-
Notifications
You must be signed in to change notification settings - Fork 119
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: add support for Argo Rollouts #1098
feat: add support for Argo Rollouts #1098
Conversation
Signed-off-by: Soumil Paranjpay <[email protected]> fix: add untracked files Signed-off-by: Soumil Paranjpay <[email protected]> fix: use argo rollouts library structs Signed-off-by: Soumil Paranjpay <[email protected]>
Signed-off-by: Soumil Paranjpay <[email protected]>
Thanks for making a pull request! 😃 |
Signed-off-by: Soumil Paranjpay <[email protected]>
k8s.io/apiserver v0.26.2 // indirect | ||
k8s.io/cli-runtime v0.23.1 // indirect | ||
k8s.io/cli-runtime v0.25.8 // indirect |
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.
Are we going with v0.24.2
or v0.25.8
? Also, please recheck if everything works well if you change the version.
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.
If I'm not wrong, these are indirect dependencies so they have to be 25.8
Signed-off-by: Soumil Paranjpay <[email protected]>
Signed-off-by: Soumil Paranjpay <[email protected]>
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1098 +/- ##
==========================================
- Coverage 14.81% 14.76% -0.06%
==========================================
Files 90 90
Lines 8321 8379 +58
==========================================
+ Hits 1233 1237 +4
- Misses 6770 6824 +54
Partials 318 318 ☔ View full report in Codecov by Sentry. |
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
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
No description provided.