Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: configurable cache cleanup via env (CODER_PROVISIONER_CACHE_*)#20051

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
angrycub wants to merge2 commits intomain
base:main
Choose a base branch
Loading
fromblink/configurable-terraform-cache-cleanup

Conversation

angrycub
Copy link
Contributor

@angrycubangrycub commentedSep 30, 2025
edited
Loading

This change makes Terraform plugin cache cleanup user-configurable with minimal surface area:

  • Replace hard-coded 30d retention with a configurable variable
  • Read environment in provisioner Serve():
    • CODER_PROVISIONER_CACHE_CLEANUP (default true)
    • CODER_PROVISIONER_CACHE_RETENTION (e.g. 720h)
  • Guard the cleanup call in Plan() with the toggle
  • Update docs (best practices) to describe the new env vars

Behavior:

  • Defaults remain unchanged (cleanup enabled; ~30d retention)
  • Works for both built-in and external provisioners (env read in Serve)

Notes:

  • Tests in provisioner/terraform require a Terraform binary; build succeeds locally. Happy to add CI setup or adjust tests if needed.

…eplace hard-coded 30d retention with configurable var\n- Read env in provisioner Serve():\n - CODER_TERRAFORM_PLUGIN_CLEANUP (default true)\n - CODER_TERRAFORM_PLUGIN_RETENTION (e.g. 720h)\n- Guard cleanup call with toggle\n- Update docs: speed-up-templates to describe settings\n\nCo-authored-by: angrycub <464492+angrycub@users.noreply.github.com>
@angrycubangrycub changed the titleterraform: configurable plugin cache cleanup (env-controlled)feature: configurable plugin cache cleanup (env-controlled)Oct 1, 2025
@angrycubangrycub changed the titlefeature: configurable plugin cache cleanup (env-controlled)feature: make plugin cache cleanup configurable (env-controlled)Oct 1, 2025
@angrycubangrycub changed the titlefeature: make plugin cache cleanup configurable (env-controlled)feat: make plugin cache cleanup configurable (env-controlled)Oct 1, 2025
…PROVISIONER_CACHE_CLEANUP\n- CODER_PROVISIONER_CACHE_RETENTION\n\nUpdate docs accordingly.\n\nCo-authored-by: angrycub <464492+angrycub@users.noreply.github.com>
@angrycubangrycub changed the titlefeat: make plugin cache cleanup configurable (env-controlled)provisioner: configurable cache cleanup via env (CODER_PROVISIONER_CACHE_*)Oct 1, 2025
@angrycubangrycub changed the titleprovisioner: configurable cache cleanup via env (CODER_PROVISIONER_CACHE_*)feat: configurable cache cleanup via env (CODER_PROVISIONER_CACHE_*)Oct 1, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees

@angrycubangrycub

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

1 participant
@angrycub

[8]ページ先頭

©2009-2025 Movatter.jp