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 edge cases in PostgreSQL maintenance #43

Merged
merged 1 commit into from
Jul 19, 2023
Merged

Fix edge cases in PostgreSQL maintenance #43

merged 1 commit into from
Jul 19, 2023

Conversation

glrf
Copy link
Contributor

@glrf glrf commented Jul 19, 2023

This commit fixes two separate issue with the PostreSQL maintenance script. Both are related to waiting for the upgrade to complete.

  • Fixes a panic in case we receive an event with an SGDbOps without any conditions
  • Fixes an issue where the script will run repack prematurely, in case there is an event on a different, completed, SGDbOps in the namespace.

The second point is fairly theoretical, but the second one can be reproduced locally, even though it does not seem to occur in production. Probably a timing issue.

Checklist

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • Update tests.

This commit fixes two separate issue with the PostreSQL maintenance
script. Both are related to waiting for the upgrade to complete.

* Fixes a panic in case we receive an event with an SGDbOps without any
  conditions
* Fixes an issue where the script will run repack prematurely, in case
  there is an event on a different, completed, SGDbOps in the namespace.
@glrf glrf added the bug Something isn't working label Jul 19, 2023
@glrf glrf self-assigned this Jul 19, 2023
@glrf glrf marked this pull request as ready for review July 19, 2023 12:08
@glrf glrf requested a review from zugao July 19, 2023 12:08
@zugao zugao self-requested a review July 19, 2023 12:13
Copy link
Collaborator

@zugao zugao left a comment

Choose a reason for hiding this comment

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

LGTM

@glrf glrf merged commit bdf54f9 into master Jul 19, 2023
3 checks passed
@glrf glrf deleted the fix/pg-main-panic branch July 19, 2023 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants