-
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
Format quota print #205
Format quota print #205
Conversation
for k, v := range hardLimits { | ||
fmt.Fprintf(w, "%s\t%s\t\n", k, v.(string)) | ||
} | ||
fmt.Fprintf(w, "Memory\t%s\t\n", hardLimits["requests.memory"]) |
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.
Pod limit should be printed, but can use the name: "MaxParallism"
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.
Done.
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.
Done.
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++
* tutorial and usage update * first add * rm not need * fix bugs * add logging * add docker file * fix by yancey's comment * update * save parameter * fix yaml * fix bugs * fix bugs * fix convert bug * fix by wuyi's comment * add readme * rm logging.confg * modify README.md * modify README.md * add start command * update * update by comments * upload files with recursion * prettify output * remove replica set name * recursion to loop * dlnel index page (#194) * switch to dlnel index page * mount template files * delete unused comments * add port env * update tutorial (#202) * update tutorial * update * remove unused line * update * update * update * test format * modify travis.yaml * fix * fix * fix sudo * add travis * add glide * add gimme * fix style * fix style * add files * modify sh * fix by wuyi's comments * fix pre-commit bugs * Format quota print (#205) * forat quota print * update * add sleep for pserver get ready (#216) * Update readme (#214) * update readme * update * Enable ingress notebook access (#219) * enable ingress notebook access * update notebook resource * test ok * fix * fix * fix login url (#229) * Fix invalid job path (#227) * fix invalid job path * update doc * check job name in clint (#231) * [Done]Fault tolerant job (#212) * fault tolerant job * ft job * update * tested job submit with ft job * update
* tutorial and usage update * first add * rm not need * fix bugs * add logging * add docker file * fix by yancey's comment * update * save parameter * fix yaml * fix bugs * fix bugs * fix convert bug * fix by wuyi's comment * add readme * rm logging.confg * modify README.md * modify README.md * add start command * update * update by comments * upload files with recursion * prettify output * remove replica set name * recursion to loop * dlnel index page (#194) * switch to dlnel index page * mount template files * delete unused comments * add port env * update tutorial (#202) * update tutorial * update * remove unused line * update * update * update * test format * modify travis.yaml * fix * fix * fix sudo * add travis * add glide * add gimme * fix style * fix style * add files * modify sh * fix by wuyi's comments * fix pre-commit bugs * Format quota print (#205) * forat quota print * update * add sleep for pserver get ready (#216) * Update readme (#214) * update readme * update * Enable ingress notebook access (#219) * enable ingress notebook access * update notebook resource * test ok * fix * fix * fix login url (#229) * Fix invalid job path (#227) * fix invalid job path * update doc * check job name in clint (#231) * [Done]Fault tolerant job (#212) * fault tolerant job * ft job * update * tested job submit with ft job * update * fix ci
Just modify quota print as flowing: