Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Update NSTemplateTier CRD so that it contains the Spec.Parameters and the Status.Revisions fields #444

Merged
merged 15 commits into from
Sep 17, 2024

Conversation

mfrancisc
Copy link
Contributor

@mfrancisc mfrancisc commented Sep 12, 2024

Description

This PR adds two optional fields:

  • NSTemplateTier.Spec.Parameters will contain all the parameters that will be evaluated in the TierTemplates referenced by NSTemplateTier. Those parameters will be evaluated at creation time, thus when the NSTemplateTier and TierTemplate CRs are created in the cluster.

  • NSTemplateTier.Status.Revisions is a list containing the names of the last applied TierTemplateRevisions CR and they will be used to generate the unique hash as we do now with the TierTemplate names.

For more details see NSTemplateSet creation/update scenario in the design doc

Jira: https://issues.redhat.com/browse/KUBESAW-136

Checks

  1. Did you run make generate target? yes

  2. Did make generate change anything in other projects (host-operator)? yes

  3. In case of new CRD, did you the following? N/A

  4. In case other projects are changed, please provides PR links.

Copy link
Contributor

@MatousJobanek MatousJobanek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍 Some suggestions for the comments

api/v1alpha1/nstemplatetier_types.go Outdated Show resolved Hide resolved
api/v1alpha1/nstemplatetier_types.go Outdated Show resolved Hide resolved
api/v1alpha1/nstemplatetier_types.go Show resolved Hide resolved
api/v1alpha1/nstemplatetier_types.go Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Sep 17, 2024

@mfrancisc mfrancisc merged commit 978fc39 into codeready-toolchain:master Sep 17, 2024
3 checks passed
@mfrancisc mfrancisc deleted the nstemplatetierupdate branch September 17, 2024 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants