| openstackgerrit | Joshua Harlow proposed openstack/nova: Update nova api.auth tests to work with newer oslo.context https://review.openstack.org/395261 | 00:00 |
|---|---|---|
| *** browne has quit IRC | 00:02 | |
| *** jamesdenton has joined #openstack-nova | 00:06 | |
| *** esberglu has quit IRC | 00:10 | |
| *** hongbin has quit IRC | 00:10 | |
| *** jamesdenton has quit IRC | 00:11 | |
| *** thorst_ has joined #openstack-nova | 00:12 | |
| openstackgerrit | Sujitha proposed openstack/nova: Document for zero downtime upgrade process https://review.openstack.org/395269 | 00:13 |
| *** ijw has joined #openstack-nova | 00:14 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Add zero down time upgrade steps https://review.openstack.org/393931 | 00:14 |
| *** catintheroof has quit IRC | 00:14 | |
| *** catintheroof has joined #openstack-nova | 00:15 | |
| *** portdirect has quit IRC | 00:16 | |
| *** oomichi_ has joined #openstack-nova | 00:17 | |
| openstackgerrit | Michael Glaser proposed openstack/nova: Transform instance.resize_confirm notification https://review.openstack.org/390654 | 00:18 |
| *** ijw has quit IRC | 00:19 | |
| *** portdirect has joined #openstack-nova | 00:19 | |
| *** catintheroof has quit IRC | 00:19 | |
| *** Dong_Zhi has joined #openstack-nova | 00:19 | |
| *** david-lyle has quit IRC | 00:20 | |
| *** oomichi has quit IRC | 00:22 | |
| *** Dong_Zhi has quit IRC | 00:24 | |
| *** Dong_Zhi has joined #openstack-nova | 00:25 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix a 'port' parameter in os-consoles.inc https://review.openstack.org/384742 | 00:28 |
| *** oomichi_ has quit IRC | 00:28 | |
| *** jwcroppe has joined #openstack-nova | 00:29 | |
| *** gmann has joined #openstack-nova | 00:30 | |
| *** jwcroppe has quit IRC | 00:30 | |
| *** annegentle has quit IRC | 00:30 | |
| *** jwcroppe has joined #openstack-nova | 00:30 | |
| *** Dong_Zhi has quit IRC | 00:32 | |
| openstackgerrit | Moshe Levi proposed openstack/nova-specs: User-controlled SR-IOV ports allocation https://review.openstack.org/182242 | 00:34 |
| *** jamesdenton has joined #openstack-nova | 00:34 | |
| *** browne has joined #openstack-nova | 00:38 | |
| *** gmann has quit IRC | 00:39 | |
| *** moshele has quit IRC | 00:40 | |
| *** oomichi has joined #openstack-nova | 00:43 | |
| *** gmann has joined #openstack-nova | 00:46 | |
| *** hoangcx has joined #openstack-nova | 00:48 | |
| *** amotoki has quit IRC | 00:49 | |
| *** tovin07 has quit IRC | 00:49 | |
| *** hieulq has quit IRC | 00:49 | |
| *** womygod has joined #openstack-nova | 00:51 | |
| *** hieulq has joined #openstack-nova | 00:52 | |
| *** diogogmt has quit IRC | 00:52 | |
| *** tovin07 has joined #openstack-nova | 00:54 | |
| *** ducttape_ has joined #openstack-nova | 00:54 | |
| *** Zer0Byte__ has quit IRC | 00:54 | |
| *** jamesdenton has quit IRC | 00:55 | |
| *** ijw has joined #openstack-nova | 00:55 | |
| *** rama_y has quit IRC | 00:55 | |
| *** mlavalle has quit IRC | 00:57 | |
| *** maticue_ has quit IRC | 00:57 | |
| *** tuanluong has joined #openstack-nova | 00:57 | |
| *** gmann_ has joined #openstack-nova | 01:02 | |
| *** gmann has quit IRC | 01:03 | |
| *** ducttape_ has quit IRC | 01:05 | |
| *** Swami_ has quit IRC | 01:06 | |
| *** Swami has quit IRC | 01:06 | |
| *** hoonetorg has quit IRC | 01:11 | |
| *** bezilla has quit IRC | 01:16 | |
| *** gmann has joined #openstack-nova | 01:17 | |
| *** jamesdenton has joined #openstack-nova | 01:18 | |
| *** gmann_ has quit IRC | 01:19 | |
| *** markvoelker has quit IRC | 01:22 | |
| *** Sukhdev has quit IRC | 01:23 | |
| *** hoonetorg has joined #openstack-nova | 01:24 | |
| *** stewie925 has quit IRC | 01:30 | |
| *** gmann_ has joined #openstack-nova | 01:30 | |
| *** gmann has quit IRC | 01:33 | |
| *** yuntongjin has joined #openstack-nova | 01:33 | |
| *** diogogmt has joined #openstack-nova | 01:39 | |
| *** Daisy has joined #openstack-nova | 01:39 | |
| *** Daisy has quit IRC | 01:39 | |
| *** Daisy has joined #openstack-nova | 01:39 | |
| *** Apoorva has quit IRC | 01:42 | |
| *** amotoki has joined #openstack-nova | 01:43 | |
| *** browne has quit IRC | 01:44 | |
| *** jamesdenton has quit IRC | 01:46 | |
| *** zhangyufei has joined #openstack-nova | 01:47 | |
| *** armax has quit IRC | 01:47 | |
| *** jamesdenton has joined #openstack-nova | 01:48 | |
| *** amotoki has quit IRC | 01:48 | |
| *** ayoung has quit IRC | 01:54 | |
| *** smatzek has joined #openstack-nova | 01:56 | |
| *** jamesdenton has quit IRC | 01:57 | |
| oomichi | mriedem: hi, can you take a look at https://review.openstack.org/#/c/383049 ? That seems different API behaviors between backend hypervisors | 02:00 |
| mriedem | oomichi: that's because the hyper-v driver doesn't implement the same wait code that the libvirt driver does, which is waiting for neutron to ack that the vifs are plugged and the ports are active, which then continues the spawn() operation in the virt driver before the vm_state goes to ACTIVE | 02:03 |
| mriedem | i guess that's noted in the bug report: | 02:04 |
| mriedem | "Currently, the nova Hyper-V driver does not wait for neutron vif plug events (or, in case of other drivers, nova-compute can be configured not to wait for neutron vif plug events)," | 02:04 |
| mriedem | ideally the hyperv driver would implement that wait code fo the vif plugged event | 02:04 |
| *** ijw has quit IRC | 02:04 | |
| oomichi | mriedem: yeah, I agree with you | 02:04 |
| *** nic has quit IRC | 02:05 | |
| oomichi | mriedem: the bug is registered as tempest bug anyways ;) | 02:05 |
| *** ducttape_ has joined #openstack-nova | 02:05 | |
| mriedem | clearly, CLEARLY, this is all tempest's fault. :) | 02:05 |
| mriedem | so, as claudiu pointed out in the review, and bug report, you can configure nova-compute to not wait for the vif plugged events, but if you're using neutron and have a virt driver that supports that, i don't know why you'd do that | 02:06 |
| oomichi | mriedem: I cannot vote on that :) | 02:06 |
| mriedem | but that is a configuration change on the compute that impacts the behavior of the api | 02:06 |
| mriedem | so for libvirt the tests are probably working fine b/c when we get to that point the ports are already active b/c we've waited for the server to be active, which waited for the ports to be active | 02:07 |
| oomichi | mriedem: yeah, actually that seems difficult to make API behaviors same between drivers | 02:07 |
| mriedem | it's probably not unreasonable to expect the client to wait for the port status to be active before operations, like associating a floating IP to the port, can be performed on it | 02:08 |
| *** unicell has quit IRC | 02:08 | |
| mriedem | what would happen if we associated the floating IP to the server via neutron APIs rather than the proxy APIs in nova? would neutron require the port to be active? | 02:08 |
| *** ljxiash has joined #openstack-nova | 02:09 | |
| *** ducttape_ has quit IRC | 02:10 | |
| oomichi | hopefully, all cilents could be waiting for the port status as the patch, but actually it is difficult to expect that because tempest doesn't do that. | 02:12 |
| mriedem | oomichi: i just dumped some words into that review also | 02:17 |
| *** david-lyle has joined #openstack-nova | 02:20 | |
| *** david-lyle has quit IRC | 02:20 | |
| mriedem | oomichi: while i've got you, can you take a look at this https://review.openstack.org/#/c/394524/ | 02:22 |
| *** david-lyle has joined #openstack-nova | 02:22 | |
| *** markvoelker has joined #openstack-nova | 02:23 | |
| *** zhangyufei has quit IRC | 02:23 | |
| *** zhangyufei has joined #openstack-nova | 02:23 | |
| *** zhangyufei has quit IRC | 02:23 | |
| *** zhangyufei has joined #openstack-nova | 02:24 | |
| *** sach-m has quit IRC | 02:24 | |
| *** sach-m has joined #openstack-nova | 02:25 | |
| *** markvoelker has quit IRC | 02:28 | |
| *** zhangyufei has quit IRC | 02:28 | |
| *** sach-m has quit IRC | 02:30 | |
| *** Zer0Byte__ has joined #openstack-nova | 02:37 | |
| *** thorst_ has quit IRC | 02:39 | |
| *** thorst_ has joined #openstack-nova | 02:40 | |
| *** mriedem has quit IRC | 02:42 | |
| *** Zer0Byte__ has quit IRC | 02:43 | |
| *** Zer0Byte__ has joined #openstack-nova | 02:43 | |
| *** Zer0Byte__ has quit IRC | 02:45 | |
| *** zhangyufei has joined #openstack-nova | 02:45 | |
| *** Zer0Byte__ has joined #openstack-nova | 02:45 | |
| *** thorst_ has quit IRC | 02:48 | |
| *** Zer0Byte__ has quit IRC | 02:50 | |
| *** Zer0Byte__ has joined #openstack-nova | 02:54 | |
| openstackgerrit | Choe, Cheng-Dae proposed openstack/nova: Query availability zone in a single query https://review.openstack.org/389759 | 02:56 |
| *** bkopilov has quit IRC | 02:58 | |
| *** ljxiash_ has joined #openstack-nova | 02:59 | |
| *** yulong|away has quit IRC | 03:01 | |
| *** yulong|away has joined #openstack-nova | 03:02 | |
| *** ljxiash has quit IRC | 03:02 | |
| *** yulong|away has quit IRC | 03:05 | |
| *** yuntongjin has quit IRC | 03:18 | |
| *** Zer0Byte__ has quit IRC | 03:20 | |
| *** ducttape_ has joined #openstack-nova | 03:21 | |
| *** haplo37 has quit IRC | 03:22 | |
| *** gcb has quit IRC | 03:22 | |
| *** haplo37 has joined #openstack-nova | 03:30 | |
| *** ducttape_ has quit IRC | 03:31 | |
| *** ducttape_ has joined #openstack-nova | 03:34 | |
| *** smatzek has quit IRC | 03:36 | |
| *** gcb has joined #openstack-nova | 03:38 | |
| *** dave-mccowan has quit IRC | 03:38 | |
| *** Daisy has quit IRC | 03:39 | |
| *** Daisy has joined #openstack-nova | 03:40 | |
| *** houming has joined #openstack-nova | 03:41 | |
| *** Daisy has quit IRC | 03:41 | |
| *** hongbin has joined #openstack-nova | 03:41 | |
| *** Daisy has joined #openstack-nova | 03:42 | |
| *** portdirect has quit IRC | 03:44 | |
| *** yamahata has joined #openstack-nova | 03:46 | |
| *** thorst_ has joined #openstack-nova | 03:46 | |
| *** Daisy has quit IRC | 03:47 | |
| *** Daisy has joined #openstack-nova | 03:48 | |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Fix hypervisor-stats show invalid info of baremetal resource https://review.openstack.org/391415 | 03:51 |
| *** hongbin has quit IRC | 03:51 | |
| *** adisky has joined #openstack-nova | 03:52 | |
| *** Daisy has quit IRC | 03:52 | |
| *** thorst_ has quit IRC | 03:54 | |
| *** Daisy has joined #openstack-nova | 03:54 | |
| openstackgerrit | John Hua proposed openstack/nova: WIP: fix memory usage miscalculation https://review.openstack.org/394857 | 03:57 |
| *** Daisy has quit IRC | 03:58 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP] Support tag instances when boot https://review.openstack.org/394321 | 03:59 |
| *** Daisy has joined #openstack-nova | 04:00 | |
| *** ljxiash_ has quit IRC | 04:02 | |
| *** ljxiash has joined #openstack-nova | 04:02 | |
| *** Daisy has quit IRC | 04:04 | |
| *** gmann_ has quit IRC | 04:04 | |
| *** Daisy has joined #openstack-nova | 04:06 | |
| *** ljxiash has quit IRC | 04:07 | |
| *** kylek3h has quit IRC | 04:08 | |
| *** Daisy has quit IRC | 04:10 | |
| *** fabian4 has quit IRC | 04:13 | |
| *** Daisy has joined #openstack-nova | 04:16 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/395192 | 04:20 |
| *** fabian4 has joined #openstack-nova | 04:20 | |
| *** Daisy has quit IRC | 04:21 | |
| *** Daisy has joined #openstack-nova | 04:22 | |
| *** sudipto_ has joined #openstack-nova | 04:22 | |
| *** sudipto has joined #openstack-nova | 04:22 | |
| macsz | auggy: i modified the bug triage wiki and updated the link to bug dashboard for the time being; until infra gets to host it the address is http://172.99.106.101 | 04:22 |
| *** Sukhdev has joined #openstack-nova | 04:22 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements https://review.openstack.org/395347 | 04:22 |
| *** alex_xu has quit IRC | 04:23 | |
| *** Dong_Zhi has joined #openstack-nova | 04:23 | |
| *** markvoelker has joined #openstack-nova | 04:24 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/395371 | 04:24 |
| *** Dong_Zhi_ has joined #openstack-nova | 04:24 | |
| *** Dong_Zhi has quit IRC | 04:25 | |
| *** Dong_Zhi__ has joined #openstack-nova | 04:25 | |
| *** fragatin_ has joined #openstack-nova | 04:26 | |
| *** hshiina has joined #openstack-nova | 04:26 | |
| *** ljxiash has joined #openstack-nova | 04:27 | |
| *** bks has joined #openstack-nova | 04:28 | |
| *** rama_y has joined #openstack-nova | 04:28 | |
| *** markvoelker has quit IRC | 04:28 | |
| *** Dong_Zhi_ has quit IRC | 04:29 | |
| *** Daisy_ has joined #openstack-nova | 04:29 | |
| *** alex_xu has joined #openstack-nova | 04:29 | |
| *** houming has quit IRC | 04:29 | |
| *** fragatina has quit IRC | 04:29 | |
| *** GB21 has joined #openstack-nova | 04:30 | |
| *** Daisy_ has quit IRC | 04:30 | |
| *** nicolasbock has quit IRC | 04:30 | |
| *** Daisy_ has joined #openstack-nova | 04:31 | |
| *** fragatin_ has quit IRC | 04:31 | |
| *** ljxiash has quit IRC | 04:31 | |
| *** yuntongjin has joined #openstack-nova | 04:32 | |
| *** Daisy has quit IRC | 04:32 | |
| *** houming has joined #openstack-nova | 04:33 | |
| *** gmann has joined #openstack-nova | 04:33 | |
| *** Dong_Zhi__ has quit IRC | 04:33 | |
| *** Daisy_ has quit IRC | 04:35 | |
| *** links has joined #openstack-nova | 04:38 | |
| *** Daisy has joined #openstack-nova | 04:43 | |
| *** noama_ has joined #openstack-nova | 04:44 | |
| *** sandanar has joined #openstack-nova | 04:45 | |
| *** bkopilov has joined #openstack-nova | 04:47 | |
| *** Daisy has quit IRC | 04:47 | |
| *** sudipto_ has quit IRC | 04:48 | |
| *** sudipto has quit IRC | 04:48 | |
| *** GB21 has quit IRC | 04:49 | |
| *** Daisy has joined #openstack-nova | 04:49 | |
| *** dikonoor has joined #openstack-nova | 04:50 | |
| *** prateek has joined #openstack-nova | 04:51 | |
| *** sree has joined #openstack-nova | 04:52 | |
| *** thorst_ has joined #openstack-nova | 04:52 | |
| *** Daisy has quit IRC | 04:53 | |
| *** Daisy has joined #openstack-nova | 04:55 | |
| *** thorst_ has quit IRC | 04:59 | |
| *** prashkre has joined #openstack-nova | 04:59 | |
| *** Daisy has quit IRC | 05:00 | |
| *** GB21 has joined #openstack-nova | 05:01 | |
| *** takashin has left #openstack-nova | 05:01 | |
| *** Daisy has joined #openstack-nova | 05:01 | |
| *** lpetrut has joined #openstack-nova | 05:01 | |
| *** pbandark has joined #openstack-nova | 05:02 | |
| *** Daisy has quit IRC | 05:03 | |
| *** Daisy has joined #openstack-nova | 05:03 | |
| *** fragatina has joined #openstack-nova | 05:03 | |
| *** gmann_ has joined #openstack-nova | 05:04 | |
| *** gmann has quit IRC | 05:06 | |
| *** dikonoor has quit IRC | 05:06 | |
| *** fragatina has quit IRC | 05:08 | |
| *** kylek3h has joined #openstack-nova | 05:09 | |
| *** rama_y has quit IRC | 05:09 | |
| *** rama_y has joined #openstack-nova | 05:10 | |
| *** claudiub has joined #openstack-nova | 05:11 | |
| *** kylek3h has quit IRC | 05:15 | |
| *** Hosam has joined #openstack-nova | 05:15 | |
| *** alex_xu has quit IRC | 05:17 | |
| *** alex_xu has joined #openstack-nova | 05:18 | |
| *** noama_ has quit IRC | 05:18 | |
| *** houming has quit IRC | 05:20 | |
| *** houming has joined #openstack-nova | 05:21 | |
| *** gcb has quit IRC | 05:22 | |
| *** david-lyle has quit IRC | 05:22 | |
| *** fragatina has joined #openstack-nova | 05:22 | |
| *** gcb has joined #openstack-nova | 05:22 | |
| *** david-lyle has joined #openstack-nova | 05:23 | |
| *** Daisy has quit IRC | 05:25 | |
| *** Daisy has joined #openstack-nova | 05:26 | |
| *** fragatina has quit IRC | 05:27 | |
| *** sudipto has joined #openstack-nova | 05:31 | |
| *** Daisy_ has joined #openstack-nova | 05:31 | |
| *** sudipto_ has joined #openstack-nova | 05:31 | |
| *** prashkre has quit IRC | 05:33 | |
| *** manasm has joined #openstack-nova | 05:34 | |
| *** Daisy has quit IRC | 05:34 | |
| *** ljxiash has joined #openstack-nova | 05:35 | |
| *** ducttape_ has quit IRC | 05:36 | |
| *** ducttape_ has joined #openstack-nova | 05:36 | |
| *** dikonoor has joined #openstack-nova | 05:37 | |
| *** gcb has quit IRC | 05:37 | |
| *** gcb has joined #openstack-nova | 05:38 | |
| *** Daisy_ has quit IRC | 05:39 | |
| *** Daisy has joined #openstack-nova | 05:40 | |
| *** ducttape_ has quit IRC | 05:40 | |
| *** ducttape_ has joined #openstack-nova | 05:45 | |
| *** GB21 has quit IRC | 05:48 | |
| *** ducttape_ has quit IRC | 05:50 | |
| *** vladani has joined #openstack-nova | 05:51 | |
| *** vladani_ has quit IRC | 05:51 | |
| *** lpetrut has quit IRC | 05:52 | |
| *** gcb has quit IRC | 05:54 | |
| *** ratailor has joined #openstack-nova | 05:56 | |
| *** thorst_ has joined #openstack-nova | 05:56 | |
| *** oanson has joined #openstack-nova | 05:59 | |
| *** GB21 has joined #openstack-nova | 06:01 | |
| *** vladani_ has joined #openstack-nova | 06:01 | |
| *** vladani has quit IRC | 06:02 | |
| *** gmann has joined #openstack-nova | 06:03 | |
| *** thorst_ has quit IRC | 06:04 | |
| *** klkumar has joined #openstack-nova | 06:04 | |
| *** gmann_ has quit IRC | 06:05 | |
| *** haplo37_ has quit IRC | 06:06 | |
| *** GB21 has quit IRC | 06:06 | |
| *** gcb has joined #openstack-nova | 06:06 | |
| *** GB21 has joined #openstack-nova | 06:06 | |
| *** GB21 has quit IRC | 06:07 | |
| *** GB21 has joined #openstack-nova | 06:07 | |
| *** lpetrut has joined #openstack-nova | 06:07 | |
| *** links has quit IRC | 06:11 | |
| *** eandersson_ has quit IRC | 06:11 | |
| *** haplo37 has quit IRC | 06:13 | |
| *** houming has quit IRC | 06:14 | |
| openstackgerrit | NITIN GUPTA proposed openstack/python-novaclient: Modified "nova delete" command usage and test case. https://review.openstack.org/395011 | 06:16 |
| *** Daisy has quit IRC | 06:17 | |
| *** sahid has joined #openstack-nova | 06:18 | |
| *** houming has joined #openstack-nova | 06:21 | |
| *** haplo37 has joined #openstack-nova | 06:23 | |
| openstackgerrit | NITIN GUPTA proposed openstack/python-novaclient: Modified "nova delete" command usage and test case. https://review.openstack.org/395011 | 06:23 |
| *** markvoelker has joined #openstack-nova | 06:24 | |
| *** lpetrut has quit IRC | 06:25 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP] Support tag instances when boot https://review.openstack.org/394321 | 06:25 |
| *** haplo37_ has joined #openstack-nova | 06:25 | |
| *** markvoelker has quit IRC | 06:29 | |
| *** namnh has joined #openstack-nova | 06:32 | |
| *** sridharg has joined #openstack-nova | 06:33 | |
| *** Hosam has quit IRC | 06:34 | |
| *** moshele has joined #openstack-nova | 06:34 | |
| *** mirrorbo1 is now known as mirrorbox | 06:34 | |
| *** kaisers has joined #openstack-nova | 06:35 | |
| *** ljxiash has quit IRC | 06:36 | |
| *** ljxiash has joined #openstack-nova | 06:36 | |
| *** zhangyufei has quit IRC | 06:38 | |
| *** Kevin_Zheng has joined #openstack-nova | 06:39 | |
| *** links has joined #openstack-nova | 06:39 | |
| *** zhangyufei has joined #openstack-nova | 06:39 | |
| *** raunak has joined #openstack-nova | 06:40 | |
| *** Hosam has joined #openstack-nova | 06:41 | |
| *** ljxiash has quit IRC | 06:41 | |
| *** sahid has quit IRC | 06:42 | |
| *** Hosam_ has joined #openstack-nova | 06:44 | |
| *** moshele has quit IRC | 06:45 | |
| *** ducttape_ has joined #openstack-nova | 06:46 | |
| *** rama_y has quit IRC | 06:46 | |
| *** houming has quit IRC | 06:47 | |
| *** Hosam has quit IRC | 06:47 | |
| *** houming has joined #openstack-nova | 06:48 | |
| *** esberglu has joined #openstack-nova | 06:49 | |
| *** mnestratov has joined #openstack-nova | 06:50 | |
| *** ducttape_ has quit IRC | 06:51 | |
| openstackgerrit | zhongshengping proposed openstack/nova: Change cfg.set_defaults into cors.set_defaults https://review.openstack.org/395419 | 06:52 |
| *** esberglu has quit IRC | 06:53 | |
| *** rcernin has joined #openstack-nova | 06:54 | |
| *** qwertyco has joined #openstack-nova | 06:57 | |
| *** markus_z has joined #openstack-nova | 06:57 | |
| qwertyco | hello team! I need your help.. I had deployed Mitaka everyting works fine , I can create instance also I can create volume from dashboard but when I try to attach created volume to instance I get "Volume 08abf736-7f6c-4e92-ab46-9a452c6dec84 could not be found." | 06:58 |
| qwertyco | Do you have any idea? | 06:58 |
| *** klkumar has quit IRC | 07:00 | |
| *** ekuris has joined #openstack-nova | 07:00 | |
| *** thorst_ has joined #openstack-nova | 07:01 | |
| *** zenoway has joined #openstack-nova | 07:05 | |
| *** ihrachys has joined #openstack-nova | 07:07 | |
| *** unicell has joined #openstack-nova | 07:08 | |
| *** thorst_ has quit IRC | 07:09 | |
| *** Jeffrey4l has joined #openstack-nova | 07:09 | |
| *** tesseract has joined #openstack-nova | 07:11 | |
| *** takashin has joined #openstack-nova | 07:11 | |
| *** tesseract is now known as Guest39033 | 07:11 | |
| *** kylek3h has joined #openstack-nova | 07:11 | |
| *** kylek3h has quit IRC | 07:16 | |
| *** raunak has quit IRC | 07:17 | |
| *** ljxiash has joined #openstack-nova | 07:18 | |
| *** tlian has quit IRC | 07:18 | |
| *** garyk1 has joined #openstack-nova | 07:19 | |
| *** garyk1 has quit IRC | 07:20 | |
| *** garyk1 has joined #openstack-nova | 07:20 | |
| *** garyk1 has quit IRC | 07:20 | |
| *** garyk1 has joined #openstack-nova | 07:21 | |
| *** garyk1 has quit IRC | 07:21 | |
| *** garyk1 has joined #openstack-nova | 07:21 | |
| *** garyk1 has quit IRC | 07:21 | |
| *** shihanzhang has quit IRC | 07:21 | |
| openstackgerrit | NITIN GUPTA proposed openstack/python-novaclient: Modified "nova delete" command usage and test case. https://review.openstack.org/395011 | 07:22 |
| *** markvoelker has joined #openstack-nova | 07:25 | |
| openstackgerrit | Huan Xie proposed openstack/nova: XenAPI: Do not rely on XenAPI to get device names https://review.openstack.org/213112 | 07:27 |
| openstackgerrit | Huan Xie proposed openstack/nova: XenAPI: Do not rely on XenAPI to get device names https://review.openstack.org/213112 | 07:28 |
| *** Sukhdev has quit IRC | 07:28 | |
| *** andreas_s has joined #openstack-nova | 07:29 | |
| *** markvoelker has quit IRC | 07:30 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436 | 07:33 |
| *** gcb has quit IRC | 07:35 | |
| *** zhangyufei has quit IRC | 07:35 | |
| markus_z | qwertyco: Please use the channel #openstack for help with deployments. This channel here is for development. | 07:36 |
| qwertyco | oh ok thank you | 07:37 |
| *** gcb has joined #openstack-nova | 07:38 | |
| *** garyk1 has joined #openstack-nova | 07:39 | |
| *** owalsh has quit IRC | 07:39 | |
| *** belmoreira has joined #openstack-nova | 07:45 | |
| *** moshele has joined #openstack-nova | 07:46 | |
| *** esberglu has joined #openstack-nova | 07:48 | |
| *** CristinaPauna has quit IRC | 07:48 | |
| *** prashkre has joined #openstack-nova | 07:52 | |
| *** matrohon has joined #openstack-nova | 07:52 | |
| *** mars has joined #openstack-nova | 07:52 | |
| *** esberglu has quit IRC | 07:53 | |
| *** manasm has quit IRC | 07:53 | |
| *** pilgrimstack1 has joined #openstack-nova | 07:57 | |
| *** pilgrimstack has quit IRC | 07:59 | |
| *** klkumar has joined #openstack-nova | 08:00 | |
| openstackgerrit | zhongshengping proposed openstack/nova: Change cfg.set_defaults into cors.set_defaults https://review.openstack.org/395419 | 08:01 |
| *** mars has left #openstack-nova | 08:02 | |
| *** manasm has joined #openstack-nova | 08:03 | |
| *** takashin has left #openstack-nova | 08:05 | |
| *** prashkre has quit IRC | 08:06 | |
| *** ihrachys has quit IRC | 08:06 | |
| *** thorst_ has joined #openstack-nova | 08:08 | |
| *** eandersson_ has joined #openstack-nova | 08:08 | |
| *** vladani_ has quit IRC | 08:08 | |
| *** vladani has joined #openstack-nova | 08:08 | |
| *** thorst_ has quit IRC | 08:13 | |
| *** rmart04 has joined #openstack-nova | 08:16 | |
| *** ducttape_ has joined #openstack-nova | 08:17 | |
| *** namnh has quit IRC | 08:21 | |
| *** ducttape_ has quit IRC | 08:22 | |
| *** zhangyufei has joined #openstack-nova | 08:22 | |
| *** yuntongjin has quit IRC | 08:23 | |
| *** acabot has joined #openstack-nova | 08:24 | |
| *** jaosorior has joined #openstack-nova | 08:25 | |
| *** ralonsoh has joined #openstack-nova | 08:26 | |
| *** houming has quit IRC | 08:27 | |
| *** houming has joined #openstack-nova | 08:30 | |
| *** edand has joined #openstack-nova | 08:31 | |
| *** gcb has quit IRC | 08:33 | |
| *** CristinaPauna has joined #openstack-nova | 08:38 | |
| *** gcb has joined #openstack-nova | 08:47 | |
| *** openstackgerrit has quit IRC | 08:48 | |
| *** openstackgerrit has joined #openstack-nova | 08:48 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Use base64.decodebytes() in Python3 environment https://review.openstack.org/395468 | 08:52 |
| *** prashkre has joined #openstack-nova | 08:52 | |
| *** prashkre has quit IRC | 08:52 | |
| *** claudiub has quit IRC | 08:53 | |
| *** prashkre has joined #openstack-nova | 08:57 | |
| *** danpb has joined #openstack-nova | 08:58 | |
| *** Dong_Zhi__ has joined #openstack-nova | 08:59 | |
| *** hshiina has quit IRC | 08:59 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** lpetrut has joined #openstack-nova | 09:00 | |
| *** zzzeek has joined #openstack-nova | 09:00 | |
| *** alexchadin has joined #openstack-nova | 09:00 | |
| *** claudiub has joined #openstack-nova | 09:04 | |
| *** Dong_Zhi__ has quit IRC | 09:05 | |
| *** jed56 has joined #openstack-nova | 09:06 | |
| *** lucas-afk is now known as lucasagomes | 09:07 | |
| *** ihrachys has joined #openstack-nova | 09:11 | |
| *** thorst_ has joined #openstack-nova | 09:11 | |
| *** kylek3h has joined #openstack-nova | 09:12 | |
| *** kylek3h has quit IRC | 09:17 | |
| *** vladani_ has joined #openstack-nova | 09:17 | |
| *** xinliang has quit IRC | 09:17 | |
| *** vladani has quit IRC | 09:18 | |
| *** thorst_ has quit IRC | 09:19 | |
| *** claudiub has quit IRC | 09:19 | |
| *** stvnoyes1 has joined #openstack-nova | 09:19 | |
| *** stvnoyes has quit IRC | 09:19 | |
| *** gcb has quit IRC | 09:20 | |
| *** xinliang has joined #openstack-nova | 09:22 | |
| *** josecastroleon has joined #openstack-nova | 09:23 | |
| *** gmann has quit IRC | 09:24 | |
| *** gcb has joined #openstack-nova | 09:25 | |
| *** dimtruck is now known as zz_dimtruck | 09:25 | |
| *** edand has quit IRC | 09:26 | |
| *** markvoelker has joined #openstack-nova | 09:26 | |
| *** yamahata has quit IRC | 09:27 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: encryptors: Introduce encryption provider constants https://review.openstack.org/393901 | 09:29 |
| *** zhangyufei has quit IRC | 09:29 | |
| *** markvoelker has quit IRC | 09:31 | |
| *** vladani_ has quit IRC | 09:32 | |
| *** cdent has joined #openstack-nova | 09:32 | |
| *** vladani_ has joined #openstack-nova | 09:33 | |
| *** sudipto_ has quit IRC | 09:33 | |
| *** sudipto has quit IRC | 09:33 | |
| openstackgerrit | NITIN GUPTA proposed openstack/python-novaclient: Added a new nova CLI command "nova delete-all" https://review.openstack.org/393923 | 09:33 |
| *** zhangyufei has joined #openstack-nova | 09:33 | |
| *** Hosam_ has quit IRC | 09:33 | |
| *** ihrachys has quit IRC | 09:34 | |
| *** Hosam has joined #openstack-nova | 09:34 | |
| *** yassine has joined #openstack-nova | 09:36 | |
| *** yassine is now known as Guest83582 | 09:36 | |
| *** vladani_ has quit IRC | 09:37 | |
| *** liverpooler has joined #openstack-nova | 09:38 | |
| *** Hosam has quit IRC | 09:38 | |
| *** derekh has joined #openstack-nova | 09:39 | |
| *** qwertyco has quit IRC | 09:42 | |
| *** GB21 has quit IRC | 09:42 | |
| *** liverpooler has quit IRC | 09:45 | |
| *** liverpooler has joined #openstack-nova | 09:45 | |
| *** rmart04 has quit IRC | 09:45 | |
| *** ducttape_ has joined #openstack-nova | 09:47 | |
| *** zhangyufei has quit IRC | 09:49 | |
| *** zhangyufei has joined #openstack-nova | 09:51 | |
| *** ducttape_ has quit IRC | 09:52 | |
| *** tuanluong has quit IRC | 10:09 | |
| *** bkopilov has quit IRC | 10:14 | |
| *** Dong_Zhi__ has joined #openstack-nova | 10:14 | |
| *** brault has quit IRC | 10:15 | |
| *** thorst_ has joined #openstack-nova | 10:16 | |
| *** Dong_Zhi__ has quit IRC | 10:17 | |
| *** sree has quit IRC | 10:19 | |
| *** gouthamr has joined #openstack-nova | 10:19 | |
| *** sree has joined #openstack-nova | 10:19 | |
| *** ihrachys has joined #openstack-nova | 10:21 | |
| *** sahid has joined #openstack-nova | 10:23 | |
| *** thorst_ has quit IRC | 10:23 | |
| *** klkumar has quit IRC | 10:27 | |
| *** owalsh has joined #openstack-nova | 10:29 | |
| *** gcb has quit IRC | 10:29 | |
| *** gcb has joined #openstack-nova | 10:29 | |
| *** mvk has quit IRC | 10:30 | |
| *** roeyc has joined #openstack-nova | 10:31 | |
| *** esberglu has joined #openstack-nova | 10:32 | |
| *** tojuvone has quit IRC | 10:34 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform aggregate.create notification https://review.openstack.org/394511 | 10:34 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform aggregate.delete notification https://review.openstack.org/394512 | 10:34 |
| *** tojuvone has joined #openstack-nova | 10:34 | |
| *** roeyc has quit IRC | 10:35 | |
| *** esberglu has quit IRC | 10:36 | |
| *** gouthamr has quit IRC | 10:42 | |
| cdent | prateek: are you actively working on https://bugs.launchpad.net/nova/+bug/1638681 | 10:43 |
| openstack | Launchpad bug 1638681 in OpenStack Compute (nova) "resource tracker sets wrong max_unit in placement Inventory" [Undecided,Triaged] - Assigned to Prateek Arora (parora) | 10:43 |
| prateek | cdent, yes i want to. I am trying to get the info on how to hit the placement API's since there is no cmd for it, can you help me with that regard ? | 10:44 |
| *** gouthamr has joined #openstack-nova | 10:45 | |
| cdent | prateek: for that, that's not what needs to happen. The info is already present in the compute node info in the resource tracker, it just needs to be used. let me locate some links | 10:46 |
| mdbooth | danpb: Thanks! | 10:46 |
| cdent | prateek: this is the code that needs to change: https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L62 | 10:46 |
| *** avolkov has joined #openstack-nova | 10:47 | |
| cdent | and there's a previously abandoned changeset that does it more correctly here: line 58 of https://review.openstack.org/#/c/363583/1/nova/scheduler/client/report.py | 10:48 |
| cdent | how max_unit is being set in those objects | 10:48 |
| cdent | s/objects/calls/ | 10:49 |
| cdent | prateek: from the compute_node object | 10:49 |
| prateek | cdent, just give me 5 minutes to comprehend all this stuff :) | 10:50 |
| cdent | :) | 10:50 |
| *** womygod has quit IRC | 10:51 | |
| *** dikonoor has quit IRC | 10:53 | |
| prateek | cdent, that means the change that I need to do here is just change max_units=1 to max_units=compute_node.vcpus ? | 10:55 |
| prateek | *what I need to do | 10:55 |
| cdent | prateek: pretty much yes, but it needs to cover all three resource classes: disk_db, vcpu, memory_mb | 10:55 |
| prateek | yups and the compute_node object seems to have all three values already present with it | 10:56 |
| cdent | (disk_gb!) vcpu and memory ought to be straightforward, but disk a bit more complex | 10:56 |
| cdent | (but that complexity may be masked from elsewhere) | 10:56 |
| prateek | that would mean writing a helper method to do that and using that value directly here, is it ? | 10:57 |
| cdent | there may be some issues that neither of us know about, but they can be addressed in the review process when folk like jaypipes and dansmith come along | 10:57 |
| *** alexchadin has quit IRC | 10:58 | |
| *** gszasz has joined #openstack-nova | 10:58 | |
| *** amotoki has joined #openstack-nova | 10:58 | |
| prateek | cdent, that sounds like a good plan :), i would start with that, thanks for the great help, one more favor , can you tell me how do you test the placement API's, are there any cmd's or you hit the REST API's directly ? | 10:58 |
| cdent | prateek: I think for now it is probably sufficient to just change the max_unit line from 1 to : compute_node.{memory_mb,local_gb,vcpus} | 10:59 |
| *** hoangcx has quit IRC | 10:59 | |
| prateek | ok cdent | 10:59 |
| cdent | the are some unit tests in nova/tests/unit/scheduler/client/test_report.py | 11:00 |
| cdent | and there is a functional test in nova/tests/functional/api/openstack/placement/test_report_client.py that might be useful | 11:01 |
| prateek | cdent, ok, Thanks ! | 11:01 |
| cdent | thank you! | 11:02 |
| *** mvk has joined #openstack-nova | 11:03 | |
| *** Hosam has joined #openstack-nova | 11:05 | |
| *** Hosam_ has joined #openstack-nova | 11:05 | |
| *** rfolco has joined #openstack-nova | 11:05 | |
| *** roeyc has joined #openstack-nova | 11:07 | |
| *** Hosam has quit IRC | 11:09 | |
| *** zhangyufei has quit IRC | 11:11 | |
| *** kylek3h has joined #openstack-nova | 11:13 | |
| *** dikonoor has joined #openstack-nova | 11:14 | |
| *** klkumar has joined #openstack-nova | 11:14 | |
| *** zhangyufei has joined #openstack-nova | 11:16 | |
| *** kylek3h has quit IRC | 11:17 | |
| *** mbirru has quit IRC | 11:20 | |
| *** thorst_ has joined #openstack-nova | 11:20 | |
| *** brault has joined #openstack-nova | 11:27 | |
| *** thorst_ has quit IRC | 11:27 | |
| *** alexchadin has joined #openstack-nova | 11:34 | |
| *** nicolasbock has joined #openstack-nova | 11:36 | |
| *** portdirect has joined #openstack-nova | 11:37 | |
| *** tbachman has quit IRC | 11:41 | |
| *** dikonoor has quit IRC | 11:42 | |
| cdent | bauzas, edleafe, jaypipes: here's a placement related change that alaski started but I've picked up. It could do with some review: https://review.openstack.org/#/c/382640/ | 11:43 |
| *** amotoki has quit IRC | 11:43 | |
| *** Dong_Zhi__ has joined #openstack-nova | 11:45 | |
| *** aloga has quit IRC | 11:46 | |
| *** aloga has joined #openstack-nova | 11:46 | |
| *** cdent has quit IRC | 11:47 | |
| *** Dong_Zhi__ has quit IRC | 11:49 | |
| *** dikonoor has joined #openstack-nova | 11:49 | |
| *** jvgrant_ has joined #openstack-nova | 11:50 | |
| *** zhangyufei has quit IRC | 11:50 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Use double quotes instead of singles for consistency https://review.openstack.org/395551 | 11:51 |
| *** jvgrant__ has quit IRC | 11:53 | |
| *** zhangyufei has joined #openstack-nova | 11:58 | |
| manasm | bauzas, jaypipes, mriedem: https://review.openstack.org/#/c/392620/1 is awaiting review from cores, please take a look. | 12:00 |
| *** moshele has quit IRC | 12:01 | |
| *** rodrigods has quit IRC | 12:04 | |
| *** rodrigods has joined #openstack-nova | 12:04 | |
| *** moshele has joined #openstack-nova | 12:05 | |
| pkholkin | johnthetubaguy: bauzas: hey folks, waiting for your review here https://review.openstack.org/#/c/381912/, thanks in advance ! | 12:06 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: compute: Do not create bdm before reserving the volume https://review.openstack.org/290793 | 12:07 |
| *** smatzek has joined #openstack-nova | 12:08 | |
| *** esberglu has joined #openstack-nova | 12:11 | |
| *** amotoki has joined #openstack-nova | 12:13 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: compute: Do not create bdm before reserving the volume https://review.openstack.org/290793 | 12:14 |
| *** scsnow has joined #openstack-nova | 12:16 | |
| *** bvanhav_ has joined #openstack-nova | 12:16 | |
| *** esberglu has quit IRC | 12:16 | |
| *** catintheroof has joined #openstack-nova | 12:16 | |
| *** ratailor has quit IRC | 12:16 | |
| *** maticue_ has joined #openstack-nova | 12:18 | |
| *** cdent has joined #openstack-nova | 12:18 | |
| *** nicolasbock has quit IRC | 12:19 | |
| *** nicolasbock has joined #openstack-nova | 12:19 | |
| *** catinthe_ has joined #openstack-nova | 12:20 | |
| *** nicolasbock has quit IRC | 12:21 | |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild notification https://review.openstack.org/384922 | 12:21 |
| *** nicolasbock has joined #openstack-nova | 12:21 | |
| *** catintheroof has quit IRC | 12:21 | |
| *** nicolasbock has quit IRC | 12:22 | |
| *** nicolasbock has joined #openstack-nova | 12:22 | |
| *** rtheis has joined #openstack-nova | 12:22 | |
| *** nicolasbock has quit IRC | 12:22 | |
| *** nicolasbock has joined #openstack-nova | 12:22 | |
| *** claudiub has joined #openstack-nova | 12:25 | |
| *** mkoderer has joined #openstack-nova | 12:26 | |
| *** amotoki has quit IRC | 12:28 | |
| *** klkumar has quit IRC | 12:30 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Add spec to use Cinder's new attachment APIs https://review.openstack.org/373203 | 12:33 |
| *** owalsh has quit IRC | 12:34 | |
| *** klkumar has joined #openstack-nova | 12:35 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Mark sibling CPUs as 'used' for cpu_thread_policy = 'isolated' https://review.openstack.org/391416 | 12:38 |
| *** ljxiash has quit IRC | 12:39 | |
| *** prashkre has quit IRC | 12:40 | |
| *** tyrola has joined #openstack-nova | 12:42 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Add spec to use Cinder's new attachment APIs https://review.openstack.org/373203 | 12:46 |
| *** bkopilov has joined #openstack-nova | 12:46 | |
| *** ljxiash has joined #openstack-nova | 12:46 | |
| *** ljxiash has quit IRC | 12:46 | |
| *** amotoki has joined #openstack-nova | 12:46 | |
| *** lucasagomes is now known as lucas-hungry | 12:47 | |
| *** thorst_ has joined #openstack-nova | 12:48 | |
| *** thorst_ has quit IRC | 12:48 | |
| *** thorst_ has joined #openstack-nova | 12:48 | |
| *** liverpooler has quit IRC | 12:51 | |
| *** liverpooler has joined #openstack-nova | 12:51 | |
| *** gmann has joined #openstack-nova | 12:51 | |
| *** manasm has quit IRC | 12:51 | |
| *** gcb_ has joined #openstack-nova | 12:52 | |
| *** gcb has quit IRC | 12:52 | |
| *** Hosam_ has quit IRC | 12:53 | |
| *** pradk has joined #openstack-nova | 12:53 | |
| *** gmann_ has joined #openstack-nova | 12:53 | |
| *** baoli has joined #openstack-nova | 12:54 | |
| *** brault_ has joined #openstack-nova | 12:54 | |
| *** brault has quit IRC | 12:55 | |
| alex_xu | nova api meeting is in 5 mins at #openstack-meeting-4 | 12:55 |
| *** GB21 has joined #openstack-nova | 12:55 | |
| *** yuntongjin has joined #openstack-nova | 12:55 | |
| *** gmann has quit IRC | 12:56 | |
| *** baoli_ has joined #openstack-nova | 12:57 | |
| *** baoli has quit IRC | 12:59 | |
| *** rama_y has joined #openstack-nova | 13:03 | |
| *** yuntongjin has quit IRC | 13:03 | |
| *** portdirect has quit IRC | 13:04 | |
| *** gcb_ has quit IRC | 13:04 | |
| *** cdent has quit IRC | 13:07 | |
| *** ducttape_ has joined #openstack-nova | 13:07 | |
| *** gmann has joined #openstack-nova | 13:08 | |
| *** kylek3h has joined #openstack-nova | 13:09 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: change get_console_output as prep work for bp/libvirt-virtlogd https://review.openstack.org/394525 | 13:09 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices https://review.openstack.org/323765 | 13:09 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: read rotated "console.log" files https://review.openstack.org/385366 | 13:09 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: refactor console device creation methods https://review.openstack.org/385367 | 13:09 |
| *** lamt has joined #openstack-nova | 13:10 | |
| *** gmann_ has quit IRC | 13:11 | |
| *** gmann has quit IRC | 13:11 | |
| *** tbachman has joined #openstack-nova | 13:11 | |
| *** gmann has joined #openstack-nova | 13:11 | |
| *** edmondsw has joined #openstack-nova | 13:15 | |
| *** jwcroppe has quit IRC | 13:16 | |
| *** amotoki has quit IRC | 13:17 | |
| *** krtaylor has joined #openstack-nova | 13:20 | |
| *** shuquan has joined #openstack-nova | 13:20 | |
| *** tblakes has joined #openstack-nova | 13:20 | |
| markus_z | claudiub: During the summit I noted down the TODO for me to give you this link: http://openstack.markmail.org/thread/xrq2ao56t6cw4doc | 13:21 |
| markus_z | claudiub: We talked shortly about the idea. Do whatever you want with that piece of information :) | 13:22 |
| sfinucan | danpb: Could I get a post-merge conflict reapproval on this patch, please? https://review.openstack.org/#/c/356067/ | 13:22 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform aggregate.create notification https://review.openstack.org/394511 | 13:23 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform aggregate.delete notification https://review.openstack.org/394512 | 13:23 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: console: introduce framework for RFB authentication https://review.openstack.org/345397 | 13:24 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | 13:24 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: console: provide an RFB security proxy implementation https://review.openstack.org/345399 | 13:24 |
| *** zhangyufei has quit IRC | 13:24 | |
| *** lamt has quit IRC | 13:25 | |
| *** jistr is now known as jistr|brb | 13:26 | |
| *** cdent has joined #openstack-nova | 13:27 | |
| *** houming has quit IRC | 13:27 | |
| *** bks has quit IRC | 13:28 | |
| *** baoli_ has quit IRC | 13:29 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: Move api options to a group https://review.openstack.org/384490 | 13:31 |
| *** lamt has joined #openstack-nova | 13:31 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: Move api options to a group https://review.openstack.org/384490 | 13:32 |
| *** baoli has joined #openstack-nova | 13:32 | |
| *** ducttape_ has quit IRC | 13:33 | |
| *** ducttape_ has joined #openstack-nova | 13:33 | |
| *** dave-mccowan has joined #openstack-nova | 13:34 | |
| *** jheroux has joined #openstack-nova | 13:34 | |
| *** alexchadin has quit IRC | 13:35 | |
| *** sree has quit IRC | 13:37 | |
| *** sree has joined #openstack-nova | 13:38 | |
| *** ducttape_ has quit IRC | 13:38 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: fix code indentation for consistency in API https://review.openstack.org/395594 | 13:38 |
| *** klkumar has quit IRC | 13:40 | |
| *** klkumar has joined #openstack-nova | 13:41 | |
| *** sree has quit IRC | 13:42 | |
| *** lpetrut has quit IRC | 13:44 | |
| *** jistr|brb is now known as jistr | 13:46 | |
| kashyap | markus_z: If you've run today's DevStack, wonder if you noticed something like this "AssertionError: Egg-link does not match installed location of nova" - http://paste.openstack.org/show/588555/ | 13:47 |
| *** catintheroof has joined #openstack-nova | 13:47 | |
| *** paul-carlton2 has joined #openstack-nova | 13:49 | |
| * markus_z reads the paste | 13:50 | |
| *** ducttape_ has joined #openstack-nova | 13:50 | |
| *** catinthe_ has quit IRC | 13:50 | |
| *** jwcroppe has joined #openstack-nova | 13:51 | |
| *** lucas-hungry is now known as lucasagomes | 13:51 | |
| *** pbandark has quit IRC | 13:51 | |
| markus_z | kashyap: I've never seen the issue before. Let me try it locally. | 13:51 |
| *** klkumar has quit IRC | 13:53 | |
| *** bvanhav_ has quit IRC | 13:54 | |
| claudiub | markus_z: thanks for the link. :) | 13:54 |
| kashyap | markus_z: Thanks. (The only change in my env. is that I'm running with a patch locally from Matt Booth) | 13:54 |
| openstackgerrit | Chris Dent proposed openstack/nova: Optional separate database for placement API https://review.openstack.org/362766 | 13:54 |
| markus_z | kashyap: I've found https://github.com/pypa/pip/issues/3784 but it's already some months old | 13:55 |
| kashyap | markus_z: Hmm. | 13:55 |
| mdbooth | I'm assuming that intel-nfv-ci is sad today. | 13:55 |
| wznoinsk | mdbooth, very | 13:56 |
| wznoinsk | we're catching up tho | 13:56 |
| kashyap | mdbooth: When I applied your NFS patch series locally, I see the above (http://paste.openstack.org/show/588555/) | 13:56 |
| mdbooth | kashyap: Whoa. Are you sure they're related? | 13:57 |
| *** jed56 has quit IRC | 13:57 | |
| kashyap | But perhaps it's my env. But I'm kinda bummed with DevStack for the last two days | 13:57 |
| wznoinsk | mdbooth, + tempest fix to intel-nfv-ci-test was merged 3h ago https://review.openstack.org/#/c/395493/ although there's nova server internal error we were hitting not sure that's yet troubleshooted | 13:58 |
| kashyap | mdbooth: Yeah, not yet pointing fingers at that patch. I just updated Git & ran it | 13:58 |
| *** GB21 has quit IRC | 13:59 | |
| mdbooth | kashyap: Are you planning to setup an nfs testing environment, btw? | 14:00 |
| *** portdirect has joined #openstack-nova | 14:00 | |
| *** xyang1 has joined #openstack-nova | 14:00 | |
| *** tlian has joined #openstack-nova | 14:00 | |
| johnthetubaguy | alex_xu: what about the tenant_id_2 getting a link if the following page has more of its info on it? | 14:00 |
| markus_z | kashyap: my local devstack is running, I'll keep you posted. | 14:01 |
| johnthetubaguy | alex_xu: I added a comment about that in there: https://review.openstack.org/#/c/386771 | 14:01 |
| kashyap | mdbooth: I already have one | 14:01 |
| mdbooth | Cool. Are you testing my stuff? | 14:01 |
| *** lennyb has quit IRC | 14:02 | |
| bauzas | dansmith: once you're up, I'd be very interested in understanding the problem about secgroups that you're trying to solve with https://review.openstack.org/#/c/394656/2 | 14:02 |
| *** bvanhav_ has joined #openstack-nova | 14:02 | |
| alex_xu | johnthetubaguy: yea, I think it works, and sounds more simple | 14:03 |
| bauzas | dansmith: I saw https://review.openstack.org/#/c/367557/15/nova/conductor/api.py@133 but I need to understand the scope of that information, hence me thinking originally as the field being in the BuildRequest | 14:03 |
| *** mriedem has joined #openstack-nova | 14:03 | |
| kashyap | markus_z: Danke | 14:03 |
| kashyap | mdbooth: Yes, I am | 14:03 |
| markus_z | ;) | 14:03 |
| kashyap | mdbooth: (Or more accurately, I intend to) | 14:03 |
| *** oanson has quit IRC | 14:03 | |
| gmann | johnthetubaguy: alex_xu so we can check with just tenant id to add link there | 14:04 |
| mdbooth | kashyap: Awesome. Can you keep me closely in the loop? If you hit any issues, stick a -1 on there even if you haven't debugged yet and let me know. | 14:05 |
| alex_xu | johnthetubaguy: but how the dashboard to build the second page link? | 14:05 |
| alex_xu | johnthetubaguy: the dashboard is page by tenant I guess, won't be by instance id | 14:05 |
| *** ducttape_ has quit IRC | 14:05 | |
| *** diga has joined #openstack-nova | 14:05 | |
| kashyap | mdbooth: Of course, as a lowly tester, I'll come running to you if I hit an issue :-) | 14:05 |
| *** owalsh has joined #openstack-nova | 14:06 | |
| johnthetubaguy | alex_xu: so the dashboard could follow the link to complete the tenant_id, or if just cache the beginning of the next page and don't show that to the user, depending | 14:06 |
| johnthetubaguy | alex_xu: yeah | 14:06 |
| *** coreywright has quit IRC | 14:06 | |
| mdbooth | Um... you're not a lowly tester: I'll still expect you to debug anyway :P I just want a heads up. | 14:06 |
| kashyap | mdbooth: BTW, I'm using this super helpful NFS DevStack plugin that setups everything | 14:06 |
| kashyap | enable_plugin devstack-plugin-nfs https://git.openstack.org/openstack/devstack-plugin-nfs | 14:06 |
| *** rama_y has quit IRC | 14:07 | |
| kashyap | mdbooth: Hehe, I hear you. Not trying to shirk my duties | 14:07 |
| alex_xu | johnthetubaguy: if dashboard only show 1 item in the page, the first page get tenant_1 and part of tenant_2. For the second page, it still need to get the nova api first page, and the second page. | 14:07 |
| alex_xu | johnthetubaguy: that is why I hope the first page only return tenant_1. For the dashboard, it needn't cache the part of tenant_2 usage. | 14:09 |
| johnthetubaguy | alex_xu: we could do that, but we have to also worry about the case were tenant_1 is three pages long | 14:10 |
| diana_clarke | alex_xu, johnthetubaguy: Apologies for not answering the ping during the API meeting. My 10 year old is home sick today & I was away from the computer. | 14:11 |
| kashyap | markus_z: I think I'm too late to ask I guess :-( Would be super useful, if you could try with this: `git review -d 383859` | 14:11 |
| kashyap | markus_z: Don't worry if can't get to it. | 14:11 |
| alex_xu | johnthetubaguy: ah yea, for the tenant_1 is three pages long, I think it is ok have link to show this isn't the ending. | 14:11 |
| diana_clarke | (I'm reviewing the meeting minutes now) | 14:11 |
| *** mdrabe has joined #openstack-nova | 14:11 | |
| alex_xu | diana_clarke: no worries | 14:11 |
| johnthetubaguy | diana_clarke: honestly, they don't say too much, we still going through your spec | 14:11 |
| alex_xu | take care the baby is first thing | 14:11 |
| paul-carlton2 | mriedem, remember you re approved https://review.openstack.org/#/c/340874, the three patches to implement this are ready for review now - https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/detach-boot-volume | 14:13 |
| *** prateek has quit IRC | 14:13 | |
| *** gmann has quit IRC | 14:13 | |
| BobBall | mriedem: Just to say, if you have further questions on the os-xenapi spec do feel free to ping me on irc too | 14:15 |
| BobBall | (spec link https://review.openstack.org/#/c/379279/ ) | 14:15 |
| mriedem | paul-carlton2: tempest testing? | 14:15 |
| mriedem | BobBall: ack | 14:15 |
| paul-carlton2 | mriedem, ah, found this https://review.openstack.org/#/c/332050/ | 14:17 |
| *** coreywright has joined #openstack-nova | 14:17 | |
| mriedem | paul-carlton2: ok, can we get that cleaned up and using the same topic branch as the nova changes? | 14:18 |
| paul-carlton2 | mriedem, I'll fix it | 14:18 |
| *** lennyb has joined #openstack-nova | 14:19 | |
| pkoniszewski | paul-carlton2: hi, can you remove your -1 from https://review.openstack.org/#/c/386676/ ? I assume that your concern is no longer valid as you abandoned the revert | 14:20 |
| paul-carlton2 | done | 14:20 |
| pkoniszewski | thanks! | 14:20 |
| johnthetubaguy | mriedem: that cinder related spec should be in a better shape now, with some pictures: https://review.openstack.org/373203 | 14:21 |
| mriedem | BobBall: replied in https://review.openstack.org/#/c/379279/ | 14:22 |
| mriedem | BobBall: the py3 thing is a bit of a concern, because i think for nova to claim py3 support, nova and it's dependencies have to be py3 compatible | 14:23 |
| mriedem | the plugins probably don't need to be based on how those are used, but os-xenapi itself surely would need to be py3 compatible | 14:23 |
| BobBall | Absolutely, it is already and will remain so. The py2.4 components will *only* be the plugins. | 14:23 |
| * johnthetubaguy nods approvingly at BobBall | 14:24 | |
| *** victorhe has joined #openstack-nova | 14:24 | |
| mriedem | os-xenapi is already py3 compatible? | 14:24 |
| BobBall | mriedem: We already have py3 UT running, with the exclusions copied from Nova (e.g. XenAPI.py isn't currently py3 compatible, but we're currently fixing it to remove the final 2 exclusions) | 14:24 |
| mriedem | because i didn't think it existed yet | 14:24 |
| mriedem | ok, that's ^ what i was getting at | 14:25 |
| mriedem | i didn't think the xenapi stuff in nova was fully py3 compatible yet given our UT blacklist | 14:25 |
| mriedem | but i think we're on the same page | 14:25 |
| mriedem | ok +W | 14:26 |
| BobBall | https://git.openstack.org/cgit/openstack/os-xenapi/tree/exclusion_py3.txt is the current exclusion list - but we're trying to fix XenAPI.py. Unfortunately the pypi package is owned by an ex-employee who no longer has access to his account. We're trying to get either his account re-activated with a new email address or get the package transferred. Failing that, we'll move XenAPI to os-xenapi and | 14:26 |
| BobBall | remove the XenAPI from requirements.txt so we can fix | 14:26 |
| *** coreywright has quit IRC | 14:30 | |
| *** prashkre has joined #openstack-nova | 14:30 | |
| mriedem | johnthetubaguy: gonna try to tackle some lower hanging fruit before digging into the cinder spec :) | 14:31 |
| edleafe | cdent: bauzas: dansmith: jaypipes: Reminder for hangout in 30 minutes | 14:31 |
| johnthetubaguy | mriedem: totally! | 14:31 |
| *** jvgrant__ has joined #openstack-nova | 14:31 | |
| cdent | edleafe: ✔ | 14:31 |
| mriedem | danpb: johnthetubaguy: also, in case you haven't gone through them yet (i haven't on all of them), we should make a pass on the security specs in https://etherpad.openstack.org/p/ocata-nova-summit-security before it gets too late | 14:31 |
| mriedem | danpb: specifically you and that cert validation one | 14:32 |
| kashyap | mdbooth: Hmm, that error is not related to your patch. (But still I could reproduce it in my env w/ Git master & still have to resolve it) | 14:32 |
| *** smatzek has quit IRC | 14:32 | |
| *** esberglu has joined #openstack-nova | 14:33 | |
| *** jaosorior has quit IRC | 14:33 | |
| *** jaosorior has joined #openstack-nova | 14:33 | |
| danpb | mriedem: iirc, dane said to me after that session that they'd almost certainly only tackle the problem of tempest testing of existing functionality for ocata, given it was a short cycle | 14:34 |
| danpb | mriedem: so we should check with him, whether they really do need those specs approved for ocata or not | 14:34 |
| *** jvgrant_ has quit IRC | 14:34 | |
| mriedem | hmm, ok, because there are several specs | 14:34 |
| cdent | diana_clarke: as you predicted fixing the database poison stuff in resource providers has proved messy: https://review.openstack.org/#/c/394944/ | 14:34 |
| mriedem | but having said that, i agree that getting the CI job in place is probably going to take the whole release | 14:35 |
| danpb | mriedem: cert validation spec was the first one, and the api / trusted flavours was an add-on iiuc | 14:35 |
| danpb | my websockify spec has merged - not sure if you approved the blueprint yet or not | 14:35 |
| mriedem | there was also image verification | 14:35 |
| mriedem | danpb: yeah did that after the summit | 14:35 |
| danpb | ok | 14:35 |
| *** prashkre has quit IRC | 14:36 | |
| mriedem | danpb: and i think thorst_'s team was still working on the vnc changes | 14:36 |
| alex_xu | johnthetubaguy: I replied one more idea for the simple tenant usage pagination | 14:36 |
| mriedem | for devstack/tempest | 14:36 |
| mriedem | sean-k-mooney: this seems to have stalled https://review.openstack.org/#/c/334048/ - are you planning on updating that? | 14:37 |
| thorst_ | mriedem: yeah, I'll follow up with mmandel on the status on that. But the powervm driver does not require that for VNC console to work, it just makes it more secure. | 14:37 |
| danpb | mriedem: i've submitted code for the vnc impl already and working on the manuals docs update now | 14:37 |
| mriedem | danpb: i meant for CI testing | 14:37 |
| mriedem | with vnc | 14:37 |
| *** rtheis_ has joined #openstack-nova | 14:38 | |
| mriedem | danpb: looks like there is some discussion in https://review.openstack.org/#/c/334048/ that might need your input | 14:39 |
| *** coreywright has joined #openstack-nova | 14:39 | |
| *** rtheis has quit IRC | 14:39 | |
| *** baoli has quit IRC | 14:40 | |
| *** baoli has joined #openstack-nova | 14:41 | |
| *** jamesdenton has joined #openstack-nova | 14:43 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: compute: Do not create bdms before successfully reserving a volume https://review.openstack.org/290793 | 14:44 |
| sean-k-mooney | mriedem: sorry yes i am i have just been side tracked with some work in kolla | 14:45 |
| mdbooth | mriedem: Fancy bumping this one over the line? https://review.openstack.org/#/c/395050/ You were looking at it yesterday, test patches. | 14:45 |
| sean-k-mooney | mriedem: im not sure if i will get to it today but ill put it to the top of my list to rebase it tomorow | 14:46 |
| *** roeyc has quit IRC | 14:46 | |
| diana_clarke | cdent: yeah... I spent a couple of hours on it twice before I decided it was probably better left to you since it's going to require some judgments calls. Sorry ;( | 14:47 |
| cdent | diana_clarke: no worries | 14:47 |
| cdent | I'm currently stuck on it, and punted to edleafe so that he and I can share the burden | 14:47 |
| mriedem | mdbooth: https://www.youtube.com/watch?v=MNKL9onYB_8 ? | 14:47 |
| cdent | I might be inclined to remove some of the tests as they don't really cover anything | 14:48 |
| mdbooth | mriedem: lol | 14:48 |
| mdbooth | "One toke over the line". Is this a song as somebody who got busted for being slightly too high? | 14:49 |
| *** links has quit IRC | 14:49 | |
| mriedem | i don't know what the hell they are talking about, going home to be with his sweetheart but he's too stoned or something | 14:50 |
| diana_clarke | cdent: Have you ever watched any of Sandi Metz's talks on tests? Specifically around deleting tests. | 14:50 |
| edleafe | cdent: it's on my list. I may need to tap into the brain diana_clarke to get up to speed on the poison | 14:50 |
| *** manasm has joined #openstack-nova | 14:51 | |
| cdent | diana_clarke: not that I'm aware | 14:51 |
| diana_clarke | cdent: I think you would really like them. I'll try to dig up some links. | 14:51 |
| *** baoli has quit IRC | 14:51 | |
| *** rtheis_ is now known as rtheis | 14:52 | |
| diana_clarke | edleafe: Tip: It's easier for work on the poison stuff when it throws exceptions rather than warnings. Like this: https://review.openstack.org/#/c/349082/4/nova/tests/fixtures.py | 14:53 |
| *** baoli has joined #openstack-nova | 14:53 | |
| diana_clarke | typo: "for work on" -> "to work on" | 14:53 |
| *** roeyc has joined #openstack-nova | 14:53 | |
| edleafe | diana_clarke: thx, I'll keep that in mind | 14:54 |
| mriedem | mdbooth: why is "fakelibvirt.disable_event_thread(self)" removed in https://review.openstack.org/#/c/395050/2..3/nova/tests/unit/virt/test_virt_drivers.py ? | 14:56 |
| *** efried has joined #openstack-nova | 14:56 | |
| *** victorhe has quit IRC | 14:56 | |
| *** victorhe has joined #openstack-nova | 14:56 | |
| mriedem | oh i guess because the fixture does it already | 14:56 |
| *** sandanar has quit IRC | 14:58 | |
| *** achinm has joined #openstack-nova | 14:58 | |
| dansmith | edleafe: I assume you'll generate/paste the hangout link? | 14:58 |
| edleafe | cdent: bauzas: dansmith: jaypipes: Hangout URL: https://hangouts.google.com/call/4kttydtt2nelxavpnyjqz3ia6ue | 14:58 |
| raj_singh | johnthetubaguy: mriedem: Any chance I can get https://review.openstack.org/#/c/387711/ on your review list. | 14:58 |
| edleafe | dansmith: you beat me to it! | 14:58 |
| bauzas | edleafe: roger, just need some kind of bathroom usage :) | 14:59 |
| cdent | edleafe: be right there, gotta find headphones | 14:59 |
| johnthetubaguy | raj_singh: thats actually open on my screen right now, but +1 | 14:59 |
| johnthetubaguy | +1 on getting it reviewed that is | 14:59 |
| raj_singh | johnthetubaguy: Thx | 14:59 |
| *** hongbin has joined #openstack-nova | 15:01 | |
| *** moshele has quit IRC | 15:01 | |
| *** sdatko has joined #openstack-nova | 15:02 | |
| sdatko | Hello | 15:02 |
| *** moshele has joined #openstack-nova | 15:02 | |
| sdatko | I have found tiny bug in Nova unit tests | 15:02 |
| sdatko | https://bugs.launchpad.net/nova/+bug/1640502 | 15:02 |
| openstack | Launchpad bug 1640502 in OpenStack Compute (nova) "Missing /etc/machine-id causes failure in unit tests" [Undecided,New] - Assigned to Szymon Datko (fenekpl) | 15:02 |
| mriedem | sdatko: that failure crops up from time to time, it means the test isn't configuring something to ignore that, we should really disable that globally in tests | 15:03 |
| *** ducttape_ has joined #openstack-nova | 15:03 | |
| sdatko | yep | 15:03 |
| sdatko | /etc/machine-id is missing | 15:04 |
| sdatko | it seems that is should be created by systemd | 15:04 |
| sdatko | https://www.freedesktop.org/software/systemd/man/machine-id.html | 15:04 |
| sdatko | but it is not on some environments (like containers, in my case) | 15:04 |
| mriedem | set CONF.libvirt.sysinfo_serial to 'none' | 15:04 |
| sdatko | my propose is to mock this file in unit tests | 15:04 |
| mriedem | in the tests | 15:04 |
| *** lbrune has joined #openstack-nova | 15:04 | |
| mriedem | no, just set that flag globally | 15:04 |
| sdatko | ach, ok | 15:05 |
| mriedem | honestly we should just set that in the fake libvirt fixture | 15:05 |
| sdatko | hmm | 15:05 |
| mriedem | tests that need to test the non-none flows can override it | 15:05 |
| *** ducttape_ has quit IRC | 15:05 | |
| mriedem | see nova.tests.unit.virt.libvirt.fakelibvirt.FakeLibvirtFixture | 15:05 |
| sdatko | ok | 15:06 |
| mriedem | that's used in all of the libvirt tests that hit the driver | 15:06 |
| mriedem | so just set the config option to 'none' in there | 15:06 |
| sdatko | I will create a patch and I will send it | 15:06 |
| mriedem | sdatko: you might want to base it on top of this https://review.openstack.org/#/c/395050/ | 15:07 |
| mriedem | mdbooth: comments in https://review.openstack.org/#/c/395050/ | 15:09 |
| mriedem | a couple of small things to cleanup | 15:09 |
| mdbooth | mriedem: Thanks | 15:09 |
| mriedem | i'll be happy to fast approve that when those are addressed | 15:09 |
| *** cmartin has joined #openstack-nova | 15:09 | |
| *** lpetrut has joined #openstack-nova | 15:11 | |
| mdbooth | mriedem: Coming up shortly. | 15:12 |
| mriedem | payback https://review.openstack.org/#/c/394585/ | 15:13 |
| *** paw has quit IRC | 15:13 | |
| mriedem | ^ fixes a race in functional versioned notification tests i hit in zuul | 15:13 |
| cfriesen | mriedem: updated bug 1600304 with an explanation of how to hit it with mainline code | 15:13 |
| openstack | bug 1600304 in OpenStack Compute (nova) "_update_usage_from_migrations() can end up processing stale migrations" [Medium,In progress] https://launchpad.net/bugs/1600304 - Assigned to Chris Friesen (cbf123) | 15:13 |
| *** diogogmt has quit IRC | 15:13 | |
| *** achinm has quit IRC | 15:13 | |
| mdbooth | mriedem: Sure thing, will look. | 15:13 |
| mriedem | cfriesen: hmm, seems we should cleanup any failed migrations when evacuating the instance in questoin | 15:15 |
| mriedem | *question | 15:15 |
| mriedem | rather than leaving those hanging around to screw up the RT | 15:15 |
| mriedem | or, do something like that on startup of the failed compute node since i think that's already looking at migrations | 15:15 |
| *** garyk1 has quit IRC | 15:16 | |
| *** prashkre has joined #openstack-nova | 15:16 | |
| openstackgerrit | Daniel Berrange proposed openstack/nova: console: introduce framework for RFB authentication https://review.openstack.org/345397 | 15:17 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | 15:17 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: console: provide an RFB security proxy implementation https://review.openstack.org/345399 | 15:17 |
| cfriesen | mriedem: I'm not sure we can do that across the board since depending on where it failed it might actually still be in progress...there could be RPC calls queued up that nova-compute is about to process. | 15:18 |
| markus_z | kashyap: sorry for the late response. I could stack normally, with and without the patch you mentioned. | 15:18 |
| mriedem | cfriesen: i'm specifically thinkin of this method https://github.com/openstack/nova/blob/569e463f02b0a631d341b06849217657a22c4569/nova/compute/manager.py#L629 | 15:18 |
| cfriesen | mriedem: might be possible to clean up some migrations depending on the state it's in | 15:18 |
| mriedem | this comment is also confusing https://github.com/openstack/nova/blob/569e463f02b0a631d341b06849217657a22c4569/nova/compute/manager.py#L636 since it mentions not touching instances that are migrating/resize states, but doesn't actually check for those | 15:19 |
| mriedem | dansmith: when you're off your call, you might know the answer to ^ | 15:19 |
| *** paul-carlton2 has quit IRC | 15:20 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Fix incorrect libvirt library patching in tests https://review.openstack.org/395050 | 15:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Call host connection callbacks asynchronously https://review.openstack.org/393869 | 15:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859 | 15:20 |
| dansmith | mriedem: which question? whether we should clean up migrations? | 15:20 |
| mriedem | dansmith: the comment in there about not cleaning up instances in a certain state, | 15:21 |
| cfriesen | mriedem: back in kilo it actually checked the migration states | 15:21 |
| mriedem | but that code doesn't check any vm_state | 15:21 |
| cfriesen | mriedem: then we changed it to look for evacuations, | 15:21 |
| mriedem | this code was one toke over the line back in kilo | 15:21 |
| mriedem | so the comment just doesn't reflect reality anymore | 15:21 |
| openstackgerrit | Christopher Martin proposed openstack/nova: Fix for resize vm_state stopped rollback https://review.openstack.org/370861 | 15:21 |
| mriedem | ok i have to run to a meeting | 15:21 |
| mriedem | cfriesen: final question before i go - with https://review.openstack.org/#/c/339715/6/nova/compute/resource_tracker.py - we'll always hit those same busted migrations right? at some point, something should be cleaning those up it seems, and that code isn't doing it | 15:22 |
| mriedem | so we're going to process those migration records on that node every 60 seconds | 15:22 |
| bauzas | dansmith: saw my ping for https://review.openstack.org/#/c/394656/2 ? | 15:22 |
| cfriesen | mriedem: yes | 15:22 |
| bauzas | dansmith: I just wanted to understand the problem, ie. we're needing secgroups, but I don't get where | 15:23 |
| bauzas | dansmith: of course in the conductor manager side, but I wonder where it fails | 15:23 |
| dansmith | cfriesen: mriedem: tbh, creating a new migration should probably clean any outstanding migrations for that instance | 15:23 |
| dansmith | bauzas: there's lots of failed CI runs with logs for the earlier patches, but basically we stop passing the list of secgroup names in the call, which means we have no secgroups in compute when we create the port | 15:24 |
| dansmith | bauzas: which means the tests fail because we're not able to access the instance | 15:25 |
| bauzas | dansmith: which are the objects looking up for those secgroups ? | 15:25 |
| dansmith | bauzas: not sure what you mean.. the compute call needs them, and compute uses them while creating the ports | 15:26 |
| cmartin | Hey if anybody has some time available I have had the first patch for this bugfix up for a month now with no reviews: https://review.openstack.org/#/c/386756/ | 15:26 |
| *** mriedem1 has joined #openstack-nova | 15:26 | |
| mriedem1 | dansmith: yeah maybe | 15:26 |
| bauzas | dansmith: wait | 15:26 |
| mriedem1 | seems like there is a better fix for this somewhere | 15:26 |
| dansmith | bauzas: I have something to do in three minutes, back after that | 15:26 |
| bauzas | dansmith: I think I got it, those groups are stored in the instance object | 15:26 |
| dansmith | mriedem1: right, cleaning them up in some systematic way is much better, IMHO | 15:26 |
| bauzas | dansmith: okay, take time, will investigate | 15:26 |
| dansmith | bauzas: they are not | 15:26 |
| *** gszasz has quit IRC | 15:26 | |
| dansmith | bauzas: never when we're running neutron | 15:26 |
| dansmith | biab | 15:27 |
| bauzas | dansmith: okay, I need to understand the current situation before I can understand why you need to persist those | 15:27 |
| bauzas | dansmith: ++ | 15:27 |
| *** mriedem has quit IRC | 15:27 | |
| markus_z | BobBall: got a minute? It's a CI question. I'm wondering if and how you skip specific tempest tests in your CI. | 15:27 |
| mriedem1 | bauzas: for nova-net we store security groups in the nova db and can look them up via instance.security_groups | 15:27 |
| BobBall | Sure - we do. | 15:27 |
| mriedem1 | bauzas: but with neutron we have to get the security groups indirectly via the ports attached to the instance | 15:27 |
| BobBall | We have a tempest exclusion list which we translate into a REGEXP | 15:27 |
| bauzas | mriedem1: so we lazy-load them if n-net but we explicitly do a REST call if the other ? | 15:28 |
| BobBall | markus_z: https://git.openstack.org/cgit/openstack/xenapi-os-testing/tree/tempest_exclusion_list | 15:28 |
| markus_z | BobBall: Can I see that somewhere? We do something where I'm not sure if this is right. | 15:28 |
| * markus_z is reading | 15:28 | |
| *** baoli has quit IRC | 15:28 | |
| BobBall | markus_z: https://git.openstack.org/cgit/openstack/xenapi-os-testing/tree/run_tests.sh#n52 | 15:29 |
| *** gszasz has joined #openstack-nova | 15:29 | |
| mriedem1 | bauzas: yeah | 15:29 |
| bauzas | mmmm | 15:29 |
| cfriesen | dansmith: mriedem1: I kind of like the idea of using the existance of a new migration for an instance to cause all earlier ones to be somehow no longer in-progress. That logic could be used in the resource audit. | 15:30 |
| markus_z | BobBall: I see, cool. | 15:30 |
| mriedem1 | bauzas: yeah it's a known grossness, see https://bugs.launchpad.net/nova/+bug/1567655 | 15:30 |
| openstack | Launchpad bug 1567655 in OpenStack Compute (nova) "500 error when trying to list instances and neutron-server is down" [Medium,In progress] - Assigned to Prateek Arora (parora) | 15:30 |
| *** mriedem1 is now known as mriedem | 15:30 | |
| markus_z | BobBall: You don't have a clone of devstack-gate and fix that with custom stuff, do you? | 15:30 |
| bauzas | mriedem: okay, I'm more trying to understand why dan is trying to persist those in the Spec object https://review.openstack.org/#/c/394656/2 | 15:30 |
| bauzas | mriedem: given https://review.openstack.org/#/c/367557/15/nova/conductor/api.py@133 | 15:31 |
| BobBall | markus_z: uhhh... ... ... Well... It sounds like you want the answer to be no? | 15:31 |
| markus_z | BobBall: yes :) | 15:31 |
| markus_z | BobBall: My real question is, where is the entry point for 3d party CIs to do custom stuff. | 15:31 |
| markus_z | BobBall: I try to make sense of how it is supposed to be. | 15:32 |
| bauzas | mriedem: oh nvm, found the diff https://review.openstack.org/#/c/367557/15..17/nova/conductor/manager.py | 15:32 |
| BobBall | markus_z: TBH the only reason we have a custom d-g is because some of our test hosts don't have access to the zuul, so we checkout from review.openstack.org and merge rather than using zuul-merger's output | 15:32 |
| bauzas | okay, so we need to find them at the conductor level so we can pass them down to the compute | 15:32 |
| bauzas | that's all I need to understand | 15:32 |
| markus_z | BobBall: OK, sounds differently than the reasons why we do that. | 15:32 |
| BobBall | markus_z: Ah - pre_test_hook? https://github.com/citrix-openstack/os-ext-testing/blob/master/jenkins/jobs/jobs.yaml#L17 | 15:32 |
| *** zz_dimtruck is now known as dimtruck | 15:33 | |
| markus_z | BobBall: And the jobs.yaml gets pulled by Jenkins? | 15:33 |
| mriedem | bauzas: right | 15:33 |
| BobBall | markus_z: We use jenkins job builder to publish it to jenkins, yes. | 15:34 |
| cfriesen | mriedem: dansmith: maybe we could extend migration_get_in_progress_by_host_and_node() to eliminate ones where there is a newer migration for the same instance? | 15:34 |
| markus_z | BobBall: ok, got it, I think I have a picture. | 15:34 |
| bauzas | mriedem: k, so given we can't get them by the instance object, we need to store them explicitely, correct? | 15:34 |
| *** ducttape_ has joined #openstack-nova | 15:34 | |
| *** ekuris has quit IRC | 15:34 | |
| markus_z | BobBall: Thanks a lot for your time! | 15:35 |
| *** shuquan has quit IRC | 15:35 | |
| BobBall | markus_z: Coolio. Let me know if there are more questions. | 15:35 |
| mriedem | cfriesen: well, i think the point was you can only have a single migration running at a time per instance right? | 15:35 |
| bauzas | mriedem: my original thinking was to use the BuildRequest.instance object for getting those secgroups, but given what you said, it would only work for n-net, correct? | 15:35 |
| mriedem | cfriesen: so if migration 1 failed and you migrated again, when you start doing migration 2, cleanup migration 1 if it's residual | 15:35 |
| mriedem | bauzas: yeah, because BuildRequest.instance is the serialized instance | 15:36 |
| markus_z | BobBall: A last one :) Where do you store the tempest.conf you use in the check pipeline? | 15:36 |
| cfriesen | mriedem: yes, that would work | 15:36 |
| markus_z | I mean, where is the source of that | 15:36 |
| bauzas | mriedem: correct, so it wouldn't even work for n-net | 15:36 |
| mriedem | bauzas: i'm not actually sure if that includes the joined fields | 15:36 |
| BobBall | markus_z: I think it's just generated by devstack | 15:36 |
| bauzas | mriedem: pretty sure it's not, unless you explicitely want to get them AFAIKR | 15:36 |
| bauzas | AFAIR* | 15:36 |
| markus_z | BobBall: And the localrc for devstack? | 15:36 |
| markus_z | BobBall: simple example, we need to deactivate the VNC console for s390x. | 15:37 |
| BobBall | markus_z: Can be set up in the pre_test_hook | 15:37 |
| markus_z | BobBall: got it, that was all I needed, thanks a lot again! | 15:38 |
| cfriesen | mriedem: presumably we'd set any in-progress previous migrations to FAILED? | 15:38 |
| *** victorhe has quit IRC | 15:38 | |
| *** gouthamr has quit IRC | 15:38 | |
| *** gouthamr has joined #openstack-nova | 15:39 | |
| BobBall | markus_z: (btw - before you get too deeply into it; I actually showed you parts from two CI systems - the jenkins job definitions for the XenServer CI are in an internal repo, so showing the ones from the XenProject CI was quicker. You may spot things that don't quite make sense, so let me know if you want cleaner references) | 15:39 |
| *** mlavalle has joined #openstack-nova | 15:39 | |
| mriedem | bauzas: it looks like ovo will serialize all of the fields | 15:40 |
| mriedem | so the serialized instance object in build_requests should have the security_groups for n-net | 15:40 |
| bauzas | mriedem: but only for that case, not neutron | 15:40 |
| mriedem | bauzas: but, given n-net is deprecated we shouldn't care all that much for too long | 15:40 |
| mriedem | bauzas: right | 15:40 |
| bauzas | mriedem: okay, so we need to persist those secgroups explicitely | 15:40 |
| markus_z | BobBall: Thanks for the hint. Btw, is there a 3rd party IRC channel/meeting I should be aware of? | 15:40 |
| mriedem | cfriesen: i suppose | 15:41 |
| dansmith | mriedem: bauzas: instance has secgroups indeed, but we can't save them with it, so we need to persist them in the reqspec if we want to know them later | 15:41 |
| mriedem | keeping in mind that failed != error wrt migrations | 15:41 |
| johnthetubaguy | dansmith: mriedem: I put up a spec for the neutron refactor stuff that we said probably should be a priority: https://review.openstack.org/#/c/393888/ | 15:41 |
| dansmith | mriedem: bauzas which is why I put it there | 15:41 |
| bauzas | dansmith: okay, I understand now | 15:41 |
| bauzas | dansmith: my only worries could be a reschedule | 15:41 |
| mriedem | johnthetubaguy: ok | 15:41 |
| dansmith | bauzas: there is a comment in the instance object to this effect | 15:41 |
| bauzas | dansmith: given we get the original requestspec object when we do a move operation, we could potentially send down to the compute something not accurate | 15:42 |
| bauzas | dansmith: if we begin to care about the move operations using the cellsv2 path of course | 15:42 |
| BobBall | markus_z: Yeah - http://eavesdrop.openstack.org/#Third_Party_Meeting - but I haven't been to them for quite some time. | 15:42 |
| BobBall | markus_z: Looks like some folk are still attending | 15:42 |
| mriedem | bauzas: the security groups wouldn't change between reschedules right? | 15:42 |
| bauzas | mriedem: between reschedules, probably not | 15:43 |
| mriedem | or move | 15:43 |
| bauzas | mriedem: between a boot and a resize, that's less probable | 15:43 |
| mriedem | requested security groups aren't going to change | 15:43 |
| bauzas | oooh, that's not the *existing ones*, rather the ones you ask ? | 15:43 |
| mriedem | yeah, i suppose you could add/remove security groups after the server is created | 15:43 |
| bauzas | that's very different | 15:43 |
| dansmith | bauzas: I don't understand.. why would we not want to have the original secgroups on a move? | 15:44 |
| bauzas | I supposed that list was caching the list of existing SGs, not the requested ones | 15:44 |
| markus_z | BobBall: Uh, even an actively maintained agenda, nice. :) cool cool, I owe you one. | 15:44 |
| bauzas | dansmith: we're getting the original Spec object at the API level and pass it down to the conductor method responsible for the move, so we can schedule based on the original query | 15:44 |
| BobBall | Actively maintained agenda? Not seen one of those for a while... | 15:44 |
| BobBall | :) | 15:44 |
| bauzas | dansmith: here, we only need to persist what the user asked, right? | 15:45 |
| bauzas | (per your TODO)à | 15:45 |
| * dansmith is confused | 15:46 | |
| bauzas | dansmith: here I think there is a big difference between a requested string and a real object that we looked up | 15:46 |
| bauzas | dansmith: lemme explain | 15:46 |
| bauzas | dansmith: the RequestSpec object only stores the user requested strings, not exactly what's in the nova db | 15:47 |
| bauzas | dansmith: eg. we're not storing flavors, we're rather storing multiple "resource class amounts" | 15:47 |
| mriedem | bauzas: which on a server create, there is nothing in the db | 15:47 |
| mriedem | nothing existing anyway | 15:47 |
| mriedem | it's all what the user requested | 15:47 |
| bauzas | mriedem: right | 15:47 |
| bauzas | zactly that | 15:47 |
| bauzas | so | 15:47 |
| bauzas | I think we should only store what the user asked for security groups, ie. the secgroup names, nope? | 15:48 |
| dansmith | bauzas: you're arguing that we should store the names and not the secgroup objects? | 15:48 |
| bauzas | storing the whole object could lead to inconsistencies if we say that security groups can be modified over time | 15:48 |
| mriedem | that's all we get in the request isn't it? | 15:48 |
| bauzas | dansmith: yup | 15:48 |
| dansmith | bauzas: okay then you should just say that | 15:49 |
| bauzas | dansmith: I was thinking out loud :) | 15:49 |
| bauzas | 2 mins before, I wasn't understanding why the Spec object was useful for your case | 15:49 |
| bauzas | now, I'll leave an updated comment :) | 15:49 |
| dansmith | bauzas: the deal is we resolve them in compute now, but later we will resolve them in conductor early | 15:49 |
| *** prateek has joined #openstack-nova | 15:49 | |
| mriedem | on the request we get a list of names or uuids https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/security_groups.py#L25 | 15:49 |
| mriedem | just fyi | 15:49 |
| dansmith | bauzas: I'm not sure why storing resolved secgroups isn't right, because if we were to rename one, wouldn't we want "the same secgroup" on rebuild instead of "the same-named secgroup" ? | 15:50 |
| *** aloga_ has joined #openstack-nova | 15:51 | |
| bauzas | dansmith: if we say the conductor would resolve the secgroups, then it would lookup those objects by the Spec secgroup names, which is fine to me | 15:51 |
| *** markus_z has quit IRC | 15:51 | |
| bauzas | dansmith: and for a rebuild, we would still do the lookup in the conductor, based on that secgroup name | 15:52 |
| dansmith | bauzas: maybe, but maybe not | 15:52 |
| bauzas | we wouldn't need to update the Spec object for amending the secgroup in case they're updated | 15:52 |
| dansmith | if we store the objects, we have the option to do either | 15:52 |
| dansmith | if we store the names, we're locked into a symbolic link going forward | 15:52 |
| *** Sukhdev has joined #openstack-nova | 15:53 | |
| bauzas | dansmith: there is something which is not fully clear on the Spec object about fields that can be immutable and fields that can be different over time | 15:53 |
| *** ayoung has joined #openstack-nova | 15:54 | |
| *** tbachman has quit IRC | 15:55 | |
| bauzas | dansmith: for example, the project_id field is immutable but the force_hosts field needs to be changed if we're getting the original request for a move operation | 15:55 |
| dansmith | bauzas: yeah | 15:55 |
| *** eharney has joined #openstack-nova | 15:55 | |
| *** rrecio has joined #openstack-nova | 15:56 | |
| *** rfolco_ has joined #openstack-nova | 15:56 | |
| openstackgerrit | Julian proposed openstack/nova: Create schema generation for AddressBase https://review.openstack.org/391546 | 15:57 |
| *** jerrygb has joined #openstack-nova | 15:57 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Improve help text for ldap_dns_opts https://review.openstack.org/341984 | 15:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Fix config option types https://review.openstack.org/168481 | 15:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Fix indentation of network https://review.openstack.org/395682 | 15:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Deprecate all nova-net related opts https://review.openstack.org/395683 | 15:57 |
| *** _ducttape_ has joined #openstack-nova | 15:58 | |
| bauzas | dansmith: we don't have a special handler for saying during a move operation "please update those fields, they're not accurate" but the one named "reset_forced_destinations()" | 15:58 |
| *** rfolco has quit IRC | 15:58 | |
| bauzas | dansmith: so if we say we're going to store the whole object for preventing ourselve to do further lookups, I'm fine but I want to make sure we're forcing ourselves to refresh that object if we do a move operation | 15:59 |
| *** roeyc has quit IRC | 15:59 | |
| bauzas | ourselves* | 15:59 |
| mriedem | what does the refresh do? just to see if the security group with that id still exists? | 16:00 |
| mriedem | or if it's rules changed? | 16:00 |
| *** mdrabe has quit IRC | 16:01 | |
| *** marst has joined #openstack-nova | 16:01 | |
| *** Zer0Byte__ has joined #openstack-nova | 16:01 | |
| *** ducttape_ has quit IRC | 16:01 | |
| *** ayoung has quit IRC | 16:01 | |
| dansmith | mriedem: I'm assuming he means re-resolve the name-to-id, but I dunno | 16:01 |
| dansmith | storing the object gives us the most flexibility, which is why I think it's probably the best move | 16:02 |
| *** jaugustine_ has joined #openstack-nova | 16:02 | |
| *** roeyc has joined #openstack-nova | 16:02 | |
| bauzas | mriedem: the latter | 16:02 |
| bauzas | mriedem: what if we update a secgroup with adding a new rule | 16:03 |
| dansmith | bauzas: what? | 16:03 |
| dansmith | bauzas: nova needs to look at the secgroup rules to see if they have changed? | 16:03 |
| dansmith | that doesn't make sense to me *at all* | 16:03 |
| bauzas | dansmith: sec, looking at the object | 16:03 |
| bauzas | dansmith: we're not storing those rules, so I'm good then | 16:03 |
| bauzas | dansmith: my point is, can we update some security group information that would create a situation where we're getting the old information (from the Spec object if not updated) instead ? | 16:04 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova-specs: Add offset to console log https://review.openstack.org/390027 | 16:04 |
| bauzas | from the nova side | 16:04 |
| dansmith | bauzas: only the linkage matters at this point | 16:05 |
| dansmith | bauzas: manipulating the secgroup underneath the instance is not something that needs tracking here | 16:05 |
| mriedem | sahid: provided an alternative wording in https://review.openstack.org/#/c/395067/ - see what you think | 16:05 |
| mriedem | dansmith: i agree with that statement | 16:06 |
| *** stewie925 has joined #openstack-nova | 16:06 | |
| mriedem | i don't care if we store the object itself or just the name/uuid from the request - the object is only n-net though, | 16:07 |
| mriedem | and the SecurityGroup object only has an int id field or nova-net | 16:07 |
| mriedem | so wouldn't work for neutron where you can specify a uuid on the request | 16:07 |
| mriedem | so i'm not sure how we're going to store the SecurityGroupList for neutron requests that use uuids | 16:07 |
| dansmith | mriedem: yeah, but I thought we had previously discussed storing the uuid there so that we can use it for both? | 16:07 |
| bauzas | dansmith: mriedem: so why not just storing the list of secgroup uuids ? | 16:07 |
| dansmith | because we have some hacks in place for that right now, IIRC | 16:08 |
| mriedem | dansmith: so add a uuid field to SecurityGroup obj? | 16:08 |
| dansmith | bauzas: because we don't resolve them there currently, and that won't work for n-net either | 16:08 |
| dansmith | mriedem: we previously discussed it right? | 16:08 |
| bauzas | dansmith: oh right, an 'id" | 16:08 |
| bauzas | snap | 16:08 |
| mriedem | we do something similar for some other network resource, like fixed ips or something | 16:08 |
| *** moshele has quit IRC | 16:08 | |
| mriedem | ah NeutronFloatingIP | 16:09 |
| dansmith | ah, maybe that's what I'm thinking of | 16:09 |
| mriedem | that's the neutron variant of the floating IP object where the id is a uuid | 16:09 |
| dansmith | either way, | 16:09 |
| dansmith | if we store a list of strings, we are limited in the future.. if we store an object it's easier to evolve I think | 16:09 |
| mriedem | dansmith: i don't remember discussing this previously, no, fwiw | 16:09 |
| bauzas | dansmith: I think I don't disagree with that | 16:10 |
| dansmith | if it doesn't matter, then okay, but I dunno.. I'd rather hedge | 16:10 |
| dansmith | mriedem: I'm probably thinking about the ip one | 16:10 |
| mriedem | storing an object would future proof us in case we add new fields to that object that we care about later | 16:10 |
| dansmith | mriedem: right | 16:10 |
| bauzas | dansmith: my only concern is the mutability of that object | 16:10 |
| bauzas | if we persist it | 16:10 |
| mriedem | bauzas: i think you'd need to provide a use case for that going wrong | 16:11 |
| dansmith | bauzas: but as you said, we just have to make a rule and stick to it | 16:11 |
| mriedem | to understand your concern | 16:11 |
| dansmith | bauzas: a list of names has the same question | 16:11 |
| *** belmoreira has quit IRC | 16:11 | |
| bauzas | dansmith: good point, but I guess we're forbidding the user to provide a different secgroup request when moving an instance, right? | 16:11 |
| mriedem | tbh, i don't care either way if we store the list of strings (names or uuids) from the initial REST API request, or a SecurityGroupList | 16:11 |
| claudiub | mriedem: hi. i've noticed that the config option instance_name_template is in the mitaka config option documentation http://docs.openstack.org/mitaka/config-reference/compute/config-options.html , but it doesn't exist anymore in the newton one. any ideas? there is no deprecation notice either. | 16:11 |
| *** Zer0Byte__ has quit IRC | 16:11 | |
| dansmith | bauzas: right, so why is that any different for the list of objects? | 16:12 |
| mriedem | bauzas: you can't change security groups when migrating an instance today right? | 16:12 |
| mriedem | bauzas: i'd rather cross bridges when they exist | 16:12 |
| mriedem | if we need to | 16:12 |
| bauzas | mriedem: dansmith: okay, I guess my problem is because I don't really know the SecurityGroup object like you | 16:12 |
| bauzas | mriedem: dansmith: I can see a description and a name fields that look to me possibly being changed | 16:13 |
| bauzas | mriedem: dansmith: but I guess those are only filled if n-net | 16:13 |
| bauzas | mriedem: dansmith: my concern are : | 16:13 |
| *** rcernin has quit IRC | 16:13 | |
| bauzas | 1/ an user boots an instance asking for group A | 16:13 |
| bauzas | 2/ group A description is being modified | 16:14 |
| dansmith | but .. the name is just as mutable as a lit of names, and the other fields aren't used, certainly not for a rebuild type thing | 16:14 |
| bauzas | dansmith: okay, I guess they're only used for show/list | 16:14 |
| dansmith | *list | 16:14 |
| bauzas | that's a good point | 16:14 |
| dansmith | bauzas: and only for n-net | 16:14 |
| *** mdrabe has joined #openstack-nova | 16:15 | |
| bauzas | okay, looks to me then there is no reason for us to really care about what's in the object | 16:15 |
| mriedem | storing a name that is mutable is a bad idea for sure | 16:15 |
| mriedem | so for n-net we need to store the id (int), for neutron we need to store the uuid | 16:15 |
| mriedem | b/c those don't change | 16:15 |
| openstackgerrit | Sujitha proposed openstack/nova: Document for zero downtime upgrade process https://review.openstack.org/395269 | 16:15 |
| *** liverpooler has quit IRC | 16:16 | |
| mriedem | so...we add uuid to SecurityGroup object, or create a NeutronSecurityGroup object like we have for the fixed IP object, and then persist that list in the request spec | 16:16 |
| mriedem | the name/description be damned | 16:16 |
| mriedem | we care about the thing that won't change so we can look it up later | 16:16 |
| dansmith | mriedem: we don't resolve the n-net id in the api either, FWIW | 16:16 |
| mriedem | dansmith: we don't lookup the requested security groups by name from the request? | 16:17 |
| mriedem | in the API? | 16:17 |
| dansmith | mriedem: but if adding a uuid so we can potentially update it once we've resolved it is the right thing, then that's cool | 16:17 |
| *** liverpooler has joined #openstack-nova | 16:17 | |
| dansmith | mriedem: the n-net "populate_security_groups()" call does not, | 16:17 |
| dansmith | which is what ends up in this field | 16:17 |
| openstackgerrit | Lucian Petrut proposed openstack/nova: HyperV: use os-brick for volume related operations https://review.openstack.org/273504 | 16:17 |
| dansmith | it may have checked them separately for exist-ness | 16:17 |
| *** al_indigo has joined #openstack-nova | 16:17 | |
| dansmith | but we don't have the ids at the populate call | 16:17 |
| mriedem | ok looking | 16:18 |
| mriedem | _check_requested_secgroups validates the request | 16:19 |
| bauzas | dansmith: mriedem: back up a sec, we're doing the Neutron call by passing an uuid that is found by the secgroup name, correct? | 16:19 |
| mriedem | then later yeah we convert the list of names to a SecurityGroupList object | 16:19 |
| mriedem | with just the names set | 16:19 |
| dansmith | mriedem: right | 16:19 |
| mriedem | bauzas: you can pass name or uuid for neutron | 16:19 |
| dansmith | mriedem: for neutron we just pass the names down and sort out the bodies in compute | 16:20 |
| mriedem | bauzas: but in neutron you can have multiple security groups with the same name | 16:20 |
| mriedem | bauzas: if that's the case, you get a conflict i think b/c we can't decide which to use | 16:20 |
| dansmith | mriedem: that's another good reason to use an object, so we can is_uuid_like() the thing in the api, populate the object accordingly and pass it down | 16:20 |
| bauzas | mriedem: and the API resolves the uuid if the name(s) are provided? | 16:20 |
| dansmith | instead of mixing them which I think happens now | 16:20 |
| bauzas | mriedem: or are we calling neutron with the name ? | 16:20 |
| *** sree has joined #openstack-nova | 16:21 | |
| mriedem | bauzas: we call neutron with the name | 16:22 |
| mriedem | bauzas: if it's not unique neutron raises NeutronClientNoUniqueMatch | 16:22 |
| mriedem | and we raise that back up as a failure | 16:22 |
| bauzas | okay | 16:22 |
| mriedem | just like with networks | 16:22 |
| bauzas | makes sense yeah | 16:22 |
| mriedem | if you don't provide a specific network and there are multiple available to the tenant, then we fail saying you need to provide a specific network | 16:22 |
| bauzas | mriedem: so that's the problem I mentioned | 16:22 |
| mriedem | dansmith: so i'm cool with storing the objects | 16:23 |
| mriedem | it would be good if we actually stored the neutron secgroup uuid that we looked up if a name were provided | 16:23 |
| mriedem | in _check_requested_secgroups | 16:23 |
| bauzas | how can we get the correct security group for a move operation at the moment if the user modified the name in Neutron ? | 16:23 |
| mriedem | basically if you requested via name, we found via name, then we swap name for uuid and store that later in populate_security_groups() | 16:24 |
| dansmith | bauzas: after johnthetubaguy moves the port allocation, we can update the object with a uuid, | 16:24 |
| dansmith | so that we're pinned going forward | 16:24 |
| mdbooth | mriedem: Jenkins re-blessed https://review.openstack.org/#/c/395050/ if you get a sec | 16:24 |
| mriedem | dansmith: ^ agree on storing the uuid and not the name for neutron? | 16:24 |
| mriedem | then we don't care about the name changing | 16:24 |
| openstackgerrit | Sujitha proposed openstack/nova: Remove deprecated manager classes in service.py https://review.openstack.org/338553 | 16:24 |
| dansmith | mriedem: I would wait and do it as part of johnthetubaguy's work, but if you want to resolve them now and store uuids, that's fine | 16:25 |
| *** Guest39033 has quit IRC | 16:25 | |
| dansmith | mriedem: storing uuid if a uuid is provided now would be good though, yeah | 16:25 |
| dansmith | mriedem: detecting uuidness and storing it in the object as a uuid instead of a uuid in the name field, I mean | 16:25 |
| bauzas | dansmith: mriedem: I think we can store the object now and care about that problem later | 16:25 |
| *** gszasz has quit IRC | 16:25 | |
| bauzas | mriedem: dansmith: because the move operations are not yet following the cellsv2 path | 16:26 |
| mriedem | dansmith: we already do the work of looking up the secgroup from neutron by name so we'd have the uuid, so i'd store it now | 16:26 |
| *** swamireddy has quit IRC | 16:26 | |
| *** smatzek has joined #openstack-nova | 16:26 | |
| dansmith | mriedem: okay, I haven't looked through that, but that's cool | 16:26 |
| dansmith | mriedem: bauzas so I will add uuid to secgroup and then work that change in before the final one there somewhere | 16:26 |
| mriedem | i could look at making that little swap patch and see what breaks | 16:27 |
| mriedem | should be trivial | 16:27 |
| *** sridharg has quit IRC | 16:27 | |
| sahid | mriedem: replied, thanks to notice me | 16:27 |
| mriedem | mdbooth: +W | 16:27 |
| bauzas | dansmith: fine by me, I'll remove my -1 that was about storing it in the Spec object | 16:27 |
| mdbooth | mriedem: Thanks | 16:27 |
| cdent | bauzas, edleafe: could you guys give this change from prateek a review. Improves 404 messages in placement api: https://review.openstack.org/#/c/387674/ | 16:27 |
| *** tbachman has joined #openstack-nova | 16:27 | |
| mdbooth | Incidentally, the next 2 are a meatier bugfix. johnthetubaguy, are you interested in nfs volume mount/unmount races? danpb is already +2 | 16:28 |
| mdbooth | Top change is here: https://review.openstack.org/#/c/383859/ relies on the 1 before it. | 16:28 |
| openstackgerrit | Lucian Petrut proposed openstack/nova: HyperV: use os-brick for volume related operations https://review.openstack.org/273504 | 16:29 |
| *** aloga_ has quit IRC | 16:29 | |
| mriedem | cdent: comment in there | 16:30 |
| mriedem | i cared about NFS issues about a month ago... | 16:30 |
| cdent | thanks mriedem | 16:30 |
| openstackgerrit | Lucian Petrut proposed openstack/nova: HyperV: use os-brick for volume related operations https://review.openstack.org/273504 | 16:31 |
| *** markvoelker has joined #openstack-nova | 16:31 | |
| mdbooth | mriedem: It would be awesome if you had time to look. | 16:31 |
| mriedem | mdbooth: so.... | 16:31 |
| mriedem | mdbooth: people have been working on the nfs job that's in the experimental queue | 16:31 |
| *** prateek has quit IRC | 16:31 | |
| *** dikonoor has quit IRC | 16:31 | |
| mriedem | especially since cinder is working on adding snapshot support to nfs in cinder | 16:31 |
| openstackgerrit | jichenjc proposed openstack/python-novaclient: Add version pin for image related function https://review.openstack.org/392523 | 16:31 |
| mriedem | mdbooth: it would be cool to be running that nfs job on those changes | 16:32 |
| mriedem | in the nova experimental queue of course | 16:32 |
| *** paw has joined #openstack-nova | 16:32 | |
| mdbooth | Is it already in the experimental basket? | 16:32 |
| * mdbooth has never dipped his toes into CI config, tbh | 16:32 | |
| mriedem | gate-tempest-dsvm-full-devstack-plugin-nfs-nv is the job name | 16:33 |
| mriedem | mdbooth: copy this http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml#n6234 | 16:33 |
| mriedem | and dump somewhere in this section for nova experimental queue http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml#n12264 | 16:33 |
| mriedem | easy peasy | 16:33 |
| mdbooth | mriedem: K | 16:34 |
| mriedem | brb | 16:34 |
| edmondsw | mriedem, a couple days ago you'd said you thought we should avoid isinstance while fixing https://launchpad.net/bugs/1636109 but we got a -1 wanting us to use that... can you take a look and add your thoughts to the review? | 16:34 |
| openstack | Launchpad bug 1636109 in OpenStack Compute (nova) "Reschedule of a virtual machine fails with 'NetworkInfo' object has no 'wait'" [Undecided,In progress] - Assigned to Esha Seth (eshaseth) | 16:34 |
| edmondsw | mriedem https://review.openstack.org/#/c/393669/ | 16:34 |
| edmondsw | sahid ^ | 16:36 |
| *** kfarr has joined #openstack-nova | 16:36 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: remove deprecated nova quota options https://review.openstack.org/395706 | 16:36 |
| *** sree has quit IRC | 16:36 | |
| edmondsw | of course I hit ENTER right as you said "brb" ;) | 16:37 |
| *** gszasz has joined #openstack-nova | 16:37 | |
| *** mriedem1 has joined #openstack-nova | 16:38 | |
| *** mriedem1 has quit IRC | 16:38 | |
| *** mriedem1 has joined #openstack-nova | 16:39 | |
| sean-k-mooney | exit | 16:39 |
| *** sean-k-mooney has quit IRC | 16:39 | |
| *** mriedem has quit IRC | 16:39 | |
| *** bkopilov has quit IRC | 16:39 | |
| mriedem1 | dansmith: cells v2 meeting in 20 minutes? | 16:39 |
| *** sean-k-mooney has joined #openstack-nova | 16:40 | |
| dansmith | yeah I guess | 16:40 |
| *** fragatina has joined #openstack-nova | 16:42 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: remove deprecated nova quota options https://review.openstack.org/395706 | 16:42 |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: Move api options to a group https://review.openstack.org/384490 | 16:42 |
| mriedem1 | edmondsw: comments in ps3 | 16:43 |
| *** mriedem1 is now known as mriedem | 16:43 | |
| mriedem | dansmith: great! i'm looking forward to it! | 16:43 |
| dansmith | heh | 16:44 |
| *** bkopilov has joined #openstack-nova | 16:46 | |
| edmondsw | mriedem, tx | 16:46 |
| edmondsw | sahid, please see those comments | 16:47 |
| mriedem | oh bobby caldwell, take me away | 16:48 |
| *** aloga_ has joined #openstack-nova | 16:48 | |
| *** Zer0Byte__ has joined #openstack-nova | 16:49 | |
| mdbooth | mriedem: This isn't what you asked for, but out of curiosity I'm wondering if it does what I hope: https://review.openstack.org/#/c/395709/ | 16:49 |
| *** manasm has quit IRC | 16:50 | |
| * mdbooth is about to submit a proper patch against project-config | 16:50 | |
| mriedem | looks like it will | 16:50 |
| mriedem | the job might fail for unrelated reasons, i'm not sure what state it's in | 16:50 |
| mriedem | i wanted to disable the snapshot tests in it since they didn't work, and clone tests, but i think i got some pushback from cinder folks on that | 16:51 |
| *** _ducttape_ has quit IRC | 16:51 | |
| mriedem | but attach/detach volume should work | 16:51 |
| sahid | edmondsw: yeah i can see that, you win :) let's add a comment at least so | 16:51 |
| *** ducttape_ has joined #openstack-nova | 16:51 | |
| *** Swami has joined #openstack-nova | 16:52 | |
| *** lbeliveau_ has joined #openstack-nova | 16:52 | |
| *** bnemec has quit IRC | 16:52 | |
| *** lbeliveau has quit IRC | 16:52 | |
| *** aloga_ has quit IRC | 16:53 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: pci: Validate device name https://review.openstack.org/395710 | 16:53 |
| edmondsw | sahid, sure, I'll strengthen the comment mriedem referenced | 16:53 |
| *** yamahata has joined #openstack-nova | 16:53 | |
| *** bvanhav__ has joined #openstack-nova | 16:53 | |
| *** bvanhav_ has quit IRC | 16:54 | |
| *** bnemec has joined #openstack-nova | 16:55 | |
| *** gouthamr has quit IRC | 16:56 | |
| *** gouthamr has joined #openstack-nova | 16:56 | |
| *** yamahata has quit IRC | 16:56 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: aggregate_metadata_get_by_host cleanup https://review.openstack.org/395713 | 16:56 |
| *** yamahata has joined #openstack-nova | 16:58 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: DNM - encryptors: Switch to os-brick encryptor classes https://review.openstack.org/391597 | 17:01 |
| *** cdent has left #openstack-nova | 17:02 | |
| pkholkin | mriedem: hello Matt! please take a look my spec please if you have time https://review.openstack.org/#/c/381912/, waiting for John and Sylvain there | 17:02 |
| *** lbrune has quit IRC | 17:02 | |
| *** sahid has quit IRC | 17:03 | |
| kashyap | mdbooth: Heh, that commit message https://review.openstack.org/#/c/395709/ | 17:03 |
| mdbooth | kashyap: Honesty first. | 17:03 |
| *** zenoway has quit IRC | 17:04 | |
| *** zenoway has joined #openstack-nova | 17:04 | |
| kashyap | mdbooth: I have to head out early (already 18) today, will get to testing that race series in the morning | 17:05 |
| sdatko | mriedem: hey, are you still there? :) | 17:05 |
| mriedem | sdatko: in a meeting, but post if you want | 17:05 |
| mdbooth | kashyap: Thanks | 17:05 |
| sdatko | mriedem: i just realised (after few tests) that I described the bug a bit incorrectly | 17:07 |
| *** andreas_s has quit IRC | 17:07 | |
| sdatko | mriedem: the tests are fine if there is no /etc/machine-id; the exception is there (in tests) when the content is empty | 17:08 |
| *** zenoway has quit IRC | 17:08 | |
| sdatko | mriedem: and this is what happens inside my Docker container | 17:08 |
| mriedem | sdatko: still, unless the tests actually care about that flow, we should just set the tests globally to not care | 17:09 |
| mriedem | so setting that option to 'none' | 17:09 |
| *** roeyc has quit IRC | 17:09 | |
| danpb | sdatko: which test(s) fails with that ? | 17:09 |
| *** gouthamr has quit IRC | 17:10 | |
| sdatko | danpb: nova.tests.unit.virt.libvirt.test_driver.LibvirtDriverTestCase.test_rescue | 17:10 |
| danpb | there's already code to deal with this - see the 'patch_exists' method in that file | 17:11 |
| danpb | so probably just needs to be called at the right place in test_rescue | 17:11 |
| mriedem | danpb: that's a bandaid and will just blow up later on new tests | 17:11 |
| mriedem | danpb: we should nix that thing globally as it depends on system state | 17:12 |
| mriedem | tests that are about those code flows should set the flag appropriately | 17:12 |
| danpb | oh you mean set a flag to prevent nova using machine-id at all | 17:12 |
| *** gouthamr has joined #openstack-nova | 17:12 | |
| *** tbachman_ has joined #openstack-nova | 17:13 | |
| sdatko | okay | 17:13 |
| sdatko | i just wanted to be sure that this does not change your original vote | 17:13 |
| *** roeyc has joined #openstack-nova | 17:14 | |
| sdatko | thanks :) | 17:14 |
| *** matrohon has quit IRC | 17:14 | |
| *** roeyc has quit IRC | 17:15 | |
| *** tbachman has quit IRC | 17:15 | |
| *** tbachman_ is now known as tbachman | 17:15 | |
| lyarwood | dansmith: https://review.openstack.org/#/c/290793/16/nova/compute/api.py@3405 - if you have anytime today I'd appreciate your take on adding a bdm.uuid field to help cover all the corner cases here. IIRC you had a change up for review a while ago to add this. | 17:16 |
| *** lbrune has joined #openstack-nova | 17:18 | |
| mriedem | lyarwood: i can find that series and you can update it | 17:20 |
| *** roeyc has joined #openstack-nova | 17:22 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: handle os-brick InvalidConnectorProtocol on init https://review.openstack.org/395067 | 17:22 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Move driver_dict_from_config to libvirt driver https://review.openstack.org/395008 | 17:22 |
| mriedem | lyarwood: https://review.openstack.org/#/c/242602/ | 17:24 |
| mriedem | lyarwood: and https://review.openstack.org/#/c/242603/ | 17:24 |
| mriedem | lyarwood: want me to restore those so you can fix them up? | 17:24 |
| *** tbachman has quit IRC | 17:24 | |
| mriedem | lyarwood: don't talk so much ok :) but i restored those | 17:25 |
| *** browne has joined #openstack-nova | 17:26 | |
| *** baoli has joined #openstack-nova | 17:28 | |
| *** nic has joined #openstack-nova | 17:28 | |
| dansmith | lyarwood: yep, I'm good with a uuid obviously, and you're welcome to run with that patch | 17:29 |
| lyarwood | mriedem: lol sorry my wife walked through the door just after I sent that | 17:30 |
| lyarwood | mriedem: thanks, yeah if it's needed I'll take a swing at that tomorrow | 17:30 |
| lyarwood | mriedem: I just wasn't sure if it was | 17:30 |
| lyarwood | dansmith: thanks :) | 17:30 |
| stewie925 | hello, I am getting a functional test error while testing instance resize_confirm notification - posted the issue in http://paste.openstack.org/show/588598/. Input appreciated. | 17:31 |
| *** Sukhdev has quit IRC | 17:31 | |
| *** jaugustine_ has quit IRC | 17:31 | |
| *** TravT has joined #openstack-nova | 17:33 | |
| *** gouthamr has quit IRC | 17:35 | |
| dansmith | mriedem: at what point does that series win the prize for the most yanked around, abandoned-restored-abandoned series ever? | 17:37 |
| *** pilgrimstack1 has quit IRC | 17:37 | |
| *** bkopilov has quit IRC | 17:37 | |
| mriedem | hnmm | 17:37 |
| *** gouthamr has joined #openstack-nova | 17:37 | |
| *** prashkre has quit IRC | 17:38 | |
| mriedem | dansmith: this is the prize http://cdn.someecards.com/someecards/filestorage/rather-pot-decided-shit-wedding-ecard-someecards.jpg | 17:38 |
| dansmith | awesome | 17:38 |
| *** baoli has quit IRC | 17:39 | |
| *** baoli has joined #openstack-nova | 17:39 | |
| *** Apoorva has joined #openstack-nova | 17:40 | |
| *** danpb has quit IRC | 17:41 | |
| *** gouthamr has quit IRC | 17:41 | |
| *** gouthamr has joined #openstack-nova | 17:42 | |
| *** victorhe has joined #openstack-nova | 17:43 | |
| *** victorhe has quit IRC | 17:45 | |
| *** victorhe has joined #openstack-nova | 17:46 | |
| *** gouthamr has quit IRC | 17:50 | |
| *** Hosam has joined #openstack-nova | 17:51 | |
| *** Hosam has quit IRC | 17:51 | |
| *** Hosam has joined #openstack-nova | 17:51 | |
| *** gouthamr has joined #openstack-nova | 17:52 | |
| mdbooth | mriedem: http://logs.openstack.org/09/395709/1/check/gate-tempest-dsvm-full-devstack-plugin-nfs-nv/81331d0/ | 17:53 |
| mdbooth | There are a lot of failures in there, but you mentioned it wasn't working anyway? | 17:53 |
| *** bkopilov has joined #openstack-nova | 17:54 | |
| mdbooth | I flicked through and nothing looked obviously related to my patch, but it's hard to tell... | 17:54 |
| mriedem | mdbooth: yeah the snapshot and clone tests need to be disabled in that job | 17:54 |
| mriedem | nfs in cinder doesn't support those operations | 17:54 |
| mriedem | i basically need to put up a patch to the devstack-plugin-nfs repo to configure tempest to disable those in tempest.conf | 17:55 |
| mdbooth | Ok, I could take a run through and ensure that every failure has either 'snapshot' or 'clone' in the name | 17:55 |
| johnthetubaguy | stewie925: seems like 204 is the expected return code: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L808 | 17:55 |
| *** gszasz has quit IRC | 17:56 | |
| *** derekh has quit IRC | 17:57 | |
| *** sach-m has joined #openstack-nova | 17:57 | |
| sneti | johnthetubaguy, can you please review these when you have time: https://review.openstack.org/#/c/384724/ and https://review.openstack.org/#/c/392305/ | 17:59 |
| *** swamireddy has joined #openstack-nova | 17:59 | |
| *** baoli has quit IRC | 18:00 | |
| *** baoli has joined #openstack-nova | 18:00 | |
| *** gouthamr has quit IRC | 18:00 | |
| *** ralonsoh has quit IRC | 18:02 | |
| *** mkoderer has quit IRC | 18:03 | |
| *** fragatin_ has joined #openstack-nova | 18:04 | |
| *** rama_y has joined #openstack-nova | 18:04 | |
| *** kuzko has quit IRC | 18:05 | |
| *** mvk has quit IRC | 18:05 | |
| mdbooth | mriedem: List is here: https://review.openstack.org/#/c/383859/ | 18:06 |
| *** gouthamr has joined #openstack-nova | 18:06 | |
| openstackgerrit | Matthew Edmonds proposed openstack/nova: Add a no-op wait method to NetworkInfo https://review.openstack.org/393669 | 18:07 |
| *** fragatina has quit IRC | 18:07 | |
| *** lucasagomes is now known as lucas-afk | 18:07 | |
| edmondsw | sahid, mriedem, made the changes as discussed ^ | 18:07 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Rename security_group parameter in compute.API:create https://review.openstack.org/395746 | 18:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: Transform requested secgroup names to uuids https://review.openstack.org/395747 | 18:08 |
| mriedem | dansmith: ^ i need to add unit tests for the top change | 18:08 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.rebuild notification https://review.openstack.org/384922 | 18:08 |
| mriedem | but that's what i had in mind | 18:08 |
| mriedem | bottom change is optional but that's always bugged me | 18:09 |
| *** kuzko has joined #openstack-nova | 18:09 | |
| *** htruta has joined #openstack-nova | 18:09 | |
| *** nicolasbock has quit IRC | 18:09 | |
| *** unicell has quit IRC | 18:09 | |
| dansmith | mriedem: okay looking at alex_xu's spec right now then I'll look | 18:09 |
| *** eharney has quit IRC | 18:10 | |
| *** nicolasbock has joined #openstack-nova | 18:11 | |
| mriedem | i'm going to forage for lunch | 18:11 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: Transform requested secgroup names to uuids https://review.openstack.org/395747 | 18:11 |
| *** bnemec has quit IRC | 18:11 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Transform instance.reboot notifications https://review.openstack.org/382959 | 18:12 |
| *** prashkre has joined #openstack-nova | 18:12 | |
| *** bnemec has joined #openstack-nova | 18:13 | |
| *** sach-m has quit IRC | 18:14 | |
| stewie925 | johnthetubaguy: thank you | 18:14 |
| stewie925 | johnthetubaguy: I'll take a look at it | 18:14 |
| *** gouthamr has quit IRC | 18:15 | |
| *** gouthamr has joined #openstack-nova | 18:16 | |
| *** bvanhav__ has quit IRC | 18:17 | |
| *** bvanhav__ has joined #openstack-nova | 18:17 | |
| *** prashkre has quit IRC | 18:19 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: aggregate_metadata_get_by_host cleanup https://review.openstack.org/395713 | 18:19 |
| *** baoli has quit IRC | 18:21 | |
| *** ihrachys has quit IRC | 18:22 | |
| *** baoli has joined #openstack-nova | 18:22 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Add zero down time upgrade steps https://review.openstack.org/393931 | 18:22 |
| *** portdirect is now known as portdirect_away | 18:22 | |
| *** gouthamr_ has joined #openstack-nova | 18:23 | |
| *** gouthamr has quit IRC | 18:23 | |
| *** tbachman has joined #openstack-nova | 18:23 | |
| stewie925 | johnthetubaguy: that is still strange though - HTTP 2xx are non-error status codes, but my functional test shows "nova.tests.functional.api.client.OpenStackApiException: Unexpected status code Status Code: 204" | 18:24 |
| *** lpetrut has quit IRC | 18:24 | |
| stewie925 | http://paste.openstack.org/show/588598/ | 18:24 |
| mriedem | stewie925: it's because the api client thing used in the functional tests only expects i think 200 | 18:24 |
| openstackgerrit | Sujitha proposed openstack/nova: Document for zero downtime upgrade process https://review.openstack.org/395269 | 18:24 |
| *** rodrigods has quit IRC | 18:24 | |
| mriedem | everything else is considered a failure | 18:24 |
| mriedem | stewie925: https://github.com/openstack/nova/blob/master/nova/tests/functional/api/client.py#L188 | 18:25 |
| mriedem | stewie925: is it a GET? https://github.com/openstack/nova/blob/master/nova/tests/functional/api/client.py#L208 | 18:25 |
| stewie925 | mriedem: no its a POST actually - here is the information: http://paste.openstack.org/show/588598/ | 18:26 |
| *** bdeschenes has joined #openstack-nova | 18:26 | |
| mriedem | POST doesn't expect a 204 in there https://github.com/openstack/nova/blob/master/nova/tests/functional/api/client.py#L218 | 18:26 |
| *** jaosorior has quit IRC | 18:26 | |
| melwitt | hah, that reminds me of when I had to add 204 for a functional test I was writing as part of https://review.openstack.org/#/c/355091/7/nova/tests/functional/api/client.py@218 | 18:27 |
| mriedem | so you'd have to add 204 to https://github.com/openstack/nova/blob/master/nova/tests/functional/api/client.py#L218 | 18:27 |
| *** rodrigods has joined #openstack-nova | 18:27 | |
| stewie925 | oh thank you - let me try adding that | 18:27 |
| mriedem | http://developer.openstack.org/api-ref/compute/?expanded=confirm-resized-server-confirmresize-action-detail | 18:28 |
| mriedem | Normal response codes: 204 | 18:28 |
| mriedem | so it's totally OK to add that, the client is busted | 18:28 |
| stewie925 | mriedem I wonder what is the significance of this: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L808 | 18:28 |
| *** roeyc has quit IRC | 18:29 | |
| stewie925 | brb | 18:29 |
| mriedem | that's the response | 18:29 |
| mriedem | well, response status code | 18:29 |
| mriedem | as noted there it's a bug in the v2 API but was carried into v2.1 for backward compatibility | 18:30 |
| *** gouthamr_ is now known as gouthamr | 18:30 | |
| mriedem | we could microversion that out | 18:30 |
| *** mirrorbo1 has joined #openstack-nova | 18:31 | |
| *** mirrorbox has quit IRC | 18:31 | |
| stewie925 | ahhh I see, so these are all related | 18:31 |
| stewie925 | thanks, I'll go ahead and add 204 to the response code as you suggested earlier | 18:32 |
| *** jaugustine_ has joined #openstack-nova | 18:32 | |
| *** ayoung has joined #openstack-nova | 18:32 | |
| *** diga has quit IRC | 18:34 | |
| *** efried has quit IRC | 18:35 | |
| *** unicell has joined #openstack-nova | 18:36 | |
| *** mnestratov has quit IRC | 18:37 | |
| *** jaugustine_ has quit IRC | 18:37 | |
| openstackgerrit | Sujitha proposed openstack/nova: Document for zero downtime upgrade process https://review.openstack.org/395269 | 18:40 |
| *** baoli has quit IRC | 18:42 | |
| *** adisky has quit IRC | 18:43 | |
| *** baoli has joined #openstack-nova | 18:43 | |
| *** gouthamr has quit IRC | 18:44 | |
| *** gouthamr has joined #openstack-nova | 18:45 | |
| *** fragatin_ has quit IRC | 18:47 | |
| cfriesen | danpb: when doing block live migration is there any way to cap the amount of memory used? ie slow down the guest IO rather than consume unlimited buffer space? | 18:47 |
| *** fragatina has joined #openstack-nova | 18:48 | |
| *** baoli has quit IRC | 18:49 | |
| melwitt | when I enable placement-api in devstack, where do the inventories and resource_class_id come from? | 18:49 |
| *** gouthamr has quit IRC | 18:49 | |
| mriedem | melwitt: the resource tracker | 18:49 |
| mriedem | well, the scheduler client used in the resource tracker | 18:49 |
| *** gouthamr has joined #openstack-nova | 18:49 | |
| mriedem | https://github.com/openstack/nova/blob/569e463f02b0a631d341b06849217657a22c4569/nova/scheduler/client/report.py#L360 | 18:50 |
| *** baoli has joined #openstack-nova | 18:50 | |
| *** mvk has joined #openstack-nova | 18:51 | |
| melwitt | thanks mriedem | 18:51 |
| *** amotoki has joined #openstack-nova | 18:55 | |
| *** lpetrut has joined #openstack-nova | 18:55 | |
| *** jamesdenton has quit IRC | 18:57 | |
| *** mbirru has joined #openstack-nova | 18:57 | |
| mriedem | edmondsw: i had a couple of nits in that change if you wanted to address | 18:58 |
| mriedem | not so bad to -1 | 18:58 |
| *** rodrigods has quit IRC | 18:58 | |
| *** harlowja has quit IRC | 18:59 | |
| *** rcernin has joined #openstack-nova | 18:59 | |
| *** amotoki has quit IRC | 18:59 | |
| *** baoli has quit IRC | 19:00 | |
| *** Sukhdev has joined #openstack-nova | 19:00 | |
| *** baoli has joined #openstack-nova | 19:01 | |
| *** ijw has joined #openstack-nova | 19:02 | |
| *** diogogmt has joined #openstack-nova | 19:03 | |
| *** lpetrut has quit IRC | 19:04 | |
| *** eharney has joined #openstack-nova | 19:08 | |
| *** gouthamr has quit IRC | 19:09 | |
| *** catinthe_ has joined #openstack-nova | 19:10 | |
| *** liangy_ has quit IRC | 19:11 | |
| *** oanson has joined #openstack-nova | 19:11 | |
| *** ijw has quit IRC | 19:11 | |
| *** catintheroof has quit IRC | 19:12 | |
| *** armax has joined #openstack-nova | 19:14 | |
| *** gouthamr has joined #openstack-nova | 19:15 | |
| *** coolsvap has quit IRC | 19:17 | |
| *** lpetrut has joined #openstack-nova | 19:17 | |
| *** amotoki has joined #openstack-nova | 19:21 | |
| openstackgerrit | Matthew Edmonds proposed openstack/nova: Add a no-op wait method to NetworkInfo https://review.openstack.org/393669 | 19:21 |
| edmondsw | mriedem tx... addressed ^ | 19:21 |
| *** ijw has joined #openstack-nova | 19:22 | |
| dansmith | edmondsw: isn't that only useful for n-net? | 19:22 |
| *** baoli has quit IRC | 19:22 | |
| edmondsw | dansmith, no | 19:22 |
| dansmith | edmondsw: where do we do non-async allocation with neutron? | 19:23 |
| edmondsw | dansmith on reschedules where the network allocation was already made on the first build attempt | 19:23 |
| dansmith | ah | 19:23 |
| edmondsw | this broke reschedules | 19:24 |
| dansmith | edmondsw: can you point me at where/how that happens? I'm not seeing it | 19:24 |
| dansmith | oh, hmm requested_networks.no_allocate maybe? | 19:24 |
| *** dave-mccowan has quit IRC | 19:25 | |
| dansmith | hrm, no | 19:25 |
| *** amotoki has quit IRC | 19:25 | |
| *** dave-mccowan has joined #openstack-nova | 19:26 | |
| edmondsw | dansmith: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1428-L1437 | 19:27 |
| edmondsw | that yields a NetworkInfo instead of NetworkInfoAsyncWrapper | 19:27 |
| *** rodrigods has joined #openstack-nova | 19:28 | |
| *** baoli_ has joined #openstack-nova | 19:28 | |
| *** baoli__ has joined #openstack-nova | 19:29 | |
| dansmith | ah yeah, I remember this now | 19:29 |
| *** mnestratov has joined #openstack-nova | 19:30 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Fix database poison warnings in resource providers https://review.openstack.org/394944 | 19:32 |
| *** baoli_ has quit IRC | 19:32 | |
| mriedem | and then kablammo https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2072 | 19:32 |
| *** jaugustine_ has joined #openstack-nova | 19:33 | |
| mriedem | edmondsw: +W | 19:34 |
| dansmith | edmondsw: got it | 19:34 |
| mriedem | oops | 19:34 |
| mriedem | +2 | 19:34 |
| dansmith | +W from me | 19:34 |
| dansmith | the system works! | 19:34 |
| edmondsw | tx guys! | 19:34 |
| mriedem | will want that cherry picked back to newton | 19:34 |
| edmondsw | if you have another few minutes, https://review.openstack.org/#/c/393805/ fixes a related-but-different bug | 19:34 |
| dansmith | edmondsw: ooh, so close. classic mistake. | 19:35 |
| edmondsw | related in that it also has to do with network allocation and reschedules | 19:35 |
| edmondsw | nooo!!! | 19:35 |
| *** dave-mccowan has quit IRC | 19:36 | |
| mriedem | what's the classic mistake? double dipping? | 19:36 |
| edmondsw | I thought he was going to pull the +W, but it's still there :) | 19:37 |
| dansmith | mriedem: heh, yeah :) | 19:37 |
| dansmith | edmondsw: -1 on that one, comment left | 19:37 |
| dansmith | should be an easy change | 19:37 |
| edmondsw | +1 | 19:38 |
| edmondsw | tx for looking | 19:39 |
| *** jaugustine_ has quit IRC | 19:39 | |
| *** efried has joined #openstack-nova | 19:42 | |
| mriedem | another very sparse session etherpad https://etherpad.openstack.org/p/ocata-nova-summit-vendoradatav2 | 19:44 |
| mriedem | e.g. i don't remember what the 'hard failure' was for again | 19:44 |
| mriedem | we were going to pass roles on server create, and then re-use those initial roles later on subsequent gets | 19:45 |
| mriedem | even if the roles had changed | 19:45 |
| edmondsw | dansmith, so your assertion is that people are more likely to do obj_clone than to do their own deepcopy? Because if we move that change to the obj_clone, anyone doing their own deepcopy wouldn't get this | 19:45 |
| edmondsw | something to worry about, or not a concern? | 19:46 |
| dansmith | edmondsw: my assertion is that doing your own deepcopy on instance is invalid | 19:46 |
| dansmith | edmondsw: I'd -2 for that | 19:46 |
| edmondsw | k | 19:46 |
| *** clenimar has joined #openstack-nova | 19:46 | |
| dansmith | it's specifically what obj_clone() is for | 19:46 |
| mriedem | mikal: i know you're sitting on the couch but can you clarify at some point what L23 was for in here https://etherpad.openstack.org/p/ocata-nova-summit-vendoradatav2 ? | 19:47 |
| *** ijw has quit IRC | 19:49 | |
| *** paw has quit IRC | 19:50 | |
| mriedem | dansmith: do you remember? ^ | 19:51 |
| dansmith | mriedem: you mean the hard failure one? | 19:51 |
| mriedem | yeah | 19:51 |
| mriedem | i remember talking about something like that but don't remember the context | 19:52 |
| dansmith | mriedem: whether or not we kill the build if the external service returns something other than 200 | 19:52 |
| dansmith | mriedem: some will want it to fail, some will want it to proceed | 19:52 |
| *** stvnoyes has joined #openstack-nova | 19:52 | |
| mriedem | oh when creating config drive | 19:52 |
| dansmith | so it needs to be in config | 19:52 |
| dansmith | mriedem: when polling that service for the vendordata, which may be during configdrive yeah | 19:52 |
| *** clenimar has quit IRC | 19:52 | |
| dansmith | mriedem: see, I remember things :) | 19:52 |
| mriedem | the majority of my sinus cavity was draining on thursday so i'm a bit foggy on the detials | 19:53 |
| mriedem | *details | 19:53 |
| dansmith | heh | 19:53 |
| openstackgerrit | Sujitha proposed openstack/nova: Remove deprecated manager classes in service.py https://review.openstack.org/338553 | 19:53 |
| mriedem | until, until | 19:53 |
| mriedem | the sudafed hookup | 19:53 |
| dansmith | \o/ | 19:53 |
| *** stvnoyes1 has quit IRC | 19:55 | |
| *** oanson has quit IRC | 19:55 | |
| *** rcernin has quit IRC | 19:55 | |
| openstackgerrit | Brent Tang proposed openstack/nova: Instance obj_clone leaves metadata as changed https://review.openstack.org/393805 | 19:57 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Fix database poison warnings in resource providers https://review.openstack.org/394944 | 19:58 |
| *** armax has quit IRC | 20:08 | |
| mriedem | dansmith: if you're not using config drive though, then it's just cloud-init calling the metadata service for vendordata right? and at that point if we fail, the guest is already created, but nova doesn't know about that failure in the guest, so you can still end up with a busted instance. | 20:09 |
| mriedem | so the only thing nova can control to fail the server build is when nova is making the request to build the config drive | 20:10 |
| dansmith | mriedem: yeah, I s'pose so | 20:10 |
| mriedem | ok | 20:10 |
| *** vilobhmm11 has joined #openstack-nova | 20:11 | |
| vilobhmm11 | dansmith : ping | 20:11 |
| vilobhmm11 | bauzas : ping | 20:11 |
| dansmith | vilobhmm11: ack | 20:12 |
| vilobhmm11 | is there a way in nova to just reimage the os and not the data part of it ? nova rebuild <uuid> <new image> will reimage as well as wipe off the data | 20:13 |
| vilobhmm11 | dansmith : ^^ | 20:13 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: Transform requested secgroup names to uuids https://review.openstack.org/395747 | 20:13 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Rename security_group parameter in compute.API:create https://review.openstack.org/395746 | 20:13 |
| mriedem | vilobhmm11: attach a volume? | 20:13 |
| vilobhmm11 | mriedem : considering the fact you don't have remote storage or cinder running :( | 20:14 |
| vilobhmm11 | pretty dumb deployment but it is what it is :( | 20:14 |
| vilobhmm11 | mriedem : ^^ | 20:15 |
| mriedem | there is a preserve_ephemeral flag but it's only used in the case of ironic | 20:15 |
| *** ijw has joined #openstack-nova | 20:15 | |
| mriedem | use shared storage? | 20:16 |
| *** ijw has quit IRC | 20:17 | |
| vilobhmm11 | mriedem : nothing for virtual machines is it ? no there is no shared storage in place …vm disk stored on the HV | 20:17 |
| vilobhmm11 | ephemeral storage = vm disk is created on the hv | 20:17 |
| *** ayoung has quit IRC | 20:17 | |
| vilobhmm11 | mriedem : ^^ | 20:18 |
| mriedem | i was looking at this https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2782 | 20:19 |
| cfriesen | vilobhmm11: in general nova doesn't know what's on the image disk, and so can't rebuild only part of it. | 20:19 |
| *** tlian has quit IRC | 20:20 | |
| *** tlian has joined #openstack-nova | 20:21 | |
| cfriesen | vilobhmm11: libvirt doesn't implement the rebuild() function, so it deletes the old instance and spawns a new one, which kills your ephemeral/swap | 20:22 |
| mriedem | nothing implements rebuild() except ironic | 20:23 |
| *** ijw has joined #openstack-nova | 20:25 | |
| *** jaugustine_ has joined #openstack-nova | 20:28 | |
| *** claudiub has quit IRC | 20:29 | |
| *** ijw has quit IRC | 20:30 | |
| *** aloga_ has joined #openstack-nova | 20:30 | |
| *** ijw has joined #openstack-nova | 20:31 | |
| cfriesen | we have "max_concurrent_builds" and "max_concurrent_live_migrations". Is there an equivalent for cold migrations to prevent disk thrashing on the computes? | 20:32 |
| mriedem | nope | 20:33 |
| *** tblakes has quit IRC | 20:33 | |
| cfriesen | also, would it make sense to have "max_concurrent_live_migrations" apply to incoming migrations as well as outgoing? | 20:33 |
| mriedem | huh? | 20:33 |
| mriedem | what is an outgoing live migration? | 20:33 |
| mriedem | except for democrats moving to canada | 20:34 |
| cfriesen | sorry, make it apply to dest nodes as well as source nodes | 20:34 |
| dansmith | cfriesen: that would require retooling our rpc calls quite a bit I think | 20:34 |
| dirk | mriedem: could you please re-+2 https://review.openstack.org/#/c/389042/ ? there is no change except for updated commit message | 20:36 |
| cfriesen | dansmith: maybe bump the sema in pre_live_migration | 20:36 |
| *** ijw has quit IRC | 20:36 | |
| dansmith | cfriesen: right, but you have a thing waiting on that rpc call at that point, which will time out | 20:36 |
| cfriesen | yeah, fair enough | 20:37 |
| dirk | dear friendly nova core, please review https://review.openstack.org/#/c/389042/ - this blocks a g-r change from going in for almost a month, and it takes 30s to review | 20:37 |
| melwitt | mriedem: do you know what is meant here by "tests are now failing?" https://review.openstack.org/#/c/395261/ the change only went into oslo.context recently, how could we have picked it up? | 20:37 |
| *** jaugustine_ has quit IRC | 20:37 | |
| *** bvanhav__ has quit IRC | 20:38 | |
| *** jaugustine_ has joined #openstack-nova | 20:38 | |
| *** dave-mccowan has joined #openstack-nova | 20:38 | |
| mriedem | dirk: done | 20:38 |
| mriedem | melwitt: the olso gang has a job which runs nova unit tests against all of the unreleased oslo libs | 20:38 |
| mriedem | they run that before releasing the olso libs to make sure they don't break nova unit tests | 20:39 |
| dirk | mriedem: thank you. can you recommend another friendly core who could +2/W+1 it? I've been unsuccessfully pinging here for the last 3 weeks or so on getting this change merged | 20:39 |
| mriedem | dirk: i think melwitt would be exstatic to +W that | 20:39 |
| melwitt | mriedem: ah, okay. didn't know that | 20:39 |
| mriedem | *ecstatic | 20:39 |
| melwitt | that's right, I'm looking at it now | 20:39 |
| mriedem | i don't have a link to the failed job but harlowja might | 20:39 |
| dirk | melwitt: would you mind checking out https://review.openstack.org/#/c/389042/ when you have two minutes of spare time? it is adjusting the requires so that we can finally update to newer oslotest release. | 20:41 |
| melwitt | dirk: yes, done | 20:41 |
| melwitt | thanks for letting us know about it | 20:41 |
| *** aloga_ has quit IRC | 20:43 | |
| *** bdeschenes has quit IRC | 20:44 | |
| *** lbrune has quit IRC | 20:45 | |
| *** Guest83582 has quit IRC | 20:47 | |
| *** baoli__ has quit IRC | 20:47 | |
| *** hamzy has quit IRC | 20:48 | |
| melwitt | mriedem: found one http://logs.openstack.org/periodic/periodic-nova-py27-with-oslo-master/9cf668f/testr_results.html.gz looks like my change to sort failed tests at the top doesn't work totally right :P one of the failures isn't at the top | 20:49 |
| *** baoli has joined #openstack-nova | 20:49 | |
| *** lpetrut has quit IRC | 20:50 | |
| *** baoli_ has joined #openstack-nova | 20:50 | |
| *** lbrune has joined #openstack-nova | 20:51 | |
| mriedem | melwitt: didn't know that was a thing | 20:51 |
| *** baoli__ has joined #openstack-nova | 20:51 | |
| melwitt | yeah, I kept it low key | 20:52 |
| *** baoli has quit IRC | 20:54 | |
| *** baoli_ has quit IRC | 20:55 | |
| *** rnoriega has quit IRC | 20:56 | |
| dirk | melwit: thanks! | 20:57 |
| *** slagle has quit IRC | 20:57 | |
| *** Prophidys has quit IRC | 20:58 | |
| *** jschlueter has joined #openstack-nova | 20:59 | |
| *** tbachman has quit IRC | 21:00 | |
| *** kfarr has quit IRC | 21:02 | |
| *** cmartin has quit IRC | 21:03 | |
| *** jaugustine_ has quit IRC | 21:03 | |
| *** baoli__ has quit IRC | 21:07 | |
| *** baoli has joined #openstack-nova | 21:08 | |
| *** jaugustine_ has joined #openstack-nova | 21:11 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: aggregate_metadata_get_by_host cleanup https://review.openstack.org/395713 | 21:12 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Placement api: 404 response do not indicate what was not found https://review.openstack.org/387674 | 21:12 |
| *** rtheis has quit IRC | 21:14 | |
| *** Hosam has quit IRC | 21:21 | |
| *** gouthamr has quit IRC | 21:24 | |
| *** yamahata has quit IRC | 21:27 | |
| *** Prophidys has joined #openstack-nova | 21:28 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Remove mox in test_block_device_mapping_v1.py https://review.openstack.org/308746 | 21:29 |
| *** smatzek has quit IRC | 21:30 | |
| *** rnoriega has joined #openstack-nova | 21:30 | |
| *** slagle has joined #openstack-nova | 21:31 | |
| *** gouthamr has joined #openstack-nova | 21:32 | |
| *** Jeffrey4l has quit IRC | 21:34 | |
| *** edmondsw has quit IRC | 21:37 | |
| *** vilobhmm11 has quit IRC | 21:39 | |
| *** ayoung has joined #openstack-nova | 21:39 | |
| *** vilobhmm11 has joined #openstack-nova | 21:39 | |
| *** hamzy has joined #openstack-nova | 21:44 | |
| *** ijw has joined #openstack-nova | 21:48 | |
| *** harlowja has joined #openstack-nova | 21:49 | |
| *** thorst_ has quit IRC | 21:49 | |
| *** amotoki has joined #openstack-nova | 21:50 | |
| *** amotoki has quit IRC | 21:53 | |
| *** ijw has quit IRC | 21:56 | |
| *** jwcroppe has quit IRC | 21:57 | |
| *** jwcroppe has joined #openstack-nova | 21:58 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Placement api: Add informative message to 404 response. https://review.openstack.org/392321 | 21:58 |
| *** pkoniszewski_ has joined #openstack-nova | 21:59 | |
| *** rama_y has quit IRC | 21:59 | |
| *** sree has joined #openstack-nova | 22:00 | |
| *** rama_y has joined #openstack-nova | 22:00 | |
| *** vilobhmm11 has quit IRC | 22:01 | |
| *** ijw has joined #openstack-nova | 22:02 | |
| *** jwcroppe has quit IRC | 22:03 | |
| *** gouthamr has quit IRC | 22:03 | |
| *** jheroux has quit IRC | 22:04 | |
| *** sree has quit IRC | 22:04 | |
| *** kuzko has quit IRC | 22:04 | |
| *** mlavalle has quit IRC | 22:04 | |
| *** mlavalle has joined #openstack-nova | 22:05 | |
| *** kuzko has joined #openstack-nova | 22:05 | |
| *** ijw has quit IRC | 22:07 | |
| *** baoli has quit IRC | 22:09 | |
| *** rodrigods has quit IRC | 22:11 | |
| *** rodrigods has joined #openstack-nova | 22:11 | |
| *** pkoniszewski_ has quit IRC | 22:12 | |
| *** baoli has joined #openstack-nova | 22:18 | |
| *** gouthamr has joined #openstack-nova | 22:18 | |
| *** mriedem has quit IRC | 22:19 | |
| *** vilobhmm11 has joined #openstack-nova | 22:23 | |
| *** baoli has quit IRC | 22:24 | |
| *** baoli has joined #openstack-nova | 22:26 | |
| *** baoli has quit IRC | 22:26 | |
| *** victorhe has quit IRC | 22:29 | |
| *** tblakes has joined #openstack-nova | 22:35 | |
| *** yamahata has joined #openstack-nova | 22:37 | |
| *** thorst_ has joined #openstack-nova | 22:39 | |
| *** slaweq_ has quit IRC | 22:39 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Change database poison warning to an exception https://review.openstack.org/349082 | 22:42 |
| *** mdrabe has quit IRC | 22:42 | |
| *** mdrabe has joined #openstack-nova | 22:45 | |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild notification https://review.openstack.org/384922 | 22:45 |
| *** jaugustine_ has quit IRC | 22:46 | |
| *** jaugustine has joined #openstack-nova | 22:46 | |
| *** melwitt has left #openstack-nova | 22:47 | |
| *** melwitt has joined #openstack-nova | 22:47 | |
| *** tbachman has joined #openstack-nova | 22:47 | |
| *** jaugustine has quit IRC | 22:51 | |
| *** ijw has joined #openstack-nova | 22:53 | |
| *** ijw has quit IRC | 22:57 | |
| *** Swami has quit IRC | 22:58 | |
| *** lbrune has quit IRC | 22:58 | |
| *** ijw has joined #openstack-nova | 22:59 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/395862 | 23:01 |
| *** xyang1 has quit IRC | 23:03 | |
| *** ijw has quit IRC | 23:03 | |
| *** catinthe_ has quit IRC | 23:04 | |
| *** esberglu has quit IRC | 23:07 | |
| *** harlowja_ has joined #openstack-nova | 23:08 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Fix database poison warnings in resource providers https://review.openstack.org/394944 | 23:09 |
| *** rrecio has quit IRC | 23:09 | |
| edleafe | diana_clarke: ^^ Updated to make the mocks clearer | 23:11 |
| *** _ducttape_ has joined #openstack-nova | 23:11 | |
| *** harlowja has quit IRC | 23:12 | |
| *** maticue_ has quit IRC | 23:12 | |
| *** ducttape_ has quit IRC | 23:15 | |
| *** _ducttape_ has quit IRC | 23:15 | |
| *** jerrygb has quit IRC | 23:16 | |
| *** ayoung has quit IRC | 23:16 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Live migration rollback on failure https://review.openstack.org/395866 | 23:16 |
| *** mdrabe has quit IRC | 23:18 | |
| *** ijw has joined #openstack-nova | 23:20 | |
| *** esberglu has joined #openstack-nova | 23:21 | |
| *** jerrygb has joined #openstack-nova | 23:23 | |
| *** jwcroppe has joined #openstack-nova | 23:24 | |
| *** jwcroppe has quit IRC | 23:25 | |
| *** esberglu has quit IRC | 23:25 | |
| *** jwcroppe has joined #openstack-nova | 23:25 | |
| *** takashin has joined #openstack-nova | 23:28 | |
| *** ayoung has joined #openstack-nova | 23:28 | |
| *** gmann has joined #openstack-nova | 23:28 | |
| *** jwcroppe has quit IRC | 23:30 | |
| *** Jeffrey4l has joined #openstack-nova | 23:31 | |
| *** rama_y has quit IRC | 23:31 | |
| *** thorst_ has quit IRC | 23:32 | |
| *** Jeffrey4l has quit IRC | 23:32 | |
| *** thorst_ has joined #openstack-nova | 23:32 | |
| *** Jeffrey4l has joined #openstack-nova | 23:32 | |
| *** Jeffrey4l has quit IRC | 23:33 | |
| *** Jeffrey4l has joined #openstack-nova | 23:34 | |
| *** gmann has quit IRC | 23:34 | |
| *** armax has joined #openstack-nova | 23:37 | |
| *** thorst_ has quit IRC | 23:40 | |
| *** gmann has joined #openstack-nova | 23:41 | |
| *** mlavalle has quit IRC | 23:41 | |
| *** lamt has quit IRC | 23:42 | |
| *** Swami has joined #openstack-nova | 23:44 | |
| *** Swami_ has joined #openstack-nova | 23:46 | |
| *** gmann has quit IRC | 23:49 | |
| *** gmann has joined #openstack-nova | 23:52 | |
| *** vilobhmm11 has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
