Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Backport of "Fix: Saved plan apply hangs with -auto-approve flag using cloud backend" #36532

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

dbanck
Copy link
Member

@dbanck dbanck commented Feb 19, 2025

This PR does a manual backport of #36453 since the backport assistant failed

It seems a bit unusual that 1664348 is a merge commit, and I hope that backporting it will not cause any problems. I picked 857d188 as the parent during cherry picking

Target Release

1.11.x

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

…ngs-with-auto-approve-flag

Fix: Saved plan apply hangs with `-auto-approve` flag using cloud backend
@dbanck dbanck requested a review from a team as a code owner February 19, 2025 10:50
@@ -0,0 +1,5 @@
kind: BUG FIXES
body: Fixes hanging behavior seen when applying a saved plan with -auto-approve using the cloud backend
Copy link
Member

Choose a reason for hiding this comment

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

Although there are no special characters, I am somewhat surprised this isn't quoted 🤔 but it appears that changie dry run check is happy so no biggie 🤷🏻

Copy link
Member Author

Choose a reason for hiding this comment

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

YAML 🤷 😂

Thanks for checking

@dbanck dbanck merged commit 1996c85 into v1.11 Feb 19, 2025
7 checks passed
@dbanck dbanck deleted the dbanck/backport-apply-fix branch February 19, 2025 10:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants