Skip to content

feat: Gather namespace Roles and RoleBindings - #43

Merged
olivergondza merged 2 commits into
redhat-developer:mainfrom
olivergondza:gather-rbac
Aug 24, 2026
Merged

feat: Gather namespace Roles and RoleBindings#43
olivergondza merged 2 commits into
redhat-developer:mainfrom
olivergondza:gather-rbac

Conversation

@olivergondza

@olivergondza olivergondza commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

What type of PR is this?

Uncomment only one /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug
/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

What does this PR do / why we need it:

│       ├── rbac.authorization.k8s.io
│       │   ├── rolebindings
│       │   │   ├── backplane-ai-agent-0.yaml
│       │   │   ├── backplane-cee-0.yaml
│       │   │   ├── backplane-cse-0.yaml
│       │   │   ├── backplane-csm-0.yaml
│       │   │   ├── backplane-lpsre-0.yaml
│       │   │   ├── backplane-lpsre-package-operator-1.yaml
│       │   │   ├── backplane-mcs-tier-two-0.yaml
│       │   │   ├── backplane-mobb-0.yaml
│       │   │   ├── backplane-srep-0.yaml
│       │   │   ├── backplane-srep-1.yaml
│       │   │   ├── backplane-srep-ro-0.yaml
│       │   │   ├── backplane-srep-ro-1.yaml
│       │   │   ├── backplane-tam-0.yaml
│       │   │   ├── openshift-gitops-applicationset-controller.yaml
│       │   │   ├── openshift-gitops-argocd-application-controller.yaml
│       │   │   ├── openshift-gitops-argocd-dex-server.yaml
│       │   │   ├── openshift-gitops-argocd-redis-ha.yaml
│       │   │   ├── openshift-gitops-argocd-redis.yaml
│       │   │   ├── openshift-gitops-argocd-server.yaml
│       │   │   ├── openshift-gitops-prometheus-k8s-read-binding.yaml
│       │   │   ├── system:deployers.yaml
│       │   │   ├── system:image-builders.yaml
│       │   │   └── system:image-pullers.yaml
│       │   └── roles
│       │       ├── openshift-gitops-applicationset-controller.yaml
│       │       ├── openshift-gitops-argocd-application-controller.yaml
│       │       ├── openshift-gitops-argocd-dex-server.yaml
│       │       ├── openshift-gitops-argocd-redis-ha.yaml
│       │       ├── openshift-gitops-argocd-redis.yaml
│       │       ├── openshift-gitops-argocd-server.yaml
│       │       └── openshift-gitops-read.yaml

Have you updated the necessary documentation?

  • Documentation update is required by this PR.
  • Documentation has been updated.

Which issue(s) this PR fixes:

https://redhat.atlassian.net/browse/GITOPS-10548

Test acceptance criteria:

  • Unit Test
  • E2E Test

How to test changes / Special notes to the reviewer:

PTAL @jparsai

Signed-off-by: Oliver Gondža <ogondza@gmail.com>
Comment thread gather_gitops.sh
while read -r NAMESPACE; do
echo "gather_gitops:$LINENO] inspecting namespace $NAMESPACE .." | tee -a ${LOGS_DIR}/gather_gitops.log
oc adm inspect --dest-dir=${LOGS_DIR} "ns/$NAMESPACE" > /dev/null
oc adm inspect --dest-dir=${LOGS_DIR} -n "$NAMESPACE" roles,rolebindings > /dev/null

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is it not possible collect ClusterRoles and ClusterRoleBinding?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Good call. Can be done. It quadruples the number of files for my dummy deployment, but it only grows the uncompressed bundle by ~30% in size. :shrugs:

Signed-off-by: Oliver Gondža <ogondza@gmail.com>

@jparsai jparsai left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks @olivergondza LGTM

@olivergondza
olivergondza merged commit 8e59206 into redhat-developer:main Aug 24, 2026
4 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.

2 participants