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

(PE-14934) Test if puppetdb is up when running puppet agent on pdb node #2

Merged
merged 1 commit into from
Apr 4, 2016

Conversation

demophoon
Copy link
Contributor

Before this commit we were running into an issue where puppetdb would
sometimes not be up and running after puppet agent restarted the
service. This commit waits for the puppetdb service to be up after
running puppet agent on the database node so that the next agent run
doesn't fail.

Before this commit we were running into an issue where puppetdb would
sometimes not be up and running after puppet agent restarted the
service. This commit waits for the puppetdb service to be up after
running puppet agent on the database node so that the next agent run
doesn't fail.
@jpartlow
Copy link
Contributor

jpartlow commented Apr 4, 2016

Have you had a chance to do a test run with this?

And what about the upgrade case?

@puppetlabs-jenkins
Copy link
Contributor

Can one of the admins verify this patch?

@demophoon
Copy link
Contributor Author

Attempting to run this right now, just getting eyes on it early would be nice

@jpartlow
Copy link
Contributor

jpartlow commented Apr 4, 2016

Looks like upgrade_pe_on eventually hands to do_install as well.

@demophoon
Copy link
Contributor Author

Frankenbuilder running now with the changes in this PR patched into beaker on ubuntu-14.04-amd64.

@tvpartytonight
Copy link
Contributor

@puppetlabs-jenkins test this please

@jpartlow
Copy link
Contributor

jpartlow commented Apr 4, 2016

👍

@demophoon
Copy link
Contributor Author

Frankenbuilder passed \o/

@tvpartytonight
Copy link
Contributor

👍

@tvpartytonight
Copy link
Contributor

mario pacing

@tvpartytonight tvpartytonight merged commit 4e0b668 into puppetlabs:master Apr 4, 2016
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