| *** dviroel|rover|afk is now known as dviroel|out | 00:00 | |
| opendevreview | MaAoyu proposed openstack/cinder master: Signature of base method in class 'AccelBase' https://review.opendev.org/c/openstack/cinder/+/814100 | 03:00 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/813746 | 06:30 |
| opendevreview | kourosh vivan proposed openstack/cinder master: [ubuntu] Add tgt package for cinder storage https://review.opendev.org/c/openstack/cinder/+/814129 | 10:46 |
| *** dviroel|out is now known as dviroel|rover | 11:41 | |
| *** dviroel|rover is now known as dviroel|afk | 12:54 | |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: Use file locks in connectors https://review.opendev.org/c/openstack/os-brick/+/814139 | 12:55 |
| simondodsley | Can I get a final +2 and +W for this patch: https://review.opendev.org/c/openstack/cinder/+/812210 | 13:49 |
| simondodsley | rosmaita, is there a known issue with cinder in devstack? CI is failing with suds-jurko and oslo_vmware conflicts | 13:55 |
| rosmaita | simondodsley: yes, but i thought that was only when running on fedora | 13:55 |
| rosmaita | there's a patch up changing oslo.vmware to use "suds-community", and then a new version has to be released, and then the u-c need to be updated | 13:56 |
| rosmaita | not sure where all that is ATM | 13:56 |
| simondodsley | rosmaita, i'm using ubuntu 20.04 | 13:59 |
| rosmaita | well, hopefully the fix for fedora hasn't broken focal | 14:00 |
| simondodsley | fingers crossed on that one... | 14:00 |
| rosmaita | reminder: festival of XS reviews at 1400 UTC | 14:02 |
| rosmaita | join us in meetpad: https://meetpad.opendev.org/cinder-festival-of-reviews | 14:02 |
| rosmaita | direct link to the etherpad: https://etherpad.opendev.org/p/cinder-festival-of-reviews | 14:02 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna ^^ | 14:02 |
| jungleboyj | rosmaita: Thanks. Will join if the meeting I am in wraps up early. | 14:03 |
| simondodsley | rosmaita, you can see the failure here: http://openstack-logs.purestorage.com/84/813984/2/thirdparty-check/pure-devstack-cinder-tempest-fc-aio/8e79ee6/job-output.txt.gz. Search for `suds` | 14:04 |
| rosmaita | simondodsley: looks like oslo.vmware 3.10.0 was released about an hour ago: https://review.opendev.org/c/openstack/releases/+/814012 | 14:07 |
| rosmaita | will need to be included in upper-constraints | 14:07 |
| rosmaita | https://review.opendev.org/c/openstack/requirements/+/814148 | 14:08 |
| simondodsley | rosmaita, i don't think that will solve it as the suds-jurko version is tied to 0.6 and the new oslo-vmware requires higher than that | 14:12 |
| rosmaita | simondodsley: i believe the new oslo.vmware doesn't use suds-jurko at all | 14:13 |
| rosmaita | it uses "suds-community" instead | 14:13 |
| simondodsley | ok - i'll watch it and see if my CI is happy after it merges | 14:14 |
| rosmaita | simondodsley: sounds good | 14:14 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Increase min version of oslo.vmware to 3.10.0 https://review.opendev.org/c/openstack/cinder/+/814174 | 14:23 |
| enriquetaso | Replace suds-jurko with suds-community: https://review.opendev.org/c/openstack/oslo.vmware/+/813377 | 14:27 |
| ecsantos[m] | about the suds-jurko conflict issue: oslo.vmware needs a new release with the fix [1], and it's already proposed and approved [2], it only needs its constraints to be updated [3] and it's probably good to go. thanks for fungi and Clark from opendev for this info.... (full message at https://matrix.org/_matrix/media/r0/download/matrix.org/jYVqMhPpTlOTfnbEdxEPRyxs) | 14:45 |
| ecsantos[m] | if it's useful to you guys. I believe cinder was affected by this issue as well | 14:45 |
| ecsantos[m] | oh, rosmaita already linked to those, sorry! :) | 14:46 |
| fungi | yeah, and i've asked prometheanfire to expedite approval on the constraints change for it, but can approve it in a pinch if nobody else with requirements-core perms is available | 14:47 |
| rosmaita | fungi: ty, looks like simondodsley has hit that issue with ubuntu focal | 14:51 |
| rosmaita | so i guess it's not just a fedora 34 problem | 14:51 |
| ecsantos[m] | I hit the issue on focal as well | 14:53 |
| fungi | it's an everywhere problem when setuptools>=58 is used to install suds-jurko from the source tarball on pypi | 14:55 |
| fungi | most of opendev's jobs were spared because we pre-build wheels and had some still cached which had been built with an older version of setuptools | 14:56 |
| fungi | the reason fedora-34 jobs were hitting it is that we added the platform and wheel builder job for it after the setuptools 58.0.0 release | 14:56 |
| fungi | the way pip works is that it will seek out a compiled (wheel) package suitable for your platform first, then if it finds none it will retrieve the source package instead and attempt to build a compiled package from it to install. the problem is that suds-jurko (and many other ancient, unmaintained packages on pypi) were designed to be built with a transitional feature of setuptools | 15:01 |
| fungi | which was eventually deprecated and finally removed in 58.0.0, making it no longer possible to install those from pypi if you have the current setuptools version | 15:01 |
| fungi | we try to avoid making most jobs which run in opendev build python packages from source, because some of them take a really long time and waste quota which could go toward running other jobs, so we pre-build wheels for our test platforms if suitable ones aren't already available on pypi, and put those in a cache which we configure pip on our job nodes to look in when trying to decide | 15:03 |
| fungi | what to install | 15:03 |
| fungi | but any systems without a similar cache and wheels of thinks like suds-jurko built prior to setuptools 58 are now thoroughly broken | 15:12 |
| fungi | s/thinks/things/ | 15:13 |
| rosmaita | fungi: thanks for the explanation | 15:13 |
| opendevreview | Merged openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/813746 | 15:14 |
| opendevreview | Ade Lee proposed openstack/cinder master: WIP/DNM: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 15:24 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Fix: Race between attachment and volume deletion https://review.opendev.org/c/openstack/cinder/+/801913 | 15:47 |
| *** dviroel|afk is now known as dviroel|rover | 15:52 | |
| rosmaita | Looks like the festival is over! Thanks for participating! | 16:02 |
| opendevreview | Merged openstack/cinder master: mypy: update retry decorator in utils.py https://review.opendev.org/c/openstack/cinder/+/805077 | 16:15 |
| opendevreview | Merged openstack/cinder master: Raise min version of ddt https://review.opendev.org/c/openstack/cinder/+/812518 | 16:15 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: Use file locks in connectors https://review.opendev.org/c/openstack/os-brick/+/814139 | 16:20 |
| opendevreview | Carlos Eduardo proposed openstack/devstack-plugin-ceph master: Fix Ganesha repository broken urls https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/813446 | 16:33 |
| simondodsley | Can we get https://review.opendev.org/c/openstack/cinder/+/814174 merged? The main requirements patch has now merged. We need this for all 3rd party devstack CIs | 16:54 |
| opendevreview | Merged openstack/python-cinderclient master: Remove the unused tool scripts https://review.opendev.org/c/openstack/python-cinderclient/+/788121 | 17:00 |
| opendevreview | Ade Lee proposed openstack/cinder master: WIP/DNM: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 17:00 |
| eharney | simondodsley: done | 17:01 |
| simondodsley | eharney, thanks | 17:04 |
| opendevreview | Merged openstack/cinder master: Increase min version of oslo.vmware to 3.10.0 https://review.opendev.org/c/openstack/cinder/+/814174 | 17:53 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline starting in 5 minutes, at 18:00 UTC, for scheduled project rename maintenance, which should last no more than an hour (but will likely be much shorter): http://lists.opendev.org/pipermail/service-announce/2021-October/000024.html | 17:58 | |
| *** dviroel|rover is now known as dviroel|rover|afk | 18:01 | |
| *** dviroel|rover|afk is now known as dviroel|rover | 18:09 | |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: DNM: test NetApp CI https://review.opendev.org/c/openstack/cinder/+/814214 | 18:52 |
| *** dviroel|rover is now known as dviroel|rover|afk | 21:08 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!