| *** isq has joined #openstack-nova | 00:00 | |
| *** oomichi has joined #openstack-nova | 00:04 | |
| *** annegentle has joined #openstack-nova | 00:06 | |
| *** armax has joined #openstack-nova | 00:06 | |
| *** Nikolay_St has quit IRC | 00:06 | |
| *** Nikolay_St has joined #openstack-nova | 00:07 | |
| *** annegentle has quit IRC | 00:11 | |
| *** jwcroppe has quit IRC | 00:12 | |
| gmann | mriedem: yea, actually that was bug which got fixed in V2.1 only. And we did undo the fix to have same behavior for v2 and v2.1. | 00:12 |
|---|---|---|
| mriedem | gmann: so....should that commented out code be removed from the v2.1 servers extension? | 00:13 |
| gmann | mriedem: but i feel as it is just bug we should fix it without microversion in both code base- v2 as well in v2.1 | 00:13 |
| mriedem | gmann: oh | 00:13 |
| gmann | mriedem: whats you say? | 00:13 |
| mriedem | gmann: i didn't realize there was an outstanding bug | 00:14 |
| *** ano89 has joined #openstack-nova | 00:14 | |
| *** ano89 has quit IRC | 00:14 | |
| gmann | mriedem: https://bugs.launchpad.net/nova/+bug/1185290 | 00:16 |
| openstack | Launchpad bug 1185290 in OpenStack Compute (nova) "nova list's --tenant flag also requires --all-tenants" [Undecided,Fix released] - Assigned to Aarti Kriplani (aarti-kriplani) | 00:16 |
| mriedem | gmann: so that's marked as fix released | 00:17 |
| mriedem | but it's not fixed? | 00:17 |
| mriedem | or only fixed in v2.1? | 00:17 |
| *** ijw_ has joined #openstack-nova | 00:18 | |
| mriedem | and was then reverted b/c it was a difference from how v2 worked? i guess i don't know all of the history, obviously | 00:18 |
| gmann | mriedem: yes, it was fixed in v3 only and reverted in v2.1 | 00:18 |
| mriedem | well, s/reverted/commented out/ | 00:18 |
| gmann | mriedem: yes, that was reason to revert back | 00:18 |
| mriedem | gmann: so is there a confirmed/triaged bug still tracking this? | 00:19 |
| gmann | mriedem: i think no. | 00:19 |
| mriedem | heh | 00:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Support Network objects in set_network_host https://review.openstack.org/192396 | 00:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/194325 | 00:20 |
| mriedem | alright, we should probably update that bug saying it was reverted/commented out at some point | 00:20 |
| mriedem | and that the bug isn't actually fixed | 00:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/194889 | 00:21 |
| *** dims has joined #openstack-nova | 00:23 | |
| *** yamahata has joined #openstack-nova | 00:24 | |
| gmann | mriedem: yup | 00:28 |
| gmann | mriedem: so how we fix it as microversion or just bug fix in both v2 nd v2.1 | 00:29 |
| *** otter768 has quit IRC | 00:32 | |
| *** electrocucaracha has quit IRC | 00:32 | |
| *** otter768 has joined #openstack-nova | 00:33 | |
| *** bradjones has quit IRC | 00:34 | |
| *** bradjones has joined #openstack-nova | 00:35 | |
| *** bradjones has quit IRC | 00:35 | |
| *** bradjones has joined #openstack-nova | 00:35 | |
| *** Nikolay_St has quit IRC | 00:36 | |
| *** Nikolay_St has joined #openstack-nova | 00:37 | |
| *** tjones1 has joined #openstack-nova | 00:39 | |
| *** thomasem has quit IRC | 00:40 | |
| *** thomasem has joined #openstack-nova | 00:41 | |
| *** yingjun has joined #openstack-nova | 00:42 | |
| *** otter768 has quit IRC | 00:43 | |
| *** jasondotstar has quit IRC | 00:43 | |
| *** chen12 has left #openstack-nova | 00:44 | |
| *** yamahata has quit IRC | 00:46 | |
| *** annashen has joined #openstack-nova | 00:48 | |
| *** ijw_ has quit IRC | 00:48 | |
| *** dramakri has quit IRC | 00:51 | |
| *** wade1 has joined #openstack-nova | 00:52 | |
| mriedem | gmann: i'd think just a bug fix | 00:54 |
| *** yamahata has joined #openstack-nova | 00:57 | |
| *** wade1 has quit IRC | 00:58 | |
| *** wade1 has joined #openstack-nova | 00:59 | |
| *** mriedem1 has joined #openstack-nova | 00:59 | |
| *** Nikolay_St has quit IRC | 01:00 | |
| *** Nikolay_St has joined #openstack-nova | 01:00 | |
| *** mriedem has quit IRC | 01:01 | |
| *** wanghao has joined #openstack-nova | 01:03 | |
| *** ijw_ has joined #openstack-nova | 01:05 | |
| *** dsanders has quit IRC | 01:05 | |
| *** annegentle has joined #openstack-nova | 01:07 | |
| mriedem1 | dansmith: heh, 0 unit tests for release_fixed_ip in NetworkManager today | 01:11 |
| mriedem1 | 0 tests for dhcpagent | 01:11 |
| *** wade1 has quit IRC | 01:12 | |
| *** annegentle has quit IRC | 01:12 | |
| *** yingjun has quit IRC | 01:12 | |
| *** yingjun has joined #openstack-nova | 01:12 | |
| dansmith | mriedem1: eesh | 01:13 |
| *** thorst has joined #openstack-nova | 01:15 | |
| *** yamahata has quit IRC | 01:17 | |
| *** tjones1 has quit IRC | 01:18 | |
| *** tjones1 has joined #openstack-nova | 01:18 | |
| *** yingjun_ has joined #openstack-nova | 01:19 | |
| *** yingjun has quit IRC | 01:23 | |
| *** Nikolay_St has quit IRC | 01:28 | |
| *** thorst has quit IRC | 01:28 | |
| *** mriedem1 is now known as mriedem | 01:28 | |
| *** Nikolay_St has joined #openstack-nova | 01:29 | |
| *** jyuso1 has quit IRC | 01:34 | |
| *** jdurgin has joined #openstack-nova | 01:38 | |
| *** Marga_ has joined #openstack-nova | 01:39 | |
| *** yingjun_ has quit IRC | 01:42 | |
| *** jwcroppe has joined #openstack-nova | 01:47 | |
| *** Raildo_ has joined #openstack-nova | 01:48 | |
| *** yamahata has joined #openstack-nova | 01:48 | |
| *** Raildo_ has quit IRC | 01:48 | |
| *** baoli has joined #openstack-nova | 01:49 | |
| *** Raildo_ has joined #openstack-nova | 01:49 | |
| *** husanu has quit IRC | 01:49 | |
| *** baoli has quit IRC | 01:49 | |
| *** baoli has joined #openstack-nova | 01:50 | |
| *** husanu has joined #openstack-nova | 01:51 | |
| *** jwcroppe has quit IRC | 01:54 | |
| *** jwcroppe has joined #openstack-nova | 01:55 | |
| *** mwagner_afk has joined #openstack-nova | 01:55 | |
| *** Nikolay_St has quit IRC | 01:56 | |
| *** Nikolay_St has joined #openstack-nova | 01:56 | |
| *** jwcroppe has quit IRC | 01:59 | |
| *** patrickeast has quit IRC | 02:00 | |
| *** yingjun has joined #openstack-nova | 02:01 | |
| *** yingjun has quit IRC | 02:02 | |
| *** sdake has joined #openstack-nova | 02:04 | |
| *** tobe has joined #openstack-nova | 02:06 | |
| *** park_heijlong has joined #openstack-nova | 02:06 | |
| *** annegentle has joined #openstack-nova | 02:08 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Check mac for instance before disassociate in release_fixed_ip https://review.openstack.org/194843 | 02:10 |
| mriedem | merry xmas nova.tests.unit.network.test_manager | 02:11 |
| *** annegentle has quit IRC | 02:13 | |
| *** zhenguo has joined #openstack-nova | 02:18 | |
| *** jasondotstar has joined #openstack-nova | 02:18 | |
| *** annegentle has joined #openstack-nova | 02:22 | |
| *** fifieldt_ has quit IRC | 02:23 | |
| *** baoli has quit IRC | 02:26 | |
| *** annegentle has quit IRC | 02:28 | |
| *** annashen has quit IRC | 02:28 | |
| *** unicell has quit IRC | 02:28 | |
| *** annegentle has joined #openstack-nova | 02:32 | |
| *** dsanders has joined #openstack-nova | 02:32 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/objects/test*.py https://review.openstack.org/187785 | 02:33 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Add rootwrap daemon mode support https://review.openstack.org/180695 | 02:33 |
| *** sdake has quit IRC | 02:33 | |
| *** otter768 has joined #openstack-nova | 02:44 | |
| *** otter768 has quit IRC | 02:49 | |
| *** varunarya has joined #openstack-nova | 02:50 | |
| *** asettle has joined #openstack-nova | 02:50 | |
| *** dims has quit IRC | 02:51 | |
| *** Longgeek_ has joined #openstack-nova | 02:52 | |
| *** jdurgin has quit IRC | 02:52 | |
| *** Longgeek has quit IRC | 02:52 | |
| *** Longgeek has joined #openstack-nova | 02:53 | |
| *** veena has joined #openstack-nova | 02:53 | |
| *** Longgeek has quit IRC | 02:55 | |
| *** Longgeek has joined #openstack-nova | 02:55 | |
| mriedem | dansmith: so i reviewed https://review.openstack.org/#/c/193668/ | 02:56 |
| *** Longgeek_ has quit IRC | 02:56 | |
| mriedem | i think that's ambitious to the point that it probably wont land in liberty | 02:56 |
| mriedem | but the library could be started i guess and wip's could be worked on in nova to test the changes out | 02:56 |
| *** apuimedo has quit IRC | 02:57 | |
| mriedem | i just don't see nova/neutron cores having that all settled by thursday | 02:57 |
| dansmith | mriedem: agree, even if there was no deadline looming I think it's unlikely to make liberty | 02:58 |
| mriedem | it's essentially very similar to what i was proposing with making versioned objects in os-brick for the connection_info dict, but there is no pluggability stuff in my case, so mine is much more mechanical and easy to pull off probably | 02:58 |
| *** Longgeek has quit IRC | 02:59 | |
| *** patrickeast has joined #openstack-nova | 02:59 | |
| *** Longgeek_ has joined #openstack-nova | 02:59 | |
| mriedem | and with that i must now retire to my sleeping chamber for tv watching | 03:00 |
| mriedem | adieu | 03:00 |
| dansmith | heh, later | 03:01 |
| *** jyuso1 has joined #openstack-nova | 03:03 | |
| *** Longgeek has joined #openstack-nova | 03:04 | |
| *** Longgeek_ has quit IRC | 03:08 | |
| *** bkopilov has quit IRC | 03:08 | |
| *** jyuso1 has quit IRC | 03:09 | |
| *** Jeffrey4l has joined #openstack-nova | 03:11 | |
| *** bradjones has quit IRC | 03:16 | |
| *** changbl_ has joined #openstack-nova | 03:17 | |
| *** bradjones has joined #openstack-nova | 03:18 | |
| *** bradjones has quit IRC | 03:18 | |
| *** bradjones has joined #openstack-nova | 03:18 | |
| *** bkopilov has joined #openstack-nova | 03:19 | |
| *** iamjarvo has joined #openstack-nova | 03:19 | |
| *** krtaylor has joined #openstack-nova | 03:19 | |
| *** Longgeek has quit IRC | 03:20 | |
| *** Longgeek has joined #openstack-nova | 03:23 | |
| *** boris-42 has joined #openstack-nova | 03:23 | |
| *** tpeoples has quit IRC | 03:26 | |
| *** Raildo_ has quit IRC | 03:27 | |
| openstackgerrit | Anne Gentle proposed openstack/nova: Changes conf.py for Sphinx build because oslosphinx now contains GA https://review.openstack.org/172525 | 03:29 |
| *** dave-mccowan has quit IRC | 03:31 | |
| *** jasondotstar has quit IRC | 03:33 | |
| *** varunarya has quit IRC | 03:33 | |
| *** dave-mccowan has joined #openstack-nova | 03:34 | |
| *** ayoung is now known as ayoung_ZZZZzzz-- | 03:34 | |
| openstackgerrit | Rui Chen proposed openstack/nova-specs: CreateVM supports subnet specified https://review.openstack.org/169638 | 03:35 |
| *** Longgeek has quit IRC | 03:37 | |
| *** Longgeek has joined #openstack-nova | 03:37 | |
| *** sdake has joined #openstack-nova | 03:38 | |
| *** cfriesen_ has joined #openstack-nova | 03:41 | |
| *** Longgeek_ has joined #openstack-nova | 03:42 | |
| *** Longgee__ has joined #openstack-nova | 03:43 | |
| *** annegentle has quit IRC | 03:45 | |
| *** Longgeek has quit IRC | 03:46 | |
| *** bkopilov has quit IRC | 03:47 | |
| *** Longgeek_ has quit IRC | 03:47 | |
| *** Longgeek has joined #openstack-nova | 03:48 | |
| *** dramakri has joined #openstack-nova | 03:50 | |
| *** Longgee__ has quit IRC | 03:52 | |
| *** dave-mccowan has quit IRC | 03:52 | |
| *** husanu has quit IRC | 03:54 | |
| *** ildikov has quit IRC | 03:54 | |
| *** jwcroppe has joined #openstack-nova | 03:55 | |
| *** husanu has joined #openstack-nova | 03:56 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: virt: introduce libosinfo library to set hardware policy https://review.openstack.org/149625 | 03:59 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring network model https://review.openstack.org/149627 | 03:59 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: adding a class to retrieve hardware properties https://review.openstack.org/149626 | 03:59 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring the disk bus https://review.openstack.org/149628 | 03:59 |
| *** prashup has joined #openstack-nova | 04:00 | |
| *** husanu has quit IRC | 04:01 | |
| *** bkopilov has joined #openstack-nova | 04:02 | |
| *** coolsvap|away is now known as coolsvap | 04:03 | |
| *** husanu has joined #openstack-nova | 04:03 | |
| *** kiran-r has joined #openstack-nova | 04:05 | |
| *** ayoung_ZZZZzzz-- has quit IRC | 04:06 | |
| *** armax has quit IRC | 04:08 | |
| *** Longgeek has quit IRC | 04:11 | |
| *** Longgeek has joined #openstack-nova | 04:12 | |
| *** ams70 has joined #openstack-nova | 04:12 | |
| *** kiranr has joined #openstack-nova | 04:14 | |
| ams70 | Hi, can i create an instance that spans multiple nodes? | 04:15 |
| *** kiranr has quit IRC | 04:15 | |
| *** slagle_ has joined #openstack-nova | 04:16 | |
| *** coolsvap has quit IRC | 04:17 | |
| *** slagle has quit IRC | 04:18 | |
| *** iamjarvo has quit IRC | 04:22 | |
| *** patrickeast has quit IRC | 04:22 | |
| *** arnaud____ has joined #openstack-nova | 04:22 | |
| *** rajesht has joined #openstack-nova | 04:22 | |
| *** dsanders has quit IRC | 04:23 | |
| *** slagle has joined #openstack-nova | 04:23 | |
| *** coolsvap has joined #openstack-nova | 04:23 | |
| *** slagle_ has quit IRC | 04:26 | |
| *** Marga_ has quit IRC | 04:27 | |
| *** lan has quit IRC | 04:27 | |
| *** lan has joined #openstack-nova | 04:28 | |
| *** Nikolay_St has quit IRC | 04:29 | |
| *** kiran-r has quit IRC | 04:30 | |
| *** Nikolay_St has joined #openstack-nova | 04:30 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge preserve_ephemeral_rebuild tests between v2 & v2.1 https://review.openstack.org/173284 | 04:32 |
| *** cfriesen_ has quit IRC | 04:33 | |
| *** fawadkhaliq has joined #openstack-nova | 04:36 | |
| *** kiwik-chenrui has quit IRC | 04:36 | |
| *** kiwik-chenrui has joined #openstack-nova | 04:36 | |
| *** ams70 has quit IRC | 04:38 | |
| *** Marga_ has joined #openstack-nova | 04:40 | |
| *** prashup has quit IRC | 04:40 | |
| *** Nikolay_St has quit IRC | 04:42 | |
| *** achanda has joined #openstack-nova | 04:44 | |
| *** otter768 has joined #openstack-nova | 04:45 | |
| *** otter768 has quit IRC | 04:49 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs: Database connection switching for cells https://review.openstack.org/194935 | 04:50 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge security groups tests between v2 and v2.1 https://review.openstack.org/173671 | 04:50 |
| *** whenry has joined #openstack-nova | 04:53 | |
| *** unicell has joined #openstack-nova | 04:55 | |
| *** _gryf|afk is now known as _gryf | 04:55 | |
| *** nkrinner has joined #openstack-nova | 05:05 | |
| *** achanda has quit IRC | 05:06 | |
| *** unicell1 has joined #openstack-nova | 05:21 | |
| *** arnaud____ has quit IRC | 05:21 | |
| *** unicell has quit IRC | 05:22 | |
| *** fawadkhaliq has quit IRC | 05:24 | |
| *** tojuvone has joined #openstack-nova | 05:24 | |
| *** pixelb has quit IRC | 05:25 | |
| *** hshiina has joined #openstack-nova | 05:27 | |
| *** pixelb has joined #openstack-nova | 05:27 | |
| *** tobe has quit IRC | 05:29 | |
| *** husanu has quit IRC | 05:29 | |
| *** tobe has joined #openstack-nova | 05:29 | |
| *** achanda has joined #openstack-nova | 05:30 | |
| *** husanu has joined #openstack-nova | 05:31 | |
| *** ams70 has joined #openstack-nova | 05:33 | |
| *** achanda has quit IRC | 05:35 | |
| *** prashup has joined #openstack-nova | 05:39 | |
| *** mgarza_ has joined #openstack-nova | 05:41 | |
| *** lan has quit IRC | 05:43 | |
| *** doron_ has joined #openstack-nova | 05:45 | |
| *** doron_ has left #openstack-nova | 05:45 | |
| *** changbl_ has quit IRC | 05:46 | |
| *** ian_brown has joined #openstack-nova | 05:48 | |
| *** deepthi has joined #openstack-nova | 05:48 | |
| openstackgerrit | Nikhil Komawar proposed openstack/nova-specs: Add support for Glance v2 API https://review.openstack.org/194945 | 05:48 |
| *** sudipto has joined #openstack-nova | 05:49 | |
| *** lan has joined #openstack-nova | 05:49 | |
| *** ianbrown has quit IRC | 05:49 | |
| *** lpetrut has joined #openstack-nova | 05:51 | |
| *** asettle has quit IRC | 05:52 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge V2 and V2.1 keypair functional tests https://review.openstack.org/176177 | 05:54 |
| *** garyk has quit IRC | 05:56 | |
| *** Marga_ has quit IRC | 06:00 | |
| *** prashup has quit IRC | 06:00 | |
| *** puniuee has joined #openstack-nova | 06:02 | |
| *** zhenguo has quit IRC | 06:03 | |
| *** Marga_ has joined #openstack-nova | 06:06 | |
| *** pixelb has quit IRC | 06:09 | |
| *** tjones1 has quit IRC | 06:09 | |
| *** fawadkhaliq has joined #openstack-nova | 06:11 | |
| *** Kennan has quit IRC | 06:12 | |
| *** radek__ has joined #openstack-nova | 06:12 | |
| *** destiny has joined #openstack-nova | 06:12 | |
| *** Kennan has joined #openstack-nova | 06:13 | |
| *** kiran-r has joined #openstack-nova | 06:16 | |
| *** belmoreira has joined #openstack-nova | 06:18 | |
| *** kiranr has joined #openstack-nova | 06:18 | |
| *** oomichi has quit IRC | 06:18 | |
| *** kiranr has quit IRC | 06:18 | |
| *** _kiran_ has joined #openstack-nova | 06:18 | |
| *** bradjones has quit IRC | 06:20 | |
| *** kiran-r has quit IRC | 06:21 | |
| *** chaochin has quit IRC | 06:22 | |
| *** kiranr has joined #openstack-nova | 06:22 | |
| *** bradjones has joined #openstack-nova | 06:22 | |
| *** bradjones has quit IRC | 06:22 | |
| *** bradjones has joined #openstack-nova | 06:22 | |
| *** chaochin has joined #openstack-nova | 06:22 | |
| *** _kiran_ has quit IRC | 06:23 | |
| *** kiranr has quit IRC | 06:23 | |
| *** kiran-r has joined #openstack-nova | 06:24 | |
| *** abhishekk has joined #openstack-nova | 06:26 | |
| *** kiranr has joined #openstack-nova | 06:27 | |
| *** kiran-r has quit IRC | 06:29 | |
| *** amotoki has joined #openstack-nova | 06:29 | |
| *** jyuso1 has joined #openstack-nova | 06:29 | |
| *** kiranr has quit IRC | 06:32 | |
| *** puniuee has quit IRC | 06:34 | |
| *** mpaolino has joined #openstack-nova | 06:36 | |
| *** ssurana has quit IRC | 06:36 | |
| *** dsanders has joined #openstack-nova | 06:36 | |
| *** bradjones has quit IRC | 06:36 | |
| *** ian_brown has quit IRC | 06:37 | |
| *** bradjones has joined #openstack-nova | 06:39 | |
| *** bradjones has quit IRC | 06:39 | |
| *** bradjones has joined #openstack-nova | 06:39 | |
| *** garyk has joined #openstack-nova | 06:42 | |
| *** park_heijlong has quit IRC | 06:43 | |
| *** scheuran has joined #openstack-nova | 06:43 | |
| *** mpaolino has quit IRC | 06:43 | |
| *** chen12 has joined #openstack-nova | 06:44 | |
| chen12 | hello, anyone here ?? I try to run "tox" to run unit tests, but get faied when download requirements. It looks like a network issue, but I already set http_proxy & https_proxy in my enviroment, anyone know how to fix that ? | 06:44 |
| *** otter768 has joined #openstack-nova | 06:46 | |
| *** varunarya has joined #openstack-nova | 06:46 | |
| *** sahid has joined #openstack-nova | 06:46 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove/deprecate unused parts of the compute node object https://review.openstack.org/186757 | 06:48 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Make ComputeNode.service_id nullable to match db schema https://review.openstack.org/169284 | 06:48 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Avoid querying for Service in resource tracker https://review.openstack.org/169288 | 06:49 |
| *** otter768 has quit IRC | 06:50 | |
| *** puniuee has joined #openstack-nova | 06:51 | |
| *** varunarya has quit IRC | 06:51 | |
| *** ociuhandu has quit IRC | 06:51 | |
| garyk | any chance of begging for a spec review for a parity driver feature - https://review.openstack.org/192675 | 06:52 |
| *** ociuhandu has joined #openstack-nova | 06:52 | |
| *** varunarya has joined #openstack-nova | 06:52 | |
| *** browne has quit IRC | 06:55 | |
| *** varunarya has quit IRC | 06:55 | |
| *** markus_z has joined #openstack-nova | 06:56 | |
| *** dramakri has quit IRC | 07:00 | |
| *** coolsvap has quit IRC | 07:01 | |
| *** mpavone has joined #openstack-nova | 07:02 | |
| *** slagle_ has joined #openstack-nova | 07:02 | |
| *** jlanoux has joined #openstack-nova | 07:04 | |
| *** haypo has joined #openstack-nova | 07:04 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove unused refresh_security_group_members() call https://review.openstack.org/182632 | 07:04 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Make security group refresh with refresh_instance_security_groups() https://review.openstack.org/182633 | 07:04 |
| *** slagle has quit IRC | 07:05 | |
| *** moshele has joined #openstack-nova | 07:06 | |
| *** fawadk has joined #openstack-nova | 07:09 | |
| *** coolsvap has joined #openstack-nova | 07:10 | |
| *** coolsvap is now known as coolsvap|away | 07:11 | |
| *** park_heijlong has joined #openstack-nova | 07:11 | |
| *** coolsvap|away is now known as coolsvap | 07:12 | |
| *** fawadkhaliq has quit IRC | 07:12 | |
| *** kiran-r has joined #openstack-nova | 07:13 | |
| *** ociuhandu has quit IRC | 07:13 | |
| *** kiran-r has quit IRC | 07:14 | |
| *** coolsvap is now known as coolsvap|away | 07:15 | |
| *** dramakri has joined #openstack-nova | 07:16 | |
| *** kiran-r has joined #openstack-nova | 07:17 | |
| *** exploreshaifali has joined #openstack-nova | 07:18 | |
| *** dramakri has quit IRC | 07:18 | |
| *** coolsvap|away is now known as coolsvap | 07:18 | |
| *** dsanders has quit IRC | 07:19 | |
| *** park_heijlong has quit IRC | 07:23 | |
| *** ildikov has joined #openstack-nova | 07:23 | |
| *** exploreshaifali has quit IRC | 07:25 | |
| *** exploreshaifali_ has joined #openstack-nova | 07:25 | |
| *** doude_ has quit IRC | 07:27 | |
| *** alex_klimov has joined #openstack-nova | 07:27 | |
| openstackgerrit | shihanzhang proposed openstack/nova: Optimize validate_networks https://review.openstack.org/175742 | 07:30 |
| *** lpetrut has quit IRC | 07:32 | |
| *** mgarza_ has quit IRC | 07:33 | |
| *** Nikolay_St has joined #openstack-nova | 07:33 | |
| openstackgerrit | junxu proposed openstack/nova: Add hard stop server support in rest api https://review.openstack.org/192972 | 07:34 |
| *** pask81 has joined #openstack-nova | 07:34 | |
| *** park_heijlong has joined #openstack-nova | 07:35 | |
| *** romainh has joined #openstack-nova | 07:35 | |
| *** doude has joined #openstack-nova | 07:39 | |
| openstackgerrit | Qin Zhao proposed openstack/nova: Use flavor object in compute manager https://review.openstack.org/194761 | 07:40 |
| *** dsanders has joined #openstack-nova | 07:41 | |
| *** yonglihe has quit IRC | 07:43 | |
| *** Nikolay_St has quit IRC | 07:43 | |
| *** salv-orl_ has joined #openstack-nova | 07:46 | |
| *** Nikolay_St has joined #openstack-nova | 07:47 | |
| *** zhenguo has joined #openstack-nova | 07:50 | |
| *** salv-orlando has quit IRC | 07:50 | |
| *** pixelb has joined #openstack-nova | 07:54 | |
| haypo | sdague, hi. i think you work on the work, right? what do you think of adding a py34 gate job, in addition to the existing py34 check job? https://review.openstack.org/#/c/194128/ my patch adds a py34 gate and make both py34 jobs voting | 07:54 |
| *** lucasagomes has joined #openstack-nova | 08:04 | |
| *** puniuee has quit IRC | 08:05 | |
| *** pask81 has quit IRC | 08:06 | |
| *** fawadkhaliq has joined #openstack-nova | 08:06 | |
| *** derekh has joined #openstack-nova | 08:07 | |
| *** fawadk has quit IRC | 08:09 | |
| *** BobBall_AWOL is now known as BobBall | 08:10 | |
| *** dsanders has quit IRC | 08:10 | |
| *** zxen has joined #openstack-nova | 08:12 | |
| johnthetubaguy | kfox1111: as far as I know, your spec is in the queue, have you got a link to it? | 08:13 |
| *** yassine_ has joined #openstack-nova | 08:14 | |
| *** zxen has left #openstack-nova | 08:14 | |
| *** jlanoux has quit IRC | 08:15 | |
| markus_z | What's the prerequisite for my spec to appear here: http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/ ? | 08:16 |
| markus_z | My blueprint is that one: https://blueprints.launchpad.net/nova/+spec/libvirt-log-file-and-serial-console | 08:16 |
| *** danpb has joined #openstack-nova | 08:17 | |
| markus_z | It's my first blueprint, so I'm not sure about the process | 08:17 |
| *** kiran has joined #openstack-nova | 08:18 | |
| kashyap | markus_z: Your spec seems to be not yet merged? - https://review.openstack.org/#/c/194222/ | 08:25 |
| abhishekk | markus_z: you need to create nova-specs and submit it to gerrit for review (refer github openstack/nova-specs) | 08:25 |
| *** matrohon has joined #openstack-nova | 08:25 | |
| kashyap | abhishekk: He did. It needs proper reviews. | 08:25 |
| abhishekk | kashyap: right, just saw it | 08:25 |
| markus_z | kashyap: Ah, ok, I thought it is enough to be checked by the gate | 08:25 |
| markus_z | Thanks :) | 08:25 |
| kashyap | markus_z: No, you need to go through the Kabuki dance of approvals. | 08:25 |
| markus_z | kashyap: well well, were are my dancing shoes... | 08:25 |
| *** moshele has quit IRC | 08:25 | |
| *** openstack has quit IRC | 08:25 | |
| *** openstack has joined #openstack-nova | 08:26 | |
| *** atuvenie has joined #openstack-nova | 08:28 | |
| *** jcoufal has joined #openstack-nova | 08:31 | |
| kashyap | markus_z: For later, your mapping of features -> functions doc here https://wiki.openstack.org/wiki/HypervisorSupportMatrix/DriverAPI can be clubbed into this one? -- http://docs.openstack.org/developer/nova/support-matrix.html | 08:31 |
| kashyap | Not urgent at all, just came across it and thought I'd mention it. | 08:32 |
| markus_z | kashyap: Yes, I forgot about that wiki page. I will push a view changes today regarding the support matrix, I will include this. Thanks for the reminder! | 08:33 |
| *** moshele has joined #openstack-nova | 08:33 | |
| markus_z | *"view=few" ... | 08:34 |
| *** garyk1 has quit IRC | 08:34 | |
| kashyap | Cool, thanks. | 08:34 |
| *** garyk has joined #openstack-nova | 08:35 | |
| *** rushiagr_away is now known as rushiagr | 08:38 | |
| *** fawadkhaliq has quit IRC | 08:40 | |
| *** fawadkhaliq has joined #openstack-nova | 08:40 | |
| *** kiran-r has quit IRC | 08:41 | |
| *** kiran is now known as kiran-r | 08:41 | |
| openstackgerrit | Rui Chen proposed openstack/nova-specs: Split network plane for live migration https://review.openstack.org/194990 | 08:43 |
| *** fawadk has joined #openstack-nova | 08:46 | |
| *** otter768 has joined #openstack-nova | 08:46 | |
| *** bradjones has quit IRC | 08:49 | |
| *** fawadkhaliq has quit IRC | 08:49 | |
| *** bradjones has joined #openstack-nova | 08:51 | |
| *** bradjones has quit IRC | 08:51 | |
| *** bradjones has joined #openstack-nova | 08:51 | |
| *** otter768 has quit IRC | 08:51 | |
| *** rotbeard has joined #openstack-nova | 08:59 | |
| *** salv-orl_ has quit IRC | 09:01 | |
| *** salv-orlando has joined #openstack-nova | 09:02 | |
| *** ndipanov has joined #openstack-nova | 09:02 | |
| *** Marga_ has quit IRC | 09:08 | |
| sahid | johnthetubaguy: hola, any chance you remove -2 from the serie here to let a chance to get some reviews https://review.openstack.org/#/c/185910/ | 09:09 |
| sahid | :) | 09:09 |
| *** bradjones has quit IRC | 09:09 | |
| johnthetubaguy | sahid: now its approved I can, sorry missed that one in the mass of paper work | 09:09 |
| *** rotbeard has quit IRC | 09:10 | |
| *** matrohon has quit IRC | 09:10 | |
| sahid | johnthetubaguy: np really | 09:10 |
| *** chaochin has quit IRC | 09:10 | |
| *** chaochin has joined #openstack-nova | 09:11 | |
| johnthetubaguy | sahid: I have a script that tells me which un-approved blueprints don't have a -2, I should probably code up the reverse at some point | 09:11 |
| *** Marga_ has joined #openstack-nova | 09:11 | |
| *** bradjones has joined #openstack-nova | 09:11 | |
| *** bradjones has quit IRC | 09:11 | |
| *** bradjones has joined #openstack-nova | 09:11 | |
| sahid | johnthetubaguy: i guess controbutors will ping for this case :) | 09:12 |
| *** oro has joined #openstack-nova | 09:12 | |
| johnthetubaguy | sahid: for some reason, quite a few people wait until the week before feature freeze to ask, I will try sort that out next week | 09:12 |
| *** park_heijlong has quit IRC | 09:16 | |
| *** Kevin_Zheng has joined #openstack-nova | 09:21 | |
| *** Administrator__ has joined #openstack-nova | 09:21 | |
| *** boris-42 has quit IRC | 09:22 | |
| ams70 | Hi, in compute node if i set "virt_type" in nova-compute.conf file wrongly, which part of openstack goes wrong? | 09:26 |
| *** Marga_ has quit IRC | 09:27 | |
| *** alex_klimov has quit IRC | 09:37 | |
| *** puniuee has joined #openstack-nova | 09:38 | |
| *** rotbeard has joined #openstack-nova | 09:41 | |
| *** afazekas has joined #openstack-nova | 09:42 | |
| *** varunarya has joined #openstack-nova | 09:45 | |
| BobBall | johnthetubaguy: How is BFV meant to work with 3-part images with XS? It seems that we just create an SR pointing to the root disk however tempest is configured to set a seperate kernel_id and ramdisk_id in the volume_image_metadata which doesn't seem to be used? | 09:45 |
| sdague | haypo: I think the gate job is probably not needed because of clean check | 09:46 |
| johnthetubaguy | BobBall: no idea, I doubt it has ever been properly tested, the BFV code is very dodgy in the xenapi driver, it has been patched up recently but really needs attention | 09:46 |
| *** pbrooko has joined #openstack-nova | 09:47 | |
| *** Marga_ has joined #openstack-nova | 09:48 | |
| *** varunarya has quit IRC | 09:49 | |
| *** jasondotstar has joined #openstack-nova | 09:49 | |
| haypo | sdague, you have to discuss with jogo, he has a different opinion :) | 09:49 |
| *** dims has joined #openstack-nova | 09:50 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Add support for Glance v2 API https://review.openstack.org/194945 | 09:52 |
| openstackgerrit | Puneet proposed openstack/nova: implements block volume quota per instance https://review.openstack.org/192973 | 09:54 |
| *** rotbeard has quit IRC | 09:54 | |
| hanzhang | hi all, who can help review this patch #https://review.openstack.org/#/c/175742/ | 09:56 |
| *** obondarev has joined #openstack-nova | 09:58 | |
| *** Marga_ has quit IRC | 10:05 | |
| *** Marga_ has joined #openstack-nova | 10:06 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Add support for Glance v2 API https://review.openstack.org/194945 | 10:07 |
| garyk | danpb: any chance you can please look at https://review.openstack.org/192675 | 10:10 |
| *** zhenguo has quit IRC | 10:13 | |
| *** rotbeard has joined #openstack-nova | 10:14 | |
| *** chen12 has quit IRC | 10:16 | |
| openstackgerrit | Jordan Pittier proposed openstack/nova: libvirt:when deleting snapshot use qemu-img to blockRebase if domain is stopped https://review.openstack.org/192736 | 10:16 |
| *** chen12 has joined #openstack-nova | 10:17 | |
| *** rotbeard has quit IRC | 10:20 | |
| openstackgerrit | garyk proposed openstack/nova: Cells: add instance cell registration utility to nova-manage https://review.openstack.org/193889 | 10:21 |
| *** Marga_ has quit IRC | 10:28 | |
| *** coolsvap is now known as coolsvap|away | 10:28 | |
| *** Marga_ has joined #openstack-nova | 10:28 | |
| *** alex_klimov has joined #openstack-nova | 10:29 | |
| johnthetubaguy | haypo: have you seen this, any chance you can dig into this please, particularly around things like policy.py now having graduated from incubator? https://blueprints.launchpad.net/nova/+spec/liberty-olso-sync | 10:29 |
| *** jlanoux has joined #openstack-nova | 10:33 | |
| *** tteggel_ has quit IRC | 10:36 | |
| *** kiranr has joined #openstack-nova | 10:36 | |
| *** tteggel has joined #openstack-nova | 10:37 | |
| *** gszasz has joined #openstack-nova | 10:37 | |
| *** kzaitsev_mb has joined #openstack-nova | 10:37 | |
| kzaitsev_mb | folks, we're getting wierd errors from calling `nova floating-ip-list` on our gate? | 10:38 |
| kzaitsev_mb | is this a known issue? | 10:38 |
| kashyap | kzaitsev_mb: That sounds a bit vague. If you're running the latest stable, you might want to search in the bug tracker to see if your error turn up there. | 10:40 |
| *** puniuee has quit IRC | 10:40 | |
| *** kiranr has quit IRC | 10:40 | |
| kzaitsev_mb | kashyap: sorry =) | 10:41 |
| kzaitsev_mb | kashyap: ImportError: cannot import name i18n we started getting this error | 10:41 |
| kzaitsev_mb | https://murano-ci.mirantis.com/jenkins/job/gate-murano-dashboard-selenium/1112/console here's a fast link to our ci console output, it's really short | 10:42 |
| kzaitsev_mb | apparently it's installing python-novaclient<2.24.0,>=2.22.0 | 10:42 |
| kashyap | kzaitsev_mb: Quick look gives me this - https://bugs.launchpad.net/oslo.i18n/+bug/1409998 | 10:42 |
| openstack | Launchpad bug 1409998 in oslo.i18n "ImportError: cannot import name i18n" [High,Fix released] - Assigned to Dan Prince (dan-prince) | 10:42 |
| *** hshiina has quit IRC | 10:43 | |
| kzaitsev_mb | thanx! wasn't really sure where to look, and decided to ask here 1st =) | 10:43 |
| *** claudiub has joined #openstack-nova | 10:45 | |
| *** tobe has quit IRC | 10:45 | |
| kzaitsev_mb | hm but that bug is Fix-released. =/ | 10:45 |
| *** otter768 has joined #openstack-nova | 10:47 | |
| *** otter768 has quit IRC | 10:52 | |
| dims | haypo: https://review.openstack.org/#/c/187785/ is ready again with a few tweaks you mentioned | 11:08 |
| *** hshiina has joined #openstack-nova | 11:09 | |
| *** cfriesen_ has joined #openstack-nova | 11:11 | |
| *** mpavone has quit IRC | 11:12 | |
| *** whenry has quit IRC | 11:14 | |
| *** ZZelle-away has quit IRC | 11:14 | |
| *** ZZelle-away has joined #openstack-nova | 11:14 | |
| *** IanGovett has joined #openstack-nova | 11:18 | |
| *** Marga_ has quit IRC | 11:20 | |
| *** jasondotstar has quit IRC | 11:21 | |
| *** lucasagomes is now known as lucas-hungry | 11:23 | |
| *** mpavone has joined #openstack-nova | 11:27 | |
| *** bradjones has quit IRC | 11:29 | |
| *** bradjones has joined #openstack-nova | 11:30 | |
| *** bradjones has quit IRC | 11:30 | |
| *** bradjones has joined #openstack-nova | 11:30 | |
| *** Jeffrey4l has quit IRC | 11:30 | |
| *** husanu has quit IRC | 11:32 | |
| *** husanu has joined #openstack-nova | 11:34 | |
| *** VW_ has joined #openstack-nova | 11:37 | |
| *** aix has quit IRC | 11:37 | |
| *** salv-orlando has quit IRC | 11:39 | |
| *** kzaitsev_mb has quit IRC | 11:41 | |
| *** thorst has joined #openstack-nova | 11:42 | |
| *** Jeffrey4l has joined #openstack-nova | 11:42 | |
| *** cfriesen_ has quit IRC | 11:47 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Adds Hyper-V Shielded VMs spec https://review.openstack.org/192892 | 11:47 |
| *** kiranr has joined #openstack-nova | 11:48 | |
| *** tojuvone has quit IRC | 11:50 | |
| *** kzaitsev_mb has joined #openstack-nova | 11:51 | |
| *** kiranr has quit IRC | 11:52 | |
| *** alex_klimov has quit IRC | 11:54 | |
| *** alex_klimov has joined #openstack-nova | 11:55 | |
| *** rotbeard has joined #openstack-nova | 11:56 | |
| *** alexpilotti has joined #openstack-nova | 11:56 | |
| *** dprince has joined #openstack-nova | 11:57 | |
| *** Jeffrey4l has quit IRC | 11:57 | |
| alexpilotti | danpb johnthetubaguy: hi guys, we hit a potential race conditions source in the manager and I’d like to hear what do you think | 11:59 |
| alexpilotti | specifically in handle_lifecycle_event: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1200-L1237 | 11:59 |
| alexpilotti | the problem came up in Tempest, noticing that somethimes it was processingevents AFTER other subsequent events completed | 12:00 |
| johnthetubaguy | alexpilotti: it looks to me like you could procces the power_state syncs out of order, is that what you are seeing? | 12:00 |
| alexpilotti | yes | 12:00 |
| alexpilotti | e.g. I do stop -> start -> suspend | 12:00 |
| alexpilotti | and I end up processing the stop after the suspend | 12:01 |
| alexpilotti | resulting in a forced shutdown :-) | 12:01 |
| johnthetubaguy | alexpilotti: so I think a bit of compare and swap magic on the updates might help, like check the expected previous state | 12:01 |
| alexpilotti | you mean in the driver or in the manager? | 12:02 |
| johnthetubaguy | alexpilotti: in the manager | 12:02 |
| danpb | hmm, with libvirt at least, events are serialized | 12:03 |
| alexpilotti | one option could be using incremental IDs in the events and skip every event with id lower than the last expected operation | 12:03 |
| johnthetubaguy | part of me is thinking really we should just check the VM state on each lifecycle event, rather than trust the event blindly | 12:03 |
| ams70 | Hi, in compute node if i set "virt_type" in nova-compute.conf file wrongly, which part of openstack goes wrong? | 12:04 |
| johnthetubaguy | alexpilotti: yeah, but then you need to create the incremental ids, there should be an easier fix | 12:04 |
| danpb | johnthetubaguy: you mean check with the hypervisor what the current state is ? | 12:04 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: enhance method to return pointer_model from image prop https://review.openstack.org/174854 | 12:05 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: deprecate use_usb_tablet in favor of pointer_model https://review.openstack.org/176242 | 12:05 |
| johnthetubaguy | danpb: yeah, so just check the latest state and update after the detection of an event, or does that defeat the object of it somehow? | 12:05 |
| danpb | even that has a race condition | 12:05 |
| johnthetubaguy | ams70: all the stuff related to that compute node, I suspect | 12:05 |
| danpb | because another event can arrive between time you do the check and time you take action based on the resulkts of the check | 12:05 |
| *** deepthi has quit IRC | 12:05 | |
| danpb | though admittedly that is probably less likely | 12:05 |
| johnthetubaguy | danpb: it is, but you just want the most up to date state right, we don't need a record of all the transitions, or am I missing something, oh I think I am... | 12:05 |
| alexpilotti | the solution we though about is to decorate driver methods | 12:05 |
| alexpilotti | and stop generating events for a given VM during that operation | 12:06 |
| danpb | FWIW, the original problem scenario addresed by these events was primarily around detecting guest initiated shutdown | 12:06 |
| johnthetubaguy | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5761 | 12:06 |
| danpb | eg, let nova see that the guest admin turned off their guest, without waiting 60 seconds (or whatever it was/is) for the power state refresh cycle | 12:06 |
| johnthetubaguy | danpb: yeah, thats why I was thinking just re-read the shutdown each time | 12:06 |
| danpb | so yeah, i guess we could simply double check that it is truely shut down | 12:07 |
| johnthetubaguy | danpb: I mean, on an event just "check" the power state to see if its changed, but you get what I mean I think | 12:07 |
| johnthetubaguy | alexpilotti: does that give you some ideas to try? | 12:07 |
| danpb | all other lifecycle changes should be a direct reslt of Nova having done some action itself | 12:07 |
| danpb | so we probably would not even need to try processing them i general | 12:07 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova-specs: Fix resource tracking for operations that move instances between hosts https://review.openstack.org/193576 | 12:08 |
| alexpilotti | it’s still race prone IMO | 12:08 |
| johnthetubaguy | danpb: of some out of band management system, but yes | 12:08 |
| johnthetubaguy | alexpilotti: but whats the problem with those races? | 12:08 |
| johnthetubaguy | alexpilotti: you just get the latest state of the VM in the DB, which is what the aim was in the first place right? | 12:08 |
| danpb | johnthetubaguy: i think we generally assume that nothing out of band will be starting guests behinds nova's back | 12:08 |
| johnthetubaguy | danpb: agreed, but the events do detect those, probably | 12:09 |
| *** ajayaa has joined #openstack-nova | 12:09 | |
| danpb | johnthetubaguy: ytep | 12:09 |
| johnthetubaguy | alexpilotti: you can add locking around a smaller bit of the sync that way | 12:09 |
| alexpilotti | the biggest issue is of course the fact that the manager calls stop() on the driver if it detects a stutdown event, which at least in Hyper-V case wouldnt be necessary (as the instance would be already stut down) | 12:09 |
| *** jasondotstar has joined #openstack-nova | 12:09 | |
| alexpilotti | the minor issue is that a client might see some inaccurate sequence of statuse changes | 12:09 |
| johnthetubaguy | alexpilotti: I think that might relate to some volume things, but I don't remember why it does that | 12:10 |
| johnthetubaguy | alexpilotti: that should be expected | 12:10 |
| *** jasondotstar has quit IRC | 12:11 | |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Test patch libvirt race condition (do not merge) https://review.openstack.org/191732 | 12:11 |
| johnthetubaguy | alexpilotti: the client will just not see an edge, and given then are polling an API, they might miss that edge anyways, but our API doesn't pretend to make those guarantees anyways, although I suspect thats not documented anywhere | 12:11 |
| *** svg has joined #openstack-nova | 12:11 | |
| alexpilotti | ok, that’s the minor issue anyway | 12:12 |
| alexpilotti | about the amin one: whay is the manager calling stop() on the driver if it gets a shutdown event? | 12:12 |
| openstackgerrit | Puneet proposed openstack/nova: implements block volume quota per instance https://review.openstack.org/192973 | 12:13 |
| alexpilotti | I mean, the event slready states that the VM is shut down, so the manager should just update the instance status | 12:13 |
| alexpilotti | this is for the actual “guest initiated shutdown” case | 12:13 |
| *** rushiagr is now known as rushiagr_away | 12:14 | |
| svg | good day all | 12:14 |
| alexpilotti | or are there drivers that need to do additional operations after a guest initiated shutdown? | 12:14 |
| alexpilotti | danpb: maybe you know more about libvirt for this ^ | 12:14 |
| svg | got a question about nova.conf [glance] api_servers | 12:15 |
| *** rotbeard has quit IRC | 12:15 | |
| *** kzaitsev_mb has left #openstack-nova | 12:15 | |
| svg | this can be a list of glance servers, but I can't find information on how nova accesses these different servers | 12:15 |
| svg | should each of those endpoints be higly available, or will nova handle one of them going offline? | 12:16 |
| *** VW_ has quit IRC | 12:16 | |
| svg | In my experience, nova returns an error immediately in such a case | 12:16 |
| kashyap | ams70: Depending on _what_ you set wrongly, e.g. if you used plain QEMU instead of KVM, _while_ KVM support is present on your machine, your Nova instances will be very, very slow. | 12:16 |
| *** VW_ has joined #openstack-nova | 12:17 | |
| alexpilotti | johnthetubaguy danpb: this is teh actual line wher the stop() is called: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5763 | 12:17 |
| ams70 | no instead of xen i set QEMU | 12:18 |
| ams70 | kashyap: no instead of xen i set QEMU | 12:18 |
| *** Jeffrey4l has joined #openstack-nova | 12:18 | |
| johnthetubaguy | ams70: so instead of xen VMs you will have got QEMU based VMs I assume? | 12:19 |
| danpb | alexpilotti: you need to tear down networking vifs, clean up cinder storage, and other random stuff | 12:19 |
| danpb | basically do everything you'd do except actually need to stop the guest | 12:19 |
| ams70 | it means not effect on compute node work? | 12:20 |
| johnthetubaguy | ams70: it only affects the compute node | 12:20 |
| *** alex_klimov has quit IRC | 12:20 | |
| alexpilotti | danpb: ok, vifs in our case are handled by the neutron agent, but makes sense | 12:20 |
| *** alex_klimov has joined #openstack-nova | 12:21 | |
| ams70 | so i need to correct it | 12:21 |
| *** sdake_ has joined #openstack-nova | 12:21 | |
| *** sdake has quit IRC | 12:22 | |
| *** LukeHinds has joined #openstack-nova | 12:22 | |
| garyk | johnthetubaguy: is there any use to try and beg borrow or steal review for https://review.openstack.org/192675 - or should we wait till the thurday meeting and hope that there is time for that on the agenda? | 12:22 |
| johnthetubaguy | svg: not sure if thats virt driver depend, it should spread the load between them, in theory | 12:22 |
| johnthetubaguy | garyk: I am half way through that | 12:23 |
| garyk | johnthetubaguy: ok, thanks | 12:23 |
| johnthetubaguy | garyk: the bad news is it feels like the wrong way forward, long term at least, it maybe the only way short term | 12:23 |
| svg | johnthetubaguy: That is what i would expect, and what my upstream-installer (OSAD) expects, but they were not sure if that perhaps changed with kilo | 12:23 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs: Support --all-tenants in floatingips-list https://review.openstack.org/171389 | 12:23 |
| garyk | johnthetubaguy: not sure i understand | 12:24 |
| svg | either way, we easily reproduce this not being the case when bringing a glance server down.. | 12:24 |
| *** Jeffrey4l has quit IRC | 12:24 | |
| garyk | johnthetubaguy: it is how the driver deals with things - so we are saying that vmware cannot have any of this support? | 12:24 |
| alexpilotti | johnthetubaguy danpb: the race condition came up during rescue / unrescue, as an example | 12:24 |
| alexpilotti | johnthetubaguy danpb: do you think it makes sense to send shutdown events when the driver does “housekeeping” during those actions? | 12:25 |
| garyk | johnthetubaguy: it is a valid use case for nfv | 12:25 |
| johnthetubaguy | garyk: no, I am saying long term, everyone should implement the same abstract concept all users can understand | 12:25 |
| alexpilotti | r we should just skip them> | 12:25 |
| alexpilotti | ? | 12:25 |
| garyk | johnthetubaguy: the problem is that drivers have different concepts | 12:26 |
| johnthetubaguy | garyk: right, I didn't say it was easy to have a single concept that spans them all, it certainly seems possible | 12:26 |
| *** dave-mccowan has joined #openstack-nova | 12:26 | |
| alexpilotti | I mean, the fact that I need to shudown a VM for performing a rescue, does not mean that the instance has to get into shutoff state | 12:27 |
| johnthetubaguy | garyk: my concern is the end user visible bits | 12:27 |
| garyk | johnthetubaguy: i understand the concerns but 90% of the flavor extra specs are very driver specific | 12:27 |
| garyk | the admin who is providing these things should be aware of what she.he is exposing at the back end. | 12:28 |
| garyk | sadly it is not a black box | 12:28 |
| johnthetubaguy | alexpilotti: so it probably should, in a way, I think its a fairly easy fix just to fetch the power state before updating the DB, while holding a lock, to make sure each update just updates the DB with the latest state | 12:28 |
| garyk | sometimes you can drive a ferrari and other times you need to take a bus | 12:28 |
| *** ams70 has quit IRC | 12:28 | |
| johnthetubaguy | garyk: thats why I want 90% of flavor extra specs to die in a big fire and be replace by cross hypervisor concepts, although thats about three releases of work at this point | 12:29 |
| *** baoli has joined #openstack-nova | 12:29 | |
| johnthetubaguy | garyk: sure, but they both have "fast" or "faster" versions, and thats all the end user probably needs to know about | 12:29 |
| alexpilotti | johnthetubaguy: what if I’m in the middle of a rescue (copying images, etc) and the VM is still in shutdown mode? | 12:30 |
| johnthetubaguy | garyk: now the admin concepts might not be common between the hypervisors, and thats sucky, but thats life, its messy | 12:30 |
| garyk | the user knows about the speed but the admin has the control of the keys | 12:30 |
| alexpilotti | johnthetubaguy: this would trigger a stop() | 12:30 |
| johnthetubaguy | alexpilotti: right, its not bad (because its correct), but its not really required either (as its assumed as part of the operation) | 12:31 |
| *** baoli has quit IRC | 12:31 | |
| *** baoli has joined #openstack-nova | 12:32 | |
| garyk | johnthetubaguy: so how is this specs support for the limits, shares for vifs, different from https://review.openstack.org/#/c/128825/13/specs/liberty/approved/libvirt-virtiomq.rst,cm | 12:32 |
| johnthetubaguy | alexpilotti: seems like you might want your driver to surpress events, I don't really know that bit of the code well enough to help any more I am afraid, without going to read all the code and work out what libvirt is up to, don't have this stuff on the XenAPI side, at least didn't have it last time I checked | 12:32 |
| alexpilotti | johnthetubaguy: suppressing events was my first idea, as those events make no sense IMO during those operations | 12:33 |
| alexpilotti | johnthetubaguy: that’s why I asked if we still need to report the shutdown during a rescue :-) | 12:33 |
| johnthetubaguy | garyk: its the first one, but its basically the same, I am not a massive fan of either right now | 12:33 |
| *** Jeffrey4l has joined #openstack-nova | 12:34 | |
| *** lucas-hungry is now known as lucasagomes | 12:34 | |
| johnthetubaguy | garyk: yours has the unfortunate, "look I am different" look about it, which makes it harder to accept, for not particularly good reasons | 12:34 |
| *** packet has joined #openstack-nova | 12:34 | |
| garyk | johnthetubaguy: alexpilotti: sorry to but in but do people actually use rescue. | 12:34 |
| alexpilotti | garyk: well Tempest does :-) | 12:34 |
| garyk | :) | 12:35 |
| johnthetubaguy | garyk: going back to what I said before, its the wrong thing *long* term, doesn't actually block it now, if we accept that fact | 12:35 |
| garyk | johnthetubaguy: ok, thanks | 12:35 |
| johnthetubaguy | garyk: our users requested it, when they are doing things like changing grub boot records after they screwed up their instance | 12:35 |
| johnthetubaguy | garyk: not saying its very cloudy, but it does help you fix broken snapshots, and things | 12:36 |
| garyk | ok, nice to know where it comes in useful | 12:36 |
| *** busterswt has joined #openstack-nova | 12:36 | |
| *** edmondsw has joined #openstack-nova | 12:39 | |
| *** salv-orlando has joined #openstack-nova | 12:40 | |
| openstackgerrit | Jordan Pittier proposed openstack/nova: libvirt:when deleting snapshot use qemu-img to blockRebase if domain is stopped https://review.openstack.org/192736 | 12:41 |
| johnthetubaguy | garyk: I wish we had this all documented somewhere in specs, but yeah, thats a massive effort | 12:41 |
| johnthetubaguy | garyk: I kinda hope I will get around to writing down some one lines for the crazier ones, like evacuate, rescue, etc | 12:42 |
| garyk | yeah, that would be very useful | 12:43 |
| *** sudipto has quit IRC | 12:43 | |
| *** pbrooko has quit IRC | 12:44 | |
| *** romainh has left #openstack-nova | 12:48 | |
| *** ferest has joined #openstack-nova | 12:48 | |
| *** teju has joined #openstack-nova | 12:48 | |
| *** otter768 has joined #openstack-nova | 12:48 | |
| *** fawadk has quit IRC | 12:49 | |
| haypo | dims, cool. i will review https://review.openstack.org/#/c/187785/ | 12:49 |
| *** aysyanne has joined #openstack-nova | 12:49 | |
| *** VW_ has quit IRC | 12:50 | |
| *** EmilienM is now known as EmilienM|off | 12:50 | |
| johnthetubaguy | haypo: did you get my email about looking into this blueprint: https://blueprints.launchpad.net/nova/+spec/liberty-olso-sync | 12:50 |
| haypo | johnthetubaguy, hi. for https://blueprints.launchpad.net/nova/+spec/liberty-olso-sync i don't understand the link with http://lists.openstack.org/pipermail/openstack-dev/2015-June/067006.html | 12:51 |
| *** pbrooko has joined #openstack-nova | 12:51 | |
| *** VW_ has joined #openstack-nova | 12:51 | |
| johnthetubaguy | haypo: oh, oops, thats a total cut and paste error, sorry! | 12:51 |
| haypo | johnthetubaguy, i can take care of oslo.policy. are you aware of other pending tasks related to oslo? | 12:52 |
| johnthetubaguy | haypo: basically we need someone to go and check them all | 12:52 |
| johnthetubaguy | haypo: dims posted a good check list, just trying to find the link to that | 12:53 |
| *** ferest has quit IRC | 12:53 | |
| johnthetubaguy | haypo: here we go: http://lists.openstack.org/pipermail/openstack-dev/2015-June/067131.html | 12:53 |
| *** otter768 has quit IRC | 12:54 | |
| johnthetubaguy | haypo: this is all getting quite urgent, as its soon going to be getting quite late in the cycle to make these updates, it would be good to get it all done quite soon, ideally | 12:55 |
| *** rfolco has joined #openstack-nova | 12:55 | |
| openstackgerrit | garyk proposed openstack/nova-specs: Propose VMware limits, reservation and shares https://review.openstack.org/192675 | 12:55 |
| *** aix has joined #openstack-nova | 12:56 | |
| *** ociuhandu has joined #openstack-nova | 12:56 | |
| *** eharney has joined #openstack-nova | 12:57 | |
| *** rlrossit has joined #openstack-nova | 12:57 | |
| *** Marga_ has joined #openstack-nova | 12:57 | |
| *** mdrabe has joined #openstack-nova | 12:57 | |
| dims | johnthetubaguy: haypo: i have pending reviews/bps for oslo.rootwrap, oslo.service, oslo-config-generator | 12:57 |
| haypo | johnthetubaguy, i see too many files in nova/openstack/common/ | 12:57 |
| johnthetubaguy | haypo: right that needs fixing, sounds like dims has got a good chunk of the work started for you though :) | 12:58 |
| dims | haypo: once we throw in oslo.reports and merge others above, we should be able to get rid of a bunch | 12:58 |
| haypo | dims, i'm filling https://blueprints.launchpad.net/nova/+spec/liberty-olso-sync whitebord. i added your conf generator BP. where are the BP for oslo.service & oslo.rootwrap? | 12:58 |
| dims | https://blueprints.launchpad.net/nova/+spec/oslo-config-generator | 12:59 |
| *** kylek3h has joined #openstack-nova | 12:59 | |
| johnthetubaguy | haypo: we don't need blueprints for all these, that blueprint is just to make sure we don't forget to do the sync really | 12:59 |
| dims | https://blueprints.launchpad.net/nova/+spec/rootwrap-daemon-mode | 12:59 |
| dims | haypo: need to file one for oslo.service - review is ready | 13:00 |
| dims | johnthetubaguy: ++ | 13:00 |
| johnthetubaguy | dims: that rootwrap spec, could you quickly reply to the -1 on there for me, and I can merge that: https://review.openstack.org/#/c/105404/11/specs/liberty/approved/nova-rootwrap-daemon-mode.rst,cm | 13:00 |
| haypo | dims, do you know if oslo.log is used with oslo.context in nova? | 13:00 |
| dims | haypo: i've cut test versions of oslo.cache, oslo.reports today, so if you want to kick the tires, welcome | 13:00 |
| dims | haypo: yes, already took care of that | 13:01 |
| haypo | dims, i saw oslo.context in the code, but i wasn't sure. cool :) | 13:01 |
| dims | johnthetubaguy: ack | 13:01 |
| *** ajayaa has quit IRC | 13:02 | |
| *** doude has quit IRC | 13:02 | |
| *** doude has joined #openstack-nova | 13:02 | |
| dims | johnthetubaguy: replied to Yuriy, don't want to update the file but concern is already fixed in the review | 13:03 |
| *** rotbeard has joined #openstack-nova | 13:04 | |
| johnthetubaguy | dims: funny think, you are probably about to give nova-network a massive performance boost, but lets over look that | 13:04 |
| *** rushiagr_away has quit IRC | 13:04 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Adds Hyper-V vTPM devices spec https://review.openstack.org/195068 | 13:05 |
| dims | johnthetubaguy: hahaha | 13:06 |
| claudiub | johnthetubaguy: hi. Can you take a look at this small blueprint? The concept is not new, so I don't think it requires a spec: https://blueprints.launchpad.net/nova/+spec/hyper-v-host-maintenance | 13:06 |
| *** Jeffrey4l has quit IRC | 13:07 | |
| haypo | johnthetubaguy, i listed some items on https://blueprints.launchpad.net/nova/+spec/liberty-olso-sync and i will try to implement some of them, no idea how much time it will take | 13:07 |
| haypo | johnthetubaguy, maybe dims will help ;) | 13:07 |
| johnthetubaguy | claudiub: please add it to the meeting agenda of spec less blueprints | 13:08 |
| claudiub | johnthetubaguy: k. :) | 13:08 |
| mriedem | danpb: looks like xen also tags it's guests with a nova flag https://github.com/openstack/nova/blob/master/nova/virt/xenapi/host.py#L71 | 13:08 |
| *** hshiina has quit IRC | 13:08 | |
| mriedem | claudiub: do you know if the hyper-v driver somehow identifies the guests it creates as being created by nova or openstack? | 13:08 |
| dims | haypo: we'll find more troops :) | 13:08 |
| haypo | dims, \o/ | 13:08 |
| *** mpavone has quit IRC | 13:09 | |
| *** VW_ has quit IRC | 13:09 | |
| johnthetubaguy | haypo: OK, feel free to find some others to help out, I think thekhey thing is to make sure we get it done what needs doing | 13:09 |
| alexpilotti | mriedem: there’s the UUID in the instance notes that can be used for that | 13:09 |
| johnthetubaguy | haypo: lol, +1 what dims said basically | 13:09 |
| mriedem | alexpilotti: ok, is that set during spawn()? | 13:09 |
| alexpilotti | mriedem: yes | 13:10 |
| mriedem | ok, cool | 13:10 |
| danpb | mriedem: i'm not sure where you're intending to make use of this though - if its for filtering lists of domains, then i still think we should be doing that based on the list of UUIDs that nova has for instnaces its owns | 13:10 |
| mriedem | danpb: yeah, it would be part of that | 13:11 |
| johnthetubaguy | mriedem: alexpilotti; FWIW, I would prefer we remove that XenAPI feature than add it in for the other drivers, its a very odd way of competing with other API calls we have. | 13:11 |
| danpb | mriedem: even for libvirt, we can't actually rely on that metadata being present, unless we assume people haven't left VMs running long term from previous version of nova | 13:11 |
| *** Nikolay_St has quit IRC | 13:11 | |
| danpb | if they upgraded nova while keping VMs running, there's non-zero chance of the metadata being missing | 13:11 |
| *** exploreshaifali_ has quit IRC | 13:11 | |
| *** fawadkhaliq has joined #openstack-nova | 13:11 | |
| mriedem | danpb: yeah, true, but there is a thing in the domain meta returned from the bulk stats call that has the path to the instance on the file system and that has a uuid in it | 13:11 |
| mriedem | which would be correlated with the uuid in the nova db | 13:12 |
| *** mpavone has joined #openstack-nova | 13:12 | |
| danpb | a while back I tried to add filter in libvirt based on instance name, until i found that instance name was configurable :-( | 13:12 |
| mriedem | looks like xen and hyperv are squirreling away the instance uuid also | 13:12 |
| mriedem | garyk: does the vmware driver store something about the nova instance uuid with the vcenter guest vm so it can be filtered later? | 13:12 |
| *** xyang1 has joined #openstack-nova | 13:13 | |
| garyk | mriedem: yes. i will send you a link of r areview that enables the serach | 13:14 |
| *** ivasev has joined #openstack-nova | 13:16 | |
| *** pbrooko has quit IRC | 13:17 | |
| *** achanda has joined #openstack-nova | 13:20 | |
| claudiub | danpb: hi. Can you also take a look at the Hyper-V uefi secure boot spec? Hopefully, all your comments were addressed. :) https://review.openstack.org/#/c/190997/ | 13:20 |
| *** jordanP has joined #openstack-nova | 13:21 | |
| jordanP | ping sahid. About your latest comment in https://review.openstack.org/#/c/192736/5/nova/tests/unit/virt/libvirt/test_driver.py,cm | 13:21 |
| jordanP | If i try your change I get an error: | 13:21 |
| jordanP | AttributeError: Mock object has no attribute 'blockRebase' | 13:21 |
| sahid | jordanP: this method is probably missed from the fake | 13:22 |
| jordanP | sahid, I guess that's because the Domain class in unit/virt/libvirt/fakelibvirt.py has no blockRevasemethod | 13:22 |
| sahid | in fakelibvirt | 13:22 |
| jordanP | sahid, should I add it ? | 13:22 |
| sahid | jordanP: sure ! | 13:22 |
| sahid | :à | 13:22 |
| sahid | :) | 13:22 |
| jordanP | okay, will do | 13:22 |
| *** ijuwang_away is now known as ijuwang | 13:24 | |
| openstackgerrit | Lei Li proposed openstack/nova: Add the rule of separate plugin for Nova REST API in devref https://review.openstack.org/162913 | 13:25 |
| *** rotbeard has quit IRC | 13:25 | |
| mriedem | oh ndipanov | 13:26 |
| ndipanov | oh mriedem | 13:26 |
| ndipanov | :) | 13:26 |
| mriedem | ndipanov: i think you'll find there is no problem with this spec https://review.openstack.org/#/c/194774/ | 13:26 |
| ndipanov | mriedem, exception that proves the rule | 13:27 |
| ndipanov | and it's also simultaniously sad that it's not a clever/funny python hack, but a spec doc | 13:28 |
| *** Jeffrey4l has joined #openstack-nova | 13:28 | |
| ndipanov | like the get_off_my_lawn() method from yesterday :) | 13:28 |
| *** zzzeek has joined #openstack-nova | 13:28 | |
| *** fawadkhaliq has quit IRC | 13:29 | |
| mriedem | i didn't see that one | 13:29 |
| *** Marga_ has quit IRC | 13:29 | |
| ndipanov | it's a method on objects.DanSmith object | 13:29 |
| *** Marga_ has joined #openstack-nova | 13:30 | |
| openstackgerrit | Jordan Pittier proposed openstack/nova: libvirt:when deleting snapshot use qemu-img to blockRebase if domain is stopped https://review.openstack.org/192736 | 13:30 |
| *** veena has quit IRC | 13:32 | |
| openstackgerrit | dane-fichter proposed openstack/nova-specs: Nova Support of Glance Image Signing https://review.openstack.org/195078 | 13:33 |
| dims | ndipanov: did we just unmask a bug in existing implementation? inst_cell.siblings seems to be an iterator and not a list as it should be? | 13:33 |
| ndipanov | dims, in python2 it uses a map which returns a list | 13:33 |
| *** husanu has quit IRC | 13:33 | |
| ndipanov | if anything we should be forcing that into a list there | 13:34 |
| *** iamjarvo has joined #openstack-nova | 13:34 | |
| ndipanov | or at every call site | 13:34 |
| *** iamjarvo has quit IRC | 13:34 | |
| ndipanov | but just flipping it in tests so that they pass is... | 13:34 |
| ndipanov | well :) | 13:34 |
| dims | ndipanov: makes sense, that the whole idea of the review process right? :) | 13:34 |
| ndipanov | right :) | 13:34 |
| dims | thanks :) | 13:34 |
| *** rushiagr_away has joined #openstack-nova | 13:35 | |
| *** vilobhmm has joined #openstack-nova | 13:35 | |
| *** dave-mccowan has quit IRC | 13:35 | |
| *** teju has quit IRC | 13:35 | |
| *** husanu has joined #openstack-nova | 13:35 | |
| garyk | mriedem: please see commit 2c45a4f59bb26a0f1fada62306deafbca242565a | 13:36 |
| *** kiranr has joined #openstack-nova | 13:36 | |
| garyk | in the annotation filed we will in all kinds of metadata for the instance. then you can search the VC with this. | 13:36 |
| garyk | if you want i can explain in more detail | 13:36 |
| garyk | mriedem: that code only landed in L a few weeks ago | 13:37 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Check mac for instance before disassociate in release_fixed_ip https://review.openstack.org/194843 | 13:38 |
| mriedem | garyk: ok, got to track down a new neutron race bug in the gate | 13:38 |
| *** ctrath has joined #openstack-nova | 13:38 | |
| garyk | mriedem: there are no races in neutron | 13:38 |
| *** ijuwang has quit IRC | 13:41 | |
| *** kiranr has quit IRC | 13:41 | |
| garyk | mriedem: the problem with the patch you posted with the macs is that nova generates a random mad address - maybe it uses the same mac twice and that screws up everything | 13:41 |
| garyk | in nuetron we vliadte that the mac address is not already used | 13:41 |
| openstackgerrit | Alexander Schmidt proposed openstack/nova: libvirt: enable live migration with serial console https://review.openstack.org/191035 | 13:41 |
| *** jecarey has joined #openstack-nova | 13:44 | |
| *** ijuwang has joined #openstack-nova | 13:47 | |
| jaypipes | jroll: "Okay; shall we change the entire document I guess?" <-- my answer would be "no". | 13:48 |
| *** vilobhmm has quit IRC | 13:49 | |
| *** zzzeek has quit IRC | 13:49 | |
| *** burt has joined #openstack-nova | 13:49 | |
| *** vilobhmm has joined #openstack-nova | 13:49 | |
| *** toure has joined #openstack-nova | 13:50 | |
| *** dave-mccowan has joined #openstack-nova | 13:51 | |
| *** Raildo_ has joined #openstack-nova | 13:52 | |
| edleafe | jaypipes: answered your comment on https://review.openstack.org/187739 | 13:54 |
| *** dave-mccowan has quit IRC | 13:56 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Create class hierarchy for tasks in conductor https://review.openstack.org/184495 | 13:57 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Split instance resize operation into 2 phases https://review.openstack.org/195088 | 13:57 |
| *** zz_jgrimm is now known as jgrimm | 13:57 | |
| *** nkrinner has quit IRC | 13:59 | |
| ndipanov | mriedem, actually I'm sad that we will discuss about how documentation is good (no s**t) instead of real issues we have | 13:59 |
| *** whenry has joined #openstack-nova | 13:59 | |
| johnthetubaguy | alaski: I am curious about your comments on this API one: https://review.openstack.org/#/c/173243/17/specs/liberty/approved/api-relax-validation.rst,cm | 13:59 |
| *** mleroy_ has joined #openstack-nova | 14:00 | |
| alaski | johnthetubaguy: my main contention is that it's not really compatible with v2 | 14:00 |
| *** changbl_ has joined #openstack-nova | 14:00 | |
| *** jasondot_ has joined #openstack-nova | 14:00 | |
| alaski | I would favor not stripping out additionalProperties in the relaxation | 14:00 |
| johnthetubaguy | alaski: well if only using in tree components its compatible right? | 14:01 |
| *** _gryf is now known as _gryf|afk | 14:01 | |
| johnthetubaguy | alaski: the worry is around the extra security we get from validation, we ideally don't want to loose that for v2 | 14:01 |
| alaski | johnthetubaguy: for most things yes, for scheduler hints no | 14:01 |
| johnthetubaguy | alaski: we can just relax the validation for scheduler hints, but it the bits where we pass a dict into the DB layer that I am worried about | 14:02 |
| *** lyarwood has quit IRC | 14:02 | |
| alaski | johnthetubaguy: my only real concern is scheduler hints. though we don't support out of tree things in general, I think that's one area we do want to at least offer a passthrough | 14:02 |
| *** lyarwood has joined #openstack-nova | 14:02 | |
| *** dims has quit IRC | 14:02 | |
| johnthetubaguy | alaski: I am not sure that was intentional, but we can add a spec to add that I think? | 14:02 |
| mriedem | ndipanov: ? | 14:02 |
| *** dims has joined #openstack-nova | 14:03 | |
| jroll | jaypipes: yeah, followed up to that. I thought it was your comment at first, got really confused :) | 14:03 |
| mriedem | ndipanov: > 0 is good, yes | 14:03 |
| mriedem | there will always be problems | 14:03 |
| alaski | johnthetubaguy: add what? | 14:03 |
| *** dboik has joined #openstack-nova | 14:03 | |
| johnthetubaguy | alaski: add the validation relax into the v2.1 and v2 api for scheduler hints (not sure what validation we have at the moment | 14:03 |
| ndipanov | mriedem, > 0 docs? | 14:04 |
| alaski | johnthetubaguy: there's strict validation in 2.1, which I think needs to be re-examined | 14:04 |
| alaski | johnthetubaguy: I just want to raise the issue before it's added to v2 | 14:04 |
| mriedem | ndipanov: i was just saying the docs generated from specs are a nice byproduct | 14:05 |
| mriedem | wikis are not stored in code, etherpads are lost easily | 14:05 |
| *** husanu has quit IRC | 14:05 | |
| garyk | alaski: you have a few minutes to chat about the instance mapping? | 14:05 |
| mriedem | ndipanov: i realize your frustration with the specs process, but i feel like we go through this fire drill every 6-8 months | 14:05 |
| alaski | garyk: sure | 14:06 |
| garyk | alaski: i understand that we may want to do it in chunks. but i am not sure how to do the filtering | 14:06 |
| ndipanov | mriedem, well so far we only had 3 cycles with specs | 14:06 |
| garyk | that is, it would be nice if there was some instance value that we could use. but that is not the case | 14:07 |
| garyk | alaski: i am aslo thinking that it is a once off thing - so how much do we need to invest in optimizations here | 14:07 |
| garyk | what do you think | 14:07 |
| *** husanu has joined #openstack-nova | 14:07 | |
| *** logan2 has quit IRC | 14:08 | |
| johnthetubaguy | alaski: yeah, I think we should talk about that separately, its worth a separate discussion | 14:08 |
| *** artom_ has joined #openstack-nova | 14:08 | |
| alaski | johnthetubaguy: yep | 14:08 |
| *** dave-mccowan has joined #openstack-nova | 14:09 | |
| alaski | garyk: I wouldn't worry too much about optimization, I don't think we're doing something that's more time consuming than expected | 14:09 |
| *** ajayaa has joined #openstack-nova | 14:10 | |
| alaski | garyk: what's your thinking on using filtering? I was thinking just page through a sorted list of instances | 14:10 |
| *** artom has quit IRC | 14:10 | |
| *** hshiina has joined #openstack-nova | 14:10 | |
| garyk | alaski: for that i was thinking that we need to return some value from each iteration that that can be used int he next one - that will enable the admin to run it N times in chunks. | 14:12 |
| *** iamjarvo has joined #openstack-nova | 14:12 | |
| *** tonytan4ever has joined #openstack-nova | 14:12 | |
| garyk | it may just need to be something that is documented if you are ok with that | 14:12 |
| *** artom_ has quit IRC | 14:14 | |
| *** rlrossit has left #openstack-nova | 14:15 | |
| alaski | garyk: I might be, but I think I need to see it to make a decision. My thought was to leverage the limit/marker params for the db query to iterate through. if they're sorted by created_at then no instances should be missed as they're paged through | 14:15 |
| openstackgerrit | Puneet proposed openstack/nova: implements block volume quota per instance https://review.openstack.org/192973 | 14:16 |
| *** thangp has joined #openstack-nova | 14:17 | |
| *** READ10 has joined #openstack-nova | 14:17 | |
| *** dboik_ has joined #openstack-nova | 14:17 | |
| garyk | alaski: ok, understood. i just think that we need to return the markers. i'll code it and ping you when i am done | 14:17 |
| alaski | garyk: cool, thanks for working on that | 14:18 |
| garyk | sure np | 14:20 |
| *** dboik has quit IRC | 14:20 | |
| bauzas | alaski: about sched hints, I made the change we discussed https://review.openstack.org/#/c/145528/23/nova/objects/request_spec.py,cm | 14:21 |
| *** Jeffrey4l has quit IRC | 14:22 | |
| bauzas | alaski: like I said, I'm just wondering if all of that is good or not | 14:22 |
| bauzas | johnthetubaguy: ^ | 14:22 |
| bauzas | johnthetubaguy: I saw your reply to alaski's point for hints | 14:22 |
| bauzas | johnthetubaguy: and I exactly have the same problem for the ReqSpec object | 14:22 |
| *** husanu1 has joined #openstack-nova | 14:23 | |
| bauzas | johnthetubaguy: while I know the nova policy about out-of-tree support, I'm still wondering if that's fait to cut support for them, given we have objects | 14:23 |
| bauzas | fair* | 14:23 |
| openstackgerrit | dane-fichter proposed openstack/nova-specs: Nova Support of Glance Image Signing https://review.openstack.org/188874 | 14:24 |
| danpb | bauzas: cut support for what ? | 14:24 |
| alaski | bauzas: johnthetubaguy another way to think about it is, if a scheduler filter with a hint is added to Nova, should we have to upgrade objects and the API to use it? | 14:25 |
| jaypipes | danpb: hey, while I catch you... I'd really love to get the vif plugin library spec approved this week. I know a number of folks have comments/questions on the spec review. anything I can do to assist you with it? | 14:25 |
| johnthetubaguy | bauzas: I think we just need to be more explicit about adding support | 14:26 |
| danpb | jaypipes: yeah, i'm working on updating the spec now | 14:26 |
| danpb | aim to repost it before the end of today my time | 14:26 |
| jaypipes | danpb: coolio. thx | 14:26 |
| danpb | there wre alot of comments, but generally it seems they are supportive of the direction proposed | 14:26 |
| bauzas | danpb: since scheduler_hints was a free-form dict, it was really easy for a new filter to just consume a specific key from that dict | 14:26 |
| alaski | bauzas: johnthetubaguy ultimately I just don't think the hint API belongs in Nova at all and so locking it down, which implies support, is the wrong thing to do | 14:26 |
| alaski | I'd rather leave it free-form and just remove it at some later time if we can | 14:27 |
| bauzas | danpb: now, there is an API validation for v3 explicitly mentioning the supported hints | 14:27 |
| *** achanda has quit IRC | 14:27 | |
| danpb | ah i see | 14:27 |
| bauzas | danpb: and I'm providing an Hints object doing the same | 14:27 |
| johnthetubaguy | alaski: hmm, OK, not given it too much thought right now | 14:27 |
| bauzas | s/v3/v2.1 of course... | 14:27 |
| bauzas | alaski: I'm not really liking that | 14:28 |
| bauzas | alaski: because that's both a security flaw and something confusing and errorprone | 14:28 |
| *** VW_ has joined #openstack-nova | 14:28 | |
| bauzas | alaski: the same goes for image metadata for example | 14:28 |
| bauzas | alaski: and how we key those metadata information | 14:29 |
| garyk | alaski: ok, the penny has dropped - maybe you just heard a big bang. | 14:29 |
| bauzas | alaski: so, I really want to say "for in-tree, here are the official keys and formats, for out-of-tree good luck to you" | 14:29 |
| *** fawadkhaliq has joined #openstack-nova | 14:29 | |
| bauzas | alaski: and just leave the door for unversioned stuff, provided it's not read in-tree | 14:30 |
| *** ctrath has left #openstack-nova | 14:30 | |
| *** ctrath has joined #openstack-nova | 14:30 | |
| *** Murali has quit IRC | 14:30 | |
| *** logan2 has joined #openstack-nova | 14:31 | |
| *** tpeoples has joined #openstack-nova | 14:32 | |
| *** tdp has joined #openstack-nova | 14:33 | |
| *** tpeoples has quit IRC | 14:33 | |
| alaski | bauzas: I'm not sure I see the security flaw for scheduling hints. but I agree with confusing and error prone, however I don't think our validation helps that much | 14:33 |
| alaski | it ensures you don't mispell a hint, but doesn't do anything about whether or not filters are loaded or honored | 14:33 |
| alaski | which is a bigger issue IMO | 14:33 |
| *** tdp has quit IRC | 14:34 | |
| *** tpeoples has joined #openstack-nova | 14:34 | |
| bauzas | alaski: that's an interesting point, whether the user has to know if his SLA (hint) will be honored or not - given the filter is enabled | 14:34 |
| *** rlrossit has joined #openstack-nova | 14:34 | |
| bauzas | alaski: from my perspective, we can say to the user 'ask what you want, we'll find you a destination' | 14:35 |
| jaypipes | danpb: lol at your review on keep-jogo spec :) that's awesome. | 14:35 |
| *** fawadkhaliq has quit IRC | 14:35 | |
| bauzas | alaski: but if in the middle, there is no filter reading the hint, then that's not really a problem | 14:35 |
| alaski | bauzas: so why limit what can be asked? | 14:35 |
| dansmith | jaypipes: lol, indeed | 14:35 |
| bauzas | alaski: you can ask tomatoes=2 if you want, that doesn't necessarly mean Nova is counting tomatoes | 14:36 |
| markus_z | johnthetubaguy: I worked on the devref/architecture until I remembered that you already had a patch set for that: https://review.openstack.org/#/c/177374/ | 14:36 |
| johnthetubaguy | markus_z: I am tracking that stuff in a BP I linked to you, if you need to check | 14:36 |
| markus_z | johnthetubaguy: I did the picture with graphviz: http://www.tiikoni.com/tis/view/?id=b8fe951 | 14:36 |
| markus_z | And can give you the code for that | 14:36 |
| bauzas | alaski: from my perspective, that's the big difference between an API contract and a scheduler hint | 14:37 |
| *** tpeoples has quit IRC | 14:37 | |
| bauzas | alaski: but giving a formal type for what is in-tree sounds necessary | 14:37 |
| mriedem | markus_z: jogo also has a patch for updating the nova arch image in devref | 14:37 |
| alaski | bauzas: but on v2.1 I can't. I think a ML thread may be needed here. I think Nova needs to abstract the user from the hypervisor so I agree with the principal of locking down extra things, but I'm not sure we need to completely hide the scheduler. Or if we do I think the current method is the wrong approach | 14:37 |
| bauzas | alaski: because we can afford tomatoes=2, but not tomatoes="foo" | 14:38 |
| mriedem | markus_z: https://review.openstack.org/#/c/194840/ | 14:38 |
| markus_z | mriedem: ok, I'll look at that. I must have missed that. | 14:38 |
| markus_z | Thanks | 14:38 |
| *** tjones1 has joined #openstack-nova | 14:38 | |
| bauzas | alaski: are you considering image metadata specifying the cpu model as something mandatory or optional ? | 14:39 |
| bauzas | I mean, is that a discrete number of possibilities, or can it be free-form ? | 14:39 |
| *** alex_klimov has quit IRC | 14:39 | |
| *** Marga_ has quit IRC | 14:40 | |
| bauzas | alaski: sorry if I'm unclear, I'm trying to consider any stuff that the user can provide identically | 14:41 |
| * bauzas is really lost in trying to explain his thoughts | 14:42 | |
| *** Marga_ has joined #openstack-nova | 14:42 | |
| alaski | bauzas: that should be discrete possibilities. But if we have that enforced as discrete possibilites, will I need to update an object and API microversion to add a new cpu model? | 14:43 |
| alaski | I'm only arguing for the hints part of the API, because I think we're completely off on what's exposed there. So I don't want to codify it | 14:44 |
| *** artom has joined #openstack-nova | 14:45 | |
| bauzas | alaski: I don't see a big problem here, we can just accept additional properties to that validation schema | 14:45 |
| *** achanda has joined #openstack-nova | 14:45 | |
| *** oro has quit IRC | 14:46 | |
| bauzas | alaski: I mean, we can say "for that specific set of keys, here are the accepted formats, but for any other key, leave it free-form" | 14:46 |
| *** jasondot_ has quit IRC | 14:47 | |
| *** whenry has quit IRC | 14:48 | |
| *** Marga_ has quit IRC | 14:48 | |
| alaski | bauzas: sure, if we accept additionalProperties my concern is smaller. But we're still embedding support for something like 'build_near_host_ip' into the API as a real thing. Which seems meaningless to me | 14:48 |
| bauzas | alaski: okay, I see your concern | 14:49 |
| *** otter768 has joined #openstack-nova | 14:49 | |
| *** tongli has joined #openstack-nova | 14:49 | |
| bauzas | alaski: but I think it's valid to limit it to a specific format, because that's a real thing in-tree | 14:49 |
| bauzas | alaski: and if I want to modify the corresponding filter, then I have to bump an API version | 14:50 |
| bauzas | alaski: because if not, I could have a PITA | 14:50 |
| alaski | bauzas: I would like to say that modifying filters just isn't allowed until we can solve the issue of feedback to users | 14:51 |
| alaski | and I think API validation is a poor proxy for that | 14:52 |
| bauzas | alaski: gotcha | 14:52 |
| bauzas | alaski: so then the RequestSpec is a better mediator for it | 14:52 |
| bauzas | alaski: just because it's something the nova-scheduler waits | 14:53 |
| bauzas | alaski: meaning that the filter accepts a specific format | 14:53 |
| bauzas | alaski: changing how the filter reads the hint changes also the contract | 14:53 |
| bauzas | alaski: so nova-conductor and nova-scheduler have to find a backward compat path for talking the same | 14:54 |
| *** Marga_ has joined #openstack-nova | 14:54 | |
| *** otter768 has quit IRC | 14:54 | |
| *** jasondotstar has joined #openstack-nova | 14:54 | |
| bauzas | alaski: if we leave it unformatted, it would just raise an exception or whatever in the filter which sounds definitely not a Good Thing | 14:54 |
| *** mestery has joined #openstack-nova | 14:57 | |
| *** sdake_ is now known as sdake | 14:58 | |
| *** Marga_ has quit IRC | 15:00 | |
| *** Marga_ has joined #openstack-nova | 15:01 | |
| *** diogogmt has joined #openstack-nova | 15:01 | |
| *** Marga_ has quit IRC | 15:01 | |
| *** Marga_ has joined #openstack-nova | 15:01 | |
| alaski | bauzas: I don't think we should talk about versioning hints. they're not an api imo, it's a black hole you can send things to and sometimes it might do something meaningful | 15:02 |
| bauzas | alaski: probably a ML thread like you said | 15:02 |
| *** annegentle has joined #openstack-nova | 15:02 | |
| dansmith | oh geez | 15:02 |
| dansmith | (versioning hints) | 15:03 |
| bauzas | alaski: sorry but my brain is fried by johnthetubaguy's https://review.openstack.org/#/c/191914/5/specs/backlog/approved/parallel-scheduler.rst,cm | 15:03 |
| *** rwsu has joined #openstack-nova | 15:04 | |
| *** thedodd has joined #openstack-nova | 15:04 | |
| *** svg has left #openstack-nova | 15:05 | |
| bauzas | dansmith: that's already kinda done https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/v3/scheduler_hints.py :) | 15:05 |
| *** r-daneel has joined #openstack-nova | 15:05 | |
| dansmith | bauzas: that's not versioning them like I was thinking :) | 15:06 |
| bauzas | dansmith: and I did pretty the same here https://review.openstack.org/#/c/145528/23/nova/objects/request_spec.py,cm | 15:06 |
| alaski | the api and the object both concern me, for different reasons | 15:07 |
| *** logan2 has quit IRC | 15:07 | |
| *** jasondotstar has quit IRC | 15:08 | |
| *** abhishekk has quit IRC | 15:09 | |
| *** alex_klimov has joined #openstack-nova | 15:09 | |
| *** garyk1 has joined #openstack-nova | 15:10 | |
| *** garyk has quit IRC | 15:11 | |
| *** mpavone has quit IRC | 15:11 | |
| *** electrocucaracha has joined #openstack-nova | 15:11 | |
| *** jasondotstar has joined #openstack-nova | 15:12 | |
| kashyap | markus_z Hey, | 15:14 |
| kashyap | With your patch for serial console log applied, | 15:14 |
| kashyap | testing from git throws me this cryptic - 2015-06-24 10:41:16.013 CRITICAL nova [-] AssertionError: There is no script for 294 version | 15:14 |
| *** lyarwood has quit IRC | 15:15 | |
| *** lyarwood has joined #openstack-nova | 15:15 | |
| markus_z | kashyap: That's news to me. That's during launch I guess? | 15:15 |
| kashyap | markus_z: Just applied the patch to Nova git and `make stack` in DevStack | 15:16 |
| markus_z | kashyap: "make stack"? Is that new? | 15:17 |
| *** rwsu has quit IRC | 15:17 | |
| *** patrickeast has joined #openstack-nova | 15:17 | |
| kashyap | markus_z: Just an alias in DevStack to ./stack.sh | 15:17 |
| *** rwsu has joined #openstack-nova | 15:17 | |
| johnthetubaguy | bauzas: happy to help with brain frying, ideas to make this clearer welcome too :) | 15:17 |
| bauzas | johnthetubaguy: that's just 1 hour I'm reviewing your change... :) | 15:17 |
| bauzas | not kidding :) | 15:17 |
| markus_z | kashyap: Learning never stops. Thanks for the hint! I'll test it again and research the cause. | 15:18 |
| bauzas | hopefully, I'm multitasked | 15:18 |
| kashyap | markus_z: That's the complete trace - http://paste.openstack.org/show/320156/ | 15:18 |
| johnthetubaguy | bauzas: yeah, sorry about that, its kinda hairy to explain, I think its probably more a devref... oops | 15:18 |
| *** Piet has joined #openstack-nova | 15:18 | |
| kashyap | markus_z: I'll re-test in a few, bbiab. | 15:18 |
| *** unicell has joined #openstack-nova | 15:18 | |
| *** logan2 has joined #openstack-nova | 15:18 | |
| bauzas | johnthetubaguy: np, your spec is good, just trying to sort out all the things in my mind before replying | 15:19 |
| *** browne has joined #openstack-nova | 15:19 | |
| *** VW_ has quit IRC | 15:19 | |
| *** unicell1 has quit IRC | 15:19 | |
| kashyap | markus_z: DevStack recently got a Makefile with a few frequently used targets. :-) | 15:20 |
| *** ijw_ has quit IRC | 15:20 | |
| *** ijw_ has joined #openstack-nova | 15:21 | |
| markus_z | kashyap: I really like devstack. Always wished to have something like this in the past. | 15:21 |
| *** vilobhmm has quit IRC | 15:22 | |
| *** hshiina has quit IRC | 15:23 | |
| mriedem | kashyap: 2015-06-24 10:41:16.013 CRITICAL nova [-] AssertionError: There is no script for 294 version | 15:23 |
| mriedem | probably means your db sync is failing b/c your code is out of sync | 15:23 |
| mriedem | 294 would be a db migration script | 15:23 |
| kashyap | mriedem: It's just a day's old Nova git | 15:24 |
| kashyap | mriedem: I feared I'd hear something like that, I'll re-test with current-git-from-a-minute-ago | 15:24 |
| mriedem | 294 has been for a week or more | 15:24 |
| mriedem | 295 is the latest http://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/migrate_repo/versions | 15:25 |
| kashyap | mriedem: Thanks for decrypting that error | 15:25 |
| claudiub | i encountered something similar in the past. I think that you have a 294_asd.pyc in the migrrations folder, but no 294_asd.py. Had this because of checking out an older version. | 15:26 |
| *** ijw_ has quit IRC | 15:26 | |
| mriedem | hrm | 15:26 |
| mriedem | find . -name *.pyc -delete? | 15:26 |
| claudiub | yeah. | 15:26 |
| claudiub | that's how I "fixed" it. :) | 15:27 |
| *** VW_ has joined #openstack-nova | 15:27 | |
| *** ajayaa has quit IRC | 15:27 | |
| *** achanda_ has joined #openstack-nova | 15:27 | |
| *** scheuran has quit IRC | 15:28 | |
| kashyap | mriedem: Going to try that in a few. | 15:28 |
| *** marun has joined #openstack-nova | 15:29 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Split instance resize operation into 2 phases https://review.openstack.org/195088 | 15:29 |
| *** achanda has quit IRC | 15:29 | |
| *** agireud has joined #openstack-nova | 15:30 | |
| *** husanu1 has quit IRC | 15:30 | |
| danpb | dansmith: johnthetubaguy IMHO this particular spec is a reasonable candidate for trivial approval https://review.openstack.org/#/c/194221/ | 15:31 |
| *** VW_ has quit IRC | 15:31 | |
| edleafe | bauzas: lxsli: jaypipes: n0ano: johnthetubaguy: new scheduler meeting time poll: http://doodle.com/cuxa6xk63yb5y5tn | 15:31 |
| dansmith | danpb: johnthetubaguy: or just drop the spec and go with the code if it was just an unimplemented part of the previous spec | 15:31 |
| danpb | yep, that would work too | 15:32 |
| *** ijw_ has joined #openstack-nova | 15:32 | |
| bauzas | edleafe: oooooh UTC, much better | 15:32 |
| *** achanda_ has quit IRC | 15:32 | |
| johnthetubaguy | dansmith: I would say yes, but I guess its written now, so we can merge it I guess? i am easy either way I guess | 15:32 |
| dansmith | whichever | 15:32 |
| bauzas | edleafe: I was a little confused by the automatic TZ change from doodle | 15:32 |
| edleafe | bauzas: yeah, that's only possible with free text. Not supported in calendars | 15:32 |
| *** VW_ has joined #openstack-nova | 15:36 | |
| *** xyhuang has joined #openstack-nova | 15:36 | |
| *** rajesht has quit IRC | 15:37 | |
| *** logan2 has quit IRC | 15:39 | |
| *** yamahata has quit IRC | 15:39 | |
| kashyap | mriedem: markus_z : Hmm, no dice after this `find . -name '*.pyc' -exec rm -f \{} +`. | 15:41 |
| kashyap | But, a git pull` on Nova git brought in a massive diffstat | 15:42 |
| kashyap | Testing with that. | 15:42 |
| *** dramakri has joined #openstack-nova | 15:43 | |
| *** salv-orl_ has joined #openstack-nova | 15:45 | |
| *** zul has quit IRC | 15:45 | |
| *** zul has joined #openstack-nova | 15:46 | |
| *** patrickeast has quit IRC | 15:46 | |
| *** salv-orlando has quit IRC | 15:48 | |
| *** pask81 has joined #openstack-nova | 15:49 | |
| *** jasondotstar has quit IRC | 15:50 | |
| openstackgerrit | Dan Smith proposed openstack/nova-specs: Add service-version-number spec https://review.openstack.org/195156 | 15:52 |
| *** jasondot_ has joined #openstack-nova | 15:54 | |
| *** jasondot_ has quit IRC | 15:55 | |
| *** tongli has quit IRC | 15:55 | |
| *** tongli has joined #openstack-nova | 15:56 | |
| *** fawadkhaliq has joined #openstack-nova | 15:59 | |
| *** lyarwood has quit IRC | 15:59 | |
| *** lyarwood has joined #openstack-nova | 15:59 | |
| kfox1111 | johnthetubaguy: how are we doing with the nova instance user spec for the spec freeze? the barbican and keystone ptl's have both +1'ed it. | 16:00 |
| *** erlon has joined #openstack-nova | 16:00 | |
| jogo | markus_z: too bad we ended up doing the same thing :/ | 16:00 |
| *** tongli has quit IRC | 16:00 | |
| johnthetubaguy | kfox1111: I don't think its got any further right now | 16:00 |
| jogo | markus_z: I am most interested in getting a non ascii diagram, and don't really care whose we use | 16:01 |
| kfox1111 | think it will be ok to get a spec freeze exception for it? | 16:01 |
| johnthetubaguy | jogo: markus_z: I created the blueprint to try and help co-ordinate those efforts, please use it if it helps | 16:01 |
| markus_z | jogo: no harm done. Survival of the most beautiful diagram ;) | 16:01 |
| johnthetubaguy | markus_z: jogo: lol, that works | 16:01 |
| jogo | johnthetubaguy: I don't think we need a blueprint do we? | 16:01 |
| johnthetubaguy | jogo: its not required in away way shape or form, its just to help get it more reviews, and track what each other are up to | 16:02 |
| jogo | johnthetubaguy: what is the BP link? | 16:02 |
| jogo | markus_z johnthetubaguy: thoughts on http://docs-draft.openstack.org/40/194840/1/check/gate-nova-docs/3c74957//doc/build/html/architecture.html ? | 16:02 |
| johnthetubaguy | jogo: markus_z: this is the BP: https://blueprints.launchpad.net/nova/+spec/devref-refresh-liberty | 16:02 |
| *** tongli has joined #openstack-nova | 16:02 | |
| markus_z | jogo: I'd like to go with your SVG diagram and see what the responses are from the Nova folks. | 16:02 |
| johnthetubaguy | jogo: sorry trying to keep my mind on the specs right now | 16:02 |
| jogo | ahh the general BP, for a moment I thought you meant just for the diagram | 16:02 |
| *** patrickeast_ has joined #openstack-nova | 16:03 | |
| johnthetubaguy | jogo: ah, gotcha, hell no :) | 16:03 |
| *** patrickeast_ is now known as patrickeast | 16:03 | |
| johnthetubaguy | kfox1111: its in our backlog of 100 specs right now, I am afraid | 16:03 |
| markus_z | johnthetubaguy: Thanks, I'll used that already | 16:03 |
| kfox1111 | jaypipes: Did the most recent update resolve your concerns? | 16:03 |
| openstackgerrit | Joe Gordon proposed openstack/nova: Replace ascii art architecture diagram with svg image https://review.openstack.org/194840 | 16:03 |
| jwcroppe | johnthetubaguy: Re: https://review.openstack.org/#/c/141219/ - did you want me to revive the 'resize to same host' spec independently? | 16:03 |
| jogo | markus_z: your review on ^ would be useful | 16:03 |
| *** fawadkhaliq has quit IRC | 16:03 | |
| kfox1111 | johnthetubaguy: Thats why I'm asking for an exception. We've all put in a lot of work already into it and to get it killed in the backlog ins increadibly discuraging. | 16:03 |
| kfox1111 | we still have to write the code and review that too. that should be ok to do in parallel if the general idea is ok with everyone and we continue to polish the spec? | 16:04 |
| *** iamjarvo has quit IRC | 16:04 | |
| markus_z | jogo: I put myself to the reviewer list. In a few minutes I have to leave for today. I'll review it tomorrow morning. | 16:05 |
| johnthetubaguy | kfox1111: the risk is, if we can't get it approved for liberty, keeping the code alive is very painful | 16:05 |
| jaypipes | kfox1111: sorry, haven't had a chance to re-review... will try today, | 16:06 |
| jwcroppe | jaypipes: On your spec - https://review.openstack.org/#/c/183837/ - do we think that's still viable for Liberty, or looking like M? | 16:06 |
| kfox1111 | jaypipes: thanks. | 16:06 |
| jogo | markus_z: thanks | 16:06 |
| kfox1111 | johnthetubaguy: so's not having the feature. :/ | 16:06 |
| *** aix has quit IRC | 16:06 | |
| jaypipes | jwcroppe: thx for the reminder :) will try to push a rev today. not sure it's feasible for liberyt, but we cn try | 16:06 |
| kfox1111 | its really important to the app catalog I'm afraid. :/ | 16:06 |
| johnthetubaguy | kfox1111: I am hoping that will still get reviewed in time, its a good idea, overall, its just completing for time right now with everything else, sadly | 16:06 |
| *** dsanders has joined #openstack-nova | 16:06 | |
| kfox1111 | yeah. I understand. just trying to figure out the best way forward. | 16:07 |
| jwcroppe | jaypipes: no problem... I think it'll be interesting, but I think we still have a # of things to hash through :) | 16:07 |
| *** tongli_ has joined #openstack-nova | 16:07 | |
| *** jlanoux has quit IRC | 16:07 | |
| *** tongli has quit IRC | 16:08 | |
| openstackgerrit | Chris Gacsal proposed openstack/nova: virt: Allow multiple volume attachments https://review.openstack.org/193135 | 16:08 |
| openstackgerrit | Chris Gacsal proposed openstack/nova: comp: Allow multiple volume attachments https://review.openstack.org/193134 | 16:08 |
| openstackgerrit | Chris Gacsal proposed openstack/nova: REST API changes to support multiple volume attach https://review.openstack.org/153038 | 16:08 |
| *** annegentle has quit IRC | 16:08 | |
| kfox1111 | johnthetubaguy: no matter how things go, thanks for the help you've given. | 16:09 |
| *** garyk1 has quit IRC | 16:09 | |
| kfox1111 | the spec's been greatly improved because of it. | 16:09 |
| *** thorst has quit IRC | 16:09 | |
| *** yamahata has joined #openstack-nova | 16:10 | |
| jwcroppe | johnthetubaguy: I left a comment in https://review.openstack.org/#/c/141219/ and would be happy to bring the other spec for 'resize + same host' back to life... just let me know how you'd like to best proceed, whenever your vcpus aren't overcommitted. :) | 16:10 |
| openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Adds Hyper-V Shielded VMs spec https://review.openstack.org/192892 | 16:10 |
| openstackgerrit | Peter Savage proposed openstack/python-novaclient: Added marker functionality to flavours and images https://review.openstack.org/193212 | 16:12 |
| *** tongli_ has quit IRC | 16:12 | |
| *** edmondsw has quit IRC | 16:12 | |
| *** tjones2 has joined #openstack-nova | 16:12 | |
| *** belmoreira has quit IRC | 16:12 | |
| *** unicell has quit IRC | 16:12 | |
| *** arnaud____ has joined #openstack-nova | 16:13 | |
| *** chen12 has quit IRC | 16:15 | |
| *** davideagnello has quit IRC | 16:16 | |
| *** hemnafk is now known as hemna | 16:16 | |
| *** pask81 has quit IRC | 16:17 | |
| bauzas | uh uh, in order to follow the trend, could some people leave some comments in https://review.openstack.org/#/c/179224/ ? | 16:18 |
| bauzas | (that was my daily review request ping, I'm gonna back in my batcave) | 16:18 |
| claudiub | bauzas: is it a cave inside of a bat? | 16:19 |
| claudiub | it can't be that big. :D | 16:19 |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: Fix rebuild of an instance with a volume attached https://review.openstack.org/176891 | 16:20 |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: rebuild: make sure server is shut down before volumes are detached https://review.openstack.org/176892 | 16:20 |
| bauzas | claudiub: https://en.wikipedia.org/wiki/Batcave | 16:20 |
| bauzas | :) | 16:20 |
| *** arnaud____ has quit IRC | 16:20 | |
| *** destiny has quit IRC | 16:20 | |
| claudiub | bauzas: i know, I'm a fan of the batcave myself. :D When I was in vancouver, I've got myself some batman comics. :D | 16:20 |
| *** ZZelle has joined #openstack-nova | 16:21 | |
| claudiub | bauzas: anyways, you have a comment from johnthetubaguy on the commit | 16:21 |
| *** moshele has quit IRC | 16:21 | |
| bauzas | claudiub: sssssshut | 16:21 |
| *** mleroy_ has quit IRC | 16:21 | |
| *** nihilifer has quit IRC | 16:22 | |
| *** chen12 has joined #openstack-nova | 16:22 | |
| bauzas | claudiub: that doesn't count, he cheated by reviewing an old PS :p | 16:22 |
| claudiub | :)) | 16:22 |
| *** markus_z has quit IRC | 16:22 | |
| *** afazekas has quit IRC | 16:23 | |
| *** jordanP has quit IRC | 16:23 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova: Fixes delayed instance lifecycle events issue https://review.openstack.org/195180 | 16:24 |
| *** jcoufal has quit IRC | 16:26 | |
| *** sahid has quit IRC | 16:27 | |
| *** tongli has joined #openstack-nova | 16:28 | |
| *** vipuls is now known as vipul | 16:28 | |
| *** veena has joined #openstack-nova | 16:30 | |
| *** tongli has quit IRC | 16:33 | |
| *** salv-orl_ has quit IRC | 16:36 | |
| *** ijw_ has quit IRC | 16:36 | |
| *** ijw_ has joined #openstack-nova | 16:37 | |
| alexpilotti | johnthetubaguy danpb: https://review.openstack.org/#/c/195180/ the discussed manager fix for the event sync issue | 16:37 |
| *** haypo has quit IRC | 16:38 | |
| *** rushiagr_away is now known as rushiagr | 16:38 | |
| *** ijw_ has quit IRC | 16:41 | |
| *** armax has joined #openstack-nova | 16:43 | |
| *** ijw_ has joined #openstack-nova | 16:44 | |
| mriedem | dansmith: ha, no merge conflicts, i'm amazed https://review.openstack.org/#/c/195190/ | 16:47 |
| *** unicell has joined #openstack-nova | 16:47 | |
| *** jgrimm is now known as zz_jgrimm | 16:47 | |
| dansmith | nice | 16:47 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Fix Python 3 issues in nova.db.sqlalchemy https://review.openstack.org/195191 | 16:48 |
| *** alex_klimov has quit IRC | 16:48 | |
| *** otter768 has joined #openstack-nova | 16:50 | |
| *** tongli has joined #openstack-nova | 16:50 | |
| *** iamjarvo has joined #openstack-nova | 16:50 | |
| *** Sukhdev has joined #openstack-nova | 16:51 | |
| *** achanda has joined #openstack-nova | 16:52 | |
| *** dontalton has joined #openstack-nova | 16:52 | |
| mriedem | danpb: addressed the generic cinder encrypted volume thing here https://review.openstack.org/#/c/193673/ | 16:53 |
| mriedem | danpb: good call on that | 16:53 |
| *** zzzeek has joined #openstack-nova | 16:53 | |
| *** lsmola has quit IRC | 16:53 | |
| *** armax has quit IRC | 16:54 | |
| *** otter768 has quit IRC | 16:54 | |
| *** lucasagomes is now known as lucas-dinner | 16:55 | |
| *** derekh has quit IRC | 16:55 | |
| *** jasondot_ has joined #openstack-nova | 16:56 | |
| *** ijw__ has joined #openstack-nova | 16:57 | |
| *** mriedem is now known as mriedem_away | 16:59 | |
| *** nic has joined #openstack-nova | 16:59 | |
| *** ijw_ has quit IRC | 17:00 | |
| ildikov | jaypipes: ping | 17:01 |
| *** whenry has joined #openstack-nova | 17:01 | |
| *** marun_ has joined #openstack-nova | 17:02 | |
| *** marun has quit IRC | 17:03 | |
| *** doude has quit IRC | 17:03 | |
| *** achanda has quit IRC | 17:03 | |
| jaypipes | ildikov: on a call... leave me a message? :) | 17:04 |
| *** marun has joined #openstack-nova | 17:04 | |
| *** annegentle has joined #openstack-nova | 17:04 | |
| ildikov | jaypipes: sure, will do :) | 17:05 |
| *** davideagnello has joined #openstack-nova | 17:06 | |
| ildikov | jaypipes: I basically wanted to ask you about this spec: https://review.openstack.org/#/c/192755/ , as the deadline is close now | 17:06 |
| *** marun_ has quit IRC | 17:07 | |
| *** fawadkhaliq has joined #openstack-nova | 17:07 | |
| ildikov | jaypipes: it would be good to have it along with the base multi-attach BP, as thi sextension only covers the API changes that was left out from the original one | 17:07 |
| *** jasondot_ has quit IRC | 17:07 | |
| jaypipes | ildikov: please do involve johnthetubaguy, ndipanov and others in this. | 17:08 |
| ildikov | jaypipes: so I wonder if it somehow does not make it until the deadline, could we go for a freeze exception to be able to do all the required changes in this cycle? | 17:09 |
| ndipanov | I only briefly read the multi-attach spec | 17:09 |
| ndipanov | I am not sure I fully understand the usecase for it | 17:09 |
| ndipanov | I do understand the changes you need in Nova though | 17:09 |
| ildikov | ndipanov: it turned out in one review that the API changes are not covered properly in the base spec, so we uploaded an extension | 17:09 |
| *** garyk has joined #openstack-nova | 17:10 | |
| ildikov | ndipanov: you mean why we need to be able to attach one volume to multiple instances? | 17:10 |
| jaypipes | ildikov: gah, you just took the prize for the first person to use "freeze exception" this cycle :( | 17:10 |
| ndipanov | ildikov, and the orriginal spec is murged? | 17:10 |
| ndipanov | er merged | 17:10 |
| ndipanov | ? | 17:10 |
| ildikov | ndipanov: yes, here it is: https://review.openstack.org/#/c/179104/ | 17:11 |
| openstackgerrit | Daniel Berrange proposed openstack/nova-specs: VIF port config versioned objects and driver plugin library https://review.openstack.org/193668 | 17:11 |
| ildikov | jaypipes: I saw it in a mail earlier this week IIRC :S | 17:11 |
| *** danpb has quit IRC | 17:11 | |
| *** iamjarvo has quit IRC | 17:11 | |
| jaypipes | ildikov: :) | 17:11 |
| ildikov | jaypipes: I really wouldn't want to end up there! :) | 17:11 |
| ndipanov | I see. | 17:12 |
| *** jasondot_ has joined #openstack-nova | 17:12 | |
| *** whenry has quit IRC | 17:12 | |
| ndipanov | jaypipes, not sure if it's late or not but daily ping https://review.openstack.org/#/c/193576/ | 17:14 |
| ndipanov | ildikov, so what's the issue with this I don't understand | 17:15 |
| ildikov | ndipanov: that the extension spec is not approved yet :) | 17:15 |
| ildikov | ndipanov: so it's blocking some parts of the proposed code at the moment | 17:16 |
| ndipanov | ildikov, I | 17:17 |
| ndipanov | would refer you to the ML to see my opinion on the whole thing ;) | 17:17 |
| *** oro has joined #openstack-nova | 17:17 | |
| *** iamjarvo has joined #openstack-nova | 17:18 | |
| *** dane-fichter has joined #openstack-nova | 17:18 | |
| ildikov | ndipanov: do you mean the "spec administartion" thread that I haven't read yet? | 17:19 |
| ndipanov | I do mean that | 17:19 |
| ildikov | ndipanov: if it's about making the process somehow more flexible, than believe me, I'm on your side :) | 17:20 |
| ndipanov | ildikov, hehe thanks :) | 17:20 |
| ildikov | ndipanov: is there anything else I can do to make it happen? | 17:21 |
| *** ssurana has joined #openstack-nova | 17:22 | |
| ndipanov | well I'd help you but I really didn't look into that spec in enough detail - I can try to do it tomorrow | 17:22 |
| ndipanov | I honestly think that in this case your patches should just land | 17:22 |
| ndipanov | and then you can update the spec later | 17:22 |
| ndipanov | unless you completely failed to mention the API impact in the first draft | 17:22 |
| ndipanov | that merged | 17:22 |
| dane-fichter | ndipanov: that email was great. burn the specs | 17:23 |
| ndipanov | well actually even I don't agree with that :) | 17:23 |
| ndipanov | but we should fix the rpocess | 17:23 |
| ndipanov | a lot | 17:23 |
| ildikov | ndipanov: the merged spec is originated in Kilo, I think before the microversioning | 17:23 |
| ndipanov | so did you have to re=propose it? | 17:24 |
| ndipanov | I mean i don't know | 17:25 |
| ndipanov | it just seems that if people agreed that multi attach is a good idea | 17:25 |
| ildikov | ndipanov: it was re-proposed and is already merged for Liberty, but the content wasn't changed almost at all | 17:25 |
| ndipanov | so just go ahead and post patches | 17:25 |
| *** sudipto has joined #openstack-nova | 17:25 | |
| ndipanov | do you have -2s on them? | 17:25 |
| *** jasondot_ has quit IRC | 17:26 | |
| ildikov | ndipanov: the extension spec originates from Alex Xu's comment on this patch: https://review.openstack.org/#/c/153038/ | 17:27 |
| ildikov | ndipanov: not a -2, it was just a -1, I just wouldn't want to end up with surprises, when we're almost there to cover the whole feature | 17:28 |
| *** sudipto_ has joined #openstack-nova | 17:29 | |
| ildikov | ndipanov: I usually try to follow the processes on the first place, if possible :) | 17:29 |
| ildikov | dane-fichter: lol :) | 17:29 |
| ildikov | dane-fichter: IIRC the shortes Launchpad blueprint I've ever seen was one sentence long. And it wasn't even a long sentence. :) | 17:30 |
| ildikov | dane-fichter: we need to find some balance | 17:31 |
| *** veena has quit IRC | 17:31 | |
| *** jasondot_ has joined #openstack-nova | 17:31 | |
| *** VW_ has quit IRC | 17:32 | |
| *** ociuhandu has quit IRC | 17:32 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Remove compat_instance() https://review.openstack.org/192708 | 17:33 |
| sudipto | dansmith, johnthetubaguy I wanted to write up the 'whitelist' as recommended during the reviews for https://review.openstack.org/#/c/192760/ but i am struggling to understand few of the alembic constructs that would help me achieve this. Help? | 17:34 |
| *** salv-orlando has joined #openstack-nova | 17:34 | |
| ndipanov | ildikov, agreed | 17:34 |
| sudipto | dansmith, johnthetubaguy the changes would be in the test_migrations.py code to exclude fields that are not present in the model but actually exist in the DB. | 17:35 |
| dansmith | sudipto: I don't remember which whitelist .. it's been a couple weeks | 17:35 |
| ndipanov | ildikov, so I think that retroactively all specs need to be re-approved | 17:36 |
| sudipto | dansmith, Umm, sorry - was held up in other things. It's the one with the 'scheduled_at' attribute. You recommended that we shouldn't have a migration script and not drop this column from the db. | 17:36 |
| ndipanov | but API changes are as we've seen important | 17:36 |
| ndipanov | I keep wondering if we should have an API stabilization period | 17:37 |
| sudipto | dansmith, the problem with removing it from the model.py and not removing it from the db - arises in the test_migrations.py - that tries to compare the model with the db schema and complains that - they don't match. | 17:37 |
| dansmith | sudipto: ah right | 17:37 |
| dansmith | sudipto: I'd have to go examine that test to see what to do, I'm not super familiar with how it works | 17:37 |
| dansmith | sudipto: you might ask sdague | 17:38 |
| *** openstackgerrit has quit IRC | 17:38 | |
| sudipto | dansmith, ok sure, | 17:38 |
| *** jasondot_ has quit IRC | 17:38 | |
| dansmith | sudipto: I might be able to dig in next week if you're still stuck on it, but I'm overcommitted this week for sure | 17:38 |
| *** atuvenie has quit IRC | 17:38 | |
| *** openstackgerrit has joined #openstack-nova | 17:38 | |
| sudipto | dansmith, no problem, i just thought that there's a way to pass a list of attributes that could be excluded during that comparison. I will dig in more unless sdague has sometime to help :) | 17:39 |
| sdague | yeh, as far as I know that's enforced in oslo.db base tests | 17:39 |
| ildikov | danp: jogo: johnthetubaguy: jaypipes: tl;dr: We had to extend the multi-attach spec because of the lack of the API impact description in it. We uploaded a small new spec to cover this. Please take a look and approve or give some guidelines how to proceed with the administartion to get the feature fully implemented in Liberty. The spec is here: https://review.openstack.org/#/c/192755/ Thanks! | 17:39 |
| sdague | unfortunately most of the logic of test_migrations.py is no longer in nova, it's in oslo.db, and there are some assumptions which are a bit odd | 17:40 |
| dansmith | sdague: ugh | 17:40 |
| dansmith | sdague: he wants to remove a thing from the models so we can drop the column in the next cycle, but he can't get past that test :/ | 17:40 |
| sdague | yeh, I expect that's going to require loosening things in oslo.db | 17:41 |
| ildikov | danp: jogo: johnthetubaguy: jaypipes: s/extension spec/extension patch/ | 17:41 |
| *** whenry has joined #openstack-nova | 17:41 | |
| dansmith | sudipto: maybe dims could help you with the oslo bit of that | 17:41 |
| sudipto | dims, help!! | 17:42 |
| *** lpetrut has quit IRC | 17:42 | |
| * dims reading scrollback | 17:42 | |
| sdague | that was locked down because there was so much skew between models and db in the past, and convergence was actually a desired thing | 17:42 |
| dansmith | yeah, which is helpful | 17:43 |
| dansmith | the problem is, we can't drop things in the schema and still be able to apply the schema updates to running code where the model has a thing | 17:43 |
| *** gfa has quit IRC | 17:43 | |
| dansmith | sudipto: the other option might be to defer the thing on the model so that it's not loaded, which we could do now, and drop the column next time with the model drop, but that's really less desirable | 17:44 |
| dansmith | fixing the tests is much more preferable, IMHO | 17:44 |
| ildikov | ndipanov: how do you mean the API stabilization period? | 17:44 |
| ndipanov | well a period where we advertise that an API can change just in case we got it wrong | 17:45 |
| ndipanov | I don't know - it's tough | 17:45 |
| *** mgarza_ has joined #openstack-nova | 17:45 | |
| dansmith | ndipanov: I think that has been -2ed several times | 17:46 |
| ndipanov | dansmith, well it probably makes less sense for nova | 17:46 |
| sdague | right, because it's a chicken and egg game. | 17:46 |
| ndipanov | but componenets which pretend to be public APIs | 17:46 |
| sdague | if you advertise an API as not stable, no one uses it | 17:46 |
| ndipanov | but are actually only used by nova | 17:46 |
| ndipanov | like a ton of cinder stuff | 17:47 |
| sdague | so you never know if you got it wrong | 17:47 |
| ndipanov | sdague, right | 17:47 |
| sdague | if people start using things, you can't break them, if you want to keep users in the future | 17:47 |
| ildikov | so the cinder part of this change is ready | 17:47 |
| ndipanov | but that's not true for some bits of the cinder api | 17:47 |
| *** Mike_D_laptop has joined #openstack-nova | 17:47 | |
| ndipanov | for example | 17:47 |
| *** sudipto has quit IRC | 17:47 | |
| *** sudipto_ has quit IRC | 17:47 | |
| ildikov | and the impact is quite small on the Nova API | 17:47 |
| *** Raildo_ has quit IRC | 17:47 | |
| ndipanov | but we hold that to the same standard - I had a session about this in Atlanta | 17:47 |
| ildikov | as it is partially prepared for this change | 17:47 |
| ndipanov | and all of you guys thought I had too much fried chicken :) | 17:48 |
| *** sudipto has joined #openstack-nova | 17:48 | |
| ildikov | lol :) | 17:48 |
| sudipto | dansmith, sorry my laptop froze, it's been troubling me off late :( | 17:48 |
| *** fawadkhaliq has quit IRC | 17:49 | |
| dansmith | dims: nothing since you asked dims for help | 17:49 |
| dansmith | you didn't miss anything | 17:49 |
| sudipto | ok :) | 17:50 |
| dansmith | sudipto: maybe continue discussion in #openstack-oslo? | 17:51 |
| sudipto | dansmith, ok sure. | 17:51 |
| *** yassine_ has quit IRC | 17:51 | |
| *** toure has quit IRC | 17:52 | |
| dansmith | alaski: on the min version thing, I can add something to the spec mentioning the possibility, but we don't need to define how that logic works until M when we can depend on this being in place, right? | 17:53 |
| *** toure has joined #openstack-nova | 17:53 | |
| *** lyarwood has quit IRC | 17:54 | |
| *** lyarwood has joined #openstack-nova | 17:54 | |
| dims | sudipto: 192760 does not have the updated test that shows a failure, right? | 17:55 |
| *** zul has quit IRC | 17:55 | |
| *** amotoki has quit IRC | 17:55 | |
| *** achanda has joined #openstack-nova | 17:56 | |
| sudipto | dims, if i remove the scheduled_at from the DB and not add the migration script, then the problem is seen in the test_migrations.py - 192760 is not updated with that. It has the migration script on it - so the tests work. | 17:56 |
| sudipto | dims, sorry my bad - i meant if i remove scheduled_at from the Model | 17:57 |
| *** moshele has joined #openstack-nova | 17:58 | |
| *** agireud has quit IRC | 17:58 | |
| *** thedodd has quit IRC | 17:59 | |
| *** thangp has quit IRC | 17:59 | |
| *** logan2 has joined #openstack-nova | 18:00 | |
| *** harlowja has quit IRC | 18:00 | |
| *** thangp has joined #openstack-nova | 18:01 | |
| dims | sudipto: we could do some stuff in oslo.db, not sure what exactly yet (will have to get roman/mike), can you update the review to create the failure and add a bug in oslo.db please? | 18:01 |
| sudipto | dims, will do! Thanks!! | 18:01 |
| sudipto | dansmith, sdague is it a good idea to remove this constraint of not comparing the model and the db schemas in the future? | 18:02 |
| *** lpetrut has joined #openstack-nova | 18:02 | |
| dansmith | sudipto: no, we just need the constraint to be the right direction | 18:02 |
| openstackgerrit | Dan Smith proposed openstack/nova-specs: Add service-version-number spec https://review.openstack.org/195156 | 18:02 |
| *** jwcroppe_ has joined #openstack-nova | 18:03 | |
| *** harlowja has joined #openstack-nova | 18:04 | |
| sudipto | dansmith, was just wondering if I add the bug, should it be about provisioning a way that would help us such constraints like a whitelist of sorts - that should be skipped? | 18:04 |
| dansmith | sudipto: yep, I think so | 18:04 |
| *** oro has quit IRC | 18:04 | |
| sudipto | sorry about the typos on the above line. Ok sure. | 18:04 |
| dansmith | sudipto: maybe call it "allow_missing_from_model=" and "allow_missing_from_db=" or something | 18:04 |
| dansmith | the second we don't really care about, but might be good for symmetry | 18:04 |
| sudipto | dansmith, perfect. | 18:05 |
| *** ad has joined #openstack-nova | 18:05 | |
| *** jodah- is now known as jodah | 18:06 | |
| *** jwcroppe has quit IRC | 18:06 | |
| ad | can anyone help me with the metadata service? in my VM my keys are not getting injected and i have an error "calling http://169.254.169.254/2009-04-04/meta-data/instance-id' bad status code [400] | 18:06 |
| *** electroc_ has joined #openstack-nova | 18:07 | |
| *** electrocucaracha has quit IRC | 18:10 | |
| alaski | dansmith: yeah, that's a good point | 18:10 |
| *** husanu has quit IRC | 18:11 | |
| *** tpeoples has joined #openstack-nova | 18:11 | |
| *** whenry has quit IRC | 18:11 | |
| *** agireud has joined #openstack-nova | 18:12 | |
| *** ildikov has quit IRC | 18:12 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup https://review.openstack.org/192760 | 18:13 |
| *** husanu has joined #openstack-nova | 18:13 | |
| *** Nikolay_St has joined #openstack-nova | 18:14 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup https://review.openstack.org/192760 | 18:15 |
| *** atuvenie has joined #openstack-nova | 18:17 | |
| *** VW_ has joined #openstack-nova | 18:17 | |
| *** annashen has joined #openstack-nova | 18:18 | |
| *** xyhuang has quit IRC | 18:18 | |
| *** moshele has quit IRC | 18:18 | |
| *** boris-42 has joined #openstack-nova | 18:18 | |
| *** Nikolay_St has quit IRC | 18:19 | |
| *** jasondot_ has joined #openstack-nova | 18:21 | |
| *** achanda has quit IRC | 18:27 | |
| *** achanda has joined #openstack-nova | 18:28 | |
| *** armax has joined #openstack-nova | 18:29 | |
| *** matrohon has joined #openstack-nova | 18:31 | |
| *** achanda has quit IRC | 18:33 | |
| *** thedodd has joined #openstack-nova | 18:33 | |
| *** achanda has joined #openstack-nova | 18:40 | |
| *** matrohon has quit IRC | 18:42 | |
| *** matrohon has joined #openstack-nova | 18:43 | |
| *** salv-orlando has quit IRC | 18:44 | |
| *** logan2 has quit IRC | 18:46 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: Multipath commands with error messages in stdout fail to parse https://review.openstack.org/172660 | 18:46 |
| openstackgerrit | Dan Smith proposed openstack/nova: Remove compat_instance() https://review.openstack.org/192708 | 18:47 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: Multipath device action incorrectly parsed as the device name https://review.openstack.org/193119 | 18:47 |
| *** tongli has quit IRC | 18:48 | |
| *** Sukhdev has quit IRC | 18:51 | |
| *** otter768 has joined #openstack-nova | 18:51 | |
| *** fawadkhaliq has joined #openstack-nova | 18:52 | |
| jaypipes | ndipanov: k, https://review.openstack.org/#/c/193576/ reviewed. | 18:53 |
| ndipanov | oh the suspense of gerrit slow loading | 18:54 |
| ndipanov | hmmm you raise interesting points | 18:55 |
| jaypipes | ndipanov: just add a tiny bit more clarity in a couple areas, but nothing major. | 18:56 |
| *** otter768 has quit IRC | 18:56 | |
| *** amotoki has joined #openstack-nova | 18:56 | |
| jaypipes | ndipanov: this patch series from tdurakov will also be of interest to you:https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1455460,n,z | 18:56 |
| ndipanov | jaypipes, so did you happen to see this https://review.openstack.org/#/c/163440/ | 18:57 |
| jaypipes | ndipanov: no, I hadn't. thx for letting me know about it. | 18:57 |
| ndipanov | last 3 comments discuss why it cannot be reconstructed after it has been claimed | 18:57 |
| ndipanov | simply from flavor and image | 18:58 |
| jaypipes | k, reading | 18:58 |
| *** electrocucaracha has joined #openstack-nova | 19:00 | |
| *** amotoki has quit IRC | 19:01 | |
| jaypipes | ndipanov: ok, I'm honestly befuddled by your comments there... | 19:03 |
| ndipanov | I knew it :D | 19:03 |
| *** matrohon has quit IRC | 19:03 | |
| *** electroc_ has quit IRC | 19:03 | |
| *** artom has quit IRC | 19:04 | |
| jaypipes | ndipanov: are you saying that if I launch a VM on a host with NUMA support, that I can only migrate that VM to another host that has the *exact* CPU pinset and numa node selection available that the original host did? | 19:04 |
| *** matrohon has joined #openstack-nova | 19:04 | |
| ndipanov | no | 19:04 |
| ndipanov | but think of the way how resource tracking works | 19:04 |
| ndipanov | it goes through a list of migrations and applies their usage tto the resources | 19:04 |
| *** jwcroppe has joined #openstack-nova | 19:05 | |
| *** jasondot_ has quit IRC | 19:05 | |
| ndipanov | if you re-calculate the placement from flavor and image every time | 19:05 |
| ndipanov | it can change from run to run | 19:05 |
| jaypipes | ndipanov: I'm not following why you need to consider the exact NUMA node and pinset selection in the migration placement/scheduling decision. | 19:06 |
| ndipanov | because once you have claimed it there are situations where a previously successfull claim could be no longer successful | 19:06 |
| *** jwcroppe_ has quit IRC | 19:06 | |
| ndipanov | for example a instance that does not care which numa node a PCI device belongs to | 19:07 |
| ndipanov | might take it from the one that does care | 19:07 |
| ndipanov | even though the first one could use a different one | 19:07 |
| ndipanov | so you _have to_ reserve the exact ones with a migration claim too | 19:07 |
| ndipanov | it actually took me a while to figure it out too | 19:07 |
| ndipanov | as can be seen from the fact that my original code does not account for it :D | 19:08 |
| jaypipes | ndipanov: But don't we already have a Migration object that we can put this information into? Why put it on the Instance itself? | 19:08 |
| ndipanov | ah so | 19:08 |
| jaypipes | ndipanov: sorry, I moved on to my next question... you answered the firt, thanks :) | 19:08 |
| ndipanov | that's by dansmith suggestion on that patch | 19:08 |
| ndipanov | yes | 19:08 |
| dansmith | yeah, I wanted that | 19:08 |
| ndipanov | and I kind of agree with it | 19:09 |
| ndipanov | I like instance owning that stuff | 19:09 |
| *** Longgeek has quit IRC | 19:09 | |
| ndipanov | I think it makes for an easier programming model | 19:09 |
| dansmith | it's also stuff that is meaningless after the migration | 19:09 |
| dansmith | we keep migration records for history, but don't need to keep that stuff | 19:09 |
| dansmith | we could delete it off the migration object of course | 19:09 |
| dansmith | but it seems clearly to be the instance-specific context information about the migration | 19:10 |
| *** dramakri has quit IRC | 19:10 | |
| ndipanov | so yes - it would also make sense on the migration object | 19:10 |
| jaypipes | dansmith, ndipanov: got it. OK, that's a great point. | 19:10 |
| jaypipes | ndipanov: cool, I'm good with this. | 19:10 |
| jaypipes | thx for walking me through it. | 19:10 |
| *** electrocucaracha has quit IRC | 19:11 | |
| ndipanov | jaypipes, anytime :) | 19:12 |
| ndipanov | I will update the rest soon | 19:13 |
| jaypipes | ndipanov: cheers mate | 19:14 |
| jaypipes | ndipanov: wait... so the spec process might actually be useful here? ;P | 19:14 |
| *** harlowja has quit IRC | 19:15 | |
| *** harlowja has joined #openstack-nova | 19:15 | |
| ndipanov | jaypipes, yeah I mean I didn;t want to throw the baby out with the mother... (or something) | 19:16 |
| ndipanov | just start a discussion | 19:16 |
| jaypipes | ndipanov: I'm just pulling your chain :) | 19:16 |
| *** mgarza_ has quit IRC | 19:17 | |
| *** ildikov has joined #openstack-nova | 19:17 | |
| *** VW_ has quit IRC | 19:18 | |
| *** lyarwood has quit IRC | 19:18 | |
| *** lyarwood has joined #openstack-nova | 19:19 | |
| *** matrohon has quit IRC | 19:19 | |
| *** matrohon has joined #openstack-nova | 19:19 | |
| *** annashen has quit IRC | 19:20 | |
| *** arnaud____ has joined #openstack-nova | 19:20 | |
| *** READ10 has quit IRC | 19:21 | |
| *** Marga_ has quit IRC | 19:21 | |
| *** LukeHinds has quit IRC | 19:22 | |
| openstackgerrit | dane-fichter proposed openstack/nova-specs: Nova Support of Glance Image Signing https://review.openstack.org/188874 | 19:22 |
| *** atuvenie has quit IRC | 19:22 | |
| *** gszasz has quit IRC | 19:23 | |
| *** achanda has quit IRC | 19:23 | |
| *** mgarza_ has joined #openstack-nova | 19:24 | |
| dane-fichter | hey all, if you get a chance, please take a look at https://review.openstack.org/188874 . I'm a first-time contributor trying to get this spec approved before the spec/feature freezes | 19:24 |
| *** arnaud____ has quit IRC | 19:25 | |
| openstackgerrit | Michael Sambol proposed openstack/nova: Move existing V2 to legacy-v2 https://review.openstack.org/193725 | 19:26 |
| *** alexpilotti has quit IRC | 19:26 | |
| *** annegentle has quit IRC | 19:26 | |
| *** salv-orlando has joined #openstack-nova | 19:27 | |
| *** annegentle has joined #openstack-nova | 19:28 | |
| *** Marga_ has joined #openstack-nova | 19:31 | |
| *** sudipto has quit IRC | 19:31 | |
| edleafe | sambol: you around? | 19:32 |
| *** Nikolay_St has joined #openstack-nova | 19:32 | |
| sambol | Yes, hi Ed | 19:32 |
| edleafe | Saw your patchset. I'm wondering if there might be a problem in requiring a change in api-paste.ini | 19:33 |
| edleafe | sambol: alex_xu_ had noted that we can't rely on users updating that file | 19:33 |
| edleafe | sambol: that's why we couldn't rename APIRouter, etc. | 19:33 |
| *** Sukhdev has joined #openstack-nova | 19:33 | |
| *** Marga_ has quit IRC | 19:34 | |
| *** Marga_ has joined #openstack-nova | 19:34 | |
| *** nkrinner has joined #openstack-nova | 19:34 | |
| sambol | @edleafe: shoot, I think you're right. that was definitely the error, however. all tests are passing now. | 19:35 |
| edleafe | sambol: understood. Passing tests is good; not breaking deployments when they upgrade is even better :) | 19:36 |
| edleafe | sambol: we'll probably have to wait until alex_xu_ is back from his baby vacation to figure this out. | 19:36 |
| *** jasondotstar has joined #openstack-nova | 19:36 | |
| sambol | edleafe: sounds good. | 19:38 |
| *** jwcroppe has quit IRC | 19:39 | |
| anish | jogo: danpb: can I get a review on https://review.openstack.org/#/c/193414/ re bug https://bugs.launchpad.net/nova/+bug/1466730 | 19:40 |
| openstack | Launchpad bug 1466730 in OpenStack Compute (nova) "iscsi transport support broken due to use of incorrect oslo_config choices support" [Undecided,In progress] - Assigned to Anish Bhatt (anish7) | 19:40 |
| *** zul has joined #openstack-nova | 19:40 | |
| *** jwcroppe has joined #openstack-nova | 19:40 | |
| *** dramakri has joined #openstack-nova | 19:40 | |
| *** zul has quit IRC | 19:44 | |
| *** zul has joined #openstack-nova | 19:45 | |
| *** fawadkhaliq has quit IRC | 19:46 | |
| *** Marga_ has quit IRC | 19:46 | |
| dansmith | dane-fichter: it would be awesome if you could make sure you cover the items other reviewers have asked for before you ask for more review here | 19:47 |
| *** zsais has joined #openstack-nova | 19:48 | |
| *** packet has quit IRC | 19:53 | |
| dane-fichter | dansmith: I included what Dan Berrange referenced in the 'Problem description' section. I guess I don't understand what the purpose of the 'Use case' section of a spec is for | 19:54 |
| *** Marga_ has joined #openstack-nova | 19:55 | |
| jaypipes | dane-fichter: it's for describing the use cases that would be fulfilled by the code you are proposing to add to Nova. | 19:55 |
| dane-fichter | jaypipes: I feel as if I am describing exactly how the feature will be used | 19:57 |
| *** amotoki has joined #openstack-nova | 19:57 | |
| dane-fichter | jaypipes: in my view, the use cases are: 1) a user would like to boot a signed image 2) a user would like to sign an image | 19:58 |
| jaypipes | dane-fichter: a use case should be of the form "As a <ROLE>, I wish to do <ACTION> in order to <EXPECTED_RESULT> | 19:58 |
| jaypipes | dane-fichter: that doesn't explain *why* a user would want to sign an image or boot a signed image. | 19:59 |
| *** dansmith has quit IRC | 19:59 | |
| dane-fichter | jaypipes: ok, I will rephrase. | 19:59 |
| jaypipes | dane-fichter: ok. | 19:59 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/objects/test*.py https://review.openstack.org/187785 | 19:59 |
| *** Marga_ has quit IRC | 20:00 | |
| *** dansmith has joined #openstack-nova | 20:00 | |
| *** amotoki has quit IRC | 20:01 | |
| *** Marga_ has joined #openstack-nova | 20:02 | |
| *** shihanzhang has joined #openstack-nova | 20:02 | |
| *** electrocucaracha has joined #openstack-nova | 20:04 | |
| *** oro has joined #openstack-nova | 20:05 | |
| *** hanzhang has quit IRC | 20:05 | |
| *** dane-fichter has quit IRC | 20:06 | |
| *** afazekas has joined #openstack-nova | 20:08 | |
| *** earlephilhower has quit IRC | 20:09 | |
| *** annashen has joined #openstack-nova | 20:09 | |
| *** ijw_ has joined #openstack-nova | 20:11 | |
| *** afazekas has quit IRC | 20:12 | |
| *** BobBall is now known as BobBall_AWOL | 20:12 | |
| *** tonytan4ever has quit IRC | 20:13 | |
| *** ijw__ has quit IRC | 20:14 | |
| *** VW_ has joined #openstack-nova | 20:17 | |
| *** VW_ has quit IRC | 20:17 | |
| *** harlowja has quit IRC | 20:18 | |
| *** VW_ has joined #openstack-nova | 20:18 | |
| *** harlowja has joined #openstack-nova | 20:18 | |
| *** oro has quit IRC | 20:19 | |
| *** VW_ has quit IRC | 20:19 | |
| *** VW_ has joined #openstack-nova | 20:19 | |
| *** iamjarvo has quit IRC | 20:22 | |
| *** thangp has quit IRC | 20:26 | |
| *** nkrinner has quit IRC | 20:29 | |
| *** jasondotstar has quit IRC | 20:29 | |
| *** rushiagr is now known as rushiagr_away | 20:31 | |
| *** tonytan4ever has joined #openstack-nova | 20:33 | |
| *** matrohon has quit IRC | 20:33 | |
| *** mestery has quit IRC | 20:35 | |
| *** matrohon has joined #openstack-nova | 20:35 | |
| *** jasondotstar has joined #openstack-nova | 20:36 | |
| *** alex_klimov has joined #openstack-nova | 20:36 | |
| *** annegentle has quit IRC | 20:37 | |
| *** thorst has joined #openstack-nova | 20:37 | |
| bauzas | jaypipes: hey, given your workload, could you please try to find some time for reviewing https://review.openstack.org/#/c/179224/13/specs/liberty/approved/check-destination-on-migrations.rst,cm | 20:37 |
| bauzas | jaypipes: I hate asking people for that but you know that I want to have this spec prioritary | 20:38 |
| jaypipes | bauzas: yup, it's near top of the queue. | 20:38 |
| bauzas | jaypipes: coolness | 20:38 |
| *** Nikolay_St has quit IRC | 20:41 | |
| *** ad has quit IRC | 20:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/194325 | 20:43 |
| *** earlephilhower has joined #openstack-nova | 20:47 | |
| openstackgerrit | Marian Horban proposed openstack/nova: Added processing /compute URL https://review.openstack.org/195303 | 20:49 |
| *** mestery has joined #openstack-nova | 20:49 | |
| *** jecarey has quit IRC | 20:50 | |
| *** dprince has quit IRC | 20:51 | |
| *** otter768 has joined #openstack-nova | 20:52 | |
| *** jasondotstar has quit IRC | 20:54 | |
| *** ZZelle has quit IRC | 20:54 | |
| *** tonytan4ever has quit IRC | 20:55 | |
| *** mriedem has joined #openstack-nova | 20:55 | |
| *** tonytan4ever has joined #openstack-nova | 20:56 | |
| *** mriedem_away has quit IRC | 20:57 | |
| *** otter768 has quit IRC | 20:57 | |
| *** salv-orl_ has joined #openstack-nova | 20:57 | |
| *** belmoreira has joined #openstack-nova | 20:58 | |
| *** ndipanov has quit IRC | 20:59 | |
| *** tonytan4ever has quit IRC | 21:00 | |
| *** salv-orlando has quit IRC | 21:01 | |
| *** ZZelle has joined #openstack-nova | 21:01 | |
| *** ivasev has quit IRC | 21:02 | |
| *** matrohon has quit IRC | 21:02 | |
| *** tonytan4ever has joined #openstack-nova | 21:04 | |
| *** rlrossit has left #openstack-nova | 21:04 | |
| *** VW_ has quit IRC | 21:05 | |
| *** Longgeek has joined #openstack-nova | 21:09 | |
| *** dave-mccowan has quit IRC | 21:12 | |
| *** dramakri has quit IRC | 21:14 | |
| *** Longgeek has quit IRC | 21:15 | |
| openstackgerrit | Marian Horban proposed openstack/nova: Logging corrected https://review.openstack.org/195321 | 21:15 |
| mriedem | alaski: there are backports proposed to stable/kilo for these: | 21:15 |
| mriedem | https://review.openstack.org/#/q/status:merged+project:openstack/nova+branch:master+topic:bug/1404268,n,z | 21:15 |
| mriedem | alaski: in the bug report, you call out https://review.openstack.org/#/c/160348/ as being part of the bug fix | 21:15 |
| mriedem | alaski: i'm trying to figure out if https://review.openstack.org/#/c/160348/ is related to the point that it would also need to be backported for the fix to be complete in stable/kilo | 21:16 |
| mriedem | alaski: ignore, https://review.openstack.org/#/c/160348/ is in stable/kilo already :) | 21:16 |
| *** vishwana_ has quit IRC | 21:17 | |
| *** vishwanathj has joined #openstack-nova | 21:18 | |
| alaski | mriedem: one sec, in cells meeting. but https://review.openstack.org/#/c/160348/ may have been the cause. need to look again | 21:18 |
| *** dramakri has joined #openstack-nova | 21:19 | |
| *** erlon has quit IRC | 21:20 | |
| *** lpetrut has quit IRC | 21:22 | |
| *** morganfainberg is now known as ayspryn | 21:24 | |
| *** ayspryn is now known as morganfainberg | 21:24 | |
| mriedem | chaochin: i just commented on https://review.openstack.org/#/c/183144/ - if i read that correctly, anytime someone wants to use greenthread.spawn in nova they should be using nova.utils.spawn, right? if so, we should have a hacking check for that | 21:25 |
| *** dboik_ has quit IRC | 21:27 | |
| mriedem | chaochin: i opened bug 1468513 for a hacking check gap | 21:28 |
| openstack | bug 1468513 in OpenStack Compute (nova) "hacking check needed for using greenthread.spawn()" [Low,Triaged] https://launchpad.net/bugs/1468513 | 21:28 |
| mriedem | ctrath: ^ low-hanging-fruit | 21:28 |
| *** jasondotstar has joined #openstack-nova | 21:29 | |
| *** alexpilotti has joined #openstack-nova | 21:30 | |
| alaski | mriedem: okay, I'm up to speed now I think | 21:33 |
| alaski | mriedem: and I think I can ignore it, as you said :) | 21:33 |
| *** Sukhdev has quit IRC | 21:33 | |
| mriedem | alaski: cool, i +2'ed the 2 backports | 21:34 |
| *** dave-mccowan has joined #openstack-nova | 21:36 | |
| alaski | mriedem: there were two issues with the same symptom, context info missing from the logs. so the fixes aren't intertwined despite the lp comment | 21:36 |
| mriedem | alaski: ok | 21:37 |
| *** alexpilotti has quit IRC | 21:40 | |
| ctrath | mriedem: ok | 21:41 |
| *** alexpilotti has joined #openstack-nova | 21:41 | |
| ctrath | mriedem: just looked though that. Looks like that is going to get called when tox gets run? | 21:41 |
| *** tonytan4ever has quit IRC | 21:42 | |
| mriedem | ctrath: hacking checks are run as part of pep8 | 21:43 |
| mriedem | kashyap: i think https://review.openstack.org/#/c/193497/ might be worth the backport now, but can you fix the pep8/py27 problems? | 21:43 |
| ctrath | mriedem: looks pretty simple to implement. Only if I could :-/ | 21:48 |
| mriedem | ctrath: that's why i assigned it to rlrossit :) | 21:48 |
| ctrath | mriedem: because it's easy? ;-) | 21:48 |
| mriedem | that and he's ok with the CLA | 21:48 |
| ctrath | mriedem: is there a way that I can add myself to the reviewer list for rlrossit's code or do I just tell him to add me as a reviewer? | 21:51 |
| mriedem | ctrath: you can add yourself if/when he has a change up for review | 21:51 |
| ctrath | ok, so, I can't add myself to the bug | 21:51 |
| mriedem | ctrath: you could | 21:52 |
| mriedem | there is a subscribe link | 21:52 |
| ctrath | does that automatically attach me to be a reviewer of the code, then? | 21:52 |
| mriedem | ctrath: no, but you should be notified when there is an LP comment that a change is up for review | 21:52 |
| mriedem | although LP has been flaky lately on putting those comments in the bug reports | 21:52 |
| ctrath | mriedem: ok. thanks. I'll attempt it and see what happens | 21:53 |
| *** lucas-dinner has quit IRC | 21:53 | |
| *** annashen has quit IRC | 21:54 | |
| *** sdake_ has joined #openstack-nova | 21:55 | |
| *** ian_brown has joined #openstack-nova | 21:57 | |
| *** amotoki has joined #openstack-nova | 21:58 | |
| *** ijuwang has quit IRC | 21:58 | |
| *** sdake has quit IRC | 21:59 | |
| *** dboik has joined #openstack-nova | 22:01 | |
| *** IanGovett has quit IRC | 22:01 | |
| *** Piet has quit IRC | 22:02 | |
| *** amotoki has quit IRC | 22:04 | |
| *** ijuwang has joined #openstack-nova | 22:05 | |
| *** annashen has joined #openstack-nova | 22:05 | |
| *** busterswt has quit IRC | 22:08 | |
| *** agireud has quit IRC | 22:09 | |
| *** jasondotstar has quit IRC | 22:09 | |
| *** Longgeek has joined #openstack-nova | 22:11 | |
| mriedem | dansmith: are you still ok with this backport after ndipanov updated it? https://review.openstack.org/#/c/185591/ | 22:14 |
| mriedem | dansmith: it's big and ugly and nfv'y | 22:14 |
| cburgess_ | NFV needs another character so it can be a 4 letter word. | 22:15 |
| mriedem | NFFV | 22:16 |
| mriedem | FNFV? | 22:16 |
| cburgess_ | Yeah | 22:16 |
| cburgess_ | That was what I was going to suggest.. | 22:16 |
| *** Longgeek has quit IRC | 22:16 | |
| cburgess_ | Wait.. why do I have _... | 22:16 |
| mriedem | as long as it's opt-in, meh | 22:16 |
| *** belmoreira has quit IRC | 22:19 | |
| dansmith | mriedem: will look | 22:19 |
| *** alexpilotti has quit IRC | 22:19 | |
| cburgess_ | dansmith: Thank you so much for teaching me git review -d in Paris. I don't know how i lived without this. | 22:20 |
| dansmith | heh | 22:20 |
| *** thorst has quit IRC | 22:21 | |
| dansmith | mriedem: I don't think he did anything other than rebase it, right? | 22:21 |
| mriedem | dansmith: i didn't dig in | 22:21 |
| dansmith | seems like it to me | 22:21 |
| mriedem | i saw you were +2 and wanted to see if you were still after that | 22:21 |
| mriedem | before i dug in, b/c i don't want to dig into this one | 22:21 |
| *** thedodd has quit IRC | 22:21 | |
| dansmith | mriedem: just added it back | 22:22 |
| mriedem | but if i don't there will be angry threads about how stable review process is broken :) | 22:22 |
| *** chen12 has quit IRC | 22:22 | |
| *** ijw_ has quit IRC | 22:22 | |
| dansmith | I think there will be angry threads regardless | 22:22 |
| *** Sukhdev has joined #openstack-nova | 22:23 | |
| *** VW_ has joined #openstack-nova | 22:23 | |
| *** ijw_ has joined #openstack-nova | 22:23 | |
| *** annashen has quit IRC | 22:23 | |
| *** radek__ has quit IRC | 22:23 | |
| *** mdrabe has quit IRC | 22:24 | |
| cburgess_ | NFV is the path to the dark side. NFV leads to anger. Anger leads to hate. Hate leads to suffering. | 22:26 |
| *** nic has quit IRC | 22:27 | |
| *** zsais has quit IRC | 22:28 | |
| *** annashen has joined #openstack-nova | 22:28 | |
| *** ijw_ has quit IRC | 22:29 | |
| *** belmoreira has joined #openstack-nova | 22:29 | |
| *** VW_ has quit IRC | 22:31 | |
| *** salv-orl_ has quit IRC | 22:34 | |
| *** salv-orlando has joined #openstack-nova | 22:35 | |
| *** belmoreira has quit IRC | 22:35 | |
| *** ijw_ has joined #openstack-nova | 22:38 | |
| openstackgerrit | Peter Savage proposed openstack/python-novaclient: Added marker functionality to flavours and images https://review.openstack.org/193212 | 22:39 |
| *** ZZelle has quit IRC | 22:40 | |
| *** tjones2 has quit IRC | 22:42 | |
| *** annashen has quit IRC | 22:42 | |
| *** Piet has joined #openstack-nova | 22:43 | |
| mriedem | claudiub: fyi, you were waiting to +2 https://review.openstack.org/#/c/189328/ until the backport to stable/kilo was approved - i approved the stable/kilo backport today | 22:44 |
| *** ctrath has quit IRC | 22:45 | |
| claudiub | mriedem: thanks for letting me know. I will aprove it when it merges in kilo. :) | 22:46 |
| *** ijuwang has quit IRC | 22:46 | |
| *** dboik has quit IRC | 22:46 | |
| *** tjones1 has quit IRC | 22:47 | |
| *** otter768 has joined #openstack-nova | 22:53 | |
| *** belmoreira has joined #openstack-nova | 22:53 | |
| *** aysyanne has quit IRC | 22:56 | |
| *** sorrison_laptop has joined #openstack-nova | 22:56 | |
| *** mgarza_ has quit IRC | 22:57 | |
| *** otter768 has quit IRC | 22:57 | |
| openstackgerrit | Victor Stinner proposed openstack/nova: Fix Python 3 issues in nova.db.sqlalchemy https://review.openstack.org/195191 | 22:58 |
| *** mmedvede has quit IRC | 22:58 | |
| *** amotoki has joined #openstack-nova | 22:59 | |
| *** zzzeek has quit IRC | 23:01 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova: FC Eliminate need to store devices list https://review.openstack.org/195350 | 23:02 |
| *** amotoki has quit IRC | 23:04 | |
| *** salv-orlando has quit IRC | 23:04 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Adds Hyper-V Shielded VMs spec https://review.openstack.org/192892 | 23:05 |
| *** mleroy has quit IRC | 23:11 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Adds Hyper-V vTPM devices spec https://review.openstack.org/195068 | 23:11 |
| *** mleroy has joined #openstack-nova | 23:13 | |
| *** IanGovett has joined #openstack-nova | 23:14 | |
| *** belmoreira has quit IRC | 23:15 | |
| *** exploreshaifali has joined #openstack-nova | 23:16 | |
| *** ijw__ has joined #openstack-nova | 23:22 | |
| *** ijw_ has quit IRC | 23:25 | |
| *** otter768 has joined #openstack-nova | 23:25 | |
| *** bradjones has quit IRC | 23:26 | |
| *** bradjones has joined #openstack-nova | 23:26 | |
| *** bradjones has quit IRC | 23:26 | |
| *** bradjones has joined #openstack-nova | 23:26 | |
| *** idevops has joined #openstack-nova | 23:27 | |
| *** idevops has quit IRC | 23:27 | |
| *** alex_klimov has quit IRC | 23:29 | |
| *** Marga_ has quit IRC | 23:30 | |
| *** xyang1 has quit IRC | 23:32 | |
| *** patrickeast_ has joined #openstack-nova | 23:34 | |
| *** patrickeast has quit IRC | 23:34 | |
| *** patrickeast_ is now known as patrickeast | 23:34 | |
| *** armax has quit IRC | 23:36 | |
| *** mestery has quit IRC | 23:36 | |
| *** sdake_ has quit IRC | 23:40 | |
| *** Sukhdev has quit IRC | 23:40 | |
| *** otter768 has quit IRC | 23:43 | |
| *** cfriesen_ has joined #openstack-nova | 23:47 | |
| *** dboik has joined #openstack-nova | 23:47 | |
| *** otter768 has joined #openstack-nova | 23:48 | |
| *** exploreshaifali has quit IRC | 23:49 | |
| *** mtanino has joined #openstack-nova | 23:50 | |
| *** kylek3h has quit IRC | 23:51 | |
| *** asettle has joined #openstack-nova | 23:51 | |
| *** ssurana has quit IRC | 23:53 | |
| *** hemna is now known as hemnafk | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!