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

Docker updates #98

Merged
merged 6 commits into from
May 28, 2020
Merged

Docker updates #98

merged 6 commits into from
May 28, 2020

Conversation

logicminds
Copy link
Contributor

These changes fix a a few minor annoyances I have found so far when operating the docker example containers.

I also made some speed enhancements with the sign_csr.rb task so it would just load Puppet and use the settings instead of shelling out multiple times.

  * When running puppetdb queries in the container it is helpful to
    have the the config file setup already. This commit configures
    the puppetdb.conf file on the puppet servers.
  * The docker container for bolt was having some encoding issues
    after running a plan.  This fixes that issue so the plan/command
    does not fail.

  * Only applies to the example docker containers
  * This only applies to our example docker containers.
    If the docker host reboots, the containers will now
    restart upon reboot. Was just too annoying before to always
    have to start up the machines again.
@logicminds logicminds requested a review from a team as a code owner May 27, 2020 23:45
  * previously if a puppet 5 agent had already submitted a CSR
    the submit_csr task would fail.  This fixes that by checking
    if the host making the request has a cert already.

    Note this does not verify the cert is actually valid but only if the
    cert exists or not.  In puppet 6 we use ssl verify to do actual
    verification.
  * Specifies how to start a container so it can use the PE stack.
@reidmv reidmv merged commit 1622d61 into puppetlabs:master May 28, 2020
@logicminds logicminds deleted the docker_updates branch June 29, 2020 19:28
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.

2 participants