cert-manager / trust-manager

trust-manager is an operator for distributing trust bundles across a Kubernetes cluster.
https://cert-manager.io/docs/projects/trust-manager/
Apache License 2.0
251 stars 66 forks source link

Empty Target field in kubectl get bundle #343

Closed qds-x closed 5 months ago

qds-x commented 5 months ago

After having upgraded to 0.9.2 from 0.5.0 we see the 'Target' field empty in the following output

➜  k get bundle
NAME                   TARGET   SYNCED   REASON   AGE
test-bundle-injected            True     Synced   4m27s

Everything functions as expected.

Tested this on a fresh minikube instance and got the same thing. I used a very simple bundle defn:

apiVersion: trust.cert-manager.io/v1alpha1
kind: Bundle
metadata:
  name: "test-bundle-injected" 
spec:
  sources:
  - inLine: |
      -----BEGIN CERTIFICATE-----
      ...
      -----END CERTIFICATE-----
  target:
    configMap:
      key: cert.pem

Happy to provide more information if required.

qds-x commented 5 months ago

I'm pretty sure this is because the field is taken from status.target.configMap.key which doesn't exist

erikgb commented 5 months ago

Thanks for reporting this! I submitted the PR to remove target from status. 😊 Will take a look!

qds-x commented 5 months ago

Thanks for reporting this! I submitted the PR to remove target from status. 😊 Will take a look!

No problemo. Obviously it doesn't really affect anything but it made us think for a second that something was off with our bundle defns. Thanks for looking at it so quickly!