Improvement for k8s.io/docs/concepts/overview/working-with-objects/common-labels/ #46007
Labels
good first issue
Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.
help wanted
Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.
language/en
Issues or PRs related to English language
triage/accepted
Indicates an issue or PR is ready to be actively worked on.
There is a suggested annotation of:
But Helm actually recommends:
Which will insert a value of "Helm", with a first character in uppercase.
According to this research, labels are case-sensitive, except for the prefix.
(You can notice this discrepancy if you are using the
helm diff
plug-in, as your chart with a lower-case word will always be replaced with upper-case, so a three-way-merge diff shows this as changing every time. I also imagine you could find this if you tried to use the label as a selector)Therefore I recommend that the suggested annotation in this document be replaced with:
Caveat: I have no idea if the case changed between helm v2 and v3, but v2 is defunct now, so should be safe to make this change.
The text was updated successfully, but these errors were encountered: