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

utils_sys.py: Add function for finding process PIDS #3983

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

Slancaster1
Copy link
Contributor

Added small utility function get_pids_for(<list>). This function reads the output of 'ps aux' and gets the pids for any process in <list>.

@Slancaster1
Copy link
Contributor Author

@dzhengfy I do not know why the static checks fail

@dzhengfy
Copy link
Contributor

@dzhengfy I do not know why the static checks fail

From the error message, you can see it is ** Running signedoff-check... error. There is incorrect sign-off message in your commit with root committed 12 hours ago now. But you should give like Signed-off-by: Slancaster1 <[email protected]>

virttest/utils_sys.py Show resolved Hide resolved
virttest/utils_sys.py Show resolved Hide resolved
virttest/utils_sys.py Outdated Show resolved Hide resolved
virttest/utils_sys.py Outdated Show resolved Hide resolved
virttest/utils_sys.py Outdated Show resolved Hide resolved
@Slancaster1 Slancaster1 force-pushed the get_pids_for branch 3 times, most recently from e0a3dbc to 553b61b Compare August 29, 2024 12:31
@Slancaster1 Slancaster1 force-pushed the get_pids_for branch 3 times, most recently from a6ed128 to a986d2d Compare August 30, 2024 13:17
Copy link
Contributor

@dzhengfy dzhengfy left a comment

Choose a reason for hiding this comment

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

Others LGTM

virttest/utils_sys.py Outdated Show resolved Hide resolved
Added small utility function get_pids_for(<list>). This
function reads the output of 'ps aux' and gets the pids
for any process in <list>.

Signed-off-by: Slancaster1 <[email protected]>
@Yingshun Yingshun merged commit e1d7d52 into avocado-framework:master Sep 6, 2024
50 checks 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.

3 participants