Friday, 2019-10-04

*** tetsuro has joined #openstack-release00:15
*** jtomasek has quit IRC00:30
*** jtomasek has joined #openstack-release00:40
*** armax has joined #openstack-release00:48
*** tetsuro has quit IRC01:19
*** d34dh0r53 has quit IRC01:30
*** d34dh0r53 has joined #openstack-release01:32
*** ricolin has joined #openstack-release02:27
*** ykarel|away has joined #openstack-release02:36
*** ykarel|away has quit IRC04:05
*** ekcs has quit IRC04:14
*** gouthamr has quit IRC04:15
*** gouthamr has joined #openstack-release04:16
*** ykarel|away has joined #openstack-release04:25
*** ykarel|away is now known as ykarel04:31
*** ykarel is now known as ykarel|afk04:50
*** tetsuro has joined #openstack-release05:11
*** ykarel|afk is now known as ykarel06:00
*** spsurya has joined #openstack-release06:33
*** udesale has joined #openstack-release06:46
*** tetsuro has quit IRC07:09
*** rpittau|afk is now known as rpittau07:10
*** pcaruana has joined #openstack-release07:15
*** tosky has joined #openstack-release07:18
*** e0ne has joined #openstack-release07:22
*** e0ne has quit IRC07:22
*** amoralej|off is now known as amoralej07:25
*** witek has joined #openstack-release07:35
openstackgerritMerged openstack/releases master: Redirects to master should always be temporary  https://review.opendev.org/68649208:11
*** zbr is now known as zbr|ruck08:23
*** lpetrut has joined #openstack-release08:25
*** tetsuro has joined #openstack-release08:31
openstackgerritMerged openstack/releases master: Release cinder 14.0.2  https://review.opendev.org/68607108:46
openstackgerritMerged openstack/releases master: nova: release stein 19.0.3  https://review.opendev.org/68630108:51
*** e0ne has joined #openstack-release08:59
openstackgerritMerged openstack/releases master: Add stable/train for tacker-horizon  https://review.opendev.org/68616209:11
*** SergeyLukjanov has quit IRC09:44
*** SergeyLukjanov has joined #openstack-release09:51
*** mugsie has quit IRC10:03
*** mugsie has joined #openstack-release10:05
*** prometheanfire has quit IRC10:22
*** prometheanfire has joined #openstack-release10:23
*** zbr|ruck is now known as zbr|lunch11:15
*** EmilienM has quit IRC11:53
*** EmilienM has joined #openstack-release11:53
openstackgerritKota Tsuyuzaki proposed openstack/releases master: Release storlets rc2 for train  https://review.opendev.org/68657812:01
*** spsurya has quit IRC12:10
*** udesale has quit IRC12:16
openstackgerritJean-Philippe Evrard proposed openstack/releases master: Create train branch for OSA roles  https://review.opendev.org/68658112:16
*** udesale has joined #openstack-release12:16
ttxevrardjp: reporting for duty from a lounge. Let me know if you need me for anything12:19
evrardjpnot right now, I am helping OSA figuring out their branching :p12:19
evrardjpI have plenty of reviews to do but nothing seem more urgent than usual12:20
openstackgerritJean-Philippe Evrard proposed openstack/releases master: [DNM] Example patch -- Create OSA train  https://review.opendev.org/68658312:23
openstackgerritJean-Philippe Evrard proposed openstack/releases master: [DNM] Example patch -- Create OSA train  https://review.opendev.org/68658312:24
evrardjpttx: enjoy the lounge :)12:36
openstackgerritMerged openstack/releases master: Release python-troveclient 2.17.1 for stable/stein  https://review.opendev.org/68650212:46
openstackgerritThierry Carrez proposed openstack/releases master: Release storlets rc2 for train  https://review.opendev.org/68657812:48
evrardjpttx: mmm we missed some releases?12:53
*** camelCaser has quit IRC12:53
evrardjpoh cool everything is explained in the commit message!12:53
ttxInorite12:53
openstackgerritMerged openstack/releases master: Release cinder 13.0.7  https://review.opendev.org/68607012:54
openstackgerritMerged openstack/releases master: stable/rocky release oslo.messaging 8.1.4  https://review.opendev.org/68645412:54
*** camelCaser has joined #openstack-release12:55
openstackgerritMerged openstack/releases master: Release cinder 12.0.9  https://review.opendev.org/68606912:56
*** udesale has quit IRC12:56
openstackgerritHervĂ© Beraud proposed openstack/releases master: stable/train release oslo.utils 3.41.2  https://review.opendev.org/68659812:56
openstackgerritMerged openstack/releases master: stable/queens release oslo.messaging 5.35.6  https://review.opendev.org/68639512:56
EmilienMevrardjp: https://review.opendev.org/#/c/686431/12:57
EmilienMI guess you didn't see my comment on PS112:57
EmilienM"Note that 15.3.0 was missed to be released, which isn't a big deal. The metadata is now at 15.4.0, this will be our RC1."12:57
openstackgerritHervĂ© Beraud proposed openstack/releases master: stable/stein release oslo.utils 3.40.4  https://review.opendev.org/68660112:58
EmilienMthe PTL updated metadata with 15.3.0 as "m4", not sure what that meant but then pushed later 15.4.0 which is the RC112:58
EmilienMwhich is why we skip 15.3.012:58
EmilienMthanks for the reviews!12:58
evrardjpEmilienM: I saw the message afterwards12:59
evrardjpEmilienM: I even commented on the fact that I missed it, and that I was sorry12:59
EmilienMevrardjp: no prob :P12:59
EmilienMevrardjp: thx for the help12:59
evrardjpI am so used to do it my reviews this way, I am always missing those hahah12:59
*** camelCaser has quit IRC12:59
evrardjplet's see if those merge without hiccups, and then you can thank me :p13:00
openstackgerritMerged openstack/releases master: Add Glance Train highlights  https://review.opendev.org/68597913:01
openstackgerritMerged openstack/releases master: Add stable/train for karbor  https://review.opendev.org/68615613:01
openstackgerritMerged openstack/releases master: Add stable/train for karbor-dashboard  https://review.opendev.org/68615913:01
openstackgerritMerged openstack/releases master: Add stable/train for python-openstackclient  https://review.opendev.org/68615713:01
openstackgerritMerged openstack/releases master: Add stable/train branch for heat-agents  https://review.opendev.org/68597013:01
openstackgerritMerged openstack/releases master: Update common info for make-tracking-pad  https://review.opendev.org/68245213:14
openstackgerritMerged openstack/releases master: Release Puppet OpenStack Train RC1  https://review.opendev.org/68643113:14
ttxevrardjp: is there any reason to hold on gmann's QA-branching patches?13:16
ttx(we can probably wait for Monday and him removing his W-1)13:16
*** zbr|lunch is now known as zbr|ruck13:25
*** camelCaser has joined #openstack-release13:43
mordredhello release friends!13:44
*** efried has joined #openstack-release13:45
mordredwe found a bug that's a fun intersection between nova, keystoneauth and openstacksdk - and were wondering if it's still possible to get a backport of the fix into train and bump lower-constraints for nova13:46
mordredI'm assuming I'd need to request a FFE somewhere, right?13:46
mordredcc: efried and cmurphy13:47
efriedsmcginnis, evrardjp: ^13:47
mordredthe ksa and sdk patches in question are: https://review.opendev.org/#/q/topic:fix-auth-discovery13:48
*** amoralej is now known as amoralej|lunch13:50
*** mlavalle has joined #openstack-release13:56
mordredok. now updated with stable/train backport patches too13:57
*** gmann_afk is now known as gmann13:58
openstackgerritGhanshyam Mann proposed openstack/releases master: Create stable/train for devstack  https://review.opendev.org/68540014:01
ttxmordred: everything is possible14:09
mordredttx: you're possible14:09
ttxmordred: it might trigger a bunch of RCs (to pick up the new constraint beyond nova)14:09
ttxbut there is still time for that14:09
ttxAnd yes, it starts with discussing the trade-off (pain of rerolling RCs vs. severity of the bug on release day 0)14:10
ttxon a openstack-discuss thread14:10
mordredok, cool. so - what steps should I take? focus on landing the stable/train patches and cutting point releases from them first?14:10
mordredgotcha14:10
mordredefried: ^^ you might be in a better position to talk about that tradeoff - do you have time to start a discuss thread? or do you want me to try and you respond to it?14:10
ttxso (0) fix in master (1) thread (2) backports (3) re-releases14:11
ttxwe should probably branch/unfreeze requirements first14:11
ttxso that is out of the way14:11
mordredcool. I'll focus on 0 and 2 and efried and I can figure out 114:11
efriedmordred: do you see this being needed by anyone other than nova?14:12
ttxprometheanfire should be able to assess how much will need to be re-released14:12
efriedNova is getting a new RC for sure in any case14:12
efriedsorry14:13
efriedwe don't need it for nova14:13
efriedunless the authed version discovery is happening for ironic14:13
efriedwhich it must not be.14:13
mordredefried: yeah - I thnik you discovered it working on cyborg calling nova or something, right?14:13
mordredefried: if you don't need it in nova - do we even need it as a train backport?14:14
efriedmordred: I think the concern was needing it in cyborg14:16
efriedbut14:16
efriedwe discovered we could work around it in cyborg by not using strict_proxies=True (which may actually be subject to the extra-query-every-time thing you noticed, not sure)14:16
efriedand14:16
efriedthe code where cyborg talks to nova isn't even going to be used in train14:16
efriedso14:16
efriedI think we can do the backport, but after RC, because it's not critical to get it in the base train release14:16
mordredcool. that's much easier14:18
prometheanfirettx: sup14:18
mordredefried: and I think it can just roll out as a bugfix that will just cause that extra query call to go away - but that call is really low cost anyway, so it sounds non-urgent14:18
mordredprometheanfire: I think we have determined that we don't need to do anything nasty14:19
efriedmordred: I concur14:19
mordred\o.14:19
mordredlgah14:19
mordred\o/14:19
mordredsometimes raising both arms is hard14:19
prometheanfirenice14:19
* prometheanfire is gonna look to see what releases are blocking our branching today, swift branched so that's a start :D14:20
*** ricolin_ has joined #openstack-release14:25
*** ricolin has quit IRC14:26
ttxwe should now have everything :)14:28
evrardjpttx: I don't see a reason to hold on qa branching patches now that swift is branched14:31
evrardjp(sorry catching up)14:31
evrardjpmordred: oh lovely, a bug.14:31
ttxok maybe wait Monday to avoid breaking the world on a Friday14:31
ttxok, jumping on next plane now14:32
evrardjphaha yeah14:32
evrardjpok14:32
evrardjpenjoy your flight ttx14:33
evrardjpyou're in CDG ?14:33
ttxyes14:33
evrardjpand yes I agree, maybe we should branch first, so that it's easier to track this14:33
evrardjpbranch requirements*14:33
evrardjpI will read the next of the convo14:34
gmannam i missing the new format(if changed) to cut the devstack stable branch - https://review.opendev.org/#/c/685400/14:35
evrardjpoh nice feedback from efried :) I love the easier paths :p14:35
*** amoralej|lunch is now known as amoralej14:35
efriedefried for release core!14:35
efried(KIDDING!)14:36
evrardjpefried: I will remove the kidding line, and you'll get new duties, for free!14:36
evrardjphow do you like this ? :p14:36
evrardjphahah14:36
* efried should learn to keep big mouth shut14:36
evrardjpno really it was very good feedback :p14:37
evrardjpmore seriously, will nova require another RC, for different reason?14:37
evrardjpjust curious14:37
gmannsmcginnis: evrardjp do i need to update anything one this, its failing on  - https://review.opendev.org/#/c/68540014:39
evrardjpI will have a look14:40
evrardjpsmcginnis: is unavailable today14:41
evrardjplet me double check the format for tagless14:41
evrardjpand the error message :)14:42
evrardjpI can update the patch for you14:42
gmannthanks, it was same till last release14:42
gmannsure14:42
prometheanfirejust patrole is missing a train branch14:42
prometheanfirewasn't that going to get demoted or something?14:42
evrardjpgmann: the syntax is right, but I think you refer to a change id, not a sha. Let me double check14:43
prometheanfirelist-deliverables --series train --no-stable-branch --cycle-based-no-trailing --deliverables-dir deliverables14:43
gmannprometheanfire: patrole does not need branch. it is brachless like tempest14:43
evrardjp(the I makes me wonder )14:43
gmannavah14:43
evrardjpprometheanfire: patrole doesn't need branching14:43
prometheanfiregmann: kk, it was listed in the above command14:43
efriedevrardjp: sorry, yes, we will be cutting another RC14:43
openstackgerritGhanshyam Mann proposed openstack/releases master: Create stable/train for devstack  https://review.opendev.org/68540014:43
prometheanfireso maybe it's deliverable type should be changed :P14:44
evrardjpprometheanfire: yeah, I guess we should discuss this with smcginnis to change deliverable14:44
evrardjpprometheanfire: hahah yeah :P14:44
gmannevrardjp: thx, need coffee :)14:44
prometheanfireyarp14:44
evrardjpgmann: I didn't check the validation job yet so I hope it was that :)14:44
gmannprometheanfire: it was changed i think, evrardjp ?14:44
evrardjpgmann: I think patrole was stil c-w-i14:45
evrardjplet me check14:45
efriedevrardjp: at the moment we're waiting to see whether this fix will land14:45
efriedhttps://review.opendev.org/685756 SEV busted for a certain combination of flags (master: https://review.opendev.org/684825 ) bug https://bugs.launchpad.net/nova/+bug/184598614:45
efriedor whether we have to document that it's busted: https://review.opendev.org/#/c/686414/14:45
efriedBut there are at least four other changes already merged into train since RC1 that will need to be included in RC214:45
efriedIf you want to follow along: https://etherpad.openstack.org/p/nova-train-release-todo14:45
openstackLaunchpad bug 1845986 in OpenStack Compute (nova) "SEV does not enable IOMMU on SCSI controller" [High,In progress] - Assigned to Boris Bobrov (bbobrov)14:45
gmannevrardjp: but with branchless info ttx added. Tempest is also c-w-i14:45
evrardjpgmann: I guess you need the tags, right?14:46
efrieds/four/five/14:46
evrardjpgmann: I thought maybe independent could do, not really sure14:46
evrardjpefried: I see14:46
evrardjpI will keep an eye on this14:46
gmannevrardjp: for patrole ?14:46
evrardjpgmann: yeah, maybe it's a crazy idea14:47
efriedevrardjp: the SEV thing appears nontrivial, so it might come down to the wire :(14:47
evrardjpgmann: I don't know14:47
evrardjpI have to think about it further :)14:47
evrardjpefried: is Adam working on it?14:47
gmannevrardjp: we do tag but with c-w-i same as Tempest.  we have discussed it many times in past and c-w-i is not best model for them but better than other14:48
efriedevrardjp: as far as I know. He chimed in yesterday on the "document it's busted" patch.14:48
evrardjpgmann: yeah exactly "not best" but "better than the rest" :)14:48
evrardjpefried: haha good14:48
efriedI shall commence nagging him daily14:49
*** ricolin_ has quit IRC14:49
evrardjpefried: not sure if that's gonna change something, but why not :)14:49
efriedcan't hurt14:49
efriedI mean, I'm pretty sure I can take him.14:49
evrardjpefried: SEV support was announced IIRC so I suppose it's better to fix it inside release, but maybe this can be done in stable and could be done after rc time if really necessary?14:50
evrardjp(assuming other patch documenting passes?)14:50
evrardjpjust curious, didn't read the etherpad yet14:50
evrardjp:D14:50
efriedyeah, that's pretty much how it be.14:51
*** bnemec is now known as beekneemech14:54
*** armstrong has joined #openstack-release14:58
*** armstrong has quit IRC14:59
openstackgerritMatthew Thode proposed openstack/releases master: Release and branch request for requirements train  https://review.opendev.org/68403915:04
evrardjpgmann: that looks greener :)15:04
openstackgerritGhanshyam Mann proposed openstack/releases master: Add stable-branch-type as 'none' for Tempest an Patrole  https://review.opendev.org/68674315:07
gmannevrardjp: cool.  so that is ready now and next will be other QA step for release which I will follow once that is merged.15:07
gmannevrardjp:  prometheanfire : this will make clear for Patrole - https://review.opendev.org/68674315:07
evrardjpI am not sure which one should go first, requirements branching or devstack ... let me not push any of those until my brain is not farting.15:08
* evrardjp fetches a hot chocolate15:08
prometheanfireevrardjp: they shouldn't mater for ordering15:08
gmannyeah.15:09
prometheanfirefor constraints we already have a trains link (and a ussuri one too)15:10
prometheanfireinstall_command = pip install -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/train} {opts} {packages}15:11
*** ykarel is now known as ykarel|afk15:15
evrardjpthat I know, I just don't know devstack well, so I don't know why it needed to wait for some things15:17
evrardjpI suppose it was for be representative of a branch not really be a mixed model, but because I am not sure I will stop speaking and listen :)15:18
evrardjpok read a few things, got it now15:20
evrardjpyeah so I now feel confident about those. Thanks gmann and prometheanfire for updating the patches :)15:21
prometheanfireyarp15:22
evrardjpprometheanfire: your patch is incorrect15:23
evrardjpit's asking for a tag, you want me to patch it?15:23
prometheanfireya, I thought I only changed the sha15:23
prometheanfiresure15:23
evrardjpoh15:23
evrardjpmy bad then15:23
evrardjpit raises the fact that tagless were taken with my mass patch15:23
prometheanfireya, that is all I changed15:23
evrardjpyeah that's my fault then15:23
prometheanfire:D15:23
openstackgerritJean-Philippe Evrard proposed openstack/releases master: Release and branch request for requirements train  https://review.opendev.org/68403915:24
evrardjpwell the process is wrong. I will blame ttx for that :p15:24
evrardjphe is in flight so he cannot deny15:24
evrardjpprometheanfire: would https://review.opendev.org/#/c/684039/ be still the right sha?15:25
prometheanfireyour review had the wrong sha15:25
evrardjpmy pebkacs are real.15:25
prometheanfiremy review has the right one :D15:25
evrardjpdon't we have the same sha?15:26
evrardjpgod I am definitely blind15:26
prometheanfirefor openstack_releases/_redirections.py do I need to do something to get the redirect for ussuri working?15:28
prometheanfirewant me to update that sha?15:29
evrardjpI compared to patchset 215:30
evrardjpbut yeah, feel free to :)15:30
evrardjpnow that I fixed the structure, I trust you on the sha :p15:30
evrardjpfor the redirections... I have no clue. It's a gap in our process.15:30
prometheanfireoh, it looks fine, maybe I'm blind this time15:30
evrardjpif there is something to do I mean15:30
evrardjpor maybe I am blind again15:30
evrardjpprometheanfire: will make a good team.15:31
prometheanfireblind leading the blind then?15:31
prometheanfireya, it looks fine15:31
prometheanfirehttps://github.com/openstack/releases/blob/master/openstack_releases/_redirections.py#L48-L5015:33
prometheanfireI guess the question is if ussuri is in future_releases15:33
evrardjpussuri is in series_status, not sure what's future_releases yet15:34
evrardjpI can check15:34
prometheanfirehttps://github.com/openstack/releases/blob/master/doc/source/_exts/deliverables.py#L490-L49615:34
prometheanfireyarp15:34
evrardjpit is not15:35
evrardjpcause here it's development as status15:35
evrardjpso you think there is something wrong there?15:35
prometheanfirewhat else is development?15:36
prometheanfirewe'd like the link from ussuri to master to be done like the train to master one was done15:36
evrardjpI am not sure if we had that code yet. I will double check all that don't worry15:37
prometheanfiretrain should point to the train branch once we branch as well15:37
evrardjpI also see I completely broke the patch for branching15:37
prometheanfire:D15:37
prometheanfirefun time of year15:37
evrardjpI will investigate all that.15:37
evrardjphaha yeah15:37
prometheanfirethanks15:38
openstackgerritJean-Philippe Evrard proposed openstack/releases master: Release and branch request for requirements train  https://review.opendev.org/68403915:38
evrardjpprometheanfire: could you revote ^ ?15:39
evrardjpofc I was blind, and forgot a whole block of yaml15:39
prometheanfireyep15:39
evrardjpI had the thing in front of me when I patched it. It's seriously concering. Maybe I should buy new glasses.15:39
*** rpittau is now known as rpittau|afk15:45
openstackgerritKota Tsuyuzaki proposed openstack/releases master: Release storlets rc2 for train  https://review.opendev.org/68657815:51
*** ekcs has joined #openstack-release15:58
*** adriant has quit IRC15:59
*** cmurphy is now known as cmorpheus16:01
*** tosky has quit IRC16:05
*** ykarel|afk is now known as ykarel16:07
*** witek has quit IRC16:10
openstackgerritJean-Philippe Evrard proposed openstack/releases master: Make Ussuri the future series  https://review.opendev.org/68676416:19
evrardjpsmcginnis: ^ I have proposed an approach to generate ussuri constraints redirection, it seems it's not in our process. I have proposed an alternative in the patch commit message too. Tell me what you think.16:20
evrardjpsmcginnis: I have also reduced my vote on https://review.opendev.org/#/c/684039 for awareness16:22
evrardjpttx: if you could have a look at this part of the convo ^16:22
*** ykarel has quit IRC16:24
*** e0ne has quit IRC16:32
*** zbr|ruck has quit IRC16:39
*** lpetrut has quit IRC16:41
openstackgerritCarlos Goncalves proposed openstack/releases master: Releases for Octavia Queens, Rocky and Stein  https://review.opendev.org/68320216:48
*** zbr has joined #openstack-release16:51
openstackgerritGhanshyam Mann proposed openstack/releases master: Create stable/train for grenade  https://review.opendev.org/68677116:52
*** ykarel has joined #openstack-release16:53
*** lpetrut has joined #openstack-release17:15
*** mjblack has left #openstack-release17:19
openstackgerritAlfredo Moralejo proposed openstack/releases master: Release Train RC1 for Puppet modules - follow-up  https://review.opendev.org/68677817:24
*** jtomasek has quit IRC17:31
*** lpetrut has quit IRC17:49
*** efried has quit IRC17:56
*** ykarel is now known as ykarel|away17:56
*** amoralej is now known as amoralej|off17:57
*** pcaruana has quit IRC18:51
*** Sundar has joined #openstack-release18:52
*** Sundar has quit IRC18:56
openstackgerritGhanshyam Mann proposed openstack/releases master: Release Tempest 22.0.0 for Train  https://review.opendev.org/68540619:22
*** ykarel|away has quit IRC19:31
*** zbr has quit IRC19:46
*** ekcs has quit IRC19:46
*** e0ne has joined #openstack-release19:50
*** e0ne has quit IRC20:09
*** EmilienM is now known as EvilienM20:18
*** EvilienM is now known as containerizes_hi20:19
*** containerizes_hi is now known as containerized20:19
*** containerized is now known as EvilienM20:19
*** ekcs has joined #openstack-release21:07
*** prometheanfire has quit IRC21:16
*** prometheanfire has joined #openstack-release21:17
*** tosky has joined #openstack-release21:31
*** EvilienM is now known as EmilienM21:52
openstackgerritTerry Wilson proposed openstack/releases master: Release ovsdbapp 0.18.0  https://review.opendev.org/68684522:15
*** ekcs has quit IRC22:53
*** ekcs has joined #openstack-release23:19
*** ekcs has quit IRC23:29
*** tosky has quit IRC23:39

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