Skip to content

Conversation

@richm
Copy link
Contributor

@richm richm commented Jul 23, 2025

The other files in /etc/pcp/openmetrics/ are all 0644. The latest
pmdaopenmetrics service will get a permission denied trying to open spark.url
without this mode change.

Signed-off-by: Rich Megginson [email protected]

Copy link
Member

@natoscott natoscott left a comment

Choose a reason for hiding this comment

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

LGTM, you could maybe tweak the commit message a little (s/pmcd/pmdaopenmetrics/g) if you like, but I'm sure people will get the gist as-is too.

The other files in /etc/pcp/openmetrics/ are all 0644.  The latest
pmdaopenmetrics service will get a permission denied trying to open spark.url
without this mode change.

Signed-off-by: Rich Megginson <[email protected]>
@richm richm force-pushed the fix-spark-url-must-be-0644 branch from c982ca0 to 72ee8ec Compare July 23, 2025 22:32
@richm
Copy link
Contributor Author

richm commented Jul 23, 2025

pmdaopenmetrics was recently changed to run 'unprivileged' (under the 'pcp' user account) - previously it ran as root - this is some unintended fallout (and unforeseen, I'm wondering if that change might affect anything else now - its the first diagnosed case of this I've seen fortunately, so hopefully this isn't common).

@natoscott does this mean we should keep the permissions at 0600 for older versions of pmdaopenmetrics? If so, which version has the change?

@natoscott
Copy link
Member

pmdaopenmetrics was recently changed to run 'unprivileged' (under the 'pcp' user account) - previously it ran as root - this is some unintended fallout (and unforeseen, I'm wondering if that change might affect anything else now - its the first diagnosed case of this I've seen fortunately, so hopefully this isn't common).

@natoscott does this mean we should keep the permissions at 0600 for older versions of pmdaopenmetrics?

There's no reason to do so I can think of - root can read it either way and its not sensitive information.

If so, which version has the change?

This is the relevant changelog entry for upstream pcp-6.3.2:
https://github.com/performancecopilot/pcp/blob/87920ab31083f3e7bc10de498f78da26534ce7e8/CHANGELOG#L114

Not sure which RHEL releases this has been back-ported to off the top of my head.

@richm richm force-pushed the fix-spark-url-must-be-0644 branch from 0aed4c5 to 72ee8ec Compare July 24, 2025 00:20
@richm richm merged commit 1c5a08e into performancecopilot:main Jul 24, 2025
10 of 12 checks passed
@richm richm deleted the fix-spark-url-must-be-0644 branch July 24, 2025 00:50
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