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

roachprod: update default CockroachDB logging configuration #133089

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

jbowens
Copy link
Collaborator

@jbowens jbowens commented Oct 21, 2024

Update the default logging configuration used for roachprod clusters to disable auditable logs on logs going to file sinks. Some roachtests use the buffered:true configuration to withstand disk stall events. This setting is incompatible with auditable logs on file sinks and recently introduced validation (#132742) prohibits the settings from being used together.

Release note: none
Informs #129922.
Informs #132988.
Epic: none

Update the default logging configuration used for roachprod clusters to disable
auditable logs on logs going to file sinks. Some roachtests use the
buffered:true configuration to withstand disk stall events. This setting is
incompatible with auditable logs on file sinks and recently introduced
validation (cockroachdb#132742) prohibits the settings from being used together.

Release note: none
Informs cockroachdb#129922.
Informs cockroachdb#132988.
Epic: none
@jbowens jbowens requested review from dhartunian, a team and RaduBerinde and removed request for RaduBerinde October 21, 2024 18:23
@jbowens jbowens requested a review from a team as a code owner October 21, 2024 18:23
@jbowens jbowens requested review from nameisbhaskar and vidit-bhat and removed request for a team October 21, 2024 18:23
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@jbowens jbowens added the backport-24.3.x Flags PRs that need to be backported to 24.3 label Oct 21, 2024
@jbowens
Copy link
Collaborator Author

jbowens commented Oct 21, 2024

TFTR!

bors r=dhartunian

@craig craig bot merged commit 47acd83 into cockroachdb:master Oct 21, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-24.3.x Flags PRs that need to be backported to 24.3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants