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

feat: Support caching provisioner assets #574

Merged
merged 5 commits into from
Mar 28, 2022
Merged

feat: Support caching provisioner assets #574

merged 5 commits into from
Mar 28, 2022

Conversation

kylecarbs
Copy link
Member

This caches the Terraform binary, and Terraform plugins.
Eventually, it could cache other temporary files.

This allows zero-trust authentication for all AWS instances.

Prior to this, AWS instances could be used by passing `CODER_TOKEN`
as an environment variable to the startup script. AWS explicitly
states that secrets should not be passed in startup scripts because
it's user-readable.
@kylecarbs kylecarbs requested a review from coadler March 26, 2022 05:04
@kylecarbs kylecarbs self-assigned this Mar 26, 2022
This caches the Terraform binary, and Terraform plugins.
Eventually, it could cache other temporary files.
@codecov
Copy link

codecov bot commented Mar 26, 2022

Codecov Report

Merging #574 (04d9fe3) into main (a502a5f) will decrease coverage by 0.20%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##             main     #574      +/-   ##
==========================================
- Coverage   63.87%   63.67%   -0.21%     
==========================================
  Files         197      197              
  Lines       11524    11537      +13     
  Branches       85       85              
==========================================
- Hits         7361     7346      -15     
- Misses       3389     3411      +22     
- Partials      774      780       +6     
Flag Coverage Δ
unittest-go- 62.72% <44.44%> (-0.20%) ⬇️
unittest-go-macos-latest 58.43% <44.44%> (-0.14%) ⬇️
unittest-go-ubuntu-latest 61.31% <44.44%> (-0.14%) ⬇️
unittest-go-windows-2022 57.65% <44.44%> (-0.04%) ⬇️
unittest-js 63.32% <ø> (ø)
Impacted Files Coverage Δ
provisioner/terraform/provision.go 72.29% <0.00%> (-1.37%) ⬇️
provisioner/terraform/serve.go 50.00% <25.00%> (ø)
cli/start.go 64.17% <100.00%> (+0.37%) ⬆️
provisionersdk/transport.go 74.46% <0.00%> (-6.39%) ⬇️
provisioner/echo/serve.go 54.40% <0.00%> (-2.40%) ⬇️
peer/conn.go 76.90% <0.00%> (-2.29%) ⬇️
coderd/provisionerdaemons.go 62.75% <0.00%> (-1.01%) ⬇️
provisionerd/provisionerd.go 80.76% <0.00%> (-0.45%) ⬇️
peerbroker/listen.go 87.28% <0.00%> (+3.38%) ⬆️

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 a502a5f...04d9fe3. Read the comment docs.

@misskniss
Copy link

@kylecarbs can this be closed or does it still need to be done?

@kylecarbs
Copy link
Member Author

Erm there was no ticket for this @misskniss, so I'm not sure what you mean.

Base automatically changed from awsauth to main March 28, 2022 19:31
@kylecarbs kylecarbs merged commit 13cef7d into main Mar 28, 2022
@kylecarbs kylecarbs deleted the plugincache branch March 28, 2022 19:57
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.

None yet

4 participants