Skip to content

feat: Support caching provisioner assets#574

Merged
kylecarbs merged 5 commits into
mainfrom
plugincache
Mar 28, 2022
Merged

feat: Support caching provisioner assets#574
kylecarbs merged 5 commits into
mainfrom
plugincache

Conversation

@kylecarbs

Copy link
Copy Markdown
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

codecov Bot commented Mar 26, 2022

Copy link
Copy Markdown

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
Copy Markdown

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

@kylecarbs

Copy link
Copy Markdown
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.

4 participants