-
Notifications
You must be signed in to change notification settings - Fork 21
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: added button to delete tag #401
feat: added button to delete tag #401
Conversation
b84f125
to
3a1ea9c
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #401 +/- ##
==========================================
+ Coverage 87.26% 87.49% +0.22%
==========================================
Files 53 55 +2
Lines 1594 1623 +29
Branches 436 433 -3
==========================================
+ Hits 1391 1420 +29
- Misses 190 192 +2
+ Partials 13 11 -2 ☔ View full report in Codecov by Sentry. |
ffd49a4
to
dd1a24c
Compare
dd1a24c
to
68f6817
Compare
@raulkele fixed all comments, thank you! |
68f6817
to
8b25eeb
Compare
Signed-off-by: Petu Eusebiu <[email protected]>
8b25eeb
to
07dd11c
Compare
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.
lgtm
Thanks @eviishondell for PR #391 |
What type of PR is this?
Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs from IPAMD/CNI showing the issue:
Testing done on this change:
Automation added to e2e:
Will this break upgrades or downgrades. Has updating a running cluster been tested?:
Does this change require updates to the CNI daemonset config files to work?:
Does this PR introduce any user-facing change?:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.