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
util/log/eventpb: direct cluster settings events to the OPS channel #117923
util/log/eventpb: direct cluster settings events to the OPS channel #117923
Conversation
It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR? 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @abarganier)
The OPS channel is documented as reporting "point" operational events such as those initiated by human interactions, including cluster settings. Emitting cluster setting events to the DEV channel was likely a mistake, not intentional. Epic: none Release note (ops change): Events for cluster setting changes are now emitted to the OPS channel rather than the DEV channel.
58bbb10
to
ca11487
Compare
TFTR! bors r+ |
1 similar comment
TFTR! bors r+ |
Already running a review |
TFTR! bors r+ |
Already running a review |
Build succeeded: |
The OPS channel is documented as reporting "point" operational events such as
those initiated by human interactions, including cluster settings. Emitting
cluster setting events to the DEV channel was likely a mistake, not
intentional.
Epic: none
Release note (ops change): Events for cluster setting changes are now emitted
to the OPS channel rather than the DEV channel.