-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add support to read resources from kubernetes manifests. Closes #128 #129
Conversation
…into parse-kube-manifest
…est is used to create the resource
…nfig is set to list both deployed and manifest resource
@@ -1,4 +1,4 @@ | |||
# Table: kubernetes_{custom_resource_singular_name} | |||
# Table: kubernetes\_{custom_resource_singular_name} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Table: kubernetes\_{custom_resource_singular_name} | |
# Table: kubernetes_{custom_resource_singular_name} |
kubernetesConfig := GetConfig(c) | ||
filterCrds := kubernetesConfig.CustomResourceTables | ||
if len(filterCrds) == 0 { | ||
return nil, nil | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, @misraved, we need this code block to validate if a list of custom resource tables is set in the config before creating the client.
// | ||
// Check for deployed resources | ||
// |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// | |
// Check for deployed resources | |
// | |
// Check for deployed resources | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Subhajit97 please take a look at the minor review comments.
Also please make sure that we have tested the manifest changes against the filewatcher property.
Example query results
Results