cloud-bulldozer / orion

The constellation of a hunter
Apache License 2.0
4 stars 8 forks source link

Adding baseline and uuid options #13

Closed paigerube14 closed 6 months ago

paigerube14 commented 8 months ago

Type of change

Description

A lot of the times we have a set uuid (current run) or one in mind that it would be nice to not have to go get the metadata of the run, create a config file and then run. This would allow a user to set a uuid on command line and have the config file only define the metrics wanted to be collected

Related Tickets & Documents

Checklist before requesting a review

Testing

Example: orion --uuid dc9431e4-43c9-482d-861b-87ab11afbd3a --baseline 2076cfe3-00fc-4fdd-bf76-016bb31264f1

paigerube14 commented 7 months ago

@shashank-boyapally @jtaleric PTAL and let me know your thoughts on this change. Would allow for a user to set a uuid on the command line instead of filling out the metadata of it in a yaml file. Think this would be helpful for our CI runs that would just be able to set uuid, it'll fetch the metadata of that uuid, find any comparison uuid(s) and run the metric collection/comparison on it.

paigerube14 commented 7 months ago

/assign @shashank-boyapally /assign @jtaleric

paigerube14 commented 6 months ago

@shashank-boyapally any final thoughts on this?

shashank-boyapally commented 6 months ago

/lgtm, Great work @paigerube14!!