P->O upgrade fails on Property error: resources.NovaLibvirtBase.properties: Property LibvirtTLSPassword not assigned

[Sofer Athlan-Guyot]

All is in the title.

Seen there https://logs.rdoproject.org/00/523700/2/openstack-check/gate-tripleo-ci-centos-7-containers-multinode-upgrades-pike/Z11e0aa15006e4d6885acda643ba1871d/undercloud/home/jenkins/failed_upgrade.log.txt.gz

overcloud.ControllerServiceChain.ServiceChain.37:
  resource_type: OS::TripleO::Services::NovaLibvirt
  physical_resource_id:
  status: CREATE_FAILED
  status_reason: |
    resources[37]: Property error: resources.NovaLibvirtBase.properties: Property LibvirtTLSPassword not assigned

[Jose Luis Franco (jfrancoa)]

From the list of installed packages we can see that tripleo-common comes from delorean, while tripleo-heat-templates is picked from gating-repo:
https://logs.rdoproject.org/00/523700/2/openstack-check/gate-tripleo-ci-centos-7-containers-multinode-upgrades-pike/Z11e0aa15006e4d6885acda643ba1871d/undercloud/var/log/extra/yum-list-installed.txt.gz

openstack-tripleo-common.noarch 7.6.5-0.20171122034430.37f3f45.el7.centos @delorean
openstack-tripleo-heat-templates.noarch 7.0.6-0.20171129085514.cef58e3.el7.centos @gating-repo

Also, the package version for tripleo-common does not include the Depends-On change submitted in the original patch which adds the LibvirtTLSPassword property:

https://review.openstack.org/#/c/522848/

Depends-On: https://review.openstack.org/#/c/522846/ which is 15 commits over the last patch included in the package openstack-tripleo-common.noarch 7.6.5-0.20171122034430.37f3f45.el7.centos, patch 37f3f45 (https://github.com/openstack/tripleo-common/commits/stable/pike).

For that reason, the password is not set and the mistral workflow fails.

[Jose Luis Franco (jfrancoa)]

When I say, "doesn't include the Depends-On", it is a way to refer to the commit which has a dependency to.

My main point is that the installed package openstack-tripleo-common.noarch 7.6.5-0.20171122034430.37f3f45.el7.centos is not including the commit https://review.openstack.org/#/c/522846/, which is needed by https://review.openstack.org/#/c/522848/

[Sofer Athlan-Guyot (sofer-athlan-guyot)]

https://review.openstack.org/524147 tries to re-introduce current-pike.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Athlan-Guyot sofer (<email address hidden>) on branch: master
Review: https://review.openstack.org/524147
Reason: This is for the undercloud, and thus is not useful for this scenario.

[Sofer Athlan-Guyot (sofer-athlan-guyot)]

oki so this is really a ci problem. See the comment in the revived https://review.openstack.org/#/c/524147/ review for more

[OpenStack Infra (hudson-openstack)]

Change abandoned by Athlan-Guyot sofer (<email address hidden>) on branch: master
Review: https://review.openstack.org/524147

[Sofer Athlan-Guyot (sofer-athlan-guyot)]

Fix by using delorean current in the repo config.