| opendevreview | Rabi Mishra proposed openstack/heat stable/wallaby: Don't use master constraints for stable branches https://review.opendev.org/c/openstack/heat/+/802986 | 03:56 |
|---|---|---|
| ramishra | tkajinam: ^^ let's see how this works.. | 03:57 |
| tkajinam | ramishra, thanks | 03:57 |
| tkajinam | ramishra, my only concern is that upper constraints might break tempest and heat-tempest-plugin, but let's see how it goes | 03:58 |
| tkajinam | in that case we can pin tempest | 03:58 |
| opendevreview | Rabi Mishra proposed openstack/heat stable/wallaby: WIP Don't use master constraints for stable branches https://review.opendev.org/c/openstack/heat/+/802986 | 06:54 |
| *** kleini_ is now known as kleini | 08:49 | |
| ramishra | gmann: Hey! All heat stable branches are broken after https://github.com/openstack/python-cinderclient/commit/3502a5591a654ae57741c6738994ffa9d8457696#diff-278ab3e257fee8afad8352ebc46f85ea585a3ca1e610c191c0902dbf1fbc383f | 13:00 |
| ramishra | I think that's because tempest uses latest python-cinderclient everywhere from train onwards | 13:01 |
| ramishra | We can't use latest cinderclient in stable branches. Is there a way to use relevant uc for the branch? I tried with https://review.opendev.org/c/openstack/heat/+/802986 which does not seem to work | 13:02 |
| ramishra | ricolin_: ^^ | 13:02 |
| -opendevstatus- NOTICE: There will be a brief outage of the Gerrit service on review.opendev.org starting at 15:00 UTC today as part of a routine project rename maintenance: http://lists.opendev.org/pipermail/service-announce/2021-July/000023.html | 14:10 | |
| -opendevstatus- NOTICE: There will be a brief outage of the Gerrit service on review.opendev.org in the next few minutes as part of a routine project rename maintenance: http://lists.opendev.org/pipermail/service-announce/2021-July/000023.html | 14:59 | |
| gmann | ramishra: we do not use python-cinderclient in Tempest, tempest has its own service clients to call API | 15:39 |
| gmann | ramishra: checking the failure | 15:39 |
| gmann | ramishra: tis heat using cinder v2 still, tkajinam has fixes up https://review.opendev.org/q/topic:%2522volumev2-removal%2522+(status:open+OR+status:merged)+projects:openstack/heat | 15:48 |
| gmann | *its | 15:48 |
| gmann | ramishra: in stable branch I think, python-cinderclient old version should be used as heat test are mixed with tempest tests and python-client calling tests. | 15:49 |
| gmann | tempest tests are all fine for cinder v2 removal | 15:50 |
| gmann | ramishra: seems like it is complex issue and mixing the both different type of tests needs to split into separate job, i commented on your patch, please check if that make sense https://review.opendev.org/c/openstack/heat/+/802986/2/roles/run-heat-tests/tasks/main.yaml#13 | 17:13 |
| *** kleini_ is now known as kleini | 21:16 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!