Skip to content
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 system inventory export flow #547

Merged
merged 2 commits into from
Oct 30, 2024
Merged

Add system inventory export flow #547

merged 2 commits into from
Oct 30, 2024

Conversation

apinkert
Copy link
Contributor

For RHCLOUD-35637. @justinorringer @tahmidefaz this is my first time working in VA. Let me know if I missed anything!

@@ -26,6 +26,7 @@ intents:
- intent_core_here
- intent_core_something_else
- intent_core_other_languages
- intent_core_export_inventory
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure if this is the right place for these, but I couldn't find any other inventory references so I put them next to the language flows since they're in the same section in the Miro board.

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm about to push a PR with an insights/inventory folder haha, we can move it there later

Copy link
Contributor

@justinorringer justinorringer left a comment

Choose a reason for hiding this comment

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

Looking good! Just a couple things

@@ -26,6 +26,7 @@ intents:
- intent_core_here
- intent_core_something_else
- intent_core_other_languages
- intent_core_export_inventory
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm about to push a PR with an insights/inventory folder haha, we can move it there later

steps:
- intent: intent_core_export_inventory
- action: utter_core_export_inventory_1
- action: utter_core_export_inventory_2
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add the form_closing stuff to the end of this rule?

- Can I export this list of servers somehow?
- How to export a csv list from systems inventory?
- How to export to a file de list of systems?
- how can i export a list of inventory systems
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's add some more examples - bot is dumb:

  • export a csv
  • export my systems
  • export systems inventory
  • list my systems
  • dump systems data to file
  • export systems to a file

Copy link
Contributor

@justinorringer justinorringer left a comment

Choose a reason for hiding this comment

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

Looks good!

@justinorringer justinorringer merged commit 4e073fd into main Oct 30, 2024
10 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants