Friday, 2017-12-08

*** gmann_afk is now known as gmann00:01
*** threestrands has joined #openstack-nova00:01
*** hemna_ has quit IRC00:02
*** yamamoto has quit IRC00:03
*** itlinux has joined #openstack-nova00:03
gmannmriedem: dansmith please check these stable branch patches for zuulv3 in novaclient - https://review.openstack.org/#/q/I43a8435485751748ca6228f67d401945cb32652e00:04
gmannproject config patch is waiting for those to merge - https://review.openstack.org/#/c/522101/00:04
*** liverpooler has joined #openstack-nova00:04
*** hongbin has quit IRC00:08
openstackgerritArmando Migliaccio proposed openstack/nova stable/pike: VMware: Factor out relocate_vm()  https://review.openstack.org/52654900:08
*** liverpooler has quit IRC00:09
*** chyka has quit IRC00:11
*** baoli has quit IRC00:13
*** slaweq has joined #openstack-nova00:22
*** rcernin has quit IRC00:25
*** rcernin has joined #openstack-nova00:25
*** gouthamr has quit IRC00:25
*** hshiina has joined #openstack-nova00:26
*** slaweq has quit IRC00:26
*** sdague has quit IRC00:29
*** mlavalle has quit IRC00:36
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3)  https://review.openstack.org/52655700:36
openstackgerritSam Morrison proposed openstack/nova master: Allow ability for non admin users to use all filters on server list.  https://review.openstack.org/52655800:41
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352600:46
*** slaweq has joined #openstack-nova00:57
*** huanxie has joined #openstack-nova00:59
*** yamamoto has joined #openstack-nova00:59
*** gyee has quit IRC01:01
*** slaweq has quit IRC01:02
*** phuongnh has joined #openstack-nova01:04
*** yamamoto has quit IRC01:04
*** jichen has joined #openstack-nova01:08
*** yangyapeng has joined #openstack-nova01:09
*** esberglu has joined #openstack-nova01:16
openstackgerritTetsuro Nakamura proposed openstack/nova master: Add aggregates check in allocation candidates  https://review.openstack.org/52240701:17
openstackgerritTetsuro Nakamura proposed openstack/nova master: Add functional test for get_all_with_shared()  https://review.openstack.org/52240801:17
openstackgerritTetsuro Nakamura proposed openstack/nova master: Fix missing rps in allocation candidates  https://review.openstack.org/52240901:17
openstackgerritTetsuro Nakamura proposed openstack/nova master: placement: func tests for multiple shared RPs  https://review.openstack.org/49873701:17
*** david-lyle has joined #openstack-nova01:18
*** esberglu has quit IRC01:21
*** mikal has quit IRC01:24
*** Dinesh_Bhor has joined #openstack-nova01:28
*** mikal has joined #openstack-nova01:30
*** slaweq has joined #openstack-nova01:32
*** Apoorva_ has joined #openstack-nova01:33
*** fragatina has quit IRC01:33
*** Apoorva has quit IRC01:36
*** david-lyle has quit IRC01:36
*** slaweq has quit IRC01:37
*** Apoorva_ has quit IRC01:38
*** fragatina has joined #openstack-nova01:46
*** fragatina has quit IRC01:51
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add instance action db and obj pagination support.  https://review.openstack.org/52642201:52
*** yamamoto has joined #openstack-nova02:00
*** harlowja has quit IRC02:03
*** yamamoto has quit IRC02:04
*** hongbin has joined #openstack-nova02:05
*** slaweq has joined #openstack-nova02:12
*** slaweq has quit IRC02:17
openstackgerritwanghongxu proposed openstack/nova master: Update cinder info when attach volume that attached  https://review.openstack.org/52554002:17
*** annp has joined #openstack-nova02:26
openstackgerritJiao Pengju proposed openstack/python-novaclient master: Fix listing available images example error in nova.rst  https://review.openstack.org/52656902:27
*** dgonzalez_ has joined #openstack-nova02:27
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632602:28
*** dgonzalez_ has quit IRC02:29
*** namnh has joined #openstack-nova02:38
*** zhurong has joined #openstack-nova02:47
*** slaweq has joined #openstack-nova02:48
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Document Queens review priorities  https://review.openstack.org/52657302:49
*** slaweq has quit IRC02:52
*** fragatina has joined #openstack-nova03:01
*** yamamoto has joined #openstack-nova03:01
*** yamamoto has quit IRC03:06
*** bbbbzhao_ has joined #openstack-nova03:14
*** slaweq has joined #openstack-nova03:19
*** fragatina has quit IRC03:19
*** tidwellr has joined #openstack-nova03:19
*** abhishekk has joined #openstack-nova03:24
*** slaweq has quit IRC03:24
*** slaweq has joined #openstack-nova03:25
*** Dinesh_Bhor has quit IRC03:25
*** dave-mccowan has quit IRC03:26
*** slaweq has quit IRC03:30
openstackgerritMatt Riedemann proposed openstack/nova master: doc: link in some Sydney summit content  https://review.openstack.org/52658703:30
*** Apoorva has joined #openstack-nova03:33
*** mriedem has quit IRC03:36
*** links has joined #openstack-nova03:42
*** coreywright has quit IRC03:53
*** udesale has joined #openstack-nova03:54
*** slaweq has joined #openstack-nova03:57
*** BryanS68 has joined #openstack-nova04:01
*** slaweq has quit IRC04:02
*** yamamoto has joined #openstack-nova04:03
*** Dinesh_Bhor has joined #openstack-nova04:06
*** yamamoto has quit IRC04:06
*** armax has quit IRC04:06
*** yamamoto has joined #openstack-nova04:06
*** armax has joined #openstack-nova04:07
*** armax has quit IRC04:07
*** coreywright has joined #openstack-nova04:10
*** trungnv has quit IRC04:10
*** TuanLA has joined #openstack-nova04:13
*** chyka has joined #openstack-nova04:13
openstackgerritChen Hanxiao proposed openstack/nova master: trivial: more suitable log in set_admin_password  https://review.openstack.org/52659204:13
*** gbarros has quit IRC04:17
*** chyka has quit IRC04:19
*** Dinesh_Bhor has quit IRC04:19
*** Dinesh_Bhor has joined #openstack-nova04:19
*** andreas_s has joined #openstack-nova04:22
*** zhurong has quit IRC04:26
*** andreas_s has quit IRC04:27
*** huanxie has quit IRC04:32
*** gbarros has joined #openstack-nova04:32
*** adisky_ has joined #openstack-nova04:34
*** slaweq has joined #openstack-nova04:36
*** gbarros has quit IRC04:39
*** slaweq has quit IRC04:41
*** hieulq has quit IRC04:43
*** hieulq has joined #openstack-nova04:44
*** adreznec has joined #openstack-nova04:54
*** adreznec has quit IRC04:55
*** karthiks has joined #openstack-nova04:56
*** adreznec has joined #openstack-nova04:57
*** diga has joined #openstack-nova04:59
*** sree has joined #openstack-nova05:00
*** tidwellr has quit IRC05:02
*** huanxie has joined #openstack-nova05:02
*** BryanS68 has quit IRC05:06
*** slaweq has joined #openstack-nova05:07
*** Apoorva has quit IRC05:09
*** rcernin has quit IRC05:11
*** slaweq has quit IRC05:11
*** BryanS68 has joined #openstack-nova05:12
*** slaweq_ has joined #openstack-nova05:13
*** ratailor has joined #openstack-nova05:13
*** Apoorva has joined #openstack-nova05:13
*** Apoorva has quit IRC05:15
*** slaweq_ has quit IRC05:18
*** yikun_ has quit IRC05:22
*** Lingwu has joined #openstack-nova05:24
*** threestrands has quit IRC05:24
*** takashin has left #openstack-nova05:30
*** harlowja has joined #openstack-nova05:37
*** hongbin has quit IRC05:38
*** threestrands has joined #openstack-nova05:39
*** threestrands has quit IRC05:39
*** threestrands has joined #openstack-nova05:39
*** BryanS68 has quit IRC05:39
*** slaweq has joined #openstack-nova05:40
*** zhurong has joined #openstack-nova05:40
*** slaweq has quit IRC05:45
*** armax has joined #openstack-nova05:45
*** fragatina has joined #openstack-nova05:52
*** armax has quit IRC05:55
*** armax has joined #openstack-nova05:55
*** armax has quit IRC05:55
*** armax has joined #openstack-nova05:56
*** armax has quit IRC05:56
*** fragatina has quit IRC05:57
*** mikal has quit IRC05:57
*** mikal has joined #openstack-nova05:59
*** janki has joined #openstack-nova05:59
*** rcernin has joined #openstack-nova06:11
*** sree has quit IRC06:12
*** sree has joined #openstack-nova06:13
*** harlowja has quit IRC06:14
*** slaweq has joined #openstack-nova06:16
*** sree has quit IRC06:17
*** _heena_py_ has joined #openstack-nova06:19
_heena_py_Hi team, could anyone please help me with command where I have the server name and I need to know the floating IP06:20
*** slaweq has quit IRC06:21
*** xinliang has quit IRC06:22
*** diga has quit IRC06:24
openstackgerritjichenjc proposed openstack/nova master: deprecate fping_path config option  https://review.openstack.org/52660206:26
*** threestrands has quit IRC06:28
*** fragatina has joined #openstack-nova06:29
*** fragatina has quit IRC06:30
*** fragatina has joined #openstack-nova06:30
*** xinliang has joined #openstack-nova06:35
*** hui has joined #openstack-nova06:36
*** Shunli has joined #openstack-nova06:39
masber_heena_py_, jut run ¨openstack server list¨ and it will show your server and the floating IP06:43
*** sree has joined #openstack-nova06:43
masbergood afternoon, I have a host which is misbehaving. How could I tell nova not to schedule new vms into that host? I tried disabling the nova service but didnt work06:46
masberany idea?06:46
*** zhurong has quit IRC06:47
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632606:47
*** vivsoni__ has quit IRC06:51
*** slaweq has joined #openstack-nova06:55
*** andreas_s has joined #openstack-nova06:57
*** sree has quit IRC06:57
*** sree has joined #openstack-nova06:58
*** slaweq has quit IRC07:00
*** sree has quit IRC07:03
-openstackstatus- NOTICE: Due to some unforseen Zuul issues the gate is under very high load and extremely unstable at the moment. This is likely to persist until PST morning07:03
*** ChanServ changes topic to "Due to some unforseen Zuul issues the gate is under very high load and extremely unstable at the moment. This is likely to persist until PST morning"07:03
*** serverascode has quit IRC07:07
*** serverascode has joined #openstack-nova07:07
*** kencjohnston has quit IRC07:08
*** melwitt has quit IRC07:08
*** sree has joined #openstack-nova07:10
*** melwitt has joined #openstack-nova07:10
*** melwitt is now known as Guest905407:11
*** kencjohnston has joined #openstack-nova07:13
*** bbbbzhao_ has quit IRC07:14
*** sree has quit IRC07:17
*** john51 has quit IRC07:17
*** john51 has joined #openstack-nova07:17
*** sree has joined #openstack-nova07:17
*** jamesdenton has quit IRC07:18
*** sree has quit IRC07:25
*** mdnadeem has joined #openstack-nova07:25
*** phuongnh has quit IRC07:27
*** moshele has joined #openstack-nova07:27
*** swamireddy has quit IRC07:30
*** huanxie has quit IRC07:31
*** slaweq has joined #openstack-nova07:34
*** slaweq has quit IRC07:39
*** sahid has joined #openstack-nova07:44
*** swamireddy has joined #openstack-nova07:46
*** slaweq has joined #openstack-nova07:46
*** sree has joined #openstack-nova07:47
*** slaweq has quit IRC07:50
*** slaweq has joined #openstack-nova07:50
*** slaweq has quit IRC07:51
*** david-lyle has joined #openstack-nova07:52
*** slaweq has joined #openstack-nova07:53
openstackgerritEric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI  https://review.openstack.org/52609408:06
*** slaweq has quit IRC08:12
*** moshele has quit IRC08:15
*** zhurong has joined #openstack-nova08:16
*** huanxie has joined #openstack-nova08:16
*** tesseract has joined #openstack-nova08:20
openstackgerritEli Qiao proposed openstack/nova master: Api-guide: Add Block Device Mapping  https://review.openstack.org/52208408:31
*** sapd__ has joined #openstack-nova08:31
*** sapd_ has quit IRC08:31
*** adreznec has quit IRC08:32
*** adreznec has joined #openstack-nova08:32
*** vivsoni has joined #openstack-nova08:34
*** damien_r has joined #openstack-nova08:41
*** qsyqian has joined #openstack-nova08:47
*** hshiina has quit IRC08:47
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances  https://review.openstack.org/52367608:48
*** lucas-afk is now known as lucasagomes08:57
*** gszasz has joined #openstack-nova09:00
*** itlinux has quit IRC09:01
*** hoangcx has quit IRC09:05
*** owalsh has quit IRC09:10
*** andreas_s has quit IRC09:11
*** sapd__ has quit IRC09:11
*** owalsh has joined #openstack-nova09:12
*** andreas_s has joined #openstack-nova09:12
*** sapd has joined #openstack-nova09:12
*** andreas_s has quit IRC09:17
*** alexchadin has joined #openstack-nova09:17
*** karthiks is now known as karthiks_afk09:20
*** sapd_ has joined #openstack-nova09:22
*** sapd has quit IRC09:22
*** moshele has joined #openstack-nova09:22
*** andreas_s has joined #openstack-nova09:23
*** phuongnh has joined #openstack-nova09:24
*** andreas_s has quit IRC09:28
*** Shunli has quit IRC09:29
*** andreas_s has joined #openstack-nova09:37
*** Dinesh_Bhor has quit IRC09:37
*** yamamoto has quit IRC09:41
*** andreas_s has quit IRC09:42
*** yamamoto has joined #openstack-nova09:43
*** moshele has quit IRC09:46
*** gmann is now known as gmann_afk09:48
*** andreas_s has joined #openstack-nova09:51
*** andreas_s has quit IRC09:56
*** Alex_Staf has joined #openstack-nova09:56
*** andreas_s has joined #openstack-nova10:03
openstackgerritBalazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407  https://review.openstack.org/52609510:05
openstackbug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer)10:05
*** annp has quit IRC10:07
*** brault has quit IRC10:11
*** brault has joined #openstack-nova10:11
*** gszasz has quit IRC10:12
*** brault has quit IRC10:12
*** andreas_s has quit IRC10:12
*** dtantsur|afk is now known as dtantsur10:16
*** Alex_Staf has quit IRC10:16
*** pooja-jadhav is now known as pooja_jadhav10:19
*** namnh has quit IRC10:19
openstackgerritJiao Pengju proposed openstack/python-novaclient master: Fix listing available images example error in nova.rst  https://review.openstack.org/52656910:19
*** andreas_s has joined #openstack-nova10:19
*** ociuhandu has joined #openstack-nova10:20
openstackgerritTheodoros Tsioutsias proposed openstack/python-novaclient master: CommandError is raised for invalid server fields  https://review.openstack.org/52511010:21
*** andreas_s has quit IRC10:24
*** sdague has joined #openstack-nova10:28
*** gszasz has joined #openstack-nova10:31
*** jichen has quit IRC10:33
*** andreas_s has joined #openstack-nova10:33
*** phuongnh has quit IRC10:39
*** ociuhandu has quit IRC10:40
*** zhurong has quit IRC10:40
*** derekh has joined #openstack-nova10:40
*** danpawlik has quit IRC10:42
*** danpawlik has joined #openstack-nova10:43
*** hoonetorg has quit IRC10:44
*** abhishekk has quit IRC10:45
*** gszasz has quit IRC10:45
*** hoonetorg has joined #openstack-nova10:45
*** andreas_s has quit IRC10:48
*** andreas_s has joined #openstack-nova10:48
*** gibi is now known as giblet10:57
*** andreas_s has quit IRC10:57
*** andreas_s has joined #openstack-nova10:58
*** danpawlik has quit IRC10:59
*** danpawlik has joined #openstack-nova11:00
*** danpawlik has quit IRC11:01
*** danpawlik has joined #openstack-nova11:03
*** qsyqian has quit IRC11:05
*** danpawlik has quit IRC11:06
*** karthiks_afk is now known as karthiks11:08
*** danpawlik has joined #openstack-nova11:10
*** qsyqian has joined #openstack-nova11:17
*** alexchadin has quit IRC11:17
*** openstackgerrit has quit IRC11:17
*** yamamoto has quit IRC11:21
*** sree has quit IRC11:25
*** chyka has joined #openstack-nova11:26
*** links has quit IRC11:28
*** claudiub has joined #openstack-nova11:28
*** udesale has quit IRC11:29
*** qsyqian has quit IRC11:30
*** chyka has quit IRC11:30
*** brault has joined #openstack-nova11:38
*** artom_ has joined #openstack-nova11:38
*** artom_ has quit IRC11:39
*** artom_ has joined #openstack-nova11:39
*** artom has quit IRC11:41
*** links has joined #openstack-nova11:41
*** liuzz has quit IRC11:44
*** dave-mccowan has joined #openstack-nova11:45
*** gszasz has joined #openstack-nova11:47
*** yamamoto has joined #openstack-nova11:49
*** tesseract has quit IRC11:50
*** tesseract has joined #openstack-nova11:51
*** tbachman has quit IRC11:52
*** adisky_ has quit IRC11:53
*** cdent has joined #openstack-nova11:59
*** kalyan has quit IRC12:02
*** andreas_s has quit IRC12:04
*** andreas_s has joined #openstack-nova12:05
*** andreas_s has quit IRC12:05
*** andreas_s has joined #openstack-nova12:05
*** artom_ has quit IRC12:06
gibletwhat a slow day12:07
cdentgah! now you’ve done it12:07
cdentthe quiet peace will now be broken12:07
*** janki has quit IRC12:07
cdentIs the gate still unhappy12:08
gibletyeah the gate contributes a lot to the slowness of the day12:09
*** claudiub has quit IRC12:14
*** andreas_s has quit IRC12:17
*** andreas_s has joined #openstack-nova12:17
*** yangyapeng has quit IRC12:24
*** TuanLA has quit IRC12:26
*** andreas_s has quit IRC12:27
*** efried is now known as fried_rice12:29
fried_riceMorning folks.12:29
fried_riceWow, still nothing merged since last night.12:30
*** openstackgerrit has joined #openstack-nova12:30
openstackgerritMerged openstack/nova master: XenAPI: update the picture in Xen hypervisor document  https://review.openstack.org/52336012:30
gibletfried_rice: welcome in the slow Friday land :)12:30
cdentfried_rice: gate has issues that likely won’t be resolved until morning time pst12:30
*** yikun_ has joined #openstack-nova12:31
cdentdansmith: when you come on scene can you help decide if the cells-related issue in https://bugs.launchpad.net/nova/+bug/1736101 ought to be considered a proper bug (comments welcome from everyone else too)12:31
openstackLaunchpad bug 1736101 in OpenStack Compute (nova) "nova placement resource_providers DBDuplicateEntry when name repeat" [Undecided,New]12:31
*** ratailor has quit IRC12:32
fried_riceJust caught up on -infra, looks like they might have sorted it out around 3h ago.  Now we just gotta wait for the backlog to push through.12:36
fried_riceSokay, I only had 17 patches in queue.12:39
cdentclearly this is all your fault. you gotta baby the queue12:40
*** andreas_s has joined #openstack-nova12:42
*** salv-orlando has joined #openstack-nova12:42
gibletI hope by rechecking all the bounced patches we will DDoS the whole gate down again ;]12:45
fried_riceI'm deliberately holding off on 10 of them.12:45
*** pchavva has joined #openstack-nova12:46
openstackgerritBalazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407  https://review.openstack.org/52609512:46
openstackbug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer)12:46
openstackgerritBalazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild  https://review.openstack.org/52524212:46
gibletoops, two more :)12:46
fried_ricecdent I've got about 4-4.5h in me today, then on vacation.  Were you serious in your offer to babysit my code while I'm out?12:47
*** andreas__ has joined #openstack-nova12:48
cdentfried_rice: a) yes (in part because I want to understand it better ), b) I thought you were supposed to already be gone12:48
sean-k-m2giblet: it wont ddos the gate. zuul will just enqueue them and run them when nodepool says there is an env availably to it12:48
*** sean-k-m2 is now known as sean-k-mooney12:49
gibletsean-k-mooney: what a relief12:49
*** andreas_s has quit IRC12:49
* cdent tries coffee as cold medicine12:49
gibletfried_rice, cdent: I'm here and active until 20th of Dec so I can offer at least my review help on those patches12:50
sean-k-mooneycdent: im pretty sure you can by coffee enfused cold medicine12:50
fried_ricecdent giblet Okay cool, so what I was gonna say is, if y'all got time over the next couple hours to look at those, I can answer any questions or explain any weirdness.12:50
*** alexchadin has joined #openstack-nova12:50
fried_ricesean-k-mooney Don't they call that "Irish coffee"?  ;P12:51
*** andreas_s has joined #openstack-nova12:51
sean-k-mooneycdent: but if that does not work you can try the irish solution of brandy/wisky infused coffe12:51
*** yamamoto has quit IRC12:51
*** andreas__ has quit IRC12:52
fried_ricecdent Can you explain this delta?  https://review.openstack.org/#/c/414230/2..6/nova/scheduler/client/report.py@180  In particular, why did we feel it necessary to check for both `openstack-request-id` and `x-openstack-request-id`?12:52
cdentsean-k-mooney: that will be later in the day, while I think a weekly resource providers update supported by whisky might be entertaining, it might not be all that useful12:54
*** alexchadin has quit IRC12:54
cdentfried_rice: because there was a brief time when both was going to be supported (as in moving to not using the x- prefix) but then that didn’t happen12:55
*** alexchadin has joined #openstack-nova12:55
*** yangyapeng has joined #openstack-nova12:55
fried_riceOkay, so https://review.openstack.org/#/c/523606/ is kosher?12:55
*** andreas_s has quit IRC12:55
sean-k-mooneyx-openstack-request-id is the http header we use for the osprofiler stuff right?12:56
fried_riceoh, you're saying we still might move to the non-x version at some point, whereupon the above would have to be reverted?12:56
sean-k-mooneyi taught all non standard http headers were required to have an x prefix12:56
cdentit is kosher, as I said on my +1 on ps312:56
cdentsean-k-mooney: they got rid of that requirement somewhere in the last 3 years or so12:57
*** andreas_s has joined #openstack-nova12:57
cdentfried_rice: even if we did move to wanting the new style, we would have to continue producing the old style, because of compatibility between version12:57
fried_riceOh, I didn't see your review.  Cool.12:57
sean-k-mooneycdent: i guess to prevent peopel from updating all there code to support both when its standariesed12:57
cdentit’s like how the microversion headers in nova always return and support both the old and new styles12:58
cdentsean-k-mooney: yes12:58
sean-k-mooneyin anycase key.endswith('-request-id') was a little too broad before i like the v6 version more that said wont this rais an exception if neither form are found13:00
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref  https://review.openstack.org/52300713:01
*** andreas_s has quit IRC13:01
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060813:02
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948313:04
*** liverpooler has joined #openstack-nova13:05
*** stephenfin is now known as finucannot13:05
sean-k-mooneyactully never mind if .get does not find the key and no default is given it returns None instead of raising an exception13:05
*** diga has joined #openstack-nova13:06
*** jaosorior has quit IRC13:12
*** jaosorior has joined #openstack-nova13:13
openstackgerritSilvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts  https://review.openstack.org/52224513:17
*** claudiub has joined #openstack-nova13:20
*** jaypipes has joined #openstack-nova13:28
*** salv-orlando has quit IRC13:29
*** jaypipes is now known as leakypipes13:29
*** salv-orlando has joined #openstack-nova13:30
*** trozet has quit IRC13:31
gibletfried_rice: regarding https://review.openstack.org/#/c/521605 Can we change the traits of an RP without bumping its generation?13:32
gibletfried_rice: I wondering about having the default None in https://review.openstack.org/#/c/521605/5/nova/compute/provider_tree.py@134 is necessary13:33
*** salv-orlando has quit IRC13:34
fried_ricegiblet So13:35
fried_riceIIUC, the generation is intended to be used to detect async updates when we push stuff back to placement.13:35
fried_riceat the moment, I feel like it's not well defined how it's supposed to get used.13:36
*** edleafe is now known as figleaf13:36
fried_riceI.e. who's responsible for bumping the generation, who's responsible for checking for inconsistencies, what the result of an inconsistency should be (exception? HTTP error? Retry? ...)13:36
gibletfried_rice: I think the client should send in bumped generation and the server is responsible to reject update with outdated generation13:37
fried_riceI think that's about the gist of it, yeah.13:37
leakypipesfried_rice: the nova.objects.resource_provider._increment_provider_generation() function pretty well explains it.13:38
* fried_rice looks...13:38
*** lpetrut has joined #openstack-nova13:39
fried_riceleakypipes So the server side is responsible for incrementing the generation.13:39
fried_riceIt should never be done by the client13:40
*** markvoelker has quit IRC13:40
fried_riceThe client is supposed to take the generation from the server side, make whatever changes to the *other* fields, and push back the object with the *same* generation, which lets the server detect whether a concurrent update occurred.13:40
fried_riceIOW, the only time we should be updating the generation in ProviderTree is when we've freshly retrieved the object from placement.13:41
gibletleakypipes, fried_rice: thanks, now it make sense13:42
gibletfried_rice: so the case when generation is None in https://review.openstack.org/#/c/521605/5/nova/compute/provider_tree.py@134 is when the client update the trait of the RP13:43
*** markvoelker has joined #openstack-nova13:43
leakypipesfried_rice: the client is responsible for refreshing its knowledge of the latest generation, yes.13:43
fried_ricegiblet Yeah, so the code path (in a subsequent patch in the series) is going to be a GET of the RP's traits.  Uhm, I need to check whether that response includes the RP generation I suppose.13:44
leakypipesfried_rice: it does not.13:44
fried_riceOkay.  Then we would have no way to know what to update it to.13:44
fried_riceIn fact, I'm not even sure we should have the generation arg there at all...13:45
fried_riceSame for aggregates, I assume.13:45
*** _heena_py_ has quit IRC13:45
leakypipesfried_rice: changing aggregates doesn't increment the generation.13:46
*** peter-hamilton has joined #openstack-nova13:47
leakypipesfried_rice: because when you associate or disassociate an aggregate, you aren't changing the state of the resource provider's own attributes (like traits or inventories). there could be an argument made for incrementing the generation when aggs change on a rp, but I'm a bit skeptical about that13:47
*** tesseract has quit IRC13:47
fried_riceleakypipes But generation would change if you update traits?13:48
gibletleakypipes: the worst thing is that two parallel aggregate update on the same RP overwrites each others13:48
*** mriedem has joined #openstack-nova13:49
leakypipesgiblet: sure, understood.13:50
*** mlavalle has joined #openstack-nova13:50
leakypipesfried_rice: yes.13:50
leakypipesfried_rice: for instance, if/when we support setting distances between aggregates (for affinity/anti-affinity), some attribute of the aggregate is being changed that could affect placement decisions and therefore the generation should be incremented.13:50
*** tesseract has joined #openstack-nova13:50
fried_riceThat makes sense.  But in that case, the RP generation should really come back with the GET of traits/aggs, nah?13:51
fried_riceleakypipes But then we run into the issue of: if I have my RP cached at generation 1, and I GET traits and they come back at generation 2, I really ought to re-GET the RP - and on and on until the generation agrees on all the GETs.13:52
*** yamamoto has joined #openstack-nova13:52
leakypipesfried_rice: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/trait.py#L21513:52
leakypipesfried_rice: the generation is return when listing traits.13:52
mriedemso i guess zuul just totally crapped out last night huh13:53
leakypipesmriedem: yup13:53
leakypipesfried_rice: correctomundo.13:53
leakypipesfried_rice: which is why your proposed (GET /resource_providers_and_all_the_subresources call makes sense to me.13:53
fried_ricemriedem They restarted some stuff that may have fixed it around 4h ago, but backlog will take a while to clear.13:55
mriedemyikes13:55
mriedemwe've had things sitting in the gate for 2 days now13:55
mriedembut ok13:55
*** yamahata has joined #openstack-nova13:55
fried_riceleakypipes Okay, I had gotten the impression that that guy wasn't going to get traction; the code in my current series does all of that via a dozen separate calls (but without the generation checking).13:56
*** yamamoto has quit IRC13:58
leakypipesfried_rice: ack. which is fine, frankly. it may seem like there's a potential for data corruption, but aside from the aggregates, there isn't.13:58
cdentfried_rice: i think the lack of traction is not because it is a fundamentally bad idea, but because there’s insufficient sharing of info on what all that stuff’s doing (as I’ve noted in the rp updated pushed out just now)13:58
*** alexchadin has quit IRC14:00
*** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Queens_Release_Schedule"14:00
-openstackstatus- NOTICE: The issues have been fixed, Zuul is operating fine again but has a large backlog. You can recheck jobs that failed.14:00
fried_ricecdent sorry, noted where?14:00
*** yamamoto has joined #openstack-nova14:01
cdentfried_rice: in the nested providers section of http://lists.openstack.org/pipermail/openstack-dev/2017-December/125318.html14:01
*** kuzko has quit IRC14:02
leakypipesfried_rice: cdent is referring to his weekly status email14:03
leakypipesoh, jinx14:03
fried_riceaha14:03
*** baoli has joined #openstack-nova14:04
leakypipescdent, giblet, fried_rice: in any case, to just wrap this conversation up in a tidy little bow, the resource provider generation protects the resource provider's important attributes (inventories, traits, allocations, etc) against concurrent writes. It means that yes, you need to occasionally refresh that generation if you get a 409 Conflict about a concurrent write.14:04
cdentleakypipes: we still haven’t done server side retry of allocations have we? reckon we should get that in sooner than later or does it matter?14:05
fried_riceleakypipes At the moment the only thing that's actually doing RP updates (qua updates - not fresh creates) is inventory.  And there's no concurrency issues yet because it's always 1:1 compute node to RP.14:06
leakypipesbut also keep in mind that that refresh/concurrent update on the provider generation is exceedingly rare in the ProviderTree (since the compute host is what houses the ProviderTree and all operations against it are currently done while holding a semaphore in the compute manager. The scheduler's claim_resources() block is where the contention is, but it's designed for quick retries when it receives a 409 Conflict for either a concurrent update14:06
leakypipesor a capacity exceeded due to a concurrent claim14:06
leakypipesfried_rice: claim_resources() in the scheduler is the only hot-spot currently.14:07
leakypipesfried_rice: and that is by design.14:07
fried_ricewait, claims don't update a RP generation, do they??14:07
leakypipescdent: nope, only client-side (in the reportclient.claim_resources(*)14:07
leakypipesfried_rice: they absolutely do.14:08
fried_riceoh, dang, then we definitely need to do more than we're doing.14:08
leakypipesfried_rice: https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L209814:08
cdentleakypipes: Is that “no we shouldn’t bother” or “ack, I agree we aren’t”14:09
*** kuzko has joined #openstack-nova14:09
*** tbachman has joined #openstack-nova14:09
figleafmriedem: heh, you are already rechecking the alternate host series14:09
leakypipescdent: on the server-side retry thing? that was a "no, we just haven't gotten to that yet and I'm not too worried because we have client-side retries"14:10
leakypipescdent: i.e. the server-side would be a welcome optimization but isn't unsafe14:10
figleafmriedem: I had them open, and didn't see your update until I rechecked 'em14:10
cdent14:10
mriedemi have rechecked all things14:10
figleafmriedem: cool, thanks14:10
*** sahid has quit IRC14:11
leakypipesdiga: whatup? :)14:11
*** trozet has joined #openstack-nova14:11
digaleakypipes: Hi14:11
leakypipesdiga: the thing ew've been discussing -- server-side retries of the claim_resources() logic -- is something I think you might be interested in contributing.14:11
*** lyan has joined #openstack-nova14:12
leakypipesdiga: it all stems from the following TODO left in the code:14:12
leakypipeshttps://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L2148-L215214:12
digaleakypipes: yep, this can be gr8 opportunity for me to pitch in :)14:12
digaleakypipes: let me go through this14:12
digaleakypipes: How can I test this first14:13
*** kuzko has quit IRC14:13
leakypipesdiga: currently, we are doing a retry of the "claim resources" transaction from the client side (i.e. in the scheduler). It would be an optimization to be able to retry the transaction from the placement server side (thus saving the round-trip HTTP call)14:13
*** sahid has joined #openstack-nova14:14
fried_ricediga The real life scenario would be to have two separate threads doing claims, and make sure they do them in a certain sequence.14:15
openstackgerritSurya Seetharaman proposed openstack/nova master: update_cell allows more than once cell to have the same db/transport url  https://review.openstack.org/51855614:15
fried_ricediga For test purposes, you could simulate it by cloning the local RP data, doing a claim with one copy, and then doing a different claim with the second copy (which will have the old generation).14:15
*** kuzko has joined #openstack-nova14:16
*** sahid has quit IRC14:19
cdentsigh, I was wrong about deploy.deploy() being a public interface, it’s deploy.loadapp() that I was thinking of. I’ll fix things appropriately.14:19
*** kuzko has quit IRC14:20
*** tssurya has quit IRC14:22
*** gbarros has joined #openstack-nova14:23
*** sahid has joined #openstack-nova14:24
*** esberglu has joined #openstack-nova14:24
*** dansmith is now known as superdan14:25
*** tbachman_ has joined #openstack-nova14:31
*** diga has quit IRC14:32
*** tbachman has quit IRC14:33
*** tbachman_ is now known as tbachman14:33
*** links has quit IRC14:37
*** trozet has quit IRC14:38
*** kuzko has joined #openstack-nova14:38
mriedemfried_rice: do we need to hold https://review.openstack.org/#/c/522112/ for https://review.openstack.org/#/c/508345/ or can yours be stacked on top?14:39
mriedemdiga left huh14:40
mriedemwas going to point them at how we deal with this client-side today https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L9814:40
*** awaugama has joined #openstack-nova14:41
*** kuzko has quit IRC14:43
openstackgerritChris Dent proposed openstack/nova master: [placement] annotate loadapp as public interface  https://review.openstack.org/52669114:45
*** diga has joined #openstack-nova14:47
digaleakypipes: Hi14:47
*** yamahata has quit IRC14:47
digasorry got disconnected14:47
*** gbarros has quit IRC14:48
digaleakypipes: I will try to test it as per your suggestion, if anything requires, will ping you14:48
openstackgerritStephen Finucane proposed openstack/nova master: tests: fixes mock autospec usage  https://review.openstack.org/44750514:51
*** artom has joined #openstack-nova14:51
fried_ricemriedem Either way will work.  Mine isn't working yet, so do the other first.14:52
digacdent: I will ping you if need help14:52
*** Guest9054 is now known as melwitt14:52
mriedemfried_rice: ok - did you see anything wrong with the way they are doing the ksa options in that one?14:52
fried_ricemriedem No, looks fine.  Mine will need a manual rebase to accomodate, of course.14:53
*** rcernin has quit IRC14:55
fried_ricemriedem I left a +1.14:56
mriedemthanks14:56
*** andreas_s has joined #openstack-nova15:02
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.57 - Add trusted_image_certificates  https://review.openstack.org/50039615:02
*** salv-orlando has joined #openstack-nova15:04
*** hongbin has joined #openstack-nova15:07
*** BryanS68 has joined #openstack-nova15:09
*** hemna_ has joined #openstack-nova15:09
*** burt has joined #openstack-nova15:10
*** tssurya has joined #openstack-nova15:12
*** tidwellr has joined #openstack-nova15:13
*** diga has quit IRC15:14
*** tssurya has quit IRC15:17
*** hemna_ has quit IRC15:18
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.57 - Add trusted_image_certificates  https://review.openstack.org/50039615:19
*** trozet has joined #openstack-nova15:20
*** tssurya has joined #openstack-nova15:20
openstackgerritMatt Riedemann proposed openstack/nova master: Deprecate file injection  https://review.openstack.org/52202715:22
*** Apoorva has joined #openstack-nova15:22
*** tssurya has quit IRC15:23
*** burt has quit IRC15:23
*** armax has joined #openstack-nova15:24
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.57 - Add trusted_image_certificates  https://review.openstack.org/50039615:25
*** tssurya has joined #openstack-nova15:25
*** burt has joined #openstack-nova15:26
*** hemna_ has joined #openstack-nova15:27
*** claudiub has quit IRC15:28
*** Apoorva has quit IRC15:28
melwittmriedem: I utilitized the quotas follow up patch https://review.openstack.org/#/c/52423415:30
mriedemok, currently re-reviewing superdan's live migration allocations thing15:31
melwittsahid: would you be able to take a look at this review where there's a bug with disk device addresses being changed during a live migration? https://review.openstack.org/#/c/51802215:34
*** salv-orlando has quit IRC15:34
melwittit looks like it's a regression caused by a different bug fix from the past https://review.openstack.org/#/c/45974115:35
*** salv-orlando has joined #openstack-nova15:35
openstackgerritChris Dent proposed openstack/nova master: [placement] add name to resource provider create error  https://review.openstack.org/52671015:36
sahidmelwitt: i think i did that this morning15:36
sahidis there something not clear you want to discuss?15:37
melwittmdbooth: it would be good to have your review on this change for fixing the cache value with regard to O_DIRECT support during an image conversion https://review.openstack.org/#/c/52355415:37
mdboothmelwitt: I vaguely recall having looked at something similar, if not this15:38
melwittsahid: oh geez, sorry. I didn't see that you reviewed it this morning. I had it on my todo to ask you the past few days and just did it blindly. thanks15:38
*** amodi has joined #openstack-nova15:38
*** liangy has joined #openstack-nova15:39
*** lyan has quit IRC15:39
*** salv-orlando has quit IRC15:40
openstackgerritMerged openstack/nova master: Avoid stashed connector lookup for new style detach  https://review.openstack.org/52618215:43
melwittmdbooth: most of the patch is moving the supports_direct_io check to nova/utils.py so that it could be used by both virt/images and libvirt/driver. but the main part of the fix is in here https://review.openstack.org/#/c/523554/6/nova/virt/images.py15:43
* superdan opens the live migration patch and waits for mriedem's next -1 to roll in15:44
mriedemsuperdan: done15:47
mriedemhowever, it's an easy one15:47
superdanreceived15:47
*** felipemonteiro_ has joined #openstack-nova15:47
*** tssurya has left #openstack-nova15:49
*** tssurya has joined #openstack-nova15:49
*** tssurya has quit IRC15:50
leakypipesmriedem: I'm trying to square your code comment here: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4591-L4595 with the following in the functional test case setup here: https://github.com/openstack/nova/blob/master/nova/tests/functional/compute/test_host_api.py#L76-L7915:50
leakypipesmriedem: the reason is I have some PoC code I'm working on locally that would allow host aggregates to be associated to compute nodes (via UUID) in addition to compute *services* by service hostname15:51
cdentsuperdan: not sure if you saw my earlier ping on this potentially cells related bug that needs some evaluation: https://bugs.launchpad.net/nova/+bug/173610115:51
openstackLaunchpad bug 1736101 in OpenStack Compute (nova) "nova placement resource_providers DBDuplicateEntry when name repeat" [Undecided,New]15:51
superdanI did not15:51
mriedemleakypipes: https://github.com/openstack/nova/blob/master/nova/tests/functional/compute/test_host_api.py#L78 is just test data, the hostmapping.host wouldn't be a uuid15:51
*** felipemonteiro__ has joined #openstack-nova15:51
mriedemleakypipes: at least, not a uuid that matches the computenode.uuid15:52
leakypipesmriedem: sorry, I'm not following you...15:52
leakypipesmriedem: you're creating a HostMapping with the host attribute of the mapping set to the compute node's UUID.15:52
superdancdent: so, two hosts with the same name is not a valid thing, but it sounds like they only have that situation because they're moving between cells?15:53
leakypipesmriedem: which is what the code comment in HostsAPI says we're not doing?15:53
mriedemleakypipes: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4591-L4595 is just saying, we're looking up the compute node (in a cell) by uuid, (microversion >=2.53), if we had the uuid stored in the host mapping (api db), then we wouldn't have to iterate the cells querying each db for a compute node with that uuid15:53
mriedemwe could just get the host mapping from the api db which would tell us which cell that compute node is in15:53
cdentsuperdan: I struggled to parse it and wasn’t entirely clear on the naming constraints15:53
mriedemleakypipes: https://github.com/openstack/nova/blob/master/nova/tests/functional/compute/test_host_api.py#L78 - the hostmapping.host field could be "turd.ferguson"15:53
superdancdent: the conflict is in the placement db, right?15:53
cdentyes, rp names are supposed to be unique15:54
superdancdent: the rp name comes from what though?15:54
leakypipesmriedem: ok. so are we adding HostMapping records for compute nodes now or are we not doing that?15:54
mriedemleakypipes: the test is just using something random, which in this case is the uuid that matches the compute node in the cell - confusing the test i guess15:54
leakypipesmriedem: note: compute nodes, not service hosts.15:54
mriedemleakypipes: we are adding host mappings for service hosts15:54
mriedemnot compute nodes15:54
leakypipesmriedem: ok, that's what I needed to know, thanks man15:54
cdentsuperdan: that’s what I’m not clear on. some portion of the host? If it’s not the fqdn, then that’s a clear potential for problems15:54
leakypipesmriedem: I'm writing func tests for this code I'm proposing, thus the question...15:55
mriedemleakypipes: this is what creates the host mappings http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/host_mapping.py#n19215:55
superdancdent: I wonder if it is something like the hostname, and that moving between cells has caused the name to be the same but the uuid change?15:55
mriedemhttp://git.openstack.org/cgit/openstack/nova/tree/nova/objects/host_mapping.py#n17715:55
leakypipesmriedem: I copied test_hosts_api.py and saw that after reading the comment in api.py about us not storing compute node records in host mappings table and was like... uh?15:55
mriedemhost_mapping = HostMapping(                 ctxt, host=compute.host,                 cell_mapping=cm)             host_mapping.create()15:55
*** felipemonteiro_ has quit IRC15:55
mriedemleakypipes: the api comment says "We wouldn't have to loop over cells if we stored the ComputeNode.uuid in the HostMapping" - specifically the uuid, but yeah i can see how that's confusing now15:56
mriedembecause even if we did, it'd be wrong15:56
mriedemwe should probably remove that comment15:56
* mriedem patches15:56
leakypipesmriedem: right. I'm hacking around the host mapping stuff, though, and calling ComputeNode.get_all() on all cells via the context.scatter_gather_skip_cell0() function, since I figured it would not be possible to look up compute node information in the host_mappings table.15:57
cdentsuperdan: I haven’t had much of a chance to think about it, I was just going through placement tagged bugs and that one was making a dance between invalid and new and thought I better try to make sure it wasn’t lost if it is a real issue15:57
leakypipess/get_all()/get_by_uuid()15:57
mriedemleakypipes: yeah correct - only the compute_nodes.host field is in there, and if you're ironic that doesn't help you15:57
mriedembecause 1:M15:57
superdancdent: well, I imagine it's a real issue, but I'm not sure exactly what to tell them.. I guess you could provide them the curl commands to grab a token and delete the old provider record to confirm that it heals up properly afterwards right?15:58
leakypipesmriedem: FYI, this is to support PUT /os-aggregates/{agg_id} and passing a compute node UUID instead of a service hostname.15:58
leakypipesmriedem: since our friends in Ironic can't use host aggregates due to the service host coupling.15:58
leakypipesmriedem: and our Ironic friends can't use any aggregate or affinity filter that relies on host aggregates being associated with an Ironic resource node.15:59
openstackgerritMatt Riedemann proposed openstack/nova master: Remove confusing comment in compute_node_get API method  https://review.openstack.org/52671816:00
mriedemleakypipes: see if this floats your boat ^16:00
*** tbachman has quit IRC16:00
leakypipesmriedem: of course, I am going to need to backport all of this to Ocata, when none of the context.scatter_gather stuff was in. so that should be super-fun.16:00
mriedemleakypipes: you can't add aggregates support for ironic in nova because mogan16:00
sean-k-mooneyleakypipes: FYI i talked to infra and at least for now they are ok with enableing legacy jobs so ill submit a patch for os-vif fuctional jobs on monday. ill see if i can migrate os-vif to in repo zuul v3 jobs before the PTG16:00
leakypipesmriedem: hmm?16:00
mriedemleakypipes: you mean backport internally...16:00
leakypipesmriedem: correcto.16:00
mriedemfun16:00
mriedemoath?16:00
leakypipesyuppers.16:00
mriedemgreat16:00
cdentsuperdan: when I say “ I haven’t had much of a chance to think about it” I really mean that. I’ve not processed what they are doing at all. If you’re trying to encourage me to take some responsibility for flagging something up, that’s great, and I can, but it’s not on my mental stack, despite the presumption of trying to stick ti on yours… :)16:00
leakypipesI blame the penick.16:01
mriedemleakypipes: oh right you weren't at the mogen session in sydney16:01
mriedemaggregates support in mogan was one of the differentiators16:01
leakypipesmriedem: I wasn't in Sydney. period. :)16:01
mriedemi could have sworn...16:01
mriedemleakypipes: so,16:01
mriedemare you going to do this upstream with a new microversion?16:01
leakypipesmriedem: well, frankly, a placement aggregate doesn't have the service host coupling that a nova host-aggregate currently does, so I'm just trying to get a patch in place for Ocata Nova that kind of gets nova host-aggs looking/functioning more like placement aggs16:02
mriedemwhat happens when oath upgrades to pike or queens?16:02
leakypipesmriedem: I'll throw my code up and have you take a looksie, of course.16:02
mriedemwell,16:02
leakypipesmriedem: when oath upgrades to pike/queens, we'll likely forward-port this thing I'm wokring on16:03
mriedemmy point is, backporting microversions opens up some bad issues unless you backport all intermediate microversions too16:03
leakypipesmriedem: until placement aggs are fully utilized.16:03
*** artom has quit IRC16:03
leakypipesmriedem: I'm not doing this as a microversion :)16:03
mriedemok16:03
leakypipesmriedem: I'm not changing the API at all.16:03
mriedem"this is to support PUT /os-aggregates/{agg_id} and passing a compute node UUID instead of a service hostname."16:03
leakypipesmriedem: just looking to see if hostname is like a UUID and if so, looking up a compute node instead of a service host.16:03
mriedemok,16:04
mriedemwas just going to say that :)16:04
leakypipesit's cheeky, I know.16:04
mriedemtricksy16:04
leakypipescheeky cheeky monkey.16:04
mriedemdoes oath also want to be able to cold migrate/resize baremetal instances?16:04
sean-k-mooneysuperdan: i was under the impression that we required the hostID to be gloally unique across the entire cloud not jsut within a cell so if they are useing hostnames as the hostid .i.e the default then they would be required to have unique host names across all cells16:05
superdansean-k-mooney: correct, but they're moving a host between cells16:05
mriedemleakypipes: because https://review.openstack.org/#/c/449155/16:06
sean-k-mooneysuperdan: yes but i was under the impression that the host id had to be unique period e.g. its invalid to have to hosts with the same hostname in different cells if you are using the hostname as the hostid16:06
leakypipesmriedem: that hasn't been brought up, no16:06
superdansean-k-mooney: yes, that's true, I'm not sure what you're getting at16:07
superdansean-k-mooney: they are moving hosts between cells, which means the host gets a new UUID, and I think the problem is they're reporting a new uuid with the same name, and placement is kicking it out as a result16:07
finucannotmelwitt: Could you take another look at this some time today/early next week? https://review.openstack.org/#/q/topic:bp/websocket-proxy-to-host-security+status:open16:09
cdenthow does someone move a host between cells? Is that a command they run to which “remove the resource provider” needs to be added?16:09
sean-k-mooneysuperdan: oh sorry they are moving the physical host between cells. not that they actlly have 2 phyical host with the same hostname16:09
superdancorrect16:09
dmsimardmriedem, kashyap: would love if we could iron out https://bugs.launchpad.net/nova/+bug/1735823 soon, it's making a non-negligible amount of jobs flap16:09
openstackLaunchpad bug 1735823 in OpenStack Compute (nova) "Nova can hang when creating a VM with disk injection" [Medium,In progress] - Assigned to Matt Riedemann (mriedem)16:09
mriedemcdent: very carefully16:09
cdentas is so often the case16:09
superdancdent: that's my point16:09
melwittfinucannot: been meaning to do that, yes I will16:09
kashyapdmsimard: Hey, I've seen it; but a bit overwhelmed today16:09
kashyapdmsimard: I haven't forgotten it16:09
superdancdent: if they're doing this thing, its going to leave residue we likely don't have a cleanup command for16:09
dmsimardkashyap: I know the feeling, just a friendly reminder :)16:09
mriedemdmsimard: are these only centos jobs?16:09
kashyapYeah, was wondering the same.16:10
cdentsuperdan: ah okay, I think I’m _finally_ caught up then. I was way way back there in the weeds.16:10
mriedemi'm assuming the problem is some difference in the guestfs package versions16:10
*** liangy has quit IRC16:10
cdentI blame society. Or my head cold. Or just being stupid.16:10
mriedemdmsimard: because https://review.openstack.org/#/c/524748/ has been busted for a long time, and we know it doesn't fix the issue16:10
dmsimardmriedem: the data that we have has problems on centos and opensuse, I couldn't find any on Ubuntu but that doesn't mean it doesn't happen there16:10
mriedemi think the lock issue is the fact we're using eventlet.tpool16:11
*** liangy has joined #openstack-nova16:11
dmsimardmriedem: well, it kind of fixes the issue but then generates new ones ? right ? the patch with the depends-on fails horribly16:11
*** amodi has quit IRC16:11
mriedemhttps://review.openstack.org/#/c/524750/ ?16:11
kashyapYeah, that seems to be the test16:12
finucannotsean-k-mooney: What's this about functional tests?16:12
finucannotI assume you mean enabling ralonsoh's patches?16:12
mriedemoh boy http://logs.openstack.org/50/524750/1/check/legacy-tempest-dsvm-neutron-full/eb8d805/logs/screen-n-api.txt.gz?level=TRACE#_Dec_04_13_34_20_63587416:13
sean-k-mooneyfinucannot: yes im going to add a job to the experimental pipline for os-vif so we can run rodolfos tests16:13
sean-k-mooneyfinucannot: once they are merged i can move it to the gate pipline16:13
*** jaosorior has quit IRC16:13
finucannotsean-k-mooney: Ideal16:13
finucannotgate for os-vif or nova too?16:14
sean-k-mooneycheck and gate for os-vif16:14
superdanleakypipes: finucannot: melwitt: either one of you can mindlessly approve this trivial thing without worry: https://review.openstack.org/#/c/507874/116:14
*** itlinux has joined #openstack-nova16:14
leakypipessuperdan: BFV with BDMs?16:14
finucannotsuperdan: If there's one thing I know how to do...16:14
superdanleakypipes: not even16:15
leakypipes:)16:15
superdanfinucannot: I knew I could count on you16:15
finucannotsean-k-mooney: Cool. We already run oslo.versionedobjects tests in the experimental pipeline for nova. Might be worth adding os-vif there too16:15
finucannotEventually, anyway16:15
*** weshay|ruck is now known as weshay|ruck|MOD16:15
sean-k-mooneyi would also like to move all our os-vif jobs to zuul v3 in repo config and add a linux-bridge job for os-vif but it will be a few week before i have time to do that16:15
sean-k-mooneyfinucannot: os-vif is already in the nova experimental pipline16:16
sean-k-mooneycheck experimtal on a nova patch will test that patch with the head of the os-vif master branch in a seperate job16:16
finucannotsuperdan: I know you asked for a mindless approval but...no bug? https://review.openstack.org/#/c/507874/16:16
superdanfinucannot: it's not a bug yet16:17
sean-k-mooneyfinucannot: the one thing it does not do is work with depend-on where the depends-on is an unmerged os-vif change16:17
superdanfinucannot: it will be after my live migrations patch merges, it's just an omission right now, the last of a few fixes ahead of time16:17
finucannotsuperdan: Yup, just saw jianghuaw_'s comment now. Cool, +Wd16:17
superdanfinucannot: thanks16:18
mriedemwell it might be a bug16:18
finucannotsean-k-mooney: All the os-vif jobs or just the unit tests?16:18
finucannot*unit test jobs16:18
mriedemif the driver doesn't pass migrate_data, we don't set the migration.status to 'error'16:18
sean-k-mooneyfinucannot: ill try and move them all i think. we will see what infra advise16:19
sean-k-mooneyfinucannot: what has nova done?16:19
mriedemnothing16:19
mriedemnovaclient on the other hand https://review.openstack.org/#/q/I43a8435485751748ca6228f67d401945cb32652e16:19
finucannotmriedem: Well https://review.openstack.org/#/q/status:open+branch:master+topic:zuulv3-nova16:19
mriedemsean-k-mooney: there are very few ci jobs that nova runs which are just run against nova16:19
finucannotWe don't have to move much, thankfully16:19
mriedemi think the nova-next job is the only one16:19
finucannotBecause of that, yes16:19
mriedemoh lxc/lvm16:20
mriedemumm, we can drop the wsgi-full job, unless we run that on stable16:20
*** tidwellr has quit IRC16:20
mriedemwsgi is the default in devstack since pike16:20
finucannotWe don't run any of them on stable. Drop that entirely so?16:20
sean-k-mooneyah cool well i dont think it will be that hard to move os-vif. is there a plan to move nova eventually or just rely on the infra maintained jobs16:21
mriedemidk, i'd have to go through this with a comb16:21
mriedemci job shit is always a minefield16:21
finucannotsean-k-mooney: Those patches simply copy in the legacy ones. I haven't wrapped my head around translating a job to zuulv3 yet16:21
finucannotdespite mordred's best efforts16:22
melwittwould appreciate reviews on this relatively simple bug fix to set the right cache mode during image conversion. most of the patch is moving a function that checks for O_DIRECT support to nova/utils so it can be used by both virt/images and libvirt/driver. the actual fix is in https://review.openstack.org/#/c/523554/6/nova/virt/images.py16:22
*** MasterOfBugs has joined #openstack-nova16:22
mordredI didn't do it16:22
*** pramodrj07 has joined #openstack-nova16:22
sean-k-mooneyfinucannot: ya just too a look at your patches now.16:23
mordredfinucannot: oh - well - I'm working on converting glanceclient jobs to be native at the moment (and finding a weird issue) - once I'm done with that, I'll circle back around and help update yours16:23
finucannotmriedem: The migration made it especially fun https://review.openstack.org/#/c/514310/3/zuul.d/zuul-legacy-jobs.yaml@609216:24
sean-k-mooneyfinucannot: there is a sports and social Q4 party tonight in the stand so ime going to head home soon but ill take a look at what you have started on monday16:24
finucannotmordred: I'd appreciate that. Will do whatever I can to help16:24
finucannotsean-k-mooney: Enjoy. We've the Dublin office party tonight too16:24
sean-k-mooneyoh fun dublin has more interesting venues then the strand to choose form16:25
openstackgerritMerged openstack/python-novaclient master: inject file: add method of showing quota value of injecting files for 'rebuild' command  https://review.openstack.org/52458816:25
*** takedakn has joined #openstack-nova16:26
*** jangutter has quit IRC16:26
mordredfinucannot: it should actually be *fairly* easy ... once I figure out why the glanceclient job is somehow getting a broken mysql - but shade and python-openstackclient work fine16:26
finucannotAye. The biggest issue I had was figuring out who was using what (and why). That's probably general infra- inexperience though16:27
sean-k-mooneymordred: is it insatlling in a venv and therefor may be using a different pymysql package16:27
finucannotand I haven't gone further down the zuulv3 rabbit hole that attempting to diff jobs before and after (I didn't get far)16:27
*** baoli has quit IRC16:27
*** felipemonteiro__ has quit IRC16:28
*** felipemonteiro__ has joined #openstack-nova16:28
*** takedakn has quit IRC16:29
mriedemdmsimard: are the centos or opensuse jobs voting anywhere?16:29
dmsimardmriedem: the devstack ones ? I'm not sure, but it's impacting non-devstack stuff like packstack16:30
dmsimardmriedem: tripleo doesn't run disk injection tests in tempest so they're not running into the issue16:31
mriedemdmsimard: so, has anyone tried downgrading the libguestfs package on centos to the version that we have in the UCA jobs and see if they still fail?16:31
*** gszasz has quit IRC16:31
dmsimardI haven't really tried reaching out to OSA and kolla if they've been noticing it too.16:31
*** gszasz has joined #openstack-nova16:31
dmsimardmriedem: no, not that I am aware of -- I'm not sure if that version is even available on EL7, I'd need to check.16:32
mordredsean-k-mooney: well - eventually it should - but it's actually failing in the devstack layer before it even gets there. it's super SUPER strange, but I'm certain I just left out a comma or something16:34
*** lyan_ has joined #openstack-nova16:35
*** mlavalle has quit IRC16:35
*** liangy has quit IRC16:35
*** yangyape_ has joined #openstack-nova16:36
dmsimardmriedem: yeah we don't have 1.32.2, closest I see is libguestfs-1.32.7-3.el7.centos.x86_64.rpm and that's in an archived release16:36
*** lyan_ has quit IRC16:36
*** yangyapeng has quit IRC16:36
dmsimardmriedem: it might be complicated to downgrade the package, it probably doesn't come on its own16:36
finucannotmelwitt: I'll take a look now16:37
finucannotleakypipes: Could you take a look at https://review.openstack.org/#/c/390520/ again at some point in the next few days?16:38
* finucannot figures you're the man for the job 16:38
melwittthanks finucannot16:38
*** baoli has joined #openstack-nova16:38
leakypipesfinucannot: yes, wil try my best16:38
finucannotleakypipes: Excellent. Appreciate it16:39
*** AlexeyAbashkin has joined #openstack-nova16:40
mriedemsdague: superdan: methinks we should consider deprecating the libvirt.inject_key option16:41
mriedem(1) depends on file injection16:41
mriedem(2) doesn't work for volume-backed instances16:42
mriedem(3) use config drive16:42
*** itlinux has quit IRC16:42
superdani16:42
superdanam down16:42
mriedemsame with inject_password16:43
*** itlinux has joined #openstack-nova16:43
*** lyan has joined #openstack-nova16:43
mriedemoh fun we pass down the libvirt virt_type when injecting network info because of lxc16:44
*** AlexeyAbashkin has quit IRC16:44
finucannotI've got a rather simply bugfix here that resolves an issue we've seen with dodgy glance configurations, if anyone's bored https://review.openstack.org/#/c/52632916:45
*** lyan has quit IRC16:46
*** gszasz has quit IRC16:46
finucannotmriedem: Do we still support lxc?16:46
*** lyan has joined #openstack-nova16:47
mriedemfinucannot: where does https://docs.openstack.org/python-glanceclient/latest/reference/api/glanceclient.v2.images.html say something about 0 data?16:47
finucannotmriedem: https://docs.openstack.org/python-glanceclient/latest/reference/api/glanceclient.v2.images.html#glanceclient.v2.images.Controller.data16:47
finucannotI trimmed the URL because I didn't want to pollute everyone's git-log16:48
finucannot(more than I'd already done)16:48
finucannotHa! python-glanceclient is broken itself16:54
openstackgerritEd Leafe proposed openstack/nova master: Change RPC for select_destinations()  https://review.openstack.org/51670716:56
openstackgerritEd Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils  https://review.openstack.org/51135716:56
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135816:56
openstackgerritEd Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize  https://review.openstack.org/52643616:56
*** tidwellr has joined #openstack-nova16:57
*** fragatina has quit IRC16:57
*** david-lyle has quit IRC16:57
*** liusheng has quit IRC16:57
*** liusheng has joined #openstack-nova16:57
*** baoli_ has joined #openstack-nova16:59
*** fragatina has joined #openstack-nova17:00
*** itlinux has quit IRC17:00
*** andreas_s has quit IRC17:00
*** chyka has joined #openstack-nova17:00
*** andreas_s has joined #openstack-nova17:00
*** baoli has quit IRC17:01
*** amodi has joined #openstack-nova17:02
*** tbachman has joined #openstack-nova17:03
melwittmriedem: I noticed you abandoned these https://review.openstack.org/#/c/508163 did we not want to keep them and not skip that test now that test_volume_backed_live_migration works? to guard against regression?17:04
mriedemfewer double negatives please17:04
*** fragatina has quit IRC17:05
*** itlinux has joined #openstack-nova17:05
melwittsorry. like, we were using those DNM patches to test a fix for volume backed live migration17:05
*** damien_r has quit IRC17:05
*** andreas_s has quit IRC17:05
melwittnow that volume backed live migration is fixed, shouldn't we have removed DNM from those to unskip the test?17:05
melwittdon't we want to unskip the test now that it will pass, that is17:07
*** felipemonteiro_ has joined #openstack-nova17:07
mriedemnow that volume backed live migration is fixed....17:07
mriedemit is?17:07
mriedemyou mean https://review.openstack.org/#/c/507202/ ?17:08
melwittyeah, I thought that fixed it. you had those DNM patches depend on it and the unskipped tests passed17:08
melwittdid I miss something?17:08
mriedemmelwitt: i believe those tests will intermittently pass regardless of that patch17:08
melwittokay, that's what I was missing then17:09
mriedemthe volume-backed and iscsi live migration tests were skipped in tempest long ago due to random failures17:09
mriedemthat no one could sleuth because qemu17:09
mriedemusing UCA now might have changed that17:09
melwittI see17:09
mriedembug 152489817:09
openstackbug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,In progress] https://launchpad.net/bugs/152489817:09
melwittokay, I thought maybe that was related to the recent fix we had17:10
mriedemeven if we enable the test in tempest again,17:10
mriedemtempest is branchless so it would run on stable branches and could fail there if we're using older packages that had a problem17:10
mriedemhard to tell17:10
mriedemi would very much like to get volume-backed live migration testing enabled in the gate though yes17:10
melwittoh, right.17:10
mriedemwe should probably change the global skip on the test in tempest to a config option,17:11
mriedemso that we can enable the test per-branch17:11
melwittI can't recall if we skip certain tests for stable-only, or if that's even possible17:11
mriedemwe can configure tempest per-branch, per-job17:11
*** felipemonteiro__ has quit IRC17:11
mriedemwe can configure tempest in our in-tree live migration scripts17:11
mriedemthe post_test_hook17:11
mriedemhttps://github.com/openstack/nova/blob/master/nova/tests/live_migration/hooks/run_tests.sh#L3317:12
mriedemwith the magic of ansible17:12
*** derekh has quit IRC17:12
*** lucasagomes is now known as lucas-afk17:12
melwittokay, so we'd add another option volume_backed_live_migration or such17:13
mriedemso if you wanted to be cool, propose a change to tempest to add a config option for volume-backed live migration to replace that skip line, default to false for stable, and then set it to true in our live migration job post_test_hook script17:13
mriedemand then be available when it melts the gate17:13
*** lyan has quit IRC17:14
melwittcan do17:14
*** gyee has joined #openstack-nova17:14
melwittthanks17:14
mriedemit is my pleasure17:16
mriedemthat's a weird phrase17:16
mriedemdmsimard: interesting, even when it appears that we do set force_tcg mode, guestfs returns None rather than an int17:18
mriedemhttp://logs.openstack.org/50/524750/1/check/legacy-tempest-dsvm-neutron-full-centos-7/a7f051e/logs/screen-n-cpu.txt.gz#_Dec_04_13_43_19_92309817:18
melwittlots of zvm:recheck spam going on17:18
mriedemFailed to force guestfs TCG mode. guestfs_set_backend_settings returned: None17:18
dmsimardweird17:18
dmsimardmriedem: it seems like there's two set_backend_settings happening there17:19
*** karthiks has quit IRC17:19
dmsimardevent=trace eh=0 buf='set_backend_settings "force_tcg"' array=[]17:19
dmsimardand event=trace eh=0 buf='set_backend_settings = 0' array=[] ?17:20
*** baoli_ has quit IRC17:20
mriedemi think one is the value being set?17:21
mriedemor the return value?17:21
mriedemi don't know what the array is17:21
mriedembut like "add_drive = 0"17:21
*** lyan has joined #openstack-nova17:24
mriedemhttp://libguestfs.org/guestfs.3.html#guestfs_set_event_callback17:24
*** dtantsur is now known as dtantsur|afk17:24
*** mdnadeem has quit IRC17:26
*** baoli has joined #openstack-nova17:27
*** mlavalle has joined #openstack-nova17:28
*** tesseract has quit IRC17:29
*** lpetrut has quit IRC17:30
*** tidwellr has quit IRC17:30
mriedemok this is where we have the eventlet switch that blows up http://logs.openstack.org/50/524750/1/check/legacy-tempest-dsvm-neutron-full-centos-7/a7f051e/logs/screen-n-cpu.txt.gz#_Dec_04_13_43_19_94186917:33
mriedemthat's right in the middle of the guestfs.launch call17:33
mriedemerror: cannot switch to a different thread17:33
mriedemheh http://logs.openstack.org/50/524750/1/check/legacy-tempest-dsvm-neutron-full-centos-7/a7f051e/logs/screen-n-cpu.txt.gz#_Dec_04_15_25_29_32075317:34
mriedemso, i think the logging is what's maybe making it switch17:34
mriedemso you enable guestfs debug to get callbacks from guestfs to log them,17:35
mriedemmeanwhile, guestfs is running in a thread17:35
*** david-lyle has joined #openstack-nova17:36
mriedemthat doesn't explain why we were hanging before we enabled debug though17:36
mriedembut i can push a patch to only launch in a thread if guestfs.debug is False17:36
fried_ricecdent You going to be around in a couple hours?  For a "warm handoff"?17:38
mriedemgross17:38
* cdent shivers17:38
cdentfried_rice: yessir, I should be around, or at least nearby, make a ping and I’ll (eventually) come running17:39
fried_ricecdent ack17:40
fried_ricecdent Leaving a bit more of a mess than anticipated, I'm afraid.  But nothing insurmountable.17:40
*** fried_rice is now known as fried_rolls17:41
*** fragatina has joined #openstack-nova17:43
*** smatzek has joined #openstack-nova17:47
*** gbarros has joined #openstack-nova17:47
mnaseris there anyone here who worked on glance image signature verification?17:49
mnaserwe have a change in puppet-nova to add the ability to configure it, but i'd like to know if its a global nova config (aka, api/etc need it) or nova-compute only17:50
mnaserfrom my understanding of code, the option seems to be used in the glance image download code which probably only happens on computes?17:50
*** salv-orlando has joined #openstack-nova17:50
*** claudiub has joined #openstack-nova17:51
*** tbachman has quit IRC17:52
mriedempeter-hamilton: ^17:52
mriedemmnaser: dane-fichter is who i'd usually go to but haven't seem him in channel in a long time17:52
alaskimriedem: in case this didn't already get answered, instance actions are not synced between a cell and the api level in v1. There are spurious records at the API level that never get used, but it would have taken a lot of effort to not write them there. Requests for instance actions are proxied to the cell and results are returned from there.17:52
mnasermriedem: ok cool, thanks for the info.. i'll research the code a tad bit more17:52
mriedemalaski: hmm, i'm not sure how @wrap_instance_event in the compute service would ever work then17:53
mriedemalaski: because the api creates the action record17:53
mriedemand the compute tries to look it up to record events against it17:53
mriedembut if the action isn't in the cell db, how are they ever found?17:53
superdanmriedem: they get replayed in the cell I imagine17:53
mriedemsuperdan: where/how?17:53
superdanso they're down there, but the api never looks at them, only at the stale ones that are left by the start17:53
superdanmriedem: because we replay calls into compute/api down in the cell17:54
superdanwhich would create them there too17:54
mriedemso if a compute service fails to find an action created in the api, it could just be because of a timing issue?17:54
*** peter-hamilton has quit IRC17:54
superdanno, I don't think so,17:54
mriedemok, was going to say - i find that hard to believe17:55
mriedembecause we'd be randomly failing all the time17:55
superdanbecause it should still get created in the cell before you kick off anything17:55
mriedemok that doesn't seem to be happening17:56
mriedemi dug through the cellsv1 code for awhile yesterday and don't see anything special about instance actions in there thoguh17:56
mriedem*though, like how they are handled17:56
superdanright17:56
superdanbecause they're not17:56
superdanmriedem: for example, this one: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3098-L309917:57
superdanmriedem: we run that in the api node as part of the cells_api indirection17:57
openstackgerritMerged openstack/nova-specs master: Document Queens review priorities  https://review.openstack.org/52657317:57
superdanmriedem: then we pass it down to the cell and run that same code again in the cell,17:57
superdanthen we call to compute, which would finish it, but only in the cell17:57
superdanand then you're done17:58
mriedemand that's all synchronous?17:58
superdanif you go look at it from the api, you'd only see the start, because you never finished it in the api cell, and nothing ever sync'd the finish17:58
superdanmriedem: is what synchronous? the api cell before the child cell? sure, but they don't know about each other really anyway, so it doesn't matter17:58
*** tbachman has joined #openstack-nova17:59
mriedemis the replay of the instance action create from the top level to the child cell synchronous17:59
mriedemwas my question18:00
mriedemtrying to figure out what is special that makes this fail the cellsv1 job https://review.openstack.org/#/c/52367618:00
mriedemhttp://logs.openstack.org/76/523676/11/check/legacy-tempest-dsvm-cells/1d155c6/logs/screen-n-cpu.txt.gz?level=TRACE#_Dec_07_09_02_09_57999118:00
superdanthe thing that is replayed is the call to compute/api::confirm_resize()18:00
superdanthe action_create is18:00
superdannot replayed itself,18:00
superdanthe whole call to that method is18:00
*** salv-orlando has quit IRC18:01
mriedemok, so backup_instance would all be replayed in the child cell api too right?18:01
*** tidwellr has joined #openstack-nova18:01
superdanyeah18:01
superdanI don't see where the cells job ran on that18:01
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L259418:02
mriedemthe latest patch timed out in on the cells job18:02
superdanokay18:02
superdanwould have thought that would be reported18:02
mriedemzuul in the last 48 hours...18:02
*** yamamoto has quit IRC18:02
superdanoh I see, it is i the comment18:02
superdanbut not the summary18:02
*** openstackgerrit has quit IRC18:03
alaskistepped away, but it's exactly as superdan describes18:03
superdan\o/18:03
mriedemso i wonder if it has something to do with this comment from comstud https://github.com/openstack/nova/blob/master/nova/compute/api.py#L259418:04
mriedembut i think that is just about setting the task_state18:04
superdanmriedem: that's a remote call, so maybe it's calling to conductor which is looking in the wrong db?18:04
superdaner, a remote exception I mean18:05
mriedemwell,18:06
superdannot sure that makes sense actually18:06
mriedemi don't know why it would just fail on that one lookup and not all of the others18:06
mriedemthat's what i don't understand about this18:06
mriedemfried_rolls: have you ever mocked something that can't be imported?18:06
superdanoh Iknow why18:07
superdanmriedem: backup_instance doesn't get replayed in the cell for some reason, unlike everything else, which means it calls straight to the compute I think: https://github.com/openstack/nova/blob/master/nova/cells/messaging.py#L915-L92518:07
superdanmriedem: compare that to rebuild right below18:07
mriedemok yeah that's what i was looking at18:08
mriedemhttps://github.com/openstack/nova/commit/2be50f0f3f02830798c25442dc54e92faa18106b18:08
superdanI'd be extremely concerned about changing _any_ of that stuff at this point18:09
mriedemweird, ok, so i guess the note about "any changes made here have to be made there" is what that's about18:09
mriedemincluding creating the action record18:10
mriedemwell i see 2 options18:10
mriedem1. do the action create for backup and snapshot in the cells messaging code like it says,18:10
mriedem2. do something in the EventReporter where if cells is enabled, we blacklist some actions18:10
mriedemwe could remove #2 when we remove cellsv118:10
*** peter-hamilton has joined #openstack-nova18:11
superdanI don't love either of those18:11
peter-hamiltonmriedem: thanks!18:11
mriedemsuperdan: me neither18:11
peter-hamiltonmnaser: bpoulos is who you want to talk to; I'll ping her for you18:11
mriedemsuperdan: but i'm not sure what else to do at this point18:12
mnaserpeter-hamilton: cool, thanks!18:12
mriedemso, i'll go to lunch and not think about this18:12
*** r-daneel has joined #openstack-nova18:15
*** yangyape_ has quit IRC18:16
*** sdague has quit IRC18:16
*** yangyapeng has joined #openstack-nova18:16
*** yamamoto has joined #openstack-nova18:16
*** openstackgerrit has joined #openstack-nova18:17
openstackgerritMatt Riedemann proposed openstack/nova master: Don't launch guestfs in a thread pool if guestfs.debug is enabled  https://review.openstack.org/52674818:17
mriedemdmsimard: clarkb: ^18:17
*** tidwellr has quit IRC18:19
*** david-lyle has quit IRC18:21
dmsimardmriedem: so we toggled the bug by setting it to debug ? :/18:21
*** tssurya has joined #openstack-nova18:21
*** yamamoto has quit IRC18:21
dmsimardmriedem: the force_tcg patch would work without debug ?18:21
*** bpoulos has joined #openstack-nova18:23
bpoulosmnaser: only nova-compute needs the verify_glance_signatures option set in order to enable glance image signature verification18:25
mnaserbpoulos: awesome, thank you so much (fyi, asking in context of https://review.openstack.org/#/c/526514/ )18:26
bpoulosmnaser: no problem, happy to help18:27
*** yamamoto has joined #openstack-nova18:32
*** penick has joined #openstack-nova18:35
*** yamamoto has quit IRC18:36
*** bpoulos has quit IRC18:39
*** sdague has joined #openstack-nova18:40
*** yamamoto has joined #openstack-nova18:44
*** yamamoto has quit IRC18:44
*** yamamoto has joined #openstack-nova18:44
*** yamamoto has quit IRC18:45
*** penick has quit IRC18:55
openstackgerritJay Pipes proposed openstack/nova master: allow compute nodes to be associated with host agg  https://review.openstack.org/52675318:56
*** penick has joined #openstack-nova18:58
*** Guest99196 has quit IRC19:04
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/52479519:10
*** pramodrj07 has quit IRC19:13
*** MasterOfBugs has quit IRC19:13
*** BryanS68 has quit IRC19:13
*** BryanS68 has joined #openstack-nova19:14
openstackgerritMerged openstack/nova master: Updated from global requirements  https://review.openstack.org/52638619:16
*** harlowja has joined #openstack-nova19:21
*** peter-hamilton has quit IRC19:23
*** moshele has joined #openstack-nova19:24
*** tidwellr has joined #openstack-nova19:25
openstackgerritMerged openstack/nova master: Fix 'force' parameter in os-quota-sets PUT schema  https://review.openstack.org/52319419:27
*** MasterOfBugs has joined #openstack-nova19:27
*** pramodrj07 has joined #openstack-nova19:27
*** tssurya has quit IRC19:31
*** gbarros has quit IRC19:34
*** moshele has quit IRC19:36
*** moshele has joined #openstack-nova19:37
*** AlexeyAbashkin has joined #openstack-nova19:46
*** yamamoto has joined #openstack-nova19:46
mriedemdmsimard: that's what i hope to find out19:47
fried_rollsmriedem No, I don't think I have ever (successfully) mocked something that can't be imported.  It will fail at "compile" time.19:50
fried_rollsmriedem You can cheat by mocking it after it's imported, though.19:50
fried_rollsI assume this is in a situation where the thing imports correctly on the prod side but for some reason not on the test side?  Which... why?19:52
*** yamamoto has quit IRC19:52
*** fried_rolls is now known as fried_rice19:53
fried_ricecdent How's now?19:53
cdentone moment caller19:53
cdentblargh, timeouts in migration tests: http://logs.openstack.org/39/521639/8/gate/openstack-tox-functional/f6880ac/testr_results.html.gz19:54
*** AlexeyAbashkin has quit IRC19:56
*** tssurya has joined #openstack-nova19:57
*** tssurya has quit IRC19:58
cdentfried_rice: okay, sorry, had to feed that cat and put dinner dishes aweay20:02
fried_riceHow domestic.20:03
fried_riceShould we do this here, or maybe snag an empty meeting channel so we don't consume this one?20:03
figleafstay here - I'm eavesdropping20:04
fried_riceight20:04
cdentagree with figleaf, probably best to spread the word as much as possible20:04
*** gouthamr has joined #openstack-nova20:04
*** jmlowe has quit IRC20:04
fried_riceI'll lead by saying I've got several balls in the air, including the 'granular' effort and a non-placement-related bp for ksa adapter stuff; but I'll be happy if just the nrp series gets shepherded while I'm away.20:05
fried_riceThat's the series starting at the hopefully-just-about-to-merge https://review.openstack.org/#/c/385693/9120:05
fried_riceThe culmination of this series will be the ability for the compute driver to tell the scheduler (almost) everything about how it wants to perceive its world, placement-wise.20:06
cdent“the scheduler”? or “placment” or both?20:07
fried_riceAs currently conceived in the top patch in the series (https://review.openstack.org/#/c/520246/12/nova/compute/resource_tracker.py) the resource tracker will hand off the ProviderTree to the compute driver; the compute driver will mangle it as it sees fit; and the resource tracker will flush any changes back to placement.20:07
*** lpetrut has joined #openstack-nova20:07
fried_ricecdent I should have said "resource tracker" instead of "scheduler".  The main point there is that the compute driver does not directly tell placement anything; that is brokered by resource tracker via ProviderTree.20:08
fried_rice...via report client.20:08
*** penick has quit IRC20:08
*** BryanS68 has quit IRC20:08
cdentokay, so the virt driver tells the compute manager and its resource tracker about what it knows about its world20:08
fried_riceYup.20:09
cdent20:09
fried_riceI think there are reasons we don't want virt talking to placement directly.  leakypipes and superdan have those in their heads.  But I think it has to do with narrowing the points of control.20:09
leakypipesfried_rice: ya20:10
*** jmlowe has joined #openstack-nova20:10
leakypipesfried_rice: the scheduler reporting client is a good bright-line interface for nova-compute with the placement API.20:10
fried_riceNod.20:11
cdentyeah, my concern was not so much about violating that line, rather trying to clarify the use of the term scheduler, which is ambiguous, but it’s clear now20:11
fried_riceAs currently written in the series, the first time around, resource tracker => report client will do get_provider_tree to bootstrap the ProviderTree, which will _ensure_resource_provider with the compute nodename & uuid.  That will create the entry in placement for the compute host RP and populate the local ProviderTree with that.20:11
fried_riceAnd then pass that ProviderTree to virt, via ComputeDriver.update_provider_tree.20:12
fried_riceSo it'll have just the compute RP as a root (no children), no inventory, no traits, no aggs, no sharing RPs.20:12
*** BryanS68 has joined #openstack-nova20:13
cdent20:13
fried_ricevirt does whatever it wants, as long as it's possible via the public ProviderTree interfaces.  This includes creating new root RPs (which will probably typically be sharing RPs, though we're not enforcing that) or child RPs (usually descendants of the compute RP, though we're not enforcing that).20:14
fried_riceIt also includes setting traits and inventories on any of those guys.20:14
fried_riceAnd assigning aggregate UUIDs.20:14
*** sbezverk has joined #openstack-nova20:14
fried_ricevirt is responsible for returning True or False from that method to let us know whether it changed anything.  (TBH, I'm not sure that's necessary, cause it's pretty easy for us to figure it out, and that might be better than trusting virt to get it right.)20:15
fried_riceAnyway, resource tracker gets that provider tree back and diffs it with what it passed in originally (which, I should have mentioned, is a copy of what it had built up); and then makes report client calls to push those changes back to the placement db.20:16
*** david-lyle has joined #openstack-nova20:16
cdentcan I pause you for a moment?20:16
fried_ricesho20:16
*** ChanServ has quit IRC20:17
*** lpetrut has quit IRC20:18
cdentthis is making sense, mostly, so far, except for one thing. Can you give me a scenario or two in which the provider tree will have more than one root? Is this for something like ironic, of clustering hypervisor or something else or something in addition to that?20:18
*** tssurya has joined #openstack-nova20:19
fried_riceI can't say what'll happen for ironic.  It's possible they'll have their root be their (I'm going to get the terminology wrong, but) compute "host" with children being the compute "nodes".  Or maybe the compute "nodes" will all be roots.  I dunno.20:19
fried_riceA real scenario for multiple roots is sharing RPs.20:20
*** tssurya has quit IRC20:20
fried_riceSo if I have a shared disk provider that my virt driver knows about, the compute manager won't know about it yet.20:20
openstackgerritMatt Riedemann proposed openstack/nova master: Add instance action record for backup instances  https://review.openstack.org/52367620:20
*** tssurya has joined #openstack-nova20:21
fried_ricevirt will populate the provider tree with that shared disk provider as a new root with the MISC_SHARES_VIA_AGGREGATE trait and tag both it and the compute RP with the aggregate UUID.20:21
cdentlet me know when you are EOF, so I can ask my followup, but I don’t want to do it if you are stil typing away20:21
fried_riceAnd the rt will have to handle the case where we go to create that guy in placement and find it already exists, because some other compute node that *also* shares with it already created it.20:21
fried_riceBy the same token, the various virts sharing a given sharing RP will have to make sure they use the same name and/or UUID (and we should probably document some rules for them doing that) so that we can in fact do that coordination.20:22
*** penick has joined #openstack-nova20:22
fried_riceWe've been saying we won't support sharing in Q, but we at least need to consider this stuff so we don't paint ourselves into a corner.20:22
fried_riceEOF20:22
*** huanxie has quit IRC20:24
*** ChanServ has joined #openstack-nova20:24
*** barjavel.freenode.net sets mode: +o ChanServ20:24
cdentHmm, okay. I see there are benefits to that model for sharing, especially for virtdrivers from our respective employers, but it surprises me a bit. I was imagining that a compute manager would only ever be responsble for managing stuff that it can be said to own, and ownership in this case is weird because of the -shared- aspect of things. over20:25
*** penick has quit IRC20:25
*** sahid has quit IRC20:26
fried_riceI think the flexibility of this model is good here, because it allows for some third party to manage the sharing RPs, but if such a third party doesn't exist for a given driver, it also allows for it all to be coordinated by the virt driver.20:26
*** huanxie has joined #openstack-nova20:26
fried_riceI can speak for PowerVM, when it comes to our shared storage pool impl, that there's no third party process that's going to be able to do that stuff (unless it's at the behest of the virt driver itself under the covers, which would pretty much violate the "virt driver not talking to placement" thing).20:27
fried_riceover20:27
fried_riceor I can proceed with brain dump if we don't need to discuss this aspect further at the moment.20:28
*** sridharg has quit IRC20:28
*** sbezverk has quit IRC20:29
cdentI’ll just make one more comment, as I agree that the flexibility is appropriate given constraints, and then we can move on: I’d rather see the virt drivers have license to talk to placement, especially since we can’t (and shouldn’t want) to control that anyway. All you need is the right keystone token. carry on my wayward son20:30
fried_riceYou'd have to take that up with not-me, for sure.20:30
* cdent nods20:31
fried_riceTo the point of whether virt can create roots, we could document (and even enforce) what virt drivers are and are not allowed to do to the ProviderTree, if we wanted to.  But note that the code that lets you create a new root in a ProviderTree is already merged :)20:31
*** sbezverk has joined #openstack-nova20:31
fried_riceSo anyway, I pretty much started with that, wrote that patch first, and then worked my way backwards to all the plumbing we need underneath it to make it workable.20:32
*** david-lyle has quit IRC20:32
* cdent nods20:32
fried_riceIt's in the neighborhood of 16 patches at the moment.  I think we only need one more :)20:32
fried_riceOh, two.20:33
fried_riceSo here's where I go into my "state of the series" address.20:33
* cdent sits comfortably20:34
fried_riceThe bottom-most WIP was only still WIP because I wanted to add some more meat to the functional test: https://review.openstack.org/#/c/521098/10/nova/tests/functional/api/openstack/placement/test_report_client.py@33220:34
fried_riceThat's a nice big functional test that creates a bunch of interconnected providers such as a (fairly complex) virt driver might, including inventory.  See ascii diagram at the top.20:35
* cdent nods20:36
fried_riceThe crux is then where we call get_provider_tree, and make sure what we get back has all the stuff in it.20:36
fried_riceBTW, the above story only talked about the first time we call into this stuff; the general case whenever we do update compute node (or whatever it's called in the periodic task) will be the opportunity for virt to tell us what, if anything, has changed in the provider world.  It can shuffle inventories, aggregates, etc.20:37
* cdent nods20:37
fried_riceOne would hope such changes to be rare to the point of almost-never, because virt doesn't manage allocations.20:37
fried_riceSo we're talking about if someone hotplugs storage or CoDs a bank of CPUs or something.20:37
fried_riceanyway...20:37
cdent(if virt could manage allocations, that would be cooler, for hybrid environments)20:38
fried_riceAs I was adding the code for that TODO at the bottom, I found that the traits were in fact not showing up for certain providers.20:38
* cdent nods20:38
fried_riceWhich turns out to be because the traits PUT is 400ing.20:38
*** Apoorva has joined #openstack-nova20:38
*** Apoorva has quit IRC20:38
cdentbut nobody can tell because of None?20:39
fried_riceNo20:39
* cdent shucks20:39
fried_riceBecause we don't have a report client method for creating traits.20:39
*** Apoorva has joined #openstack-nova20:39
fried_riceor aggregates20:39
*** penick has joined #openstack-nova20:39
fried_riceso I'm doing that directly in the test, and not checking the result.20:39
* cdent nods20:39
fried_riceSo that's one (or maybe two) still-needed patch(es).20:39
fried_riceWhile we're talking about None, there's another patch (possibly more than one) needed, which could actually be done outside of this series, to make sure we're raising where we should.20:40
fried_ricePer discussion yesterday, we only need to do it for methods where we don't always check the return20:41
fried_riceI identified that at least the one that gets inventories qualifies.20:41
fried_riceThere may be others.20:41
*** penick has quit IRC20:42
* cdent nods20:42
*** penick has joined #openstack-nova20:42
fried_riceNow, I think I identified a change that's needed to what's already in gerrit for the get_provider_tree method to make sure we're pulling the latest traits & aggs for all "associated" providers to populate the ProviderTree.  But I won't know until I figure out & fix whatever that PUT /traits deal is.20:43
fried_rice(not PUT /traits -- PUT /resource_provider/%u/traits or whatever - you get the idea)20:43
* cdent nods20:43
fried_riceOkay, next to-do: The report client method that diffs what virt driver returns and pushes those changes back to placement.  It'll go between the top two patches in the series, I think.  Working title for that method is update_from_provider_tree, per https://review.openstack.org/#/c/520246/12/nova/compute/resource_tracker.py@83920:45
*** penick has quit IRC20:45
*** dgonzalez has left #openstack-nova20:46
fried_riceThe last major thing that affects the series is how we handle generations, which we started talking about some this morning.20:46
*** yamamoto has joined #openstack-nova20:48
fried_riceHonestly, I would need to relook at pretty much the whole report client with that in mind to figure out what we're gonna need to do about it.20:48
fried_riceAbsent the proposed get-everything-about-a-provider API (https://review.openstack.org/#/c/521875/) we would have to do stuff like:20:49
cdentdo you have a summary statement of the the “generations problem”?20:49
fried_ricewhen we GET a provider's traits, that response payload contains the RP generation as known to the server.  If that doesn't match what we have in the local ProviderTree, we should force-refresh everything we know about that provider.20:50
cdentor more generally: any time we get a generation in a response, if it is not a match to us, refresh?20:51
fried_riceyeah, I think that sums it up at least to the 80/20 point :)20:52
*** yamamoto has quit IRC20:52
fried_riceAt the moment, my code that retrieves traits & aggs is ignoring the rp generation field in the response.20:52
*** nore_rabel has joined #openstack-nova20:53
* cdent nods20:53
fried_riceIn particular, what it's *not* doing is calling the set_*() method with the generation from that response, which would actually be the *wrong* thing to do I think.20:54
fried_riceBecause that would be updating the generation based on thing A that changed without taking into account that thing B (whose content we still have at the old generation level) might have also changed.20:54
* fried_rice takes notes for Dr. Seuss-esque book)20:55
*** itlinux has quit IRC20:55
cdentquite20:55
fried_riceBeyond all that, while restacking, there are some nice review comments to address from finucannot, giblet, figleaf, takashin...20:56
* cdent nods20:57
*** penick has joined #openstack-nova20:57
*** ttx has quit IRC20:57
*** ttx has joined #openstack-nova20:57
cdentIf you had to identify the scariest-here-be-dragons aspect, what would it be?20:57
fried_riceAnd writing up an executive summary, as you suggested in your email today, which could largely comprise text from the foregoing.20:57
* figleaf doesn't feel his comments were all that nice20:57
fried_riceScariest as in potential for largest amount of rework of stuff that's not already marked WIP?  The generation thing.20:58
*** itlinux has joined #openstack-nova20:58
cdentk20:59
cdentLet me see if I can sort of translate some things to do:21:00
cdenta) decode the last hour of the log to extract salient links, main points, action items21:01
*** moshele has quit IRC21:01
cdentb) attend to comments from reviews whilst restacking21:01
*** tasker has quit IRC21:01
cdentc) make the linked functional test, notably it’s ascii diagram “work” and make the code do that tree and allow that tree to change, and deal with generation changes21:01
*** pchavva has quit IRC21:02
*** sbezverk has quit IRC21:02
cdenteof21:03
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: mask InjectionInfo.admin_pass  https://review.openstack.org/52677221:03
cdent(since c) ended up being sort of d) and e) as well)21:03
fried_ricePretty much.  WRT (c), the only part of the diagram that the code doesn't "do" right is the traits on *some* providers.21:04
fried_riceWhich may be solved by resolving that 400, but I actually think there's more to it.  I'm leaving some code comments with more info.21:04
* cdent nods21:04
*** gouthamr has quit IRC21:07
fried_riceDone: https://review.openstack.org/#/c/521098/1021:08
cdentcool21:08
*** penick has quit IRC21:09
fried_riceOkay, I gotta run get the kids.  Realistically, I'll be working a few scattered hours over the course of the next week, so drop me an email if you want to set up some more discussion time.21:09
cdentI will endeavor to keep things moving. I suspect once I start breaking it it will make considerably more sense than now and happiness will abound and unicorns will leap and frolic etc21:09
figleafAny word from HR?21:09
figleafoops, wrong channel21:09
fried_riceYeah, don't tell HR I'm working.21:09
fried_riceThey're like, "you have HOW much vacation time left?  Yeah, you gotta go."21:10
figleafDon't tell them I'm not :)21:10
leakypipesfigleaf: I've reported you to HR.21:10
fried_riceleakypipes Dunno if you followed any/all of that, but would be interested to know whether you concur on the high points.21:11
figleafleakypipes: aw, c'mon - it was just a little hug21:11
leakypipesfigleaf: :)21:11
leakypipesfried_rice: I was not paying attention, no...21:11
fried_riceleakypipes And likewise, shoot me an email if we need to get together for an hour or whatever.21:11
cdentleakypipes: the plan is to replace the scheduler, placement, the compute managers with etcd watches21:12
*** fried_rice is now known as efried_cya_jan21:12
leakypipescdent: ah, cool. so k8s then.21:12
figleafleakypipes: running on Cassandra, of course21:12
cdentbefore christmas21:12
* efried_cya_jan waves21:12
figleafcya21:12
*** penick has joined #openstack-nova21:12
leakypipesefried_cya_jan: cya!21:12
cdenthappy holidays or whathaveyou efried_cya_jan21:12
efried_cya_janY'all too21:12
* leakypipes just finished up the PostgreSQL benchmark changes for that UUID vs. integer PK benchmark.21:13
cdenti assume it is glorious?21:13
cdentactually, instead of etcd, we’ve got new technology that inverts cgroups to create global datacentre-wide share process spaces, and you can make global consensual RAM21:14
leakypipescdent: very similar to Percona/MySQL21:14
leakypipescdent: results-wise.21:14
leakypipescdent: gonna run the full bench tonight while nothing else is running on my puter.21:14
leakypipescdent: global consensual RAM sounds kinky.21:15
cdentdoes a bit21:16
cdentI will avoid a google search21:16
cdentmy eyes are done21:19
cdentgoodnight all, have good weekends21:19
*** cdent has quit IRC21:19
*** david-lyle has joined #openstack-nova21:24
*** dave-mccowan has quit IRC21:25
*** smatzek has quit IRC21:27
*** david-lyle has quit IRC21:29
leakypipesand I'm signing off too... ciao folks.21:33
*** leakypipes has quit IRC21:33
*** hui has quit IRC21:37
*** nore_rabel has quit IRC21:38
*** Apoorva has quit IRC21:39
*** rcernin has joined #openstack-nova21:44
*** gbarros has joined #openstack-nova21:46
*** Apoorva has joined #openstack-nova21:48
*** yamamoto has joined #openstack-nova21:49
*** yamamoto has quit IRC21:54
*** edmondsw has joined #openstack-nova21:55
*** edmondsw has quit IRC21:56
*** smatzek has joined #openstack-nova21:57
*** tssurya has quit IRC21:57
*** ihrachys has joined #openstack-nova21:58
ihrachyswhat could be the reason that I get this in placement-api (and compute service also spills something like that)21:59
ihrachysPlacement API returning an error response: Unable to update inventory for resource provider 2c677702-34b2-4e93-9124-a512bb97230f: Invalid inventory for 'MEMORY_MB' on resource provider '2c677702-34b2-4e93-9124-a512bb97230f'. The reserved value is greater than or equal to total.21:59
ihrachysand the node can't boot instances, returning the usual No hosts found error. I suspect the placement api error is relevant.22:00
mriedemihrachys: check the reserved_host_memory_mb config option22:00
*** itlinux has quit IRC22:01
mriedemthe virt driver is saying there is x amount of total memory but the config is trying to reserve more than that for the host22:01
*** damien_r has joined #openstack-nova22:01
*** damien_r has quit IRC22:01
mriedemthere is probably something in the debug logs before that failure in nova-compute where it dumps what it's thinking the values are22:01
*** smatzek has quit IRC22:01
*** tidwellr has quit IRC22:03
*** rcernin has quit IRC22:03
*** rcernin has joined #openstack-nova22:03
ihrachysmriedem, reserved_host_memory_mb        = 409622:04
*** tidwellr has joined #openstack-nova22:04
ihrachysoh so it tries to lock all this memory?22:05
ihrachysI don't have it on the node, though I added some swap to make it running22:05
ihrachysmaybe I should try to reduce that22:07
melwittyeah it will make it unavailable https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.reserved_host_disk_mb22:07
melwittsorry that was disk. memory is https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.reserved_host_memory_mb22:08
*** tidwellr has quit IRC22:08
*** awaugama has quit IRC22:10
mriedemihrachys: this is the method that sets up the inventory data to send to placement https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L8322:10
*** felipemonteiro_ has quit IRC22:11
openstackgerritmelanie witt proposed openstack/nova master: Add API and nova-manage tests that use the NoopQuotaDriver  https://review.openstack.org/52627022:11
openstackgerritmelanie witt proposed openstack/nova master: Follow up on removing old-style quotas code  https://review.openstack.org/52423422:11
*** tbachman has quit IRC22:12
ihrachysmriedem, reduced to 512, restarted, and now it seems to have started the instance as expected. I guess the issue was because swap is not considered for the locking22:12
*** gouthamr has joined #openstack-nova22:12
ihrachysthanks mriedem  for the pointer to the option!22:12
mriedemyw22:14
*** itlinux has joined #openstack-nova22:17
*** itlinux has quit IRC22:18
openstackgerritmelanie witt proposed openstack/nova master: DNM: Test websocketproxy with TLS  https://review.openstack.org/51316022:21
*** gmann_afk is now known as gmann22:21
*** lyan has quit IRC22:21
*** tssurya has joined #openstack-nova22:24
*** baoli has quit IRC22:33
*** BryanS68 has quit IRC22:33
melwittmriedem: I had to rebase the noop quotas test coverage bc of your quota force bug fix shenanigans https://review.openstack.org/#/c/52627022:34
*** lyan has joined #openstack-nova22:34
*** baoli has joined #openstack-nova22:34
*** baoli has quit IRC22:35
*** vladikr has quit IRC22:36
*** amodi has quit IRC22:39
mriedemdone22:41
melwittthanks \o/22:41
*** lyan has quit IRC22:46
mriedemsuperdan: were you going to update https://review.openstack.org/#/c/507638/ or save it for monday or are you ok with me updating it?22:49
superdanmriedem: yeah I started locally but likely won't finish until monday22:49
superdanthat dependent one is in the gate atm too22:49
mriedemalright22:50
*** yamamoto has joined #openstack-nova22:50
mriedemwell i guess i'll just quit and spend quality time with my family22:50
superdangood call22:51
melwittsuperdan: were you gonna remove one of the dupe tests in this "unmap compute nodes" one or should I do it? https://review.openstack.org/#/c/52464122:52
superdanmelwitt: yes I was22:52
superdanso what you're both saying is, I suck, is that about right?22:52
melwittokay. no, of course not!22:52
*** figleaf is now known as edleafe22:53
*** yamamoto has quit IRC22:54
openstackgerritDan Smith proposed openstack/nova master: Unmap compute nodes when deleting host mapping  https://review.openstack.org/52464122:55
superdanmelwitt: ^22:55
melwittsaweet22:56
melwittmriedem: if you're still around ^22:57
superdanI think you can slam it, he was +2 before right?22:59
melwittyeah he was. I'm conservative about doing that so thought I'd ask in case he's still there23:00
*** huanxie has quit IRC23:00
superdanwell, no pressure from me23:01
superdanI'm spent for today anyway23:01
melwitt++23:01
*** burt has quit IRC23:08
*** esberglu has quit IRC23:10
*** baoli has joined #openstack-nova23:11
*** imacdonn has quit IRC23:14
*** imacdonn has joined #openstack-nova23:14
*** penick has quit IRC23:15
*** baoli has quit IRC23:16
*** tssurya has quit IRC23:21
*** purplerbot has quit IRC23:29
*** yamahata has joined #openstack-nova23:37
*** tbachman has joined #openstack-nova23:38
*** felipemonteiro has joined #openstack-nova23:39
openstackgerritMerged openstack/nova stable/pike: Add regression test for rebuild with new image doubling allocations  https://review.openstack.org/52321323:44
*** markvoelker has quit IRC23:49
*** markvoelker has joined #openstack-nova23:50
*** yamamoto has joined #openstack-nova23:51
*** markvoelker has quit IRC23:54
*** yamamoto has quit IRC23:55

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