Skip to content

chore(module): allow ClusterAdmin to read internal virtualization resources#2513

Open
fl64 wants to merge 1 commit into
mainfrom
chore/module/cluster-admin-read-internal-virt
Open

chore(module): allow ClusterAdmin to read internal virtualization resources#2513
fl64 wants to merge 1 commit into
mainfrom
chore/module/cluster-admin-read-internal-virt

Conversation

@fl64

@fl64 fl64 commented Jun 20, 2026

Copy link
Copy Markdown
Member

Description

Расширил ClusterRole d8:user-authz:virtualization:cluster-admin: добавил права get/list/watch на внутренние ресурсы виртуализации (api-группы internal.virtualization.deckhouse.io и cdi.internal.virtualization.deckhouse.io).

Why do we need it, and what problem does it solve?

ClusterAdmin не мог читать внутренние ресурсы виртуализации, которые нужны для сбора отладочной информации. Теперь у него есть read-only доступ к тем же ресурсам, которые модуль уже использует внутри себя.

What is the expected result?

ClusterAdmin может kubectl get внутренние ресурсы виртуализации для диагностики.

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: module
type: chore
summary: "Allow ClusterAdmin to read internal virtualization resources for troubleshooting."
impact_level: low

…ources

Add get/list/watch on internal.virtualization.deckhouse.io
(internalvirtualizationkubevirts, virtualmachines, virtualmachineinstances,
virtualmachineinstancemigrations) and cdi.internal.virtualization.deckhouse.io
(internalvirtualizationdatavolumes) to d8:user-authz:virtualization:cluster-admin.

These resources are already read by module controllers and the collectdebuginfo
CLI; exposing them read-only to ClusterAdmin aids troubleshooting.
@fl64 fl64 requested a review from Isteb4k as a code owner June 20, 2026 18:39
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