Comment 1 for bug 1933448

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

I posted a test fix at [1] so when featureset_override is provided with a delorean hash it is used directly for the md5.

I am trying to test with [2] except it isn't included with depends-on. It is ci-config and besides the fact that delorean probably can't build the ci-config repo (? I cannot see any related distgit @ https://github.com/rdo-packages?q=ci-config&type=&language=&sort= ), the conditional fails there [3] because zuul.override_repo is stable/ussuri for the periodic-tripleo-ci-centos-8-standalone-on-multinode-ipa-ussuri job running in my testproject e.g. logs @ [4]

Not sure how to proceed... I have considered if we can enhance build-test-packages to apply changes from git directly to a checkout? Even if we do want to do that I don't think it is somehting that can be done in this ruck|rover cycle and needs discussion etc.

We could try merging https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34275 - the conditional there means it only happens when user specifies the hash. It might fix the issue, or we may need something further which we can iterate for.

[1] https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34275/1/ci-scripts/infra-setup/roles/get_hash/tasks/get_hash.yaml
[2] https://review.rdoproject.org/r/c/testproject/+/34256
[3] https://opendev.org/openstack/tripleo-quickstart-extras/src/commit/e423bb068a96ed919ead08226f9447b2cdfc0332/roles/build-test-packages/tasks/main.yml#L200
[4] https://logserver.rdoproject.org/56/34256/3/check/periodic-tripleo-ci-centos-8-standalone-on-multinode-ipa-ussuri/7385797/zuul-info/inventory.yaml