Skip to content

Conversation

@dyl10s
Copy link
Collaborator

@dyl10s dyl10s commented Aug 25, 2025

Description

This updates the Vault Audit receiver to use LoggingReceiverMacro

This is one of the few cases that actually needs a separate definition of LoggingReceiverMacro because there are custom parameters.

This also has a similar issue as #2043 where the multiline rules are not working correctly. Will wait for an update there to see what I need to fix.

Related issue

How has this been tested?

Checklist:

  • Unit tests
    • Unit tests do not apply.
    • Unit tests have been added/modified and passed for this PR.
  • Integration tests
    • Integration tests do not apply.
    • Integration tests have been added/modified and passed for this PR.
  • Documentation
    • This PR introduces no user visible changes.
    • This PR introduces user visible changes and the corresponding documentation change has been made.
  • Minor version bump
    • This PR introduces no new features.
    • This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
    • This PR bumps the version.

@dyl10s dyl10s marked this pull request as ready for review August 25, 2025 19:56
@franciscovalentecastro franciscovalentecastro added the kokoro:force-run Forces kokoro to run integration tests on a CL label Aug 28, 2025
@stackdriver-instrumentation-release stackdriver-instrumentation-release removed the kokoro:force-run Forces kokoro to run integration tests on a CL label Aug 28, 2025
@github-actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Oct 28, 2025
@github-actions
Copy link

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions bot removed the Stale label Nov 24, 2025
@franciscovalentecastro franciscovalentecastro merged commit 6742077 into GoogleCloudPlatform:master Nov 24, 2025
68 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.

3 participants