-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
chore(cli): Migrate argo logs
to use API client. See #2116
#2177
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2177 +/- ##
=========================================
Coverage ? 11.57%
=========================================
Files ? 71
Lines ? 27596
Branches ? 0
=========================================
Hits ? 3193
Misses ? 23998
Partials ? 405 Continue to review full report at Codecov.
|
@@ -1,517 +1,110 @@ | |||
package commands |
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.
this is basically a re-write, so I'd review the new file
@@ -87,6 +87,7 @@ message WorkflowWatchEvent { | |||
|
|||
message LogEntry { | |||
string content = 1; | |||
string podName = 2; |
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.
need this for printing
argo logs
to use API client. See #2116argo logs
to use API client. See #2116
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. I have updated the image argo log -f
is not showing full log when wf is running.
Checklist:
"fix(controller): Updates such and such. Fixes #1234"
.Please merge this as soon as builds are green and it is approved.