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

ps.pgrep raises NoSuchProcess #23021

Closed
kevinboulain opened this issue Apr 24, 2015 · 1 comment
Closed

ps.pgrep raises NoSuchProcess #23021

kevinboulain opened this issue Apr 24, 2015 · 1 comment
Labels
Bug broken, incorrect, or confusing behavior Execution-Module fixed-pls-verify fix is linked, bug author to confirm fix P3 Priority 3 Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Milestone
Approved

Comments

@kevinboulain
Copy link
Contributor

Hi there,

Sometimes ps.pgrep can raise an exception NoSuchProcess.
https://github.com/saltstack/salt/blob/develop/salt/modules/ps.py#L324-331

Some try/except have been added after being reported in #9274, but maybe the ps module misses some more catches?

Regards.
@kevinboulain kevinboulain changed the title ps.pgrep rasises NoSuchProcess ps.pgrep raises NoSuchProcess Apr 24, 2015
@jfindlay jfindlay mentioned this issue Apr 24, 2015
Merged
@jfindlay jfindlay added Execution-Module Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around P3 Priority 3 labels Apr 24, 2015
@jfindlay jfindlay added this to the Approved milestone Apr 24, 2015
@jfindlay jfindlay added the fixed-pls-verify fix is linked, bug author to confirm fix label Apr 24, 2015
@jfindlay
Copy link
Contributor

@ether42, thanks for the report.

@jfindlay jfindlay added the Platform Relates to OS, containers, platform-based utilities like FS, system based apps label May 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug broken, incorrect, or confusing behavior Execution-Module fixed-pls-verify fix is linked, bug author to confirm fix P3 Priority 3 Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Projects
None yet
Milestone
Approved
Development

No branches or pull requests
2 participants
@jfindlay @kevinboulain