Tuesday, 2018-11-27

« Monday, 2018-11-26 Index Wednesday, 2018-11-28 »
*** _alastor_ has quit IRC00:02
*** wolverineav has quit IRC00:04
*** tetsuro has joined #openstack-nova00:04
*** wolverineav has joined #openstack-nova00:08
*** slaweq has joined #openstack-nova00:16
*** threestrands has joined #openstack-nova00:22
*** mriedem_away has quit IRC00:23
*** slaweq has quit IRC00:24
openstackgerritArtom Lifshitz proposed openstack/nova master: Give drop_move_claim() correct docstring  https://review.openstack.org/62017000:35
*** bhagyashris has joined #openstack-nova00:54
*** _alastor_ has joined #openstack-nova00:54
*** _alastor_ has quit IRC00:59
openstackgerritVladyslav Drok proposed openstack/nova stable/pike: [pike-only] Fix resize_instance rpcapi call  https://review.openstack.org/60343901:01
openstackgerritVladyslav Drok proposed openstack/nova stable/pike: [pike-only] Fix resize_instance rpcapi call  https://review.openstack.org/60343901:06
*** slaweq has joined #openstack-nova01:16
*** slaweq has quit IRC01:24
*** threestrands has quit IRC01:28
*** threestrands has joined #openstack-nova01:32
*** naichuans has quit IRC01:32
*** wolverin_ has joined #openstack-nova01:41
*** wolverin_ has quit IRC01:42
*** wolverineav has quit IRC01:42
*** wolverineav has joined #openstack-nova01:42
*** Dinesh_Bhor has joined #openstack-nova01:50
*** hongbin has joined #openstack-nova01:55
*** slaweq has joined #openstack-nova02:13
*** wolverineav has quit IRC02:16
*** wolverineav has joined #openstack-nova02:16
*** wolverineav has quit IRC02:16
*** k_mouza has joined #openstack-nova02:17
*** wolverineav has joined #openstack-nova02:17
*** takashin has joined #openstack-nova02:19
*** bzhao__ has joined #openstack-nova02:20
*** wolverineav has quit IRC02:21
*** mrsoul has joined #openstack-nova02:23
*** slaweq has quit IRC02:24
*** k_mouza has quit IRC02:24
*** mschuppert has quit IRC02:27
*** bzhao__ has quit IRC02:27
*** _alastor_ has joined #openstack-nova02:34
*** bzhao__ has joined #openstack-nova02:34
*** _alastor_ has quit IRC02:38
*** psachin has joined #openstack-nova02:40
*** mhen has quit IRC02:43
*** mhen has joined #openstack-nova02:47
*** whoami-rajat has joined #openstack-nova02:50
*** Dinesh_Bhor has quit IRC03:02
*** Dinesh_Bhor has joined #openstack-nova03:13
*** slaweq has joined #openstack-nova03:16
*** artom has quit IRC03:21
*** artom has joined #openstack-nova03:24
*** slaweq has quit IRC03:24
*** sridharg has joined #openstack-nova03:33
*** mvkr has quit IRC03:47
*** udesale has joined #openstack-nova03:54
*** hongbin has quit IRC04:06
*** Dinesh_Bhor has quit IRC04:10
*** slaweq has joined #openstack-nova04:11
*** jarodwl has quit IRC04:17
*** slaweq has quit IRC04:24
*** udesale has quit IRC04:31
*** Dinesh_Bhor has joined #openstack-nova04:31
*** ivve has joined #openstack-nova04:31
*** janki has joined #openstack-nova04:49
*** bhagyashris has quit IRC05:00
*** bhagyashris has joined #openstack-nova05:00
*** ratailor has joined #openstack-nova05:39
*** moshele has joined #openstack-nova05:45
openstackgerritMerged openstack/nova master: Add missing ws seperator between words  https://review.openstack.org/61849105:48
*** moshele has quit IRC06:06
*** slaweq has joined #openstack-nova06:11
*** cfriesen has quit IRC06:16
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove Placement API reference  https://review.openstack.org/61443706:21
*** slaweq has quit IRC06:24
*** ccamacho has quit IRC06:30
*** _alastor_ has joined #openstack-nova06:35
*** _alastor_ has quit IRC06:40
*** Luzi has joined #openstack-nova06:45
*** moshele has joined #openstack-nova06:46
*** dpawlik has joined #openstack-nova06:47
*** moshele has quit IRC06:51
*** adrianc has joined #openstack-nova06:52
*** dpawlik has quit IRC06:54
*** rcernin has quit IRC06:58
*** moshele has joined #openstack-nova06:59
*** mschuppert has joined #openstack-nova07:08
*** adrianc has quit IRC07:08
*** adrianc has joined #openstack-nova07:09
*** moshele has quit IRC07:18
*** dpawlik has joined #openstack-nova07:23
*** pcaruana has joined #openstack-nova07:23
*** dpawlik has quit IRC07:38
*** slaweq has joined #openstack-nova07:42
*** dpawlik has joined #openstack-nova07:44
*** threestrands has quit IRC07:44
*** ccamacho has joined #openstack-nova07:56
*** moshele has joined #openstack-nova08:02
*** sahid has joined #openstack-nova08:02
*** maciejjozefczyk has joined #openstack-nova08:07
*** hshiina has quit IRC08:10
openstackgerritYikun Jiang proposed openstack/nova master: Change the default values of XXX_allocation_ratio  https://review.openstack.org/60280308:12
*** tssurya has joined #openstack-nova08:13
*** lpetrut has joined #openstack-nova08:21
*** priteau has joined #openstack-nova08:23
*** ralonsoh has joined #openstack-nova08:29
*** k_mouza has joined #openstack-nova08:32
*** k_mouza has quit IRC08:37
*** maciejjozefczyk has quit IRC08:46
*** maciejjozefczyk has joined #openstack-nova08:47
*** tssurya has quit IRC08:49
*** cdent has joined #openstack-nova09:01
*** tssurya has joined #openstack-nova09:01
*** xek has joined #openstack-nova09:01
*** tobberydberg has joined #openstack-nova09:02
*** udesale has joined #openstack-nova09:09
*** jackivanov has joined #openstack-nova09:16
*** k_mouza has joined #openstack-nova09:20
*** ccamacho has quit IRC09:20
*** ccamacho has joined #openstack-nova09:30
*** derekh has joined #openstack-nova09:40
openstackgerritChris Dent proposed openstack/nova master: Use external placement in functional tests  https://review.openstack.org/61794109:41
openstackgerritChris Dent proposed openstack/nova master: WIP: Delete the placement code  https://review.openstack.org/61821509:41
*** xek has quit IRC09:44
*** bhagyashris has quit IRC09:44
*** xek has joined #openstack-nova09:48
*** ondrejme has joined #openstack-nova09:53
*** takashin has left #openstack-nova10:04
*** xek has quit IRC10:18
*** xek has joined #openstack-nova10:23
openstackgerritChris Dent proposed openstack/nova master: Use external placement in functional tests  https://review.openstack.org/61794110:25
openstackgerritChris Dent proposed openstack/nova master: Delete the placement code  https://review.openstack.org/61821510:25
*** erlon has joined #openstack-nova10:26
openstackgerritBrin Zhang proposed openstack/nova master: Remove useless sample and add the lack of tests in v266  https://review.openstack.org/61467110:29
*** moshele has quit IRC10:33
*** udesale has quit IRC10:33
*** udesale has joined #openstack-nova10:34
*** moshele has joined #openstack-nova10:34
*** _alastor_ has joined #openstack-nova10:36
*** udesale has quit IRC10:39
*** k_mouza has quit IRC10:41
*** _alastor_ has quit IRC10:41
*** k_mouza has joined #openstack-nova10:41
*** adrianc has quit IRC10:42
*** moshele has quit IRC10:48
*** jlvillal has joined #openstack-nova10:58
*** moshele has joined #openstack-nova10:58
*** sean-k-mooney has quit IRC11:04
*** udesale has joined #openstack-nova11:06
*** adrianc has joined #openstack-nova11:07
*** sean-k-mooney has joined #openstack-nova11:08
*** dosaboy has quit IRC11:24
*** dosaboy has joined #openstack-nova11:31
*** tbachman has quit IRC11:32
*** sambetts_ is now known as sambetts|afk11:32
*** udesale has quit IRC11:38
*** xek has quit IRC11:40
*** udesale has joined #openstack-nova11:45
*** udesale has quit IRC11:45
*** udesale has joined #openstack-nova11:46
openstackgerritMerged openstack/nova stable/rocky: Make supports_direct_io work on 4096b sector size  https://review.openstack.org/61925111:52
*** udesale has quit IRC11:52
*** udesale has joined #openstack-nova11:53
*** ratailor has quit IRC11:53
*** janki has quit IRC11:59
*** xek has joined #openstack-nova12:11
*** ccamacho has quit IRC12:12
*** tetsuro has quit IRC12:13
*** Dinesh_Bhor has quit IRC12:13
*** udesale has quit IRC12:20
openstackgerritYikun Jiang proposed openstack/nova master: Change the default values of XXX_allocation_ratio  https://review.openstack.org/60280312:29
openstackgerritYikun Jiang proposed openstack/nova master: Use new ``initial_xxx_allocation_ratio`` CONF  https://review.openstack.org/60280412:29
*** Luzi has quit IRC12:32
*** udesale has joined #openstack-nova12:32
*** udesale has quit IRC12:32
*** udesale has joined #openstack-nova12:34
*** ccamacho has joined #openstack-nova12:41
*** k_mouza has quit IRC12:45
*** k_mouza has joined #openstack-nova12:46
*** Luzi has joined #openstack-nova12:47
*** erlon has quit IRC12:48
*** udesale has quit IRC12:54
*** takamatsu has quit IRC12:58
*** erlon has joined #openstack-nova12:59
*** ccamacho has quit IRC13:04
*** ccamacho has joined #openstack-nova13:04
*** amab has joined #openstack-nova13:11
amabI've changed the default console from vnc to spice, but I get the error is this link: https://paste.ubuntu.com/p/MF6mFQBJ88/13:12
*** tbachman has joined #openstack-nova13:13
*** takamatsu has joined #openstack-nova13:21
*** k_mouza has quit IRC13:24
*** _alastor_ has joined #openstack-nova13:31
*** takamatsu has quit IRC13:37
*** k_mouza has joined #openstack-nova13:37
*** k_mouza_ has joined #openstack-nova13:38
*** moshele has quit IRC13:40
*** k_mouza has quit IRC13:42
*** takamatsu has joined #openstack-nova13:43
*** jhesketh_ has joined #openstack-nova13:44
*** whoami-rajat has quit IRC13:49
*** jhesketh has quit IRC13:50
*** hughsaunders has quit IRC13:50
mnaserhttps://review.openstack.org/#/c/619351/ stable/rocky fix merged, stable/queens up (sorry for my daily bugging alert :])13:53
*** davidsha has joined #openstack-nova13:53
*** udesale has joined #openstack-nova13:56
*** Dinesh_Bhor has joined #openstack-nova14:04
*** Dinesh_Bhor has quit IRC14:05
*** udesale has quit IRC14:05
*** udesale has joined #openstack-nova14:05
*** mriedem has joined #openstack-nova14:18
*** eharney has joined #openstack-nova14:19
*** mchlumsky has joined #openstack-nova14:23
openstackgerritMatt Riedemann proposed openstack/nova master: Default zero disk flavor to RULE_ADMIN_API in Stein  https://review.openstack.org/60391014:24
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Refactor handling of PCIe root ports  https://review.openstack.org/62032714:28
*** cdent has quit IRC14:29
*** xek has quit IRC14:30
*** tbachman has quit IRC14:31
*** _alastor_ has quit IRC14:33
dansmithmriedem: http://logs.openstack.org/04/620104/1/check/neutron-grenade/832c98f/logs/grenade.sh.txt.gz#_2018-11-26_19_52_27_04214:33
dansmithmriedem: looks reasonable I think14:33
*** xek has joined #openstack-nova14:34
mriedemso the placement api check failed as expected14:35
dansmithyep, didn't explode and "unable to discover endpoint" is reasonably clear I think14:36
*** mordred has joined #openstack-nova14:36
*** xek_ has joined #openstack-nova14:43
*** xek has quit IRC14:46
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Refactor handling of PCIe root ports  https://review.openstack.org/62032714:49
*** adrianc has quit IRC14:49
*** adrianc has joined #openstack-nova14:57
*** awaugama has joined #openstack-nova14:59
*** edmondsw has joined #openstack-nova14:59
*** awaugama has joined #openstack-nova15:01
*** mlavalle has joined #openstack-nova15:02
*** tbachman has joined #openstack-nova15:02
*** tbachman_ has joined #openstack-nova15:04
*** dpawlik has quit IRC15:04
*** tbachman has quit IRC15:07
*** tbachman_ has quit IRC15:09
*** lpetrut has quit IRC15:18
*** eharney has quit IRC15:19
mriedemis mdbooth out?15:21
kashyapmriedem: He's actually in; I was wondering why isn't he here too15:22
kashyapmriedem: BTW, while I have your attention, are things like these useful? -- https://review.openstack.org/#/c/620327/15:22
*** mdbooth has joined #openstack-nova15:22
kashyapThe original code was hurting my head, so I _had_ to spin that code out into a separate function, and simplify the crap15:22
* mdbooth waves15:23
mdboothWeird, didn't get reconnected15:23
artomkashyap, for what it's worth, I approve15:23
artomThat original 'if' is... evil15:23
kashyapartom: Thanks, at least one other person finds it useful, then.15:23
kashyapartom: Yeah, fucking awful.  I was feeling stupid having to re-read it multiple times.15:23
artomNo, I'm staring at it now and my brain is complaining already15:24
kashyapI held my nose, and cleaned up the excrements15:24
mdboothkashyap: What code's this?15:24
kashyapmdbooth: Okay, now you're Cced :D it is this: https://review.openstack.org/#/c/620327/15:24
kashyapmdbooth: BTW, mriedem was pinging you for something else, perhaps.15:24
mriedemmdbooth: fyi, some nits in https://review.openstack.org/#/c/591733/ which might be worth a follow up15:24
mdboothmriedem: Looking15:25
kashyapartom: Since you did click, please go the next step and review.  And if need be, complain if you need a test, etc15:25
artomkashyap, yep, doing  that now15:25
artom(reviewing, not complainig)15:25
kashyapThanks!15:25
mdboothmriedem: Awesome, thanks!15:26
*** eharney has joined #openstack-nova15:26
kashyapartom: That TODO item15:26
kashyap(Err hit send too soon)15:27
artomI call that PCRS15:27
artomPremature Carriage Return Syndrome15:27
*** tbachman has joined #openstack-nova15:27
kashyapartom: ... is part of a larger work item that needs cleaning up of assumptions about machine types in Nova.  And coordinating w/ relevant libvirt work15:27
kashyapartom: Heh15:28
*** tbachman has quit IRC15:35
mriedemmdbooth: what's the difference between the 'disk' and 'disk.local' entries in the block_device_info disk 'mapping' entry?15:38
mdboothmriedem: 'disk' is the root disk15:38
mdbooth'disk.local' is legacy, and no longer generated15:38
mdboothBut it used to be the single ephemeral disk15:38
mriedemunless...15:38
mdboothNow we have disk.eph0, disk.eph1, etc15:39
mriedemif we have code for it, we must support it somehow...15:39
mriedemis it for bdmv1?15:39
mriedemor just really old bdm records in the db?15:39
mdboothIt's a long time since I've checked the details on this, but I recall convincing myself in the past the current code can't generate this15:39
mdboothYeah, it's old BDM records15:39
*** Cardoe has joined #openstack-nova15:41
*** Cardoe is now known as Cardoe_15:49
*** Cardoe_ is now known as Cardoe15:49
*** dpawlik has joined #openstack-nova15:50
*** Luzi has quit IRC15:51
*** udesale has quit IRC15:56
*** udesale has joined #openstack-nova15:57
*** udesale has quit IRC15:58
*** ttsiouts has joined #openstack-nova15:58
*** dansmith has quit IRC16:02
*** dansmith has joined #openstack-nova16:02
*** munimeha1 has joined #openstack-nova16:06
mriedemmdbooth: comments inline https://review.openstack.org/#/c/578846/16:10
*** _alastor_ has joined #openstack-nova16:11
mriedemdansmith: efried: mdbooth: i'd like to move forward with getting this evacuate-related bug fixed https://review.openstack.org/#/c/606122/ - i think the open question was do we use dansmith's more generic solution https://review.openstack.org/#/c/575190/ or keep my "tactical" fix for backports and then rebase dan's on top to replace my specific fix with the generic one16:14
mriedembecause that's holding up getting us to do evacuate testing with both rbd and lvm backends https://review.openstack.org/#/c/604400/16:15
dansmithmriedem: if you'll do the rebase of mine and cleanup of this spot after we merge yours that's fine16:18
dansmithI would backport my generic solution too, but if you don't want to that's fine16:18
mdboothmriedem: ^^^ wfm.16:20
*** artom has quit IRC16:22
*** artom has joined #openstack-nova16:22
*** eharney has quit IRC16:23
mriedemdansmith: ack will do16:23
efriedI'll need to reread everything to refresh my memory of what's going on here.16:29
*** lpetrut has joined #openstack-nova16:30
efriedso wait, which are we doing? just dan's thing, master & backports; or matt's thing for backports and dan's thing for master?16:33
dansmithwe're doing whichever of those mriedem wants16:35
dansmithwhich I think is the latter16:35
mriedemthe latter16:35
efriedokay, so my job is to +2 mriedem's patch (done) and then wait for mdbooth to be satisfied with dansmith's patch before reviewing it.16:36
dansmithefried: I think the -1 on mine isn't actually a thing16:37
*** cfriesen has joined #openstack-nova16:38
efriedyeah, but I'm drowning, so looking for any excuse to defer16:39
mdboothdansmith: It's not a thing?16:41
dansmithmdbooth: the security groups thing?16:41
mdboothdansmith: Yeah. IIRC I would have been happy with a command saying: # We broke this, but we don't care because it's not a thing any more.16:42
mdbooths/comment16:42
dansmithI don't understand why it's a thing at all anyway.. if the instance is deleted, we're not going to do anything with the security groups we do fetch out of the db (if there are any anyway),16:43
dansmithand, in any case where we did load the instance with a deleted=yes context,16:43
dansmithwe'd have gotten the same16:43
dansmithso the only case where we'd do something different is where we had an instance, which wasn't deleted when we started, but was deleted underneath us and we go to lazy-load security groups.. if we were to go on and save that instance later, it'd fail and I can't think of anything else we'd do with it that would trigger some different behavior than we have now16:44
dansmithwhich I think was also mriedem's reply16:45
mriedemi was specifically worried about instance.services but mel said it was something else there16:47
mriedemi.e. the bug she was working on16:47
mdboothdansmith: I think think so. I think the bug is that if you've got an instance which isn't deleted and you lazy load security groups, you get deleted security groups. I don't think you need to delete the instance in between or anything.16:47
mdbooths/think think/don't think/16:47
mdboothdansmith: But we only care about that if we care about nova security groups, which IIUC are nova-network only.16:48
mriedemthey are nova-net only16:49
dansmithmdbooth: ah, right, I remember you clarifying this again post initial review16:50
dansmiththey're n-net only, and don't work with cellsv1, right? so we really shouldn't have any supported cases where they're there anymore16:50
mriedemso,16:51
mriedemSecurityGroup.get_by_instance calls db api security_group_get_by_instance which hard-codes read_deleted="no"16:51
mriedem*SecurityGroupList.get_by_instance16:51
mriedemso isn't the hard-coding in the DB API preventing that16:51
dansmithoh does it?16:52
* dansmith looks16:52
mdboothmriedem: Looks like I missed that. That's convenient :)16:53
dansmithsure does16:53
*** jangutter has joined #openstack-nova16:55
*** Cardoe has quit IRC16:56
sean-k-mooneyspeaking of things in nova net the function that enables trusted vfs was added to the nova net module16:57
dansmithrecorded for posterity16:57
*** Cardoe has joined #openstack-nova16:57
sean-k-mooneyit proably should be moved to an sriov plugin in os-vif16:57
sean-k-mooneyspecifically https://github.com/openstack/nova/blob/c6218428e9b29a2c52808ec7d27b4b21aadc0299/nova/network/linux_net.py#L1773 should be moved16:59
*** gyee has joined #openstack-nova16:59
sean-k-mooneymriedem: by the way i havent forgot about looking into can we remove NeutronLinuxBridgeInterfaceDriver too but i havent got around to it yet either17:00
mriedemken'ichi says it's ok https://review.openstack.org/#/c/616995/17:01
mdboothmriedem: Going back a patch, I was wondering about adding an 'intent' to driver.cleanup and/or driver.destroy, i.e. driver.destroy(reason=MIGRATION|DELETE). We currently have destroy_disks, but the problem with this is that we second guess it in ComputeManager, which makes it basically useless.17:01
*** lpetrut has quit IRC17:01
mdboothWhat we want the driver to do is, for delete: delete everything. For migrate/evacuate: delete everything local to this hypervisor.17:02
mdboothI think the definition of 'local to the hypervisor' is more cleanly defined internally by the hypervisor.17:03
slaweqhi nova cores, can You take alook at https://review.openstack.org/#/c/619061/ - it should fix neutron-tempest-postgres-full periodic job in gate so it's quite important for us :)17:03
slaweqthx in advance17:03
sean-k-mooneymriedem: oh ok cool i think there is more in that file we are not using that we could remove too17:03
* mdbooth should send an email about that instead, really.17:03
mriedemmdbooth: i don't expect you'll get much in the way of replies on that in the ML17:03
mriedempeople, like me, will likely say they don't want to think about it17:04
mdboothmriedem: It's a source of a class of real bugs, thoough17:04
mriedemmdbooth: so having said that, i assume you aren't proposing to change your current fix to incorporate that right?17:04
mriedemmdbooth: oh i don't doubt it17:04
mdboothmriedem: Hell, no17:04
mdboothmriedem: I was thinking about lyarwood's original proposal to fix an extra thing in my patch17:04
mriedemthere is definitely a russian roulette of disk deletion in compute/virt17:04
mdboothI looked harder at that, and I don't think we can fix it like that17:05
*** ttsiouts has quit IRC17:06
*** ttsiouts has joined #openstack-nova17:07
openstackgerritJack Ding proposed openstack/nova master: Improve libvirt image and snapshot handling  https://review.openstack.org/61669217:07
*** lpetrut has joined #openstack-nova17:10
*** ttsiouts has quit IRC17:11
openstackgerritJack Ding proposed openstack/nova-specs master: [WIP] Flavor Extra Spec and Image Properties Validation  https://review.openstack.org/61854217:12
*** dpawlik has quit IRC17:13
*** dpawlik has joined #openstack-nova17:14
mdboothmriedem: Practically speaking, what would be the most practical way to go about making that sort of change? Just post a patch? Problem is it wouldn't be a small amount of work, so I'd be cautious of that without some level of buy in beforehand. BP?17:14
* mdbooth notices the time and runs away17:16
*** cdent has joined #openstack-nova17:16
mriedembp doesn't give you much in the way of details, spec allows you to really write the details down and get review on it without getting too far into implementation, but idk17:17
mriedemgonna be a hard sell to get people to want to spend time/brain power on this17:17
*** dpawlik has quit IRC17:18
*** psachin has quit IRC17:21
*** derekh has quit IRC17:23
*** adrianc has quit IRC17:24
*** k_mouza has joined #openstack-nova17:29
*** Sundar has joined #openstack-nova17:29
*** dpawlik has joined #openstack-nova17:29
*** moshele has joined #openstack-nova17:30
*** sahid has quit IRC17:30
*** jmlowe has quit IRC17:32
*** k_mouza_ has quit IRC17:33
*** NostawRm has joined #openstack-nova17:33
*** jmlowe has joined #openstack-nova17:33
*** jmlowe has quit IRC17:34
*** dpawlik has quit IRC17:34
*** tbachman has joined #openstack-nova17:37
*** davidsha has quit IRC17:37
sean-k-mooneymdbooth: blueprints tend to be more for tracking then design if you want input you will want a spec or at least a etherpad + hangout if you have a quorum of people that are interested17:38
*** k_mouza has quit IRC17:38
openstackgerritMatt Riedemann proposed openstack/nova master: Fix InstanceNotFound during _destroy_evacuated_instances  https://review.openstack.org/60612217:44
openstackgerritMatt Riedemann proposed openstack/nova master: Refactor TestEvacuateDeleteServerRestartOriginalCompute  https://review.openstack.org/60870517:44
openstackgerritMatt Riedemann proposed openstack/nova master: Always read-deleted=yes on lazy-load  https://review.openstack.org/57519017:44
openstackgerritMatt Riedemann proposed openstack/nova master: Run evacuate tests with local/lvm and shared/rbd storage  https://review.openstack.org/60440017:44
mriedemdansmith: rebasearoo17:44
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Run against mriedem's evacuate test  https://review.openstack.org/60442317:45
openstackgerritArtom Lifshitz proposed openstack/nova-specs master: Re-propose numa-aware-live-migration spec  https://review.openstack.org/59958717:49
artommriedem, ^^ I think I got everything, when you have a moment17:49
mriedemok17:51
artom(Oh and the docstring patch as well)17:52
artomBut that's not as important17:52
bauzasmriedem: artom: I'll try to look at this spec tomorrow17:52
artomwoot17:52
artomCheers bauzas!17:53
*** imacdonn has quit IRC17:54
bauzasartom: before cheering me, wait for my comments :)17:55
*** imacdonn has joined #openstack-nova17:55
artomI have 100% confidence they'll be constructive and useful.17:55
artomWhy no, my nose isn't brown, whydja ask? ;)17:55
*** jmlowe has joined #openstack-nova18:05
*** bnemec has quit IRC18:06
*** bnemec has joined #openstack-nova18:06
*** sridharg has quit IRC18:09
*** moshele has quit IRC18:16
*** gary_perkins has quit IRC18:17
*** gary_perkins has joined #openstack-nova18:18
*** burt has joined #openstack-nova18:18
*** jangutter has quit IRC18:19
*** Sundar has quit IRC18:21
*** moshele has joined #openstack-nova18:23
*** wolverineav has joined #openstack-nova18:24
*** lpetrut has quit IRC18:33
*** dpawlik has joined #openstack-nova18:37
*** dpawlik has quit IRC18:42
*** adrianc has joined #openstack-nova18:48
*** tssurya has quit IRC18:50
*** dpawlik has joined #openstack-nova18:53
*** eharney has joined #openstack-nova18:54
*** dpawlik has quit IRC18:57
*** wolverin_ has joined #openstack-nova19:03
*** wolverineav has quit IRC19:05
mriedemartom: dansmith: there you go https://review.openstack.org/#/c/599587/19:06
*** ralonsoh has quit IRC19:07
*** erlon has quit IRC19:17
*** moshele has quit IRC19:24
*** hughsaunders has joined #openstack-nova19:35
*** dpawlik has joined #openstack-nova19:38
artommriedem, thank you!19:40
*** wolverin_ has quit IRC19:41
mriedembelmoreira: hey, remember talking in berlin about a problem where if you have a lot of disabled computes for an upgrade, but with max_placement_results set low, you could get novalidhost because placement returns all disabled computes? you were going to open a bug for that but i never saw one.19:41
*** wolverineav has joined #openstack-nova19:41
*** wolverineav has quit IRC19:42
mriedemi remember dansmith and i kicked around some ideas about that in berlin, but my memory is getting hazy19:42
*** wolverineav has joined #openstack-nova19:42
mriedemneed another core on this concurrent i/o disk semaphore thing in runways https://review.openstack.org/#/c/609180/19:45
mriedemit's pretty simple19:45
*** jmlowe has quit IRC19:48
*** jmlowe has joined #openstack-nova19:50
*** wolverineav has quit IRC19:53
*** adrianc has quit IRC19:59
*** _hemna has quit IRC20:04
*** Sundar has joined #openstack-nova20:08
*** lennyb_ has joined #openstack-nova20:17
*** lennyb has quit IRC20:20
cdentAre hooks still a thing that are being used in nova, or is it more that they haven't been cleared away yet?20:21
*** awaugama has quit IRC20:21
mriedemdon't build on them if you can help it20:22
mriedemrely on notifications if possible20:22
mriedemhooks are use at your own risk (we can break them and not care to fix them), and the last time someone complained about the interface breaking on one of the hooks, upstream got real serious about just yanking them20:23
mriedemkind of one of those squeeky wheel kinds of things - best not to talk about it20:23
cdentyeah, I was wondering if the yanking was still an option20:23
mriedemif we had a good reason to20:24
cdentbecause the race I'm watching just showed up with a trace from one of them, but it appears it is more likely a coincidence20:24
mriedemand someone that cared enough20:24
*** wolverineav has joined #openstack-nova20:24
* cdent would like to care but already cares too much20:25
cdentmy running theory at this point is a totally unsurprising "it's probably eventlet"20:25
mriedemno one was ever fired for blaming eventlet20:28
mriedemGET /jaypipes20:32
mriedemhttps://review.openstack.org/#/c/603352/720:32
*** wolverineav has quit IRC20:33
*** wolverineav has joined #openstack-nova20:34
mriedemi imagine jay is fighting with code using hooks within oath20:36
mriedemhooks that call off to some chef thing20:36
*** imacdonn has quit IRC20:37
*** imacdonn has joined #openstack-nova20:37
*** slaweq_ has joined #openstack-nova20:44
*** eharney has quit IRC20:45
*** jmlowe has quit IRC20:45
openstackgerritEric Fried proposed openstack/nova master: SIGHUP n-cpu to clear provider tree cache  https://review.openstack.org/61564620:49
openstackgerritEric Fried proposed openstack/nova master: Reduce calls to placement from _ensure  https://review.openstack.org/61567720:49
openstackgerritEric Fried proposed openstack/nova master: Consolidate inventory refresh  https://review.openstack.org/61569520:49
openstackgerritEric Fried proposed openstack/nova master: Commonize _update code path  https://review.openstack.org/61570520:49
openstackgerritEric Fried proposed openstack/nova master: Rip the report client out of SchedulerClient  https://review.openstack.org/61704220:49
openstackgerritEric Fried proposed openstack/nova master: Rip out the SchedulerClient  https://review.openstack.org/61704920:49
efriedblayum20:49
cdenthuzzah20:50
artomSo eventlets are the reverse IBM? No one got fired for either buying IBM, or blaming eventlets?20:54
mriedemthat was the joke yes20:59
* artom looks up to see it flying high above :(21:01
tbachman<delayed boom-splash>21:02
*** slaweq_ has quit IRC21:03
*** flwang has joined #openstack-nova21:03
flwangcould anybody please help me understand that if i can swift from v2 endpoint to v2.1 endpoint directly?21:07
flwang'directly' means change the endpoint url in keystone catalyst directly, thanks a lot21:07
mriedemflwang: https://docs.openstack.org/nova/latest/reference/stable-api.html and https://specs.openstack.org/openstack/nova-specs/specs/liberty/implemented/api-relax-validation.html might help, but i think you also have to do something with the api-paste.ini21:11
mriedemsince that is config, it will require a change on upgrade to use v2.121:11
mriedemi'm thinking about this from liberty https://github.com/openstack/nova/commit/4a18f7d3bafcdbede48500aac389e0a770b8e6a8#diff-c29e518f0f69a48dc80705f43cf9a3b421:13
flwangmriedem: thanks for reply, we have enabled it based on my testing, the v2.1 endpoint works21:14
flwangbut i'm not sure how can we migrate the catalyst in keystone21:14
flwangcatalog21:14
mriedemyou can't just make the compute endpoint point at v2.1 rather than v2?21:15
flwangbecause when you initialize a nova client, the code will talk to keystone catalog to get the endpoint21:16
flwangand currently, we're using /v2/ in the endpint21:17
flwangnot v121:17
flwangfor example, for horizon21:17
flwangit's doing that21:17
*** swamireddy has joined #openstack-nova21:17
*** swamireddy has quit IRC21:17
*** swamireddy has joined #openstack-nova21:17
flwangand as a result, there are some features we can't enable without upgrading the endpint21:17
flwangmriedem: i just checked, we have done the correct work for nova (api-paste.ini)21:20
*** imacdonn has quit IRC21:20
flwangand as I mentioned above, the endpoint is working correctly. im just not sure if we can change the keystone catalog now21:21
mriedemthe client just asks for the compute endpoint, right? which is an alias to some URL21:21
*** ivve has quit IRC21:21
mriedemthe client shouldn't care if it's v1 or v2 or v2.1 really unless they have version-specific requests21:21
flwangmriedem: yes, that's my understanding21:23
flwangi think, as admin, it should be safe to switch the endpoint, but just wanna get more ideas at here21:23
*** imacdonn has joined #openstack-nova21:24
*** _hemna has joined #openstack-nova21:24
mriedemwell it's been awhile since i've talked to any operators that went through this and the old kilo/liberty release notes are pretty sparse,21:25
mriedembut the api-paste.ini was the big thing i knew of that hit people, like when oath upgraded,21:25
mriedembut jpenick is out, and jaypipes insn't here21:25
mriedem*isn't21:25
mriedemjroll: were you around for the oath upgrade to ocata?21:25
mriedemand compute api v2.1?21:25
mriedemflwang: otherwise #openstack-operators is probably best21:26
Sundarcdent: Please ping me when you have the time. Thanks.21:27
*** skatsaounis has quit IRC21:28
*** eharney has joined #openstack-nova21:28
flwangmriedem: got it, thank you so much21:28
cdentSundar: I'll try, but currently very busy. If it's something for which email will work: cdent@anticdent.org21:29
*** wolverineav has quit IRC21:32
*** jmlowe has joined #openstack-nova21:32
*** wolverineav has joined #openstack-nova21:34
*** xek_ has quit IRC21:35
*** wolverineav has quit IRC21:38
*** wolverineav has joined #openstack-nova21:38
*** pcaruana has quit IRC21:46
*** priteau has quit IRC21:47
*** dpawlik has quit IRC21:54
*** dpawlik has joined #openstack-nova21:55
*** dpawlik has quit IRC22:00
*** slaweq has quit IRC22:03
*** slaweq has joined #openstack-nova22:19
*** slaweq has quit IRC22:24
*** wolverineav has quit IRC22:36
*** wolverineav has joined #openstack-nova22:37
*** wolverineav has quit IRC22:39
*** wolverin_ has joined #openstack-nova22:39
*** munimeha1 has quit IRC22:45
*** wolverin_ has quit IRC22:49
*** wolverineav has joined #openstack-nova22:50
*** swamireddy has quit IRC22:50
*** eharney has quit IRC22:54
*** priteau has joined #openstack-nova22:54
*** rcernin has joined #openstack-nova22:57
*** priteau has quit IRC22:58
*** wolverineav has quit IRC22:59
*** wolverineav has joined #openstack-nova23:02
*** wolverin_ has joined #openstack-nova23:12
*** wolverin_ has quit IRC23:15
*** wolverin_ has joined #openstack-nova23:15
*** wolverineav has quit IRC23:15
*** Sundar has quit IRC23:16
*** slaweq has joined #openstack-nova23:29
*** slaweq has quit IRC23:33
*** mriedem has quit IRC23:46
*** jhesketh_ is now known as jhesketh23:51
« Monday, 2018-11-26 Index Wednesday, 2018-11-28 »

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