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

PWX-38541 | Add GitHub Actions to Enforce Sync Between StorageCluster CRD and Helm Templates #1640

Open
wants to merge 5 commits into
base: PWX-29986
Choose a base branch
from

Conversation

hitesh-wani-px
Copy link
Collaborator

@hitesh-wani-px hitesh-wani-px commented Aug 12, 2024

What this PR does / why we need it:
This PR implements a GitHub Actions workflow to validate synchronization between changes in the StorageCluster CRD and the corresponding Helm storagecluster CRD and template.

If changes are detected in deploy/crds/core_v1_storagecluster_crd.yaml file, the its required ti include following in the PR description:

  1. Helm PR Link: If there are changes to the StorageCluster CRD, provide the Helm PR link in the following format:
    HELM_PR_LINK = <your helm PR>
  2. Configurable Field Flag: Indicate whether the new field in the StorageCluster CRD is configurable by the user and it needs to be added in storagecluster template and values.yaml
    IS_USER_CONFIGURABLE_FIELD_ADDED = YES | NO
    • Set IS_USER_CONFIGURABLE_FIELD_ADDED to YES if the field is configurable or editable by the user.
    • Set IS_USER_CONFIGURABLE_FIELD_ADDED to NO if the field is not configurable by the user.

Which issue(s) this PR fixes (optional)
Closes #https://purestorage.atlassian.net/browse/PWX-38541

Special notes for your reviewer:

…ile for user configurable field changes

Signed-off-by: hitesh-wani-px <[email protected]>
@hitesh-wani-px hitesh-wani-px force-pushed the PWX-38541 branch 12 times, most recently from fa995c2 to a24abc9 Compare August 13, 2024 19:21
Signed-off-by: hitesh-wani-px <[email protected]>
Copy link

This PR is stale because it has been in review for 3 days with no activity.

@hitesh-wani-px hitesh-wani-px changed the base branch from PWX-29986 to master August 26, 2024 12:01
@hitesh-wani-px hitesh-wani-px changed the base branch from master to PWX-29986 August 26, 2024 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant