Wednesday, 2021-03-31

*** armstrong has quit IRC00:12
*** iurygregory has quit IRC01:16
*** iurygregory has joined #openstack-release01:17
*** iurygregory has quit IRC01:18
*** iurygregory has joined #openstack-release01:18
*** iurygregory has quit IRC02:08
*** iurygregory has joined #openstack-release02:09
*** evrardjp has quit IRC02:33
*** evrardjp has joined #openstack-release02:33
*** diablo_rojo has quit IRC02:40
*** armax has quit IRC02:55
*** armax has joined #openstack-release02:59
*** dulek has quit IRC03:09
*** dulek has joined #openstack-release03:19
*** armax has quit IRC03:57
*** ykarel|away has joined #openstack-release04:20
*** vishalmanchanda has joined #openstack-release04:30
*** ykarel|away is now known as ykarel04:39
hberaudfungi: thanks05:07
*** whoami-rajat has joined #openstack-release05:17
*** adriant has quit IRC05:43
*** slaweq has joined #openstack-release06:10
*** sboyron has joined #openstack-release06:21
*** eolivare has joined #openstack-release06:30
*** openstackgerrit has joined #openstack-release06:54
openstackgerritHervé Beraud proposed openstack/releases master: Release ansible-role-atos-hsm for Wallaby  https://review.opendev.org/c/openstack/releases/+/78390806:54
openstackgerritHervé Beraud proposed openstack/releases master: Release ansible-role-thales-hsm for Wallaby  https://review.opendev.org/c/openstack/releases/+/78390506:55
openstackgerritTakashi Kajinami proposed openstack/releases master: Puppet OpenStack Wallaby RC1  https://review.opendev.org/c/openstack/releases/+/78401407:21
*** tosky has joined #openstack-release07:33
openstackgerritTakashi Kajinami proposed openstack/releases master: Puppet OpenStack Wallaby RC1  https://review.opendev.org/c/openstack/releases/+/78401407:34
*** e0ne has joined #openstack-release07:35
openstackgerritTakashi Kajinami proposed openstack/releases master: Puppet OpenStack Wallaby RC1  https://review.opendev.org/c/openstack/releases/+/78401407:39
*** e0ne has quit IRC07:58
*** ykarel has quit IRC08:01
*** vishalmanchanda has quit IRC08:39
*** ykarel has joined #openstack-release08:43
*** ykarel is now known as ykarel|lunch08:43
openstackgerritTakashi Kajinami proposed openstack/releases master: Puppet OpenStack Wallaby RC1  https://review.opendev.org/c/openstack/releases/+/78401408:58
openstackgerritTakashi Kajinami proposed openstack/releases master: Puppet OpenStack Wallaby RC1  https://review.opendev.org/c/openstack/releases/+/78401408:59
*** vishalmanchanda has joined #openstack-release09:11
*** e0ne has joined #openstack-release09:14
openstackgerritMerged openstack/releases master: Release Bifrost 10.2.0 and create stable/wallaby  https://review.opendev.org/c/openstack/releases/+/78366509:16
*** gibi is now known as gibi_away09:52
*** e0ne has quit IRC09:59
*** ykarel|lunch is now known as ykarel10:04
hberaudttx, elod: o/ Any idea concerning this error? => https://zuul.opendev.org/t/openstack/build/3c94d9fe7dbc41fb82030a7e5adbf88a/log/job-output.txt#4257 those both patches face the same problem => https://review.opendev.org/c/openstack/releases/+/783905 https://review.opendev.org/c/openstack/releases/+/78390810:09
hberaudyet I think I've properly configuered templates => https://review.opendev.org/c/openstack/project-config/+/78401110:10
ttxlooking10:10
ttxLooks like rate limiting on Pypi10:11
ttxNot great news for us if it sticks10:13
elod:S10:14
ttxyeah, I suspect it's Pypi returning 429 (too many requests) on that XMLrpc requests10:15
ttxSee https://github.com/pypa/warehouse/issues/875310:17
ttxwe may have to introduce a sleep in there10:18
ttxor an incremental retry10:22
ttxor change the way we retrieve roles since XMLrpc seems to only be used for that10:24
ttxhttps://opendev.org/openstack/releases/src/branch/master/openstack_releases/pythonutils.py#L141 for those following at home10:25
ttxah, wait10:25
hberaudhm ok... but then why other job pass?10:27
hberaudmore or less at the same time10:27
ttxjust a sec10:27
hberaudsure10:27
ttxtesting an assumption here10:28
ttxhttps://zuul.opendev.org/t/openstack/build/3c94d9fe7dbc41fb82030a7e5adbf88a/log/job-output.txt#4227 shows that it retrieves roles twice10:30
*** e0ne has joined #openstack-release10:31
ttxso that explains why it hits the TooManyRequests limit10:31
ttxFirst request fails, and it tries again10:31
ttxFor https://review.opendev.org/c/openstack/releases/+/783908 the error seems different?10:32
elodno, it's the same: https://zuul.opendev.org/t/openstack/build/f72a8875b48d4eef9b9f6f94ff5636c4/log/job-output.txt#4218-421910:36
elodI've checked a passing patch's log and it has only one 'retrieving roles for' debug log: https://zuul.opendev.org/t/openstack/build/fb8d23ea25c24ae6888d08c87f60adef/log/job-output.txt#420310:39
elodnow the question is why we have two in the failing cases :)10:40
openstackgerritThierry Carrez proposed openstack/releases master: Avoid PyPI ratelimit on roles retrieval retry  https://review.opendev.org/c/openstack/releases/+/78406810:40
ttxIt's because the package does not exist10:40
ttxhttps://pypi.org/project/ansible-role-atos-hsm/10:40
ttxso it tries to retrieve roles, that fails, then it retries with the canonicalized version of the name (which in this case is the same)10:41
hberaudyes they were not released yet10:41
elodsounds logical10:41
ttxOK the patch above should fix it10:42
hberaudhowever that's not the first time that we created a new deliverable10:42
ttxOther validation runs do not hit that because they are created already10:42
ttxSure, but the rate limit was added relatively recently10:42
ttxmaybe our new host is fatser etc10:43
hberaudah I see10:43
hberaudyes that make sense10:43
ttxA more comprehensive improvement would be to only retry if the canonicalized name is different from the name on the 1st try10:44
ttxbut I guess retying can't hurt, maybe other transient failures can make it fail on 1st try10:44
ttxlunchtime!10:45
hberaudbon appetit10:46
hberaudFor now let's observe if that help10:46
elodand I saw that xmlrpc is not recommended and json api would be better ( https://wiki.python.org/moin/PyPIXmlRpc )10:49
elodbut for now this fix hopefully solves our issue10:49
hberaud+110:49
hberaudimprovements could comes with follow up patches10:49
elodand: Bon appétit! :)10:50
hberaudMerci10:50
hberaud:)10:50
*** mugsie__ is now known as mugsie11:01
openstackgerritMerged openstack/releases master: Avoid PyPI ratelimit on roles retrieval retry  https://review.opendev.org/c/openstack/releases/+/78406811:15
hberaudwell, retrying11:15
elodit works \o/11:25
hberaud\o/11:26
hberaudkudos ttx11:26
elodyes-yes, nice catch :)11:27
hberaudelod, ttx: so now these patches just need a final +2 https://review.opendev.org/c/openstack/releases/+/783908 https://review.opendev.org/c/openstack/releases/+/78390511:31
hberaudafter this point only 2 ironic* stable branches will still miss11:32
hberaudhowever, according to our process, I think that when those 2 patches ^ will be merged we will be able to branch requirements11:34
*** dtantsur|afk is now known as dtantsur11:34
hberaudThe branching missing projects and the branching of requirements seem concurrent and independent tasks https://releases.openstack.org/reference/process.html#r-2-week11:36
hberauds/branching missing/branching of missing/11:37
ttx+2a11:43
elodoh, I was late.11:45
elodso the 1st one is now super-approved.11:45
elod:]11:45
hberaudthanks guys11:48
fungicatching up, looks like you were able to work out a likely cause for the pypi error. nothing needs to be rerun now?11:59
hberaudfungi: it was a false positive, https://review.opendev.org/c/openstack/releases/+/784068 . We don't have things to reenqueue12:05
fungigood, good...12:05
hberaudfungi: thanks for your diligence12:06
fungijust making sure as i try to work through priorities for my morning12:06
fungino sweat, always happy to help12:06
openstackgerritMerged openstack/releases master: Release ansible-role-atos-hsm for Wallaby  https://review.opendev.org/c/openstack/releases/+/78390812:07
openstackgerritMerged openstack/releases master: Release ansible-role-thales-hsm for Wallaby  https://review.opendev.org/c/openstack/releases/+/78390512:08
openstackgerritMerged openstack/releases master: Release manila-tempest-plugin for wallaby  https://review.opendev.org/c/openstack/releases/+/78336812:08
hberaud:)12:18
*** nweinber has joined #openstack-release12:28
openstackgerritMerged openstack/releases master: Release barbican-tempest-plugin for wallaby  https://review.opendev.org/c/openstack/releases/+/78334112:30
*** smcginnis has quit IRC12:53
hberaudgmann: o/, How is the situation from the QA team perspective concerning your release duties? https://wiki.openstack.org/wiki/QA/releases12:59
dtantsurhberaud: hi! we're working towards proposing ironic and IPA releases by EOD13:13
hberauddtantsur: excellent, thanks13:13
*** armstrong has joined #openstack-release13:27
gmannhberaud: are all projects branched? if so QA can start the process13:38
hberaud2 projects remains unbranched for now ^13:39
hberaudgmann: https://review.opendev.org/c/openstack/releases/+/783675 https://review.opendev.org/c/openstack/releases/+/78369613:39
hberaudgmann: I'll ping you when they will be merged13:40
gmannhberaud: ok, I will keep eyes on these and start QA process accordingly, thanks13:40
hberaudThat's an option13:40
hberaudthank you gmann13:40
gmannnp!13:40
armstrong@hberaud   I  pinged  the projects on IRC and one only one project  have responded13:41
hberaudarmstrong: ack thanks for the heads up13:41
armstrongnp13:42
hberaudarmstrong: I seen your private message13:42
armstrongis there something else I  can do?13:42
hberaudnot much for now13:43
armstrongok13:43
*** rpioso is now known as rpioso|afk13:55
openstackgerritHervé Beraud proposed openstack/releases master: Improve R-2 documentation  https://review.opendev.org/c/openstack/releases/+/78410314:11
*** armax has joined #openstack-release14:18
hberaudelod, ttx: FYI I don't think that we need to expect PTL/liaison validation for this one https://review.opendev.org/c/openstack/releases/+/783365 https://zuul.openstack.org/build/7305e115025243779531bf323d8b171e14:32
dtantsurJFYI I'll leave earlier today, Julia will coordinate the final ironic releases14:33
hberaudack14:33
*** ykarel is now known as ykarel|away14:40
openstackgerritIury Gregory Melo Ferreira proposed openstack/releases master: Creating stable wallaby for ironic-python-agent  https://review.opendev.org/c/openstack/releases/+/78367514:47
ttxok approved14:48
iurygregorydtantsur, hberaud fyi ^14:48
hberaudack14:49
hberaudiurygregory: thanks14:49
iurygregorynp14:50
dtantsuriurygregory, TheJulia, I wonder if https://docs.openstack.org/releasenotes/ironic-python-agent/unreleased.html#upgrade-notes and the vmedia change justify a major bump.14:50
* iurygregory looks14:51
iurygregorydtantsur, yeah I agree i can be a major bump to 7.0.014:54
* iurygregory waits for TheJulia to confirm =)14:54
TheJuliahmm14:55
TheJulialooking14:55
ykarel|awayhberaud, looks like tarballs are not being pushed https://tarballs.opendev.org/openstack/barbican-tempest-plugin/?C=M;O=D14:55
TheJuliaYes, major version bump please14:55
TheJulia7.0.0 for IPA14:55
iurygregoryack14:55
ykarel|awaysame for https://tarballs.opendev.org/openstack/manila-tempest-plugin/?C=M;O=D14:56
ykarel|awayrelease jobs passed14:56
ykarel|awayhttps://e263e188bf593a1accc9-15a9cdd27c1ff21de3e948d542da4662.ssl.cf1.rackcdn.com/d887d76c1f7d4e3a0630b5b9886c39c1bfb05263/release/release-openstack-python/7c42f4c/job-output.txt14:56
openstackgerritIury Gregory Melo Ferreira proposed openstack/releases master: Creating stable wallaby for ironic-python-agent  https://review.opendev.org/c/openstack/releases/+/78367514:57
openstackgerritMerged openstack/releases master: Release keystone-tempest-plugin for wallaby  https://review.opendev.org/c/openstack/releases/+/78336515:00
fungiykarel|away: this was the build for that: https://zuul.opendev.org/t/openstack/build/7c42f4c7e93545918a321da1e91f941a15:01
fungiand /afs/.openstack.org/project/tarballs.opendev.org/openstack/barbican-tempest-plugin/barbican_tempest_plugin-1.3.0.tar.gz does exist, but we may be behind on syncing the read-only replica15:04
fungii'll look into it15:04
hberaudWe didn't get related notification about job failure so not sure that's a job failure http://lists.openstack.org/pipermail/release-job-failures/2021-March/thread.html15:05
ykarel|awayfungi, Thanks15:05
ykarel|awayhberaud, job passed but tarball not yet exists15:05
hberaudyes15:05
hberaudlooks more a replication issue15:05
fungihberaud: ykarel|away: yeah, i don't think there was a job failure. also the tarball does exist, it's just not being served by the tarballs site yet15:06
fungiit's in the read-write afs volume15:06
fungiyep, confirmed, i think we didn't consider that our decision to add a new replica on afs01.ord for better redundancy would delay volume updates for the other replicas15:08
fungiso nothing's "broken" i don't think, this is a self-imposed delay in the publication process15:09
hberaudack15:10
*** noonedeadpunk has quit IRC15:27
*** noonedeadpunk has joined #openstack-release15:28
*** ykarel|away has quit IRC15:37
*** mtreinish has joined #openstack-release15:53
prometheanfirereqs branch waiting on IPA and the like?15:56
*** rpioso|afk is now known as rpioso16:00
hberaudprometheanfire: yes16:00
hberaudIPA is on the rails but it still miss the other ironic patch https://review.opendev.org/c/openstack/releases/+/78369616:01
prometheanfirekk16:01
hberaudprometheanfire: and normally we also need to wait for the QA team duties before branching reqs, however they also wait for the ironic patches16:02
hberaudelod, ttx: Please can one of you can validate this patch => https://review.opendev.org/c/openstack/releases/+/783675/16:04
*** diablo_rojo has joined #openstack-release16:06
elodhberaud: +2+W'd16:22
hberaudelod: thanks16:22
elodnp16:22
iurygregoryhberaud, re ironic patch, we are waiting for some changes to merge on master and I will update16:29
hberaudack16:29
hberaudiurygregory: which patches?16:29
hberauds/patches/changes/16:29
iurygregory1min let me get the links16:29
iurygregoryI think they are on gate already16:30
hberaudok16:30
iurygregoryhttps://review.opendev.org/c/openstack/ironic/+/744117 -  just got merged | https://review.opendev.org/c/openstack/ironic/+/744117  - on gate16:31
hberaudok thanks16:31
iurygregoryI'm reviewing to see if there is something else16:31
hberaudboth patches are the same ^16:32
iurygregoryenr XD16:33
iurygregoryhttps://review.opendev.org/c/openstack/ironic/+/72159316:36
iurygregoryand probably https://review.opendev.org/c/openstack/ironic/+/78375916:36
hberaudiurygregory: ok. Do you think it will be possible to merge the last one ASAP?16:38
hberaudhttps://review.opendev.org/c/openstack/ironic/+/78375916:38
iurygregoryyeah CI should be happy16:38
hberaudok16:38
iurygregoryI ask Julia =)16:38
iurygregoryI will*16:38
hberaudthanks for your feedback16:38
hberaudok16:38
openstackgerritMarios Andreou proposed openstack/releases master: WIP Make tripleo repos release for stable/ussuri branch  https://review.opendev.org/c/openstack/releases/+/78412416:40
openstackgerritMerged openstack/releases master: Creating stable wallaby for ironic-python-agent  https://review.opendev.org/c/openstack/releases/+/78367516:42
*** eolivare has quit IRC16:43
openstackgerritGhanshyam Mann proposed openstack/releases master: Create stable/wallaby for devstack  https://review.opendev.org/c/openstack/releases/+/78413217:00
openstackgerritGhanshyam Mann proposed openstack/releases master: Create stable/wallaby for grenade  https://review.opendev.org/c/openstack/releases/+/78413417:03
gmannhberaud: ^^ pushed devstack/grenade branch with depends-on on Ironic patch. once that is merged these are all good to go17:04
hberaudgmann: ak17:04
hberaudack17:04
hberaudthanks17:04
*** dtantsur is now known as dtantsur|afk17:04
openstackgerritHervé Beraud proposed openstack/releases master: add branch for openstack requirements  https://review.opendev.org/c/openstack/releases/+/77983217:10
hberaudprometheanfire: I rebased the requirements branch on HEAD => https://opendev.org/openstack/requirements/commit/040e24a73f93c5f1c81fc2312a3552f4cb972cc5 and I added two depends-on gmann is patches ^17:11
hberaudprometheanfire: these patches themselves have a depends-on on => https://review.opendev.org/c/openstack/releases/+/783696 (the missing branch)17:13
* hberaud AFK for awhile, be back later17:13
openstackgerritMichael Johnson proposed openstack/releases master: Release Octavia Wallaby RC2  https://review.opendev.org/c/openstack/releases/+/78413917:21
openstackgerritGhanshyam Mann proposed openstack/releases master: Create stable/wallaby for devstack  https://review.opendev.org/c/openstack/releases/+/78413217:33
openstackgerritGhanshyam Mann proposed openstack/releases master: Create stable/wallaby for grenade  https://review.opendev.org/c/openstack/releases/+/78413417:34
*** vishalmanchanda has quit IRC17:51
prometheanfirehberaud: ok19:03
hberaudiurygregory, TheJulia: The CI seems happy here => https://review.opendev.org/c/openstack/ironic/+/783759 and the other patch is now merged19:20
TheJuliahberaud: help me understand real quick, is the pressure because you guys want to branch requirements and see us being branched first as a requirement to that?19:23
hberaudTheJulia: We need the ironic's wallaby stable branch to cut requirements19:25
TheJuliais ironic in requirements?19:26
TheJuliaI mean, its kind because traditionally QA/release teams have just branch and broken devstack right out from under us at the end of the cycle19:26
TheJuliaI'm just confused why now19:26
TheJuliaI'm surely missing something19:27
hberaudI don't think that ironic is in requs, concerning myself is more that our process ask for all stable branches before cutting reqs => https://releases.openstack.org/reference/process.html#r-2-week19:28
hberaudbut maybe you are right and we don't really need this one... and process are outdated or misleading19:28
hberaudprometheanfire: any opinion? ^19:29
TheJuliaI mean, that is openstack at large's process, doesn't mean everyone must conform precisely to it.19:29
TheJuliaGranted, I want to get the release out and lay down and sleep for a day19:29
TheJuliabut yeah19:29
hberaudTo drive the coordinated releases we need referential to organize each steps and our existing process (https://releases.openstack.org/reference/process.html) is my referential for now19:31
TheJuliaI've got a patch that has to be backported in the queue now, regardless, so its not a big deal to cut without that, and I've kind of given a contributor a little bit of time to fix a lack of an api version check we spotted at the last minute.19:32
prometheanfirehberaud: it's more that the projects can start pointing at the stable-reqs branch once it's cut I think19:32
TheJuliaprometheanfire: that was my understanding as well, fwiw19:32
TheJuliahberaud: i guess this is your first time driving the process...19:33
hberaudTheJulia: exactly19:33
hberaudI think that we can wait until tomorrow or early on Friday19:33
TheJuliaSean would have just done it and ignored us which explains why I feel more frustrated about releases this year19:33
TheJuliawell, cycle19:33
TheJuliahberaud: yeah, my hope is by EOD at this momen19:34
prometheanfireya, not a huge deal, but I'd like to unfreeze soon19:34
TheJuliaI basically said "has to pass on first go" to the contributor19:34
TheJuliaprometheanfire: ++19:34
hberaudOk let's wait a bit more19:34
TheJuliayou guys are free to go ahead and branch requirements I think, its super unlikely to break us immediately19:34
TheJuliaI guess what I'm saying is "we're used to getting broken, it won't come as surprising to us at the last moment"19:35
TheJuliaespescialy when things ran longer than we expected19:36
* TheJulia sees a 2nd attempt appear and starts getting worried about ci19:40
prometheanfirewe've backported stuff to stable branches before as well19:45
prometheanfireto branch reqs faster19:45
TheJuliawe had to do that with like 16 patches for ironic once19:45
TheJuliait was a nightmare19:45
prometheanfire:D19:45
*** sboyron has quit IRC20:12
*** nweinber has quit IRC20:14
johnsomJust adding $0.02. Octavia also had a hard time with all of the ci outages (ddos, caches down, memory issues, DNS problems, etc.) over the last few weeks trying to get in a few very important bugs. It is the nature of our release cycle. Yeah, it's not pleasant to have to again try to get the backports in for another release, but that is what we have decided is in the best interest of the stakeholders.20:36
johnsomOctavia has been proposed to not be a part of a release in the past because we couldn't get a patch through the gates for a stable cut.20:36
clarkbjohnsom: what were the caches, memory, and dns issues? (I was out last week and aware of the gitea troulbes, but the others are not immediately familiar)20:37
johnsomWe reluctantly decided we needed to move forward with RC1 on schedule and planned for RC2 (proposed today).20:37
clarkboh zuul memory issues?20:38
johnsomclarkb Yes.20:38
johnsomddos was someone going direct to opendev git for a deployment, killing git checkouts20:38
johnsommemory issues took down parts of zuul causing jobs to time out20:39
clarkbya I know that one. Still trying ti girue out what caches and dns problems were20:39
clarkbmore accurately the zuul memory leak causes the jobs to restart but has a similar impact20:39
johnsomcache issues were package not found20:39
johnsomThat one might have been tracked back to a cdn outage20:40
johnsomDNS was the resolution over IPv6 was not resolving the mirror addresses (no answer flag in unbound log).20:40
fungithere was an observed job failure where a dns lookup in ovh returned a pypi lookup failure, logs collected from the node indicated that unbound had gotten a valid record and handed it back to the requesting process, so unclear why it seemed to think the name was not found (suspicion is that the error reported by pip was misleading)20:41
johnsomThe one I saw unbound didn't get an answer to the query20:41
fungialso not sure why that job was going directly to pypi instead of our mirror servers20:41
fungijohnsom: oh, the second one, yeah that was in limestone and looked more like local network connectivity failure in the provider's network20:42
johnsomYeah, it happens, we have seen it before20:42
johnsomBasically the trail of tears is captured on this patch: https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/774157 and it's pair in the octavia repo.20:42
johnsomAnyway, this is why we have a few weeks after the stable cut to get any critical patches in.20:44
clarkbok, just making sure we're trying to stay on top of things. I know fungi worked through gitea last week and we're poking at zuul now to improve that. Sounds like the other issue are less persistent and may be issues adjacent to us but not directly in our control20:44
johnsomYeah, the DNS issue is one you and I have looked at before as well.20:45
fungiwe may want to consider turning off our use of the limestone cloud if connectivity continues to be spotty in there20:46
*** whoami-rajat has quit IRC20:47
johnsomIf you want to run it all to ground I can extract the important bits for each issue into an etherpad.20:47
clarkbthe focus is definitely on the zuul memory issues right now, but ya I can try and take a look at it again when the othre issues settle a bit20:48
johnsomThe gotcha is the logs may expire. But if you want I'm happy to try to extract the highlights.20:49
*** e0ne has quit IRC21:00
*** dhellmann_ has joined #openstack-release21:12
*** dhellmann has quit IRC21:12
*** dhellmann_ is now known as dhellmann21:14
*** dhellmann has quit IRC21:16
*** dhellmann has joined #openstack-release21:17
*** dhellmann has quit IRC21:23
*** dhellmann has joined #openstack-release21:25
*** dansmith has quit IRC21:33
*** dansmith has joined #openstack-release21:38
*** e0ne has joined #openstack-release21:40
*** e0ne has quit IRC21:51
johnsomclarkb fungi Here are the ones that were easy to find: https://etherpad.opendev.org/p/wallaby-RC1-ci-challenges22:02
clarkbthanks!22:02
*** tosky has quit IRC23:30

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