Friday, 2023-02-10

*** chandankumar is now known as chkumar|rover05:07
*** blarnath is now known as d34dh0r5306:56
*** amoralej|off is now known as amoralej07:04
*** amoralej is now known as amoralej|lunch13:45
ttx#startmeeting releaseteam14:00
opendevmeetMeeting started Fri Feb 10 14:00:01 2023 UTC and is due to finish in 60 minutes.  The chair is ttx. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'releaseteam'14:00
ttxPing list: hberaud armstrong elodilles14:00
elodilleso/14:00
ttxMeeting agenda at: https://etherpad.opendev.org/p/antelope-relmgt-tracking14:00
ttxdown to ~line 30614:00
ttxwaiting a minute to see if we get anyone else14:01
ttx#topic Review task completion14:02
hberaudo/14:02
ttx*     Propose autoreleases for cycle-with-intermediary libraries (excluding client libraries) (elod)14:02
elodillesyepp14:03
elodillesi've proposed the patches:14:03
elodilleshttps://review.opendev.org/q/topic:antelope-final-non-client-libs14:03
elodilles~half of them got responses and merged14:03
ttxok looks like we could process a couple PTL+1s more14:04
elodillesi've reviewed all the generated patches and the rest now seems fine to me to merge14:04
elodilles+2'd them14:04
elodillesonly one -1 from a team left14:04
ttxOK14:05
elodillesso as we promised in the commit message (those without responses) will be merged today14:05
ttx* Check with the Technical Committee that Zuul job templates have been created to include those runtimes (elod)14:06
elodillesso basically the patches need a 2nd +2 from either of you :)14:06
elodillesok, so that one:14:06
elodillespatch is proposed but needs to be merged NO SOONER than 2023.1 final release has passed:14:07
elodilleshttps://review.opendev.org/c/openstack/openstack-zuul-jobs/+/87305114:07
elodillesthis is because the process changed since last cycle14:07
ttxnoted14:07
elodillesin connection, another patch is proposed to remove the script for autogenerated BOT patches:14:08
elodilleshttps://review.opendev.org/c/openstack/openstack-zuul-jobs/+/87305114:08
ttxhmm same one?14:08
elodillesindeed14:08
elodillessry, let me search for the right one :S14:08
ttxhttps://review.opendev.org/c/openstack/project-config/+/873016 ?14:08
elodillesyepp ^^^ that one :) thx14:09
ttxOK I reviewed that one iirc14:09
elodillesthx \o/14:09
ttxok last task was:14:09
ttx- Send countdown email (ttx)14:09
ttxwhich we'll review in a sec14:09
ttx#topic Assign next week tasks14:10
ttxI'm OOO all week so would rather not take tasks14:10
elodillesack14:10
elodillesall taken, thx hberaud :)14:11
hberaudnp14:11
ttxThanks guys!14:11
ttx#topic Review countdown email 14:11
ttx#link https://etherpad.opendev.org/p/relmgmt-weekly-emails14:12
ttxEtherpad shows the areas I edited to facilitate your review14:12
ttxWe all agree the branch will be named stable/2023.1 14:12
ttxand I edited the text around cycle highlights since the deadline is the week after now14:13
hberaudyes for the branch name14:13
hberaudLGTM14:13
ttxAlso took the liberty of mentioning the early bird deadline as it may save the acute reader a few bucks14:14
elodillesthanks, LGTM, too14:14
opendevreviewMerged openstack/releases master: Release final os-win for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87278014:15
elodilles+1 to mention the early bird14:15
ttxAlright, last topic14:15
ttx#topic Open Discussion14:15
ttxanything to discuss?14:15
hberaudnope14:15
elodillesi've added the 2023.{1,2} just in case :X14:15
elodilles(for the release name)14:15
bauzashey release team, I have a question14:15
elodillesbauzas: o/14:15
elodillesshoot :)14:16
bauzasoh snap, you're on meeting14:16
bauzasmy bad14:16
bauzasI can wait until the end14:16
opendevreviewMerged openstack/releases master: Release final ceilometermiddleware for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87276614:16
ttxwell it's open discussion, so great timing14:16
bauzaslol14:16
elodilles:)14:16
bauzasmy question was about how the proposal bot works14:16
ttxit uses magic beans, I think14:16
bauzashttps://review.opendev.org/c/openstack/requirements/+/854821 was proposed but I don't see any open change for os-traits 2.1014:16
bauzasttx: I need the recipe14:17
bauzasoh wait14:18
elodilleshmmm, then there must be a bug :S14:18
bauzasI think I know the answer14:18
bauzasI don't see any bump yet14:18
bauzashmmm https://review.opendev.org/c/openstack/releases/+/87310614:19
elodilleshmmm, we don't have a pypi release either14:19
elodilles:S14:19
bauzasdespite this, https://review.opendev.org/q/project:openstack/requirements+os-traits14:19
bauzaselodilles: yup, and the tag isn't present14:19
bauzasthe git tag14:19
elodillesneither have we a 'release job failure' reported :-o14:20
ttxthe tag is missing in git?14:21
elodilleseven that14:21
bauzasyes, I quickly checked on gh, but I'll look at opendev14:21
bauzashttps://opendev.org/openstack/os-traits/tags14:21
bauzassee, no 2.10.0 tag14:22
* bauzas didn't want to derail the meeting14:22
fungiprobably the highlight from the infra channel is also about this, i haven't looked yet but trying to dig up the log for it now14:22
bauzasfungi: thanks14:22
* fungi was ignoring the infra channel to pay attention to this meeting14:22
ttxIt's normal that we don;t have a release on PyPI as those get triggered by the tag14:23
bauzasfwiw, we have a feature patch awaiting for the u-c patch to land in order to have it merged this cycle14:23
fungiso this was the merge commit which should have triggered the tagging job: https://opendev.org/openstack/releases/commit/f2626efa6b9b75ae7e5886292b5b1b0f0b99dfe214:24
elodillesyepp14:24
ttxHrm that pipeline look stuck since Feb 3 https://zuul.opendev.org/t/openstack/builds?job_name=tag-releases14:26
elodillesit seems we have quite many of these silently failing releases :(14:26
ttxThe job does not even run so there is no error14:27
fungithe zuul builds interface finds nothing triggered in any pipeline for that merge commit14:27
ttxfungi: no tag-releases job ran for a whole week14:27
ttxand that was a busy week :)14:27
elodillesfreezer-tempest-plugin -- Fri, 03 Feb 2023 16:16:11 GMT -- 3.0.014:27
bauzas\o/14:27
opendevreviewMerged openstack/releases master: Release final oslo.i18n for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87289214:28
elodilleswas the last one that was really released /o\14:28
opendevreviewMerged openstack/releases master: Release final oslo.metrics for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87278414:28
opendevreviewMerged openstack/releases master: Release final oslo.middleware for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87278514:28
opendevreviewMerged openstack/releases master: Release final oslo.context for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87278214:28
opendevreviewMerged openstack/releases master: Release final oslo.limit for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87278314:28
opendevreviewMerged openstack/releases master: Release final oslo.policy for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87278614:28
ttxhmm I think we should stop approving stuff :/14:28
opendevreviewMerged openstack/releases master: Release final oslo.serialization for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87278914:28
opendevreviewMerged openstack/releases master: Release final oslo.upgradecheck for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87279114:28
fungii don't see any obvious job configuration errors for the releases repo14:28
ttxnone of those will tag (or publish)14:28
ttxOk I think we can move that one off-meeting14:29
ttxanything else before I close the meeting officially?14:29
elodillesthis will be enough i think :D14:30
elodilles:/14:30
ttxalright then14:30
opendevreviewMerged openstack/releases master: Release final oslo.vmware for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87279314:30
elodillesso let's close14:30
ttx#endmeeting14:30
opendevmeetMeeting ended Fri Feb 10 14:30:28 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:30
opendevmeetMinutes:        https://meetings.opendev.org/meetings/releaseteam/2023/releaseteam.2023-02-10-14.00.html14:30
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/releaseteam/2023/releaseteam.2023-02-10-14.00.txt14:30
opendevmeetLog:            https://meetings.opendev.org/meetings/releaseteam/2023/releaseteam.2023-02-10-14.00.log.html14:30
fungilooks like nothing has run in the release-post pipeline for a week: https://zuul.opendev.org/t/openstack/builds?pipeline=release-post&skip=0 so it's not just the tag jobs specifically14:30
bauzasI usually hate when someone rushes into our meetings and drops a bomb14:30
bauzas:)14:30
ttxelod: is tag-releases the only post-job that does not run when we merge openstack/releases changes?14:31
elodillesbauzas: especially in friday afternoon :]14:31
bauzasyeah tgif14:31
bauzasg being whatever you appreciate14:32
elodillesttx: good question14:32
ttxReference: https://releases.openstack.org/reference/release_infra.html14:32
fungii don't see any obvious recent changes to job configuration for releases14:32
fungittx: see the link i posted14:32
opendevreviewMerged openstack/releases master: Release final oslo.service for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87279014:33
opendevreviewMerged openstack/releases master: Release final oslo.versionedobjects for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87279214:33
opendevreviewMerged openstack/releases master: Release final automaton for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87276314:33
opendevreviewMerged openstack/releases master: Release final castellan for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87276514:33
opendevreviewMerged openstack/releases master: Release final stevedore for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87279514:33
fungiit shows all the builds in the release-post pipeline so you can see what's been run in it historically14:33
ttxpublish-release-docs also did not run since Feb 3, so that's the whole pipeline that is stuck https://zuul.opendev.org/t/openstack/builds?pipeline=release-post&skip=014:33
fungiright, or at least no longer being matched14:34
ttx(I mean, publish-tox-docs-releases)14:34
*** amoralej|lunch is now known as amoralej14:35
elodillesso it seems that the patch merge did not trigger the post pipeline at all, right?14:35
ttxyeah, il looks like no job is run from the release-post pipeline at all14:37
elodillesi'm looking at it's definition in project-config repo: https://opendev.org/openstack/project-config/src/branch/master/zuul.d/pipelines.yaml#L28214:39
fungilooks like there may have been a schema/model change impacting that pipeline, or possibly some data corruption in zk: https://paste.opendev.org/show/bcGaLU1wkHZzXgcZ4agi/14:40
fungii'll take this to the zuul matrix channel14:40
elodillesfungi: thanks14:40
fungithe good news is i'm pretty sure the tag-releases and publish-tox-docs-releases jobs are idempotent, so as soon as we solve this the next releases change to land should catch everything up14:41
elodillesfingers crossed X14:42
ttxweird that release-post pipeline would not get run as it has the same trigger as the post pipeline, which runs alright14:42
ttxi.e. ref-updated with  ref: ^refs/heads/.*$14:42
ttxone is supercedent and the other is independent butthat's about the only diff14:43
bauzasfungi: others, I get that if those jobs aren't idempotent, the question would be "what happens with a revert then ?"14:43
bauzasI mean, what are the post-jobs of a releases revert patch ? 14:44
ttxwe do not revert releases14:44
ttxonce released, always there14:44
bauzasok, so technically, if we were needed to tag releases again, a revert and then a revert of revert would do the trick 14:46
fungittx: yeah, my first guess is something may have changed in the zk schema in a backward-incompatible way, maybe only affecting supercedent pipelines14:46
ttx(post is supercedent, release-post is independent)14:47
ttxbeats me14:47
fungibauzas: the fact that it's a supercedent pipeline means we already skip all queued items between the running one and the most recent to be added, which implies the jobs are assumed to all be idempotent14:47
bauzasTIL14:48
fungioh, hmm so release-post isn't supercedent?14:48
fungii'll double-check in a moment14:48
fungii was misunderstanding which one you meant was which, i guess14:48
ttxhttps://opendev.org/openstack/project-config/src/branch/master/zuul.d/pipelines.yaml#L28614:50
fungifirst occurrence of "json.decoder.JSONDecodeError: Extra data:" in the scheduler logs appears to be 2023-02-06 17:15:19,025 utc14:50
ttxafaict it's not, but that's the only difference it has with the 'post' pipeline14:51
ttxfungi: that would match, no release over the weekend anyway14:51
fungilast successful use of the pipeline was when 868588 merged 2023-02-03 16:08:4014:58
fungiso an approximate 3-day window where things appear to have broken14:58
fungier, started breaking14:58
elodilles(yepp, that's the freezer-tempest-plugin i think)14:59
elodilles(yepp: https://review.opendev.org/c/openstack/releases/+/868588/ )15:01
fungiwe automatically perform a graceful upgrade of all of zuul over the weekend, which is likely when the problem came in15:03
fungithe other pipeline which seems to be impacted is deploy, which explains a seemingly unrelated but lower priority mystery i had on my to do list to look into (we're apparently a few days behind on deploying infrastructure changes)15:06
bauzasfungi: ttx: then when can we expect to have the release-post pipeline to catch up ?15:07
fungibauzas: it might be a day, if we need to patch zuul to recognize existing data in zk under the prior model15:08
bauzasack, noted15:10
bauzasfungi: as I said, my only concern is about making sure we're not holding someone else's series to be merged in nova due to a missing lib upgrade15:11
bauzasbut we'll freeze on thur15:11
bauzasI just hope the requirements team will be able to catch up the flow of releases bumps15:12
fungiit could be sooner if it's just some data corruption we need to manually clean up15:12
bauzas++15:24
opendevreviewBrian Rosmaita proposed openstack/releases master: Release final os-brick for 2023.1 Antelope  https://review.opendev.org/c/openstack/releases/+/87277815:30
fungiokay, so the good news is that this is corrupted znodes which are in the process of being cleared now so we should be able to get stuff moving again in a few minutes. the better news is that this is probably a race condition which was identified over a week ago and has a fix at https://review.opendev.org/872482 which will hopefully merge today and get pulled in with our automatic15:37
fungiupgrades tomorrow, hopefully preventing recurrence15:37
bauzasfungi: great news !15:42
elodillesthanks for the info fungi \o/15:52
fungii'll let you know when we're ready to try approving another release request and see if everything which merged previously ends up getting tagged15:56
elodillesfungi: ack, we have a patch that is waiting for approval, so that sounds good \o/16:00
fungielodilles: may not even be necessary. apparently the schedulers stored up all the items they had been unable to enqueue while the pipeline cache was broken, and they're running now16:01
elodillesfungi: perfect \o/16:03
elodillesbauzas gibi : from pypi: os-traits -- Fri, 26 Aug 2022 17:09:34 GMT -- 2.9.016:25
elodillesbah16:25
gibiyeah we still need 2.1016:25
elodillessry :D16:25
elodillesfalse alarm :D16:26
gibiI know you are excited :) so much releases :D16:26
bauzaselodilles: don't tell me the past, I want the future :p16:27
elodillesguilty as charged o: )16:30
elodillesbauzas gibi : nah, there you have it: os-traits -- Fri, 10 Feb 2023 16:30:49 GMT -- 2.10.016:32
gibi\o/ then we can all go home and enjoy our weekends ;)16:34
gibithanks folks16:34
bauzasbravo and kudos for the hard work16:34
bauzaselodilles: I'll look at the requirements patches to see whether the bot posted them16:35
elodillesbauzas: it should be on the way: https://review.opendev.org/q/project:openstack/requirements+topic:new-release+is:open16:35
bauzasthe requirements team will enjoy16:36
fungisorry about the missed problem for nearly a week, but glad it was fixed without having to revert anything16:49
opendevreviewElod Illes proposed openstack/releases master: Create Train and Ussuri EOL tags  https://review.opendev.org/c/openstack/releases/+/87185716:50
fungiand as i said, after this weekend the race condition which caused that corruption should hopefully no longer be present16:50
elodillesack, thanks16:52
elodillesi time to time check the releases on pypi so it's an unfortunate situation that it wasn't me who realised the issue :/16:53
elodillesi even checked os-traits for nova team, but this time missed to look pypi /o\16:53
elodillescould have been noticed earlier... :/16:53
elodillesrelease-job-failure mail is so convenient that we forgot to be more careful o:)16:55
*** amoralej is now known as amoralej|off17:08
funginah, it's really not all that often we have a bug which causes jobs to not run and not report a failure. easy to overlook unfortunately18:56
fungithe backup in the tag and release-post pipelines reminds me that we still have serialized contention around the release notes and releases site content publication jobs19:59
fungiwe're about 4 hours in and still have 7 items in tag and 24 in release-post waiting for their turn to run those jobs20:00
opendevreviewEric Arellano proposed openstack/reno master: Add support for subsections  https://review.opendev.org/c/openstack/reno/+/87316520:32

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