feat(crd-generator): Deprecation warning for CRD versions

[baloo42]

Description

Add support for deprecation warnings in CRDGenerator.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• Feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change
• Chore (non-breaking change which doesn't affect codebase;
  test, version modification, documentation, etc.)

Checklist

• Code contributed by me aligns with current project license: Apache 2.0
• I Added CHANGELOG entry regarding this change
• I have implemented unit tests to cover my changes
• I have added/updated the javadocs and other documentation accordingly
• No new bugs, code smells, etc. in SonarCloud report
• I tested my code in Kubernetes
• I tested my code in OpenShift

[sonarqubecloud]

Quality Gate failed

Failed conditions
51.4% Coverage on New Code (required ≥ 80%)
12.8% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

[andreaTP]

LGTM, thanks for improving this aspect!

[manusa]

LGTM, thx! ❤️

[baloo42]

Thanks for merging and sorry for the missing changelog entry...

One follow up question:
At the moment the deprecated flag and the deprecationWarning are only added to the generated manifests if they are required (if deprecated=true).
Do you think it's a good strategy to keep the resulting manifests as small as possible or do you prefer to add the deprecated attribute explicitly (like served or storage)?

[manusa]

Thanks for merging and sorry for the missing changelog entry...

No worries

One follow up question: At the moment the deprecated flag and the deprecationWarning are only added to the generated manifests if they are required (if deprecated=true). Do you think it's a good strategy to keep the resulting manifests as small as possible or do you prefer to add the deprecated attribute explicitly (like served or storage)?

@andreaTP @metacosm ?

[andreaTP]

I think that the current strategy is aligned with the other fields, thanks for asking!