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.

Sign up for GitHub

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

Jump to bottom

Export of PSI metrics to a PCP archive does not work #357

Closed
kurik opened this issue May 4, 2023 · 1 comment · Fixed by #358
Closed

Export of PSI metrics to a PCP archive does not work #357

kurik opened this issue May 4, 2023 · 1 comment · Fixed by #358

Comments

@kurik
Copy link
Contributor

kurik commented May 4, 2023

When PSI metrics are exported from a SA archive to a PCP archive, using sadf the export fails and no PSI metrics are present in the generated PCP archive.

Here is a test case:

# Generate a small SA archive, containing the PSI data
sar -o ~/sar.data -q MEM,CPU,IO 2 10
# Check if PSI data are present in the SA archive
sadf -r ~/sar.data -- -q MEM,IO,CPU | grep -e smem- -e scpu-  -e sio-
# The command above returns a bunch of records with PSI data

# Convert the sar archive to a pcp archive
rm -f ~/pcp.data.*
sadf -l -O pcparchive=~/pcp.data ~/sar.data -- -q MEM,IO,CPU
# The command above responds with the message "PCP: pmiWrite: No data to output" and returns error code 4

# Check the content of the new pcp archive
pmdumplog ~/pcp.data | grep pressure
# The command above returns nothing
@kurik kurik mentioned this issue May 4, 2023
Merged
@kurik
Copy link
Contributor Author

kurik commented May 4, 2023

The pull request above resolves this issue. Unfortunately I did not fully understand the relationship between a test in tests directory and the tests/TLIST file. So, to avoid some testsuite break I decided to describe the test in the description of this issue instead. If anyone can help me to understand how this works, or point me to some documentation, I can extend the pull request with a testcase.

@sysstat sysstat closed this as completed in e9b0d0c May 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed export of PSI metrics to a PCP archive kurik/sysstat
1 participant
@kurik