Add a new config to override kubebuilder printcolumn:name

[a-hilaly]

Issue N/A

In some CRDs like GlobalTable adding a is_printable: true to the GlobalTableStatus causes kubectl get to print very large/redundant column names.

Description of changes:

• This patch adds a new config to allow overriding kubebuilder printcolumn:name
• Adds metav1.Time to the acceptableColumnMap

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[RedbackThomson]

Hmm I wish I had the foresight to add this initially, because I'd probably have made the config a struct containing the boolean and the string - rather than them both being on the parent resource level. Regardless, without breaking backwards compatibility, this is good.

/approve

[a-hilaly]

Hmm I wish I had the foresight to add this initially, because I'd probably have made the config a struct containing the boolean and the string - rather than them both being on the parent resource level. Regardless, without breaking backwards compatibility, this is good.

Agreed. I hesitated to introduce a struct config for it.. but as you said it will break the compatibility .. maybe one day a generator config v2 :)

[jaypipes]

Nice, Amine. :)

[jaypipes]

/lgtm

[ack-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: A-Hilaly, jaypipes, RedbackThomson

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [A-Hilaly,RedbackThomson,jaypipes]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment