-
Notifications
You must be signed in to change notification settings - Fork 54
(PE-37235) Verify and update documentation on add_compiler #429
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
Merged
Merged
Changes from 8 commits
Commits
Show all changes
10 commits
Select commit
Hold shift + click to select a range
4b49cd5
PE-37235 Verify and update documentation on add_compiler if needed an…
cathal41 49c309b
@cathal41
cathal41 495ec7a
Update plans/add_compiler.pp
cathal41 b9d3090
Update documentation/add_compiler.md
cathal41 c22656e
Update documentation/add_compiler.md
cathal41 be239b3
Update documentation/add_compiler.md
cathal41 3bfa284
Update documentation/add_compiler.md
cathal41 532f434
Update documentation/add_compiler.md
cathal41 869fc70
Updating add compiler docs to show optional params
043a3cd
Updating docs with text corrections
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
# Add Compiler | ||
|
||
The `peadm::add_compiler` plan can be used to add a new compiler to a PE cluster or replace an existing one with new configuration. | ||
cathal41 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
## Add a compiler to an existing PE cluster | ||
cathal41 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
Prepare to run the plan using a params.json file such as this one: | ||
cathal41 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
```json | ||
{ | ||
"avail_group_letter": "A", | ||
"compiler_host": "pe-xl-compiler-0.lab1.puppet.vm", | ||
"dns_alt_names": [ "puppet", "puppet.lab1.puppet.vm" ], | ||
"primary_host": "pe-xl-core-0.lab1.puppet.vm", | ||
"primary_postgresql_host": "pe-psql-6251cd-0.us-west1-a.c.slice-cody.internal", | ||
ragingra marked this conversation as resolved.
Show resolved
Hide resolved
|
||
} | ||
``` | ||
|
||
See the [install](install.md#reference-architectures) documentation for a list of supported architectures. | ||
cathal41 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
|
||
## Running the add_compiler plan | ||
``` | ||
bolt plan run peadm::add_compiler --params @params.json | ||
``` | ||
|
||
This command will retrieve the current PEADM configuration to determine the setup rules needed for a compiler's secondary PuppetDB instances. The plan will configure the primary with appropriate rules for allowing access from the new compiler. On the primary, the `puppet` service is stopped and the `pe-postgresql` service is reloaded. If required, a puppet agent is be installed. The compiler agent's certificate is be regenerated to include required data with `peadm::subplans::component_install`. Puppet agent will run on the following components | ||
cathal41 marked this conversation as resolved.
Show resolved
Hide resolved
ragingra marked this conversation as resolved.
Show resolved
Hide resolved
|
||
* _\<compiler-host\>_ | ||
* _\<primary_postgresql_host\>_ | ||
* _\<replica host\>_ | ||
* _\<primary_postgresql_host\>_ | ||
cathal41 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
The `puppet` service is then restarted. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.