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

DAOS-16781 client: Allow daos_metrics read via pid (#15448) #15466

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

mjmac
Copy link
Contributor

@mjmac mjmac commented Nov 7, 2024

In cases where the client telemetry has been manually
enabled, daos_metrics should be able to read it as
long as the client's PID is known and the user has
read access to the shared memory segment.

Moves the daos_metrics utility into the common daos
package for use from both server and client sides.

Required-githooks: true

Change-Id: Ibdcbb88de450b02bf31513ca7fadf9d73f16bd97
Signed-off-by: Michael MacDonald mjmac@google.com

Copy link

github-actions bot commented Nov 7, 2024

Ticket title is 'Allow daos_metrics to read client metrics by PID'
Status is 'In Review'
https://daosio.atlassian.net/browse/DAOS-16781

In cases where the client telemetry has been manually
enabled, daos_metrics should be able to read it as
long as the client's PID is known and the user has
read access to the shared memory segment.

Moves the daos_metrics utility into the common daos
package for use from both server and client sides.

Required-githooks: true

Change-Id: Ibdcbb88de450b02bf31513ca7fadf9d73f16bd97
Signed-off-by: Michael MacDonald <mjmac@google.com>
@mjmac mjmac force-pushed the mjmac/DAOS-16781-google-2.6 branch from 4451bff to 1992c4f Compare November 7, 2024 13:58
@mjmac mjmac merged commit 310c75c into google/2.6 Nov 7, 2024
47 of 49 checks passed
@mjmac mjmac deleted the mjmac/DAOS-16781-google-2.6 branch November 7, 2024 18:59
jolivier23 pushed a commit that referenced this pull request Feb 3, 2025
In cases where the client telemetry has been manually
enabled, daos_metrics should be able to read it as
long as the client's PID is known and the user has
read access to the shared memory segment.

Moves the daos_metrics utility into the common daos
package for use from both server and client sides.

Required-githooks: true

Change-Id: I1fab0e2fb4fe90228c5c5ff2aabc43122ace0c1c
Signed-off-by: Michael MacDonald <mjmac@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants