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

Fix - Initial version of hmc-templates release gets updated concurrently #515

Closed
wants to merge 1 commit into from

Conversation

kylewuolle
Copy link
Contributor

Simple change to resolve #449.

Testing performed:

  1. Updated hmc_values.yaml to set the createRelease flag to true

  2. Executed make hmc-chart-release

  3. Ran make kind-deploy registry-deploy dev-push dev-deploy dev-templates (make dev-apply without the dev-release target)

  4. Verified the release was created correctly

  5. Updated hmc_values.yml to set the createRelease flag to false

  6. Ran make kind-deploy registry-deploy dev-push dev-deploy dev-templates (make dev-apply without the dev-release target)

  7. Verified the release was not created and there was no loop attempting to create it

  8. ran make dev-release and verified the release was installed correctly

Also ran make dev-apply with the createRelease flag set to false (clean version of hmc_values.yaml) and verified this works as well.

…o it is passed in along with the helm values
@squizzi
Copy link

squizzi commented Oct 17, 2024

Rebase to pick up the e2e controller test, it might need to be fixed up for this, it may even be useful to add a test case to the controller test.

@@ -252,7 +252,7 @@ func (r *ReleaseReconciler) reconcileHMCTemplates(ctx context.Context, releaseNa
},
}

if releaseName == "" {
if releaseName == "" || r.CreateRelease {
Copy link
Collaborator

Choose a reason for hiding this comment

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

we only need Release to be created as part of the hcm-templates chart during the initial hmc installation. We should probably rename it to avoid confusion (along with create-management, create-template-management) (although it's mentioned in the variable description)

@kylewuolle
Copy link
Contributor Author

Looks like this was fixed in another PR

@kylewuolle kylewuolle closed this Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Initial version of hmc-templates release gets updated concurrently
4 participants