pgbackrest: fix condition for task ssh_keys

no need to configure ssh key-based authentication if "gcs" or "azure" is specified. And update comment for pgbackrest_repo_type variable.
vitabaks · Feb 23, 2023 · 48a7ecc · 48a7ecc
1 parent b64d8b0
commit 48a7ecc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/roles/pgbackrest/tasks/main.yml b/roles/pgbackrest/tasks/main.yml
@@ -104,7 +104,7 @@
 # if pgbackrest_repo_type: "posix" and pgbackrest_repo_host is set
 - import_tasks: ssh_keys.yml
   when:
-    - pgbackrest_repo_type|lower != "s3"
+    - pgbackrest_repo_type|lower == "posix"
     - pgbackrest_repo_host is defined
     - pgbackrest_repo_host | length > 0
   tags: pgbackrest, pgbackrest_ssh_keys

diff --git a/vars/main.yml b/vars/main.yml
@@ -400,7 +400,7 @@ wal_g_patroni_cluster_bootstrap_command: "wal-g backup-fetch {{ postgresql_data_
 pgbackrest_install: false  # or 'true'
 pgbackrest_install_from_pgdg_repo: true  # or 'false'
 pgbackrest_stanza: "stanza_name"  # specify your --stanza
-pgbackrest_repo_type: "posix"  # or "s3"
+pgbackrest_repo_type: "posix"  # or "s3", "gcs", "azure"
 pgbackrest_repo_host: "10.128.64.50"  # dedicated repository host (if repo_type: "posix")
 pgbackrest_repo_user: "postgres"  # if "repo_host" is set
 pgbackrest_conf_file: "/etc/pgbackrest/pgbackrest.conf"