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

[v22.3.x] tests/redpanda: do not include command with no args into pid list #10431

Conversation

vbotbuildovich
Copy link
Collaborator

Backport of PR #10154
Fixes #10430,

When `ps` is unable to determine process arguments it wraps the command
name in square brackets like this:
```
262250 pts/278  R+     0:00 [redpanda]
```

this was incorrectly interpreted as running Redpanda, the node was never
brought back up.
Fixed issue by filtering out incorrect `ps` output lines.

Fixes: redpanda-data#8753

Signed-off-by: Michal Maslanka <[email protected]>
(cherry picked from commit 81c37c5)
@vbotbuildovich vbotbuildovich added this to the v22.3.x-next milestone Apr 28, 2023
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Apr 28, 2023
@mmaslankaprv mmaslankaprv marked this pull request as ready for review April 28, 2023 10:07
@michael-redpanda
Copy link
Contributor

Closing as obsolete due to #10520

@vshtokman vshtokman modified the milestones: v22.3.x-next, v22.3.18 May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/backport PRs targeting a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants