-
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 parse Helm charts #139
Conversation
…into parse-kube-manifest
…into parse-kube-manifest
…est is used to create the resource
… show absolute path to the files
…ailable in the current cluster context
…s parent raw template file and show it when the resource is queried using the respective kube resource tables
…ured multiple times
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 see doc comments, thanks!
@@ -54,100 +60,35 @@ Download and install the latest Kubernetes plugin: | |||
steampipe plugin install kubernetes | |||
``` | |||
|
|||
### Configuration |
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.
Get Started
Install
Configuration
Include simplest example of using kube config with the default kube config file location and the default context (or if they need to specify one, use an example)
Additional configuration examples are included below and for a full list of configuration arguments, please see the default configuration file.
Run a Query
Show steampipe query, the query, results
Configuring Kubernetes Cluster Credentials
...
docs/index.md
Outdated
@@ -16,6 +16,12 @@ og_image: "/images/plugins/turbot/kubernetes-social-graphic.png" | |||
|
|||
[Kubernetes](https://kubernetes.io) is an open-source system for automating deployment, scaling, and management of containerized applications. | |||
|
|||
## Overview |
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.
## Overview |
@@ -199,21 +128,17 @@ connection "kubernetes_all" { | |||
} | |||
``` | |||
|
|||
## Custom Resource Definitions | |||
### Custom Resource Definitions |
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.
### Custom Resource Definitions | |
### Custom resource definitions | |
### Custom Resource Definitions |
When deciding between using title case or not, this one is confusing for me IMO
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.
At a high level, the changes look great and the doc changes seem appropriate. I'm approving but will leave the final review and merge up to @misraved
…into add-support-to-parse-helm-charts
Example query results
Results