*** alexpilotti has quit IRC | 00:00 | |
*** Marga_ has joined #openstack-nova | 00:00 | |
*** ferest has quit IRC | 00:00 | |
*** ferest has joined #openstack-nova | 00:01 | |
*** ferest has quit IRC | 00:02 | |
*** promulo_ has quit IRC | 00:03 | |
*** ferest has joined #openstack-nova | 00:04 | |
*** Marga_ has quit IRC | 00:05 | |
*** ferest has quit IRC | 00:05 | |
*** ferest has joined #openstack-nova | 00:06 | |
*** ferest has quit IRC | 00:10 | |
*** ferest has joined #openstack-nova | 00:10 | |
*** ferest has quit IRC | 00:16 | |
*** ferest has joined #openstack-nova | 00:17 | |
*** mwagner_lap has joined #openstack-nova | 00:19 | |
*** ferest has quit IRC | 00:19 | |
*** ferest has joined #openstack-nova | 00:19 | |
*** suresh12 has joined #openstack-nova | 00:20 | |
*** ferest has quit IRC | 00:20 | |
*** ferest has joined #openstack-nova | 00:21 | |
*** ferest has quit IRC | 00:21 | |
*** ferest has joined #openstack-nova | 00:22 | |
*** suresh12 has quit IRC | 00:27 | |
*** ferest has quit IRC | 00:29 | |
*** ferest has joined #openstack-nova | 00:30 | |
*** r-daneel has joined #openstack-nova | 00:37 | |
*** ferest has quit IRC | 00:37 | |
*** ferest has joined #openstack-nova | 00:38 | |
*** r-daneel has quit IRC | 00:44 | |
*** salv-orlando has joined #openstack-nova | 00:45 | |
*** ferest has quit IRC | 00:45 | |
*** ferest has joined #openstack-nova | 00:46 | |
*** salv-orl_ has joined #openstack-nova | 00:49 | |
*** ferest has quit IRC | 00:50 | |
*** salv-orlando has quit IRC | 00:50 | |
*** ferest has joined #openstack-nova | 00:50 | |
*** ferest has quit IRC | 00:50 | |
*** ferest has joined #openstack-nova | 00:52 | |
*** salv-orl_ has quit IRC | 00:54 | |
*** ferest has quit IRC | 00:56 | |
*** ferest has joined #openstack-nova | 00:56 | |
*** dims has joined #openstack-nova | 00:58 | |
*** ferest has quit IRC | 00:58 | |
*** ferest has joined #openstack-nova | 00:59 | |
*** ferest has quit IRC | 01:00 | |
*** Marga_ has joined #openstack-nova | 01:01 | |
*** Marga_ has quit IRC | 01:05 | |
*** MentalRay has quit IRC | 01:10 | |
*** gabriel-bezerra has quit IRC | 01:11 | |
*** annashen has joined #openstack-nova | 01:11 | |
*** samueldmq has quit IRC | 01:11 | |
*** ericksonsantos has quit IRC | 01:11 | |
*** pixelbeat_ has quit IRC | 01:13 | |
*** Kennan has quit IRC | 01:13 | |
*** htruta has quit IRC | 01:14 | |
*** htruta has joined #openstack-nova | 01:15 | |
*** samueldmq has joined #openstack-nova | 01:15 | |
*** ericksonsantos has joined #openstack-nova | 01:15 | |
*** Kennan has joined #openstack-nova | 01:16 | |
*** gabriel-bezerra has joined #openstack-nova | 01:17 | |
*** suresh12 has joined #openstack-nova | 01:23 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 01:26 |
---|---|---|
*** jasondotstar has joined #openstack-nova | 01:26 | |
*** davideagnello has joined #openstack-nova | 01:27 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Split out resize_image logic from create_image https://review.openstack.org/187857 | 01:28 |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 01:28 |
*** suresh12 has quit IRC | 01:28 | |
*** krotsche_vaca is now known as krotscheck | 01:28 | |
*** shyu has joined #openstack-nova | 01:35 | |
openstackgerrit | John Hua proposed openstack/nova: Support extra (t)gz images that only has a single VHD file in it https://review.openstack.org/205831 | 01:38 |
*** hanzhang has joined #openstack-nova | 01:47 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Get py34 subunit.run test discovery to work https://review.openstack.org/204723 | 01:51 |
*** suresh12 has joined #openstack-nova | 01:51 | |
*** eliqiao has left #openstack-nova | 01:51 | |
*** salv-orlando has joined #openstack-nova | 01:53 | |
openstackgerrit | sajeesh cs proposed openstack/nova: Nested Quota Driver: Adding v2 and v3(v2.1) support https://review.openstack.org/149828 | 01:54 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Get py34 subunit.run test discovery to work https://review.openstack.org/204723 | 01:56 |
*** ijw has joined #openstack-nova | 01:57 | |
*** salv-orl_ has joined #openstack-nova | 02:01 | |
*** Marga_ has joined #openstack-nova | 02:02 | |
*** salv-orlando has quit IRC | 02:03 | |
*** salv-orl_ has quit IRC | 02:06 | |
*** Marga_ has quit IRC | 02:06 | |
*** zhenguo has joined #openstack-nova | 02:07 | |
openstackgerrit | sajeesh cs proposed openstack/nova: Nested Quota Driver: Adding v2 and v3(v2.1) support https://review.openstack.org/149828 | 02:09 |
*** RuiChen has joined #openstack-nova | 02:13 | |
*** MentalRay has joined #openstack-nova | 02:16 | |
*** ayoung has quit IRC | 02:17 | |
*** suresh12 has quit IRC | 02:18 | |
*** liusheng has joined #openstack-nova | 02:25 | |
*** suresh12 has joined #openstack-nova | 02:25 | |
*** suresh12 has quit IRC | 02:30 | |
*** shuquan has joined #openstack-nova | 02:33 | |
*** annashen has quit IRC | 02:35 | |
*** annashen has joined #openstack-nova | 02:37 | |
*** coreywright has quit IRC | 02:38 | |
*** annashen_ has joined #openstack-nova | 02:40 | |
*** annashen_ has quit IRC | 02:41 | |
*** annashen has quit IRC | 02:42 | |
*** annashen has joined #openstack-nova | 02:42 | |
openstackgerrit | sajeesh cs proposed openstack/nova: Nested Quota Driver: Adding v2 and v3(v2.1) support https://review.openstack.org/149828 | 02:45 |
*** annashen has quit IRC | 02:45 | |
*** annashen has joined #openstack-nova | 02:46 | |
*** yuntong has quit IRC | 02:48 | |
*** annashen has quit IRC | 02:50 | |
*** annashen has joined #openstack-nova | 02:50 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Get py34 subunit.run test discovery to work https://review.openstack.org/204723 | 02:51 |
*** hakimo has joined #openstack-nova | 02:52 | |
*** hakimo_ has quit IRC | 02:54 | |
*** markvoelker has joined #openstack-nova | 02:58 | |
*** dims has quit IRC | 02:59 | |
*** armax has joined #openstack-nova | 03:02 | |
*** markvoelker has quit IRC | 03:02 | |
*** kylek3h has quit IRC | 03:02 | |
*** Marga_ has joined #openstack-nova | 03:03 | |
*** kcaj has quit IRC | 03:07 | |
*** Marga_ has quit IRC | 03:07 | |
*** y_sawai has joined #openstack-nova | 03:08 | |
*** salv-orlando has joined #openstack-nova | 03:08 | |
*** annashen has quit IRC | 03:09 | |
*** annashen has joined #openstack-nova | 03:12 | |
*** salv-orlando has quit IRC | 03:12 | |
*** salv-orlando has joined #openstack-nova | 03:13 | |
*** kcaj has joined #openstack-nova | 03:14 | |
openstackgerrit | Fei Long Wang proposed openstack/nova: Return project id for floating ip https://review.openstack.org/205880 | 03:14 |
*** Jeffrey4l has joined #openstack-nova | 03:16 | |
*** salv-orlando has quit IRC | 03:17 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add instance action events for live migration https://review.openstack.org/195372 | 03:18 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Resize: Return to initial state instead of ACTIVE when resize fails https://review.openstack.org/202379 | 03:18 |
*** changbl has quit IRC | 03:19 | |
*** jasondotstar has quit IRC | 03:20 | |
*** Marga_ has joined #openstack-nova | 03:20 | |
*** Marga_ has quit IRC | 03:25 | |
*** chaochin has quit IRC | 03:26 | |
*** suresh12 has joined #openstack-nova | 03:26 | |
*** y_sawai has quit IRC | 03:26 | |
*** coolsvap|away is now known as coolsvap | 03:30 | |
*** otter768 has quit IRC | 03:31 | |
*** armax has quit IRC | 03:31 | |
*** suresh12 has quit IRC | 03:32 | |
*** changbl has joined #openstack-nova | 03:32 | |
*** Sukhdev has joined #openstack-nova | 03:44 | |
*** shuquan_ has joined #openstack-nova | 03:48 | |
*** annashen has quit IRC | 03:48 | |
*** annashen has joined #openstack-nova | 03:48 | |
*** shuquan has quit IRC | 03:48 | |
*** shuquan_ is now known as shuquan | 03:48 | |
*** ssurana has joined #openstack-nova | 03:50 | |
*** zul has quit IRC | 03:52 | |
*** zul has joined #openstack-nova | 03:53 | |
*** dims has joined #openstack-nova | 04:00 | |
*** liusheng has quit IRC | 04:00 | |
*** shuquan has quit IRC | 04:00 | |
*** haomaiwang has quit IRC | 04:02 | |
*** liusheng has joined #openstack-nova | 04:03 | |
*** achanda has quit IRC | 04:03 | |
*** shuquan has joined #openstack-nova | 04:03 | |
*** haomaiwang has joined #openstack-nova | 04:04 | |
*** dims has quit IRC | 04:06 | |
*** hanzhang has quit IRC | 04:07 | |
*** hanzhang has joined #openstack-nova | 04:07 | |
*** shuquan has quit IRC | 04:08 | |
*** shuquan has joined #openstack-nova | 04:08 | |
*** jecarey has joined #openstack-nova | 04:12 | |
*** deepthi has joined #openstack-nova | 04:13 | |
*** suresh12 has joined #openstack-nova | 04:15 | |
*** annashen has quit IRC | 04:16 | |
*** annashen has joined #openstack-nova | 04:16 | |
*** fawadkhaliq has joined #openstack-nova | 04:17 | |
*** annashen has quit IRC | 04:19 | |
*** Kennan2 has joined #openstack-nova | 04:20 | |
*** Kennan has quit IRC | 04:20 | |
*** Marga_ has joined #openstack-nova | 04:21 | |
*** suresh12 has quit IRC | 04:24 | |
*** achanda has joined #openstack-nova | 04:24 | |
*** Marga_ has quit IRC | 04:25 | |
*** Sukhdev has quit IRC | 04:25 | |
*** achanda has quit IRC | 04:26 | |
*** dave-mcc_ has quit IRC | 04:27 | |
*** annashen has joined #openstack-nova | 04:28 | |
*** annashen has quit IRC | 04:35 | |
*** rotbeard has joined #openstack-nova | 04:39 | |
*** marzif has joined #openstack-nova | 04:41 | |
*** garyk has quit IRC | 04:41 | |
*** ssurana has quit IRC | 04:43 | |
*** jecarey has quit IRC | 04:47 | |
*** tpeoples has quit IRC | 04:55 | |
*** markvoelker has joined #openstack-nova | 04:58 | |
*** shuquan has quit IRC | 05:00 | |
*** salv-orlando has joined #openstack-nova | 05:01 | |
*** browne has joined #openstack-nova | 05:02 | |
*** markvoelker has quit IRC | 05:03 | |
*** salv-orl_ has joined #openstack-nova | 05:03 | |
*** salv-orlando has quit IRC | 05:05 | |
*** achanda has joined #openstack-nova | 05:07 | |
*** Sukhdev has joined #openstack-nova | 05:08 | |
openstackgerrit | Shiina, Hironori proposed openstack/nova: libvirt:Add a driver API to inject an NMI https://review.openstack.org/202380 | 05:08 |
*** marzif has quit IRC | 05:11 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Register JSON-Home data for resources with parent https://review.openstack.org/205905 | 05:11 |
*** salv-orl_ has quit IRC | 05:11 | |
*** salv-orlando has joined #openstack-nova | 05:12 | |
*** kumar has joined #openstack-nova | 05:12 | |
*** fawadkhaliq has quit IRC | 05:13 | |
openstackgerrit | Meghal Gosalia proposed openstack/nova-specs: Provide support for creating dynamic quota resources https://review.openstack.org/205906 | 05:13 |
*** fawadkhaliq has joined #openstack-nova | 05:13 | |
*** pradipta has joined #openstack-nova | 05:13 | |
openstackgerrit | Meghal Gosalia proposed openstack/nova-specs: Provide support for creating dynamic quota resources https://review.openstack.org/205906 | 05:15 |
*** salv-orlando has quit IRC | 05:16 | |
*** chaochin has joined #openstack-nova | 05:16 | |
*** cfriesen has joined #openstack-nova | 05:18 | |
*** salv-orlando has joined #openstack-nova | 05:19 | |
*** Marga_ has joined #openstack-nova | 05:22 | |
*** Marga_ has quit IRC | 05:26 | |
*** davideagnello has quit IRC | 05:27 | |
*** kmARC has joined #openstack-nova | 05:29 | |
*** MentalRay has quit IRC | 05:31 | |
*** otter768 has joined #openstack-nova | 05:32 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Show 'locked_by' information in server Show & List Detail https://review.openstack.org/202391 | 05:33 |
*** lpetrut has joined #openstack-nova | 05:34 | |
*** annashen has joined #openstack-nova | 05:35 | |
*** Marga_ has joined #openstack-nova | 05:36 | |
*** otter768 has quit IRC | 05:37 | |
*** Marga_ has quit IRC | 05:37 | |
*** Marga_ has joined #openstack-nova | 05:38 | |
*** annashen has quit IRC | 05:40 | |
*** shuquan has joined #openstack-nova | 05:40 | |
*** browne has quit IRC | 05:41 | |
*** browne has joined #openstack-nova | 05:41 | |
*** frickler_ is now known as frickler | 05:42 | |
*** SourabhP has joined #openstack-nova | 05:43 | |
*** ankit_ag has joined #openstack-nova | 05:44 | |
*** shuquan has quit IRC | 05:45 | |
*** shuquan has joined #openstack-nova | 05:45 | |
*** SourabhP_ has joined #openstack-nova | 05:46 | |
openstackgerrit | Meghal Gosalia proposed openstack/nova-specs: Provide support for creating dynamic quota resources https://review.openstack.org/205906 | 05:46 |
*** deepakcs has joined #openstack-nova | 05:47 | |
*** SourabhP has quit IRC | 05:47 | |
*** SourabhP_ is now known as SourabhP | 05:47 | |
*** nkrinner has joined #openstack-nova | 05:50 | |
*** garyk has joined #openstack-nova | 05:52 | |
openstackgerrit | Meghal Gosalia proposed openstack/nova-specs: Provide support for creating dynamic quota resources https://review.openstack.org/205906 | 05:55 |
*** unicell1 has joined #openstack-nova | 05:57 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Support for registering JSON-Home data- show/delete/update https://review.openstack.org/205001 | 05:57 |
*** unicell has quit IRC | 05:57 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Add resource_template_with_parent JSON-Home methods https://review.openstack.org/205392 | 05:57 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Register JSON-Home data for resources with parent https://review.openstack.org/205905 | 05:58 |
*** Murali has joined #openstack-nova | 05:59 | |
*** jwcroppe has joined #openstack-nova | 06:00 | |
*** fifieldt has joined #openstack-nova | 06:00 | |
*** dims has joined #openstack-nova | 06:03 | |
*** cfriesen has quit IRC | 06:04 | |
*** atuvenie has joined #openstack-nova | 06:07 | |
*** fawadk has joined #openstack-nova | 06:07 | |
*** fawadkhaliq has quit IRC | 06:08 | |
*** dims has quit IRC | 06:09 | |
*** markus_z has joined #openstack-nova | 06:10 | |
deepakcs | anyone seeing a neutron 'network not found' error in the jenkins neutron job, lately ? | 06:12 |
deepakcs | I am seeing this for my patch, which I don't see any relation to: http://logs.openstack.org/38/194538/6/check/gate-tempest-dsvm-neutron-full/a00b9a2/console.html.gz#_2015-07-24_15_56_19_862 | 06:12 |
*** lsmola has joined #openstack-nova | 06:12 | |
openstackgerrit | Vipin Balachandran proposed openstack/nova: VMware: Delete vmdk UUID during volume detach https://review.openstack.org/204622 | 06:14 |
*** annashen has joined #openstack-nova | 06:14 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/197213 | 06:16 |
*** markus_z has quit IRC | 06:17 | |
*** moshele has joined #openstack-nova | 06:18 | |
*** vilobhmm has joined #openstack-nova | 06:19 | |
*** rdopiera has joined #openstack-nova | 06:20 | |
*** abhijeetm has joined #openstack-nova | 06:20 | |
*** annashen has quit IRC | 06:23 | |
*** shuquan has quit IRC | 06:25 | |
*** alexpilotti has joined #openstack-nova | 06:26 | |
*** lan has quit IRC | 06:27 | |
*** annashen has joined #openstack-nova | 06:29 | |
*** fawadk has quit IRC | 06:30 | |
*** fawadkhaliq has joined #openstack-nova | 06:30 | |
*** vilobhmm1 has joined #openstack-nova | 06:30 | |
*** scheuran has joined #openstack-nova | 06:30 | |
*** mestery has quit IRC | 06:31 | |
*** annashen has quit IRC | 06:31 | |
*** vilobhmm has quit IRC | 06:33 | |
*** annashen has joined #openstack-nova | 06:33 | |
openstackgerrit | liaonanhai proposed openstack/nova: Add user_id info: add 'user_id' to REST os-simple-tenant-usage output https://review.openstack.org/151522 | 06:34 |
*** annashen has quit IRC | 06:39 | |
*** vilobhmm1 has quit IRC | 06:43 | |
*** SourabhP has quit IRC | 06:45 | |
*** Marga_ has quit IRC | 06:45 | |
*** doude has joined #openstack-nova | 06:46 | |
*** doude has quit IRC | 06:46 | |
*** doude has joined #openstack-nova | 06:46 | |
*** Marga_ has joined #openstack-nova | 06:46 | |
*** annashen has joined #openstack-nova | 06:46 | |
*** shuquan has joined #openstack-nova | 06:47 | |
*** gmann has quit IRC | 06:47 | |
*** kmARC has quit IRC | 06:47 | |
*** annashen has quit IRC | 06:49 | |
openstackgerrit | Eli Qiao proposed openstack/nova: Object: https://review.openstack.org/205917 | 06:50 |
*** annashen has joined #openstack-nova | 06:51 | |
*** rushiagr_away is now known as rushiagr | 06:53 | |
*** annashen has quit IRC | 06:53 | |
*** gszasz has joined #openstack-nova | 06:57 | |
*** vilobhmm has joined #openstack-nova | 06:59 | |
*** markvoelker has joined #openstack-nova | 06:59 | |
*** sahid has joined #openstack-nova | 07:00 | |
*** matrohon has joined #openstack-nova | 07:02 | |
openstackgerrit | Eli Qiao proposed openstack/nova: Object: Fix KeyError when loading instance from db https://review.openstack.org/205917 | 07:04 |
*** markvoelker has quit IRC | 07:04 | |
*** slong has quit IRC | 07:05 | |
*** achanda has quit IRC | 07:07 | |
*** Sukhdev has quit IRC | 07:08 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Resize: Return to initial state instead of ACTIVE when resize fails https://review.openstack.org/202379 | 07:08 |
*** lpetrut has quit IRC | 07:09 | |
*** markus_z has joined #openstack-nova | 07:11 | |
openstackgerrit | garyk proposed openstack/nova: Remove unused import of the my_ip option from the manager https://review.openstack.org/205923 | 07:12 |
*** achanda has joined #openstack-nova | 07:12 | |
*** fawadk has joined #openstack-nova | 07:13 | |
*** fawadkhaliq has quit IRC | 07:13 | |
*** rajesht has joined #openstack-nova | 07:14 | |
*** Marga_ has quit IRC | 07:14 | |
*** matrohon has quit IRC | 07:14 | |
*** lpetrut has joined #openstack-nova | 07:16 | |
*** chhavi has joined #openstack-nova | 07:19 | |
*** karimb has joined #openstack-nova | 07:19 | |
*** kiwik-chenrui has joined #openstack-nova | 07:21 | |
*** lsmola has quit IRC | 07:21 | |
*** lsmola has joined #openstack-nova | 07:21 | |
*** kiwik-chenrui has quit IRC | 07:22 | |
*** matrohon has joined #openstack-nova | 07:22 | |
*** kiwik-chenrui has joined #openstack-nova | 07:22 | |
*** kiwik-chenrui has quit IRC | 07:23 | |
*** RuiChen has quit IRC | 07:24 | |
*** mpaolino has joined #openstack-nova | 07:26 | |
*** davideagnello has joined #openstack-nova | 07:27 | |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Added removing of tags from instance after its deletion https://review.openstack.org/204053 | 07:28 |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Added removing of tags from instance after its deletion https://review.openstack.org/204053 | 07:30 |
*** jwcroppe has quit IRC | 07:30 | |
*** mpavone has joined #openstack-nova | 07:30 | |
*** mpavone has quit IRC | 07:31 | |
*** jwcroppe has joined #openstack-nova | 07:31 | |
*** davideagnello has quit IRC | 07:32 | |
*** otter768 has joined #openstack-nova | 07:33 | |
openstackgerrit | garyk proposed openstack/nova: libvirt: test capability for supports_migrate_to_same_host https://review.openstack.org/205928 | 07:33 |
*** RuiChen has joined #openstack-nova | 07:37 | |
*** otter768 has quit IRC | 07:37 | |
*** vilobhmm has quit IRC | 07:38 | |
*** salv-orlando has quit IRC | 07:39 | |
openstackgerrit | Sachi King proposed openstack/nova: Add constraint target to tox.ini https://review.openstack.org/205931 | 07:40 |
*** fifieldt has quit IRC | 07:42 | |
*** jlanoux has joined #openstack-nova | 07:43 | |
*** fawadkhaliq has joined #openstack-nova | 07:44 | |
*** fawadk has quit IRC | 07:44 | |
*** jistr has joined #openstack-nova | 07:44 | |
*** salv-orlando has joined #openstack-nova | 07:45 | |
lifeless | Nakato: \o/ | 07:45 |
openstackgerrit | Zhenguo Niu proposed openstack/nova: [Ironic]Not count available resources of deployed ironic node https://review.openstack.org/204088 | 07:46 |
*** rotbeard has quit IRC | 07:47 | |
*** Marga_ has joined #openstack-nova | 07:52 | |
*** lpetrut has quit IRC | 07:53 | |
*** xgsa has quit IRC | 07:58 | |
*** jwcroppe has quit IRC | 08:01 | |
*** scheuran has quit IRC | 08:01 | |
*** jwcroppe has joined #openstack-nova | 08:02 | |
*** ttx has quit IRC | 08:03 | |
*** ttx has joined #openstack-nova | 08:03 | |
*** jwcroppe has quit IRC | 08:06 | |
*** lucasagomes has joined #openstack-nova | 08:07 | |
*** aarefiev has joined #openstack-nova | 08:07 | |
*** ndipanov has joined #openstack-nova | 08:08 | |
*** mnestratov has joined #openstack-nova | 08:08 | |
*** achanda has quit IRC | 08:08 | |
*** achanda has joined #openstack-nova | 08:09 | |
*** gmann has joined #openstack-nova | 08:10 | |
*** jwcroppe has joined #openstack-nova | 08:15 | |
*** haypo has joined #openstack-nova | 08:16 | |
*** romainh has joined #openstack-nova | 08:18 | |
*** Marga_ has quit IRC | 08:18 | |
*** fawadkhaliq has quit IRC | 08:20 | |
*** Marga_ has joined #openstack-nova | 08:20 | |
*** coolsvap is now known as coolsvap|away | 08:21 | |
*** rdopiera has quit IRC | 08:22 | |
*** rdopiera has joined #openstack-nova | 08:22 | |
*** rdopiera has quit IRC | 08:23 | |
openstackgerrit | Pavel Kholkin proposed openstack/nova: Fixed indentation https://review.openstack.org/205529 | 08:23 |
*** rdopiera has joined #openstack-nova | 08:23 | |
*** MarcF has joined #openstack-nova | 08:23 | |
*** kmARC has joined #openstack-nova | 08:24 | |
*** gszasz has quit IRC | 08:26 | |
garyk | johnthetubaguy: https://review.openstack.org/#/c/170641/ has a CI vote. It green green green … | 08:27 |
*** danpb has joined #openstack-nova | 08:28 | |
*** coolsvap|away is now known as coolsvap | 08:29 | |
*** Jhelf has joined #openstack-nova | 08:29 | |
*** achanda has quit IRC | 08:31 | |
*** coolsvap is now known as coolsvap|away | 08:31 | |
*** aix has joined #openstack-nova | 08:31 | |
*** promulo_ has joined #openstack-nova | 08:33 | |
BobBall | I have disabled the libvirt+xen CI - seems it's failing to merge all changes, not sure why | 08:33 |
*** achanda has joined #openstack-nova | 08:33 | |
*** achanda has quit IRC | 08:34 | |
*** romainh has quit IRC | 08:35 | |
*** romainh has joined #openstack-nova | 08:35 | |
*** browne has quit IRC | 08:36 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Register JSON-Home data for resources with inherits https://review.openstack.org/205944 | 08:37 |
*** annashen has joined #openstack-nova | 08:38 | |
*** romainh has quit IRC | 08:39 | |
*** romainh1 has joined #openstack-nova | 08:42 | |
*** lpetrut has joined #openstack-nova | 08:43 | |
*** romainh1 has quit IRC | 08:43 | |
*** coolsvap|away is now known as coolsvap | 08:43 | |
openstackgerrit | Shiina, Hironori proposed openstack/nova: libvirt:Add a driver API to inject an NMI https://review.openstack.org/202380 | 08:44 |
openstackgerrit | Shiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI https://review.openstack.org/202605 | 08:45 |
*** romainh has joined #openstack-nova | 08:46 | |
openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 08:46 |
*** Nikolay_St has joined #openstack-nova | 08:46 | |
bauwser | morning nova | 08:46 |
*** bauwser is now known as bauzas | 08:46 | |
openstackgerrit | Shiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI https://review.openstack.org/202605 | 08:47 |
*** romainh1 has joined #openstack-nova | 08:49 | |
*** romainh2 has joined #openstack-nova | 08:49 | |
*** romainh2 has quit IRC | 08:50 | |
*** romainh has quit IRC | 08:50 | |
*** Marga_ has quit IRC | 08:52 | |
*** romainh2 has joined #openstack-nova | 08:52 | |
*** romainh2 has quit IRC | 08:53 | |
*** romainh1 has quit IRC | 08:53 | |
*** ZZelle has joined #openstack-nova | 08:53 | |
*** Marga_ has joined #openstack-nova | 08:54 | |
*** romainh1 has joined #openstack-nova | 08:54 | |
*** subscope has joined #openstack-nova | 08:55 | |
*** igordcard_ has quit IRC | 08:55 | |
*** igordcard_ has joined #openstack-nova | 08:56 | |
markus_z | Could a core look at my patch please? https://review.openstack.org/#/c/188058/ Non-Priority FF is near. | 08:56 |
*** Marga_ has quit IRC | 08:56 | |
*** Marga_ has joined #openstack-nova | 08:56 | |
*** romainh1 has quit IRC | 08:58 | |
*** markvoelker has joined #openstack-nova | 09:00 | |
*** romainh has joined #openstack-nova | 09:00 | |
*** romainh has quit IRC | 09:01 | |
*** romainh has joined #openstack-nova | 09:02 | |
* johnthetubaguy is slowly waking up | 09:02 | |
*** romainh1 has joined #openstack-nova | 09:03 | |
*** igordcard_ has quit IRC | 09:03 | |
*** romainh1 has quit IRC | 09:03 | |
*** igordcard_ has joined #openstack-nova | 09:03 | |
*** karimb has quit IRC | 09:04 | |
*** markvoelker has quit IRC | 09:05 | |
*** igordcard_ has quit IRC | 09:06 | |
*** echirivella has quit IRC | 09:06 | |
*** promulo_ has quit IRC | 09:06 | |
*** dims has joined #openstack-nova | 09:06 | |
openstackgerrit | Zhenguo Niu proposed openstack/nova: Remove unused decorator on attach/detach volume https://review.openstack.org/165839 | 09:06 |
bauzas | markus_z: the "read the full spec" link gives me 404 on https://blueprints.launchpad.net/nova/+spec/libvirt-log-file-and-serial-console because it points to a jenkins build probably recycled | 09:06 |
bauzas | markus_z: could you please fix it ? | 09:06 |
*** romainh has quit IRC | 09:07 | |
bauzas | markus_z: ooooh, no need for a spec | 09:07 |
*** igordcard_ has joined #openstack-nova | 09:07 | |
markus_z | bauzas: Sorry, I forgot to update this after the clarification with john that this is a trivial blueprint. | 09:07 |
*** igordcard_ has joined #openstack-nova | 09:07 | |
bauzas | markus_z: yeah, it makes sense it doesn't need a spec, but I stupidely followed the link | 09:08 |
johnthetubaguy | I just deleted the URL, I must have forgotten to do that when I approved it | 09:08 |
*** romainh1 has joined #openstack-nova | 09:08 | |
bauzas | markus_z: what's a bit worrying me is that you propose to have a consolelog *anyway* | 09:08 |
markus_z | bauzas: What concerns you? | 09:09 |
*** romainh1 has quit IRC | 09:10 | |
*** romainh2 has joined #openstack-nova | 09:10 | |
*** romainh2 has quit IRC | 09:10 | |
bauzas | markus_z: sorry, I'm a bit unclear | 09:10 |
bauzas | markus_z: by removing the else statement, you proposed to create the file pointer by any time | 09:11 |
*** dims has quit IRC | 09:11 | |
bauzas | instead of using a remote tcp | 09:11 |
markus_z | bauzas: Right. That's how it was before the serial console feature ("tcp device") was introduced. | 09:12 |
*** romainh1 has joined #openstack-nova | 09:12 | |
markus_z | bauzas: If I only have the tcp device, I won't have the boot messages persisted. | 09:12 |
bauzas | markus_z: what if I don't want to have a file descriptor for each instance I'm gonna create ? | 09:12 |
bauzas | well, nvm | 09:13 |
bauzas | it still takes a fd for a tcp device | 09:13 |
markus_z | bauzas: I would see that as a new requirement. To enable the tcp device in order to prevent the file descriptor creation seems odd to me. | 09:13 |
bauzas | yeah | 09:14 |
*** romainh1 has quit IRC | 09:14 | |
*** romainh has joined #openstack-nova | 09:14 | |
bauzas | markus_z: sorry for that, just thinking of all the operator possibilities :) | 09:14 |
bauzas | markus_z: like you said in your commit msg, it will now create a new device by default for any new instance | 09:15 |
bauzas | markus_z: I just wonder whether it's safe or not | 09:15 |
markus_z | bauzas: You mean if it's safe to have both devices at the same time? | 09:17 |
openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: scheduler: weight nodes based on their cached images https://review.openstack.org/173304 | 09:17 |
openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: Add new image cache monitor in compute nodes https://review.openstack.org/173303 | 09:17 |
openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: vmware: add image_cache_manager property https://review.openstack.org/187602 | 09:17 |
bauzas | markus_z: I wonder if it's safe to have the instance log file anyway :-) | 09:17 |
openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: libvirt: make live_migration_uri flag dependent on virt_type https://review.openstack.org/175780 | 09:18 |
*** annashen has quit IRC | 09:18 | |
*** echirivella has joined #openstack-nova | 09:18 | |
markus_z | bauzas: A instance will have this logfile every time, unless you enable the "serial console" feature (=tcp device). | 09:20 |
bauzas | markus_z: I understood that, I'm just considering whether it's good or not | 09:20 |
bauzas | markus_z: or in other words, if having the logfile should be managed by yet another confflag like 'instance_log' | 09:22 |
markus_z | bauzas: The logfile will have the "historical" readonly output from the guest. The tcp device will enable an interaction to the guest in case of an emergency. | 09:22 |
bauzas | markus_z: yeah, I got that | 09:22 |
markus_z | bauzas: OK, I'm just double-checking. It's hard to know each thing in detail. | 09:23 |
markus_z | bauzas: When would you see a negative impact on having the log file? | 09:23 |
openstackgerrit | Shiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI https://review.openstack.org/202617 | 09:23 |
bauzas | markus_z: see danpb's comment | 09:23 |
*** rotbeard has joined #openstack-nova | 09:24 | |
* markus_z reading | 09:24 | |
bauzas | markus_z: I was not aware of such limitation, but that was something that was worrying me | 09:24 |
markus_z | bauzas: me neither. Good that danpb catched it. I'm looking for a solution. | 09:26 |
*** lpetrut has quit IRC | 09:26 | |
*** melwitt has quit IRC | 09:26 | |
danpb | as i mention in the review, i think this idea is flawed regardless of the max port count | 09:26 |
*** lpetrut has joined #openstack-nova | 09:27 | |
*** pixelbeat_ has joined #openstack-nova | 09:27 | |
bauzas | danpb: honestly, you gave an excellent reason, ty | 09:27 |
*** atuvenie has quit IRC | 09:27 | |
*** afazekas has joined #openstack-nova | 09:28 | |
johnthetubaguy | garyk: I was unsure about the commit message, did the comments make any sense? | 09:28 |
*** davideagnello has joined #openstack-nova | 09:28 | |
markus_z | danpb: Do you mean the whole idea of the "serial console" is flawed or the blueprint I proposed? | 09:29 |
*** mpaolino has quit IRC | 09:30 | |
danpb | just the change | 09:31 |
danpb | we need to be able to record the data being sent over an existing serial port, not add more serial ports | 09:31 |
garyk | johnthetubaguy: no really. i was under the impression that the commit message was self explanatory and it is a real trivial fix. what do you suggest that i change it to? | 09:31 |
*** jasondotstar has joined #openstack-nova | 09:32 | |
*** davideagnello has quit IRC | 09:33 | |
*** otter768 has joined #openstack-nova | 09:34 | |
johnthetubaguy | garyk: I added a question in gerrit, it seems like you can't ever get None back from that method, so the old code is not buggy, but the change is still fine, just worried about the misleading git commit message, I might just be miss reading that though, I was really just following up on jogo's previous comment | 09:34 |
johnthetubaguy | danpb: markus_z: from what I remember, I think thats what was assumed in the blueprint review, it was logging on an existing serial console, hence the we should review the details of the implementation argument to the spec-less approval, but yeah, thats not what is happening here I guess | 09:36 |
garyk | johnthetubaguy: the will return None is invalid. I''l update the commit message. | 09:37 |
*** shyu has quit IRC | 09:37 | |
markus_z | danpb: I understand the possible regression now. Do you have a pointer for the upper limit of 4 serial ports on x86? | 09:37 |
garyk | kind of took 2 months to look at this so another 2 to look at it again is par fo rhte course | 09:37 |
johnthetubaguy | garyk: I know other bits of the code are making the reverse change in places, but I don't mind either way around | 09:37 |
johnthetubaguy | garyk: I just saw it on the etherpad, so I took a look | 09:38 |
*** otter768 has quit IRC | 09:38 | |
garyk | the team has kind of given up on the etherpad as there have been no reviews. should we continue to update it. it just feels futile at the moment | 09:38 |
danpb | markus_z: its just a long standing limit of the x86 ISA chipset | 09:38 |
danpb | been that way for at least 15 years | 09:38 |
johnthetubaguy | garyk: yes please, keep updating it, I am still pushing that | 09:38 |
garyk | ok, we'll continue to update. | 09:39 |
markus_z | johnthetubaguy: I wasn't aware of the upper limit danpb mentioned, that's why I didn't consider it in the blueprint. | 09:39 |
garyk | feedback on our side is that we updated it off the bat and no reviews were being done so we stopped. | 09:39 |
johnthetubaguy | garyk: I will send an update on the expectations of what goes in that list | 09:39 |
garyk | ok, thanks | 09:39 |
garyk | i review code and hope to expect that people review mine. what you give is what you get :) | 09:39 |
johnthetubaguy | garyk: its going to take time to become normal, there were a good few folks at the midcycle who said they should now have time to look at more of that list | 09:40 |
garyk | johnthetubaguy: i hope so. | 09:40 |
garyk | but looking at the review numbers it looks unlikely | 09:41 |
lxsli | johnthetubaguy: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule#My_code_reviews_seems_to_have_got_stuck.2C_what_can_I_do.3F | 09:41 |
lxsli | johnthetubaguy: that still advises pinging on IRC+email and adding to Nova meeting | 09:42 |
openstackgerrit | garyk proposed openstack/nova: VMware: remove redundant check for block devices https://review.openstack.org/170641 | 09:42 |
lxsli | Is that what you want currently? | 09:42 |
johnthetubaguy | lxsli: well, it advises pinging on IRC if people don't get back to you after giving you a -1 | 09:42 |
*** bradjones has joined #openstack-nova | 09:42 | |
*** bradjones has quit IRC | 09:42 | |
*** bradjones has joined #openstack-nova | 09:42 | |
johnthetubaguy | lxsli: at least thats my intent... it kinda doesn't read like that, oops | 09:43 |
*** mpaolino has joined #openstack-nova | 09:45 | |
*** mpaolino has quit IRC | 09:45 | |
*** kumar has quit IRC | 09:45 | |
lxsli | johnthetubaguy: I can send you a pastebin of a rewrite if you like? | 09:46 |
garyk | johnthetubaguy: i have updated the commit message. hope that suffices. | 09:46 |
johnthetubaguy | lxsli: that would be awesome, I did a quick emergency edit just now, in case that helps | 09:46 |
garyk | fwiw the link http://russellbryant.net/openstack-stats/nova-openreviews.html does not work (not sure if that is from the office) | 09:49 |
openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: libvirt: Nova part of volume snapshot improvements https://review.openstack.org/172813 | 09:50 |
openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: objects: Add support for OnlineSnapshotInfo object https://review.openstack.org/194538 | 09:50 |
*** stefanb has quit IRC | 09:51 | |
*** jwcroppe has quit IRC | 09:54 | |
deepakcs | johnthetubaguy: Pls help bless the patches @ https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/volume-snapshot-improvements,n,z . Thanks in advance | 09:54 |
openstackgerrit | Jordan Pittier proposed openstack/nova: libvirt:when deleting snapshot use qemu-img to blockRebase if domain is stopped https://review.openstack.org/192736 | 09:55 |
*** gszasz has joined #openstack-nova | 09:55 | |
*** jwcroppe has joined #openstack-nova | 09:56 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Show 'locked_by' information in server Show & List Detail https://review.openstack.org/202391 | 09:58 |
*** lyanchih has joined #openstack-nova | 10:00 | |
*** jordanP has joined #openstack-nova | 10:00 | |
jordanP | deepakcs, yes I am :) | 10:00 |
deepakcs | jordanP: welcome, ur latest patch was just a rebase I assume ? | 10:01 |
deepakcs | jordanP: nvm,i see ur comment that its a rebase, now. I already gave +1 | 10:02 |
*** shuquan has quit IRC | 10:03 | |
jordanP | deepakcs, thanks. Indeed just a rebase | 10:03 |
jordanP | this libvirt/driver.py is really a moving target :) | 10:03 |
mnestratov | hi all | 10:03 |
deepakcs | jordanP: u bet!, i have been doing rebases 80% of the time, in the last 2 months! as my patches touch driver.py tooo | 10:04 |
*** dims has joined #openstack-nova | 10:04 | |
*** stefanb has joined #openstack-nova | 10:04 | |
deepakcs | danpb: thanks for ur review on the onlinesnapshotinfo patch.. just 1 quick comment. I used the direct classes and the intermediate variables, to simplify pep8/flake8 passing :), otherwise, its becoming too long and un-friendly to read | 10:04 |
mnestratov | can anyone from core take a look at bugfix https://review.openstack.org/#/c/202442/ it already has one +2 | 10:05 |
danpb | deepakcs: just put line breaks in where needed | 10:06 |
*** dims_ has joined #openstack-nova | 10:06 | |
deepakcs | danpb: sure, i had in the past had people disliking line breaks, 'guess need to use () then | 10:07 |
*** karimb has joined #openstack-nova | 10:09 | |
*** dims has quit IRC | 10:10 | |
*** BadCub_ has joined #openstack-nova | 10:18 | |
*** aweeks_ has joined #openstack-nova | 10:19 | |
*** aweeks has quit IRC | 10:21 | |
*** vishy has quit IRC | 10:21 | |
*** jmccrory has quit IRC | 10:21 | |
*** BadCub has quit IRC | 10:21 | |
*** rushiagr has quit IRC | 10:21 | |
*** karimb has quit IRC | 10:21 | |
*** Guest66585 has quit IRC | 10:21 | |
*** kevinbenton has quit IRC | 10:21 | |
*** ildikov has quit IRC | 10:21 | |
*** kevinbenton has joined #openstack-nova | 10:22 | |
*** BadCub_ is now known as BadCub | 10:22 | |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw_cpu_threads_policy' to ImageMetaProps https://review.openstack.org/202647 | 10:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=separate' scheduling https://review.openstack.org/202652 | 10:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Make use of 'InstanceNUMACell.cpu_policy' field https://review.openstack.org/202653 | 10:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Make use of 'CPUAllocation' enum values https://review.openstack.org/202654 | 10:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=isolate' scheduling https://review.openstack.org/202655 | 10:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'cpu_policy' and 'cpu_threads_policy' fields https://review.openstack.org/202648 | 10:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=avoid' filtering https://review.openstack.org/202649 | 10:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Add some logs to 'numa_topology_filter' https://review.openstack.org/202650 | 10:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=require' scheduling https://review.openstack.org/202651 | 10:22 |
*** ildikov has joined #openstack-nova | 10:22 | |
*** jmccrory has joined #openstack-nova | 10:22 | |
*** aweeks_ is now known as aweeks | 10:22 | |
*** karimb has joined #openstack-nova | 10:23 | |
*** dan_ has joined #openstack-nova | 10:23 | |
*** dan_ is now known as Guest47142 | 10:24 | |
*** salv-orlando has quit IRC | 10:24 | |
*** vishy has joined #openstack-nova | 10:26 | |
*** promulo_ has joined #openstack-nova | 10:29 | |
*** promulo_ has quit IRC | 10:29 | |
*** mwagner_lap has quit IRC | 10:34 | |
*** coolsvap is now known as coolsvap|away | 10:35 | |
johnthetubaguy | garyk: is they any chance if you could see if this is a real failure? https://review.openstack.org/#/c/204059 | 10:37 |
johnthetubaguy | garyk: I added a recheck, but just wondering if you wanted to push on this one? | 10:37 |
lucasagomes | hi all, can I get some eyes on https://review.openstack.org/#/c/204995/ should be a quick review just adding some better handlers for errors in the Ironic driver | 10:42 |
*** alex_klimov has joined #openstack-nova | 10:42 | |
*** yamahata has joined #openstack-nova | 10:43 | |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=separate' scheduling https://review.openstack.org/202652 | 10:44 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Make use of 'InstanceNUMACell.cpu_policy' field https://review.openstack.org/202653 | 10:44 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Make use of 'CPUAllocation' enum values https://review.openstack.org/202654 | 10:44 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=isolate' scheduling https://review.openstack.org/202655 | 10:44 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Add some logs to 'numa_topology_filter' https://review.openstack.org/202650 | 10:44 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=require' scheduling https://review.openstack.org/202651 | 10:44 |
pczesno | johnthetubaguy: Hi, regarding: https://blueprints.launchpad.net/nova/+spec/virt-driver-cpu-thread-pinning | 10:44 |
pczesno | johnthetubaguy: can we get your approval back, please? | 10:45 |
garyk | johnthetubaguy: i will check that CI on that one. | 10:45 |
johnthetubaguy | garyk: sweet, thank you | 10:45 |
johnthetubaguy | pczesno: it missed the feature proposal freeze, I thought? | 10:47 |
pczesno | yes, we talked about that one week ago and i thought you agreed to reapprove? | 10:47 |
pczesno | it miseed the dadline by few hours | 10:47 |
johnthetubaguy | pczesno: well I said if its fixed by the end of my day, yes, I possibly just missed your IRC ping or something? | 10:48 |
pczesno | johnthetubaguy: i assumed no pingin was neccessary | 10:49 |
johnthetubaguy | pczesno: ah... sorry, I have tracking around 80+ blueprints right now, its easy to miss one | 10:49 |
johnthetubaguy | pczesno: so while I could approve it now, if its not already got a +2 today, its very very unlikely to make liberty-2 I am afraid | 10:50 |
pczesno | johnthetubaguy: i guess that's not easy | 10:50 |
johnthetubaguy | pczesno: I am sorry this slipped through the gaps :( | 10:50 |
*** Marga_ has quit IRC | 10:51 | |
pczesno | johnthetubaguy: i understand that it's late | 10:51 |
pczesno | johnthetubaguy: if you approve, then at least people will take a look at it, and let us now if it's okish or crap | 10:52 |
*** kiran-r has joined #openstack-nova | 10:53 | |
johnthetubaguy | pczesno: thats true, its just I would rather they started by looking at the 50 or so that made the deadline, and in some cases, also missed the last two releases, sorry, I hope that makes sense :( | 10:53 |
sahid | cpu thread policy is only the implementation of a feature which have already been approved by spec | 10:53 |
pczesno | johnthetubaguy: i get that, thanks | 10:54 |
johnthetubaguy | sahid: I am sure its all good stuff, but there are almost 50 that met the deadlines, and it seems fair to concentrate on those first, I know thats painful | 10:54 |
johnthetubaguy | pczesno: I think the best thing to look at here is the object conversion stuff danpb is working on. I think getting that ground work solid should make it quicker next time? | 10:55 |
pczesno | johnthetubaguy: thanks i'll take a look | 10:56 |
johnthetubaguy | pczesno: good news, is we have the spec approved, so you should be good to go as soon as we open for the M release | 10:56 |
johnthetubaguy | pczesno: just ping me on here when we open master for M, and we can get this all approved and up for review that day, hopefully | 10:57 |
pczesno | johnthetubaguy: thanks, we will be ready | 10:57 |
openstackgerrit | Jeffrey Zhang proposed openstack/nova: Add total_vms field in API https://review.openstack.org/203382 | 10:57 |
openstackgerrit | Jeffrey Zhang proposed openstack/nova: Add total_vms field to ComputeNode https://review.openstack.org/202472 | 10:57 |
johnthetubaguy | pczesno: I am hoping we get specs merged before the summit, where possible, so folks are not blocked like this so much in the future, its worth a try, anyways | 10:58 |
lxsli | Does anyone know if sqlalchemy queries with limits+offset are point-in-time or if they reflect underlying changes please? | 10:58 |
lxsli | I'm looking at some code which is deleting rows as it goes and I'm wondering if it's correct to add the previous limit to the offset each loop | 10:59 |
*** fawadkhaliq has joined #openstack-nova | 10:59 | |
sahid | johnthetubaguy: unfortunatly you are the only one who can to approve bleuprint and it's a huge work i can understand | 11:00 |
*** markvoelker has joined #openstack-nova | 11:01 | |
johnthetubaguy | sahid: anyone in nova-drivers can do that, I would for someone to take on my old blueprint czar / release CPL role, that would really help spread the load, but I am failing to find anyone who is keen right now :( | 11:02 |
johnthetubaguy | garyk: alaski: I think this is implemented now, is that correct? https://blueprints.launchpad.net/nova/+spec/cells-instance-migration | 11:05 |
*** markvoelker has quit IRC | 11:05 | |
sahid | https://launchpad.net/~nova-drivers/+members from here ? | 11:06 |
garyk | johnthetubaguy: yes, that is correct. i has been merged | 11:06 |
garyk | johnthetubaguy: i iwll update | 11:06 |
johnthetubaguy | garyk: sweet, thank you for helping out with that priority effort | 11:06 |
openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: libvirt: Nova part of volume snapshot improvements https://review.openstack.org/172813 | 11:06 |
openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: objects: Add support for OnlineSnapshotInfo object https://review.openstack.org/194538 | 11:06 |
deepakcs | danpb: ^^ complied with ur review and re-posted. Thanks | 11:07 |
garyk | sure np | 11:07 |
johnthetubaguy | sahid: yes, I think thats correct, nova-drivers (should be the same as nova-specs-core, in theory) are the folks who approve blueprints, technically speaking | 11:07 |
johnthetubaguy | sahid: its usually the blueprint czar who sorts that all out, which right now, is also me :( | 11:08 |
sahid | johnthetubaguy: hum perhaps you should send an email or refresh this list some of them are not specifically working on drivers | 11:09 |
sahid | i can do that work if you want, i'm really disapointed to see all of these good blueprints not approved | 11:10 |
johnthetubaguy | sahid: me too | 11:10 |
johnthetubaguy | sahid: that list is mostly correct, I just removed jogo, we left him there on purpose, as he transitioned away | 11:11 |
sahid | ahahhum it seems to me dan is motsly working on objecs related things alaski on cells, matt on CI/drivers | 11:12 |
*** pradipta has quit IRC | 11:12 | |
sahid | s/ahahhum/hum | 11:13 |
johnthetubaguy | sahid: there are good reasons for this approach, the one we agreed at the summit, to match what we did in the last release, I have tried to document them here: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule#Why_is_there_a_non-priority_Feature_Freeze_in_Nova.3F | 11:13 |
* sahid looking | 11:13 | |
sahid | unfortunatly i was not here | 11:14 |
johnthetubaguy | sahid: the others are all part of nova-specs-core, and yes, they can all approve blueprints once they approve the spec or similar, mostly we discuss it with those folks in the nova meeting | 11:15 |
*** Marga_ has joined #openstack-nova | 11:16 | |
johnthetubaguy | sahid: would love to catch up on a google hangout, btw, really open to ideas here | 11:20 |
*** fawadkhaliq has quit IRC | 11:21 | |
*** fawadk has joined #openstack-nova | 11:21 | |
*** thorst1 has joined #openstack-nova | 11:22 | |
sahid | johnthetubaguy: yes it's something we can do | 11:26 |
*** salv-orlando has joined #openstack-nova | 11:26 | |
*** scheuran has joined #openstack-nova | 11:28 | |
*** davideagnello has joined #openstack-nova | 11:29 | |
*** jwcroppe_ has joined #openstack-nova | 11:30 | |
*** Marga_ has quit IRC | 11:30 | |
*** salv-orlando has quit IRC | 11:30 | |
*** salv-orlando has joined #openstack-nova | 11:32 | |
*** deepthi has quit IRC | 11:33 | |
*** jwcroppe has quit IRC | 11:33 | |
*** davideagnello has quit IRC | 11:34 | |
openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 11:34 |
*** otter768 has joined #openstack-nova | 11:34 | |
*** markus_z has quit IRC | 11:35 | |
*** Marga_ has joined #openstack-nova | 11:37 | |
*** kiran-r has quit IRC | 11:38 | |
*** otter768 has quit IRC | 11:39 | |
*** lucasagomes is now known as lucas-hungry | 11:44 | |
*** jwcroppe_ has quit IRC | 11:46 | |
*** deepakcs has quit IRC | 11:46 | |
garyk | johnthetubaguy: https://review.openstack.org/#/c/204059/ has the VMware CI +1 | 11:48 |
*** markus_z has joined #openstack-nova | 11:48 | |
*** dguitarbite has quit IRC | 11:50 | |
*** Mike_D_laptop has joined #openstack-nova | 11:51 | |
*** dguitarbite has joined #openstack-nova | 11:56 | |
*** VW_ has joined #openstack-nova | 11:56 | |
*** salv-orlando has quit IRC | 11:58 | |
*** mwagner_lap has joined #openstack-nova | 12:01 | |
*** marcusvrn has joined #openstack-nova | 12:02 | |
*** zhenguo has quit IRC | 12:02 | |
*** raildo has joined #openstack-nova | 12:04 | |
*** thangp has joined #openstack-nova | 12:06 | |
*** baoli has joined #openstack-nova | 12:08 | |
*** jasondotstar has quit IRC | 12:10 | |
*** shuquan has joined #openstack-nova | 12:12 | |
johnthetubaguy | garyk: awesome, thank you | 12:12 |
*** fawadk has quit IRC | 12:13 | |
openstackgerrit | Alex Xu proposed openstack/python-novaclient: Adds support for x509 certificates as keypairs https://review.openstack.org/136458 | 12:14 |
openstackgerrit | Alex Xu proposed openstack/python-novaclient: Add version discover and check in CLI https://review.openstack.org/184281 | 12:14 |
openstackgerrit | Alex Xu proposed openstack/python-novaclient: Add "get_current" method to VersionManager https://review.openstack.org/202296 | 12:14 |
openstackgerrit | Alex Xu proposed openstack/python-novaclient: Set "latest" as default compute api version https://review.openstack.org/184658 | 12:14 |
openstackgerrit | Alex Xu proposed openstack/python-novaclient: Implements 'microversions' api type - Part 2 https://review.openstack.org/167408 | 12:14 |
alex_xu | johnthetubaguy: sdague melwitt ^ microversion client is ready for review | 12:15 |
*** jiaxi_ has joined #openstack-nova | 12:15 | |
jiaxi_ | Hello | 12:15 |
johnthetubaguy | alex_xu: afraid I am making the blueprints about to be hit by the freeze over the next few days my priority right now | 12:15 |
alex_xu | johnthetubaguy: can you remove -2 for api relax validation patch https://review.openstack.org/193857 also, this is ready for review also | 12:15 |
jiaxi_ | Hello.Is anyone here | 12:16 |
*** baoli has quit IRC | 12:16 | |
garyk | jiaxi_: that depends | 12:16 |
jiaxi_ | No one review my patch set for many days .. | 12:16 |
jiaxi_ | many many days | 12:16 |
jiaxi_ | https://review.openstack.org/#/c/204944/ | 12:16 |
*** baoli has joined #openstack-nova | 12:16 | |
alex_xu | johnthetubaguy: emm...sorry, I didn't get your mean... | 12:17 |
*** shuquan has quit IRC | 12:17 | |
johnthetubaguy | alex_xu: removed the -2, didn't know that was still on there | 12:17 |
alex_xu | johnthetubaguy: thanks! | 12:17 |
johnthetubaguy | alex_xu: there is the non-priority feature freeze coming up, so concentrating on that | 12:17 |
jiaxi_ | https://review.openstack.org/#/c/200512/, please have a look at my patch set . Thank you | 12:17 |
garyk | jiaxi_: i suggest reaching out to people who woek on the client - mel witt and kevin mitchell. good luck! | 12:17 |
*** apuimedo has joined #openstack-nova | 12:18 | |
alex_xu | johnthetubaguy: ah, I see now, right, the freeze is the focus now | 12:18 |
garyk | jiaxi_: i suggest that you post that in the keystone channel - if you are not getting nova reviews here i am not sure you will get any keystone ones in this channel :) | 12:18 |
*** annashen has joined #openstack-nova | 12:19 | |
jiaxi_ | garyk: This is a nova related question. | 12:19 |
jiaxi_ | garyk: keystone is too quiet. | 12:19 |
garyk | ok | 12:20 |
johnthetubaguy | jiaxi_: I am afraid we have over 700 patches in review right now, with an average wait time of 23 days right now, this might help you: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule#How_can_I_get_my_code_merged_faster.3F (it seems your patch doesn't pass all our tests, I would look into getting some help with that) | 12:20 |
johnthetubaguy | jiaxi_: there might be more keystone folks around later on today I guess, but if its nova related, we can sure try to help | 12:22 |
jiaxi_ | johnthetubaguy: The tempest is ill. The failed log has on relationship with my code. | 12:22 |
*** sudipto has joined #openstack-nova | 12:22 | |
*** Mike_D_laptop has quit IRC | 12:23 | |
garyk | jiaxi_: if you review peoples patches then hopefully someone may review yours | 12:23 |
*** annashen has quit IRC | 12:23 | |
*** rfolco has joined #openstack-nova | 12:24 | |
jiaxi_ | garyk: Good suggestion. | 12:24 |
johnthetubaguy | jiaxi_: without a jenkins +1 very few people are going to review, I would do another rebase or do a recheck. | 12:25 |
jiaxi_ | johnthetubaguy: It's very kind of you. Thank you for your work. | 12:26 |
openstackgerrit | John Garbutt proposed openstack/python-novaclient: Should change 'injected_file_content_bytes' into 'injected_file_size' https://review.openstack.org/204944 | 12:26 |
johnthetubaguy | jiaxi_: I usually just click the rebase button in gerrit, which should make it easier, it does the hard work for us, sometimes :) | 12:27 |
*** shuquan has joined #openstack-nova | 12:27 | |
jiaxi_ | johnthetubaguy: oh. got it. | 12:27 |
scheuran | danpb, could you please have another look on https://review.openstack.org/#/c/182283/ (macvtap vif) and check if your -1 is still valid? I did some additional research and answered via a comment. | 12:27 |
johnthetubaguy | jiaxi_: +1 to garyk 's tip about reviews also, its in that wiki page, and I added more details here: https://wiki.openstack.org/wiki/Nova/Mentoring#Why_do_code_reviews_if_I_am_not_in_nova-core.3F | 12:28 |
*** kaisers has quit IRC | 12:28 | |
jiaxi_ | johnthetubaguy: Ok, I will read it | 12:29 |
*** Jeffrey4l has quit IRC | 12:30 | |
*** setuid has quit IRC | 12:30 | |
*** edmondsw has joined #openstack-nova | 12:30 | |
*** aysyd has joined #openstack-nova | 12:31 | |
bauzas | johnthetubaguy: eh, missed the "oldest BP patches waiting for" section in http://5885fef486164bb8596d-41634d3e64ee11f37e8658ed1b4d12ec.r44.cf3.rackcdn.com/nova-openreviews.html | 12:32 |
bauzas | johnthetubaguy: great stuff ! | 12:32 |
*** jasondotstar has joined #openstack-nova | 12:32 | |
bauzas | johnthetubaguy: I suspect you use the Implements tag ? | 12:32 |
*** rushiagr_away has joined #openstack-nova | 12:33 | |
bauzas | johnthetubaguy: because I found a BP not exactly having the same patches | 12:33 |
johnthetubaguy | bauzas: yep, its just auto generated using the tag | 12:33 |
johnthetubaguy | bauzas: that was for the review day a few weeks back | 12:33 |
bauzas | johnthetubaguy: okay, because I'm following https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/soft-affinity-for-server-group,n,z | 12:34 |
johnthetubaguy | bauzas: you can get a newer one at reviews.johnthetubaguy.com | 12:34 |
bauzas | johnthetubaguy: coolness | 12:34 |
bauzas | johnthetubaguy: lemme check if there is a bug :) | 12:34 |
johnthetubaguy | bauzas: its not auto updated right now | 12:34 |
*** shuquan has quit IRC | 12:34 | |
bauzas | mmm ok | 12:35 |
*** setuid has joined #openstack-nova | 12:35 | |
bauzas | johnthetubaguy: I remember having such discussion with you a couple of weeks ago, yeah | 12:35 |
bauzas | johnthetubaguy: so, no launchpadlib check ? | 12:35 |
johnthetubaguy | bauzas: can't remember, I think thats still only in my other tool right now | 12:36 |
bauzas | johnthetubaguy: because I feel something wrong for soft-affinity-for-server-group like I said | 12:36 |
*** kylek3h has joined #openstack-nova | 12:36 | |
bauzas | johnthetubaguy: its marked 1 patch, but I can find 7 | 12:36 |
johnthetubaguy | bauzas: you can see some stuff here: http://status.openstack.org/release/ | 12:36 |
bauzas | johnthetubaguy: 6 are -1'd from Jenkins tho | 12:36 |
bauzas | johnthetubaguy: okay, my take is to see | 12:37 |
johnthetubaguy | bauzas: it could have been one patch when I generated the list, it doesn't filter out jenkins things, on purpose | 12:37 |
bauzas | #1, I want to review non-prio specs | 12:37 |
*** eharney has quit IRC | 12:37 | |
bauzas | #2, I want to review prio specs | 12:37 |
johnthetubaguy | bauzas: see this: http://status.openstack.org/release/ | 12:37 |
bauzas | #3, I want to review bugs | 12:37 |
bauzas | johnthetubaguy: yup, I know this webpage | 12:38 |
bauzas | johnthetubaguy: that just shows me which ones are marked Needs-Code-Review | 12:38 |
johnthetubaguy | bauzas: and the list of patches, but yes | 12:39 |
bauzas | johnthetubaguy: from my POV, a quickwin would be to modify your reviewsstats to call launchpadlib | 12:39 |
*** vladikr has joined #openstack-nova | 12:39 | |
bauzas | so to get the list of changes and put a link to gerrit | 12:39 |
bauzas | oh, missed the changelist | 12:40 |
bauzas | that's icons, right? | 12:40 |
johnthetubaguy | bauzas: its not really my tool, I just hacked russells to add the extra bits | 12:40 |
johnthetubaguy | bauzas: yeah, its in the icons | 12:40 |
bauzas | johnthetubaguy: oh nice | 12:40 |
bauzas | johnthetubaguy: that makes me change my mind | 12:41 |
bauzas | johnthetubaguy: thanks for having noticed me that, I was feeling that webpage not useful, this is not the case | 12:42 |
*** kaisers has joined #openstack-nova | 12:43 | |
*** lucas-hungry is now known as lucasagomes | 12:44 | |
*** Marga_ has quit IRC | 12:48 | |
*** kaisers has quit IRC | 12:50 | |
*** kaisers1 has joined #openstack-nova | 12:50 | |
*** matrohon has quit IRC | 12:50 | |
*** jiaxi_ has quit IRC | 12:52 | |
*** annegentle has joined #openstack-nova | 12:54 | |
*** rotbeard has quit IRC | 12:56 | |
openstackgerrit | Sudipta Biswas proposed openstack/nova: Base enablement for memory bandwidth monitor https://review.openstack.org/202623 | 12:56 |
*** rlrossit has joined #openstack-nova | 12:58 | |
*** markus_z has quit IRC | 12:59 | |
*** kaisers1 has quit IRC | 13:00 | |
*** zhenguo has joined #openstack-nova | 13:01 | |
*** eglynn has joined #openstack-nova | 13:02 | |
*** dave-mccowan has joined #openstack-nova | 13:03 | |
*** markus_z has joined #openstack-nova | 13:03 | |
*** fawadkhaliq has joined #openstack-nova | 13:03 | |
*** rushiagr_away is now known as rushiagr | 13:03 | |
*** cpschult has joined #openstack-nova | 13:07 | |
*** edmondsw has quit IRC | 13:08 | |
*** tongli has joined #openstack-nova | 13:08 | |
*** achanda has joined #openstack-nova | 13:08 | |
*** diogogmt has quit IRC | 13:09 | |
garyk | is there any chance to be for a review for https://review.openstack.org/#/c/188789 | 13:10 |
*** annegentle has quit IRC | 13:10 | |
*** annegentle has joined #openstack-nova | 13:11 | |
*** diogogmt has joined #openstack-nova | 13:11 | |
johnthetubaguy | garyk: I have that open in a tab right now | 13:11 |
*** achanda has quit IRC | 13:13 | |
*** diogogmt has quit IRC | 13:15 | |
*** thorst1 has quit IRC | 13:16 | |
*** thorst1 has joined #openstack-nova | 13:16 | |
*** ctrath has joined #openstack-nova | 13:17 | |
*** ivasev has joined #openstack-nova | 13:18 | |
*** browne has joined #openstack-nova | 13:19 | |
*** Marga_ has joined #openstack-nova | 13:21 | |
*** eharney has joined #openstack-nova | 13:22 | |
*** kiran-r has joined #openstack-nova | 13:22 | |
*** kaisers has joined #openstack-nova | 13:23 | |
*** chuckcarmack has joined #openstack-nova | 13:26 | |
*** _kiran_ has joined #openstack-nova | 13:27 | |
*** mpaolino has joined #openstack-nova | 13:27 | |
*** browne has quit IRC | 13:28 | |
openstackgerrit | garyk proposed openstack/nova: VMware: add in net_io resource limitations https://review.openstack.org/180202 | 13:28 |
openstackgerrit | garyk proposed openstack/nova: VMware: add support for cores per socket https://review.openstack.org/187942 | 13:28 |
openstackgerrit | garyk proposed openstack/nova: VMware: support limits via image meta data https://review.openstack.org/180942 | 13:28 |
openstackgerrit | garyk proposed openstack/nova: Objects: add limits, reservations and shares to ImageMetaProps object https://review.openstack.org/188321 | 13:28 |
openstackgerrit | garyk proposed openstack/nova: VMware: create common object for limits, reservations and shares https://review.openstack.org/179059 | 13:28 |
openstackgerrit | garyk proposed openstack/nova: VMware: Resource limits for memory https://review.openstack.org/179060 | 13:28 |
openstackgerrit | garyk proposed openstack/nova: virt: convert hardware module to use nova.objects.ImageMeta https://review.openstack.org/188789 | 13:28 |
openstackgerrit | garyk proposed openstack/nova: VMware: store extra_specs object https://review.openstack.org/179061 | 13:28 |
openstackgerrit | garyk proposed openstack/nova: VMware: add resource limits for disk https://review.openstack.org/179062 | 13:28 |
*** n0ano has joined #openstack-nova | 13:30 | |
*** kiran-r has quit IRC | 13:30 | |
*** davideagnello has joined #openstack-nova | 13:30 | |
*** mjturek1 has joined #openstack-nova | 13:31 | |
lxsli | what should someone do if they fixed a bug but their code hasn't been reviewed after a month? | 13:32 |
lxsli | I mean by core - assume it already has +1s | 13:32 |
*** markvoelker has joined #openstack-nova | 13:34 | |
*** davideagnello has quit IRC | 13:35 | |
openstackgerrit | Ryan Rossiter proposed openstack/nova: Change List objects to use obj_relationships https://review.openstack.org/198730 | 13:35 |
*** otter768 has joined #openstack-nova | 13:35 | |
dims_ | lxsli: which one? | 13:35 |
*** samueldmq has quit IRC | 13:35 | |
lxsli | dims_: non-specific, I'm rewriting the advice on "what to do if my patch got stuck?" | 13:36 |
*** samueldmq has joined #openstack-nova | 13:36 | |
dims_ | lxsli: ah. | 13:36 |
lxsli | For now I've put "ping IRC each week" | 13:36 |
*** sudipto has quit IRC | 13:38 | |
*** annegent_ has joined #openstack-nova | 13:38 | |
dims_ | lxsli: "look through the git history and find folks (especially cores) who may have worked in this area and ping them on IRC" | 13:38 |
lxsli | I think we're trying to discourage direct core-pinging | 13:38 |
*** _kiran_ has quit IRC | 13:39 | |
dims_ | lxsli: random request never works...right? | 13:39 |
openstackgerrit | garyk proposed openstack/nova: Metadata: support proxying loadbalancers https://review.openstack.org/147539 | 13:39 |
openstackgerrit | garyk proposed openstack/nova: Neutron: add in API method for updating VNIC index https://review.openstack.org/147126 | 13:39 |
*** lucas__ has joined #openstack-nova | 13:39 | |
*** otter768 has quit IRC | 13:40 | |
lxsli | not sure what you mean by random request; an open request is best I find, as you can get +/- 1 and might get lucky with +2 | 13:40 |
garyk | lxsli: we are all between a rock and a hard place here | 13:40 |
lxsli | garyk: yup yup | 13:41 |
*** ijuwang has joined #openstack-nova | 13:41 | |
garyk | lxsli: we also need to prepare ourselves for the up and coming dry season - august will most probably be vacations | 13:41 |
*** annegentle has quit IRC | 13:41 | |
bauzas | lxsli: I guess your bugfix is non trivial ? | 13:42 |
bauzas | lxsli: if not, which severity has been assigned ? | 13:43 |
bauzas | lxsli: in general, bugs with severity High and Critical deserve good reviews | 13:43 |
garyk | bauzas: critical bug fixes are also waiting for months... | 13:43 |
bauzas | garyk: not sure I understand - we're reviewing weekly the critical ones during the nova meeting | 13:43 |
*** jungleboyj has quit IRC | 13:44 | |
bauzas | we also identify a stakeholder per tag, here | 13:44 |
*** lucas__ has quit IRC | 13:44 | |
bauzas | lxsli: https://wiki.openstack.org/wiki/Nova/BugTriage#Step_2:_Triage_Tagged_Bugs | 13:44 |
*** mdrabe has joined #openstack-nova | 13:44 | |
bauzas | lxsli: so, here is the thing | 13:44 |
openstackgerrit | Alexis Lee proposed openstack/nova: DO NOT MERGE https://review.openstack.org/206070 | 13:45 |
bauzas | lxsli: 1/ check the severity | 13:45 |
lxsli | johnthetubaguy: https://review.openstack.org/206070 | 13:45 |
garyk | bauzas: yeah, in the nova meeting, but not in gerrit ... | 13:45 |
bauzas | lxsli: 2/ bug the corresponding stackholder | 13:45 |
bauzas | lxsli: 3/ the stakeholder can put attention on a pile of bugfixes if those are worth of interesrt | 13:46 |
*** alexpilotti has quit IRC | 13:46 | |
bauzas | garyk: um, do you have some examples ? | 13:46 |
*** coolsvap|away is now known as coolsvap | 13:47 | |
bauzas | garyk: since criticals are weekly reviewed in meetings, why aren't the proposers not bugging cores during that meeting ? | 13:47 |
bauzas | garyk: if they feel it's not properly handled ? | 13:47 |
garyk | bauzas: i have so many that i have even lost track. seriously. | 13:47 |
garyk | but we are laboring the point. | 13:48 |
johnthetubaguy | yeah, we check the critical bugs in every meeting, we don't review the high list, but yeah | 13:48 |
bauzas | garyk: well, I'm now more focusing on bugs since last January midcycle, and I don't remember a specific Critical facing a huge review lag | 13:48 |
garyk | yes, it is not properly handled | 13:48 |
*** ijuwang has quit IRC | 13:48 | |
*** READ10 has joined #openstack-nova | 13:48 | |
*** jasondotstar has quit IRC | 13:48 | |
johnthetubaguy | lxsli: honestly, its likely to be next week when I can review that, liberty-2 is distracting me this week | 13:48 |
bauzas | johnthetubaguy: High severity bugs should be reviewed frequently by bug stakeholders | 13:48 |
johnthetubaguy | lxsli: I am planning on pushing that whole wiki into devref, so its possible worth trying that | 13:48 |
bauzas | and yeah, we know that we drag bugs mostly after FF | 13:49 |
andrearosa | danpb: are you around for a question about machine-id? ref bug 1337349 | 13:49 |
openstack | bug 1337349 in OpenStack Security Notes "Nova qemu hypervisor host smbios serial number is leaked to guest" [Medium,Fix released] https://launchpad.net/bugs/1337349 - Assigned to Nathan Kinder (nkinder) | 13:49 |
johnthetubaguy | bauzas: agreed, the priorities etherpad is where the subteams get to push their top few bugs fixes to the core team, the subteam can have a bug tag to track them, ideally | 13:49 |
bauzas | and we have https://wiki.openstack.org/wiki/Nova/BugTriage#Step_2:_Triage_Tagged_Bugs | 13:49 |
johnthetubaguy | bauzas: I think markus_z is going to digging into the bug situation for us, to help with this | 13:50 |
markus_z | bauzas: I have committed myself to the bug czar role and like to streamline a few things regarding bugs | 13:50 |
lxsli | johnthetubaguy: no rush, whenever you want it fixed :) | 13:50 |
bauzas | that's a POC list that bug owners can take for bugging people | 13:50 |
johnthetubaguy | lxsli: well I want it fixed last week, but yeah, snowed under this week with all the release tracking and reviewing load | 13:50 |
*** zzzeek has joined #openstack-nova | 13:50 | |
markus_z | I need this week to properly write the things down I have in mind. | 13:50 |
bauzas | so, solution found : hassle markus_z ! :p | 13:51 |
markus_z | One of them is to address bugs and theirs reviews by impact/importance | 13:51 |
markus_z | bauzas: waaahhh! ;) | 13:51 |
*** aysyd has quit IRC | 13:51 | |
danpb | andrearosa: what about it ? | 13:51 |
markus_z | bauzas: I have a lot of patches in my stash, maybe they fit to some of the problems ;) | 13:51 |
bauzas | markus_z: like I said, my review attention is really depending on the cycle | 13:52 |
bauzas | markus_z: I mean, most of the reviewers are also committers :p | 13:52 |
*** jistr has quit IRC | 13:52 | |
*** ijuwang has joined #openstack-nova | 13:53 | |
bauzas | markus_z: so in general, I defer reviewing of bugs till FF | 13:53 |
bauzas | markus_z: and I suspect I'm not alone | 13:53 |
andrearosa | quick question: in the case of we use the 'os' option we return the value in the /etc/machine-id file and we raise an error if the fhe file is missed. what we should do if the file is there but empty? I think we should just return an empty string, what do you think? | 13:53 |
markus_z | bauzas: I understand that and I don't doubt that everyone works as best and fast as possible. | 13:53 |
*** dane-fichter has joined #openstack-nova | 13:53 | |
johnthetubaguy | lxsli: maybe add that change into this BP for me, so I don't forget about it? https://blueprints.launchpad.net/nova/+spec/devref-refresh-liberty although probably best just to ping me next week, if you can, it looks like a good update at a quick look through | 13:53 |
bauzas | markus_z: so, what I see is that we have a pile of bugs triaged by severity and component | 13:53 |
*** jistr has joined #openstack-nova | 13:54 | |
markus_z | bauzas: true | 13:54 |
bauzas | markus_z: based on that, we can at least make sure that we properly consider high severity bugs correctly with the subteams | 13:54 |
openstackgerrit | garyk proposed openstack/nova: VMware: improve instance names on VC https://review.openstack.org/166608 | 13:55 |
openstackgerrit | garyk proposed openstack/nova: VMware: add in folder support on VC https://review.openstack.org/165060 | 13:55 |
markus_z | bauzas: right | 13:55 |
bauzas | markus_z: but we can't really act on reviewing a bugfix unless it's either trivial or when we have more bandwidth | 13:55 |
lxsli | johnthetubaguy: sure thing | 13:55 |
*** aysyd has joined #openstack-nova | 13:56 | |
johnthetubaguy | lxsli: cool, thank you | 13:56 |
markus_z | bauzas: the bandwidth is still an issue, that's true | 13:57 |
*** mriedem has joined #openstack-nova | 13:57 | |
*** edmondsw has joined #openstack-nova | 13:57 | |
*** thangp_ has joined #openstack-nova | 13:57 | |
bauzas | markus_z: so, like I said, we already do Criticals on Nova meetings, we could ask the subteams to weekly review Highes - if they do weekly meetings | 13:58 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/206081 | 13:58 |
bauzas | markus_z: for the subteam, that's just part of gaining credibility from the nova-core team | 13:58 |
bauzas | s/credibility/trust | 13:58 |
lxsli | markus_z: do all these criticals have patches proposed already? Are those patches of mergeable quality? | 13:59 |
markus_z | lxsli: I'm still investigating how we are doing right now. I don't have numbers. | 13:59 |
lxsli | markus_z: OK, that'd be good to know, they're very different problems | 14:00 |
bauzas | lxsli: we weekly review Launchpad | 14:00 |
openstackgerrit | Timofey Durakov proposed openstack/nova: Split instance resize operation into 2 phases https://review.openstack.org/195088 | 14:00 |
openstackgerrit | Timofey Durakov proposed openstack/nova: Create class hierarchy for tasks in conductor https://review.openstack.org/184495 | 14:00 |
markus_z | lxsli: bauzas: That's what I try to figure out this week. | 14:00 |
bauzas | markus_z: you never went to a nova meeting ? never saw mriedem or me saying 'nothing for this week" ? | 14:01 |
lxsli | it's really only a core bandwidth problem once there's a high-quality fix proposed | 14:01 |
*** thangp has quit IRC | 14:01 | |
*** nelsnelson has joined #openstack-nova | 14:01 | |
markus_z | bauzas: Nope | 14:02 |
bauzas | markus_z: ok, look at that one | 14:02 |
bauzas | http://eavesdrop.openstack.org/meetings/nova/2015/nova.2015-07-16-21.01.log.html | 14:02 |
*** boris-42 has joined #openstack-nova | 14:02 | |
bauzas | starting 21:08:34 | 14:02 |
johnthetubaguy | lxsli: yeah, its the usual things at that point, need to see more use of the subteam etherpad, and I really want to see more of this too: https://wiki.openstack.org/wiki/Nova/Mentoring#Why_do_code_reviews_if_I_am_not_in_nova-core.3F | 14:02 |
*** jungleboyj has joined #openstack-nova | 14:02 | |
johnthetubaguy | it the high list got under control | 14:03 |
johnthetubaguy | we could review that list | 14:03 |
johnthetubaguy | or at least review the stats | 14:03 |
markus_z | bauzas: Yes, I've seen that you do a lot of work there. | 14:03 |
andrearosa | danpb: sorry I put my question above ^^^ without quote you. | 14:03 |
johnthetubaguy | but yeah, lets see how it goes | 14:03 |
*** exploreshaifali has joined #openstack-nova | 14:03 | |
bauzas | markus_z: I'm just explaining that we can raise attention to the team at that time | 14:04 |
markus_z | bauzas: Yes, thanks, I understand that. | 14:04 |
bauzas | markus_z: if one Critical bug doesn't get Fix Committed from one week to another, we should have been noticed it | 14:04 |
*** MentalRay has joined #openstack-nova | 14:04 | |
n0ano | bauzas, edleafe lxsli scheduler meeting? | 14:04 |
bauzas | that's why I'm a bit surprised by what garyk said about criticals | 14:04 |
bauzas | n0ano: *facepalm* | 14:05 |
garyk | bauzas: take a look at the bug list. that would be a start. | 14:05 |
n0ano | bauzas, 20 lashes with a wet noodle :-) | 14:05 |
andrearosa | danpb: basically when we read UUID from machine-id if the file is nto there we raise an error, what we should do when the file is there but empty? I think we should return an empty string, do you agre? | 14:05 |
garyk | then take alook at how many of the high and critical are in review | 14:05 |
*** lucas__ has joined #openstack-nova | 14:05 | |
bauzas | garyk: I'm not talking about High bugs, just criticals | 14:05 |
bauzas | garyk: like I said, I'm reviewing on a weekly basis both Nova and Python-Novaclient Launchpad buglists | 14:06 |
garyk | bauzas: thanks for reviewing. | 14:06 |
danpb | andrearosa: where do you see us raising an error - the code checks to see if machine-id exists and if not fetches the uuid from libvirt | 14:07 |
markus_z | bauzas: I see the overall amount of bugs as an issue I'd like to tackle. | 14:07 |
markus_z | bauzas: Mainly the turnaround time from reporting to closed/fixed. | 14:07 |
bauzas | markus_z: we know that some components face more bugs than others | 14:07 |
*** erlon has joined #openstack-nova | 14:08 | |
markus_z | bauzas: Yes, that's my observation too | 14:08 |
andrearosa | danpb: that happens olny if we call _get_host_sysinfo_serial_auto but if we call _get_host_sysinfo_serial_os there is no check | 14:09 |
bauzas | markus_z: sorry, sched subteam meeting now, ETOOMANYCONVOS | 14:09 |
*** lucas__ has quit IRC | 14:09 | |
andrearosa | danpb: I thought it was done on purpose as in the commit message of the but it is clearly specified: "'os' - always use /etc/machine-id, error if missing" | 14:10 |
markus_z | bauzas: no worries, I'd like to talk to you about that in a later point in time. | 14:10 |
danpb | andrearosa: so where's the actual bug then ? if you set sysinfo_serial=os, and machine-id is missing, then it is is correct to raise an error | 14:11 |
*** tonytan4ever has joined #openstack-nova | 14:11 | |
bauzas | markus_z: keep online, I'll drag you on the meeting later on | 14:11 |
bauzas | markus_z: because I'll talk about your stuff by the open discussion time | 14:11 |
andrearosa | danpb: correct, my question is about what to do if the file is there but it is empty | 14:12 |
markus_z | bauzas: which channel? | 14:12 |
openstackgerrit | Alexis Lee proposed openstack/nova: DO NOT MERGE - halp.wiki https://review.openstack.org/206070 | 14:12 |
danpb | andrearosa: IMHO that is simply a broken OS install | 14:13 |
danpb | the admin should fix that | 14:13 |
*** lucas__ has joined #openstack-nova | 14:13 | |
bauzas | #openstack-meeting-alt | 14:14 |
*** xyang2 has joined #openstack-nova | 14:14 | |
bauzas | markus_z: ^ | 14:14 |
*** mpaolino has quit IRC | 14:15 | |
*** jecarey has joined #openstack-nova | 14:15 | |
*** Murali has quit IRC | 14:16 | |
*** cfriesen has joined #openstack-nova | 14:16 | |
andrearosa | danpb: ok I am asking because I found this bug https://bugs.launchpad.net/ubuntu/+source/nova/+bug/1413293 | 14:16 |
openstack | Launchpad bug 1413293 in nova (Ubuntu) "Unable to start instances, empty /etc/machine-id file." [High,Triaged] | 14:16 |
*** ankit_ag has quit IRC | 14:18 | |
*** karimb has quit IRC | 14:19 | |
andrearosa | danpb: and one of the options was to teach nova how to manage an empty file | 14:20 |
*** salv-orlando has joined #openstack-nova | 14:21 | |
*** karimb has joined #openstack-nova | 14:25 | |
danpb | andrearosa: we should raise an error if it is empty then | 14:25 |
*** atuvenie has joined #openstack-nova | 14:26 | |
*** exploreshaifali_ has joined #openstack-nova | 14:26 | |
*** ferest has joined #openstack-nova | 14:27 | |
*** armax has joined #openstack-nova | 14:27 | |
*** ferest has quit IRC | 14:27 | |
*** jasondotstar has joined #openstack-nova | 14:28 | |
*** ferest has joined #openstack-nova | 14:28 | |
*** exploreshaifali has quit IRC | 14:28 | |
*** ferest has quit IRC | 14:29 | |
*** scheuran has quit IRC | 14:29 | |
*** ferest has joined #openstack-nova | 14:30 | |
*** ferest has quit IRC | 14:30 | |
*** armax has quit IRC | 14:30 | |
*** armax has joined #openstack-nova | 14:30 | |
openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 14:31 |
*** ferest has joined #openstack-nova | 14:31 | |
*** changbl has quit IRC | 14:32 | |
andrearosa | danpb: thank you very much for your time | 14:32 |
*** ferest has quit IRC | 14:32 | |
mriedem | no dansmith in channel huh? | 14:33 |
kashyap | mriedem: It's superdan I guess | 14:34 |
*** ferest has joined #openstack-nova | 14:34 | |
mriedem | nope | 14:34 |
mriedem | it's dansmith404 | 14:34 |
*** vilobhmm has joined #openstack-nova | 14:35 | |
*** ferest has quit IRC | 14:35 | |
*** Guest14296 is now known as dansmith | 14:35 | |
*** salv-orlando has quit IRC | 14:35 | |
*** ferest has joined #openstack-nova | 14:35 | |
dansmith | silly freenode | 14:35 |
*** ferest has quit IRC | 14:36 | |
*** ijw has quit IRC | 14:36 | |
rlrossit | dansmith: don't lie, you were just trying to hide from mriedem | 14:36 |
*** armax has quit IRC | 14:36 | |
*** vilobhmm1 has joined #openstack-nova | 14:36 | |
*** ferest has joined #openstack-nova | 14:37 | |
dansmith | heh | 14:37 |
mriedem | it was unimportant | 14:37 |
mriedem | deep cut from the 70s on a monday morning https://www.youtube.com/watch?v=TOjblP64HoI | 14:38 |
*** ferest has quit IRC | 14:38 | |
*** vilobhmm has quit IRC | 14:39 | |
ndipanov | slam dunk https://review.openstack.org/#/c/202737/ | 14:39 |
*** ferest has joined #openstack-nova | 14:40 | |
*** ferest has quit IRC | 14:41 | |
*** ferest has joined #openstack-nova | 14:41 | |
*** ftersin has joined #openstack-nova | 14:42 | |
*** ferest has quit IRC | 14:42 | |
*** scheuran has joined #openstack-nova | 14:42 | |
*** ferest has joined #openstack-nova | 14:43 | |
*** fawadkhaliq has quit IRC | 14:43 | |
*** ferest has quit IRC | 14:43 | |
*** salv-orlando has joined #openstack-nova | 14:44 | |
*** ferest has joined #openstack-nova | 14:44 | |
*** ferest has quit IRC | 14:45 | |
*** r-daneel has joined #openstack-nova | 14:46 | |
*** ferest has joined #openstack-nova | 14:46 | |
*** ferest has quit IRC | 14:47 | |
scheuran | danpb, could you please have another look on https://review.openstack.org/#/c/182283/ (macvtap vif) and check if your -1 is still valid? I did some additional research and answered via a comment. | 14:47 |
*** ferest has joined #openstack-nova | 14:48 | |
*** ferest has quit IRC | 14:48 | |
*** mtanino has joined #openstack-nova | 14:48 | |
*** jaypipes has joined #openstack-nova | 14:48 | |
*** lucas__ has quit IRC | 14:49 | |
*** sajeesh has joined #openstack-nova | 14:49 | |
*** ferest has joined #openstack-nova | 14:49 | |
*** shaifali__ has joined #openstack-nova | 14:50 | |
*** ferest has quit IRC | 14:50 | |
*** sajeesh has quit IRC | 14:50 | |
*** ferest has joined #openstack-nova | 14:51 | |
*** lucas__ has joined #openstack-nova | 14:51 | |
*** ferest has quit IRC | 14:51 | |
*** exploreshaifali_ has quit IRC | 14:52 | |
*** ferest has joined #openstack-nova | 14:52 | |
mtreinish | mriedem: ooh, my sphinx patch landed, now it supports latex check marks | 14:53 |
*** annegent_ has quit IRC | 14:54 | |
*** ferest has quit IRC | 14:55 | |
*** ferest has joined #openstack-nova | 14:56 | |
*** tpeoples has joined #openstack-nova | 14:56 | |
*** ferest has quit IRC | 14:56 | |
*** kbyrne has quit IRC | 14:56 | |
*** rajesht has quit IRC | 14:57 | |
mriedem | mtreinish: still waiting on mine that actually makes the apidoc generation pass | 14:57 |
mriedem | ndipanov: slam dunk https://review.openstack.org/#/c/199297/ | 14:57 |
mtreinish | mriedem: heh, mine was kinda stupid simple: https://github.com/sphinx-doc/sphinx/pull/1971 | 14:58 |
mtreinish | I imagine yours is a bit more involved | 14:58 |
mriedem | although at this point, https://review.openstack.org/#/c/199297/ probably fails gate-nova-docs | 14:58 |
mriedem | well, with https://review.openstack.org/#/c/199297/ - if gate-nova-docs works, then it works | 14:58 |
mriedem | and http://docs-draft.openstack.org/97/199297/2/check/gate-nova-docs/86c41dd//doc/build/html/py-modindex.html | 14:58 |
mriedem | since that doesn't work before | 14:58 |
mtreinish | oh, right that patch. It makes the latex output super useful :) | 15:00 |
*** mestery has joined #openstack-nova | 15:00 | |
mriedem | mtreinish: we could probably configure that somehow | 15:00 |
mriedem | to ignore doc/build/api/autoindex when building latex | 15:00 |
mtreinish | yeah, probably but it's not like it really matters too much | 15:00 |
mriedem | anyway, got a meeting | 15:00 |
lxsli | edleafe: we should design from scratch, then figure out how to get there from here | 15:00 |
mtreinish | how many people are going to bother with using latex to generate a pdf of the nova docs | 15:01 |
lxsli | bauzas: "if booting an instance, that sounds very Nova to me" | 15:01 |
bauzas | lxsli: yup ? | 15:02 |
lxsli | bauzas: if claiming a bundle of resources, that sounds not very Nova-specific to me | 15:02 |
*** alexpilotti has joined #openstack-nova | 15:02 | |
edleafe | lxsli: that would be ideal, but we have to deal with an existing community | 15:02 |
edleafe | lxsli: "rip it out and start fresh" is a complete non-starter | 15:02 |
bauzas | lxsli: that's just talking about UX | 15:02 |
edleafe | lxsli: no matter how much better it would be | 15:03 |
lxsli | edleafe: that's almost entirely not what I said | 15:03 |
bauzas | lxsli: ie. a nova boot command or no | 15:03 |
*** ddaskal has joined #openstack-nova | 15:03 | |
edleafe | lxsli: sure, but I'm giving the extreme version | 15:03 |
lxsli | edleafe: that's a strawman | 15:03 |
edleafe | lxsli: not at all | 15:03 |
bauzas | lxsli: you could possibly express something like nova boot --volume=<> instance and make sure that a VolumeAffinityFilter would put your instance close to your volume | 15:03 |
edleafe | lxsli: just a reality check | 15:04 |
lxsli | edleafe: "the extreme version is very stupid" "let's not do the extreme version then" | 15:04 |
edleafe | lxsli: you misunderstood | 15:04 |
*** exploreshaifali_ has joined #openstack-nova | 15:04 | |
bauzas | lxsli: but a cinder create volume is certainly not something Nova | 15:04 |
*** shaifali__ has quit IRC | 15:04 | |
edleafe | lxsli: if you propose starting from scratch, you will face a *ton* of resistance | 15:04 |
lxsli | edleafe: I am advocating a clean design then gradual iteration towards that | 15:04 |
edleafe | lxsli: ok, good | 15:05 |
edleafe | lxsli: I'd like your feedback when I propose my clean design shortly | 15:05 |
lxsli | edleafe: certainly :) | 15:05 |
*** diogogmt has joined #openstack-nova | 15:06 | |
lxsli | bauzas: just untwingling the convo | 15:06 |
*** exploreshaifali_ has quit IRC | 15:06 | |
lxsli | bauzas: so to me, we need a generic optimising bin-packer to which we can supply hard and soft constraints | 15:06 |
lxsli | bauzas: when we want to boot an instance, one hard constraint is that we get a new instance | 15:07 |
bauzas | on a call now :' | 15:07 |
bauzas | lxsli: ^ | 15:07 |
lxsli | bauzas: shucks, ok | 15:07 |
*** nkrinner has quit IRC | 15:09 | |
*** kbyrne has joined #openstack-nova | 15:10 | |
*** achanda has joined #openstack-nova | 15:11 | |
*** moshele has quit IRC | 15:12 | |
*** ildikov has quit IRC | 15:12 | |
jaypipes | bauzas: marooning. haven't gotten to emails yet. what is the status of the request-spec series? | 15:12 |
bauzas | jaypipes: reviewable for your pleasure | 15:13 |
*** yamahata has quit IRC | 15:13 | |
bauzas | CI is happy, you give a shot | 15:13 |
bauzas | you can$ | 15:13 |
jaypipes | bauzas: rock on. ok, I will do that top priority review today. | 15:13 |
bauzas | coolio | 15:14 |
bauzas | (on a call now) | 15:14 |
lxsli | jaypipes: in the nicest possible way - why isn't your resource objects work merged yet? | 15:14 |
lxsli | jaypipes: did dan just not have time or is there an issue? | 15:15 |
jaypipes | lxsli: because bauzas doesn't like it. | 15:15 |
*** achanda has quit IRC | 15:15 | |
lxsli | oh, THAT guy | 15:16 |
*** salv-orlando has quit IRC | 15:16 | |
*** browne has joined #openstack-nova | 15:16 | |
jaypipes | lxsli: no, I'm being serious. | 15:17 |
lxsli | jaypipes: yeah sorry, I see his comments and am responding | 15:18 |
*** baoli has quit IRC | 15:18 | |
bauzas | jaypipes: lxsli: don't shoot the messenger, feel free to engage with him if needed :) | 15:18 |
*** kmartin has joined #openstack-nova | 15:18 | |
*** baoli has joined #openstack-nova | 15:18 | |
jaypipes | bauzas: :) I will. just haven't had a chance to respond yet. | 15:19 |
* jaypipes invites lxsli to engage with bauzas on the resource-objects patches. ;) | 15:19 | |
*** dannywilson has joined #openstack-nova | 15:19 | |
lxsli | jaypipes: can I ask you why you didn't add is_negative at least to the Amount ABC? | 15:19 |
*** burt has joined #openstack-nova | 15:19 | |
lxsli | I'd put __eq__ etc on there too but you mumbled something about that not being possible? Is this a Python limitation I don't know? | 15:20 |
jaypipes | lxsli: because it isn't a method. | 15:20 |
lxsli | Hmm you could have a @property which raises UnsupportedOperationException though right? | 15:20 |
kashyap | danpb: Wow, just discovered this little gem -- https://wiki.openstack.org/wiki/LibvirtVIFTypeXMLConfigs | 15:20 |
* lxsli betrays his heritage | 15:20 | |
jaypipes | lxsli: perhaps. | 15:20 |
*** eharney has quit IRC | 15:20 | |
jaypipes | lxsli: you could submit a patch revision if you feel strongly about that. I wouldn't mind. | 15:21 |
bauzas | okay, if we have a Python limitation, I'd certainly accept some explanatioin | 15:21 |
*** Sukhdev has joined #openstack-nova | 15:21 | |
*** dims_ has quit IRC | 15:21 | |
lxsli | jaypipes: I'm a bit uncomfortable that two people have questioned it now but on the other hand it seems natural to have a unifying interface across these similarly-behaving things | 15:21 |
*** tsekiyama has joined #openstack-nova | 15:21 | |
*** dims has joined #openstack-nova | 15:21 | |
*** aix has quit IRC | 15:22 | |
jaypipes | lxsli: it used to have an abstract base class but a number of people questioned its usefulness. see the first patches in the series.... | 15:22 |
lxsli | ha | 15:22 |
*** rushiagr is now known as rushiagr_away | 15:22 | |
*** zhenguo has quit IRC | 15:22 | |
*** thedodd has joined #openstack-nova | 15:22 | |
thorst1 | Hey. Wondering if a core could take a look at https://review.openstack.org/#/c/203846/ I just hit the issue myself, and it looks like this proposed fix has been out for a few days. | 15:25 |
*** kiran-r has joined #openstack-nova | 15:25 | |
*** _kiran_ has joined #openstack-nova | 15:26 | |
jaypipes | thorst1: looking... | 15:29 |
*** inara` has quit IRC | 15:29 | |
*** kylek3h has quit IRC | 15:29 | |
*** jecarey has quit IRC | 15:30 | |
*** kylek3h has joined #openstack-nova | 15:30 | |
thorst1 | jaypipes: thx | 15:30 |
*** kiran-r has quit IRC | 15:30 | |
*** rlrossit has quit IRC | 15:30 | |
*** jecarey has joined #openstack-nova | 15:30 | |
*** suresh12 has joined #openstack-nova | 15:30 | |
*** davideagnello has joined #openstack-nova | 15:31 | |
*** rlrossit has joined #openstack-nova | 15:31 | |
*** armax has joined #openstack-nova | 15:32 | |
jaypipes | thorst1: +2 from me. mriedem, please see above request from thorst1. looks like a prettystraightforward fix. | 15:32 |
thorst1 | jaypipes: I cherry picked it down too, just to verify it worked. IPv6 only was fine then. | 15:33 |
*** eharney has joined #openstack-nova | 15:35 | |
*** davideagnello has quit IRC | 15:35 | |
*** afazekas has quit IRC | 15:36 | |
*** otter768 has joined #openstack-nova | 15:36 | |
*** atuvenie_ has joined #openstack-nova | 15:37 | |
*** inara has joined #openstack-nova | 15:37 | |
*** atuvenie has quit IRC | 15:39 | |
*** ijw has joined #openstack-nova | 15:40 | |
*** otter768 has quit IRC | 15:41 | |
*** mpaolino has joined #openstack-nova | 15:41 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Get py34 subunit.run test discovery to work https://review.openstack.org/204723 | 15:43 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 15:43 |
*** ijw_ has joined #openstack-nova | 15:43 | |
bauzas | thorst1: also added to https://etherpad.openstack.org/p/liberty-nova-priorities-tracking but given jaypipes's pitch, not sure it would help more than that :) | 15:44 |
*** rushiagr_away is now known as rushiagr | 15:45 | |
thorst1 | bauzas: TIL. Thank you for directing me to that. | 15:45 |
*** ijw has quit IRC | 15:45 | |
*** ijw has joined #openstack-nova | 15:46 | |
*** scheuran has quit IRC | 15:47 | |
*** ildikov has joined #openstack-nova | 15:48 | |
*** nkrinner has joined #openstack-nova | 15:48 | |
*** ijw_ has quit IRC | 15:49 | |
*** atuvenie has joined #openstack-nova | 15:50 | |
*** baoli has quit IRC | 15:51 | |
*** atuvenie_ has quit IRC | 15:51 | |
*** Mike_D_laptop has joined #openstack-nova | 15:51 | |
*** _kiran_ has quit IRC | 15:52 | |
*** abhijeetm has quit IRC | 15:53 | |
mriedem | at least we don't have to backport the fix | 15:53 |
mriedem | since it was only introduced in liberty | 15:54 |
*** xyang2 has quit IRC | 15:55 | |
mriedem | thorst1: jaypipes: approved | 15:55 |
mriedem | adam_g: thanks for the fix | 15:55 |
thorst1 | mriedem: adam_g: jaypipes: Thx! | 15:55 |
*** unicell has joined #openstack-nova | 15:56 | |
*** thorst1 is now known as thorst1_away | 15:56 | |
*** cpschult has quit IRC | 15:57 | |
*** tonytan4ever has quit IRC | 15:57 | |
mriedem | garyk: did you see my separate comments on https://review.openstack.org/#/c/190047/ and ideas about refactoring _sync_instance_power_state so it's not a mess of conditionals? | 15:57 |
mriedem | maybe better to do a ML thread on that, i can start one | 15:58 |
*** unicell1 has quit IRC | 15:58 | |
n0ano | jaypipes, so are you good with me starting the ML thread on host capabilities? | 15:58 |
garyk | mriedem: i saw them.but they did not make sense. maybe as a follow on we can do the refactoring. | 15:59 |
mriedem | chuckcarmack: here is that thread on sending instance diag notifications http://lists.openstack.org/pipermail/openstack-dev/2015-June/067283.html | 15:59 |
jaypipes | n0ano: sure thing. | 15:59 |
jaypipes | n0ano: that should give me the time I need to write up a spec ;) | 16:00 |
chuckcarmack | mriedem: thanks, reading | 16:00 |
garyk | mriedem: i can build on this patch with the refactoring. at the moment i feel that we have some real issues here and we should try and backport this too. having a refactor makes that even more difficult | 16:00 |
n0ano | jaypipes, thought so but wanted to check | 16:00 |
garyk | mriedem: also the latest update is a few line of code | 16:00 |
*** mgarza_ has joined #openstack-nova | 16:00 | |
mriedem | garyk: so if the admin is concerned about that task shutting down his instances, why doesn't he just disable the task? | 16:00 |
mriedem | garyk: ^ which is why i think this is low priority | 16:01 |
mriedem | but added gorp | 16:01 |
*** vishwanathj has joined #openstack-nova | 16:01 | |
n0ano | also, did you see that lxsli volunteer to do the work on select migration destination? | 16:01 |
garyk | mriedem: i think that it is importnat to know when there is an inconsistency. in addition to this the task does other things | 16:01 |
garyk | but shutting down the instances is just catostrophic in certain cases | 16:02 |
garyk | mriedem: when you configure a firewall for example you can define the action as drop, notify etc. the notify help you understand that there is an issue or was one | 16:02 |
garyk | so here the admin can investigate without having someone call and shout at them thatthe CI is down for example | 16:02 |
mriedem | garyk: would implementing the lifecycle callback for the vmware driver help in this case? | 16:03 |
garyk | i need to investigate that. i am not sure that it will address this case. having said that i need to understand the lifecycle callback better | 16:04 |
garyk | mriedem: that is in heat no? is it in nova? | 16:05 |
mriedem | it's in nova | 16:05 |
mriedem | implemented by libvirt and hyperv (i thought by now) drivers | 16:05 |
*** fawadkhaliq has joined #openstack-nova | 16:05 | |
garyk | can you please point me to an example | 16:05 |
mriedem | and you can disable that also | 16:05 |
*** mpaolino has quit IRC | 16:05 | |
mriedem | see the handle_lifecycle_event method in the compute manager | 16:06 |
rlrossit | dansmith: mriedem: *sigh*.... no dice on bringing in that o.vo change: http://logs.openstack.org/30/198730/14/check/gate-grenade-dsvm-partial-ncpu/a29229b/logs/old/screen-n-cpu.txt.gz?level=INFO#_2015-07-27_15_32_10_701 | 16:06 |
bauzas | jaypipes: oooooh, I see your technical problem about ABC and is_negative | 16:06 |
bauzas | jaypipes: lemme bet on it, that's because is_negative is a property, right? | 16:06 |
*** ijw has quit IRC | 16:07 | |
jaypipes | bauzas: yes. | 16:07 |
jaypipes | bauzas: but honestly, I don't feel strongly about the ABC stuff. | 16:08 |
garyk | mriedem: problem is that the lifecycle events call _sync_instance_power_state which is where we are at the moment… | 16:08 |
bauzas | jaypipes: did you try with https://docs.python.org/2/library/abc.html and six ? | 16:08 |
lxsli | it turns out it works fine | 16:08 |
jaypipes | bauzas: and neither does dansmith which is why it looks the way it does. | 16:08 |
lxsli | if you have a property on A(object) and B(A), the one on B overrides correctly | 16:08 |
mriedem | garyk: but if the instance is down, nova should reflect that right? | 16:08 |
mriedem | i.e. vm_state shouldn't be ACTIVE if the hypervisor is down | 16:09 |
bauzas | jaypipes: fair point, then if you don't feel that strong... :) | 16:09 |
garyk | mriedem: in the vmware/hyperv case it is different. | 16:09 |
lxsli | bauzas: re: validation in __init__, I'm thinking I'll provide @property setters which check self.frozen and raise if set | 16:09 |
garyk | mriedem: say for example you have a host in the cluster that goes down and HA moves that VM to another host. legitimate? | 16:10 |
*** ferest has joined #openstack-nova | 16:10 | |
*** mpaolino has joined #openstack-nova | 16:10 | |
*** ferest has quit IRC | 16:10 | |
garyk | in that case the instance will eventually be up but nova decides that it is down and shuts it down | 16:10 |
jaypipes | bauzas, lxsli: right, like I said, if you guys wanted to push a revision that adds the changes you are talking about, feel free. | 16:10 |
garyk | it can also happen in libvirt if the admin does fencing | 16:10 |
lxsli | bauzas: I'll try not to overengineer as it sounds like that'll go in o.vo 3.0 but it may as well be neat until then | 16:10 |
lxsli | jaypipes: yes just going through the rest of bauzas' comments then I'll push something | 16:11 |
bauzas | lxsli: why aren't you coercing using a new field type ? | 16:11 |
mriedem | garyk: HA via nova's rebuild/evacuate API? | 16:11 |
mriedem | or something completely isolated to vcenter? | 16:11 |
lxsli | bauzas: I don't like having a new field type for every little thing | 16:11 |
*** ferest has joined #openstack-nova | 16:11 | |
dansmith | rlrossit: are you sure that the old code is running your change? | 16:11 |
dansmith | rlrossit: not sure how it could be requesting 1.16 if it is... | 16:11 |
bauzas | lxsli: and what if that object becomes remotable ? :) | 16:12 |
*** ferest has quit IRC | 16:12 | |
lxsli | bauzas: I don't know the implications of that | 16:12 |
garyk | mriedem: i think that with those there could also be edge cases when the instance could be shutdown | 16:12 |
dansmith | bauzas: +1 | 16:12 |
bauzas | lxsli: objects are passed over the wire by primitiving/deprimitiving | 16:12 |
*** ferest has joined #openstack-nova | 16:13 | |
*** alex_klimov has quit IRC | 16:13 | |
*** ferest has quit IRC | 16:13 | |
mriedem | garyk: that's what this check is for at the beginning of the method if self.host != db_instance.host: | 16:14 |
*** jlanoux has quit IRC | 16:14 | |
bauzas | lxsli: so here is the thing | 16:14 |
bauzas | lxsli: the compute node will build a Resource object | 16:14 |
*** ferest has joined #openstack-nova | 16:14 | |
garyk | mriedem: in the case where there are nodes active this issue is more prevalent. | 16:14 |
*** gszasz has quit IRC | 16:14 | |
garyk | mriedem: so it could happen with ironic | 16:15 |
mriedem | which issue? | 16:15 |
*** lucas__ has quit IRC | 16:15 | |
mriedem | the original in the bug? | 16:15 |
garyk | mriedem: that nova thinks that the instance is shutdown but it is actually up | 16:15 |
garyk | yes | 16:15 |
bauzas | lxsli: once the object will be created, the RT will populate it and convert it back to DB entries by calling its save() method | 16:15 |
mriedem | garyk: but nova is asking the virt driver what the power_state is | 16:15 |
mriedem | which goes to the hypervisor | 16:15 |
*** n0ano has quit IRC | 16:15 | |
bauzas | lxsli: once the other hand, the scheduler will populate the corresponding Resource object given what it takes from the DB, right? | 16:16 |
openstackgerrit | Laura Moore proposed openstack/nova: Update port binding information for ironic ports https://review.openstack.org/206135 | 16:16 |
*** ferest has quit IRC | 16:16 | |
bauzas | s/once/on | 16:16 |
garyk | mriedem: it goes like this. the first time nova asks it gets that the instance is down. it then updates the state. then it asks again at the next iteration and the instance is up. then nova shuts it down. | 16:16 |
*** romainh has left #openstack-nova | 16:16 | |
lxsli | bauzas: OK, none of this suggests any problem | 16:17 |
*** ferest has joined #openstack-nova | 16:17 | |
garyk | same can happen with libvirt is say for example the libvirtd process may be restarted when the call is made | 16:17 |
lxsli | bauzas: otoh it looks trivial to add Positive{Integer,Float}Field so I don't mind | 16:17 |
*** ferest has quit IRC | 16:17 | |
garyk | mriedem: having nova shut down instances is what causes the problem. hence the option of just getting the warning | 16:17 |
garyk | maybe i am not explanaing myself well enough | 16:17 |
lxsli | bauzas: especially as dansmith agrees | 16:17 |
*** ijw has joined #openstack-nova | 16:17 | |
bauzas | lxsli: okay, I defer that decision to you | 16:18 |
*** Nikolay_St has quit IRC | 16:18 | |
*** atuvenie has quit IRC | 16:18 | |
bauzas | ;Ã | 16:18 |
bauzas | :) | 16:18 |
*** ferest has joined #openstack-nova | 16:18 | |
lxsli | I seem to have found work to do | 16:18 |
*** mragupat has joined #openstack-nova | 16:18 | |
mriedem | "then it asks again at the next iteration and the instance is up. then nova shuts it down." | 16:19 |
*** cpschult has joined #openstack-nova | 16:19 | |
mriedem | garyk: in that case ^ | 16:19 |
*** ferest has quit IRC | 16:19 | |
rlrossit | dansmith: according to jenkins they got pulled in: https://jenkins03.openstack.org/job/gate-grenade-dsvm-partial-ncpu/730/ | 16:19 |
mriedem | the vm_state should be STOPPED from the previous stop API call, and if the virt driver says the power_state is running nova wouldn't do anything, | 16:19 |
mriedem | i guess the user would have to call the start or reboot API in that case | 16:19 |
*** ferest has joined #openstack-nova | 16:19 | |
*** ferest has quit IRC | 16:19 | |
*** ferest has joined #openstack-nova | 16:20 | |
garyk | mriedem: i have reproduced this endless times. do you want me to show you | 16:20 |
garyk | mriedem: it also happened in production with our CI which was down for a few days | 16:21 |
*** ferest has quit IRC | 16:21 | |
*** thangp_ has quit IRC | 16:21 | |
garyk | the fact that nova decide to shutdown the instance is problematic | 16:21 |
*** fawadkhaliq has quit IRC | 16:21 | |
*** unicell has quit IRC | 16:21 | |
*** ferest has joined #openstack-nova | 16:21 | |
mriedem | garyk: so disable the task | 16:22 |
mriedem | garyk: it doesn't do anything else besides log things and stop instances | 16:22 |
*** ferest has quit IRC | 16:22 | |
garyk | mriedem: but disable it is problematic as states get upated | 16:23 |
mriedem | huh? | 16:23 |
mriedem | where? | 16:23 |
garyk | what is wrong with what is proposed? | 16:23 |
*** ferest has joined #openstack-nova | 16:23 | |
garyk | give me a few minues and i will trace the flow | 16:23 |
mriedem | garyk: i think it's adding unnecessary complexity to an already complex spaghetti mess and a new config option all for something that if you don't want it to do it's thing, you can just disable it | 16:23 |
mriedem | garyk: also, isn't the nsx ci running kvm? | 16:24 |
*** salv-orlando has joined #openstack-nova | 16:24 | |
garyk | mriedem: an if for a log message over a instance shutdown is not adding complexity. | 16:24 |
garyk | that is offering an admin a tool to understand what is going on | 16:24 |
garyk | mriedem: it is running a hybrid env with kvm and vsphere | 16:24 |
*** n0ano has joined #openstack-nova | 16:25 | |
garyk | mriedem: the NSx CI is run on an instance which was running on the vshere backend | 16:25 |
*** patrickeast has joined #openstack-nova | 16:25 | |
mriedem | garyk: so if you have a detailed scenario, it'd be helpful to have that in the commit message | 16:25 |
*** ferest has quit IRC | 16:25 | |
mriedem | rather than just 'this is bad, let's make it loggable instead' | 16:26 |
garyk | mriedem: i will add those. | 16:26 |
openstackgerrit | Chuck Carmack proposed openstack/nova: Add documentation for nova-cells and nova-idmapshift commands https://review.openstack.org/206139 | 16:26 |
mriedem | i understand this task kind of blows, i've seen several bugs around it wrt reboot via the lifecycle callback | 16:26 |
*** salv-orl_ has joined #openstack-nova | 16:26 | |
*** ferest has joined #openstack-nova | 16:26 | |
*** ferest has quit IRC | 16:26 | |
mriedem | and the docstring in the power_state module is basically a lie b/c this task exists | 16:26 |
garyk | mriedem: you added a log message to say that the task is disabled - why is that of interst to an admin. cause the task does something that is important | 16:27 |
garyk | so actually showing what the difefernces are is very important | 16:27 |
garyk | it will also help us debug and troubleshoot | 16:27 |
*** vilobhmm1 has quit IRC | 16:27 | |
*** neiljerram has joined #openstack-nova | 16:27 | |
mriedem | i added an option to disable the lifecycle handler callback | 16:28 |
*** ferest has joined #openstack-nova | 16:28 | |
mriedem | just like how you can disable the periodic task | 16:28 |
garyk | tomorrow i will update the commit message and i hope that will provide enough information | 16:28 |
*** cpschult has quit IRC | 16:28 | |
*** ferest has quit IRC | 16:28 | |
garyk | but we are not disabling the task | 16:28 |
mriedem | yes i realize | 16:28 |
*** ijw has quit IRC | 16:28 | |
garyk | i have to run off to a meeting. i'll update the commit messag with more details | 16:29 |
*** salv-orlando has quit IRC | 16:29 | |
*** ferest has joined #openstack-nova | 16:29 | |
*** armax has quit IRC | 16:30 | |
*** ferest has quit IRC | 16:30 | |
*** suresh12 has quit IRC | 16:30 | |
*** ijw has joined #openstack-nova | 16:30 | |
*** ferest has joined #openstack-nova | 16:31 | |
*** tonytan4ever has joined #openstack-nova | 16:31 | |
*** ferest has quit IRC | 16:32 | |
*** sajeesh has joined #openstack-nova | 16:32 | |
*** ferest has joined #openstack-nova | 16:32 | |
*** nic has joined #openstack-nova | 16:33 | |
chuckcarmack | apmelton: I just uploaded a new rst file for nova-idmapshift. Could you review it? https://review.openstack.org/#/c/206139/ | 16:33 |
jaypipes | dansmith, mriedem: would you have time to check out https://review.openstack.org/#/c/188789/ ? | 16:33 |
*** kiran-r has joined #openstack-nova | 16:34 | |
*** mpaolino has quit IRC | 16:34 | |
*** ferest has quit IRC | 16:34 | |
*** jasondotstar has quit IRC | 16:34 | |
mriedem | jaypipes: did you want to address johnthetubaguy's comments? | 16:34 |
mriedem | s/address/reply/ | 16:34 |
*** _kiran_ has joined #openstack-nova | 16:35 | |
*** ferest has joined #openstack-nova | 16:35 | |
*** ferest has quit IRC | 16:35 | |
jaypipes | mriedem: I believe danpb already did. | 16:35 |
*** moshele has joined #openstack-nova | 16:36 | |
jaypipes | mriedem: oh, sorry, seems I missed a spot. | 16:36 |
*** tonytan4ever has quit IRC | 16:36 | |
mriedem | yar | 16:36 |
jaypipes | mriedem: k, gimme a bit. | 16:36 |
jaypipes | sorry bout that | 16:36 |
*** afazekas has joined #openstack-nova | 16:36 | |
mriedem | quickest review ever | 16:36 |
*** ferest has joined #openstack-nova | 16:36 | |
*** ferest has quit IRC | 16:37 | |
*** ferest has joined #openstack-nova | 16:38 | |
*** lucas__ has joined #openstack-nova | 16:38 | |
*** ferest has quit IRC | 16:38 | |
*** raph_ has quit IRC | 16:38 | |
*** lucas__ has quit IRC | 16:39 | |
*** kiran-r has quit IRC | 16:39 | |
*** lucas__ has joined #openstack-nova | 16:39 | |
*** thangp has joined #openstack-nova | 16:39 | |
*** ferest has joined #openstack-nova | 16:39 | |
*** jasondotstar has joined #openstack-nova | 16:40 | |
*** tonytan4ever has joined #openstack-nova | 16:41 | |
chuckcarmack | All: for anyone who understands how the nova-idmapshift command works, I would appreciate of a review of https://review.openstack.org/#/c/206139/. | 16:41 |
openstackgerrit | Jay Pipes proposed openstack/nova: ironic: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189362 | 16:42 |
openstackgerrit | Jay Pipes proposed openstack/nova: objects: convert config drive to use ImageMeta object https://review.openstack.org/189323 | 16:42 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188119 | 16:42 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: make instance compulsory in blockinfo APIs https://review.openstack.org/188118 | 16:42 |
openstackgerrit | Jay Pipes proposed openstack/nova: hyperv: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189302 | 16:42 |
openstackgerrit | Jay Pipes proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 16:42 |
openstackgerrit | Jay Pipes proposed openstack/nova: virt: convert hardware module to use nova.objects.ImageMeta https://review.openstack.org/188789 | 16:42 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 16:42 |
openstackgerrit | Jay Pipes proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 16:42 |
openstackgerrit | Jay Pipes proposed openstack/nova: xenapi: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188790 | 16:42 |
*** ferest has quit IRC | 16:42 | |
jaypipes | mriedem: k, done. | 16:42 |
*** dave-mccowan has quit IRC | 16:42 | |
*** armax has joined #openstack-nova | 16:43 | |
*** ferest has joined #openstack-nova | 16:43 | |
*** kmARC has quit IRC | 16:43 | |
mriedem | chuckcarmack: so things get reviewed as they get reviewed, you don't have to ask for reviews on a change that you just recently put upt | 16:43 |
mriedem | *up | 16:43 |
*** ferest has quit IRC | 16:43 | |
chuckcarmack | mriedem: I was trying to get the author of the command to review it | 16:44 |
mriedem | chuckcarmack: for all we know he may not even work on openstack anymore | 16:44 |
chuckcarmack | ok | 16:44 |
chuckcarmack | apmelton | 16:44 |
mriedem | chuckcarmack: also i'd split those into separate changes | 16:45 |
*** ferest has joined #openstack-nova | 16:45 | |
mriedem | the people that konw the cells stuff and can review that maybe different from idmapshift | 16:45 |
chuckcarmack | ok | 16:46 |
*** ferest has quit IRC | 16:46 | |
*** ferest has joined #openstack-nova | 16:46 | |
*** ferest has quit IRC | 16:46 | |
*** mpaolino has joined #openstack-nova | 16:46 | |
jaypipes | mriedem: meh, I need to rebase to master and fix conflicts from the fileutils merge... gimme a few. sorry | 16:46 |
mriedem | jaypipes: no rush, i'm in the middle of a big series rebase myself | 16:47 |
mriedem | plus annoyed with other things atm | 16:47 |
mriedem | i think a gdi is in order | 16:47 |
*** ferest has joined #openstack-nova | 16:48 | |
*** unicell has joined #openstack-nova | 16:48 | |
jaypipes | heh | 16:48 |
*** jwcroppe has joined #openstack-nova | 16:49 | |
*** ferest has quit IRC | 16:49 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtISCSIVolumeDriver into it's own module https://review.openstack.org/202301 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtISERVolumeDriver into it's own module https://review.openstack.org/197311 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move volume/remotefs/quobyte modules under volume subdir https://review.openstack.org/193223 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtNETVolumeDriver into it's own module https://review.openstack.org/197341 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtNFSVolumeDriver into it's own module https://review.openstack.org/196859 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtScalityVolumeDriver into it's own module https://review.openstack.org/193249 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move lvm/dmcrypt/rbd_utils modules under storage subdir https://review.openstack.org/193195 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtAOEVolumeDriver into it's own module https://review.openstack.org/196276 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtQuobyteVolumeDriver into the quobyte module https://review.openstack.org/193236 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert SMBFS vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202779 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert NFS volume driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202778 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtFibreChannelVolumeDriver into it's own module https://review.openstack.org/193278 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtGlusterfsVolumeDriver into it's own module https://review.openstack.org/193380 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Introduce LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202777 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Scality vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202783 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move fibre channel specific utilities into fc module https://review.openstack.org/193317 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Quobyte driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202782 | 16:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtGPFSVolumeDriver into it's own module https://review.openstack.org/193240 | 16:49 |
*** cpschult has joined #openstack-nova | 16:49 | |
lxsli | dansmith: how'd you feel about adding __hash__ to NovaObject, which returns the result of hash() called with a tuple of all the field values? | 16:50 |
*** ferest has joined #openstack-nova | 16:50 | |
*** markus_z has quit IRC | 16:50 | |
*** ferest has quit IRC | 16:50 | |
*** tjones1 has joined #openstack-nova | 16:51 | |
lxsli | dansmith: maybe not as __hash__ but via a helper method to make it easy for individual objects to provide __hash__ if they want that behaviour? | 16:51 |
*** ferest has joined #openstack-nova | 16:52 | |
*** gus has quit IRC | 16:52 | |
rlrossit | lxsli: what did you have in mind for the hash (just out of curiosity)? | 16:52 |
*** mpaolino has quit IRC | 16:52 | |
rlrossit | like uses for it | 16:52 |
lxsli | rlrossit: it's used when you put things in dicts and sets | 16:52 |
*** ferest has quit IRC | 16:52 | |
*** moshele has quit IRC | 16:53 | |
*** ferest has joined #openstack-nova | 16:53 | |
*** nkrinner has quit IRC | 16:54 | |
*** gus has joined #openstack-nova | 16:54 | |
*** ferest has quit IRC | 16:54 | |
*** SourabhP has joined #openstack-nova | 16:54 | |
*** jistr has quit IRC | 16:55 | |
*** SourabhP_ has joined #openstack-nova | 16:55 | |
*** ferest has joined #openstack-nova | 16:55 | |
*** _kiran_ has quit IRC | 16:55 | |
*** ferest has quit IRC | 16:55 | |
*** atuvenie has joined #openstack-nova | 16:56 | |
*** meghal has joined #openstack-nova | 16:56 | |
*** ferest has joined #openstack-nova | 16:57 | |
*** ferest has quit IRC | 16:57 | |
lxsli | rlrossit: http://paste.openstack.org/show/405941/ | 16:57 |
*** afazekas has quit IRC | 16:58 | |
*** achanda has joined #openstack-nova | 16:58 | |
openstackgerrit | Anthony Lee proposed openstack/nova: Fix live-migrations usage of the wrong connector information https://review.openstack.org/202770 | 16:58 |
*** ferest has joined #openstack-nova | 16:58 | |
*** ferest has quit IRC | 16:58 | |
lxsli | mostly I'm just a stickler for statements like "if you implement __eq__, you should also implement __hash__" | 16:58 |
*** annashen has joined #openstack-nova | 16:59 | |
*** SourabhP has quit IRC | 16:59 | |
*** SourabhP_ is now known as SourabhP | 16:59 | |
*** ferest has joined #openstack-nova | 17:00 | |
*** ferest has quit IRC | 17:00 | |
*** afazekas has joined #openstack-nova | 17:00 | |
*** neiljerram has quit IRC | 17:01 | |
*** ferest has joined #openstack-nova | 17:01 | |
*** ferest has quit IRC | 17:02 | |
*** jordanP has quit IRC | 17:02 | |
*** ferest has joined #openstack-nova | 17:03 | |
cfriesen | danpb: ndipanov: the spec at http://specs.openstack.org/openstack/nova-specs/specs/juno/approved/virt-driver-cpu-pinning.html mentions the "hw:cpu_threads_policy" option for controlling hyperthreading. That doesn't seem to have been implemented...was there a design decision to cancel it, did you just run out of time, or something else? Are there any plans to add it back in? | 17:03 |
ndipanov | cfriesen, yeah I ran out of time | 17:04 |
*** ferest has quit IRC | 17:04 | |
ndipanov | there is a bp to fix it | 17:04 |
ndipanov | and it even has code up | 17:04 |
lxsli | dansmith: nevermind, objects which provide __hash__ need to be immutable and there are few enough of those | 17:04 |
ndipanov | but mriedem blocked it on the grounds of being late for the FPF | 17:04 |
*** ferest has joined #openstack-nova | 17:04 | |
cfriesen | ndipanov: got a link? | 17:05 |
ndipanov | https://review.openstack.org/#/c/202647/ | 17:05 |
cfriesen | save searching time | 17:05 |
*** ferest has quit IRC | 17:05 | |
cfriesen | ndipanov: cool...will take a look. | 17:05 |
*** coolsvap is now known as coolsvap|away | 17:06 | |
*** ferest has joined #openstack-nova | 17:06 | |
openstackgerrit | Jay Pipes proposed openstack/nova: ironic: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189362 | 17:07 |
*** ferest has quit IRC | 17:07 | |
openstackgerrit | Jay Pipes proposed openstack/nova: objects: convert config drive to use ImageMeta object https://review.openstack.org/189323 | 17:07 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188119 | 17:07 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: make instance compulsory in blockinfo APIs https://review.openstack.org/188118 | 17:07 |
openstackgerrit | Jay Pipes proposed openstack/nova: hyperv: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189302 | 17:07 |
openstackgerrit | Jay Pipes proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 17:07 |
openstackgerrit | Jay Pipes proposed openstack/nova: virt: convert hardware module to use nova.objects.ImageMeta https://review.openstack.org/188789 | 17:07 |
johnthetubaguy | cfriesen: ndipanov: yeah I unapproved that blueprint for the non-priority FPF, I just caught up with the folks doing that this morning | 17:07 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 17:07 |
openstackgerrit | Jay Pipes proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 17:07 |
openstackgerrit | Jay Pipes proposed openstack/nova: xenapi: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188790 | 17:07 |
* jaypipes chuckles at the IRC storm. | 17:07 | |
*** BobBall is now known as BobBall_AWOL | 17:07 | |
*** SourabhP has quit IRC | 17:07 | |
ndipanov | johnthetubaguy, and? | 17:08 |
*** ferest has joined #openstack-nova | 17:08 | |
ndipanov | imho posting patches on the 16th for the FF that is on the 16th is ok | 17:08 |
*** thorst1_away is now known as thorst1 | 17:08 | |
ndipanov | but that's just me | 17:08 |
mriedem | ndipanov: maybe, however, it sounded like in the spec review that this was approved for kilo, so why did it take until the due date for FPF to get code up? | 17:09 |
*** ferest has quit IRC | 17:09 | |
*** dave-mccowan has joined #openstack-nova | 17:09 | |
ndipanov | mriedem, lol | 17:09 |
ndipanov | I don't know I didn't write the code | 17:09 |
mriedem | i know, just saying | 17:09 |
ndipanov | and I'm glad I didn't cuz you sound scary | 17:09 |
*** ferest has joined #openstack-nova | 17:09 | |
ndipanov | :) | 17:09 |
mriedem | no, it's just frustrating | 17:10 |
mriedem | all around | 17:10 |
ndipanov | yeah I know - tbh I didn't look at the code yet | 17:10 |
ndipanov | but seems to be adding fields etc. | 17:10 |
*** ferest has quit IRC | 17:10 | |
ndipanov | not something that is just likely to land in a day or 2 | 17:11 |
mriedem | i just saw NFV-y type things which makes me ew | 17:11 |
mriedem | right | 17:11 |
*** ferest has joined #openstack-nova | 17:11 | |
*** ferest has quit IRC | 17:12 | |
johnthetubaguy | ndipanov: agreed its crazy, but we have loads (probably like 50) BPs that met the deadline, so it was unfair to let them through | 17:12 |
ndipanov | well I try to keep that kind of bias out of my -2s ... | 17:12 |
*** SourabhP has joined #openstack-nova | 17:13 | |
johnthetubaguy | ndipanov: I did ask them to ping me if they got it up by the end of their day, and I would let them through, but they only reached out to me this morning | 17:13 |
*** ferest has joined #openstack-nova | 17:13 | |
johnthetubaguy | ndipanov: the -2 was purely procedural right? it was an unapproved blueprint at that point | 17:13 |
mriedem | ndipanov: it was a policy -2 | 17:13 |
johnthetubaguy | ndipanov: the late ping was a miss communication between us both, I think they assumed I would have seen the patches, and I didn't see them, as it wasn't on my radar any more | 17:14 |
ndipanov | well I really haven't looked at the code, and from the little I kinda have, it's probably not easy to land | 17:14 |
*** sahid has quit IRC | 17:15 | |
*** ferest has quit IRC | 17:15 | |
*** Nikolay_St has joined #openstack-nova | 17:16 | |
openstackgerrit | Ming Yang proposed openstack/nova: Case-insensitive filters https://review.openstack.org/201725 | 17:16 |
johnthetubaguy | ndipanov: for a singe (small) patchset BP, I would be OK bending the rules as its clearly useful, but there like way over 5 patch sets (i.e. I didn't count the number), so it just seemed unfair to all the other blueprint authors to bend the rules for that one :( | 17:16 |
*** ferest has joined #openstack-nova | 17:17 | |
ndipanov | johnthetubaguy, yes agreed... | 17:17 |
*** ferest has quit IRC | 17:17 | |
meghal | hello all - I submitted a spec yesterday night, for allowing creation of dynamic quota resources…so that cloud-admin gets the capability of assigning quota for resources like flavor, az or any random resource that a cloud-admin wants to restrict total number of instances on. Link - https://review.openstack.org/#/c/205906/ | 17:18 |
*** davideagnello has joined #openstack-nova | 17:18 | |
meghal | wanted to get feedback and see what you all think | 17:18 |
*** davideagnello has joined #openstack-nova | 17:18 | |
*** ferest has joined #openstack-nova | 17:18 | |
mriedem | meghal: move that to the backlog or line up behind the mitaka change | 17:18 |
mriedem | https://review.openstack.org/#/c/205280/ | 17:18 |
*** whenry has joined #openstack-nova | 17:19 | |
*** ferest has quit IRC | 17:19 | |
meghal | mriedem: should I move it here then - https://github.com/openstack/nova-specs/tree/master/specs/backlog/approved ? | 17:19 |
*** ferest has joined #openstack-nova | 17:19 | |
*** ferest has quit IRC | 17:19 | |
mriedem | meghal: i'd make it depend on https://review.openstack.org/#/c/205280/ which opens specs for mitaka | 17:20 |
mriedem | alaski: johnthetubaguy: dansmith: danpb: ^ we should get that landed | 17:20 |
mriedem | so people can move stuff there | 17:20 |
*** vilobhmm has joined #openstack-nova | 17:20 | |
meghal | mriedem: will do that. Thanks | 17:21 |
*** Sukhdev has quit IRC | 17:21 | |
johnthetubaguy | mriedem: yeah, let me take a peak | 17:21 |
ildikov | johnthetubaguy: hi. do you have a few seconds? | 17:21 |
*** ferest has joined #openstack-nova | 17:21 | |
*** garyk has quit IRC | 17:21 | |
johnthetubaguy | ildikov: yes | 17:21 |
*** ferest has quit IRC | 17:21 | |
ildikov | johnthetubaguy: do you plan to have a follow up review day or smth like for old patches up for review? | 17:22 |
*** afazekas has quit IRC | 17:22 | |
*** ferest has joined #openstack-nova | 17:22 | |
johnthetubaguy | ildikov: there is a bug related one pencilled in for later in the release | 17:23 |
johnthetubaguy | ildikov: the current ideas are all in the wiki page: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule#Special_review_days | 17:23 |
*** ferest has quit IRC | 17:23 | |
*** mpaolino has joined #openstack-nova | 17:23 | |
johnthetubaguy | ildikov: I am thinking about changing the day given what mriedem was saying about Fridays | 17:24 |
*** ferest has joined #openstack-nova | 17:24 | |
*** ferest has quit IRC | 17:24 | |
mriedem | fridays are not productive review days imo | 17:24 |
mriedem | reviewing blueprints is about the last thing i want to do on a friday | 17:24 |
ildikov | johnthetubaguy: is there anything for non priority BPs too? | 17:24 |
lxsli | +1 | 17:24 |
johnthetubaguy | ildikov: they all have to be merged by tomorrow, that was really what the last review day was for | 17:25 |
ildikov | johnthetubaguy: I know that there was a day for that but it turned out to be a gate fixing day as far as I can remember | 17:25 |
*** danpb has quit IRC | 17:25 | |
johnthetubaguy | mriedem: yeah, maybe I should go for a tuesday? we should bring that up in the nova meeting | 17:25 |
*** ferest has joined #openstack-nova | 17:25 | |
johnthetubaguy | ildikov: yeah, sadly the midcycle got in the way of scheduling something last week, and this week is the deadline, it didn't work out too well | 17:25 |
*** ferest has quit IRC | 17:25 | |
*** xyang2 has joined #openstack-nova | 17:26 | |
*** armax has quit IRC | 17:26 | |
ildikov | johnthetubaguy: is there anything we could do about the soft affinity bp? | 17:26 |
johnthetubaguy | ildikov: although a few folks did do a little push, it didn't work as well as I had hoped in the end | 17:26 |
*** ferest has joined #openstack-nova | 17:26 | |
ildikov | johnthetubaguy: it was up for review before the Kilo deadline :S | 17:26 |
*** ferest has quit IRC | 17:27 | |
johnthetubaguy | ildikov: so right now, there are lots of -1s from Jenkins it seems on those patches, without a jenkins +1 most folks will avoid reviewing those patches | 17:27 |
mriedem | johnthetubaguy: tuesday-thursday is generally good imo | 17:28 |
mriedem | mondays and fridays i don't want to be 'working' :) | 17:28 |
*** ferest has joined #openstack-nova | 17:28 | |
*** SourabhP has quit IRC | 17:28 | |
mriedem | the other 60% of the week i'm all yours | 17:28 |
*** ferest has quit IRC | 17:28 | |
johnthetubaguy | ildikov: sadly, its not the only blueprint in that situation, but thats the best idea I have right now :( | 17:28 |
ildikov | johnthetubaguy: yeap, I need to recheck those, but we didn't really get review comments even with green tests | 17:28 |
johnthetubaguy | mriedem: lol | 17:28 |
ildikov | johnthetubaguy: it was quite on the top of your list of old BPs up for review IIRC | 17:29 |
johnthetubaguy | ildikov: agreed, its a general problem right now, we don't have enough people doing reviews (and I don't mean core reviews, I mean all people) | 17:29 |
edleafe | mriedem: https://www.youtube.com/watch?v=rBJLoYd8xak | 17:29 |
*** ferest has joined #openstack-nova | 17:29 | |
ildikov | johnthetubaguy: any idea how to get his merged in a reasonable timeframe? | 17:29 |
*** ferest has quit IRC | 17:30 | |
openstackgerrit | Meghal Gosalia proposed openstack/nova-specs: Provide support for creating dynamic quota resources https://review.openstack.org/206160 | 17:30 |
*** ferest has joined #openstack-nova | 17:31 | |
*** vishwanathj has quit IRC | 17:32 | |
*** ferest has quit IRC | 17:32 | |
*** ferest has joined #openstack-nova | 17:32 | |
*** armax has joined #openstack-nova | 17:32 | |
johnthetubaguy | ildikov: I don't have a good answer I am afraid, beyond the usual things: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule#How_can_I_get_my_code_merged_faster.3F | 17:33 |
*** yamahata has joined #openstack-nova | 17:34 | |
*** ferest has quit IRC | 17:34 | |
*** eglynn has quit IRC | 17:34 | |
meghal | mriedem: wow…that was a quick merge for opening up mitaka specs. Have added my spec under mitaka now - https://review.openstack.org/#/c/206160/ Is it under the right location now ? | 17:35 |
mriedem | edleafe: yes, turtle necks would be helpful | 17:35 |
ildikov | johnthetubaguy: ok | 17:35 |
*** ferest has joined #openstack-nova | 17:35 | |
ildikov | johnthetubaguy: is the freeze execption for these things? | 17:35 |
ildikov | sdague: hi. are you around? | 17:36 |
mriedem | ildikov: non-priority feature freeze is thursday | 17:36 |
mriedem | meghal: looks fine, you didn't need to drop the old change, just needed to move it | 17:36 |
*** ferest has quit IRC | 17:36 | |
mriedem | but whatever | 17:36 |
*** penick has joined #openstack-nova | 17:36 | |
mriedem | meghal: don't expect anyone to review this for awhile | 17:36 |
*** nelsnelson has quit IRC | 17:36 | |
ildikov | mriedem: yeap, thanks | 17:37 |
openstackgerrit | Laura Moore proposed openstack/nova: Update ironic virt driver to use portgroups https://review.openstack.org/206163 | 17:37 |
ildikov | mriedem: do you have a strategy for how to get reviews maybe? :) | 17:37 |
*** otter768 has joined #openstack-nova | 17:37 | |
*** ferest has joined #openstack-nova | 17:37 | |
mriedem | ildikov: i don't | 17:37 |
*** lpetrut has quit IRC | 17:37 | |
mriedem | do reviews yourself to lighten the load | 17:38 |
*** ferest has quit IRC | 17:38 | |
*** melwitt has joined #openstack-nova | 17:38 | |
*** armax has quit IRC | 17:38 | |
meghal | mriedem: hmm…is it because of the work going on in liberty ? so will folks start looking at it after Liberty is out :( or earlier ? | 17:38 |
*** kmARC has joined #openstack-nova | 17:38 | |
mriedem | meghal: right, everyone is heads down on liberty | 17:38 |
ildikov | mriedem: ok, thanks | 17:39 |
*** ferest has joined #openstack-nova | 17:39 | |
mriedem | meghal: except ops people maybe, they are always good to get reviews on new specs | 17:39 |
mriedem | meghal: which is why i added jpenick | 17:39 |
*** boris-42 has quit IRC | 17:40 | |
openstackgerrit | Alexis Lee proposed openstack/nova: Adds resource classes https://review.openstack.org/128999 | 17:40 |
openstackgerrit | Alexis Lee proposed openstack/nova: Add object models for comparing resource amounts https://review.openstack.org/128992 | 17:40 |
*** ferest has quit IRC | 17:40 | |
*** ferest has joined #openstack-nova | 17:41 | |
*** otter768 has quit IRC | 17:41 | |
meghal | mriedem: thanks. yes…atleast it would be great to get initial feedback on how ops perceive it and what community thinks about the change. | 17:42 |
*** VW_ has quit IRC | 17:42 | |
*** VW_ has joined #openstack-nova | 17:42 | |
*** ferest has quit IRC | 17:42 | |
*** ferest has joined #openstack-nova | 17:43 | |
*** penick has quit IRC | 17:43 | |
*** winston-d has joined #openstack-nova | 17:44 | |
*** ferest has quit IRC | 17:44 | |
lxsli | bauzas: new patchsets up ^^ | 17:44 |
lxsli | g'night | 17:44 |
*** mgarza_ has quit IRC | 17:44 | |
*** tonytan4ever has quit IRC | 17:45 | |
*** ferest has joined #openstack-nova | 17:45 | |
*** ZZelle_ has joined #openstack-nova | 17:45 | |
*** ferest has quit IRC | 17:45 | |
*** baoli has joined #openstack-nova | 17:46 | |
*** ferest has joined #openstack-nova | 17:46 | |
*** VW_ has quit IRC | 17:46 | |
*** penick has joined #openstack-nova | 17:48 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova: Switch to using os-brick https://review.openstack.org/175569 | 17:48 |
*** ferest has quit IRC | 17:48 | |
*** ferest has joined #openstack-nova | 17:49 | |
*** smathews has joined #openstack-nova | 17:49 | |
*** afazekas has joined #openstack-nova | 17:49 | |
*** ferest has quit IRC | 17:49 | |
*** apuimedo has quit IRC | 17:50 | |
*** ferest has joined #openstack-nova | 17:51 | |
*** winston-d is now known as winston-d_zZZ | 17:51 | |
*** ferest has quit IRC | 17:51 | |
*** READ10 has quit IRC | 17:51 | |
*** ferest has joined #openstack-nova | 17:52 | |
*** ferest has quit IRC | 17:52 | |
*** harlowja has joined #openstack-nova | 17:52 | |
*** ferest has joined #openstack-nova | 17:53 | |
*** VW_ has joined #openstack-nova | 17:53 | |
*** tonytan4ever has joined #openstack-nova | 17:53 | |
*** ferest has quit IRC | 17:53 | |
*** ferest has joined #openstack-nova | 17:55 | |
*** yamahata has quit IRC | 17:57 | |
*** ferest has quit IRC | 17:57 | |
*** vishwanathj has joined #openstack-nova | 17:58 | |
*** annegentle has joined #openstack-nova | 17:58 | |
dannywilson | Does anyone know how to check an instance to see what the guest contains? I can get the guest via self._host.get_guest(instance) but I am not sure how to tell what controllers are currently attached. | 17:58 |
*** ferest has joined #openstack-nova | 17:58 | |
dannywilson | I am trying to provide discard/unmap/trim support for cinder attached volumes and having trouble with reusing the correct scsi controller | 17:59 |
dannywilson | ^^ https://review.openstack.org/#/c/205726/ | 17:59 |
*** ferest has quit IRC | 17:59 | |
*** ferest has joined #openstack-nova | 18:00 | |
*** suresh12 has joined #openstack-nova | 18:00 | |
*** pixelbeat_ has quit IRC | 18:01 | |
mriedem | dannywilson: it seems nova.virt.libvirt.volume.disconnect_volume would need to know that in the iscsi case, so i'd look there to see how it's filtering | 18:02 |
*** ferest has quit IRC | 18:02 | |
*** Marga_ has quit IRC | 18:02 | |
ndipanov | dannywilson, it's amost surely a mistake to parse the instance xml | 18:02 |
ndipanov | nova should have all the info already | 18:02 |
ndipanov | if it doesn't it should be added | 18:02 |
rlrossit | mriedem: take a look at my comment on https://review.openstack.org/#/c/200280/4. I might totally be off-base with my concern, but something about this doesn't seem right... | 18:02 |
ndipanov | we have countless bugs around that very thing | 18:02 |
dannywilson | ndipanov: thanks, good to know | 18:02 |
*** ferest has joined #openstack-nova | 18:03 | |
mriedem | rlrossit: yeah, so the compat thing on enum field changes is a question i have also | 18:03 |
mriedem | and hope dansmith has the answer | 18:03 |
mriedem | rlrossit: i think i posted comments on that in ps1 | 18:03 |
rlrossit | *phew* I thought I was going crazy | 18:03 |
mriedem | https://review.openstack.org/#/c/200280/1/nova/objects/image_meta.py | 18:04 |
*** coreywright has joined #openstack-nova | 18:04 | |
*** ferest has quit IRC | 18:04 | |
rlrossit | yep I see them now. I'm trying to figure this out as to how it will work from testing against this too to make sure the version gets bumped | 18:04 |
*** ferest has joined #openstack-nova | 18:05 | |
mriedem | rlrossit: so my thinking was, if i have an ImageMetaProps at version 1.3 and it has hw_cdrom_bus with value 'lxc' and i'm going to version 1.2, that would need to blow up | 18:06 |
mriedem | however, it's weird with enums since they are not versioned themselves | 18:06 |
*** ferest has quit IRC | 18:06 | |
rlrossit | mriedem: let me look at the bug quick, but basically, what is happening now because those fields are missing? | 18:06 |
mriedem | rlrossit: the bug just says these are valid enum values and were missing | 18:07 |
rlrossit | mriedem: if it's blow up terribly now vs. blow up more gracefully with this, then I'm cool | 18:07 |
mriedem | so add them | 18:07 |
*** Marga_ has joined #openstack-nova | 18:07 | |
*** ssurana has joined #openstack-nova | 18:07 | |
*** ferest has joined #openstack-nova | 18:07 | |
rlrossit | so I mean we wouldn't want to blow up because it wouldn't in the past right? | 18:07 |
mriedem | rlrossit: i *think* what would happen is if you have some legacy image properties and try to load it up with this new object model, and one of those fields has lxc or uml, it'll blow up with a ValueError | 18:07 |
mriedem | rlrossit: right | 18:07 |
mriedem | in the past it was a big dict | 18:07 |
mriedem | with whatever you wanted in it | 18:07 |
*** lucasagomes is now known as lucas-dinner | 18:07 | |
*** ferest has quit IRC | 18:07 | |
*** apuimedo has joined #openstack-nova | 18:08 | |
mriedem | so i think my point in ps1 with converting back to < 1.3 was we shouldn't since those were busted during the object conversoin and we don't want to go back there | 18:08 |
*** ferest has joined #openstack-nova | 18:08 | |
mriedem | rlrossit: if that makes sense, i should update the commit message | 18:08 |
*** ferest has quit IRC | 18:08 | |
rlrossit | mriedem: thinking about this.... | 18:09 |
*** baoli has quit IRC | 18:09 | |
kfox1111_ | Just hit another case where a vm wants to manage openstack resources. HA Chef, so that it can move the cinder volume from machine to machine. :/ | 18:09 |
*** ferest has joined #openstack-nova | 18:09 | |
*** ferest has quit IRC | 18:09 | |
kfox1111_ | How do we make forward progress on instance users? | 18:09 |
*** baoli has joined #openstack-nova | 18:10 | |
mriedem | kfox1111_: is this that barbican thing? | 18:10 |
mriedem | that stalled | 18:10 |
mnestratov | mriedem: dansmith: hi, can you please look at https://review.openstack.org/#/c/184311/ - I had to rebase | 18:10 |
mriedem | mnestratov: sure | 18:10 |
*** yamahata has joined #openstack-nova | 18:10 | |
rlrossit | mriedem: if this is just a value in the DB, just like integers, do we care if it's something we have now that we didn't before? (kinda playing devil's advocate) | 18:11 |
*** ferest has joined #openstack-nova | 18:11 | |
*** ferest has quit IRC | 18:11 | |
mriedem | have now that we didn't have before? | 18:11 |
mriedem | we always supported disk_bus of lxc and uml | 18:11 |
mriedem | just never validated disk_bus | 18:11 |
rlrossit | then do we care about compat? | 18:12 |
kfox1111_ | mriedem: related to barbican, but useful for lots of different things. | 18:12 |
*** ferest has joined #openstack-nova | 18:12 | |
mriedem | well, we did validate i guess, down in the libvirt driver | 18:12 |
*** ferest has quit IRC | 18:12 | |
*** changbl has joined #openstack-nova | 18:12 | |
mriedem | rlrossit: so i think that's what i'm saying, i don't think we care about compat | 18:12 |
mriedem | and i need to update the commit message for that | 18:12 |
kfox1111_ | yeah. spec here: https://review.openstack.org/#/c/186617/ | 18:12 |
mriedem | kfox1111_: so you're probably looking at a summit session at this point | 18:12 |
rlrossit | mriedem: ok. I do want dansmith's input on this too also, I'm not the objects expert (yet) | 18:12 |
mriedem | kfox1111_: since it's not going to happen in liberty | 18:13 |
mriedem | rlrossit: let's just merge it and see if he objects later (get it) | 18:13 |
openstackgerrit | Chuck Carmack proposed openstack/nova: Add documentation for the nova-idmapshift command. https://review.openstack.org/206139 | 18:13 |
kfox1111_ | Not sure I'm going to be able to make the summit. :/ | 18:13 |
*** ferest has joined #openstack-nova | 18:13 | |
mriedem | kfox1111_: are there other stakeholders on this spec that will be? | 18:13 |
rlrossit | mriedem: wait... so if we don't care about compat, do we have to bump versions? | 18:13 |
dansmith | mnestratov: I just found an issue in that patch | 18:13 |
jogo | amazon has placement groups: | 18:13 |
kfox1111_ | not sure. :/ | 18:13 |
dansmith | mnestratov: that I didn't think about before.. I'll comment | 18:13 |
jogo | amazon has placement groups: http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/placement-groups.html | 18:13 |
kfox1111_ | there are tons of folks that want it, afte talking with them. | 18:13 |
*** ferest has quit IRC | 18:13 | |
*** salv-orl_ has quit IRC | 18:14 | |
mriedem | jogo: o/ | 18:14 |
mnestratov | dansmith: ok | 18:14 |
kfox1111_ | not too many with the big picture though. :/ | 18:14 |
mriedem | so we could have AZs, server groups and placement groups, oh my | 18:14 |
*** ferest has joined #openstack-nova | 18:14 | |
mriedem | rlrossit: i was told yes to bumping the version on enum changes | 18:15 |
*** mpaolino has quit IRC | 18:15 | |
mriedem | rlrossit: at least so we could do compat checking if needed | 18:15 |
*** ferest has quit IRC | 18:15 | |
*** annegentle has quit IRC | 18:15 | |
kfox1111_ | I guess I'll see if management will send me. this activity seems very difficult without face time. :/ | 18:16 |
*** ferest has joined #openstack-nova | 18:16 | |
mriedem | kfox1111_: yeah, would have been good for the nova meetup that happened last week | 18:16 |
*** annashen has quit IRC | 18:16 | |
mriedem | kfox1111_: where are you located? | 18:16 |
*** ferest has quit IRC | 18:16 | |
kfox1111_ | Tricities Washington. | 18:16 |
mriedem | kfox1111_: so, you should have come to the nova meetup last week in MN :) | 18:17 |
dansmith | mnestratov: commented.. should be straightforward I think.. sorry I didn't catch it before | 18:17 |
kfox1111_ | east side of the state. | 18:17 |
mriedem | cheaper and higher bandwidth for discussions like this | 18:17 |
kfox1111_ | I was out on vacation. :/ | 18:17 |
mriedem | kfox1111_: well that's your 2nd problem then | 18:17 |
*** ferest has joined #openstack-nova | 18:18 | |
dansmith | kfox1111_: you mean washington state? | 18:18 |
jogo | mriedem: o/ | 18:18 |
*** ssurana has left #openstack-nova | 18:18 | |
jogo | mriedem: figured AWS placement groups make an interesting datapoint | 18:18 |
*** ferest has quit IRC | 18:18 | |
*** annashen has joined #openstack-nova | 18:18 | |
kfox1111_ | dansmith: Yeah. | 18:18 |
dansmith | okay | 18:18 |
mriedem | jogo: i figured it was a passing by "i'm going to just mention this riiiiiight.....here" and leave kind of thing :) | 18:19 |
mriedem | stirring the pot | 18:19 |
kfox1111_ | I work for Pacific Northwest National Lab | 18:19 |
mriedem | rlrossit: so this is the same thing we were just talking about i guess https://review.openstack.org/#/c/184311/22/nova/tests/unit/objects/test_objects.py | 18:19 |
*** ferest has joined #openstack-nova | 18:19 | |
*** afazekas has quit IRC | 18:19 | |
dansmith | melwitt: around? | 18:20 |
*** Mike_D_laptop has quit IRC | 18:20 | |
kfox1111_ | Part of the problem with this spec is its involving so many other projects. | 18:20 |
melwitt | dansmith: yes | 18:20 |
kfox1111_ | I'm not sure a nova specific meetup would have worked anyway. | 18:20 |
*** ferest has quit IRC | 18:20 | |
dansmith | melwitt: so now that I can think straight I was looking at this: https://review.openstack.org/#/c/197770/5/nova/tests/unit/api/openstack/compute/test_servers.py,cm | 18:21 |
dansmith | melwitt: specifically the test_tenant_id_filter_converts .. () method | 18:21 |
rlrossit | mriedem: yep. So how did you not get the need for a hash change? | 18:21 |
rlrossit | do we not hash all enums? | 18:21 |
jogo | mriedem: yup :) | 18:21 |
dansmith | melwitt: so the reason I removed that is twofold: (1) it is checking behavior that is fully within compute/api and not at all related to servers.py and (b) I would have to un-stub compute/api::get_all() in order to run the code it's trying to test | 18:22 |
*** ferest has joined #openstack-nova | 18:22 | |
*** ferest has quit IRC | 18:22 | |
*** ferest has joined #openstack-nova | 18:23 | |
dansmith | melwitt: so I'm going to try to write an equivalent test in test_compute_api, which is where it should be anyway.. you okay with that? | 18:23 |
*** ferest has quit IRC | 18:23 | |
*** ferest has joined #openstack-nova | 18:24 | |
*** yamahata has quit IRC | 18:25 | |
*** jasondot_ has joined #openstack-nova | 18:25 | |
*** ferest has quit IRC | 18:25 | |
*** yamahata has joined #openstack-nova | 18:25 | |
mriedem | rlrossit: the hash did change for ImageMetaProps which is what's using the DiskBusField (enum) | 18:25 |
*** fawadkhaliq has joined #openstack-nova | 18:25 | |
mnestratov | dansmith: that's ok. by the way when another hypervisors were added recently they didn't bump the version (Docker and IRONIC for instance). I just followed this way. | 18:25 |
mriedem | rlrossit: the hash didn't change for ImageMeta since none of it's fields changed | 18:25 |
dansmith | mnestratov: those were added before it was embedded in the object I think | 18:25 |
mriedem | right | 18:26 |
dansmith | mnestratov: and if not, then shame on them, yes :D | 18:26 |
*** ferest has joined #openstack-nova | 18:26 | |
*** apuimedo has quit IRC | 18:26 | |
openstackgerrit | Sudipta Biswas proposed openstack/nova: Allow NUMA based reporting for Monitors https://review.openstack.org/198972 | 18:26 |
rlrossit | mriedem: ok yeah that's what threw me off. I was thinking ImageMeta needed to have its hash updated too but I got it now | 18:26 |
*** ferest has quit IRC | 18:26 | |
rlrossit | doing too many objects-related reviews that everything is starting to blend together | 18:27 |
*** annegentle has joined #openstack-nova | 18:27 | |
melwitt | dansmith: I think I see your point. I was thinking the test would become a stubbing out of get_all and the verification would be the query params are correctly in search_opts | 18:27 |
rlrossit | I'm just avoiding going back to look at my infinite recursion because I just get a headache thinking about it... | 18:27 |
*** jasondotstar has quit IRC | 18:27 | |
*** ferest has joined #openstack-nova | 18:28 | |
mnestratov | dansmith: indeed | 18:28 |
openstackgerrit | Nicolas Simonds proposed openstack/nova-specs: RBD instance snapshots https://review.openstack.org/188244 | 18:28 |
melwitt | dansmith: but that's without taking a step back to understand what the test is trying to test. I don't always understand the api tests | 18:28 |
*** kmARC has quit IRC | 18:28 | |
*** ferest has quit IRC | 18:28 | |
openstackgerrit | Chuck Carmack proposed openstack/nova: Add documentation for the nova-idmapshift command. https://review.openstack.org/206179 | 18:28 |
dansmith | melwitt: right, but the problem is, the thing it's testing isn't against the get_all call, it's against what get_all does | 18:28 |
dansmith | melwitt: it looks like the test would test that servers.py calls get_all() in a certain way, but it's actually testing that get_all() calls the DB in a certain way | 18:29 |
dansmith | since I moved this all to mocking out the compute_api layer instead of the DB layer, that's no longer a thing | 18:29 |
openstackgerrit | Chuck Carmack proposed openstack/nova: Add documentation for the nova-cells command. https://review.openstack.org/206179 | 18:29 |
*** ferest has joined #openstack-nova | 18:29 | |
openstackgerrit | Cale Rath proposed openstack/nova: Purge soft-deleted instances cmd https://review.openstack.org/203751 | 18:29 |
*** ferest has quit IRC | 18:31 | |
*** casusbelli has joined #openstack-nova | 18:31 | |
*** ferest has joined #openstack-nova | 18:32 | |
*** karimb has quit IRC | 18:32 | |
melwitt | dansmith: yeah. okay. I see it now | 18:33 |
melwitt | dansmith: in get_all is where it converts filter "tenant_id" to "project_id" | 18:34 |
*** ferest has quit IRC | 18:34 | |
dansmith | melwitt: yeah | 18:34 |
openstackgerrit | Vilobh Meshram proposed openstack/nova-specs: Quota Management for Dynamic Quota Resources https://review.openstack.org/206160 | 18:34 |
*** ferest has joined #openstack-nova | 18:35 | |
*** ferest has quit IRC | 18:36 | |
*** ferest has joined #openstack-nova | 18:37 | |
*** nelsnelson has joined #openstack-nova | 18:39 | |
*** ferest has quit IRC | 18:39 | |
openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 18:39 |
*** ferest has joined #openstack-nova | 18:40 | |
*** ferest has quit IRC | 18:40 | |
*** liusheng has quit IRC | 18:41 | |
*** ferest has joined #openstack-nova | 18:41 | |
*** ferest has quit IRC | 18:41 | |
*** liusheng has joined #openstack-nova | 18:41 | |
*** boris-42 has joined #openstack-nova | 18:42 | |
*** agireud has joined #openstack-nova | 18:42 | |
*** ferest has joined #openstack-nova | 18:43 | |
*** ferest has quit IRC | 18:43 | |
*** annegentle has quit IRC | 18:43 | |
*** ferest has joined #openstack-nova | 18:43 | |
*** ivasev has quit IRC | 18:44 | |
*** ferest has quit IRC | 18:44 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: objects: add missing enum values to DiskBus field https://review.openstack.org/200280 | 18:45 |
mriedem | rlrossit: see if that makes sense ^ | 18:45 |
*** ferest has joined #openstack-nova | 18:45 | |
openstackgerrit | Sudipta Biswas proposed openstack/nova: Base enablement for memory bandwidth monitor https://review.openstack.org/202623 | 18:46 |
*** ferest has quit IRC | 18:46 | |
*** romainh has joined #openstack-nova | 18:47 | |
*** ferest has joined #openstack-nova | 18:47 | |
*** agireud has quit IRC | 18:47 | |
*** agireud has joined #openstack-nova | 18:48 | |
*** ferest has quit IRC | 18:48 | |
*** ferest has joined #openstack-nova | 18:49 | |
*** ferest has quit IRC | 18:49 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: objects: add missing enum values to DiskBus field https://review.openstack.org/200280 | 18:49 |
*** ferest has joined #openstack-nova | 18:49 | |
*** mragupat_ has joined #openstack-nova | 18:50 | |
*** rushiagr is now known as rushiagr_away | 18:50 | |
*** ferest has quit IRC | 18:50 | |
*** ferest has joined #openstack-nova | 18:51 | |
*** casusbelli has quit IRC | 18:52 | |
*** lpetrut has joined #openstack-nova | 18:52 | |
*** apuimedo has joined #openstack-nova | 18:53 | |
*** mragupat has quit IRC | 18:54 | |
*** ferest has quit IRC | 18:54 | |
*** ferest has joined #openstack-nova | 18:55 | |
*** cpschult has quit IRC | 18:55 | |
openstackgerrit | Dan Smith proposed openstack/nova: Fix test_security_groups to use Instance object properly https://review.openstack.org/198765 | 18:56 |
openstackgerrit | Dan Smith proposed openstack/nova: Refactor test_servers to use instance objects https://review.openstack.org/197770 | 18:56 |
openstackgerrit | Dan Smith proposed openstack/nova: Remove legacy flavor compatibility code from Instance https://review.openstack.org/197771 | 18:56 |
openstackgerrit | Dan Smith proposed openstack/nova: Fix Instance object usage in test_extended_ips tests https://review.openstack.org/198767 | 18:56 |
openstackgerrit | Dan Smith proposed openstack/nova: Fix test_extended_server_attributes for proper Instance object usage https://review.openstack.org/198766 | 18:56 |
*** ferest has quit IRC | 18:57 | |
*** ferest has joined #openstack-nova | 18:57 | |
jroll | tonyb: re IPA on a bmc... just saw https://www.openstack.org/summit/tokyo-2015/vote-for-speakers/Presentation/6520 | 18:58 |
*** ferest has quit IRC | 18:59 | |
*** ferest has joined #openstack-nova | 19:00 | |
*** ferest has quit IRC | 19:02 | |
*** annegentle has joined #openstack-nova | 19:02 | |
openstackgerrit | Raildo Mascena de Sousa Filho proposed openstack/nova: Nested Quota Driver: Adding v2 and v3(v2.1) support https://review.openstack.org/149828 | 19:02 |
*** ferest has joined #openstack-nova | 19:03 | |
*** ferest has quit IRC | 19:03 | |
*** ferest has joined #openstack-nova | 19:04 | |
*** Mike_D_laptop has joined #openstack-nova | 19:04 | |
*** ferest has quit IRC | 19:06 | |
*** apuimedo has quit IRC | 19:06 | |
*** ferest has joined #openstack-nova | 19:07 | |
*** ferest has quit IRC | 19:08 | |
*** ferest has joined #openstack-nova | 19:09 | |
*** ferest has quit IRC | 19:09 | |
*** ferest has joined #openstack-nova | 19:10 | |
*** salv-orlando has joined #openstack-nova | 19:11 | |
*** ssurana has joined #openstack-nova | 19:11 | |
*** annegentle has quit IRC | 19:11 | |
*** ferest has quit IRC | 19:11 | |
*** samueldmq is now known as not_samueldmq | 19:11 | |
*** annegentle has joined #openstack-nova | 19:12 | |
*** ferest has joined #openstack-nova | 19:12 | |
*** ferest has quit IRC | 19:12 | |
*** ferest has joined #openstack-nova | 19:13 | |
*** cpschult has joined #openstack-nova | 19:14 | |
*** ferest has quit IRC | 19:14 | |
*** ferest has joined #openstack-nova | 19:16 | |
*** ferest has quit IRC | 19:17 | |
*** not_samueldmq is now known as samueldmq | 19:17 | |
*** lpetrut has quit IRC | 19:18 | |
*** ferest has joined #openstack-nova | 19:18 | |
*** baoli has quit IRC | 19:19 | |
*** baoli has joined #openstack-nova | 19:19 | |
*** ferest has quit IRC | 19:19 | |
*** xyang2 has quit IRC | 19:20 | |
*** ferest has joined #openstack-nova | 19:20 | |
*** ferest has quit IRC | 19:21 | |
*** Nastooh_ has joined #openstack-nova | 19:21 | |
*** kmARC has joined #openstack-nova | 19:21 | |
*** ferest has joined #openstack-nova | 19:21 | |
*** ferest has quit IRC | 19:22 | |
*** ijw has quit IRC | 19:23 | |
*** ferest has joined #openstack-nova | 19:23 | |
*** ferest has quit IRC | 19:23 | |
*** xyang2 has joined #openstack-nova | 19:23 | |
*** ferest has joined #openstack-nova | 19:23 | |
*** ferest has quit IRC | 19:25 | |
*** ferest has joined #openstack-nova | 19:26 | |
*** meghal has quit IRC | 19:27 | |
*** ferest has quit IRC | 19:27 | |
*** ferest has joined #openstack-nova | 19:27 | |
dims | haypo: ping | 19:27 |
dims | haypo: please take a look when you get a chance? https://review.openstack.org/#/q/project:openstack/nova+topic:bp/nova-python3+is:open,n,z | 19:28 |
*** ildikov has quit IRC | 19:28 | |
*** ferest has quit IRC | 19:28 | |
*** zsais has joined #openstack-nova | 19:28 | |
*** ferest has joined #openstack-nova | 19:29 | |
*** ferest has quit IRC | 19:32 | |
*** ferest has joined #openstack-nova | 19:32 | |
*** ferest has quit IRC | 19:32 | |
*** ferest has joined #openstack-nova | 19:34 | |
*** ferest has quit IRC | 19:34 | |
*** mwagner_lap has quit IRC | 19:35 | |
Nastooh_ | Trying to build a local nova-docker. Cloned git+https://github.com/stackforge/nova-docker and issued "sudo python setup.py install"; however, local changes are not visible, e.g., changing name to name = "nova-test-" + instance['uuid']. What am I missing? | 19:35 |
*** lpetrut has joined #openstack-nova | 19:36 | |
*** ferest has joined #openstack-nova | 19:36 | |
*** thorst1 has quit IRC | 19:36 | |
*** ferest has quit IRC | 19:36 | |
*** ssurana has left #openstack-nova | 19:36 | |
*** vishwanathj has quit IRC | 19:37 | |
*** ferest has joined #openstack-nova | 19:37 | |
*** otter768 has joined #openstack-nova | 19:38 | |
*** artom has joined #openstack-nova | 19:39 | |
*** ferest has quit IRC | 19:39 | |
*** artom has quit IRC | 19:39 | |
*** artom has joined #openstack-nova | 19:39 | |
*** ferest has joined #openstack-nova | 19:40 | |
*** otter768 has quit IRC | 19:42 | |
*** meghal has joined #openstack-nova | 19:43 | |
*** ferest has quit IRC | 19:43 | |
*** ferest has joined #openstack-nova | 19:44 | |
*** thangp has quit IRC | 19:44 | |
*** fawadkhaliq has quit IRC | 19:44 | |
*** ferest has quit IRC | 19:45 | |
*** harlowja has quit IRC | 19:45 | |
*** vilobhmm has quit IRC | 19:45 | |
*** ferest has joined #openstack-nova | 19:45 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add libvirt-python to test-requirements.txt to enforce min version https://review.openstack.org/183706 | 19:47 |
*** ferest has quit IRC | 19:47 | |
*** achanda has quit IRC | 19:47 | |
*** meghal has quit IRC | 19:47 | |
*** ferest has joined #openstack-nova | 19:48 | |
*** angdraug has joined #openstack-nova | 19:49 | |
*** ferest has quit IRC | 19:49 | |
*** ferest has joined #openstack-nova | 19:50 | |
*** ferest has quit IRC | 19:50 | |
*** romainh has left #openstack-nova | 19:51 | |
*** tonytan4ever has quit IRC | 19:51 | |
*** ferest has joined #openstack-nova | 19:51 | |
*** ferest has quit IRC | 19:51 | |
*** ferest has joined #openstack-nova | 19:52 | |
*** ferest has quit IRC | 19:52 | |
*** ferest has joined #openstack-nova | 19:53 | |
openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 19:54 |
*** ferest has quit IRC | 19:54 | |
*** mestery has quit IRC | 19:54 | |
openstackgerrit | Tomoki Sekiyama proposed openstack/nova: Fix multipath iSCSI encrypted volume attach failure https://review.openstack.org/196482 | 19:55 |
*** ferest has joined #openstack-nova | 19:55 | |
*** thorst1 has joined #openstack-nova | 19:55 | |
*** ferest has quit IRC | 19:56 | |
*** vilobhmm has joined #openstack-nova | 19:56 | |
*** ferest has joined #openstack-nova | 19:57 | |
*** Sukhdev has joined #openstack-nova | 19:58 | |
*** ferest has quit IRC | 19:59 | |
*** suresh12 has quit IRC | 19:59 | |
*** baoli has quit IRC | 20:00 | |
*** ferest has joined #openstack-nova | 20:00 | |
*** baoli has joined #openstack-nova | 20:00 | |
*** chhavi has quit IRC | 20:00 | |
*** harlowja has joined #openstack-nova | 20:01 | |
*** tonytan4ever has joined #openstack-nova | 20:01 | |
*** pixelbeat_ has joined #openstack-nova | 20:02 | |
*** davideagnello has quit IRC | 20:03 | |
*** ferest has quit IRC | 20:03 | |
*** davideagnello has joined #openstack-nova | 20:03 | |
*** ferest has joined #openstack-nova | 20:04 | |
*** suresh12 has joined #openstack-nova | 20:04 | |
*** ferest has quit IRC | 20:05 | |
*** ferest has joined #openstack-nova | 20:05 | |
*** andreykurilin__ has joined #openstack-nova | 20:07 | |
*** ferest has quit IRC | 20:07 | |
*** ferest has joined #openstack-nova | 20:08 | |
*** ferest has quit IRC | 20:08 | |
andreykurilin__ | melwitt: hi! I left a comment for you at https://review.openstack.org/#/c/167408/ | 20:08 |
*** ferest has joined #openstack-nova | 20:08 | |
melwitt | andreykurilin__: thanks! just saw it. wanted to make sure you were okay with alex_xu's change on your patch | 20:09 |
*** ferest has quit IRC | 20:09 | |
*** ferest has joined #openstack-nova | 20:10 | |
dansmith | mriedem: does my response here make sense? https://review.openstack.org/#/c/198765/4/nova/tests/unit/api/openstack/compute/contrib/test_security_groups.py,cm | 20:11 |
*** ferest has quit IRC | 20:11 | |
*** ferest has joined #openstack-nova | 20:12 | |
mriedem | magic is always an acceptable response | 20:13 |
*** ferest has quit IRC | 20:13 | |
dansmith | is it magic? | 20:14 |
*** ferest has joined #openstack-nova | 20:14 | |
dansmith | mriedem: I can do context=None if that would be more clear | 20:14 |
*** pixelbeat_ has quit IRC | 20:14 | |
mriedem | sec | 20:14 |
*** ferest has quit IRC | 20:14 | |
melwitt | dansmith, mriedem: I'm surprised that stub_instance_obj requires a context to be passed | 20:15 |
*** achanda has joined #openstack-nova | 20:15 | |
*** ferest has joined #openstack-nova | 20:15 | |
*** VW__ has joined #openstack-nova | 20:16 | |
mriedem | i think that's more or less why it's magical | 20:16 |
mriedem | if we don't need it, maybe stub_instance_obj should either not require it or default to None | 20:16 |
mriedem | so tests don't need to pass None | 20:16 |
dansmith | melwitt: because it calls _from_db_object which requires it, and since we use it for calls that pass context and expect a non-orphaned instance back | 20:16 |
melwitt | dansmith: oh | 20:16 |
*** pixelbeat_ has joined #openstack-nova | 20:16 | |
*** ferest has quit IRC | 20:16 | |
*** VW__ has quit IRC | 20:17 | |
dansmith | and it calls it out because it calls stub_instance() which doesn't take the context | 20:17 |
dansmith | so we don't have to fish it out of args | 20:17 |
*** ferest has joined #openstack-nova | 20:17 | |
*** VW__ has joined #openstack-nova | 20:17 | |
*** ferest has quit IRC | 20:18 | |
mriedem | yeah, ok | 20:18 |
*** diogogmt_ has joined #openstack-nova | 20:19 | |
*** ferest has joined #openstack-nova | 20:19 | |
*** VW_ has quit IRC | 20:19 | |
*** flwang has quit IRC | 20:19 | |
*** lucas__ has quit IRC | 20:20 | |
*** ferest has quit IRC | 20:20 | |
*** ferest has joined #openstack-nova | 20:20 | |
*** lucas__ has joined #openstack-nova | 20:20 | |
*** ferest has quit IRC | 20:20 | |
openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 20:21 |
*** ferest has joined #openstack-nova | 20:21 | |
*** diogogmt has quit IRC | 20:22 | |
*** ferest has quit IRC | 20:22 | |
*** diogogmt_ is now known as diogogmt | 20:22 | |
*** yassine has joined #openstack-nova | 20:23 | |
*** ferest has joined #openstack-nova | 20:23 | |
*** ferest has quit IRC | 20:24 | |
*** ferest has joined #openstack-nova | 20:24 | |
*** lucas__ has quit IRC | 20:25 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtISCSIVolumeDriver into it's own module https://review.openstack.org/202301 | 20:26 |
*** ferest has quit IRC | 20:26 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtISERVolumeDriver into it's own module https://review.openstack.org/197311 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move volume/remotefs/quobyte modules under volume subdir https://review.openstack.org/193223 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtNETVolumeDriver into it's own module https://review.openstack.org/197341 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtNFSVolumeDriver into it's own module https://review.openstack.org/196859 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtScalityVolumeDriver into it's own module https://review.openstack.org/193249 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move lvm/dmcrypt/rbd_utils modules under storage subdir https://review.openstack.org/193195 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtAOEVolumeDriver into it's own module https://review.openstack.org/196276 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtQuobyteVolumeDriver into the quobyte module https://review.openstack.org/193236 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert SMBFS vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202779 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert NFS volume driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202778 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtFibreChannelVolumeDriver into it's own module https://review.openstack.org/193278 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtGlusterfsVolumeDriver into it's own module https://review.openstack.org/193380 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Introduce LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202777 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Scality vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202783 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move fibre channel specific utilities into fc module https://review.openstack.org/193317 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Quobyte driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202782 | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtGPFSVolumeDriver into it's own module https://review.openstack.org/193240 | 20:26 |
*** lucas__ has joined #openstack-nova | 20:27 | |
*** ferest has joined #openstack-nova | 20:27 | |
ctrath | mriedem: regarding proxy to neutron quota and usage APIs for limits, do we want to do this in the v2 api only? | 20:27 |
*** ferest has quit IRC | 20:27 | |
mriedem | ctrath: v2.1 | 20:27 |
mriedem | microversion | 20:27 |
ctrath | leave 2 alone? | 20:28 |
mriedem | yes | 20:28 |
*** lucas__ has quit IRC | 20:28 | |
mriedem | ctrath: info on microversions is here http://docs.openstack.org/developer/nova/devref/api_microversions.html | 20:28 |
ctrath | thanks.. What's the reasoning for not doing it in v2? Just a support reason? | 20:28 |
*** kmARC has quit IRC | 20:29 | |
mriedem | it's not discoverable | 20:29 |
*** ferest has joined #openstack-nova | 20:29 | |
*** lucas__ has joined #openstack-nova | 20:29 | |
*** vishwanathj has joined #openstack-nova | 20:29 | |
*** ferest has quit IRC | 20:29 | |
mriedem | this was a fairly straight-forward microversion as a guide https://review.openstack.org/#/c/168966/ | 20:29 |
mriedem | you wouldn't have any request/response changes, but would show you how the version check happens | 20:30 |
ctrath | thanks | 20:30 |
ctrath | what do you mean by "discoverable"? | 20:30 |
*** ferest has joined #openstack-nova | 20:30 | |
mriedem | from the client perspective they don't know if the quotas/limits values they are getting back from the v2 api would be proxied from neutron or not | 20:31 |
mriedem | whereas with the microversion, you're basically opting in to that data | 20:31 |
*** ferest has quit IRC | 20:31 | |
*** lucas____ has joined #openstack-nova | 20:31 | |
*** lpetrut has quit IRC | 20:31 | |
ctrath | alright, so what happens if they call v2 and they're using Neutron? fail? | 20:31 |
mriedem | so you know, in v2.x they proxy to neutron for these values, so if my client is using at least v2.x i can expect certain results from that api | 20:31 |
mriedem | ctrath: same as they have for the last 2+ years | 20:32 |
ctrath | ok, gotcha | 20:32 |
mriedem | it doesn't fail | 20:32 |
mriedem | but it's wrong | 20:32 |
*** ferest has joined #openstack-nova | 20:32 | |
ctrath | it just uses the nova value | 20:32 |
*** armax has joined #openstack-nova | 20:32 | |
dansmith | mriedem: this doesn't break config because they actually specify the left half right? https://review.openstack.org/#/c/193223/13/nova/virt/libvirt/driver.py,cm | 20:32 |
mriedem | so i suspect client code using that api is ignoring the values for security groups/floating ips if using neutron and then calling the neutron api to get that info | 20:32 |
*** ferest has quit IRC | 20:32 | |
*** flwang has joined #openstack-nova | 20:32 | |
mriedem | dansmith: well, cinder provides the key | 20:33 |
dansmith | mriedem: okay | 20:33 |
*** lucas__ has quit IRC | 20:33 | |
dansmith | mriedem: why are those just strings and not a dict or something? | 20:33 |
mriedem | as the driver_volume_type in the connection_info dict passed back from cinder | 20:33 |
mriedem | dansmith: i wondered the same myself | 20:33 |
*** ferest has joined #openstack-nova | 20:33 | |
dansmith | mriedem: does that list get parsed into a dict somewhere? | 20:34 |
mriedem | b/c it's fun to parse key value pairs around = ? | 20:34 |
mriedem | i think so | 20:34 |
*** dmsimard has joined #openstack-nova | 20:34 | |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: add hw_vif_multiqueue_enabled image property https://review.openstack.org/202168 | 20:34 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: tests: change assertEqual args to expected, observed for VIF https://review.openstack.org/204190 | 20:34 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: enable virtio-net multiqueue https://review.openstack.org/128829 | 20:34 |
*** dmsimard has left #openstack-nova | 20:34 | |
*** ferest has quit IRC | 20:34 | |
dansmith | driver_dict_from_config() | 20:34 |
dansmith | gross | 20:34 |
mriedem | s/gross/super awesome/ | 20:34 |
openstackgerrit | dane-fichter proposed openstack/nova: libvirt: Raise error when booting from encrypted volumes https://review.openstack.org/203784 | 20:35 |
*** ferest has joined #openstack-nova | 20:35 | |
dansmith | riight | 20:35 |
*** lucas____ has quit IRC | 20:35 | |
*** ferest has quit IRC | 20:35 | |
*** ferest has joined #openstack-nova | 20:36 | |
*** mragupat_ has quit IRC | 20:36 | |
dansmith | mriedem: and, I assume we don't do anything stupid like store the whole class path in the db, right? we just store the iscsi= bit or something? | 20:37 |
*** mragupat has joined #openstack-nova | 20:37 | |
*** ferest has quit IRC | 20:37 | |
*** apuimedo has joined #openstack-nova | 20:37 | |
*** mragupat has quit IRC | 20:37 | |
mriedem | right, the iscsi part is the type value in connection_info which is part of the bdms table | 20:37 |
mriedem | the mapping is all internal to the driver | 20:37 |
*** annegentle has quit IRC | 20:37 | |
*** mragupat has joined #openstack-nova | 20:37 | |
dansmith | yeah, okay cool | 20:38 |
*** vladikr has quit IRC | 20:38 | |
*** mragupat has quit IRC | 20:38 | |
*** ferest has joined #openstack-nova | 20:38 | |
mriedem | and in the long long future i hope to objectify all of that connection_info mess in os-brick | 20:38 |
dansmith | good man | 20:39 |
*** mragupat has joined #openstack-nova | 20:39 | |
mriedem | should help us avoid more of these https://review.openstack.org/#/c/202795/ | 20:40 |
*** ferest has quit IRC | 20:40 | |
mriedem | if i had that in a versioned response from cinder, | 20:40 |
mriedem | then i'd konw if i can rely on those keys on a later change in that bp series | 20:40 |
*** ferest has joined #openstack-nova | 20:41 | |
*** ferest has quit IRC | 20:41 | |
*** mwagner_lap has joined #openstack-nova | 20:42 | |
dansmith | crazy talk | 20:42 |
*** ferest has joined #openstack-nova | 20:43 | |
raildo | hey guys, me and some Keystone folks are proposing a talk about hierarchical multitenancy, and I think that will be useful for other projects like nova. so if you want to vote in this session, we will be very happy :) https://www.openstack.org/summit/tokyo-2015/vote-for-speakers/presentation/6378 | 20:46 |
mriedem | melwitt: i guess that's you ^ | 20:46 |
*** ferest has quit IRC | 20:46 | |
*** ferest has joined #openstack-nova | 20:47 | |
*** harlowja_ has joined #openstack-nova | 20:47 | |
raildo | mriedem: ++ this will be special for her haha | 20:48 |
mriedem | gabriel-bezerra: did you notice there was an ironic job failure on this https://review.openstack.org/#/c/190675/ ? | 20:49 |
mriedem | http://logs.openstack.org/75/190675/2/check/gate-tempest-dsvm-ironic-pxe_ssh-full-nv/2c65f3f/console.html | 20:49 |
*** ferest has quit IRC | 20:49 | |
mriedem | jroll: i guess we know that test never passes since it's nonvoting ^ ? | 20:49 |
jroll | mriedem: yep, that just was added friday | 20:50 |
jroll | thanks for checking :) | 20:50 |
*** vilobhmm has quit IRC | 20:50 | |
mriedem | gd, 65 failed tests, why is it even in the check queue? | 20:50 |
jroll | hoping to hack on that with mtreinish this week | 20:50 |
*** ferest has joined #openstack-nova | 20:50 | |
jroll | uhh | 20:50 |
mriedem | sorry, 31 failed tests | 20:50 |
jroll | mriedem: I totally forgot experimental existed | 20:50 |
mriedem | oh it does | 20:50 |
mriedem | big time | 20:50 |
jroll | yeah | 20:50 |
*** ferest has quit IRC | 20:50 | |
jroll | heh | 20:50 |
jroll | :x | 20:50 |
* jroll fixes | 20:51 | |
mtreinish | jroll: I'm more around this week, I don't have to try to get to MN or back | 20:51 |
mtreinish | which is apparently something I fail at doing | 20:51 |
*** meghal has joined #openstack-nova | 20:51 | |
*** harlowja has quit IRC | 20:51 | |
mriedem | there is a kind of bermuda triange between MN, philly and wherever mtreinish lives in NY | 20:51 |
*** yamahata has quit IRC | 20:51 | |
*** ferest has joined #openstack-nova | 20:51 | |
mriedem | *triangle | 20:51 |
*** ferest has quit IRC | 20:51 | |
jroll | mtreinish: heh, I'm mostly around too but juggling lots of things. you think we should make this experimental for now or do you think we can get it working fairly quickly | 20:51 |
*** yamahata has joined #openstack-nova | 20:51 | |
mriedem | lots of goodies here http://logs.openstack.org/75/190675/2/check/gate-tempest-dsvm-ironic-pxe_ssh-full-nv/2c65f3f/logs/screen-n-cpu.txt.gz?level=TRACE | 20:52 |
mtreinish | jroll: I think we can get it fairly quickly | 20:52 |
mtreinish | but it doesn't hurt to move it to experimental | 20:52 |
jroll | ok, I'll move it quick | 20:53 |
*** ferest has joined #openstack-nova | 20:53 | |
*** eharney has quit IRC | 20:53 | |
mtreinish | mriedem: heh, well sdague didn't seem to have nearly as much trouble as I did | 20:53 |
mriedem | mtreinish: he didn't go through philly or NC right? | 20:53 |
*** ferest has quit IRC | 20:53 | |
mtreinish | mriedem: yeah that was the diff | 20:54 |
mriedem | right | 20:54 |
mriedem | he skirted the triangle like an experienced traveler | 20:54 |
mriedem | however, no water slides for him! | 20:54 |
*** atuvenie has quit IRC | 20:54 | |
mtreinish | jroll: hmm, ironic is returning 409s | 20:54 |
*** ferest has joined #openstack-nova | 20:54 | |
jroll | mtreinish: ironic LOVES 409s | 20:54 |
jroll | mtreinish: https://review.openstack.org/#/c/206218/ btw | 20:54 |
*** moshele has joined #openstack-nova | 20:54 | |
mtreinish | mriedem: yeah, I can cross staying at the waterpark of america off my bucket list :) | 20:55 |
*** ferest has quit IRC | 20:55 | |
mriedem | jroll: your change moves it to nova's gate queue | 20:56 |
mriedem | so umm | 20:56 |
jroll | what | 20:56 |
mriedem | change that please :) | 20:56 |
*** ferest has joined #openstack-nova | 20:56 | |
jroll | jesus | 20:56 |
rlrossit | dansmith: possibly totally unreasonable test, but if there is any change in an object adding a field, there has to be something in obj_make_compatible to take out that field if it goes to the previous version right? So we should be testing that | 20:56 |
*** ferest has quit IRC | 20:56 | |
jroll | mriedem: fixed, ty | 20:56 |
jroll | now the " | 20:57 |
jroll | now the "I'm dumb" in the commit message is fully qualified | 20:57 |
*** tongli has quit IRC | 20:57 | |
*** ferest has joined #openstack-nova | 20:57 | |
*** ferest has quit IRC | 20:57 | |
*** ferest has joined #openstack-nova | 20:59 | |
*** ferest has quit IRC | 20:59 | |
flwang | mriedem: ping | 21:00 |
*** raildo has quit IRC | 21:00 | |
flwang | mriedem: could you pls take a look at this https://review.openstack.org/#/c/205880/ ? | 21:00 |
flwang | mriedem: does it need a bp/spec? | 21:00 |
*** ferest has joined #openstack-nova | 21:00 | |
flwang | what I'm doing is add a project_id in the api response | 21:00 |
mriedem | flwang: yes, all api changes require a spec, see alex_xu's comments in there | 21:01 |
*** ferest has quit IRC | 21:01 | |
mriedem | especially if you're changing the response | 21:01 |
*** ferest has joined #openstack-nova | 21:02 | |
mriedem | flwang: like this https://review.openstack.org/#/c/168966/ | 21:02 |
mriedem | http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/show-reserved-status-in-os-fixed-ips-api.html | 21:02 |
openstackgerrit | Eric Brown proposed openstack/nova: Replace openssl calls with cryptography lib https://review.openstack.org/198246 | 21:02 |
flwang | mriedem: ok, cool, it's a good sample for me, cheers | 21:03 |
*** gary-smith has joined #openstack-nova | 21:03 | |
mriedem | flwang: np | 21:03 |
*** ferest has quit IRC | 21:05 | |
*** lucas__ has joined #openstack-nova | 21:05 | |
*** ferest has joined #openstack-nova | 21:05 | |
*** annegentle has joined #openstack-nova | 21:06 | |
*** moshele has quit IRC | 21:07 | |
*** ferest has quit IRC | 21:07 | |
*** rlrossit has left #openstack-nova | 21:08 | |
kfox1111_ | any idea why nova notifications in celiometer don't have valid project_id's? | 21:08 |
kfox1111_ | I'm seeing stuff like: b1e966858ed0caf9c020d5633d635295 instead of dad318e0-a2fb-4663-ab43-18c595818faa | 21:08 |
*** ferest has joined #openstack-nova | 21:08 | |
*** ferest has quit IRC | 21:09 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova: Add os_brick-based VolumeDriver for HGST connector https://review.openstack.org/194412 | 21:10 |
*** lucas__ has quit IRC | 21:10 | |
*** ferest has joined #openstack-nova | 21:10 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova: Add os-brick based LibvirtVolumeDriver for ScaleIO https://review.openstack.org/194454 | 21:10 |
*** chuckcarmack has left #openstack-nova | 21:11 | |
mriedem | kfox1111_: from the notifications code in nova that i'm looking at, the tenant_id is just pulled from instance.project_id, so as long as it's the right form in the nova instances table it should be correct in the notification | 21:11 |
*** ferest has quit IRC | 21:11 | |
kfox1111_ | the ceilometer folks claimed to be just passing it along too. how strange... | 21:12 |
*** ferest has joined #openstack-nova | 21:12 | |
kfox1111_ | and yet: select * from instances where project_id = 'b1e966858ed0caf9c020d5633d635295'; | 21:13 |
*** ferest has quit IRC | 21:13 | |
mriedem | kfox1111_: fwiw i'm looking at nova.notifications.info_from_instance | 21:13 |
kfox1111_ | shows no rows. | 21:13 |
*** ferest has joined #openstack-nova | 21:14 | |
kfox1111_ | wait.... | 21:14 |
*** annashen has quit IRC | 21:14 | |
*** ctrath has quit IRC | 21:14 | |
kfox1111_ | select project_id from instances group by project_id limit 10; does show projects without any -'s. | 21:14 |
kfox1111_ | yeah. so the strange projectid's are in the nova instance database. | 21:15 |
*** achanda has quit IRC | 21:15 | |
*** ferest has quit IRC | 21:15 | |
*** rdopiera has quit IRC | 21:16 | |
*** ferest has joined #openstack-nova | 21:16 | |
*** ferest has quit IRC | 21:16 | |
*** ferest has joined #openstack-nova | 21:18 | |
openstackgerrit | dane-fichter proposed openstack/nova: libvirt: Raise error when booting from encrypted volumes https://review.openstack.org/203784 | 21:20 |
*** ferest has quit IRC | 21:20 | |
*** gary-smith has left #openstack-nova | 21:21 | |
*** xyang2 has quit IRC | 21:21 | |
*** ferest has joined #openstack-nova | 21:21 | |
*** ferest has quit IRC | 21:22 | |
*** ferest has joined #openstack-nova | 21:23 | |
*** dane-fichter has quit IRC | 21:24 | |
*** ferest has quit IRC | 21:24 | |
*** ddaskal has quit IRC | 21:25 | |
*** ferest has joined #openstack-nova | 21:25 | |
*** ferest has quit IRC | 21:26 | |
*** pixelbeat_ has quit IRC | 21:26 | |
*** baoli has quit IRC | 21:26 | |
*** ferest has joined #openstack-nova | 21:26 | |
*** baoli has joined #openstack-nova | 21:27 | |
openstackgerrit | Danny Wilson proposed openstack/nova: libvirt: add virtio-scsi support for attached volumes https://review.openstack.org/205726 | 21:27 |
*** ferest has quit IRC | 21:31 | |
*** aysyd has quit IRC | 21:31 | |
*** ferest has joined #openstack-nova | 21:32 | |
*** ferest has quit IRC | 21:33 | |
*** baoli has quit IRC | 21:33 | |
*** ferest has joined #openstack-nova | 21:34 | |
*** ferest has quit IRC | 21:35 | |
tonyb | jroll: Hmmm interesting. | 21:36 |
*** ferest has joined #openstack-nova | 21:36 | |
*** ferest has quit IRC | 21:37 | |
*** vilobhmm has joined #openstack-nova | 21:38 | |
openstackgerrit | Eric Brown proposed openstack/nova: Replace openssl calls with cryptography lib https://review.openstack.org/198246 | 21:38 |
*** ferest has joined #openstack-nova | 21:38 | |
*** annashen has joined #openstack-nova | 21:39 | |
*** ferest has quit IRC | 21:39 | |
*** otter768 has joined #openstack-nova | 21:39 | |
*** salv-orlando has quit IRC | 21:39 | |
*** ferest has joined #openstack-nova | 21:40 | |
*** pixelbeat_ has joined #openstack-nova | 21:40 | |
openstackgerrit | Fei Long Wang proposed openstack/nova: Fix nova backup for volume-backed instance https://review.openstack.org/164494 | 21:40 |
*** ferest has quit IRC | 21:40 | |
*** ferest has joined #openstack-nova | 21:41 | |
openstackgerrit | Eric Brown proposed openstack/nova: Replace openssl calls with cryptography lib https://review.openstack.org/198246 | 21:42 |
*** yassine has quit IRC | 21:42 | |
*** ferest has quit IRC | 21:42 | |
*** ferest has joined #openstack-nova | 21:43 | |
kfox1111_ | anyone know why projectid's are strainge in nova.instance? | 21:43 |
*** armax has quit IRC | 21:43 | |
*** ferest has quit IRC | 21:43 | |
*** otter768 has quit IRC | 21:44 | |
*** ferest has joined #openstack-nova | 21:44 | |
*** ferest has quit IRC | 21:45 | |
*** ferest has joined #openstack-nova | 21:46 | |
*** ferest has quit IRC | 21:46 | |
*** ferest has joined #openstack-nova | 21:47 | |
*** ferest has quit IRC | 21:48 | |
*** cpschult has quit IRC | 21:49 | |
*** ferest has joined #openstack-nova | 21:49 | |
*** ferest has quit IRC | 21:50 | |
*** rfolco has quit IRC | 21:51 | |
*** ferest has joined #openstack-nova | 21:51 | |
*** ferest has quit IRC | 21:51 | |
*** meghal has quit IRC | 21:52 | |
*** ferest has joined #openstack-nova | 21:52 | |
*** meghal has joined #openstack-nova | 21:52 | |
*** ferest has quit IRC | 21:53 | |
*** ferest has joined #openstack-nova | 21:53 | |
mriedem | kfox1111_: well they are in your nova db right? can you trace those instances back to what created them? | 21:53 |
*** armax has joined #openstack-nova | 21:54 | |
*** agireud has quit IRC | 21:54 | |
*** ferest has quit IRC | 21:54 | |
*** penick has quit IRC | 21:54 | |
med_ | so, if Nova IRC weekly meetings are for "stuck reviews" and not for reviews lacking attention, how does one garner the appropriate attention? | 21:54 |
*** ferest has joined #openstack-nova | 21:55 | |
*** ferest has quit IRC | 21:55 | |
mriedem | med_: sacrifices to the dark lord | 21:55 |
* med_ thought the newbie guide said to sit in the IRC meeting and try and get it brought up | 21:55 | |
med_ | mriedem, I should have bought the cheeseheads then.... | 21:55 |
* med_ knew that was a critical mistake. | 21:55 | |
mriedem | med_: link? | 21:56 |
*** Piet has joined #openstack-nova | 21:56 | |
med_ | https://wiki.openstack.org/wiki/UX | 21:56 |
med_ | not that | 21:56 |
openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: filter None port_ids from ports list in _unbind_ports https://review.openstack.org/206236 | 21:56 |
med_ | https://review.openstack.org/#/c/203214/ | 21:56 |
*** ferest has joined #openstack-nova | 21:56 | |
med_ | sorry, wrong project the first time. | 21:56 |
med_ | 203214 is the right one.... | 21:56 |
*** ferest has quit IRC | 21:56 | |
mriedem | so, that's been up for about 1.5 weeks | 21:57 |
med_ | It's part of the filters devref. | 21:57 |
med_ | do I need to wait another 2.5 weeks? | 21:57 |
* med_ really has no idea of the nova speed/expectations | 21:57 | |
kfox1111_ | mriedem: yeah, the users via horizon. | 21:57 |
*** mragupat has quit IRC | 21:57 | |
mriedem | no, just saying, that's not terribly long to be around | 21:57 |
med_ | gotcha | 21:57 |
kfox1111_ | everything's working as expected, which is very strange. | 21:57 |
mriedem | med_: we have the trivial bug fix list in the priorities etherpad | 21:57 |
mriedem | med_: but with small stuff like this i think it's fine to ask for a review in channel | 21:58 |
med_ | and it said not to add your own... | 21:58 |
mriedem | yeah, i know | 21:58 |
med_ | that there were triagers. | 21:58 |
*** Sukhdev has quit IRC | 21:58 | |
mriedem | i blame bauzas for not adding yours | 21:58 |
*** ferest has joined #openstack-nova | 21:58 | |
med_ | wfm. :^) | 21:58 |
bauzas | mriedem: mmm ? :) | 21:58 |
*** zsais has quit IRC | 21:58 | |
mriedem | we also said at the midcycle to start doing the trivial bug list after FPF | 21:58 |
bauzas | oh | 21:58 |
*** ferest has quit IRC | 21:58 | |
med_ | nod, no bandwidth before that. Good point. | 21:58 |
med_ | thanks for the attention mriedem / bauzas and #chan | 21:59 |
kfox1111_ | hmm... on our other cloud, the project_id's in nova also look strange. | 21:59 |
bauzas | yeah, I'm quite off for reviewing bugs | 21:59 |
bauzas | until we are past FF | 21:59 |
kfox1111_ | how does nova relate keystone projects to its own project_id's? | 21:59 |
*** ferest has joined #openstack-nova | 21:59 | |
med_ | bauzas, you already +1d just didn't get added to trivial ehterpad list. | 22:00 |
bauzas | med_: any change I should review ? | 22:00 |
bauzas | med_: okay, which one ? | 22:00 |
*** ferest has quit IRC | 22:00 | |
med_ | https://review.openstack.org/#/c/203214/ is what I'm referring to... was hoping to get someone to +2 or comment | 22:00 |
med_ | (very trivial) | 22:00 |
*** pixelbeat_ has quit IRC | 22:00 | |
*** mriedem is now known as mriedem_away | 22:00 | |
med_ | but the process was informative. | 22:00 |
*** meghal has quit IRC | 22:00 | |
bauzas | med_: oh man, I missed to see that it was a bug | 22:01 |
med_ | and I'm the guy that joined occasionally to the Hangout last week (mostly so I could see what section of the etherpad John was at.) | 22:01 |
*** jungleboyj has quit IRC | 22:01 | |
*** ferest has joined #openstack-nova | 22:01 | |
dansmith | mriedem_away: oops, I guess I missed rlrossit | 22:01 |
mriedem_away | dansmith: he'll be around tomorrow | 22:01 |
dansmith | k | 22:02 |
*** Sukhdev has joined #openstack-nova | 22:02 | |
*** ferest has quit IRC | 22:02 | |
mriedem_away | i will be in meetings all day | 22:02 |
mriedem_away | \o/ | 22:02 |
med_ | :-( | 22:02 |
*** ddaskal has joined #openstack-nova | 22:02 | |
*** ferest has joined #openstack-nova | 22:03 | |
bauzas | mriedem_away: med_: okay, added it to https://etherpad.openstack.org/p/liberty-nova-priorities-tracking | 22:03 |
med_ | grazie. | 22:03 |
openstackgerrit | Jay Pipes proposed openstack/nova: ironic: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189362 | 22:03 |
openstackgerrit | Jay Pipes proposed openstack/nova: objects: convert config drive to use ImageMeta object https://review.openstack.org/189323 | 22:03 |
openstackgerrit | Jay Pipes proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 22:03 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 22:03 |
openstackgerrit | Jay Pipes proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 22:03 |
bauzas | see you guys | 22:03 |
*** ferest has quit IRC | 22:03 | |
*** penick has joined #openstack-nova | 22:03 | |
*** ferest has joined #openstack-nova | 22:04 | |
*** burt has quit IRC | 22:06 | |
*** ferest has quit IRC | 22:06 | |
*** armax has quit IRC | 22:06 | |
*** ferest has joined #openstack-nova | 22:07 | |
*** ferest has quit IRC | 22:07 | |
*** unicell has quit IRC | 22:08 | |
*** unicell has joined #openstack-nova | 22:08 | |
*** ferest has joined #openstack-nova | 22:08 | |
*** salv-orlando has joined #openstack-nova | 22:09 | |
*** ferest has quit IRC | 22:09 | |
*** ferest has joined #openstack-nova | 22:10 | |
*** ferest has quit IRC | 22:10 | |
*** ferest has joined #openstack-nova | 22:11 | |
*** xyang2 has joined #openstack-nova | 22:11 | |
*** ferest has quit IRC | 22:11 | |
*** ferest has joined #openstack-nova | 22:12 | |
kfox1111_ | oh... strange. its keystone maybe that's doing tiat. | 22:13 |
*** edmondsw has quit IRC | 22:14 | |
*** ferest has quit IRC | 22:14 | |
*** Piet has quit IRC | 22:15 | |
*** ferest has joined #openstack-nova | 22:15 | |
*** achanda has joined #openstack-nova | 22:15 | |
*** jasondot_ has quit IRC | 22:15 | |
*** ferest has quit IRC | 22:15 | |
*** ferest has joined #openstack-nova | 22:17 | |
*** tongli has joined #openstack-nova | 22:18 | |
*** Nikolay_St has quit IRC | 22:20 | |
*** harlowja_ has quit IRC | 22:21 | |
*** harlowja has joined #openstack-nova | 22:21 | |
*** ferest has quit IRC | 22:21 | |
*** tpeoples has quit IRC | 22:23 | |
*** xyang2 has quit IRC | 22:24 | |
*** VW__ has quit IRC | 22:24 | |
*** VW_ has joined #openstack-nova | 22:25 | |
*** marcusvrn has quit IRC | 22:25 | |
*** ferest has joined #openstack-nova | 22:26 | |
*** VW_ has quit IRC | 22:29 | |
*** ferest has quit IRC | 22:30 | |
*** ferest has joined #openstack-nova | 22:31 | |
*** kylek3h has quit IRC | 22:32 | |
*** ferest has quit IRC | 22:35 | |
*** ferest has joined #openstack-nova | 22:35 | |
*** ijuwang has quit IRC | 22:36 | |
*** ferest has quit IRC | 22:36 | |
*** penick has quit IRC | 22:37 | |
*** ferest has joined #openstack-nova | 22:37 | |
*** penick has joined #openstack-nova | 22:37 | |
*** ferest has quit IRC | 22:38 | |
*** ferest has joined #openstack-nova | 22:39 | |
*** ferest has quit IRC | 22:39 | |
openstackgerrit | Ed Leafe proposed openstack/nova: Add non-negative integer and float fields https://review.openstack.org/206249 | 22:39 |
*** ferest has joined #openstack-nova | 22:40 | |
*** ferest has quit IRC | 22:41 | |
*** rfolco has joined #openstack-nova | 22:41 | |
*** haypo has left #openstack-nova | 22:42 | |
dansmith | melwitt: thanks for hitting those patches again | 22:42 |
*** ferest has joined #openstack-nova | 22:43 | |
*** ferest has quit IRC | 22:43 | |
dansmith | melwitt: I assume you noticed that I put back in all the other tests I had snipped out (the bad_flavor, etc ones) | 22:43 |
openstackgerrit | Ed Leafe proposed openstack/nova: Add object models for comparing resource amounts https://review.openstack.org/128992 | 22:43 |
dansmith | melwitt: those seemed equally meaningless in the new way, but the point about the exception translation seems reason enough to keep them | 22:44 |
*** ferest has joined #openstack-nova | 22:44 | |
*** meghal has joined #openstack-nova | 22:45 | |
*** ferest has quit IRC | 22:45 | |
melwitt | dansmith: I did. yeah, I agree they test a lot less with the new way. thanks for helping think about whether they still have some value | 22:46 |
*** ferest has joined #openstack-nova | 22:46 | |
*** ferest has quit IRC | 22:47 | |
*** ferest has joined #openstack-nova | 22:48 | |
dansmith | ferest: it'd be good if you can figure out why you keep joining/quitting IRC please. it's very distracting | 22:48 |
*** ferest has quit IRC | 22:49 | |
dansmith | *sigh* | 22:49 |
*** ferest has joined #openstack-nova | 22:50 | |
*** annegentle has quit IRC | 22:50 | |
dansmith | ferest: it'd be good if you can figure out why you keep joining/quitting IRC please. it's very distracting | 22:51 |
*** ferest has quit IRC | 22:52 | |
*** ferest has joined #openstack-nova | 22:53 | |
*** ferest has quit IRC | 22:53 | |
*** alexpilotti has quit IRC | 22:54 | |
*** ferest has joined #openstack-nova | 22:54 | |
*** ferest has quit IRC | 22:56 | |
*** ferest has joined #openstack-nova | 22:58 | |
*** penick has quit IRC | 23:00 | |
*** ferest has quit IRC | 23:00 | |
*** ferest has joined #openstack-nova | 23:00 | |
*** davideagnello has quit IRC | 23:01 | |
*** meghal has quit IRC | 23:01 | |
*** davideagnello has joined #openstack-nova | 23:01 | |
*** r-daneel has quit IRC | 23:02 | |
*** ferest has quit IRC | 23:02 | |
*** jwcroppe has quit IRC | 23:02 | |
*** jwcroppe has joined #openstack-nova | 23:03 | |
*** ferest has joined #openstack-nova | 23:03 | |
*** meghal has joined #openstack-nova | 23:04 | |
*** ferest has quit IRC | 23:04 | |
*** MentalRay has quit IRC | 23:05 | |
*** suresh12 has quit IRC | 23:05 | |
*** slong has joined #openstack-nova | 23:06 | |
*** ferest has joined #openstack-nova | 23:06 | |
*** ferest has quit IRC | 23:06 | |
*** penick has joined #openstack-nova | 23:06 | |
*** arahal has joined #openstack-nova | 23:06 | |
*** zzzeek has quit IRC | 23:07 | |
*** ferest has joined #openstack-nova | 23:07 | |
*** jasondotstar has joined #openstack-nova | 23:08 | |
*** ferest has quit IRC | 23:08 | |
*** ferest has joined #openstack-nova | 23:09 | |
openstackgerrit | Jay Pipes proposed openstack/nova: ironic: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189362 | 23:11 |
*** ferest has quit IRC | 23:11 | |
openstackgerrit | Jay Pipes proposed openstack/nova: objects: convert config drive to use ImageMeta object https://review.openstack.org/189323 | 23:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188119 | 23:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: make instance compulsory in blockinfo APIs https://review.openstack.org/188118 | 23:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: hyperv: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189302 | 23:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 23:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: virt: convert hardware module to use nova.objects.ImageMeta https://review.openstack.org/188789 | 23:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 23:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 23:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: xenapi: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188790 | 23:11 |
*** thedodd has quit IRC | 23:12 | |
*** thorst1 has quit IRC | 23:12 | |
*** ferest has joined #openstack-nova | 23:12 | |
*** annashen has quit IRC | 23:12 | |
*** jasondotstar has quit IRC | 23:13 | |
*** ferest has quit IRC | 23:13 | |
*** yamahata has quit IRC | 23:14 | |
*** ferest has joined #openstack-nova | 23:14 | |
*** ferest has quit IRC | 23:17 | |
*** annashen has joined #openstack-nova | 23:17 | |
*** lucas__ has joined #openstack-nova | 23:17 | |
*** VW_ has joined #openstack-nova | 23:18 | |
*** ferest has joined #openstack-nova | 23:18 | |
*** ZZelle_ has quit IRC | 23:18 | |
*** Piet has joined #openstack-nova | 23:18 | |
*** ferest has quit IRC | 23:18 | |
*** jaypipes has quit IRC | 23:19 | |
*** meghal has quit IRC | 23:19 | |
*** ferest has joined #openstack-nova | 23:19 | |
*** rfolco has quit IRC | 23:19 | |
*** ferest has quit IRC | 23:22 | |
*** lucas__ has quit IRC | 23:22 | |
*** ferest has joined #openstack-nova | 23:23 | |
*** ferest has quit IRC | 23:23 | |
*** annashen has quit IRC | 23:23 | |
*** tonytan4ever has quit IRC | 23:23 | |
*** ferest has joined #openstack-nova | 23:24 | |
*** ferest has quit IRC | 23:26 | |
*** ferest has joined #openstack-nova | 23:27 | |
*** suresh12 has joined #openstack-nova | 23:27 | |
*** jasondot_ has joined #openstack-nova | 23:28 | |
*** ferest has quit IRC | 23:28 | |
*** ferest has joined #openstack-nova | 23:29 | |
*** penick has quit IRC | 23:29 | |
*** ferest has quit IRC | 23:29 | |
*** ferest has joined #openstack-nova | 23:30 | |
*** jecarey has quit IRC | 23:31 | |
*** salv-orlando has quit IRC | 23:32 | |
*** penick has joined #openstack-nova | 23:33 | |
*** ferest has quit IRC | 23:33 | |
*** ferest has joined #openstack-nova | 23:34 | |
*** Jeffrey4l has joined #openstack-nova | 23:35 | |
*** ferest has quit IRC | 23:35 | |
*** ferest has joined #openstack-nova | 23:36 | |
*** arahal has quit IRC | 23:37 | |
*** arahal has joined #openstack-nova | 23:37 | |
*** ferest has quit IRC | 23:37 | |
*** arahal has quit IRC | 23:37 | |
*** ferest has joined #openstack-nova | 23:38 | |
*** lucas-dinner has quit IRC | 23:38 | |
*** ferest has quit IRC | 23:39 | |
*** meghal has joined #openstack-nova | 23:39 | |
*** ferest has joined #openstack-nova | 23:40 | |
*** otter768 has joined #openstack-nova | 23:40 | |
*** markvoelker has quit IRC | 23:40 | |
*** VW_ has quit IRC | 23:41 | |
*** VW_ has joined #openstack-nova | 23:41 | |
*** ferest has quit IRC | 23:41 | |
*** ferest has joined #openstack-nova | 23:42 | |
*** ferest has quit IRC | 23:42 | |
*** erlon has quit IRC | 23:43 | |
*** jasondotstar has joined #openstack-nova | 23:44 | |
*** ferest has joined #openstack-nova | 23:44 | |
*** diogogmt has quit IRC | 23:44 | |
*** ferest has quit IRC | 23:44 | |
*** tonytan4ever has joined #openstack-nova | 23:45 | |
*** otter768 has quit IRC | 23:45 | |
*** ferest has joined #openstack-nova | 23:45 | |
*** ferest has quit IRC | 23:46 | |
*** VW_ has quit IRC | 23:46 | |
*** ferest has joined #openstack-nova | 23:46 | |
*** jasondot_ has quit IRC | 23:47 | |
*** ferest has quit IRC | 23:47 | |
*** ferest has joined #openstack-nova | 23:48 | |
*** ferest has quit IRC | 23:48 | |
*** ferest has joined #openstack-nova | 23:49 | |
*** thorst has joined #openstack-nova | 23:51 | |
*** ferest has quit IRC | 23:51 | |
*** otter768 has joined #openstack-nova | 23:52 | |
*** ferest has joined #openstack-nova | 23:53 | |
*** tongli has quit IRC | 23:54 | |
*** ferest has quit IRC | 23:54 | |
*** ferest has joined #openstack-nova | 23:55 | |
*** ferest has quit IRC | 23:55 | |
*** ferest has joined #openstack-nova | 23:56 | |
*** tonytan4ever has quit IRC | 23:57 | |
*** ferest has quit IRC | 23:57 | |
*** ferest has joined #openstack-nova | 23:58 | |
*** dannywilson has quit IRC | 23:59 | |
*** ferest has quit IRC | 23:59 | |
openstackgerrit | Shiina, Hironori proposed openstack/nova: libvirt:Add a driver API to inject an NMI https://review.openstack.org/202380 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!