diff --git a/roles/restore_backups/tasks/main.yml b/roles/restore_backups/tasks/main.yml index 8767b4d..4d6865f 100644 --- a/roles/restore_backups/tasks/main.yml +++ b/roles/restore_backups/tasks/main.yml @@ -1,8 +1,8 @@ --- # This role will attempt to fetch and load the backup manifest from a -# remote HTTP or S3 URL. On successful completion the contents of JSON -# or YAML document will be in the var `atl_backup_manifest`. +# remote S3 URL. On successful completion the contents of JSON or YAML +# document will be in the var `atl_backup_manifest`. # # PREREQUISITES: # * `atl_backup_manifest_url` points at the manifest. @@ -12,6 +12,9 @@ # # NOTE: The actual DB/FS restore operations could potentially be split # out into discrete roles, but currently that is not required. +# +# TODO: Support HTTPS with authentication. Deferred until after the +# initial testing release. - block: @@ -43,12 +46,6 @@ dest: "{{ atl_backup_manifest_dest }}" when: atl_backup_manifest_url.scheme == 's3' - - name: Fetch the manifest from remote host - get_url: - url: "{{ atl_backup_manifest_url }}" - dest: "{{ atl_backup_manifest_dest }}" - when: atl_backup_manifest_url.scheme != 's3' - - name: Load parameters from manifest include_vars: file: "{{ atl_backup_manifest_dest }}"