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

Lbjenn patch 1 #2

Merged
merged 74 commits into from
Aug 12, 2024
Merged

Lbjenn patch 1 #2

merged 74 commits into from
Aug 12, 2024

Conversation

lbjenn
Copy link
Collaborator

@lbjenn lbjenn commented Aug 3, 2024

Made several changes so that workflows would run successfully and gather report artifacts. run_changes and run_targeted_rollback are using Liquibase 4.28.0 explicitly instead of latest.

lbjenn and others added 30 commits August 1, 2024 13:27
updated log level for trouble shooting
… lbjenn-patch-1

changes in remote are for GHA workflows only and local change is only for QC config file version
changed the actions/upload-artifact calls for DEV & TEST to use v4
changed docker pull to get latest LB version
updated checkout calls to actions/checkout@v4
Updated snapshot files to be 4.28.0 and docker to use 4.28.0
Removed grep command to show diff as we now have drift report
Added parameters to rollback command to product a rollback report
added in reports enabled and env name for report file
Changed the log artifact name for the zip file to include the env label so update-artifacts@v4 does not fail saying there is an artifact with this name in this run
UPdated the actions/checkout github function to v4
Updated actions/checkout to v4
Added artifact upload step in each env to capture the rollback report
Updated actions/checkout to v4
Added rollback stmt back in
Copy link
Owner

@molivasdat molivasdat left a comment

Choose a reason for hiding this comment

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

LGTM

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