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

Adding baseline and uuid options #13

Merged
merged 2 commits into from
Mar 20, 2024
Merged

Conversation

paigerube14
Copy link
Collaborator

Type of change

  • Refactor
  • New feature
  • Bug fix
  • Optimization
  • Documentation Update

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

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please describe the System Under Test.
  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

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

@paigerube14
Copy link
Collaborator Author

@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 paigerube14 force-pushed the uuid branch 3 times, most recently from 9a05a85 to cb1424c Compare March 6, 2024 20:43
@paigerube14
Copy link
Collaborator Author

/assign @shashank-boyapally
/assign @jtaleric

orion.py Outdated Show resolved Hide resolved
Copy link
Member

@jtaleric jtaleric left a comment

Choose a reason for hiding this comment

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

lgtm

@paigerube14
Copy link
Collaborator Author

@shashank-boyapally any final thoughts on this?

examples/small-scale-cluster-density.yaml Outdated Show resolved Hide resolved
orion.py Outdated Show resolved Hide resolved
orion.py Outdated Show resolved Hide resolved
orion.py Outdated Show resolved Hide resolved
orion.py Show resolved Hide resolved
utils/orion_funcs.py Show resolved Hide resolved
utils/orion_funcs.py Outdated Show resolved Hide resolved
Signed-off-by: Paige Rubendall <[email protected]>
@shashank-boyapally
Copy link
Contributor

/lgtm, Great work @paigerube14!!

Signed-off-by: Paige Rubendall <[email protected]>
@jtaleric jtaleric merged commit da1c2fe into cloud-bulldozer:main Mar 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants