Skip to content

Add procps-ng as RPM dependency to fix pxf stop command#93

Merged
tuhaihe merged 1 commit intoapache:mainfrom
tuhaihe:update-rpm-ps-cmd
Apr 3, 2026
Merged

Add procps-ng as RPM dependency to fix pxf stop command#93
tuhaihe merged 1 commit intoapache:mainfrom
tuhaihe:update-rpm-ps-cmd

Conversation

@tuhaihe
Copy link
Copy Markdown
Member

@tuhaihe tuhaihe commented Apr 3, 2026

The pxf script uses the 'ps' command in the isRunning() function to check if the PXF process is running. However, minimal container images like Rocky Linux 9 do not include the 'ps' command by default.

This causes the 'pxf stop' command to fail with:
/usr/local/cloudberry-pxf/bin/pxf: line 162: ps: command not found

Add explicit package dependencies to ensure the 'ps' command is available when PXF is installed:

  • RPM packages: procps-ng
  • DEB packages: procps

The 'ps' command is used in:

  • server/pxf-service/src/scripts/pxf:162 (isRunning function)

closes: #ISSUE_Number


Change logs

Describe your change clearly, including what problem is being solved or what document is being added or updated.

Contributor's checklist

Here are some reminders before you submit your pull request:

The pxf script uses the 'ps' command in the isRunning() function
to check if the PXF process is running. However, minimal container
images like Rocky Linux 9 do not include the 'ps' command by default.

This causes the 'pxf stop' command to fail with:
  /usr/local/cloudberry-pxf/bin/pxf: line 162: ps: command not found

Add explicit package dependencies to ensure the 'ps' command is
available when PXF is installed:
- RPM packages: procps-ng
- DEB packages: procps

The 'ps' command is used in:
- server/pxf-service/src/scripts/pxf:162 (isRunning function)
@tuhaihe tuhaihe marked this pull request as draft April 3, 2026 02:38
@tuhaihe tuhaihe marked this pull request as ready for review April 3, 2026 09:05
@tuhaihe
Copy link
Copy Markdown
Member Author

tuhaihe commented Apr 3, 2026

Hi @MisterRaindrop @ostinru PTAL. Thanks!

@tuhaihe tuhaihe merged commit 3942c55 into apache:main Apr 3, 2026
42 of 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.

2 participants