| *** ijw has quit IRC | 00:00 | |
| *** ijw has joined #openstack-nova | 00:00 | |
| *** shz has quit IRC | 00:11 | |
| *** shz has joined #openstack-nova | 00:11 | |
| *** otter768 has joined #openstack-nova | 00:14 | |
| *** r-daneel has quit IRC | 00:16 | |
| *** annegentle has quit IRC | 00:18 | |
| *** diogogmt has joined #openstack-nova | 00:18 | |
| *** otter768 has quit IRC | 00:19 | |
| *** boris-42 has quit IRC | 00:20 | |
| *** penick has quit IRC | 00:25 | |
| *** ijw has quit IRC | 00:26 | |
| openstackgerrit | Meghal Gosalia proposed openstack/nova-specs: Quota Management for Dynamic Quota Resources https://review.openstack.org/206160 | 00:31 |
|---|---|---|
| *** baoli has joined #openstack-nova | 00:34 | |
| *** bapalm has joined #openstack-nova | 00:37 | |
| *** gholler has joined #openstack-nova | 00:37 | |
| openstackgerrit | Jesse J. Cook proposed openstack/nova: Cells: report capped units and max builds https://review.openstack.org/195756 | 00:37 |
| *** thorst has joined #openstack-nova | 00:37 | |
| *** leeantho has quit IRC | 00:37 | |
| *** thorst has quit IRC | 00:37 | |
| *** bapalm has quit IRC | 00:39 | |
| openstackgerrit | Meghal Gosalia proposed openstack/nova-specs: Quota Management for Dynamic Quota Resources https://review.openstack.org/206160 | 00:40 |
| *** zzzeek has joined #openstack-nova | 00:44 | |
| *** tsekiyam_ has joined #openstack-nova | 00:45 | |
| *** heyongli has joined #openstack-nova | 00:46 | |
| *** mtanino has quit IRC | 00:46 | |
| heyongli | hi, Intel PCI CI found that all patches start from a moment begin to failed, we got error message like this: self[key] = field.from_primitive(self, key, value) | 00:48 |
| heyongli | TypeError: 'PciDevice' object does not support item assignment | 00:48 |
| *** tsekiyama has quit IRC | 00:48 | |
| heyongli | here is the full exception: http://paste.openstack.org/show/410220/ | 00:49 |
| *** baoli has quit IRC | 00:49 | |
| heyongli | i don't know which exactly patch cause this error message, still searching, any update is welcome. | 00:50 |
| *** tsekiyam_ has quit IRC | 00:50 | |
| *** vilobhmm has quit IRC | 00:51 | |
| heyongli | here is a example test results: http://52.27.155.124/208706/4/screen-logs/n-cpu.log.gz | 00:54 |
| *** shyu has joined #openstack-nova | 00:57 | |
| *** browne has quit IRC | 01:03 | |
| *** baoli has joined #openstack-nova | 01:06 | |
| *** echoingu_ has quit IRC | 01:06 | |
| *** meghal1 has quit IRC | 01:06 | |
| *** zhenguo has joined #openstack-nova | 01:09 | |
| *** annegentle has joined #openstack-nova | 01:14 | |
| *** Sukhdev has quit IRC | 01:15 | |
| *** zzzeek has quit IRC | 01:16 | |
| *** lyanchih has joined #openstack-nova | 01:16 | |
| *** wanghua has joined #openstack-nova | 01:17 | |
| *** humble_ has joined #openstack-nova | 01:18 | |
| *** dims_ has joined #openstack-nova | 01:21 | |
| *** wanghua has quit IRC | 01:22 | |
| *** dims has quit IRC | 01:22 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 01:31 |
| openstackgerrit | lyanchih proposed openstack/nova: libvirt: serial console ports count upper limit needs to be checked https://review.openstack.org/207373 | 01:32 |
| *** gholler has quit IRC | 01:38 | |
| *** jdandrea has quit IRC | 01:39 | |
| *** SourabhP has quit IRC | 01:39 | |
| *** shz has quit IRC | 01:39 | |
| *** hanzhang has joined #openstack-nova | 01:40 | |
| *** echoingumesh has joined #openstack-nova | 01:41 | |
| *** hshiina has joined #openstack-nova | 01:48 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Set task_state=None when booting instance failed https://review.openstack.org/187145 | 01:49 |
| *** jogo has quit IRC | 01:49 | |
| *** jogo has joined #openstack-nova | 01:50 | |
| *** gholler has joined #openstack-nova | 01:52 | |
| *** unicell has quit IRC | 01:52 | |
| *** unicell has joined #openstack-nova | 01:53 | |
| openstackgerrit | Shiina, Hironori proposed openstack/nova: libvirt: improve error handling of injecting an NMI https://review.openstack.org/208794 | 01:54 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Use Version Independent Identity Plugins for Neutron handshake https://review.openstack.org/209599 | 01:54 |
| *** atuvenie has joined #openstack-nova | 01:55 | |
| *** unicell has quit IRC | 01:58 | |
| *** atuvenie_ has quit IRC | 01:59 | |
| *** achanda has joined #openstack-nova | 02:00 | |
| *** dims has joined #openstack-nova | 02:03 | |
| *** dims_ has quit IRC | 02:05 | |
| *** dims_ has joined #openstack-nova | 02:06 | |
| *** logan2 has quit IRC | 02:08 | |
| *** logan2 has joined #openstack-nova | 02:09 | |
| *** dims has quit IRC | 02:09 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: add support for muti_statuses_query from migration https://review.openstack.org/169631 | 02:09 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: DB:support for migration objects multi-statuses https://review.openstack.org/169604 | 02:09 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: resize-revert can break anti-affinity policy https://review.openstack.org/160110 | 02:09 |
| *** markvoelker has quit IRC | 02:11 | |
| *** baoli has quit IRC | 02:12 | |
| *** markvoelker has joined #openstack-nova | 02:12 | |
| *** baoli has joined #openstack-nova | 02:12 | |
| *** otter768 has joined #openstack-nova | 02:15 | |
| *** angdraug has quit IRC | 02:17 | |
| *** unicell has joined #openstack-nova | 02:18 | |
| *** echoingumesh has quit IRC | 02:19 | |
| *** otter768 has quit IRC | 02:19 | |
| *** dannywilson has quit IRC | 02:20 | |
| *** echoingumesh has joined #openstack-nova | 02:20 | |
| *** humble__ has joined #openstack-nova | 02:20 | |
| *** ssurana has quit IRC | 02:21 | |
| *** unicell1 has joined #openstack-nova | 02:22 | |
| *** irenab_ has joined #openstack-nova | 02:22 | |
| *** dims_ has quit IRC | 02:22 | |
| *** unicell has quit IRC | 02:22 | |
| *** diogogmt has quit IRC | 02:22 | |
| *** Piet has joined #openstack-nova | 02:23 | |
| *** humble_ has quit IRC | 02:23 | |
| *** echoingumesh has quit IRC | 02:24 | |
| *** irenab has quit IRC | 02:24 | |
| *** irenab_ is now known as irenab | 02:24 | |
| *** tjones has joined #openstack-nova | 02:26 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 02:27 |
| *** annashen has joined #openstack-nova | 02:28 | |
| *** rushil has joined #openstack-nova | 02:29 | |
| *** claudiub has quit IRC | 02:34 | |
| *** irenab has quit IRC | 02:40 | |
| *** irenab has joined #openstack-nova | 02:40 | |
| *** dave-mccowan has joined #openstack-nova | 02:40 | |
| openstackgerrit | lyanchih proposed openstack/nova: xenapi: soft reboot should attempt hard reboot on failure https://review.openstack.org/203513 | 02:41 |
| openstackgerrit | Shiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI https://review.openstack.org/202617 | 02:45 |
| *** BobBall_AWOL is now known as BobBall | 02:50 | |
| *** hakimo has joined #openstack-nova | 02:52 | |
| *** meghal has joined #openstack-nova | 02:55 | |
| *** hakimo_ has quit IRC | 02:55 | |
| *** rfolco has quit IRC | 02:56 | |
| *** meghal1 has joined #openstack-nova | 02:57 | |
| *** meghal has quit IRC | 02:59 | |
| *** krtaylor has quit IRC | 02:59 | |
| *** browne has joined #openstack-nova | 03:01 | |
| *** artom has quit IRC | 03:04 | |
| *** markvoelker has quit IRC | 03:04 | |
| *** krtaylor has joined #openstack-nova | 03:12 | |
| *** tjones has quit IRC | 03:13 | |
| *** logan2 has quit IRC | 03:14 | |
| *** irenab has quit IRC | 03:17 | |
| *** irenab has joined #openstack-nova | 03:22 | |
| *** baoli has quit IRC | 03:26 | |
| *** boris-42 has joined #openstack-nova | 03:27 | |
| *** sakaYK has joined #openstack-nova | 03:29 | |
| *** meghal1 has quit IRC | 03:30 | |
| *** hemna has joined #openstack-nova | 03:30 | |
| *** logan2 has joined #openstack-nova | 03:31 | |
| *** gholler_ has joined #openstack-nova | 03:33 | |
| *** hemna has quit IRC | 03:34 | |
| *** gholler has quit IRC | 03:35 | |
| *** marzif__ has joined #openstack-nova | 03:35 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Fix resource leaking when consume_from_instance raise exception https://review.openstack.org/182165 | 03:37 |
| *** gholler_ has quit IRC | 03:37 | |
| *** irenab has quit IRC | 03:40 | |
| *** rushil has quit IRC | 03:41 | |
| *** dave-mccowan has quit IRC | 03:42 | |
| *** MentalRay has quit IRC | 03:43 | |
| *** achanda has quit IRC | 03:44 | |
| *** irenab has joined #openstack-nova | 03:45 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix typos of 'address', 'current' and 'concurrent' https://review.openstack.org/208749 | 03:50 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 03:52 |
| *** jamiec has quit IRC | 03:57 | |
| *** jamiec has joined #openstack-nova | 03:58 | |
| openstackgerrit | Vipin Balachandran proposed openstack/nova: VMware: Delete vmdk UUID during volume detach https://review.openstack.org/204622 | 03:58 |
| *** coolsvap|away is now known as coolsvap | 03:58 | |
| *** vilobhmm has joined #openstack-nova | 04:01 | |
| *** irenab has quit IRC | 04:02 | |
| *** irenab has joined #openstack-nova | 04:03 | |
| *** markvoelker has joined #openstack-nova | 04:04 | |
| *** rajesht has joined #openstack-nova | 04:05 | |
| *** rajesht has quit IRC | 04:06 | |
| *** jecarey has joined #openstack-nova | 04:06 | |
| *** rajesht has joined #openstack-nova | 04:06 | |
| *** markvoelker has quit IRC | 04:09 | |
| *** irenab has quit IRC | 04:10 | |
| *** irenab has joined #openstack-nova | 04:11 | |
| *** lyanchih has quit IRC | 04:15 | |
| *** otter768 has joined #openstack-nova | 04:16 | |
| *** annashen has quit IRC | 04:17 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 04:18 |
| *** sakaYK has quit IRC | 04:19 | |
| *** sakaYK has joined #openstack-nova | 04:20 | |
| *** otter768 has quit IRC | 04:21 | |
| *** edmund has joined #openstack-nova | 04:21 | |
| *** Vikas has quit IRC | 04:23 | |
| *** sakaYK has quit IRC | 04:25 | |
| *** sakaYK has joined #openstack-nova | 04:26 | |
| *** garyk has quit IRC | 04:36 | |
| *** vivekd has joined #openstack-nova | 04:39 | |
| *** annegentle has quit IRC | 04:41 | |
| openstackgerrit | sonu proposed openstack/nova: Replacing unichr() with six.unichr() and reduce with six.moves.reduce() https://review.openstack.org/209752 | 04:41 |
| *** deepakcs has joined #openstack-nova | 04:42 | |
| *** tjones has joined #openstack-nova | 04:44 | |
| *** edmund has quit IRC | 04:47 | |
| *** sudipto has joined #openstack-nova | 04:48 | |
| *** irenab_ has joined #openstack-nova | 04:49 | |
| *** achanda has joined #openstack-nova | 04:50 | |
| *** irenab has quit IRC | 04:50 | |
| *** irenab_ is now known as irenab | 04:50 | |
| *** RuiChen has quit IRC | 04:53 | |
| *** RuiChen has joined #openstack-nova | 04:54 | |
| *** vladikr has joined #openstack-nova | 04:54 | |
| *** irenab_ has joined #openstack-nova | 05:00 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: resize-revert can break anti-affinity policy https://review.openstack.org/160110 | 05:00 |
| *** irenab has quit IRC | 05:00 | |
| *** irenab_ is now known as irenab | 05:01 | |
| *** hhuang has joined #openstack-nova | 05:01 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix cinder volume stuck in swap_volume https://review.openstack.org/207385 | 05:01 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 05:03 |
| *** lyanchih has joined #openstack-nova | 05:04 | |
| *** irenab has quit IRC | 05:05 | |
| *** irenab has joined #openstack-nova | 05:06 | |
| *** emagana has joined #openstack-nova | 05:08 | |
| *** sudipto_ has joined #openstack-nova | 05:08 | |
| *** tjones has quit IRC | 05:08 | |
| *** emagana has quit IRC | 05:10 | |
| *** sudipto has quit IRC | 05:11 | |
| *** irenab has quit IRC | 05:13 | |
| *** sakaYK has quit IRC | 05:14 | |
| *** shyama has joined #openstack-nova | 05:15 | |
| _gryf | sdague, so, should I change the method to use stacking validation, or we leave it as it is? | 05:18 |
| *** jkraj has joined #openstack-nova | 05:21 | |
| *** takedakn has joined #openstack-nova | 05:24 | |
| *** achanda has quit IRC | 05:25 | |
| *** moshele has joined #openstack-nova | 05:27 | |
| *** ftersin has joined #openstack-nova | 05:27 | |
| openstackgerrit | Zachary Sais proposed openstack/nova: Move existing V2 to legacy_v2 https://review.openstack.org/193725 | 05:28 |
| *** takedakn has quit IRC | 05:28 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: tests: change assertEqual args to expected, observed for VIF https://review.openstack.org/204190 | 05:28 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: enable virtio-net multiqueue https://review.openstack.org/128829 | 05:28 |
| *** achanda has joined #openstack-nova | 05:31 | |
| dansmith | _gryf: he wants you to collapse the two methods, use the schema stacking decorators | 05:31 |
| _gryf | dansmith, yes, i got it. qst is should i do this in this patch? i think the merge was just started | 05:33 |
| dansmith | _gryf: ah, no in a separate patch | 05:33 |
| dansmith | _gryf: don't endanger this one from merging :) | 05:33 |
| _gryf | ok :) | 05:33 |
| *** prashup has joined #openstack-nova | 05:34 | |
| _gryf | dansmith, actually, it's merged already | 05:34 |
| dansmith | cool | 05:35 |
| * dansmith -> bed | 05:35 | |
| lyanchih | Hi, I found one version of oslo.versionedobjects will cause master nova create instance failed in nova network. How can I check it more detail. | 05:36 |
| *** ftersin has quit IRC | 05:38 | |
| *** jecarey has quit IRC | 05:38 | |
| hhuang | can someone kindly give me a heads up about the duplicated controller class QuotaSetsController in nova/api/openstack/compute/contrib/quotas.py and nova/api/openstack/compute/plugins/v3/quota_sets.py files? | 05:38 |
| lyanchih | They are difference api version | 05:39 |
| hhuang | the v3 one is used by api v2.1? and the contrib one is for older version? | 05:39 |
| lyanchih | I forget whether v2.1 is using v3 or not. But contrib is really older api version. | 05:40 |
| hhuang | okay, does it mean I can ignore the extensions in contrib if I would like to work on latest api version? | 05:42 |
| *** lsmola has joined #openstack-nova | 05:42 | |
| lyanchih | If I don’t remeber wrong, v2 is freeze. | 05:43 |
| *** yamahata has quit IRC | 05:43 | |
| lyanchih | Let me check website | 05:43 |
| hhuang | yes, that's what I found as well | 05:43 |
| hhuang | thanks a lot lyanchih | 05:43 |
| *** tojuvone has joined #openstack-nova | 05:45 | |
| *** tpeoples has quit IRC | 05:46 | |
| *** SourabhP has joined #openstack-nova | 05:50 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Libvirt: correct libvirt reference url link when live-migration failed https://review.openstack.org/165671 | 05:50 |
| *** tpeoples has joined #openstack-nova | 05:51 | |
| *** hanzhang has quit IRC | 05:51 | |
| *** markvoelker has joined #openstack-nova | 05:51 | |
| *** hanzhang has joined #openstack-nova | 05:52 | |
| heyongli | lyanchih, might be same problem with PCI, some object can not be convert to primitive, is that problem you mentioned? | 05:52 |
| lyanchih | Oh, almost same. But i was wrong with from_primitive | 05:53 |
| *** chhavi has joined #openstack-nova | 05:53 | |
| *** vilobhmm1 has joined #openstack-nova | 05:53 | |
| *** vilobhmm has quit IRC | 05:53 | |
| lyanchih | File "/usr/local/lib/python2.7/dist-packages/oslo_versionedobjects/base.py", line 205, in wrapper | 05:54 |
| lyanchih | 2015-08-06 05:08:31.264 TRACE oslo_messaging.rpc.dispatcher self[key] = field.from_primitive(self, key, value) | 05:54 |
| lyanchih | 2015-08-06 05:08:31.264 TRACE oslo_messaging.rpc.dispatcher TypeError: 'VirtualInterface' object does not support item assignment | 05:54 |
| lyanchih | I was using 5.2 version. But it had no problem when I upgrade to latest | 05:55 |
| *** josecastroleon has joined #openstack-nova | 05:55 | |
| *** markvoelker has quit IRC | 05:56 | |
| *** vilobhmm has joined #openstack-nova | 05:56 | |
| *** vilobhmm1 has quit IRC | 05:57 | |
| heyongli | VirtualInterface' object does not support item assignment, what's this mean, is it mean the virtualInterface need change something? | 05:57 |
| *** cfriesen has quit IRC | 05:58 | |
| lyanchih | It was occur during create VirtualInterface object. Some works are behine oslo_objectversioned. | 05:59 |
| *** dims has joined #openstack-nova | 06:03 | |
| *** prashup has quit IRC | 06:04 | |
| deepakcs | dansmith: Hi, there ? | 06:04 |
| heyongli | lyanchih, don't know how to fix it now, could say more, thanks. | 06:05 |
| *** moshele has quit IRC | 06:05 | |
| heyongli | is this mean, we must supporting ___getitems_ in all objects? | 06:07 |
| *** Nikolay_St has quit IRC | 06:07 | |
| *** hshiina has quit IRC | 06:07 | |
| *** hshiina has joined #openstack-nova | 06:08 | |
| openstackgerrit | Roman Dobosz proposed openstack/nova: Squashing down update method https://review.openstack.org/209766 | 06:08 |
| lyanchih | heyongli: I’m still trace different code between master and 5.2 tag of versionedobjects’s code. | 06:09 |
| *** dims has quit IRC | 06:09 | |
| *** sakaYK has joined #openstack-nova | 06:13 | |
| *** irenab has joined #openstack-nova | 06:14 | |
| *** otter768 has joined #openstack-nova | 06:17 | |
| *** sakaYK has quit IRC | 06:19 | |
| *** boris-42 has quit IRC | 06:20 | |
| heyongli | lyanchih, thanks very much, it does break several things, seems to put all object to somewhere, unfortunately, PciDevice and several other object does not supporting this operations. | 06:20 |
| *** chuckcarmack has joined #openstack-nova | 06:20 | |
| lyanchih | heyongli: I found problem | 06:21 |
| lyanchih | But I still check more detail | 06:21 |
| *** otter768 has quit IRC | 06:21 | |
| *** sakaYK has joined #openstack-nova | 06:21 | |
| heyongli | lyanchih, share with me please, -:) | 06:22 |
| *** irenab has quit IRC | 06:22 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 06:22 |
| lyanchih | In few days ago, VirtualInterface had remove supoer class “base.NovaObjectDictCompat” | 06:23 |
| lyanchih | s/supoer/super/ | 06:23 |
| *** irenab has joined #openstack-nova | 06:24 | |
| *** vivekd has quit IRC | 06:24 | |
| *** vivekd has joined #openstack-nova | 06:24 | |
| *** prashup has joined #openstack-nova | 06:24 | |
| lyanchih | But versionedobjects 5.2 was using “self[key] = field.from_primitive(self, key, value)” | 06:24 |
| lyanchih | It was modified to setattr(self, key, field.from_primitive(self, key, value)) at least | 06:25 |
| *** sakaYK has quit IRC | 06:25 | |
| *** sakaYK has joined #openstack-nova | 06:26 | |
| lyanchih | The object is not dict like, therefore it doesn’t support item assign in old version. | 06:26 |
| lyanchih | This is what I though. | 06:26 |
| heyongli | PciDevice never use the base.NovaObjectDictCompat, but not broken before, this confuse me. | 06:27 |
| *** jianghuaw has joined #openstack-nova | 06:28 | |
| heyongli | PciDevice-> base.NovaObject -> ovoo_base.VersionedObject, not depend on base.NovaObjectDictCompat, | 06:29 |
| lyanchih | It was happen with remotable decorator. | 06:29 |
| openstackgerrit | Alex Xu proposed openstack/nova: Skip additionalProperties checks when LegacyV2CompatibleWrapper enabled https://review.openstack.org/193858 | 06:32 |
| openstackgerrit | Alex Xu proposed openstack/nova: Fix the wrong file name for legacy v2 compatible wrapper functional test https://review.openstack.org/206326 | 06:32 |
| openstackgerrit | Alex Xu proposed openstack/nova: Return v2 version info with v2 legacy compatible wrapper https://review.openstack.org/206930 | 06:32 |
| openstackgerrit | Alex Xu proposed openstack/nova: Add scenario for API sample tests with legacy v2 compatible wrapper https://review.openstack.org/203485 | 06:32 |
| heyongli | PciDevice need base on base.NovaObjectDictCompat, i think, let me try. | 06:33 |
| *** prashup has quit IRC | 06:33 | |
| lyanchih | I had search PciDevice, looks like it’s called in n-conductor. Maybe you have different issue. | 06:34 |
| lyanchih | In fact, I still not sure those super class’s purpose….. | 06:35 |
| *** afazekas_ has joined #openstack-nova | 06:37 | |
| *** sudipto_ has quit IRC | 06:37 | |
| *** marzif__ has quit IRC | 06:39 | |
| *** Marga_ has joined #openstack-nova | 06:41 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/209780 | 06:41 |
| *** sakaYK has quit IRC | 06:44 | |
| *** sakaYK has joined #openstack-nova | 06:45 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Fix launch from volume-backed image https://review.openstack.org/208932 | 06:49 |
| *** sakaYK has quit IRC | 06:50 | |
| openstackgerrit | Roman Dobosz proposed openstack/nova: Squashing down update method https://review.openstack.org/209766 | 06:51 |
| *** kumar has joined #openstack-nova | 06:51 | |
| *** sakaYK has joined #openstack-nova | 06:51 | |
| *** belmoreira has joined #openstack-nova | 06:51 | |
| *** scheuran has joined #openstack-nova | 06:52 | |
| *** prashup has joined #openstack-nova | 06:52 | |
| *** ftersin has joined #openstack-nova | 06:52 | |
| *** moshele has joined #openstack-nova | 06:52 | |
| *** sakaYK has quit IRC | 06:57 | |
| *** sakaYK has joined #openstack-nova | 06:58 | |
| *** Marga_ has quit IRC | 07:00 | |
| *** Marga_ has joined #openstack-nova | 07:01 | |
| *** takedakn has joined #openstack-nova | 07:01 | |
| *** ishant has joined #openstack-nova | 07:01 | |
| *** sakaYK has quit IRC | 07:03 | |
| *** sakaYK has joined #openstack-nova | 07:04 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Remove instance group member when delete instance https://review.openstack.org/173664 | 07:04 |
| heyongli | lyanchih, after i add the compact base class, now i ruining to the fail lure you are working on. | 07:05 |
| *** ildikov has quit IRC | 07:07 | |
| *** Murali has joined #openstack-nova | 07:07 | |
| *** irenab has quit IRC | 07:08 | |
| *** irenab has joined #openstack-nova | 07:08 | |
| *** romainh has joined #openstack-nova | 07:09 | |
| *** vladikr has quit IRC | 07:10 | |
| *** tpeoples has quit IRC | 07:14 | |
| *** vladikr has joined #openstack-nova | 07:14 | |
| *** rushiagr_away is now known as rushiagr | 07:16 | |
| *** rdopiera has joined #openstack-nova | 07:16 | |
| *** pixelbeat has joined #openstack-nova | 07:17 | |
| *** vladikr has quit IRC | 07:20 | |
| *** claudiub has joined #openstack-nova | 07:20 | |
| *** sakaYK has quit IRC | 07:23 | |
| *** markus_z has joined #openstack-nova | 07:24 | |
| *** sakaYK has joined #openstack-nova | 07:24 | |
| *** Marga_ has quit IRC | 07:25 | |
| *** ildikov has joined #openstack-nova | 07:26 | |
| *** Murali has quit IRC | 07:27 | |
| *** browne has quit IRC | 07:27 | |
| *** takedakn has quit IRC | 07:28 | |
| *** romainh has left #openstack-nova | 07:28 | |
| *** sakaYK has quit IRC | 07:30 | |
| *** Nikolay_St has joined #openstack-nova | 07:30 | |
| *** sakaYK has joined #openstack-nova | 07:30 | |
| *** Murali has joined #openstack-nova | 07:32 | |
| *** sakaYK has quit IRC | 07:36 | |
| *** ftersin has quit IRC | 07:37 | |
| *** sakaYK has joined #openstack-nova | 07:37 | |
| *** Marga_ has joined #openstack-nova | 07:38 | |
| lyanchih | heyongli: Really?! Maybe we are face different issue…. | 07:40 |
| *** garyk has joined #openstack-nova | 07:40 | |
| *** Marga_ has quit IRC | 07:41 | |
| *** Marga_ has joined #openstack-nova | 07:41 | |
| *** gszasz has joined #openstack-nova | 07:42 | |
| *** sakaYK has quit IRC | 07:43 | |
| *** sakaYK_ has joined #openstack-nova | 07:43 | |
| *** jcoufal has joined #openstack-nova | 07:45 | |
| *** alex_klimov has joined #openstack-nova | 07:45 | |
| *** vilobhmm has quit IRC | 07:46 | |
| *** ftersin has joined #openstack-nova | 07:46 | |
| *** sahid has joined #openstack-nova | 07:47 | |
| *** SourabhP has quit IRC | 07:47 | |
| *** sakaYK_ has quit IRC | 07:48 | |
| *** sakaYK has joined #openstack-nova | 07:49 | |
| *** jlanoux has joined #openstack-nova | 07:50 | |
| *** RuiChen has quit IRC | 07:50 | |
| *** RuiChen has joined #openstack-nova | 07:50 | |
| *** sayali has joined #openstack-nova | 07:50 | |
| *** markvoelker has joined #openstack-nova | 07:52 | |
| *** sakaYK has quit IRC | 07:53 | |
| *** sakaYK has joined #openstack-nova | 07:54 | |
| *** ishant_ has joined #openstack-nova | 07:55 | |
| *** markvoelker has quit IRC | 07:56 | |
| *** sakaYK has quit IRC | 07:56 | |
| *** alex_klimov has quit IRC | 07:57 | |
| *** sakaYK has joined #openstack-nova | 07:57 | |
| *** apuimedo has joined #openstack-nova | 07:59 | |
| *** ishant has quit IRC | 07:59 | |
| heyongli | lyanchih, yeah , might different, PciDevice need that base class. | 07:59 |
| *** sakaYK has quit IRC | 08:00 | |
| *** Murali has quit IRC | 08:00 | |
| *** sakaYK has joined #openstack-nova | 08:00 | |
| *** rdopiera has quit IRC | 08:02 | |
| *** kmARC has joined #openstack-nova | 08:03 | |
| heyongli | class VirtualInterface(base.NovaPersistentObject, base.NovaObject, base.NovaObjectDictCompat): | 08:04 |
| *** kmARC_ has joined #openstack-nova | 08:04 | |
| *** RA_ has quit IRC | 08:04 | |
| heyongli | lyanchih, add this "class VirtualInterface(base.NovaPersistentObject, base.NovaObject, base.NovaObjectDictCompat)" make it passed testing , verified by intel PCI CI, we need make the object all dict compatable, it seems. | 08:04 |
| *** sakaYK has quit IRC | 08:05 | |
| *** sakaYK has joined #openstack-nova | 08:05 | |
| heyongli | i'm going to proposal fix for PCI, would you like to fix virtualInterface? | 08:05 |
| lyanchih | But I had saw some blueprints are ready to cleanup NovaObjectDictCompat one object by one object. | 08:05 |
| lyanchih | You can checkout this review: https://review.openstack.org/#/c/200823/ | 08:07 |
| heyongli | that's another story, you clean that, please fix the oslo, before that, we need this to make sure things work | 08:07 |
| heyongli | i'm very not sure if is the problem need be fixed in the versionobject, but i will proposal this change first. | 08:08 |
| *** ttrumm has joined #openstack-nova | 08:09 | |
| *** prashup has quit IRC | 08:09 | |
| *** kiran-r has joined #openstack-nova | 08:10 | |
| *** prashup has joined #openstack-nova | 08:11 | |
| *** rdopiera has joined #openstack-nova | 08:13 | |
| *** achanda has quit IRC | 08:14 | |
| *** rdopiera has quit IRC | 08:14 | |
| *** rdopiera has joined #openstack-nova | 08:14 | |
| *** jistr has joined #openstack-nova | 08:14 | |
| *** yassine_ has joined #openstack-nova | 08:14 | |
| openstackgerrit | Yongli He proposed openstack/nova: make PciDevice and VirtualInterface compatable with dict https://review.openstack.org/209812 | 08:17 |
| *** achanda has joined #openstack-nova | 08:18 | |
| *** otter768 has joined #openstack-nova | 08:18 | |
| johnthetubaguy | kfox1111: its a case of getting a spec up ASAP, that one sounds like we need it to be quite cross project in some ways, but lets get something up and reviewed very soon | 08:18 |
| *** alex_klimov has joined #openstack-nova | 08:18 | |
| johnthetubaguy | mikal: hi, did you want to catch up around exceptions? | 08:18 |
| *** heyongli has quit IRC | 08:19 | |
| *** GeraldK has joined #openstack-nova | 08:19 | |
| *** heyongli has joined #openstack-nova | 08:19 | |
| heyongli | johnthetubaguy, PCI CI been halted till problem https://review.openstack.org/209812 been addressed, cause this make all test failed, then given a fail message to every patch. just F.Y.I | 08:21 |
| *** Marga_ has quit IRC | 08:21 | |
| johnthetubaguy | heyongli: do you want to make a sub group in here, for these kinds of issues? https://etherpad.openstack.org/p/liberty-nova-priorities-tracking | 08:22 |
| openstackgerrit | Jeffrey Zhang proposed openstack/nova: tenant_id implies all_tenants for servers list https://review.openstack.org/209813 | 08:22 |
| *** danpb has joined #openstack-nova | 08:22 | |
| *** otter768 has quit IRC | 08:23 | |
| *** Jeffrey4l has joined #openstack-nova | 08:23 | |
| mikal | johnthetubaguy: I do, but have to go to dinner in 8 minutes | 08:23 |
| *** lucasagomes has joined #openstack-nova | 08:23 | |
| mikal | johnthetubaguy: want me to ping you when I get back? | 08:23 |
| heyongli | johnthetubaguy, do you mean CI break problem? | 08:23 |
| johnthetubaguy | mikal: which ever is easier, did you see the email I sent out with the list of exception freezes? | 08:24 |
| mikal | johnthetubaguy: yep, I did | 08:24 |
| johnthetubaguy | heyongli: either PCI or third party CI breaks, I created a new this for third party CI breaks, and added your patch | 08:24 |
| johnthetubaguy | mikal: did that all make sense? | 08:24 |
| johnthetubaguy | heyongli: there a a few big issues with that patch | 08:25 |
| *** Marga_ has joined #openstack-nova | 08:25 | |
| heyongli | johnthetubaguy, good ideas | 08:25 |
| johnthetubaguy | heyongli: there is no bug attached, seems like we should attach a bug | 08:25 |
| openstackgerrit | Jeffrey Zhang proposed openstack/nova: tenant_id implies all_tenants for servers list https://review.openstack.org/209813 | 08:25 |
| *** ankit_ag has joined #openstack-nova | 08:25 | |
| heyongli | johnthetubaguy, yeah, let's me open a bug for that | 08:25 |
| johnthetubaguy | heyongli: also, that removal of the dict compat was intentional | 08:26 |
| johnthetubaguy | heyongli: can't you just fix the code that is using the object? | 08:26 |
| mikal | johnthetubaguy: yep, it works for me | 08:26 |
| * mikal has been pulled into a bunch of Rackspace stuff this week which has been distracting unfortunately | 08:26 | |
| heyongli | johnthetubaguy, might good fix is in the osolo versioned object, but that take long time | 08:26 |
| johnthetubaguy | mikal: cool, I mean we actually got a lot more merged that it was looking like this time last week, which cheered me up a little | 08:26 |
| johnthetubaguy | mikal: yeah, I can understand that :( | 08:27 |
| heyongli | johnthetubaguy, if fixed it , and then begin to remove them all , how about this procedure? | 08:27 |
| johnthetubaguy | mikal: so feel free to catch up when its best for you, like don't feel obligated to catch up later if that doesn't make sense, we can catch up tomorrow, or at a better point in time | 08:27 |
| mikal | johnthetubaguy: yeah, not a problem. I need to come back after dinner anyways, so I will ping you when I return | 08:28 |
| johnthetubaguy | mikal: cool | 08:28 |
| johnthetubaguy | heyongli: how bad is the problem, I would have expected unit tests to break, for example? | 08:29 |
| *** achanda has quit IRC | 08:29 | |
| heyongli | all test cases in our CI for PCI/SRIOV is failed because of this | 08:29 |
| heyongli | so these 2 CI been stopped. | 08:29 |
| johnthetubaguy | heyongli: agreed, I am trying to work out why our unit tests in tree are not failing | 08:30 |
| johnthetubaguy | seems like we have bad unit tests there, or a lack of coverage | 08:30 |
| johnthetubaguy | they though have caught these sorts of errors | 08:30 |
| *** ndipanov has joined #openstack-nova | 08:30 | |
| heyongli | and the version for VirtualInterface, mean you can not create networking for a VM, seem unit test must be failed , but don't know why | 08:30 |
| johnthetubaguy | heyongli: so here is an idea... | 08:30 |
| johnthetubaguy | heyongli: lets get a bug open, one for adding the dictcompat, one for the fix up, and get a WIP patch up for the second one, to show how bad the problem is | 08:31 |
| johnthetubaguy | heyongli: do you have a stack trace from the compute logs, so its clear what is breaking? | 08:31 |
| heyongli | i have | 08:31 |
| heyongli | http://paste.openstack.org/show/410220/ | 08:32 |
| johnthetubaguy | heyongli: perfect, lets get that in into the bug report you are opening | 08:32 |
| heyongli | johnthetubaguy, i'm going to fill one, | 08:32 |
| *** subscope has quit IRC | 08:33 | |
| _gryf | johnthetubaguy, hi | 08:33 |
| johnthetubaguy | heyongli: what version of oslo_versionedobjects do you have in your CI system? | 08:33 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add os_admin_user to use with set admin password https://review.openstack.org/185921 | 08:33 |
| heyongli | newest, according to 'requiemets.txt' | 08:33 |
| _gryf | johnthetubaguy, there was a change requested by sdague, and while the original patch was already merged I've created another one for mark-host-down | 08:34 |
| _gryf | johnthetubaguy, could you please take a look? - https://review.openstack.org/#/c/209766/ | 08:34 |
| johnthetubaguy | heyongli: is there a quick way to check the exact version, I am seeing this bug is fixed in the current code base | 08:35 |
| *** Murali has joined #openstack-nova | 08:35 | |
| johnthetubaguy | _gryf: opened it in a tab | 08:36 |
| heyongli | i'm checking it now | 08:36 |
| lyanchih | johnthetubaguy: The issue was occured before 6.0 | 08:36 |
| johnthetubaguy | lyanchih: yeah, I am just seeing this: https://github.com/openstack/oslo.versionedobjects/commit/58070642b52a2332605ab5d834b7de323da9ba01 | 08:37 |
| johnthetubaguy | heyongli: seems like you need this fix in your CI: https://github.com/openstack/oslo.versionedobjects/commit/58070642b52a2332605ab5d834b7de323da9ba01 | 08:37 |
| heyongli | johnthetubaguy, seems this is the right fix, but this problem just pop up yesterday, i'm trying to check what happens there. | 08:38 |
| johnthetubaguy | heyongli: its possible we only dropped the dict compat stuff yesterday | 08:39 |
| sahid | danpb: when you have moment if you can take a look at this https://review.openstack.org/#/c/207373/ we set by default a pty console and i do not know if when we ask for serial console whether we should remove it | 08:40 |
| openstackgerrit | Earle F. Philhower, III proposed openstack/nova: Add os_brick-based VolumeDriver for HGST connector https://review.openstack.org/194412 | 08:40 |
| lyanchih | heyongli: You can try to upgrade your oslo.versionedobject | 08:40 |
| lyanchih | Yes, dict compat was dropped halt day ago | 08:40 |
| *** achanda has joined #openstack-nova | 08:40 | |
| heyongli | lyanchih, i do upgrade that, accutally every new patch been testing with all packages reinstalled. | 08:40 |
| johnthetubaguy | heyongli: I think this is the patch that "caused" you need for oslo_versionedobjects>0.6 https://github.com/openstack/nova/commit/e464267085ca45129ef9b092db41112697ddf3ca | 08:41 |
| openstackgerrit | Emma Foley proposed openstack/nova: Added 2MB and 1GB page size to hw:mem_page_size https://review.openstack.org/209564 | 08:42 |
| heyongli | johnthetubaguy, i do suspect this patch, cause after this one , all testcase failed, the 'clean up' in the CI should be improved, i think. | 08:42 |
| johnthetubaguy | heyongli: OK, let me know how your digging goes, it certainly seems like your CI is using an old version of oslo_versionedobjects, for some reason | 08:44 |
| johnthetubaguy | heyongli: I think the change you want is to bump the min version to 0.6.0? https://github.com/openstack/nova/blob/master/requirements.txt#L52 | 08:45 |
| lyanchih | heyongli: Oh, you had your own CI? I originally though you mean openstack jenkin CI | 08:45 |
| heyongli | johnthetubaguy, i upgrade the oslo by hand , let me check it | 08:46 |
| heyongli | lyanchih, yeah, i mean the PCI third party CI | 08:46 |
| heyongli | johnthetubaguy, it works | 08:46 |
| johnthetubaguy | heyongli: yeah, so I think we need to make 0.6.0 the min version in requirements: https://github.com/openstack/requirements/blob/master/global-requirements.txt#L91 | 08:47 |
| heyongli | we had a list to clean up before every patch been tested, seems the list should be all global-requiements.txt | 08:47 |
| *** NithyaG has joined #openstack-nova | 08:48 | |
| *** Kevin_Zheng_ has joined #openstack-nova | 08:49 | |
| *** achanda has quit IRC | 08:51 | |
| johnthetubaguy | heyongli: are you good making progress on this now? | 08:52 |
| heyongli | yeah | 08:53 |
| heyongli | CI will recover | 08:53 |
| heyongli | oslo.versionedobjects!=0.5.0,>=0.3.0, is this ok? | 08:53 |
| lyanchih | It is latest code in global-requirement | 08:54 |
| *** marzif__ has joined #openstack-nova | 08:54 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: scheduler: weight nodes based on their cached images https://review.openstack.org/173304 | 08:54 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: Add new image cache monitor in compute nodes https://review.openstack.org/173303 | 08:54 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: vmware: implement get_image_cache_manager https://review.openstack.org/187602 | 08:54 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: virt: add method for getting driver image cache manager https://review.openstack.org/209823 | 08:54 |
| *** chhavi has quit IRC | 08:55 | |
| *** ftersin has quit IRC | 08:56 | |
| heyongli | global: oslo.versionedobjects>=0.3.0,!=0.5.0 | 08:58 |
| *** hshiina has quit IRC | 08:58 | |
| heyongli | PCI CI machine is 0.5.2, but don't not work | 08:58 |
| lyanchih | heyongli: yes, 0.5.2 will fail | 08:59 |
| heyongli | yeah, we need upgrade that requiements in the nova/requirements.txt | 09:00 |
| lyanchih | Maybe your CI will upgrade package according to global requirements. | 09:00 |
| lyanchih | But the condition is match so it doesn’t upgrade. | 09:00 |
| heyongli | devstack will do that i think | 09:00 |
| lyanchih | But it wouldn’t auto upgrade | 09:01 |
| heyongli | lyanchih, johnthetubaguy so the real problem is nova/requirements.txt ? | 09:01 |
| heyongli | every time devstack will install that requirements.txt i remember | 09:01 |
| heyongli | our system meeting the nova/requirements.txt so it not upgrade | 09:02 |
| lyanchih | Yes, but it will not upgrade to latest version of each library | 09:02 |
| heyongli | lyanchih, it will not upgrade to latest version because nova//requirements.txt no requied to do so, i think. | 09:02 |
| lyanchih | yes, 0.5.2 is still match requirement’s version restrict | 09:03 |
| *** jkraj has quit IRC | 09:04 | |
| *** sakaYK has quit IRC | 09:05 | |
| openstackgerrit | Yongli He proposed openstack/nova: use new oslo.versionedobjects https://review.openstack.org/209812 | 09:08 |
| heyongli | johnthetubaguy, i think we need use oslo.versionedobjects>0.6.0 instead of oslo.versionedobjects>=0.3.0,!=0.5.0 which currently in the nova requirements. new patch sent. | 09:09 |
| johnthetubaguy | heyongli: yes, agreed, thats what I suggested in my code review of your patch, and what I was attempting to say earlier | 09:10 |
| heyongli | johnthetubaguy, thanks, but why unit test passed? | 09:10 |
| johnthetubaguy | heyongli: the unit tests are using a newer version, I assume | 09:10 |
| johnthetubaguy | heyongli: Collecting oslo.versionedobjects!=0.5.0,>=0.3.0 (from -r /home/jenkins/workspace/gate-nova-python27/requirements.txt (line 48)) | 09:11 |
| johnthetubaguy | Downloading http://pypi.region-b.geo-1.openstack.org/packages/py2.py3/o/oslo.versionedobjects/oslo.versionedobjects-0.7.0-py2.py3-none-any.whl (57kB) | 09:11 |
| johnthetubaguy | that what I see in the jenkins logs | 09:11 |
| heyongli | but this is not based on the requements | 09:12 |
| johnthetubaguy | heyongli: it is, it grabs the newest, which matches the requirements, the full log is here: http://logs.openstack.org/94/164494/27/check/gate-nova-python27/b3cd017/tox/py27-1.log | 09:12 |
| heyongli | johnthetubaguy, so, such a test method will missing to catch problem like this one | 09:13 |
| *** Marga_ has quit IRC | 09:13 | |
| lyanchih | heyongli: Maybe you can’t directly modify requirement. It is auto upgrade by global-requiremenets. | 09:13 |
| *** sakaYK has joined #openstack-nova | 09:13 | |
| heyongli | for case the installed version meeting the requirements. | 09:14 |
| johnthetubaguy | heyongli: the correct fix to to raise the min version of olso.versionedobjects right? | 09:14 |
| heyongli | lyanchih, yeah, johnthetubaguy suggesting to do so, i updated the patch. | 09:14 |
| johnthetubaguy | heyongli: OK, cool | 09:14 |
| johnthetubaguy | heyongli: I am not quite sure about the test stragegy here, I though there was talk of testing the min versions of things | 09:16 |
| heyongli | from this time begin, our CI re install these packages every time, but such a testing to check if the requirements valided or not is needed. | 09:16 |
| johnthetubaguy | I lost track of the ML discussion around that | 09:16 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Clean the deprecated noauth middleware https://review.openstack.org/184424 | 09:16 |
| johnthetubaguy | heyongli: I would look at making sure your CI system uses the same pip command that the gate uses, that should help you, I think | 09:16 |
| *** kjelly has joined #openstack-nova | 09:17 | |
| lyanchih | Besides I had post review before. https://review.openstack.org/#/c/209797/ | 09:17 |
| heyongli | johnthetubaguy, suppose i install 0.5.3, this ok according the requirements.txt, but accually the requirements.txt is wrong, the gate method will not be able to find this problem | 09:17 |
| johnthetubaguy | heyongli: agreed | 09:18 |
| heyongli | johnthetubaguy, global-requirements.txt not come from individual project? i think the nova need be fixed, am i wrong? | 09:20 |
| johnthetubaguy | heyongli: once you update global, it should get sync into nova | 09:20 |
| johnthetubaguy | global must be done first, as I understand it | 09:20 |
| heyongli | got it | 09:20 |
| *** haomaiwang has quit IRC | 09:21 | |
| *** prashup has quit IRC | 09:21 | |
| lyanchih | The error was occured during remote call, gate usually only focus on unit test. | 09:21 |
| *** haomaiwang has joined #openstack-nova | 09:21 | |
| lyanchih | Maybe this is why gate don’t find the problem. | 09:22 |
| johnthetubaguy | lyanchih: they gate runs a full functional test, in several configurations, using tempest | 09:23 |
| johnthetubaguy | s/they/the/ | 09:23 |
| heyongli | johnthetubaguy, i hope you can review this patch , it's there long time, revised very much, https://review.openstack.org/#/c/131321/ | 09:28 |
| *** kumar has quit IRC | 09:28 | |
| johnthetubaguy | heyongli: the bug it says it is fixing is marked as released? | 09:29 |
| *** humble__ has quit IRC | 09:30 | |
| heyongli | johnthetubaguy, no , that's not fixed it, just related, | 09:30 |
| johnthetubaguy | heyongli: should we not have a new bug for that patch then? | 09:31 |
| heyongli | only this new management merged that bug could be fully addressed, beside this, the pci resource management really need this patch , make it seems same thing with vcpu | 09:31 |
| *** matrohon has joined #openstack-nova | 09:31 | |
| heyongli | johnthetubaguy, that patch not fix the but mentioned here, i don't know why marked fixed. | 09:32 |
| heyongli | s/but/bug/ | 09:32 |
| johnthetubaguy | heyongli: looks like its was marked a duplicate of this fixed bug: https://bugs.launchpad.net/nova/+bug/1415768 | 09:33 |
| openstack | Launchpad bug 1415768 in OpenStack Compute (nova) juno "the pci deivce assigned to instance is inconsistent with DB record when restarting nova-compute" [High,Fix committed] - Assigned to Nikola Đipanov (ndipanov) | 09:33 |
| rajesht | johnthetubaguy: could ou please review this patch https://review.openstack.org/#/c/170118/ | 09:34 |
| heyongli | johnthetubaguy, yeah, but that really a different problem, sorry not point this out, so we need a new bug? | 09:34 |
| *** GeraldK has quit IRC | 09:34 | |
| heyongli | the bug my patch to fix a scenario which the pci devices is missing , not the pci in the db been clear. | 09:36 |
| ndipanov | heyongli, I marked it as fixed | 09:36 |
| johnthetubaguy | heyongli: maybe just update the bug with that detail I guess? | 09:36 |
| heyongli | ok | 09:36 |
| johnthetubaguy | heyongli: I think the code looks good, just getting my head around some details before I vote on it | 09:37 |
| ndipanov | johnthetubaguy, heyongli so I believe that patch should be merged | 09:37 |
| johnthetubaguy | rajesht: the bug attached to your patch has apparently been fixed already https://review.openstack.org/#/c/170118 | 09:38 |
| ndipanov | but it does not fix the bug anymore | 09:38 |
| johnthetubaguy | I mean: https://launchpad.net/bugs/1296414 | 09:38 |
| openstack | Launchpad bug 1296414 in OpenStack Compute (nova) "quotas not updated when periodic tasks or startup finish deletes" [High,Fix released] - Assigned to Rajesh Tailor (rajesh-tailor) | 09:38 |
| ndipanov | the bug is fixed by paul' | 09:38 |
| ndipanov | s refactoring | 09:38 |
| johnthetubaguy | ndipanov: yeah, thats what I thought | 09:38 |
| ndipanov | however heyongli 's patch is also very good and makes things more clear and probably fixes some more edge cases | 09:39 |
| ndipanov | that we don't know about | 09:39 |
| ndipanov | yet | 09:39 |
| ndipanov | johnthetubaguy, I will re-review it today and vote since I (used to be) familiar with taht code :) | 09:39 |
| ndipanov | unless you beat me to it | 09:39 |
| ndipanov | it just fell off of my radar since it hasn't been updated lately I think | 09:40 |
| *** jkraj has joined #openstack-nova | 09:40 | |
| johnthetubaguy | ndipanov: that would be good, saves me trying to read up on that, its on my TODO list to take a look so it was tempting, but I need to do some spec stuff today really | 09:40 |
| heyongli | johnthetubaguy, ndipanov , thanks, i'd leave home , and will address anything tomorrow. | 09:40 |
| heyongli | bye | 09:40 |
| rajesht | johnthetubaguy: can you give me reference of paul's refactoring | 09:41 |
| ndipanov | heyongli, bye - drive/ride safely! | 09:41 |
| ndipanov | rajesht, it's linked on the bug | 09:41 |
| rajesht | johnthetubaguy, ndipanov: I have submitted two patches for this | 09:41 |
| johnthetubaguy | rajesht: sorry, I think you got caught up in a different conversation | 09:41 |
| ndipanov | https://review.openstack.org/148904 | 09:41 |
| johnthetubaguy | rajesht: so is the second of two patches to fix this bug? https://launchpad.net/bugs/1296414 | 09:42 |
| openstack | Launchpad bug 1296414 in OpenStack Compute (nova) "quotas not updated when periodic tasks or startup finish deletes" [High,Fix released] - Assigned to Rajesh Tailor (rajesh-tailor) | 09:42 |
| rajesht | johnthetubaguy: yes | 09:42 |
| openstackgerrit | Bartosz Fic proposed openstack/nova: Live migration failure in API doesn't set vm_state to Error https://review.openstack.org/168916 | 09:43 |
| ndipanov | rajesht, that doesn't seem related tbh | 09:43 |
| rajesht | ndipanov: thanks for reference, but I don't think that solves problem which I wanted to solve in https://review.openstack.org/#/c/170118/ | 09:44 |
| ndipanov | rajesht, I don't think it does... | 09:45 |
| ndipanov | not sure why you thought it did | 09:46 |
| *** lyanchih has quit IRC | 09:46 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova-specs: Create nova-bug-report cmd utility https://review.openstack.org/209847 | 09:46 |
| ndipanov | quotas sux | 09:46 |
| rajesht | johnthetubaguy: https://review.openstack.org/#/c/170118/ patch fixes issue where during resizing, if user tries to delete instance and it got stuck in deleting task state | 09:47 |
| *** deepakcs has quit IRC | 09:47 | |
| rajesht | in that case, on restart of compute service resources should be updated according to the new flavor | 09:47 |
| rajesht | earlier it was updating resources according to the older flavor information | 09:48 |
| *** dims has joined #openstack-nova | 09:49 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova-specs: Create nova-bug-report cmd utility https://review.openstack.org/209847 | 09:49 |
| rajesht | johnthetubaguy: thanks, you got it right :) | 09:50 |
| *** markvoelker has joined #openstack-nova | 09:53 | |
| *** markvoelker has quit IRC | 09:57 | |
| *** hakimo has quit IRC | 09:58 | |
| *** nagyz_ has joined #openstack-nova | 09:58 | |
| *** hakimo has joined #openstack-nova | 09:59 | |
| *** nagyz has quit IRC | 10:01 | |
| *** yassine_ has quit IRC | 10:03 | |
| mikal | . | 10:05 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/128940 | 10:06 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Fixed incorrect behaviour of method _check_instance_exists https://review.openstack.org/203037 | 10:06 |
| *** shyu has quit IRC | 10:06 | |
| snikitin | johnthetubaguy: Hi, John! I rebased my patch which you just approved. Could you approve this again, please? https://review.openstack.org/#/c/203037/ | 10:08 |
| *** lyanchih has joined #openstack-nova | 10:08 | |
| johnthetubaguy | snikitin: sure | 10:09 |
| snikitin | johnthetubaguy: thanks! | 10:09 |
| *** kmARC has quit IRC | 10:09 | |
| *** kmARC_ has quit IRC | 10:09 | |
| johnthetubaguy | snikitin: did you have a question on the extension_info stuff in that patch? | 10:10 |
| johnthetubaguy | snikitin: I think its correct that it doesn't appear in the v2 extension list, that bit looks correct right? | 10:11 |
| snikitin | johnthetubaguy: right | 10:11 |
| *** zhenguo has quit IRC | 10:12 | |
| *** otter768 has joined #openstack-nova | 10:18 | |
| *** sakaYK_ has joined #openstack-nova | 10:21 | |
| *** sakaYK has quit IRC | 10:22 | |
| *** oomichi has quit IRC | 10:22 | |
| *** otter768 has quit IRC | 10:23 | |
| *** sahid has quit IRC | 10:25 | |
| *** sahid has joined #openstack-nova | 10:26 | |
| *** kmARC has joined #openstack-nova | 10:27 | |
| *** GeraldK has joined #openstack-nova | 10:27 | |
| *** kmARC_ has joined #openstack-nova | 10:27 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Use Version Independent Identity Plugins for Neutron handshake https://review.openstack.org/209599 | 10:28 |
| snikitin | johnthetubaguy: I answered on your questions :) | 10:29 |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: wip: gate debug lp 1480698 https://review.openstack.org/209858 | 10:29 |
| openstack | Launchpad bug 1480698 in oslo.db "MySQL error - too many connections" [Undecided,New] https://launchpad.net/bugs/1480698 | 10:29 |
| dims | rpodolyaka: nice trick! | 10:30 |
| rpodolyaka1 | dims: heh, I'm desperate :P | 10:31 |
| dims | i can see :) | 10:31 |
| openstackgerrit | Sean Dague proposed openstack/nova: Introduce LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202777 | 10:35 |
| *** josecastroleon has quit IRC | 10:36 | |
| *** mwagner_lap has quit IRC | 10:39 | |
| sdague | zomg, that mriedem_away patch series | 10:39 |
| *** sakaYK_ has quit IRC | 10:42 | |
| *** shuquan has joined #openstack-nova | 10:42 | |
| _gryf | sdague, hi, I have changed the update method as you suggested - https://review.openstack.org/#/c/209766/ - could you take a look? | 10:45 |
| sdague | _gryf: great, thanks | 10:45 |
| _gryf | sdague, :) | 10:46 |
| *** dims_ has joined #openstack-nova | 10:51 | |
| openstackgerrit | John Garbutt proposed openstack/nova: adding a volume detach_force call https://review.openstack.org/184537 | 10:51 |
| *** ftersin has joined #openstack-nova | 10:52 | |
| *** dims has quit IRC | 10:53 | |
| *** markvoelker has joined #openstack-nova | 10:54 | |
| *** IanGovett has joined #openstack-nova | 10:54 | |
| openstackgerrit | Shuquan Huang proposed openstack/nova: nova.utils._get_root_helper() should be public https://review.openstack.org/207366 | 10:54 |
| dims_ | sdague: found a hole in nova's keystone v3 story | 10:55 |
| dims_ | sdague: added some notes here - https://etherpad.openstack.org/p/keystone-v3-adoption-barriers | 10:56 |
| *** markvoelker has quit IRC | 10:58 | |
| sdague | ok, honestly, it's early enough that I'm not going to be able to process that | 10:59 |
| *** smatzek has joined #openstack-nova | 11:00 | |
| *** sahid has quit IRC | 11:00 | |
| sdague | plus, honestly, it seems really weird to me that we've create keystone so that we don't have to hand user and password all over the place, but then we have to put admin credentials to the system in plain text on every host | 11:01 |
| *** jkraj has quit IRC | 11:01 | |
| * dims_ hands sdague some coffee :) | 11:02 | |
| *** rpodolyaka1 is now known as roman_hungry | 11:06 | |
| johnthetubaguy | sdague: the debates about giving each instance a token are very related to that idea | 11:06 |
| *** Marga_ has joined #openstack-nova | 11:07 | |
| johnthetubaguy | sdague: you could generate a per instance cert that point back to a single user, although I think that causes issues in the keystone data model or something, so they are trying to avoid that, I can't remember the details | 11:07 |
| *** liusheng has quit IRC | 11:08 | |
| *** josecastroleon has joined #openstack-nova | 11:09 | |
| *** vladikr has joined #openstack-nova | 11:15 | |
| *** shihanzhang has joined #openstack-nova | 11:17 | |
| *** hanzhang has quit IRC | 11:18 | |
| *** ftersin has quit IRC | 11:18 | |
| sdague | yeh, it just feels pretty gorpy still | 11:18 |
| *** jkraj has joined #openstack-nova | 11:19 | |
| sdague | dims_: so I think that having to read a bunch of patches and irc comments to figure out the thing is less useful. Can you summarize "here is what's wrong" and "here's what needs to be fixed" | 11:21 |
| *** thorst has joined #openstack-nova | 11:24 | |
| *** kmARC_ has quit IRC | 11:26 | |
| *** kmARC has quit IRC | 11:26 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Allow resize root_gb to 0 for volume-backed instances https://review.openstack.org/207794 | 11:28 |
| *** hshiina has joined #openstack-nova | 11:28 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Add missing rules in policy.json https://review.openstack.org/188235 | 11:30 |
| *** shuquan has quit IRC | 11:30 | |
| *** Jeffrey4l has quit IRC | 11:33 | |
| *** kmARC has joined #openstack-nova | 11:34 | |
| Kevin_Zheng_ | Hi All, I have a fix like this: return also project_id when ask for server-group information, as for an admin user, it is allowed to list server-groups for all projects through CLI command "nova server-group-list --all-project" and also through REST API. When an admin user does this, All the server-group will return and it is really hard to identify which | 11:35 |
| Kevin_Zheng_ | server-group belong to which project, so I believe it will be really useful if we also return the project_id information. I have already submitted a patch about it: https://review.openstack.org/#/c/209018/ . I'm aware that this kind of fix may need to submit a spec and also microversions, if that is so, where should I submit the spec? in the backlog? or | 11:35 |
| Kevin_Zheng_ | anywhere else? Thanks a lot. | 11:35 |
| *** mdrabe has quit IRC | 11:39 | |
| *** dprince has joined #openstack-nova | 11:40 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: tests: change assertEqual args to expected, observed for VIF https://review.openstack.org/204190 | 11:40 |
| *** kmARC_ has joined #openstack-nova | 11:40 | |
| openstackgerrit | Shiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI https://review.openstack.org/202605 | 11:41 |
| openstackgerrit | Shiina, Hironori proposed openstack/nova: libvirt: improve error handling of injecting an NMI https://review.openstack.org/208794 | 11:42 |
| *** yassine_ has joined #openstack-nova | 11:42 | |
| *** ftersin has joined #openstack-nova | 11:45 | |
| *** Marga_ has quit IRC | 11:46 | |
| *** lyanchih has quit IRC | 11:46 | |
| openstackgerrit | Arata Notsu proposed openstack/nova: More specific error message for OverQuota on BDM https://review.openstack.org/209897 | 11:49 |
| *** ishant_ has quit IRC | 11:53 | |
| *** diogogmt has joined #openstack-nova | 11:54 | |
| *** Jeffrey4l has joined #openstack-nova | 11:56 | |
| *** alexschm has joined #openstack-nova | 11:56 | |
| *** kmARC has quit IRC | 11:56 | |
| *** kmARC_ has quit IRC | 11:56 | |
| *** rushiagr is now known as rushiagr_away | 11:56 | |
| *** nelsnelson has quit IRC | 11:57 | |
| *** busterswt has joined #openstack-nova | 12:00 | |
| ftersin | danpb: hi, could you again set +2 to https://review.openstack.org/#/c/208932/ | 12:01 |
| ftersin | danpb: it failed to merge tonight due to conflict with another change of ImageMeta | 12:01 |
| *** vladikr has quit IRC | 12:02 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add instance action events for live migration https://review.openstack.org/195372 | 12:04 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Resize: Return to initial state instead of ACTIVE when resize fails https://review.openstack.org/202379 | 12:04 |
| *** ferest has joined #openstack-nova | 12:05 | |
| *** roman_hungry is now known as rpodolyaka2 | 12:07 | |
| *** vivekd has quit IRC | 12:08 | |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: libvirt: Nova part of volume snapshot improvements https://review.openstack.org/172813 | 12:09 |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: objects: Add support for OnlineSnapshotInfo object https://review.openstack.org/194538 | 12:09 |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: libvirt: Refactor _volume_snapshot_delete function https://review.openstack.org/209904 | 12:09 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova-specs: Create nova-bug-report cmd utility https://review.openstack.org/209847 | 12:09 |
| *** ferest has quit IRC | 12:09 | |
| jaypipes | johnthetubaguy: on force detach volume, yes, I will review all the code ASAP. | 12:09 |
| johnthetubaguy | jaypipes: awesome, thank you, I honestly hadn't seen the patch was up already till I went digging, sorry about that | 12:10 |
| jaypipes | np | 12:10 |
| jaypipes | johnthetubaguy: and sorry for losing my cool yesterday. some $work stuff was really affecting me. | 12:10 |
| johnthetubaguy | jaypipes: no worries, stuff happens, I just found the spec thats up for that extension stuff, I super want that in liberty if we can | 12:11 |
| *** tojuvone has quit IRC | 12:11 | |
| Kevin_Zheng_ | johnthetubaguy: Hi, I have a fix like this: return also project_id when ask for server-group information, as for an admin user, it is allowed to list server-groups for all projects through CLI command "nova server-group-list --all-project" and also through REST API. When an admin user does this, All the server-group will return and it is really hard to | 12:13 |
| Kevin_Zheng_ | identify which server-group belong to which project, so I believe it will be really useful if we also return the project_id information. I have already submitted a patch about it: https://review.openstack.org/#/c/209018/ . I'm aware that this kind of fix may need to submit a spec and also microversions, if that is so, where should I submit the spec? in the | 12:13 |
| Kevin_Zheng_ | backlog? or somewhere else? Thanks a lot. | 12:13 |
| *** sahid has joined #openstack-nova | 12:13 | |
| johnthetubaguy | Kevin_Zheng_: good question, so its a bug fix, so I think we can still attempt to add this into liberty if we are quick, have you seen the other API specs folks have already done? | 12:14 |
| Kevin_Zheng_ | johnthetubaguy: not yet, I will try to find some | 12:16 |
| johnthetubaguy | Kevin_Zheng_: no worries, I will try find you a good example | 12:16 |
| *** markvoelker has joined #openstack-nova | 12:16 | |
| Kevin_Zheng_ | johnthetubaguy: Thanks a lot | 12:16 |
| johnthetubaguy | Kevin_Zheng_: so here is an example at least: http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/show-reserved-status-in-os-fixed-ips-api.html | 12:17 |
| johnthetubaguy | https://git.openstack.org/cgit/openstack/nova-specs/tree/specs/liberty/approved/show-reserved-status-in-os-fixed-ips-api.rst | 12:17 |
| johnthetubaguy | Kevin_Zheng_: so we do specs for API changes, because once we merge an API change, we assume it could be deployed somewhere, so we have to support it for ever, so we are being careful not to land "bad" API changes. I hope that makes sense. | 12:18 |
| *** otter768 has joined #openstack-nova | 12:19 | |
| Kevin_Zheng_ | johnthetubaguy: thanks a lot, I will check it | 12:20 |
| *** unicell has joined #openstack-nova | 12:20 | |
| *** unicell1 has quit IRC | 12:21 | |
| *** mwagner_lap has joined #openstack-nova | 12:22 | |
| Kevin_Zheng_ | johnthetubaguy: one more question, should I also submit a bp like the link you showed me, or do I just do the spec | 12:23 |
| openstackgerrit | Shiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI https://review.openstack.org/202617 | 12:23 |
| johnthetubaguy | Kevin_Zheng_: lets just get the spec up, link that to your bug, and see how it looks, I think thats the easiest way at this point | 12:23 |
| *** dims has joined #openstack-nova | 12:24 | |
| *** otter768 has quit IRC | 12:24 | |
| Kevin_Zheng_ | johnthetubaguy: alright, thanks | 12:25 |
| *** dims_ has quit IRC | 12:27 | |
| *** Marga_ has joined #openstack-nova | 12:29 | |
| *** jdandrea has joined #openstack-nova | 12:30 | |
| *** diogogmt has quit IRC | 12:31 | |
| *** dane-fichter has joined #openstack-nova | 12:31 | |
| *** VW_ has joined #openstack-nova | 12:34 | |
| *** baoli has joined #openstack-nova | 12:35 | |
| openstackgerrit | dane-fichter proposed openstack/nova-specs: Nova Support of Glance Image Signing https://review.openstack.org/188874 | 12:35 |
| *** aysyd has joined #openstack-nova | 12:36 | |
| *** edmondsw has joined #openstack-nova | 12:36 | |
| *** lyanchih has joined #openstack-nova | 12:37 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: scheduler: weight nodes based on their cached images https://review.openstack.org/173304 | 12:37 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: compute: add new image cache monitor https://review.openstack.org/173303 | 12:37 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: imagecache: set base_dir as a property https://review.openstack.org/209912 | 12:37 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: imagecache: implement listing of cached images https://review.openstack.org/209913 | 12:37 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: virt: add method for obtaining the list of cached images https://review.openstack.org/209914 | 12:37 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: libvirt: implement get_cached_images method https://review.openstack.org/209915 | 12:37 |
| *** andrey-mp has joined #openstack-nova | 12:38 | |
| jaypipes | andrearosa: around? | 12:38 |
| jaypipes | andrearosa: regarding https://review.openstack.org/#/c/184537/, what is your estimate of time needed to convert that patch to not use a REST API but instead use only a nova-manage db volume-force-detach CLI command? | 12:39 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Use Version Independent Identity Plugins for Neutron handshake https://review.openstack.org/209599 | 12:39 |
| andrey-mp | hi core! who can close this review - https://review.openstack.org/#/c/208932/ ? it was closed once but merge has failed... gating of out ec2 project is broken without this and we can't commit anything :( | 12:39 |
| ndipanov | failure of our review process in all it's glory: https://review.openstack.org/#/c/182165/10 | 12:39 |
| *** raildo has joined #openstack-nova | 12:39 | |
| ndipanov | 4 different patchests - 4 different and only tangentially related things to the actual bug being discussed, no consideration for previous comments | 12:40 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: libvirt: implement get_cached_images_sha1 method https://review.openstack.org/209915 | 12:41 |
| ndipanov | imho a glorious example of what not to do when reviewing ^ | 12:41 |
| jaypipes | ndipanov: thx. | 12:41 |
| ndipanov | jaypipes, just sayin' | 12:41 |
| ndipanov | I am guilty of all of these things too | 12:41 |
| ndipanov | it's partly abou the tools | 12:41 |
| ndipanov | but not discussing it is not going to make it better | 12:42 |
| ndipanov | jaypipes, don't hate me :) | 12:42 |
| jaypipes | ndipanov: I don't :) | 12:42 |
| andrey-mp | jaypipes, ndipanov: please, could you look to? | 12:43 |
| *** bapalm has joined #openstack-nova | 12:44 | |
| jaypipes | ndipanov: +2 from me on that patch after RuiChen pushed my requested changes. | 12:44 |
| johnthetubaguy | ndipanov: the toggle CI button has made that a lot easier, but its not perfect for sure | 12:44 |
| jaypipes | andrey-mp: looking... | 12:44 |
| openstackgerrit | Chuck Carmack proposed openstack/nova: Add documentation for the nova-cells command. https://review.openstack.org/206179 | 12:46 |
| jaypipes | johnthetubaguy: I set bug 1481220 to critical... | 12:46 |
| openstack | bug 1481220 in OpenStack Compute (nova) "Cannot boot from volume-backed instance snapshot" [Critical,In progress] https://launchpad.net/bugs/1481220 - Assigned to Feodor Tersin (ftersin) | 12:46 |
| *** zul has joined #openstack-nova | 12:46 | |
| *** baoli_ has joined #openstack-nova | 12:47 | |
| johnthetubaguy | jaypipes: yeah, its probably more a high++, but thats cool, it needs attention | 12:47 |
| johnthetubaguy | jaypipes: I get the feeling we need a new tempest test somehwere | 12:47 |
| jaypipes | johnthetubaguy: yes we do. | 12:48 |
| jaypipes | andrey-mp: ^^ | 12:48 |
| jaypipes | johnthetubaguy: +2 from me on the patch from andrey-mp | 12:48 |
| andrey-mp | jaypipes: thanks! | 12:48 |
| jaypipes | johnthetubaguy: there was a previous +2 from danpb, FYI | 12:48 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Add project id information in os-server-group API https://review.openstack.org/209917 | 12:48 |
| *** baoli has quit IRC | 12:49 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova-specs: Create nova-bug-report cmd utility https://review.openstack.org/209847 | 12:50 |
| *** afazekas_ has quit IRC | 12:50 | |
| *** krtaylor has quit IRC | 12:50 | |
| _gryf | ndipanov, jaypipes, johnthetubaguy can I get the attention for the small patch: https://review.openstack.org/#/c/209766/ ? | 12:51 |
| *** rushil has joined #openstack-nova | 12:51 | |
| *** VW_ has quit IRC | 12:52 | |
| *** VW_ has joined #openstack-nova | 12:52 | |
| *** mriedem_away has quit IRC | 12:53 | |
| *** rushil_ has joined #openstack-nova | 12:53 | |
| *** marzif__ has quit IRC | 12:53 | |
| *** VW_ has quit IRC | 12:53 | |
| jaypipes | _gryf: done :) | 12:53 |
| _gryf | jaypipes, thank you :) | 12:54 |
| jaypipes | np! | 12:54 |
| *** marzif__ has joined #openstack-nova | 12:54 | |
| *** VW_ has joined #openstack-nova | 12:54 | |
| ndipanov | johnthetubaguy, would feel better if you did the +A | 12:55 |
| ndipanov | https://review.openstack.org/#/c/182165/ | 12:55 |
| ndipanov | johnthetubaguy, ^ | 12:55 |
| *** shuquan has joined #openstack-nova | 12:56 | |
| *** rushil has quit IRC | 12:56 | |
| *** VW_ has quit IRC | 12:56 | |
| *** VW_ has joined #openstack-nova | 12:56 | |
| shuquan | hi guys, can you help to review this https://review.openstack.org/#/c/207366/ ?? Thanks | 12:57 |
| *** artom has joined #openstack-nova | 12:58 | |
| johnthetubaguy | ndipanov: OK, I need to sit down and look at that | 12:59 |
| ndipanov | johnthetubaguy, yeah a tricky one | 13:00 |
| *** VW_ has quit IRC | 13:01 | |
| *** dave-mccowan has joined #openstack-nova | 13:01 | |
| *** rlrossit has joined #openstack-nova | 13:02 | |
| openstackgerrit | dane-fichter proposed openstack/nova-specs: Nova Support of Glance Image Signing https://review.openstack.org/188874 | 13:03 |
| *** Marga_ has quit IRC | 13:03 | |
| *** krtaylor has joined #openstack-nova | 13:03 | |
| jdandrea | Is the tenant id available from within a scheduler filter? (Suspecting "not at the moment") | 13:03 |
| *** dave-mcc_ has joined #openstack-nova | 13:04 | |
| *** tojuvone has joined #openstack-nova | 13:04 | |
| johnthetubaguy | ndipanov: I have a meeting right now, its going to be after the nova meeting I am afraid, as a heads up | 13:05 |
| ndipanov | thanks! johnthetubaguy | 13:06 |
| ndipanov | jaypipes, am I missing something here? https://review.openstack.org/#/c/208932/ | 13:06 |
| *** dave-mccowan has quit IRC | 13:07 | |
| *** ivasev has joined #openstack-nova | 13:08 | |
| *** lyanchih has quit IRC | 13:09 | |
| *** abhishekk has joined #openstack-nova | 13:10 | |
| *** kiran-r has quit IRC | 13:11 | |
| *** ivasev has quit IRC | 13:12 | |
| *** lpetrut has joined #openstack-nova | 13:12 | |
| *** browne has joined #openstack-nova | 13:15 | |
| *** shuquan has quit IRC | 13:15 | |
| *** kumar has joined #openstack-nova | 13:17 | |
| *** rfolco has joined #openstack-nova | 13:18 | |
| rlrossit | dansmith: I hope I spelled everything right now: https://review.openstack.org/#/c/198904/ | 13:21 |
| *** dustins has joined #openstack-nova | 13:22 | |
| *** ivasev has joined #openstack-nova | 13:23 | |
| openstackgerrit | Simona Iuliana Toader proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot https://review.openstack.org/209581 | 13:23 |
| *** thumpba has joined #openstack-nova | 13:23 | |
| *** thumpba_ has joined #openstack-nova | 13:26 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: CONF.allow_resize_to_same_host should check only once in controller https://review.openstack.org/186236 | 13:26 |
| *** shuquan has joined #openstack-nova | 13:27 | |
| *** thumpba has quit IRC | 13:30 | |
| *** marcusvrn has joined #openstack-nova | 13:30 | |
| *** thumpba_ has quit IRC | 13:30 | |
| *** mriedem has joined #openstack-nova | 13:31 | |
| *** shuquan has quit IRC | 13:32 | |
| mriedem | dims: didn't you have a change to make some imagemeta fields nullable ? | 13:32 |
| mriedem | https://review.openstack.org/#/c/208932/ | 13:32 |
| *** mestery has quit IRC | 13:32 | |
| *** Jeffrey4l has quit IRC | 13:34 | |
| *** thumpba has joined #openstack-nova | 13:36 | |
| mriedem | how the hell didn't test_volume_boot_pattern catch https://bugs.launchpad.net/nova/+bug/1481220 ? | 13:37 |
| openstack | Launchpad bug 1481220 in OpenStack Compute (nova) "Cannot boot from volume-backed instance snapshot" [Critical,In progress] - Assigned to Feodor Tersin (ftersin) | 13:37 |
| mriedem | mtreinish: ^? | 13:37 |
| *** Marga_ has joined #openstack-nova | 13:37 | |
| *** shuquan has joined #openstack-nova | 13:37 | |
| *** mestery has joined #openstack-nova | 13:38 | |
| *** VW_ has joined #openstack-nova | 13:38 | |
| *** shuquan has quit IRC | 13:38 | |
| jaypipes | ndipanov: going from nullable=False to nullable=True should always be backwards-compatible... | 13:39 |
| jaypipes | ndipanov: it's the opposite that isn't. | 13:39 |
| ndipanov | jaypipes, yes but we need some backporting | 13:40 |
| jaypipes | ndipanov: though I suppose that older code that expects non-nullable things would blow up ... hmm. | 13:40 |
| ndipanov | right | 13:40 |
| *** mnestratov has joined #openstack-nova | 13:40 | |
| *** zzzeek has joined #openstack-nova | 13:40 | |
| ndipanov | may not be applicable in this case though - was just trying to figure that out but then needed coffee | 13:40 |
| jaypipes | ndipanov: no, you and mriedem are correct. I am wrong, again. | 13:42 |
| rlrossit | jaypipes: ndipanov: interjecting with my 2 cents.... you need to up the version of the object just so you can butt in an obj_make_compatible() rule that sets those to some value in order to backport | 13:42 |
| ndipanov | yeah but it's not as simple | 13:43 |
| ndipanov | how do you backport a null to something | 13:43 |
| jaypipes | right.. | 13:43 |
| ndipanov | "fake-value-disregard" | 13:43 |
| rlrossit | I'm getting deja vu so hard right now.... | 13:43 |
| jaypipes | the older computes will continue to blow up. | 13:43 |
| *** mriedem has quit IRC | 13:43 | |
| jaypipes | but is that something we care about? not sure... | 13:43 |
| rlrossit | dansmith: was talking about this to someone before because they made a new field nullable, but they had to assign it to something else in the backport | 13:43 |
| jaypipes | ndipanov: see, the thing is... the compute nodes blow up *now*. So, this actually wouldn't be changing anything in that regard. | 13:44 |
| jaypipes | ndipanov: and it would fix things for the new compute nodes. | 13:44 |
| dansmith | yeah, it's hard to go *to* nullable because you can't synthesize a value | 13:44 |
| openstackgerrit | Marian Horban proposed openstack/nova: libvirt:Rsync remote FS driver was added https://review.openstack.org/146887 | 13:44 |
| jaypipes | dansmith: well, it's hard to go either way actually :) | 13:45 |
| jaypipes | dansmith: and good morning, early riser. :) | 13:45 |
| dansmith | heh | 13:45 |
| rlrossit | dansmith: yeah, the situation from that one patch was adding a mapping, so you could just steal something off the mapping and stuff it in | 13:45 |
| dansmith | jaypipes: yeah, either way is hard for sure | 13:45 |
| dansmith | rlrossit: yeah, that's lucky | 13:45 |
| *** mriedem has joined #openstack-nova | 13:46 | |
| *** jswarren has joined #openstack-nova | 13:46 | |
| rlrossit | is there any way we could set it to some sort of sentinel or stub or something to prevent previous computes from blowing up? | 13:46 |
| *** edmondsw has quit IRC | 13:46 | |
| ndipanov | maybe "" could be used here but we would need to make sure this does not trip up a check somewhere or something like that | 13:47 |
| * jaypipes wonders why volume-backed images have a nullable checksum.. | 13:47 | |
| rlrossit | ndipanov: was thinking the same thing | 13:47 |
| ndipanov | jaypipes, well because they are just placeholders for block device mapping right? | 13:48 |
| dims | mriedem: yes, it was clashing with Feodor's so i abandoned mine | 13:48 |
| ndipanov | as in there is no image - just metadata about the instance | 13:48 |
| jaypipes | ndipanov: well, when the snapshot is taken of the root disk (the booted volume), it is uploaded to Glance, and Glance should checksum the resulting bits... | 13:48 |
| mriedem | dims: did yours have the backward compat code? | 13:48 |
| dansmith | do we need it to be null? | 13:48 |
| dansmith | can we leave it unset if we don't have one? | 13:49 |
| dansmith | that was my suggestion in the patch, IIRC | 13:49 |
| *** gholler has joined #openstack-nova | 13:49 | |
| *** lpetrut has quit IRC | 13:49 | |
| *** dguryanov has joined #openstack-nova | 13:49 | |
| ndipanov | jaypipes, this snapshot is done in cinder and just block devices are added (I think) | 13:49 |
| dims | mriedem: don't remember, let me check | 13:49 |
| jaypipes | ndipanov: ah, yes, I remember now... | 13:49 |
| dims | mriedem: https://review.openstack.org/#/c/207679/5/nova/objects/image_meta.py,cm | 13:50 |
| * rlrossit wonders if dansmith having rlrossit working on objects was a punishment | 13:51 | |
| *** schwicke has joined #openstack-nova | 13:51 | |
| mriedem | huh, never seen this before http://logs.openstack.org/77/202777/13/gate/gate-nova-python27/af2fc54/console.html#_2015-08-06_12_39_46_246 | 13:52 |
| *** ijw has joined #openstack-nova | 13:52 | |
| *** READ10 has joined #openstack-nova | 13:52 | |
| dims | rlrossit: when my older kid's soccer team lost all games in the season i termed it "character-building exercise" :) | 13:53 |
| jaypipes | ffs, I need to boot into Windows to join some stupid meeting. back in a bit. | 13:53 |
| mriedem | and it's new | 13:53 |
| *** ankit_ag has quit IRC | 13:53 | |
| *** jaypipes has quit IRC | 13:53 | |
| mriedem | sdague: http://logs.openstack.org/77/202777/13/gate/gate-nova-python27/af2fc54/console.html#_2015-08-06_12_39_46_246 ever seen a thing like that? | 13:53 |
| mriedem | inprogress? | 13:53 |
| *** jecarey has joined #openstack-nova | 13:54 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add test to check relations at or below current https://review.openstack.org/203642 | 13:54 |
| *** meghal has joined #openstack-nova | 13:54 | |
| rlrossit | dansmith: ^ I found another broken object | 13:54 |
| mriedem | seg fault, nice | 13:54 |
| dansmith | rlrossit: okay, will look in a sec | 13:54 |
| mriedem | mtreinish: is this os_testr's issue? http://logs.openstack.org/77/202777/13/gate/gate-nova-python27/af2fc54/console.html#_2015-08-06_12_39_46_250 | 13:54 |
| dims | mriedem: "Segmentation fault (core dumped)" | 13:55 |
| garyk | mriedem: dansmith: regarding the objecst stuff i posted https://review.openstack.org/209583 and an fixing up the objecst pacth following dans suggestions | 13:55 |
| dims | mriedem: on "2015-08-06 12:39:20.984" | 13:55 |
| mriedem | dims: yeah | 13:55 |
| *** gholler has quit IRC | 13:55 | |
| sdague | mriedem: looking | 13:55 |
| mriedem | gdi, nova meeting in 5 min | 13:55 |
| *** lucas___ has joined #openstack-nova | 13:55 | |
| sdague | mriedem: yeh, so that means the subunit stream isn't finished | 13:56 |
| sdague | right, the segmentation fault is the key issue | 13:56 |
| *** ctrath has joined #openstack-nova | 13:56 | |
| sdague | it's a pretty good indication that that was the test where the segmentation fault happened | 13:56 |
| mriedem | k, looking for recent changes | 13:56 |
| *** n0ano has joined #openstack-nova | 13:57 | |
| *** meghal1 has joined #openstack-nova | 13:58 | |
| mriedem | bug reported https://bugs.launchpad.net/nova/+bug/1482230 | 13:58 |
| openstack | Launchpad bug 1482230 in OpenStack Compute (nova) "LibvirtConnTestCase.test_clean_shutdown_first_time causes subunit segfault" [Undecided,New] | 13:58 |
| *** lyanchih has joined #openstack-nova | 13:58 | |
| *** andrey-mp has quit IRC | 13:59 | |
| *** meghal has quit IRC | 13:59 | |
| *** dustins has quit IRC | 13:59 | |
| johnthetubaguy | mikal tjones cburgess jgrimm adrian_otto funzo mjturek jcookekhugen irina_pov krtaylor danpb alexpilotti flip214 jaypipes gilliard garyk edleafe dims moshele anteaya Nisha sileht claudiub lxsli neiljerram markus_z swamireddy alevine tonyb andreykurilin ndipanov sc68cal akuriata artom jlvillal mnestratov kashyap aloga rgeragnov bauzas xyang tpatil med_ nic scottda its nova meeting time | 14:00 |
| *** meghal1 is now known as meghal | 14:00 | |
| openstackgerrit | lyanchih proposed openstack/nova: libvirt: serial console ports count upper limit needs to be checked https://review.openstack.org/207373 | 14:01 |
| *** r-daneel has joined #openstack-nova | 14:01 | |
| *** mdrabe has joined #openstack-nova | 14:01 | |
| *** cfriesen has joined #openstack-nova | 14:04 | |
| *** ijuwang has joined #openstack-nova | 14:06 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Change docstring in test to comment https://review.openstack.org/209948 | 14:06 |
| sdague | mriedem: also, if gerrit shows a merge conflict, recheck is just going to fail | 14:06 |
| sdague | but hitting the rebase button in the web ui might fix things | 14:07 |
| mriedem | sdague: ok, i just rebased locally but didn't push it up b/c of your reaction yesterday | 14:07 |
| *** lyanchih has quit IRC | 14:07 | |
| sdague | mriedem: oh, sorry, I didn't mean to discourage you | 14:07 |
| sdague | I'm just mad that a change like this has taken so much manual time | 14:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert SMBFS vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202779 | 14:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert NFS volume driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202778 | 14:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Introduce LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202777 | 14:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Scality vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202783 | 14:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Quobyte driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202782 | 14:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert GPFS volume driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202781 | 14:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert GlusterFS driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202780 | 14:08 |
| mriedem | so be it | 14:08 |
| mriedem | i can +W myself | 14:08 |
| sdague | mriedem: yeh | 14:09 |
| sdague | that's valid | 14:09 |
| openstackgerrit | Eduard Biceri-Matei proposed openstack/nova: libvirt: New driver for volumes backed by local file. https://review.openstack.org/134134 | 14:10 |
| *** ftersin has quit IRC | 14:12 | |
| sdague | johnthetubaguy: can we get this merged - https://review.openstack.org/#/c/198622/ - because the functional patch was approved by you, but blocked in merging on the spec change | 14:12 |
| openstackgerrit | Eduard Biceri-Matei proposed openstack/nova: libvirt: New driver for volumes backed by local file. https://review.openstack.org/134134 | 14:13 |
| *** tpeoples has joined #openstack-nova | 14:13 | |
| mtreinish | mriedem: err yeah that error is coming from subunit-trace but it's probable because the stream died part way through | 14:15 |
| *** xgsa has joined #openstack-nova | 14:16 | |
| *** dustins has joined #openstack-nova | 14:17 | |
| *** kylek3h has joined #openstack-nova | 14:18 | |
| openstackgerrit | Eduard Biceri-Matei proposed openstack/nova: libvirt: New driver for volumes backed by local file. https://review.openstack.org/134134 | 14:18 |
| *** xgsa1 has quit IRC | 14:18 | |
| openstackgerrit | Daniel Berrange proposed openstack/nova: ironic: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189362 | 14:20 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: objects: convert config drive to use ImageMeta object https://review.openstack.org/189323 | 14:20 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188119 | 14:20 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: make instance compulsory in blockinfo APIs https://review.openstack.org/188118 | 14:20 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: hyperv: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189302 | 14:20 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 14:20 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 14:20 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 14:20 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: xenapi: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188790 | 14:20 |
| *** otter768 has joined #openstack-nova | 14:20 | |
| *** ttrumm has quit IRC | 14:21 | |
| *** irenab has quit IRC | 14:21 | |
| openstackgerrit | Simona Iuliana Toader proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot https://review.openstack.org/209581 | 14:22 |
| *** otter768 has quit IRC | 14:25 | |
| *** dims has quit IRC | 14:25 | |
| *** MentalRay has joined #openstack-nova | 14:25 | |
| *** dims has joined #openstack-nova | 14:26 | |
| *** afazekas_ has joined #openstack-nova | 14:27 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Add notification for service status change https://review.openstack.org/182350 | 14:28 |
| *** edmondsw has joined #openstack-nova | 14:28 | |
| openstackgerrit | garyk proposed openstack/nova: Compute: use instance object for refresh_instance_security_rules https://review.openstack.org/187537 | 14:28 |
| *** baoli_ has quit IRC | 14:30 | |
| sdague | mtreinish: there is a stack trace in os-testr at the end because of timestamp assumptions | 14:31 |
| sdague | that should probably be fixed | 14:31 |
| sdague | worker 5 has no end time | 14:31 |
| openstackgerrit | Marian Horban proposed openstack/nova: Rsync compression removed https://review.openstack.org/209957 | 14:32 |
| sdague | for some reason I tripped over https://review.openstack.org/#/c/144875/ which is glance v2 review, that's completely unreviewable. It's in a procedural -2 from Kilo (which it probably shouldn't be), but it's also clearly unlandable in current form | 14:32 |
| *** xgsa1 has joined #openstack-nova | 14:33 | |
| *** xgsa has quit IRC | 14:33 | |
| sdague | is glance v2 a thing that's still somewhere in the priorities? | 14:33 |
| mtreinish | sdague: sure, we could be a bit more defensive about that it subunit-trace | 14:34 |
| *** jordanP has joined #openstack-nova | 14:34 | |
| abhishekk | ndipanov: hi | 14:35 |
| ndipanov | abhishekk, so the fix (not ideal but good) is that we cannot pass none for block devices in _get_instance_disk_info | 14:35 |
| ndipanov | now that's not so easy | 14:36 |
| ndipanov | because it's called from the resource tracker | 14:36 |
| *** edmund has joined #openstack-nova | 14:36 | |
| abhishekk | ndipanov: any alternative for this? | 14:36 |
| ndipanov | meaning the RT would also need to gett block devices for each instance | 14:36 |
| lxsli | meghal: just wanted to say hi o/ | 14:36 |
| ndipanov | well the alternative is to never call it | 14:36 |
| ndipanov | :) | 14:36 |
| ndipanov | I don't think the information it gives us is useful | 14:37 |
| abhishekk | ndipanov: :) | 14:37 |
| ndipanov | another approach would be to make it possible to join block devices on instances in a single query | 14:37 |
| meghal | lxsli: hi…thanks for your prompt feedback | 14:37 |
| abhishekk | ndipanov: could you please reply to this ML http://lists.openstack.org/pipermail/openstack-dev/2015-July/070736.html | 14:37 |
| meghal | lxsli: am working on rest of the comments | 14:37 |
| johnthetubaguy | danpb: dansmith: alaski: mikal: we have a few liberty spec revisions/fixups that need a little attention if you get chance, this is one sdague just pointed out: https://review.openstack.org/#/c/198622/ | 14:37 |
| lxsli | meghal: cool :) | 14:37 |
| ndipanov | abhishekk, OK - I will also comment on the bug | 14:37 |
| raildo | johnthetubaguy: mriedem Can you help me about what is the next steps, who don't have a FFE accepted? We need to resubmit the bp to mitaka and/or resubmit the spec to backlog? | 14:38 |
| *** diogogmt has joined #openstack-nova | 14:38 | |
| ndipanov | abhishekk, sorry about this taking so long :/ | 14:38 |
| johnthetubaguy | raildo: please move it into the mitaka directory | 14:38 |
| abhishekk | ndipanov: sure, thank you for your time, I will aslo try to dig it out from my side | 14:38 |
| johnthetubaguy | raildo: have you got a link to the spec you are talking about? | 14:38 |
| raildo | johnthetubaguy: https://github.com/openstack/nova-specs/blob/master/specs/liberty/approved/nested-quota-driver-api.rst | 14:39 |
| dansmith | johnthetubaguy: how is that a spec revision/ | 14:39 |
| johnthetubaguy | dansmith: ah, so I think I got my wires crossed then, just digging | 14:40 |
| johnthetubaguy | dansmith: ah, totally my bad, that was a bug that forced into a spec due to an API change | 14:40 |
| dansmith | okay | 14:40 |
| openstackgerrit | Alex Xu proposed openstack/nova: Return v2 version info with v2 legacy compatible wrapper https://review.openstack.org/206930 | 14:41 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Remove shared storage flag in evacuate API https://review.openstack.org/183033 | 14:42 |
| *** lpetrut has joined #openstack-nova | 14:42 | |
| mriedem | raildo: steps are here for re-proposing an approved spec http://git.openstack.org/cgit/openstack/nova-specs/tree/README.rst#n117 | 14:43 |
| dansmith | sdague: so, a question on that one | 14:43 |
| raildo | mriedem: thanks for the link :) | 14:43 |
| *** marzif__ has quit IRC | 14:43 | |
| dansmith | sdague: I'm cool with adding it in with the clean name, but I kinda wonder if we shouldn't just mirror that with the NAMESPACE:net_id as well, always set to the same thing | 14:43 |
| *** afazekas_ has quit IRC | 14:43 | |
| dansmith | sdague: just so that this isn't an accidental place where v2.0 users can't use v2.1 without an annoying change | 14:43 |
| *** abhishekk has left #openstack-nova | 14:43 | |
| *** marzif__ has joined #openstack-nova | 14:44 | |
| dansmith | sdague: like, if people haven't tripped over this because they haven't tried yet, we could avoid the trip if we just add this back in as "we forgot this, but since we're doing a microversion, we're also adding it in the clean way" | 14:44 |
| dansmith | I certainly don't want to set a precedent, but.. | 14:44 |
| *** patrickeast has joined #openstack-nova | 14:44 | |
| *** tsekiyama has joined #openstack-nova | 14:44 | |
| mriedem | jordanP: comments posted in your change | 14:44 |
| jordanP | mriedem, I saw. | 14:45 |
| mriedem | jordanP: danpb is really going to have to be on that review given the qemu-img stuff | 14:45 |
| *** kiran-r has joined #openstack-nova | 14:45 | |
| openstackgerrit | garyk proposed openstack/nova: Compute: use instance object for refresh_instance_security_rules https://review.openstack.org/187537 | 14:45 |
| *** dustins_ has joined #openstack-nova | 14:45 | |
| danpb | mriedem: what review ? | 14:45 |
| *** annegentle has joined #openstack-nova | 14:45 | |
| mriedem | danpb: https://review.openstack.org/#/c/192736/ | 14:45 |
| jordanP | Dmitry Guryanov from virtuozzo confirmed it works for him on a comment he left on Jul, 20 | 14:45 |
| gibi | dansmith: Could you please remove your -2 from https://review.openstack.org/183033 as the spec moved to specs/mitaka/approved dir? | 14:45 |
| *** MVenesio has joined #openstack-nova | 14:45 | |
| mriedem | jordanP: i'd rather see links to actual ci test logs showing it's passing with that patch | 14:46 |
| dansmith | gibi: yep | 14:46 |
| mriedem | jordanP: because then i can see the debug logging you added and see it's hitting your error handling path properly | 14:46 |
| *** thangp has joined #openstack-nova | 14:46 | |
| gibi | dansmith: thanks! | 14:46 |
| jordanP | dguryanov, ^^ | 14:46 |
| *** kurtmartin has joined #openstack-nova | 14:47 | |
| *** dustins has quit IRC | 14:47 | |
| *** hemna has joined #openstack-nova | 14:47 | |
| jordanP | could you post a link to one of your CI run on https://review.openstack.org/#/c/192736 please ? | 14:47 |
| *** xyang has joined #openstack-nova | 14:49 | |
| openstackgerrit | Raildo Mascena de Sousa Filho proposed openstack/nova-specs: Moviing Nested Quota Driver to Mitaka https://review.openstack.org/209969 | 14:51 |
| openstackgerrit | Raildo Mascena de Sousa Filho proposed openstack/nova-specs: Moving Nested Quota Driver to Mitaka https://review.openstack.org/209969 | 14:51 |
| *** gszasz has quit IRC | 14:52 | |
| *** kurtmartin has quit IRC | 14:54 | |
| *** hemna has quit IRC | 14:55 | |
| *** mriedem has quit IRC | 14:58 | |
| *** ftersin has joined #openstack-nova | 14:58 | |
| *** GeraldK has quit IRC | 14:58 | |
| *** r-daneel has quit IRC | 14:58 | |
| *** mriedem has joined #openstack-nova | 14:58 | |
| ftersin | ndipanov: mriedem: i answered you in https://review.openstack.org/#/c/208932/ | 14:58 |
| *** jkraj has quit IRC | 14:58 | |
| *** achanda has joined #openstack-nova | 14:59 | |
| *** kiran-r has quit IRC | 14:59 | |
| *** bapalm_ has joined #openstack-nova | 15:00 | |
| *** dustins_ is now known as dustins | 15:00 | |
| *** eglynn has joined #openstack-nova | 15:01 | |
| *** r-daneel has joined #openstack-nova | 15:01 | |
| *** bapalm has quit IRC | 15:01 | |
| *** hemna has joined #openstack-nova | 15:01 | |
| openstackgerrit | Marc Fouché proposed openstack/nova-specs: Inheritable admin image properties https://review.openstack.org/206431 | 15:02 |
| *** Murali has quit IRC | 15:02 | |
| mriedem | danpb: can you elaborate on why the versions of ImageMeta and ImageMetaProps have to be tied together here? https://review.openstack.org/#/c/208932/ | 15:02 |
| *** ishant has joined #openstack-nova | 15:04 | |
| *** jaypipes has joined #openstack-nova | 15:05 | |
| danpb | mriedem: that's how dansmith wanted us to handle it | 15:05 |
| dansmith | mriedem: it just makes it easier since we'll be bumping the properties one a lot | 15:06 |
| mriedem | dansmith: can you elaborate on why the versions of ImageMeta and ImageMetaProps have to be tied together here? https://review.openstack.org/#/c/208932/ :) | 15:06 |
| dansmith | danpb: also, way to throw me under the bus :) | 15:06 |
| danpb | lol | 15:06 |
| *** baoli has joined #openstack-nova | 15:07 | |
| *** Piet has quit IRC | 15:08 | |
| *** kurtmartin has joined #openstack-nova | 15:08 | |
| *** MentalRay has quit IRC | 15:08 | |
| *** burt has joined #openstack-nova | 15:08 | |
| dansmith | mriedem: let's slam this in whilst things are working: https://review.openstack.org/#/c/198904/12 | 15:08 |
| *** mtanino has joined #openstack-nova | 15:09 | |
| *** achanda has quit IRC | 15:09 | |
| mriedem | we should have a big fat comment in there that says that | 15:09 |
| mriedem | since it's a special unicorn | 15:09 |
| *** hemna has quit IRC | 15:10 | |
| *** zul has quit IRC | 15:10 | |
| dansmith | mriedem: well, it's a pattern we should probably be using more often | 15:11 |
| markus_z | johnthetubaguy: For the bug review day next week, does it make sense to add reviews to the liberty-nova-priorities-tracking etherpad? | 15:11 |
| dansmith | mriedem: the list object for a particular type might as well be tied to the base object, for example | 15:11 |
| markus_z | I've seen Ironic did this for Kilo. | 15:11 |
| openstackgerrit | Zoltan Arnold Nagy proposed openstack/nova: libvirt: add encrypted rbd volume support https://review.openstack.org/206576 | 15:11 |
| mriedem | danpb: i went through the fields in ImageMeta and compared to the image db model in glance, there are several other fields in the nova object that aren't nullable which are nullable in the glance model - was that a conscious decision when creating the object? i pointed them out in https://review.openstack.org/#/c/208932/ | 15:12 |
| jaypipes | dansmith: +W'd that patch from rlrossit | 15:12 |
| dansmith | jaypipes: thanks homey | 15:12 |
| jaypipes | no problem | 15:12 |
| rlrossit | thanks dansmith and jaypipes! | 15:12 |
| rlrossit | I feel so productive today | 15:13 |
| *** krtaylor has quit IRC | 15:13 | |
| jaypipes | hehe | 15:13 |
| jaypipes | rlrossit: don't worry. jenkins will barf on it and we'll forget about the patch for a month. that'll learn ya. | 15:14 |
| mriedem | gdi, http://status.openstack.org/elastic-recheck/index.html#1482230 is hitting me locally now too | 15:14 |
| mriedem | wtf | 15:14 |
| rlrossit | jaypipes: well seeing as that happened to my parent to this patch, it wouldn't surprise me... | 15:14 |
| *** patrickeast has quit IRC | 15:14 | |
| jaypipes | rlrossit: :) | 15:15 |
| *** diogogmt has quit IRC | 15:16 | |
| *** diogogmt has joined #openstack-nova | 15:17 | |
| *** patrickeast has joined #openstack-nova | 15:18 | |
| *** angdraug has joined #openstack-nova | 15:18 | |
| *** MentalRay has joined #openstack-nova | 15:19 | |
| *** vishwanathj has joined #openstack-nova | 15:19 | |
| dguryanov | jordanP: we used workaround in tempest | 15:19 |
| jordanP | dguryanov, how did you do then ? | 15:20 |
| dansmith | sdague: ping | 15:20 |
| *** dannywilson has joined #openstack-nova | 15:20 | |
| dguryanov | jordanP: I'll replace tempest workaround with your patch, recheck some review | 15:20 |
| sdague | dansmith: pong | 15:20 |
| jordanP | dguryanov, that would be great ! thanks | 15:20 |
| dansmith | sdague: remember how you were complaining about object version screwups? | 15:21 |
| sdague | dansmith: yep | 15:21 |
| dansmith | sdague: put your money where your mouth is: https://review.openstack.org/#/c/203642/2 | 15:21 |
| *** jgriffith has quit IRC | 15:21 | |
| jordanP | but for my info, how did you work around that ? (I am also a tempest contributor, so mayve there"s something I can fix) | 15:21 |
| dansmith | oh wait | 15:21 |
| dansmith | I forgot I had a thing on that | 15:21 |
| dansmith | hah | 15:21 |
| dansmith | damn I suck | 15:22 |
| dguryanov | jordanP: We're just deleting servers instead of stopping - https://review.openstack.org/#/c/191590/ | 15:22 |
| dansmith | sdague: thanks | 15:22 |
| rlrossit | dangit dansmith you got my hopes up | 15:23 |
| *** xyang2 has joined #openstack-nova | 15:23 | |
| dansmith | rlrossit: it's approved anyway | 15:23 |
| jordanP | dguryanov, okay :) | 15:23 |
| mriedem | scottda: where is the nova change for this force detach nova-manage command? | 15:23 |
| dguryanov | jordanP: I think it would be great to add a separate testcase to tempest, which checks only if it possible to delete snapshot of a volume if an instance is stopped | 15:23 |
| *** tonytan4ever has joined #openstack-nova | 15:23 | |
| mriedem | the spec links to the wrong blueprint | 15:23 |
| rlrossit | dansmith: does this mean I get a cheap commit? :) | 15:24 |
| mriedem | you're going to add the comma back? | 15:24 |
| openstackgerrit | sajeesh cs proposed openstack/nova-specs: Quota Management in Nested Projects https://review.openstack.org/209989 | 15:24 |
| scottda | mriedem: looking... | 15:24 |
| jordanP | dguryanov, the scenario does that.... but not only that's why that's a scenario | 15:24 |
| rlrossit | mriedem: I was kidding, but if it really is a big deal I can | 15:24 |
| mriedem | it's not | 15:25 |
| mriedem | just makes the next change a 2 line change rather than 1 | 15:25 |
| *** tjones has joined #openstack-nova | 15:25 | |
| rlrossit | all of the objects are super inconsistent with the commas so I never remember what the "original" is in those | 15:25 |
| *** jgriffith has joined #openstack-nova | 15:25 | |
| mriedem | well we should obviously have a hacking check for that | 15:25 |
| mriedem | all entries in a multi-line list/container need to have a trailing comma | 15:25 |
| *** jgriffith is now known as Guest57802 | 15:25 | |
| rlrossit | I'm pretty sure that would be a super gigantic change to abide by that check | 15:26 |
| rlrossit | I'm super curious and want to do it though | 15:26 |
| *** rajesht has quit IRC | 15:26 | |
| dansmith | rlrossit: I'm pretty consistent with mine I think | 15:26 |
| *** lucas___ has quit IRC | 15:27 | |
| lxsli | You can also have one in function calls | 15:27 |
| dansmith | mriedem: agree with that check | 15:27 |
| rlrossit | I can open up a bug and take it if you guys want | 15:27 |
| mriedem | dansmith: rlrossit: i'm joking | 15:27 |
| dansmith | I'm not | 15:27 |
| openstackgerrit | sajeesh cs proposed openstack/nova-specs: Quota Management in Nested Projects https://review.openstack.org/209989 | 15:27 |
| scottda | mriedem: Here's an out-of-date patch: https://review.openstack.org/#/c/184537/ | 15:28 |
| scottda | It still contains an API change, and that will need to be converted to using nova-manage | 15:28 |
| mriedem | yeah totally not what is in the spec | 15:28 |
| *** meghal has quit IRC | 15:28 | |
| mriedem | scottda: ok, so the nova-manage thing isn't written | 15:28 |
| mriedem | for some weird reason the bp link in the spec review doesn't work | 15:28 |
| mriedem | but looks like it's the right bp name given the link in the spec at the bottom | 15:29 |
| scottda | yes, and the spec should be abandoned and the bug used for tracking. | 15:29 |
| *** scheuran has quit IRC | 15:29 | |
| mriedem | wanghao: ^ | 15:29 |
| mriedem | wanghao: please abandon this https://review.openstack.org/#/c/84048/ | 15:29 |
| mriedem | xyang: you have to rebase your scaleio shim patch to nova | 15:30 |
| *** xgsa has joined #openstack-nova | 15:30 | |
| *** xgsa1 has quit IRC | 15:31 | |
| *** Guest55 has joined #openstack-nova | 15:32 | |
| wanghao | mriedem: sure | 15:33 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add hacking check for greenthread.spawn() and eventlet.spawn() https://review.openstack.org/195713 | 15:33 |
| rlrossit | oh boy I get to beg for some reapproval now! | 15:34 |
| *** xyang2 has quit IRC | 15:35 | |
| mriedem | looking | 15:36 |
| ftersin | mriedem: i investigated Nova code, i think we can set '' or 0 insteand None | 15:39 |
| mriedem | rlrossit: btw, your begging needs some work | 15:39 |
| rlrossit | mriedem: puh puh puh puh puuuhhhhhleeeeeeeeeeeeeeeaaaaaaase? | 15:40 |
| rlrossit | is that better? | 15:40 |
| *** kiran-r has joined #openstack-nova | 15:40 | |
| ftersin | mriedem: most fields (like name) are not used at all, because they are not returned by get_image_from_system_metadata | 15:40 |
| *** kumar has quit IRC | 15:40 | |
| mriedem | begging usually doesn't involve so much sarcasam | 15:40 |
| *** mpaolino has joined #openstack-nova | 15:41 | |
| mriedem | ftersin: ok, i haven't checked myself yet | 15:41 |
| *** alexschm has quit IRC | 15:41 | |
| ndipanov | mriedem, ftersin why do we keep those 2 versions in sync again? | 15:42 |
| *** krtaylor has joined #openstack-nova | 15:42 | |
| *** jaypipes has quit IRC | 15:43 | |
| openstackgerrit | Zachary Sais proposed openstack/nova: Move existing V2 to legacy_v2 https://review.openstack.org/193725 | 15:43 |
| mriedem | garyk: can you remove the NOTEs in here? https://review.openstack.org/#/c/209583/ | 15:43 |
| ftersin | mriedem: so what i should to do? wait for you, or update review for substitute Nones for all respective fields? | 15:43 |
| mriedem | ndipanov: see above convo with dansmith and danpb | 15:43 |
| *** meghal has joined #openstack-nova | 15:43 | |
| *** tongli has joined #openstack-nova | 15:44 | |
| mriedem | ftersin: update with what you think is appropriate for now and document why in the commit message | 15:44 |
| mriedem | ftersin: i'm also wondering why many of those other fields aren't nullable since they are nullable in the glance db model for images | 15:44 |
| mriedem | which is maybe not a question for you to answer | 15:44 |
| ndipanov | ok makes sense | 15:44 |
| ftersin | absolutely | 15:44 |
| ndipanov | then just add the None -> "" and I'm cool with that patch ftersin | 15:44 |
| garyk | mriedem: sure. i was originally asked by dims to add that but i'll drop it | 15:44 |
| xyang | mriedem: sure. Thanks | 15:48 |
| *** Guest55 has quit IRC | 15:48 | |
| *** xyang2 has joined #openstack-nova | 15:48 | |
| openstackgerrit | garyk proposed openstack/nova: Compute: replace incorrect instance object with dict https://review.openstack.org/209583 | 15:50 |
| *** jgriffith has joined #openstack-nova | 15:51 | |
| dansmith | mriedem: ftersin: ndipanov: did you see what I said about just unsetting it? | 15:54 |
| dansmith | or, hmm, is this a thing we're passing to a compute node? | 15:55 |
| ndipanov | dansmith, I didn't sorry | 15:56 |
| *** jaypipes has joined #openstack-nova | 15:56 | |
| openstackgerrit | Dmitry Guryanov proposed openstack/nova: libvirt: add vzstorage volumes suppport https://review.openstack.org/190843 | 15:58 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: ironic: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189362 | 15:59 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: objects: convert config drive to use ImageMeta object https://review.openstack.org/189323 | 15:59 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188119 | 15:59 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: make instance compulsory in blockinfo APIs https://review.openstack.org/188118 | 15:59 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: hyperv: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189302 | 15:59 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 15:59 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 15:59 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 15:59 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: xenapi: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188790 | 15:59 |
| *** yamahata has joined #openstack-nova | 15:59 | |
| *** ishant has quit IRC | 16:01 | |
| *** jlanoux has quit IRC | 16:02 | |
| *** marzif__ has quit IRC | 16:03 | |
| *** Marga_ has quit IRC | 16:03 | |
| *** belmoreira has quit IRC | 16:03 | |
| *** alex_klimov has quit IRC | 16:04 | |
| *** xgsa has quit IRC | 16:05 | |
| *** xgsa has joined #openstack-nova | 16:05 | |
| *** tjones has quit IRC | 16:05 | |
| *** hogepodge has quit IRC | 16:06 | |
| *** matrohon has quit IRC | 16:07 | |
| *** hogepodge has joined #openstack-nova | 16:07 | |
| openstackgerrit | Zoltan Arnold Nagy proposed openstack/nova: libvirt: add encrypted rbd volume support https://review.openstack.org/206576 | 16:07 |
| *** Nikolay_St has quit IRC | 16:11 | |
| *** sahid has quit IRC | 16:11 | |
| *** jistr has quit IRC | 16:12 | |
| *** yassine_ has quit IRC | 16:12 | |
| *** kiran-r has quit IRC | 16:12 | |
| *** jgriffith has left #openstack-nova | 16:13 | |
| *** echoingumesh has joined #openstack-nova | 16:13 | |
| *** danpb has quit IRC | 16:13 | |
| *** mpaolino has quit IRC | 16:14 | |
| *** kurtmartin is now known as kmartin | 16:16 | |
| *** tjones has joined #openstack-nova | 16:16 | |
| *** hshiina has quit IRC | 16:17 | |
| *** lucas___ has joined #openstack-nova | 16:18 | |
| *** SourabhP has joined #openstack-nova | 16:18 | |
| *** tjones1 has joined #openstack-nova | 16:18 | |
| *** unicell has quit IRC | 16:18 | |
| *** lucas___ has quit IRC | 16:18 | |
| *** jordanP has quit IRC | 16:18 | |
| *** unicell has joined #openstack-nova | 16:19 | |
| *** tjones has quit IRC | 16:19 | |
| *** leeantho has joined #openstack-nova | 16:20 | |
| *** ftersin has quit IRC | 16:20 | |
| *** unicell has quit IRC | 16:20 | |
| *** lucas___ has joined #openstack-nova | 16:21 | |
| *** otter768 has joined #openstack-nova | 16:21 | |
| *** vilobhmm has joined #openstack-nova | 16:21 | |
| *** SourabhP_ has joined #openstack-nova | 16:22 | |
| mriedem | garyk: looks like you messed up the rebase? | 16:23 |
| *** vilobhmm has quit IRC | 16:23 | |
| *** SourabhP has quit IRC | 16:24 | |
| *** SourabhP_ is now known as SourabhP | 16:24 | |
| *** mdrabe has quit IRC | 16:24 | |
| mriedem | garyk: on https://review.openstack.org/#/c/209583/ | 16:24 |
| *** exploreshaifali has joined #openstack-nova | 16:24 | |
| *** mdrabe has joined #openstack-nova | 16:24 | |
| *** echoingumesh has quit IRC | 16:25 | |
| openstackgerrit | Anthony Lee proposed openstack/nova: Fix live-migrations usage of the wrong connector information https://review.openstack.org/202770 | 16:25 |
| *** mdrabe has quit IRC | 16:25 | |
| *** lucas___ has quit IRC | 16:25 | |
| *** otter768 has quit IRC | 16:26 | |
| *** mdrabe has joined #openstack-nova | 16:26 | |
| *** lucas___ has joined #openstack-nova | 16:26 | |
| *** rook has quit IRC | 16:27 | |
| garyk | mriedem: ok, tahnks. i will look | 16:28 |
| *** lpetrut has quit IRC | 16:29 | |
| *** kbyrne has quit IRC | 16:29 | |
| *** wyatt88 has joined #openstack-nova | 16:29 | |
| *** nic has joined #openstack-nova | 16:30 | |
| *** SourabhP has quit IRC | 16:30 | |
| mriedem | mnestratov: were you planning on updating https://review.openstack.org/#/c/204231/ soonish? i'm keen to see that get in so we can backport to kilo | 16:32 |
| *** apuimedo has quit IRC | 16:32 | |
| mriedem | i got the cinderclient patch up that's related to the error handling | 16:32 |
| *** patrickeast has quit IRC | 16:32 | |
| *** kbyrne has joined #openstack-nova | 16:33 | |
| *** vivekd has joined #openstack-nova | 16:33 | |
| openstackgerrit | garyk proposed openstack/nova: Compute: replace incorrect instance object with dict https://review.openstack.org/209583 | 16:34 |
| sdague | mriedem: so something is up, every unit test is segfaulting now | 16:35 |
| sdague | sorry, every run | 16:35 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova-specs: RBD instance snapshots https://review.openstack.org/188244 | 16:35 |
| *** rook has joined #openstack-nova | 16:36 | |
| mriedem | sdague: yeah, i hit it locally on a full tox -r -e py27 run | 16:36 |
| sdague | oh, is it a change we've done? | 16:37 |
| mriedem | the last i combed through recent libvirt changes was prior to the nova meeting | 16:37 |
| mriedem | nothing jumped out | 16:37 |
| *** tjones1 has quit IRC | 16:37 | |
| *** ildikov has quit IRC | 16:38 | |
| sdague | can we just skip the test for now, and get danpb to take a look tomorrow? | 16:38 |
| *** rpodolyaka2 has quit IRC | 16:38 | |
| *** annegentle has quit IRC | 16:38 | |
| sdague | because, we don't really need another "can't merge any code" day | 16:38 |
| mriedem | wtf is this mock.patch.object(host.Host, "has_min_version", return_value=True) | 16:38 |
| *** jdandrea has quit IRC | 16:38 | |
| mriedem | https://review.openstack.org/#/c/128829/12/nova/tests/unit/virt/libvirt/test_driver.py | 16:39 |
| mriedem | that's obviously fucked | 16:39 |
| *** jdandrea has joined #openstack-nova | 16:39 | |
| mriedem | sorry, i'm entering full swear mode | 16:39 |
| mriedem | let me remove that turd | 16:40 |
| mnestratov | mriedem: I saw it, thanks. Won't monday be late? I'm on short vacation currently | 16:40 |
| mriedem | mnestratov: monday is probably fine at this point | 16:40 |
| mriedem | i have in-laws coming this weekend so i'll be unable to do anything but fake laugh and fake smile | 16:41 |
| *** smatzek has quit IRC | 16:41 | |
| *** penick has joined #openstack-nova | 16:41 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Use Version Independent Identity Plugins for Neutron handshake https://review.openstack.org/209599 | 16:41 |
| mnestratov | mriedem: :) | 16:41 |
| *** rdopiera has quit IRC | 16:42 | |
| *** takedakn has joined #openstack-nova | 16:42 | |
| *** takedakn has quit IRC | 16:42 | |
| *** tjones has joined #openstack-nova | 16:43 | |
| *** unicell has joined #openstack-nova | 16:43 | |
| *** tonytan4ever has quit IRC | 16:43 | |
| *** sudipto_ has joined #openstack-nova | 16:43 | |
| *** iamjarvo has joined #openstack-nova | 16:43 | |
| sdague | mriedem: oh, so it's that patch? | 16:44 |
| openstackgerrit | garyk proposed openstack/nova: Compute: use instance object for refresh_instance_security_rules https://review.openstack.org/187537 | 16:44 |
| sdague | so revert? | 16:44 |
| mriedem | sdague: i'm just looking at the things that changed test_driver in the last couple of days | 16:44 |
| mriedem | not yet | 16:44 |
| mriedem | testing locally | 16:44 |
| mriedem | wondering if someone added something that doesn't use fakelibvirt maybe | 16:45 |
| *** lpetrut has joined #openstack-nova | 16:45 | |
| mriedem | nova.virt.libvirt.(driver|host).libvirt should be set to fakelibvirt in any tests that actually call libvirt | 16:45 |
| *** moshele has quit IRC | 16:45 | |
| mriedem | i know there has been some refactoring so i look for things added that don't have that | 16:45 |
| *** lpetrut has quit IRC | 16:46 | |
| johnthetubaguy | markus_z: yeah, we could replace that older blueprint section of that tracking etherpad with bugs, if it helps I have a list of old bug patches, but my script doesn't extract the priority yet: http://5885fef486164bb8596d-41634d3e64ee11f37e8658ed1b4d12ec.r44.cf3.rackcdn.com/nova-openreviews.html | 16:46 |
| mriedem | sdague: i feel like there should be a better way to do this with a fixture or something but i'm not smart enough | 16:47 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/tests/unit/virt/libvirt/test_driver.py#L99 | 16:47 |
| sdague | yeh, I don't now. I'd have to think that one through. | 16:47 |
| *** patrickeast has joined #openstack-nova | 16:48 | |
| openstackgerrit | xing-yang proposed openstack/nova: Add os-brick based LibvirtVolumeDriver for ScaleIO https://review.openstack.org/194454 | 16:48 |
| mriedem | i figured maybe the fakelibvirt fixture could somehow stub that stuff out via module import/stub magic | 16:49 |
| sdague | it could | 16:49 |
| xyang | mriedem: I rebased it | 16:49 |
| *** ftersin has joined #openstack-nova | 16:50 | |
| mriedem | xyang: cool | 16:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: objects: add missing enum values to DiskBus field https://review.openstack.org/200280 | 16:50 |
| mriedem | cores ^ needs review, had a +2 from danpb but needed a rebase, would be good to not explode on our wrong object enums | 16:50 |
| markus_z | johnthetubaguy: I could tweak that if you give me access to the source. I'm also looking into http://status.openstack.org/reviews/ | 16:50 |
| cfriesen | in gerrit, is there a way to figure out the difference between two patch sets? Suppose someone submits patch set X, then does a minor fixup and submits patch set X+1, as a reviewer I'd like to be able to see that the minor fixup is the only difference. | 16:50 |
| *** hemna has joined #openstack-nova | 16:51 | |
| cfriesen | I see the "reference version" pulldown, but that doesn't really help when the new patch set is rebased off a new codebase. What I'd like is a way to essentially compare the diffs. | 16:52 |
| mriedem | cfriesen: see the 'patch sets' link at the top when viewing a file | 16:52 |
| mriedem | click that and select the patch set to compare to | 16:52 |
| *** patrickeast has quit IRC | 16:53 | |
| *** rpodolyaka1 has joined #openstack-nova | 16:54 | |
| *** vilobhmm has joined #openstack-nova | 16:54 | |
| markus_z | FYI: bug triage day results: https://wiki.openstack.org/wiki/Nova/BugTriage#Liberty | 16:55 |
| cfriesen | mriedem: thanks. now I feel like an idiot. :) I've actually used that but forgot about it and was looking at the top level. | 16:55 |
| *** patrickeast has joined #openstack-nova | 16:55 | |
| *** iamjarvo has quit IRC | 16:56 | |
| *** josecastroleon has quit IRC | 16:56 | |
| mriedem | np | 16:56 |
| vilobhmm | johnthetubaguy, alaski, jaypipes : meghal and I have written a spec for "Quota by X" encompasses Quota by Flavor, Quota by AZ, Quota by "Foo"….whenever you get a chance please have a look…you inputs on this will be valueable https://review.openstack.org/#/c/206160/11 | 16:57 |
| openstackgerrit | Tracy Jones proposed openstack/nova: Cleanup of compute api reboot method https://review.openstack.org/127427 | 16:57 |
| tjones | rebase hell ^^ | 16:57 |
| jaypipes | vilobhmm: I'll see what I can do. | 16:57 |
| vilobhmm | jaypipes : sure…thanks! | 16:58 |
| *** meghal has quit IRC | 16:58 | |
| *** meghal has joined #openstack-nova | 16:58 | |
| cfriesen | mreidem: so while that's helpful, it's not quite what I was looking for since it still highlights differences due to the rebase. Ideally I'd like to basically compare the diffs (to try and hide the differences due to rebase as much as possible). | 17:00 |
| mriedem | sdague: so this https://review.openstack.org/#/c/183706/ | 17:00 |
| *** htruta_ has quit IRC | 17:00 | |
| mriedem | been sitting around forever and now i apparently have to rebase again, | 17:00 |
| mriedem | but you'll see it makes sure that fakelibvirt is used in some tests that weren't setting it | 17:00 |
| *** htruta has joined #openstack-nova | 17:01 | |
| mriedem | and https://pypi.python.org/pypi/libvirt-python/1.2.18 was released on 8/4 | 17:01 |
| *** bapalm_ has quit IRC | 17:01 | |
| *** mwagner_lap has quit IRC | 17:02 | |
| *** lucas___ has quit IRC | 17:03 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: remove invalid mock from LibvirtConnTestCase https://review.openstack.org/210036 | 17:04 |
| mriedem | sdague: dansmith: ndipanov: ^ fixed that weird seg fault for me locally | 17:04 |
| *** busterswt has quit IRC | 17:04 | |
| *** lucasagomes has quit IRC | 17:06 | |
| ndipanov | mriedem, very interesting in a wtf way | 17:06 |
| mriedem | yeah, idk | 17:06 |
| dansmith | mriedem: why is it obviously wrong? | 17:06 |
| ndipanov | why is that obviously wrong though? | 17:06 |
| ndipanov | yeah exactly | 17:06 |
| mriedem | well, it's obviously useless | 17:07 |
| dansmith | because the test passes without it | 17:07 |
| dansmith | ? | 17:07 |
| mriedem | i assume that was meant to be a decorator at one point | 17:07 |
| mriedem | yes | 17:07 |
| dansmith | well, that doesn't mean it's useless | 17:07 |
| dansmith | because it's ensuring that the thing is true, if we were to change the default or something the tests would fail unnecessarily | 17:07 |
| ndipanov | well it should be taken care of by the fake driver right | 17:07 |
| dansmith | is host.Host libvirt-related? | 17:08 |
| mriedem | alright, well, i checked the last few test_driver.py changes in the last 24 hours and that jumped out at me, and it's in the same test class, and i recreated locally before and removing that and i didn't recreate | 17:08 |
| ndipanov | yeah | 17:08 |
| dansmith | like maybe you just shook the crash to tsomething else? | 17:08 |
| mriedem | so if others want to take over, go nuts | 17:08 |
| *** vilobhmm has quit IRC | 17:08 | |
| *** kumar has joined #openstack-nova | 17:08 | |
| *** garyk has quit IRC | 17:08 | |
| mriedem | i'm also rebasing https://review.openstack.org/#/c/183706/ | 17:08 |
| mriedem | so that we dno't use the libvirt-python that was released on tuesday | 17:08 |
| mriedem | but tuesday doesn't line up with what logstash is showing for when this started | 17:09 |
| cfriesen | how would removing that fix any segfault? | 17:09 |
| ndipanov | mriedem, but do we ever use libvirt-python in tests? | 17:09 |
| mriedem | ndipanov: yes | 17:09 |
| ndipanov | we do? | 17:09 |
| mriedem | if it's on the system | 17:09 |
| ndipanov | sight I thought you fixed that | 17:10 |
| mriedem | see above | 17:10 |
| mriedem | i've had a fix out for months | 17:10 |
| dansmith | cfriesen: yeah, I'm guessing maybe it just moved a thing | 17:10 |
| *** vivekd has quit IRC | 17:10 | |
| mriedem | it could also be os-testr | 17:10 |
| mriedem | https://pypi.python.org/pypi/os-testr | 17:10 |
| mriedem | https://pypi.python.org/pypi/os-testr/0.2.0 released on 6/30, there was a logstash hit for this thing on 6/29 | 17:10 |
| mriedem | *7/29 | 17:10 |
| mriedem | so nevermind | 17:10 |
| ndipanov | mriedem, I would personally prefer that change to land then random test changes... | 17:13 |
| mriedem | lifeless: are you around? | 17:14 |
| *** Kevin_Zheng_ has quit IRC | 17:15 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: WIP - convert direct db calls to objects https://review.openstack.org/194851 | 17:15 |
| *** ildikov has joined #openstack-nova | 17:15 | |
| *** VW_ has quit IRC | 17:16 | |
| *** VW_ has joined #openstack-nova | 17:16 | |
| *** iamjarvo has joined #openstack-nova | 17:18 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Honour nullability constraints of Glance schema in ImageMeta https://review.openstack.org/208932 | 17:18 |
| *** MentalRay has quit IRC | 17:19 | |
| *** lucas___ has joined #openstack-nova | 17:19 | |
| *** n0ano has quit IRC | 17:20 | |
| mriedem | "patch.object() can be used as a decorator, class decorator or a context manager." | 17:20 |
| *** VW_ has quit IRC | 17:21 | |
| mriedem | "anything else results in crazy ass seg faults LOL!" | 17:21 |
| melwitt | mriedem: I was just looking at that, what's supposed to happen if it's used as none of those things | 17:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add libvirt-python to test-requirements.txt to enforce min version https://review.openstack.org/183706 | 17:23 |
| mriedem | ndipanov: there you go ^ | 17:23 |
| mriedem | melwitt: undefined? | 17:23 |
| mriedem | melwitt: i assume it's useless at best | 17:23 |
| mriedem | i could be totally wrong | 17:23 |
| *** yamahata has quit IRC | 17:24 | |
| *** yamahata has joined #openstack-nova | 17:24 | |
| ndipanov | mriedem, although you are right that that;s not a proper way to use a patch() | 17:24 |
| melwitt | mriedem: yeah, undefined. and I thought it would just do nothing | 17:24 |
| ndipanov | if in setUp - it should use the start()/stop() no | 17:24 |
| mriedem | i haven't actually used the start stop style that oslotest has | 17:25 |
| ftersin | mriedem: ndipanov: i've updated https://review.openstack.org/#/c/208932/ does it look like what you wanted? | 17:25 |
| ndipanov | ftersin, just saw that | 17:25 |
| ndipanov | but does it actually fix the bug? | 17:25 |
| ndipanov | setting None would still fail | 17:25 |
| dansmith | mriedem: I'm confused, are you concerned about using patch.object on a class? | 17:26 |
| ftersin | i hope yes. will know by recheck of https://review.openstack.org/#/c/206955/ | 17:26 |
| mriedem | dansmith: no | 17:26 |
| ndipanov | using it in setup and never udnoing it | 17:27 |
| mriedem | dansmith: i'm concerned about it's usage as not a decorator or context manager | 17:27 |
| *** MentalRay has joined #openstack-nova | 17:27 | |
| *** lucas___ has quit IRC | 17:27 | |
| ndipanov | if it's a decorator or a context manager it's scope is known | 17:27 |
| ndipanov | here - I am not sure what it does | 17:27 |
| ndipanov | you're supposed to use start() and have a stop in tearDown me thinks | 17:27 |
| dansmith | mriedem: it returns a patcher that you have to undo when you're done | 17:27 |
| dansmith | yeah, what ndipanov said | 17:27 |
| *** iamjarvo has quit IRC | 17:28 | |
| ndipanov | so mriedem your first patch might be onto something | 17:28 |
| mriedem | i haven't looked but maybe in some previous patch set it was doing something like that, idk, but now the same thing is patched in method decorators that need it | 17:28 |
| mriedem | so it seems completely useless | 17:28 |
| *** kumar has quit IRC | 17:28 | |
| ftersin | ndipanov: i added a test for initialization from dict... but do you mean direct assignement like image_meta.disk_format = None? | 17:29 |
| mriedem | ftersin: well your bug report has a trace on NoneType | 17:29 |
| mriedem | so setting the imagemeta.checksum value to None wouldn't be good either | 17:29 |
| ndipanov | no no from dict - same like it fails in https://launchpad.net/bugs/1481220 | 17:29 |
| openstack | Launchpad bug 1481220 in OpenStack Compute (nova) "Cannot boot from volume-backed instance snapshot" [Critical,In progress] - Assigned to Feodor Tersin (ftersin) | 17:29 |
| mriedem | or container_format, whatever it was | 17:29 |
| melwitt | mriedem: yeah, the way it is there is definitely wrong/not doing anything, not mocking anything, at best | 17:29 |
| ndipanov | that would still be happening with yuour patch I think ftersin | 17:30 |
| ndipanov | no? | 17:30 |
| *** achanda has joined #openstack-nova | 17:30 | |
| ftersin | mriedem: ndipanov: there is from_dict in the trace | 17:31 |
| ndipanov | yes | 17:31 |
| ndipanov | ok let me look at the test :) | 17:31 |
| dansmith | ndipanov: see my comment on that patch | 17:31 |
| *** Nikolay_St has joined #openstack-nova | 17:31 | |
| ndipanov | hmm I didn't know about that - OK I guess it's cool | 17:32 |
| dansmith | well, | 17:32 |
| dansmith | I think we should just make from_dict super explicit and not rely on this behavior | 17:33 |
| dansmith | it will give us a place to put a verbose comment | 17:33 |
| ndipanov | yes | 17:33 |
| dansmith | and maybe even remove the defaults from the fields | 17:33 |
| dansmith | just defaulting them ourselves in that translation | 17:33 |
| dansmith | actually, yeah, that's better I think | 17:33 |
| dansmith | size(default=0) just seems bad | 17:33 |
| ndipanov | lol | 17:34 |
| *** baoli has quit IRC | 17:34 | |
| ndipanov | is_alive(default=False) | 17:34 |
| ndipanov | #depressingdefaultvaluew | 17:34 |
| *** salv-orlando has quit IRC | 17:35 | |
| ftersin | but size will be 0 anywere | 17:35 |
| *** lucas___ has joined #openstack-nova | 17:35 | |
| *** pv__ has joined #openstack-nova | 17:35 | |
| *** baoli has joined #openstack-nova | 17:35 | |
| pv__ | hey guys, im trying to figure out an error i keep getting when launching a nova instance, the logs state "ERROR nova.compute.manager [-] Instance failed network setup after 1 attempt(s)" and it keeps failing to boot from an image | 17:35 |
| ftersin | is there a difference how to set it? | 17:36 |
| pv__ | i have 2 different machines im running devstack on and the error seems to only be on one of them | 17:36 |
| pv__ | which happens to be our HBA FC enabled machine | 17:36 |
| dansmith | ftersin: yes, please do it in from_dict() | 17:36 |
| pv__ | which i need to test my cinder driver | 17:36 |
| dansmith | ftersin: it's much cleaner and more clear what is going on there, IMHO | 17:36 |
| pv__ | anyone seen this error before? | 17:36 |
| pv__ | the nova.conf(s) from both machines are exactly identical other than the machine-specific fields | 17:37 |
| *** ssurana has joined #openstack-nova | 17:38 | |
| *** nic has quit IRC | 17:38 | |
| *** VW_ has joined #openstack-nova | 17:38 | |
| *** nic has joined #openstack-nova | 17:39 | |
| *** n0ano has joined #openstack-nova | 17:39 | |
| sdague | dansmith: iptables question, in trying to debug these grenade failures where the guest used to ping, and now it doesn't | 17:39 |
| openstackgerrit | Johnson koil raj proposed openstack/nova: Provide an option to set disk.EnableUUID in vmx https://review.openstack.org/205269 | 17:40 |
| *** moshele has joined #openstack-nova | 17:40 | |
| dansmith | sdague: okay | 17:40 |
| sdague | is there a go strategy in figuring out why the ping isn't getting through. Some level of detailed logging we could run with? | 17:40 |
| dansmith | sdague: you can add log rules before drops and such | 17:41 |
| dansmith | and those show up in dmesg | 17:41 |
| dansmith | iptables -A INPUT 5 -j LOG --log-prefix='got here' | 17:41 |
| dansmith | inserts a thing at line 5 in INPUT, which will log the packet and the prefix to dmesg | 17:41 |
| dansmith | not sure that's a "good strategy" but it's a strategy | 17:41 |
| *** alexpilotti has joined #openstack-nova | 17:42 | |
| sdague | ok, we're going to have to wedge that into nova though, right? because nova dumps and restores the entire iptables space | 17:42 |
| _gryf | dansmith, hi | 17:43 |
| dansmith | sdague: oh, I guess I thought you were doing some local testing or something | 17:43 |
| *** marcusvrn has quit IRC | 17:43 | |
| dansmith | sdague: you could definitely write a patch that has a flag to put log rules into some of the chains we create | 17:43 |
| *** markvoelker has quit IRC | 17:44 | |
| dansmith | it'd be some work, but might be useful again someday | 17:44 |
| russellb | dansmith: you'll be happy to know that the new version of cascading looks more like cells v1 | 17:44 |
| russellb | sigh | 17:44 |
| _gryf | dansmith, can we somehow fix the situation when jenkins failed to merge the (patch https://review.openstack.org/#/c/209766/2) ? | 17:44 |
| dansmith | _gryf: yeah, lemme look | 17:45 |
| _gryf | dansmith, okay | 17:45 |
| sdague | dansmith: yeh, we're going to need it in the main nova code to get enough data to make it useful. So if you think it's a reasonable add, I'll work on it. | 17:45 |
| *** jcoufal has quit IRC | 17:45 | |
| *** mwagner_lap has joined #openstack-nova | 17:45 | |
| sdague | because, the whole guest randomly not responding on the network thing is something which I'd really like to have an answer on | 17:46 |
| dansmith | mriedem: hmm, is that patch from _gryf hitting some known thing? obj | 17:46 |
| dansmith | dammit | 17:46 |
| *** garyk has joined #openstack-nova | 17:46 | |
| dansmith | http://logs.openstack.org/66/209766/2/gate/gate-nova-python27/59457b3/console.html#_2015-08-06_14_24_17_827 | 17:46 |
| ndipanov | mriedem, I did a mriedem here https://review.openstack.org/#/c/202770/ | 17:46 |
| mriedem | dansmith: yeah, so you know the thing you guys have been questioning me about for a half hour? | 17:46 |
| ndipanov | but good to keep this one on your list I think since it's brick-ish related | 17:46 |
| dansmith | mriedem: oh, I thought that was a segfault | 17:46 |
| dansmith | mriedem: sorry | 17:46 |
| mriedem | look before the testr summary | 17:47 |
| dansmith | mriedem: aaaand, I see the segv | 17:47 |
| dansmith | mriedem: sorry, I suck | 17:47 |
| mriedem | http://logs.openstack.org/66/209766/2/gate/gate-nova-python27/59457b3/console.html#_2015-08-06_14_23_55_303 | 17:47 |
| dansmith | mriedem: is it worth rechecking or are things blocked now? | 17:47 |
| *** exploreshaifali has quit IRC | 17:47 | |
| *** VW__ has joined #openstack-nova | 17:47 | |
| mriedem | blocked | 17:47 |
| mriedem | fucked | 17:47 |
| mriedem | hemna: we probably want your review on this one https://review.openstack.org/#/c/202770/ | 17:48 |
| dansmith | _gryf: ^ | 17:48 |
| _gryf | dansmith, doesn't sounds good :/ | 17:48 |
| *** kumar has joined #openstack-nova | 17:48 | |
| hemna | mriedem, ok I'll check it out | 17:48 |
| sdague | mriedem: it's actually only 4 hits in your er query, I wonder if it's too narrow | 17:49 |
| dansmith | russellb: sounds like a winner to me | 17:49 |
| *** meghal has quit IRC | 17:49 | |
| mriedem | sdague: it is | 17:49 |
| mriedem | sdague: i'll update it | 17:49 |
| openstackgerrit | Jesse J. Cook proposed openstack/nova: Cells: report capped units and max builds https://review.openstack.org/195756 | 17:50 |
| *** MentalRay has quit IRC | 17:50 | |
| *** VW_ has quit IRC | 17:51 | |
| dansmith | sdague: it seems like it would probably be useful to me | 17:52 |
| hemna | mriedem, done | 17:52 |
| *** exploreshaifali has joined #openstack-nova | 17:52 | |
| sdague | dansmith: sold, I'll pop that on the near term stack | 17:53 |
| *** markus_z has quit IRC | 17:53 | |
| dansmith | sdague: you could also make the flag be something like: debug_packet_protocols=icmp, and then just insert rules that add that | 17:53 |
| dansmith | to avoid the spam you'll get in dmesg from logging tons of packets in a reachable-only-by-network machine | 17:53 |
| sdague | dansmith: sure, let me figure out a base patch, and we can hack on it from there | 17:54 |
| dansmith | cool | 17:54 |
| *** exploreshaifali has quit IRC | 17:54 | |
| *** thumpba has quit IRC | 17:54 | |
| sdague | it will be at least a tomorrow thing. I think I'm about done for the day. | 17:55 |
| *** meghal has joined #openstack-nova | 17:56 | |
| mriedem | hemna: thanks | 17:56 |
| *** zul has joined #openstack-nova | 17:56 | |
| mriedem | sdague: maybe you want to gander at this before you head out https://review.openstack.org/#/c/210036/ | 17:57 |
| *** thumpba has joined #openstack-nova | 17:57 | |
| *** patrickeast has quit IRC | 17:57 | |
| *** praneshp has joined #openstack-nova | 17:57 | |
| *** echoingumesh has joined #openstack-nova | 17:58 | |
| *** xyang2 has quit IRC | 17:59 | |
| *** kumar has quit IRC | 17:59 | |
| *** rushil_ has quit IRC | 17:59 | |
| *** VW__ has quit IRC | 17:59 | |
| *** VW_ has joined #openstack-nova | 18:00 | |
| mriedem | any ops people around? | 18:02 |
| mriedem | cburgess_: nic: med_: penick: https://review.openstack.org/#/c/207942/ | 18:03 |
| *** kmartin has quit IRC | 18:03 | |
| cburgess_ | I'm around is | 18:03 |
| penick | *looks* | 18:03 |
| mriedem | look at review you should | 18:03 |
| cburgess_ | Umm.... kind of random to just throw this into the defaults all of the sudden after 4 years. | 18:03 |
| *** bapalm has joined #openstack-nova | 18:04 | |
| penick | it shouldn’t be a problem for people that’ve already deployed | 18:04 |
| *** pixelbeat has quit IRC | 18:04 | |
| *** markvoelker has joined #openstack-nova | 18:05 | |
| mriedem | cburgess_: i was going to ask if you were already using it, but you use ceph i believe so shouldn't matter | 18:05 |
| penick | most of us have had to go and add it into our filter chain anyway | 18:05 |
| mriedem | penick: that's what i was wondering | 18:05 |
| cburgess_ | For the few customers that use local storage (we discourage this) we enable it. | 18:06 |
| nic | mriedem: we have some Ceph deployments, but not everywhere | 18:06 |
| cburgess_ | I actually wish it would work with shared storage. It would be create to have a disk filter that was shared storage aware. | 18:06 |
| cburgess_ | Well more correctly its not the disk filters fault but the host state stuff. | 18:06 |
| nic | ^ That | 18:06 |
| dansmith | cburgess_: yeah, we talked about some ways to fix that at the midcucle | 18:07 |
| cburgess_ | But... it doesn't really break anything if this is the default. | 18:07 |
| dansmith | er, midcycle even | 18:07 |
| cburgess_ | DAMN | 18:07 |
| cburgess_ | Sorry I couldn't be there | 18:07 |
| *** sudipto_ has quit IRC | 18:07 | |
| mriedem | i guess since lvm is the reference impl for cinder i'm on board with this statement "and is likely to be something most new operators will want | 18:07 |
| mriedem | enabled. We should default to include this." | 18:07 |
| cburgess_ | Because that would be a killer feature. | 18:07 |
| cburgess_ | dansmith: I gave it a +1 but only because I like you. | 18:09 |
| dansmith | cburgess_: <3 | 18:09 |
| mriedem | rlrossit: if you want something to hack on, figure out how to do this globally in the fakelibvirt fixture | 18:10 |
| mriedem | https://review.openstack.org/#/c/183706/7/nova/tests/unit/virt/libvirt/test_driver.py | 18:10 |
| mriedem | so we don't have to whack that mole | 18:10 |
| mriedem | http://git.openstack.org/cgit/openstack/nova/tree/nova/tests/unit/virt/libvirt/fakelibvirt.py#n1328 | 18:11 |
| mriedem | somewhere in there ^ | 18:11 |
| *** irenab has joined #openstack-nova | 18:12 | |
| rlrossit | mriedem: alrighty | 18:13 |
| * rlrossit looks up what a fixture is | 18:13 | |
| dansmith | looking up, you probably will see a fixture | 18:14 |
| dansmith | a light fixture | 18:14 |
| *** thumpba_ has joined #openstack-nova | 18:14 | |
| clarkb | dansmith wgat if we only have eye level lamps? | 18:14 |
| rlrossit | dansmith: I don't have lights in my office | 18:15 |
| dansmith | clarkb: see "probably" | 18:15 |
| *** eglynn has quit IRC | 18:15 | |
| rlrossit | I work in a dungeon | 18:15 |
| rlrossit | do iron bars count as a fixture? | 18:15 |
| dansmith | okay, I give up | 18:15 |
| *** thumpba has quit IRC | 18:17 | |
| *** apuimedo has joined #openstack-nova | 18:17 | |
| _gryf | dansmith, is there anything we can do with this blocked gate? | 18:18 |
| dansmith | _gryf: yes, you can go review code to help with that load :) | 18:19 |
| dansmith | _gryf: but at the moment if we're wedged, nothing is going to merge | 18:19 |
| dansmith | _gryf: but your patch is just a cleanup to your previous one, so not critical anyway, right? | 18:19 |
| _gryf | dansmith, well yeah, but otoh i put it into patchset for mark-host-down feature | 18:20 |
| _gryf | if that matters… | 18:20 |
| *** Sukhdev has joined #openstack-nova | 18:20 | |
| dansmith | _gryf: I'm not sure what you are concerned about... | 18:20 |
| _gryf | that little patch is the final one for mhd functionalty, as it was labeled as partially implements | 18:21 |
| dansmith | _gryf: it's cool, your feature has landed, you've been assigned a microversion number, there's no turning back now :) | 18:22 |
| cgoncalves | core members, could you please review https://review.openstack.org/#/c/185849/ ? it has got already a +2 from Kevin and is the only last patch on bp/mark-host-down | 18:22 |
| _gryf | dansmith, if it failed to merge, it cannot be treated as a whole bp implementation, right? | 18:22 |
| *** otter768 has joined #openstack-nova | 18:22 | |
| cgoncalves | oh, you are discussing that blueprint, cool! :-) | 18:23 |
| dansmith | _gryf: are you worried that no patch has landed that claims the bp is fully implemented? | 18:23 |
| *** josecastroleon has joined #openstack-nova | 18:23 | |
| _gryf | dansmith, to reverse the sentence: bp mark-host-down implementation is not completed withot that patch | 18:23 |
| _gryf | dansmith, yup | 18:23 |
| dansmith | _gryf: it is complete, that's just a cleanup | 18:23 |
| dansmith | _gryf: fear not :) | 18:23 |
| *** tojuvone has quit IRC | 18:24 | |
| _gryf | dansmith, oh. ok. | 18:24 |
| dansmith | _gryf: "complete" is not fully driven by tags on commits and such | 18:24 |
| _gryf | dansmith, and btw, what are the next steps for completed bp implementation? should I somehow update the launchpad? | 18:24 |
| dansmith | _gryf: we should probably wait until the client patch lands and then we're good I'd say | 18:25 |
| _gryf | okay | 18:25 |
| dansmith | _gryf: not sure if you can mark it as complete, but if not I can do it | 18:25 |
| dansmith | _gryf: but either way, it's just bookkeeping.. it's not going to change anything | 18:25 |
| _gryf | dansmith, cool | 18:25 |
| _gryf | dansmith, great :) thanks for the explanation :) | 18:26 |
| *** ijw has quit IRC | 18:26 | |
| dansmith | np :) | 18:26 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Honour nullability constraints of Glance schema in ImageMeta https://review.openstack.org/208932 | 18:26 |
| *** otter768 has quit IRC | 18:27 | |
| ftersin | dansmith: ^^^^^^^^^^^ | 18:27 |
| *** rushiagr_away is now known as rushiagr | 18:28 | |
| *** davideag_ has joined #openstack-nova | 18:29 | |
| dansmith | ftersin: looks better | 18:30 |
| dansmith | ftersin: bonus points if you fix my two nits real quick :) | 18:30 |
| dansmith | ftersin: the version comment could go right below the top comment, so leave the period if you add it there :) | 18:30 |
| *** pixelbeat has joined #openstack-nova | 18:30 | |
| ftersin | :) | 18:31 |
| ftersin | but where to put comments about nullability? at each field line? | 18:32 |
| *** baoli has quit IRC | 18:33 | |
| *** davideagnello has quit IRC | 18:33 | |
| ftersin | dansmith: and... did you read my answer on PS5? | 18:33 |
| *** tjones has quit IRC | 18:34 | |
| dansmith | ftersin: I did, I want jaypipes to read and approve that specifically :) | 18:34 |
| pv__ | i keep getting an error in logs saying 2015-08-05 14:26:25.262 2774 ERROR nova.compute.manager MessagingTimeout: Timed out waiting for a reply to message ID a6bc170b95b249a6af022ee1a491cdcb | 18:35 |
| pv__ | in the traceback for 2015-08-05 14:26:25.262 2774 ERROR nova.compute.manager [-] Instance failed network setup after 1 attempt(s) | 18:35 |
| dansmith | ftersin: left a comment about the extra comment placement | 18:35 |
| pv__ | i think it has to do with rabbitMQ but im not sure how to proceed and fix this error | 18:35 |
| pv__ | any ideas? | 18:35 |
| *** tjones has joined #openstack-nova | 18:35 | |
| dansmith | pv__: that probably means it failed to ask conductor to do something | 18:36 |
| *** irenab has quit IRC | 18:36 | |
| dansmith | pv__: check conductor logs to see if there is a crash | 18:36 |
| dansmith | make sure conductor is running, etc | 18:36 |
| pv__ | ok ill go check | 18:36 |
| dansmith | but yes, that is a failure to talk to something over rabbit, but not necessarily something wrong with your rabbit | 18:37 |
| *** mtreinish has quit IRC | 18:37 | |
| *** meghal has quit IRC | 18:37 | |
| *** salv-orlando has joined #openstack-nova | 18:37 | |
| *** baoli has joined #openstack-nova | 18:37 | |
| *** mtreinish has joined #openstack-nova | 18:39 | |
| *** gyee has joined #openstack-nova | 18:40 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Honour nullability constraints of Glance schema in ImageMeta https://review.openstack.org/208932 | 18:41 |
| dansmith | ftersin: so close :) | 18:42 |
| *** rushiagr is now known as rushiagr_away | 18:43 | |
| *** MentalRay has joined #openstack-nova | 18:43 | |
| ftersin | old blind eyes... | 18:43 |
| *** salv-orlando has quit IRC | 18:44 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Honour nullability constraints of Glance schema in ImageMeta https://review.openstack.org/208932 | 18:44 |
| *** iamjarvo has joined #openstack-nova | 18:46 | |
| *** pixelbeat has quit IRC | 18:47 | |
| *** vilobhmm has joined #openstack-nova | 18:49 | |
| *** artom has quit IRC | 18:50 | |
| *** Nikolay_St has quit IRC | 18:52 | |
| *** baoli has quit IRC | 18:53 | |
| *** josecastroleon has quit IRC | 18:53 | |
| dguryanov | jordanP: mriedem: CI logs with fix for snapshots in shutdown state - http://openstack-3rd-party-ci-logs.odin.net/42/203642/2/check/dsvm-tempest-cinder/a5a1c5d/ | 18:54 |
| *** baoli_ has joined #openstack-nova | 18:54 | |
| *** skylerberg has joined #openstack-nova | 18:55 | |
| mriedem | dguryanov: cool, can you post that in the review comments? | 18:55 |
| *** harlowja has quit IRC | 18:56 | |
| dguryanov | mriedem: OK | 18:56 |
| mriedem | dguryanov: very helpful though, once the other comments are cleaned up i think this will be good to go | 18:57 |
| dguryanov | mriedem: Thanks, I hope it will get into libery | 18:58 |
| *** lucas___ has quit IRC | 19:02 | |
| *** iamjarvo has quit IRC | 19:04 | |
| *** achanda has quit IRC | 19:05 | |
| *** achanda has joined #openstack-nova | 19:05 | |
| mriedem | dansmith: i think this is ready now https://review.openstack.org/#/c/186537/ | 19:07 |
| jaypipes | dansmith: done. lgtm | 19:08 |
| *** zsais has joined #openstack-nova | 19:08 | |
| dansmith | jaypipes: cool | 19:08 |
| dansmith | mriedem: okay, will look | 19:09 |
| dansmith | need food | 19:09 |
| *** annegentle has joined #openstack-nova | 19:09 | |
| *** tonytan4ever has joined #openstack-nova | 19:09 | |
| mriedem | i ate my feelings from this morning so now i feel terrible | 19:09 |
| *** vilobhmm1 has joined #openstack-nova | 19:09 | |
| dansmith | heh | 19:10 |
| lifeless | mriedem: hi | 19:10 |
| mriedem | lifeless: my ping was just about the same thing from -qa | 19:11 |
| *** xyang2 has joined #openstack-nova | 19:11 | |
| *** kmartin has joined #openstack-nova | 19:11 | |
| *** vilobhmm has quit IRC | 19:12 | |
| *** patrickeast has joined #openstack-nova | 19:14 | |
| *** iamjarvo has joined #openstack-nova | 19:15 | |
| *** lucas__ has joined #openstack-nova | 19:17 | |
| *** lucas__ has quit IRC | 19:18 | |
| *** lucas__ has joined #openstack-nova | 19:18 | |
| *** rook has quit IRC | 19:19 | |
| *** lucas__ has quit IRC | 19:19 | |
| *** ftersin has quit IRC | 19:19 | |
| *** lucas__ has joined #openstack-nova | 19:20 | |
| *** lucas__ has quit IRC | 19:24 | |
| lifeless | mriedem: ack | 19:25 |
| *** lucas__ has joined #openstack-nova | 19:26 | |
| *** wyatt88 is now known as jgriffith | 19:26 | |
| *** READ10 has quit IRC | 19:27 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Add functional test for server group https://review.openstack.org/147514 | 19:30 |
| openstackgerrit | Cale Rath proposed openstack/nova: Make generate_password more secure https://review.openstack.org/210092 | 19:31 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: scheduler:refactor group info in filter_properties https://review.openstack.org/148275 | 19:31 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Add functional tests to soft affinity capabilities https://review.openstack.org/147521 | 19:31 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: scheduler: add num of instance per host in a group https://review.openstack.org/148277 | 19:31 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: object:count members per hosts in InstanceGroup https://review.openstack.org/147517 | 19:31 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: scheduler: add soft-(anti-)affinity weighers https://review.openstack.org/147516 | 19:31 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: api: add soft-affinity policies for server groups https://review.openstack.org/210093 | 19:31 |
| *** baoli_ has quit IRC | 19:32 | |
| *** armax has quit IRC | 19:32 | |
| *** baoli has joined #openstack-nova | 19:34 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: api: add soft-affinity policies for server groups https://review.openstack.org/147520 | 19:34 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Add functional tests to soft affinity capabilities https://review.openstack.org/147521 | 19:34 |
| *** thumpba_ has quit IRC | 19:34 | |
| *** thumpba has joined #openstack-nova | 19:35 | |
| *** claudiub has quit IRC | 19:36 | |
| *** hemna_ has joined #openstack-nova | 19:37 | |
| *** hemna__ has joined #openstack-nova | 19:37 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: [WIP] Add functional tests to soft affinity capabilities https://review.openstack.org/147521 | 19:37 |
| *** kmARC has joined #openstack-nova | 19:37 | |
| *** claudiub has joined #openstack-nova | 19:38 | |
| *** kmARC_ has joined #openstack-nova | 19:38 | |
| *** hemna__ has quit IRC | 19:38 | |
| *** boris-42 has joined #openstack-nova | 19:39 | |
| *** fifieldt_ has joined #openstack-nova | 19:40 | |
| *** hhuang has quit IRC | 19:42 | |
| *** fifieldt has quit IRC | 19:42 | |
| *** dane-fichter has quit IRC | 19:43 | |
| *** iamjarvo has quit IRC | 19:44 | |
| *** angdraug has quit IRC | 19:46 | |
| *** hemna has quit IRC | 19:46 | |
| *** hemna_ is now known as hemna | 19:46 | |
| *** irenab has joined #openstack-nova | 19:46 | |
| *** salv-orlando has joined #openstack-nova | 19:47 | |
| *** thumpba has quit IRC | 19:48 | |
| *** thumpba has joined #openstack-nova | 19:48 | |
| *** salv-orlando has quit IRC | 19:51 | |
| *** thumpba has quit IRC | 19:53 | |
| *** thumpba has joined #openstack-nova | 19:55 | |
| *** hhuang has joined #openstack-nova | 19:55 | |
| openstackgerrit | Anthony Lee proposed openstack/nova: Fix live-migrations usage of the wrong connector information https://review.openstack.org/202770 | 19:55 |
| _gryf | melwitt, around? could you please take a look in https://review.openstack.org/#/c/185849/5 ? | 19:55 |
| *** thumpba has quit IRC | 19:56 | |
| openstackgerrit | Joel Coffman proposed openstack/nova: libvirt: Add unit tests for methods https://review.openstack.org/198880 | 19:56 |
| *** thumpba has joined #openstack-nova | 19:57 | |
| *** dprince has quit IRC | 19:58 | |
| *** tjones1 has joined #openstack-nova | 19:59 | |
| *** ssurana has quit IRC | 20:01 | |
| *** ssurana has joined #openstack-nova | 20:01 | |
| *** tjones has quit IRC | 20:02 | |
| *** ftersin has joined #openstack-nova | 20:05 | |
| *** ijw has joined #openstack-nova | 20:06 | |
| *** lucas___ has joined #openstack-nova | 20:09 | |
| *** VW_ has quit IRC | 20:09 | |
| *** mnestratov has quit IRC | 20:09 | |
| *** VW_ has joined #openstack-nova | 20:09 | |
| *** ijw_ has joined #openstack-nova | 20:10 | |
| *** xyang2 has quit IRC | 20:10 | |
| *** moshele has quit IRC | 20:10 | |
| *** lucas___ has quit IRC | 20:10 | |
| *** lucas__ has quit IRC | 20:11 | |
| *** raildo has quit IRC | 20:11 | |
| *** nic has quit IRC | 20:11 | |
| *** raildo has joined #openstack-nova | 20:11 | |
| *** achanda_ has joined #openstack-nova | 20:12 | |
| *** ijw has quit IRC | 20:12 | |
| *** ivasev has quit IRC | 20:12 | |
| *** achanda has quit IRC | 20:12 | |
| *** logan2 has quit IRC | 20:12 | |
| *** logan2 has joined #openstack-nova | 20:12 | |
| *** thumpba_ has joined #openstack-nova | 20:12 | |
| *** lucas__ has joined #openstack-nova | 20:12 | |
| *** lucas__ has quit IRC | 20:14 | |
| *** VW_ has quit IRC | 20:14 | |
| *** sulo has quit IRC | 20:15 | |
| *** iamjarvo has joined #openstack-nova | 20:16 | |
| *** thumpba has quit IRC | 20:16 | |
| *** MentalRay has quit IRC | 20:16 | |
| *** sulo has joined #openstack-nova | 20:16 | |
| *** tpeoples has quit IRC | 20:16 | |
| *** marzif has joined #openstack-nova | 20:17 | |
| *** alex_klimov has joined #openstack-nova | 20:19 | |
| *** lucas__ has joined #openstack-nova | 20:19 | |
| *** Sukhdev has quit IRC | 20:20 | |
| *** DCWilliams_VA has joined #openstack-nova | 20:21 | |
| *** Piet has joined #openstack-nova | 20:22 | |
| *** otter768 has joined #openstack-nova | 20:23 | |
| *** dims_ has joined #openstack-nova | 20:23 | |
| pv__ | still stuck with 2015-08-05 14:26:25.262 2774 ERROR nova.compute.manager [-] Instance failed network setup after 1 attempt(s) and a Message timeout that appears in its traceback in the logs | 20:23 |
| pv__ | this is for when I try an boot up an instance | 20:23 |
| pv__ | any help? | 20:24 |
| pv__ | i checked the conductor logs | 20:24 |
| pv__ | and im not seeing any errors there | 20:24 |
| *** dims_ has quit IRC | 20:24 | |
| *** dims_ has joined #openstack-nova | 20:25 | |
| *** dims has quit IRC | 20:26 | |
| *** DCWilliams_VA has quit IRC | 20:26 | |
| *** echoingumesh has quit IRC | 20:27 | |
| *** otter768 has quit IRC | 20:27 | |
| *** echoingumesh has joined #openstack-nova | 20:28 | |
| *** jgriffith has quit IRC | 20:31 | |
| *** ijw_ has quit IRC | 20:32 | |
| *** echoingumesh has quit IRC | 20:32 | |
| *** ijw has joined #openstack-nova | 20:32 | |
| *** rook has joined #openstack-nova | 20:33 | |
| *** jecarey has quit IRC | 20:34 | |
| *** tongli has quit IRC | 20:34 | |
| *** patrickeast has quit IRC | 20:35 | |
| *** penick has quit IRC | 20:38 | |
| *** tonytan4ever has quit IRC | 20:39 | |
| dansmith | pv__: can you paste a chunk of the compute log with that trace and the 20 or so lines above it? | 20:39 |
| *** patrickeast has joined #openstack-nova | 20:40 | |
| pv__ | sure thing one sec | 20:41 |
| *** rook has quit IRC | 20:41 | |
| pv__ | 2015-08-06 16:16:50.598 DEBUG oslo_service.periodic_task [req-29d2196a-6971-4f85-a802-0e719865b9ae None None] Running periodic task ComputeManager._check_instance_build_time run_periodic_tasks /usr/lib/python2.7/site-packages/oslo_service/periodic_task.py:213 2015-08-06 16:16:50.598 DEBUG oslo_service.loopingcall [req-29d2196a-6971-4f85-a802-0e719865b9ae None None] Dynamic interval looping call <bound method Service.periodic_ta | 20:44 |
| *** lucas__ has quit IRC | 20:45 | |
| pv__ | @dansmith here http://pastebin.com/0R1zXSbs | 20:45 |
| *** rook has joined #openstack-nova | 20:46 | |
| *** echoingumesh has joined #openstack-nova | 20:46 | |
| dansmith | pv__: ah, that's talking to network not conductor | 20:46 |
| dansmith | so check that network is running and/or look for errors in that log | 20:46 |
| dansmith | sorry to lead you astray before | 20:46 |
| *** claudiub has quit IRC | 20:47 | |
| *** Marga_ has joined #openstack-nova | 20:48 | |
| *** logan2 has quit IRC | 20:48 | |
| pv__ | no problem! | 20:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: ironic: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189362 | 20:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: objects: convert config drive to use ImageMeta object https://review.openstack.org/189323 | 20:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188119 | 20:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: make instance compulsory in blockinfo APIs https://review.openstack.org/188118 | 20:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: hyperv: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189302 | 20:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 20:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 20:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 20:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: xenapi: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188790 | 20:49 |
| pv__ | ill lok at the network log rn | 20:49 |
| *** bapalm has quit IRC | 20:50 | |
| pv__ | http://pastebin.com/mmHfBcLH | 20:52 |
| pv__ | thats the end of my n-net.log | 20:52 |
| *** rlrossit has left #openstack-nova | 20:52 | |
| pv__ | the traceback | 20:52 |
| *** salv-orlando has joined #openstack-nova | 20:52 | |
| *** dannywilson has quit IRC | 20:54 | |
| *** dannywilson has joined #openstack-nova | 20:55 | |
| *** nic has joined #openstack-nova | 20:55 | |
| *** mwagner_lap has quit IRC | 20:55 | |
| *** raildo has quit IRC | 20:57 | |
| *** claudiub has joined #openstack-nova | 20:58 | |
| *** ssurana has quit IRC | 20:59 | |
| *** dustins has quit IRC | 20:59 | |
| *** salv-orlando has quit IRC | 21:00 | |
| dansmith | pv__: yep, there you go :) | 21:00 |
| dansmith | that's preventing n-net from starting I think, which is your problem | 21:00 |
| dansmith | not sure we should be installing things on the lo device, so you might check your config | 21:01 |
| dansmith | if that's correct, run the command by itself to see why it's failing | 21:01 |
| *** chuckcarmack has left #openstack-nova | 21:01 | |
| *** rfolco has quit IRC | 21:02 | |
| *** vilobhmm1 has quit IRC | 21:03 | |
| *** meghal has joined #openstack-nova | 21:04 | |
| *** penick has joined #openstack-nova | 21:04 | |
| *** thangp has quit IRC | 21:05 | |
| *** lucas__ has joined #openstack-nova | 21:05 | |
| *** vilobhmm has joined #openstack-nova | 21:06 | |
| *** patrickeast has quit IRC | 21:06 | |
| *** mriedem is now known as mriedem_away | 21:07 | |
| openstackgerrit | Zoltan Arnold Nagy proposed openstack/nova: libvirt: add encrypted rbd volume support https://review.openstack.org/206576 | 21:07 |
| *** thumpba_ has quit IRC | 21:08 | |
| *** meghal has quit IRC | 21:08 | |
| *** dannywilson has quit IRC | 21:08 | |
| *** aysyd has quit IRC | 21:08 | |
| *** penick has quit IRC | 21:08 | |
| *** dannywilson has joined #openstack-nova | 21:08 | |
| openstackgerrit | Chuck Carmack proposed openstack/nova: Fix DeprecationWarning when using BaseException.message https://review.openstack.org/207560 | 21:09 |
| *** lucas__ has quit IRC | 21:10 | |
| *** patrickeast has joined #openstack-nova | 21:11 | |
| *** VW_ has joined #openstack-nova | 21:13 | |
| *** vilobhmm has quit IRC | 21:15 | |
| *** vilobhmm has joined #openstack-nova | 21:15 | |
| *** penick has joined #openstack-nova | 21:16 | |
| ctlaugh | Is there a way to configure the default volume bus type to use? | 21:17 |
| *** iamjarvo has quit IRC | 21:18 | |
| *** annegentle has quit IRC | 21:18 | |
| ctlaugh | I am running on arm64 using 'virt' machine type. Currently, the 'virtio' bus is used, but I would like to use 'scsi' (virtio-scsi). How can I configure this? | 21:18 |
| openstackgerrit | j_king proposed openstack/nova: Add wait for neutron vif unplug events on destroy https://review.openstack.org/210126 | 21:19 |
| *** lucas__ has joined #openstack-nova | 21:21 | |
| *** lucas__ has quit IRC | 21:22 | |
| *** markvoelker has quit IRC | 21:22 | |
| *** armax has joined #openstack-nova | 21:22 | |
| *** MVenesio has quit IRC | 21:24 | |
| *** ijuwang has quit IRC | 21:24 | |
| *** patrickeast has quit IRC | 21:24 | |
| *** dave-mcc_ has quit IRC | 21:26 | |
| *** meghal has joined #openstack-nova | 21:27 | |
| *** patrickeast has joined #openstack-nova | 21:27 | |
| *** lpetrut has joined #openstack-nova | 21:29 | |
| openstackgerrit | Fei Long Wang proposed openstack/nova-specs: Add spec for volume backed backup https://review.openstack.org/190919 | 21:32 |
| *** harlowja has joined #openstack-nova | 21:34 | |
| *** belmoreira has joined #openstack-nova | 21:35 | |
| openstackgerrit | j_king proposed openstack/nova: Add wait for neutron vif unplug events on destroy https://review.openstack.org/210126 | 21:36 |
| *** ftersin has quit IRC | 21:37 | |
| *** flwang1 has joined #openstack-nova | 21:38 | |
| flwang1 | johnthetubaguy: ping | 21:38 |
| *** dannywilson has quit IRC | 21:39 | |
| *** harlowja has quit IRC | 21:39 | |
| *** harlowja has joined #openstack-nova | 21:39 | |
| flwang1 | jaypipes: ping | 21:39 |
| *** angdraug has joined #openstack-nova | 21:39 | |
| openstackgerrit | j_king proposed openstack/nova: libvirt: Add wait for vif unplug events on destroy https://review.openstack.org/210126 | 21:40 |
| *** dannywilson has joined #openstack-nova | 21:40 | |
| *** lpetrut has quit IRC | 21:44 | |
| *** ijw_ has joined #openstack-nova | 21:44 | |
| *** zul has quit IRC | 21:45 | |
| *** ijw has quit IRC | 21:47 | |
| *** penick has quit IRC | 21:48 | |
| *** claudiub has quit IRC | 21:48 | |
| *** jecarey has joined #openstack-nova | 21:48 | |
| *** armax has quit IRC | 21:53 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move existing V2 to legacy_v2 https://review.openstack.org/193725 | 21:54 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move the v3 files into the main directories https://review.openstack.org/210131 | 21:54 |
| *** armax has joined #openstack-nova | 21:54 | |
| *** ijw has joined #openstack-nova | 21:54 | |
| *** zul has joined #openstack-nova | 21:55 | |
| *** thorst has quit IRC | 21:55 | |
| openstackgerrit | Pengtao Huang proposed openstack/nova: change the volue "vol" to "bdm" https://review.openstack.org/208211 | 21:56 |
| *** ijw_ has quit IRC | 21:56 | |
| *** penick has joined #openstack-nova | 21:57 | |
| *** Marga_ has quit IRC | 21:59 | |
| *** Marga__ has joined #openstack-nova | 21:59 | |
| *** salv-orlando has joined #openstack-nova | 21:59 | |
| openstackgerrit | Pengtao Huang proposed openstack/nova: change the value "all_mappings" to "block_device_mapping" https://review.openstack.org/208511 | 22:05 |
| *** cfriesen_ has joined #openstack-nova | 22:05 | |
| *** cfriesen has quit IRC | 22:06 | |
| *** lucas__ has joined #openstack-nova | 22:06 | |
| *** burt has quit IRC | 22:06 | |
| *** marzif has quit IRC | 22:07 | |
| *** salv-orlando has quit IRC | 22:08 | |
| *** lucas__ has quit IRC | 22:10 | |
| *** armax has quit IRC | 22:15 | |
| *** patrickeast has quit IRC | 22:17 | |
| *** cfriesen__ has joined #openstack-nova | 22:17 | |
| *** cfriesen_ has quit IRC | 22:17 | |
| *** ctrath has quit IRC | 22:18 | |
| *** cfriesen__ is now known as cfriesen | 22:19 | |
| *** edmondsw has quit IRC | 22:20 | |
| *** otter768 has joined #openstack-nova | 22:24 | |
| *** jecarey has quit IRC | 22:25 | |
| *** VW_ has quit IRC | 22:25 | |
| *** VW_ has joined #openstack-nova | 22:26 | |
| *** patrickeast has joined #openstack-nova | 22:27 | |
| *** otter768 has quit IRC | 22:28 | |
| *** VW_ has quit IRC | 22:30 | |
| *** IanGovett has quit IRC | 22:31 | |
| *** nic has quit IRC | 22:31 | |
| *** meghal has quit IRC | 22:32 | |
| *** vilobhmm has quit IRC | 22:33 | |
| openstackgerrit | Belmiro Moreira proposed openstack/nova: Creates flavor* tables in API database https://review.openstack.org/201606 | 22:33 |
| *** vilobhmm has joined #openstack-nova | 22:34 | |
| *** VW_ has joined #openstack-nova | 22:35 | |
| *** zul has quit IRC | 22:35 | |
| *** VW__ has joined #openstack-nova | 22:36 | |
| *** moshele has joined #openstack-nova | 22:36 | |
| *** VW_ has quit IRC | 22:39 | |
| *** meghal has joined #openstack-nova | 22:39 | |
| *** VW__ has quit IRC | 22:40 | |
| *** marzif has joined #openstack-nova | 22:42 | |
| melwitt | mriedem_away: the invalid mock patch got hit by the seg fault http://logs.openstack.org/36/210036/1/gate/gate-nova-python27/98825c9/console.html#_2015-08-06_22_04_48_362 | 22:43 |
| *** moshele has quit IRC | 22:45 | |
| *** iamjarvo has joined #openstack-nova | 22:48 | |
| *** zsais has quit IRC | 22:49 | |
| *** dims_ has quit IRC | 22:50 | |
| *** markvoelker has joined #openstack-nova | 22:50 | |
| *** alexpilotti has quit IRC | 22:52 | |
| *** Marga__ has quit IRC | 22:53 | |
| *** baoli has quit IRC | 22:54 | |
| cfriesen | alaski: I just reported a strange DB issue (bug 1482416) and I was wondering if you had any suggestions. | 22:54 |
| openstack | bug 1482416 in OpenStack Compute (nova) "bug blocks DB migration that changes column type" [Undecided,New] https://launchpad.net/bugs/1482416 | 22:54 |
| *** lpetrut has joined #openstack-nova | 22:55 | |
| *** lpetrut has left #openstack-nova | 22:55 | |
| *** alexpilotti has joined #openstack-nova | 22:58 | |
| *** marzif has quit IRC | 23:01 | |
| *** erlon has quit IRC | 23:03 | |
| *** kylek3h has quit IRC | 23:04 | |
| *** logan2 has joined #openstack-nova | 23:06 | |
| *** patrickeast has quit IRC | 23:08 | |
| *** sc68cal has quit IRC | 23:09 | |
| *** dims_ has joined #openstack-nova | 23:10 | |
| openstackgerrit | Shiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI https://review.openstack.org/202605 | 23:13 |
| *** sc68cal has joined #openstack-nova | 23:14 | |
| *** r-daneel has quit IRC | 23:17 | |
| *** atuvenie has quit IRC | 23:17 | |
| *** atuvenie has joined #openstack-nova | 23:17 | |
| *** meghal has quit IRC | 23:19 | |
| *** jgriffith has joined #openstack-nova | 23:19 | |
| *** alex_klimov has quit IRC | 23:19 | |
| *** zhenguo has joined #openstack-nova | 23:20 | |
| *** meghal has joined #openstack-nova | 23:20 | |
| *** mriedem has joined #openstack-nova | 23:21 | |
| *** kmartin has quit IRC | 23:22 | |
| *** meghal has quit IRC | 23:22 | |
| *** meghal has joined #openstack-nova | 23:22 | |
| mriedem | melwitt: well that sucks | 23:22 |
| melwitt | mriedem: yeah :( | 23:22 |
| *** mriedem_away has quit IRC | 23:23 | |
| *** edmund has quit IRC | 23:23 | |
| mriedem | melwitt: so did the change to use fakelibvirt everywhere https://review.openstack.org/#/c/183706/ | 23:23 |
| *** kurtmartin has joined #openstack-nova | 23:23 | |
| *** kurtmartin has quit IRC | 23:24 | |
| melwitt | mriedem: oof | 23:24 |
| mriedem | well, | 23:24 |
| mriedem | we could try blocking latest python-libvirt that was released this week, but if we are probably using fakelibvirt everywhere it shouldn't be a problem | 23:24 |
| mriedem | i might be missing some usage in https://review.openstack.org/#/c/183706/ since it's been around for awhile | 23:24 |
| mriedem | could be stale | 23:25 |
| *** ijw_ has joined #openstack-nova | 23:25 | |
| melwitt | hm, yeah | 23:25 |
| *** kmARC has quit IRC | 23:26 | |
| *** kmARC_ has quit IRC | 23:27 | |
| *** salv-orlando has joined #openstack-nova | 23:27 | |
| *** ijw has quit IRC | 23:28 | |
| *** echoingumesh has quit IRC | 23:28 | |
| *** echoingumesh has joined #openstack-nova | 23:29 | |
| *** tjones1 has quit IRC | 23:31 | |
| *** jgriffith has quit IRC | 23:31 | |
| *** morganfainberg is now known as morgan_503 | 23:31 | |
| *** xyang2 has joined #openstack-nova | 23:32 | |
| mriedem | well, i've got a house full of relatives tonight so i won't be able to do anything | 23:32 |
| *** xyang2 has quit IRC | 23:33 | |
| *** xyang2 has joined #openstack-nova | 23:33 | |
| *** echoingumesh has quit IRC | 23:33 | |
| jamielennox | dims_: around? | 23:34 |
| *** xyang2 has quit IRC | 23:35 | |
| *** xyang2 has joined #openstack-nova | 23:35 | |
| *** ijw_ has quit IRC | 23:36 | |
| *** xyang2 has quit IRC | 23:36 | |
| *** ijw has joined #openstack-nova | 23:37 | |
| *** VW_ has joined #openstack-nova | 23:40 | |
| *** ijw has quit IRC | 23:41 | |
| *** ijw has joined #openstack-nova | 23:42 | |
| *** pv__ has quit IRC | 23:44 | |
| *** xyang2 has joined #openstack-nova | 23:46 | |
| *** iamjarvo has quit IRC | 23:47 | |
| openstackgerrit | Vilobh Meshram proposed openstack/nova-specs: Quota Management for Dynamic Quota Resources https://review.openstack.org/206160 | 23:47 |
| *** VW_ has quit IRC | 23:49 | |
| *** Sukhdev has joined #openstack-nova | 23:53 | |
| *** penick has quit IRC | 23:53 | |
| *** dannywil_ has joined #openstack-nova | 23:54 | |
| *** pixelbeat has joined #openstack-nova | 23:56 | |
| *** kylek3h has joined #openstack-nova | 23:57 | |
| *** dannywilson has quit IRC | 23:57 | |
| *** dannywil_ has quit IRC | 23:58 | |
| *** alexpilotti has quit IRC | 23:58 | |
| *** achanda has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!