(PE-40691) review updates

Author: davidmalloncares

documentation/recovery/automated_recovery.md

@@ -4,7 +4,7 @@ These instructions provide automated procedures for recovering from select failu
 
 Manual procedures are documented in [recovery.md](recovery.md)
 
-## Recover from failed primary Puppet server
+## Recover from failed Puppet primary server
 
 1. Promote the replica ([official docs](https://puppet.com/docs/pe/2019.8/dr_configure.html#dr-promote-replica))
 2. [Replace missing or failed replica Puppet server](#replace-missing-or-failed-replica-puppet-server)
@@ -13,7 +13,7 @@ Manual procedures are documented in [recovery.md](recovery.md)
 
 This procedure uses the following placeholder references.
 
-* _\<primary-server-fqdn\>_ - The FQDN and certname of the primary Puppet server
+* _\<primary-server-fqdn\>_ - The FQDN and certname of the Puppet primary server
 * _\<replica-postgres-server-fqdn\>_ - The FQDN and certname of the PE-PostgreSQL server which resides in the same availability group as the replacement replica Puppet server
 * _\<replacement-replica-fqdn\>_ - The FQDN and certname of the replacement replica Puppet server
 
@@ -28,12 +28,12 @@ This procedure uses the following placeholder references.
 
 ## Replace failed PE-PostgreSQL server (A or B side)
 
-The procedure for replacing a failed PE-PostgreSQL server is the same regardless of which PE-PostgreSQL server is missing or if the name of the PE-PostgreSQL server is the same or different. This procedure uses the following placeholder references.
+The procedure for replacing a failed PE-PostgreSQL server is the same regardless of which PE-PostgreSQL server is missing or whether the name of the PE-PostgreSQL server is the same or different. This procedure uses the following placeholder references.
 
 * _\<replacement-postgres-server-fqdn\>_ - The FQDN and certname of the new server being brought in to replace the failed PE-PostgreSQL server
 * _\<working-postgres-server-fqdn\>_ - The FQDN and certname of the still-working PE-PostgreSQL server
 * _\<failed-postgres-server-fqdn\>_ - The FQDN and certname of the failed PE-PostgreSQL server
-* _\<primary-server-fqdn\>_ - The FQDN and certname of the primary Puppet server
+* _\<primary-server-fqdn\>_ - The FQDN and certname of the Puppet primary server
 * _\<replica-server-fqdn\>_ - The FQDN and certname of the replica Puppet server
 
 Procedure:
@@ -47,11 +47,11 @@ Procedure:
                 failed_postgresql_host=<failed-postgres-server-fqdn> \
                 replacement_postgresql_host=<replacement-postgres-server-fqdn>
 
-## Replace failed replica puppet server AND failed replica pe-postgresql server
+## Replace failed replica Puppet server AND failed replica PE-PostgreSQL server
 
 This procedure uses the following placeholder references.
 
-* _\<primary-server-fqdn\>_ - The FQDN and certname of the primary Puppet server
+* _\<primary-server-fqdn\>_ - The FQDN and certname of the Puppet primary server
 * _\<failed-replica-fqdn\>_ - The FQDN and certname of the failed replica Puppet server
 
 1. Ensure the old replica server is forgotten.
@@ -68,7 +68,7 @@ This procedure uses the following placeholder references.
 * _\<avail-group-letter\>_ - Either A or B; whichever of the two letter designations the compiler is being assigned to
 * _\<compiler-hosts\>_ - A comma-separated list of FQDN and certname of the new compiler(s)
 * _\<dns-alt-names\>_ - A comma-separated list of DNS alt names for the compiler
-* _\<primary-server-fqdn\>_ - The FQDN and certname of the primary Puppet server
+* _\<primary-server-fqdn\>_ - The FQDN and certname of the Puppet primary server
 * _\<postgresql-server-fqdn\>_ - The FQDN and certname of the PE-PostgreSQL server with availability group _\<avail-group-letter\>_
 
 Procedure:
@@ -82,14 +82,14 @@ Procedure:
                 dns_alt_names=<dns-alt-names> \
                 primary_postgresql_host=<postgresql-server-fqdn>
 
-Please note, the optional parameters and values of the plan are as follows.
+Please note, the optional parameters and values of the plan are as follows:
 
 <!-- table -->
 
 | Parameter                 | Default value | Description                                                                                                                    |
 | ------------------------- | ------------- | ------------------------------------------------------------------------------------------------------------------------------ |
 | `avail_group_letter`      | `A`           | By default, each compiler will be added to the primary group A.                                                                |
 | `dns_alt_names`           | `undef`       |                                                                                                                                |
-| `primary_postgresql_host` | `undef`       | By default, this will pre-populate to the required value depending if your architecture contains HA and or external databases. |
+| `primary_postgresql_host` | `undef`       | By default, this will pre-populate to the required value depending on whether your architecture contains HA and or external databases. |
 
 For more information around adding compilers to your infrastructure [Expanding Your Deployment](expanding.md#adding-compilers-with-peadmadd_compiler)

documentation/recovery/recovery.md

@@ -6,7 +6,7 @@ The new system needs to be provisioned with the same certificate name as the sys
 
 Automated procedures are documented in [automated_recovery.md](automated_recovery.md)
 
-## Recover from failed primary Puppet server
+## Recover from failed Puppet primary server
 
 1. Promote the replica ([official docs](https://puppet.com/docs/pe/2019.8/dr_configure.html#dr-promote-replica))
 2. Purge the failed primary server
@@ -20,7 +20,7 @@ Automated procedures are documented in [automated_recovery.md](automated_recover
 
 This procedure uses the following placeholder references.
 
-* _\<primary-server-fqdn\>_ - The FQDN and certname of the primary Puppet server
+* _\<primary-server-fqdn\>_ - The FQDN and certname of the Puppet primary server
 * _\<old-replica-fqdn\>_ - The FQDN and certname of the old replica Puppet server that has failed or is missing
 * _\<replacement-replica-fqdn\>_ - The FQDN and certname of the new replica Puppet server
 * _\<replacement-avail-group-letter\>_ - Either A or B; whichever of the two letter designations is appropriate for the replacement server. It will be the opposite of the primary server. 
@@ -83,7 +83,7 @@ The procedure for replacing a failed PE-PostgreSQL server is the same regardless
 * _\<replacement-postgres-server-fqdn\>_ - The FQDN and certname of the new server being brought in to replace the failed PE-PostgreSQL server
 * _\<working-postgres-server-fqdn\>_ - The FQDN and certname of the still-working PE-PostgreSQL server
 * _\<replacement-avail-group-letter\>_ - Either A or B; whichever of the two letter designations is appropriate for the server being replaced. It will be the opposite of the still-working PE-PostgreSQL server
-* _\<primary-server-fqdn\>_ - The FQDN and certname of the primary Puppet server
+* _\<primary-server-fqdn\>_ - The FQDN and certname of the Puppet primary server
 
 Procedure:
 
@@ -202,7 +202,7 @@ This procedure uses the following placeholder references.
 * _\<avail-group-letter\>_ - Either A or B; whichever of the two letter designations the compiler is being assigned to
 * _\<new-compiler-fqdn\>_ - The FQDN and certname of the new compiler
 * _\<dns-alt-names\>_ - A comma-separated list of DNS alt names for the compiler
-* _\<primary-server-fqdn\>_ - The FQDN and certname of the primary Puppet server
+* _\<primary-server-fqdn\>_ - The FQDN and certname of the Puppet primary server
 * _\<postgresql-server-fqdn\>_ - The FQDN and certname of the PE-PostgreSQL server with availability group _\<avail-group-letter\>_
 
 1. On _\<postgresql-server-fqdn\>_: