milvus-helm icon indicating copy to clipboard operation
milvus-helm copied to clipboard

feat:object storage aksk uses k8s secret storage

Open doubledna opened this issue 2 months ago • 3 comments

What this PR does / why we need it: milvus connect external s3 aksk uses k8s secret storage. / s3 aksk stored in helm value is prone to leakage.

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • [x ] Chart Version bumped
  • [ ] Variables are documented in the README.md
  • [ ] Title of the PR starts with chart name (e.g. [mychartname])
  • [ ] PR only contains changes for one chart

doubledna avatar Oct 10 '25 08:10 doubledna

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: doubledna To complete the pull request process, please assign loveeachday after the PR has been reviewed. You can assign the PR to them by writing /assign @loveeachday in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

sre-ci-robot avatar Oct 10 '25 08:10 sre-ci-robot

Welcome @doubledna! It looks like this is your first PR to zilliztech/milvus-helm 🎉

sre-ci-robot avatar Oct 10 '25 08:10 sre-ci-robot

bump! it's really needed

mateusz-mielewczyk-it avatar Oct 31 '25 14:10 mateusz-mielewczyk-it