deprecate additionalMetadata

[oliverbaehler]

Describe the feature

With the merge of #1339 we have a more powerful option to assign metadata to namespaces. I would like to preserve this approach for the stable spec but encourage users to no longer use the additionalMetadata block on it's own. It should be deprecated and removed

[rxinui]

Hi @oliverbaehler, If I understand correctly additionalMetedata should be deprecated for all version prior v1beta2 but what about the "to be removed" part?

[Svarrogh1337]

Hi @oliverbaehler, If I understand correctly additionalMetedata should be deprecated for all version prior v1beta2 but what about the "to be removed" part?

If I may chime in, I’m not entirely convinced we should remove it outright, as it would introduce a breaking change.
What I’m thinking instead is using a conversion webhook to convert additionalMetadata to additionalMetadataList.
@oliverbaehler WDYT?

[rxinui]

@Svarrogh1337 following that direction, deprecating it remains relevant (no removal). Perhaps, on top of it a webhook could do, although that can be a lot of effort for a temporary use case IMO?

[oliverbaehler]

It's planned for v1, a breaking change API.

[github-actions]

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days.