| *** sdake has joined #openstack-release | 00:03 | |
| *** dave-mccowan has quit IRC | 00:07 | |
| *** mriedem has quit IRC | 00:13 | |
| *** mriedem has joined #openstack-release | 00:19 | |
| *** lbragstad has quit IRC | 00:32 | |
| *** dangtrinhnt has quit IRC | 00:40 | |
| *** sdake has quit IRC | 01:08 | |
| *** dangtrinhnt has joined #openstack-release | 01:17 | |
| openstackgerrit | Tony Breeds proposed openstack/releases master: Generate the constraints redirections from the deliverable data https://review.openstack.org/639011 | 01:31 |
|---|---|---|
| openstackgerrit | Tony Breeds proposed openstack/releases master: Generate the constraints redirections from the deliverable data https://review.openstack.org/639011 | 01:50 |
| *** sdake has joined #openstack-release | 02:09 | |
| *** hongbin has joined #openstack-release | 02:32 | |
| *** sdake has quit IRC | 02:46 | |
| *** ekcs has quit IRC | 02:47 | |
| *** udesale has joined #openstack-release | 03:15 | |
| *** sdake has joined #openstack-release | 03:22 | |
| *** sdake has quit IRC | 03:23 | |
| *** sdake has joined #openstack-release | 03:27 | |
| *** mriedem has quit IRC | 03:32 | |
| *** sdake has quit IRC | 03:40 | |
| *** spsurya has joined #openstack-release | 04:14 | |
| openstackgerrit | dharmendra kushwaha proposed openstack/releases master: tacker-horizon stein release 0.13.0 https://review.openstack.org/639634 | 04:18 |
| *** ykarel|away has joined #openstack-release | 04:19 | |
| *** ykarel|away is now known as ykarel | 04:20 | |
| *** armax has quit IRC | 05:00 | |
| *** hongbin has quit IRC | 05:16 | |
| *** sdake has joined #openstack-release | 05:31 | |
| *** sdake has quit IRC | 06:02 | |
| *** sdake has joined #openstack-release | 06:22 | |
| *** sdake has quit IRC | 06:26 | |
| *** sdake has joined #openstack-release | 06:42 | |
| *** sdake has quit IRC | 07:15 | |
| *** sdake has joined #openstack-release | 07:18 | |
| *** sdake has quit IRC | 07:19 | |
| *** aojea has joined #openstack-release | 07:25 | |
| *** tesseract has joined #openstack-release | 08:00 | |
| *** jpich has joined #openstack-release | 08:00 | |
| *** jtomasek has joined #openstack-release | 08:01 | |
| openstackgerrit | baisen proposed openstack/releases master: Relase tricircle for Stein https://review.openstack.org/639945 | 08:17 |
| *** pcaruana has joined #openstack-release | 08:32 | |
| *** tosky has joined #openstack-release | 08:39 | |
| *** hberaud|gone is now known as hberaud | 08:40 | |
| *** ykarel is now known as ykarel|lunch | 08:43 | |
| openstackgerrit | Akihiro Motoki proposed openstack/releases master: release neutron-lib 1.25.0 https://review.openstack.org/639255 | 08:55 |
| *** jaosorior has joined #openstack-release | 09:06 | |
| *** electrofelix has joined #openstack-release | 09:11 | |
| *** sdake has joined #openstack-release | 09:14 | |
| *** olivierbourdon38 has joined #openstack-release | 09:20 | |
| *** olivierbourdon38 has quit IRC | 09:22 | |
| *** udesale has quit IRC | 09:28 | |
| *** udesale has joined #openstack-release | 09:29 | |
| *** e0ne has joined #openstack-release | 09:32 | |
| *** sdake has quit IRC | 09:46 | |
| *** ykarel|lunch is now known as ykarel | 09:48 | |
| *** dtantsur|afk is now known as dtantsur | 09:49 | |
| openstackgerrit | Chris Dent proposed openstack/releases master: Release 0.3.0 of os-resource-classes https://review.openstack.org/639971 | 10:03 |
| *** sohny has joined #openstack-release | 10:21 | |
| *** udesale has quit IRC | 10:24 | |
| *** udesale has joined #openstack-release | 10:24 | |
| *** sohny has quit IRC | 10:34 | |
| *** sdake has joined #openstack-release | 10:37 | |
| *** luizbag has joined #openstack-release | 10:38 | |
| *** smrcascao has quit IRC | 10:40 | |
| sean-k-mooney | smcginnis: tonyb o/ melanie has +1'd https://review.openstack.org/#/c/639860/ over night. when ye get online could ye take a look again. | 11:00 |
| * ttx waves | 11:01 | |
| *** sdake has quit IRC | 11:18 | |
| *** sdake has joined #openstack-release | 11:20 | |
| *** e0ne has quit IRC | 11:30 | |
| *** e0ne has joined #openstack-release | 11:36 | |
| *** jpich has quit IRC | 11:47 | |
| openstackgerrit | Javier Peña proposed openstack/releases master: Release pymod2pkg 0.17.2 https://review.openstack.org/640010 | 11:53 |
| openstackgerrit | Thomas Bechtold proposed openstack/releases master: pymod2pkg: release 0.17.2 https://review.openstack.org/640011 | 11:53 |
| *** sdake has quit IRC | 11:55 | |
| *** hberaud is now known as hberaud|lunch | 12:14 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/releases master: TripleO: minor release for rocky https://review.openstack.org/637858 | 12:55 |
| *** luizbag has quit IRC | 12:58 | |
| *** dave-mccowan has joined #openstack-release | 13:11 | |
| *** luizbag has joined #openstack-release | 13:16 | |
| dhellmann | evrardjp : I was going to +2 https://review.openstack.org/#/c/639000/ (where you have a -1) and wanted to talk about your question first. Do you have a minute? | 13:16 |
| *** hberaud|lunch is now known as hberaud | 13:27 | |
| evrardjp | hey | 13:27 |
| evrardjp | removing my -1 first because it got attention | 13:27 |
| evrardjp | I am now listening | 13:27 |
| dhellmann | hey | 13:28 |
| dhellmann | I think the commit message says what you're asking it to say, but in a way that may not be clear if you don't have the background for what tonyb is doing | 13:28 |
| dhellmann | the follow-up patch is the thing that needs this data, did you have time to review that? | 13:29 |
| evrardjp | I think I opened it and read it without leaving a comment | 13:29 |
| dhellmann | k | 13:29 |
| evrardjp | the redirection bit, right? | 13:29 |
| dhellmann | yeah | 13:29 |
| dhellmann | the idea is to have a URL per series for the constraints file and have that keep working after the branch is closed | 13:30 |
| evrardjp | please let me check something real quick first | 13:30 |
| evrardjp | ok understood it | 13:30 |
| dhellmann | also to simplify dealing with the timing of when we change the URL in the tox.ini files | 13:30 |
| evrardjp | when I reviewed it I was in the train no pun there | 13:30 |
| dhellmann | haha | 13:30 |
| evrardjp | I was wondering if newton was still an open branch, vs an eol tag, because I thought newton was kaput | 13:31 |
| dhellmann | the newton branch of the requirements repo is still open | 13:31 |
| evrardjp | apparently for requirements it is not | 13:31 |
| evrardjp | that explains | 13:31 |
| evrardjp | ocata and above already have their content I guess | 13:31 |
| evrardjp | let me check real quick in releases | 13:31 |
| dhellmann | right | 13:31 |
| dhellmann | there have been some issues with getting newton closed down completely | 13:31 |
| evrardjp | got it | 13:32 |
| dhellmann | right, so this is just adding the missing data to produce enough useful redirects | 13:32 |
| evrardjp | ok I understand the state better | 13:33 |
| evrardjp | but wouldn't releases: projects requirements be enough to tag something ? | 13:33 |
| dhellmann | good. It seemed like this was a case of us not filling in "the new guys" about the work in progress :-) | 13:33 |
| evrardjp | haha | 13:34 |
| dhellmann | I'm not sure what you mean? | 13:34 |
| evrardjp | well I have so many questions, where to start ... :D | 13:34 |
| dtantsur | hi folks! is it possible to list the deliverables under a project that have the final release today? | 13:34 |
| evrardjp | dtantsur: there sure is a command for that | 13:34 |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release metalsmith 0.11.0 and create stable/stein https://review.openstack.org/640028 | 13:35 |
| evrardjp | sorry I am checking if there is something in process that can help you at the same time | 13:36 |
| dhellmann | we should add some docs about the commands under https://releases.openstack.org/reference/using.html | 13:37 |
| dhellmann | we used to have docs in the readme, but I don't see where they moved | 13:38 |
| dhellmann | oh, no, that was the other repo, release-tools, where we did that | 13:38 |
| evrardjp | dtantsur: we do not have that documented -- I guess you could use the existing venv commands | 13:38 |
| dhellmann | documenting a bunch of useful incantations of list-deliverables would be a good project for train | 13:38 |
| dtantsur | ++ | 13:39 |
| dhellmann | today is the non-client library deadline, right? | 13:39 |
| evrardjp | I have a few ideas there, because I think we have multiple places to link from and to | 13:39 |
| evrardjp | dhellmann: yes | 13:39 |
| dhellmann | that would be --type library to list-deliverables | 13:39 |
| dhellmann | so something like this: tox -e venv -- list-deliverables --type library --team NAME | 13:39 |
| evrardjp | dtantsur: you can do t-x -e venv -- list-deliverables --help to see arguments you can pass | 13:40 |
| evrardjp | dhellmann: shouldn't the horizon libaries also be evaluated? | 13:40 |
| evrardjp | I mean horizon plugins | 13:40 |
| evrardjp | we discussed this | 13:40 |
| evrardjp | my memory is already flushed | 13:40 |
| dhellmann | oh, maybe? | 13:41 |
| dhellmann | so add --type horizon-plugin to that command, dtantsur | 13:41 |
| evrardjp | ahah I am not alone then | 13:41 |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic-lib 2.16.2 and create stable/stein https://review.openstack.org/640029 | 13:41 |
| * dtantsur tries | 13:41 | |
| evrardjp | I think we maybe decided to not have the --type horizon-plugin to do it later, but my memory is... flaky there | 13:41 |
| evrardjp | it's very complex for me, I need one week-end to read the whole thing, draw diagrams, and let it soak. | 13:42 |
| dhellmann | yeah, I could see us considering those "clients" or even leaving them to the final deadline | 13:42 |
| dhellmann | do we want the stable branches created this week? | 13:42 |
| evrardjp | anyway, I will return to my series of questions then :p | 13:42 |
| evrardjp | dhellmann: that's early ? | 13:43 |
| evrardjp | I thought that was rc time | 13:43 |
| dhellmann | yes, we don't branch the services until rc1 | 13:43 |
| dhellmann | I suppose if there is feature work to be done in a library we would want to branch it now | 13:43 |
| dhellmann | I would have to find smcginnis' email to see what he recommended | 13:44 |
| dhellmann | ah, point 5 of https://releases.openstack.org/reference/process.html#final-library-release-week-before-milestone-3 | 13:44 |
| dhellmann | "allow ... do not require" | 13:44 |
| evrardjp | makes sense | 13:45 |
| dhellmann | evrardjp : so, back to the redirect stuff? | 13:47 |
| *** lbragstad has joined #openstack-release | 13:54 | |
| openstackgerrit | Merged openstack/releases master: pymod2pkg: release 0.17.2 https://review.openstack.org/640011 | 14:03 |
| openstackgerrit | Merged openstack/releases master: Release Senlin Stein b1 https://review.openstack.org/639806 | 14:10 |
| *** mriedem has joined #openstack-release | 14:13 | |
| *** ykarel has quit IRC | 14:15 | |
| *** smrcascao has joined #openstack-release | 14:15 | |
| *** ykarel has joined #openstack-release | 14:15 | |
| dtantsur | oh, forgot to say thanks you: list-deliverables helped me | 14:15 |
| openstackgerrit | Merged openstack/releases master: tacker-horizon stein release 0.13.0 https://review.openstack.org/639634 | 14:17 |
| openstackgerrit | Merged openstack/releases master: release os-vif 1.15.1 https://review.openstack.org/639860 | 14:17 |
| evrardjp | dtantsur: good to hear that : ) | 14:17 |
| smcginnis | I believe we've always treated horizon plugins and client libs. Since they are kind of clients. | 14:20 |
| *** ianychoi has joined #openstack-release | 14:20 | |
| dhellmann | that seems right to me | 14:22 |
| ttx | in the past we treated them as "other" deliverables -- meaning they could be released late | 14:23 |
| ttx | Nothing "depends" on them | 14:23 |
| ttx | they are at the top of the foodchain | 14:23 |
| openstackgerrit | Colleen Murphy proposed openstack/releases master: Release keystoneauth 3.12.0 https://review.openstack.org/640041 | 14:26 |
| openstackgerrit | Merged openstack/releases master: Release cinder 11.2.0 https://review.openstack.org/636401 | 14:27 |
| openstackgerrit | Colleen Murphy proposed openstack/releases master: Release keystonemiddleware 6.0.0 https://review.openstack.org/640045 | 14:32 |
| openstackgerrit | Colleen Murphy proposed openstack/releases master: Release python-keystoneclient 3.19.0 https://review.openstack.org/640047 | 14:35 |
| smcginnis | cmurphy: Client lib deadline isn't until next week. Now is fine, but just making sure you're aware in case there might be more the team would want to get in. | 14:38 |
| cmurphy | smcginnis: we have another keystonemiddleware change that depends on the client change, we're thinking maybe we can squeeze that in today but understand if we can't | 14:39 |
| cmurphy | in any case getting the ci green for the ksm change would be nice | 14:39 |
| smcginnis | cmurphy: So the non-client lib has a dependency on the client lib? | 14:40 |
| cmurphy | smcginnis: yes :/ | 14:40 |
| smcginnis | cmurphy: I see. | 14:40 |
| openstackgerrit | Chris Dent proposed openstack/releases master: Release 0.3.0 of os-resource-classes https://review.openstack.org/639971 | 14:52 |
| *** mlavalle has joined #openstack-release | 14:56 | |
| evrardjp | that's a pickle | 15:13 |
| *** e0ne has quit IRC | 15:16 | |
| *** e0ne has joined #openstack-release | 15:19 | |
| evrardjp | I see a session called PTL tips and tricks in the denver forum planning -- is there something we should do for release in that context? Gathering feedback from ptl/release liaison just after that one would be useful. | 15:22 |
| smcginnis | Not a bad idea. | 15:24 |
| smcginnis | We did want to try to get feedback on the changes we've made this cycle to see if they feel it's helped them or not. | 15:24 |
| smcginnis | Might be a good opportunity to do that. | 15:24 |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release sushy 1.8.0 and create stable/stein https://review.openstack.org/640061 | 15:28 |
| bnemec | As the proposer of that session I would be good with having a section for release discussion. :-) | 15:32 |
| bnemec | In fact, that's sort of why I'm in this channel right now. | 15:32 |
| bnemec | I made a note for myself last cycle that I should make sure to release all the Oslo projects, even if they only had test-related changes. | 15:33 |
| bnemec | The reason was that some of our stable branch jobs broke last time around because we branched off old commits that were missing necessary test fixes. | 15:33 |
| bnemec | Since this week is non-client library final releases, do I need to do that now or is there some other way to handle it? | 15:34 |
| dhellmann | bnemec : it seems like a good thing to do now | 15:37 |
| bnemec | dhellmann: Okay, thanks. I'll get that proposed ASAP. | 15:38 |
| evrardjp | bnemec: thanks | 15:38 |
| dhellmann | ++ | 15:38 |
| *** armax has joined #openstack-release | 15:46 | |
| openstackgerrit | Ben Nemec proposed openstack/releases master: Final Oslo releases for Stein https://review.openstack.org/640076 | 15:56 |
| bnemec | That's a whole buncha releases. | 15:57 |
| smcginnis | Wow, dang, that is. :) | 15:57 |
| *** ykarel is now known as ykarel|away | 15:58 | |
| smcginnis | #startmeeting releaseteam | 16:00 |
| openstack | Meeting started Thu Feb 28 16:00:03 2019 UTC and is due to finish in 60 minutes. The chair is smcginnis. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| *** openstack changes topic to " (Meeting topic: releaseteam)" | 16:00 | |
| openstack | The meeting name has been set to 'releaseteam' | 16:00 |
| smcginnis | Ping list: smcginnis ttx dhellmann diablo_rojo hberaud evrardjp fungi armstrong | 16:00 |
| smcginnis | #link https://etherpad.openstack.org/p/stein-relmgt-tracking Agenda | 16:00 |
| dhellmann | o/ | 16:00 |
| fungi | aloha | 16:00 |
| smcginnis | Getting really far down in that tracking etherpad now. | 16:00 |
| *** armstrong has joined #openstack-release | 16:00 | |
| diablo_rojo | o/ | 16:00 |
| evrardjp | o/ | 16:01 |
| ttx | o/ | 16:01 |
| smcginnis | #topic TripleO release issues | 16:02 |
| *** openstack changes topic to "TripleO release issues (Meeting topic: releaseteam)" | 16:02 | |
| smcginnis | #link https://review.openstack.org/#/c/637857 | 16:02 |
| smcginnis | #link https://review.openstack.org/#/c/637858/ | 16:02 |
| smcginnis | #link https://review.openstack.org/#/c/638459/ | 16:02 |
| smcginnis | Tasks update | 16:02 |
| smcginnis | Oops, a little too much grabbed on that last one. :) | 16:02 |
| smcginnis | Not sure who added this topic. | 16:03 |
| hberaud | o/ | 16:03 |
| smcginnis | http://logs.openstack.org/57/637857/4/check/openstack-tox-validate/502eeab/job-output.txt.gz#_2019-02-22_10_12_23_715253 | 16:03 |
| smcginnis | Looks like they need a release job defined and they have errors in their readme. | 16:04 |
| evrardjp | it looks like ttx might have added this? | 16:04 |
| smcginnis | ttx: This one yours? | 16:04 |
| evrardjp | see also last comments on https://review.openstack.org/#/c/637857 | 16:04 |
| ttx | yes | 16:04 |
| ttx | It looks like this pile of things are blocked | 16:05 |
| ttx | First do fail due to a validation error | 16:05 |
| ttx | (missing release job for instack-undercloud) | 16:06 |
| ttx | but apparently fixing that is not the solution | 16:06 |
| ttx | see https://review.openstack.org/#/c/638459/ | 16:06 |
| smcginnis | I thought that was actually what was needed. | 16:07 |
| ttx | yeah, me too | 16:07 |
| smcginnis | Maybe we just need to let Andreas know that? | 16:07 |
| ttx | but sending Juan Antonio falling down the cracks between our teams is not the best we can do | 16:07 |
| ttx | "You have everything set up to release - the release repo check just does not know ;)" | 16:07 |
| smcginnis | https://github.com/openstack/tripleo-image-elements/blob/stable/queens/zuul.d/layout.yaml#L8 | 16:08 |
| evrardjp | sorry it was kinda my fault to ping then | 16:08 |
| evrardjp | it's been laying around so I wanted some fresh eyes :) | 16:08 |
| smcginnis | I didn't think we could do that in-repo due to secrets. | 16:08 |
| dhellmann | yeah, we require the template so we don't have to know all of the other places jobs can be attached to projects | 16:08 |
| smcginnis | dhellmann: Can you comment on https://review.openstack.org/#/c/638459/ | 16:08 |
| ttx | maybe our repo check is a bit too restrictive. Since we did have a light agenda I suggested we look into it and give them an answer :) | 16:08 |
| dhellmann | the issue is that they have listed individual jobs in the release queue so ajaeger doesn't want to add the template | 16:08 |
| dhellmann | commented | 16:09 |
| dhellmann | I also restored the patch | 16:09 |
| smcginnis | So we either need to expand our validation checking for whatever combination folks try to add, or we enforce that they are consistent with how this is done. | 16:09 |
| smcginnis | I like consistency. | 16:09 |
| ttx | expanding our validation is unfortunately not easy | 16:10 |
| diablo_rojo | +2 for consistency | 16:10 |
| smcginnis | But, they do still need to get their README fixed too. | 16:10 |
| fungi | anybody know the backstory on why they aren't using the template today? | 16:10 |
| ttx | without reimplementing the full Zuul logic for it | 16:10 |
| fungi | like is it just historical baggage or are there jobs in the template they don't want to (or can't) run? | 16:10 |
| dhellmann | fwiw, this hasn't been a problem since I made the decision to require using the templates at the very beginning of writing the validation | 16:10 |
| smcginnis | Looks like that patch could be updated to also remove the individual jobs if they are adding the template. | 16:10 |
| evrardjp | couldn't the freeze job details help here? | 16:11 |
| dhellmann | there are some cross-dependencies listed there. maybe those are redundant? | 16:11 |
| smcginnis | Those seem... odd. | 16:11 |
| ttx | yes, that patch should drop the individual job def | 16:11 |
| fungi | i've lost track apparently. where are you looking at cross-dependencies? | 16:12 |
| *** udesale has quit IRC | 16:13 | |
| dhellmann | https://review.openstack.org/#/c/638459/1/zuul.d/projects.yaml line 4523 and below | 16:13 |
| ttx | "these have to dupe the publish-to-pypi jobs | 16:13 |
| ttx | # because of the branch designation for the release job | 16:13 |
| ttx | " | 16:13 |
| dhellmann | not "cross" but dependencies | 16:13 |
| fungi | oh, i didn't think to look at context further down below the diff | 16:13 |
| dhellmann | ajaeger's objection to the patch seemed to be that the template was redundant | 16:14 |
| ttx | so the duplication was intentional | 16:14 |
| smcginnis | Which is valid. But our validation enforces that convention. | 16:14 |
| *** dangtrinhnt has quit IRC | 16:15 | |
| dhellmann | I suppose this wouldn't be an issue if the master branch wasn't closed? | 16:15 |
| smcginnis | I think that's the only gotcha in all that. | 16:15 |
| *** dangtrinhnt has joined #openstack-release | 16:15 | |
| ttx | The issue here is that the template does not really match what they have. So we should add another template to match what they have, and add that to the validation test | 16:15 |
| fungi | why does it matter that they don't run jobs on master if they're not accepting patches for master anyway? | 16:16 |
| ttx | some publish-tripleo template mabe | 16:16 |
| dhellmann | ttx: in what way does it not match? | 16:16 |
| *** e0ne has quit IRC | 16:16 | |
| fungi | dhellmann: the branch exclusions | 16:16 |
| smcginnis | fungi: Yeah, there won't be any releases on master, so does it really matter? | 16:16 |
| dhellmann | I don't see any branch exclusions in this patch, are those in the template? | 16:17 |
| fungi | dhellmann: the lines just after the diff | 16:17 |
| dhellmann | those are the check jobs | 16:17 |
| dhellmann | I don't see any exclusions in the release jobs | 16:17 |
| ttx | fungi: maybe the check/gate could stay | 16:17 |
| *** dave-mccowan has quit IRC | 16:17 | |
| fungi | yeah, those are the ones i was talking about, not the release jobs | 16:18 |
| dhellmann | oh, I see, those are release test jobs | 16:18 |
| fungi | just wondering why they even need to exclude master if they're not accepting changes for master | 16:18 |
| dhellmann | I suppose they had to exclude the job from master in order to close it and change the readme in the first place | 16:18 |
| dhellmann | they could probably remove all of that now | 16:18 |
| fungi | right, cleanup | 16:18 |
| dhellmann | wfm | 16:18 |
| evrardjp | I am confused there | 16:19 |
| ttx | ok, so remove all jobs and replace them with the template | 16:19 |
| smcginnis | This is getting into zuul configuration, so confusion is normal. :) | 16:19 |
| dhellmann | s/normal/required/ | 16:19 |
| evrardjp | I would hope not to me :p | 16:19 |
| dhellmann | here's what I think happened: | 16:19 |
| dhellmann | 1. one day, a long time ago, this was a normal project. | 16:20 |
| evrardjp | but they don't want to run those jobs for master, which makes sense? | 16:20 |
| fungi | evrardjp: the complexity seems to be that they've retired the instack-undercloud repo, but because it wasn't retired until the current cycle they had to leave testing in place to accept fixes on stable branches | 16:20 |
| dhellmann | 2. they closed master to new development, but kept the stable branches open | 16:20 |
| dhellmann | 3. to do that, they had to disable some jobs on master | 16:20 |
| dhellmann | 4. they did that by listing them all out explicitly instead of using the template | 16:20 |
| dhellmann | 5. now the release validation is complaining because the template is not there | 16:20 |
| evrardjp | because they would need to override per branch in step 4 | 16:20 |
| fungi | right | 16:21 |
| dhellmann | right | 16:21 |
| fungi | i agree with dhellmann's probable timeline there | 16:21 |
| evrardjp | me too, that's my understanding | 16:21 |
| evrardjp | I don't understand how cleaning would help :p | 16:21 |
| ttx | ok, should we propose the fix? Or go to write a lengthy explanation and expect someone else to push it? | 16:21 |
| smcginnis | So since they are not accepting changes to master, now that they've gotten past that final merge to master there is no need to have special handling. | 16:21 |
| dhellmann | the fix is to clean up the explicit job listings and apply the template | 16:21 |
| dhellmann | the jobs will run on master and fail, but we don't care | 16:21 |
| fungi | we're basically missing a step after retiring the master branch to put the jobs back the way they were | 16:21 |
| dhellmann | evrardjp : because it removes ajeager's objection to having redundant information in the config | 16:21 |
| evrardjp | smcginnis: oh ok | 16:21 |
| dhellmann | ttx: perhaps we should do it | 16:22 |
| ttx | ++ | 16:22 |
| smcginnis | This is an odd case where a repo was retired but they still care about stable. | 16:22 |
| ttx | I can push it if we agree | 16:22 |
| dhellmann | ttx: ++ | 16:22 |
| evrardjp | thanks ttx | 16:22 |
| smcginnis | ttx: That would be great. | 16:22 |
| ttx | (I'll just remove lines 4506 and below) | 16:22 |
| ttx | that will do it right | 16:22 |
| dhellmann | ttx: perhaps with a nice big comment in there with the template explaining why it's ok | 16:22 |
| dhellmann | yes | 16:22 |
| ttx | I'll explain in commit message | 16:23 |
| evrardjp | I still believe this is not the ultimate right way, but I am not sure we have the right tools now | 16:23 |
| *** e0ne has joined #openstack-release | 16:23 | |
| smcginnis | evrardjp: What is you suggestion? | 16:23 |
| dhellmann | what would you do instead? | 16:23 |
| evrardjp | but hey, we all agree :) | 16:23 |
| evrardjp | I have nothing for now, but I would think changing the validation to use the future zuul's freeze api would help | 16:24 |
| smcginnis | Future zuul fixes everything. :) | 16:24 |
| evrardjp | because we could freeze the job info per branch, so it means for the release we are looking for we could know or not if that job exists | 16:24 |
| dhellmann | perhaps. the point of using the templates was to enforce an *interface* for releasing without enforcing details that tend to change a lot | 16:24 |
| evrardjp | fair :) | 16:24 |
| dhellmann | i.e., we added test jobs to that template this cycle | 16:25 |
| evrardjp | in any way, that's only about the future :D | 16:25 |
| ttx | fungi: should I keep a noop in check nd gate pipelines? | 16:25 |
| fungi | evrardjp: i might agree with you if we even had consensus on the patch series for those zuul features yet | 16:25 |
| evrardjp | sorry to digress here | 16:25 |
| fungi | ttx: nope, the noop job can go away | 16:25 |
| evrardjp | fungi: :D | 16:25 |
| smcginnis | I see they are in the process of merging a README fix, so the other issue should be resolved shortly too. | 16:25 |
| dhellmann | agreed | 16:25 |
| smcginnis | Any more on these? | 16:26 |
| smcginnis | #topic Tasks update | 16:26 |
| fungi | if they're adjusting the readme on master, then we shouldn't remove the jobs in 638459 until that's done | 16:26 |
| *** openstack changes topic to "Tasks update (Meeting topic: releaseteam)" | 16:26 | |
| dhellmann | I hope they're only adjusting it in the releasable branches? | 16:26 |
| smcginnis | fungi: No, master and stable/rocky are done. They just hadn't fixed queens. | 16:26 |
| fungi | okay, in that case it's fine | 16:26 |
| smcginnis | diablo_rojo: Did you follow up with QA? | 16:27 |
| diablo_rojo | smcginnis, I did | 16:27 |
| diablo_rojo | sounded like he had it handled and didn't need anything from us | 16:27 |
| ttx | Done @ https://review.openstack.org/#/c/638459/ | 16:27 |
| smcginnis | diablo_rojo: Excellent, thanks. | 16:27 |
| smcginnis | And thanks ttx | 16:27 |
| diablo_rojo | smcginnis, no problem :) | 16:28 |
| smcginnis | fungi: Any updates on signing key? | 16:28 |
| smcginnis | The other task is for generating release requests for any missed c-w-i libs. | 16:29 |
| fungi | #link https://sks-keyservers.net/pks/lookup?op=vindex&search=0xcdc08088c3cb45a9be08332b2354069e5b504663&fingerprint=on OpenStack Infra (Train Cycle) <infra-root@openstack.org> | 16:29 |
| smcginnis | I can take that unless someone else want it. | 16:29 |
| smcginnis | fungi: Great, thanks! | 16:29 |
| fungi | i'm waiting for it to appear on the keyservers i'm pulling from so i can attest to it | 16:29 |
| fungi | so hopefully later today you'll see a signature from me on it there | 16:30 |
| fungi | but it's already signed by the stein cycle key | 16:30 |
| ttx | fungi: wondering why we still need the queue defined in https://review.openstack.org/#/c/638459/ but hey I'll add it | 16:30 |
| smcginnis | No one clamoring for doing generating the release patches, so I'll do it tomorrow once we're past the deadline and see what's left. | 16:31 |
| fungi | ttx: that's so the tripleo jobs all share a named change queue | 16:31 |
| ttx | ah ok | 16:31 |
| smcginnis | Looking ahead to next week's tasks, same thing for generating the client libs that are missed. | 16:32 |
| fungi | basically anything running those (so any changes for instack-undercloud) will end up in the tripleo queue even if they don't share jobs with other projects in that queue | 16:32 |
| smcginnis | I've put prometheanfire down in there as a reminder for requirements freeze. | 16:33 |
| dhellmann | smcginnis : perhaps we can sign up one of our new team members for that task next week, since they have more notice | 16:33 |
| smcginnis | dhellmann: I like that idea. :) | 16:33 |
| * dhellmann looks meaningfully at evrardjp and diablo_rojo | 16:33 | |
| diablo_rojo | Ha ha so this is just generating another list? | 16:34 |
| smcginnis | It's a matter of listing what deliverables of --type=client-lib have not been released yet and generating release patches for any that have any commits. | 16:34 |
| dhellmann | right, not just a list but creating the patches to trigger the missing releases | 16:34 |
| evrardjp | diablo_rojo: and updating the docs I would say | 16:34 |
| smcginnis | Actually, at this point any regardless of commits as we need a stable/stein branchign point. | 16:34 |
| dhellmann | there are scripts to do it all, I think, and if not we have the parts to build the scripts | 16:34 |
| diablo_rojo | evrardjp, maybe we can work on this together again? | 16:35 |
| evrardjp | I would be fine to sync with diablo_rojo on that | 16:35 |
| evrardjp | yeah | 16:35 |
| smcginnis | The parts are there, but I never had time to start to automate it. | 16:35 |
| diablo_rojo | So we can fumble through it together? | 16:35 |
| diablo_rojo | I think we can figure that out. | 16:35 |
| evrardjp | fine for me | 16:35 |
| ttx | At this point it's ok to approve stein branch creations for libraries, right? | 16:35 |
| evrardjp | we'll ping for validation before sending a mass change | 16:35 |
| smcginnis | Cool, thanks. I've added your nicks to the task in the etherpad. | 16:35 |
| dhellmann | we'll help if you run into trouble, too | 16:35 |
| dhellmann | thank you! | 16:35 |
| dhellmann | ttx: it is, although I've been reminding folks that they can also wait to rc1 | 16:36 |
| smcginnis | It will be a good exercise. | 16:36 |
| dhellmann | our docs say "allow ... but do not require" branches this week | 16:36 |
| smcginnis | If they're pretty sure they are done, no reason not to branch. | 16:36 |
| dhellmann | yeah, it's just a trade-off about potentially having to backport changes to fix bugs | 16:36 |
| smcginnis | But if there's some concern there might still be an important bug fix, branching just causes extra work at this point. | 16:37 |
| ttx | Since deadline is this week and we prefer not to release libs on Fridays... | 16:37 |
| dhellmann | right | 16:37 |
| smcginnis | Last task on the list is to run the aclissues checker. | 16:37 |
| dhellmann | the constraints list protects us from bad releases going into the gate | 16:37 |
| dhellmann | at least mostly | 16:37 |
| ttx | I can do that, unless someone else wants to try that | 16:38 |
| dhellmann | that feels like another case where it would be good to have someone "new" do it to verify the instructions | 16:38 |
| smcginnis | Anyone want to walk through with ttx as he does it to learn? Like maybe someone in a similar timezone? | 16:38 |
| smcginnis | :) | 16:39 |
| ttx | (subtle hint) | 16:39 |
| evrardjp | Wow that was targetted | 16:39 |
| smcginnis | hehe | 16:39 |
| dhellmann | I don't know, where is diablo_rojo this week? | 16:39 |
| evrardjp | I am fine with doing a session with ttx | 16:39 |
| ttx | I'm jetlagges so I'm currently UTC-3 | 16:39 |
| diablo_rojo | dhellmann, physically? or mentall? | 16:39 |
| evrardjp | I think it's simply running a shell script | 16:39 |
| diablo_rojo | *mentally | 16:39 |
| evrardjp | iirc | 16:39 |
| dhellmann | diablo_rojo : temporally | 16:39 |
| smcginnis | evrardjp: Yeah, not a complicated task, but a few things to know. | 16:39 |
| diablo_rojo | dhellmann, in between waking up and going to take a nap | 16:39 |
| dhellmann | +1 for naps | 16:40 |
| diablo_rojo | evrardjp, we could add that to the list too and verify against each other | 16:40 |
| smcginnis | OK, I've put down ttx and evrardjp for that last task. | 16:40 |
| diablo_rojo | since we will both need to know how to do all these things anyway | 16:40 |
| diablo_rojo | Oh | 16:40 |
| * diablo_rojo shuts up | 16:40 | |
| smcginnis | diablo_rojo: If you can join too, that's great. | 16:40 |
| diablo_rojo | smcginnis, might be a good idea | 16:41 |
| evrardjp | the more the merrier | 16:41 |
| smcginnis | Just trying to have a good transfer of knowledge where we can. | 16:41 |
| dhellmann | ++ | 16:41 |
| smcginnis | The other tasks in the process doc are reminders that I've put in the countdown email. | 16:42 |
| smcginnis | Just need to format them into actual content. | 16:42 |
| smcginnis | That should be it for tasks. | 16:42 |
| *** v12aml has quit IRC | 16:42 | |
| smcginnis | #topic Open | 16:42 |
| *** openstack changes topic to "Open (Meeting topic: releaseteam)" | 16:42 | |
| smcginnis | Anything else we should discuss in-meeting? | 16:43 |
| smcginnis | OK, we can get back to work then. | 16:43 |
| smcginnis | Thanks everyone! | 16:43 |
| dhellmann | there was some discussion of forum/ptg slots, how did that end up? | 16:43 |
| smcginnis | Oh, right. | 16:43 |
| *** v12aml has joined #openstack-release | 16:43 | |
| smcginnis | bnemec: Had a topic submitted for the forum that will be a good time to try to get release change feedback from PTLs. | 16:44 |
| fungi | thanks smcginnis! | 16:44 |
| smcginnis | Anyone have a link for that? | 16:44 |
| bnemec | I haven't actually submitted it yet. | 16:44 |
| smcginnis | OK. Let me know if you need anything from us, otherwise we can watch for the session and plan to be there. | 16:45 |
| bnemec | Yeah, I was kind of waiting to submit it to see if there was interest. | 16:46 |
| bnemec | Since there is, I'll go ahead. | 16:46 |
| smcginnis | ++ | 16:46 |
| smcginnis | OK, I guess that's it. Thanks everyone for participating in the release team. | 16:46 |
| smcginnis | #endmeeting | 16:46 |
| *** openstack changes topic to "OpenStack Release Managers office - Come here to discuss how to release OpenStack components - Logged at http://eavesdrop.openstack.org/irclogs/%23openstack-release/" | 16:47 | |
| openstack | Meeting ended Thu Feb 28 16:46:59 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:47 |
| evrardjp | thanks you | 16:47 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/releaseteam/2019/releaseteam.2019-02-28-16.00.html | 16:47 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/releaseteam/2019/releaseteam.2019-02-28-16.00.txt | 16:47 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/releaseteam/2019/releaseteam.2019-02-28-16.00.log.html | 16:47 |
| ttx | thanks! | 16:47 |
| evrardjp | to you* | 16:47 |
| smcginnis | :) | 16:47 |
| evrardjp | and everyone in fact :) | 16:47 |
| diablo_rojo | Thanks smcginnis :) | 16:48 |
| smcginnis | Oh, wanted to remind everyone to please take a look at the proposed Train schedule I put up and see if it is sane. | 16:48 |
| *** dtantsur is now known as dtantsur|afk | 16:55 | |
| openstackgerrit | Merged openstack/releases master: Release metalsmith 0.11.0 and create stable/stein https://review.openstack.org/640028 | 16:57 |
| openstackgerrit | Merged openstack/releases master: Release ironic-lib 2.16.2 and create stable/stein https://review.openstack.org/640029 | 16:57 |
| openstackgerrit | Merged openstack/releases master: Release keystoneauth 3.12.0 https://review.openstack.org/640041 | 16:59 |
| openstackgerrit | Merged openstack/releases master: release neutron-lib 1.25.0 https://review.openstack.org/639255 | 16:59 |
| *** electrofelix has quit IRC | 16:59 | |
| *** e0ne has quit IRC | 17:05 | |
| openstackgerrit | Abhishek Kekane proposed openstack/releases master: Release glance_store 1.0.0 https://review.openstack.org/640103 | 17:06 |
| *** ykarel|away has quit IRC | 17:09 | |
| *** luizbag has quit IRC | 17:16 | |
| *** luizbag has joined #openstack-release | 17:16 | |
| openstackgerrit | Erno Kuvaja proposed openstack/releases master: Release glance_store 1.0.0 https://review.openstack.org/640103 | 17:38 |
| openstackgerrit | Erno Kuvaja proposed openstack/releases master: Release glance_store 0.28.0 https://review.openstack.org/640103 | 17:41 |
| openstackgerrit | Merged openstack/releases master: Release 0.3.0 of os-resource-classes https://review.openstack.org/639971 | 17:46 |
| openstackgerrit | Merged openstack/releases master: Final Oslo releases for Stein https://review.openstack.org/640076 | 17:52 |
| openstackgerrit | Merged openstack/releases master: Release keystonemiddleware 6.0.0 https://review.openstack.org/640045 | 17:53 |
| *** luizbag has quit IRC | 17:56 | |
| openstackgerrit | Merged openstack/releases master: Release sushy 1.8.0 and create stable/stein https://review.openstack.org/640061 | 17:59 |
| openstackgerrit | Merged openstack/releases master: Release python-keystoneclient 3.19.0 https://review.openstack.org/640047 | 18:03 |
| openstackgerrit | Andrey Kurilin proposed openstack/releases master: [rally] Release 1.4.1 https://review.openstack.org/640134 | 18:06 |
| *** aojea has quit IRC | 18:10 | |
| *** ykarel|away has joined #openstack-release | 18:16 | |
| prometheanfire | smcginnis: sorry, meeting... | 18:24 |
| prometheanfire | smcginnis: action needed or just your countdown email? | 18:24 |
| *** sdake has joined #openstack-release | 18:45 | |
| *** armstrong has quit IRC | 18:50 | |
| *** e0ne has joined #openstack-release | 18:51 | |
| *** hberaud is now known as hberaud|gone | 18:56 | |
| *** luizbag has joined #openstack-release | 18:57 | |
| *** ykarel|away has quit IRC | 19:01 | |
| *** luizbag_ has joined #openstack-release | 19:02 | |
| *** e0ne has quit IRC | 19:03 | |
| *** luizbag has quit IRC | 19:04 | |
| smcginnis | prometheanfire: I was just bringing up the task from bullet 2 in our release process for milestone-3 | 19:13 |
| smcginnis | https://releases.openstack.org/reference/process.html#milestone-3 | 19:13 |
| prometheanfire | ah, k | 19:13 |
| *** pcaruana has quit IRC | 19:14 | |
| smcginnis | prometheanfire: Just let us know if there's any other requirements related things the release team needs to be aware of. | 19:14 |
| prometheanfire | atm no | 19:15 |
| *** jtomasek has quit IRC | 19:16 | |
| *** whoami-rajat has quit IRC | 19:20 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release glance_store 0.28.0 https://review.openstack.org/640103 | 20:07 |
| *** sdake has quit IRC | 20:07 | |
| *** jtomasek has joined #openstack-release | 20:11 | |
| *** luizbag_ has quit IRC | 20:13 | |
| *** e0ne has joined #openstack-release | 20:16 | |
| *** tesseract has quit IRC | 20:22 | |
| fungi | the train release signing key is showing a signature by me now: https://sks-keyservers.net/pks/lookup?op=vindex&search=0xcdc08088c3cb45a9be08332b2354069e5b504663&fingerprint=on | 20:26 |
| openstackgerrit | Merged openstack/releases master: Release glance_store 0.28.0 https://review.openstack.org/640103 | 20:31 |
| *** sdake has joined #openstack-release | 21:03 | |
| *** dave-mccowan has joined #openstack-release | 21:11 | |
| *** e0ne has quit IRC | 21:19 | |
| *** dave-mccowan has quit IRC | 21:22 | |
| *** sdake has quit IRC | 21:23 | |
| *** sdake has joined #openstack-release | 21:25 | |
| *** jtomasek has quit IRC | 21:42 | |
| *** e0ne has joined #openstack-release | 21:42 | |
| *** e0ne has quit IRC | 21:44 | |
| *** e0ne has joined #openstack-release | 21:45 | |
| *** zul has quit IRC | 21:53 | |
| *** e0ne has quit IRC | 22:01 | |
| openstackgerrit | Monty Taylor proposed openstack/releases master: Release 0.25.0 of openstacksdk https://review.openstack.org/640202 | 22:04 |
| *** sdake has quit IRC | 22:13 | |
| lbragstad | smcginnis what's the deadline for proposing libraries? | 22:24 |
| lbragstad | we have one patch from mordred we want to get into keystoneauth before cutting another release for it | 22:25 |
| smcginnis | Non-client lib deadline is today. How far along is that fix? | 22:26 |
| mordred | smcginnis: it was green, we just pushed up a rev to fix a review comment - but it should be clear to land as long as zuul is friendly | 22:28 |
| smcginnis | mordred: Awesome. Gate doesn't seem too backed up, so hopefully that shouldn't take long to get through. | 22:29 |
| mordred | smcginnis: fingers crossed | 22:29 |
| openstackgerrit | Merged openstack/releases master: [rally] Release 1.4.1 https://review.openstack.org/640134 | 22:29 |
| openstackgerrit | Witold Bedyk proposed openstack/releases master: Release monasca-common 2.13.0 https://review.openstack.org/640206 | 22:35 |
| openstackgerrit | Merged openstack/releases master: Release 0.25.0 of openstacksdk https://review.openstack.org/640202 | 22:37 |
| openstackgerrit | Witold Bedyk proposed openstack/releases master: Release monasca-statsd 1.11.0 https://review.openstack.org/640208 | 22:39 |
| dtroyer | so question I really should know the answer to but I'm comming up blank… the sha in a release record should not be a merge commit? | 23:04 |
| clarkb | dtroyer: not a release team member but a merge commit should be fine since you may have to tag/release the result of a merge | 23:06 |
| dtroyer | clarkb: thanks, for some reason I had a doubt about that… | 23:06 |
| openstackgerrit | Dean Troyer proposed openstack/releases master: Release osc-lib 1.12.1 https://review.openstack.org/640220 | 23:07 |
| *** armstrong has joined #openstack-release | 23:23 | |
| *** armstrong has quit IRC | 23:35 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!