Wednesday, 2018-08-15

openstackgerritxhzhf proposed openstack/nova master: [vmware] add missing os types in vSphere sdk 6.5  https://review.openstack.org/59177100:20
*** takashin has joined #openstack-placement00:28
*** Nel1x has joined #openstack-placement00:31
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: placement: ignore policy scope check failures if not enforcing scope  https://review.openstack.org/59187200:43
openstackgerritMerged openstack/nova master: Make Xen code py3-compatible  https://review.openstack.org/59106102:01
openstackgerritMerged openstack/nova master: Update really old comments about vmware hosts managing multiple nodes  https://review.openstack.org/58966602:01
openstackgerritMerged openstack/nova master: Ironic: report 0 for vcpus/memory_mb/disk_gb resources  https://review.openstack.org/56584102:01
*** tetsuro has joined #openstack-placement02:01
*** lei-zh has joined #openstack-placement02:09
*** openstack has joined #openstack-placement02:37
*** ChanServ sets mode: +o openstack02:37
*** lei-zh has quit IRC02:50
*** lei-zh has joined #openstack-placement02:55
*** takashin has left #openstack-placement03:00
*** Nel1x has quit IRC03:11
*** tetsuro has quit IRC03:31
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Force refresh instance info_cache during heal  https://review.openstack.org/59160703:48
*** lei-zh has quit IRC04:01
openstackgerritChen proposed openstack/nova master: Revisons on notifications doc  https://review.openstack.org/59162104:11
*** lei-zh has joined #openstack-placement05:27
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Explicitly fail if trying to attach SR-IOV port  https://review.openstack.org/59189805:40
*** e0ne has joined #openstack-placement06:16
*** mriedem has joined #openstack-placement06:19
*** mriedem has quit IRC07:05
openstackgerritMerged openstack/nova master: Remove blacklisted py3 xen tests  https://review.openstack.org/59141907:10
*** s10 has joined #openstack-placement07:11
*** s10 has quit IRC07:22
*** alex_xu has quit IRC07:50
openstackgerritBalazs Gibizer proposed openstack/nova master: consumer gen: move_allocations  https://review.openstack.org/59181008:07
openstackgerritBalazs Gibizer proposed openstack/nova master: consumer gen: more tests for delete allocation cases  https://review.openstack.org/59181108:07
openstackgerritBalazs Gibizer proposed openstack/nova master: consumer gen: support claim_resources  https://review.openstack.org/58366708:07
*** cdent has joined #openstack-placement08:14
*** tssurya has joined #openstack-placement08:35
*** alex_xu has joined #openstack-placement08:48
openstackgerritBrin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter  https://review.openstack.org/59197609:04
*** lei-zh has quit IRC10:01
openstackgerritBrin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter  https://review.openstack.org/59197610:14
openstackgerritBrin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter  https://review.openstack.org/59197610:39
*** s10 has joined #openstack-placement10:41
openstackgerritGhanshyam Mann proposed openstack/nova master: Remove the deprecated API extensions policies  https://review.openstack.org/58687211:15
openstackgerritGhanshyam Mann proposed openstack/nova master: Remove the deprecated API extensions policies  https://review.openstack.org/58687211:16
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge extended availability zone response into server controller  https://review.openstack.org/50285911:49
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge config drive extension response into server controller  https://review.openstack.org/58422312:03
jaypipesmorning gibi, cdent and efried12:24
* cdent waves12:25
jaypipesefried, gibi, cdent: just FYI, I'll be out on PTO Friday through Wednesday.12:25
cdentwise choice12:25
jaypipesno access to IRC or email12:25
jaypipesI'm being forced to socialize with the outside world.12:26
jrollyou poor soul12:27
cdentis at least an outside world that is located somewhere good?12:29
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge extended server attributes extension response  https://review.openstack.org/58459012:31
gibijaypipes: have a good time off12:46
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge keypair extension response into server view builder  https://review.openstack.org/58474812:47
jaypipesgibi: cheers mate12:50
jaypipesjroll: :P12:50
jroll:)12:50
jaypipescdent: north carolina mountains...12:50
cdentah, nice. there are _far_ worse places in the world. Enjoy.12:51
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge server usage extension response into server view builder  https://review.openstack.org/58526213:00
jaypipesefried: I'm fixing up the nit that tetsuro noted on 590041.13:11
openstackgerritJay Pipes proposed openstack/nova master: [placement] split gigantor SQL query, add logging  https://review.openstack.org/59004113:14
jaypipesefried: et voila ^13:14
jaypipesalex_xu: ^ fixed up that one nit13:14
alex_xujaypipes: i guess not hurry to +2, since efried already sent email to ask feedback13:15
jaypipesalex_xu: you were already +2 on that I think :)13:16
alex_xuyea...13:16
jaypipesno worries though13:16
alex_xudone \o/13:17
efriedIf we don't hear anything back in a day or two, I say merge it. We can always improve on it later.13:17
efriedjaypipes: Did you run that locally? I'm wondering if you need to do list(traits.keys()) else you'd wind up with output like <Generator at 0x0a7e0a> etc.13:23
jaypipesefried: ah, in py3...13:24
jaypipesefried: yeah, passed locally on py2713:24
jaypipesefried: lemme change.13:24
efriedwell, it'll pass either way.13:24
efriedjaypipes: If you're changing, list(required_traits) should work - no need to .list() unless you're just feeling explicit.13:25
openstackgerritJay Pipes proposed openstack/nova master: [placement] split gigantor SQL query, add logging  https://review.openstack.org/59004113:25
jaypipesefried: meh?13:26
openstackgerritJay Pipes proposed openstack/nova master: [placement] split gigantor SQL query, add logging  https://review.openstack.org/59004113:29
efriedhm, actually, oslo_log seems to have thought about it. It was printing dict_keys([actual list]) instead of <generator xxxx>13:29
efriedlist() is still better, but good to know.13:29
openstackgerritMerged openstack/nova master: api-ref: fix GET /flavors?is_public description  https://review.openstack.org/58809213:33
efriedalex_xu: it's ready again.13:36
openstackgerritGaudenz Steinlin proposed openstack/nova master: Ignore some updates from virt driver  https://review.openstack.org/52300613:43
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge security groups extension response into server view builder  https://review.openstack.org/58547513:45
openstackgerritEric Fried proposed openstack/nova master: [placement] Regex consts for placement schema  https://review.openstack.org/59186313:48
openstackgerritMatthew Booth proposed openstack/nova master: Add regression test for bug 1550919  https://review.openstack.org/59173313:52
openstackbug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)13:52
*** alex_xu has quit IRC13:52
openstackgerritMatthew Booth proposed openstack/nova master: Py3 fix in fake image service  https://review.openstack.org/59205613:54
openstackgerritxhzhf proposed openstack/nova master: VMware: add missing os types in vSphere sdk 6.5  https://review.openstack.org/59177113:59
openstackgerritGhanshyam Mann proposed openstack/nova master: Remove the deprecated API extensions policies  https://review.openstack.org/58687214:07
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge extended availability zone response into server controller  https://review.openstack.org/50285914:08
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge config drive extension response into server controller  https://review.openstack.org/58422314:09
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge extended server attributes extension response  https://review.openstack.org/58459014:09
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge keypair extension response into server view builder  https://review.openstack.org/58474814:09
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge server usage extension response into server view builder  https://review.openstack.org/58526214:10
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge security groups extension response into server view builder  https://review.openstack.org/58547514:21
cdentefried: I think maybe you interrupted an already running recheck of the perf thing?14:21
efriedI checked the timestamps, but I could have effed it up.14:22
cdentit's been in the gate for an hour and 17 minutes already14:22
cdentand it's still there, so I guess we are potentially safe14:22
efriedsorry about that14:25
cdentno harm done14:25
cdentTIL: It's weird that the server is interposing a stylesheet on a .txt doc at all.14:26
cdentit is weird indeed14:26
openstackgerritDan Smith proposed openstack/nova stable/queens: Wait for network-vif-plugged before starting live migration  https://review.openstack.org/59173514:27
openstackgerritDan Smith proposed openstack/nova stable/queens: DNM: Debug patch to test live migration waiting  https://review.openstack.org/59177514:27
openstackgerritMatthew Booth proposed openstack/nova master: Add regression test for bug 1550919  https://review.openstack.org/59173314:30
openstackgerritMatthew Booth proposed openstack/nova master: Py3 fix in fake image service  https://review.openstack.org/59205614:30
openstackgerritMatthew Booth proposed openstack/nova master: Don't delete disks on shared storage during evacuate  https://review.openstack.org/57884614:30
openstackbug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)14:30
openstackgerritMatthew Booth proposed openstack/nova master: Don't delete disks on shared storage during evacuate  https://review.openstack.org/57884614:34
openstackgerritEric Fried proposed openstack/nova master: [placement] Add /reshaper handler for POST  https://review.openstack.org/57692714:47
openstackgerritEric Fried proposed openstack/nova master: reshaper: Look up provider if not in inventories  https://review.openstack.org/58503314:47
openstackgerritEric Fried proposed openstack/nova master: Make get_allocations_for_resource_provider sane  https://review.openstack.org/58459814:47
openstackgerritEric Fried proposed openstack/nova master: Report client: Real get_allocs_for_consumer  https://review.openstack.org/58459914:47
openstackgerritEric Fried proposed openstack/nova master: Report client: get_allocations_for_provider_tree  https://review.openstack.org/58464814:47
openstackgerritEric Fried proposed openstack/nova master: Report client: _reshape helper, placement min bump  https://review.openstack.org/58503414:47
openstackgerritEric Fried proposed openstack/nova master: Report client: update_from_provider_tree w/reshape  https://review.openstack.org/58504914:47
openstackgerritEric Fried proposed openstack/nova master: Compute: Handle reshaped provider trees  https://review.openstack.org/57623614:47
openstackgerritEric Fried proposed openstack/nova master: [placement] Regex consts for placement schema  https://review.openstack.org/59186314:47
efriedRebase for merge conflict ^14:47
efriedgibi, jaypipes: Please re+2 https://review.openstack.org/#/c/584598/ (where the manual part of that rebase needed to happen ^)14:51
jaypipesefried: is there a reason we still have a -2 on the bottom of that series?14:53
efriedjaypipes: To wait until the whole pile has been reviewed, in case we find something at the top that requires a change in the API patch on bottom.14:54
efried...so we don't burn a microversion14:54
gibiefried: done14:57
efriedthx14:57
openstackgerritMatthew Booth proposed openstack/nova master: Add regression test for bug 1550919  https://review.openstack.org/59173314:58
openstackgerritMatthew Booth proposed openstack/nova master: Py3 fix in fake image service  https://review.openstack.org/59205614:58
openstackgerritMatthew Booth proposed openstack/nova master: Don't delete disks on shared storage during evacuate  https://review.openstack.org/57884614:58
openstackbug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)14:58
*** cdent_ has joined #openstack-placement15:15
jaypipescdent_: welcome back. :P15:15
cdent_my internets assploded15:15
*** cdent has quit IRC15:15
*** cdent_ is now known as cdent15:15
openstackgerritDan Smith proposed openstack/nova stable/queens: Wait for network-vif-plugged before starting live migration  https://review.openstack.org/59173515:25
openstackgerritDan Smith proposed openstack/nova stable/queens: DNM: Debug patch to test live migration waiting  https://review.openstack.org/59177515:26
openstackgerritDan Smith proposed openstack/nova stable/queens: Fix cancel_all_events event name parsing  https://review.openstack.org/59208615:26
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge extended_status extension response into server view builder  https://review.openstack.org/59209215:35
cdentjaypipes, efried : shall I address the -1 from jay on the bottom of the reshaper stack, or efried would you like to get it later when revisiting the whole stack (in which there are some new suggestions)15:44
openstackgerritChen proposed openstack/nova master: Revisons on notifications doc  https://review.openstack.org/59162115:50
jaypipescdent: hold up pls. still reviewing rest of series15:54
jaypipescdent: I've got a few more nits that may as well be addressed at same time15:55
cdentjaypipes: yeah, that's basically why I was asking, knew you were in there too15:55
cdentmy today is going to expire pretty soon, so I'll let efried (or you) have the whole ball15:55
efriedjaypipes, cdent: I was working on putting one or more nitbuster patches on top of the series, but if we need to rework one or more of them in-series, I can do that.16:03
efried(The UUID regex refactor was the first of those)16:03
cdenti have no preference about where the fixes go16:04
openstackgerritDan Smith proposed openstack/nova stable/queens: Wait for network-vif-plugged before starting live migration  https://review.openstack.org/59173516:04
openstackgerritDan Smith proposed openstack/nova stable/queens: DNM: Debug patch to test live migration waiting  https://review.openstack.org/59177516:04
* cdent comes back later16:14
*** cdent has quit IRC16:14
*** efried is now known as efried_rollin16:28
openstackgerritDoug Hellmann proposed openstack/nova master: use static pages for mitaka and newton release notes  https://review.openstack.org/59215016:41
*** e0ne has quit IRC16:47
*** openstackstatus has joined #openstack-placement16:57
*** ChanServ sets mode: +v openstackstatus16:57
*** s10 has quit IRC17:11
openstackgerritMerged openstack/nova-specs master: Create specs directory for Stein  https://review.openstack.org/57360217:27
*** tssurya has quit IRC17:28
openstackgerritMerged openstack/nova stable/queens: Revert "libvirt: slow live-migration to ensure network is ready"  https://review.openstack.org/59176117:32
openstackgerritDmitry Sutyagin proposed openstack/nova master: libvirt: add disabling KSM via flavor extra specs  https://review.openstack.org/59185417:40
*** e0ne has joined #openstack-placement17:56
*** openstackgerrit has quit IRC19:19
*** efried_rollin is now known as efried19:53
*** e0ne has quit IRC20:03
*** openstackgerrit has joined #openstack-placement20:50
openstackgerritDoug Hellmann proposed openstack/nova master: use static pages for mitaka and newton release notes  https://review.openstack.org/59215020:50
openstackgerritDmitry Sutyagin proposed openstack/nova master: libvirt: add disabling KSM via flavor extra specs  https://review.openstack.org/59185420:54
openstackgerritMerged openstack/nova master: Add placement perf info gathering hook to end of nova-next  https://review.openstack.org/59136721:52
openstackgerritKen'ichi Ohmichi proposed openstack/nova master: Merge extended availability zone response into server controller  https://review.openstack.org/50285922:25
openstackgerritDmitry Sutyagin proposed openstack/nova master: libvirt: add disabling KSM via flavor extra specs  https://review.openstack.org/59185423:08
openstackgerritMerged openstack/nova stable/pike: Revert "libvirt: slow live-migration to ensure network is ready"  https://review.openstack.org/59176223:16
*** takashin has joined #openstack-placement23:53

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