| *** moshele has joined #openstack-nova | 00:01 | |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP: Test cellsv1 https://review.openstack.org/418141 | 00:04 |
|---|---|---|
| *** dave-mccowan has quit IRC | 00:07 | |
| *** jamielennox is now known as jamielennox|away | 00:07 | |
| *** harlowja has quit IRC | 00:09 | |
| *** LeoZhang has quit IRC | 00:09 | |
| *** LeoZhang has joined #openstack-nova | 00:10 | |
| *** CristinaPauna has quit IRC | 00:10 | |
| *** CristinaPauna has joined #openstack-nova | 00:10 | |
| *** purplerbot has quit IRC | 00:11 | |
| *** jamielennox|away is now known as jamielennox | 00:11 | |
| *** ansiwen has quit IRC | 00:11 | |
| *** andrewbogott has quit IRC | 00:11 | |
| *** baoli has quit IRC | 00:12 | |
| *** efried has joined #openstack-nova | 00:12 | |
| *** macsz has quit IRC | 00:12 | |
| *** markvoelker has quit IRC | 00:12 | |
| *** vdrok has quit IRC | 00:12 | |
| *** haukebruno has quit IRC | 00:12 | |
| *** rpodolyaka has quit IRC | 00:12 | |
| *** johnsom has quit IRC | 00:12 | |
| *** macsz has joined #openstack-nova | 00:12 | |
| *** purplerbot has joined #openstack-nova | 00:12 | |
| *** med_ has quit IRC | 00:12 | |
| *** afazekas has quit IRC | 00:12 | |
| *** markvoelker has joined #openstack-nova | 00:12 | |
| *** pkoniszewski has quit IRC | 00:13 | |
| *** ansiwen has joined #openstack-nova | 00:13 | |
| *** ildikov has quit IRC | 00:13 | |
| *** haukebruno has joined #openstack-nova | 00:13 | |
| *** jroll has quit IRC | 00:14 | |
| *** Dave_____ has joined #openstack-nova | 00:14 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient: Microversion 2.42 - Enable cold migration with target host https://review.openstack.org/406707 | 00:15 |
| *** andrewbogott has joined #openstack-nova | 00:15 | |
| *** hongbin has quit IRC | 00:15 | |
| *** pkoniszewski has joined #openstack-nova | 00:16 | |
| *** med_ has joined #openstack-nova | 00:16 | |
| *** med_ is now known as Guest47520 | 00:17 | |
| *** afazekas has joined #openstack-nova | 00:17 | |
| *** Guest47520 is now known as medberry | 00:19 | |
| *** medberry has quit IRC | 00:19 | |
| *** medberry has joined #openstack-nova | 00:19 | |
| *** Dave has quit IRC | 00:19 | |
| *** rpodolyaka has joined #openstack-nova | 00:20 | |
| *** jroll has joined #openstack-nova | 00:21 | |
| *** ildikov has joined #openstack-nova | 00:21 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 00:21 |
| *** vdrok has joined #openstack-nova | 00:22 | |
| *** johnsom has joined #openstack-nova | 00:22 | |
| *** vdrok has quit IRC | 00:25 | |
| *** vdrok has joined #openstack-nova | 00:25 | |
| *** johnsom has quit IRC | 00:25 | |
| *** johnsom has joined #openstack-nova | 00:25 | |
| *** thorst has joined #openstack-nova | 00:25 | |
| *** harlowja has joined #openstack-nova | 00:25 | |
| *** mlavalle has quit IRC | 00:31 | |
| *** tbachman has quit IRC | 00:32 | |
| *** jwcroppe has quit IRC | 00:32 | |
| *** chyka has quit IRC | 00:33 | |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance-exists notification https://review.openstack.org/403660 | 00:33 |
| *** aslezil has quit IRC | 00:34 | |
| *** alezil is now known as aslezil | 00:34 | |
| *** mriedem has joined #openstack-nova | 00:35 | |
| *** woodster_ has quit IRC | 00:35 | |
| *** mriedem1 has joined #openstack-nova | 00:38 | |
| *** thorst has quit IRC | 00:42 | |
| *** mriedem has quit IRC | 00:42 | |
| *** thorst has joined #openstack-nova | 00:42 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: WIP: Add service_token for nova-glance interaction https://review.openstack.org/412634 | 00:43 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: DNM: Test service token https://review.openstack.org/413232 | 00:43 |
| *** jaypipes has quit IRC | 00:44 | |
| *** dtp has quit IRC | 00:46 | |
| *** thorst has quit IRC | 00:46 | |
| *** tbachman has joined #openstack-nova | 00:49 | |
| *** tuanluong has joined #openstack-nova | 00:53 | |
| *** oomichi has quit IRC | 00:58 | |
| *** mriedem1 has quit IRC | 00:58 | |
| *** oomichi has joined #openstack-nova | 01:00 | |
| *** phuongnh has joined #openstack-nova | 01:04 | |
| *** nic has quit IRC | 01:06 | |
| *** huanxie has joined #openstack-nova | 01:06 | |
| *** ssurana has quit IRC | 01:09 | |
| *** LeoZhang has quit IRC | 01:11 | |
| *** thorst has joined #openstack-nova | 01:11 | |
| *** dave-mccowan has joined #openstack-nova | 01:12 | |
| *** ssurana has joined #openstack-nova | 01:13 | |
| oomichi | alex_xu: can you take a look at https://review.openstack.org/#/c/320375 ? it is a little difficult to understand for me | 01:14 |
| *** ssurana has left #openstack-nova | 01:14 | |
| *** thorst has quit IRC | 01:16 | |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP: Test cellsv1 https://review.openstack.org/418141 | 01:18 |
| *** zz_dimtruck is now known as dimtruck | 01:22 | |
| *** yuntongjin has joined #openstack-nova | 01:25 | |
| openstackgerrit | melanie witt proposed openstack/nova: WIP Count resources to check quota for cells https://review.openstack.org/416521 | 01:25 |
| openstackgerrit | melanie witt proposed openstack/nova: Make Quotas object favor the API database https://review.openstack.org/410945 | 01:25 |
| openstackgerrit | melanie witt proposed openstack/nova: Add online migration to move quotas to API database https://review.openstack.org/410946 | 01:25 |
| openstackgerrit | Huan Xie proposed openstack/nova: XenAPI Use os-xenapi lib for nova https://review.openstack.org/406059 | 01:27 |
| *** kevinz has joined #openstack-nova | 01:28 | |
| *** aweeks has joined #openstack-nova | 01:32 | |
| *** raunak has quit IRC | 01:33 | |
| *** test222 has joined #openstack-nova | 01:33 | |
| *** tjones has joined #openstack-nova | 01:34 | |
| *** briancurtin has joined #openstack-nova | 01:35 | |
| *** namnh has joined #openstack-nova | 01:39 | |
| *** markvoelker has quit IRC | 01:43 | |
| *** markvoelker has joined #openstack-nova | 01:44 | |
| *** markvoelker has quit IRC | 01:49 | |
| *** sree has joined #openstack-nova | 01:52 | |
| oomichi | alex_xu: johnthetubaguy: can you take a look at https://review.openstack.org/#/c/389314 ? That seems making sense for me. | 01:54 |
| *** cdelatte has quit IRC | 01:54 | |
| *** sree has quit IRC | 01:56 | |
| *** unicell has quit IRC | 02:01 | |
| *** thorst has joined #openstack-nova | 02:02 | |
| *** thorst has quit IRC | 02:02 | |
| *** efried has quit IRC | 02:03 | |
| alex_xu | oomichi: will review the patch after the meeting | 02:06 |
| *** sacharya has quit IRC | 02:07 | |
| *** fengzhr has joined #openstack-nova | 02:07 | |
| openstackgerrit | Huan Xie proposed openstack/nova: XenAPI Remove useless files when use os-xenapi lib https://review.openstack.org/411241 | 02:08 |
| *** browne has quit IRC | 02:08 | |
| *** jose-phillips has quit IRC | 02:12 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Plumbing for tagged nic attachment https://review.openstack.org/415622 | 02:12 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | 02:12 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Plumbing for tagged volume attachment https://review.openstack.org/391941 | 02:12 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: API support for tagged device attachment https://review.openstack.org/376021 | 02:12 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | 02:12 |
| *** efried has joined #openstack-nova | 02:13 | |
| *** dimtruck is now known as zz_dimtruck | 02:15 | |
| *** yingjun has joined #openstack-nova | 02:16 | |
| *** rama_y has joined #openstack-nova | 02:22 | |
| *** yingjun has quit IRC | 02:26 | |
| *** yingjun has joined #openstack-nova | 02:28 | |
| *** mdrabe has quit IRC | 02:29 | |
| *** voelzmo has joined #openstack-nova | 02:32 | |
| *** voelzmo has quit IRC | 02:37 | |
| *** raunak has joined #openstack-nova | 02:49 | |
| *** sree has joined #openstack-nova | 02:51 | |
| gmann | oomichi: alex_xu johnthetubaguy : yea even i did not find why we enforced volume policy in attachment APIs. | 02:53 |
| gmann | oomichi: alex_xu johnthetubaguy : for clear view we should separate volume attachment controller in separate file for easy maintenance. | 02:54 |
| openstackgerrit | huangtianhua proposed openstack/python-novaclient: Make _console() public https://review.openstack.org/414364 | 02:54 |
| *** hongbin has joined #openstack-nova | 02:55 | |
| *** sree has quit IRC | 02:55 | |
| *** sacharya has joined #openstack-nova | 03:02 | |
| openstackgerrit | zhangyujun proposed openstack/nova: Use JSON-Schema to validate bmd https://review.openstack.org/418654 | 03:07 |
| *** sacharya has quit IRC | 03:07 | |
| openstackgerrit | zhangyujun proposed openstack/nova: Use JSON-Schema to validate bdm https://review.openstack.org/418654 | 03:09 |
| *** moshele has quit IRC | 03:10 | |
| *** zz_dimtruck is now known as dimtruck | 03:11 | |
| *** yamahata has quit IRC | 03:14 | |
| *** rama_y has quit IRC | 03:14 | |
| *** dave-mccowan has quit IRC | 03:17 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | 03:17 |
| *** bdeschenes has quit IRC | 03:18 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 03:20 |
| *** thorst has joined #openstack-nova | 03:20 | |
| *** thorst has quit IRC | 03:20 | |
| *** gouthamr has quit IRC | 03:21 | |
| *** amotoki has quit IRC | 03:21 | |
| openstackgerrit | zhangyujun proposed openstack/nova: Use JSON-Schema to validate bdm https://review.openstack.org/418654 | 03:26 |
| *** diga has joined #openstack-nova | 03:27 | |
| *** amotoki has joined #openstack-nova | 03:29 | |
| *** nicolasbock has quit IRC | 03:31 | |
| *** salv-orl_ has quit IRC | 03:32 | |
| *** salv-orlando has joined #openstack-nova | 03:32 | |
| *** amotoki has quit IRC | 03:33 | |
| *** tbachman has quit IRC | 03:35 | |
| *** markvoelker has joined #openstack-nova | 03:36 | |
| *** yingjun has quit IRC | 03:46 | |
| *** links has joined #openstack-nova | 03:47 | |
| *** gongysh has joined #openstack-nova | 03:47 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Plumbing for tagged nic attachment https://review.openstack.org/415622 | 04:07 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | 04:07 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Plumbing for tagged volume attachment https://review.openstack.org/391941 | 04:07 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: API support for tagged device attachment https://review.openstack.org/376021 | 04:07 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | 04:07 |
| *** sacharya has joined #openstack-nova | 04:15 | |
| *** yamahata has joined #openstack-nova | 04:15 | |
| *** sacharya has quit IRC | 04:20 | |
| *** salv-orl_ has joined #openstack-nova | 04:23 | |
| *** salv-orlando has quit IRC | 04:26 | |
| *** jose-phillips has joined #openstack-nova | 04:27 | |
| *** jose-phillips has quit IRC | 04:30 | |
| *** sudipto_ has joined #openstack-nova | 04:32 | |
| *** sudipto has joined #openstack-nova | 04:32 | |
| *** voelzmo has joined #openstack-nova | 04:33 | |
| *** sheel has joined #openstack-nova | 04:34 | |
| *** ijw has quit IRC | 04:34 | |
| *** mdnadeem has joined #openstack-nova | 04:35 | |
| *** sree has joined #openstack-nova | 04:36 | |
| *** psachin has joined #openstack-nova | 04:40 | |
| *** voelzmo has quit IRC | 04:46 | |
| *** phuongnh has quit IRC | 04:48 | |
| *** aslezil_ has joined #openstack-nova | 04:58 | |
| *** dikonoor has joined #openstack-nova | 05:07 | |
| *** raunak_ has joined #openstack-nova | 05:08 | |
| *** thorst has joined #openstack-nova | 05:09 | |
| *** raunak has quit IRC | 05:09 | |
| *** raunak_ is now known as raunak | 05:09 | |
| *** adisky_ has joined #openstack-nova | 05:11 | |
| *** gcb has quit IRC | 05:13 | |
| *** thorst has quit IRC | 05:14 | |
| *** pbandark has joined #openstack-nova | 05:15 | |
| *** sudipto_ has quit IRC | 05:17 | |
| *** sudipto has quit IRC | 05:17 | |
| *** hongbin has quit IRC | 05:18 | |
| openstackgerrit | zhangyujun proposed openstack/nova: Update JSON Schema for flavor access https://review.openstack.org/418683 | 05:20 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: DNM: just test https://review.openstack.org/415413 | 05:20 |
| *** tbachman has joined #openstack-nova | 05:21 | |
| *** tbachman has quit IRC | 05:26 | |
| *** yonglihe has quit IRC | 05:27 | |
| *** tbachman has joined #openstack-nova | 05:28 | |
| *** ayogi has joined #openstack-nova | 05:28 | |
| *** prateek has joined #openstack-nova | 05:34 | |
| *** raunak has quit IRC | 05:35 | |
| *** dikonoor has quit IRC | 05:40 | |
| *** aslezil_ has quit IRC | 05:41 | |
| *** sudipto_ has joined #openstack-nova | 05:42 | |
| *** sudipto has joined #openstack-nova | 05:42 | |
| *** ratailor has joined #openstack-nova | 05:45 | |
| *** raunak has joined #openstack-nova | 05:52 | |
| *** gongysh has quit IRC | 05:54 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: WIP(2): Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | 05:58 |
| *** nkrinner_afk is now known as nkrinner | 06:04 | |
| *** gcb has joined #openstack-nova | 06:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 06:05 |
| *** raunak has quit IRC | 06:05 | |
| *** tbachman has quit IRC | 06:05 | |
| *** tlian has quit IRC | 06:06 | |
| *** bkopilov has joined #openstack-nova | 06:06 | |
| *** unicell has joined #openstack-nova | 06:07 | |
| *** tbachman has joined #openstack-nova | 06:08 | |
| alex_xu | oomichi: johnthetubaguy re:https://review.openstack.org/#/c/389314/ I added comment about a solution for deprecate the rule first and remove it next release, if we care about this should be process for removing a policy rule. | 06:08 |
| *** tbachman_ has joined #openstack-nova | 06:09 | |
| *** raunak has joined #openstack-nova | 06:10 | |
| *** lpetrut has joined #openstack-nova | 06:10 | |
| *** tbachman has quit IRC | 06:12 | |
| *** tbachman_ is now known as tbachman | 06:12 | |
| *** gokhan has joined #openstack-nova | 06:13 | |
| *** dims has quit IRC | 06:13 | |
| *** unicell has quit IRC | 06:14 | |
| *** unicell has joined #openstack-nova | 06:14 | |
| *** tovin07 has joined #openstack-nova | 06:16 | |
| *** sacharya has joined #openstack-nova | 06:16 | |
| *** moshele has joined #openstack-nova | 06:16 | |
| *** raunak has quit IRC | 06:16 | |
| *** ijw has joined #openstack-nova | 06:18 | |
| *** sacharya has quit IRC | 06:20 | |
| *** gongysh has joined #openstack-nova | 06:21 | |
| *** tbachman has quit IRC | 06:22 | |
| *** ijw has quit IRC | 06:22 | |
| gokhan | Hi Team, I tried to create virtuozzo containers on nova but I am getting errors.My production is mitaka. I have a compute node which is installed virtuoozo.These are logs http://paste.openstack.org/show/594498/ Can you help me ? | 06:23 |
| *** pcaruana has joined #openstack-nova | 06:25 | |
| *** efried has quit IRC | 06:26 | |
| *** satyar has joined #openstack-nova | 06:31 | |
| *** sridharg has joined #openstack-nova | 06:32 | |
| *** moshele has quit IRC | 06:33 | |
| *** sheel has quit IRC | 06:37 | |
| mnestratov | gokhan:hi | 06:38 |
| mnestratov | I can help you | 06:38 |
| *** efried has joined #openstack-nova | 06:38 | |
| *** gongysh has quit IRC | 06:39 | |
| mnestratov | gokhan: what virtuozzo version do you have? | 06:39 |
| gokhan | mnestratov I have 7.0.1 I installed from https://download.openvz.org/virtuozzo/releases/openvz-7.0.1-554/x86_64/iso/openvz-iso-7.0.1.iso | 06:41 |
| gokhan | and for openstack I followed https://help.virtuozzo.com/customer/portal/articles/2524013 | 06:41 |
| mnestratov | gokhan: so you are trying devstack, right? | 06:43 |
| *** lpetrut has quit IRC | 06:43 | |
| gokhan | mnestratov, no not devstack, I am using it on production | 06:43 |
| mnestratov | gokhan: ok I see | 06:44 |
| *** mjura has joined #openstack-nova | 06:44 | |
| gokhan | mnestratov, I followed also https://openvz.org/Basic_operations_in_OpenVZ_environment for creating container. I have created container successfully. But with openstack, I didn't achieve it. | 06:47 |
| *** ekuris has joined #openstack-nova | 06:49 | |
| *** bhagyashris has joined #openstack-nova | 06:51 | |
| *** ijw has joined #openstack-nova | 06:54 | |
| openstackgerrit | Roman Dobosz proposed openstack/nova-specs: Provide detailed error information for placement API https://review.openstack.org/418393 | 06:54 |
| *** hfu has joined #openstack-nova | 06:56 | |
| *** lpetrut has joined #openstack-nova | 07:00 | |
| mnestratov | gokhan: looks like you need commit d4aa455d53c91c6dfebbf9a9850f7b6c3fef4545 to fix this issue | 07:06 |
| mnestratov | gokhan: review https://review.openstack.org/#/c/182257/ | 07:06 |
| mnestratov | gokhan: there was a lot of changes merged since mitaka regarding virtuozzo support | 07:07 |
| *** lpetrut has quit IRC | 07:08 | |
| *** tesseract has joined #openstack-nova | 07:08 | |
| gokhan | thanks mnestratov, I will try it | 07:09 |
| mnestratov | gokhan: thus, I'd recommend you to use newton | 07:09 |
| *** tjones has quit IRC | 07:09 | |
| *** unicell has quit IRC | 07:10 | |
| *** unicell has joined #openstack-nova | 07:10 | |
| mnestratov | gokhan: did you create a container image yourself or use our test one? | 07:11 |
| gokhan | mnestratov: I used test one | 07:12 |
| mnestratov | gokhan: ok, just wanted to clarify | 07:13 |
| gokhan | mnestratov: ok firstly I will commit these changes . If it doesn't work, I will try newton | 07:16 |
| mnestratov | gokhan: same error? | 07:16 |
| mnestratov | oh I thought you tried already | 07:17 |
| *** lpetrut has joined #openstack-nova | 07:17 | |
| gokhan | no I haven't tried yet. | 07:17 |
| *** udesale has joined #openstack-nova | 07:19 | |
| *** yuntongjin has quit IRC | 07:20 | |
| *** yuntongjin has joined #openstack-nova | 07:20 | |
| *** andreas_s has joined #openstack-nova | 07:21 | |
| *** nherciu has joined #openstack-nova | 07:21 | |
| *** sapcc-bot has joined #openstack-nova | 07:22 | |
| *** carthaca_ has joined #openstack-nova | 07:22 | |
| *** dimtruck is now known as zz_dimtruck | 07:24 | |
| *** carthaca_ has quit IRC | 07:24 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Change parent of diagnostics objects to NovaObject class https://review.openstack.org/394480 | 07:26 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 07:26 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | 07:27 |
| openstackgerrit | avnish proposed openstack/nova: Remove useless ddt requirements https://review.openstack.org/418747 | 07:27 |
| *** amotoki has joined #openstack-nova | 07:28 | |
| *** kaisers has quit IRC | 07:32 | |
| *** lpetrut has quit IRC | 07:33 | |
| *** kaisers1 has quit IRC | 07:33 | |
| *** hwoarang has quit IRC | 07:35 | |
| *** markus_z has joined #openstack-nova | 07:36 | |
| *** moshele has joined #openstack-nova | 07:37 | |
| *** kevinz has quit IRC | 07:40 | |
| *** kevinz has joined #openstack-nova | 07:41 | |
| *** tojuvone has joined #openstack-nova | 07:41 | |
| *** hwoarang has joined #openstack-nova | 07:43 | |
| *** zenoway has joined #openstack-nova | 07:48 | |
| *** ralonsoh has joined #openstack-nova | 07:48 | |
| *** kaisers has joined #openstack-nova | 07:50 | |
| *** kaisers1 has joined #openstack-nova | 07:50 | |
| *** yonglihe has joined #openstack-nova | 07:50 | |
| *** bkopilov has quit IRC | 07:55 | |
| *** hogepodge has quit IRC | 07:56 | |
| *** sahid has joined #openstack-nova | 07:59 | |
| *** sacharya has joined #openstack-nova | 08:00 | |
| *** yuntongjin has quit IRC | 08:02 | |
| *** sapcc-bot has quit IRC | 08:03 | |
| *** sapcc-bot has joined #openstack-nova | 08:03 | |
| *** carthaca_ has joined #openstack-nova | 08:03 | |
| *** yamahata has quit IRC | 08:05 | |
| *** carthaca_ has quit IRC | 08:05 | |
| *** sacharya has quit IRC | 08:06 | |
| *** amotoki has quit IRC | 08:16 | |
| *** owalsh has joined #openstack-nova | 08:18 | |
| *** gcb has quit IRC | 08:19 | |
| *** sudipto has quit IRC | 08:21 | |
| *** salv-orl_ has quit IRC | 08:21 | |
| *** sudipto_ has quit IRC | 08:23 | |
| *** eglynn has joined #openstack-nova | 08:23 | |
| *** sudipto has joined #openstack-nova | 08:23 | |
| *** sudipto_ has joined #openstack-nova | 08:23 | |
| *** lpetrut has joined #openstack-nova | 08:24 | |
| *** voelzmo has joined #openstack-nova | 08:25 | |
| *** voelzmo has quit IRC | 08:28 | |
| *** lyarwood is now known as lyarwood_ | 08:30 | |
| *** gcb has joined #openstack-nova | 08:33 | |
| *** Matias has quit IRC | 08:37 | |
| *** sapcc-bot has quit IRC | 08:37 | |
| *** carthaca_ has joined #openstack-nova | 08:38 | |
| *** dgonzalez_ has joined #openstack-nova | 08:38 | |
| *** sapcc-bot has joined #openstack-nova | 08:38 | |
| *** gcb has quit IRC | 08:40 | |
| *** dgonzalez_ has quit IRC | 08:40 | |
| *** carthaca_ has quit IRC | 08:40 | |
| *** Matias has joined #openstack-nova | 08:43 | |
| *** John341_ has joined #openstack-nova | 08:48 | |
| *** bkopilov has joined #openstack-nova | 08:51 | |
| *** ijw has quit IRC | 08:52 | |
| *** haplo37_ has quit IRC | 08:54 | |
| *** Dave_____ is now known as Dave | 08:55 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 08:57 |
| *** haplo37_ has joined #openstack-nova | 08:57 | |
| *** sridhargaddam has joined #openstack-nova | 08:59 | |
| *** sridharg has quit IRC | 08:59 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-nova | 09:00 | |
| alex_xu | bauzas: johnthetubaguy good morning, I found this patch similar to the force param on evacuate https://review.openstack.org/#/c/408955/11/nova/compute/api.py@3145, but i feel that if the instance without request_spec obj, then the API just don't the thing user expected silently. Does it sounds ok? will Hydrate a fake spec very complex? | 09:12 |
| bauzas | alex_xu: not sure I understand your concern | 09:13 |
| bauzas | alex_xu: FWIW, all the instances are now having a RequestSpec | 09:14 |
| bauzas | since Newton | 09:14 |
| bauzas | because I wrote a DB migration for creating a RequestSpec for all the already created instances | 09:14 |
| alex_xu | bauzas: oops, we have data migration for it? I saw this comment https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3897, just thought it may still have that case | 09:15 |
| alex_xu | bauzas: ah, got it, cool | 09:15 |
| bauzas | alex_xu: yup, sec | 09:15 |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers https://review.openstack.org/312488 | 09:17 |
| bauzas | alex_xu: https://github.com/openstack/nova/commit/09f2d4d5ec3a699176d70c2407ced0ce7cd58197 | 09:19 |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers https://review.openstack.org/312488 | 09:20 |
| *** namnh has quit IRC | 09:20 | |
| *** avolkov has joined #openstack-nova | 09:21 | |
| alex_xu | bauzas: thanks, i see now. we probably just need some cleanup later | 09:22 |
| *** amotoki has joined #openstack-nova | 09:22 | |
| bauzas | alex_xu: what's missing is that we don't delete yet the RequetSpec if the instance is deleted | 09:22 |
| alex_xu | bauzas: is there any blocker? or we just miss it? | 09:25 |
| bauzas | nothing but time :) | 09:26 |
| alex_xu | bauzas: ok :) | 09:26 |
| *** roeyc has joined #openstack-nova | 09:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 09:30 |
| *** mvk has quit IRC | 09:32 | |
| *** lucas-afk is now known as lucasagomes | 09:38 | |
| *** sree has quit IRC | 09:40 | |
| *** BobBall_AWOL is now known as BobBall | 09:40 | |
| *** sree has joined #openstack-nova | 09:41 | |
| *** derekh has joined #openstack-nova | 09:42 | |
| *** mnestratov has quit IRC | 09:44 | |
| openstackgerrit | Abhishek Kekane proposed openstack/nova: Use is_valid_cidr and is_valid_ipv6_cidr from oslo_utils https://review.openstack.org/320375 | 09:44 |
| gmann | alex_xu: why we want to handle _* separately in https://review.openstack.org/#/c/408571/36 | 09:47 |
| gmann | alex_xu: i mean just 400 in server.py will be enough right instead of allowing it in schema and later allow to go from strip_additional_property | 09:48 |
| gmann | alex_xu: also how about treating that same as other additional property, just ignore | 09:49 |
| *** hfu has quit IRC | 09:51 | |
| *** ijw has joined #openstack-nova | 09:52 | |
| lyarwood_ | mdbooth: morning, was anything blocking https://review.openstack.org/#/c/383859/ aside from core reviews? | 09:53 |
| alex_xu | gmann: if we didn't put _* in the schema, they will be stripped out | 09:54 |
| alex_xu | gmann: we have discuss it whether we can just ignore at last api meeting http://eavesdrop.openstack.org/meetings/nova_api/2017/nova_api.2017-01-04-13.00.log.html | 09:56 |
| alex_xu | gmann: start from "13:07:35" | 09:56 |
| alex_xu | gmann: but yes, I'm not sure people like current implement. | 09:57 |
| *** ijw has quit IRC | 09:57 | |
| *** sridharg has joined #openstack-nova | 09:59 | |
| * alex_xu will back after few hours... | 09:59 | |
| *** ijw has joined #openstack-nova | 10:00 | |
| *** lyarwood_ is now known as lyarwood | 10:02 | |
| *** sridhargaddam has quit IRC | 10:02 | |
| gmann | alex_xu: oh yea, for db model object internal attributes. | 10:03 |
| *** lyarwood is now known as lyarwood_ | 10:03 | |
| gmann | alex_xu: but only allow in strip_additional and later 400 should be enough may be instead of explicitly allow in original schema ? | 10:04 |
| *** ijw has quit IRC | 10:05 | |
| *** ayogi has quit IRC | 10:05 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 10:06 |
| *** ayogi has joined #openstack-nova | 10:07 | |
| *** mvk has joined #openstack-nova | 10:10 | |
| *** zenoway has quit IRC | 10:11 | |
| *** zenoway has joined #openstack-nova | 10:12 | |
| *** zenoway has quit IRC | 10:16 | |
| *** udesale has quit IRC | 10:17 | |
| openstackgerrit | zhangyujun proposed openstack/nova: Update JSON Schema for flavor access https://review.openstack.org/418683 | 10:23 |
| *** mdnadeem has quit IRC | 10:24 | |
| *** prateek is now known as Prateek|afk | 10:24 | |
| *** kevinz has quit IRC | 10:27 | |
| *** alexpilotti has quit IRC | 10:27 | |
| *** alexpilotti has joined #openstack-nova | 10:28 | |
| *** satyar has quit IRC | 10:29 | |
| *** carthaca_ has joined #openstack-nova | 10:30 | |
| *** dgonzalez_ has joined #openstack-nova | 10:30 | |
| *** dgonzalez_ has quit IRC | 10:31 | |
| *** carthaca_ has quit IRC | 10:31 | |
| *** rfolco has joined #openstack-nova | 10:32 | |
| *** cristicalin has joined #openstack-nova | 10:34 | |
| *** claudiub has joined #openstack-nova | 10:38 | |
| *** zenoway has joined #openstack-nova | 10:40 | |
| mdbooth | lyarwood_: Actually I've got it open right now. I was writing some docs for it at the suggestion of sfinucan and noticed a race, which I'm addressing. | 10:40 |
| *** mdnadeem has joined #openstack-nova | 10:40 | |
| mdbooth | But basically no. | 10:40 |
| *** claudiub|2 has quit IRC | 10:41 | |
| *** sree has quit IRC | 10:43 | |
| *** sree has joined #openstack-nova | 10:43 | |
| openstackgerrit | Jiri Suchomel proposed openstack/nova: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | 10:44 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 10:45 |
| *** rmart04 has joined #openstack-nova | 10:46 | |
| *** tuanluong has quit IRC | 10:46 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 10:46 |
| *** ygl has joined #openstack-nova | 10:47 | |
| *** ygl has left #openstack-nova | 10:47 | |
| *** aarefiev has joined #openstack-nova | 10:47 | |
| *** sree has quit IRC | 10:48 | |
| *** vladikr has quit IRC | 10:48 | |
| openstackgerrit | avnish proposed openstack/nova: Removed unnecessary utf-8 encoding https://review.openstack.org/418838 | 10:49 |
| openstackgerrit | avnish proposed openstack/python-novaclient: Removed unnecessary utf-8 encoding https://review.openstack.org/418839 | 10:49 |
| *** aarefiev_afk has quit IRC | 10:50 | |
| sfinucan | Morning/afternoon, folks. On this, review day, could I get some eyes on two bug fixes: https://review.openstack.org/#/c/396184/ and https://review.openstack.org/#/c/392643/ | 10:52 |
| sfinucan | Both should be somewhat self explanatory, but I'm here if anyone has any questions | 10:53 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489 | 10:54 |
| *** zenoway has quit IRC | 11:00 | |
| *** gcb has joined #openstack-nova | 11:00 | |
| *** cdent has joined #openstack-nova | 11:00 | |
| *** huanxie has quit IRC | 11:02 | |
| *** ijw has joined #openstack-nova | 11:02 | |
| *** sacharya has joined #openstack-nova | 11:02 | |
| *** sree has joined #openstack-nova | 11:06 | |
| *** zenoway has joined #openstack-nova | 11:06 | |
| *** ijw has quit IRC | 11:07 | |
| *** sacharya has quit IRC | 11:07 | |
| *** CristinaPauna has quit IRC | 11:09 | |
| *** sudipto has quit IRC | 11:15 | |
| *** sudipto_ has quit IRC | 11:15 | |
| *** sdague has joined #openstack-nova | 11:15 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Populate network data in _validate_requested_network_ids https://review.openstack.org/418511 | 11:15 |
| openstackgerrit | John Garbutt proposed openstack/nova: Pass objects into _create_ports_for_instance https://review.openstack.org/418512 | 11:15 |
| openstackgerrit | John Garbutt proposed openstack/nova: Pass objects into _update_ports_for_instance https://review.openstack.org/418513 | 11:15 |
| *** smatzek has joined #openstack-nova | 11:18 | |
| *** CristinaPauna has joined #openstack-nova | 11:19 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers https://review.openstack.org/312488 | 11:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 11:20 |
| *** smatzek has quit IRC | 11:21 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 11:21 |
| *** smatzek has joined #openstack-nova | 11:21 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 11:22 |
| *** bhagyashris has quit IRC | 11:22 | |
| *** sapcc-bot has quit IRC | 11:27 | |
| *** sapcc-bot1 has joined #openstack-nova | 11:27 | |
| *** dgonzalez_ has joined #openstack-nova | 11:27 | |
| *** carthaca_ has joined #openstack-nova | 11:27 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 11:29 |
| alex_xu | gmann: I feel explicity in the schema is better than hide some hacky deep into some method. | 11:29 |
| *** dgonzalez_ has quit IRC | 11:29 | |
| *** carthaca_ has quit IRC | 11:29 | |
| *** nicolasbock has joined #openstack-nova | 11:35 | |
| *** dims has joined #openstack-nova | 11:39 | |
| *** JoseMello has joined #openstack-nova | 11:40 | |
| *** haplo37_ has quit IRC | 11:40 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract preallocate_for_instance https://review.openstack.org/418876 | 11:40 |
| *** cristicalin has quit IRC | 11:41 | |
| *** snikitin_ has joined #openstack-nova | 11:42 | |
| *** haplo37_ has joined #openstack-nova | 11:43 | |
| *** gszasz has joined #openstack-nova | 11:43 | |
| openstackgerrit | zhangyujun proposed openstack/nova: Query parameter validation for hosts API https://review.openstack.org/418880 | 11:49 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization https://review.openstack.org/398509 | 11:49 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 11:50 |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: Call pre_allocate_for_instance before scheduler https://review.openstack.org/413124 | 11:51 |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: introduce preallocate_for_instance method. https://review.openstack.org/418881 | 11:51 |
| *** ociuhandu has joined #openstack-nova | 11:52 | |
| *** claudiub|2 has joined #openstack-nova | 11:54 | |
| *** sapcc-bot1 has quit IRC | 11:56 | |
| *** ratailor has quit IRC | 11:57 | |
| *** claudiub has quit IRC | 11:57 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Make allocate_for_instance take consistent args https://review.openstack.org/418024 | 11:58 |
| *** mvk has quit IRC | 12:03 | |
| *** mpaolino has joined #openstack-nova | 12:03 | |
| *** gszasz has quit IRC | 12:06 | |
| openstackgerrit | zhangyujun proposed openstack/nova: Update JSON Schema for flavor access https://review.openstack.org/418683 | 12:06 |
| *** kevinz has joined #openstack-nova | 12:06 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Pass objects into _validate_requested_network_ids https://review.openstack.org/418027 | 12:07 |
| openstackgerrit | John Garbutt proposed openstack/nova: Add and use ProcessedNetworkRequest object https://review.openstack.org/418025 | 12:07 |
| *** rtheis has joined #openstack-nova | 12:09 | |
| *** zenoway has quit IRC | 12:09 | |
| *** nik__ has joined #openstack-nova | 12:13 | |
| *** amotoki has quit IRC | 12:14 | |
| *** mvk has joined #openstack-nova | 12:15 | |
| *** moshele has quit IRC | 12:16 | |
| *** moshele has joined #openstack-nova | 12:16 | |
| *** zenoway has joined #openstack-nova | 12:17 | |
| *** ekuris has quit IRC | 12:19 | |
| *** moshele has quit IRC | 12:20 | |
| *** ekuris has joined #openstack-nova | 12:20 | |
| *** moshele has joined #openstack-nova | 12:23 | |
| *** bvanhav has joined #openstack-nova | 12:24 | |
| *** amotoki has joined #openstack-nova | 12:26 | |
| *** omnipresent has joined #openstack-nova | 12:26 | |
| *** zenoway has quit IRC | 12:29 | |
| *** yuntongjin has joined #openstack-nova | 12:31 | |
| *** kevinz has quit IRC | 12:31 | |
| *** vladikr has joined #openstack-nova | 12:31 | |
| *** kevinz has joined #openstack-nova | 12:32 | |
| *** gszasz has joined #openstack-nova | 12:34 | |
| *** thorst has joined #openstack-nova | 12:41 | |
| *** bdeschenes has joined #openstack-nova | 12:42 | |
| *** sree_ has joined #openstack-nova | 12:43 | |
| *** edmondsw has joined #openstack-nova | 12:43 | |
| *** ayogi has quit IRC | 12:43 | |
| *** sree_ is now known as Guest36399 | 12:43 | |
| *** gcb has quit IRC | 12:45 | |
| *** sree has quit IRC | 12:45 | |
| *** omnipresent has quit IRC | 12:45 | |
| *** omnipresent has joined #openstack-nova | 12:46 | |
| *** liuyulong_ has joined #openstack-nova | 12:48 | |
| *** kevinz has quit IRC | 12:49 | |
| *** lucasagomes is now known as lucas-hungry | 12:50 | |
| *** mdnadeem has quit IRC | 12:50 | |
| *** kevinz has joined #openstack-nova | 12:50 | |
| *** amotoki has quit IRC | 12:50 | |
| *** liuyulong has quit IRC | 12:51 | |
| *** claudiub has joined #openstack-nova | 12:51 | |
| alex_xu | nova api sub-team meeting is 9mins at #openstack-meeting-4 | 12:51 |
| *** claudiub|2 has quit IRC | 12:54 | |
| *** sudipto has joined #openstack-nova | 12:56 | |
| *** sudipto_ has joined #openstack-nova | 12:56 | |
| gmann | alex_xu: i see point now, for skipping _* from strip_additional we need in schema | 12:57 |
| gmann | alex_xu: thanks :) | 12:57 |
| alex_xu | gmann: yea, but that is a little ugly | 12:58 |
| *** sahid has quit IRC | 12:58 | |
| *** owalsh_ has joined #openstack-nova | 12:58 | |
| Kevin_Zheng | lol | 12:59 |
| gmann | alex_xu true, that's side effect of maintaining lot of versions and supported codes :) | 12:59 |
| *** owalsh has quit IRC | 12:59 | |
| *** owalsh_ has quit IRC | 12:59 | |
| alex_xu | gmann: yea :) | 12:59 |
| *** owalsh_ has joined #openstack-nova | 12:59 | |
| *** amotoki has joined #openstack-nova | 13:00 | |
| *** rfolco has quit IRC | 13:01 | |
| *** amotoki has quit IRC | 13:02 | |
| *** smatzek has quit IRC | 13:02 | |
| *** sacharya has joined #openstack-nova | 13:03 | |
| *** omnipresent has quit IRC | 13:04 | |
| *** LeoZhang has joined #openstack-nova | 13:04 | |
| *** zenoway has joined #openstack-nova | 13:04 | |
| *** liverpooler has quit IRC | 13:05 | |
| *** amotoki has joined #openstack-nova | 13:05 | |
| *** sacharya has quit IRC | 13:08 | |
| *** omnipresent has joined #openstack-nova | 13:08 | |
| *** tovin07_ has joined #openstack-nova | 13:09 | |
| *** zenoway has quit IRC | 13:09 | |
| *** LeoZhang has quit IRC | 13:10 | |
| *** LeoZhang has joined #openstack-nova | 13:10 | |
| *** smatzek has joined #openstack-nova | 13:12 | |
| *** ducttape_ has quit IRC | 13:14 | |
| *** tovin07 has quit IRC | 13:15 | |
| *** tovin07_ is now known as tovin07 | 13:15 | |
| *** tovin07_ has joined #openstack-nova | 13:15 | |
| *** ducttape_ has joined #openstack-nova | 13:17 | |
| *** mvk has quit IRC | 13:21 | |
| *** omnipresent has quit IRC | 13:21 | |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance-exists notification https://review.openstack.org/403660 | 13:24 |
| *** hfu has joined #openstack-nova | 13:26 | |
| *** efried has quit IRC | 13:27 | |
| *** amotoki has quit IRC | 13:28 | |
| bauzas | artom: I have a question for you in https://review.openstack.org/#/c/415520/5 | 13:29 |
| bauzas | (reviewing all the blueprints for the review day) | 13:29 |
| bauzas | artom: at least, I put -1 because of the confusing docstring | 13:29 |
| bauzas | artom: heh, jinx for the recheck :) | 13:32 |
| artom | bauzas, aha! Yeah, that probably makes sense | 13:32 |
| artom | bauzas, heh, a lot of volumes tests failed on that one, so I got worried, but it looks like it's an unrelated MySQL failure and not my code | 13:33 |
| bauzas | zactmy | 13:33 |
| bauzas | artom: about the first change, amirite if I think you just need to signal something ? | 13:33 |
| *** mvk has joined #openstack-nova | 13:33 | |
| bauzas | you don't really care about the value itsefl | 13:33 |
| bauzas | if so, is the tag info something we can get from the vif info ? | 13:34 |
| bauzas | artom: ^ | 13:34 |
| artom | bauzas, I don't think it's in vif info... | 13:34 |
| bauzas | okay | 13:34 |
| *** bdeschenes has quit IRC | 13:35 | |
| bauzas | if so, we need it in the signature indeed | 13:35 |
| artom | Vif info is that weird network model class, right? If so (and IIRC it is) it's not in there | 13:35 |
| *** mpaolino has quit IRC | 13:35 | |
| artom | bauzas, strictly speaking, we need it as a boolean flag, so... just check whether it's in kwargs? You think that'd be better? | 13:36 |
| *** CristinaPauna has quit IRC | 13:36 | |
| bauzas | artom: I'm thinking about the interface | 13:37 |
| bauzas | I mean that method signature | 13:37 |
| bauzas | and whether a single boolean is all that good | 13:37 |
| bauzas | I tend to avoid adding extra arguments just for a single conditional | 13:37 |
| *** jheroux has joined #openstack-nova | 13:37 | |
| bauzas | so I tend to regroup all the needed into objects | 13:38 |
| artom | Objects or dicts? Because we know how the latter worked out ;) | 13:38 |
| bauzas | when you can actually lookup the needed within the methods | 13:38 |
| bauzas | nah, objects | 13:38 |
| bauzas | I mean | 13:38 |
| bauzas | just amending a virt signature for passing a boolean is a huge thing | 13:38 |
| artom | Yeah, I agree with that | 13:39 |
| *** swamireddy has quit IRC | 13:39 | |
| *** amotoki has joined #openstack-nova | 13:39 | |
| *** aloga has quit IRC | 13:39 | |
| bauzas | that's why I'm trying to understand whether we can either #1 get that elsewhere but not passed in as arg, or #2 pass a better argument that would help more than just a boolean | 13:39 |
| *** mdrabe has joined #openstack-nova | 13:40 | |
| artom | I guess we can add it as a field in the vif info? | 13:40 |
| bauzas | well | 13:40 |
| bauzas | if so, that means a new os-vif version, right ? | 13:40 |
| *** owalsh_ is now known as owalsh-lunch | 13:40 | |
| artom | Also, other virt drivers might need it, I suppose | 13:40 |
| *** satyar has joined #openstack-nova | 13:40 | |
| artom | And we pass the tag itself from the compute manager, not just a boolean flag | 13:40 |
| *** lucas-hungry has quit IRC | 13:41 | |
| *** hfu has quit IRC | 13:41 | |
| artom | bauzas, hold up, daughter's awake, back a bit later | 13:42 |
| bauzas | np | 13:43 |
| *** Jeffrey4l_ has quit IRC | 13:45 | |
| *** huanxie has joined #openstack-nova | 13:45 | |
| *** annegentle has joined #openstack-nova | 13:46 | |
| *** bvanhav_ has joined #openstack-nova | 13:46 | |
| *** bvanhav has quit IRC | 13:47 | |
| *** lucasagomes has joined #openstack-nova | 13:47 | |
| *** yuntongjin has quit IRC | 13:51 | |
| *** winstonsullen has joined #openstack-nova | 13:51 | |
| *** snikitin_ has quit IRC | 13:52 | |
| *** ethenejumpy has joined #openstack-nova | 13:52 | |
| *** tyrola has quit IRC | 13:53 | |
| huanxie | Hi sdague, I have rebased my nova patch of using os-xenapi https://review.openstack.org/#/c/406059/, could you help review again? This patch already got +2/+w before, but there is a unit test conflict with upstream, so I rebased and fixed, now it's ready. Thanks a lot :) | 13:53 |
| *** aloga has joined #openstack-nova | 13:54 | |
| *** tyrola has joined #openstack-nova | 13:54 | |
| *** CristinaPauna has joined #openstack-nova | 13:55 | |
| *** owalsh-lunch has quit IRC | 13:56 | |
| *** david-lyle has quit IRC | 13:56 | |
| *** tongli has joined #openstack-nova | 13:59 | |
| *** amotoki has quit IRC | 13:59 | |
| *** david-lyle has joined #openstack-nova | 13:59 | |
| *** mat128 has quit IRC | 13:59 | |
| *** baoli has joined #openstack-nova | 13:59 | |
| *** baoli has quit IRC | 14:00 | |
| *** tblakes has joined #openstack-nova | 14:00 | |
| *** mat128 has joined #openstack-nova | 14:00 | |
| *** baoli has joined #openstack-nova | 14:00 | |
| *** lamt has joined #openstack-nova | 14:01 | |
| *** links has quit IRC | 14:02 | |
| *** kylek3h has quit IRC | 14:03 | |
| *** mriedem has joined #openstack-nova | 14:03 | |
| *** Prateek|afk has quit IRC | 14:03 | |
| *** kaisers_ has joined #openstack-nova | 14:04 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Change parent of diagnostics objects to NovaObject class https://review.openstack.org/394480 | 14:04 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 14:04 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | 14:04 |
| *** diga has quit IRC | 14:06 | |
| huanxie | Hi mriedem, I have rebased my patch https://review.openstack.org/#/c/406059/ which is to use os-xenapi lib, could you help have a review again? Thanks a lot! | 14:08 |
| *** claudiub|2 has joined #openstack-nova | 14:12 | |
| sdague | huanxie: no prob | 14:13 |
| *** tlian has joined #openstack-nova | 14:13 | |
| huanxie | sdague :) | 14:13 |
| *** ducttape_ has quit IRC | 14:14 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Add and use ProcessedNetworkRequest object https://review.openstack.org/418025 | 14:14 |
| openstackgerrit | John Garbutt proposed openstack/nova: Make allocate_for_instance take consistent args https://review.openstack.org/418024 | 14:14 |
| *** jamielennox is now known as jamielennox|away | 14:14 | |
| *** claudiub has quit IRC | 14:16 | |
| *** liverpooler has joined #openstack-nova | 14:16 | |
| *** xinliang has quit IRC | 14:16 | |
| *** david-lyle has quit IRC | 14:16 | |
| *** annegentle has quit IRC | 14:18 | |
| cdent | Kevin_Zheng: are you still working on https://review.openstack.org/#/c/410128/ ? | 14:19 |
| *** winstonsullen has quit IRC | 14:20 | |
| *** claudiub has joined #openstack-nova | 14:20 | |
| *** Swami has joined #openstack-nova | 14:21 | |
| *** zz_dimtruck is now known as dimtruck | 14:22 | |
| *** udesale has joined #openstack-nova | 14:22 | |
| *** claudiub|2 has quit IRC | 14:23 | |
| *** karthiks has quit IRC | 14:26 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Update instance group during live migration https://review.openstack.org/339588 | 14:26 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: live-migration test hook for serial console https://review.openstack.org/347471 | 14:27 |
| *** xinliang has joined #openstack-nova | 14:30 | |
| *** dave-mccowan has joined #openstack-nova | 14:31 | |
| *** kevinz has quit IRC | 14:31 | |
| *** tblakes has quit IRC | 14:32 | |
| *** jgrassler has joined #openstack-nova | 14:33 | |
| *** amotoki has joined #openstack-nova | 14:39 | |
| *** lamt has quit IRC | 14:40 | |
| *** Guest36399 has quit IRC | 14:42 | |
| *** huanxie has quit IRC | 14:44 | |
| *** ducttape_ has joined #openstack-nova | 14:45 | |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP: Test cellsv1 https://review.openstack.org/418141 | 14:46 |
| *** jwcroppe has joined #openstack-nova | 14:46 | |
| mriedem | i smell a feature review sprint coming on https://etherpad.openstack.org/p/nova-ocata-feature-freeze | 14:46 |
| bauzas | dansmith: I stopped reviewing cellsv1 effort since a while | 14:47 |
| *** owalsh-lunch has joined #openstack-nova | 14:47 | |
| bauzas | dansmith: is there some change I should first look at | 14:47 |
| bauzas | ? | 14:47 |
| dansmith | bauzas: that patch is trying to fix cellsv1 when cellsv2 is enabled | 14:47 |
| dansmith | and it's still a ways off from working, so.. ignore it for now | 14:47 |
| bauzas | mriedem: FWIW, looking at the BP whiteboards is somehow difficult | 14:47 |
| bauzas | dansmith: okay, that was because today means I should help others | 14:48 |
| *** owalsh-lunch is now known as owalsh | 14:48 | |
| *** huanxie has joined #openstack-nova | 14:48 | |
| *** dimtruck is now known as zz_dimtruck | 14:49 | |
| *** tongli has quit IRC | 14:49 | |
| dansmith | bauzas: yeah, you should do that instead of looking at my not-working patch :) | 14:49 |
| bauzas | roger. | 14:49 |
| bauzas | dansmith: today's cells meeting is at 2100UTC, right? | 14:50 |
| *** huanxie has quit IRC | 14:50 | |
| bauzas | I *need* to attend that one after lot of times missing it | 14:50 |
| dansmith | bauzas: are you asking me to be your calendar? today's meeting is in the later slot. | 14:50 |
| *** huanxie has joined #openstack-nova | 14:50 | |
| bauzas | dansmith: no, just verifying my own calendar that I don't trust :) | 14:51 |
| *** cleong has joined #openstack-nova | 14:51 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: xenapi: support the hotplug of a neutron port https://review.openstack.org/365505 | 14:52 |
| *** gouthamr has joined #openstack-nova | 14:52 | |
| mriedem | bauzas: how would you like me to custom tailor the etherpad to your delicate sensibilities? :) | 14:52 |
| mriedem | bauzas: https://review.openstack.org/#/c/365505/ needs another core | 14:53 |
| bauzas | mriedem: nothing, just a comment :) | 14:53 |
| bauzas | mriedem: we should think about providing a dashboard for the next cycle | 14:53 |
| bauzas | but meh | 14:53 |
| mriedem | huanxie: got it | 14:54 |
| *** tbachman has joined #openstack-nova | 14:54 | |
| huanxie | mriedem thanks! | 14:55 |
| mriedem | huanxie: note that i already updated the release note on the vif hot plugging patch | 14:55 |
| mriedem | so that's ready to go | 14:55 |
| *** zhangyujun has quit IRC | 14:56 | |
| *** Jinxing has joined #openstack-nova | 14:58 | |
| *** ljxiash has joined #openstack-nova | 14:58 | |
| *** zhangyujun has joined #openstack-nova | 15:00 | |
| huanxie | mriedem, I was just trying to ssh to my linux and change this, you have already done, many many thanks:) | 15:01 |
| mriedem | huanxie: in case you haven't done it yet, you can use the edit button in the gerrit UI, or the 'fix' button next to a comment and make changes in the gerrit UI directly | 15:01 |
| mriedem | that will make it diverge from your local copy so i only do it for trivial things that are ready to get approved | 15:02 |
| *** tblakes has joined #openstack-nova | 15:02 | |
| *** efried has joined #openstack-nova | 15:04 | |
| huanxie | mridem, I see, I haven't modified directly in gerrit UI, will have a try in my other patch, it's useful for very small change | 15:04 |
| *** sacharya has joined #openstack-nova | 15:04 | |
| *** zhangyujun has quit IRC | 15:06 | |
| *** hongbin has joined #openstack-nova | 15:06 | |
| *** prateek has joined #openstack-nova | 15:06 | |
| cdent | anywhere in particular I should put my eyes for sprinting, or just let the etherpad be my guide? | 15:07 |
| *** moshele has quit IRC | 15:07 | |
| *** sacharya has quit IRC | 15:08 | |
| huanxie | mriedem could you also help review this patch https://review.openstack.org/#/c/406059/, this patch is the first one when using os-xenapi, thanks a lot | 15:08 |
| *** nkrinner is now known as nkrinner_afk | 15:09 | |
| mriedem | huanxie: that's already approved | 15:09 |
| mriedem | cdent: the diagnostics series could probably use a fresh set of eyes | 15:10 |
| mriedem | as i think it was recently rebased | 15:10 |
| mriedem | cdent: https://review.openstack.org/#/q/topic:bp/restore-vm-diagnostics,n,z | 15:10 |
| cdent | thanks mriedem | 15:11 |
| *** rama_y has joined #openstack-nova | 15:11 | |
| *** claudiub|2 has joined #openstack-nova | 15:12 | |
| huanxie | mriedem, I see, remember each patch needs to get two core reviewers +2 or one +2 is enough? | 15:12 |
| *** zhangyujun has joined #openstack-nova | 15:12 | |
| mriedem | huanxie: the bottom change was already approved, you just rebased it so sdague fast-approved it again | 15:13 |
| mriedem | which is ok | 15:13 |
| *** jaosorior has joined #openstack-nova | 15:13 | |
| *** psachin has quit IRC | 15:14 | |
| *** pradk has joined #openstack-nova | 15:15 | |
| *** claudiub has quit IRC | 15:16 | |
| huanxie | mriedem, oh, I see, thanks for explaining this | 15:16 |
| *** david-lyle has joined #openstack-nova | 15:17 | |
| *** kylek3h has joined #openstack-nova | 15:17 | |
| mriedem | raj_singh: some easy things to cleanup in https://review.openstack.org/#/c/397399/ - mostly remove the reset_globals() thing in service_auth.py | 15:17 |
| mriedem | raj_singh: we don't want to repeat that pattern | 15:17 |
| mriedem | raj_singh: and on the devstack change https://review.openstack.org/#/c/409329/ that needs to depend on the nova change which defines the config | 15:19 |
| raj_singh | mriedem: ok, will respin it now. Thx for the review | 15:19 |
| artom | bauzas, heya, back now. So yeah, I agree the method signature is not pretty, but I'm not sure I have a better idea | 15:20 |
| *** zz_dimtruck is now known as dimtruck | 15:21 | |
| bauzas | artom: yup, I understand | 15:21 |
| bauzas | artom: me too honestly | 15:21 |
| bauzas | fortunately, it's not a RPC API | 15:21 |
| *** prateek has quit IRC | 15:21 | |
| *** david-lyle has quit IRC | 15:21 | |
| bauzas | so probably we could just add it now and discuss later on how to have a better argument | 15:21 |
| artom | bauzas, I'll definitely update the docstring though | 15:22 |
| *** yamahata has joined #openstack-nova | 15:22 | |
| mriedem | gibi: did you have a bug for the versioned notifications lazy-load issue? | 15:23 |
| *** marst has joined #openstack-nova | 15:23 | |
| *** huanxie has quit IRC | 15:23 | |
| *** ekuris has quit IRC | 15:24 | |
| gibi | mriedem: yes | 15:24 |
| gibi | mriedem: here it is https://review.openstack.org/#/c/415857/ | 15:24 |
| mriedem | thanks | 15:25 |
| *** sacharya has joined #openstack-nova | 15:25 | |
| *** READ10 has joined #openstack-nova | 15:26 | |
| *** raj_singh_ has joined #openstack-nova | 15:26 | |
| *** raj_singh_ has left #openstack-nova | 15:27 | |
| *** prateek has joined #openstack-nova | 15:27 | |
| *** raj_singh_ has joined #openstack-nova | 15:27 | |
| *** ljxiash has quit IRC | 15:27 | |
| *** markvoelker has quit IRC | 15:28 | |
| *** baoli has quit IRC | 15:28 | |
| cdent | snikitin: I left these as a comment, but how come the new NovaObject based DiskDiagnostics doesn't get an id, when the old one did? | 15:29 |
| *** markvoelker has joined #openstack-nova | 15:29 | |
| *** sridharg has quit IRC | 15:31 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Pass objects into _validate_requested_network_ids https://review.openstack.org/418027 | 15:32 |
| *** nklenke_ has joined #openstack-nova | 15:33 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Plumbing for tagged nic attachment https://review.openstack.org/415622 | 15:34 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | 15:34 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Plumbing for tagged volume attachment https://review.openstack.org/391941 | 15:34 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: API support for tagged device attachment https://review.openstack.org/376021 | 15:34 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | 15:34 |
| *** markvoelker has quit IRC | 15:35 | |
| openstackgerrit | Christopher Martin proposed openstack/nova: Stop using mox in test_compute_cells.py https://review.openstack.org/389750 | 15:35 |
| *** udesale has quit IRC | 15:36 | |
| *** raunak has joined #openstack-nova | 15:37 | |
| *** prateek has quit IRC | 15:37 | |
| *** prateek has joined #openstack-nova | 15:38 | |
| *** raunak has quit IRC | 15:40 | |
| *** Jeffrey4l has joined #openstack-nova | 15:40 | |
| *** lucasagomes has quit IRC | 15:44 | |
| *** lucasagomes has joined #openstack-nova | 15:46 | |
| *** hamzy has joined #openstack-nova | 15:46 | |
| artom | mriedem, heya - you mentioned bumping the RPC version for the tagged attachment patches. I'm wondering whether I squish both volume and nic stuff into one patch and have a bigger patch but only one version bump, or keep it split for easier to digest patches but 2 version bumps (one for each, nic and volume) | 15:47 |
| *** Swami_ has joined #openstack-nova | 15:47 | |
| artom | s/squish/squash/ (though squish is funnier) | 15:47 |
| *** prateek has quit IRC | 15:47 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Refactor _post_live_migratiion https://review.openstack.org/413021 | 15:48 |
| *** mvk has quit IRC | 15:48 | |
| *** burt has joined #openstack-nova | 15:49 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Add service_token for nova-cinder interaction https://review.openstack.org/397399 | 15:49 |
| *** Swami has quit IRC | 15:49 | |
| mriedem | cdent: that came up in the spec review, i asked the same thing, i think it was ultimately decided that the disk id wasn't useful, but we could add one in later if needed, i.e. via bdm uuid or something | 15:50 |
| *** Sukhdev has joined #openstack-nova | 15:50 | |
| cdent | so how do you tell one disk from another? | 15:50 |
| mriedem | cdent: https://review.openstack.org/#/c/357884/ if it helps | 15:50 |
| * cdent looks | 15:51 | |
| mriedem | cdent: yeah i asked the same questions in the spec review | 15:51 |
| mriedem | would have to go back through the conversation on that review to get the answer | 15:51 |
| cdent | heh :) | 15:51 |
| * cdent is reading | 15:51 | |
| *** aslezil_ has joined #openstack-nova | 15:51 | |
| *** armax has joined #openstack-nova | 15:52 | |
| cdent | the gist appears to be: we never had good info there in the first place, and making it could be expensive, so we'll get rid of it for now | 15:52 |
| *** eglynn has quit IRC | 15:52 | |
| mriedem | cdent: https://review.openstack.org/#/c/357884/9/specs/ocata/approved/restore-vm-diagnostics.rst@142 | 15:52 |
| cdent | sort-of-jinx | 15:52 |
| mriedem | i'm pretty sure we also asked if/how people were using it in the ops list and didn't get much of a response | 15:53 |
| mriedem | so we're kind of flying blind | 15:53 |
| *** ethenejumpy has quit IRC | 15:54 | |
| *** shaner has quit IRC | 15:54 | |
| *** shaner has joined #openstack-nova | 15:54 | |
| mriedem | there was some response i guess http://lists.openstack.org/pipermail/openstack-operators/2016-October/011793.html | 15:54 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: WIP: Add service_token for nova-glance interaction https://review.openstack.org/412634 | 15:55 |
| mriedem | cdent: so i guess people use it as a poor mans way of bypassing ceilometer? | 15:55 |
| mriedem | even though it's entire existence in the first place is for ceilometer i believe | 15:55 |
| cdent | apparently ceilometer is now going to be using something to bypass nova ;) | 15:55 |
| mriedem | dragon added it way back when | 15:55 |
| mriedem | nice | 15:55 |
| cdent | yeah, chaos everywhere, cats and dogs, etc | 15:56 |
| *** markvoelker has joined #openstack-nova | 15:57 | |
| *** claudiub has joined #openstack-nova | 15:59 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add new configuration option live_migration_scheme https://review.openstack.org/410817 | 16:01 |
| *** claudiub|2 has quit IRC | 16:02 | |
| *** pbandark has quit IRC | 16:02 | |
| *** lamt has joined #openstack-nova | 16:03 | |
| *** ijw has joined #openstack-nova | 16:06 | |
| *** salv-orlando has joined #openstack-nova | 16:09 | |
| *** mjura has quit IRC | 16:10 | |
| *** ijw has quit IRC | 16:11 | |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP: Test cellsv1 https://review.openstack.org/418141 | 16:12 |
| *** annegentle has joined #openstack-nova | 16:13 | |
| *** dtp has joined #openstack-nova | 16:13 | |
| *** tovin07 has quit IRC | 16:15 | |
| *** tblakes has quit IRC | 16:15 | |
| mriedem | wow, sudafed + 2 cups of coffee and i feel like a changed man | 16:15 |
| mriedem | thanks drugs! | 16:16 |
| *** bkopilov has quit IRC | 16:16 | |
| *** tblakes has joined #openstack-nova | 16:16 | |
| dansmith | sudafed ftw | 16:16 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Update notification for flavor https://review.openstack.org/413846 | 16:16 |
| artom | Breaking news: open source project leader secretly a drug addict. | 16:21 |
| *** jaypipes has joined #openstack-nova | 16:21 | |
| cdent | I think we'd be more surprised about an open source project leader that wasn't a drug addict | 16:21 |
| *** salv-orl_ has joined #openstack-nova | 16:22 | |
| *** Jinxing has quit IRC | 16:23 | |
| bauzas | I need to be AFK for a few hours but will be back around 2000UTC | 16:23 |
| gibi | mriedem: regarding https://review.openstack.org/#/c/413846/ not only flavor.project but flavor.extra_specs is also missing from the emitted payload | 16:24 |
| gibi | mriedem: but it is expected to be solved by the same bug fix | 16:25 |
| bauzas | in the meatime, I'd appreciate some reviews for https://review.openstack.org/#/c/392569/25 | 16:25 |
| bauzas | ^ placement API microversion for RP get | 16:25 |
| *** salv-orlando has quit IRC | 16:25 | |
| bauzas | folks , \o | 16:25 |
| markus_z | mriedem: Hand on heart, the newton backports for fixing live-migration with serial console won't make it, right? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton+topic:%22bug+1455252%22 | 16:26 |
| markus_z | I will only cry a little if you say "yes" | 16:26 |
| mriedem | gibi: you forgot that flavor.extra_specs is loaded up by default when we pull the flavor from the database https://github.com/openstack/nova/blob/fd91a25324b7b7fed1f3f10c5b56668b20a170f4/nova/objects/flavor.py#L402 | 16:26 |
| *** bvanhav_ has quit IRC | 16:26 | |
| mriedem | gibi: that one is the special snowflake | 16:27 |
| mriedem | markus_z: hand on whose heart? mine or yours? | 16:27 |
| *** rama_y has quit IRC | 16:27 | |
| markus_z | mine of course ;P (probably I got the idiom wrong) | 16:27 |
| *** bkopilov has joined #openstack-nova | 16:28 | |
| *** mdrabe has quit IRC | 16:28 | |
| gibi | mriedem: but it is still not shows up in the flavor.create payload | 16:28 |
| gibi | mriedem: which seems contradicting to me | 16:29 |
| *** bvanhav_ has joined #openstack-nova | 16:29 | |
| mriedem | markus_z: i just have reservations about that one given the size of the series, the conflicts, and all of the kerfuffle around that stuff in newton, like i thought there were other things going on related to that series in newton, - it was just really hard to sort out at the time | 16:29 |
| mriedem | markus_z: so i'd need someone that was much closer to how all of that went down in newton to vouch for it, like pkoniszewski | 16:29 |
| mriedem | gibi: well flavor create is different | 16:29 |
| mriedem | the known issue in that reno is talking about the instance payload | 16:30 |
| mriedem | not flavor.create | 16:30 |
| mriedem | instance payload will have the flavor sub-object based on Flavor.get_by_* | 16:30 |
| mriedem | which loads extra_specs | 16:30 |
| mriedem | hmm, why are we running a functional py35 job when it's just going to always fail/ | 16:31 |
| mriedem | seems like a waste of resources | 16:31 |
| mriedem | dims: ^? since gcb isn't around. | 16:31 |
| gibi | mriedem: sorry I still not get it. neither flavor.create nor instance.<action> samples shows the flavor.extra_specs but your reference points out that it is loaded. This seems a bug to me | 16:32 |
| markus_z | pkoniszewski: Do you see a chance to have a look at those patches for stable newton? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton+topic:%22bug+1455252%22 When there are concerns about the risks, I'm fully OK with abandoning them. | 16:32 |
| mriedem | gibi: https://review.openstack.org/#/c/413846/10/doc/notification_samples/instance-create-end.json | 16:33 |
| mriedem | that has the extra_specs key in the payload | 16:33 |
| mriedem | it's just empty, because the fake flavor used in those tests don't have any extra specs :) | 16:33 |
| nik__ | cdent, thank you for review! I fixed you comments | 16:33 |
| *** nik__ is now known as snikitin_ | 16:34 | |
| cdent | cool, will go back and have a look | 16:34 |
| gibi | mriedem: you are right my eyes ticked me | 16:35 |
| mriedem | gibi: https://github.com/openstack/nova/blob/fd91a25324b7b7fed1f3f10c5b56668b20a170f4/nova/tests/functional/notification_sample_tests/notification_sample_base.py#L138 | 16:35 |
| gibi | mriedem: instance.<action> has the extra spec | 16:35 |
| mriedem | yeah, so if we added an extra spec to that fake flavor those instance samples would have it | 16:35 |
| mriedem | it might be a nice clarification to add that in a separate change | 16:35 |
| mriedem | hw:watchdog_action=disable or something | 16:36 |
| *** mdrabe has joined #openstack-nova | 16:37 | |
| artom | mriedem, I guess you missed my question earlier about 2 rpc versions bumps but smaller patches vs 1 bump but bigger patch in the tagged attachment series. Any opinions, or shall I plow on with what I think is best and let code review sort it out later? | 16:37 |
| gibi | mriedem: I agree, let's add that in a separate patch | 16:37 |
| *** cdent has quit IRC | 16:38 | |
| gibi | mriedem: so what is left over the existing reno limitation is that the flavor.create doesn't have the projects and the extra_specs field but I assume that the lazy-load bugfix will solve that as well | 16:39 |
| *** andreas_s has quit IRC | 16:39 | |
| mriedem | gibi: do you feel the need to update that release note to get into that amount of detail? | 16:40 |
| mriedem | gibi: on flavor create i don't think we have access or extra specs anyway, as those are different APIs to update an existing flavor | 16:40 |
| gibi | mriedem: I think the current patch can go in, worst case we add another reno if the bug fix does not solve the flavor.create | 16:41 |
| mriedem | gibi: ok but like i said, flavor.create won't have access or extra specs, | 16:41 |
| mriedem | those aren't available when the flavor is created | 16:41 |
| mriedem | those are later POST actions on an existing flavor | 16:41 |
| pkoniszewski | markus_z: yeah, added series to my TODO list for tomorrow | 16:41 |
| mriedem | so those would be update notifications | 16:41 |
| mriedem | artom: i think i'd prefer separate | 16:42 |
| mriedem | artom: that series is easier to review when we're dealing with bdms and vifs separately | 16:42 |
| mriedem | but that's just me | 16:42 |
| artom | mriedem, I think I agree | 16:43 |
| gibi | mriedem: I agree that the flavor.extra_spec is empty at flavor.create but that empty dict still should be in the emited flavor.create payload | 16:43 |
| artom | mriedem, human time > computer time | 16:43 |
| artom | Or in this case, computer complexity | 16:43 |
| * artom thanks mriedem for the validation :) | 16:43 | |
| markus_z | pkoniszewski: Thanks | 16:43 |
| *** roeyc has quit IRC | 16:44 | |
| diana_clarke | mriedem, sdague, alex_xu: I'm not sure if you saw my last comment here, but I'm not sure this should be shipped as-is. Thoughts? https://review.openstack.org/#/c/398755/ | 16:44 |
| *** FL1SK has quit IRC | 16:45 | |
| sdague | diana_clarke: it would be nice to make that a better error message | 16:45 |
| dansmith | mriedem: sdague: do you know what tox target I should run in tempest to be like what the cells job gets in the gate? if I do -efull I get ~100 more tests than we run | 16:46 |
| mriedem | diana_clarke: hmm, i hadn't seen that | 16:46 |
| mriedem | dansmith: you need to use the rc file in the nova tree | 16:46 |
| mriedem | dansmith: https://github.com/openstack/nova/blob/master/devstack/tempest-dsvm-cells-rc | 16:47 |
| mriedem | sec, | 16:47 |
| *** FL1SK has joined #openstack-nova | 16:47 | |
| mriedem | that is sourced here https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate-wrap.sh#L601 | 16:48 |
| *** moshele has joined #openstack-nova | 16:48 | |
| mriedem | dansmith: and then used here https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L789 | 16:49 |
| dansmith | mriedem: do I need to re-run devstack for that or can I just run tempest and point it at the regex? | 16:49 |
| dansmith | ah | 16:49 |
| gibi | mriedem: I just cherry-picked the bugfix top of the flavor update patch and the notificaion sample test blows up that the extra_spec field of the flavore in None and the extra_specs field of the falvor payload cannot be none. So I think we have an issue here | 16:49 |
| mriedem | dansmith: are you talking local runs? | 16:49 |
| dansmith | mriedem: yeah | 16:49 |
| mriedem | dansmith: then i think you just source that rc file to get the env variable exported, | 16:49 |
| mriedem | and then tox -eall $DEVSTACK_GATE_TEMPEST_REGEX | 16:50 |
| mriedem | well: tox -e all -- $DEVSTACK_GATE_TEMPEST_REGEX | 16:50 |
| dansmith | oh okay | 16:50 |
| dansmith | confusing | 16:50 |
| *** nherciu has quit IRC | 16:51 | |
| gibi | mriedem: worst case I have to bump the version of the falvor payload to make the extra_specs nullable | 16:51 |
| *** cdent has joined #openstack-nova | 16:51 | |
| gibi | mriedem: is it still OK as a bugfix? | 16:51 |
| mriedem | dansmith: alternatively, just do what the cellsv1 job does | 16:51 |
| mriedem | http://logs.openstack.org/55/398755/18/check/gate-tempest-dsvm-cells-ubuntu-xenial/22625aa/console.html.gz#_2017-01-04_09_21_05_122446 | 16:51 |
| dansmith | it's working with sourcing that thing | 16:51 |
| dansmith | thanks | 16:51 |
| mriedem | diana_clarke: maybe create a bug to track the error message cruft | 16:52 |
| mriedem | gibi: hmm, 'extra_specs': fields.DictOfStringsField(), so that's not nullable in the Flavor object itself | 16:53 |
| mriedem | so i imagine that's why it's not nullable in the flavor payload either | 16:53 |
| mriedem | gibi: is the api sample test setting it to None? | 16:53 |
| mriedem | because that would be an invalid test | 16:53 |
| diana_clarke | mriedem: okie dokie | 16:53 |
| gibi | mriedem: checking.. | 16:54 |
| *** tblakes has quit IRC | 16:55 | |
| *** dikonoor has joined #openstack-nova | 16:57 | |
| *** tblakes has joined #openstack-nova | 16:57 | |
| gibi | mriedem: as far as I see all the api sample tests are using flavors that has the extra_specs set to someting. This is possible because flavor.extra_specs are returned by a different api resource than the normal flavor object | 16:57 |
| gibi | mriedem: so in a normal GET os-flavors there is no extra_specs ever | 16:58 |
| *** tblakes_ has joined #openstack-nova | 17:01 | |
| gibi | mriedem: and the GET os-flavor-extra-specs will call objects.Flavor.get_by_flavor_id | 17:01 |
| *** tblakes has quit IRC | 17:02 | |
| *** tblakes_ is now known as tblakes | 17:02 | |
| melwitt | dansmith, mriedem: I was just doing like the gate to run locally, just have to put the regex in single quotes: tempest run --regex '<regex stuff>' | 17:02 |
| gibi | mriedem: so that works the same way as the code that emits the instance.<action> notifications | 17:02 |
| diana_clarke | mriedem, sdague, alex_xu: https://bugs.launchpad.net/nova/+bug/1655710 | 17:03 |
| openstack | Launchpad bug 1655710 in OpenStack Compute (nova) "Query parameter validation using json schema - error messages" [Undecided,New] | 17:03 |
| *** READ10 has quit IRC | 17:04 | |
| mriedem | thanks | 17:05 |
| mriedem | diana_clarke: so do you get a 400 or a 500 now? | 17:05 |
| mriedem | i assume you still get a 400 response | 17:05 |
| diana_clarke | mriedem: not sure off the top of my head, but I can check | 17:06 |
| gibi | mriedem: we can fix flavor.create by adding the extra_specs in the expected_attr here https://github.com/openstack/nova/blob/master/nova/objects/flavor.py#L494 | 17:06 |
| *** markus_z has quit IRC | 17:06 | |
| mriedem | gibi: wouldn't that already be done because of this? https://github.com/openstack/nova/blob/master/nova/objects/flavor.py#L490 | 17:07 |
| openstackgerrit | avnish proposed openstack/python-novaclient: Removed unnecessary utf-8 encoding https://review.openstack.org/418839 | 17:07 |
| mriedem | extra_specs is in OPTIONAL_FIELDS | 17:07 |
| gibi | mriedem: I think not because extra_specs is not in updates at https://github.com/openstack/nova/blob/master/nova/objects/flavor.py#L491 | 17:07 |
| *** Sukhdev has quit IRC | 17:08 | |
| *** ijw has joined #openstack-nova | 17:08 | |
| mriedem | oh right i missed the in updates check | 17:08 |
| gibi | mriedem: Is it OK for you if I just fix this as part of the bugfix https://review.openstack.org/#/c/415857/ | 17:10 |
| *** briancurtin has quit IRC | 17:11 | |
| mriedem | gibi: that's dealing with aggregates, so i'd think it'd be part of a separate change in that series? i guess it's hard for me to say without knowing in detail what the actual change is going to be - i don't have the context in my head right now | 17:11 |
| mriedem | raj_singh: fyi i updated the devstack series that's testing the service user token stuff | 17:12 |
| *** raunak has joined #openstack-nova | 17:12 | |
| mriedem | i want to see how that runs before i +2 the bottom change in the nova series | 17:12 |
| *** ijw has quit IRC | 17:12 | |
| *** jaypipes has quit IRC | 17:12 | |
| gibi | mriedem: right, that patch getting mixed up with different object fixes due to the same root cause | 17:12 |
| *** nic has joined #openstack-nova | 17:12 | |
| *** pcaruana has quit IRC | 17:12 | |
| gibi | mriedem: I can split it into a series of patches | 17:12 |
| *** Zealoussnow_ has joined #openstack-nova | 17:13 | |
| raj_singh_ | mriedem: I am using https://review.openstack.org/#/c/413232/ to test the series, which is dependent on the devstack patch as well. BTW it works | 17:13 |
| *** LeoZhang has quit IRC | 17:13 | |
| raj_singh_ | mriedem: http://logs.openstack.org/32/413232/4/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/fcaf67a/console.html | 17:13 |
| *** nic has quit IRC | 17:13 | |
| *** rmart04 has quit IRC | 17:13 | |
| *** jose-phillips has joined #openstack-nova | 17:13 | |
| raj_singh_ | tempest.api.compute.admin.test_live_migration.LiveBlockMigrationTestJSON.test_live_block_migration [412.934140s] ... ok | 17:13 |
| raj_singh_ | I put a sleep of 400 and this test took more than 400 with service token enabled. I will upadate the commit msg on DNM patches to avoid confusion | 17:14 |
| *** Swami_ has quit IRC | 17:14 | |
| *** FL1SK has quit IRC | 17:15 | |
| *** mlavalle has joined #openstack-nova | 17:16 | |
| *** FL1SK has joined #openstack-nova | 17:16 | |
| diana_clarke | mriedem: yup, 400 b/c exceptions.ValidationError inherits from exceptions.Invalid which has 400 for code | 17:17 |
| mriedem | diana_clarke: ok | 17:17 |
| mriedem | raj_singh: yup cool | 17:17 |
| dansmith | melwitt: okay, running with the config as the gate does, I now get fewer tests, and many more fails | 17:19 |
| *** tesseract has quit IRC | 17:20 | |
| *** snikitin__ has joined #openstack-nova | 17:20 | |
| mriedem | raj_singh: actually that's kind of odd that it's passing in your test patch, | 17:20 |
| mriedem | raj_singh: because we don't currently run volume-backed live migration tests in the live migration job | 17:20 |
| mriedem | those are skipped for now due to bugs | 17:21 |
| *** moshele has quit IRC | 17:21 | |
| *** snikitin___ has joined #openstack-nova | 17:21 | |
| openstackgerrit | Sujitha proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 17:21 |
| raj_singh_ | :) | 17:21 |
| *** zhangyujun has quit IRC | 17:21 | |
| *** unicell has quit IRC | 17:23 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Refactor InstancePayload creation https://review.openstack.org/418488 | 17:24 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489 | 17:24 |
| *** FL1SK has quit IRC | 17:24 | |
| *** yamahata has quit IRC | 17:25 | |
| openstackgerrit | Sujitha proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 17:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP: Test cellsv1 https://review.openstack.org/418141 | 17:28 |
| *** zhangyujun has joined #openstack-nova | 17:28 | |
| *** FL1SK has joined #openstack-nova | 17:28 | |
| melwitt | dansmith: hm, odd. I guess I didn't make note of how many total tests vs gate when I was running it | 17:31 |
| *** FL1SK has quit IRC | 17:32 | |
| *** FL1SK has joined #openstack-nova | 17:33 | |
| dansmith | melwitt: a lot of my fails are keystone and cinder, so maybe it's not as bad | 17:33 |
| dansmith | but the total is still different | 17:33 |
| openstackgerrit | Sujitha proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 17:35 |
| openstackgerrit | Sujitha proposed openstack/nova: Do not merge: Test newly introduced zero-downtime-upgrade gate job https://review.openstack.org/419066 | 17:35 |
| *** baoli has joined #openstack-nova | 17:36 | |
| *** snikitin_ has quit IRC | 17:36 | |
| *** snikitin___ is now known as snikitin_ | 17:36 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: DNM: Test service token https://review.openstack.org/413232 | 17:36 |
| openstackgerrit | Sujitha proposed openstack/nova: Do not merge: Test newly introduced zero-downtime-upgrade gate job https://review.openstack.org/419066 | 17:37 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: xenapi: support the hotplug of a neutron port https://review.openstack.org/365505 | 17:38 |
| *** lpetrut has quit IRC | 17:38 | |
| *** tjones has joined #openstack-nova | 17:39 | |
| mriedem | GET jaypipes returns a 404 | 17:42 |
| mriedem | :( | 17:42 |
| mriedem | GET/nova-core/jaypipes I guess i should say | 17:43 |
| * mriedem waits for cdent and edleafe to pounce | 17:43 | |
| diana_clarke | I've been working on adding microversion 2.40 support to the python-openstack client, but it currently doesn't do anything special for any of the compute microversions... | 17:43 |
| mriedem | diana_clarke: yeah that's kind of a losing battle right now i think | 17:43 |
| mriedem | diana_clarke: we don't have any microversion support in osc yet | 17:43 |
| cdent | mriedem: ZOMG YOU ARE DOING IT COMPLETELY WRONG, HOW COULD YOU, DIE IN A FIRE | 17:43 |
| edleafe | cdent: +1 | 17:44 |
| diana_clarke | mriedem: okay, so perhaps I'll just abort that mission. Or it could be my new baby ;) | 17:44 |
| mriedem | diana_clarke: raising your human baby might be easier than getting that done :) | 17:44 |
| diana_clarke | LOL | 17:44 |
| *** eharney has joined #openstack-nova | 17:44 | |
| mriedem | far less crying | 17:44 |
| openstackgerrit | Sujitha proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 17:45 |
| mriedem | to be fair at some point we really need someone to actually dig into making microversions work in osc, | 17:46 |
| mriedem | i just haven't given it the attention it needs | 17:46 |
| *** browne has joined #openstack-nova | 17:46 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: DNM: Test service token https://review.openstack.org/413232 | 17:48 |
| *** slaweq has quit IRC | 17:48 | |
| *** slaweq has joined #openstack-nova | 17:49 | |
| *** FL1SK has quit IRC | 17:50 | |
| *** sudipto has quit IRC | 17:50 | |
| *** sudipto_ has quit IRC | 17:50 | |
| *** lucasagomes is now known as lucas-afk | 17:50 | |
| mriedem | diana_clarke: i added something to https://etherpad.openstack.org/p/nova-ptg-pike for compute microversions in osc | 17:50 |
| *** jaypipes has joined #openstack-nova | 17:52 | |
| openstackgerrit | Sujitha proposed openstack/nova: Adding tags field to InstancePayload https://review.openstack.org/407228 | 17:52 |
| *** FL1SK has joined #openstack-nova | 17:53 | |
| *** aarefiev is now known as aarefiev_afk | 17:54 | |
| *** yamahata has joined #openstack-nova | 17:54 | |
| diana_clarke | mriedem: thx! | 17:54 |
| *** Swami has joined #openstack-nova | 17:54 | |
| *** tbachman has quit IRC | 17:55 | |
| *** avolkov has quit IRC | 17:57 | |
| diana_clarke | mriedem: I added a bit of additional info. I won't be traveling to trumpland for the PTG, but I'll follow along via the etherpads. | 17:58 |
| *** vipuls has joined #openstack-nova | 17:58 | |
| *** mvk has joined #openstack-nova | 17:59 | |
| *** ociuhandu has quit IRC | 18:01 | |
| *** ralonsoh has quit IRC | 18:03 | |
| *** dikonoor has quit IRC | 18:03 | |
| *** nic has joined #openstack-nova | 18:03 | |
| openstackgerrit | Jiri Suchomel proposed openstack/nova: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | 18:04 |
| *** liverpooler has quit IRC | 18:05 | |
| *** tbachman has joined #openstack-nova | 18:05 | |
| *** ijw has joined #openstack-nova | 18:08 | |
| *** nic has quit IRC | 18:08 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Add service_token for nova-neutron interaction https://review.openstack.org/410394 | 18:09 |
| *** gszasz has quit IRC | 18:09 | |
| *** derekh has quit IRC | 18:10 | |
| *** liverpooler has joined #openstack-nova | 18:12 | |
| *** ijw has quit IRC | 18:13 | |
| *** claudiub|2 has joined #openstack-nova | 18:15 | |
| *** david-lyle has joined #openstack-nova | 18:15 | |
| *** david-lyle has quit IRC | 18:16 | |
| *** david-lyle has joined #openstack-nova | 18:16 | |
| *** unicell has joined #openstack-nova | 18:17 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization https://review.openstack.org/398509 | 18:18 |
| *** claudiub has quit IRC | 18:19 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: DNM: Test service token https://review.openstack.org/413232 | 18:24 |
| *** kfarr has joined #openstack-nova | 18:25 | |
| *** rama_y has joined #openstack-nova | 18:25 | |
| *** lpetrut has joined #openstack-nova | 18:31 | |
| *** amotoki has quit IRC | 18:32 | |
| *** FL1SK has quit IRC | 18:32 | |
| *** dharinic is now known as dharinic|lunch | 18:36 | |
| *** adisky_ has quit IRC | 18:39 | |
| *** aslezil_ has quit IRC | 18:39 | |
| *** raj_singh_ has quit IRC | 18:41 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract preallocate_for_instance https://review.openstack.org/418876 | 18:45 |
| openstackgerrit | John Garbutt proposed openstack/nova: Pass objects into _create_ports_for_instance https://review.openstack.org/418512 | 18:45 |
| openstackgerrit | John Garbutt proposed openstack/nova: Pass objects into _update_ports_for_instance https://review.openstack.org/418513 | 18:45 |
| *** ociuhandu has joined #openstack-nova | 18:45 | |
| artom | claudiub|2, around? Can you take a look at https://review.openstack.org/#/c/415520/ and https://review.openstack.org/#/c/418100/? It's the virt driver patches for the continuation of device tagging, tagged attachment of nics and volumes | 18:46 |
| *** owalsh is now known as owalsh-afk | 18:46 | |
| artom | I'm not sure I have a specific ask as of yet, as their shape could change in response to reviews, but would something like that be doable for hyper-v? | 18:46 |
| claudiub|2 | looking | 18:50 |
| claudiub|2 | meanwhile, happy new year. :D | 18:50 |
| *** owalsh-afk has quit IRC | 18:50 | |
| *** FL1SK has joined #openstack-nova | 18:51 | |
| artom | claudiub|2, thanks, same to you :) And happy orthodox christmas, if you're into that stuff ^_^ | 18:51 |
| claudiub|2 | hm, i am orthodox, but my christmas was in 25th dec. :D | 18:52 |
| claudiub|2 | artom: question: shouldn't the configdrive be regenerated, to also include the new tag? | 18:52 |
| artom | claudiub|2, yeah, but that's trickier, since we need to wait for the instance to be off | 18:53 |
| artom | We can't just yank it from underneath the guest while it's running | 18:53 |
| artom | It may be mounted, etc | 18:53 |
| *** cdent has quit IRC | 18:53 | |
| *** kfarr has quit IRC | 18:54 | |
| *** dave-mccowan has quit IRC | 18:54 | |
| *** dave-mcc_ has joined #openstack-nova | 18:54 | |
| claudiub|2 | hm, how about regenerating it on interface_attach, but don't attach it yet, and then attach the new one on power_on? | 18:55 |
| *** lpetrut has quit IRC | 18:55 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Improve flavor sample in notification sample tests https://review.openstack.org/419086 | 18:56 |
| claudiub|2 | artom: also, i suppose you will want to merge this before the feature-freeze, right? | 18:56 |
| artom | claudiub|2, in my wildest dreams, yeah :) | 18:56 |
| artom | But I'm aware this is pretty last minute | 18:56 |
| *** abalutoiu has joined #openstack-nova | 18:58 | |
| claudiub|2 | anyways. it is doable for hyper-v | 18:58 |
| gibi | mriedem: I improved the flavor extra_specs part of the instance.<action> notification samples test with the watchdog device as you suggested https://review.openstack.org/#/c/419086 | 18:58 |
| openstackgerrit | Dan Smith proposed openstack/nova: Don't bypass cellsv1 replication if cellsv2 maps are in place https://review.openstack.org/418141 | 18:58 |
| *** aslezil_ has joined #openstack-nova | 18:58 | |
| claudiub|2 | i'll have to send up a patch for it then. :) | 18:58 |
| claudiub|2 | doesn't seem like much to be done, imo. only that configdrive regeneration is a bit trickier | 18:59 |
| claudiub|2 | artom: also, regarding device tagging, are pci devices tagged as well? | 19:00 |
| artom | claudiub|2, don't think so... | 19:00 |
| artom | There's no API for it | 19:00 |
| *** cmartin has joined #openstack-nova | 19:01 | |
| *** satyar has quit IRC | 19:01 | |
| mriedem | gibi: ok | 19:01 |
| mriedem | GET /nova-core/jaypipes -> 200! | 19:02 |
| *** mtanino has joined #openstack-nova | 19:02 | |
| mriedem | jaypipes: when you have 10 free minutes, could you please manhandle this? https://review.openstack.org/#/c/140045/ | 19:02 |
| mriedem | claudiub|2: while you're around there were some comments in https://review.openstack.org/#/c/282407/ - i'm not sure if they are all valid though, so would be cool to respond to those sooner than later | 19:03 |
| * mriedem goes to plow snow | 19:04 | |
| dansmith | mriedem: um, I think you mean personhandle | 19:05 |
| mriedem | dansmith: do women womanhandle? | 19:06 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 19:06 |
| dansmith | mriedem: I feel like this is a trap | 19:06 |
| mriedem | it is, and you haven't yet taken your corporate training course on this yet for 2017 | 19:06 |
| dansmith | true story | 19:06 |
| *** dtp has quit IRC | 19:06 | |
| jaypipes | mriedem: hey yeah, I'm on it. spurious network connectivity and human bandwidth issues all this week, sorry. | 19:08 |
| *** briancurtin has joined #openstack-nova | 19:09 | |
| *** chyka has joined #openstack-nova | 19:11 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.volume_attach notification https://review.openstack.org/401992 | 19:15 |
| edleafe | We need HA Jay | 19:15 |
| *** dave-mcc_ has quit IRC | 19:15 | |
| jaypipes | edleafe: *I* need HA Jay :( | 19:17 |
| *** vladikr has quit IRC | 19:17 | |
| * jroll has a hotspare Jay you can borrow | 19:18 | |
| * jroll hands JayF to jaypipes | 19:18 | |
| jaypipes | :) | 19:18 |
| claudiub|2 | artom: i see. :) | 19:19 |
| JayF | I don't think you could even lift me, much less hand me over to someone ;) | 19:19 |
| claudiub|2 | mriedem: replied. :) | 19:19 |
| claudiub|2 | mriedem: although, not exactly sure how to treat a certain comment: Andrey Volkov was asking if there could be a way to warn operators that if they want to use hyper-v vnuma, they'd have to disable the numa spanning on the host. is there any other better way to do this (except release notes)? | 19:21 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.volume_detach notification https://review.openstack.org/408676 | 19:24 |
| *** satyar has joined #openstack-nova | 19:24 | |
| *** dharinic|lunch is now known as dharinic | 19:26 | |
| *** amotoki has joined #openstack-nova | 19:32 | |
| *** krtaylor has quit IRC | 19:37 | |
| *** vladikr has joined #openstack-nova | 19:38 | |
| EmilienM | I think I found something "weird" in nova-compute & nova-placement | 19:38 |
| EmilienM | if you start n-cpu *before* n-placement, n-compute will never retry to register the placement services | 19:39 |
| diana_clarke | dansmith: ^ was that what you were talking about yesterday? That you were going to change to keep retrying? Ignore me if I've missunderstood. | 19:41 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Ensure to_xml always returns an ASCII string https://review.openstack.org/402382 | 19:41 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: utils.get_disk return the target dev of the root device https://review.openstack.org/402716 | 19:41 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the mirror element to detect when to pivot https://review.openstack.org/399735 | 19:41 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Make guest.BlockDevice only use the target dev of a disk https://review.openstack.org/402163 | 19:41 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Wait for abort_job to complete when not resizing https://review.openstack.org/382449 | 19:41 |
| diana_clarke | dansmith: perhaps this patch? https://review.openstack.org/#/c/418590/ | 19:42 |
| dansmith | EmilienM: diana_clarke yep | 19:43 |
| EmilienM | oh wow | 19:43 |
| EmilienM | but it's not only scheduler | 19:43 |
| EmilienM | it's also compute | 19:43 |
| dansmith | EmilienM: it's only compute, and it was intentional, but a little naive | 19:43 |
| EmilienM | https://review.openstack.org/#/c/418590/4/nova/scheduler/client/report.py is run by nova-compute? | 19:43 |
| dansmith | EmilienM: yes | 19:43 |
| EmilienM | nice | 19:44 |
| EmilienM | thanks! | 19:44 |
| dansmith | EmilienM: it's the scheduler *client* which is the client of the scheduler, which would be compute | 19:44 |
| EmilienM | we need this patch :D | 19:44 |
| dansmith | EmilienM: I wrote it for you dude :) | 19:44 |
| EmilienM | dansmith: yeah? I just found the bug in tripleo :-P | 19:44 |
| EmilienM | we deploy placement but in some random cases, it was not starting. | 19:45 |
| *** salv-orl_ has quit IRC | 19:45 | |
| dansmith | EmilienM: I did, specifically to make the upgrade case a little more graceful | 19:45 |
| dansmith | EmilienM: planning to backport that to newton | 19:45 |
| *** jaosorior has quit IRC | 19:46 | |
| EmilienM | dansmith++ | 19:46 |
| EmilienM | dansmith: do you think https://review.openstack.org/#/c/418590/ can merge today? | 19:47 |
| *** bvanhav__ has joined #openstack-nova | 19:47 | |
| *** bvanhav_ has quit IRC | 19:49 | |
| *** voelzmo has joined #openstack-nova | 19:51 | |
| dansmith | EmilienM: you should hassle someone that can do something about it | 19:52 |
| dansmith | EmilienM: jaypipes once was +2 on that and may grant your wish | 19:52 |
| EmilienM | jaypipes: in case you have some time to review, please: https://review.openstack.org/#/c/418590/ | 19:56 |
| *** jaosorior has joined #openstack-nova | 19:56 | |
| *** morgan_ is now known as morgan | 19:58 | |
| *** nic has joined #openstack-nova | 20:01 | |
| mriedem | claudiub|2: you could emit a warning in the logs... | 20:01 |
| mriedem | claudiub|2: i'd also get it into the numa docs in the admin guide: http://docs.openstack.org/admin-guide/compute-cpu-topologies.html - so tag the patch with DocImpact and leave a comment in the commit message about updating those docs? | 20:02 |
| diana_clarke | mriedem: I wasn't sure what to work on instead, so I am going to keep playing with microversions & osc. We'll see how it goes. https://review.openstack.org/#/c/419108/ | 20:02 |
| mriedem | dansmith: are you going to tweek the docs in a follow up to https://review.openstack.org/#/c/418590/ ? | 20:03 |
| mriedem | i.e. http://git.openstack.org/cgit/openstack/nova/tree/doc/source/placement.rst?id=f5d7ea52c681611c2d85f344da643f9f630d1ef9#n108 | 20:03 |
| dansmith | mriedem: tbh, it's not high on my list considering time available and the rush to unfsck the current stuff | 20:04 |
| dansmith | mriedem: if you think it's a priority for right now then I will, | 20:04 |
| dansmith | but if not, maybe we could find someone interested in doing it? | 20:04 |
| diana_clarke | dansmith, mriedem: I can try and do it. | 20:05 |
| *** tbachman has quit IRC | 20:05 | |
| dansmith | diana_clarke: cool, thanks | 20:05 |
| mriedem | i don't really see how spamming the logs fixes the complaints i was hearing from the red hat people yesterday, | 20:06 |
| mriedem | but if it's something you guys have agreed it makes your lives easier somehow, then ok | 20:06 |
| mriedem | that's all, re: why i'm not rushing it in | 20:06 |
| dansmith | mriedem: spamming the logs doesn't fix anything for us, continuing to try does | 20:07 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make metadata server know about cell mappings https://review.openstack.org/417962 | 20:07 |
| openstackgerrit | Dan Smith proposed openstack/nova: WTFWIP: Multicell support for instance listing https://review.openstack.org/396775 | 20:07 |
| openstackgerrit | Dan Smith proposed openstack/nova: Don't bypass cellsv1 replication if cellsv2 maps are in place https://review.openstack.org/418141 | 20:07 |
| mriedem | ok so in a point release rhops rolls out creds to the newton computes and restarts nova-compute on them, right? which as long as they keep trying to connect to placement, it just happens whenever rhosp rolls out the placement service in an update for newton? i guess i just figured the restart that you need to do for the computes would come after placement is available, at which point we don't need the continual retry | 20:09 |
| *** dtp has joined #openstack-nova | 20:09 | |
| dansmith | mriedem: we can't really enforce that ordering, | 20:09 |
| dansmith | in general, and for a couple different approaches to the upgrade | 20:10 |
| mriedem | ok so you/we might avoid an unnecessary restart of the computes depending on the order of the upgrade | 20:10 |
| mriedem | which is the point | 20:10 |
| *** moshele has joined #openstack-nova | 20:10 | |
| mriedem | w/o the patch, you could have deployed creds updates to the computes, restarted them, then rolled out placement, at which point you have to go back and restart all of the computes again | 20:11 |
| mriedem | and it's that potential 2nd mass restart that is the point of the patch (trying to avoid it) | 20:11 |
| dansmith | at least that yeah | 20:12 |
| mriedem | ok | 20:13 |
| mriedem | i think i can smell what you're cooking now | 20:13 |
| dansmith | so, that and this gets us a little more nag spam in the logs to make sure that people running newton see that this is required before they have kicked off their upgradeand decided to skim the release notes once things are broken | 20:14 |
| dansmith | which goes back to our not so awesome messaging, retroactive "oh and also..." thing in the point release, etc | 20:14 |
| *** dtp has quit IRC | 20:21 | |
| EmilienM | dansmith: look at what we have to do when we missed nova cells v2 in newton https://review.openstack.org/#/c/419121/1/elements/puppet-stack-config/puppet-stack-config.pp | 20:21 |
| jaypipes | mriedem, claudiub|2: hyper-v osvif patch reviewed. | 20:22 |
| *** satyar has quit IRC | 20:22 | |
| dansmith | EmilienM: what about it? | 20:23 |
| edleafe | jaypipes: got a resource provider question if you have a few minutes | 20:23 |
| EmilienM | dansmith: I just wanted to show you the complexity of things about nova cells v2 upgrade from newton to ocata | 20:23 |
| dansmith | EmilienM: to make me feel bad? | 20:23 |
| EmilienM | dansmith: no :-) so you can maybe give feedback | 20:24 |
| EmilienM | and yes, hopefully you feel very sad | 20:24 |
| EmilienM | because we missed it in tripleo/newton, now we have to do that thing | 20:25 |
| edleafe | jaypipes: if I have a reference to a provider, say 'rp', and call rp.set_inventory(inv), that 'rp' reference is now stale, since it now 1 generation behind the DB. | 20:25 |
| openstackgerrit | Diana Clarke proposed openstack/nova: docs - Connect to placement service & retries https://review.openstack.org/419123 | 20:25 |
| EmilienM | jaypipes: thx for the +w :-) | 20:25 |
| edleafe | jaypipes: my question is: should methods that change generation also update the generation field in the current object? | 20:26 |
| claudiub|2 | jaypipes: ty, will upload a new ps shortly. :D | 20:26 |
| diana_clarke | dansmith: It's hard to write about things you don't know much about, but there goes ;) ^^ | 20:26 |
| dansmith | diana_clarke: looks good tome | 20:26 |
| dansmith | also to me | 20:26 |
| dansmith | diana_clarke: mriedem will -1 one time to eff up my stats, but otherwise I think it covers all the things | 20:27 |
| *** browne has quit IRC | 20:28 | |
| mriedem | dansmith: EmilienM: jaypipes: postmortem https://review.openstack.org/#/c/418590/ +1 fwiw with a more detailed recap of my understanding of the issue the patch is trying to avoid/fix | 20:29 |
| mriedem | i just got distracted with PTG talk in another channel | 20:29 |
| dansmith | mriedem: it's not dead yet.. it's like peri-mortem | 20:30 |
| EmilienM | mriedem: ack | 20:30 |
| EmilienM | I like the idea of https://review.openstack.org/#/c/418590/ being backported fwiw | 20:30 |
| EmilienM | I'm pretty sure it will really help deployers | 20:30 |
| jaypipes | edleafe: https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L306 | 20:31 |
| jaypipes | EmilienM: np :) | 20:31 |
| edleafe | jaypipes: that's a static method that only updates the DB | 20:32 |
| mriedem | EmilienM: so in retrospec, i guess we're going to need to figure out a working relationship on how to communicate to the tripleo guys that even though something is optional now, you're going to want to bake it into your tooling, because release notes aren't going to cut it | 20:32 |
| edleafe | jaypipes: the object that called it is now stale | 20:32 |
| mriedem | *retrospect | 20:32 |
| *** voelzmo has quit IRC | 20:32 | |
| *** nic has quit IRC | 20:32 | |
| EmilienM | mriedem: I do agree | 20:32 |
| mriedem | i.e. if nova has to touch grenade for ANY reason, someone from tripleo needs to know | 20:32 |
| *** browne has joined #openstack-nova | 20:33 | |
| mriedem | s/tripleo/any of the 40 openstack deployment projects/ | 20:33 |
| *** voelzmo has joined #openstack-nova | 20:33 | |
| dansmith | mriedem: I was just about to say, it's in the grenade history | 20:33 |
| jaypipes | edleafe: https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L372 | 20:33 |
| jaypipes | edleafe: the object that called it is self. | 20:33 |
| edleafe | jaypipes: IOW, if you make two set_inventory calls with the same object, the second will throw a ConcurrentUpdateDetected | 20:33 |
| dansmith | mriedem: tempted to start up a twitter account and git commit hook :) | 20:33 |
| mriedem | so i'll add something to the nova docs in our review guide - albeit chances are low anyone will read that | 20:33 |
| mriedem | dansmith: you or me? | 20:34 |
| dansmith | mriedem: I'm tempted | 20:34 |
| jaypipes | edleafe: I don't believe that is true. | 20:34 |
| EmilienM | mriedem: we are building a grenade like job in tripleo, where for the first time we'll test in gate upgrades with complex deployments | 20:34 |
| EmilienM | mriedem: it might be useful to have the job in experimental pipeline in nova (one day) | 20:34 |
| edleafe | jaypipes: I'm getting it in some of my tests. | 20:34 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds Hyper-V OVS ViF driver https://review.openstack.org/140045 | 20:35 |
| edleafe | jaypipes: let me see if there is somewhere else it is getting changed | 20:35 |
| mriedem | EmilienM: well the thing is, placement and cells v2 were optional in newton, | 20:36 |
| mriedem | so we had to explicitly enable those in specific jobs in newton to test them out | 20:37 |
| *** bvanhav__ has quit IRC | 20:37 | |
| mriedem | EmilienM: which takes the forethought to know you have to toggle those bits, we're missing that as part of our process right now for non-devstack/grenade things | 20:37 |
| *** voelzmo has quit IRC | 20:37 | |
| *** krtaylor has joined #openstack-nova | 20:38 | |
| *** jaosorior has quit IRC | 20:39 | |
| *** snikitin__ has quit IRC | 20:42 | |
| *** snikitin_ has quit IRC | 20:42 | |
| *** nklenke_ has quit IRC | 20:42 | |
| *** dtp_ has joined #openstack-nova | 20:43 | |
| *** nic has joined #openstack-nova | 20:44 | |
| mriedem | diana_clarke: i'll approve the doc update once https://review.openstack.org/#/c/418590/ is merged | 20:47 |
| diana_clarke | mriedem: thanks, I probably should have has a depends on | 20:48 |
| diana_clarke | had | 20:48 |
| openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | 20:48 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: docs - Connect to placement service & retries https://review.openstack.org/419123 | 20:49 |
| mriedem | diana_clarke: done | 20:49 |
| openstackgerrit | Eric Fried proposed openstack/nova: WIP(2): Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | 20:50 |
| edleafe | jaypipes: found it. _check_capacity_exceeded() creates new RP objects, and these are the ones whose generation gets updated instead of the originals | 20:50 |
| edleafe | jaypipes: https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L1078-L1081 | 20:50 |
| openstackgerrit | Eric Fried proposed openstack/nova: WIP(3): Introduce base PowerVM driver (console) https://review.openstack.org/409402 | 20:51 |
| dansmith | diana_clarke: mriedem: thanks | 20:51 |
| EmilienM | diana_clarke: fyi, depends-on works for inter-projects connection, but for the same project, you better to use git rebase | 20:52 |
| mriedem | dansmith: dash doesn't support multiple changes for the -c option does it...? | 20:52 |
| mriedem | EmilienM: it's fine either way in this case | 20:53 |
| EmilienM | right, just fyi again | 20:53 |
| dansmith | mriedem: dunno, I never use that | 20:54 |
| dansmith | mriedem: patches accepted :) | 20:54 |
| openstackgerrit | Eric Fried proposed openstack/nova: WIP(4): Introduce base PowerVM driver (cfg drv) https://review.openstack.org/409404 | 20:54 |
| diana_clarke | EmilienM: Oh right. I did know that, but I just drank some flu meds so I'm a bit groggy ;) | 20:54 |
| EmilienM | diana_clarke: oops :-) | 20:54 |
| mdrabe | dansmith: Is something enforcing having cells v2 in ocata? | 20:54 |
| dansmith | mdrabe: various bits of cellsv2 have been required each release recently | 20:55 |
| mriedem | mdrabe: yes, a db schema migration | 20:55 |
| dansmith | mdrabe: there is more in ocata sure | 20:55 |
| *** xyang1 has joined #openstack-nova | 20:55 | |
| mriedem | mdrabe: you can't upgrade the api db schema without doing some setup first | 20:55 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: adds REST API for nested providers https://review.openstack.org/384807 | 20:56 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: allow filter providers in tree https://review.openstack.org/377215 | 20:56 |
| openstackgerrit | Jay Pipes proposed openstack/nova: pci: add uuid field to PciDevice object https://review.openstack.org/414768 | 20:56 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | 20:56 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693 | 20:56 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: add nested resource providers https://review.openstack.org/377138 | 20:56 |
| jaypipes | edleafe: le sigh... yeah, line 1169 needs to be changed to: | 20:58 |
| mdrabe | mriedem: What if you're starting fresh on ocata? | 20:58 |
| jaypipes | rp.generation = _increment... | 20:58 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: utils.get_disk return the target dev of the root device https://review.openstack.org/402716 | 20:59 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the mirror element to detect when to pivot https://review.openstack.org/399735 | 20:59 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Make guest.BlockDevice only use the target dev of a disk https://review.openstack.org/402163 | 20:59 |
| mriedem | mdrabe: same deal | 20:59 |
| mriedem | mdrabe: you still have to have a cell0 cell mapping record in the api db | 20:59 |
| mriedem | actually... | 20:59 |
| dansmith | mriedem: the blocker doesn't enforce that it's done first, | 20:59 |
| mriedem | i think the db schema migration ends early | 20:59 |
| dansmith | but you still need that | 20:59 |
| mriedem | if there are no flavors in the api db then it's a noop | 20:59 |
| mriedem | mdrabe: we're also working on this nova-status upgrade check CLI for checking this stuff at any time prior to install/upgrade | 21:00 |
| mriedem | just haven't integrated that yet into the grenade tooling we have | 21:00 |
| *** jamielennox|away is now known as jamielennox | 21:01 | |
| *** lyarwood_ is now known as lyarwood | 21:02 | |
| *** dtp_ is now known as dtp | 21:02 | |
| *** rama_y has quit IRC | 21:05 | |
| *** rama_y_ has joined #openstack-nova | 21:05 | |
| mdrabe | mriedem: So say I didn't have any cells in my deployment down the road, things are certainly not going to work right? The cells requirement isn't just for want? | 21:06 |
| *** tbachman has joined #openstack-nova | 21:06 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: pci: add uuid field to PciDevice object https://review.openstack.org/414768 | 21:07 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | 21:07 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693 | 21:07 |
| dansmith | mdrabe: you can have a cell of one, of course, if that's what you mean | 21:07 |
| dansmith | mdrabe: everyone will be transitioning to a single-cell deployment all the time | 21:07 |
| *** dimtruck is now known as zz_dimtruck | 21:07 | |
| mdrabe | dansmith: But if I had none like we could in newton that's not gonna fly? | 21:08 |
| dansmith | mdrabe: I'm not sure what you mean.. "one" cell is just a single row in the database -- nothing else is different | 21:08 |
| dansmith | mdrabe: meaning, moving from a newton deployment to an ocata one with just one cell is just adding a row to the database, which says your existing stuff is "cell 1" | 21:09 |
| dansmith | mdrabe: no extra services or configuration like normal cells (i.e. cellsv1) requires | 21:10 |
| mdrabe | dansmith: But if I didn't have a single cell could I still deploy? | 21:11 |
| dansmith | mdrabe: no, there's no such thing as "no cells" anymore | 21:11 |
| dansmith | mdrabe: well, you could deploy with no cells, but you could never create any instances.. :P | 21:12 |
| *** smatzek has quit IRC | 21:13 | |
| mdrabe | because the instance has to be built to a specific cell | 21:14 |
| *** jwcroppe has quit IRC | 21:14 | |
| dansmith | yes | 21:14 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Fix missing RP generation update https://review.openstack.org/419137 | 21:16 |
| edleafe | jaypipes: The fix ^^ | 21:16 |
| jaypipes | edleafe: cheers man | 21:17 |
| jaypipes | mriedem: I'm good with https://review.openstack.org/#/c/140045/ now. | 21:17 |
| jaypipes | feel free to wallaby. | 21:17 |
| *** tbachman has quit IRC | 21:18 | |
| jaypipes | edleafe, reviewed. | 21:19 |
| *** moshele has quit IRC | 21:19 | |
| *** cdent has joined #openstack-nova | 21:19 | |
| edleafe | jaypipes: the functional test that was failing was in the later patch that removed Allocation.create() | 21:20 |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | 21:20 |
| edleafe | jaypipes: I'll see what I can add that would be close | 21:21 |
| jaypipes | edleafe: cool, thanks Ed. | 21:21 |
| cdent | more stuff borked? | 21:21 |
| edleafe | cdent: one effect of removing Allocation.create() is that AllocationList.create_all() is much more stringent in its checking | 21:22 |
| edleafe | cdent: and that uncovered a slew of ResourceProvider references that were stale as far as their generation was concerned | 21:23 |
| cdent | in? | 21:23 |
| *** tbachman has joined #openstack-nova | 21:24 | |
| edleafe | cdent: see https://review.openstack.org/419137 for what I fixed | 21:24 |
| cdent | cool thanks | 21:24 |
| *** salv-orlando has joined #openstack-nova | 21:25 | |
| edleafe | cdent: the tests in nova/tests/functional/db/test_resource_provider.py were creating allocations before there was any inventory | 21:25 |
| edleafe | cdent: Allocation.create() allowed that. AllocationList.create_all() doesn't. | 21:26 |
| *** zz_dimtruck is now known as dimtruck | 21:27 | |
| cdent | gyres with gyres or something | 21:29 |
| *** lpetrut has joined #openstack-nova | 21:33 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: pci: add uuid field to PciDevice object https://review.openstack.org/414768 | 21:34 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | 21:34 |
| *** Jeffrey4l_ has joined #openstack-nova | 21:34 | |
| *** Jeffrey4l has quit IRC | 21:35 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds vNUMA implementation https://review.openstack.org/282407 | 21:36 |
| *** rtheis has quit IRC | 21:38 | |
| *** lpetrut has quit IRC | 21:38 | |
| *** abalutoiu has quit IRC | 21:41 | |
| mriedem | claudiub|2: i was just about done commenting on that one :) | 21:42 |
| dansmith | mriedem: melwitt: don't mind me, just bragging: https://review.openstack.org/#/c/418141/ | 21:43 |
| claudiub|2 | ha, oups. :D | 21:43 |
| *** nic has quit IRC | 21:44 | |
| *** jaosorior has joined #openstack-nova | 21:44 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Fix missing RP generation update https://review.openstack.org/419137 | 21:44 |
| mriedem | claudiub|2: comments posted | 21:44 |
| mriedem | dansmith: nice | 21:44 |
| mriedem | dansmith: i can see your nippies from here | 21:45 |
| melwitt | hah, yay | 21:45 |
| mriedem | i guess that should have been in a PM | 21:45 |
| dansmith | haha | 21:45 |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | 21:46 |
| *** cleong has quit IRC | 21:49 | |
| bauzas | mriedem: jaypipes: could you maybe look at https://review.openstack.org/#/c/392569/ soon ? | 21:51 |
| bauzas | given it's adding a microversion, I'd make sure that this change could be quickly merged | 21:52 |
| bauzas | edleafe had some comments, but I could follow-up the change | 21:52 |
| *** krtaylor has quit IRC | 21:54 | |
| *** dave-mccowan has joined #openstack-nova | 21:54 | |
| bauzas | mriedem: do you know if we have a problem with the Xen 3rd party CI? | 21:54 |
| bauzas | wrt https://review.openstack.org/#/c/365505/ | 21:54 |
| bauzas | I'm waiting to see it there | 21:54 |
| *** thorst has quit IRC | 21:54 | |
| mriedem | bauzas: it was good before | 21:54 |
| mriedem | bauzas: i've rechecked https://review.openstack.org/#/c/416797/ which runs the attach/detach interfaces tests on that patch | 21:55 |
| mriedem | bauzas: and yeah on the placement API change, i'm trying to wrap up some of the hyperv patches and then can move on | 21:55 |
| mriedem | bauzas: we should also get alex_xu and sdague to help review https://review.openstack.org/#/c/392569/ | 21:56 |
| bauzas | mriedem: cool by me | 21:56 |
| *** nic has joined #openstack-nova | 21:56 | |
| bauzas | mriedem: last run of XenServer for the above change was -1 with http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/05/365505/22/check/dsvm-tempest-neutron-network/e2bfe3d/ | 21:56 |
| bauzas | for PS22 | 21:57 |
| mriedem | bauzas: that was some devstack setup issue http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/05/365505/22/check/dsvm-tempest-neutron-network/e2bfe3d/logs/devstacklog.txt.gz | 21:57 |
| *** thorst has joined #openstack-nova | 21:57 | |
| *** ijw has joined #openstack-nova | 21:57 | |
| bauzas | oh right | 21:57 |
| bauzas | meh | 21:58 |
| mriedem | http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/97/416797/3/check/dsvm-tempest-neutron-network/d75b62d/run_tests.log was happy on 1/9 | 21:58 |
| *** dtp has quit IRC | 21:58 | |
| mriedem | bauzas: if you want to +2 and leave the +W to me that's fine too | 21:58 |
| mriedem | given time of day i mean | 21:59 |
| bauzas | mriedem: just looking at the XenServer change | 21:59 |
| bauzas | I mean the tempest test removal | 21:59 |
| bauzas | well, the exception removal rather | 21:59 |
| bauzas | mriedem: how can we be sure that https://review.openstack.org/#/c/416797/ is run by XenServer CI ? | 22:00 |
| bauzas | I'm verifying that | 22:00 |
| *** tblakes has quit IRC | 22:01 | |
| bauzas | actually looking at the regex here http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/97/416797/3/check/dsvm-tempest-neutron-network/d75b62d/run_tests.log | 22:01 |
| *** thorst has quit IRC | 22:02 | |
| bauzas | awesome {2} tempest.api.compute.servers.test_attach_interfaces.AttachInterfacesTestJSON.test_create_list_show_delete_interfaces [349.753425s] ... ok | 22:02 |
| bauzas | mriedem: okay, +Wing it | 22:02 |
| bauzas | because of ^ | 22:02 |
| mriedem | http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/97/416797/3/check/dsvm-tempest-neutron-network/d75b62d/logs/devstacklog.txt.gz | 22:02 |
| mriedem | 2017-01-10 03:15:31.180 | + functions-common:git_clone:566 : git show --oneline 2017-01-10 03:15:31.194 | 823b053 xenapi: support the hotplug of a neutron port | 22:03 |
| mriedem | tells you which nova change it's testing | 22:03 |
| mriedem | which is our happy friend the vif hotplug patch | 22:03 |
| mriedem | btw, easy tangential patch to the flavor notification series https://review.openstack.org/#/c/419086/ | 22:04 |
| mriedem | but i'm going to mark that bp complete since the update patch is merged | 22:04 |
| *** tblakes has joined #openstack-nova | 22:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 22:11 |
| mriedem | claudiub|2: tweak needed here :) https://review.openstack.org/#/c/140045/ | 22:13 |
| mriedem | or i'll just do it if you're trying to end your day | 22:13 |
| dansmith | "end your day" .. sounds ominous | 22:14 |
| * cdent ends his day | 22:15 | |
| *** cdent has quit IRC | 22:15 | |
| claudiub|2 | mriedem: i'm not trying to end my days. :D | 22:16 |
| mriedem | if you're trying to rest your weary head | 22:16 |
| *** jaypipes has quit IRC | 22:16 | |
| mriedem | the plurality is significant | 22:17 |
| claudiub|2 | still, VirtualInterfaceUnplugException does have a reason formatting argument, while the VirtualInterfacePlugException doesn't. | 22:17 |
| claudiub|2 | i was expecting both of them to have a reason argument. | 22:18 |
| claudiub|2 | https://github.com/openstack/nova/blob/f55815b2f90b934801a88e10bf750876b0587328/nova/exception.py#L132 | 22:18 |
| zzzeek | mriedem: dumb Q as I am clueless. does the nova.task_log table grow unbounded and if so is that part of truncate/cleanup | 22:20 |
| mriedem | claudiub|2: yeah i just noticed the same and left a comment to correct that unplug type | 22:20 |
| mriedem | zzzeek: let me look | 22:20 |
| *** salv-orl_ has joined #openstack-nova | 22:22 | |
| mriedem | zzzeek: so it's going to be unbounded, | 22:23 |
| mriedem | runs in a periodic task as long as CONF.instance_usage_audit is True | 22:23 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds Hyper-V OVS ViF driver https://review.openstack.org/140045 | 22:24 |
| mriedem | zzzeek: the archive command should move it from the nova.task_log table to the nova.shadow_task_log table | 22:24 |
| mriedem | zzzeek: but nothing deletes those records as far as i know | 22:24 |
| zzzeek | mriedem: OK so on a highly used nova DB i might have 30-40M rows in this thing, at least in the shadow table | 22:24 |
| mriedem | sure | 22:25 |
| mriedem | my understanding of this thing is it's for the REST API to pull these audit logs which ceilometer consumes | 22:25 |
| zzzeek | mriedem: right, but the DB size is never mitigated | 22:25 |
| mriedem | http://developer.openstack.org/api-ref/compute/#server-usage-audit-log-os-instance-usage-audit-log | 22:25 |
| mriedem | zzzeek: nope | 22:26 |
| *** salv-orlando has quit IRC | 22:26 | |
| mriedem | neither is...anything except for nova.tags and anything in the nova_api db | 22:26 |
| zzzeek | mriedem: and...not anytime soon, but people might someday like that DB tables like these don't grow to be billions of rows? or that's how they want it | 22:26 |
| mriedem | all of the old tables in the nova db use the soft delete mixin | 22:26 |
| zzzeek | mriedem: right but task_log seems to be, accumulate rows just for doing things, so it's a fast grower | 22:27 |
| mriedem | zzzeek: i think most operators have their own purge scripts, i know there are some in the osops tooling repos | 22:27 |
| zzzeek | mriedem: OK | 22:27 |
| mriedem | there was a bp at one point to add a purge cli to nova | 22:27 |
| mriedem | for not archiving, no shadow tables, but delete things over a certain age | 22:27 |
| mriedem | last i remember someone from red hat was looking into implementing that | 22:28 |
| mriedem | i'll try to find it | 22:28 |
| mriedem | zzzeek: v | 22:28 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/purge-deleted-instances-cmd.html | 22:28 |
| *** markvoelker has quit IRC | 22:29 | |
| mriedem | ^ only addresses instances though, so that'd have to be tweaked to handle task_log | 22:29 |
| mriedem | or to add something to nova to self-clean that table | 22:29 |
| mriedem | like only keep audit logs under a certain age | 22:29 |
| *** baoli has quit IRC | 22:30 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: create aggregate map in report client https://review.openstack.org/407309 | 22:30 |
| *** markvoelker has joined #openstack-nova | 22:32 | |
| mriedem | zzzeek: what's even funnier/sadder about that thing, | 22:32 |
| mriedem | zzzeek: is the periodic task emits a notification, which someone (ceilometer) is consuming and probably throwing that data into a time series db | 22:33 |
| *** annegentle has quit IRC | 22:33 | |
| mriedem | so nova storing it is probably much less relevant, except for nova's own REST API | 22:33 |
| *** markvoelker has quit IRC | 22:34 | |
| zzzeek | mriedem: im on a search to figure out how big people's databases might be reasonably getting | 22:34 |
| *** markvoelker has joined #openstack-nova | 22:34 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Make metadata server know about cell mappings https://review.openstack.org/417962 | 22:35 |
| openstackgerrit | Dan Smith proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775 | 22:35 |
| mriedem | zzzeek: at one point not too long ago dansmith had a fairly large db for testing the archive command | 22:36 |
| * dansmith crosses all crossable appendages | 22:36 | |
| mriedem | dansmith: did that have task_log stuff in it? | 22:36 |
| dansmith | mriedem: probably | 22:36 |
| dansmith | it's in a bug somewhere | 22:36 |
| mriedem | mikal might be able to get his hands on some large db with task_log in it since that feature was all added by rax people | 22:37 |
| dansmith | the dump I mean | 22:37 |
| mriedem | zzzeek: or jhesketh | 22:37 |
| mriedem | if i had 1% more energy right now i might email the operators list asking if anyone uses the nova REST API for this feature | 22:38 |
| *** jheroux has quit IRC | 22:38 | |
| *** markvoelker_ has joined #openstack-nova | 22:39 | |
| *** markvoelker has quit IRC | 22:39 | |
| zzzeek | mriedem: yeah i can manufacture fake large DBs with a script here am more wondering, if i went into the nova DB of some huge customer, how big would these tables be. because migrating from mariadb5.5 -> 10.1 requires a dump+ restore | 22:39 |
| zzzeek | and for tens of millions of rows it's a hit | 22:39 |
| mriedem | zzzeek: so i'd ask jhesketh b/c he has some large dbs for just that kind of question | 22:40 |
| mriedem | zzzeek: otherwise we'd have to ask some godaddy or cern people probably | 22:40 |
| zzzeek | mriedem: thanks | 22:40 |
| *** kberger has joined #openstack-nova | 22:41 | |
| *** marst has quit IRC | 22:44 | |
| mriedem | dansmith: so i guess on https://review.openstack.org/#/c/416585/ we have some kind of weird catch-22 because the cellsv1 job fails on that patch, but passes on your nova one that uses that | 22:46 |
| mriedem | http://logs.openstack.org/85/416585/8/experimental/gate-tempest-dsvm-cells-ubuntu-xenial/e9f1fe9/console.html#_2017-01-11_19_25_38_191005 | 22:47 |
| dansmith | mriedem: we might be able to merge the nova patch before the devstack one, if you want to try | 22:47 |
| dansmith | or we just queue the nova one up and let the devstack one gate it, which I think is probably fine given the stability of things right now | 22:47 |
| dansmith | your call | 22:47 |
| dansmith | I needed the depends-on to test them together | 22:48 |
| mriedem | so we don't need the depends-on to make the nova one pass? | 22:48 |
| dansmith | I don't know | 22:48 |
| dansmith | we might not | 22:48 |
| dansmith | probably don't | 22:48 |
| mriedem | ok, well the cells v1 job is gating of nova and other repos (not devstack though sadly) so if we land the devstack change first, it's going to break things until the nova one lands | 22:49 |
| mriedem | s/of/on/ | 22:49 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Add service_token for nova-glance interaction https://review.openstack.org/412634 | 22:49 |
| dansmith | yeah, but we can get them both in the gate together | 22:49 |
| dansmith | but I'll remove it and let is run to see | 22:49 |
| *** catintheroof has quit IRC | 22:50 | |
| mriedem | ok | 22:50 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make metadata server know about cell mappings https://review.openstack.org/417962 | 22:51 |
| openstackgerrit | Dan Smith proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775 | 22:51 |
| openstackgerrit | Dan Smith proposed openstack/nova: Don't bypass cellsv1 replication if cellsv2 maps are in place https://review.openstack.org/418141 | 22:51 |
| *** catintheroof has joined #openstack-nova | 22:51 | |
| *** cmartin has quit IRC | 22:51 | |
| *** catintheroof has quit IRC | 22:55 | |
| openstackgerrit | Sujitha proposed openstack/nova: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | 22:56 |
| *** Swami has quit IRC | 22:56 | |
| openstackgerrit | Sujitha proposed openstack/nova: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | 22:58 |
| *** dtp has joined #openstack-nova | 23:00 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds vNUMA implementation https://review.openstack.org/282407 | 23:00 |
| *** krtaylor has joined #openstack-nova | 23:02 | |
| *** tblakes has quit IRC | 23:04 | |
| *** raunak has quit IRC | 23:05 | |
| *** mdrabe has quit IRC | 23:06 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds vNUMA implementation https://review.openstack.org/282407 | 23:07 |
| *** nic has quit IRC | 23:07 | |
| claudiub|2 | well, i'm out. thanks for the reviews! | 23:07 |
| *** efried has quit IRC | 23:09 | |
| mriedem | zzzeek: http://lists.openstack.org/pipermail/openstack-operators/2017-January/012425.html | 23:09 |
| *** burt has quit IRC | 23:09 | |
| zzzeek | mriedem: wow thanks! | 23:10 |
| *** dave-mccowan has quit IRC | 23:11 | |
| *** slaweq has quit IRC | 23:11 | |
| *** lamt has quit IRC | 23:11 | |
| *** mlavalle has quit IRC | 23:14 | |
| *** claudiub|2 has quit IRC | 23:16 | |
| *** raunak has joined #openstack-nova | 23:18 | |
| openstackgerrit | Sujitha proposed openstack/nova: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | 23:20 |
| openstackgerrit | Jay Pipes proposed openstack/nova: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 23:29 |
| *** jaypipes has joined #openstack-nova | 23:30 | |
| jaypipes | edleafe: good pickup on the location of the except block in https://review.openstack.org/#/c/392569/. Pushed a fix for that. Please re-review at your earliest. | 23:30 |
| jaypipes | mriedem: ^ is mostly written by me so I don't want to +2 it. | 23:31 |
| jaypipes | if you and dansmith could tackle, that would be great. | 23:31 |
| *** markvoelker_ has quit IRC | 23:32 | |
| mriedem | jaypipes: yeah it's on my list, would also like alex_xu and sdague to look | 23:32 |
| jaypipes | mriedem: thx | 23:33 |
| jaypipes | mriedem: going through the feature freeze review list now, trying to get reviews done. | 23:33 |
| *** JoseMello has quit IRC | 23:35 | |
| *** edmondsw has quit IRC | 23:36 | |
| mriedem | cool we've made some solid progress today, 1 merged and have 3 in the queue | 23:36 |
| *** pradk has quit IRC | 23:38 | |
| *** markvoelker has joined #openstack-nova | 23:40 | |
| *** FL1SK has quit IRC | 23:41 | |
| *** dimtruck is now known as zz_dimtruck | 23:47 | |
| *** thorst_ has joined #openstack-nova | 23:48 | |
| *** salv-orl_ has quit IRC | 23:52 | |
| *** thorst_ has quit IRC | 23:53 | |
| *** jaosorior has quit IRC | 23:54 | |
| *** xyang1 has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!