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

.helm: also delete hook resources before-hook-creation #3930

Merged
merged 1 commit into from
Oct 14, 2023

Conversation

BacLuc
Copy link
Contributor

@BacLuc BacLuc commented Oct 7, 2023

That you don't have to manually delete the resources before you retry the deployment with the updated configuration. Worked very good for the hook_db_restore resources.

Else the deployment fails with the message:
UPGRADE FAILED: post-upgrade hooks failed: warning: Hook post-upgrade ecamp3/templates/hook_db_migrate.yaml failed: 1 error occurred:
* jobs.batch "ecamp3-dev-db-migrate" already exists

That you don't have to manually delete the resources before
you retry the deployment with the updated configuration.
Worked very good for the hook_db_restore resources.

Else the deployment fails with the message:
 UPGRADE FAILED: post-upgrade hooks failed: warning: Hook post-upgrade ecamp3/templates/hook_db_migrate.yaml failed: 1 error occurred:
	* jobs.batch "ecamp3-dev-db-migrate" already exists
@usu usu added this pull request to the merge queue Oct 14, 2023
Merged via the queue into ecamp:devel with commit 12a8083 Oct 14, 2023
28 checks passed
@BacLuc BacLuc deleted the helm-improve-delete-policy-for-hooks branch October 14, 2023 09:51
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.

None yet

3 participants