-
Notifications
You must be signed in to change notification settings - Fork 41
jobreport #113
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
jobreport #113
Conversation
|
preview available: https://docs.tds.cscs.ch/113 |
|
preview available: https://docs.tds.cscs.ch/113 |
henrique
left a comment
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!
msimberg
left a comment
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.
Only minor cosmetic requests. Thanks @boeschf for porting this!
docs/running/jobreport.md
Outdated
| ## Downloading the job summary report | ||
|
|
||
| Precompiled binary for the `jobreport` utility can be obtained directly from the [repository](https://github.com/eth-cscs/alps-jobreport/releases) or via the command line: | ||
|
|
||
| ```console | ||
| $ wget https://github.com/eth-cscs/alps-jobreport/releases/download/v0.1/jobreport | ||
| $ chmod +x ./jobreport | ||
| ``` |
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.
Not for this PR: @simonpintarelli since you've been deploying uenv binaries on the systems for a while, how easy do you think it would be to do the same for jobreport? It's a bit crazy that we need to ask users to download this tool manually.
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.
It should be pretty easy.
It would have to use the /user-tools mount so that it can be used alongside application/prgenv uenv.
Co-authored-by: Mikael Simberg <mikael.simberg@iki.fi>
Co-authored-by: Mikael Simberg <mikael.simberg@iki.fi>
Co-authored-by: Mikael Simberg <mikael.simberg@iki.fi>
Co-authored-by: Mikael Simberg <mikael.simberg@iki.fi>
|
preview available: https://docs.tds.cscs.ch/113 |
Co-authored-by: Mikael Simberg <mikael.simberg@iki.fi>
|
preview available: https://docs.tds.cscs.ch/113 |
Co-authored-by: Mikael Simberg <mikael.simberg@iki.fi>
|
preview available: https://docs.tds.cscs.ch/113 |
3 similar comments
|
preview available: https://docs.tds.cscs.ch/113 |
|
preview available: https://docs.tds.cscs.ch/113 |
|
preview available: https://docs.tds.cscs.ch/113 |
|
preview available: https://docs.tds.cscs.ch/113 |
|
preview available: https://docs.tds.cscs.ch/113 |
1 similar comment
|
preview available: https://docs.tds.cscs.ch/113 |
No description provided.