| *** gcb has quit IRC | 00:03 | |
| *** awaugama has joined #openstack-nova | 00:07 | |
| *** artom has quit IRC | 00:10 | |
| *** artom has joined #openstack-nova | 00:11 | |
| *** artom has quit IRC | 00:12 | |
| *** mvk has quit IRC | 00:12 | |
| *** artom has joined #openstack-nova | 00:12 | |
| openstackgerrit | Dan Peschman proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025 | 00:13 |
|---|---|---|
| *** crushil has quit IRC | 00:17 | |
| *** catintheroof has quit IRC | 00:19 | |
| *** amotoki has joined #openstack-nova | 00:20 | |
| *** MasterOfBugs has quit IRC | 00:22 | |
| *** pramodrj07 has quit IRC | 00:23 | |
| *** mtanino_ has joined #openstack-nova | 00:23 | |
| *** mvk has joined #openstack-nova | 00:25 | |
| *** mtanino has quit IRC | 00:25 | |
| *** iceyao has joined #openstack-nova | 00:26 | |
| *** cdelatte has joined #openstack-nova | 00:30 | |
| *** cdelatte has quit IRC | 00:30 | |
| *** mvk has quit IRC | 00:34 | |
| *** mvk has joined #openstack-nova | 00:34 | |
| *** hongbin has quit IRC | 00:36 | |
| *** phuongnh has joined #openstack-nova | 00:42 | |
| *** yamahata has joined #openstack-nova | 00:43 | |
| *** dtp has quit IRC | 00:44 | |
| *** crushil has joined #openstack-nova | 00:47 | |
| *** baoli has joined #openstack-nova | 00:48 | |
| *** baoli has quit IRC | 00:49 | |
| *** ssurana has quit IRC | 00:50 | |
| *** baoli has joined #openstack-nova | 00:50 | |
| *** mvk has quit IRC | 00:51 | |
| *** kaisers has joined #openstack-nova | 00:52 | |
| *** zhurong has joined #openstack-nova | 00:54 | |
| *** kaisers has quit IRC | 00:57 | |
| *** mdrabe has quit IRC | 00:57 | |
| *** Apoorva_ has joined #openstack-nova | 01:02 | |
| *** cNilesh has joined #openstack-nova | 01:02 | |
| *** cNilesh has quit IRC | 01:03 | |
| *** cNilesh has joined #openstack-nova | 01:03 | |
| *** mvk has joined #openstack-nova | 01:03 | |
| *** gyee has quit IRC | 01:04 | |
| *** Apoorva has quit IRC | 01:06 | |
| *** Apoorva_ has quit IRC | 01:06 | |
| *** mvk has quit IRC | 01:10 | |
| *** mvk has joined #openstack-nova | 01:10 | |
| *** gcb has joined #openstack-nova | 01:16 | |
| *** tovin07_ has joined #openstack-nova | 01:20 | |
| *** thorst has quit IRC | 01:30 | |
| *** thorst has joined #openstack-nova | 01:30 | |
| *** Sukhdev has quit IRC | 01:33 | |
| *** thorst has quit IRC | 01:34 | |
| *** NikhilS has joined #openstack-nova | 01:35 | |
| *** edmondsw has joined #openstack-nova | 01:39 | |
| *** edmondsw has quit IRC | 01:43 | |
| *** awaugama has quit IRC | 01:54 | |
| *** dixiaoli has joined #openstack-nova | 01:54 | |
| *** dixiaoli has quit IRC | 01:58 | |
| *** esberglu has joined #openstack-nova | 01:58 | |
| *** nic has quit IRC | 02:00 | |
| *** dixiaoli has joined #openstack-nova | 02:02 | |
| *** esberglu has quit IRC | 02:03 | |
| *** yonglihe has joined #openstack-nova | 02:03 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs master: Deprecate multinic proxy API https://review.openstack.org/384261 | 02:06 |
| alex_xu | mriedem: ^ updated | 02:06 |
| *** nic has joined #openstack-nova | 02:09 | |
| *** mtanino_ has quit IRC | 02:11 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200 | 02:12 |
| *** nic has quit IRC | 02:12 | |
| *** tuanluong has joined #openstack-nova | 02:15 | |
| *** catintheroof has joined #openstack-nova | 02:16 | |
| *** kevinz has joined #openstack-nova | 02:26 | |
| *** hongbin has joined #openstack-nova | 02:26 | |
| *** thorst has joined #openstack-nova | 02:31 | |
| *** fragatina has joined #openstack-nova | 02:34 | |
| *** catintheroof has quit IRC | 02:34 | |
| *** diga has joined #openstack-nova | 02:34 | |
| *** catintheroof has joined #openstack-nova | 02:35 | |
| *** fragatin_ has quit IRC | 02:37 | |
| *** iceyao has quit IRC | 02:37 | |
| *** fragatina has quit IRC | 02:38 | |
| *** catintheroof has quit IRC | 02:40 | |
| openstackgerrit | Spencer Yu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value https://review.openstack.org/447766 | 02:40 |
| *** fragatina has joined #openstack-nova | 02:41 | |
| *** fragatina has quit IRC | 02:42 | |
| *** diga has quit IRC | 02:48 | |
| *** diga has joined #openstack-nova | 02:49 | |
| *** thorst has quit IRC | 02:51 | |
| *** iceyao has joined #openstack-nova | 02:55 | |
| *** Shunli has joined #openstack-nova | 02:55 | |
| *** iceyao has quit IRC | 02:59 | |
| *** armax has joined #openstack-nova | 03:00 | |
| *** dixiaoli has quit IRC | 03:00 | |
| *** dixiaoli has joined #openstack-nova | 03:01 | |
| *** iceyao has joined #openstack-nova | 03:03 | |
| *** diga has quit IRC | 03:03 | |
| *** diga has joined #openstack-nova | 03:10 | |
| *** nicolasbock has quit IRC | 03:10 | |
| *** chyka has quit IRC | 03:14 | |
| mriedem | alex_xu: thanks, a couple of small things were missed | 03:18 |
| mriedem | but overall should be easy to fix and then i'm +2 | 03:18 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix api-ref for create servers response https://review.openstack.org/451673 | 03:20 |
| mriedem | tommylikehu: re https://review.openstack.org/#/c/453272/3/specs/pike/approved/nova-support-attached-volume-extend.rst@26 nova does not yet support volume multiattach upstream | 03:25 |
| mriedem | so a VM is correct there | 03:25 |
| *** sridharg has joined #openstack-nova | 03:26 | |
| *** amotoki has quit IRC | 03:26 | |
| tommylikehu | mriedem: thanks | 03:27 |
| *** crushil has quit IRC | 03:32 | |
| *** dimtruck is now known as zz_dimtruck | 03:38 | |
| *** tbachman has quit IRC | 03:38 | |
| openstackgerrit | Bob Ball proposed openstack/nova master: XenAPI: device tagging https://review.openstack.org/333781 | 03:39 |
| *** amotoki has joined #openstack-nova | 03:40 | |
| mriedem | tommylikehu: mgagne: comments inline https://review.openstack.org/#/c/453272/3 | 03:41 |
| *** esberglu has joined #openstack-nova | 03:47 | |
| *** iceyao has quit IRC | 03:49 | |
| *** amotoki has quit IRC | 03:50 | |
| *** gouthamr has quit IRC | 03:51 | |
| *** esberglu has quit IRC | 03:52 | |
| *** kaisers has joined #openstack-nova | 03:54 | |
| tommylikehu | mriedem: great, thanks! | 03:55 |
| *** mriedem has quit IRC | 03:56 | |
| *** amotoki has joined #openstack-nova | 03:58 | |
| *** diga has quit IRC | 03:58 | |
| *** kaisers has quit IRC | 03:58 | |
| *** vks1 has joined #openstack-nova | 04:00 | |
| *** kaisers has joined #openstack-nova | 04:00 | |
| *** kaisers has quit IRC | 04:03 | |
| *** abhi89 has joined #openstack-nova | 04:03 | |
| *** kaisers has joined #openstack-nova | 04:04 | |
| *** chyka has joined #openstack-nova | 04:06 | |
| *** Dinesh_Bhor has quit IRC | 04:07 | |
| *** fragatina has joined #openstack-nova | 04:09 | |
| *** yamamoto_ has quit IRC | 04:09 | |
| *** kaisers has quit IRC | 04:09 | |
| *** markvoelker has quit IRC | 04:09 | |
| *** yamamoto has joined #openstack-nova | 04:09 | |
| *** chyka has quit IRC | 04:11 | |
| *** hongbin has quit IRC | 04:12 | |
| *** awaugama has joined #openstack-nova | 04:12 | |
| *** fragatina has quit IRC | 04:13 | |
| *** iceyao has joined #openstack-nova | 04:13 | |
| *** amotoki has quit IRC | 04:13 | |
| *** Sukhdev has joined #openstack-nova | 04:14 | |
| *** baoli has quit IRC | 04:14 | |
| *** trinaths has joined #openstack-nova | 04:16 | |
| *** yamahata has quit IRC | 04:17 | |
| *** dixiaoli has quit IRC | 04:17 | |
| *** dixiaoli has joined #openstack-nova | 04:19 | |
| *** dixiaoli has quit IRC | 04:23 | |
| *** dharinic has quit IRC | 04:27 | |
| *** vks1 has quit IRC | 04:28 | |
| *** amotoki has joined #openstack-nova | 04:29 | |
| *** psachin has joined #openstack-nova | 04:31 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272 | 04:35 |
| *** fragatina has joined #openstack-nova | 04:37 | |
| *** diga has joined #openstack-nova | 04:37 | |
| *** fragatina has quit IRC | 04:37 | |
| *** fragatina has joined #openstack-nova | 04:38 | |
| *** vks1 has joined #openstack-nova | 04:42 | |
| *** diga has quit IRC | 04:44 | |
| *** ayogi has joined #openstack-nova | 04:46 | |
| *** thorst has joined #openstack-nova | 04:48 | |
| *** thorst has quit IRC | 04:52 | |
| *** iceyao has quit IRC | 04:57 | |
| *** awaugama has quit IRC | 05:02 | |
| *** kaisers has joined #openstack-nova | 05:05 | |
| *** kaisers has quit IRC | 05:09 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272 | 05:10 |
| *** Dinesh_Bhor has joined #openstack-nova | 05:11 | |
| *** Sukhdev has quit IRC | 05:12 | |
| *** iceyao has joined #openstack-nova | 05:15 | |
| *** rcernin has joined #openstack-nova | 05:15 | |
| *** Jack_Iv has joined #openstack-nova | 05:16 | |
| *** iceyao has quit IRC | 05:20 | |
| *** udesale has joined #openstack-nova | 05:20 | |
| *** prateek has joined #openstack-nova | 05:20 | |
| *** Jack_Iv has quit IRC | 05:21 | |
| *** dixiaoli has joined #openstack-nova | 05:26 | |
| *** mdnadeem has joined #openstack-nova | 05:26 | |
| *** prateek has quit IRC | 05:29 | |
| *** prateek has joined #openstack-nova | 05:29 | |
| *** abhi89 has quit IRC | 05:29 | |
| *** abhi89 has joined #openstack-nova | 05:30 | |
| *** tuanluong has quit IRC | 05:33 | |
| *** psachin has quit IRC | 05:34 | |
| *** esberglu has joined #openstack-nova | 05:37 | |
| *** esberglu has quit IRC | 05:41 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272 | 05:42 |
| *** adisky_ has joined #openstack-nova | 05:44 | |
| *** fragatina has quit IRC | 05:44 | |
| *** spotz is now known as spotz_zzz | 05:46 | |
| *** thorst has joined #openstack-nova | 05:48 | |
| *** spotz_zzz is now known as spotz | 05:48 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs master: Deprecate multinic proxy API https://review.openstack.org/384261 | 05:51 |
| *** sridharg has quit IRC | 05:51 | |
| *** Jack_Iv has joined #openstack-nova | 05:52 | |
| *** thorst has quit IRC | 05:53 | |
| *** jaosorior_away is now known as jaosorior | 05:55 | |
| *** tuanluong has joined #openstack-nova | 05:58 | |
| *** chyka has joined #openstack-nova | 06:01 | |
| *** iceyao has joined #openstack-nova | 06:01 | |
| *** markus_z has joined #openstack-nova | 06:04 | |
| *** chyka has quit IRC | 06:05 | |
| *** kaisers has joined #openstack-nova | 06:06 | |
| openstackgerrit | Lei Zhang proposed openstack/nova master: Add sync traits command for placement https://review.openstack.org/450125 | 06:07 |
| *** vks1 has quit IRC | 06:08 | |
| *** nirendra has joined #openstack-nova | 06:12 | |
| *** nirendra has left #openstack-nova | 06:13 | |
| *** mkoderer_ has joined #openstack-nova | 06:13 | |
| *** Oku_OS-away is now known as Oku_OS | 06:13 | |
| *** nirendra has joined #openstack-nova | 06:14 | |
| *** mkoderer_ has quit IRC | 06:15 | |
| *** satyar has joined #openstack-nova | 06:15 | |
| nirendra | Hi, Nova placement API checks for max_units which is local_gb for the host. This makes allocation_ratio useless. So if I'm trying to create a VM with 80GB root disk and disk_allocation_ratio as 100, it fails for a host which does not have 80GB local storage | 06:16 |
| nirendra | Is this a known bug? I couldn't find one in Launchpad | 06:16 |
| *** andreas_s has joined #openstack-nova | 06:18 | |
| *** iceyao has quit IRC | 06:23 | |
| *** Jack_Iv has quit IRC | 06:24 | |
| *** Jack_Iv has joined #openstack-nova | 06:24 | |
| *** voelzmo has joined #openstack-nova | 06:26 | |
| *** vks1 has joined #openstack-nova | 06:27 | |
| *** Jack_Iv has quit IRC | 06:29 | |
| *** avolkov has joined #openstack-nova | 06:30 | |
| *** psachin has joined #openstack-nova | 06:31 | |
| *** Jack_Iv has joined #openstack-nova | 06:32 | |
| *** ltomasbo|away is now known as ltomasbo | 06:41 | |
| *** nirendra has quit IRC | 06:41 | |
| *** trinaths has quit IRC | 06:41 | |
| *** tesseract has joined #openstack-nova | 06:47 | |
| *** iceyao has joined #openstack-nova | 06:47 | |
| *** thorst has joined #openstack-nova | 06:49 | |
| *** mgagne has quit IRC | 06:52 | |
| *** iceyao has quit IRC | 06:52 | |
| *** Jack_Iv has quit IRC | 06:53 | |
| *** Jack_Iv has joined #openstack-nova | 06:54 | |
| *** thorst has quit IRC | 06:54 | |
| *** Jack_Iv has quit IRC | 06:55 | |
| *** Jack_Iv has joined #openstack-nova | 06:56 | |
| *** Jack_Iv has quit IRC | 06:56 | |
| *** Jack_Iv has joined #openstack-nova | 06:56 | |
| *** chyka has joined #openstack-nova | 07:01 | |
| *** sridharg has joined #openstack-nova | 07:05 | |
| *** chyka has quit IRC | 07:06 | |
| *** pcaruana has joined #openstack-nova | 07:06 | |
| *** garyk1 has quit IRC | 07:10 | |
| openstackgerrit | YuYang proposed openstack/nova master: VMware:fix get VM root device conditional verify https://review.openstack.org/454441 | 07:16 |
| *** karimb has joined #openstack-nova | 07:17 | |
| *** xinliang has quit IRC | 07:19 | |
| *** Jack_Iv has quit IRC | 07:19 | |
| *** Jack_Iv_ has joined #openstack-nova | 07:20 | |
| *** Jack_Iv_ has quit IRC | 07:20 | |
| openstackgerrit | YuYang proposed openstack/nova master: VMware:fix get VM root device conditional verify https://review.openstack.org/454441 | 07:20 |
| *** Jack_Iv has joined #openstack-nova | 07:20 | |
| *** Jack_Iv has quit IRC | 07:21 | |
| *** Jack_Iv has joined #openstack-nova | 07:21 | |
| *** chenying has joined #openstack-nova | 07:22 | |
| *** garyk has joined #openstack-nova | 07:23 | |
| *** hieulq has quit IRC | 07:27 | |
| *** garyk has quit IRC | 07:27 | |
| *** hieulq has joined #openstack-nova | 07:27 | |
| *** hieulq has quit IRC | 07:29 | |
| *** hieulq has joined #openstack-nova | 07:29 | |
| *** hieulq has quit IRC | 07:29 | |
| *** garyk has joined #openstack-nova | 07:29 | |
| *** hieulq has joined #openstack-nova | 07:29 | |
| *** hieulq is now known as Guest33220 | 07:30 | |
| *** hieulq_ has joined #openstack-nova | 07:30 | |
| *** trinaths has joined #openstack-nova | 07:30 | |
| *** garyk has quit IRC | 07:30 | |
| *** hieulq_ has quit IRC | 07:30 | |
| *** hferenc has quit IRC | 07:31 | |
| *** xinliang has joined #openstack-nova | 07:32 | |
| *** garyk has joined #openstack-nova | 07:33 | |
| *** hferenc has joined #openstack-nova | 07:33 | |
| *** garyk has quit IRC | 07:33 | |
| *** Jack_Iv has quit IRC | 07:36 | |
| *** Jack_Iv has joined #openstack-nova | 07:37 | |
| *** karimb has quit IRC | 07:39 | |
| *** hieulq_ has joined #openstack-nova | 07:41 | |
| *** aarefiev_afk is now known as aarefiev | 07:41 | |
| *** Jack_Iv has quit IRC | 07:41 | |
| *** mlakat has quit IRC | 07:46 | |
| *** rpodolyaka_afk is now known as rpodolyaka | 07:47 | |
| *** alexpilotti has quit IRC | 07:47 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (1/4) https://review.openstack.org/455563 | 07:49 |
| *** thorst has joined #openstack-nova | 07:50 | |
| *** mlakat has joined #openstack-nova | 07:51 | |
| *** frickler has quit IRC | 07:51 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Fix HTTP 500 raised for getColsolelog for stopped instance https://review.openstack.org/455183 | 07:52 |
| *** frickler has joined #openstack-nova | 07:52 | |
| *** derekh has joined #openstack-nova | 07:54 | |
| *** Jack_Iv has joined #openstack-nova | 07:54 | |
| *** hieulq_ has quit IRC | 07:54 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (2/4) https://review.openstack.org/455568 | 07:55 |
| *** hieulq_ has joined #openstack-nova | 07:56 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (3/4) https://review.openstack.org/455570 | 08:00 |
| *** hieulq_ has quit IRC | 08:01 | |
| *** chyka has joined #openstack-nova | 08:02 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (4/4) https://review.openstack.org/455573 | 08:04 |
| *** lucas-afk is now known as lucasagomes | 08:06 | |
| *** chyka has quit IRC | 08:07 | |
| *** takashin has left #openstack-nova | 08:09 | |
| *** thorst has quit IRC | 08:09 | |
| *** karimb has joined #openstack-nova | 08:11 | |
| *** efoley_ has joined #openstack-nova | 08:18 | |
| *** ralonsoh has joined #openstack-nova | 08:20 | |
| *** efoley__ has joined #openstack-nova | 08:21 | |
| *** Kevin_Zheng has quit IRC | 08:23 | |
| *** efoley_ has quit IRC | 08:24 | |
| *** priteau has joined #openstack-nova | 08:27 | |
| *** haplo37_ has quit IRC | 08:30 | |
| *** gszasz has joined #openstack-nova | 08:38 | |
| *** haplo37 has quit IRC | 08:40 | |
| *** rmart04 has joined #openstack-nova | 08:43 | |
| *** Jack_Iv has quit IRC | 08:46 | |
| *** Jack_Iv has joined #openstack-nova | 08:49 | |
| *** Jack_Iv has quit IRC | 08:49 | |
| *** Jack_Iv has joined #openstack-nova | 08:49 | |
| *** iceyao has joined #openstack-nova | 08:50 | |
| *** kaisers has quit IRC | 08:50 | |
| *** karimb has quit IRC | 08:53 | |
| *** haplo37 has joined #openstack-nova | 08:54 | |
| *** iceyao has quit IRC | 08:54 | |
| *** haplo37_ has joined #openstack-nova | 08:54 | |
| *** voelzmo has quit IRC | 09:00 | |
| *** voelzmo has joined #openstack-nova | 09:02 | |
| *** cdent has joined #openstack-nova | 09:02 | |
| d9k | when instance is destroyed neutron deletes ipsets while nova is modifying iptables and https://ptpb.pw/16S8 happens, acquiring neutron-ipset lock in (network/linux_net.py) IptablesManager._apply seems to fix the problem | 09:03 |
| *** openstackgerrit has quit IRC | 09:03 | |
| *** karimb has joined #openstack-nova | 09:03 | |
| *** lpetrut has joined #openstack-nova | 09:04 | |
| *** ociuhandu has quit IRC | 09:05 | |
| *** thorst has joined #openstack-nova | 09:06 | |
| *** abalutoiu has quit IRC | 09:08 | |
| *** thorst has quit IRC | 09:11 | |
| *** jsheeren has joined #openstack-nova | 09:14 | |
| *** esberglu has joined #openstack-nova | 09:14 | |
| jsheeren | hi, question: is it possible to limit image A to be only able to use flavor X and Y, image B only flavor Z? | 09:15 |
| *** esberglu has quit IRC | 09:19 | |
| *** abhi89 has quit IRC | 09:19 | |
| *** abhi89 has joined #openstack-nova | 09:20 | |
| *** sambetts|afk is now known as sambetts | 09:26 | |
| *** prabhuraj has joined #openstack-nova | 09:27 | |
| prabhuraj | Hi team, Someone could review the following changes : | 09:28 |
| prabhuraj | https://review.openstack.org/#/c/397471/ | 09:28 |
| prabhuraj | https://review.openstack.org/#/c/397459/ | 09:29 |
| *** BobBall has joined #openstack-nova | 09:37 | |
| BobBall | Howdy - Trying to use device tagging I'm getting an "Invalid input for field/attribute networks" - nova cli says I need to use ",tag=<value>" but schema validation fails if I add it. Openstack cli doesn't seem to support tags (According to help text anyway)... | 09:40 |
| BobBall | Can someone remind me how to use tagging? | 09:40 |
| johnthetubaguy | openstack cli doesn't support microversions yet | 09:41 |
| johnthetubaguy | afaik | 09:41 |
| johnthetubaguy | did you try with nova client? | 09:42 |
| BobBall | Yes - that's what failed the schema validation when running it | 09:42 |
| BobBall | http://paste.openstack.org/show/606121/ - I'm clearly using the wrong syntax | 09:44 |
| *** naichuans has joined #openstack-nova | 09:44 | |
| BobBall | (or device tagging is broken...) | 09:45 |
| *** kaisers has joined #openstack-nova | 09:46 | |
| *** belmoreira has joined #openstack-nova | 09:47 | |
| abhi89 | johnthetubaguy: can you please review https://review.openstack.org/#/c/455118/2 | 09:49 |
| *** prabhuraj has quit IRC | 09:49 | |
| *** ishant has joined #openstack-nova | 09:54 | |
| *** tovin07_ has quit IRC | 09:55 | |
| *** ociuhandu has joined #openstack-nova | 09:57 | |
| *** tbachman has joined #openstack-nova | 09:57 | |
| johnthetubaguy | BobBall: that doesn't seem good | 09:58 |
| BobBall | OK - so the nova client does not now support nova microversion 2.42 - for some reason 7.1.0 was installed | 09:58 |
| BobBall | it's just weird that novaclient 7.1.0 lists tag in the help text but doesn't support the microversion which introduced the API | 09:59 |
| johnthetubaguy | BobBall: try do the full debug and see what version is requested | 09:59 |
| johnthetubaguy | yeah, thats... inconsistent | 09:59 |
| johnthetubaguy | probably only got some of the patches landed | 09:59 |
| BobBall | nova client is requesting 2.41 - which doesn't support tagging... *jumps with joy* | 09:59 |
| johnthetubaguy | BobBall: did you try explicitly request the new version, it might be we couldn't raise the maximum at that point | 09:59 |
| BobBall | Yes - python-novaclient says it's not supported. | 10:00 |
| johnthetubaguy | not 100% sure thats a thing | 10:00 |
| johnthetubaguy | ha | 10:00 |
| johnthetubaguy | abhi89: I think this might be the code you need to change for your bug: https://github.com/openstack/nova/blob/3092d2adcb6bab0bf39b01ea0561ec7e6a3990fd/nova/compute/manager.py#L1585 | 10:00 |
| *** karimb has quit IRC | 10:01 | |
| Dinesh_Bhor | sfinucan: Hi, addressed your review comments. Already got one +2 please take a look at it whenever you are free: https://review.openstack.org/#/c/364687/ | 10:01 |
| sfinucan | Dinesh_Bhor: Will do | 10:01 |
| Dinesh_Bhor | sfinucan: thanks | 10:01 |
| *** tuanluong_ has joined #openstack-nova | 10:02 | |
| *** tuanluong has quit IRC | 10:05 | |
| ishant | Hi All, I am creating instance and it always sets availability_zone as None even though default_availability_zone value is 'nova' | 10:05 |
| ishant | beacuse of this port device_owner is always set as compute:None | 10:06 |
| BobBall | Even the latest python-novaclient does not support the required API micro version!!! http://git.openstack.org/cgit/openstack/python-novaclient/tree/novaclient/__init__.py | 10:06 |
| BobBall | In other words, I think no one can use python-novaclient to specify device tagging at all. | 10:06 |
| *** NikhilS has quit IRC | 10:07 | |
| *** karimb has joined #openstack-nova | 10:09 | |
| *** zhurong has quit IRC | 10:10 | |
| *** zhurong has joined #openstack-nova | 10:11 | |
| *** chyka has joined #openstack-nova | 10:14 | |
| sfinucan | ralonsoh: You around. Question on https://review.openstack.org/#/c/444415 for you | 10:15 |
| *** cNilesh has quit IRC | 10:16 | |
| ralonsoh | sfinucan: hi! | 10:16 |
| sfinucan | ralonsoh: How come we're using the 'extra_info' column in that review rather than adding a new capabilities column? | 10:18 |
| sfinucan | I didn't even question it until you said that nothing else was using the column now. If it was important enough to move out the other stuff, why re-add new stufF? | 10:18 |
| ralonsoh | sfinucan: that is the easy way, to add a new field | 10:19 |
| *** chyka has quit IRC | 10:19 | |
| ralonsoh | sfinucan: instead of this, we decided not to modify the DB schema and add this information into extra_info | 10:20 |
| *** zsli_ has joined #openstack-nova | 10:20 | |
| ralonsoh | sfinucan: extra_info is initially a bucket for "other" information, not so relevant | 10:20 |
| sfinucan | ralonsoh: OK. What's the long term strategy then? Drop that entire table in favour of resource providers? | 10:22 |
| *** openstackgerrit has joined #openstack-nova | 10:22 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Add description to policies in hypervisors.py https://review.openstack.org/451294 | 10:22 |
| ralonsoh | sfinucan: yes, move every item (network:capabilities, etc) to resource providers | 10:22 |
| ralonsoh | sfinucan: this migration will modify a small part of this feature | 10:23 |
| ralonsoh | sfinucan: and should be fast to implement | 10:23 |
| *** Shunli has quit IRC | 10:23 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova-specs master: Proposed block accepting AZ renames https://review.openstack.org/446446 | 10:24 |
| sfinucan | ralonsoh: Oh yeah - I agree with doing it this way. Just making sure I've the big picture right in my head first :) | 10:24 |
| abhi89 | johnthetubaguy: at https://github.com/openstack/nova/blob/5a6d4e089dca2515c0f7e083a8b903ac7a166773/nova/volume/cinder.py#L202, after catching the OverLimit exception, we are raising OverQuota exception with overs='volumes'. But here the overs could have been volumes, gigabytes or per_volume_gigabytes. In each case there would be different error messages specific to the quota exceeded | 10:24 |
| ralonsoh | sfinucan: sure! this is the last working point of the spec, but I'll do it | 10:24 |
| *** nicolasbock has joined #openstack-nova | 10:24 | |
| johnthetubaguy | abhi89: I agree with you, it just the fix is in a different place to your current patch | 10:25 |
| sfinucan | ralonsoh: Ok, thanks for the info. I'm going to add a few points about including context comments (so people can understand what's going on) but that looks good to me otherwise | 10:25 |
| ralonsoh | sfinucan: thanks! | 10:25 |
| *** trinaths has left #openstack-nova | 10:27 | |
| *** satyar has quit IRC | 10:27 | |
| BobBall | johnthetubaguy / artom: Found the problem - it's relating to Ia0869dc6f7f5bd347ccbd0930d1d668d37695a22 (device tag disappearing) - because nova re-added the device tag in the 2.42 version but novaclient doesn't support that version, we have no way of using device tagging currently. | 10:28 |
| jsheeren | anyone know if it is possible to limit image A to be only able to use flavor X and Y, image B only flavor Z? | 10:28 |
| johnthetubaguy | BobBall: oh I forgot about that being for tags | 10:28 |
| BobBall | Not sure what the fix is currently -I did try hacking novaclient but it still can't cope with device tags for block devices... | 10:29 |
| johnthetubaguy | jsheeren: there are various image properties that can help for specific requirements, not sure there is a blanket only flavor X thing. | 10:29 |
| johnthetubaguy | BobBall: can't you raise the max supported API version up to 2.42? | 10:29 |
| *** haplo37_ has quit IRC | 10:31 | |
| BobBall | works for nics but not for block devices (volume_id is not recognised) | 10:32 |
| johnthetubaguy | BobBall: I am curious what the API tests are doing, I don't remember the context around that stuff | 10:35 |
| *** Drankis has joined #openstack-nova | 10:35 | |
| BobBall | :P | 10:36 |
| *** kevinz has quit IRC | 10:39 | |
| *** haplo37_ has joined #openstack-nova | 10:40 | |
| *** sdague has joined #openstack-nova | 10:42 | |
| *** kaisers has quit IRC | 10:48 | |
| *** Jack_Iv has quit IRC | 10:48 | |
| *** Jack_Iv has joined #openstack-nova | 11:02 | |
| *** phuongnh has quit IRC | 11:03 | |
| *** Jack_Iv has quit IRC | 11:04 | |
| *** Jack_Iv has joined #openstack-nova | 11:05 | |
| *** smatzek has joined #openstack-nova | 11:07 | |
| *** thorst has joined #openstack-nova | 11:08 | |
| *** alexpilotti has joined #openstack-nova | 11:12 | |
| *** alexpilotti has quit IRC | 11:12 | |
| *** prateek_ has joined #openstack-nova | 11:13 | |
| *** thorst has quit IRC | 11:13 | |
| *** prateek has quit IRC | 11:15 | |
| *** karimb has quit IRC | 11:15 | |
| *** zsli_ has quit IRC | 11:16 | |
| *** tbachman has quit IRC | 11:16 | |
| *** lucasagomes is now known as lucas-bbl | 11:20 | |
| *** dixiaoli has quit IRC | 11:21 | |
| *** dixiaoli has joined #openstack-nova | 11:21 | |
| *** prateek__ has joined #openstack-nova | 11:23 | |
| sfinucan | vladikr, sahid: Could you guys take a look at some point, please? https://review.openstack.org/#/q/topic:bump-min-libvirt+status:open | 11:24 |
| *** dixiaoli has quit IRC | 11:25 | |
| *** prateek_ has quit IRC | 11:26 | |
| *** alexpilotti has joined #openstack-nova | 11:26 | |
| *** Guest49144 has quit IRC | 11:30 | |
| *** ekhugen has quit IRC | 11:30 | |
| *** ekhugen_alt has quit IRC | 11:32 | |
| jsheeren | johnthetubaguy: thanks | 11:32 |
| *** patriciadomin has quit IRC | 11:32 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova master: Added PCI NUMA policies https://review.openstack.org/390520 | 11:32 |
| vladikr | sfinucan, sure. will drop off my kids to school and then | 11:33 |
| *** egarbade has joined #openstack-nova | 11:36 | |
| *** patriciadomin has joined #openstack-nova | 11:37 | |
| *** egarbade is now known as Guest26195 | 11:37 | |
| *** ekhugen has joined #openstack-nova | 11:37 | |
| *** ekhugen_alt has joined #openstack-nova | 11:37 | |
| *** venki has quit IRC | 11:40 | |
| *** venki has joined #openstack-nova | 11:41 | |
| artom | BobBall, if you pin the version to 2.32 it should work for both nic and bdm tags | 11:43 |
| artom | Until we merge the 2.42 novaclient change | 11:43 |
| *** kaisers has joined #openstack-nova | 11:43 | |
| *** zz_dimtruck is now known as dimtruck | 11:43 | |
| *** dillaman has quit IRC | 11:43 | |
| *** karimb has joined #openstack-nova | 11:45 | |
| *** baoli has joined #openstack-nova | 11:47 | |
| *** thorst has joined #openstack-nova | 11:47 | |
| gaurangt | hi, I'm facing issue in the nova instance creation and it is failing with following error in nova-compute.log | 11:48 |
| gaurangt | Failed to create resource provider record in placement API for UUID | 11:48 |
| gaurangt | The requested URL /placement/resource_providers was not found on this server | 11:48 |
| gaurangt | I actually added the <Directory> section in nova-placement-api.conf, but still the same issue. | 11:49 |
| *** mvk has quit IRC | 11:49 | |
| *** Jack_Iv has quit IRC | 11:49 | |
| *** Jack_Iv has joined #openstack-nova | 11:49 | |
| gaurangt | any idea what might be going wrong here? | 11:50 |
| gaurangt | nova-placement-api.conf looks like this - http://paste.openstack.org/show/606143/ | 11:50 |
| cdent | gaurangt: "/nova-placement-api" should be "/placement" in both places it shows up | 11:51 |
| cdent | actually the Alias is wrong in other ways too, just a sec | 11:51 |
| *** baoli has quit IRC | 11:52 | |
| cdent | gaurangt: we're you following some docs to set this up? | 11:53 |
| gaurangt | cdent, yes, I'm following the official OpenStack guide | 11:53 |
| gaurangt | https://docs.openstack.org/ocata/install-guide-rdo/nova-controller-install.html | 11:53 |
| *** Jack_Iv has quit IRC | 11:55 | |
| *** Jack_Iv has joined #openstack-nova | 11:55 | |
| cdent | gaurangt: it looks like the rdo packages may be configuring apache somewhat incorrectly | 11:55 |
| gaurangt | cdent, oh | 11:56 |
| cdent | changes the <Location ..> entry to be <Location /placement> and reload apache and see what happens | 11:56 |
| gaurangt | cdent, what should be the workaround here? | 11:56 |
| cdent | what I just said | 11:56 |
| *** markvoelker has joined #openstack-nova | 11:56 | |
| gaurangt | what about this - Alias /nova-placement-api /usr/bin/nova-placement-api? | 11:56 |
| gaurangt | just above the <Location..> | 11:56 |
| *** dillaman has joined #openstack-nova | 11:57 | |
| cdent | there too: Alias /placement /usr/bin/nova-placement-api | 11:57 |
| gaurangt | cdent, hmm ok | 11:58 |
| gaurangt | any nova service needs to be restarted? | 11:59 |
| cdent | no just apache | 11:59 |
| gaurangt | ok | 11:59 |
| gaurangt | ah, now it says - You don't have permission to access /placement/resource_providers | 11:59 |
| *** karimb has quit IRC | 12:01 | |
| gaurangt | cdent, anything still wrong? | 12:02 |
| *** chyka has joined #openstack-nova | 12:03 | |
| *** tbachman has joined #openstack-nova | 12:03 | |
| *** markvoelker has quit IRC | 12:03 | |
| cdent | move that <Directory> section outside of the VirtualHost | 12:06 |
| cdent | gaurangt: ^ | 12:06 |
| *** nkorabli has joined #openstack-nova | 12:06 | |
| gaurangt | cdent, ok | 12:06 |
| *** chyka has quit IRC | 12:07 | |
| *** tuanluong_ has quit IRC | 12:08 | |
| *** vks1 has quit IRC | 12:08 | |
| *** rmart04 has quit IRC | 12:09 | |
| cdent | gaurangt: once you get this figured out, I'd suggest you submit a bug to the rdo packagers. It seems like there might some confusion between what's being set the apache configuration and what's being set in the service catalog. Either the packages or the docs are wrong | 12:10 |
| *** amotoki has quit IRC | 12:11 | |
| *** edmondsw has joined #openstack-nova | 12:12 | |
| gaurangt | cdent, thanks it worked | 12:13 |
| *** venki has quit IRC | 12:13 | |
| cdent | gaurangt: ah, good, glad to hear it | 12:13 |
| gaurangt | cdent, I will submit a bug now | 12:13 |
| cdent | awesome! | 12:13 |
| gaurangt | cdent, thanks for the help ! | 12:13 |
| *** venki has joined #openstack-nova | 12:13 | |
| *** gouthamr has joined #openstack-nova | 12:14 | |
| *** iceyao has joined #openstack-nova | 12:15 | |
| *** lucas-bbl is now known as lucasagomes | 12:16 | |
| *** vladikr has quit IRC | 12:17 | |
| *** breton has quit IRC | 12:17 | |
| *** breton has joined #openstack-nova | 12:18 | |
| *** iceyao has quit IRC | 12:20 | |
| *** iceyao has joined #openstack-nova | 12:20 | |
| *** amotoki has joined #openstack-nova | 12:23 | |
| *** Jack_Iv has quit IRC | 12:24 | |
| *** Madasi has quit IRC | 12:24 | |
| *** Jack_Iv has joined #openstack-nova | 12:26 | |
| *** jsheeren has quit IRC | 12:26 | |
| *** hshiina has joined #openstack-nova | 12:27 | |
| *** Jack_Iv has quit IRC | 12:30 | |
| *** egonzalez has joined #openstack-nova | 12:31 | |
| *** iceyao has quit IRC | 12:31 | |
| *** iceyao has joined #openstack-nova | 12:32 | |
| *** ayogi has quit IRC | 12:32 | |
| *** cmurphy has left #openstack-nova | 12:35 | |
| *** prateek__ has quit IRC | 12:35 | |
| *** amotoki has quit IRC | 12:39 | |
| *** venki has left #openstack-nova | 12:40 | |
| *** vks1 has joined #openstack-nova | 12:41 | |
| *** iceyao_ has joined #openstack-nova | 12:42 | |
| *** Jack_Iv has joined #openstack-nova | 12:42 | |
| *** amotoki has joined #openstack-nova | 12:43 | |
| *** cleong has joined #openstack-nova | 12:44 | |
| *** scottda has joined #openstack-nova | 12:45 | |
| *** iceyao has quit IRC | 12:45 | |
| *** vks1 has quit IRC | 12:45 | |
| *** garyk has joined #openstack-nova | 12:45 | |
| *** amotoki has quit IRC | 12:49 | |
| egonzalez | Hi, it's been a recent change in master who change the behaviour in nova service-list? in current master output is empty with no service registered | 12:50 |
| *** zhurong has quit IRC | 12:54 | |
| *** vladikr has joined #openstack-nova | 12:54 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (4/4) https://review.openstack.org/455573 | 12:59 |
| *** Jack_Iv has quit IRC | 13:00 | |
| *** vks1 has joined #openstack-nova | 13:00 | |
| *** Jack_Iv has joined #openstack-nova | 13:00 | |
| *** esberglu has joined #openstack-nova | 13:00 | |
| *** iceyao_ has quit IRC | 13:00 | |
| *** chyka has joined #openstack-nova | 13:03 | |
| *** mriedem has joined #openstack-nova | 13:04 | |
| *** satyar has joined #openstack-nova | 13:05 | |
| *** jsheeren has joined #openstack-nova | 13:06 | |
| jsheeren | hi again, i'm back with my question about pinning an image to a flavor | 13:07 |
| jsheeren | we can set the min-disk and min-ram on the image, is there something similar with os_distro or os_type? | 13:07 |
| *** mdrabe has joined #openstack-nova | 13:07 | |
| *** chyka has quit IRC | 13:08 | |
| *** bpoulos has joined #openstack-nova | 13:10 | |
| *** jaypipes has joined #openstack-nova | 13:10 | |
| *** mdrabe has quit IRC | 13:11 | |
| BobBall | artom: Thanks but I don't think it will - nova client is rejecting the volume_id based on internal code, not checking the nova API version | 13:11 |
| *** amotoki has joined #openstack-nova | 13:12 | |
| *** lyan has joined #openstack-nova | 13:13 | |
| *** jsheeren has quit IRC | 13:13 | |
| *** karimb has joined #openstack-nova | 13:13 | |
| *** garyk has quit IRC | 13:13 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers-actions.inc https://review.openstack.org/454565 | 13:13 |
| *** peter-hamilton has joined #openstack-nova | 13:14 | |
| peter-hamilton | johnthetubaguy: do you have a minute to discuss cert validation? | 13:15 |
| *** rfolco has joined #openstack-nova | 13:15 | |
| *** garyk1 has joined #openstack-nova | 13:15 | |
| *** liverpooler has joined #openstack-nova | 13:18 | |
| *** Jack_Iv has quit IRC | 13:18 | |
| *** gabor_antal has quit IRC | 13:19 | |
| *** abhi89 has quit IRC | 13:20 | |
| *** abhishek has joined #openstack-nova | 13:20 | |
| *** gabor_antal has joined #openstack-nova | 13:20 | |
| *** dane-fichter has joined #openstack-nova | 13:22 | |
| *** Dinesh_Bhor has quit IRC | 13:22 | |
| *** bhagyashris has quit IRC | 13:22 | |
| andymccr | cdent: are you about? | 13:23 |
| cdent | andymccr: hi, yes | 13:23 |
| andymccr | cdent: hey :) if you have a second - is there a reason why CONTENT_LENGTH has to be set to an int? https://github.com/openstack/nova/commit/6dd047a3307a1056077608fd5bc2d1c3b3285338 | 13:23 |
| *** Dinesh_Bhor has joined #openstack-nova | 13:24 | |
| andymccr | im having issues in that content_length is present, but empty so its failing with a valueError: https://bugs.launchpad.net/nova/+bug/1681843 | 13:24 |
| openstack | Launchpad bug 1681843 in OpenStack Compute (nova) "Nova-placement returns "ValueError: invalid literal for int() with base 10: ''"" [Undecided,New] | 13:24 |
| andymccr | i figure the easiest fix is just to remove the int() (which works) but wanted to make sure im not overlooking anything | 13:24 |
| *** erlon has joined #openstack-nova | 13:24 | |
| *** bhagyashris has joined #openstack-nova | 13:25 | |
| *** iceyao has joined #openstack-nova | 13:25 | |
| cdent | andymccr: need to catch up and get my brain right, just a sec | 13:25 |
| andymccr | cdent: no problem. lemme know if i can explain anything better/further | 13:25 |
| peter-hamilton | jaypipes: do you have a sec to discuss cert validation? | 13:25 |
| jaypipes | peter-hamilton: here on IRC? or in a hangout or something? | 13:26 |
| peter-hamilton | jaypipes: here's fine, just some quick questions | 13:26 |
| jaypipes | peter-hamilton: go for it. | 13:26 |
| *** jianghuaw has joined #openstack-nova | 13:26 | |
| peter-hamilton | jaypipes: are you ok with adding two conf options, (1) to enable/disable cert validation, and (2) to provide a default list of trusted cert IDs | 13:27 |
| peter-hamilton | jaypipes: the goal would be to smooth the migration path and user experience | 13:27 |
| peter-hamilton | jaypipes: these were recommended by johnthetubaguy | 13:27 |
| jaypipes | peter-hamilton: if that smooths the integration path, I'm all for it.' | 13:27 |
| peter-hamilton | jaypipes: cool, i'm folding those in | 13:28 |
| jaypipes | rock on brother. | 13:28 |
| *** READ10 has joined #openstack-nova | 13:29 | |
| *** iceyao has quit IRC | 13:29 | |
| *** Jack_Iv has joined #openstack-nova | 13:30 | |
| openstackgerrit | Andy McCrae proposed openstack/nova master: Don't force CONTENT_LENGTH to be an int() https://review.openstack.org/455710 | 13:30 |
| cdent | andymccr: hmmm. that's icky behavior by nginx. what that check is trying to do is detect that there is a body with some length, without actually looking at the body, and then complain if there's no content-type header | 13:31 |
| cdent | so if nginx is sending both, but with empty values, neither line is correct | 13:31 |
| andymccr | cdent: agreed, although im confused as to why we need to force it to be an int? if there are no integer comparisons after that | 13:32 |
| cdent | andymccr: probably because i was being overzealous: the spec says that if the header is present it's value is made up of DIGITS | 13:33 |
| *** mvk has joined #openstack-nova | 13:33 | |
| andymccr | ahh | 13:33 |
| *** smatzek has quit IRC | 13:33 | |
| andymccr | cdent: any issue with removing the int? alternatively, we could get it, check it's not None and then make sure it's an int? | 13:33 |
| andymccr | (or something slightly more efficient i assume) | 13:34 |
| cdent | so yeah, your fix is probably okay, but if you're in there, addressing nginx'isms then you probably need to fix the content-type thing too | 13:34 |
| *** al498u has joined #openstack-nova | 13:34 | |
| *** eharney has joined #openstack-nova | 13:34 | |
| *** Jack_Iv has quit IRC | 13:35 | |
| cdent | a) drop the int b) if not environ.get('CONTENT_TYPE', ''): | 13:35 |
| cdent | andymccr: that would get the correct behavior wouldn't it? | 13:35 |
| *** Jack_Iv has joined #openstack-nova | 13:36 | |
| andymccr | cdent: hmm thinking. | 13:36 |
| andymccr | cdent: how it is now worked on content_type, because i guess it is defined, so they're both defined (but empty) - trying to think if there is another use case (that isnt this) where that would cause an issue | 13:37 |
| mriedem | jaypipes: johnthetubaguy: sdague: at some point i need to talk with you about the os-interface response body and how we want that to look | 13:37 |
| cdent | andymccr: the thing we are trying to prevent here is an empty content-type | 13:37 |
| cdent | and I was thinking that the environ would only have the key if the header was present | 13:38 |
| andymccr | cdent: so when CONTENT_LENGTH is not empty, make sure CONTENT_TYPE is also not empty | 13:38 |
| cdent | yes | 13:38 |
| andymccr | ohh i see what you mean | 13:38 |
| andymccr | ok lemme fix! | 13:38 |
| *** udesale has quit IRC | 13:40 | |
| cdent | andymccr: thank you! | 13:40 |
| *** ecelik has joined #openstack-nova | 13:40 | |
| *** ecelik has left #openstack-nova | 13:40 | |
| *** diana_clarke has joined #openstack-nova | 13:41 | |
| frickler | did nova stop writing libvirt.xml into the instance-dir with ocata? or do I just need a specific option for it? | 13:42 |
| *** Jack_Iv has quit IRC | 13:44 | |
| *** Jack_Iv has joined #openstack-nova | 13:45 | |
| *** mlavalle has joined #openstack-nova | 13:48 | |
| *** Jack_Iv has quit IRC | 13:48 | |
| *** Jack_Iv has joined #openstack-nova | 13:48 | |
| openstackgerrit | Andy McCrae proposed openstack/nova master: Don't force CONTENT_LENGTH to be an int() https://review.openstack.org/455710 | 13:49 |
| *** crushil has joined #openstack-nova | 13:50 | |
| *** Jack_Iv has quit IRC | 13:50 | |
| *** Jack_Iv has joined #openstack-nova | 13:51 | |
| *** Jack_Iv has quit IRC | 13:52 | |
| *** Jack_Iv has joined #openstack-nova | 13:52 | |
| *** dimtruck is now known as zz_dimtruck | 13:53 | |
| *** lpetrut has quit IRC | 13:53 | |
| *** Jack_Iv has quit IRC | 13:54 | |
| *** Jack_Iv has joined #openstack-nova | 13:54 | |
| *** awaugama has joined #openstack-nova | 13:55 | |
| sdague | mriedem: I was looking through that this morning | 13:58 |
| jaypipes | mriedem: sure, just gimme a few. need to restart my workstation.. | 13:58 |
| *** jaypipes has quit IRC | 13:58 | |
| sdague | mriedem: I guess the question is, what do we think people need to do with this? | 13:58 |
| *** Jack_Iv has quit IRC | 13:58 | |
| mriedem | sdague: alex_xu: just left replies in https://review.openstack.org/#/c/455023/ | 14:00 |
| mriedem | sdague: to list ports attached to their server | 14:00 |
| *** kaisers has quit IRC | 14:00 | |
| mriedem | sdague: same reason we have os-volume_attachments | 14:00 |
| sdague | so that they can go operate on those ports in neutron? | 14:00 |
| mriedem | or detach a port from the server in the same api | 14:00 |
| sdague | like after they get that, what do they do with it next? | 14:00 |
| *** jaypipes has joined #openstack-nova | 14:01 | |
| mriedem | GET /os-interface - nothing attached, POST /os-interface/port_id - attach a port, GET /os-interface, see my port is attached, DELETE /os-interface/port_id - detach the port | 14:01 |
| *** Jack_Iv has joined #openstack-nova | 14:01 | |
| jaypipes | mriedem: ok, back... | 14:01 |
| mriedem | jaypipes: replied to comments inline | 14:01 |
| johnthetubaguy | peter-hamilton: thanks for taking a look at those | 14:01 |
| peter-hamilton | johnthetubaguy: not a problem | 14:02 |
| mriedem | alex_xu: pointed out duplication with the /ips API such that we could drop the fixed_ips and mac_addr part of os-interface | 14:02 |
| *** p01nt3r75 has joined #openstack-nova | 14:02 | |
| peter-hamilton | johnthetubaguy: can you provide more info on versionedobjects? | 14:02 |
| mriedem | we could drop port_state since it doesn't really make sense | 14:02 |
| mriedem | or help | 14:02 |
| peter-hamilton | johnthetubaguy: i'm not that familiar with them | 14:02 |
| mriedem | peter-hamilton: https://github.com/openstack/oslo.versionedobjects | 14:02 |
| sdague | /ips is all super weird, and that's all in the main server doc iirc | 14:02 |
| peter-hamilton | mriedem: thanks! | 14:02 |
| *** smatzek has joined #openstack-nova | 14:02 | |
| johnthetubaguy | peter-hamilton: sure thing, the rest of those instance extra entries are basically serialised versioned objects, helps with upgrades, etc, long term | 14:02 |
| mriedem | sdague: it is, in fact, more of /ips is in the main server doc than it is in /ips | 14:03 |
| mriedem | sdague: details wise | 14:03 |
| peter-hamilton | johnthetubaguy: ok, so it would just match the current convention for InstanceExtras | 14:03 |
| johnthetubaguy | peter-hamilton: yeah | 14:03 |
| mriedem | peter-hamilton: the nova objects are in https://github.com/openstack/nova/tree/master/nova/objects | 14:03 |
| peter-hamilton | johnthetubaguy: thanks | 14:03 |
| mriedem | instance_extra stuff is used in https://github.com/openstack/nova/blob/master/nova/objects/instance.py | 14:03 |
| *** garyk1 has quit IRC | 14:03 | |
| *** Jack_Iv has quit IRC | 14:04 | |
| peter-hamilton | johnthetubaguy: i'm fine with making that a versionedobject then | 14:04 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/models.py#L372 | 14:04 |
| peter-hamilton | johnthetubaguy: i'm working through some updates to the spec now | 14:04 |
| *** Jack_Iv has joined #openstack-nova | 14:04 | |
| johnthetubaguy | peter-hamilton: the extra bit of detail might be here, in case that helps with context: https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L718 | 14:04 |
| mriedem | if we store something in the nova database and it's passed over rpc, it should be a versioned object | 14:04 |
| peter-hamilton | johnthetubaguy: thanks, i'll look it over | 14:04 |
| johnthetubaguy | peter-hamilton: sweet, will try hit that as soon as its updated | 14:04 |
| johnthetubaguy | +1 mriedem | 14:04 |
| *** jaosorior is now known as jaosorior_away | 14:06 | |
| dansmith | mriedem: is this docs failure known? http://logs.openstack.org/92/443292/15/check/gate-nova-docs-ubuntu-xenial/a9a4dc5/console.html | 14:08 |
| dansmith | seems like the kind of thing that would be wide failure | 14:08 |
| mriedem | dansmith: not by me | 14:08 |
| dansmith | okay | 14:08 |
| mriedem | lemme checkade logstash | 14:09 |
| sdague | mriedem: I don't understand the dropping of mac address, how would you determine what guest interface is what port if you don't have that? | 14:09 |
| alex_xu | sdague: good point~ | 14:09 |
| mriedem | sdague: I only mentioned that b/c the /ips API also has it | 14:10 |
| mriedem | so maybe we remove mac from /ips and leave in os-interface | 14:10 |
| sdague | dansmith: it looks like that job ran on python 3.5, which is the issue | 14:10 |
| dansmith | sdague: talking in infra now, it's known | 14:10 |
| sdague | mriedem: well, if you have > 1 port I'm not sure how ips helps | 14:10 |
| mriedem | yeah http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22AttributeError%3A%20'dict_keys'%20object%20has%20no%20attribute%20'sort'%5C%22&from=7d | 14:10 |
| dansmith | pabelanger said they'd send out a note shortly | 14:11 |
| *** lpetrut has joined #openstack-nova | 14:11 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677 | 14:11 |
| mriedem | the /ips api is also a bit goofed up since it keys off the network label | 14:11 |
| mriedem | so it's less about showing fixed/floating ips for a server and more about showing addresses for a server within a network | 14:12 |
| *** markvoelker has joined #openstack-nova | 14:12 | |
| *** markvoelker has quit IRC | 14:12 | |
| *** felipemonteiro has joined #openstack-nova | 14:12 | |
| mriedem | and of course we have /os-fixed-ips/ | 14:13 |
| mriedem | but that's not tied to the server | 14:13 |
| *** haplo37_ has quit IRC | 14:13 | |
| *** markvoelker has joined #openstack-nova | 14:13 | |
| mriedem | and /os-floating-ips but that's not tied to a server either | 14:13 |
| sdague | ok, so, let's not go through everything that's wrong, that will just take us in loops :) | 14:13 |
| mriedem | right, | 14:13 |
| mriedem | so, | 14:13 |
| sdague | how compat are you trying to be here? | 14:14 |
| mriedem | can we agree that at some point in the future, we want to expose bdm and vif tags out of the rest api? | 14:14 |
| mriedem | since those are a nova-only concept | 14:14 |
| mriedem | well, i guess you can set tags on ports in neutron too | 14:15 |
| mriedem | but you can't tag volumes in cinder, and they aren't used the same way in nova anyway | 14:15 |
| sdague | https://etherpad.openstack.org/p/nova-os-interfaces - scratch space, because there seem like there are so many options | 14:16 |
| sdague | yeh, I think tags exposed would be good | 14:16 |
| *** abalutoiu has joined #openstack-nova | 14:16 | |
| mriedem | sdague: my goal is one day expose tags for both bdms and ports in a single microversion, and i planned on exposing vif tags out of the os-interface API | 14:16 |
| sdague | sure | 14:16 |
| mriedem | i'm totally fine with putting the os-interface response on a diet | 14:16 |
| mriedem | i.e. remove port_status and fixed_ips | 14:16 |
| mriedem | leave port_id, net_id, and mac_addr | 14:16 |
| sdague | so, the real question I wonder is if we are making enough change to those, if we change the resource url at the same time | 14:17 |
| mriedem | GET /servers/ports? | 14:17 |
| jaypipes | mriedem: honestly, I hate all the existing APIs for dealing with both BDMs and vNICs. | 14:17 |
| mriedem | GET /servers/{server_id}/ports? | 14:17 |
| sdague | I was going to say interfaces | 14:17 |
| *** lucasxu has joined #openstack-nova | 14:17 | |
| sdague | jaypipes: ok, well hate only gets so far :) | 14:17 |
| mriedem | jaypipes: yeah, provide your alternatie | 14:17 |
| mriedem | *alternative | 14:18 |
| jaypipes | fine, I will. | 14:18 |
| mriedem | because i'm about to ragequit on this spec | 14:18 |
| sdague | it seems like this is the way to understand relationships so you can chain through | 14:18 |
| sdague | well, etherpad sketch things first | 14:18 |
| sdague | is net_id readily available? | 14:22 |
| sdague | like how much of this data exists cheaply for us? | 14:22 |
| *** haplo37_ has joined #openstack-nova | 14:22 | |
| mriedem | it's cheap | 14:23 |
| mriedem | it's all in the instance info_cache | 14:23 |
| openstackgerrit | Andy McCrae proposed openstack/nova master: Don't force CONTENT_LENGTH to be an int() https://review.openstack.org/455710 | 14:24 |
| *** abalutoiu_ has joined #openstack-nova | 14:24 | |
| jaypipes | sdague: honestly, I'm thinking in terms of the end user here. Which end user needs to know the subnet_id or the net_id? | 14:24 |
| openstackgerrit | Andy McCrae proposed openstack/nova master: Allow CONTENT_LENGTH to be present but empty https://review.openstack.org/455710 | 14:25 |
| sdague | jaypipes: if you have multiple networks and some can get out and others cant? | 14:25 |
| mriedem | jaypipes: we agreed to drop fixed_ips from the response, which drops the subnet_id | 14:25 |
| openstackgerrit | Mario Villaplana proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | 14:25 |
| sdague | maybe mordred's pov would be good | 14:25 |
| jaypipes | sdague: it's just downright confusing to the end user, IMHO. I'd rather see an API that exposes things the end user is interested in in relation to the instance itself (not the port object in Neutron, which is what MANO/VNFM users are interested in) | 14:25 |
| sdague | as he fights with network discovery all the time | 14:25 |
| mordred | I do indeed! | 14:26 |
| * mordred reads scrollback | 14:26 | |
| -openstackstatus- NOTICE: latest base images have mistakenly put python3 in some places expecting python2 causing widespread failure of docs patches - fixes are underway | 14:27 | |
| sdague | mordred: we're trying to fix the interfaces API in nova, because it remains full of some nova-net gorp. And the real question is what should it return in it's new state | 14:27 |
| mordred | so - as a I read - a few quick thoughts: the addresses dict as it is today is horrible to work with, I'd love something new - I'd also love for port_ids to be returned inside of the server resource for all ips reported | 14:27 |
| jaypipes | sdague: what I'm saying is not that network info isn't interesting to some users (it is for VNF managers and orchestrators) but for the most part, none of that is interesting for a normal cloud user. What they want to see is simply the IP[v46] address information, the routes and that's pretty much it. | 14:27 |
| *** abalutoiu has quit IRC | 14:27 | |
| jaypipes | honestly, the VIF tag stuff should never have been added to Nova. It's a Neutron port profile thing iMHO. | 14:28 |
| mordred | jaypipes: a normal cloud user cannot use a vm in a cloud without knowing a TON of informaiton about the networks and ports | 14:28 |
| *** lucasxu has quit IRC | 14:28 | |
| alex_xu | Emm...thinking of most user should distinguish the interface by ip address, so they have to have two API call to delete a interface | 14:28 |
| jaypipes | mordred: other than routing and addressing information, what information about networks and ports do they need? | 14:28 |
| mordred | the number of neutron calls I have to do after getting a server from nova to figure out what the hell is going on with it is mind boggling | 14:28 |
| sdague | alex_xu: which is all well and good until their interface dosen't actually get an IP because of things | 14:28 |
| mordred | jaypipes: well, the port_id for one | 14:29 |
| dansmith | jaypipes: you mean the user-specified tag or the vlan tag? | 14:29 |
| *** lucasxu has joined #openstack-nova | 14:29 | |
| jaypipes | mordred: I'm thinking of myself as a normal cloud user here. In all my time using AWS, I've never needed anything other than routing and address info. | 14:29 |
| mordred | jaypipes: also whether or not the IP came from a network that can route off of the cloud or not | 14:29 |
| mordred | jaypipes: that's because AWS is only one install | 14:29 |
| jaypipes | dansmith: both. | 14:29 |
| sdague | jaypipes: because they have one very specific way of setting up networking | 14:29 |
| *** mdnadeem has quit IRC | 14:29 | |
| mordred | jaypipes: and they give you your ip in DNS | 14:29 |
| jaypipes | sdague: no, they have many actually. | 14:29 |
| dansmith | jaypipes: the user-specified tag seems extremely useful to me | 14:29 |
| mordred | in openstack, your cloud could be doing networking a bilion different ways | 14:29 |
| mordred | and doesn't use dns to return your ip | 14:29 |
| jaypipes | dansmith: sure, on the Neutron *port* object, not on the server instance's information. | 14:30 |
| dansmith | jaypipes: not for public cloud so much, but for just base IaaS where you are trying to arrange your I in a particular way... | 14:30 |
| mordred | jaypipes: I REALLY need the port id on server objects | 14:30 |
| dansmith | jaypipes: but the whole point is to know which interface in the guest is the one you tied to which neutron network | 14:30 |
| mordred | it would save at least 4 additional API calls, including 2 lists | 14:30 |
| dansmith | jaypipes: otherwise it's just ethN and you don't know what to do with it | 14:30 |
| mordred | that's per server | 14:30 |
| mordred | yah | 14:30 |
| mordred | I have no way of knowing just from an address on the server if it's routable or if it's attached to the server or is NATed | 14:31 |
| mordred | those are important things to know | 14:31 |
| mordred | "fixed vs. floating" doesn't tell me anytihng by itself | 14:31 |
| sdague | mordred: so... network_id also valuable in that context? I think that's kind of any outstanding question | 14:31 |
| mordred | sdague: yes | 14:31 |
| mordred | it is | 14:31 |
| sdague | https://etherpad.openstack.org/p/nova-os-interfaces | 14:31 |
| sdague | some sketching | 14:32 |
| jaypipes | mordred: I still don't understand why this is information a user of Nova needs to know (versus just calling Neutron if you're a power user or a VNFM) | 14:32 |
| mordred | because if an IP is not floating but is fixed, I need to be able to ask neutron whether or not the network the ip is from is externally routable or not | 14:32 |
| openstackgerrit | Andy McCrae proposed openstack/nova master: Allow CONTENT_LENGTH to be present but empty https://review.openstack.org/455710 | 14:32 |
| mordred | jaypipes, sdague: network id would be nice and save me a roundtrip - but port_id is the more important thing | 14:32 |
| mordred | because port_id is my foreign key into my neutron resources | 14:33 |
| jaypipes | mordred: you're equating floating IP to "publically routable" but OK... still doesn't answer why the *Nova* API needs to expose this stuff when Neutron's API already does. | 14:33 |
| sdague | mordred: yeh, port_id seems mandatory | 14:33 |
| mordred | jaypipes: port_id is the essential - I can live wihtout network being on the server | 14:33 |
| jaypipes | mordred: I'm not arguing against port_id being in the return.. | 14:33 |
| mordred | awesome | 14:33 |
| sdague | but, network_id also doesn't cost us anything | 14:33 |
| mordred | if we can agree on port id, then we're at a good baseline :) | 14:33 |
| jaypipes | mordred: I'm arguing against crap like Neutron's "port_state" which is totally out of sync with the actual state of the VIF on the VM. | 14:34 |
| sdague | and it would save mordred round trips in real world use cases | 14:34 |
| jaypipes | mordred: which we just dump out into our os-interface/ return. | 14:34 |
| *** dave-mccowan has joined #openstack-nova | 14:34 | |
| mriedem | jaypipes: we also agreed to drop port_state | 14:34 |
| jaypipes | also, WTF is it os-interface and not os-interfaces/ or, I don't know... just /interfaces. :( guh. | 14:34 |
| mordred | oh- so - I'm saying please add port_id to the server | 14:34 |
| mordred | not to /servers/{server}/interfaces | 14:34 |
| mordred | adding it to /interfaces doens't save me anything | 14:35 |
| jaypipes | mordred: you mean a list of port_ids I assume? | 14:35 |
| mordred | I want a port id with every ip address in addresses | 14:35 |
| sdague | mordred: ok, we weren't talking about serviers | 14:35 |
| mriedem | mordred: right which gets us into fixed the addresses monster | 14:35 |
| mordred | (although I really want a new thing that isn't addresses, but I'd take just adding a port_id to addresses) | 14:35 |
| mriedem | which we weren't getting into for this spec | 14:35 |
| mordred | gotcha | 14:35 |
| mordred | so - for me, /interfaces is not useful at all | 14:35 |
| mordred | because I've got neutron if I want to query that info | 14:35 |
| jaypipes | mordred: EXACTLY. | 14:36 |
| mordred | I want to know how to contact my server without an additional roundtrip | 14:36 |
| *** eharney has quit IRC | 14:36 | |
| jaypipes | mordred: why are we even trying to return any of this other than a list of port_ids (or even hyperlinks to a Neutron URL). | 14:36 |
| *** eharney has joined #openstack-nova | 14:36 | |
| mordred | and the server record as it is today is not sufficient for that | 14:36 |
| mordred | jaypipes: don't return hyperlinks, they're gross | 14:36 |
| mriedem | so GET /ports/?device_id=<server uuid> is preferred? | 14:37 |
| mordred | mriedem: that's what I have to do today | 14:37 |
| mordred | so - let me give a specific use case, that might help | 14:37 |
| jaypipes | mordred: please. | 14:37 |
| mordred | I boot a server and I now want to attach a new floating ip | 14:37 |
| mordred | in order to do that, I need the port id of the fixed ip on the server so that I can pass it to the neutron create_floating_ip call | 14:38 |
| mordred | but in order to be able to do that, Ineed to call GET /ports/?device_id=<server uuid> | 14:38 |
| *** andreas_s has quit IRC | 14:38 | |
| mordred | so I can find the port id | 14:38 |
| mordred | so it's "boot sever, list ports, create fip" - instead of "boot server, create fip" | 14:38 |
| *** lucasxu has quit IRC | 14:38 | |
| mordred | (that's the super basic use case and doesn't get in to "how do I know if I need a FIP or not" | 14:39 |
| mriedem | POST /servers/{'networks': 'auto-floating'} :) | 14:39 |
| mriedem | get me a network + floating IP | 14:39 |
| mordred | mriedem: I'd still need the port id | 14:39 |
| mordred | OR | 14:39 |
| jaypipes | mordred: if that's the use case, why does any of that information need to be exposed to the end user? I mean, why not just say "hey, boot me a server with external connectivity". oh wait... that's kind of what "get me a network" was supposed to be. | 14:39 |
| mordred | neutron auto-network-and-float would need to grow the abilityto take a server id | 14:40 |
| mordred | jaypipes: yes. cloud.create_server(auto_ip=True) is what I _actually_ want | 14:40 |
| alex_xu | mordred: after 'boot server' you can't get the network info, because the port created in nova-compute. So at least you need to 'boot server, show server, create fip' | 14:40 |
| *** armax has quit IRC | 14:40 | |
| mriedem | mordred: i'm not sure why you'd need the port id if nova takes care of this for you at server create | 14:40 |
| mriedem | anyway, we're sort of going off course here | 14:40 |
| mordred | alex_xu: yes - wel - Ihave to do a poll loop on show server anyway to tell when it's active | 14:40 |
| mordred | mriedem: because nova does not take care of this for me at server create | 14:41 |
| jaypipes | alex_xu: the port doesn't *have* to be created in nova-compute. in fact, for virtually all NFV use cases, the ports are created separately and before any VM is launched. | 14:41 |
| jaypipes | alex_xu: ala service function chaining | 14:41 |
| mriedem | mordred: i know, today it doesn't, i'm talking about requested support | 14:41 |
| mriedem | expanding get me a network | 14:41 |
| mordred | mriedem: unless nova has grown a "please make me a FIP if I need a fip on this cloud otherwise please do not get me a FIP" | 14:41 |
| mordred | mriedem: dear god in heaven yes that would be a BILLION times better | 14:41 |
| mordred | mriedem: but that logic, I'll warn you, does get rather complex :) | 14:42 |
| mordred | so I'll take port_id in addresses in the meantime :) | 14:42 |
| mriedem | right so we're digressing | 14:42 |
| alex_xu | jaypipes: emm..yea, and you creat the port by your own, then you already know the port id | 14:42 |
| mriedem | mordred has pointed out that he doesn't care about or use GET /os-interface/{server_id} | 14:42 |
| mriedem | sorry, GET /servers/{server_id}/os-interface | 14:43 |
| jaypipes | mriedem: is the choice here about whether to reduce os-interface/ to just a list of port_ids or to have it repeat some/all of the informatio nin the instance_info cache? | 14:43 |
| sdague | or.... delete os-interface entirely | 14:44 |
| sdague | and put port_id in addresses | 14:44 |
| *** mdrabe has joined #openstack-nova | 14:44 | |
| jaypipes | sdague: ++ | 14:44 |
| *** lucasxu has joined #openstack-nova | 14:44 | |
| alex_xu | sdague: +1 | 14:44 |
| mriedem | where are you going to move the POST and DELETE methods? | 14:44 |
| alex_xu | move to action? | 14:44 |
| alex_xu | our ugly action | 14:44 |
| sdague | well... | 14:44 |
| mriedem | and eventually when we want to expose vif tags, those are in 'addresses' as well? | 14:45 |
| mordred | sdague: ++ | 14:46 |
| sdague | mriedem: yeh, probably, they are honestly more useful than a lot of things we have in the server doc | 14:46 |
| jaypipes | mriedem: Neutron ports can be tagged in the port binding details. | 14:46 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read NIC features in libvirt https://review.openstack.org/444415 | 14:46 |
| mriedem | jaypipes: we don't use those tags | 14:46 |
| mriedem | jaypipes: not for the 2.32 microversion anyway | 14:47 |
| jaypipes | mriedem: yeah, unfortunately. now we have two places to look for them :( | 14:47 |
| -openstackstatus- NOTICE: we have rolled back centos-7, fedora-25 and ubuntu-xenial images to the previous days release. Feel free to recheck your jobs now. | 14:47 | |
| alex_xu | jaypipes: do you mean there is a copy in port binding detail for tag? or the tag in port binding detail is separated thing than nova's one | 14:48 |
| mriedem | and binding:profile in the port is completely free form, | 14:48 |
| mriedem | and a wild west of what could or might not be in there depending on version and backend of the neutron you're talking to | 14:48 |
| mriedem | so that's also a ton of fun | 14:48 |
| *** baoli has joined #openstack-nova | 14:49 | |
| *** mtanino has joined #openstack-nova | 14:49 | |
| *** adisky_ has quit IRC | 14:49 | |
| mordred | mriedem: because I don't follow everything y'all do - "when we want to expose vif tags" - can you point me to something where I can learn what vif tags are and what I might use them for? | 14:50 |
| jaypipes | alex_xu: what mriedem said. but I will add that *Neutron* is the owner of all information about a port, so regardless of what we think about Neutron's API, it is what is the canonical source of truth for things like os-vif. | 14:50 |
| mordred | ++ | 14:51 |
| jaypipes | mordred: it's NFV fun. | 14:51 |
| mriedem | mordred: https://docs.openstack.org/developer/nova/api_microversion_history.html#id29 | 14:51 |
| mordred | jaypipes: ah. so I do not care aboutthem :) | 14:51 |
| jaypipes | mordred: well, you should. | 14:51 |
| mordred | jaypipes: yah. I was being snarky | 14:51 |
| *** chyka has joined #openstack-nova | 14:51 | |
| mordred | those seem potentially cool | 14:52 |
| *** jlvillal_pto is now known as jlvillal | 14:52 | |
| sdague | mriedem / jaypipes so... I guess the one question I still have, which jaypipes and I are cycling on is there is some delay between "attach" and it being attached | 14:52 |
| mriedem | mordred: one example for a bdm tag i think is that, you know how we return a device_name for bdms attached to a server? and that's a total lie? people could think they can correlate that mountpoint in their guest to what the api says volume x is attached to, but that could be totally wrong | 14:52 |
| sdague | and what are we indicating in that interim | 14:52 |
| sdague | especially if there is no port_status | 14:52 |
| mriedem | i believe the bdm tag can actually be used to correlate within the guest a given volume attached to the server | 14:52 |
| mordred | mriedem: that would be neat | 14:54 |
| *** lucasxu has quit IRC | 14:54 | |
| jaypipes | sdague: well, the user could easily poll on GET /ports/{port_id} for the attachment state. | 14:54 |
| sdague | jaypipes: yeh, that's probably reasonable | 14:54 |
| mriedem | sdague: i think ^ is what tempest does | 14:54 |
| * mordred thinks the server should not be active until the ports are attached | 14:54 | |
| mriedem | i added the code to do it i think, will look | 14:54 |
| sdague | so it only shows up in servers once it's really there | 14:54 |
| mriedem | mordred: we already do that | 14:55 |
| sdague | mordred: you can add ports later too | 14:55 |
| *** lucasxu has joined #openstack-nova | 14:55 | |
| mriedem | we wait for the vif plug callback from neutron | 14:55 |
| jaypipes | mordred: we're talking about the live attach/detach of a port. | 14:55 |
| *** nkorabli has quit IRC | 14:55 | |
| mordred | sdague: yes - post-create port attach I think polling the port in neutron for attach state is right - or port not showing up on server until it's attache | 14:55 |
| mordred | either way | 14:55 |
| mriedem | sdague: https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_attach_interfaces.py#L100 | 14:56 |
| *** marst has quit IRC | 14:56 | |
| *** chyka has quit IRC | 14:56 | |
| mriedem | sdague: https://github.com/openstack/tempest/blob/master/tempest/common/waiters.py#L264 | 14:56 |
| *** kaisers has joined #openstack-nova | 14:56 | |
| mriedem | so tempest totally relies on the port_state in the os-interface response to see when the port is ACTIVE | 14:56 |
| mriedem | and the attach is done | 14:56 |
| sdague | mriedem: ok, sure | 14:57 |
| sdague | so, you down at the bottom of the etherpad | 14:57 |
| sdague | I think we could get that whole thing with the ports sub resource that jay proposed | 14:57 |
| mordred | mriedem: maybe it's time to switch tempest to poll neutron port state instead? | 14:57 |
| mriedem | mordred: i'm not saying that can't be change | 14:58 |
| *** hongbin has joined #openstack-nova | 14:58 | |
| mriedem | the test doesn't even run if you're using nova-net https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_attach_interfaces.py#L35 | 14:58 |
| *** belmoreira has quit IRC | 14:59 | |
| mordred | mriedem: awesome. so it's only a neutron test anyway :) | 14:59 |
| sdague | mriedem: sure, but also realize this isn't contract code, it's the incidental code | 14:59 |
| mriedem | mordred: os-interface GET ONLY works for neutron | 14:59 |
| mriedem | it's a NotImplementedError for nova-net | 14:59 |
| sdague | yeh | 14:59 |
| mriedem | and os-virtual-interfaces ONLY works for nova-net, and is a NotImplementedError for neutron | 14:59 |
| mordred | mriedem: I don't know that I even knew os-interface existed | 14:59 |
| mriedem | hence me wanting to collapse these | 14:59 |
| sdague | mriedem: anyway, on your level of rage, what about the add ports to server document, provide this new minimalist interface for the port attach/detach/attaching status | 15:00 |
| jaypipes | mriedem, sdague: sounds like we are converging on something here.. | 15:00 |
| johnthetubaguy | I always wish the volume attachments and network attachments kinda look/feel similar in the API, not sure I like either right now though | 15:00 |
| mordred | as I say that ... | 15:00 |
| mriedem | johnthetubaguy: i was trying to get them closer, this gets them much further apart | 15:00 |
| mordred | https://developer.openstack.org/api-ref/compute/?expanded=list-port-interfaces-detail is pretty | 15:00 |
| *** zz_dimtruck is now known as dimtruck | 15:01 | |
| sdague | mordred: you just said that did you no good :P | 15:01 |
| mriedem | sdague: honestly it's a lot more than i was planning to do for this | 15:01 |
| johnthetubaguy | mriedem: would we likely take everything there eventually? | 15:01 |
| sdague | mordred: litterally, that's where we started | 15:01 |
| mriedem | sdague: and if it comes to that, i'm basically going to punt for pike | 15:01 |
| mriedem | and just deprecate os-virtual-interfaces as the easy thing we have to do anyway, we could lump that in with deprecating multinic | 15:02 |
| sdague | mriedem: that's fine | 15:02 |
| mriedem | johnthetubaguy: who knwos | 15:02 |
| mriedem | johnthetubaguy: maybe eventually we do similar for bdms | 15:02 |
| mordred | sdague: well, yah - it doesn't - because it's an extra call - I'm just saying it _is_ nice looking, so whoever wrote it didn't do a bad job or anything | 15:02 |
| mriedem | GET /servers/{server_id}/volumes | 15:02 |
| mriedem | johnthetubaguy: but in 2 years time when we talk about doing that, we'll have either all moved on or likely changed our minds and come to some other conclusion | 15:03 |
| johnthetubaguy | mriedem: it feels like when they seems similar, we are closer to where we want to be | 15:03 |
| mordred | sdague: if the addresses dict was instead list of those, that might be nice | 15:03 |
| johnthetubaguy | +1 for deprecating the broken bits | 15:03 |
| *** hshiina has quit IRC | 15:03 | |
| mordred | although I'd have to go poke for a minute to see how much code I could delete if it was | 15:03 |
| sdague | ok, so I think that's where we stand, just deprecate the broken thing | 15:03 |
| mriedem | how about microversion 2.43: completely rewrite the entire gd compute api? | 15:03 |
| cdent | mriedem++ | 15:03 |
| *** sridhargaddam has joined #openstack-nova | 15:03 | |
| alex_xu | :) | 15:04 |
| *** sridhargaddam has quit IRC | 15:04 | |
| mriedem | because what i always think is an incremental step forward turns into a giant rewrite | 15:04 |
| * johnthetubaguy does sarcastic nod | 15:04 | |
| sdague | mriedem: yeh, they do have the natural tendancy to grow | 15:05 |
| sdague | anyway, I think deprecate in a mv is fine | 15:05 |
| sdague | that's a small change | 15:05 |
| *** rcernin has quit IRC | 15:05 | |
| mriedem | alex_xu: if we just deprecate os-virtual-interfaces, i think we should combine that in the same microversion as deprecating multinic | 15:06 |
| *** armax has joined #openstack-nova | 15:06 | |
| mriedem | so we don't have 20 separate microversions for just deprecating APIs | 15:06 |
| alex_xu | mriedem: i'm ok with that | 15:06 |
| mriedem | because that gets super annoying | 15:06 |
| jaypipes | ya | 15:06 |
| jaypipes | on the MV for 2 deprecations. | 15:06 |
| *** marst has joined #openstack-nova | 15:07 | |
| alex_xu | do you want me merge that two spec? or we just approve the two spec separately, then I can help to code them together? | 15:07 |
| alex_xu | and fyi, one more deprecation spec https://review.openstack.org/386888 :) | 15:08 |
| mriedem | alex_xu: i don't think i ever liked that one | 15:08 |
| mriedem | alex_xu: because of the very odd flow that the user has to do on the neutron side | 15:09 |
| alex_xu | mriedem: ok, I probably remember your point | 15:09 |
| mriedem | to hook up a server with a floating ip via neutron | 15:09 |
| mriedem | it's all technically possible using the APIs, but the UX seems pretty terrible | 15:09 |
| jaypipes | mriedem: pls see summary of mine on bottom of etherpad. | 15:09 |
| mriedem | alex_xu: i've abandoned https://review.openstack.org/#/c/455023/ and we can just merge it into https://review.openstack.org/#/c/384261/ | 15:10 |
| mriedem | alex_xu: since it's getting late for you, do you want me to just combine those? | 15:10 |
| mriedem | then i can harass sean or jay or john to approve that today | 15:10 |
| alex_xu | mriedem: will appreciate for that | 15:11 |
| mordred | mriedem: the nova fip proxy api should never be used | 15:11 |
| * johnthetubaguy nods | 15:11 | |
| mordred | mriedem: it's racy and impossible to use atomically | 15:11 |
| mordred | its use should be discouraged strongly | 15:11 |
| johnthetubaguy | fip = floating ip? | 15:11 |
| mordred | yah | 15:11 |
| jaypipes | yes | 15:11 |
| johnthetubaguy | mriedem: I think you removed that from alex_xu's original deprecation spec right? | 15:11 |
| jaypipes | mordred, mriedem, sdague: does that summary on the etherpad make sense? | 15:11 |
| *** garyk has joined #openstack-nova | 15:12 | |
| mriedem | johnthetubaguy: originally alex_xu had deprecating floating ip actions and multinic in the same spec, and they were separated yes | 15:12 |
| *** cNilesh has joined #openstack-nova | 15:12 | |
| johnthetubaguy | ah, separated, I don't remember finding the other one I guess. | 15:12 |
| egonzalez | Hi, it's been a recent change in master who change the behaviour in nova service-list? in current master output is empty with no service registered | 15:13 |
| mriedem | mordred: you should read this then https://review.openstack.org/#/c/386888/4 | 15:14 |
| mriedem | jaypipes: seems so | 15:14 |
| johnthetubaguy | mriedem: do we want to include floating ip in your mega merged deprecated spec? | 15:15 |
| amotoki | I totally agree that nova fip proxy is discouraged. on the other hand, it is sometimes useful i.e. in case of an instance has only one port. | 15:16 |
| mriedem | johnthetubaguy: when i originally read the spec about deprecating the ability to associate/disassocate floating ips to servers and what you had to do in neutron for that, it seemed really complicated which is why i avoided it | 15:16 |
| *** Drankis has quit IRC | 15:16 | |
| mriedem | if it's really just: | 15:17 |
| mriedem | neutron floatingip-create EXT_NET_ID | 15:17 |
| *** cNilesh has quit IRC | 15:17 | |
| mriedem | neutron floatingip-associate FLOATING_IP_ID VM_PORT_ID | 15:17 |
| mriedem | then that doesn't seem as bad | 15:17 |
| amotoki | it is about CLI perspective. my comment is not applied to the API perspective and I thought so from when I implemented the neutron proxy... | 15:17 |
| *** Jack_Iv has quit IRC | 15:17 | |
| johnthetubaguy | mriedem : I think thats correct | 15:17 |
| amotoki | yeah. I believe OSC can support it | 15:17 |
| mriedem | it could also be, | 15:18 |
| mordred | mriedem: that's the bad version | 15:18 |
| johnthetubaguy | mriedem: the network changed event bit was missing out that original spec, not sure if that was also a possible blocker | 15:18 |
| mordred | mriedem: neutron floatingip-create EXT_NET_ID VM_PORT_ID | 15:18 |
| mriedem | whenever i need to read through the docs on how to associate a floating ip to a server in docs.o.o, i can't figure it out | 15:18 |
| mordred | is actually the good way | 15:18 |
| mordred | it's a single call | 15:18 |
| johnthetubaguy | mordred: bad version? I am curious? | 15:19 |
| johnthetubaguy | oh | 15:19 |
| mriedem | johnthetubaguy: no i remember that part | 15:19 |
| mriedem | mordred: but neutron cli is deprecated :) | 15:19 |
| sdague | jaypipes: I think that 2 & 3 have to happen at the same time along with droping the os-interface resource | 15:19 |
| mordred | johnthetubaguy: the reason is that if you have multiple api accessors, and any of them want to attempt to re-use unattached floating ips, they have no way of knowing if a fip that is in the flow of being created/attached is actually up for grabs or not | 15:19 |
| mordred | mriedem: :) | 15:19 |
| jaypipes | sdague: os-interface, sure, but not os-virtual-interfaces, right? | 15:20 |
| mordred | if you use the single-api-call create-with-port version in places you know you want to create-and-attach, then it is possible to know that unattached fips are available | 15:20 |
| sdague | jaypipes: correct | 15:20 |
| *** baoli_ has joined #openstack-nova | 15:20 | |
| mriedem | so "neutron floatingip-create EXT_NET_ID VM_PORT_ID" creates a floating ip on EXT_NET_ID and then associates that to a fixed ip on VM_PORT_ID? | 15:20 |
| mordred | if you do the create/attach dance, you are always racing | 15:20 |
| johnthetubaguy | mordred: yeah, make sense | 15:20 |
| mordred | mriedem: yes- but it does it in one step | 15:20 |
| jaypipes | sdague: and technically, we could add a ports/ subresource listing before deprecating os-interface. | 15:20 |
| mordred | mriedem: so you never have a fip show up in a non-attached state | 15:20 |
| sdague | jaypipes: we could, but it would mostly create 1 mv where there was a funny duplication | 15:21 |
| mordred | mriedem: (we used to have to run a fip cleaner thread in nodepool before switching to the single-call neutron api) | 15:21 |
| jaypipes | sdague: sure, understood. | 15:21 |
| johnthetubaguy | mordred: not sure if the generation id they are adding to ports helps with that, I guess its a pattern they could use elsewhere | 15:21 |
| *** Jack_Iv has joined #openstack-nova | 15:21 | |
| mordred | johnthetubaguy: I haven't heard about this generation id | 15:21 |
| jaypipes | sdague: though we'd need to deprecate os-interface for a while anyway, so there'd still be a duplication period. | 15:22 |
| *** aarefiev is now known as aarefiev_afk | 15:22 | |
| johnthetubaguy | mordred: kevinbenton I think knows about that, for atomic port updates | 15:22 |
| mordred | neat. I like atomic things | 15:22 |
| *** baoli has quit IRC | 15:22 | |
| sdague | jaypipes: we actually don't | 15:22 |
| jaypipes | sdague: no? | 15:22 |
| sdague | jaypipes: no, it's a microversion change | 15:22 |
| johnthetubaguy | mordred: I was curious on your comment about server metadata getting out of sync, is that given the clouds you are using? They might not have the bug fix in them | 15:23 |
| mordred | johnthetubaguy: speaking of kevinbenton ... I just realized that my ssh-hostkey spec should actually be a neutron feature not a nova feature, so I'm going to be bugging kevinbenton about it and not you | 15:23 |
| rpodolyaka | mriedem: hey! you asked about https://review.openstack.org/#/c/381912/ yesterday. It's not on the top of my list, but yes, I was hoping either me or snikitin_ will take over it from pkholkin | 15:23 |
| * johnthetubaguy mordred: I believe this is the one: https://bugs.launchpad.net/neutron/+bug/1642918 | 15:23 | |
| openstack | Launchpad bug 1642918 in neutron "floating-ip nova notification is broken" [High,Fix released] - Assigned to YAMAMOTO Takashi (yamamoto) | 15:23 |
| mriedem | rpodolyaka: ok | 15:23 |
| mordred | johnthetubaguy: it's entirely possible, yes - but we have since learned to not trust that data as it's proxy data and just go directly to the source | 15:23 |
| mordred | johnthetubaguy: AWESOME | 15:23 |
| sdague | jaypipes: the old version continues to work as well, this is just the new prefered way. The old one isn't going to get dropped any time soon. | 15:23 |
| johnthetubaguy | mordred: yeah, that makes sense | 15:23 |
| alex_xu | mordred: under 'neutron floatingip-create EXT_NET_ID VM_PORT_ID' is one API or multiple API? | 15:23 |
| mordred | johnthetubaguy: that does, in fact, sound like the problem we saw :) | 15:23 |
| *** gcb has quit IRC | 15:24 | |
| mordred | alex_xu: well, I honestly don't know about the neutron command line client- but there is a single REST API for create-with-port | 15:24 |
| mordred | one sec, lemme find it for you | 15:24 |
| amotoki | alex_xu: it is a single API except name lookup | 15:24 |
| alex_xu | mordred: amotoki, ah, that is cool | 15:24 |
| mriedem | alex_xu: mordred: https://developer.openstack.org/api-ref/networking/v2/index.html?expanded=show-port-details-detail,create-floating-ip-detail#create-floating-ip | 15:25 |
| mriedem | POST /v2.0/floatingips with a port_id in the body | 15:25 |
| mordred | mriedem: yah | 15:25 |
| mordred | that'sit | 15:25 |
| mriedem | "To associate the floating IP with an internal port, specify the port ID attribute in the request body. If you do not specify a port ID in the request, you can issue a PUT request instead of a POST request." | 15:25 |
| *** psachin has quit IRC | 15:25 | |
| mordred | (this API is 100% of the reason I want port_ids on fixed ips reported in the nova server record, fwiw) | 15:26 |
| alex_xu | amotoki: I remember we can't do that in single API in long time before? | 15:26 |
| *** lucasxu has quit IRC | 15:26 | |
| mordred | sdague, jaypipes: so - don't hate me - but I do actually think net_id and subnet_id are also useful alongside the port_id in the addresses dict - largely because if I only have port_id I'll still have to go do a GET on the port to find out what network it's from so I can match it to cached information I've figured out about which networks are public and which are private | 15:28 |
| mordred | SO | 15:28 |
| mordred | if it's possible to add it when you add port_id without any additional cost - it is very useful (and still isn't duplicating all the additinonal info) | 15:28 |
| sdague | mordred: so... I think it's pushing it to put all of that in the server doc | 15:29 |
| sdague | in the subresource, sure | 15:29 |
| mordred | sdague: adding three ids adjacent to the ip is pushing it? | 15:29 |
| jaypipes | sdague: yeah, that's what I was referring to... as in "we still will duplicate information for a time being" :) | 15:30 |
| mordred | I mean, you already have the port_id in ram when you're building addresses and just delete it | 15:30 |
| jaypipes | mordred: sure, I hear ya. it's more of an efficiency thing, as we discussed. | 15:30 |
| sdague | mordred: it's mostly about the constant everyone wants everything they might ever need in the server document | 15:30 |
| sdague | and we're trying to resist that a bit | 15:31 |
| mordred | sure. but it's not everything they might ever need -it's the two foreign keys you need to know how to take action on the server from a networking perspective. | 15:31 |
| sdague | mordred: so, I think it's just time to walk away on this one, because the whole thing got dropped from pike about 30 minutes ago when it overran mriedem's rage level | 15:32 |
| mordred | ok | 15:32 |
| mordred | I'd _love_ to dig in to find the right thing whenver it's the right time to do that - but I totally grok rage level on this topic | 15:32 |
| mriedem | johnthetubaguy: alex_xu: so given the above discussion, maybe we do just merge the deprecation of add/removeFloatingIP with add/removeFixedIP (multinic), and also throw os-virtual-interfaces deprecation in there as well | 15:33 |
| johnthetubaguy | mriedem: I am very tempted by that | 15:33 |
| mriedem | johnthetubaguy: don't let me seduce you, you're married proper now | 15:34 |
| mriedem | mordred: sdague: jaypipes: yeah i'm running as far away from fixing bdms and ports in the compute API as i can for pike | 15:34 |
| mriedem | because both attempts crushed both balls | 15:34 |
| jaypipes | mriedem: fine with me. | 15:35 |
| jaypipes | mriedem: sorry for complaining. | 15:35 |
| *** eharney_ has joined #openstack-nova | 15:35 | |
| johnthetubaguy | mriedem: :) | 15:35 |
| mriedem | oh btw https://review.openstack.org/#/c/453272/ | 15:35 |
| mordred | mriedem: well, please let me know if/when the topic comes up again. I'm definitely both very interested, very opinionated and often very confused | 15:35 |
| sdague | mordred: yeh, well the issue is not a lack of opinions for sure.... | 15:36 |
| jaypipes | mriedem: apparently, Tommy *doesn't* Like. | 15:36 |
| * mriedem cues rimshot | 15:36 | |
| *** Jack_Iv has quit IRC | 15:37 | |
| mordred | sdague: :) | 15:37 |
| *** dane-fichter has quit IRC | 15:37 | |
| *** artom has quit IRC | 15:38 | |
| *** artom has joined #openstack-nova | 15:38 | |
| *** crushil has quit IRC | 15:39 | |
| *** eharney has quit IRC | 15:39 | |
| *** eharney_ is now known as eharney | 15:39 | |
| *** artom has quit IRC | 15:40 | |
| *** Jack_Iv has joined #openstack-nova | 15:40 | |
| *** artom has joined #openstack-nova | 15:40 | |
| mriedem | jaypipes: since you championed server tags in the comptue api, you might want to chime in on this cinder thread to do similar for volume tags http://lists.openstack.org/pipermail/openstack-dev/2017-March/114618.html | 15:40 |
| *** Jack_Iv has quit IRC | 15:40 | |
| mriedem | because the reply right now is, 'use searchlight' | 15:40 |
| jaypipes | mriedem: that would mean I want to tell DuncanT that he's wrong again. :) | 15:41 |
| *** Jack_Iv has joined #openstack-nova | 15:41 | |
| mordred | mriedem: are clouds using searchlight yet? | 15:42 |
| *** chyka has joined #openstack-nova | 15:42 | |
| *** tuan_luong has joined #openstack-nova | 15:42 | |
| *** tuanla_ has joined #openstack-nova | 15:43 | |
| mriedem | mordred: no idea | 15:43 |
| mriedem | mordred: searchlight wasn't listed in the oct 2016 user survey report anyway | 15:45 |
| * mordred likes searchlight as an idea, just hasn't seen uptick enough yet to turn it into the answer for things | 15:46 | |
| mriedem | mordred: well let me introduce you to https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/list-instances-using-searchlight.html | 15:46 |
| mriedem | mordred: you're going to end up wasting your entire gd day in this channel | 15:47 |
| *** Jack_Iv has quit IRC | 15:48 | |
| mordred | mriedem: luckily I like all of you people | 15:48 |
| sdague | mriedem: after a couple of hours yesterday, I did figure out why my flake8 in buffer checking didn't like the order import extension. It's that it's out of the box behavior isn't what we use | 15:48 |
| sdague | and that if the file being run on it is outside of the tox tree (and there are reasons tools do that) it then behaves differently | 15:49 |
| mordred | mriedem: neat! | 15:49 |
| sdague | anyway I have a work around, but as this is the second time that flake8-import-order caused an issue, I do kind of wonder if we really want to keep it, https://review.openstack.org/#/c/454853/ | 15:50 |
| mriedem | sdague: i have no opinion on it, and haven't had issues with it | 15:51 |
| *** Jack_Iv has joined #openstack-nova | 15:51 | |
| *** dtp has joined #openstack-nova | 15:51 | |
| sdague | mriedem: it broke the intel ci, again, because it does things by just being installed | 15:51 |
| mriedem | because it's in nova? | 15:51 |
| sdague | well, it used to be in test-requirements | 15:52 |
| sdague | which means it impacted things system wide | 15:52 |
| mriedem | it's not in test-requirements anymore though | 15:53 |
| sdague | it's not | 15:53 |
| sdague | now it's this other problem | 15:53 |
| amotoki | alex_xu: sorry for late. IIRC we can do that from the beginning: 'neutron floatingip-create --port-id <port-id> <floating-ip-net> | 15:53 |
| amotoki | alex_xu: neutron didn't change the floating IP API from Folsom, but I am not sure the neutron proxy in nova supports all of the API | 15:55 |
| *** kaisers has quit IRC | 15:55 | |
| sdague | mriedem: at this point it's in the land of "we shouldn't make it any harder to develop than we have to" | 15:56 |
| DuncanT | jaypipes: I've spent enough time telling you over the years that you're wrong, turn about is fair play. I'm often wrong, I just hope I'm right even more often :-) | 15:56 |
| jaypipes | :) | 15:57 |
| *** trinaths has joined #openstack-nova | 15:58 | |
| mriedem | sdague: i've had 0 problem with it | 15:58 |
| mriedem | i don't use tramp over emacs or whatever you said | 15:59 |
| mriedem | so i don't have an opinion | 15:59 |
| sdague | mriedem: that's fine, I've lost a full day of development | 15:59 |
| *** jistr is now known as jistr|mtg | 15:59 | |
| sdague | debugging why this does the wrong thing | 15:59 |
| mriedem | sorry about that. i've lost a full day on abandoned specs. | 15:59 |
| mriedem | it sucks | 15:59 |
| mriedem | i'm not -1 on it | 16:00 |
| *** karimb has quit IRC | 16:01 | |
| *** Jack_Iv has quit IRC | 16:01 | |
| DuncanT | jaypipes: Having caught up with what I said, it seems daft to have two apis (metadata and tags) for the same purpose... I guess if you /really/ want tags, we just make it a convenience API that updates one specific metadata key (tags), rather than having a whole new bunch of stuff in the db... not sure if that will make search prohibatively difficult | 16:02 |
| DuncanT | though... SQL has in 'in' operator, right? | 16:02 |
| *** Jack_Iv has joined #openstack-nova | 16:02 | |
| *** voelzmo has quit IRC | 16:02 | |
| jaypipes | DuncanT: that's not how the IN operator works in SQL. | 16:02 |
| *** bpoulos has quit IRC | 16:03 | |
| DuncanT | jaypipes: Step 1, fix SQL ;-) | 16:03 |
| *** voelzmo has joined #openstack-nova | 16:03 | |
| *** Jack_Iv has quit IRC | 16:03 | |
| jaypipes | DuncanT: we (in Nova-land) tend to separately define tags as "user-defined simple strings" whereas the metadata key/values can (and often are) added by the system itself and have a particular encoding of information in them. | 16:04 |
| jaypipes | DuncanT: so, tags on a server instance are just that... simple string tags ala the typical Web definition of the term. | 16:04 |
| *** crushil has joined #openstack-nova | 16:04 | |
| jaypipes | DuncanT: and the system never sets tags on anything. | 16:05 |
| jaypipes | DuncanT: only the user. | 16:05 |
| mriedem | the nova 'system' doesn't set anything on server.metadata | 16:05 |
| mriedem | it uses server.system_metadata | 16:05 |
| DuncanT | jaypipes: metadata in cinder should never be set by anything other than the user... we have admin-metadata for sematically significant things | 16:05 |
| mriedem | the single-tenant NFV monolith might be the system that uses server.metadata, idk | 16:05 |
| DuncanT | jaypipes: I think there's a historical lack of definition about what the 'metadata' fields are there for | 16:06 |
| jaypipes | mriedem: well, yes, sorry I was referring to the entire concept of key/value metadata. there is server metadata, system metadata on the server, image metadata, "metadetails" on the aggregate, etc. | 16:06 |
| *** Jack_Iv has joined #openstack-nova | 16:06 | |
| *** armax has quit IRC | 16:06 | |
| mriedem | don't forget about flavor extra specs | 16:07 |
| *** Jack_Iv has quit IRC | 16:07 | |
| mriedem | and resource provider traits | 16:07 |
| jaypipes | mriedem: :) how could I forget about them? :) | 16:07 |
| DuncanT | jaypipes: I don't think the best fix for historical lack of definition is to slap something new on the edge though | 16:07 |
| *** voelzmo has quit IRC | 16:07 | |
| jaypipes | mriedem: one of the reasons we didn't call traits "tags" for resource providers was specifically because it is the system/admin that sets traits on a resource provider, not a user. | 16:07 |
| DuncanT | I think we've only got volume.metadata, volume.admin_metadata, snapshot.metadata and backup.metadata, though I haven't checked carefully | 16:08 |
| jaypipes | DuncanT: like I said, I didn't want to get in a conversation with you about how you're wrong. :P | 16:08 |
| jaypipes | DuncanT: in case it's not clear, you're not wrong at all... | 16:09 |
| DuncanT | jaypipes: You actually had me worried for a moment :-) | 16:09 |
| sdague | mordred: with your user hat on, you want to check on this one - https://review.openstack.org/#/c/400883/ | 16:09 |
| jaypipes | DuncanT: the concept of a simple string tag is actually pretty universal in the web. The fact that OpenStack inherited the AWS EC2 concept of metadata key/value pairs is really just a vestigiality of the early Nova API more than anything else. | 16:10 |
| *** Jack_Iv has joined #openstack-nova | 16:10 | |
| mriedem | heh, i can even apply tags to VSIs in softlayer for gods sake | 16:10 |
| mriedem | and they have the worst api | 16:10 |
| jaypipes | mriedem: sshh, don't tell. | 16:11 |
| *** jianghuaw has quit IRC | 16:11 | |
| mriedem | they know | 16:11 |
| mordred | sdague: we do not seem to use that informaiton in glean at all | 16:13 |
| *** Jack_Iv has quit IRC | 16:13 | |
| *** annegentle has joined #openstack-nova | 16:13 | |
| sdague | mordred: ok, great | 16:13 |
| *** Jack_Iv has joined #openstack-nova | 16:13 | |
| sdague | I just wanted to make sure with a consumer before cutting that off | 16:13 |
| clarkb | sdague: mordred you sure that won't affect the udev rules for glean? | 16:14 |
| *** Jack_Iv has quit IRC | 16:14 | |
| *** baoli_ has quit IRC | 16:14 | |
| clarkb | I guess udev doesn't rely on metadata only what is actualy exposed by kernel | 16:14 |
| mordred | clarkb: I don't see us doing anything with link type in glean at all | 16:14 |
| clarkb | mordred: well glean does things with the link type in udev, but thats provided by linux kernel not metadata | 16:15 |
| mordred | right | 16:15 |
| mordred | I mean doesn't do anything with the link type from config-drive | 16:15 |
| *** p01nt3r75 has quit IRC | 16:16 | |
| jaypipes | mriedem, sdague, johnthetubaguy, dansmith, bauzas: https://review.openstack.org/#/c/376200/ ready for your eyeballs if you have some time today. | 16:17 |
| *** Jack_Iv has joined #openstack-nova | 16:17 | |
| *** nic has joined #openstack-nova | 16:17 | |
| sdague | jaypipes / cdent what's the status on api-ref for placement? | 16:19 |
| *** udesale has joined #openstack-nova | 16:20 | |
| cdent | sdague: https://review.openstack.org/#/q/topic:cd/placement-api-ref | 16:20 |
| *** browne has joined #openstack-nova | 16:20 | |
| *** Apoorva has joined #openstack-nova | 16:21 | |
| cdent | sdague: avolkov is working on getting the publishing jobs linked up right. the html drafts are created but not linking into the test results yet | 16:21 |
| sdague | cdent: is the publishing job up for review anywhere? | 16:21 |
| mriedem | markus_z: https://bugs.launchpad.net/nova/+bug/1680363 | 16:22 |
| openstack | Launchpad bug 1680363 in OpenStack Compute (nova) "Raise Unexpected API Error when get console log from stopped instance" [Undecided,In progress] - Assigned to Zhenyu Zheng (zhengzhenyu) | 16:22 |
| mriedem | markus_z: UnboundLocalError on https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2801 | 16:22 |
| sdague | traits seems big enough I don't want us to be digging too giant of a hole on the docs side, as we know how long it takes to dig out from the past nova experience | 16:22 |
| cdent | sdague: the initial ones have merged, last I talked to avolkov there's more that's WIP, but I don't currently have the details | 16:23 |
| *** derekh has quit IRC | 16:23 | |
| cdent | https://review.openstack.org/#/q/project:openstack-infra/project-config+owner:avolkov%2540mirantis.com+status:merged | 16:24 |
| *** baoli has joined #openstack-nova | 16:24 | |
| sdague | cdent: yeh, doesn't seem like any open patches | 16:25 |
| *** gyee has joined #openstack-nova | 16:25 | |
| cdent | yeah, I think in this case WIP means "on his disk" | 16:25 |
| sdague | ok | 16:26 |
| *** sridharg has quit IRC | 16:27 | |
| cdent | sdague: since we're in the same place at the same time, did you see that I left some notes on the uwsgi etherpad, just some playing around with options https://etherpad.openstack.org/p/devstack-uwsgi ? | 16:28 |
| sdague | cdent: sure | 16:28 |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Amend spec for placement project/user association https://review.openstack.org/454352 | 16:28 |
| *** egonzalez has quit IRC | 16:29 | |
| sdague | so the line 85 bit | 16:29 |
| cdent | it's opened some question on what we're actually trying to accomplish with uwsgi, logs, etc. because, as ever, there are options... | 16:29 |
| sdague | can you do that without ports | 16:29 |
| *** pcaruana has quit IRC | 16:29 | |
| *** Jack_Iv has quit IRC | 16:30 | |
| sdague | what I'd really like to do is find a way to not have the custom ports used anywhere | 16:30 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make all timestamps formats equal https://review.openstack.org/421981 | 16:30 |
| sdague | which I think means, 1 http server with a lot of /paths, talking to a bunch of uwsgi processes over the local app socket | 16:31 |
| *** trinaths has left #openstack-nova | 16:31 | |
| cdent | line 85 can do that | 16:32 |
| sdague | cdent: that I think gets us the most win | 16:32 |
| cdent | is the port issue to make sure that nothing connects to the port (even over localhost) because the first example binds to localhost only, so isn't externally exposed. | 16:33 |
| *** nkorabli has joined #openstack-nova | 16:33 | |
| *** Jack_Iv has joined #openstack-nova | 16:33 | |
| markus_z | mriedem: OK, will review it tomorrow. Thanks for the notice | 16:33 |
| *** Jack_Iv has quit IRC | 16:34 | |
| mordred | sdague: btw - if you get rid of the custom ports, I will buy you a unicornpony | 16:35 |
| mriedem | markus_z: nvm it's latent anyway | 16:35 |
| mriedem | markus_z: easy fix, it's not a regression | 16:35 |
| sdague | cdent: I want nothing to connect to it, because we still have collisions if we require them anywhere | 16:36 |
| *** markus_z has quit IRC | 16:36 | |
| sdague | cdent: I'm starting a new bit at the top about behaviors of the system | 16:36 |
| cdent | sdague: cool, I'll build the example of the l85 version tonight or tomorrow morning, and dump its implications when I do. should be simple-ish | 16:36 |
| *** Jack_Iv has joined #openstack-nova | 16:37 | |
| *** jistr|mtg is now known as jistr | 16:37 | |
| *** Jack_Iv has quit IRC | 16:37 | |
| *** Jack_Iv has joined #openstack-nova | 16:38 | |
| sdague | cdent: yeh, before you run off, look at my list of behaviors at the top of that etherpad | 16:39 |
| sdague | and make sure they make sense | 16:39 |
| *** lpetrut has quit IRC | 16:39 | |
| *** alexpilotti has quit IRC | 16:39 | |
| cdent | they do make sense, thanks for that, will leave some comments | 16:39 |
| *** alexpilotti has joined #openstack-nova | 16:40 | |
| *** udesale has quit IRC | 16:40 | |
| cdent | sdague: one question about logs: coming from a webby background I think in terms of an access log and and error log, as showin in one of the examples lower in the doc. uwsgi can have specific request logs if configured to do so but I'm not sure if those logs can go to systemd in the same was as the error log, does that matter | 16:41 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: fix null_safe_int when called with 0 https://review.openstack.org/455778 | 16:43 |
| sdague | cdent: we're going to be hooking the journal directly in the oslo.log setup I think | 16:43 |
| *** Jack_Iv has quit IRC | 16:43 | |
| sdague | i.e. I think we're going to straight bypass that ... but we'll see | 16:43 |
| cdent | okay, but what about request logs (of the access_log vareity). do they matter? | 16:44 |
| *** alexpilotti has quit IRC | 16:44 | |
| cdent | for me they've always been at least as important as the log messages generated by LOG.*('some message') | 16:44 |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Amend spec for placement project/user association https://review.openstack.org/454352 | 16:45 |
| *** lucasagomes is now known as lucas-afk | 16:45 | |
| *** burt has joined #openstack-nova | 16:46 | |
| *** gouthamr has quit IRC | 16:47 | |
| *** crushil has quit IRC | 16:48 | |
| sdague | cdent: honestly, I don't think the access logs are all that useful one way or another, because they are lacking all the identifiable information about request-id / project-id / user-id that let you dig any deeper about what was going on | 16:49 |
| sdague | mostly only really for high level statistics I think | 16:49 |
| bauzas | p!spy | 16:49 |
| sdague | at least that's how I react when I look at something like this - http://logs.openstack.org/31/450431/3/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/38bb8c6/logs/apache/tls-proxy_access.txt.gz | 16:49 |
| cdent | huh, I think they're the least noisy way to answer the question: is the service getting the hits I expect | 16:49 |
| cdent | but it's not really an issue, that info is a variety of places | 16:50 |
| *** garyk has quit IRC | 16:50 | |
| *** fragatina has joined #openstack-nova | 16:51 | |
| sdague | yeh, and with the journal, that should be a better way to see it. The logging standards are that those things should be in the INFO block, which should be easy to query. | 16:52 |
| *** kaisers has joined #openstack-nova | 16:52 | |
| melwitt | has anyone seen a docs job failure like this? "WARNING: no "man_pages" config value found; no manual pages will be written" | 16:52 |
| dansmith | melwitt: my docs job is failing too | 16:53 |
| *** fragatina has quit IRC | 16:53 | |
| *** fragatina has joined #openstack-nova | 16:53 | |
| *** karimb has joined #openstack-nova | 16:55 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: fix null_safe_int when called with 0 https://review.openstack.org/455778 | 16:56 |
| *** ralonsoh has quit IRC | 16:56 | |
| *** armax has joined #openstack-nova | 16:58 | |
| jaypipes | bauzas: have you checked out https://etherpad.openstack.org/p/new-instance-reservation ? | 16:59 |
| *** mgagne has joined #openstack-nova | 17:00 | |
| sdague | jaypipes / bauzas / cdent - https://bugs.launchpad.net/nova/+bug/1681658 | 17:00 |
| openstack | sdague: Error: Could not gather data from Launchpad for bug #1681658 (https://launchpad.net/bugs/1681658). The error has been logged | 17:00 |
| *** Jack_Iv has joined #openstack-nova | 17:00 | |
| sdague | oh... thank you openstack bot :P | 17:00 |
| cdent | sdague: that's correct behavior: max_unit means you can't have more than max_unit | 17:01 |
| cdent | it's doesn't depend on allocation_ratio | 17:01 |
| *** alexpilotti has joined #openstack-nova | 17:01 | |
| cdent | total capacity does | 17:01 |
| *** Jack_Iv has quit IRC | 17:02 | |
| sdague | cdent: ok, cool, can you update the bug with why it's a Won't Fix then? | 17:02 |
| cdent | sdague: you wanna get that or shall I? | 17:02 |
| cdent | jinxish | 17:02 |
| cdent | yeah, will do | 17:02 |
| sfinucan | dansmith, melwitt: It's a broken gate. There was a bot message on infra or docs a while back | 17:03 |
| dansmith | sfinucan: that was for the py3 thing | 17:04 |
| dansmith | which was supposedly already fixed | 17:04 |
| sfinucan | Oh, wait - different issue | 17:04 |
| *** amotoki has quit IRC | 17:04 | |
| sfinucan | the no man_pages thing can be fixed by removing same from Sphinx's conf.py or setting 'builders = html' in '[build_sphinx]' of 'setup.cfg' | 17:05 |
| sfinucan | blame pbr being too clever for its own good :( | 17:06 |
| sfinucan | https://github.com/openstack-dev/pbr/blob/master/pbr/builddoc.py#L66 | 17:06 |
| dansmith | sfinucan: are you saying that something has changed in some package such that we're dead until we start doing that? | 17:07 |
| sfinucan | dansmith: Depends on what project the issue is happening in. This isn't nova, is it? | 17:07 |
| *** abalutoiu_ is now known as abalutoiu | 17:08 | |
| dansmith | sfinucan: it's neutron, we're just talking about it in the nova channel | 17:08 |
| dansmith | sfinucan: kidding, yes nova :P | 17:08 |
| *** Jack_Iv has joined #openstack-nova | 17:08 | |
| dansmith | actually, melwitt, mine seems to still be the py3 thing | 17:08 |
| sdague | yeh, the known broken thing was python3 going on by default | 17:09 |
| *** Jack_Iv has quit IRC | 17:09 | |
| melwitt | hmm | 17:09 |
| dansmith | one of my changes is working though so maybe the puppet didn't propagate fully yet | 17:09 |
| jaypipes | sdague: done. | 17:09 |
| *** gouthamr has joined #openstack-nova | 17:10 | |
| sfinucan | dansmith: Yeah, it shouldn't happen for nova. All 'source/conf.py's (doc, api-ref, api-guide) have 'man_page' removed or set to something | 17:10 |
| sfinucan | Hence "this isn't nova" :) | 17:10 |
| *** ayogi has joined #openstack-nova | 17:11 | |
| melwitt | sfinucan: my failure is happening in nova-specs | 17:15 |
| *** sambetts is now known as sambetts|afk | 17:15 | |
| *** efoley__ has quit IRC | 17:16 | |
| melwitt | I see 'man_pages = []' under doc/source/conf.py | 17:16 |
| *** tuanla_ has quit IRC | 17:16 | |
| *** tuan_luong has quit IRC | 17:16 | |
| *** crushil has joined #openstack-nova | 17:16 | |
| melwitt | so it seems like it shouldn't be failing | 17:16 |
| melwitt | it's not failing locally, only in the gate | 17:16 |
| melwitt | and I rebased it already | 17:17 |
| melwitt | ¯\_(ツ)_/¯ | 17:18 |
| jaypipes | melwitt: yeah, I had the same reaction looking at your spec amendment's docs failure :( | 17:18 |
| *** gszasz has quit IRC | 17:18 | |
| *** mtanino has quit IRC | 17:21 | |
| openstackgerrit | Mario Villaplana proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | 17:24 |
| *** mvk has quit IRC | 17:25 | |
| *** tesseract has quit IRC | 17:26 | |
| *** tjones has joined #openstack-nova | 17:28 | |
| *** tjones has left #openstack-nova | 17:28 | |
| *** mlavalle has quit IRC | 17:31 | |
| *** kaisers has quit IRC | 17:32 | |
| *** kaisers has joined #openstack-nova | 17:32 | |
| *** vks1 has quit IRC | 17:36 | |
| cdent | sdague: hiccup on mod_proxy_uwsgi: we need this version https://uwsgi-docs.readthedocs.io/en/latest/Changelog-2.0.13.html to get unix socket support, xenial is 2.0.12 | 17:38 |
| cdent | do we have a usual solution to such issues? | 17:39 |
| *** garyk has joined #openstack-nova | 17:39 | |
| *** ltomasbo is now known as ltomasbo|away | 17:41 | |
| *** MasterOfBugs has joined #openstack-nova | 17:41 | |
| *** pramodrj07 has joined #openstack-nova | 17:41 | |
| clarkb | cdent: you can ask ubuntu to backport, but generally if its not a bug I don't think they do. The other option is to get it into cloud archive (and then use cloud archive). Or maybe just use tcp? | 17:41 |
| cdent | clarkb: not using tcp is a design goal | 17:42 |
| cdent | clarkb: https://etherpad.openstack.org/p/devstack-uwsgi | 17:42 |
| openstackgerrit | Andy McCrae proposed openstack/nova master: Allow CONTENT_LENGTH to be present but empty https://review.openstack.org/455710 | 17:42 |
| clarkb | cdent: to avoid the port allcoation madness? | 17:43 |
| cdent | in part yeah | 17:43 |
| clarkb | another option may be to use not apache if another tls terminator has better uwsgi support? | 17:43 |
| cdent | clarkb: yeah, nginx was the original plan, but we're doing discovery on apache options to see about limit the surface of the change | 17:44 |
| *** ociuhandu has quit IRC | 17:45 | |
| *** armax has quit IRC | 17:47 | |
| *** armax has joined #openstack-nova | 17:50 | |
| sdague | cdent: we have a certain amount of flexibility in making it a reasonable dev environment, even if it means more change. I'm totally find with the nginx cut over at the same time | 17:55 |
| *** fragatina has quit IRC | 17:57 | |
| *** mvk has joined #openstack-nova | 17:57 | |
| clarkb | can I selfishly ask that https://bugs.launchpad.net/nova/+bug/1676363 and its associated changes get reviews/input/etc? This is the sort of issue in the pike release that will break us (us being infra) when the noav change that precipitated this hits clouds | 17:58 |
| openstack | Launchpad bug 1676363 in OpenStack Compute (nova) "The network metadata should be more useful" [Medium,In progress] - Assigned to Dr. Jens Rosenboom (j-rosenboom-j) | 17:58 |
| *** baoli has quit IRC | 17:59 | |
| *** hshiina has joined #openstack-nova | 17:59 | |
| *** baoli has joined #openstack-nova | 17:59 | |
| *** armax has quit IRC | 18:01 | |
| *** hshiina has quit IRC | 18:03 | |
| jaypipes | clarkb: looking now. | 18:05 |
| *** karimb has quit IRC | 18:06 | |
| *** mtanino has joined #openstack-nova | 18:06 | |
| cdent | sdague: added some notes on the hackarounds I did to get mox_proxy_uwsgi working, going to switch to some other things for a while: https://etherpad.openstack.org/p/devstack-uwsgi | 18:16 |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification https://review.openstack.org/455801 | 18:18 |
| *** yamahata has joined #openstack-nova | 18:18 | |
| *** MasterOfBugs has quit IRC | 18:21 | |
| *** pramodrj07 has quit IRC | 18:21 | |
| *** nkorabli has quit IRC | 18:24 | |
| *** abalutoiu has quit IRC | 18:25 | |
| *** nkorabli has joined #openstack-nova | 18:25 | |
| *** tbachman has quit IRC | 18:25 | |
| *** lucasxu has joined #openstack-nova | 18:25 | |
| *** ociuhandu has joined #openstack-nova | 18:25 | |
| *** Jack_Iv has joined #openstack-nova | 18:26 | |
| *** Sukhdev has joined #openstack-nova | 18:26 | |
| *** priteau has quit IRC | 18:26 | |
| *** abalutoiu has joined #openstack-nova | 18:27 | |
| *** nkorabli has quit IRC | 18:28 | |
| *** prabhuraj has joined #openstack-nova | 18:28 | |
| prabhuraj | Hi all, Warm Greetings, someone could review my code https://review.openstack.org/#/c/397471/ | 18:29 |
| prabhuraj | https://review.openstack.org/#/c/397459/ | 18:29 |
| mriedem | prabhuraj: wrong channel? | 18:29 |
| mriedem | those are fuel patches | 18:29 |
| mriedem | this is nova | 18:29 |
| *** abalutoiu_ has joined #openstack-nova | 18:31 | |
| *** Apoorva_ has joined #openstack-nova | 18:31 | |
| *** dtp has quit IRC | 18:32 | |
| *** prabhuraj has quit IRC | 18:33 | |
| *** crushil has quit IRC | 18:33 | |
| clarkb | jaypipes: tldr on glean was cloud init couldn't configure rax interfaces and monty went off and made a thing that could. But also cloud init has historically made some very bad choices for us so I think mordred was just looking for an excuse at that point. (distro specific users, mounting volumes in places where we did not wnat them mounted are the two big ones that I recall) | 18:34 |
| *** abalutoiu has quit IRC | 18:34 | |
| *** Apoorva has quit IRC | 18:34 | |
| jaypipes | clarkb: hey, cool by me. I wasn't complaining about NIH or anything. was genuinely learning about a new thing :) | 18:34 |
| clarkb | ya, just thought I'd explain after seeing your comment :) | 18:35 |
| *** mgagne_ has joined #openstack-nova | 18:35 | |
| *** mgagne_ is now known as Guest89317 | 18:35 | |
| clarkb | jaypipes: there are days I am grumpy about it (especially when something breaks due to sillyness in glean) but then there are times when we don't break due to something cloud init would have done when I am happy | 18:35 |
| clarkb | :) | 18:35 |
| jaypipes | no worries clarkb :) | 18:36 |
| clarkb | but its become quite common with ironic uses I guess because it groks vlans and stuff | 18:36 |
| jaypipes | clarkb: doesn't cloud-init grok vlans as well? I think it does... | 18:36 |
| clarkb | I think it didn't for a long time? | 18:37 |
| jaypipes | clarkb: ok, done with reviews on that series. | 18:38 |
| jaypipes | clarkb: only thing I'd like to see is prefixlen be an int not a string. | 18:38 |
| clarkb | this is mostly second hand I don't actually do ironic things. VLANs and link aggregation were the two big features they like | 18:38 |
| jaypipes | clarkb: otherwise, looks fine to me. | 18:38 |
| clarkb | jaypipes: awesome thank you for taking a look | 18:38 |
| jaypipes | any time. | 18:38 |
| jaypipes | clarkb: link agg: https://git.launchpad.net/cloud-init/tree/cloudinit/sources/helpers/openstack.py#n590 and vlans: https://git.launchpad.net/cloud-init/tree/cloudinit/sources/helpers/openstack.py#n614 :) | 18:39 |
| *** ociuhandu has quit IRC | 18:40 | |
| *** ociuhandu has joined #openstack-nova | 18:41 | |
| mordred | jaypipes: yah -so - when we started in on glean, cloud-init was on a very explicit freeze not accepting new things becaue they were working on a v2 | 18:44 |
| mordred | jaypipes: which meant adding support for the network_info.json stuff was a no-go | 18:44 |
| mordred | they have since unfrozen, which is great for everyone | 18:45 |
| jaypipes | cool beans. | 18:45 |
| *** abhishek has quit IRC | 18:45 | |
| mordred | so at this point the main difference is size - glean assumes the only thing you want it to do is get networking set up and inject ssh keys - and that you NEVER want to talk to non-config-drive metadata service | 18:45 |
| mordred | cloud-init is much more flexible and lets you inject arbitrary scripts | 18:46 |
| mordred | which some people think is cool | 18:46 |
| *** Guest89317 has quit IRC | 18:46 | |
| mordred | and I support them in that - I, for myself, _never_ want to run scripts via cloud-init because I can run scripts over ssh much more easily and more debuggably :) | 18:47 |
| dansmith | mordred: orly, glean is configdrive-only? | 18:47 |
| *** kaisers has quit IRC | 18:47 | |
| mordred | dansmith: yup | 18:47 |
| dansmith | interesting | 18:47 |
| mordred | we have had nothing but terrible experiences with the ec2-style metadata service | 18:47 |
| mordred | crashing clouds and instances not booting and instances timing out and whatnot | 18:47 |
| mordred | so we avoid it like the plague | 18:47 |
| mordred | config-drive is teh ossum | 18:48 |
| dansmith | mordred: yeah, especially when you have puppet running every 15 minutes on every instance against central metadata services | 18:48 |
| mordred | dansmith: yup | 18:48 |
| dansmith | and when I say every fifteen minutes I mean "for 14 minutes total until smoke" | 18:49 |
| *** baoli has quit IRC | 18:49 | |
| mordred | dansmith: since cloud-init does support config-drive, i'd LOVE it if nova just defaulted config-drive to on, removed the option to turn it off, and then removed the ec2 metadata service | 18:49 |
| dansmith | yeah, the only problem is the staticness of it | 18:50 |
| mordred | dansmith: but I suppose that's at odds with my stance about not changing things, so maybe I'm a hypocrit :) | 18:50 |
| dansmith | i.e. volume attach and nic attach | 18:50 |
| mordred | right- so ... | 18:50 |
| jaypipes | lmao | 18:50 |
| mordred | network attach should just work, since clouds should just be offering dhcp, which means there is no need for config-drive to be updated (unless the cloud is broken) | 18:51 |
| *** mlavalle has joined #openstack-nova | 18:51 | |
| dansmith | mordred: unless you want/need metadata about that nic | 18:51 |
| mordred | but if you're adding a NIC after boot, you're adding the nic, so you as the orchestrator likely already know you're doing it and should be able to direct the instance to do something different,no? | 18:52 |
| dansmith | unless you have an agent that looks for nics with tags of "multipath-to-db" and enslaves them into a bond, | 18:52 |
| mordred | I mean, the only reason we need config-drive for network info anyway is for clouds without dhcp - for the rest I recommend using your orchestration tool to talk to the nova api | 18:52 |
| dansmith | not necessarily | 18:52 |
| jaypipes | mordred: right, unless you're a VNFM or MANO system that is dynamically swapping metadata about your workload's vNICs at runtime and having the workload reconfigure service function chains. | 18:52 |
| dansmith | that ^ | 18:53 |
| *** fragatina has joined #openstack-nova | 18:53 | |
| *** kaisers has joined #openstack-nova | 18:53 | |
| *** crushil has joined #openstack-nova | 18:53 | |
| * jaypipes drops NFV into water supply, poisons everyone. | 18:53 | |
| mordred | I'm still not sure why that isn't just an API call on the service API - which would of course be hard because we haven't bothered with per-instance credentials :) | 18:53 |
| dansmith | jaypipes: it does kinda sound like a chemical | 18:53 |
| jaypipes | mordred: bingo. | 18:54 |
| mordred | but ec2 metadata services is a poor workaround for having an agent in an instance just make API calls to the control plane in a sane fashion | 18:54 |
| jaypipes | mordred: also correct. | 18:54 |
| dansmith | so you want the instance to hit the api, find out the ports that are on the instance, and corellate them all by mac address to know what to do with which actual device in the OS? | 18:55 |
| mordred | dansmith: well, the list ports call also lists the mac addresses of the ports | 18:55 |
| mordred | but yah - I do that for every instance we spin up in nodepool | 18:56 |
| mordred | because I have to to figure out if the instance needs a floating ip or not | 18:56 |
| *** abalutoiu has joined #openstack-nova | 18:56 | |
| mordred | so sure! | 18:56 |
| dansmith | that works if you've got creds in the instance | 18:57 |
| *** alexpilotti has quit IRC | 18:57 | |
| mordred | we could also just have an API call that gives a rollup report similar to what's in the metadata service - I'm just saying it doesn't need to be a special network port that's only usable to talk to the special service that doesn't scale | 18:57 |
| *** baoli has joined #openstack-nova | 18:57 | |
| mordred | and instead we should focus on getting creds into the instance | 18:57 |
| *** kaisers has quit IRC | 18:57 | |
| mordred | which would make many people happy | 18:57 |
| *** alexpilotti has joined #openstack-nova | 18:57 | |
| dansmith | well, I dunno, given where we are, tags on the devices we add seems like a reasonable thing to me | 18:58 |
| dansmith | especially given that there are some devices for which there is no service to talk to | 18:58 |
| dansmith | like a raw pci device | 18:58 |
| mordred | wouldn't that raw pci device be there at boot? or be a thing you added to the vm via nova? | 18:59 |
| dansmith | if cinder gives us the serial number of the block device that we're going to add or something, then I guess we can do the same for cinder, | 18:59 |
| *** kaisers has joined #openstack-nova | 18:59 | |
| dansmith | but otherwise that's a problem | 18:59 |
| *** marst has quit IRC | 18:59 | |
| mordred | I mean - I hear you | 18:59 |
| dansmith | mordred: what does it matter if it was there at boot? | 18:59 |
| mordred | if it's there at boot, it can have data in config-drive without any issues | 19:00 |
| *** abalutoiu_ has quit IRC | 19:00 | |
| mordred | the static nature of config-drive and dynamic devices was the issue | 19:00 |
| *** fragatina has quit IRC | 19:00 | |
| dansmith | sure, I thought we were talking about not needing metadata at al | 19:00 |
| mordred | oh golly no | 19:00 |
| mordred | metadata is great | 19:00 |
| dansmith | cinder doesn't show me a device serial number or aything, | 19:00 |
| *** karimb has joined #openstack-nova | 19:00 | |
| dansmith | so not sure that I can do the same correlation inside the guest for it | 19:00 |
| mordred | well, mounting volumes is hard no matter what | 19:00 |
| mordred | because how they're presented to the guest varies by hypervisor | 19:01 |
| mordred | and the api can lie to you about the device name | 19:01 |
| dansmith | well, same for nic, except we have an address in the case of ethernet | 19:01 |
| mordred | dansmith: WHAT ABOUT TOKEN RING??? | 19:01 |
| dansmith | mordred: indeed, what about token ring indeed, | 19:01 |
| dansmith | but while you say that, | 19:01 |
| dansmith | there are definitely requests for weird custom L2 things | 19:02 |
| mordred | oh, I'm sure there are | 19:02 |
| dansmith | at which I laugh, but.. | 19:02 |
| *** alexpilotti has quit IRC | 19:02 | |
| *** garyk1 has joined #openstack-nova | 19:03 | |
| *** ociuhandu has quit IRC | 19:03 | |
| mordred | so ... my world view has given up on magical agents sitting inside of guests looking for metadata updates to automagically reconfigure themselves | 19:03 |
| mordred | that might color my opinion on how these things should work | 19:03 |
| *** ociuhandu has joined #openstack-nova | 19:03 | |
| dansmith | well, in that case you don't really need instance creds, right? | 19:03 |
| mordred | having systems that sit outside of the guests, that konw what's going on and that can reach in and execute commands in the guests works much better in my world so far | 19:03 |
| mordred | dansmith: that is correct, I do not | 19:04 |
| *** garyk has quit IRC | 19:04 | |
| dansmith | anyway, I just think that being consistent about how you plumb things at boot or after boot requires metadata, and I like configdrive for that for all cases other than that it can't be updated after boot | 19:04 |
| mordred | however, if instance creds are there, then in-instance things have access to the same power I have while sitting outside of the guest | 19:04 |
| mordred | dansmith: ++ | 19:04 |
| *** gjayavelu has joined #openstack-nova | 19:04 | |
| mordred | and I'd like more metadata - especially in cases where our current metadata is missing or lies | 19:04 |
| mordred | "this is the volume you named bob" | 19:05 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Idempotent PUT /resource_classes/{name} https://review.openstack.org/448791 | 19:05 |
| dansmith | mordred: right and we have that now, but only at boot for configdrive | 19:05 |
| *** tbachman has joined #openstack-nova | 19:05 | |
| dansmith | the volume thing I mean | 19:05 |
| dansmith | because we can generate that data only in the hypervisor driver | 19:05 |
| mordred | dansmith: we ALSO have the volume thing in libvirt with device names | 19:05 |
| dansmith | and because it's virt-specific, we only show that to the guest not the external API | 19:05 |
| dansmith | mordred: not with volume tags which we now have | 19:05 |
| dansmith | mordred: now you get a metadata blob that says "your vdb IS the volume you tagged as foo" | 19:06 |
| mordred | and that tag does't show up in /dev/disk? | 19:06 |
| dansmith | meaning /dev/disk/by-tag/? | 19:06 |
| mordred | yah | 19:07 |
| dansmith | something in the guest would have to do that for us, of course | 19:07 |
| *** garyk1 has quit IRC | 19:07 | |
| *** READ10 has quit IRC | 19:08 | |
| mordred | we get labels for normal cinder volumes now though - but I guess a disk can only have one lable and could have many tags | 19:08 |
| *** lucasxu has quit IRC | 19:08 | |
| dansmith | we get labels if they lay a filesystem down, right? | 19:08 |
| *** abalutoiu_ has joined #openstack-nova | 19:08 | |
| mordred | we get a by-$something thathas the volume name in somewhere - one sec, lemme find the thing I'm talking about | 19:08 |
| dansmith | I don't think so | 19:09 |
| *** abalutoiu__ has joined #openstack-nova | 19:09 | |
| dansmith | that would be by-label, and if they put a filesystem in and labeled it | 19:09 |
| dansmith | or are you talking about the virtio-$neutered_uuid ? | 19:10 |
| *** lucasxu has joined #openstack-nova | 19:11 | |
| mordred | dansmith: ok. I think I may just be smoking crack | 19:12 |
| *** abalutoiu has quit IRC | 19:12 | |
| dansmith | I would totally believe that | 19:12 |
| dansmith | :) | 19:12 |
| *** Jack_Iv has quit IRC | 19:12 | |
| mordred | dansmith: it's 19:12 UTC, which everyone knows is crack-o'clock | 19:12 |
| dansmith | obvi | 19:12 |
| *** abalutoiu_ has quit IRC | 19:12 | |
| *** Jack_Iv has joined #openstack-nova | 19:13 | |
| * dansmith assumes that by crack, mordred means "maple candy from vermont" | 19:14 | |
| mordred | of course! | 19:16 |
| mordred | dansmith: is there some other type I should know about? | 19:16 |
| dansmith | mordred: nah, stick to the safe stuff | 19:16 |
| *** dharinic has joined #openstack-nova | 19:16 | |
| *** mgagne_ has joined #openstack-nova | 19:18 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate CONF.api.allow_instance_snapshots https://review.openstack.org/455100 | 19:18 |
| *** mgagne_ is now known as Guest4366 | 19:19 | |
| *** dave-mccowan has quit IRC | 19:21 | |
| *** armax has joined #openstack-nova | 19:22 | |
| *** mgagne has quit IRC | 19:24 | |
| *** Guest4366 is now known as mgagne | 19:25 | |
| *** mgagne has quit IRC | 19:25 | |
| *** mgagne has joined #openstack-nova | 19:25 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272 | 19:27 |
| *** liusheng has quit IRC | 19:29 | |
| *** liusheng has joined #openstack-nova | 19:29 | |
| *** bmace has quit IRC | 19:30 | |
| *** bmace has joined #openstack-nova | 19:30 | |
| *** tbachman has quit IRC | 19:31 | |
| mgagne | gibi: mriedem asked me to ask you about your input/opinion on notification impact on this proposed spec: https://review.openstack.org/#/c/453272/ | 19:35 |
| mgagne | gibi: in PS3: "Hmm, this seems like something for which we should send a notification. I don't think we send notifications for any external events that we handle today though. It might be worth talking to gibi about this. I wouldn't block this spec on it, but it seems useful." | 19:35 |
| *** pramodrj07 has joined #openstack-nova | 19:36 | |
| *** MasterOfBugs has joined #openstack-nova | 19:36 | |
| *** alexpilotti has joined #openstack-nova | 19:37 | |
| cfriesen | does anything test with "force_raw_images" set to false, or with glance image formats other than raw/qcow2 ? | 19:39 |
| mriedem | of course not http://codesearch.openstack.org/?q=force_raw_images&i=nope&repos=nova | 19:41 |
| *** awaugama has quit IRC | 19:41 | |
| mriedem | that reminds me, i noticed we don't test with that as False once | 19:41 |
| *** alexpilotti has quit IRC | 19:41 | |
| *** fragatina has joined #openstack-nova | 19:43 | |
| cfriesen | it seems to work okay for raw/qcow2, but apparently it doesn't work with a vmdk format image file. (admittedly a qcow2 image with a vmdk backing file is a bit odd, but we don't do anything to prevent it) | 19:43 |
| *** dtp has joined #openstack-nova | 19:45 | |
| mriedem | cfriesen: did you mean integration test? because we don't even test that with a unit test | 19:45 |
| mriedem | vmdk with libvirt imagebackend? | 19:45 |
| *** satyar has quit IRC | 19:45 | |
| mriedem | vmdk is all vmware | 19:46 |
| mordred | cfriesen: we do vhd uploads in production | 19:46 |
| mordred | but that's not helpful for gate testing :) | 19:47 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025 | 19:47 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add online data migration for populating services.uuid https://review.openstack.org/454899 | 19:47 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 19:48 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Modify PciPassthroughFilter to accept lists https://review.openstack.org/444946 | 19:49 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 19:49 |
| mriedem | cfriesen: you've got a little unit test failure to fix up in https://review.openstack.org/#/c/447055/ | 19:53 |
| *** smatzek has quit IRC | 19:53 | |
| *** markvoelker has quit IRC | 19:56 | |
| *** kaisers has quit IRC | 19:57 | |
| cfriesen | mriedem: okay, will sort it out. I'm putting out customer fires. | 19:59 |
| *** cdent has quit IRC | 20:00 | |
| cfriesen | mriedem: mordred: apparently libvirt/qemu will boot a vmdk-format glance image. | 20:00 |
| mriedem | https://en.wikibooks.org/wiki/QEMU/Images#Image_types | 20:01 |
| mriedem | i don't see ploop in that list | 20:02 |
| mordred | cfriesen: fascinating | 20:02 |
| *** ayogi has quit IRC | 20:02 | |
| cfriesen | I have a report that it boots up, but then fails when we try to live-migrate the instance. | 20:03 |
| cfriesen | I'll raise a bug once we have the info nailed down a bit more securely | 20:03 |
| mriedem | please include customer credit card numbers in the bug report thanks | 20:03 |
| cfriesen | wouldn't that be nice. :) | 20:04 |
| cfriesen | not actually expecting it to get fixed, but at least someone else can see the bug report and know it's broken. :) | 20:04 |
| *** eharney has quit IRC | 20:04 | |
| jaypipes | wait, mriedem, your customers pay you? | 20:07 |
| mriedem | i don't have customers | 20:07 |
| *** Jack_Iv has quit IRC | 20:08 | |
| *** Jack_Iv has joined #openstack-nova | 20:08 | |
| *** Jack_Iv has quit IRC | 20:08 | |
| *** Jack_Iv has joined #openstack-nova | 20:08 | |
| d9k | is there any code for different components to share locks?, i dont see anything | 20:08 |
| d9k | also oslo, which uses fasteners does not have any code for shared locks either | 20:09 |
| *** Jack_Iv has quit IRC | 20:09 | |
| *** Jack_Iv has joined #openstack-nova | 20:10 | |
| d9k | i posted earlier about a race nova and neutron introduce with iptables and ipsets, both need to acquire shared locks from each other | 20:10 |
| sdague | d9k: there is no mechanism for that today, is there an open bug on that issue? | 20:11 |
| *** xyang1 has joined #openstack-nova | 20:13 | |
| *** Jack_Iv has quit IRC | 20:14 | |
| d9k | no not yet, acquiring neutron-ipset lock in the iptables-save/restore bit fixes https://ptpb.pw/16S8, but https://ptpb.pw/aukQ needs locks too | 20:16 |
| *** eharney has joined #openstack-nova | 20:18 | |
| mriedem | http://forumtopics.openstack.org/cfp/details/11 | 20:19 |
| d9k | also this happens fairly often when running tempest compute.servers | 20:21 |
| *** kaisers has joined #openstack-nova | 20:22 | |
| *** kaisers has quit IRC | 20:24 | |
| *** kaisers has joined #openstack-nova | 20:24 | |
| *** armax has quit IRC | 20:26 | |
| *** READ10 has joined #openstack-nova | 20:26 | |
| *** tbachman has joined #openstack-nova | 20:27 | |
| *** lucasxu has quit IRC | 20:27 | |
| *** armax has joined #openstack-nova | 20:27 | |
| *** gouthamr has quit IRC | 20:28 | |
| sdague | d9k: a bug with details would be good, sounds like you've done a lot of the debugging, and it would be great to get all those details down | 20:30 |
| *** lucasxu has joined #openstack-nova | 20:30 | |
| *** lucasxu has quit IRC | 20:34 | |
| *** dimtruck is now known as zz_dimtruck | 20:35 | |
| *** lucasxu has joined #openstack-nova | 20:35 | |
| swebster | Has anybody run the latest py27 unit tests on a Centos system? I'm getting a failure on the nova.tests.unit.objects.test_objects.TestObjectVersions.test_versions test which seems to be related to the recent "Use cursive for signature verification" change: https://review.openstack.org/#/c/351232/. The test passes on my Ubuntu Xenial box, with cursive versions both at 0.1.2. | 20:38 |
| *** crushil has quit IRC | 20:44 | |
| mriedem | swebster: it's likely something in the cursive library, what is the actual error? (in a paste) | 20:44 |
| *** karimb has quit IRC | 20:45 | |
| swebster | mriedem: Here's a paste of the error(s) seen https://pastebin.com/1rqJaTPs | 20:46 |
| *** jpena|off has quit IRC | 20:48 | |
| *** dmellado has quit IRC | 20:48 | |
| mriedem | my guess would be something with this https://github.com/openstack/cursive/blob/master/cursive/signature_utils.py#L177 | 20:48 |
| mriedem | that changes what's in REGISTERED_TYPES | 20:49 |
| mriedem | which changes our enum list in nova | 20:49 |
| *** ltomasbo|away has quit IRC | 20:50 | |
| mriedem | this https://github.com/openstack/nova/blob/master/nova/objects/fields.py#L430 | 20:50 |
| mriedem | swebster: to find out, import cursive 0.1.2 in centos and xenial and compare signature_utils.SignatureKeyType.REGISTERED_TYPES.keys() | 20:51 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Deprecate multinic, os-virtual-interfaces, and floating IP action APIs https://review.openstack.org/384261 | 20:54 |
| mriedem | sdague: jaypipes: mordred: alex_xu: johnthetubaguy: ^ there you go | 20:54 |
| mriedem | big bag of deprecation | 20:54 |
| jaypipes | mriedem: cool. | 20:55 |
| swebster | mriedem: Very right. Centos: ['RSA-PSS', 'ECC_SECP521R1', 'ECC_SECP384R1', 'DSA']. Ubuntu: ['RSA-PSS', 'ECC_SECP521R1', 'ECC_SECT409R1', 'ECC_SECT409K1', 'ECC_SECT571R1', 'ECC_SECP384R1', 'ECC_SECT571K1', 'DSA'] | 20:55 |
| *** armax has quit IRC | 20:56 | |
| mriedem | swebster: that's super fun | 20:56 |
| *** dharinic has quit IRC | 20:56 | |
| mriedem | they used to be hardcoded https://review.openstack.org/#/c/351232/16/nova/objects/fields.py | 20:56 |
| mriedem | before we were using cursive | 20:56 |
| mriedem | swebster: please report a bug | 20:57 |
| mriedem | https://bugs.launchpad.net/nova/ | 20:57 |
| swebster | mriedem: And now it looks like they could be different system to system. Will do, and thanks | 20:57 |
| *** dharinic has joined #openstack-nova | 20:58 | |
| mriedem | yeah that sucks, and i was afraid of something like that happening | 20:58 |
| mriedem | we should probably just hard-code the superset | 20:58 |
| *** ssurana has joined #openstack-nova | 20:59 | |
| *** xyang1 has quit IRC | 21:00 | |
| *** xyang1 has joined #openstack-nova | 21:00 | |
| *** rfolco has quit IRC | 21:01 | |
| mriedem | melwitt: so you hit "sphinx.errors.SphinxWarning: WARNING: no "man_pages" config value found; no manual pages will be written" right? | 21:01 |
| mriedem | did you just recheck it? | 21:01 |
| melwitt | mriedem: cdent rechecked it and it failed again | 21:02 |
| *** zz_dimtruck is now known as dimtruck | 21:02 | |
| mriedem | well that's just great | 21:03 |
| mriedem | i'm going to go eat my feelings | 21:03 |
| *** karimb has joined #openstack-nova | 21:05 | |
| *** thorst has quit IRC | 21:07 | |
| *** cleong has quit IRC | 21:08 | |
| *** armax has joined #openstack-nova | 21:10 | |
| *** lucasxu has quit IRC | 21:12 | |
| dansmith | last time I tasted feelings they had a distinct "like butt" quality | 21:13 |
| *** gouthamr has joined #openstack-nova | 21:13 | |
| dansmith | but, I'm not very adventurous with my mouth | 21:13 |
| dansmith | er, tastes | 21:13 |
| *** jdurgin has quit IRC | 21:14 | |
| *** lucasxu has joined #openstack-nova | 21:16 | |
| *** eharney has quit IRC | 21:19 | |
| *** catintheroof has joined #openstack-nova | 21:19 | |
| *** esberglu has quit IRC | 21:19 | |
| *** esberglu has joined #openstack-nova | 21:20 | |
| *** catintheroof has quit IRC | 21:21 | |
| *** Apoorva_ has quit IRC | 21:21 | |
| *** catintheroof has joined #openstack-nova | 21:21 | |
| *** Apoorva has joined #openstack-nova | 21:22 | |
| *** vladikr has quit IRC | 21:22 | |
| *** abalutoiu_ has joined #openstack-nova | 21:23 | |
| *** esberglu has quit IRC | 21:24 | |
| mriedem | these feelings tasted like pretzels | 21:25 |
| *** abalutoiu__ has quit IRC | 21:27 | |
| *** READ10 has quit IRC | 21:30 | |
| *** alexpilotti has joined #openstack-nova | 21:30 | |
| *** thorst has joined #openstack-nova | 21:30 | |
| *** edmondsw has quit IRC | 21:34 | |
| *** thorst has quit IRC | 21:34 | |
| *** edmondsw has joined #openstack-nova | 21:35 | |
| *** esberglu has joined #openstack-nova | 21:35 | |
| mriedem | ok, on to writing a spec to deprecate os-hosts now?! | 21:36 |
| *** kaisers has quit IRC | 21:37 | |
| *** edmondsw has quit IRC | 21:39 | |
| *** ociuhandu has quit IRC | 21:43 | |
| *** jdurgin has joined #openstack-nova | 21:44 | |
| dansmith | mriedem: uuid should be nullable on the model but not on the object right? https://review.openstack.org/#/c/453025/6/nova/objects/service.py | 21:48 |
| dansmith | AFAICT, there's no need for it to actually be None anywhere | 21:49 |
| dansmith | but if it's nullable everything needs to be defensive | 21:49 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025 | 21:52 |
| mriedem | yes | 21:52 |
| mriedem | you're right | 21:52 |
| mriedem | compute nodes uuid is read-only https://github.com/openstack/nova/blob/master/nova/objects/compute_node.py#L55 | 21:52 |
| mriedem | but not nullable | 21:52 |
| mriedem | aggregate uuid is nullable=False, but not read-only https://github.com/openstack/nova/blob/master/nova/objects/aggregate.py#L244 | 21:53 |
| *** esberglu has quit IRC | 21:53 | |
| *** ociuhandu has joined #openstack-nova | 21:53 | |
| dansmith | yeah | 21:53 |
| dansmith | there was a reason for the agg one not being read-only | 21:53 |
| mriedem | so should be nullable=False and read_only=True? | 21:53 |
| dansmith | tbh, just make it nullable=False (i.e. don't specify it) | 21:54 |
| dansmith | the readonly has been problematic recently, | 21:54 |
| mriedem | ok | 21:54 |
| dansmith | so I'd leave that off, which is why it is in aggregate | 21:54 |
| * dansmith runs off for a bit | 21:55 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598 | 21:57 |
| efried | mriedem ^^ back atcha. | 21:57 |
| efried | Still a couple uncertainties inline. | 21:58 |
| *** lucasxu has quit IRC | 21:58 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025 | 21:59 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add online data migration for populating services.uuid https://review.openstack.org/454899 | 21:59 |
| *** jaypipes has quit IRC | 22:02 | |
| *** edmondsw has joined #openstack-nova | 22:04 | |
| *** mdrabe has quit IRC | 22:04 | |
| *** edmondsw has quit IRC | 22:08 | |
| *** tbachman has quit IRC | 22:12 | |
| *** burt has quit IRC | 22:14 | |
| *** dgonzalez has quit IRC | 22:24 | |
| *** xyang1 has quit IRC | 22:26 | |
| *** dgonzalez has joined #openstack-nova | 22:28 | |
| *** ociuhandu has quit IRC | 22:29 | |
| *** mtanino has quit IRC | 22:30 | |
| *** ociuhandu has joined #openstack-nova | 22:31 | |
| *** Guest89312 has quit IRC | 22:35 | |
| mriedem | mgagne: http://lists.openstack.org/pipermail/openstack-dev/2017-April/115292.html | 22:36 |
| mriedem | browne: efried: BobBall: would be good to check out https://review.openstack.org/#/c/453272/ and see if vmware, powervm and xen drivers can support this API eventually | 22:37 |
| mgagne | mriedem: Thanks for starting the conversation and doing proper follow up | 22:37 |
| mriedem | claudiub: you too but i think hyperv already does | 22:37 |
| *** kaisers has joined #openstack-nova | 22:37 | |
| mriedem | mgagne: does the cinder extend volume api have a confirm / reject state like nova resize does? | 22:38 |
| *** r-daneel has joined #openstack-nova | 22:38 | |
| mgagne | mriedem: cinder doesn't have this confirmation system | 22:38 |
| mgagne | mriedem: as you mention, there are pending questions: capabilities from virt driver. But also error handling in Nova since event system is async, Cinder can't query back for status. | 22:39 |
| mriedem | mgagne: the nova api could fail fast in one case, | 22:39 |
| mriedem | that the computes aren't new enough | 22:39 |
| claudiub | mriedem: yeah, that's something i was going to comment on. | 22:40 |
| *** felipemonteiro has quit IRC | 22:40 | |
| mriedem | i.e. if we know the instance is running on an ocata compute, we know it's going to fail, and we could fail the request to cinder with a 409 or something | 22:40 |
| mgagne | mriedem: this means a sync call to compute node? or could capabilities be cached somewhere in database? | 22:40 |
| claudiub | it seems that it only talks about iscsi volumes. | 22:40 |
| mriedem | mgagne: we have the nova-compute service version in the database | 22:40 |
| mriedem | mgagne: so we know if the compute host is running ocata or pike code, | 22:40 |
| claudiub | while for hyper-v we support iscsi, smb, and fibre channel volumes. | 22:40 |
| mgagne | cool, didn't know about that | 22:40 |
| mriedem | or what the minimum nova-compute is in the deployment | 22:41 |
| claudiub | for smb, afaik, the hyper-v driver might have to do the volume resize. didn't read the whole spec to see if that's possible in nova | 22:41 |
| mgagne | claudiub: yes, the proposed implementation is very "optimist" by assuming only libvirt+iscsi exists in its presumed perfect world =) | 22:41 |
| mriedem | claudiub: yeah good point - mgagne: how do we know which brick connector to get? do we have to get the connection_info dict from cinder for the given volume? | 22:41 |
| mgagne | mriedem: iirc, extend_volume takes connection_info in parameter so I assume it knows that to do with that | 22:42 |
| mgagne | mriedem: I can check, just a bit busy with some openstack upgrade atm =) | 22:42 |
| mriedem | oh, hmm, | 22:43 |
| mriedem | i guess we have the bdm.connection_info | 22:43 |
| mriedem | and can lookup the bdm by instance and volume id | 22:43 |
| mgagne | yes, nova has it afaik in bdm | 22:43 |
| *** dixiaoli has joined #openstack-nova | 22:43 | |
| *** al498u has quit IRC | 22:43 | |
| claudiub | mriedem: anyways. since there's blueprints talk. i was going to propose a sriov + hyperv spec to nova. i finally have some hardware to work / test on. :D it would be a pretty straight-forward implementation IMO, especially since we already support pci passthrough in hyper-v. do I need a spec for that? | 22:46 |
| mriedem | claudiub: i thought you had a spec for that at one point? | 22:47 |
| mriedem | maybe it was just a blueprint and a patch that was abandoned? | 22:47 |
| mriedem | claudiub: i think i'm fine with a specless feature parity blueprint though | 22:47 |
| claudiub | i don't think I had, never had the hardware to test that feature before. | 22:47 |
| claudiub | cool, will submit the blueprint and some POC code by the end of the week. | 22:48 |
| *** r-daneel has quit IRC | 22:59 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272 | 23:03 |
| *** chyka has quit IRC | 23:05 | |
| *** chyka has joined #openstack-nova | 23:05 | |
| *** lyan has quit IRC | 23:06 | |
| *** sdague has quit IRC | 23:09 | |
| *** tbachman has joined #openstack-nova | 23:10 | |
| *** annegentle has quit IRC | 23:12 | |
| *** artom has quit IRC | 23:14 | |
| *** artom has joined #openstack-nova | 23:14 | |
| *** erlon has quit IRC | 23:15 | |
| *** artom has quit IRC | 23:16 | |
| *** artom has joined #openstack-nova | 23:16 | |
| *** Apoorva_ has joined #openstack-nova | 23:16 | |
| *** Apoorva has quit IRC | 23:20 | |
| *** thorst has joined #openstack-nova | 23:20 | |
| *** baoli has quit IRC | 23:21 | |
| *** thorst has quit IRC | 23:23 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (1/4) https://review.openstack.org/455563 | 23:24 |
| *** claudiub has quit IRC | 23:25 | |
| *** catintheroof has quit IRC | 23:29 | |
| *** catintheroof has joined #openstack-nova | 23:30 | |
| *** catintheroof has quit IRC | 23:30 | |
| *** chyka has quit IRC | 23:30 | |
| *** dixiaoli has quit IRC | 23:36 | |
| *** mlavalle has quit IRC | 23:38 | |
| *** crushil has joined #openstack-nova | 23:38 | |
| *** jamielennox is now known as jamielennox|away | 23:48 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (2/4) https://review.openstack.org/455568 | 23:48 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (3/4) https://review.openstack.org/455570 | 23:48 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (4/4) https://review.openstack.org/455573 | 23:48 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers-actions.inc https://review.openstack.org/454565 | 23:48 |
| *** baoli has joined #openstack-nova | 23:49 | |
| *** baoli has quit IRC | 23:49 | |
| *** baoli has joined #openstack-nova | 23:50 | |
| *** jamielennox|away is now known as jamielennox | 23:51 | |
| *** thorst has joined #openstack-nova | 23:53 | |
| *** hongbin has quit IRC | 23:54 | |
| *** takashin has joined #openstack-nova | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!