| rdogerrit | rdo-trunk created nfvinfo master: Promote CBS tags update for nfvsig-2.13-release https://review.rdoproject.org/r/c/nfvinfo/+/38567 | 00:06 |
|---|---|---|
| rdogerrit | rdo-trunk created nfvinfo master: Promote CBS tags update for nfvsig-2.16-release https://review.rdoproject.org/r/c/nfvinfo/+/38568 | 00:06 |
| rdogerrit | rdo-trunk proposed rdoinfo master: Bump rdoinfo yoga-uc tags to latest upper-constraints https://review.rdoproject.org/r/c/rdoinfo/+/38513 | 00:10 |
| rdogerrit | rdo-trunk created rdoinfo master: Promote CBS tags update for xena-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/38569 | 00:11 |
| rdogerrit | rdo-trunk created rdoinfo master: Promote CBS tags update for wallaby-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/38570 | 00:12 |
| rdogerrit | rdo-trunk created rdoinfo master: Promote CBS tags update for victoria-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/38571 | 00:15 |
| marios | anyone know if there is an issue with rdo gerrit? | 07:40 |
| marios | cant vote on rdo-jobs 'internal server error' | 07:40 |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for victoria-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/38571 | 08:05 |
| *** amoralej|off is now known as amoralej | 08:05 | |
| rdogerrit | Merged config master: Add CS9 jobs to weirdo repos https://review.rdoproject.org/r/c/config/+/38563 | 08:09 |
| rdogerrit | Alfredo Moralejo proposed rdo-jobs master: Install tox in CentOS 9 nodes running weirdo https://review.rdoproject.org/r/c/rdo-jobs/+/38562 | 08:11 |
| rdogerrit | Merged rdo-jobs master: Move periodic-cloudsig-* jobs to CS8 https://review.rdoproject.org/r/c/rdo-jobs/+/38526 | 08:14 |
| jcapitao[m] | marios: yeah, it should be ok now, infra team fixed it | 08:17 |
| rdogerrit | Merged config master: Update tripleo-ci regular jobs for CentOS 9 https://review.rdoproject.org/r/c/config/+/38544 | 08:21 |
| rdogerrit | Alfredo Moralejo proposed rdoinfo master: Promote CBS tags update for wallaby-8s-testing https://review.rdoproject.org/r/c/rdoinfo/+/38150 | 08:21 |
| rdogerrit | Merged config master: Update tripleo projects for CentOS 9 OVB https://review.rdoproject.org/r/c/config/+/38545 | 08:21 |
| marios | thanks jcapitao[m] yeah seems to be ok now | 08:22 |
| rdogerrit | Merged config master: Update OVB repo with OVB check jobs https://review.rdoproject.org/r/c/config/+/38553 | 08:22 |
| rdogerrit | Alfredo Moralejo proposed rdoinfo master: Bump tempest to 28.1.0 in Victoria https://review.rdoproject.org/r/c/rdoinfo/+/33101 | 08:24 |
| rdogerrit | Merged nfvinfo master: Promote CBS tags update for nfvsig-2.13-release https://review.rdoproject.org/r/c/nfvinfo/+/38567 | 08:24 |
| rdogerrit | Merged nfvinfo master: Promote CBS tags update for nfvsig-2.16-release https://review.rdoproject.org/r/c/nfvinfo/+/38568 | 08:24 |
| rdogerrit | Merged rdoinfo master: Bump rdoinfo yoga-uc tags to latest upper-constraints https://review.rdoproject.org/r/c/rdoinfo/+/38513 | 08:29 |
| *** anbanerj is now known as frenzyfriday | 08:35 | |
| *** jpena|off is now known as jpena | 08:38 | |
| rdogerrit | Alfredo Moralejo created rdoinfo master: Pin scciclient in ussuri to last known good commit https://review.rdoproject.org/r/c/rdoinfo/+/38573 | 08:40 |
| rdogerrit | Fabien Boucher proposed config master: removing tripleo-downstream-trigger-nested-virt from replication https://review.rdoproject.org/r/c/config/+/38386 | 08:44 |
| rdogerrit | Fabien Boucher proposed config master: removing tripleo-downstream-trigger-nested-virt from replication https://review.rdoproject.org/r/c/config/+/38386 | 09:05 |
| rdogerrit | Fabien Boucher proposed config master: removing tripleo-downstream-trigger-nested-virt from replication https://review.rdoproject.org/r/c/config/+/38386 | 09:18 |
| rdogerrit | Fabien Boucher created config master: Add ignore replication for tripleo-downstream-trigger-nested-virt https://review.rdoproject.org/r/c/config/+/38575 | 09:18 |
| rdogerrit | Sagi Shnaidman proposed config master: Update tripleo-quickstart for CentOS 9 https://review.rdoproject.org/r/c/config/+/38555 | 09:18 |
| rdogerrit | Sagi Shnaidman proposed rdo-jobs master: Update OVB jobs templates for CentOS 9 https://review.rdoproject.org/r/c/rdo-jobs/+/38543 | 09:19 |
| rdogerrit | Sagi Shnaidman proposed config master: Update logs and extras CI repos for CentOS 9 https://review.rdoproject.org/r/c/config/+/38554 | 09:19 |
| rdogerrit | Merged nfvinfo master: Promote CBS tags update for nfvsig-2.15-testing https://review.rdoproject.org/r/c/nfvinfo/+/38283 | 10:51 |
| rdogerrit | Merged config master: Add ignore replication for tripleo-downstream-trigger-nested-virt https://review.rdoproject.org/r/c/config/+/38575 | 10:53 |
| rdogerrit | Emma Foley created rdo-jobs master: [WIP] Test whether opstools-rpmbuild job works on Centos9Stream https://review.rdoproject.org/r/c/rdo-jobs/+/38577 | 11:03 |
| rdogerrit | Merged rdoinfo master: Pin scciclient in ussuri to last known good commit https://review.rdoproject.org/r/c/rdoinfo/+/38573 | 11:06 |
| *** rlandy|out is now known as rlandy|ruck | 11:12 | |
| *** dviroel|afk is now known as dviroel | 11:21 | |
| rdogerrit | Merged rdo-jobs master: Install tox in CentOS 9 nodes running weirdo https://review.rdoproject.org/r/c/rdo-jobs/+/38562 | 11:33 |
| rdogerrit | Merged rdo-jobs master: Update tripleo-next-os template jobs https://review.rdoproject.org/r/c/rdo-jobs/+/38564 | 11:33 |
| rdogerrit | Merged rdo-jobs master: Update OVB jobs templates for CentOS 9 https://review.rdoproject.org/r/c/rdo-jobs/+/38543 | 11:38 |
| rdogerrit | rdo-trunk created openstack/ironic-inspector-distgit victoria-rdo: openstack-ironic-inspector-10.4.2-1 https://review.rdoproject.org/r/c/openstack/ironic-inspector-distgit/+/38579 | 11:45 |
| rdogerrit | rdo-trunk created openstack/ironic-python-agent-distgit victoria-rdo: openstack-ironic-python-agent-6.4.4-1 https://review.rdoproject.org/r/c/openstack/ironic-python-agent-distgit/+/38580 | 11:45 |
| rdogerrit | rdo-trunk created openstack/ironic-distgit victoria-rdo: openstack-ironic-16.0.4-1 https://review.rdoproject.org/r/c/openstack/ironic-distgit/+/38581 | 11:51 |
| rdogerrit | rdo-trunk created openstack/horizon-distgit victoria-rdo: python-django-horizon-18.6.3-1 https://review.rdoproject.org/r/c/openstack/horizon-distgit/+/38583 | 12:01 |
| rdogerrit | Alfredo Moralejo created deps/python-amqp c9s-yoga-rdo: Update to python-amqp-5.0.9-1 https://review.rdoproject.org/r/c/deps/python-amqp/+/38589 | 12:24 |
| rdogerrit | Sagi Shnaidman proposed config master: Update tripleo-quickstart for CentOS 9 https://review.rdoproject.org/r/c/config/+/38555 | 13:00 |
| rdogerrit | Sagi Shnaidman proposed config master: Update logs and extras CI repos for CentOS 9 https://review.rdoproject.org/r/c/config/+/38554 | 13:04 |
| efoley | Hi folks, I'm trying to get the opstools distgit jobs to run CentOS 9 Stream builds. I've passed the correct target (opstools9s-collectd-5-el9s), however, the build still kind thinks it's on CentOS 8, (rhel and centos macros equal 8). I made an update for the srpm part, but now it seems that the issue is in cbs, https://review.rdoproject.org/zuul/build/89881ad881ad4b9c8517e71980160645/console | 13:04 |
| efoley | Any pointers onthe above would be much appreciated, the relevant review(s) and info are here: https://review.rdoproject.org/r/c/centos-opstools/collectd/+/37812 | 13:06 |
| efoley | At the moment the distgit jobs need to be run on CentOS8Stream because the required packages aren't in CentOS9Stream, but this shouldn't be a problem in theory, since the build is kicked off for CBS | 13:07 |
| efoley | That is, the required packages in the distgit jobs for running pkg-validate, etc | 13:07 |
| *** amoralej is now known as amoralej|lunch | 13:09 | |
| rdogerrit | Merged deps/python-amqp c9s-yoga-rdo: Update to python-amqp-5.0.9-1 https://review.rdoproject.org/r/c/deps/python-amqp/+/38589 | 13:24 |
| rdogerrit | Douglas Viroel proposed rdo-jobs master: Add playbook to setup compose-repos https://review.rdoproject.org/r/c/rdo-jobs/+/36893 | 13:36 |
| rdogerrit | Douglas Viroel proposed rdo-jobs master: Add CentOs compose dependency jobs https://review.rdoproject.org/r/c/rdo-jobs/+/37788 | 13:36 |
| *** dasm|off is now known as dasm | 13:48 | |
| tobias-urdin | can we get https://review.rdoproject.org/r/c/rdoinfo/+/38150 through so we get 12.1.0 magnum soon :) | 13:55 |
| *** amoralej|lunch is now known as amoralej | 13:59 | |
| amoralej | meeting time | 14:00 |
| spotz | tobias-urdin: looks like it's got +w | 14:00 |
| spotz | o/ | 14:00 |
| amoralej | tobias-urdin, we'll have problems with the promotion to release | 14:00 |
| amoralej | #startmeeting RDO meeting - 2022-01-26 | 14:01 |
| opendevmeet | Meeting started Wed Jan 26 14:01:24 2022 UTC and is due to finish in 60 minutes. The chair is amoralej. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'rdo_meeting___2022_01_26' | 14:01 |
| amoralej | #topic roll call | 14:01 |
| amoralej | who is around? | 14:01 |
| PagliaccisCloud | O/ | 14:03 |
| amoralej | #chair PagliaccisCloud | 14:03 |
| opendevmeet | Current chairs: PagliaccisCloud amoralej | 14:03 |
| amoralej | spotz, kkula jcapitao[m] are you around? | 14:03 |
| kkula | yes! | 14:03 |
| jcapitao[m] | o/ | 14:03 |
| kkula | hello | 14:04 |
| amoralej | #chair kkula jcapitao[m] | 14:04 |
| opendevmeet | Current chairs: PagliaccisCloud amoralej jcapitao[m] kkula | 14:04 |
| amoralej | feel free to add topics to https://etherpad.opendev.org/p/RDO-Meeting | 14:05 |
| amoralej | i'll start with first ones | 14:05 |
| amoralej | #topic introduce new RDOers | 14:05 |
| amoralej | #info kkula join RDO team, welcome!!! | 14:05 |
| amoralej | so, kkula will be contributing to RDO | 14:06 |
| kkula | hi all RDOers! | 14:06 |
| amoralej | i hope you have a nice onboarding :) | 14:06 |
| jcapitao[m] | welcome kkula ! :) | 14:06 |
| amoralej | and feel free to ask us anything you need | 14:07 |
| spotz | Wellcome kkula!!! | 14:07 |
| dpawlik | hey kkula! | 14:07 |
| amoralej | ok, let's move on | 14:08 |
| amoralej | #topic Devconf Meetup Friday at 17:00 UTC - https://www.devconf.info/cz/ | 14:08 |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for wallaby-8s-testing https://review.rdoproject.org/r/c/rdoinfo/+/38150 | 14:09 |
| amoralej | spotz, stage is yours | 14:09 |
| PagliaccisCloud | Welcome kkula!!! ٩( ᐛ )و | 14:09 |
| spotz | Please join in on Friday, opportunity to just hang out, answer any questions we might get and some SWAG giveouts from our stash:) | 14:09 |
| rdogerrit | Douglas Viroel created rdo-jobs master: WIP - Adding multinode and build-images jobs def https://review.rdoproject.org/r/c/rdo-jobs/+/38596 | 14:10 |
| spotz | That's it for that really:) | 14:10 |
| rdogerrit | Douglas Viroel proposed rdo-jobs master: WIP - Adding multinode and build-images jobs def https://review.rdoproject.org/r/c/rdo-jobs/+/38596 | 14:10 |
| amoralej | spotz, so RDO has some virtual booth? | 14:11 |
| amoralej | or something? | 14:11 |
| spotz | We have a 1 hour meetup slot | 14:11 |
| spotz | The decision was no booths because it was supposed to be live but they'd have meetups | 14:11 |
| amoralej | ok | 14:12 |
| amoralej | meetup is in hopin platform too? | 14:12 |
| amoralej | ah, i see it | 14:12 |
| amoralej | good | 14:12 |
| amoralej | https://devconfcz2022.sched.com/event/sjOE/rdo-meetup | 14:12 |
| spotz | I suspect it'll be as quiet as a booth:( But It'll be nice to have some hangout space even virtually | 14:12 |
| amoralej | yeah | 14:12 |
| amoralej | good to meet there | 14:13 |
| spotz | Yep | 14:13 |
| amoralej | btw, it's a bit weird to do virtual events in saturday? | 14:13 |
| amoralej | or it's just me :) | 14:13 |
| amoralej | anyway | 14:13 |
| rdogerrit | Merged openstack/ironic-python-agent-distgit victoria-rdo: openstack-ironic-python-agent-6.4.4-1 https://review.rdoproject.org/r/c/openstack/ironic-python-agent-distgit/+/38580 | 14:13 |
| spotz | I think because historically those are the days in person they kept them. Even FOSDEM will still have weekend stuff | 14:14 |
| amoralej | #action everyone Join in virtual RDO Meetup on devconf.cz on friday! it's an opportunity to just hang out, answer any questions we might get and some SWAG giveouts from our stash :) | 14:15 |
| amoralej | to get it into minutes :) | 14:15 |
| spotz | :) | 14:15 |
| amoralej | thanks spotz, let's move to next mtg | 14:15 |
| amoralej | yeah, i thought the same about fosdem | 14:15 |
| amoralej | #topic Dependencies update in Yoga | 14:16 |
| amoralej | i think it'd be good to do a dependencies refresh at this time for Yoga | 14:16 |
| amoralej | i've created https://review.rdoproject.org/etherpad/p/update-deps-yoga-cs9 | 14:16 |
| amoralej | and kkula will start working on it | 14:17 |
| amoralej | so, you'll see reviews coming to the deps repos :) | 14:17 |
| jcapitao[m] | ok good | 14:17 |
| amoralej | #info https://review.rdoproject.org/etherpad/p/update-deps-yoga-cs9 | 14:17 |
| jcapitao[m] | should we wait for review to land in Fedora ? | 14:18 |
| amoralej | #action kkula will work on updating dependencies for Yoga based on https://dashboards.rdoproject.org/report-uc-cs9 | 14:18 |
| amoralej | jcapitao[m], yes | 14:18 |
| jcapitao[m] | or we can act in parallel ? | 14:18 |
| amoralej | unless there is some special need | 14:18 |
| jcapitao[m] | ok | 14:18 |
| amoralej | i've asked kkula to start with the deps which are already updated in fedora | 14:19 |
| jcapitao[m] | b/c sometime patches are dying opened | 14:19 |
| jcapitao[m] | ok that should help us to catch up fast | 14:20 |
| amoralej | yep | 14:20 |
| amoralej | let's see how many can we update | 14:21 |
| amoralej | btw, we'll try to build for cs8 from the same distgits | 14:21 |
| amoralej | to align cs8 and cs9 | 14:21 |
| jcapitao[m] | ok | 14:21 |
| amoralej | mmm i'm thinking if we may create a check job to do scratch build in cs8 | 14:22 |
| jcapitao[m] | and if it fails ? | 14:22 |
| amoralej | from updates in cs9 branch | 14:22 |
| amoralej | non-voting | 14:22 |
| jcapitao[m] | ah yes | 14:22 |
| jcapitao[m] | worth it to add it | 14:22 |
| amoralej | #action add a new non-voting job deps-cbs-validate-cs8 to scratch build in cs8 BR from updates in c9s branches | 14:23 |
| amoralej | btw, it could be a good oportunity to update https://www.rdoproject.org/documentation/requirements/#updating-a-requirement-in-rdo-cloudsig-repositories | 14:25 |
| amoralej | and get it to the same level of adding a new one | 14:25 |
| amoralej | with diagram and so on | 14:25 |
| amoralej | kkula, ^ wdyt? | 14:25 |
| amoralej | would you mind to update the procedure to update requirements? | 14:25 |
| amoralej | no need to do it now | 14:25 |
| spotz | Good idea, I'll also try to carve some time out to make sure we're up to date for saying stream, etc | 14:26 |
| amoralej | but once you are familiar with the procedure | 14:26 |
| amoralej | kkula, fyi website content is maintained using gerrit with markdown https://review.rdoproject.org/r/c/rdo-website/+/37937 | 14:28 |
| amoralej | that's a recent update from jcapitao[m] | 14:28 |
| kkula | i'll take a look | 14:28 |
| amoralej | anyway, i'd recomend you to do some deps updates to understand the process first and then update | 14:29 |
| amoralej | next topic | 14:29 |
| amoralej | #topic Move out puppet promotion pipeline from Jenkins to Zuul | 14:29 |
| amoralej | that's from jcapitao[m] | 14:29 |
| jcapitao[m] | as we've already did for CS9, we can move CS8 pipelines to zuul now | 14:30 |
| jcapitao[m] | this will help us to prevent hitting CICO infra issue, hopefully | 14:30 |
| jcapitao[m] | we're requesting too many nodes | 14:30 |
| amoralej | we need to find out how to distribute the pipelines | 14:31 |
| jcapitao[m] | in multiple phases you mean ? | 14:32 |
| amoralej | i mean that, if we are adding 4 more pipelines+ | 14:32 |
| amoralej | each one runs 5 nodes | 14:32 |
| amoralej | not to run all of them together | 14:32 |
| jcapitao[m] | right | 14:32 |
| amoralej | for the cs9 ones, i've used the tripleo pipelines | 14:33 |
| amoralej | the ones they have for their promotions | 14:33 |
| amoralej | i think they have more | 14:33 |
| amoralej | let's discuss in the reviews | 14:33 |
| jcapitao[m] | ack | 14:34 |
| amoralej | that's it about this topic? | 14:35 |
| jcapitao[m] | so I'll propose patches and handle the migration | 14:35 |
| rdogerrit | Merged rdo-website master: Fix line breaks issue https://review.rdoproject.org/r/c/rdo-website/+/38325 | 14:36 |
| amoralej | i guess, will be similar to cs8 ones, right? | 14:36 |
| amoralej | btw, i think i still need to enable the one for centos9-wallaby | 14:36 |
| jcapitao[m] | yes similar to cs9 ones | 14:37 |
| jcapitao[m] | right, we did it for X, not yet for W | 14:38 |
| amoralej | yep | 14:38 |
| opendevreview | Alfredo Moralejo proposed x/packstack master: Move nova database connection config to nova::db https://review.opendev.org/c/x/packstack/+/826476 | 14:38 |
| amoralej | #topic OIF Summit! CFP is open!!! | 14:39 |
| amoralej | spotz, something to add? | 14:39 |
| spotz | Please submit talks! | 14:39 |
| amoralej | #link https://cfp.openinfra.dev/app/berlin-2022 | 14:40 |
| spotz | Would love to have an RDO presence in the talks and I'm also planning for a social and some sponsoring of the OPS Meetup | 14:40 |
| spotz | Not sure what to submit reach out I've got tons of ideas:) | 14:40 |
| spotz | Need a co-speaker cause you don't want to talk alone I'll do it or find you someone | 14:41 |
| spotz | That's it:) | 14:41 |
| amoralej | thans spotz | 14:41 |
| amoralej | #topic CentOS Cloud SiG | 14:42 |
| spotz | After last weeks meeting I talked with the Fedora Cloud SiG folks. CCentOS Cloud SiG will meet the second thursday of the month and Fedora Cloud SiG will meet the first and third so anyone who wants to participate in both can | 14:43 |
| amoralej | nice | 14:43 |
| spotz | The goal is that this move will grow the SiG beyond just RDO | 14:43 |
| spotz | The 2 groups aren't a 1 to 1 matching on scope but that's ok | 14:44 |
| spotz | We're also up for the newsletter:) | 14:44 |
| amoralej | yeah | 14:44 |
| amoralej | did you send patch for the calendar? | 14:44 |
| amoralej | i mean https://www.centos.org/community/calendar/ | 14:44 |
| spotz | That's it for Cloud SiG I just wanted to bring it up here as we meet more often then the SiG | 14:44 |
| spotz | Yeah patch was sent, we foound an issue with another SiG. I'll poke to get it merged | 14:45 |
| amoralej | ack, good | 14:45 |
| rdogerrit | Joel Capitao created config master: Add puppet-promotion jobs for Wallaby in openstack-periodic-daily https://review.rdoproject.org/r/c/config/+/38597 | 14:45 |
| amoralej | thanks for arranging that | 14:45 |
| spotz | I think we got sidetracked fixing that:) | 14:45 |
| spotz | My pleasure | 14:45 |
| amoralej | yeah, i remembered you sent the patch, that's why i asked :) | 14:46 |
| spotz | https://github.com/CentOS/Calendar/pull/52 | 14:46 |
| amoralej | #topic chair for next week | 14:47 |
| amoralej | next week is video call meeting | 14:47 |
| amoralej | anyone willing to chair? | 14:47 |
| spotz | I should be able to | 14:48 |
| amoralej | #action spotz to chair next meeting | 14:48 |
| amoralej | #topic open floor | 14:48 |
| amoralej | anything else you'd like to share? | 14:48 |
| jcapitao[m] | nothing else from me | 14:50 |
| amoralej | ok, i'll give you some minutes back | 14:50 |
| amoralej | thanks all for joining! | 14:50 |
| amoralej | #endmeeting | 14:50 |
| opendevmeet | Meeting ended Wed Jan 26 14:50:44 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:50 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/rdo_meeting___2022_01_26/2022/rdo_meeting___2022_01_26.2022-01-26-14.01.html | 14:50 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/rdo_meeting___2022_01_26/2022/rdo_meeting___2022_01_26.2022-01-26-14.01.txt | 14:50 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/rdo_meeting___2022_01_26/2022/rdo_meeting___2022_01_26.2022-01-26-14.01.log.html | 14:50 |
| spotz | Thanks amoralej, thanks everyone for coming and welcome kkula!!! | 14:51 |
| amoralej | efoley, according to logs in https://cbs.centos.org/koji/taskinfo?taskID=2659150 | 14:51 |
| jcapitao[m] | thx amoralej | 14:52 |
| rdogerrit | Merged openstack/ironic-inspector-distgit victoria-rdo: openstack-ironic-inspector-10.4.2-1 https://review.rdoproject.org/r/c/openstack/ironic-inspector-distgit/+/38579 | 14:53 |
| *** dviroel is now known as dviroel|lunch | 15:01 | |
| rdogerrit | Alfredo Moralejo created rdo-jobs master: Remove distgit-cbs-validate-centos8 jobs from distgit gates https://review.rdoproject.org/r/c/rdo-jobs/+/38598 | 15:04 |
| rdogerrit | Alfredo Moralejo proposed rdo-jobs master: Remove distgit-cbs-validate-centos8 jobs from distgit gates https://review.rdoproject.org/r/c/rdo-jobs/+/38598 | 15:04 |
| amoralej | efoley, i left a comment in your review, apparently the job is working fine | 15:06 |
| amoralej | i'd say it's building in the expected tag | 15:06 |
| efoley | amoralej: The rhel and centos macros are not set correctly, they should be 9, and not 8 | 15:07 |
| amoralej | which macros? | 15:08 |
| efoley | In the specfile. | 15:08 |
| amoralej | mmm lemme check again | 15:08 |
| efoley | There's a %dump in the specfile, and it's showing centos = 8 and rhel = 8, the build checks %if 0%{?rhel} > 8 in some cases, and those bits are being skipped | 15:09 |
| amoralej | efoley, so you have a if %{?rhel} > 8 | 15:09 |
| amoralej | and it's getting 9 in centos9 | 15:10 |
| rdogerrit | Merged openstack/horizon-distgit victoria-rdo: python-django-horizon-18.6.3-1 https://review.rdoproject.org/r/c/openstack/horizon-distgit/+/38583 | 15:10 |
| efoley | amoralej: It appears to be getting 8 in centos9 | 15:10 |
| amoralej | i don't think so | 15:10 |
| amoralej | actually the build is failing because the patch which is in the conditional for centos9 is missing from the distgit | 15:11 |
| amoralej | Bad file: /chroot_tmpdir/srpm_unpacked/SOURCES/collectd-remove-des-support-from-snmp-plugin.patch: No such file or directory | 15:11 |
| amoralej | https://review.rdoproject.org/r/c/centos-opstools/collectd/+/37812/18/collectd.spec#150 | 15:11 |
| amoralej | that's in if %{rhel} > 8 | 15:12 |
| amoralej | so that's getting 9 | 15:12 |
| efoley | amoralej: That file is in the distgit repo. Are there additional steps required to add it to the distgit | 15:12 |
| amoralej | mmmm, lemme check the content | 15:12 |
| amoralej | it must not being added to the src.rpm | 15:14 |
| amoralej | ahhh | 15:16 |
| amoralej | efoley, i understood you now | 15:16 |
| amoralej | so yes | 15:16 |
| amoralej | the problem is with the macros when building the srpm | 15:17 |
| amoralej | in the job | 15:17 |
| amoralej | not when bulding the actual packages from the srpm in cbs | 15:17 |
| amoralej | got it now | 15:17 |
| amoralej | mmm | 15:17 |
| amoralej | lemme check how we are doing it | 15:17 |
| amoralej | we may be having the same issue but didn't catch because we may not have a patch in a rhel conditional | 15:18 |
| amoralej | we may add --define 'rhel 9' to the rpmbuild command | 15:19 |
| amoralej | but, we may hit with other macros, it's kind of corner case tbh | 15:19 |
| efoley | amoralej: Shouldn't that be inherited when you set the target distro? | 15:20 |
| amoralej | no | 15:20 |
| amoralej | ok, let me explaini | 15:20 |
| amoralej | i mean the cbs build part is working perfectly | 15:20 |
| amoralej | the problem is in the content of the srpm | 15:20 |
| amoralej | the srpm is created locally using rpmbuild command | 15:20 |
| spotz | Calendar change merged amoralej | 15:20 |
| amoralej | efoley, look at https://logserver.rdoproject.org/12/37812/18/check/centos-opstools-collectd-5-distgit-cbs-validate-centos9s/6d9fc2e/job-output.txt | 15:21 |
| amoralej | look for commaned "rpmbuild --define 'dist .el9s' -bs ./collectd.spec" | 15:21 |
| amoralej | that's far before running cbs | 15:21 |
| rdogerrit | Merged openstack/glance-distgit rpm-master: Require python3-sqlalchemy >= 1.4.18 https://review.rdoproject.org/r/c/openstack/glance-distgit/+/38008 | 15:21 |
| efoley | amoralej, that's what I changed yesterday, IINM. | 15:21 |
| amoralej | that's just creating the srpm that will be later in command cbs build --scratch opstools9s-collectd-5-el9s /home/zuul/rpmbuild/SRPMS/collectd-5.12.0-7.el9s.src.rpm | 15:22 |
| amoralej | but | 15:22 |
| amoralej | rpmbuild has no notion of tags | 15:22 |
| amoralej | just creates a srpm locally | 15:22 |
| amoralej | and in that command macros are got from the local env | 15:22 |
| amoralej | so rhel=8 | 15:23 |
| amoralej | so patch is not included | 15:23 |
| amoralej | in the srpm | 15:23 |
| efoley | Ah, so the dist and rhel values are not related | 15:23 |
| amoralej | nop | 15:23 |
| amoralej | see | 15:24 |
| amoralej | 2022-01-26 11:40:30.463845 | rdo-centos-stream | -13= rhel8 | 15:24 |
| amoralej | maybe you could add one more --difine 'rhel 9' in rpmbuild | 15:24 |
| amoralej | other option would be to use mock to create srpms | 15:25 |
| amoralej | instead of pure rpmbuild | 15:25 |
| efoley | amoralej: isn't that generally recommended? | 15:25 |
| amoralej | or to change that conditional and make it depend on dist instead of rhel, but that'd be tricky :) | 15:25 |
| amoralej | efoley, well, we try to keep stuff as simple as possible and using rpmbuild is simpler | 15:26 |
| amoralej | but yes, it can lead to this kind of issues | 15:26 |
| amoralej | the point is that the srpm is rebuilt later in cbs | 15:26 |
| amoralej | with the right macros | 15:26 |
| amoralej | but, if the rhel conditional affect to the files included in the srpm | 15:27 |
| amoralej | as in this particular case, it can break it | 15:27 |
| efoley | amoralej, I'd imagine that more of this will be seen as more packages are built for centos9s | 15:27 |
| efoley | however, opstools might be the only ones doing this | 15:28 |
| amoralej | no no | 15:28 |
| amoralej | we are building centos9 packages all the time | 15:28 |
| amoralej | but we didn't have a case like this one | 15:28 |
| amoralej | where a Patchxx in included in a %{rhel} conditional | 15:29 |
| amoralej | it's not a common case | 15:29 |
| efoley | We're using the same file to build both c9s and c8s, instead of having different branches for different releases, which is the norm IIUC | 15:30 |
| amoralej | efoley, https://review.rdoproject.org/r/c/deps/python-amqp/+/38589 | 15:30 |
| amoralej | that's build for cs9, i.e | 15:30 |
| amoralej | yeah, that's not bad | 15:30 |
| rdogerrit | Alfredo Moralejo proposed gating_scripts master: Override packages from modules in packages validation https://review.rdoproject.org/r/c/gating_scripts/+/38384 | 15:37 |
| rdogerrit | Alfredo Moralejo created gating_scripts master: Set rhel macro when creating srpm based on TARGET https://review.rdoproject.org/r/c/gating_scripts/+/38599 | 15:37 |
| amoralej | efoley, ^ may you test this with a Depends-On in your patch? | 15:37 |
| efoley | amoralej: can do. | 15:38 |
| rdogerrit | Merged gating_scripts master: Override packages from modules in packages validation https://review.rdoproject.org/r/c/gating_scripts/+/38384 | 15:38 |
| opendevreview | Alfredo Moralejo proposed x/packstack master: Move nova database connection config to nova::db https://review.opendev.org/c/x/packstack/+/826476 | 15:40 |
| rdogerrit | Joel Capitao created rdo-jobs master: [WIP] Add CentOS8 Yoga puppet promotion pipeline https://review.rdoproject.org/r/c/rdo-jobs/+/38600 | 15:41 |
| rdogerrit | Joel Capitao proposed rdo-jobs master: [WIP] Add CentOS8 Yoga puppet promotion pipeline https://review.rdoproject.org/r/c/rdo-jobs/+/38600 | 15:44 |
| rdogerrit | Emma Foley proposed rdo-jobs master: [centos-opstools] Pass TARGET into pkg-{validate,export}.sh in distgit jobs https://review.rdoproject.org/r/c/rdo-jobs/+/38532 | 15:45 |
| efoley | amoralej: Thanks for your help. | 15:53 |
| rdogerrit | User marios created config master: Drop integration-pipeline-main-centos8 - no c8 for master https://review.rdoproject.org/r/c/config/+/38601 | 15:57 |
| rdogerrit | User marios created rdo-jobs master: Remove integration-pipeline-main-centos8.yaml - no c8/master https://review.rdoproject.org/r/c/rdo-jobs/+/38602 | 15:57 |
| amoralej | efoley, at least this time is passing but rebuildSRPM task https://cbs.centos.org/koji/taskinfo?taskID=2662536 | 15:59 |
| efoley | amoralej: That's progress! I am expecting the job to fail based on missing packages in c9s, but so far I'm being surprised | 16:03 |
| amoralej | efoley, build passed for x86_64 in c9 | 16:03 |
| amoralej | :) | 16:03 |
| amoralej | failed in c8 because of missing buildrequires | 16:03 |
| efoley | amoralej: c8 is okay, that was happening before, I still need to look at that. Thanks for your help! This is a big blocker cleared | 16:10 |
| *** dviroel|lunch is now known as dviroel | 16:23 | |
| rdogerrit | Alfredo Moralejo created rdo-jobs master: Move jobs running on rdo-centos-8 to rdo-centos-8-stream https://review.rdoproject.org/r/c/rdo-jobs/+/38604 | 16:27 |
| rdogerrit | Alfredo Moralejo proposed rdo-jobs master: Move jobs running on rdo-centos-8 to rdo-centos-8-stream https://review.rdoproject.org/r/c/rdo-jobs/+/38604 | 16:28 |
| *** artom is now known as artom_not_the_sdk_one | 16:28 | |
| *** artom_not_the_sdk_one is now known as artom | 16:29 | |
| rdogerrit | Joel Capitao created rdo-jobs master: Fix CentOS9 Xena puppet promotion job https://review.rdoproject.org/r/c/rdo-jobs/+/38605 | 16:32 |
| rdogerrit | Alfredo Moralejo created rdo-jobs master: Use CS8 in rdoinfo-DLRN-check-centos8 job https://review.rdoproject.org/r/c/rdo-jobs/+/38606 | 16:40 |
| rdogerrit | Alfredo Moralejo proposed rdo-jobs master: Use CS8 in rdoinfo-DLRN-check-centos8 job https://review.rdoproject.org/r/c/rdo-jobs/+/38606 | 16:41 |
| amoralej | jcapitao[m], https://review.rdoproject.org/r/q/topic:cl8+status:open if you have a chance | 16:43 |
| rdogerrit | Merged rdo-jobs master: Fix CentOS9 Xena puppet promotion job https://review.rdoproject.org/r/c/rdo-jobs/+/38605 | 16:49 |
| jcapitao[m] | amoralej: I merged them | 16:49 |
| jcapitao[m] | there is also https://review.rdoproject.org/r/c/config/+/38597 | 16:50 |
| rdogerrit | Merged rdo-jobs master: Remove distgit-cbs-validate-centos8 jobs from distgit gates https://review.rdoproject.org/r/c/rdo-jobs/+/38598 | 16:54 |
| jcapitao[m] | amoralej: wrt https://review.opendev.org/c/x/packstack/+/826476 we need to handle https://review.opendev.org/c/openstack/puppet-aodh/+/825961 | 17:01 |
| jcapitao[m] | I can propose new PS if you want | 17:02 |
| amoralej | yes please | 17:03 |
| rdogerrit | Merged config master: Add puppet-promotion jobs for Wallaby in openstack-periodic-daily https://review.rdoproject.org/r/c/config/+/38597 | 17:03 |
| rdogerrit | Alfredo Moralejo proposed rdo-jobs master: Move jobs running on rdo-centos-8 to rdo-centos-8-stream https://review.rdoproject.org/r/c/rdo-jobs/+/38604 | 17:05 |
| opendevreview | Joel Capitao proposed x/packstack master: Move nova and aodh database connection config to ::db https://review.opendev.org/c/x/packstack/+/826476 | 17:08 |
| rdogerrit | Joel Capitao proposed rdo-jobs master: Use CS8 in rdoinfo-DLRN-check-centos8 job https://review.rdoproject.org/r/c/rdo-jobs/+/38606 | 17:17 |
| rdogerrit | Merged rdo-jobs master: Move jobs running on rdo-centos-8 to rdo-centos-8-stream https://review.rdoproject.org/r/c/rdo-jobs/+/38604 | 17:26 |
| *** jpena is now known as jpena|off | 17:44 | |
| *** sshnaidm is now known as sshnaidm|afk | 17:49 | |
| *** amoralej is now known as amoralej|off | 17:53 | |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for xena-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/38569 | 18:03 |
| rdogerrit | Dariusz created config master: Fixed typo https://review.rdoproject.org/r/c/config/+/38610 | 19:30 |
| *** dviroel is now known as dviroel|afk | 21:06 | |
| *** dasm is now known as dasm|off | 23:24 | |
| *** rlandy|ruck is now known as rlandy|out | 23:41 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!