additionalTags with whitespace in the key

[mimmus]

/kind bug

Unfortunately, I have a company's mandatory tag with a space in the key:

apiVersion: infrastructure.cluster.x-k8s.io/v1beta1
kind: AWSCluster
...
spec:
  additionalTags:
    "Patch Group": nopatch
...

but I'm getting:
The AWSCluster "mycluster-dr" is invalid: spec.additionalTags: Invalid value: "Patch Group": key cannot have characters other than alphabets, numbers, spaces and _ . : / = + - @

• Cluster-api-provider-aws version:
  1.5.1

[richardcase]

As the error message says we should be allowing space, so this needs fixing.

/triage accepted
/priority important-soon
/help
/good-first-issue

[k8s-ci-robot]

@richardcase:
This request has been marked as suitable for new contributors.

Guidelines

Please ensure that the issue body includes answers to the following questions:

• Why are we solving this issue?
• To address this issue, are there any code changes? If there are code changes, what needs to be done in the code and what places can the assignee treat as reference points?
• Does this issue have zero to low barrier of entry?
• How can the assignee reach out to you for help?

For more details on the requirements of such an issue, please see here and ensure that they are met.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.

Details

In response to this:

As the error message says we should be allowing space, so this needs fixing.

/triage accepted
/priority important-soon
/help
/good-first-issue

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[richardcase]

For anyone picking this issue up, this is where we do the validation:

cluster-api-provider-aws/api/v1beta2/tags.go

Line 77 in 14ca4d8

func (t Tags) Validate() []*field.Error {

[Msesto]

Hi @richardcase
I'm new contributing to k8s and would love to take on this bug (I'm not familiar with the time expectation of the tags, but I believe I can have a PR before next week)
I've been going through the dev guide, but wanted to ask you if I should go with tilt (never used it before) over setting the local environment? Also there is room for improvement on the developer guide, if that's an acceptable issue to create for a docs pr I will take on it after this one.
Thanks!

[Msesto]

Alright, I managed to get my local environment running, and digging into the issue, it was fixed with #3702
Not entirely sure how you guys manage the versioning nor when something stops beings maintained, I've yet to dig deep into the docs..

[mimmus]

Great! An extra "" in regex!

Only another question:

• is this bug a regression or was it always present?

I'm not so good to look into Github history.