opendevreview | OpenStack Proposal Bot proposed openstack/glance master: Imported Translations from Zanata https://review.opendev.org/c/openstack/glance/+/923808 | 03:53 |
---|---|---|
opendevreview | Pranali Deore proposed openstack/python-glanceclient master: DNM: Test glanceclient with new Location APIs https://review.opendev.org/c/openstack/python-glanceclient/+/899453 | 07:53 |
elodilles | mnaser: +2+W'd the patch https://review.opendev.org/c/openstack/glance/+/923310 | 14:30 |
elodilles | mnaser: but we can't merge yet the rest of the patches, | 14:31 |
elodilles | mnaser: as there are failing 'import' jobs as far as i can see | 14:31 |
elodilles | :/ | 14:31 |
elodilles | if anyone has any hint how those failures could be eliminated, then please let me know o:) https://review.opendev.org/c/openstack/glance/+/917473/3?tab=change-view-tab-header-zuul-results-summary | 14:32 |
elodilles | abhishekk wrote that probably those are related to inject metadata plugin, but i haven't got there to figure out the fix (or the related existing patch that needs to be backported to fix these errors) | 14:33 |
abhishekk | He needs to fix nova-ceph-multistore glance tests, other failures there are not related imo | 14:35 |
abhishekk | once those fixed then we can make nova job non voting to get it merged | 14:35 |
abhishekk | abhishekk | 14:36 |
abhishekk | Ok, to fix glance related failures you need to have these two patches | 14:36 |
abhishekk | abhishekk | 14:36 |
abhishekk | https://review.opendev.org/c/openstack/glance/+/923341?usp=search | 14:36 |
abhishekk | abhishekk | 14:36 |
abhishekk | and https://review.opendev.org/c/openstack/tempest/+/923357?usp=search and https://review.opendev.org/q/project:openstack/tempest+owner:akekane@redhat.com+status:merged | 14:36 |
abhishekk | abhishekk | 14:36 |
abhishekk | didn't found any strace related to 'test_nova_image_snapshot_dependency' in glance logs | 14:36 |
abhishekk | https://review.opendev.org/c/openstack/tempest/+/923357?usp=search and https://review.opendev.org/c/openstack/tempest/+/923352?usp=search | 14:37 |
elodilles | test_nova_image_snapshot_dependency is a new test in tempest, so we either override-checkout tempest to zed-last, or exclude that test (i don't know yet if that can be fixed otherwise with any config option or something) | 14:38 |
abhishekk | these two tempest patches and this glance patch is definitely needed, https://review.opendev.org/c/openstack/glance/+/923341?usp=search | 14:38 |
elodilles | the nova-ceph-multistore looks like passing | 14:38 |
abhishekk | no it will fail for sure | 14:38 |
elodilles | ACK, then since on zed we should at some point use zed-last tagged tempest(s) anyway, i guess the necessary workaround is just to exclude that failing test case. I'll sort it out, thanks! | 14:41 |
opendevreview | Cyril Roelandt proposed openstack/python-glanceclient master: Python3.12: do not use ssl.wrap_socket https://review.opendev.org/c/openstack/python-glanceclient/+/922866 | 14:51 |
abhishekk | ack | 15:12 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!