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
(MODULES-4241) Enable Windows acceptance testing #201
Conversation
Still a WIP, but getting pretty close. I'm having trouble getting the uninstall working. |
955600d
to
2c8b4fe
Compare
CLA signed by all contributors. |
2c8b4fe
to
4ad9493
Compare
Darn, one scenario still fails. |
4ad9493
to
7bba9a3
Compare
@@ -0,0 +1,9 @@ | |||
Facter.add(:puppet_agent_appdata) do |
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.
Oh you should rebase these changes out of this PR.
@@ -45,6 +45,12 @@ | |||
default => "${::system32}\\cmd.exe" | |||
} | |||
|
|||
if (member($::puppet_agent::service_names, 'puppet')) { |
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.
And these. :)
Ensure the Puppet service is enabled and started on Windows after an upgrade when it's included in the `service` param by setting `PUPPET_AGENT_STARTUP_MODE=Automatic`. Fixes puppetlabs#197.
c54c492
to
c424c95
Compare
Update test setup to work with Windows paths and install behavior.
c424c95
to
a07948e
Compare
Just disabled the whole mcollective section, it needs some work. This should now pass. |
I ran with
|
👍 lgtm |
Update test setup to work with Windows paths and install behavior.
Fixes #197. Depends on #200.