Fixes #39050 - PostgreSQL process name pattern is wrong in PCP foreman-hotproc config file#10815
Merged
evgeni merged 1 commit intotheforeman:developfrom Feb 2, 2026
Merged
Conversation
evgeni
reviewed
Jan 30, 2026
82fe681 to
1de067b
Compare
evgeni
reviewed
Feb 2, 2026
extras/pcp/foreman-hotproc.conf
Outdated
| fname == "redis-server" || | ||
| (fname == "postgres" && psargs ~ /-D/) || | ||
| (fname == "postmaster" && psargs ~ /-D/) || | ||
| psargs ~ /postgres/ || |
Member
There was a problem hiding this comment.
Suggested change
| psargs ~ /postgres/ || |
Please drop that one :)
Member
|
Is there a redmine we could reference in the commit message? the bot won't let us merge without :) |
Contributor
Author
|
I've just created https://projects.theforeman.org/issues/39050 . |
Member
|
Okay, if you now amend your commit message to read (sorry, commit rules are strict in here) |
b502917 to
c36701a
Compare
Contributor
Author
|
Is the Redmine check failing because of the maybe longer than expected description? |
Member
|
No, |
…n-hotproc config file Fix PostgreSQL process name patterns in config to match current reality: ``` [root@satellite ~]# ps auxw | grep -w -e postgres -e postmaster postgres 139026 0.1 0.3 4429260 112736 ? Ss Jan08 1:49 /usr/bin/postmaster -D /var/lib/pgsql/data postgres 139027 0.0 0.0 91644 20752 ? Ss Jan08 0:00 postgres: logger postgres 139029 0.0 7.7 4430672 2531748 ? Ss Jan08 0:36 postgres: checkpointer postgres 139030 0.0 0.2 4429648 81124 ? Ss Jan08 0:04 postgres: background writer postgres 139031 0.0 0.0 4429260 21700 ? Ss Jan08 0:15 postgres: walwriter postgres 139032 0.0 0.0 4430632 7704 ? Ss Jan08 0:02 postgres: autovacuum launcher postgres 139033 0.1 0.0 69516 6172 ? Ss Jan08 1:21 postgres: stats collector postgres 139034 0.0 0.0 4430336 6388 ? Ss Jan08 0:00 postgres: logical replication launcher postgres 140719 0.0 0.4 4436176 153476 ? Ss Jan08 0:15 postgres: pulp pulpcore ::1(56720) idle postgres 140735 0.0 0.4 4434076 148580 ? Ss Jan08 0:16 postgres: pulp pulpcore ::1(56722) idle . . . ```
Contributor
Author
|
Thanks!!! I get it now :) |
evgeni
approved these changes
Feb 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
To match current reality:
Fixes: https://issues.redhat.com/browse/SAT-41438