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

Fix perf #839

Merged
merged 4 commits into from
May 16, 2024
Merged

Fix perf #839

merged 4 commits into from
May 16, 2024

Conversation

PraveenPenguin
Copy link
Collaborator

No description provided.

@PraveenPenguin PraveenPenguin mentioned this pull request May 13, 2024
@PraveenPenguin
Copy link
Collaborator Author

@abdhaleegit can you please check this PR , as helped me to get this in

@abdhaleegit abdhaleegit self-assigned this May 13, 2024
common/OpTestHMC.py Show resolved Hide resolved
common/OpTestHMC.py Outdated Show resolved Hide resolved
common/OpTestHMC.py Outdated Show resolved Hide resolved
common/OpTestHMC.py Show resolved Hide resolved
…on attribute set to LPAR profile

Added library function which retun True/False if performance collection attribute set to LPAR profile

Signed-off-by: Praveen K Pandey <[email protected]>
…to LPAR using HMC command

Added a library function which help to set perf collection attribute to LPAR using HMC command

Signed-off-by: Praveen K Pandey <[email protected]>
…a dynamic profile

Added support in Machine config as perf collection data can be set via dynamic profile

Signed-off-by: Praveen K Pandey <[email protected]>
Remove non relevant function docstring

Signed-off-by: Praveen K Pandey <[email protected]>
@PraveenPenguin
Copy link
Collaborator Author

@abdhaleegit can you please review again

Copy link
Collaborator

@abdhaleegit abdhaleegit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@abdhaleegit abdhaleegit merged commit 0a4cc62 into open-power:master May 16, 2024
1 check passed
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.

2 participants