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.

Sign up for GitHub

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

Jump to bottom

kctrl: Allow users to drop a values secret being consumed by a package install #675

Closed
4 tasks
100mik opened this issue May 10, 2022 · 1 comment · Fixed by #683
Closed
4 tasks

kctrl: Allow users to drop a values secret being consumed by a package install #675

100mik opened this issue May 10, 2022 · 1 comment · Fixed by #683
Assignees
@100mik
Labels
carvel-accepted This issue should be considered for future work and that the triage process has been completed cli Issue for kapp-controller cli enhancement This issue is a feature request

Comments

@100mik
Copy link
Contributor

100mik commented May 10, 2022

Describe the problem/challenge you have
Allow users to dereference and garbage collect a secret suopplied to a package install craeted by kctrl

Describe the solution you'd like
A flag --no-values-file which deletes and dereferences value secrets created by kctrl

Acceptance criteria

  • Add flag to drop values if the user does not supply a values file
  • Dereference
  • Assert ownership before deletion
  • Tests

Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.
@100mik 100mik added enhancement This issue is a feature request carvel-triage This issue has not yet been reviewed for validity labels May 10, 2022
@100mik 100mik mentioned this issue May 10, 2022
Closed
62 tasks
@100mik 100mik added carvel-accepted This issue should be considered for future work and that the triage process has been completed cli Issue for kapp-controller cli priority/unprioritized-backlog Higher priority than priority/awaiting-more-evidence but not planned. Contributions are welcome. and removed carvel-triage This issue has not yet been reviewed for validity labels May 10, 2022
@100mik 100mik self-assigned this May 10, 2022
@100mik 100mik removed the priority/unprioritized-backlog Higher priority than priority/awaiting-more-evidence but not planned. Contributions are welcome. label May 10, 2022
@100mik
Copy link
Contributor Author

100mik commented May 10, 2022

Branched out from #412

@100mik 100mik mentioned this issue May 11, 2022
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@100mik 100mik

Labels
carvel-accepted This issue should be considered for future work and that the triage process has been completed cli Issue for kapp-controller cli enhancement This issue is a feature request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add --values to kctrl package installed update carvel-dev/kapp-controller
1 participant
@100mik