Skip to content
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

Misc improvements #13

Merged
merged 8 commits into from
Dec 18, 2023
Merged

Misc improvements #13

merged 8 commits into from
Dec 18, 2023

Conversation

mahendrapaipuri
Copy link
Owner

  • Update README
  • Read /proc fs in go routines to improve perf
  • Use jobidraw in sacct instead of jobid. This is useful for array jobs to properly identify correct job ID
  • Remove run.with.sudo flag for IPMI collector. Redundant.
  • Add example config files
  • Rename batchjob_stats_api to batchjob_stats_server

Signed-off-by: Mahendra Paipuri <[email protected]>
* Add tests using fake procfs

* Rename CLI opts to be more intuitive

* Update test data files accordingly

Signed-off-by: Mahendra Paipuri <[email protected]>
* jobidraw will show real job id for array jobs

* Improve wording on CLI help

Signed-off-by: Mahendra Paipuri <[email protected]>
* Users can simply use sudo in command cli option. CLI flag is redundant

Signed-off-by: Mahendra Paipuri <[email protected]>
Signed-off-by: Mahendra Paipuri <[email protected]>
Signed-off-by: Mahendra Paipuri <[email protected]>
* batchjob_stats_server is more appropriate than stats_api

Signed-off-by: Mahendra Paipuri <[email protected]>
* Formatted with golines -w --max-len=120 .

Signed-off-by: Mahendra Paipuri <[email protected]>
@mahendrapaipuri mahendrapaipuri merged commit 5edb17c into main Dec 18, 2023
5 checks passed
@mahendrapaipuri mahendrapaipuri deleted the new_feats branch December 18, 2023 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant