Skip to content

Fix: Saved plan apply hangs with -auto-approve flag using cloud backend#36453

Merged
Maed223 merged 4 commits intomainfrom
IPL-7602/saved-plan-apply-hangs-with-auto-approve-flag
Feb 10, 2025
Merged

Fix: Saved plan apply hangs with -auto-approve flag using cloud backend#36453
Maed223 merged 4 commits intomainfrom
IPL-7602/saved-plan-apply-hangs-with-auto-approve-flag

Conversation

@Maed223
Copy link
Copy Markdown
Contributor

@Maed223 Maed223 commented Feb 6, 2025

Description

When applying a saved plan with the -auto-approve flag using the cloud backend, we would see the apply hang in the console. The only way to proceed was to confirm the run using the UI. Without the flag, the apply runs without any user confirmation as is the intended behavior. This bug fix avoids this hanging behavior by kicking off the apply regardless of whether automatic approval is given if a saved plan is being used– as is the behavior locally.

Target Release

1.11.x

CHANGELOG entry

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

@Maed223 Maed223 marked this pull request as ready for review February 6, 2025 21:03
@Maed223 Maed223 requested a review from a team as a code owner February 6, 2025 21:03
sebasslash
sebasslash previously approved these changes Feb 6, 2025
Copy link
Copy Markdown
Contributor

@sebasslash sebasslash left a comment

Choose a reason for hiding this comment

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

Smoke tested

Copy link
Copy Markdown
Contributor

@liamcervante liamcervante left a comment

Choose a reason for hiding this comment

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

If you want to release this in v1.11, you need to add the relevant backport label and move the changelog description into .changes/backported.

@Maed223 Maed223 added the 1.11-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged label Feb 7, 2025
@Maed223 Maed223 merged commit 1664348 into main Feb 10, 2025
@Maed223 Maed223 deleted the IPL-7602/saved-plan-apply-hangs-with-auto-approve-flag branch February 10, 2025 16:01
dbanck pushed a commit that referenced this pull request Feb 19, 2025
…ngs-with-auto-approve-flag

Fix: Saved plan apply hangs with `-auto-approve` flag using cloud backend
dbanck added a commit that referenced this pull request Feb 19, 2025
…ngs-with-auto-approve-flag (#36532)

Fix: Saved plan apply hangs with `-auto-approve` flag using cloud backend

Co-authored-by: Mark DeCrane <mark.decrane@hashicorp.com>
@github-actions
Copy link
Copy Markdown
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

1.11-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants