Friday, 2018-07-27

openstackgerritMichael Johnson proposed openstack/releases master: Release Octavia projects for Rocky MS3  https://review.openstack.org/58594500:03
*** harlowja has joined #openstack-release00:05
*** linkmark has quit IRC00:07
*** bobh has quit IRC00:08
openstackgerritMerged openstack/releases master: Release Octavia projects for Rocky MS3  https://review.openstack.org/58594500:23
*** harlowja has quit IRC01:17
openstackgerritzhurong proposed openstack/releases master: Release Murano projects for rocky-3  https://review.openstack.org/58638301:27
lbragstad_smcginnis: rechecked https://review.openstack.org/#/c/586374/1 (its dependency merged) it should be good to go01:44
*** ricolin has joined #openstack-release01:45
*** bobh has joined #openstack-release01:46
*** yamahata has quit IRC01:48
smcginnislbragstad_: Thanks01:51
*** bobh has quit IRC01:52
openstackgerritMerged openstack/releases master: Release keystone rocky-3  https://review.openstack.org/58637402:03
openstackgerritzhurong proposed openstack/releases master: Release Murano projects for rocky-3  https://review.openstack.org/58638302:17
*** gongysh has joined #openstack-release02:22
*** rosmaita has quit IRC02:55
*** armax has quit IRC03:00
*** armax has joined #openstack-release03:04
*** armax has quit IRC03:12
*** armax has joined #openstack-release03:16
openstackgerritJay Bryant proposed openstack/releases master: Release python-cinderclient 4.0.0  https://review.openstack.org/58640303:23
*** dave-mccowan has quit IRC03:24
*** armax has quit IRC03:29
openstackgerritmelanie witt proposed openstack/releases master: os-vif: release rocky 1.11.1  https://review.openstack.org/58640403:35
*** gongysh has quit IRC03:52
*** yamahata has joined #openstack-release03:53
*** ekcs has quit IRC03:54
*** mriedem has quit IRC04:04
*** pcaruana has joined #openstack-release04:28
*** pcaruana has quit IRC04:30
*** gongysh has joined #openstack-release04:50
*** zigo_ has joined #openstack-release05:53
*** zigo has quit IRC05:53
*** alexchadin has joined #openstack-release06:15
*** lvdombrkr has joined #openstack-release06:18
openstackgerritJeffrey Zhang proposed openstack/releases master: Release kolla rocky 7.0.0.0b3  https://review.openstack.org/58644706:19
openstackgerritJeffrey Zhang proposed openstack/releases master: Release kolla-ansible rocky 7.0.0.0b3  https://review.openstack.org/58644906:22
*** lvdombrkr89 has joined #openstack-release06:31
*** lvdombrkr has quit IRC06:33
openstackgerritAbhishek Kekane proposed openstack/releases master: Release Glance 17.0.0.0b3  https://review.openstack.org/58635706:47
*** amoralej|off is now known as amoralej06:50
*** dtantsur|afk is now known as dtantsur07:21
openstackgerritMerged openstack/releases master: Release keystone 13.0.1  https://review.openstack.org/58624207:22
openstackgerritMerged openstack/releases master: Release keystone 12.0.1  https://review.openstack.org/58624107:29
openstackgerritMerged openstack/releases master: Release keystone 11.0.4  https://review.openstack.org/58624007:29
openstackgerritMerged openstack/releases master: Release python-manilaclient 1.21.1 for stable/queens  https://review.openstack.org/58583407:30
openstackgerritMerged openstack/releases master: Release manila stable/queens 6.0.2  https://review.openstack.org/58579607:33
*** shardy has joined #openstack-release07:37
*** jtomasek has joined #openstack-release07:48
*** jtomasek has quit IRC07:48
*** jtomasek has joined #openstack-release07:49
*** alexchadin has quit IRC07:52
*** dtantsur is now known as dtantsur|bbl08:00
dhellmanntonyb : I honestly have no idea what's going on with reproducing that git issue for openstackdocstheme :-/08:04
dhellmannI'm at europython this week and the conference wifi is poor, so I haven't been able to do more than look at the logs from the previous failures.08:05
*** alexchadin has joined #openstack-release08:05
*** jpich has joined #openstack-release08:06
tonybdhellmann: Sorry I didn't mean to bug you.  and I certainly don't wnat to looking at it while treaveling.  I was just enjoying the conversation and puzzle08:17
dhellmanntonyb : you're not bugging -- I'm sorry I couldn't explain what's going on08:18
tonybsmcginnis, dhellmann: something to talk about later, would you object to addding a 'branch_point' key into series_status.yaml ? I'd kinda like it for soemthign I want to do in the elections repo.08:19
*** bauzas is now known as PapaOurs08:19
tonybsmcginnis, dhellmann: I know I can poke around in other yaml files or git to derive it but it'd be nice to have it in a nice easy to find spot08:20
dhellmanntonyb : what information would it hold?08:20
dhellmanna date?08:20
tonybdhellmann: glad I'm not bugging08:20
tonybdhellmann: Yeah basically it'd hold a 'best guess' average date for when most projects branched RC108:20
tonybdhellmann: which in turn is a proxy for when $series opened for development08:21
dhellmannsure. we could get that from the schedule data file, too, but I can see the benefit of just reading the one file.08:22
*** e0ne has joined #openstack-release08:22
dhellmannwe should think about whether we want the $series entry to include the start date for $series or the branch date for stable/$series08:22
tonybYeah right now I'm just sluping series_status.yaml from git as I'd like avoid too much coupling between election and releases08:23
dhellmann++08:23
tonybdhellmann: If I want to cover all of queens, I'd like to find a date in the pike cycle where we tagged pike RC1 which is typically when we created stable/pike so that'd work best for me08:25
tonybanyway it seems like it isn't too terrible of an idea and I wont really need it for another 3 months so we can noodle it out and work out how to create it without adding a manual step into the process for createing a new series08:26
dhellmannyeah, I don't see an issue with adding that info08:27
tonybdhellmann: Thanks.08:27
tonybdhellmann: enjoy europython :)08:27
dhellmanntoday's the last day, but so far the talks have been good08:27
tonybdhellmann: ahh cool.  I hope the travel back isn't too bad :)08:28
* tonyb bails for today08:29
*** lifeless has quit IRC08:54
*** d0ugal has joined #openstack-release09:09
*** lifeless has joined #openstack-release09:11
*** andymccr- has joined #openstack-release09:47
*** andymccr_ has quit IRC09:50
*** johnthetubaguy has quit IRC09:52
*** andymccr has quit IRC10:04
*** andymccr- is now known as andymccr10:05
*** alexchadin has quit IRC10:26
*** pkovar has joined #openstack-release10:30
*** alexchadin has joined #openstack-release10:39
tbarronsmcginnis: as soon as https://review.openstack.org/#/c/535178 merges I10:43
tbarronsmcginnis: I'll push the release; need it no client microversion lines up with our features10:44
tbarrons/no/so/10:44
*** gongysh has quit IRC10:47
*** dtantsur|bbl is now known as dtantsur10:49
*** dave-mccowan has joined #openstack-release11:06
*** pkovar has quit IRC11:29
*** pkovar has joined #openstack-release11:30
*** zul has quit IRC11:34
*** alexchadin has quit IRC11:34
*** alexchadin has joined #openstack-release11:35
*** alexchadin has quit IRC11:35
*** alexchadin has joined #openstack-release11:36
*** alexchadin has quit IRC11:36
*** alexchadin has joined #openstack-release11:36
*** alexchadin has quit IRC11:37
*** d0ugal has quit IRC11:41
*** witek_ has joined #openstack-release11:43
*** witek has quit IRC11:44
*** witek_ is now known as witek11:44
*** amoralej is now known as amoralej|lunch11:54
*** linkmark has joined #openstack-release12:03
openstackgerritTom Barron proposed openstack/releases master: Release python-manilaclient 1.24.0  https://review.openstack.org/58652212:12
*** alexchadin has joined #openstack-release12:16
*** edmondsw has joined #openstack-release12:17
*** johnthetubaguy has joined #openstack-release12:17
*** alexchadin has quit IRC12:20
*** zul has joined #openstack-release12:22
*** zul has quit IRC12:28
*** zul has joined #openstack-release12:30
*** alexchadin has joined #openstack-release12:33
*** mriedem has joined #openstack-release12:35
*** zul has quit IRC12:36
*** zul has joined #openstack-release12:37
*** zul has quit IRC12:44
*** zul has joined #openstack-release12:45
*** amoralej|lunch is now known as amoralej12:47
openstackgerritRico Lin proposed openstack/releases master: Release heatclient 1.17.0  https://review.openstack.org/58418712:49
smcginnistonyb: I wonder if you could just pull in doc/source/$SERIES/schedule.yaml similar to how we grab governance project data - https://github.com/openstack/releases/blob/master/openstack_releases/governance.py#L2312:51
*** zul has quit IRC12:51
*** zul has joined #openstack-release12:53
*** zul has quit IRC12:58
*** zul has joined #openstack-release13:00
*** rpioso|afk is now known as rpioso13:03
openstackgerritMerged openstack/releases master: Release python-manilaclient 1.24.0  https://review.openstack.org/58652213:06
openstackgerritMerged openstack/releases master: Release Glance 17.0.0.0b3  https://review.openstack.org/58635713:06
openstackgerritMerged openstack/releases master: Release python-cinderclient 4.0.0  https://review.openstack.org/58640313:06
openstackgerritMerged openstack/releases master: Release Murano projects for rocky-3  https://review.openstack.org/58638313:06
openstackgerritMerged openstack/releases master: Release kolla-ansible rocky 7.0.0.0b3  https://review.openstack.org/58644913:09
*** zul has quit IRC13:14
*** zul has joined #openstack-release13:15
openstackgerritMerged openstack/releases master: Release kolla rocky 7.0.0.0b3  https://review.openstack.org/58644713:19
*** zul has quit IRC13:21
*** zul has joined #openstack-release13:23
openstackgerritTom Barron proposed openstack/releases master: Release Manila 7.0.0.0b3  https://review.openstack.org/58654713:26
*** zul has quit IRC13:28
*** rosmaita has joined #openstack-release13:30
*** zul has joined #openstack-release13:30
*** alexchadin has quit IRC13:36
*** alexchadin has joined #openstack-release13:42
*** zul has quit IRC13:43
*** alexchadin has quit IRC13:49
smcginnisricolin: Hey, looking at the heatclient changes, is there a reason that isn't 1.16.1? It looks like the only commits of release significants were a couple of small bugfixes.13:53
*** ianychoi has joined #openstack-release13:56
*** mlavalle has joined #openstack-release13:57
*** zul has joined #openstack-release13:58
ricolinsmcginnis, Indeed only few bug fixes, and I'm fine to release it as 1.16.113:59
*** ianychoi_ has quit IRC14:00
openstackgerritRico Lin proposed openstack/releases master: Release heatclient 1.17.0  https://review.openstack.org/58418714:00
ricolinsmcginnis, done14:00
smcginnisrosmaita: Perfect, thanks!14:00
smcginnisrosmaita: Sorry, wrong tab completion. :D14:01
smcginnisricolin: Can you update the version in the commit message so there's no confusion too?14:01
ricolinsmcginnis, also I will release heat later today in case you're curious14:01
ricolinoops14:01
smcginnisricolin: What is the delay with heat?14:01
openstackgerritRico Lin proposed openstack/releases master: Release heatclient 1.16.1  https://review.openstack.org/58418714:02
ricolinsmcginnis, we got a lot of features landed just now, now there are some small patches in gate, would like to give them a chance if that's fine with you14:03
smcginnisricolin: Yes, just unfortunate that the deadline was yesterday. Was there last minute issues that came up?14:04
ricolinsmcginnis, there's no last minute issues, just most features not getting any reviews from core:/14:07
ricolinsmcginnis, will release now, and sorry about the delay14:07
*** annabelleB has joined #openstack-release14:08
smcginnisricolin: If you have things in the gate queue we can wait until they land. I'm just trying to get a feel for what led to missing the deadline to see if there is anything that can be done about it.14:08
smcginnisricolin: Sounds like the common thing of not getting enough cores to focus on things until right before the deadline hits.14:08
smcginnistbarron: I should ask you too. Is there something that caused manila to be late?14:10
ricolinsmcginnis, or still not get enough review even right at the deadline14:10
smcginnisricolin: True! I know the feeling.14:10
ricolinsmcginnis, anyway I will release now14:10
smcginnisricolin: OK, great. If you're ready that would be good.14:10
*** annabelleB has quit IRC14:11
tbarronsmcginnis: yeah, I failed to push to get a python-manilaclient patch merged in time that we needed for14:11
tbarronsmcginnis: the client to match the server14:12
openstackgerritRico Lin proposed openstack/releases master: Release Heat 11.0.0.0b3 (Rocky m3)  https://review.openstack.org/58656914:12
smcginnistbarron: OK, thanks.14:12
ricolinsmcginnis, ^^^14:12
smcginnisricolin: Thank you!14:12
tbarronsmcginnis: didn't want to be in the position of having a client that couldn't support the full rocky featureset14:13
tbarronfor manila14:13
smcginnismelwitt: Any update on the nova release?14:13
smcginnistbarron: Might be better if that case happens again to at least release the service side and just have a delay on the client.14:14
tbarronsmcginnis: noted, can do that14:14
melwittsmcginnis: we have a bad gate bug that started yesterday afternoon that we're trying to remediate, it's been preventing things we need for the release from merging14:14
smcginnismelwitt: OK, I'll keep my fingers crossed.14:15
smcginnisEmilienM: Did those pypi jobs get set up for tripleo yet?14:16
EmilienMsmcginnis: they are now14:16
EmilienMI fixed it last night.14:16
EmilienMbut I have README issues now14:16
EmilienMwhy are we checking readme in release jobs?14:16
EmilienMI'm in a meeting right now but I need to update the release patch with latest SHA1 and see what the errors are now14:16
smcginnisEmilienM: Pypi added enforcement, so we added checks in our jobs to make sure the release doesn't fail at the end when we go to upload it.14:16
smcginnisEmilienM: You can run those checks locally to the repo by running "python setup.py check -r -s".14:17
EmilienMI know how to run them14:17
EmilienMbut I need to figure the errors now14:17
*** jlvacation is now known as jlvillal14:18
*** shardy has quit IRC14:18
smcginnissamP: Did pypi get set up for masakari-dashboard yet?14:18
*** alexchadin has joined #openstack-release14:26
openstackgerritRico Lin proposed openstack/releases master: Release heat-dashboard 1.3.0  https://review.openstack.org/58657614:26
rosmaitasmcginnis: i am always happy to accept, "Perfect, thanks!"14:30
smcginnisrosmaita: Hah ;)14:30
*** alexchadin has quit IRC14:30
*** shardy has joined #openstack-release14:40
evrardjpDid you know it's super hard to find this team weekly's meeting agenda and time?14:41
evrardjpon google14:42
*** pkovar has quit IRC14:42
evrardjpI had to go on eavesdrop!14:42
smcginnisevrardjp: Hah, well eavesdrop is the official place for listing all of those. :)14:42
evrardjpthat's true.14:43
smcginnisMaybe I could add that info to my countdown emails so maybe google would pick it up.14:43
smcginnisevrardjp: We did cancel this weeks meeting though since everyone else is out of the office this week.14:43
smcginnisevrardjp: Was there something you wanted to bring up there?14:43
evrardjpnothing really, I just realise I haven't been really participating enough, wanted to change that.14:44
evrardjpFYI on OSA side for M3, we've been struggling for two days with slow gating, so it's still pending. Sorry for the delay14:45
smcginnisevrardjp: Thanks, I appreciate that.14:45
evrardjpthat's normal14:45
smcginnisevrardjp: That does seem to be a common theme today.14:45
evrardjpyeah14:45
*** pkovar has joined #openstack-release14:50
*** mlavalle has quit IRC14:52
*** mlavalle has joined #openstack-release14:52
*** alexchadin has joined #openstack-release15:04
*** e0ne has quit IRC15:16
*** annabelleB has joined #openstack-release15:16
*** alexchadin has quit IRC15:24
prometheanfireevrardjp: you and everyone else :P15:32
prometheanfireit's that time of cycle15:32
evrardjpyeah15:32
evrardjpprometheanfire: we have long jobs though -- 2h per part (so 4h for a full gating) is HARD when you fail.15:32
evrardjpit's been two days now.15:33
*** andymccr has quit IRC15:34
*** andymccr has joined #openstack-release15:35
prometheanfireevrardjp: we have cross jobs, we probably need to just increase the time, again, spectre is sad15:35
prometheanfireevrardjp: also, they are restarting gate in a couple min15:35
smcginnisAnd all jobs are now failing in the gate and will need to be rechecked.15:35
prometheanfirewooo15:35
smcginnisevrardjp: This is your punishment for missing the deadline. :P15:36
prometheanfirelol15:36
prometheanfireevrardjp: you going to ptg?15:36
evrardjpI have to book stuff but yes I'd like to. Passing the baton! :p15:36
evrardjpsmcginnis: I've seen that.15:37
evrardjpsmcginnis: you jinxed it.15:37
evrardjp:p15:37
smcginnis:)15:37
prometheanfireevrardjp: indeed15:37
evrardjp2 days for this darned patch!15:37
evrardjpI see you continue prometheanfire :)15:37
*** hongbin_ has joined #openstack-release15:39
-openstackstatus- NOTICE: A zuul config error slipped through and caused a pile of job failures with retry_limit - a fix is being applied and should be back up in a few minutes15:39
*** ChanServ changes topic to "A zuul config error slipped through and caused a pile of job failures with retry_limit - a fix is being applied and should be back up in a few minutes"15:39
prometheanfireI continue?15:40
evrardjpprometheanfire: PTLing15:42
prometheanfireah, yep, this cycle should be good15:43
*** lvdombrkr89 has quit IRC15:45
johnsomHi, it looks like the python-octaviaclient release notes didn't render for the 1.6.0 release we just did: https://docs.openstack.org/releasenotes/python-octaviaclient/unreleased.html15:54
johnsomhttps://review.openstack.org/#/c/585943/ was the release15:54
johnsomOur release notes gate was fine on the last patch that went in: http://logs.openstack.org/90/539390/3/check/build-openstack-releasenotes/466ecb1/html/unreleased.html15:55
clarkbjohnsom: possibly fallout from the zuul misconfiguration that just happened (see the status notice above)15:57
clarkbwe may have to reenqueue jobs for those artificats15:58
clarkbmordred: pabelanger ^ fyi15:58
johnsomHmm, that went in yesterday morning, so not sure about that15:58
clarkbah15:58
smcginnisclarkb, johnsomL I think it's actually because there is no rocky landing page yet.15:58
smcginnisThere are no unreleased release notes since they've all been released.15:58
smcginnisjohnsom: This needs to land: https://review.openstack.org/#/c/586216/15:59
johnsomsmcginnis We usually point them to the "unreleased" page in the project area until the main release notes job changes them to a release specific page.  All of our M3 releases worked correctly and rendered.16:00
*** shardy has quit IRC16:01
smcginnisjohnsom: Yes, but python-octaviaclient has branched with the final rocky release, so those rocky release notes are now only displayed under rocky and not unreleased.16:01
johnsomYeah, we didn't set it to rocky. I don't think i even passes validation if you put in the release specific path before that is in.16:01
smcginnisjohnsom: Yep, that's why the bot proposes those patches at the time it happens.16:01
johnsomAh, I follow you now.16:02
johnsomhmm, guess I hadn't noticed that in the past releases.16:02
*** openstackgerrit has quit IRC16:04
*** jpich has quit IRC16:06
mordredsmcginnis, dhellmann: if a reno note is in both stable/rocky and in master (because of backports) - reno is gonna do the right thing and include it in the release notes for releases cut from stable/rocky right?16:08
smcginnismordred: Yes, it will show up in rocky. But unless behavior has changed, it will also show up in the unreleased (then stein) release notes as well.16:09
mordredyah - that's totally fine16:10
mordredjust wanted to make sure it would show up in rocky - because I need a bugfix release and that'll be the only release note for the release, so it would otherwise be a confusing set of releaenotes16:10
smcginnismordred: Yep, then you're all good.16:10
mordred"why does this release exist?" :)16:10
smcginnisThat question might be asked of a few things. ;)16:10
mordredsmcginnis: in case you were wondering - you can't call len() on an interator16:11
smcginnismordred: I actually just saw that somewhere.16:11
smcginnismordred: Maybe it was on your patch.16:11
*** lbragstad_ is now known as lbragstad16:11
*** amoralej is now known as amoralej|off16:11
smcginnisThe tricky/confusing part can be backports where you don't want it to show up on in the next release's notes.16:12
smcginnisThen you have to break normal stable policy and go right to the stable branch.16:12
smcginnishttps://docs.openstack.org/reno/latest/user/usage.html#updating-stable-branch-release-notes16:12
mordredyah. I had originally thought to just have the reno note in the stable patch16:12
mordredyah16:12
*** mriedem is now known as mriedem_away16:15
*** openstackgerrit has joined #openstack-release16:17
openstackgerritMerged openstack/releases master: Release heatclient 1.16.1  https://review.openstack.org/58418716:17
*** armax has joined #openstack-release16:21
openstackgerritMerged openstack/releases master: Release heat-dashboard 1.3.0  https://review.openstack.org/58657616:26
*** harlowja has joined #openstack-release16:27
*** pkovar has quit IRC16:46
openstackgerritMerged openstack/releases master: Release Manila 7.0.0.0b3  https://review.openstack.org/58654716:52
*** ricolin has quit IRC16:59
*** yamahata has quit IRC17:07
*** dtantsur is now known as dtantsur|afk17:10
*** mriedem_away is now known as mriedem17:23
openstackgerritMerged openstack/releases master: os-vif: release rocky 1.11.1  https://review.openstack.org/58640417:23
*** harlowja has quit IRC17:31
openstackgerritMerged openstack/releases master: Release Heat 11.0.0.0b3 (Rocky m3)  https://review.openstack.org/58656917:34
*** yamahata has joined #openstack-release17:43
*** jtomasek has quit IRC18:10
*** mriedem1 has joined #openstack-release18:14
*** mriedem has quit IRC18:14
*** harlowja has joined #openstack-release18:15
*** mriedem1 is now known as mriedem18:22
*** Qiming has quit IRC18:27
*** Qiming has joined #openstack-release18:29
*** jtomasek has joined #openstack-release19:19
*** jtomasek has quit IRC19:19
*** annabelleB has quit IRC19:32
*** lbragstad_ has joined #openstack-release19:40
*** lbragstad has quit IRC19:41
*** lbragstad_ is now known as lbragstad19:56
*** e0ne has joined #openstack-release20:06
*** rosmaita has quit IRC20:11
prometheanfiresmcginnis: didn't we talk about os-vif needing a ffe?20:21
*** e0ne has quit IRC20:22
*** dtruong_ has quit IRC20:23
*** dtruong_ has joined #openstack-release20:26
*** annabelleB has joined #openstack-release20:30
clarkbsmcginnis: looks like there are no more release jobs queued I think we are going to try and restart zuul now. Demand has fallen off as well so we should catch up on job backlog quickly20:46
*** edmondsw has quit IRC21:04
prometheanfiresmcginnis: tbarron: not sure who else from manilla is in the channel, but manilaclient is going to need a re-release, we can't merge the requirements update because of https://github.com/openstack/python-manilaclient/blob/master/requirements.txt#L2021:05
*** edmondsw has joined #openstack-release21:05
prometheanfire=== in requirements :(21:05
*** yamahata has quit IRC21:05
prometheanfiresomeone has watched https://video.fosdem.org/2018/K.3.201/how_to_make_package_managers_cry.webm21:05
clarkbok restarting zuul now.21:06
prometheanfireclarkb: more rechecks?21:06
clarkbprometheanfire: no, now its deploying a weeks worth of bugfixes we couldn't deploy due to feature freeze21:06
smcginnisprometheanfire: os-vif we had talked about a couple days ago with melwitt I thought.21:06
smcginnistbarron: See above for issue with manilaclient.21:06
smcginnisclarkb: Thanks, I had to step away for awhile, so it probably was a good window to restart.21:07
prometheanfireclarkb: :D21:07
tbarronprometheanfire: smcginnis: looking21:07
prometheanfiremelwitt: iirc you said you'd submit the FFE for the os-vif release?21:08
melwittsmcginnis, prometheanfire: yes, we had a ffe and we acked it on the dev ML and it was released today, version 1.11.1. is there something wrong?21:08
prometheanfireoh, that's the release we talked about21:08
prometheanfirek, thought it was another one21:08
melwittok21:08
*** edmondsw has quit IRC21:10
smcginnisprometheanfire: That looks like a good presentation. I'll have to watch it all later when I have some time.21:10
prometheanfire:D21:11
prometheanfireya, I watch it every now and then21:11
prometheanfirethe guix one is good too21:11
*** e0ne has joined #openstack-release21:16
*** yamahata has joined #openstack-release21:18
tonybsmcginnis: I could do that for sure, it's just easier to get one file than several files.21:18
tonybsmcginnis: I'll code it up with sevreal and see what it looks like.  I may just be being lazy21:19
smcginnistonyb: Yeah, I'm just looking for less data duplication, but maybe that's not worth it.21:19
tbarronprometheanfire: smcginnis: so shld I push a new patch on master reverting the change that introduced the problematic requirement and21:22
tbarronprometheanfire: smcginnis then propose a new release for rocky for python-manilaclient?21:23
tbarronprometheanfire: rocky branch already exists, so push patch on master reverting and backport?21:24
tonybsmcginnis: Fair point.21:24
tonybtbarron: That'd be the process yes, master, backport to rocky re-releazse with an implied FFE21:25
tbarrontonyb: kk, thanks21:25
tonybtbarron: I can't see the error in the scrollback and I didn't notice anything when I reviewd the release, can you point me at the problem?21:25
smcginnistbarron: Looks like zuul is still in the process of being restarted, but if you get something through I will watch for an updated release.21:26
smcginnistonyb: prometheanfire said it was this - https://github.com/openstack/python-manilaclient/blob/master/requirements.txt#L2021:26
clarkbsmcginnis: I think we've got it up and queues restored now. Mostly just monitoring for any problems at this point21:26
clarkbwill give the all clear when we are happy21:26
smcginnisclarkb: Excellent, thank you.21:26
tbarronsmcginnis: tonyb prometheanfire so I'm a bit dumb here, should we have known on that review that the docutils requirements change in python-manilaclient was problematic?21:27
tbarronprometheanfire: i'll watch the video later tonight with popcorn21:27
tonybtbarron: Let me look21:29
*** tellesnobrega has quit IRC21:31
clarkbtbarron: I thinkthe concern is that the various distros out there can't all package that exact version of docutils21:32
*** tellesnobrega has joined #openstack-release21:32
tonybtbarron: Yeah we should have.  we can't pin (with == or ===) any package other than those in http://git.openstack.org/cgit/openstack/requirements/tree/blacklist.txt (well the first half the othera are 'special') and even then only intest-requirements.txt21:32
clarkbtbarron: so we want to be flexible in our requirements giving the various packagers room to package our software with their other packaged software21:32
tonybtbarron: clearly our tooling (the requiremets-check job) has a blind spot there that we'll need to fix21:32
* tonyb will check the job logs of the chnage that added it21:33
tbarronTIL be careful with requirements changes that pin ...21:33
tonybtbarron: Yeah like I said the requiremenst-check job should make it very obvious but #blindspot21:34
clarkbtonyb: it wouldn't hvae been an issue in the past when we enforced that we match global reuqirements exactly21:34
smcginnisProper way is to raise lower-constraints to be >-0.11, rigth?21:34
tonybOh requirements-check didn't run21:35
clarkblooks like docutils has no version specified on global requirements21:36
tonybsmcginnis: Yeah lower-constratints.tx docutils===0.11; requirements.txt docutils>=0.11 (only with the correct semver for the package)21:36
* smcginnis misses requirements syncing21:36
tonybclarkb: for rocky+ we don't specify minimums in g-r21:37
clarkbtonyb: ah21:38
tonybtbarron: So at somer point soon we shoudl think about adding the requirement-check job to openstack/*mailla* I *think* that would have voted -1 on that change21:38
tonybtbarron: my earlier statements about a blindspot assumed that it was running21:38
tbarrontonyb: ack21:38
*** annabelleB has quit IRC21:39
smcginnismelwitt: How are those gate failures coming along?21:39
tonybclarkb: because we can do per-project minimums, and if needed we have a tool that can generate a guess at the globally acceptable minimums21:39
clarkbtonyb: the lower constraints are effectively that lower bound now21:39
clarkbright?21:39
*** annabelleB has joined #openstack-release21:40
tbarrontonyb: smcginnis: prometheanfire: revert proposed on python-manilaclient master - https://review.openstack.org/#/c/586709/21:40
melwittsmcginnis: fix for the gate bug is 3rd in line in the gate. gate reset recently so it's going to be awhile longer21:40
melwittthen I'll be able to recheck the patches we need for the release if they need it, then we should be ok21:40
tonybclarkb: well they're in $project/requirements.txt but we can also do $project/lower-constratints.txt21:40
smcginnistbarron: I don't think you need to revert it completely, just fix the requirements spec.21:41
smcginnismelwitt: OK, thanks.21:41
clarkbmelwitt: is that the live migration bug?21:42
melwittclarkb: yes21:42
tbarronsmcginnis: I just figured we'd revisit it, but I"ll propose another parallel review that just does a docutils>=0.1121:42
smcginnistbarron: Yeah, I suppose revert for now is the most straightforward way to resolve this.21:42
clarkbtonyb: but your mins have to equal or above the mins in the global lower reqs right?21:43
smcginnistbarron: README doesn't get updated often, so I'm sure you're safe without that for awhile.21:43
clarkbtonyb: we enforce you are within the valid global range, just not that you match it exactly iirc21:43
tonybtbarron: added a big comment, +121:44
tonybclarkb: No we no longer have the concept of a global minimum, but we do do global exclusions.21:44
clarkbok that was a change on top of the original reworking dhellmann proposed right?21:45
tonybclarkb: a project is free to set a minimum to anything they like as long as 1) it isn't a globbally banned version and b) it's compatible with what's in u-c21:45
tonybclarkb: Yeah.21:45
clarkbok good I didn't completely misunderstand the original refactor then :)21:46
tonyb\o/21:46
* tonyb goes to have breakfast with the kids21:47
tonybalso checking IRC on Saturday morning is a bad plan21:47
smcginnistonyb: Yeah, get out of here! :)21:47
tonyb+1 had to do, https://twitter.com/tonybreeds/status/1022961841729937408 first ;P21:48
*** annabelleB has quit IRC21:50
smcginnistonyb: Hah!21:54
clarkbsmcginnis: seems like zuul is still happy21:55
clarkbI think it is probably safe to say all clear now21:55
smcginnisThanks clarkb21:59
*** e0ne has quit IRC22:00
tbarronclarkb: thanks for 'splainin about the hard-pinned requirments above ...22:03
smcginnisprometheanfire: Does this need to be in lower-constraints too? https://review.openstack.org/#/c/58670922:04
tbarronsmcginnis: prometheanfire it's already there22:05
smcginnistbarron: Oh! Sorry, I should have checked that part. < prometheanfire22:05
prometheanfire:D22:06
tbarronsmcginnis: well it wasn't in either change set, probably we added lower-constraints after the first one22:06
smcginnistbarron: If it got pulled in to l-c, you probably already have it installed through a dependency's dependency. But best to declare it in requirements if you know you need it directly.22:07
*** annabelleB has joined #openstack-release22:08
tbarronsmcginnis: yeah, that must be it, it went in to lower-constraints first according to git blame22:09
openstackgerritMerged openstack/releases master: Release puppet-manila 12.5.0 for stable/queens  https://review.openstack.org/58583122:10
prometheanfiresmcginnis: tbarron with a new release I'm going to abandon https://review.openstack.org/58656322:12
smcginnisprometheanfire: Makes sense.22:12
smcginnisprometheanfire: Should tbarron push that change through? Or is Monday good?22:12
tbarronsmcginnis: thanks, was just going to ask whether I should override our normal review process22:13
tbarronin the name of the global good or whether if we get the reviews and backports over the course of the weekend that would be sufficient22:13
prometheanfiresmcginnis: whenever22:14
tbarronprometheanfire: thanks, it's moving along then, I can ping reviewers etc. and get it done by normal process then22:15
prometheanfirek22:16
*** annabelleB has quit IRC22:21
*** annabelleB has joined #openstack-release22:37
*** hongbin_ has quit IRC22:42
*** annabelleB has quit IRC22:54
*** rpioso is now known as rpioso|afk23:09
*** harlowja has quit IRC23:09
tonybtbarron: I think I can supply a +2/+W on the stable backport as needed23:18
tonybtbarron: just ping me here23:18
*** gongysh has joined #openstack-release23:43

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!