-
Notifications
You must be signed in to change notification settings - Fork 4
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
@ #17 | should install ansible through pip in guest mode #18
@ #17 | should install ansible through pip in guest mode #18
Conversation
please check this PR! Thank you! |
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.
@hieptranquoc pls check my comments
a1dd643
to
d69e7af
Compare
@hoatle I've updated. Please check this PR again. Thank you! |
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.
@hieptranquoc pls check my comments, never forget backward compatibility changes.
d69e7af
to
05a3e99
Compare
@hoatle I've updated. Please check this PR again.! |
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.
@hieptranquoc pls check my comments
|
||
if certs_config['ansible']['mode'] == 'guest' | ||
ansible_install_mode = certs_config['ansible']['install_mode'] | ||
ansible_install_mode = 'pip' unless TeracyDev::Util.exist?(certs_config['ansible']['install_mode']) |
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.
set the default "pip" for yaml config instead (to fix #12)
"_id" => "certs-ansible", | ||
"type" => ansible_type, | ||
"extra_vars" => extra_vars, | ||
"install_mode" => ansible_install_mode, |
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.
set install_mode if ansible_install_mode is valid (not empty)
ansible_install_mode = 'pip' unless TeracyDev::Util.exist?(certs_config['ansible']['install_mode']) | ||
|
||
ansible_install_version = certs_config['ansible']['version'] | ||
ansible_install_version = 'latest' unless TeracyDev::Util.exist?(certs_config['ansible']['version']) |
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.
don't set "latest" as default (similar to the related issue on teracy-dev-k8s)
"type" => ansible_type, | ||
"extra_vars" => extra_vars, | ||
"install_mode" => ansible_install_mode, | ||
"version" => ansible_install_version |
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.
set this only if ansible_install_version is valid (not empty)
merged, I'm going to add more improvements later on. |
connect #17
related teracyhq-incubator/teracy-dev-entry-k8s#10