Comment 2 for bug 1990012

Revision history for this message
Marios Andreou (marios-b) wrote (last edit ):

...found some time to dig into this some more.

The problem is definitely at [1] where based on the ansible_distribution_major_version of the node running that code (undercloud) we query the relevant dlrn endpoint and fetch the hash from it.

In the 8-9 job, we need to report for centos8, but the node creating the hash_info and reporting to delorean is centos9.

One relatively simple way to fix it is to just override 'distro' at [2]. I have proposed [3] to do that. The problem is that [3] is in a config repo so we cannot test it before merging.

I propose that we merge [3] and I'll run testproject for those jobs as soon as it lands. Depending on the result we can keep it or revert it immediately.

[1] https://github.com/rdo-infra/ci-config/blob/43d194a98c0aafe40559706d85a538bdf60e9dff/ci-scripts/infra-setup/roles/get_hash/defaults/main.yml#L51
[2] https://github.com/rdo-infra/review.rdoproject.org-config/blob/cb5636cf022718bc2f612c49a22d51269d4bfaae/playbooks/tripleo-ci-periodic-base/pre.yaml#L7-L9
[3] https://review.rdoproject.org/r/c/config/+/45259