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(liquibase): fix checkSum errors occurring with spinnaker upgrade (backport #6217) #6232

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Jun 11, 2024

This PR fixes checkSum errors caused after liquibase upgrade(to 4.24.0) as the checksum calculation underwent some changes causing this issue.

This PR fixes the checksum errors by adding validChecksum tags to the failing changesets.

Added an integration test to verify the migration scenario with postgres. This will be useful to identify and avoid issues when liquibase is upgraded or if any changesets are modified.

Similar orca PR for reference: spinnaker/orca#4727


This is an automatic backport of pull request #6217 done by [Mergify](https://mergify.com).

…6217)

* fix(tests): added postgres migration test

* fix(tests): hardcoded the docker image to create initial conditions

* fix(liquibase): add validCheckSum values to fix failing changesets

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit ad1a8ef)
@dbyron-sf dbyron-sf added the ready to merge Approved and ready for a merge label Jun 11, 2024
@mergify mergify bot added the auto merged Merged automatically by a bot label Jun 11, 2024
@mergify mergify bot merged commit 4d1782e into release-1.34.x Jun 11, 2024
24 checks passed
@mergify mergify bot deleted the mergify/bp/release-1.34.x/pr-6217 branch June 11, 2024 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged Merged automatically by a bot ready to merge Approved and ready for a merge target-release/1.34
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants