Images only referenced in preflight check collectors are not included in airgap packages

[laverya]

apiVersion: troubleshoot.replicated.com/v1beta1
kind: Preflight
metadata:
  name: airgap-preflight-checks
spec:
  analyzers:
    - textAnalyze:
        checkName: a name
        fileName: ping.txt/run-ping.log
        regexGroups: '(?P<Transmitted>\d+) packets? transmitted, (?P<Received>\d+) packets? received, (?P<Loss>\d+\.\d+)% packet loss'
        outcomes:
          - pass:
              when: "Loss < 5.0"
              message: google.com resolves correctly
          - fail:
              message: High packet loss  
  collectors:
    - run:
        collectorName: "run-ping"
        image: busybox:1
        name: ping.txt
        namespace: default
        command: ["ping"]
        args: ["-w", "5", 'google.com']
        imagePullPolicy: IfNotPresent

the 'busybox:1' image is not pulled, causing the preflight check to spin forever.

[laverya]

Also, we don't rewrite the preflight check images to refer to local registries, etc

[ndrafahl]

Hey @laverya or @divolgin not to revive an older issue, but was there ever a fix or workaround in place for this?

[cbodonnell]

@ndrafahl can you confirm that this is still an issue, and if so, provide additional reproduction steps? Currently, an airgap bundle built for an application with the above preflight spec includes the busybox:1 image and is rewritten to the local registry.