Skip to content

Commit

Permalink
Merge pull request #169 from lars-t-hansen/w-89-trying-again
Browse files Browse the repository at this point in the history
Fix #89 - revamp job id, rollup, emit ppid
  • Loading branch information
bast authored Apr 24, 2024
2 parents 82a60a2 + 735d877 commit 791a6c7
Show file tree
Hide file tree
Showing 17 changed files with 448 additions and 461 deletions.
2 changes: 1 addition & 1 deletion .zenodo.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,5 +25,5 @@
"id": "GPL-3.0"
},
"title": "sonar: Tool to profile usage of HPC resources by regularly probing processes using ps.",
"version": "0.6.2"
"version": "0.11.0"
}
2 changes: 1 addition & 1 deletion Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[package]
name = "sonar"
version = "0.10.0"
version = "0.11.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
Expand Down
19 changes: 18 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,9 +50,18 @@ affect the output nevertheless, ie, most changes not covered by changes to the m

These rules are new with v0.8.0.

### Changes in v0.11.x

**Better `ps` data**. More data points. (v0.11.0)

### Changes in v0.10.x

**Less output**. Removed the `cores` and `memtotalkib` fields, as they are supplied by `sonar sysinfo`.
**Less output**. Removed the `cores` and `memtotalkib` fields, as they are supplied by `sonar
sysinfo`. (v0.10.0)

**Batchless job ID**. The meaning of the `job` field for systems without a batch queue (`sonar ps
--batchless`) has changed from being the pid of the process below the session leader to being the
more conventional process group id. In most situations this won't make a difference. (v0.10.1)

### Changes in v0.9.x

Expand Down Expand Up @@ -170,6 +179,12 @@ v=0.7.0,time=2023-08-10T11:09:41+02:00,host=somehost,cores=8,user=someone,job=0,
v=0.7.0,time=2023-08-10T11:09:41+02:00,host=somehost,cores=8,user=someone,job=0,cmd=slack,cpu%=3.9,cpukib=716924,gpus=none,gpu%=0,gpumem%=0,gpukib=0,cputime_sec=266
```

### Version 0.11.0 `ps` output format

Version 0.11.0 adds one field:

`ppid` (optional, default "0"): The parent process ID of the job, a positive integer.

### Version 0.10.0 `ps` output format

The fields `cores` and `memtotalkib` were removed, as they were unused by all clients and are
Expand Down Expand Up @@ -252,6 +267,8 @@ ID. Multiple records with the job ID 0 should never be merged into a single job
(see `rolledup` below) this has value zero. Otherwise, this record represents one process and so
the field holds the process ID.

`ppid` (optional, default "0"): The parent process ID of the job, a positive integer.

`cpu%` (optional, default "0"): The running average CPU percentage over the true lifetime of the
process (ie computed independently of the sonar log), a nonnegative floating-point number. 100.0
corresponds to "one full core's worth of computation".
Expand Down
Loading

0 comments on commit 791a6c7

Please sign in to comment.