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

[v24.2.x] auditing: construct metrics probe during service construction #24127

Conversation

vbotbuildovich
Copy link
Collaborator

@vbotbuildovich vbotbuildovich commented Nov 14, 2024

Backport of PR #24108

Fixes: CORE-8251

Move the construction of the audit probe from service start to service
construction. Previously, the audit probe was created as a unique pointer
when the service was started and destroyed when the service stopped.

Since the audit service has the same lifespan as the application, creating
the probe during service construction reduces the risk of dereferencing a
null pointer if the probe is accessed before the service has started.

Signed-off-by: Michael Boquard <michael@redpanda.com>
(cherry picked from commit 862e549)
@vbotbuildovich vbotbuildovich added this to the v24.2.x-next milestone Nov 14, 2024
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Nov 14, 2024
Copy link
Contributor

@michael-redpanda michael-redpanda left a comment

Choose a reason for hiding this comment

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

clean backport

@michael-redpanda michael-redpanda self-assigned this Nov 14, 2024
@vbotbuildovich
Copy link
Collaborator Author

@michael-redpanda michael-redpanda merged commit 4a9144b into redpanda-data:v24.2.x Nov 15, 2024
18 checks passed
@piyushredpanda piyushredpanda modified the milestones: v24.2.x-next, v24.2.12 Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/redpanda kind/backport PRs targeting a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants