-
Notifications
You must be signed in to change notification settings - Fork 2
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
List plugins to report their status #124
Conversation
Thanks @iainelder for raising this PR! I ran the checks and seems like the @pskrbasu For the 2 goals @iainelder raised above, is |
Done. |
Running
Running |
@pskrbasu , thank you for the feedback, and thank you for raising the What are the next steps for this PR? |
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.
@iainelder Please see requested changes, thanks!
Co-authored-by: cbruno10 <[email protected]>
@cbruno10 commented in a resolved thread:
I was about to argue for keeping the They are not really independent actions, though. In practice there is no reason to list the plugins when none are installed. You've persuaded me to fuse the blocks together. |
You only need to list if you installed any plugins.
@cbruno10 , I made changes to address all your feedback. How does it look now? |
Thanks @iainelder for the quick changes! I'm out of office for the rest of this week, but will merge in this change and release a new action version early next week. |
Thanks again @iainelder ! |
Run
steampipe plugin list
after installing any plugin. It has two use cases:information_schema
metadataI made a new GitHub repo called steampipe-csv-init with repros for the described behavior as custom workflows using production version v1.5.2. Below I link to an example workflow run for each use case.
If you need unit tests, please provide guidance.
Report configuration errors
For the repro see run 7 of job report-errors.
If you set up the CSV plugin without a
path
config:Then the new command reports
plugin failed to start
.When the you add the correct path config:
Then the list command reports that the plugin is correctly installed.
A future PR could act on the output of this query to make the workflow fail with an error. Details of why the plugin failed to start may be found in the plugin log file.
Initalizes
information_schema
metadataFor the repro see run 7 of job init-information-schema.
A Steampipe user may use the
information_schema
metadata to the state of Steampipe's database in a GitHub workflow.For example, a user may run this query to show which CSV files Steampipes exposes as tables.
The user has placed a CSV called
acme.csv
.Before running the new command, the
csv
metadata is empty.After running the new command, the
csv
metadata shows theacme
table.