Skip to content

Remove Register groups section from gbac-dp partial#1654

Open
micheleRP wants to merge 1 commit intomainfrom
DOC-1809-remove-register-groups-from-gbac-dp
Open

Remove Register groups section from gbac-dp partial#1654
micheleRP wants to merge 1 commit intomainfrom
DOC-1809-remove-register-groups-from-gbac-dp

Conversation

@micheleRP
Copy link
Copy Markdown
Contributor

Summary

  • Remove the Cloud-only Register groups in Redpanda Cloud section (ifdef::env-cloud[]) from modules/manage/partials/gbac-dp.adoc
  • This section has been moved to the cloud-docs repo under security/authorization/gbac/gbac.adoc as part of the GBAC restructuring in DOC-1809

Test plan

  • Verify the self-hosted GBAC page still renders correctly without the removed section
  • Verify the Cloud GBAC data plane page renders correctly (the section now lives in cloud-docs)

🤖 Generated with Claude Code

This Cloud-only section has been moved to the cloud-docs repo under
security/authorization/gbac/gbac.adoc as part of the GBAC restructuring
in DOC-1809.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@micheleRP micheleRP requested a review from a team as a code owner April 1, 2026 22:20
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 1, 2026

Deploy Preview for redpanda-docs-preview ready!

Name Link
🔨 Latest commit 3ae640b
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-docs-preview/deploys/69cd9a23c62e8500083ec78c
😎 Deploy Preview https://deploy-preview-1654--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
Copy Markdown
Contributor

coderabbitai bot commented Apr 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: eced48d5-e6da-43cc-b8fb-c498a77a342c

📥 Commits

Reviewing files that changed from the base of the PR and between c4f9842 and 3ae640b.

📒 Files selected for processing (1)
  • modules/manage/partials/gbac-dp.adoc
💤 Files with no reviewable changes (1)
  • modules/manage/partials/gbac-dp.adoc

📝 Walkthrough

Walkthrough

This pull request removes a documentation section from the GBAC documentation file that provided instructions for registering IdP groups in Redpanda Cloud. The removed section included both Cloud UI and Control Plane API guidance (POST /v1/groups). The document structure now proceeds directly from the "No wildcard ACLs for groups" limitation to the "Create group-based ACLs" section without cloud-specific group-registration instructions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • rockwotj
  • kbatuigas
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is missing required sections from the template, including JIRA ticket link, review deadline, and page previews. Add the Resolves JIRA ticket link (DOC-1809), review deadline, and page preview URLs for the modified documentation pages.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and specifically describes the main change: removing a cloud-specific section from a documentation file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch DOC-1809-remove-register-groups-from-gbac-dp

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

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.

1 participant