| *** ssurana has joined #openstack-nova | 00:00 | |
| *** fragatina has quit IRC | 00:01 | |
| *** matrohon has joined #openstack-nova | 00:02 | |
| *** shaohe_feng has quit IRC | 00:02 | |
| *** fragatin_ has quit IRC | 00:02 | |
| *** shaohe_feng has joined #openstack-nova | 00:03 | |
| *** salv-orlando has quit IRC | 00:04 | |
| *** ssurana has quit IRC | 00:04 | |
| *** thorst has quit IRC | 00:04 | |
| *** salv-orlando has joined #openstack-nova | 00:05 | |
| *** fragatina has joined #openstack-nova | 00:05 | |
| *** sdake_ has quit IRC | 00:05 | |
| *** jamesdenton has joined #openstack-nova | 00:06 | |
| *** fragatina has quit IRC | 00:07 | |
| *** fragatin_ has joined #openstack-nova | 00:07 | |
| *** rama_y has quit IRC | 00:08 | |
| *** fragatin_ has quit IRC | 00:08 | |
| *** jamesdenton has quit IRC | 00:11 | |
| *** shaohe_feng has quit IRC | 00:12 | |
| *** shaohe_feng has joined #openstack-nova | 00:13 | |
| *** amotoki has joined #openstack-nova | 00:18 | |
| *** rahuls has quit IRC | 00:18 | |
| *** markvoelker has joined #openstack-nova | 00:20 | |
| *** shaohe_feng has quit IRC | 00:23 | |
| *** shaohe_feng has joined #openstack-nova | 00:24 | |
| *** markvoelker has quit IRC | 00:25 | |
| *** ssurana has joined #openstack-nova | 00:27 | |
| *** FL1SK has joined #openstack-nova | 00:32 | |
| *** shaohe_feng has quit IRC | 00:33 | |
| *** shaohe_feng has joined #openstack-nova | 00:34 | |
| *** mtanino has quit IRC | 00:34 | |
| *** sdake has joined #openstack-nova | 00:38 | |
| *** r-daneel has quit IRC | 00:39 | |
| *** Sukhdev has quit IRC | 00:42 | |
| *** hoangcx has joined #openstack-nova | 00:43 | |
| *** shaohe_feng has quit IRC | 00:43 | |
| *** shaohe_feng has joined #openstack-nova | 00:44 | |
| *** ducttape_ has quit IRC | 00:48 | |
| *** shaohe_feng has quit IRC | 00:53 | |
| *** salv-orlando has quit IRC | 00:53 | |
| *** salv-orlando has joined #openstack-nova | 00:54 | |
| *** shaohe_feng has joined #openstack-nova | 00:54 | |
| *** mnestratov has quit IRC | 00:55 | |
| *** thorst has joined #openstack-nova | 00:56 | |
| *** jwcroppe has quit IRC | 00:56 | |
| *** cheneydc has joined #openstack-nova | 00:57 | |
| *** thorst has quit IRC | 00:58 | |
| *** penick has joined #openstack-nova | 00:58 | |
| *** thorst has joined #openstack-nova | 00:58 | |
| *** sacharya has joined #openstack-nova | 00:59 | |
| *** Daisy has joined #openstack-nova | 01:00 | |
| *** Sukhdev has joined #openstack-nova | 01:03 | |
| *** Sukhdev has quit IRC | 01:04 | |
| *** shaohe_feng has quit IRC | 01:04 | |
| *** jaypipes has joined #openstack-nova | 01:04 | |
| *** sacharya has quit IRC | 01:04 | |
| *** shaohe_feng has joined #openstack-nova | 01:05 | |
| *** thorst has quit IRC | 01:07 | |
| *** harlowja has joined #openstack-nova | 01:08 | |
| *** amotoki has quit IRC | 01:08 | |
| *** nic has quit IRC | 01:10 | |
| *** anteaya has quit IRC | 01:10 | |
| *** shaohe_feng has quit IRC | 01:14 | |
| *** shaohe_feng has joined #openstack-nova | 01:15 | |
| *** gjayavelu has quit IRC | 01:16 | |
| *** browne1 has quit IRC | 01:17 | |
| *** jasondotstar has quit IRC | 01:18 | |
| openstackgerrit | Fei Chen proposed openstack/nova-specs: FPGA Device Management https://review.openstack.org/323130 | 01:21 |
|---|---|---|
| *** Qiming has joined #openstack-nova | 01:23 | |
| *** tbachman has quit IRC | 01:23 | |
| *** shaohe_feng has quit IRC | 01:24 | |
| *** tbachman has joined #openstack-nova | 01:24 | |
| *** shaohe_feng has joined #openstack-nova | 01:25 | |
| *** Apoorva_ has joined #openstack-nova | 01:29 | |
| *** tbachman has quit IRC | 01:29 | |
| *** vishwanathj has joined #openstack-nova | 01:31 | |
| *** Apoorva has quit IRC | 01:32 | |
| *** Apoorva_ has quit IRC | 01:33 | |
| *** Qiming_ has joined #openstack-nova | 01:34 | |
| *** shaohe_feng has quit IRC | 01:34 | |
| *** EinstCrazy has joined #openstack-nova | 01:35 | |
| *** amotoki has joined #openstack-nova | 01:35 | |
| *** shaohe_feng has joined #openstack-nova | 01:35 | |
| *** ababich1 has quit IRC | 01:40 | |
| *** ababich has joined #openstack-nova | 01:40 | |
| *** ssurana has quit IRC | 01:40 | |
| *** ducttape_ has joined #openstack-nova | 01:44 | |
| *** shaohe_feng has quit IRC | 01:45 | |
| *** zz_dimtruck is now known as dimtruck | 01:45 | |
| *** shaohe_feng has joined #openstack-nova | 01:45 | |
| *** alt_oomichi has joined #openstack-nova | 01:46 | |
| *** Qiming_ has quit IRC | 01:46 | |
| *** chenfei has joined #openstack-nova | 01:48 | |
| *** sdake has quit IRC | 01:50 | |
| *** yamahata has quit IRC | 01:51 | |
| *** ababich has quit IRC | 01:52 | |
| *** raunak has quit IRC | 01:52 | |
| *** ababich has joined #openstack-nova | 01:52 | |
| *** shaohe_feng has quit IRC | 01:55 | |
| *** shaohe_feng has joined #openstack-nova | 01:55 | |
| *** sacharya has joined #openstack-nova | 01:59 | |
| *** amotoki has quit IRC | 02:00 | |
| *** gjayavelu has joined #openstack-nova | 02:04 | |
| *** thorst has joined #openstack-nova | 02:05 | |
| *** gjayavelu has quit IRC | 02:05 | |
| *** shaohe_feng has quit IRC | 02:05 | |
| *** alt_oomichi has quit IRC | 02:06 | |
| *** shaohe_feng has joined #openstack-nova | 02:06 | |
| *** ducttape_ has quit IRC | 02:09 | |
| *** ducttape_ has joined #openstack-nova | 02:09 | |
| *** penick has quit IRC | 02:10 | |
| *** sdake has joined #openstack-nova | 02:11 | |
| *** thorst has quit IRC | 02:12 | |
| *** amotoki has joined #openstack-nova | 02:12 | |
| *** alt_oomichi has joined #openstack-nova | 02:15 | |
| *** shaohe_feng has quit IRC | 02:15 | |
| *** shaohe_feng has joined #openstack-nova | 02:16 | |
| *** browne has joined #openstack-nova | 02:18 | |
| *** sdake_ has joined #openstack-nova | 02:18 | |
| *** ducttape_ has quit IRC | 02:19 | |
| *** raunak has joined #openstack-nova | 02:19 | |
| *** sdake has quit IRC | 02:20 | |
| *** markvoelker has joined #openstack-nova | 02:21 | |
| *** markvoelker has quit IRC | 02:26 | |
| *** shaohe_feng has quit IRC | 02:26 | |
| *** shaohe_feng has joined #openstack-nova | 02:26 | |
| *** jamielennox is now known as jamielennox|away | 02:27 | |
| *** jamielennox|away is now known as jamielennox | 02:35 | |
| *** ducttape_ has joined #openstack-nova | 02:35 | |
| *** shaohe_feng has quit IRC | 02:36 | |
| *** shaohe_feng has joined #openstack-nova | 02:36 | |
| *** stewie925 has quit IRC | 02:39 | |
| *** sudipto has joined #openstack-nova | 02:42 | |
| *** mrkz has quit IRC | 02:45 | |
| *** shaohe_feng has quit IRC | 02:46 | |
| *** shaohe_feng has joined #openstack-nova | 02:47 | |
| *** phuongnh has joined #openstack-nova | 02:49 | |
| *** houming has joined #openstack-nova | 02:50 | |
| *** matrohon has quit IRC | 02:51 | |
| *** gjayavelu has joined #openstack-nova | 02:56 | |
| *** shaohe_feng has quit IRC | 02:56 | |
| *** shaohe_feng has joined #openstack-nova | 02:57 | |
| *** houming has quit IRC | 02:59 | |
| *** adisky has joined #openstack-nova | 03:00 | |
| *** sheel has joined #openstack-nova | 03:01 | |
| *** houming has joined #openstack-nova | 03:06 | |
| *** shaohe_feng has quit IRC | 03:07 | |
| *** shaohe_feng has joined #openstack-nova | 03:07 | |
| *** gjayavelu has quit IRC | 03:08 | |
| *** fragatina has joined #openstack-nova | 03:10 | |
| *** thorst has joined #openstack-nova | 03:10 | |
| *** fragatin_ has joined #openstack-nova | 03:11 | |
| *** phuongnh has quit IRC | 03:13 | |
| *** rama_y has joined #openstack-nova | 03:14 | |
| *** fragatina has quit IRC | 03:15 | |
| *** Daisy has quit IRC | 03:16 | |
| *** Daisy has joined #openstack-nova | 03:17 | |
| *** thorst has quit IRC | 03:17 | |
| *** shaohe_feng has quit IRC | 03:17 | |
| *** shaohe_feng has joined #openstack-nova | 03:18 | |
| *** ducttape_ has quit IRC | 03:23 | |
| openstackgerrit | Brian Moss proposed openstack/nova: Copy edit feature classification https://review.openstack.org/323671 | 03:24 |
| *** Daisy_ has joined #openstack-nova | 03:24 | |
| *** shaohe_feng has quit IRC | 03:27 | |
| *** Daisy has quit IRC | 03:28 | |
| *** shaohe_feng has joined #openstack-nova | 03:28 | |
| *** Daisy_ has quit IRC | 03:28 | |
| *** Daisy has joined #openstack-nova | 03:29 | |
| *** tbachman has joined #openstack-nova | 03:30 | |
| *** Daisy_ has joined #openstack-nova | 03:30 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Remove deprecated config option volume_api_class https://review.openstack.org/319530 | 03:31 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Remove deprecated configuration option network_device_mtu https://review.openstack.org/319543 | 03:33 |
| *** Daisy has quit IRC | 03:34 | |
| *** tbachman has quit IRC | 03:34 | |
| *** Daisy_ has quit IRC | 03:35 | |
| *** tbachman has joined #openstack-nova | 03:36 | |
| *** shaohe_feng has quit IRC | 03:37 | |
| *** sdake_ has quit IRC | 03:38 | |
| *** shaohe_feng has joined #openstack-nova | 03:39 | |
| *** tbachman has quit IRC | 03:41 | |
| *** zerda2 has joined #openstack-nova | 03:43 | |
| openstackgerrit | Anh Tran proposed openstack/nova: Fix wrong patch of unittest in unit/test_metadata.py https://review.openstack.org/326218 | 03:43 |
| *** jamielennox is now known as jamielennox|away | 03:44 | |
| *** shaohe_feng has quit IRC | 03:48 | |
| *** shaohe_feng has joined #openstack-nova | 03:49 | |
| *** jamesdenton has joined #openstack-nova | 03:52 | |
| *** jamesdenton has quit IRC | 03:52 | |
| *** links has joined #openstack-nova | 03:56 | |
| *** shaohe_feng has quit IRC | 03:58 | |
| *** jamielennox|away is now known as jamielennox | 03:59 | |
| *** shaohe_feng has joined #openstack-nova | 04:01 | |
| openstackgerrit | Tomi Juvonen proposed openstack/nova-specs: Maintenance reason to server https://review.openstack.org/310510 | 04:02 |
| *** tbachman has joined #openstack-nova | 04:04 | |
| *** yamahata has joined #openstack-nova | 04:04 | |
| *** divya___ has quit IRC | 04:05 | |
| *** ijw has quit IRC | 04:06 | |
| alt_oomichi | Dinesh_Bhor: please check sdague comment on https://review.openstack.org/#/c/315964/ | 04:08 |
| *** woodster_ has quit IRC | 04:08 | |
| *** shaohe_feng has quit IRC | 04:08 | |
| *** tbachman has quit IRC | 04:10 | |
| *** zhongshengping has joined #openstack-nova | 04:10 | |
| *** shaohe_feng has joined #openstack-nova | 04:11 | |
| *** pcaruana has joined #openstack-nova | 04:11 | |
| *** zhongshengping has left #openstack-nova | 04:11 | |
| *** tbachman has joined #openstack-nova | 04:12 | |
| *** thorst has joined #openstack-nova | 04:15 | |
| *** dimtruck is now known as zz_dimtruck | 04:18 | |
| *** Daisy has joined #openstack-nova | 04:18 | |
| *** shaohe_feng has quit IRC | 04:18 | |
| *** tbachman_ has joined #openstack-nova | 04:19 | |
| *** shaohe_feng has joined #openstack-nova | 04:19 | |
| *** tbachman has quit IRC | 04:20 | |
| *** tbachman_ is now known as tbachman | 04:20 | |
| *** ducnc has joined #openstack-nova | 04:21 | |
| *** markvoelker has joined #openstack-nova | 04:22 | |
| *** thorst has quit IRC | 04:22 | |
| *** pcaruana has quit IRC | 04:24 | |
| *** tbachman has quit IRC | 04:25 | |
| *** salv-orl_ has joined #openstack-nova | 04:25 | |
| *** markvoelker has quit IRC | 04:26 | |
| *** salv-orlando has quit IRC | 04:27 | |
| *** rama_y has quit IRC | 04:29 | |
| *** shaohe_feng has quit IRC | 04:29 | |
| *** haukebruno has joined #openstack-nova | 04:30 | |
| *** shaohe_feng has joined #openstack-nova | 04:30 | |
| *** mdrabe has joined #openstack-nova | 04:33 | |
| *** rama_y has joined #openstack-nova | 04:33 | |
| *** pcaruana has joined #openstack-nova | 04:34 | |
| *** dave-mccowan has quit IRC | 04:35 | |
| *** sacharya has quit IRC | 04:36 | |
| *** sacharya has joined #openstack-nova | 04:37 | |
| *** prateek has joined #openstack-nova | 04:37 | |
| *** shaohe_feng has quit IRC | 04:39 | |
| *** shaohe_feng has joined #openstack-nova | 04:40 | |
| *** pcaruana has quit IRC | 04:40 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Test: Don't merge https://review.openstack.org/326223 | 04:41 |
| *** prateek has quit IRC | 04:42 | |
| *** sacharya has quit IRC | 04:42 | |
| *** sudipto has quit IRC | 04:43 | |
| *** ljxiash has joined #openstack-nova | 04:47 | |
| *** shaohe_feng has quit IRC | 04:49 | |
| *** shaohe_feng has joined #openstack-nova | 04:50 | |
| *** Daisy has quit IRC | 04:52 | |
| *** mdrabe has quit IRC | 04:52 | |
| *** alt_oomichi has quit IRC | 04:56 | |
| *** shaohe_feng has quit IRC | 04:59 | |
| *** shaohe_feng has joined #openstack-nova | 05:00 | |
| *** ljxiash has quit IRC | 05:04 | |
| *** ljxiash has joined #openstack-nova | 05:04 | |
| *** jaosorior has joined #openstack-nova | 05:05 | |
| *** moshele has joined #openstack-nova | 05:07 | |
| *** ljxiash has quit IRC | 05:09 | |
| *** shaohe_feng has quit IRC | 05:10 | |
| *** shaohe_feng has joined #openstack-nova | 05:11 | |
| *** Daisy has joined #openstack-nova | 05:13 | |
| *** Daisy has quit IRC | 05:14 | |
| *** Daisy has joined #openstack-nova | 05:15 | |
| *** ratailor has joined #openstack-nova | 05:16 | |
| *** ljxiash has joined #openstack-nova | 05:17 | |
| *** lpetrut has joined #openstack-nova | 05:18 | |
| *** shaohe_feng has quit IRC | 05:20 | |
| *** Sukhdev has joined #openstack-nova | 05:21 | |
| *** shaohe_feng has joined #openstack-nova | 05:22 | |
| *** thorst has joined #openstack-nova | 05:22 | |
| *** markvoelker has joined #openstack-nova | 05:23 | |
| *** ducttape_ has joined #openstack-nova | 05:23 | |
| *** diga has joined #openstack-nova | 05:24 | |
| *** markvoelker has quit IRC | 05:27 | |
| *** thorst has quit IRC | 05:28 | |
| *** ducttape_ has quit IRC | 05:28 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/nova: Keep py3.X compatibility for urllib https://review.openstack.org/261034 | 05:30 |
| *** diga_ has joined #openstack-nova | 05:30 | |
| *** shaohe_feng has quit IRC | 05:30 | |
| *** shaohe_feng has joined #openstack-nova | 05:31 | |
| moshele | dansmith, jaypipes: can you review some pci patches https://review.openstack.org/#/c/310990/, https://review.openstack.org/#/c/312028/ | 05:32 |
| *** Daisy_ has joined #openstack-nova | 05:33 | |
| *** kaisers1 has quit IRC | 05:34 | |
| *** gjayavelu has joined #openstack-nova | 05:35 | |
| *** rcernin has joined #openstack-nova | 05:36 | |
| *** yamahata has quit IRC | 05:36 | |
| *** Daisy has quit IRC | 05:36 | |
| *** yamahata has joined #openstack-nova | 05:37 | |
| *** sacharya has joined #openstack-nova | 05:38 | |
| *** sree has joined #openstack-nova | 05:39 | |
| *** salv-orl_ has quit IRC | 05:40 | |
| *** salv-orlando has joined #openstack-nova | 05:40 | |
| *** shaohe_feng has quit IRC | 05:40 | |
| *** Sukhdev has quit IRC | 05:41 | |
| *** sudipto has joined #openstack-nova | 05:41 | |
| *** shaohe_feng has joined #openstack-nova | 05:41 | |
| *** Sukhdev has joined #openstack-nova | 05:41 | |
| *** rama_y has quit IRC | 05:42 | |
| *** sacharya has quit IRC | 05:43 | |
| *** ankit__ has joined #openstack-nova | 05:43 | |
| *** Daisy_ has quit IRC | 05:48 | |
| *** Daisy has joined #openstack-nova | 05:48 | |
| *** josecastroleon has joined #openstack-nova | 05:49 | |
| *** chandankumar has joined #openstack-nova | 05:49 | |
| *** shaohe_feng has quit IRC | 05:51 | |
| *** shaohe_feng has joined #openstack-nova | 05:52 | |
| *** jaosorior has quit IRC | 05:52 | |
| *** mjura has joined #openstack-nova | 05:52 | |
| *** Daisy has quit IRC | 05:53 | |
| *** jaosorior has joined #openstack-nova | 05:53 | |
| *** ljxiash_ has joined #openstack-nova | 05:57 | |
| *** ecelik has joined #openstack-nova | 05:59 | |
| *** jianghuaw has joined #openstack-nova | 05:59 | |
| *** mjura has quit IRC | 05:59 | |
| *** mjura has joined #openstack-nova | 06:00 | |
| *** ljxiash has quit IRC | 06:01 | |
| *** shaohe_feng has quit IRC | 06:01 | |
| *** shaohe_feng has joined #openstack-nova | 06:01 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Fix race for compute node create record, on compute-start https://review.openstack.org/304415 | 06:02 |
| *** srushti has joined #openstack-nova | 06:03 | |
| *** jaosorior has quit IRC | 06:06 | |
| *** jaosorior has joined #openstack-nova | 06:07 | |
| *** apoorv has joined #openstack-nova | 06:08 | |
| *** lpetrut has quit IRC | 06:08 | |
| *** diga_ has quit IRC | 06:09 | |
| *** diga has quit IRC | 06:10 | |
| *** sudipto has quit IRC | 06:10 | |
| *** jaosorior has quit IRC | 06:11 | |
| *** shaohe_feng has quit IRC | 06:11 | |
| *** jaosorior has joined #openstack-nova | 06:12 | |
| *** shaohe_feng has joined #openstack-nova | 06:12 | |
| *** haukebruno has quit IRC | 06:12 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 06:13 |
| *** Daisy has joined #openstack-nova | 06:18 | |
| *** gongysh has joined #openstack-nova | 06:18 | |
| *** jaosorior has quit IRC | 06:21 | |
| *** shaohe_feng has quit IRC | 06:21 | |
| *** mjura has quit IRC | 06:22 | |
| *** ljxiash has joined #openstack-nova | 06:22 | |
| *** shaohe_feng has joined #openstack-nova | 06:22 | |
| *** xiucai has joined #openstack-nova | 06:23 | |
| *** ijw has joined #openstack-nova | 06:23 | |
| *** jaosorior has joined #openstack-nova | 06:24 | |
| *** thorst has joined #openstack-nova | 06:25 | |
| *** shihanzhang has joined #openstack-nova | 06:25 | |
| *** ljxiash_ has quit IRC | 06:26 | |
| *** haukebruno has joined #openstack-nova | 06:26 | |
| *** sudipto has joined #openstack-nova | 06:26 | |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Stop using mox from objects/test_instance.py (1) https://review.openstack.org/326245 | 06:27 |
| *** raunak has quit IRC | 06:27 | |
| tdurakov | hello everyone | 06:27 |
| *** ljxiash has quit IRC | 06:28 | |
| *** liusheng has quit IRC | 06:28 | |
| *** moshele has quit IRC | 06:31 | |
| *** thorst has quit IRC | 06:32 | |
| *** shaohe_feng has quit IRC | 06:32 | |
| *** shaohe_feng has joined #openstack-nova | 06:32 | |
| *** sudipto has quit IRC | 06:33 | |
| openstackgerrit | Chen Fan proposed openstack/nova: filters: remove the local full/part msg variable https://review.openstack.org/325329 | 06:33 |
| openstackgerrit | Chen Fan proposed openstack/nova: update corresponding filter cases caused by output changed https://review.openstack.org/326249 | 06:33 |
| *** fragatin_ has quit IRC | 06:34 | |
| *** vtech has quit IRC | 06:40 | |
| *** shaohe_feng has quit IRC | 06:42 | |
| *** shaohe_feng has joined #openstack-nova | 06:43 | |
| *** xiucai has left #openstack-nova | 06:46 | |
| *** ljxiash has joined #openstack-nova | 06:49 | |
| *** sudipto has joined #openstack-nova | 06:49 | |
| *** andreas_s has joined #openstack-nova | 06:52 | |
| *** Daisy has quit IRC | 06:52 | |
| *** shaohe_feng has quit IRC | 06:52 | |
| *** Daisy has joined #openstack-nova | 06:53 | |
| *** shaohe_feng has joined #openstack-nova | 06:53 | |
| *** ljxiash has quit IRC | 06:54 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Don't use VIR_MIGRATE_TUNNELLED for block migration with volumes https://review.openstack.org/326258 | 06:57 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/325541 | 06:58 |
| tangchen | tdurakov: Hi Timofey, I think the migration state machine job will be delayed to the next cycle because of its influence. So, what do you think I can do in this cycle for the conductor refactor or any work related to it ? | 06:59 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end) https://review.openstack.org/250283 | 06:59 |
| *** vtech has joined #openstack-nova | 07:00 | |
| *** jaosorior has quit IRC | 07:00 | |
| *** jaosorior has joined #openstack-nova | 07:00 | |
| *** mpavone has joined #openstack-nova | 07:01 | |
| *** sridharg has joined #openstack-nova | 07:01 | |
| *** jaosorior has quit IRC | 07:01 | |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Stop using mox from objects/test_instance.py (2) https://review.openstack.org/326260 | 07:02 |
| tdurakov | tangchen: hi, I think it worth to try adressing all comments you received first, maybe some PoC to mitigate concerns about upgrade issues | 07:02 |
| *** shaohe_feng has quit IRC | 07:02 | |
| *** shaohe_feng has joined #openstack-nova | 07:03 | |
| *** salv-orlando has quit IRC | 07:04 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Revert "Optimize _cleanup_incomplete_migrations periodic task" https://review.openstack.org/326262 | 07:05 |
| *** salv-orlando has joined #openstack-nova | 07:05 | |
| tangchen | tdurakov: By upgrade, do you mean upgrade from one release to the next ? like Mitaka -> Newton | 07:05 |
| tdurakov | tangchen: yup | 07:05 |
| tangchen | tdurakov: Sure. Will try it first. :) | 07:05 |
| tangchen | tdurakov: Thanks. | 07:06 |
| tdurakov | tangchen: have you seen my comment on your spec? | 07:06 |
| tangchen | tdurakov: Oh, not the latest one. Reading it. | 07:06 |
| *** Sukhdev has quit IRC | 07:06 | |
| *** mjura_ has joined #openstack-nova | 07:08 | |
| *** mjura has joined #openstack-nova | 07:08 | |
| tangchen | tdurakov: BTW, how to update nova dev-ref ? I can post the diagram first. | 07:08 |
| *** mjura has quit IRC | 07:09 | |
| tdurakov | tangchen: I expect this should be it https://github.com/openstack/nova/blob/master/doc/source/ | 07:09 |
| *** chhavi has joined #openstack-nova | 07:09 | |
| tdurakov | johnthetubaguy:hi, could you confirm that^ | 07:10 |
| *** vishwanathj has quit IRC | 07:11 | |
| alex_xu | tdurakov: tangchen yes, devref is under that directory | 07:11 |
| tdurakov | alex_xu: thanks! | 07:11 |
| alex_xu | tdurakov: np | 07:12 |
| *** jichen has joined #openstack-nova | 07:12 | |
| *** sahid has joined #openstack-nova | 07:12 | |
| tangchen | tdurakov, alex_xu: Thanks. Let me start it from here :) | 07:12 |
| gmann | alex_xu: 1 quick question, host list will give all host including down host also right? | 07:13 |
| openstackgerrit | Alex Xu proposed openstack/nova: Remove policy checkpoints for SecurityGroupAPI and NetworkAPI https://review.openstack.org/325641 | 07:13 |
| openstackgerrit | Alex Xu proposed openstack/nova: Remove the skip_policy_check flags https://review.openstack.org/325684 | 07:13 |
| *** shaohe_feng has quit IRC | 07:13 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Remove policy checkpoints from ComputeAPI https://review.openstack.org/320752 | 07:13 |
| alex_xu | gmann: yes | 07:13 |
| *** shaohe_feng has joined #openstack-nova | 07:13 | |
| gmann | alex_xu: ok | 07:14 |
| *** ljxiash has joined #openstack-nova | 07:17 | |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Stop using mox from objects/test_instance.py (3) https://review.openstack.org/326267 | 07:18 |
| *** ljxiash has quit IRC | 07:21 | |
| *** shaohe_feng has quit IRC | 07:23 | |
| *** matrohon has joined #openstack-nova | 07:23 | |
| *** markvoelker has joined #openstack-nova | 07:23 | |
| *** shaohe_feng has joined #openstack-nova | 07:24 | |
| *** jaosorior has joined #openstack-nova | 07:24 | |
| *** ducttape_ has joined #openstack-nova | 07:25 | |
| *** Daisy_ has joined #openstack-nova | 07:27 | |
| *** markvoelker has quit IRC | 07:28 | |
| *** liverpooler has joined #openstack-nova | 07:28 | |
| *** winston-d_ has joined #openstack-nova | 07:29 | |
| *** paul-carlton2 has joined #openstack-nova | 07:29 | |
| *** thorst has joined #openstack-nova | 07:30 | |
| *** ducttape_ has quit IRC | 07:30 | |
| *** ankit__ has quit IRC | 07:31 | |
| *** Daisy has quit IRC | 07:31 | |
| *** shaohe_feng has quit IRC | 07:33 | |
| *** liverpooler has quit IRC | 07:34 | |
| *** edand has joined #openstack-nova | 07:34 | |
| *** shaohe_feng has joined #openstack-nova | 07:34 | |
| *** jlanoux has joined #openstack-nova | 07:34 | |
| *** jed56 has joined #openstack-nova | 07:34 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs: New Nova Scheduler filter: AggregateInstanceTypeFilter https://review.openstack.org/314097 | 07:34 |
| *** gjayavel_ has joined #openstack-nova | 07:35 | |
| *** gjayavelu has quit IRC | 07:36 | |
| *** thorst has quit IRC | 07:37 | |
| *** markus_z has joined #openstack-nova | 07:38 | |
| *** liusheng has joined #openstack-nova | 07:40 | |
| markus_z | nova bugs team meeting in 20 min in #openstack-meeting-4 https://wiki.openstack.org/wiki/Meetings/Nova/BugsTeam | 07:40 |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Remove mox from api/openstack/compute/test_image_size.py https://review.openstack.org/322690 | 07:41 |
| *** roeyc has joined #openstack-nova | 07:42 | |
| *** ihrachys has joined #openstack-nova | 07:42 | |
| *** ijw has quit IRC | 07:43 | |
| *** shaohe_feng has quit IRC | 07:43 | |
| *** shaohe_feng has joined #openstack-nova | 07:44 | |
| *** mfedosin has joined #openstack-nova | 07:45 | |
| openstackgerrit | Chen Fan proposed openstack/nova: filters: remove the local full/part msg variable https://review.openstack.org/325329 | 07:52 |
| *** shaohe_feng has quit IRC | 07:54 | |
| *** shaohe_feng has joined #openstack-nova | 07:54 | |
| *** salv-orlando has quit IRC | 07:56 | |
| *** salv-orlando has joined #openstack-nova | 07:56 | |
| ratailor | Can anyone review https://review.openstack.org/#/c/312431/ | 07:57 |
| *** zzzeek has quit IRC | 08:00 | |
| markus_z | nova bugs team meeting in #openstack-meeting-4 https://wiki.openstack.org/wiki/Meetings/Nova/BugsTeam | 08:00 |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** gjayavel_ has quit IRC | 08:02 | |
| *** shaohe_feng has quit IRC | 08:04 | |
| *** shaohe_feng has joined #openstack-nova | 08:05 | |
| *** mjura_ has quit IRC | 08:06 | |
| *** nkrinner has joined #openstack-nova | 08:07 | |
| *** flip214 has quit IRC | 08:08 | |
| *** laszloh has joined #openstack-nova | 08:09 | |
| *** flip214 has joined #openstack-nova | 08:10 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
| *** danpb has joined #openstack-nova | 08:10 | |
| *** markus_z has quit IRC | 08:11 | |
| *** ljxiash has joined #openstack-nova | 08:11 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Archive migrations table https://review.openstack.org/323684 | 08:13 |
| *** iceyao has joined #openstack-nova | 08:14 | |
| *** shaohe_feng has quit IRC | 08:14 | |
| openstackgerrit | Li Yingjun proposed openstack/nova: Implement hypervisor notifications https://review.openstack.org/315312 | 08:14 |
| *** shaohe_feng has joined #openstack-nova | 08:15 | |
| *** ljxiash has quit IRC | 08:17 | |
| *** hoangcx has quit IRC | 08:20 | |
| *** paul-carlton2 has quit IRC | 08:21 | |
| *** mvk has quit IRC | 08:22 | |
| *** jaosorior has quit IRC | 08:23 | |
| *** jaosorior has joined #openstack-nova | 08:24 | |
| *** tyrola has joined #openstack-nova | 08:24 | |
| *** roeyc has quit IRC | 08:24 | |
| *** shaohe_feng has quit IRC | 08:25 | |
| *** shaohe_feng has joined #openstack-nova | 08:25 | |
| *** ljxiash has joined #openstack-nova | 08:26 | |
| *** mjura has joined #openstack-nova | 08:28 | |
| *** zul has quit IRC | 08:32 | |
| *** shaohe_feng has quit IRC | 08:35 | |
| *** thorst has joined #openstack-nova | 08:35 | |
| *** paul-carlton2 has joined #openstack-nova | 08:35 | |
| *** shaohe_feng has joined #openstack-nova | 08:36 | |
| tdurakov | danpb: hi, thank you for comment on https://review.openstack.org/#/c/326258 will resubmit patch soon | 08:36 |
| *** Daisy_ has quit IRC | 08:36 | |
| *** Daisy has joined #openstack-nova | 08:37 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: rest api version bumped for async pre live migration checks https://review.openstack.org/314932 | 08:37 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Make live migration checks async https://review.openstack.org/311116 | 08:37 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Added instance actions for conductor https://review.openstack.org/311115 | 08:37 |
| *** jianghuaw has quit IRC | 08:39 | |
| *** MarcF has joined #openstack-nova | 08:40 | |
| *** Daisy has quit IRC | 08:41 | |
| *** thorst has quit IRC | 08:42 | |
| *** zenoway has joined #openstack-nova | 08:44 | |
| *** Daisy has joined #openstack-nova | 08:45 | |
| *** p01nt3r75 has joined #openstack-nova | 08:45 | |
| *** shaohe_feng has quit IRC | 08:45 | |
| *** zul has joined #openstack-nova | 08:46 | |
| *** shaohe_feng has joined #openstack-nova | 08:46 | |
| *** derekh has joined #openstack-nova | 08:46 | |
| *** hoangcx has joined #openstack-nova | 08:46 | |
| *** liusheng has quit IRC | 08:48 | |
| *** liusheng has joined #openstack-nova | 08:48 | |
| *** paul-carlton2 has quit IRC | 08:48 | |
| *** paul-carlton2 has joined #openstack-nova | 08:49 | |
| *** mvk has joined #openstack-nova | 08:50 | |
| *** tesseract has joined #openstack-nova | 08:50 | |
| *** moshele has joined #openstack-nova | 08:51 | |
| *** yassine has joined #openstack-nova | 08:52 | |
| *** yassine has quit IRC | 08:52 | |
| *** yassine has joined #openstack-nova | 08:52 | |
| *** cdent has joined #openstack-nova | 08:54 | |
| *** shaohe_feng has quit IRC | 08:55 | |
| *** Daisy_ has joined #openstack-nova | 08:56 | |
| *** Jeffrey4l has joined #openstack-nova | 08:57 | |
| *** shaohe_feng has joined #openstack-nova | 08:58 | |
| *** Daisy has quit IRC | 08:58 | |
| *** takashin has left #openstack-nova | 09:01 | |
| *** chandankumar has quit IRC | 09:02 | |
| *** dmk0202 has joined #openstack-nova | 09:05 | |
| *** gongysh has quit IRC | 09:05 | |
| *** shaohe_feng has quit IRC | 09:06 | |
| *** moshele has quit IRC | 09:06 | |
| *** BobBall_AWOL has quit IRC | 09:06 | |
| *** shaohe_feng has joined #openstack-nova | 09:06 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Update libvirt to use the new imagebackend methods https://review.openstack.org/282580 | 09:06 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Use oslo_config new type PortOpt for port options https://review.openstack.org/238347 | 09:06 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Base enablement for memory bandwidth monitor https://review.openstack.org/266773 | 09:07 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: add memory bandwidth monitor base class https://review.openstack.org/270079 | 09:07 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: define NUMA memory bw monitor https://review.openstack.org/270344 | 09:07 |
| *** Daisy_ has quit IRC | 09:07 | |
| *** Daisy has joined #openstack-nova | 09:08 | |
| *** Daisy has quit IRC | 09:11 | |
| *** Daisy_ has joined #openstack-nova | 09:11 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Update libvirt to use the new imagebackend methods https://review.openstack.org/282580 | 09:13 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Delete old imagebackend methods & tests https://review.openstack.org/322974 | 09:15 |
| *** shaohe_feng has quit IRC | 09:16 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Remove the last bits of mox from test_imagebackend https://review.openstack.org/323038 | 09:16 |
| *** shaohe_feng has joined #openstack-nova | 09:17 | |
| *** p01nt3r75 has quit IRC | 09:17 | |
| *** sfinucan has joined #openstack-nova | 09:19 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Base enablement for memory bandwidth monitor https://review.openstack.org/266773 | 09:22 |
| *** sambetts|afk is now known as sambetts | 09:23 | |
| *** markvoelker has joined #openstack-nova | 09:24 | |
| *** shaohe_feng has quit IRC | 09:26 | |
| *** ducttape_ has joined #openstack-nova | 09:26 | |
| *** mgoddard_ has quit IRC | 09:26 | |
| *** shaohe_feng has joined #openstack-nova | 09:27 | |
| *** Daisy_ has quit IRC | 09:28 | |
| *** Daisy has joined #openstack-nova | 09:28 | |
| *** markvoelker has quit IRC | 09:29 | |
| *** yamahata has quit IRC | 09:30 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP] Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | 09:30 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transfrom wrap_exception notification to versioned format https://review.openstack.org/310700 | 09:31 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Enhance notification sample test base https://review.openstack.org/321610 | 09:31 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Handle multiple samples per versioned notification https://review.openstack.org/321609 | 09:31 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.update notification https://review.openstack.org/325923 | 09:31 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Do not register notification objects https://review.openstack.org/309454 | 09:31 |
| *** ducttape_ has quit IRC | 09:31 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.delete notifications https://review.openstack.org/313654 | 09:31 |
| *** houming has quit IRC | 09:33 | |
| *** chhavi has quit IRC | 09:34 | |
| *** caowei has joined #openstack-nova | 09:34 | |
| *** roeyc has joined #openstack-nova | 09:35 | |
| *** shaohe_feng has quit IRC | 09:36 | |
| *** caowei has quit IRC | 09:37 | |
| *** shaohe_feng has joined #openstack-nova | 09:37 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Fix imagecache.get_cache_fname() to work in python3 https://review.openstack.org/321579 | 09:37 |
| *** caowei has joined #openstack-nova | 09:37 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Qcow2: add create_from_image & create_from_func https://review.openstack.org/320610 | 09:37 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Lvm: add create_from_image & create_from_func https://review.openstack.org/320611 | 09:37 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Image: add create_from_image & create_from_func https://review.openstack.org/318948 | 09:37 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Rbd: add create_from_image & create_from_func https://review.openstack.org/317694 | 09:37 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Introduce ImageCacheLocalDir https://review.openstack.org/279669 | 09:37 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Add check_backing_from_func & check_backing_from_image https://review.openstack.org/322285 | 09:37 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Flat: add create_from_image & create_from_func https://review.openstack.org/317701 | 09:37 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Ploop: add create_from_image & create_from_func https://review.openstack.org/317689 | 09:37 |
| *** moshele has joined #openstack-nova | 09:38 | |
| *** thorst has joined #openstack-nova | 09:39 | |
| *** roeyc has quit IRC | 09:39 | |
| *** sacharya has joined #openstack-nova | 09:40 | |
| *** nkrinner has quit IRC | 09:40 | |
| *** Daisy has quit IRC | 09:42 | |
| openstackgerrit | Daniel Berrange proposed openstack/nova: virt: set address space & CPU time limits when running qemu-img https://review.openstack.org/307663 | 09:42 |
| *** Daisy has joined #openstack-nova | 09:42 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Remove manual creation of console.log https://review.openstack.org/326335 | 09:43 |
| mdbooth | lyarwood_ danpb: ^^^ | 09:43 |
| mdbooth | That's a bit speculative, because I have no idea why we touch console.log in pre_live_migration | 09:44 |
| mdbooth | But CI should exercise that in the live migration job, so we'll find out. | 09:44 |
| * mdbooth suspects it is not required. | 09:44 | |
| *** sacharya has quit IRC | 09:45 | |
| *** mvk has quit IRC | 09:46 | |
| *** chandankumar has joined #openstack-nova | 09:46 | |
| johnthetubaguy | mdbooth: I think that used to be a permissions issue | 09:46 |
| johnthetubaguy | mdbooth: if we created, nova can access it, if the live-migrate stuff created it, nova failed to access it after live-migrate, maybe... my memory of that is cloudy | 09:46 |
| *** shaohe_feng has quit IRC | 09:47 | |
| mdbooth | johnthetubaguy: I *think* the permissions issue is covered. | 09:47 |
| *** mvk has joined #openstack-nova | 09:47 | |
| danpb | johnthetubaguy: no, that's not right - check the commit message | 09:47 |
| johnthetubaguy | I am miss-remembering them | 09:47 |
| danpb | johnthetubaguy: even if nova creates it, whatever permissions nova sets will be changed by libvirt | 09:47 |
| johnthetubaguy | danpb: ah, OK | 09:47 |
| johnthetubaguy | sounds kinda pointless then | 09:47 |
| *** Daisy has quit IRC | 09:47 | |
| *** thorst has quit IRC | 09:47 | |
| danpb | nova only needs care about permissions at the the time it reads the file | 09:47 |
| johnthetubaguy | danpb: OK, do we just chmod as required then? | 09:48 |
| danpb | yeah, that code already exists | 09:48 |
| *** nkrinner has joined #openstack-nova | 09:48 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/nova: Remove mox from nova/tests/unit/db/test_db_api.py https://review.openstack.org/323165 | 09:49 |
| johnthetubaguy | mdbooth: so my memory was playing tricks with me | 09:50 |
| *** shaohe_feng has joined #openstack-nova | 09:51 | |
| *** vishwanathj has joined #openstack-nova | 09:54 | |
| *** sree has quit IRC | 09:56 | |
| *** shaohe_feng has quit IRC | 09:57 | |
| *** permalac has joined #openstack-nova | 09:57 | |
| *** vishwanathj has quit IRC | 09:57 | |
| *** vishwanathj has joined #openstack-nova | 09:58 | |
| *** shaohe_feng has joined #openstack-nova | 09:58 | |
| sudipto | johnthetubaguy, Hi, since the spec is approved for mitaka, i was wondering if you could lift the -2s from these patches: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/memory-bw pls? | 09:59 |
| lyarwood_ | mdbooth: sorry wasn't watching irc, LGTM thanks! | 10:00 |
| johnthetubaguy | sudipto: the spec needs to be approved for newton | 10:00 |
| sudipto | johnthetubaguy, sorry my bad - i meant Newton :-| | 10:01 |
| johnthetubaguy | sudipto: no worries, I keep doing that, its approved anyways, I will remove my -2 | 10:01 |
| sudipto | johnthetubaguy, thanks a lot! | 10:01 |
| *** cdent has quit IRC | 10:03 | |
| johnthetubaguy | sudipto: not sure if all the patches are still active, but I removed the -2 from all of these: https://review.openstack.org/#/q/topic:bp/memory-bw | 10:03 |
| *** cheneydc has quit IRC | 10:03 | |
| *** vishwanathj has quit IRC | 10:04 | |
| sudipto | johnthetubaguy, thanks. those are the ones indeed. Thanks!! | 10:04 |
| sudipto | johnthetubaguy, infact, i have almost all code up now for review. | 10:04 |
| johnthetubaguy | good stuff :) | 10:04 |
| *** nkrinner has quit IRC | 10:05 | |
| *** shaohe_feng has quit IRC | 10:07 | |
| *** lpetrut has joined #openstack-nova | 10:07 | |
| *** shaohe_feng has joined #openstack-nova | 10:08 | |
| *** irenab has joined #openstack-nova | 10:08 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 10:10 |
| *** shaohe_feng has quit IRC | 10:17 | |
| *** shaohe_feng has joined #openstack-nova | 10:18 | |
| johnthetubaguy | bauzas: just checking, is this blueprint complete now? https://blueprints.launchpad.net/nova/+spec/check-destination-on-migrations-newton | 10:19 |
| bauzas | johnthetubaguy: almost complete, I just need to write 2 changes for novaclient for the new force field | 10:20 |
| johnthetubaguy | bauzas: ah, gotcha, just the nova-client bits now, cools | 10:21 |
| bauzas | yup :) | 10:21 |
| bauzas | I was waiting to have the API changes being merged to make sure I wouldn't modify the client ones in case I needed to bump the microversion :) | 10:21 |
| johnthetubaguy | bauzas: good call | 10:22 |
| johnthetubaguy | bauzas: its possible I could get someone from OSIC to take a look at those patches, if that helps free up time for the scheduler work? | 10:22 |
| bauzas | johnthetubaguy: well, tbh, I never yet provided a novaclient patch :p | 10:24 |
| bauzas | even if I'm able to review those :) | 10:24 |
| johnthetubaguy | bauzas: :) | 10:24 |
| *** sdague has joined #openstack-nova | 10:24 | |
| *** paul-carlton2 has quit IRC | 10:25 | |
| *** jed56 has quit IRC | 10:25 | |
| *** salv-orl_ has joined #openstack-nova | 10:25 | |
| johnthetubaguy | bauzas: either way, the offer is there, if thats useful | 10:25 |
| *** winston-d_ has quit IRC | 10:25 | |
| bauzas | johnthetubaguy: sure :) | 10:26 |
| bauzas | johnthetubaguy: tbh, OSIC guys can also help us for working on the RPC API now that we merged providing the RequestSpec | 10:26 |
| *** jlanoux_ has joined #openstack-nova | 10:27 | |
| bauzas | johnthetubaguy: because most of the calls are now providing both the legacy request_spec and filt_props dict, plus the RequestSpec object | 10:27 |
| bauzas | while I think we should just pass the object | 10:27 |
| johnthetubaguy | bauzas: oh, you mean the tidy up stuff | 10:27 |
| bauzas | also, something is missing, we're not passing yet the ReqSpec object to the compute nodes | 10:27 |
| bauzas | johnthetubaguy: yeah | 10:27 |
| *** shaohe_feng has quit IRC | 10:28 | |
| bauzas | so, it's a bit of cleaning up, but that's needed | 10:28 |
| *** salv-orlando has quit IRC | 10:28 | |
| bauzas | also, I had lots of TODO because the utils module are only using the legacy dicts | 10:28 |
| johnthetubaguy | bauzas: don't we have to wait a cycle and an RPC bump for the object clean up, oh, right the utils bits | 10:28 |
| bauzas | yeah, I think we can just tidy up at least the utils helper modules to use the ReqSpec without waiting one cycle | 10:29 |
| bauzas | at least sched.utils | 10:29 |
| bauzas | that's something I have in my todolist | 10:30 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Break up _live_migration_monitor https://review.openstack.org/325997 | 10:30 |
| *** jlanoux has quit IRC | 10:30 | |
| johnthetubaguy | bauzas: cool, I will ask about that, this is one example I guess: https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L489 | 10:30 |
| *** shaohe_feng has joined #openstack-nova | 10:30 | |
| *** roeyc has joined #openstack-nova | 10:30 | |
| *** cdent has joined #openstack-nova | 10:31 | |
| openstackgerrit | Allen Gao proposed openstack/nova: Use IPOpt and PortOpt in vnc options https://review.openstack.org/300716 | 10:34 |
| *** dave-mccowan has joined #openstack-nova | 10:35 | |
| kashyap | mdbooth: Nice, your console.log code-removal change. | 10:36 |
| mdbooth | kashyap: Thanks. Just hope it's correct :) Quite a lot of archeology required to remove a couple of lines of apparently obsolete code. | 10:37 |
| *** nkrinner has joined #openstack-nova | 10:38 | |
| *** shaohe_feng has quit IRC | 10:38 | |
| kashyap | mdbooth: Yeah, was reading the commits linked in your commit message. | 10:38 |
| *** hoangcx has quit IRC | 10:38 | |
| * mdbooth is especially interested in the result of gate-grenade-dsvm-multinode | 10:39 | |
| *** shaohe_feng has joined #openstack-nova | 10:39 | |
| mdbooth | I wouldn't be at all surprised to discover that there's a non-obvious interaction somewhere. | 10:39 |
| *** kaisers has quit IRC | 10:42 | |
| *** jlanoux has joined #openstack-nova | 10:42 | |
| *** kaisers has joined #openstack-nova | 10:44 | |
| *** jlanoux_ has quit IRC | 10:45 | |
| *** rk4n has joined #openstack-nova | 10:46 | |
| *** itamarl has joined #openstack-nova | 10:46 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Make the base options definitions consistent https://review.openstack.org/321471 | 10:46 |
| *** thorst has joined #openstack-nova | 10:47 | |
| *** shaohe_feng has quit IRC | 10:48 | |
| *** GnomeKing has joined #openstack-nova | 10:49 | |
| *** shaohe_feng has joined #openstack-nova | 10:49 | |
| *** rk4n_ has joined #openstack-nova | 10:49 | |
| *** smatzek has joined #openstack-nova | 10:52 | |
| *** rk4n has quit IRC | 10:53 | |
| *** thorst has quit IRC | 10:53 | |
| *** gszasz has joined #openstack-nova | 10:55 | |
| tdurakov | danpb: are you around? | 10:55 |
| *** chhavi has joined #openstack-nova | 10:57 | |
| *** shaohe_feng has quit IRC | 10:58 | |
| *** shaohe_feng has joined #openstack-nova | 10:59 | |
| *** amotoki has quit IRC | 11:00 | |
| *** ljxiash has quit IRC | 11:00 | |
| *** ljxiash has joined #openstack-nova | 11:01 | |
| *** sudipto has quit IRC | 11:01 | |
| *** GnomeKing is now known as BobBall | 11:02 | |
| *** vladikr has joined #openstack-nova | 11:03 | |
| *** roeyc1 has joined #openstack-nova | 11:04 | |
| openstackgerrit | John Garbutt proposed openstack/nova: api-ref: os-certificates.inc https://review.openstack.org/323365 | 11:04 |
| *** roeyc has quit IRC | 11:04 | |
| *** shihanzhang has quit IRC | 11:04 | |
| *** salv-orl_ has quit IRC | 11:04 | |
| *** salv-orlando has joined #openstack-nova | 11:05 | |
| *** ljxiash has quit IRC | 11:05 | |
| *** rtheis has joined #openstack-nova | 11:06 | |
| *** ducttape_ has joined #openstack-nova | 11:08 | |
| *** shaohe_feng has quit IRC | 11:09 | |
| *** shaohe_feng has joined #openstack-nova | 11:11 | |
| *** zioproto has joined #openstack-nova | 11:13 | |
| *** ducnc has quit IRC | 11:16 | |
| *** salv-orl_ has joined #openstack-nova | 11:18 | |
| *** shaohe_feng has quit IRC | 11:19 | |
| *** shaohe_feng has joined #openstack-nova | 11:19 | |
| *** amotoki has joined #openstack-nova | 11:20 | |
| *** jinquan has joined #openstack-nova | 11:22 | |
| *** ducttape_ has quit IRC | 11:22 | |
| *** salv-orlando has quit IRC | 11:23 | |
| *** iceyao has quit IRC | 11:23 | |
| *** iceyao has joined #openstack-nova | 11:24 | |
| *** markvoelker has joined #openstack-nova | 11:25 | |
| *** mgoddard has joined #openstack-nova | 11:28 | |
| openstackgerrit | Zhihai Song proposed openstack/nova: testfollowup https://review.openstack.org/326388 | 11:28 |
| *** shaohe_feng has quit IRC | 11:29 | |
| *** markvoelker has quit IRC | 11:29 | |
| *** zioproto has quit IRC | 11:29 | |
| *** shaohe_feng has joined #openstack-nova | 11:30 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Update libvirt to use the new imagebackend methods https://review.openstack.org/282580 | 11:30 |
| *** ducttape_ has joined #openstack-nova | 11:36 | |
| *** shaohe_feng has quit IRC | 11:39 | |
| *** shaohe_feng has joined #openstack-nova | 11:40 | |
| *** lucasagomes is now known as lucas-hungry | 11:40 | |
| *** sacharya has joined #openstack-nova | 11:41 | |
| *** sridharg has quit IRC | 11:45 | |
| *** sacharya has quit IRC | 11:46 | |
| *** thorst_ has joined #openstack-nova | 11:46 | |
| openstackgerrit | Chen Fan proposed openstack/nova: flavor: add check for pci request in extraspecs https://review.openstack.org/326400 | 11:49 |
| openstackgerrit | Chen Fan proposed openstack/nova: flavor_extra_specs: add test cases for checking pci requests https://review.openstack.org/326401 | 11:49 |
| *** shaohe_feng has quit IRC | 11:50 | |
| *** shaohe_feng has joined #openstack-nova | 11:50 | |
| *** salv-orl_ has quit IRC | 11:51 | |
| *** gcb has quit IRC | 11:52 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Fix nova-compute start failed when reserved_huge_pages has value. https://review.openstack.org/324379 | 11:52 |
| *** salv-orlando has joined #openstack-nova | 11:52 | |
| sdague | dims: for reference ^^^ | 11:52 |
| openstackgerrit | Sean Dague proposed openstack/nova: libvirt: handle reserved pages size https://review.openstack.org/292500 | 11:53 |
| sdague | johnthetubaguy: I added that unit test to https://review.openstack.org/324379 for reference | 11:54 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP resource-providers/placement api https://review.openstack.org/293104 | 11:54 |
| sdague | it was more complicated than I thought due to where we have to inject with oslo.config | 11:54 |
| *** ducttape_ has quit IRC | 11:57 | |
| *** cdent has quit IRC | 11:58 | |
| *** salv-orlando has quit IRC | 11:58 | |
| *** salv-orlando has joined #openstack-nova | 11:58 | |
| *** Jeffrey4l has quit IRC | 11:58 | |
| *** Jeffrey4l has joined #openstack-nova | 11:59 | |
| *** mpavone has quit IRC | 12:00 | |
| *** shaohe_feng has quit IRC | 12:00 | |
| *** amotoki has quit IRC | 12:00 | |
| *** shaohe_feng has joined #openstack-nova | 12:00 | |
| *** baoli has joined #openstack-nova | 12:01 | |
| jaypipes | moshele: on 310990, I'm stil confused why that doesn't need a version bump and a removal of the is_new field in obj_mak_compatible() for versions greater than 1.1... | 12:02 |
| jaypipes | dansmith: ^ | 12:02 |
| *** baoli_ has joined #openstack-nova | 12:02 | |
| *** zarkone has joined #openstack-nova | 12:03 | |
| *** gcb has joined #openstack-nova | 12:05 | |
| *** armax has joined #openstack-nova | 12:05 | |
| *** baoli has quit IRC | 12:05 | |
| *** markvoelker has joined #openstack-nova | 12:06 | |
| sdague | johnthetubaguy / danpb - https://review.openstack.org/#/c/324379/ would make reserved_huge_pages option actually work if set, which would be great :) | 12:06 |
| *** armax has quit IRC | 12:06 | |
| *** mpavone has joined #openstack-nova | 12:08 | |
| *** laszloh has quit IRC | 12:08 | |
| *** amotoki has joined #openstack-nova | 12:09 | |
| *** rodrigods has quit IRC | 12:09 | |
| *** caowei has quit IRC | 12:09 | |
| *** rodrigods has joined #openstack-nova | 12:09 | |
| *** rk4n_ has quit IRC | 12:10 | |
| zarkone | hello all! I'm running liberty and trying to use nova service-force-down.. I can't get it to work both from python-novaclient and nova CLI. In first case I've got 404, and in second -- keystone plugin error. What should I check to figure out what's the reason of this? | 12:10 |
| *** zerda2 has quit IRC | 12:10 | |
| *** shaohe_feng has quit IRC | 12:10 | |
| *** PaulMurray has joined #openstack-nova | 12:10 | |
| openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref, os-availability-zone.inc https://review.openstack.org/321847 | 12:10 |
| *** shaohe_feng has joined #openstack-nova | 12:11 | |
| *** diogogmt has quit IRC | 12:14 | |
| johnthetubaguy | sdague: yeah, that looks nice, thank you | 12:15 |
| *** diogogmt has joined #openstack-nova | 12:16 | |
| *** roeyc1 has quit IRC | 12:16 | |
| *** paul-carlton2 has joined #openstack-nova | 12:17 | |
| *** Hosam has joined #openstack-nova | 12:18 | |
| *** shaohe_feng has quit IRC | 12:20 | |
| *** ivase has joined #openstack-nova | 12:21 | |
| *** aysyd has joined #openstack-nova | 12:23 | |
| *** roeyc has joined #openstack-nova | 12:23 | |
| *** laszloh has joined #openstack-nova | 12:23 | |
| *** rk4n has joined #openstack-nova | 12:23 | |
| *** shaohe_feng has joined #openstack-nova | 12:23 | |
| *** cdent has joined #openstack-nova | 12:23 | |
| *** diogogmt has quit IRC | 12:24 | |
| *** smatzek has quit IRC | 12:26 | |
| *** ducttape_ has joined #openstack-nova | 12:27 | |
| *** apoorv has quit IRC | 12:28 | |
| thorst_ | mriedem_away: I'm told that thing adding comments (under Lian's name) is now turned off. Let me know if you see it again and I'll follow up with them. | 12:28 |
| *** shaohe_feng has quit IRC | 12:31 | |
| *** garyk1 has joined #openstack-nova | 12:31 | |
| *** thorst_ is now known as thorst | 12:31 | |
| garyk1 | sdague: can you pleas elook at https://review.openstack.org/#/c/318145/ - this has thumbs up from a number of people | 12:31 |
| *** shaohe_feng has joined #openstack-nova | 12:31 | |
| *** kylek3h has joined #openstack-nova | 12:32 | |
| *** pwnall has joined #openstack-nova | 12:32 | |
| *** flip214_ has joined #openstack-nova | 12:34 | |
| sahid | sdague, johnthetubaguy you might want re+W this one https://review.openstack.org/#/c/292500/ | 12:34 |
| *** flip214 has quit IRC | 12:35 | |
| *** flip214_ is now known as flip214 | 12:35 | |
| *** flip214 has quit IRC | 12:35 | |
| *** flip214 has joined #openstack-nova | 12:35 | |
| *** Hosam has quit IRC | 12:36 | |
| openstackgerrit | jichenjc proposed openstack/nova: Avoid unnessary _get_power_state call https://review.openstack.org/326431 | 12:37 |
| *** diogogmt has joined #openstack-nova | 12:37 | |
| *** lucas-hungry is now known as lucasagomes | 12:40 | |
| *** EinstCrazy has quit IRC | 12:41 | |
| *** shaohe_feng has quit IRC | 12:41 | |
| *** shaohe_feng has joined #openstack-nova | 12:41 | |
| *** EinstCrazy has joined #openstack-nova | 12:41 | |
| pkoniszewski | tdurakov: hey, are u there? | 12:41 |
| tdurakov | pkoniszewski: hi | 12:41 |
| *** jwcroppe has joined #openstack-nova | 12:42 | |
| pkoniszewski | tdurakov: im looking through your changes regarding this migrateToURI3 bit | 12:42 |
| pkoniszewski | tdurakov: isn't this https://review.openstack.org/#/c/310707 fixing the issue? | 12:42 |
| *** eharney has quit IRC | 12:42 | |
| *** salv-orlando has quit IRC | 12:42 | |
| *** salv-orlando has joined #openstack-nova | 12:43 | |
| tdurakov | pkoniszewski: haven't seen it yet | 12:43 |
| *** ducttape_ has quit IRC | 12:44 | |
| tdurakov | pkoniszewski: have you read danpb comments on my change? | 12:44 |
| *** iceyao has quit IRC | 12:45 | |
| pkoniszewski | tdurakov: yeah, but i don't understand how we can validate configuration during init_host without checking all VMs in the environment | 12:46 |
| *** EinstCrazy has quit IRC | 12:46 | |
| tdurakov | pkoniszewski: got same question | 12:46 |
| *** smatzek has joined #openstack-nova | 12:46 | |
| tdurakov | pkoniszewski: let me review your patch carefully | 12:47 |
| *** kylek3h has quit IRC | 12:47 | |
| tdurakov | p.s. I think it worth to mention this bug/fix in ml | 12:47 |
| *** ljxiash has joined #openstack-nova | 12:47 | |
| pkoniszewski | tdurakov: thanks, it isn't perfect as it leaves doors open for potential issues, but i haven't found better way yet | 12:47 |
| pkoniszewski | tdurakov: okay, i will do | 12:47 |
| *** kfarr has joined #openstack-nova | 12:48 | |
| *** ljxiash_ has joined #openstack-nova | 12:49 | |
| openstackgerrit | jichenjc proposed openstack/nova: api-ref: verify os-server-groups.inc https://review.openstack.org/325269 | 12:49 |
| *** smatzek_ has joined #openstack-nova | 12:50 | |
| tdurakov | pkoniszewski: marked this as a duplicate https://bugs.launchpad.net/nova/+bug/1589591 | 12:51 |
| openstack | Launchpad bug 1576093 in OpenStack Compute (nova) "duplicate for #1589591 block migration fail with libvirt 1.3.1" [High,In progress] - Assigned to Eli Qiao (taget-9) | 12:51 |
| *** links has quit IRC | 12:51 | |
| *** shaohe_feng has quit IRC | 12:51 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 12:51 |
| *** ljxiash has quit IRC | 12:52 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: Add InventoryList.find() method https://review.openstack.org/326439 | 12:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova: Add set_inventory() method on ResourceProvider https://review.openstack.org/326440 | 12:52 |
| jaypipes | cdent: et voila. ^ | 12:52 |
| *** shaohe_feng has joined #openstack-nova | 12:52 | |
| *** links has joined #openstack-nova | 12:52 | |
| cdent | jaypipes: rockin | 12:52 |
| moshele | jaypipes: dansmith had a good when I talked to him about this in summit, but I forgot it :( | 12:52 |
| jaypipes | cdent: set_inventory() has one failing test... still trying to figure it out. gonna need rpodolyaka orzzzeek's help I think | 12:52 |
| rpodolyaka | jaypipes: looking | 12:53 |
| jaypipes | moshele: well, dansmith is OK on that patch which makes me wonder what I'm missing :) | 12:53 |
| cdent | jaypipes: these changes will mostly obsolete the "correct inventory" stuff, or something else? | 12:53 |
| mdbooth | johnthetubaguy: You ran check experimental against my patch earlier, which just finished. It's kinda inconclusive, which is disappointing. | 12:53 |
| *** ljxiash_ has quit IRC | 12:53 | |
| *** jwcroppe has quit IRC | 12:54 | |
| mdbooth | What would you like to do about that? One of the failures is non-deterministic, and probably won't happen again. | 12:54 |
| jaypipes | cdent: yes, they will mostly obselete that patch. | 12:54 |
| mdbooth | The other looks like it might be deterministic, but not related to this change. | 12:54 |
| mdbooth | In which case it would fail again. Again, inconclusive. | 12:54 |
| johnthetubaguy | mdbooth: I know that job is having issues right now, I thought mriedem_away had some patches up for that though | 12:54 |
| *** smatzek has quit IRC | 12:54 | |
| mdbooth | johnthetubaguy: Cool, if mriedem_away could positively identify the failure in gate-tempest-dsvm-multinode-live-migration as a known issue, that would be a good thing. | 12:55 |
| *** zhs_s has joined #openstack-nova | 12:56 | |
| *** chandankumar has quit IRC | 12:56 | |
| *** Jeffrey4l has quit IRC | 12:56 | |
| *** gmann_ has joined #openstack-nova | 12:57 | |
| moshele | jaypipes: I think it related to old instance in old compute will use the is_new, but I think that because it is broken for several cycles it doesn't matter. Anyway I didn't mind to remove it on version bump | 12:57 |
| *** kylek3h has joined #openstack-nova | 12:57 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP] Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | 12:58 |
| *** gjayavelu has joined #openstack-nova | 12:58 | |
| *** smatzek_ has quit IRC | 12:58 | |
| *** zhs has quit IRC | 12:59 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Break up _live_migration_monitor https://review.openstack.org/325997 | 12:59 |
| moshele | sean-k-mooney, sfinucan can you join the sriov meeting ? | 13:01 |
| *** edmondsw has joined #openstack-nova | 13:01 | |
| *** mriedem_away is now known as mriedem | 13:01 | |
| *** shaohe_feng has quit IRC | 13:01 | |
| mriedem | mdbooth: johnthetubaguy: yeah known failures in the live migration job, the details are in the live migration meeting agenda | 13:02 |
| mriedem | meeting is in 58 minutes | 13:02 |
| *** ctrath has joined #openstack-nova | 13:02 | |
| *** shaohe_feng has joined #openstack-nova | 13:02 | |
| mdbooth | mriedem: See you there :) | 13:02 |
| *** xyang1 has joined #openstack-nova | 13:02 | |
| *** ctrath has quit IRC | 13:02 | |
| openstackgerrit | jichenjc proposed openstack/nova-specs: Add host-command-with-POST https://review.openstack.org/303388 | 13:02 |
| mriedem | there are a few issues actually, the first is the tunneling one, tdurakov has a patch for that here https://review.openstack.org/#/c/326258/ | 13:03 |
| mriedem | the other issue is the job is failing to setup NFS | 13:03 |
| tdurakov | mriedem: hey, actually folks have been working on it, see https://bugs.launchpad.net/nova/+bug/1576093 | 13:03 |
| openstack | Launchpad bug 1576093 in OpenStack Compute (nova) "block migration fail with libvirt since version 1.2.17" [High,In progress] - Assigned to Eli Qiao (taget-9) | 13:03 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: compute: Skip volume driver detach calls during evacuation https://review.openstack.org/318266 | 13:03 |
| *** kfarr_ has joined #openstack-nova | 13:04 | |
| cdent | jaypipes: something I'm not quite grokking on this _set_inventory stuff: If the input is the inv_list we want the resource provider to have when this transaction is done, and any failure should rollback everything, why is it important to _delete, _add, _update, why can't we just clobber what's there, and put the new stuff in? Is it because want to preserve timestamp metadata? | 13:04 |
| *** chandankumar has joined #openstack-nova | 13:05 | |
| sdague | BobBall: we need functional review on - https://review.openstack.org/#/c/324536 from a xen perspective (glance v2) | 13:05 |
| sdague | that's kind of the last patch in the chain before this is all sorted | 13:06 |
| *** rlrossit has joined #openstack-nova | 13:07 | |
| *** kfarr has quit IRC | 13:07 | |
| sfinucan | moshele: We both have conflicting meetings at the moment :( sean-k-mooney might be able to join in a bit, but I'm not sure | 13:08 |
| *** kfarr_ has quit IRC | 13:09 | |
| *** gmann_ has quit IRC | 13:09 | |
| *** sfinucan has quit IRC | 13:09 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: define NUMA memory bw monitor https://review.openstack.org/270344 | 13:09 |
| *** sudipto has joined #openstack-nova | 13:10 | |
| *** r-daneel has joined #openstack-nova | 13:11 | |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: verify keypairs https://review.openstack.org/325308 | 13:11 |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: add version 2.3 parameters to servers https://review.openstack.org/325502 | 13:11 |
| *** shaohe_feng has quit IRC | 13:12 | |
| mriedem | tdurakov: i think you can abandon https://review.openstack.org/#/c/326111/ | 13:12 |
| mriedem | tdurakov: since there are 3 patches up for this now we should figure out which one to work on :) | 13:12 |
| tdurakov | mriedem: yup, reviewing this one https://review.openstack.org/#/c/310707 | 13:12 |
| mriedem | yeah i just did | 13:12 |
| *** garyk1 has quit IRC | 13:12 | |
| *** amotoki has quit IRC | 13:13 | |
| mriedem | sdague: should i add the api-ref to my 2.30 microversion change for get-me-a-network now, or is it bad form to jump around on those in api-ref? | 13:13 |
| sdague | mriedem: yeh, I would add them in | 13:13 |
| sdague | I was starting to backfill some of them | 13:14 |
| mriedem | ok | 13:14 |
| *** mrkz has joined #openstack-nova | 13:14 | |
| *** shaohe_feng has joined #openstack-nova | 13:15 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: add memory bandwidth monitor base class https://review.openstack.org/270079 | 13:15 |
| *** jaosorior has quit IRC | 13:15 | |
| *** amotoki has joined #openstack-nova | 13:15 | |
| *** jaosorior has joined #openstack-nova | 13:16 | |
| johnthetubaguy | sdague: got a quick question on that update: https://review.openstack.org/#/c/325308/5 | 13:16 |
| mriedem | sdague: ok. i'm not totally sure how i'm going to represent that change, might just be something in the server.networks.uuid field description | 13:16 |
| sdague | johnthetubaguy: well, it's only displayed for admin users | 13:17 |
| sdague | so... required: false seems correct | 13:17 |
| *** akshai has joined #openstack-nova | 13:17 | |
| johnthetubaguy | oh, I miss read the code, I thought it was displayed for everyone | 13:17 |
| sdague | I was pretty sure it was only admin users, let me go look again | 13:18 |
| mriedem | sahid: thanks for the review on the get-me-a-network series btw | 13:18 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Base enablement for memory bandwidth monitor https://review.openstack.org/266773 | 13:18 |
| jroll | jaypipes: should I be waiting for you to update the g-r-c spec before updating my ironic-multiple-compute spec, or just go ahead? | 13:19 |
| *** jichen has quit IRC | 13:20 | |
| sdague | johnthetubaguy: no, I think you are right | 13:20 |
| sdague | I read a comment wrong | 13:20 |
| sahid | mriedem: no problem, you did a really good code well documented, still have to review last patches of the serie | 13:20 |
| johnthetubaguy | sdague: no worries | 13:20 |
| *** mdrabe has joined #openstack-nova | 13:21 | |
| rpodolyaka | jaypipes: ah, cdent beat me to this :P left you a couple of comments | 13:21 |
| *** shaohe_feng has quit IRC | 13:22 | |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: verify keypairs https://review.openstack.org/325308 | 13:22 |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: add version 2.3 parameters to servers https://review.openstack.org/325502 | 13:22 |
| sdague | johnthetubaguy: there we go | 13:22 |
| *** shaohe_feng has joined #openstack-nova | 13:22 | |
| johnthetubaguy | sdague: sweet, that looks correct now | 13:23 |
| johnthetubaguy | sdague: only had a tiny nit in the follow up one, otherwise looks good | 13:23 |
| sdague | yeh, good catch | 13:23 |
| tdurakov | pkoniszewski, mriedem: lgtm, let's wait for experimental job results | 13:24 |
| mriedem | dansmith: https://review.openstack.org/#/c/326111/ would be a good upgrade regression fix to get into mitaka before we cut a release this week | 13:24 |
| *** r-daneel has quit IRC | 13:24 | |
| *** doug-fish has joined #openstack-nova | 13:25 | |
| sdague | mfedosin: you going to do the split up of the xenserver patch today? because, I think that's basically the last bit then we can land it + hyperv (already approved). | 13:25 |
| tdurakov | mriedem: could we merge this too https://review.openstack.org/#/q/34b8336a41e2896f0639c402f64a05247f475fea,n,z | 13:25 |
| mriedem | tdurakov: i'll take a look | 13:26 |
| tdurakov | mriedem: cool, thank you | 13:26 |
| pkoniszewski | tdurakov: mriedem: i can change this condition to elif or put version check on top if you want to be more comfortable about it | 13:27 |
| *** sdake has joined #openstack-nova | 13:27 | |
| tdurakov | pkoniszewski: I'd leave things as is, just add debug | 13:28 |
| *** ctrath has joined #openstack-nova | 13:28 | |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: add version 2.3 parameters to servers https://review.openstack.org/325502 | 13:28 |
| pkoniszewski | tdurakov: okay, but let's wait for experimental job | 13:28 |
| tdurakov | pkoniszewski: sure | 13:28 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova-specs: VMware: add SR-IOV support https://review.openstack.org/317126 | 13:29 |
| mriedem | pkoniszewski: no that's fine, i replied | 13:29 |
| *** maurosr has quit IRC | 13:30 | |
| dansmith | mriedem: that's DNM and abandoned? | 13:31 |
| mriedem | crap | 13:31 |
| mriedem | dansmith: https://review.openstack.org/#/q/I21107ea0a148b66bee81e57cdce08e3006a60aee,n,z | 13:31 |
| openstackgerrit | John Garbutt proposed openstack/nova: api-ref: add version 2.3 parameters to servers https://review.openstack.org/325502 | 13:31 |
| dansmith | mriedem: oh yeah, agree, I'll get it | 13:31 |
| *** shaohe_feng has quit IRC | 13:32 | |
| *** mmandel has joined #openstack-nova | 13:32 | |
| *** shaohe_feng has joined #openstack-nova | 13:33 | |
| *** eharney has joined #openstack-nova | 13:33 | |
| *** sdake_ has joined #openstack-nova | 13:34 | |
| *** cleong has joined #openstack-nova | 13:34 | |
| *** mnestratov has joined #openstack-nova | 13:35 | |
| *** gjayavel_ has joined #openstack-nova | 13:35 | |
| *** sdake has quit IRC | 13:36 | |
| *** r-daneel has joined #openstack-nova | 13:36 | |
| *** mpavone has quit IRC | 13:37 | |
| *** gjayavelu has quit IRC | 13:37 | |
| mriedem | dansmith: and this one https://review.openstack.org/#/c/312812/ | 13:37 |
| rlrossit | johnthetubaguy: FYI this got updated to add the unit test, so should be a pretty easy re +2 for you https://review.openstack.org/#/c/310700/16 | 13:37 |
| *** READ10 has joined #openstack-nova | 13:38 | |
| *** READ10 has quit IRC | 13:38 | |
| *** READ10 has joined #openstack-nova | 13:38 | |
| *** maurosr has joined #openstack-nova | 13:40 | |
| *** ratailor has quit IRC | 13:40 | |
| *** iceyao has joined #openstack-nova | 13:41 | |
| *** shaohe_feng has quit IRC | 13:42 | |
| *** shaohe_feng has joined #openstack-nova | 13:43 | |
| *** ducttape_ has joined #openstack-nova | 13:44 | |
| *** mgoddard_ has joined #openstack-nova | 13:45 | |
| *** chandankumar has quit IRC | 13:45 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add set_inventory() method on ResourceProvider https://review.openstack.org/326440 | 13:45 |
| jaypipes | jroll: gimme a couple more days, sorry man :( | 13:46 |
| jroll | jaypipes: no worries, just getting anxious :) | 13:46 |
| jaypipes | jroll: understood, trust me. | 13:47 |
| jroll | heh | 13:47 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add set_inventory() method on ResourceProvider https://review.openstack.org/326440 | 13:48 |
| openstackgerrit | John Garbutt proposed openstack/nova: api-ref: parameter verification for os-hypervisors https://review.openstack.org/316681 | 13:48 |
| cdent | jaypipes: I've fixed the minor problems on that ^^ but there's one main issue, which is do we need to pass in a generation? | 13:48 |
| *** mgoddard has quit IRC | 13:48 | |
| *** amotoki has quit IRC | 13:48 | |
| cdent | (see my -1 comment on patchset 1) | 13:48 |
| alex_xu | sdague: johnthetubaguy, reservation_id isn't an internal id, it returned by multiple create, and list support to filter by reservation_id. If you guys agree, i will remove the 'internal' quickly, and +w the patch | 13:48 |
| dansmith | cdent: were you going to un-nullifiy the name property one? I'm looking at it right now if you want me to do it | 13:49 |
| mriedem | more xenapi py3 mox unit test failures http://logs.openstack.org/69/321169/3/check/gate-nova-python34-db/c1c9add/testr_results.html.gz | 13:49 |
| *** mpavone has joined #openstack-nova | 13:49 | |
| cdent | dansmith: it was on my list, but I've not had a chance to get to it, but besides un-nulling name it should also expose generation | 13:49 |
| *** ducttape_ has quit IRC | 13:50 | |
| cdent | dansmith: so maybe that should be done there. I can go ahead unless you're super eager | 13:50 |
| dansmith | cdent: and it's called generation in the db as well right? | 13:50 |
| cdent | yes | 13:50 |
| jaypipes | cdent: yes, we need to pass in a generation. it's in the resource provider... | 13:50 |
| dansmith | cdent: I'm not super eager, I'm just willing | 13:50 |
| cdent | please proceed dansmith, if you could that would be just dandy | 13:51 |
| sdague | alex_xu: sure, I was about to dive into the multiple create doc | 13:51 |
| cdent | jaypipes: how is that generation the generation that the http request provided? | 13:51 |
| sdague | alex_xu: but we can fix that patch now | 13:51 |
| alex_xu | sdague: yea, i will edit it on the gerrit directly | 13:52 |
| cdent | jaypipes: when we were talking about this yesterday we're going round in circles (during the etags discussion) about how we don't want to get the resource provider object twice | 13:52 |
| sdague | alex_xu: sounds great | 13:52 |
| jaypipes | cdent, rpodolyaka: in answer to your question about the "why not just blow everything away"... it's more efficient to not do that actually and introduces way less churn in the database. Plus, I want to eventually be able to just add or delete a single inventory record and the _delete, _add, _update subroutines make that easier long-term. | 13:52 |
| *** paul-carlton2a has joined #openstack-nova | 13:52 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: xen: skip two more racey mox py34 test classes https://review.openstack.org/326476 | 13:52 |
| mriedem | sdague: johnthetubaguy: ^ should get that in quickly to avoid more race failures | 13:52 |
| *** shaohe_feng has quit IRC | 13:53 | |
| *** ametts has joined #openstack-nova | 13:53 | |
| jaypipes | cdent: I'm not sure I understand why you think we're getting the resource provider twice? | 13:53 |
| * johnthetubaguy nods | 13:53 | |
| *** sree has joined #openstack-nova | 13:53 | |
| sdague | mriedem: +A | 13:53 |
| dansmith | cdent: so, hmm, there's no save() on resource_provider, so how are we going to update things like generation? just at the super low sql level? | 13:54 |
| openstackgerrit | Alex Xu proposed openstack/nova: api-ref: add version 2.3 parameters to servers https://review.openstack.org/325502 | 13:54 |
| jaypipes | dansmith: yes. | 13:54 |
| jaypipes | dansmith: that is intended. | 13:54 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Fix revert on migration with SR-IOV https://review.openstack.org/326174 | 13:54 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Update binding:profile for SR-IOV ports https://review.openstack.org/242573 | 13:54 |
| dansmith | jaypipes: so should we initialize generation to zero on create? | 13:55 |
| *** sridharg has joined #openstack-nova | 13:55 | |
| jaypipes | dansmith: yeah, or 1. or some integer value, just not None :) | 13:55 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: api-ref: parameter verification for os-hypervisors https://review.openstack.org/316681 | 13:55 |
| *** burt has joined #openstack-nova | 13:55 | |
| dansmith | jaypipes: hmm, generation is default=0, but it looks like it is nullable in the db... that's wrong right? | 13:55 |
| *** paul-carlton2a has quit IRC | 13:56 | |
| *** shaohe_feng has joined #openstack-nova | 13:56 | |
| *** paul-carlton2 has quit IRC | 13:56 | |
| jaypipes | dansmith: yeah, should be non-nullable. | 13:56 |
| dansmith | Column('generation', Integer, default=0), | 13:56 |
| dansmith | that makes it nullable AFAIK | 13:56 |
| *** paul-carlton2 has joined #openstack-nova | 13:57 | |
| openstackgerrit | John Garbutt proposed openstack/nova: api-ref: parameter verification for os-hypervisors https://review.openstack.org/316681 | 13:57 |
| andreykurilin1 | mriedem sdague: hi! Could you look at small patch to novaclient? It makes usage of microversions for resources in more clear way - https://review.openstack.org/#/c/308848/ | 13:57 |
| cdent | jaypipes: when I PUT some new inventory I need to provide a resource_provider_generation. Then I load the resource provider object so I can call set_inventory. Before I enter set_inventory I must check resource_provider_generation == rp.generation or 409 so that I know I'm even allowed to call set_inventory. Then I call set_inventory and it does it's stuff and at the end it might exit saying "no", try again. | 13:57 |
| jaypipes | dansmith: it was likely added as nullable because generation field was added after the initial db schema was done. | 13:58 |
| cdent | If I don't need to do that with resource_provider_generation, why have it? | 13:58 |
| dansmith | jaypipes: yeah, but then we moved it to the api db, so we had a chance to fix it :/ | 13:58 |
| dansmith | I just didn't realize | 13:58 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Catch PciRequestAliasNotDefined exception https://review.openstack.org/317926 | 13:58 |
| cdent | jaypipes: when we were talking etags there was some hope would could enclose everything I said above in one transaction | 13:59 |
| cdent | rather than having some action before the transaction in set_inventory | 13:59 |
| jaypipes | cdent: no, *you* do not need to check if resource_provider_generation is < rp.generation. The server needs to do that. | 13:59 |
| *** smatzek_ has joined #openstack-nova | 13:59 | |
| *** claudiub has joined #openstack-nova | 13:59 | |
| cdent | I'm talking about the server | 13:59 |
| PaulMurray | Live migration meeting on #openstack-meeting-3 in a minute: PaulMurray paul-carlton2 eliqiao shaohe_feng pkoniszewski jlanoux yuntongjin mdbooth kashyap kmartin claudiub tobasco tdurakov andreas_s abhishekk diana_clarke (keeping people happy :) | 14:00 |
| cdent | The "I" is the "api server" | 14:00 |
| jaypipes | cdent: in other words, the request payload is sent, the first thing that should be done is ResourceProvider.get_by_uuid(...), and then call set_inventory() on that resource provider object. | 14:00 |
| jaypipes | cdent: so I'm failing to see where we "read resource provider again" | 14:00 |
| *** paul-carlton1 has joined #openstack-nova | 14:01 | |
| *** paul-carlton1 has quit IRC | 14:01 | |
| cdent | I misspoke, I meant 'attend to the generation again'. The underlying question still stands: When does resource_provider_generation (in the request body) get used? | 14:01 |
| *** yamahata has joined #openstack-nova | 14:02 | |
| jaypipes | cdent: ah, yes. for that we will need a ResourceProvider.get_by_uuid_and_generation(). | 14:03 |
| *** shaohe_feng has quit IRC | 14:03 | |
| cdent | isn't that racy? | 14:03 |
| *** gjayavel_ has quit IRC | 14:03 | |
| cdent | wouldn't: ResourceProvider.get_by_uuid_and_generation_and_update_inventory(...) be less so? | 14:03 |
| *** shaohe_feng has joined #openstack-nova | 14:04 | |
| jaypipes | cdent: no. that would raise NotFound back to the caller to indicate a concurrent writer has updated the inventory of the resource provider (which is exactly what the raising of ConcurrentInventoryUpdate signals, just later on in set_inventory(). | 14:04 |
| sdague | alex_xu: you want to just fix https://review.openstack.org/#/c/325308/6/api-ref/source/os-keypairs.inc with the issues as well? | 14:04 |
| sdague | you modified the stack on top, so my local tree is no longer in sync | 14:04 |
| alex_xu | sdague: yea, i'm editing | 14:04 |
| sdague | alex_xu: ok, cool, thank you | 14:04 |
| *** ecelik has quit IRC | 14:04 | |
| alex_xu | sdague: np | 14:04 |
| cdent | jaypipes: Hmmm. I don't feel like I'm making my point as well as I could, hold on a sec, I need to roll it around in my brain for a rephrase. | 14:05 |
| openstackgerrit | Alex Xu proposed openstack/nova: api-ref: verify keypairs https://review.openstack.org/325308 | 14:05 |
| *** chandankumar has joined #openstack-nova | 14:06 | |
| jaypipes | dansmith, cdent: while we work out the placement API and client stuff, it's probably a better idea to have the nova-compute resource tracker keep a ResourceProvider and InventoryList object in memory and call set_inventory() on that in-memory object. That would, yes, call the nova-conductor which would interact with the API database directly, but we can do that for some time period until the API | 14:06 |
| jaypipes | service is done and replace the in-memory object with calls to the placement client. you follow? | 14:06 |
| bauzas | jaypipes: cdent: well, it's almost impossible to follow your convo guys | 14:07 |
| bauzas | unless I'm looking at the channel | 14:08 |
| cdent | (I had a mirror of that thought with regard how info about aggregates are going to be managed in placement api.) | 14:08 |
| *** tbachman has joined #openstack-nova | 14:08 | |
| bauzas | cdent: I'm -1 with having the aggregates resources in the placement API (yet) | 14:09 |
| *** MikeG451 has joined #openstack-nova | 14:09 | |
| *** takedakn has joined #openstack-nova | 14:09 | |
| *** jamesdenton has joined #openstack-nova | 14:11 | |
| *** jed56 has joined #openstack-nova | 14:11 | |
| *** shaohe_feng has quit IRC | 14:13 | |
| jaypipes | bauzas: why -1 on agg in placement API? how else will the scheduler understand the relationship between sets of resource providers? | 14:13 |
| *** jwcroppe has joined #openstack-nova | 14:13 | |
| bauzas | jaypipes: because it's something yet in Nova | 14:13 |
| *** shaohe_feng has joined #openstack-nova | 14:14 | |
| bauzas | and aggregates can be used for more than just resource providers | 14:14 |
| jaypipes | bauzas: sorry, I'm not following you... what do you mean by "yet in Nova"? | 14:14 |
| jaypipes | bauzas: no, aggregates are used pretty much *only* for associating groups of resource providers. | 14:14 |
| bauzas | yet in Nova == already there by like 3 years | 14:14 |
| jaypipes | yes... | 14:14 |
| *** takedakn has quit IRC | 14:15 | |
| *** zhs_ss has joined #openstack-nova | 14:15 | |
| jaypipes | bauzas: I'm still not following you :) | 14:15 |
| *** rlrossit has quit IRC | 14:16 | |
| bauzas | jaypipes: people use aggregates for more than just splitting resource providers | 14:16 |
| bauzas | they want either to group things that can't possibly yet resource providers | 14:17 |
| bauzas | or they want to tag a list of hosts | 14:17 |
| bauzas | with the same key | 14:17 |
| *** tonytan4ever has joined #openstack-nova | 14:17 | |
| bauzas | sure, that's something being in use by the scheduler, and I think it should be mid-term in the placement API | 14:17 |
| bauzas | but | 14:18 |
| *** zhs_s has quit IRC | 14:18 | |
| bauzas | I'm pretty sad to see that *now* | 14:18 |
| *** jinquan has quit IRC | 14:18 | |
| jaypipes | bauzas: the *only* thing you can group with aggregates is a set of resource providers. | 14:18 |
| cdent | bauzas: it isn't now. it is just something I asked about so that we can plan effectively. it is something we're going to need to manage, at some point. | 14:18 |
| jaypipes | bauzas: compute nodes are resource providers. | 14:19 |
| bauzas | jaypipes: sure, I see your logic | 14:19 |
| bauzas | jaypipes: what I'm saying is that I feel we should keep the aggregates API in the os-compute endpoint until we clearly state that the scheduler is split out | 14:20 |
| cdent | jaypipes: I think I resolved my transaction issues above (by trying to write it down), basically my plan just moves a problem that will always exist, and doesn't help. But the get_by_uuid_and_generation will need to by tidy to be sure it can both 404 and 409, otherwise it's not being friendly across the api. | 14:20 |
| bauzas | jaypipes: because operators know hosts, not resource providers | 14:20 |
| *** irenab has quit IRC | 14:21 | |
| jaypipes | cdent: yes. the placement API server would catch NotFound from get_by_uuid_and_generation() and return a 409 Conflict. | 14:22 |
| jaypipes | cdent: there isn't any corresponding HTTP API call for "get me the resource provider with this UUID and generation". | 14:22 |
| cdent | jaypipes: _no_. It has to be able to return both 404 and 409 | 14:22 |
| cdent | a uri has to be able to 404 | 14:23 |
| cdent | that's the point I'm trying to make | 14:23 |
| jaypipes | cdent: if there is no resource provider with that UUID you mean? | 14:23 |
| cdent | exactly | 14:23 |
| *** rama_y has joined #openstack-nova | 14:23 | |
| *** shaohe_feng has quit IRC | 14:23 | |
| jaypipes | cdent: so just do a get_by_uuid() and compare the generation manually. yes? | 14:23 |
| cdent | yes | 14:24 |
| jaypipes | cool with me :) | 14:24 |
| cdent | that's all I mean by "tidy" | 14:24 |
| *** jinquan has joined #openstack-nova | 14:24 | |
| *** shaohe_feng has joined #openstack-nova | 14:26 | |
| jaypipes | bauzas: the placement API needs to know aggregate information. There will be an aggregate component to the placement API but only the aggregate UUIDs are used in the placement API at this time. | 14:26 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: tests: make XMLMatches work with Python3 https://review.openstack.org/300509 | 14:26 |
| bauzas | jaypipes: that looks fine by me then | 14:26 |
| jaypipes | bauzas: in other words, the Compute API can keep aggregate information for the time being. all the placement API needs is the relationship of an agg UUID to a resource provider UUID, in the context of the resource provider REST resource, not the aggregate REST resource. | 14:27 |
| *** mriedem is now known as mriedem_afk | 14:27 | |
| bauzas | jaypipes: yeah, got that and I think it's a reasonable trade-off :) | 14:27 |
| *** irenab has joined #openstack-nova | 14:27 | |
| jaypipes | bauzas: for this week at least :P | 14:27 |
| bauzas | jaypipes: I just don't want to duplicate the aggregate/host/metadata CRUD operations in the placement API (yet) | 14:28 |
| bauzas | but setting/getting the relationship between RPs and aggs is something that needs to be set in the placement API, I agree | 14:28 |
| *** woodster_ has joined #openstack-nova | 14:30 | |
| bauzas | I think I even +1d that in the REST API proposal you made :) | 14:30 |
| *** ctrath has quit IRC | 14:30 | |
| *** jinquan has left #openstack-nova | 14:30 | |
| *** jamesdenton has quit IRC | 14:30 | |
| *** sree has quit IRC | 14:30 | |
| bauzas | jaypipes: sorry if my confusion between a single service running multiple endpoints and multiple services each running a single endpoint made you think I was not having the same consensus by each day | 14:31 |
| *** zz_dimtruck is now known as dimtruck | 14:31 | |
| bauzas | s/consensus/thoughts (better) | 14:31 |
| *** jlanoux has quit IRC | 14:32 | |
| *** ctrath has joined #openstack-nova | 14:32 | |
| openstackgerrit | Tom Patzig proposed openstack/python-novaclient: Add option to disable local disks in flavors https://review.openstack.org/300541 | 14:33 |
| *** mgoddard_ has quit IRC | 14:33 | |
| openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors https://review.openstack.org/280301 | 14:33 |
| *** tonytan4ever has quit IRC | 14:33 | |
| *** shaohe_feng has quit IRC | 14:34 | |
| *** nkrinner has quit IRC | 14:34 | |
| *** shaohe_feng has joined #openstack-nova | 14:34 | |
| *** nkrinner has joined #openstack-nova | 14:35 | |
| *** sheel has quit IRC | 14:35 | |
| *** nkrinner has quit IRC | 14:35 | |
| *** sdake_ has quit IRC | 14:36 | |
| *** ljxiash has joined #openstack-nova | 14:37 | |
| *** tinbar has joined #openstack-nova | 14:39 | |
| *** mfisch has joined #openstack-nova | 14:40 | |
| *** mfisch has quit IRC | 14:40 | |
| *** mfisch has joined #openstack-nova | 14:40 | |
| *** zenoway has quit IRC | 14:40 | |
| *** roeyc has quit IRC | 14:42 | |
| jaypipes | bauzas: no worries man. | 14:42 |
| *** mgoddard has joined #openstack-nova | 14:43 | |
| *** shaohe_feng has quit IRC | 14:44 | |
| *** shaohe_feng has joined #openstack-nova | 14:45 | |
| *** laszloh has quit IRC | 14:47 | |
| JayF | mriedem_afk: If you get a chance, I'd love a review on these two backports for major bugs in the ironic driver: https://review.openstack.org/#/c/323477/ https://review.openstack.org/#/c/323196/ | 14:49 |
| openstackgerrit | Gage Hugo proposed openstack/nova: WIP - Remove DictCompact from Block_Device https://review.openstack.org/316184 | 14:50 |
| *** ducttape_ has joined #openstack-nova | 14:50 | |
| moshele | jaypipes: can you review the pci: Allow to update pci_requests in instance_extra https://review.openstack.org/#/c/312028/ ? | 14:51 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add set_inventory() method on ResourceProvider https://review.openstack.org/326440 | 14:51 |
| *** renato_r has joined #openstack-nova | 14:51 | |
| *** amotoki has joined #openstack-nova | 14:53 | |
| *** zenoway has joined #openstack-nova | 14:53 | |
| *** renato_r_ has joined #openstack-nova | 14:54 | |
| *** shaohe_feng has quit IRC | 14:54 | |
| *** chandankumar has quit IRC | 14:54 | |
| *** shaohe_feng has joined #openstack-nova | 14:55 | |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Fix revert on migration with SR-IOV https://review.openstack.org/326174 | 14:55 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 14:56 |
| *** renato_r has quit IRC | 14:57 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova: Remove mox from tests/unit/conductor/tasks/test_live_migrate.py(4) https://review.openstack.org/312589 | 14:58 |
| *** zenoway has quit IRC | 14:58 | |
| *** jlanoux has joined #openstack-nova | 14:59 | |
| *** mdrabe has quit IRC | 15:00 | |
| *** whenry has joined #openstack-nova | 15:00 | |
| *** sree has joined #openstack-nova | 15:01 | |
| jaypipes | moshele: queue.push(moshele). | 15:02 |
| moshele | jaypipes: I hope it is thread safe :) | 15:02 |
| jaypipes | moshele: of course it is. :) | 15:03 |
| *** tinbar has quit IRC | 15:04 | |
| *** shaohe_feng has quit IRC | 15:04 | |
| *** shaohe_feng has joined #openstack-nova | 15:05 | |
| sdague | alaski: looks like one last thing on the oslo.policy stack - https://review.openstack.org/#/c/321243/4/oslo_policy/generator.py | 15:06 |
| openstackgerrit | Sujitha proposed openstack/nova: Improve help text for glance options https://review.openstack.org/324007 | 15:06 |
| *** vishwanathj has joined #openstack-nova | 15:06 | |
| alaski | sdague: cool. those are easily addressable | 15:07 |
| sdague | alaski: yeh, so close | 15:08 |
| *** aarefiev_ has joined #openstack-nova | 15:08 | |
| *** caowei has joined #openstack-nova | 15:08 | |
| *** rk4n_ has joined #openstack-nova | 15:09 | |
| *** ababich1 has joined #openstack-nova | 15:10 | |
| *** vnogin has joined #openstack-nova | 15:10 | |
| *** thumpba has joined #openstack-nova | 15:10 | |
| *** andreykurilin has joined #openstack-nova | 15:11 | |
| *** ababich has quit IRC | 15:11 | |
| *** andreykurilin1 has quit IRC | 15:11 | |
| *** rk4n has quit IRC | 15:12 | |
| *** aarefiev has quit IRC | 15:12 | |
| *** vnogin1 has quit IRC | 15:12 | |
| *** tonytan4ever has joined #openstack-nova | 15:14 | |
| *** syjulian has joined #openstack-nova | 15:14 | |
| *** mdrabe has joined #openstack-nova | 15:14 | |
| *** shaohe_feng has quit IRC | 15:15 | |
| *** shaohe_feng has joined #openstack-nova | 15:15 | |
| *** vdrok-afk is now known as vdrok | 15:16 | |
| *** Jeffrey4l has joined #openstack-nova | 15:16 | |
| jaypipes | claudiub: \o/ remotefx is merged. | 15:17 |
| jaypipes | claudiub: only 17 months. | 15:17 |
| *** rcernin has quit IRC | 15:17 | |
| *** sheel has joined #openstack-nova | 15:17 | |
| *** rlrossit has joined #openstack-nova | 15:17 | |
| openstackgerrit | Sujitha proposed openstack/nova: Improve help text for availability zones options https://review.openstack.org/316300 | 15:17 |
| claudiub | jaypipes: \o/. What do you mean 17 months? | 15:18 |
| *** zenoway has joined #openstack-nova | 15:18 | |
| jaypipes | claudiub: more? | 15:18 |
| claudiub | jaypipes: the patch was first sent in 2013. :) | 15:18 |
| jaypipes | claudiub: I thought that patch was originally proposed 17 months ago>? | 15:18 |
| jaypipes | claudiub: ok, so longer then ;) | 15:18 |
| *** kfarr has joined #openstack-nova | 15:18 | |
| jaypipes | claudiub: you should get an award. | 15:18 |
| cdent | remotefx? Is that so I can control the blinkenlights on the host over the api? | 15:19 |
| claudiub | jaypipes: lol | 15:19 |
| jaypipes | cdent: ack on your last comment about checking allocations before deleting stuff. didn't add that in there because, well, there's no allocations yet ;) | 15:19 |
| * cdent nods | 15:19 | |
| claudiub | cdent: yep, it's especially useful in clubs and discos. :D | 15:20 |
| cdent | jaypipes: I'm currently migrating my way through several separate stacks of changes seeing where the real dependencies lie and trying to figure out an ordering. | 15:20 |
| *** zenoway has quit IRC | 15:22 | |
| openstackgerrit | Anna Babich proposed openstack/python-novaclient: Functional tests for server's description https://review.openstack.org/321031 | 15:23 |
| *** thumpba has quit IRC | 15:24 | |
| *** shaohe_feng has quit IRC | 15:25 | |
| *** shaohe_feng has joined #openstack-nova | 15:25 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Improve help text for xenapi_session_opts https://review.openstack.org/322322 | 15:25 |
| *** links has quit IRC | 15:26 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add set_inventory() method on ResourceProvider https://review.openstack.org/326440 | 15:28 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add InventoryList.find() method https://review.openstack.org/326439 | 15:28 |
| openstackgerrit | Chris Dent proposed openstack/nova: Adjust field types and defaults on Inventory https://review.openstack.org/326578 | 15:28 |
| bauzas | jaypipes: dansmith: cdent: alaski: I saw you guys discussing about the name field and how a bad resource provider could trample other resource inventories | 15:28 |
| *** itamarl has quit IRC | 15:28 | |
| *** sdake has joined #openstack-nova | 15:28 | |
| bauzas | is this something with a consensus yet or not ? | 15:28 |
| cdent | bauzas: that wasn't quite it, it was that a service, like ironic, could provide bad information | 15:29 |
| dansmith | bauzas: right nothing related to name | 15:29 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add new configuration option to turn auto converge on/off https://review.openstack.org/266001 | 15:29 |
| cdent | but the agreement was that we already have that problem in spades | 15:29 |
| bauzas | okay | 15:29 |
| bauzas | so the name field is just for UX ? | 15:30 |
| jaypipes | bauzas: yes. | 15:30 |
| jaypipes | bauzas: more of a tag than anything. | 15:30 |
| bauzas | k | 15:30 |
| bauzas | I'm tempted to +W it but dansmith had a concern, that's hopefully sorted out, right? | 15:30 |
| *** rama_y has quit IRC | 15:31 | |
| dansmith | bauzas: I'm changing the name patch right now | 15:31 |
| *** zenoway has joined #openstack-nova | 15:31 | |
| bauzas | oh okay | 15:31 |
| *** ljxiash has quit IRC | 15:32 | |
| bauzas | dansmith: about what ? having the name not optional ? | 15:32 |
| dansmith | bauzas: and adding generation | 15:33 |
| bauzas | coolness | 15:33 |
| *** thumpba has joined #openstack-nova | 15:33 | |
| *** stewie925 has joined #openstack-nova | 15:34 | |
| *** Swami has joined #openstack-nova | 15:34 | |
| *** Swami_ has joined #openstack-nova | 15:34 | |
| *** mjura has quit IRC | 15:35 | |
| *** shaohe_feng has quit IRC | 15:35 | |
| *** zenoway has quit IRC | 15:36 | |
| *** coolsvap has joined #openstack-nova | 15:36 | |
| *** lifeless has quit IRC | 15:37 | |
| *** chhavi has quit IRC | 15:38 | |
| bauzas | doffm: around ? | 15:38 |
| *** shaohe_feng has joined #openstack-nova | 15:38 | |
| bauzas | doffm: I suppose https://review.openstack.org/#/c/284895/36 would hurt you a bit ? | 15:38 |
| bauzas | possibly just a rebase tho, but I don't know which order to do | 15:38 |
| *** chandankumar has joined #openstack-nova | 15:39 | |
| bauzas | we could possibly accept the above, since it's just a query, and do the aggregate migration too | 15:39 |
| *** syjulian has quit IRC | 15:39 | |
| bauzas | but I want to make sure we don't forget that one | 15:39 |
| *** syjulian has joined #openstack-nova | 15:40 | |
| *** sree has quit IRC | 15:40 | |
| *** sameo has joined #openstack-nova | 15:40 | |
| sdague | claudiub: can you review - https://review.openstack.org/#/c/324670/ to make sure it's covered - glance v2 with hyperv | 15:42 |
| *** roeyc has joined #openstack-nova | 15:42 | |
| claudiub | sdague: will do. thanks for the heads up. :) | 15:42 |
| *** sacharya has joined #openstack-nova | 15:42 | |
| *** mtanino has joined #openstack-nova | 15:44 | |
| doffm | bauzas: A rebase and a bit of work. I have been aware of that patch for a while. That going in first is probably best. | 15:44 |
| bauzas | k | 15:44 |
| bauzas | I also think we don't need to have https://review.openstack.org/#/c/281945/45 as a dep | 15:45 |
| bauzas | k, will make sure we merge it | 15:45 |
| *** syjulian has quit IRC | 15:45 | |
| *** shaohe_feng has quit IRC | 15:45 | |
| *** diga has joined #openstack-nova | 15:45 | |
| cdent | bauzas: the messy ordering there has been because of the api work that is at the top of the stack: it needed both of those | 15:46 |
| *** shaohe_feng has joined #openstack-nova | 15:46 | |
| cdent | but that stack is now in flux because of the changes that jay introduced today | 15:46 |
| *** syjulian has joined #openstack-nova | 15:47 | |
| cdent | so some reordering and extraction can probably happen | 15:47 |
| cdent | I had assumed doffm's stuff was going to make it in before the get_by_uuid patch, but if not, then it might make sense to pull it out of that dependency stack | 15:47 |
| *** sacharya has quit IRC | 15:47 | |
| jaypipes | cdent: I'm working on the patches. adding the patch that adds generation to RP now... | 15:48 |
| cdent | jaypipes: dansmith said he was doing that? | 15:48 |
| jaypipes | cdent: whilst also participating in 7 meetyings today. | 15:48 |
| dansmith | sigh | 15:48 |
| jaypipes | cdent: oh? | 15:48 |
| dansmith | I'm in the middle of that | 15:48 |
| jaypipes | dansmith: np then. I had just started. | 15:49 |
| cdent | gerrit needs locks :) | 15:49 |
| jaypipes | dansmith: I will wait to review yours | 15:49 |
| jaypipes | dansmith: sorry, didn't realize. | 15:49 |
| *** rlrossit has quit IRC | 15:49 | |
| dansmith | jaypipes: YOU BASTARD | 15:49 |
| BobBall | jaypipes: danpb suggested your input would be useful on http://lists.openstack.org/pipermail/openstack-dev/2016-June/096684.html - could you have a look at some point? | 15:49 |
| *** ctrath has quit IRC | 15:49 | |
| *** lifeless has joined #openstack-nova | 15:49 | |
| dansmith | jaypipes: I will forgive you this once | 15:49 |
| jaypipes | thx | 15:49 |
| dansmith | heh | 15:49 |
| cdent | jaypipes: I added a patch between your find() and set_inventory() | 15:50 |
| cdent | and abandoned "correct inventory" | 15:50 |
| jaypipes | BobBall: will try my best. kinda swamped right now :) | 15:50 |
| BobBall | Not surprised! Thanks :) | 15:50 |
| *** ihrachys has quit IRC | 15:51 | |
| *** BjoernT has joined #openstack-nova | 15:51 | |
| *** smatzek_ has quit IRC | 15:52 | |
| *** rlrossit_ has joined #openstack-nova | 15:52 | |
| * cdent does something else for a while so as not to violate any locks | 15:52 | |
| *** andreas_s has quit IRC | 15:53 | |
| *** Swami_ has quit IRC | 15:54 | |
| *** ctrath has joined #openstack-nova | 15:54 | |
| *** Swami has quit IRC | 15:54 | |
| *** shaohe_feng has quit IRC | 15:56 | |
| *** raunak_ has joined #openstack-nova | 15:56 | |
| mriedem_afk | JayF: ok, only another hour of meetings and then i'm on it :)( | 15:56 |
| *** shaohe_feng has joined #openstack-nova | 15:56 | |
| *** armax has joined #openstack-nova | 15:58 | |
| JayF | thanks! | 15:59 |
| *** gokrokve has joined #openstack-nova | 16:02 | |
| *** rdopiera has left #openstack-nova | 16:03 | |
| *** sacharya has joined #openstack-nova | 16:03 | |
| *** shaohe_feng has quit IRC | 16:06 | |
| *** shaohe_feng has joined #openstack-nova | 16:06 | |
| *** sridharg has quit IRC | 16:06 | |
| *** smatzek_ has joined #openstack-nova | 16:09 | |
| *** mpavone has quit IRC | 16:09 | |
| *** mgoddard has quit IRC | 16:10 | |
| *** smatzek_ has quit IRC | 16:10 | |
| *** yamahata has quit IRC | 16:11 | |
| *** armax has quit IRC | 16:12 | |
| *** paul-carlton1 has joined #openstack-nova | 16:12 | |
| *** zenoway has joined #openstack-nova | 16:13 | |
| *** tesseract has quit IRC | 16:13 | |
| *** Apoorva has joined #openstack-nova | 16:14 | |
| *** matrohon has quit IRC | 16:14 | |
| *** Apoorva has quit IRC | 16:15 | |
| *** armax has joined #openstack-nova | 16:16 | |
| *** shaohe_feng has quit IRC | 16:16 | |
| *** Apoorva has joined #openstack-nova | 16:16 | |
| *** shaohe_feng has joined #openstack-nova | 16:16 | |
| *** zenoway has quit IRC | 16:17 | |
| *** r-daneel has quit IRC | 16:17 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Add versioning to response https://review.openstack.org/305369 | 16:20 |
| *** caowei has quit IRC | 16:23 | |
| sdague | lxsli: can you add the blueprint to the commit message for this stack - https://review.openstack.org/#/c/280851 ? | 16:23 |
| sdague | the topic doesn't update launchpad I don't think | 16:23 |
| *** amotoki has quit IRC | 16:24 | |
| *** jaosorior has quit IRC | 16:24 | |
| *** jaosorior has joined #openstack-nova | 16:24 | |
| *** armax has quit IRC | 16:25 | |
| *** salv-orl_ has joined #openstack-nova | 16:25 | |
| *** ssurana has joined #openstack-nova | 16:25 | |
| *** zarkone has quit IRC | 16:25 | |
| *** jaosorior has quit IRC | 16:26 | |
| *** r-daneel has joined #openstack-nova | 16:26 | |
| *** shaohe_feng has quit IRC | 16:26 | |
| *** shaohe_feng has joined #openstack-nova | 16:27 | |
| *** salv-orlando has quit IRC | 16:27 | |
| *** chandankumar has quit IRC | 16:28 | |
| *** thumpba has quit IRC | 16:29 | |
| *** dmk0202 has quit IRC | 16:30 | |
| sdague | dansmith: can you hit claudiub's follow up nit fix patch - https://review.openstack.org/#/c/320599/1 ? | 16:31 |
| dansmith | yups | 16:31 |
| *** coolsvap has quit IRC | 16:33 | |
| *** fragatina has joined #openstack-nova | 16:34 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: config options: Improve help for remote debug https://review.openstack.org/324750 | 16:34 |
| *** yamahata has joined #openstack-nova | 16:34 | |
| *** fragatina has quit IRC | 16:35 | |
| *** fragatina has joined #openstack-nova | 16:35 | |
| *** mriedem_afk is now known as mriedem | 16:36 | |
| *** shaohe_feng has quit IRC | 16:37 | |
| *** jlanoux has quit IRC | 16:37 | |
| *** shaohe_feng has joined #openstack-nova | 16:37 | |
| *** rama_y has joined #openstack-nova | 16:40 | |
| *** derekh has quit IRC | 16:40 | |
| *** tonytan4ever has quit IRC | 16:41 | |
| *** mmandel has quit IRC | 16:41 | |
| *** tpepper has joined #openstack-nova | 16:41 | |
| dansmith | cdent: that patch'll be coming shortly.. unfortunately, had to account for some more details around name than I was expecting | 16:43 |
| *** FL1SK has quit IRC | 16:43 | |
| dansmith | cdent: working around them for the moment and maybe a follow-on to really fix | 16:43 |
| *** haukebruno has quit IRC | 16:43 | |
| auggy | good morning! | 16:44 |
| cdent | dansmith: sorry for your pain. don't rush on my account, I'm stopping for dinner soon and then maybe dying for the evening from allergies. | 16:44 |
| *** chandankumar has joined #openstack-nova | 16:44 | |
| dansmith | cdent: I'm not rushing, just explaining why it's taking a bit | 16:44 |
| dansmith | cdent: enjoy dinner/death | 16:44 |
| cdent | I _always_ look forward to death. | 16:45 |
| auggy | cdent: cake or death? | 16:45 |
| cdent | (and thanks) | 16:45 |
| dansmith | auggy: cake then death | 16:45 |
| cdent | auggy: it's all death all the time round here | 16:45 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add name and generation to ResourceProvider object https://review.openstack.org/281945 | 16:46 |
| *** shaohe_feng has quit IRC | 16:47 | |
| *** nic has joined #openstack-nova | 16:47 | |
| * danpb mutters something about 'cake' being a dangerous drug .... | 16:47 | |
| *** Swami has joined #openstack-nova | 16:47 | |
| *** shaohe_feng has joined #openstack-nova | 16:47 | |
| *** Swami has quit IRC | 16:48 | |
| *** Swami has joined #openstack-nova | 16:48 | |
| danpb | ... at UK TV programme once tricked a whole bunch of celebrity sports, tv people and politicians to film clips condeming use of a made up drug called "cacke" | 16:48 |
| *** rk4n_ has quit IRC | 16:49 | |
| paul-carlton1 | mriedem, see Kevin Mitchell's comment on https://review.openstack.org/#/c/316026/7, is there something we can do to fix this? | 16:49 |
| auggy | danpb: i think that's like the penn and teller episode where they were getting people all outraged about dihydrogen monoxide | 16:49 |
| danpb | hehe yeah | 16:50 |
| *** rk4n has joined #openstack-nova | 16:50 | |
| *** iceyao has quit IRC | 16:51 | |
| cdent | dansmith: huh. just to balance out my brain going "ick", remind me again why we decided that name can't be null? | 16:51 |
| *** chandankumar has quit IRC | 16:51 | |
| dansmith | cdent: like, you want me to call up the logs from this channel yesterday where we discussed it? | 16:51 |
| *** bharath has joined #openstack-nova | 16:52 | |
| *** lucasagomes is now known as lucas-dinner | 16:52 | |
| *** roeyc has quit IRC | 16:52 | |
| mriedem | paul-carlton1: first, you need fewer irc nicks | 16:53 |
| cdent | dansmith: heh, no, I can do that. I was just hoping, blandly, for the ten word summary, but don't worry, I can do that. | 16:53 |
| mriedem | tab-complete makes me sad on your nick | 16:53 |
| mriedem | paul-carlton1: second, i commented in the review - Vek is correct, the microversion support in the client has to go in order | 16:53 |
| dansmith | cdent: becauseI think it's important for the humans that have to run, debug, and maintain this system (and I think alaski agrees) | 16:54 |
| cdent | dansmith thanks, that's an excellent summary | 16:54 |
| *** prateek has joined #openstack-nova | 16:55 | |
| *** prateek has quit IRC | 16:55 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: config options: Improve help for base https://review.openstack.org/324559 | 16:56 |
| diana_clarke | I can't seem to get past 'XenProject CI' merge errors, even when Jenkins says a-okay. And 'recheck' doesn't seem to trigger 'XenProject CI' (as far as I can tell). Does it have it's own recheck command? | 16:57 |
| *** shaohe_feng has quit IRC | 16:57 | |
| dansmith | diana_clarke: it does | 16:57 |
| dansmith | diana_clarke: why we haven't required them to be standardized is beyond me | 16:58 |
| dansmith | diana_clarke: ISTR it is "xen: recheck" or something | 16:58 |
| *** shaohe_feng has joined #openstack-nova | 16:58 | |
| dansmith | or "xen: check yosef before you wreck yosef" | 16:58 |
| diana_clarke | dansmith: Unlike the others, it also doesn't list the recheck command it when it comments on the patch. | 16:58 |
| diana_clarke | dansmith: Thanks, I'll try that. | 16:58 |
| dansmith | diana_clarke: direct your frustration^Hcomments at BobBall | 16:59 |
| BobBall | dansmith: My understanding was that the 'standardisation' was to have jenkins only reply to recheck and all third party CI's to be namespaced on <key>: <action> | 16:59 |
| *** ijw has joined #openstack-nova | 16:59 | |
| *** rlrossit_ has quit IRC | 16:59 | |
| *** bharath has quit IRC | 16:59 | |
| *** mrkz has quit IRC | 16:59 | |
| dansmith | BobBall: oh really? I thought xen was the only one that used that pattern | 16:59 |
| *** rlrossit has joined #openstack-nova | 16:59 | |
| dansmith | BobBall: if so, and you're standards-compliant then I retract my jab :) | 16:59 |
| BobBall | Not listing a recheck command? Gah! That's frustrating. Can I deflect the frustration to anthonyper though? :) | 16:59 |
| *** mrkz has joined #openstack-nova | 17:00 | |
| dansmith | of course | 17:00 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Fix revert on migration with SR-IOV https://review.openstack.org/326174 | 17:00 |
| BobBall | dansmith: I like to think of us as thought leaders in this space. | 17:00 |
| *** vishwanathj has quit IRC | 17:00 | |
| dansmith | BobBall: heh | 17:00 |
| BobBall | dansmith: https://review.openstack.org/#/c/278537/ | 17:00 |
| *** lpetrut has quit IRC | 17:01 | |
| dansmith | BobBall: ouch, okay, you win :) | 17:01 |
| diana_clarke | Lest I make any enemies before I get a chance to know you folks, I wasn't actually frustrated ;) cc BobBall | 17:01 |
| mriedem | the xenproject ci lists a recheck command in the comment it leaves | 17:01 |
| mriedem | i know because i always have to look | 17:02 |
| BobBall | diana_clarke: Got a patch where it didn't leave one? | 17:02 |
| BobBall | Oh - perhaps it doesn't leave them on merge failures... Interesting... | 17:02 |
| *** r-daneel has quit IRC | 17:02 | |
| paul-carlton1 | mriedem, sorry I have two machines in the office and my laptop, the office machines are always running I should logout of irc before I go home | 17:02 |
| BobBall | I have a feeling I saw that on jenkins merge failures too; but if teh xenproject CI is merge failing when jenkins succeeds then I think a fix would be useful. | 17:02 |
| dansmith | hyper-v doesn't use the system:command syntax at least | 17:02 |
| dansmith | which is probably why I didn't think we had a standard | 17:03 |
| diana_clarke | BobBall: https://review.openstack.org/#/c/317694/ It just says "Patch Set 17: Verified-1 This change depends on a change that failed to merge." | 17:03 |
| BobBall | is claudiub the one who might fix that? :) | 17:03 |
| paul-carlton1 | mriedem, How do we get people who introduced api microversions 2.27 to 2.30 to submit python/novaclient microversion bumps? | 17:04 |
| dansmith | diana_clarke: generally rechecking on merge conflict isn't useful, which might be why | 17:04 |
| mriedem | paul-carlton1: ask them? | 17:04 |
| *** thumpba has joined #openstack-nova | 17:04 | |
| BobBall | Curious... anthonyper - any ideas why https://review.openstack.org/#/c/317694 failed to merge only for xenproject CI? | 17:04 |
| *** mmandel has joined #openstack-nova | 17:05 | |
| paul-carlton1 | I did ask cdent and also someone else I forgotten their name now | 17:05 |
| bzurawski | quit | 17:05 |
| *** bzurawski has quit IRC | 17:05 | |
| cdent | paul-carlton1: I did it. | 17:05 |
| BobBall | (fwiw, anthonyper may have gone home already, hence I hope we will get a reply tomorrow) | 17:05 |
| *** Apoorva_ has joined #openstack-nova | 17:06 | |
| paul-carlton1 | Maybe we should not approve nova changes with new api version unless there is a python-novaclient version bump change submitted too? | 17:06 |
| paul-carlton1 | cdent, so mriedem's suggestion works some of the time, thanks | 17:06 |
| anthonyper | BobBall: I will look at it | 17:07 |
| BobBall | Ah! awesome | 17:07 |
| *** eharney has quit IRC | 17:07 | |
| *** shaohe_feng has quit IRC | 17:07 | |
| diana_clarke | anthonyper: Thanks!! | 17:08 |
| *** Apoorva has quit IRC | 17:08 | |
| *** shaohe_feng has joined #openstack-nova | 17:10 | |
| *** mvk has quit IRC | 17:11 | |
| *** tonytan4ever has joined #openstack-nova | 17:13 | |
| *** moshele has quit IRC | 17:14 | |
| *** r-daneel has joined #openstack-nova | 17:15 | |
| *** chandankumar has joined #openstack-nova | 17:16 | |
| *** paul-carlton1 has quit IRC | 17:16 | |
| *** paul-carlton1 has joined #openstack-nova | 17:16 | |
| *** shaohe_feng has quit IRC | 17:18 | |
| *** shaohe_feng has joined #openstack-nova | 17:18 | |
| *** tyrola has quit IRC | 17:19 | |
| anthonyper | diana_clarke, BobBall: about the merge faillure, I guess something change before the CI start to test the commit, I only see this in the log: "zuul.IndependentPipelineManager: Dequeuing change 317694,17 because it can no longer merge". I thing a xen: recheck should be enough. | 17:21 |
| paul-carlton1 | pkholkin, Can you submit a python-novaclient change to bump the microversion to 2.28 pls | 17:21 |
| *** chandankumar has quit IRC | 17:21 | |
| paul-carlton1 | see Kevin Mitchell's comment on https://review.openstack.org/#/c/316026/7 | 17:22 |
| alaski | cdent: dansmith +1 to what you said | 17:22 |
| cdent | ✔ | 17:23 |
| *** tyrola has joined #openstack-nova | 17:24 | |
| *** amotoki has joined #openstack-nova | 17:24 | |
| *** ctrath has quit IRC | 17:25 | |
| claudiub | dansmith: BobBall: just reading now. So, the hyper-v ci is triggered either with a recheck, or with a check hyper-v. We chose 'check hyper-v', because that's how most of the 3rd party CIs were being triggered back in the early days. Since then, other CIs changed their trigger magic spell, but we didn't find any reason to do so. :) | 17:25 |
| cdent | alaski: I was just confirming the reasoning because the change dan had to make was more complicated than expected. | 17:25 |
| dansmith | cdent: more than I expected too | 17:26 |
| cdent | that's what I mean | 17:26 |
| dansmith | cdent: all of your silly tests got in the way | 17:26 |
| alaski | ahh, okay | 17:26 |
| dansmith | I mean "the silly tests" | 17:26 |
| cdent | dansmith: pish, at least there weren't many mocks to decode | 17:27 |
| claudiub | dansmith: BobBall: At the moment, there's no standardization regarding the 3rd party recheck method, neither on nova, or cinder, or any other. I'm fine with changing it, if we are going to standardize it. :) | 17:27 |
| andrearosa | mriedem: as PaulMurray suggested my friday's nickname is going to be paul-carlton4 | 17:27 |
| PaulMurray | mriedem, we decided for friday nicks everyone in hpe is going to be called paul-carlton | 17:27 |
| dansmith | claudiub: BobBall asserted that there is a standard | 17:27 |
| *** mriedem has quit IRC | 17:27 | |
| *** mriedem has joined #openstack-nova | 17:27 | |
| *** thumpba has quit IRC | 17:27 | |
| PaulMurray | andrearosa, I bag pual-carlton7 | 17:28 |
| dansmith | mriedem: I think you missed: [10:27:33] <PaulMurray>mriedem, we decided for friday nicks everyone in hpe is going to be called paul-carlton | 17:28 |
| *** shaohe_feng has quit IRC | 17:28 | |
| *** shaohe_feng has joined #openstack-nova | 17:28 | |
| *** amotoki has quit IRC | 17:29 | |
| *** ctrath has joined #openstack-nova | 17:29 | |
| mriedem | gah | 17:30 |
| mriedem | wouldn't surprise me | 17:30 |
| mriedem | maybe some of those paul-carltons can implement microversion support in the novaclient | 17:30 |
| andrearosa | lol | 17:31 |
| mriedem | 2 more stable/mitaka backports ready for +W https://review.openstack.org/#/q/topic:bug/1503453+status:open | 17:31 |
| *** vtech has quit IRC | 17:33 | |
| claudiub | dansmith: BobBall: there is no mention of any recheck standard on this page, other than the all-mighty 'recheck' recheck, or at the very least, not in the 'Requirements' section: http://docs.openstack.org/infra/system-config/third_party.html#requirements | 17:34 |
| claudiub | unless there's some newer, updated version of that doc? | 17:34 |
| claudiub | auggy: Do you like bridges? In your opinion, what is the best name for a bridge? :D | 17:37 |
| *** shaohe_feng has quit IRC | 17:38 | |
| *** shaohe_feng has joined #openstack-nova | 17:39 | |
| *** keekz_ is now known as nicholas | 17:40 | |
| *** nicholas is now known as keekz | 17:40 | |
| *** keekz is now known as nicholas | 17:40 | |
| mriedem | dansmith: you should probably +W this https://review.openstack.org/#/c/324401/ | 17:40 |
| mriedem | w/o looking | 17:40 |
| sdague | sudipto: when you fix up the glance v2 xenserver patch, can you rebase and put hyperv in front of it | 17:40 |
| *** salv-orl_ has quit IRC | 17:41 | |
| *** danpb has quit IRC | 17:41 | |
| sdague | because I think hyperv is ready to go | 17:41 |
| openstackgerrit | Paul Murray proposed openstack/nova: Add periodic task to clean expired console tokens https://review.openstack.org/325381 | 17:41 |
| openstackgerrit | Paul Murray proposed openstack/nova: Add console auth token records to the database https://review.openstack.org/325414 | 17:41 |
| sdague | and it would be better to just land that then hold it on xenserver | 17:41 |
| openstackgerrit | Paul Murray proposed openstack/nova: Add console auth object https://review.openstack.org/320063 | 17:41 |
| *** salv-orlando has joined #openstack-nova | 17:41 | |
| openstackgerrit | Paul Murray proposed openstack/nova: Add instance uuid to console URLs https://review.openstack.org/325382 | 17:41 |
| openstackgerrit | Paul Murray proposed openstack/nova: Add console auth tokens db api methods https://review.openstack.org/319964 | 17:41 |
| openstackgerrit | Paul Murray proposed openstack/nova: Add console auth tokens table and model https://review.openstack.org/319327 | 17:41 |
| sudipto | sdague, ok sure. | 17:41 |
| sdague | sudipto: thanks | 17:41 |
| *** edand has quit IRC | 17:42 | |
| sdague | mfedosin: any idea what the vmware story is here? is that something trapped down in oslo.vmware? | 17:42 |
| sdague | and is anyone from vmware active enough to keep an eye on it? | 17:42 |
| *** paul-carlton1 has quit IRC | 17:43 | |
| *** nicholas is now known as keekz | 17:44 | |
| *** sambetts is now known as sambetts|afk | 17:44 | |
| *** tonytan4ever has quit IRC | 17:44 | |
| pkholkin | paul-carlton1: hi, I am on pto this week, will do after that | 17:44 |
| dansmith | mriedem: whaaa | 17:45 |
| *** Sukhdev has joined #openstack-nova | 17:46 | |
| mfedosin | sdague: what's wrong with vmware? | 17:46 |
| PaulMurray | jaypipes, mriedem would like to get your opinion on table schema in https://review.openstack.org/319327 - should instance_uuid be a foreign key on instances table ? | 17:47 |
| dansmith | mfedosin: that's really a long and complicated answer :) | 17:47 |
| *** ducttape_ has quit IRC | 17:47 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot https://review.openstack.org/209581 | 17:47 |
| sdague | mriedem / dansmith: (╯°□°）╯︵ ┻━┻ | 17:47 |
| dansmith | sdague: is that for the flavor revert? | 17:48 |
| mfedosin | dansmith: :) are there some tests failed? | 17:48 |
| sdague | dansmith: yes | 17:48 |
| mriedem | i thought it was funny | 17:48 |
| dansmith | mfedosin: I meant the philosophical answer to that general question :P | 17:48 |
| *** Sukhdev has quit IRC | 17:48 | |
| sdague | mfedosin: no, the point is that I assume vmware cares about glance somewhere :) | 17:48 |
| *** shaohe_feng has quit IRC | 17:48 | |
| *** shaohe_feng has joined #openstack-nova | 17:49 | |
| sdague | it's not in the patch series | 17:49 |
| mriedem | tjones: ^? | 17:49 |
| mriedem | tjones: where does the vmware driver handle snapshots? | 17:49 |
| mfedosin | sabari: ^ | 17:49 |
| sdague | I'm personally fine that it's on the vmware team to show up and do a thing | 17:49 |
| mfedosin | sdague: +1 | 17:50 |
| sdague | and not going to block moving forward here, but would at least like to know if the story is 1) it's all covered 2) someone is looking at it 3) glance-v2-what-now? | 17:50 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot https://review.openstack.org/209581 | 17:50 |
| *** PaulMurray has quit IRC | 17:50 | |
| auggy | claudiub: bridges are pretty neat :) | 17:50 |
| alaski | Is there a way to run tests locally with the same ordering that Jenkins used for a job? There's a py34 failure on my patch that isn't happening locally and I suspect a race | 17:51 |
| sdague | mtreinish: ^^^ ? | 17:52 |
| auggy | alaski: i've been working with this tool, not sure if it would be helpful - http://docs.openstack.org/infra/system-config/devstack-gate.html | 17:52 |
| *** diga has quit IRC | 17:52 | |
| alaski | auggy: that's a different level than I'm at | 17:52 |
| alaski | I'm looking for testrepository ordering | 17:53 |
| *** mvk has joined #openstack-nova | 17:53 | |
| alaski | and I'm assuming that lifeless is not around at this time | 17:53 |
| sdague | alaski: with lifeless around less, mtreinish probably understands testr better than anyone else | 17:53 |
| auggy | ah ok | 17:54 |
| alaski | gotcha, good to now | 17:54 |
| alaski | *note | 17:54 |
| * auggy just has devstack-gate on the brain right now ;) | 17:54 | |
| sdague | my guess though is that given we're starting from a blank seed (I think) you probably will get the same order if you run at the same parallelism | 17:54 |
| sdague | because testr is actually quite deterministic in test ordering | 17:55 |
| sdague | you'd need to delete your .testrepository before running though | 17:55 |
| alaski | okay. by seed do you mean the times.dbm file it uses to sort by run time? | 17:55 |
| sdague | yeh | 17:55 |
| sdague | I'm pretty sure we're blank in the gate for that | 17:56 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make Aggregate metadata functions work with API db https://review.openstack.org/297350 | 17:56 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Block aggregate creation until main database empty https://review.openstack.org/315235 | 17:56 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add data migration methods for Aggregate. https://review.openstack.org/314782 | 17:56 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make Aggregate.save work with the API db https://review.openstack.org/305466 | 17:56 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Aggregate create and destroy work against API db https://review.openstack.org/314781 | 17:56 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make Aggregate host operations work against API db https://review.openstack.org/296597 | 17:56 |
| *** sudipto has quit IRC | 17:56 | |
| claudiub | auggy: They sure are. :) I believe the best name for a bridge is 'linuxbridge'. :D | 17:56 |
| *** fragatina has quit IRC | 17:56 | |
| sdague | so delete the local file you have in your nova test tree, and it might end up the same | 17:56 |
| alaski | sdague: I'll give it a go, though I have to half my concurrency for it :( | 17:56 |
| sdague | alaski: yeh, well, them's the ropes :) | 17:56 |
| auggy | claudiub: https://i.chzbgr.com/full/6752160512/h185CF54E/ | 17:57 |
| *** rlrossit has quit IRC | 17:57 | |
| mriedem | wow | 17:57 |
| mriedem | so, | 17:57 |
| claudiub | auggy: lol | 17:57 |
| mriedem | https://review.openstack.org/#/c/309614/ merged, bp and spec not approved | 17:57 |
| mriedem | spec was abandoned b/c the code was merged | 17:57 |
| mriedem | ha | 17:57 |
| cfriesen | is Przemyslaw Czesnowicz on here? I think I found a problem with the ISOLATE thread policy on non-HT hosts | 17:58 |
| *** eharney has joined #openstack-nova | 17:58 | |
| mriedem | no release note, nothing | 17:58 |
| *** shaohe_feng has quit IRC | 17:59 | |
| *** shaohe_feng has joined #openstack-nova | 18:00 | |
| bauzas | jaypipes: dansmith: if you agree, I'd separate https://review.openstack.org/#/c/284895/36 from the series so you guys could review it (and +W it) without needing to wait for https://review.openstack.org/#/c/281945/45 | 18:00 |
| bauzas | jaypipes: dansmith: cdent: I'm about to do the above, just lemme know before I upload it | 18:00 |
| JayF | mriedem: TY for the stable review. Uf you have any suggestions for another reviewer to ping to get those Ironic fixes in, I'd appreciate it. :) | 18:00 |
| sdague | mriedem: ummmm | 18:01 |
| mriedem | i'm not really sure how backward compatible this is | 18:01 |
| jaypipes | bauzas: I wouldn't bother. dansmith is pushing a quick fix to the name patch and then I hope to get that merged. | 18:01 |
| mriedem | dane-fichter was raising that concern in the spec | 18:01 |
| dansmith | literally pushing right now | 18:01 |
| bauzas | okay | 18:01 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Enable mutable config in Nova https://review.openstack.org/280851 | 18:01 |
| mriedem | https://review.openstack.org/#/c/309614/4/nova/keymgr/__init__.py has comments about it being backward compat | 18:01 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add name and generation to ResourceProvider object https://review.openstack.org/281945 | 18:01 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Make LM downtime params mutable https://review.openstack.org/319254 | 18:02 |
| jaypipes | bauzas: et volia ^ | 18:02 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Make LM timeout params mutable https://review.openstack.org/319203 | 18:03 |
| * bauzas cleaning up his glasses :p | 18:03 | |
| sdague | lxsli: thanks for the commit message fixes | 18:03 |
| lxsli | sdague: np, thanks for review | 18:04 |
| sdague | mriedem: ok, well I'm assuming we don't want to revert the keymanager, so maybe figure out who is on the hook to show up with a reno | 18:04 |
| bauzas | jaypipes: dansmith: okay, so the bottom change is not using the latest https://review.openstack.org/#/c/284895/2, I'll cut the latter from the series | 18:04 |
| lxsli | I could use help on https://review.openstack.org/#/c/325997/ | 18:04 |
| mriedem | sdague: i guess i'll hunt down kaitlinn farr | 18:05 |
| dansmith | mriedem: kfarr here I think | 18:05 |
| mriedem | kfarr: so i see https://review.openstack.org/#/c/309614/ merged w/o the blueprint or spec being approved, | 18:06 |
| mriedem | kfarr: are you going to provide a release note in nova for the deprecated options and new usage of castellan | 18:06 |
| mriedem | ? | 18:06 |
| *** paul-carlton1 has joined #openstack-nova | 18:06 | |
| mriedem | do we still need a test dependency on barbicanclient? | 18:07 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Add Hyper-V storage QoS support https://review.openstack.org/175479 | 18:07 |
| jaypipes | dansmith: +2 from me on 281945 | 18:07 |
| dansmith | jaypipes: ack | 18:07 |
| *** moshele has joined #openstack-nova | 18:08 | |
| *** tyrola has quit IRC | 18:08 | |
| *** shaohe_feng has quit IRC | 18:09 | |
| *** diogogmt has quit IRC | 18:11 | |
| *** gyee has joined #openstack-nova | 18:11 | |
| *** mpavone has joined #openstack-nova | 18:11 | |
| *** shaohe_feng has joined #openstack-nova | 18:12 | |
| jaypipes | bauzas: can you review dansmith's patch 281945 at your earliest please :) | 18:12 |
| *** sameo is now known as sameo_afk | 18:12 | |
| bauzas | jaypipes: already on it | 18:12 |
| bauzas | I'm a bit blocking on the short name | 18:12 |
| jaypipes | coolio, thx | 18:12 |
| dansmith | it's cdent's | 18:13 |
| bauzas | I'm just verifying where we add the host name | 18:13 |
| bauzas | because AFAIR, we don't FQDN | 18:13 |
| *** diogogmt has joined #openstack-nova | 18:13 | |
| *** jichen has joined #openstack-nova | 18:13 | |
| dansmith | bauzas: it's just safety to make sure we don't end up with an FQDN in there.. it's graceful if already short | 18:13 |
| bauzas | (for the host field of the ComputeNode object tbc) | 18:13 |
| *** vtech has joined #openstack-nova | 18:13 | |
| *** mvk_ has joined #openstack-nova | 18:14 | |
| bauzas | dansmith: sure, I understand that, but I thought we were already doing that, sec | 18:14 |
| jaypipes | bauzas: compute_nodes.host is whatever is returned by the hypervisor's get_host_ip() method | 18:14 |
| bauzas | yeah | 18:14 |
| jaypipes | or get_hostname(), can't remember... | 18:15 |
| bauzas | it's when we init the RT | 18:15 |
| bauzas | by the compute node init | 18:15 |
| bauzas | I mean the service startup | 18:15 |
| *** doug-fis_ has joined #openstack-nova | 18:15 | |
| *** diogogmt has quit IRC | 18:15 | |
| *** thumpba has joined #openstack-nova | 18:16 | |
| jaypipes | bauzas: what is your question on dansmith's patch, though? | 18:16 |
| dansmith | jaypipes: he's concerned I'm duplicating that single line of work I think | 18:16 |
| openstackgerrit | Ed Leafe proposed openstack/nova-specs: Accept multiple hosts when calling live migration https://review.openstack.org/276840 | 18:16 |
| bauzas | dansmith: zactly | 18:17 |
| *** mvk has quit IRC | 18:17 | |
| bauzas | but that's not yet a question, just a verif :) | 18:17 |
| bauzas | okay, I'm clear now | 18:18 |
| dansmith | it looks to me like it comes from the service | 18:18 |
| bauzas | jaypipes: FWIW, self.host taken from RT is not given by the hypervisor, but rather set by the main Manager | 18:18 |
| *** doug-fish has quit IRC | 18:18 | |
| *** READ10 has quit IRC | 18:18 | |
| bauzas | yeah, the CONF.host value | 18:18 |
| bauzas | which could be then a FQDN | 18:19 |
| jaypipes | bauzas: yeah, sorry, was thinking about hypervisor_hostname | 18:19 |
| bauzas | and we don't treat that, so it's nice to have the short_name done by the patch | 18:19 |
| *** shaohe_feng has quit IRC | 18:19 | |
| jaypipes | bauzas: I'm not following you. what did you want the patch to do? | 18:19 |
| dansmith | jaypipes: exactly what I do now | 18:20 |
| bauzas | jaypipes: no worries, it's hypervisor_hostname which is obtained from the virt driver | 18:20 |
| *** shaohe_feng has joined #openstack-nova | 18:20 | |
| jaypipes | dansmith: ah, gotcha. | 18:20 |
| *** lucas___ has joined #openstack-nova | 18:20 | |
| bauzas | jaypipes: I was thinking out loud, because I said "I'm at bit blocked atm by shortening the hostname in the cn object" | 18:20 |
| bauzas | jaypipes: but now, I'm all good - for that point | 18:20 |
| melwitt | alaski: have you tried loading the subunit stream from the jenkins run? http://testrepository.readthedocs.io/en/latest/MANUAL.html just guessing based on the doc | 18:20 |
| bauzas | gosh, I love subunit for that | 18:22 |
| alaski | melwitt: I have not, but will attempt that now | 18:22 |
| bauzas | in particular for reproducing something weird | 18:22 |
| alaski | my attempt to replicate via matching concurrency failed. possibly related to a different number of skipped tests | 18:23 |
| *** syjulian has quit IRC | 18:23 | |
| *** mpavone has quit IRC | 18:24 | |
| bauzas | alaski: yeah missed your original point, but you know there is a possibility to bisect a failing test thanks to subunit ? | 18:24 |
| bauzas | lemme find out that awesome feature | 18:24 |
| alaski | bauzas: yeah, but I haven't gotten it to fail locally yet | 18:24 |
| paul-carlton1 | pkholkin, bauzas did you see my email about the missing python-novaclient version bumps? | 18:24 |
| *** ducttape_ has joined #openstack-nova | 18:24 | |
| bauzas | I used that once for that precise issue | 18:24 |
| bauzas | paul-carlton1: yeah I replied to you | 18:24 |
| bauzas | saying I should do that by tomorrow unless some big comet lands on earth | 18:25 |
| *** rk4n has quit IRC | 18:25 | |
| *** amotoki has joined #openstack-nova | 18:26 | |
| bauzas | alaski: http://testrepository.readthedocs.io/en/latest/MANUAL.html#automated-test-isolation-bisection | 18:26 |
| bauzas | ^ that is what I used which worked like a charm | 18:26 |
| *** syjulian has joined #openstack-nova | 18:27 | |
| *** ametts has quit IRC | 18:27 | |
| paul-carlton1 | bauzas, see it now, EOW is fine thanks | 18:27 |
| *** paul-carlton1 has quit IRC | 18:28 | |
| alaski | melwitt: loading the subunit stream seems to have been useful, thanks. I thought I needed a test list | 18:29 |
| *** shaohe_feng has quit IRC | 18:29 | |
| *** amotoki has quit IRC | 18:30 | |
| *** shaohe_feng has joined #openstack-nova | 18:30 | |
| melwitt | alaski: nice. running failing tests from the stream should be 'testr run --failing' and then you can get fancy by doing the test isolation stuff bauzas mentioned | 18:31 |
| *** chandankumar has joined #openstack-nova | 18:31 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds vNUMA implementation https://review.openstack.org/282407 | 18:32 |
| *** gokrokve has quit IRC | 18:32 | |
| *** baoli_ has quit IRC | 18:32 | |
| alaski | yeah, I'm running the analyze isolation command now and hoping it spits out something useful | 18:33 |
| openstackgerrit | Samuel Matzek proposed openstack/nova: Detach volume after deleting instance with no host https://review.openstack.org/256059 | 18:33 |
| *** lpetrut has joined #openstack-nova | 18:34 | |
| *** Apoorva_ has quit IRC | 18:34 | |
| *** edand has joined #openstack-nova | 18:34 | |
| *** Apoorva has joined #openstack-nova | 18:35 | |
| smcginnis | Anyone know if this is a known issue? From the scheduler: | 18:36 |
| smcginnis | ram: 3441MB disk: -1024MB io_ops: 0 instances: 0 does not have 0 MB usable disk, it only has -1024.0 MB usable disk | 18:37 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: numa: Fixes NUMA topology related unit tests https://review.openstack.org/278835 | 18:37 |
| *** rk4n has joined #openstack-nova | 18:37 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: libvirt: fixes python 3 related unit tests https://review.openstack.org/278834 | 18:37 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes python 3 unit tests https://review.openstack.org/278810 | 18:37 |
| *** lucas___ has quit IRC | 18:38 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: compute: fixes python 3 related unit tests https://review.openstack.org/300262 | 18:39 |
| kfarr | dansmith, mriedem just saw these messages | 18:39 |
| mriedem | kfarr: sent you an email | 18:39 |
| *** lucas___ has joined #openstack-nova | 18:39 | |
| *** shaohe_feng has quit IRC | 18:40 | |
| mriedem | kfarr: basically we need a release note for the deprecated options introduced in that castellan integration change that landed in n-1 | 18:40 |
| *** shaohe_feng has joined #openstack-nova | 18:40 | |
| *** ametts has joined #openstack-nova | 18:40 | |
| kfarr | mriedem, ok thanks, got your email too | 18:40 |
| kfarr | I can propose a release note, will add you as a reviewer when I get it up | 18:41 |
| kfarr | Thanks for not reverting the patch :) | 18:41 |
| diana_clarke | anthonyper: Yup, 'xen: recheck' just worked, despite claiming earlier that it couldn't merge it. It keeps doing this for patch series though. Anyhoo, thanks! | 18:42 |
| *** lucas___ has quit IRC | 18:44 | |
| *** zenoway has joined #openstack-nova | 18:45 | |
| bauzas | dansmith: cdent: jaypipes: probably a miss, but it seems we're removing a assertRaises() which is still useful in https://review.openstack.org/#/c/281945/47 | 18:45 |
| bauzas | an* | 18:45 |
| bauzas | or I missed something | 18:45 |
| mtreinish | alaski: the ordering locally after nuking .testrepository should be the same in the gate | 18:46 |
| mtreinish | assuming the same concurrency | 18:46 |
| *** Sukhdev has joined #openstack-nova | 18:47 | |
| alaski | mtreinish: I think that should have worked, except I end up with 42 skipped tests locally and 25 in the gate which may have affected things | 18:47 |
| alaski | because the failure didn't reproduce | 18:47 |
| dansmith | bauzas: I dunno, that was in the patch before I took it | 18:48 |
| dansmith | bauzas: it looks oddly placed anyway | 18:48 |
| bauzas | dansmith: yep, I know, I verified that | 18:48 |
| bauzas | hence my call to cdent :p | 18:48 |
| kfarr | mriedem, this is someone else's patch for a release note, but does it work? | 18:49 |
| kfarr | https://review.openstack.org/#/c/317684/9 ^^ | 18:49 |
| *** zenoway has quit IRC | 18:49 | |
| *** shaohe_feng has quit IRC | 18:50 | |
| cdent | bauzas: i'm just digging through the several thousand changes to find where it changed, because I didn't do it at the very start | 18:50 |
| bauzas | possibly a merge | 18:50 |
| *** shaohe_feng has joined #openstack-nova | 18:51 | |
| dansmith | coming up: | 18:51 |
| mtreinish | alaski: hmm, the scheduler without any data in .testrepository should just take the sorted list and iterate over it and put things in N bins: https://github.com/testing-cabal/testrepository/blob/master/testrepository/testcommand.py#L441-L445 | 18:51 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add name and generation to ResourceProvider object https://review.openstack.org/281945 | 18:51 |
| mtreinish | if you can't get it to behave exactly like the gate there is a mode test test isolation which might be a better way to go | 18:52 |
| mtreinish | although I've had limited success using that personally | 18:52 |
| *** FL1SK has joined #openstack-nova | 18:53 | |
| mriedem | kfarr: the release note looks ok, the rest of that change looks busted thogh | 18:53 |
| mriedem | *though | 18:53 |
| *** jwcroppe has quit IRC | 18:54 | |
| jaypipes | bauzas: can we add a new test method for verifying the not found in a followup patch please? | 18:54 |
| *** lucas___ has joined #openstack-nova | 18:54 | |
| alaski | mtreinish: okay, based on the testr code it should have reproduced locally so maybe something else is going on here. I'm trying the isolation thing now but I don't have high hopes. | 18:55 |
| kfarr | mriedem, hm ok, I will propose a separate patch then | 18:55 |
| bauzas | jaypipes: well, I usually tend to say that for new features not fully covered, not for test coverage regression, but idk honestly | 18:55 |
| dansmith | jaypipes: I already did | 18:55 |
| mriedem | kfarr: or just fix that one | 18:55 |
| jaypipes | dansmith: oh, gotcha. | 18:55 |
| mriedem | kfarr: comments inline, i haven't reviewed the whole thing, but have a meeting now | 18:55 |
| mtreinish | alaski: so the ordering might have been the same but it if it's an interaction between tests it might just be a race and not completely based on the ordering | 18:56 |
| dansmith | jaypipes: er, already updated the patch | 18:56 |
| *** ctrath has quit IRC | 18:56 | |
| mtreinish | so the different execute speed being different could mean you don't hit it | 18:56 |
| jaypipes | dansmith: +2d. | 18:56 |
| alaski | mtreinish: my understanding is that concurrent runners are totally isolated and tests are run serially in a runner so I didn't think speed would matter | 18:57 |
| *** fragatina has joined #openstack-nova | 18:58 | |
| alaski | but I'm not really sure on that | 18:58 |
| mtreinish | alaski: I don't know what the specific issue is here. But, yes the tests run in multiple processes, but if there is some other state (like a file or db) that could be an issue | 18:58 |
| bauzas | dansmith: just one point before +Wing https://review.openstack.org/#/c/281945/48/nova/objects/compute_node.py , the FIXME means that we would change the behaviour by updating the original name later once we add .save() op right? | 18:58 |
| bauzas | dansmith: for the moment, we keep the old name | 18:58 |
| dansmith | bauzas: yes | 18:59 |
| *** mriedem1 has joined #openstack-nova | 18:59 | |
| alaski | mtreinish: fair point. the issue is that http://logs.openstack.org/27/263927/39/check/gate-nova-python34-db/f2013c3/console.html#_2016-06-07_16_00_46_666 isn't failing for me locally. But that traceback may give me something to go in without reproducing | 18:59 |
| bauzas | that's a quite big change, so I'm fine adding the missing functional test later in a follow-up but I still think we should cover that | 18:59 |
| bauzas | dansmith: ^ | 18:59 |
| *** ihrachys has joined #openstack-nova | 18:59 | |
| dansmith | bauzas: what is a big change for later? | 18:59 |
| bauzas | dansmith: updating the RP name | 19:00 |
| *** ctrath has joined #openstack-nova | 19:00 | |
| dansmith | bauzas: why? | 19:00 |
| bauzas | in case it's conflicting | 19:00 |
| dansmith | bauzas: nothing can see the name right now | 19:00 |
| bauzas | right, good point | 19:00 |
| *** doug-fis_ has quit IRC | 19:00 | |
| *** shaohe_feng has quit IRC | 19:00 | |
| bauzas | but once we add an API resource for RPs, it will become a big change right? :) | 19:00 |
| *** gszasz has quit IRC | 19:00 | |
| dansmith | tbh, I don't see why | 19:00 |
| *** mriedem has quit IRC | 19:01 | |
| jaypipes | bauzas, dansmith, alaski: this one should be easy: https://review.openstack.org/#/c/326439/ | 19:01 |
| *** doug-fish has joined #openstack-nova | 19:01 | |
| *** doug-fish has quit IRC | 19:01 | |
| dansmith | this is auto-generated from the compute host.. the uuid is the really unique thing here, the name is for easy correlation by humans | 19:01 |
| mtreinish | alaski: so one thing I should add is that testr controls how the tests are grouped for each worker. But, it doesn't actually determine the execute order in the serial worker process | 19:01 |
| *** shaohe_feng has joined #openstack-nova | 19:01 | |
| mtreinish | that's the domain of the actual test runner being used. (in this case subunit.run) | 19:01 |
| *** doug-fish has joined #openstack-nova | 19:01 | |
| bauzas | dansmith: I know, I just mean that for example say operator changes the service host name, it will still keep the original RP name, correct ? well, until we write your FIXME | 19:02 |
| *** raunak_ has quit IRC | 19:03 | |
| dansmith | bauzas: yes, but all that means is that the hostname part of the name for humans will be out of date, but... | 19:03 |
| alaski | mtreinish: gotcha. I did load the subunit output which spit the failures out at me. but then I started the isolation thing which has not been useful yet. presumably I could run the tests after loading the subunit stream and reproduce | 19:03 |
| *** moshele has quit IRC | 19:03 | |
| bauzas | dansmith: anyway, I don't want to shave a yak | 19:03 |
| dansmith | bauzas: I think the yak is already bare | 19:04 |
| dansmith | bauzas: but yeah, lets not shave any more or we'll draw blood | 19:04 |
| openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref, os-availability-zone.inc https://review.openstack.org/321847 | 19:04 |
| *** rahuls has joined #openstack-nova | 19:04 | |
| bauzas | heh | 19:04 |
| auggy | i'm all for yak shaving ;) http://reywafibers.com/about/yak-down | 19:05 |
| *** chandankumar has quit IRC | 19:05 | |
| alaski | jaypipes: could an inventory list have two of the same resource classes? | 19:05 |
| *** sheel has quit IRC | 19:05 | |
| bauzas | alaski: MHO would say no, but I leave jaypipes here :) | 19:06 |
| jaypipes | alaski: no. | 19:08 |
| jaypipes | alaski: there is a unique constraint on the inventories table that prevents that. | 19:08 |
| *** edand has quit IRC | 19:08 | |
| jaypipes | alaski: so I see no reason to worry about multiple entries for a resource class in find(). | 19:09 |
| jaypipes | alaski: if that's what you're getting at :) | 19:09 |
| alaski | yep. the UC alleviates my concern | 19:09 |
| *** ctrath has quit IRC | 19:10 | |
| *** shaohe_feng has quit IRC | 19:10 | |
| *** shaohe_feng has joined #openstack-nova | 19:11 | |
| *** rlrossit has joined #openstack-nova | 19:13 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Adjust field types and defaults on Inventory https://review.openstack.org/326578 | 19:14 |
| *** timello has quit IRC | 19:15 | |
| *** Kennan has quit IRC | 19:15 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add ResourceProvider.save() https://review.openstack.org/326706 | 19:15 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make create_inventory() handle name change https://review.openstack.org/326707 | 19:15 |
| dansmith | bauzas: happy now? ^ | 19:15 |
| bauzas | huh | 19:16 |
| *** Kennan has joined #openstack-nova | 19:16 | |
| *** lucas___ has quit IRC | 19:16 | |
| *** ihrachys has quit IRC | 19:18 | |
| *** ihrachys has joined #openstack-nova | 19:18 | |
| jaypipes | dansmith: +2 from me on both. | 19:19 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Update libvirt to use the new imagebackend methods https://review.openstack.org/282580 | 19:19 |
| dansmith | jaypipes: I only ran spot checks on tests, so we'll see how good I am :) | 19:19 |
| *** lucas___ has joined #openstack-nova | 19:19 | |
| dansmith | anyway, I'm in no hurry to merge those, just trying to appease bauzas :) | 19:20 |
| *** timello has joined #openstack-nova | 19:20 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Delete old imagebackend methods & tests https://review.openstack.org/322974 | 19:20 |
| bauzas | dansmith: but I was really okay :) | 19:20 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Remove the last bits of mox from test_imagebackend https://review.openstack.org/323038 | 19:20 |
| *** shaohe_feng has quit IRC | 19:21 | |
| *** lucas____ has joined #openstack-nova | 19:21 | |
| dansmith | pep8 fix coming | 19:21 |
| *** shaohe_feng has joined #openstack-nova | 19:21 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add ResourceProvider.save() https://review.openstack.org/326706 | 19:21 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make create_inventory() handle name change https://review.openstack.org/326707 | 19:21 |
| *** liusheng has quit IRC | 19:22 | |
| *** liusheng has joined #openstack-nova | 19:22 | |
| *** lucas___ has quit IRC | 19:24 | |
| *** mriedem has joined #openstack-nova | 19:24 | |
| *** ctrath has joined #openstack-nova | 19:24 | |
| *** lucas____ has quit IRC | 19:25 | |
| *** mriedem1 has quit IRC | 19:25 | |
| Swanson | Seeing an error where nova is going negative on available disk space. Scheduler then fails with -1024MB is not enough for 0MB. Anyone seen this? | 19:26 |
| *** ihrachys has quit IRC | 19:28 | |
| *** baoli has joined #openstack-nova | 19:28 | |
| *** ihrachys has joined #openstack-nova | 19:29 | |
| *** shaohe_feng has quit IRC | 19:31 | |
| BobBall | claudiub / dansmith: When setting up the Xen CI I was asked to avoid the global namespace: http://lists.openstack.org/pipermail/openstack-dev/2014-July/041238.html based on that we updated the template puppet scripts https://review.openstack.org/#/c/278537 | 19:31 |
| cdent | jaypipes, dansmith, bauzas: do we need to reinstate (and fix for move to api database) this inventory change: https://review.openstack.org/#/c/316031/ I would think "yes" because get_all_by_resource_provider_uuid() went out in early Feb so even though it isn't our preferred interface, it exists. The fix is straightforward (and already stashed in my git). | 19:31 |
| cdent | However, I don't seem to have the power to unabandon that change. | 19:31 |
| *** shaohe_feng has joined #openstack-nova | 19:31 | |
| BobBall | claudiub: Not documented clearly of course, but hopefully future CIs will just follow the template puppet scripts which have now been updated | 19:32 |
| BobBall | claudiub: It's clearly not an urgent issue, but it's quite straight forward to add a second recheck syntax and trigger on both for a while so it should be easy to transition | 19:33 |
| jaypipes | cdent: that patch was abandoned because the revert of the cell DB inventories table joins fixed the bug that patch was intending to fix. | 19:33 |
| Swanson | Okay, I'm guessing maybe I should open a bug. | 19:33 |
| cdent | the query that is done for that method is incorrect | 19:33 |
| *** lucas___ has joined #openstack-nova | 19:34 | |
| bauzas | jaypipes: because it was not really a git revert, right ? :p | 19:34 |
| jaypipes | bauzas: correct. | 19:35 |
| cdent | jaypipes: so I'm assuming we don't want an incorrect query lying around | 19:35 |
| cdent | even if the method is obsolte | 19:35 |
| *** permalac has quit IRC | 19:36 | |
| jaypipes | cdent: ah, I got you now. | 19:36 |
| jaypipes | cdent: yeah, feel free to push a patch that corrects that particular query. | 19:37 |
| *** zenoway has joined #openstack-nova | 19:37 | |
| cdent | jaypipes: can you unabandon the existing fix so I can just doctor that? | 19:37 |
| dansmith | cdent: I unabandoned for you | 19:37 |
| jaypipes | cdent: that particular patch was still operating on the assumption that the models were in the cell DB, not the API db. | 19:37 |
| cdent | dansmith: thanks | 19:37 |
| dansmith | jaypipes: he said that :) | 19:37 |
| dansmith | jaypipes: er, he said "fix that in the process" | 19:37 |
| jaypipes | yeah, sorry... too many balls up in the air. | 19:38 |
| cdent | we got there in the end | 19:38 |
| *** shaohe_feng has quit IRC | 19:41 | |
| *** shaohe_feng has joined #openstack-nova | 19:41 | |
| *** zenoway has quit IRC | 19:41 | |
| *** mkrcmari__ has joined #openstack-nova | 19:44 | |
| *** rk4n has quit IRC | 19:44 | |
| *** yassine has quit IRC | 19:45 | |
| claudiub | BobBall: I see. Sure, we can do that. Though, it would be nice to have the format documented somewhere, like here http://docs.openstack.org/infra/system-config/third_party.html#requirements , so newer CIs will use the same format. | 19:45 |
| BobBall | claudiub: Agreed; I'll see if I can find where to change that tomorrow. | 19:46 |
| claudiub | BobBall: great, thanks. :) | 19:46 |
| *** mvk_ has quit IRC | 19:47 | |
| *** shaohe_feng has quit IRC | 19:51 | |
| *** morgabra_ is now known as morgabra | 19:52 | |
| *** morgabra has joined #openstack-nova | 19:52 | |
| *** lucas___ has quit IRC | 19:52 | |
| *** shaohe_feng has joined #openstack-nova | 19:52 | |
| *** lucas___ has joined #openstack-nova | 19:55 | |
| *** lucas___ has quit IRC | 19:57 | |
| mriedem | doesn't fuel upgrade from release to release? fuel doesn't have a kilo/7.0 release? https://review.openstack.org/#/c/326446/ | 19:58 |
| *** rlrossit has quit IRC | 19:58 | |
| *** lucas___ has joined #openstack-nova | 19:59 | |
| *** raunak has joined #openstack-nova | 19:59 | |
| *** mvk_ has joined #openstack-nova | 20:01 | |
| *** shaohe_feng has quit IRC | 20:02 | |
| *** Sukhdev has quit IRC | 20:02 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Added support for new block device format in Hyper-V https://review.openstack.org/246298 | 20:03 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Added support for new block device format in vmops https://review.openstack.org/246299 | 20:03 |
| *** clenimar has joined #openstack-nova | 20:04 | |
| *** jwcroppe has joined #openstack-nova | 20:04 | |
| *** mkrcmari__ has quit IRC | 20:05 | |
| openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref, os-services.inc https://review.openstack.org/321123 | 20:06 |
| *** lucas___ has quit IRC | 20:06 | |
| *** shaohe_feng has joined #openstack-nova | 20:07 | |
| *** lucas___ has joined #openstack-nova | 20:10 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Fix update inventory for multiple providers https://review.openstack.org/316031 | 20:11 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add name and generation to ResourceProvider object https://review.openstack.org/281945 | 20:11 |
| cdent | bauzas: I accidently blew your +W off https://review.openstack.org/#/c/281945/ with a rebase that I didn't realize was going to do that, could you reapprove please? | 20:11 |
| *** baoli has quit IRC | 20:12 | |
| *** shaohe_feng has quit IRC | 20:12 | |
| *** baoli has joined #openstack-nova | 20:12 | |
| *** shaohe_feng has joined #openstack-nova | 20:13 | |
| *** timello has quit IRC | 20:14 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Improve help text for compute db_opts https://review.openstack.org/326718 | 20:15 |
| cdent | jaypipes: could you pull your -2 off https://review.openstack.org/#/c/316031/ please? | 20:17 |
| *** ihrachys has quit IRC | 20:17 | |
| *** zenoway has joined #openstack-nova | 20:18 | |
| *** ihrachys has joined #openstack-nova | 20:19 | |
| *** timello has joined #openstack-nova | 20:20 | |
| mriedem | dansmith: feelings on this for stable/liberty? it doesn't fix the bug, just makes it fail faster, so i'm inclined to -1/-2 it for liberty given the support phase we're in https://review.openstack.org/#/c/307219/ | 20:20 |
| jaypipes | cdent: done | 20:20 |
| cdent | thanks muchly | 20:21 |
| *** lucas___ has quit IRC | 20:21 | |
| *** lucas___ has joined #openstack-nova | 20:22 | |
| dansmith | mriedem: yeah, I think it's fine to reject it because it's not a critical fix | 20:22 |
| *** shaohe_feng has quit IRC | 20:22 | |
| *** shaohe_feng has joined #openstack-nova | 20:23 | |
| *** Apoorva has quit IRC | 20:28 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Delete old imagebackend methods & tests https://review.openstack.org/322974 | 20:31 |
| *** mpavone has joined #openstack-nova | 20:32 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Remove the last bits of mox from test_imagebackend https://review.openstack.org/323038 | 20:32 |
| *** shaohe_feng has quit IRC | 20:32 | |
| *** mpavone has quit IRC | 20:33 | |
| *** BjoernT has quit IRC | 20:33 | |
| *** shaohe_feng has joined #openstack-nova | 20:33 | |
| *** eharney has quit IRC | 20:33 | |
| *** xyang1 has quit IRC | 20:36 | |
| *** claudiub has quit IRC | 20:37 | |
| mriedem | tonyb: fyi, waiting on 2 changes in the gate queue to merge for stable/liberty then i'll request that release | 20:39 |
| *** burt has quit IRC | 20:39 | |
| *** burt has joined #openstack-nova | 20:40 | |
| *** lucas___ has quit IRC | 20:40 | |
| *** lucas___ has joined #openstack-nova | 20:40 | |
| *** lucas___ has quit IRC | 20:40 | |
| *** xyang1 has joined #openstack-nova | 20:41 | |
| *** rk4n has joined #openstack-nova | 20:42 | |
| *** shaohe_feng has quit IRC | 20:43 | |
| *** cleong has quit IRC | 20:43 | |
| *** shaohe_feng has joined #openstack-nova | 20:43 | |
| *** sameo_afk has quit IRC | 20:44 | |
| *** ayoung has quit IRC | 20:44 | |
| *** ihrachys has quit IRC | 20:48 | |
| *** Apoorva has joined #openstack-nova | 20:48 | |
| *** adisky has quit IRC | 20:50 | |
| *** baoli has quit IRC | 20:52 | |
| *** baoli has joined #openstack-nova | 20:52 | |
| *** sameo has joined #openstack-nova | 20:53 | |
| *** shaohe_feng has quit IRC | 20:53 | |
| *** gjayavelu has joined #openstack-nova | 20:53 | |
| *** lucas___ has joined #openstack-nova | 20:53 | |
| *** shaohe_feng has joined #openstack-nova | 20:53 | |
| *** jaypipes has quit IRC | 20:53 | |
| *** kylek3h has quit IRC | 20:54 | |
| *** lucas____ has joined #openstack-nova | 20:54 | |
| *** jwcroppe has quit IRC | 20:54 | |
| *** lucas____ has quit IRC | 20:55 | |
| *** lucas has joined #openstack-nova | 20:56 | |
| *** baoli has quit IRC | 20:56 | |
| *** baoli has joined #openstack-nova | 20:56 | |
| *** lucas___ has quit IRC | 20:58 | |
| andrewbogott | I have a (recently-working) nova-compute service that is hanging inside virt.libvirt._get_filter_uuid. Is that a familiar problem to anyone? | 20:59 |
| *** aysyd has quit IRC | 20:59 | |
| *** jichen has quit IRC | 21:00 | |
| *** ihrachys has joined #openstack-nova | 21:00 | |
| *** syjulian has quit IRC | 21:00 | |
| *** lucas has quit IRC | 21:00 | |
| *** baoli has quit IRC | 21:01 | |
| andrewbogott | It's calling nwfilterLookupByName('nova-no-nd-reflection') | 21:01 |
| *** thumpba has quit IRC | 21:01 | |
| *** baoli has joined #openstack-nova | 21:01 | |
| openstackgerrit | Emile Snyder proposed openstack/nova-specs: Attach/detach instance access from Manila shares https://review.openstack.org/310050 | 21:02 |
| *** lucas has joined #openstack-nova | 21:02 | |
| *** clenimar has quit IRC | 21:02 | |
| sdague | andrewbogott: your libvirt lock up? that looks like the only thing that could cause a hang there | 21:02 |
| andrewbogott | sdague: Yeah, that's what it seems like... | 21:03 |
| andrewbogott | How would I kick libvirt? I've tried restarting libvirt-bin | 21:03 |
| andrewbogott | but maybe there are lock files &c? | 21:03 |
| *** shaohe_feng has quit IRC | 21:03 | |
| *** mfedosin has quit IRC | 21:03 | |
| *** doug-fish has quit IRC | 21:03 | |
| sdague | you probably have to restart the nova-compute as well, right? | 21:03 |
| sdague | *shrug* I'm not sure | 21:03 |
| andrewbogott | yes — the way I'm reproducing the error is by restarting nova-compute… it connects to rabbit and then gets down to checking in with the first instance and hangs. | 21:04 |
| *** shaohe_feng has joined #openstack-nova | 21:04 | |
| *** doug-fish has joined #openstack-nova | 21:04 | |
| mriedem | JayF: on your stable/mitaka question, try hitting up mikal when he's around | 21:04 |
| JayF | mriedem: thanks! I will, he's the one who initially suggested I backport it | 21:04 |
| *** ayoung has joined #openstack-nova | 21:04 | |
| JayF | mriedem: and impressive async response capabilities, my cache timed out hours ago :P | 21:05 |
| *** doug-fish has quit IRC | 21:05 | |
| *** doug-fish has joined #openstack-nova | 21:05 | |
| *** lucas has quit IRC | 21:06 | |
| mriedem | dansmith: sdague: mikal: tonyb: at your convenience, would be good to get this top patch approved for mitaka before approving the bottom patch - the bottom introduced a bug fixed in the top, which is why i nacked the bottom one until the top is approved https://review.openstack.org/#/c/312312/ | 21:09 |
| mriedem | but the bug fixed in the original is something we should get into mitaka | 21:09 |
| *** r-daneel has quit IRC | 21:09 | |
| *** diogogmt has joined #openstack-nova | 21:12 | |
| *** thorst has quit IRC | 21:12 | |
| andrewbogott | sdague: indeed, 'virsh list' seems to hang forever. So, nova is probably blameless | 21:12 |
| *** yamahata has quit IRC | 21:13 | |
| andrewbogott | hm, nope, there it goes :( | 21:13 |
| *** thorst has joined #openstack-nova | 21:13 | |
| *** shaohe_feng has quit IRC | 21:13 | |
| *** shaohe_feng has joined #openstack-nova | 21:14 | |
| *** yamahata has joined #openstack-nova | 21:15 | |
| *** liunian has quit IRC | 21:17 | |
| *** thorst has quit IRC | 21:17 | |
| *** lpetrut has quit IRC | 21:17 | |
| *** claudiub has joined #openstack-nova | 21:17 | |
| *** ihrachys has quit IRC | 21:18 | |
| *** liunian has joined #openstack-nova | 21:19 | |
| mriedem | belliott: do you care about this on stable/mitaka? https://review.openstack.org/#/c/317662/ | 21:19 |
| *** diogogmt has quit IRC | 21:19 | |
| *** ihrachys has joined #openstack-nova | 21:20 | |
| mriedem | lyarwood_: ^ maybe you want to clean that up? | 21:20 |
| *** diogogmt has joined #openstack-nova | 21:22 | |
| *** shaohe_feng has quit IRC | 21:24 | |
| *** shaohe_feng has joined #openstack-nova | 21:24 | |
| lyarwood_ | mriedem: sure thing | 21:25 |
| *** burt has quit IRC | 21:26 | |
| lyarwood_ | mriedem: btw you and the other stable-maint members for nova might find https://review.openstack.org/#/c/322273/ from ihar interesting | 21:26 |
| *** amotoki has joined #openstack-nova | 21:27 | |
| *** burt has joined #openstack-nova | 21:27 | |
| *** r-daneel has joined #openstack-nova | 21:30 | |
| *** ihrachys has quit IRC | 21:31 | |
| mriedem | yeah it came up in the stable meeting yesterday | 21:31 |
| mriedem | however, i have less and less time for stable thingies | 21:31 |
| *** ihrachys has joined #openstack-nova | 21:31 | |
| *** ijw has quit IRC | 21:32 | |
| *** ijw has joined #openstack-nova | 21:32 | |
| *** mvk has joined #openstack-nova | 21:33 | |
| *** amotoki has quit IRC | 21:33 | |
| *** ijw has quit IRC | 21:33 | |
| *** rtheis has quit IRC | 21:34 | |
| *** shaohe_feng has quit IRC | 21:34 | |
| *** mkrcmari__ has joined #openstack-nova | 21:34 | |
| *** doug-fish has quit IRC | 21:34 | |
| *** shaohe_feng has joined #openstack-nova | 21:35 | |
| *** mkrcmari__ has quit IRC | 21:35 | |
| *** r-daneel has quit IRC | 21:35 | |
| *** mvk_ has quit IRC | 21:35 | |
| *** mriedem has quit IRC | 21:36 | |
| *** mvk has quit IRC | 21:37 | |
| *** thorst has joined #openstack-nova | 21:38 | |
| *** doug-fish has joined #openstack-nova | 21:40 | |
| *** rk4n has quit IRC | 21:41 | |
| *** thorst has quit IRC | 21:42 | |
| *** shaohe_feng has quit IRC | 21:44 | |
| *** doug-fis_ has joined #openstack-nova | 21:46 | |
| *** ctrath has quit IRC | 21:47 | |
| *** shaohe_feng has joined #openstack-nova | 21:47 | |
| *** ihrachys has quit IRC | 21:48 | |
| *** r-daneel has joined #openstack-nova | 21:48 | |
| *** rk4n has joined #openstack-nova | 21:49 | |
| *** doug-fish has quit IRC | 21:50 | |
| *** doug-fis_ has quit IRC | 21:51 | |
| *** rk4n has quit IRC | 21:52 | |
| *** cdent has quit IRC | 21:53 | |
| *** ayoung has quit IRC | 21:54 | |
| *** Sukhdev has joined #openstack-nova | 21:54 | |
| *** shaohe_feng has quit IRC | 21:54 | |
| *** shaohe_feng has joined #openstack-nova | 21:55 | |
| *** Sukhdev has quit IRC | 21:57 | |
| *** ctrath has joined #openstack-nova | 21:58 | |
| *** ihrachys has joined #openstack-nova | 21:59 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: enhance method to return pointer_model from image prop https://review.openstack.org/174854 | 22:00 |
| *** sahid has quit IRC | 22:00 | |
| *** sdake_ has joined #openstack-nova | 22:03 | |
| *** timello has quit IRC | 22:03 | |
| *** mmandel has quit IRC | 22:04 | |
| *** doug-fish has joined #openstack-nova | 22:04 | |
| *** shaohe_feng has quit IRC | 22:05 | |
| *** Sukhdev has joined #openstack-nova | 22:05 | |
| *** shaohe_feng has joined #openstack-nova | 22:05 | |
| *** sdake has quit IRC | 22:06 | |
| *** sdake has joined #openstack-nova | 22:06 | |
| *** ayoung has joined #openstack-nova | 22:08 | |
| *** doug-fish has quit IRC | 22:09 | |
| *** sdake_ has quit IRC | 22:10 | |
| *** Sukhdev has quit IRC | 22:12 | |
| *** Sukhdev has joined #openstack-nova | 22:12 | |
| *** shaohe_feng has quit IRC | 22:15 | |
| *** shaohe_feng has joined #openstack-nova | 22:15 | |
| *** kfarr has quit IRC | 22:15 | |
| *** burt has quit IRC | 22:15 | |
| *** Sukhdev has quit IRC | 22:16 | |
| *** Sukhdev has joined #openstack-nova | 22:16 | |
| *** r-daneel has quit IRC | 22:17 | |
| *** ijw has joined #openstack-nova | 22:19 | |
| *** ivase has quit IRC | 22:19 | |
| *** ihrachys has quit IRC | 22:19 | |
| *** ihrachys has joined #openstack-nova | 22:20 | |
| *** ijw has quit IRC | 22:21 | |
| *** Apoorva_ has joined #openstack-nova | 22:21 | |
| *** diogogmt has quit IRC | 22:22 | |
| *** ijw_ has joined #openstack-nova | 22:23 | |
| *** ivase has joined #openstack-nova | 22:23 | |
| *** ducttape_ has quit IRC | 22:24 | |
| *** salv-orl_ has joined #openstack-nova | 22:24 | |
| *** Apoorva has quit IRC | 22:25 | |
| *** shaohe_feng has quit IRC | 22:25 | |
| *** shaohe_feng has joined #openstack-nova | 22:26 | |
| *** BjoernT has joined #openstack-nova | 22:26 | |
| *** Apoorva_ has quit IRC | 22:26 | |
| *** salv-orlando has quit IRC | 22:27 | |
| *** ijw_ has quit IRC | 22:27 | |
| *** ivase has quit IRC | 22:28 | |
| *** fawadkhaliq has joined #openstack-nova | 22:28 | |
| *** Apoorva has joined #openstack-nova | 22:29 | |
| *** ijw has joined #openstack-nova | 22:29 | |
| *** mriedem has joined #openstack-nova | 22:29 | |
| *** amotoki has joined #openstack-nova | 22:30 | |
| *** ametts has quit IRC | 22:30 | |
| melwitt | I would appreciate a sanity check on a review dealing with 'delete/local delete/compute delete' flow considering instance.host changing underneath us https://review.openstack.org/#/c/256059/12/nova/compute/api.py | 22:31 |
| *** ihrachys has quit IRC | 22:32 | |
| *** mrkz has quit IRC | 22:33 | |
| *** efried has quit IRC | 22:34 | |
| *** amotoki has quit IRC | 22:34 | |
| *** shaohe_feng has quit IRC | 22:35 | |
| *** shaohe_feng has joined #openstack-nova | 22:36 | |
| *** ihrachys has joined #openstack-nova | 22:38 | |
| *** mdrabe has quit IRC | 22:41 | |
| *** sdake has quit IRC | 22:43 | |
| mriedem | sdague: in api-ref, server networks are not required, but with get-me-a-network and v2.31 (current microversion), they are, so not sure how to handle the 'required' attribute for that parameter in api-ref. is that best to just document in the description? | 22:44 |
| *** shaohe_feng has quit IRC | 22:46 | |
| *** shaohe_feng has joined #openstack-nova | 22:47 | |
| *** edmondsw has quit IRC | 22:47 | |
| *** stewie925 has quit IRC | 22:48 | |
| *** liunian has quit IRC | 22:50 | |
| *** liunian has joined #openstack-nova | 22:55 | |
| *** shaohe_feng has quit IRC | 22:56 | |
| alaski | melwitt: added to my queue but I likely won't get to it until tomorrow | 22:56 |
| melwitt | alaski: awesome, thanks! | 22:57 |
| *** ctrath has quit IRC | 22:57 | |
| *** dimtruck is now known as zz_dimtruck | 22:59 | |
| *** shaohe_feng has joined #openstack-nova | 22:59 | |
| *** ihrachys has quit IRC | 23:05 | |
| *** rahuls has quit IRC | 23:05 | |
| *** Sukhdev has quit IRC | 23:05 | |
| *** Sukhdev has joined #openstack-nova | 23:05 | |
| *** shaohe_feng has quit IRC | 23:06 | |
| *** baoli has quit IRC | 23:07 | |
| *** shaohe_feng has joined #openstack-nova | 23:07 | |
| *** IanGovett has joined #openstack-nova | 23:08 | |
| *** Sukhdev has quit IRC | 23:09 | |
| *** rahuls has joined #openstack-nova | 23:09 | |
| *** sdake has joined #openstack-nova | 23:12 | |
| *** markvoelker has quit IRC | 23:13 | |
| *** tjones has left #openstack-nova | 23:14 | |
| *** sacharya_ has joined #openstack-nova | 23:15 | |
| *** sacharya has quit IRC | 23:15 | |
| *** rahuls has quit IRC | 23:15 | |
| *** BjoernT has quit IRC | 23:16 | |
| *** sdague has quit IRC | 23:16 | |
| *** shaohe_feng has quit IRC | 23:16 | |
| *** shaohe_feng has joined #openstack-nova | 23:17 | |
| *** ihrachys has joined #openstack-nova | 23:18 | |
| *** Sukhdev has joined #openstack-nova | 23:19 | |
| *** xyang1 has quit IRC | 23:19 | |
| *** ijw has quit IRC | 23:21 | |
| *** ijw has joined #openstack-nova | 23:22 | |
| *** lucas has joined #openstack-nova | 23:23 | |
| *** lucas has quit IRC | 23:23 | |
| *** lucas has joined #openstack-nova | 23:24 | |
| *** zenoway has quit IRC | 23:25 | |
| *** sacharya_ has quit IRC | 23:25 | |
| *** rahuls has joined #openstack-nova | 23:26 | |
| *** shaohe_feng has quit IRC | 23:27 | |
| *** shaohe_feng has joined #openstack-nova | 23:28 | |
| *** lucas has quit IRC | 23:28 | |
| *** ducttape_ has joined #openstack-nova | 23:29 | |
| *** sameo has quit IRC | 23:29 | |
| *** lucas has joined #openstack-nova | 23:30 | |
| *** Sukhdev has quit IRC | 23:30 | |
| *** amotoki has joined #openstack-nova | 23:30 | |
| *** Sukhdev has joined #openstack-nova | 23:32 | |
| *** lucas___ has joined #openstack-nova | 23:32 | |
| *** amotoki has quit IRC | 23:35 | |
| *** lucas has quit IRC | 23:35 | |
| *** mvk has joined #openstack-nova | 23:36 | |
| *** ihrachys has quit IRC | 23:36 | |
| *** zenoway has joined #openstack-nova | 23:37 | |
| *** lucas___ has quit IRC | 23:37 | |
| *** shaohe_feng has quit IRC | 23:37 | |
| *** shaohe_feng has joined #openstack-nova | 23:37 | |
| *** fawadkhaliq has quit IRC | 23:37 | |
| *** ihrachys has joined #openstack-nova | 23:38 | |
| *** IanGovett has quit IRC | 23:38 | |
| *** IanGovett has joined #openstack-nova | 23:40 | |
| *** Sukhdev has quit IRC | 23:40 | |
| *** Sukhdev has joined #openstack-nova | 23:40 | |
| *** zenoway has quit IRC | 23:41 | |
| *** MikeG451 has quit IRC | 23:41 | |
| *** salv-orl_ has quit IRC | 23:43 | |
| *** salv-orlando has joined #openstack-nova | 23:43 | |
| *** IanGovett has quit IRC | 23:45 | |
| *** sdake has quit IRC | 23:46 | |
| *** shaohe_feng has quit IRC | 23:47 | |
| *** shaohe_feng has joined #openstack-nova | 23:48 | |
| *** clenimar has joined #openstack-nova | 23:49 | |
| *** p01nt3r75 has joined #openstack-nova | 23:50 | |
| *** renato_r_ has quit IRC | 23:51 | |
| *** fawadkhaliq has joined #openstack-nova | 23:52 | |
| *** ducttape_ has quit IRC | 23:53 | |
| *** ducttape_ has joined #openstack-nova | 23:56 | |
| *** thorst has joined #openstack-nova | 23:56 | |
| *** fawadkhaliq has quit IRC | 23:56 | |
| *** thorst has quit IRC | 23:56 | |
| *** thorst has joined #openstack-nova | 23:57 | |
| *** shaohe_feng has quit IRC | 23:57 | |
| *** shaohe_feng has joined #openstack-nova | 23:58 | |
| openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref: os-volumes.inc https://review.openstack.org/320048 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!