Tuesday, 2021-11-30

rdogerritrdo-trunk proposed rdoinfo master: Bump rdoinfo yoga-uc tags to latest upper-constraints  https://review.rdoproject.org/r/c/rdoinfo/+/3690200:22
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for ussuri-8-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696300:38
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for victoria-8-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696400:39
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for xena-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696500:39
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for wallaby-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696600:41
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for victoria-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696700:42
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for ussuri-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696800:43
*** rlandy_ is now known as rlandy|ruck01:43
*** rlandy|ruck is now known as rlandy|out01:48
*** rcastillo is now known as rcastillo|out04:20
*** pojadhav|afk is now known as pojadhav05:17
*** ysandeep|out is now known as ysandeep|ruck05:24
*** ysandeep|ruck is now known as ysandeep|lunch07:31
rdogerritTakashi Kajinami created openstack/gnocchi-distgit rpm-master: Copy api-paste.ini to /etc/gnocci  https://review.rdoproject.org/r/c/openstack/gnocchi-distgit/+/3696907:38
*** pojadhav is now known as pojadhav|lunch07:42
rdogerritTakashi Kajinami created openstack/aodh-distgit rpm-master: Copy api-paste.ini to /etc/aodh  https://review.rdoproject.org/r/c/openstack/aodh-distgit/+/3697007:43
rdogerritTakashi Kajinami proposed openstack/gnocchi-distgit rpm-master: Copy api-paste.ini to /etc/gnocci  https://review.rdoproject.org/r/c/openstack/gnocchi-distgit/+/3696907:43
rdogerritJoel Capitao proposed rdoinfo master: Bump rdoinfo yoga-uc tags to latest upper-constraints  https://review.rdoproject.org/r/c/rdoinfo/+/3690208:21
rdogerritMerged openstack/ironic-inspector-client-distgit rpm-master: Use stestr during %check  https://review.rdoproject.org/r/c/openstack/ironic-inspector-client-distgit/+/3692308:22
*** ysandeep|lunch is now known as ysandeep08:22
*** amoralej|off is now known as amoralej08:24
rdogerritMerged rdoinfo master: Promote CBS tags update for ussuri-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696808:25
rdogerritrdo-trunk created openstack/ironic-inspector-client-distgit rpm-master: python-ironic-inspector-client: failed to build 82649e1  https://review.rdoproject.org/r/c/openstack/ironic-inspector-client-distgit/+/3697108:31
rdogerritMerged config master: Add missing deps to send deps-uc review  https://review.rdoproject.org/r/c/config/+/3696108:40
*** bhagyashris_ is now known as bhagyashris08:42
rdogerritmathieu bultel proposed rdo-jobs master: Enable validation on wallaby ovb component jobs  https://review.rdoproject.org/r/c/rdo-jobs/+/3685008:49
rdogerritMerged nfvinfo master: Promote CBS tags update for nfvsig-2.16-release  https://review.rdoproject.org/r/c/nfvinfo/+/3692108:57
rdogerritMerged rdoinfo master: Promote CBS tags update for ussuri-8-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696309:02
rdogerritMerged rdoinfo master: Promote CBS tags update for victoria-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696709:02
rdogerritMerged rdoinfo master: Promote CBS tags update for victoria-8-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696409:02
rdogerritMerged rdoinfo master: Promote CBS tags update for xena-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696509:02
rdogerritMerged rdoinfo master: Promote CBS tags update for wallaby-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/3696609:02
rdogerritMerged nfvinfo master: Promote CBS tags update for nfvsig-2021-release  https://review.rdoproject.org/r/c/nfvinfo/+/3692209:07
*** pojadhav|lunch is now known as pojadhav09:10
rdogerritMerged nfvinfo master: Promote CBS tags update for nfvsig-2.15-release  https://review.rdoproject.org/r/c/nfvinfo/+/3692009:21
*** jpena|off is now known as jpena09:27
rdogerritAlfredo Moralejo proposed nfvinfo master: Promote CBS tags update for nfvsig-2.13-release  https://review.rdoproject.org/r/c/nfvinfo/+/3691909:33
*** rlandy is now known as rlandy|ruck10:44
*** ysandeep is now known as ysandeep|afk10:51
*** jcapitao1 is now known as jcapitao11:00
*** ysandeep|afk is now known as ysandeep|ruck11:06
*** dviroel|out is now known as dviroel11:15
jcapitaoamoralej, tvignaud: https://pagure.io/centos-infra/issue/54011:27
amoralejgood11:28
rdogerritMerged rdo-jobs master: Enable validation on wallaby ovb component jobs  https://review.rdoproject.org/r/c/rdo-jobs/+/3685011:48
rdogerritArx Cruz created rdo-infra/ci-config master: Add centos 9 components data  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3697212:14
*** amoralej is now known as amoralej|lunch12:17
*** ysandeep|ruck is now known as ysandeep|afk13:09
rdogerritmathieu bultel proposed rdo-jobs master: Enable Validation on Clients and Security OVB jobs for Wallaby  https://review.rdoproject.org/r/c/rdo-jobs/+/3685513:18
*** sshnaidm|afk is now known as sshnaidm13:29
*** ysandeep|afk is now known as ysandeep13:34
*** ysandeep is now known as ysandeep|ruck13:34
*** amoralej|lunch is now known as amoralej13:35
hjensasGood day #rdo. I have question on metalsmith packaging for Wallaby.14:01
hjensasWe have today: - project: metalsmith14:01
hjensas  tags:14:01
hjensas    wallaby:14:01
hjensas      source-branch: 1.4.214:01
hjensashttps://github.com/redhat-openstack/rdoinfo/blob/321f44594a33895e3a64e081b6e235dfcedeed05/tags/wallaby.yml#L322-L32514:01
amoralejhjensas, what's the question?14:02
hjensasAs we are landing some bugfixes in stable/wallaby branch. Would it make sense to change that, removing the "source-branch"?14:02
hjensasI'm thinking, to not have to keep doing releases?14:02
amoralejhjensas, the best option is to create a new release in wallaby and update it in upper-constrainst.txt14:03
amoralejhttps://github.com/openstack/requirements/blob/stable/wallaby/upper-constraints.txt#L30814:03
amoralejhjensas, for projects included in upper-constraints.txt we follow what is there14:03
amoralejas it's what is used in other projects gates14:03
hjensasamoralej: ok, so source-branch would bunp automatically in rdoinfo based on upper-constraints? 14:04
amoralejthat's where 1.4.2 is coming from14:04
amoralejyes14:04
amoralejwe have a nightly job to automatically propose updates from u-c14:04
hjensasok, yeah then let's follow process and keep doing that.14:04
amoralejhjensas, example https://review.rdoproject.org/r/c/rdoinfo/+/3690114:05
amoralejif you want to speed it up, you can send the update to rdoinfo yourself but once it's in u-c if possible14:05
ysandeep|ruckhjensas, your patch in master just merged, we will need a new tag in master as well.14:05
ysandeep|ruckhjensas,  i can propose ^^ if you want, but not sure if anyone can do that or just PTL14:13
ysandeep|rucki wonder if its worth adding a third party ovb jobs check jobs in openstack/metalsmith repo - to make sure it don't break us.14:24
ysandeep|ruckhjensas, rlandy|ruck ^^ what do you think - frequency of patches on that repo is not much 14:25
ysandeep|ruck+ it will give hint to us before release that latest metalsmith will break us, and we avoid unnecessary releases just for ci bug fixes14:27
ysandeep|rucki mean creating new release tags*14:27
rlandy|ruckysandeep|ruck: ok by me14:34
ysandeep|ruckrlandy|ruck, ack thanks, lets wait for hjensas to share his thoughts, if he sees value in adding third party ovb jobs check jobs in openstack/metalsmith repo.14:36
*** pojadhav is now known as pojadhav|brb14:37
*** rcastillo|out is now known as rcastillo14:38
*** dviroel is now known as dviroel|lunch14:51
*** pojadhav|brb is now known as pojadhav15:14
amoralejwe have nfvsig packages for CS9 in official mirrors now http://mirror.stream.centos.org/SIGs/9-stream/nfv/x86_64/openvswitch-2/Packages/o/15:20
amoralejapevec[m], jcapitao[m] ^ 15:20
apevec[m]nice15:21
apevec[m]amoralej: btw next FDP.I seems to break oooci, please don't push it until tested in tripleo deps pipeline15:22
amoralejok15:22
amoralejit'd be a good oportunity to test our CI :)15:22
mwhahahaamoralej: what would be the best way to get grpcio bumped (and maybe also include grpcio-tools)15:24
apevec[m]ovn 21.06.0 is the still good one15:26
amoralejmwhahaha, what version would you like to have?15:26
mwhahahasomething newer than 1.26 (at least 1.32 though current is 1.46 i think15:26
apevec[m]is that from Fedora?15:26
mwhahahait's in fedora15:26
mwhahahathough not certain if the newer version is15:27
mwhahahayea looks like it's still 1.26 in fedora15:27
apevec[m]let's bump it there first - what's the SRPM name?15:28
mwhahahahttps://fedora.pkgs.org/34/fedora-x86_64/python3-grpcio-1.26.0-13.fc34.x86_64.rpm.html15:28
mwhahaha?15:28
*** ykarel is now known as ykarel|away15:28
amoralejgrpc-1.41.1-4.fc36 is in fedorahttps://koji.fedoraproject.org/koji/buildinfo?buildID=185257915:28
apevec[m]grpc15:28
amoralejhttps://koji.fedoraproject.org/koji/buildinfo?buildID=1852579 15:28
amoralejmwhahaha, would that be fine? ^15:28
mwhahahayes it would15:29
apevec[m]we can pull from Rawhide15:29
mwhahahathey added the ability to not have to generate python code in 1.32 but you need grpcio-tools too15:29
amoralejok, gimme a while15:29
mwhahahasure whenever15:29
apevec[m]amoralej: or you let mwhahaha bump it in rdoinfo themselves?15:30
apevec[m]self-service :)15:30
mwhahahai couldn't find it15:30
mwhahahaself-service if you know where to look15:30
mwhahahai only found the el8 tags15:30
mwhahahahence my question as to how would i go about doing it :D15:31
apevec[m]hmm, it's not fedora-dependency ?15:32
mwhahahanot that i saw15:32
apevec[m]https://github.com/redhat-openstack/rdoinfo/blob/master/deps.yml#L193815:33
apevec[m]rdo-dependency15:33
apevec[m]so it was forked for some reason15:33
mwhahahait's needed by kuryr-kubernetes15:34
mwhahahaafaict15:34
apevec[m]https://review.rdoproject.org/r/q/project:deps/grpc15:34
mwhahahaso it may not have been available back then15:34
apevec[m]so we already have a bump under review15:34
apevec[m]https://review.rdoproject.org/r/c/deps/grpc/+/3654815:35
apevec[m]still unclear to me why we just do it as fedora-dependecy15:35
mwhahahamaybe we can blame EmilienM while we're at it15:36
apevec[m]looks like only to not build py3 on EL7  https://github.com/rdo-common/grpc/commit/5978d76a36b5a479bfe49c0592bbe4a77ef2d92015:36
apevec[m]that was needed 3yrs ago, not anymore15:36
mwhahahafeels like we should probably switch it though need to check with the kuryr folks problem15:37
apevec[m]EmilienM is an easy target, he was all over everything :)15:37
mwhahahait's the only one that currently depends on it15:37
mwhahahas/problem/probably15:37
EmilienMlet me get someone from Kuryr15:39
* EmilienM invokes maysams's Kuryr PTL15:39
maysamshello o/15:40
EmilienMmwhahaha, apevec[m] ^ so what's the issue with Kuryr deps?15:40
mwhahahamaysams: we're discussing bumping python3-grpcio15:40
mwhahahacurrently is 1.26, i'd like to use it but get a newer version15:40
mwhahahakuryr-kubernetes currently depends on it >=1.18 i think15:40
mwhahahawould bumping it to something more current be a problem15:40
mwhahahahttps://github.com/rdo-packages/kuryr-kubernetes-distgit/blob/6bd93bf6a39acef5b13e659398cec48a05db7183/openstack-kuryr-kubernetes.spec#L9615:42
*** ysandeep|ruck is now known as ysandeep|out15:42
*** dviroel|lunch is now known as dviroel15:44
dulekGood evening!15:45
maysamsmwhahaha: I believe we wouldn't be fine with upgrading grpcio.15:45
mwhahaharight hat's why i asked15:45
* dulek attemts to catch up with logs at opendev.org.15:46
mwhahahaseems like something we need to coordinate15:46
maysamsmwhahaha: I think It's currently used by a feature which is not maintained15:46
mwhahaha1.32 allows for no code-gen which would make the version dep less a problem15:46
maysamsdulek: might have more inputs around it15:46
mwhahahahttps://github.com/openstack/kuryr-kubernetes/tree/master/kuryr_kubernetes/pod_resources15:46
mwhahahais where it's used15:46
mwhahahasince generated code is being used (in my limited experience) that can be problematic with version bumps around grpcio15:47
maysamsdulek: Folks want to bump the version of grpcio15:47
dulekWhat's the reasons for the bump? Just out of curiosity.15:47
mwhahahai want to use teh no-code gen feature in 1.3215:48
EmilienMmwhahaha's shenanigans15:48
EmilienM:P15:48
mwhahahaand my personal shenanigans15:48
maysamsmwhahaha: I meant we would be fine, sorry15:48
dulekmwhahaha: Well, I can't say that no-code gen isn't interesting.15:48
dulekThe problem on our side is probably that we don't really know how to test that thing anymore, the guys behind it left upstream.15:49
EmilienMdulek: in that case, I guess you (we) maintain it as best effort and fix breaks if they come.15:49
dulekMeaning that we probably don't have any moral right to block the upgrade.15:50
mwhahahais the code path used? i mean i can assist a bit15:50
mwhahahathe implementation there is really basic15:50
dulekAs most likely if it's not tested, it's broken anyway. So I'd say go for it.15:50
mwhahahaha15:50
mwhahahaok15:50
EmilienM:D +115:50
dulekmwhahaha: But if you can take a look and attempt to adapt the code to higher grpcio, it'd be really appreciated.15:51
mwhahahayea i will15:51
maysamsthanks mwhahaha :)15:51
EmilienMdulek: mwhahaha will probably break it FYI15:51
mwhahahapfft15:51
mwhahahaactually i don't know if you even use that code15:52
dulekmwhahaha: https://review.opendev.org/c/openstack/kuryr-kubernetes/+/812727 - this is my attempt at a bit of a bump.15:52
mwhahahaok yea i'll help out there15:52
mwhahahathe issue is the need for the generated code to match the installed lib15:53
mwhahahawhich is why i wanted no-codegen so it would matter les15:53
mwhahahaless15:53
mwhahahathere's some breakage in the generated code between 1.26 and 1.41 somewhere15:53
dulekThe code that uses this thing in Kuryr is there: https://github.com/openstack/kuryr-kubernetes/blob/1c2cd789662b447c199b1ff1ec47313b32e833d1/kuryr_kubernetes/cni/binding/sriov.py#L66-L6715:54
dulekAFAIR the point was to allow kuryr-cni to reach kubelet and get info on SR-IOV resources available.15:55
mwhahahaall that code to just list something15:55
mwhahahafun15:55
dulekmwhahaha: Telco stuff is always like this. ;)15:55
mwhahahadulek: so which order do you want to fix this? bump the version in kuryr-kubernetes but rdo packages will be broken? or the opposite?15:58
EmilienMIMO we should bump in the kuryr repo first, make sure tests etc work fine; then bump packaging if all green15:59
mwhahahaEmilienM: there seem to be no tests which is why I asked15:59
dulekRDO package can be broken, I strongly doubt anyone uses it, Kuryr is really a container thing now.15:59
mwhahahak15:59
dulekmaysams: Does it work with you? We might have a bit of issue downstream, where we only have python3-grpcio 1.18.0, but I think it's for us to figure it out.16:00
mwhahahawe only have 1.18 downstream? we have 1.26 in rdo so it should be 1.2616:01
mwhahahaugh it comes from rhocp16:01
dulekmwhahaha: We're constrainted to what's in a released OSP, so I think we're using OSP16.2 repos.16:01
dulekmwhahaha: Ha, might actually be us that requested it.16:02
dulekKuryr is part of OCP.16:02
mwhahahaseems to be ocp only16:02
mwhahahathat might be problemmatic16:02
amoralejapevec[m], mwhahaha i wanted to use that update to test our scripts for new deps process16:03
mwhahahadulek: so if kuryr-kubernetes doesn't use RDO (and by extension OSP) depenedencies, it likely wouldn't be broken downstream. but bumping it downstream would need some other packaging fixes for OCP to get a newer version16:19
maysamsmwhahaha, dulek: I believe it's fine to try the bumping16:19
dulekmwhahaha: Oh, we do use OSP dependencies, OSP 16.2 to be precise.16:20
mwhahahadulek: it's not shipped in OSP16:20
mwhahahaso no, you're pulling it from OCP16:20
mwhahaha(i asked, osp doesn't ship it)16:21
dulekYes, GRPC comes from OCP per our request. But we do use OSP deps for other stuff.16:21
mwhahahahttps://access.redhat.com/downloads/content/python3-grpcio/1.18.0-4.el8ost/x86_64/fd431d51/package16:21
dulekJust to be precise. ;)16:21
mwhahahaso yea we'd need to match it up some how i guess16:21
amoralejmwhahaha, so you want to get it updated ini both cs8 and cs9?16:21
mwhahahaif we start shipping (likely not for 16.2), you'd need a version16:21
mwhahahaamoralej: yea would be best16:21
mwhahahayou'd need a matching version to wallaby i guess16:22
mwhahahaif shipped in 916:22
dulekmwhahaha: So what got to be upgraded for an OCP bump?16:22
amoralejyep16:22
mwhahahadulek: the issue being that the generated files need to be generated using a version you're actually building with. so if we ship a newer grpcio (in rdo/osp) for say wallaby and it gets pulled in by your code at build time (e.g. you don't specifically set == 1.18), you'll likely get execution failures.16:24
mwhahahadulek: so downstream if OSP starts shipping a newer version of grpcio, you'll need to make sure it's either not pulled in at build time or update the code to handle the newer verison16:24
dulekmwhahaha: That's fine, I think, we update code upstream->downstream pretty often.16:30
mwhahahaok so i'm going to test out the code gen bits and if that works i can propose a patch to kuryr-kubernetes that leverages that and we can go from there16:31
mwhahahain the mean time we can bump the rdo packages and figure out where it breaks in the process later16:31
dulekCool!16:32
*** xek_ is now known as xek16:48
amoralejmwhahaha, it'll take me som time to build that16:56
amoralejit adds more dependencies16:57
mwhahahaaww ok16:57
mwhahahastupid dependencies16:57
amoralejDEBUG util.py:444:  No matching package to install: 'pkgconfig(libxxhash)'16:59
amoralejDEBUG util.py:444:  No matching package to install: 'pkgconfig(re2)'16:59
amoralejDEBUG util.py:444:  No matching package to install: 'python3dist(xds-protos) >= 0.0.7'16:59
amoralejDEBUG util.py:444:  No matching package to install: 'wyhash_final1-devel'16:59
amoralejDEBUG util.py:444:  No matching package to install: 'wyhash_final1-static'16:59
amoralejDEBUG util.py:444:  No matching package to install: 'xxhash-static'16:59
amoralejDEBUG util.py:444:  No matching package to install: 'abseil-cpp-devel'16:59
mwhahahaamoralej: ok let's hold off for a bit. likely need to verify these news deps are fips friendly as well17:01
rdogerritRafael Castillo created rdo-jobs master: Add definition for scenario002 jobs for C9  https://review.rdoproject.org/r/c/rdo-jobs/+/3697417:02
amoralejyes, i was thinking about security concerns too17:02
*** jlibosva is now known as jlibosva|lunch17:14
*** jlibosva|lunch is now known as jlibosva17:14
rdogerritSagi Shnaidman created rdo-jobs master: Use ansible-core for ansible-lint  https://review.rdoproject.org/r/c/rdo-jobs/+/3697517:31
*** jpena is now known as jpena|off17:35
rdogerritMerged rdoinfo master: Bump rdoinfo yoga-uc tags to latest upper-constraints  https://review.rdoproject.org/r/c/rdoinfo/+/3690217:56
*** amoralej is now known as amoralej|off17:59
*** dviroel is now known as dviroel|afk20:13

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!