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: Preserve state on rollback #2131

Merged
merged 6 commits into from
Nov 4, 2024
Merged

Conversation

jachym-tousek-keboola
Copy link
Contributor

Jira: https://keboola.atlassian.net/browse/PSGO-918

Changes:

  • Preserve enabled/disabled/deleted/undeleted on rollback

@jachym-tousek-keboola jachym-tousek-keboola changed the title Jt psgo 918 rollback fix: Preserve state on rollback Oct 31, 2024
@jachym-tousek-keboola jachym-tousek-keboola changed the base branch from main to jt-psgo-916-undelete October 31, 2024 14:58
@jachym-tousek-keboola jachym-tousek-keboola force-pushed the jt-psgo-918-rollback branch 2 times, most recently from a561c2d to b384483 Compare November 1, 2024 11:53
Copy link
Contributor Author

@jachym-tousek-keboola jachym-tousek-keboola Nov 1, 2024

Choose a reason for hiding this comment

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

Unrelated test fix. This e2e test was randomly failing.

test: https://github.com/keboola/keboola-as-code/tree/992fef0c1c99d4677c4c1ae945653f176e7d6f65/test/templates/api/projects/instance-delete
failure: https://github.com/keboola/keboola-as-code/actions/runs/11629666014/job/32387060642?pr=2131

After checking it I noticed that there are two directories prefixed with 005. The list-instances should be 008 so that it's executed after both polls are finished.

Copy link
Contributor

@Matovidlo Matovidlo left a comment

Choose a reason for hiding this comment

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

LGTM. Won't block it now, i'll think about code duplication

Base automatically changed from jt-psgo-916-undelete to main November 4, 2024 11:00
@jachym-tousek-keboola jachym-tousek-keboola merged commit d25244c into main Nov 4, 2024
10 checks passed
@jachym-tousek-keboola jachym-tousek-keboola deleted the jt-psgo-918-rollback branch November 4, 2024 11:00
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.

3 participants