diff --git a/common/container-puppet.sh b/common/container-puppet.sh index c7d5a289e0..2d37877e8b 100644 --- a/common/container-puppet.sh +++ b/common/container-puppet.sh @@ -160,9 +160,9 @@ if [ -z "$NO_ARCHIVE" ]; then excluded_original_passwords+=" --exclude=/var/lib/config-data/*${p}" fi done - # We need to exclude the swift ring backups as those change over time and - # containers do not need to restart if they change - EXCLUDE=--exclude='*/etc/swift/backups/*'\ --exclude='*/etc/libvirt/passwd.db'\ ${excluded_original_passwords} + # We need to exclude the swift rings and backups as those change over time + # and containers do not need to restart if they change + EXCLUDE=--exclude='*/etc/swift/backups/*'\ --exclude='*/etc/swift/*.ring.gz'\ --exclude='*/etc/swift/*.builder'\ --exclude='*/etc/libvirt/passwd.db'\ ${excluded_original_passwords} # We need to repipe the tar command through 'tar xO' to force text # output because otherwise the sed command cannot work. The sed is diff --git a/deployment/swift/swift-ringbuilder-container-puppet.yaml b/deployment/swift/swift-ringbuilder-container-puppet.yaml index 206a09c185..cb6574e718 100644 --- a/deployment/swift/swift-ringbuilder-container-puppet.yaml +++ b/deployment/swift/swift-ringbuilder-container-puppet.yaml @@ -65,6 +65,12 @@ parameters: default: '' description: A temporary Swift URL to upload rings to. type: string + DeployIdentifier: + default: '' + type: string + description: > + Setting this to a unique value will re-run any deployment tasks which + perform configuration on a Heat stack-update. conditions: swift_use_local_dir: @@ -124,6 +130,10 @@ outputs: volumes: - /var/lib/config-data/puppet-generated/swift/etc/swift:/etc/swift:rw,z - /var/lib/config-data/swift_ringbuilder:/swift_ringbuilder:ro + environment: + # NOTE: this should force this container to re-run on each + # update (scale-out, etc.) + TRIPLEO_DEPLOY_IDENTIFIER: {get_param: DeployIdentifier} external_upgrade_tasks: - when: - step|int == 1 diff --git a/deployment/swift/swift-storage-container-puppet.yaml b/deployment/swift/swift-storage-container-puppet.yaml index 34b352a893..63ab3f1c8d 100644 --- a/deployment/swift/swift-storage-container-puppet.yaml +++ b/deployment/swift/swift-storage-container-puppet.yaml @@ -77,6 +77,12 @@ parameters: default: 0 description: Number of workers for Swift account service. type: string + DeployIdentifier: + default: '' + type: string + description: > + Setting this to a unique value will re-run any deployment tasks which + perform configuration on a Heat stack-update. # DEPRECATED options for compatibility with overcloud.yaml # This should be removed and manipulation of the ControllerServices list @@ -311,6 +317,10 @@ outputs: command: ['chown', '-R', 'swift:', '/srv/node'] volumes: - /srv/node:/srv/node:z + environment: + # NOTE: this should force this container to re-run on each + # update (scale-out, etc.) + TRIPLEO_DEPLOY_IDENTIFIER: {get_param: DeployIdentifier} step_4: map_merge: - if: