rdogerrit | rdo-trunk created rdoinfo master: Bump rdoinfo wallaby tags to latest upper-constraints https://review.rdoproject.org/r/c/rdoinfo/+/43781 | 00:10 |
---|---|---|
rdogerrit | rdo-trunk created rdoinfo master: Promote CBS tags update for xena-8s-testing https://review.rdoproject.org/r/c/rdoinfo/+/43782 | 00:34 |
rdogerrit | rdo-trunk proposed rdoinfo master: Promote CBS tags update for wallaby-8s-testing https://review.rdoproject.org/r/c/rdoinfo/+/43701 | 00:37 |
rdogerrit | rdo-trunk created nfvinfo master: Promote CBS tags update for nfvsig-2.15-testing https://review.rdoproject.org/r/c/nfvinfo/+/43783 | 00:37 |
rdogerrit | rdo-trunk created nfvinfo master: Promote CBS tags update for nfvsig-2.16-testing https://review.rdoproject.org/r/c/nfvinfo/+/43784 | 00:37 |
*** akahat|out is now known as akahat|ruck | 05:15 | |
rdogerrit | Martin Kopec proposed rdoinfo master: [DFG][HardProv] Bumping ironic plugin to 2.3.0 https://review.rdoproject.org/r/c/rdoinfo/+/43603 | 07:08 |
*** amoralej|off is now known as amoralej | 07:34 | |
*** jpena|off is now known as jpena | 07:39 | |
rdogerrit | Joel Capitao proposed rdoinfo master: Promote CBS tags update for wallaby-8s-testing https://review.rdoproject.org/r/c/rdoinfo/+/43701 | 07:39 |
rdogerrit | Merged rdoinfo master: Promote CBS tags update for xena-8s-testing https://review.rdoproject.org/r/c/rdoinfo/+/43782 | 07:42 |
rdogerrit | Merged openstack/neutron-distgit rpm-master: lock_path should be set in [oslo_concurrency] https://review.rdoproject.org/r/c/openstack/neutron-distgit/+/43769 | 07:43 |
rdogerrit | Merged nfvinfo master: Promote CBS tags update for nfvsig-2.16-testing https://review.rdoproject.org/r/c/nfvinfo/+/43784 | 07:58 |
rdogerrit | Merged rdo-jobs master: Add rpmlint exception for cinderlib https://review.rdoproject.org/r/c/rdo-jobs/+/43771 | 07:58 |
rdogerrit | Martin Kopec created rdoinfo master: Bump tempest to (almost) 31.1.0 https://review.rdoproject.org/r/c/rdoinfo/+/43785 | 08:10 |
rdogerrit | Joel Capitao proposed openstack/cinderlib-distgit xena-rdo: Exclude mangling shebang https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43753 | 08:10 |
rdogerrit | Joel Capitao proposed openstack/cinderlib-distgit yoga-rdo: python-cinderlib-4.2.0-1 https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43751 | 08:13 |
rdogerrit | Joel Capitao proposed openstack/cinderlib-distgit wallaby-rdo: Exclude mangling shebang https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43757 | 08:14 |
rdogerrit | Merged openstack/cinderlib-distgit rpm-master: Exclude mangling shebang https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43730 | 08:24 |
rdogerrit | Joel Capitao proposed openstack/cinderlib-distgit yoga-rdo: python-cinderlib-4.2.0-1 https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43751 | 08:33 |
rdogerrit | Joel Capitao created openstack/cinderlib-distgit rpm-master: Add importlib-resources as Requires https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43786 | 08:43 |
jcapitao[m] | amoralej: we need to package dep importlib-resources for cs9 (needed by cinderlib), do you remember why we haven't build it for CS9 (we have it in cs8 deps) ? | 09:31 |
*** rlandy|out is now known as rlandy | 09:33 | |
amoralej | jcapitao[m], it's not required after python 3.8 | 10:05 |
amoralej | it's in built-in libraries | 10:05 |
amoralej | we don't need it in cs9 | 10:05 |
amoralej | should be patched upstream if needed | 10:05 |
jcapitao[m] | amoralej: right, but cinderlib uses a new feature released since py3.10.. https://review.opendev.org/c/openstack/cinderlib/+/847170/7/cinderlib/cinderlib.py#415 | 10:07 |
amoralej | mmm | 10:08 |
jcapitao[m] | I tried to use the built-in importlib but unit test fail | 10:08 |
amoralej | i doubt that should be 10 | 10:08 |
amoralej | so, that means importib_resources is different that importlib.resources? | 10:08 |
amoralej | in 3.8? | 10:08 |
amoralej | lemme check | 10:08 |
jcapitao[m] | I patched by replacing 10 by 7 in l.415 | 10:10 |
jcapitao[m] | in order to import importlib.resources | 10:10 |
amoralej | and failed, right? | 10:11 |
jcapitao[m] | yep | 10:11 |
jcapitao[m] | https://paste.centos.org/view/8ec88919 | 10:12 |
amoralej | mmm | 10:14 |
amoralej | but from importlib.resources import files | 10:14 |
amoralej | works in cs9 | 10:14 |
jcapitao[m] | yes as it has been introduced since 3.9 https://docs.python.org/3.11/library/importlib.resources.html#importlib.resources.files | 10:15 |
jcapitao[m] | but according to comment it's the feature "namespace packages" they need | 10:16 |
jcapitao[m] | but I can't find it in https://docs.python.org/3.11/library/importlib.resources.html though | 10:16 |
rdogerrit | Joel Capitao created openstack/cinderlib-distgit yoga-rdo: DNM python-cinderlib-4.2.0-1 https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43787 | 10:19 |
jcapitao[m] | amoralej: I pushed my local changes to have a common base to test ^ | 10:20 |
amoralej | geguileo, wrt cinderlib in old releases (before yoga) should we follow master branch or pin to specific tags? | 10:39 |
amoralej | jcapitao[m], ^ | 10:39 |
geguileo | amoralej: we backported the fix back to wallaby upstream because we run cinderlib functional tests in the cinder lvm and ceph jobs, and cinderlib was breaking cinder stable jobs | 10:46 |
geguileo | amoralej: I think we can ignore that patch for older RDO releases for the time being since RDO doesn't have that issue | 10:47 |
geguileo | and we won't fix the venv issue on those releases | 10:47 |
geguileo | if it becomes a problem in the future we can improve cinderlib to not use namespaces for the venv script so it can run in 3.9 with importlib | 10:48 |
amoralej | ah, sorry, for some reason i thought cinderlib is branchless | 10:49 |
amoralej | i see it now | 10:49 |
geguileo | amoralej: it's release trailing, which is why yoga was releases so "late" | 10:58 |
*** dviroel|out is now known as dviroel | 11:20 | |
rdogerrit | User karolinku proposed openstack/gnocchi-distgit rpm-master: Remove amqpd condition and add new BR https://review.rdoproject.org/r/c/openstack/gnocchi-distgit/+/43759 | 11:32 |
rdogerrit | Chandan Kumar proposed rdo-jobs master: Rename centos-9-quay job to ubi-9-quay job https://review.rdoproject.org/r/c/rdo-jobs/+/43754 | 11:40 |
*** amoralej is now known as amoralej|lunch | 12:06 | |
*** blarnath is now known as d34dh0r53 | 12:18 | |
*** pojadhav is now known as pojadhav|afk | 12:52 | |
*** amoralej|lunch is now known as amoralej | 13:15 | |
rdogerrit | Joel Capitao proposed openstack/cinderlib-distgit yoga-rdo: python-cinderlib-4.2.0-1 https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43751 | 13:25 |
jcapitao[m] | RDO meeting in 12 minutes - https://etherpad.opendev.org/p/RDO-Meeting | 13:48 |
rdogerrit | Joel Capitao proposed openstack/cinderlib-distgit xena-rdo: Exclude mangling shebang https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43753 | 13:49 |
rdogerrit | Joel Capitao proposed openstack/cinderlib-distgit wallaby-rdo: Exclude mangling shebang https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43757 | 13:51 |
rdogerrit | Joel Capitao proposed openstack/cinderlib-distgit rpm-master: WIP Revert "Disable unit tests for master branch" https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43786 | 13:55 |
jcapitao[m] | #startmeeting RDO meeting - 2022-06-29 | 14:00 |
opendevmeet | Meeting started Wed Jun 29 14:00:42 2022 UTC and is due to finish in 60 minutes. The chair is jcapitao[m]. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
opendevmeet | The meeting name has been set to 'rdo_meeting___2022_06_29' | 14:00 |
jcapitao[m] | #topic roll call | 14:00 |
kkula | o/ | 14:01 |
jcapitao[m] | #chair spotz | 14:01 |
opendevmeet | Current chairs: jcapitao[m] spotz | 14:01 |
spotz | o/ | 14:02 |
jcapitao[m] | oops | 14:02 |
jcapitao[m] | #chair kkula | 14:02 |
opendevmeet | Current chairs: jcapitao[m] kkula spotz | 14:02 |
*** dasm|afk is now known as dasm | 14:03 | |
amoralej | o/ | 14:04 |
jcapitao[m] | #chair amoralej | 14:04 |
opendevmeet | Current chairs: amoralej jcapitao[m] kkula spotz | 14:04 |
jcapitao[m] | let's start with first topic | 14:05 |
jcapitao[m] | #topic FTBFS in fedora massive rebuilds for Fedora 37 with python 3.11 | 14:05 |
jcapitao[m] | #link https://review.rdoproject.org/etherpad/p/FTBFS-fedora-37 | 14:05 |
jcapitao[m] | I checked the list again | 14:05 |
jcapitao[m] | I mean the list published by Fedora folks | 14:06 |
amoralej | yeah | 14:06 |
jcapitao[m] | there is one remaining python-murano-pkg-check | 14:06 |
amoralej | some of them were not actual issues i think, but ok | 14:06 |
amoralej | i've reported the yappi one upstream | 14:07 |
jcapitao[m] | yes after some iteration it's ok for most of them | 14:08 |
amoralej | yep | 14:08 |
amoralej | unicodecsv, last release 2015 :( | 14:10 |
jcapitao[m] | ok, so almost good | 14:10 |
jcapitao[m] | murano-pkg-check is a bit tricky too | 14:10 |
amoralej | what's the problem with it? | 14:10 |
jcapitao[m] | at first it was just adding safe loader to yaml, but now I'm hitting other issue with mock | 14:11 |
amoralej | also very low activity | 14:12 |
amoralej | https://review.opendev.org/q/project:openstack/murano-pkg-check | 14:12 |
jcapitao[m] | when importing mock decoration helper it fails | 14:12 |
amoralej | but it seems maintained | 14:12 |
amoralej | we may report it upstream if needed | 14:12 |
jcapitao[m] | yeah I think I'm gonna report it | 14:13 |
jcapitao[m] | ah, for python-unicodecsv we have to retire it too I guess | 14:15 |
amoralej | it's not longer used? | 14:15 |
amoralej | i see we didnĀ”'t import it in cs9 | 14:15 |
jcapitao[m] | it's not required by anything | 14:17 |
amoralej | then, i'd bet for retiring it | 14:19 |
amoralej | be careful | 14:20 |
amoralej | https://src.fedoraproject.org/rpms/python-registry/blob/rawhide/f/python-registry.spec#_30 | 14:20 |
amoralej | jcapitao[m], ^ | 14:20 |
amoralej | and https://src.fedoraproject.org/rpms/python-rows/blob/rawhide/f/python-rows.spec#_37 | 14:21 |
jcapitao[m] | ah! | 14:22 |
jcapitao[m] | ok I'll dig in later | 14:22 |
amoralej | so, even if we don't use it in rdo | 14:23 |
amoralej | let's check if those actually need it | 14:23 |
jcapitao[m] | right | 14:23 |
jcapitao[m] | I'll check the codebase | 14:24 |
rdogerrit | Merged openstack/cinderlib-distgit yoga-rdo: python-cinderlib-4.2.0-1 https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43751 | 14:24 |
rdogerrit | Merged openstack/cinderlib-distgit xena-rdo: Exclude mangling shebang https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43753 | 14:24 |
amoralej | it's drop in for standard csv | 14:25 |
amoralej | maybe it's not longer needed | 14:25 |
amoralej | i'm not sure | 14:25 |
spotz | If we drop and it's an issue will tests catch it? Or could we possibly mark it for deprecation and see who says something? | 14:26 |
amoralej | i think it will appear in next reports | 14:28 |
amoralej | of packages marked to be retired | 14:28 |
amoralej | but as we know who is using them | 14:28 |
amoralej | let's try to ping the maintainers directly | 14:28 |
spotz | +2 | 14:29 |
amoralej | https://github.com/williballenthin/python-registry/blob/master/samples/amcache.py#L24 | 14:29 |
amoralej | python-registry is also using it | 14:29 |
jcapitao[m] | ok let's ask them | 14:31 |
amoralej | both packages are from the same maintainer | 14:31 |
amoralej | and he's on #fedora-devel :) | 14:31 |
amoralej | so, it'll be easy | 14:32 |
rdogerrit | Merged openstack/cinderlib-distgit wallaby-rdo: Exclude mangling shebang https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/43757 | 14:33 |
jcapitao[m] | yes he's very active | 14:33 |
amoralej | yep | 14:33 |
jcapitao[m] | any questions on this topic ? | 14:34 |
*** pojadhav|afk is now known as pojadhav | 14:35 | |
jcapitao[m] | let's move on | 14:35 |
jcapitao[m] | #topic next chair | 14:35 |
jcapitao[m] | any volunteers | 14:35 |
jcapitao[m] | ? | 14:35 |
jcapitao[m] | it will be video call though | 14:35 |
amoralej | i can take it | 14:36 |
jcapitao[m] | thanks amoralej | 14:37 |
jcapitao[m] | #action amoralej will chair the next meeting | 14:37 |
jcapitao[m] | #topic Open Floor | 14:37 |
tosky | thanks all the team for the work with keeping the packages up-to-date (like the countless times I've ask amoralej for a review), and special thanks today for jcapitao[m] for the help unblocking cinderlib | 14:37 |
amoralej | jcapitao++ | 14:38 |
jcapitao[m] | I'm glad to help :) | 14:40 |
amoralej | and thanks tosky for being such a nice maintainer :) | 14:40 |
jcapitao[m] | yeah! | 14:40 |
spotz | Thank you tosky for coming here and saying thanks! | 14:40 |
spotz | I think we sometimes forget to say that enough | 14:41 |
amoralej | totally | 14:41 |
amoralej | wrt cs8 removal | 14:42 |
amoralej | once https://review.opendev.org/c/openstack/tripleo-ansible/+/839688/ is merged we'll be able to finally remove centos8 from master | 14:42 |
jcapitao[m] | great it's gating | 14:43 |
spotz | Nice | 14:43 |
amoralej | yep | 14:43 |
tosky | great! | 14:44 |
amoralej | finally :) | 14:45 |
jcapitao[m] | yeah :) | 14:46 |
amoralej | fyi, we'll need to bump python-setuptools_scm for centos8 to get gnocchi 4.4 branch working | 14:49 |
amoralej | i've been working with kkula about it | 14:49 |
amoralej | it's wip | 14:49 |
jcapitao[m] | ok | 14:51 |
jcapitao[m] | I'm going to close the meeting if nothing more to add | 14:51 |
amoralej | nothing from my side | 14:52 |
jcapitao[m] | let's close | 14:53 |
jcapitao[m] | #endmeeting | 14:53 |
opendevmeet | Meeting ended Wed Jun 29 14:53:14 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:53 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/rdo_meeting___2022_06_29/2022/rdo_meeting___2022_06_29.2022-06-29-14.00.html | 14:53 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/rdo_meeting___2022_06_29/2022/rdo_meeting___2022_06_29.2022-06-29-14.00.txt | 14:53 |
opendevmeet | Log: https://meetings.opendev.org/meetings/rdo_meeting___2022_06_29/2022/rdo_meeting___2022_06_29.2022-06-29-14.00.log.html | 14:53 |
jcapitao[m] | thank you all for joining | 14:53 |
spotz[m] | Thanks jcapitao for | 14:53 |
spotz[m] | Leading and everyone for attending | 14:53 |
amoralej | thanks jcapitao[m]! | 14:54 |
rdogerrit | Michael Johnson created openstack/octavia-distgit rpm-master: Update Octavia spec for secure-RBAC https://review.rdoproject.org/r/c/openstack/octavia-distgit/+/43788 | 15:09 |
ykarel | amoralej, when you get chance, please check https://review.rdoproject.org/r/q/topic:requirements-sync | 15:21 |
ykarel | and also https://review.rdoproject.org/r/c/nfvinfo/+/43666 | 15:22 |
*** dviroel is now known as dviroel|lunch | 15:24 | |
rdogerrit | rdo-trunk created openstack/cloudkitty-distgit wallaby-rdo: openstack-cloudkitty-14.0.2-1 https://review.rdoproject.org/r/c/openstack/cloudkitty-distgit/+/43790 | 15:45 |
rdogerrit | rdo-trunk created openstack/ec2-api-distgit wallaby-rdo: openstack-ec2-api-12.1.0-1 https://review.rdoproject.org/r/c/openstack/ec2-api-distgit/+/43791 | 15:45 |
*** marios is now known as marios|out | 15:48 | |
*** rlandy is now known as rlandy|biab | 15:51 | |
*** rlandy|biab is now known as rlandy | 16:27 | |
*** jpena is now known as jpena|off | 16:34 | |
*** dviroel|lunch is now known as dviroel | 16:37 | |
rdogerrit | Merged openstack/cloudkitty-distgit wallaby-rdo: openstack-cloudkitty-14.0.2-1 https://review.rdoproject.org/r/c/openstack/cloudkitty-distgit/+/43790 | 16:50 |
*** amoralej is now known as amoralej|off | 16:52 | |
rdogerrit | Merged openstack/neutron-distgit yoga-rdo: Requirement Sync https://review.rdoproject.org/r/c/openstack/neutron-distgit/+/43774 | 17:06 |
rdogerrit | Merged openstack/neutron-distgit xena-rdo: Requirement Sync https://review.rdoproject.org/r/c/openstack/neutron-distgit/+/43775 | 17:09 |
rdogerrit | Merged openstack/neutron-distgit wallaby-rdo: Requirement Sync https://review.rdoproject.org/r/c/openstack/neutron-distgit/+/43776 | 17:09 |
*** undefined_ is now known as Guest3684 | 17:55 | |
*** Guest3684 is now known as rcastillo_ | 17:55 | |
*** rcastillo_ is now known as rcastillo | 17:57 | |
rdogerrit | Jon Schlueter created rdoinfo master: add upstream for dogpile-cache https://review.rdoproject.org/r/c/rdoinfo/+/43792 | 18:43 |
*** dviroel is now known as dviroel|biab | 20:30 | |
rdogerrit | rdo-trunk created openstack/designate-tempest-plugin-distgit rpm-master: python-designate-tests-tempest: failed to build 3fe1bd1 https://review.rdoproject.org/r/c/openstack/designate-tempest-plugin-distgit/+/43794 | 21:26 |
rdogerrit | Merged config master: Update auth url for vexxhost https://review.rdoproject.org/r/c/config/+/43793 | 21:29 |
rdogerrit | Ronelle Landy created config master: Revert "Update auth url for vexxhost" https://review.rdoproject.org/r/c/config/+/43679 | 21:38 |
rdogerrit | Merged config master: Revert "Update auth url for vexxhost" https://review.rdoproject.org/r/c/config/+/43679 | 22:00 |
*** dviroel|biab is now known as dviroel | 22:09 | |
*** dasm is now known as dasm|off | 22:15 | |
*** rlandy is now known as rlandy|bbl | 22:25 | |
*** dviroel is now known as dviroel|out | 22:28 | |
rdogerrit | rdo-trunk created puppet/puppet-swift-distgit rpm-master: puppet-swift: failed to build e004e46 https://review.rdoproject.org/r/c/puppet/puppet-swift-distgit/+/43795 | 22:44 |
*** gce108__ is now known as gce108 | 22:51 | |
rdogerrit | rdo-trunk created openstack/tripleo-heat-templates-distgit rpm-master: openstack-tripleo-heat-templates: failed to build 8b0260e3b https://review.rdoproject.org/r/c/openstack/tripleo-heat-templates-distgit/+/43796 | 23:29 |
rdogerrit | rdo-trunk created openstack/tripleo-ansible-distgit rpm-master: tripleo-ansible: failed to build 6d757154 https://review.rdoproject.org/r/c/openstack/tripleo-ansible-distgit/+/43797 | 23:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!