Skip to content

fix: Allow terraform provisions to be gracefully cancelled #3526

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.

Sign up for GitHub

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

Merged
merged 14 commits into from
Aug 18, 2022
Merged

fix: Allow terraform provisions to be gracefully cancelled #3526

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
fd47d05
fix: Allow terraform provisions to be gracefully cancelled
mafredri Aug 16, 2022
72f723e
Cleanup
mafredri Aug 16, 2022
00c7746
Revert protoc version bumps
mafredri Aug 16, 2022
7df45e1
fix: Remove unused import from Windows
mafredri Aug 16, 2022
211b7f9
fix: Add build tags
mafredri Aug 16, 2022
3121f6a
fix: Amend PR comments
mafredri Aug 16, 2022
27fe6f0
chore: Revert proto changes
mafredri Aug 16, 2022
1d7f825
chore: Revert newline in proto
mafredri Aug 16, 2022
5cbb712
fix: Remove incorrect minimumTerraformVersion variable
mafredri Aug 17, 2022
3e4005a
fix: Add ExitTimeout to terraform.ServeOptions
mafredri Aug 17, 2022
685c583
fix: Add test for interrupt/cancellation
mafredri Aug 17, 2022
3303ee1
Allow init to return provision complete response
mafredri Aug 17, 2022
db41a65
Add testdata script
mafredri Aug 17, 2022
80a4912
Remove superfluous require
mafredri Aug 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Remove superfluous require
  • Loading branch information
@mafredri
mafredri committed Aug 17, 2022
commit 80a4912e9e9426a62929a870daa2d6fc573d4377
1 change: 0 additions & 1 deletion provisioner/terraform/provision_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,6 @@ func TestProvision_Cancel(t *testing.T) {
if log == nil {
goto LoopStart
}
require.NotNil(t, log)
require.Equal(t, line, log.Output)
}

Expand Down