ship icon indicating copy to clipboard operation
ship copied to clipboard

Apply Common app.kubernetes.io labels

Open dexhorthy opened this issue 5 years ago • 4 comments

Would be cool if ship could add common app labels like app.kubernetes.io/managed-by: ship, as described in https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/

Probably something we could add in at the overlays/ship level? Not sure.

dexhorthy avatar Apr 12 '19 20:04 dexhorthy

It would be nice if there was a 'generated-by' tag - I think that would fit what Ship does a bit better.

IMO 'managed-by' would be something like flux - the thing that actually delivers yaml to the cluster.

laverya avatar Apr 15 '19 17:04 laverya

Not opposed to that. I think generated-by is in line with the sort of heritage: Tiller stuff that pops up in server-side or client-side helm-produced manifests.

dexhorthy avatar Apr 15 '19 21:04 dexhorthy

@laverya I did some more looking around, and I think we should still consider including the standard managed-by tag. Kustomize team uses it in their example like so: https://github.com/pst/multi-cluster-multi-tenant-kustomize/blob/master/bases/app/kustomization.yaml#L1-L5

dexhorthy avatar Apr 17 '19 16:04 dexhorthy

I think the problem with reusing the managed-by tag is that it might get overwritten by whatever system the end user is using to deploy to their cluster. Perhaps use both?

laverya avatar May 06 '19 23:05 laverya