openstackgerrit | Colleen Murphy proposed openstack/releases master: Auto release request for keystone https://review.opendev.org/684065 | 00:11 |
---|---|---|
*** KeithMnemonic has quit IRC | 00:18 | |
openstackgerrit | Merged openstack/releases master: python-tackerclient 0.16.1 https://review.opendev.org/683851 | 00:37 |
*** diablo_rojo has quit IRC | 00:37 | |
*** armax has quit IRC | 00:46 | |
*** ekcs has quit IRC | 00:47 | |
*** ekcs has joined #openstack-release | 01:53 | |
*** ekcs has quit IRC | 02:14 | |
*** weshay has quit IRC | 02:17 | |
*** ekcs has joined #openstack-release | 02:33 | |
*** ekcs has quit IRC | 02:34 | |
*** ekcs has joined #openstack-release | 02:36 | |
*** ricolin has joined #openstack-release | 02:49 | |
*** ykarel has joined #openstack-release | 02:55 | |
*** ekcs has quit IRC | 03:04 | |
*** dave-mccowan has quit IRC | 03:24 | |
openstackgerrit | Sundar Nadathur proposed openstack/releases master: Auto release request for cyborg https://review.opendev.org/684056 | 04:21 |
*** radeks has joined #openstack-release | 04:55 | |
*** udesale has joined #openstack-release | 05:00 | |
*** tetsuro has quit IRC | 05:37 | |
*** tetsuro has joined #openstack-release | 05:39 | |
*** ykarel is now known as ykarel|afk | 05:40 | |
*** ykarel|afk has quit IRC | 05:45 | |
*** ykarel|afk has joined #openstack-release | 06:00 | |
*** ykarel|afk is now known as ykarel | 06:08 | |
openstackgerrit | Colleen Murphy proposed openstack/releases master: Auto release request for keystone https://review.opendev.org/684065 | 06:22 |
*** e0ne has joined #openstack-release | 06:45 | |
*** e0ne has quit IRC | 06:50 | |
*** hberaud|gone is now known as hberaud | 06:58 | |
*** witek has joined #openstack-release | 07:04 | |
*** rpittau|afk is now known as rpittau | 07:05 | |
*** amoralej|off is now known as amoralej | 07:18 | |
*** pcaruana has joined #openstack-release | 07:21 | |
rakhmerov | hi, I left a comment in https://review.opendev.org/#/c/684072 | 07:36 |
rakhmerov | just a bit confused about the deadline | 07:36 |
rakhmerov | in my understanding it's Sep 27 so we should have some time to squeeze a couple of patches | 07:37 |
*** tosky has joined #openstack-release | 07:37 | |
rakhmerov | please confirm | 07:37 |
*** priteau has joined #openstack-release | 07:43 | |
tosky | evrardjp: hi! Sorry, I missed the reviews for the release of sahara bits, I'm going to approve them - but can I edit them to include slightly newer commits? | 07:50 |
AJaeger | tosky: read http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009721.html - it answers your questions with "Yes" ;) | 07:56 |
tosky | AJaeger: ups, thanks and sorry - I've been (and still am) in a big workshop/meeting for the entire week and I'm still a bit behind with several stuff | 07:57 |
AJaeger | perfect timing ;/ | 08:04 |
*** priteau has quit IRC | 08:09 | |
*** lpetrut has joined #openstack-release | 08:13 | |
openstackgerrit | Luigi Toscano proposed openstack/releases master: Auto release request for sahara-dashboard https://review.opendev.org/684092 | 08:17 |
openstackgerrit | Luigi Toscano proposed openstack/releases master: Auto release request for sahara-plugin-ambari https://review.opendev.org/684095 | 08:19 |
ttx | rakhmerov: commented | 08:21 |
openstackgerrit | Luigi Toscano proposed openstack/releases master: Auto release request for sahara-plugin-cdh https://review.opendev.org/684096 | 08:21 |
ttx | rakhmerov: the general idea is this -- Monday: post autorelease patches, Tuesday-Thursday: merge the ones with PTL+1, Friday: merge the ones without answer, Friday-Monday: merge the remaining ones | 08:22 |
ttx | knowing that you can always do a RC2 | 08:23 |
openstackgerrit | Luigi Toscano proposed openstack/releases master: Auto release request for sahara-plugin-mapr https://review.opendev.org/684097 | 08:23 |
openstackgerrit | Luigi Toscano proposed openstack/releases master: Auto release request for sahara-plugin-spark https://review.opendev.org/684098 | 08:24 |
tosky | ttx: PTL or release liason, I guess :) | 08:25 |
openstackgerrit | Luigi Toscano proposed openstack/releases master: Auto release request for sahara-plugin-storm https://review.opendev.org/684099 | 08:26 |
openstackgerrit | Luigi Toscano proposed openstack/releases master: Auto release request for sahara-plugin-vanilla https://review.opendev.org/684100 | 08:27 |
ttx | yes | 08:30 |
AJaeger | amotoki: does the midonet team really want to release something that looks so broken? https://review.opendev.org/#/c/684615/ (this happens on master as well) | 08:30 |
ttx | witek: looks like https://review.opendev.org/#/c/674812/ merged, so you can update https://review.opendev.org/#/c/684035/ now? | 08:32 |
amotoki | AJaeger: the neutron team needs to contact the midonet team I think. The midonet development looks not active. The gate is health at least one month ago. | 08:33 |
ttx | witek: for monasca-api I feel like https://review.opendev.org/#/c/651249/ is further away... and might be too late for train? | 08:34 |
AJaeger | amotoki: ;( | 08:35 |
ttx | amotoki: could you +1 https://review.opendev.org/#/c/684064/ ? | 08:38 |
ttx | (horizon RC1 posted by Ivan) | 08:38 |
amotoki | ttx: sure. I thought Ivan is the horizon PTL for train, so his post is enough. Anyway I will check it. | 08:39 |
ttx | oh ok, that will do | 08:39 |
ttx | tosky: FYI you can propose new hashes on any of those autorelease patches. | 08:41 |
*** ykarel is now known as ykarel|lunch | 08:41 | |
*** hberaud has quit IRC | 08:42 | |
*** hberaud has joined #openstack-release | 08:44 | |
openstackgerrit | Witold Bedyk proposed openstack/releases master: Auto release request for monasca-notification https://review.opendev.org/684035 | 08:47 |
openstackgerrit | Thierry Carrez proposed openstack/releases master: Auto release request for senlin https://review.opendev.org/684104 | 08:48 |
witek | ttx: updated notification release | 08:48 |
ttx | witek: great, thanks | 08:49 |
openstackgerrit | Merged openstack/releases master: Release request for tacker https://review.opendev.org/684108 | 08:50 |
witek | https://review.opendev.org/#/c/651249/ is good to merge, the monascalog-tempest is very fragile and fails randomly, we probably have too little memory for this job | 08:50 |
ttx | witek: ah ok, so it needs to be rechecked | 08:51 |
ttx | we can wait a bit | 08:51 |
*** tosky has quit IRC | 08:51 | |
*** pcaruana has quit IRC | 08:52 | |
openstackgerrit | Merged openstack/releases master: Auto release request for horizon https://review.opendev.org/684064 | 08:52 |
openstackgerrit | Merged openstack/releases master: Auto release request for nova https://review.opendev.org/684086 | 08:52 |
openstackgerrit | Merged openstack/releases master: Release Cinder Train RC-1 https://review.opendev.org/684050 | 08:53 |
openstackgerrit | Merged openstack/releases master: Auto release request for cyborg-tempest-plugin https://review.opendev.org/684004 | 08:57 |
openstackgerrit | Merged openstack/releases master: Auto release request for cyborg https://review.opendev.org/684056 | 08:57 |
openstackgerrit | Merged openstack/releases master: Auto release request for keystone https://review.opendev.org/684065 | 08:59 |
*** pcaruana has joined #openstack-release | 09:03 | |
*** ykarel|lunch is now known as ykarel | 09:12 | |
*** dtantsur|afk is now known as dtantsur | 09:18 | |
*** rpittau is now known as rpittau|bbl | 09:24 | |
*** ricolin has quit IRC | 09:24 | |
*** dtantsur is now known as dtantsur|lunch | 09:24 | |
*** e0ne has joined #openstack-release | 09:25 | |
openstackgerrit | Merged openstack/releases master: Release request for designate https://review.opendev.org/684058 | 09:27 |
openstackgerrit | Merged openstack/releases master: Auto release request for sahara-image-elements https://review.opendev.org/684094 | 09:27 |
openstackgerrit | Merged openstack/releases master: Auto release request for sahara-dashboard https://review.opendev.org/684092 | 09:27 |
openstackgerrit | Merged openstack/releases master: Auto release request for sahara-plugin-ambari https://review.opendev.org/684095 | 09:27 |
openstackgerrit | Merged openstack/releases master: Auto release request for sahara-plugin-mapr https://review.opendev.org/684097 | 09:27 |
openstackgerrit | Merged openstack/releases master: Auto release request for sahara-plugin-cdh https://review.opendev.org/684096 | 09:27 |
openstackgerrit | Merged openstack/releases master: Auto release request for sahara-plugin-storm https://review.opendev.org/684099 | 09:27 |
openstackgerrit | Merged openstack/releases master: Auto release request for sahara-plugin-spark https://review.opendev.org/684098 | 09:27 |
openstackgerrit | Merged openstack/releases master: Auto release request for sahara-plugin-vanilla https://review.opendev.org/684100 | 09:27 |
openstackgerrit | Merged openstack/releases master: Auto release request for monasca-notification https://review.opendev.org/684035 | 09:27 |
openstackgerrit | Mark Goddard proposed openstack/releases master: Ironic inspector: release 9.2.0 and branch stable/train https://review.opendev.org/684272 | 09:30 |
AJaeger | evrardjp: did your scripts skip doing karbor? | 09:31 |
*** e0ne has quit IRC | 09:45 | |
evrardjp | AJaeger: I am in a workshop, sorry for the limited availability. Karbor doesn't seem to appear in the list | 09:50 |
evrardjp | they had a release 1.4.0 | 09:50 |
evrardjp | 12 days ago | 09:51 |
AJaeger | But no train branch? | 09:51 |
AJaeger | evrardjp: figure it out later ;) I was looking for their train branch and couldn't find it, so that made me check and ask. | 09:52 |
evrardjp | we didn't require branching at that time | 09:52 |
evrardjp | I have to catch up on monday about what happened during last release meeting. | 09:53 |
evrardjp | if it's not in there, we should add this to the agenda of the next meeting or fix it before then. | 09:53 |
evrardjp | sorry I can't say more | 09:53 |
*** hberaud is now known as hberaud|lunch | 09:53 | |
openstackgerrit | Zhang Chi proposed openstack/releases master: release Tricircle for Trian https://review.opendev.org/685305 | 09:54 |
openstackgerrit | Mark Goddard proposed openstack/releases master: Release 1.0.0 for ironic-python-agent-builder https://review.opendev.org/685306 | 09:56 |
AJaeger | evrardjp: I think it has time until Monday ;) I suggest that you double check that all branches are created where needed... Enjoy your workshop! | 09:57 |
openstackgerrit | Luigi Toscano proposed openstack/releases master: Auto release request for sahara https://review.opendev.org/684101 | 10:18 |
brtknr | anyone here able to bless this release: https://review.opendev.org/#/c/683398/ | 10:26 |
*** cdent has joined #openstack-release | 10:31 | |
openstackgerrit | Bharat Kunwar proposed openstack/releases master: Release magnum 8.1.0 https://review.opendev.org/683398 | 10:36 |
ttx | evrardjp: any chance you could review https://review.opendev.org/#/c/683234/ ? Would make my subsequent change a lot easier :) | 10:37 |
openstackgerrit | Bharat Kunwar proposed openstack/releases master: Release magnum 8.1.0 https://review.opendev.org/683398 | 10:54 |
*** dtantsur|lunch is now known as dtantsur | 11:03 | |
ttx | evrardjp: ignore me, I'll just approve it. It's been proposed by our fearless leader after all. | 11:10 |
openstackgerrit | Merged openstack/releases master: Update lib release instructions for freeze https://review.opendev.org/683234 | 11:22 |
*** hberaud|lunch is now known as hberaud | 11:29 | |
*** cdent has quit IRC | 11:34 | |
*** jhesketh has quit IRC | 11:34 | |
*** jhesketh has joined #openstack-release | 11:35 | |
*** dhellmann_ has joined #openstack-release | 11:37 | |
*** dhellmann has quit IRC | 11:37 | |
*** dhellmann_ is now known as dhellmann | 11:37 | |
openstackgerrit | Thomas Bechtold proposed openstack/releases master: Release pymod2pkg 0.21.2 https://review.opendev.org/685326 | 11:44 |
toabctl | evrardjp, smcginnis hey. could you have a look at https://review.opendev.org/685326 please? I know it's friday but this is blocking rpm-packaging from updating to latest RC releases and it's only affecting rpm-packaging itself (no other consumers). have a nice weekend :) | 11:46 |
openstackgerrit | Thierry Carrez proposed openstack/releases master: Turn process into week-actionable items https://review.opendev.org/685327 | 11:50 |
ttx | toabctl: I'll review it as soon as the test results are posted | 11:50 |
*** weshay has joined #openstack-release | 11:55 | |
ttx | toabctl: should this be 0.22.0? (see my review) | 12:01 |
*** e0ne has joined #openstack-release | 12:04 | |
openstackgerrit | Thomas Bechtold proposed openstack/releases master: Release pymod2pkg 0.22.0 https://review.opendev.org/685326 | 12:04 |
*** cdent has joined #openstack-release | 12:04 | |
toabctl | ttx, true. fixed now. thanks! | 12:04 |
ttx | +2 | 12:04 |
*** tetsuro has quit IRC | 12:07 | |
openstackgerrit | Merged openstack/releases master: Auto release request for sahara https://review.opendev.org/684101 | 12:07 |
*** tetsuro has joined #openstack-release | 12:08 | |
*** bnemec has quit IRC | 12:10 | |
*** bnemec has joined #openstack-release | 12:11 | |
openstackgerrit | Merged openstack/releases master: Auto release request for sahara-extra https://review.opendev.org/684093 | 12:15 |
evrardjp | ttx sorry I am late to the game. Thanks for dealing with it. | 12:29 |
ttx | evrardjp: you can review https://review.opendev.org/685326 instead :) | 12:30 |
evrardjp | I am already | 12:31 |
*** iurygregory has joined #openstack-release | 12:31 | |
*** rpittau|bbl is now known as rpittau | 12:36 | |
openstackgerrit | Merged openstack/releases master: Release pymod2pkg 0.22.0 https://review.opendev.org/685326 | 12:48 |
*** udesale has quit IRC | 12:58 | |
*** udesale has joined #openstack-release | 12:58 | |
*** prometheanfire has quit IRC | 13:02 | |
*** prometheanfire has joined #openstack-release | 13:03 | |
*** pcaruana has quit IRC | 13:09 | |
*** mriedem has joined #openstack-release | 13:10 | |
*** amoralej is now known as amoralej|lunch | 13:10 | |
*** ricolin has joined #openstack-release | 13:30 | |
*** dave-mccowan has joined #openstack-release | 13:31 | |
*** cdent has quit IRC | 13:41 | |
smcginnis | Back at my desk. ttx, any fires I can help put out? | 13:50 |
smcginnis | I like the more perscriptive by-week updates to the process doc. Or at least the idea, I haven't read through the changes yet. | 13:51 |
ttx | smcginnis: nope everything under control | 13:54 |
*** udesale has quit IRC | 13:56 | |
openstackgerrit | Pierre Riteau proposed openstack/releases master: Auto release request for blazar https://review.opendev.org/684047 | 13:57 |
*** dave-mccowan has quit IRC | 13:58 | |
*** mlavalle has joined #openstack-release | 13:59 | |
*** pcaruana has joined #openstack-release | 14:04 | |
*** amoralej|lunch is now known as amoralej | 14:12 | |
*** witek has quit IRC | 14:15 | |
*** hberaud is now known as hberaud|school-r | 14:25 | |
*** lpetrut has quit IRC | 14:33 | |
*** pcaruana has quit IRC | 14:35 | |
*** hberaud|school-r is now known as hberaud | 14:37 | |
*** lpetrut has joined #openstack-release | 14:39 | |
*** armax has joined #openstack-release | 14:39 | |
*** witek has joined #openstack-release | 14:47 | |
*** radeks has quit IRC | 14:49 | |
openstackgerrit | Merged openstack/releases master: Ironic inspector: release 9.2.0 and branch stable/train https://review.opendev.org/684272 | 14:57 |
*** cdent has joined #openstack-release | 14:58 | |
smcginnis | #startmeeting releaseteam | 15:00 |
openstack | Meeting started Fri Sep 27 15:00:24 2019 UTC and is due to finish in 60 minutes. The chair is smcginnis. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
*** openstack changes topic to " (Meeting topic: releaseteam)" | 15:00 | |
openstack | The meeting name has been set to 'releaseteam' | 15:00 |
smcginnis | ping ttx dhellmann diablo_rojo hberaud evrardjp armstrong tonyb | 15:00 |
hberaud | o/ | 15:00 |
smcginnis | Welcome to our special edition release team meeting. | 15:00 |
smcginnis | #link https://etherpad.openstack.org/p/train-relmgt-tracking Agenda | 15:00 |
smcginnis | Around line 564 | 15:01 |
ttx | o/ | 15:01 |
evrardjp | I am in a workshop, so I can't attend this meeting, sorry. | 15:01 |
smcginnis | Actually, line 515. | 15:01 |
smcginnis | I'm getting ahead of myself. | 15:01 |
smcginnis | evrardjp: ack, thanks. | 15:01 |
smcginnis | #topic autoreleases without any feedback | 15:02 |
*** openstack changes topic to "autoreleases without any feedback (Meeting topic: releaseteam)" | 15:02 | |
smcginnis | I posted a reminder yesterday and then again today for these. | 15:02 |
ttx | So according to my newly-posted process we should approve those without feedback now | 15:02 |
ttx | I mean, not during meeting, but just after | 15:03 |
smcginnis | I agree, the ones without an explicit -1 should just go. | 15:03 |
smcginnis | We've tried. | 15:03 |
* ttx checks recent updates | 15:03 | |
smcginnis | I tried to get clarification, but I saw jokke mention glance would be "sometime early next week". | 15:04 |
ttx | yeah, so we should approve the ones were the PTL did not post anything, just after this meeting | 15:05 |
smcginnis | I would guess with the ones without any response, more than likely the tempest-plugin ones are good. | 15:05 |
ttx | FOr the ones with a standing -1. I propose we review them now | 15:05 |
smcginnis | dashboard probably too. | 15:05 |
smcginnis | +1 | 15:05 |
smcginnis | #topic Discuss standing -1s on autoreleases | 15:05 |
*** openstack changes topic to "Discuss standing -1s on autoreleases (Meeting topic: releaseteam)" | 15:05 | |
smcginnis | Barbican | 15:05 |
smcginnis | #link https://review.opendev.org/#/c/684045/ | 15:06 |
smcginnis | Says waiting for FFE, but I am not aware of one out there. | 15:06 |
ttx | Hmm yeah, let's ask for an update. and approve on Monday if there is none | 15:07 |
smcginnis | Just commented on that patch. | 15:07 |
smcginnis | Glance | 15:07 |
smcginnis | #link https://review.opendev.org/684060 | 15:07 |
smcginnis | I believe that is held up by needing a FFE for glance_store. | 15:08 |
smcginnis | #link https://review.opendev.org/685050 | 15:08 |
*** lpetrut has quit IRC | 15:08 | |
ttx | yeah, and a backport | 15:08 |
*** ykarel is now known as ykarel|afk | 15:08 | |
smcginnis | So that one is probably early next week based on needing to get the lib update out and then address whatever depends on that within glance itself. | 15:08 |
smcginnis | blazar-tempest-plugin | 15:09 |
smcginnis | #link https://review.opendev.org/684000 | 15:09 |
ttx | yeah, maybe try to talk to erno/Abishek today about it | 15:09 |
smcginnis | I've pinged Erno but no response yet. | 15:09 |
ttx | Pierre is reactive, so I' inclined to trust him | 15:10 |
smcginnis | Blazar tempest plugin comment is that they are still merging some fixes. | 15:10 |
ttx | Surprised though | 15:10 |
smcginnis | OK, we can just wait on that one then. | 15:10 |
ttx | Maybe ask for an update | 15:10 |
smcginnis | Last update was on the 24th. | 15:10 |
smcginnis | I'll comment on there asking for the latest. | 15:10 |
ttx | might have slipped between the cracks | 15:10 |
smcginnis | Yeah, that's my hope. | 15:10 |
smcginnis | Heat | 15:10 |
smcginnis | #link https://review.opendev.org/684062 | 15:11 |
smcginnis | ricolin commented it wasn't ready yet not long ago. | 15:11 |
ttx | woiuld be good to know what it is waiting on | 15:11 |
ttx | (in general) | 15:11 |
ttx | maybe ask for what it is waiting on | 15:11 |
smcginnis | There is also https://review.opendev.org/684008 waiting on ack for the tempest plugin. | 15:11 |
ttx | At this point we should know | 15:11 |
smcginnis | Agreed | 15:12 |
ttx | well that other one we'll approve now | 15:12 |
ttx | (post-meeting) | 15:12 |
smcginnis | monasca-api | 15:13 |
smcginnis | #link https://review.opendev.org/#/c/684031/ | 15:13 |
ttx | witek is waiting for a substantial patch to merge. | 15:13 |
ttx | it's stuck in gate. Let's wait until Monday on this oen | 15:13 |
smcginnis | OK. | 15:14 |
smcginnis | #link https://review.opendev.org/#/c/684003/ congress-tempest-plugin | 15:14 |
smcginnis | That one Eric updated with a new hash but then left -1. | 15:14 |
smcginnis | I wonder if that was meant to be a +1. | 15:14 |
ttx | Maybe aslet me see the patch queue there | 15:15 |
ttx | yeah, it's emnpty | 15:15 |
ttx | so probably a misfire | 15:15 |
smcginnis | Only patches out there don't look like they are close to merging. | 15:16 |
smcginnis | Well, I guess we can wait a bit and see if we get a response. | 15:16 |
ttx | we should override the -1 | 15:16 |
ttx | there is no patch up | 15:16 |
ttx | I +2, your call | 15:17 |
smcginnis | I commented on https://review.opendev.org/#/c/684003/2 if you want to approve. | 15:17 |
ttx | yes on it | 15:17 |
smcginnis | Oh, I see we both did. | 15:17 |
smcginnis | OK, mischief managed on that one I guess. | 15:17 |
smcginnis | #link https://review.opendev.org/#/c/684036/ monasca-persister | 15:18 |
ttx | On that one, knwoing what we are waiting on would be good | 15:18 |
smcginnis | I wonder if this is also held up by that large patch. | 15:18 |
ttx | could be. It's a pretty massive patch | 15:18 |
smcginnis | Wait a little longer for a response? | 15:18 |
openstackgerrit | Merged openstack/releases master: Auto release request for blazar https://review.opendev.org/684047 | 15:18 |
ttx | main reason why it does not make much sense to cut the stable branch before | 15:18 |
ttx | yes, but ask what is holding it up | 15:19 |
smcginnis | I had just commented their earlier, so we can wait for a response from witek. | 15:19 |
ttx | ok I'll ask | 15:19 |
witek | I think persister is ready, will update the hash | 15:20 |
ttx | hah, great! One down | 15:20 |
*** dtantsur is now known as dtantsur|afk | 15:20 | |
smcginnis | #link https://review.opendev.org/#/c/684072/ mistral | 15:21 |
smcginnis | Thank you witek | 15:21 |
ttx | On mistral, rakhmerov communicated this morning and we know what patches to track, so OK to wait a bit | 15:21 |
smcginnis | Looks like the two referenced are still in progress. | 15:22 |
ttx | yes | 15:22 |
smcginnis | #link https://review.opendev.org/#/c/685305/ Tricircle (take 2) | 15:22 |
smcginnis | We have the auto generated patch, and a new patch. | 15:23 |
ttx | the way I interpret that one is that it should wait until https://review.opendev.org/#/c/684159/ is approved | 15:23 |
ttx | but it's a bit unclear | 15:23 |
ttx | Let's regroup on Monday once China is awake | 15:23 |
smcginnis | No reviews for 3 days on the referenced change id. | 15:24 |
smcginnis | And it's a trivial patch that should not hold up anything. | 15:24 |
ttx | it is true | 15:24 |
smcginnis | Nothing else looks close to merging. | 15:24 |
ttx | Let me see if the autorelease still points to head | 15:24 |
ttx | yes it still does | 15:25 |
smcginnis | Yep | 15:25 |
smcginnis | I think there may just be some confusion there. | 15:25 |
ttx | Maybe comment on that other patch that we will approve https://review.opendev.org/#/c/684109 on Monday | 15:25 |
ttx | unless he comments there that we are waiting for another patch. | 15:26 |
smcginnis | Maybe we should just approve it today. Nothing inflight to wait for that I can see. | 15:26 |
ttx | I think he meant to include that constraint update patch | 15:26 |
smcginnis | Only think that looks sort of important is the py3 testing https://review.opendev.org/#/c/681481/ | 15:26 |
ttx | which is generally a good thing to have | 15:27 |
smcginnis | That's nothing that should hold up a release. | 15:27 |
smcginnis | It's ultimately sort of a no-op. | 15:27 |
ttx | sure, but I think that is what he tried to express | 15:27 |
ttx | So I'm willing to give him until Monday to get that in | 15:27 |
smcginnis | OK, we can wait. | 15:27 |
ttx | But we need to leave a clearer / more actionable comment | 15:27 |
smcginnis | The lack of any core reviews on those concerns me though. | 15:27 |
smcginnis | And they are all easy enough to backport later if desired. | 15:28 |
ttx | the fact that he missed the autorelease concerns me | 15:28 |
smcginnis | Was he added as a reviewer? | 15:28 |
ttx | yes | 15:28 |
smcginnis | Hmm | 15:28 |
openstackgerrit | Merged openstack/releases master: Auto release request for congress-tempest-plugin https://review.opendev.org/684003 | 15:28 |
openstackgerrit | Pierre Riteau proposed openstack/releases master: Auto release request for blazar-tempest-plugin https://review.opendev.org/684000 | 15:28 |
smcginnis | So multiple causes for multiple concerns there. | 15:28 |
ttx | OK, let me leave a message on that one | 15:29 |
smcginnis | Thanks | 15:29 |
ttx | with clear action in case he does not react | 15:29 |
smcginnis | ++ | 15:30 |
smcginnis | #topic Other train releases | 15:30 |
*** openstack changes topic to "Other train releases (Meeting topic: releaseteam)" | 15:30 | |
smcginnis | #link https://review.opendev.org/685050 glance_store | 15:30 |
smcginnis | FFE posted to the ML not long ago. | 15:30 |
smcginnis | I haven't checked if there was a response to that yet. | 15:30 |
smcginnis | I did ping prometheanfire to make sure he was aware of it though. | 15:31 |
smcginnis | My assumption is that is what is holding up the glance release. | 15:31 |
ttx | I did abandon the confused tricircle review so that we don;t have two | 15:31 |
smcginnis | #link https://review.opendev.org/#/c/682269/ | 15:32 |
smcginnis | OK, good. | 15:32 |
openstackgerrit | Graham Hayes proposed openstack/releases master: Auto release request for designate-dashboard https://review.opendev.org/684057 | 15:32 |
smcginnis | Puppet OpenStack Train Milestone 3. | 15:32 |
fungi | #link http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009752.html | 15:32 |
smcginnis | I was looking at that right before the meeting. | 15:32 |
*** mriedem has quit IRC | 15:32 | |
smcginnis | I think that's OK to go. | 15:32 |
smcginnis | fungi: Thanks | 15:32 |
ttx | smcginnis: I already +2d so please go ahead | 15:32 |
smcginnis | Done | 15:33 |
smcginnis | I know we keep having these semver discussions with the packaging projects, but not sure what to do about it. | 15:33 |
ttx | mugsie just in time to avoid autorelease | 15:33 |
smcginnis | It's just going to keep coming up every time. | 15:33 |
fungi | and yeah, no replies to jokke's glance_store ffe request yet | 15:33 |
ttx | I don;t care that much, wahetever works for them | 15:33 |
smcginnis | I don't know where we can document "packaging projects are weird and don't bother them about not properly following semver" so new reviewers don't always flag it. | 15:35 |
smcginnis | But maybe now evrardjp will know at least. :) | 15:35 |
ttx | in the reviewers tips. And yes, he is well placed to document that | 15:35 |
ttx | coming from that weirdo crowd and all | 15:35 |
smcginnis | Haha | 15:35 |
smcginnis | OK, I think that's all for pending releases for now then. | 15:36 |
smcginnis | Monday will be busy. | 15:36 |
ttx | I'll +2 all the oens that we should autorelease now | 15:36 |
smcginnis | ++ | 15:36 |
smcginnis | #topic R-2 email content final review | 15:36 |
*** openstack changes topic to "R-2 email content final review (Meeting topic: releaseteam)" | 15:36 | |
ttx | Not that much | 15:36 |
smcginnis | #link https://etherpad.openstack.org/p/relmgmt-weekly-emails | 15:36 |
*** pcaruana has joined #openstack-release | 15:36 | |
smcginnis | Line 499 | 15:36 |
smcginnis | Not sure if we should be promoting holding off on stable branch creation like we have in that draft. | 15:37 |
smcginnis | But I suppose if it makes things easier for them. | 15:37 |
ttx | smcginnis: I guess we could check for new commits on those we are about to force. But that sounds like a lot of work | 15:38 |
smcginnis | Yeah | 15:38 |
* fungi is curious what makes this meeting a special edition, but his flight is boarding so will find out later | 15:38 | |
smcginnis | fungi: It's Friday and an hour early since I was traveling all week. | 15:39 |
*** ykarel|afk is now known as ykarel | 15:39 | |
smcginnis | And RC's are just so darn special. | 15:39 |
smcginnis | OK, the draft looks good to me. I can send that after the meeting. | 15:40 |
*** mriedem has joined #openstack-release | 15:40 | |
smcginnis | #topic Open discussion | 15:40 |
*** openstack changes topic to "Open discussion (Meeting topic: releaseteam)" | 15:40 | |
smcginnis | #link https://review.opendev.org/#/c/685327 | 15:40 |
smcginnis | I like the clarity of that. | 15:40 |
smcginnis | Less room for errors in interpretation on when something should be done. | 15:41 |
smcginnis | ttx: Anything you wanted to comment on that? | 15:41 |
ttx | no, just that it is incremental work | 15:41 |
ttx | I'll clarify other weeks | 15:41 |
smcginnis | OK, thanks. | 15:41 |
ttx | Just a LOT of work to sync | 15:41 |
smcginnis | Yeah | 15:42 |
ttx | but it will avoid having to translate all the time | 15:42 |
ttx | Like I've done the work this time, would rather not leave it to interpretation for next time | 15:42 |
smcginnis | That will be important. We need things to be less work, even if we are doing more. | 15:42 |
ttx | I also included the temail template | 15:42 |
ttx | s | 15:42 |
ttx | that way it avoid duplicating info like "mention in the email" | 15:43 |
smcginnis | One other subject I wanted to bring up, at least to start thinking about. | 15:43 |
smcginnis | I'm going to email tonyb and see how things are going. | 15:44 |
smcginnis | I'm hoping he is just getting caught up on his day job and will be back. | 15:44 |
smcginnis | But either way, we may need to think about how we handle stable releases without really having regular "stable" people to review. | 15:44 |
smcginnis | Maybe mriedem can take over Tony's Monday stable review timeslot, but I think we actually just need to start treating them like normal releases that just require a little more scrutiny. | 15:45 |
smcginnis | So nothing we need to debate or decide here today, but I wanted to raise the topic for folks to start thinking about. | 15:46 |
openstackgerrit | Merged openstack/releases master: Puppet OpenStack Train Milestone 3 https://review.opendev.org/682269 | 15:46 |
mriedem | feel free to ping me for stable release reviews | 15:46 |
smcginnis | Since at least for the last couple months, stable releases have kind of been up in the air with not having a clear process to follow. | 15:46 |
ttx | ++ | 15:47 |
smcginnis | mriedem: Tony used to have a regular review task on Monday's so we could follow a process, but yeah, maybe we can just ping you for anything that we are not entirely sure on. | 15:47 |
ttx | smcginnis: you should be able to +2 all the ones with a +2.. except https://review.opendev.org/#/c/684031/ which should be considered a -1 | 15:47 |
dulek | smcginnis: Hey, does it matter if kuryr-kubernetes has a release, but it's not having "rc1" suffix on its name? | 15:47 |
ttx | depends on its release model, let me check | 15:47 |
smcginnis | dulek: No, it's cycle-with-intermediary. | 15:48 |
smcginnis | Those do not do release candidates. So you should be all set. | 15:48 |
smcginnis | dulek: Thanks for checking! | 15:48 |
ttx | you just do releases, and teh latest is included in the global "openstack" release | 15:48 |
*** rpittau is now known as rpittau|afk | 15:48 | |
dulek | Okay, thanks! | 15:48 |
ttx | smcginnis: once all the silent are approved, we are not looking at too many exceptions | 15:49 |
ttx | The largest would be glance, which will block the post-RC1 process (since it's in the devstack default set I assume) | 15:49 |
smcginnis | Oh right. | 15:50 |
ttx | oh we should look at next week tasks | 15:50 |
smcginnis | Hopefully that doesn't take too long then. | 15:50 |
ttx | forgot to add that to agenda | 15:50 |
smcginnis | #topic Review next week's tasks | 15:50 |
*** openstack changes topic to "Review next week's tasks (Meeting topic: releaseteam)" | 15:50 | |
ttx | so there is another round of autoreleases... to cut missing branches this time | 15:51 |
smcginnis | Proposing stable/train branches for deliverables that missed it. | 15:51 |
smcginnis | I can do that if no one else can, but would also be very happy to let someone else handle that. | 15:51 |
smcginnis | We have a nice script to do it at least. | 15:51 |
ttx | otherwise there is the whole post-RC1 process to coordinate with Ghanshyam | 15:51 |
ttx | once we have all the eggs in the basket | 15:52 |
smcginnis | Though last time I did them all in one patch and ended up with a job timeout, so maybe we should be more careful with how many we try to do at once. | 15:52 |
ttx | I probably won;t do much as I'm traveling and taking a day off on Londay too | 15:52 |
ttx | Monday | 15:52 |
smcginnis | Nordics should be fun. | 15:52 |
ttx | So I'' only take the email template | 15:52 |
smcginnis | Looks like it will be just me and evrardjp next week for the most part. | 15:53 |
ttx | yes | 15:53 |
smcginnis | OK, I will take care of the branching. | 15:53 |
smcginnis | The post-rc1 tasks for us are mainly reminding qa and requirements to do their things. | 15:53 |
smcginnis | I can also take the task to add release note links. That's another easy script. | 15:54 |
smcginnis | "Encourage liaisons" is just a countdown email reminder. | 15:54 |
ttx | Post-RC1 tasks you will need to coordinate with gmann | 15:54 |
ttx | ++ moves there | 15:55 |
smcginnis | OK, I think that's pretty much it. | 15:55 |
ttx | Added a "handle standing RC1 exceptions" task to the top | 15:56 |
ttx | ok | 15:56 |
ttx | I think we are good! | 15:56 |
smcginnis | OK, we can wrap it up then. Thanks! | 15:56 |
smcginnis | #endmeeting | 15:56 |
*** 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/" | 15:56 | |
openstack | Meeting ended Fri Sep 27 15:56:40 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:56 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/releaseteam/2019/releaseteam.2019-09-27-15.00.html | 15:56 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/releaseteam/2019/releaseteam.2019-09-27-15.00.txt | 15:56 |
openstack | Log: http://eavesdrop.openstack.org/meetings/releaseteam/2019/releaseteam.2019-09-27-15.00.log.html | 15:56 |
smcginnis | Safe travels next week. I hope the event goes well. | 15:56 |
ttx | thanks! | 15:57 |
ttx | smcginnis: I'll let you process the autoreleases now | 15:57 |
smcginnis | Next on my todo list. | 15:57 |
*** diablo_rojo has joined #openstack-release | 15:58 | |
*** diablo_rojo has quit IRC | 16:03 | |
*** diablo_rojo has joined #openstack-release | 16:03 | |
openstackgerrit | Merged openstack/releases master: Auto release request for blazar-tempest-plugin https://review.opendev.org/684000 | 16:03 |
*** e0ne has quit IRC | 16:04 | |
cdent | smcginnis: can I assume there's basically nothing to do with regard to placement and releases if no bugs come along, except for approve the final release? | 16:10 |
*** armstrong has joined #openstack-release | 16:15 | |
gmann | ttx: smcginnis ack. I am going to start those today. | 16:22 |
diablo_rojo | Sorry I missed the meeting today, I knew it had moved days, but I guess I missed that it was going to be an hour earlier. | 16:34 |
*** ykarel has quit IRC | 16:34 | |
smcginnis | cdent: Yep, you should be all set. | 16:38 |
cdent | rad, thanks | 16:39 |
cdent | diablo_rojo: time is an illusion | 16:39 |
openstackgerrit | Ghanshyam Mann proposed openstack/releases master: Create stable/train for devstack https://review.opendev.org/685400 | 16:39 |
smcginnis | diablo_rojo: We moved it an hour earlier to make it easier on the EU folks. No one wants to stick around on a Friday to talk about release management. | 16:40 |
diablo_rojo | WAT? I thought it was everyone's favorite subject! | 16:40 |
openstackgerrit | Witold Bedyk proposed openstack/releases master: Auto release request for monasca-persister https://review.opendev.org/684036 | 16:41 |
diablo_rojo | Yeah I figured it was in the email you sent out, but I didn't update my calendar I just went, oh I'll remember and obviously failed. | 16:41 |
*** zbr|ruck is now known as zbr | 16:41 | |
smcginnis | Haha, oh well. | 16:42 |
*** cdent has quit IRC | 16:42 | |
diablo_rojo | And now I know my brain can't be trusted with these things. | 16:43 |
* diablo_rojo goes to open logs and catch up | 16:43 | |
*** witek has quit IRC | 16:46 | |
*** witek has joined #openstack-release | 16:48 | |
*** ykarel has joined #openstack-release | 16:50 | |
openstackgerrit | Merged openstack/releases master: Auto release request for senlin https://review.opendev.org/684104 | 16:54 |
openstackgerrit | Merged openstack/releases master: Auto release request for designate-dashboard https://review.opendev.org/684057 | 16:54 |
openstackgerrit | Merged openstack/releases master: Auto release request for heat-dashboard https://review.opendev.org/684061 | 16:54 |
openstackgerrit | Merged openstack/releases master: Auto release request for magnum-ui https://review.opendev.org/684029 | 16:54 |
openstackgerrit | Merged openstack/releases master: Auto release request for freezer https://review.opendev.org/684028 | 16:54 |
openstackgerrit | Merged openstack/releases master: Auto release request for magnum https://review.opendev.org/684067 | 16:54 |
ricolin | smcginnis, ttx FYI will +1 on heat release once https://review.opendev.org/#/c/683309/ merged | 16:57 |
smcginnis | ricolin: Thanks. | 17:00 |
openstackgerrit | Merged openstack/releases master: Auto release request for freezer-tempest-plugin https://review.opendev.org/684007 | 17:00 |
smcginnis | ricolin: Just a reminder - you will need to update the commit hash on that patch once that lands too. | 17:01 |
openstackgerrit | Ghanshyam Mann proposed openstack/releases master: Release Tempest 22.0.0 for Train https://review.opendev.org/685406 | 17:02 |
ricolin | smcginnis, got it | 17:03 |
*** weshay is now known as weshay|ruck | 17:08 | |
*** amoralej is now known as amoralej|off | 17:10 | |
openstackgerrit | Ghanshyam Mann proposed openstack/releases master: Release Patrole 0.7.0 for Train https://review.opendev.org/685430 | 17:13 |
*** witek has quit IRC | 17:16 | |
*** radeks has joined #openstack-release | 17:20 | |
*** ekcs has joined #openstack-release | 17:23 | |
*** pcaruana has quit IRC | 17:23 | |
openstackgerrit | Merged openstack/releases master: Auto release request for freezer-web-ui https://review.opendev.org/684027 | 17:27 |
openstackgerrit | Merged openstack/releases master: Auto release request for masakari-monitors https://review.opendev.org/684070 | 17:28 |
openstackgerrit | Merged openstack/releases master: Auto release request for masakari-dashboard https://review.opendev.org/684069 | 17:33 |
openstackgerrit | Merged openstack/releases master: Auto release request for heat-tempest-plugin https://review.opendev.org/684008 | 17:33 |
openstackgerrit | Merged openstack/releases master: Auto release request for mistral-tempest-plugin https://review.opendev.org/684014 | 17:33 |
openstackgerrit | Merged openstack/releases master: Auto release request for masakari https://review.opendev.org/684071 | 17:33 |
*** ekcs has quit IRC | 17:39 | |
*** diablo_rojo has quit IRC | 17:46 | |
*** diablo_rojo has joined #openstack-release | 17:46 | |
*** hberaud is now known as hberaud|gone | 17:50 | |
*** e0ne has joined #openstack-release | 17:50 | |
*** ekcs has joined #openstack-release | 17:54 | |
openstackgerrit | Witold Bedyk proposed openstack/releases master: Auto release request for monasca-api https://review.opendev.org/684031 | 18:00 |
openstackgerrit | Merged openstack/releases master: Auto release request for monasca-persister https://review.opendev.org/684036 | 18:09 |
*** mriedem has quit IRC | 18:12 | |
*** mriedem has joined #openstack-release | 18:14 | |
openstackgerrit | Ghanshyam Mann proposed openstack/releases master: Create stable/train for devstack https://review.opendev.org/685400 | 18:16 |
cmurphy | I don't understand the commit message in https://review.opendev.org/685264 "Use pbr instruction to increment the minor version number automatically" that change didn't touch anything related to pbr? | 18:40 |
*** KeithMnemonic has joined #openstack-release | 18:41 | |
AJaeger | cmurphy: check the commit message, it contains "Sem-Ver: feature" ;) | 18:42 |
smcginnis | cmurphy: It's the "Sem-Ver: feature" line. | 18:42 |
smcginnis | It's a special string the PBR looks for, and if found, it takes that into consideration to calculate the new version. | 18:42 |
cmurphy | oh i see | 18:43 |
smcginnis | So basically just makes sure "builds" before the next release are guaranteed to be versioned higher than anything on stable/train. | 18:43 |
AJaeger | smcginnis, cmurphy : is that documented in pbr docs? If not, want to? | 18:44 |
smcginnis | AJaeger: I think I found out about it from Doug, Jeremy, or Clark. I don't think I ever went looking for actual documentation after that. | 18:45 |
smcginnis | It may not be very clearly documented since it does introduce some risk in using it. | 18:45 |
smcginnis | Once there is a commit in the tree that has that string, not really a good way to undo what it's done. | 18:45 |
AJaeger | oh, found it - https://docs.openstack.org/pbr/latest/user/features.html#version | 18:46 |
smcginnis | Clear enough I guess, once you find it. | 18:48 |
AJaeger | yeah, once you find it ;) | 18:48 |
AJaeger | perhaps the commit message should link to it? | 18:49 |
smcginnis | Eh, I'd rather not advertise it too much given the risks. | 18:49 |
AJaeger | I see - ok, you'll have to continue to answer questions then ;) | 18:50 |
smcginnis | This has been the first time anyone has asked. :) | 18:51 |
AJaeger | oh | 18:53 |
*** armstrong has quit IRC | 18:55 | |
*** ricolin has quit IRC | 19:03 | |
openstackgerrit | Renat Akhmerov proposed openstack/releases master: Auto release request for mistral https://review.opendev.org/684072 | 19:05 |
*** radeks has quit IRC | 19:12 | |
prometheanfire | smcginnis: where be that ffe? | 19:15 |
smcginnis | prometheanfire: Here be - http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009752.html | 19:16 |
smcginnis | prometheanfire: Oh, here's the review if that's what you were looking for - https://review.opendev.org/685050 | 19:19 |
smcginnis | But that currently has some issues. | 19:19 |
prometheanfire | ah, missed the reqs tag :P | 19:21 |
smcginnis | Oh right, didn't notice that. | 19:21 |
*** KeithMnemonic has quit IRC | 19:23 | |
prometheanfire | next I need to see if all the projects have branched (not including cycle-trailing) | 19:24 |
*** ykarel is now known as ykarel|away | 19:29 | |
openstackgerrit | Merged openstack/releases master: Auto release request for barbican https://review.opendev.org/684045 | 20:05 |
openstackgerrit | Merged openstack/releases master: Auto release request for mistral https://review.opendev.org/684072 | 20:05 |
openstackgerrit | Merged openstack/releases master: Auto release request for monasca-api https://review.opendev.org/684031 | 20:07 |
*** ykarel|away has quit IRC | 20:16 | |
openstackgerrit | Sean McGinnis proposed openstack/releases master: Proposed release schedule for Ussuri https://review.opendev.org/679822 | 20:23 |
*** ekcs has quit IRC | 20:24 | |
*** e0ne has quit IRC | 20:45 | |
*** jtomasek has quit IRC | 20:55 | |
*** ekcs has joined #openstack-release | 21:20 | |
*** ekcs has quit IRC | 21:24 | |
*** ekcs has joined #openstack-release | 21:26 | |
*** mriedem has quit IRC | 21:37 | |
*** ekcs has quit IRC | 21:39 | |
*** mtreinish has quit IRC | 22:07 | |
*** mtreinish has joined #openstack-release | 22:10 | |
*** ekcs has joined #openstack-release | 22:17 | |
brtknr | Any chance someone could +W this: https://review.opendev.org/#/c/683398/ | 22:57 |
*** armax has quit IRC | 23:09 | |
*** diablo_rojo has quit IRC | 23:44 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!