Skip to content

Conversation

@sago2k8
Copy link
Contributor

@sago2k8 sago2k8 commented Oct 2, 2025

Description

The assign role limitation is not true anymore, you can assign roles via the console UI

Resolves https://redpandadata.atlassian.net/browse/
Review deadline:

Page previews

Checks

  • New feature
  • Content gap
  • Support Follow-up
  • Small fix (typos, links, copyedits, etc)

The assign role limitation is not true anymore, you can assign roles via
de console UI

Signed-off-by: santiago <santiago@redpanda.com>
@sago2k8 sago2k8 requested a review from a team as a code owner October 2, 2025 23:10
@netlify
Copy link

netlify bot commented Oct 2, 2025

Deploy Preview for redpanda-docs-preview ready!

Name Link
🔨 Latest commit 96b8f08
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-docs-preview/deploys/68df0672e1dc670007ee5cb4
😎 Deploy Preview https://deploy-preview-1371--redpanda-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 2, 2025

📝 Walkthrough

Walkthrough

A single documentation update in modules/manage/partials/security/oidc/limitations.adoc removes a bullet stating that Redpanda Console cannot assign roles to OIDC principals and must rely on rpk. No other content changes are made, and there are no code or API modifications.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The PR description includes the required sections from the template but leaves the Jira ticket placeholder unresolved and provides no actual page preview URLs, resulting in incomplete required information. Replace the `<jira-ticket>` placeholder with the actual ticket number in the “Resolves” line and add the Netlify preview URLs under “## Page previews” to demonstrate the documentation change.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title succinctly and accurately reflects the primary change by indicating that the OIDC documentation statement about role assignment is being removed, follows the standard “docs(scope): summary” format, and is clear and specific without extraneous detail.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sj/fix-incorrect-statement

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between f324697 and 96b8f08.

📒 Files selected for processing (1)
  • modules/manage/partials/security/oidc/limitations.adoc (0 hunks)
💤 Files with no reviewable changes (1)
  • modules/manage/partials/security/oidc/limitations.adoc
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Redirect rules - redpanda-docs-preview
  • GitHub Check: Header rules - redpanda-docs-preview
  • GitHub Check: Pages changed - redpanda-docs-preview

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@micheleRP micheleRP left a comment

Choose a reason for hiding this comment

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

thank you @sago2k8!

@sago2k8 sago2k8 merged commit 7840a9c into main Oct 3, 2025
7 checks passed
@sago2k8 sago2k8 deleted the sj/fix-incorrect-statement branch October 3, 2025 08:54
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