| *** ivase has joined #openstack-nova | 00:02 | |
| *** _ducttape_ has quit IRC | 00:03 | |
| *** Kennan_Vacation is now known as Kennan | 00:03 | |
| *** dims has quit IRC | 00:04 | |
| *** ndipanov has quit IRC | 00:04 | |
| *** mylu has quit IRC | 00:06 | |
| *** su_zhang has joined #openstack-nova | 00:07 | |
| *** ivase has quit IRC | 00:07 | |
| *** dims has joined #openstack-nova | 00:11 | |
| *** thorst has joined #openstack-nova | 00:11 | |
| *** mylu has joined #openstack-nova | 00:13 | |
| *** thorst_ has joined #openstack-nova | 00:14 | |
| *** mylu has quit IRC | 00:15 | |
| *** mrkz has quit IRC | 00:17 | |
| *** thorst has quit IRC | 00:17 | |
| *** mylu has joined #openstack-nova | 00:22 | |
| *** pradk has quit IRC | 00:30 | |
| *** EinstCra_ has quit IRC | 00:31 | |
| *** su_zhang has quit IRC | 00:32 | |
| *** markvoelker has joined #openstack-nova | 00:35 | |
| *** salv-orl_ has quit IRC | 00:38 | |
| *** markvoelker has quit IRC | 00:39 | |
| *** ducttape_ has joined #openstack-nova | 00:40 | |
| *** ducttape_ has quit IRC | 00:45 | |
| *** su_zhang has joined #openstack-nova | 00:47 | |
| *** aginwala has quit IRC | 00:51 | |
| *** rfolco has quit IRC | 00:58 | |
| *** thorst_ has quit IRC | 00:59 | |
| *** thorst has joined #openstack-nova | 01:00 | |
| *** ZZelle_ has quit IRC | 01:01 | |
| *** ijw has joined #openstack-nova | 01:02 | |
| *** dims has quit IRC | 01:04 | |
| *** mtanino has quit IRC | 01:06 | |
| *** dims has joined #openstack-nova | 01:08 | |
| *** thorst has quit IRC | 01:08 | |
| *** achanda has quit IRC | 01:13 | |
| *** aginwala has joined #openstack-nova | 01:17 | |
| *** achanda has joined #openstack-nova | 01:17 | |
| *** mylu has quit IRC | 01:17 | |
| *** rotbeard has joined #openstack-nova | 01:19 | |
| *** nelsnelson has joined #openstack-nova | 01:20 | |
| *** achanda has quit IRC | 01:20 | |
| *** david-lyle has joined #openstack-nova | 01:23 | |
| *** ducttape_ has joined #openstack-nova | 01:29 | |
| *** yonglihe has joined #openstack-nova | 01:36 | |
| *** jaypipes has quit IRC | 01:36 | |
| *** eil397 has quit IRC | 01:39 | |
| *** jacky-zhang has joined #openstack-nova | 01:39 | |
| *** haomaiwa_ has joined #openstack-nova | 01:42 | |
| *** ducttape_ has quit IRC | 01:44 | |
| *** ducttape_ has joined #openstack-nova | 01:47 | |
| *** mylu has joined #openstack-nova | 01:48 | |
| *** ivase has joined #openstack-nova | 01:48 | |
| *** ivase has quit IRC | 01:52 | |
| *** rotbeard has quit IRC | 01:55 | |
| *** edmondsw_ has quit IRC | 01:56 | |
| *** rotbeard has joined #openstack-nova | 01:59 | |
| *** zz_dimtruck is now known as dimtruck | 02:00 | |
| *** haomaiwa_ has quit IRC | 02:01 | |
| *** haomaiwang has joined #openstack-nova | 02:01 | |
| *** gcb has joined #openstack-nova | 02:03 | |
| *** thorst has joined #openstack-nova | 02:07 | |
| *** thorst has quit IRC | 02:14 | |
| *** bkero has joined #openstack-nova | 02:16 | |
| bkero | Hey folks. Out of curiosity did something change with the import of 'oslo.middlware'? I'm trying to run the puppet integration tests on centos7 and am getting this in nova-api.log: "ERROR nova ImportError: No module named oslo.middleware" | 02:17 |
|---|---|---|
| *** ganso has quit IRC | 02:19 | |
| *** krtaylor has quit IRC | 02:20 | |
| bkero | python2-oslo-middlware is installed | 02:25 |
| *** su_zhang has quit IRC | 02:26 | |
| *** armax has joined #openstack-nova | 02:26 | |
| *** su_zhang has joined #openstack-nova | 02:26 | |
| dims | bkero : check your api-paste.ini... the package name in the library is oslo_middleware | 02:26 |
| *** houming has joined #openstack-nova | 02:26 | |
| bkero | im[root@localhost nova]# cat api-paste.ini |grep middle | 02:26 |
| bkero | paste.filter_factory = oslo.middleware:RequestId.factory | 02:26 |
| bkero | dims: ^ | 02:26 |
| dims | yep, edit that line | 02:27 |
| bkero | I wonder why puppet generated that line :/ | 02:27 |
| bkero | Doesn't really help if I fix it one time, need to find out why that file said it in the first place :) | 02:28 |
| *** ducttape_ has quit IRC | 02:30 | |
| *** krtaylor has joined #openstack-nova | 02:31 | |
| *** markvoelker has joined #openstack-nova | 02:36 | |
| greghaynes | bkero: are you deploying a stable release config with latest deps? | 02:37 |
| bkero | greghaynes: I think I had some additional deps and this wasn't the clean image I thought it was. | 02:37 |
| bkero | So I'm going to solve that first | 02:38 |
| bkero | (delorean kilo repo) | 02:39 |
| *** markvoelker has quit IRC | 02:40 | |
| *** su_zhang has quit IRC | 02:43 | |
| *** mylu has quit IRC | 02:44 | |
| *** mylu has joined #openstack-nova | 02:48 | |
| *** mylu has quit IRC | 02:52 | |
| *** mylu has joined #openstack-nova | 02:55 | |
| *** nic has quit IRC | 03:00 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-nova | 03:01 | |
| *** armax has quit IRC | 03:03 | |
| *** ducttape_ has joined #openstack-nova | 03:09 | |
| *** thorst has joined #openstack-nova | 03:12 | |
| *** gongysh has joined #openstack-nova | 03:14 | |
| *** salv-orlando has joined #openstack-nova | 03:16 | |
| *** neophy has joined #openstack-nova | 03:17 | |
| *** thorst has quit IRC | 03:18 | |
| *** aginwala has quit IRC | 03:19 | |
| *** Marga__ has joined #openstack-nova | 03:20 | |
| *** ducttape_ has quit IRC | 03:23 | |
| *** Marga_ has quit IRC | 03:23 | |
| *** Marga__ has quit IRC | 03:25 | |
| *** ducttape_ has joined #openstack-nova | 03:28 | |
| *** salv-orlando has quit IRC | 03:28 | |
| *** ccard__ has joined #openstack-nova | 03:31 | |
| *** ijw has quit IRC | 03:33 | |
| *** ccard_ has quit IRC | 03:34 | |
| *** ccard_ has joined #openstack-nova | 03:35 | |
| *** ccard__ has quit IRC | 03:36 | |
| *** dims has quit IRC | 03:37 | |
| *** aginwala has joined #openstack-nova | 03:38 | |
| *** jacky-zhang has quit IRC | 03:44 | |
| *** cfriesen has joined #openstack-nova | 03:46 | |
| *** rotbeard has quit IRC | 03:56 | |
| *** amotoki has joined #openstack-nova | 03:56 | |
| *** amotoki has quit IRC | 03:56 | |
| *** irenab_ has joined #openstack-nova | 03:57 | |
| *** ducttape_ has quit IRC | 03:57 | |
| *** jwcroppe has joined #openstack-nova | 03:58 | |
| *** irenab has quit IRC | 03:59 | |
| *** irenab_ is now known as irenab | 03:59 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-nova | 04:01 | |
| *** daemontool has quit IRC | 04:02 | |
| *** david-lyle has quit IRC | 04:03 | |
| *** ccarmack has joined #openstack-nova | 04:15 | |
| *** mylu has quit IRC | 04:16 | |
| *** thorst has joined #openstack-nova | 04:16 | |
| *** mylu has joined #openstack-nova | 04:17 | |
| *** mylu has quit IRC | 04:19 | |
| *** ccarmack has quit IRC | 04:20 | |
| *** dimtruck is now known as zz_dimtruck | 04:23 | |
| *** thorst has quit IRC | 04:24 | |
| *** mylu has joined #openstack-nova | 04:28 | |
| *** mylu has quit IRC | 04:29 | |
| *** mylu has joined #openstack-nova | 04:31 | |
| *** vilobhmm11 has joined #openstack-nova | 04:31 | |
| *** markvoelker has joined #openstack-nova | 04:36 | |
| *** bkumar has joined #openstack-nova | 04:38 | |
| *** sudipto has joined #openstack-nova | 04:38 | |
| *** markvoelker has quit IRC | 04:41 | |
| *** sudipto has quit IRC | 04:42 | |
| *** jacky-zhang has joined #openstack-nova | 04:45 | |
| *** dave-mccowan has quit IRC | 04:47 | |
| *** achanda has joined #openstack-nova | 04:48 | |
| *** jwcroppe has quit IRC | 04:51 | |
| *** jwcroppe has joined #openstack-nova | 04:51 | |
| *** fawadkhaliq has joined #openstack-nova | 04:52 | |
| *** sridharg has joined #openstack-nova | 04:54 | |
| *** oomichi has joined #openstack-nova | 04:55 | |
| *** david-lyle has joined #openstack-nova | 04:55 | |
| *** jwcroppe has quit IRC | 04:56 | |
| *** shyama has joined #openstack-nova | 04:58 | |
| *** edtubill has joined #openstack-nova | 04:59 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-nova | 05:01 | |
| *** ijw has joined #openstack-nova | 05:04 | |
| *** edtubill has quit IRC | 05:08 | |
| *** subscope has joined #openstack-nova | 05:09 | |
| *** thorst has joined #openstack-nova | 05:21 | |
| *** mylu has quit IRC | 05:24 | |
| *** mylu has joined #openstack-nova | 05:25 | |
| *** baoli_ has quit IRC | 05:26 | |
| *** takashin has joined #openstack-nova | 05:29 | |
| *** oomichi has quit IRC | 05:29 | |
| *** thorst has quit IRC | 05:29 | |
| *** irenab has quit IRC | 05:31 | |
| *** ijw has quit IRC | 05:39 | |
| *** mylu has quit IRC | 05:42 | |
| *** mubirru has joined #openstack-nova | 05:44 | |
| *** ankit_ag has joined #openstack-nova | 05:44 | |
| *** jichen has joined #openstack-nova | 05:45 | |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Pass bdm info to _get_instance_disk_info method https://review.openstack.org/221162 | 05:46 |
| *** aagrawal has quit IRC | 05:47 | |
| *** shihanzhang has quit IRC | 05:49 | |
| *** shihanzhang has joined #openstack-nova | 05:49 | |
| *** mylu has joined #openstack-nova | 05:51 | |
| *** mylu has quit IRC | 05:54 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: Validate subs in api samples base class to improve error handling https://review.openstack.org/278221 | 05:55 |
| *** coolsvap|away is now known as coolsvap | 05:56 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwa_ has joined #openstack-nova | 06:01 | |
| *** su_zhang has joined #openstack-nova | 06:03 | |
| *** Sukhdev has joined #openstack-nova | 06:07 | |
| *** sridharg has quit IRC | 06:08 | |
| *** neophy has quit IRC | 06:15 | |
| *** chhavi has joined #openstack-nova | 06:17 | |
| *** ekuris has joined #openstack-nova | 06:19 | |
| *** jaosorior has joined #openstack-nova | 06:20 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: WIP - Is this faster? https://review.openstack.org/280512 | 06:20 |
| *** thorst has joined #openstack-nova | 06:27 | |
| *** zenoway has joined #openstack-nova | 06:28 | |
| *** irenab has joined #openstack-nova | 06:30 | |
| *** vilobhmm11 has quit IRC | 06:33 | |
| *** subscope has quit IRC | 06:34 | |
| *** thorst has quit IRC | 06:34 | |
| *** markvoelker has joined #openstack-nova | 06:37 | |
| *** aginwala has quit IRC | 06:38 | |
| *** achanda has quit IRC | 06:39 | |
| *** david-lyle has quit IRC | 06:39 | |
| *** david-lyle has joined #openstack-nova | 06:40 | |
| *** markvoelker has quit IRC | 06:42 | |
| *** salv-orlando has joined #openstack-nova | 06:43 | |
| *** zzzeek has quit IRC | 06:44 | |
| *** salv-orlando has quit IRC | 06:46 | |
| openstackgerrit | javeme proposed openstack/nova: VMware: Get read_size by FileHandle.get_size() in start_transfer() https://review.openstack.org/280007 | 06:48 |
| *** lpetrut has joined #openstack-nova | 06:50 | |
| openstackgerrit | ShaoHe Feng proposed openstack/nova: add index/show API for server migrations https://review.openstack.org/258771 | 06:51 |
| *** subscope has joined #openstack-nova | 06:52 | |
| *** subscope has quit IRC | 06:55 | |
| *** sridharg has joined #openstack-nova | 06:55 | |
| *** coolsvap is now known as coolsvap|away | 06:58 | |
| *** shyama has quit IRC | 06:58 | |
| *** shyama has joined #openstack-nova | 06:59 | |
| *** achanda has joined #openstack-nova | 06:59 | |
| *** unicell has joined #openstack-nova | 07:00 | |
| *** Marga_ has joined #openstack-nova | 07:00 | |
| *** haomaiwa_ has quit IRC | 07:01 | |
| *** cfriesen has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-nova | 07:01 | |
| *** Marga__ has joined #openstack-nova | 07:02 | |
| *** Marga_ has quit IRC | 07:02 | |
| *** e0ne has joined #openstack-nova | 07:02 | |
| openstackgerrit | ShaoHe Feng proposed openstack/nova: reset task_state after select_destinations failed. https://review.openstack.org/275650 | 07:05 |
| *** Marga__ has quit IRC | 07:06 | |
| *** Marga_ has joined #openstack-nova | 07:06 | |
| *** esracelik has joined #openstack-nova | 07:07 | |
| *** shyama has quit IRC | 07:13 | |
| *** shyama has joined #openstack-nova | 07:13 | |
| *** sahid has joined #openstack-nova | 07:17 | |
| openstackgerrit | Béla Vancsics proposed openstack/nova: Reduce code duplication https://review.openstack.org/270911 | 07:18 |
| openstackgerrit | sahid proposed openstack/nova: virt: reserved hugepages on compute host https://review.openstack.org/277422 | 07:20 |
| *** drankis has joined #openstack-nova | 07:21 | |
| *** dedery has joined #openstack-nova | 07:24 | |
| *** scheuran has joined #openstack-nova | 07:25 | |
| openstackgerrit | John Hua proposed openstack/nova: xenapi: Image cache cannot be disabled https://review.openstack.org/262661 | 07:28 |
| *** kiranr_ has joined #openstack-nova | 07:32 | |
| *** thorst has joined #openstack-nova | 07:32 | |
| *** gongysh has quit IRC | 07:33 | |
| *** kiranr_ is now known as kiran-r | 07:33 | |
| *** mjura has joined #openstack-nova | 07:34 | |
| *** rcernin has joined #openstack-nova | 07:37 | |
| *** thorst has quit IRC | 07:38 | |
| *** clduser_ has quit IRC | 07:43 | |
| *** jbernard has quit IRC | 07:43 | |
| *** clduser has joined #openstack-nova | 07:43 | |
| *** jbernard has joined #openstack-nova | 07:43 | |
| *** ekarlso has quit IRC | 07:44 | |
| *** tjones_zzzz has quit IRC | 07:44 | |
| *** Anticimex has quit IRC | 07:45 | |
| *** liusheng has quit IRC | 07:45 | |
| *** aarefiev has quit IRC | 07:45 | |
| *** jeblair has quit IRC | 07:45 | |
| *** wznoinsk has quit IRC | 07:45 | |
| *** gzhai2 has quit IRC | 07:46 | |
| *** mpavone has joined #openstack-nova | 07:46 | |
| *** aarefiev has joined #openstack-nova | 07:46 | |
| *** wznoinsk has joined #openstack-nova | 07:46 | |
| *** kragniz has quit IRC | 07:46 | |
| *** Sukhdev has quit IRC | 07:46 | |
| *** liusheng has joined #openstack-nova | 07:46 | |
| *** kragniz has joined #openstack-nova | 07:46 | |
| *** Anticimex has joined #openstack-nova | 07:46 | |
| *** jeblair has joined #openstack-nova | 07:46 | |
| *** gzhai2 has joined #openstack-nova | 07:47 | |
| *** tjones has joined #openstack-nova | 07:48 | |
| *** fawadkhaliq has quit IRC | 07:51 | |
| *** fawadkhaliq has joined #openstack-nova | 07:52 | |
| *** fawadkhaliq has quit IRC | 07:52 | |
| *** fawadkhaliq has joined #openstack-nova | 07:52 | |
| *** fawadkhaliq has quit IRC | 07:53 | |
| *** fawadkhaliq has joined #openstack-nova | 07:53 | |
| *** bjornar_ has joined #openstack-nova | 07:54 | |
| *** ekarlso has joined #openstack-nova | 07:57 | |
| *** haomaiwang has quit IRC | 08:01 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: api: discoverable policy https://review.openstack.org/280539 | 08:01 |
| *** haomaiwa_ has joined #openstack-nova | 08:01 | |
| *** mjura has quit IRC | 08:01 | |
| *** jed56 has joined #openstack-nova | 08:08 | |
| *** coolsvap|away is now known as coolsvap | 08:09 | |
| *** lpetrut has quit IRC | 08:10 | |
| *** unicell has quit IRC | 08:11 | |
| *** unicell has joined #openstack-nova | 08:11 | |
| *** unicell has quit IRC | 08:11 | |
| *** ociuhandu has joined #openstack-nova | 08:11 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: numa: Fixes NUMA topology related unit tests https://review.openstack.org/278835 | 08:13 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: libvirt: fixes python 3 related unit tests https://review.openstack.org/278834 | 08:13 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes python 3 unit tests https://review.openstack.org/278810 | 08:13 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes python 3 urllib quote / unquote usage https://review.openstack.org/278809 | 08:13 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes hex decoding related unit tests https://review.openstack.org/278808 | 08:13 |
| *** e0ne has quit IRC | 08:13 | |
| *** mjura has joined #openstack-nova | 08:16 | |
| *** lpetrut has joined #openstack-nova | 08:19 | |
| *** subscope has joined #openstack-nova | 08:21 | |
| *** ociuhandu has quit IRC | 08:21 | |
| *** rdopiera has joined #openstack-nova | 08:25 | |
| *** ihrachys has joined #openstack-nova | 08:25 | |
| *** coolsvap is now known as coolsvap|away | 08:26 | |
| *** lpetrut has quit IRC | 08:26 | |
| *** su_zhang has quit IRC | 08:27 | |
| *** matrohon has joined #openstack-nova | 08:28 | |
| *** jwcroppe has joined #openstack-nova | 08:29 | |
| *** salv-orlando has joined #openstack-nova | 08:30 | |
| *** liverpooler has joined #openstack-nova | 08:30 | |
| *** zakora has joined #openstack-nova | 08:32 | |
| *** salv-orlando has quit IRC | 08:32 | |
| *** oomichi has joined #openstack-nova | 08:33 | |
| *** liverpooler has quit IRC | 08:33 | |
| *** liverpooler has joined #openstack-nova | 08:34 | |
| *** jwcroppe has quit IRC | 08:34 | |
| *** liverpoo1er has joined #openstack-nova | 08:34 | |
| *** roeyc has joined #openstack-nova | 08:35 | |
| *** mfedosin has joined #openstack-nova | 08:37 | |
| *** oomichi has quit IRC | 08:37 | |
| bauzas | good morning Nova | 08:37 |
| *** jlanoux has joined #openstack-nova | 08:38 | |
| openstackgerrit | javeme proposed openstack/nova: VMware: Get read_size by FileHandle.get_size() in start_transfer() https://review.openstack.org/280007 | 08:38 |
| *** thorst has joined #openstack-nova | 08:39 | |
| *** ihrachys has quit IRC | 08:39 | |
| openstackgerrit | javeme proposed openstack/nova: VMware: Refactor fetching all kinds of images https://review.openstack.org/280013 | 08:42 |
| *** jlanoux has quit IRC | 08:42 | |
| *** jlanoux has joined #openstack-nova | 08:44 | |
| *** thorst has quit IRC | 08:44 | |
| *** sudipto has joined #openstack-nova | 08:48 | |
| *** abhishekk_ has joined #openstack-nova | 08:49 | |
| *** subscope has quit IRC | 08:50 | |
| *** jlanoux has quit IRC | 08:59 | |
| *** haomaiwa_ has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| *** lyarwood has joined #openstack-nova | 09:01 | |
| *** jlanoux has joined #openstack-nova | 09:02 | |
| *** achanda has quit IRC | 09:03 | |
| *** ihrachys has joined #openstack-nova | 09:03 | |
| *** subscope has joined #openstack-nova | 09:04 | |
| *** achanda has joined #openstack-nova | 09:06 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/python-novaclient: Support for forcing live migration to complete https://review.openstack.org/279647 | 09:06 |
| *** salv-orlando has joined #openstack-nova | 09:08 | |
| *** kiran-r has quit IRC | 09:08 | |
| *** kiran-r has joined #openstack-nova | 09:09 | |
| *** danpb has joined #openstack-nova | 09:15 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs: Xenapi: a new VDI store via streaming https://review.openstack.org/274045 | 09:16 |
| *** e0ne has joined #openstack-nova | 09:16 | |
| *** yamahata has quit IRC | 09:22 | |
| *** lucas-dinner is now known as lucasagomes | 09:24 | |
| *** klkumar has joined #openstack-nova | 09:25 | |
| *** yassine__ has joined #openstack-nova | 09:25 | |
| *** efoley has joined #openstack-nova | 09:27 | |
| *** mvk has quit IRC | 09:27 | |
| *** achanda has quit IRC | 09:28 | |
| *** ndipanov has joined #openstack-nova | 09:33 | |
| *** danpb1 has joined #openstack-nova | 09:34 | |
| *** sfinucan has joined #openstack-nova | 09:34 | |
| *** danpb has quit IRC | 09:34 | |
| *** mubirru has quit IRC | 09:37 | |
| *** ndipanov has quit IRC | 09:38 | |
| *** ndipanov has joined #openstack-nova | 09:38 | |
| *** ihrachys has quit IRC | 09:39 | |
| *** subscope has quit IRC | 09:40 | |
| dedery | ndipanov: thank you for the review | 09:41 |
| *** thorst has joined #openstack-nova | 09:42 | |
| *** jacky-zhang has quit IRC | 09:42 | |
| *** al_indig_ has joined #openstack-nova | 09:42 | |
| *** lpetrut has joined #openstack-nova | 09:42 | |
| *** ociuhandu has joined #openstack-nova | 09:43 | |
| ndipanov | dedery, no problem - nice fix! | 09:44 |
| al_indig_ | Hello guys! I have multiple compute nodes with single neutron server, release liberty. Each compute node has 4 physical nics. I want to use all of them to have some fault-tolerance&performance improvement. Can't do LACP (there are reasons). What's the best way to do it? | 09:44 |
| openstackgerrit | Tom Patzig proposed openstack/nova: Feature: Add option to disable root_disk in flavor https://review.openstack.org/280301 | 09:44 |
| dedery | ndipanov: thanks :) | 09:46 |
| *** subscope has joined #openstack-nova | 09:47 | |
| dedery | al_indig_: "Can't do LACP" == can't create a linux bond on the compute nodes? | 09:48 |
| openstackgerrit | Pawel Koniszewski proposed openstack/python-novaclient: Support for forcing live migration to complete https://review.openstack.org/279647 | 09:48 |
| *** fawadkhaliq has quit IRC | 09:48 | |
| *** ihrachys has joined #openstack-nova | 09:48 | |
| *** fawadkhaliq has joined #openstack-nova | 09:48 | |
| *** thorst has quit IRC | 09:49 | |
| al_indig_ | dedery: yeap. Nics are assigned to different physical switches (they are blade servers) and Dell hardware has a known bug with using multichassis aggregation groups with large MTU | 09:49 |
| *** ociuhandu has quit IRC | 09:50 | |
| *** fawadkhaliq has quit IRC | 09:50 | |
| al_indig_ | dedery: so I can do two bonded interfaces (by two nics) but that's all. They may fix it, but it can take years :_ | 09:50 |
| *** fawadkhaliq has joined #openstack-nova | 09:50 | |
| dedery | al_indig_: and the "fault-tolerance&performance improvement" requirement is for the purpose of the compute nodes or for the hosted VMs? (or both) | 09:50 |
| *** lpetrut has quit IRC | 09:53 | |
| *** ociuhandu has joined #openstack-nova | 09:53 | |
| *** mvk has joined #openstack-nova | 09:54 | |
| *** fawadkhaliq has quit IRC | 09:54 | |
| openstackgerrit | xhzhf proposed openstack/nova: Add missing os types: suseGuest64/suseGuest https://review.openstack.org/280578 | 09:54 |
| *** fawadkhaliq has joined #openstack-nova | 09:55 | |
| al_indig_ | dedery: for the hosted vms primarily - nics are 10GbE each so I wanted each to each VM talking at 10GbE with using all 4 channels (so I could host more vms with high each-to-each bandwidth) | 09:55 |
| *** chhavi has quit IRC | 09:56 | |
| al_indig_ | dedery: if there is a way to get 40GbE on one VM, that would be good too (but I doubt that it's really possible) | 09:56 |
| dedery | al_indig_: so for the fault aspect you can create a VM level bond between 4 ports (it's a little cumbersome since you'll need to create 4 networks each on a different physical network, their respective subnets and ports) | 09:58 |
| *** ganso has joined #openstack-nova | 09:59 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| al_indig_ | dedery: And can't I for example move all interfaces as ports of br-int in ovs? Shouldn't it work? | 10:01 |
| al_indig_ | dedery: I don't really understand how this part works actually | 10:02 |
| *** salv-orl_ has joined #openstack-nova | 10:02 | |
| *** salv-orlando has quit IRC | 10:05 | |
| *** pras has joined #openstack-nova | 10:07 | |
| *** andymaier has joined #openstack-nova | 10:09 | |
| pras | JOIN vmware-nsx | 10:09 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix an unnecessary interpolation https://review.openstack.org/278186 | 10:10 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add os-interface functional negative tests https://review.openstack.org/276197 | 10:11 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add some validations for BDM https://review.openstack.org/258788 | 10:11 |
| *** haomaiwang has quit IRC | 10:11 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Pass BDM in swap_volume method of LibvirtDriver https://review.openstack.org/257135 | 10:12 |
| dedery | al_indig_: I'm not sure that moving all interfaces as ports of br-int will work. Did you look at lacp using ovs? http://blog.scottlowe.org/2012/10/19/link-aggregation-and-lacp-with-open-vswitch/ | 10:14 |
| al_indig_ | dedery: I keep this option in mind but it seems to be risky: it seems for me that ovs lacp uses the same mechanism as common linux bond => I will see the same bug from dell hardware. Dell FTOS fails to support LACP (even software-based - non 803ad) with non-standard MTU | 10:16 |
| *** paul-carlton1 has joined #openstack-nova | 10:16 | |
| *** marutikamat has joined #openstack-nova | 10:17 | |
| al_indig_ | dedery: and the second question is what bridge will use ovs bond? br-int? | 10:17 |
| *** marutikamat has quit IRC | 10:17 | |
| *** andreykurilin has quit IRC | 10:18 | |
| al_indig_ | dedery: I mean that now it works in some magical way - I don't even see what physical interfaces are used in ovs (here is running config from it: https://gist.github.com/al-indigo/bb662b763dd1ecbadfd4) | 10:19 |
| *** pras has left #openstack-nova | 10:19 | |
| openstackgerrit | javeme proposed openstack/nova: VMware: Refactor fetching all kinds of images https://review.openstack.org/280013 | 10:20 |
| openstackgerrit | Bhagyashri Shewale proposed openstack/nova: Remove duplicate key from dictionary https://review.openstack.org/280592 | 10:21 |
| *** lpetrut has joined #openstack-nova | 10:22 | |
| *** andymaier has quit IRC | 10:23 | |
| al_indig_ | dedery: and only ifconfig traffic statistics makes me think that only the first interface is used | 10:23 |
| *** jlanoux has quit IRC | 10:26 | |
| *** jlanoux has joined #openstack-nova | 10:27 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Finish post live migration even when exception occurs https://review.openstack.org/235994 | 10:30 |
| *** jlanoux has quit IRC | 10:33 | |
| *** andreykurilin has joined #openstack-nova | 10:34 | |
| *** jlanoux has joined #openstack-nova | 10:34 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Use imageutils from oslo.utils https://review.openstack.org/257232 | 10:37 |
| *** dosaboy has quit IRC | 10:38 | |
| *** gongysh has joined #openstack-nova | 10:38 | |
| dedery | al_indig_: going back to adding all of the interfaces to br-int: this is a pre-requisite if you want to enable exactly what i described as the VM level bond. It's the ovs side of defining RegionOne/Two/Three/Four (or phyNet1 etc.. or whatever naming that you want) and have connectivity. You may be able to create the networks using neutron (if you define the bridge mapping correctly e.g. ['RegionOne:br-eth1','RegionTw | 10:41 |
| dedery | o:br-eth2']) but you won't have connectivity unless all are part of br-int | 10:41 |
| *** dosaboy has joined #openstack-nova | 10:41 | |
| dedery | al_indig_: As for ovs lacp - I'm not the expert. never did it, just read about it | 10:42 |
| *** rotbeard has joined #openstack-nova | 10:42 | |
| al_indig_ | dedery: Thanks, I see | 10:44 |
| *** rfolco has joined #openstack-nova | 10:45 | |
| *** thorst has joined #openstack-nova | 10:47 | |
| *** subscope has quit IRC | 10:47 | |
| *** erlon has joined #openstack-nova | 10:48 | |
| *** pece has joined #openstack-nova | 10:50 | |
| *** roeyc has quit IRC | 10:50 | |
| *** dims has joined #openstack-nova | 10:51 | |
| *** thorst has quit IRC | 10:54 | |
| dedery | al_indig_: I see some discussions about "cascading bond devices" (google it) I don't know if it works (never tried it) but it sounds like something that may workaround your problem | 10:55 |
| *** cdent has joined #openstack-nova | 10:56 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [microversions] Add support for 2.19 https://review.openstack.org/278385 | 11:01 |
| *** gszasz has joined #openstack-nova | 11:03 | |
| *** dave-mccowan has joined #openstack-nova | 11:04 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Enhance value check for option notify_on_state_change https://review.openstack.org/242717 | 11:05 |
| *** houming has quit IRC | 11:06 | |
| *** aysyd has joined #openstack-nova | 11:06 | |
| openstackgerrit | Daniel Berrange proposed openstack/os-vif: Provide plugins an oslo_config group for their setup https://review.openstack.org/275281 | 11:09 |
| openstackgerrit | Daniel Berrange proposed openstack/os-vif: vif_plug_ovs: Disable IPv6 on bridge devices https://review.openstack.org/275293 | 11:09 |
| openstackgerrit | Daniel Berrange proposed openstack/os-vif: import openvswitch plugin implementation https://review.openstack.org/273610 | 11:09 |
| openstackgerrit | Daniel Berrange proposed openstack/os-vif: import linux bridge plugin implementation https://review.openstack.org/273609 | 11:09 |
| *** dims has quit IRC | 11:12 | |
| *** bkumar has quit IRC | 11:13 | |
| *** paul-carlton1 has quit IRC | 11:14 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Update ComputeNode values with disk allocation ratios in the RT https://review.openstack.org/277953 | 11:14 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add disk_allocation_ratio to ComputeNode https://review.openstack.org/277934 | 11:14 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Update HostManager and DiskFilter to use ComputeNode disk ratio https://review.openstack.org/278112 | 11:14 |
| *** bkumar has joined #openstack-nova | 11:14 | |
| *** ildikov_ has quit IRC | 11:15 | |
| *** bkumar4 has joined #openstack-nova | 11:17 | |
| *** bkumar4 has quit IRC | 11:17 | |
| *** subscope has joined #openstack-nova | 11:18 | |
| *** bkumar has quit IRC | 11:20 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Catch errors while creating veth pair https://review.openstack.org/264146 | 11:21 |
| *** jwcroppe has joined #openstack-nova | 11:23 | |
| *** dims has joined #openstack-nova | 11:24 | |
| *** sridharg has quit IRC | 11:25 | |
| *** jlanoux has quit IRC | 11:27 | |
| *** sdague has joined #openstack-nova | 11:27 | |
| *** jlanoux has joined #openstack-nova | 11:28 | |
| *** achanda has joined #openstack-nova | 11:28 | |
| *** ociuhandu has quit IRC | 11:29 | |
| *** igordcard_ has joined #openstack-nova | 11:31 | |
| *** igordcard_ has quit IRC | 11:31 | |
| *** igordcard_ has joined #openstack-nova | 11:32 | |
| *** igordcard_ has joined #openstack-nova | 11:33 | |
| *** igordcard_ has quit IRC | 11:33 | |
| *** efoley_ has joined #openstack-nova | 11:33 | |
| *** roeyc has joined #openstack-nova | 11:33 | |
| *** efoley has quit IRC | 11:34 | |
| *** achanda has quit IRC | 11:34 | |
| *** sridharg has joined #openstack-nova | 11:36 | |
| *** rodrigods has quit IRC | 11:38 | |
| *** rodrigods has joined #openstack-nova | 11:38 | |
| al_indig_ | dedery: thanks, it might be a needed trick | 11:38 |
| *** dims has quit IRC | 11:38 | |
| *** haomaiwang has joined #openstack-nova | 11:38 | |
| *** gibi has quit IRC | 11:39 | |
| *** gibi has joined #openstack-nova | 11:39 | |
| *** sdague has quit IRC | 11:39 | |
| *** sdague has joined #openstack-nova | 11:44 | |
| *** ildikov_ has joined #openstack-nova | 11:44 | |
| *** sudipto has quit IRC | 11:44 | |
| *** igordcard_ has joined #openstack-nova | 11:44 | |
| *** ccesario has quit IRC | 11:44 | |
| dedery | al_indig_: hopefully you won't hit the bug... | 11:44 |
| openstackgerrit | Ralf Haferkamp proposed openstack/python-novaclient: Allow to specify a network for functional tests https://review.openstack.org/239971 | 11:44 |
| *** fawadkhaliq has quit IRC | 11:45 | |
| *** takashin has left #openstack-nova | 11:45 | |
| *** fawadkhaliq has joined #openstack-nova | 11:45 | |
| *** fawadkhaliq has quit IRC | 11:46 | |
| *** fawadkhaliq has joined #openstack-nova | 11:46 | |
| *** fawadkhaliq has quit IRC | 11:47 | |
| *** fawadkhaliq has joined #openstack-nova | 11:47 | |
| *** jlanoux has quit IRC | 11:47 | |
| *** fawadkhaliq has quit IRC | 11:47 | |
| *** rodrigods has quit IRC | 11:47 | |
| *** rodrigods has joined #openstack-nova | 11:48 | |
| *** fawadkhaliq has joined #openstack-nova | 11:48 | |
| *** thorst has joined #openstack-nova | 11:52 | |
| *** gmann_ has joined #openstack-nova | 11:53 | |
| *** Daisy has joined #openstack-nova | 11:54 | |
| openstackgerrit | javeme proposed openstack/nova: VMware: Refactor fetching all kinds of images https://review.openstack.org/280013 | 11:55 |
| *** ildikov_ has quit IRC | 11:59 | |
| *** jlanoux has joined #openstack-nova | 11:59 | |
| *** thorst has quit IRC | 11:59 | |
| *** haomaiwang has quit IRC | 12:01 | |
| *** Daisy has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-nova | 12:01 | |
| *** jwcroppe has quit IRC | 12:02 | |
| *** Daisy has joined #openstack-nova | 12:02 | |
| *** jwcroppe has joined #openstack-nova | 12:02 | |
| *** ociuhandu has joined #openstack-nova | 12:04 | |
| *** fawadkhaliq has quit IRC | 12:06 | |
| *** fawadk has joined #openstack-nova | 12:06 | |
| *** jwcroppe has quit IRC | 12:06 | |
| *** efoley__ has joined #openstack-nova | 12:09 | |
| *** efoley_ has quit IRC | 12:09 | |
| *** smatzek has joined #openstack-nova | 12:10 | |
| *** raildo-afk is now known as raildo | 12:11 | |
| *** rqw has joined #openstack-nova | 12:11 | |
| *** fawadk has quit IRC | 12:11 | |
| *** mvk has quit IRC | 12:13 | |
| *** ericksonsantos has joined #openstack-nova | 12:13 | |
| *** gcb has quit IRC | 12:13 | |
| *** gcb has joined #openstack-nova | 12:14 | |
| *** mvk has joined #openstack-nova | 12:15 | |
| *** rqw has quit IRC | 12:16 | |
| *** raildo is now known as raildo-afk | 12:16 | |
| *** rqw has joined #openstack-nova | 12:16 | |
| *** Jeffrey4l has quit IRC | 12:18 | |
| *** pcaruana has joined #openstack-nova | 12:18 | |
| *** raildo-afk is now known as raildo | 12:20 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [microversions] Add support for 2.19 https://review.openstack.org/278385 | 12:21 |
| *** Daisy has quit IRC | 12:22 | |
| *** Daisy has joined #openstack-nova | 12:23 | |
| *** raildo is now known as raildo-afk | 12:25 | |
| *** Daisy has quit IRC | 12:27 | |
| *** efoley__ has quit IRC | 12:28 | |
| *** Jeffrey4l has joined #openstack-nova | 12:31 | |
| *** raildo-afk is now known as raildo | 12:31 | |
| *** achanda has joined #openstack-nova | 12:31 | |
| *** ccesario has joined #openstack-nova | 12:31 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: [dnm] cleanup live-migrations https://review.openstack.org/280653 | 12:34 |
| *** efoley__ has joined #openstack-nova | 12:35 | |
| *** lucasagomes is now known as lucas-hungry | 12:35 | |
| *** rqw has quit IRC | 12:36 | |
| *** ccesario has quit IRC | 12:36 | |
| *** rqw has joined #openstack-nova | 12:36 | |
| *** achanda has quit IRC | 12:37 | |
| *** chhavi has joined #openstack-nova | 12:38 | |
| *** ccarmack has joined #openstack-nova | 12:38 | |
| *** Daisy has joined #openstack-nova | 12:39 | |
| *** gmann_ has quit IRC | 12:39 | |
| *** archers has joined #openstack-nova | 12:41 | |
| *** Daisy has quit IRC | 12:43 | |
| *** archers has quit IRC | 12:43 | |
| *** rqw has quit IRC | 12:44 | |
| *** rqw has joined #openstack-nova | 12:44 | |
| ankit_ag | ndipanov: hi | 12:45 |
| eliqiao_ | hi BobBall johnthetubaguy , can you please check https://review.openstack.org/#/c/254101/20/nova/virt/xenapi/vmops.py, seems that you have opposite suggestion on it. | 12:45 |
| BobBall | Will do | 12:46 |
| BobBall | Ah - it needs johnthetubaguy to review | 12:46 |
| *** kylek3h has joined #openstack-nova | 12:46 | |
| BobBall | I _think_ he's got his comments the wrong way round - so just need him to check and if he still thinks they are the right way round then we need a chat on irc | 12:47 |
| *** paul-carlton1 has joined #openstack-nova | 12:48 | |
| *** baoli has joined #openstack-nova | 12:48 | |
| eliqiao_ | BobBall: yeah, seems johnthetubaguy is not around now, I'v been told he is in ops meetup :( | 12:49 |
| *** thorst has joined #openstack-nova | 12:50 | |
| *** thorst_ has joined #openstack-nova | 12:51 | |
| *** annasort has quit IRC | 12:52 | |
| *** ildikov has joined #openstack-nova | 12:52 | |
| *** baoli has quit IRC | 12:53 | |
| *** mpavone has quit IRC | 12:55 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Update instance host in post live migration even when exception occurs https://review.openstack.org/235994 | 12:55 |
| *** thorst has quit IRC | 12:55 | |
| *** boris-42 has joined #openstack-nova | 12:57 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [microversions] Add support for 2.19 https://review.openstack.org/278385 | 12:57 |
| *** ankit_ag has quit IRC | 12:57 | |
| openstackgerrit | Sean Dague proposed openstack/nova: always use pip constraints https://review.openstack.org/267096 | 12:58 |
| openstackgerrit | Sean Dague proposed openstack/nova: reduce pep8 requirements to just hacking https://review.openstack.org/267097 | 12:58 |
| ndipanov | anyone up for a quick sriov meeting | 12:58 |
| ndipanov | go through bugs and pending patches? | 12:58 |
| *** haomaiwang has quit IRC | 13:01 | |
| *** cfriesen has joined #openstack-nova | 13:01 | |
| *** mjura has quit IRC | 13:01 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: remove default=None for config options https://review.openstack.org/280670 | 13:01 |
| *** haomaiwa_ has joined #openstack-nova | 13:01 | |
| *** mvk has quit IRC | 13:02 | |
| *** mpavone has joined #openstack-nova | 13:02 | |
| *** edmondsw has joined #openstack-nova | 13:03 | |
| *** ihrachys has quit IRC | 13:04 | |
| *** mvk has joined #openstack-nova | 13:05 | |
| *** andrearo1 has joined #openstack-nova | 13:06 | |
| *** ihrachys has joined #openstack-nova | 13:08 | |
| *** ekuris has quit IRC | 13:09 | |
| *** andrearosa has quit IRC | 13:09 | |
| *** cfriesen has quit IRC | 13:10 | |
| *** smatzek_ has joined #openstack-nova | 13:10 | |
| *** ducttape_ has joined #openstack-nova | 13:12 | |
| *** mjura has joined #openstack-nova | 13:14 | |
| *** smatzek has quit IRC | 13:14 | |
| *** doug-fish has joined #openstack-nova | 13:14 | |
| *** ildikov has quit IRC | 13:18 | |
| *** roeyc has quit IRC | 13:20 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: imagecache: set base_dir as a property https://review.openstack.org/209912 | 13:23 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: imagecache: implement listing of cached images https://review.openstack.org/209913 | 13:23 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: virt: add method for obtaining the list of cached images https://review.openstack.org/209914 | 13:23 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: libvirt: implement get_cached_images method https://review.openstack.org/209915 | 13:23 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: compute: track cached images SHA1s https://review.openstack.org/280687 | 13:24 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: scheduler: weight nodes based on their cached images https://review.openstack.org/280688 | 13:24 |
| *** takedakn has joined #openstack-nova | 13:24 | |
| *** ducttape_ has quit IRC | 13:25 | |
| *** lucas-hungry is now known as lucasagomes | 13:27 | |
| *** efoley__ has quit IRC | 13:28 | |
| *** abhishekk_ has quit IRC | 13:28 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: [dnm] cleanup live-migrations https://review.openstack.org/280653 | 13:30 |
| *** salv-orl_ has quit IRC | 13:32 | |
| *** baoli has joined #openstack-nova | 13:32 | |
| johnthetubaguy | eliqiao_: BobBall: in manchester today at the ops meetup, I can try take a look | 13:33 |
| *** efoley__ has joined #openstack-nova | 13:33 | |
| *** doug-fish has quit IRC | 13:34 | |
| *** eglynn has joined #openstack-nova | 13:34 | |
| *** ildikov has joined #openstack-nova | 13:35 | |
| *** thorst_ is now known as thorst | 13:35 | |
| *** salv-orlando has joined #openstack-nova | 13:36 | |
| *** roeyc has joined #openstack-nova | 13:36 | |
| *** markvoelker has joined #openstack-nova | 13:39 | |
| sdague | fyi, this is ready to go now, to get our unit tests running under the same requirements as the devstack jobs - https://review.openstack.org/#/c/267096/ | 13:41 |
| *** EinstCrazy has joined #openstack-nova | 13:42 | |
| *** snikitin has joined #openstack-nova | 13:43 | |
| *** tdurakov has quit IRC | 13:44 | |
| *** tdurakov has joined #openstack-nova | 13:45 | |
| *** Guest70049 has joined #openstack-nova | 13:45 | |
| cdent | sdague: is it possible to set UPPER_CONSTRAINTS_FILE to empty string to get it to not use constraints or does it need to be an empty file? | 13:45 |
| *** dims has joined #openstack-nova | 13:45 | |
| *** ivase has joined #openstack-nova | 13:46 | |
| *** rlrossit has joined #openstack-nova | 13:50 | |
| *** dims has quit IRC | 13:51 | |
| *** subscope has quit IRC | 13:53 | |
| bauzas | johnthetubaguy: how was the ops feedback this morning ? saw a few stuff | 13:53 |
| *** ljxiash has joined #openstack-nova | 13:55 | |
| sdague | cdent: it must be a non empty file | 13:55 |
| tdurakov | paul-carlton2 eliqiao shaohe_feng pkoniszewski jlanoux yuntongjin mdbooth kashyap kmartin claudiub tobasco live migration subteam meeting starts in 4 minutes, please join #openstack-meeting-3 | 13:56 |
| *** andrearo1 is now known as andrearosa | 13:57 | |
| bauzas | meh, seems we had some storage issues with -infra :( http://logs.openstack.org/34/277934/5/check/gate-nova-pep8/8c6bb6f/ | 13:57 |
| eliqiao | tdurakov: gotcha | 13:58 |
| sdague | alex_xu: you still up? | 14:00 |
| alex_xu | sdague: yup | 14:00 |
| sdague | https://bugs.launchpad.net/nova/+bug/1541691 is an interesting, and problematic bug | 14:00 |
| openstack | Launchpad bug 1541691 in OpenStack Compute (nova) "server boot with leading and trailing white spaces in name, displays weird error message" [High,Confirmed] - Assigned to Srushti Gadadare (srushti-gadadare) | 14:00 |
| *** haomaiwa_ has quit IRC | 14:01 | |
| *** zz_dimtruck is now known as dimtruck | 14:01 | |
| sdague | the valid name regex is currently about 2 MB | 14:01 |
| *** haomaiwang has joined #openstack-nova | 14:01 | |
| *** jwcroppe has joined #openstack-nova | 14:01 | |
| sdague | because it's built by iterating the entire 4 byte utf8 range and listing every character one at a time | 14:01 |
| sdague | so when it fails, it dumps this giant thing, that no one understands | 14:02 |
| sdague | which is really 2 issues | 14:02 |
| alex_xu | sdague: yes...we have a huge regex.. | 14:02 |
| sdague | 1, it's 2 MB | 14:02 |
| sdague | 2, it's not a useful error | 14:02 |
| *** claudiub has joined #openstack-nova | 14:02 | |
| *** doug-fish has joined #openstack-nova | 14:02 | |
| sdague | regex supports ranges | 14:02 |
| sdague | why isn't that just building ranges | 14:02 |
| *** liusheng has quit IRC | 14:03 | |
| sdague | which would at least make it smaller | 14:03 |
| *** raildo is now known as raildo-afk | 14:03 | |
| sdague | that's got to be a few MB per worker we're storing there | 14:03 |
| sdague | given regex object overhead | 14:03 |
| *** liusheng has joined #openstack-nova | 14:03 | |
| alex_xu | sdague: actually I'm not very good at regex | 14:03 |
| sdague | ok | 14:03 |
| sdague | well, let's go to problem #2, I can look at #1 | 14:04 |
| alex_xu | sdague: how much size we can small with range? | 14:04 |
| *** changbl has quit IRC | 14:04 | |
| sdague | good question, I'll have to write some tests | 14:04 |
| sdague | alex_xu: anyway, for cryptic regexes, it would be really good if the validation types had a custom error message | 14:04 |
| johnthetubaguy | bauzas: didn't get too many concrete things, mostly some documentation bits, and a few specific bugs, and related things | 14:05 |
| bauzas | johnthetubaguy: ack | 14:05 |
| alex_xu | for #2, I need check if we have way to use useful message instead json-schema generated. or we just validate name in the python code | 14:05 |
| sdague | https://github.com/openstack/nova/blob/dbdf27021cc23bf8cec8d3f51fa03c6dac5da53a/nova/api/validation/parameter_types.py#L163 | 14:05 |
| sdague | alex_xu: right, well in our code if we could specify an additional param there which would be the custom error message | 14:06 |
| sdague | if we failed jsonschema validation, send that out instead | 14:06 |
| sdague | this is also dumping those 2MB into the logs on each of these calls | 14:06 |
| *** vladikr has joined #openstack-nova | 14:06 | |
| sdague | so it becomes a bit of a concern on log fill up | 14:06 |
| alex_xu | sdague: yes | 14:07 |
| *** mpavone has quit IRC | 14:07 | |
| *** salv-orlando has quit IRC | 14:08 | |
| *** kiran-r has quit IRC | 14:08 | |
| *** ccesario has joined #openstack-nova | 14:08 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: Removed duplicated_key_in_test_vmops https://review.openstack.org/280714 | 14:08 |
| *** irenab has quit IRC | 14:09 | |
| *** mvk has quit IRC | 14:09 | |
| *** xyang1 has joined #openstack-nova | 14:09 | |
| *** ctrath has joined #openstack-nova | 14:10 | |
| *** takedakn has quit IRC | 14:10 | |
| *** raildo-afk is now known as raildo | 14:11 | |
| *** Guest76507 is now known as med_ | 14:12 | |
| *** med_ has quit IRC | 14:12 | |
| *** med_ has joined #openstack-nova | 14:12 | |
| *** mdrabe has joined #openstack-nova | 14:12 | |
| *** MCoLo__ has quit IRC | 14:14 | |
| *** MCoLo has joined #openstack-nova | 14:15 | |
| *** jaosorior has quit IRC | 14:16 | |
| *** jaosorior has joined #openstack-nova | 14:17 | |
| alex_xu | sdague: do you like we get rid fo those regex instead with this https://github.com/openstack/nova/blob/master/nova/api/validation/validators.py#L67 ? | 14:18 |
| alex_xu | but I still need test if we can pass custom message, there can pass in a custom exception https://python-jsonschema.readthedocs.org/en/latest/validate/#jsonschema.FormatChecker.cls_checks, maybe we can pass custom message, but i need test | 14:19 |
| sdague | alex_xu: maybe | 14:20 |
| sdague | so if we build up ranges, this gets a ton smaller | 14:20 |
| *** mpavone has joined #openstack-nova | 14:20 | |
| *** jichen has quit IRC | 14:20 | |
| sdague | http://paste.openstack.org/show/487117/ | 14:20 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [microversions] Add support for 2.19 https://review.openstack.org/278385 | 14:20 |
| *** mvk has joined #openstack-nova | 14:21 | |
| *** ekuris has joined #openstack-nova | 14:21 | |
| *** subscope has joined #openstack-nova | 14:22 | |
| sdague | alex_xu: len(regex) is 1910 in the non_ws case | 14:24 |
| sdague | actually, in both cases | 14:24 |
| alex_xu | sdague: so...it still huge | 14:24 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Add a way to discover only contrib extensions https://review.openstack.org/280725 | 14:25 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Turn on extensions from contrib dir for v2.1 https://review.openstack.org/280726 | 14:25 |
| *** EinstCra_ has joined #openstack-nova | 14:25 | |
| sdague | vs. 54263 | 14:26 |
| sdague | which is the current issue | 14:26 |
| *** eharney has quit IRC | 14:26 | |
| *** EinstCrazy has quit IRC | 14:28 | |
| alex_xu | yes | 14:28 |
| *** rdopiera is now known as rdopiera_adk | 14:30 | |
| *** rdopiera_adk is now known as rdopiera_afk | 14:30 | |
| *** smatzek_ has quit IRC | 14:31 | |
| *** zengine has joined #openstack-nova | 14:32 | |
| *** klkumar has quit IRC | 14:32 | |
| *** mriedem_away has quit IRC | 14:32 | |
| sdague | alex_xu: ok, so how would you like to proceed here? | 14:33 |
| sdague | because while this regex is shorter | 14:34 |
| sdague | it's not any easier to understand | 14:34 |
| sdague | http://paste.openstack.org/show/487119/ | 14:35 |
| *** achanda has joined #openstack-nova | 14:35 | |
| *** zzzeek has joined #openstack-nova | 14:35 | |
| *** cfriesen has joined #openstack-nova | 14:36 | |
| *** ildikov has quit IRC | 14:36 | |
| alex_xu | sdague: this is short a lot. I'm ok with this, maybe I can find out a way to add custom validator to jsonschema lib, then we can get a custom message | 14:37 |
| *** achanda has quit IRC | 14:40 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add StableObjectJsonFixture and use it in our base test class https://review.openstack.org/279242 | 14:40 |
| *** yed has joined #openstack-nova | 14:43 | |
| sdague | alex_xu: if we were going to do a custom validator, we'd still use the regex for that? | 14:43 |
| *** dimtruck is now known as zz_dimtruck | 14:44 | |
| alex_xu | sdague: yes, probably just add some custom message. but I prefer we can use format checker instead of regex totally | 14:44 |
| sdague | ok, well how about I change the code to give us smaller regexes, because we'll still use that | 14:45 |
| sdague | and you can take a look at custom checker tomorrow | 14:45 |
| sdague | and we'll consider them both to be part of the fix | 14:45 |
| alex_xu | sdague: ok, that's cool | 14:45 |
| alex_xu | sdague: i will find out both whether custom validator can custom message and whether format checker works tomorrow | 14:46 |
| sdague | ok, great | 14:46 |
| sdague | have a good evening | 14:46 |
| alex_xu | sdague: thanks | 14:47 |
| *** ildikov has joined #openstack-nova | 14:48 | |
| openstackgerrit | Brian Elliott proposed openstack/nova: Reap idle SQL connections after 60 seconds https://review.openstack.org/276325 | 14:49 |
| * alex_xu hate this slow vpn connection... | 14:50 | |
| *** esracelik has quit IRC | 14:51 | |
| bauzas | dansmith: around already ? I was wondering whether https://review.openstack.org/#/c/266233/4/nova/objects/image_meta.py was possible to be tested | 14:52 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (2) https://review.openstack.org/280438 | 14:52 |
| *** burt has joined #openstack-nova | 14:52 | |
| dansmith | bauzas: okay, I'll look in a sec | 14:52 |
| *** ihrachys has quit IRC | 14:53 | |
| bauzas | dansmith: it's a general pattern about testing backwars compatible objects | 14:53 |
| bauzas | IIRC, I provided one of them | 14:53 |
| *** pradk has joined #openstack-nova | 14:53 | |
| *** nkrinner has joined #openstack-nova | 14:54 | |
| *** aarefiev has quit IRC | 14:54 | |
| *** nkrinner has quit IRC | 14:54 | |
| *** mriedem has joined #openstack-nova | 14:55 | |
| *** aarefiev has joined #openstack-nova | 14:55 | |
| *** aarefiev has quit IRC | 14:55 | |
| *** mrkz has joined #openstack-nova | 14:55 | |
| *** aarefiev has joined #openstack-nova | 14:55 | |
| rlrossit | bauzas: I'm about to disappear for a while, but there's no way for testing (as far as I know) to know the previous history of the object with respect to what fields were on it in previous versions | 14:55 |
| dansmith | well, | 14:56 |
| bauzas | rlrossit: so I wrote one of them | 14:56 |
| dansmith | right, unless we keep it | 14:56 |
| bauzas | sec | 14:56 |
| dansmith | which means we have to remember to do that, or try to make a test to note when new fields were added | 14:56 |
| dansmith | which we might be able to do | 14:56 |
| *** thorst_ has joined #openstack-nova | 14:56 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [microversions] Turn off check for header in response https://review.openstack.org/280746 | 14:56 |
| dansmith | I tried something a while back and never liked what I came up with | 14:56 |
| *** ducttape_ has joined #openstack-nova | 14:57 | |
| *** ducttape_ has quit IRC | 14:57 | |
| *** ducttape_ has joined #openstack-nova | 14:57 | |
| bauzas | rlrossit: dansmith: https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_compute_node.py#L389-L392 | 14:57 |
| bauzas | I was thinking of this kind of check | 14:57 |
| *** mtanino has joined #openstack-nova | 14:57 | |
| dansmith | bauzas: sure, but we have to write it each time, unless we have some test that can tell you forgot | 14:57 |
| *** coolsvap|away has quit IRC | 14:57 | |
| *** sridharg has quit IRC | 14:57 | |
| bauzas | dansmith: that's my point | 14:58 |
| rlrossit | dansmith: yep | 14:58 |
| bauzas | dansmith: I wonder if we should agree to merge https://review.openstack.org/#/c/266233/4/nova/objects/image_meta.py without either an explicit test (like the one I wrote) or wait for a generic detection pattern | 14:58 |
| *** edtubill has joined #openstack-nova | 14:58 | |
| *** subscope has quit IRC | 14:59 | |
| sdague | alex_xu: also, the current validation is all wonkers | 14:59 |
| dansmith | bauzas: I have definitely not been writing static tests like that for all the attributes I add, because they're fairly thin.. like our db migrations tests that check to see that columns were added :) | 14:59 |
| *** thorst has quit IRC | 14:59 | |
| dansmith | bauzas: let me think on another automated test for a bit | 14:59 |
| *** nkrinner has joined #openstack-nova | 14:59 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: [dnm] cleanup live-migrations https://review.openstack.org/280653 | 15:00 |
| bauzas | dansmith: yeah that's my concern | 15:00 |
| *** nkrinner has quit IRC | 15:00 | |
| dansmith | bauzas: that I'll think? :) | 15:00 |
| bauzas | dansmith: nah, that we should somehow have a general pattern for detecting that miss | 15:00 |
| *** nkrinner has joined #openstack-nova | 15:00 | |
| bauzas | and not provide a test for each attribute backport | 15:00 |
| *** tdurakov has quit IRC | 15:01 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** coolsvap|away has joined #openstack-nova | 15:01 | |
| *** sfinucan has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-nova | 15:01 | |
| bauzas | in the meantime, I'll accept the above change, there is no reason to wait for an automatic test :) | 15:01 |
| *** smatzek_ has joined #openstack-nova | 15:02 | |
| *** ljxiash has quit IRC | 15:02 | |
| *** ljxiash has joined #openstack-nova | 15:02 | |
| mriedem | bauzas: fwiw, i think it's perfectly fine to want to see a unit test for a change like this https://review.openstack.org/#/c/266233/2/nova/objects/image_meta.py | 15:02 |
| bauzas | mriedem: but we don't ask atm | 15:03 |
| mriedem | to make sure the primitive is popped if target_version < 1.10 | 15:03 |
| mriedem | are people afraid to ask? | 15:03 |
| bauzas | mriedem: I even don't write that with my changes, except the one I provided above | 15:03 |
| bauzas | mriedem: nah, it's a matter of checking each attribute in a single test | 15:03 |
| mriedem | w/o a unit test someone could come in later and change the logic in the compat method and regress it somehow | 15:03 |
| bauzas | so, see, I'm torn :D | 15:03 |
| *** EinstCra_ has quit IRC | 15:04 | |
| *** tdurakov has joined #openstack-nova | 15:04 | |
| mriedem | chances are slim, but it's possible | 15:04 |
| *** EinstCrazy has joined #openstack-nova | 15:04 | |
| bauzas | so, should we ask to write an UT for each time someone is adding a field ? | 15:04 |
| rlrossit | mriedem: the problem there is you're adding a lot of reviewer burden | 15:04 |
| *** whenry has joined #openstack-nova | 15:04 | |
| alex_xu | sdague: what means wonker? | 15:04 |
| mriedem | rlrossit: reviewer burden? | 15:04 |
| bauzas | mriedem: like https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_compute_node.py#L389-L392 | 15:04 |
| rlrossit | mriedem: you always gotta make sure you're looking for it | 15:05 |
| rlrossit | the onus is on the reviewer to find it | 15:05 |
| mriedem | well, | 15:05 |
| mriedem | it's pretty simple | 15:05 |
| mriedem | 1. someone is fixing a bug | 15:05 |
| dansmith | mriedem: bauzas: I definitely don't think there's anything *wrong* with asking for a test | 15:05 |
| mriedem | 2. bug fixes should nearly always have a unit test with them | 15:05 |
| mriedem | we have that somewhere in the docs | 15:05 |
| mriedem | bug fixes should have a test change | 15:05 |
| bauzas | okay, lemme be the messenger :) | 15:06 |
| *** ljxiash has quit IRC | 15:06 | |
| mriedem | b/c by definition they weren't caught with tests to begin with | 15:06 |
| *** subscope has joined #openstack-nova | 15:06 | |
| *** efoley__ is now known as efoley | 15:06 | |
| mriedem | so the only burden on the reviewer is asking for the test | 15:06 |
| dansmith | mriedem: can we merge this? https://review.openstack.org/#/c/279242/ | 15:06 |
| dansmith | mriedem: I bet bauzas would go in on it with you | 15:07 |
| mriedem | +2 | 15:07 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Add a way to discover only contrib extensions https://review.openstack.org/280725 | 15:07 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Turn on extensions from contrib dir for v2.1 https://review.openstack.org/280726 | 15:07 |
| sdague | alex_xu: if I am reading this correctly | 15:07 |
| sdague | valid_name_regex_base | 15:07 |
| sdague | valid_name_regex_base = '^(?![%s])[%s]*(?<![%s])$' | 15:08 |
| dansmith | mriedem: thanks | 15:08 |
| *** klkumar has joined #openstack-nova | 15:08 | |
| sdague | which is fed in printable_ws, get_printable, printable_ws | 15:08 |
| sdague | meaning that only printable_ws characters are excluded from the starting position | 15:08 |
| *** rqw has quit IRC | 15:09 | |
| sdague | non printable zw characters, control characters, could still be there | 15:09 |
| *** moshele has joined #openstack-nova | 15:09 | |
| bauzas | dansmith: mriedem: heh :) | 15:09 |
| rlrossit | mriedem: so that means I should've -1'd dansmith on https://review.openstack.org/#/c/277554/ ? | 15:09 |
| *** tdurakov has quit IRC | 15:09 | |
| rlrossit | (just to play devil's advocate) | 15:09 |
| * rlrossit runs away | 15:10 | |
| ndipanov | dansmith, https://review.openstack.org/#/c/278078/5 | 15:10 |
| *** gongysh has quit IRC | 15:11 | |
| *** EinstCrazy has quit IRC | 15:12 | |
| bauzas | rlrossit: so I feel mriedem's point was rather that the bugfix wasn't shipping an UT | 15:12 |
| dansmith | ndipanov: pushed it on the stack | 15:12 |
| bauzas | dansmith: so, you made a fixture in https://review.openstack.org/#/c/279242/3/nova/tests/fixtures.py because you don't want to change the behaviour for the object serialization itself ? | 15:13 |
| dansmith | bauzas: definitely not, yeah | 15:13 |
| dansmith | bauzas: it only matters for unit tests, and I don't want to burn the cycles to sort on every rpc call (see the commit message) | 15:13 |
| bauzas | dansmith: okay, just for my knowledge, it would be a terrible idea, because.. ? | 15:13 |
| bauzas | ah | 15:13 |
| bauzas | yeah indeed | 15:13 |
| dansmith | because it's useless and wasted effort? :) | 15:13 |
| bauzas | because you would have to sort for every rpc call | 15:14 |
| *** EinstCrazy has joined #openstack-nova | 15:14 | |
| dansmith | right | 15:14 |
| mriedem | surprise surprise, softlayer moved to liberty and didn't understand or know about the upper-constraints file | 15:15 |
| sdague | mriedem: and what happened? | 15:15 |
| sdague | mriedem: I would like story time | 15:16 |
| mriedem | they are using the latest versions of things like os-brick that aren't tested on liberty | 15:16 |
| mriedem | well, i don't think that's really their problem, their immediate problem is they are doing scale testing with multipath attached volumes and it's shitting the bed | 15:16 |
| *** mkoderer__ has quit IRC | 15:16 | |
| mriedem | but i wouldn't be surprised if they start blindly picking up mitaka versions of packages b/c they aren't aware of upper-constraints | 15:16 |
| mriedem | i honestly don't know how consumers / deployers are really supposed to know about that magical unicorn | 15:17 |
| *** rotbeard has quit IRC | 15:18 | |
| *** mkoderer__ has joined #openstack-nova | 15:18 | |
| *** zz_dimtruck is now known as dimtruck | 15:18 | |
| mriedem | 4) multipathd crashes are causing volumes stuck in "detaching" state. | 15:18 |
| mriedem | sounds fun | 15:18 |
| sdague | mriedem: yeh, that's probably a good point. Worth bringing it up on the ML and poking lifeless | 15:18 |
| mriedem | multipath is for HA right? | 15:18 |
| mriedem | but multipath prevents you from being able to operate, so HA doesn't matter, how ironic | 15:19 |
| *** su_zhang has joined #openstack-nova | 15:20 | |
| *** irenab has joined #openstack-nova | 15:20 | |
| *** tdurakov_ has joined #openstack-nova | 15:21 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: imagecache: set base_dir as a property https://review.openstack.org/209912 | 15:23 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: imagecache: implement listing of cached images https://review.openstack.org/209913 | 15:23 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: virt: add method for obtaining the list of cached images https://review.openstack.org/209914 | 15:23 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: libvirt: implement get_cached_images method https://review.openstack.org/209915 | 15:23 |
| *** tdurakov_ has quit IRC | 15:23 | |
| *** paul-carlton1 has quit IRC | 15:23 | |
| *** tongli has joined #openstack-nova | 15:25 | |
| sdague | in my experience a lot of HA techniques end up causing more downtime because they add a ton of complexity | 15:25 |
| *** EinstCrazy has quit IRC | 15:25 | |
| *** gszasz has quit IRC | 15:25 | |
| andrearosa | sdague: that is my experience too | 15:25 |
| kashyap | sdague: Heh, I heard someone else say the samething recently | 15:25 |
| kashyap | "With HA you get more downtime than without" | 15:26 |
| *** mdrabe has quit IRC | 15:26 | |
| ndipanov | dansmith, that means it's next in line... I think you meant to say added it to the queue | 15:26 |
| dansmith | ndipanov: as soon as I said that I knew you were going to CS101 me | 15:27 |
| *** whenry has quit IRC | 15:27 | |
| ndipanov | you've been wellactuallied | 15:27 |
| dansmith | ndipanov: but, my "queue" is currently length of one, not including the item I'm on right now, stack == queue | 15:27 |
| dansmith | just hope nobody else pushes something on my _stack_ | 15:28 |
| * cdent smashes dansmith's stack | 15:29 | |
| *** whenry has joined #openstack-nova | 15:29 | |
| *** gszasz has joined #openstack-nova | 15:30 | |
| mriedem | sdague: bombs away on the email | 15:30 |
| *** eharney has joined #openstack-nova | 15:32 | |
| *** mnestratov has joined #openstack-nova | 15:34 | |
| *** fawadkhaliq has joined #openstack-nova | 15:34 | |
| *** fawadkhaliq has quit IRC | 15:34 | |
| *** fawadkhaliq has joined #openstack-nova | 15:35 | |
| *** yamahata has joined #openstack-nova | 15:36 | |
| *** jlanoux has quit IRC | 15:39 | |
| *** EinstCrazy has joined #openstack-nova | 15:39 | |
| *** irenab has quit IRC | 15:39 | |
| *** salv-orl_ has joined #openstack-nova | 15:40 | |
| *** salv-orl_ has quit IRC | 15:40 | |
| *** irenab has joined #openstack-nova | 15:41 | |
| *** salv-orlando has joined #openstack-nova | 15:41 | |
| *** roeyc has quit IRC | 15:41 | |
| doffm | Probably most people that install openstack are still doing so using distributions. When we were packaging it the first thing we did was look at the requirements repo. It contains the projects file. We were not using pip. It was very painful. | 15:41 |
| *** zenoway has quit IRC | 15:43 | |
| *** mdrabe has joined #openstack-nova | 15:43 | |
| *** zenoway has joined #openstack-nova | 15:44 | |
| *** sfinucan has joined #openstack-nova | 15:44 | |
| *** nkrinner has quit IRC | 15:45 | |
| *** irenab has quit IRC | 15:45 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: scheduler: weight nodes based on their cached images https://review.openstack.org/280688 | 15:46 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: compute: track cached images SHA1s https://review.openstack.org/280687 | 15:46 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: scheduler: weight nodes based on their cached images https://review.openstack.org/173304 | 15:47 |
| mriedem | doffm: softlayer is using wheels + venvs | 15:48 |
| *** zenoway has quit IRC | 15:48 | |
| mriedem | which is sane | 15:48 |
| *** aloga has quit IRC | 15:49 | |
| mriedem | apparently they are pinning the versions in their ansible scripts, and they just apparently pinned to the latest versions available in some cases, like os-brick | 15:49 |
| mriedem | b/c they didn't know upper-constraints was a thing | 15:49 |
| mriedem | venvs or containers is where i think a lot of us want deployers to get | 15:49 |
| *** pushkaru has joined #openstack-nova | 15:51 | |
| *** dave-mccowan has quit IRC | 15:53 | |
| *** changbl has joined #openstack-nova | 15:56 | |
| *** mgoddard_ has joined #openstack-nova | 16:00 | |
| *** su_zhang has quit IRC | 16:00 | |
| ndipanov | sfinucan, hey - I think I might have an idea about what's happening with pinning and shelving | 16:00 |
| ndipanov | I updated the bug | 16:00 |
| ndipanov | will post some patches soon | 16:00 |
| ndipanov | https://bugs.launchpad.net/nova/+bug/1545675 | 16:01 |
| openstack | Launchpad bug 1545675 in OpenStack Compute (nova) "Resizing a pinned VM results in inconsistent state" [Undecided,In progress] - Assigned to Stephen Finucane (sfinucan) | 16:01 |
| *** haomaiwang has quit IRC | 16:01 | |
| ndipanov | sfinucan, let me know if the explanation makes sense... | 16:01 |
| *** haomaiwang has joined #openstack-nova | 16:01 | |
| *** baoli has quit IRC | 16:01 | |
| *** moshele has quit IRC | 16:01 | |
| *** salv-orl_ has joined #openstack-nova | 16:02 | |
| *** baoli has joined #openstack-nova | 16:03 | |
| *** mgoddard has quit IRC | 16:03 | |
| *** rdopiera_afk is now known as rdopiera | 16:04 | |
| *** Bjoern has joined #openstack-nova | 16:05 | |
| *** salv-orlando has quit IRC | 16:05 | |
| *** haplo37 has joined #openstack-nova | 16:06 | |
| *** Swanson has joined #openstack-nova | 16:06 | |
| sfinucan | ndipanov: I'll check it out. Cheers :) | 16:07 |
| Swanson | Hi. Using nova networking and have stacked devstack on top of devstack. The virtualized-devstack VMs do not seem to be able to get a dhcp lease. Any idea what I should look for? (Or which IRC channel I should ask..) | 16:08 |
| *** ekuris has quit IRC | 16:09 | |
| *** gszasz has quit IRC | 16:12 | |
| *** dave-mccowan has joined #openstack-nova | 16:12 | |
| *** su_zhang has joined #openstack-nova | 16:13 | |
| *** Marga_ has quit IRC | 16:13 | |
| *** drankis has quit IRC | 16:13 | |
| *** EinstCrazy has quit IRC | 16:15 | |
| *** EinstCrazy has joined #openstack-nova | 16:16 | |
| *** pcaruana has quit IRC | 16:18 | |
| dansmith | bauzas: rlrossit : | 16:19 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP: Test to validate obj_make_compatible() strips old attributes https://review.openstack.org/280795 | 16:19 |
| dansmith | someone needs to flesh out the mapping | 16:19 |
| dansmith | which kinda sounds like "the guy that called for the test" but .. :) | 16:20 |
| *** zenoway has joined #openstack-nova | 16:20 | |
| *** EinstCrazy has quit IRC | 16:20 | |
| mriedem | he who dealt it? | 16:20 |
| mriedem | smelt it? | 16:20 |
| mriedem | i know that's backward | 16:21 |
| dansmith | smelt it delt it | 16:21 |
| dansmith | heh | 16:21 |
| *** mvk has quit IRC | 16:21 | |
| *** bkumar has joined #openstack-nova | 16:22 | |
| *** irenab has joined #openstack-nova | 16:22 | |
| mriedem | sdague: any chance you can clean this up this week? it's related to the cells v2 stuff and i don't think it's very far off, https://review.openstack.org/#/c/265282/ | 16:22 |
| sdague | mriedem: I can, though we said it was punted until newton | 16:23 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: Validate subs in api samples base class to improve error handling https://review.openstack.org/278221 | 16:23 |
| sdague | so it was low on my priority list, and I wasn't going to circle around until m3 | 16:23 |
| mriedem | ok, i guess it's not a blocker for the cells v2 flavor tables in the api db stuff | 16:24 |
| *** su_zhang has quit IRC | 16:24 | |
| alaski | it's a blocker for the migration, but not the table addition | 16:24 |
| *** zenoway has quit IRC | 16:24 | |
| *** su_zhang has joined #openstack-nova | 16:25 | |
| *** su_zhang has quit IRC | 16:25 | |
| *** jlanoux has joined #openstack-nova | 16:28 | |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add manage command for Cell0. https://review.openstack.org/275266 | 16:29 |
| doffm | Gah. 'Cell0 -> cell0' | 16:29 |
| *** drankis has joined #openstack-nova | 16:30 | |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add manage command for cell0. https://review.openstack.org/275266 | 16:30 |
| *** jdurgin1 has joined #openstack-nova | 16:30 | |
| mriedem | alaski: why a blocker for the migration? and how would we enforce that in the code? | 16:31 |
| *** agireud has quit IRC | 16:31 | |
| mriedem | alaski: just b/c the microversion is in the API doesn't mean anyone is at that version | 16:31 |
| mriedem | or that we're checking a microversion in the object code | 16:31 |
| *** sdague has quit IRC | 16:31 | |
| *** agireud has joined #openstack-nova | 16:32 | |
| *** Swami has joined #openstack-nova | 16:32 | |
| mriedem | alaski: do we expect https://review.openstack.org/#/c/213041/ to get into mitaka b/c it looks like it's in bad shape and hasn't had any love in several weeks | 16:33 |
| *** boris-42 has quit IRC | 16:34 | |
| alaski | mriedem: We don't need to wait, but if we wait then when we do the migration and wipe out the ability to show deleted flavors at all there's at least an alternative available. Though you make a fair point about whether or not clients will use it | 16:34 |
| *** sdague has joined #openstack-nova | 16:34 | |
| mriedem | well, it looks like we'll probably be waiting anyway if that flavor migration change isn't updated | 16:35 |
| alaski | mriedem: not that one, no. https://review.openstack.org/#/c/201606/23 is targetted for M with the one you posted waiting on the think from sdague | 16:35 |
| alaski | s/think/thing/ | 16:35 |
| mriedem | yeah, that should be an easy one, i plan on reviewing today | 16:35 |
| mriedem | was just reading your ML thread on cells v2 status, which is now probably outdated | 16:35 |
| alaski | heh, not too bad. there's just additional reviews from doffm and melwitt now for nova-manage commands | 16:36 |
| *** mpavone has quit IRC | 16:36 | |
| *** rcernin has quit IRC | 16:37 | |
| *** achanda has joined #openstack-nova | 16:38 | |
| *** armax has joined #openstack-nova | 16:38 | |
| mriedem | but we're cool with the flavors tables in the api db being not soft deletable, it's not being used now anyway | 16:39 |
| alaski | yep. puts us one step closer to removing soft-delete everywhere | 16:41 |
| openstackgerrit | Thomas Maddox proposed openstack/nova: [WIP - DO NOT MERGE] Testing LXC Gate https://review.openstack.org/274792 | 16:41 |
| mriedem | 1/~35 | 16:42 |
| mriedem | not bad | 16:42 |
| mriedem | :) | 16:42 |
| *** achanda has quit IRC | 16:44 | |
| *** zenoway has joined #openstack-nova | 16:44 | |
| *** baoli has quit IRC | 16:45 | |
| *** baoli has joined #openstack-nova | 16:46 | |
| alaski | :) looking at the list doffm put together we might migrate about half of those into cells so hopefully we pull this same trick for each | 16:47 |
| mriedem | so, https://github.com/openstack/nova/blob/master/nova/objects/base.py#L197-L198 | 16:48 |
| mriedem | it's a good thing those are nullable | 16:48 |
| mriedem | defaulting 'deleted' to False when there is no 'deleted' attribute seems weird, or misleading, but i guess if it's just not used it's not a big deal | 16:48 |
| alaski | We haven't been using that mixin for cells v2 objects | 16:49 |
| mriedem | sure, but those are new objects, like InstanceMapping right? | 16:50 |
| mriedem | that are only in the api db | 16:50 |
| alaski | yeah | 16:50 |
| alaski | are you thinking of removing it from current objects that have it? | 16:50 |
| mriedem | no | 16:51 |
| mriedem | i just noticed this now too https://github.com/openstack/nova/blob/master/nova/objects/base.py#L177 | 16:51 |
| *** klkumar has quit IRC | 16:51 | |
| doffm | alaski: Yeah, we are going to need to live migrate alot of stuff. Might be worth pulling out a frameworky thing from the flavor migration patch. | 16:51 |
| alaski | yeah, I added that for the cells objects | 16:51 |
| mriedem | seems odd that the tag object doesn't have the timestamp mixin https://github.com/openstack/nova/blob/master/nova/objects/tag.py | 16:51 |
| openstackgerrit | Sean Dague proposed openstack/nova: build smaller name regexes for validation https://review.openstack.org/280814 | 16:51 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Cell switch in manage db sync command for cell0 https://review.openstack.org/274885 | 16:52 |
| mriedem | oh i guess the tag model doesn't use the timestamp mixin | 16:52 |
| alaski | mriedem: I don't think the timestamp mixin existed at that point | 16:53 |
| mriedem | well they could have defined the created_at/updated_at columns in the db model explicitly | 16:53 |
| *** ssurana has joined #openstack-nova | 16:53 | |
| mriedem | but anyway, i'm on a bad tangent | 16:53 |
| *** eharney has quit IRC | 16:53 | |
| alaski | right. I'm not sure why that didn't happen. it probably should have | 16:53 |
| mriedem | tags aren't in the api yet anyway | 16:53 |
| alaski | doffm: that would be great if we could do that. Currently much of that migration work occurs within the objects, and I'm not sure if we'll gain much to pull it out but it's worth a look. | 16:54 |
| mriedem | if a 2nd migration type has common code, pull it up to common code at that point | 16:55 |
| *** yassine__ has quit IRC | 16:55 | |
| *** subscope has quit IRC | 16:56 | |
| alaski | right, but looking at https://review.openstack.org/#/c/213041/24/nova/objects/flavor.py nothing is jumping out at me as something that would be common with another migration | 16:56 |
| doffm | Its the common code along the lines of 'look for item in API db, don't find, migrate it' But the 'look for item' is a different function in each object which makes things difficult. | 16:57 |
| *** matrohon has quit IRC | 16:57 | |
| doffm | I'd LIKE there to be common code, since its a common pattern. But i'm not sure there will be. | 16:57 |
| *** vishwanathj has joined #openstack-nova | 16:57 | |
| *** zakora has quit IRC | 16:57 | |
| alaski | +1 | 16:58 |
| alaski | flavor may be a bad example to look at as it's actually three tables which complicates it a bit | 16:58 |
| mriedem | like the getter? | 17:00 |
| mriedem | you could pass the db api function in | 17:00 |
| *** haomaiwang has quit IRC | 17:01 | |
| *** haomaiwang has joined #openstack-nova | 17:01 | |
| *** asselin has quit IRC | 17:03 | |
| *** mgoddard_ has quit IRC | 17:03 | |
| *** mgoddard has joined #openstack-nova | 17:03 | |
| *** e0ne has quit IRC | 17:04 | |
| *** mjura has quit IRC | 17:04 | |
| *** Marga_ has joined #openstack-nova | 17:05 | |
| *** Marga_ has quit IRC | 17:05 | |
| *** Marga_ has joined #openstack-nova | 17:05 | |
| alaski | certainly worth looking at to see if a framework emerges. I'm not against it, just not seeing it quite yet | 17:06 |
| *** apoorvad has joined #openstack-nova | 17:07 | |
| *** apoorvad has quit IRC | 17:08 | |
| *** eharney has joined #openstack-nova | 17:09 | |
| *** aginwala has joined #openstack-nova | 17:09 | |
| *** rcernin has joined #openstack-nova | 17:09 | |
| *** cfriesen has quit IRC | 17:09 | |
| *** apoorvad has joined #openstack-nova | 17:10 | |
| *** asselin has joined #openstack-nova | 17:10 | |
| *** scheuran has quit IRC | 17:13 | |
| *** mylu has joined #openstack-nova | 17:13 | |
| mriedem | zzzeek: does this automagically add the index to the uuid column definition so it hooks back into the table create later? https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_migrations/migrate_repo/versions/001_cell_mapping.py#L45 | 17:15 |
| zzzeek | mriedem: without knowing if sqlalchemy-migrate gets in the way, in general yes a table.create() would include that index | 17:16 |
| zzzeek | err | 17:16 |
| zzzeek | yes. table.create() would do it. | 17:16 |
| zzzeek | if this were alembic, alembic doesn't use table.create() | 17:16 |
| *** penick has joined #openstack-nova | 17:17 | |
| *** mylu has quit IRC | 17:18 | |
| bauzas | doffm: not sure you saw my comment on the database connection string https://review.openstack.org/#/c/275266/2..4/nova/cmd/manage.py | 17:19 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add manage command for cell0. https://review.openstack.org/275266 | 17:20 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add a cell and host mapping utility to nova-manage https://review.openstack.org/270565 | 17:20 |
| openstackgerrit | Mark Doffman proposed openstack/nova: WIP: PoC: Add cell mapping for instances https://review.openstack.org/280834 | 17:20 |
| bauzas | doffm: alaski: I'm also unclear, is cell0 a fresh new 3rd DB or just a logical implementation for a cell_mapping table ? | 17:20 |
| *** su_zhang has joined #openstack-nova | 17:21 | |
| alaski | bauzas: cell0 should be considered an additional cell that just happens to not have a lot of things, like compute hosts | 17:21 |
| bauzas | alaski: that I know | 17:22 |
| bauzas | alaski: and I agree | 17:22 |
| doffm | bauzas: Sorry about that. I did miss the connection string comment. | 17:22 |
| bauzas | alaski: I actually see that just an abstract | 17:22 |
| alaski | bauzas: so ideally it's a 3rd db, that matches the other cell db | 17:22 |
| bauzas | alaski: what if we have a AIO or a cell DB and an API DB on the same host ? | 17:23 |
| alaski | there is nothing that keeps that database from living on the same host as the api db, or the current cell db | 17:23 |
| alaski | if you change the db name you could colocate with another cell db | 17:23 |
| bauzas | I agree it's a connection string | 17:23 |
| bauzas | but a 3rd db sync ? | 17:23 |
| doffm | bauzas: This option is required as cell0 will be using a separate db. Like alaski said, one that matches the other cells dbs (for now). The same migrations will be performed on the cell0 db as a normal cell db. | 17:23 |
| doffm | bauzas: Yep a 3rd db sync. | 17:23 |
| bauzas | huh | 17:23 |
| bauzas | I was just thinking it was just like some tables in the API DB saying "meh, it's for the cell0 abstraction" | 17:24 |
| alaski | that was an option early on | 17:24 |
| bauzas | so I missed that convo, my bad | 17:25 |
| alaski | doing it that way meant that we need to special case the lookups there | 17:25 |
| bauzas | right, I remember us in Tokyo | 17:25 |
| bauzas | haha, I see | 17:26 |
| *** su_zhang has quit IRC | 17:26 | |
| *** su_zhang has joined #openstack-nova | 17:26 | |
| *** jdurgin1 has quit IRC | 17:27 | |
| *** su_zhang has quit IRC | 17:27 | |
| *** sfinucan has quit IRC | 17:28 | |
| doffm | Of course the way the code has ended up you could provide the db connection string of your first proper cell. Then you only do 2 db syncs. Not sure if that is a good idea, but right now you could. | 17:28 |
| ndipanov | alaski, got a sec? | 17:29 |
| *** su_zhang has joined #openstack-nova | 17:29 | |
| alaski | ndipanov: yep | 17:29 |
| ndipanov | question about shelved offloaded - are we supposed to keep resources "reserved" on the host? | 17:29 |
| alaski | not if it's offloaded | 17:29 |
| alaski | unshelving in that case will reschedule | 17:30 |
| ndipanov | right... | 17:30 |
| bauzas | doffm: yeah, that's just a connection string, I agree | 17:30 |
| ndipanov | alaski, context: https://bugs.launchpad.net/nova/+bug/1545675 | 17:30 |
| openstack | Launchpad bug 1545675 in OpenStack Compute (nova) "Resizing a pinned VM results in inconsistent state" [Undecided,In progress] - Assigned to Stephen Finucane (sfinucan) | 17:30 |
| ndipanov | if you have a sec read through my comment and see if my analysis wrt shelving makes sense | 17:30 |
| bauzas | alaski: sorry, I forgot that if you were just saying "meh, cell0 is there", you litterally need a 3rd DB | 17:30 |
| alaski | ndipanov: will do | 17:31 |
| *** mylu has joined #openstack-nova | 17:31 | |
| alaski | bauzas: right, will cell0 you really have two cells rather than one | 17:31 |
| bauzas | in case of a AIO yes | 17:32 |
| *** roeyc has joined #openstack-nova | 17:32 | |
| *** shyama has quit IRC | 17:32 | |
| bauzas | because you have the api db, the cell1 db, and then you would litteraly need a copy of cell1 aka. cell0 | 17:33 |
| bauzas | okay, I left some comments in a devref that I should update :) | 17:33 |
| alaski | heh, ok | 17:34 |
| *** aginwala has quit IRC | 17:34 | |
| *** dims has joined #openstack-nova | 17:34 | |
| *** ociuhandu has quit IRC | 17:35 | |
| *** aginwala has joined #openstack-nova | 17:36 | |
| *** lpetrut has quit IRC | 17:36 | |
| *** spandhe has joined #openstack-nova | 17:37 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "guestfs" https://review.openstack.org/248963 | 17:37 |
| *** bkumar has quit IRC | 17:38 | |
| *** nic has joined #openstack-nova | 17:39 | |
| *** Jeffrey4l has quit IRC | 17:40 | |
| *** sahid has quit IRC | 17:42 | |
| *** paul-carlton1 has joined #openstack-nova | 17:43 | |
| mriedem | alaski: went through https://review.openstack.org/#/c/201606/ - i think we should take the opportunity to define a bunch of those columns as non-nullable now | 17:44 |
| mriedem | since the API reqiures them to be non-nullable and they have unique constraints on them. | 17:44 |
| *** delattec has joined #openstack-nova | 17:45 | |
| *** nic has quit IRC | 17:46 | |
| alaski | ndipanov: commented. tl;dr is that I agree with your analysis | 17:46 |
| *** nic has joined #openstack-nova | 17:47 | |
| *** e0ne has joined #openstack-nova | 17:47 | |
| bauzas | oh snaaaaaap https://review.openstack.org/#/c/277934/5/nova/tests/unit/objects/test_compute_node.py L460 | 17:48 |
| ndipanov | alaski, many thanks! :) | 17:48 |
| alaski | mriedem: yeah, that would be a great cleanup | 17:48 |
| *** cdelatte has quit IRC | 17:48 | |
| bauzas | I need to modify the primitive target thanks to dansmith :p | 17:48 |
| bauzas | dansmith: I'm just <3 your https://review.openstack.org/#/c/280795/1 | 17:49 |
| *** mriedem is now known as mriedem_lunch | 17:49 | |
| *** abhishekk has joined #openstack-nova | 17:52 | |
| *** fyxim has joined #openstack-nova | 17:55 | |
| *** browne has joined #openstack-nova | 17:56 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Update ComputeNode values with disk allocation ratios in the RT https://review.openstack.org/277953 | 17:57 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add disk_allocation_ratio to ComputeNode https://review.openstack.org/277934 | 17:57 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Update HostManager and DiskFilter to use ComputeNode disk ratio https://review.openstack.org/278112 | 17:57 |
| *** achanda has joined #openstack-nova | 17:59 | |
| *** kfarr has joined #openstack-nova | 17:59 | |
| *** shyama has joined #openstack-nova | 18:01 | |
| *** haomaiwang has quit IRC | 18:01 | |
| *** delattec has quit IRC | 18:01 | |
| *** haomaiwang has joined #openstack-nova | 18:01 | |
| *** sridharg has joined #openstack-nova | 18:01 | |
| *** Marga__ has joined #openstack-nova | 18:01 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Make 'debug' option reconfigurable in Nova https://review.openstack.org/280851 | 18:02 |
| *** drankis has quit IRC | 18:02 | |
| *** delattec has joined #openstack-nova | 18:03 | |
| *** ssurana1 has joined #openstack-nova | 18:05 | |
| *** Marga_ has quit IRC | 18:05 | |
| *** achanda has quit IRC | 18:06 | |
| *** ssurana has quit IRC | 18:06 | |
| *** mriedem_lunch is now known as mriedem | 18:06 | |
| *** kfarr has quit IRC | 18:07 | |
| *** abhishekk has quit IRC | 18:07 | |
| *** subscope has joined #openstack-nova | 18:07 | |
| *** kfarr has joined #openstack-nova | 18:10 | |
| dansmith | bauzas: heh, cool | 18:12 |
| bauzas | dansmith: I'll review it later tonight | 18:13 |
| bauzas | I see it's a WIP tho | 18:13 |
| dansmith | we have to finish the attribute map for it to be right | 18:13 |
| bauzas | yeah I saw | 18:13 |
| dansmith | the map is just fake right now | 18:13 |
| dansmith | okay | 18:13 |
| bauzas | I have a few things to do also, but if I can help, I'll do | 18:14 |
| *** paul-carlton1 has quit IRC | 18:14 | |
| *** drankis has joined #openstack-nova | 18:15 | |
| bauzas | dansmith: so if I understand, we need to update the attribute map by adding for each version the new attr, right ? | 18:16 |
| *** jlanoux has quit IRC | 18:16 | |
| bauzas | for the moment, it's a basic 1.0 which is fake | 18:16 |
| *** davideagnello has joined #openstack-nova | 18:18 | |
| *** ctrath has quit IRC | 18:19 | |
| *** drankis has quit IRC | 18:19 | |
| *** ctrath has joined #openstack-nova | 18:22 | |
| *** achanda has joined #openstack-nova | 18:23 | |
| *** chhavi has quit IRC | 18:23 | |
| *** spandhe has quit IRC | 18:23 | |
| *** mylu has quit IRC | 18:24 | |
| sdague | if anyone is feeling like they'd like to prove their regex skills, they could figure out if I borked up this - https://review.openstack.org/#/c/280814/ | 18:24 |
| *** efoley has quit IRC | 18:25 | |
| *** spandhe has joined #openstack-nova | 18:26 | |
| *** david-lyle has quit IRC | 18:27 | |
| dansmith | bauzas: right, so if you look at the tests you'll see why/how it fails because 1.0 is not the right number for anything added to an object since | 18:27 |
| *** vilobhmm11 has joined #openstack-nova | 18:27 | |
| bauzas | okay so I understood correctly | 18:28 |
| bauzas | yeah was looking at the UTs | 18:28 |
| bauzas | that seems an handful of necessary changes tho, I wonder how we could just use that for a fraction of objects | 18:29 |
| bauzas | and then add the new ones etc. | 18:29 |
| openstackgerrit | Ed Leafe proposed openstack/nova: config options: centralize section "database" + "api_database" https://review.openstack.org/263804 | 18:29 |
| *** shyama has quit IRC | 18:30 | |
| *** vilobhmm11 has quit IRC | 18:30 | |
| cdent | sdague: how far away is \w, with UNICODE flag set, from what you need? | 18:30 |
| sdague | cdent: \w doesn't include punctuation or spaces | 18:31 |
| cdent | yes, but isn't there a character class for that? | 18:31 |
| sdague | that I don't know | 18:31 |
| cdent | maybe I'm missing perl too much | 18:31 |
| * cdent looks around | 18:31 | |
| *** tdurakov_ has joined #openstack-nova | 18:31 | |
| sdague | we also have a pretty wide regex right now, so restricting it woudl potentially make existing sites explode | 18:32 |
| sdague | but, if you do find a better option, that would be great | 18:33 |
| *** paul-carlton1 has joined #openstack-nova | 18:33 | |
| sdague | this is incrementally better than listing every character, which is what the old one did | 18:33 |
| *** e0ne has quit IRC | 18:33 | |
| *** drankis has joined #openstack-nova | 18:34 | |
| cdent | It's \p (for posix character classes) but its not in yet, could import this: https://pypi.python.org/pypi/regex | 18:34 |
| *** mriedem has quit IRC | 18:35 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: [dnm] cleanup live-migrations https://review.openstack.org/280653 | 18:35 |
| *** vilobhmm11 has joined #openstack-nova | 18:35 | |
| *** david-lyle has joined #openstack-nova | 18:35 | |
| *** ccarmack has quit IRC | 18:36 | |
| sdague | cdent: we could, that requires native code, which makes it more of a burden on packaging / requirements. So unless you think it's a huge win, I'd rather go with just using stdlib | 18:36 |
| cdent | probably better to go with your thing | 18:36 |
| *** tdurakov_ has quit IRC | 18:38 | |
| *** ijw has joined #openstack-nova | 18:40 | |
| *** vilobhmm11 has quit IRC | 18:40 | |
| *** ijw_ has joined #openstack-nova | 18:42 | |
| *** ijw has quit IRC | 18:42 | |
| *** danpb1 has quit IRC | 18:42 | |
| *** mriedem has joined #openstack-nova | 18:42 | |
| *** diana_clarke has quit IRC | 18:42 | |
| *** lpetrut has joined #openstack-nova | 18:44 | |
| *** diana_clarke has joined #openstack-nova | 18:47 | |
| openstackgerrit | Russell Bryant proposed openstack/nova-specs: Add OVS fallback to VIF_TYPE_VHOSTUSER. https://review.openstack.org/280881 | 18:48 |
| *** spandhe has quit IRC | 18:51 | |
| *** ssurana1 has quit IRC | 18:52 | |
| *** ccarmack has joined #openstack-nova | 18:54 | |
| *** spandhe has joined #openstack-nova | 18:56 | |
| *** ctrath has quit IRC | 18:57 | |
| *** haomaiwang has quit IRC | 19:01 | |
| *** ctrath has joined #openstack-nova | 19:01 | |
| *** haomaiwang has joined #openstack-nova | 19:01 | |
| *** Bjoern has quit IRC | 19:01 | |
| sdague | doffm / alaski - the cell0 db and the way we're doing nova manage commands is really making the cell of 1 case pretty complicated - https://review.openstack.org/#/c/276903/2/lib/nova | 19:03 |
| *** haplo37 has quit IRC | 19:03 | |
| sdague | is there a way we can make the base case simpler, even if that means nova-manage is doing a few things under the covers | 19:03 |
| *** jichen has joined #openstack-nova | 19:03 | |
| alaski | sdague: your concern is just the number of nova-manage commands needed? | 19:05 |
| sdague | yeh, say I'm a small site, a few nodes. Now I've got to manage 3 dbs and remember to upgrade them all on every upgrade to have a couple dozen vms | 19:06 |
| *** gibi has quit IRC | 19:06 | |
| *** gibi has joined #openstack-nova | 19:07 | |
| *** vilobhmm11 has joined #openstack-nova | 19:07 | |
| alaski | "nova-manage db sync" could take an option to loop through all cells | 19:07 |
| sdague | sure, it could, or do that by default. | 19:07 |
| sdague | make the complicated thing of phased upgrades the non default and take options | 19:08 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: Log successful reverts_task_state calls https://review.openstack.org/280889 | 19:08 |
| *** gibi has quit IRC | 19:08 | |
| sdague | because the experts are likely the ones doing it | 19:08 |
| alaski | do we want to optimize for the large or small deployment? | 19:08 |
| sdague | large deployments have more skills | 19:08 |
| alaski | one would hope :) | 19:09 |
| alaski | but that works for me | 19:09 |
| sdague | I don't want to inhibit the infrastructure for cells v2, I just want the base case to be simpler to administer, because I'm assuming that the majority of clouds are only going to have 1 cell. | 19:10 |
| doffm | sdague: I basically agree. If you want the sync command to run through all cells that seems OK. Running over cell0 by default sounds like a really good idea. | 19:10 |
| sdague | by cloud count | 19:10 |
| sdague | doffm: yeh, I think that's better, and leave the options to do them one at a time there for people doing interesting things in large deploys | 19:10 |
| *** mylu has joined #openstack-nova | 19:11 | |
| alaski | another optimization we could make would be to have the initial cell setup create the cell0 db on the api db host | 19:11 |
| *** mtanino__ has quit IRC | 19:12 | |
| *** gibi has joined #openstack-nova | 19:12 | |
| alaski | but that's a bit much I think | 19:12 |
| alaski | rolling https://review.openstack.org/#/c/275266/5/nova/cmd/manage.py into https://review.openstack.org/#/c/270565/5/nova/cmd/manage.py for the default case | 19:12 |
| *** sridharg has quit IRC | 19:12 | |
| sdague | I also wonder if there can be a sensible cell0 set of defaults based on nova db location so that you don't have to specify it by default | 19:13 |
| *** rdopiera has quit IRC | 19:13 | |
| doffm | sdague: Plausibly. Take the regular connection string and append 'cell0' to the db name. | 19:13 |
| *** daemontool has joined #openstack-nova | 19:14 | |
| sdague | again, let people override if it they want to. But sensible defaults I think will get us a long way | 19:14 |
| *** ijw has joined #openstack-nova | 19:14 | |
| alaski | doffm: the api connection string, right? | 19:14 |
| doffm | alaski: Yeah sorry. The API connection string. | 19:15 |
| *** ijw_ has quit IRC | 19:15 | |
| *** bkero has left #openstack-nova | 19:16 | |
| *** daemontool_ has joined #openstack-nova | 19:16 | |
| *** dimtruck is now known as zz_dimtruck | 19:17 | |
| *** eharney has quit IRC | 19:17 | |
| *** ssurana has joined #openstack-nova | 19:18 | |
| doffm | Ok, so with sdague's suggestions - deployers will only have to call `nova-manage cell_v2 map_cell0` followed by `nova-manage cell_v2 map_cell_and_hosts --transport-url <url>` | 19:19 |
| *** daemontool has quit IRC | 19:19 | |
| doffm | (They will also have to call nova-manage db sync, but they have to do that anyway) | 19:20 |
| alaski | doffm: I think map_cell_and_hosts is intended purely for migrations of pre v2 deploys | 19:20 |
| *** mylu has quit IRC | 19:20 | |
| sdague | yeh, what does `nova-manage cell_v2 map_cell0` actually do? | 19:20 |
| *** mylu has joined #openstack-nova | 19:21 | |
| doffm | sdague: Maps all the current instances to the instance_mapping table. | 19:21 |
| *** zz_dimtruck is now known as dimtruck | 19:21 | |
| alaski | doffm: although looking at it it could be used as a bootstrap command as well | 19:21 |
| sdague | is that something we could just embed in a migration? | 19:22 |
| sdague | are they going to be able to function unless it's done | 19:22 |
| sdague | sorry if these are dumb questions, just the devstack patch made me start thinking through some of the implications | 19:23 |
| alaski | the mappings ask for user input (database connection string in the case of cell0) | 19:23 |
| sdague | right, but we said we could probably have a sensible default | 19:24 |
| alaski | if we embedded in a migration and pulled a sensible default we disallow the complex case | 19:24 |
| sdague | ah, ok, that makes sense | 19:24 |
| alaski | and for the other cells they need a messaging transport, and pulling a sensible value from the conf has been troublesome so far | 19:25 |
| *** unicell has joined #openstack-nova | 19:26 | |
| doffm | sdague: No, these are good questions. No, when cell switching is turned on things won't work properly until this is done. (map_cells_and_hosts command) | 19:26 |
| alaski | oslo.messaging synthesizes the value and then doesn't expose it | 19:26 |
| *** lbelivea has joined #openstack-nova | 19:26 | |
| doffm | alaski: Oslo messaging doesn't really use the transport url. It parses it for the pieces and then never uses it again. The rabbit driver has separate password and other options, so never puts an url together at all. | 19:27 |
| doffm | The other drivers use a transport url for configuration, so for those it seems difficult to get to, but available. | 19:28 |
| alaski | doffm: sdague I had been thinking there would be two commands for non cell0 cells, map_cells_and_hosts would migrate an existing deployment and some other thing like map_cell would be used for a new cell | 19:28 |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "guestfs" https://review.openstack.org/248963 | 19:28 |
| openstackgerrit | Tracy Jones proposed openstack/nova: Add better help to the guestfs option https://review.openstack.org/255578 | 19:28 |
| alaski | because map_cells_and_hosts does extra migration work for existing compute nodes | 19:28 |
| alaski | would it be preferable to roll those into one? | 19:28 |
| alaski | doffm: ahh okay re:messaging. I wasn't clear on the details there | 19:29 |
| mriedem | so i'm dumb, | 19:31 |
| mriedem | can someone tell me if "" means anyone? https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L468 | 19:31 |
| mriedem | or does "" mean default? | 19:31 |
| alaski | default | 19:31 |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "volume" https://review.openstack.org/253747 | 19:31 |
| mriedem | alaski: and the fact some use "" and some use "rule:admin_or_owner" is just random | 19:32 |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/nova: Implement setup_networks_on_host for Neutron to support Migration https://review.openstack.org/275073 | 19:32 |
| mriedem | where "rule:admin_or_owner" is the default | 19:32 |
| alaski | mriedem: yes. At least I've never seen any reasoning behind it | 19:32 |
| *** ijw has quit IRC | 19:32 | |
| mriedem | ok. | 19:32 |
| mriedem | thanks | 19:32 |
| *** ijw has joined #openstack-nova | 19:32 | |
| mriedem | i'll ask again in 12 months | 19:33 |
| doffm | Lets go through the two cases. Upgrade scenario and new deployment. 1) upgrade * map_cell0 followed by map_cells_and_hosts. 2) new deployment * map_cell0 followed by map_cell. | 19:33 |
| alaski | mriedem: that will hopefully be fixed when/if these get embedded in the code. though that effort stalled for a bit | 19:33 |
| *** su_zhang has quit IRC | 19:33 | |
| doffm | map_cell0 seems like it could be rolled in to the other one. Create it if it doesn't exist, use some sensible defaults. But I'd prefer to keep `map_cells_and_hosts` separate from `map_cell`. | 19:33 |
| alaski | doffm: I agree. I'm not yet sold on rolling map_cell0 in but I do think the other two should be separate | 19:34 |
| sdague | mriedem: hopefully in 12 months that file will be empty | 19:37 |
| *** eharney has joined #openstack-nova | 19:37 | |
| *** mylu has quit IRC | 19:40 | |
| *** mylu has joined #openstack-nova | 19:43 | |
| *** eil397 has joined #openstack-nova | 19:44 | |
| *** Sukhdev has joined #openstack-nova | 19:47 | |
| edmondsw | mriedem alaski "" means "allow anyone" not "use default" | 19:48 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create BuildRequest object during boot process https://review.openstack.org/278125 | 19:49 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add build_requests database table and model https://review.openstack.org/263926 | 19:49 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add BuildRequest object https://review.openstack.org/278124 | 19:49 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Make InstanceMappings.cell_id nullable https://review.openstack.org/278598 | 19:49 |
| openstackgerrit | Andrew Laski proposed openstack/nova: WIP Populate instance_mappings during boot. https://review.openstack.org/263925 | 19:49 |
| alaski | edmondsw: that hasn't been my experience. '@' means allow everyone | 19:49 |
| *** haplo37 has joined #openstack-nova | 19:49 | |
| edmondsw | alaski that does as well | 19:50 |
| edmondsw | http://docs.openstack.org/developer/oslo.policy/api/oslo_policy.html#policy-rule-expressions | 19:50 |
| edmondsw | see the "Finally" paragraph at the end of that link | 19:50 |
| sdague | alaski: ok, so what if in the base case "db sync" synced all the dbs | 19:51 |
| alaski | that would be a huge security issue with the default policy | 19:51 |
| sdague | edmondsw / alaski - re policy, perhaps we should build a testcase for that? | 19:52 |
| *** toure has quit IRC | 19:53 | |
| mriedem | i'm checking the code | 19:53 |
| mriedem | maybe docs are old | 19:53 |
| *** toure has joined #openstack-nova | 19:53 | |
| *** mylu has quit IRC | 19:53 | |
| alaski | hmm, the code does seem to equate '' with '@' | 19:54 |
| alaski | and I don't think that used to be the case | 19:55 |
| sdague | hmmm | 19:55 |
| sdague | yeh, that seems interesting | 19:55 |
| mriedem | alaski: where do you see that? | 19:55 |
| alaski | especially since that makes our default policy file useless now | 19:55 |
| alaski | mriedem: _parser.py:_parse_test_rule() | 19:56 |
| alaski | text_rule | 19:56 |
| mriedem | # Empty rule means always accept | 19:56 |
| mriedem | if not rule: | 19:56 |
| mriedem | return _checks.TrueCheck() | 19:56 |
| mriedem | yar | 19:56 |
| mriedem | fun | 19:56 |
| mriedem | well i'm glad i asked the dumb question :) | 19:56 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Creates flavor* tables in API database https://review.openstack.org/201606 | 19:57 |
| alaski | when did we move to oslo.policy? | 19:57 |
| sdague | last cycle | 19:57 |
| mriedem | https://github.com/openstack/nova/blob/stable/liberty/requirements.txt | 19:58 |
| sdague | about half way through | 19:58 |
| mriedem | oslo.policy is not in stable/liberty reqs | 19:58 |
| alaski | okay, trying to pull down that branch to see what old policy code did | 19:58 |
| sdague | mriedem: really? | 19:58 |
| *** Marga__ has quit IRC | 19:58 | |
| mriedem | yeah | 19:58 |
| stevemar | whua... we had it out back then | 19:58 |
| mriedem | so that means it's mitaka | 19:58 |
| stevemar | https://github.com/openstack/releases/blob/master/deliverables/liberty/oslo.policy.yaml | 19:58 |
| *** Marga_ has joined #openstack-nova | 19:58 | |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP Nova object for ResourcePool https://review.openstack.org/280906 | 19:59 |
| mriedem | added in m-2 | 19:59 |
| stevemar | oh nova's reqs, yeah, it landed in mitaka | 19:59 |
| *** spandhe has quit IRC | 19:59 | |
| mriedem | https://github.com/openstack/nova/commit/a4b2ade4794d5952527181abf1ee482dd16b8295 | 19:59 |
| dims | sdague : y, oslo.policy was post liberty | 20:00 |
| *** haomaiwang has quit IRC | 20:01 | |
| *** 21WAA3UQL has joined #openstack-nova | 20:01 | |
| *** rfolco has quit IRC | 20:01 | |
| mriedem | same in that old code https://review.openstack.org/#/c/198065/7/nova/openstack/common/policy.py | 20:01 |
| mriedem | L583 | 20:01 |
| alaski | saw that too, looks like it's been that way since 2012 | 20:01 |
| alaski | I'm still fairly sure my experience differed from that | 20:02 |
| *** e0ne has joined #openstack-nova | 20:02 | |
| alaski | and why would our example policy file have "" if that's wide open | 20:02 |
| mriedem | alaski: rax specific policy code? | 20:02 |
| *** ctrath has quit IRC | 20:03 | |
| *** paul-carlton1 has quit IRC | 20:03 | |
| *** spandhe has joined #openstack-nova | 20:03 | |
| alaski | I modified the policy.json file heavily there, but I don't recall any modifications made to the policy code | 20:03 |
| mriedem | ha https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L73-L75 | 20:04 |
| mriedem | anyone can resize your instance by default | 20:04 |
| mriedem | nice | 20:04 |
| *** jwang_ has joined #openstack-nova | 20:04 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Add better help text to scheduler options (6) https://review.openstack.org/255517 | 20:04 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Add better help text to scheduler options (7) https://review.openstack.org/255518 | 20:04 |
| mriedem | set admin password? sure: https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L54 | 20:05 |
| *** e0ne has quit IRC | 20:05 | |
| *** aginwala has quit IRC | 20:05 | |
| *** jwang has quit IRC | 20:08 | |
| *** aginwala has joined #openstack-nova | 20:08 | |
| *** jaosorior has quit IRC | 20:09 | |
| *** eil397 has quit IRC | 20:11 | |
| alaski | okay, a test proves me wrong. At least back in Kilo '' meant '@'. | 20:12 |
| *** aysyd has quit IRC | 20:13 | |
| mriedem | alaski: wrong again https://github.com/openstack/nova/blob/stable/kilo/nova/openstack/common/policy.py#L584 | 20:14 |
| mriedem | :P | 20:14 |
| mriedem | has to be out of tree rax goodies | 20:14 |
| *** lucasagomes is now known as lucas-dinner | 20:14 | |
| mriedem | still begs the question, why does the default nova policy file have so many ""? | 20:15 |
| alaski | maybe I change I didn't realize wasn't upstream | 20:15 |
| mriedem | it's unusable basically | 20:15 |
| alaski | yeah, I'm doing a quick substitution to push up | 20:15 |
| *** ctrath has joined #openstack-nova | 20:17 | |
| *** david-lyle has quit IRC | 20:18 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Update example policy.json to remove "" policies https://review.openstack.org/280913 | 20:18 |
| alaski | sdague: mriedem ^ | 20:19 |
| mriedem | alaski: did you run unit tests yet? | 20:19 |
| alaski | nope, hold please | 20:20 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: RT: Decrese usage for offloaded instances https://review.openstack.org/280914 | 20:20 |
| *** drankis has quit IRC | 20:21 | |
| openstackgerrit | Ronald Bradford proposed openstack/nova: Remove duplicated oslo.log configuration setup https://review.openstack.org/245345 | 20:22 |
| *** claudiub has quit IRC | 20:24 | |
| *** mfedosin has quit IRC | 20:26 | |
| alaski | wth, there's a test to ensure that those rules are empty | 20:27 |
| sdague | alaski: a real one? | 20:27 |
| sdague | or the fake policy | 20:28 |
| mriedem | alaski: you can use wtf, this is a safe space | 20:28 |
| *** jwang__ has joined #openstack-nova | 20:28 | |
| alaski | it checks the sample file to ensure they're set that way | 20:28 |
| sdague | or fff | 20:28 |
| alaski | mriedem: :) | 20:28 |
| alaski | ffs | 20:28 |
| sdague | because, pretty much, that's what this seems like | 20:28 |
| alaski | okay, the test was good intentioned but the wrong thing to do | 20:29 |
| *** lpetrut has quit IRC | 20:29 | |
| *** jwang has joined #openstack-nova | 20:30 | |
| *** jwang_ has quit IRC | 20:32 | |
| sdague | oh, gdi, policy.json file being json means it can't have comments | 20:32 |
| sdague | because the thing you kind of really want to do here is actually make a version with these bits commented out | 20:32 |
| *** jwang__ has quit IRC | 20:33 | |
| *** rlrossit has quit IRC | 20:33 | |
| *** mylu has joined #openstack-nova | 20:33 | |
| *** drankis has joined #openstack-nova | 20:34 | |
| sdague | alaski: wait a second though on this policy.json change. Remember, we're context scoped | 20:34 |
| sdague | so it operates only within a valid project | 20:34 |
| sdague | that's not quite expressed here | 20:35 |
| alaski | is that true if it's not in the policy rule? | 20:35 |
| sdague | compute:get | 20:35 |
| sdague | yeh, it happens way up in the wsgi layer | 20:35 |
| sdague | we actually have tests that deny access to cross tenant resources | 20:36 |
| sdague | compute:get - as an example | 20:36 |
| *** bknudson_ has joined #openstack-nova | 20:36 | |
| alaski | hmm, well that's good | 20:36 |
| sdague | admin_or_owner would make it so that you couldn't ``nova show`` computes in your project created by someone else | 20:37 |
| alaski | I think it would still be useful to have that expressed in the policy layer | 20:37 |
| sdague | possibly, though that would be a bigger change, and open up another hole | 20:37 |
| alaski | sdague: owner is a project_id check | 20:37 |
| edmondsw | sdague, bknudson is working on allowing policy to use yaml so we can have comments | 20:37 |
| sdague | alaski: oh, sorry | 20:37 |
| sdague | right, so we're already project scoped before we get here | 20:38 |
| *** Zaina has joined #openstack-nova | 20:38 | |
| sdague | which means I think that "" effectively is admin or owner | 20:38 |
| *** jwang_ has joined #openstack-nova | 20:38 | |
| alaski | so this change wouldn't be limiting anything, but isn't really necessary | 20:39 |
| edmondsw | I personally think you've got a scoped token, the code should enforce that scope, no need to duplicate scope checks in policy | 20:39 |
| edmondsw | that assumes the code does the appropriate scope checks, of course | 20:39 |
| sdague | alaski: yeh, I think so | 20:39 |
| sdague | edmondsw: yeh, we have tests for this actually | 20:39 |
| edmondsw | I think for most things it does. the ?all_tenants kills me, though | 20:39 |
| sdague | edmondsw: sure but that's scoped differently | 20:40 |
| alaski | why wouldn't policy be the place to check the scope? | 20:40 |
| sdague | "compute:get_all_tenants": "is_admin:True", | 20:40 |
| edmondsw | if I've got a token scoped to one tenant, I should not be able to use it to get info for other tenants just because I'm an admin in this one | 20:40 |
| sdague | you are not an admin of a tenant | 20:40 |
| sdague | you are an admin of openstack | 20:40 |
| sdague | is_admin is global admin | 20:40 |
| edmondsw | I don't think that's true | 20:41 |
| sdague | edmondsw: yes, it is | 20:41 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Update example policy.json to remove "" policies https://review.openstack.org/280913 | 20:41 |
| *** mylu has quit IRC | 20:41 | |
| sdague | admin is a global role | 20:41 |
| *** subscope has quit IRC | 20:42 | |
| sdague | if you give someone admin bit, they can do admin things on all your services, like shut off nova computes | 20:42 |
| *** jwang has quit IRC | 20:42 | |
| edmondsw | sdague, no admin is not a global role. There is a concept of cloud admin, which is the admin for all tenants, but that requires being the admin on a specific special project defined in keystone's conf... not just any admin | 20:42 |
| alaski | I'm going to leave the change up because I still think it's useful, and we can argue it's merit there. Consistency between the rules would be nice, either all admin_or_owner to "" or all "" to admin_or_owner | 20:42 |
| alaski | in this context admin is whatever policy.json says it is, and it's not necessarily scoped to a tenant or defaulted that way | 20:43 |
| *** thorst_ is now known as thorst | 20:44 | |
| sdague | edmondsw: "context_is_admin": "role:admin", | 20:44 |
| edmondsw | if you want to check for the global admin, it's is_admin_project:True | 20:44 |
| *** roeyc has quit IRC | 20:44 | |
| edmondsw | https://github.com/openstack/keystone/blob/ac35c7eaa0024d5922a2c5d7b503e91cc3ab3af8/etc/policy.v3cloudsample.json#L3 | 20:44 |
| sdague | edmondsw: keystone has been pushing that direction, but most people don't implement that yet | 20:44 |
| edmondsw | true enough there | 20:45 |
| sdague | edmondsw: that's a pretty new construct | 20:45 |
| alaski | mriedem: fwiw that change did not break any unit tests except the specific check that those rules were empty | 20:45 |
| alaski | checking functional now though | 20:45 |
| edmondsw | yes... replacing the domain_id:admin_domain_id construct that wasn't as good | 20:45 |
| *** aginwala has quit IRC | 20:46 | |
| edmondsw | sdague, it's a question of what was done in practice (lots of folks considering admin role to be global) vs. what was intended (roles are scoped to projects, and admin is no different, as shown by all role assignments including admin being scoped to a project or domain) | 20:48 |
| openstackgerrit | Gage Hugo proposed openstack/nova: Fixed arguement order in remove_volume_connection https://review.openstack.org/280923 | 20:48 |
| sdague | edmondsw: the keystone policy is an attempt to use an admin domain as well to scope things | 20:48 |
| sdague | which had been discussed at summit, but there wasn't a ton of agreement on it | 20:49 |
| *** aginwala has joined #openstack-nova | 20:49 | |
| sdague | which is why you'll see all the projects ship policy files that set "context_is_admin": "role:admin", | 20:49 |
| edmondsw | I believe the admin domain thing is going away, because it required putting the admin domain id in every project's configuration... is_admin_project does not require that | 20:49 |
| sdague | cinder - https://github.com/openstack/cinder/blob/ab7ea8aae903b07b180daeda933e976eff5c75de/etc/cinder/policy.json#L2 | 20:50 |
| sdague | neutron - https://github.com/openstack/neutron/blob/3dec972fcd2c653ae1c4e2ecde8dda9e2372d83b/etc/policy.json#L2 | 20:50 |
| sdague | glance - https://github.com/openstack/glance/blob/67d3ff0c359a08f4628fed63f0369713eea0f2d6/etc/policy.json#L2 | 20:51 |
| sdague | so I get there is a different thing keystone is doing, but that doesn't mean that bleeds into other projects | 20:51 |
| *** mfedosin has joined #openstack-nova | 20:51 | |
| *** Sukhdev has quit IRC | 20:54 | |
| edmondsw | sdague, it has to... not all in one day of course, but everyone is in a bad place from a security perspective as things are today. | 20:54 |
| *** roeyc has joined #openstack-nova | 20:54 | |
| sdague | maybe. When it came up in the room last time all the large deployers were pretty good with global admin. Because you have one group of people that can fix your openstack. | 20:55 |
| sdague | and they kind of need all the permissions on all of it | 20:55 |
| edmondsw | sdague puts a lot of burden on a few people. The larger you get, the more you need to delegate. If the only admins are global admins, you can't effectively do that | 20:56 |
| ndipanov | alaski, if you get a moment https://review.openstack.org/#/c/280914/ | 20:57 |
| sdague | edmondsw: is there a place for another tier, sure. But that is different than letting global admin have access to all of this. | 20:57 |
| *** jwang__ has joined #openstack-nova | 20:58 | |
| sdague | anyway, circling around to the original thing, in nova we're fine because we have other code that restricts to project scope | 20:58 |
| edmondsw | sdague, are you arguing that we should leave things implemented to assume role:admin is a global admin, and implement something else to indicate someone is a project admin? | 20:59 |
| *** lbelivea has quit IRC | 20:59 | |
| edmondsw | I'd say nova is inconsistent because it has other code that restricts to project scope... even for admin role | 20:59 |
| sdague | I think changing the meaning of role:admin is bad form and a breaking change | 20:59 |
| edmondsw | so in some cases it treats admin like it's global, and it others it does not | 21:00 |
| sdague | edmondsw: as do almost all the projects | 21:00 |
| edmondsw | yes... I'm not trying to put the blame on nova | 21:00 |
| *** spandhe has quit IRC | 21:00 | |
| *** 21WAA3UQL has quit IRC | 21:01 | |
| *** jwang_ has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-nova | 21:01 | |
| *** raildo is now known as raildo-afk | 21:01 | |
| *** ndipanov has quit IRC | 21:03 | |
| *** apoorvad has quit IRC | 21:04 | |
| *** spandhe has joined #openstack-nova | 21:04 | |
| *** bjornar_ has quit IRC | 21:05 | |
| *** mfedosin has quit IRC | 21:07 | |
| *** sdague has quit IRC | 21:07 | |
| *** apoorvad has joined #openstack-nova | 21:12 | |
| mriedem | coincidentally they are talking about policy in #openstack-meeting-cp | 21:15 |
| *** smatzek_ has quit IRC | 21:16 | |
| *** daemontool__ has joined #openstack-nova | 21:17 | |
| *** fawadkhaliq has quit IRC | 21:18 | |
| *** daemontool_ has quit IRC | 21:20 | |
| *** Marga_ has quit IRC | 21:21 | |
| *** Marga_ has joined #openstack-nova | 21:21 | |
| *** ctrath has quit IRC | 21:25 | |
| *** Marga_ has quit IRC | 21:25 | |
| *** rlrossit has joined #openstack-nova | 21:25 | |
| *** ctrath has joined #openstack-nova | 21:25 | |
| *** su_zhang has joined #openstack-nova | 21:26 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Skip expensive config fixtures (where possible) https://review.openstack.org/280512 | 21:27 |
| *** lbelivea has joined #openstack-nova | 21:27 | |
| *** Marga_ has joined #openstack-nova | 21:28 | |
| *** baoli has quit IRC | 21:29 | |
| *** Marga_ has quit IRC | 21:33 | |
| *** Sukhdev has joined #openstack-nova | 21:33 | |
| *** Sukhdev has quit IRC | 21:35 | |
| *** dims has quit IRC | 21:37 | |
| *** burt has quit IRC | 21:38 | |
| *** daemontool has joined #openstack-nova | 21:38 | |
| *** lpetrut has joined #openstack-nova | 21:39 | |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Allow saving empty pci_device_pools in ComputeNode object https://review.openstack.org/216049 | 21:40 |
| *** daemontool__ has quit IRC | 21:42 | |
| *** drankis has quit IRC | 21:46 | |
| *** dedery has quit IRC | 21:48 | |
| *** dedery has joined #openstack-nova | 21:48 | |
| *** dims has joined #openstack-nova | 21:50 | |
| *** gyee has joined #openstack-nova | 21:50 | |
| dedery | dear cores, can you please review https://review.openstack.org/#/c/262341 (Fix create snapshot failure on VMs with SRIOV). It was +2'd by ndipanov and now waiting for another review | 21:51 |
| doffm | alaski: Any idea whats happening with resource pools cells? Are we punting this till newton? (Resource pools won't make mitaka?) | 21:53 |
| *** mriedem has quit IRC | 21:55 | |
| cdent | doffm I started some patches, despite the spec not being accepted yet, top of the stack: https://review.openstack.org/#/c/280333/ | 21:55 |
| cdent | that puts the tables in the cell db for now | 21:55 |
| cdent | because that's where the conversation keeps ending | 21:55 |
| *** whenry has quit IRC | 21:55 | |
| doffm | cdent: jaypipes was not plussed by the idea of adding a cell_uuid to the API, which pretty much precludes putting the resource pools table in there. | 21:56 |
| cdent | yeah, I saw that but then they seemed to come up with some kind of solution | 21:56 |
| cdent | but I'm not sure what it was | 21:56 |
| dansmith | using aggregate | 21:56 |
| alaski | doffm: I thought the latest is that he would get around that using aggregates to map cells to pools | 21:56 |
| alaski | or pools to cells | 21:56 |
| dansmith | right | 21:57 |
| doffm | I guess I'm just missing the latest. Is it in the spec? | 21:57 |
| cdent | not yet | 21:57 |
| *** jichen has quit IRC | 21:58 | |
| alaski | I'm really waiting for you and he to make ascii diagrams of all of this so it's all clear :) | 21:58 |
| doffm | cdent: Could you put it there under a cells section so I can take a look? | 21:58 |
| cdent | doffm: I don't know what it is | 21:58 |
| doffm | The ascii diagrams will make this go away. | 21:58 |
| cdent | I believe jaypipes, dansmith and alaski have the keys to that castle | 21:58 |
| *** tongli has quit IRC | 21:59 | |
| doffm | I'll forget about it for now then. Hopefully the spec will get updated at some point. Guess its not urgent. (Newton thing) | 21:59 |
| cdent | that probably needs a "?" on the end. | 21:59 |
| cdent | It's still mitaka | 21:59 |
| alaski | I understand the bits in isolation but I haven't formed the coherent whole in my head yet, that's why I'd like a diagram | 22:00 |
| cdent | If somebody sees Jay before I do, ask him to update the spec? | 22:00 |
| cdent | I would but I simply don't know what the plan is. | 22:00 |
| *** haomaiwang has quit IRC | 22:01 | |
| cdent | I heard he had some weather to deal with last night that threw things into chaos. | 22:01 |
| *** haomaiwang has joined #openstack-nova | 22:01 | |
| *** zengine has quit IRC | 22:01 | |
| alaski | doffm: I believe the idea is similar, or the same as, the idea you proposed last week in the cells meeting | 22:01 |
| *** salv-orlando has joined #openstack-nova | 22:03 | |
| *** thorst has quit IRC | 22:03 | |
| *** thorst has joined #openstack-nova | 22:03 | |
| *** spandhe has quit IRC | 22:05 | |
| * cdent retires | 22:05 | |
| *** daemontool has quit IRC | 22:05 | |
| *** salv-orl_ has quit IRC | 22:05 | |
| *** cdent has quit IRC | 22:06 | |
| *** ZZelle_ has joined #openstack-nova | 22:06 | |
| doffm | Ok. I'll wait to see what jaypipes thinks before giving it any more head space. Has to be somewhat different to what I was thinking as I still had the cell_id being specified in the resource pools API. | 22:06 |
| doffm | I suppose you could get around that by creating the resource pool record in EVERY cell. | 22:07 |
| *** mylu has joined #openstack-nova | 22:07 | |
| *** thorst_ has joined #openstack-nova | 22:07 | |
| *** jdurgin has quit IRC | 22:07 | |
| *** vishwanathj has quit IRC | 22:08 | |
| *** jdurgin has joined #openstack-nova | 22:08 | |
| *** thorst has quit IRC | 22:08 | |
| *** spandhe has joined #openstack-nova | 22:08 | |
| *** aginwala has quit IRC | 22:08 | |
| *** thorst_ has quit IRC | 22:11 | |
| *** baoli has joined #openstack-nova | 22:12 | |
| *** baoli_ has joined #openstack-nova | 22:12 | |
| alaski | doffm: oh yeah, because the aggregate association is done after the pool create that's true | 22:12 |
| *** kairo has joined #openstack-nova | 22:13 | |
| alaski | if the association was handled during the pool create then it should work out | 22:13 |
| alaski | I'm not sure if that's what he intended | 22:13 |
| *** mylu has quit IRC | 22:14 | |
| doffm | We will find out... | 22:15 |
| *** baoli has quit IRC | 22:16 | |
| *** Marga_ has joined #openstack-nova | 22:16 | |
| *** Marga_ has quit IRC | 22:17 | |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Failed migration shoudn't be reported as in progress https://review.openstack.org/277451 | 22:17 |
| *** Marga_ has joined #openstack-nova | 22:17 | |
| *** Marga_ has quit IRC | 22:18 | |
| *** lbelivea has quit IRC | 22:18 | |
| *** Marga_ has joined #openstack-nova | 22:18 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Update example policy.json to remove "" policies https://review.openstack.org/280913 | 22:19 |
| *** thorst has joined #openstack-nova | 22:24 | |
| *** dave-mccowan has quit IRC | 22:26 | |
| *** kairo has left #openstack-nova | 22:26 | |
| *** ccarmack has quit IRC | 22:27 | |
| *** thorst has quit IRC | 22:28 | |
| *** mylu has joined #openstack-nova | 22:32 | |
| *** zenoway has quit IRC | 22:38 | |
| *** dedery has quit IRC | 22:41 | |
| *** lpetrut has quit IRC | 22:45 | |
| *** ctrath has quit IRC | 22:46 | |
| *** j_king has quit IRC | 22:47 | |
| *** dims has quit IRC | 22:47 | |
| *** daemontool has joined #openstack-nova | 22:48 | |
| *** Nakato has quit IRC | 22:50 | |
| *** Nakato has joined #openstack-nova | 22:50 | |
| *** ctrath has joined #openstack-nova | 22:50 | |
| *** kfarr has quit IRC | 22:50 | |
| *** zenoway has joined #openstack-nova | 22:50 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Skip expensive config fixtures (where possible) https://review.openstack.org/280512 | 22:52 |
| *** changbl has quit IRC | 22:52 | |
| *** haplo37 has quit IRC | 22:52 | |
| *** ccarmack has joined #openstack-nova | 22:54 | |
| *** pradk has quit IRC | 22:55 | |
| *** zenoway has quit IRC | 22:55 | |
| *** Bjoern has joined #openstack-nova | 22:55 | |
| *** pece has quit IRC | 22:57 | |
| *** unicell has quit IRC | 22:57 | |
| *** unicell has joined #openstack-nova | 22:58 | |
| *** aginwala has joined #openstack-nova | 23:00 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwang has joined #openstack-nova | 23:01 | |
| *** vladikr has quit IRC | 23:02 | |
| *** rlrossit has quit IRC | 23:02 | |
| *** spandhe has quit IRC | 23:03 | |
| *** edtubill has quit IRC | 23:05 | |
| *** jwcroppe has quit IRC | 23:06 | |
| *** ivase has quit IRC | 23:06 | |
| *** Sukhdev has joined #openstack-nova | 23:07 | |
| *** apoorvad has quit IRC | 23:09 | |
| *** apoorvad has joined #openstack-nova | 23:12 | |
| *** ivase has joined #openstack-nova | 23:13 | |
| *** ivase has quit IRC | 23:17 | |
| *** mdrabe has quit IRC | 23:17 | |
| *** rcernin has quit IRC | 23:18 | |
| *** pushkaru has quit IRC | 23:20 | |
| *** pushkaru has joined #openstack-nova | 23:20 | |
| *** spandhe has joined #openstack-nova | 23:20 | |
| *** mylu has quit IRC | 23:24 | |
| *** zenoway has joined #openstack-nova | 23:26 | |
| *** zengine has joined #openstack-nova | 23:27 | |
| *** tojuvone_ has joined #openstack-nova | 23:27 | |
| *** eil397 has joined #openstack-nova | 23:28 | |
| *** xyang1_ has joined #openstack-nova | 23:29 | |
| *** carl_baldwin_ has joined #openstack-nova | 23:29 | |
| *** gyee_ has joined #openstack-nova | 23:30 | |
| *** andrewbogott_ has joined #openstack-nova | 23:30 | |
| *** DuncanT_ has joined #openstack-nova | 23:30 | |
| *** ducttape_ has quit IRC | 23:31 | |
| *** zenoway has quit IRC | 23:31 | |
| *** dave-mccowan has joined #openstack-nova | 23:31 | |
| *** mylu has joined #openstack-nova | 23:31 | |
| *** tpeoples_ has joined #openstack-nova | 23:32 | |
| *** gibi_ has joined #openstack-nova | 23:32 | |
| *** nelsnels_ has joined #openstack-nova | 23:33 | |
| *** kgalanov_ has joined #openstack-nova | 23:33 | |
| *** dimtruck is now known as zz_dimtruck | 23:35 | |
| *** cburgess has joined #openstack-nova | 23:35 | |
| *** wuhao_ has joined #openstack-nova | 23:35 | |
| *** ByPasS_ has joined #openstack-nova | 23:35 | |
| *** EmilienM_ has joined #openstack-nova | 23:35 | |
| *** another_larsks has joined #openstack-nova | 23:36 | |
| *** clayton_ has joined #openstack-nova | 23:36 | |
| *** wasmum- has joined #openstack-nova | 23:36 | |
| *** gyee has quit IRC | 23:37 | |
| *** gibi has quit IRC | 23:37 | |
| *** xyang1 has quit IRC | 23:37 | |
| *** ccesario has quit IRC | 23:37 | |
| *** andreykurilin has quit IRC | 23:37 | |
| *** ekarlso has quit IRC | 23:37 | |
| *** nelsnelson has quit IRC | 23:37 | |
| *** ByPasS has quit IRC | 23:37 | |
| *** wasmum has quit IRC | 23:37 | |
| *** wyang has quit IRC | 23:37 | |
| *** clayton has quit IRC | 23:37 | |
| *** wuhao has quit IRC | 23:37 | |
| *** andrewbogott has quit IRC | 23:37 | |
| *** tpeoples has quit IRC | 23:37 | |
| *** EmilienM has quit IRC | 23:37 | |
| *** acabot has quit IRC | 23:37 | |
| *** larsks has quit IRC | 23:37 | |
| *** tojuvone has quit IRC | 23:37 | |
| *** bswartz has quit IRC | 23:37 | |
| *** lbeliveau has quit IRC | 23:37 | |
| *** mdavidson has quit IRC | 23:37 | |
| *** carl_baldwin has quit IRC | 23:37 | |
| *** anthonyper has quit IRC | 23:37 | |
| *** cburgess_ has quit IRC | 23:37 | |
| *** DuncanT has quit IRC | 23:37 | |
| *** kgalanov has quit IRC | 23:37 | |
| *** ameade has quit IRC | 23:37 | |
| *** angular_mike__ has quit IRC | 23:37 | |
| *** johnthetubaguy has quit IRC | 23:37 | |
| *** xyang1_ is now known as xyang1 | 23:37 | |
| *** wuhao_ is now known as wuhao | 23:37 | |
| *** clayton_ is now known as clayton | 23:37 | |
| *** EmilienM_ is now known as EmilienM | 23:37 | |
| *** lbeliveau has joined #openstack-nova | 23:37 | |
| *** EmilienM is now known as Guest28836 | 23:37 | |
| *** ByPasS_ is now known as ByPasS | 23:37 | |
| *** kgalanov_ is now known as kgalanov | 23:38 | |
| *** carl_baldwin_ is now known as carl_baldwin | 23:38 | |
| *** andrewbogott_ is now known as andrewbogott | 23:40 | |
| *** tpeoples_ is now known as tpeoples | 23:40 | |
| *** Guest28836 has quit IRC | 23:41 | |
| *** Guest28836 has joined #openstack-nova | 23:41 | |
| *** angular_mike___ has joined #openstack-nova | 23:41 | |
| *** Guest28836 is now known as EmilienM | 23:42 | |
| *** ccarmack1 has joined #openstack-nova | 23:42 | |
| *** Bjoern has quit IRC | 23:42 | |
| *** mdavidson has joined #openstack-nova | 23:43 | |
| *** wyang has joined #openstack-nova | 23:43 | |
| *** ameade has joined #openstack-nova | 23:43 | |
| *** acabot has joined #openstack-nova | 23:43 | |
| *** DuncanT_ is now known as DuncanT | 23:43 | |
| *** ccesario has joined #openstack-nova | 23:43 | |
| *** dims_ has joined #openstack-nova | 23:44 | |
| *** pushkaru has quit IRC | 23:44 | |
| *** andreykurilin has joined #openstack-nova | 23:44 | |
| *** andrewbogott has quit IRC | 23:45 | |
| *** andrewbogott has joined #openstack-nova | 23:45 | |
| *** Bjoern has joined #openstack-nova | 23:45 | |
| *** ccarmack has quit IRC | 23:45 | |
| *** xyang1 has quit IRC | 23:46 | |
| *** Bjoern has quit IRC | 23:47 | |
| *** mylu has quit IRC | 23:47 | |
| *** ekarlso has joined #openstack-nova | 23:47 | |
| *** johnthetubaguy has joined #openstack-nova | 23:47 | |
| *** anthonyper has joined #openstack-nova | 23:47 | |
| *** ekarlso has quit IRC | 23:48 | |
| *** ekarlso has joined #openstack-nova | 23:48 | |
| *** ctrath has quit IRC | 23:50 | |
| *** mylu has joined #openstack-nova | 23:50 | |
| *** ccarmack1 has quit IRC | 23:54 | |
| *** harlowja_at_home has joined #openstack-nova | 23:55 | |
| *** browne has quit IRC | 23:56 | |
| *** mylu has quit IRC | 23:56 | |
| *** mrkz has quit IRC | 23:59 | |
| *** tjones has left #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!