*** slagle has quit IRC | 00:20 | |
*** brault has joined #openstack-nova | 00:24 | |
*** mdrabe has joined #openstack-nova | 00:28 | |
*** brault has quit IRC | 00:28 | |
*** purplerbot has quit IRC | 00:34 | |
*** purplerbot has joined #openstack-nova | 00:34 | |
*** tbachman has quit IRC | 00:41 | |
*** wxy has joined #openstack-nova | 00:42 | |
*** thorst has joined #openstack-nova | 00:45 | |
*** baoli has joined #openstack-nova | 00:47 | |
*** Apoorva_ has quit IRC | 00:47 | |
*** hieulq has joined #openstack-nova | 00:47 | |
*** tovin07 has joined #openstack-nova | 00:48 | |
*** mdrabe has quit IRC | 00:48 | |
*** thorst has quit IRC | 00:50 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (2/4) https://review.openstack.org/455568 | 00:51 |
---|---|---|
openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (3/4) https://review.openstack.org/455570 | 00:51 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (4/4) https://review.openstack.org/455573 | 00:52 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers-actions.inc https://review.openstack.org/454565 | 00:52 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971 | 00:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: cinder: add attachment_create method https://review.openstack.org/459111 | 00:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: cinder: add attachment_update method https://review.openstack.org/459113 | 00:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971 | 00:57 |
*** phuongnh has joined #openstack-nova | 00:57 | |
*** liusheng has quit IRC | 01:05 | |
*** kevinz has joined #openstack-nova | 01:11 | |
*** Nakato has quit IRC | 01:14 | |
*** hongbin has quit IRC | 01:17 | |
*** thorst has joined #openstack-nova | 01:21 | |
*** thorst has quit IRC | 01:22 | |
*** XueFeng has joined #openstack-nova | 01:27 | |
*** jamielennox is now known as jamielennox|away | 01:33 | |
*** yamamoto has quit IRC | 01:33 | |
*** yamamoto has joined #openstack-nova | 01:33 | |
*** nkorabli has joined #openstack-nova | 01:35 | |
*** iceyao has joined #openstack-nova | 01:36 | |
*** slaweq has joined #openstack-nova | 01:36 | |
*** amotoki has quit IRC | 01:36 | |
*** kaisers_ has joined #openstack-nova | 01:37 | |
*** nkorabli has quit IRC | 01:39 | |
*** kaisers has quit IRC | 01:40 | |
*** jamielennox|away is now known as jamielennox | 01:44 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Rework descriptions in os-hypervisors https://review.openstack.org/452030 | 01:47 |
*** thorst has joined #openstack-nova | 01:47 | |
*** thorst has quit IRC | 01:47 | |
*** zhenguo has joined #openstack-nova | 01:48 | |
*** slaweq has quit IRC | 01:55 | |
*** Nakato has joined #openstack-nova | 01:56 | |
*** dimtruck is now known as zz_dimtruck | 02:02 | |
*** zz_dimtruck is now known as dimtruck | 02:02 | |
*** yamamoto has quit IRC | 02:03 | |
*** imacdonn has quit IRC | 02:03 | |
*** fandi has quit IRC | 02:06 | |
*** fandi has joined #openstack-nova | 02:07 | |
*** lyan has quit IRC | 02:09 | |
*** dimtruck is now known as zz_dimtruck | 02:12 | |
*** imacdonn has joined #openstack-nova | 02:16 | |
*** thorst has joined #openstack-nova | 02:18 | |
*** armax has quit IRC | 02:19 | |
*** nicolasbock has quit IRC | 02:21 | |
*** baoli has quit IRC | 02:28 | |
*** yamamoto has joined #openstack-nova | 02:28 | |
*** takashin has left #openstack-nova | 02:36 | |
*** takashin has quit IRC | 02:36 | |
*** thorst has quit IRC | 02:36 | |
*** takashin has joined #openstack-nova | 02:36 | |
*** Guest76746 is now known as med_ | 02:37 | |
*** med_ has joined #openstack-nova | 02:38 | |
*** winston-d_ has joined #openstack-nova | 02:41 | |
*** yamamoto has quit IRC | 02:50 | |
*** weshay is now known as weshay_PTO | 02:51 | |
*** yamamoto has joined #openstack-nova | 02:53 | |
*** slaweq has joined #openstack-nova | 02:53 | |
*** takashin_ has joined #openstack-nova | 02:55 | |
*** Kevin_Zheng has joined #openstack-nova | 02:56 | |
*** takashin has quit IRC | 02:56 | |
*** takashin has joined #openstack-nova | 02:57 | |
*** slaweq has quit IRC | 02:57 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876 | 02:58 |
*** psachin has joined #openstack-nova | 02:59 | |
*** takashin_ has quit IRC | 03:00 | |
*** baoli has joined #openstack-nova | 03:03 | |
*** yamamoto has quit IRC | 03:04 | |
*** Shunli has joined #openstack-nova | 03:07 | |
*** salv-orlando has joined #openstack-nova | 03:07 | |
*** hongbin has joined #openstack-nova | 03:07 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Trivial Fix a typo https://review.openstack.org/459149 | 03:08 |
*** baoli has quit IRC | 03:08 | |
*** salv-orlando has quit IRC | 03:12 | |
*** ssurana has joined #openstack-nova | 03:14 | |
*** kaisers_ has quit IRC | 03:24 | |
*** kaisers has joined #openstack-nova | 03:24 | |
*** tovin07_ has joined #openstack-nova | 03:27 | |
*** thorst has joined #openstack-nova | 03:33 | |
*** dikonoor has joined #openstack-nova | 03:36 | |
*** dave-mccowan has quit IRC | 03:37 | |
*** thorst has quit IRC | 03:37 | |
*** abhi89 has joined #openstack-nova | 03:41 | |
*** kaisers has quit IRC | 03:44 | |
*** abhishek has joined #openstack-nova | 03:44 | |
*** abhi89 has quit IRC | 03:48 | |
*** diga has joined #openstack-nova | 04:00 | |
*** vks1 has joined #openstack-nova | 04:02 | |
*** thomasem_ has quit IRC | 04:04 | |
*** Dinesh_Bhor has joined #openstack-nova | 04:06 | |
*** yamamoto has joined #openstack-nova | 04:07 | |
*** salv-orlando has joined #openstack-nova | 04:08 | |
*** salv-orlando has quit IRC | 04:13 | |
*** slaweq has joined #openstack-nova | 04:14 | |
*** diga has quit IRC | 04:18 | |
*** adisky_ has joined #openstack-nova | 04:19 | |
*** slaweq has quit IRC | 04:19 | |
*** baoli has joined #openstack-nova | 04:30 | |
*** iceyao has quit IRC | 04:32 | |
*** dikonoor has quit IRC | 04:32 | |
*** thorst has joined #openstack-nova | 04:34 | |
*** baoli has quit IRC | 04:34 | |
*** XueFeng has quit IRC | 04:37 | |
*** thorst has quit IRC | 04:38 | |
*** hongbin has quit IRC | 04:39 | |
*** ayogi has joined #openstack-nova | 04:40 | |
*** sree has joined #openstack-nova | 04:42 | |
*** dikonoor has joined #openstack-nova | 04:43 | |
*** kaisers has joined #openstack-nova | 04:44 | |
*** xinliang has quit IRC | 04:44 | |
*** trinaths has joined #openstack-nova | 04:45 | |
*** oomichi has quit IRC | 04:46 | |
*** kaisers has quit IRC | 04:49 | |
*** Apoorva has joined #openstack-nova | 04:53 | |
*** salv-orlando has joined #openstack-nova | 04:57 | |
*** xinliang has joined #openstack-nova | 04:57 | |
*** xinliang has joined #openstack-nova | 04:57 | |
*** thomasem_ has joined #openstack-nova | 05:00 | |
*** nkorabli has joined #openstack-nova | 05:04 | |
*** abhishek has quit IRC | 05:05 | |
*** claudiub|2 has joined #openstack-nova | 05:11 | |
*** claudiub has joined #openstack-nova | 05:11 | |
*** gouthamr has joined #openstack-nova | 05:13 | |
*** gcb has joined #openstack-nova | 05:13 | |
*** marst has quit IRC | 05:16 | |
*** trinaths has left #openstack-nova | 05:17 | |
*** slaweq has joined #openstack-nova | 05:18 | |
*** slaweq has quit IRC | 05:19 | |
openstackgerrit | Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova. https://review.openstack.org/459166 | 05:22 |
*** ekuris has joined #openstack-nova | 05:23 | |
*** bkopilov has quit IRC | 05:26 | |
*** Apoorva has quit IRC | 05:27 | |
*** brault has joined #openstack-nova | 05:31 | |
*** thorst has joined #openstack-nova | 05:34 | |
*** udesale has joined #openstack-nova | 05:35 | |
*** fandi has quit IRC | 05:35 | |
*** brault has quit IRC | 05:36 | |
*** thorst has quit IRC | 05:39 | |
*** arunman has joined #openstack-nova | 05:39 | |
*** dikonoor has quit IRC | 05:43 | |
*** kaisers has joined #openstack-nova | 05:46 | |
*** iceyao has joined #openstack-nova | 05:46 | |
*** bkopilov has joined #openstack-nova | 05:48 | |
*** gouthamr has quit IRC | 05:49 | |
*** iceyao has quit IRC | 05:50 | |
*** cloud-liang has joined #openstack-nova | 05:51 | |
*** cloud-liang has left #openstack-nova | 05:51 | |
*** dikonoor has joined #openstack-nova | 05:51 | |
*** mdnadeem has joined #openstack-nova | 05:53 | |
*** cloud-liang has joined #openstack-nova | 05:54 | |
*** abhi89 has joined #openstack-nova | 05:56 | |
*** takashin has quit IRC | 06:09 | |
*** takashin has joined #openstack-nova | 06:11 | |
*** Dinesh_Bhor has quit IRC | 06:13 | |
*** Oku_OS-away is now known as Oku_OS | 06:13 | |
*** thomasem_ has quit IRC | 06:16 | |
*** slaweq has joined #openstack-nova | 06:20 | |
*** thomasem_ has joined #openstack-nova | 06:20 | |
*** ratailor has joined #openstack-nova | 06:21 | |
*** slaweq has quit IRC | 06:24 | |
*** ltomasbo|away is now known as ltomasbo | 06:25 | |
*** moshele has joined #openstack-nova | 06:25 | |
*** andreas_s has joined #openstack-nova | 06:27 | |
*** Dinesh_Bhor has joined #openstack-nova | 06:27 | |
*** huanxie has joined #openstack-nova | 06:29 | |
*** takashin_ has joined #openstack-nova | 06:30 | |
*** takashin has quit IRC | 06:32 | |
*** takashin_ has left #openstack-nova | 06:33 | |
*** takashin has joined #openstack-nova | 06:33 | |
*** thorst has joined #openstack-nova | 06:35 | |
*** david_1 has quit IRC | 06:36 | |
*** carthaca_1 has quit IRC | 06:36 | |
*** sapcc-bot1 has quit IRC | 06:36 | |
*** carthaca_ has joined #openstack-nova | 06:37 | |
*** sapcc-bot has joined #openstack-nova | 06:37 | |
*** david_3 has joined #openstack-nova | 06:37 | |
*** thorst has quit IRC | 06:39 | |
*** nkorabli has quit IRC | 06:42 | |
*** prateek has joined #openstack-nova | 06:46 | |
*** yongjiexu has joined #openstack-nova | 06:48 | |
*** salv-orlando has quit IRC | 06:50 | |
*** sridharg has joined #openstack-nova | 06:51 | |
*** iceyao has joined #openstack-nova | 06:53 | |
*** voelzmo has joined #openstack-nova | 06:53 | |
*** kaisers has quit IRC | 06:57 | |
*** iceyao has quit IRC | 06:57 | |
*** satyar has joined #openstack-nova | 06:58 | |
*** Qiming has quit IRC | 06:58 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Rework descriptions in os-hypervisors https://review.openstack.org/452030 | 06:59 |
*** markus_z has joined #openstack-nova | 07:02 | |
*** voelzmo has quit IRC | 07:02 | |
*** Qiming has joined #openstack-nova | 07:03 | |
*** bhagyashris has joined #openstack-nova | 07:10 | |
*** voelzmo has joined #openstack-nova | 07:12 | |
*** kiwi_rot has joined #openstack-nova | 07:13 | |
*** ralonsoh has joined #openstack-nova | 07:14 | |
*** sridharg has quit IRC | 07:16 | |
*** nkorabli has joined #openstack-nova | 07:18 | |
bhagyashris | Hi all, I need some guidance regarding, configuration: Is it possible to set single resource provider in multinode setup for nfs backend? How should I configure the single resource provider in multinode environment which uses the nfs backend? | 07:19 |
*** slaweq has joined #openstack-nova | 07:21 | |
*** sridharg has joined #openstack-nova | 07:21 | |
openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225 | 07:23 |
*** syjulian has quit IRC | 07:23 | |
*** pcaruana has joined #openstack-nova | 07:24 | |
*** syjulian has joined #openstack-nova | 07:24 | |
*** damien_r has joined #openstack-nova | 07:25 | |
*** ssurana has quit IRC | 07:26 | |
*** lpetrut has joined #openstack-nova | 07:26 | |
*** arunman has quit IRC | 07:26 | |
*** ssurana has joined #openstack-nova | 07:27 | |
*** damien_r has quit IRC | 07:29 | |
*** damien_r has joined #openstack-nova | 07:30 | |
*** ssurana has quit IRC | 07:31 | |
*** jaosorior has joined #openstack-nova | 07:33 | |
*** arunman has joined #openstack-nova | 07:34 | |
*** rmart04 has joined #openstack-nova | 07:34 | |
*** slaweq_ has joined #openstack-nova | 07:37 | |
*** thorst has joined #openstack-nova | 07:37 | |
*** claudiub|2 has quit IRC | 07:38 | |
*** claudiub has quit IRC | 07:38 | |
*** karthiks has joined #openstack-nova | 07:39 | |
*** rcernin has joined #openstack-nova | 07:43 | |
*** lpetrut has quit IRC | 07:43 | |
*** fragatina has quit IRC | 07:43 | |
*** slaweq has quit IRC | 07:44 | |
*** jpena|off is now known as jpena | 07:44 | |
*** fragatina has joined #openstack-nova | 07:45 | |
*** fragatina has quit IRC | 07:46 | |
*** fragatina has joined #openstack-nova | 07:46 | |
*** karthiks has quit IRC | 07:46 | |
*** iceyao has joined #openstack-nova | 07:48 | |
*** arunman has quit IRC | 07:49 | |
*** iceyao has quit IRC | 07:52 | |
*** moshele has quit IRC | 07:53 | |
*** kaisers has joined #openstack-nova | 07:53 | |
*** thorst has quit IRC | 07:56 | |
*** salv-orlando has joined #openstack-nova | 07:58 | |
*** karthiks has joined #openstack-nova | 07:59 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-nova | 08:00 | |
*** iceyao has joined #openstack-nova | 08:01 | |
openstackgerrit | jichenjc proposed openstack/nova-specs master: Add os-quota-class-sets-query-update https://review.openstack.org/459185 | 08:04 |
*** efoley__ has joined #openstack-nova | 08:11 | |
*** lpetrut has joined #openstack-nova | 08:12 | |
*** efoley_ has joined #openstack-nova | 08:16 | |
*** dixiaoli has joined #openstack-nova | 08:16 | |
*** moshele has joined #openstack-nova | 08:16 | |
*** amotoki has joined #openstack-nova | 08:18 | |
*** efoley__ has quit IRC | 08:19 | |
*** ircuser-1 has quit IRC | 08:20 | |
*** oanson has quit IRC | 08:23 | |
*** rcernin has quit IRC | 08:23 | |
*** oanson has joined #openstack-nova | 08:24 | |
*** yassine has joined #openstack-nova | 08:25 | |
*** yassine is now known as Guest55762 | 08:25 | |
moshele | sfinucan: hi | 08:26 |
sfinucan | moshele: Heya | 08:26 |
moshele | sfinucan: regarding https://review.openstack.org/#/c/398277/ so I am not sure that vif_type is the best place to document the ovs plugin | 08:28 |
*** ssurana has joined #openstack-nova | 08:29 | |
sfinucan | moshele: No? Would there be somewhere better suited? | 08:29 |
moshele | sfinucan: vif_type describe the version objects of the os-vif. I guess we should have also documentation per plugin ovs and lb | 08:31 |
*** sridharg has quit IRC | 08:33 | |
moshele | sean-k-mooney: what do you think ^ | 08:34 |
johnthetubaguy | there is the longer term plan around os-vif type binding | 08:34 |
johnthetubaguy | I was catch up with kevinbenton about that on friday I think | 08:34 |
*** ssurana has quit IRC | 08:35 | |
moshele | can you point me to the discussion? | 08:35 |
*** cdent has joined #openstack-nova | 08:35 | |
openstackgerrit | jichenjc proposed openstack/nova-specs master: Add os-quota-class-sets-query-update https://review.openstack.org/459185 | 08:35 |
johnthetubaguy | moshele: the wider context is implement the vif binding negotiation thing in the original os-vif spec, maybe thats not too relevant here, except for the versioning | 08:36 |
moshele | johnthetubaguy: ok I will read the os-vif spec | 08:39 |
sfinucan | moshele: The main thing I was looking for was a comparison of that particular VIF type to other VIF types | 08:39 |
sfinucan | I took a stab at documenting a glossary, but that doesn't really solve the problem here | 08:39 |
sfinucan | So if that's in the form of a new doc, so be it | 08:40 |
johnthetubaguy | sfinucan: docs are a good point here, I think that is slowing down the discussions with neutron | 08:40 |
johnthetubaguy | there is something that Nova calls to inspect the host to say what is supported, I guess we have to check the kernel to see if this new feature is supported | 08:41 |
johnthetubaguy | Nova then (eventually) will send to neutron the os-vif response to say what the host supports | 08:41 |
johnthetubaguy | neutron then picks from that what it thinks its best from its side of things | 08:41 |
johnthetubaguy | the port binding info is then returned as a serialized os-vif object, that needs to tell os-vif exactly what needs to get setup | 08:42 |
johnthetubaguy | or something like that (wibble inspect the instance and the host, etc) | 08:42 |
johnthetubaguy | its not clear how this all fits into that pattern, for me at least | 08:42 |
*** derekh has joined #openstack-nova | 08:42 | |
johnthetubaguy | sfinucan: is that the kinda stuff you were thinking, or is that orthogonal? | 08:43 |
moshele | johnthetubaguy: this is orthogonal because we to doument the plug/unplug of each vif plugin | 08:47 |
*** sridharg has joined #openstack-nova | 08:47 | |
moshele | johnthetubaguy: basiclly doumenting parts like https://github.com/openstack/os-vif/blob/master/vif_plug_ovs/ovs.py#L154-L174 as it become to match complex. sfinucan are we on the same page? | 08:50 |
*** arunman has joined #openstack-nova | 08:52 | |
johnthetubaguy | moshele: ah, understood, that certainly looks out of hand | 08:53 |
johnthetubaguy | moshele: I was kinda expecting each of those vif objects to do the thing they needed, rather than having massive If statement | 08:54 |
*** dikonoor has quit IRC | 08:55 | |
moshele | johnthetubaguy: the ovs-plugin just say it used with the OVS mechanism driver, but there are several ways to achieve binding (plain ovs/ hybrid with lb/DPDK/windows) each one is an if statement | 08:57 |
*** karthiks has quit IRC | 08:57 | |
sfinucan | moshele: Yeah, exactly | 08:58 |
sfinucan | As a start | 08:58 |
johnthetubaguy | moshele: yeah thats certainly the current state, that seems bad, you could use the object structure to remove the all the ifs I would have thought | 08:58 |
johnthetubaguy | I guess I was expecting those to all get advertised as separate things in some ways, just sharing a common set of utils | 08:58 |
sfinucan | johnthetubaguy: There's definitely good reason to go the way you're suggesting but that'd be a different series | 08:58 |
johnthetubaguy | sfinucan: oh, totally | 08:58 |
johnthetubaguy | just thinking out loud really | 08:58 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Rework descriptions in os-hypervisors https://review.openstack.org/452030 | 08:58 |
sfinucan | moshele: So if you wanted to add a document with description of that particular VIF driver, I'd be happy enough for now | 08:59 |
*** karthiks has joined #openstack-nova | 08:59 | |
sfinucan | We could collectively expand it out to all the other VIF types after the fact | 09:00 |
moshele | sfinucan: sure I will start something and let see where we go with it | 09:01 |
sfinucan | (y) Cheers :) | 09:01 |
*** karthiks has quit IRC | 09:05 | |
abhi89 | johnthetubaguy: Hi | 09:13 |
johnthetubaguy | abhi89: hi | 09:14 |
abhi89 | i have thought of a solution to https://review.openstack.org/#/c/455118/3 & commented after your comments.. can you please take a look & let me know about whether that would be the right way to proceed | 09:16 |
*** kaisers has quit IRC | 09:17 | |
*** sree_ has joined #openstack-nova | 09:19 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Always disconnect_volume after rebase failures https://review.openstack.org/458807 | 09:19 |
johnthetubaguy | abhi89: can we get a stack trace out the system that is seeing this error | 09:19 |
*** sree_ is now known as Guest10425 | 09:19 | |
johnthetubaguy | abhi89: your current patch still doesn't look like it would fix the problem, based on what is described | 09:19 |
*** kaisers has joined #openstack-nova | 09:19 | |
abhi89 | johnthetubaguy: stacktrace will have VolumeLimitExceeded() exception, i will attach the stacktrace also | 09:20 |
abhi89 | johnthetubaguy: i haven't made those changes to my current patch. i have just proposed a solution that i thought of | 09:21 |
johnthetubaguy | abhi89: right, thats what I expected, the code you have added is not reachable | 09:21 |
johnthetubaguy | abhi89: ah... sorry, let me read the comments not the code | 09:21 |
*** dikonoor has joined #openstack-nova | 09:21 | |
*** karthiks has joined #openstack-nova | 09:22 | |
abhi89 | yes.. please go through my comment of proposed solution once | 09:22 |
*** sree has quit IRC | 09:22 | |
johnthetubaguy | abhi89: we do need to translate the exception, its a common patter we have for all cinder errors. What I would do is just add additional information into the translated exception | 09:23 |
*** rmart04_ has joined #openstack-nova | 09:23 | |
*** lpetrut has quit IRC | 09:24 | |
*** rmart04 has quit IRC | 09:24 | |
*** rmart04_ is now known as rmart04 | 09:24 | |
sfinucan | johnthetubaguy: Could you stick these on your review queue? https://review.openstack.org/#/q/status:open+topic:bp/move-nova-cmds-to-cliff | 09:24 |
johnthetubaguy | abhi89: I would recommend you instead look at this line of code: https://github.com/openstack/nova/blob/5a6d4e089dca2515c0f7e083a8b903ac7a166773/nova/volume/cinder.py#L203 | 09:24 |
sfinucan | I'd like to get them merged this cycle so I can push on with that bp next cycle | 09:24 |
*** kaisers has quit IRC | 09:25 | |
*** kaisers has joined #openstack-nova | 09:26 | |
abhi89 | johnthetubaguy: yes. we can even add additional information to the translated exception at https://github.com/openstack/nova/blob/5a6d4e089dca2515c0f7e083a8b903ac7a166773/nova/volume/cinder.py#L203 | 09:26 |
johnthetubaguy | sfinucan: possibly, my queue is getting a bit silly right now, particularly given I don't have a job by the time the summit happens, AFAIK :( | 09:27 |
abhi89 | johnthetubaguy: i will try this out | 09:27 |
johnthetubaguy | abhi89: thanks, I think thats probably going to be easiest way to fix it | 09:27 |
abhi89 | johnthetubaguy: yes, thanks for your suggestion | 09:27 |
johnthetubaguy | abhi89: no problems, sorry I didn't add that on the first set of comments, I should have. | 09:28 |
johnthetubaguy | sfinucan: oh, wait, I see, its tiny | 09:28 |
*** rmart04 has quit IRC | 09:29 | |
*** rmart04 has joined #openstack-nova | 09:30 | |
*** kaisers has quit IRC | 09:30 | |
sfinucan | johnthetubaguy: Yeah they're tidy enough. If you've enough on your plate though then they certainly wait | 09:31 |
*** dikonoor has quit IRC | 09:32 | |
openstackgerrit | jichenjc proposed openstack/nova-specs master: Add os-quota-class-sets-query-update https://review.openstack.org/459185 | 09:38 |
*** takashin has left #openstack-nova | 09:39 | |
*** Shunli has quit IRC | 09:42 | |
*** pooja_jadhav has joined #openstack-nova | 09:42 | |
*** Guest55762 has quit IRC | 09:45 | |
*** sambetts|afk is now known as sambetts | 09:47 | |
*** salv-orl_ has joined #openstack-nova | 09:49 | |
*** satyar has quit IRC | 09:52 | |
*** salv-orlando has quit IRC | 09:52 | |
*** thorst has joined #openstack-nova | 09:53 | |
*** mvk has quit IRC | 09:54 | |
*** masber has quit IRC | 09:54 | |
*** dixiaoli_ has joined #openstack-nova | 09:54 | |
*** dixiaoli has quit IRC | 09:56 | |
*** thorst has quit IRC | 09:57 | |
*** nkorabli has quit IRC | 09:58 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types https://review.openstack.org/430608 | 10:01 |
*** Guest55762 has joined #openstack-nova | 10:02 | |
*** zenoway has joined #openstack-nova | 10:03 | |
*** tbachman has joined #openstack-nova | 10:06 | |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for limits endpoint instead of stevedore https://review.openstack.org/458805 | 10:06 |
*** nicolasbock has joined #openstack-nova | 10:06 | |
*** iceyao has quit IRC | 10:07 | |
*** karthiks has quit IRC | 10:09 | |
*** ociuhandu has quit IRC | 10:15 | |
*** ratailor has quit IRC | 10:17 | |
*** tovin07_ has quit IRC | 10:18 | |
bhagyashris | cdent: Hi, | 10:18 |
*** Jack_Iv has joined #openstack-nova | 10:20 | |
*** aloga has quit IRC | 10:21 | |
*** aloga has joined #openstack-nova | 10:21 | |
*** Guest10425 is now known as sree | 10:21 | |
cdent | hi bhagyashris, what's up? | 10:21 |
*** karthiks has joined #openstack-nova | 10:22 | |
*** mvk has joined #openstack-nova | 10:23 | |
*** Guest55762 has quit IRC | 10:25 | |
*** kaisers has joined #openstack-nova | 10:27 | |
*** udesale has quit IRC | 10:29 | |
*** sree has quit IRC | 10:29 | |
*** iceyao has joined #openstack-nova | 10:30 | |
*** rnoriega has quit IRC | 10:31 | |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for hosts endpoint instead of stevedore https://review.openstack.org/459231 | 10:31 |
*** ssurana has joined #openstack-nova | 10:31 | |
bhagyashris | cdent: Fine. I have one doubt. Actually I have configured the multinode setup and tried to use the nfs shared storage. so How can I used the single resource provider in multinode setup? | 10:32 |
cdent | bhagyashris: are you saying you want to track the nfs share as a resource provider? If so, that's not yet supported. It's still the case that the compute-node will report the disk to which it has access. Shared resource providers is something we hope to get done before the end of this current cycle | 10:34 |
cdent | bhagyashris: if you mean something else, what is it? | 10:34 |
*** kevinz has quit IRC | 10:35 | |
bhagyashris | cdent: yeah i want to track nfs share as a resource provider. | 10:35 |
*** ssurana has quit IRC | 10:36 | |
cdent | bhagyashris: there's a blueprint for that here, but progress is stalled while we work some other things out first: https://blueprints.launchpad.net/nova/+spec/shared-resources-pike | 10:36 |
*** oanson has quit IRC | 10:37 | |
bhagyashris | cdent: ok. means on current master we can not configure the shared disk resource provider for multinode set up right? | 10:37 |
cdent | you could, if you wanted, track the nfs share as a resource provider and manage its inventory with your own tooling, now, but that information wouldn't yet be used in scheduling decisions | 10:37 |
cdent | correct | 10:37 |
bhagyashris | cdent: ok i will go through the blueprint. | 10:37 |
*** rnoriega has joined #openstack-nova | 10:37 | |
*** oanson has joined #openstack-nova | 10:37 | |
*** sdague has joined #openstack-nova | 10:38 | |
bhagyashris | cdent: i look in to the blueprint regading the resource pool but not sure that the implementation is completed. | 10:38 |
bhagyashris | cent: thank you for information | 10:38 |
*** phuongnh has quit IRC | 10:42 | |
*** dikonoor has joined #openstack-nova | 10:42 | |
openstackgerrit | jichenjc proposed openstack/nova master: use plain routes list for hypervisor endpoint instead of stevedore https://review.openstack.org/459235 | 10:44 |
*** zenoway has quit IRC | 10:44 | |
*** ababich has joined #openstack-nova | 10:47 | |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for limits endpoint instead of stevedore https://review.openstack.org/458805 | 10:49 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for flavors-access endpoint instead of stevedore https://review.openstack.org/456589 | 10:49 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for '/servers' endpoint instead of stevedore https://review.openstack.org/445864 | 10:49 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549 | 10:49 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for hosts endpoint instead of stevedore https://review.openstack.org/459231 | 10:49 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for '/os-aggregates' endpoint instead of stevedore https://review.openstack.org/456825 | 10:49 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for flavors-extraspecs endpoint instead of stevedore https://review.openstack.org/456585 | 10:49 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for '/os-keypairs' endpoint instead of stevedore https://review.openstack.org/456478 | 10:49 |
openstackgerrit | jichenjc proposed openstack/nova master: use plain routes list for hypervisor endpoint instead of stevedore https://review.openstack.org/459235 | 10:49 |
*** zenoway has joined #openstack-nova | 10:49 | |
*** ociuhandu has joined #openstack-nova | 10:50 | |
*** Jack_Iv has quit IRC | 10:52 | |
*** Jack_Iv has joined #openstack-nova | 10:53 | |
*** thorst has joined #openstack-nova | 10:53 | |
*** kaisers has quit IRC | 10:54 | |
*** zenoway has quit IRC | 10:54 | |
*** iceyao has quit IRC | 10:54 | |
*** Jack_Iv has quit IRC | 10:57 | |
*** thorst has quit IRC | 10:58 | |
*** kaisers has joined #openstack-nova | 11:01 | |
*** zenoway has joined #openstack-nova | 11:01 | |
*** yongjiexu has quit IRC | 11:01 | |
*** yongjiexu has joined #openstack-nova | 11:01 | |
*** Jack_Iv has joined #openstack-nova | 11:02 | |
*** lucasagomes is now known as lucas-hungry | 11:06 | |
*** dave-mccowan has joined #openstack-nova | 11:09 | |
*** dikonoor has quit IRC | 11:11 | |
*** salv-orl_ has quit IRC | 11:15 | |
*** satyar has joined #openstack-nova | 11:16 | |
*** zhurong has joined #openstack-nova | 11:17 | |
*** yongjiexu has quit IRC | 11:21 | |
*** gszasz has joined #openstack-nova | 11:21 | |
*** smatzek has joined #openstack-nova | 11:22 | |
*** oanson has quit IRC | 11:28 | |
*** dikonoor has joined #openstack-nova | 11:29 | |
*** oanson has joined #openstack-nova | 11:29 | |
*** Jack_Iv has quit IRC | 11:31 | |
*** Jack_Iv has joined #openstack-nova | 11:31 | |
*** lpetrut has joined #openstack-nova | 11:38 | |
*** thorst has joined #openstack-nova | 11:42 | |
*** kevinz has joined #openstack-nova | 11:44 | |
*** slaweq has joined #openstack-nova | 11:45 | |
*** damien_r has quit IRC | 11:46 | |
*** lpetrut_ has joined #openstack-nova | 11:47 | |
*** dixiaoli_ has quit IRC | 11:48 | |
*** slaweq has quit IRC | 11:50 | |
*** rfolco has joined #openstack-nova | 11:51 | |
*** kevinz has quit IRC | 11:52 | |
*** kevinz has joined #openstack-nova | 11:53 | |
*** damien_r has joined #openstack-nova | 11:53 | |
*** hferenc has quit IRC | 11:55 | |
*** hferenc has joined #openstack-nova | 11:58 | |
*** zenoway has quit IRC | 11:58 | |
*** zenoway has joined #openstack-nova | 11:59 | |
*** nkrinner has joined #openstack-nova | 11:59 | |
*** iceyao has joined #openstack-nova | 11:59 | |
*** slagle has joined #openstack-nova | 12:00 | |
*** zenoway has quit IRC | 12:00 | |
*** salv-orlando has joined #openstack-nova | 12:00 | |
*** vks1 has quit IRC | 12:00 | |
*** zenoway has joined #openstack-nova | 12:00 | |
openstackgerrit | Moshe Levi proposed openstack/os-vif master: HW offload support for openvswitch https://review.openstack.org/398277 | 12:00 |
openstackgerrit | Moshe Levi proposed openstack/os-vif master: add documentation for vif plug ovs https://review.openstack.org/459258 | 12:00 |
*** lpetrut_ has quit IRC | 12:01 | |
*** jschlueter has quit IRC | 12:02 | |
*** brault has joined #openstack-nova | 12:03 | |
*** hshiina has joined #openstack-nova | 12:03 | |
*** lpetrut_ has joined #openstack-nova | 12:03 | |
*** lpetrut_ has quit IRC | 12:03 | |
*** lpetrut_ has joined #openstack-nova | 12:03 | |
*** nkrinner has quit IRC | 12:07 | |
*** lyan has joined #openstack-nova | 12:07 | |
*** yassine has joined #openstack-nova | 12:09 | |
*** arunman has quit IRC | 12:09 | |
*** yassine is now known as Guest31890 | 12:09 | |
*** lucas-hungry is now known as lucasagomes | 12:11 | |
*** winston-d_ has quit IRC | 12:13 | |
openstackgerrit | jichenjc proposed openstack/nova master: Add description for /consoles https://review.openstack.org/459266 | 12:14 |
*** slunkad has quit IRC | 12:18 | |
*** moshele has quit IRC | 12:19 | |
*** lpetrut_ has quit IRC | 12:19 | |
*** lpetrut_ has joined #openstack-nova | 12:19 | |
*** haplo37_ has quit IRC | 12:23 | |
*** kaisers has quit IRC | 12:24 | |
*** slunkad has joined #openstack-nova | 12:24 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types https://review.openstack.org/430608 | 12:25 |
*** edmondsw has joined #openstack-nova | 12:26 | |
*** liusheng has joined #openstack-nova | 12:27 | |
*** xyang1 has joined #openstack-nova | 12:29 | |
*** haplo37_ has joined #openstack-nova | 12:34 | |
*** mriedem has joined #openstack-nova | 12:34 | |
*** ssurana has joined #openstack-nova | 12:34 | |
*** pchavva has joined #openstack-nova | 12:34 | |
*** catintheroof has joined #openstack-nova | 12:37 | |
*** jpena is now known as jpena|lunch | 12:38 | |
*** ssurana has quit IRC | 12:38 | |
*** zhurong has quit IRC | 12:39 | |
*** dillaman has quit IRC | 12:42 | |
mriedem | lyarwood: i replied in https://review.openstack.org/#/c/456896/ - i'm not sure i get your concern though | 12:43 |
*** liverpooler has joined #openstack-nova | 12:44 | |
lyarwood | mriedem: morning, yeah I missed the optimisation, I just assumed we were doing this to allow non-local connectors to be passed in as a kwarg that we would default to None when we wanted to use the local connector. | 12:45 |
*** slaweq has joined #openstack-nova | 12:46 | |
*** salv-orlando has quit IRC | 12:47 | |
*** cdent has quit IRC | 12:47 | |
mriedem | lyarwood: when i looked at the change that were passing a connector in, none of them had a local connector | 12:49 |
mriedem | *was | 12:49 |
*** slaweq has quit IRC | 12:51 | |
*** ayogi has quit IRC | 12:51 | |
lyarwood | mriedem: into _terminate_volume_connections? Sorry I don't follow. | 12:52 |
mriedem | earlier version of the patch https://review.openstack.org/#/c/456896/3/nova/compute/manager.py@3615 | 12:52 |
mriedem | was going out of it's way to get the connector and then pass it in to justify the kwarg | 12:52 |
mriedem | i don't know why | 12:52 |
mriedem | so until we actually need a connector=None kwarg on _terminate_volume_connections, i've omitted that | 12:53 |
lyarwood | mriedem: yeah that didn't make sense but that's the local compute connector, thus my confusion above ;) | 12:53 |
*** cdent has joined #openstack-nova | 12:53 | |
lyarwood | mriedem: and yeah, until we have a stashed connector to pass in the kwarg doesn't make sense | 12:54 |
*** lpetrut_ has quit IRC | 12:54 | |
lyarwood | mriedem: I just missed the optimisation and assumed we wanted to add the kwarg early | 12:54 |
lyarwood | anyway, back to these LM failures on stable | 12:55 |
*** jdurgin has joined #openstack-nova | 12:55 | |
mriedem | lyarwood: so are you still -1 on https://review.openstack.org/#/c/456896/ ? | 12:55 |
lyarwood | mriedem: no, I'll clean that up now | 12:55 |
mriedem | thanks | 12:56 |
*** ociuhandu has quit IRC | 12:56 | |
*** lpetrut has quit IRC | 12:57 | |
mriedem | jgriffith: when you're around, i had some questions marked with TODOs in the swap volume change for using the new cinder attachment APIs https://review.openstack.org/#/c/456971/4/nova/compute/manager.py | 12:58 |
mriedem | like, will deleting an attachment mess up migrate_volume_completion on the cinder side? | 12:58 |
mriedem | https://review.openstack.org/#/c/456971/4/nova/compute/manager.py@4993 and https://review.openstack.org/#/c/456971/4/nova/compute/manager.py@5017 specifically | 12:59 |
lyarwood | mdbooth: https://review.openstack.org/#/c/458807/ updated if you have anytime this afternoon | 13:00 |
lyarwood | kashyap: ^ also | 13:00 |
* kashyap clicks | 13:00 | |
lyarwood | kashyap: could you also take a look at https://review.openstack.org/#/c/458854/1 | 13:00 |
kashyap | lyarwood: Struggling with something else at the moment :-( | 13:01 |
lyarwood | kashyap: np | 13:01 |
kashyap | lyarwood: But I didn't forget your ping here | 13:01 |
kashyap | lyarwood: Oh, the latter one I have the context in my head | 13:02 |
kashyap | Should be easy :-) | 13:02 |
*** cloud-liang has quit IRC | 13:02 | |
*** kencjohnston has quit IRC | 13:02 | |
kashyap | It's in 1.2.7; which is well above the min version we have. | 13:02 |
kashyap | Nice find! | 13:02 |
*** jerrygb has joined #openstack-nova | 13:02 | |
*** ralonsoh_ has joined #openstack-nova | 13:02 | |
*** ralonsoh has quit IRC | 13:04 | |
*** ralonsoh_ is now known as ralonsoh | 13:04 | |
*** shaner has quit IRC | 13:05 | |
kashyap | lyarwood: Done for the second one; the 'rebase' failures are from an iSCSI-related bug? | 13:06 |
lyarwood | kashyap: yup, from the SELinux issue | 13:07 |
kashyap | Yeah, at least the linked bug says so | 13:07 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971 | 13:07 |
*** shaner has joined #openstack-nova | 13:11 | |
*** mdrabe has joined #openstack-nova | 13:11 | |
*** cleong has joined #openstack-nova | 13:15 | |
lyarwood | http://logs.openstack.org/43/458843/1/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/145ed83/console.html#_2017-04-24_13_15_00_520308 - Can anyone comment on why we would want to skip block live migration tests here? | 13:17 |
lyarwood | ah nvm I think it's https://bugs.launchpad.net/nova/+bug/1398999 | 13:18 |
openstack | Launchpad bug 1398999 in nova (Ubuntu Trusty) "Block migrate with attached volumes copies volumes to themselves" [High,Triaged] | 13:18 |
*** jaypipes has joined #openstack-nova | 13:21 | |
jaypipes | cdent: thx much for the review on the get_inventory() for libvirt patch. I'll address your comments after a meeting in a few minutes. | 13:22 |
cdent | ✔ | 13:23 |
mriedem | lyarwood: we've never run those as far as i know | 13:23 |
mriedem | ah yeah because of that bug | 13:24 |
mriedem | in the config option help text | 13:24 |
*** trinaths has joined #openstack-nova | 13:25 | |
*** READ10 has joined #openstack-nova | 13:25 | |
*** kaisers has joined #openstack-nova | 13:26 | |
openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Add documentation for OVS plugin https://review.openstack.org/459258 | 13:26 |
openstackgerrit | Stephen Finucane proposed openstack/os-vif master: HW offload support for openvswitch https://review.openstack.org/398277 | 13:26 |
openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Add documentation for Linux Bridge plugin https://review.openstack.org/459291 | 13:26 |
*** moshele has joined #openstack-nova | 13:27 | |
*** amotoki has quit IRC | 13:27 | |
mriedem | lyarwood: looks like that bug is fixed https://review.openstack.org/#/c/227278/ | 13:27 |
mriedem | if you're using new enough libvirt | 13:27 |
mriedem | so we could probably actually turn that on in master now since we're using UCA | 13:28 |
*** lpetrut has joined #openstack-nova | 13:28 | |
mriedem | we're using libvirt 2.5.0 now | 13:28 |
*** baoli has joined #openstack-nova | 13:29 | |
*** Shashi has joined #openstack-nova | 13:29 | |
*** smatzek has quit IRC | 13:29 | |
kashyap | lyarwood: Just looking at the first change related to 'rebase' failure & dangling devices; so the disconnect_volume() is coming from os_brick (os_brick/initiator/connectors/iscsi.py) | 13:29 |
* kashyap responds on the review | 13:29 | |
lyarwood | mriedem: yeah just reading through the bug now and thinking the same | 13:29 |
mriedem | i'll post a change to the live migration job to enable that test and see what happens | 13:30 |
lyarwood | cool thanks | 13:30 |
lyarwood | kashyap: well it's calling down into os-brick to disconnect the new volume, yeah. | 13:30 |
Shashi | Hello, anyone run into nova issues with the error "AttributeError: 'module' object has no attribute 'VIR_MIGRATE_POSTCOPY'" | 13:30 |
*** crudo has joined #openstack-nova | 13:31 | |
Shashi | I could see n-cpu services are not running because of this error | 13:31 |
kashyap | lyarwood: Was just writing out loud for my own edification, please be a little gentle :-) | 13:31 |
kashyap | mriedem: About those QEMU assertion failures we were debugging on Friday night (for me at least), I brought it up on upstream QEMU list - http://lists.nongnu.org/archive/html/qemu-devel/2017-04/msg03923.html | 13:31 |
Shashi | facing this issue on stable/ocata by the way | 13:31 |
jaypipes | Shashi: see /topic. best to ask usage questions on the openstack@ mailing list. But that sounds like you need a more recent version of libvirt. | 13:31 |
mriedem | Shashi: you have an old libvirt-python | 13:32 |
kashyap | mriedem: A Huwawei dev points out that they seem to have a downstream-specific patch that fixes for them | 13:32 |
kashyap | Anyway, I suggested if they want to post it as an upstream backport for 2.8 stable - http://lists.nongnu.org/archive/html/qemu-devel/2017-04/msg04090.html | 13:33 |
Shashi | I understand if I remove exusting libvirt and install the new one should resolve this issue right? | 13:33 |
*** lpetrut has quit IRC | 13:34 | |
jaypipes | Shashi: the new python-libvirt package and a new libvirt package, yes. | 13:35 |
*** awaugama has joined #openstack-nova | 13:35 | |
Shashi | Thanks jaypipes | 13:35 |
jaypipes | np | 13:35 |
rfolco | Shashi, and make sure you clean up your wheels (.whl) for libvirt-python, force a rebuild. | 13:36 |
*** baoli has quit IRC | 13:36 | |
*** esberglu has joined #openstack-nova | 13:37 | |
*** trinaths has quit IRC | 13:37 | |
Shashi | ok rfolco | 13:37 |
*** jpena|lunch is now known as jpena | 13:40 | |
*** jdwidari has joined #openstack-nova | 13:41 | |
*** hongbin has joined #openstack-nova | 13:41 | |
*** trinaths has joined #openstack-nova | 13:43 | |
*** erhudy has joined #openstack-nova | 13:44 | |
*** moshele has quit IRC | 13:45 | |
figleaf | Scheduler subteam meeting in 15 minutes in #openstack-meeting-alt | 13:45 |
figleaf | oh, wait, it's not Friday anymore | 13:45 |
*** baoli has joined #openstack-nova | 13:45 | |
*** figleaf is now known as edleafe | 13:45 | |
Tahvok | Guys, how should I configure the apache2 placement vhost? | 13:46 |
Tahvok | Can someone give me a working example? | 13:46 |
*** superdan is now known as dansmith | 13:47 | |
*** slaweq has joined #openstack-nova | 13:47 | |
*** eharney has joined #openstack-nova | 13:48 | |
*** vks1 has joined #openstack-nova | 13:51 | |
*** kaisers has quit IRC | 13:51 | |
*** burt has joined #openstack-nova | 13:52 | |
efried | Tahvok I think you may want to ask that question in #openstack | 13:55 |
*** gouthamr has joined #openstack-nova | 13:57 | |
*** smatzek has joined #openstack-nova | 13:57 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Enable test_iscsi_volume in live migration job https://review.openstack.org/459316 | 13:57 |
mriedem | tdurakov: lyarwood: ^ let's see what happens | 13:57 |
*** gouthamr has quit IRC | 13:59 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Enable test_iscsi_volume in live migration job https://review.openstack.org/459316 | 14:00 |
*** sree has joined #openstack-nova | 14:01 | |
mriedem | kashyap: which version of qemu has native iscsi support? | 14:01 |
mriedem | would be interesting to see if we can test against https://bugs.launchpad.net/nova/+bug/1524898 again | 14:01 |
openstack | Launchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,Confirmed] | 14:01 |
kashyap | mriedem: I don't know top off my head, let me look | 14:01 |
bhagyashris | cdent: Hi, actually I am willing or curious to track the nfs share as a resource provider and manage its inventory with my own tooling as per you suggest me so How can I proceed Can you please give me some guide line or steps to do this? | 14:02 |
*** iceyao has quit IRC | 14:02 | |
*** crushil has joined #openstack-nova | 14:03 | |
cdent | bhagyashris: I can show you how it is described in the spec, but you'll need some tooling of your own, and the information won't be very useful for two reasons: | 14:04 |
cdent | nothing will be using the info for scheduling, and nothing will be automatically writing allocations when disk is used | 14:05 |
kashyap | mriedem: If you're talking about the built-in iSCSI driver, it's there since "eons" -- version 1.0 (as the first commit dates to 2011) -- http://git.qemu.org/?p=qemu.git;a=commitdiff;h=c589b24 | 14:05 |
cdent | bhagyashris: some of the info here may be useful: http://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/generic-resource-pools.html#scenario-1-shared-disk-storage-used-for-vm-disk-images | 14:06 |
bhagyashris | cdent:ok. | 14:06 |
*** arunman has joined #openstack-nova | 14:06 | |
mriedem | kashyap: it wasn't in the ubuntu package though | 14:06 |
mriedem | kashyap: https://specs.openstack.org/openstack/nova-specs/specs/kilo/implemented/qemu-built-in-iscsi-initiator.html | 14:06 |
mriedem | looks like it should be there now though http://packages.ubuntu.com/xenial/qemu-block-extra | 14:06 |
bhagyashris | cdent: yeah I have checked the generic-resource-pool spec long back ago. | 14:07 |
kashyap | mriedem: What is this 'qemu-block-extra'? | 14:07 |
kashyap | It's a separate package for Ubuntu? | 14:08 |
mriedem | yeah | 14:08 |
mriedem | i guess | 14:08 |
bhagyashris | cdent: thanks for information | 14:08 |
mriedem | kashyap: see https://bugs.launchpad.net/nova/+bug/1524898/comments/17 | 14:08 |
openstack | Launchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,Confirmed] | 14:08 |
*** psachin has quit IRC | 14:08 | |
* kashyap clicks | 14:08 | |
mriedem | kashyap: long ago i think danpb said that might not fix that bug, but it might provide better details when it fails if we're using native iscsi | 14:09 |
*** iceyao has joined #openstack-nova | 14:09 | |
*** coreywright has quit IRC | 14:09 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova-specs master: Claims in the scheduler https://review.openstack.org/437424 | 14:10 |
bauzas | jaypipes: edleafe: cdent: mriedem: dansmith: others ^ | 14:10 |
cdent | thanks bauzas | 14:10 |
bauzas | some help would be accepted for the wording :) | 14:10 |
*** ddeja has joined #openstack-nova | 14:10 | |
*** slaweq has quit IRC | 14:11 | |
*** iceyao has quit IRC | 14:11 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Enable test_volume_backed_live_migration with block migration https://review.openstack.org/459321 | 14:11 |
*** iceyao has joined #openstack-nova | 14:11 | |
*** baoli has quit IRC | 14:12 | |
*** ddeja has left #openstack-nova | 14:12 | |
*** baoli has joined #openstack-nova | 14:12 | |
*** kaisers has joined #openstack-nova | 14:13 | |
*** ociuhandu has joined #openstack-nova | 14:14 | |
*** zz_dimtruck is now known as dimtruck | 14:15 | |
*** baoli has quit IRC | 14:17 | |
*** arne_r has joined #openstack-nova | 14:18 | |
*** baoli has joined #openstack-nova | 14:20 | |
*** beekneemech is now known as bnemec | 14:20 | |
*** mlavalle has joined #openstack-nova | 14:21 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [WIP] Register osapi_compute when nova-api is wsgi https://review.openstack.org/457283 | 14:21 |
*** prateek has quit IRC | 14:23 | |
*** coreywright has joined #openstack-nova | 14:23 | |
*** felipemonteiro has joined #openstack-nova | 14:24 | |
*** ociuhandu has quit IRC | 14:24 | |
*** felipemonteiro has quit IRC | 14:25 | |
*** felipemonteiro has joined #openstack-nova | 14:25 | |
kashyap | mriedem: Yeah, I recall this bug, and I se you've just enabled the said test given the recent version bumps | 14:26 |
mriedem | yeah | 14:27 |
jaypipes | bauzas: done. | 14:28 |
*** amotoki has joined #openstack-nova | 14:28 | |
* bauzas bows to jaypipes | 14:28 | |
bauzas | jaypipes: thanks for the wording details, lovely appreciated | 14:29 |
bauzas | for the fun story, we have a 2nd election round happening when I land in BOS, I have 39% of chances to stay there :p | 14:30 |
*** iceyao has quit IRC | 14:32 | |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for limits endpoint instead of stevedore https://review.openstack.org/458805 | 14:33 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for flavors-access endpoint instead of stevedore https://review.openstack.org/456589 | 14:33 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for '/servers' endpoint instead of stevedore https://review.openstack.org/445864 | 14:33 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549 | 14:33 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for hosts endpoint instead of stevedore https://review.openstack.org/459231 | 14:33 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for '/os-aggregates' endpoint instead of stevedore https://review.openstack.org/456825 | 14:33 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for flavors-extraspecs endpoint instead of stevedore https://review.openstack.org/456585 | 14:33 |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for '/os-keypairs' endpoint instead of stevedore https://review.openstack.org/456478 | 14:33 |
openstackgerrit | jichenjc proposed openstack/nova master: use plain routes list for hypervisor endpoint instead of stevedore https://review.openstack.org/459235 | 14:33 |
*** amotoki has quit IRC | 14:33 | |
jaypipes | bauzas: heh, indeed... | 14:34 |
jaypipes | bauzas: when is the second round vote day? | 14:34 |
bauzas | jaypipes: May the 7th | 14:34 |
jaypipes | hmm. | 14:34 |
bauzas | I'd hope it would have been May the 4th... | 14:34 |
jaypipes | good luck! | 14:34 |
jaypipes | heh | 14:34 |
jaypipes | indeed. | 14:34 |
*** ekuris has quit IRC | 14:34 | |
*** lpetrut has joined #openstack-nova | 14:35 | |
bauzas | anyway, my wife will proxy my vote, so I'm fine with my mind | 14:35 |
*** trinaths has left #openstack-nova | 14:35 | |
*** trinaths1 has joined #openstack-nova | 14:36 | |
*** mdrabe_ has joined #openstack-nova | 14:38 | |
*** arne_r has quit IRC | 14:40 | |
*** Jack_Iv has quit IRC | 14:41 | |
*** mdrabe has quit IRC | 14:41 | |
*** Jack_Iv has joined #openstack-nova | 14:41 | |
*** chyka has joined #openstack-nova | 14:42 | |
*** trinaths1 has left #openstack-nova | 14:42 | |
*** yamahata_ has joined #openstack-nova | 14:43 | |
*** moshele has joined #openstack-nova | 14:43 | |
*** sree_ has joined #openstack-nova | 14:45 | |
*** sree_ is now known as Guest34850 | 14:46 | |
*** armax has joined #openstack-nova | 14:47 | |
*** sree has quit IRC | 14:47 | |
mriedem | alex_xu: Kevin_Zheng: it's very confusing but I think i know why you're trying to stash the tags on the BuildRequest in https://review.openstack.org/#/c/394321/36/nova/compute/api.py | 14:52 |
mriedem | i'm leaving a ton of comments :) | 14:52 |
mriedem | but i think we're doing this the hard way | 14:52 |
mriedem | and i think some of the code in the api for this is all dead now | 14:52 |
mriedem | so i'm going to clean it up | 14:52 |
mriedem | in a separate patch | 14:52 |
mriedem | i.e. i think this was a workaround until we were creating instances and bdms in conductor | 14:52 |
mriedem | i'll have dansmith take a look later too in case i'm missing something | 14:53 |
Kevin_Zheng | mriedem: cool thanks, I will check tomorrow | 14:53 |
*** iceyao has joined #openstack-nova | 14:53 | |
bauzas | mriedem: jaypipes: FWIW, I'm entering into the hell zone of deletes wrt claims | 14:54 |
mriedem | bauzas: fwiw, i spent 6 hours yesterday in the hell zone that is swap volume | 14:54 |
bauzas | mriedem: jaypipes: AFAICT, we currently unclaim the resources (ie. we DELETE allocations) only by the compute service, right ? | 14:54 |
jaypipes | bauzas: yes, AFAIK. | 14:55 |
bauzas | mriedem: jaypipes: so, what happens in the case of a local delete where we gently instance.delete() ? | 14:55 |
bauzas | do we DELETE /allocations too ? None from what I can tell | 14:55 |
jaypipes | bauzas: only in the periodic audit interval task (update_available_resource) | 14:55 |
mriedem | bauzas: we don't delete allocations for local delete, | 14:56 |
mriedem | remember i opened that bug? | 14:56 |
bauzas | jaypipes: mriedem: okay, so then should we keep the same behaviour ? | 14:56 |
mriedem | bauzas: and yes, the only place we delete allocations today is the scheduler report client via the RT | 14:56 |
bauzas | I mean, should we really not be concerned by DELETE allocations in the scheduler claims spec ? | 14:56 |
*** brault has quit IRC | 14:56 | |
bauzas | or we could do that earlier in the compute API worker | 14:56 |
bauzas | well, only if not soft-deletes, of course | 14:57 |
mriedem | the fact we don't cleanup allocations during local delete is a bug | 14:57 |
*** armax has quit IRC | 14:57 | |
*** lpetrut has quit IRC | 14:57 | |
mriedem | regardless of claims in the scheduler | 14:57 |
bauzas | mriedem: I vaguely remembered one bug, yup | 14:57 |
mriedem | bauzas: https://bugs.launchpad.net/nova/+bug/1679750 | 14:57 |
openstack | Launchpad bug 1679750 in OpenStack Compute (nova) "Allocations are not cleaned up in placement for instance 'local delete' case" [Medium,Confirmed] | 14:57 |
*** iceyao has quit IRC | 14:58 | |
bauzas | okay, so my question is | 14:58 |
mriedem | we talked about doing the cleanup in the api service, or leaving it to init_host on the compute, | 14:58 |
bauzas | should we modify the current behaviour of when we delete allocations, regardless of the above bug ? | 14:58 |
mriedem | i.e. if the db says we have deleted instances for this host when nova-compute starts up, then go through and attempt to delete their allocation records for that host | 14:58 |
mriedem | modify how? | 14:59 |
bauzas | or should we keep the compute service responsible for notifying placement ? | 14:59 |
bauzas | mriedem: call placement in the compute worker | 14:59 |
bauzas | mriedem: I mean the api worker | 14:59 |
bauzas | mriedem: before sending the RPC call to compute | 14:59 |
mriedem | i thought people wanted to do it in compute | 14:59 |
mriedem | hold up | 15:00 |
mriedem | you're conflating two things | 15:00 |
mriedem | 1. normal delete where we rpc cast to the compute to do the delete | 15:00 |
bauzas | right, whatever the delete type is | 15:00 |
mriedem | 2. local delete in the api where the compute service is down, or the instance is shelved offloaded | 15:00 |
bauzas | yup, I'm not conflating | 15:00 |
bauzas | I just wonder if we wanted to notify placement earlier than now | 15:01 |
mriedem | no | 15:01 |
mriedem | we don't want to tie up the api | 15:01 |
bauzas | okay, so I'll just wipe out the paragraph I wrote about deletes | 15:01 |
bauzas | because it's unrelated | 15:01 |
mriedem | we only do that in the api for local deletes, but there isn't clear agreement on deleting allocations in the api even in the local delete case | 15:01 |
mriedem | the options are in the bug report above | 15:01 |
bauzas | I'll just mention the bug as a necessary prereq | 15:01 |
mriedem | yeah the bug needs to be fixed regardless | 15:02 |
mriedem | and most likely backported | 15:02 |
*** adrian_otto has joined #openstack-nova | 15:02 | |
*** salv-orlando has joined #openstack-nova | 15:02 | |
bauzas | okay, thanks for cleaning up the mess in my brain | 15:03 |
*** mdnadeem has quit IRC | 15:03 | |
bauzas | I'm all clear | 15:03 |
bauzas | jaypipes: so, you told me we don't DELETE allocations right now but only when we self-heal ? | 15:05 |
openstackgerrit | Stephen Finucane proposed openstack/os-vif master: HW offload support for openvswitch https://review.openstack.org/398277 | 15:05 |
openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Add documentation for Linux Bridge plugin https://review.openstack.org/459291 | 15:05 |
bauzas | jaypipes: that's something that needs to change then | 15:05 |
*** crudo has left #openstack-nova | 15:05 | |
jaypipes | bauzas: no, sorry, I was referring to instances that were terminated but the compute host was down at the time. | 15:06 |
*** Guest34850 has quit IRC | 15:06 | |
*** jamesdenton has joined #openstack-nova | 15:06 | |
jaypipes | bauzas: in a "normal" DELETE /servers call, the allocations are indeed deleted. | 15:06 |
bauzas | okay, looking at code | 15:07 |
sean-k-mooney | sfinucan: moshele: on the vnic type i dont think this should be documented seperatly per plugin as they are common across all plugins | 15:07 |
sfinucan | sean-k-mooney: Have you seen the patches themselves | 15:08 |
sfinucan | They're common, yes, but not all VIF types are supported across all plugins | 15:08 |
sfinucan | I didn't grasp that nuance until this morning | 15:08 |
bauzas | jaypipes: mmm, can't find where we call DELETE /allocations in the compute manager, but only in the RT | 15:09 |
sean-k-mooney | sfinucan: no i havent atleast not lately. correct that is why im working on https://review.openstack.org/#/c/441590/ | 15:09 |
sean-k-mooney | i am hoping to implement the nova neutron negociation this cycle if i can ever finish the supprot for ovs-dpdk in kolla | 15:10 |
jaypipes | bauzas: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L716-L731 | 15:10 |
*** baoli has quit IRC | 15:10 | |
jaypipes | bauzas: that's called at the end of the manager's _delete_instance() method. | 15:10 |
bauzas | ok, seeing it noxw | 15:10 |
bauzas | we force a RT update | 15:10 |
bauzas | which will delete the allocation, gotcha | 15:11 |
jaypipes | bauzas: the _update_resource_tracker() call eventually calls the scheduler client. | 15:11 |
jaypipes | right. | 15:11 |
sean-k-mooney | sfinucan: the host_info object discribes what vif types are supported by each plugin and it will with that patch allso describe what port profiles are supporte db each vif_type per plugin | 15:11 |
bauzas | jaypipes: okay, no explicit allocation delete, RT handles that | 15:11 |
bauzas | jaypipes: thanks, so nothing needed from a scheduler claims spec perspective | 15:11 |
*** baoli has joined #openstack-nova | 15:11 | |
*** baoli has quit IRC | 15:11 | |
bauzas | jaypipes: except the local delete problem | 15:11 |
sean-k-mooney | sfinucan: this object will be serialsed into the neutron binding profile and read by the mechanisum drivers as part of the port binding call. | 15:12 |
*** Shashi has quit IRC | 15:12 | |
jaypipes | bauzas: no, that's still called for soft deletes. | 15:12 |
sfinucan | sean-k-mooney: Nice. I wasn't aware of host_info either | 15:12 |
bauzas | jaypipes: local delete != soft delete | 15:12 |
* sfinucan goes to see if _that's_ documented | 15:12 | |
jaypipes | bauzas: no? | 15:12 |
bauzas | jaypipes: if the RPC compute service is down, we local delete in the api worker | 15:12 |
*** kevinz has quit IRC | 15:12 | |
bauzas | totally unrelated to soft deletes, which are just a specific VM state | 15:12 |
jaypipes | bauzas: right. the api worker sets instance to SOFT_DELETED | 15:12 |
bauzas | no no | 15:13 |
jaypipes | no? | 15:13 |
bauzas | sec | 15:13 |
bauzas | yet another fun | 15:13 |
sfinucan | sean-k-mooney: Do you think there's value in documenting the built-in plugins, in that case, as done here https://review.openstack.org/#/c/459258/2 | 15:13 |
*** Oku_OS is now known as Oku_OS-away | 15:13 | |
bauzas | jaypipes: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2001 and https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2027-L2032 | 15:14 |
sfinucan | i.e. the plugins themselves along with a little summary of how a given VIF type works in the given plugin/networking backend | 15:14 |
sean-k-mooney | sfinucan: so the other part of this is that in the future the mapping we currently do between teh existing vif types and the os-vif ones will go away with neutron specifying plugin,vif_type and port prifile in a single os-vif object as a responce to nova in the port binding call | 15:14 |
sean-k-mooney | sfinucan: ill take a look at it now | 15:14 |
mriedem | melwitt: dansmith: some easy dead code cleanup in conductor https://review.openstack.org/#/c/434615/ | 15:14 |
bauzas | jaypipes: mriedem mentioned a bug report because we do delete the instance but we don't delete the allocation | 15:14 |
bauzas | jaypipes: https://bugs.launchpad.net/nova/+bug/1679750 | 15:14 |
openstack | Launchpad bug 1679750 in OpenStack Compute (nova) "Allocations are not cleaned up in placement for instance 'local delete' case" [Medium,Confirmed] | 15:14 |
bauzas | FWIW, I'll assign myself to the bug since it's related to the claims spec | 15:15 |
*** ssurana has joined #openstack-nova | 15:15 | |
mriedem | jaypipes: we don't soft delete in the api if the compute host is down | 15:15 |
*** baoli has joined #openstack-nova | 15:15 | |
mriedem | jaypipes: we send out notifications saying we're soft deleting, but really we don't | 15:15 |
mriedem | we hard delete that mofo | 15:15 |
sean-k-mooney | sfinucan: so i dont think we should have redefinition of the vif types per plugin but i would be ok with referencing the vif types that are definded globally | 15:15 |
mriedem | this has been keeping gibi awake at night | 15:15 |
sean-k-mooney | sfinucan: essentailly like you have done with the glossary | 15:16 |
sean-k-mooney | define each fo them as a term or link back to the singel definition. | 15:16 |
dansmith | mriedem: I don't think that's dead code | 15:16 |
mriedem | jaypipes: https://review.openstack.org/#/c/443764/1/nova/compute/api.py@2066 | 15:16 |
bauzas | oh man, just got an email asking me to take all my holidays before end of May | 15:16 |
bauzas | that's going to be legendary | 15:16 |
sfinucan | sean-k-mooney: Basic VIF definitions do exist - see 'doc/source/vif_types' | 15:16 |
sfinucan | bauzas: dibs! | 15:16 |
mriedem | dansmith: you can't upgrade the schema if you don't have cell0 in ocata | 15:17 |
sean-k-mooney | each plugin could discibe the the vif types it support but i would prefer to auto generate that documention form https://github.com/openstack/os-vif/blob/e133b8a3771ce6a49594af83b90586bdd88630ae/vif_plug_ovs/ovs.py#L67-L83 if we could | 15:17 |
bauzas | I'm always ashamed to speak about that here, but if I would take all my holidays, it could probably require me to stop working right after I come back from BOS :/ | 15:17 |
dansmith | mriedem: unless you have an empty database because you're installing ocata for the first time | 15:18 |
bauzas | like not working for May | 15:18 |
dansmith | mriedem: and if you miss putting that record in, you get a unhelpful error condition | 15:18 |
mriedem | dansmith: if you have any empty db, how are you creating instances? | 15:18 |
jaypipes | mriedem: lovely.. | 15:18 |
sean-k-mooney | sfinucan: any chance we could use something like your sphix click intergration to do taht | 15:18 |
sean-k-mooney | if not im sure i coudl script something up | 15:18 |
dansmith | mriedem: you had an empty database when applying the schema migrations, which made it past the cell0 check, then you start creating instances without doing your cell record setup | 15:18 |
mriedem | dansmith: this https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_migrations/migrate_repo/versions/030_require_cell_setup.py | 15:19 |
dansmith | mriedem: right but that doesn't do anything for a fresh install | 15:19 |
mriedem | oh the gd flavor thing | 15:19 |
mriedem | ffs | 15:19 |
*** dixiaoli has joined #openstack-nova | 15:19 | |
*** ssurana has quit IRC | 15:19 | |
mriedem | jaypipes: yes soft delete is a turd | 15:19 |
mriedem | which should be flushed | 15:19 |
mdbooth | +1 | 15:21 |
sean-k-mooney | sfinucan: basically in the docs job all we would need to do is load os-vif. loop over all the in tree plugins, call discribe and template our a descripts into a folder and then have sphinx auto include it in the docs that way its automatic and always up to date | 15:21 |
*** crushil has quit IRC | 15:21 | |
sfinucan | sean-k-mooney: I could but it might be overkill | 15:22 |
*** dixiaoli has quit IRC | 15:22 | |
sfinucan | Realistically, how often are those lines going to change | 15:22 |
sfinucan | and, iirc, the plan is not to add any more plugins in os-vif (they should live in separate repos) | 15:23 |
sean-k-mooney | not that often we could just make it a requirement for adding more vif_types or port profiles to existing plugins as part of ongoing reviews | 15:23 |
sean-k-mooney | yep | 15:23 |
mriedem | dansmith: ok left a reply - but if we don't have cell0 to bury, we still have the build requests to list 'instances' and delete that 'instance', but we don't update the vm_state on the build request to ERROR | 15:24 |
mriedem | so those 'instances' just get stuck in scheduling state until you delete them (i think) | 15:24 |
sean-k-mooney | more vif types will be added to both plugins though and more port poriles but it wont be a large overhead to manually update the docs. | 15:24 |
*** dixiaoli has joined #openstack-nova | 15:24 | |
sfinucan | sean-k-mooney: Yeah, I don't think it would be too much of a requirement | 15:25 |
sfinucan | docs usually help reviews anyway because they give way more context | 15:25 |
*** baoli has quit IRC | 15:26 | |
sean-k-mooney | sfinucan: one of the requests i got from the neutron folks was to allow os-vif to plug non vm interfaces so they could use it to plug the dhcp,l3 and tunnel ports when its intergrated into neutron | 15:26 |
sean-k-mooney | i opened some bugs for those with too do to fill them out later | 15:26 |
dansmith | mriedem: yep, but we have to have the big red warning I think | 15:27 |
*** dixiaoli has quit IRC | 15:28 | |
mriedem | yeah i suppose. | 15:28 |
sean-k-mooney | we will need about 5 or 6 new vif_types to cover that (patch,vif_pair,vxlan,gre,geneve) i think im missing one but basically same pattern as vms and form and os vif point of view we dont really care what uses the port anyway. | 15:28 |
mriedem | dansmith: if we wanted to get the instance into ERROR state, we'd have to deserialize the build_request.instance, set it's vm/task state and then update the build_request right? | 15:29 |
sfinucan | sean-k-mooney: nova wouldn't care about those new ones, right? | 15:29 |
*** david-lyle has joined #openstack-nova | 15:29 | |
dansmith | mriedem: I dunno if that would do it, but perhaps.. I'm not too concerned about that I guess, since this is early setup you failed to do, so API semantics aren't that big of a deal, but.. | 15:29 |
sean-k-mooney | on the one i was missing was tap. correct nova should not care though the tap vif type which is basically the existing generic vif type(i would proably just use that) could be used with calico | 15:30 |
*** sridharg has quit IRC | 15:30 | |
sfinucan | sean-k-mooney: Yup, sounds good to me so | 15:31 |
sfinucan | sean-k-mooney: Going back to those docs then, I could probably put a note in saying that those supported vif_types are exposed using that host_info object, right? | 15:32 |
sean-k-mooney | sfinucan: yes if you could make them a link or somehow include the previous defintions of the docs from the global definiton that would also be good | 15:33 |
sean-k-mooney | i would prefer not to have multiple definitions of the same vif_type that could get out of sync if sphix supports that | 15:33 |
sean-k-mooney | if not its fine we can just keep them in sync with reviews | 15:33 |
*** krtaylor has quit IRC | 15:34 | |
*** vks1 has quit IRC | 15:34 | |
mriedem | dansmith: fair enough | 15:34 |
mriedem | at some point if i care i might see how hard it is to reproduce and change the build request to error, but i might not care later either | 15:35 |
dansmith | heh | 15:36 |
*** gcb has quit IRC | 15:37 | |
sfinucan | sean-k-mooney: Ah, there is a difference. The general vif_type overview is provided in doc/source/vif_types | 15:37 |
sfinucan | The ones that we've added refer to how a given VIF is implemented for a given backend | 15:38 |
sean-k-mooney | sfinucan: thats the thing the vif_type is ment to be completly independent form that backend impelmtation | 15:38 |
mriedem | lyarwood: huzzah! https://review.openstack.org/#/c/459316/ | 15:38 |
mriedem | mtreinish: sdague: ^ test_iscsi_volume in action | 15:39 |
mriedem | after so many years | 15:39 |
sean-k-mooney | the vif_type refers to how the hypervisor virtualise the interface not how that is connected to the datapalne but i can see why you would want to describe that also | 15:39 |
sfinucan | sean-k-mooney: Yes and no. It should expose the same API, but it has to work differently under the hood | 15:39 |
*** baoli has joined #openstack-nova | 15:41 | |
*** zhurong has joined #openstack-nova | 15:41 | |
*** gouthamr has joined #openstack-nova | 15:42 | |
*** kaisers has quit IRC | 15:43 | |
*** slaweq has joined #openstack-nova | 15:46 | |
*** burt has quit IRC | 15:46 | |
lyarwood | mriedem: nice :) | 15:47 |
*** vks1 has joined #openstack-nova | 15:49 | |
*** salv-orl_ has joined #openstack-nova | 15:49 | |
*** jvgrant has joined #openstack-nova | 15:49 | |
*** zenoway has quit IRC | 15:49 | |
*** jvgrant has left #openstack-nova | 15:50 | |
*** zenoway has joined #openstack-nova | 15:50 | |
*** slaweq has quit IRC | 15:50 | |
*** kaisers has joined #openstack-nova | 15:51 | |
*** rmart04 has quit IRC | 15:51 | |
*** salv-orlando has quit IRC | 15:52 | |
*** dillaman has joined #openstack-nova | 15:53 | |
*** kevinz has joined #openstack-nova | 15:53 | |
*** armax has joined #openstack-nova | 15:54 | |
*** zenoway has quit IRC | 15:54 | |
*** moshele has quit IRC | 15:54 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove unused _create_block_device_mapping method from API https://review.openstack.org/459389 | 15:55 |
*** amotoki has joined #openstack-nova | 15:56 | |
*** zhurong has quit IRC | 15:57 | |
*** kaisers has quit IRC | 15:57 | |
jaypipes | vladikr, ralonsoh, sean-k-mooney, sfinucan, bauzas, edleafe and cdent: updated https://review.openstack.org/#/c/448283/. pls re-review at your convenience. | 15:57 |
*** slaweq has joined #openstack-nova | 15:57 | |
sfinucan | (y) | 15:58 |
ralonsoh | ralonsoh: in a moment | 15:58 |
*** andreas_s has quit IRC | 15:58 | |
ralonsoh | jaypipes: in a moment (pfff I'm talking to myself) | 15:59 |
*** kevinz has quit IRC | 15:59 | |
sean-k-mooney | jaypipes: running to meeting but will do after | 15:59 |
*** hshiina has quit IRC | 16:00 | |
*** gyee has joined #openstack-nova | 16:00 | |
sfinucan | jaypipes: nice easy ones for you here too https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bump-min-libvirt | 16:01 |
*** voelzmo has quit IRC | 16:02 | |
*** dikonoor has quit IRC | 16:03 | |
*** zhurong has joined #openstack-nova | 16:03 | |
edleafe | jaypipes: ack | 16:04 |
*** fragatina has quit IRC | 16:04 | |
jaypipes | sfinucan: done. | 16:04 |
sfinucan | ta | 16:05 |
*** abhi89 has quit IRC | 16:07 | |
*** zhurong has quit IRC | 16:07 | |
*** karthiks has quit IRC | 16:08 | |
*** marst has joined #openstack-nova | 16:09 | |
*** ericyoung has quit IRC | 16:10 | |
*** kfarr has joined #openstack-nova | 16:14 | |
openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Add documentation for OVS plugin https://review.openstack.org/459258 | 16:16 |
openstackgerrit | Stephen Finucane proposed openstack/os-vif master: HW offload support for openvswitch https://review.openstack.org/398277 | 16:16 |
openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Add documentation for Linux Bridge plugin https://review.openstack.org/459291 | 16:16 |
*** jerrygb has quit IRC | 16:16 | |
*** ltomasbo is now known as ltomasbo|away | 16:19 | |
*** Apoorva has joined #openstack-nova | 16:20 | |
*** Apoorva has quit IRC | 16:20 | |
*** Apoorva has joined #openstack-nova | 16:21 | |
*** Guest31890 has quit IRC | 16:21 | |
*** slaweq has quit IRC | 16:22 | |
*** damien_r has quit IRC | 16:23 | |
*** lucasagomes is now known as lucas-afk | 16:24 | |
*** annegentle has joined #openstack-nova | 16:24 | |
*** efried has quit IRC | 16:25 | |
*** salv-orl_ has quit IRC | 16:26 | |
mriedem | sfinucan: nice easy cleanup win here https://review.openstack.org/#/c/391597/ | 16:27 |
mriedem | and closes a bp to boot | 16:27 |
*** markus_z has quit IRC | 16:28 | |
*** dikonoor has joined #openstack-nova | 16:29 | |
*** fragatina has joined #openstack-nova | 16:29 | |
*** krtaylor has joined #openstack-nova | 16:31 | |
*** Jack_Iv has quit IRC | 16:32 | |
*** gyee has quit IRC | 16:32 | |
*** rmart04 has joined #openstack-nova | 16:33 | |
*** crushil has joined #openstack-nova | 16:33 | |
*** jerrygb has joined #openstack-nova | 16:33 | |
*** kfarr_ has joined #openstack-nova | 16:34 | |
*** jpena is now known as jpena|away | 16:37 | |
*** rmart04 has quit IRC | 16:38 | |
*** mvk has quit IRC | 16:39 | |
*** mtanino has joined #openstack-nova | 16:42 | |
*** arunman has quit IRC | 16:42 | |
*** vladikr has joined #openstack-nova | 16:42 | |
*** gyee has joined #openstack-nova | 16:42 | |
*** kaisers has joined #openstack-nova | 16:43 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove BuildRequest.block_device_mapping clone workaround https://review.openstack.org/459406 | 16:45 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove BuildRequest.block_device_mapping clone workaround https://review.openstack.org/459406 | 16:47 |
mriedem | dansmith: melwitt: ^ dredged up a lot of confusion over why we needed to do that in the first place https://review.openstack.org/#/c/394321/36/nova/compute/api.py@1056 | 16:48 |
mriedem | i don't want to do the same thing for tags | 16:48 |
mriedem | i think we just pass the original dirty tag list to conductor like we do for bdms and we should be fine | 16:49 |
*** gszasz has quit IRC | 16:51 | |
*** iceyao has joined #openstack-nova | 16:55 | |
*** arunman has joined #openstack-nova | 16:56 | |
*** kiwi_rot has quit IRC | 16:58 | |
*** jaosorior is now known as jaosorior_away | 16:59 | |
*** fragatin_ has joined #openstack-nova | 17:00 | |
*** iceyao has quit IRC | 17:00 | |
*** cuhler has joined #openstack-nova | 17:01 | |
*** derekh has quit IRC | 17:01 | |
cuhler | Is it a known problem that nova.tests.unit.virt.libvirt.volume.test_iscsi.LibvirtISCSIVolumeDriverTestCase.test_iscsiadm_discover_parsing is failing? | 17:02 |
*** fragatina has quit IRC | 17:02 | |
cuhler | It is holding up https://review.openstack.org/#/c/458688/ | 17:02 |
cuhler | Is someone looking at it already? | 17:03 |
*** kfarr_ has quit IRC | 17:04 | |
*** claudiub has joined #openstack-nova | 17:05 | |
*** adrian_otto1 has joined #openstack-nova | 17:06 | |
cuhler | The problem is that LibvirtISCSIVolumeDriver connector's _get_target_portals_from_iscsiadm_output method now returns a list instead of an array as it used to. | 17:06 |
cuhler | Either that's a bug or the unit test didn't get updated. | 17:06 |
*** arunman has quit IRC | 17:07 | |
*** burt has joined #openstack-nova | 17:09 | |
*** adrian_otto has quit IRC | 17:09 | |
*** efoley_ has quit IRC | 17:11 | |
*** prateek has joined #openstack-nova | 17:12 | |
*** kaisers has quit IRC | 17:13 | |
*** iceyao has joined #openstack-nova | 17:16 | |
*** slaweq has joined #openstack-nova | 17:18 | |
dansmith | mriedem: do you plan to write tests for that cleanup patch? | 17:18 |
*** mvk has joined #openstack-nova | 17:18 | |
dansmith | doesn't fail anything now and it'd be an odd assertion in isolation I guess | 17:18 |
openstackgerrit | Chris Dent proposed openstack/nova master: Add pbr-installed wsgi application for metadata api https://review.openstack.org/459413 | 17:19 |
*** yamahata_ has quit IRC | 17:20 | |
*** iceyao has quit IRC | 17:20 | |
*** Swami has joined #openstack-nova | 17:21 | |
*** ssurana has joined #openstack-nova | 17:23 | |
*** adrian_otto1 has quit IRC | 17:23 | |
*** ekuris has joined #openstack-nova | 17:24 | |
*** arunman has joined #openstack-nova | 17:25 | |
*** kaisers has joined #openstack-nova | 17:26 | |
*** ralonsoh has quit IRC | 17:28 | |
*** vks1 has quit IRC | 17:29 | |
*** arunman has quit IRC | 17:30 | |
*** mtanino has quit IRC | 17:32 | |
rfolco | jaypipes, start using new delete inventories method implies in chaning put to delete in https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L625 - sounds correct ? I am just wondering where is the delete_inventory for a single inventory. | 17:34 |
*** Swami has quit IRC | 17:38 | |
*** salv-orlando has joined #openstack-nova | 17:38 | |
sean-k-mooney | jaypipes: https://review.openstack.org/#/c/448283/5 looks good to me in it current form | 17:38 |
mriedem | dansmith: i didn't plan on it no | 17:38 |
mriedem | functional tests passing should be sufficient | 17:39 |
*** Jack_Iv has joined #openstack-nova | 17:39 | |
dansmith | okay | 17:40 |
mriedem | dansmith: unrelated, but looking at https://review.openstack.org/#/c/457338/ i didn't even think about how server groups could have members in different cells | 17:41 |
mriedem | at least for anti-affinity | 17:41 |
dansmith | yup | 17:41 |
*** dikonoor has quit IRC | 17:41 | |
*** gjayavelu has joined #openstack-nova | 17:43 | |
*** jpena|away is now known as jpena | 17:45 | |
*** fragatin_ has quit IRC | 17:45 | |
*** fragatina has joined #openstack-nova | 17:46 | |
*** fragatina has quit IRC | 17:51 | |
*** cdent has quit IRC | 17:52 | |
*** prateek has quit IRC | 17:54 | |
*** amotoki has quit IRC | 17:54 | |
*** cuhler has quit IRC | 17:55 | |
mriedem | dansmith: ok comments in that patch | 17:55 |
mriedem | -1 for a test thing | 17:55 |
*** abhi89 has joined #openstack-nova | 17:55 | |
dansmith | thanks, will look in a sec | 17:55 |
*** READ10 has quit IRC | 17:56 | |
*** yamahata_ has joined #openstack-nova | 17:56 | |
edleafe | rfolco: there isn't a way to delete a single type of inventory for a RP. You should use set_inventory_for_provider() instead, and only include the inventory for the types that the RP has. | 17:57 |
rfolco | edleafe, that explains, thank you. Also, I am using a specific error message (previously returning db api error) when 409 conflict for inventory in use... but there is no way to emulate it in gabbits since they run serially. The only emulation possible is bad generation id. Any thoughts ? | 18:00 |
sdague | mriedem: you were too hopeful that the uuid thing wasn't a problem :) | 18:00 |
*** catintheroof has quit IRC | 18:01 | |
edleafe | rfolco: you would allocate against that inventory, checking for success. Then the next test would try to delete that same inventory, which should raise the 409 | 18:01 |
mriedem | sdague: the jsonschema thing? | 18:02 |
sdague | mriedem: yep | 18:02 |
mriedem | i was looking for an easy button | 18:02 |
sdague | I responded | 18:02 |
rfolco | edleafe, cool, will check on allocation then, thx++ | 18:03 |
sdague | the format isn't jsonschema native, it just means it calls a dedicated validating function | 18:03 |
mriedem | sdague: ha | 18:05 |
mriedem | sob | 18:05 |
mriedem | i see your reply and raise you | 18:06 |
sdague | mriedem: yeh, we all see different parts of the elephant | 18:06 |
*** crushil has quit IRC | 18:07 | |
*** Swami has joined #openstack-nova | 18:13 | |
*** mtanino has joined #openstack-nova | 18:13 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: remove redundant and broken iscsi volume test https://review.openstack.org/459420 | 18:14 |
mriedem | ^ fixes cuhler's issue pointed out earlier, and unblocks us from using brick 1.12.0 | 18:14 |
smcginnis | Following the docs.o.o install instructions for Ubuntu. Getting this error :The requested URL /placement/resource_providers was not found on this server" | 18:15 |
smcginnis | Any pointers? | 18:15 |
mriedem | smcginnis: hello new user, please see the channel topic and gtfo | 18:16 |
mriedem | :) | 18:16 |
mriedem | smcginnis: where are you getting the error? which service? | 18:16 |
smcginnis | mriedem: Just remember that when you want us to adopt placement usage too. :P | 18:16 |
smcginnis | mriedem: In the nova-compute log when trying to create a first test instance. | 18:16 |
*** kaisers has quit IRC | 18:17 | |
smcginnis | Actually, regular intervals it looks like. | 18:17 |
mriedem | if compute couldn't talk to placement it shouldn't have started up in the first place | 18:17 |
*** kaisers has joined #openstack-nova | 18:17 | |
mriedem | yeah, it should be spewing errors | 18:17 |
mriedem | are you able to make curl requests to placement from your compute node? | 18:17 |
*** toure is now known as toure|bbl | 18:17 | |
smcginnis | From this: Failed to create resource provider record in placement API | 18:17 |
mriedem | smcginnis: source the same credentials that are in nova.conf on your compute host, and then create a token for those creds and try a curl request to the same placement endpoint url in the service catalog | 18:17 |
mriedem | to see if it's compute misconfig or something else | 18:18 |
mriedem | also, is the placement service logging anything? | 18:18 |
mriedem | it runs under apache | 18:18 |
smcginnis | Where does that go? | 18:18 |
mriedem | /var/log/httpd? i think it has to be configured | 18:19 |
mriedem | devstack puts it into it's own log file | 18:19 |
smcginnis | Ah, /var/log/apache/nova_placement* | 18:19 |
smcginnis | Hmm, "nova" placement. Yeah, good luck getting me to use this. | 18:19 |
smcginnis | jk | 18:19 |
mriedem | https://github.com/openstack-dev/devstack/blob/master/lib/placement#L168 | 18:19 |
*** iceyao has joined #openstack-nova | 18:20 | |
mriedem | yeah, i think the ubuntu packagers were throwing nova in the name since that's where the code is sourced from | 18:20 |
mriedem | which is going to be a fun packaging rename later | 18:20 |
smcginnis | mriedem: Actual deployment looks like it puts them in nova_placement_error.log and nova_placement_access.log on Ubuntu. | 18:20 |
smcginnis | Nothing in either file though. | 18:21 |
mriedem | is it running? | 18:21 |
smcginnis | mriedem: Nope. Looks like the packages must not configure Apache correctly. | 18:22 |
smcginnis | mriedem: Loaded: not-found (Reason: No such file or directory) | 18:22 |
smcginnis | OK, go the clue I needed. I'll dig from here. | 18:22 |
smcginnis | Taking it this is not a known issue though. | 18:22 |
mriedem | i had heard of people complaining that the ubuntu packages were messed up | 18:22 |
mriedem | at least while they were working on the docs, | 18:22 |
mriedem | if there was a known issue, i'd think it would have been capture in the install guide though, but i've been out of the loop there | 18:22 |
mriedem | cdent might know but he's gone | 18:23 |
smcginnis | mriedem: If I find anything I'll try to update the docs. Thanks. | 18:23 |
smcginnis | :q | 18:23 |
*** abhi89 has quit IRC | 18:23 | |
*** iceyao has quit IRC | 18:24 | |
*** moshele has joined #openstack-nova | 18:27 | |
*** ircuser-1 has joined #openstack-nova | 18:27 | |
*** crushil has joined #openstack-nova | 18:28 | |
*** Matias has quit IRC | 18:28 | |
*** adisky_ has quit IRC | 18:29 | |
mriedem | sdague: can we get this into stable/ocata to unblock the LM job there? https://review.openstack.org/#/c/458843/ | 18:30 |
*** moshele has quit IRC | 18:30 | |
*** fragatina has joined #openstack-nova | 18:32 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Make server groups api aware of multiple cells for membership https://review.openstack.org/457338 | 18:35 |
openstackgerrit | Dan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path https://review.openstack.org/458636 | 18:35 |
openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Move to proper target_cell calling convention https://review.openstack.org/458634 | 18:35 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635 | 18:35 |
openstackgerrit | Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | 18:35 |
openstackgerrit | Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487 | 18:35 |
openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | 18:35 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | 18:35 |
*** ZZelle_ has joined #openstack-nova | 18:37 | |
*** catintheroof has joined #openstack-nova | 18:38 | |
*** cdent has joined #openstack-nova | 18:39 | |
smcginnis | mriedem: FYI, Ubuntu packages put the placement stuff under port 8778. | 18:43 |
smcginnis | mriedem: I seem to have that resolved now, but it's failing to find resource types when updating, etc. | 18:44 |
smcginnis | mriedem: Is there an installation step that needs to be run again to get resource info populated in placement? | 18:44 |
mriedem | resource types? | 18:44 |
*** kaisers has quit IRC | 18:44 | |
mriedem | got an error message? | 18:44 |
mriedem | or log | 18:44 |
smcginnis | Failed to retrieve filtered list of resource providers from placement API for filters | 18:45 |
smcginnis | Resource could not be found. | 18:45 |
*** salv-orlando has quit IRC | 18:45 | |
dansmith | mriedem: <3 | 18:45 |
mriedem | dansmith: that's my way of paying you back for bugging you all morning | 18:45 |
dansmith | heh | 18:46 |
*** efried has joined #openstack-nova | 18:46 | |
mriedem | smcginnis: hmm, you might not have the resource classes, or any resource providers, loaded into placement | 18:46 |
mriedem | resource provider == compute node | 18:46 |
mriedem | smcginnis: so now that placement is working, are you seeing things populating it in the nova-compute logs? | 18:47 |
mriedem | smcginnis: the update_available_resource periodic task in the compute service will update placement | 18:47 |
mriedem | every 1 min | 18:47 |
smcginnis | Failed to create resource provider record in placement API for UUID e3c2178a-8fc6-47d7-9ee1-c01a067213aa. Got 404: 404 Not Found | 18:47 |
mriedem | hmm wtf | 18:48 |
mriedem | smcginnis: nothing in the placement logs? | 18:49 |
*** kaisers has joined #openstack-nova | 18:49 | |
*** jpena is now known as jpena|away | 18:49 | |
smcginnis | mriedem: Just a bunch of 404s. | 18:50 |
mriedem | i don't see where we'd ever return a 404 when trying to POST /resource_providers | 18:50 |
mriedem | unless you're not making it to placement? | 18:50 |
mriedem | can you make a curl request against the placement endpoint in the service catalog? | 18:50 |
mriedem | i wonder if you're getting an endpoint not found from keystone or something | 18:50 |
smcginnis | I get : {"error": {"message": "The request you have made requires authentication.", "code": 401, "title": "Unauthorized"}} | 18:51 |
mriedem | ok check your nova.conf [placement] section | 18:51 |
mriedem | on the compute host | 18:51 |
mriedem | well, you'll need a token to curl to placement | 18:51 |
mriedem | did you pass a token? | 18:52 |
smcginnis | mriedem: No, was just seeing if I could even hit it. | 18:52 |
mriedem | source the credentials that you have in nova.conf under the [placement] group using the normal OS_* variables | 18:52 |
mriedem | get a token and pass that to the curl request headers | 18:53 |
smcginnis | mriedem: Happen to have an example of that curl command line? | 18:54 |
mriedem | yeah sec | 18:54 |
mriedem | hmm seems the docs have changed for the api guide and now i've lost my reference | 18:56 |
mriedem | here we go https://developer.openstack.org/api-guide/quick-start/api-quick-start.html | 18:57 |
mriedem | OS_TOKEN=$(openstack token issue -c id -f value) | 18:57 |
*** abhi89 has joined #openstack-nova | 18:57 | |
mriedem | is what i use to get the token and store into a variable | 18:58 |
smcginnis | OK, I see X-Auth-Token... | 18:58 |
mriedem | curl -s -H "X-Auth-Token: $OS_TOKEN" <whatever your placement endpoint is in the service catalog> | 18:58 |
mriedem | "openstack catalog list" to get the service catalog | 18:59 |
smcginnis | mriedem: Still 404: The resource could not be found. | 18:59 |
mriedem | are you trying to hit it on port 8778 ? | 19:00 |
smcginnis | yes | 19:01 |
smcginnis | mriedem: I wonder if there is something else wrong with the apache config the Ubuntu package has. | 19:02 |
mriedem | i assume there is | 19:02 |
mriedem | but i'm like a baby lamb when it comes to apache config | 19:02 |
*** toure|bbl is now known as toure | 19:02 | |
mriedem | but i see cdent snuck back in.... | 19:03 |
*** jamesdenton has quit IRC | 19:03 | |
cdent | uh oh | 19:03 |
cdent | that placement not found business is usually the result of a missing <Directory /path/to/the/wsgi/script> clause | 19:04 |
smcginnis | cdent: Do you happen to know of issues with the way Ubuntu packages set up placement? | 19:04 |
smcginnis | cdent: They set up a virtual host on *:8778 | 19:04 |
cdent | yeah, that's okay as long as the service catalog is right but without the Directory clause the script is not accessible | 19:04 |
cdent | lemme locate something, one sec | 19:04 |
smcginnis | cdent: They have WSGIScriptAlias / /usr/bin/nova-placement-api | 19:05 |
smcginnis | cdent: Then <Directory /usr/bin> | 19:05 |
cdent | is that inside our outside the virtual host? | 19:05 |
cdent | s/our/or/ | 19:05 |
smcginnis | Inside. | 19:05 |
cdent | try moving it outside | 19:06 |
cdent | I seem to recall that being one of several fixes (there's been a slew of bugs and changes related to this and I don't think the actually released package has caught up) | 19:06 |
mriedem | https://docs.openstack.org/ocata/install-guide-ubuntu/nova-controller-install.html doesn't mention anything about known issues | 19:06 |
mriedem | or apache config at all, i guess because the install guide relies on the packages being ok | 19:07 |
cdent | mriedem: exactly, there's been some gaps | 19:07 |
smcginnis | Here's the full current Apache config that the package includes if you're interested: http://paste.openstack.org/show/607726/ | 19:08 |
cdent | yeah, other than the directory clause needing to be top level that looks okay | 19:08 |
mriedem | i only see 2 reported bugs for placement in the nova ubuntu packges on LP | 19:09 |
mriedem | https://bugs.launchpad.net/ubuntu/+source/nova/+bug/1671509 | 19:09 |
openstack | Launchpad bug 1671509 in Ubuntu Cloud Archive "placement api configuration in apache configure CustomLog incorrectly" [High,Fix committed] - Assigned to Corey Bryant (corey.bryant) | 19:09 |
mriedem | https://bugs.launchpad.net/ubuntu/+source/nova/+bug/1674465 | 19:09 |
openstack | Launchpad bug 1674465 in nova (Ubuntu) "wsgi scripts shouldn't grant on /usr/bin" [Undecided,Confirmed] | 19:09 |
mriedem | neither of those sound like what you're hitting | 19:10 |
cdent | strange I swear I had talked to corey about this, but I suppose it could have been one of the rdo people as they have had similar problems | 19:11 |
smcginnis | Nope, still getting 404's after moving the Directory piece out of the VirtualServer. | 19:11 |
mriedem | smcginnis: did you restart it? | 19:11 |
smcginnis | s/VirtualServer/VirtualHost/ | 19:11 |
smcginnis | yeah, restarted apache, then tried restarting the host. | 19:11 |
*** smatzek has quit IRC | 19:12 | |
cdent | smcginnis: what's your curl? | 19:14 |
smcginnis | curl -s -H "X-Auth-Token: $OS_TOKEN" http://192.168.1.230:8778/placement | 19:15 |
*** smatzek has joined #openstack-nova | 19:15 | |
cdent | smcginnis: oh, yeah, drop the 'placement' | 19:16 |
cdent | when it is on a port, there's no prefix | 19:16 |
*** karimb has joined #openstack-nova | 19:16 | |
cdent | or change the WSGIScriptAlias from / to /placement | 19:16 |
* cdent sighs | 19:16 | |
smcginnis | Shernuf: {"versions": [{"min_version": "1.0", "max_version": "1.4", "id": "v1.0"}]} | 19:16 |
* cdent sighs again | 19:16 | |
cdent | there's a docs bug which discusses that the service catalog entry is wrong when on a port | 19:17 |
smcginnis | cdent: So if I just change my catalog endpoint to not include /placement, are there going to be an issues with code expecting that in the path? | 19:17 |
cdent | lemme see if I can find that | 19:17 |
cdent | the code always uses the service catalog, so should be no problems | 19:17 |
mriedem | smcginnis: we don't put that in code anywhere | 19:17 |
mriedem | right | 19:17 |
smcginnis | Good, that's what I was hoping. | 19:17 |
mriedem | gah | 19:18 |
mriedem | url | http://controller/placement | 19:18 |
mriedem | ^ is in the install guide | 19:18 |
cdent | smcginnis mriedem https://bugs.launchpad.net/openstack-manuals/+bug/1683585 | 19:18 |
openstack | Launchpad bug 1683585 in openstack-manuals "Placement endpoint configuration incorrect as written" [High,Confirmed] - Assigned to Brian Moss (bmoss) | 19:18 |
mriedem | cdent: is that what you're referring to? | 19:18 |
smcginnis | mriedem: yep | 19:18 |
*** satyar has quit IRC | 19:18 | |
smcginnis | Or Ubuntu shouldn't be putting it on a different port. | 19:19 |
cdent | there's a lot of confusion on how to manage the wsgi-based stuff under apache | 19:19 |
cdent | which I (incorrectly) had assumed was a known thing | 19:19 |
mriedem | coreycb: are you still with canonical? | 19:20 |
*** karimb has quit IRC | 19:20 | |
coreycb | mriedem, hey, yes | 19:21 |
mriedem | coreycb: can you comment on the state of the nova-placement-api package wrt https://bugs.launchpad.net/openstack-manuals/+bug/1683585 / | 19:21 |
openstack | Launchpad bug 1683585 in openstack-manuals "Placement endpoint configuration incorrect as written" [High,Confirmed] - Assigned to Brian Moss (bmoss) | 19:21 |
mriedem | ? | 19:21 |
coreycb | mriedem, sure, taking a look | 19:21 |
mriedem | smcginnis: sounds like you should be able to do the workaround in that bug report by deleting your endpoints and recreating them with the 8778 port | 19:22 |
*** iceyao has joined #openstack-nova | 19:22 | |
smcginnis | mriedem: Just finished recreating them, let's see how this goes. | 19:22 |
* mriedem crosses fingers | 19:22 | |
*** jamesdenton has joined #openstack-nova | 19:22 | |
coreycb | smcginnis, i've always created them with the port numbers, i need to what chris was talking about | 19:24 |
coreycb | that is, i need to look into | 19:24 |
cdent | coreycb: I think the fundamental issue was that I said (in a review of the install guide): in the long term we want to use prefixes like /placement instead of ports | 19:25 |
cdent | but that got translated into changing the docs | 19:25 |
smcginnis | coreycb: Great. Yeah, I was just following the docs.o.o ubuntu instructions, and that doesn't work right now. | 19:25 |
cdent | causing a mismatch with the packges | 19:25 |
smcginnis | Hmm, not I see this in the scheduler log: The placement API endpoint not found. Placement is optional in Newton, but required in Ocata. Please enable the placement service before upgrading. | 19:26 |
*** iceyao has quit IRC | 19:26 | |
mriedem | smcginnis: you're getting an endpoint not found error from ksa now | 19:26 |
mriedem | smcginnis: the service type and endpoint name are 'placement' right? | 19:26 |
coreycb | cdent, yep | 19:27 |
mriedem | cdent: unless the ubuntu packages are going to change to take the port number out of the apache config it seems we should get the install guide updated to create the endpoint with the port number | 19:27 |
cdent | mriedem: yes | 19:28 |
coreycb | mriedem, cdent: yes agreed | 19:28 |
cdent | that's pretty much where asettle and I landed when talking about it last week | 19:28 |
mriedem | if canonical changed the apache config in a new package release, that would be confusing (at least for ocata) | 19:28 |
mriedem | you could probably mirate the config somehow at install/update time for the new package, but... | 19:28 |
mriedem | *migrate | 19:28 |
smcginnis | Well, if our long term goal is to get rid of ports, shouldn't we make sure new services aren't getting set up that way by default? | 19:28 |
mriedem | smcginnis: sure, but then canonical has to change the config in their package and provide a migration i'd think | 19:29 |
mriedem | which is probably a PITA after ocata GA'ed | 19:29 |
coreycb | mriedem, smcginnis: yeah we'd definitely want to do that on a release boundary if we did | 19:29 |
smcginnis | mriedem: OK, instance still failing because it's filtering out all my one hosts. I should be able to figure it out from here though. Placement seems to be sorted. | 19:32 |
smcginnis | mriedem, cdent, coreycb: Thanks for the help. | 19:32 |
coreycb | smcginnis, maybe cells v2 is needed next | 19:32 |
smcginnis | coreycb: Any docs.o.o instruction issues with that? I followed the docs on that one too. | 19:33 |
coreycb | smcginnis, not that i know of | 19:33 |
smcginnis | coreycb: OK, I'll let you know if I find otherwise. | 19:34 |
mriedem | yes | 19:34 |
coreycb | smcginnis, ok thanks | 19:34 |
mriedem | it's in the compute install guide | 19:34 |
mriedem | you have to discover the host | 19:34 |
mriedem | smcginnis: https://docs.openstack.org/ocata/install-guide-ubuntu/nova-compute-install.html | 19:34 |
smcginnis | mriedem: Yep, did that. | 19:34 |
mriedem | coreycb: if you can comment in that bug i can move forward with updating the install guide | 19:34 |
smcginnis | mriedem: Unless that needs to be done again since placement was hosed? | 19:34 |
mriedem | basically, we create the placement endpoint in keystone with ports for ocata | 19:34 |
mriedem | and we could reconsider package changes for the apache config in pike | 19:35 |
*** dimtruck is now known as zz_dimtruck | 19:35 | |
mriedem | smcginnis: is the nova-compute log showing that it's creating the resource provider in placement now? | 19:35 |
*** salv-orlando has joined #openstack-nova | 19:35 | |
smcginnis | mriedem: Created resource provider record via placement API for resource provider with UUID | 19:36 |
mriedem | ok nice | 19:36 |
mriedem | smcginnis: are you using a flavor that's too big or something? | 19:36 |
mriedem | for your compute node? | 19:36 |
*** dane-fichter has joined #openstack-nova | 19:36 | |
coreycb | mriedem, done | 19:36 |
mriedem | coreycb: thanks | 19:36 |
smcginnis | mriedem: Nope, tiniest one. Enabled debug now, so I can see what's going on with the filtering. | 19:37 |
smcginnis | mriedem: Well, whatever it was, restarting to enable debug seems to have fixed the issue. Got an ACTIVE running instance. | 19:38 |
smcginnis | mriedem: I'll go back in my own corner now. Thanks! | 19:38 |
mriedem | hmm | 19:38 |
mriedem | maybe needed to wait for the inventory to get updated in the compute periodic interval | 19:39 |
mriedem | or something was stashed in the scheduler worker | 19:39 |
mriedem | and restarting it flushed that out so we checked again | 19:39 |
*** baoli has quit IRC | 19:39 | |
smcginnis | Yeah, I don't think I really changed anything, so maybe just needed to wait or restart. | 19:39 |
*** baoli has joined #openstack-nova | 19:40 | |
*** jerrygb has quit IRC | 19:42 | |
*** damien_r has joined #openstack-nova | 19:45 | |
sean-k-mooney | mriedem: hi, im just heading home but i know you had some question regarding the intel cis. ill try and catch up with you later in the week and answer what i can. | 19:45 |
mriedem | sean-k-mooney: ok | 19:46 |
*** karimb has joined #openstack-nova | 19:47 | |
* cdent sneaks back out | 19:52 | |
*** cdent has quit IRC | 19:52 | |
*** sree has joined #openstack-nova | 19:55 | |
*** jamesdenton has quit IRC | 19:57 | |
*** ijw has quit IRC | 19:58 | |
*** jamesdenton has joined #openstack-nova | 19:58 | |
*** sree has quit IRC | 20:00 | |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP/PoC: nova.utils.get_service_url(group) https://review.openstack.org/458257 | 20:00 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate Multinic, floatingip action and os-virtual-interface API https://review.openstack.org/457181 | 20:00 |
*** sambetts is now known as sambetts|afk | 20:01 | |
*** pcaruana has quit IRC | 20:01 | |
*** baoli has quit IRC | 20:01 | |
*** baoli has joined #openstack-nova | 20:02 | |
mriedem | claudiub: are you around/ | 20:02 |
mriedem | ? | 20:02 |
claudiub | o/ | 20:02 |
mriedem | claudiub: can you get https://review.openstack.org/#/c/458843/ so we can unblock stable/ocata changes? | 20:03 |
claudiub | ack | 20:03 |
mriedem | thanks | 20:03 |
*** harlowja has quit IRC | 20:03 | |
*** dtp has joined #openstack-nova | 20:03 | |
*** dane-fichter has quit IRC | 20:05 | |
claudiub | mriedem: can you also push this in? :D https://review.openstack.org/#/c/445540/ | 20:07 |
mriedem | i'll take a look | 20:07 |
*** MasterOfBugs has joined #openstack-nova | 20:08 | |
mriedem | no unit tests huh | 20:12 |
mriedem | nice | 20:12 |
mriedem | real nice | 20:12 |
mriedem | claudiub: approved. let me know if you need me to do a patch release for stable/ocata os-vif | 20:13 |
claudiub | that would actually help. :) | 20:13 |
*** jamielennox is now known as jamielennox|away | 20:13 | |
mriedem | ok i'll submit one | 20:14 |
claudiub | anyways, just an fyi, there is a silent hyper-v + ovs ci running on neutron. currently collecting history, and checking for stability. in time, it will replace the current neutron hyper-v ci. | 20:15 |
mriedem | ok | 20:15 |
mriedem | btw, i'm not sure about this https://review.openstack.org/#/c/431683/3/requirements.txt | 20:15 |
*** damien_r1 has joined #openstack-nova | 20:16 | |
*** jerrygb has joined #openstack-nova | 20:17 | |
mriedem | looks like we missed ovo 1.17.0 in os-vif before the final ocata release | 20:17 |
mriedem | if we release that in stable/ocata we need to bump the minor version, which could conflict with what's on master | 20:17 |
claudiub | yeah we did. the last requirements updates before that in os-vif was in november | 20:18 |
claudiub | and we need that pbr update too | 20:18 |
mriedem | we could separate out the pbr update | 20:18 |
claudiub | in ocata, all projects are <2.0.0 | 20:18 |
mriedem | we haven't done an os-vif release in pike yet, and the last release for os-vif in ocata was 1.4.0 | 20:19 |
mriedem | we could bring the ovo change in and release it in stable/ocata as 1.5.0, | 20:19 |
mriedem | but that's kind of odd | 20:19 |
claudiub | true, but it works, imo. | 20:19 |
mriedem | unless we actually need it on stable/ocata, which we don't, i think we should just ignore that one and do pbr separately | 20:19 |
*** jamielennox|away is now known as jamielennox | 20:20 | |
claudiub | hm, i was going to say that upper constraints are being installed anyways, but apparently in ocata they aren't for os-vif | 20:20 |
*** damien_r has quit IRC | 20:20 | |
*** jerrygb has quit IRC | 20:21 | |
claudiub | that is actually going to be a problem in the future | 20:21 |
claudiub | as the requirements will eventually start to conflict at some point | 20:21 |
mriedem | hmm yeah | 20:21 |
claudiub | it isn't in master either | 20:21 |
mriedem | jaypipes: is there any reason not to be using upper-constraints in os-vif tox.ini? | 20:22 |
jaypipes | mriedem: no, just an oversight likely. | 20:22 |
mriedem | claudiub: here is the pbr sync by itself https://review.openstack.org/#/c/459442/ | 20:26 |
*** liverpooler has quit IRC | 20:27 | |
claudiub | left comment | 20:30 |
*** zz_dimtruck is now known as dimtruck | 20:31 | |
mriedem | claudiub: replied. that's not what the bot does though. | 20:32 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/451049 | 20:32 |
claudiub | hm, weird. I've seen the bot do it one time though a long time ago. | 20:33 |
*** smatzek has quit IRC | 20:34 | |
*** crushil has quit IRC | 20:34 | |
*** karimb has quit IRC | 20:37 | |
*** kfarr has quit IRC | 20:38 | |
*** Sukhdev has joined #openstack-nova | 20:44 | |
*** cleong has quit IRC | 20:45 | |
*** rfolco has quit IRC | 20:46 | |
*** awaugama has quit IRC | 20:47 | |
*** krtaylor has quit IRC | 20:51 | |
*** Jack_Iv has quit IRC | 20:54 | |
*** Jack_Iv has joined #openstack-nova | 20:56 | |
*** dixiaoli has joined #openstack-nova | 20:58 | |
*** harlowja has joined #openstack-nova | 20:59 | |
*** Apoorva_ has joined #openstack-nova | 21:00 | |
*** thorst has quit IRC | 21:01 | |
mriedem | mgagne: i've given you more information than you ever wanted to know https://review.openstack.org/#/c/454322/ | 21:01 |
*** Jack_Iv has quit IRC | 21:01 | |
*** Jack_Iv has joined #openstack-nova | 21:02 | |
*** adrian_otto has joined #openstack-nova | 21:02 | |
mgagne | mriedem: thanks, will look right now | 21:03 |
*** Apoorva has quit IRC | 21:04 | |
*** Jack_Iv has quit IRC | 21:06 | |
*** annegentle has quit IRC | 21:08 | |
*** nkorabli has joined #openstack-nova | 21:11 | |
*** damien_r1 has quit IRC | 21:11 | |
*** catintheroof has quit IRC | 21:15 | |
*** pchavva has quit IRC | 21:17 | |
*** adrian_otto has quit IRC | 21:19 | |
*** dimtruck is now known as zz_dimtruck | 21:19 | |
*** zz_dimtruck is now known as dimtruck | 21:19 | |
*** adrian_otto has joined #openstack-nova | 21:20 | |
mriedem | jaypipes: you're doing the get_inventory changes under some random topic branch right? | 21:22 |
mriedem | jaypipes: looks like you did get_inventory for ironic under this https://review.openstack.org/#/q/topic:bp/custom-resource-classes-pike,n,z | 21:23 |
jaypipes | mriedem: correct. | 21:23 |
mriedem | jaypipes: or should we consider that bp closed? | 21:23 |
*** burt has quit IRC | 21:23 | |
*** toure is now known as toure|gone | 21:23 | |
jaypipes | that bp is closed yet. | 21:23 |
mriedem | and the get_inventory for the other drivers is just currency? | 21:23 |
mriedem | s/yet/yes/? | 21:23 |
jaypipes | mriedem: correct. I would like to get the get_inventory() methods implemented for the other virt drivers before the nested resource providers is done. will make my life easier. | 21:24 |
mriedem | ok | 21:24 |
jaypipes | mriedem: not having to redo a bunch of stuff. | 21:24 |
mriedem | i'll close custom-resource-classes-pike then | 21:24 |
mriedem | thanks | 21:24 |
*** abalutoiu has quit IRC | 21:26 | |
jaypipes | mriedem: no, thank YOU. | 21:31 |
mriedem | ha | 21:31 |
*** nkorabli has quit IRC | 21:31 | |
*** fragatin_ has joined #openstack-nova | 21:33 | |
*** jamiec has quit IRC | 21:35 | |
*** annegentle has joined #openstack-nova | 21:36 | |
*** fragatina has quit IRC | 21:37 | |
*** jamiec has joined #openstack-nova | 21:38 | |
*** Apoorva_ has quit IRC | 21:38 | |
*** Apoorva has joined #openstack-nova | 21:39 | |
*** krtaylor has joined #openstack-nova | 21:39 | |
*** eharney has quit IRC | 21:40 | |
mriedem | sdague: do we have anything left to do with https://blueprints.launchpad.net/nova/+spec/validate-project-with-keystone ? | 21:40 |
*** baoli has quit IRC | 21:40 | |
mriedem | maybe that is https://review.openstack.org/#/c/435432/ ? | 21:41 |
*** thorst has joined #openstack-nova | 21:41 | |
*** jamesdenton has quit IRC | 21:42 | |
mriedem | i seem to remember some fallout from https://review.openstack.org/#/c/435010/ with tripleo, but i think any attempts to change were abandoned | 21:43 |
mriedem | around the time of the ptg | 21:43 |
*** felipemonteiro has quit IRC | 21:44 | |
*** thorst has quit IRC | 21:46 | |
*** salv-orl_ has joined #openstack-nova | 21:49 | |
*** salv-orlando has quit IRC | 21:51 | |
*** jdurgin has quit IRC | 21:52 | |
*** esberglu has quit IRC | 21:54 | |
*** esberglu has joined #openstack-nova | 21:55 | |
*** ijw has joined #openstack-nova | 21:55 | |
*** takashin has joined #openstack-nova | 21:56 | |
*** esberglu has quit IRC | 21:56 | |
*** abalutoiu has joined #openstack-nova | 21:56 | |
*** esberglu has joined #openstack-nova | 21:56 | |
*** esberglu has quit IRC | 21:56 | |
*** esberglu has joined #openstack-nova | 21:56 | |
*** esberglu has quit IRC | 21:57 | |
*** efried has quit IRC | 21:59 | |
*** Jack_Iv has joined #openstack-nova | 22:03 | |
*** Jack_Iv has quit IRC | 22:07 | |
openstackgerrit | Matthew Treinish proposed openstack/nova master: Remove nova-cert https://review.openstack.org/459460 | 22:11 |
mikal | Has anyone seen this "cannot imprt name pool_states" error coming out of the xenapi driver? | 22:20 |
mikal | http://logs.openstack.org/66/459166/1/check/gate-nova-python27-ubuntu-xenial/30e6fc7/console.html#_2017-04-24_05_24_59_601212 | 22:21 |
*** jlk has joined #openstack-nova | 22:21 | |
openstackgerrit | Matthew Treinish proposed openstack/nova master: Remove nova-cert https://review.openstack.org/459460 | 22:24 |
*** abhi89 has quit IRC | 22:24 | |
*** takashin_ has joined #openstack-nova | 22:25 | |
*** takashin_ has left #openstack-nova | 22:25 | |
*** takashin has quit IRC | 22:26 | |
*** mlavalle has quit IRC | 22:26 | |
*** iceyao has joined #openstack-nova | 22:28 | |
*** iceyao has quit IRC | 22:32 | |
*** fragatin_ has quit IRC | 22:33 | |
*** slaweq has quit IRC | 22:33 | |
*** fragatina has joined #openstack-nova | 22:33 | |
*** chyka has quit IRC | 22:42 | |
*** chyka has joined #openstack-nova | 22:42 | |
*** jamesdenton has joined #openstack-nova | 22:51 | |
*** adrian_otto has quit IRC | 22:52 | |
*** kiwi_rot has joined #openstack-nova | 22:59 | |
*** lyan has quit IRC | 23:00 | |
openstackgerrit | Matthew Treinish proposed openstack/nova master: Remove nova-cert https://review.openstack.org/459460 | 23:02 |
*** Jack_Iv has joined #openstack-nova | 23:03 | |
*** sdague has quit IRC | 23:03 | |
*** kiwi_rot has quit IRC | 23:03 | |
openstackgerrit | Matthew Treinish proposed openstack/nova master: Remove nova-cert https://review.openstack.org/459460 | 23:04 |
*** Jack_Iv has quit IRC | 23:08 | |
*** dimtruck is now known as zz_dimtruck | 23:10 | |
*** salv-orl_ has quit IRC | 23:10 | |
*** ZZelle_ has quit IRC | 23:18 | |
*** jlk has left #openstack-nova | 23:21 | |
*** mriedem has left #openstack-nova | 23:24 | |
*** dave-mccowan has quit IRC | 23:30 | |
*** sree has joined #openstack-nova | 23:31 | |
*** spotz_ has quit IRC | 23:34 | |
*** dixiaoli has quit IRC | 23:35 | |
*** sree has quit IRC | 23:35 | |
*** takashin has joined #openstack-nova | 23:35 | |
*** claudiub has quit IRC | 23:36 | |
*** zz_dimtruck is now known as dimtruck | 23:36 | |
*** mriedem has joined #openstack-nova | 23:36 | |
openstackgerrit | Matthew Treinish proposed openstack/nova master: WIP: Remove cloudpipe APIs https://review.openstack.org/459470 | 23:44 |
*** annegentle has quit IRC | 23:46 | |
*** iceyao has joined #openstack-nova | 23:47 | |
*** dtp has quit IRC | 23:48 | |
*** iceyao has quit IRC | 23:51 | |
*** spotz_ has joined #openstack-nova | 23:56 | |
mriedem | you know what's a nice surprise? being reminded you have $5000 in a dependent care spending account for the previous tax year just waiting to be claimed. | 23:57 |
dansmith | mriedem: you know what sucks? not being rewarded on taxes for keeping the population down. | 23:58 |
*** spotz_ has quit IRC | 23:58 | |
mriedem | however, | 23:58 |
mriedem | you still win money-wise in the end | 23:59 |
*** annegentle has joined #openstack-nova | 23:59 | |
dansmith | and in overall happiness | 23:59 |
dansmith | and in number of unsoiled walls, carpets, etc | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!