| *** yamamoto has quit IRC | 00:01 | |
| *** yamamoto has joined #openstack-release | 00:05 | |
| *** ricolin has joined #openstack-release | 00:14 | |
| *** mriedem has quit IRC | 00:30 | |
| *** armax has quit IRC | 00:34 | |
| *** mriedem has joined #openstack-release | 00:38 | |
| *** harlowja has quit IRC | 00:38 | |
| *** zhurong has joined #openstack-release | 00:43 | |
| *** thorst_afk has joined #openstack-release | 00:46 | |
| *** rajathagasthya has quit IRC | 00:53 | |
| *** thorst_afk has quit IRC | 00:54 | |
| *** liujiong has joined #openstack-release | 01:08 | |
| *** sdague has quit IRC | 01:08 | |
| *** yamamoto has quit IRC | 01:25 | |
| *** thorst_afk has joined #openstack-release | 01:29 | |
| *** yamamoto has joined #openstack-release | 01:29 | |
| *** gcb has joined #openstack-release | 01:30 | |
| *** ekcs has quit IRC | 01:33 | |
| *** ricolin has quit IRC | 01:34 | |
| *** gcb has quit IRC | 01:37 | |
| *** thorst_afk has quit IRC | 01:38 | |
| *** thorst_afk has joined #openstack-release | 01:38 | |
| *** gcb has joined #openstack-release | 01:39 | |
| *** thorst_afk has quit IRC | 01:43 | |
| *** https_GK1wmSU has joined #openstack-release | 01:50 | |
| *** https_GK1wmSU has left #openstack-release | 01:50 | |
| *** yamamoto has quit IRC | 01:51 | |
| *** armax has joined #openstack-release | 01:52 | |
| *** thorst_afk has joined #openstack-release | 01:55 | |
| *** thorst_afk has quit IRC | 01:55 | |
| *** yamamoto has joined #openstack-release | 01:56 | |
| *** yamamoto has quit IRC | 02:07 | |
| *** yamahata has quit IRC | 02:08 | |
| *** mriedem has quit IRC | 02:10 | |
| *** thorst_afk has joined #openstack-release | 02:11 | |
| *** rajathagasthya has joined #openstack-release | 02:17 | |
| *** yamamoto has joined #openstack-release | 02:19 | |
| *** hongbin has joined #openstack-release | 02:21 | |
| *** harlowja has joined #openstack-release | 02:22 | |
| *** thorst_afk has quit IRC | 02:25 | |
| *** bobh has joined #openstack-release | 02:26 | |
| *** rajathagasthya has quit IRC | 02:34 | |
| *** ricolin has joined #openstack-release | 02:42 | |
| *** harlowja has quit IRC | 02:44 | |
| *** liujiong has quit IRC | 02:50 | |
| *** liujiong has joined #openstack-release | 02:51 | |
| *** hongbin_ has joined #openstack-release | 02:55 | |
| *** prometheanfire has joined #openstack-release | 02:55 | |
| prometheanfire | don't know if tony asked already, but can we get an [n]ack for a FFE on https://review.openstack.org/489376 | 02:56 |
|---|---|---|
| *** hongbin has quit IRC | 02:57 | |
| *** hongbin has joined #openstack-release | 02:59 | |
| *** hongbin_ has quit IRC | 03:00 | |
| *** yamamoto has quit IRC | 03:01 | |
| *** yamamoto has joined #openstack-release | 03:06 | |
| *** lbragstad_ has joined #openstack-release | 03:14 | |
| *** lbragstad has quit IRC | 03:14 | |
| *** yamamoto has quit IRC | 03:21 | |
| *** bobh has quit IRC | 03:23 | |
| *** yamamoto has joined #openstack-release | 03:24 | |
| *** yamamoto has quit IRC | 03:36 | |
| *** harlowja has joined #openstack-release | 03:38 | |
| *** udesale has joined #openstack-release | 03:43 | |
| *** yamamoto has joined #openstack-release | 03:44 | |
| *** dave-mcc_ has quit IRC | 03:48 | |
| *** yamamoto has quit IRC | 04:04 | |
| *** hongbin has quit IRC | 04:05 | |
| *** yamamoto has joined #openstack-release | 04:09 | |
| *** ekcs has joined #openstack-release | 04:16 | |
| *** ekcs has quit IRC | 04:23 | |
| *** thorst_afk has joined #openstack-release | 04:26 | |
| *** thorst_afk has quit IRC | 04:31 | |
| *** udesale__ has joined #openstack-release | 04:32 | |
| *** udesale has quit IRC | 04:34 | |
| *** harlowja has quit IRC | 05:10 | |
| *** jtomasek has joined #openstack-release | 05:20 | |
| *** https___GK1wmSU has joined #openstack-release | 05:25 | |
| *** https___GK1wmSU has left #openstack-release | 05:26 | |
| *** gcb has quit IRC | 05:29 | |
| *** gcb_ has joined #openstack-release | 05:29 | |
| *** harlowja has joined #openstack-release | 05:51 | |
| *** yamahata has joined #openstack-release | 06:01 | |
| *** wolverineav has joined #openstack-release | 06:18 | |
| *** wolverineav has quit IRC | 06:22 | |
| *** thorst_afk has joined #openstack-release | 06:28 | |
| *** thorst_afk has quit IRC | 06:32 | |
| ttx | prometheanfire: lgtm | 06:48 |
| *** harlowja has quit IRC | 06:51 | |
| prometheanfire | thanks | 06:52 |
| *** alexchadin has joined #openstack-release | 06:56 | |
| *** dtantsur|afk is now known as dtantsur | 07:01 | |
| *** pcaruana has joined #openstack-release | 07:06 | |
| *** dtantsur is now known as dtantsur|bbl | 07:47 | |
| *** clarkb has quit IRC | 08:04 | |
| *** clarkb has joined #openstack-release | 08:17 | |
| *** lucas-afk is now known as lucasagomes | 08:26 | |
| *** thorst_afk has joined #openstack-release | 08:27 | |
| *** thorst_afk has quit IRC | 08:32 | |
| *** pbandark has joined #openstack-release | 08:40 | |
| *** electrofelix has joined #openstack-release | 08:44 | |
| *** ricolin has quit IRC | 08:51 | |
| *** alexchadin has quit IRC | 08:53 | |
| *** alexchadin has joined #openstack-release | 08:56 | |
| *** yamahata has quit IRC | 09:17 | |
| *** sdague has joined #openstack-release | 09:20 | |
| *** tosky has joined #openstack-release | 09:24 | |
| *** coolsvap has joined #openstack-release | 09:32 | |
| *** yamamoto has quit IRC | 09:37 | |
| *** udesale__ has quit IRC | 09:56 | |
| *** yamamoto has joined #openstack-release | 09:57 | |
| *** wolverineav has joined #openstack-release | 09:59 | |
| *** yamamoto has quit IRC | 10:08 | |
| *** dtantsur|bbl is now known as dtantsur | 10:11 | |
| *** liujiong has quit IRC | 10:12 | |
| *** mugsie has quit IRC | 10:22 | |
| *** mugsie has joined #openstack-release | 10:25 | |
| *** mugsie has quit IRC | 10:25 | |
| *** mugsie has joined #openstack-release | 10:25 | |
| *** yamamoto has joined #openstack-release | 10:25 | |
| *** thorst_afk has joined #openstack-release | 10:28 | |
| *** mugsie has quit IRC | 10:31 | |
| *** thorst_afk has quit IRC | 10:32 | |
| *** mugsie has joined #openstack-release | 10:33 | |
| *** mugsie has quit IRC | 10:33 | |
| *** mugsie has joined #openstack-release | 10:33 | |
| *** alexchadin has quit IRC | 10:35 | |
| *** alexchadin has joined #openstack-release | 10:36 | |
| dims | ttx : dhellmann : smcginnis : o/ | 10:51 |
| *** tellesnobrega has quit IRC | 10:51 | |
| *** alexchadin has quit IRC | 11:08 | |
| *** alexchadin has joined #openstack-release | 11:12 | |
| *** yamamoto has quit IRC | 11:17 | |
| *** alexchadin has quit IRC | 11:22 | |
| ttx | dims, dhellmann, smcginnis: o/ | 11:30 |
| *** yamamoto has joined #openstack-release | 11:32 | |
| *** tellesnobrega has joined #openstack-release | 11:33 | |
| *** thorst_afk has joined #openstack-release | 11:40 | |
| *** efried has quit IRC | 11:40 | |
| *** yamamoto has quit IRC | 11:49 | |
| *** lucasagomes is now known as lucas-hungry | 11:52 | |
| *** efried has joined #openstack-release | 11:52 | |
| *** claudiub has joined #openstack-release | 12:06 | |
| *** thorst_afk has quit IRC | 12:22 | |
| *** udesale has joined #openstack-release | 12:23 | |
| ttx | Recent group-based-policy-tarball job fail is a weird job timeout 1 hour after the task completed | 12:25 |
| ttx | could be unrelated infra issues | 12:26 |
| *** mriedem has joined #openstack-release | 12:27 | |
| *** tellesnobrega has quit IRC | 12:45 | |
| *** tellesnobrega has joined #openstack-release | 12:45 | |
| *** yamamoto has joined #openstack-release | 12:49 | |
| *** lucas-hungry is now known as lucasagomes | 12:50 | |
| *** electrofelix has quit IRC | 12:52 | |
| *** dave-mccowan has joined #openstack-release | 12:54 | |
| *** mriedem has quit IRC | 12:57 | |
| *** thorst_afk has joined #openstack-release | 12:57 | |
| *** electrofelix has joined #openstack-release | 12:58 | |
| *** yamamoto has quit IRC | 12:59 | |
| *** mriedem has joined #openstack-release | 13:06 | |
| openstackgerrit | Lance Bragstad proposed openstack/releases master: Release oslo.policy 1.25.1 https://review.openstack.org/489599 | 13:20 |
| lbragstad_ | dhellmann: looks like those backports to oslo.policy landed - i've proposed a release of oslo.policy stable/pike | 13:22 |
| lbragstad_ | https://review.openstack.org/#/c/489599/ | 13:22 |
| *** claudiub has quit IRC | 13:25 | |
| *** tosky has quit IRC | 13:31 | |
| *** hongbin has joined #openstack-release | 13:38 | |
| smcginnis | o/ | 13:45 |
| smcginnis | lbragstad_: Issue with one part of that release patch. | 13:47 |
| *** liujiong has joined #openstack-release | 13:48 | |
| smcginnis | ttx: WRT that group-based-policy-tarball failure - what needs to be done when a job like that fails? | 13:50 |
| openstackgerrit | Lance Bragstad proposed openstack/releases master: Release oslo.policy 1.25.1 https://review.openstack.org/489599 | 13:54 |
| lbragstad_ | smcginnis: good catch - thanks! | 13:54 |
| *** lbragstad_ is now known as lbragstad | 14:04 | |
| dhellmann | dims, ttx, smcginnis : o/ | 14:08 |
| dhellmann | smcginnis : for that case, since it's not an official project, we don't do anything unless they come to ask for help | 14:09 |
| dhellmann | smcginnis : for official projects, we debug the failure and either retrigger the job (ask infra to), or work with the team to fix the issue and tag a new release with a new number | 14:09 |
| dhellmann | ttx: did you have a chance to look at the yaml normalization patches? topic "queens-indentation" | 14:09 |
| smcginnis | dhellmann: Thanks. Yeah, wasn't sure how to retrigger a job like that. Makes sense. | 14:10 |
| dhellmann | smcginnis : fungi usually does that, although some of the other infra cores can do it, too. I forget exactly which superpower you need to do it. | 14:11 |
| fungi | any of the infra-root sysadmins can. it's just a tool you have to run from the command-line on the zuul.openstack.org server (the main scheduler for the ci system) | 14:12 |
| smcginnis | dhellmann, fungi: Nice, thanks for the info. Learning what I can. ;) | 14:12 |
| dims | smcginnis : don't know if you know, "infra-root" is a keyword on the -infra channel which all the folks with super powers watch for | 14:12 |
| fungi | which tool depends on the situation (whether you need to retrigger just a single job or want to reenqueue an entire ref) | 14:12 |
| fungi | so for example if a tarball build in the release pipeline fails, we would want to reenqueue that tag ref into the release pipeline so that all the jobs for it get rerun properly | 14:13 |
| smcginnis | dims: That the magic incantation. :) | 14:14 |
| fungi | however if the release announcement job fails in that situation, we would want to just retrigger that job because the tarballs/wheels were already built and uploaded to pypi so can't be rerun | 14:14 |
| dhellmann | fungi : it would be pretty cool to have a way to let us propose retriggering jobs like that, with infra approving, instead of having to take manual action. Is there anything like that on the zuul3 roadmap? | 14:15 |
| * dhellmann feels like he may have asked that question once before | 14:15 | |
| *** liujiong has quit IRC | 14:15 | |
| fungi | not that i'm aware of (and there's not much roadmap left for zuul v3 anyway, we're talking about go-live right before/at the ptg) | 14:15 |
| fungi | but there are a lot more options we can leverage in v3 to more easily implement something like that | 14:16 |
| dhellmann | cool, maybe that's a conversation to have in denver, then | 14:17 |
| fungi | however, that sort of runs counter to our design philosophy. if jobs fail often enough that you need to do something to address that situation, spend your time making the jobs more robust rather than designing elegant workarounds for broken things | 14:18 |
| fungi | because making the brokenness less painful just gives you more excuses not to address the root cause | 14:19 |
| fungi | and leads to crappy software | 14:19 |
| dhellmann | sure, that makes sense. maybe we should review the way we have some of these jobs chained, so we can retrigger them by landing another patch in the releases repo | 14:22 |
| fungi | so it's a balance to strike between problems we can solve in software and problems outside our control (like external to the environment or intrinsic to standard protocols) | 14:22 |
| dhellmann | we could avoid a lot of the release announce failures by testing that in the validation job in the releases repo | 14:23 |
| dhellmann | that one has been on my todo list for a while | 14:23 |
| ttx | dhellmann: looking | 14:24 |
| dhellmann | ttx: I'm trying to address the indentation issue that trips us up periodically because of differences in tools and in our existing data files | 14:25 |
| ttx | ok | 14:25 |
| dhellmann | that should make it easier to have commands to do batch operations like adding a bunch of branches or tagging the final release | 14:25 |
| *** udesale has quit IRC | 14:26 | |
| fungi | dhellmann: what angle are you taking? just straight-up parser correctness addressable by something like yamllint or do you need schema validation a la voluptuous? | 14:26 |
| dhellmann | fungi : we have both in place, but the linter was not configured to enforce the same indentation everywhere so it was only enforcing consistency within a file. That means some files have lists with members that are not indented, and others have lists with members that are indented. | 14:27 |
| fungi | ahh | 14:28 |
| dhellmann | adding an element to the list then either resulted in incorrect indentation (when we treated the file as text and just appended to it) or reindented the entire file (when we treated the file as a serialized data structure and reserialized it) | 14:28 |
| fungi | got it, so you mostly have a solution implemented and just need to tweak it to catch some things it currently misses | 14:29 |
| dhellmann | this series of patches nudges us down the road toward serialized data structures | 14:29 |
| dhellmann | yeah | 14:29 |
| dhellmann | there's a jsonschema check and a linter check they're just incomplete | 14:29 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add new-release and edit-deliverable to helpers list in readme https://review.openstack.org/489405 | 14:30 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add release-notes URLs for some pike deliverables https://review.openstack.org/489403 | 14:30 |
| ttx | reviewed half, jumped on a call, will finish after | 14:35 |
| jeblair | smcginnis: oh hi! /me wakes up and checks infra-root bat-signal | 14:41 |
| fungi | jeblair: no emergency, mostly zuul(v3) details/discussion | 14:44 |
| dhellmann | ttx: thanks | 14:45 |
| *** tellesnobrega has quit IRC | 14:48 | |
| smcginnis | jeblair: :) | 14:50 |
| *** bobh has joined #openstack-release | 15:06 | |
| *** pcaruana has quit IRC | 15:22 | |
| ttx | dhellmann: all reviewed | 15:24 |
| ttx | dhellmann approve before it merge-conflicts :) | 15:24 |
| ttx | clarkb, dhellmann: do you need me around the infra team meeting tonight ? I can probably attend if needed | 15:26 |
| dhellmann | ttx: maybe smcginnis or dims can do the approvals | 15:29 |
| ttx | maybe, yes :) | 15:29 |
| dhellmann | ttx, clarkb : if it's just a matter of when it's safe to take gerrit down for an upgrade, we could talk about that now and I can relay | 15:29 |
| *** claudiub has joined #openstack-release | 15:29 | |
| clarkb | ttx: only if you are particularly concerned about the scheduling of a downtime and upgrade | 15:30 |
| dhellmann | clarkb : do you have a proposed date? | 15:30 |
| clarkb | basially trying to find a good spot between enough lead time for announing to community, release, and ptg | 15:30 |
| ttx | dhellmann: what's the status on stable/pike branch creation ? | 15:31 |
| * dhellmann notes the PTG date is not on https://releases.openstack.org/pike/schedule.html | 15:31 | |
| ttx | The only big train of things we have is that large patchseries | 15:31 |
| clarkb | dhellmann: week before ptg and week after ptg are what I think may be easiest on everyone but is quite a ways out | 15:31 |
| dhellmann | ttx: all of the libs are branched. We wait for the services rc1 to branch. | 15:31 |
| ttx | so this week is probably the best moment anyway | 15:31 |
| dhellmann | ptg is sept 11, right? | 15:31 |
| ttx | as long as it's off any releases repo change | 15:31 |
| ttx | yes | 15:31 |
| ttx | It appears in the Queens schedule | 15:31 |
| ttx | (tentative) | 15:32 |
| dhellmann | ah | 15:32 |
| * smcginnis pulls up dhellmann's list of patches... | 15:32 | |
| dhellmann | oh, right, since it's the queens ptg | 15:32 |
| smcginnis | Right, I thought I saw it in the queens schedule. | 15:32 |
| dhellmann | I may add a cross-reference anyway | 15:32 |
| smcginnis | I wonder if those we should try to mention in both? Maybe overkill. | 15:32 |
| dhellmann | smcginnis : yeah, that's what I was thinking | 15:32 |
| dhellmann | I suppose the danger is that we'd end up with it in the ical file twice | 15:32 |
| dhellmann | maybe I can make it not do that | 15:33 |
| smcginnis | dhellmann: Not the end of the world if there's two cal entries. | 15:33 |
| ttx | I think it should only be on the queens one | 15:33 |
| smcginnis | At least my calendar is a mess anyways. :) | 15:33 |
| ttx | since it's a queens thing | 15:33 |
| dhellmann | clarkb : before or after the ptg seem like good timing | 15:33 |
| dhellmann | ttx: that's fair | 15:33 |
| smcginnis | ttx: True. | 15:33 |
| dhellmann | I was thinking of a cross-reference, since there's overlap between the calendars | 15:34 |
| dhellmann | but it's not a big deal | 15:34 |
| dhellmann | we don't technically need gerrit for the final release, but I would hate to have it down if we had a critical bug to fix in that timeframe | 15:35 |
| clarkb | and no concerns with it early in queens? | 15:35 |
| * dims peeks | 15:35 | |
| *** armax has quit IRC | 15:36 | |
| smcginnis | dhellmann: We don't have a risk of any automation seeing these deliverable merges and kicking off something because they appear to be changes, right? | 15:37 |
| dhellmann | clarkb : early in the cycle is usually better for something like this, right? | 15:38 |
| dhellmann | clarkb : smcginnis should weigh in, since he's ptl for queens :-) | 15:38 |
| smcginnis | dhellmann: Well, not yet. Could have a dark horse. :) | 15:38 |
| smcginnis | clarkb, dhellmann: But yeah, I think early in queens would be safer. | 15:39 |
| openstackgerrit | Merged openstack/releases master: fix format-yaml to work with python 3 https://review.openstack.org/488606 | 15:39 |
| smcginnis | No need to shoot ourselves in the foot if something goes wrong and we are without it. | 15:39 |
| *** pcaruana has joined #openstack-release | 15:40 | |
| openstackgerrit | Merged openstack/releases master: use yamlutils.dumps() in new-release command https://review.openstack.org/488845 | 15:40 |
| openstackgerrit | Merged openstack/releases master: use yamlutils.dumps() in init-series https://review.openstack.org/488846 | 15:40 |
| openstackgerrit | Merged openstack/releases master: add wrapper for yamllint https://review.openstack.org/489380 | 15:40 |
| openstackgerrit | Merged openstack/releases master: ensure version numbers always have quotes https://review.openstack.org/489381 | 15:40 |
| *** armax has joined #openstack-release | 15:40 | |
| clarkb | smcginnis: dhellmann ok that is what I figured but wanted to make double sure | 15:41 |
| openstackgerrit | Merged openstack/releases master: use yes/no for boolean values in reformatter https://review.openstack.org/489382 | 15:41 |
| dhellmann | clarkb : do you still need us in the meeting today? | 15:42 |
| clarkb | dhellmann: I don't think so, not if either of those weeks work for you. Will ping if that doesn't work for infra for some currently unknown reason and we can go from there | 15:43 |
| dhellmann | clarkb : wfm, thanks | 15:43 |
| smcginnis | ++ | 15:44 |
| ttx | ++ | 15:55 |
| *** claudiub has quit IRC | 16:08 | |
| *** ricolin has joined #openstack-release | 16:17 | |
| *** lucasagomes is now known as lucas-afk | 16:23 | |
| *** rajathagasthya has joined #openstack-release | 16:30 | |
| *** yamahata has joined #openstack-release | 16:31 | |
| *** ragasthya has joined #openstack-release | 16:33 | |
| openstackgerrit | Merged openstack/releases master: do not add line wraps to text when reformatting https://review.openstack.org/489383 | 16:34 |
| *** rajathagasthya has quit IRC | 16:35 | |
| openstackgerrit | Merged openstack/releases master: normalize independent deliverable files https://review.openstack.org/489384 | 16:35 |
| smcginnis | dhellmann: Want to send something brief to the ML letting folks know about the deliverable format changes? | 16:40 |
| smcginnis | dhellmann: Just thinking next time around we might have people confused why something that worked that last few releases suddenly doesn't. | 16:40 |
| dhellmann | smcginnis : makes sense | 16:40 |
| smcginnis | dhellmann: Or I can mention it. Just thinking it should be noted somewhere. | 16:41 |
| dhellmann | yes, definitely | 16:45 |
| *** bobh has quit IRC | 16:47 | |
| openstackgerrit | Merged openstack/releases master: normalize oslo deliverable files https://review.openstack.org/489385 | 16:48 |
| openstackgerrit | Merged openstack/releases master: normalize client deliverable files https://review.openstack.org/489386 | 16:48 |
| smcginnis | dhellmann: Quick question on here: https://review.openstack.org/#/c/489401/3 | 16:50 |
| openstackgerrit | Merged openstack/releases master: normalize tripleo deliverable files https://review.openstack.org/489387 | 16:50 |
| openstackgerrit | Merged openstack/releases master: normalize puppet deliverable files https://review.openstack.org/489388 | 16:54 |
| openstackgerrit | Merged openstack/releases master: normalize misc deliverable files https://review.openstack.org/489389 | 16:54 |
| openstackgerrit | Merged openstack/releases master: standardize indentation at 2 spaces https://review.openstack.org/488607 | 16:54 |
| openstackgerrit | Merged openstack/releases master: add linter rules for vertical whitespace https://review.openstack.org/488608 | 16:54 |
| openstackgerrit | Merged openstack/releases master: remove obsolete tool script for adding team names https://review.openstack.org/489400 | 16:54 |
| *** tellesnobrega has joined #openstack-release | 16:58 | |
| *** bobh has joined #openstack-release | 16:58 | |
| *** pcaruana has quit IRC | 16:58 | |
| *** dtantsur is now known as dtantsur|afk | 17:13 | |
| *** ragasthya has quit IRC | 17:34 | |
| *** yamahata has quit IRC | 17:43 | |
| *** electrofelix has quit IRC | 17:43 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: treat deliverable files as utf-8 in edit-deliverable https://review.openstack.org/489684 | 17:44 |
| *** rajathagasthya has joined #openstack-release | 17:48 | |
| openstackgerrit | Merged openstack/releases master: add edit-deliverable command https://review.openstack.org/489401 | 17:49 |
| openstackgerrit | Merged openstack/releases master: use edit-deliverable in script to set the release notes URLs https://review.openstack.org/489402 | 17:50 |
| openstackgerrit | Merged openstack/releases master: add release-notes URLs for some pike deliverables https://review.openstack.org/489403 | 17:52 |
| openstackgerrit | Merged openstack/releases master: add new-release and edit-deliverable to helpers list in readme https://review.openstack.org/489405 | 17:53 |
| openstackgerrit | Merged openstack/releases master: treat deliverable files as utf-8 in edit-deliverable https://review.openstack.org/489684 | 17:54 |
| openstackgerrit | Lance Bragstad proposed openstack/releases master: Release oslo.policy 1.25.1 https://review.openstack.org/489599 | 17:55 |
| *** ekcs has joined #openstack-release | 18:03 | |
| dhellmann | smcginnis : http://lists.openstack.org/pipermail/openstack-dev/2017-August/120423.html | 18:07 |
| smcginnis | dhellmann: Excellent! | 18:09 |
| *** EmilienM is now known as emacchi | 18:14 | |
| *** emacchi is now known as EmilienM | 18:14 | |
| smcginnis | lbragstad: Hey, was there discussion on IRC or the ML about releasing that new oslo.policy? Just looking for background and reason for doing it past lib freeze. | 18:14 |
| lbragstad | smcginnis: yeah - dhellmann and i were talking about it last week in -dev, let me see if i can find it | 18:17 |
| lbragstad | smcginnis: http://eavesdrop.openstack.org/irclogs/%23openstack-dev/%23openstack-dev.2017-07-28.log.html#t2017-07-28T15:58:23 | 18:17 |
| smcginnis | lbragstad: Thanks | 18:18 |
| lbragstad | smcginnis: no problem | 18:20 |
| *** yamahata has joined #openstack-release | 18:20 | |
| smcginnis | lbragstad: Sorry, I know you're in your team meeting right now, so no rush. | 18:23 |
| smcginnis | lbragstad: Is this needed now? Seems safe enough to release after the freeze is over. | 18:23 |
| *** pbandark has quit IRC | 18:42 | |
| lbragstad | smcginnis: the main reason we saw the importance of backporting those fixes was because it would allow us to use the show-policy directive supplied by oslo.policy to render our policy documentation in the configuration reference (for the doc-migration) work | 18:56 |
| lbragstad | smcginnis: for example - https://review.openstack.org/#/c/488508/ | 18:57 |
| lbragstad | smcginnis: https://bugs.launchpad.net/keystone/+bug/1707246 contains a bunch of context too if you haven't already seen it | 18:57 |
| openstack | Launchpad bug 1707246 in OpenStack Identity (keystone) "Configuration guide references configuration options for policy instead of sample policy file" [Low,In progress] - Assigned to Lance Bragstad (lbragstad) | 18:57 |
| smcginnis | Does seem very low risk. dhellmann, dims: what has been our stance on something like this in the past? | 18:58 |
| *** bobh has quit IRC | 18:58 | |
| dhellmann | smcginnis, lbragstad : we try to minimize risk. in this case, the change is only in the doc tool, and it would be easy enough to block the constraint update if it causes problems, so I think this risk is relatively low compared to the benefit we get from having those docs in place. | 19:01 |
| dhellmann | we should coordinate with the team in #openstack-requirements, too, though | 19:01 |
| smcginnis | Probably still a little early for tonyb. | 19:02 |
| smcginnis | It does seem very low risk. | 19:02 |
| dhellmann | I left a +2 on the release patch, but didn't approve it since we should have consensus first | 19:03 |
| dims | smcginnis : let's talk to tonyb also, as this has to get into u-c. i like the fact that it's just 2 commits that matter and they are not changing any code that is likely to hit at runtime - http://logs.openstack.org/99/489599/3/check/gate-releases-tox-list-changes-ubuntu-xenial/5bc688d/console.html#_2017-08-01_17_58_59_872987 | 19:04 |
| dims | lbragstad : better drop an FFE email as well to -dev mailing list | 19:04 |
| lbragstad | dims: sounds good - FFE tagged with [oslo] only? | 19:10 |
| lbragstad | dims: or should i include release in there, too? | 19:10 |
| dims | [release][requirements] too | 19:10 |
| dhellmann | dims,. smcginnis : maybe we want to document our ffe process this time around? | 19:16 |
| smcginnis | dhellmann: We really should have that somewhere. Thoughts on location for that? | 19:20 |
| dhellmann | smcginnis : project team guide? | 19:20 |
| dhellmann | we do have a release section there | 19:21 |
| dhellmann | it just doesn't have a ton of process info | 19:21 |
| smcginnis | +1 that makes sense. | 19:21 |
| dhellmann | I think we do have a section that talks about a typical release cycle? that might be a good place | 19:22 |
| smcginnis | dhellmann: Probably under the "Feature Freeze -1" section here? https://docs.openstack.org/project-team-guide/release-management.html#typical-development-cycle-schedule | 19:25 |
| dhellmann | yes, that would be a good spot | 19:26 |
| smcginnis | dhellmann: https://review.openstack.org/#/c/489723/ | 19:41 |
| *** rosmaita has joined #openstack-release | 19:43 | |
| dhellmann | smcginnis : I think I'm still not quite clear what you're trying to say in that second sentence | 19:47 |
| dhellmann | are you saying that raising the minimum version is another reason a release may be required? | 19:48 |
| rosmaita | dhellmann: smcginnis: are you talking about glance_store by any chance? | 19:48 |
| smcginnis | dhellmann: Yes, since that would not be allowed later if a stable release is done. | 19:48 |
| dhellmann | rosmaita : heh, no, we're trying to write down a policy :-) | 19:49 |
| smcginnis | rosmaita: No, actually just trying to better document how all that works when we need to bump a lib after the freeze. | 19:49 |
| dhellmann | smcginnis : ok | 19:49 |
| smcginnis | dhellmann: Maybe I should just remove that sentence? | 19:49 |
| rosmaita | sorry i joined in media res ... i have a question when you're done, no hurry | 19:49 |
| dhellmann | that situation doesn't necessarily require a new release, it may require a g-r update during the freeze period | 19:50 |
| dhellmann | so maybe those are separate concerns? | 19:50 |
| smcginnis | rosmaita: No problem, what's up? | 19:50 |
| dhellmann | separate but related | 19:50 |
| smcginnis | dhellmann: Well, we had the issue with os-brick where we needed to raise the minimum. | 19:50 |
| smcginnis | And stable does not allow the mins to be raised after a stable release has gone out. | 19:50 |
| rosmaita | it's about this patch that just merged raising the minimum of os_brick: https://review.openstack.org/#/c/489603/ | 19:50 |
| rosmaita | it merged in master | 19:50 |
| smcginnis | Speak of the devil. :) | 19:51 |
| rosmaita | after stable/pike was cut | 19:51 |
| rosmaita | so i guess i need to propose a backport to stable/pike ? | 19:51 |
| dhellmann | smcginnis : yes, I guess the key thing there was that the feature release was deemed "critical" for the rest of the projects, so we had to approve the release and the g-r change together | 19:51 |
| dhellmann | rosmaita: we haven't branched requirements yet, but when we do the bot will propose the update | 19:51 |
| rosmaita | oh, ok, so nothing for me to do yet | 19:52 |
| smcginnis | dhellmann: Yes, it was just extra ammo that if we did not do it before release, we would not be able to rely on consumers having the right version after the release as they could still grab the old (broken) version. | 19:52 |
| * dhellmann verifies that we haven't branched requirements yet | 19:52 | |
| dhellmann | rosmaita : yes, not yet | 19:52 |
| smcginnis | rosmaita: Good question! :) | 19:52 |
| dhellmann | smcginnis : true. maybe that case needs to be explained in more detail to make that clear. | 19:53 |
| dhellmann | case 1: there is a critical bug fix | 19:53 |
| rosmaita | ok, cool ... one last thing, is this the kind of thing we'd need to add a release note for, so people know why the new release? | 19:53 |
| dhellmann | case 2: there is a feature deemed critical for a service, which means a feature release and a g-r min change to ensure it is present | 19:53 |
| *** wolverineav has quit IRC | 19:53 | |
| smcginnis | dhellmann: Or maybe I should just remove it. If it confused you, then I think my chances of confusing others reading it will be pretty high. :) | 19:53 |
| dhellmann | rosmaita : a release note in glance? or os-brick? or both? | 19:53 |
| *** wolverineav has joined #openstack-release | 19:54 | |
| dhellmann | smcginnis : true, we could just be really general here | 19:54 |
| dhellmann | "impacting project releases" should cover us | 19:54 |
| smcginnis | dhellmann: Yeah, I think they'll figure it out based on their specific scenario. | 19:54 |
| smcginnis | Update coming.. | 19:54 |
| rosmaita | well, i was thinking glance_store, glance rc-1 hasn't been cut yet, so it should get the correct version of glance_store, so i didn't think it worth mentioning for glance | 19:54 |
| smcginnis | rosmaita: Yeah, I don't think a release note should be needed for that. | 19:55 |
| rosmaita | even better! thanks | 19:55 |
| smcginnis | rosmaita: g-r will get updated to have the minimum version. Not sure if anything extra needs to be called out on that. | 19:56 |
| rosmaita | smcginnis: works for me | 19:56 |
| smcginnis | dhellmann: When is requirements typically branched? | 19:56 |
| dhellmann | smcginnis : "after all cycle-with-milestone projects have their branches created" | 19:59 |
| *** bobh has joined #openstack-release | 20:00 | |
| dhellmann | hmm, those instructions say to create the branch by hand | 20:01 |
| dhellmann | we have openstack_requirements in the deliverables files though | 20:01 |
| smcginnis | Those artisinal branch instructions are everywhere. :) | 20:02 |
| smcginnis | That's one concern I had changing those feature branch instructions - if it every changes again it will be a pain to track down all references. | 20:03 |
| smcginnis | But I suppose at least now they point to the README, so even if the source page is wrong, there's a chance they will get to the correct current info. | 20:03 |
| *** wolverineav has quit IRC | 20:17 | |
| *** wolverineav has joined #openstack-release | 20:17 | |
| *** wolverineav has quit IRC | 20:17 | |
| *** jtomasek has quit IRC | 20:20 | |
| openstackgerrit | Dave McCowan proposed openstack/releases master: Release python-barbicanclient 4.5.2 https://review.openstack.org/489729 | 20:20 |
| dave-mccowan | dims fyi ^^ | 20:21 |
| dims | dave-mccowan : can you please throw a quick email to -dev for an FFE? "[release][requirements]" tags please | 20:25 |
| smcginnis | dave-mccowan: And if you have the time, provide feedback on here if the instructions make sense: https://review.openstack.org/#/c/489723/ | 20:25 |
| dave-mccowan | dims sure. i'll need a patch to global requirements first, right? since that will be the exemption we need? | 20:26 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: update directions for branching requirements repo https://review.openstack.org/489734 | 20:31 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: make openstack_requirements cycle-with-intermediary https://review.openstack.org/489735 | 20:31 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: teach edit-deliverable to add stable branch data https://review.openstack.org/489736 | 20:31 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add deliverable argument to discover_branch_points.py https://review.openstack.org/489737 | 20:31 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: update import_branch_points.sh to use edit-deliverable https://review.openstack.org/489738 | 20:31 |
| dhellmann | dave-mccowan : you need the exception for the release and for the constraint update, so start with the email pointing to the release | 20:32 |
| dhellmann | dave-mccowan : do you need to raise the minimum version for barbicanclient in global requirements? | 20:33 |
| dave-mccowan | dhellmann i think we should block 4.5.0 and 4.5.1 as bad | 20:34 |
| dhellmann | dave-mccowan : ok, include that info in the email | 20:34 |
| lbragstad | dhellmann: smcginnis dims done - http://lists.openstack.org/pipermail/openstack-dev/2017-August/120434.html | 20:36 |
| smcginnis | lbragstad: Perfect. I'll respond on there with my opinion. | 20:39 |
| openstackgerrit | Merged openstack/releases master: update directions for branching requirements repo https://review.openstack.org/489734 | 20:40 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: fix typos in yamlutils https://review.openstack.org/489745 | 20:52 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: do not let yamlutils emit alias references https://review.openstack.org/489746 | 20:52 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: have propose-final-releases use yamlutils instead of appending to the file https://review.openstack.org/489747 | 20:52 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: Release tooz 1.48.2 for Ocata https://review.openstack.org/486108 | 20:55 |
| *** ekcs has quit IRC | 21:01 | |
| *** yamamoto has joined #openstack-release | 21:17 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: remove unused data from new-release https://review.openstack.org/489752 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: use yamlutils.loads() everywhere instead of yaml.safe_load() https://review.openstack.org/489753 | 21:18 |
| *** yamamoto has quit IRC | 21:21 | |
| *** yamamoto has joined #openstack-release | 21:21 | |
| *** rosmaita has left #openstack-release | 21:22 | |
| *** yamamoto has quit IRC | 21:26 | |
| *** dave-mccowan has quit IRC | 21:27 | |
| jeblair | heads up that the pypi mirrors haven't updated since 2:15 UTC due to a bug on pypi.python.org. i reported it and they fixed the problem; mirror updates are currently in progress. | 21:27 |
| dhellmann | jeblair : thanks | 21:48 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: use yamlutils.loads() everywhere instead of yaml.safe_load() https://review.openstack.org/489753 | 21:51 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: remove unused data from new-release https://review.openstack.org/489752 | 21:51 |
| *** ekcs has joined #openstack-release | 21:57 | |
| tonyb | which change are we talking about? oslo.policy 1.25.1 ? | 21:58 |
| *** thorst_afk has quit IRC | 22:05 | |
| *** ekcs has quit IRC | 22:12 | |
| *** bobh has quit IRC | 22:16 | |
| jeblair | the pypi mirror is up to date now | 22:16 |
| *** thorst_afk has joined #openstack-release | 22:19 | |
| *** yamamoto has joined #openstack-release | 22:23 | |
| *** thorst_afk has quit IRC | 22:24 | |
| *** yamamoto has quit IRC | 22:25 | |
| *** yamamoto has joined #openstack-release | 22:25 | |
| *** sdague has quit IRC | 22:35 | |
| *** dave-mccowan has joined #openstack-release | 22:44 | |
| dhellmann | tonyb : both oslo.policy and python-barbicanclient have asked for FFE releases | 22:44 |
| dhellmann | jeblair : thanks | 22:44 |
| clarkb | hello release team. We are currently looking at a september 18 gerrit upgrade to get that done before everyone returns to work from the ptg. In the infra meeting we were wondering if that is something that the release team would want to advertise on the release schedule? its not a confirmed hard date but the one we are looking at so no rush. More something to think about | 22:58 |
| *** https_GK1wmSU has joined #openstack-release | 23:02 | |
| *** https_GK1wmSU has left #openstack-release | 23:02 | |
| smcginnis | clarkb: Not sure if we should publish that on the schedule. At a minimum though, I think several ML posts would be good leading up to it. ;) | 23:03 |
| clarkb | smcginnis: yup definitely intend on giving people lots of warning on the mailing list | 23:04 |
| clarkb | once we firm up the schedule the first notice will go out | 23:04 |
| clarkb | (hopefully tomorrow evening) | 23:04 |
| smcginnis | clarkb: Awesome. Is there a convenient list of differences somewhere with the new version? Like a release notes page or something? | 23:05 |
| * smcginnis hasn't been keeping up with upstream gerrit | 23:05 | |
| clarkb | smcginnis: https://www.gerritcodereview.com/releases/2.12.md and https://www.gerritcodereview.com/releases/2.13.md for end users it should largely be the same. 2.14 will be a big change so we aren't jumpign to it just yet | 23:06 |
| smcginnis | clarkb: Cool, thanks for the links. | 23:06 |
| clarkb | mobile browsers work a little better under 2.13. You don't have to remove approvals and reapply them if a change already has a positive verified vote, you can just recheck and it will now do the right thing | 23:06 |
| clarkb | minor stuff like that | 23:07 |
| clarkb | smcginnis: you can test your workflow at https://review-dev.openstack.org too | 23:07 |
| smcginnis | clarkb: Hmm, if mobile works better (actually works) that's more than minor to me. ;) | 23:07 |
| clarkb | no it doesn't actually work unfortunately | 23:08 |
| clarkb | it just works better :) | 23:08 |
| smcginnis | Haha, fair enough. | 23:08 |
| clarkb | you still can't inline comment with it at least not on my phone but the rendering is much more readable for diffs on small screens | 23:08 |
| smcginnis | clarkb: Well, that is something at least. | 23:09 |
| clarkb | the metrics stuff is potentially a useful new feature as well | 23:09 |
| smcginnis | The most important question - can I press 'f' again to get a pop up of the file list? | 23:10 |
| clarkb | no | 23:10 |
| clarkb | not that fortunate | 23:10 |
| smcginnis | That's it, I quit. | 23:10 |
| smcginnis | :) | 23:11 |
| clarkb | 2.14 adds proper support for the polygerrit interface which is yet another rewrite of the UI | 23:11 |
| smcginnis | I saw an earlier preview of that I think. Looked promising. | 23:11 |
| clarkb | I think it has gotten all of the UI attention for several releases so 2.13 doesn't really have much | 23:11 |
| clarkb | its a lot more like the old change screen | 23:11 |
| clarkb | and written with proper javascript rather than gwt | 23:11 |
| smcginnis | I think that will make a lot of folks happy. | 23:11 |
| clarkb | let me find a link for you | 23:13 |
| clarkb | https://gerrit-review.googlesource.com/c/112531 | 23:13 |
| clarkb | it has gone back to a change oriented change screen instead of patchset oriented | 23:13 |
| clarkb | and has single page diff (hit the show diffs button) | 23:14 |
| smcginnis | A little different, but shouldn't take too long to get used to. | 23:14 |
| clarkb | we decided not to go to that version right away because it introduces an entire new database system, requires java 8 (an os upgrade for us), polygerrit, etc. Went with easier upgrade to build momentum but hope to move on to 2.14 in the not too distant future | 23:15 |
| smcginnis | Yeah, makes sense. | 23:16 |
| clarkb | also it has built in CI comment hiding so we can drop our hacky javascript | 23:16 |
| smcginnis | Oh nice! | 23:17 |
| *** hongbin has quit IRC | 23:30 | |
| tonyb | dhellmann: okay. I'm looking at both of them now. I'll reply on list | 23:38 |
| -openstackstatus- NOTICE: osic nodes have been removed from nodepool due to a problem with the mirror host beginning around 22:20 UTC. please recheck any jobs with failures installing packages. | 23:48 | |
| dims | tonyb : when you wake up, please take a look at the oslo.policy and python-barbicanclient FFE/release request | 23:58 |
| tonyb | dims: :) Sure | 23:59 |
| dims | ah there you are :) | 23:59 |
| dims | good morning | 23:59 |
| * tonyb tries not to point out I just said that to dhellmann ;P | 23:59 | |
| tonyb | dims: morning | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!