-
Notifications
You must be signed in to change notification settings - Fork 75
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
Prettify output #200
Prettify output #200
Conversation
elif [ $ret -eq 1 ] ; then | ||
echo "General Error" > /dev/termination-log | ||
elif [ $ret -eq 134 ] ; then | ||
echo "Program Abort" > /dev/termination-log |
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.
Deal with an unknown return code?
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.
unknown就不现实message了,ret code还是可以看到的
go/paddlecloud/get.go
Outdated
w := tabwriter.NewWriter(os.Stdout, 0, 0, 3, ' ', 0) | ||
if len(items) >= 0 { | ||
fmt.Fprintf(w, "NUM\tNAME\tSUCC\tFAIL\tSTART\tCOMP\tACTIVE\t\n") | ||
fmt.Fprintf(w, "NAME\tACTIVE\tSUCC\tFAIL\tSTART\tCOMP\tPS_NAME\tPS_READY\tPS_TOTAL\t\n") |
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.
What's the PS_NAME used for? Maybe only print the stats of PServer?
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.
replicaset的name不过对于用户来说没什么用了倒是。可以去掉
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
Fix #195
Note here's still a problem when train exited with error, submit with:
will not show message when
paddlecloud get workers [jobname]