Description of problem: oc describe on several machineconfig apis displays labels names with corrupting spacing and capitalization. For example (from `oc describe machineconfigpool worker`): >>>>> Spec: Machine Config Selector: Match Labels: Machineconfiguration . Openshift . Io / Role: worker Machine Selector: Match Labels: Node - Role . Kubernetes . Io / Worker: <<<<< Version-Release number of selected component (if applicable): 4.0.0-0.9 How reproducible: 100% Steps to Reproduce: 1. oc describe machineconfigpool worker 2. 3. Actual results: Label names corrupted with spacing and capitalization.. Machineconfiguration . Openshift . Io / Role: worker Expected results: machineconfiguration.openshift.io/role: worker Additional details: Other resources are also affected (e.g. machinesets)
Spun up a cluster and can confirm that the output is the same as described.
Figured out the problem: So this is related to kubectl's describe : https://github.com/kubernetes/kubernetes/blob/fd98e7abf042251bb094a5535d67f53d36021246/pkg/kubectl/describe/versioned/describe.go#L306 The smartLabelFor() function is what is adding the capitalization and the spaces. However ~2 weeks ago, a PR was merged into the upstream that fixes this issue: https://github.com/kubernetes/kubernetes/pull/75483 To fix this we'd have to update our oc which is using 1.13 currently. I'm going to ask about this in #forum-devex and will update to the resolution.
According to the original issue filing upstream, the fix will be in v1.15: https://github.com/kubernetes-sigs/cluster-api-provider-aws/issues/260#issuecomment-478585501
Update: as of right now 1.15 (containing this fix) would be pulled into 4.3, so we'd get it in the fall. Right now master team is dealing with blockers but might be able to get it in a bit sooner, but we'd have to follow up later.
Opened #1698235 to track this in master since they are in charge of this part of `oc` This BZ can be closed.
See Bug 1698235 (hoping this creates a link)
Closing as this is not MCO specific but a fix should land in oc itself.