Skip to content

[v25.3.x] Fix audit log client on upgrade#28927

Merged
michael-redpanda merged 4 commits intoredpanda-data:v25.3.xfrom
vbotbuildovich:backport-pr-28852-v25.3.x-252
Dec 16, 2025
Merged

[v25.3.x] Fix audit log client on upgrade#28927
michael-redpanda merged 4 commits intoredpanda-data:v25.3.xfrom
vbotbuildovich:backport-pr-28852-v25.3.x-252

Conversation

@vbotbuildovich
Copy link
Copy Markdown
Collaborator

Backport of PR #28852

When building a list of authorized operations for a metadata response,
describe cluster response or a describe groups response, make the authz
checks quiet so they do not clutter the log at INFO level.

Signed-off-by: Michael Boquard <michael@redpanda.com>
(cherry picked from commit c74f284)
Not all uses of the kafka client need metadata or describe_cluster
requests to include the bitmask of authorized operations.  If the
authenticated principal used by the client doesn't have the appropriate
level of authorization to get that information this clutters up the
broker logs with failed authorization checks.  This new option permits
users of k/client to opt out of getting that information.

Signed-off-by: Michael Boquard <michael@redpanda.com>
(cherry picked from commit 5b9ff4c)
All uses of the kafka client outside of Shadow Linking do not require
the authorized operations flag to be set.  This change ensures that any
use disables that flag.

Signed-off-by: Michael Boquard <michael@redpanda.com>
(cherry picked from commit 8a929cc)
Signed-off-by: Michael Boquard <michael@redpanda.com>
(cherry picked from commit d7f6816)
@vbotbuildovich vbotbuildovich requested a review from a team as a code owner December 10, 2025 18:01
@vbotbuildovich vbotbuildovich added this to the v25.3.x-next milestone Dec 10, 2025
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Dec 10, 2025
Copy link
Copy Markdown
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

@vbotbuildovich
Copy link
Copy Markdown
Collaborator Author

CI test results

test results on build#77663
test_class test_method test_arguments test_kind job_url test_status passed reason test_history
ScalingUpTest test_moves_with_local_retention {"use_topic_property": false} integration https://buildkite.com/redpanda/redpanda/builds/77663#019b099c-ac66-4bb6-afd8-c178f15bb527 FLAKY 19/21 upstream reliability is '96.63865546218487'. current run reliability is '90.47619047619048'. drift is 6.16246 and the allowed drift is set to 50. The test should PASS https://redpanda.metabaseapp.com/dashboard/87-tests?tab=142-dt-individual-test-history&test_class=ScalingUpTest&test_method=test_moves_with_local_retention
WriteCachingFailureInjectionTest test_unavoidable_data_loss null integration https://buildkite.com/redpanda/redpanda/builds/77663#019b099c-ac66-4bb6-afd8-c178f15bb527 FLAKY 18/21 upstream reliability is '94.4055944055944'. current run reliability is '85.71428571428571'. drift is 8.69131 and the allowed drift is set to 50. The test should PASS https://redpanda.metabaseapp.com/dashboard/87-tests?tab=142-dt-individual-test-history&test_class=WriteCachingFailureInjectionTest&test_method=test_unavoidable_data_loss

@tyson-redpanda tyson-redpanda modified the milestones: v25.3.2, v25.3.x-next Dec 12, 2025
@michael-redpanda michael-redpanda merged commit befbbec into redpanda-data:v25.3.x Dec 16, 2025
19 checks passed
@piyushredpanda piyushredpanda modified the milestones: v25.3.x-next, v25.3.3 Dec 20, 2025
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.

4 participants