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

Roll service_report_generator.py into the frost CLI #385

Open
ajvb opened this issue Oct 9, 2020 · 2 comments
Open

Roll service_report_generator.py into the frost CLI #385

ajvb opened this issue Oct 9, 2020 · 2 comments
Milestone

Comments

@ajvb
Copy link
Contributor

ajvb commented Oct 9, 2020

Now that we have a frost CLI it might be nice to roll https://github.com/mozilla/frost/blob/master/service_report_generator.py into it.

@ajvb
Copy link
Contributor Author

ajvb commented Oct 27, 2020

Alongside this, we should add documentation around the JSON output that server_report_generator.py creates, beyond what is documented in the script itself.

@ajvb ajvb mentioned this issue Oct 27, 2020
@hwine
Copy link
Contributor

hwine commented Oct 27, 2020

So this is an interesting architectural decision for me with GitHub. I don't really understand the script, so was planning on consuming the JSON directly from frost in my post-test-actions.

What are the reasons I should reconsider that decision?

  • much easier to parse output <== sufficient!

Looking forward to the integration!

@ajvb ajvb added this to the 0.5.0 milestone Oct 30, 2020
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

No branches or pull requests

2 participants