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

[MSYS-1040] Added validation for image_os_type and connection_port #153

Conversation

kapilchouhan99
Copy link

Signed-off-by: Kapil Chouhan kapil.chouhan@msystechnologies.com

Description

  • whenever we are trying to bootstrap a windows node either --connection_protocol winrm or --connection_protocol ssh, at that time we need to reset windows instance password, so that's why at this stage we need to use image_os_type
  • Added validation for image_os_type and connection_port

Kapil Chouhan and others added 3 commits June 14, 2019 15:57
Fix for Travis failure

Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
Signed-off-by: NAshwini <ashwini.nehate@msystechnologies.com>
Signed-off-by: NAshwini <ashwini.nehate@msystechnologies.com>
@NAshwini NAshwini changed the title Added validation for image_os_type and connection_port [MSYS-1040] Added validation for image_os_type and connection_port Jul 2, 2019
Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
@kapilchouhan99 kapilchouhan99 force-pushed the Kapil/Added_validation_for_image_os_type_and_connection_port branch from d5d30e4 to 9a015fd Compare July 2, 2019 11:31
ui.warn("Auto-migrate disabled for preemptible instance") if preemptible? && locate_config_value(:auto_migrate)
ui.warn("Auto-restart disabled for preemptible instance") if preemptible? && locate_config_value(:auto_restart)
ui.warn("[DEPRECATED] --bootstrap-protocol option is deprecated. Use --connection-protocol option instead.") if locate_config_value(:bootstrap_protocol)
ui.warn("[DEPRECATED] --bootstrap-protocol option is deprecated. Use --connection-protocol option instead.") if locate_config_value(:bootstrap_protocol)
Copy link
Contributor

Choose a reason for hiding this comment

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

Now that we require Chef 15 we may be able to use the new deprecation functionality in mixlib-cli to handle the deprecation right in the option definition. See how we did it directly in knife here: https://github.com/chef/chef/pull/8637/files

Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
@kapilchouhan99 kapilchouhan99 force-pushed the Kapil/Added_validation_for_image_os_type_and_connection_port branch from 9a015fd to 793f099 Compare July 3, 2019 10:27
Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
@btm btm merged commit a27bac0 into chef:master Aug 1, 2019
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

Successfully merging this pull request may close these issues.

4 participants