Register read_only_base shortname in managed_role_templates - #1023
Register read_only_base shortname in managed_role_templates#1023cshiels-ie wants to merge 1 commit into
Conversation
ManagedReadOnlyBase existed as a class but was not accessible via the ANSIBLE_BASE_MANAGED_ROLE_REGISTRY shortname system, forcing consuming apps to instantiate and register it manually in their AppConfig.ready(). Adding 'read_only_base' to managed_role_templates lets apps declare read-only managed roles through settings alone, consistent with how admin_base and action_base are already exposed. Assisted-by: Claude Code (claude-sonnet-4-6) Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
DVCS PR Check Results: Could not find JIRA key(s) in PR title, branch name, or commit messages |
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## devel #1023 +/- ##
=======================================
Coverage 94.70% 94.70%
=======================================
Files 253 253
Lines 13495 13495
Branches 2043 2043
=======================================
Hits 12781 12781
Misses 714 714
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
|



ManagedReadOnlyBase existed as a class but was not accessible via the ANSIBLE_BASE_MANAGED_ROLE_REGISTRY shortname system, forcing consuming apps to instantiate and register it manually in their AppConfig.ready().
Adding 'read_only_base' to managed_role_templates lets apps declare read-only managed roles through settings alone, consistent with how admin_base and action_base are already exposed.
Assisted-by: Claude Code (claude-sonnet-4-6)
Description
Type of Change
Self-Review Checklist
Testing Instructions
Prerequisites
Steps to Test
Expected Results
Additional Context
Required Actions
Screenshots/Logs