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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: argocd app delete apps by label #10118
Conversation
Codecov ReportBase: 45.70% // Head: 45.66% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #10118 +/- ##
==========================================
- Coverage 45.70% 45.66% -0.04%
==========================================
Files 236 236
Lines 28668 28690 +22
==========================================
Hits 13102 13102
- Misses 13772 13794 +22
Partials 1794 1794
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@danielhelfand if you resolve conflicts, I'll be happy to merge! |
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Thanks again for looking everything over. I have resolved the conflicts. |
Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @danielhelfand!
* feat: argocd app delete apps by label Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: add path to e2e test Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: use guestbook path in e2e test Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: change operation status assertions in e2e test Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: change sync status to synced Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: assert on cli output for err in e2e test Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: assert on err not output in e2e test Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: ignore error to allow assertion to take place in test Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: assert on Contains instead of Equal Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: swtich args to assert.Contains Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: selector and args should not be used together Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: add app to delete example Signed-off-by: Daniel Helfand <helfand.4@gmail.com> * fix: return err instead of nil Signed-off-by: Daniel Helfand <helfand.4@gmail.com> Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
Closes #10091
Allow
argocd app delete
command to delete apps having a common label. UX isargocd app delete -l foo=bar
. Similar toargocd app sync -l
experience.Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist: