-
Notifications
You must be signed in to change notification settings - Fork 293
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
teuthology-suite: add default machine type(smithi) #1625
Conversation
set teuthology_default_machine_type to smithi. see: ceph/teuthology#1625 Signed-off-by: Deepika Upadhyay <[email protected]>
1aae78d
to
b07046c
Compare
b07046c
to
b59d20a
Compare
b59d20a
to
6b3ba72
Compare
@susebot run deploy |
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.
nit: rather than 'multi' check this looks good for me.
I am extremely sorry about I forgot to review this PR earlier. |
Commit 6b3ba72 is OK. |
6b3ba72
to
4e49f42
Compare
updated, thanks! |
I'm fine with the concept. I'll leave the python review up to others. If Kyr's happy, I'm happy. |
@susebot run deploy |
Commit 4e49f42 is OK. |
@kshtsk good to merge? |
yep... |
ouch... probably commit message is outdated and does not correspond the patch content... |
…fied explicitly right now, users have to always pass --machine-type when scheduling a run, when not specified, command fails with no machine type specified error. Instead of failing, we can have `default_machine_type` which in our case should pick smithi, specified in /etc/teuthology.yml Signed-off-by: Deepika Upadhyay <[email protected]>
4e49f42
to
2a15c89
Compare
aah, sorry about that; fixed it! |
@susebot run deploy |
Commit 2a15c89 is OK. |
right now the default_machine_type fallback is set to None via some internal
mangaling, this makes users always have to pass --machine_type which mostly we
prefer to smithi's . Hence, making it a default unless otherwise, should be
helpful for users.
Signed-off-by: Deepika Upadhyay [email protected]