Skip to content

Conversation

@alongwill
Copy link
Member

Show the list of sensitive Talos resources that cannot be viewed for Omni managed clusters.

Show the list of sensitive Talos resources that cannot be viewed for Omni managed clusters.

Signed-off-by: Andrew Longwill <[email protected]>
Copy link
Member

@smira smira left a comment

Choose a reason for hiding this comment

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

but I would also ask @Iheanacho-ai to review

@github-project-automation github-project-automation bot moved this from In Review to Approved in Planning Oct 20, 2025
@smira smira removed this from Planning Oct 27, 2025

## Sensitive resources

Omni prevents viewing machine configuration which might contain sensitive PKI. To view the list of resources that cannot be viewed you can use this command:
Copy link
Member

Choose a reason for hiding this comment

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

i personally think that there are too many "view" in this paragraph

would this be a better rewrite?

'Omni restricts access to machine configurations that may contain sensitive PKI information. To see which resources are restricted, run the following command:'


## Sensitive resources

Omni prevents viewing machine configuration which might contain sensitive PKI. To view the list of resources that cannot be viewed you can use this command:
Copy link
Member

Choose a reason for hiding this comment

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

i added this comment yesterday but for some reason it has been pending since, but here are my thoughts:

i personally think that there are too many "view" in this paragraph

would this be a better rewrite?

'Omni restricts access to machine configurations that may contain sensitive PKI information. To see which resources are restricted, run the following command:'

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.

5 participants