| ralonsoh | hi folks, if you don't mind, please check https://review.opendev.org/c/openstack/devstack/+/887790. That will unblock the Neutron CI in Wallaby | 10:01 |
|---|---|---|
| ralonsoh | thanks! | 10:01 |
| ralonsoh | BTW, tested (and working) in https://review.opendev.org/c/openstack/neutron/+/887791 | 10:02 |
| ykarel | frickler, dansmith hi can you please check the backport https://review.opendev.org/c/openstack/devstack/+/887790 | 12:18 |
| opendevreview | sean mooney proposed openstack/devstack-plugin-ceph stable/2023.1: Enable validation and disable block-migration https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/887879 | 12:44 |
| ykarel | sean-k-mooney, Thanks for ^ but nova-ceph-multistore not inheriting that job but instead devstack-plugin-ceph-tempest-py3 | 12:58 |
| ykarel | so wondering how that's related | 12:59 |
| sean-k-mooney | oh well its the same problem basically the tempest patch that made validation a hard dep should have updated these other jobs before it was merged | 12:59 |
| sean-k-mooney | we shoudl be able ot copy paste the validation enabling ot the base ceph job | 13:00 |
| ykarel | ohkk master jobs have it via https://opendev.org/openstack/devstack-plugin-ceph/commit/41b6a8c227190a9b52d29a078425321f96240d92 | 13:04 |
| ykarel | so for this backport case we would need only validation enable part in base job | 13:04 |
| sean-k-mooney | that has some other changes in it too. | 13:10 |
| sean-k-mooney | for the cephadm supprot which renames some jobs | 13:10 |
| sean-k-mooney | so i dont think we can backprot that as is | 13:10 |
| ykarel | yeap we would just need validation enable part | 13:10 |
| sean-k-mooney | elodilles: ^ should we do a stable only patch with jsut the | 13:10 |
| sean-k-mooney | TEMPEST_RUN_VALIDATION: false | 13:10 |
| sean-k-mooney | TEMPEST_RUN_VALIDATION: True | 13:11 |
| sean-k-mooney | MYSQL_REDUCE_MEMORY: True | 13:11 |
| sean-k-mooney | in the base job | 13:11 |
| sean-k-mooney | i actully think the chagne in the base job are all ok to backport so lines 21-28 | 13:12 |
| sean-k-mooney | but the rest likely shoud be dropped. | 13:12 |
| sean-k-mooney | so a stable only patch refernce this change id might be the best way to go | 13:12 |
| elodilles | sean-k-mooney: no need for "stable-only" tag, just drop the unrelevant parts and add a "note" to the commit message that it is a partial backport | 13:14 |
| sean-k-mooney | ok ill do that now | 13:14 |
| sean-k-mooney | will i abandon the other backport since if we do it in the base job that is not needed or put this on top of that | 13:15 |
| sean-k-mooney | do we also need https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/881764 | 13:25 |
| sean-k-mooney | i dont know if that is branched or branchless | 13:26 |
| sean-k-mooney | looks like its branchless so no i can remove that depends on | 13:27 |
| opendevreview | sean mooney proposed openstack/devstack-plugin-ceph stable/2023.1: [Partial Backport] Revert "Temporary pin the ceph jobs nodeset to Focal" https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/887955 | 13:30 |
| sean-k-mooney | ok i think something like ^ should work i updated the depends-on for the nova-lvm patch too | 13:31 |
| sean-k-mooney | oh actully no we might need both becasue we still need to disable block migration | 13:36 |
| opendevreview | sean mooney proposed openstack/devstack-plugin-ceph stable/2023.1: Enable validation and disable block-migration https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/887879 | 13:42 |
| sean-k-mooney | i need to swap to some downstream stuff but ill check the zuul results later today | 13:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!