| *** IanGovett has joined #openstack-nova | 00:00 | |
| *** markvoelker has quit IRC | 00:02 | |
| *** dims_ has joined #openstack-nova | 00:02 | |
| *** dimsum__ has quit IRC | 00:04 | |
| *** adam_g has quit IRC | 00:09 | |
| *** adam_g has joined #openstack-nova | 00:10 | |
| *** adam_g has quit IRC | 00:10 | |
| *** adam_g has joined #openstack-nova | 00:10 | |
| *** annashen has joined #openstack-nova | 00:24 | |
| *** annashen has quit IRC | 00:28 | |
| *** jasondotstar has joined #openstack-nova | 00:28 | |
| *** tpeoples has quit IRC | 00:35 | |
| *** pixelbeat_ has quit IRC | 00:43 | |
| *** mriedem has quit IRC | 00:43 | |
| *** salv-orlando has joined #openstack-nova | 00:46 | |
| *** dims_ has quit IRC | 00:50 | |
| *** dimsum__ has joined #openstack-nova | 00:53 | |
| openstackgerrit | sajeesh cs proposed openstack/nova: NestedQuotaDriver:Modification of settable quotas of nested projects https://review.openstack.org/200342 | 00:54 |
|---|---|---|
| *** rmk has quit IRC | 00:55 | |
| *** salv-orlando has quit IRC | 00:55 | |
| openstackgerrit | Hiroyuki Eguchi proposed openstack/nova: qemu-img convert should be skipped when migrating https://review.openstack.org/138979 | 00:59 |
| openstackgerrit | Hiroyuki Eguchi proposed openstack/nova: qemu-img convert should be skipped when migrating https://review.openstack.org/138979 | 01:02 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Fix link's href to consider osapi_compute_link_prefix https://review.openstack.org/207702 | 01:04 |
| *** rmk has joined #openstack-nova | 01:07 | |
| *** jamielennox is now known as jamielennox|away | 01:08 | |
| *** haigang has joined #openstack-nova | 01:10 | |
| *** artom has quit IRC | 01:14 | |
| *** kiwik-chenrui has quit IRC | 01:17 | |
| *** RuiChen has joined #openstack-nova | 01:18 | |
| *** park_heijlong has quit IRC | 01:19 | |
| *** park_heijlong has joined #openstack-nova | 01:22 | |
| openstackgerrit | sajeesh cs proposed openstack/nova: Nested Quota Driver: Finding parent_id and immediate child list https://review.openstack.org/200941 | 01:23 |
| *** meghal has joined #openstack-nova | 01:23 | |
| *** wanghua has joined #openstack-nova | 01:24 | |
| *** tpeoples has joined #openstack-nova | 01:24 | |
| *** achanda has joined #openstack-nova | 01:25 | |
| *** dimsum__ has quit IRC | 01:25 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtISCSIVolumeDriver into it's own module https://review.openstack.org/202301 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtISERVolumeDriver into it's own module https://review.openstack.org/197311 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtNFSVolumeDriver into it's own module https://review.openstack.org/196859 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtNETVolumeDriver into it's own module https://review.openstack.org/197341 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtAOEVolumeDriver into it's own module https://review.openstack.org/196276 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert SMBFS vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202779 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtFibreChannelVolumeDriver into it's own module https://review.openstack.org/193278 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtGlusterfsVolumeDriver into it's own module https://review.openstack.org/193380 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert NFS volume driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202778 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Introduce LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202777 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Scality vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202783 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Quobyte driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202782 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert GPFS volume driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202781 | 01:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtSMBFSVolumeDriver into it's own module https://review.openstack.org/196380 | 01:26 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert GlusterFS driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202780 | 01:26 |
| *** dimsum__ has joined #openstack-nova | 01:28 | |
| *** shihanzhang has quit IRC | 01:28 | |
| *** mmedvede has quit IRC | 01:30 | |
| *** mmedvede has joined #openstack-nova | 01:31 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: DB:support for migration objects multi-statuses https://review.openstack.org/169604 | 01:33 |
| *** tpeoples has quit IRC | 01:33 | |
| *** tpeoples has joined #openstack-nova | 01:33 | |
| *** humble_ has joined #openstack-nova | 01:34 | |
| *** shuquan has joined #openstack-nova | 01:35 | |
| *** haigang has quit IRC | 01:35 | |
| *** haigang has joined #openstack-nova | 01:36 | |
| *** wanghua has quit IRC | 01:37 | |
| *** tpeoples has quit IRC | 01:38 | |
| *** shuquan_ has joined #openstack-nova | 01:40 | |
| *** shuquan has quit IRC | 01:40 | |
| *** shuquan_ is now known as shuquan | 01:40 | |
| *** meghal has quit IRC | 01:44 | |
| *** lyanchih_ has joined #openstack-nova | 01:46 | |
| *** lyanchih_ has left #openstack-nova | 01:46 | |
| *** jyuso1 has joined #openstack-nova | 01:46 | |
| *** lyanchih_ has joined #openstack-nova | 01:47 | |
| *** lyanchih_ has quit IRC | 01:50 | |
| *** lyanchih_ has joined #openstack-nova | 01:50 | |
| *** lyanchih_ has left #openstack-nova | 01:51 | |
| *** lyanchih_ has joined #openstack-nova | 01:53 | |
| *** salv-orlando has joined #openstack-nova | 01:54 | |
| *** shuquan_ has joined #openstack-nova | 01:58 | |
| *** shuquan has quit IRC | 01:58 | |
| *** shuquan_ is now known as shuquan | 01:58 | |
| *** markvoelker has joined #openstack-nova | 01:59 | |
| *** shuquan has quit IRC | 02:02 | |
| *** dimsum__ has quit IRC | 02:02 | |
| *** humble__ has joined #openstack-nova | 02:02 | |
| *** shuquan has joined #openstack-nova | 02:02 | |
| *** markvoelker has quit IRC | 02:03 | |
| *** humble_ has quit IRC | 02:05 | |
| *** salv-orlando has quit IRC | 02:06 | |
| *** whenry has joined #openstack-nova | 02:08 | |
| *** haomaiw__ has joined #openstack-nova | 02:09 | |
| openstackgerrit | lyanchih proposed openstack/nova: Remove error instance fail when enable serial_consol https://review.openstack.org/208268 | 02:09 |
| *** IanGovett has quit IRC | 02:10 | |
| *** haomaiwang has quit IRC | 02:11 | |
| *** baoli has joined #openstack-nova | 02:14 | |
| *** shuquan has quit IRC | 02:15 | |
| *** shuquan has joined #openstack-nova | 02:15 | |
| *** whenry has quit IRC | 02:24 | |
| *** annashen has joined #openstack-nova | 02:25 | |
| *** haomaiwang has joined #openstack-nova | 02:25 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix cinder volume stuck in swap_volume https://review.openstack.org/207385 | 02:26 |
| *** kiran-r has joined #openstack-nova | 02:27 | |
| *** kiran-r has quit IRC | 02:27 | |
| *** haomaiw__ has quit IRC | 02:28 | |
| *** annashen has quit IRC | 02:29 | |
| *** shuquan has quit IRC | 02:31 | |
| *** baoli has quit IRC | 02:32 | |
| *** shuquan has joined #openstack-nova | 02:33 | |
| *** shuquan has quit IRC | 02:36 | |
| *** otter768 has joined #openstack-nova | 02:36 | |
| *** shuquan has joined #openstack-nova | 02:37 | |
| *** humble_ has joined #openstack-nova | 02:39 | |
| *** shuquan has quit IRC | 02:41 | |
| *** humble__ has quit IRC | 02:42 | |
| *** shuquan has joined #openstack-nova | 02:44 | |
| *** hakimo_ has joined #openstack-nova | 02:52 | |
| *** hakimo has quit IRC | 02:54 | |
| *** baoli has joined #openstack-nova | 02:54 | |
| *** baoli has quit IRC | 03:05 | |
| *** diogogmt has quit IRC | 03:08 | |
| *** alejandrito has quit IRC | 03:12 | |
| *** tjones has joined #openstack-nova | 03:15 | |
| *** tpeoples has joined #openstack-nova | 03:23 | |
| *** shuquan has quit IRC | 03:26 | |
| *** otter768 has quit IRC | 03:29 | |
| *** liusheng has joined #openstack-nova | 03:30 | |
| *** jecarey has joined #openstack-nova | 03:33 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Adds instance_uuid index for instance_system_metadata https://review.openstack.org/208277 | 03:36 |
| *** coolsvap|away is now known as coolsvap | 03:40 | |
| *** shuquan has joined #openstack-nova | 03:40 | |
| *** mdrabe has joined #openstack-nova | 03:42 | |
| openstackgerrit | sajeesh cs proposed openstack/nova: Nested Quota Driver: Adding v2 and v3 support https://review.openstack.org/149828 | 03:46 |
| *** FL1SK has quit IRC | 03:52 | |
| *** Jeffrey4l has joined #openstack-nova | 03:52 | |
| *** salv-orlando has joined #openstack-nova | 03:52 | |
| *** mdrabe has quit IRC | 03:53 | |
| *** salv-orlando has quit IRC | 03:58 | |
| *** salv-orl_ has joined #openstack-nova | 04:00 | |
| *** markvoelker has joined #openstack-nova | 04:00 | |
| openstackgerrit | lyanchih proposed openstack/nova: libvirt: serial console ports count upper limit needs to be checked https://review.openstack.org/207373 | 04:00 |
| *** shuquan has quit IRC | 04:02 | |
| *** shuquan has joined #openstack-nova | 04:03 | |
| *** salv-orl_ has quit IRC | 04:04 | |
| *** markvoelker has quit IRC | 04:04 | |
| *** lyanchih_ has quit IRC | 04:05 | |
| *** sajeesh has quit IRC | 04:06 | |
| *** ijw has joined #openstack-nova | 04:07 | |
| *** tjones has quit IRC | 04:14 | |
| *** shuquan has quit IRC | 04:15 | |
| *** shuquan has joined #openstack-nova | 04:16 | |
| *** jecarey has quit IRC | 04:17 | |
| *** boris-42 has joined #openstack-nova | 04:18 | |
| *** coreywright has quit IRC | 04:24 | |
| *** annashen has joined #openstack-nova | 04:26 | |
| *** shyama has joined #openstack-nova | 04:27 | |
| *** unicell1 has joined #openstack-nova | 04:28 | |
| *** unicell has quit IRC | 04:28 | |
| *** ijw has quit IRC | 04:29 | |
| *** annashen has quit IRC | 04:31 | |
| *** coreywright has joined #openstack-nova | 04:34 | |
| *** Kennan has quit IRC | 04:35 | |
| *** Kennan has joined #openstack-nova | 04:36 | |
| *** armax has joined #openstack-nova | 04:37 | |
| *** shuquan has quit IRC | 04:38 | |
| *** jamielennox|away is now known as jamielennox | 04:38 | |
| *** deepakcs has joined #openstack-nova | 04:39 | |
| *** davideagnello has joined #openstack-nova | 04:41 | |
| *** FL1SK has joined #openstack-nova | 04:45 | |
| *** davideagnello has quit IRC | 04:46 | |
| *** rotbeard has joined #openstack-nova | 04:50 | |
| *** tpeoples has quit IRC | 04:52 | |
| openstackgerrit | Jeffrey Zhang proposed openstack/nova: Use urlencode instead of dict_to_query_str function https://review.openstack.org/207468 | 04:54 |
| *** shuquan has joined #openstack-nova | 04:56 | |
| *** slong has quit IRC | 04:57 | |
| *** armax has quit IRC | 05:01 | |
| *** kiran-r has joined #openstack-nova | 05:03 | |
| *** salv-orlando has joined #openstack-nova | 05:03 | |
| *** slong has joined #openstack-nova | 05:04 | |
| *** salv-orlando has quit IRC | 05:06 | |
| *** salv-orlando has joined #openstack-nova | 05:11 | |
| *** chhavi has joined #openstack-nova | 05:13 | |
| *** lyanchih_ has joined #openstack-nova | 05:16 | |
| *** sudipto has joined #openstack-nova | 05:19 | |
| *** Sukhdev has joined #openstack-nova | 05:21 | |
| *** fawadkhaliq has joined #openstack-nova | 05:23 | |
| openstackgerrit | lyanchih proposed openstack/nova: libvirt: serial console ports count upper limit needs to be checked https://review.openstack.org/207373 | 05:25 |
| *** otter768 has joined #openstack-nova | 05:30 | |
| *** ig0r_ has joined #openstack-nova | 05:33 | |
| *** otter768 has quit IRC | 05:35 | |
| *** kmARC has joined #openstack-nova | 05:37 | |
| *** afazekas has joined #openstack-nova | 05:37 | |
| *** harlowja has quit IRC | 05:44 | |
| *** ig0r__ has joined #openstack-nova | 05:45 | |
| *** afazekas has quit IRC | 05:46 | |
| *** afazekas has joined #openstack-nova | 05:49 | |
| openstackgerrit | jichenjc proposed openstack/nova: Handle MessageTimeout to MigrationPreCheckError https://review.openstack.org/168635 | 05:49 |
| *** meghal has joined #openstack-nova | 05:53 | |
| *** ig0r_ has quit IRC | 05:53 | |
| *** lyanchih_ has quit IRC | 05:55 | |
| *** ig0r_ has joined #openstack-nova | 05:55 | |
| *** fawadk has joined #openstack-nova | 05:56 | |
| *** fawadkhaliq has quit IRC | 05:58 | |
| *** josecastroleon has joined #openstack-nova | 05:58 | |
| *** lyanchih_ has joined #openstack-nova | 05:59 | |
| *** ishant has joined #openstack-nova | 06:00 | |
| *** liusheng has quit IRC | 06:00 | |
| *** markvoelker has joined #openstack-nova | 06:01 | |
| *** liusheng has joined #openstack-nova | 06:02 | |
| *** meghal has quit IRC | 06:02 | |
| *** lennyb has joined #openstack-nova | 06:02 | |
| *** haigang has quit IRC | 06:03 | |
| *** haigang has joined #openstack-nova | 06:04 | |
| *** markvoelker has quit IRC | 06:05 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs: Clarifying more about lock interface in spec https://review.openstack.org/208326 | 06:08 |
| *** Kennan has quit IRC | 06:09 | |
| *** pcaruana has quit IRC | 06:09 | |
| *** Kennan has joined #openstack-nova | 06:11 | |
| *** shuquan has quit IRC | 06:13 | |
| *** shuquan has joined #openstack-nova | 06:14 | |
| *** haigang has quit IRC | 06:16 | |
| *** shuquan has quit IRC | 06:17 | |
| openstackgerrit | jichenjc proposed openstack/nova: Refactor test cases for live-migrate error case https://review.openstack.org/179048 | 06:17 |
| *** ankit_ag has joined #openstack-nova | 06:20 | |
| *** haigang has joined #openstack-nova | 06:20 | |
| *** shuquan has joined #openstack-nova | 06:22 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/197213 | 06:22 |
| *** scheuran has joined #openstack-nova | 06:23 | |
| *** belmoreira has joined #openstack-nova | 06:25 | |
| *** afazekas has quit IRC | 06:27 | |
| *** fawadk has quit IRC | 06:27 | |
| *** Murali has joined #openstack-nova | 06:30 | |
| *** afazekas has joined #openstack-nova | 06:31 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Allow resize root_gb to 0 for volume-backed instances https://review.openstack.org/207794 | 06:35 |
| *** Murali_ has joined #openstack-nova | 06:38 | |
| *** shuquan has quit IRC | 06:38 | |
| *** Murali has quit IRC | 06:39 | |
| *** Murali_ is now known as Murali | 06:40 | |
| *** Murali_ has joined #openstack-nova | 06:42 | |
| *** rdopiera has joined #openstack-nova | 06:44 | |
| *** Murali has quit IRC | 06:44 | |
| *** Murali_ is now known as Murali | 06:45 | |
| *** sudipto has quit IRC | 06:45 | |
| openstackgerrit | Radomir Dopieralski proposed openstack/nova: Handle SSL termination proxies for version list https://review.openstack.org/206479 | 06:51 |
| *** kmARC has quit IRC | 06:56 | |
| *** gibi has joined #openstack-nova | 06:57 | |
| *** sahid has joined #openstack-nova | 07:00 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Move "versions" functional tests in v2.1 tests https://review.openstack.org/201439 | 07:00 |
| *** sahid has quit IRC | 07:02 | |
| *** garyk has joined #openstack-nova | 07:02 | |
| *** browne1 has quit IRC | 07:04 | |
| *** sahid has joined #openstack-nova | 07:06 | |
| *** Sukhdev has quit IRC | 07:07 | |
| *** markus_z has joined #openstack-nova | 07:09 | |
| *** boris-42 has quit IRC | 07:10 | |
| *** achanda has quit IRC | 07:13 | |
| *** achanda has joined #openstack-nova | 07:17 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: DB:support for migration objects multi-statuses https://review.openstack.org/169604 | 07:18 |
| *** salv-orlando has quit IRC | 07:22 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Fix link's href to consider osapi_compute_link_prefix https://review.openstack.org/207702 | 07:22 |
| *** jcoufal has joined #openstack-nova | 07:26 | |
| *** otter768 has joined #openstack-nova | 07:31 | |
| *** shuquan has joined #openstack-nova | 07:33 | |
| *** vivekd has joined #openstack-nova | 07:33 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Move "versions" functional tests in v2.1 tests https://review.openstack.org/201439 | 07:34 |
| *** otter768 has quit IRC | 07:36 | |
| *** achanda has quit IRC | 07:36 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Adds instance_uuid index for instance_system_metadata https://review.openstack.org/208277 | 07:36 |
| *** kjelly has quit IRC | 07:37 | |
| *** pcaruana has joined #openstack-nova | 07:40 | |
| *** jlanoux has joined #openstack-nova | 07:47 | |
| *** salv-orlando has joined #openstack-nova | 07:48 | |
| *** kmARC has joined #openstack-nova | 07:54 | |
| *** echirivella has quit IRC | 07:55 | |
| *** jamielennox is now known as jamielennox|away | 07:57 | |
| *** romainh has joined #openstack-nova | 07:58 | |
| *** moshele has joined #openstack-nova | 08:01 | |
| *** fawadkhaliq has joined #openstack-nova | 08:01 | |
| *** markvoelker has joined #openstack-nova | 08:02 | |
| *** moshele has quit IRC | 08:02 | |
| *** moshele_ has joined #openstack-nova | 08:02 | |
| *** markvoelker has quit IRC | 08:06 | |
| *** echirivella has joined #openstack-nova | 08:08 | |
| *** subscope has joined #openstack-nova | 08:10 | |
| openstackgerrit | Shiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI https://review.openstack.org/202605 | 08:13 |
| *** jistr has joined #openstack-nova | 08:15 | |
| *** danpb has joined #openstack-nova | 08:15 | |
| *** alex_klimov has joined #openstack-nova | 08:16 | |
| *** aix has joined #openstack-nova | 08:18 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: Fix deleting instance with incorrectly attached volume https://review.openstack.org/204231 | 08:18 |
| *** yassine has joined #openstack-nova | 08:20 | |
| garyk | johnthetubaguy: not sure if you are around. addressed the comments - https://review.openstack.org/180202 | 08:22 |
| *** annashen has joined #openstack-nova | 08:27 | |
| *** MarcF has joined #openstack-nova | 08:28 | |
| *** fawadkhaliq has quit IRC | 08:28 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: Fix deleting instance with incorrectly attached volume https://review.openstack.org/204231 | 08:28 |
| *** zhenguo has joined #openstack-nova | 08:29 | |
| *** fawadkhaliq has joined #openstack-nova | 08:30 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova: VMware: add serial port device https://review.openstack.org/149203 | 08:31 |
| *** annashen has quit IRC | 08:32 | |
| *** kjelly has joined #openstack-nova | 08:34 | |
| *** ftersin has joined #openstack-nova | 08:35 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/128940 | 08:37 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Fixed incorrect behaviour of method _check_instance_exists https://review.openstack.org/203037 | 08:37 |
| *** Kevin_Zheng_ has joined #openstack-nova | 08:39 | |
| *** huanxie has joined #openstack-nova | 08:39 | |
| Kevin_Zheng_ | Hi, could someone kindly review these patches: https://review.openstack.org/#/c/195372/ and https://review.openstack.org/#/c/202379/ , they are very useful fix and there are also a lot of +1s but yet not approved. Thanks a lot. | 08:39 |
| *** humble__ has joined #openstack-nova | 08:40 | |
| *** davideagnello has joined #openstack-nova | 08:43 | |
| *** humble_ has quit IRC | 08:44 | |
| *** davideagnello has quit IRC | 08:47 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova: VMware: map one nova-compute to one VC cluster https://review.openstack.org/103916 | 08:47 |
| *** jianghjw has joined #openstack-nova | 08:48 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Fixes Bug "destroy_vm fails with HyperVException" https://review.openstack.org/204007 | 08:55 |
| *** rubasov has joined #openstack-nova | 09:00 | |
| *** Nikolay_St has joined #openstack-nova | 09:02 | |
| *** fawadkhaliq has quit IRC | 09:03 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Fix collection of metadata for a snapshot of a volume-backed instance https://review.openstack.org/170243 | 09:05 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Consolidate initialization of instance snapshot metadata https://review.openstack.org/198959 | 09:05 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Simplify interface for creating snapshot of volume-backed instance https://review.openstack.org/198958 | 09:05 |
| *** fawadkhaliq has joined #openstack-nova | 09:06 | |
| *** ftersin has left #openstack-nova | 09:08 | |
| *** ftersin has joined #openstack-nova | 09:08 | |
| *** andreykurilin__ has joined #openstack-nova | 09:10 | |
| *** humble_ has joined #openstack-nova | 09:16 | |
| *** humble__ has quit IRC | 09:19 | |
| lexloofer | Morning | 09:20 |
| *** lexloofer is now known as lxsli | 09:20 | |
| lxsli | Ahem | 09:20 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Fixed random failing of test_describe_instances_with_filters_tags https://review.openstack.org/208253 | 09:20 |
| *** atuvenie has joined #openstack-nova | 09:25 | |
| *** Jeffrey4l has quit IRC | 09:26 | |
| *** humble__ has joined #openstack-nova | 09:27 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Fixed random failing of test_describe_instances_with_filters_tags https://review.openstack.org/208253 | 09:28 |
| *** abhishekk has joined #openstack-nova | 09:30 | |
| *** humble_ has quit IRC | 09:31 | |
| *** kumar has joined #openstack-nova | 09:31 | |
| *** otter768 has joined #openstack-nova | 09:32 | |
| *** sudipto has joined #openstack-nova | 09:32 | |
| *** fawadkhaliq has quit IRC | 09:32 | |
| *** fawadkhaliq has joined #openstack-nova | 09:33 | |
| *** jianghjw has quit IRC | 09:35 | |
| *** otter768 has quit IRC | 09:37 | |
| *** Jeffrey4l has joined #openstack-nova | 09:39 | |
| *** rushiagr_away is now known as rushiagr | 09:39 | |
| *** kumar has quit IRC | 09:39 | |
| *** tojuvone has joined #openstack-nova | 09:40 | |
| *** gszasz has joined #openstack-nova | 09:40 | |
| *** fawadkhaliq has quit IRC | 09:45 | |
| *** fawadkhaliq has joined #openstack-nova | 09:45 | |
| *** marzif_ has quit IRC | 09:46 | |
| *** marzif_ has joined #openstack-nova | 09:47 | |
| *** moshele_ has quit IRC | 09:48 | |
| *** marzif_ has quit IRC | 09:48 | |
| *** shuquan has quit IRC | 09:49 | |
| *** kjelly has quit IRC | 09:50 | |
| *** romainh has quit IRC | 09:53 | |
| *** rdopiera has quit IRC | 09:54 | |
| openstackgerrit | lyanchih proposed openstack/nova: Add unit tests for libvirt fibrechannel/iser/iscsi/aoe volume drivers after os-brick https://review.openstack.org/208401 | 09:56 |
| *** marzif has joined #openstack-nova | 09:57 | |
| *** ftersin_ has joined #openstack-nova | 09:57 | |
| *** ftersin_ has quit IRC | 09:58 | |
| *** fawadkhaliq has quit IRC | 09:58 | |
| *** ftersin has quit IRC | 09:59 | |
| *** salv-orlando has quit IRC | 09:59 | |
| *** jcoufal_ has joined #openstack-nova | 10:02 | |
| *** markvoelker has joined #openstack-nova | 10:02 | |
| *** eliqiao has joined #openstack-nova | 10:03 | |
| *** sabari- has joined #openstack-nova | 10:06 | |
| *** kylek3h_ has joined #openstack-nova | 10:06 | |
| *** aix has quit IRC | 10:06 | |
| *** markvoelker has quit IRC | 10:07 | |
| *** ndipanov has joined #openstack-nova | 10:07 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Adds RemoteFX support to the Hyper-V driver https://review.openstack.org/42529 | 10:07 |
| *** Kennan has quit IRC | 10:07 | |
| *** FL1SK has quit IRC | 10:07 | |
| *** sabari has quit IRC | 10:08 | |
| *** alex_klimov has quit IRC | 10:08 | |
| *** jcoufal has quit IRC | 10:08 | |
| *** jmccrory has quit IRC | 10:08 | |
| *** logan2 has quit IRC | 10:08 | |
| *** mwagner_lap has quit IRC | 10:08 | |
| *** rushiagr has quit IRC | 10:08 | |
| *** timello has quit IRC | 10:08 | |
| *** HenryG has quit IRC | 10:08 | |
| *** garyk has quit IRC | 10:08 | |
| *** sabari- is now known as sabari | 10:08 | |
| *** garyk has joined #openstack-nova | 10:08 | |
| *** jmccrory has joined #openstack-nova | 10:08 | |
| *** HenryG has joined #openstack-nova | 10:08 | |
| *** kylek3h has quit IRC | 10:08 | |
| *** timello has joined #openstack-nova | 10:08 | |
| *** mwagner_lap has joined #openstack-nova | 10:08 | |
| *** logan2 has joined #openstack-nova | 10:08 | |
| *** pixelbeat_ has joined #openstack-nova | 10:08 | |
| *** alex_klimov has joined #openstack-nova | 10:09 | |
| *** Kennan has joined #openstack-nova | 10:09 | |
| openstackgerrit | lyanchih proposed openstack/nova: libvirt: serial console ports count upper limit needs to be checked https://review.openstack.org/207373 | 10:09 |
| *** Jeffrey4l has quit IRC | 10:10 | |
| haigang | anybody? | 10:13 |
| *** rdo has quit IRC | 10:14 | |
| *** rdo has joined #openstack-nova | 10:16 | |
| garyk | any cores around to look at https://review.openstack.org/179066 - it has 2 +2 and lost the +A as it was rebased | 10:16 |
| *** ftersin has joined #openstack-nova | 10:17 | |
| *** jyuso1 has quit IRC | 10:17 | |
| *** josecastroleon has quit IRC | 10:20 | |
| *** huanxie has quit IRC | 10:20 | |
| *** fawadkhaliq has joined #openstack-nova | 10:21 | |
| *** lyanchih_ has quit IRC | 10:22 | |
| sudipto | lyanchih, hi, w.r.t your comment on https://review.openstack.org/#/c/207373/4/nova/virt/libvirt/driver.py - i didn't mean to ask you to remove the virt_type check - did i? | 10:25 |
| openstackgerrit | garyk proposed openstack/nova: VMware: improve instance names on VC https://review.openstack.org/166608 | 10:28 |
| openstackgerrit | garyk proposed openstack/nova: VMware: add in folder support on VC https://review.openstack.org/165060 | 10:28 |
| *** annashen has joined #openstack-nova | 10:28 | |
| *** eliqiao has quit IRC | 10:29 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: Fix deleting instance with incorrectly attached volume https://review.openstack.org/204231 | 10:31 |
| *** moshele_ has joined #openstack-nova | 10:31 | |
| *** moshele_ has quit IRC | 10:32 | |
| *** moshele_ has joined #openstack-nova | 10:32 | |
| *** annashen has quit IRC | 10:33 | |
| moshele_ | danpb: can you review again https://review.openstack.org/#/c/187052/ I merge conflict and also I addressed all you comments in https://review.openstack.org/#/c/199488/ | 10:33 |
| *** dimsum__ has joined #openstack-nova | 10:34 | |
| garyk | are people also find it difficult to upload patches? | 10:44 |
| *** haigang has quit IRC | 10:44 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: ensure that get_info raises the correct exception https://review.openstack.org/191558 | 10:45 |
| mnestratov | garyk: yep, it takes a bit longer than usual | 10:46 |
| _gryf | any cores around to look at the patch https://review.openstack.org/#/c/197063/ ? I need the rebase on the patch that is depend on it and don't want loose +2 already there… | 10:46 |
| garyk | mnestratov: ok, thanks. it is really bad at the moment | 10:46 |
| *** haigang has joined #openstack-nova | 10:48 | |
| *** kjelly has joined #openstack-nova | 10:49 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: treat deletion exception with attached volumes https://review.openstack.org/198674 | 10:50 |
| *** pcaruana has quit IRC | 10:57 | |
| *** mwagner_lap has quit IRC | 10:57 | |
| *** josecastroleon has joined #openstack-nova | 11:02 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds VolumeOps unit tests https://review.openstack.org/139797 | 11:02 |
| *** claudiub has joined #openstack-nova | 11:03 | |
| *** coolsvap is now known as coolsvap|away | 11:06 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: treat deletion exception with attached volumes https://review.openstack.org/198674 | 11:08 |
| openstackgerrit | garyk proposed openstack/nova: VMware: ensure that get_info raises the correct exception https://review.openstack.org/191558 | 11:08 |
| *** marzif has quit IRC | 11:09 | |
| *** marzif has joined #openstack-nova | 11:10 | |
| *** Marga__ has quit IRC | 11:13 | |
| *** rushiagr has joined #openstack-nova | 11:13 | |
| *** pcaruana has joined #openstack-nova | 11:14 | |
| *** dheeraj-gupta-4 has joined #openstack-nova | 11:16 | |
| *** thorst has joined #openstack-nova | 11:17 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/128940 | 11:17 |
| *** dims_ has joined #openstack-nova | 11:18 | |
| *** dimsum__ has quit IRC | 11:20 | |
| garyk | johnthetubaguy: can you please look at https://review.openstack.org/180202 | 11:21 |
| *** marzif has quit IRC | 11:27 | |
| *** jaypipes has joined #openstack-nova | 11:27 | |
| *** edmondsw has joined #openstack-nova | 11:31 | |
| *** kiran-r has quit IRC | 11:31 | |
| openstackgerrit | Shiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI https://review.openstack.org/202617 | 11:32 |
| *** otter768 has joined #openstack-nova | 11:33 | |
| *** markvoelker has joined #openstack-nova | 11:33 | |
| *** otter768 has quit IRC | 11:37 | |
| *** markvoelker has quit IRC | 11:38 | |
| *** rdo has quit IRC | 11:41 | |
| *** sudipto has quit IRC | 11:41 | |
| *** dheeraj-gupta-4 has quit IRC | 11:42 | |
| *** rdo has joined #openstack-nova | 11:42 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Fixed random failing of test_describe_instances_with_filters_tags https://review.openstack.org/208253 | 11:47 |
| *** smatzek has joined #openstack-nova | 11:48 | |
| *** marcusvrn has joined #openstack-nova | 11:49 | |
| *** markvoelker has joined #openstack-nova | 11:50 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add instance action events for live migration https://review.openstack.org/195372 | 11:50 |
| *** wanghua has joined #openstack-nova | 11:50 | |
| *** garyk has quit IRC | 11:50 | |
| *** shyama has quit IRC | 11:50 | |
| *** romainh has joined #openstack-nova | 11:51 | |
| *** shyama has joined #openstack-nova | 11:51 | |
| *** humble__ has quit IRC | 11:53 | |
| *** markvoelker_ has joined #openstack-nova | 11:54 | |
| *** markvoelker has quit IRC | 11:54 | |
| *** dims_ has quit IRC | 11:54 | |
| *** dimsum__ has joined #openstack-nova | 11:55 | |
| *** claudiub has quit IRC | 11:55 | |
| *** htruta_ has joined #openstack-nova | 11:58 | |
| *** coolsvap|away is now known as coolsvap | 11:58 | |
| *** pixelbeat_ has quit IRC | 12:00 | |
| *** garyk has joined #openstack-nova | 12:01 | |
| *** FL1SK has joined #openstack-nova | 12:03 | |
| *** fawadkhaliq has quit IRC | 12:04 | |
| *** Marga_ has joined #openstack-nova | 12:05 | |
| *** dimsum__ has quit IRC | 12:08 | |
| *** dprince has joined #openstack-nova | 12:08 | |
| *** chhavi has quit IRC | 12:08 | |
| garyk | danpb: any chance you can please look at https://review.openstack.org/179066 (it was rebased) | 12:13 |
| *** raildo has joined #openstack-nova | 12:18 | |
| *** rfolco has joined #openstack-nova | 12:21 | |
| *** dimsum__ has joined #openstack-nova | 12:22 | |
| *** jasondotstar has quit IRC | 12:22 | |
| *** marzif has joined #openstack-nova | 12:24 | |
| *** marzif has quit IRC | 12:24 | |
| *** marzif has joined #openstack-nova | 12:25 | |
| *** andreykurilin__ has quit IRC | 12:25 | |
| *** dims_ has joined #openstack-nova | 12:27 | |
| *** apuimedo has joined #openstack-nova | 12:28 | |
| *** dims__ has joined #openstack-nova | 12:29 | |
| *** dimsum__ has quit IRC | 12:29 | |
| *** annashen has joined #openstack-nova | 12:29 | |
| *** Marga_ has quit IRC | 12:30 | |
| *** dimsum__ has joined #openstack-nova | 12:32 | |
| *** marzif_ has joined #openstack-nova | 12:32 | |
| *** dims_ has quit IRC | 12:32 | |
| *** jasondotstar has joined #openstack-nova | 12:32 | |
| johnthetubaguy | garyk: been away, in a field, but back now, I can take a quick look at the ones that need +2 reapplying, etc | 12:33 |
| *** marzif has quit IRC | 12:33 | |
| *** annashen has quit IRC | 12:33 | |
| *** dims__ has quit IRC | 12:34 | |
| garyk | johnthetubaguy: camping? | 12:34 |
| johnthetubaguy | garyk: yeah, well at a folk festival, and camping their | 12:35 |
| *** ferest has joined #openstack-nova | 12:35 | |
| *** ferest has quit IRC | 12:36 | |
| *** pixelbeat_ has joined #openstack-nova | 12:36 | |
| johnthetubaguy | s/their/there/ | 12:36 |
| johnthetubaguy | one day, I will learn english | 12:37 |
| * StevenK waits for "Today is not this day" | 12:37 | |
| johnthetubaguy | garyk: have you got any patches that lost their +W from rebasing, etc? | 12:38 |
| garyk | johnthetubaguy: https://review.openstack.org/179066 | 12:38 |
| *** Jeffrey4l has joined #openstack-nova | 12:39 | |
| _gryf | johnthetubaguy, hi | 12:39 |
| garyk | nice. hope the weather was fine | 12:39 |
| johnthetubaguy | garyk: some how, luck was on my side, it was nice but not too sunny, pretty much perfect, I mean the nights got colder than I would have chosen, but nothing a jumper didn't fix :) | 12:40 |
| johnthetubaguy | _gryf: hi | 12:40 |
| _gryf | johnthetubaguy, i have a qst: if the implementation for my non-priority BP is in the middle of revew (two patches merged, one have one +2 and the last still is ongoing) that is mean that I have to wait for the next window (m release)? | 12:40 |
| *** rdopiera has joined #openstack-nova | 12:41 | |
| *** rdopiera has quit IRC | 12:41 | |
| *** rdopiera_ has joined #openstack-nova | 12:41 | |
| johnthetubaguy | _gryf: possibly, I am going to start looking through all the blueprints, and starting to postpone things today | 12:42 |
| johnthetubaguy | _gryf: I will let you know how close it looks | 12:42 |
| _gryf | johnthetubaguy, hm. okay, although it doesn't sounds good :/ | 12:42 |
| johnthetubaguy | _gryf: given all the problems we had with Zuul at the end of last week, I am going easy on the deadline, but we need to wrap things up this week, if possible | 12:42 |
| *** andreykurilin__ has joined #openstack-nova | 12:43 | |
| *** zhenguo has quit IRC | 12:43 | |
| _gryf | johnthetubaguy, I'm waiting with the implementation for ~2 months to be reviewed | 12:44 |
| johnthetubaguy | _gryf: yeah, apologies, we need more folks to do more reviews really, we are hurting because of that right now. I am trying to encourage more reviews, but its proving hard work | 12:44 |
| *** davideagnello has joined #openstack-nova | 12:44 | |
| *** dims_ has joined #openstack-nova | 12:45 | |
| johnthetubaguy | _gryf: so this doesn't make it any better or easier, but for context we have things that were up for review during kilo (and possibly in same cases juno?), that are getting delayed to M at this point, all of them super useful, just not part of the agreed priority list right now, and that sucks, but we need to get bugs fixed, and need to get our priorities worked on :( | 12:45 |
| *** dimsum__ has quit IRC | 12:46 | |
| *** aysyd has joined #openstack-nova | 12:47 | |
| moshele_ | johnthetubaguy: can you review https://review.openstack.org/#/c/187052/ it had +2 from danpb but it is lost in the rebase, also I think the commit it ready for core reviewers https://review.openstack.org/#/c/199488/ | 12:48 |
| johnthetubaguy | jaypipes: I think we were doing a bit of ping-pong on this review, does the outcome make sense to you? https://review.openstack.org/#/c/180202 | 12:49 |
| *** mnestratov has quit IRC | 12:49 | |
| *** davideagnello has quit IRC | 12:49 | |
| *** mnestratov has joined #openstack-nova | 12:49 | |
| *** baoli has joined #openstack-nova | 12:49 | |
| *** kylek3h_ has quit IRC | 12:49 | |
| johnthetubaguy | moshele_: sorry, I can't seem that happening today, I have hour and hours of work to do sorting out the blueprint freeze, and seeing where we are all at, just reapplying votes I made previously really | 12:50 |
| *** mdrabe has joined #openstack-nova | 12:50 | |
| *** Jeffrey4l_ has joined #openstack-nova | 12:51 | |
| moshele_ | johnthetubaguy: I will be happy if it will this week :), but take your time | 12:52 |
| *** dims_ has quit IRC | 12:53 | |
| jaypipes | johnthetubaguy: me today: http://www.memes.com/img/510571 | 12:53 |
| *** dimsum__ has joined #openstack-nova | 12:54 | |
| *** Jeffrey4l has quit IRC | 12:54 | |
| jaypipes | johnthetubaguy: I thought I was pretty adamant about that being net_io_, not vif_ ... | 12:54 |
| johnthetubaguy | jaypipes: OK so I have a vision of you and a load of pugs all lying on the floor in the same pose | 12:54 |
| jaypipes | johnthetubaguy: is there a reason it was changed back? | 12:55 |
| *** dimsum__ is now known as dims | 12:55 | |
| *** jeffrey__ has joined #openstack-nova | 12:55 | |
| jaypipes | johnthetubaguy: heh, yes, that definitely is the case some days :) | 12:55 |
| johnthetubaguy | jaypipes: I think just because I said I think the other way is better for consistency, but thats not really a good answer, I think the spec review was thinking we want it to match the ones we have in libvirt that are kinda similar | 12:56 |
| johnthetubaguy | jaypipes: I suspect there is something big I am missing here | 12:56 |
| jaypipes | johnthetubaguy: the reason I was adamant about that renaming is because a) vif_limit makes it looks like its a number of virtual interfaces that is the limit, not the network I/O bandwidth, and b) disk_io_limit is used for disk I/O. | 12:56 |
| *** browne has joined #openstack-nova | 12:57 | |
| *** Jeffrey4l_ has quit IRC | 12:58 | |
| johnthetubaguy | jaypipes: yeah, I actually agree with you, although that does run the risk of network throughput vs nic through put when you have two nics on the same network, but I have less issues with that one, its just libvirt already uses the vif_*_limit name, so I was thinking it should be equally strange to save confusion (I think hyper-v has a spec approved to copy that name too) | 12:58 |
| jaypipes | johnthetubaguy: hmmm... | 12:59 |
| jaypipes | danpb: around? | 12:59 |
| *** flip214 is now known as flip562 | 12:59 | |
| *** flip562 is now known as flip214 | 12:59 | |
| lxsli | good morning Jay! when you're done discussing vif | 12:59 |
| *** otter768 has joined #openstack-nova | 13:00 | |
| jaypipes | lxsli: yup (though we have scheduler meeting in an hour right?) | 13:00 |
| lxsli | jaypipes: how about that, indeed | 13:01 |
| *** markvoelker_ has quit IRC | 13:02 | |
| lxsli | sometime today would be nice, otherwise all good | 13:02 |
| *** markvoelker has joined #openstack-nova | 13:04 | |
| johnthetubaguy | jaypipes: I should clarify, I think its our libvirt driver that uses that, rather than libvirt | 13:04 |
| *** rlrossit has joined #openstack-nova | 13:08 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Moves OVS related code to a common location https://review.openstack.org/179727 | 13:08 |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver https://review.openstack.org/140045 | 13:08 |
| *** aix has joined #openstack-nova | 13:11 | |
| *** dustins has joined #openstack-nova | 13:13 | |
| *** browne has quit IRC | 13:13 | |
| garyk | anyone know why gerrit is so painfully slow | 13:13 |
| StevenK | Oh, I thought it was just this hotel wifi being terrible. It's not just me! | 13:14 |
| *** dane-fichter has joined #openstack-nova | 13:16 | |
| *** haigang has quit IRC | 13:17 | |
| *** kylek3h has joined #openstack-nova | 13:17 | |
| *** romainh has left #openstack-nova | 13:18 | |
| garyk | nope. reviewing and posting patches is taking so long …. | 13:20 |
| *** chuckcarmack has joined #openstack-nova | 13:20 | |
| *** zul has joined #openstack-nova | 13:21 | |
| *** boris-42 has joined #openstack-nova | 13:22 | |
| johnthetubaguy | garyk: this doesn't help really, but same here :( | 13:22 |
| johnthetubaguy | it was my internet connection the other day, but this is different | 13:23 |
| johnthetubaguy | or feels like it is | 13:23 |
| garyk | it has been like this for the last 2 days. but today is far worse | 13:23 |
| *** haomaiw__ has joined #openstack-nova | 13:24 | |
| garyk | maybe winging on infra will help | 13:24 |
| dane-fichter | gerrit is like molasses today | 13:24 |
| *** zzzeek has joined #openstack-nova | 13:24 | |
| tojuvone | johnthetubaguy: to be on safe side and not to overload reviews today, I guess do not have to have medium priority BP code reviewed today, just if BP with low priority? | 13:25 |
| garyk | ttx: maybe you know who can look into gerrit performance issue…. openstack is imploding | 13:26 |
| johnthetubaguy | tojuvone: in theory you should be fine, certainly for today you should be fine | 13:26 |
| *** ig0r_ has quit IRC | 13:26 | |
| *** ig0r__ has quit IRC | 13:27 | |
| *** haomaiwang has quit IRC | 13:27 | |
| *** bapalm has joined #openstack-nova | 13:27 | |
| ttx | looks like they are not up yet | 13:28 |
| tojuvone | Thanks :) | 13:28 |
| *** abhishekk has quit IRC | 13:28 | |
| *** mjturek1 has joined #openstack-nova | 13:31 | |
| *** pedroalvarez has quit IRC | 13:32 | |
| *** ig0r_ has joined #openstack-nova | 13:32 | |
| *** otter768 has quit IRC | 13:33 | |
| *** ig0r__ has joined #openstack-nova | 13:33 | |
| *** ctrath has joined #openstack-nova | 13:33 | |
| *** pedroalvarez has joined #openstack-nova | 13:33 | |
| lxsli | lascii: would you be so kind as to twiddle +A on https://review.openstack.org/#/c/202640/ please? It seems to have fallen out of the gate queue and reverify hasn't worked | 13:35 |
| lxsli | oh, belay that | 13:36 |
| *** rushiagr is now known as rushiagr_away | 13:36 | |
| *** erlon has joined #openstack-nova | 13:36 | |
| *** lascii is now known as alaski | 13:39 | |
| *** mwagner_lap has joined #openstack-nova | 13:39 | |
| *** VW_ has joined #openstack-nova | 13:40 | |
| alaski | lxsli: it appears to have a merge conflict | 13:40 |
| raildo | melwitt:ping, I intend to send a FFE for nested quota driver, until tomorrow, so I saw an email in the ML about it. We just need to add the bp in the etherpad https://etherpad.openstack.org/p/liberty-nova-non-priority-feature-freeze and send an email explaining the reason? | 13:40 |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org has been restarted in an attempt to improve performance. | 13:40 | |
| *** jistr is now known as jistr|mtg | 13:41 | |
| *** tpeoples has joined #openstack-nova | 13:42 | |
| jaypipes | if anyone has a change to review https://review.openstack.org/#/c/184495/ and https://review.openstack.org/#/c/195088/, I'm sure tdurakov would be most appreciative. :) | 13:42 |
| *** Marga_ has joined #openstack-nova | 13:42 | |
| jaypipes | alaski, danpb, johnthetubaguy: ^^ | 13:42 |
| *** lucas___ has joined #openstack-nova | 13:42 | |
| tdurakov | jaypipes, thx, it would be great if these become merged:) | 13:43 |
| *** marzif_ has quit IRC | 13:43 | |
| jaypipes | tdurakov: thanks for hanging in there and being patient | 13:44 |
| *** ig0r__ has quit IRC | 13:44 | |
| *** marzif_ has joined #openstack-nova | 13:44 | |
| *** wverdugo500 has joined #openstack-nova | 13:44 | |
| *** ankit_ag has quit IRC | 13:45 | |
| garyk | alaski: can you please see the comments on https://review.openstack.org/187537. the security groups return a dict. i have added a follow up patch changing that to a object | 13:46 |
| superdan | garyk: so just drop the first patch and focus on the second, right? | 13:47 |
| superdan | no reason to change it and then re-change it, right? | 13:47 |
| *** superdan is now known as dansmith | 13:47 | |
| *** h00327910__ has quit IRC | 13:48 | |
| *** lyanchih_ has joined #openstack-nova | 13:48 | |
| *** sayali has joined #openstack-nova | 13:48 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: add in vif resource limitations https://review.openstack.org/180202 | 13:48 |
| openstackgerrit | garyk proposed openstack/nova: VMware: support limits via image meta data https://review.openstack.org/180942 | 13:48 |
| openstackgerrit | garyk proposed openstack/nova: Objects: add limits, reservations and shares to ImageMetaProps object https://review.openstack.org/188321 | 13:48 |
| alaski | agreed. just do it all at once | 13:49 |
| garyk | dansmith: alaski: ok, will do. | 13:49 |
| *** Nikolay_St has quit IRC | 13:49 | |
| *** browne has joined #openstack-nova | 13:50 | |
| *** vivekd has quit IRC | 13:50 | |
| *** Jeffrey4l_ has joined #openstack-nova | 13:51 | |
| *** bapalm_ has joined #openstack-nova | 13:51 | |
| *** READ10 has joined #openstack-nova | 13:51 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Raise NovaException for missing/empty machine-id https://review.openstack.org/202640 | 13:52 |
| *** jeffrey__ has quit IRC | 13:53 | |
| openstackgerrit | garyk proposed openstack/nova: Compute: use instance object for refresh_instance_security_rules https://review.openstack.org/187537 | 13:53 |
| garyk | dansmith: alaski: ^ | 13:53 |
| dansmith | garyk: see my review on that? | 13:54 |
| *** bapalm has quit IRC | 13:54 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation https://review.openstack.org/138255 | 13:54 |
| *** sayali_ has joined #openstack-nova | 13:55 | |
| garyk | dansmith: i just saw it now when i was abandoning. i'll update. thanks | 13:55 |
| *** sayali_ has quit IRC | 13:55 | |
| *** Jeffrey4l_ has quit IRC | 13:56 | |
| *** rotbeard has quit IRC | 13:56 | |
| *** tpeoples has quit IRC | 13:58 | |
| *** annegentle has joined #openstack-nova | 13:58 | |
| *** sayali has quit IRC | 13:58 | |
| *** andrewbogott has joined #openstack-nova | 13:59 | |
| *** sayalilunkad has joined #openstack-nova | 13:59 | |
| *** deepakcs has quit IRC | 13:59 | |
| sayalilunkad | whois sayalilunkad | 14:00 |
| *** haomaiw__ has quit IRC | 14:01 | |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: libvirt: Nova part of volume snapshot improvements https://review.openstack.org/172813 | 14:01 |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: objects: Add support for OnlineSnapshotInfo object https://review.openstack.org/194538 | 14:01 |
| *** haomaiwang has joined #openstack-nova | 14:02 | |
| openstackgerrit | garyk proposed openstack/nova: Compute: use instance object for refresh_instance_security_rules https://review.openstack.org/187537 | 14:02 |
| *** josecastroleon has quit IRC | 14:02 | |
| *** mriedem has joined #openstack-nova | 14:03 | |
| *** SourabhP has joined #openstack-nova | 14:05 | |
| *** sayalilunkad has quit IRC | 14:06 | |
| *** sayali has joined #openstack-nova | 14:06 | |
| *** browne has quit IRC | 14:08 | |
| *** Murali has quit IRC | 14:08 | |
| *** burt has joined #openstack-nova | 14:08 | |
| *** sayali has quit IRC | 14:09 | |
| *** sayali has joined #openstack-nova | 14:09 | |
| *** garyk has quit IRC | 14:09 | |
| *** sayali has quit IRC | 14:10 | |
| *** sayalilunkad has joined #openstack-nova | 14:10 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/128940 | 14:10 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Fixed incorrect behaviour of method _check_instance_exists https://review.openstack.org/203037 | 14:10 |
| *** afazekas has quit IRC | 14:12 | |
| lyanchih_ | Hi guys, please help to review our patch https://review.openstack.org/#/c/202436/ | 14:12 |
| lyanchih_ | Which is race conditions about security-group additions | 14:12 |
| *** xyang2 has joined #openstack-nova | 14:13 | |
| lyanchih_ | I was using utils synchronized to avoid race condtion, if it is not adaptive please tell me | 14:13 |
| *** SourabhP_ has joined #openstack-nova | 14:14 | |
| *** rdopiera_ has quit IRC | 14:14 | |
| *** ig0r__ has joined #openstack-nova | 14:14 | |
| *** SourabhP has quit IRC | 14:16 | |
| *** SourabhP_ is now known as SourabhP | 14:16 | |
| *** xyang2 has quit IRC | 14:19 | |
| *** MarcF has left #openstack-nova | 14:19 | |
| *** xyang2 has joined #openstack-nova | 14:20 | |
| *** jistr|mtg is now known as jistr | 14:20 | |
| dansmith | lyanchih_: so, this adds a lock around quite a bit of stuff that gets called from the API | 14:20 |
| dansmith | lyanchih_: which I think we try to avoid because it blocks the API server | 14:20 |
| *** kjelly has quit IRC | 14:21 | |
| dansmith | I don't actually see that we do that anywhere else | 14:21 |
| dansmith | alaski: thoughts on that? I'm sure the patch fixes some races, but a lot of add/remove secgroup calls could block a bunch of api workers waiting on neutron.. | 14:22 |
| *** tojuvone has quit IRC | 14:23 | |
| raildo | jaypipes: jogo, melwitt can you put this patch in your review list? :) https://review.openstack.org/#/c/151327/ | 14:24 |
| jaypipes | raildo: FYI, jogo no longer works on OpenStack... :) he's at Pinterest now doing Pinteresting things. | 14:24 |
| alaski | dansmith: yeah. locking code that's called from the API is going to be troublesome for us | 14:24 |
| lyanchih_ | synchronized will block all API!? | 14:24 |
| dansmith | alaski: okay, that's what I figured | 14:25 |
| alaski | well, that's the other thing. there's no guarantee that this will help because if there are multiple services running the lock doesn't cross them all | 14:25 |
| raildo | jaypipes: ops, I didn't know that... thanks for the informaatino | 14:25 |
| dansmith | alaski: yeah, it assumes shared storage for the locks | 14:25 |
| raildo | information* | 14:25 |
| dansmith | which is never going to be the case for api | 14:25 |
| openstackgerrit | Jeremy Stanley proposed openstack/nova: IGNORE: Add bindep dependencies list https://review.openstack.org/164444 | 14:25 |
| *** rdopiera has joined #openstack-nova | 14:25 | |
| *** unicell has joined #openstack-nova | 14:26 | |
| johnthetubaguy | sahid: do you know where we are at with this patch? https://review.openstack.org/#/c/196620 | 14:26 |
| alaski | lyanchih_: it would block another call that needs the lock, but there are timeouts on the API so locking could cause those to get hit | 14:26 |
| lyanchih_ | Should we call the function by compute instead api? | 14:26 |
| alaski | lyanchih_: compute is the only place I think it's reasonable to lock right now | 14:27 |
| *** unicell1 has quit IRC | 14:27 | |
| dansmith | yes, but | 14:27 |
| lyanchih_ | If I don’t remeber wrong, nova-network security group was handle by compute | 14:28 |
| dansmith | funneling all neutron security group operations down to a compute is not that great either | 14:28 |
| dansmith | the better solution would be to refactor the code so that it doesn't try to do a full load/save of the security groups for a port such that we don't have the problem at all | 14:28 |
| alaski | yeah. not if it's only to get a lock | 14:28 |
| *** lucas___ has quit IRC | 14:29 | |
| dansmith | which probably requires a change to neutron's api, but that's really a much better solution in the long run | 14:29 |
| *** annashen has joined #openstack-nova | 14:30 | |
| alaski | agreed | 14:30 |
| sahid | johnthetubaguy: the whole serie is waiting to be approved | 14:30 |
| johnthetubaguy | sahid: that patch needs updating those, I think its a quick change | 14:31 |
| sahid | johnthetubaguy: the comment from Stephen as already been addressed at the beginning | 14:31 |
| johnthetubaguy | sahid: alex_xu was pointing out this stuff, I think: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6300 | 14:31 |
| *** xyang2 has quit IRC | 14:31 | |
| mriedem | dansmith: i had a trivial rebase for this series over the weekend, https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/consolidate-libvirt-fs-volume-drivers,n,z it was approved, can you re-slap your +W on those? | 14:31 |
| dansmith | yeah | 14:31 |
| *** xyang2 has joined #openstack-nova | 14:32 | |
| *** afazekas has joined #openstack-nova | 14:32 | |
| openstackgerrit | Simona Iuliana Toader proposed openstack/nova: Hyper-V: Adds host maintenance implementation https://review.openstack.org/204157 | 14:32 |
| *** rushiagr_away is now known as rushiagr | 14:32 | |
| rlrossit | dansmith: mriedem: Jenkins is finally once again happy with https://review.openstack.org/#/c/198730/ after rebasing and rechecking, so can I get your guys's blessing again? | 14:32 |
| johnthetubaguy | sahid: and here: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2232 | 14:33 |
| *** lucas___ has joined #openstack-nova | 14:33 | |
| *** lucas___ has quit IRC | 14:33 | |
| *** lucas___ has joined #openstack-nova | 14:33 | |
| *** annashen has quit IRC | 14:35 | |
| dansmith | rlrossit: yep | 14:35 |
| johnthetubaguy | sahid: does that make sense? | 14:35 |
| sahid | johnthetubaguy: ok let me addresse a patch to remove NovaException and be more specific on my change | 14:35 |
| *** jecarey has joined #openstack-nova | 14:37 | |
| sahid | johnthetubaguy: actually QemuGuestAgentNotEnabled inherits from NovaExcetpion so that should be OK but we can be more specific yes | 14:37 |
| johnthetubaguy | sahid: i think its actually quite a good to have the separate exception, just that breaks the error handling in its current form | 14:37 |
| openstackgerrit | Pengtao Huang proposed openstack/nova: change the value "all_mappings" to "block_device_mapping" https://review.openstack.org/208511 | 14:38 |
| *** dannywilson has joined #openstack-nova | 14:39 | |
| lyanchih_ | dansmith: I also though it’s greate to add neutron API which can add/remove security-gropu instead of setting full data. So do I need to send a blue print to neutron first? | 14:39 |
| sahid | well yes, the code is not false as it is, we can just be more sepcific and so an other change could be adressed after this one to avoid re-looking for +2 | 14:39 |
| dansmith | lyanchih_: I don't know their process, but probably something like that yeah | 14:40 |
| sahid | hum yes you already -1ed, ok will do that change | 14:40 |
| *** sayalilunkad has quit IRC | 14:40 | |
| *** sayali has joined #openstack-nova | 14:41 | |
| johnthetubaguy | sahid: the code is false in its current form though right, you get an exception raised, where it should just be returning false right? | 14:42 |
| lyanchih_ | dansmith: Ok, thanks. I will checkout it out. | 14:42 |
| dansmith | lyanchih: cool, thanks | 14:42 |
| johnthetubaguy | sahid: or maybe I miss-read the logic? | 14:42 |
| sahid | johnthetubaguy: ah i did not noticed that, looking | 14:42 |
| *** wanghua has quit IRC | 14:43 | |
| johnthetubaguy | sahid: its this bit https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2232 | 14:43 |
| nagyz | mriedem, so the tempest scenario runs: http://logs.openstack.org/76/206576/4/check/gate-tempest-dsvm-full-ceph/8b15cbb/console.html.gz#_2015-07-31_21_48_03_043 | 14:43 |
| nagyz | mriedem, I guess the next step is for me to write the unit tests and refactor it into it's own module under volumes/? | 14:44 |
| johnthetubaguy | sahid: it would get an unexpected RPC error, I think, in its current form, and would just raise that | 14:44 |
| *** Kevin_Zheng_ has quit IRC | 14:45 | |
| *** ig0r_ has quit IRC | 14:45 | |
| *** davideagnello has joined #openstack-nova | 14:45 | |
| *** davideagnello has quit IRC | 14:49 | |
| *** SourabhP has quit IRC | 14:50 | |
| sahid | johnthetubaguy: yeah... now i underqtand we should to cache that exception since in the except we look at os_require_quiesce to know whether that feature have been requested | 14:51 |
| sahid | if it's the case we re raise if not we skip | 14:51 |
| sahid | s/cache/catch | 14:51 |
| mriedem | nagyz: looks pretty good | 14:51 |
| lyanchih_ | Hi, where can I find metting note in Philly midcycle | 14:51 |
| mriedem | nagyz: so yeah, i'd say push forward | 14:52 |
| lyanchih_ | There is one comment in https://bugs.launchpad.net/nova/+bug/1420274, which said the bug will discuss in midcycle. | 14:52 |
| openstack | Launchpad bug 1420274 in OpenStack Compute (nova) "The ephemeral disk and the swap disk locally in the nova-compute node when image_type=rbd" [Low,Confirmed] - Assigned to Chung Chih, Hung (lyanchih) | 14:52 |
| *** achanda has joined #openstack-nova | 14:53 | |
| *** schwicke has joined #openstack-nova | 14:53 | |
| *** mtanino has joined #openstack-nova | 14:54 | |
| nagyz | mriedem, ok I'll take tomorrow to write the tests, thanks | 14:55 |
| *** shyama has quit IRC | 14:56 | |
| johnthetubaguy | sahid: yeah, and needs to be noted in compute manager, so its makes it across the RPC wire right? | 14:57 |
| *** barra204 has joined #openstack-nova | 14:57 | |
| sahid | johnthetubaguy: yep | 14:58 |
| *** haomaiwang has quit IRC | 15:01 | |
| *** tpeoples has joined #openstack-nova | 15:01 | |
| *** SourabhP has joined #openstack-nova | 15:01 | |
| *** tonytan4ever has joined #openstack-nova | 15:02 | |
| *** haomaiwang has joined #openstack-nova | 15:02 | |
| *** jasondotstar has quit IRC | 15:02 | |
| *** achanda has quit IRC | 15:04 | |
| *** agireud has joined #openstack-nova | 15:04 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Don't check flavor disk attributes when booting from volume https://review.openstack.org/204635 | 15:06 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Use original image size in volume image metadata https://review.openstack.org/208523 | 15:06 |
| *** mdrabe has quit IRC | 15:07 | |
| *** mdrabe has joined #openstack-nova | 15:07 | |
| *** ddaskal has joined #openstack-nova | 15:07 | |
| *** ig0r__ has quit IRC | 15:09 | |
| *** barra204 has quit IRC | 15:09 | |
| *** afazekas has quit IRC | 15:10 | |
| mdbooth | Sigh, just realised I forgot to run pep8 :( | 15:10 |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Moves OVS related code to a common location https://review.openstack.org/179727 | 15:12 |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver https://review.openstack.org/140045 | 15:12 |
| *** pcaruana has quit IRC | 15:13 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation https://review.openstack.org/138255 | 15:14 |
| *** thumpba has joined #openstack-nova | 15:14 | |
| *** Sukhdev has joined #openstack-nova | 15:14 | |
| *** shakamunyi has joined #openstack-nova | 15:14 | |
| *** jasondotstar has joined #openstack-nova | 15:15 | |
| *** shakamunyi is now known as barra204 | 15:16 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Remove legacy flavor compatibility code from Instance https://review.openstack.org/197771 | 15:25 |
| dansmith | would really like to get that merged ^ | 15:25 |
| *** barra204 has quit IRC | 15:26 | |
| *** pixelbeat_ has quit IRC | 15:27 | |
| alaski | ooh, I would too. I'll take a look in a bit | 15:27 |
| *** prometheanfire has joined #openstack-nova | 15:27 | |
| dansmith | woohoo! | 15:27 |
| *** thedodd has joined #openstack-nova | 15:28 | |
| prometheanfire | mikal: if you are still up, was told you might know if nova supports running multiple arches in the same cell/region, aka x86_64 and armv8(arm64)? | 15:28 |
| *** pixelbeat_ has joined #openstack-nova | 15:28 | |
| prometheanfire | question for others as well I guess :D | 15:28 |
| *** meghal has joined #openstack-nova | 15:28 | |
| *** sajeesh has joined #openstack-nova | 15:29 | |
| *** tjones has joined #openstack-nova | 15:30 | |
| dansmith | prometheanfire: yeah, that should be doable | 15:30 |
| tdurakov | alaski, ping | 15:31 |
| prometheanfire | cool, friend has an arm64 box as a build box and wanted nova on it :D | 15:31 |
| tdurakov | seen your comment: While I agree that we can be better about alerting users to migrations that will fail I don't want to establish a precedent of having API calls consult with drivers before returning a response. - could you explain why it is wrong? | 15:32 |
| *** dannywilson has quit IRC | 15:33 | |
| *** otter768 has joined #openstack-nova | 15:34 | |
| alaski | tdurakov: it gets to a place where the same request can have a different response based on which virt driver is in use | 15:34 |
| alaski | which is info that Nova shouldn't (need to) expose | 15:34 |
| *** armax has joined #openstack-nova | 15:35 | |
| tdurakov | alaski, you are talking of a case when nova have different compute nodes type, right? | 15:36 |
| *** ishant has quit IRC | 15:36 | |
| *** xyang2 has quit IRC | 15:36 | |
| alaski | tdurakov: that, or two different clouds which use different hypervisors | 15:36 |
| *** moshele_ has quit IRC | 15:37 | |
| dansmith | tdurakov: or different versions of the same hypervisor | 15:37 |
| *** shyama has joined #openstack-nova | 15:37 | |
| *** schwicke has quit IRC | 15:38 | |
| tdurakov | alaski, dansmith, ok, thanks. another question | 15:38 |
| *** otter768 has quit IRC | 15:38 | |
| prometheanfire | dansmith: the sceduler can handle the diferent image types and cpu arches? | 15:39 |
| dansmith | prometheanfire: the scheduler can honor host aggregates, which you can use to segment things | 15:39 |
| mdbooth | Whoa! https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertTrue | 15:39 |
| dansmith | there might be an arch filter too, I'm not positive | 15:39 |
| tdurakov | if you attempt to resize instance in such cloud now you can get same situation, but there would be no info for user | 15:39 |
| mdbooth | You modify a method to return an object instead of a boolean, and its unit test continues to pass by coincidence! | 15:39 |
| tdurakov | what is a profit in such hiding? | 15:39 |
| *** sayali has quit IRC | 15:40 | |
| tdurakov | alaski, dansmith^ | 15:40 |
| prometheanfire | http://docs.openstack.org/developer/nova/devref/filter_scheduler.html maybe | 15:40 |
| prometheanfire | will have to look | 15:40 |
| *** barra204 has joined #openstack-nova | 15:40 | |
| johnthetubaguy | dansmith: I was trying to find the unit test for that remaining load flavor logic, do you know where I should look?: https://review.openstack.org/#/c/197771/8/nova/objects/instance.py,cm | 15:40 |
| dansmith | johnthetubaguy: for which remaining flavor load logic? | 15:41 |
| mdbooth | So interestingly, I recall being encouraged to replace assertEqual(True, foo) with assertTrue(foo) | 15:41 |
| mdbooth | However, those 2 aren't at all equivalent | 15:41 |
| dansmith | mdbooth: no the other way around | 15:41 |
| *** mestery is now known as mestery_afk_toda | 15:42 | |
| alaski | tdurakov: there's no value in hiding the information. I don't have concerns with providing some feedback, just the way it is done. I think it warrants a bp/ML post/meeting discussion | 15:42 |
| mdbooth | And the docs suggest assertIs(foo, True) in that case | 15:42 |
| dansmith | mdbooth: our hacking rules try to discourage you from using assertTrue | 15:42 |
| *** mestery_afk_toda is now known as mestery_afk | 15:42 | |
| rlrossit | dansmith: taking a look at your flavor change... you're removing a lot of methods but you aren't bumping the version of Instance. I'm guessing these methods aren't used anymore but it still seems scary to just leave the version as the same. | 15:42 |
| mdbooth | dansmith: orly? All is well in the world :) | 15:42 |
| dansmith | johnthetubaguy: L510 maybe? | 15:42 |
| johnthetubaguy | dansmith: line 412 in that review list | 15:42 |
| openstackgerrit | lyanchih proposed openstack/nova: Can't remove error instance which boot from image with new volume https://review.openstack.org/208536 | 15:42 |
| dansmith | johnthetubaguy: er, right | 15:43 |
| dansmith | johnthetubaguy: I dunno, let me look.. that's bringing something out of _migrate_flavor() that we already had, but I can look to see if we have something specific to cover that | 15:43 |
| dansmith | rlrossit: they're not remotable | 15:43 |
| johnthetubaguy | dansmith: cool, thanks | 15:43 |
| dansmith | rlrossit: so they don't have any bearing on RPC, which means no version bump | 15:43 |
| tdurakov | alaski, when i disscussed that with johnthetubaguy, bp wasn't required. maybe ml? | 15:44 |
| rlrossit | dansmith: ah, never noticed the remotable decorators before. TIL | 15:44 |
| dansmith | rlrossit: :) | 15:45 |
| dansmith | johnthetubaguy: actually there is a test, but it says: | 15:45 |
| dansmith | if 'flavor' in attr: # FIXME(danms): This isn't implemented yet | 15:45 |
| *** belmoreira has quit IRC | 15:45 | |
| johnthetubaguy | dansmith: yeah, just spotted that, line 170 | 15:45 |
| dansmith | johnthetubaguy: :) | 15:46 |
| dansmith | johnthetubaguy: so I should see if I can remove that exclusion now | 15:46 |
| johnthetubaguy | dansmith: yeah, I guess you want old_ and new_ in there too | 15:46 |
| *** mriedem has quit IRC | 15:46 | |
| dansmith | johnthetubaguy: the "if flavor in" was catching all three on purpose | 15:46 |
| alaski | tdurakov: that would be good, or bring it up at a nova meeting. I think it's worth discussing in some form with a wider audience before proceeding | 15:46 |
| johnthetubaguy | dansmith: ah, didn't read it that way, makes sense | 15:47 |
| dansmith | johnthetubaguy: yeah, that test passes without the exclusion now, so lemme push that up | 15:47 |
| johnthetubaguy | dansmith: sweet, thanks | 15:47 |
| openstackgerrit | Dan Smith proposed openstack/nova: Remove legacy flavor compatibility code from Instance https://review.openstack.org/197771 | 15:47 |
| prometheanfire | we getting a backport for https://launchpad.net/bugs/1387543 ? | 15:49 |
| openstack | Launchpad bug 1387543 in OpenStack Security Advisory "Resize/delete combo allows to overload nova-compute (CVE-2015-3241)" [Medium,In progress] | 15:49 |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Fixes Bug "destroy_vm fails with HyperVException" https://review.openstack.org/204007 | 15:50 |
| *** dannywilson has joined #openstack-nova | 15:50 | |
| *** dannywilson has quit IRC | 15:53 | |
| *** openstackgerrit_ has joined #openstack-nova | 15:54 | |
| *** tjones has quit IRC | 15:54 | |
| *** jasondotstar has quit IRC | 15:54 | |
| *** scheuran has quit IRC | 15:57 | |
| *** wendar has quit IRC | 15:57 | |
| *** wendar has joined #openstack-nova | 15:57 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Fixes Bug "destroy_vm fails with HyperVException" https://review.openstack.org/204007 | 15:58 |
| *** lucas___ has quit IRC | 15:58 | |
| *** tpeoples has quit IRC | 15:59 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-nova | 16:02 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Don't check flavor disk attributes when booting from volume https://review.openstack.org/204635 | 16:02 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Use original image size in volume image metadata https://review.openstack.org/208523 | 16:02 |
| *** tpeoples has joined #openstack-nova | 16:03 | |
| *** prometheanfire has left #openstack-nova | 16:03 | |
| *** jlanoux has quit IRC | 16:04 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup https://review.openstack.org/192760 | 16:05 |
| *** tsekiyama has joined #openstack-nova | 16:06 | |
| *** rdo has quit IRC | 16:07 | |
| *** sudipto has joined #openstack-nova | 16:07 | |
| *** rdo has joined #openstack-nova | 16:08 | |
| *** ig0r_ has joined #openstack-nova | 16:08 | |
| *** dguitarbite has quit IRC | 16:08 | |
| *** lsmola has quit IRC | 16:11 | |
| *** leeantho has joined #openstack-nova | 16:13 | |
| *** unicell has quit IRC | 16:13 | |
| *** tjones has joined #openstack-nova | 16:14 | |
| *** echoingumesh has joined #openstack-nova | 16:15 | |
| *** Marga_ has quit IRC | 16:16 | |
| *** mriedem has joined #openstack-nova | 16:16 | |
| *** fawadkhaliq has joined #openstack-nova | 16:16 | |
| *** tjones has quit IRC | 16:17 | |
| echoingumesh | nova-api stops when i add log_config=/etc/nova/logging.conf. my logging.conf is default. Any clues ? | 16:17 |
| *** tjones has joined #openstack-nova | 16:17 | |
| *** sahid has quit IRC | 16:18 | |
| *** alex_klimov has quit IRC | 16:20 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Fix test_relationships() for subobject versions https://review.openstack.org/198904 | 16:20 |
| *** browne has joined #openstack-nova | 16:21 | |
| jogo | jaypipes: I am making sure mordred can keep pinning his recipes | 16:21 |
| dansmith | jogo: ah, pinteresting | 16:23 |
| jaypipes | jogo: :) | 16:25 |
| *** yamahata has quit IRC | 16:25 | |
| *** marzif_ has quit IRC | 16:25 | |
| jogo | dansmith: they love putting the word pin in everything here | 16:26 |
| dansmith | I bet | 16:26 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: cleanup NovaObjectDictCompat from security_group_rule https://review.openstack.org/201282 | 16:26 |
| jogo | pinfo | 16:26 |
| jaypipes | that's pincredible. | 16:26 |
| *** Marga_ has joined #openstack-nova | 16:27 | |
| jroll | jogo: I used to work at zerocater, I now have thousands of food puns in my back pocket :| | 16:27 |
| *** jistr has quit IRC | 16:28 | |
| *** dustins has quit IRC | 16:28 | |
| *** shyama has quit IRC | 16:29 | |
| jaypipes | jroll: were you ever called spring rollenhagen? | 16:29 |
| jroll | somehow, I don't think so | 16:29 |
| jaypipes | jroll: I believe I have just identified your casual friday nick. jspringroll | 16:29 |
| jroll | my nick on hipchat there was JimmyJohns | 16:29 |
| *** jdandrea has joined #openstack-nova | 16:29 | |
| jaypipes | or jeggroll if you prefer ;) | 16:29 |
| jroll | nah, if I participate in casual nick friday the clear choice is jtroll | 16:30 |
| jaypipes | haha | 16:30 |
| jaypipes | nice. | 16:30 |
| *** jcoufal_ has quit IRC | 16:30 | |
| *** annashen has joined #openstack-nova | 16:31 | |
| *** diogogmt has joined #openstack-nova | 16:32 | |
| *** otter768 has joined #openstack-nova | 16:32 | |
| *** moshele_ has joined #openstack-nova | 16:33 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: compute: improve exceptions related to disk size checks https://review.openstack.org/206565 | 16:34 |
| *** exploreshaifali has joined #openstack-nova | 16:35 | |
| *** annashen has quit IRC | 16:35 | |
| *** unicell has joined #openstack-nova | 16:37 | |
| *** lucas__ has joined #openstack-nova | 16:37 | |
| *** dustins has joined #openstack-nova | 16:38 | |
| *** lpetrut has joined #openstack-nova | 16:39 | |
| *** lucas__ has quit IRC | 16:42 | |
| jogo | jroll: haha | 16:42 |
| mgagne | melwitt: I'm trying to create a keypair and use it in a cell enabled environment. I'm getting KeypairNotFound exception and it looks like nova-compute is trying to load the key (via conductor) from the compute cell database instead of the api cell. Any idea on where to start debugging? | 16:42 |
| openstackgerrit | Dan Smith proposed openstack/nova: Remove legacy flavor compatibility code from Instance https://review.openstack.org/197771 | 16:43 |
| openstackgerrit | Dan Smith proposed openstack/nova: Fix test mock that abuses objects https://review.openstack.org/208559 | 16:43 |
| *** moshele_ has quit IRC | 16:43 | |
| jroll | mgagne: I feel like that's a known issue | 16:43 |
| dansmith | alaski: addressed your concern, and had to fix another test that crept in :/ | 16:43 |
| mgagne | jroll: can you share the known solution? :D | 16:43 |
| jroll | looking :P | 16:44 |
| mgagne | found https://review.openstack.org/#/c/197514/ | 16:44 |
| mgagne | don't know why LP didn't find anything :-/ | 16:44 |
| jroll | mgagne: here's the bug: https://bugs.launchpad.net/nova/+bug/1443816 | 16:45 |
| openstack | Launchpad bug 1443816 in OpenStack Compute (nova) "cells: config drive doesn't work with cells when injecting an ssh key" [High,In progress] - Assigned to Chung Chih, Hung (lyanchih) | 16:45 |
| *** browne has quit IRC | 16:45 | |
| mgagne | jroll: tyvm for your help, will try now | 16:45 |
| jroll | mgagne: downstream we've just commented it out, as it's only for the metadata service, which we don't use https://gist.github.com/jimrollenhagen/058dcf2674b8b3fd62e7 | 16:46 |
| *** kiran-r has joined #openstack-nova | 16:46 | |
| mgagne | jroll: what about configdrive? | 16:46 |
| jroll | mgagne: it still works for configdrive | 16:46 |
| *** asselin_ has joined #openstack-nova | 16:46 | |
| jroll | I haven't read enough of the code to know why, but it does :) | 16:46 |
| *** tonytan4ever has quit IRC | 16:46 | |
| mgagne | will try now | 16:47 |
| *** shyama has joined #openstack-nova | 16:48 | |
| openstackgerrit | Anthony PERARD proposed openstack/nova: libvirt-xen: Add xen_vif_bridge_script_path config option. https://review.openstack.org/199093 | 16:48 |
| *** ig0r_ has quit IRC | 16:49 | |
| *** jasondot_ has joined #openstack-nova | 16:51 | |
| *** jasondot_ has quit IRC | 16:52 | |
| *** jasondot_ has joined #openstack-nova | 16:52 | |
| *** exploreshaifali has quit IRC | 16:53 | |
| johnthetubaguy | mgagne: jroll: that breaks because it tries to read the ssh key from the "wrong" database | 16:54 |
| *** jasondot_ is now known as jasondotstar_ | 16:54 | |
| *** jasondotstar_ is now known as jasondotstar__ | 16:54 | |
| mgagne | johnthetubaguy: yea, we figured that part | 16:54 |
| *** jasondotstar__ is now known as jasondotstar_ | 16:54 | |
| *** ig0r_ has joined #openstack-nova | 16:54 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] issue with latest libraries https://review.openstack.org/208561 | 16:54 |
| *** lucas__ has joined #openstack-nova | 16:54 | |
| mgagne | jroll: it works now. thanks. I'm not sure what's the difference between public_keys and keys in the metadata though... looks like the instance object already has a partial copy of the keypair data anyway. I don't know why it bother to load it again. key_data and key_name are already available. Maybe it loads it only to get the key type... | 16:54 |
| *** otter768 has quit IRC | 16:55 | |
| *** lucas__ has quit IRC | 16:55 | |
| johnthetubaguy | mgagne: jroll: ah, so sorry, I miss read the conversation | 16:55 |
| jroll | johnthetubaguy: right, the thing is that it's not immediately obvious to me why it still works with configdrive, but I'll take it as a known unknown | 16:55 |
| *** lucas__ has joined #openstack-nova | 16:55 | |
| jroll | mgagne: yeah, it's odd | 16:55 |
| *** ig0r_ has quit IRC | 16:55 | |
| *** Marga_ has quit IRC | 16:55 | |
| johnthetubaguy | jroll: oh, so it should fail with both, good point, unless we are missing data in one of them | 16:55 |
| mgagne | jroll: configdrive still loads InstanceMetadata. I guess it's cloud-init that reads public_keys and not keys which is now missing. | 16:55 |
| jroll | maybe configdrive doesn't require key type | 16:56 |
| jroll | mgagne: ah, I see, interesting | 16:56 |
| mgagne | configdrive doesn't care, only cloud-init or whatever agent is in there | 16:56 |
| johnthetubaguy | jroll: yeah, thats a bug if thats true, but I guess thats it | 16:56 |
| *** Marga_ has joined #openstack-nova | 16:56 | |
| johnthetubaguy | they should have identical data, as I understand it | 16:56 |
| *** tjones has quit IRC | 16:56 | |
| * jroll nod | 16:57 | |
| mgagne | mostly, one has the key type too in its struct | 16:57 |
| johnthetubaguy | the key_type is missing in the instance object, thats the key issue I guess | 16:57 |
| *** rdo has quit IRC | 16:57 | |
| mgagne | no pun intended :P | 16:57 |
| *** rdopiera has quit IRC | 16:57 | |
| johnthetubaguy | its possible config drive just always has None for the type or something horrible like that | 16:57 |
| johnthetubaguy | thats worth a check | 16:57 |
| *** coolsvap is now known as coolsvap|away | 16:58 | |
| mgagne | I'm not sure why it worked with icehouse, something changed in the metadata struct | 16:58 |
| mgagne | in juno: https://github.com/openstack/nova/blob/stable/juno/nova/api/metadata/base.py#L317-L320 | 16:59 |
| mgagne | in kilo: https://github.com/openstack/nova/blob/stable/kilo/nova/api/metadata/base.py#L309-L321 | 16:59 |
| mgagne | so yea, someone added a new key to the metadata | 16:59 |
| *** rdo has joined #openstack-nova | 16:59 | |
| mriedem | scottda: i was out friday afternoon, you linked me something about a cinder related bug, data loss something something? | 17:00 |
| mgagne | and type can be ssh or x509 | 17:00 |
| *** READ10 has quit IRC | 17:01 | |
| *** haomaiwang has quit IRC | 17:01 | |
| scottda | mriedem: just a minute...let me track that down... | 17:01 |
| *** haomaiwang has joined #openstack-nova | 17:02 | |
| *** READ10 has joined #openstack-nova | 17:02 | |
| *** harlowja has joined #openstack-nova | 17:03 | |
| *** tongli has joined #openstack-nova | 17:03 | |
| *** ig0r_ has joined #openstack-nova | 17:04 | |
| *** dguitarbite has joined #openstack-nova | 17:04 | |
| *** tonytan4ever has joined #openstack-nova | 17:05 | |
| mgagne | jroll: I'm sure there is a way to "recompute" the type from the key data | 17:05 |
| scottda | mriedem: So, the issue is that Cinder would like to start returning an exception when the API is called with a volume that is in a busy '*-ing' state (creating/deleting/detaching/attaching) and Nova would need changes so that calls to Cinder volume_api would catch exceptions.... | 17:05 |
| mgagne | jroll: like key_data.startswith('ssh') | 17:05 |
| scottda | https://bugs.launchpad.net/nova/+bug/1458958 is the bug. | 17:05 |
| openstack | Launchpad bug 1460044 in OpenStack Compute (nova) "duplicate for #1458958 Data loss can occur if cinder attach fails" [High,In progress] - Assigned to Vipin Balachandran (vbala) | 17:05 |
| *** pcaruana has joined #openstack-nova | 17:06 | |
| scottda | mriedem: I'm wondering your opinion if 1) this does not need a spec, but is a valid bug 2) Can we land these changes in Nova for Liberty? | 17:06 |
| mriedem | scottda: first, that says it shouldn't be a duplicate, so can we just fix lp firsT? | 17:06 |
| *** tjones has joined #openstack-nova | 17:06 | |
| *** tjones has quit IRC | 17:06 | |
| scottda | It's not really a duplicate, because the bug it points to is more specific, for only the detaching case. | 17:07 |
| scottda | so, do you mean fix the duplicate part? sure, I'll see if I can do that... | 17:07 |
| scottda | ok...I think I've un-duped it. | 17:08 |
| mriedem | so i guess i'll look at https://review.openstack.org/#/c/186742/ first | 17:09 |
| johnthetubaguy | mgagne: I think we need to store the keytype in the instance record, that was an oversight from the patch that added the type field, I think | 17:09 |
| scottda | ok. I don't know if you've been following in the ML, but this ties into the discussion on making Cinder Active-Active HA, and the need for a DLM... | 17:09 |
| mgagne | johnthetubaguy: sure. now I would like a patch for my setup in the next 10 minutes so thinking out loud about a solution =) | 17:10 |
| mriedem | ew jesus, i have not | 17:10 |
| *** davideagnello has joined #openstack-nova | 17:11 | |
| dansmith | rlrossit: does this need freshening after the last week of fixing the nova version? https://review.openstack.org/#/c/203831/2 | 17:11 |
| scottda | It's lengthy...and complicated...and contentious. Anyway, we're at the cinder mid-cycle this week, so removing local locks in Cinder, Cinder API returning an exception, and Nova catching it are part of a bigger discussion. | 17:11 |
| johnthetubaguy | mgagne: gotcha, I would just hard code ssh, if you don't use windows guests, if you need a quick fix? I guess the check in the keydata would do no halm | 17:11 |
| *** moshele_ has joined #openstack-nova | 17:11 | |
| * johnthetubaguy feels kinda dirty after saying that | 17:12 | |
| *** kmARC has quit IRC | 17:13 | |
| mgagne | https://gist.github.com/jimrollenhagen/058dcf2674b8b3fd62e7#gistcomment-1541307 | 17:13 |
| *** lyanchih_ has quit IRC | 17:13 | |
| mriedem | scottda: well, isn't that kind of a backwards incomptable change with the cinder API? | 17:13 |
| *** tjones has joined #openstack-nova | 17:14 | |
| mriedem | have we ever supported newer cinder than nova in deployments? | 17:14 |
| *** Sukhdev has quit IRC | 17:14 | |
| *** dannywilson has joined #openstack-nova | 17:14 | |
| mriedem | like liberty cinder starts sending errors back to kilo nova via the cinder api | 17:14 |
| *** diogogmt has quit IRC | 17:15 | |
| *** Sukhdev has joined #openstack-nova | 17:15 | |
| scottda | not that I know of, and I don't think we'd start now... | 17:16 |
| jroll | mgagne: seems like it would work | 17:16 |
| *** unicell1 has joined #openstack-nova | 17:16 | |
| *** unicell has quit IRC | 17:16 | |
| scottda | so Cinder would have to land their changes first, but only if Nova were ready to catch the exceptions (soon after Cinder lands). Kinda tricky... | 17:16 |
| *** tonytan4ever has quit IRC | 17:17 | |
| *** tonytan4ever has joined #openstack-nova | 17:17 | |
| scottda | and then there's the backward compatible (cinder L, Nova k) stuff. | 17:17 |
| mgagne | jroll: hmmm unless there is a way to make SSH keypair not start with the ssh string | 17:17 |
| mgagne | jroll: http://man.he.net/man5/authorized_keys | 17:18 |
| mgagne | jroll: nova doesn't do much validation on the content... | 17:18 |
| jroll | nod | 17:19 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: cleanup NovaObjectDictCompat from virtual_interface https://review.openstack.org/200823 | 17:19 |
| scottda | mriedem: I've no answers here, just an ever expanding list of questions. | 17:19 |
| *** echoingumesh has quit IRC | 17:20 | |
| *** dannywilson has quit IRC | 17:20 | |
| *** echoingumesh has joined #openstack-nova | 17:20 | |
| *** echoingumesh has quit IRC | 17:21 | |
| mriedem | scottda: did a nova person ever sign up to come to the cinder meetup? | 17:21 |
| scottda | no, I think we got the message out too late... | 17:21 |
| scottda | but we'll be using a google hangout. | 17:22 |
| *** echoingumesh has joined #openstack-nova | 17:22 | |
| scottda | And I'll be pinging this room when we discuss Nova related issues, i.e. the Nova <-> Cinder API | 17:22 |
| *** achanda has joined #openstack-nova | 17:25 | |
| *** ig0r_ has quit IRC | 17:25 | |
| *** salv-orlando has joined #openstack-nova | 17:25 | |
| *** achanda_ has joined #openstack-nova | 17:26 | |
| *** achanda has quit IRC | 17:26 | |
| *** atuvenie has quit IRC | 17:26 | |
| *** SourabhP has quit IRC | 17:27 | |
| johnthetubaguy | scottda: cool, I should be able to drop in during the morning, if thats helpful, although I don't really have the strongest cinder knowledge, but I can help some some bits | 17:28 |
| scottda | johnthetubaguy: What time zone are you in? | 17:28 |
| johnthetubaguy | scottda: UTC+1 | 17:28 |
| scottda | ok, mid-cycle is in MDT UTC-6 so we'll shoot for early in our morning here, which would be late afternoon there. | 17:29 |
| johnthetubaguy | scottda: I do have some meetings tomorrow, that will be hard to get out off, but I will do my best to get to make time, possibly wednesday morning would work best | 17:29 |
| johnthetubaguy | scottda: yeah, sounds good | 17:29 |
| *** browne has joined #openstack-nova | 17:30 | |
| scottda | We'll try to schedule for fir/st thing Wed morning, then. No later than 10:00 AM MDT which is 16:00 UTC | 17:30 |
| mriedem | scottda: ok, anyway, https://review.openstack.org/#/c/186742/ needs some work | 17:30 |
| mriedem | i'd also like ndipanov to review that ^ | 17:30 |
| *** yassine has quit IRC | 17:31 | |
| *** vilobhmm has joined #openstack-nova | 17:32 | |
| mriedem | i'd also think that this impacts tempest | 17:33 |
| *** echoingumesh has quit IRC | 17:33 | |
| mriedem | "When a volume is in an 'ING' state, Cinder API calls that interact with that volume will return an exception notifying a caller that that volume is busy. There may be other calls to the Cinder API (that deal with volumes) that Nova does not handle an exception from. Exceptions from those calls will need to be handled, too." | 17:33 |
| *** ig0r_ has joined #openstack-nova | 17:33 | |
| *** jasondotstar_ has quit IRC | 17:34 | |
| *** diogogmt has joined #openstack-nova | 17:34 | |
| mriedem | scottda: vilobhmm: do you know if anyone has started working on this part that jaypipes mentioned, which we should be able to agree on https://bugs.launchpad.net/nova/+bug/1458958/comments/1 | 17:34 |
| openstack | Launchpad bug 1458958 in OpenStack Compute (nova) "Exceptions from Cinder detach volume API not handled" [Wishlist,In progress] - Assigned to Michal Dulko (michal-dulko-f) | 17:34 |
| mriedem | "The Cinder Volume REST API should, at the very least, catch VolumeInUse and return an HTTP 409 Conflict status code, which the cinderclient, called via nova-compute, can translate to a non-500 Internal Server Error exception." | 17:34 |
| *** andreykurilin__ has quit IRC | 17:35 | |
| scottda | yes, cinderclient can/will be changed to translate the HTTP 409... | 17:35 |
| *** danpb has quit IRC | 17:37 | |
| *** Piet_ has joined #openstack-nova | 17:37 | |
| *** praneshp has joined #openstack-nova | 17:38 | |
| vilobhmm | mriedem : I guess scottda is working on it… | 17:38 |
| *** Nikolay_St has joined #openstack-nova | 17:39 | |
| *** tonytan4ever has quit IRC | 17:39 | |
| scottda | vilobhmm: working on which part? | 17:40 |
| *** lpetrut has quit IRC | 17:41 | |
| vilobhmm | on nova-cinder interaction | 17:41 |
| scottda | cinderclient? Yes, I'll probably do that. Nova try blocks? maybe, or will work with winston-d_zZZ who had shown an interest.. | 17:41 |
| vilobhmm | sure | 17:41 |
| vilobhmm | winston-d intiially decided to pick it up in cinder weekly meeting | 17:41 |
| scottda | mriedem: that work is not started, but I'll take that on next week. | 17:43 |
| dansmith | hmm | 17:43 |
| dansmith | wonder if we're seeing a new unit test fail on master | 17:43 |
| dansmith | http://logs.openstack.org/59/208559/1/check/gate-nova-python27/61df9b7/console.html#_2015-08-03_17_35_16_893 | 17:44 |
| dansmith | I see it locally if I run on master | 17:44 |
| *** ig0r_ has quit IRC | 17:44 | |
| *** tonytan4ever has joined #openstack-nova | 17:45 | |
| *** echoingumesh has joined #openstack-nova | 17:45 | |
| *** lpetrut has joined #openstack-nova | 17:47 | |
| *** liusheng has quit IRC | 17:48 | |
| *** liusheng has joined #openstack-nova | 17:48 | |
| *** lucas__ has quit IRC | 17:49 | |
| *** lucas__ has joined #openstack-nova | 17:49 | |
| sudipto | dansmith, think i should remove the "Conflicts" from the commit message: https://review.openstack.org/#/c/192760/ ? | 17:52 |
| *** dane-fichter has quit IRC | 17:52 | |
| dansmith | sudipto: yeah, we leave those in for a backport, but not master patches | 17:52 |
| dansmith | if we left them in, | 17:52 |
| dansmith | rlrossit's patches would have like 200 lines in the commit message :) | 17:52 |
| rlrossit | what did I do? | 17:53 |
| *** yamahata has joined #openstack-nova | 17:53 | |
| rlrossit | oh yeah definitely | 17:53 |
| sudipto | dansmith, ok...will remove them...also would ask for a review when you have sometime :) | 17:53 |
| dansmith | sudipto: did you get an oslo.db change to make it so we can remove those from the model before the schema? | 17:53 |
| sudipto | dansmith, yup | 17:54 |
| sudipto | dansmith, that's why it took a while i believe for jenkins to pass. | 17:54 |
| dansmith | okay cool | 17:54 |
| rlrossit | dansmith: I don't *think* https://review.openstack.org/#/c/203831/2 needs any refreshers because I took that code straight from the original object's method of getting subobject versions. I can toss more of the tests that I wrote in nova in that change though. I kinda just whipped that up one afternoon and would like your opinions on the style of how I wrote it because that was my attempt at being creative | 17:55 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup https://review.openstack.org/192760 | 17:56 |
| dansmith | okay, I just wasn't sure if I should look at it or hold off | 17:56 |
| mnestratov | johnthetubaguy: hi, I saw you've just changed Approved => Pending Approval for https://blueprints.launchpad.net/nova/+spec/virtuozzo-instance-resize-support, does it mean that I should to send FFE to ML? | 17:56 |
| mnestratov | or https://etherpad.openstack.org/p/liberty-nova-non-priority-feature-freeze is enough? | 17:56 |
| *** moshele_ has quit IRC | 17:57 | |
| *** kmARC has joined #openstack-nova | 17:58 | |
| openstackgerrit | garyk proposed openstack/nova: Neutron: add in API method for updating VNIC index https://review.openstack.org/147126 | 17:58 |
| *** garyk has joined #openstack-nova | 17:59 | |
| *** salv-orl_ has joined #openstack-nova | 17:59 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Cells: Handle instance_destroy_at_top failure https://review.openstack.org/207587 | 18:00 |
| *** Marga_ has quit IRC | 18:00 | |
| *** haomaiwang has quit IRC | 18:01 | |
| *** salv-orlando has quit IRC | 18:01 | |
| *** Marga_ has joined #openstack-nova | 18:01 | |
| *** haomaiwang has joined #openstack-nova | 18:02 | |
| openstackgerrit | Baodong (Robert) Li proposed openstack/nova: Handle port delete initiated by neutron https://review.openstack.org/179390 | 18:03 |
| *** markus_z has quit IRC | 18:06 | |
| *** openstackgerrit_ has quit IRC | 18:07 | |
| *** tonytan4ever has quit IRC | 18:08 | |
| *** lpetrut has quit IRC | 18:10 | |
| *** dannywilson has joined #openstack-nova | 18:11 | |
| *** dannywilson has quit IRC | 18:11 | |
| *** dannywilson has joined #openstack-nova | 18:11 | |
| *** Marga_ has quit IRC | 18:11 | |
| *** rushiagr is now known as rushiagr_away | 18:14 | |
| *** achanda has joined #openstack-nova | 18:14 | |
| *** tcammann has quit IRC | 18:15 | |
| *** achanda_ has quit IRC | 18:16 | |
| *** tcammann has joined #openstack-nova | 18:16 | |
| sudipto | dansmith, regardging your comment, i guess you meant the 'scheduled_at' field is deprecated and but yet not removed, hence we need to discard it - is that the right way to understand it? | 18:16 |
| *** SourabhP has joined #openstack-nova | 18:17 | |
| sudipto | dansmith, sorry - i understood your statement properly now :) | 18:17 |
| mriedem | dansmith: https://github.com/openstack/nova/commit/ed0196ebb6e09b0baac783981da480849ee16f36 | 18:18 |
| dansmith | mriedem: what about it? | 18:19 |
| mriedem | that's probably what introduced that new test bug | 18:19 |
| dansmith | sudipto: okay :) | 18:19 |
| dansmith | mriedem: the ec2 thing? | 18:19 |
| mriedem | no nova.tests.unit.cmd.test_baseproxy.BaseProxyTestCase.test_proxy | 18:19 |
| dansmith | I'm confused | 18:20 |
| mriedem | (12:44:31 PM) dansmith: http://logs.openstack.org/59/208559/1/check/gate-nova-python27/61df9b7/console.html#_2015-08-03_17_35_16_893 | 18:20 |
| mriedem | (12:43:58 PM) dansmith: wonder if we're seeing a new unit test fail on master | 18:20 |
| dansmith | oh, I see.. I'm actually seeing a different failure locally | 18:21 |
| dansmith | I thought was the same | 18:21 |
| dansmith | heh | 18:21 |
| *** harlowja has quit IRC | 18:21 | |
| * dansmith recreates .tox | 18:21 | |
| *** Kennan has quit IRC | 18:21 | |
| *** harlowja has joined #openstack-nova | 18:21 | |
| *** chaochin has quit IRC | 18:21 | |
| *** park_heijlong has quit IRC | 18:22 | |
| mriedem | unfortunately logstash is all backed up so this is too new to shows up | 18:22 |
| mriedem | anyway, https://review.openstack.org/#/c/204723/ includes a bump to websockify which is apparently parsing things randomly, something racey in there | 18:24 |
| mtreinish | mriedem: https://review.openstack.org/#/c/208561/ seems related to that | 18:25 |
| mriedem | another jaypipes revert :) | 18:25 |
| dansmith | maybe we just automatically propose reverts for jaypipes reviews so we have them ready to go if we need them? | 18:26 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup https://review.openstack.org/192760 | 18:27 |
| mriedem | dansmith: do you have a line to directxman12? | 18:27 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1481043 | 18:27 |
| openstack | Launchpad bug 1481043 in OpenStack Compute (nova) "nova.tests.unit.cmd.test_baseproxy.BaseProxyTestCase.test_proxy randomly fails" [Undecided,New] | 18:27 |
| *** park_heijlong has joined #openstack-nova | 18:28 | |
| dansmith | maybe | 18:28 |
| *** chaochin has joined #openstack-nova | 18:28 | |
| *** Kennan has joined #openstack-nova | 18:29 | |
| *** rotbeard has joined #openstack-nova | 18:29 | |
| dansmith | mriedem: ah, yep, with new .tox I get the same failure as gate and the other fail is gone | 18:29 |
| dansmith | awesome | 18:29 |
| dansmith | "awesome" | 18:29 |
| *** directxman12 has joined #openstack-nova | 18:30 | |
| mriedem | that is awesome | 18:30 |
| mriedem | with logstash backed up i can't tell how pervasive this is | 18:30 |
| dansmith | mriedem: ^ | 18:30 |
| mriedem | i know | 18:30 |
| dansmith | mriedem: it's 100% for me | 18:30 |
| mriedem | i'm hanging on your every word :) | 18:30 |
| rlrossit | mriedem: it got me | 18:30 |
| openstackgerrit | garyk proposed openstack/nova: Metadata: support proxying loadbalancers https://review.openstack.org/147539 | 18:30 |
| mriedem | so um? | 18:30 |
| mriedem | revert? | 18:30 |
| dansmith | mriedem: I meant ^ directxman12 | 18:30 |
| mriedem | directxman12: 1481043 | 18:31 |
| mriedem | oops https://bugs.launchpad.net/nova/+bug/1481043 | 18:31 |
| openstack | Launchpad bug 1481043 in OpenStack Compute (nova) "nova.tests.unit.cmd.test_baseproxy.BaseProxyTestCase.test_proxy randomly fails" [High,Confirmed] | 18:31 |
| *** fawadkhaliq has quit IRC | 18:31 | |
| * directxman12 looks | 18:31 | |
| *** thangp has joined #openstack-nova | 18:31 | |
| *** annashen has joined #openstack-nova | 18:32 | |
| *** sajeesh has quit IRC | 18:32 | |
| *** htruta_ has quit IRC | 18:32 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] issue with latest libraries https://review.openstack.org/208561 | 18:33 |
| *** annashen has quit IRC | 18:36 | |
| *** josecastroleon has joined #openstack-nova | 18:36 | |
| directxman12 | mriedem: hmm... if I had to guess, it's because `traffic=CONF.verbose and not CONF.daemon`, so if it's failing randomly, there's some sort of configuration being left around | 18:37 |
| directxman12 | dansmith: ^ | 18:37 |
| *** kmARC has quit IRC | 18:38 | |
| *** kmARC_ has joined #openstack-nova | 18:38 | |
| *** kmARC_ has quit IRC | 18:38 | |
| *** sudipto has quit IRC | 18:38 | |
| *** kmARC has joined #openstack-nova | 18:39 | |
| *** apuimedo has quit IRC | 18:39 | |
| directxman12 | (it just looks like a mismatch between the traffic parameter) | 18:41 |
| *** doffm has joined #openstack-nova | 18:42 | |
| mriedem | directxman12: traffic and verbose | 18:43 |
| *** htruta_ has joined #openstack-nova | 18:44 | |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: Create a new dictionary for type_data in VMwareAPIVMTestCase class https://review.openstack.org/202768 | 18:45 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: create vif with resource limitations https://review.openstack.org/196171 | 18:45 |
| *** kiran-r has quit IRC | 18:45 | |
| *** shakamunyi has joined #openstack-nova | 18:46 | |
| directxman12 | mriedem: yeah. It doesn't look like it's a problem with anything changed in websockify, though. If you set those two conf options at the beginning of the test, does it fix the problem? | 18:46 |
| *** dprince has quit IRC | 18:48 | |
| *** patrickeast has joined #openstack-nova | 18:49 | |
| *** Sukhdev has quit IRC | 18:49 | |
| mriedem | not sure, would need to reproduce locally | 18:50 |
| mriedem | i'm also totally not understanding why some of the mock stuff is changed the way it is here https://review.openstack.org/#/c/204723/ | 18:50 |
| mriedem | like https://review.openstack.org/#/c/204723/19/nova/tests/unit/cmd/test_baseproxy.py <- why? | 18:50 |
| mriedem | dims: ^? | 18:51 |
| lifeless | why are you patching individual methods :) | 18:51 |
| *** Marga_ has joined #openstack-nova | 18:52 | |
| lifeless | mock.patch.object vs mock.patch are equivalent here | 18:52 |
| jdandrea | Scheduler plugin question: If I understand correctly, these plugins can each be assigned a weight to aid in determining "which one wins" (yes/no/sorta?). | 18:55 |
| jaypipes | dansmith, mriedem: :( | 18:55 |
| jdandrea | If so, is there a way for a given plugin to either pass on its vote per se, or if it WANTS to vote, to be "winner take all" in that regard (e.g., have such a high weight that it always wins)? | 18:55 |
| *** otter768 has joined #openstack-nova | 18:56 | |
| dansmith | jaypipes: I was just kiddin' | 18:56 |
| dansmith | of course | 18:56 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: Configure memory resource allocation during instance resize operation https://review.openstack.org/204219 | 18:58 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: Configure disk resource allocation during instance resize operation https://review.openstack.org/208612 | 18:58 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Fix problem with BaseProxyTestCase.test_proxy https://review.openstack.org/208561 | 18:58 |
| *** tonytan4ever has joined #openstack-nova | 18:59 | |
| mriedem | jaypipes: sdague: added you guys to this cinder spec https://review.openstack.org/#/c/149894/ | 18:59 |
| mriedem | scottda: ^ i also went through and commented on that spec | 18:59 |
| jaypipes | mriedem: thx | 19:00 |
| *** haomaiwang has quit IRC | 19:01 | |
| *** otter768 has quit IRC | 19:01 | |
| *** yamahata has quit IRC | 19:01 | |
| *** yamahata has joined #openstack-nova | 19:01 | |
| mriedem | proposes to return errors from cinder api which used to return success | 19:02 |
| mriedem | w/o an api version bump | 19:02 |
| mriedem | we kind of sort of care about that | 19:02 |
| *** haomaiwang has joined #openstack-nova | 19:02 | |
| *** josecastroleon has quit IRC | 19:06 | |
| mriedem | directxman12: is "traffic=CONF.verbose and not CONF.daemon" in websockify? | 19:06 |
| mriedem | oops nvm | 19:06 |
| mriedem | i see it in nova now | 19:06 |
| directxman12 | mrieden: it's in nova.cmd.baseproxy | 19:06 |
| mriedem | yup | 19:07 |
| *** exploreshaifali has joined #openstack-nova | 19:07 | |
| mriedem | yeah so i think https://review.openstack.org/#/c/204723/ changed something to do a global mock and that's what's causing the race | 19:07 |
| mriedem | so probably need to stub CONF.verbose like you said in the test setup | 19:07 |
| mriedem | or not mock global things | 19:08 |
| *** flwang has quit IRC | 19:08 | |
| *** patrickeast_ has joined #openstack-nova | 19:10 | |
| *** patrickeast_ has quit IRC | 19:11 | |
| *** zul has quit IRC | 19:11 | |
| *** patrickeast has quit IRC | 19:11 | |
| *** Sukhdev has joined #openstack-nova | 19:12 | |
| *** READ10 has quit IRC | 19:13 | |
| *** tonytan4ever has quit IRC | 19:13 | |
| lifeless | directxman12: mriedem was puzzled on 204723 (see up a page or so), and I answered haypo's question about using testr in it, if thats interesting | 19:15 |
| lifeless | directxman12: sorry, fail tab complete | 19:16 |
| lifeless | dims: ^ | 19:16 |
| mriedem | i'm about to push a fix | 19:16 |
| * dims peeks | 19:16 | |
| dims | i was trying a change out locally...looks like mriedem may have a better fix | 19:16 |
| *** jasondot_ has joined #openstack-nova | 19:17 | |
| *** shyama has quit IRC | 19:17 | |
| dims | lifeless: ack, will look at the testr comment | 19:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Make BaseProxyTestCase.test_proxy deterministic wrt traffic/verbose https://review.openstack.org/208621 | 19:17 |
| mriedem | directxman12: dims: dansmith: jaypipes: rlrossit: ^ | 19:17 |
| *** angdraug has joined #openstack-nova | 19:17 | |
| lifeless | dims: I just realised the list might be file names, in which case an aggregate regex is needed, not a load-list; but its doable | 19:17 |
| dims | lifeless: i was asking mtreinish about adding a whitelist to os-testr | 19:18 |
| lifeless | dims: that said, I think what is there is ok while we're working up 3.4 support | 19:18 |
| lifeless | dims: just 'testr run filename.one filename.two' will DTRT | 19:19 |
| dansmith | mriedem: hmm | 19:19 |
| lifeless | dims: because the implicit combinator of regex patterns is | | 19:19 |
| dims | lifeless: 204723 was primarily to stop folks from checking in incompatible code/syntax/import for all tests | 19:19 |
| lifeless | dims: yep | 19:19 |
| directxman12 | mriedem: FWIW, LGTM | 19:19 |
| dims | thanks mriedem | 19:19 |
| mriedem | directxman12: i hope so since i put your name on it | 19:19 |
| directxman12 | heh | 19:19 |
| *** Piet_ has quit IRC | 19:20 | |
| *** busterswt has joined #openstack-nova | 19:22 | |
| *** flwang has joined #openstack-nova | 19:22 | |
| *** jasondot_ has quit IRC | 19:26 | |
| *** gszasz has quit IRC | 19:27 | |
| *** angdraug has quit IRC | 19:27 | |
| *** pixelbeat_ has quit IRC | 19:27 | |
| *** pixelbeat_ has joined #openstack-nova | 19:28 | |
| *** agireud_ has joined #openstack-nova | 19:29 | |
| *** MentalRay has joined #openstack-nova | 19:29 | |
| *** agireud has quit IRC | 19:30 | |
| *** agireud_ is now known as agireud | 19:30 | |
| *** emagana has joined #openstack-nova | 19:33 | |
| *** angdraug has joined #openstack-nova | 19:37 | |
| openstackgerrit | Marian Horban proposed openstack/nova: libvirt:Rsync remote FS driver was added https://review.openstack.org/146887 | 19:37 |
| openstackgerrit | Roman Dobosz proposed openstack/nova: New nova API call to mark nova-compute down https://review.openstack.org/184086 | 19:41 |
| openstackgerrit | Vikhyat Umrao proposed openstack/nova: RBD: Reading rbd_default_features from ceph.conf https://review.openstack.org/186537 | 19:42 |
| *** ftersin has quit IRC | 19:44 | |
| *** patrickeast has joined #openstack-nova | 19:48 | |
| *** angdraug has quit IRC | 19:48 | |
| *** angdraug has joined #openstack-nova | 19:48 | |
| *** raildo has quit IRC | 19:49 | |
| *** echoingumesh has quit IRC | 19:50 | |
| *** echoingumesh has joined #openstack-nova | 19:51 | |
| *** geekinutah has joined #openstack-nova | 19:51 | |
| *** patrickeast has quit IRC | 19:52 | |
| *** agireud has quit IRC | 19:52 | |
| nagyz | does anybody have some information on https://blueprints.launchpad.net/nova/+spec/add-local-cache-of-cinder-volumes-to-nova ? | 19:54 |
| *** agireud has joined #openstack-nova | 19:54 | |
| nagyz | here's the actual review: https://review.openstack.org/#/c/138619/ | 19:55 |
| dansmith | mriedem: do we understand at all why that helps? | 19:55 |
| dansmith | because based on what was changed in the supposed evil patch, I don't get it | 19:55 |
| nagyz | I know it's basically dead but for Mitaka would there be interest? | 19:55 |
| *** Piet has joined #openstack-nova | 19:58 | |
| mriedem | dansmith: i don't even know why https://review.openstack.org/#/c/204723/19/nova/tests/unit/cmd/test_baseproxy.py needed to bechanged | 19:58 |
| mriedem | like, can i not use @mock.patch.object now? | 19:58 |
| mriedem | dims: ^ | 19:58 |
| dansmith | hmm | 19:58 |
| *** haomaiwang has quit IRC | 20:01 | |
| *** patrickeast has joined #openstack-nova | 20:01 | |
| *** angdraug has quit IRC | 20:02 | |
| *** haomaiwang has joined #openstack-nova | 20:02 | |
| *** angdraug has joined #openstack-nova | 20:02 | |
| dims | mriedem: either one should work, am trying to remember why i needed that change | 20:03 |
| *** dustins has quit IRC | 20:05 | |
| mriedem | https://docs.python.org/3/library/unittest.mock.html#id4 | 20:09 |
| *** baoli has quit IRC | 20:09 | |
| *** rushiagr_away has quit IRC | 20:13 | |
| *** tonytan4ever has joined #openstack-nova | 20:14 | |
| mriedem | idk either, it fails if i change it back to this | 20:15 |
| mriedem | @mock.patch.object(websocketproxy.NovaWebSocketProxy, '__init__', return_value=None) | 20:15 |
| mriedem | without self.flags(verbose=False) | 20:15 |
| dansmith | mriedem: which fixes it? the mock change or the flags change? | 20:16 |
| mriedem | dansmith: the flags change | 20:16 |
| dansmith | the mock okay, that's what I thought, and that's the confusing thing to me | 20:17 |
| *** tonytan_brb has joined #openstack-nova | 20:20 | |
| *** tonytan4ever has quit IRC | 20:21 | |
| *** kurtmartin has joined #openstack-nova | 20:21 | |
| mriedem | dansmith: alternatively, oslo.logging 1.8.0 https://pypi.python.org/pypi/oslo.log/1.8.0 | 20:24 |
| mriedem | released today | 20:24 |
| mriedem | did you see this before today? | 20:24 |
| dansmith | no | 20:24 |
| mriedem | ha | 20:25 |
| mriedem | https://github.com/openstack/oslo.log/commit/f464d9f2e1e31de4e916f1c8089853d93b1bba61 | 20:25 |
| mriedem | bingo | 20:25 |
| mriedem | i'll update the commit message on my fix | 20:25 |
| *** lpetrut has joined #openstack-nova | 20:27 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Make BaseProxyTestCase.test_proxy deterministic wrt traffic/verbose https://review.openstack.org/208621 | 20:27 |
| mriedem | dansmith: jaypipes: sdague: dims: ^ | 20:27 |
| mriedem | updated commit message | 20:27 |
| jaypipes | mriedem: done | 20:29 |
| sdague | mriedem: ah, that commit message makes a bit more sense. Though it's still weird that it's actually impacting things poorly | 20:30 |
| mriedem | sdague: it's not very weird actually, | 20:30 |
| *** tmckay has joined #openstack-nova | 20:30 | |
| mriedem | it was False by default before, then changed to True by default | 20:30 |
| mriedem | the test wasn't explicitly stubbing that config option out so it was completely based on what the default was | 20:30 |
| sdague | oh, why don't you just make verbose=False => mock.ANY ? | 20:31 |
| mriedem | because i can just stub it | 20:31 |
| dansmith | yeah, if this is the problem, why force it back? | 20:31 |
| tmckay | Hi nova folks. We just found out in Sahara that if we have "Generated by Sahara" at the end of our public key, it breaks key serialization because the code in nova is splitting key components on a space and thinking there are 5 components | 20:31 |
| tmckay | so who is wrong? Sahara or nova? | 20:31 |
| mriedem | verbose='true'? | 20:31 |
| sdague | because, we don't care in the least about that value | 20:31 |
| * tmckay throws a grenage | 20:31 | |
| tmckay | can't spell. but you get the idea | 20:31 |
| sdague | so, I'm still +2 on the patch, because it's not hugely important, but it seems like mock.ANY on verbose in the assert would have been my approach | 20:32 |
| *** annashen has joined #openstack-nova | 20:33 | |
| dims | tmckay: have an example? (paste.openstack.org please) | 20:34 |
| tmckay | yep, hold on | 20:34 |
| dims | mriedem: good detective work :) | 20:35 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Xen: convert image auto_disk_config value to bool before compare https://review.openstack.org/208642 | 20:35 |
| *** ig0r_ has joined #openstack-nova | 20:36 | |
| *** annashen has quit IRC | 20:37 | |
| *** alexpilotti has joined #openstack-nova | 20:37 | |
| tmckay | dims, http://paste.openstack.org/show/406793/ | 20:38 |
| alaski | dansmith: is there a way to unset an attribute on an object? I can see a way to do it by mucking with object internals which I don't want to do | 20:39 |
| tmckay | dims, I can find the "split" line if I look again .... | 20:39 |
| mriedem | alaski: like with what changed? | 20:40 |
| mriedem | http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/base.py#n469 | 20:40 |
| mriedem | or so that obj_attr_is_set returns False? | 20:40 |
| alaski | the latter | 20:40 |
| alaski | delattr(object, get_attrname(attrname)) should do it, but is invasive | 20:41 |
| dims | tmckay: please open a bug, i think i know where it is | 20:41 |
| *** dannywilson has quit IRC | 20:41 | |
| *** patrickeast has quit IRC | 20:41 | |
| dims | tmckay: recently changed code, so possible regression | 20:42 |
| openstackgerrit | Earle F. Philhower, III proposed openstack/nova: Add os_brick-based VolumeDriver for HGST connector https://review.openstack.org/194412 | 20:42 |
| tmckay | /usr/lib/python2.7/site-packages/cryptography/hazmat/primitives/serialization.py(36)load_ssh_public_key() | 20:42 |
| mriedem | alaski: i guess you could convert to a primitive, pop the entry from the dict and then create a new object instance from that primitive, but equally gross | 20:42 |
| tmckay | dims, ^^ | 20:42 |
| *** prometheanfire has joined #openstack-nova | 20:43 | |
| *** patrickeast has joined #openstack-nova | 20:43 | |
| tmckay | dims, hmm, I wonder if spaces in the comment are legal at all | 20:44 |
| prometheanfire | what does nova use to attach to iscsi for cinder? need to know for packaging dep reasons | 20:44 |
| anish | open-iscsi | 20:44 |
| *** hemna has joined #openstack-nova | 20:45 | |
| *** patrickeast has quit IRC | 20:46 | |
| *** directxman12 has left #openstack-nova | 20:47 | |
| *** patrickeast has joined #openstack-nova | 20:47 | |
| *** ig0r__ has joined #openstack-nova | 20:50 | |
| *** ig0r_ has quit IRC | 20:50 | |
| *** patrickeast has quit IRC | 20:52 | |
| anish | prometheanfire: was that what you wanted ? yes/no/retry ? | 20:53 |
| dims | tmckay: trying to track down RFC(s)...nova seems to be using Generated-by-Nova, so you could do Generated-by-Sahara :) | 20:53 |
| tmckay | dims, ah, okay | 20:54 |
| *** annegentle has quit IRC | 20:54 | |
| tmckay | dims, filing the bug, just in case | 20:54 |
| dims | tmckay: "The optional comment field continues to the end of the line, and is not used." from sshd man page | 20:54 |
| tmckay | dims, so who wrote the hazmat stuff? Is that part of openstack, or another project that needs to be fixed? | 20:55 |
| tmckay | well, I guess it's the cryptography package | 20:55 |
| dims | tmckay: there's code in crypto.py which splits before the other code is called | 20:56 |
| tmckay | ah, I see | 20:56 |
| tmckay | gotcha | 20:56 |
| *** rushiagr_away has joined #openstack-nova | 20:56 | |
| *** thumpba has quit IRC | 20:56 | |
| *** otter768 has joined #openstack-nova | 20:56 | |
| *** patrickeast has joined #openstack-nova | 20:57 | |
| prometheanfire | anish: ya, now I need to figure out why I can't boot from volume :( | 20:58 |
| tmckay | dims, okay, https://bugs.launchpad.net/nova/+bug/1481084 | 20:59 |
| openstack | Launchpad bug 1481084 in OpenStack Compute (nova) "Keypair creation fails when ssh public key comment contains spaces" [Undecided,New] | 20:59 |
| prometheanfire | also, it seems that when I fail to boot from volume I can't just do a nova-delete | 20:59 |
| tmckay | let me know if you need any elaboration | 20:59 |
| prometheanfire | there is some manual cleanup? | 20:59 |
| *** SourabhP has quit IRC | 20:59 | |
| prometheanfire | nova is telling cinder to detach a volume that's already detached | 21:00 |
| prometheanfire | so cinder 500s | 21:00 |
| *** SourabhP has joined #openstack-nova | 21:00 | |
| prometheanfire | not sure if this is a cinder or a nova bug | 21:00 |
| prometheanfire | this is stable/kilo | 21:00 |
| *** bapalm_ has quit IRC | 21:00 | |
| *** haomaiwang has quit IRC | 21:01 | |
| *** lucas__ has quit IRC | 21:01 | |
| *** thorst has quit IRC | 21:01 | |
| *** otter768 has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-nova | 21:01 | |
| mriedem | prometheanfire: possibly related https://review.openstack.org/#/c/186742/ | 21:07 |
| mriedem | prometheanfire: and https://review.openstack.org/#/c/204231/ | 21:08 |
| mriedem | ^ sounds like what you're looking for specifically | 21:08 |
| prometheanfire | ya, that last one is good | 21:08 |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Moves OVS related code to a common location https://review.openstack.org/179727 | 21:09 |
| *** patrickeast has quit IRC | 21:09 | |
| prometheanfire | guess I review :P | 21:11 |
| prometheanfire | mriedem: offhand, you know how I need to clean up? | 21:11 |
| *** ndipanov has quit IRC | 21:12 | |
| *** Sukhdev has quit IRC | 21:12 | |
| mriedem | eek, not really | 21:13 |
| mriedem | force-delete doesn't fix a thing like this | 21:14 |
| mriedem | you could delete the related block_device_mapping entries from the nova database.... | 21:14 |
| anish | prometheanfire: there's one more patch out there fixing a similar issue, can't find it | 21:15 |
| anish | the end solution on that one at least was this is already fixed in os-brick | 21:15 |
| *** tmckay has quit IRC | 21:15 | |
| *** smatzek has quit IRC | 21:16 | |
| mriedem | anish: nova doesn't use os-brick in kilo | 21:16 |
| *** Sukhdev has joined #openstack-nova | 21:16 | |
| *** mjturek1 has quit IRC | 21:17 | |
| *** ctrath has quit IRC | 21:18 | |
| *** thangp has quit IRC | 21:19 | |
| mriedem | scottda: there is a cinderclient improvement to be had here https://review.openstack.org/#/c/204231/ | 21:20 |
| *** meghal has quit IRC | 21:21 | |
| *** ig0r__ has quit IRC | 21:21 | |
| *** meghal has joined #openstack-nova | 21:21 | |
| scottda | mriedem: thanks, I'll have a look | 21:22 |
| mriedem | added cinderclient to the bug | 21:22 |
| *** meghal has quit IRC | 21:22 | |
| *** meghal has joined #openstack-nova | 21:23 | |
| *** meghal has quit IRC | 21:23 | |
| *** meghal has joined #openstack-nova | 21:23 | |
| *** meghal has quit IRC | 21:23 | |
| *** meghal has joined #openstack-nova | 21:24 | |
| *** rlrossit has left #openstack-nova | 21:24 | |
| *** meghal has quit IRC | 21:24 | |
| *** meghal has joined #openstack-nova | 21:25 | |
| anish | mriedem: I know. this is for a liberty I believe | 21:25 |
| *** meghal has quit IRC | 21:25 | |
| anish | but the symptoms were the same as waht prometheanfire is seeing | 21:25 |
| *** Marga_ has quit IRC | 21:25 | |
| *** meghal has joined #openstack-nova | 21:25 | |
| *** meghal has quit IRC | 21:26 | |
| *** meghal has joined #openstack-nova | 21:26 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Better handling of SSH key comments https://review.openstack.org/208661 | 21:28 |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: Servicegroup foundational refactoring for Control Plane https://review.openstack.org/202714 | 21:29 |
| prometheanfire | well, guess it's time to dive into the db | 21:29 |
| *** hemna has quit IRC | 21:31 | |
| *** mriedem is now known as mriedem_away | 21:31 | |
| prometheanfire | anish: mriedem_away: this might my a related but difernt bug | 21:36 |
| *** vilobhmm has quit IRC | 21:36 | |
| *** vilobhmm has joined #openstack-nova | 21:36 | |
| *** hemna has joined #openstack-nova | 21:37 | |
| *** pixelbeat_ has quit IRC | 21:43 | |
| *** ijw has joined #openstack-nova | 21:44 | |
| *** mnestratov has quit IRC | 21:44 | |
| *** annegentle has joined #openstack-nova | 21:47 | |
| prometheanfire | mriedem_away: deleting from the mapping table worked, thanks | 21:48 |
| *** vladikr has joined #openstack-nova | 21:54 | |
| *** jecarey has quit IRC | 21:56 | |
| *** Jeffrey4l has joined #openstack-nova | 21:58 | |
| *** burt has quit IRC | 21:58 | |
| *** tongli has quit IRC | 21:59 | |
| *** kurtmartin is now known as kmartin | 21:59 | |
| *** mwagner_lap has quit IRC | 21:59 | |
| *** meghal has quit IRC | 21:59 | |
| *** haomaiwang has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-nova | 22:02 | |
| *** agireud has quit IRC | 22:02 | |
| *** thedodd has quit IRC | 22:04 | |
| *** ijw has quit IRC | 22:06 | |
| *** ijw has joined #openstack-nova | 22:07 | |
| *** slong has quit IRC | 22:13 | |
| *** agireud has joined #openstack-nova | 22:14 | |
| *** agireud has quit IRC | 22:18 | |
| *** kylek3h has quit IRC | 22:18 | |
| *** thumpba has joined #openstack-nova | 22:19 | |
| *** lpetrut has quit IRC | 22:20 | |
| *** hemna has quit IRC | 22:21 | |
| *** atuvenie has joined #openstack-nova | 22:23 | |
| *** busterswt has quit IRC | 22:25 | |
| *** VW_ has quit IRC | 22:27 | |
| *** thumpba has quit IRC | 22:29 | |
| *** vilobhmm has quit IRC | 22:33 | |
| *** annashen has joined #openstack-nova | 22:33 | |
| *** geekinutah has quit IRC | 22:35 | |
| *** annegentle has quit IRC | 22:35 | |
| *** kmartin has quit IRC | 22:36 | |
| *** vilobhmm has joined #openstack-nova | 22:37 | |
| *** annashen has quit IRC | 22:38 | |
| *** mdrabe has quit IRC | 22:38 | |
| *** mdrabe has joined #openstack-nova | 22:38 | |
| *** thumpba has joined #openstack-nova | 22:41 | |
| *** thumpba has quit IRC | 22:41 | |
| *** vilobhmm has quit IRC | 22:41 | |
| *** kmARC has quit IRC | 22:42 | |
| *** annegentle has joined #openstack-nova | 22:43 | |
| *** shakamunyi has quit IRC | 22:51 | |
| *** dims_ has joined #openstack-nova | 22:51 | |
| *** moshele_ has joined #openstack-nova | 22:52 | |
| *** dims has quit IRC | 22:52 | |
| prometheanfire | so... | 22:52 |
| *** dims has joined #openstack-nova | 22:53 | |
| prometheanfire | for nova boot from a cinder volume, cinder is complaining that it's not getting the initiator | 22:53 |
| *** mwagner_lap has joined #openstack-nova | 22:53 | |
| prometheanfire | which is sent from nova I think | 22:53 |
| prometheanfire | does nova need a special setup to work with cinder? | 22:55 |
| anish | no | 22:56 |
| *** dims_ has quit IRC | 22:56 | |
| *** otter768 has joined #openstack-nova | 22:57 | |
| *** patrickeast has joined #openstack-nova | 22:59 | |
| *** salv-orlando has joined #openstack-nova | 22:59 | |
| prometheanfire | well, here's some nova log for the failure (compute host side). https://gist.githubusercontent.com/prometheanfire/c9b70bc4b2abd7d384e6/raw/ae344feff8ef6c36f7685d1433b8e22b811cb03b/gistfile1.txt | 23:01 |
| *** haomaiwang has quit IRC | 23:01 | |
| prometheanfire | I almost want to blame keystone | 23:01 |
| *** vilobhmm has joined #openstack-nova | 23:01 | |
| *** haomaiwa_ has joined #openstack-nova | 23:02 | |
| *** otter768 has quit IRC | 23:02 | |
| prometheanfire | I'm not seeing any errors in keystone's log though | 23:02 |
| *** salv-orl_ has quit IRC | 23:02 | |
| *** tonytan_brb has quit IRC | 23:03 | |
| anish | prometheanfire: you're sure it's not just a firewall issue yes ? | 23:06 |
| prometheanfire | no fw between hosts | 23:06 |
| *** VW_ has joined #openstack-nova | 23:06 | |
| *** moshele_ has quit IRC | 23:07 | |
| prometheanfire | well, neutron-linuxbridge manages iptables on the comupte hosts | 23:08 |
| *** emagana has quit IRC | 23:08 | |
| openstackgerrit | vikas choudhary proposed openstack/nova: Update hypervisor resources correctly in booting failed case https://review.openstack.org/208680 | 23:09 |
| *** aix has quit IRC | 23:12 | |
| *** vilobhmm has quit IRC | 23:13 | |
| prometheanfire | anish: added some cinder-volume log to the bottom https://gist.githubusercontent.com/prometheanfire/c9b70bc4b2abd7d384e6/raw/475e3267e67ccdf2bc803324b41f27ddf3900c02/gistfile1.txt | 23:14 |
| *** VW_ has quit IRC | 23:15 | |
| *** praneshp has quit IRC | 23:15 | |
| prometheanfire | anish: any other logs you think would be applicable? | 23:15 |
| *** VW_ has joined #openstack-nova | 23:15 | |
| *** Hazelesque has joined #openstack-nova | 23:16 | |
| anish | I feel like you'd see an error on the nova side as well if you have no initiator info | 23:16 |
| anish | this is kilo release right, not devstack ? | 23:16 |
| *** Sukhdev has quit IRC | 23:16 | |
| prometheanfire | ya, kilo | 23:17 |
| prometheanfire | running from stable/kilo | 23:17 |
| openstackgerrit | Anthony Lee proposed openstack/nova: Fix live-migrations usage of the wrong connector information https://review.openstack.org/202770 | 23:17 |
| prometheanfire | the top log lines are from nova-compute on the compute host | 23:17 |
| prometheanfire | so that is from nova | 23:17 |
| anish | not sure, sorry. just tried it on mine, do not see the issue | 23:18 |
| prometheanfire | damn | 23:18 |
| *** dannywilson has joined #openstack-nova | 23:18 | |
| prometheanfire | anish: does the nova compute host expect any cinder config options to be added? | 23:19 |
| *** VW_ has quit IRC | 23:20 | |
| anish | I've never done any changes on the nova side for iscsi | 23:20 |
| anish | unless you have a borked open-iscsi setup on the nova side | 23:20 |
| prometheanfire | well open-iscsi is installed with no mods | 23:21 |
| anish | check if this file exists : /etc/iscsi/initiatorname.iscsi | 23:21 |
| prometheanfire | exists | 23:21 |
| prometheanfire | nothing uncommented, but exists | 23:22 |
| *** Sukhdev has joined #openstack-nova | 23:22 | |
| *** aysyd has quit IRC | 23:22 | |
| anish | prometheanfire: supposed to look like this http://ix.io/k5u | 23:22 |
| prometheanfire | open-iscsi is a service that needs starting? | 23:23 |
| anish | yes | 23:24 |
| prometheanfire | well, that explains at least some of it | 23:24 |
| anish | iscsid & iscsid.socket if you have systemd | 23:24 |
| *** annegentle has quit IRC | 23:24 | |
| anish | open-iscsi if you've installed in manually | 23:24 |
| prometheanfire | but not running on the cinder node | 23:24 |
| anish | instead of repos | 23:24 |
| anish | no | 23:24 |
| anish | open-iscsi is only the initiator | 23:24 |
| prometheanfire | I'm packaging it for gentoo | 23:24 |
| prometheanfire | right | 23:25 |
| anish | cinder node is tgt/lio | 23:25 |
| anish | or iet | 23:25 |
| prometheanfire | ya, tgt here | 23:25 |
| *** patrickeast has quit IRC | 23:25 | |
| prometheanfire | hmm, I do have InitiatorName=iqn.2015-08.:openiscsi-83bfdf0741c8f8120d9324fa38c3f7e5 | 23:26 |
| anish | I would add something between the . & the : | 23:26 |
| *** Vikas_ has joined #openstack-nova | 23:26 | |
| anish | idk if that makes a difference, but some targets will reject that | 23:27 |
| prometheanfire | ok | 23:27 |
| *** Sukhdev has quit IRC | 23:28 | |
| *** dims has quit IRC | 23:29 | |
| *** thorst has joined #openstack-nova | 23:30 | |
| *** marcusvrn has quit IRC | 23:33 | |
| *** ddaskal has quit IRC | 23:38 | |
| *** ijw_ has joined #openstack-nova | 23:39 | |
| *** echoingumesh has quit IRC | 23:40 | |
| *** echoingumesh has joined #openstack-nova | 23:41 | |
| *** ijw has quit IRC | 23:42 | |
| *** echoingumesh has quit IRC | 23:45 | |
| *** atuvenie has quit IRC | 23:46 | |
| *** tpeoples has quit IRC | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!