| *** amotoki has quit IRC | 00:07 | |
| *** crushil has quit IRC | 00:09 | |
| *** hongbin has quit IRC | 00:10 | |
| *** thorst has joined #openstack-nova | 00:12 | |
| *** Jack_Iv has joined #openstack-nova | 00:17 | |
| *** tbachman has quit IRC | 00:19 | |
| *** thorst has quit IRC | 00:19 | |
| *** amotoki has joined #openstack-nova | 00:20 | |
| *** Jack_Iv has quit IRC | 00:21 | |
| *** esberglu has joined #openstack-nova | 00:21 | |
| *** esberglu has quit IRC | 00:21 | |
| *** esberglu has joined #openstack-nova | 00:22 | |
| *** esberglu has quit IRC | 00:22 | |
| *** Sukhdev has joined #openstack-nova | 00:22 | |
| *** esberglu has joined #openstack-nova | 00:22 | |
| Sukhdev | TheJulia: here is the bug I filed for incorrect dhcp options - https://bugs.launchpad.net/ironic/+bug/1682309 | 00:22 |
|---|---|---|
| openstack | Launchpad bug 1682309 in Ironic "Couple of DHCP Options seem to be incorrect" [Undecided,New] | 00:22 |
| TheJulia | Sukhdev: Wrong channel, but thank you :) | 00:23 |
| Sukhdev | TheJulia : opps - sorry | 00:23 |
| *** armax has quit IRC | 00:26 | |
| *** armax has joined #openstack-nova | 00:26 | |
| *** armax has quit IRC | 00:26 | |
| *** esberglu has quit IRC | 00:26 | |
| *** amotoki has quit IRC | 00:27 | |
| *** dtp has quit IRC | 00:29 | |
| *** armax has joined #openstack-nova | 00:31 | |
| *** armax has quit IRC | 00:32 | |
| *** gyee has quit IRC | 00:32 | |
| *** amotoki has joined #openstack-nova | 00:33 | |
| *** winston-d_ has joined #openstack-nova | 00:35 | |
| *** armax has joined #openstack-nova | 00:36 | |
| *** ediardo has quit IRC | 00:37 | |
| *** marst has joined #openstack-nova | 00:38 | |
| *** zhurong has joined #openstack-nova | 00:43 | |
| *** armax has quit IRC | 00:43 | |
| *** armax has joined #openstack-nova | 00:44 | |
| *** armax has quit IRC | 00:48 | |
| *** MasterOfBugs has quit IRC | 00:49 | |
| *** pramodrj07 has quit IRC | 00:49 | |
| *** thorst has joined #openstack-nova | 00:50 | |
| *** Sukhdev has quit IRC | 00:50 | |
| *** mriedem has quit IRC | 00:51 | |
| *** liusheng has joined #openstack-nova | 00:53 | |
| *** amotoki has quit IRC | 00:54 | |
| *** masber has quit IRC | 00:54 | |
| *** masber has joined #openstack-nova | 00:55 | |
| *** suresh has quit IRC | 00:56 | |
| openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811 | 00:57 |
| *** phuongnh has joined #openstack-nova | 00:58 | |
| *** thorst has quit IRC | 01:01 | |
| *** amotoki has joined #openstack-nova | 01:02 | |
| *** takashin has quit IRC | 01:04 | |
| *** takashin has joined #openstack-nova | 01:05 | |
| *** lyan has quit IRC | 01:06 | |
| *** al498u_ has left #openstack-nova | 01:06 | |
| *** takashin has quit IRC | 01:09 | |
| *** NikhilS has joined #openstack-nova | 01:10 | |
| *** takashin has joined #openstack-nova | 01:11 | |
| *** zz_dimtruck is now known as dimtruck | 01:12 | |
| *** crushil has joined #openstack-nova | 01:12 | |
| *** rfolco has quit IRC | 01:13 | |
| *** takashin_ has joined #openstack-nova | 01:15 | |
| *** takashin has quit IRC | 01:16 | |
| *** mriedem has joined #openstack-nova | 01:18 | |
| *** takashin_ has quit IRC | 01:20 | |
| *** mtanino has joined #openstack-nova | 01:21 | |
| *** takashin has joined #openstack-nova | 01:22 | |
| *** zeroDivisible has quit IRC | 01:24 | |
| *** zeroDivisible has joined #openstack-nova | 01:29 | |
| *** takashin has quit IRC | 01:30 | |
| *** gongysh has quit IRC | 01:31 | |
| *** Apoorva has quit IRC | 01:31 | |
| *** takashin has joined #openstack-nova | 01:34 | |
| *** tovin07_ has joined #openstack-nova | 01:35 | |
| *** takashin has quit IRC | 01:38 | |
| *** gongysh has joined #openstack-nova | 01:40 | |
| *** priteau has joined #openstack-nova | 01:44 | |
| *** mnestratov has quit IRC | 01:44 | |
| *** takashin has joined #openstack-nova | 01:44 | |
| *** priteau has quit IRC | 01:48 | |
| *** thorst has joined #openstack-nova | 01:49 | |
| *** thorst has quit IRC | 01:49 | |
| *** yonglihe has joined #openstack-nova | 01:52 | |
| *** litao has joined #openstack-nova | 01:55 | |
| openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811 | 02:05 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Explicitly define reboot 'type' as string in schema https://review.openstack.org/456430 | 02:08 |
| *** gcb has quit IRC | 02:13 | |
| *** rcernin has joined #openstack-nova | 02:20 | |
| *** thorst has joined #openstack-nova | 02:21 | |
| openstackgerrit | 赵明俊 proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram https://review.openstack.org/456050 | 02:23 |
| openstackgerrit | Jon Bernard proposed openstack/nova master: Add tempest-dsvm-ceph-rc https://review.openstack.org/456292 | 02:25 |
| *** gcb has joined #openstack-nova | 02:25 | |
| *** mtanino has quit IRC | 02:25 | |
| *** rcernin has quit IRC | 02:26 | |
| *** rcernin has joined #openstack-nova | 02:26 | |
| *** mkoderer has quit IRC | 02:27 | |
| *** gongysh has quit IRC | 02:28 | |
| *** sofie_ has joined #openstack-nova | 02:30 | |
| *** Sukhdev has joined #openstack-nova | 02:39 | |
| *** thorst has quit IRC | 02:39 | |
| openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.rescue notifications https://review.openstack.org/385644 | 02:46 |
| *** gongysh has joined #openstack-nova | 02:54 | |
| *** mriedem has quit IRC | 02:54 | |
| *** rfolco has joined #openstack-nova | 02:56 | |
| *** rfolco has quit IRC | 02:56 | |
| *** vladikr has quit IRC | 03:01 | |
| *** Sukhdev has quit IRC | 03:01 | |
| *** lucasxu has joined #openstack-nova | 03:01 | |
| *** gjayavelu has quit IRC | 03:02 | |
| *** coreywright has quit IRC | 03:06 | |
| *** cNilesh has joined #openstack-nova | 03:06 | |
| *** lucasxu has quit IRC | 03:09 | |
| *** nicolasbock has quit IRC | 03:12 | |
| *** nic has joined #openstack-nova | 03:12 | |
| *** liujiong has joined #openstack-nova | 03:19 | |
| *** coreywright has joined #openstack-nova | 03:20 | |
| *** ssurana has quit IRC | 03:21 | |
| *** ssurana has joined #openstack-nova | 03:21 | |
| *** ssurana has quit IRC | 03:21 | |
| *** fragatin_ has joined #openstack-nova | 03:26 | |
| *** fragatina has quit IRC | 03:29 | |
| *** links has joined #openstack-nova | 03:36 | |
| *** nic has quit IRC | 03:40 | |
| *** priteau has joined #openstack-nova | 03:44 | |
| *** priteau has quit IRC | 03:49 | |
| *** trinaths has joined #openstack-nova | 03:51 | |
| *** dimtruck is now known as zz_dimtruck | 03:51 | |
| *** udesale has joined #openstack-nova | 03:54 | |
| *** ssurana has joined #openstack-nova | 03:56 | |
| openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram https://review.openstack.org/456050 | 03:56 |
| *** sridharg has joined #openstack-nova | 03:57 | |
| *** zhurong has quit IRC | 03:57 | |
| *** jamielennox is now known as jamielennox|away | 04:01 | |
| *** amotoki_ has joined #openstack-nova | 04:08 | |
| *** amotoki has quit IRC | 04:10 | |
| *** abhishek has joined #openstack-nova | 04:12 | |
| *** zhurong has joined #openstack-nova | 04:17 | |
| *** Jack_Iv has joined #openstack-nova | 04:17 | |
| *** Jack_Iv has quit IRC | 04:22 | |
| *** rcernin has quit IRC | 04:29 | |
| *** jamielennox|away is now known as jamielennox | 04:29 | |
| *** psachin has joined #openstack-nova | 04:29 | |
| *** thorst has joined #openstack-nova | 04:36 | |
| *** ayogi has joined #openstack-nova | 04:41 | |
| *** thorst has quit IRC | 04:41 | |
| *** fragatin_ has quit IRC | 04:41 | |
| *** fragatina has joined #openstack-nova | 04:42 | |
| *** fandi has joined #openstack-nova | 04:54 | |
| *** prateek has joined #openstack-nova | 04:54 | |
| *** pramodrj07 has joined #openstack-nova | 05:05 | |
| *** MasterOfBugs has joined #openstack-nova | 05:05 | |
| *** fandi has quit IRC | 05:10 | |
| *** nkorabli has joined #openstack-nova | 05:10 | |
| *** bmace has quit IRC | 05:11 | |
| *** bmace has joined #openstack-nova | 05:11 | |
| *** nkorabli_ has quit IRC | 05:13 | |
| *** claudiub has joined #openstack-nova | 05:17 | |
| *** lucasxu has joined #openstack-nova | 05:19 | |
| *** lucasxu has quit IRC | 05:20 | |
| *** baoli has quit IRC | 05:20 | |
| *** baoli has joined #openstack-nova | 05:21 | |
| *** ratailor has joined #openstack-nova | 05:21 | |
| *** rcernin has joined #openstack-nova | 05:37 | |
| *** garyk has joined #openstack-nova | 05:38 | |
| *** mdnadeem has joined #openstack-nova | 05:39 | |
| *** Jack_Iv has joined #openstack-nova | 05:39 | |
| *** grumpycatt has quit IRC | 05:41 | |
| *** priteau has joined #openstack-nova | 05:45 | |
| *** amotoki has joined #openstack-nova | 05:46 | |
| *** amotoki_ has quit IRC | 05:50 | |
| *** priteau has quit IRC | 05:50 | |
| *** Jack_Iv has quit IRC | 05:50 | |
| *** Jack_Iv has joined #openstack-nova | 05:51 | |
| *** baoli has quit IRC | 05:53 | |
| *** ssurana has quit IRC | 05:54 | |
| *** tuanluong has joined #openstack-nova | 05:58 | |
| *** fragatina has quit IRC | 06:00 | |
| *** ecelik has joined #openstack-nova | 06:00 | |
| *** crushil has quit IRC | 06:01 | |
| *** gjayavelu has joined #openstack-nova | 06:06 | |
| *** dixiaoli has joined #openstack-nova | 06:08 | |
| *** jaosorior_away is now known as jaosorior | 06:18 | |
| *** sridharg has quit IRC | 06:18 | |
| *** ecelik has left #openstack-nova | 06:18 | |
| *** satyar has joined #openstack-nova | 06:24 | |
| *** yasemin has joined #openstack-nova | 06:31 | |
| yasemin | hi, i installed multinode ocata devstack, when i added compute nodes, my instances didint connect to internet ? any idea ? | 06:33 |
| *** markus_z has joined #openstack-nova | 06:35 | |
| *** fragatina has joined #openstack-nova | 06:41 | |
| *** zzzeek has quit IRC | 06:41 | |
| *** fragatina has quit IRC | 06:41 | |
| *** avolkov has joined #openstack-nova | 06:41 | |
| *** fragatina has joined #openstack-nova | 06:42 | |
| *** fragatina has quit IRC | 06:42 | |
| *** fragatina has joined #openstack-nova | 06:42 | |
| *** fandi has joined #openstack-nova | 06:43 | |
| *** fragatina has quit IRC | 06:45 | |
| *** fragatina has joined #openstack-nova | 06:46 | |
| *** ekuris has quit IRC | 06:46 | |
| *** udesale__ has joined #openstack-nova | 06:47 | |
| *** udesale has quit IRC | 06:49 | |
| *** voelzmo has joined #openstack-nova | 06:50 | |
| *** jamielennox is now known as jamielennox|away | 06:52 | |
| *** faizy has joined #openstack-nova | 06:54 | |
| *** belmoreira has joined #openstack-nova | 06:56 | |
| *** zzzeek has joined #openstack-nova | 06:59 | |
| *** pcaruana has joined #openstack-nova | 07:01 | |
| *** rcernin has quit IRC | 07:01 | |
| *** ekuris has joined #openstack-nova | 07:01 | |
| *** rcernin has joined #openstack-nova | 07:01 | |
| *** pramodrj07 has quit IRC | 07:02 | |
| *** MasterOfBugs has quit IRC | 07:02 | |
| *** pramodrj07 has joined #openstack-nova | 07:03 | |
| *** MasterOfBugs has joined #openstack-nova | 07:03 | |
| *** PramodJ has joined #openstack-nova | 07:04 | |
| *** MasterOfBugs has quit IRC | 07:05 | |
| *** pramodrj07 has quit IRC | 07:05 | |
| *** MasterOfBugs has joined #openstack-nova | 07:05 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [WIP] [placement] Add api-ref resource provider https://review.openstack.org/456176 | 07:06 |
| *** tesseract has joined #openstack-nova | 07:12 | |
| *** adisky_ has joined #openstack-nova | 07:19 | |
| *** Jack_Iv has quit IRC | 07:26 | |
| *** Jack_Iv has joined #openstack-nova | 07:27 | |
| *** Jack_Iv has quit IRC | 07:27 | |
| *** sridharg has joined #openstack-nova | 07:31 | |
| *** fragatina has quit IRC | 07:33 | |
| *** rmart04 has joined #openstack-nova | 07:34 | |
| *** abhishek has quit IRC | 07:36 | |
| *** Jack_Iv has joined #openstack-nova | 07:41 | |
| *** peter-hamilton has quit IRC | 07:46 | |
| *** udesale has joined #openstack-nova | 07:47 | |
| *** priteau has joined #openstack-nova | 07:47 | |
| *** ekuris has quit IRC | 07:47 | |
| *** udesale__ has quit IRC | 07:49 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for '/servers' endpoint instead of stevedore https://review.openstack.org/445864 | 07:49 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for '/os-keypairs' endpoint instead of stevedore https://review.openstack.org/456478 | 07:49 |
| openstackgerrit | Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/456481 | 07:51 |
| *** priteau has quit IRC | 07:52 | |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Netronome SmartNIC Enablement https://review.openstack.org/456483 | 07:55 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add a functional test for 'removeFloatingIp' action https://review.openstack.org/453026 | 07:57 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 07:57 |
| *** MasterOfBugs has quit IRC | 07:59 | |
| *** PramodJ has quit IRC | 07:59 | |
| *** gjayavelu has quit IRC | 07:59 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 07:59 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 08:00 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 08:01 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.42 - Fix tag attribute disappearing https://review.openstack.org/429512 | 08:01 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43 - Enable cold migration with target host https://review.openstack.org/406707 | 08:01 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43: List/Show all server migration types https://review.openstack.org/430839 | 08:02 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876 | 08:02 |
| *** ekuris has joined #openstack-nova | 08:03 | |
| *** efoley has joined #openstack-nova | 08:05 | |
| *** efoley has quit IRC | 08:05 | |
| *** efoley_ has joined #openstack-nova | 08:05 | |
| *** ralonsoh has joined #openstack-nova | 08:06 | |
| *** efoley__ has joined #openstack-nova | 08:07 | |
| *** efoley_ has quit IRC | 08:11 | |
| *** lucas-afk is now known as lucasagomes | 08:13 | |
| *** huanxie has joined #openstack-nova | 08:14 | |
| *** ratailor is now known as ratailor|Lunch | 08:14 | |
| gmann | alex_xu: ping | 08:15 |
| *** mlakat has joined #openstack-nova | 08:18 | |
| *** mlakat has quit IRC | 08:18 | |
| *** mlakat has joined #openstack-nova | 08:18 | |
| *** mlakat has quit IRC | 08:18 | |
| *** mlakat has joined #openstack-nova | 08:19 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 08:19 |
| *** zenoway has joined #openstack-nova | 08:20 | |
| openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | 08:22 |
| alex_xu | gmann: pong | 08:22 |
| *** jangutter has joined #openstack-nova | 08:23 | |
| gmann | alex_xu: on extension removal works | 08:23 |
| alex_xu | gmann: yup | 08:23 |
| gmann | alex_xu: i am doing on falvor - https://github.com/openstack/nova/blob/master/setup.cfg#L99-L103 | 08:24 |
| gmann | alex_xu: to avoid any conflict in case we end up doing on same APIs | 08:24 |
| alex_xu | gmann: thanks :) | 08:24 |
| alex_xu | gmann: do you think we are worth to sync at somewhere? | 08:24 |
| alex_xu | like etherpad | 08:25 |
| *** ljjjustin has joined #openstack-nova | 08:25 | |
| alex_xu | or it is just fine, we can ping eachother anytime | 08:25 |
| gmann | alex_xu: may be or like i start from end and you from starting in setup.cfg | 08:25 |
| gmann | alex_xu: or etherpad also fine for me we can keep entering name on APIs we start | 08:26 |
| alex_xu | gmann: ok, how about etherpad, we already have one https://etherpad.openstack.org/p/api-no-more-extensions-pike | 08:27 |
| gmann | alex_xu: yup, writing there :) | 08:27 |
| gmann | alex_xu: added flavor i should be able to push patch today | 08:28 |
| alex_xu | gmann: cool, thanks :) | 08:28 |
| *** garyk has quit IRC | 08:29 | |
| gmann | alex_xu: np!, thank you | 08:29 |
| *** abhishek has joined #openstack-nova | 08:35 | |
| *** Shunli has joined #openstack-nova | 08:35 | |
| *** yassine has joined #openstack-nova | 08:38 | |
| *** yassine is now known as Guest58097 | 08:39 | |
| *** thorst has joined #openstack-nova | 08:41 | |
| *** zenoway has quit IRC | 08:42 | |
| *** zenoway has joined #openstack-nova | 08:42 | |
| *** belmoreira has quit IRC | 08:46 | |
| *** rmart04 has quit IRC | 08:46 | |
| *** priteau has joined #openstack-nova | 08:47 | |
| *** cdent has joined #openstack-nova | 08:47 | |
| *** belmoreira has joined #openstack-nova | 08:47 | |
| *** huanxie has quit IRC | 08:49 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Deprecate os-hosts API https://review.openstack.org/456504 | 08:50 |
| openstackgerrit | jichenjc proposed openstack/nova master: Update inc to deprecate os-hosts https://review.openstack.org/456505 | 08:50 |
| *** takashin_ has joined #openstack-nova | 08:51 | |
| *** coreywright has quit IRC | 08:51 | |
| -openstackstatus- NOTICE: zuul was restarted due to an unrecoverable disconnect from gerrit. If your change is missing a CI result and isn't listed in the pipelines on http://status.openstack.org/zuul/ , please recheck | 08:51 | |
| *** takashin has quit IRC | 08:53 | |
| *** Jack_Iv has quit IRC | 08:53 | |
| *** belmoreira has quit IRC | 08:56 | |
| *** zenoway has quit IRC | 08:56 | |
| *** zenoway has joined #openstack-nova | 08:56 | |
| *** huanxie has joined #openstack-nova | 08:56 | |
| *** ratailor|Lunch is now known as ratailor | 08:56 | |
| *** takashin has joined #openstack-nova | 08:57 | |
| *** takashin_ has quit IRC | 08:57 | |
| *** aarefiev_afk is now known as aarefiev | 08:57 | |
| *** tuanluong_ has joined #openstack-nova | 08:58 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Check instance existing before check in mapping https://review.openstack.org/433470 | 08:58 |
| *** thorst has quit IRC | 08:59 | |
| *** huanxie has quit IRC | 08:59 | |
| *** efoley_ has joined #openstack-nova | 08:59 | |
| *** Jack_Iv has joined #openstack-nova | 08:59 | |
| *** tuanluong has quit IRC | 09:00 | |
| *** efoley__ has quit IRC | 09:02 | |
| cdent | bauzas, johnthetubaguy, alex_xu: if you guys have some input on this spec about changing putting resource classes, that would be appreciated. it's a bit stalled, but the associated code is mostly done and useful: https://review.openstack.org/#/c/453732/ | 09:03 |
| bauzas | okay | 09:03 |
| johnthetubaguy | cdent: is that part of making things more efficient / less chatty? | 09:04 |
| cdent | yes | 09:04 |
| johnthetubaguy | I am sold then, I will take a peak at that today | 09:04 |
| cdent | thanks | 09:04 |
| *** ljjjustin has quit IRC | 09:05 | |
| *** sambetts|afk is now known as sambetts | 09:06 | |
| alex_xu | cdent: got it | 09:07 |
| cdent | thanks | 09:07 |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification https://review.openstack.org/455801 | 09:07 |
| * cdent changes location | 09:07 | |
| *** cdent has quit IRC | 09:07 | |
| alex_xu | cdent: one more for you https://review.openstack.org/450125, it is last one for traits API, i'm afraid people forget that patch | 09:07 |
| alex_xu | ok, I sent message late one second | 09:08 |
| takashin | johnthetubaguy: Would you review https://review.openstack.org/#/c/407237/ again? | 09:08 |
| johnthetubaguy | takashin: yes, I must get to that one again, thanks for the update | 09:09 |
| takashin | johnthetubaguy: Thank you. | 09:09 |
| *** coreywright has joined #openstack-nova | 09:10 | |
| *** Guest58097 has quit IRC | 09:12 | |
| *** Guest58097 has joined #openstack-nova | 09:13 | |
| jangutter | Hi, in the spirit of last-minute mad dashes, we finally obtained the clearance to submit the w-i-p code for the spec: https://review.openstack.org/#/c/456386/ | 09:15 |
| jangutter | Would anyone here with SR-IOV and/or OVS experience be interested in reviewing? | 09:16 |
| openstackgerrit | jichenjc proposed openstack/nova master: Deprecate os-hosts API https://review.openstack.org/456504 | 09:16 |
| openstackgerrit | jichenjc proposed openstack/nova master: Update inc to deprecate os-hosts https://review.openstack.org/456505 | 09:16 |
| *** lpetrut has joined #openstack-nova | 09:21 | |
| *** jaosorior has quit IRC | 09:26 | |
| *** yamahata has quit IRC | 09:30 | |
| *** jaosorior has joined #openstack-nova | 09:30 | |
| *** tbachman has joined #openstack-nova | 09:30 | |
| *** Jack_Iv has quit IRC | 09:32 | |
| *** trinaths has quit IRC | 09:32 | |
| *** trinaths has joined #openstack-nova | 09:32 | |
| *** Jack_Iv has joined #openstack-nova | 09:33 | |
| *** mvk has joined #openstack-nova | 09:38 | |
| openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add doc for resource provider usages https://review.openstack.org/450105 | 09:38 |
| *** takashin has left #openstack-nova | 09:38 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: encryptors: Switch to os-brick encryptor classes https://review.openstack.org/391597 | 09:41 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: encryptors: Do not overwrite file volumes on attach_volume https://review.openstack.org/452938 | 09:42 |
| lyarwood | mdbooth: ^ if you have time this morning, I'm looking to land the bugfix for attaching encrypted files so I can backport it _before_ the change to rm -rf the code from our codebase. | 09:43 |
| mdbooth | lyarwood: If my connection stays up long enough | 09:43 |
| lyarwood | mdbooth: cheers | 09:44 |
| openstackgerrit | Stephen Finucane proposed openstack/os-vif master: doc: Add glossary https://review.openstack.org/452666 | 09:44 |
| openstackgerrit | Stephen Finucane proposed openstack/os-vif master: docs: Stop building anything but html output https://review.openstack.org/456535 | 09:44 |
| *** udesale__ has joined #openstack-nova | 09:50 | |
| *** cdent has joined #openstack-nova | 09:50 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Fix HTTP 500 raised for getConsoleLog for stopped instance https://review.openstack.org/455183 | 09:51 |
| *** udesale has quit IRC | 09:51 | |
| cdent | alex_xu: thanks for the remind on the sync command. I thought I had commented on that already,but I must have dismissed it without saving. Will look at it asap | 09:52 |
| *** NikhilS has quit IRC | 09:55 | |
| *** cNilesh has quit IRC | 09:59 | |
| mdbooth | lyarwood: Just thinking about https://review.openstack.org/#/c/452938/ | 10:04 |
| lyarwood | mdbooth: ah the hackaround | 10:04 |
| mdbooth | I don't like that either, tbh, because I don't think Nova should be messing with volumes owned by cinder | 10:04 |
| *** tovin07_ has quit IRC | 10:04 | |
| mdbooth | How about we just explicitly don't support it? | 10:04 |
| mdbooth | It wouldn't be a regression, as it previously blew away your data anyway | 10:05 |
| lyarwood | mdbooth: yeah it's hackaround or block it tbh | 10:05 |
| lyarwood | mdbooth: there are downstream users jumping up and down asking for this | 10:05 |
| mdbooth | I think the fix should be to pass a different path to qemu | 10:05 |
| mdbooth | lyarwood: Hmm, we're still risking their data, though | 10:06 |
| mdbooth | Surely cinder operations are going to be broken while Nova is doing its thing | 10:06 |
| mdbooth | For eg | 10:06 |
| lyarwood | mdbooth: yeah with this renaming they will be | 10:07 |
| mdbooth | Ideally we'd leave it alone entirely, create a dev mapper device on top of it, then use the dev mapper device instead of the original device, which remains untouched | 10:07 |
| mdbooth | IIRC we're only doing this hack because the existing code doesn't allow the encryptor to modify the path used by other Nova code | 10:08 |
| lyarwood | mdbooth: yeah because we do all of this _after_ creating the libvirtdiskconfig objects iirc | 10:09 |
| mdbooth | I'm really not comfortable with Nova moving cinder volumes around physically, especially when we don't tell cinder about it | 10:09 |
| lyarwood | ack that's ah great point that I missed tbh | 10:09 |
| lyarwood | a* | 10:09 |
| mdbooth | What's the plan for a real fix, btw? | 10:10 |
| *** sdague has joined #openstack-nova | 10:10 | |
| lyarwood | well, my QEMU 2.6 native LUKS spec landed yesterday so that's one way of `fixing` this | 10:10 |
| *** MarkMielke has quit IRC | 10:11 | |
| *** MarkMielke has joined #openstack-nova | 10:11 | |
| mdbooth | lyarwood: Of course. | 10:12 |
| lyarwood | really not sure what we can do for the existing encryptors aside from block this for file based volumes for now | 10:12 |
| mdbooth | lyarwood: Honestly my short term fix would be to raise an unsupported exception instead of deleting the user's volume | 10:13 |
| *** Jack_Iv has quit IRC | 10:13 | |
| *** Jack_Iv has joined #openstack-nova | 10:14 | |
| mdbooth | lyarwood: Long term qemu native luks | 10:14 |
| *** tuanluong_ has quit IRC | 10:14 | |
| lyarwood | mdbooth: cool, I'll post a new change for that later today. | 10:14 |
| mdbooth | If we're going to fix it as it is, I think a Nova refactor to allow us to switch out the path | 10:14 |
| *** liujiong has quit IRC | 10:15 | |
| mdbooth | Disgusting hack potential: change the nfs volume driver to use a loopback device instead of the file directly | 10:15 |
| * mdbooth hasn't fully thought that through, btw, just throwing it out there | 10:16 | |
| *** zhurong has quit IRC | 10:17 | |
| *** trinaths has left #openstack-nova | 10:17 | |
| *** phuongnh has quit IRC | 10:18 | |
| *** satyar has quit IRC | 10:23 | |
| *** damien_r has joined #openstack-nova | 10:25 | |
| johnthetubaguy | cdent: so the PUT request has no body in your spec, is that correct? | 10:28 |
| cdent | johnthetubaguy: that's right, because there's nothing useful to put in it | 10:28 |
| *** nicolasbock has joined #openstack-nova | 10:28 | |
| johnthetubaguy | cdent: saves the need to check it matches the URL or fail, makes sense I think | 10:29 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549 | 10:30 |
| cdent | johnthetubaguy, bauzas: I'll make a new version in few minutes to include the info that bauzas asked for | 10:31 |
| *** udesale has joined #openstack-nova | 10:35 | |
| *** udesale__ has quit IRC | 10:36 | |
| sfinucan | cdent: In the absence of jaypipes (wherever he's gone to), could you look at https://review.openstack.org/#/c/361140/ again and see if there's any must-have changes needed today? | 10:40 |
| cdent | sfinucan: sure, will do right after fixing this other spec | 10:41 |
| sfinucan | cdent: spot on | 10:41 |
| *** zhurong has joined #openstack-nova | 10:45 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549 | 10:51 |
| sfinucan | pkoniszewski: You around? Question on the 'live_migration_scheme' and 'live_migraton_inbound_addr' options | 10:53 |
| pkoniszewski | sfinucan: yes, what's up? | 10:53 |
| openstackgerrit | Chris Dent proposed openstack/nova-specs master: A spec for idempotent custom resource class creation https://review.openstack.org/453732 | 10:54 |
| sfinucan | pkoniszewski: What was your rationale behind df334b4 ? | 10:54 |
| sfinucan | I'm guessing it's a usability thing, but was there anything more subtle than that? | 10:54 |
| sfinucan | pkoniszewski: https://github.com/openstack/nova/commit/df334b4f414b8c341709df37ca10065d3b50fcef for reference | 10:54 |
| *** gszasz has joined #openstack-nova | 10:56 | |
| *** thorst has joined #openstack-nova | 10:56 | |
| pkoniszewski | sfinucan: so my point was to deprecate and finally remove live_migration_uri which is used for hacking the interface used for live migrations, now there is live_migration_inbound_addr to choose an interface | 10:56 |
| pkoniszewski | also rationale behind df334b4 was to keep an option to change scheme for live migration, some of operators use SSH instead of TCP | 10:56 |
| pkoniszewski | i didn't want to force anyone to switch to TCP from SSH | 10:56 |
| sfinucan | pkoniszewski: Sounds fair. Unfortunately though some operators also use extraparamters, e.g. '?command=xyz' https://bugs.launchpad.net/nova/+bug/1671288 | 10:57 |
| openstack | Launchpad bug 1671288 in OpenStack Compute (nova) "live_migration_uri to live_migration_scheme SSH settings" [Undecided,Confirmed] - Assigned to Stephen Finucane (stephenfinucane) | 10:57 |
| pkoniszewski | yeah, i noticed that like two weeks ago | 10:58 |
| pkoniszewski | so it's a trade off right now | 10:58 |
| sfinucan | pkoniszewski: trade-off between...? :) | 10:58 |
| sfinucan | pkoniszewski: I was going to add a 'live_migration_extraparamters' DictOpt to allow folks to specify those options if they want them | 11:00 |
| pkoniszewski | keep live_migration_uri and document how it can be used/let people google how to use it, or implement a few configuration options to cover operators use cases (like SSH username and key) and remove live_migration_uri entirely | 11:00 |
| pkoniszewski | or a DictOpt, it also sounds good | 11:01 |
| sfinucan | ...but then I though, wouldn't it be easier to keep live_migration_uri and mark it as 'advanced' | 11:01 |
| sfinucan | https://docs.openstack.org/developer/oslo.config/opts.html#oslo_config.cfg.Opt | 11:01 |
| *** erlon has joined #openstack-nova | 11:01 | |
| *** thorst has quit IRC | 11:01 | |
| sfinucan | pkoniszewski: seems we're weighing up similar trade-offs so :) | 11:01 |
| *** jangutter has quit IRC | 11:02 | |
| *** winston-d_ has quit IRC | 11:03 | |
| johnthetubaguy | sfinucan: pkoniszewski: I was thinking about that before as well | 11:03 |
| johnthetubaguy | honestly, folks seem used to using the URL, maybe we just keep it | 11:03 |
| johnthetubaguy | and add a description about how you change SSH params in there | 11:03 |
| sfinucan | right, I'll revert the deprecation so, mark it as advanced, and expand the help with a link to the libvirt docs | 11:04 |
| sfinucan | that'll keep both standard and "advanced" operators happy | 11:04 |
| sfinucan | with minimal code changes (read: easy backporting) | 11:04 |
| johnthetubaguy | I am tempted to include the SSH case in the description | 11:04 |
| sfinucan | johnthetubaguy: e.g.? | 11:05 |
| johnthetubaguy | makes sense to have the same key for migration and livemigration | 11:05 |
| johnthetubaguy | sfinucan: it was in one of the bugs somewhere | 11:05 |
| johnthetubaguy | sfinucan: here we go live_migration_uri = qemu+ssh://nova@%s/system?keyfile=/var/lib/nova/.ssh/id_rsa&no_tty=1&no_verify=1 | 11:05 |
| *** mlakat has quit IRC | 11:06 | |
| *** efoley__ has joined #openstack-nova | 11:06 | |
| *** abalutoiu has quit IRC | 11:08 | |
| * sfinucan reads the libvirt docs and wonders why we don't default to 'no_tty' | 11:08 | |
| *** efoley_ has quit IRC | 11:09 | |
| * johnthetubaguy goes for some lunch | 11:10 | |
| *** abalutoiu has joined #openstack-nova | 11:10 | |
| johnthetubaguy | sfinucan: yeah, maybe leave those stranger bits off the example | 11:10 |
| * johnthetubaguy is not a fan of no_verify | 11:11 | |
| *** Jack_Iv has quit IRC | 11:12 | |
| *** djohnsto has quit IRC | 11:15 | |
| *** damien_r has quit IRC | 11:17 | |
| *** Jack_Iv has joined #openstack-nova | 11:18 | |
| *** jamielennox|away is now known as jamielennox | 11:18 | |
| *** lpetrut has quit IRC | 11:20 | |
| *** claudiub has quit IRC | 11:22 | |
| openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | 11:26 |
| *** Jack_Iv has quit IRC | 11:27 | |
| voelzmo | Hey there | 11:27 |
| voelzmo | A while back, there was some blueprint about updateable metadata for servers. Is that still a thing that's being pursued? | 11:28 |
| *** Jack_Iv has joined #openstack-nova | 11:29 | |
| *** dixiaoli has quit IRC | 11:31 | |
| voelzmo | s/metadata/userdata/ | 11:31 |
| voelzmo | I guess this might be the one https://blueprints.launchpad.net/nova/+spec/userdata-modification | 11:31 |
| openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | 11:32 |
| *** lpetrut has joined #openstack-nova | 11:33 | |
| *** lucasagomes is now known as lucas-hungry | 11:34 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Don't deprecate the 'live_migration_uri' https://review.openstack.org/456571 | 11:35 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Gather 'live_migration_scheme', 'live_migration_inbound_addr' https://review.openstack.org/456572 | 11:35 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Convert 'live_migration_inbound_addr' to HostAddressOpt https://review.openstack.org/456573 | 11:35 |
| sfinucan | johnthetubaguy, pkoniszewski, owalsh: ^ | 11:35 |
| *** iceyao has joined #openstack-nova | 11:38 | |
| *** ociuhandu has quit IRC | 11:38 | |
| *** Shunli has quit IRC | 11:39 | |
| *** Jack_Iv has quit IRC | 11:40 | |
| *** Shunli has joined #openstack-nova | 11:41 | |
| *** Jack_Iv has joined #openstack-nova | 11:42 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: encryptors: Switch to os-brick encryptor classes https://review.openstack.org/391597 | 11:43 |
| openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | 11:45 |
| *** yingjun has joined #openstack-nova | 11:46 | |
| *** smatzek has joined #openstack-nova | 11:48 | |
| *** iceyao has quit IRC | 11:49 | |
| EmilienM | hey folks, this patch is really critical for TripleO CI to run correctly again: https://review.openstack.org/#/c/456227/ | 11:50 |
| EmilienM | https://bugs.launchpad.net/tripleo/+bug/1682001 | 11:51 |
| openstack | Launchpad bug 1682001 in tripleo "[promotion] nova hypervisors/detail fails when host is not mapped to any cell" [Critical,In progress] - Assigned to Oliver Walsh (owalsh) | 11:51 |
| *** iceyao has joined #openstack-nova | 11:52 | |
| *** fandi has quit IRC | 11:53 | |
| *** thorst has joined #openstack-nova | 11:53 | |
| *** fandi has joined #openstack-nova | 11:55 | |
| *** sridharg has quit IRC | 11:56 | |
| *** udesale has quit IRC | 11:56 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavors-extraspecs endpoint instead of stevedore https://review.openstack.org/456585 | 11:57 |
| *** iceyao has quit IRC | 11:58 | |
| *** mtanino has joined #openstack-nova | 11:59 | |
| *** iceyao has joined #openstack-nova | 11:59 | |
| *** damien_r has joined #openstack-nova | 12:03 | |
| *** ratailor has quit IRC | 12:03 | |
| *** jamielennox is now known as jamielennox|away | 12:05 | |
| *** lpetrut has quit IRC | 12:05 | |
| *** peter-hamilton has joined #openstack-nova | 12:07 | |
| openstackgerrit | NidhiMittalHada proposed openstack/python-novaclient master: Nova volume-attach doesn't care for device name https://review.openstack.org/454048 | 12:07 |
| peter-hamilton | johnthetubaguy: if you have any further questions on the cert validation spec just ping me | 12:09 |
| johnthetubaguy | peter-hamilton: cool, I should hit that one now | 12:09 |
| *** edmondsw has joined #openstack-nova | 12:11 | |
| *** faizy has quit IRC | 12:11 | |
| *** artom has quit IRC | 12:12 | |
| *** artom has joined #openstack-nova | 12:12 | |
| *** cdent has quit IRC | 12:12 | |
| *** lpetrut has joined #openstack-nova | 12:13 | |
| *** artom has quit IRC | 12:14 | |
| johnthetubaguy | peter-hamilton: do we want to return a 400 error when a user gives us a list of certs, but our cert checking is turned off? | 12:14 |
| *** artom has joined #openstack-nova | 12:14 | |
| *** amotoki has quit IRC | 12:15 | |
| peter-hamilton | johnthetubaguy: good question; does it make sense to throw a warning and store the certs with the instance anyway? | 12:16 |
| johnthetubaguy | peter-hamilton: I am bit worried the user would assume the cloud has done the verification, but it hasn't actually done it | 12:17 |
| peter-hamilton | johnthetubaguy: that way cert validation could be enabled at a later date and boot would still work? | 12:17 |
| johnthetubaguy | although we do allow folks to specify the cert list when the image doesn't require verification anyways | 12:17 |
| johnthetubaguy | its probably more consistent to just log a warning | 12:17 |
| johnthetubaguy | I mean the user doesn't see that I guess | 12:18 |
| peter-hamilton | johnthetubaguy: i could go either way i think | 12:18 |
| *** lucas-hungry is now known as lucasagomes | 12:18 | |
| johnthetubaguy | peter-hamilton: honestly, me too | 12:18 |
| johnthetubaguy | wonders which side of the fence to sit | 12:18 |
| peter-hamilton | johnthetubaguy: should we hash this out during implementation? | 12:18 |
| *** nicolasbock has quit IRC | 12:19 | |
| johnthetubaguy | peter-hamilton: thats probably the right call | 12:19 |
| peter-hamilton | johnthetubaguy: i'll make a note of it for when i get to the API patches | 12:20 |
| *** mtanino has quit IRC | 12:23 | |
| *** nicolasbock has joined #openstack-nova | 12:25 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavors-access endpoint instead of stevedore https://review.openstack.org/456589 | 12:26 |
| *** gongysh has quit IRC | 12:26 | |
| *** mdrabe has joined #openstack-nova | 12:27 | |
| *** gongysh has joined #openstack-nova | 12:27 | |
| *** gongysh has quit IRC | 12:27 | |
| *** Daviey has quit IRC | 12:28 | |
| *** annegentle has joined #openstack-nova | 12:28 | |
| *** bpoulos has joined #openstack-nova | 12:28 | |
| *** amotoki has joined #openstack-nova | 12:31 | |
| openstackgerrit | Yasufumi Ogawa proposed openstack/nova-specs master: Merge "Libvirt options for DPDK" https://review.openstack.org/456593 | 12:31 |
| *** iceyao has quit IRC | 12:33 | |
| *** annegentle has quit IRC | 12:33 | |
| *** iceyao has joined #openstack-nova | 12:34 | |
| *** Jack_Iv has quit IRC | 12:34 | |
| *** Shunli has quit IRC | 12:35 | |
| *** Shunli has joined #openstack-nova | 12:36 | |
| *** gongysh has joined #openstack-nova | 12:38 | |
| *** annegentle has joined #openstack-nova | 12:38 | |
| *** mriedem has joined #openstack-nova | 12:38 | |
| *** ayogi has quit IRC | 12:39 | |
| *** dane-fichter has joined #openstack-nova | 12:42 | |
| dane-fichter | johnthetubaguy: Can you take a look at peter's spec when you get a chance? https://review.openstack.org/#/c/357151/ | 12:43 |
| peter-hamilton | dane-fichter: he's already on it :) | 12:44 |
| johnthetubaguy | dane-fichter: yeah, have thinking through that now :) | 12:44 |
| dane-fichter | oh lol sorry | 12:44 |
| johnthetubaguy | no problem, its the right day to chase me down :) | 12:44 |
| openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | 12:44 |
| peter-hamilton | johnthetubaguy: awesome, thanks for the +2! | 12:47 |
| *** amotoki has quit IRC | 12:47 | |
| *** awaugama has joined #openstack-nova | 12:47 | |
| johnthetubaguy | peter-hamilton: no problem, glad we are getting there, its a good feature folks need! | 12:48 |
| peter-hamilton | johnthetubaguy: i agree :) | 12:48 |
| peter-hamilton | johnthetubaguy: now, where is jaypipes... | 12:48 |
| *** iceyao has quit IRC | 12:48 | |
| *** cleong has joined #openstack-nova | 12:48 | |
| johnthetubaguy | hopefully he is having a coffee, putting on his flip flops, and about to open his laptop | 12:49 |
| *** Jack_Iv has joined #openstack-nova | 12:50 | |
| *** Jack_Iv has quit IRC | 12:50 | |
| *** Jack_Iv has joined #openstack-nova | 12:50 | |
| *** iceyao has joined #openstack-nova | 12:50 | |
| *** sofie_ has quit IRC | 12:51 | |
| *** baoli has joined #openstack-nova | 12:54 | |
| *** Daviey has joined #openstack-nova | 12:55 | |
| *** lyan has joined #openstack-nova | 12:55 | |
| *** mlakat has joined #openstack-nova | 12:56 | |
| openstackgerrit | Yasufumi Ogawa proposed openstack/nova-specs master: Merge "Libvirt options for DPDK" https://review.openstack.org/456601 | 12:57 |
| *** Shunli has quit IRC | 13:00 | |
| johnthetubaguy | sean-k-mooney: sfinucan: any chance we could get use cases added in here quickly? https://review.openstack.org/#/c/361140 | 13:01 |
| sfinucan | johnthetubaguy: Sure. Gimme 10 | 13:01 |
| johnthetubaguy | sean-k-mooney: sfinucan: something to describe the scenario(s) where each type of behaviour is what you want | 13:01 |
| johnthetubaguy | sfinucan: sweet, I have got all confused after re-reading them, particular around the good points cdent raised in there | 13:02 |
| *** lpetrut has quit IRC | 13:03 | |
| *** annegentle has quit IRC | 13:03 | |
| *** efried has joined #openstack-nova | 13:04 | |
| *** liverpooler has joined #openstack-nova | 13:05 | |
| sfinucan | johnthetubaguy: Dumb question, but is there another name of the service that boots a VM other than 'nova-compute'? | 13:06 |
| johnthetubaguy | sfinucan: hmm, duno really, n-cpu? | 13:07 |
| sfinucan | i.e. we'd say "the scheduler" rather than 'nova-scheduler' | 13:07 |
| *** eharney has quit IRC | 13:07 | |
| johnthetubaguy | "the compute node" gets used a bit | 13:07 |
| johnthetubaguy | or host | 13:07 |
| johnthetubaguy | or just "in compute" | 13:07 |
| johnthetubaguy | not sure thats a good thing though | 13:08 |
| *** Guest3412 has joined #openstack-nova | 13:08 | |
| sfinucan | maybe I should just use nova-compute and nova-scheduler consistently instead | 13:09 |
| *** jcook has quit IRC | 13:09 | |
| *** esberglu has joined #openstack-nova | 13:13 | |
| *** jcook has joined #openstack-nova | 13:14 | |
| johnthetubaguy | sfinucan: I guess thats always clear | 13:14 |
| *** satyar has joined #openstack-nova | 13:15 | |
| *** crushil has joined #openstack-nova | 13:15 | |
| *** Guest58097 has quit IRC | 13:16 | |
| bauzas | sfinucan: if you talk about a running service having a RPC MQ, it's nova-compute | 13:17 |
| *** amotoki has joined #openstack-nova | 13:17 | |
| bauzas | sfinucan: if you talk about an hypervisor, it's a node | 13:17 |
| *** kfarr has joined #openstack-nova | 13:19 | |
| *** fandi has quit IRC | 13:21 | |
| *** artom has quit IRC | 13:21 | |
| *** tbachman has quit IRC | 13:21 | |
| *** artom has joined #openstack-nova | 13:22 | |
| *** artom has quit IRC | 13:24 | |
| *** artom has joined #openstack-nova | 13:24 | |
| *** djohnsto has joined #openstack-nova | 13:24 | |
| *** Guest58097 has joined #openstack-nova | 13:25 | |
| *** vks1 has joined #openstack-nova | 13:25 | |
| *** eharney has joined #openstack-nova | 13:28 | |
| *** gouthamr has joined #openstack-nova | 13:28 | |
| *** esberglu has quit IRC | 13:29 | |
| *** links has quit IRC | 13:29 | |
| *** iceyao has quit IRC | 13:29 | |
| *** baoli has quit IRC | 13:29 | |
| *** iceyao has joined #openstack-nova | 13:30 | |
| *** smatzek has quit IRC | 13:31 | |
| *** cdent has joined #openstack-nova | 13:33 | |
| *** lpetrut has joined #openstack-nova | 13:34 | |
| *** jaypipes has joined #openstack-nova | 13:34 | |
| *** ociuhandu has joined #openstack-nova | 13:35 | |
| peter-hamilton | jaypipes: if you have any questions on cert validation just ping me | 13:37 |
| jaypipes | peter-hamilton: will do. was on PTO yesterday. catching up today. | 13:37 |
| mriedem | johnthetubaguy: as you probably saw i marked up your remove scope check patch, let me know if you want to talk about it | 13:38 |
| johnthetubaguy | mriedem: so I didn't see that, looking now | 13:39 |
| mriedem | you might want to sit down | 13:39 |
| johnthetubaguy | funny you say that, I just adjusted my desk to sit down mode | 13:40 |
| *** tbachman has joined #openstack-nova | 13:41 | |
| *** Dinesh_Bhor has quit IRC | 13:41 | |
| *** Dinesh_Bhor has joined #openstack-nova | 13:43 | |
| * artom has a problem. | 13:44 | |
| *** catintheroof has joined #openstack-nova | 13:44 | |
| *** zz_dimtruck is now known as dimtruck | 13:44 | |
| artom | For tagged attach, I need to invalidate the metadata API's cache - however, as far as I can tell, its cache is just a dict local to that process. | 13:45 |
| *** catintheroof has quit IRC | 13:45 | |
| artom | I'm trying to thinking of a good way to solve that, without suddenly requiring all deployment to run memcache.' | 13:45 |
| *** catintheroof has joined #openstack-nova | 13:45 | |
| bauzas | jaypipes: in case you have some brain time free, I'd love to get your insights on https://review.openstack.org/#/c/450122/15 | 13:46 |
| bauzas | jaypipes: my biggest concern is how a virt driver can return a single dict for a common RC if it has two distinct devices behind it | 13:46 |
| bauzas | unless it's not really a problem and then meh. | 13:47 |
| *** coreycb_vaca is now known as coreycb | 13:47 | |
| mriedem | artom: i wonder if we could set a flag in the db to signal when the cache should be dumped | 13:48 |
| mriedem | and only look it up when requesting a version new enough for the tagged attach stuff, | 13:48 |
| *** esberglu has joined #openstack-nova | 13:48 | |
| mriedem | although i think with config drive that would be every time | 13:48 |
| artom | mriedem, it's not in the DB anywhere... | 13:48 |
| mriedem | artom: i know | 13:48 |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: PCI NUMA Policies https://review.openstack.org/361140 | 13:48 |
| *** baoli has joined #openstack-nova | 13:48 | |
| sfinucan | johnthetubaguy: & | 13:48 |
| mriedem | artom: is the cache per instance? | 13:49 |
| mriedem | i'm not terribly familiar with this | 13:49 |
| artom | mriedem, I think it's per process | 13:49 |
| *** esberglu has quit IRC | 13:49 | |
| artom | https://github.com/openstack/nova/blob/master/nova/cache_utils.py | 13:49 |
| johnthetubaguy | sfinucan: cool, queued it | 13:49 |
| *** esberglu has joined #openstack-nova | 13:49 | |
| sfinucan | Also: cdent, sean-k-mooney, snikitin_, sgordon - https://review.openstack.org/361140 | 13:49 |
| sfinucan | I went with required, preferred and legacy | 13:49 |
| *** mlavalle has joined #openstack-nova | 13:49 | |
| artom | mriedem, the metadata handler calls get_client(), which, depending on conf, configures a cache region in a certain way | 13:49 |
| *** Guest3412 has quit IRC | 13:50 | |
| *** catintheroof has quit IRC | 13:50 | |
| artom | It's using dogpile.cache, in which a region is basically a frontend/client: https://dogpilecache.readthedocs.io/en/latest/usage.html | 13:50 |
| sfinucan | jaypipes: You don't get to escape either :P https://review.openstack.org/361140 | 13:50 |
| * sfinucan is happy to see jaypipes online. He has so much work for him | 13:51 | |
| cdent | ha | 13:51 |
| *** catintheroof has joined #openstack-nova | 13:51 | |
| artom | And I think the backend is basically a dict: https://github.com/openstack/oslo.cache/blob/master/oslo_cache/backends/dictionary.py | 13:51 |
| artom | So unless the REST API request just happened to land on the same process that eventually handles the metadata request, we need some form of IPC | 13:51 |
| mriedem | artom: default cache is just in-memory per process dict? | 13:52 |
| mriedem | i think it's dogpile yeah? | 13:52 |
| artom | mriedem, that's what I gather, yeah | 13:52 |
| *** Drankis has joined #openstack-nova | 13:54 | |
| mriedem | well this is a sticky wicket | 13:55 |
| *** dimtruck is now known as zz_dimtruck | 13:55 | |
| mriedem | we have to look up an instance when it's not cache, or the cache is expired | 13:55 |
| mriedem | i was thinking we could have set a flag in instance_extra that would tell metadata api to refresh the cache, but that defeats the purpose because you have to do the db lookup to find out | 13:55 |
| mriedem | default cache expiration is 15 seconds right? | 13:56 |
| snikitin_ | sfinucan: wow. Thank you for update! btw 'legacy' looks good because this behavior it really a little bit strange | 13:56 |
| mriedem | does the guest need to see the tagged block device immediately? | 13:56 |
| artom | mriedem, ah, you're saying just wait it out? | 13:57 |
| frickler | mriedem: regarding your ML question, johnthetubaguy has a patch up to clarify things: https://review.openstack.org/#/c/438887/3 | 13:57 |
| mriedem | artom: that's what i'm asking | 13:57 |
| *** zhurong has quit IRC | 13:57 | |
| mriedem | artom: i.e. this is already a latent issue right? | 13:57 |
| sfinucan | snikitin_: no problem | 13:57 |
| *** mtanino has joined #openstack-nova | 13:57 | |
| mriedem | if your guest is relying on changes via metadata api, and it's already cached on a timer, then your guest logic probably already needs to poll for changes | 13:57 |
| mriedem | i don't really know the use cases though | 13:58 |
| sfinucan | snikitin_: Personally I'd like to drop that policy because it makes no sense. If you care about NUMA affinity, you should have 100% NUMA-aware nodes. If you don't, you clearly don't care enough :) | 13:58 |
| johnthetubaguy | frickler: mriedem: thats right, I new that was familiar, I think bauzas said he would fix up that patch. | 13:58 |
| *** yingjun has quit IRC | 13:58 | |
| sfinucan | But this is a good half way measure to avoid breaking existing deployments | 13:58 |
| artom | mriedem, I suppose it is... but... well, did we ever have a user case where the metadata changed during an instance's lifetime? | 13:58 |
| mriedem | artom: i'm sure there are all sorts of things that change | 13:59 |
| snikitin_ | sfinucan: yea, me too. But this damn backward compatibility... | 13:59 |
| *** fragatina has joined #openstack-nova | 13:59 | |
| mriedem | but let me check | 13:59 |
| bauzas | johnthetubaguy: it just went out of my radar, because I'm not the Gerrit owner :( | 13:59 |
| johnthetubaguy | bauzas: yeah, I have done that a few times also | 13:59 |
| mriedem | artom: security groups could change, password could change | 13:59 |
| sfinucan | snikitin_: I know, right? How dare our customers ask for backwards compatibility :P Let's hope that naming clears up everything | 13:59 |
| mriedem | artom: network metadata could change couldn't it depending on attaching/detaching ports? | 14:00 |
| mriedem | not to mention dynamic vendor data | 14:00 |
| bauzas | mriedem: if your ML thread is related to my az spec, nevermind it, we could just wait for Queens | 14:00 |
| mriedem | bauzas: it was the impetus yes | 14:00 |
| mriedem | i never knew about default_scheduler_zone | 14:00 |
| artom | mriedem, that's probably correct... | 14:00 |
| bauzas | mriedem: default_schedul/e/_zone but yeah | 14:01 |
| mriedem | artom: so if you accept as a limitation that you're not getting the latest changes immediately when using metadata api, then this is probably ok | 14:01 |
| bauzas | mriedem: hope my reply made things clearer for you | 14:01 |
| mriedem | bauzas: it made me feel like there was a purpose | 14:01 |
| artom | mriedem, I think that's acceptable in the short term, at least | 14:01 |
| mriedem | bauzas: however, the help text on those options is still murky | 14:01 |
| *** Jack_Iv has quit IRC | 14:01 | |
| mriedem | and should probably be updated | 14:01 |
| bauzas | mriedem: yup, hence johnthetubaguy and frickler point | 14:01 |
| mriedem | bauzas: plus, | 14:01 |
| artom | mriedem, and if someone complains we can always do a "real" cache later | 14:01 |
| snikitin_ | sfinucan: :D | 14:01 |
| *** smatzek has joined #openstack-nova | 14:01 | |
| mriedem | this big fat scary warning https://docs.openstack.org/developer/nova/aggregates.html#availability-zones-azs | 14:02 |
| mriedem | should probably be in the api-ref | 14:02 |
| mriedem | where api users would see it | 14:02 |
| mriedem | rather than way down in devref | 14:02 |
| artom | Or they can run memcache and we can add a bit where we invalidate it | 14:02 |
| bauzas | mriedem: well, when I wrote by that time, I wasn't seasoned with api-ref :) | 14:02 |
| mriedem | artom: memcache doesn't help the timing problem right? just the consistency issue, | 14:02 |
| bauzas | it should even be in the operators guide | 14:02 |
| mriedem | i.e. depending on which metadata api worker i hit with my request, i could get different results | 14:02 |
| bauzas | anyway, spec deadline day, I'm turned into review day for me | 14:03 |
| mriedem | or would memcache be updated globally for all? i suppose it would | 14:03 |
| mriedem | nevermind then | 14:03 |
| artom | mriedem, I'm assuming memcache would be a global singleton | 14:03 |
| artom | So it's much easier to invalidate an item from a different place in the code | 14:03 |
| mriedem | bauzas: "Consequently, it is highly recommended for users to never ever ask for booting an instance by specifying an explicit AZ named nova and for operators to never set the AZ metadata for an aggregate to nova." | 14:03 |
| mriedem | ^ is all about the end user | 14:03 |
| mriedem | artom: yeah true | 14:03 |
| mriedem | artom: so yeah if you were a deployment that heavily depended on metadata api, you're probably using memcahe | 14:04 |
| bauzas | mriedem: setting AZ metadata is an operator thing, right? not sure I get your point | 14:04 |
| mriedem | *memcache - that's why the dogpile per-process dict cache is the cheap and easy default | 14:04 |
| mriedem | bauzas: read again "Consequently, it is highly recommended for users to never ever ask for booting an instance by specifying an explicit AZ named nova and for operators to never set the AZ metadata for an aggregate to nova." | 14:04 |
| *** jangutter has joined #openstack-nova | 14:04 | |
| bauzas | mriedem: most of the bugs I see are related to people saying "I did nova boot --az nova" and now I'm stuck | 14:04 |
| mriedem | bauzas: yes, that's exactly what that warning says | 14:04 |
| bauzas | mriedem: or people saying "I did created aggregate with AZ=nova and now I'm broken" | 14:05 |
| mriedem | "don't do nova boot --availability-zone nova" | 14:05 |
| bauzas | mriedem: I know, *I* wrote that big fat warning :)à | 14:05 |
| mriedem | the user doesn't read the operators guide | 14:05 |
| mriedem | https://github.com/openstack/nova/blob/master/api-ref/source/parameters.yaml#L3499 | 14:05 |
| bauzas | ah okay, get your point | 14:05 |
| bauzas | gotcha | 14:06 |
| bauzas | I can forge a patch | 14:06 |
| johnthetubaguy | lbragstad: mriedem: sounds like we need to talk about "is_admin_project:True", I am assuming that basically means god_mode=True | 14:06 |
| *** hongbin has joined #openstack-nova | 14:07 | |
| mriedem | johnthetubaguy: my understanding in talking with lbragstad was is_admin_project was assigning someone as admin to a project, not the deployment | 14:07 |
| mriedem | but it's really confusing so i probably screwed it up | 14:07 |
| mriedem | or is is_admin_project a flag on a project that says, this is the god project, if yo'ure a member, welcome to heaven | 14:08 |
| lbragstad | mriedem that - but the other way around | 14:08 |
| *** lpetrut has quit IRC | 14:08 | |
| lbragstad | the admin_project is this super-duper special project within keystone and if you have admin on it, you have admin on the deployment | 14:08 |
| lbragstad | (it's weird and a little strange) | 14:08 |
| mriedem | so what is the case where i'm just admin on a non-admin project? | 14:09 |
| mriedem | is that just, role:admin on project foo? | 14:09 |
| mriedem | not role:admin on project admin? | 14:09 |
| lbragstad | mriedem yeah | 14:09 |
| *** iceyao has quit IRC | 14:09 | |
| lbragstad | mriedem the case would be that you'd be able to manage the project, but not the deployment | 14:09 |
| mriedem | ok | 14:09 |
| lbragstad | (think creating subprojects) | 14:09 |
| mriedem | johnthetubaguy: i don't know if you need that explained in your spec or not, it would have helped me, and in simple terms | 14:10 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova-specs master: PCI NUMA Policies https://review.openstack.org/361140 | 14:10 |
| mriedem | * god-mode: user as admin role on the special admin project | 14:10 |
| lbragstad | or if you wanted to manage role assignments for things within the project you have admin on | 14:10 |
| johnthetubaguy | mriedem: I tried to describe that in there, I probably failed. Its why I spoke about a "special" admin project. | 14:11 |
| lbragstad | i mean - it's a step in the right direction, but i kinda think of it as a bandaid | 14:11 |
| mriedem | * fief lord: user has admin role on non-admin tenant project | 14:11 |
| *** lucasxu has joined #openstack-nova | 14:11 | |
| johnthetubaguy | mriedem: +1 | 14:11 |
| mriedem | or demigod if you like | 14:11 |
| mriedem | fief lord seems more appropriate | 14:12 |
| lbragstad | yeah | 14:12 |
| mriedem | johnthetubaguy: my main question was can we split fixing the db issue (the trove service user 404 case) from the scope check overhaul? | 14:12 |
| mriedem | because the former i think i understand the mechanics of, the latter i don't so much | 14:13 |
| johnthetubaguy | mriedem: I don't believe we can | 14:13 |
| johnthetubaguy | mriedem: I think one check is there to replace the other | 14:13 |
| mriedem | i figured that would be the answer | 14:13 |
| johnthetubaguy | mriedem: is it worth a quick hangout? | 14:13 |
| mriedem | in that case, you have to answer my gauntlet of questions in the scope check section | 14:13 |
| mriedem | we could, | 14:14 |
| johnthetubaguy | yeah, heading down them now | 14:14 |
| mriedem | would be good if it were more than just me though | 14:14 |
| *** zz_dimtruck is now known as dimtruck | 14:15 | |
| johnthetubaguy | mriedem: very true, I wonder if sdague and lbragstad would be up for it? | 14:15 |
| sdague | ?? | 14:16 |
| lbragstad | ? | 14:16 |
| *** Jack_Iv has joined #openstack-nova | 14:16 | |
| johnthetubaguy | wondering about a quick hangout | 14:17 |
| johnthetubaguy | around policy stuff, with mriedem | 14:17 |
| sdague | sure, give me 2 minutes, then I could do it | 14:17 |
| lbragstad | I have some time now - meeting in 45 | 14:17 |
| johnthetubaguy | lbragstad: same here | 14:17 |
| johnthetubaguy | well, same meeting | 14:17 |
| openstackgerrit | Monty Taylor proposed openstack/python-novaclient master: Add novaclient client_name and client_version to user-agent https://review.openstack.org/456638 | 14:17 |
| lbragstad | johnthetubaguy :) | 14:17 |
| sdague | ok, well let me know when | 14:18 |
| johnthetubaguy | policy hangout: https://hangouts.google.com/call/5zxc7sggwndgrcihsydb5o37mmy | 14:19 |
| johnthetubaguy | mriedem: lbragstad: sdague: ^ | 14:19 |
| mriedem | johnthetubaguy: i'll also be totally honest and say i worry about starting a new big slog of changes like this, when we have some things we haven't completed yet from previous releases | 14:19 |
| mordred | jamielennox|away: ^^ see 456638 and double-check me that that's right before I do it for the other clients | 14:20 |
| johnthetubaguy | mriedem: I kinda thought we finished the conf now, hence opening this up | 14:20 |
| *** iceyao has joined #openstack-nova | 14:20 | |
| mriedem | my av is all screwed up | 14:21 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova-specs master: PCI NUMA Policies https://review.openstack.org/361140 | 14:21 |
| johnthetubaguy | mriedem: your mic was working great | 14:21 |
| *** felipemonteiro has joined #openstack-nova | 14:21 | |
| *** iceyao has quit IRC | 14:21 | |
| mriedem | oh, no audio for me :) | 14:21 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | 14:22 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | 14:22 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | 14:22 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | 14:22 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | 14:22 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | 14:22 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | 14:22 |
| *** claudiub has joined #openstack-nova | 14:23 | |
| *** krtaylor has joined #openstack-nova | 14:25 | |
| *** Jack_Iv has quit IRC | 14:26 | |
| *** udesale has joined #openstack-nova | 14:29 | |
| *** jianghuaw_ has joined #openstack-nova | 14:30 | |
| mriedem | johnthetubaguy: i meant https://blueprints.launchpad.net/nova/+spec/prep-for-network-aware-scheduling-pike specifically, but we have other things too | 14:31 |
| mriedem | sfinucan: dansmith: bauzas: easy fix https://review.openstack.org/#/c/455183/ | 14:33 |
| sfinucan | mriedem: I see your change and raise you https://review.openstack.org/#/c/456227/ | 14:34 |
| * sfinucan doesn't want two RH +2s | 14:34 | |
| mriedem | clarkb: looks like we have a new set of live migration failures since the UCA change http://logs.openstack.org/83/455183/5/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/38fe0cd/logs/screen-n-cpu.txt.gz?level=TRACE#_2017-04-13_11_58_36_986 - not sure if it's related to the other things we've already seen and UCA was meant to fix, i.e. the i/o error | 14:34 |
| bauzas | mriedem: looks like the live-mig job is bonkers | 14:34 |
| mriedem | sfinucan: that tab was already open :) | 14:35 |
| mriedem | it is http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Live%20Migration%20failure%3A%20operation%20failed%3A%20Failed%20to%20connect%20to%20remote%20libvirt%20URI%5C%22%20AND%20message%3A%5C%22End%20of%20file%20while%20reading%20data%3A%20sh%3A%201%3A%20nc%3A%20not%20found%3A%20Input%2Foutput%20error%5C%22%20AND%20tags%3A%5C%22screen-n-cpu.txt%5C%22&from=7d | 14:35 |
| *** lpetrut has joined #openstack-nova | 14:36 | |
| kashyap | sfinucan: I'm obviously not a "core", but there's nothing inherently wrong with "two RH +2s" | 14:37 |
| kashyap | Assuming they're acting in good faith | 14:38 |
| sfinucan | kashyap: True, but it's just good practice | 14:38 |
| kashyap | Having a blind rule (I'm not saying there is) like "We don't want two RH +2s" is just plain silly | 14:38 |
| sfinucan | especially when the patch was RH authored | 14:38 |
| kashyap | sfinucan: Ah, like that | 14:38 |
| kashyap | sfinucan: Sure, it's a "nice to have" to keep biases in check | 14:38 |
| mriedem | i'll review it when i'm off this call | 14:39 |
| *** zenoway has quit IRC | 14:39 | |
| kashyap | But if they're are mature adults, then it's okay to assume they're acting in good faith :-) | 14:39 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova-specs master: PCI NUMA Policies https://review.openstack.org/361140 | 14:39 |
| mriedem | kashyap: they are also rushing because of tripleo pressure | 14:39 |
| mriedem | so there is a conflict | 14:39 |
| mriedem | just hold up 30 minutes please :) | 14:39 |
| sfinucan | kashyap: z'actly :) | 14:39 |
| kashyap | sfinucan: Case in point: Upstream QEMU block layer has lots of RH-authored patches reviewed by RHers | 14:39 |
| *** zenoway has joined #openstack-nova | 14:39 | |
| kashyap | And the quality of those discussions is one of the highest I've ever seen. | 14:39 |
| kashyap | And they disagree all the time with each other | 14:40 |
| jaypipes | bauzas, sfinucan: will try my best. really not feeling well today :( | 14:40 |
| kashyap | mriedem: Ah, I see - didn't know the TripleO context | 14:40 |
| bauzas | jaypipes: no worries man | 14:40 |
| bauzas | jaypipes: take rest | 14:40 |
| *** zenoway has quit IRC | 14:40 | |
| *** zenoway has joined #openstack-nova | 14:40 | |
| sfinucan | jaypipes: Aye, take it easy in that case | 14:44 |
| jangutter | Massive apologies again for the insane lateness of "Spec to enable Netronome SmartNIC support" https://review.openstack.org/#/c/456386/ | 14:45 |
| *** prateek has quit IRC | 14:45 | |
| jangutter | At least I should be online for the next couple of hours if anyone is still willing to review it. | 14:46 |
| *** iceyao has joined #openstack-nova | 14:46 | |
| *** alexpilotti has quit IRC | 14:49 | |
| *** alexpilotti has joined #openstack-nova | 14:50 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Fix AZ related API docs https://review.openstack.org/456655 | 14:50 |
| *** iceyao has quit IRC | 14:50 | |
| bauzas | mriedem: johnthetubaguy: attempt to clarify our API docs wrt: AZs https://review.openstack.org/456655 | 14:50 |
| *** sridharg has joined #openstack-nova | 14:50 | |
| * bauzas disappears for a couple of minutes | 14:50 | |
| *** catintheroof has quit IRC | 14:52 | |
| *** catintheroof has joined #openstack-nova | 14:52 | |
| *** aarefiev is now known as aarefiev_afk | 14:54 | |
| mriedem | live migration job is spiking at 75%+ fail, i'm going to make it non-voting | 14:55 |
| kashyap | mriedem: Any specific bug (or a set of bugs) that's causing it? | 14:56 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1682453 | 14:56 |
| openstack | Launchpad bug 1682453 in OpenStack Compute (nova) "Live Migration failure: operation failed: Failed to connect to remote libvirt URI - End of file while reading data: sh: 1: nc: not found: Input/output error" [High,Confirmed] | 14:56 |
| mriedem | libvirt craps it's pants | 14:56 |
| *** tbachman has quit IRC | 14:56 | |
| mriedem | with newer libvirt/qemu | 14:56 |
| mriedem | which we were told by some (cough cough) it would fix all of our problems :) | 14:57 |
| *** mtreinish has quit IRC | 14:57 | |
| *** armax has joined #openstack-nova | 14:57 | |
| *** swebster has quit IRC | 14:58 | |
| *** mtreinish has joined #openstack-nova | 14:58 | |
| * kashyap hangs his head in shame | 14:58 | |
| kashyap | mriedem: I never make blanket promises, certainly to _you_ that it will "fix all the problems" | 14:58 |
| mriedem | https://review.openstack.org/456656 | 14:59 |
| *** eharney has quit IRC | 14:59 | |
| kashyap | s/to/not to/ | 14:59 |
| mriedem | kashyap: i'm just teasing | 14:59 |
| kashyap | I know, I forgot to add a smiley | 14:59 |
| *** yingjun has joined #openstack-nova | 15:01 | |
| *** voelzmo has quit IRC | 15:01 | |
| *** marst has quit IRC | 15:02 | |
| *** marst has joined #openstack-nova | 15:02 | |
| *** mdrabe has quit IRC | 15:03 | |
| *** khappone has joined #openstack-nova | 15:04 | |
| *** obre has joined #openstack-nova | 15:05 | |
| *** annegentle has joined #openstack-nova | 15:05 | |
| *** jowisz_ has joined #openstack-nova | 15:06 | |
| *** belliott_ has joined #openstack-nova | 15:06 | |
| *** lifeless_ has joined #openstack-nova | 15:07 | |
| owalsh | kashyap: that looks like a packaging issue, nc not installed | 15:07 |
| *** mdnadeem has quit IRC | 15:07 | |
| *** toanster has joined #openstack-nova | 15:08 | |
| *** ericyoung has joined #openstack-nova | 15:08 | |
| jangutter | :O "nc not installed" conjures the same horror in me as "gcc not installed". | 15:08 |
| *** mdrabe has joined #openstack-nova | 15:08 | |
| *** siva_kri- has joined #openstack-nova | 15:08 | |
| *** marst has quit IRC | 15:09 | |
| *** afazekas_ has joined #openstack-nova | 15:09 | |
| *** dutsmoc has joined #openstack-nova | 15:09 | |
| *** melwitt_ has joined #openstack-nova | 15:09 | |
| *** alaski_ has joined #openstack-nova | 15:09 | |
| *** lbragstad_ has joined #openstack-nova | 15:09 | |
| *** leifz- has joined #openstack-nova | 15:09 | |
| *** dimtruck_ has joined #openstack-nova | 15:09 | |
| *** esmiurium_ has joined #openstack-nova | 15:09 | |
| *** zigo_ has joined #openstack-nova | 15:09 | |
| *** spotz_ has joined #openstack-nova | 15:09 | |
| *** EmilienM_ has joined #openstack-nova | 15:09 | |
| *** pcaruana has quit IRC | 15:10 | |
| *** kencjohnston_ has joined #openstack-nova | 15:10 | |
| *** kashyap` has joined #openstack-nova | 15:10 | |
| sdague | mriedem: the fix is merged now, right? | 15:10 |
| *** mordred1 has joined #openstack-nova | 15:10 | |
| *** stvnoyes has quit IRC | 15:10 | |
| *** alaski has quit IRC | 15:10 | |
| *** toan has quit IRC | 15:10 | |
| *** lbragstad has quit IRC | 15:10 | |
| *** melwitt has quit IRC | 15:10 | |
| *** McNinja has quit IRC | 15:10 | |
| *** khappone_ has quit IRC | 15:10 | |
| *** siva_krishnan has quit IRC | 15:10 | |
| *** mordred has quit IRC | 15:10 | |
| *** snikitin_ has quit IRC | 15:10 | |
| *** TheJulia has quit IRC | 15:10 | |
| *** mariojv has quit IRC | 15:10 | |
| *** belliott has quit IRC | 15:10 | |
| *** kashyap has quit IRC | 15:10 | |
| *** njohnston has quit IRC | 15:10 | |
| *** obre_ has quit IRC | 15:10 | |
| *** mhenkel_ has quit IRC | 15:10 | |
| *** kencjohnston has quit IRC | 15:10 | |
| *** VAhl has quit IRC | 15:10 | |
| *** lifeless has quit IRC | 15:10 | |
| *** jgriffith has quit IRC | 15:10 | |
| *** esmiurium has quit IRC | 15:10 | |
| *** stevemar has quit IRC | 15:10 | |
| *** JayF has quit IRC | 15:10 | |
| *** EmilienM has quit IRC | 15:10 | |
| *** leifz has quit IRC | 15:10 | |
| *** comstud has quit IRC | 15:10 | |
| *** dimtruck has quit IRC | 15:10 | |
| *** afazekas has quit IRC | 15:10 | |
| *** spotz has quit IRC | 15:10 | |
| *** phealy has quit IRC | 15:10 | |
| *** serverascode has quit IRC | 15:10 | |
| *** zigo has quit IRC | 15:10 | |
| *** jowisz has quit IRC | 15:10 | |
| *** dimtruck_ is now known as dimtruck | 15:10 | |
| mriedem | clarks? | 15:10 |
| *** leifz- is now known as leifz | 15:10 | |
| mriedem | that merged last night | 15:10 |
| mriedem | this is something else | 15:10 |
| *** njohnston has joined #openstack-nova | 15:11 | |
| *** eharney has joined #openstack-nova | 15:11 | |
| *** kashyap` is now known as kashyap | 15:11 | |
| sdague | mriedem: https://review.openstack.org/#/c/456388/ | 15:11 |
| *** mariojv_ has joined #openstack-nova | 15:11 | |
| *** EmilienM_ is now known as 17WAAOC7S | 15:11 | |
| *** mhenkel has joined #openstack-nova | 15:11 | |
| sdague | no, it didn't merge until this morning | 15:11 |
| *** annegentle has quit IRC | 15:11 | |
| *** eharney has quit IRC | 15:11 | |
| *** eharney has joined #openstack-nova | 15:11 | |
| *** McNinja has joined #openstack-nova | 15:11 | |
| *** 21WAAA2JF has joined #openstack-nova | 15:11 | |
| *** VAhl has joined #openstack-nova | 15:11 | |
| *** JayF has joined #openstack-nova | 15:11 | |
| sdague | also, it seems totally weird to me that libvirt would shell out to nc | 15:11 |
| *** stevemar has joined #openstack-nova | 15:11 | |
| *** snikitin_ has joined #openstack-nova | 15:12 | |
| *** stvnoyes has joined #openstack-nova | 15:12 | |
| *** zigo_ is now known as zigo | 15:12 | |
| dansmith | sdague: it has definitely used nc for things | 15:12 |
| *** burgerk has joined #openstack-nova | 15:12 | |
| *** zigo is now known as Guest52586 | 15:12 | |
| dansmith | sdague: used to be you couldn't connect a fedora virt-manager to an ubuntu host without installing netcat, or something like that | 15:13 |
| dansmith | the reverse maybe, I forget | 15:13 |
| *** hshiina has joined #openstack-nova | 15:13 | |
| *** rcernin has quit IRC | 15:13 | |
| *** lbragstad_ is now known as lbragstad | 15:14 | |
| *** TheJulia has joined #openstack-nova | 15:18 | |
| *** annegentle has joined #openstack-nova | 15:19 | |
| sdague | so I think this is probably the problematic commit for us - https://libvirt.org/git/?p=libvirt.git;a=commitdiff;h=22eaee8e011645162e4b00d313bf2abdd4eb54e5 | 15:19 |
| *** litao has quit IRC | 15:19 | |
| sdague | probably good to get an ubuntu packaging bug openned up on that | 15:19 |
| *** lpetrut has quit IRC | 15:20 | |
| sdague | clarkb: did you do that yet? | 15:20 |
| *** serverascode has joined #openstack-nova | 15:20 | |
| *** Drankis has quit IRC | 15:21 | |
| owalsh | sdague: that's for qemu+libssh, the qemu+ssh driver always requried netcat | 15:21 |
| *** mlakat has quit IRC | 15:21 | |
| *** vladikr has joined #openstack-nova | 15:24 | |
| *** rajinir has quit IRC | 15:24 | |
| sfinucan | owalsh: Regarding https://review.openstack.org/456571, are you saying we need user and port options even if we don't deprecate 'live_migration_uri' | 15:25 |
| *** rajinir has joined #openstack-nova | 15:25 | |
| * sfinucan would have figured you could specify them as part of that option | 15:25 | |
| *** Jack_Iv has joined #openstack-nova | 15:26 | |
| sdague | it looks like the libvirt-daemon recommends netcat-openbsd, but does not require it | 15:27 |
| *** catintheroof has quit IRC | 15:27 | |
| sdague | Recommends: qemu-kvm | qemu (>= 0.9.1), libxml2-utils, netcat-openbsd | 15:27 |
| *** catintheroof has joined #openstack-nova | 15:27 | |
| owalsh | sfinucan: need to use live_migration_inbound_addr to set the IP network for live migration. We can't do this with live_migration_uri AFAIK | 15:27 |
| *** catintheroof has quit IRC | 15:27 | |
| owalsh | sfinucan: I assume that's one of the main reasons for deprecating live_migration_uri | 15:29 |
| *** nkorabli has quit IRC | 15:30 | |
| *** bnemec has quit IRC | 15:30 | |
| owalsh | sfinucan: but to use SSH we need to pass user, maybe port too, and almost certainly extraoptions, and there is no way to set this when using live_migration_inbound_addr | 15:30 |
| *** bnemec has joined #openstack-nova | 15:30 | |
| *** Jack_Iv has quit IRC | 15:31 | |
| *** XueFeng has quit IRC | 15:32 | |
| *** dutsmoc has quit IRC | 15:34 | |
| *** comstud has joined #openstack-nova | 15:34 | |
| *** Jack_Iv has joined #openstack-nova | 15:35 | |
| *** Oku_OS is now known as Oku_OS-away | 15:35 | |
| *** Jack_Iv has quit IRC | 15:36 | |
| *** Jack_Iv has joined #openstack-nova | 15:36 | |
| *** psachin has quit IRC | 15:37 | |
| sfinucan | owalsh: So for 'live_migration_uri', you could conceivably set it to something like 'xen+ssh://rjones@%s/?no_verify=1'. That has both a user and extra_options | 15:37 |
| sfinucan | owalsh: Do we want the same functionality for whatever's using live_migration_inbound_addr? | 15:37 |
| *** chyka has joined #openstack-nova | 15:39 | |
| *** chyka has quit IRC | 15:39 | |
| owalsh | sfinucan: yes, currently do set the user via live_migration_uri, but we can't control which network it users. %s is always replaced with the remote hostname | 15:39 |
| *** chyka has joined #openstack-nova | 15:41 | |
| *** JayF has quit IRC | 15:41 | |
| sfinucan | owalsh: ...and live_migration_inbound_addr provides an alternative to that, but it doesn't allow you to a username or extra_specs currently, right? | 15:42 |
| * sfinucan is clearly confused | 15:42 | |
| *** stevelle has left #openstack-nova | 15:43 | |
| sfinucan | owalsh: If we allowed you to _not_ specify '%s' for 'live_migration_uri' and instead set a static destination/IP network, would that achieve what you want? | 15:43 |
| cdent | edleafe: are your resource tracker related functional tests sort of on hold? | 15:44 |
| owalsh | sfinucan: you have to specify %s - it's replaced with the hostname of the live miragation target | 15:44 |
| sfinucan | owalsh: That's what I'm saying. What if I removed that requirement | 15:45 |
| *** JayF has joined #openstack-nova | 15:45 | |
| sfinucan | Or are the two complimentary somehow? | 15:45 |
| *** mordred1 is now known as mordred | 15:46 | |
| *** damien_r has quit IRC | 15:46 | |
| owalsh | so live_migration_uri = foo://bar@%s/ will tranlate to foo://bar@nova1/ when migrating to nova1, and foo:/bar@nova2/ when migrating to nova2 | 15:46 |
| *** swebster has joined #openstack-nova | 15:47 | |
| *** markus_z has quit IRC | 15:47 | |
| owalsh | when using the new approach when migrating to nova1 we get live_migration_inbound addr etc.. for nova1 to build the url to migrate to it | 15:48 |
| *** amotoki has quit IRC | 15:48 | |
| owalsh | IIRC anyway | 15:48 |
| * owalsh is probably confused now too | 15:48 | |
| owalsh | it been a while since I looked at it but I think there was consensus that the new options are required | 15:49 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | 15:49 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | 15:49 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | 15:49 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | 15:49 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | 15:49 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | 15:49 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | 15:49 |
| *** liverpooler has quit IRC | 15:56 | |
| edleafe | cdent: yeah, more because I got distracted with other stuff while waiting for more feedback | 15:57 |
| cdent | edleafe: yeah, just checking. I had a reminder to look at them but didn't have the current brain to say much in this round. | 15:58 |
| edleafe | cdent: ok, remind me next week if you think of it | 15:58 |
| cdent | ✔ | 15:58 |
| *** liverpooler has joined #openstack-nova | 15:59 | |
| mriedem | dansmith: before you change things in https://review.openstack.org/#/c/456227/ i've left some comments | 15:59 |
| mriedem | looks like some were at the same time you were replying to sfinucan | 15:59 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix hypervisors api missing HostMappingNotFound handlers https://review.openstack.org/456227 | 15:59 |
| dansmith | hah | 15:59 |
| mriedem | doh | 15:59 |
| mriedem | gdi | 15:59 |
| dansmith | was literally just pushing when you said that | 15:59 |
| *** yonglihe has quit IRC | 15:59 | |
| *** artom has quit IRC | 16:00 | |
| *** artom has joined #openstack-nova | 16:00 | |
| *** tbachman has joined #openstack-nova | 16:01 | |
| *** abhishek has quit IRC | 16:02 | |
| *** ngupta has joined #openstack-nova | 16:02 | |
| ildikov | mriedem: johnthetubaguy: are you avaiable for the Cinder-Nova meeting? | 16:02 |
| mriedem | yeah | 16:02 |
| *** artom has quit IRC | 16:06 | |
| *** artom has joined #openstack-nova | 16:06 | |
| *** zenoway has quit IRC | 16:07 | |
| *** gszasz has quit IRC | 16:10 | |
| *** udesale has quit IRC | 16:15 | |
| *** vladikr has quit IRC | 16:17 | |
| openstackgerrit | Daniel Pawlik proposed openstack/nova master: Changed network_info variable in test_create_configdrive https://review.openstack.org/436953 | 16:18 |
| *** felipemonteiro has quit IRC | 16:20 | |
| *** dtp has joined #openstack-nova | 16:20 | |
| dtp | good morning. can we get a ruling on https://blueprints.launchpad.net/nova/+spec/conf-network-metadata-always-static-ip, please? | 16:21 |
| *** crushil has quit IRC | 16:21 | |
| kashyap | mriedem: That live mig job failure is due to missing `nc` (Netcat), which seems to be resolved by clarkb's -- https://review.openstack.org/#/c/456388/ ("Install netcat for libvirt live migration") | 16:23 |
| *** thorst has quit IRC | 16:23 | |
| kashyap | That merged about 3 hours ago | 16:23 |
| mriedem | kashyap: ok, i had thought that merged last night but forgot about the devstack dependency | 16:23 |
| mriedem | i'll check logstash in a bit to see if it's dropping off | 16:24 |
| kashyap | mriedem: See! You seem to blame libvirt at the drop of a hat :P | 16:24 |
| *** satyar has quit IRC | 16:24 | |
| kashyap | You actually didn't | 16:24 |
| ngupta | Hi everyone, new to this channel. I want to know where does OpenStack stand with supporting encryption for Block Storage using the RBD driver. | 16:25 |
| *** thorst has joined #openstack-nova | 16:25 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/455960 | 16:27 |
| *** openstack has joined #openstack-nova | 16:34 | |
| mriedem | dansmith: is it just me, or is it weird that we don't terminate volume connections on the host when we shelve offload? | 16:34 |
| mriedem | since when you unshelve, you can end up on another host | 16:34 |
| *** openstackstatus has joined #openstack-nova | 16:34 | |
| *** ChanServ sets mode: +v openstackstatus | 16:34 | |
| *** tesseract has joined #openstack-nova | 16:35 | |
| *** erlon has quit IRC | 16:35 | |
| *** yingjun has joined #openstack-nova | 16:35 | |
| *** catintheroof has joined #openstack-nova | 16:36 | |
| *** scottda has joined #openstack-nova | 16:36 | |
| *** jvgrant has joined #openstack-nova | 16:36 | |
| dansmith | mriedem: yeah.. I assume that'd be a bug | 16:36 |
| *** armax_ has joined #openstack-nova | 16:36 | |
| mriedem | like, the most latent bug ever... | 16:36 |
| *** jvgrant has left #openstack-nova | 16:36 | |
| dansmith | I think there might've been some things that were left in place on the off chance it landed on the same host, but can't imagine that's right | 16:37 |
| mriedem | i keep thinking i'm missing something obvious | 16:37 |
| dansmith | unless we call to the old compute somehow, but we've lost that info I think | 16:37 |
| dansmith | yeah, seems too obviously broken | 16:37 |
| *** nic has joined #openstack-nova | 16:37 | |
| mriedem | conductor manager for unshelve calls out to scheduler to get new hosts | 16:37 |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support VGPU https://review.openstack.org/450122 | 16:37 |
| mriedem | plus, you could have unshelved from a host that is now down or gone | 16:38 |
| mriedem | *shelved | 16:38 |
| dansmith | yeah | 16:38 |
| *** armax has quit IRC | 16:39 | |
| *** armax_ is now known as armax | 16:39 | |
| *** lucasagomes is now known as lucas-afk | 16:41 | |
| mriedem | oh btw, pike-1 milestone today | 16:41 |
| mriedem | i had forgotten about the need for the tag | 16:41 |
| *** haplo37 has joined #openstack-nova | 16:42 | |
| *** Apoorva has joined #openstack-nova | 16:42 | |
| *** Apoorva has quit IRC | 16:42 | |
| jianghuaw_ | jaypipes, sbauza: could you help to look at this VGPU spec? | 16:42 |
| jianghuaw_ | https://review.openstack.org/450122 | 16:42 |
| melwitt_ | ngupta: I found this old spec about it https://review.openstack.org/#/c/239798/ and the consensus there mentions the way forward for rbd encrypted volumes should be with qemu native rbd encryption, but I don't think anyone worked on it yet | 16:42 |
| *** Apoorva has joined #openstack-nova | 16:42 | |
| *** melwitt_ is now known as melwitt | 16:43 | |
| mriedem | dansmith: also looks like we don't unbind ports from the host on shelve offload | 16:43 |
| mriedem | this is all just very odd | 16:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP! conf: Add three new '[libvirt] live_migration_*' options https://review.openstack.org/456571 | 16:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Gather 'live_migration_scheme', 'live_migration_inbound_addr' https://review.openstack.org/456572 | 16:44 |
| melwitt | mriedem: yeah, that's "intentional"? I noticed it just leaves everything there as-is, network connection, volume connection, etc | 16:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Convert 'live_migration_inbound_addr' to HostAddressOpt https://review.openstack.org/456573 | 16:44 |
| sfinucan | owalsh: I've to go to tag, so that's my WIP patches. Lemme know if you think that's the right direction. Will finish them tonight or Tuesday | 16:44 |
| *** catintheroof has quit IRC | 16:44 | |
| *** crushil has joined #openstack-nova | 16:44 | |
| *** jaosorior has quit IRC | 16:44 | |
| *** tbachman_ has joined #openstack-nova | 16:45 | |
| melwitt | it assumes it will go back on the same host, but I don't know if it does or can do anything to guarantee that. with the network part, I could see it wanting to keep the IP allocated so that it can't be given out (and network possibly run out of IPs) when the person comes back to unshelve. same concept as keeping the quota occupied during offload | 16:45 |
| *** tbachman has quit IRC | 16:46 | |
| *** tbachman_ is now known as tbachman | 16:46 | |
| *** bpoulos has quit IRC | 16:46 | |
| owalsh | sfinucan: cool, thanks | 16:46 |
| dansmith | mriedem: must be missing something | 16:47 |
| *** Swami_ has quit IRC | 16:48 | |
| mriedem | i honestly hope i am missing something | 16:50 |
| johnthetubaguy | melwitt: oh, interesting, once its offloaded I kinda assumed it wasn't going back there, in all likelylood | 16:52 |
| *** Swami_ has joined #openstack-nova | 16:53 | |
| *** kfarr has quit IRC | 16:53 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support VGPU https://review.openstack.org/450122 | 16:53 |
| *** jaosorior has joined #openstack-nova | 16:54 | |
| dansmith | melwitt: yeah, once offloaded it's not supposed to (have to) go back to the same host | 16:57 |
| dansmith | that's the whole point | 16:57 |
| melwitt | yeah | 16:57 |
| melwitt | I do see it cleaning up network here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L4378 | 16:57 |
| melwitt | so maybe someone just forgot about volumes? | 16:57 |
| *** priteau_ has quit IRC | 16:58 | |
| openstackgerrit | Rajath Agasthya proposed openstack/nova master: Fix Bandit jinja2 warnings https://review.openstack.org/456709 | 16:59 |
| mriedem | melwitt: that's a noop for neutron | 16:59 |
| melwitt | I would not have expected that | 16:59 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L2332 | 16:59 |
| mriedem | melwitt: there are like 5 different methods in the base network api like that | 17:00 |
| *** baoli has quit IRC | 17:00 | |
| *** vladikr has joined #openstack-nova | 17:00 | |
| mriedem | melwitt: and of course some are only neutron and some are only nova-net | 17:00 |
| melwitt | guh, what the hell | 17:00 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/network/api.py#L529 | 17:00 |
| *** baoli has joined #openstack-nova | 17:01 | |
| melwitt | sigh | 17:01 |
| *** ralonsoh has quit IRC | 17:02 | |
| *** jgriffith has joined #openstack-nova | 17:03 | |
| *** jaosorior has quit IRC | 17:03 | |
| *** 21WAAA2JF is now known as EmilienM | 17:05 | |
| *** EmilienM has joined #openstack-nova | 17:05 | |
| *** ekuris has quit IRC | 17:06 | |
| *** gyee has joined #openstack-nova | 17:06 | |
| *** liverpooler has quit IRC | 17:07 | |
| *** ngupta has quit IRC | 17:09 | |
| *** efoley__ has quit IRC | 17:10 | |
| swebster | ^ Interesting that the neutron cleanup doesn't start a migration. This actually causes you to be completely screwed when unshelving with PCI devices. I fixed a symptom of that with https://review.openstack.org/#/c/453938 | 17:10 |
| *** ngupta has joined #openstack-nova | 17:11 | |
| *** hshiina has quit IRC | 17:11 | |
| *** jgriffith has quit IRC | 17:13 | |
| *** chenying has quit IRC | 17:13 | |
| mriedem | hmm, yeah, so we don't unbind and we don't terminate volume connections on shelve offload, | 17:14 |
| mriedem | so unshelve is just.... | 17:14 |
| mriedem | roll the dice | 17:14 |
| *** ociuhandu has quit IRC | 17:14 | |
| mriedem | plus on unshelve i believe we'll also create a new volume, but would have to check that | 17:14 |
| mriedem | if it's bfv | 17:14 |
| *** chenying has joined #openstack-nova | 17:14 | |
| *** gongysh has quit IRC | 17:15 | |
| *** yamahata has joined #openstack-nova | 17:15 | |
| openstackgerrit | Steven Webster proposed openstack/nova master: Fix mitaka online migration for PCI devices https://review.openstack.org/456397 | 17:17 |
| mriedem | yeah so on unshelve, we prep block devices but pass do_check_attach=False, because the volume is still in-use b/c it's still attached to the instance, by design, | 17:18 |
| mriedem | we make a new connection on the new host, | 17:18 |
| mriedem | and if volume['attach_status'] == "detached": is False, | 17:18 |
| mriedem | so we don't try to attach again | 17:18 |
| melwitt | how can it stay attached if the virt domain was destroyed? | 17:19 |
| *** jianghuaw-m has joined #openstack-nova | 17:20 | |
| mriedem | yeah good point, let me see what driver.destroy() does | 17:20 |
| *** baoli has quit IRC | 17:20 | |
| mriedem | that calls cleanup() which will by default unplug vifs | 17:21 |
| *** baoli has joined #openstack-nova | 17:21 | |
| mriedem | we get the bdms and disconnect the volumes | 17:21 |
| mriedem | we never call terminate_connection in cinder | 17:22 |
| mriedem | but maybe that doesn't matter? | 17:22 |
| mriedem | smcginnis: hemna: what does terminate_connection actually do on the cinder side? | 17:22 |
| *** ekuris has joined #openstack-nova | 17:22 | |
| *** sambetts is now known as sambetts|afk | 17:26 | |
| mriedem | melwitt: also, when we unplug the vifs during cleanup() that should trigger a network-changed event from the neutron side, but since the instance is now shelved offloaded, the server-external-events api can't rpc cast into the compute to refresh it's nw info cache | 17:26 |
| *** yingjun has quit IRC | 17:28 | |
| melwitt | O.o | 17:28 |
| *** jgriffith has joined #openstack-nova | 17:32 | |
| *** ekuris has quit IRC | 17:33 | |
| hemna | mriedem, it's job is to tear down the external connection from the backend to the host | 17:38 |
| hemna | effectively unexport the volume | 17:38 |
| peter-hamilton | jaypipes: i'll be afk for a bit; if you have any questions on cert validation, i'll answer once i get back | 17:40 |
| *** mtanino has quit IRC | 17:40 | |
| *** rcernin has joined #openstack-nova | 17:42 | |
| *** tbachman has quit IRC | 17:43 | |
| *** tbachman has joined #openstack-nova | 17:43 | |
| *** vks1 has quit IRC | 17:46 | |
| *** jianghuaw-m has quit IRC | 17:47 | |
| *** Guest58097 has quit IRC | 17:48 | |
| *** felipemonteiro has joined #openstack-nova | 17:52 | |
| *** gjayavelu has joined #openstack-nova | 17:52 | |
| *** pchavva has joined #openstack-nova | 17:55 | |
| mriedem | hemna: so seems like that would be important when an instance with an attached volume disconnects from it's host and eventually shows up elsewhere (another compute host) yeah? | 17:57 |
| mriedem | or does it depend on the backend? | 17:57 |
| hemna | not sure I'm following | 17:57 |
| mriedem | hemna: are you familiar with the shelve offload operation in nova? | 17:58 |
| hemna | not imtimately | 17:58 |
| mriedem | let me introduce you | 17:58 |
| hemna | :P ok | 17:58 |
| mriedem | basically keep your volume attached in the cinder db, | 17:58 |
| mriedem | but we destroy the guest vm from the compute host | 17:58 |
| mriedem | keep ports and volumes attached in dbs | 17:58 |
| mriedem | when we destroy the guest, we disconnect any volumes | 17:59 |
| mriedem | then the instance is in this shelve-offloaded state, where it doesn't have a host | 17:59 |
| mriedem | (we also take a snapshot of it before offloading it) | 17:59 |
| hemna | ok | 17:59 |
| hemna | so it's like a dormant vm | 17:59 |
| *** lpetrut has joined #openstack-nova | 17:59 | |
| mriedem | when you unshelve the instance, we take the snapshot image and boot the server on a new host | 17:59 |
| *** ociuhandu has joined #openstack-nova | 17:59 | |
| mriedem | sort of yeah | 17:59 |
| mriedem | it might land on the same host as before, but it might not | 17:59 |
| *** baoli has quit IRC | 17:59 | |
| hemna | yah, so I believe terminate_connection should be called then in this case, to ensure that the backend disconnects from the host | 18:00 |
| mriedem | when the instance is created on a new host, we call initialize_connection again | 18:00 |
| dane-fichter | jaypipes: also I can answer most cert validation questions while peter-hamilton is afk | 18:00 |
| hemna | makes sense | 18:00 |
| openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272 | 18:00 |
| *** Jack_Iv has quit IRC | 18:01 | |
| *** baoli has joined #openstack-nova | 18:01 | |
| mriedem | hemna: the funny thing is on unshelve, | 18:01 |
| jgriffith | mriedem hemna there is no terminate_connection, only “attachment_delete” or “update" | 18:02 |
| *** Jack_Iv has joined #openstack-nova | 18:02 | |
| jgriffith | :) | 18:02 |
| hemna | :P | 18:02 |
| mriedem | is i don't think we ever call connect_volume in brick on the new host | 18:02 |
| hemna | ! | 18:02 |
| mriedem | so even though we initialize the connection, we don't connect the volume | 18:02 |
| mriedem | as far as i can tell | 18:02 |
| hemna | oops | 18:02 |
| mriedem | let me check the driver.spawn() method though | 18:02 |
| mriedem | maybe it's done there | 18:02 |
| mriedem | some of this garbage is hidden | 18:02 |
| hemna | well if the volume is showing up there, then it has to get called at some point | 18:03 |
| mriedem | i don't know if it does, this is all via code inspection, | 18:03 |
| mriedem | i don't have a devstack handy to recreate | 18:03 |
| mriedem | yeah i don't see brick called during spawn during unshelve | 18:04 |
| *** Jack_Iv has quit IRC | 18:07 | |
| *** Jack_Iv has joined #openstack-nova | 18:07 | |
| mriedem | ha https://bugs.launchpad.net/nova/+bug/1547142 | 18:08 |
| openstack | Launchpad bug 1547142 in OpenStack Compute (nova) "A shelved_offload VM's volumes are still attached to a host" [Medium,Confirmed] | 18:08 |
| mriedem | the fix on the nova side for this was blocked by some changes on the cinder side | 18:09 |
| mriedem | of course | 18:09 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix hypervisors api missing HostMappingNotFound handlers https://review.openstack.org/456227 | 18:10 |
| hemna | oh missing the remove_export | 18:10 |
| hemna | well It's backend dependent really | 18:12 |
| hemna | terminate_connection will do it for most of the cinder backends | 18:12 |
| hemna | lvm of course uses remove_export to do the real work, and does nothing in terminate_connection | 18:14 |
| mriedem | hemna: that's what the guy on the bug originally asked, can't we at least call terminate connection | 18:14 |
| mriedem | does terminate_connection call remove_export? | 18:15 |
| hemna | you can | 18:15 |
| hemna | but in the case of lvm tgt, terminate_connection does nothing | 18:15 |
| hemna | nope | 18:15 |
| hemna | the volume manager calls both of those | 18:15 |
| mriedem | so what calls remove_export for lvm tgt? detach? | 18:15 |
| hemna | volume manager's remove_export does | 18:16 |
| hemna | and some other edge cases | 18:16 |
| mriedem | yeah but what calls that from the api entry point? | 18:16 |
| hemna | looking | 18:16 |
| mriedem | detach? delete volume? | 18:16 |
| mriedem | ok | 18:16 |
| hemna | I don't see anything in the api that calls it..... | 18:17 |
| *** armax_ has joined #openstack-nova | 18:17 | |
| *** mtanino has joined #openstack-nova | 18:18 | |
| * hemna is confused | 18:18 | |
| *** tbachman has quit IRC | 18:19 | |
| *** liverpooler has joined #openstack-nova | 18:19 | |
| mriedem | btw, i looked up shelved + volumes in nova bugs, | 18:19 |
| mriedem | what a treasure trove of broken | 18:19 |
| hemna | detach_volume calls it | 18:19 |
| mriedem | hemna: ah ok, | 18:20 |
| mriedem | and we don't call detach during shelve offload | 18:20 |
| *** armax has quit IRC | 18:20 | |
| *** armax_ is now known as armax | 18:20 | |
| mriedem | because we want the volume to logically stay attached to the instance | 18:20 |
| hemna | well detach_volume calls the driver's remove_export at least | 18:20 |
| hemna | jgriffith, does the new API account for being able to unexport the volume w/o marking it as unattached ? | 18:21 |
| *** felipemonteiro has quit IRC | 18:21 | |
| hemna | or we'll have to add a new Cinder API entry point for remove export | 18:21 |
| hemna | or...does it make sense that terminate_connection in the volume manager would automatically call remove_export too | 18:22 |
| hemna | ... | 18:22 |
| openstackgerrit | Daniel Pawlik proposed openstack/nova master: Changed network_info variable in test_create_configdrive https://review.openstack.org/436953 | 18:23 |
| hemna | honestly I never understood why there was terminate_connection vs remove_export, and not just a single merged capability | 18:23 |
| mriedem | gmann: i just found https://bugs.launchpad.net/nova/+bug/1554440 and i don't agree that it requires a microversion to fix a regression introduced with 2.20 | 18:23 |
| openstack | Launchpad bug 1554440 in OpenStack Compute (nova) "Show volume attachment exception does not looks good if server is in shelved_offloaded state" [Medium,Confirmed] | 18:23 |
| mriedem | hemna: yeah that would be nicer | 18:23 |
| mriedem | oomichi: jaypipes: why do you think we need a microversion bump to fix this bug? https://review.openstack.org/#/c/290196/ | 18:23 |
| mriedem | oomichi: jaypipes: it's clearly a regression in 2.20 | 18:23 |
| *** lpetrut has quit IRC | 18:26 | |
| *** mlavalle has quit IRC | 18:26 | |
| *** mvk has quit IRC | 18:29 | |
| jangutter | Is it to late to get "Netronome SmartNIC Enablement" spec https://review.openstack.org/#/c/456386/ approved before Pike? | 18:31 |
| artom | jangutter, spec freeze is today, so I think it's unlikely for Pike, yes | 18:33 |
| artom | Unless you whip the spec cores into a flurry of activity ;) | 18:33 |
| mriedem | sean-k-mooney: can you take a look at that netronome smartnic spec? ^ | 18:34 |
| mriedem | sean-k-mooney: that seems right up your alley | 18:34 |
| sean-k-mooney | mriedem: sure i think i was already reviewing some os-vif changes related to this | 18:35 |
| *** ociuhandu has quit IRC | 18:36 | |
| jangutter | Unfortunately it took much, much longer than expected to get the internal clearance for this. I believe Mellanox would also like to get something similar merged, but their dev is OOO at the moment. | 18:37 |
| mriedem | jangutter: fwiw, this is a very nicely written spec | 18:37 |
| mriedem | so congratulations on that | 18:37 |
| jangutter | Hey, I can't take credit for that, my barbarian gibberish was transformed into flowery language by our internal review. | 18:38 |
| artom | "barbarian gibberish" :D | 18:38 |
| artom | I shall endeavour to use that somewhere | 18:38 |
| mriedem | +2 | 18:39 |
| mriedem | jaypipes: you're probably good for https://review.openstack.org/#/c/456386/ - it's pretty straight forward vendor vif type stuff | 18:39 |
| mriedem | jangutter: well i commend your internal team for getting their ducks in a row then | 18:40 |
| jangutter | It's my first time using gerrit... The scary thing is that I feel nearly the same way about it as I felt about first encountering "git". | 18:41 |
| jangutter | "where has this been all my life". | 18:41 |
| mriedem | heh | 18:41 |
| artom | Do we need a new API version for that kind of possible parameter value change? | 18:42 |
| mriedem | artom: api parameter value change where? | 18:43 |
| *** nic has quit IRC | 18:43 | |
| mriedem | that's all on the neutron side isn't it? | 18:43 |
| artom | https://review.openstack.org/#/c/456386/5/specs/pike/approved/netronome-smartnic-enablement.rst@134 | 18:43 |
| artom | I may be reading it completely wrong | 18:43 |
| *** lucasxu has quit IRC | 18:43 | |
| jangutter | Well, there's one new "VNIC" type we're proposing. | 18:44 |
| mriedem | artom: there are no rest api changes to the compute api | 18:44 |
| mriedem | i would have just left that section as None to avoid confusion | 18:44 |
| *** avolkov has quit IRC | 18:44 | |
| mriedem | artom: but otherwise i think of this like https://github.com/openstack/nova/commit/32069c8f397e395134c3b1142c2d852640685fef | 18:44 |
| artom | mriedem, ah, right, you pass that to Neutron when you create the port | 18:44 |
| mriedem | yeah, how neutron handles that i don't know | 18:45 |
| jangutter | Yeah, and that changes to text basically. | 18:45 |
| *** cdent has quit IRC | 18:45 | |
| jangutter | And Neutron has a matching bit of text to pick up the same type of VNIC. | 18:45 |
| jangutter | So, technically, the protocol changes by "new allowed value combinations". Where previously those would error out. | 18:46 |
| jangutter | Meaning an opportunity for extra error handling and nicer exceptions, of course. | 18:47 |
| artom | jangutter, right, but those new allowed value combinations aren't in the Nova REST API, right? | 18:48 |
| jangutter | Oh hang on, I'm confused, Neutron tells Nova, it's the other way around. Blame a week of late nights. | 18:48 |
| mriedem | right, | 18:48 |
| mriedem | you create the port in neutron, | 18:48 |
| artom | User tells Neutron, Neutron tells Nova is what I think the workflow is | 18:48 |
| mriedem | nova gets the vif type from the port and determines how to plug it on the host using os-vif | 18:48 |
| jangutter | OK, so newbie question, where's the REST API defined.... | 18:49 |
| *** fragatina has quit IRC | 18:49 | |
| artom | jangutter, defined? All over the place ;) Documented? https://developer.openstack.org/api-ref/compute/ for Nova | 18:49 |
| artom | https://developer.openstack.org/api-ref/networking/ for Neutron | 18:50 |
| jangutter | Hence "APIImpact" | 18:50 |
| artom | Right, except I don't think your spec actually has one | 18:50 |
| *** ociuhandu has joined #openstack-nova | 18:51 | |
| jangutter | :-O | 18:51 |
| artom | jangutter, you'd be changing the vnic-type here: https://developer.openstack.org/api-ref/networking/v2/index.html?expanded=create-port-detail, right? | 18:53 |
| *** sridharg has quit IRC | 18:53 | |
| *** sridharg has joined #openstack-nova | 18:53 | |
| jangutter | artom: yeah, binding:vnic_type, port binding extended attribute. | 18:54 |
| artom | So that's the Neutron REST API - Nova just consumes that | 18:54 |
| *** esberglu has quit IRC | 18:55 | |
| jangutter | Aaah, so that's where the core API impact rests. | 18:55 |
| *** esberglu has joined #openstack-nova | 18:55 | |
| artom | Yeah, I think that's where you'd add your type | 18:55 |
| *** sridharg has quit IRC | 18:56 | |
| jangutter | OK, so that means we'd need to submit a Neutron API spec change too? | 18:57 |
| mriedem | jangutter: neutron uses launchpad bugs tagged as RFEs | 18:57 |
| mriedem | for smaller things | 18:57 |
| mriedem | i don't think they require a spec for everything | 18:57 |
| mriedem | but i'm not totally familiar with their process | 18:58 |
| mriedem | jangutter: https://docs.openstack.org/developer/neutron/policies/blueprints.html | 18:58 |
| *** gjayavelu has quit IRC | 18:58 | |
| *** gjayavelu has joined #openstack-nova | 18:58 | |
| *** MasterOfBugs has joined #openstack-nova | 18:59 | |
| *** armax_ has joined #openstack-nova | 18:59 | |
| *** esberglu has quit IRC | 18:59 | |
| jangutter | mriedem: OK. Should I add that task to the spec or the blueprint? | 18:59 |
| mriedem | jangutter: not the nova one no | 19:00 |
| *** armax has quit IRC | 19:00 | |
| *** armax_ is now known as armax | 19:00 | |
| mriedem | well, you could just call out a dependency on needing to get that support into neutron, but it's kind of implied | 19:00 |
| jangutter | OK. It's a bit of a chicken/egg situation. Both sides need the support at roughly the same time. | 19:01 |
| artom | jangutter, also, and I'm only superficially familiar with this sort of thing, but do you need a new vnic *and* vif type? | 19:01 |
| *** d9k_ has joined #openstack-nova | 19:01 | |
| jangutter | I think sean-k-mooney pointed out that we don't, just new portbinding types. | 19:02 |
| *** d9k has quit IRC | 19:03 | |
| jangutter | It would mean the OVS vif type is now one of the most overloaded ones in the universe. | 19:03 |
| *** eharney has quit IRC | 19:04 | |
| *** armax has quit IRC | 19:04 | |
| vladikr | what are the chances for anyone looking at the spice spec again? https://review.openstack.org/#/c/442040 ? | 19:06 |
| *** nkorabli has joined #openstack-nova | 19:06 | |
| artom | Heh, while we're doing last-minute spec merged ;) mriedem ^^ | 19:07 |
| artom | *merges | 19:07 |
| artom | To be completely honest, I feel like we should have bugged alex_xu for the API part earlier | 19:07 |
| *** armax has joined #openstack-nova | 19:08 | |
| vladikr | the spec was there for a long while - definitely not a last minute spec | 19:08 |
| *** esberglu has joined #openstack-nova | 19:08 | |
| *** esberglu has quit IRC | 19:08 | |
| *** esberglu has joined #openstack-nova | 19:08 | |
| mriedem | spice-native? | 19:09 |
| mriedem | does anyone use spice? | 19:09 |
| mriedem | spice-uncut | 19:09 |
| mriedem | spice-freerange | 19:09 |
| mriedem | spice-organic | 19:10 |
| artom | The whole of interstellar travel runs on spice;) | 19:10 |
| mriedem | yes i love dune references | 19:10 |
| artom | So clearly we need faster proxies | 19:10 |
| vladikr | mriedem, yes, well, more like 'will', especially with the introduction of the vgpus | 19:11 |
| vladikr | mriedem, it's not just about a faster proxy , but for the ability to connect native clients | 19:12 |
| mriedem | introduction of vgpus? into nova? or in general? | 19:13 |
| artom | mriedem, I think the hope is that eventually we send the vGPU output to the client through SPICE | 19:13 |
| vladikr | mriedem, well, in general and in nova in future :) | 19:13 |
| artom | Ie, full workstation virt | 19:13 |
| vladikr | speaking about vgpu: https://review.openstack.org/#/c/450122/ | 19:16 |
| *** gouthamr has quit IRC | 19:17 | |
| openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | 19:21 |
| *** voelzmo has joined #openstack-nova | 19:22 | |
| *** ociuhandu has quit IRC | 19:22 | |
| *** fragatina has joined #openstack-nova | 19:23 | |
| *** alaski_ is now known as alaski | 19:24 | |
| *** erhudy has joined #openstack-nova | 19:24 | |
| *** coreywright has quit IRC | 19:24 | |
| *** voelzmo has quit IRC | 19:26 | |
| *** cdent has joined #openstack-nova | 19:27 | |
| *** voelzmo has joined #openstack-nova | 19:28 | |
| *** Jack_Iv has quit IRC | 19:28 | |
| *** john5223 has quit IRC | 19:28 | |
| sean-k-mooney | jangutter: actully the ovs-vif type is really just a special case of the generic tap vif type but ya it has a pretty broad meaning | 19:29 |
| *** Jack_Iv has joined #openstack-nova | 19:30 | |
| *** annegentle has joined #openstack-nova | 19:31 | |
| jangutter | In trying to amend the spec, I seem to have plonked in a unicode character somewhere. | 19:31 |
| clarkb | mriedem: is the livemigration stuff happy now with my fixes in? | 19:33 |
| mriedem | clarkb: yeah, i didn't realize that fix merged this morning | 19:33 |
| clarkb | cool just double checking I don't have more to look at now that I am back to a computer | 19:34 |
| *** Jack_Iv has quit IRC | 19:35 | |
| *** Swami_ has quit IRC | 19:36 | |
| openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | 19:36 |
| *** coreywright has joined #openstack-nova | 19:36 | |
| *** ngupta has quit IRC | 19:37 | |
| *** Swami_ has joined #openstack-nova | 19:37 | |
| *** ngupta has joined #openstack-nova | 19:37 | |
| *** xyang1 has joined #openstack-nova | 19:38 | |
| *** Jack_Iv has joined #openstack-nova | 19:40 | |
| *** Swami_ has quit IRC | 19:40 | |
| *** dimtruck is now known as zz_dimtruck | 19:42 | |
| *** dane-fichter has quit IRC | 19:44 | |
| *** Jack_Iv has quit IRC | 19:46 | |
| *** Jack_Iv has joined #openstack-nova | 19:47 | |
| *** mlavalle has joined #openstack-nova | 19:47 | |
| *** pchavva has quit IRC | 19:49 | |
| sean-k-mooney | did gerrit just go down | 19:50 |
| clarkb | yes | 19:50 |
| sean-k-mooney | its back | 19:50 |
| clarkb | the notice is on its way out | 19:50 |
| clarkb | but too many irc channels to do it all at once | 19:51 |
| *** Jack_Iv has quit IRC | 19:51 | |
| sean-k-mooney | clarkb: is the maitenace still ongoing or was that restart the only one to expect | 19:51 |
| -openstackstatus- NOTICE: The Gerrit service on http://review.openstack.org is being restarted to address hung remote replication tasks. | 19:52 | |
| *** coreycb is now known as coreycb_vaca | 19:52 | |
| clarkb | hopefully the only one | 19:52 |
| sean-k-mooney | cool ill continue reviewing then | 19:53 |
| *** armax has quit IRC | 19:54 | |
| *** Jack_Iv has joined #openstack-nova | 19:55 | |
| *** armax has joined #openstack-nova | 19:56 | |
| *** annegentle has quit IRC | 19:57 | |
| *** annegentle has joined #openstack-nova | 19:58 | |
| *** cdent has quit IRC | 19:59 | |
| jangutter | I'm trying to find out why tox is somehow getting stuck on a unicode character in the spec. It's not happening if I run it locally :-( | 20:00 |
| *** liverpooler has quit IRC | 20:00 | |
| *** Jack_Iv has quit IRC | 20:00 | |
| sean-k-mooney | jangutter: which spec? this one https://review.openstack.org/#/c/456386/5/specs/pike/approved/netronome-smartnic-enablement.rst | 20:01 |
| sean-k-mooney | i can test it locally if you like and see if i get the same error | 20:01 |
| clarkb | jangutter: its likely a locale issue. I think you get C.utf-8 in the jenkins runs | 20:01 |
| *** cleong has quit IRC | 20:01 | |
| clarkb | but maybe not | 20:01 |
| *** zz_dimtruck is now known as dimtruck | 20:03 | |
| jangutter | Yeah, if I look at the unicode character it's complaining about it's an "en dash" "\u2013". | 20:03 |
| jangutter | I've been bitten by them before: cutting and pasting from IM apps that believe the world should be a prettier place. | 20:04 |
| *** crushil has quit IRC | 20:04 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix 404 for no bdm.device_name when showing volume attachment details https://review.openstack.org/290196 | 20:05 |
| mriedem | gmann: jaypipes: oomichi: ^ updated this very old change, i don't think requires a microversion | 20:05 |
| clarkb | oh regardless of locale it seems like the docs build expect ascii for some reason | 20:05 |
| mriedem | jaypipes: could you drop the -2 on that? | 20:05 |
| *** baoli_ has joined #openstack-nova | 20:06 | |
| *** baoli has quit IRC | 20:09 | |
| *** adisky_ has quit IRC | 20:09 | |
| *** gouthamr has joined #openstack-nova | 20:10 | |
| jangutter | sean-k-mooney: I've updated it to https://review.openstack.org/#/c/456386/7/specs/pike/approved/netronome-smartnic-enablement.rst | 20:11 |
| jangutter | sean-k-mooney: would you mind checking? tox completes fine on my side... | 20:12 |
| peter-hamilton | jaypipes: do you think you'll be able to get to the cert validation spec today? | 20:12 |
| peter-hamilton | jaypipes: don't want to be pest, obv | 20:12 |
| mriedem | i think he's under the weather | 20:12 |
| peter-hamilton | mriedem: ah, ok | 20:12 |
| peter-hamilton | mriedem: do you mind giving it a look over then? he's been a +2 for the past couple versions | 20:13 |
| mriedem | i've got some other things queued up first | 20:13 |
| peter-hamilton | mriedem: gotcha | 20:15 |
| sean-k-mooney | jangutter: sure im just pulling it down now | 20:16 |
| *** itlinux has joined #openstack-nova | 20:17 | |
| mriedem | me too | 20:17 |
| *** artom has quit IRC | 20:18 | |
| *** artom has joined #openstack-nova | 20:18 | |
| *** artom has quit IRC | 20:20 | |
| sean-k-mooney | jangutter: ya i get the same error as the gate | 20:20 |
| *** voelzmo has quit IRC | 20:20 | |
| *** artom has joined #openstack-nova | 20:20 | |
| mriedem | me too | 20:21 |
| mriedem | after i rebased | 20:21 |
| mriedem | and did a tox -r -e docs | 20:21 |
| *** artom has quit IRC | 20:22 | |
| mriedem | i'm wondering if it has something to do with this https://github.com/openstack/nova-specs/commit/0a4c1bcaeb600229103cc61d574d57cac320ba84 | 20:22 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix hypervisors api missing HostMappingNotFound handlers https://review.openstack.org/456227 | 20:22 |
| clarkb | oh chardet updated | 20:22 |
| clarkb | and I know openstack-manuals ran into a similar problem | 20:22 |
| *** artom has joined #openstack-nova | 20:22 | |
| mriedem | ah yeah | 20:22 |
| mriedem | 3.0.2 | 20:22 |
| *** ngupta has quit IRC | 20:23 | |
| jangutter | Are those the dashes we are looking for? | 20:23 |
| mriedem | *2.3.0 | 20:23 |
| sean-k-mooney | so we have to cap chardet or is there a way to fix it? | 20:23 |
| mriedem | oh wait no 3.0.2 is new, but u-c is using 2.3.0 | 20:24 |
| *** ngupta has joined #openstack-nova | 20:24 | |
| *** crushil has joined #openstack-nova | 20:24 | |
| mriedem | clarkb: i don't see chardet used in this job | 20:24 |
| mriedem | http://logs.openstack.org/86/456386/7/check/gate-nova-specs-docs-ubuntu-xenial/9c8b9dd/tox/venv-3.log | 20:24 |
| jangutter | I've had _fun_ with pbr and tox before. | 20:25 |
| *** artom has quit IRC | 20:26 | |
| *** artom has joined #openstack-nova | 20:26 | |
| mriedem | someone remind me next time i mark my entire nova folder as read in thunderbird, that i need to not set view > threads > threads with unread | 20:27 |
| mriedem | because i got scared | 20:27 |
| sdague | heh | 20:27 |
| *** ngupta has quit IRC | 20:27 | |
| *** ngupta has joined #openstack-nova | 20:28 | |
| sean-k-mooney | mriedem: clarkb downgradeing chardet to 2.3.0 has no effect | 20:28 |
| mriedem | yeah we have chardet 2.3.0 in u-c already | 20:29 |
| mriedem | that's not new | 20:29 |
| mriedem | reverting https://github.com/openstack/nova-specs/commit/0a4c1bcaeb600229103cc61d574d57cac320ba84 makes no difference either | 20:29 |
| mriedem | nova meeting in 30 minutes | 20:30 |
| sean-k-mooney | mriedem: looking at https://github.com/openstack/nova-specs/blob/master/tox.ini i dont think we are using the upper constraints in nova-specs | 20:31 |
| mriedem | oh that's right | 20:31 |
| mriedem | and i can't remember why | 20:31 |
| sean-k-mooney | well it has no "code" so i kindof makes sense that it would not be needed | 20:31 |
| melwitt | mriedem: for adding "upgrade impact" to the spec template, should I do it only for the queens template or for pike and queens? | 20:32 |
| *** zhenguo has quit IRC | 20:32 | |
| mriedem | melwitt: quenes | 20:32 |
| mriedem | *queens | 20:32 |
| melwitt | thanks | 20:32 |
| clarkb | yasfb made a release today | 20:33 |
| clarkb | checking if a downgrade fixes | 20:34 |
| jangutter | There is definitely a lot of non-ascii characters in the specs. | 20:34 |
| mriedem | me too | 20:34 |
| *** smatzek has quit IRC | 20:34 | |
| mriedem | clarkb: yeah that works | 20:36 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Cap yasfb at 0.6.0 https://review.openstack.org/456754 | 20:37 |
| *** ediardo has joined #openstack-nova | 20:37 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Cap yasfb at 0.6.0 https://review.openstack.org/456754 | 20:38 |
| sean-k-mooney | mriedem: do we need to cap that in upper constraits too | 20:38 |
| *** ngupta_ has joined #openstack-nova | 20:38 | |
| mriedem | sean-k-mooney: we don't use it from u-c | 20:38 |
| mriedem | yasfb isn't in u-c | 20:38 |
| mriedem | you know, i think the last time i figured out that nova-specs doesn't use u-c, it was because of yasfb | 20:39 |
| sean-k-mooney | mriedem: yes but i assume this will hit others out side of nova? | 20:39 |
| sean-k-mooney | haha | 20:39 |
| mriedem | https://github.com/openstack/nova-specs/commit/8c0a70ec54c3e4c764fdf34cd0db59fa9c6cd095 | 20:39 |
| mriedem | i don't think we even need to use it | 20:39 |
| sean-k-mooney | is there any reason that we cant make the specs utf-8 out of interest | 20:39 |
| clarkb | sounds like because yasfb | 20:40 |
| mriedem | i'm bugging doug | 20:40 |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Add a new section: "Upgrade impact" to the template https://review.openstack.org/456756 | 20:40 |
| mriedem | dougbug | 20:40 |
| sean-k-mooney | didnt stephen just disable building all sphinx target except html | 20:40 |
| mriedem | yeah | 20:41 |
| sean-k-mooney | are we still building the rss xml output? | 20:41 |
| mriedem | i guess? https://review.openstack.org/#/c/456754/ | 20:41 |
| mriedem | oops | 20:41 |
| mriedem | http://specs.openstack.org | 20:41 |
| *** ngupta has quit IRC | 20:41 | |
| itlinux | hello all, has anyone adopted a way to automatically register a name of the VM to a DNS server without using Deseginete? | 20:41 |
| sean-k-mooney | but presumable that is done as a post merge job. it could just be installed in the job. | 20:42 |
| *** artom has quit IRC | 20:42 | |
| *** ngupta_ has quit IRC | 20:43 | |
| *** artom has joined #openstack-nova | 20:43 | |
| itlinux | should we have cloud-init do this.. | 20:43 |
| mriedem | ok fast approved https://review.openstack.org/#/c/456754/ | 20:43 |
| *** artom has quit IRC | 20:44 | |
| *** artom has joined #openstack-nova | 20:44 | |
| mriedem | +2 on handling volume extend events from cinder https://review.openstack.org/#/c/453272/ spec if someone else can +W | 20:47 |
| *** edmondsw has quit IRC | 20:47 | |
| *** ijw has joined #openstack-nova | 20:47 | |
| *** vladikr has quit IRC | 20:47 | |
| *** edmondsw has joined #openstack-nova | 20:48 | |
| *** itlinux has quit IRC | 20:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | 20:50 |
| mriedem | jangutter: you should be fixed now | 20:50 |
| mriedem | just need to address sean-k-mooney's comments | 20:50 |
| jangutter | Thanks. | 20:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Add a new section: "Upgrade impact" to the template https://review.openstack.org/456756 | 20:50 |
| sean-k-mooney | mriedem: jangutter my only real feedback is on the logation fo the os-vif code. if its not upstream in the main ovs repo the pluging changes need to be in an external plugin | 20:51 |
| jangutter | sean-k-mooney: I understand that the VIF types would likely need to be forked because we're not upstream (yet). | 20:52 |
| *** nkorabli has quit IRC | 20:52 | |
| *** itlinux has joined #openstack-nova | 20:52 | |
| sean-k-mooney | jangutter: well the vif object definition should go in os-vif even for external plugins | 20:52 |
| *** nkorabli has joined #openstack-nova | 20:52 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Amend spec for placement project/user association https://review.openstack.org/454352 | 20:52 |
| sean-k-mooney | jangutter: that is to ensure we all have a common set of datamodels | 20:53 |
| jangutter | sean-k-mooney: yep, agreed, but it would still be acceptable? | 20:53 |
| *** edmondsw has quit IRC | 20:53 | |
| sean-k-mooney | createing the port profiles in os-vif yes | 20:53 |
| sean-k-mooney | jangutter: the changes to vif_plug_ovs in https://review.openstack.org/#/c/456483/ should be done in an external os-vif plugin | 20:54 |
| *** kukacz has quit IRC | 20:54 | |
| jangutter | forgive me if I'm operating a bit on the wonky side, I claim a severe lack of sleep and the wrong timezone... | 20:54 |
| *** takashin has joined #openstack-nova | 20:55 | |
| sean-k-mooney | the changes to the os_vif objects should got to os-vif | 20:56 |
| jangutter | OK, so an extra task would be: create a new external plugin (replicating what the OVS plugin does). | 20:56 |
| sean-k-mooney | jangutter: yes but only replicating the part you need | 20:56 |
| sean-k-mooney | os-vif was structured so that you can clone it and the cp one of the plugin sub directos and use that as the base for a new plugin. | 20:57 |
| jangutter | Yep, I noticed that too, it's pretty awesome. | 20:58 |
| sean-k-mooney | jangutter: of example i dont think you need to support iptables/hybrid plug in you case correct | 20:58 |
| jangutter | Correct: that part of the kernel is short-circuited. | 20:58 |
| jangutter | We do support the ovs conntrac rules on the chip though. | 20:59 |
| sean-k-mooney | so the plugin itself should be failly small | 20:59 |
| sean-k-mooney | cool | 20:59 |
| jangutter | Thanks, amending spec. | 20:59 |
| *** burt1 has quit IRC | 21:00 | |
| sean-k-mooney | contrack should not effect the plug dirver code though. if you want me to review the plugin or help you with it let me know | 21:00 |
| mriedem | nova meeting itme | 21:00 |
| mriedem | *time | 21:00 |
| mriedem | in #openstack-meeting | 21:00 |
| *** thorst has quit IRC | 21:02 | |
| *** cdent has joined #openstack-nova | 21:04 | |
| *** armax has quit IRC | 21:06 | |
| *** armax has joined #openstack-nova | 21:07 | |
| *** sdague has quit IRC | 21:11 | |
| *** crushil has quit IRC | 21:14 | |
| openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | 21:15 |
| *** lpetrut has joined #openstack-nova | 21:20 | |
| sean-k-mooney | mriedem: that was a short and sweet meeting | 21:23 |
| jangutter | sean-k-mooney: yep, nova is blithely oblivious to conntrack. | 21:23 |
| *** cdent has quit IRC | 21:25 | |
| bauzas | mriedem: I guess https://review.openstack.org/#/c/448253/ is very latent for Ubuntu given they still provide an old libvirt ? | 21:28 |
| openstackgerrit | Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavor API https://review.openstack.org/449288 | 21:29 |
| openstackgerrit | Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavor API https://review.openstack.org/449288 | 21:30 |
| sean-k-mooney | jangutter: minor nit in latest version of https://review.openstack.org/#/c/456386/9 but otherwise this looks good to me | 21:31 |
| jangutter | The main reason I left that in there was to coordinate with the Mellanox guys. | 21:32 |
| bauzas | mriedem: yeah confirmed, xenial LTS is having libvirt-1.3.1 http://packages.ubuntu.com/xenial/libvirt-bin | 21:32 |
| mriedem | bauzas: yes, outside of UCA | 21:32 |
| mriedem | bauzas: we regressed ubuntu with a fix for centos 7.3 | 21:32 |
| mriedem | that assumed a newer libvirt | 21:32 |
| mriedem | which was ok for centos, but broke xenial | 21:32 |
| bauzas | okay | 21:32 |
| mriedem | so we straddle the versions now | 21:32 |
| *** ngupta has joined #openstack-nova | 21:32 | |
| jangutter | sean-k-mooney: If we can come up with a common standard, it might get us that much closer to upstream by default into OVS. | 21:33 |
| sean-k-mooney | jangutter: ya the melonox work seams to be getting traction upstream in ovs | 21:33 |
| bauzas | mriedem: so, if you deploy Nova on Xenial without using the UCA packages, then kaboom, correct? | 21:34 |
| *** itlinux has quit IRC | 21:34 | |
| *** armax has quit IRC | 21:34 | |
| sean-k-mooney | if you are able to make that work for ye then all of these changes could go into the openvswitch plugin in os-vif | 21:34 |
| *** armax has joined #openstack-nova | 21:35 | |
| jangutter | sean-k-mooney: yep, but the TC stuff is kernel hackery, and the bar for merging that is VERY high. | 21:35 |
| sean-k-mooney | ya i know we have looked at doing something similar with dpdk in the past | 21:36 |
| sean-k-mooney | we canceled those nics though so that got pushed back | 21:36 |
| jangutter | sean-k-mooney: As have we. The funny thing is that there doesn't seem to be something at this stage that does everything well. | 21:37 |
| jangutter | *looked at DPDK. | 21:37 |
| sean-k-mooney | i know of at least 3 vendors who have done something in this space now so it would be nice to have a singel api in ovs for this. | 21:38 |
| mriedem | bauzas: and you have one of these vif types | 21:38 |
| mriedem | that set that config | 21:38 |
| jangutter | sean-k-mooney, I definitely owe you your libation of choice. | 21:38 |
| mriedem | bauzas: but yeah, they were picking up the latest newton patch release which regressed them | 21:38 |
| bauzas | okay, anyway, +2d | 21:39 |
| mriedem | cool thanks | 21:39 |
| sean-k-mooney | jangutter: ah no worries im happy to help. | 21:39 |
| dtp | can i get a ruling on https://blueprints.launchpad.net/nova/+spec/conf-network-metadata-always-static-ip, please? | 21:39 |
| dtp | unsure what to do next w/ it | 21:40 |
| *** itlinux has joined #openstack-nova | 21:40 | |
| mriedem | clarkb: mordred: are you interested in https://review.openstack.org/#/c/336293/ ? | 21:40 |
| *** burgerk_ has joined #openstack-nova | 21:40 | |
| bauzas | mriedem: dansmith: I won't have time to correctly review https://review.openstack.org/#/c/456227/ but in case you don't land it tonight, I'll be reviewing it tomorrow morning my time | 21:40 |
| mriedem | bauzas: i'll get it | 21:40 |
| *** lpetrut has quit IRC | 21:40 | |
| mriedem | with melwitt's help | 21:40 |
| bauzas | k | 21:41 |
| * melwitt emerges from cave | 21:41 | |
| clarkb | mriedem: I want dhcp to be required :) | 21:42 |
| clarkb | mriedem: which I think that sort of says it should be (as fallback?) | 21:42 |
| sean-k-mooney | jangutter: strangly enough the reason we first looked at ovs manageing vf was so that we could manage sriov via odl for generic hardware. if you have smart nics that is even more valuable | 21:43 |
| mriedem | dtp: can you answer clarkb's question? | 21:43 |
| mriedem | dtp: clarkb and mordred are your ticket to config drive goodness | 21:44 |
| *** burgerk has quit IRC | 21:44 | |
| jangutter | sean-k-mooney: heh, our very first OpenStack foray was with ODL! | 21:44 |
| *** burgerk_ has quit IRC | 21:44 | |
| sean-k-mooney | my first encouter was deploying havana in the rc phase on centos 6.x with a beta version fo packstack. i obvioly did something bad in a past life | 21:46 |
| *** annegentle has quit IRC | 21:46 | |
| jangutter | sean-k-mooney: wow. Mine was Liberty at least... | 21:47 |
| mriedem | sean-k-mooney: it was punishment for bringing up 50 blueprint ideas at the ptg | 21:47 |
| mriedem | all involving new scheduler filters or extra specs | 21:47 |
| *** tesseract has quit IRC | 21:47 | |
| sean-k-mooney | mriedem: haha, it was worth it then | 21:48 |
| dtp | clarkb i don't follow. do you mean static network configuration should never be used? | 21:48 |
| jangutter | So, https://review.openstack.org/#/c/456386 has been thoroughly reviewed. Thank you so much! | 21:49 |
| clarkb | dtp: it should never be required, I'm fine if you want to opt into it | 21:49 |
| mriedem | jangutter: +2 | 21:51 |
| dtp | clarkb in our network we prefer static config, but if you need dhcp for a first boot or something, we want to support that as well | 21:51 |
| clarkb | dtp: its mostly convenience. Its something that just works everywhere | 21:52 |
| clarkb | and doesn't require me to change/learn how every distro does network configuration | 21:52 |
| sean-k-mooney | jangutter: out of interest looking at the poc code you have up across all the repos is that a working solution | 21:52 |
| clarkb | but there are some cases (like link aggregation and vlans) where opting in might make sense as a user | 21:52 |
| clarkb | *opting in to needing static config I should say | 21:53 |
| sean-k-mooney | jangutter: it seams pretty small codewise of a failiy big performace bump though most of the really work is done in the nic/ovs | 21:53 |
| jangutter | sean-k-mooney: it's working now on newton, but there's still something hinky that happens during live-migration. | 21:53 |
| dtp | got to agree on that - it is a nice convenience. what change are you suggesting to this code? | 21:53 |
| jangutter | sean-k-mooney: I have yet to track down why the pci address isn't being reallocated on the destination compute node. | 21:54 |
| sean-k-mooney | jangutter: oh i might know why that is | 21:54 |
| clarkb | dtp: I don't think it needs to change. at least my initial reading is this is compatbile with what I have said above | 21:55 |
| clarkb | dtp: dhcp will still work. But nova might ask you to use static | 21:55 |
| sean-k-mooney | jangutter: it has to do with the way nova has to rebind the port in migration | 21:55 |
| sean-k-mooney | jangutter: it generates the libvirt xml for the destination before it calls to neutorn to rebind the host on the new host | 21:55 |
| jangutter | sean-k-mooney: I have some scribblings a madman did here, something about "populate-neutron-binding-profile"... | 21:55 |
| dtp | clarkb yessir. basically, just allowing nova to provide static config even if DHCP is enabled, is what it does. | 21:56 |
| dtp | clients can use whichever they prefer | 21:56 |
| sean-k-mooney | jangutter: as a result it does not update the pci adress before the xml is generated | 21:56 |
| jangutter | sean-k-mooney: yes, exactly what we're seeing. Did that happen during Mitaka? | 21:57 |
| sean-k-mooney | jangutter: yes as part of the port bind nova need to populate the neutron binging profile with the detail of the new vf | 21:57 |
| sean-k-mooney | jangutter: yes it happened in mitaka | 21:57 |
| sean-k-mooney | i think there is a spec to fix it | 21:57 |
| sean-k-mooney | it came up as a topic in the nova neutron cross project session | 21:57 |
| jangutter | wow, you just made my day.... | 21:58 |
| jangutter | (only 2 minutes left!) | 21:58 |
| *** ngupta has quit IRC | 21:58 | |
| sean-k-mooney | well i think there is a spec/bug for this i dont know if its fixed. | 21:58 |
| sean-k-mooney | i belive the resolution involved allowing multiple port bindings during a live migrate | 21:58 |
| mriedem | dtp: i left some comments/questions, | 21:59 |
| mriedem | dtp: i'm green in this area so maybe my questions don't make sense | 21:59 |
| jangutter | sean-k-mooney: would you know some keywords I can google? | 21:59 |
| dtp | mriedem thanks! | 21:59 |
| *** erhudy has quit IRC | 22:00 | |
| mriedem | melwitt: i'm +2 on https://review.openstack.org/#/c/456227/ - we need to get that in for p1 | 22:00 |
| mriedem | but no pressure | 22:01 |
| sean-k-mooney | jangutter: i belive its https://review.openstack.org/#/c/309416/ | 22:01 |
| *** ijw has quit IRC | 22:01 | |
| *** tjones has joined #openstack-nova | 22:01 | |
| *** mdrabe has quit IRC | 22:01 | |
| melwitt | mriedem: hah. looking at it now | 22:01 |
| *** ijw has joined #openstack-nova | 22:01 | |
| jangutter | sean-k-mooney: thanks! | 22:02 |
| jangutter | would anyone else have cycles to do the final review on https://review.openstack.org/#/c/456386 ? | 22:03 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Blacklist yasfb 0.6.0 https://review.openstack.org/456769 | 22:06 |
| sean-k-mooney | jaypipes: perhaps you could take a skim over jangutter path https://review.openstack.org/#/c/456386 if you have time. form an os-vif point of view it makes sense to me. | 22:06 |
| *** felipemonteiro has joined #openstack-nova | 22:08 | |
| *** thorst has joined #openstack-nova | 22:10 | |
| melwitt | mriedem: I think I'm missing how HostMappingNotFound is raised from things that don't appear to raise it | 22:12 |
| mriedem | melwitt: the decorator | 22:12 |
| mriedem | melwitt: which one specifically? | 22:12 |
| melwitt | mriedem: like service_get_by_compute_host, but I see the decorator now. duh | 22:12 |
| mriedem | ok | 22:13 |
| melwitt | meaning duh to me, I wasn't seeing where the lookup was coming from | 22:13 |
| mriedem | melwitt: i took that as a personal 'duh' to me | 22:13 |
| mriedem | i was highly offended | 22:13 |
| melwitt | I knew it! | 22:14 |
| *** dimtruck is now known as zz_dimtruck | 22:15 | |
| *** nic has joined #openstack-nova | 22:17 | |
| *** jamielennox|away is now known as jamielennox | 22:19 | |
| melwitt | mriedem: I don't see how it can be raised from the compute_node_get here https://review.openstack.org/#/c/456227/4/nova/api/openstack/compute/hypervisors.py@192 and then things could fail at L195 not raising the webob exception | 22:19 |
| *** thorst has quit IRC | 22:20 | |
| *** felipemonteiro has quit IRC | 22:21 | |
| *** armax has quit IRC | 22:21 | |
| *** xyang1 has quit IRC | 22:23 | |
| mriedem | melwitt: hmm yeah i don't see HostMappingNotFound raised from there either | 22:26 |
| melwitt | not harmful for the except block, but harmful not to have the try-except around the other call that can raise it | 22:27 |
| mriedem | well, it's not good to be handling exceptions that won't happen either | 22:27 |
| mriedem | melwitt: want to -1? | 22:27 |
| melwitt | yeah, I will. I'm just looking through the tests now | 22:28 |
| melwitt | yes, agreed not good | 22:28 |
| *** nkorabli has quit IRC | 22:28 | |
| *** Jack_Iv has joined #openstack-nova | 22:29 | |
| *** nkorabli has joined #openstack-nova | 22:29 | |
| *** Apoorva_ has joined #openstack-nova | 22:30 | |
| *** itlinux has quit IRC | 22:31 | |
| *** annegentle has joined #openstack-nova | 22:33 | |
| *** nkorabli has quit IRC | 22:33 | |
| *** annegentle has quit IRC | 22:33 | |
| *** Jack_Iv has quit IRC | 22:34 | |
| *** Apoorva has quit IRC | 22:34 | |
| *** lyan has quit IRC | 22:36 | |
| *** lyan has joined #openstack-nova | 22:38 | |
| *** zz_dimtruck is now known as dimtruck | 22:43 | |
| *** thorst has joined #openstack-nova | 22:43 | |
| *** thorst has quit IRC | 22:44 | |
| *** jangutter has quit IRC | 22:46 | |
| *** tjones1 has joined #openstack-nova | 22:47 | |
| *** esberglu has quit IRC | 22:47 | |
| *** esberglu has joined #openstack-nova | 22:48 | |
| mriedem | dansmith: i don't want to pester but are we missing anything on https://review.openstack.org/#/c/456227/ ? | 22:50 |
| *** tjones has quit IRC | 22:50 | |
| dansmith | gdi you people | 22:50 |
| *** tjones1 has quit IRC | 22:51 | |
| dansmith | yeah I think I might've mixed up compute_get and service_get | 22:51 |
| dansmith | ffs | 22:51 |
| *** artom has quit IRC | 22:52 | |
| dansmith | I think I even explained it right in one of the earlier sets and then did the wrong thing | 22:52 |
| melwitt | got too excited | 22:52 |
| *** esberglu has quit IRC | 22:52 | |
| *** artom has joined #openstack-nova | 22:52 | |
| *** artom has quit IRC | 22:54 | |
| mriedem | it would be nice if we could just put a try/except around the entire os-hypervisors/os-services/os-hosts APIs | 22:54 |
| *** artom has joined #openstack-nova | 22:54 | |
| mriedem | heh, you know, we actually could | 22:54 |
| *** hamzy has quit IRC | 22:55 | |
| melwitt | now that would be sweet | 22:55 |
| *** john51 has quit IRC | 22:56 | |
| mriedem | well we do have a catch all place in the wsgi layer | 22:56 |
| mriedem | this i think https://github.com/openstack/nova/blob/master/nova/api/openstack/wsgi.py#L396 | 22:57 |
| mriedem | so, if we wanted to be lazy, we could just handle HostMappingNotFound there and return httpnotfound | 22:58 |
| mriedem | melwitt: dansmith: ^ | 22:58 |
| mriedem | kind of a cop out though | 22:58 |
| dansmith | you'd rather backport that change? :) | 22:58 |
| mriedem | this is going to ocata? | 22:58 |
| mriedem | https://review.openstack.org/#/c/442162/ was pike | 22:59 |
| dansmith | oh yeaq | 22:59 |
| dansmith | I was thinking the p-1 | 22:59 |
| mriedem | ok, | 22:59 |
| dansmith | you'd rather do that? | 22:59 |
| dansmith | I think I have this changed, fwiw | 22:59 |
| mriedem | i thought red hat backported multi-cell support for rhosp ocata :) | 22:59 |
| *** john51 has joined #openstack-nova | 22:59 | |
| mriedem | how about we do the safe thing now | 22:59 |
| mriedem | and look at the lazy way later | 22:59 |
| claudiub|2 | mriedem: hellou. just peeking around here for a bit. this blueprint was approved in ocata, and had the implementation up for review, but it didn't merge in ocata: https://blueprints.launchpad.net/nova/+spec/hyper-v-nested-virtualization | 22:59 |
| melwitt | it seems like doing that would be weird. though it is doing like Forbidden and Invalid that way but not NotFound. dunno the history on that | 23:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix hypervisors api missing HostMappingNotFound handlers https://review.openstack.org/456227 | 23:00 |
| claudiub|2 | can it get re-approved? :D | 23:00 |
| mriedem | claudiub|2: for you? anything. | 23:00 |
| claudiub|2 | <3 | 23:00 |
| claudiub|2 | seems like an early easter. :) | 23:00 |
| mriedem | melwitt: i think it's basically just a catch all for global types of failures | 23:00 |
| mriedem | which hostmappingnotfound kind of is | 23:00 |
| *** MasterOfBugs has quit IRC | 23:00 | |
| dansmith | mriedem: yeah that one is pretty easy to catch and always do the right thing | 23:00 |
| mriedem | claudiub|2: jesus hasn't come up and seen his shadow yet so don't count your chickens | 23:01 |
| dansmith | it's fairly specific about what happened and what should be done | 23:01 |
| melwitt | yeah. I was just noting it for whatever reason intentionally left out the base class for NotFound | 23:01 |
| mriedem | this easter could be different | 23:01 |
| dansmith | mriedem: could be super confusing for someone trying to track down the 404 though | 23:01 |
| mriedem | dansmith: yeah i totally agree | 23:01 |
| mriedem | dansmith: i never knew that existed until about a year ago when laski pointed it out | 23:01 |
| mriedem | ^ is generally my story for everything | 23:01 |
| claudiub|2 | well, it is a bit cloud-y lately | 23:01 |
| melwitt | laski, that rascal | 23:01 |
| *** lyan has quit IRC | 23:02 | |
| claudiub|2 | also, i've submitted the sr-iov bp: https://blueprints.launchpad.net/nova/+spec/hyperv-sriov no implementation yet i'm afraid, apparently there's a whole guide to follow just to configure the nics properly. | 23:03 |
| mriedem | claudiub|2: sounds about right | 23:04 |
| mriedem | claudiub|2: do you get a long weekend? | 23:04 |
| claudiub|2 | well, technically i'm free on monday, i'll still be around | 23:05 |
| claudiub|2 | i don't like vacations too much. :D | 23:05 |
| mriedem | you can read the sriov manual | 23:05 |
| claudiub|2 | yeah, that is in my plan. :D | 23:05 |
| claudiub|2 | and question: can we live-migrate vms with sr-iov? | 23:06 |
| mriedem | claudiub|2: no | 23:06 |
| mriedem | resource tracker issue | 23:06 |
| claudiub|2 | hm, interesting | 23:06 |
| mriedem | *issues | 23:06 |
| mriedem | cfriesen knows about it | 23:06 |
| claudiub|2 | is that a nova limitation, or a hypervisor limitation? | 23:06 |
| * alaski heard his name | 23:09 | |
| claudiub|2 | so, alaski is like beetlejuice - say his name too many times and he shows up. :D | 23:09 |
| alaski | I'm glad I appear to have explained things to folks in the past, because I don't remember much these days | 23:10 |
| *** thorst has joined #openstack-nova | 23:15 | |
| *** awaugama has quit IRC | 23:15 | |
| melwitt | you missed out on some cerebral shelve feature talk today | 23:15 |
| alaski | I hope it was good. I am very much not proud of introducing that feature. | 23:20 |
| *** gouthamr has quit IRC | 23:24 | |
| melwitt | hm ... define good. mriedem found that upon shelve offload, volumes are not disconnected and then also when network is cleaned up, that only does something for nova-net through the network_api, the neutron implementation is a no-op pass | 23:25 |
| *** thorst has quit IRC | 23:26 | |
| melwitt | so it pretty much relies on the unshelve putting the instance back on the same host it was shelved offloaded from, which seems really unlikely | 23:27 |
| alaski | yeah, I would not expect that to happen much in practice | 23:29 |
| gmann | mriedem: thank i almost forgot about that bug, ll check patch | 23:33 |
| *** chyka has quit IRC | 23:33 | |
| *** chyka has joined #openstack-nova | 23:33 | |
| *** gouthamr has joined #openstack-nova | 23:35 | |
| cfriesen | claudiub|2: the first issue is that we need to get resource tracking in general working over live migration. (see https://review.openstack.org/#/c/244489 and the follow-on patches) | 23:37 |
| *** mlavalle has quit IRC | 23:38 | |
| *** chyka has quit IRC | 23:38 | |
| *** nic has quit IRC | 23:39 | |
| cfriesen | do people actually use shelve? If so, why hasn't anyone noticed this? | 23:39 |
| alaski | maybe not using volumes, or neutron? | 23:40 |
| alaski | it did seem to have a small following, despite discouragement about using it | 23:40 |
| *** thorst has joined #openstack-nova | 23:41 | |
| melwitt | yeah, I would guess similar. I have heard of it being used and it would work fine for nova-net and no volumes | 23:42 |
| *** ngupta has joined #openstack-nova | 23:50 | |
| mriedem | cfriesen: someone has, long time ago | 23:53 |
| mriedem | the stratoscale guys | 23:53 |
| mriedem | from the bug in the ML | 23:53 |
| *** edmondsw has joined #openstack-nova | 23:54 | |
| *** ijw has quit IRC | 23:55 | |
| *** itlinux has joined #openstack-nova | 23:57 | |
| *** edmondsw has quit IRC | 23:59 | |
| *** takashin has quit IRC | 23:59 | |
| *** ijw has joined #openstack-nova | 23:59 | |
| *** nkorabli has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!