PE-38801 Fixed formatting of rules in warning message

Author: AaronShannon

plans/convert.pp

@@ -261,8 +261,9 @@
     # the existing groups are correct enough to function until the upgrade is
     # performed.
     if (versioncmp($pe_version, '2019.7.0') >= 0) {
-      out::message('WARNING: The following existing rules on the PE Infrastructure Agent group will be overwritten with default values:')
-      run_task('peadm::get_group_rules', $primary_target)
+      $rules = run_task('peadm::get_group_rules', $primary_target).first.value['_output']
+      $rules_formatted = stdlib::to_json_pretty(parsejson($rules))
+      out::message("WARNING: The following existing rules on the PE Infrastructure Agent group will be overwritten with default values:\n ${rules_formatted}")
 
       apply($primary_target) {
         class { 'peadm::setup::node_manager_yaml':

plans/upgrade.pp

@@ -326,8 +326,9 @@
       default                                      => $primary_postgresql_target.peadm::certname(),
     }
 
-    out::message('WARNING: The following existing rules on the PE Infrastructure Agent group will be overwritten with default values:')
-    run_task('peadm::get_group_rules', $primary_target)
+    $rules = run_task('peadm::get_group_rules', $primary_target).first.value['_output']
+    $rules_formatted = stdlib::to_json_pretty(parsejson($rules))
+    out::message("WARNING: The following existing rules on the PE Infrastructure Agent group will be overwritten with default values:\n ${rules_formatted}")
 
     apply($primary_target) {
       class { 'peadm::setup::node_manager_yaml':