*** ckopper has joined #openstack-nova | 00:00 | |
*** ckopper has quit IRC | 00:00 | |
*** ijw__ has quit IRC | 00:01 | |
*** suresh12 has quit IRC | 00:02 | |
*** suresh12 has joined #openstack-nova | 00:05 | |
*** Apoorva_ has quit IRC | 00:05 | |
*** gmann has quit IRC | 00:06 | |
*** gmann has joined #openstack-nova | 00:06 | |
*** Apoorva has joined #openstack-nova | 00:06 | |
*** openstack has joined #openstack-nova | 00:17 | |
*** hieulq has joined #openstack-nova | 00:17 | |
*** sambetts_ has quit IRC | 00:17 | |
*** rha has quit IRC | 00:17 | |
*** patriciadomin_ has quit IRC | 00:17 | |
*** aignatov has quit IRC | 00:17 | |
*** breton has joined #openstack-nova | 00:19 | |
*** edmondsw has quit IRC | 00:19 | |
*** sambetts_ has joined #openstack-nova | 00:19 | |
*** aignatov has joined #openstack-nova | 00:20 | |
*** patriciadomin has joined #openstack-nova | 00:20 | |
*** tbachman has quit IRC | 00:20 | |
*** tbachman_ is now known as tbachman | 00:20 | |
*** rha has joined #openstack-nova | 00:20 | |
*** rha has quit IRC | 00:20 | |
*** rha has joined #openstack-nova | 00:20 | |
*** igordcard has joined #openstack-nova | 00:20 | |
*** PramodJ has quit IRC | 00:22 | |
*** inara has quit IRC | 00:23 | |
*** john51 has joined #openstack-nova | 00:23 | |
*** s1061123 has joined #openstack-nova | 00:23 | |
*** strigazi has joined #openstack-nova | 00:24 | |
*** 7JTABFHYX has joined #openstack-nova | 00:24 | |
*** zigo has joined #openstack-nova | 00:24 | |
*** gyee has joined #openstack-nova | 00:24 | |
*** vipul has joined #openstack-nova | 00:24 | |
*** 7JTABFHYX has quit IRC | 00:24 | |
*** szaher has joined #openstack-nova | 00:25 | |
*** thorst has joined #openstack-nova | 00:26 | |
*** thorst has quit IRC | 00:27 | |
*** tbachman_ has joined #openstack-nova | 00:29 | |
*** inara has joined #openstack-nova | 00:30 | |
*** yingjun has joined #openstack-nova | 00:32 | |
*** tbachman has quit IRC | 00:33 | |
*** tbachman_ is now known as tbachman | 00:33 | |
*** ckopper has joined #openstack-nova | 00:33 | |
*** Apoorva_ has joined #openstack-nova | 00:35 | |
openstackgerrit | Merged openstack/nova master: api-ref: cleanup PUT /os-hypervisors/statistics docs https://review.openstack.org/474250 | 00:35 |
---|---|---|
*** Apoorva has quit IRC | 00:37 | |
*** jaosorior has joined #openstack-nova | 00:39 | |
*** Apoorva_ has quit IRC | 00:39 | |
*** ijw has joined #openstack-nova | 00:41 | |
*** amotoki_away is now known as amotoki | 00:41 | |
*** gbarros has quit IRC | 00:41 | |
*** ijw has quit IRC | 00:43 | |
*** ijw has joined #openstack-nova | 00:43 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938 | 00:45 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938 | 00:48 |
*** Apoorva has joined #openstack-nova | 00:49 | |
*** claudiub has quit IRC | 00:52 | |
*** acormier has quit IRC | 00:52 | |
*** acormier has joined #openstack-nova | 00:52 | |
*** Apoorva has quit IRC | 00:53 | |
*** Apoorva has joined #openstack-nova | 00:54 | |
*** hoonetorg has quit IRC | 00:54 | |
*** amotoki is now known as amotoki_away | 00:55 | |
*** mdnadeem has joined #openstack-nova | 00:56 | |
*** tonyb_ is now known as tonyb | 00:57 | |
*** dtp has quit IRC | 00:58 | |
*** Apoorva has quit IRC | 00:58 | |
alex_xu | mriedem: sorry, I commented at wrong place. We can implement that check in the python code. | 00:59 |
*** ijw has quit IRC | 01:02 | |
*** gbarros has joined #openstack-nova | 01:03 | |
*** harlowja has joined #openstack-nova | 01:04 | |
*** ijw has joined #openstack-nova | 01:09 | |
*** hoonetorg has joined #openstack-nova | 01:11 | |
*** sree has joined #openstack-nova | 01:11 | |
*** xyang1 has quit IRC | 01:12 | |
*** liujiong has joined #openstack-nova | 01:13 | |
*** ijw has quit IRC | 01:15 | |
*** sree has quit IRC | 01:16 | |
*** zzzeek has quit IRC | 01:18 | |
*** zzzeek has joined #openstack-nova | 01:18 | |
*** yangyapeng has joined #openstack-nova | 01:20 | |
*** mingyu has joined #openstack-nova | 01:20 | |
*** ijw has joined #openstack-nova | 01:21 | |
*** StevenK_ is now known as StevenK | 01:22 | |
*** ijw has quit IRC | 01:26 | |
*** markvoelker has quit IRC | 01:27 | |
*** Kevin_Zheng has joined #openstack-nova | 01:30 | |
*** hoonetorg has quit IRC | 01:30 | |
*** Swami has quit IRC | 01:31 | |
*** xinliang has quit IRC | 01:31 | |
*** zz_dimtruck is now known as dimtruck | 01:32 | |
*** smatzek has joined #openstack-nova | 01:32 | |
*** mriedem has quit IRC | 01:33 | |
*** gbarros has quit IRC | 01:33 | |
*** xinliang has joined #openstack-nova | 01:34 | |
*** xinliang has quit IRC | 01:34 | |
*** xinliang has joined #openstack-nova | 01:34 | |
*** rajathag_ has quit IRC | 01:37 | |
*** thorst has joined #openstack-nova | 01:38 | |
*** coreywright has quit IRC | 01:38 | |
*** rfolco has joined #openstack-nova | 01:39 | |
*** ijw has joined #openstack-nova | 01:40 | |
*** suresh12 has quit IRC | 01:44 | |
*** xinliang_ has joined #openstack-nova | 01:44 | |
*** yangyape_ has joined #openstack-nova | 01:48 | |
*** ijw has quit IRC | 01:48 | |
*** mingyu has quit IRC | 01:49 | |
*** yangyapeng has quit IRC | 01:52 | |
*** hoonetorg has joined #openstack-nova | 01:52 | |
*** yamahata_ has quit IRC | 01:53 | |
*** acormier has quit IRC | 01:54 | |
*** acormier has joined #openstack-nova | 01:54 | |
*** litao__ has joined #openstack-nova | 01:55 | |
*** thorst has quit IRC | 01:56 | |
*** yangyape_ has quit IRC | 01:56 | |
*** yangyapeng has joined #openstack-nova | 01:56 | |
*** thorst has joined #openstack-nova | 01:56 | |
*** coreywright has joined #openstack-nova | 01:56 | |
*** hoonetorg has quit IRC | 01:58 | |
*** acormier has quit IRC | 01:59 | |
*** esberglu has quit IRC | 02:00 | |
*** thorst has quit IRC | 02:00 | |
*** amotoki_away is now known as amotoki | 02:01 | |
*** xinliang has quit IRC | 02:02 | |
*** edmondsw has joined #openstack-nova | 02:03 | |
*** tiendc has joined #openstack-nova | 02:05 | |
*** edmondsw has quit IRC | 02:08 | |
*** mingyu has joined #openstack-nova | 02:10 | |
*** amotoki is now known as amotoki_away | 02:11 | |
*** hoonetorg has joined #openstack-nova | 02:12 | |
Kevin_Zheng | ping mriedem | 02:16 |
*** yamamoto has joined #openstack-nova | 02:17 | |
*** armax has joined #openstack-nova | 02:19 | |
*** acormier has joined #openstack-nova | 02:19 | |
*** _ioni is now known as ioni | 02:21 | |
*** gomarivera has joined #openstack-nova | 02:23 | |
*** yassine has quit IRC | 02:24 | |
*** yassine has joined #openstack-nova | 02:24 | |
*** Guest8722 is now known as sdake | 02:29 | |
*** sdake has joined #openstack-nova | 02:29 | |
*** hongbin has joined #openstack-nova | 02:31 | |
*** raginbajin has quit IRC | 02:33 | |
*** masber has joined #openstack-nova | 02:34 | |
*** amotoki_away is now known as amotoki | 02:36 | |
*** masuberu has quit IRC | 02:37 | |
*** mingyu has quit IRC | 02:37 | |
*** mingyu has joined #openstack-nova | 02:37 | |
*** masuberu has joined #openstack-nova | 02:38 | |
*** raginbajin has joined #openstack-nova | 02:39 | |
*** gyee has quit IRC | 02:41 | |
*** masber has quit IRC | 02:42 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix to return server groups quota in quota class set https://review.openstack.org/467999 | 02:44 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix to return server groups quota in quota class set https://review.openstack.org/467999 | 02:50 |
*** gomarivera has quit IRC | 02:58 | |
*** gongysh has joined #openstack-nova | 02:59 | |
*** gomarivera has joined #openstack-nova | 03:01 | |
*** armax has quit IRC | 03:04 | |
*** smatzek has quit IRC | 03:05 | |
*** armax has joined #openstack-nova | 03:10 | |
*** wxy has joined #openstack-nova | 03:11 | |
*** acormier has quit IRC | 03:12 | |
openstackgerrit | Moshe Levi proposed openstack/os-vif master: hardware offload support for openvswitch https://review.openstack.org/460278 | 03:12 |
*** acormier has joined #openstack-nova | 03:12 | |
*** rfolco has quit IRC | 03:13 | |
*** mingyu has quit IRC | 03:14 | |
*** moshele has joined #openstack-nova | 03:15 | |
*** acormier has quit IRC | 03:17 | |
*** moshele has quit IRC | 03:17 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568 | 03:18 |
*** mingyu has joined #openstack-nova | 03:23 | |
*** amotoki is now known as amotoki_away | 03:24 | |
*** nicolasbock has joined #openstack-nova | 03:27 | |
*** markvoelker has joined #openstack-nova | 03:28 | |
*** tristanC has quit IRC | 03:29 | |
*** tristanC has joined #openstack-nova | 03:31 | |
*** sridharg has joined #openstack-nova | 03:34 | |
*** diga has joined #openstack-nova | 03:43 | |
*** gomarivera has quit IRC | 03:48 | |
*** gomarivera has joined #openstack-nova | 03:49 | |
*** mingyu has quit IRC | 03:51 | |
*** edmondsw has joined #openstack-nova | 03:51 | |
*** ratailor has joined #openstack-nova | 03:52 | |
*** ratailor has quit IRC | 03:52 | |
*** ratailor has joined #openstack-nova | 03:55 | |
*** edmondsw has quit IRC | 03:56 | |
*** thorst has joined #openstack-nova | 03:57 | |
*** links has joined #openstack-nova | 03:58 | |
gmann | alex_xu, oomichi updated quota class fix, check now - https://review.openstack.org/#/c/467999/ | 04:00 |
*** catintheroof has quit IRC | 04:01 | |
*** thorst has quit IRC | 04:01 | |
*** gouthamr has quit IRC | 04:03 | |
*** mingyu has joined #openstack-nova | 04:04 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add missing parameters in limits.inc https://review.openstack.org/475640 | 04:05 |
*** mingyu has quit IRC | 04:06 | |
*** mingyu has joined #openstack-nova | 04:08 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/475563 | 04:12 |
*** qwertyco has joined #openstack-nova | 04:13 | |
*** mingyu has quit IRC | 04:13 | |
*** hongbin has quit IRC | 04:17 | |
*** gomarivera has quit IRC | 04:29 | |
*** gomarivera has joined #openstack-nova | 04:30 | |
*** takashin has left #openstack-nova | 04:30 | |
*** gomarivera has quit IRC | 04:33 | |
*** gomarivera has joined #openstack-nova | 04:33 | |
*** adisky__ has joined #openstack-nova | 04:34 | |
*** jerrygb has joined #openstack-nova | 04:36 | |
*** mdrabe has quit IRC | 04:40 | |
alex_xu | gmann: cool, will try to reach it asap | 04:43 |
gmann | alex_xu, thanks | 04:43 |
alex_xu | gmann: np, thanks for the patch also | 04:43 |
*** amotoki_away is now known as amotoki | 04:48 | |
*** salv-orlando has joined #openstack-nova | 04:48 | |
*** salv-orlando has quit IRC | 04:49 | |
*** salv-orlando has joined #openstack-nova | 04:49 | |
*** salv-orlando has quit IRC | 04:51 | |
*** salv-orlando has joined #openstack-nova | 04:51 | |
*** gongysh has quit IRC | 04:56 | |
*** yamahata_ has joined #openstack-nova | 04:58 | |
*** amotoki is now known as amotoki_away | 05:00 | |
*** jerrygb_ has joined #openstack-nova | 05:01 | |
*** jerrygb has quit IRC | 05:03 | |
*** psachin has joined #openstack-nova | 05:09 | |
*** pcaruana has joined #openstack-nova | 05:14 | |
*** salv-orl_ has joined #openstack-nova | 05:15 | |
*** ltomasbo|away is now known as ltomasbo | 05:17 | |
*** salv-orlando has quit IRC | 05:18 | |
*** moshele has joined #openstack-nova | 05:20 | |
*** gomarivera has quit IRC | 05:20 | |
*** gomarivera has joined #openstack-nova | 05:21 | |
*** prateek has joined #openstack-nova | 05:21 | |
*** gomarivera has quit IRC | 05:25 | |
*** sree has joined #openstack-nova | 05:29 | |
*** pcaruana has quit IRC | 05:30 | |
*** gongysh has joined #openstack-nova | 05:30 | |
*** vks1 has joined #openstack-nova | 05:35 | |
*** frickler_ is now known as frickler | 05:35 | |
*** edmondsw has joined #openstack-nova | 05:39 | |
*** edmondsw has quit IRC | 05:44 | |
*** suresh12 has joined #openstack-nova | 05:46 | |
*** ioni is now known as _ioni | 05:55 | |
*** _ioni is now known as ioni | 05:55 | |
*** thorst has joined #openstack-nova | 05:58 | |
*** markvoelker has quit IRC | 06:01 | |
*** thorst has quit IRC | 06:02 | |
*** gaurangt has quit IRC | 06:05 | |
*** gaurangt has joined #openstack-nova | 06:05 | |
*** bkopilov has quit IRC | 06:07 | |
*** armax has quit IRC | 06:10 | |
*** Oku_OS-away is now known as Oku_OS | 06:14 | |
*** markus_z has joined #openstack-nova | 06:23 | |
*** d3n14l has joined #openstack-nova | 06:26 | |
*** claudiub has joined #openstack-nova | 06:26 | |
*** armax has joined #openstack-nova | 06:27 | |
*** d3n14l has left #openstack-nova | 06:30 | |
*** fnordahl_ is now known as fnordahl | 06:31 | |
*** udesale has joined #openstack-nova | 06:31 | |
*** udesale has quit IRC | 06:31 | |
*** amotoki_away is now known as amotoki | 06:31 | |
*** udesale has joined #openstack-nova | 06:32 | |
*** abalutoiu_ has quit IRC | 06:34 | |
*** artom has joined #openstack-nova | 06:35 | |
*** tesseract has joined #openstack-nova | 06:40 | |
*** gongysh has quit IRC | 06:41 | |
*** qwertyco has quit IRC | 06:49 | |
*** suresh12 has quit IRC | 06:49 | |
*** amotoki is now known as amotoki_away | 06:49 | |
*** links has quit IRC | 06:49 | |
*** pcaruana has joined #openstack-nova | 06:49 | |
*** rcernin_ has joined #openstack-nova | 06:53 | |
*** cNilesh has joined #openstack-nova | 06:55 | |
*** salv-orl_ has quit IRC | 07:01 | |
*** salv-orlando has joined #openstack-nova | 07:02 | |
*** gongysh has joined #openstack-nova | 07:04 | |
*** links has joined #openstack-nova | 07:05 | |
*** amotoki_away is now known as amotoki | 07:06 | |
*** jgrassle1 is now known as jgrassler | 07:06 | |
*** salv-orlando has quit IRC | 07:07 | |
*** gaurangt has quit IRC | 07:08 | |
*** salv-orlando has joined #openstack-nova | 07:09 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568 | 07:10 |
*** mingyu has joined #openstack-nova | 07:13 | |
*** alexchadin has joined #openstack-nova | 07:13 | |
*** mingyu has quit IRC | 07:14 | |
*** gaurangt has joined #openstack-nova | 07:14 | |
*** rmart04 has joined #openstack-nova | 07:15 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix to return server groups quota in quota class set https://review.openstack.org/467999 | 07:17 |
*** yamahata_ has quit IRC | 07:18 | |
*** ijw has joined #openstack-nova | 07:21 | |
*** amotoki is now known as amotoki_away | 07:22 | |
*** abalutoiu has joined #openstack-nova | 07:24 | |
*** ijw has quit IRC | 07:25 | |
*** edmondsw has joined #openstack-nova | 07:27 | |
*** amotoki_away is now known as amotoki | 07:29 | |
*** dtantsur|afk is now known as dtantsur | 07:30 | |
*** dave-mccowan has quit IRC | 07:30 | |
*** tbachman has quit IRC | 07:30 | |
*** sbezverk has quit IRC | 07:31 | |
*** markus_z has quit IRC | 07:31 | |
*** huanxie has joined #openstack-nova | 07:32 | |
*** edmondsw has quit IRC | 07:32 | |
*** pcaruana|afk| has joined #openstack-nova | 07:35 | |
*** pcaruana|afk| has quit IRC | 07:35 | |
*** pcaruana has quit IRC | 07:36 | |
*** dave-mccowan has joined #openstack-nova | 07:36 | |
*** jpena|off is now known as jpena | 07:37 | |
*** amotoki is now known as amotoki_away | 07:39 | |
*** pcaruana has joined #openstack-nova | 07:40 | |
*** markus_z has joined #openstack-nova | 07:50 | |
*** ralonsoh has joined #openstack-nova | 07:50 | |
*** amotoki_away is now known as amotoki | 07:53 | |
*** tbachman has joined #openstack-nova | 07:53 | |
*** links has quit IRC | 07:55 | |
*** thorst has joined #openstack-nova | 07:58 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-nova | 08:00 | |
*** vvargaszte has joined #openstack-nova | 08:03 | |
*** thorst has quit IRC | 08:03 | |
*** alextricity25 has quit IRC | 08:04 | |
*** alextricity25_ has joined #openstack-nova | 08:04 | |
*** oomichi has quit IRC | 08:05 | |
ralonsoh | Hello fellow reviewers, can I take your attention to https://review.openstack.org/#/c/449257/? | 08:06 |
*** oomichi has joined #openstack-nova | 08:07 | |
ralonsoh | The spec was merged months ago, but I still have two out of four patches pending for reviews | 08:07 |
ralonsoh | thanks in advance | 08:07 |
*** links has joined #openstack-nova | 08:08 | |
*** udesale has quit IRC | 08:08 | |
*** danpawlik has quit IRC | 08:13 | |
*** Guest39557 is now known as asettle | 08:16 | |
*** bkopilov has joined #openstack-nova | 08:17 | |
*** mvk has quit IRC | 08:24 | |
*** priteau has joined #openstack-nova | 08:27 | |
*** derekh has joined #openstack-nova | 08:32 | |
*** lucas-afk is now known as lucasagomes | 08:34 | |
*** klipka_ is now known as klipka | 08:34 | |
*** danpawlik has joined #openstack-nova | 08:35 | |
*** efoley has joined #openstack-nova | 08:39 | |
*** udesale has joined #openstack-nova | 08:41 | |
*** markus_z has quit IRC | 08:46 | |
*** openstackgerrit has quit IRC | 08:48 | |
*** suresh12 has joined #openstack-nova | 08:50 | |
*** mvk has joined #openstack-nova | 08:53 | |
*** suresh12 has quit IRC | 08:54 | |
*** openstackgerrit has joined #openstack-nova | 08:55 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.soft_delete notifications https://review.openstack.org/460029 | 08:55 |
*** danpawlik has quit IRC | 08:58 | |
*** sdatko has quit IRC | 08:58 | |
*** markus_z has joined #openstack-nova | 08:58 | |
*** karimb has joined #openstack-nova | 08:58 | |
*** mvk has quit IRC | 08:59 | |
*** sree has quit IRC | 09:03 | |
*** sree has joined #openstack-nova | 09:04 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.soft_delete notifications https://review.openstack.org/460029 | 09:07 |
*** sree has quit IRC | 09:08 | |
gabor_antal | sfinucan, bauzas: would you please give back +2s on my patchset https://review.openstack.org/460029 ? I had to modify the json because https://review.openstack.org/#/c/451789/ was merged a few days ago. | 09:08 |
*** salv-orlando has quit IRC | 09:09 | |
*** salv-orlando has joined #openstack-nova | 09:09 | |
*** mvk has joined #openstack-nova | 09:11 | |
*** udesale has quit IRC | 09:13 | |
*** salv-orlando has quit IRC | 09:13 | |
openstackgerrit | Merged openstack/nova master: Provide error message in MismatchError for api-samples tests https://review.openstack.org/475489 | 09:15 |
*** edmondsw has joined #openstack-nova | 09:15 | |
*** udesale has joined #openstack-nova | 09:16 | |
*** sambetts_ is now known as sambetts | 09:19 | |
*** edmondsw has quit IRC | 09:20 | |
*** sdatko has joined #openstack-nova | 09:21 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix to return server groups quota in quota class set https://review.openstack.org/467999 | 09:22 |
*** danpawlik has joined #openstack-nova | 09:23 | |
*** vvargaszte has quit IRC | 09:25 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add datapath type information to OVS vif objects https://review.openstack.org/474892 | 09:30 |
*** openstackgerrit has quit IRC | 09:33 | |
*** openstackgerrit has joined #openstack-nova | 09:34 | |
openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP traits https://review.openstack.org/474550 | 09:34 |
*** yingjun has quit IRC | 09:38 | |
gabor_antal | sfinucan: thanks! | 09:39 |
*** yamamoto has quit IRC | 09:41 | |
openstackgerrit | Andrey Volkov proposed openstack/nova master: Add ability to view quota details for project owner https://review.openstack.org/386008 | 09:41 |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | 09:44 |
*** ijw has joined #openstack-nova | 09:44 | |
*** mdnadeem has quit IRC | 09:46 | |
*** sdatko has quit IRC | 09:47 | |
*** danpawlik has quit IRC | 09:48 | |
*** ftersin has joined #openstack-nova | 09:49 | |
*** mingyu has joined #openstack-nova | 09:49 | |
*** thorst has joined #openstack-nova | 09:49 | |
*** ijw has quit IRC | 09:49 | |
*** sdatko has joined #openstack-nova | 09:49 | |
*** danpawlik has joined #openstack-nova | 09:50 | |
*** gjayavelu has joined #openstack-nova | 09:52 | |
*** alexchadin has quit IRC | 09:53 | |
*** alexchadin has joined #openstack-nova | 09:53 | |
*** vladikr has joined #openstack-nova | 09:55 | |
*** dtantsur is now known as dtantsur|bbl | 09:55 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Read datapath_type from VIF object https://review.openstack.org/474914 | 09:56 |
*** yamamoto has joined #openstack-nova | 09:56 | |
*** mnestratov has joined #openstack-nova | 09:56 | |
*** thorst has quit IRC | 09:59 | |
*** udesale has quit IRC | 10:02 | |
*** liujiong has quit IRC | 10:02 | |
*** jerrygb_ has quit IRC | 10:03 | |
*** udesale has joined #openstack-nova | 10:03 | |
*** yamamoto has quit IRC | 10:06 | |
*** alexchadin has quit IRC | 10:07 | |
openstackgerrit | Moshe Levi proposed openstack/os-vif master: hardware offload support for openvswitch https://review.openstack.org/460278 | 10:13 |
*** gszasz has joined #openstack-nova | 10:15 | |
*** dims has quit IRC | 10:16 | |
*** yangyapeng has quit IRC | 10:16 | |
*** yangyapeng has joined #openstack-nova | 10:17 | |
*** thorst has joined #openstack-nova | 10:17 | |
*** gongysh has quit IRC | 10:18 | |
*** dims has joined #openstack-nova | 10:21 | |
*** yangyapeng has quit IRC | 10:21 | |
*** gomarivera has joined #openstack-nova | 10:22 | |
*** yangyapeng has joined #openstack-nova | 10:22 | |
*** yamamoto has joined #openstack-nova | 10:25 | |
*** mingyu has quit IRC | 10:25 | |
*** gomarivera has quit IRC | 10:26 | |
*** yangyapeng has quit IRC | 10:27 | |
*** jpena is now known as jpena|off | 10:27 | |
*** jpena|off is now known as jpena | 10:31 | |
*** pewp has quit IRC | 10:34 | |
*** moshele has quit IRC | 10:36 | |
*** andreas_s has joined #openstack-nova | 10:36 | |
*** pewp has joined #openstack-nova | 10:36 | |
*** mingyu has joined #openstack-nova | 10:38 | |
*** dtantsur|bbl is now known as dtantsur | 10:45 | |
*** vvargaszte has joined #openstack-nova | 10:46 | |
openstackgerrit | Merged openstack/nova master: Consolidate index and detail methods in HypervisorsController https://review.openstack.org/475458 | 10:47 |
*** karimb has quit IRC | 10:50 | |
*** vks1 has quit IRC | 10:51 | |
*** karimb has joined #openstack-nova | 10:51 | |
*** qwertyco has joined #openstack-nova | 10:54 | |
*** smatzek has joined #openstack-nova | 10:55 | |
*** vvargaszte has quit IRC | 11:03 | |
*** edmondsw has joined #openstack-nova | 11:03 | |
*** edmondsw has quit IRC | 11:08 | |
*** mingyu has quit IRC | 11:08 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576 | 11:10 |
*** bkopilov has quit IRC | 11:12 | |
*** gjayavelu has quit IRC | 11:12 | |
*** huanxie has quit IRC | 11:15 | |
openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625 | 11:16 |
*** thorst has quit IRC | 11:16 | |
*** tbachman has quit IRC | 11:17 | |
*** vks1 has joined #openstack-nova | 11:18 | |
*** vladikr has quit IRC | 11:21 | |
*** eric_young has quit IRC | 11:23 | |
*** lucasagomes is now known as lucas-hungry | 11:24 | |
*** moshele has joined #openstack-nova | 11:24 | |
*** boolman has joined #openstack-nova | 11:25 | |
boolman | Hi guys, I'm having trouble with a new deployment.. I'm using juju to deploy ocata. and I seem to have issues with my compute-nodes/placement API, http://ix.io/xHP http://ix.io/xHO | 11:25 |
openstackgerrit | Béla Vancsics proposed openstack/nova master: Use more specific asserts in tests https://review.openstack.org/466609 | 11:27 |
*** yangyapeng has joined #openstack-nova | 11:28 | |
*** karimb has quit IRC | 11:31 | |
*** acormier has joined #openstack-nova | 11:33 | |
*** ijw has joined #openstack-nova | 11:37 | |
*** diga has quit IRC | 11:39 | |
*** ijw has quit IRC | 11:42 | |
*** vks1 has quit IRC | 11:44 | |
*** thorst has joined #openstack-nova | 11:47 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/475563 | 11:49 |
*** suresh12 has joined #openstack-nova | 11:51 | |
*** thorst has quit IRC | 11:52 | |
openstackgerrit | Moshe Levi proposed openstack/os-vif master: hardware offload support for openvswitch https://review.openstack.org/460278 | 11:53 |
*** jpena is now known as jpena|lunch | 11:54 | |
*** suresh12 has quit IRC | 11:55 | |
*** udesale has quit IRC | 11:55 | |
*** udesale has joined #openstack-nova | 11:55 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929 | 11:56 |
*** acormier has quit IRC | 12:01 | |
*** acormier has joined #openstack-nova | 12:02 | |
*** acormier has quit IRC | 12:06 | |
*** thorst has joined #openstack-nova | 12:08 | |
*** tbachman has joined #openstack-nova | 12:09 | |
*** rfolco has joined #openstack-nova | 12:14 | |
*** edmondsw has joined #openstack-nova | 12:15 | |
*** salv-orlando has joined #openstack-nova | 12:19 | |
*** efried has joined #openstack-nova | 12:21 | |
*** gomarivera has joined #openstack-nova | 12:27 | |
*** jaypipes has joined #openstack-nova | 12:30 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: separate normalize_resources_qs_param https://review.openstack.org/475599 | 12:30 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: support GET /allocation_candidates https://review.openstack.org/475448 | 12:30 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/475600 | 12:30 |
*** yingjun has joined #openstack-nova | 12:30 | |
*** kylek3h has joined #openstack-nova | 12:31 | |
*** lucas-hungry is now known as lucasagomes | 12:32 | |
*** efried has quit IRC | 12:34 | |
*** tiendc has quit IRC | 12:34 | |
bhagyashris | boolman: Hi, you can make sure that the [placement] section is there in the /etc/nova/nova.conf if its not then you should add this and restart the nova compute service. | 12:36 |
*** esberglu has joined #openstack-nova | 12:37 | |
*** cNilesh has quit IRC | 12:37 | |
*** mriedem has joined #openstack-nova | 12:38 | |
*** esberglu has quit IRC | 12:39 | |
*** esberglu has joined #openstack-nova | 12:39 | |
mriedem | sdague: when you get some time, can you go through the bottom set of +2ed changes in melwitt's quotas series? starts here: https://review.openstack.org/#/c/446241/ - they are mostly mechanical conversions from old style to new style quota checks. | 12:40 |
sfinucan | moshele, jaypipes: Am I missing something with https://review.openstack.org/#/c/460278/? | 12:40 |
rfolco | jaypipes, could you please review https://review.openstack.org/#/c/460147/ ? | 12:40 |
*** acormier has joined #openstack-nova | 12:40 | |
sfinucan | fwict, you have nested try-excepts for the same exception. Surely there's no reason for that and the whole thing could be flattened? | 12:40 |
sfinucan | ...but I can't seem to get that point across in Gerrit, unfortunately :) | 12:41 |
*** acormier has quit IRC | 12:41 | |
sdague | mriedem: yeh, probably an hour from now | 12:41 |
*** mdrabe has joined #openstack-nova | 12:42 | |
*** bkopilov has joined #openstack-nova | 12:42 | |
*** gomarivera has quit IRC | 12:43 | |
*** vks1 has joined #openstack-nova | 12:43 | |
*** esberglu has quit IRC | 12:44 | |
ftersin | mdbooth: hi. any news? | 12:44 |
*** pchavva has joined #openstack-nova | 12:49 | |
*** lbragstad_ has quit IRC | 12:49 | |
*** gomarivera has joined #openstack-nova | 12:49 | |
*** gomarivera has quit IRC | 12:50 | |
dtantsur | dansmith: hi! remember I was talking about nova-api traceback on adding an ironic nova-compute host to an aggregate? here it is https://paste.fedoraproject.org/paste/giaZIJ~cymkC3nNXO1spaA | 12:51 |
*** esberglu has joined #openstack-nova | 12:51 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | 12:51 |
*** salv-orlando has quit IRC | 12:52 | |
*** salv-orlando has joined #openstack-nova | 12:52 | |
*** efried has joined #openstack-nova | 12:53 | |
*** alexchadin has joined #openstack-nova | 12:54 | |
*** jwcroppe has quit IRC | 12:55 | |
-openstackstatus- NOTICE: restarting gerrit to address slowdown issues | 12:56 | |
*** salv-orlando has quit IRC | 12:56 | |
*** jpena|lunch is now known as jpena | 12:57 | |
*** efoley has quit IRC | 12:58 | |
*** efoley has joined #openstack-nova | 12:58 | |
*** lucasxu has joined #openstack-nova | 13:00 | |
moshele | sfinucan: to me it just a style issue, but I will do the change | 13:00 |
moshele | sfinucan: sure I will put you as Co-Authored-by :) I will put the patch workflow -1 for now | 13:01 |
*** lyan has joined #openstack-nova | 13:02 | |
*** mingyu has joined #openstack-nova | 13:02 | |
*** awaugama has joined #openstack-nova | 13:06 | |
*** mingyu_ has joined #openstack-nova | 13:09 | |
*** mingyu has quit IRC | 13:09 | |
alex_xu | jaypipes: bauzas edleafe, would you remind me about why we put the resource-classes in the extra_specs? I'm thinking of the same question for required_traits in the flavor. Put something new in the extra specs, that means something undiscoverable for the API user, so I didn't understand why. | 13:09 |
*** lpetrut has joined #openstack-nova | 13:09 | |
*** karimb has joined #openstack-nova | 13:10 | |
*** yamamoto has quit IRC | 13:10 | |
*** ratailor has quit IRC | 13:10 | |
*** yamamoto has joined #openstack-nova | 13:11 | |
*** salv-orlando has joined #openstack-nova | 13:12 | |
*** gouthamr has joined #openstack-nova | 13:12 | |
*** alexchadin has quit IRC | 13:13 | |
dtantsur | dansmith (or anyone): isn't it doomed to fail: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4383 ? even my basic deployment has 2 cells (cell0 and default), and the service can be found in both (for whatever reason..) | 13:13 |
*** zz9pzza has quit IRC | 13:14 | |
*** zz9pzza has joined #openstack-nova | 13:14 | |
*** zz9pzza has quit IRC | 13:15 | |
*** zz9pzza has joined #openstack-nova | 13:15 | |
*** yamamoto has quit IRC | 13:15 | |
*** jerrygb has joined #openstack-nova | 13:15 | |
*** njohnston has left #openstack-nova | 13:17 | |
*** links has quit IRC | 13:18 | |
edleafe | alex_xu: IIRC, it's simply an interim solution until we can overhaul the handling of flavors. | 13:18 |
*** jerrygb_ has joined #openstack-nova | 13:20 | |
*** baoli has joined #openstack-nova | 13:20 | |
*** jerrygb has quit IRC | 13:20 | |
*** eharney has joined #openstack-nova | 13:23 | |
sfinucan | edleafe: Remind me, most of the operations you refer to in the "Trying to understand the proposed direction" occur in, what, cell 0? | 13:26 |
sfinucan | except for the actual booting of VM, which occurs in the target cell, of course | 13:26 |
*** mingyu_ has quit IRC | 13:27 | |
sfinucan | or the super cell, even | 13:28 |
*** mingyu has joined #openstack-nova | 13:28 | |
*** yamamoto has joined #openstack-nova | 13:29 | |
*** yamamoto has quit IRC | 13:29 | |
*** gszasz has quit IRC | 13:30 | |
edleafe | sfinucan: they occur in placement/scheduler. It is the super conductor that moderates the flow, and eventually dispatches the build to the cell chosen by the scheduler | 13:31 |
*** yamamoto has joined #openstack-nova | 13:31 | |
sfinucan | and the scheduler is running outside of the cells? | 13:32 |
sfinucan | i.e. there isn't a super-scheduler? | 13:32 |
edleafe | sfinucan: yeah, just one. No per-cell schedulers | 13:32 |
*** jwcroppe has joined #openstack-nova | 13:32 | |
sfinucan | Gotcha. Thank you :) | 13:32 |
edleafe | np | 13:32 |
* sfinucan is trying to formalize what you said in that mail into a more permanent doc | 13:32 | |
dtantsur | speaking of which, is there something like "nova cells 101" for absolute beginners? | 13:33 |
sfinucan | dtantsur: dansmith's talk from the ATL summit it excellent | 13:33 |
edleafe | sfinucan: remember that what I called the "current flow" isn't what has merged into master; it was the design agreed to before the recent changes | 13:34 |
kashyap | Yeah, was about to suggest that | 13:34 |
dtantsur | sfinucan: cool! I've found something from ATX right now | 13:34 |
*** yingjun has quit IRC | 13:34 | |
*** yamahata_ has joined #openstack-nova | 13:34 | |
sfinucan | edleafe: Yup, I haven't bothered including the "current flow" - just the new one | 13:34 |
sfinucan | and can -WIP it til code is merged | 13:34 |
mriedem | dtantsur: those are also linked here https://docs.openstack.org/developer/nova/cells.html | 13:34 |
edleafe | sfinucan: ok cool | 13:34 |
*** efried has quit IRC | 13:35 | |
dtantsur | sweet. thanks mriedem | 13:35 |
boolman | bhagyashris: the [placment] section is there, configured and all. ( if missing or incorrect the nova-compute wont start ) | 13:35 |
dansmith | dtantsur: oh you probably have some control services in your cell0 and some computes in cell1, with the same id | 13:35 |
dansmith | dtantsur: you need mriedem | 13:35 |
dansmith | er | 13:35 |
dtantsur | dansmith: this is definitely the case | 13:35 |
dtantsur | or wait | 13:35 |
dansmith | dtantsur: you need mriedem's set which is converting that API to uuids | 13:35 |
dtantsur | what is cell1? | 13:35 |
dansmith | dtantsur: I just mean "the other cell that is not cell0, the real one" | 13:36 |
dtantsur | aha | 13:36 |
dtantsur | so, sorry, what do I need? is there some patch up? | 13:36 |
mriedem | dtantsur: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/service-hyper-uuid-in-api | 13:36 |
mriedem | is the wip for converting os-services and os-hypervisors APIs to use uuids | 13:37 |
dansmith | mriedem: the -2 on that is for microversion sequencing I assume? | 13:37 |
mriedem | dansmith: yes | 13:37 |
mriedem | those need to go together | 13:37 |
*** damien_r has quit IRC | 13:37 | |
mriedem | well, they do'nt need to | 13:37 |
mriedem | but we said they would | 13:37 |
dansmith | oh I see the one behind it? | 13:37 |
mriedem | only thing left in os-hypervisors at the top is docs and paging | 13:37 |
*** zz9pzza has quit IRC | 13:38 | |
dtantsur | does it mean that I'll have to stop referring to host names when adding them to aggregates? please pardon my ignorance, I'm not sure how it solves the problem in question | 13:38 |
dansmith | he needs services | 13:38 |
*** yingjun has joined #openstack-nova | 13:38 | |
dansmith | dtantsur: oh are you getting that while querying for a service by hostname? | 13:38 |
dtantsur | dansmith: I'm doing $ openstack aggregate add host baremetal-hosts overcloud-controller-0.localdomain | 13:39 |
dtantsur | do add the ironic host to an aggregate | 13:39 |
*** cleong has joined #openstack-nova | 13:39 | |
dansmith | I see | 13:39 |
dtantsur | which may be not a perfect approach to a problem of hybrid bm-vm environment, but that's what we do now | 13:39 |
*** pchavva has quit IRC | 13:39 | |
dansmith | let me look a bit | 13:39 |
* dtantsur gets back to cell videos meanwhile | 13:40 | |
*** gomarivera has joined #openstack-nova | 13:40 | |
dansmith | ohhh | 13:40 |
dtantsur | I don't like this "ohhh" :) | 13:40 |
dansmith | because you have multiple per host maybe | 13:40 |
dansmith | hang on, I have no coffee in my system. this is not good. | 13:40 |
dtantsur | multiple what? I definitely have only one nova-compute process per host.. | 13:41 |
*** zz9pzza has joined #openstack-nova | 13:41 | |
boolman | Hi guys, I'm having trouble with a new deployment.. I'm using juju to deploy ocata, so all the configuration is automated, and yes I have the [placement] section. and I seem to have issues with my compute-nodes/placement API, http://ix.io/xHP http://ix.io/xHO | 13:41 |
dtantsur | dansmith: sure, lack of coffee is an urgent priority bug :) | 13:41 |
dansmith | dtantsur: hmm, no this should work.. are you running control services on the same node as a compute? | 13:42 |
dtantsur | dansmith: I am | 13:42 |
dansmith | dtantsur: with different config files I assume? | 13:42 |
*** efried has joined #openstack-nova | 13:42 | |
*** liverpooler has joined #openstack-nova | 13:42 | |
dtantsur | dansmith: I don't think so actually | 13:42 |
dansmith | dtantsur: if not then they wouldn't be in different cells... | 13:43 |
dtantsur | are cells assigned via nova.conf? | 13:43 |
*** gbarros has joined #openstack-nova | 13:43 | |
dansmith | no, but which cell you check into is determined by which database url you have configured | 13:43 |
dtantsur | hmm, ok. these hosts may indeed end up in a wrong cell.. however, it does not quite explain to me why we have duplicate hosts exception | 13:44 |
dansmith | well, | 13:44 |
dansmith | if they | 13:44 |
dansmith | are in different cells, | 13:44 |
dansmith | and they both report the same hostname, then there are multiple services with the same hostname but in different cells | 13:45 |
dtantsur | they = who? for now it seems that I have nova-api, scheduler and ironic's computes in one cell, then real compute in another | 13:46 |
dansmith | right | 13:46 |
dansmith | they = any service (n-api, n-cpu, etc) | 13:46 |
dansmith | since they have the same hostname as what n-cpu is reporting, | 13:46 |
*** gomarivera has quit IRC | 13:46 | |
dansmith | we could look that service up to find the cell, but end up getting the wrong one | 13:47 |
dansmith | which is why we now check for dupes and fail if the result is ambiguous | 13:47 |
dansmith | dtantsur: can we check to see if they are using the same config and thus reporting into the same cell for the moment? | 13:47 |
dtantsur | dansmith: I have only one nova.conf (provided by puppet, I guess), so I think that's the case | 13:49 |
*** mingyu has quit IRC | 13:49 | |
dtantsur | I still don't quite understand why the result is ambiguos | 13:49 |
dansmith | actually, that query should be looking specifically for a nova-compute service, which shouldn't be duplicated if you only have one compute | 13:49 |
dansmith | dtantsur: that's what we're trying to find out | 13:49 |
dtantsur | right | 13:49 |
dansmith | dtantsur: can you dump the services table for your cell0 and other cell databases for me? | 13:49 |
dtantsur | lemme try | 13:49 |
*** sree has joined #openstack-nova | 13:51 | |
dtantsur | dansmith: there are 3 databases, I guess I need one that is "nova", not "nova_api" or "nova_placement". right? | 13:51 |
*** gszasz has joined #openstack-nova | 13:51 | |
*** pchavva has joined #openstack-nova | 13:51 | |
*** crushil has joined #openstack-nova | 13:51 | |
alex_xu | edleafe: not sure what I should do for required traits | 13:52 |
*** suresh12 has joined #openstack-nova | 13:52 | |
dansmith | dtantsur: you should have nova and nova_cell0 | 13:53 |
dtantsur | ok, lemme see | 13:54 |
*** udesale has quit IRC | 13:56 | |
*** qwertyco has quit IRC | 13:58 | |
*** karimb has quit IRC | 13:58 | |
*** gouthamr has quit IRC | 13:58 | |
*** suresh12 has quit IRC | 14:00 | |
dtantsur | dansmith: ok, cell0 is empty. lemme fetch another one for you | 14:01 |
dansmith | cell0.services you mean right? | 14:02 |
dansmith | that's what I would expect if you're all pointing at the other cell | 14:02 |
dansmith | but I'm not sure how we're hitting that not unique error then | 14:02 |
dansmith | oh, | 14:02 |
*** gomarivera has joined #openstack-nova | 14:02 | |
edleafe | alex_xu: I would mimic what was done for custom resources, but using the 'traits:' prefix. Not too sure how to distinguish required vs. preferred | 14:03 |
dansmith | unless you have two cell records for the same cell? | 14:03 |
dansmith | dtantsur: also in nova_api, dump out the cell_mapping table please | 14:03 |
dtantsur | sure, gimme a second | 14:03 |
dtantsur | I need to figure out how to escape column names in mysql (binary is a reserved word, apparently) | 14:03 |
*** gomarivera has quit IRC | 14:04 | |
*** suresh12 has joined #openstack-nova | 14:04 | |
alex_xu | edleafe: but really want to understand the reason why we continue put something in the extra spec first | 14:04 |
dansmith | dtantsur: heh | 14:04 |
*** gomarivera has joined #openstack-nova | 14:04 | |
dansmith | dtantsur: I think if you use the full path it'll work, i.e.: SELECT services.binary FROM services | 14:04 |
*** prateek has quit IRC | 14:05 | |
dtantsur | dansmith: thnx! ok, so here are services: https://paste.fedoraproject.org/paste/ZK~YdTqfsWnpIImEF6TxLQ | 14:05 |
*** gbarros has quit IRC | 14:05 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document the scheduler workflow https://review.openstack.org/475810 | 14:06 |
sfinucan | edleafe, jaypipes: ^ probably not suitable for merge yet, but there you go | 14:06 |
jaypipes | sfinucan: hey, cool, thanks for working on that | 14:06 |
dtantsur | dansmith: cell_mappings has 2 cells | 14:07 |
dansmith | dtantsur: do they both point to the same database_connection ? | 14:08 |
dtantsur | dansmith: nope | 14:08 |
*** felipemonteiro has joined #openstack-nova | 14:09 | |
dansmith | well, then I'm not sure how you're getting that exception | 14:09 |
*** acormier has joined #openstack-nova | 14:09 | |
* dtantsur bangs his head on the table | 14:09 | |
openstackgerrit | Maciej Kucia proposed openstack/nova master: [doc] Updated sqlalchemy URL in migrate readme https://review.openstack.org/472549 | 14:09 |
dansmith | dtantsur: is this a machine I can get on to add some debug? or can you just sprinkle some debug statements in that _find_service_in_cell() function? | 14:09 |
*** lyan has quit IRC | 14:10 | |
*** karimb has joined #openstack-nova | 14:10 | |
dtantsur | dansmith: I can do both, the latter is easier, I guess | 14:10 |
dtantsur | just after the current meeting | 14:10 |
*** lyan has joined #openstack-nova | 14:10 | |
dansmith | hmm | 14:12 |
edleafe | alex_xu: because right now flavors are the only means to specify what you want Nova to build. Until we can clean up that technical debt, we are stuck using flavors, which means cramming more and more stuff into extra_specs | 14:13 |
*** jdillaman has joined #openstack-nova | 14:14 | |
*** eharney has quit IRC | 14:14 | |
dtantsur | dansmith: what exactly would you like to see printed? I think I have an idea, but just in case you have something else in mind | 14:15 |
dansmith | dtantsur: that for loop should iterate twice.. once over each cell. if cell0 is empty, it should never set "service" to something, and thus in the second/last iteration, it should not hit the ServiceNotUnique raise, because service hasn't been set yet | 14:16 |
dansmith | dtantsur: so just anything around service getting set, what it's set to, etc | 14:17 |
dtantsur | yep, this is my assumption too. I'll try to check it. | 14:17 |
*** gbarros has joined #openstack-nova | 14:20 | |
*** tidwellr has joined #openstack-nova | 14:22 | |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Allow tuple as for nics value https://review.openstack.org/475816 | 14:23 |
*** yamahata_ has quit IRC | 14:23 | |
*** salv-orlando has quit IRC | 14:24 | |
sdague | melwitt / mriedem - in https://review.openstack.org/#/c/446247/23 ... auto_allocated floating ips are excluded from quota? | 14:24 |
sdague | is that they way it's always been? | 14:24 |
*** moshele has quit IRC | 14:25 | |
dtantsur | dansmith: it looks like both Cell objects (cell0 and default) have the same database connection (poining to "nova") Oo | 14:26 |
*** suresh12 has quit IRC | 14:26 | |
dansmith | dtantsur: okay, that makes sense for getting that not unique error, so now we need to figure out why the db is different | 14:27 |
dtantsur | wait, no | 14:27 |
sdague | ah, I found other code that indicates yes | 14:27 |
sdague | mriedem: ok, I went up until the first jenkins failing patch | 14:27 |
mriedem | sdague: that's the way it's always been | 14:27 |
sdague | and they look good | 14:27 |
*** salv-orlando has joined #openstack-nova | 14:28 | |
sdague | mriedem: yeh, I didn't realize, but the follow on patch touched a line where it was more obvious | 14:28 |
dtantsur | dansmith: sorry for confusing. the cells are ok. I'm just getting the same host out of both.. | 14:29 |
mriedem | thanks for hitting those | 14:29 |
dansmith | dtantsur: okay | 14:29 |
*** gouthamr has joined #openstack-nova | 14:30 | |
*** salv-orlando has quit IRC | 14:30 | |
*** yangyape_ has joined #openstack-nova | 14:30 | |
*** salv-orlando has joined #openstack-nova | 14:30 | |
dtantsur | but 'services' table is empty on nova_cell0, I've just double-checked >_< | 14:31 |
sdague | mriedem: my guess is that got caught up in the 6 way octopus merge limits with git | 14:31 |
sdague | mriedem: so honestly rebasing everything once the base set lands is probably in order anyway | 14:32 |
dansmith | dtantsur: and just to be clear, the api_connection is pointed at nova_api and not nova_placement right? | 14:32 |
dtantsur | lemme check too | 14:32 |
*** eharney has joined #openstack-nova | 14:33 | |
sdague | mriedem: my only wish in those patches (which is long past as it landed in march) is that the config option was strict_quotas instead of recheck_quotas, just from a nice name perspective | 14:33 |
*** yangyapeng has quit IRC | 14:33 | |
*** udesale has joined #openstack-nova | 14:34 | |
*** knikolla_phone has joined #openstack-nova | 14:34 | |
dtantsur | dansmith: [api_database]connection points to nova_api | 14:34 |
dansmith | dtantsur: okay so in that for loop, is it finding/setting | 14:34 |
dansmith | service to anything on the cell0 pass? | 14:35 |
dtantsur | dansmith: yep. lemme show you the output of my added logging | 14:35 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667 | 14:35 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077 | 14:35 |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Allow tuple as for nics value https://review.openstack.org/475816 | 14:36 |
knikolla_phone | How does nova calculate quota usage? After manually setting an instance as deleted in the database 'instances' table the resources still show up as used. | 14:36 |
dansmith | heh | 14:37 |
dtantsur | dansmith: https://paste.fedoraproject.org/paste/r26SqCAGrk8dXPlOk-sfjg | 14:37 |
*** thorst has quit IRC | 14:37 | |
*** annegentle has joined #openstack-nova | 14:38 | |
*** lbragstad_ has joined #openstack-nova | 14:38 | |
*** jdwidari has quit IRC | 14:39 | |
dansmith | dtantsur: that's kinda hard to make sense of, especially without seeing the code, but.. why the HostMappingNotFound messages interleaved with the iteration of that for loop? | 14:39 |
dtantsur | dansmith: I think it was a bug in oslo.log | 14:39 |
dansmith | oh eff me | 14:39 |
dansmith | dtantsur: I see the bug | 14:40 |
*** yingjun has quit IRC | 14:40 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Fix using targeted cell context when finding services in cells https://review.openstack.org/475830 | 14:41 |
dansmith | dtantsur: try that ^ :) | 14:41 |
*** felipemonteiro has quit IRC | 14:41 | |
dtantsur | thanks! | 14:41 |
*** suresh12 has joined #openstack-nova | 14:41 | |
*** lbragstad_ is now known as lbragstad | 14:42 | |
dansmith | dtantsur: it was iterating but not actually looking in the other database :P | 14:42 |
dtantsur | I see | 14:42 |
*** yamahata_ has joined #openstack-nova | 14:42 | |
*** thorst has joined #openstack-nova | 14:42 | |
*** lyan has quit IRC | 14:43 | |
*** lyan has joined #openstack-nova | 14:43 | |
*** felipemonteiro has joined #openstack-nova | 14:44 | |
*** hongbin has joined #openstack-nova | 14:46 | |
dansmith | dtantsur: if this fixes you, then: sorry for wasting your time | 14:46 |
*** udesale has quit IRC | 14:46 | |
*** thorst has quit IRC | 14:47 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/475600 | 14:47 |
*** rcernin_ has quit IRC | 14:47 | |
dtantsur | dansmith: w00t, it works with this patch :) thanks! | 14:48 |
dansmith | dtantsur: cool, I shall write a test and then try to get that in asap | 14:48 |
dtantsur | great | 14:48 |
jaypipes | knikolla_phone: you need to run the nova-manage quota refresh command. | 14:48 |
dansmith | cc: mriedem | 14:48 |
jaypipes | knikolla_phone: unfortunately, quotas uses different tables than instances to store usage. | 14:49 |
dansmith | jaypipes: yesterday that was true | 14:49 |
dansmith | jaypipes: today it may not be | 14:49 |
jaypipes | wait, what? | 14:49 |
*** markvoelker has joined #openstack-nova | 14:49 | |
jaypipes | oh, yeah. | 14:49 |
* dansmith looks at all the counting quotas patches in the gate | 14:49 | |
jaypipes | but pretty sure knikolla_phone doesn't have yesterday's code deployed ;) | 14:49 |
dansmith | GASP | 14:49 |
mriedem | hells bells https://review.openstack.org/#/c/475830/1/nova/compute/api.py | 14:50 |
dansmith | jaypipes: I know, but it's amusing timing given the flurry of them in the gate atm | 14:50 |
knikolla_phone | Jaypipes: we're running ocata | 14:50 |
jaypipes | knikolla_phone: yup, so you need to do the nova-manage quota refresh thing :) | 14:50 |
jaypipes | dansmith: ack | 14:50 |
knikolla_phone | Jaypipes: thanks. Will give it a try now. | 14:51 |
*** annegentle has quit IRC | 14:53 | |
mriedem | mgagne: sorry it took awhile, but i finally got another thorough review on https://review.openstack.org/#/c/454322/ | 14:53 |
mriedem | mgagne: lots of edge cases in there that need to be dealt with | 14:53 |
mriedem | dansmith: so, | 14:53 |
mgagne | ok, will read and address them asap | 14:53 |
mriedem | something i've been meaning to pester other cores about for their opinions on https://review.openstack.org/#/c/454322/ | 14:53 |
*** yamamoto has quit IRC | 14:53 | |
mriedem | dansmith: ^ is a new compute driver capability, | 14:53 |
mriedem | called by cinder when a volume is resized | 14:54 |
mriedem | if you're doing this on an instance running on an ocata compute, that's going to be ignored | 14:54 |
mriedem | with no indication to the user/admin | 14:54 |
*** yamamoto has joined #openstack-nova | 14:54 | |
*** yamamoto has quit IRC | 14:54 | |
knikolla_phone | jaypipes: that didn't work. Still ghost resources. | 14:55 |
mriedem | dansmith: well, take that back, you'll get a warning in the compute logs on the ocata node for "Received unexpected event %(event)s for instance" | 14:55 |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform keypair.delete notification https://review.openstack.org/463946 | 14:55 |
knikolla_phone | I know i asked this once and mriedem answered. But what does the deleted column in the table need to be set for a deleted instance? I can't remember what it was, sorry to ask again :( | 14:56 |
mriedem | dansmith: i was pondering doing a minimum compute service version check in the api for these types of events, since we know that you need a minimum of pike to support those events, and we could return a 400 to cinder otherwise, | 14:56 |
mriedem | and they could rollback the volume resize on their side | 14:56 |
jaypipes | knikolla_phone: to its ID value. | 14:56 |
mriedem | knikolla_phone: instances.deleted = instances.id | 14:56 |
mriedem | if it's deleted | 14:56 |
jaypipes | knikolla_phone: but honestly, you shouldn't be setting stuff manually in the DB. Dragons be there. | 14:56 |
mriedem | dansmith: but i'm wondering if that's too heavy handed, doing the service version check in the api | 14:56 |
dansmith | mriedem: yeah, I mean, it's a warning and the assumption was that we'd add events over time, so the warning isn't terrible | 14:57 |
dansmith | mriedem: just depends on whether or not we have different behavior before/after | 14:57 |
dansmith | if it's just an optimization that will continue to work but slower, then no need to be heavy-handed I think | 14:57 |
mriedem | my thinking was as a user i do this volume resize thing, and it all says it's fine, | 14:57 |
mriedem | but the block device in my guest isn't resized to the new size, so i report a support ticket | 14:57 |
mriedem | the admin has to go figure out through the logs that the event was basically ignored on the ocata compute because it didn't recognize it | 14:58 |
knikolla_phone | jaypipes: i like dragons. | 14:58 |
dansmith | ah, yeah, but.. they triggered the resize from the cinder side right? | 14:58 |
mriedem | we also don't have any policy check for this to effectively disable it per event until your computes are all upgraded | 14:58 |
mriedem | dansmith: yes | 14:58 |
dansmith | mriedem: blocking the event from cinder isn't going to tell the user anything right? | 14:58 |
mriedem | but cinder is pike and the instance is runnign on an ocata compute | 14:59 |
knikolla_phone | We were left with an inconsistent db after upgrade. Some instances failed to create and delete. But everything else was fine. | 14:59 |
mriedem | dansmith: i was thinking that if we could detect in the api that this isn't going to work based on service version, then we return a 400 to cinder and cinder rolls back the volume resize | 14:59 |
mriedem | which is the indication to the user | 14:59 |
dansmith | mriedem: that's super expensive | 14:59 |
dansmith | mriedem: and.. can you even roll it back? often times block device resizes are one-way | 15:00 |
mriedem | alternatively, if cinder has a policy rule for extending an attached volume, then the admin can disable this from the cinder side until they know they can handle it | 15:00 |
mriedem | mgagne: does the cinder side of this have a unique policy rule for extending attached volumes? | 15:00 |
mriedem | or is it just the same as os-extend for 'available' non-attached volumes? | 15:00 |
jaypipes | knikolla_phone: here's a dragon for you: http://bit.ly/2sLVKtR | 15:01 |
mriedem | mgagne: because if i were a xen or vmware or hyperv shop, and this just will never work, i'd want to disable the cinder api behavior globally | 15:01 |
*** takedakn has joined #openstack-nova | 15:01 | |
*** gomarivera has quit IRC | 15:01 | |
*** ckopper has quit IRC | 15:02 | |
knikolla_phone | jaypipes: the nova-manage project quota_refresh_usage command didn't do anything. | 15:02 |
*** gomarivera has joined #openstack-nova | 15:02 | |
knikolla_phone | Newton -> ocata was the only problematic upgrade so far for us. :( | 15:02 |
*** mingyu has joined #openstack-nova | 15:03 | |
*** yamamoto has joined #openstack-nova | 15:04 | |
jaypipes | knikolla_phone: other than manually going through your DB, I'm not entirely sure I can help you on this channel. There's some mismatch somewhere between the project_quota_usages or quota_usages table and the "true" resource tables like instances, instance_extra, pci_devices, etc | 15:04 |
*** READ10 has joined #openstack-nova | 15:04 | |
jaypipes | knikolla_phone: is this isolated to a particular tenant? or general? | 15:04 |
*** annegentle has joined #openstack-nova | 15:05 | |
knikolla_phone | jaypipes: no, that's okay, you've helped more than enough. This is only limited to one tenant. | 15:06 |
jaypipes | knikolla_phone: and only for their instances quota? | 15:06 |
*** gbarros has quit IRC | 15:06 | |
*** gomarivera has quit IRC | 15:06 | |
*** suresh12 has quit IRC | 15:07 | |
knikolla_phone | jaypipes: whatever resources two of the instances we manually deleted used (instances, vcpu, ram, etc) | 15:07 |
jaypipes | hmm | 15:08 |
mriedem | dansmith: ok, so i think policy blocker on the cinder side is key, i've put that in the cinder change https://review.openstack.org/#/c/454287/ | 15:08 |
jaypipes | knikolla_phone: but does the instances usage show correct? meaning, does it show the tenant using instances that were deleted? | 15:08 |
dansmith | mriedem: ack | 15:08 |
mriedem | dansmith: and then we just report the shit out of when it fails on the nova compute side in pike | 15:08 |
*** sbezverk has joined #openstack-nova | 15:08 | |
mriedem | using action events and faults | 15:08 |
*** yamamoto has quit IRC | 15:11 | |
knikolla_phone | jaypipes: yes. The usage is correct to what it would be if those two instances weren't deleted (which is weird because we manually deleted five of them). Quota_usages table reflects that amount, and updating it to random values and then using quota_refresh_usage command, it preserves the random amount. | 15:11 |
*** yamamoto has joined #openstack-nova | 15:11 | |
jaypipes | knikolla_phone: what's in the reservations table for that project? | 15:13 |
*** rmart04 has quit IRC | 15:14 | |
*** sree has quit IRC | 15:14 | |
*** sree has joined #openstack-nova | 15:15 | |
*** yamamoto has quit IRC | 15:15 | |
*** sree has quit IRC | 15:18 | |
*** jerrygb_ has quit IRC | 15:20 | |
*** gszasz has quit IRC | 15:20 | |
knikolla_phone | jaypipes: what does uuid in that table (reservations) map to? Is it an instance uuid? | 15:21 |
jaypipes | knikolla_phone: yep | 15:21 |
*** jerrygb has joined #openstack-nova | 15:21 | |
*** ckopper has joined #openstack-nova | 15:21 | |
*** Oku_OS is now known as Oku_OS-away | 15:22 | |
*** takedakn has quit IRC | 15:23 | |
knikolla_phone | jaypipes: okay, i see a deleted row. Going to try setting that too. | 15:23 |
knikolla_phone | Thanks for all your help. | 15:23 |
mnaser | i've been doing some digging around and i cannot seem to find where root_device_name gets assigned for vms.. can someone give a hint :( | 15:23 |
jaypipes | knikolla_phone: hey, no prob :) | 15:23 |
mnaser | i've discovered a bug with when using virtio-scsi with nova | 15:23 |
jaypipes | mnaser: depends on the hypervisor. | 15:24 |
jaypipes | mnaser: you talking about libvirt? | 15:24 |
mnaser | root_device_name gets set to /dev/vda .. but when attaching cinder volumes, it tries to attach them at /dev/sdX, and tries to assign the same pci address | 15:24 |
mnaser | yep | 15:24 |
jaypipes | mnaser: hmm... | 15:24 |
jaypipes | mnaser: might need to bring in mdbooth on that one. | 15:24 |
mnaser | i turned off the vm, attached the cinder block device and then started it | 15:24 |
jaypipes | mdbooth: Matt, any ideas? | 15:24 |
mnaser | and it actually generated invalid libvirt xml | 15:24 |
mnaser | i have a copy of it, let me paste | 15:25 |
* mdbooth reads back | 15:25 | |
mnaser | well looks like paste.openstack.org isn't loading | 15:26 |
mriedem | mnaser: _validate_and_build_base_options in nova.compute.api.API | 15:26 |
*** knikolla_phone has quit IRC | 15:26 | |
mnaser | https://gist.github.com/mnaser/8b000d20590e4343e4d7a73459d85210 | 15:26 |
mriedem | sets it in the base_options dict which eventually gets set on the instance (really the BuildRequest now) | 15:26 |
mnaser | notice the address of the first 2 disks | 15:26 |
mnaser | is the same | 15:26 |
mnaser | so qemu never boots the vm | 15:26 |
*** claudiub has quit IRC | 15:27 | |
mnaser | i added a LOG.error(disk_info) in the libvirt tooling | 15:27 |
mnaser | and this is what came out of it too - https://gist.github.com/mnaser/aedd44b9df018786a144f7c719ec5a13 | 15:27 |
*** jerrygb_ has joined #openstack-nova | 15:27 | |
mnaser | notice how 'dev': 'vda' (which is ephermal) and the 'dev': 'sda' (which is attached cinder block device) | 15:27 |
mnaser | i am guessing that's where the issue is stemming, the first one should be sda too (because using virtio-scsi with the image hw_ parameters) | 15:28 |
mriedem | mnaser: https://github.com/openstack/nova/blob/9045a98145f0834446d668e0282ddb1c6f2d136c/nova/compute/api.py#L921 is eventually what gets set in the instance.root_device_name field | 15:28 |
*** jerrygb has quit IRC | 15:29 | |
mnaser | i've confirmed with two different installs that booting an image with hw_scsi_model=virtio-scsi and hw_disk_bus=scsi will work but no cinder volumes will successfully attach: libvirtError: internal error: unable to execute QEMU command '__com.redhat_drive_add': Duplicate ID 'drive-scsi0-0-0-0' for drive | 15:29 |
mdbooth | mnaser: what's the symptom of the weird xml? | 15:29 |
mdbooth | Does it barf? | 15:29 |
mnaser | so, if you try to attach it to an already running vm, you get the libvirt error above (duplicate id) | 15:29 |
*** claudiub has joined #openstack-nova | 15:30 | |
mnaser | if you turn the vm off, attach (really a no-op if its offline), start it, it'll generate that invalid xml and fail | 15:30 |
*** tidwellr has quit IRC | 15:30 | |
mnaser | so im not sure if libvirt or nova is to blame in putting the duplicate address ther | 15:30 |
mdbooth | We really shouldn't be looking at root_device_name | 15:30 |
*** tidwellr has joined #openstack-nova | 15:31 | |
*** sapcc-bot has quit IRC | 15:31 | |
*** david_3 has quit IRC | 15:31 | |
*** databus23_ has quit IRC | 15:31 | |
*** carthaca_ has quit IRC | 15:31 | |
mnaser | i've just been kinda trying to understand the code and i dont know if i can point the finger at nova or libvirt yet | 15:31 |
*** jerrygb has joined #openstack-nova | 15:31 | |
*** mingyu has quit IRC | 15:31 | |
*** david_4 has joined #openstack-nova | 15:31 | |
*** sapcc-bot has joined #openstack-nova | 15:31 | |
*** carthaca_ has joined #openstack-nova | 15:31 | |
*** databus23_ has joined #openstack-nova | 15:31 | |
mnaser | but it's obviously just an issue when using virtio-scsi | 15:32 |
*** jerrygb_ has quit IRC | 15:32 | |
*** karimb has quit IRC | 15:33 | |
*** thorst has joined #openstack-nova | 15:33 | |
*** thorst has quit IRC | 15:34 | |
*** ltomasbo is now known as ltomasbo|away | 15:34 | |
*** thorst has joined #openstack-nova | 15:34 | |
*** markus_z has quit IRC | 15:36 | |
*** jerrygb_ has joined #openstack-nova | 15:36 | |
*** jerrygb has quit IRC | 15:37 | |
mgagne | mriedem: supports_volume_extend vs supports_extend_volume ? RE https://review.openstack.org/#/c/454322/18/nova/virt/driver.py | 15:37 |
*** gyee has joined #openstack-nova | 15:38 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Fix using targeted cell context when finding services in cells https://review.openstack.org/475830 | 15:38 |
dansmith | dtantsur: mriedem ^ | 15:38 |
dtantsur | nice | 15:39 |
openstackgerrit | Kaitlin Farr proposed openstack/nova master: Remove deprecated keymgr code https://review.openstack.org/439855 | 15:40 |
mriedem | melwitt: i think you might have gotten delete happy in the tests here https://review.openstack.org/#/c/457862/ | 15:41 |
mriedem | mgagne: i'm ok with either - i suppose supports_extend_volume to match the driver's method name | 15:41 |
mgagne | that's my opinion | 15:41 |
*** crushil has quit IRC | 15:42 | |
*** jerrygb_ has quit IRC | 15:42 | |
*** jerrygb has joined #openstack-nova | 15:43 | |
*** gomarivera has joined #openstack-nova | 15:47 | |
mriedem | mdbooth: i think that root_device_name thing is a legacy bdm v1 thing, but not totally sure | 15:50 |
*** tidwellr has quit IRC | 15:50 | |
mriedem | some kind of ec2 carry over | 15:50 |
*** mriedem is now known as mriedem_bowlcut | 15:51 | |
mdbooth | mriedem: I haven't followed the code all the way down, but I suspect we could ignore it | 15:51 |
*** sridharg has quit IRC | 15:52 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/475600 | 15:54 |
*** tbachman has quit IRC | 15:56 | |
*** efried has quit IRC | 15:56 | |
*** andreas_s has quit IRC | 15:56 | |
dansmith | jaypipes: quick +W bait: https://review.openstack.org/#/c/475830/ | 15:56 |
*** tbachman has joined #openstack-nova | 15:56 | |
dansmith | jaypipes: fixes the ironic people | 15:57 |
jaypipes | dansmith: wallaby'd | 15:57 |
dansmith | thanks | 15:57 |
dtantsur | thanks all! | 15:57 |
dansmith | I guess cdent still isn't around today? | 15:57 |
* jaypipes trying to figure out gibi's faults bug... | 15:57 | |
jaypipes | dansmith: he's on PTO for next two weeksd. | 15:58 |
dansmith | oh my | 15:58 |
jaypipes | dansmith: after which he will be a VMWare employee. | 15:58 |
*** vks1 has quit IRC | 15:59 | |
dansmith | oh | 15:59 |
dansmith | my | 15:59 |
dansmith | GAWD | 15:59 |
*** nic has joined #openstack-nova | 16:00 | |
*** tidwellr has joined #openstack-nova | 16:00 | |
openstackgerrit | Merged openstack/nova master: Count networks to check quota https://review.openstack.org/446241 | 16:01 |
gibi_ | jaypipes: good luck with the api.fault bug | 16:02 |
*** eharney has quit IRC | 16:02 | |
jaypipes | gibi_: yeah, I might need to pass on that :( | 16:02 |
jaypipes | gibi_: that's some old-ass code in there :) | 16:02 |
*** vks1 has joined #openstack-nova | 16:03 | |
*** armax has quit IRC | 16:03 | |
gibi_ | jaypipes: I hope somebody can say 'delete that code!' | 16:03 |
*** eharney has joined #openstack-nova | 16:03 | |
jaypipes | gibi_: it's possible. some day. https://twitter.com/dims/status/542002018970529792 | 16:04 |
*** kaisers_ has joined #openstack-nova | 16:04 | |
*** artom has quit IRC | 16:05 | |
gibi_ | :) | 16:06 |
openstackgerrit | Merged openstack/nova master: Add SecurityGroupList.get_counts() https://review.openstack.org/446245 | 16:07 |
*** dtantsur is now known as dtantsur|afk | 16:08 | |
openstackgerrit | Merged openstack/nova master: Count security groups to check quota https://review.openstack.org/457860 | 16:08 |
openstackgerrit | Merged openstack/nova master: Add FixedIPList.get_count_by_project() https://review.openstack.org/446246 | 16:09 |
*** thorst has quit IRC | 16:09 | |
*** mvk has quit IRC | 16:09 | |
openstackgerrit | Merged openstack/nova master: Count fixed ips to check quota https://review.openstack.org/457861 | 16:09 |
openstackgerrit | Merged openstack/nova master: Add FloatingIPList.get_count_by_project() https://review.openstack.org/446247 | 16:10 |
dansmith | wow, just count those counting quotas merges | 16:10 |
*** Apoorva has joined #openstack-nova | 16:11 | |
*** psachin has quit IRC | 16:13 | |
*** mdrabe has quit IRC | 16:13 | |
*** ltomasbo|away is now known as ltomasbo | 16:13 | |
*** sudipto has joined #openstack-nova | 16:17 | |
*** sudipto_ has joined #openstack-nova | 16:17 | |
*** yamamoto has joined #openstack-nova | 16:17 | |
*** mdrabe has joined #openstack-nova | 16:19 | |
*** pcaruana has quit IRC | 16:19 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (3) https://review.openstack.org/452820 | 16:21 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data https://review.openstack.org/452818 | 16:21 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (2) https://review.openstack.org/452819 | 16:21 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data https://review.openstack.org/475860 | 16:21 |
*** dtp has joined #openstack-nova | 16:22 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform keypair.delete notification https://review.openstack.org/463946 | 16:25 |
*** thorst has joined #openstack-nova | 16:25 | |
*** suresh12 has joined #openstack-nova | 16:27 | |
*** yamamoto has quit IRC | 16:28 | |
openstackgerrit | Alex Szarka proposed openstack/nova master: [WIP]Transform instance-live_migration_abort notification https://review.openstack.org/420453 | 16:32 |
*** jwcroppe has quit IRC | 16:32 | |
*** thorst has quit IRC | 16:35 | |
openstackgerrit | Alex Szarka proposed openstack/nova master: [WIP]Transform instance-live_migration_abort notification https://review.openstack.org/420453 | 16:36 |
openstackgerrit | Alex Szarka proposed openstack/nova master: [WIP]Transform instance-live_migration_abort notification https://review.openstack.org/420453 | 16:37 |
openstackgerrit | Alex Szarka proposed openstack/nova master: [WIP]Transform instance-live_migration_abort notification https://review.openstack.org/420453 | 16:40 |
*** lpetrut has quit IRC | 16:40 | |
*** ltomasbo is now known as ltomasbo|away | 16:42 | |
*** tbachman has quit IRC | 16:44 | |
*** sambetts is now known as sambetts|afk | 16:45 | |
*** suresh12 has quit IRC | 16:45 | |
*** kfarr has joined #openstack-nova | 16:46 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677 | 16:47 |
*** rajathagasthya has joined #openstack-nova | 16:47 | |
*** sbezverk has quit IRC | 16:51 | |
*** thorst has joined #openstack-nova | 16:52 | |
*** itlinux_ has joined #openstack-nova | 16:52 | |
*** crushil has joined #openstack-nova | 16:54 | |
*** zzzeek has quit IRC | 16:55 | |
*** thorst has quit IRC | 16:55 | |
*** moshele has joined #openstack-nova | 16:55 | |
*** derekh has quit IRC | 16:56 | |
*** yamamoto has joined #openstack-nova | 16:57 | |
*** ralonsoh has quit IRC | 16:58 | |
*** thorst has joined #openstack-nova | 17:00 | |
*** tesseract has quit IRC | 17:00 | |
*** moshele has quit IRC | 17:01 | |
*** efried has joined #openstack-nova | 17:02 | |
*** jwcroppe has joined #openstack-nova | 17:03 | |
*** lucasagomes is now known as lucas-afk | 17:03 | |
*** yamamoto has quit IRC | 17:03 | |
*** rajathag_ has joined #openstack-nova | 17:03 | |
*** liangy has joined #openstack-nova | 17:04 | |
*** nic has quit IRC | 17:04 | |
*** rajathagasthya has quit IRC | 17:04 | |
*** moshele has joined #openstack-nova | 17:06 | |
*** Swami has joined #openstack-nova | 17:09 | |
*** nic1 has joined #openstack-nova | 17:12 | |
*** felipemonteiro has quit IRC | 17:13 | |
*** vks1 has quit IRC | 17:16 | |
*** vks1 has joined #openstack-nova | 17:18 | |
*** suresh12 has joined #openstack-nova | 17:19 | |
*** qwertyco has joined #openstack-nova | 17:19 | |
*** lyan has quit IRC | 17:19 | |
*** tbachman has joined #openstack-nova | 17:20 | |
*** slaweq has joined #openstack-nova | 17:20 | |
openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/475563 | 17:22 |
*** suresh12 has quit IRC | 17:23 | |
*** amotoki is now known as amotoki_away | 17:23 | |
*** suresh12 has joined #openstack-nova | 17:24 | |
*** catintheroof has joined #openstack-nova | 17:24 | |
*** slaweq has quit IRC | 17:25 | |
*** suresh12_ has joined #openstack-nova | 17:25 | |
*** slaweq has joined #openstack-nova | 17:25 | |
*** suresh12 has quit IRC | 17:29 | |
*** slaweq has quit IRC | 17:30 | |
*** artom has joined #openstack-nova | 17:30 | |
*** READ10 has quit IRC | 17:31 | |
*** suresh12_ has quit IRC | 17:32 | |
*** rajathag_ has quit IRC | 17:32 | |
*** vks1 has quit IRC | 17:35 | |
*** jwcroppe_ has joined #openstack-nova | 17:35 | |
*** jwcroppe has quit IRC | 17:37 | |
*** suresh12 has joined #openstack-nova | 17:38 | |
*** sudipto has quit IRC | 17:40 | |
*** sudipto_ has quit IRC | 17:40 | |
*** priteau has quit IRC | 17:42 | |
*** priteau has joined #openstack-nova | 17:42 | |
*** jwcroppe_ has quit IRC | 17:43 | |
*** claudiub has quit IRC | 17:45 | |
*** priteau has quit IRC | 17:47 | |
*** zzzeek has joined #openstack-nova | 17:47 | |
*** efried has quit IRC | 17:47 | |
*** rajathagasthya has joined #openstack-nova | 17:49 | |
*** priteau has joined #openstack-nova | 17:49 | |
*** priteau has quit IRC | 17:53 | |
*** priteau has joined #openstack-nova | 17:54 | |
*** priteau has quit IRC | 17:54 | |
*** jpena is now known as jpena|off | 17:54 | |
*** priteau has joined #openstack-nova | 17:54 | |
*** artom has quit IRC | 17:56 | |
*** tbachman has quit IRC | 17:56 | |
*** zzzeek has quit IRC | 17:57 | |
*** qwertyco has quit IRC | 17:57 | |
*** kfarr has quit IRC | 17:58 | |
*** suresh12 has quit IRC | 17:59 | |
*** suresh12 has joined #openstack-nova | 17:59 | |
*** mriedem_bowlcut is now known as mriedem | 17:59 | |
*** priteau has quit IRC | 17:59 | |
*** moshele has quit IRC | 17:59 | |
*** priteau has joined #openstack-nova | 18:02 | |
*** zzzeek has joined #openstack-nova | 18:03 | |
*** catinthe_ has joined #openstack-nova | 18:03 | |
*** thorst has quit IRC | 18:05 | |
*** catintheroof has quit IRC | 18:05 | |
*** MasterOfBugs has joined #openstack-nova | 18:06 | |
*** suresh12 has quit IRC | 18:10 | |
*** efoley has quit IRC | 18:11 | |
*** nic1 has quit IRC | 18:12 | |
*** winston-d_ has quit IRC | 18:12 | |
*** suresh12 has joined #openstack-nova | 18:18 | |
*** jwcroppe has joined #openstack-nova | 18:20 | |
*** priteau has quit IRC | 18:23 | |
*** priteau has joined #openstack-nova | 18:23 | |
*** jwcroppe has quit IRC | 18:25 | |
*** tbachman has joined #openstack-nova | 18:26 | |
mriedem | gibi_: i know this was already approved, but needed to be rebased and while it's sitting there i had a question inline https://review.openstack.org/#/c/460029/8/doc/notification_samples/instance-soft_delete-start.json | 18:26 |
mriedem | about transforming soft delete notifications to versioned payloads | 18:27 |
*** nic has joined #openstack-nova | 18:27 | |
*** priteau has quit IRC | 18:28 | |
*** rajathagasthya has quit IRC | 18:29 | |
*** suresh12 has quit IRC | 18:32 | |
*** sbezverk has joined #openstack-nova | 18:33 | |
melwitt | mriedem: I deleted those tests on purpose because after changing floating_ips to a CountableResource, there are no more "project scoped quotas" that are refreshable or syncable. the majority of the tests in nova/tests/unit/test_quota.py rely on the global QUOTAS engine and once there are no more real examples of a certain type of quota, we would have to synthesize fake resources for the tests to still work | 18:36 |
melwitt | instead, I figured delete them since they're no longer applicable to anything real. the quota refresh command only refreshes ReservableResources | 18:37 |
mriedem | i thought the refresh command was using the _sync_*_quotas stuff still, which is in the db api, but would have to dig | 18:38 |
*** salv-orlando has quit IRC | 18:38 | |
mriedem | if it's only using ReservableResources, and we don't have any of those, that's also grounds to remove it - but i need to check | 18:38 |
*** jwcroppe has joined #openstack-nova | 18:38 | |
*** salv-orlando has joined #openstack-nova | 18:39 | |
melwitt | mriedem: it does, but the _sync_* functions are only used on ReservableResources. the connection is made in nova/quota.py and each patch that removes the connection stops it from working that way | 18:39 |
melwitt | mriedem: we still have ReservableResources, they are just only user scoped and those unit tests are testing project scoped. it's confusing :( | 18:39 |
openstackgerrit | Kaitlin Farr proposed openstack/nova master: Remove deprecated keymgr code https://review.openstack.org/439855 | 18:39 |
*** eharney has quit IRC | 18:40 | |
mriedem | i'll dig a bit | 18:40 |
melwitt | the patch that removes the last ReservableResource is "Count instances" and we're not there yet. k. the "user scoped quotas" are the ones in the "project_user_quotas_table" and the "project scoped quotas" are in the "quotas" table (floating_ips, fixed_ips, networks). that's what the tests refer to when it says "project key" it means one of: 'floating_ips', 'fixed_ips', or 'networks' | 18:42 |
mriedem | melwitt: ok so https://github.com/openstack/nova/blob/master/nova/quota.py#L900 checks https://github.com/openstack/nova/blob/master/nova/quota.py#L416 | 18:43 |
melwitt | there's also the global var in nova/db/sqlalchemy/api.py PER_PROJECT_QUOTAS that show the project keys | 18:43 |
mriedem | and since per_project_resources is fixedips, floatingips and networks, | 18:43 |
mriedem | and none of those are ReservableResources | 18:43 |
melwitt | yes | 18:44 |
mriedem | but we could still have user-scoped resources, | 18:44 |
mriedem | like server groups | 18:44 |
*** jwcroppe has quit IRC | 18:44 | |
melwitt | yes, and we still do. we still have 'instances' 'cores' 'ram' as of that patch I think | 18:44 |
*** ericyoung has joined #openstack-nova | 18:45 | |
mriedem | let me check the tests being removed again | 18:45 |
melwitt | so those will still use reserve/commit/rollback and be synced/refreshed | 18:45 |
melwitt | k | 18:45 |
*** eharney has joined #openstack-nova | 18:46 | |
mriedem | rfolco: some issues in here https://review.openstack.org/#/c/460147/ | 18:46 |
openstackgerrit | Merged openstack/nova master: Fix using targeted cell context when finding services in cells https://review.openstack.org/475830 | 18:46 |
mriedem | rfolco: i'd really appreciate to leave the translation marker removal to a separate patch devoted to that | 18:46 |
melwitt | the only one we could argue is test_usage_refresh_invalid_user_key. I could do s/floating_ips/fake_resource_key/ there and the test would still work. the point of it had been, "verify you get Invalid if you try to refresh a valid key that isn't a valid user key" | 18:47 |
melwitt | and since there are no more project keys, we can't choose any valid keys. the test on L3270 is sort of identical | 18:48 |
rfolco | mriedem, looking | 18:48 |
openstackgerrit | Merged openstack/nova master: api-ref: fix hypervisor_hostname description for Ironic https://review.openstack.org/474723 | 18:51 |
*** mnestratov has quit IRC | 18:51 | |
openstackgerrit | Merged openstack/nova master: [doc] Updated sqlalchemy URL in migrate readme https://review.openstack.org/472549 | 18:51 |
mriedem | melwitt: removed my -1 for that - are we ok with https://review.openstack.org/#/c/457862/22/nova/network/floating_ips.py@225 ? | 18:52 |
mriedem | i think i talked myself into being ok with it | 18:52 |
mriedem | but i need you to close the deal | 18:52 |
melwitt | mriedem: thanks. the test on L3270 is the same test really anyway. looking at the next question ... | 18:55 |
*** thorst has joined #openstack-nova | 18:58 | |
*** mdrabe_ has joined #openstack-nova | 18:59 | |
*** mdrabe has quit IRC | 19:00 | |
melwitt | mriedem: looks like that's similar to the fixed ip one. I didn't think to call the self.deallocate* methods bc I thought to just use the inverse object functions to undo the create. but, I think technically maybe someone could create a floating ip and a dns domain in parallel and maybe could end up with orphaned dns domains that way. so it's probably most correct to call self.deallocate* | 19:00 |
melwitt | I'm not sure if associating the dns domain requires anything from the floating ip create response maybe. but either way it doesn't hurt to call the self.deallocate function | 19:01 |
mriedem | yeah i'm having a hard time figuring out how/where the dns domains and floating ips are correlated | 19:02 |
mriedem | just looking at https://developer.openstack.org/api-ref/compute/?expanded=create-allocate-floating-ip-address-detail#floating-ip-dns-records-os-floating-ip-dns-deprecated isn't helping me | 19:03 |
melwitt | yeah, I just cracked open the api-ref | 19:03 |
melwitt | heh "Floating IP DNS records (os-floating-ip-dns) (DEPRECATED)" | 19:03 |
*** jwcroppe has joined #openstack-nova | 19:03 | |
melwitt | maybe it's this one, you provide an IP and it creates an entry Floating IP DNS records (os-floating-ip-dns) (DEPRECATED) | 19:04 |
melwitt | gah | 19:05 |
melwitt | https://developer.openstack.org/api-ref/compute/?expanded=create-or-update-dns-domain-detail,create-or-update-dns-entry-detail#create-or-update-dns-entry | 19:05 |
*** jwcroppe has quit IRC | 19:06 | |
melwitt | I would guess that you would have to receive the address back from the API after creating the floating ip before you could create dns entries | 19:06 |
mriedem | wow https://developer.openstack.org/api-ref/compute/?expanded=create-or-update-dns-entry-detail#create-or-update-dns-entry | 19:07 |
mriedem | the request params table there is all sorts of wrong | 19:08 |
melwitt | oh, well that's the stuff from the url looks like | 19:08 |
mgagne | mriedem: actively working on the online volume extend change, adding all relevant unit tests atm. will complete tomorrow. I addressed most if not all comments. | 19:08 |
*** suresh12 has joined #openstack-nova | 19:08 | |
melwitt | but the other stuff isn't in the table. yeah | 19:09 |
mriedem | mgagne: sounds good | 19:09 |
*** slaweq has joined #openstack-nova | 19:10 | |
mriedem | mgagne: i imagine you may be a bit sore | 19:10 |
mgagne | all fine, just a bit out of time for today ;) | 19:10 |
mriedem | melwitt: yeah it's this i think https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/floating_ip_dns.py#L225 | 19:11 |
melwitt | ah, yep, that looks like it | 19:11 |
mriedem | melwitt: so you can't add a floating ip address dns entry to the dns domain if you don't already have the floating ip address, | 19:12 |
mriedem | so, | 19:12 |
mriedem | we can probably just delete the floating ip if we go over-quota, rather than call deallocate_floating_ip | 19:12 |
mriedem | like you're doing | 19:12 |
melwitt | yeah, so it just looks like the same deal as the fixed ip alloc/dealloc thing, you can't have made other associations before the REST call returns something to the user for the user to call back and make associations | 19:12 |
mriedem | ok, and if we mess this up, the .002% of people that will likely hit it can yell at us | 19:13 |
melwitt | so I probably lucked out on that. and yes, that is also true heh | 19:13 |
mriedem | we'll just tell them dansmith told us it was ok | 19:13 |
*** itlinux_ has quit IRC | 19:13 | |
melwitt | :D | 19:13 |
melwitt | sdague had a good point about calling that option "strict_quotas" instead of "recheck_quota." now that's gonna bug me | 19:13 |
melwitt | I know I could technically go ahead and deprecate "recheck_quota" and add strict_quotas and have both work for awhile | 19:14 |
*** thorst has quit IRC | 19:15 | |
*** gbarros has joined #openstack-nova | 19:15 | |
sdague | melwitt: ++ | 19:15 |
*** thorst has joined #openstack-nova | 19:15 | |
sdague | it's not in a stable yet, so now would be the time to do it | 19:15 |
mriedem | melwitt: +W on the change, | 19:15 |
mriedem | you're getting eerily close to seeing light at the end of this quota tunnel | 19:15 |
*** catinthe_ has quit IRC | 19:16 | |
melwitt | sdague: yeah, I think I'll do it then. that name is so much better. I just couldn't think of anything good | 19:16 |
*** slaweq has quit IRC | 19:17 | |
melwitt | mriedem: \o/ can't believe it! also, I need to add all of the recheck quota test cases for instances now, that's gonna be a pain | 19:17 |
*** jwcroppe has joined #openstack-nova | 19:18 | |
melwitt | I meant to have done it already but was busy and sort of avoiding it :( | 19:18 |
*** thorst has quit IRC | 19:20 | |
*** nicolasbock has quit IRC | 19:21 | |
*** priteau has joined #openstack-nova | 19:25 | |
mriedem | that's ok, i'm putting off supporting compute node paging across multiple cells | 19:26 |
*** priteau has joined #openstack-nova | 19:26 | |
*** gbarros_ has joined #openstack-nova | 19:28 | |
*** ericyoung has quit IRC | 19:29 | |
*** edmondsw has quit IRC | 19:29 | |
*** gbarros has quit IRC | 19:30 | |
sdague | melwitt: so, given the bug dan was debugging this morning, is this bit right - https://review.openstack.org/#/c/416521/46/nova/compute/api.py@3022 | 19:33 |
sdague | I also kind of wonder if there is going to need to be some standard scatter/gather callers here | 19:33 |
*** hoonetorg has quit IRC | 19:34 | |
dansmith | sdague: I just replied to both | 19:34 |
melwitt | sdague: it is now wrong since dansmith's change to how target_cell works. the change is really old | 19:35 |
melwitt | so, good catch | 19:35 |
*** gomarivera has quit IRC | 19:36 | |
*** gomarivera has joined #openstack-nova | 19:36 | |
*** liangy has quit IRC | 19:37 | |
*** jwcroppe has quit IRC | 19:37 | |
*** rajathagasthya has joined #openstack-nova | 19:38 | |
*** dimtruck is now known as zz_dimtruck | 19:39 | |
openstackgerrit | Merged openstack/nova master: placement: separate normalize_resources_qs_param https://review.openstack.org/475599 | 19:40 |
*** gomarivera has quit IRC | 19:41 | |
openstackgerrit | Merged openstack/nova master: Use more specific asserts in tests https://review.openstack.org/466609 | 19:41 |
*** qwertyco has joined #openstack-nova | 19:43 | |
*** gbarros_ has quit IRC | 19:48 | |
*** qwertyco has quit IRC | 19:50 | |
*** hoonetorg has joined #openstack-nova | 19:50 | |
*** gomarivera has joined #openstack-nova | 19:53 | |
*** gomarivera has quit IRC | 19:53 | |
*** zzzeek has quit IRC | 19:53 | |
*** gomarivera has joined #openstack-nova | 19:54 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971 | 19:57 |
*** edmondsw has joined #openstack-nova | 20:00 | |
*** jwcroppe has joined #openstack-nova | 20:04 | |
*** Apoorva_ has joined #openstack-nova | 20:05 | |
*** Apoorva has quit IRC | 20:08 | |
*** smatzek has quit IRC | 20:09 | |
*** slaweq has joined #openstack-nova | 20:11 | |
*** jwcroppe has quit IRC | 20:12 | |
*** jwcroppe has joined #openstack-nova | 20:13 | |
*** hoonetorg has quit IRC | 20:15 | |
*** kylek3h has quit IRC | 20:16 | |
*** jwcroppe has quit IRC | 20:17 | |
*** mdrabe_ is now known as mdrabe | 20:18 | |
*** jerrygb has quit IRC | 20:19 | |
*** efried has joined #openstack-nova | 20:20 | |
*** amotoki_away is now known as amotoki | 20:24 | |
*** liverpooler has quit IRC | 20:24 | |
*** mnestratov has joined #openstack-nova | 20:27 | |
*** suresh12 has quit IRC | 20:28 | |
*** suresh12 has joined #openstack-nova | 20:29 | |
*** pchavva has quit IRC | 20:31 | |
cfriesen | has anyone looked at what the qemu logs look like when block live-migrating under heavy localdisk IO? does it show up in the "Migration running for..." log from the live migration monitor? | 20:31 |
*** amotoki is now known as amotoki_away | 20:33 | |
*** xyang1 has joined #openstack-nova | 20:33 | |
*** hoonetorg has joined #openstack-nova | 20:36 | |
*** smatzek has joined #openstack-nova | 20:38 | |
*** suresh12 has quit IRC | 20:39 | |
*** zzzeek has joined #openstack-nova | 20:42 | |
*** awaugama has quit IRC | 20:42 | |
*** suresh12 has joined #openstack-nova | 20:44 | |
*** ijw has joined #openstack-nova | 20:45 | |
*** jwcroppe has joined #openstack-nova | 20:46 | |
*** suresh12 has quit IRC | 20:46 | |
*** suresh12 has joined #openstack-nova | 20:46 | |
*** eharney has quit IRC | 20:46 | |
mriedem | melwitt: were you looking for a release note on this? https://review.openstack.org/#/c/393695/ | 20:48 |
melwitt | mriedem: not really, just wanted a sanity check that we can be assured adding a minimum wouldn't break anyone. maybe a release note is the right thing? | 20:49 |
melwitt | I don't know why anyone would use -1 but what if they did. I didn't check yet what a -1 would do for that interval, maybe it just wouldn't even work | 20:50 |
mriedem | melwitt: don't know, but i left a reply in there too - -1 has special meanings in things | 20:50 |
mriedem | like retry forever, | 20:50 |
mriedem | or disable | 20:50 |
melwitt | yeah. so maybe the question is, does -1 work presently or not. if not, then the point is moot | 20:51 |
*** jwcroppe has quit IRC | 20:51 | |
*** jwcroppe has joined #openstack-nova | 20:51 | |
*** cleong has quit IRC | 20:53 | |
*** jwcroppe has quit IRC | 20:56 | |
openstackgerrit | Merged openstack/nova master: Count floating ips to check quota https://review.openstack.org/457862 | 20:57 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: hypervisor_hostname must match get_available_nodes https://review.openstack.org/475926 | 21:03 |
*** lucasxu has quit IRC | 21:05 | |
*** gbarros has joined #openstack-nova | 21:07 | |
*** tidwellr has quit IRC | 21:08 | |
*** gbarros has quit IRC | 21:08 | |
openstackgerrit | Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API https://review.openstack.org/449288 | 21:09 |
*** owalsh_ has joined #openstack-nova | 21:10 | |
*** gbarros has joined #openstack-nova | 21:12 | |
*** owalsh has quit IRC | 21:12 | |
*** gbarros has quit IRC | 21:14 | |
*** gbarros has joined #openstack-nova | 21:15 | |
openstackgerrit | Merged openstack/nova master: Remove translation of log messages https://review.openstack.org/467948 | 21:16 |
*** salv-orl_ has joined #openstack-nova | 21:17 | |
openstackgerrit | Merged openstack/nova master: [placement] Update placement devref to modern features https://review.openstack.org/469047 | 21:17 |
*** smatzek has quit IRC | 21:17 | |
*** efried has quit IRC | 21:18 | |
*** salv-orlando has quit IRC | 21:20 | |
*** armax has joined #openstack-nova | 21:22 | |
*** jwcroppe has joined #openstack-nova | 21:22 | |
*** crushil has quit IRC | 21:22 | |
*** efried has joined #openstack-nova | 21:22 | |
*** crushil has joined #openstack-nova | 21:22 | |
*** catintheroof has joined #openstack-nova | 21:25 | |
*** crushil has quit IRC | 21:27 | |
*** gouthamr has quit IRC | 21:31 | |
*** smatzek has joined #openstack-nova | 21:32 | |
*** thorst has joined #openstack-nova | 21:32 | |
*** dave-mccowan has quit IRC | 21:35 | |
*** pramodrj07 has joined #openstack-nova | 21:37 | |
*** kylek3h has joined #openstack-nova | 21:38 | |
*** gbarros has quit IRC | 21:40 | |
*** baoli has quit IRC | 21:40 | |
*** MasterOfBugs has quit IRC | 21:40 | |
*** rfolco has quit IRC | 21:42 | |
*** rajathagasthya has quit IRC | 21:45 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Fix and optimize external_events for multiple cells https://review.openstack.org/445142 | 21:47 |
*** thorst has quit IRC | 21:47 | |
*** gouthamr has joined #openstack-nova | 21:47 | |
*** mingyu has joined #openstack-nova | 21:49 | |
openstackgerrit | Eric Fried proposed openstack/nova master: hypervisor_hostname must match get_available_nodes https://review.openstack.org/475926 | 21:52 |
*** gbarros has joined #openstack-nova | 21:53 | |
*** mingyu has quit IRC | 21:53 | |
*** armax has quit IRC | 21:54 | |
*** slaweq has quit IRC | 21:55 | |
*** slaweq has joined #openstack-nova | 21:56 | |
*** markvoelker has quit IRC | 21:56 | |
*** slaweq has quit IRC | 21:56 | |
*** slaweq has joined #openstack-nova | 21:57 | |
*** thorst has joined #openstack-nova | 21:57 | |
*** smatzek has quit IRC | 21:59 | |
*** slaweq has quit IRC | 21:59 | |
*** priteau has quit IRC | 22:00 | |
*** slaweq has joined #openstack-nova | 22:00 | |
*** gbarros has quit IRC | 22:00 | |
mriedem | dansmith: we need to talk about https://review.openstack.org/#/c/391941/ | 22:03 |
mriedem | the easy one first https://review.openstack.org/#/c/391941/50/nova/compute/cells_api.py | 22:03 |
*** armax has joined #openstack-nova | 22:03 | |
mriedem | i'm thinking we don't need to be passing that tag kwarg through for cells v1 | 22:03 |
dansmith | totes agree | 22:04 |
*** markvoelker has joined #openstack-nova | 22:04 | |
mriedem | ok now the hard one https://review.openstack.org/#/c/391941/50/nova/compute/api.py | 22:04 |
mriedem | the way artom has this written, | 22:04 |
dansmith | if we're checking service version and failing, | 22:04 |
mriedem | if you try to attach a volume with a tag to a shelved offloaded instance, he fails with a 400 | 22:04 |
dansmith | we should just always fail in cells_api if you pass a tag | 22:04 |
*** thorst has quit IRC | 22:05 | |
mriedem | good point | 22:05 |
mriedem | mayhap you'd like to leave that comment | 22:05 |
*** thorst has joined #openstack-nova | 22:05 | |
dansmith | mayhap I already did | 22:05 |
mriedem | ok, so the shelved offloaded case | 22:05 |
*** slaweq has quit IRC | 22:05 | |
mriedem | he's got this failing for a shelved offloaded instance since you can't call reserve_block_device_name on the compute | 22:06 |
mriedem | which is what's checking the driver flag | 22:06 |
mriedem | here https://review.openstack.org/#/c/391941/50/nova/compute/manager.py | 22:06 |
*** mdrabe has quit IRC | 22:06 | |
dansmith | ugh, we support regular volume attach for offloaded instances? | 22:07 |
mriedem | i'm trying to decide if we should support attaching a volume with a tag to a shelved offloaded instance. his argument was it's not good to mysteriously fail to apply the tag when unshelving | 22:07 |
mriedem | eff yes we do | 22:07 |
mriedem | HPE MFer | 22:07 |
*** thorst has quit IRC | 22:07 | |
dansmith | yeah, I understand his point, but.. oye | 22:07 |
*** esberglu has quit IRC | 22:07 | |
mriedem | so with attach to a shelve offloaded instance, we prep the bdms (attach them) to the guest when unshelving | 22:07 |
*** gbarros has joined #openstack-nova | 22:08 | |
mriedem | basically server create, but you know | 22:08 |
mriedem | like bfv for an already booted instance - it's the best | 22:08 |
*** esberglu has joined #openstack-nova | 22:08 | |
*** efried has quit IRC | 22:08 | |
mriedem | anyway, my thinking was we could support attaching volumes to a shelved offloaded instance in basically the same way as we treat normal create | 22:08 |
dansmith | right, but it would suck to have destined your instance to certain doom while offloaded, | 22:08 |
dansmith | and then be unable to ever boot it ever because you have no supporting hypervisors | 22:08 |
mriedem | it wouldn't be certain doom | 22:09 |
mriedem | and the same can be said for creating a server with a tagged bdm | 22:09 |
mriedem | if it ends up on a compute that doesn't support tagged devices, it's a BuildAbortException | 22:09 |
mriedem | remember me freaking out about this earlier in the week? | 22:09 |
*** david_4 has quit IRC | 22:10 | |
*** sapcc-bot has quit IRC | 22:10 | |
*** carthaca_ has quit IRC | 22:10 | |
*** databus23_ has quit IRC | 22:10 | |
mriedem | on normal server create we check here https://github.com/openstack/nova/blob/4e6a7c18b1c8340b9fdc93ee02e56b5a29a99648/nova/compute/manager.py#L1930 | 22:10 |
mriedem | if that fails (it's a vmware host), we abort the build | 22:11 |
mriedem | fin | 22:11 |
mriedem | i was thinking we could allow attaching a tagged bdm to a shelved offloaded instance, and during unshelve we basically have the same flag check, and if the compute doesn't support it then we fail the unshelve | 22:12 |
*** openstack has joined #openstack-nova | 22:16 | |
*** david_ has joined #openstack-nova | 22:16 | |
*** Apoorva_ has quit IRC | 22:17 | |
*** dansmith_ is now known as dansmith | 22:17 | |
dansmith | mriedem: well, the options are: | 22:17 |
dansmith | 1. refuse to attach with a tag to offloaded instances | 22:17 |
dansmith | 2. allow it and fail on unshelve | 22:17 |
*** oomichi has quit IRC | 22:17 | |
*** Apoorva has joined #openstack-nova | 22:17 | |
*** dansmith is now known as Guest96259 | 22:17 | |
Guest96259 | ...I guess that's it, right? | 22:17 |
mriedem | Guest96259: basically | 22:18 |
*** david_ has quit IRC | 22:18 | |
*** databus23_2 has quit IRC | 22:18 | |
*** carthaca_2 has quit IRC | 22:18 | |
*** sapcc-bot3 has quit IRC | 22:18 | |
*** owalsh_ is now known as owalsh | 22:18 | |
*** databus23_ has joined #openstack-nova | 22:19 | |
*** sapcc-bot has joined #openstack-nova | 22:19 | |
*** david_ has joined #openstack-nova | 22:19 | |
*** carthaca_ has joined #openstack-nova | 22:19 | |
Guest96259 | I dunno, #1 seems not that terrible to me, given how much stuff doesn't really work with shelve | 22:19 |
*** david_ has quit IRC | 22:19 | |
*** databus23_ has quit IRC | 22:19 | |
*** carthaca_ has quit IRC | 22:19 | |
mriedem | Guest96259: failing in unshelve seems like a can of worms given we don't do anything with the instance if spawn fails | 22:19 |
*** oomichi has joined #openstack-nova | 22:19 | |
*** sapcc-bot has quit IRC | 22:20 | |
Guest96259 | yeah | 22:20 |
*** dosaboy has joined #openstack-nova | 22:20 | |
mriedem | we'll revert the task_state, | 22:20 |
mriedem | record a fault | 22:20 |
mriedem | but we don't clean anything up, i.e. https://bugs.launchpad.net/nova/+bug/1627694 | 22:20 |
mriedem | i guess you don't want the instance in ERROR state b/c then you can't attempt to unshelve it again | 22:20 |
*** david_ has joined #openstack-nova | 22:20 | |
*** sapcc-bot has joined #openstack-nova | 22:20 | |
openstack | Launchpad bug 1627694 in OpenStack Compute (nova) "unshelving an instance doesn't rollback volumes connections on failure" [Low,In progress] - Assigned to Shoham Peller (shoham-peller) | 22:21 |
*** edmondsw has quit IRC | 22:23 | |
*** gomarivera has joined #openstack-nova | 22:23 | |
mriedem | mordred: is your new api user ML a place i can ask if people are actually using some of the compute APIs? like shelve? | 22:24 |
*** StevenK has joined #openstack-nova | 22:25 | |
*** suresh12 has quit IRC | 22:28 | |
*** gomarivera has quit IRC | 22:29 | |
*** slaweq has joined #openstack-nova | 22:30 | |
*** esberglu has joined #openstack-nova | 22:32 | |
*** Apoorva_ has joined #openstack-nova | 22:33 | |
*** Apoorva has quit IRC | 22:33 | |
*** slaweq has quit IRC | 22:37 | |
*** esberglu has quit IRC | 22:37 | |
openstackgerrit | Merged openstack/nova master: libvirt: Extract method _guest_add_video_device https://review.openstack.org/470783 | 22:37 |
openstackgerrit | Merged openstack/nova master: libvirt: Extract method _guest_add_pci_devices https://review.openstack.org/470784 | 22:37 |
*** slaweq has joined #openstack-nova | 22:38 | |
openstackgerrit | Merged openstack/nova master: libvirt: Extract method _guest_add_watchdog_action https://review.openstack.org/470785 | 22:41 |
*** acormier has quit IRC | 22:41 | |
openstackgerrit | Merged openstack/nova master: libvirt: Extract method _guest_add_memory_balloon https://review.openstack.org/470786 | 22:42 |
*** sbezverk has quit IRC | 22:42 | |
*** acormier has joined #openstack-nova | 22:42 | |
*** jwcroppe has joined #openstack-nova | 22:43 | |
*** slaweq has quit IRC | 22:43 | |
*** jwcroppe has quit IRC | 22:43 | |
*** suresh12_ has joined #openstack-nova | 22:44 | |
openstackgerrit | Merged openstack/nova master: libvirt: Extract method _guest_add_spice_channel https://review.openstack.org/470787 | 22:44 |
*** acormier has quit IRC | 22:46 | |
*** mriedem1 has joined #openstack-nova | 22:48 | |
*** gbarros has quit IRC | 22:49 | |
mordred | mriedem: sure! | 22:49 |
*** mriedem has quit IRC | 22:50 | |
*** pramodrj07 has quit IRC | 22:52 | |
*** jwcroppe has joined #openstack-nova | 22:52 | |
*** pramodrj07 has joined #openstack-nova | 22:53 | |
openstackgerrit | Merged openstack/nova master: remove null_safe_int from module scope https://review.openstack.org/455778 | 22:54 |
*** rajathagasthya has quit IRC | 22:54 | |
*** xyang1 has quit IRC | 22:55 | |
*** rajathagasthya has joined #openstack-nova | 22:55 | |
* Guest96259 rages | 22:56 | |
Guest96259 | oh double rage | 22:56 |
*** Guest96259 is now known as dansmith | 22:56 | |
*** gbarros has joined #openstack-nova | 22:57 | |
*** annegentle has quit IRC | 22:58 | |
*** salv-orl_ has quit IRC | 22:59 | |
*** salv-orlando has joined #openstack-nova | 22:59 | |
*** gbarros has quit IRC | 23:00 | |
*** Apoorva has joined #openstack-nova | 23:02 | |
*** gbarros has joined #openstack-nova | 23:02 | |
*** jwcroppe has quit IRC | 23:03 | |
*** salv-orlando has quit IRC | 23:03 | |
*** gomarivera has joined #openstack-nova | 23:03 | |
*** jwcroppe has joined #openstack-nova | 23:04 | |
*** Apoorva_ has quit IRC | 23:05 | |
*** jwcroppe has quit IRC | 23:09 | |
*** gomarivera has quit IRC | 23:09 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971 | 23:09 |
*** slaweq has joined #openstack-nova | 23:14 | |
*** catintheroof has quit IRC | 23:18 | |
*** slaweq has quit IRC | 23:19 | |
*** harlowja has quit IRC | 23:19 | |
*** annegentle has joined #openstack-nova | 23:19 | |
*** annegentle has quit IRC | 23:22 | |
*** Apoorva has quit IRC | 23:24 | |
*** armax has quit IRC | 23:26 | |
*** mvk has joined #openstack-nova | 23:27 | |
*** gomarivera has joined #openstack-nova | 23:27 | |
openstackgerrit | Merged openstack/nova master: Use plain routes list for os-remote-consoles instead of stevedore https://review.openstack.org/474930 | 23:27 |
gmann | alex_xu, mriedem1 its ready now - https://review.openstack.org/#/c/467999/ | 23:28 |
mriedem1 | ok | 23:31 |
*** mriedem1 is now known as mriedem | 23:31 | |
*** ijw has quit IRC | 23:33 | |
*** slaweq has joined #openstack-nova | 23:35 | |
mriedem | alex_xu: i'm not sure if i'll be in the next api meeting but i added something to open discussion in the agenda https://wiki.openstack.org/wiki/Meetings/NovaAPI#Agenda | 23:36 |
*** gbarros has quit IRC | 23:37 | |
*** gbarros has joined #openstack-nova | 23:38 | |
*** slaweq has quit IRC | 23:40 | |
openstackgerrit | melanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object https://review.openstack.org/446244 | 23:40 |
openstackgerrit | melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242 | 23:40 |
openstackgerrit | melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | 23:40 |
openstackgerrit | melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521 | 23:40 |
openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 23:40 |
openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 23:40 |
openstackgerrit | melanie witt proposed openstack/nova master: Add scatter_gather_all_cells() utility https://review.openstack.org/475957 | 23:40 |
*** annegentle has joined #openstack-nova | 23:42 | |
*** sbezverk has joined #openstack-nova | 23:44 | |
*** gomarivera has quit IRC | 23:46 | |
*** annegentle has quit IRC | 23:47 | |
*** acormier has joined #openstack-nova | 23:51 | |
*** slaweq has joined #openstack-nova | 23:56 | |
*** mnestratov has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!