| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: DNM: test bookworm https://review.opendev.org/c/openstack/devstack/+/887547 | 05:18 |
|---|---|---|
| lucasagomes | Hi folks, can some devstack stable branch maintainer take a quick look at https://review.opendev.org/c/openstack/devstack/+/887184 . Without this the neutron stable/wallaby branch is broken | 12:26 |
| lucasagomes | ihrachys, ^ just a fyi | 12:26 |
| ihrachys | lucasagomes why did it break | 13:59 |
| lucasagomes | ihrachys, cause the --branch from clone does not accept a commit hash | 14:04 |
| lucasagomes | ihrachys, and in neutron-temptest we have a hash for OVN_BRANCH | 14:05 |
| lucasagomes | this patch fixes it | 14:05 |
| lucasagomes | I guess we could also change neutron-tempest | 14:05 |
| lucasagomes | but since this was already uploaded and merged on other branches, it made sense to also get it in stable/wallaby | 14:05 |
| lucasagomes | https://paste.opendev.org/show/bg9ZuR5nbrEmnNsVbYRp/ | 14:06 |
| frickler | the problem with that patch is that it introduces a regression because GIT_DEPTH no longer works and until we have a fix for that, I'm hesitating to do more backports of it | 15:09 |
| frickler | I'm also not sure that changing the OVN version that is deployed is a good idea for old stable branches | 15:10 |
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: DNM: test bookworm https://review.opendev.org/c/openstack/devstack/+/887547 | 15:12 |
| lucasagomes | frickler, ack, I will propose a change to the OVN_BRANCH in neutron-tempest then | 15:37 |
| lucasagomes | to use a tag instead of the commit hash or something | 15:37 |
| ihrachys | I think maybe there's already an ovn release that includes the necessary stateless ACL change that spurred the pin to commit in the first place and this may be unneeded now | 15:42 |
| ihrachys | just need to check what ovn released recently | 15:42 |
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: DNM: test bookworm https://review.opendev.org/c/openstack/devstack/+/887547 | 15:48 |
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: DNM: test bookworm https://review.opendev.org/c/openstack/devstack/+/887547 | 16:53 |
| opendevreview | Merged openstack/tempest master: Add tests to list hidden images https://review.opendev.org/c/openstack/tempest/+/884610 | 22:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!