Refactor windows userdata scripts. #6477

Merged
merged 1 commit into from Dec 2, 2016

Conversation

Projects
None yet
5 participants
Contributor

hoenirvili commented Oct 19, 2016

Contains modifications in the windows userdata scripts so
it can be run multiple times during provisioning without changing it's final outcome.

Contributor

perrito666 commented Oct 19, 2016

!!build!!

Owner

mitechie commented Oct 19, 2016

!!build!!

Contributor

hoenirvili commented Oct 19, 2016

!!build!!

axw approved these changes Oct 27, 2016

LGTM. Can you please describe what steps you took to test this? I'm presuming this is to support manual provisioning of Windows.

Contributor

hoenirvili commented Oct 27, 2016

@axw
The manual windows provisioning patch does not depend on this patch, but during the development I observerd this wierd behaviour.

The story of this patch come from, trying multiple times to provision the same windows machine after every juju remove-machine <number of machine>

Concicise example

juju add-machine --debug winrm:Administrator@192.168.11.11  
juju remove-machine <number-of-that-machine>

It worked, it creates the machine and fully removes it without problems, and after that I tried again with:

juju add-machine --debug winrm:Administrator@192.168.11.11

It complained that some registry keys, paths are already there, the jujud user already exist, so executing the provisioning script failes. After fixing the script now, I can provision multiple times the same machine adding and removing it without any problems.

Here's the tricky part:

Some providers don't allow to send scripts that are larger than 16 kbys so the script needs to be tested on Azure,GCE and AWS.

Azure passed, need to test this also on AWS, and GCE.

Member

axw commented Oct 27, 2016

Some providers don't allow to send scripts that are larger than 16 kbys so the script needs to be tested on Azure,GCE and AWS.

Azure passed, need to test this also on AWS, and GCE.

OK, and thanks for the details. Please let us know when this is ready to merge.

Contributor

hoenirvili commented Nov 29, 2016

!!build!!

Contributor

hoenirvili commented Dec 2, 2016

!!build!!

Refactor windows userdata scripts.
Contains modifications in the windows userdata scripts so
it can be run multiple times during provisioning without changing it's final outcome.

Signed-off-by: Salvatore Giulitti <sgiulitti@cloudbasesolutions.com>
Contributor

hoenirvili commented Dec 2, 2016

$$merge$$

Contributor

jujubot commented Dec 2, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit f1585f3 into juju:develop Dec 2, 2016

1 check passed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details

@hoenirvili hoenirvili deleted the hoenirvili:refactor-userdata branch Dec 2, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment