15:00:03 #startmeeting releaseteam 15:00:04 Meeting started Fri Jun 1 15:00:03 2018 UTC and is due to finish in 60 minutes. The chair is smcginnis. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:00:05 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:00:07 The meeting name has been set to 'releaseteam' 15:00:10 Ping list: dhellmann, dims, fungi, tonyb, lbragstad, ttx, armstrong, annabelleB 15:00:16 o/ 15:00:17 Hi! 15:00:23 #link https://etherpad.openstack.org/p/rocky-relmgt-tracking Agenda 15:00:23 howdy 15:00:24 o/ 15:01:12 looks like we're around line 216 today? 15:01:31 Yep, looks right. 15:01:37 #topic Rocky-2 tasks 15:01:44 #link https://github.com/openstack/releases/blob/master/PROCESS.rst 15:02:10 So other than tagging the milestone releases, I guess I should have included a mention of the unrelease changes. 15:02:25 I actually did run the report, but didn't include that. 15:02:28 I'll be traveling Tue-Thu, but shoudl be around to take my share on teh Wednesday 15:02:33 I'll get that next tune, 15:02:36 *time 15:02:47 ttx: Great, thanks. 15:03:01 Any other issues/concerns for next week's milestone activities? 15:03:27 OK, I don't think I have anything going on, so should be good on my end. 15:03:36 #topic Tagging right fixes progress 15:03:37 no concerns here 15:03:52 no planned infrastructure disruption either 15:04:02 fungi: OK, great! 15:04:10 ttx: Was this your topic? 15:05:42 yes 15:06:01 so... ec2api was done already 15:06:23 For dragonflow, I looked it up and they did not do a queens release 15:06:42 that's unfortunate 15:06:44 Surprised we missed that until now. 15:06:48 which makes me question what to do there 15:06:51 i thnik they may be one of those projects which trail substantially 15:06:54 er, think 15:07:18 as in they develop against the latest neutron release rather than against master? 15:07:27 given where it sits on the map (neutron plugin) I would be inclined to consider it for unofficalnessity 15:07:41 At least I don't see them listed as part of the coordinated release: https://releases.openstack.org/queens/index.html 15:07:49 probably one to prioritize in the "health" TC activity 15:08:09 last release was 9 months ago 15:08:10 I'll make a note of that now 15:08:32 They have a bunch of commits though, so it's not "dead" 15:08:35 Ah, I seem to recall some internal mention of this. 15:09:11 2017-09-01 for their 4.0 release 15:09:12 they have 393 commits since that last release 15:09:25 that seems pretty active, are they just not tagging? 15:09:35 It's hard to consider it a part of openstack if it's not released basically 15:09:40 yeah 15:09:56 so they basically released 4.0 immediately after the coordinated pile release 15:10:05 I'm not saying it should die... I just question the value of it being seen as a part of the product rather than a product add-on 15:10:07 er, pike release 15:10:16 So do we propose to move them out of governance? 15:10:29 smcginnis: I'd wait for someone from teh TC to dive into it 15:10:35 fungi : quite the freudian slip that was :-) 15:10:39 hah 15:10:42 :D 15:10:43 We just need to prioritize it up 15:10:46 and get some feedback from the dragonflow devs too, of course 15:11:09 I've added a note to the "actively monitoring teams" section of the tc tracker in the wiki 15:11:09 next up: 15:11:16 looks like oanson is probably the most active contributor there (and has been pushing their tags) 15:11:16 I can start a ML thread after the meeting raising the question. 15:11:24 I intended to raise that in office hours next week when most everyone is back online 15:11:30 dhellmann: ++ 15:11:33 Rally -- I overlooked their response to the last ML thread about it 15:11:41 http://lists.openstack.org/pipermail/openstack-dev/2018-April/129405.html 15:11:41 smcginnis : a ML thread from the release PTL would be a good start, too 15:11:57 basically arguing that non-cycle-based things should retain tagging rights 15:11:58 I don't find either of those arguments particularly convincing 15:12:04 #action smcginnis to start ML thread asking for Dragonflow status update 15:12:21 but yeah, if they don't want to be listed on the releases.o.o site I guess I don't want to argue over it 15:12:37 We do state now in the documentation that coming under governance means the release team must be the ones handling tagging. 15:12:37 dhellmann: yeah, so my initial "ACL compliance" target was the openstack / openstack-operations buckets from the map[tm] 15:12:58 so rally wants to keep their release notes in the tag message rather than in their repository content? 15:13:15 they want to keep not depending on us, I think 15:13:43 that seems like the real reason, yes 15:14:11 we have some old data about releases. we could just remove that so the site doesn't appear to be out of date. 15:14:18 I'm fine with the "I should be able to do things how I damn well please" mentality, but I also think things can live outside of the "openstack" official space 15:14:25 right, the message indicates a dislike for the rules about when releases are appropriate, and a concern that there might not be release team members around to approve (urgent?) release requests 15:14:30 ttx: Agree 15:14:39 wanting both is what I disagree with 15:14:52 ++ 15:14:53 Sounds like this one should be "OpenStack hosted" and not officially governed if that's what they want. 15:15:03 we hold "openstack deliverables" to a given standard, and that standard now includes managed released 15:15:32 if only so that you can not make non-peer-reviewed release blunders that would stain the openstyck name 15:15:52 infra being the main outlier, and as noted in the most recent ml thread we want to move most of the infra repos out of openstack officially too 15:16:16 so i think it's a fine precedent 15:16:38 so, does someone need to reply to that ML thread proposing the 2 options? either letting us manage the releases or removing rally from governance? 15:16:59 Sounds like something the TC chair should address. :) 15:17:04 I'd say that's a TC-level discussion 15:17:08 agreed 15:17:16 enforcing that rule that is now in governance or not 15:17:17 ok, maybe our release team PTL will raise that then? :-) 15:17:36 Hehe, OK. 15:18:11 I think the key is what are the minimal stquality andards we want to enforce for things to be associated with the "openstack" 15:18:25 damn it weechat+mosh stop eating my characters 15:18:49 smcginnis : if you lay out the reasons why we have the rule, I will offer the option of rally leaving governance 15:19:25 I think those quality standards now include peer-reviewed releases basically 15:20:05 and that is what that resolution said 15:20:18 dhellmann: Are you thinking as a reply on that thread? Or should I bring this up in the next office hours? 15:21:06 reply to thread would be good 15:21:10 smcginnis : let's start by reviving the thread. I'll make sure the tc-members see it. 15:21:16 i think if it's brought up on the thread as a discussion, that works 15:21:26 maybe add [tc] to the subject if it isn't already there 15:21:27 Andrey felt like his answer went into deaf ears 15:21:27 OK, sounds like a plan. 15:21:55 I for one missed it 15:22:05 ok, next topic! 15:22:10 I saw it, but didn't plan on responding right away. 15:22:10 yeah, i honestly don't remember seeing his reply 15:22:23 i probably did read it and immediately forgot 15:23:04 yeah, I'm not sure why I glossed over that one 15:23:17 #action smcginnis to respond to http://lists.openstack.org/pipermail/openstack-dev/2018-April/129405.html with reasons why governed projects need to use the release team 15:26:02 ttx: Want to discuss the aclfixer patch? 15:26:15 so what's the deal with the puppet-pacemaker addition then? just needs to have the acl updated to conform or is that an indefinite blacklist? 15:26:15 should be straightforward 15:26:27 just needs to be fixed, I think 15:26:30 yeah, I wanted to ask about that, too 15:26:45 should we just fix their acls? 15:26:47 I could post the fix and update that aclfixer patch 15:27:10 just wanted to have time to dive deeper into it see where that came from 15:27:18 to prevent it from happening again 15:27:33 ok 15:27:42 I'll post the patch and update the aclfixer list 15:27:51 so don't approve just now 15:28:03 OK, sounds good. 15:28:23 #topic Stable library maintenance 15:28:26 OK, so I realized this morning that I signed up for that one, and wanted to discuss it for a bit 15:28:50 what would be your definition of a stable library ? Wouldn't we expect req bumps on those ? 15:29:05 since we no longer sync them, we might not have any 15:29:09 think oslo.i18n 15:29:33 that's basically never going to see new features, and will only get bug fixes if there are any, but it's a thin wrapper around gettext, so there aren't likely to be many 15:29:39 I think we would need them. Aren't the older ones still just using g-r and u-c? 15:30:04 we don't change g-r on older branches 15:30:10 they do use u-c, but those aren't synced 15:30:26 ok 15:30:33 so in this case... 15:31:00 the less-disruptive way is probably to allow multiple stable branches to be created from a given master release 15:31:14 we'll have a lot of somewhat-but-less stable libs, but oslo.i18n seems likely to stay as it is for ages 15:31:31 We would just branch it from the last master release if nothing was pushed there since last stable 15:31:59 In case new things land in master, we just need to jump n stable branches in that versioning 15:32:05 x.y+n.0 15:32:19 Any obvious issue with that strawman ? 15:32:33 I feel like not creating branches until they are needed is actually more painful 15:32:37 it sounds like extra backporting work 15:32:48 backporting? 15:33:03 if we have a queens version that ends up being used for 3 cycles, then we have a fix, would we backport it to all of those other empty branches? 15:33:13 why not just backport from master to queens and do 2 releases? 15:33:20 seems like it might be challenging to predict version numbers for those branches? 15:34:09 hmm let me think... we rely on constraints rather than branch names in tests, right 15:34:35 in unit tests, yes 15:34:36 the branch names are used when testing changes to the libraries in the integration tests 15:34:44 ideally we'd be able to use version numbers instead of series names, but that's a lot of tooling updates 15:34:53 stable/2.0 instead of stable/queens 15:34:56 I fear that tracking which branch is used for where might be a bit challenging 15:35:03 What we really need is branch aliases :) 15:35:05 the telemetry team tried that and it turned into a lot of work 15:35:13 say we branch stable/rocky and then later branch stable/stein from the same commit... later we discover a bug (security vulnerability fix?) which needs to be committed in master and backported... what do we tag for the stable branch release versions? 15:35:51 so.. master has 1.18.0 15:35:57 if the version before stable/rocky was 1.0.0 then the version on stable/rocky is 1.1.0 and the version on stable/stein is 1.2.0 15:35:59 you create stable/rocky from that 15:36:04 the release tools already require that 15:36:17 then create stable/stein from that 15:36:27 I have to admit, I'm not following all this. Why do we need to do something different than our current stable library releases? 15:36:32 then you fix the thing in master and release 1.20.0 15:36:38 so as long as we pick the stable/rocky version number before we pick the stable/steni version number, it could work out 15:36:41 then 1.18.1 and 1.19.0 15:36:44 fungi : yes 15:37:08 but we need to make sure we don't tag something in stable/stein before stable/rocky 15:37:14 what dhellmann says is that it's preferable to do 1.19.0 on master and 1.18.1 for rocky AND stein 15:37:19 or we could paint ourselves into a corner with version numbering 15:37:28 smcginnis : I guess the question is, do we need to create empty branches when there were no changes for several series? 15:37:35 fungi: that's why I suggest 1.19.0 15:37:51 ttx: right, we would just use the rocky version in the stein series, too 15:38:02 dhellmann: Just speaking from experience with os-brick, we always create a stable branch. 15:38:04 I don't know if that's going to be confusing for downstream folks, but that's how we treat 3rd party tools 15:38:24 Of course, it always does have changes, so I guess that's the key here. 15:38:32 ttx: right, my example could be extended to include stable/t and we need to make sure that we pick version numbers across r, s and t which are in order then 15:38:39 right 15:38:45 But I think we should still create branches each release for any "active" libs. 15:38:50 I'd say the 3 options are 15:38:55 so can't just rely on incrementing y in x.y.z 15:39:05 fungi : yeah, we'll have to be careful with the reviews. or we could make the tool smart enough to detect those errors 15:39:12 1/ force "artificial" releases even if there are no changes and just reuse same process 15:39:30 if we create a branch, there will always be at least 1 patch containing the .gitreview update 15:39:35 2/ do not force releases, but still create branches from latest releases before they are even needed 15:39:51 3/ do not force releases, and reuse stable branches from cycle to cycle 15:40:34 3 is conceptually the best, but I have a hard time to wrap my head around potential consequences 15:40:35 1 seems the safest to me. 15:40:48 1 is the safest but weirdest when there are no more changes 15:40:56 2 is a trade-off 15:41:29 2 is the case where we have to be careful with versions, as fungi pointed out 15:41:29 do we agree on the 3 options ? 15:41:37 yeah, #1 while safe deos also seem like a treadmill of process-imposed busywork 15:41:46 I suppose option 4 is to not worry about stable branches at all for those things 15:41:47 s/deos/does/ 15:42:01 switch them to independent ? 15:42:10 and option 5 is to not do anything and create the stable branches when we need them 15:42:39 (option 5 is like 2, except for the timing) 15:42:46 dhellmann: you mean create all the missing branches once you need them ? 15:42:50 right 15:43:04 I'm not sure it's a good idea, but for completeness sake I thought I'd mention it 15:43:04 It's a 2bis 15:43:07 i want to say there were some drawbacks to #4 and #5 (we did them in the past at various times) but trying to recall what the problems were now 15:43:37 is #4 like switching things to be independent ? 15:43:45 * ttx documents in etherpad 15:43:50 certainly not creating stable branches leads to problems when you have a new release you don't want used by stable branches of other projects 15:43:56 well, we have stable branches for independent projects sometimes 15:44:19 fungi : with the constraints system in place, new releases can be controlled 15:44:24 though i suppose constraints on those stable branches help 15:44:29 yeah, that 15:44:54 Yeah, we're probably in better shape now to handle these. 15:44:54 we didn't really have constraints widely used back when we were encountering the issues which caused us to force stable branches for all libs 15:44:54 in fact 3 would require extra manual work to update u-c in stable branches 15:45:06 right 15:45:24 so... 4/5 actually seem sort of okay and would certainly be less work 15:45:46 Not to be lazy, but less work is probably better at this point. 15:45:52 option 5 feels appealing, because it lets us do the work we need to fix something at the time it's broken without having to do extra work up front 15:46:00 i think it would end up being #5 because there will be critical fixes you need backported to an older version used by stable-branch projects 15:46:12 otoh, doing it up front means it's done as part of a batch, so we're ready when things are broken 15:46:42 so I'm sort of torn between 5 and 2 15:46:43 yeah, it's self-imposed work, but part of a routine 15:46:46 well, release requests can come with branching requests, right? so it's not really that much extra convenience to have the branches precreated 15:46:53 I renamed 5 to 2bis in the etherpad 15:47:02 fungi : they need the branch before they can backport a fix 15:47:14 ok 15:47:17 since it's close to 2 and helps estimating disruption potential 15:47:21 oh, good point. so it's two release requests 15:47:35 create the branch, backport the fix, tag the release 15:48:05 they're likely going to want a release from master, and the branch request could come in that patch, but at that point it's not really much extra effort to make it separately 15:48:21 maybe we try 2bis and if we run into problems we switch to 2? 15:48:21 i still feel like #5 is preferable to #2 _if_ the frequency of backports is very low 15:48:26 We don't have to decide now, just wanted to lay down the options to give it some extra thought 15:48:39 Was just going to say, we don't need to decide today. 15:48:41 I'm expecting a very small number of things in this category, and for those things to be very stable. 15:48:54 Let's give it a bit and let things sink in and see if we think of any other concerns. 15:48:58 One issue is how we "mark" those 15:49:05 ++ to contemplation 15:49:16 ttx: yeah, maybe a new release model? 15:49:31 it's also possible to switch from #2 to #2bis in the future if we discover that #2 isn't buying us much for the additional effort 15:49:38 fungi : good point 15:49:53 it seems like we can switch between any of the options, really 15:50:06 sure 15:50:08 Yeah, 2bis is really just lazy loaded 2. 15:50:17 ok... let's it simmer a bit. Next topic ? 15:50:19 ++ for being lazy 15:50:36 Any aspect of extended maintenance lib releases we should consider with this? 15:50:46 I thought that was actually the topic at first glance. 15:51:01 Probably the same applies? 15:51:20 brain cells level: 0 15:51:22 I think we decided to change the devstack job to install everything from source by default for EM branches, didn't we? 15:51:47 Was that the decision in the session. I couldn't remember if we came to a decision. 15:52:01 that was the consensus there yes 15:52:07 I felt like it was the preferred option, although I'm not sure if someone stepped up to write that change. 15:52:20 it may have been another case of dealing with it when it actually comes up 15:52:34 OK, that works then. We can move on. 15:52:45 #topic Planned time off 15:53:11 I realized my summer vacation is on FF week, again 15:53:20 I know I will be gone R-7, but that should be a quiet week. 15:53:21 also in Europython week 15:53:30 I shouldn't have any conflicts on FF week (R-5). 15:53:48 I'll be traipsing all over scotland that week 15:53:51 I refreshed my travel schedule for the rest of the cycle 15:53:56 dhellmann: Nice! 15:54:00 EuroPython \o/ 15:54:16 \o/ 15:54:18 well,not so much "all over" as "Edinburgh and Glasgow" 15:54:49 So I guess I'll be holding down the fort that week with whatever help I can get. 15:55:08 And will be definitely leaning on fungi if any release/infra issues come up. :) 15:55:48 i'm gone for the second half of july... trying to see what that works out to on the release schedule 15:55:50 This cycle has been much more reliable with those things, so hopefully this is no issue. 15:55:53 * smcginnis knocks on wood 15:56:11 it's a good test at least 15:56:22 OK, then I can just cry for help in -infra if something happens. :D 15:56:32 But not feeling too worried. 15:56:36 yeah, i disappear in the middle of r5 and return in the middle of r3 15:56:40 on conference days (the end of the week) I'll check in on irc periodically 15:56:49 er, i mean r6 and r4 15:57:12 dhellmann: That would be great. Don't want to distract you, but might be good if any questions come up. 15:57:33 I'm on twitter and signal, too 15:57:51 so yeah, my travel is july 18-31 according to my notes, though i'll have internet access during some of that 15:57:53 I think I can hunt most of you down if I really need to. 15:58:13 * dhellmann turns on his cloaking device 15:58:19 Hah ;) 15:58:29 Anything else in the last minute+? 15:59:11 nothing from me 15:59:13 We need to talk succession at some point, but we have time yet. But time today. 15:59:23 But not time today I mean. 15:59:34 OK, thanks everyone! 15:59:45 #endmeeting