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

cluster: Sanction redpanda.leaders.preference on invalid license - create topic #24074

Merged

Conversation

IoannisRP
Copy link
Contributor

@IoannisRP IoannisRP commented Nov 8, 2024

Implements: CORE-8042 - Sanction 22

Implement sanction(s) on leadership pinning properties when there is no valid license:

  • Requests to create a topic with a restricted redpanda.leaders.preference value will be denied.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v24.2.x
  • v24.1.x
  • v23.3.x

Release Notes

  • none

@IoannisRP IoannisRP requested review from BenPope and a team November 8, 2024 14:10
BenPope
BenPope previously approved these changes Nov 8, 2024
pgellert
pgellert previously approved these changes Nov 8, 2024
@IoannisRP
Copy link
Contributor Author

Copy link
Member

@BenPope BenPope left a comment

Choose a reason for hiding this comment

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

LGTM

src/v/cluster/topics_frontend.cc Outdated Show resolved Hide resolved
- Extend test to allow properties of any type.
@IoannisRP IoannisRP force-pushed the ik-sanction-pin-leadership-create-topic branch from 498f397 to c5cb776 Compare November 11, 2024 13:27
@IoannisRP
Copy link
Contributor Author

  • Rebased to dev
  • split test refactoring into separate commit
  • Replaced ad-hoc checking for restricted leadership_preference value with call to enterprise::check_restricted

@michael-redpanda
Copy link
Contributor

@ztlpn would appreciate your look here as well, thank you!

src/v/cluster/topics_frontend.cc Outdated Show resolved Hide resolved
BenPope
BenPope previously approved these changes Nov 11, 2024
@IoannisRP IoannisRP force-pushed the ik-sanction-pin-leadership-create-topic branch from c5cb776 to e78e7d1 Compare November 11, 2024 14:41
Implements CORE-8042 - Sanction-22

If there is no valid license, request to create a topic with
redpanda.leaders.preference enabled will be rejected.
@IoannisRP IoannisRP force-pushed the ik-sanction-pin-leadership-create-topic branch from e78e7d1 to 1ddda34 Compare November 11, 2024 14:44
@IoannisRP
Copy link
Contributor Author

Addressed #24074 (comment) and #24074 (comment)

@IoannisRP IoannisRP merged commit 997168d into redpanda-data:dev Nov 13, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants