rdogerrit | rdo-trunk proposed rdoinfo master: Promote CBS tags update for zed-9s-release https://review.rdoproject.org/r/c/rdoinfo/+/47014 | 00:21 |
---|---|---|
rdogerrit | rdo-trunk proposed nfvinfo master: Promote CBS tags update for nfvsig-openvswitch2.17-release https://review.rdoproject.org/r/c/nfvinfo/+/47015 | 00:22 |
rdogerrit | rdo-trunk proposed rdoinfo master: Promote CBS tags update for yoga-9s-release https://review.rdoproject.org/r/c/rdoinfo/+/47016 | 00:22 |
rdogerrit | rdo-trunk proposed rdoinfo master: Promote CBS tags update for yoga-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/47017 | 00:23 |
rdogerrit | rdo-trunk proposed rdoinfo master: Promote CBS tags update for xena-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/47018 | 00:25 |
rdogerrit | rdo-trunk proposed rdoinfo master: Promote CBS tags update for wallaby-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/47019 | 00:25 |
rdogerrit | rdo-trunk proposed deps/python-daiquiri c9s-antelope-rdo: Update to 3.2.1 https://review.rdoproject.org/r/c/deps/python-daiquiri/+/47020 | 00:27 |
rdogerrit | Merged rdo-jobs master: Replace with_items by loop keyword. https://review.rdoproject.org/r/c/rdo-jobs/+/46861 | 06:24 |
*** amoralej|off is now known as amoralej | 07:13 | |
rdogerrit | Cédric Jeanneret proposed rdo-jobs master: Clean some of the playbook https://review.rdoproject.org/r/c/rdo-jobs/+/47033 | 08:13 |
rdogerrit | Merged rdoinfo master: Promote CBS tags update for yoga-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/47017 | 08:15 |
rdogerrit | Merged rdoinfo master: Promote CBS tags update for wallaby-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/47019 | 08:15 |
rdogerrit | Merged config master: Build ansiblee and meta operator images https://review.rdoproject.org/r/c/config/+/47002 | 08:22 |
rdogerrit | Merged rdo-jobs master: Properly lock the firewall, adding some security https://review.rdoproject.org/r/c/rdo-jobs/+/46924 | 08:23 |
rdogerrit | Merged rdoinfo master: Promote CBS tags update for xena-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/47018 | 08:24 |
rdogerrit | Merged rdoinfo master: Promote CBS tags update for zed-9s-release https://review.rdoproject.org/r/c/rdoinfo/+/47014 | 08:24 |
rdogerrit | Merged rdoinfo master: Promote CBS tags update for yoga-9s-release https://review.rdoproject.org/r/c/rdoinfo/+/47016 | 08:24 |
*** jpena|off is now known as jpena | 08:25 | |
rdogerrit | Merged deps/python-daiquiri c9s-antelope-rdo: Update to 3.2.1 https://review.rdoproject.org/r/c/deps/python-daiquiri/+/47020 | 08:36 |
rdogerrit | Daniel Pawlik proposed rdo-infra/rdo-infra-playbooks master: Remove leftovers after removing registry.rdoproject.org https://review.rdoproject.org/r/c/rdo-infra/rdo-infra-playbooks/+/47034 | 08:43 |
rdogerrit | Cédric Jeanneret proposed rdo-jobs master: Clean some of the playbook https://review.rdoproject.org/r/c/rdo-jobs/+/47033 | 08:47 |
rdogerrit | Cédric Jeanneret proposed rdo-jobs master: Clean some of the playbook https://review.rdoproject.org/r/c/rdo-jobs/+/47033 | 09:11 |
rdogerrit | Cédric Jeanneret proposed config master: We need some more dependencies https://review.rdoproject.org/r/c/config/+/47035 | 09:32 |
rdogerrit | Cédric Jeanneret proposed rdo-jobs master: Clean some of the playbook https://review.rdoproject.org/r/c/rdo-jobs/+/47033 | 09:33 |
rdogerrit | Cédric Jeanneret proposed config master: Dependencies are needed earlier https://review.rdoproject.org/r/c/config/+/47035 | 09:37 |
*** jpena is now known as jpena|off | 09:39 | |
rdogerrit | Merged config master: Dependencies are needed earlier https://review.rdoproject.org/r/c/config/+/47035 | 09:46 |
jcapitao[m] | amoralej: wrt https://review.rdoproject.org/r/c/openstack/glare-distgit/+/47007 | 09:59 |
jcapitao[m] | maybe we should retire glare | 09:59 |
amoralej | did they release in zed? | 10:00 |
jcapitao[m] | I was trying to make %check to work | 10:00 |
amoralej | yeah | 10:00 |
jcapitao[m] | but last commit is 3 years ago | 10:00 |
amoralej | actually last commit is 3 years old ... | 10:00 |
amoralej | https://opendev.org/x/glare | 10:00 |
jcapitao[m] | we haven't release a new NVR it since a while now | 10:01 |
jcapitao[m] | it's py37 | 10:01 |
jcapitao[m] | py39 is not even supported | 10:01 |
jcapitao[m] | well it might but not 100% sure | 10:01 |
jcapitao[m] | it's not explicitly retired though | 10:01 |
amoralej | let's do it | 10:02 |
amoralej | send a mail to rdo ML | 10:02 |
amoralej | before | 10:02 |
amoralej | announcing we are about to retire | 10:02 |
jcapitao[m] | ok lemme write the email | 10:03 |
*** jpena|off is now known as jpena | 10:06 | |
rdogerrit | Joel Capitao proposed rdoinfo master: Retire Glare https://review.rdoproject.org/r/c/rdoinfo/+/47036 | 10:18 |
rdogerrit | Merged openstack/cinderclient-distgit rpm-master: Remove simplejson dependency https://review.rdoproject.org/r/c/openstack/cinderclient-distgit/+/47012 | 10:26 |
rdogerrit | Merged nfvinfo master: Promote CBS tags update for nfvsig-openvswitch2.17-release https://review.rdoproject.org/r/c/nfvinfo/+/47015 | 10:36 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Install tox in cs8 for periodic jobs https://review.rdoproject.org/r/c/rdo-jobs/+/47037 | 10:42 |
jcapitao[m] | amoralej ^ to unblock periodic jobs on CS8 | 10:43 |
amoralej | didn't we fix that in weirdo itself? | 10:44 |
amoralej | apparently not :( | 10:44 |
jcapitao[m] | for the check jobs | 10:44 |
jcapitao[m] | https://review.rdoproject.org/zuul/builds?project=rdoinfo&pipeline=openstack-periodic-integration-stable3 | 10:44 |
jcapitao[m] | but not periodic ones | 10:44 |
jcapitao[m] | maybe we should have put it in pre job ? | 10:44 |
amoralej | right, we call weirdo with tox so we need to fix it before | 10:46 |
amoralej | we don't share any pre-run between periodic and not periodic | 10:48 |
amoralej | so, unless we create a new common parent for all weirdo, we don't have a single place to fix | 10:48 |
jcapitao[m] | yeah I was checking | 10:48 |
jcapitao[m] | the "pre" we have is base/pre.yaml | 10:49 |
jcapitao[m] | which is too general | 10:49 |
amoralej | yeah, i proposed initially to fix it in base/pre.yaml but it'd affect all jobs | 10:50 |
amoralej | the best option would have been adding it to the image but i couldn't figure out how out of dib | 10:50 |
jcapitao[m] | right | 10:52 |
jcapitao[m] | so we need to add another pre | 10:53 |
jcapitao[m] | common to weirdo buildsys-tags and periodic | 10:53 |
jcapitao[m] | lemme propose a patch | 10:53 |
*** dviroel|out is now known as dviroel | 10:58 | |
amoralej | jcapitao[m], so, will you create a new weirdo common base job with pre-run ? | 10:59 |
jcapitao[m] | yes, sounds good ? | 11:02 |
amoralej | jcapitao[m], sure, but then we should check which part of current playbooks we may move there | 11:08 |
rdogerrit | Cédric Jeanneret proposed rdo-jobs master: Clean some of the playbook https://review.rdoproject.org/r/c/rdo-jobs/+/47033 | 11:11 |
jcapitao[m] | amoralej: yeah, the refactoring is not that small | 11:28 |
jcapitao[m] | there is weirdo integration too | 11:28 |
jcapitao[m] | in playbooks/weirdo/* | 11:28 |
amoralej | yes, those are the ones we use to test weirdo itself | 11:29 |
rdogerrit | Marios Andreou proposed rdo-jobs master: WIP adding job definition for data_plane_adoption job and nodeset https://review.rdoproject.org/r/c/rdo-jobs/+/46877 | 12:14 |
rdogerrit | Chandan Kumar proposed rdo-jobs master: [DNM] openstack operator status check https://review.rdoproject.org/r/c/rdo-jobs/+/47038 | 12:29 |
rdogerrit | Chandan Kumar proposed rdo-jobs master: [DNM] openstack operator status check https://review.rdoproject.org/r/c/rdo-jobs/+/47038 | 12:31 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Consolidate weirdo playbooks https://review.rdoproject.org/r/c/rdo-jobs/+/47039 | 13:03 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Consolidate weirdo playbooks https://review.rdoproject.org/r/c/rdo-jobs/+/47039 | 13:04 |
*** dasm|off is now known as dasm | 13:10 | |
rdogerrit | Joel Capitao proposed rdo-infra/ansible-role-weirdo-common master: WIP testing 47039 patch https://review.rdoproject.org/r/c/rdo-infra/ansible-role-weirdo-common/+/47040 | 13:11 |
rdogerrit | Joel Capitao proposed rdoinfo master: DNM testing 47039 patch for validating buildsys-tags changes https://review.rdoproject.org/r/c/rdoinfo/+/47041 | 13:20 |
rdogerrit | Chandan Kumar proposed config master: Update meta operator makefile to use custom-bundle.Dockerfile https://review.rdoproject.org/r/c/config/+/47042 | 13:23 |
*** amoralej is now known as amoralej|lunch | 13:24 | |
rdogerrit | Joel Capitao proposed rdo-jobs master: Consolidate weirdo playbooks https://review.rdoproject.org/r/c/rdo-jobs/+/47039 | 13:25 |
rdogerrit | Merged config master: Update meta operator makefile to use custom-bundle.Dockerfile https://review.rdoproject.org/r/c/config/+/47042 | 13:35 |
rdogerrit | Amol Kahat proposed rdo-infra/ci-config master: Added molecule tests for new infra roles https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/46680 | 13:40 |
rdogerrit | Joel Capitao proposed rdoinfo master: DNM testing 47039 patch for validating buildsys-tags changes https://review.rdoproject.org/r/c/rdoinfo/+/47041 | 13:51 |
rdogerrit | Joel Capitao proposed rdoinfo master: DNM testing 47039 patch for validating buildsys-tags changes https://review.rdoproject.org/r/c/rdoinfo/+/47041 | 13:52 |
*** amoralej|lunch is now known as amoralej | 14:02 | |
rdogerrit | Joel Capitao proposed rdo-jobs master: Consolidate weirdo playbooks https://review.rdoproject.org/r/c/rdo-jobs/+/47039 | 15:36 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Consolidate weirdo playbooks https://review.rdoproject.org/r/c/rdo-jobs/+/47039 | 15:49 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Consolidate weirdo playbooks https://review.rdoproject.org/r/c/rdo-jobs/+/47039 | 16:02 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Consolidate weirdo playbooks https://review.rdoproject.org/r/c/rdo-jobs/+/47039 | 16:07 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Consolidate weirdo playbooks https://review.rdoproject.org/r/c/rdo-jobs/+/47039 | 16:22 |
apevec | jcapitao: we EOLed oslo.rootwrap in RDO Wallaby, right, same as upstream? | 16:25 |
apevec | asking b/c I noticed it re-appeared in Antelope, by mistake I guess? https://releases.openstack.org/antelope/index.html#antelope-oslo-rootwrap | 16:26 |
apevec | it was functionally replaced by oslo.privsetp afaik | 16:26 |
apevec | oh we did not, still in Zed https://cbs.centos.org/koji/buildinfo?buildID=38086 | 16:27 |
apevec | Tue Sep 13 14:45:47 2022 python-oslo-rootwrap-6.3.1-1.el9s tagged into cloud9s-openstack-zed-candidate by karolinku | 16:28 |
apevec | why was that? | 16:28 |
jcapitao[m] | ah ! | 16:29 |
jcapitao[m] | so we should retire it | 16:30 |
apevec | hmm wait, it is still required at least by neutron, looks like privsep migration was not completed everywhere? | 16:31 |
apevec | it's weird, rootwrap was not included since Wallaby release and now it showed up in AA | 16:32 |
apevec | even weirder there are both Requires on privsep and rootwrap | 16:34 |
jcapitao[m] | apevec: and the diff between 6.3.1 (packaged since Wallaby) and 7.0.0 is not that big | 17:04 |
jcapitao[m] | only 4 commits | 17:04 |
jcapitao[m] | https://github.com/openstack/oslo.rootwrap/compare/6.3.1...7.0.0 | 17:04 |
jcapitao[m] | major bump is due to dropping py36 and py37 support | 17:05 |
jcapitao[m] | https://review.opendev.org/c/openstack/releases/+/872895 | 17:05 |
apevec | yeah I guess 7.0.0 tag was just a mistake | 17:05 |
apevec | ah | 17:05 |
apevec | but still weird there weren't any rootwrap releases W-Z | 17:05 |
jcapitao[m] | but yeah I think it's a mistake too | 17:06 |
apevec | so it was not EOLed just neglected :) | 17:06 |
jcapitao[m] | it seems so ! | 17:07 |
amoralej | it's 6.3.1 in u-c | 17:08 |
apevec | yeah and still used in Neutron https://github.com/rdo-packages/neutron-distgit/blob/rpm-master/neutron-dist.conf#L6 | 17:09 |
apevec | by default at least, maybe tripleo changes config to privsep | 17:09 |
amoralej | and y many others | 17:09 |
amoralej | at least in requirements.txt | 17:09 |
amoralej | https://codesearch.opendev.org/?q=rootwrap&i=nope&literal=nope&files=.*requirements.txt&excludeFiles=&repos= | 17:09 |
amoralej | https://github.com/openstack/cinder/blob/master/requirements.txt#L27 | 17:10 |
amoralej | we never retired it in rdo | 17:10 |
apevec | ok, thanks for checking, it looked weird, I came across it when in one ticket it was pointed out there wasn't stable/wallaby branch so I assumed it was EOLed | 17:19 |
apevec | s/it/oslo.rootwrap/ | 17:19 |
amoralej | apevec, it was moved to independent | 17:23 |
amoralej | https://opendev.org/openstack/releases/src/branch/master/deliverables/_independent/oslo.rootwrap.yaml | 17:23 |
*** jpena is now known as jpena|off | 17:24 | |
rdogerrit | Joel Capitao proposed rdo-infra/weirdo master: Update docs building https://review.rdoproject.org/r/c/rdo-infra/weirdo/+/47043 | 17:25 |
apevec | uhm where then 7.0.0 came from? | 17:25 |
amoralej | because it was just moved back to cycle-with-intermediate | 17:25 |
amoralej | https://review.opendev.org/c/openstack/releases/+/872901 | 17:25 |
amoralej | :) | 17:25 |
apevec | https://opendev.org/openstack/releases/src/branch/master/deliverables/antelope/oslo.rootwrap.yaml | 17:26 |
amoralej | or it's being | 17:26 |
amoralej | yeah | 17:26 |
amoralej | apevec, the explanation is in https://review.opendev.org/c/openstack/releases/+/864095 | 17:27 |
rdogerrit | Joel Capitao proposed rdo-infra/ansible-role-weirdo-common master: WIP testing 47039 patch https://review.rdoproject.org/r/c/rdo-infra/ansible-role-weirdo-common/+/47040 | 17:27 |
apevec | ok we need that comment patch asap | 17:27 |
jcapitao[m] | aah ok | 17:35 |
jcapitao[m] | apevec: sorry I didn't get your last comment | 17:37 |
jcapitao[m] | apevec: ^ | 17:37 |
jcapitao[m] | amoralej: https://review.rdoproject.org/r/c/rdo-jobs/+/47039 I set it ready for reviews | 17:38 |
amoralej | i will check tomorrow jcapitao[m] | 17:41 |
apevec | * patch asap: https://review.opendev.org/c/openstack/releases/+/872901 | 17:41 |
jcapitao[m] | yeah sure | 17:42 |
apevec | it was about this patch https://review.opendev.org/c/openstack/releases/+/872901 | 17:42 |
amoralej | jcapitao[m], btw did you try with ensure-tox ?= | 17:42 |
jcapitao[m] | damn.. | 17:42 |
amoralej | :) | 17:42 |
jcapitao[m] | I was obnubilate by the other issue | 17:43 |
amoralej | np | 17:43 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Consolidate weirdo playbooks https://review.rdoproject.org/r/c/rdo-jobs/+/47039 | 17:45 |
*** amoralej is now known as amoralej|off | 17:49 | |
rdogerrit | Ronelle Landy proposed rdo-jobs master: WIP: Reduce the master line - conatiner build + scenarios https://review.rdoproject.org/r/c/rdo-jobs/+/47044 | 22:38 |
rdogerrit | Ronelle Landy proposed rdo-jobs master: WIP: Reduce the master line - container build + scenarios https://review.rdoproject.org/r/c/rdo-jobs/+/47044 | 22:38 |
rdogerrit | Ronelle Landy proposed rdo-infra/ci-config master: WIP: Reduce the master line - container build + scenarios https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/47045 | 23:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!