| *** suresh12 has quit IRC | 00:01 | |
| *** kaisers has quit IRC | 00:03 | |
| *** suresh12 has joined #openstack-nova | 00:03 | |
| *** sneti_ has quit IRC | 00:07 | |
| *** gouthamr has joined #openstack-nova | 00:07 | |
| *** sneti_ has joined #openstack-nova | 00:07 | |
| *** sneti_ has quit IRC | 00:07 | |
| *** sneti_ has joined #openstack-nova | 00:08 | |
| *** karimb has quit IRC | 00:08 | |
| *** sneti_ has quit IRC | 00:08 | |
| *** crushil has quit IRC | 00:11 | |
| *** hongbin has quit IRC | 00:11 | |
| *** suresh12 has quit IRC | 00:14 | |
| *** suresh12 has joined #openstack-nova | 00:21 | |
| *** catintheroof has joined #openstack-nova | 00:25 | |
| *** wangqwsh has joined #openstack-nova | 00:28 | |
| *** chyka has joined #openstack-nova | 00:29 | |
| *** kaisers has joined #openstack-nova | 00:29 | |
| *** thorst has quit IRC | 00:31 | |
| *** thorst has joined #openstack-nova | 00:31 | |
| *** suresh12 has quit IRC | 00:33 | |
| *** suresh12 has joined #openstack-nova | 00:33 | |
| *** chyka has quit IRC | 00:33 | |
| *** thorst has quit IRC | 00:35 | |
| *** Fdaisuke has quit IRC | 00:36 | |
| *** tjones has quit IRC | 00:37 | |
| *** joanna__ has quit IRC | 00:37 | |
| *** suresh12 has quit IRC | 00:39 | |
| *** suresh12 has joined #openstack-nova | 00:40 | |
| *** timello has joined #openstack-nova | 00:40 | |
| *** adrian_otto has joined #openstack-nova | 00:41 | |
| *** timello has quit IRC | 00:43 | |
| *** esberglu has quit IRC | 00:44 | |
| *** timello has joined #openstack-nova | 00:44 | |
| *** artom__ has joined #openstack-nova | 00:44 | |
| *** suresh12 has quit IRC | 00:44 | |
| *** esberglu has joined #openstack-nova | 00:44 | |
| *** artom__ has quit IRC | 00:46 | |
| *** artom__ has joined #openstack-nova | 00:46 | |
| *** suresh12 has joined #openstack-nova | 00:46 | |
| *** tovin07_ has joined #openstack-nova | 00:48 | |
| *** artom_ has quit IRC | 00:48 | |
| *** esberglu has quit IRC | 00:49 | |
| *** timello has quit IRC | 00:49 | |
| *** timello has joined #openstack-nova | 00:49 | |
| *** timello has quit IRC | 00:50 | |
| *** timello has joined #openstack-nova | 00:51 | |
| *** esberglu has joined #openstack-nova | 00:51 | |
| *** esberglu has quit IRC | 00:52 | |
| *** esberglu has joined #openstack-nova | 00:52 | |
| *** suresh12 has quit IRC | 00:55 | |
| *** timello has quit IRC | 00:56 | |
| *** esberglu has quit IRC | 00:56 | |
| *** timello has joined #openstack-nova | 00:56 | |
| *** nic has quit IRC | 00:58 | |
| *** kaisers has quit IRC | 00:58 | |
| *** dtp has quit IRC | 01:01 | |
| *** catintheroof has quit IRC | 01:02 | |
| *** rfolco has joined #openstack-nova | 01:04 | |
| *** rfolco has quit IRC | 01:04 | |
| *** sridharg has joined #openstack-nova | 01:04 | |
| *** adrian_otto has quit IRC | 01:04 | |
| *** phuongnh has joined #openstack-nova | 01:04 | |
| *** thorst has joined #openstack-nova | 01:07 | |
| *** NikhilS has joined #openstack-nova | 01:07 | |
| *** wangqwsh has quit IRC | 01:08 | |
| *** jbernard has quit IRC | 01:12 | |
| *** jbernard has joined #openstack-nova | 01:13 | |
| *** thorst has quit IRC | 01:15 | |
| *** dave-mccowan has quit IRC | 01:17 | |
| *** mriedem has left #openstack-nova | 01:18 | |
| *** suresh12 has joined #openstack-nova | 01:18 | |
| *** suresh12 has quit IRC | 01:41 | |
| *** Apoorva_ has joined #openstack-nova | 01:41 | |
| *** crushil has joined #openstack-nova | 01:41 | |
| *** Apoorva has quit IRC | 01:44 | |
| *** Apoorva_ has quit IRC | 01:46 | |
| *** kaisers has joined #openstack-nova | 01:48 | |
| *** jdillaman has quit IRC | 01:49 | |
| *** iceyao has joined #openstack-nova | 01:50 | |
| *** crushil has quit IRC | 02:01 | |
| *** jdillaman has joined #openstack-nova | 02:05 | |
| *** harlowja has quit IRC | 02:06 | |
| *** huyang_ has joined #openstack-nova | 02:08 | |
| *** bkopilov has quit IRC | 02:09 | |
| *** adrian_otto has joined #openstack-nova | 02:10 | |
| *** Sukhdev_ has quit IRC | 02:10 | |
| *** takashin has joined #openstack-nova | 02:12 | |
| *** thorst has joined #openstack-nova | 02:15 | |
| *** jdillaman has quit IRC | 02:19 | |
| *** adrian_otto has quit IRC | 02:23 | |
| *** diga has quit IRC | 02:25 | |
| *** adrian_otto has joined #openstack-nova | 02:25 | |
| *** timello has quit IRC | 02:26 | |
| *** panbalag has joined #openstack-nova | 02:28 | |
| *** huyang_ has quit IRC | 02:28 | |
| *** thorst has quit IRC | 02:35 | |
| *** hongbin has joined #openstack-nova | 02:36 | |
| *** kaisers has quit IRC | 02:41 | |
| *** tuanluong has joined #openstack-nova | 02:44 | |
| *** Shunli has joined #openstack-nova | 02:46 | |
| *** adrian_otto has quit IRC | 02:47 | |
| *** harlowja has joined #openstack-nova | 02:50 | |
| *** panbalag has left #openstack-nova | 02:50 | |
| *** jwcroppe has quit IRC | 02:50 | |
| *** jwcroppe has joined #openstack-nova | 02:51 | |
| *** kaisers has joined #openstack-nova | 02:54 | |
| *** panbalag has joined #openstack-nova | 02:55 | |
| *** nicolasbock has quit IRC | 02:55 | |
| *** panbalag has left #openstack-nova | 02:56 | |
| *** crushil has joined #openstack-nova | 02:58 | |
| *** panbalag has joined #openstack-nova | 02:58 | |
| *** panbalag has left #openstack-nova | 02:59 | |
| *** tbachman_ has quit IRC | 03:06 | |
| *** links has joined #openstack-nova | 03:08 | |
| *** jwcroppe_ has joined #openstack-nova | 03:11 | |
| *** jwcroppe has quit IRC | 03:13 | |
| *** lyan has joined #openstack-nova | 03:15 | |
| *** f10d4 has quit IRC | 03:18 | |
| *** f10d4 has joined #openstack-nova | 03:18 | |
| *** Shunli has quit IRC | 03:20 | |
| *** lyan has quit IRC | 03:31 | |
| *** amotoki has joined #openstack-nova | 03:34 | |
| *** bkopilov has joined #openstack-nova | 03:39 | |
| *** dtp has joined #openstack-nova | 03:45 | |
| *** amotoki has quit IRC | 03:50 | |
| *** trinaths has joined #openstack-nova | 03:51 | |
| *** hongbin has quit IRC | 03:53 | |
| *** hongbin has joined #openstack-nova | 03:53 | |
| *** harlowja has quit IRC | 03:56 | |
| *** adrian_otto has joined #openstack-nova | 03:57 | |
| *** gouthamr has quit IRC | 03:57 | |
| *** gouthamr has joined #openstack-nova | 03:58 | |
| *** Sukhdev has joined #openstack-nova | 03:59 | |
| *** amotoki has joined #openstack-nova | 04:01 | |
| *** armax has joined #openstack-nova | 04:05 | |
| *** sree has joined #openstack-nova | 04:07 | |
| *** hongbin has quit IRC | 04:07 | |
| *** harlowja has joined #openstack-nova | 04:07 | |
| *** gouthamr has quit IRC | 04:09 | |
| *** nmathew has joined #openstack-nova | 04:18 | |
| *** suresh12 has joined #openstack-nova | 04:19 | |
| *** amotoki has quit IRC | 04:19 | |
| *** vks1 has joined #openstack-nova | 04:21 | |
| *** esberglu has joined #openstack-nova | 04:21 | |
| *** suresh12 has quit IRC | 04:24 | |
| *** psachin has joined #openstack-nova | 04:24 | |
| *** esberglu has quit IRC | 04:26 | |
| *** amotoki has joined #openstack-nova | 04:28 | |
| *** thorst has joined #openstack-nova | 04:32 | |
| *** harlowja has quit IRC | 04:33 | |
| *** yamahata has joined #openstack-nova | 04:34 | |
| *** thorst has quit IRC | 04:37 | |
| *** bmace has quit IRC | 04:45 | |
| *** bmace has joined #openstack-nova | 04:45 | |
| *** rcernin has joined #openstack-nova | 04:45 | |
| *** dimtruck is now known as zz_dimtruck | 04:48 | |
| vks1 | ConfigFileValueError: Value for option instances is not valid: invalid literal for int() with base 10 . Getting this error. I have set nova.conf like this: | 04:48 |
|---|---|---|
| vks1 | [quota] | 04:48 |
| vks1 | instances=-1 | 04:48 |
| vks1 | #cores=50 | 04:48 |
| vks1 | ram=-1 | 04:48 |
| vks1 | gigabytes=-1 | 04:48 |
| *** Jack_Iv has joined #openstack-nova | 04:56 | |
| *** suresh12 has joined #openstack-nova | 04:57 | |
| *** Jack_Iv_ has joined #openstack-nova | 04:57 | |
| *** adrian_otto has quit IRC | 05:00 | |
| *** Jack_Iv has quit IRC | 05:01 | |
| *** nmathew- has joined #openstack-nova | 05:04 | |
| *** kaisers_ has joined #openstack-nova | 05:05 | |
| *** vishwana_ has joined #openstack-nova | 05:05 | |
| *** toabctl has quit IRC | 05:06 | |
| *** alezil_ has joined #openstack-nova | 05:06 | |
| *** sballe__ has joined #openstack-nova | 05:07 | |
| *** khappone_ has joined #openstack-nova | 05:07 | |
| *** betherly_ has joined #openstack-nova | 05:07 | |
| *** ratailor has joined #openstack-nova | 05:09 | |
| *** Jack_Iv_ has quit IRC | 05:09 | |
| *** tuanluong_ has joined #openstack-nova | 05:09 | |
| *** rcernin has quit IRC | 05:09 | |
| *** szaher_ has joined #openstack-nova | 05:10 | |
| *** guchihiro has joined #openstack-nova | 05:10 | |
| *** toabctl has joined #openstack-nova | 05:10 | |
| *** Jack_Iv has joined #openstack-nova | 05:10 | |
| *** rcernin has joined #openstack-nova | 05:11 | |
| *** harlowja has joined #openstack-nova | 05:11 | |
| *** crushil has quit IRC | 05:12 | |
| *** rodrigod` has joined #openstack-nova | 05:12 | |
| *** McNinja_ has joined #openstack-nova | 05:12 | |
| *** owalsh_ has joined #openstack-nova | 05:12 | |
| *** MasterofJOKers_ has joined #openstack-nova | 05:13 | |
| *** patriciadomin_ has joined #openstack-nova | 05:13 | |
| *** NightKhaos_ has joined #openstack-nova | 05:13 | |
| *** nmathew has quit IRC | 05:14 | |
| *** f10d4 has quit IRC | 05:14 | |
| *** kaisers has quit IRC | 05:14 | |
| *** tuanluong has quit IRC | 05:14 | |
| *** sridharg has quit IRC | 05:14 | |
| *** szaher has quit IRC | 05:14 | |
| *** Dinesh_Bhor has quit IRC | 05:14 | |
| *** edleafe has quit IRC | 05:14 | |
| *** vishwanathj has quit IRC | 05:14 | |
| *** owalsh has quit IRC | 05:14 | |
| *** khappone has quit IRC | 05:14 | |
| *** betherly has quit IRC | 05:14 | |
| *** rodrigods has quit IRC | 05:14 | |
| *** purplerbot has quit IRC | 05:14 | |
| *** MasterofJOKers has quit IRC | 05:14 | |
| *** McNinja has quit IRC | 05:14 | |
| *** zhenguo has quit IRC | 05:14 | |
| *** patriciadomin has quit IRC | 05:14 | |
| *** NightKhaos has quit IRC | 05:14 | |
| *** thomasem has quit IRC | 05:14 | |
| *** alezil has quit IRC | 05:14 | |
| *** sballe_ has quit IRC | 05:14 | |
| *** MasterofJOKers_ is now known as MasterofJOKers | 05:14 | |
| *** McNinja_ is now known as McNinja | 05:14 | |
| *** NightKhaos_ is now known as NightKhaos | 05:14 | |
| *** edleafe has joined #openstack-nova | 05:15 | |
| *** adisky_ has joined #openstack-nova | 05:15 | |
| *** jamielennox is now known as jamielennox|away | 05:15 | |
| *** Jack_Iv has quit IRC | 05:15 | |
| *** alezil_ is now known as alezil | 05:15 | |
| *** thomasem has joined #openstack-nova | 05:15 | |
| *** diga has joined #openstack-nova | 05:15 | |
| *** esberglu has joined #openstack-nova | 05:16 | |
| *** betherly_ is now known as betherly | 05:16 | |
| *** sballe__ is now known as sballe_ | 05:16 | |
| *** f10d4 has joined #openstack-nova | 05:16 | |
| *** udesale has joined #openstack-nova | 05:17 | |
| *** esberglu has quit IRC | 05:20 | |
| *** sridharg has joined #openstack-nova | 05:21 | |
| *** Dinesh_Bhor has joined #openstack-nova | 05:21 | |
| *** amotoki has quit IRC | 05:26 | |
| *** prateek has joined #openstack-nova | 05:26 | |
| *** Jack_Iv has joined #openstack-nova | 05:29 | |
| *** dtp has quit IRC | 05:29 | |
| *** amotoki has joined #openstack-nova | 05:30 | |
| *** mdnadeem has joined #openstack-nova | 05:31 | |
| *** raj_singh has quit IRC | 05:31 | |
| *** raj_singh_ has joined #openstack-nova | 05:31 | |
| *** thorst has joined #openstack-nova | 05:33 | |
| *** madgoat has joined #openstack-nova | 05:37 | |
| *** Sukhdev has quit IRC | 05:37 | |
| *** madgoat has left #openstack-nova | 05:37 | |
| *** sree has quit IRC | 05:37 | |
| *** jamielennox|away is now known as jamielennox | 05:37 | |
| *** thorst has quit IRC | 05:38 | |
| *** sree has joined #openstack-nova | 05:38 | |
| *** tuanluong_ has quit IRC | 05:38 | |
| *** nkrinner_afk is now known as nkrinner | 05:39 | |
| *** Jack_Iv_ has joined #openstack-nova | 05:39 | |
| *** lpetrut has joined #openstack-nova | 05:39 | |
| *** lpetrut has quit IRC | 05:40 | |
| *** udesale__ has joined #openstack-nova | 05:42 | |
| *** sree has quit IRC | 05:42 | |
| *** udesale has quit IRC | 05:43 | |
| *** Jack_Iv has quit IRC | 05:43 | |
| *** sridharg has quit IRC | 05:46 | |
| *** mtreinish has quit IRC | 05:49 | |
| *** mtreinish has joined #openstack-nova | 05:50 | |
| *** mtreinish has quit IRC | 05:52 | |
| *** carthaca_ has joined #openstack-nova | 05:56 | |
| *** mtreinish has joined #openstack-nova | 05:56 | |
| *** iceyao has quit IRC | 06:00 | |
| *** iceyao_ has joined #openstack-nova | 06:00 | |
| *** jaosorior_away is now known as jaosorior | 06:01 | |
| *** harlowja has quit IRC | 06:02 | |
| *** andreas_s has joined #openstack-nova | 06:03 | |
| *** tuanluong has joined #openstack-nova | 06:06 | |
| *** Oku_OS-away is now known as Oku_OS | 06:08 | |
| *** udesale__ has quit IRC | 06:08 | |
| *** esberglu has joined #openstack-nova | 06:10 | |
| *** esberglu has quit IRC | 06:15 | |
| *** voelzmo has joined #openstack-nova | 06:19 | |
| *** voelzmo has quit IRC | 06:24 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: api-ref: fix os-extended-volumes:volumes_attached in servers responses https://review.openstack.org/449809 | 06:25 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: ix api-ref for create servers response https://review.openstack.org/451673 | 06:25 |
| *** tesseract has joined #openstack-nova | 06:26 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix api-ref for create servers response https://review.openstack.org/451673 | 06:27 |
| gmann | oomichi: updated https://review.openstack.org/#/c/451673/ | 06:27 |
| *** voelzmo has joined #openstack-nova | 06:27 | |
| *** iceyao_ has quit IRC | 06:32 | |
| *** thorst has joined #openstack-nova | 06:34 | |
| *** thorst has quit IRC | 06:38 | |
| openstackgerrit | Dan Peschman proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025 | 06:39 |
| *** jpena|away is now known as jpena | 06:39 | |
| *** claudiub has quit IRC | 06:40 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add a removeFloatingIp example https://review.openstack.org/453026 | 06:42 |
| *** suresh12 has quit IRC | 06:44 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237 | 06:45 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Specify keymap on server boot https://review.openstack.org/448830 | 06:45 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | 06:45 |
| *** rmart04 has joined #openstack-nova | 06:52 | |
| *** abhishek_k has quit IRC | 06:54 | |
| *** satyar has joined #openstack-nova | 06:56 | |
| *** rmart04 has quit IRC | 06:57 | |
| *** sridharg has joined #openstack-nova | 06:58 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Add project/user association to placement https://review.openstack.org/453030 | 06:58 |
| *** amotoki has quit IRC | 07:00 | |
| *** iceyao has joined #openstack-nova | 07:02 | |
| *** amotoki has joined #openstack-nova | 07:03 | |
| *** Jack_Iv_ has quit IRC | 07:04 | |
| *** Jack_Iv has joined #openstack-nova | 07:04 | |
| *** Jack_Iv has quit IRC | 07:05 | |
| *** Jack_Iv has joined #openstack-nova | 07:05 | |
| *** Jack_Iv_ has joined #openstack-nova | 07:06 | |
| *** aarefiev_afk is now known as aarefiev | 07:07 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova-specs master: TinyFix: Changed type of CPU ID from 'String' to 'Integer' in diagnostics spec https://review.openstack.org/445855 | 07:09 |
| *** Jack_Iv has quit IRC | 07:10 | |
| *** phuongnh has quit IRC | 07:11 | |
| *** tuanluong has quit IRC | 07:11 | |
| snikitin_ | johnthetubaguy, jaypipes: Hi guys! You approved my little fix in diagnostics spec but I had to reupload it because of pep8 error. Could you please fast reapprove it? https://review.openstack.org/#/c/445855 | 07:13 |
| *** zhenguo has joined #openstack-nova | 07:15 | |
| *** Jack_Iv has joined #openstack-nova | 07:16 | |
| *** markus_z has joined #openstack-nova | 07:17 | |
| openstackgerrit | Sihan Wang proposed openstack/nova master: Show quota detail when inject file quota exceeds https://review.openstack.org/453040 | 07:17 |
| *** Jack_Iv_ has quit IRC | 07:18 | |
| *** pcaruana has joined #openstack-nova | 07:19 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Convert consoles code to use objects framework https://review.openstack.org/453041 | 07:23 |
| *** avolkov has joined #openstack-nova | 07:26 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add api sample tests for security group rules https://review.openstack.org/452055 | 07:27 |
| *** udesale has joined #openstack-nova | 07:27 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add api sample tests for security group rules https://review.openstack.org/452055 | 07:28 |
| *** prateek_ has joined #openstack-nova | 07:31 | |
| *** prateek_ has quit IRC | 07:32 | |
| *** prateek_ has joined #openstack-nova | 07:32 | |
| *** prateek has quit IRC | 07:33 | |
| *** thorst has joined #openstack-nova | 07:34 | |
| *** amotoki has quit IRC | 07:35 | |
| *** karimb has joined #openstack-nova | 07:36 | |
| *** amotoki has joined #openstack-nova | 07:36 | |
| *** shaner has quit IRC | 07:38 | |
| *** takashin has quit IRC | 07:38 | |
| *** shaner has joined #openstack-nova | 07:39 | |
| *** thorst has quit IRC | 07:39 | |
| *** Jack_Iv has quit IRC | 07:44 | |
| *** armax has quit IRC | 07:44 | |
| *** lpetrut has joined #openstack-nova | 07:45 | |
| *** jamielennox is now known as jamielennox|away | 07:45 | |
| *** udesale__ has joined #openstack-nova | 07:48 | |
| *** prateek has joined #openstack-nova | 07:49 | |
| *** prateek_ has quit IRC | 07:49 | |
| *** udesale has quit IRC | 07:50 | |
| *** amoralej|off is now known as amoralej | 07:53 | |
| *** yamahata has quit IRC | 07:58 | |
| *** lucas-afk is now known as lucasagomes | 07:59 | |
| *** esberglu has joined #openstack-nova | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** ltomasbo|away is now known as ltomasbo | 08:01 | |
| *** lpetrut has quit IRC | 08:01 | |
| *** esberglu has quit IRC | 08:04 | |
| *** nmathew has joined #openstack-nova | 08:11 | |
| *** nmathew- has quit IRC | 08:12 | |
| *** ralonsoh has joined #openstack-nova | 08:14 | |
| lyarwood | mdbooth: https://review.openstack.org/#/c/452938/ - hackaround for an encrypted nfs volume corruption bug if you get time today | 08:22 |
| *** prateek_ has joined #openstack-nova | 08:27 | |
| *** Jack_Iv has joined #openstack-nova | 08:28 | |
| *** Jack_Iv has quit IRC | 08:28 | |
| * johnthetubaguy wishes everyone a happy specs review day | 08:29 | |
| johnthetubaguy | sfinucan: do you have context on this, I have some dumb questions: https://review.openstack.org/#/c/361140 | 08:30 |
| *** prateek has quit IRC | 08:30 | |
| *** prateek_ has quit IRC | 08:31 | |
| sfinucan | johnthetubaguy: looking | 08:33 |
| sfinucan | johnthetubaguy: Sure do. What's up? | 08:33 |
| johnthetubaguy | sfinucan: so when I read the problem, I assumed I would see the reverse solution, let me try sketch that out | 08:34 |
| johnthetubaguy | sfinucan: PCI whitelist tells Nova to only prefer NUMA affinity, because all the PCI devices are only on one NUMA node | 08:35 |
| johnthetubaguy | that way the user doesn't need to worry themselves | 08:35 |
| *** thorst has joined #openstack-nova | 08:35 | |
| johnthetubaguy | sfinucan: I am guessing there is a use case that doesn't work for, but I am not seeing it myself | 08:36 |
| sfinucan | johnthetubaguy: Is that a statement or a question? :) | 08:36 |
| sfinucan | the PCI whitelist bit, that is | 08:36 |
| *** guchihiro has quit IRC | 08:37 | |
| johnthetubaguy | sfinucan: bit of both really, lol | 08:37 |
| johnthetubaguy | snikitin_ I think this is your spec I am look at, if you are about ^ | 08:37 |
| johnthetubaguy | sfinucan: also, I wasn't sure why an image property was a good idea for this one, anyways I left some comments | 08:38 |
| sfinucan | Hmm, so I don't get your question, but I can give you my tl;dr on that spec, if it would help | 08:38 |
| *** derekh has joined #openstack-nova | 08:40 | |
| sfinucan | In short, some people want to be able to use PCI devices even if they don't have NUMA affinity | 08:40 |
| *** cdent has joined #openstack-nova | 08:41 | |
| sfinucan | but we can't force that on everyone. The current behaviour is to fail if we can't get NUMA affinitized PCI devices and that behaviour should be retained | 08:41 |
| *** purplerbot has joined #openstack-nova | 08:42 | |
| sfinucan | johnthetubaguy: Hence, the need for configurable policies. I _think_ what you're saying is why not soften the requirements for everyone. That'd be great, until people start complaining about why the performance of their VMs has dropped (by 10-15%, based on data I've seen in the past) :) | 08:43 |
| johnthetubaguy | sfinucan: no, no | 08:43 |
| johnthetubaguy | sfinucan: I mean why configure it on the flavor, when its really a host property | 08:44 |
| *** suresh12 has joined #openstack-nova | 08:45 | |
| *** prateek has joined #openstack-nova | 08:47 | |
| *** suresh12 has quit IRC | 08:49 | |
| sfinucan | johnthetubaguy: Is it though? | 08:49 |
| snikitin_ | johnthetubaguy, it's configured in flavor because it's influence on VM performance. For example you want to boot a high performance VM. You have a lot of hosts with different NUMA nodes and PCI devises. `strict` policy is guarantee you that scheduler will choose host with direct NUMA - PCI affinity. and then strict policy is guarantee you that from many PCI-NUMA combinations on current host nova compute will choose NUMA node and PCI device with | 08:49 |
| snikitin_ | derect affinity | 08:49 |
| johnthetubaguy | snikitin_: yeah, I think the mixed set of hosts is the main reason flavor wins | 08:50 |
| sfinucan | It seems very workload specific. Some workloads will be impacted by the affinitization (or lack thereof) | 08:50 |
| johnthetubaguy | snikitin_: sfinucan: the spec doesn't seems to cover scheduler filter or weights changes, I guess thats true right now, as its all picked on the host? | 08:51 |
| sfinucan | johnthetubaguy: I can't say off the top of my head, but I think snikitin_ has WIP code up that might provide guidance on implementation | 08:51 |
| johnthetubaguy | sfinucan: if you want the host to have SRIOV connections, and its only got NICs on one NUMA node, it feels like a host property | 08:52 |
| johnthetubaguy | sfinucan: it just the spec normally would cover those details, and I don't see it mentioned really | 08:52 |
| *** kevinz has joined #openstack-nova | 08:52 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova master: Added PIC NUMA policies https://review.openstack.org/390520 | 08:52 |
| snikitin_ | johnthetubaguy, here is PoC https://review.openstack.org/#/c/390520/ | 08:52 |
| sfinucan | johnthetubaguy: I don't think the spec is focussed on exposing any additional properties of SR-IOV devices. It would only manifest itself with scheduling | 08:53 |
| sfinucan | If we wanted to expose the NUMA affinity etc. globally, resource providers would be the way forward. Different issue though | 08:53 |
| *** thorst has quit IRC | 08:55 | |
| johnthetubaguy | sfinucan: snikitin_: yeah, makes sense it doesn't impact the scheduler as such, because we just don't expose this stuff right now | 08:55 |
| sfinucan | At the moment, scheduling works by identifying hosts with enough free CPUs and SR-IOV devices (no knowledge of NUMA yet) | 08:55 |
| snikitin_ | johnthetubaguy: I don't changed numa_topology filter or weight code because both filter and nova compute use method 'fit_numa_topology' (or similar) to figure out if a VM can be booted on some host or not | 08:55 |
| sfinucan | Once done, it'll try to fit the instance vCPUs onto the host pCPUs. If that's successful, it tries to get a free SR-IOV device and associate that with the guest | 08:56 |
| johnthetubaguy | sfinucan: totally, that all makes sense | 08:56 |
| johnthetubaguy | snikitin_: sfinucan: I wasn't sure why we need the image property as well as the flavor thing, is that just for completness? | 08:57 |
| snikitin_ | yes | 08:57 |
| sfinucan | johnthetubaguy: Sweet. So it's just the very last step that we're touching | 08:57 |
| sfinucan | yup - that was my suggestion | 08:57 |
| *** priteau has joined #openstack-nova | 08:57 | |
| *** carthaca_ has quit IRC | 08:58 | |
| *** carthaca_ has joined #openstack-nova | 08:58 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support VGPU https://review.openstack.org/450122 | 09:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova-specs master: Placement doing claims https://review.openstack.org/437424 | 09:02 |
| *** kevinz has quit IRC | 09:03 | |
| *** haplo37_ has quit IRC | 09:04 | |
| *** haplo37_ has joined #openstack-nova | 09:04 | |
| *** kevinz has joined #openstack-nova | 09:04 | |
| mdbooth | johnthetubaguy: Is there a master list of priority specs? | 09:04 |
| johnthetubaguy | mdbooth: not sure beyond what is in here: https://etherpad.openstack.org/p/pike-nova-priorities-tracking | 09:05 |
| *** slunkad has joined #openstack-nova | 09:05 | |
| mdbooth | johnthetubaguy: I meant specifically for the spec review | 09:06 |
| mdbooth | day | 09:06 |
| johnthetubaguy | not that I know of | 09:06 |
| *** nmathew has quit IRC | 09:08 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support VGPU https://review.openstack.org/450122 | 09:08 |
| mdbooth | johnthetubaguy: Do you have a strategy for finding important specs to review today? | 09:08 |
| johnthetubaguy | currently got a gerrit dash up with all the open reviews, looking for the interesting looking ones, particularly ones with no votes or all positive votes | 09:09 |
| * mdbooth will go with that | 09:09 | |
| *** tuanluong has joined #openstack-nova | 09:12 | |
| *** jschlueter has left #openstack-nova | 09:12 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova master: Added PIC NUMA policies https://review.openstack.org/390520 | 09:13 |
| *** edmondsw has joined #openstack-nova | 09:14 | |
| *** Jack_Iv has joined #openstack-nova | 09:15 | |
| *** Jack_Iv_ has joined #openstack-nova | 09:16 | |
| *** gszasz has joined #openstack-nova | 09:16 | |
| *** ltomasbo is now known as ltomasbo|away | 09:17 | |
| *** edmondsw has quit IRC | 09:18 | |
| *** szaher_ has quit IRC | 09:19 | |
| *** szaher has joined #openstack-nova | 09:19 | |
| *** Jack_Iv has quit IRC | 09:19 | |
| *** ratailor_ has joined #openstack-nova | 09:25 | |
| *** ratailor_ has quit IRC | 09:25 | |
| *** ratailor is now known as ratailor_ | 09:26 | |
| *** ltomasbo|away is now known as ltomasbo | 09:26 | |
| *** ratailor_ is now known as Guest6565 | 09:27 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077 | 09:27 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077 | 09:29 |
| *** mvk has quit IRC | 09:31 | |
| *** karimb_ has joined #openstack-nova | 09:35 | |
| *** nmathew has joined #openstack-nova | 09:38 | |
| *** karimb has quit IRC | 09:39 | |
| *** mnestratov has quit IRC | 09:45 | |
| *** esberglu has joined #openstack-nova | 09:48 | |
| *** thorst has joined #openstack-nova | 09:52 | |
| *** cdent has quit IRC | 09:52 | |
| *** esberglu has quit IRC | 09:52 | |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Switch Nova Quobyte volume driver to mount via systemd-run https://review.openstack.org/432344 | 09:53 |
| *** thorst has quit IRC | 09:56 | |
| *** sree has joined #openstack-nova | 09:58 | |
| *** sree has quit IRC | 09:58 | |
| *** sree has joined #openstack-nova | 09:58 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support VGPU https://review.openstack.org/450122 | 09:59 |
| *** sree has quit IRC | 09:59 | |
| *** abalutoiu__ has joined #openstack-nova | 10:00 | |
| *** sree has joined #openstack-nova | 10:00 | |
| *** mvk has joined #openstack-nova | 10:00 | |
| *** NikhilS has quit IRC | 10:01 | |
| *** NikhilS has joined #openstack-nova | 10:01 | |
| *** NikhilS has quit IRC | 10:02 | |
| *** tbachman has joined #openstack-nova | 10:03 | |
| *** abalutoiu_ has quit IRC | 10:03 | |
| *** jahsis has joined #openstack-nova | 10:04 | |
| *** sree has quit IRC | 10:04 | |
| *** sambetts|afk is now known as sambetts | 10:04 | |
| openstackgerrit | MarkMielke proposed openstack/nova master: Resource tracker should use Block Device Mappings https://review.openstack.org/430091 | 10:05 |
| *** sdague has joined #openstack-nova | 10:08 | |
| *** priteau has quit IRC | 10:11 | |
| *** iceyao has quit IRC | 10:11 | |
| *** iceyao has joined #openstack-nova | 10:13 | |
| *** satyar has quit IRC | 10:14 | |
| *** bhagyashris has left #openstack-nova | 10:14 | |
| *** bhagyashris has joined #openstack-nova | 10:14 | |
| *** avolkov has quit IRC | 10:17 | |
| *** tovin07_ has quit IRC | 10:18 | |
| *** tuanluong has quit IRC | 10:20 | |
| *** iceyao has quit IRC | 10:20 | |
| johnthetubaguy | sfinucan: doing some cleanups, I am guessing you are not still pushing this one forward, or did we merge that somewhere else? https://review.openstack.org/#/c/406081/ | 10:22 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventories https://review.openstack.org/449765 | 10:23 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventories https://review.openstack.org/452708 | 10:23 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventories https://review.openstack.org/452707 | 10:23 |
| sfinucan | johnthetubaguy: So that ties into the PCI spec you were reviewing earlier | 10:23 |
| sfinucan | I thought image metadata was used to allow you to package an application up such that you could drop it into any cloud and it would just work | 10:24 |
| sfinucan | ...assuming the cloud supported the required features | 10:25 |
| johnthetubaguy | sfinucan: just clicked abandon on it as its been left unloved for 4 months, thats all | 10:25 |
| sfinucan | johnthetubaguy: Yeah, that's fair :) I'm just curious what you think image metadata is for | 10:26 |
| sfinucan | The above is why I said the PCI policies spec should have an image metadata aspect. Strict NUMA affinity could be a required attribute for a workload/application | 10:27 |
| johnthetubaguy | sfinucan: its used for a heap of aweful things right now, and we don't define the contract anywhere, so I just hate it really | 10:27 |
| *** lpetrut has joined #openstack-nova | 10:27 | |
| johnthetubaguy | sfinucan: the use I like is specifying things that image requires in order to work correctly | 10:28 |
| *** Jack_Iv_ has quit IRC | 10:29 | |
| johnthetubaguy | sfinucan: but less is more when it comes to APIs, so if its something only the admin wants to control, I prefer not adding image metadata. For people who care we force them to add flavor extra specs now, rather than there being no way to change the defaults, possibly for zero gain. | 10:29 |
| *** priteau has joined #openstack-nova | 10:30 | |
| sfinucan | johnthetubaguy: Right. I think something like CPU thread or PCI policies falls into that category (the image would boot, but the combined performance impact could mean the application is not fit for purpose and therefore "broken") | 10:30 |
| *** Jack_Iv has joined #openstack-nova | 10:30 | |
| sfinucan | Hmm, yeah - that's the other perspective | 10:30 |
| *** udesale__ has quit IRC | 10:30 | |
| sfinucan | Long story short, that spec was applying the above idea to emulator thread policies. I don't care enough about it to pursue it though, heh | 10:31 |
| *** cdent has joined #openstack-nova | 10:31 | |
| *** nicolasbock has joined #openstack-nova | 10:33 | |
| *** Jack_Iv has quit IRC | 10:35 | |
| *** lpetrut has quit IRC | 10:37 | |
| * cdent puts on his spectacles | 10:37 | |
| *** lpetrut has joined #openstack-nova | 10:38 | |
| *** lpetrut has quit IRC | 10:38 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rewrap 'code-review' at ¬79 characters https://review.openstack.org/384960 | 10:39 |
| *** owalsh_ is now known as owalsh | 10:39 | |
| *** tbachman has quit IRC | 10:42 | |
| *** mnestratov has joined #openstack-nova | 10:43 | |
| *** avolkov has joined #openstack-nova | 10:43 | |
| *** Jack_Iv has joined #openstack-nova | 10:47 | |
| *** trinaths has left #openstack-nova | 10:49 | |
| *** diga has quit IRC | 10:53 | |
| *** lucasagomes is now known as lucas-hungry | 10:56 | |
| *** Jack_Iv_ has joined #openstack-nova | 10:59 | |
| *** Jack_Iv_ has quit IRC | 10:59 | |
| *** Jack_Iv_ has joined #openstack-nova | 10:59 | |
| *** Jack_Iv has quit IRC | 11:02 | |
| *** claudiub has joined #openstack-nova | 11:03 | |
| *** jamielennox|away is now known as jamielennox | 11:03 | |
| kashyap | sfinucan: Why rewrap? | 11:07 |
| kashyap | It's at 72 now? | 11:08 |
| sfinucan | kashyap: I think I had a change after it at one point? | 11:08 |
| kashyap | Something like that. FWIW, it looks okay to me | 11:08 |
| kashyap | I think I vaguely recall some list discussion | 11:08 |
| sfinucan | It was in merge conflict so I just fixed it :) | 11:08 |
| kashyap | Ah, okay :-) | 11:09 |
| *** ltomasbo is now known as ltomasbo|away | 11:10 | |
| *** bkopilov has quit IRC | 11:12 | |
| *** artom_ has joined #openstack-nova | 11:16 | |
| *** artom_ has quit IRC | 11:18 | |
| *** lpetrut has joined #openstack-nova | 11:18 | |
| *** artom has joined #openstack-nova | 11:19 | |
| *** artom has quit IRC | 11:20 | |
| *** artom has joined #openstack-nova | 11:20 | |
| *** artom__ has quit IRC | 11:20 | |
| *** dave-mccowan has joined #openstack-nova | 11:20 | |
| *** artom_ has joined #openstack-nova | 11:22 | |
| *** artom has quit IRC | 11:22 | |
| *** thorst has joined #openstack-nova | 11:30 | |
| *** lpetrut has quit IRC | 11:31 | |
| *** rmart04 has joined #openstack-nova | 11:34 | |
| *** mdnadeem has quit IRC | 11:36 | |
| *** timello has joined #openstack-nova | 11:36 | |
| *** kornica is now known as kornicameister | 11:37 | |
| *** artom_ has quit IRC | 11:40 | |
| *** garyk2 has joined #openstack-nova | 11:40 | |
| *** artom_ has joined #openstack-nova | 11:40 | |
| *** garyk1 has quit IRC | 11:41 | |
| *** artom__ has joined #openstack-nova | 11:44 | |
| *** artom_ has quit IRC | 11:44 | |
| *** dane-fichter has joined #openstack-nova | 11:47 | |
| *** dane-fichter has quit IRC | 11:48 | |
| *** artom_ has joined #openstack-nova | 11:48 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Libvirt: Enable QEMU native LUKs handling https://review.openstack.org/437070 | 11:48 |
| lyarwood | mdbooth, kashyap ^ if you have spec review time today, I'm not 100% sure that we will have the required versions of QEMU and libvirt in the gate for Pike tbh | 11:49 |
| kashyap | lyarwood: I did notice it, actually | 11:49 |
| kashyap | lyarwood: Yes, we don't have the reqiured versions | 11:49 |
| *** baoli has joined #openstack-nova | 11:50 | |
| *** baoli has quit IRC | 11:50 | |
| lyarwood | kashyap: right, but 17.04 will, I just don't think it's going to be used in the gate as it isn't LTS | 11:50 |
| kashyap | lyarwood: We should really get that in-progress (over a year) DevStack plugin that lets us use QEMU / libvirt from Git (and newest releases / RCs) with Nova | 11:50 |
| lyarwood | kashyap: link? I don't think I've seen that | 11:50 |
| *** baoli has joined #openstack-nova | 11:50 | |
| kashyap | lyarwood: https://review.openstack.org/#/c/108714/ | 11:51 |
| lyarwood | kashyap: thanks! | 11:51 |
| kashyap | lyarwood: Err, scratch that -- that's the OLD one that was subimtted directly as part of DevStack | 11:51 |
| *** mdnadeem has joined #openstack-nova | 11:51 | |
| kashyap | But the agreed decision was to make it a plugin | 11:51 |
| lyarwood | kk | 11:52 |
| *** artom__ has quit IRC | 11:52 | |
| *** smatzek has joined #openstack-nova | 11:52 | |
| *** jpena is now known as jpena|lunch | 11:54 | |
| *** artom_ has quit IRC | 11:54 | |
| *** Jack_Iv_ has quit IRC | 11:54 | |
| *** Jack_Iv has joined #openstack-nova | 11:54 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Libvirt: Enable QEMU native LUKs handling https://review.openstack.org/437070 | 11:55 |
| *** artom has joined #openstack-nova | 11:56 | |
| mdbooth | lyarwood: Will try to look today | 11:56 |
| lyarwood | mdbooth: thanks | 11:56 |
| *** artom has quit IRC | 11:58 | |
| *** voelzmo has quit IRC | 11:58 | |
| *** artom has joined #openstack-nova | 11:58 | |
| *** artom has quit IRC | 12:00 | |
| *** artom has joined #openstack-nova | 12:00 | |
| *** rodrigod` is now known as rodrigods | 12:01 | |
| *** rodrigods has joined #openstack-nova | 12:01 | |
| *** rodrigods has quit IRC | 12:01 | |
| *** voelzmo has joined #openstack-nova | 12:01 | |
| *** rodrigods has joined #openstack-nova | 12:02 | |
| *** lucas-hungry is now known as lucasagomes | 12:02 | |
| *** ltomasbo|away is now known as ltomasbo | 12:02 | |
| *** rodrigods has quit IRC | 12:03 | |
| *** rodrigods has joined #openstack-nova | 12:03 | |
| *** catintheroof has joined #openstack-nova | 12:05 | |
| lyarwood | kashyap: thanks for the link in the review btw | 12:05 |
| lyarwood | kashyap: I'll check that out after lunch | 12:05 |
| *** edmondsw has joined #openstack-nova | 12:06 | |
| kashyap | lyarwood: Np. I'll be looking at that plugin again today, been meaning to but couldn't get around | 12:06 |
| *** tbachman has joined #openstack-nova | 12:06 | |
| jaypipes | snikitin_: done | 12:07 |
| *** amoralej is now known as amoralej|lunch | 12:08 | |
| snikitin_ | jaypipes, thanks! | 12:08 |
| *** karimb_ has quit IRC | 12:09 | |
| *** Guest6565 has quit IRC | 12:11 | |
| *** artom has quit IRC | 12:12 | |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Removes getfattr from Quobyte Nova driver https://review.openstack.org/428646 | 12:12 |
| *** artom has joined #openstack-nova | 12:12 | |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Removes getfattr from Quobyte Nova driver https://review.openstack.org/428646 | 12:15 |
| *** zz_dimtruck is now known as dimtruck | 12:16 | |
| *** artom has quit IRC | 12:16 | |
| *** artom has joined #openstack-nova | 12:16 | |
| *** artom has quit IRC | 12:18 | |
| *** artom has joined #openstack-nova | 12:18 | |
| *** karimb has joined #openstack-nova | 12:20 | |
| *** jdillaman has joined #openstack-nova | 12:21 | |
| *** tbachman has quit IRC | 12:21 | |
| *** tbachman has joined #openstack-nova | 12:25 | |
| *** rfolco has joined #openstack-nova | 12:26 | |
| *** mdnadeem has quit IRC | 12:27 | |
| *** rfolco_ has joined #openstack-nova | 12:28 | |
| bauzas | jaypipes: you could be interested in https://review.openstack.org/#/c/446446/ | 12:28 |
| bauzas | johnthetubaguy: too ^ | 12:29 |
| *** carthaca_ has quit IRC | 12:29 | |
| *** carthaca_ has joined #openstack-nova | 12:29 | |
| *** sapcc-bot has quit IRC | 12:30 | |
| *** carthaca_ has quit IRC | 12:30 | |
| *** sapcc-bot has joined #openstack-nova | 12:31 | |
| *** rfolco has quit IRC | 12:31 | |
| *** esberglu has joined #openstack-nova | 12:31 | |
| *** peter-hamilton has joined #openstack-nova | 12:31 | |
| johnthetubaguy | bauzas: hopefully will get there, mostly just cleaning up the queue this morning | 12:31 |
| *** jianghuaw has joined #openstack-nova | 12:32 | |
| bauzas | johnthetubaguy: where are we discussing in the PTG etherpad about https://review.openstack.org/#/c/381912/10/specs/pike/approved/strict_isolation_of_group_of_hosts_for_image_and_flavor.rst ? | 12:32 |
| bauzas | johnthetubaguy: I don't remember that | 12:32 |
| johnthetubaguy | bauzas: search for the patch number, you should find it | 12:32 |
| * bauzas facepalms | 12:32 | |
| bauzas | johnthetubaguy: FWIW, it's an old discussion | 12:33 |
| *** nkorabli has joined #openstack-nova | 12:34 | |
| *** jianghuaw__ has quit IRC | 12:34 | |
| *** baoli has quit IRC | 12:35 | |
| *** ralonsoh has quit IRC | 12:36 | |
| *** esberglu has quit IRC | 12:36 | |
| *** vks1 has quit IRC | 12:38 | |
| snikitin_ | jaypipes: I don't know why bot after your reapprove gate jobs weren't started. https://review.openstack.org/#/c/445855/ maybe try again with +2 | 12:38 |
| *** sapcc-bot has quit IRC | 12:39 | |
| snikitin_ | ? | 12:39 |
| *** gouthamr has joined #openstack-nova | 12:39 | |
| *** baoli has joined #openstack-nova | 12:39 | |
| *** dimtruck is now known as zz_dimtruck | 12:40 | |
| *** tpatzig_ has joined #openstack-nova | 12:40 | |
| *** david_1 has joined #openstack-nova | 12:40 | |
| *** sapcc-bot has joined #openstack-nova | 12:40 | |
| *** carthaca_ has joined #openstack-nova | 12:40 | |
| *** dgonzalez_ has joined #openstack-nova | 12:40 | |
| *** mkoderer_ has joined #openstack-nova | 12:40 | |
| *** mkoderer_ has quit IRC | 12:40 | |
| *** sapcc-bot has quit IRC | 12:41 | |
| *** mkoderer_ has joined #openstack-nova | 12:41 | |
| *** sapcc-bot has joined #openstack-nova | 12:41 | |
| *** voelzmo has quit IRC | 12:41 | |
| sdague | snikitin_: it needs both a +2 and a +W | 12:41 |
| *** david_1 has quit IRC | 12:42 | |
| *** tpatzig_ has quit IRC | 12:42 | |
| snikitin_ | sdague, thanks! | 12:42 |
| *** dgonzalez_ has quit IRC | 12:42 | |
| *** carthaca_ has quit IRC | 12:42 | |
| *** mkoderer_ has quit IRC | 12:42 | |
| *** sapcc-bot has quit IRC | 12:43 | |
| *** Jack_Iv has quit IRC | 12:43 | |
| *** sapcc-bot has joined #openstack-nova | 12:43 | |
| *** lyan has joined #openstack-nova | 12:43 | |
| *** Jack_Iv has joined #openstack-nova | 12:43 | |
| *** voelzmo has joined #openstack-nova | 12:43 | |
| *** mdnadeem has joined #openstack-nova | 12:45 | |
| *** voelzmo has quit IRC | 12:46 | |
| *** liverpooler has joined #openstack-nova | 12:46 | |
| *** sapcc-bot has quit IRC | 12:47 | |
| *** sapcc-bot has joined #openstack-nova | 12:47 | |
| *** sapcc-bot1 has joined #openstack-nova | 12:48 | |
| *** voelzmo has joined #openstack-nova | 12:49 | |
| *** sapcc-bot1 has quit IRC | 12:49 | |
| *** sapcc-bot has quit IRC | 12:49 | |
| *** sapcc-bot1 has joined #openstack-nova | 12:49 | |
| *** jpena|lunch is now known as jpena | 12:50 | |
| *** suresh12 has joined #openstack-nova | 12:50 | |
| *** sapcc-bot1 has quit IRC | 12:50 | |
| *** sapcc-bot has joined #openstack-nova | 12:51 | |
| *** Jack_Iv has quit IRC | 12:51 | |
| *** suresh12 has quit IRC | 12:52 | |
| *** suresh12 has joined #openstack-nova | 12:52 | |
| *** sapcc-bot has quit IRC | 12:52 | |
| *** jwcroppe_ has quit IRC | 12:52 | |
| *** Jack_Iv has joined #openstack-nova | 12:52 | |
| *** IRCFrEAK has joined #openstack-nova | 12:53 | |
| *** IRCFrEAK has quit IRC | 12:53 | |
| *** Jack_Iv has quit IRC | 12:53 | |
| *** Jack_Iv has joined #openstack-nova | 12:53 | |
| *** esberglu has joined #openstack-nova | 12:57 | |
| *** nkorabli has quit IRC | 12:57 | |
| johnthetubaguy | bauzas: sfinucan: can I ask you about this one: https://review.openstack.org/#/c/438787 | 12:59 |
| sfinucan | johnthetubaguy: Hit me | 12:59 |
| johnthetubaguy | bauzas: sfinucan: why can't we use the existing image property, basically? | 12:59 |
| * bauzas looking | 13:00 | |
| sfinucan | johnthetubaguy: I think the existing property refers to a different attribute of the emulated VGA dev | 13:00 |
| johnthetubaguy | it seems to be video ram vs vga ram, they seem like the same thing to me | 13:00 |
| bauzas | johnthetubaguy: AFAICU, that seems to be a separate property | 13:01 |
| bauzas | looking at https://libvirt.org/formatdomain.html | 13:01 |
| sfinucan | johnthetubaguy: That should be in the spec. If you haven't done it already, there should probably be a comment to the same | 13:01 |
| johnthetubaguy | but why are they different? what does it mean? | 13:01 |
| sfinucan | They are different properties though. Lemme find the QEMU docs | 13:01 |
| sfinucan | johnthetubaguy: | 13:02 |
| sfinucan | johnthetubaguy: https://www.ovirt.org/documentation/draft/video-ram/ | 13:02 |
| sfinucan | vgamem is allocated within ram. Minimum ram size, as enforced by QEMU regardless the command line settings, is 2 * vgamem. vram is allocated separately. | 13:02 |
| *** awaugama has joined #openstack-nova | 13:02 | |
| bauzas | johnthetubaguy: https://libvirt.org/formatdomain.html#elementsVideo | 13:02 |
| bauzas | what sfinucan said | 13:02 |
| *** Jack_Iv has quit IRC | 13:02 | |
| bauzas | "Attribute ram ( since 1.0.2) specifies the size of the primary bar, while the attribute vram specifies the secondary bar size. If ram or vram are not supplied a default value is used. The ram should also be rounded to power of two as vram. There is also optional attribute vgamem (since 1.2.11) to set the size of VGA framebuffer for fallback mode of QXL device | 13:03 |
| bauzas | honestly, it's a very specific libvirt thing | 13:03 |
| sfinucan | fwiw, it's just another libvirt knob | 13:03 |
| sfinucan | bauzas: jinx :) | 13:03 |
| bauzas | I'm not really happy with that, hence my comment saying to maybe wait for custom RCs :) | 13:03 |
| sfinucan | bauzas: well, technically QEMU - libvirt just wraps it | 13:03 |
| *** gouthamr has quit IRC | 13:04 | |
| *** links has quit IRC | 13:04 | |
| bauzas | johnthetubaguy: sfinucan: just -1d it again | 13:06 |
| *** belmoreira has joined #openstack-nova | 13:07 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs master: SPICE native client support https://review.openstack.org/442040 | 13:08 |
| *** bkopilov has joined #openstack-nova | 13:09 | |
| johnthetubaguy | bauzas: don't feel we are being great at giving suggestions how to fix the spec right now though | 13:09 |
| *** artom has quit IRC | 13:10 | |
| *** artom has joined #openstack-nova | 13:10 | |
| *** mdrabe has joined #openstack-nova | 13:11 | |
| *** Jack_Iv has joined #openstack-nova | 13:11 | |
| *** zul has quit IRC | 13:12 | |
| *** jwcroppe has joined #openstack-nova | 13:12 | |
| *** eharney has joined #openstack-nova | 13:12 | |
| *** Jack_Iv has quit IRC | 13:12 | |
| *** amoralej|lunch is now known as amoralej | 13:14 | |
| bauzas | johnthetubaguy: I tried to explain my preferrence about providing custom resource classes against very specific image props | 13:14 |
| *** xyang1 has joined #openstack-nova | 13:14 | |
| *** tbachman has quit IRC | 13:14 | |
| *** Jack_Iv has joined #openstack-nova | 13:15 | |
| *** amotoki has quit IRC | 13:15 | |
| *** sapcc-bot has joined #openstack-nova | 13:15 | |
| *** sapcc-bot has quit IRC | 13:15 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova master: Added PCI NUMA policies https://review.openstack.org/390520 | 13:16 |
| *** sapcc-bot has joined #openstack-nova | 13:16 | |
| *** gouthamr has joined #openstack-nova | 13:17 | |
| *** efried has joined #openstack-nova | 13:18 | |
| *** cdent has quit IRC | 13:18 | |
| *** sapcc-bot has quit IRC | 13:19 | |
| *** sapcc-bot has joined #openstack-nova | 13:19 | |
| *** sapcc-bot has quit IRC | 13:20 | |
| *** sapcc-bot has joined #openstack-nova | 13:21 | |
| *** annegentle has joined #openstack-nova | 13:21 | |
| *** prateek_ has joined #openstack-nova | 13:22 | |
| *** aysyd has joined #openstack-nova | 13:23 | |
| *** liverpooler has quit IRC | 13:24 | |
| *** sapcc-bot has quit IRC | 13:24 | |
| *** prateek has quit IRC | 13:24 | |
| *** liverpooler has joined #openstack-nova | 13:24 | |
| *** sapcc-bot has joined #openstack-nova | 13:24 | |
| *** sapcc-bot has quit IRC | 13:25 | |
| *** sapcc-bot has joined #openstack-nova | 13:25 | |
| *** sapcc-bot has quit IRC | 13:26 | |
| *** annegentle has quit IRC | 13:26 | |
| *** prateek_ has quit IRC | 13:27 | |
| *** mriedem has joined #openstack-nova | 13:27 | |
| mriedem | is everyone excited for a fun spec review day?! | 13:28 |
| * artom got his spec updated just in time | 13:29 | |
| artom | Doc's appointment this morning, but will try to participate this afternoon | 13:29 |
| peter-hamilton | mriedem: indeed! | 13:29 |
| mriedem | i see johnthetubaguy has been having fun | 13:29 |
| *** cleong has joined #openstack-nova | 13:30 | |
| johnthetubaguy | yeah, I abandoned a bunch of left alone specs | 13:30 |
| johnthetubaguy | only three pages of joy now | 13:30 |
| mriedem | awesome | 13:30 |
| *** zul has joined #openstack-nova | 13:32 | |
| *** dikonoor has joined #openstack-nova | 13:32 | |
| *** crushil has joined #openstack-nova | 13:33 | |
| *** liverpooler has quit IRC | 13:34 | |
| *** liverpooler has joined #openstack-nova | 13:34 | |
| *** jaosorior is now known as jaosorior_away | 13:36 | |
| *** litao has joined #openstack-nova | 13:36 | |
| *** awaugama has quit IRC | 13:37 | |
| *** sree has joined #openstack-nova | 13:39 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add remove-nova-cert spec https://review.openstack.org/448882 | 13:39 |
| *** mlavalle has joined #openstack-nova | 13:44 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'agent' commands https://review.openstack.org/450795 | 13:45 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'shell' commands https://review.openstack.org/432280 | 13:45 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'project', 'account' commands https://review.openstack.org/453174 | 13:45 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'host' commands https://review.openstack.org/453175 | 13:45 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'log' commands https://review.openstack.org/453176 | 13:45 |
| *** sree has quit IRC | 13:46 | |
| bauzas | mriedem: I guess you don't know https://en.wikipedia.org/wiki/L%C3%A9onard_%28comics%29 ? | 13:48 |
| mriedem | nope | 13:48 |
| bauzas | mriedem: because the disciple always says "I serve science and it's my joy" | 13:49 |
| *** amotoki has joined #openstack-nova | 13:50 | |
| *** dane-fichter has joined #openstack-nova | 13:50 | |
| *** kevinz has quit IRC | 13:50 | |
| *** liangy has joined #openstack-nova | 13:51 | |
| dane-fichter | peter-hamilton: anything relevant so far? | 13:52 |
| *** nmathew has quit IRC | 13:53 | |
| peter-hamilton | dane-fichter: not yet :) | 13:53 |
| dane-fichter | peter-hamilton: cool. got caught in traffic | 13:53 |
| *** lyan has quit IRC | 13:53 | |
| *** aysyd has quit IRC | 13:53 | |
| *** felipemonteiro has joined #openstack-nova | 13:54 | |
| mriedem | peter-hamilton: dane-fichter: you guys might want to review https://review.openstack.org/#/c/448882 | 13:54 |
| mriedem | for dropping the nova-cert service | 13:54 |
| peter-hamilton | mriedem: looking now | 13:54 |
| dane-fichter | mriedem: we'll take a look, thanks | 13:54 |
| *** cdent has joined #openstack-nova | 13:56 | |
| *** hoangcx has joined #openstack-nova | 13:56 | |
| tdurakov | hi, folks, there will be no live-migration meeting this week | 13:58 |
| bhagyashris | johnthetubaguy: Hi, Could you please review patch https://review.openstack.org/#/c/409644/ ? if you have free time. | 13:58 |
| *** amotoki has quit IRC | 13:59 | |
| *** jwcroppe has quit IRC | 14:00 | |
| sdague | johnthetubaguy: why did you think 400 instead of 404 - https://review.openstack.org/#/c/448882 ? | 14:01 |
| *** sree has joined #openstack-nova | 14:01 | |
| johnthetubaguy | sdague: didn't consider 404 honestly | 14:01 |
| johnthetubaguy | sdague: I was seeing this as the implementation removed, not the API changed | 14:02 |
| *** jwcroppe has joined #openstack-nova | 14:02 | |
| sdague | well, we could always be tricksie and use 410 then | 14:03 |
| sdague | but I though that 404 is what the whole system returns if you go fetch /foobar | 14:03 |
| *** sree has quit IRC | 14:06 | |
| johnthetubaguy | sdague: oh, 410 = gone, I had not come across that. I guess it depends if we want to pretend if the old API never existed or not. 410 would help know you got the correct URL, as you always used to, its just the server that has change and no longer wants to server up that API | 14:06 |
| *** mdnadeem has quit IRC | 14:07 | |
| *** lyan_ has joined #openstack-nova | 14:07 | |
| *** namnh has joined #openstack-nova | 14:07 | |
| sdague | johnthetubaguy: yeh, though my hesitation on 410 is that no tool is every designed to understand it :) | 14:08 |
| johnthetubaguy | sdague: yeah, thats good and bad in equal measure I guess | 14:08 |
| sdague | yep | 14:09 |
| sdague | but, more importantly, I think it requires us to have a stub hanging around | 14:09 |
| *** ekuris has quit IRC | 14:09 | |
| sdague | vs. just deleting the handler | 14:09 |
| sdague | and, I'd rather just delete the handler | 14:09 |
| *** ekuris has joined #openstack-nova | 14:09 | |
| *** liangy has quit IRC | 14:09 | |
| *** dharinic has joined #openstack-nova | 14:10 | |
| *** armax has joined #openstack-nova | 14:11 | |
| *** satyar has joined #openstack-nova | 14:12 | |
| *** mlakat has quit IRC | 14:13 | |
| *** ekuris has quit IRC | 14:13 | |
| *** garyk2 has quit IRC | 14:14 | |
| *** garyk has joined #openstack-nova | 14:14 | |
| *** tbachman has joined #openstack-nova | 14:15 | |
| *** jdillaman has quit IRC | 14:15 | |
| *** amotoki has joined #openstack-nova | 14:17 | |
| cdent | sdague, johnthetubaguy I reckon deleting the handler has most win (because of getting to remove cleanly) | 14:17 |
| *** sree has joined #openstack-nova | 14:17 | |
| mriedem | do i need a spec to propose deleting the os-pci api code even though it's never been enabled or exposed? | 14:17 |
| cdent | however 410 would also be fine. Not that many tools actually "understand" response codes. They just react to them. | 14:18 |
| cdent | mriedem I think you have to ask mriedem about that | 14:19 |
| mriedem | damn | 14:19 |
| cdent | and then collapse in on yourself | 14:19 |
| *** tommylikehu_ has joined #openstack-nova | 14:19 | |
| cdent | mriedem speaking of needing specs: https://review.openstack.org/#/c/448791/ needs one, I guess? | 14:20 |
| mriedem | probably yeah | 14:20 |
| mriedem | i don't want us to get lax on api impacts for placement | 14:21 |
| cdent | fair | 14:21 |
| * cdent adds to list | 14:21 | |
| mriedem | clarkb: with the UCA change in devstack we get libvirt 2.5 right? do you know if we also are going to get an updated qemu? and if so, which version? | 14:22 |
| *** lyan_ has quit IRC | 14:22 | |
| *** sree_ has joined #openstack-nova | 14:22 | |
| sdague | mriedem: we will | 14:23 |
| *** sree_ is now known as Guest15320 | 14:23 | |
| mriedem | just wondering if it's new enough to cover what's in https://review.openstack.org/#/c/437070/ | 14:23 |
| mriedem | ^ requires qemu 2.6 | 14:23 |
| *** sree has quit IRC | 14:24 | |
| sdague | mriedem: https://launchpad.net/~ubuntu-cloud-archive/+archive/ubuntu/ocata-staging/+index?batch=75&direction=backwards&start=150 | 14:26 |
| sdague | guessing, I think that would be 2.8 | 14:26 |
| *** tommylikehu_ has quit IRC | 14:26 | |
| *** jdillaman has joined #openstack-nova | 14:28 | |
| *** awaugama has joined #openstack-nova | 14:28 | |
| *** eharney has quit IRC | 14:31 | |
| mriedem | lyarwood: this introduced a bug https://review.openstack.org/#/c/393901/ | 14:32 |
| mriedem | encryptor may not be defined when logged | 14:32 |
| *** crushil has quit IRC | 14:32 | |
| *** ralonsoh has joined #openstack-nova | 14:32 | |
| *** lyan_ has joined #openstack-nova | 14:33 | |
| lyarwood | mriedem: bug #? The os-brick differs there for some reason.. | 14:35 |
| *** READ10 has joined #openstack-nova | 14:35 | |
| mriedem | lyarwood: no bug yet, i can open one | 14:35 |
| lyarwood | mriedem: thanks, I'll get it backported, btw I'm looking to drop all of this code in Pike via https://review.openstack.org/#/c/391597/ | 14:36 |
| *** zz_dimtruck is now known as dimtruck | 14:37 | |
| mriedem | heh that's not really a bug | 14:37 |
| *** mdrabe has quit IRC | 14:37 | |
| *** crushil has joined #openstack-nova | 14:37 | |
| *** hongbin has joined #openstack-nova | 14:37 | |
| lyarwood | mriedem: ah just code review? I assumed it came from a customer, just reading the scollback it looks like you are reviewing the QEMU LUKS spec right? | 14:38 |
| mriedem | yes | 14:38 |
| lyarwood | kk, don't worry about the bug then | 14:38 |
| mriedem | already written | 14:39 |
| mriedem | i meant your os-brick encryptors change was not a bug | 14:39 |
| mriedem | to make nova use os-brick encryptors code | 14:39 |
| mriedem | is not a bug | 14:39 |
| mriedem | the NameError is definitely a bug :) | 14:39 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1679717 | 14:39 |
| openstack | Launchpad bug 1679717 in OpenStack Compute (nova) "Potential for NameError when logging volume encryptor" [Undecided,New] | 14:39 |
| lyarwood | ah | 14:39 |
| *** mriedem has left #openstack-nova | 14:40 | |
| *** tjones has joined #openstack-nova | 14:40 | |
| *** mriedem has joined #openstack-nova | 14:40 | |
| *** vks1 has joined #openstack-nova | 14:41 | |
| *** mdrabe has joined #openstack-nova | 14:43 | |
| *** huyang_ has joined #openstack-nova | 14:44 | |
| *** huyang_ has quit IRC | 14:45 | |
| *** abalutoiu has joined #openstack-nova | 14:46 | |
| *** eharney has joined #openstack-nova | 14:46 | |
| mriedem | lyarwood: your spec reminds me that the cinder API reference doesn't even mention the encryption provider stuff in the volume type api docs | 14:47 |
| openstackgerrit | Jeremy Stanley proposed openstack/nova master: Switch from pip_missing_reqs to pip_check_reqs https://review.openstack.org/453208 | 14:47 |
| *** cdent has quit IRC | 14:47 | |
| *** jahsis has quit IRC | 14:47 | |
| *** abalutoiu__ has quit IRC | 14:48 | |
| *** erhudy has joined #openstack-nova | 14:48 | |
| *** dikonoor has quit IRC | 14:48 | |
| *** armax has quit IRC | 14:51 | |
| namnh | dansmith: Hi mr Smith, are you free, could I ask you a question about rolling upgrade? | 14:51 |
| dansmith | namnh: just ask and someone here will answer if we know | 14:52 |
| lyarwood | mriedem: yeah isn't it just stashed in the type as in extra_specs? ./me doesn't have an env to check | 14:53 |
| *** yingjun has joined #openstack-nova | 14:53 | |
| *** crushil has quit IRC | 14:53 | |
| *** yingjun has quit IRC | 14:53 | |
| lyarwood | type in extra_specs* | 14:53 |
| namnh | dansmith: According to this link [1], we have to stop and start all Nova's serices except compute at the same time. | 14:53 |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova-specs master: Stop using Neutron admin token https://review.openstack.org/439890 | 14:53 |
| namnh | [1] https://docs.openstack.org/developer/nova/upgrade.html#rolling-upgrade-process | 14:54 |
| namnh | dansmith: But IMO, there is another way to upgrade is that I will upgrade the services except compute sequentially by pinning version each services like this file [2], at this time we have a new code with the value of "pin version" is old release, after that I will unpin version by restart all services at the same time. If we do this way, we only have downtime when unpining version. How do you think? | 14:54 |
| namnh | [2] https://github.com/openstack/nova/blob/master/nova/conf/upgrade_levels.py | 14:54 |
| dansmith | namnh: as we said yesterday, that only works for compute | 14:55 |
| dansmith | namnh: the other services talk directly to the database, which doesn't honor the upgrade level (those are only for rpc) | 14:55 |
| dansmith | namnh: compute is not allowed to talk directly to the database, which is why it can be old, but the others cannoy | 14:55 |
| dansmith | *cannot | 14:55 |
| *** annegentle has joined #openstack-nova | 14:56 | |
| *** armax has joined #openstack-nova | 14:56 | |
| *** crushil has joined #openstack-nova | 14:57 | |
| *** bhagyashris has quit IRC | 14:59 | |
| *** jpena is now known as jpena|away | 15:00 | |
| *** ratailor has joined #openstack-nova | 15:00 | |
| namnh | dansmith: but if I upgrade one conductor (N ->O), I will pin version is that: conductor=newton. That mean although conductor at new release, but it will continue talk with other services due to pin version. | 15:00 |
| namnh | dansmith: I understand this file [1] configuration to do this. [1] https://github.com/openstack/nova/blob/master/nova/conf/upgrade_levels.py | 15:01 |
| namnh | dansmith: we can pin version for each service | 15:02 |
| *** hoangcx has quit IRC | 15:03 | |
| *** lucasxu has joined #openstack-nova | 15:04 | |
| *** pumarani_ has joined #openstack-nova | 15:04 | |
| *** vks1 has quit IRC | 15:04 | |
| *** timello has quit IRC | 15:04 | |
| bauzas | dansmith: mriedem: so I thought about the claims spec and I pushed my thoughts in the spec https://review.openstack.org/#/c/437424/ | 15:04 |
| bauzas | dansmith: mriedem: tl;dr I think there are 2 things | 15:05 |
| *** yamahata has joined #openstack-nova | 15:05 | |
| bauzas | 1/ rescheduling is a cellsv2 problem, not really related to my spec, so we should discuss that separately | 15:05 |
| *** jwcroppe has quit IRC | 15:05 | |
| bauzas | because rescheduling (even without scheduler claims) is not possible in a cellsv2 world | 15:05 |
| *** Oku_OS is now known as Oku_OS-away | 15:06 | |
| bauzas | 2/ I think it would be a still good behaviour if computes delete allocations directly by calling placemnt API, given it would be quicker than waiting for the scheduler during the reschedule | 15:06 |
| *** jwcroppe has joined #openstack-nova | 15:07 | |
| namnh | hieulq: ping | 15:08 |
| frickler | from a deployment perspective, is there any sense in providing the placement API to customers? or would one rather create an internal endpoint only? | 15:08 |
| frickler | I'm assuming the latter and then it might make sense to change the default of placement.os_interface from public to internal, too | 15:10 |
| pumarani_ | sdague, checking your comment on https://review.openstack.org/#/c/439890/. I understand your concern of getting neutron changes in place first. | 15:10 |
| pumarani_ | sdague, we have already added RFE in neutron https://review.openstack.org/#/c/439890/ | 15:10 |
| pumarani_ | i mean https://bugs.launchpad.net/neutron/+bug/1674349 | 15:11 |
| openstack | Launchpad bug 1674349 in neutron "[RFE] Introduce a new rule with service user role in Neutron policy" [Wishlist,Triaged] - Assigned to Akihiro Motoki (amotoki) | 15:11 |
| *** yamahata has quit IRC | 15:12 | |
| *** Guest52040 has joined #openstack-nova | 15:13 | |
| *** armax has quit IRC | 15:13 | |
| *** nkrinner is now known as nkrinner_afk | 15:14 | |
| *** annegentle has quit IRC | 15:15 | |
| *** cdent has joined #openstack-nova | 15:16 | |
| *** baoli has quit IRC | 15:18 | |
| *** raj_sing- has joined #openstack-nova | 15:19 | |
| sdague | pumarani_: ok, that's a wishlist bug, does that mean it's commited to be accepted? | 15:19 |
| sdague | also, where is implementation code for it | 15:19 |
| frickler | sdague: mriedem: this has a nicer (imho) list of all the versions in Ocata UCA: http://reqorts.qa.ubuntu.com/reports/ubuntu-server/cloud-archive/ocata_versions.html | 15:19 |
| *** ralonsoh_ has joined #openstack-nova | 15:19 | |
| sdague | frickler: ah, cool, I was looking for something like that | 15:20 |
| *** raj_singh_ has quit IRC | 15:20 | |
| mriedem | lyarwood: no it's in the volume encryption model in the cinder db | 15:20 |
| *** vks1 has joined #openstack-nova | 15:20 | |
| sdague | pumarani_: I guess, it's really unclear to me what the neutron status is, and as such I'd really feel a lot better about this depending on merged neutron code. | 15:21 |
| *** jdillaman has quit IRC | 15:21 | |
| mdbooth | lyarwood: Are you wondering where the provider string comes from? | 15:22 |
| *** Jack_Iv_ has joined #openstack-nova | 15:22 | |
| mriedem | bauzas: if we can't reschedule, why do we care about how fast the allocations are deleted? | 15:22 |
| *** Jack_Iv__ has joined #openstack-nova | 15:22 | |
| *** joanna__ has joined #openstack-nova | 15:22 | |
| mriedem | that raises the question of when the allocations are deleted, but i think that's just left unchanged, when we delete the instance we remove the allocations via RT in the comptue host | 15:23 |
| mriedem | *compute | 15:23 |
| bauzas | mriedem: it's a separate problem, hence me saying if reschedules are possible, then I'd prefer to delete the allocations sooner than later | 15:23 |
| mdbooth | mriedem: Do you have a spec to delete bdm v1, btw? | 15:23 |
| mriedem | mdbooth: i don't | 15:23 |
| mriedem | bauzas: also, have we taken into account local deletes in the API? | 15:23 |
| *** ralonsoh has quit IRC | 15:23 | |
| bauzas | mriedem: I'm just putting a big hypothesis in my spec that is, reschedules are possible | 15:23 |
| mriedem | the compute api would have to delete the allocations there if the compute is dead | 15:23 |
| mdbooth | mriedem: Why does it linger, btw? | 15:23 |
| *** Jack_Iv__ has quit IRC | 15:24 | |
| mriedem | mdbooth: no good reason probably | 15:24 |
| *** belmoreira has quit IRC | 15:24 | |
| *** Jack_Iv has quit IRC | 15:24 | |
| *** Guest15320 has quit IRC | 15:24 | |
| mdbooth | It's a prime candidate for simplification. | 15:24 |
| bauzas | mriedem: well, that's possibly a good point, if the compute is dead, then the allocation would still be created | 15:24 |
| *** Jack_Iv has joined #openstack-nova | 15:24 | |
| *** sree has joined #openstack-nova | 15:24 | |
| bauzas | mriedem: so we could possibly clean up "dead" allocations | 15:24 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova master: The API defines PUT and POST as HTTP methods which need a request body. It is allowed, that this request body might be of zero length. What was missing is, that the "content-length" of the request also might be None. https://review.openstack.org/453220 | 15:25 |
| bauzas | mriedem: that said, the ComputeFilter verifies whether the service is still up | 15:25 |
| *** Jack_Iv_ has quit IRC | 15:25 | |
| *** Jack_Iv has quit IRC | 15:25 | |
| *** ralonsoh_ is now known as ralonsoh | 15:25 | |
| pumarani_ | sdague, yes. With neutron changes getting merged, we can also have some code samples on it. we are following up on neutron RFE. | 15:25 |
| *** Jack_Iv has joined #openstack-nova | 15:25 | |
| bauzas | mriedem: so the real problem is rather, say, a full compute MQ with bad performance | 15:25 |
| bauzas | mriedem: then we would have created an allocation for something not really created | 15:26 |
| bauzas | either way, I need to think more of that | 15:26 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova master: API: accept None as content-length in HTTP requests https://review.openstack.org/453220 | 15:26 |
| sdague | mriedem: in our "spec review sprint" today, are there particular specs you want eyes on instead of just scatter shotting? | 15:26 |
| *** raj_sing- is now known as raj_singh | 15:27 | |
| openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/357151 | 15:29 |
| *** sree has quit IRC | 15:29 | |
| mriedem | lyarwood: https://bugs.launchpad.net/cinder/+bug/1679736 | 15:31 |
| openstack | Launchpad bug 1679736 in Cinder "VolumeTypeEncryption API extension is not documented in api-ref site" [Undecided,New] | 15:31 |
| *** peter-hamilton_ has joined #openstack-nova | 15:31 | |
| mriedem | bauzas: i'm talking about after you've landed an instance on a host, so after scheduling, and the compute is down and you delete the instance | 15:31 |
| *** peter-hamilton has quit IRC | 15:31 | |
| mriedem | since we can't remove the allocation via RT on the compute host, where do we deallocate for that resource provider for a local delete? | 15:31 |
| mriedem | hell, is that just a latent bug we already have with placement? | 15:31 |
| mriedem | jaypipes: dansmith: ^? | 15:32 |
| bauzas | mriedem: well, me shrugs | 15:32 |
| bauzas | I guess yes | 15:32 |
| mriedem | bauzas: i don't know what "then we would have created an allocation for something not really created" means | 15:32 |
| mriedem | sdague: full monty | 15:32 |
| * jaypipes reads back | 15:33 | |
| mriedem | bauzas: for the comptue isn't down, but the instance is shelved-offloaded, | 15:33 |
| bauzas | mriedem: I was thinking of the scheduler saying "okay, this one" then posting the allocations, then the boot requests goes trampled by the RPC bus | 15:33 |
| mriedem | i guess when we shelve offload, do we deallocate? | 15:33 |
| * cdent tries to catch up too | 15:33 | |
| cdent | I think there's some misconceptions going on here | 15:33 |
| mordred | cdent: people have the wrong idea about something? bah | 15:34 |
| jaypipes | mriedem: yeah, it's a latent bug I think. we don't currently deallocate orphaned instances like we should. | 15:34 |
| mriedem | lyarwood: i think we should add a microversion to cinder v3 to fix some of the volume type encryption api stuff, like remove the need for the full classname in the provider entry, just say 'luks' or 'cryptsetup' | 15:34 |
| cdent | yes, known bug: a dead compute node's allocations persist | 15:35 |
| mriedem | lyarwood: and as part of that microversion change in cinder v3, we enforce the actual providers using schema validation, rather than let it be free-form like it is today | 15:35 |
| mriedem | cdent: jaypipes: how about shelve offload? | 15:35 |
| cdent | there's also this: https://bugs.launchpad.net/nova/+bug/1661312 | 15:36 |
| openstack | Launchpad bug 1661312 in OpenStack Compute (nova) "Evacuation will corrupt instance allocations" [Undecided,New] | 15:36 |
| mriedem | because when we unshelve, the instance could end up on a different host | 15:36 |
| jaypipes | mriedem: no idea. would need to check. not sure if the compute manager calls an RT method on shelve_offload(). | 15:36 |
| mriedem | and if it's on a different host, we're then counting allocatoins twice for the same instnace | 15:36 |
| *** pumarani_ has quit IRC | 15:37 | |
| cdent | since allocations are keyed by consumer id, and we set them (as in fully replace them) the situation is a bit cleaner than it might initially appear | 15:38 |
| mriedem | but when we do claims in the scheduler, | 15:38 |
| mriedem | aren't we aggregating the total allocations for a given resource provider? | 15:39 |
| mriedem | and if we don't remove the allocations for a shelved offloaded consumer on that compute resource provider, it's a lie about actual resource allocation right? | 15:39 |
| cdent | if we are puting an allocation, it will be for the instance uuid, no matter what it uses | 15:39 |
| *** peter-hamilton_ is now known as peter-hamilton | 15:39 | |
| jaypipes | mriedem: no. | 15:39 |
| mriedem | i guess when we do claims we're looking at inventory? | 15:40 |
| *** iceyao has joined #openstack-nova | 15:40 | |
| *** gszasz has quit IRC | 15:40 | |
| mriedem | if i have an inventory of 5 apples, and your instance claims 3 apples, i have 2 apples left for sally to claim, right? | 15:40 |
| jaypipes | mriedem: the "claim" is merely the transactional writing of allocation records against one or more resource providers for a single consumer ID. | 15:40 |
| bauzas | mriedem: I think all the move operations need to update the allocations indeed | 15:41 |
| jaypipes | mriedem: yes, that is a check that occurs entirely within the POST /allocations context in the placement API though. | 15:41 |
| bauzas | I need a second pass of thoughts | 15:41 |
| mriedem | jaypipes: don't we POST allocations to a specific resource provider though? | 15:41 |
| cdent | mriedem: no | 15:42 |
| jaypipes | mriedem: you will get a 409 if, say, Alice gets an apple before Sally's attempt to get 2 apples. | 15:42 |
| jaypipes | mriedem: no, allocations are multi-provider. | 15:42 |
| cdent | mriedem: or rather, not quite | 15:42 |
| gibi | mriedem: hi! I saw your ping from yesterday about the snapshot notification and I had some free time to put a patch togheter for that issue https://review.openstack.org/#/c/453077/ | 15:42 |
| mriedem | resource_provider_uuid = allocation['resource_provider']['uuid'] | 15:42 |
| jaypipes | mriedem: for instance, a compute host provider and a shared storage pool provider might be involved in the same POST /allocations for a consumer (instance UUID) | 15:42 |
| mriedem | but the allocation is still tied to a resource provider | 15:43 |
| mriedem | one or more | 15:43 |
| jaypipes | mriedem: the allocation record is, yes. but the entire "claim" is a *set* of allocation records for a single consumer (the instance) | 15:43 |
| mriedem | the scheduler is asking placement for the list of available resource providers given some vcpu/ram/disk constraints, right? | 15:43 |
| jaypipes | yep | 15:43 |
| mriedem | and then takes that list, gets the compute nodes from the db, and filters them | 15:43 |
| jaypipes | yep | 15:44 |
| mriedem | so then i have maybe 3 available hosts, i pick one, and make my claim | 15:44 |
| mriedem | which is a POST allocation on that resource provider, yes? | 15:44 |
| jaypipes | no | 15:44 |
| jaypipes | :) | 15:44 |
| *** lucasxu has quit IRC | 15:44 | |
| jaypipes | it would be a POST /allocations and the compute node resource provider would be one of the resource providers involved in the allocation request. | 15:44 |
| jaypipes | mriedem: again, the reason this is like that is because you can claim resources against >1 provider in a single request. | 15:45 |
| mriedem | i think i'm trying to say the same thing | 15:45 |
| mriedem | but in the less formal way | 15:45 |
| mriedem | me ugg, me want 2 apples, you mug have 3 apples, me claim 2 of your 3 apples, yes? | 15:45 |
| jaypipes | yes | 15:46 |
| mriedem | i get the multiple provider thing, that's fine, i need some compute resources, i need some shared storage resources from another provider, whatever | 15:46 |
| cdent | (just to be pedantic the currently available method is PUT /allocations/{consumer_uuid} and I think we can continue using that one for the time being) | 15:46 |
| mriedem | ok, so i think we've established that the scheduler has 'claimed' resources on a provider, or set of providers, by putting an allocation for a given instance (consumer) on those providers, | 15:47 |
| mriedem | and then sends the build request to that host | 15:47 |
| mriedem | to build the instance | 15:47 |
| mriedem | since we can't reschedule, i guess i won't ask about that | 15:48 |
| *** Jack_Iv has quit IRC | 15:48 | |
| mriedem | so when we shelve offload an instance, do we deallocate for that consumer on it's resource providers | 15:48 |
| mriedem | or when we delete the instance, and the compute host is down so we can't rpc cast to the compute to do the delete, do we deallocate from the api | 15:48 |
| mriedem | i think the answer to both is no | 15:48 |
| jaypipes | yeah, I suspect you are correct about that. but would be easy enough to fix in a bug report. | 15:49 |
| mriedem | i may be wrong about the shelve offload case, because i remember alaski pointing out a super gross tight coupling of instance vm_state checking in the RT | 15:50 |
| mriedem | yup ALLOW_RESOURCE_REMOVAL = [DELETED, SHELVED_OFFLOADED] | 15:50 |
| mriedem | RT._update_usage_from_instance | 15:50 |
| mriedem | checks that | 15:50 |
| *** chyka has joined #openstack-nova | 15:50 | |
| jaypipes | ok | 15:50 |
| *** artom has quit IRC | 15:51 | |
| mriedem | which passes sign=-1 to update_instance_allocation to call _delete_allocation_for_instance | 15:51 |
| mriedem | cool | 15:51 |
| mriedem | we still fail the local delete case probably | 15:51 |
| mriedem | which is maybe related to dansmith's bug about evacuate | 15:51 |
| *** rmart04 has quit IRC | 15:52 | |
| *** jlvillal_pto is now known as jlvillal | 15:54 | |
| *** andreas_s has quit IRC | 15:54 | |
| *** rfolco_ has quit IRC | 15:56 | |
| *** artom has joined #openstack-nova | 15:57 | |
| *** hieulq_ has joined #openstack-nova | 15:57 | |
| *** hieulq__ has joined #openstack-nova | 15:58 | |
| *** namnh has quit IRC | 15:58 | |
| *** eharney has quit IRC | 15:59 | |
| *** markus_z has quit IRC | 16:00 | |
| *** lucasagomes is now known as lucas-afk | 16:00 | |
| dansmith | sorry I was on a call, and have only skimmed the scrollback | 16:01 |
| mriedem | dansmith: in https://bugs.launchpad.net/nova/+bug/1661312 where does step 4 happen? | 16:01 |
| openstack | Launchpad bug 1661312 in OpenStack Compute (nova) "Evacuation will corrupt instance allocations" [Undecided,New] | 16:01 |
| mriedem | dansmith: jaypipes: cdent: created https://bugs.launchpad.net/nova/+bug/1679750 | 16:01 |
| openstack | Launchpad bug 1679750 in OpenStack Compute (nova) "Allocations are not cleaned up in placement for instance 'local delete' case" [Undecided,New] | 16:01 |
| cdent | jaypipes, bauzas, mriedem I left a longer comment on the spec to remind us of some of these things | 16:01 |
| dansmith | jaypipes: if we end up off the compute node due to shelve or evacuate, won't we heal our way to deleting the allocations after a bit? | 16:01 |
| *** pcaruana has quit IRC | 16:01 | |
| dansmith | jaypipes: also for local delete, I would think | 16:01 |
| cdent | thanks mriedem | 16:01 |
| jaypipes | dansmith: need to look into it. we *may* but wasn't 100% sure. | 16:01 |
| dansmith | jaypipes: I would think we do | 16:02 |
| dansmith | jaypipes: it might take a bit, but.. should happen I think | 16:02 |
| *** hieulq_ has quit IRC | 16:02 | |
| *** efoley has joined #openstack-nova | 16:02 | |
| cdent | the periodic job in the RT is always trying to heal if it sees a gap | 16:02 |
| dansmith | cdent: right | 16:02 |
| cdent | so the problem comes up when there is no RT | 16:02 |
| cdent | s/the/a/ | 16:02 |
| *** voelzmo has quit IRC | 16:03 | |
| dansmith | you mean in a scheduler-claims world? | 16:04 |
| cdent | dansmith: no, in the world where we want the representation of available resources to be correct | 16:05 |
| cdent | it doesn't always have practical impact if a compute nodes is down but in the placement db but it does mean that our global total of eg vcpus is wrong | 16:06 |
| cdent | it would be nice that when a compute node went down , it was removed from the placement db | 16:07 |
| cdent | but since they self register | 16:07 |
| cdent | We don't currently have anything in place to clean up when they die, do we? | 16:07 |
| dansmith | I dunno about that.. for instances that aren't evacuated, those are still valid allocations, yes? | 16:07 |
| dansmith | if you think about when we do quota by counting things in placement, | 16:08 |
| cdent | I was thinking more in the sense of full on node crash | 16:08 |
| dansmith | one compute node going offline shouldn't free up quota for you and then get you in the red once it comes back | 16:08 |
| dansmith | even if the compute node is off and cold, if there are unevacuated (or undeleted) instances there, the allocations should stay, IMHO | 16:08 |
| mriedem | dansmith: i agree with that | 16:08 |
| cdent | the quota case nicely complicates things | 16:09 |
| dansmith | i.e. I wouldn't think placement should know about liveness | 16:09 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1679750 is about when you delete an instance on a dead node | 16:09 |
| openstack | Launchpad bug 1679750 in OpenStack Compute (nova) "Allocations are not cleaned up in placement for instance 'local delete' case" [Undecided,New] | 16:09 |
| *** nic has joined #openstack-nova | 16:09 | |
| dansmith | if nova has an instance, then placement has a record of allocations.. nothing more than that | 16:09 |
| cdent | because if we are using allocations for usage, is a cold node supposed to count against my quota? that's not my (the user's) fault and I want to run some instances | 16:09 |
| *** Jack_Iv has joined #openstack-nova | 16:09 | |
| cdent | s/usage/quota measurment/ | 16:09 |
| dansmith | cdent: it should IMHO yeah | 16:09 |
| dansmith | cdent: you're free to delete the instance and get the quota back | 16:10 |
| dansmith | the flip side is, | 16:10 |
| dansmith | if a compute node goes down, you get more quota and boot a thing, | 16:10 |
| dansmith | when the compute node comes back, should the provider be free to nuke the instance that was running on that node since you'd now be over quota? | 16:10 |
| cdent | hmmm. yeah. weird. | 16:10 |
| cdent | blech | 16:10 |
| mriedem | i think when i get some time i'll write a functional test for the local delete /allocations bug | 16:10 |
| mriedem | back to spec review | 16:11 |
| *** karimb has quit IRC | 16:12 | |
| *** derekh has quit IRC | 16:13 | |
| *** eharney has joined #openstack-nova | 16:16 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs master: SPICE native client support https://review.openstack.org/442040 | 16:19 |
| *** hieulq__ has quit IRC | 16:21 | |
| *** hieulq__ has joined #openstack-nova | 16:22 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Show persisted scheduler hints in server details https://review.openstack.org/440580 | 16:22 |
| *** aarefiev is now known as aarefiev_afk | 16:23 | |
| *** sridharg has quit IRC | 16:23 | |
| *** READ10 has quit IRC | 16:24 | |
| *** hieulq__ has quit IRC | 16:25 | |
| gibi | mriedem, sdague, cdent, jaypipes, johnthetubaguy: hi! I updated the spec https://review.openstack.org/#/c/440580 | 16:25 |
| *** Swami has joined #openstack-nova | 16:25 | |
| gibi | mriedem, sdague, cdent, jaypipes, johnthetubaguy: I tried to answer all the concerns comments | 16:26 |
| jaypipes | gibi: thx Balasz! | 16:26 |
| cdent | gibi++ | 16:26 |
| *** rfolco has joined #openstack-nova | 16:27 | |
| *** Jack_Iv has quit IRC | 16:27 | |
| mriedem | mgagne: thanks for this https://review.openstack.org/#/c/337819/ | 16:28 |
| mriedem | mgagne: not many people take the time to update spec docs after the fact | 16:28 |
| *** dtp has joined #openstack-nova | 16:28 | |
| mgagne | mriedem: hmm, didn't get notification for that one, checking | 16:29 |
| johnthetubaguy | cdent: you are right about limiting the crust at ingress, I think that was the question for me, should we do that first. | 16:29 |
| *** litao has quit IRC | 16:29 | |
| johnthetubaguy | mriedem: ++ on the thank you and your comments there, it came up the other day in a bug I think | 16:30 |
| mgagne | cool, glad to see it got merged =) | 16:31 |
| *** lyan_ is now known as lyan | 16:31 | |
| *** psachin has quit IRC | 16:32 | |
| *** suresh12 has quit IRC | 16:35 | |
| *** suresh12 has joined #openstack-nova | 16:35 | |
| *** annegentle has joined #openstack-nova | 16:36 | |
| *** lucasxu has joined #openstack-nova | 16:38 | |
| *** iceyao has quit IRC | 16:39 | |
| *** mtanino has joined #openstack-nova | 16:39 | |
| *** Apoorva has joined #openstack-nova | 16:39 | |
| *** suresh12 has quit IRC | 16:39 | |
| *** Apoorva has quit IRC | 16:40 | |
| *** Apoorva has joined #openstack-nova | 16:40 | |
| *** gouthamr has quit IRC | 16:41 | |
| *** ltomasbo is now known as ltomasbo|away | 16:41 | |
| *** gouthamr has joined #openstack-nova | 16:45 | |
| *** Jack_Iv has joined #openstack-nova | 16:45 | |
| *** Hosam has joined #openstack-nova | 16:49 | |
| *** Jack_Iv has quit IRC | 16:50 | |
| *** ratailor has quit IRC | 16:50 | |
| *** READ10 has joined #openstack-nova | 16:56 | |
| *** nkorabli has joined #openstack-nova | 16:57 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: encryptors: Always default to the NoOpEncrytor https://review.openstack.org/453265 | 17:03 |
| *** jdillaman has joined #openstack-nova | 17:04 | |
| *** tesseract has quit IRC | 17:04 | |
| *** armax has joined #openstack-nova | 17:08 | |
| *** Jack_Iv has joined #openstack-nova | 17:09 | |
| *** Jack_Iv has quit IRC | 17:15 | |
| *** mvk has quit IRC | 17:15 | |
| *** tbachman has quit IRC | 17:16 | |
| *** timello has joined #openstack-nova | 17:16 | |
| *** vks1 has quit IRC | 17:16 | |
| *** armax has quit IRC | 17:18 | |
| *** amoralej is now known as amoralej|off | 17:20 | |
| *** tbachman has joined #openstack-nova | 17:22 | |
| *** baoli has joined #openstack-nova | 17:25 | |
| mriedem | melwitt: on https://review.openstack.org/#/c/453041/ i know it's just a re-proposal, but most of what that spec is talking about is the old justification for doing it, i.e. rpc versioning and the like. the reason you're re-proposing it is a bit different, for cells v2. i'd like to see something in there about why this is something we want for cells v2 - what cells v2 problem does it solve, etc? | 17:26 |
| *** sambetts is now known as sambetts|afk | 17:27 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Libvirt: Native LUKS file and host device decryption by QEMU https://review.openstack.org/437070 | 17:27 |
| melwitt | mriedem: okay. so add that and remove Previously-approved? | 17:27 |
| *** armax has joined #openstack-nova | 17:27 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272 | 17:27 |
| mriedem | melwitt: previously-approved is fine | 17:28 |
| mriedem | i'm just looking for something about cells v2 in the problem description | 17:28 |
| melwitt | k | 17:28 |
| openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272 | 17:28 |
| mriedem | maybe that's L53... "Because the new tokens will go in the database we need to consider | 17:28 |
| mriedem | cells v2." | 17:28 |
| *** Guest52040 has quit IRC | 17:28 | |
| * mriedem reads on | 17:28 | |
| mriedem | melwitt: so maybe that paragraph answers it | 17:29 |
| mriedem | the url contains the instance uuid | 17:29 |
| mriedem | weren't we going to use that to lookup which cell the token was in? | 17:29 |
| *** voelzmo has joined #openstack-nova | 17:30 | |
| melwitt | yeah, I'm trying to see if that change happened already yet (looking at the old merged vs abandoned patches) | 17:32 |
| melwitt | okay, so that change didn't make it https://review.openstack.org/#/c/325382/ | 17:33 |
| *** lucasxu has quit IRC | 17:33 | |
| *** lucasxu has joined #openstack-nova | 17:34 | |
| *** jvgrant_ has quit IRC | 17:35 | |
| *** jvgrant has joined #openstack-nova | 17:35 | |
| *** voelzmo has quit IRC | 17:36 | |
| mriedem | do we have to worry about old connection urls that don't have the instance uuid so we can't do the lookup for the cell? | 17:37 |
| *** dillaman has joined #openstack-nova | 17:39 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | 17:39 |
| *** voelzmo has joined #openstack-nova | 17:40 | |
| *** lucasxu has quit IRC | 17:40 | |
| *** lucasxu has joined #openstack-nova | 17:41 | |
| *** eharney has quit IRC | 17:41 | |
| *** annegentle has quit IRC | 17:42 | |
| *** Hosam has quit IRC | 17:43 | |
| melwitt | mriedem: so it looks like the idea during the transition period is to have both the consoleauth service running and have new tokens stored in cell databases. so old connection urls will be served by the consoleauth service, which has the tokens globally in memcached | 17:44 |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | 17:44 |
| *** annegentle has joined #openstack-nova | 17:46 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | 17:47 |
| *** mlavalle has quit IRC | 17:49 | |
| melwitt | mriedem: and then I think through normal token expiration, eventually all of the old memcached tokens won't be used anymore, and when that happens it would be safe to stop running the consoleauth service since no outstanding connection urls will be without the server uuid | 17:49 |
| macsz | bugs team meeting (#openstack-meeting-4) in 10 minutes, if anyone is interested | 17:52 |
| *** eharney has joined #openstack-nova | 17:55 | |
| *** prateek has joined #openstack-nova | 17:56 | |
| openstackgerrit | Dan Peschman proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025 | 18:00 |
| *** mvk has joined #openstack-nova | 18:01 | |
| *** tuan_luong has joined #openstack-nova | 18:02 | |
| *** tuanla_ has joined #openstack-nova | 18:02 | |
| *** ralonsoh has quit IRC | 18:04 | |
| mriedem | melwitt: ok is that all in the spec? i didn't read the whole thing b/c i'm lazy | 18:05 |
| mriedem | ah reading your reply | 18:05 |
| *** dillaman has quit IRC | 18:05 | |
| oomichi | gmann: thanks, LGTM now | 18:06 |
| *** dillaman has joined #openstack-nova | 18:06 | |
| melwitt | mriedem: it is, but it's not as detailed as my comment. I sort of derived it from looking at the abandoned patches plus the spec, since I'm not so familiar with consoles. the spec language appears to assume more knowledge | 18:07 |
| mriedem | yeah same here | 18:07 |
| *** efoley has quit IRC | 18:08 | |
| mriedem | you need a phd like paul murray | 18:09 |
| mriedem | anyway, approved | 18:09 |
| melwitt | heh, yeah | 18:09 |
| melwitt | cool, thanks! | 18:09 |
| openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272 | 18:10 |
| *** mdavidson has joined #openstack-nova | 18:11 | |
| *** dillaman has quit IRC | 18:12 | |
| *** felipemonteiro has quit IRC | 18:13 | |
| *** armax has quit IRC | 18:14 | |
| *** ijw_ has joined #openstack-nova | 18:14 | |
| *** felipemonteiro has joined #openstack-nova | 18:19 | |
| *** Sukhdev has joined #openstack-nova | 18:20 | |
| *** suresh12 has joined #openstack-nova | 18:21 | |
| *** sree has joined #openstack-nova | 18:22 | |
| *** suresh12 has quit IRC | 18:25 | |
| *** jamesdenton has joined #openstack-nova | 18:26 | |
| *** dillaman has joined #openstack-nova | 18:26 | |
| *** Apoorva_ has joined #openstack-nova | 18:27 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs master: SPICE native client support https://review.openstack.org/442040 | 18:29 |
| *** Apoorva has quit IRC | 18:30 | |
| *** mdrabe has quit IRC | 18:30 | |
| *** tuanla_ has quit IRC | 18:31 | |
| *** tuan_luong has quit IRC | 18:31 | |
| *** crushil has quit IRC | 18:32 | |
| *** MikeG451 has quit IRC | 18:33 | |
| *** amotoki has quit IRC | 18:34 | |
| *** Jack_Iv has joined #openstack-nova | 18:34 | |
| *** voelzmo has quit IRC | 18:35 | |
| *** Jack_Iv_ has joined #openstack-nova | 18:35 | |
| *** jamesden_ has joined #openstack-nova | 18:35 | |
| *** jamesdenton has quit IRC | 18:36 | |
| *** Jack_Iv has quit IRC | 18:36 | |
| *** Jack_Iv has joined #openstack-nova | 18:37 | |
| *** Jack_Iv_ has quit IRC | 18:37 | |
| *** Jack_Iv_ has joined #openstack-nova | 18:38 | |
| *** Jack_Iv has quit IRC | 18:40 | |
| mriedem | cfriesen: in https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/instance-flavor-api.html how are we representing a flavor that doesn't have any extra specs? are you omitting it from the response, or returning an empty list, or None? | 18:41 |
| *** nkorabli has quit IRC | 18:41 | |
| *** timello has quit IRC | 18:41 | |
| *** artom has quit IRC | 18:42 | |
| *** artom has joined #openstack-nova | 18:42 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Add project/user association to placement https://review.openstack.org/453030 | 18:43 |
| *** MikeG451 has joined #openstack-nova | 18:44 | |
| *** ekuris has joined #openstack-nova | 18:44 | |
| mriedem | looks like it will be {} | 18:45 |
| mriedem | per the api sample | 18:45 |
| *** cdent has quit IRC | 18:46 | |
| *** suresh12 has joined #openstack-nova | 18:47 | |
| *** joanna__ has quit IRC | 18:47 | |
| *** jwcroppe has quit IRC | 18:47 | |
| *** eharney has quit IRC | 18:47 | |
| *** jwcroppe has joined #openstack-nova | 18:48 | |
| *** artom has quit IRC | 18:50 | |
| *** artom has joined #openstack-nova | 18:50 | |
| *** mdrabe has joined #openstack-nova | 18:50 | |
| *** Jack_Iv has joined #openstack-nova | 18:50 | |
| *** satyar has quit IRC | 18:51 | |
| *** Jack_Iv_ has quit IRC | 18:51 | |
| *** jwcroppe has quit IRC | 18:52 | |
| *** avolkov has quit IRC | 18:55 | |
| *** mlavalle has joined #openstack-nova | 18:55 | |
| *** mkoderer has quit IRC | 18:57 | |
| *** suresh12 has quit IRC | 18:57 | |
| *** suresh12 has joined #openstack-nova | 18:58 | |
| *** mnestratov has quit IRC | 18:59 | |
| *** jamesden_ has quit IRC | 18:59 | |
| *** READ10 has quit IRC | 19:00 | |
| *** jamesdenton has joined #openstack-nova | 19:00 | |
| *** eharney has joined #openstack-nova | 19:00 | |
| *** mdrabe has quit IRC | 19:02 | |
| *** suresh12 has quit IRC | 19:02 | |
| *** mdrabe has joined #openstack-nova | 19:04 | |
| *** Jack_Iv_ has joined #openstack-nova | 19:05 | |
| *** dimtruck is now known as zz_dimtruck | 19:05 | |
| mriedem | melwitt: ever noticed that for project_id and user_id in the nova data model we allow 255? | 19:07 |
| mriedem | but keystone only allows 64 | 19:07 |
| *** sree has quit IRC | 19:07 | |
| *** lucasxu has quit IRC | 19:07 | |
| *** Jack_Iv__ has joined #openstack-nova | 19:08 | |
| *** lucasxu has joined #openstack-nova | 19:08 | |
| *** sree has joined #openstack-nova | 19:08 | |
| melwitt | mriedem: no, I didn't know we allow 255 | 19:09 |
| *** Jack_Iv has quit IRC | 19:09 | |
| mriedem | ok questions in https://review.openstack.org/#/c/453030/ | 19:11 |
| *** Jack_Iv_ has quit IRC | 19:11 | |
| *** owalsh_ has joined #openstack-nova | 19:12 | |
| *** sree has quit IRC | 19:12 | |
| melwitt | cool, thanks | 19:13 |
| *** baoli has quit IRC | 19:14 | |
| *** owalsh has quit IRC | 19:15 | |
| *** voelzmo has joined #openstack-nova | 19:17 | |
| *** crushil has joined #openstack-nova | 19:18 | |
| *** voelzmo has quit IRC | 19:19 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: PowerVM Driver: config drive https://review.openstack.org/409404 | 19:20 |
| *** harlowja has joined #openstack-nova | 19:24 | |
| *** owalsh_ is now known as owalsh | 19:26 | |
| *** voelzmo has joined #openstack-nova | 19:27 | |
| *** Jack_Iv__ has quit IRC | 19:28 | |
| *** Jack_Iv has joined #openstack-nova | 19:28 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189 | 19:29 |
| *** READ10 has joined #openstack-nova | 19:31 | |
| mriedem | f yeah down to just 1.5 pages of open spec reviews | 19:32 |
| *** Jack_Iv has quit IRC | 19:32 | |
| *** baoli has joined #openstack-nova | 19:32 | |
| *** Jack_Iv has joined #openstack-nova | 19:35 | |
| *** prateek has quit IRC | 19:36 | |
| *** Jack_Iv has quit IRC | 19:36 | |
| *** krtaylor has quit IRC | 19:45 | |
| *** voelzmo has quit IRC | 19:46 | |
| *** awaugama has quit IRC | 19:47 | |
| *** ducttape_ has joined #openstack-nova | 19:48 | |
| *** suresh12 has joined #openstack-nova | 19:52 | |
| *** cdent has joined #openstack-nova | 19:58 | |
| *** ediardo has quit IRC | 20:00 | |
| mriedem | sdague: this is an easy one to go through https://review.openstack.org/#/c/384261/ - if you agree with my comments, i can update it quick | 20:00 |
| *** ediardo has joined #openstack-nova | 20:01 | |
| sdague | mriedem: after the TC meeting I'll try to look | 20:01 |
| mriedem | ack | 20:01 |
| *** dave-mccowan has quit IRC | 20:03 | |
| *** nkorabli has joined #openstack-nova | 20:05 | |
| *** nkorabli has quit IRC | 20:05 | |
| *** mnestratov has joined #openstack-nova | 20:06 | |
| *** Sukhdev has quit IRC | 20:15 | |
| *** voelzmo has joined #openstack-nova | 20:18 | |
| *** Apoorva_ has quit IRC | 20:19 | |
| *** Apoorva has joined #openstack-nova | 20:19 | |
| *** gouthamr has quit IRC | 20:20 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Add project/user association to placement https://review.openstack.org/453030 | 20:23 |
| *** voelzmo has quit IRC | 20:23 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add additional-default-policy-roles spec https://review.openstack.org/427872 | 20:23 |
| *** ducttape_ has quit IRC | 20:24 | |
| *** suresh12 has quit IRC | 20:30 | |
| *** cleong has quit IRC | 20:30 | |
| *** armax has joined #openstack-nova | 20:30 | |
| *** suresh12 has joined #openstack-nova | 20:30 | |
| *** ducttape_ has joined #openstack-nova | 20:34 | |
| *** suresh12 has quit IRC | 20:35 | |
| *** thorst has quit IRC | 20:35 | |
| *** smatzek has quit IRC | 20:37 | |
| *** gouthamr has joined #openstack-nova | 20:38 | |
| *** mdrabe has quit IRC | 20:40 | |
| *** mdrabe has joined #openstack-nova | 20:48 | |
| *** jamesdenton has quit IRC | 20:50 | |
| *** zz_dimtruck is now known as dimtruck | 20:53 | |
| *** rcernin has quit IRC | 20:55 | |
| *** krtaylor has joined #openstack-nova | 20:59 | |
| *** jamesdenton has joined #openstack-nova | 21:00 | |
| *** marst has quit IRC | 21:00 | |
| *** marst has joined #openstack-nova | 21:01 | |
| *** Apoorva_ has joined #openstack-nova | 21:06 | |
| *** thorst has joined #openstack-nova | 21:07 | |
| mriedem | random filter question, | 21:07 |
| mriedem | i'm sure we have something that handles this, | 21:07 |
| sdague | mriedem: https://review.openstack.org/#/c/384261/6/specs/pike/approved/deprecate-multinic-proxy-api.rst ... yeh, I think your comments are fine there | 21:08 |
| mriedem | but if i have a set of host aggregates with a certain storage backend that has restrictions on it, and i want my flavors tied to that, i can do it with extra specs so the flavors are tied to those host aggregates, but how to i keep other flavors that don't have the extra spec from landing on those hosts? | 21:08 |
| mriedem | which filter does that? | 21:08 |
| *** Apoorva_ has quit IRC | 21:09 | |
| * mriedem spins the wheel of scheduler filters | 21:09 | |
| mriedem | sdague: ok i'll update it | 21:09 |
| mriedem | sdague: although at this point in the day i suppose you're done | 21:09 |
| *** Apoorva_ has joined #openstack-nova | 21:09 | |
| mriedem | i can let alex_xu update it | 21:09 |
| *** Apoorva has quit IRC | 21:10 | |
| melwitt | mriedem: I thought it would be AggregateInstanceExtraSpecsFilter | 21:10 |
| sdague | mriedem: either way, the fam is out at the store, so I have a few minutes until they get back | 21:10 |
| *** marst has quit IRC | 21:10 | |
| mriedem | melwitt: yeah but AggregateInstanceExtraSpecsFilter returns True if the flavor doesn't have extra specs too | 21:11 |
| melwitt | hm, I was going off the docstring "Check that the extra specs associated with the instance type match the metadata provided by aggregates. If not present return False." | 21:11 |
| mriedem | melwitt: i think that's if the extra spec key is not in the aggregate metadata | 21:12 |
| mriedem | if not aggregate_vals: | 21:12 |
| *** suresh12 has joined #openstack-nova | 21:12 | |
| melwitt | right, doesn't that mean if no match, host_passes is False? but I do see it returns True if extra_specs is entirely empty | 21:13 |
| *** artom has quit IRC | 21:13 | |
| *** artom has joined #openstack-nova | 21:14 | |
| mriedem | yeah | 21:14 |
| mriedem | so if the flavor has extra specs but they aren't in the aggregate meta, then host does not pass | 21:14 |
| mriedem | if the flavor doesn't have extra specs at all, then host passes | 21:14 |
| mriedem | that's the case i'm wondering about | 21:14 |
| melwitt | yeah, and I remember the latter was a relatively new addition. "relatively new" meaning I remember when it got added | 21:15 |
| melwitt | lemme see | 21:15 |
| mriedem | so the only way to really restrict everything would be to put extra specs on all of your flavors | 21:15 |
| mriedem | i think | 21:15 |
| *** awaugama has joined #openstack-nova | 21:16 | |
| *** dave-mccowan has joined #openstack-nova | 21:16 | |
| melwitt | https://review.openstack.org/#/c/241016/ was where it got added | 21:17 |
| *** READ10 has quit IRC | 21:17 | |
| mriedem | crazy yahoo people | 21:17 |
| melwitt | yeah, I think you're right. I'm trying to see if the review and/or bug mention how that's supposed to be okay | 21:17 |
| melwitt | heh | 21:18 |
| *** vladikr has quit IRC | 21:18 | |
| *** dane-fichter has quit IRC | 21:19 | |
| melwitt | I guess, if the flavor has no extra_specs at all, it's like saying "this is okay anywhere" | 21:20 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: nova-net: remove get_instance_nw_info from API subclass https://review.openstack.org/453336 | 21:20 |
| mriedem | yeah | 21:20 |
| melwitt | like, this flavor is okay to be anywhere because it has no special requirements | 21:20 |
| mriedem | even if it might land on a host that can't support it | 21:20 |
| mriedem | the flavor might not have requirements, but the host might | 21:20 |
| melwitt | well, if it has support requirements, it would have an extra_specs of some kind, no? | 21:21 |
| melwitt | oh | 21:21 |
| mriedem | this is for the scaleio ephemeral backend spec, | 21:21 |
| mriedem | which requires that root_gb on the flavor be in multiples of 8 | 21:21 |
| mriedem | in a mixed compute environment, you'd have flavors with extra specs that fit that and have an extra spec to hit those scaleio host aggregates, | 21:21 |
| mriedem | i'm trying to figure out how you get non-specific flavors for scaleio to *not* land on those host aggregates | 21:22 |
| melwitt | I see | 21:22 |
| *** cdent has quit IRC | 21:22 | |
| mriedem | in reality anyone using this is probably using all scaleio all the time everywhere b/c they are buying a converged architecture product from EMC | 21:22 |
| mriedem | and using anything else voids their support contract with EMC :) | 21:23 |
| melwitt | :) | 21:23 |
| openstackgerrit | Emilien Macchi proposed openstack/nova master: Add sem-ver flag so pbr generates correct version https://review.openstack.org/453338 | 21:23 |
| EmilienM | mriedem: ^ is it something you would accept? | 21:24 |
| mriedem | EmilienM: is there supposed to be a code change with that? | 21:24 |
| EmilienM | mriedem: nope | 21:24 |
| EmilienM | mriedem: https://github.com/openstack/ironic/blob/master/doc/source/dev/releasing.rst#things-to-do-before-releasing | 21:25 |
| EmilienM | the semver thing is just in the commit message | 21:25 |
| EmilienM | "sem-ver: feature" bumps Y | 21:26 |
| EmilienM | "sem-ver: api-break" bumps X | 21:26 |
| EmilienM | (if your version is X.Y) | 21:26 |
| mriedem | but pike is 16.0.0 | 21:26 |
| mriedem | ocata was 15.0.0 | 21:26 |
| mriedem | newton is 14.0.0 | 21:27 |
| mriedem | etc | 21:27 |
| EmilienM | ok so we need to bump X I guess | 21:27 |
| mriedem | didn't this used to all be handled in setup.cfg? | 21:28 |
| openstackgerrit | Emilien Macchi proposed openstack/nova master: Add sem-ver flag so pbr generates correct version https://review.openstack.org/453338 | 21:28 |
| mriedem | we also have tags in the release repo | 21:28 |
| EmilienM | this thing helps us to update packages without the tag | 21:28 |
| EmilienM | so our tooling can find out there is a new version from pbr | 21:29 |
| *** Guest63262 has quit IRC | 21:29 | |
| EmilienM | afik historically nova was releasing only X.0.Z++ | 21:29 |
| mriedem | generally yes | 21:30 |
| mriedem | i think we've bumped a minor version on a stable branch when we had to pull in a new dependency | 21:30 |
| mriedem | or bump a minimum dependent package version for a security fix, something like that | 21:30 |
| mriedem | and you're only doing this for nova and ironic? | 21:31 |
| EmilienM | I'm going to do it for all projects | 21:31 |
| *** edmondsw has quit IRC | 21:31 | |
| EmilienM | so tripleo CI can start testing upgrades | 21:32 |
| EmilienM | I need to leave irc now, feel free to comment or push over the patch | 21:32 |
| EmilienM | I'll catch-up later | 21:32 |
| *** rfolco has quit IRC | 21:33 | |
| *** edmondsw has joined #openstack-nova | 21:33 | |
| *** Guest63262 has joined #openstack-nova | 21:33 | |
| *** jamesdenton has quit IRC | 21:34 | |
| edleafe | charge/. | 21:34 |
| *** david-lyle has quit IRC | 21:35 | |
| *** peter-hamilton has quit IRC | 21:35 | |
| *** jamesdenton has joined #openstack-nova | 21:37 | |
| *** edmondsw has quit IRC | 21:38 | |
| *** jamesdenton has quit IRC | 21:39 | |
| mriedem | so reading https://docs.openstack.org/developer/pbr/#version we do postversioning since we don't have version in setup.cfg, | 21:40 |
| mriedem | when pbr walks the git tree it finds the last tag, which is 15.0.2 | 21:40 |
| mriedem | and gives some dev version after that i assume | 21:40 |
| mriedem | A missing Sem-Ver line is equivalent to Sem-Ver: bugfix. The bugfix symbol causes a patch level increment to the version. | 21:41 |
| mriedem | so i guess you'd get 15.0.3 actually | 21:41 |
| mriedem | and because we're in pike, that's wrong b/c pike should be 16.0.0 | 21:41 |
| *** dimtruck is now known as zz_dimtruck | 21:41 | |
| *** annegentle has quit IRC | 21:43 | |
| *** david-lyle has joined #openstack-nova | 21:43 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: docs: update description for AggregateInstanceExtraSpecsFilter https://review.openstack.org/453349 | 21:44 |
| *** annegentle has joined #openstack-nova | 21:44 | |
| *** annegentle has quit IRC | 21:45 | |
| mriedem | melwitt: so maybe if you have a mixed compute env, | 21:49 |
| mriedem | you have to put all of the hosts into aggregates, | 21:49 |
| mriedem | one for scaleio-specific, and one for non-scaleio specific, | 21:49 |
| mriedem | and then all of your flavors have to have extra specs that tie them to either of the aggregates | 21:49 |
| mriedem | that seems like the only way around it | 21:50 |
| *** artom has quit IRC | 21:53 | |
| *** artom has joined #openstack-nova | 21:54 | |
| *** smatzek has joined #openstack-nova | 21:58 | |
| *** esberglu has quit IRC | 21:58 | |
| *** thorst has quit IRC | 22:01 | |
| *** felipemonteiro has quit IRC | 22:02 | |
| *** Sukhdev has joined #openstack-nova | 22:03 | |
| *** mdrabe has quit IRC | 22:03 | |
| mriedem | melwitt: jaypipes: dansmith: i'm +2 on https://review.openstack.org/#/c/453030/ - the rest api impact section could call out the need for a microversion bump, but that could be done in a follow up, or just implied | 22:03 |
| dansmith | I haven't had a chance to look at it yet | 22:04 |
| *** Apoorva_ has quit IRC | 22:04 | |
| *** eharney has quit IRC | 22:04 | |
| *** Apoorva has joined #openstack-nova | 22:04 | |
| *** priteau has quit IRC | 22:04 | |
| *** mnestratov has quit IRC | 22:05 | |
| *** zz_dimtruck is now known as dimtruck | 22:08 | |
| edleafe | charge/.charge/. | 22:09 |
| edleafe | stupid vm windows | 22:09 |
| *** smatzek has quit IRC | 22:12 | |
| *** esberglu has joined #openstack-nova | 22:13 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | 22:17 |
| melwitt | mriedem: yeah, agreed. the only other thing I thought of is if we added a near copy of the AggregateInstanceExtraSpecsFilter that _doesn't_ do the empty extra_specs pass thing and operators would have to choose either/or, depending on the behavior desired | 22:19 |
| mriedem | dansmith: i could use some input on this remove bdm device from api spec too, since it's getting caught up in some legacy kerfuffle https://review.openstack.org/#/c/452546/ | 22:22 |
| mriedem | net is bfv with image bdm overrides f'cks it all up | 22:23 |
| *** eharney has joined #openstack-nova | 22:23 | |
| *** baoli has quit IRC | 22:23 | |
| mriedem | but, we could remove device from volume attach api i think since that doesn't have the image bdm over bfv mess | 22:23 |
| dansmith | oh I thought this was the one john had commented on, | 22:23 |
| dansmith | but this is different | 22:24 |
| mriedem | there was never a spec for this, | 22:24 |
| mriedem | we've talked about it a billion times | 22:24 |
| mriedem | and i thought someone was going to write a spec, but didn't, so i posted to the ML over the weekend and wrote a spec, | 22:24 |
| mriedem | to which ftersin pointed out this legacy hack for ec2 | 22:24 |
| dansmith | yeah | 22:25 |
| mriedem | artom: do you know if, off the top of your head, we snapshot a volume backed instance that has bdm tags, do the bdm tags also go into the block_device_mapping_v2 image metadata? | 22:25 |
| mriedem | so that when you boot a new server from that snapshot image, it does bfv with a tagged bdm? | 22:25 |
| *** lucasxu has quit IRC | 22:27 | |
| dansmith | guh what a mess | 22:30 |
| mriedem | heh yeah totally | 22:31 |
| dansmith | so I was thinking what mdbooth said before I got down and saw his comment, | 22:31 |
| dansmith | but your reply doesn't make sense to me | 22:32 |
| dansmith | I think his point is to just leave device_name in the api, but make it clear in all the docs that it is in no way related to the block device name in the guest | 22:32 |
| dansmith | meaning, change the docs and examples to something like "disk1" and such | 22:32 |
| mriedem | i meant rather than correlate on device_name, we correlate on tag | 22:34 |
| mriedem | we do actually set a device_name in the bdm based on what libvirt calculates it to be | 22:35 |
| dansmith | we overwrite what the user gave us? | 22:35 |
| mriedem | so i think having something in the api that says this volume is attached at /dev/vdb but it's actually not, is part of the problem, and why we wanted to remove it | 22:35 |
| mriedem | dansmith: yes | 22:35 |
| mriedem | the libvirt driver completely ignores what the user specified | 22:35 |
| mriedem | since liberty | 22:35 |
| *** awaugama has quit IRC | 22:35 | |
| mriedem | as noted in the spec | 22:35 |
| dansmith | but libvirt doesn't know what it is in the guest | 22:36 |
| *** eharney has quit IRC | 22:36 | |
| dansmith | it knows that it thinks it will be vdb in the guest based on where it got attached, but the guest chooses that | 22:36 |
| mriedem | then why do we even call https://review.openstack.org/#/c/189632/10/nova/virt/libvirt/driver.py@6802 ? | 22:36 |
| mriedem | i guess because we need to have *something* in the bdm table b/c it's used for a shitty primary key | 22:37 |
| dansmith | to populate that field with something semi recognizeable, I guess | 22:37 |
| dansmith | but it's definitely not the right thing even in that case | 22:37 |
| mriedem | ok | 22:38 |
| mriedem | so we could just throw a uuid in bdm.device_name | 22:38 |
| dansmith | as you say we _could_ do away with all of this and use the tag for sure | 22:38 |
| mriedem | and it wouldn't matter | 22:38 |
| dansmith | right, we could | 22:38 |
| dansmith | but the least amount of change would be to just be more ambiguous about what "device name" is I think | 22:39 |
| dansmith | although we should ask a xenapi person, | 22:39 |
| dansmith | as in xen it does actually match | 22:39 |
| dansmith | or can, or did, or whatever | 22:39 |
| *** nic has left #openstack-nova | 22:39 | |
| *** nic has joined #openstack-nova | 22:39 | |
| dansmith | and libvirt with xen may try to make it match, I have no idea | 22:39 |
| mriedem | the least amount of change will be for me to just abandon the spec :) | 22:40 |
| dansmith | hah | 22:40 |
| mriedem | if the only thing we care about is this dumb image bdm override use case, wouldn't it be incrementally better to at least remove device from the attach volume api? | 22:40 |
| *** tommylikehu_ has joined #openstack-nova | 22:40 | |
| mriedem | or is that not even worth it? | 22:40 |
| dansmith | it didn't seem clear to me that this wasn't also a problem for ec2 in that api, | 22:41 |
| dansmith | but I have a hard time understanding his words | 22:41 |
| dansmith | but yeah, if he doesn't care about that, it'd be worth doing I think | 22:42 |
| mriedem | yeah i have a hard time following what the ec2api is doing or not | 22:42 |
| mriedem | i do'nt know if it's merging the bdms before we hit the compute api or what | 22:42 |
| dansmith | yeah I dunno | 22:42 |
| mriedem | and i'll be god damned if i'm going to go code spelunking | 22:42 |
| *** hoonetorg has quit IRC | 22:43 | |
| *** hoonetorg has joined #openstack-nova | 22:44 | |
| *** esberglu has quit IRC | 22:45 | |
| *** hoonetorg has quit IRC | 22:47 | |
| *** Guest63262 has quit IRC | 22:48 | |
| mriedem | artom: found the answer to my question, we don't include bdm tags in snapshots | 22:51 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/block_device.py#L290 | 22:51 |
| mriedem | we'd have to add it to copy_from_template | 22:51 |
| mriedem | would one expect a new server created from the image snapshot with the volume snapshot bdm to have the original bdm tag? i don't know. does anyone care? probably not. | 22:52 |
| *** catintheroof has quit IRC | 22:57 | |
| *** tommylikehu_ has quit IRC | 22:57 | |
| *** hoonetorg has joined #openstack-nova | 22:58 | |
| *** Guest63262 has joined #openstack-nova | 22:59 | |
| *** thorst has joined #openstack-nova | 23:01 | |
| *** lyan has quit IRC | 23:06 | |
| *** thorst has quit IRC | 23:06 | |
| openstackgerrit | Sujitha proposed openstack/nova master: Add sync_instances action in os-cells API https://review.openstack.org/453366 | 23:06 |
| *** suresh12 has quit IRC | 23:21 | |
| artom | mriedem, always happy to have things work out in my absence;) | 23:28 |
| *** chyka has quit IRC | 23:29 | |
| *** dillaman has quit IRC | 23:30 | |
| *** mlavalle has quit IRC | 23:32 | |
| *** dimtruck is now known as zz_dimtruck | 23:35 | |
| *** suresh12 has joined #openstack-nova | 23:36 | |
| openstackgerrit | Sujitha proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779 | 23:37 |
| *** mtanino has quit IRC | 23:39 | |
| *** zz_dimtruck is now known as dimtruck | 23:46 | |
| *** ducttape_ has quit IRC | 23:46 | |
| *** ducttape_ has joined #openstack-nova | 23:46 | |
| *** dillaman has joined #openstack-nova | 23:47 | |
| *** ducttape_ has quit IRC | 23:48 | |
| *** ducttape_ has joined #openstack-nova | 23:48 | |
| *** jianghuaw has quit IRC | 23:48 | |
| *** Swami has quit IRC | 23:49 | |
| *** _ducttape_ has joined #openstack-nova | 23:50 | |
| *** ducttape_ has quit IRC | 23:50 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!