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

Node manager switch #64

Merged
merged 4 commits into from Mar 11, 2020
Merged

Node manager switch #64

merged 4 commits into from Mar 11, 2020

Commits on Mar 9, 2020

  1. Address issue where group B wasn't created when a master replica was …

    …present without a standalone puppetdb replica.
    bwilcox authored and reidmv committed Mar 9, 2020
    Copy the full SHA
    248bb5d View commit details
    Browse the repository at this point in the history
  2. More concise code.

    bwilcox authored and reidmv committed Mar 9, 2020
    Copy the full SHA
    7cf2520 View commit details
    Browse the repository at this point in the history
  3. Iterative fix improvement

    To try and avoid dealing with the complexity of PuppetDB database host
    picking at the end of the manifest, see if having a sane class parameter
    default solves the problem cleanly. If it does, we can do a simple
    switch on replica/no-replica to create the B groups.
    reidmv committed Mar 9, 2020
    Copy the full SHA
    ed7d53b View commit details
    Browse the repository at this point in the history

Commits on Mar 10, 2020

  1. Fix for PDK validate, add parameter comments

    It doesn't like having an optional parameter come before a non-optional
    one. Since parameters are moving around a bit, add some comments to
    clarify their intended usage.
    reidmv committed Mar 10, 2020
    Copy the full SHA
    b74390b View commit details
    Browse the repository at this point in the history