| *** Sukhdev has quit IRC | 00:00 | |
| *** sthillma has quit IRC | 00:02 | |
| *** shaohe_feng has quit IRC | 00:02 | |
| *** sthillma has joined #openstack-nova | 00:03 | |
| *** Sukhdev has joined #openstack-nova | 00:03 | |
| *** shaohe_feng has joined #openstack-nova | 00:03 | |
| *** salv-orl_ has quit IRC | 00:06 | |
| *** sthillma has quit IRC | 00:08 | |
| *** shaohe_feng has quit IRC | 00:08 | |
| *** sthillma has joined #openstack-nova | 00:09 | |
| *** rcernin has quit IRC | 00:12 | |
| *** tojuvone_ has quit IRC | 00:13 | |
| *** raunak has joined #openstack-nova | 00:13 | |
| *** BjoernT has joined #openstack-nova | 00:18 | |
| *** BjoernT has quit IRC | 00:23 | |
| *** thorst_away has joined #openstack-nova | 00:24 | |
| *** mriedem1 has joined #openstack-nova | 00:26 | |
| *** mriedem has quit IRC | 00:28 | |
| *** tojuvone_ has joined #openstack-nova | 00:29 | |
| *** takashin has joined #openstack-nova | 00:30 | |
| *** welldannit has quit IRC | 00:35 | |
| *** daemontool has quit IRC | 00:38 | |
| mriedem1 | eliqiao: just reviewed https://review.openstack.org/#/c/320897/ again, comments inline | 00:45 |
|---|---|---|
| mriedem1 | eliqiao: but it's close | 00:45 |
| mriedem1 | mnestratov: when you said the vz storage ci wasn't working with os-brick from g-r, what wasn't working? rootwrap calls? | 00:45 |
| *** mriedem1 is now known as mriedem | 00:45 | |
| *** shaohe_feng has joined #openstack-nova | 00:47 | |
| openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref: os-volumes.inc https://review.openstack.org/320048 | 00:49 |
| mriedem | mnestratov: yeah that has to be it, because g-r has os-brick capped below the version that added privsep support, so w/o that and not having pstorage-mount in compute.filters in the patch it's going to fail | 00:52 |
| mriedem | see http://lists.openstack.org/pipermail/openstack-dev/2016-June/097904.html | 00:52 |
| *** shaohe_feng has quit IRC | 00:53 | |
| *** shaohe_feng has joined #openstack-nova | 00:53 | |
| openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref: os-volumes.inc https://review.openstack.org/320048 | 00:56 |
| *** cheneydc has joined #openstack-nova | 00:58 | |
| *** dimtruck is now known as zz_dimtruck | 00:58 | |
| *** lchen_ has quit IRC | 00:58 | |
| *** Sukhdev has quit IRC | 01:01 | |
| *** diogogmt has quit IRC | 01:01 | |
| *** ayoung has joined #openstack-nova | 01:02 | |
| *** shaohe_feng has quit IRC | 01:03 | |
| *** shaohe_feng has joined #openstack-nova | 01:04 | |
| *** Sukhdev has joined #openstack-nova | 01:04 | |
| *** smatzek has joined #openstack-nova | 01:04 | |
| *** Hosam has joined #openstack-nova | 01:06 | |
| *** Hosam_ has joined #openstack-nova | 01:07 | |
| mriedem | mdbooth: how much should we not be doing this now? https://review.openstack.org/#/c/260636/ | 01:08 |
| *** salv-orlando has joined #openstack-nova | 01:08 | |
| mriedem | mdbooth: or will that be cleaned up when ploop is processed for the image backend refactor? | 01:08 |
| *** Sukhdev has quit IRC | 01:08 | |
| *** jaypipes has quit IRC | 01:09 | |
| *** cheneydc has quit IRC | 01:11 | |
| *** Hosam has quit IRC | 01:11 | |
| *** smatzek has quit IRC | 01:12 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to image metadata unit tests https://review.openstack.org/333625 | 01:13 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to images unit tests https://review.openstack.org/333626 | 01:13 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: Restrict Metadata to Lower Case Keys https://review.openstack.org/329697 | 01:13 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to server metadata unit tests https://review.openstack.org/333551 | 01:13 |
| *** shaohe_feng has quit IRC | 01:13 | |
| *** shaohe_feng has joined #openstack-nova | 01:14 | |
| *** salv-orlando has quit IRC | 01:15 | |
| *** Jeffrey4l has joined #openstack-nova | 01:16 | |
| *** Apoorva has quit IRC | 01:17 | |
| *** kevin-run has joined #openstack-nova | 01:17 | |
| *** ducttape_ has joined #openstack-nova | 01:19 | |
| *** ducttape_ has joined #openstack-nova | 01:19 | |
| *** zul has quit IRC | 01:22 | |
| *** zul_ has joined #openstack-nova | 01:22 | |
| *** flwang has quit IRC | 01:22 | |
| *** sdague has quit IRC | 01:23 | |
| *** shaohe_feng has quit IRC | 01:24 | |
| *** shaohe_feng has joined #openstack-nova | 01:24 | |
| *** diga has joined #openstack-nova | 01:25 | |
| *** Daisy has joined #openstack-nova | 01:27 | |
| *** pacoblox has joined #openstack-nova | 01:30 | |
| eliqiao | mriedem: nice, thank you. I will try to update it soon. | 01:31 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 01:31 |
| *** EinstCrazy has joined #openstack-nova | 01:31 | |
| *** diga has quit IRC | 01:32 | |
| *** EinstCrazy has quit IRC | 01:33 | |
| *** esmiurium has quit IRC | 01:34 | |
| *** shaohe_feng has quit IRC | 01:34 | |
| *** shaohe_feng has joined #openstack-nova | 01:35 | |
| *** lchen has joined #openstack-nova | 01:36 | |
| gus | sean-k-mooney: yes, that's me. | 01:38 |
| gus | sean-k-mooney: Right now, it can't. It would be easy to make it do so, I just haven't had time to type that code (and no-one else has contributed it yet). | 01:38 |
| gus | sean-k-mooney: right now, the privileged process chooses to emit a log or not based on the usual oslo.config severity options. Then everything that it chooses to emit gets actually-logged by the unprivileged code at WARNING (because I needed to pick some level). | 01:38 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/333181 | 01:38 |
| *** zz_dimtruck is now known as dimtruck | 01:39 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements https://review.openstack.org/331901 | 01:39 |
| gus | A better version of the code would emit a structured (json?) log rather than simple lines to stderr - then the unprivileged code could parse it to preserve the originally-intended severity. | 01:39 |
| *** amotoki has joined #openstack-nova | 01:39 | |
| *** tbachman_ has joined #openstack-nova | 01:39 | |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification for api-ref os-flavor-extra-specs https://review.openstack.org/333651 | 01:39 |
| *** tbachman has quit IRC | 01:40 | |
| *** tbachman_ is now known as tbachman | 01:40 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/333451 | 01:40 |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification for api-ref os-flavor-extra-specs https://review.openstack.org/333651 | 01:43 |
| *** shaohe_feng has quit IRC | 01:44 | |
| *** shaohe_feng has joined #openstack-nova | 01:45 | |
| *** tbachman_ has joined #openstack-nova | 01:45 | |
| *** EinstCrazy has joined #openstack-nova | 01:45 | |
| *** tbachman has quit IRC | 01:47 | |
| *** tbachman_ is now known as tbachman | 01:47 | |
| *** shaohe_feng has quit IRC | 01:54 | |
| *** shaohe_feng has joined #openstack-nova | 01:55 | |
| *** dave-mccowan has quit IRC | 01:56 | |
| *** dingboopt has quit IRC | 01:58 | |
| *** baoli has joined #openstack-nova | 02:01 | |
| *** shaohe_feng has quit IRC | 02:05 | |
| *** shaohe_feng has joined #openstack-nova | 02:06 | |
| *** mriedem has quit IRC | 02:09 | |
| *** woodster_ has quit IRC | 02:09 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Parameter verification for os-simple-tenant-usage.inc https://review.openstack.org/333654 | 02:09 |
| *** sthillma has quit IRC | 02:10 | |
| *** lujinluo has joined #openstack-nova | 02:11 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: libvirt: Add migration support for perf event support https://review.openstack.org/329339 | 02:13 |
| openstackgerrit | Eli Qiao proposed openstack/nova: libvirt: add perf event support when create instance https://review.openstack.org/320897 | 02:13 |
| *** shaohe_feng has quit IRC | 02:15 | |
| *** salv-orlando has joined #openstack-nova | 02:15 | |
| *** shaohe_feng has joined #openstack-nova | 02:16 | |
| *** thorst_away has quit IRC | 02:16 | |
| *** baoli has quit IRC | 02:16 | |
| *** thorst_away has joined #openstack-nova | 02:17 | |
| *** ijw has quit IRC | 02:22 | |
| *** pwnall has quit IRC | 02:24 | |
| *** yamahata has quit IRC | 02:24 | |
| *** thorst_away has quit IRC | 02:25 | |
| openstackgerrit | Yang Shengming proposed openstack/nova: [WIP]prevent resize down with ceph backend in libvirt driver https://review.openstack.org/325708 | 02:25 |
| *** shaohe_feng has quit IRC | 02:25 | |
| *** shaohe_feng has joined #openstack-nova | 02:26 | |
| *** salv-orlando has quit IRC | 02:27 | |
| *** houming has joined #openstack-nova | 02:29 | |
| *** raunak has quit IRC | 02:30 | |
| *** Hosam_ has quit IRC | 02:33 | |
| *** shaohe_feng has quit IRC | 02:35 | |
| *** shaohe_feng has joined #openstack-nova | 02:36 | |
| *** dave-mccowan has joined #openstack-nova | 02:37 | |
| *** Daisy has quit IRC | 02:39 | |
| *** Daisy has joined #openstack-nova | 02:40 | |
| *** lujinluo has quit IRC | 02:42 | |
| *** imcsk8 has joined #openstack-nova | 02:43 | |
| *** shaohe_feng has quit IRC | 02:46 | |
| *** ducttape_ has quit IRC | 02:48 | |
| *** shaohe_feng has joined #openstack-nova | 02:48 | |
| *** Daisy has quit IRC | 02:48 | |
| *** chenfan has joined #openstack-nova | 02:50 | |
| *** gyee has quit IRC | 02:51 | |
| *** M00nr41n has joined #openstack-nova | 02:53 | |
| *** jichen has joined #openstack-nova | 02:55 | |
| *** bswartz has joined #openstack-nova | 02:56 | |
| *** shaohe_feng has quit IRC | 02:56 | |
| *** shaohe_feng has joined #openstack-nova | 02:57 | |
| *** rrecio has quit IRC | 02:58 | |
| *** rrecio has joined #openstack-nova | 02:58 | |
| *** jichen_ has joined #openstack-nova | 02:58 | |
| *** chenfan has quit IRC | 02:59 | |
| *** jichen has quit IRC | 03:00 | |
| *** jichen_ is now known as jichen | 03:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/333181 | 03:01 |
| *** tbachman has quit IRC | 03:01 | |
| *** lujinluo has joined #openstack-nova | 03:01 | |
| *** raunak has joined #openstack-nova | 03:01 | |
| *** shaohe_feng has quit IRC | 03:06 | |
| *** shaohe_feng has joined #openstack-nova | 03:07 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 03:09 |
| openstackgerrit | jingtao liang proposed openstack/nova: Make print py3 compatible https://review.openstack.org/333669 | 03:09 |
| *** tbachman has joined #openstack-nova | 03:13 | |
| *** fragatina has joined #openstack-nova | 03:15 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/333181 | 03:15 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements https://review.openstack.org/331901 | 03:16 |
| *** shaohe_feng has quit IRC | 03:16 | |
| *** shaohe_feng has joined #openstack-nova | 03:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/333451 | 03:17 |
| *** thorst_away has joined #openstack-nova | 03:23 | |
| *** lujinluo has quit IRC | 03:26 | |
| *** amotoki has quit IRC | 03:26 | |
| *** shaohe_feng has quit IRC | 03:27 | |
| *** shaohe_feng has joined #openstack-nova | 03:28 | |
| *** salv-orlando has joined #openstack-nova | 03:29 | |
| *** thorst_away has quit IRC | 03:30 | |
| *** READ10 has quit IRC | 03:31 | |
| *** ducttape_ has joined #openstack-nova | 03:32 | |
| *** rcernin has joined #openstack-nova | 03:34 | |
| *** shaohe_feng has quit IRC | 03:37 | |
| *** salv-orlando has quit IRC | 03:37 | |
| *** shaohe_feng has joined #openstack-nova | 03:40 | |
| *** fragatina has quit IRC | 03:40 | |
| *** gongysh has joined #openstack-nova | 03:42 | |
| *** Dinesh_Bhor has quit IRC | 03:43 | |
| *** amotoki has joined #openstack-nova | 03:44 | |
| *** shaohe_feng has quit IRC | 03:47 | |
| *** shaohe_feng has joined #openstack-nova | 03:48 | |
| *** tjones has left #openstack-nova | 03:49 | |
| *** ducttape_ has quit IRC | 03:51 | |
| *** shaohe_feng has quit IRC | 03:57 | |
| *** shaohe_feng has joined #openstack-nova | 03:58 | |
| *** amotoki has quit IRC | 04:00 | |
| *** sdake_ has joined #openstack-nova | 04:00 | |
| *** lujinluo has joined #openstack-nova | 04:01 | |
| *** sdake has quit IRC | 04:02 | |
| *** rrecio has quit IRC | 04:03 | |
| *** amotoki has joined #openstack-nova | 04:03 | |
| *** M00nr41n has quit IRC | 04:03 | |
| *** Sukhdev has joined #openstack-nova | 04:07 | |
| *** shaohe_feng has quit IRC | 04:08 | |
| *** markvoelker has quit IRC | 04:11 | |
| *** shaohe_feng has joined #openstack-nova | 04:13 | |
| *** GB21 has joined #openstack-nova | 04:15 | |
| *** prateek has joined #openstack-nova | 04:18 | |
| *** shaohe_feng has quit IRC | 04:18 | |
| *** amotoki has quit IRC | 04:18 | |
| *** Wenzhi_ has joined #openstack-nova | 04:19 | |
| *** david-lyle has quit IRC | 04:20 | |
| *** david-lyle has joined #openstack-nova | 04:20 | |
| *** shaohe_feng has joined #openstack-nova | 04:21 | |
| *** sdake_ has quit IRC | 04:22 | |
| *** chenfei has quit IRC | 04:22 | |
| *** Qiming has quit IRC | 04:22 | |
| *** Wenzhi has quit IRC | 04:23 | |
| *** chenfei has joined #openstack-nova | 04:23 | |
| *** Qiming has joined #openstack-nova | 04:23 | |
| *** thorst_away has joined #openstack-nova | 04:28 | |
| *** shaohe_feng has quit IRC | 04:28 | |
| *** baoli has joined #openstack-nova | 04:28 | |
| *** shaohe_feng has joined #openstack-nova | 04:31 | |
| *** lpetrut has joined #openstack-nova | 04:32 | |
| *** baoli has quit IRC | 04:33 | |
| *** diga has joined #openstack-nova | 04:33 | |
| *** thorst_away has quit IRC | 04:35 | |
| *** Daisy has joined #openstack-nova | 04:36 | |
| *** links has joined #openstack-nova | 04:37 | |
| *** sdake has joined #openstack-nova | 04:38 | |
| *** amotoki has joined #openstack-nova | 04:38 | |
| *** shaohe_feng has quit IRC | 04:38 | |
| *** salv-orlando has joined #openstack-nova | 04:39 | |
| *** shaohe_feng has joined #openstack-nova | 04:39 | |
| *** EinstCrazy has quit IRC | 04:40 | |
| *** Daisy has quit IRC | 04:41 | |
| *** prateek has quit IRC | 04:42 | |
| *** _Fdaisuke_ has quit IRC | 04:43 | |
| *** EinstCrazy has joined #openstack-nova | 04:47 | |
| *** shaohe_feng has quit IRC | 04:49 | |
| *** shaohe_feng has joined #openstack-nova | 04:49 | |
| *** links has quit IRC | 04:50 | |
| *** links has joined #openstack-nova | 04:50 | |
| *** dave-mccowan has quit IRC | 04:50 | |
| *** salv-orlando has quit IRC | 04:51 | |
| *** ratailor has joined #openstack-nova | 04:52 | |
| *** lpetrut has quit IRC | 04:55 | |
| *** bkopilov has quit IRC | 04:58 | |
| *** shaohe_feng has quit IRC | 04:59 | |
| *** shaohe_feng has joined #openstack-nova | 05:00 | |
| *** M00nr41n has joined #openstack-nova | 05:04 | |
| *** houming has quit IRC | 05:05 | |
| *** GB21 has quit IRC | 05:05 | |
| *** Daisy has joined #openstack-nova | 05:06 | |
| *** Manikanta has quit IRC | 05:07 | |
| *** shaohe_feng has quit IRC | 05:09 | |
| *** shaohe_feng has joined #openstack-nova | 05:09 | |
| *** markvoelker has joined #openstack-nova | 05:11 | |
| *** salv-orlando has joined #openstack-nova | 05:14 | |
| *** markvoelker has quit IRC | 05:16 | |
| *** nkrinner_afk is now known as nkrinner | 05:17 | |
| *** Daisy has quit IRC | 05:18 | |
| *** Daisy has joined #openstack-nova | 05:18 | |
| openstackgerrit | jichenjc proposed openstack/nova: make purge_option to be default False https://review.openstack.org/332777 | 05:19 |
| *** shaohe_feng has quit IRC | 05:19 | |
| *** ducttape_ has joined #openstack-nova | 05:22 | |
| *** shaohe_feng has joined #openstack-nova | 05:22 | |
| *** Daisy has quit IRC | 05:23 | |
| *** lpetrut has joined #openstack-nova | 05:23 | |
| *** prateek has joined #openstack-nova | 05:25 | |
| *** links has quit IRC | 05:25 | |
| *** ducttape_ has quit IRC | 05:27 | |
| *** zerda2 has joined #openstack-nova | 05:27 | |
| *** lpetrut has quit IRC | 05:28 | |
| *** shaohe_feng has quit IRC | 05:30 | |
| *** shaohe_feng has joined #openstack-nova | 05:31 | |
| *** sdake has quit IRC | 05:32 | |
| *** zhangjn has quit IRC | 05:33 | |
| *** zhangjn has joined #openstack-nova | 05:33 | |
| *** thorst_away has joined #openstack-nova | 05:33 | |
| *** sdake has joined #openstack-nova | 05:34 | |
| *** takashin has left #openstack-nova | 05:35 | |
| *** rcernin has quit IRC | 05:35 | |
| *** links has joined #openstack-nova | 05:39 | |
| *** Sukhdev has quit IRC | 05:40 | |
| *** thorst_away has quit IRC | 05:40 | |
| *** shaohe_feng has quit IRC | 05:40 | |
| *** shaohe_feng has joined #openstack-nova | 05:41 | |
| *** houming has joined #openstack-nova | 05:43 | |
| *** sacharya has quit IRC | 05:43 | |
| *** sheel has quit IRC | 05:45 | |
| *** chandankumar has joined #openstack-nova | 05:47 | |
| *** dimtruck is now known as zz_dimtruck | 05:48 | |
| *** diga has quit IRC | 05:49 | |
| *** houming has quit IRC | 05:49 | |
| *** shaohe_feng has quit IRC | 05:50 | |
| *** pbandark has joined #openstack-nova | 05:50 | |
| *** shaohe_feng has joined #openstack-nova | 05:51 | |
| *** josecastroleon has joined #openstack-nova | 06:00 | |
| *** mpavone has joined #openstack-nova | 06:00 | |
| *** shaohe_feng has quit IRC | 06:00 | |
| *** diga has joined #openstack-nova | 06:01 | |
| *** shaohe_feng has joined #openstack-nova | 06:01 | |
| *** Daisy has joined #openstack-nova | 06:01 | |
| *** EinstCrazy has quit IRC | 06:02 | |
| *** GB21 has joined #openstack-nova | 06:04 | |
| *** prateek has quit IRC | 06:04 | |
| *** links has quit IRC | 06:05 | |
| *** Daisy has quit IRC | 06:06 | |
| *** raunak has quit IRC | 06:08 | |
| *** shaohe_feng has quit IRC | 06:11 | |
| *** lpetrut has joined #openstack-nova | 06:11 | |
| *** rcernin has joined #openstack-nova | 06:11 | |
| *** GB21 has quit IRC | 06:12 | |
| *** markvoelker has joined #openstack-nova | 06:13 | |
| *** sudipto has joined #openstack-nova | 06:13 | |
| *** shaohe_feng has joined #openstack-nova | 06:14 | |
| *** EinstCrazy has joined #openstack-nova | 06:16 | |
| *** markvoelker has quit IRC | 06:17 | |
| *** links has joined #openstack-nova | 06:18 | |
| *** prateek has joined #openstack-nova | 06:19 | |
| *** shaohe_feng has quit IRC | 06:21 | |
| *** shaohe_feng has joined #openstack-nova | 06:22 | |
| *** ducttape_ has joined #openstack-nova | 06:23 | |
| *** liverpooler has quit IRC | 06:24 | |
| *** ducttape_ has quit IRC | 06:28 | |
| *** salv-orlando has quit IRC | 06:28 | |
| *** Daisy has joined #openstack-nova | 06:30 | |
| *** goutham has joined #openstack-nova | 06:30 | |
| openstackgerrit | Tiantian Gao proposed openstack/nova-specs: Hide Exclusive Availability zones https://review.openstack.org/333764 | 06:31 |
| *** shaohe_feng has quit IRC | 06:31 | |
| *** shaohe_feng has joined #openstack-nova | 06:32 | |
| *** allen_gao has joined #openstack-nova | 06:33 | |
| openstackgerrit | Zhengguang Ou proposed openstack/nova: libvirt: fix detach volume race condition https://review.openstack.org/290510 | 06:34 |
| *** andreas_s has joined #openstack-nova | 06:35 | |
| *** links has quit IRC | 06:37 | |
| *** thorst_away has joined #openstack-nova | 06:38 | |
| *** hoangcx has joined #openstack-nova | 06:41 | |
| *** shaohe_feng has quit IRC | 06:41 | |
| *** shaohe_feng has joined #openstack-nova | 06:42 | |
| *** hoangcx has quit IRC | 06:42 | |
| *** sahid has joined #openstack-nova | 06:43 | |
| *** GB21 has joined #openstack-nova | 06:43 | |
| *** ssurana has quit IRC | 06:44 | |
| *** mpavone has quit IRC | 06:45 | |
| *** thorst_away has quit IRC | 06:45 | |
| *** vtech has quit IRC | 06:45 | |
| *** links has joined #openstack-nova | 06:49 | |
| *** edand has joined #openstack-nova | 06:52 | |
| *** shaohe_feng has quit IRC | 06:52 | |
| *** prateek has quit IRC | 06:52 | |
| *** fragatina has joined #openstack-nova | 06:54 | |
| *** shaohe_feng has joined #openstack-nova | 06:54 | |
| *** lpetrut has quit IRC | 06:55 | |
| *** tesseract- has joined #openstack-nova | 06:57 | |
| *** njirap has joined #openstack-nova | 07:00 | |
| *** shaohe_feng has quit IRC | 07:02 | |
| *** shaohe_feng has joined #openstack-nova | 07:03 | |
| *** pkoniszewski has quit IRC | 07:03 | |
| openstackgerrit | John Hua proposed openstack/nova: POC: WIP: XenAPI: device tagging https://review.openstack.org/333781 | 07:04 |
| *** prateek has joined #openstack-nova | 07:04 | |
| openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [2/3] https://review.openstack.org/331632 | 07:05 |
| openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [3/3] https://review.openstack.org/331633 | 07:05 |
| openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [1/3] https://review.openstack.org/280301 | 07:05 |
| openstackgerrit | John Hua proposed openstack/nova: POC: WIP: XenAPI: device tagging https://review.openstack.org/333781 | 07:08 |
| *** vtech has joined #openstack-nova | 07:08 | |
| *** matrohon has joined #openstack-nova | 07:10 | |
| openstackgerrit | jichenjc proposed openstack/nova: api-ref: add note about os-certificates API https://review.openstack.org/333538 | 07:11 |
| *** sdake_ has joined #openstack-nova | 07:12 | |
| *** shaohe_feng has quit IRC | 07:12 | |
| *** pkoniszewski has joined #openstack-nova | 07:12 | |
| *** shaohe_feng has joined #openstack-nova | 07:13 | |
| *** markvoelker has joined #openstack-nova | 07:13 | |
| openstackgerrit | Tiantian Gao proposed openstack/nova-specs: Hide Exclusive Availability zones https://review.openstack.org/333764 | 07:14 |
| *** sdake has quit IRC | 07:15 | |
| *** EinstCrazy has quit IRC | 07:17 | |
| *** markvoelker has quit IRC | 07:19 | |
| *** EinstCrazy has joined #openstack-nova | 07:20 | |
| *** markus_z has joined #openstack-nova | 07:21 | |
| *** shaohe_feng has quit IRC | 07:22 | |
| *** shaohe_feng has joined #openstack-nova | 07:23 | |
| *** ducttape_ has joined #openstack-nova | 07:24 | |
| *** EinstCrazy has quit IRC | 07:27 | |
| *** EinstCrazy has joined #openstack-nova | 07:28 | |
| *** ihrachys has joined #openstack-nova | 07:28 | |
| *** EinstCrazy has quit IRC | 07:28 | |
| *** EinstCrazy has joined #openstack-nova | 07:29 | |
| *** EinstCrazy has quit IRC | 07:29 | |
| *** ducttape_ has quit IRC | 07:30 | |
| *** sridharg has joined #openstack-nova | 07:30 | |
| *** jlanoux has joined #openstack-nova | 07:31 | |
| *** shaohe_feng has quit IRC | 07:33 | |
| *** fragatina has quit IRC | 07:33 | |
| *** edand has quit IRC | 07:34 | |
| *** shaohe_feng has joined #openstack-nova | 07:34 | |
| *** danpb has joined #openstack-nova | 07:34 | |
| *** yamahata has joined #openstack-nova | 07:34 | |
| *** salv-orlando has joined #openstack-nova | 07:36 | |
| *** EinstCra_ has joined #openstack-nova | 07:38 | |
| *** vtech_ has joined #openstack-nova | 07:39 | |
| *** vtech has quit IRC | 07:39 | |
| *** EinstCrazy has joined #openstack-nova | 07:40 | |
| *** jlanoux_ has joined #openstack-nova | 07:41 | |
| *** Daisy has quit IRC | 07:41 | |
| *** Daisy has joined #openstack-nova | 07:41 | |
| *** zenoway has joined #openstack-nova | 07:43 | |
| *** thorst_away has joined #openstack-nova | 07:43 | |
| *** shaohe_feng has quit IRC | 07:43 | |
| *** EinstCra_ has quit IRC | 07:43 | |
| *** shaohe_feng has joined #openstack-nova | 07:43 | |
| *** EinstCra_ has joined #openstack-nova | 07:43 | |
| *** EinstCrazy has quit IRC | 07:44 | |
| *** jlanoux has quit IRC | 07:44 | |
| *** josecastroleon has quit IRC | 07:45 | |
| openstackgerrit | YaoZheng proposed openstack/python-novaclient: make string.letters python3 compatible https://review.openstack.org/333795 | 07:45 |
| *** mpavone has joined #openstack-nova | 07:46 | |
| openstackgerrit | Tiantian Gao proposed openstack/nova-specs: Hide Exclusive Availability zones https://review.openstack.org/333764 | 07:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/331996 | 07:47 |
| *** josecastroleon has joined #openstack-nova | 07:48 | |
| *** EinstCrazy has joined #openstack-nova | 07:49 | |
| *** ihrachys has quit IRC | 07:50 | |
| *** EinstCra_ has quit IRC | 07:50 | |
| *** thorst_away has quit IRC | 07:50 | |
| *** kuzko has quit IRC | 07:53 | |
| *** shaohe_feng has quit IRC | 07:53 | |
| *** shaohe_feng has joined #openstack-nova | 07:54 | |
| *** EinstCra_ has joined #openstack-nova | 07:54 | |
| *** EinstCrazy has quit IRC | 07:54 | |
| *** kuzko has joined #openstack-nova | 07:55 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: api-ref: verify images.inc https://review.openstack.org/326871 | 07:56 |
| *** mpavone has quit IRC | 07:57 | |
| *** rcernin has quit IRC | 07:59 | |
| *** sameo has joined #openstack-nova | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** Daisy_ has joined #openstack-nova | 08:00 | |
| *** sdake_ has quit IRC | 08:02 | |
| *** lucas-afk is now known as lucasagomes | 08:02 | |
| openstackgerrit | jingtao liang proposed openstack/nova: Make print py3 compatible https://review.openstack.org/333669 | 08:03 |
| *** shaohe_feng has quit IRC | 08:03 | |
| *** shaohe_feng has joined #openstack-nova | 08:04 | |
| *** Daisy has quit IRC | 08:04 | |
| *** EinstCra_ has quit IRC | 08:04 | |
| *** kuzko has quit IRC | 08:07 | |
| *** kuzko has joined #openstack-nova | 08:09 | |
| *** zenoway has quit IRC | 08:09 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
| *** kuzko has quit IRC | 08:10 | |
| *** kuzko has joined #openstack-nova | 08:10 | |
| *** gongysh_ has joined #openstack-nova | 08:10 | |
| *** EinstCrazy has joined #openstack-nova | 08:12 | |
| *** sfinucan has joined #openstack-nova | 08:12 | |
| *** zenoway has joined #openstack-nova | 08:14 | |
| *** shaohe_feng has quit IRC | 08:14 | |
| *** shaohe_feng has joined #openstack-nova | 08:14 | |
| *** dmk0202 has joined #openstack-nova | 08:17 | |
| *** daemontool has joined #openstack-nova | 08:17 | |
| *** andrearosa is now known as andrearosa_sad | 08:17 | |
| *** zenoway has quit IRC | 08:18 | |
| openstackgerrit | Bin Zhou proposed openstack/nova: StandardError raises NameError in Python3 https://review.openstack.org/333802 | 08:18 |
| *** mingyu has joined #openstack-nova | 08:20 | |
| *** paul-carlton2 has joined #openstack-nova | 08:23 | |
| *** shaohe_feng has quit IRC | 08:24 | |
| *** shaohe_feng has joined #openstack-nova | 08:25 | |
| *** gszasz has joined #openstack-nova | 08:25 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add DeviceTaggingFilter to scheduler https://review.openstack.org/331490 | 08:31 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: attach configdrive after instance XML https://review.openstack.org/303335 | 08:31 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: Add devices_metadata to instance object https://review.openstack.org/327920 | 08:31 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: db: add a device_metadata column to instance_extra https://review.openstack.org/318883 | 08:31 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 08:31 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: new InstanceDeviceMetadata object https://review.openstack.org/318884 | 08:31 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 08:31 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Compute manager device tagging support https://review.openstack.org/264017 | 08:31 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Libvirt driver implementation of device tagging https://review.openstack.org/264016 | 08:32 |
| *** shaohe_feng has quit IRC | 08:34 | |
| *** shaohe_feng has joined #openstack-nova | 08:35 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add DeviceTaggingFilter to scheduler https://review.openstack.org/331490 | 08:35 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: attach configdrive after instance XML https://review.openstack.org/303335 | 08:35 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 08:35 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 08:36 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Compute manager device tagging support https://review.openstack.org/264017 | 08:36 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Libvirt driver implementation of device tagging https://review.openstack.org/264016 | 08:36 |
| *** matrohon has quit IRC | 08:38 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Rename the import of nova.virt.disk.api from disk to disk_api https://review.openstack.org/333521 | 08:39 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Pass path to Image base class https://review.openstack.org/333271 | 08:39 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Add Backend from_image and from_func https://review.openstack.org/333522 | 08:39 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Rename Backend snapshot and image to by_libvirt_path and by_name https://review.openstack.org/333272 | 08:39 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Pass is_block_dev explicitly from create_from_func https://review.openstack.org/333263 | 08:39 |
| *** vladikr has quit IRC | 08:42 | |
| *** zenoway has joined #openstack-nova | 08:43 | |
| *** shaohe_feng has quit IRC | 08:45 | |
| *** shaohe_feng has joined #openstack-nova | 08:45 | |
| *** GB21 has quit IRC | 08:45 | |
| *** BobBall_AWOL is now known as BobBall | 08:46 | |
| *** derekh has joined #openstack-nova | 08:46 | |
| *** BobBall is now known as Guest74267 | 08:46 | |
| *** gongysh has quit IRC | 08:47 | |
| *** thorst_away has joined #openstack-nova | 08:48 | |
| *** gongysh_ has quit IRC | 08:49 | |
| *** gongysh_ has joined #openstack-nova | 08:52 | |
| *** thorst_away has quit IRC | 08:55 | |
| *** shaohe_feng has quit IRC | 08:55 | |
| *** shaohe_feng has joined #openstack-nova | 08:55 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Deprecated networks related APIs https://review.openstack.org/333151 | 08:56 |
| openstackgerrit | Alex Xu proposed openstack/nova: Deprecate FixedIPs API https://review.openstack.org/333152 | 08:56 |
| openstackgerrit | Alex Xu proposed openstack/nova: Deprecated floating ip related proxy APIs https://review.openstack.org/333821 | 08:56 |
| openstackgerrit | Alex Xu proposed openstack/nova: Deprecate SecurityGroup related proxy API https://review.openstack.org/333822 | 08:56 |
| openstackgerrit | Alex Xu proposed openstack/nova: Deprecate volumes related APIs https://review.openstack.org/333823 | 08:56 |
| openstackgerrit | Alex Xu proposed openstack/nova: Deprecate Baremetal and fping API https://review.openstack.org/333824 | 08:56 |
| openstackgerrit | Alex Xu proposed openstack/nova: Filter network related quotas out of quotas API in 2.31 https://review.openstack.org/333825 | 08:56 |
| *** zenoway has quit IRC | 08:57 | |
| *** alexpilotti has joined #openstack-nova | 08:59 | |
| andrearosa_sad | gmann_: I pinged you yesterday but I managed to get the answers I was looking for asking in the #openstack-qa channel. Thanks | 09:02 |
| *** mingyu has quit IRC | 09:02 | |
| *** mingyu has joined #openstack-nova | 09:03 | |
| *** sean-k-mooney_aw is now known as sean-k-mooney | 09:03 | |
| *** Daisy_ has quit IRC | 09:04 | |
| *** Daisy has joined #openstack-nova | 09:04 | |
| *** shaohe_feng has quit IRC | 09:05 | |
| *** zenoway has joined #openstack-nova | 09:06 | |
| *** lujinluo has quit IRC | 09:08 | |
| *** kevin-run has quit IRC | 09:08 | |
| *** bkopilov has joined #openstack-nova | 09:08 | |
| *** shaohe_feng has joined #openstack-nova | 09:08 | |
| *** luis5tb has joined #openstack-nova | 09:08 | |
| *** Daisy has quit IRC | 09:09 | |
| lyarwood | just asked this in #openstack-vmware but on the off chance that someone knows in here... | 09:09 |
| lyarwood | I'm trying to get nova-vc6-dsvm-tempest to run against https://review.openstack.org/#/c/318266/ but the documented recheck command doesn't appear to be working | 09:09 |
| lyarwood | can anyone confirm if this is queued or if there's another command that we should be using? | 09:09 |
| openstackgerrit | Zhengguang Ou proposed openstack/nova: libvirt: fix detach volume race condition https://review.openstack.org/290510 | 09:11 |
| *** kevin_zh has joined #openstack-nova | 09:13 | |
| *** shaohe_feng has quit IRC | 09:15 | |
| *** 7GHABEJ24 has joined #openstack-nova | 09:16 | |
| *** shaohe_feng has joined #openstack-nova | 09:16 | |
| *** markvoelker has joined #openstack-nova | 09:16 | |
| *** 7GHABEJ24 has quit IRC | 09:16 | |
| *** fragatina has joined #openstack-nova | 09:17 | |
| *** lpetrut has joined #openstack-nova | 09:18 | |
| *** sfinucan has quit IRC | 09:19 | |
| *** sfinucan has joined #openstack-nova | 09:19 | |
| *** sfinucan has quit IRC | 09:20 | |
| *** markvoelker has quit IRC | 09:20 | |
| *** aarefiev has quit IRC | 09:21 | |
| *** sfinucan has joined #openstack-nova | 09:21 | |
| openstackgerrit | Gergely Csatari proposed openstack/nova: WIP api-ref: Checking the parameters of servers-actions.inc https://review.openstack.org/327112 | 09:22 |
| *** amotoki has quit IRC | 09:22 | |
| *** amotoki has joined #openstack-nova | 09:23 | |
| *** sfinucan_ has joined #openstack-nova | 09:23 | |
| *** jichen has quit IRC | 09:23 | |
| *** sfinucan_ has quit IRC | 09:23 | |
| *** kevin-run has joined #openstack-nova | 09:25 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 09:25 |
| *** shaohe_feng has quit IRC | 09:26 | |
| *** shaohe_feng has joined #openstack-nova | 09:26 | |
| *** ducttape_ has joined #openstack-nova | 09:27 | |
| openstackgerrit | YaoZheng proposed openstack/python-novaclient: make string.letters python3 compatible https://review.openstack.org/333795 | 09:27 |
| *** rhagarty__ has joined #openstack-nova | 09:28 | |
| *** sfinucan has quit IRC | 09:30 | |
| *** rhagarty_ has quit IRC | 09:31 | |
| *** ducttape_ has quit IRC | 09:31 | |
| *** sfinucan has joined #openstack-nova | 09:34 | |
| *** amotoki_ has joined #openstack-nova | 09:34 | |
| *** amotoki has quit IRC | 09:35 | |
| *** shaohe_feng has quit IRC | 09:36 | |
| *** shaohe_feng has joined #openstack-nova | 09:36 | |
| *** kevin-run has quit IRC | 09:38 | |
| *** kevin-run has joined #openstack-nova | 09:38 | |
| *** sambetts|afk is now known as sambetts | 09:39 | |
| pkholkin | dansmith: Hi Dan, are you here? I have a quick question about objects | 09:40 |
| *** kevin_zh has quit IRC | 09:40 | |
| *** kevin_zh has joined #openstack-nova | 09:40 | |
| *** PaulMurray has joined #openstack-nova | 09:41 | |
| bauzas | pkholkin: he's in the last TZ you can have, ie. PST | 09:41 |
| bauzas | maybe I could help you ? | 09:41 |
| pkholkin | bauzas: ok, please look this comment https://review.openstack.org/#/c/284868/9/nova/tests/unit/objects/test_objects.py | 09:42 |
| *** fawadkhaliq has joined #openstack-nova | 09:42 | |
| *** fawadkhaliq has quit IRC | 09:42 | |
| PaulMurray | Anyone know what the recheck comment is for XenProject CI ? | 09:42 |
| bauzas | PaulMurray: xen: recheck | 09:42 |
| bauzas | fancy | 09:42 |
| pkholkin | I've added two parameters limit=None,marker=None to the object | 09:42 |
| PaulMurray | (not citrix xenserver) | 09:42 |
| pkholkin | bauzas: should we bump new version? | 09:42 |
| bauzas | PaulMurray: yup, xen:recheck != xenserver:recheck | 09:42 |
| PaulMurray | thanks bauzas | 09:43 |
| bauzas | PaulMurray: np | 09:43 |
| bauzas | pkholkin: /me looking | 09:43 |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: Implementation of Libvirt Storage Pools https://review.openstack.org/314134 | 09:44 |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: Libvirt Storage Pools unit tests fixes https://review.openstack.org/333836 | 09:44 |
| *** shaohe_feng has quit IRC | 09:46 | |
| bauzas | pkholkin: okay, I looked at the change, and indeed you need to bump a new microversion | 09:46 |
| bauzas | because it's a remotable method | 09:47 |
| *** shaohe_feng has joined #openstack-nova | 09:47 | |
| pkholkin | bauzas: ok, and should I so separate method or I can leave it as it is with new two parameters | 09:47 |
| pkholkin | ? | 09:47 |
| pkholkin | should I do * | 09:47 |
| bauzas | pkholkin: tbc, the RPC client will call that method, but given it's a RPC call, it will go to the RPC server | 09:47 |
| bauzas | so the server will look at the asked version and if it can't support it, will use the one it knows | 09:48 |
| bauzas | pkoniszewski: here, the point is that you modify a remotable method signature | 09:48 |
| *** syfy323 has quit IRC | 09:50 | |
| *** kevin_zh has quit IRC | 09:50 | |
| *** kevin_zh has joined #openstack-nova | 09:50 | |
| pkholkin | bauzas: so, can I change 'key_pair_get_all_by_user' signature or I should create separate method? | 09:52 |
| lyarwood | danpb: https://review.openstack.org/#/c/270289/ should be good to +2 again if you're happy with the releasenote I added. | 09:52 |
| *** thorst_away has joined #openstack-nova | 09:52 | |
| bauzas | pkholkin: nope, AFAICT, you just need to bump the version, that's it | 09:54 |
| *** markvoelker has joined #openstack-nova | 09:54 | |
| lyarwood | ^ also I would obviously appreciate additional core reviews for the series - https://review.openstack.org/#/q/status:open+topic:bp/virt-rescue-stable-disk-devices | 09:54 |
| pkholkin | bauzas: I also think so, thank you ! | 09:54 |
| bauzas | pkholkin: given you already provided default nullable arguments | 09:54 |
| pkholkin | yes | 09:54 |
| *** shaohe_feng has quit IRC | 09:56 | |
| *** shaohe_feng has joined #openstack-nova | 09:57 | |
| goutham | how to start writing a new feature in nova ? | 09:58 |
| goutham | can anybody help me | 09:58 |
| *** kevin-run has quit IRC | 09:59 | |
| *** syfy323 has joined #openstack-nova | 09:59 | |
| goutham | please i am new to nova and i want to write a new feature.. | 09:59 |
| *** thorst_away has quit IRC | 10:00 | |
| *** kevin_zh has quit IRC | 10:04 | |
| bauzas | goutham: heya, you can begin with some explanations here http://docs.openstack.org/developer/nova/how_to_get_involved.html#how-do-i-get-my-feature-in and there http://docs.openstack.org/developer/nova/process.html#how-do-i-get-my-code-merged | 10:05 |
| *** shaohe_feng has quit IRC | 10:07 | |
| *** yamahata has quit IRC | 10:07 | |
| *** shaohe_feng has joined #openstack-nova | 10:08 | |
| *** sdague has joined #openstack-nova | 10:09 | |
| *** pcaruana has joined #openstack-nova | 10:09 | |
| * tbachman can’t wait to see Friday’s NICKs given recent current events (#BRExit) | 10:10 | |
| *** mfedosin has joined #openstack-nova | 10:12 | |
| *** tbachman has quit IRC | 10:12 | |
| goutham | hi <bauzas> thanks for the reply | 10:13 |
| goutham | is there any place where i can find the code flow | 10:13 |
| goutham | of a small service | 10:13 |
| *** salv-orlando has quit IRC | 10:14 | |
| *** chandankumar has quit IRC | 10:16 | |
| *** shaohe_feng has quit IRC | 10:17 | |
| *** shaohe_feng has joined #openstack-nova | 10:18 | |
| openstackgerrit | Sean Dague proposed openstack/nova: policy: Add defaults in code (part 6) https://review.openstack.org/329228 | 10:19 |
| *** manikanta has joined #openstack-nova | 10:19 | |
| *** amotoki_ has quit IRC | 10:19 | |
| *** shaohe_feng has quit IRC | 10:27 | |
| *** ducttape_ has joined #openstack-nova | 10:28 | |
| *** shaohe_feng has joined #openstack-nova | 10:28 | |
| *** claudiub has joined #openstack-nova | 10:28 | |
| *** mpavone has joined #openstack-nova | 10:28 | |
| openstackgerrit | Jens Rosenboom proposed openstack/nova: Tests in test_ipv6 are broken https://review.openstack.org/333841 | 10:29 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove personality extension https://review.openstack.org/333030 | 10:30 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove preserve-ephemeral rebuild extension https://review.openstack.org/333014 | 10:30 |
| openstackgerrit | Sean Dague proposed openstack/nova: collapse servers.ViewBuilderV21 into servers.ViewBuilder https://review.openstack.org/333574 | 10:30 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove access_ips extension https://review.openstack.org/331834 | 10:30 |
| *** ducttape_ has quit IRC | 10:33 | |
| *** josecastroleon has quit IRC | 10:33 | |
| *** chandankumar has joined #openstack-nova | 10:33 | |
| Guest74267 | nick BobBall | 10:35 |
| *** Guest74267 is now known as BobBall | 10:36 | |
| *** salv-orlando has joined #openstack-nova | 10:36 | |
| *** shaohe_feng has quit IRC | 10:37 | |
| *** shaohe_feng has joined #openstack-nova | 10:38 | |
| johnthetubaguy | markus_z: can I catch up around https://wiki.openstack.org/wiki/ConfigOptionsConsistency | 10:40 |
| *** chandankumar has quit IRC | 10:45 | |
| bauzas | oops Friday | 10:46 |
| *** bauzas is now known as bauwser | 10:46 | |
| *** Dinesh_Bhor has joined #openstack-nova | 10:47 | |
| *** fragatina has quit IRC | 10:47 | |
| *** EinstCrazy has quit IRC | 10:48 | |
| *** shaohe_feng has quit IRC | 10:48 | |
| *** shaohe_feng has joined #openstack-nova | 10:48 | |
| *** chandankumar has joined #openstack-nova | 10:48 | |
| sdague | johnthetubaguy: I fixed claudiub's patch - https://review.openstack.org/#/c/329228 and rebased my extensions folding on top of it | 10:49 |
| *** aarefiev has joined #openstack-nova | 10:51 | |
| *** goutham has quit IRC | 10:53 | |
| *** acabot has quit IRC | 10:54 | |
| *** tbachman has joined #openstack-nova | 10:55 | |
| *** edand has joined #openstack-nova | 10:56 | |
| *** sudipto has quit IRC | 10:57 | |
| markus_z | johnthetubaguy: I hadn't yet the chance to look at it | 10:58 |
| *** shaohe_feng has quit IRC | 10:58 | |
| *** shaohe_feng has joined #openstack-nova | 10:58 | |
| *** thorst_away has joined #openstack-nova | 10:58 | |
| johnthetubaguy | markus_z: OK, just wondering about omitting sections when they are empty. Looking at this it feels like we are repeating our selves in too many of the possible values sections: http://docs-draft.openstack.org/46/317046/4/check/gate-nova-docs/8a4591b//doc/build/html/sample_config.html | 10:59 |
| *** salv-orl_ has joined #openstack-nova | 10:59 | |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: merge multiple create to servers.inc https://review.openstack.org/333378 | 11:00 |
| * johnthetubaguy going to get some food, and sulk about people voting the wrong way | 11:00 | |
| sdague | johnthetubaguy: well make sure you come back after | 11:00 |
| johnthetubaguy | heh, yeah, I will leave the beer for later | 11:01 |
| *** salv-orlando has quit IRC | 11:02 | |
| *** smatzek has joined #openstack-nova | 11:02 | |
| markus_z | johnthetubaguy: yeah, if its just a repetition of the default value, then it doesn't make sense. Or if it's a boolean value like "enable". | 11:03 |
| openstackgerrit | sahid proposed openstack/nova: network: handle unauthorized exception from neutron https://review.openstack.org/312014 | 11:03 |
| markus_z | johnthetubaguy: My initial idea to use "none" for related options was to be explicit about it. | 11:04 |
| *** thorst_away has quit IRC | 11:05 | |
| *** tbachman_ has joined #openstack-nova | 11:06 | |
| openstackgerrit | Jens Rosenboom proposed openstack/nova: Tests in test_ipv6 are broken https://review.openstack.org/333841 | 11:06 |
| *** acabot has joined #openstack-nova | 11:06 | |
| *** tbachman has quit IRC | 11:07 | |
| *** tbachman_ is now known as tbachman | 11:07 | |
| *** thorst_away has joined #openstack-nova | 11:07 | |
| openstackgerrit | Kien Nguyen proposed openstack/nova: Remove mox in nova/tests/unit/cells/test_cells_manager.py https://review.openstack.org/333851 | 11:07 |
| *** shaohe_feng has quit IRC | 11:08 | |
| markus_z | johnthetubaguy: Better I just read the wiki, takes a bit | 11:08 |
| *** shaohe_feng has joined #openstack-nova | 11:09 | |
| *** csatari__ has joined #openstack-nova | 11:11 | |
| *** edand has quit IRC | 11:12 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 1) https://review.openstack.org/330033 | 11:13 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 2) https://review.openstack.org/330034 | 11:13 |
| *** thorst_away is now known as thorst_ | 11:15 | |
| *** allen_gao has quit IRC | 11:17 | |
| *** luis5tb has quit IRC | 11:18 | |
| *** shaohe_feng has quit IRC | 11:18 | |
| *** shaohe_feng has joined #openstack-nova | 11:19 | |
| *** tbachman_ has joined #openstack-nova | 11:20 | |
| *** paul-carlton2 has quit IRC | 11:20 | |
| *** M00nr41n has quit IRC | 11:20 | |
| *** paul-carlton2 has joined #openstack-nova | 11:20 | |
| *** josecastroleon has joined #openstack-nova | 11:22 | |
| *** tbachman has quit IRC | 11:23 | |
| *** tbachman_ is now known as tbachman | 11:23 | |
| *** mpavone has quit IRC | 11:23 | |
| openstackgerrit | Andreas Scheuring proposed openstack/nova-specs: Use target VIF information on live migration https://review.openstack.org/301090 | 11:24 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Pin HTTP response codes to API microversions https://review.openstack.org/305369 | 11:25 |
| andreas_s | PaulMurray, Hi Paul, I would like to continue the discussions on the live migration bp: https://blueprints.launchpad.net/nova/+spec/migration-use-target-vif | 11:26 |
| andreas_s | PaulMurray, do you think we have time for this in the next live migration meeting? | 11:26 |
| andreas_s | PaulMurray, now things on Neutron side seem to be clear | 11:27 |
| andreas_s | thanks! | 11:27 |
| PaulMurray | andreas_s, hi | 11:28 |
| *** ducttape_ has joined #openstack-nova | 11:28 | |
| PaulMurray | andreas_s, we can start discussion, but the blueprint is not approved | 11:28 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: Allow multi-attach in compute api https://review.openstack.org/271047 | 11:28 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments https://review.openstack.org/267587 | 11:28 |
| *** shaohe_feng has quit IRC | 11:29 | |
| PaulMurray | what do you want to do ? | 11:29 |
| *** mpavone has joined #openstack-nova | 11:29 | |
| *** shaohe_feng has joined #openstack-nova | 11:29 | |
| andreas_s | PaulMurray, I know. So it will probably not make Newton, but that's fair | 11:29 |
| andreas_s | PaulMurray, I want to modify the network section of the domain.xml before live migration | 11:29 |
| andreas_s | PaulMurray, therefore some work on the neutron side needs to be done. I already worked out a proposal for that | 11:30 |
| *** rtheis has joined #openstack-nova | 11:30 | |
| andreas_s | PaulMurray, the feedback from Neutron cores was, that we should make this a joint effort, as they don't want to implement things that never get exploited ;) | 11:30 |
| PaulMurray | andreas_s, yes, that's fair. | 11:31 |
| PaulMurray | andreas_s, and there is nothing wrong with planning :) | 11:31 |
| andreas_s | PaulMurray, so basically the discussion is about, IF we want to solve the problem. | 11:31 |
| andreas_s | PaulMurray, and how that would fit into the nova priorities for ocata | 11:32 |
| andreas_s | PaulMurray, if we have an agreement there, I could continue to close the design on neutron side | 11:32 |
| *** danpb has quit IRC | 11:33 | |
| *** ducttape_ has quit IRC | 11:33 | |
| PaulMurray | I'm trying to remember where we got to with this before. I remember several things to do with networking in migration | 11:33 |
| PaulMurray | I will have to read through again | 11:33 |
| PaulMurray | do you have any more written up or POC patches ? | 11:34 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 11:34 |
| andreas_s | PaulMurray, to summarize: my first proposal was to do the portbinding (setting binding:host_id) in pre_live_migration instead of post_live_migration | 11:34 |
| *** cdelatte has joined #openstack-nova | 11:35 | |
| PaulMurray | I remember - that needed neutron to be aware of the migration and multiple ports right ? | 11:35 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 4) https://review.openstack.org/330036 | 11:35 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 2) https://review.openstack.org/330034 | 11:35 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 3) https://review.openstack.org/330035 | 11:35 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 5) https://review.openstack.org/333514 | 11:35 |
| andreas_s | PaulMurray, but that does not work out, as some neutron drivers do certain things when a portbinding is set - so I invested in neutron | 11:36 |
| andreas_s | PaulMurray, right | 11:36 |
| andreas_s | the idea now is to have the port bound to both hosts during migration | 11:36 |
| andreas_s | PaulMurray, https://review.openstack.org/#/c/309416/ | 11:36 |
| andreas_s | ^ Neutron spec - but it's also linked in the nova spec | 11:36 |
| PaulMurray | I've reviewed that one before | 11:37 |
| PaulMurray | sounds ok to me | 11:37 |
| andreas_s | PaulMurray, right | 11:37 |
| *** csatari__ has quit IRC | 11:38 | |
| andreas_s | PaulMurray, today I updated the nova spec to match the neutron proposal | 11:38 |
| PaulMurray | It does seem like a good idea to bring this up now | 11:38 |
| andreas_s | PaulMurray, https://review.openstack.org/#/c/301090 | 11:38 |
| PaulMurray | we will have our mid cycle in a few weeks | 11:38 |
| *** shaohe_feng has quit IRC | 11:39 | |
| PaulMurray | if we can't work out a plan before that we can do it there | 11:39 |
| andreas_s | PaulMurray, ok, let's try. I will not be there, but maybe somebody else from our team will come - need to figure that out | 11:39 |
| PaulMurray | How much attention has this had in neutron ? | 11:39 |
| *** shaohe_feng has joined #openstack-nova | 11:40 | |
| andreas_s | PaulMurray, to be honest, not that much at the moment. | 11:40 |
| *** alaski is now known as lascii | 11:41 | |
| PaulMurray | The meeting next week is a couple of days before nova's non-proiority feature freeze | 11:41 |
| andreas_s | PaulMurray, yeah I read that | 11:41 |
| PaulMurray | so I will probably go through all the non-prio patches to see what needs to be completed in that meeting | 11:41 |
| PaulMurray | but there should be time to bring this up as well | 11:42 |
| andreas_s | PaulMurray, cool. Putting it at the end is absolutely fair! | 11:42 |
| PaulMurray | Maybe you could raise awareness on the ML as well. | 11:42 |
| johnthetubaguy | markus_z: yeah, I am torn about the being explicit about the section being empty, and not bloating the simpler ones.the file | 11:42 |
| andreas_s | PaulMurray, ok, so let's continue next week. I need to leave a little bit earlier into the weekend today | 11:43 |
| PaulMurray | I'll put it on the agenda | 11:43 |
| andreas_s | PaulMurray, thanks for your time and for putting that on the agenda! | 11:43 |
| PaulMurray | speak to you next week then | 11:43 |
| andreas_s | yes, see you | 11:43 |
| *** andreas_s has quit IRC | 11:44 | |
| markus_z | johnthetubaguy: hm, true, the sample.conf is already huge, maybe just give the ops stuff to read they should really know. | 11:49 |
| *** shaohe_feng has quit IRC | 11:49 | |
| markus_z | johnthetubaguy: IOW, omitting empty sections is OK imo | 11:49 |
| *** shaohe_feng has joined #openstack-nova | 11:49 | |
| markus_z | johnthetubaguy: I've finished reading the wiki page and I found a few things | 11:49 |
| johnthetubaguy | markus_z: yeah, this stuff is used in the docs tables and the sample config file, so I think dropping them could work | 11:50 |
| johnthetubaguy | markus_z: cool, great to get more eyes on that wiki, so we agree the direction, we have so, so many patches up | 11:50 |
| markus_z | johnthetubaguy: #fix_opt_description I'm OK with dropping "Services which consume this". You said something about "compute" options and "controller" options a few weeks back, do we want to have that instead? | 11:51 |
| johnthetubaguy | markus_z: so I think if we do that, we need to do it by creating two separate sample files, like other projects do | 11:52 |
| markus_z | johnthetubaguy: IOW, how do we create a compute sample.conf and a controller sample.conf. IIRC that's what you was suggested? | 11:52 |
| sdague | alex_xu / johnthetubaguy - doc fix that will help on the defcore confusion - https://review.openstack.org/#/c/332973/ | 11:52 |
| *** danpb has joined #openstack-nova | 11:53 | |
| *** mpavone has quit IRC | 11:53 | |
| markus_z | johnthetubaguy: Do you know from the top of your head how they do this? | 11:53 |
| johnthetubaguy | markus_z: well, it also maps to the config option docs, they have very specific sets of config for different components | 11:54 |
| johnthetubaguy | markus_z: I don't remember which project I looked at now... | 11:55 |
| markus_z | johnthetubaguy: ok, we can figure that out later I guess. I'm ok with the direction, it's the maintainable option with enough clarity for ops. | 11:55 |
| markus_z | johnthetubaguy: The 3 items below "# min value, max value, 0, -1).", is that a format issue on the wiki page? | 11:56 |
| markus_z | looks a little odd | 11:57 |
| markus_z | johnthetubaguy: just to avoid confusion, we also want to omit "related options" if it's empty? Right now it's different on the wiki page. | 11:59 |
| *** shaohe_feng has quit IRC | 11:59 | |
| johnthetubaguy | markus_z: yeah, I think lets just drop the empty sections | 12:01 |
| markus_z | +1 | 12:01 |
| johnthetubaguy | markus_z: I did only just fix the formatting of those items, did you refresh recently? | 12:01 |
| claudiub | sdague: regarding shortening the import names, agents_policies become a_policies then? or only imports that have a lot of words in their names? | 12:01 |
| markus_z | johnthetubaguy: Ah, I had an old cached version, it's better now :) | 12:01 |
| johnthetubaguy | markus_z: cool | 12:01 |
| markus_z | johnthetubaguy: about #check_opt_group_and_type, I'm a little concerned about the "mutable" flag in that check. | 12:02 |
| *** shaohe_feng has joined #openstack-nova | 12:02 | |
| markus_z | johnthetubaguy: I agree that we should check our options for mutability, but I'd prefer an extra step for that. | 12:02 |
| openstackgerrit | Jens Rosenboom proposed openstack/nova: Tests in test_ipv6 are broken https://review.openstack.org/333841 | 12:03 |
| markus_z | johnthetubaguy: I assume that functional code changes would be necessary to make the "mutable" behave correctly. | 12:03 |
| *** rcernin has joined #openstack-nova | 12:04 | |
| johnthetubaguy | markus_z: it varies | 12:05 |
| johnthetubaguy | markus_z: depends if we "cache" the read from the CONF somewhere or not | 12:06 |
| markus_z | johnthetubaguy: yes, true, I wasn't precise here | 12:06 |
| johnthetubaguy | markus_z: but yeah, with you that many will need functional changes | 12:06 |
| johnthetubaguy | manager has a sig_hup handler that does stuff already | 12:07 |
| *** RuiChen has quit IRC | 12:07 | |
| *** fragatina has joined #openstack-nova | 12:07 | |
| markus_z | johnthetubaguy: And using "mutable" is not a documentation change anymore imo. | 12:08 |
| *** rfolco has joined #openstack-nova | 12:08 | |
| markus_z | johnthetubaguy: I like to have it, but I would exclude it from the current scope to be honest | 12:08 |
| markus_z | johnthetubaguy: and rather have single changes if ops say they like to have this or that option mutable | 12:09 |
| *** pradk has joined #openstack-nova | 12:09 | |
| *** shaohe_feng has quit IRC | 12:10 | |
| *** shaohe_feng has joined #openstack-nova | 12:10 | |
| *** ducttape_ has joined #openstack-nova | 12:10 | |
| johnthetubaguy | markus_z: yeah, that is scope creep really, maybe update the wiki to exclude that change | 12:11 |
| markus_z | johnthetubaguy: My last remark is about the very last paragraph. That's oslo bug 1568820 which got fixed in oslo.config 3.10 which we use in our requirements.txt. | 12:12 |
| openstack | bug 1568820 in oslo.config "Duplicate sections in generated config" [High,Fix released] https://launchpad.net/bugs/1568820 - Assigned to Ben Nemec (bnemec) | 12:12 |
| *** ducttape_ has quit IRC | 12:12 | |
| *** diga has quit IRC | 12:12 | |
| *** ducttape_ has joined #openstack-nova | 12:12 | |
| *** dave-mccowan has joined #openstack-nova | 12:13 | |
| johnthetubaguy | markus_z: cool, I am glad we can drop that bit. We should probably say, always pass the cfg group now? | 12:13 |
| markus_z | The rest is perfectly fine imo, thanks a lot for writing that up. I lost track on this topic :( | 12:13 |
| markus_z | johnthetubaguy: yes, always use cfg group | 12:13 |
| *** liusheng has quit IRC | 12:13 | |
| markus_z | johnthetubaguy: the burndown chart based on sdague's code is on my todo list for week R-12 | 12:14 |
| markus_z | johnthetubaguy: maybe I can move it a week earlier | 12:15 |
| markus_z | johnthetubaguy: right now I have to work on the virtlogd stuff which is important for my employer | 12:15 |
| johnthetubaguy | markus_z: I can ask one of the OSIC folks can do that, if thats easier | 12:15 |
| markus_z | johnthetubaguy: I wouldn't say "no" :) Would be great if someone else could work on that. | 12:16 |
| *** liusheng has joined #openstack-nova | 12:16 | |
| johnthetubaguy | markus_z: cool, sneti wrote most of the wiki page, so might fancy taking a look at the burndown, not sure yet | 12:17 |
| markus_z | johnthetubaguy: Shouldn't be *that* much changes. After that we should put a little effort in it to make it generic for other bulk efforts. | 12:17 |
| * johnthetubaguy nods at markus_z | 12:19 | |
| *** sfinucan has quit IRC | 12:19 | |
| *** shaohe_feng has quit IRC | 12:20 | |
| *** aysyd has joined #openstack-nova | 12:21 | |
| *** shaohe_feng has joined #openstack-nova | 12:21 | |
| markus_z | ok, cool :) | 12:22 |
| *** chandankumar has quit IRC | 12:23 | |
| *** ivase has joined #openstack-nova | 12:26 | |
| *** zerda2 has quit IRC | 12:27 | |
| *** sfinucan has joined #openstack-nova | 12:28 | |
| *** Hosam has joined #openstack-nova | 12:29 | |
| *** shaohe_feng has quit IRC | 12:30 | |
| *** shaohe_feng has joined #openstack-nova | 12:30 | |
| claudiub | sdague: johnthetubaguy: hi. I've got a question regarding policies. It seems to me that some policy rules are missing. | 12:31 |
| *** fragatina has quit IRC | 12:31 | |
| claudiub | sdague: johnthetubaguy: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/extension_info.py#L205 this is for the extension_info api. | 12:31 |
| johnthetubaguy | claudiub: I would say just add the rule, but for that one, its tempting to just remove the policy check | 12:32 |
| claudiub | sdague johnthetubaguy: oups, nevermind. it uses the 'extensions' policy, not the 'extension_info' policy. | 12:32 |
| *** lucasagomes is now known as lucas-hungry | 12:32 | |
| johnthetubaguy | claudiub: ah, OK, we can remove that later | 12:33 |
| *** jaypipes has joined #openstack-nova | 12:33 | |
| claudiub | although, in the future, I think the 'extensions' policies can be refined to 'extensions:index', 'extensions:show' and so on. | 12:34 |
| *** kylek3h has joined #openstack-nova | 12:35 | |
| claudiub | johnthetubaguy: also, there was a question before regarding 'discoverable' rules. it seems to be used by the extensions_info api. it will list all the extensions that pass the discoverable rule. | 12:36 |
| claudiub | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/extension_info.py#L169 | 12:36 |
| johnthetubaguy | claudiub: yeah, I think we are dropping all of those, slowly | 12:36 |
| *** ducttape_ has quit IRC | 12:37 | |
| *** jaypipes is now known as leakypipes | 12:38 | |
| paul-carlton2 | dansmith, You around | 12:39 |
| *** kylek3h has quit IRC | 12:40 | |
| *** shaohe_feng has quit IRC | 12:40 | |
| *** shaohe_feng has joined #openstack-nova | 12:41 | |
| *** tbachman has quit IRC | 12:42 | |
| *** zul_ is now known as zul | 12:44 | |
| *** lixiaoy1 has quit IRC | 12:45 | |
| *** erlon has joined #openstack-nova | 12:45 | |
| *** lixiaoy1 has joined #openstack-nova | 12:45 | |
| sdague | claudiub: yeh, I would basically not touch those at this point, we'll basically be deleting them as we delete the extensions | 12:45 |
| *** ducttape_ has joined #openstack-nova | 12:45 | |
| *** alex_xu has quit IRC | 12:46 | |
| sdague | claudiub: my concern is just on readability, when you have a very long string the actual important part of the can() is nearly on the other side of the page. | 12:47 |
| *** Oku_OS is now known as Oku_OS-away | 12:47 | |
| claudiub | true. | 12:47 |
| *** alex_xu has joined #openstack-nova | 12:48 | |
| sdague | the long ones for sure, I guess judgement on the rest | 12:48 |
| openstackgerrit | Kengo Sakai proposed openstack/nova: Check if flavor.vcpus is more than MAX_TAP_QUEUES https://review.openstack.org/332660 | 12:48 |
| *** tangchen_ has quit IRC | 12:49 | |
| *** tangchen_ has joined #openstack-nova | 12:50 | |
| *** shaohe_feng has quit IRC | 12:51 | |
| *** shaohe_feng has joined #openstack-nova | 12:51 | |
| *** pwnall has joined #openstack-nova | 12:53 | |
| mnestratov | paul-carlton2: hi, I made a review based on your https://review.openstack.org/#/c/314134 fixing some issues with unit tests, take a look please, if it is worth merging | 12:54 |
| openstackgerrit | Jens Rosenboom proposed openstack/nova: Tests in test_ipv6 are broken https://review.openstack.org/333841 | 12:54 |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Enabling detach root volume for shelved_offloaded https://review.openstack.org/326899 | 12:54 |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Attach/detach root volume for shelved_offloaded https://review.openstack.org/317446 | 12:55 |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Adding device_present field to bdm table https://review.openstack.org/268053 | 12:55 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 1) https://review.openstack.org/330033 | 12:55 |
| *** edmondsw has joined #openstack-nova | 12:55 | |
| PaulMurray | lascii, are you around to talk about cells & the console auth patch I mentioned yesterday ? | 12:55 |
| openstackgerrit | Evgeny Antyshev proposed openstack/nova: libvirt: add nova volume driver for vzstorage https://review.openstack.org/190843 | 12:56 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 1) https://review.openstack.org/330033 | 12:58 |
| *** baoli has joined #openstack-nova | 12:58 | |
| *** ducttape_ has quit IRC | 12:59 | |
| *** links has quit IRC | 13:00 | |
| openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Add option for running nova independently from hypervisor https://review.openstack.org/332216 | 13:00 |
| openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Perform disk operations in dom0 https://review.openstack.org/331254 | 13:00 |
| openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Stream config drive to XAPI https://review.openstack.org/332215 | 13:00 |
| *** shaohe_feng has quit IRC | 13:01 | |
| *** edleafe is now known as figleaf | 13:01 | |
| *** andymaier has joined #openstack-nova | 13:01 | |
| *** shaohe_feng has joined #openstack-nova | 13:01 | |
| *** baoli_ has joined #openstack-nova | 13:02 | |
| *** baoli has quit IRC | 13:05 | |
| *** efried has joined #openstack-nova | 13:06 | |
| lascii | PaulMurray: sure, what's up? | 13:07 |
| *** ducttape_ has joined #openstack-nova | 13:07 | |
| PaulMurray | lascii, two things | 13:08 |
| *** dave-mccowan has quit IRC | 13:08 | |
| *** paul-carlton2 has quit IRC | 13:08 | |
| *** takedakn has joined #openstack-nova | 13:09 | |
| PaulMurray | I will have code in the console proxy that has an instance uuid and a token | 13:09 |
| *** paul-carlton2 has joined #openstack-nova | 13:09 | |
| PaulMurray | and I need to look up the connection info related to them - it is stored in the child cell db | 13:09 |
| PaulMurray | last night you said we will always go to child cell at the moment - is that right ? | 13:09 |
| *** tblakeslee has joined #openstack-nova | 13:10 | |
| PaulMurray | instead of needing to look up the mapping in the api database) | 13:10 |
| *** rlrossit has joined #openstack-nova | 13:10 | |
| *** shaohe_feng has quit IRC | 13:11 | |
| *** tbachman has joined #openstack-nova | 13:12 | |
| lascii | PaulMurray: it depends on which db context manager you use. | 13:12 |
| lascii | http://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/api.py#n76 | 13:13 |
| lascii | main_context_manager uses cell db, api_context_manager uses api db | 13:13 |
| *** akshai has joined #openstack-nova | 13:14 | |
| *** shaohe_feng has joined #openstack-nova | 13:14 | |
| lascii | it usually looks like http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/host_mapping.py#n76 when the db methods are in an object | 13:14 |
| lascii | though you would use the other context manager | 13:14 |
| *** kfarr has joined #openstack-nova | 13:15 | |
| PaulMurray | maybe I need to change something | 13:15 |
| markus_z | sahid: Any chance that this change broke the (already fragile) live migration of instances with a serial console? https://github.com/openstack/nova/commit/917afd2831be8107177f50f277e400e03f896f74#diff-f4019782d93a196a0d026479e6aa61b1L6001 | 13:15 |
| PaulMurray | I did this in the db api: | 13:15 |
| PaulMurray | https://review.openstack.org/#/c/319964/15/nova/db/sqlalchemy/api.py | 13:16 |
| PaulMurray | the function on line 6822 is the one that reads the dataase | 13:16 |
| *** mdrabe has joined #openstack-nova | 13:16 | |
| PaulMurray | to get the instance | 13:16 |
| *** amotoki has joined #openstack-nova | 13:16 | |
| lascii | that context manager is fine, it uses the cell db but will switch if 'with context.target_cell()' is used | 13:17 |
| markus_z | sahid: I assume that "listen_addrs" will always be None in case when the serial console is enabled (and therefore VNC is disabled). | 13:17 |
| PaulMurray | the object is https://review.openstack.org/#/c/320063/14/nova/objects/console_connection.py@98 | 13:17 |
| lascii | PaulMurray: yeah, we have been putting the db methods directly in objects now | 13:17 |
| *** chandankumar has joined #openstack-nova | 13:18 | |
| lascii | to clean up the huge db/api mess | 13:18 |
| PaulMurray | I was going to get the mapping in the proxy code using InstanceMapping.get_by_instance_uuid() | 13:18 |
| markus_z | sahid: The only assignment of "serial_listen_addr" happens at https://github.com/openstack/nova/blob/11b15e8321f73d8b4401c3e9ce386282c683f293/nova/virt/libvirt/driver.py#L6480 | 13:18 |
| markus_z | sahid: which is a later point in time IIUC | 13:19 |
| PaulMurray | and then do with context.target_cell(...) | 13:19 |
| PaulMurray | and put the call to that validate method in the block | 13:19 |
| PaulMurray | Maybe I should put that into an object method to hide it ? | 13:19 |
| markus_z | sahid: to be fair, I haven't unit tested it yet. I noticed an odd behavior in my test setup when I try to migrate an instance with serial console. | 13:19 |
| lascii | PaulMurray: yep, that should all work. pick_context_manager is the right choice for that, though untested | 13:20 |
| PaulMurray | I copied it from tags I think | 13:20 |
| PaulMurray | yesterday you said there may be no cell mapping | 13:20 |
| *** sdake has joined #openstack-nova | 13:21 | |
| lascii | right, I wouldn't worry about looking that up yet | 13:21 |
| PaulMurray | what would you do then ? | 13:21 |
| *** ducttape_ has quit IRC | 13:21 | |
| *** shaohe_feng has quit IRC | 13:21 | |
| lascii | you can if you'd like. target_cell noops if cell_mapping is None | 13:21 |
| *** shaohe_feng has joined #openstack-nova | 13:22 | |
| lascii | let me comment on the object review | 13:22 |
| PaulMurray | ok - I'll experiment with that bit. | 13:22 |
| *** danpb has quit IRC | 13:22 | |
| PaulMurray | The second part is this: | 13:22 |
| *** mpavone has joined #openstack-nova | 13:23 | |
| PaulMurray | https://github.com/openstack/nova/blob/master/nova/consoleauth/manager.py#L115-124 | 13:23 |
| PaulMurray | I need to replicate this call | 13:23 |
| *** vtech_ has quit IRC | 13:23 | |
| PaulMurray | its an rpc call to compute manager that is in existing cells code | 13:23 |
| PaulMurray | Should I just cut and paste that for now ? | 13:23 |
| *** danpb has joined #openstack-nova | 13:24 | |
| sahid | markus_z: i don't understand, this method is used to check at source host, right ? | 13:24 |
| markus_z | sahid: yes | 13:25 |
| sahid | and this method is to set migrate_data object at destination ? | 13:25 |
| sahid | https://github.com/openstack/nova/blob/11b15e8321f73d8b4401c3e9ce386282c683f293/nova/virt/libvirt/driver.py#L6480 | 13:25 |
| lascii | PaulMurray: yeah. v2 can't substitute for v1 routing so that will need to be there | 13:26 |
| *** lbeliveau has quit IRC | 13:26 | |
| *** akshai_ has joined #openstack-nova | 13:26 | |
| markus_z | sahid: yes (my explanation was confusing) | 13:26 |
| PaulMurray | lascii, ok - I will just cut and paste for now | 13:27 |
| PaulMurray | lascii, BTW I intend to continue refactoring this as we go | 13:27 |
| PaulMurray | there is a lot of code that can be dropped around the consoleauth once the db is used | 13:27 |
| markus_z | sahid: I wanted to say that "listen_addrs" here will be None: https://github.com/openstack/nova/blob/11b15e8321f73d8b4401c3e9ce386282c683f293/nova/virt/libvirt/driver.py#L5333-L5333 | 13:27 |
| dansmith | paul-carlton2: am now | 13:27 |
| sahid | markus_z: what is the problem you have when trying to migrate an instance with serial console enable ? | 13:27 |
| PaulMurray | there is also a lot of redundancy around the five different types of console | 13:27 |
| lascii | PaulMurray: awesome, deleted code is bug free code | 13:28 |
| PaulMurray | :) | 13:28 |
| markus_z | sahid: I have to disable VNC for that | 13:28 |
| markus_z | sahid: which makes "listen_addrs" None | 13:28 |
| sahid | hum... i did not test that yes | 13:28 |
| *** lbeliveau has joined #openstack-nova | 13:29 | |
| markus_z | sahid: which leads to entering the if block and raising an exception in "very_serial_console_is_disabled()" | 13:29 |
| *** kfox1111_away is now known as kfox1111 | 13:29 | |
| sahid | markus_z: can we have vnc + serial console enable ? | 13:29 |
| sahid | it's something we support ? | 13:30 |
| *** akshai has quit IRC | 13:30 | |
| *** xyang1 has joined #openstack-nova | 13:30 | |
| *** paul-carlton2 has quit IRC | 13:30 | |
| *** paul-carlton2 has joined #openstack-nova | 13:30 | |
| markus_z | sahid: Uh, good question. I guess it's possible, yes. | 13:30 |
| openstackgerrit | Mike Fedosin proposed openstack/nova: Fix identation in nova.image https://review.openstack.org/333905 | 13:30 |
| *** mriedem has joined #openstack-nova | 13:31 | |
| sahid | markus_z: yes so if that is the case i did not verify, and i don't think we have unit tests to cover that | 13:31 |
| markus_z | sahid: I'm working with the IBM system z and this system doesn't support VNC, so I have to disable it via "nova.conf". | 13:31 |
| *** dmk0202 has quit IRC | 13:31 | |
| markus_z | sahid: OK, just wanted to have another pair of eyes on it. | 13:31 |
| markus_z | sahid: to be fair, it's an edge case | 13:31 |
| markus_z | sahid: Unfortunately my employer depends on that edge case :) | 13:31 |
| *** shaohe_feng has quit IRC | 13:32 | |
| sahid | markus_z: it's always like that | 13:32 |
| sahid | :) | 13:32 |
| *** shaohe_feng has joined #openstack-nova | 13:32 | |
| sahid | feel free to ping when you have a patch ready for review | 13:32 |
| paul-carlton2 | dansmith, I have updated a test and found that I get a failure because the actual ImageMeta and my expected values do not match | 13:32 |
| markus_z | sahid: OK, cool, will do that. Not sure if I can make it today. Probably Monday. | 13:33 |
| paul-carlton2 | strange thing is that when I compare them they seem the same, including comparing all items in __disct__ | 13:33 |
| paul-carlton2 | __dict__ | 13:33 |
| sahid | markus_z: no worries i'm being crazy at fixing conflicts for https://review.openstack.org/#/c/244489/ | 13:34 |
| paul-carlton2 | I can fix this by adding ComparableVersionedObject to the ImageMeta's super classes | 13:34 |
| markus_z | sahid: Ah, you're the poor soul who inherited those changes? ;) | 13:35 |
| paul-carlton2 | That object has an override of __eq__ which uses obj_to_primitive() to compare objects | 13:35 |
| dansmith | paul-carlton2: yeah normally objects don't have equality operators defined.. can you show me the code and the failure? | 13:36 |
| paul-carlton2 | Without it the assert_called_once_with fails | 13:36 |
| sahid | markus_z: ahah well somebody have to do it and it's kind of the area where i'm working on, so.. | 13:36 |
| *** woodster_ has joined #openstack-nova | 13:37 | |
| paul-carlton2 | I'm finishing up a test before pushing patchset, would it be easier for you to review it when submitted? | 13:37 |
| paul-carlton2 | dansmith, Just take out the | 13:38 |
| paul-carlton2 | -class ImageMeta(base.NovaObject): | 13:38 |
| paul-carlton2 | +class ImageMeta(base.NovaObject, | 13:38 |
| paul-carlton2 | + ovoo_base.ComparableVersionedObject): | 13:38 |
| paul-carlton2 | to make it fail | 13:38 |
| mriedem | sdague: why pop the values out of server_dict in translate_attributes here? https://review.openstack.org/#/c/331834/6/nova/api/openstack/compute/helpers.py | 13:38 |
| dansmith | paul-carlton2: yeah just push it and I'll look there | 13:38 |
| paul-carlton2 | ta | 13:38 |
| *** pacoblox has quit IRC | 13:39 | |
| *** gszasz has quit IRC | 13:39 | |
| *** dave-mccowan has joined #openstack-nova | 13:39 | |
| *** dmellado_ is now known as dmellado | 13:40 | |
| *** ducttape_ has joined #openstack-nova | 13:40 | |
| * danpb kind of wishes all our objects inherited ovo.CompatableVersionedObject by default | 13:40 | |
| *** smatzek has quit IRC | 13:40 | |
| sdague | mriedem: I thought it was done in some of the extensions previously. We should be done with that datastructure after this point | 13:40 |
| *** ctrath has joined #openstack-nova | 13:40 | |
| *** dave-mcc_ has joined #openstack-nova | 13:40 | |
| mriedem | sdague: yeah i didn't see it as breaking anything, just thought it might be weird if something tries to use server_dict later to look sometihng up and things are missing | 13:41 |
| dansmith | danpb: I don't :) | 13:41 |
| sdague | mriedem: right, they really shouldn't be though, because we should translate to the internal structures and use those | 13:41 |
| *** shaohe_feng has quit IRC | 13:42 | |
| danpb | dansmith: well, or alternatively have an assertVersionedObjectEqual() method on the base test class to make life easier in unit tests | 13:42 |
| dansmith | danpb: sure, I'd go for that option | 13:42 |
| danpb | since that's ultimately where i end up doing lots of comparisons | 13:42 |
| *** shaohe_feng has joined #openstack-nova | 13:42 | |
| paul-carlton2 | In this case it the comparison is done inside mock as part of assert_called_once_with... | 13:44 |
| *** dave-mccowan has quit IRC | 13:44 | |
| danpb | oh yeah, that gets tedious to deal with without having native equality support | 13:44 |
| *** Daisy has joined #openstack-nova | 13:45 | |
| dansmith | a test util function would be just as easy | 13:45 |
| *** vtech has joined #openstack-nova | 13:46 | |
| *** lucas-hungry is now known as lucasagomes | 13:46 | |
| danpb | dansmith: it is possible to munge that into mock ? | 13:47 |
| *** ducttape_ has quit IRC | 13:47 | |
| dansmith | oh, for called-with, I dunno | 13:47 |
| *** cleong has joined #openstack-nova | 13:49 | |
| *** Daisy has quit IRC | 13:49 | |
| dansmith | I can certainly write a function that mirrors assert_called_with that does what one expects | 13:51 |
| *** ctrath has quit IRC | 13:52 | |
| dansmith | without having to affect our runtime object definitions | 13:52 |
| *** shaohe_feng has quit IRC | 13:52 | |
| mriedem | sdague: in https://review.openstack.org/#/c/331834/ you should drop https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/access_ips.py also | 13:53 |
| *** shaohe_feng has joined #openstack-nova | 13:53 | |
| sdague | oh, I guess I missed that | 13:53 |
| *** takedakn has quit IRC | 13:53 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Parameter verification for os-simple-tenant-usage.inc https://review.openstack.org/333654 | 13:53 |
| mriedem | sdague: there was something in one of the tests also | 13:55 |
| *** vtech has quit IRC | 13:55 | |
| *** Dinesh_Bhor has quit IRC | 13:55 | |
| sdague | ah, yeh, you know, I think I read that test as one of the disable this extension tests | 13:55 |
| sdague | let me bring that back | 13:55 |
| PaulMurray | hi al, danpb's series here has +2 and could be ready: https://review.openstack.org/#/q/topic:bp/libvirt-clean-driver | 13:56 |
| PaulMurray | and these as well: https://review.openstack.org/#/q/topic:bp/auto-live-migration-completion | 13:56 |
| PaulMurray | the seoncd lot are in series behind the first | 13:56 |
| PaulMurray | any takers? | 13:56 |
| *** mmandel has joined #openstack-nova | 13:56 | |
| *** ecelik has quit IRC | 13:57 | |
| PaulMurray | FYI - the second bp is a live migration non-prio - that's why I'm asking | 13:57 |
| PaulMurray | mriedem, or any other ^^ | 13:57 |
| diana_clarke | dansmith: Thanks for pointing out the qcow2 errors in the experimental queue. The interesting results in that queue are the LVM ones, but I'll also try to figure out what's up with the other jobs. cc: mdbooth | 13:57 |
| sdague | mriedem: you want to look at the rest of the stack and I can fix it all in one go? | 13:58 |
| mdbooth | diana_clarke: Sounds fun | 13:58 |
| mriedem | sdague: that's a tall order | 13:58 |
| *** ctrath has joined #openstack-nova | 13:58 | |
| dansmith | diana_clarke: unless the neutron job was already broken like that then they all matter.. | 13:58 |
| mriedem | need to review 331685 first | 13:58 |
| *** matrohon has joined #openstack-nova | 13:59 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 4) https://review.openstack.org/330036 | 13:59 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 2) https://review.openstack.org/330034 | 13:59 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 3) https://review.openstack.org/330035 | 13:59 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 5) https://review.openstack.org/333514 | 13:59 |
| diana_clarke | dansmith: Yes, of course. I'm running check experimental on a no-op change that pre-dates this patch series as a baseline. And looking for similar non-experimental errors around backing files. | 13:59 |
| sdague | mriedem: ok, I'll push the fixes, including the one john did for the next patch | 13:59 |
| *** ametts has joined #openstack-nova | 14:00 | |
| *** nkrinner is now known as nkrinner_afk | 14:00 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove personality extension https://review.openstack.org/333030 | 14:00 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove preserve-ephemeral rebuild extension https://review.openstack.org/333014 | 14:00 |
| openstackgerrit | Sean Dague proposed openstack/nova: collapse servers.ViewBuilderV21 into servers.ViewBuilder https://review.openstack.org/333574 | 14:00 |
| mriedem | my morning meetings are starting too, so i'll review what i can in between | 14:00 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove access_ips extension https://review.openstack.org/331834 | 14:00 |
| *** tyrola has joined #openstack-nova | 14:01 | |
| sdague | ok, well access_ips should be all fixed per comments now | 14:01 |
| *** MikeG451 has joined #openstack-nova | 14:01 | |
| sdague | and johnthetubaguy I picked up the missed schema | 14:01 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split off code for updating live migration downtime https://review.openstack.org/327763 | 14:01 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split off code for updating migration stats in the DB https://review.openstack.org/327764 | 14:01 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for processing migration tasks https://review.openstack.org/327765 | 14:01 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for recovering after migration tasks https://review.openstack.org/327766 | 14:01 |
| *** shaohe_feng has quit IRC | 14:02 | |
| *** ctrath1 has joined #openstack-nova | 14:02 | |
| *** ctrath has quit IRC | 14:03 | |
| *** shaohe_feng has joined #openstack-nova | 14:03 | |
| mriedem | sdague: can't you also remove the comment block here? https://review.openstack.org/#/c/331834/6..7/nova/tests/unit/api/openstack/compute/test_access_ips.py@52 | 14:04 |
| *** tbachman_ has joined #openstack-nova | 14:04 | |
| *** tbachman has quit IRC | 14:05 | |
| *** tbachman_ is now known as tbachman | 14:05 | |
| *** sudipto has joined #openstack-nova | 14:05 | |
| *** jwcroppe has joined #openstack-nova | 14:05 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: Microversion 2.31 adds keypairs pagination support https://review.openstack.org/284868 | 14:05 |
| *** diogogmt has joined #openstack-nova | 14:06 | |
| *** njirap has quit IRC | 14:07 | |
| *** eharney has joined #openstack-nova | 14:07 | |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for detecting live migration job type https://review.openstack.org/327761 | 14:08 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for determining if migration should abort https://review.openstack.org/327762 | 14:08 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split off code for updating live migration downtime https://review.openstack.org/327763 | 14:08 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split off code for updating migration stats in the DB https://review.openstack.org/327764 | 14:08 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for processing migration tasks https://review.openstack.org/327765 | 14:08 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for recovering after migration tasks https://review.openstack.org/327766 | 14:08 |
| sdague | mriedem: actually... I should add content to that method as well | 14:08 |
| *** ratailor has quit IRC | 14:08 | |
| sdague | also, subclassing unit tests makes delete refactoring so hard to get right | 14:09 |
| *** pacoblox has joined #openstack-nova | 14:09 | |
| *** TravT has joined #openstack-nova | 14:09 | |
| *** tyrola has quit IRC | 14:09 | |
| *** TravT has quit IRC | 14:09 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove personality extension https://review.openstack.org/333030 | 14:10 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove preserve-ephemeral rebuild extension https://review.openstack.org/333014 | 14:10 |
| openstackgerrit | Sean Dague proposed openstack/nova: collapse servers.ViewBuilderV21 into servers.ViewBuilder https://review.openstack.org/333574 | 14:10 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove access_ips extension https://review.openstack.org/331834 | 14:10 |
| *** ducttape_ has joined #openstack-nova | 14:11 | |
| SamYaple | is there any reason when oslo.service forks a new thread that the config would change? im seeing my transport_url being treated differently when not using multiple workers vs using multiple workers | 14:11 |
| *** paul-carlton2 has quit IRC | 14:11 | |
| SamYaple | i filed a bug here https://bugs.launchpad.net/nova/+bug/1595965 | 14:11 |
| openstack | Launchpad bug 1595965 in OpenStack Compute (nova) "transport_url is being parsed different when forking with oslo.service" [Undecided,New] | 14:11 |
| *** vtech has joined #openstack-nova | 14:12 | |
| *** shaohe_feng has quit IRC | 14:13 | |
| *** TravT has joined #openstack-nova | 14:13 | |
| *** TravT has quit IRC | 14:13 | |
| *** shaohe_feng has joined #openstack-nova | 14:13 | |
| *** tbachman_ has joined #openstack-nova | 14:14 | |
| sdague | SamYaple: I don't think that rabbit://<user>:<pass>@server01,<user>:<pass>@server02/openstack is a valid transport url | 14:14 |
| *** TravT has joined #openstack-nova | 14:14 | |
| *** smatzek has joined #openstack-nova | 14:15 | |
| SamYaple | sdague: it is | 14:16 |
| SamYaple | according to the docs as well | 14:16 |
| SamYaple | but it is working | 14:16 |
| SamYaple | transport://user:pass@host:port[,userN:passN@hostN:portN]/virtual_host | 14:16 |
| SamYaple | im not including the port, but the port isnt parsed at that stage anyway | 14:16 |
| *** tyrola has joined #openstack-nova | 14:17 | |
| *** tbachman has quit IRC | 14:17 | |
| *** tbachman_ is now known as tbachman | 14:17 | |
| pkholkin | mriedem: Hi Matt, I've updated my keypairs pagination patch https://review.openstack.org/#/c/284868 | 14:18 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: policy: clean-up https://review.openstack.org/333925 | 14:18 |
| pkholkin | it becomes much better :) | 14:19 |
| sdague | SamYaple: ok, well I'd ask oslo folks, that's all well parsed before nova gets anywhere near it | 14:19 |
| pkholkin | mriedem: please look, I left many comments to previous and some to this patchsets | 14:19 |
| SamYaple | sdague: well the issue is probably defined at the end of this page http://docs.openstack.org/developer/oslo.messaging/transport.html | 14:19 |
| mriedem | danpb: comments in https://review.openstack.org/#/c/327761/3 | 14:19 |
| pkholkin | mriedem: now I believe in my tests :) | 14:19 |
| SamYaple | sdague: "oslo.messaging can't ensure that forking a process that shares the same transport object is safe for the library consumer" | 14:20 |
| pkholkin | sdague: hello Sean, please look too, when you have time https://review.openstack.org/#/c/284868 | 14:20 |
| pkholkin | thanks ! | 14:20 |
| *** tbachman_ has joined #openstack-nova | 14:22 | |
| *** shaohe_feng has quit IRC | 14:23 | |
| *** paul-carlton2 has joined #openstack-nova | 14:23 | |
| *** shaohe_feng has joined #openstack-nova | 14:23 | |
| *** tbachman has quit IRC | 14:24 | |
| *** tbachman_ is now known as tbachman | 14:24 | |
| *** tonytan4ever has joined #openstack-nova | 14:24 | |
| sean-k-mooney | danpb: are you still about? was wondering if you were planning to attend the nova midcycle? | 14:25 |
| danpb | sean-k-mooney: nope, won't be there | 14:27 |
| *** pbandark has quit IRC | 14:27 | |
| *** yamahata has joined #openstack-nova | 14:27 | |
| sean-k-mooney | danpb: ok was talking to jaypipes: and he was saying it would be good to touch base on where we are with os-vif. | 14:28 |
| mriedem | danpb: what are the realistic chances anyone is going to clean up redundant tests in test_driver after this series is done? | 14:28 |
| danpb | mriedem: i can volunteer todo it | 14:28 |
| sean-k-mooney | danpb: i was not sure if i would be able to attend either but wanted to check if you would be there first | 14:28 |
| danpb | since most of the hairy tests are my fault anyway | 14:28 |
| mriedem | danpb: and what's wrong with a conditional import of libvirt or passing it around as a parameter? | 14:28 |
| mriedem | the global variable is nasty | 14:29 |
| danpb | mriedem: its different from what we do in every other libvirt/*.py module basically | 14:29 |
| mriedem | i have a problem with those too :) | 14:29 |
| danpb | i don't object to changing the way we deal with this, but it should be the same pattern everywhere | 14:29 |
| danpb | i don't want to just add a different pattern in migration.py alone | 14:29 |
| mriedem | so you'd be willing to change that but you want it all done in the same change? | 14:30 |
| danpb | yeah | 14:30 |
| mriedem | after FF i'm sure :) | 14:30 |
| *** pbandark has joined #openstack-nova | 14:30 | |
| danpb | well i don't see it as a time critical thing - it could be early in ocata if we don't want to futz with it now | 14:31 |
| mriedem | i'd feel better with a todo in that libvirt var setting saying we should clean that pattern up at some point | 14:31 |
| danpb | after all we've had this current approach for many many cycles right now | 14:31 |
| mriedem | but you probably don't want to lose jay's +2 | 14:31 |
| leakypipes | mriedem: as always, my +2 is for sale to the highest bidder. :P | 14:31 |
| danpb | mriedem: well you could just live-edit the file to add the comment & blindly add your +W at the same time | 14:32 |
| mriedem | danpb: you'd have to rebase the series | 14:32 |
| markus_z | sahid: FYI: https://bugs.launchpad.net/nova/+bug/1595962 | 14:32 |
| openstack | Launchpad bug 1595962 in OpenStack Compute (nova) "live migration with disabled vnc/spice not possible" [Undecided,New] | 14:32 |
| danpb | oh true, to hell with it, i'll just add the comment & repost | 14:32 |
| *** sfinucan has quit IRC | 14:32 | |
| mriedem | sounds like jay is ready to re apply the +2 | 14:32 |
| mriedem | i'd just +W anyway | 14:33 |
| *** pbandark has quit IRC | 14:33 | |
| *** shaohe_feng has quit IRC | 14:33 | |
| *** shaohe_feng has joined #openstack-nova | 14:34 | |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for detecting live migration job type https://review.openstack.org/327761 | 14:35 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for determining if migration should abort https://review.openstack.org/327762 | 14:35 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split off code for updating live migration downtime https://review.openstack.org/327763 | 14:35 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split off code for updating migration stats in the DB https://review.openstack.org/327764 | 14:35 |
| *** sfinucan has joined #openstack-nova | 14:35 | |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: verify images.inc https://review.openstack.org/326871 | 14:35 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for processing migration tasks https://review.openstack.org/327765 | 14:35 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for recovering after migration tasks https://review.openstack.org/327766 | 14:35 |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: verify images.inc https://review.openstack.org/326871 | 14:35 |
| beagles | mriedem: rlrossit: IIRC re versioned objects and network models - I was working on it as part of the vif plugging stuff prior to the os-vif library initiative. I was in the process of breaking up a larger patch when we switched directions. I then got pulled off on to something else and it just languished after that. IIRC, the idea was to find a way to meld/reuse the objects that were in support of | 14:36 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 14:36 |
| beagles | support of nova network | 14:36 |
| beagles | mriedem: rlrossit: which ... also IIRC was a quagmire. Not a fit mapping at all. | 14:36 |
| sahid | markus_z: noted | 14:36 |
| *** mdrabe has quit IRC | 14:37 | |
| *** rook has quit IRC | 14:39 | |
| *** rook has joined #openstack-nova | 14:40 | |
| *** gongysh has joined #openstack-nova | 14:40 | |
| mriedem | johnthetubaguy: https://review.openstack.org/#/c/331834/ should be good to go now | 14:41 |
| *** sacharya has joined #openstack-nova | 14:43 | |
| *** shaohe_feng has quit IRC | 14:43 | |
| *** mdrabe has joined #openstack-nova | 14:44 | |
| *** shaohe_feng has joined #openstack-nova | 14:44 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: API changes for live migration of rescued instance https://review.openstack.org/328280 | 14:46 |
| mriedem | sdague: i think johnthetubaguy pointed this out also, but it looks like yo'ure missing a schema change to servers rebuild here https://review.openstack.org/#/c/333014/ | 14:47 |
| sdague | mriedem: the schema change should be there | 14:47 |
| mriedem | should vs are | 14:48 |
| sdague | or did I not repush? | 14:48 |
| mriedem | hmm | 14:48 |
| openstackgerrit | Paul Carlton proposed openstack/python-novaclient: [microversion] Bump to 2.31 https://review.openstack.org/316026 | 14:48 |
| *** dave-mcc_ has quit IRC | 14:48 | |
| mriedem | sob | 14:48 |
| mriedem | i'm on an older ps | 14:48 |
| mriedem | oh wait no i'm not | 14:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split off code for updating live migration downtime https://review.openstack.org/327763 | 14:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split off code for updating migration stats in the DB https://review.openstack.org/327764 | 14:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for processing migration tasks https://review.openstack.org/327765 | 14:49 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: split out code for recovering after migration tasks https://review.openstack.org/327766 | 14:49 |
| *** sacharya has quit IRC | 14:49 | |
| mriedem | sdague: it might already be there | 14:49 |
| sdague | oh, you are right, damn | 14:49 |
| mriedem | sdague: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py#L102 | 14:50 |
| mriedem | it's already there | 14:50 |
| sdague | yeh, no it's there | 14:50 |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: fix disk size calculation for VZ container instances https://review.openstack.org/222567 | 14:50 |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance resize support https://review.openstack.org/182257 | 14:50 |
| sdague | mriedem: ok, do you want me to do the pop? | 14:50 |
| *** sacharya has joined #openstack-nova | 14:50 | |
| paul-carlton2 | dansmith, see https://review.openstack.org/#/c/308198/ for the issue with object comparison we discussed earlier, specifically nova/tests/unit/compute/test_compute_mgr.py line 2474 | 14:50 |
| *** rook has quit IRC | 14:50 | |
| mriedem | sdague: yeah, we should be consistent | 14:50 |
| *** zz_dimtruck is now known as dimtruck | 14:51 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add a test utility for checking mock calls with objects https://review.openstack.org/333947 | 14:53 |
| dansmith | paul-carlton2: ^ | 14:53 |
| *** rlrossit has quit IRC | 14:53 | |
| *** rook has joined #openstack-nova | 14:53 | |
| *** shaohe_feng has quit IRC | 14:54 | |
| sdague | mriedem: updating | 14:54 |
| sdague | and addressing the pop thing in the next patch | 14:54 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove personality extension https://review.openstack.org/333030 | 14:54 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove preserve-ephemeral rebuild extension https://review.openstack.org/333014 | 14:54 |
| openstackgerrit | Sean Dague proposed openstack/nova: collapse servers.ViewBuilderV21 into servers.ViewBuilder https://review.openstack.org/333574 | 14:54 |
| *** shaohe_feng has joined #openstack-nova | 14:55 | |
| *** xenogear has quit IRC | 14:55 | |
| *** diogogmt has quit IRC | 14:56 | |
| mriedem | sdague: it sounds like you're ordering soda in the midwest | 14:56 |
| *** diogogmt has joined #openstack-nova | 14:58 | |
| mriedem | sdague: oh what about the reno? | 14:58 |
| *** tjones has joined #openstack-nova | 14:59 | |
| *** diogogmt has quit IRC | 15:00 | |
| *** rlrossit_ has joined #openstack-nova | 15:01 | |
| *** READ10 has joined #openstack-nova | 15:01 | |
| *** ssurana has joined #openstack-nova | 15:02 | |
| *** dave-mccowan has joined #openstack-nova | 15:02 | |
| *** josecastroleon has quit IRC | 15:02 | |
| *** lucas____ has joined #openstack-nova | 15:02 | |
| *** lucas____ has quit IRC | 15:02 | |
| *** tbachman has quit IRC | 15:02 | |
| *** shaohe_feng has quit IRC | 15:04 | |
| *** shaohe_feng has joined #openstack-nova | 15:05 | |
| mriedem | danpb: this perf events base patch is pretty simple https://review.openstack.org/#/c/320897/ | 15:05 |
| *** kfarr has quit IRC | 15:05 | |
| *** devananda has joined #openstack-nova | 15:06 | |
| *** kfox1111 is now known as kfox1111_away | 15:07 | |
| paul-carlton2 | dansmith, so I rebased my change on top of that and fix, removing my change to ImageMeta object and utilizing your test utility, thanks | 15:07 |
| dansmith | paul-carlton2: cool | 15:07 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 15:08 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Add a test utility for checking mock calls with objects https://review.openstack.org/333947 | 15:08 |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips https://review.openstack.org/329704 | 15:09 |
| *** tbachman has joined #openstack-nova | 15:09 | |
| *** mpavone has quit IRC | 15:10 | |
| lascii | # TODO(danms): Remove me after Icehouse | 15:11 |
| *** ljxiash has joined #openstack-nova | 15:11 | |
| *** ljxiash has quit IRC | 15:11 | |
| lascii | dansmith: I assume it's safe to remove the object_compat wrapper in compute manager | 15:11 |
| *** eharney has quit IRC | 15:12 | |
| *** tesseract- has quit IRC | 15:12 | |
| dansmith | lascii: is it still used anywhere? | 15:13 |
| *** matrohon has quit IRC | 15:13 | |
| dansmith | lascii: if not, then yes :) | 15:13 |
| dansmith | and oh my | 15:13 |
| *** dansmith is now known as superdan | 15:13 | |
| sdague | mriedem: there is nothing to note in the reno | 15:13 |
| lascii | danpb: refresh_instance_security_rules and get_instance_diagnostics | 15:13 |
| mriedem | sdague: the extension is dropped | 15:13 |
| *** READ10 has quit IRC | 15:13 | |
| sdague | we were never noting those in the reno | 15:13 |
| lascii | gah, name change fooled tab complete | 15:13 |
| sdague | we were noting when policy control points were dropped | 15:14 |
| lascii | danpb: ignore that | 15:14 |
| superdan | lascii: well, then we have to look and see if the last release was sending objects for those | 15:14 |
| sdague | as well as when extension hooks were | 15:14 |
| *** READ10 has joined #openstack-nova | 15:14 | |
| sdague | preserve_ephemeral never had a policy control point | 15:14 |
| *** shaohe_feng has quit IRC | 15:14 | |
| *** rook has quit IRC | 15:14 | |
| mriedem | lascii: i remember thinking the same thing about object_compat when i fixed something in refresh_instance_security_rules | 15:14 |
| *** rrecio has joined #openstack-nova | 15:15 | |
| *** shaohe_feng has joined #openstack-nova | 15:15 | |
| mriedem | superdan: lascii: i think refresh_instance_security_rules was updated in mitaka to send objects by garyk | 15:15 |
| mriedem | would need to verify though | 15:15 |
| lascii | okay. I'm setting up a mitaka branch to check | 15:15 |
| sdague | https://review.openstack.org/#/c/331834/8/releasenotes/notes/extensions_remove-37e9d4092981abbe.yaml | 15:15 |
| mriedem | sdague: yeah i see now | 15:16 |
| mriedem | i wasn't making the connection that the preserve_ephemeral wasn't checking policy to show anything | 15:16 |
| mriedem | because it's not showing anything in the response | 15:17 |
| lascii | heh, get_instance_diagnostics has a TODO about moving to objects | 15:17 |
| *** tbachman_ has joined #openstack-nova | 15:17 | |
| *** tbachman has quit IRC | 15:18 | |
| *** tbachman_ is now known as tbachman | 15:18 | |
| mriedem | get_instance_diagnostics is not used in the api yet | 15:18 |
| mriedem | lascii: i have a todo to write a spec for that for ocata | 15:18 |
| mriedem | get_instance_diagnostics is the one that was for v3 and uses the standardized diagnostics object | 15:18 |
| mriedem | but the rest api doesn't use that yet | 15:18 |
| lascii | okay, so we just assume that will use objects then? | 15:19 |
| mriedem | os-diagnostics uses the old get_diagnostics method | 15:19 |
| sdague | mriedem: yeh, it only used the discoverable bit | 15:19 |
| danpb | oh dear god, do we still not have the new diagnostics plumbed in yet ?!?! | 15:19 |
| * danpb thought we fixed that ages ago | 15:19 | |
| sdague | danpb: we do not | 15:19 |
| lascii | refresh_instance_security_rules switched to objects in L | 15:19 |
| sdague | it was finally reproposed recently | 15:19 |
| *** tonytan4ever has quit IRC | 15:19 | |
| mriedem | sdague: well | 15:19 |
| mriedem | not really | 15:19 |
| sdague | mriedem: oh, it never got reproposed? | 15:20 |
| mriedem | it's on my backlog | 15:20 |
| mriedem | but it's ocata at this point anyway | 15:20 |
| mriedem | so it's lower priority | 15:20 |
| *** salv-orl_ has quit IRC | 15:20 | |
| *** tyrola has quit IRC | 15:20 | |
| mriedem | but don't worry, it's on a kanban board and i have to report status on writing that spec at least twice per week | 15:21 |
| lascii | superdan: do we need an RPC version bump to remove object_compat, or can we assume that since we're at N+2 since objects started being sent we're set? | 15:21 |
| *** diogogmt has joined #openstack-nova | 15:21 | |
| *** lchen_ has joined #openstack-nova | 15:21 | |
| mriedem | lascii: there should have been an rpc bump when the method started taking objects right? | 15:22 |
| superdan | lascii: if N drops compatibility for something, then we should bump the major yeah | 15:22 |
| lascii | yeah. sorry, I meant major version bump | 15:22 |
| superdan | yeah | 15:22 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 2) https://review.openstack.org/330034 | 15:22 |
| superdan | we should do that at the end of the cycle, which I'm happy to do | 15:23 |
| superdan | if we just make a list | 15:23 |
| lascii | okay | 15:23 |
| lascii | where's a good place for that list? | 15:23 |
| superdan | I dunno, we have a release checkllist on the wiki already, so maybe a /Newton child of that or something? | 15:24 |
| superdan | needs to be in a place mriedem remembers :) | 15:24 |
| *** prateek has quit IRC | 15:24 | |
| *** shaohe_feng has quit IRC | 15:24 | |
| *** rook has joined #openstack-nova | 15:24 | |
| *** lchen has quit IRC | 15:24 | |
| *** shaohe_feng has joined #openstack-nova | 15:25 | |
| lascii | I'm guessing you search TODOs for a version bump, would some comments in the code marking those methods be useful? | 15:25 |
| superdan | yeah, maybe, although we have and ignore lots of those already :D | 15:25 |
| superdan | but yeah, if you want to freshen them that's probably ood | 15:25 |
| mriedem | superdan: sticky note on our kitchen island, that's how laura usually reminds me of things | 15:26 |
| mriedem | it has a ~65% success rate | 15:26 |
| superdan | mriedem: fine with me | 15:27 |
| *** eharney has joined #openstack-nova | 15:27 | |
| *** thomasem_ is now known as thomasem | 15:32 | |
| *** vtech has quit IRC | 15:33 | |
| *** tbachman has quit IRC | 15:34 | |
| *** shaohe_feng has quit IRC | 15:35 | |
| *** pacoblox has quit IRC | 15:35 | |
| *** pushkaru has joined #openstack-nova | 15:35 | |
| *** mingyu has quit IRC | 15:35 | |
| *** shaohe_feng has joined #openstack-nova | 15:35 | |
| *** pacoblox has joined #openstack-nova | 15:37 | |
| *** sdake has quit IRC | 15:38 | |
| *** sdake has joined #openstack-nova | 15:38 | |
| *** sdake has quit IRC | 15:38 | |
| *** sdake has joined #openstack-nova | 15:38 | |
| *** pcaruana has quit IRC | 15:38 | |
| *** rcernin has quit IRC | 15:39 | |
| *** jlanoux_ has quit IRC | 15:40 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 4) https://review.openstack.org/330036 | 15:40 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 3) https://review.openstack.org/330035 | 15:40 |
| *** salv-orlando has joined #openstack-nova | 15:40 | |
| *** paul-carlton2 has quit IRC | 15:41 | |
| *** sridharg has quit IRC | 15:41 | |
| *** shaohe_feng has quit IRC | 15:45 | |
| *** Daisy has joined #openstack-nova | 15:45 | |
| *** shaohe_feng has joined #openstack-nova | 15:46 | |
| *** zul has quit IRC | 15:47 | |
| *** irenab has quit IRC | 15:47 | |
| *** thumpba has joined #openstack-nova | 15:48 | |
| cfriesen | I just jumped from stable/mitaka to master and now "tox -e py27" is breaking with this: | 15:48 |
| cfriesen | File "nova/policies/base.py", line 20, in <module> | 15:48 |
| cfriesen | policy.RuleDefault('context_is_admin', 'role:admin'), | 15:48 |
| cfriesen | AttributeError: 'module' object has no attribute 'RuleDefault' | 15:48 |
| cfriesen | Do I need to regenerate my venv or something? | 15:48 |
| lascii | cfriesen: yes, you need oslo.policy 1.9 | 15:49 |
| *** rlrossit_ has quit IRC | 15:49 | |
| *** irenab has joined #openstack-nova | 15:49 | |
| *** sanfern has joined #openstack-nova | 15:49 | |
| cfriesen | lascii: thanks | 15:49 |
| *** Daisy has quit IRC | 15:50 | |
| sanfern | my nova-api-metadata fails with ERROR nova AddrFormatError: failed to detect a valid IP address from controller-host | 15:51 |
| *** daemontool_ has joined #openstack-nova | 15:52 | |
| sanfern | any help to fix this issue ? | 15:52 |
| *** baoli has joined #openstack-nova | 15:53 | |
| *** baoli_ has quit IRC | 15:53 | |
| *** daemontool has quit IRC | 15:54 | |
| *** vtech has joined #openstack-nova | 15:54 | |
| mriedem | i always get this confused, is pre_live_migration on the destination host? | 15:54 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 2) https://review.openstack.org/330034 | 15:54 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 3) https://review.openstack.org/330035 | 15:55 |
| *** baoli has quit IRC | 15:55 | |
| *** shaohe_feng has quit IRC | 15:55 | |
| *** alex_xu has quit IRC | 15:55 | |
| *** baoli has joined #openstack-nova | 15:55 | |
| *** shaohe_feng has joined #openstack-nova | 15:56 | |
| *** raunak has joined #openstack-nova | 15:56 | |
| mriedem | yup, nvm | 15:56 |
| *** baoli_ has joined #openstack-nova | 15:57 | |
| openstackgerrit | Chris Friesen proposed openstack/nova: Allow logging detailed info only on sched failure https://review.openstack.org/331306 | 15:58 |
| cfriesen | ^ now with unit tests. Next version will handle simultaneous sched operations in separate eventlets | 15:58 |
| *** andymaier has quit IRC | 15:58 | |
| *** alex_xu has joined #openstack-nova | 15:58 | |
| *** baoli has quit IRC | 16:01 | |
| *** zenoway has quit IRC | 16:02 | |
| *** mmandel has quit IRC | 16:03 | |
| *** fragatina has joined #openstack-nova | 16:04 | |
| *** fragatina has quit IRC | 16:04 | |
| *** sameo_ has joined #openstack-nova | 16:04 | |
| *** fragatina has joined #openstack-nova | 16:05 | |
| *** sanfern has quit IRC | 16:05 | |
| *** shaohe_feng has quit IRC | 16:05 | |
| *** shaohe_feng has joined #openstack-nova | 16:06 | |
| *** zenoway has joined #openstack-nova | 16:06 | |
| *** gongysh has quit IRC | 16:07 | |
| *** sameo has quit IRC | 16:08 | |
| *** burt has joined #openstack-nova | 16:08 | |
| *** kfox1111_away is now known as kfox1111 | 16:09 | |
| *** akshai_ has quit IRC | 16:11 | |
| *** zenoway has quit IRC | 16:11 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: API changes for live migration of rescued instance https://review.openstack.org/328280 | 16:14 |
| *** shaohe_feng has quit IRC | 16:16 | |
| *** vtech has quit IRC | 16:17 | |
| mriedem | danpb: any idea how libvirt will handle a domain xml with an empty <perf></perf> element in it? | 16:18 |
| mriedem | as in that's in the xml but there are no child <event> elements | 16:18 |
| *** shaohe_feng has joined #openstack-nova | 16:18 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Remove max_size argument to images.fetch and fetch_to_raw https://review.openstack.org/326947 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Fix fake data returned by LibvirtDriverTestCase._disk_info https://review.openstack.org/328366 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Rename the import of nova.virt.disk.api in driver https://review.openstack.org/333521 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Pass path to Image base class https://review.openstack.org/333271 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add from_image and from_func to Backend https://review.openstack.org/333522 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add create_from_image and create_from_func to Backend https://review.openstack.org/333244 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Rename Backend snapshot and image https://review.openstack.org/333272 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Pass is_block_dev explicitly from create_from_func https://review.openstack.org/333263 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Use valid uuids and fnames in test_create_image_initrd https://review.openstack.org/333241 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Improve mocking of Image objects in test_driver https://review.openstack.org/333242 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Pass instance object to _create_images_and_backing in test https://review.openstack.org/333243 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Replace some uses of invalid uuids in test_driver https://review.openstack.org/333978 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Remove some unnecessary mocking in test_driver https://review.openstack.org/333979 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add _create_external_boot_disks https://review.openstack.org/333980 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Move config disk creation into a separate function https://review.openstack.org/333981 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Move local root disk fetch into a separate function https://review.openstack.org/333982 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Move enumeration of ephemerals into a separate function https://review.openstack.org/333983 | 16:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Move swap disk fetch into a separate function https://review.openstack.org/333984 | 16:20 |
| *** smatzek has quit IRC | 16:22 | |
| *** mriedem has quit IRC | 16:22 | |
| *** sahid has quit IRC | 16:22 | |
| *** mmandel has joined #openstack-nova | 16:22 | |
| *** zul has joined #openstack-nova | 16:23 | |
| *** luis5tb has joined #openstack-nova | 16:23 | |
| *** luis5tb has quit IRC | 16:24 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add DeviceTaggingFilter to scheduler https://review.openstack.org/331490 | 16:25 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: attach configdrive after instance XML https://review.openstack.org/303335 | 16:25 |
| *** Apoorva has joined #openstack-nova | 16:25 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 16:25 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 16:25 |
| *** eharney has quit IRC | 16:25 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Compute manager device tagging support https://review.openstack.org/264017 | 16:26 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Libvirt driver implementation of device tagging https://review.openstack.org/264016 | 16:26 |
| *** shaohe_feng has quit IRC | 16:26 | |
| cfriesen | so I'm curious...are there any plans to start validating flavor extra-specs the way we do for image properties? | 16:27 |
| *** shaohe_feng has joined #openstack-nova | 16:27 | |
| *** akshai has joined #openstack-nova | 16:28 | |
| johnthetubaguy | cfriesen: I think jaypipes has a proposal to move towards official tags, I can't remember the link to that | 16:28 |
| *** tbachman has joined #openstack-nova | 16:29 | |
| *** yamahata has quit IRC | 16:31 | |
| *** alex_xu has quit IRC | 16:32 | |
| cfriesen | also, crazy idea I had...but currently we report vCPU usage the same regardless of whether it's pinned or shared. would it make sense to adjust that so that we can tell how close a host is to being full by looking at available/used cpus? Downside is it'd mean an API change since we'd report usage differently. | 16:33 |
| cfriesen | this'd be a future release blueprint/spec of course | 16:33 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Update get_instance_diagnostics for instance objects https://review.openstack.org/333988 | 16:33 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add reminder comments for compute rpcapi version bump https://review.openstack.org/333989 | 16:33 |
| *** dmk0202 has joined #openstack-nova | 16:34 | |
| *** alex_xu has joined #openstack-nova | 16:35 | |
| *** mingyu has joined #openstack-nova | 16:35 | |
| *** chandankumar has quit IRC | 16:36 | |
| *** shaohe_feng has quit IRC | 16:36 | |
| *** shaohe_feng has joined #openstack-nova | 16:37 | |
| *** pushkaru has quit IRC | 16:37 | |
| *** kylek3h has joined #openstack-nova | 16:37 | |
| *** eharney has joined #openstack-nova | 16:38 | |
| *** mingyu_ has joined #openstack-nova | 16:38 | |
| *** mingyu has quit IRC | 16:39 | |
| *** markus_z has quit IRC | 16:40 | |
| leakypipes | johnthetubaguy: this one? https://review.openstack.org/#/c/309762/ | 16:41 |
| *** mnestratov|2 has joined #openstack-nova | 16:42 | |
| *** mnestratov has quit IRC | 16:44 | |
| *** derekh has quit IRC | 16:45 | |
| *** shaohe_feng has quit IRC | 16:46 | |
| *** shaohe_feng has joined #openstack-nova | 16:47 | |
| openstackgerrit | Paul Murray proposed openstack/nova: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | 16:47 |
| *** lucasagomes is now known as lucas-afk | 16:47 | |
| *** acabot has quit IRC | 16:50 | |
| *** mfedosin has quit IRC | 16:53 | |
| *** BjoernT has joined #openstack-nova | 16:53 | |
| *** tblakeslee has quit IRC | 16:55 | |
| *** shaohe_feng has quit IRC | 16:57 | |
| PaulMurray | lascii, this change ^^^ adds code to websocketproxy to use the new ConsoleConnection object. At the moment I restrict it to only use this if we are not using cells. Would you take a look and tell me how it could be improved please. | 16:57 |
| PaulMurray | lascii, https://review.openstack.org/333990 | 16:57 |
| *** salv-orl_ has joined #openstack-nova | 16:59 | |
| *** shaohe_feng has joined #openstack-nova | 16:59 | |
| *** acabot has joined #openstack-nova | 17:01 | |
| *** akshai has quit IRC | 17:02 | |
| *** salv-orlando has quit IRC | 17:02 | |
| *** fragatina has quit IRC | 17:02 | |
| *** sudipto has quit IRC | 17:02 | |
| *** akshai has joined #openstack-nova | 17:02 | |
| lascii | PaulMurray: where is this code called from? | 17:06 |
| tjones | 3 super easy patches looking for cores. for the remove-mox BP https://review.openstack.org/265476 https://review.openstack.org/270303 https://review.openstack.org/271069 | 17:06 |
| *** paul-carlton2 has joined #openstack-nova | 17:06 | |
| *** danpb has quit IRC | 17:06 | |
| *** shaohe_feng has quit IRC | 17:07 | |
| *** sdake_ has joined #openstack-nova | 17:07 | |
| *** shaohe_feng has joined #openstack-nova | 17:07 | |
| *** superflyy has joined #openstack-nova | 17:08 | |
| *** sdake has quit IRC | 17:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/333181 | 17:09 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements https://review.openstack.org/331901 | 17:10 |
| figleaf | cfriesen: I have some ideas for improving the logging in https://review.openstack.org/#/c/331306 | 17:10 |
| figleaf | cfriesen: would you mind if I played around with it, and if it works, push a revision? | 17:11 |
| PaulMurray | lascii, the console proxy | 17:11 |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification for api-ref os-flavor-extra-specs https://review.openstack.org/333651 | 17:11 |
| lascii | PaulMurray: okay, I'm a bit lost in the flow of using a console. but if there's already a connection to the console proxy I don't think you need to care about cellsv1. v1 mostly hooks in to redirect compute/api->conductor/manager/... calls to a cell and then the proper service. If you're already connected within a cell then v1 is out of the picture | 17:15 |
| *** jed56 has quit IRC | 17:15 | |
| *** dmk0202 has quit IRC | 17:15 | |
| *** catintheroof has joined #openstack-nova | 17:16 | |
| *** dmk0202 has joined #openstack-nova | 17:16 | |
| *** yamahata has joined #openstack-nova | 17:17 | |
| *** shaohe_feng has quit IRC | 17:17 | |
| PaulMurray | lascii, maybe I don't understand how console proxies are configured when using cells | 17:17 |
| *** shaohe_feng has joined #openstack-nova | 17:17 | |
| PaulMurray | lascii, I thought they would be at the api level | 17:17 |
| lascii | looking at it now | 17:20 |
| *** superflyy has quit IRC | 17:20 | |
| lascii | it seems that get_vnc_console goes to a cell to get connection info | 17:20 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Add a test utility for checking mock calls with objects https://review.openstack.org/333947 | 17:21 |
| lascii | but consoleauth.authorize_console is not directed to a cell | 17:21 |
| *** pacoblox has quit IRC | 17:21 | |
| PaulMurray | lascii, that's right | 17:21 |
| PaulMurray | consoleauth is a service running at the api level | 17:21 |
| PaulMurray | the compute manager gets console info from the hypervisor and sends it to the consoleauth service | 17:22 |
| PaulMurray | it also generates the URL | 17:22 |
| PaulMurray | the URL is returned to the user | 17:23 |
| PaulMurray | the user puts it in a browser and connects to console proxy | 17:23 |
| PaulMurray | which asks consoleauth to check the token in the URL and return the console info to the proxy so it can connect | 17:23 |
| lascii | okay, so console proxy could be deployed anywhere in a v1 world | 17:24 |
| lascii | I know rackspace puts one in each cell, but I guess that's not required | 17:24 |
| PaulMurray | yes, could even be one per hsot | 17:24 |
| PaulMurray | lascii, it makes sense | 17:24 |
| PaulMurray | lascii, I had assumed it would be at api - it has to be externally accessible | 17:25 |
| *** sabari has left #openstack-nova | 17:25 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 17:26 |
| lascii | PaulMurray: yeah, but being in a cell doesn't necessarily mean not accessible | 17:26 |
| catintheroof | guys, im using opensctack client with liberty, and "openstack server list" gives me "EndpointNotFound: publicURL endpoint for compute service in RegionOne region not found" using keystone v3 | 17:26 |
| *** ctrath1 has quit IRC | 17:27 | |
| catintheroof | but the nova endpoint is perfectly created because using v2 auth works perfect | 17:27 |
| PaulMurray | lascii, right, I really meant I had made an assumption there | 17:27 |
| lascii | PaulMurray: so the issue is that there's no guarantee that console proxies are configured to talk to the right database | 17:27 |
| PaulMurray | right | 17:27 |
| lascii | in general, but more so for v1 | 17:27 |
| *** shaohe_feng has quit IRC | 17:27 | |
| *** shaohe_feng has joined #openstack-nova | 17:28 | |
| PaulMurray | would there be a way to know if it is in a cell or not ? | 17:28 |
| lascii | so yeah, I think what you have is the best way right now. | 17:28 |
| lascii | PaulMurray: not that I can think of | 17:29 |
| lascii | the only difference is the access_url | 17:29 |
| *** ctrath has joined #openstack-nova | 17:29 | |
| lascii | each cell will share one, or have their own | 17:29 |
| lascii | but no good way to look that up | 17:29 |
| figleaf | catintheroof: see the channel topic. Better place to ask would be #openstack | 17:29 |
| *** GB21 has joined #openstack-nova | 17:30 | |
| catintheroof | figleaf, thanks, will do | 17:30 |
| *** claudiub|2 has joined #openstack-nova | 17:31 | |
| *** nicodemus_ has joined #openstack-nova | 17:32 | |
| *** nicodemus_ has left #openstack-nova | 17:32 | |
| openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref: os-volumes.inc https://review.openstack.org/320048 | 17:34 |
| *** daemontool_ has quit IRC | 17:34 | |
| PaulMurray | lascii, thanks - something to think about | 17:35 |
| *** claudiub has quit IRC | 17:35 | |
| *** dmk0202 has quit IRC | 17:35 | |
| *** mriedem has joined #openstack-nova | 17:36 | |
| *** tjones has left #openstack-nova | 17:37 | |
| *** shaohe_feng has quit IRC | 17:38 | |
| *** shaohe_feng has joined #openstack-nova | 17:38 | |
| *** woodster_ has quit IRC | 17:39 | |
| mriedem | superdan: i'm gonna bug you to check https://review.openstack.org/#/c/316275/ again since you were +2, i just had to rebase and fix a test | 17:39 |
| superdan | in a sec | 17:39 |
| *** dmk0202 has joined #openstack-nova | 17:39 | |
| *** vtech has joined #openstack-nova | 17:41 | |
| *** PaulMurray has quit IRC | 17:41 | |
| mriedem | why was 'personality' ever a good name for the file injection API? | 17:41 |
| mriedem | is that what aws calls it? | 17:41 |
| *** tjones has joined #openstack-nova | 17:42 | |
| *** pacoblox has joined #openstack-nova | 17:42 | |
| *** tjones has left #openstack-nova | 17:43 | |
| *** carolynvs has joined #openstack-nova | 17:44 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: API changes for live migration of rescued instance https://review.openstack.org/328280 | 17:44 |
| *** thumpba_ has joined #openstack-nova | 17:46 | |
| *** lpetrut has quit IRC | 17:46 | |
| figleaf | mriedem: hmmm, I don't remember. I remember arguing that it was a dumb name, but I lost that argument, too. | 17:46 |
| mriedem | maybe it's how you personalize your vm | 17:46 |
| *** baoli_ has quit IRC | 17:47 | |
| figleaf | mriedem: nah, I personalize mine by putting a bird on it | 17:48 |
| *** thumpba has quit IRC | 17:48 | |
| *** baoli has joined #openstack-nova | 17:48 | |
| *** shaohe_feng has quit IRC | 17:48 | |
| mriedem | you don't live in portland! | 17:48 |
| *** smatzek has joined #openstack-nova | 17:48 | |
| *** shaohe_feng has joined #openstack-nova | 17:48 | |
| *** akshai has quit IRC | 17:48 | |
| *** rlrossit has joined #openstack-nova | 17:48 | |
| *** akshai has joined #openstack-nova | 17:49 | |
| *** sambetts is now known as sambetts|afk | 17:49 | |
| figleaf | :) | 17:49 |
| *** baoli has quit IRC | 17:50 | |
| mriedem | sdague: https://review.openstack.org/#/c/333030/6/nova/tests/unit/api/openstack/compute/test_serversV21.py i think that test is showing how your change is going to break some hook | 17:51 |
| mriedem | remember that's what started the whole deprecate hooks discussion? because injected_files was broken | 17:52 |
| openstackgerrit | Julian proposed openstack/nova: Add unit tests for nova.virt.firewall.IpTablesFirewallDriver (Part 2) https://review.openstack.org/325435 | 17:53 |
| *** paul-carlton2 has quit IRC | 17:54 | |
| mriedem | sdague: yeah this https://github.com/openstack/nova/commit/be4220b37df42d4f99b5f5b8c18245a315228a8a | 17:55 |
| *** DaiDV has joined #openstack-nova | 17:58 | |
| *** DaiDV has quit IRC | 17:58 | |
| *** shaohe_feng has quit IRC | 17:58 | |
| *** shaohe_feng has joined #openstack-nova | 17:59 | |
| mriedem | so i guess we either remove hooks, which was deprecated in mitaka https://review.openstack.org/#/c/286276/ or we can't break that api just yet | 17:59 |
| *** sthillma has joined #openstack-nova | 18:01 | |
| *** zul has quit IRC | 18:02 | |
| *** openstackgerrit has quit IRC | 18:03 | |
| *** openstackgerrit has joined #openstack-nova | 18:03 | |
| *** pacoblox has quit IRC | 18:03 | |
| *** sdake_ has quit IRC | 18:04 | |
| *** tblakeslee has joined #openstack-nova | 18:05 | |
| *** pacoblox has joined #openstack-nova | 18:05 | |
| *** alexpilotti has quit IRC | 18:05 | |
| sdague | mriedem: I'm all up for removing hooks, it's untested | 18:07 |
| sdague | which is why we were going to remove it | 18:07 |
| mriedem | sdague: is there anything you aren't up for removing | 18:07 |
| *** tblakeslee_ has joined #openstack-nova | 18:08 | |
| *** shaohe_feng has quit IRC | 18:08 | |
| *** mingyu_ has quit IRC | 18:09 | |
| *** shaohe_feng has joined #openstack-nova | 18:09 | |
| sdague | what's your call, on the hooks side? | 18:09 |
| *** tblakeslee has quit IRC | 18:10 | |
| *** tblakeslee_ is now known as tblakeslee | 18:10 | |
| mriedem | sdague: as in removing in newton or ocata? | 18:10 |
| sdague | yeh | 18:11 |
| mriedem | i kind of think ocata | 18:11 |
| *** alexpilotti has joined #openstack-nova | 18:11 | |
| mriedem | so it's deprecated all through newton | 18:11 |
| sdague | ok | 18:11 |
| mriedem | being on a CD team has me in a different mindset about deprecation periods | 18:11 |
| sdague | well, we're in our CD window, it's past 90 days | 18:11 |
| sdague | if you aren't CDing at least every 90 days, you aren't very C | 18:12 |
| *** ducttape_ has quit IRC | 18:12 | |
| mriedem | true | 18:12 |
| mriedem | i can never remember the specific policy wording | 18:12 |
| mriedem | https://governance.openstack.org/reference/tags/assert_follows-standard-deprecation.html | 18:12 |
| mriedem | hmm, migration path | 18:13 |
| mriedem | i guess depends on who said they needed hooks for things and then tried to get them upstream | 18:13 |
| mriedem | the guys that hit the file injection thing had a spec, but folded that into mikal's vendordata endpoint spec | 18:14 |
| sdague | which has code posted | 18:14 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Fix revert on migration with SR-IOV https://review.openstack.org/326174 | 18:15 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Raise exception if PCI device spec is not found https://review.openstack.org/331830 | 18:15 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Allocate PCI devices on migration https://review.openstack.org/328983 | 18:15 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Update binding:profile for SR-IOV ports https://review.openstack.org/242573 | 18:15 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: PCI devices should be in 'allocated' after resize https://review.openstack.org/334024 | 18:15 |
| *** eharney has quit IRC | 18:15 | |
| mriedem | sdague: yeah, but do we drop hooks until that's in place? | 18:15 |
| *** hemna has quit IRC | 18:15 | |
| *** alexpilotti has quit IRC | 18:16 | |
| sdague | if you want to keep hooks working, I can change that patch, I did think about the hooks break when I did it, but considering there is not actual testing for hooks (because we never required testing for the fix) I didn't think we were going to try to keep it alive through newton | 18:18 |
| *** shaohe_feng has quit IRC | 18:19 | |
| *** READ10 has quit IRC | 18:19 | |
| mriedem | i'd say let's leave it in place for newton | 18:19 |
| *** shaohe_feng has joined #openstack-nova | 18:19 | |
| sdague | ok | 18:19 |
| melwitt | mriedem, sdague: IIRC from the thread on removing hooks, a number of folks responded with concerns of removing hooks before an alternative is available, to not leave them stranded in the interim | 18:20 |
| mriedem | melwitt: yeah that was really about the vendordata thing | 18:20 |
| melwitt | yes | 18:21 |
| mriedem | otherwise people could use notifications, | 18:21 |
| mriedem | or actually bring up their use case | 18:21 |
| sdague | so with all that being said, honestly, we should probably merge mikal's patch before the functional tests | 18:21 |
| melwitt | right. it is also my understanding that the alternatives are notifications and the new vendordata stuff in progress | 18:21 |
| sdague | because it would be silly to hold off the replacement for the untested thing waiting on comprehensive testing of it | 18:21 |
| *** vtech has quit IRC | 18:22 | |
| sdague | potentially missing the release | 18:22 |
| mriedem | sdague: so re: that, hooks deprecation is separate from the static vendor json metadata file right? | 18:22 |
| mriedem | or plugin | 18:22 |
| sdague | mriedem: that was supposed to be part of the solution, right? | 18:22 |
| sdague | my understanding was the solution was "here is this vendor data thing, it's even dynamic now + notifications + bring us other things you need" | 18:23 |
| sdague | no one did #3 | 18:23 |
| *** GB21 has quit IRC | 18:24 | |
| mriedem | the vendordata driver deprecation was https://review.openstack.org/#/c/288107/ | 18:24 |
| sdague | mriedem: yes, I realize that was a different thing | 18:24 |
| melwitt | AFAIK mikal's proposal covers all the non-notification-able cases | 18:25 |
| sdague | but when we started talking about solutions, the injection solution became part of that, right? | 18:25 |
| *** zul has joined #openstack-nova | 18:25 | |
| *** carolynvs has quit IRC | 18:25 | |
| sdague | melwitt: right | 18:25 |
| mriedem | sdague: yeah - and that was the only hooks case i remember coming up | 18:26 |
| mriedem | in the ML | 18:26 |
| mriedem | i'm not crazy about rushing mikel's stuff in w/o proper testing | 18:26 |
| mriedem | mikal i should say | 18:27 |
| superdan | me either | 18:27 |
| sdague | mriedem: I get that, but it's replacing a thing that has no testing | 18:27 |
| sdague | he at least has unit tests | 18:27 |
| superdan | that doesn't really do it for me :) | 18:27 |
| mriedem | yeah i'm assuming the people downstream have testing if they are relying on out of tree stuff | 18:27 |
| superdan | are functional tests super hard for this for some reason? | 18:27 |
| mriedem | i'd like to just be comfortable with the replacement before we say 'use this b/c we've dropped the alternative' | 18:28 |
| sdague | mriedem: I'm not sure why you expect that :) | 18:28 |
| mriedem | and then someone comes along and says the thing you told me to use is busted | 18:28 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: Microversion 2.32 adds keypairs pagination support https://review.openstack.org/284868 | 18:28 |
| *** shaohe_feng has quit IRC | 18:29 | |
| *** shaohe_feng has joined #openstack-nova | 18:29 | |
| sdague | mriedem: ok, I just don't want us to end up in a state where that bit stalls out, hooks is there, untested, getting broken by stuff we don't think about (because that interface is fragile as hell) | 18:29 |
| sdague | and I really did think we were deleting hooks this cycle, which is why I let the fix in without tests | 18:30 |
| superdan | his patch also only implements this for libvirt it looks like | 18:30 |
| superdan | which is a problem | 18:30 |
| mriedem | sdague: i don't have a problem telling people 'oops we broke your untested hook point' | 18:30 |
| mriedem | i just don't want to intentionally do it w/o removing hooks first | 18:30 |
| superdan | right, broken hooks is less bad than broken $new_thing | 18:30 |
| sdague | ok | 18:30 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Pass nodename to internal RT methods https://review.openstack.org/334031 | 18:30 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Pass the nodename to RT.instance_claim() https://review.openstack.org/334032 | 18:30 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Pass nodename into RT.re[build|size]_claim() https://review.openstack.org/334033 | 18:30 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Remove RT.nodename attribute entirely https://review.openstack.org/334034 | 18:31 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Add virt driver get_inventory() method https://review.openstack.org/334035 | 18:31 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Implement Hyper-V get_inventory() method https://review.openstack.org/334036 | 18:31 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Add get_inventory() implementation for vCenter https://review.openstack.org/334037 | 18:31 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Add get_inventory() method for Ironic https://review.openstack.org/334038 | 18:31 |
| sdague | I'll go unbreak hooks then in my patch | 18:31 |
| mriedem | cool, thanks | 18:31 |
| *** eharney has joined #openstack-nova | 18:34 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: WIP - rt: Manage resource provider and inventory https://review.openstack.org/334039 | 18:35 |
| leakypipes | cdent, bauwser, lascii, superdan: ^^ has my ideas on how to update inventory from the resource tracker in the new resource-providers world... | 18:35 |
| sdague | mriedem: you also want an enum? even though it's going to be helpers.CREATE instead of 'create' ? | 18:36 |
| *** rfinn has joined #openstack-nova | 18:38 | |
| *** hemna has joined #openstack-nova | 18:38 | |
| *** shaohe_feng has quit IRC | 18:39 | |
| *** ijw has joined #openstack-nova | 18:39 | |
| *** shaohe_feng has joined #openstack-nova | 18:40 | |
| *** openstackstatus has quit IRC | 18:40 | |
| *** openstack has joined #openstack-nova | 18:42 | |
| *** ijw has joined #openstack-nova | 18:42 | |
| mriedem | sdague: please | 18:43 |
| *** claudiub|2 has quit IRC | 18:45 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove personality extension https://review.openstack.org/333030 | 18:46 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove preserve-ephemeral rebuild extension https://review.openstack.org/333014 | 18:46 |
| openstackgerrit | Sean Dague proposed openstack/nova: collapse servers.ViewBuilderV21 into servers.ViewBuilder https://review.openstack.org/333574 | 18:46 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove access_ips extension https://review.openstack.org/331834 | 18:46 |
| sdague | mriedem: ok, see if you like that better | 18:47 |
| *** ducttape_ has joined #openstack-nova | 18:47 | |
| *** salv-orl_ has quit IRC | 18:49 | |
| figleaf | leakypipes: in that update patch, where does self.resource_providers[host] get updated when the resources change? | 18:49 |
| *** shaohe_feng has quit IRC | 18:49 | |
| *** shaohe_feng has joined #openstack-nova | 18:50 | |
| *** stewie925 has joined #openstack-nova | 18:51 | |
| mriedem | sdague: do we need this in this change? or at all? https://review.openstack.org/#/c/333030/7/nova/api/openstack/compute/helpers.py@99 | 18:53 |
| sdague | mriedem: it's not completely out of place because we're adding the op parameter | 18:54 |
| mriedem | sdague: yeah i figured that's why you were adding it | 18:55 |
| *** lpetrut has joined #openstack-nova | 18:55 | |
| sdague | it was in #5 | 18:55 |
| mriedem | and taking some cleanup liberties in servers.py :) | 18:55 |
| sdague | https://review.openstack.org/#/c/333030/5/nova/api/openstack/compute/helpers.py | 18:55 |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips-bulk https://review.openstack.org/333075 | 18:55 |
| sdague | mriedem: I said that in the commit message | 18:55 |
| mriedem | oh i missed that | 18:55 |
| sdague | otherwise all the translate lines become multiline | 18:55 |
| *** amotoki has quit IRC | 18:55 | |
| mriedem | yeah, ok | 18:56 |
| mriedem | body['server'] 20 times was great | 18:56 |
| sdague | heh | 18:56 |
| mriedem | alright +2 on that one | 18:57 |
| sdague | \o/ | 18:59 |
| *** pacoblox has quit IRC | 18:59 | |
| sdague | now I just need johnthetubaguy to come back from heavy drinking and reapply his +2s :) | 18:59 |
| mriedem | sdague: so around 2am last night i woke up thinking about the 404 on proxy apis | 18:59 |
| mriedem | and what that means for the client | 18:59 |
| mriedem | as in novaclient | 18:59 |
| sdague | sure | 18:59 |
| *** shaohe_feng has quit IRC | 19:00 | |
| mriedem | as in, i'm not sure if we're going to temporarily request a lower microversion on purpose for some of those CLIs | 19:00 |
| *** shaohe_feng has joined #openstack-nova | 19:00 | |
| mriedem | and deprecate the CLIs at the same time - if they aren't already | 19:00 |
| sdague | right. | 19:00 |
| mriedem | i guess a lot of them should already be deprecated | 19:00 |
| sdague | besides the network api, they are all deprecated already for sure | 19:01 |
| sdague | lets separate this into 2 halves | 19:01 |
| sdague | not network / network | 19:01 |
| *** cdelatte has quit IRC | 19:01 | |
| sdague | for not network, I think we're mostly fine just ploughing forward | 19:01 |
| *** tbachman has quit IRC | 19:01 | |
| mriedem | sdague: baremetal CLI/API hasn't been deprecated yet :( https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/contrib/baremetal.py | 19:02 |
| sdague | oh, really? | 19:02 |
| sdague | I thought that had been hit | 19:02 |
| mriedem | i think you had the todo there brotha :) | 19:02 |
| SamYaple | lol | 19:02 |
| mriedem | right after i deprecated the image proxy CLIs | 19:02 |
| sdague | ok, I'll do that :) | 19:02 |
| *** tbachman has joined #openstack-nova | 19:02 | |
| mriedem | you don't have to, i can do it too, or someone, just remembering we hadn't done it yet | 19:02 |
| sdague | however, anyone using that api... is kind of insane, as it's a read only proxy that might already be broken | 19:02 |
| sdague | network... that's a more interesting case | 19:03 |
| *** sdake has joined #openstack-nova | 19:03 | |
| *** ssurana1 has joined #openstack-nova | 19:03 | |
| sdague | because there remain legit (though small number) of nova net cases, where you definitely need that stuff, and there is no alternative | 19:03 |
| mriedem | e.g. quotas/usages? | 19:04 |
| sdague | no, I mean floating ips | 19:04 |
| mriedem | oh | 19:04 |
| sdague | if you are actually using nova-net | 19:04 |
| sdague | not using network based proxy | 19:04 |
| *** ssurana has quit IRC | 19:05 | |
| mriedem | so doesn't the api have to check which network is being used and determine the 404 based on that? | 19:06 |
| sdague | so, I wonder if we do a major version bump for the novaclient release where we implement the microversion that starts 404ing on those, and tell people that the CLI stops supporting nova-net after version 6 | 19:06 |
| *** Sukhdev has joined #openstack-nova | 19:06 | |
| sdague | and just have them hang at 5.x | 19:06 |
| openstackgerrit | sean mooney proposed openstack/nova: convert libvirt dirver to use os-vif for vhost-user with ovs. https://review.openstack.org/334048 | 19:07 |
| mriedem | but doesn't os-floatingips still have to return something if you're using nova-net? | 19:07 |
| mriedem | and 404 if using neutron? | 19:07 |
| *** shaohe_feng has quit IRC | 19:10 | |
| sdague | http://specs.openstack.org/openstack/nova-specs/specs/newton/approved/deprecate-api-proxies.html#networking-api | 19:10 |
| sdague | no | 19:10 |
| sdague | we said that we'd stop responding to network calls entirely after this microversion | 19:10 |
| *** shaohe_feng has joined #openstack-nova | 19:11 | |
| sdague | and if you wanted to use nova-net, you just have to live at a frozen api level | 19:11 |
| mriedem | b/c that is also deprecated | 19:11 |
| sdague | yes | 19:11 |
| sdague | we did hash that one over in the spec quite a bit | 19:11 |
| sdague | it also simplifies cells v2 | 19:11 |
| sdague | which was part of the motivation | 19:11 |
| mriedem | yeah | 19:11 |
| *** adreznec has quit IRC | 19:12 | |
| mriedem | ....so doing a major version release when we support this microversion in novaclient is probably a good idea - clean break | 19:12 |
| mriedem | even with novaclient 6.x you could use the CLIs if you specify x-openstack-request-version or whatever | 19:13 |
| mriedem | unless we also drop those CLIs in 6.x | 19:13 |
| *** adreznec has joined #openstack-nova | 19:13 | |
| sdague | yeh, that's a thing to think about. I'm not sure | 19:15 |
| mriedem | alright, i'll think about it ~2:30am tomorrow morning | 19:15 |
| *** baoli has joined #openstack-nova | 19:15 | |
| sdague | heh | 19:15 |
| *** salv-orlando has joined #openstack-nova | 19:16 | |
| sdague | well, we could bring it up at the midcycle. Next week I'm around in bits, though there is the TC training thing so basically completely offline tue/wed | 19:16 |
| sdague | then I'm out for 2 weeks | 19:16 |
| mriedem | i'll put it on the midcycle etherpad | 19:16 |
| *** shaohe_feng has quit IRC | 19:20 | |
| openstackgerrit | Sean Dague proposed openstack/python-novaclient: Deprecate baremetal API and CLI interfaces https://review.openstack.org/334053 | 19:21 |
| *** shaohe_feng has joined #openstack-nova | 19:21 | |
| *** cdelatte has joined #openstack-nova | 19:23 | |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips-bulk https://review.openstack.org/333075 | 19:26 |
| leakypipes | figleaf: haven't wired any of that up yet :) | 19:28 |
| sdague | mriedem: https://review.openstack.org/#/c/333378/ was my solution to multiple create documentation | 19:28 |
| leakypipes | figleaf: just WIP. | 19:28 |
| openstackgerrit | sean mooney proposed openstack/nova: convert libvirt dirver to use os-vif for vhost-user with ovs. https://review.openstack.org/334048 | 19:29 |
| mriedem | sdague: yeah i was going to review that yesterday | 19:29 |
| *** shaohe_feng has quit IRC | 19:30 | |
| *** ametts has quit IRC | 19:31 | |
| *** shaohe_feng has joined #openstack-nova | 19:31 | |
| *** Jeffrey4l_ has joined #openstack-nova | 19:31 | |
| *** sacharya_ has joined #openstack-nova | 19:32 | |
| *** sai_ has joined #openstack-nova | 19:33 | |
| *** leakypipes has quit IRC | 19:34 | |
| *** Jeffrey4l has quit IRC | 19:34 | |
| *** sacharya has quit IRC | 19:34 | |
| *** sdake has quit IRC | 19:36 | |
| *** woodster_ has joined #openstack-nova | 19:36 | |
| *** sdake has joined #openstack-nova | 19:36 | |
| *** rfinn has quit IRC | 19:39 | |
| *** baoli has quit IRC | 19:40 | |
| *** shaohe_feng has quit IRC | 19:41 | |
| *** dave-mcc_ has joined #openstack-nova | 19:43 | |
| *** shaohe_feng has joined #openstack-nova | 19:44 | |
| *** dave-mccowan has quit IRC | 19:46 | |
| *** dave-mccowan has joined #openstack-nova | 19:48 | |
| *** dave-mcc_ has quit IRC | 19:50 | |
| *** lpetrut has quit IRC | 19:50 | |
| *** rfinn has joined #openstack-nova | 19:50 | |
| figleaf | leakypipes: ok, just checking | 19:50 |
| *** shaohe_feng has quit IRC | 19:51 | |
| *** ctrath has quit IRC | 19:51 | |
| *** shaohe_feng has joined #openstack-nova | 19:52 | |
| *** pacoblox has joined #openstack-nova | 19:52 | |
| *** ctrath has joined #openstack-nova | 19:53 | |
| sdague | mriedem: apparently no one has tried to land a liberty patch recently? | 19:55 |
| mriedem | sdague: erm | 19:56 |
| mriedem | idk | 19:56 |
| mriedem | only one i know of is the cve | 19:56 |
| sdague | it still defines a grenade job | 19:56 |
| sdague | but kilo is not a thing | 19:56 |
| mriedem | sdague: as of like <24 hours | 19:56 |
| mriedem | but yeah | 19:56 |
| sdague | so they all fail | 19:56 |
| mriedem | i noticed that the other day | 19:56 |
| mriedem | there are infra patches up | 19:56 |
| sdague | yeh, that should probably have been stripped earlier | 19:56 |
| sdague | are there? I didn't find them, so I pushed new ones | 19:56 |
| sdague | so, in future, those should be pulled before the branch is deleted. | 19:57 |
| mriedem | https://review.openstack.org/#/c/333910/ | 19:58 |
| mriedem | "Holding off on openstack-dev/devstack, openstack-dev/grenade, openstack-dev/pbr and openstack/requirements as their branches aren't gone quite yet." | 19:58 |
| mriedem | wah wah | 19:58 |
| mriedem | sdague: hey, tell it to the boss | 19:58 |
| sdague | ok, I think josh didn't realize that leaves things in a broken place | 19:59 |
| mriedem | and $20 says any project that wanted to keep a kilo branch isn't doing upgrade testing anyway | 19:59 |
| *** tjones has joined #openstack-nova | 20:00 | |
| sdague | it does not | 20:00 |
| sdague | I think there is a misunderstand on where the grenade jobs apply | 20:00 |
| sdague | it really doesn't matter what the grenade branches | 20:01 |
| sdague | it's about project branches :) | 20:01 |
| *** shaohe_feng has quit IRC | 20:01 | |
| mriedem | i'm sure there is cleanup that needs to happen in d-g as well | 20:01 |
| *** shaohe_feng has joined #openstack-nova | 20:02 | |
| sdague | mriedem: sure, but that won't cause liberty fails until it's done | 20:02 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add reminder comments for compute rpcapi version bump https://review.openstack.org/333989 | 20:02 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Update get_instance_diagnostics for instance objects https://review.openstack.org/333988 | 20:02 |
| *** Apoorva has quit IRC | 20:03 | |
| *** Apoorva has joined #openstack-nova | 20:04 | |
| mriedem | lascii: can't you also remove the object_compat on get_instance_diagnostics in the compute manager? | 20:07 |
| artom | mriedem, I totally get -1'ing https://review.openstack.org/#/c/304510/ until the Tempest test passes, but you also had feedback on the Tempest test itself. Would it be acceptable to make the test pass in its current incarnation, and once we're convinced the API works, rework the test to address your comments? | 20:07 |
| lascii | mriedem: not until we have a major RPC bump as I understand it | 20:08 |
| artom | mriedem, the approaching feature freeze obviously makes me want to save time where I can :) | 20:08 |
| mriedem | artom: i don't need to see +2 on the tempest test, but i need to see it passing | 20:09 |
| mriedem | lascii: but doesn't the new compute manager get rpc 4.13? | 20:09 |
| mriedem | and take the instance object? | 20:10 |
| mriedem | the rpcapi compat check is for sending to older computes, right? | 20:10 |
| artom | mriedem, awesomesauce, thanks. Today's a holiday here in Quebec, and frankly I didn't sleep enough last night to write any sort of useful code, but I'll work on it over the weekend and hopefully have a passing test for you on Monday :) | 20:10 |
| mriedem | it's been a long time since i've reviewed one of these | 20:10 |
| mriedem | artom: ok | 20:10 |
| lascii | mriedem: the new compute could still an older 4.0-4.12 message sent to it | 20:10 |
| lascii | mriedem: yeah, that's why I asked superdan about this earlier | 20:11 |
| *** shaohe_feng has quit IRC | 20:11 | |
| *** shaohe_feng has joined #openstack-nova | 20:12 | |
| *** alexpilotti has joined #openstack-nova | 20:12 | |
| lascii | also, this is now four patches I'm aware of fighting for this service version | 20:13 |
| *** cdelatte has quit IRC | 20:14 | |
| rlrossit | mriedem: is https://review.openstack.org/#/c/313654/35/doc/notification_samples/instance-delete-start.json supposed to be the same as https://wiki.openstack.org/wiki/SystemUsageData#compute.instance.delete.start.2F.end: ? because they ain't even close | 20:14 |
| mriedem | lascii: i have one of them | 20:15 |
| mriedem | and i'm rebasing b/c i lost the microversion | 20:15 |
| *** dave-mcc_ has joined #openstack-nova | 20:15 | |
| lascii | I have two, and tdurakov has one | 20:15 |
| mriedem | rlrossit: *shrug* https://wiki.openstack.org/wiki/SystemUsageData#compute.instance.delete.start.2F.end: is old | 20:16 |
| rlrossit | yeah I was trying to find a place where it had the current notifications | 20:16 |
| *** pacoblox has quit IRC | 20:17 | |
| *** dave-mccowan has quit IRC | 20:17 | |
| *** alexpilotti has quit IRC | 20:17 | |
| lascii | mriedem: you only need to rebase the API patch right? | 20:18 |
| *** tblakeslee has quit IRC | 20:18 | |
| lascii | I can +W the service version bump | 20:18 |
| mriedem | lascii: right | 20:19 |
| lascii | okay, you win the service version race | 20:19 |
| *** pacoblox has joined #openstack-nova | 20:19 | |
| mriedem | yay | 20:19 |
| *** dave-mccowan has joined #openstack-nova | 20:20 | |
| *** burgerk has quit IRC | 20:21 | |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips-bulk https://review.openstack.org/333075 | 20:21 |
| *** shaohe_feng has quit IRC | 20:22 | |
| *** shaohe_feng has joined #openstack-nova | 20:22 | |
| *** akshai has quit IRC | 20:23 | |
| *** dave-mcc_ has quit IRC | 20:23 | |
| *** tblakeslee has joined #openstack-nova | 20:26 | |
| *** Daisy has joined #openstack-nova | 20:26 | |
| *** sdake has quit IRC | 20:26 | |
| *** ctrath has quit IRC | 20:28 | |
| *** ctrath has joined #openstack-nova | 20:29 | |
| *** Daisy has quit IRC | 20:31 | |
| *** cleong has quit IRC | 20:31 | |
| *** shaohe_feng has quit IRC | 20:32 | |
| *** shaohe_feng has joined #openstack-nova | 20:32 | |
| *** sai_ has quit IRC | 20:36 | |
| *** smatzek has quit IRC | 20:37 | |
| *** imcsk8 has quit IRC | 20:39 | |
| *** imcsk8 has joined #openstack-nova | 20:39 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Bump the service version for get-me-a-network support https://review.openstack.org/318953 | 20:40 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: handle 'auto' network request in allocate_for_instance https://review.openstack.org/316275 | 20:40 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add REST API support for get me a network https://review.openstack.org/316398 | 20:40 |
| *** shaohe_feng has quit IRC | 20:42 | |
| *** shaohe_feng has joined #openstack-nova | 20:42 | |
| *** dmk0202 has quit IRC | 20:46 | |
| *** ivase has quit IRC | 20:46 | |
| mriedem | lascii: thanks for hitting those again | 20:46 |
| mriedem | you are a true gentleman | 20:46 |
| *** ivase has joined #openstack-nova | 20:47 | |
| lascii | heh, np | 20:47 |
| mriedem | "Virtual interface unplugin failed" | 20:49 |
| mriedem | unplugin isn't a word, right? | 20:49 |
| *** rlrossit_ has joined #openstack-nova | 20:49 | |
| mriedem | s/unplugin/unplug/ | 20:49 |
| *** ivase_ has joined #openstack-nova | 20:49 | |
| *** ivase__ has joined #openstack-nova | 20:50 | |
| *** ivase has quit IRC | 20:51 | |
| mriedem | sdague: are you half in the bag yet? | 20:51 |
| *** rwmjones has quit IRC | 20:52 | |
| *** tbachman_ has joined #openstack-nova | 20:52 | |
| *** shaohe_feng has quit IRC | 20:52 | |
| *** rlrossit has quit IRC | 20:52 | |
| *** shaohe_feng has joined #openstack-nova | 20:53 | |
| *** rfinn has quit IRC | 20:53 | |
| *** ivase_ has quit IRC | 20:54 | |
| openstackgerrit | Mark Doffman proposed openstack/nova: PoC: Scheduler: Placement using random weights distribution. https://review.openstack.org/287397 | 20:54 |
| *** tbachman has quit IRC | 20:54 | |
| *** tbachman_ is now known as tbachman | 20:54 | |
| *** ivase__ has quit IRC | 20:54 | |
| mikal | mriedem: I've been off sick the last couple of days | 20:55 |
| doffm | rlrossit_: ^^ For testing fun when you have alot of retry failires. | 20:55 |
| mikal | mriedem: so only just noticed the vendordata conversation above | 20:55 |
| mikal | mriedem: I agree that we shouldn't rush it in by the way | 20:55 |
| *** rfolco has quit IRC | 20:55 | |
| mikal | mriedem: but am unsure what other testing you want? Functional I presume? | 20:55 |
| *** rwmjones has joined #openstack-nova | 20:56 | |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips-bulk https://review.openstack.org/333075 | 20:57 |
| *** smatzek has joined #openstack-nova | 20:57 | |
| mriedem | mikal: yeah i think we should have functional testing in tree to be serious about getting that in | 20:57 |
| *** alexpilotti has joined #openstack-nova | 20:58 | |
| mikal | mriedem: yeah, that's fine. I've already told sdague I'd work on that next. | 20:58 |
| mikal | mriedem: then I got the man flu. So I'll get to it when I can. | 20:58 |
| mikal | mriedem: probably sometime next week. | 20:58 |
| mikal | mriedem: if that means it misses the deadline then so be it | 20:58 |
| mriedem | mikal: how is that different from the woman flu? | 20:59 |
| mikal | mriedem: worst case its all queued up ready to land in Otaca on day 1 | 20:59 |
| mikal | mriedem: have you not seen the memes? | 20:59 |
| mriedem | i don't internet | 20:59 |
| mikal | mriedem: do they not have man flu in your country? | 20:59 |
| mikal | OMG | 20:59 |
| mikal | Please hold | 20:59 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Implement stable disk device instance rescues https://review.openstack.org/270289 | 20:59 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: compute: Allow volume backed instances to be rescued by capable virt drivers https://review.openstack.org/270288 | 20:59 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Add boot ordering to individual disks https://review.openstack.org/270286 | 20:59 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: virt: Provide block_device_info during a rescue https://review.openstack.org/270287 | 20:59 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: image_meta: Add hw_rescue_device and hw_rescue_bus https://review.openstack.org/270285 | 20:59 |
| *** kylek3h has quit IRC | 20:59 | |
| mriedem | is that where guys turn into big cry babies when they are sick? | 21:00 |
| mikal | Yep -- http://manflu.info/index.htm | 21:00 |
| *** burt has quit IRC | 21:00 | |
| mikal | LOL, "printable snotometer" | 21:00 |
| mriedem | "The Woman's Role as Carer" | 21:01 |
| mriedem | do australians not know that 'caregiver' is a word? | 21:01 |
| lyarwood | mriedem: assuming the UK hasn't sunk into the atlantic come Monday will I still able to get the above non-prio feature merged? ./me can't recall if the deadline is EOB on Monday or just Monday itself. | 21:02 |
| *** lchen_ has quit IRC | 21:02 | |
| mriedem | lyarwood: 6/30 | 21:02 |
| lyarwood | so not even monday | 21:02 |
| mriedem | thursday | 21:02 |
| lyarwood | my bad | 21:02 |
| lyarwood | thanks | 21:02 |
| *** shaohe_feng has quit IRC | 21:03 | |
| mriedem | mikal: idk man, when my wife gets sick she coughs loudly and constantly, and not into her elbow like all RNs should | 21:03 |
| *** lchen_ has joined #openstack-nova | 21:03 | |
| mikal | mriedem: my wife has actually declared herself dead for the duration | 21:03 |
| mikal | mriedem: which actually might be a fair call, she's not making womanflu look like much fun | 21:03 |
| mriedem | just load up on whatever they use to make meth | 21:04 |
| mriedem | you'll be fine | 21:04 |
| mikal | So anyways, I know I need to do docs and functional tests, I hope to have them done early next week | 21:04 |
| mikal | If that means this slips to Otaca, I am actually ok with that. I'd rather it be right than rushed. | 21:04 |
| mikal | As long as we don't remove the old thing before the new thing is a thing. Thing. | 21:05 |
| *** Daisy has joined #openstack-nova | 21:05 | |
| *** shaohe_feng has joined #openstack-nova | 21:05 | |
| mriedem | mikal: that's what we talked about today, and agreed we wouldn't do | 21:06 |
| *** rlrossit_ has quit IRC | 21:06 | |
| mriedem | remove the old thing before the new thing is said thing | 21:06 |
| mikal | Cool | 21:06 |
| mikal | So we're on the same page | 21:06 |
| mikal | Hopefully I'll still be sick at the mid-cycle so I can infect you | 21:06 |
| mikal | And then you can judge this manflu for yourself | 21:06 |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Improve help text for compute running_deleted_opts https://review.openstack.org/321320 | 21:06 |
| *** smatzek has quit IRC | 21:08 | |
| *** Daisy has quit IRC | 21:09 | |
| mriedem | mikal: i look forward to it | 21:10 |
| *** mriedem has quit IRC | 21:11 | |
| *** shaohe_feng has quit IRC | 21:13 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Refactor block_device_mapping handling during boot https://review.openstack.org/334094 | 21:15 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add block_device_mappings to BuildRequest https://review.openstack.org/334095 | 21:15 |
| *** ctrath has quit IRC | 21:17 | |
| *** shaohe_feng has joined #openstack-nova | 21:17 | |
| *** jbernard has quit IRC | 21:19 | |
| *** efried has quit IRC | 21:22 | |
| *** shaohe_feng has quit IRC | 21:23 | |
| *** shaohe_feng has joined #openstack-nova | 21:23 | |
| *** lascii is now known as alaski | 21:23 | |
| *** aysyd has quit IRC | 21:25 | |
| *** ducttape_ has quit IRC | 21:29 | |
| *** shaohe_feng has quit IRC | 21:33 | |
| *** tbachman_ has joined #openstack-nova | 21:34 | |
| *** shaohe_feng has joined #openstack-nova | 21:34 | |
| *** tbachman has quit IRC | 21:34 | |
| *** tbachman_ is now known as tbachman | 21:34 | |
| *** tbachman has quit IRC | 21:40 | |
| *** sacharya_ has quit IRC | 21:42 | |
| *** sacharya has joined #openstack-nova | 21:43 | |
| *** claudiub|2 has joined #openstack-nova | 21:44 | |
| *** shaohe_feng has quit IRC | 21:44 | |
| *** shaohe_feng has joined #openstack-nova | 21:44 | |
| *** sacharya_ has joined #openstack-nova | 21:44 | |
| *** kylek3h has joined #openstack-nova | 21:44 | |
| *** tbachman has joined #openstack-nova | 21:45 | |
| *** sacharya has quit IRC | 21:47 | |
| *** thorst_ has quit IRC | 21:49 | |
| *** tbachman has quit IRC | 21:50 | |
| *** thorst has joined #openstack-nova | 21:50 | |
| *** mdrabe has quit IRC | 21:50 | |
| *** bauwser is now known as bauza | 21:52 | |
| *** bauza is now known as bauzas | 21:52 | |
| *** figleaf is now known as edleafe | 21:52 | |
| *** shaohe_feng has quit IRC | 21:54 | |
| *** shaohe_feng has joined #openstack-nova | 21:55 | |
| *** tbachman has joined #openstack-nova | 21:56 | |
| *** Apoorva has quit IRC | 21:58 | |
| *** Apoorva has joined #openstack-nova | 21:58 | |
| *** thorst has quit IRC | 21:58 | |
| *** tbachman has quit IRC | 21:59 | |
| *** tbachman has joined #openstack-nova | 21:59 | |
| *** shaohe_feng has quit IRC | 22:04 | |
| *** mmandel has quit IRC | 22:04 | |
| *** shaohe_feng has joined #openstack-nova | 22:05 | |
| *** dimtruck is now known as zz_dimtruck | 22:05 | |
| *** tbachman has quit IRC | 22:06 | |
| *** sthillma has quit IRC | 22:06 | |
| *** tbachman has joined #openstack-nova | 22:07 | |
| *** sdague has quit IRC | 22:08 | |
| *** tbachman_ has joined #openstack-nova | 22:10 | |
| *** tbachman has quit IRC | 22:12 | |
| *** pradk has quit IRC | 22:12 | |
| *** tblakeslee has quit IRC | 22:12 | |
| *** rdo has quit IRC | 22:14 | |
| *** shaohe_feng has quit IRC | 22:14 | |
| *** rtheis has quit IRC | 22:14 | |
| *** shaohe_feng has joined #openstack-nova | 22:15 | |
| *** p01nt3r75 has joined #openstack-nova | 22:15 | |
| *** rdo has joined #openstack-nova | 22:16 | |
| *** tbachman_ has quit IRC | 22:16 | |
| *** BjoernT has quit IRC | 22:17 | |
| *** harlowja has quit IRC | 22:17 | |
| *** catintheroof has quit IRC | 22:18 | |
| *** tbachman has joined #openstack-nova | 22:20 | |
| *** thumpba_ has quit IRC | 22:24 | |
| *** thumpba has joined #openstack-nova | 22:24 | |
| *** thumpba has quit IRC | 22:24 | |
| *** shaohe_feng has quit IRC | 22:25 | |
| *** thumpba has joined #openstack-nova | 22:25 | |
| *** thumpba has quit IRC | 22:25 | |
| *** thumpba has joined #openstack-nova | 22:25 | |
| *** shaohe_feng has joined #openstack-nova | 22:25 | |
| *** thumpba has quit IRC | 22:26 | |
| *** thumpba has joined #openstack-nova | 22:26 | |
| *** thumpba has quit IRC | 22:27 | |
| *** thumpba has joined #openstack-nova | 22:27 | |
| *** thumpba has quit IRC | 22:27 | |
| *** thumpba has joined #openstack-nova | 22:28 | |
| *** thumpba has quit IRC | 22:28 | |
| *** thumpba has joined #openstack-nova | 22:28 | |
| *** thumpba has quit IRC | 22:29 | |
| *** thumpba has joined #openstack-nova | 22:29 | |
| *** harlowja has joined #openstack-nova | 22:30 | |
| *** thumpba has quit IRC | 22:30 | |
| *** thumpba_ has joined #openstack-nova | 22:30 | |
| *** tbachman_ has joined #openstack-nova | 22:30 | |
| *** thumpba_ has quit IRC | 22:30 | |
| *** pwnall has quit IRC | 22:31 | |
| *** edmondsw has quit IRC | 22:32 | |
| *** tbachman has quit IRC | 22:32 | |
| *** tbachman has joined #openstack-nova | 22:33 | |
| *** kylek3h has quit IRC | 22:33 | |
| *** tbachman_ has quit IRC | 22:35 | |
| *** shaohe_feng has quit IRC | 22:35 | |
| *** shaohe_feng has joined #openstack-nova | 22:35 | |
| *** raunak has quit IRC | 22:37 | |
| *** mriedem has joined #openstack-nova | 22:41 | |
| *** thumpba has joined #openstack-nova | 22:45 | |
| *** shaohe_feng has quit IRC | 22:45 | |
| *** ducttape_ has joined #openstack-nova | 22:45 | |
| *** thumpba has quit IRC | 22:45 | |
| *** mrhillsman is now known as mrhillsman_ | 22:46 | |
| *** yamahata has quit IRC | 22:47 | |
| *** shaohe_feng has joined #openstack-nova | 22:48 | |
| *** sacharya has joined #openstack-nova | 22:48 | |
| *** sacharya_ has quit IRC | 22:49 | |
| *** pacoblox has quit IRC | 22:50 | |
| *** raunak has joined #openstack-nova | 22:52 | |
| *** ducttape_ has quit IRC | 22:55 | |
| *** shaohe_feng has quit IRC | 22:55 | |
| *** MikeG451 has quit IRC | 22:56 | |
| *** shaohe_feng has joined #openstack-nova | 22:56 | |
| *** thorst has joined #openstack-nova | 22:57 | |
| *** Apoorva has quit IRC | 22:57 | |
| *** BjoernT has joined #openstack-nova | 22:58 | |
| *** hemna is now known as hemnavacation | 22:58 | |
| *** salv-orl_ has joined #openstack-nova | 22:58 | |
| *** BjoernT_ has joined #openstack-nova | 23:00 | |
| *** BjoernT has quit IRC | 23:01 | |
| *** salv-orlando has quit IRC | 23:01 | |
| *** markvoelker has quit IRC | 23:01 | |
| *** yamahata has joined #openstack-nova | 23:02 | |
| *** markvoelker has joined #openstack-nova | 23:02 | |
| *** thorst has quit IRC | 23:04 | |
| *** shaohe_feng has quit IRC | 23:06 | |
| *** thumpba has joined #openstack-nova | 23:06 | |
| *** thumpba has quit IRC | 23:06 | |
| *** shaohe_feng has joined #openstack-nova | 23:06 | |
| *** ducttape_ has joined #openstack-nova | 23:07 | |
| *** itlinux has quit IRC | 23:09 | |
| *** shaohe_feng has quit IRC | 23:16 | |
| *** shaohe_feng has joined #openstack-nova | 23:16 | |
| *** sacharya has quit IRC | 23:18 | |
| *** smatzek has joined #openstack-nova | 23:18 | |
| *** BjoernT_ is now known as BjoernT | 23:21 | |
| *** markvoelker has quit IRC | 23:26 | |
| *** shaohe_feng has quit IRC | 23:26 | |
| *** shaohe_feng has joined #openstack-nova | 23:26 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix error message for VirtualInterfaceUnplugException https://review.openstack.org/334114 | 23:28 |
| *** smatzek has quit IRC | 23:30 | |
| *** kylek3h has joined #openstack-nova | 23:34 | |
| *** shaohe_feng has quit IRC | 23:36 | |
| *** shaohe_feng has joined #openstack-nova | 23:37 | |
| *** superdan is now known as dansmith | 23:38 | |
| *** kylek3h has quit IRC | 23:39 | |
| *** sdague has joined #openstack-nova | 23:43 | |
| *** tjones has left #openstack-nova | 23:44 | |
| *** shaohe_feng has quit IRC | 23:47 | |
| *** salv-orl_ has quit IRC | 23:47 | |
| *** shaohe_feng has joined #openstack-nova | 23:48 | |
| *** woodster_ has quit IRC | 23:49 | |
| *** adam_g has quit IRC | 23:50 | |
| *** ducttape_ has quit IRC | 23:53 | |
| *** tbachman_ has joined #openstack-nova | 23:54 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: Microversion 2.32 adds keypairs pagination support https://review.openstack.org/284868 | 23:55 |
| *** tbachman has quit IRC | 23:56 | |
| *** tbachman_ is now known as tbachman | 23:56 | |
| *** shaohe_feng has quit IRC | 23:57 | |
| *** shaohe_feng has joined #openstack-nova | 23:58 | |
| *** sdake has joined #openstack-nova | 23:58 | |
| *** Sukhdev has quit IRC | 23:58 | |
| *** thumpba has joined #openstack-nova | 23:59 | |
| *** thumpba has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!