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 _db_setup_job.tpl #162

Merged
merged 3 commits into from
Apr 10, 2024

Conversation

andrzejgrzelak
Copy link

Fix an issue with db_setup_job that arises when script fails on any of the steps after database creation and then get stuck on "database exists" step but fails to connect using saved credentials.

Now script checks both if database exists and if credentials are properly set, if any of those things is not right then it proceeds to database setup step.

Link to JIRA ticket if there is one:

New Features

Breaking Changes

Bug Fixes

Fixed a bug in which _db_setup_job gets stuck in a permanent error unless database of a related service is removed

Improvements

Dependency updates

Deployment changes

andrzejgrzelak and others added 3 commits March 21, 2024 17:09
Fix an issue with db_setup_job that arises when script fails on any of the steps after database creation and get stuck on "database exists" step but fail to connect using saved credentials.
Fix a left "fi' after a previous fix
@jawadqur jawadqur changed the base branch from master to feat/db_setup_fix April 10, 2024 20:57
@jawadqur jawadqur merged commit 8019057 into uc-cdis:feat/db_setup_fix Apr 10, 2024
1 of 2 checks passed
@jawadqur
Copy link
Collaborator

Merging this PR to a local branch so we can do some more testing, but this looks like a good change, thanks @andrzejgrzelak

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.

2 participants