Skip to content

feat: Delegate -f/--values files to Helm's secrets:// getter when possible, avoiding wrapper-created decrypted files.#793

Draft
jkroepke wants to merge 4 commits into
mainfrom
passthrough
Draft

feat: Delegate -f/--values files to Helm's secrets:// getter when possible, avoiding wrapper-created decrypted files.#793
jkroepke wants to merge 4 commits into
mainfrom
passthrough

Conversation

@jkroepke

@jkroepke jkroepke commented Jun 9, 2026

Copy link
Copy Markdown
Owner

What this PR does / why we need it:

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged):

  • fixes #

Special notes for your reviewer:

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

@jkroepke jkroepke added the enhancement New feature or request label Jun 9, 2026
@codecov

codecov Bot commented Jun 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.71429% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.37%. Comparing base (19c28c6) to head (869da87).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
scripts/commands/helm.sh 85.71% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #793      +/-   ##
==========================================
- Coverage   86.46%   86.37%   -0.10%     
==========================================
  Files          22       22              
  Lines         872      888      +16     
==========================================
+ Hits          754      767      +13     
- Misses        118      121       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…en possible, avoiding wrapper-created decrypted files.

Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de>
Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de>
@jkroepke jkroepke marked this pull request as draft June 13, 2026 07:20
jkroepke added 2 commits June 13, 2026 09:24
Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de>
Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant