Wednesday, 2021-06-23

rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for train-testing  https://review.rdoproject.org/r/c/rdoinfo/+/3426400:21
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for train-8-testing  https://review.rdoproject.org/r/c/rdoinfo/+/3426500:22
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for ussuri-8-testing  https://review.rdoproject.org/r/c/rdoinfo/+/3426600:23
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for ussuri-8s-testing  https://review.rdoproject.org/r/c/rdoinfo/+/3426700:23
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for train-8s-testing  https://review.rdoproject.org/r/c/rdoinfo/+/3426800:24
rdogerritwes hayutin created rdo-infra/ci-config master: ensure the vexxhost clean up logs to the webdir  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3426902:40
rdogerritwes hayutin proposed rdo-infra/ci-config master: ensure the vexxhost clean up logs to the webdir  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3426902:52
*** ykarel|away is now known as ykarel04:37
rdogerritMerged openstack/kolla-distgit wallaby-rdo: openstack-kolla-12.0.0-0.2.0rc2  https://review.rdoproject.org/r/c/openstack/kolla-distgit/+/3426105:00
*** marios is now known as marios|ruck05:23
rdogerritJoel Capitao proposed rdo-infra/ci-config master: WIP Add JJB files for weirdo-promote-test-puppet-centos8-train  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3422605:50
*** ysandeep|out is now known as ysandeep05:52
rdogerritMerged rdoinfo master: Promote CBS tags update for ussuri-8s-testing  https://review.rdoproject.org/r/c/rdoinfo/+/3426706:15
rdogerritSandeep Yadav created rdo-infra/ci-config master: Improvement in cloud cleanup script  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3427006:21
*** amoralej|off is now known as amoralej06:38
*** jpena|off is now known as jpena06:56
rdogerritMerged rdo-infra/weirdo master: Push logs to logserver.r.o instead of artifacts.c.o  https://review.rdoproject.org/r/c/rdo-infra/weirdo/+/3426207:02
*** rpittau|afk is now known as rpittau07:07
rdogerritMerged config master: Set mergeContent for puppet repos to true  https://review.rdoproject.org/r/c/config/+/3425907:11
*** ChanServ sets mode: +o apevec07:49
rdogerritYatin Karel proposed puppet/puppet-tripleo-distgit rpm-master: Remove unused puppet-vcsrepo  https://review.rdoproject.org/r/c/puppet/puppet-tripleo-distgit/+/3424708:03
rdogerritMerged rdo-infra/ansible-role-dlrn master: Create dummy dlrn-deps.repo file if not present in role  https://review.rdoproject.org/r/c/rdo-infra/ansible-role-dlrn/+/3425808:17
*** bhagyashris_ is now known as bhagyashris08:27
rdogerritSandeep Yadav proposed rdo-infra/ci-config master: Improvement in cloud cleanup script  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3427008:32
rdogerritSandeep Yadav proposed rdo-infra/ci-config master: Improvement in cloud cleanup script  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3427008:40
*** ykarel is now known as ykarel|lunch08:52
rdogerritSandeep Yadav proposed rdo-infra/ci-config master: Improvement in cloud cleanup script  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3427009:01
rdogerritSandeep Yadav created rdo-infra/ci-config master: ensure the vexxhost clean up logs to the webdir  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3427209:11
rdogerritSandeep Yadav proposed rdo-infra/ci-config master: ensure the vexxhost clean up logs to the webdir  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3426909:14
rdogerritSandeep Yadav proposed rdo-infra/ci-config master: Improvement in cloud cleanup script  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3427009:53
*** ykarel|lunch is now known as ykarel09:56
toskyso... about https://review.rdoproject.org/r/c/openstack/cinder-distgit/+/34257 10:34
toskydid I do the right thing?10:35
*** pojadhav is now known as pojadhav|afk10:41
*** jcapitao is now known as jcapitao_lunch11:23
*** amoralej is now known as amoralej|lunch11:28
*** jpena is now known as jpena|lunch11:30
*** dviroel|out is now known as dviroel11:37
*** marios|ruck is now known as marios|ruck|call12:00
*** jcapitao_lunch is now known as jcapitao12:07
*** ysandeep is now known as ysandeep|afk12:26
*** amoralej|lunch is now known as amoralej12:30
*** pojadhav|afk is now known as pojadhav12:32
*** jpena|lunch is now known as jpena12:33
rdogerritMerged rdo-infra/ci-config master: Improvement in cloud cleanup script  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3427012:35
rdogerritMerged openstack/validations-common-distgit rpm-master: Dependency versions and conflicts clarified  https://review.rdoproject.org/r/c/openstack/validations-common-distgit/+/3422412:39
rdogerritMerged openstack/cinder-distgit rpm-master: Fix the dependency on libcgroup-tools (no arch)  https://review.rdoproject.org/r/c/openstack/cinder-distgit/+/3425712:45
*** marios|ruck|call is now known as marios|ruck12:47
rdogerritChandan Kumar created rdo-infra/ci-config master: [container-push] Remove unused ansible vars  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3427312:52
*** ysandeep|afk is now known as ysandeep13:04
rdogerritLuigi Toscano created openstack/cinder-distgit wallaby-rdo: Fix the dependency on libcgroup-tools (no arch)  https://review.rdoproject.org/r/c/openstack/cinder-distgit/+/3427413:22
rdogerritSandeep Yadav proposed rdo-infra/ci-config master: ensure the vexxhost clean up logs to the webdir  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3426913:23
jcapitaoapevec, amoralej, ykarel: wrt https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/message/DUGTUST6GMYZWPJ7ZHCGKIJ3FQVHQBER/ I built python-openstackdocstheme-2.3.0 in order to fix the issue13:30
jcapitaohttps://src.fedoraproject.org/rpms/python-openstackdocstheme/c/7b00336ad41e36abb67376867e151adf51acf43d?branch=rawhide13:30
jcapitaoso we won't have bugs filed wrt Sphinx-4.x change hopefully13:30
amoralejcool13:30
amoralejjcapitao, so the issue in openstackclient is fixed via openstackdockstheme?13:31
ykareljcapitao, ack13:31
apevecjcapitao++ thanks I was going to check about that today13:31
apevecamoralej, yeah both FTBFS in novaclient and OSC are coming from openstackdocstheme13:32
jcapitaoamoralej: yes, https://review.opendev.org/c/openstack/openstackdocstheme/+/793820 this patch fixed it13:32
apevecyeah I checked, that's included in 2.3.013:33
jcapitaothe RDO meeting will start in less than 5 minutes. Feel free to add items to the agenda at https://etherpad.openstack.org/p/RDO-Meeting13:57
rdogerritMerged rdo-infra/ci-config master: ensure the vexxhost clean up logs to the webdir  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3426913:58
jcapitaomeeting time !14:01
jcapitao#startmeeting RDO meeting - 2021-06-2314:01
opendevmeetMeeting started Wed Jun 23 14:01:26 2021 UTC and is due to finish in 60 minutes.  The chair is jcapitao. Information about MeetBot at http://wiki.debian.org/MeetBot.14:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:01
opendevmeetThe meeting name has been set to 'rdo_meeting___2021_06_23'14:01
ykarelo/14:03
jcapitaosorry for the short notice, we'll wait a little bit for additional topics https://etherpad.openstack.org/p/RDO-Meeting14:03
spotzo/14:03
jcapitao#chair ykarel spotz 14:04
opendevmeetCurrent chairs: jcapitao spotz ykarel14:04
jcapitaoamoralej: around ?14:08
amoralejoh14:09
amoralejright14:09
amoralejo/14:09
jcapitao#chair amoralej 14:10
opendevmeetCurrent chairs: amoralej jcapitao spotz ykarel14:10
jcapitaolet's start with first topic14:10
jcapitao#topic Gerrit Merge Strategy14:10
jcapitaoykarel: ^14:10
ykarelyes, so yesterday we discussed this, currently distgit patches goes to merge conflict even if there is no merge conflict14:11
ykareland this happens as we configure [submit]/mergeContent = false 14:12
ykarelso we are evaluating  [submit]/mergeContent = true now14:12
jcapitaosince https://review.rdoproject.org/r/c/config/+/3425914:12
ykarel#link https://review.rdoproject.org/r/c/config/+/3425914:12
rdogerritmathieu bultel proposed openstack/validations-libs-distgit rpm-master: Add validation.cfg workaround  https://review.rdoproject.org/r/c/openstack/validations-libs-distgit/+/3424114:13
jcapitaois it better ?14:13
ykarelso we expecting with this change, that gerrit will auto rebase if there is no conflict at time of submit14:13
ykareljcapitao, haven't tested it14:13
amoralejlet's create some dommy patches in one of the puppet packages14:14
amoralejdummy14:14
ykarelpatches are not merged since it https://review.rdoproject.org/r/q/project:%255Epuppet/puppet.*-distgit14:14
ykarelamoralej, i rebased some patches ^14:14
*** ysandeep is now known as ysandeep|out14:14
jcapitaowhat did change suddenly to have this issue ?14:14
ykarelso with merge of puppet-tripleo patches it should be clear14:14
amoralejah14:15
amoralejso all those are rebased on rpm-master right?14:15
ykarelyes14:15
amoralejlet me check if we can merged them as they are14:15
ykareljcapitao, issue was there for long, but it get's more visible when multiple patches are posted together14:15
ykarelyesterday tkajinam raised it and it lead to the discussion14:16
ykarelwe used to manually rebase always till now14:16
ykarelnow we are hoping to avoid manually rebases unless and until there are real conflicts14:16
jcapitaook good14:17
ykarelfor some projects we really need that option to be turned off14:18
ykarelas we run different jobs in check and gate, like rdoinfo14:18
amoralejin fact, it will also happen in distgits14:18
rdogerritUser marios created rdo-infra/ci-config master: DNM testing try to set the dlrn_hash for dlrn reporting  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3427514:18
amoraleji mean, different jobs in check and gate14:19
amoralejso, some jobs will only be executed with the individual patches14:19
amoralejbut not with the rebases14:19
ykarelyes true in some project we may have it14:19
amoralejhttps://review.rdoproject.org/r/c/puppet/puppet-tripleo-distgit/+/3385214:19
amoralejso, gate only has dlrn-rpmbuild-centos814:19
amoralejit doesn't run tripleo, centos9 or build-tripleo14:20
amoralejprobably it's fine14:20
amoralejfor distgits14:20
amoralejbecause, iiuc the automatic rebase is done only before gate jobs, right?14:21
spotzWoould anything from the Validate presentation yesterday help us?14:21
ykareli think it's at the time of merge after gate results14:21
amoralejthe one about validation framework?14:21
amoralejafter gate results?14:21
amoralejthat's weird, i think...14:21
amoralejin that case the gate jobs run with a different commits stack that the one merged14:22
amoralejif patches A and B are sent of top of commit XYZ14:22
amoralejfinal situation is XYZ/A/B14:23
jpenaiirc, patches in the gate queue are always checked as if they were merged with the current repo contents (so gate jobs are running the "right" content)14:23
amoralejso we should get that merge sequence at some point14:23
spotzamoralej: yeah that one14:23
amoralejyeah, that makes sense, thanks jpena14:23
ykarelyes ^ seems true14:23
amoralejspotz, i think it's unrelated with validation framework in this case14:24
ykareli meant specifically for those options, as the were in [submit] group14:24
amoraleji think the point from gerrit perspective is that it does the submit *before* running gates, but in an internal repo14:25
amoralejnot the final one14:25
amoralejor something like that14:25
rdogerritMerged puppet/puppet-tripleo-distgit rpm-master: puppet-sahara is no longer required  https://review.rdoproject.org/r/c/puppet/puppet-tripleo-distgit/+/3424614:25
amoralejso 1. summit internal repo - 2. run gates - 3. pushes from the "internal tested repo" to the public one14:25
amoralejok, now let's see what happens with the other review in puppet-tripleo14:26
ykarelyes let's see14:26
amoralejso, https://review.rdoproject.org/r/c/puppet/puppet-tripleo-distgit/+/34247 can be merged14:26
amoralejno conflict warning now14:26
ykarelok so it's good sign14:26
amoralejyep14:26
amoralejlet's try to merge it14:26
ykarelearlier it used to show conflict warning14:26
ykarelok +w14:27
amoralejyes, i think so14:27
ykarelhttps://review.rdoproject.org/r/gitweb?p=puppet/puppet-tripleo-distgit.git;a=log;h=refs/heads/rpm-master14:29
ykarelhttps://github.com/rdo-packages/puppet-tripleo-distgit/commits/rpm-master14:29
amoralejwe can see the spec in the gate job if the spec had the puppet-sahura removed14:29
ykarelyes14:30
*** marios|ruck is now known as marios|ruck|call14:31
jcapitaothere is no puppet-sahara14:33
ykarelyeap14:34
amoralejyep14:34
amoralejso, i'd say it works as expected14:34
ykarelyes14:34
amoralejthe only question is if current job definitions is fine14:34
rdogerritMerged puppet/puppet-tripleo-distgit rpm-master: Remove unused puppet-vcsrepo  https://review.rdoproject.org/r/c/puppet/puppet-tripleo-distgit/+/3424714:34
ykarelcommit history also looks fine14:35
ykarelhttps://review.rdoproject.org/r/gitweb?p=puppet/puppet-tripleo-distgit.git;a=log;h=refs/heads/rpm-master14:35
amoraleji'd say there's usually no big dependencies between patches in specs or complex stuff that is sensitive to this kind of issues14:35
ykarelhttps://github.com/rdo-packages/puppet-tripleo-distgit/commits/rpm-master14:35
amoralejso i think we can live with this14:36
amoralejas is14:36
ykarelyes agree14:36
amoralejkeep DLRN-rpmbuild-centos8 (and 9 in future)14:36
amoralejin gate14:36
amoralejin fact weirdo and oooq are non-voting14:36
amoralejiirc14:36
amoralejso no point in adding non-voting jobs in gate14:36
ykarelyes those are non voting14:37
ykarelnow we see slightly different comment from zuul14:38
ykarelChange has been successfully rebased and submitted as f13aa50e0459674f0d2280076d8697ad55e6638d by Zuul CI14:38
amoralejright14:39
amoralejin fact, it's a new PS14:39
amoralejgood to have visibility14:39
ykarelyes14:39
ykarelok let's move to next topic14:39
rdogerritMerged rdo-infra/ci-config master: Set extended_hash value to None instead of 'None'  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3419614:39
amoralejactually it creates a new PS with a new parent14:39
ykarelPS or commit?14:40
ykarelPS for patch set?14:40
amoralejpatchset14:40
jcapitaoyes PS3 in this case14:40
amoralejyes14:40
ykarelokk got it now14:41
jcapitaogood14:42
jcapitaolet's move to next topic then14:43
jcapitao#topic Contributor shirt update14:43
jcapitaofrom spotz14:43
spotzSo while we've had the new Wallaby release version for a while we just got the new store. I've been going through it and have narrowed it down to actually the original shirrt type I'd wanted. Just working on a color at this point. I'll note the new stoore isn't easy to tell what coloor you're looking at and the virtual preview doesn't uplooad for me to see14:44
spotzBut I hope to have something in the works end of this week or nexr14:45
ykarelcool14:46
spotzAnd there should be coozies up there soon to. For the contributor shirts we'll get codes again and include shipping14:46
ykarelatleast i haven't seen new store yet14:46
jcapitaowhat kind of orignality does this shirt have ? :)14:46
spotzNew color and Victoria replaced by Wallaby. I'm not sure we can change Spencer much each time beyond that but we have control of color for shirts and possibly we couldd change item but wanted to stick with shirrts until I get better at the store14:47
amoralejwhat coozies mean? :)14:48
spotzhttps://coolstuff.redhat.com/category/121/rdo14:49
spotzhttps://coolstuff.redhat.com/category/121/rdo#link https://coolstuff.redhat.com/category/121/rdo14:49
spotzbah!14:49
spotzCoozies are little things you can put a can in and it helps keep them cold14:49
spotz#link https://coolstuff.redhat.com/category/121/rdo14:49
spotzBut we have our own section of the store!14:49
amoralejahh i know now14:50
spotzAnd that will be non-release Spencer on there14:50
amoralejhttps://belusaweb.s3.amazonaws.com/product-images/gallery/623045_BS-Detail.jpg14:51
*** marios|ruck|call is now known as marios|ruck14:51
spotz\yep14:51
amoralejnice14:51
spotzOurs are a grey from looking at the photo I was sent14:51
amoralejit's cool to have our own section in the store!14:51
amoralejspotz++14:51
jcapitaoyeah that rocks !14:52
spotzYep! Gives us more visibility:)14:52
jcapitaook, there is no more topic for today14:54
jcapitaobefore open floor14:54
jcapitao#topic Chair for next week14:54
spotzI can take it14:54
spotzI haven't done it in a bit14:54
jcapitaothank you spotz14:55
jcapitao#action spotz to chair next week14:55
jcapitao#topic Open Floor14:55
ykarelwrt migration to new jenkins https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3422614:56
ykarelany idea how we manage credentials in current ci.centos jenkins14:56
ykarellike DLRNAPI_PASSWD14:56
ykareli was looking at credentials for rdo user, but couldn't find it14:56
amoralejmmm14:57
amoralejit uses credentials-bindings14:59
amoralejhttps://github.com/rdo-infra/ci-config/blob/0ce33c32ecf54f4df2834e6e147989129d380b15/jenkins/jobs/weirdo-defaults.yml#L25-L2815:00
ykarelyes, but where and how those creds are stored15:00
amoraleji think we create them manually15:00
amoralejin jenkins webui15:01
ykarelfor rdo user or some other?15:01
amoralejchecking15:01
ykarelack15:01
jcapitaoI'm going to close the meeting, we can continue to discuss on this topic off record15:02
ykarelack15:02
jcapitao#endmeeting15:02
opendevmeetMeeting ended Wed Jun 23 15:02:46 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:02
opendevmeetMinutes:        https://meetings.opendev.org/meetings/rdo_meeting___2021_06_23/2021/rdo_meeting___2021_06_23.2021-06-23-14.01.html15:02
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/rdo_meeting___2021_06_23/2021/rdo_meeting___2021_06_23.2021-06-23-14.01.txt15:02
opendevmeetLog:            https://meetings.opendev.org/meetings/rdo_meeting___2021_06_23/2021/rdo_meeting___2021_06_23.2021-06-23-14.01.log.html15:02
ykarelThanks jcapitao 15:02
spotzThanks jcapitao and everyone!15:03
amoralejthanks jcapitao15:03
amoralejykarel, i can find some credentials in rdo user15:06
amoralejbut not that15:06
ykarelyes same for me15:06
amoraleji remember have done something15:07
amoralejrelated to credentials15:07
amoralejlong ago15:07
ykarelmay be dmsimard have some idea, from history i see he added that15:07
amoralejis any job actually using it?15:07
rdogerritJoel Capitao created rdo-website master: RDO Meeting minutes for 2021-06-16 & 23  https://review.rdoproject.org/r/c/rdo-website/+/3427615:08
ykarelamoralej, yes phase 1 weirdo jobs use it15:08
ykareland also puppet promotion jobs15:08
amoraleji see it in the job config web ui15:09
amoralejmaybe it was created at system level15:09
amoralejby jenkins admins15:09
ykarelyes possible15:11
*** ykarel is now known as ykarel|away15:20
*** marios|ruck is now known as marios|out15:55
*** jpena is now known as jpena|off16:00
*** rpittau is now known as rpittau|afk16:08
csibbittHiya, just checking if there is anything else I need for a Workflow +1 on this https://review.rdoproject.org/r/c/openstack/tripleo-ansible-distgit/+/3424916:19
*** frenzy_friday is now known as anbanerj|rover16:19
rdogerritrdo-trunk created openstack/ironic-distgit wallaby-rdo: openstack-ironic-17.0.3-1  https://review.rdoproject.org/r/c/openstack/ironic-distgit/+/3427716:20
*** amoralej is now known as amoralej|off16:25
*** sshnaidm is now known as sshnaidm|afk16:35
rdogerritMerged openstack/ironic-distgit wallaby-rdo: openstack-ironic-17.0.3-1  https://review.rdoproject.org/r/c/openstack/ironic-distgit/+/3427717:06
rdogerritMerged rdo-infra/ci-config master: Updated the ussuri and wallaby criteria file  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3401417:47
rdogerritrdo-trunk created openstack/kolla-distgit wallaby-rdo: openstack-kolla-12.0.0-1  https://review.rdoproject.org/r/c/openstack/kolla-distgit/+/3427817:56
apevec[m]off-topic: amoralej|off jpena - congrats EspaƱa! 18:13
rdogerritwes hayutin proposed rdo-infra/ci-config master: WES [wip]Ruck/rover tool  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3425519:30
rdogerritMerged rdo-infra/ci-config master: Updates on skiplist report  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3411520:27
*** dviroel is now known as dviroel|out21:13
*** rlandy is now known as rlandy|bbl22:15

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