Skip to content

fix: Use cliui.WorkspaceBuild to prevent cancel of builds jobs #1255

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

Merged
merged 1 commit into from
May 2, 2022

Conversation

kylecarbs
Copy link
Member

Build jobs cannot gracefully terminate because Terraform generally
cannot gracefully terminate.

Build jobs cannot gracefully terminate because Terraform generally
cannot gracefully terminate.
@kylecarbs kylecarbs requested a review from deansheather May 2, 2022 13:33
@kylecarbs kylecarbs self-assigned this May 2, 2022
@codecov
Copy link

codecov bot commented May 2, 2022

Codecov Report

Merging #1255 (96b2539) into main (4ff5734) will increase coverage by 0.13%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main    #1255      +/-   ##
==========================================
+ Coverage   65.93%   66.06%   +0.13%     
==========================================
  Files         272      275       +3     
  Lines       17892    17937      +45     
  Branches      192      214      +22     
==========================================
+ Hits        11797    11850      +53     
+ Misses       4871     4855      -16     
- Partials     1224     1232       +8     
Flag Coverage Δ
unittest-go-macos-latest 53.37% <33.33%> (-0.06%) ⬇️
unittest-go-postgres- 64.96% <33.33%> (+0.12%) ⬆️
unittest-go-ubuntu-latest 55.65% <33.33%> (+0.08%) ⬆️
unittest-go-windows-2022 52.97% <0.00%> (+0.13%) ⬆️
unittest-js 70.60% <ø> (+0.63%) ⬆️
Impacted Files Coverage Δ
cli/workspacestart.go 19.23% <0.00%> (+5.71%) ⬆️
cli/workspacestop.go 19.23% <0.00%> (+6.73%) ⬆️
cli/server.go 58.79% <100.00%> (-0.21%) ⬇️
provisionersdk/serve.go 35.13% <0.00%> (-8.11%) ⬇️
provisionerd/provisionerd.go 75.76% <0.00%> (-1.61%) ⬇️
peer/channel.go 84.97% <0.00%> (-1.16%) ⬇️
site/src/theme/palettes.ts 100.00% <0.00%> (ø)
site/src/components/Icons/SearchIcon.tsx 100.00% <0.00%> (ø)
...ite/src/components/ConfirmDialog/ConfirmDialog.tsx 94.11% <0.00%> (ø)
site/src/components/Dialog/Dialog.tsx 74.41% <0.00%> (ø)
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4ff5734...96b2539. Read the comment docs.

Copy link
Member

@deansheather deansheather left a comment

Choose a reason for hiding this comment

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

You need to write a ticket to support build cancellation in the future. We can probably support it by exiting terraform, then calling destroy on each ephemeral resource manually.

@kylecarbs
Copy link
Member Author

We do support build cancellation, but it's better for infrastructure folks to rely on Terraform timeouts instead. I'm going to be making significant changes here this week.

@kylecarbs kylecarbs merged commit a79aa64 into main May 2, 2022
@kylecarbs kylecarbs deleted the fixcancel branch May 2, 2022 15:20
@misskniss misskniss added this to the V2 Beta milestone May 15, 2022
kylecarbs added a commit that referenced this pull request Jun 10, 2022
Build jobs cannot gracefully terminate because Terraform generally
cannot gracefully terminate.
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