-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ClusterResource Analyzer Improvements #1523
Comments
More examples in the docs would be very helpful! The only thing we're planning to use
...which essentially just checks that the The main goal is to simply ensure that expected resources are present in the cluster. In some cases, particularly if a helm chart fails to install altogether, the support bundle will not detect that a resource is missing from the cluster. |
Hey @diamonwiggins, Could you help me to review this PR #1547? Thx! |
Describe the rationale for the suggested feature.
Documentation Improvements
Currently, the ClusterResource analyzer only has a single example which is based on a
pvc
. More examples with different types of objects could help those using the analyzer. Also, for thekind
property it isn't documented which resources theClusterResource
analyzer supports. Lastly, for the resources that are supported, the naming isn't consistent. Some are lowercase and others are upper case emphasizing how important it is to have better documentation.https://troubleshoot.sh/docs/analyze/cluster-resource/
Code Improvements
The naming for the resources shouldn't be case sensitive
The text was updated successfully, but these errors were encountered: