Monday, 2015-08-03

*** IanGovett has joined #openstack-nova00:00
*** markvoelker has quit IRC00:02
*** dims_ has joined #openstack-nova00:02
*** dimsum__ has quit IRC00:04
*** adam_g has quit IRC00:09
*** adam_g has joined #openstack-nova00:10
*** adam_g has quit IRC00:10
*** adam_g has joined #openstack-nova00:10
*** annashen has joined #openstack-nova00:24
*** annashen has quit IRC00:28
*** jasondotstar has joined #openstack-nova00:28
*** tpeoples has quit IRC00:35
*** pixelbeat_ has quit IRC00:43
*** mriedem has quit IRC00:43
*** salv-orlando has joined #openstack-nova00:46
*** dims_ has quit IRC00:50
*** dimsum__ has joined #openstack-nova00:53
openstackgerritsajeesh cs proposed openstack/nova: NestedQuotaDriver:Modification of settable quotas of nested projects  https://review.openstack.org/20034200:54
*** rmk has quit IRC00:55
*** salv-orlando has quit IRC00:55
openstackgerritHiroyuki Eguchi proposed openstack/nova: qemu-img convert should be skipped when migrating  https://review.openstack.org/13897900:59
openstackgerritHiroyuki Eguchi proposed openstack/nova: qemu-img convert should be skipped when migrating  https://review.openstack.org/13897901:02
openstackgerritGhanshyam Mann proposed openstack/nova: Fix link's href to consider osapi_compute_link_prefix  https://review.openstack.org/20770201:04
*** rmk has joined #openstack-nova01:07
*** jamielennox is now known as jamielennox|away01:08
*** haigang has joined #openstack-nova01:10
*** artom has quit IRC01:14
*** kiwik-chenrui has quit IRC01:17
*** RuiChen has joined #openstack-nova01:18
*** park_heijlong has quit IRC01:19
*** park_heijlong has joined #openstack-nova01:22
openstackgerritsajeesh cs proposed openstack/nova: Nested Quota Driver: Finding parent_id and immediate child list  https://review.openstack.org/20094101:23
*** meghal has joined #openstack-nova01:23
*** wanghua has joined #openstack-nova01:24
*** tpeoples has joined #openstack-nova01:24
*** achanda has joined #openstack-nova01:25
*** dimsum__ has quit IRC01:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtISCSIVolumeDriver into it's own module  https://review.openstack.org/20230101:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtISERVolumeDriver into it's own module  https://review.openstack.org/19731101:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtNFSVolumeDriver into it's own module  https://review.openstack.org/19685901:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtNETVolumeDriver into it's own module  https://review.openstack.org/19734101:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtAOEVolumeDriver into it's own module  https://review.openstack.org/19627601:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert SMBFS vol driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20277901:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtFibreChannelVolumeDriver into it's own module  https://review.openstack.org/19327801:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtGlusterfsVolumeDriver into it's own module  https://review.openstack.org/19338001:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert NFS volume driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20277801:25
openstackgerritMatt Riedemann proposed openstack/nova: Introduce LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20277701:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert Scality vol driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278301:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert Quobyte driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278201:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert GPFS volume driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278101:25
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtSMBFSVolumeDriver into it's own module  https://review.openstack.org/19638001:26
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert GlusterFS driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278001:26
*** dimsum__ has joined #openstack-nova01:28
*** shihanzhang has quit IRC01:28
*** mmedvede has quit IRC01:30
*** mmedvede has joined #openstack-nova01:31
openstackgerritZhenyu Zheng proposed openstack/nova: DB:support for migration objects multi-statuses  https://review.openstack.org/16960401:33
*** tpeoples has quit IRC01:33
*** tpeoples has joined #openstack-nova01:33
*** humble_ has joined #openstack-nova01:34
*** shuquan has joined #openstack-nova01:35
*** haigang has quit IRC01:35
*** haigang has joined #openstack-nova01:36
*** wanghua has quit IRC01:37
*** tpeoples has quit IRC01:38
*** shuquan_ has joined #openstack-nova01:40
*** shuquan has quit IRC01:40
*** shuquan_ is now known as shuquan01:40
*** meghal has quit IRC01:44
*** lyanchih_ has joined #openstack-nova01:46
*** lyanchih_ has left #openstack-nova01:46
*** jyuso1 has joined #openstack-nova01:46
*** lyanchih_ has joined #openstack-nova01:47
*** lyanchih_ has quit IRC01:50
*** lyanchih_ has joined #openstack-nova01:50
*** lyanchih_ has left #openstack-nova01:51
*** lyanchih_ has joined #openstack-nova01:53
*** salv-orlando has joined #openstack-nova01:54
*** shuquan_ has joined #openstack-nova01:58
*** shuquan has quit IRC01:58
*** shuquan_ is now known as shuquan01:58
*** markvoelker has joined #openstack-nova01:59
*** shuquan has quit IRC02:02
*** dimsum__ has quit IRC02:02
*** humble__ has joined #openstack-nova02:02
*** shuquan has joined #openstack-nova02:02
*** markvoelker has quit IRC02:03
*** humble_ has quit IRC02:05
*** salv-orlando has quit IRC02:06
*** whenry has joined #openstack-nova02:08
*** haomaiw__ has joined #openstack-nova02:09
openstackgerritlyanchih proposed openstack/nova: Remove error instance fail when enable serial_consol  https://review.openstack.org/20826802:09
*** IanGovett has quit IRC02:10
*** haomaiwang has quit IRC02:11
*** baoli has joined #openstack-nova02:14
*** shuquan has quit IRC02:15
*** shuquan has joined #openstack-nova02:15
*** whenry has quit IRC02:24
*** annashen has joined #openstack-nova02:25
*** haomaiwang has joined #openstack-nova02:25
openstackgerritTakashi NATSUME proposed openstack/nova: Fix cinder volume stuck in swap_volume  https://review.openstack.org/20738502:26
*** kiran-r has joined #openstack-nova02:27
*** kiran-r has quit IRC02:27
*** haomaiw__ has quit IRC02:28
*** annashen has quit IRC02:29
*** shuquan has quit IRC02:31
*** baoli has quit IRC02:32
*** shuquan has joined #openstack-nova02:33
*** shuquan has quit IRC02:36
*** otter768 has joined #openstack-nova02:36
*** shuquan has joined #openstack-nova02:37
*** humble_ has joined #openstack-nova02:39
*** shuquan has quit IRC02:41
*** humble__ has quit IRC02:42
*** shuquan has joined #openstack-nova02:44
*** hakimo_ has joined #openstack-nova02:52
*** hakimo has quit IRC02:54
*** baoli has joined #openstack-nova02:54
*** baoli has quit IRC03:05
*** diogogmt has quit IRC03:08
*** alejandrito has quit IRC03:12
*** tjones has joined #openstack-nova03:15
*** tpeoples has joined #openstack-nova03:23
*** shuquan has quit IRC03:26
*** otter768 has quit IRC03:29
*** liusheng has joined #openstack-nova03:30
*** jecarey has joined #openstack-nova03:33
openstackgerritRui Chen proposed openstack/nova: Adds instance_uuid index for instance_system_metadata  https://review.openstack.org/20827703:36
*** coolsvap|away is now known as coolsvap03:40
*** shuquan has joined #openstack-nova03:40
*** mdrabe has joined #openstack-nova03:42
openstackgerritsajeesh cs proposed openstack/nova: Nested Quota Driver: Adding v2 and v3 support  https://review.openstack.org/14982803:46
*** FL1SK has quit IRC03:52
*** Jeffrey4l has joined #openstack-nova03:52
*** salv-orlando has joined #openstack-nova03:52
*** mdrabe has quit IRC03:53
*** salv-orlando has quit IRC03:58
*** salv-orl_ has joined #openstack-nova04:00
*** markvoelker has joined #openstack-nova04:00
openstackgerritlyanchih proposed openstack/nova: libvirt: serial console ports count upper limit needs to be checked  https://review.openstack.org/20737304:00
*** shuquan has quit IRC04:02
*** shuquan has joined #openstack-nova04:03
*** salv-orl_ has quit IRC04:04
*** markvoelker has quit IRC04:04
*** lyanchih_ has quit IRC04:05
*** sajeesh has quit IRC04:06
*** ijw has joined #openstack-nova04:07
*** tjones has quit IRC04:14
*** shuquan has quit IRC04:15
*** shuquan has joined #openstack-nova04:16
*** jecarey has quit IRC04:17
*** boris-42 has joined #openstack-nova04:18
*** coreywright has quit IRC04:24
*** annashen has joined #openstack-nova04:26
*** shyama has joined #openstack-nova04:27
*** unicell1 has joined #openstack-nova04:28
*** unicell has quit IRC04:28
*** ijw has quit IRC04:29
*** annashen has quit IRC04:31
*** coreywright has joined #openstack-nova04:34
*** Kennan has quit IRC04:35
*** Kennan has joined #openstack-nova04:36
*** armax has joined #openstack-nova04:37
*** shuquan has quit IRC04:38
*** jamielennox|away is now known as jamielennox04:38
*** deepakcs has joined #openstack-nova04:39
*** davideagnello has joined #openstack-nova04:41
*** FL1SK has joined #openstack-nova04:45
*** davideagnello has quit IRC04:46
*** rotbeard has joined #openstack-nova04:50
*** tpeoples has quit IRC04:52
openstackgerritJeffrey Zhang proposed openstack/nova: Use urlencode instead of dict_to_query_str function  https://review.openstack.org/20746804:54
*** shuquan has joined #openstack-nova04:56
*** slong has quit IRC04:57
*** armax has quit IRC05:01
*** kiran-r has joined #openstack-nova05:03
*** salv-orlando has joined #openstack-nova05:03
*** slong has joined #openstack-nova05:04
*** salv-orlando has quit IRC05:06
*** salv-orlando has joined #openstack-nova05:11
*** chhavi has joined #openstack-nova05:13
*** lyanchih_ has joined #openstack-nova05:16
*** sudipto has joined #openstack-nova05:19
*** Sukhdev has joined #openstack-nova05:21
*** fawadkhaliq has joined #openstack-nova05:23
openstackgerritlyanchih proposed openstack/nova: libvirt: serial console ports count upper limit needs to be checked  https://review.openstack.org/20737305:25
*** otter768 has joined #openstack-nova05:30
*** ig0r_ has joined #openstack-nova05:33
*** otter768 has quit IRC05:35
*** kmARC has joined #openstack-nova05:37
*** afazekas has joined #openstack-nova05:37
*** harlowja has quit IRC05:44
*** ig0r__ has joined #openstack-nova05:45
*** afazekas has quit IRC05:46
*** afazekas has joined #openstack-nova05:49
openstackgerritjichenjc proposed openstack/nova: Handle MessageTimeout to MigrationPreCheckError  https://review.openstack.org/16863505:49
*** meghal has joined #openstack-nova05:53
*** ig0r_ has quit IRC05:53
*** lyanchih_ has quit IRC05:55
*** ig0r_ has joined #openstack-nova05:55
*** fawadk has joined #openstack-nova05:56
*** fawadkhaliq has quit IRC05:58
*** josecastroleon has joined #openstack-nova05:58
*** lyanchih_ has joined #openstack-nova05:59
*** ishant has joined #openstack-nova06:00
*** liusheng has quit IRC06:00
*** markvoelker has joined #openstack-nova06:01
*** liusheng has joined #openstack-nova06:02
*** meghal has quit IRC06:02
*** lennyb has joined #openstack-nova06:02
*** haigang has quit IRC06:03
*** haigang has joined #openstack-nova06:04
*** markvoelker has quit IRC06:05
openstackgerritGhanshyam Mann proposed openstack/nova-specs: Clarifying more about lock interface in spec  https://review.openstack.org/20832606:08
*** Kennan has quit IRC06:09
*** pcaruana has quit IRC06:09
*** Kennan has joined #openstack-nova06:11
*** shuquan has quit IRC06:13
*** shuquan has joined #openstack-nova06:14
*** haigang has quit IRC06:16
*** shuquan has quit IRC06:17
openstackgerritjichenjc proposed openstack/nova: Refactor test cases for live-migrate error case  https://review.openstack.org/17904806:17
*** ankit_ag has joined #openstack-nova06:20
*** haigang has joined #openstack-nova06:20
*** shuquan has joined #openstack-nova06:22
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex  https://review.openstack.org/19721306:22
*** scheuran has joined #openstack-nova06:23
*** belmoreira has joined #openstack-nova06:25
*** afazekas has quit IRC06:27
*** fawadk has quit IRC06:27
*** Murali has joined #openstack-nova06:30
*** afazekas has joined #openstack-nova06:31
openstackgerritZhenyu Zheng proposed openstack/nova: Allow resize root_gb to 0 for volume-backed instances  https://review.openstack.org/20779406:35
*** Murali_ has joined #openstack-nova06:38
*** shuquan has quit IRC06:38
*** Murali has quit IRC06:39
*** Murali_ is now known as Murali06:40
*** Murali_ has joined #openstack-nova06:42
*** rdopiera has joined #openstack-nova06:44
*** Murali has quit IRC06:44
*** Murali_ is now known as Murali06:45
*** sudipto has quit IRC06:45
openstackgerritRadomir Dopieralski proposed openstack/nova: Handle SSL termination proxies for version list  https://review.openstack.org/20647906:51
*** kmARC has quit IRC06:56
*** gibi has joined #openstack-nova06:57
*** sahid has joined #openstack-nova07:00
openstackgerritGhanshyam Mann proposed openstack/nova: Move "versions" functional tests in v2.1 tests  https://review.openstack.org/20143907:00
*** sahid has quit IRC07:02
*** garyk has joined #openstack-nova07:02
*** browne1 has quit IRC07:04
*** sahid has joined #openstack-nova07:06
*** Sukhdev has quit IRC07:07
*** markus_z has joined #openstack-nova07:09
*** boris-42 has quit IRC07:10
*** achanda has quit IRC07:13
*** achanda has joined #openstack-nova07:17
openstackgerritZhenyu Zheng proposed openstack/nova: DB:support for migration objects multi-statuses  https://review.openstack.org/16960407:18
*** salv-orlando has quit IRC07:22
openstackgerritGhanshyam Mann proposed openstack/nova: Fix link's href to consider osapi_compute_link_prefix  https://review.openstack.org/20770207:22
*** jcoufal has joined #openstack-nova07:26
*** otter768 has joined #openstack-nova07:31
*** shuquan has joined #openstack-nova07:33
*** vivekd has joined #openstack-nova07:33
openstackgerritGhanshyam Mann proposed openstack/nova: Move "versions" functional tests in v2.1 tests  https://review.openstack.org/20143907:34
*** otter768 has quit IRC07:36
*** achanda has quit IRC07:36
openstackgerritRui Chen proposed openstack/nova: Adds instance_uuid index for instance_system_metadata  https://review.openstack.org/20827707:36
*** kjelly has quit IRC07:37
*** pcaruana has joined #openstack-nova07:40
*** jlanoux has joined #openstack-nova07:47
*** salv-orlando has joined #openstack-nova07:48
*** kmARC has joined #openstack-nova07:54
*** echirivella has quit IRC07:55
*** jamielennox is now known as jamielennox|away07:57
*** romainh has joined #openstack-nova07:58
*** moshele has joined #openstack-nova08:01
*** fawadkhaliq has joined #openstack-nova08:01
*** markvoelker has joined #openstack-nova08:02
*** moshele has quit IRC08:02
*** moshele_ has joined #openstack-nova08:02
*** markvoelker has quit IRC08:06
*** echirivella has joined #openstack-nova08:08
*** subscope has joined #openstack-nova08:10
openstackgerritShiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI  https://review.openstack.org/20260508:13
*** jistr has joined #openstack-nova08:15
*** danpb has joined #openstack-nova08:15
*** alex_klimov has joined #openstack-nova08:16
*** aix has joined #openstack-nova08:18
openstackgerritMaxim Nestratov proposed openstack/nova: Fix deleting instance with incorrectly attached volume  https://review.openstack.org/20423108:18
*** yassine has joined #openstack-nova08:20
garykjohnthetubaguy: not sure if you are around. addressed the comments - https://review.openstack.org/18020208:22
*** annashen has joined #openstack-nova08:27
*** MarcF has joined #openstack-nova08:28
*** fawadkhaliq has quit IRC08:28
openstackgerritMaxim Nestratov proposed openstack/nova: Fix deleting instance with incorrectly attached volume  https://review.openstack.org/20423108:28
*** zhenguo has joined #openstack-nova08:29
*** fawadkhaliq has joined #openstack-nova08:30
openstackgerritRadoslav Gerganov proposed openstack/nova: VMware: add serial port device  https://review.openstack.org/14920308:31
*** annashen has quit IRC08:32
*** kjelly has joined #openstack-nova08:34
*** ftersin has joined #openstack-nova08:35
openstackgerritSergey Nikitin proposed openstack/nova: Added server tags support in nova-api  https://review.openstack.org/12894008:37
openstackgerritSergey Nikitin proposed openstack/nova: Fixed incorrect behaviour of method _check_instance_exists  https://review.openstack.org/20303708:37
*** Kevin_Zheng_ has joined #openstack-nova08:39
*** huanxie has joined #openstack-nova08: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-nova08:40
*** davideagnello has joined #openstack-nova08:43
*** humble_ has quit IRC08:44
*** davideagnello has quit IRC08:47
openstackgerritRadoslav Gerganov proposed openstack/nova: VMware: map one nova-compute to one VC cluster  https://review.openstack.org/10391608:47
*** jianghjw has joined #openstack-nova08:48
openstackgerritAdelina Tuvenie proposed openstack/nova: Fixes Bug "destroy_vm fails with HyperVException"  https://review.openstack.org/20400708:55
*** rubasov has joined #openstack-nova09:00
*** Nikolay_St has joined #openstack-nova09:02
*** fawadkhaliq has quit IRC09:03
openstackgerritFeodor Tersin proposed openstack/nova: Fix collection of metadata for a snapshot of a volume-backed instance  https://review.openstack.org/17024309:05
openstackgerritFeodor Tersin proposed openstack/nova: Consolidate initialization of instance snapshot metadata  https://review.openstack.org/19895909:05
openstackgerritFeodor Tersin proposed openstack/nova: Simplify interface for creating snapshot of volume-backed instance  https://review.openstack.org/19895809:05
*** fawadkhaliq has joined #openstack-nova09:06
*** ftersin has left #openstack-nova09:08
*** ftersin has joined #openstack-nova09:08
*** andreykurilin__ has joined #openstack-nova09:10
*** humble_ has joined #openstack-nova09:16
*** humble__ has quit IRC09:19
lexlooferMorning09:20
*** lexloofer is now known as lxsli09:20
lxsliAhem09:20
openstackgerritSergey Nikitin proposed openstack/nova: Fixed random failing of test_describe_instances_with_filters_tags  https://review.openstack.org/20825309:20
*** atuvenie has joined #openstack-nova09:25
*** Jeffrey4l has quit IRC09:26
*** humble__ has joined #openstack-nova09:27
openstackgerritSergey Nikitin proposed openstack/nova: Fixed random failing of test_describe_instances_with_filters_tags  https://review.openstack.org/20825309:28
*** abhishekk has joined #openstack-nova09:30
*** humble_ has quit IRC09:31
*** kumar has joined #openstack-nova09:31
*** otter768 has joined #openstack-nova09:32
*** sudipto has joined #openstack-nova09:32
*** fawadkhaliq has quit IRC09:32
*** fawadkhaliq has joined #openstack-nova09:33
*** jianghjw has quit IRC09:35
*** otter768 has quit IRC09:37
*** Jeffrey4l has joined #openstack-nova09:39
*** rushiagr_away is now known as rushiagr09:39
*** kumar has quit IRC09:39
*** tojuvone has joined #openstack-nova09:40
*** gszasz has joined #openstack-nova09:40
*** fawadkhaliq has quit IRC09:45
*** fawadkhaliq has joined #openstack-nova09:45
*** marzif_ has quit IRC09:46
*** marzif_ has joined #openstack-nova09:47
*** moshele_ has quit IRC09:48
*** marzif_ has quit IRC09:48
*** shuquan has quit IRC09:49
*** kjelly has quit IRC09:50
*** romainh has quit IRC09:53
*** rdopiera has quit IRC09:54
openstackgerritlyanchih proposed openstack/nova: Add unit tests for libvirt fibrechannel/iser/iscsi/aoe volume drivers after os-brick  https://review.openstack.org/20840109:56
*** marzif has joined #openstack-nova09:57
*** ftersin_ has joined #openstack-nova09:57
*** ftersin_ has quit IRC09:58
*** fawadkhaliq has quit IRC09:58
*** ftersin has quit IRC09:59
*** salv-orlando has quit IRC09:59
*** jcoufal_ has joined #openstack-nova10:02
*** markvoelker has joined #openstack-nova10:02
*** eliqiao has joined #openstack-nova10:03
*** sabari- has joined #openstack-nova10:06
*** kylek3h_ has joined #openstack-nova10:06
*** aix has quit IRC10:06
*** markvoelker has quit IRC10:07
*** ndipanov has joined #openstack-nova10:07
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds RemoteFX support to the Hyper-V driver  https://review.openstack.org/4252910:07
*** Kennan has quit IRC10:07
*** FL1SK has quit IRC10:07
*** sabari has quit IRC10:08
*** alex_klimov has quit IRC10:08
*** jcoufal has quit IRC10:08
*** jmccrory has quit IRC10:08
*** logan2 has quit IRC10:08
*** mwagner_lap has quit IRC10:08
*** rushiagr has quit IRC10:08
*** timello has quit IRC10:08
*** HenryG has quit IRC10:08
*** garyk has quit IRC10:08
*** sabari- is now known as sabari10:08
*** garyk has joined #openstack-nova10:08
*** jmccrory has joined #openstack-nova10:08
*** HenryG has joined #openstack-nova10:08
*** kylek3h has quit IRC10:08
*** timello has joined #openstack-nova10:08
*** mwagner_lap has joined #openstack-nova10:08
*** logan2 has joined #openstack-nova10:08
*** pixelbeat_ has joined #openstack-nova10:08
*** alex_klimov has joined #openstack-nova10:09
*** Kennan has joined #openstack-nova10:09
openstackgerritlyanchih proposed openstack/nova: libvirt: serial console ports count upper limit needs to be checked  https://review.openstack.org/20737310:09
*** Jeffrey4l has quit IRC10:10
haiganganybody?10:13
*** rdo has quit IRC10:14
*** rdo has joined #openstack-nova10:16
garykany cores around to look at https://review.openstack.org/179066 - it has 2 +2 and lost the +A as it was rebased10:16
*** ftersin has joined #openstack-nova10:17
*** jyuso1 has quit IRC10:17
*** josecastroleon has quit IRC10:20
*** huanxie has quit IRC10:20
*** fawadkhaliq has joined #openstack-nova10:21
*** lyanchih_ has quit IRC10:22
sudiptolyanchih, 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
openstackgerritgaryk proposed openstack/nova: VMware: improve instance names on VC  https://review.openstack.org/16660810:28
openstackgerritgaryk proposed openstack/nova: VMware: add in folder support on VC  https://review.openstack.org/16506010:28
*** annashen has joined #openstack-nova10:28
*** eliqiao has quit IRC10:29
openstackgerritMaxim Nestratov proposed openstack/nova: Fix deleting instance with incorrectly attached volume  https://review.openstack.org/20423110:31
*** moshele_ has joined #openstack-nova10:31
*** moshele_ has quit IRC10:32
*** moshele_ has joined #openstack-nova10:32
*** annashen has quit IRC10: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-nova10:34
garykare people also find it difficult to upload patches?10:44
*** haigang has quit IRC10:44
openstackgerritgaryk proposed openstack/nova: VMware: ensure that get_info raises the correct exception  https://review.openstack.org/19155810:45
mnestratovgaryk: yep, it takes a bit longer than usual10:46
_gryfany 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
garykmnestratov: ok, thanks. it is really bad at the moment10:46
*** haigang has joined #openstack-nova10:48
*** kjelly has joined #openstack-nova10:49
openstackgerritgaryk proposed openstack/nova: VMware: treat deletion exception with attached volumes  https://review.openstack.org/19867410:50
*** pcaruana has quit IRC10:57
*** mwagner_lap has quit IRC10:57
*** josecastroleon has joined #openstack-nova11:02
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V: Adds VolumeOps unit tests  https://review.openstack.org/13979711:02
*** claudiub has joined #openstack-nova11:03
*** coolsvap is now known as coolsvap|away11:06
openstackgerritgaryk proposed openstack/nova: VMware: treat deletion exception with attached volumes  https://review.openstack.org/19867411:08
openstackgerritgaryk proposed openstack/nova: VMware: ensure that get_info raises the correct exception  https://review.openstack.org/19155811:08
*** marzif has quit IRC11:09
*** marzif has joined #openstack-nova11:10
*** Marga__ has quit IRC11:13
*** rushiagr has joined #openstack-nova11:13
*** pcaruana has joined #openstack-nova11:14
*** dheeraj-gupta-4 has joined #openstack-nova11:16
*** thorst has joined #openstack-nova11:17
openstackgerritSergey Nikitin proposed openstack/nova: Added server tags support in nova-api  https://review.openstack.org/12894011:17
*** dims_ has joined #openstack-nova11:18
*** dimsum__ has quit IRC11:20
garykjohnthetubaguy: can you please look at https://review.openstack.org/18020211:21
*** marzif has quit IRC11:27
*** jaypipes has joined #openstack-nova11:27
*** edmondsw has joined #openstack-nova11:31
*** kiran-r has quit IRC11:31
openstackgerritShiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI  https://review.openstack.org/20261711:32
*** otter768 has joined #openstack-nova11:33
*** markvoelker has joined #openstack-nova11:33
*** otter768 has quit IRC11:37
*** markvoelker has quit IRC11:38
*** rdo has quit IRC11:41
*** sudipto has quit IRC11:41
*** dheeraj-gupta-4 has quit IRC11:42
*** rdo has joined #openstack-nova11:42
openstackgerritSergey Nikitin proposed openstack/nova: Fixed random failing of test_describe_instances_with_filters_tags  https://review.openstack.org/20825311:47
*** smatzek has joined #openstack-nova11:48
*** marcusvrn has joined #openstack-nova11:49
*** markvoelker has joined #openstack-nova11:50
openstackgerritZhenyu Zheng proposed openstack/nova: Add instance action events for live migration  https://review.openstack.org/19537211:50
*** wanghua has joined #openstack-nova11:50
*** garyk has quit IRC11:50
*** shyama has quit IRC11:50
*** romainh has joined #openstack-nova11:51
*** shyama has joined #openstack-nova11:51
*** humble__ has quit IRC11:53
*** markvoelker_ has joined #openstack-nova11:54
*** markvoelker has quit IRC11:54
*** dims_ has quit IRC11:54
*** dimsum__ has joined #openstack-nova11:55
*** claudiub has quit IRC11:55
*** htruta_ has joined #openstack-nova11:58
*** coolsvap|away is now known as coolsvap11:58
*** pixelbeat_ has quit IRC12:00
*** garyk has joined #openstack-nova12:01
*** FL1SK has joined #openstack-nova12:03
*** fawadkhaliq has quit IRC12:04
*** Marga_ has joined #openstack-nova12:05
*** dimsum__ has quit IRC12:08
*** dprince has joined #openstack-nova12:08
*** chhavi has quit IRC12:08
garykdanpb: any chance you can please look at https://review.openstack.org/179066 (it was rebased)12:13
*** raildo has joined #openstack-nova12:18
*** rfolco has joined #openstack-nova12:21
*** dimsum__ has joined #openstack-nova12:22
*** jasondotstar has quit IRC12:22
*** marzif has joined #openstack-nova12:24
*** marzif has quit IRC12:24
*** marzif has joined #openstack-nova12:25
*** andreykurilin__ has quit IRC12:25
*** dims_ has joined #openstack-nova12:27
*** apuimedo has joined #openstack-nova12:28
*** dims__ has joined #openstack-nova12:29
*** dimsum__ has quit IRC12:29
*** annashen has joined #openstack-nova12:29
*** Marga_ has quit IRC12:30
*** dimsum__ has joined #openstack-nova12:32
*** marzif_ has joined #openstack-nova12:32
*** dims_ has quit IRC12:32
*** jasondotstar has joined #openstack-nova12:32
johnthetubaguygaryk: been away, in a field, but back now, I can take a quick look at the ones that need +2 reapplying, etc12:33
*** marzif has quit IRC12:33
*** annashen has quit IRC12:33
*** dims__ has quit IRC12:34
garykjohnthetubaguy: camping?12:34
johnthetubaguygaryk: yeah, well at a folk festival, and camping their12:35
*** ferest has joined #openstack-nova12:35
*** ferest has quit IRC12:36
*** pixelbeat_ has joined #openstack-nova12:36
johnthetubaguys/their/there/12:36
johnthetubaguyone day, I will learn english12:37
* StevenK waits for "Today is not this day"12:37
johnthetubaguygaryk: have you got any patches that lost their +W from rebasing, etc?12:38
garykjohnthetubaguy: https://review.openstack.org/17906612:38
*** Jeffrey4l has joined #openstack-nova12:39
_gryfjohnthetubaguy, hi12:39
garyknice. hope the weather was fine12:39
johnthetubaguygaryk: 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: hi12:40
_gryfjohnthetubaguy, 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-nova12:41
*** rdopiera has quit IRC12:41
*** rdopiera_ has joined #openstack-nova12:41
johnthetubaguy_gryf: possibly, I am going to start looking through all the blueprints, and starting to postpone things today12:42
johnthetubaguy_gryf: I will let you know how close it looks12:42
_gryfjohnthetubaguy, 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 possible12:42
*** andreykurilin__ has joined #openstack-nova12:43
*** zhenguo has quit IRC12:43
_gryfjohnthetubaguy, I'm waiting with the implementation for ~2 months to be reviewed12: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 work12:44
*** davideagnello has joined #openstack-nova12:44
*** dims_ has joined #openstack-nova12: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 IRC12:46
*** aysyd has joined #openstack-nova12: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
johnthetubaguyjaypipes: 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/18020212:49
*** mnestratov has quit IRC12:49
*** davideagnello has quit IRC12:49
*** mnestratov has joined #openstack-nova12:49
*** baoli has joined #openstack-nova12:49
*** kylek3h_ has quit IRC12:49
johnthetubaguymoshele_: 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 really12:50
*** mdrabe has joined #openstack-nova12:50
*** Jeffrey4l_ has joined #openstack-nova12:51
moshele_johnthetubaguy: I will be happy if it will this week :), but take your time12:52
*** dims_ has quit IRC12:53
jaypipesjohnthetubaguy: me today: http://www.memes.com/img/51057112:53
*** dimsum__ has joined #openstack-nova12:54
*** Jeffrey4l has quit IRC12:54
jaypipesjohnthetubaguy: I thought I was pretty adamant about that being net_io_, not vif_ ...12:54
johnthetubaguyjaypipes: OK so I have a vision of you and a load of pugs all lying on the floor in the same pose12:54
jaypipesjohnthetubaguy: is there a reason it was changed back?12:55
*** dimsum__ is now known as dims12:55
*** jeffrey__ has joined #openstack-nova12:55
jaypipesjohnthetubaguy: heh, yes, that definitely is the case some days :)12:55
johnthetubaguyjaypipes: 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 similar12:56
johnthetubaguyjaypipes: I suspect there is something big I am missing here12:56
jaypipesjohnthetubaguy: 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-nova12:57
*** Jeffrey4l_ has quit IRC12:58
johnthetubaguyjaypipes: 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
jaypipesjohnthetubaguy: hmmm...12:59
jaypipesdanpb: around?12:59
*** flip214 is now known as flip56212:59
*** flip562 is now known as flip21412:59
lxsligood morning Jay! when you're done discussing vif12:59
*** otter768 has joined #openstack-nova13:00
jaypipeslxsli: yup (though we have scheduler meeting in an hour right?)13:00
lxslijaypipes: how about that, indeed13:01
*** markvoelker_ has quit IRC13:02
lxslisometime today would be nice, otherwise all good13:02
*** markvoelker has joined #openstack-nova13:04
johnthetubaguyjaypipes: I should clarify, I think its our libvirt driver that uses that, rather than libvirt13:04
*** rlrossit has joined #openstack-nova13:08
openstackgerritAdelina Tuvenie proposed openstack/nova: Moves OVS related code to a common location  https://review.openstack.org/17972713:08
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver  https://review.openstack.org/14004513:08
*** aix has joined #openstack-nova13:11
*** dustins has joined #openstack-nova13:13
*** browne has quit IRC13:13
garykanyone know why gerrit is so painfully slow13:13
StevenKOh, I thought it was just this hotel wifi being terrible. It's not just me!13:14
*** dane-fichter has joined #openstack-nova13:16
*** haigang has quit IRC13:17
*** kylek3h has joined #openstack-nova13:17
*** romainh has left #openstack-nova13:18
garyknope. reviewing and posting patches is taking so long ….13:20
*** chuckcarmack has joined #openstack-nova13:20
*** zul has joined #openstack-nova13:21
*** boris-42 has joined #openstack-nova13:22
johnthetubaguygaryk: this doesn't help really, but same here :(13:22
johnthetubaguyit was my internet connection the other day, but this is different13:23
johnthetubaguyor feels like it is13:23
garykit has been like this for the last 2 days. but today is far worse13:23
*** haomaiw__ has joined #openstack-nova13:24
garykmaybe winging on infra will help13:24
dane-fichtergerrit is like molasses today13:24
*** zzzeek has joined #openstack-nova13:24
tojuvonejohnthetubaguy: 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
garykttx: maybe you know who can look into gerrit performance issue…. openstack is imploding13:26
johnthetubaguytojuvone: in theory you should be fine, certainly for today you should be fine13:26
*** ig0r_ has quit IRC13:26
*** ig0r__ has quit IRC13:27
*** haomaiwang has quit IRC13:27
*** bapalm has joined #openstack-nova13:27
ttxlooks like they are not up yet13:28
tojuvoneThanks :)13:28
*** abhishekk has quit IRC13:28
*** mjturek1 has joined #openstack-nova13:31
*** pedroalvarez has quit IRC13:32
*** ig0r_ has joined #openstack-nova13:32
*** otter768 has quit IRC13:33
*** ig0r__ has joined #openstack-nova13:33
*** ctrath has joined #openstack-nova13:33
*** pedroalvarez has joined #openstack-nova13:33
lxslilascii: 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 worked13:35
lxslioh, belay that13:36
*** rushiagr is now known as rushiagr_away13:36
*** erlon has joined #openstack-nova13:36
*** lascii is now known as alaski13:39
*** mwagner_lap has joined #openstack-nova13:39
*** VW_ has joined #openstack-nova13:40
alaskilxsli: it appears to have a merge conflict13:40
raildomelwitt: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|mtg13:41
*** tpeoples has joined #openstack-nova13:42
jaypipesif 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-nova13:42
jaypipesalaski, danpb, johnthetubaguy: ^^13:42
*** lucas___ has joined #openstack-nova13:42
tdurakovjaypipes, thx, it would be great if these become merged:)13:43
*** marzif_ has quit IRC13:43
jaypipestdurakov: thanks for hanging in there and being patient13:44
*** ig0r__ has quit IRC13:44
*** marzif_ has joined #openstack-nova13:44
*** wverdugo500 has joined #openstack-nova13:44
*** ankit_ag has quit IRC13:45
garykalaski: 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 object13:46
superdangaryk: so just drop the first patch and focus on the second, right?13:47
superdanno reason to change it and then re-change it, right?13:47
*** superdan is now known as dansmith13:47
*** h00327910__ has quit IRC13:48
*** lyanchih_ has joined #openstack-nova13:48
*** sayali has joined #openstack-nova13:48
openstackgerritgaryk proposed openstack/nova: VMware: add in vif resource limitations  https://review.openstack.org/18020213:48
openstackgerritgaryk proposed openstack/nova: VMware: support limits via image meta data  https://review.openstack.org/18094213:48
openstackgerritgaryk proposed openstack/nova: Objects: add limits, reservations and shares to ImageMetaProps object  https://review.openstack.org/18832113:48
alaskiagreed.  just do it all at once13:49
garykdansmith: alaski: ok, will do.13:49
*** Nikolay_St has quit IRC13:49
*** browne has joined #openstack-nova13:50
*** vivekd has quit IRC13:50
*** Jeffrey4l_ has joined #openstack-nova13:51
*** bapalm_ has joined #openstack-nova13:51
*** READ10 has joined #openstack-nova13:51
openstackgerritAlexis Lee proposed openstack/nova: Raise NovaException for missing/empty machine-id  https://review.openstack.org/20264013:52
*** jeffrey__ has quit IRC13:53
openstackgerritgaryk proposed openstack/nova: Compute: use instance object for refresh_instance_security_rules  https://review.openstack.org/18753713:53
garykdansmith: alaski: ^13:53
dansmithgaryk: see my review on that?13:54
*** bapalm has quit IRC13:54
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation  https://review.openstack.org/13825513:54
*** sayali_ has joined #openstack-nova13:55
garykdansmith: i just saw it now when i was abandoning. i'll update. thanks13:55
*** sayali_ has quit IRC13:55
*** Jeffrey4l_ has quit IRC13:56
*** rotbeard has quit IRC13:56
*** tpeoples has quit IRC13:58
*** annegentle has joined #openstack-nova13:58
*** sayali has quit IRC13:58
*** andrewbogott has joined #openstack-nova13:59
*** sayalilunkad has joined #openstack-nova13:59
*** deepakcs has quit IRC13:59
sayalilunkadwhois sayalilunkad14:00
*** haomaiw__ has quit IRC14:01
openstackgerritDeepak C Shetty (deepakcs) proposed openstack/nova: libvirt: Nova part of volume snapshot improvements  https://review.openstack.org/17281314:01
openstackgerritDeepak C Shetty (deepakcs) proposed openstack/nova: objects: Add support for OnlineSnapshotInfo object  https://review.openstack.org/19453814:01
*** haomaiwang has joined #openstack-nova14:02
openstackgerritgaryk proposed openstack/nova: Compute: use instance object for refresh_instance_security_rules  https://review.openstack.org/18753714:02
*** josecastroleon has quit IRC14:02
*** mriedem has joined #openstack-nova14:03
*** SourabhP has joined #openstack-nova14:05
*** sayalilunkad has quit IRC14:06
*** sayali has joined #openstack-nova14:06
*** browne has quit IRC14:08
*** Murali has quit IRC14:08
*** burt has joined #openstack-nova14:08
*** sayali has quit IRC14:09
*** sayali has joined #openstack-nova14:09
*** garyk has quit IRC14:09
*** sayali has quit IRC14:10
*** sayalilunkad has joined #openstack-nova14:10
openstackgerritSergey Nikitin proposed openstack/nova: Added server tags support in nova-api  https://review.openstack.org/12894014:10
openstackgerritSergey Nikitin proposed openstack/nova: Fixed incorrect behaviour of method _check_instance_exists  https://review.openstack.org/20303714:10
*** afazekas has quit IRC14: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 additions14:12
*** xyang2 has joined #openstack-nova14:13
lyanchih_I was using utils synchronized to avoid race condtion, if it is not adaptive please tell me14:13
*** SourabhP_ has joined #openstack-nova14:14
*** rdopiera_ has quit IRC14:14
*** ig0r__ has joined #openstack-nova14:14
*** SourabhP has quit IRC14:16
*** SourabhP_ is now known as SourabhP14:16
*** xyang2 has quit IRC14:19
*** MarcF has left #openstack-nova14:19
*** xyang2 has joined #openstack-nova14:20
*** jistr|mtg is now known as jistr14:20
dansmithlyanchih_: so, this adds a lock around quite a bit of stuff that gets called from the API14:20
dansmithlyanchih_: which I think we try to avoid because it blocks the API server14:20
*** kjelly has quit IRC14:21
dansmithI don't actually see that we do that anywhere else14:21
dansmithalaski: 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 IRC14:23
raildojaypipes: jogo, melwitt can you put this patch in your review list? :) https://review.openstack.org/#/c/151327/14:24
jaypipesraildo: FYI, jogo no longer works on OpenStack... :) he's at Pinterest now doing Pinteresting things.14:24
alaskidansmith: yeah.  locking code that's called from the API is going to be troublesome for us14:24
lyanchih_synchronized will block all API!?14:24
dansmithalaski: okay, that's what I figured14:25
alaskiwell, 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 all14:25
raildojaypipes: ops, I didn't know that... thanks for the informaatino14:25
dansmithalaski: yeah, it assumes shared storage for the locks14:25
raildoinformation*14:25
dansmithwhich is never going to be the case for api14:25
openstackgerritJeremy Stanley proposed openstack/nova: IGNORE: Add bindep dependencies list  https://review.openstack.org/16444414:25
*** rdopiera has joined #openstack-nova14:25
*** unicell has joined #openstack-nova14:26
johnthetubaguysahid: do you know where we are at with this patch? https://review.openstack.org/#/c/19662014:26
alaskilyanchih_: it would block another call that needs the lock, but there are timeouts on the API so locking could cause those to get hit14:26
lyanchih_Should we call the function by compute instead api?14:26
alaskilyanchih_: compute is the only place I think it's reasonable to lock right now14:27
*** unicell1 has quit IRC14:27
dansmithyes, but14:27
lyanchih_If I don’t remeber wrong, nova-network  security group was handle by compute14:28
dansmithfunneling all neutron security group operations down to a compute is not that great either14:28
dansmiththe 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 all14:28
alaskiyeah.  not if it's only to get a lock14:28
*** lucas___ has quit IRC14:29
dansmithwhich probably requires a change to neutron's api, but that's really a much better solution in the long run14:29
*** annashen has joined #openstack-nova14:30
alaskiagreed14:30
sahidjohnthetubaguy: the whole serie is waiting to be approved14:30
johnthetubaguysahid: that patch needs updating those, I think its a quick change14:31
sahidjohnthetubaguy: the comment from Stephen as already been addressed at the beginning14:31
johnthetubaguysahid: alex_xu was pointing out this stuff, I think: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L630014:31
*** xyang2 has quit IRC14:31
mriedemdansmith: 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
dansmithyeah14:31
*** xyang2 has joined #openstack-nova14:32
*** afazekas has joined #openstack-nova14:32
openstackgerritSimona Iuliana Toader proposed openstack/nova: Hyper-V: Adds host maintenance implementation  https://review.openstack.org/20415714:32
*** rushiagr_away is now known as rushiagr14:32
rlrossitdansmith: 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
johnthetubaguysahid: and here: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L223214:33
*** lucas___ has joined #openstack-nova14:33
*** lucas___ has quit IRC14:33
*** lucas___ has joined #openstack-nova14:33
*** annashen has quit IRC14:35
dansmithrlrossit: yep14:35
johnthetubaguysahid: does that make sense?14:35
sahidjohnthetubaguy: ok let me addresse a patch to remove NovaException and be more specific on my change14:35
*** jecarey has joined #openstack-nova14:37
sahidjohnthetubaguy: actually QemuGuestAgentNotEnabled inherits from NovaExcetpion so that should be OK but we can be more specific yes14:37
johnthetubaguysahid: i think its actually quite a good to have the separate exception, just that breaks the error handling in its current form14:37
openstackgerritPengtao Huang proposed openstack/nova: change the value "all_mappings" to "block_device_mapping"  https://review.openstack.org/20851114:38
*** dannywilson has joined #openstack-nova14: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
sahidwell 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 +214:39
dansmithlyanchih_: I don't know their process, but probably something like that yeah14:40
sahidhum yes you already -1ed, ok will do that change14:40
*** sayalilunkad has quit IRC14:40
*** sayali has joined #openstack-nova14:41
johnthetubaguysahid: 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
dansmithlyanchih: cool, thanks14:42
johnthetubaguysahid: or maybe I miss-read the logic?14:42
sahidjohnthetubaguy: ah i did not noticed that, looking14:42
*** wanghua has quit IRC14:43
johnthetubaguysahid: its this bit https://github.com/openstack/nova/blob/master/nova/compute/api.py#L223214:43
nagyzmriedem, 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_04314:43
nagyzmriedem, 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
johnthetubaguysahid: it would get an unexpected RPC error, I think, in its current form, and would just raise that14:44
*** Kevin_Zheng_ has quit IRC14:45
*** ig0r_ has quit IRC14:45
*** davideagnello has joined #openstack-nova14:45
*** davideagnello has quit IRC14:49
*** SourabhP has quit IRC14:50
sahidjohnthetubaguy: 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 requested14:51
sahidif it's the case we re raise if not we skip14:51
sahids/cache/catch14:51
mriedemnagyz: looks pretty good14:51
lyanchih_Hi, where can I find metting note in Philly midcycle14:51
mriedemnagyz: so yeah, i'd say push forward14:52
lyanchih_There is one comment in https://bugs.launchpad.net/nova/+bug/1420274, which said the bug will discuss in midcycle.14:52
openstackLaunchpad 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-nova14:53
*** schwicke has joined #openstack-nova14:53
*** mtanino has joined #openstack-nova14:54
nagyzmriedem, ok I'll take tomorrow to write the tests, thanks14:55
*** shyama has quit IRC14:56
johnthetubaguysahid: yeah, and needs to be noted in compute manager, so its makes it across the RPC wire right?14:57
*** barra204 has joined #openstack-nova14:57
sahidjohnthetubaguy: yep14:58
*** haomaiwang has quit IRC15:01
*** tpeoples has joined #openstack-nova15:01
*** SourabhP has joined #openstack-nova15:01
*** tonytan4ever has joined #openstack-nova15:02
*** haomaiwang has joined #openstack-nova15:02
*** jasondotstar has quit IRC15:02
*** achanda has quit IRC15:04
*** agireud has joined #openstack-nova15:04
openstackgerritMatthew Booth proposed openstack/nova: Don't check flavor disk attributes when booting from volume  https://review.openstack.org/20463515:06
openstackgerritMatthew Booth proposed openstack/nova: Use original image size in volume image metadata  https://review.openstack.org/20852315:06
*** mdrabe has quit IRC15:07
*** mdrabe has joined #openstack-nova15:07
*** ddaskal has joined #openstack-nova15:07
*** ig0r__ has quit IRC15:09
*** barra204 has quit IRC15:09
*** afazekas has quit IRC15:10
mdboothSigh, just realised I forgot to run pep8 :(15:10
openstackgerritAdelina Tuvenie proposed openstack/nova: Moves OVS related code to a common location  https://review.openstack.org/17972715:12
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver  https://review.openstack.org/14004515:12
*** pcaruana has quit IRC15:13
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation  https://review.openstack.org/13825515:14
*** thumpba has joined #openstack-nova15:14
*** Sukhdev has joined #openstack-nova15:14
*** shakamunyi has joined #openstack-nova15:14
*** jasondotstar has joined #openstack-nova15:15
*** shakamunyi is now known as barra20415:16
openstackgerritDan Smith proposed openstack/nova: Remove legacy flavor compatibility code from Instance  https://review.openstack.org/19777115:25
dansmithwould really like to get that merged ^15:25
*** barra204 has quit IRC15:26
*** pixelbeat_ has quit IRC15:27
alaskiooh, I would too.  I'll take a look in a bit15:27
*** prometheanfire has joined #openstack-nova15:27
dansmithwoohoo!15:27
*** thedodd has joined #openstack-nova15:28
prometheanfiremikal: 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-nova15:28
prometheanfirequestion for others as well I guess :D15:28
*** meghal has joined #openstack-nova15:28
*** sajeesh has joined #openstack-nova15:29
*** tjones has joined #openstack-nova15:30
dansmithprometheanfire: yeah, that should be doable15:30
tdurakovalaski, ping15:31
prometheanfirecool, friend has an arm64 box as a build box and wanted nova on it :D15:31
tdurakovseen 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 IRC15:33
*** otter768 has joined #openstack-nova15:34
alaskitdurakov: it gets to a place where the same request can have a different response based on which virt driver is in use15:34
alaskiwhich is info that Nova shouldn't (need to) expose15:34
*** armax has joined #openstack-nova15:35
tdurakovalaski, you are talking of a case when nova have different compute nodes type, right?15:36
*** ishant has quit IRC15:36
*** xyang2 has quit IRC15:36
alaskitdurakov: that, or two different clouds which use different hypervisors15:36
*** moshele_ has quit IRC15:37
dansmithtdurakov: or different versions of the same hypervisor15:37
*** shyama has joined #openstack-nova15:37
*** schwicke has quit IRC15:38
tdurakovalaski, dansmith, ok, thanks. another question15:38
*** otter768 has quit IRC15:38
prometheanfiredansmith: the sceduler can handle the diferent image types and cpu arches?15:39
dansmithprometheanfire: the scheduler can honor host aggregates, which you can use to segment things15:39
mdboothWhoa! https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertTrue15:39
dansmiththere might be an arch filter too, I'm not positive15:39
tdurakovif you attempt to resize instance in such cloud now you can get same situation, but there would be no info for user15:39
mdboothYou modify a method to return an object instead of a boolean, and its unit test continues to pass by coincidence!15:39
tdurakovwhat is a profit in such hiding?15:39
*** sayali has quit IRC15:40
tdurakovalaski, dansmith^15:40
prometheanfirehttp://docs.openstack.org/developer/nova/devref/filter_scheduler.html maybe15:40
prometheanfirewill have to look15:40
*** barra204 has joined #openstack-nova15:40
johnthetubaguydansmith: 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,cm15:40
dansmithjohnthetubaguy: for which remaining flavor load logic?15:41
mdboothSo interestingly, I recall being encouraged to replace assertEqual(True, foo) with assertTrue(foo)15:41
mdboothHowever, those 2 aren't at all equivalent15:41
dansmithmdbooth: no the other way around15:41
*** mestery is now known as mestery_afk_toda15:42
alaskitdurakov: 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 discussion15:42
mdboothAnd the docs suggest assertIs(foo, True) in that case15:42
dansmithmdbooth: our hacking rules try to discourage you from using assertTrue15:42
*** mestery_afk_toda is now known as mestery_afk15:42
rlrossitdansmith: 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
mdboothdansmith: orly? All is well in the world :)15:42
dansmithjohnthetubaguy: L510 maybe?15:42
johnthetubaguydansmith: line 412 in that review list15:42
openstackgerritlyanchih proposed openstack/nova: Can't remove error instance which boot from image with new volume  https://review.openstack.org/20853615:42
dansmithjohnthetubaguy: er, right15:43
dansmithjohnthetubaguy: 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 that15:43
dansmithrlrossit: they're not remotable15:43
johnthetubaguydansmith: cool, thanks15:43
dansmithrlrossit: so they don't have any bearing on RPC, which means no version bump15:43
tdurakovalaski, when i disscussed that with johnthetubaguy, bp wasn't required. maybe ml?15:44
rlrossitdansmith: ah, never noticed the remotable decorators before. TIL15:44
dansmithrlrossit: :)15:45
dansmithjohnthetubaguy: actually there is a test, but it says:15:45
dansmithif 'flavor' in attr: # FIXME(danms): This isn't implemented yet15:45
*** belmoreira has quit IRC15:45
johnthetubaguydansmith: yeah, just spotted that, line 17015:45
dansmithjohnthetubaguy: :)15:46
dansmithjohnthetubaguy: so I should see if I can remove that exclusion now15:46
johnthetubaguydansmith: yeah, I guess you want old_ and new_ in there too15:46
*** mriedem has quit IRC15:46
dansmithjohnthetubaguy: the "if flavor in" was catching all three on purpose15:46
alaskitdurakov: 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 proceeding15:46
johnthetubaguydansmith: ah, didn't read it that way, makes sense15:47
dansmithjohnthetubaguy: yeah, that test passes without the exclusion now, so lemme push that up15:47
johnthetubaguydansmith: sweet, thanks15:47
openstackgerritDan Smith proposed openstack/nova: Remove legacy flavor compatibility code from Instance  https://review.openstack.org/19777115:47
prometheanfirewe getting a backport for https://launchpad.net/bugs/1387543 ?15:49
openstackLaunchpad bug 1387543 in OpenStack Security Advisory "Resize/delete combo allows to overload nova-compute (CVE-2015-3241)" [Medium,In progress]15:49
openstackgerritAdelina Tuvenie proposed openstack/nova: Fixes Bug "destroy_vm fails with HyperVException"  https://review.openstack.org/20400715:50
*** dannywilson has joined #openstack-nova15:50
*** dannywilson has quit IRC15:53
*** openstackgerrit_ has joined #openstack-nova15:54
*** tjones has quit IRC15:54
*** jasondotstar has quit IRC15:54
*** scheuran has quit IRC15:57
*** wendar has quit IRC15:57
*** wendar has joined #openstack-nova15:57
openstackgerritAdelina Tuvenie proposed openstack/nova: Fixes Bug "destroy_vm fails with HyperVException"  https://review.openstack.org/20400715:58
*** lucas___ has quit IRC15:58
*** tpeoples has quit IRC15:59
*** haomaiwang has quit IRC16:01
*** haomaiwang has joined #openstack-nova16:02
openstackgerritMatthew Booth proposed openstack/nova: Don't check flavor disk attributes when booting from volume  https://review.openstack.org/20463516:02
openstackgerritMatthew Booth proposed openstack/nova: Use original image size in volume image metadata  https://review.openstack.org/20852316:02
*** tpeoples has joined #openstack-nova16:03
*** prometheanfire has left #openstack-nova16:03
*** jlanoux has quit IRC16:04
openstackgerritSudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup  https://review.openstack.org/19276016:05
*** tsekiyama has joined #openstack-nova16:06
*** rdo has quit IRC16:07
*** sudipto has joined #openstack-nova16:07
*** rdo has joined #openstack-nova16:08
*** ig0r_ has joined #openstack-nova16:08
*** dguitarbite has quit IRC16:08
*** lsmola has quit IRC16:11
*** leeantho has joined #openstack-nova16:13
*** unicell has quit IRC16:13
*** tjones has joined #openstack-nova16:14
*** echoingumesh has joined #openstack-nova16:15
*** Marga_ has quit IRC16:16
*** mriedem has joined #openstack-nova16:16
*** fawadkhaliq has joined #openstack-nova16:16
*** tjones has quit IRC16:17
echoingumeshnova-api stops when i add log_config=/etc/nova/logging.conf. my logging.conf is default. Any clues ?16:17
*** tjones has joined #openstack-nova16:17
*** sahid has quit IRC16:18
*** alex_klimov has quit IRC16:20
openstackgerritRyan Rossiter proposed openstack/nova: Fix test_relationships() for subobject versions  https://review.openstack.org/19890416:20
*** browne has joined #openstack-nova16:21
jogojaypipes: I am making sure mordred can keep pinning his recipes16:21
dansmithjogo: ah, pinteresting16:23
jaypipesjogo: :)16:25
*** yamahata has quit IRC16:25
*** marzif_ has quit IRC16:25
jogodansmith: they love putting the word pin in everything here16:26
dansmithI bet16:26
openstackgerritSudipta Biswas proposed openstack/nova: cleanup NovaObjectDictCompat from security_group_rule  https://review.openstack.org/20128216:26
jogopinfo16:26
jaypipesthat's pincredible.16:26
*** Marga_ has joined #openstack-nova16:27
jrolljogo: I used to work at zerocater, I now have thousands of food puns in my back pocket :|16:27
*** jistr has quit IRC16:28
*** dustins has quit IRC16:28
*** shyama has quit IRC16:29
jaypipesjroll: were you ever called spring rollenhagen?16:29
jrollsomehow, I don't think so16:29
jaypipesjroll: I believe I have just identified your casual friday nick. jspringroll16:29
jrollmy nick on hipchat there was JimmyJohns16:29
*** jdandrea has joined #openstack-nova16:29
jaypipesor jeggroll if you prefer ;)16:29
jrollnah, if I participate in casual nick friday the clear choice is jtroll16:30
jaypipeshaha16:30
jaypipesnice.16:30
*** jcoufal_ has quit IRC16:30
*** annashen has joined #openstack-nova16:31
*** diogogmt has joined #openstack-nova16:32
*** otter768 has joined #openstack-nova16:32
*** moshele_ has joined #openstack-nova16:33
openstackgerritMatthew Booth proposed openstack/nova: compute: improve exceptions related to disk size checks  https://review.openstack.org/20656516:34
*** exploreshaifali has joined #openstack-nova16:35
*** annashen has quit IRC16:35
*** unicell has joined #openstack-nova16:37
*** lucas__ has joined #openstack-nova16:37
*** dustins has joined #openstack-nova16:38
*** lpetrut has joined #openstack-nova16:39
*** lucas__ has quit IRC16:42
jogojroll: haha16:42
mgagnemelwitt: 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
openstackgerritDan Smith proposed openstack/nova: Remove legacy flavor compatibility code from Instance  https://review.openstack.org/19777116:43
openstackgerritDan Smith proposed openstack/nova: Fix test mock that abuses objects  https://review.openstack.org/20855916:43
*** moshele_ has quit IRC16:43
jrollmgagne: I feel like that's a known issue16:43
dansmithalaski: addressed your concern, and had to fix another test that crept in :/16:43
mgagnejroll: can you share the known solution? :D16:43
jrolllooking :P16:44
mgagnefound https://review.openstack.org/#/c/197514/16:44
mgagnedon't know why LP didn't find anything :-/16:44
jrollmgagne: here's the bug: https://bugs.launchpad.net/nova/+bug/144381616:45
openstackLaunchpad 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 IRC16:45
mgagnejroll: tyvm for your help, will try now16:45
jrollmgagne: 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/058dcf2674b8b3fd62e716:46
*** kiran-r has joined #openstack-nova16:46
mgagnejroll: what about configdrive?16:46
jrollmgagne: it still works for configdrive16:46
*** asselin_ has joined #openstack-nova16:46
jrollI haven't read enough of the code to know why, but it does :)16:46
*** tonytan4ever has quit IRC16:46
mgagnewill try now16:47
*** shyama has joined #openstack-nova16:48
openstackgerritAnthony PERARD proposed openstack/nova: libvirt-xen: Add xen_vif_bridge_script_path config option.  https://review.openstack.org/19909316:48
*** ig0r_ has quit IRC16:49
*** jasondot_ has joined #openstack-nova16:51
*** jasondot_ has quit IRC16:52
*** jasondot_ has joined #openstack-nova16:52
*** exploreshaifali has quit IRC16:53
johnthetubaguymgagne: jroll: that breaks because it tries to read the ssh key from the "wrong" database16:54
*** jasondot_ is now known as jasondotstar_16:54
*** jasondotstar_ is now known as jasondotstar__16:54
mgagnejohnthetubaguy: yea, we figured that part16:54
*** jasondotstar__ is now known as jasondotstar_16:54
*** ig0r_ has joined #openstack-nova16:54
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] issue with latest libraries  https://review.openstack.org/20856116:54
*** lucas__ has joined #openstack-nova16:54
mgagnejroll: 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 IRC16:55
*** lucas__ has quit IRC16:55
johnthetubaguymgagne: jroll: ah, so sorry, I miss read the conversation16:55
jrolljohnthetubaguy: 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 unknown16:55
*** lucas__ has joined #openstack-nova16:55
jrollmgagne: yeah, it's odd16:55
*** ig0r_ has quit IRC16:55
*** Marga_ has quit IRC16:55
johnthetubaguyjroll: oh, so it should fail with both, good point, unless we are missing data in one of them16:55
mgagnejroll: configdrive still loads InstanceMetadata. I guess it's cloud-init that reads public_keys and not keys which is now missing.16:55
jrollmaybe configdrive doesn't require key type16:56
jrollmgagne: ah, I see, interesting16:56
mgagneconfigdrive doesn't care, only cloud-init or whatever agent is in there16:56
johnthetubaguyjroll: yeah, thats a bug if thats true, but I guess thats it16:56
*** Marga_ has joined #openstack-nova16:56
johnthetubaguythey should have identical data, as I understand it16:56
*** tjones has quit IRC16:56
* jroll nod16:57
mgagnemostly, one has the key type too in its struct16:57
johnthetubaguythe key_type is missing in the instance object, thats the key issue I guess16:57
*** rdo has quit IRC16:57
mgagneno pun intended :P16:57
*** rdopiera has quit IRC16:57
johnthetubaguyits possible config drive just always has None for the type or something horrible like that16:57
johnthetubaguythats worth a check16:57
*** coolsvap is now known as coolsvap|away16:58
mgagneI'm not sure why it worked with icehouse, something changed in the metadata struct16:58
mgagnein juno: https://github.com/openstack/nova/blob/stable/juno/nova/api/metadata/base.py#L317-L32016:59
mgagnein kilo: https://github.com/openstack/nova/blob/stable/kilo/nova/api/metadata/base.py#L309-L32116:59
mgagneso yea, someone added a new key to the metadata16:59
*** rdo has joined #openstack-nova16:59
mriedemscottda: i was out friday afternoon, you linked me something about a cinder related bug, data loss something something?17:00
mgagneand type can be ssh or x50917:00
*** READ10 has quit IRC17:01
*** haomaiwang has quit IRC17:01
scottdamriedem: just a minute...let me track that down...17:01
*** haomaiwang has joined #openstack-nova17:02
*** READ10 has joined #openstack-nova17:02
*** harlowja has joined #openstack-nova17:03
*** tongli has joined #openstack-nova17:03
*** ig0r_ has joined #openstack-nova17:04
*** dguitarbite has joined #openstack-nova17:04
*** tonytan4ever has joined #openstack-nova17:05
mgagnejroll: I'm sure there is a way to "recompute" the type from the key data17:05
scottdamriedem: 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
mgagnejroll: like key_data.startswith('ssh')17:05
scottdahttps://bugs.launchpad.net/nova/+bug/1458958 is the bug.17:05
openstackLaunchpad 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-nova17:06
scottdamriedem: 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
mriedemscottda: first, that says it shouldn't be a duplicate, so can we just fix lp firsT?17:06
*** tjones has joined #openstack-nova17:06
*** tjones has quit IRC17:06
scottdaIt's not really a duplicate, because the bug it points to is more specific, for only the detaching case.17:07
scottdaso, do you mean fix the duplicate part? sure, I'll see if I can do that...17:07
scottdaok...I think I've un-duped it.17:08
mriedemso i guess i'll look at https://review.openstack.org/#/c/186742/ first17:09
johnthetubaguymgagne: 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 think17:09
scottdaok. 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
mgagnejohnthetubaguy: sure. now I would like a patch for my setup in the next 10 minutes so thinking out loud about a solution =)17:10
mriedemew jesus, i have not17:10
*** davideagnello has joined #openstack-nova17:11
dansmithrlrossit: does this need freshening after the last week of fixing the nova version? https://review.openstack.org/#/c/203831/217:11
scottdaIt'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
johnthetubaguymgagne: 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 halm17:11
*** moshele_ has joined #openstack-nova17:11
* johnthetubaguy feels kinda dirty after saying that17:12
*** kmARC has quit IRC17:13
mgagnehttps://gist.github.com/jimrollenhagen/058dcf2674b8b3fd62e7#gistcomment-154130717:13
*** lyanchih_ has quit IRC17:13
mriedemscottda: well, isn't that kind of a backwards incomptable change with the cinder API?17:13
*** tjones has joined #openstack-nova17:14
mriedemhave we ever supported newer cinder than nova in deployments?17:14
*** Sukhdev has quit IRC17:14
*** dannywilson has joined #openstack-nova17:14
mriedemlike liberty cinder starts sending errors back to kilo nova via the cinder api17:14
*** diogogmt has quit IRC17:15
*** Sukhdev has joined #openstack-nova17:15
scottdanot that I know of, and I don't think we'd start now...17:16
jrollmgagne: seems like it would work17:16
*** unicell1 has joined #openstack-nova17:16
*** unicell has quit IRC17:16
scottdaso 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 IRC17:17
*** tonytan4ever has joined #openstack-nova17:17
scottdaand then there's the backward compatible (cinder L, Nova k) stuff.17:17
mgagnejroll: hmmm unless there is a way to make SSH keypair not start with the ssh string17:17
mgagnejroll: http://man.he.net/man5/authorized_keys17:18
mgagnejroll: nova doesn't do much validation on the content...17:18
jrollnod17:19
openstackgerritSudipta Biswas proposed openstack/nova: cleanup NovaObjectDictCompat from virtual_interface  https://review.openstack.org/20082317:19
scottdamriedem: I've no answers here, just an ever expanding list of questions.17:19
*** echoingumesh has quit IRC17:20
*** dannywilson has quit IRC17:20
*** echoingumesh has joined #openstack-nova17:20
*** echoingumesh has quit IRC17:21
mriedemscottda: did a nova person ever sign up to come to the cinder meetup?17:21
scottdano, I think we got the message out too late...17:21
scottdabut we'll be using a google hangout.17:22
*** echoingumesh has joined #openstack-nova17:22
scottdaAnd I'll be pinging this room when we discuss Nova related issues, i.e. the Nova <-> Cinder API17:22
*** achanda has joined #openstack-nova17:25
*** ig0r_ has quit IRC17:25
*** salv-orlando has joined #openstack-nova17:25
*** achanda_ has joined #openstack-nova17:26
*** achanda has quit IRC17:26
*** atuvenie has quit IRC17:26
*** SourabhP has quit IRC17:27
johnthetubaguyscottda: 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 bits17:28
scottdajohnthetubaguy: What time zone are you in?17:28
johnthetubaguyscottda: UTC+117:28
scottdaok, 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
johnthetubaguyscottda: 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 best17:29
johnthetubaguyscottda: yeah, sounds good17:29
*** browne has joined #openstack-nova17:30
scottdaWe'll try to schedule for fir/st thing Wed morning, then. No later than 10:00 AM MDT which is 16:00 UTC17:30
mriedemscottda: ok, anyway, https://review.openstack.org/#/c/186742/ needs some work17:30
mriedemi'd also like ndipanov to review that ^17:30
*** yassine has quit IRC17:31
*** vilobhmm has joined #openstack-nova17:32
mriedemi'd also think that this impacts tempest17:33
*** echoingumesh has quit IRC17: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-nova17:33
*** jasondotstar_ has quit IRC17:34
*** diogogmt has joined #openstack-nova17:34
mriedemscottda: 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/117:34
openstackLaunchpad 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 IRC17:35
scottdayes, cinderclient can/will be changed to translate the HTTP 409...17:35
*** danpb has quit IRC17:37
*** Piet_ has joined #openstack-nova17:37
*** praneshp has joined #openstack-nova17:38
vilobhmmmriedem : I guess scottda is working on it…17:38
*** Nikolay_St has joined #openstack-nova17:39
*** tonytan4ever has quit IRC17:39
scottdavilobhmm: working on which part?17:40
*** lpetrut has quit IRC17:41
vilobhmmon nova-cinder interaction17:41
scottdacinderclient? Yes, I'll probably do that. Nova try blocks? maybe, or will work with winston-d_zZZ who had shown an interest..17:41
vilobhmmsure17:41
vilobhmmwinston-d intiially decided to pick it up in cinder weekly meeting17:41
scottdamriedem: that work is not started, but I'll take that on next week.17:43
dansmithhmm17:43
dansmithwonder if we're seeing a new unit test fail on master17:43
dansmithhttp://logs.openstack.org/59/208559/1/check/gate-nova-python27/61df9b7/console.html#_2015-08-03_17_35_16_89317:44
dansmithI see it locally if I run on master17:44
*** ig0r_ has quit IRC17:44
*** tonytan4ever has joined #openstack-nova17:45
*** echoingumesh has joined #openstack-nova17:45
*** lpetrut has joined #openstack-nova17:47
*** liusheng has quit IRC17:48
*** liusheng has joined #openstack-nova17:48
*** lucas__ has quit IRC17:49
*** lucas__ has joined #openstack-nova17:49
sudiptodansmith, think i should remove the "Conflicts" from the commit message: https://review.openstack.org/#/c/192760/ ?17:52
*** dane-fichter has quit IRC17:52
dansmithsudipto: yeah, we leave those in for a backport, but not master patches17:52
dansmithif we left them in,17:52
dansmithrlrossit's patches would have like 200 lines in the commit message :)17:52
rlrossitwhat did I do?17:53
*** yamahata has joined #openstack-nova17:53
rlrossitoh yeah definitely17:53
sudiptodansmith, ok...will remove them...also would ask for a review when you have sometime :)17:53
dansmithsudipto: did you get an oslo.db change to make it so we can remove those from the model before the schema?17:53
sudiptodansmith, yup17:54
sudiptodansmith, that's why it took a while i believe for jenkins to pass.17:54
dansmithokay cool17:54
rlrossitdansmith: 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 creative17:55
openstackgerritSudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup  https://review.openstack.org/19276017:56
dansmithokay, I just wasn't sure if I should look at it or hold off17:56
mnestratovjohnthetubaguy: 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
mnestratovor https://etherpad.openstack.org/p/liberty-nova-non-priority-feature-freeze is enough?17:56
*** moshele_ has quit IRC17:57
*** kmARC has joined #openstack-nova17:58
openstackgerritgaryk proposed openstack/nova: Neutron: add in API method for updating VNIC index  https://review.openstack.org/14712617:58
*** garyk has joined #openstack-nova17:59
*** salv-orl_ has joined #openstack-nova17:59
openstackgerritAndrew Laski proposed openstack/nova: Cells: Handle instance_destroy_at_top failure  https://review.openstack.org/20758718:00
*** Marga_ has quit IRC18:00
*** haomaiwang has quit IRC18:01
*** salv-orlando has quit IRC18:01
*** Marga_ has joined #openstack-nova18:01
*** haomaiwang has joined #openstack-nova18:02
openstackgerritBaodong (Robert) Li proposed openstack/nova: Handle port delete initiated by neutron  https://review.openstack.org/17939018:03
*** markus_z has quit IRC18:06
*** openstackgerrit_ has quit IRC18:07
*** tonytan4ever has quit IRC18:08
*** lpetrut has quit IRC18:10
*** dannywilson has joined #openstack-nova18:11
*** dannywilson has quit IRC18:11
*** dannywilson has joined #openstack-nova18:11
*** Marga_ has quit IRC18:11
*** rushiagr is now known as rushiagr_away18:14
*** achanda has joined #openstack-nova18:14
*** tcammann has quit IRC18:15
*** achanda_ has quit IRC18:16
*** tcammann has joined #openstack-nova18:16
sudiptodansmith, 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-nova18:17
sudiptodansmith, sorry - i understood your statement properly now :)18:17
mriedemdansmith: https://github.com/openstack/nova/commit/ed0196ebb6e09b0baac783981da480849ee16f3618:18
dansmithmriedem: what about it?18:19
mriedemthat's probably what introduced that new test bug18:19
dansmithsudipto: okay :)18:19
dansmithmriedem: the ec2 thing?18:19
mriedemno nova.tests.unit.cmd.test_baseproxy.BaseProxyTestCase.test_proxy18:19
dansmithI'm confused18: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_89318:20
mriedem(12:43:58 PM) dansmith: wonder if we're seeing a new unit test fail on master18:20
dansmithoh, I see.. I'm actually seeing a different failure locally18:21
dansmithI thought was the same18:21
dansmithheh18:21
*** harlowja has quit IRC18:21
* dansmith recreates .tox18:21
*** Kennan has quit IRC18:21
*** harlowja has joined #openstack-nova18:21
*** chaochin has quit IRC18:21
*** park_heijlong has quit IRC18:22
mriedemunfortunately logstash is all backed up so this is too new to shows up18:22
mriedemanyway, https://review.openstack.org/#/c/204723/ includes a bump to websockify which is apparently parsing things randomly, something racey in there18:24
mtreinishmriedem: https://review.openstack.org/#/c/208561/ seems related to that18:25
mriedemanother jaypipes revert :)18:25
dansmithmaybe we just automatically propose reverts for jaypipes reviews so we have them ready to go if we need them?18:26
openstackgerritSudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup  https://review.openstack.org/19276018:27
mriedemdansmith: do you have a line to directxman12?18:27
mriedemhttps://bugs.launchpad.net/nova/+bug/148104318:27
openstackLaunchpad 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-nova18:28
dansmithmaybe18:28
*** chaochin has joined #openstack-nova18:28
*** Kennan has joined #openstack-nova18:29
*** rotbeard has joined #openstack-nova18:29
dansmithmriedem: ah, yep, with new .tox I get the same failure as gate and the other fail is gone18:29
dansmithawesome18:29
dansmith"awesome"18:29
*** directxman12 has joined #openstack-nova18:30
mriedemthat is awesome18:30
mriedemwith logstash backed up i can't tell how pervasive this is18:30
dansmithmriedem: ^18:30
mriedemi know18:30
dansmithmriedem: it's 100% for me18:30
mriedemi'm hanging on your every word :)18:30
rlrossitmriedem: it got me18:30
openstackgerritgaryk proposed openstack/nova: Metadata: support proxying loadbalancers  https://review.openstack.org/14753918:30
mriedemso um?18:30
mriedemrevert?18:30
dansmithmriedem: I meant ^ directxman1218:30
mriedemdirectxman12: 148104318:31
mriedemoops https://bugs.launchpad.net/nova/+bug/148104318:31
openstackLaunchpad bug 1481043 in OpenStack Compute (nova) "nova.tests.unit.cmd.test_baseproxy.BaseProxyTestCase.test_proxy randomly fails" [High,Confirmed]18:31
*** fawadkhaliq has quit IRC18:31
* directxman12 looks18:31
*** thangp has joined #openstack-nova18:31
*** annashen has joined #openstack-nova18:32
*** sajeesh has quit IRC18:32
*** htruta_ has quit IRC18:32
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] issue with latest libraries  https://review.openstack.org/20856118:33
*** annashen has quit IRC18:36
*** josecastroleon has joined #openstack-nova18:36
directxman12mriedem: 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 around18:37
directxman12dansmith: ^18:37
*** kmARC has quit IRC18:38
*** kmARC_ has joined #openstack-nova18:38
*** kmARC_ has quit IRC18:38
*** sudipto has quit IRC18:38
*** kmARC has joined #openstack-nova18:39
*** apuimedo has quit IRC18:39
directxman12(it just looks like a mismatch between the traffic parameter)18:41
*** doffm has joined #openstack-nova18:42
mriedemdirectxman12: traffic and verbose18:43
*** htruta_ has joined #openstack-nova18:44
openstackgerritGiridhar Jayavelu proposed openstack/nova: Create a new dictionary for type_data in VMwareAPIVMTestCase class  https://review.openstack.org/20276818:45
openstackgerritGiridhar Jayavelu proposed openstack/nova: VMware: create vif with resource limitations  https://review.openstack.org/19617118:45
*** kiran-r has quit IRC18:45
*** shakamunyi has joined #openstack-nova18:46
directxman12mriedem: 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 IRC18:48
*** patrickeast has joined #openstack-nova18:49
*** Sukhdev has quit IRC18:49
mriedemnot sure, would need to reproduce locally18:50
mriedemi'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
mriedemlike https://review.openstack.org/#/c/204723/19/nova/tests/unit/cmd/test_baseproxy.py <- why?18:50
mriedemdims: ^?18:51
lifelesswhy are you patching individual methods :)18:51
*** Marga_ has joined #openstack-nova18:52
lifelessmock.patch.object vs mock.patch are equivalent here18:52
jdandreaScheduler 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
jaypipesdansmith, mriedem: :(18:55
jdandreaIf 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-nova18:56
dansmithjaypipes: I was just kiddin'18:56
dansmithof course18:56
openstackgerritGiridhar Jayavelu proposed openstack/nova: Configure memory resource allocation during instance resize operation  https://review.openstack.org/20421918:58
openstackgerritGiridhar Jayavelu proposed openstack/nova: Configure disk resource allocation during instance resize operation  https://review.openstack.org/20861218:58
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Fix problem with BaseProxyTestCase.test_proxy  https://review.openstack.org/20856118:58
*** tonytan4ever has joined #openstack-nova18:59
mriedemjaypipes: sdague: added you guys to this cinder spec https://review.openstack.org/#/c/149894/18:59
mriedemscottda: ^ i also went through and commented on that spec18:59
jaypipesmriedem: thx19:00
*** haomaiwang has quit IRC19:01
*** otter768 has quit IRC19:01
*** yamahata has quit IRC19:01
*** yamahata has joined #openstack-nova19:01
mriedemproposes to return errors from cinder api which used to return success19:02
mriedemw/o an api version bump19:02
mriedemwe kind of sort of care about that19:02
*** haomaiwang has joined #openstack-nova19:02
*** josecastroleon has quit IRC19:06
mriedemdirectxman12: is "traffic=CONF.verbose and not CONF.daemon" in websockify?19:06
mriedemoops nvm19:06
mriedemi see it in nova now19:06
directxman12mrieden: it's in nova.cmd.baseproxy19:06
mriedemyup19:07
*** exploreshaifali has joined #openstack-nova19:07
mriedemyeah so i think https://review.openstack.org/#/c/204723/ changed something to do a global mock and that's what's causing the race19:07
mriedemso probably need to stub CONF.verbose like you said in the test setup19:07
mriedemor not mock global things19:08
*** flwang has quit IRC19:08
*** patrickeast_ has joined #openstack-nova19:10
*** patrickeast_ has quit IRC19:11
*** zul has quit IRC19:11
*** patrickeast has quit IRC19:11
*** Sukhdev has joined #openstack-nova19:12
*** READ10 has quit IRC19:13
*** tonytan4ever has quit IRC19:13
lifelessdirectxman12: mriedem was puzzled on 204723 (see up a page or so), and I answered haypo's question about using testr in it, if thats interesting19:15
lifelessdirectxman12: sorry, fail tab complete19:16
lifelessdims: ^19:16
mriedemi'm about to push a fix19:16
* dims peeks19:16
dimsi was trying a change out locally...looks like mriedem may have a better fix19:16
*** jasondot_ has joined #openstack-nova19:17
*** shyama has quit IRC19:17
dimslifeless: ack, will look at the testr comment19:17
openstackgerritMatt Riedemann proposed openstack/nova: Make BaseProxyTestCase.test_proxy deterministic wrt traffic/verbose  https://review.openstack.org/20862119:17
mriedemdirectxman12: dims: dansmith: jaypipes: rlrossit: ^19:17
*** angdraug has joined #openstack-nova19:17
lifelessdims: I just realised the list might be file names, in which case an aggregate regex is needed, not a load-list; but its doable19:17
dimslifeless: i was asking mtreinish about adding a whitelist to os-testr19:18
lifelessdims: that said, I think what is there is ok while we're working up 3.4 support19:18
lifelessdims: just 'testr run filename.one filename.two' will DTRT19:19
dansmithmriedem: hmm19:19
lifelessdims: because the implicit combinator of regex patterns is |19:19
dimslifeless: 204723 was primarily to stop folks from checking in incompatible code/syntax/import for all tests19:19
lifelessdims: yep19:19
directxman12mriedem: FWIW, LGTM19:19
dimsthanks mriedem19:19
mriedemdirectxman12: i hope so since i put your name on it19:19
directxman12heh19:19
*** Piet_ has quit IRC19:20
*** busterswt has joined #openstack-nova19:22
*** flwang has joined #openstack-nova19:22
*** jasondot_ has quit IRC19:26
*** gszasz has quit IRC19:27
*** angdraug has quit IRC19:27
*** pixelbeat_ has quit IRC19:27
*** pixelbeat_ has joined #openstack-nova19:28
*** agireud_ has joined #openstack-nova19:29
*** MentalRay has joined #openstack-nova19:29
*** agireud has quit IRC19:30
*** agireud_ is now known as agireud19:30
*** emagana has joined #openstack-nova19:33
*** angdraug has joined #openstack-nova19:37
openstackgerritMarian Horban proposed openstack/nova: libvirt:Rsync remote FS driver was added  https://review.openstack.org/14688719:37
openstackgerritRoman Dobosz proposed openstack/nova: New nova API call to mark nova-compute down  https://review.openstack.org/18408619:41
openstackgerritVikhyat Umrao proposed openstack/nova: RBD: Reading rbd_default_features from ceph.conf  https://review.openstack.org/18653719:42
*** ftersin has quit IRC19:44
*** patrickeast has joined #openstack-nova19:48
*** angdraug has quit IRC19:48
*** angdraug has joined #openstack-nova19:48
*** raildo has quit IRC19:49
*** echoingumesh has quit IRC19:50
*** echoingumesh has joined #openstack-nova19:51
*** geekinutah has joined #openstack-nova19:51
*** patrickeast has quit IRC19:52
*** agireud has quit IRC19:52
nagyzdoes 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-nova19:54
nagyzhere's the actual review: https://review.openstack.org/#/c/138619/19:55
dansmithmriedem: do we understand at all why that helps?19:55
dansmithbecause based on what was changed in the supposed evil patch, I don't get it19:55
nagyzI know it's basically dead but for Mitaka would there be interest?19:55
*** Piet has joined #openstack-nova19:58
mriedemdansmith: i don't even know why https://review.openstack.org/#/c/204723/19/nova/tests/unit/cmd/test_baseproxy.py needed to bechanged19:58
mriedemlike, can i not use @mock.patch.object now?19:58
mriedemdims: ^19:58
dansmithhmm19:58
*** haomaiwang has quit IRC20:01
*** patrickeast has joined #openstack-nova20:01
*** angdraug has quit IRC20:02
*** haomaiwang has joined #openstack-nova20:02
*** angdraug has joined #openstack-nova20:02
dimsmriedem: either one should work, am trying to remember why i needed that change20:03
*** dustins has quit IRC20:05
mriedemhttps://docs.python.org/3/library/unittest.mock.html#id420:09
*** baoli has quit IRC20:09
*** rushiagr_away has quit IRC20:13
*** tonytan4ever has joined #openstack-nova20:14
mriedemidk either, it fails if i change it back to this20:15
mriedem@mock.patch.object(websocketproxy.NovaWebSocketProxy, '__init__', return_value=None)20:15
mriedemwithout self.flags(verbose=False)20:15
dansmithmriedem: which fixes it? the mock change or the flags change?20:16
mriedemdansmith: the flags change20:16
dansmiththe mock okay, that's what I thought, and that's the confusing thing to me20:17
*** tonytan_brb has joined #openstack-nova20:20
*** tonytan4ever has quit IRC20:21
*** kurtmartin has joined #openstack-nova20:21
mriedemdansmith: alternatively, oslo.logging 1.8.0 https://pypi.python.org/pypi/oslo.log/1.8.020:24
mriedemreleased today20:24
mriedemdid you see this before today?20:24
dansmithno20:24
mriedemha20:25
mriedemhttps://github.com/openstack/oslo.log/commit/f464d9f2e1e31de4e916f1c8089853d93b1bba6120:25
mriedembingo20:25
mriedemi'll update the commit message on my fix20:25
*** lpetrut has joined #openstack-nova20:27
openstackgerritMatt Riedemann proposed openstack/nova: Make BaseProxyTestCase.test_proxy deterministic wrt traffic/verbose  https://review.openstack.org/20862120:27
mriedemdansmith: jaypipes: sdague: dims: ^20:27
mriedemupdated commit message20:27
jaypipesmriedem: done20:29
sdaguemriedem: ah, that commit message makes a bit more sense. Though it's still weird that it's actually impacting things poorly20:30
mriedemsdague: it's not very weird actually,20:30
*** tmckay has joined #openstack-nova20:30
mriedemit was False by default before, then changed to True by default20:30
mriedemthe test wasn't explicitly stubbing that config option out so it was completely based on what the default was20:30
sdagueoh, why don't you just make verbose=False => mock.ANY ?20:31
mriedembecause i can just stub it20:31
dansmithyeah, if this is the problem, why force it back?20:31
tmckayHi 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 components20:31
tmckayso who is wrong?  Sahara or nova?20:31
mriedemverbose='true'?20:31
sdaguebecause, we don't care in the least about that value20:31
* tmckay throws a grenage20:31
tmckaycan't spell. but you get the idea20:31
sdagueso, 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 approach20:32
*** annashen has joined #openstack-nova20:33
dimstmckay: have an example? (paste.openstack.org please)20:34
tmckayyep, hold on20:34
dimsmriedem: good detective work :)20:35
openstackgerritAndrew Laski proposed openstack/nova: Xen: convert image auto_disk_config value to bool before compare  https://review.openstack.org/20864220:35
*** ig0r_ has joined #openstack-nova20:36
*** annashen has quit IRC20:37
*** alexpilotti has joined #openstack-nova20:37
tmckaydims, http://paste.openstack.org/show/406793/20:38
alaskidansmith: 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 do20:39
tmckaydims, I can find the "split" line if I look again ....20:39
mriedemalaski: like with what changed?20:40
mriedemhttp://git.openstack.org/cgit/openstack/nova/tree/nova/objects/base.py#n46920:40
mriedemor so that obj_attr_is_set returns False?20:40
alaskithe latter20:40
alaskidelattr(object, get_attrname(attrname)) should do it, but is invasive20:41
dimstmckay: please open a bug, i think i know where it is20:41
*** dannywilson has quit IRC20:41
*** patrickeast has quit IRC20:41
dimstmckay: recently changed code, so possible regression20:42
openstackgerritEarle F. Philhower, III proposed openstack/nova: Add os_brick-based VolumeDriver for HGST connector  https://review.openstack.org/19441220:42
tmckay/usr/lib/python2.7/site-packages/cryptography/hazmat/primitives/serialization.py(36)load_ssh_public_key()20:42
mriedemalaski: 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 gross20:42
tmckaydims, ^^20:42
*** prometheanfire has joined #openstack-nova20:43
*** patrickeast has joined #openstack-nova20:43
tmckaydims, hmm, I wonder if spaces in the comment are legal at all20:44
prometheanfirewhat does nova use to attach to iscsi for cinder? need to know for packaging dep reasons20:44
anishopen-iscsi20:44
*** hemna has joined #openstack-nova20:45
*** patrickeast has quit IRC20:46
*** directxman12 has left #openstack-nova20:47
*** patrickeast has joined #openstack-nova20:47
*** ig0r__ has joined #openstack-nova20:50
*** ig0r_ has quit IRC20:50
*** patrickeast has quit IRC20:52
anishprometheanfire: was that what you wanted ? yes/no/retry ?20:53
dimstmckay: trying to track down RFC(s)...nova seems to be using Generated-by-Nova, so you could do Generated-by-Sahara :)20:53
tmckaydims, ah, okay20:54
*** annegentle has quit IRC20:54
tmckaydims, filing the bug, just in case20:54
dimstmckay: "The optional comment field continues to the end of the line, and is not used." from sshd man page20:54
tmckaydims, so who wrote the hazmat stuff?  Is that part of openstack, or another project that needs to be fixed?20:55
tmckaywell, I guess it's the cryptography package20:55
dimstmckay: there's code in crypto.py which splits before the other code is called20:56
tmckayah, I see20:56
tmckaygotcha20:56
*** rushiagr_away has joined #openstack-nova20:56
*** thumpba has quit IRC20:56
*** otter768 has joined #openstack-nova20:56
*** patrickeast has joined #openstack-nova20:57
prometheanfireanish: ya, now I need to figure out why I can't boot from volume :(20:58
tmckaydims, okay, https://bugs.launchpad.net/nova/+bug/148108420:59
openstackLaunchpad bug 1481084 in OpenStack Compute (nova) "Keypair creation fails when ssh public key comment contains spaces" [Undecided,New]20:59
prometheanfirealso, it seems that when I fail to boot from volume I can't just do a nova-delete20:59
tmckaylet me know if you need any elaboration20:59
prometheanfirethere is some manual cleanup?20:59
*** SourabhP has quit IRC20:59
prometheanfirenova is telling cinder to detach a volume that's already detached21:00
prometheanfireso cinder 500s21:00
*** SourabhP has joined #openstack-nova21:00
prometheanfirenot sure if this is a cinder or a nova bug21:00
prometheanfirethis is stable/kilo21:00
*** bapalm_ has quit IRC21:00
*** haomaiwang has quit IRC21:01
*** lucas__ has quit IRC21:01
*** thorst has quit IRC21:01
*** otter768 has quit IRC21:01
*** haomaiwang has joined #openstack-nova21:01
mriedemprometheanfire: possibly related https://review.openstack.org/#/c/186742/21:07
mriedemprometheanfire: and https://review.openstack.org/#/c/204231/21:08
mriedem^ sounds like what you're looking for specifically21:08
prometheanfireya, that last one is good21:08
openstackgerritAdelina Tuvenie proposed openstack/nova: Moves OVS related code to a common location  https://review.openstack.org/17972721:09
*** patrickeast has quit IRC21:09
prometheanfireguess I review :P21:11
prometheanfiremriedem: offhand, you know how I need to clean up?21:11
*** ndipanov has quit IRC21:12
*** Sukhdev has quit IRC21:12
mriedemeek, not really21:13
mriedemforce-delete doesn't fix a thing like this21:14
mriedemyou could delete the related block_device_mapping entries from the nova database....21:14
anishprometheanfire: there's one more patch out there fixing a similar issue, can't find it21:15
anishthe end solution on that one at least was this is already fixed in os-brick21:15
*** tmckay has quit IRC21:15
*** smatzek has quit IRC21:16
mriedemanish: nova doesn't use os-brick in kilo21:16
*** Sukhdev has joined #openstack-nova21:16
*** mjturek1 has quit IRC21:17
*** ctrath has quit IRC21:18
*** thangp has quit IRC21:19
mriedemscottda: there is a cinderclient improvement to be had here https://review.openstack.org/#/c/204231/21:20
*** meghal has quit IRC21:21
*** ig0r__ has quit IRC21:21
*** meghal has joined #openstack-nova21:21
scottdamriedem: thanks, I'll have a look21:22
mriedemadded cinderclient to the bug21:22
*** meghal has quit IRC21:22
*** meghal has joined #openstack-nova21:23
*** meghal has quit IRC21:23
*** meghal has joined #openstack-nova21:23
*** meghal has quit IRC21:23
*** meghal has joined #openstack-nova21:24
*** rlrossit has left #openstack-nova21:24
*** meghal has quit IRC21:24
*** meghal has joined #openstack-nova21:25
anishmriedem: I know. this is for a liberty I believe21:25
*** meghal has quit IRC21:25
anishbut the symptoms were the same as waht prometheanfire is seeing21:25
*** Marga_ has quit IRC21:25
*** meghal has joined #openstack-nova21:25
*** meghal has quit IRC21:26
*** meghal has joined #openstack-nova21:26
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Better handling of SSH key comments  https://review.openstack.org/20866121:28
openstackgerritVilobh Meshram proposed openstack/nova: Servicegroup foundational refactoring for Control Plane  https://review.openstack.org/20271421:29
prometheanfirewell, guess it's time to dive into the db21:29
*** hemna has quit IRC21:31
*** mriedem is now known as mriedem_away21:31
prometheanfireanish: mriedem_away: this might my a related but difernt bug21:36
*** vilobhmm has quit IRC21:36
*** vilobhmm has joined #openstack-nova21:36
*** hemna has joined #openstack-nova21:37
*** pixelbeat_ has quit IRC21:43
*** ijw has joined #openstack-nova21:44
*** mnestratov has quit IRC21:44
*** annegentle has joined #openstack-nova21:47
prometheanfiremriedem_away: deleting from the mapping table worked, thanks21:48
*** vladikr has joined #openstack-nova21:54
*** jecarey has quit IRC21:56
*** Jeffrey4l has joined #openstack-nova21:58
*** burt has quit IRC21:58
*** tongli has quit IRC21:59
*** kurtmartin is now known as kmartin21:59
*** mwagner_lap has quit IRC21:59
*** meghal has quit IRC21:59
*** haomaiwang has quit IRC22:01
*** haomaiwang has joined #openstack-nova22:02
*** agireud has quit IRC22:02
*** thedodd has quit IRC22:04
*** ijw has quit IRC22:06
*** ijw has joined #openstack-nova22:07
*** slong has quit IRC22:13
*** agireud has joined #openstack-nova22:14
*** agireud has quit IRC22:18
*** kylek3h has quit IRC22:18
*** thumpba has joined #openstack-nova22:19
*** lpetrut has quit IRC22:20
*** hemna has quit IRC22:21
*** atuvenie has joined #openstack-nova22:23
*** busterswt has quit IRC22:25
*** VW_ has quit IRC22:27
*** thumpba has quit IRC22:29
*** vilobhmm has quit IRC22:33
*** annashen has joined #openstack-nova22:33
*** geekinutah has quit IRC22:35
*** annegentle has quit IRC22:35
*** kmartin has quit IRC22:36
*** vilobhmm has joined #openstack-nova22:37
*** annashen has quit IRC22:38
*** mdrabe has quit IRC22:38
*** mdrabe has joined #openstack-nova22:38
*** thumpba has joined #openstack-nova22:41
*** thumpba has quit IRC22:41
*** vilobhmm has quit IRC22:41
*** kmARC has quit IRC22:42
*** annegentle has joined #openstack-nova22:43
*** shakamunyi has quit IRC22:51
*** dims_ has joined #openstack-nova22:51
*** moshele_ has joined #openstack-nova22:52
*** dims has quit IRC22:52
prometheanfireso...22:52
*** dims has joined #openstack-nova22:53
prometheanfirefor nova boot from a cinder volume, cinder is complaining that it's not getting the initiator22:53
*** mwagner_lap has joined #openstack-nova22:53
prometheanfirewhich is sent from nova I think22:53
prometheanfiredoes nova need a special setup to work with cinder?22:55
anishno22:56
*** dims_ has quit IRC22:56
*** otter768 has joined #openstack-nova22:57
*** patrickeast has joined #openstack-nova22:59
*** salv-orlando has joined #openstack-nova22:59
prometheanfirewell, here's some nova log for the failure (compute host side).  https://gist.githubusercontent.com/prometheanfire/c9b70bc4b2abd7d384e6/raw/ae344feff8ef6c36f7685d1433b8e22b811cb03b/gistfile1.txt23:01
*** haomaiwang has quit IRC23:01
prometheanfireI almost want to blame keystone23:01
*** vilobhmm has joined #openstack-nova23:01
*** haomaiwa_ has joined #openstack-nova23:02
*** otter768 has quit IRC23:02
prometheanfireI'm not seeing any errors in keystone's log though23:02
*** salv-orl_ has quit IRC23:02
*** tonytan_brb has quit IRC23:03
anishprometheanfire: you're sure it's not just a firewall issue yes ?23:06
prometheanfireno fw between hosts23:06
*** VW_ has joined #openstack-nova23:06
*** moshele_ has quit IRC23:07
prometheanfirewell, neutron-linuxbridge manages iptables on the comupte hosts23:08
*** emagana has quit IRC23:08
openstackgerritvikas choudhary proposed openstack/nova: Update hypervisor resources correctly in booting failed case  https://review.openstack.org/20868023:09
*** aix has quit IRC23:12
*** vilobhmm has quit IRC23:13
prometheanfireanish: added some cinder-volume log to the bottom https://gist.githubusercontent.com/prometheanfire/c9b70bc4b2abd7d384e6/raw/475e3267e67ccdf2bc803324b41f27ddf3900c02/gistfile1.txt23:14
*** VW_ has quit IRC23:15
*** praneshp has quit IRC23:15
prometheanfireanish: any other logs you think would be applicable?23:15
*** VW_ has joined #openstack-nova23:15
*** Hazelesque has joined #openstack-nova23:16
anishI feel like you'd see an error on the nova side as well if you have no initiator info23:16
anishthis is kilo release right, not devstack ?23:16
*** Sukhdev has quit IRC23:16
prometheanfireya, kilo23:17
prometheanfirerunning from stable/kilo23:17
openstackgerritAnthony Lee proposed openstack/nova: Fix live-migrations usage of the wrong connector information  https://review.openstack.org/20277023:17
prometheanfirethe top log lines are from nova-compute on the compute host23:17
prometheanfireso that is from nova23:17
anishnot sure, sorry. just tried it on mine, do not see the issue23:18
prometheanfiredamn23:18
*** dannywilson has joined #openstack-nova23:18
prometheanfireanish: does the nova compute host expect any cinder config options to be added?23:19
*** VW_ has quit IRC23:20
anishI've never done any changes on the nova side for iscsi23:20
anishunless you have a borked open-iscsi setup on the nova side23:20
prometheanfirewell open-iscsi is installed with no mods23:21
anishcheck if this file exists : /etc/iscsi/initiatorname.iscsi23:21
prometheanfireexists23:21
prometheanfirenothing uncommented, but exists23:22
*** Sukhdev has joined #openstack-nova23:22
*** aysyd has quit IRC23:22
anishprometheanfire: supposed to look like this http://ix.io/k5u23:22
prometheanfireopen-iscsi is a service that needs starting?23:23
anishyes23:24
prometheanfirewell, that explains at least some of it23:24
anishiscsid & iscsid.socket if you have systemd23:24
*** annegentle has quit IRC23:24
anishopen-iscsi if you've installed in manually23:24
prometheanfirebut not running on the cinder node23:24
anishinstead of repos23:24
anishno23:24
anishopen-iscsi is only the initiator23:24
prometheanfireI'm packaging it for gentoo23:24
prometheanfireright23:25
anishcinder node is tgt/lio23:25
anishor iet23:25
prometheanfireya, tgt here23:25
*** patrickeast has quit IRC23:25
prometheanfirehmm, I do have InitiatorName=iqn.2015-08.:openiscsi-83bfdf0741c8f8120d9324fa38c3f7e523:26
anishI would add something between the . & the :23:26
*** Vikas_ has joined #openstack-nova23:26
anishidk if that makes a difference, but some targets will reject that23:27
prometheanfireok23:27
*** Sukhdev has quit IRC23:28
*** dims has quit IRC23:29
*** thorst has joined #openstack-nova23:30
*** marcusvrn has quit IRC23:33
*** ddaskal has quit IRC23:38
*** ijw_ has joined #openstack-nova23:39
*** echoingumesh has quit IRC23:40
*** echoingumesh has joined #openstack-nova23:41
*** ijw has quit IRC23:42
*** echoingumesh has quit IRC23:45
*** atuvenie has quit IRC23:46
*** tpeoples has quit IRC23:46

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