Stop trying to converge during Ceph upgrade command
This had no effect as we only updated the plan and not the
stack. We'll do a normal converge after Ceph upgrade command, which
will also take care of putting back the normal deploy
CephAnsiblePlaybook.
Furthermore it was prepending the converge env file to a list of envs
which already had the *prepare* env file at a latter position (we
reused the env list from the prepare call :) ), so it kept doing
prepare operation instead of converge.
And perhaps most importantly, the messages printed to user are
misleading since no converge was really happening.
Also adds support for --container-registry-file to ceph-upgrade command,
while not necessary, some of the functions called by ceph-upgrade
command expect to receive the container-registry parameters.
Reviewed: https:/ /review. openstack. org/566944 /git.openstack. org/cgit/ openstack/ python- tripleoclient/ commit/ ?id=5d1450a7743 2bc0c4b7d0dbb76 de7efd7d39c56b
Committed: https:/
Submitter: Zuul
Branch: stable/queens
commit 5d1450a77432bc0 c4b7d0dbb76de7e fd7d39c56b
Author: Giulio Fidente <email address hidden>
Date: Wed Apr 25 18:03:11 2018 +0200
Stop trying to converge during Ceph upgrade command
This had no effect as we only updated the plan and not the Playbook.
stack. We'll do a normal converge after Ceph upgrade command, which
will also take care of putting back the normal deploy
CephAnsible
Furthermore it was prepending the converge env file to a list of envs
which already had the *prepare* env file at a latter position (we
reused the env list from the prepare call :) ), so it kept doing
prepare operation instead of converge.
And perhaps most importantly, the messages printed to user are
misleading since no converge was really happening.
Also adds support for --container- registry- file to ceph-upgrade command,
while not necessary, some of the functions called by ceph-upgrade
command expect to receive the container-registry parameters.
Co-Authored-By: Giulio Fidente <email address hidden> f23c789e7fef1a9 e9b49947f66 a06351de2bd43d0 4e8b5e4fb6)
Change-Id: I025eac40f8bda5
Partial-Bug: #1768586
(cherry picked from commit 1612b374d7b6969