Skip to content

Remove duplicate permissions check - #869

Draft
jakduch wants to merge 1 commit into
theforeman:masterfrom
jakduch:fix/849-remove-duplicate-permissions
Draft

jakduch wants to merge 1 commit into
theforeman:masterfrom
jakduch:fix/849-remove-duplicate-permissions

Conversation

@jakduch

@jakduch jakduch commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the temporary duplicate permissions health check role
  • stop running the check for Katello and Satellite flavors
  • remove the obsolete check documentation

Merge timing

This is intentionally a draft. Foreman #11271 must be released, and the supported upgrade path must no longer need the safeguard, before this can be merged.

Testing

  • parsed both modified flavor files as YAML
  • ran ansible-lint on both modified flavor files
  • verified no check_duplicate_permissions references remain
  • ran git diff --check

Fixes #849

This pull request was prepared with assistance from OpenAI Codex.

@coderabbitai

coderabbitai Bot commented Sep 25, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

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.

@jakduch
jakduch force-pushed the fix/849-remove-duplicate-permissions branch from 4a929ed to 177d00d Compare September 26, 2026 11:43
@jakduch

jakduch commented Sep 26, 2026

Copy link
Copy Markdown
Contributor Author

The failing deployment jobs hit the Candlepin heap/OOM problem fixed on current master by #865. I rebased this branch onto that fix; CI is running again.

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.

Remove duplicate permissions check after Foreman #38465 is released

1 participant