*** haomaiwang has quit IRC | 00:01 | |
*** haomaiwang has joined #openstack-nova | 00:01 | |
*** dedery has quit IRC | 00:02 | |
*** mingyu has joined #openstack-nova | 00:02 | |
*** Sukhdev has quit IRC | 00:03 | |
*** nelsnels_ has quit IRC | 00:04 | |
*** ccarmack has joined #openstack-nova | 00:04 | |
*** Sukhdev has joined #openstack-nova | 00:05 | |
*** dims has joined #openstack-nova | 00:06 | |
*** markvoelker has quit IRC | 00:06 | |
*** nelsnelson has joined #openstack-nova | 00:06 | |
*** zz_dimtruck is now known as dimtruck | 00:07 | |
*** Bjoern_ has quit IRC | 00:08 | |
*** mingyu has quit IRC | 00:08 | |
openstackgerrit | Jim Rollenhagen proposed openstack/nova: Ironic: Clean up if configdrive build fails https://review.openstack.org/287520 | 00:08 |
---|---|---|
*** _ducttape_ has quit IRC | 00:09 | |
*** amotoki has quit IRC | 00:11 | |
*** mriedem has joined #openstack-nova | 00:12 | |
*** sdague has quit IRC | 00:18 | |
*** ByPasS has quit IRC | 00:20 | |
*** Sukhdev has quit IRC | 00:20 | |
*** mylu has joined #openstack-nova | 00:21 | |
*** hemna is now known as hemnafk | 00:24 | |
*** mingyu has joined #openstack-nova | 00:25 | |
*** Jeffrey4l has quit IRC | 00:27 | |
*** qeelee has joined #openstack-nova | 00:28 | |
*** andreykurilin__ has quit IRC | 00:33 | |
*** dimtruck is now known as zz_dimtruck | 00:36 | |
mriedem | alaski: i steered you down an evil path on https://review.openstack.org/#/c/278125/ :) | 00:37 |
*** nic has quit IRC | 00:37 | |
*** amotoki has joined #openstack-nova | 00:40 | |
*** rk4n has joined #openstack-nova | 00:43 | |
*** bjornar has quit IRC | 00:43 | |
*** thorst has joined #openstack-nova | 00:44 | |
*** ZZelle_ has quit IRC | 00:45 | |
*** rk4n has quit IRC | 00:47 | |
*** rk4n has joined #openstack-nova | 00:49 | |
*** amotoki has quit IRC | 00:57 | |
*** qeelee has quit IRC | 00:57 | |
*** haomaiwang has quit IRC | 01:01 | |
*** zz_dimtruck is now known as dimtruck | 01:01 | |
*** haomaiwang has joined #openstack-nova | 01:01 | |
*** qeelee has joined #openstack-nova | 01:03 | |
*** ducttape_ has joined #openstack-nova | 01:03 | |
*** qeelee has quit IRC | 01:07 | |
*** dims has quit IRC | 01:08 | |
*** markvoelker has joined #openstack-nova | 01:10 | |
*** geekinutah has quit IRC | 01:11 | |
*** doug-fish has joined #openstack-nova | 01:12 | |
*** zenoway has joined #openstack-nova | 01:13 | |
*** takedakn has joined #openstack-nova | 01:15 | |
*** EinstCrazy has joined #openstack-nova | 01:17 | |
*** unicell has joined #openstack-nova | 01:18 | |
*** zenoway has quit IRC | 01:18 | |
*** ducttape_ has quit IRC | 01:18 | |
openstackgerrit | Jim Rollenhagen proposed openstack/nova: Cellsv1: don't blow up keypair_get_at_top on error https://review.openstack.org/287542 | 01:19 |
*** penick has joined #openstack-nova | 01:19 | |
jroll | JayF: ^ if curious | 01:20 |
*** suresh12 has quit IRC | 01:21 | |
JayF | thanks jroll, that looks good to me and should resolve the exception you found | 01:21 |
JayF | does it need a bug on the patch? IDK how strict nova is about that | 01:21 |
jroll | probably | 01:21 |
mriedem | yes | 01:22 |
mriedem | especially if you want to backport it to liberty | 01:22 |
jroll | I just wanted to get it up now to start :) | 01:23 |
jroll | I'll file one tonight/tomorrow | 01:23 |
openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common (1/4) https://review.openstack.org/286442 | 01:23 |
openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common (2/4) https://review.openstack.org/286955 | 01:23 |
*** amotoki has joined #openstack-nova | 01:23 | |
*** edleafe has quit IRC | 01:23 | |
openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common https://review.openstack.org/286958 | 01:24 |
*** edleafe has joined #openstack-nova | 01:24 | |
*** haomaiwang has quit IRC | 01:25 | |
mriedem | ok | 01:25 |
mriedem | comment inline | 01:25 |
openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common (4/4) https://review.openstack.org/287544 | 01:27 |
*** penick has quit IRC | 01:28 | |
*** Sukhdev has joined #openstack-nova | 01:29 | |
*** ssurana has quit IRC | 01:30 | |
*** unicell1 has joined #openstack-nova | 01:30 | |
*** coreywright has quit IRC | 01:31 | |
*** Sukhdev has quit IRC | 01:31 | |
*** unicell has quit IRC | 01:32 | |
*** sc68cal_ has quit IRC | 01:33 | |
*** haukebruno has joined #openstack-nova | 01:34 | |
openstackgerrit | Feng Xi Yan proposed openstack/nova: Enable VMWare driver to attach more than one SCSI LUN to VM. https://review.openstack.org/287055 | 01:36 |
*** gongysh has joined #openstack-nova | 01:37 | |
openstackgerrit | Feng Xi Yan proposed openstack/nova: Enable VMWare driver to attach more than one SCSI LUN to VM. https://review.openstack.org/287055 | 01:38 |
jroll | mriedem: thanks man | 01:38 |
*** suresh12 has joined #openstack-nova | 01:39 | |
*** rk4n has quit IRC | 01:42 | |
*** ducttape_ has joined #openstack-nova | 01:43 | |
*** MVenesio has joined #openstack-nova | 01:43 | |
*** Marga_ has quit IRC | 01:43 | |
*** rfolco has quit IRC | 01:44 | |
*** coreywright has joined #openstack-nova | 01:44 | |
*** gongysh has quit IRC | 01:45 | |
*** Swami has quit IRC | 01:47 | |
*** openstackgerrit has quit IRC | 01:47 | |
*** openstackgerrit has joined #openstack-nova | 01:48 | |
*** MVenesio has quit IRC | 01:49 | |
*** Sukhdev has joined #openstack-nova | 01:49 | |
*** phuongnh has joined #openstack-nova | 01:51 | |
*** suresh12 has quit IRC | 01:51 | |
*** Sukhdev has quit IRC | 01:52 | |
*** zenoway has joined #openstack-nova | 01:52 | |
*** fawadkhaliq has quit IRC | 01:53 | |
*** unicell1 has quit IRC | 01:55 | |
*** claudiub|2 has joined #openstack-nova | 01:56 | |
*** zenoway has quit IRC | 01:57 | |
*** claudiub has quit IRC | 01:58 | |
*** dims has joined #openstack-nova | 01:59 | |
*** mtanino has quit IRC | 02:01 | |
*** haomaiwang has joined #openstack-nova | 02:02 | |
*** takedakn has quit IRC | 02:02 | |
*** anush has joined #openstack-nova | 02:03 | |
mriedem | dansmith: looking at this, it seems we should just eager load instance.pci_devices in a bunch of cases http://logs.openstack.org/39/284839/3/check/gate-tempest-dsvm-full/22bb744/logs/screen-n-cpu.txt.gz?level=TRACE | 02:04 |
mriedem | which kind of sucks | 02:04 |
*** efried has quit IRC | 02:05 | |
*** thorst has quit IRC | 02:10 | |
*** dimtruck is now known as zz_dimtruck | 02:10 | |
*** thorst has joined #openstack-nova | 02:10 | |
*** doug-fis_ has joined #openstack-nova | 02:16 | |
*** xek_ has joined #openstack-nova | 02:16 | |
*** armax_ has joined #openstack-nova | 02:16 | |
*** markstur_ has joined #openstack-nova | 02:16 | |
*** huats__ has joined #openstack-nova | 02:17 | |
*** Marga_ has joined #openstack-nova | 02:17 | |
*** shaohe_f- has joined #openstack-nova | 02:18 | |
*** Marga_ has quit IRC | 02:18 | |
*** thorst has quit IRC | 02:18 | |
*** _xin1 has joined #openstack-nova | 02:18 | |
*** dosaboy_ has joined #openstack-nova | 02:18 | |
*** gerhardq1x has joined #openstack-nova | 02:18 | |
*** Marga_ has joined #openstack-nova | 02:19 | |
*** mariusv_ has joined #openstack-nova | 02:19 | |
*** aloga_ has joined #openstack-nova | 02:19 | |
*** tobasco_ has joined #openstack-nova | 02:19 | |
*** eliqiao_ has joined #openstack-nova | 02:20 | |
*** alex_xu_ has joined #openstack-nova | 02:21 | |
*** claudiub|2 has quit IRC | 02:22 | |
*** sgordon` has joined #openstack-nova | 02:22 | |
*** zengine has joined #openstack-nova | 02:22 | |
*** ducttape_ has quit IRC | 02:23 | |
*** mkoderer___ has joined #openstack-nova | 02:23 | |
*** dosaboy_ has quit IRC | 02:23 | |
*** agireud has quit IRC | 02:23 | |
*** mkoderer__ has quit IRC | 02:23 | |
*** sgordon has quit IRC | 02:23 | |
*** gerhardqux has quit IRC | 02:23 | |
*** dosaboy has quit IRC | 02:23 | |
*** armax has quit IRC | 02:23 | |
*** huats_ has quit IRC | 02:23 | |
*** shaohe_feng_ has quit IRC | 02:23 | |
*** stevemar has quit IRC | 02:23 | |
*** bjornar__ has quit IRC | 02:23 | |
*** mariusv has quit IRC | 02:23 | |
*** markstur has quit IRC | 02:23 | |
*** ircuser-1 has quit IRC | 02:23 | |
*** aloga has quit IRC | 02:23 | |
*** alex_xu has quit IRC | 02:23 | |
*** eliqiao has quit IRC | 02:23 | |
*** _xin has quit IRC | 02:23 | |
*** doug-fish has quit IRC | 02:23 | |
*** ccesario has quit IRC | 02:23 | |
*** xek has quit IRC | 02:23 | |
*** tobasco has quit IRC | 02:23 | |
*** armax_ is now known as armax | 02:23 | |
*** lunarlamp is now known as mariusv | 02:23 | |
*** bjornar__ has joined #openstack-nova | 02:24 | |
*** agireud has joined #openstack-nova | 02:24 | |
*** stevemar has joined #openstack-nova | 02:24 | |
*** dosaboy has joined #openstack-nova | 02:24 | |
*** doug-fis_ has quit IRC | 02:24 | |
*** yamahata has quit IRC | 02:24 | |
*** ircuser-1 has joined #openstack-nova | 02:25 | |
*** ccesario has joined #openstack-nova | 02:26 | |
*** doug-fish has joined #openstack-nova | 02:27 | |
*** Bjoern has joined #openstack-nova | 02:27 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: DNM: see where instance.pci_devices is getting lazy-loaded https://review.openstack.org/284839 | 02:27 |
*** suresh12 has joined #openstack-nova | 02:28 | |
*** Bjoern has quit IRC | 02:33 | |
*** ducttape_ has joined #openstack-nova | 02:34 | |
*** eliqiao_ is now known as eliqiao | 02:37 | |
*** y_sawai has joined #openstack-nova | 02:38 | |
mriedem | alaski: i'm going to fix that cells failure so i can +2 that create build request in api change and keep the series moving | 02:42 |
openstackgerrit | zwei proposed openstack/nova: Nova generate nova.conf.sample configuration file is incomplete eg: Not Found option #iscsi_use_multipath in [ libvirt ] eg: NOt Found option #iser_volume_opts in [ libvirt ] https://review.openstack.org/276649 | 02:43 |
*** y_sawai has quit IRC | 02:44 | |
*** gongysh has joined #openstack-nova | 02:45 | |
mriedem | markus_z: there are 3 patches up like this, it'd be good if someone that cares about config options herded those cats https://review.openstack.org/#/c/265299/ | 02:47 |
*** hoangcx has joined #openstack-nova | 02:49 | |
*** gcb has joined #openstack-nova | 02:54 | |
*** vilobhmm11 has joined #openstack-nova | 02:54 | |
*** gongysh has quit IRC | 02:54 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Create BuildRequest object during boot process https://review.openstack.org/278125 | 02:54 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add BuildRequest object https://review.openstack.org/278124 | 02:54 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Make InstanceMappings.cell_id nullable https://review.openstack.org/278598 | 02:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Populate instance_mappings during boot https://review.openstack.org/263925 | 02:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Include CellMapping in InstanceMapping object https://review.openstack.org/286309 | 02:55 |
*** y_sawai has joined #openstack-nova | 02:56 | |
openstackgerrit | zwei proposed openstack/nova: Nova generate nova.conf.sample configuration file is incomplete eg: Not Found option #power_state_check_timeframe [hyperv] https://review.openstack.org/276649 | 02:57 |
*** y_sawai_ has joined #openstack-nova | 02:58 | |
*** y_sawai_ has quit IRC | 02:59 | |
*** y_sawai_ has joined #openstack-nova | 03:00 | |
*** haomaiwang has quit IRC | 03:01 | |
*** y_sawai has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-nova | 03:01 | |
*** houming has joined #openstack-nova | 03:02 | |
*** y_sawai has joined #openstack-nova | 03:03 | |
*** y_sawai has quit IRC | 03:05 | |
*** y_sawai has joined #openstack-nova | 03:05 | |
*** y_sawai_ has quit IRC | 03:06 | |
*** y_sawai_ has joined #openstack-nova | 03:06 | |
*** apoorvad has quit IRC | 03:07 | |
*** y_sawai_ has quit IRC | 03:07 | |
*** y_sawai has quit IRC | 03:07 | |
*** y_sawai has joined #openstack-nova | 03:08 | |
*** y_sawai_ has joined #openstack-nova | 03:08 | |
*** y_sawai_ has quit IRC | 03:10 | |
*** doug-fish has quit IRC | 03:10 | |
*** y_sawai_ has joined #openstack-nova | 03:10 | |
*** browne has quit IRC | 03:11 | |
*** achanda has joined #openstack-nova | 03:11 | |
*** y_sawai__ has joined #openstack-nova | 03:11 | |
*** y_sawa___ has joined #openstack-nova | 03:12 | |
*** y_sawai__ has quit IRC | 03:12 | |
*** y_sawai has quit IRC | 03:12 | |
*** y_sawai_ has quit IRC | 03:15 | |
*** thorst has joined #openstack-nova | 03:16 | |
*** y_sawa___ has quit IRC | 03:17 | |
*** jwang_ has joined #openstack-nova | 03:21 | |
*** MCoLo___ has joined #openstack-nova | 03:21 | |
*** jwcroppe_ has joined #openstack-nova | 03:22 | |
*** kmartin__ has joined #openstack-nova | 03:23 | |
*** thorst has quit IRC | 03:24 | |
*** suresh12 has quit IRC | 03:24 | |
*** sulo_ has joined #openstack-nova | 03:24 | |
*** jdurgin1 has quit IRC | 03:25 | |
*** ujjain- has joined #openstack-nova | 03:25 | |
*** oomichi_ has joined #openstack-nova | 03:25 | |
*** taneee has joined #openstack-nova | 03:25 | |
*** dansmith_ has joined #openstack-nova | 03:25 | |
*** diana_clarke has joined #openstack-nova | 03:25 | |
*** markmcclain1 has joined #openstack-nova | 03:25 | |
*** spring_ has joined #openstack-nova | 03:25 | |
*** strictlyb has joined #openstack-nova | 03:25 | |
*** yamahata has joined #openstack-nova | 03:26 | |
*** ducttape_ has quit IRC | 03:26 | |
*** sbelous1 has joined #openstack-nova | 03:26 | |
*** dansmith has quit IRC | 03:26 | |
*** MCoLo__ has quit IRC | 03:26 | |
*** ababich1 has quit IRC | 03:26 | |
*** sbelous has quit IRC | 03:26 | |
*** 18WAAAV0X has quit IRC | 03:26 | |
*** toure has quit IRC | 03:26 | |
*** _diana_ has quit IRC | 03:26 | |
*** ujjain has quit IRC | 03:26 | |
*** ccard has quit IRC | 03:26 | |
*** jwang__ has quit IRC | 03:26 | |
*** sb- has quit IRC | 03:26 | |
*** edleafe has quit IRC | 03:26 | |
*** sulo has quit IRC | 03:26 | |
*** markmcclain has quit IRC | 03:26 | |
*** ZZelle has quit IRC | 03:26 | |
*** kmartin_ has quit IRC | 03:26 | |
*** spring__ has quit IRC | 03:26 | |
*** tanee has quit IRC | 03:26 | |
*** gibi has quit IRC | 03:26 | |
*** mnaser has quit IRC | 03:26 | |
*** oomichi has quit IRC | 03:26 | |
*** clayton has quit IRC | 03:26 | |
*** jwcroppe has quit IRC | 03:26 | |
*** ildikov has quit IRC | 03:26 | |
*** lxsli has quit IRC | 03:26 | |
*** lennyb has quit IRC | 03:26 | |
*** ajo has quit IRC | 03:26 | |
*** HenryG has quit IRC | 03:26 | |
*** hamzy has quit IRC | 03:26 | |
*** toure has joined #openstack-nova | 03:26 | |
*** markmc has quit IRC | 03:26 | |
*** hamzy has joined #openstack-nova | 03:26 | |
*** oomichi_ is now known as oomichi | 03:26 | |
*** egarbade- has joined #openstack-nova | 03:26 | |
*** markmcclain1 is now known as markmcclain | 03:26 | |
*** ccard has joined #openstack-nova | 03:26 | |
*** HenryG has joined #openstack-nova | 03:26 | |
*** sbelous1 is now known as sbelous | 03:26 | |
*** toure has quit IRC | 03:26 | |
*** toure has joined #openstack-nova | 03:26 | |
*** egarbade- has quit IRC | 03:26 | |
*** egarbade- has joined #openstack-nova | 03:26 | |
*** ajo has joined #openstack-nova | 03:26 | |
*** dansmith_ is now known as dansmith | 03:27 | |
*** ildikov has joined #openstack-nova | 03:27 | |
*** edleafe- has joined #openstack-nova | 03:27 | |
*** lennyb has joined #openstack-nova | 03:27 | |
*** ZZelle has joined #openstack-nova | 03:27 | |
*** dansmith is now known as Guest60497 | 03:27 | |
*** clayton has joined #openstack-nova | 03:28 | |
*** ababich has joined #openstack-nova | 03:28 | |
*** gibi has joined #openstack-nova | 03:28 | |
*** markmc has joined #openstack-nova | 03:28 | |
*** links has joined #openstack-nova | 03:29 | |
*** ansiwen has quit IRC | 03:31 | |
*** mdbooth has quit IRC | 03:31 | |
*** mdbooth_ has joined #openstack-nova | 03:31 | |
*** spring_ is now known as grassy | 03:31 | |
*** ansiwen has joined #openstack-nova | 03:31 | |
*** mnaser has joined #openstack-nova | 03:32 | |
*** unicell has joined #openstack-nova | 03:32 | |
*** unicell has quit IRC | 03:32 | |
openstackgerrit | huan proposed openstack/nova: XenAPI:Resolve Nova/Neutron race condition https://review.openstack.org/241127 | 03:34 |
*** lennyb__ has joined #openstack-nova | 03:35 | |
*** tojuvone_ has joined #openstack-nova | 03:36 | |
*** y_sawai has joined #openstack-nova | 03:37 | |
*** haukebruno has quit IRC | 03:38 | |
*** lennyb has quit IRC | 03:40 | |
*** tojuvone has quit IRC | 03:40 | |
*** MVenesio has joined #openstack-nova | 03:44 | |
openstackgerrit | zwei proposed openstack/nova: Nova generate nova.conf.sample configuration file is incomplete https://review.openstack.org/276649 | 03:45 |
*** ijw has quit IRC | 03:45 | |
*** baoli has joined #openstack-nova | 03:45 | |
*** qeelee has joined #openstack-nova | 03:45 | |
*** baoli has quit IRC | 03:47 | |
*** dims has quit IRC | 03:48 | |
*** MVenesio has quit IRC | 03:49 | |
*** hemna_ has joined #openstack-nova | 03:50 | |
*** lxsli has joined #openstack-nova | 03:52 | |
mriedem | jroll: when was the ironic 1.8 microversion added? | 03:57 |
*** hemna has joined #openstack-nova | 03:58 | |
openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Add changes-since support when list servers https://review.openstack.org/286423 | 03:59 |
*** hemna_ has quit IRC | 04:00 | |
*** doug-fish has joined #openstack-nova | 04:00 | |
*** haomaiwang has quit IRC | 04:01 | |
*** y_sawai has quit IRC | 04:01 | |
*** y_sawai has joined #openstack-nova | 04:01 | |
*** qeelee has quit IRC | 04:02 | |
*** doug-fish has quit IRC | 04:05 | |
*** y_sawai has quit IRC | 04:05 | |
*** browne has joined #openstack-nova | 04:06 | |
*** haukebruno has joined #openstack-nova | 04:07 | |
*** y_sawai has joined #openstack-nova | 04:08 | |
*** y_sawai has quit IRC | 04:08 | |
*** vishwanathj_ is now known as vishwanathj_zzz | 04:15 | |
*** mriedem has quit IRC | 04:16 | |
tonyb | mriedem: 3e550c482ddef17ef1a65d4c2bc07dffa59c9a6c Fri May 15 11:17:04 2015 +0100 | 04:18 |
*** thorst has joined #openstack-nova | 04:21 | |
*** harlowja_at_home has quit IRC | 04:27 | |
*** Jeffrey4l has joined #openstack-nova | 04:27 | |
*** achanda has quit IRC | 04:28 | |
*** Daisy has joined #openstack-nova | 04:28 | |
*** thorst has quit IRC | 04:29 | |
*** haomaiwang has joined #openstack-nova | 04:32 | |
*** unicell has joined #openstack-nova | 04:32 | |
*** Daisy has quit IRC | 04:34 | |
*** Daisy has joined #openstack-nova | 04:34 | |
*** mylu has quit IRC | 04:36 | |
*** zengine has quit IRC | 04:36 | |
*** mylu has joined #openstack-nova | 04:37 | |
*** y_sawai has joined #openstack-nova | 04:39 | |
*** unicell has quit IRC | 04:41 | |
*** yonglihe has quit IRC | 04:42 | |
*** unicell has joined #openstack-nova | 04:42 | |
*** mylu has quit IRC | 04:48 | |
*** Marga_ has quit IRC | 04:50 | |
*** qeelee has joined #openstack-nova | 04:51 | |
*** y_sawai has quit IRC | 04:52 | |
*** gcb has quit IRC | 04:54 | |
*** mylu has joined #openstack-nova | 04:56 | |
*** y_sawai has joined #openstack-nova | 04:59 | |
*** Jeffrey4l has quit IRC | 04:59 | |
openstackgerrit | Sivaramakrishna Garimella proposed openstack/nova: Fix VIF to ironic port mapping https://review.openstack.org/287589 | 05:00 |
*** haomaiwang has quit IRC | 05:01 | |
*** haomaiwang has joined #openstack-nova | 05:01 | |
*** suresh12 has joined #openstack-nova | 05:04 | |
*** y_sawai has quit IRC | 05:05 | |
*** takedakn has joined #openstack-nova | 05:06 | |
*** gcb has joined #openstack-nova | 05:09 | |
*** ivase_ has quit IRC | 05:09 | |
*** qeelee has quit IRC | 05:10 | |
*** ivase has joined #openstack-nova | 05:10 | |
*** achanda has joined #openstack-nova | 05:11 | |
*** unicell has quit IRC | 05:13 | |
*** eliqiao_ has joined #openstack-nova | 05:13 | |
*** zenoway has joined #openstack-nova | 05:14 | |
*** Marga_ has joined #openstack-nova | 05:16 | |
*** Marga_ has quit IRC | 05:16 | |
*** Marga_ has joined #openstack-nova | 05:16 | |
*** eliqiao has quit IRC | 05:16 | |
*** alex_xu has joined #openstack-nova | 05:18 | |
*** zenoway has quit IRC | 05:18 | |
*** salv-orl_ has joined #openstack-nova | 05:19 | |
openstackgerrit | Feng Xi Yan proposed openstack/nova: Add missed configuration in generated nova.conf.sample. https://review.openstack.org/276649 | 05:20 |
*** alex_xu_ has quit IRC | 05:21 | |
*** salv-orlando has quit IRC | 05:22 | |
*** takedakn has quit IRC | 05:27 | |
*** thorst has joined #openstack-nova | 05:27 | |
*** claudiub|2 has joined #openstack-nova | 05:28 | |
*** haukebruno has quit IRC | 05:28 | |
*** rwmjones has quit IRC | 05:31 | |
*** takedakn has joined #openstack-nova | 05:31 | |
*** rwmjones has joined #openstack-nova | 05:32 | |
*** thorst has quit IRC | 05:33 | |
*** claudiub|2 has quit IRC | 05:34 | |
*** qeelee has joined #openstack-nova | 05:34 | |
*** takedakn has quit IRC | 05:36 | |
*** takedakn has joined #openstack-nova | 05:37 | |
*** kiran-r has joined #openstack-nova | 05:37 | |
*** unicell has joined #openstack-nova | 05:40 | |
*** bharathk has joined #openstack-nova | 05:41 | |
bharathk | hi | 05:41 |
*** haukebruno has joined #openstack-nova | 05:41 | |
*** irenab has quit IRC | 05:42 | |
bharathk | im getting an error while trying to boot, the error is coming during disk filter saying does not have usable disk | 05:42 |
bharathk | the error is: ram:16040 disk:79872 io_ops:0 instances:0 does not have 81920 MB usable disk, it only has 79872.0 MB usable disk ,Filter DiskFilter returned 0 hosts | 05:43 |
bharathk | does anyone know how to free the disk . please let me know. Thank you | 05:44 |
*** Daisy has quit IRC | 05:44 | |
*** ankit_ag has joined #openstack-nova | 05:44 | |
*** Daisy has joined #openstack-nova | 05:44 | |
*** MVenesio has joined #openstack-nova | 05:45 | |
*** Daisy has quit IRC | 05:46 | |
*** Daisy has joined #openstack-nova | 05:47 | |
*** mingyu has quit IRC | 05:47 | |
*** haukebruno has quit IRC | 05:47 | |
*** sivaramakrishna has joined #openstack-nova | 05:48 | |
*** MVenesio has quit IRC | 05:50 | |
*** alex_xu_ has joined #openstack-nova | 05:50 | |
*** gcb has quit IRC | 05:50 | |
*** Daisy has quit IRC | 05:51 | |
*** rwmjones has quit IRC | 05:51 | |
*** hemna has quit IRC | 05:52 | |
*** Daisy has joined #openstack-nova | 05:52 | |
*** alex_xu has quit IRC | 05:53 | |
*** rwmjones has joined #openstack-nova | 05:53 | |
*** Daisy has quit IRC | 05:55 | |
*** edtubill has joined #openstack-nova | 05:55 | |
*** Daisy has joined #openstack-nova | 05:55 | |
*** Daisy has quit IRC | 05:56 | |
*** Daisy has joined #openstack-nova | 05:56 | |
*** suresh12 has quit IRC | 05:57 | |
*** baoli has joined #openstack-nova | 05:59 | |
*** haukebruno has joined #openstack-nova | 06:00 | |
haukebruno | got disconnected, bharathk did my messages reach you? | 06:00 |
*** haomaiwang has quit IRC | 06:01 | |
*** dave-mccowan has quit IRC | 06:01 | |
*** haomaiwang has joined #openstack-nova | 06:01 | |
*** edtubill has quit IRC | 06:03 | |
*** baoli has quit IRC | 06:04 | |
*** armax has quit IRC | 06:07 | |
*** edtubill has joined #openstack-nova | 06:07 | |
*** gcb has joined #openstack-nova | 06:10 | |
*** gongysh has joined #openstack-nova | 06:10 | |
*** edtubill has quit IRC | 06:11 | |
*** yonglihe has joined #openstack-nova | 06:14 | |
*** zenoway has joined #openstack-nova | 06:14 | |
*** takedakn has quit IRC | 06:14 | |
*** zenoway has quit IRC | 06:17 | |
*** zenoway has joined #openstack-nova | 06:18 | |
*** takedakn has joined #openstack-nova | 06:19 | |
*** sheel has joined #openstack-nova | 06:20 | |
*** edtubill has joined #openstack-nova | 06:20 | |
*** yarkot_ has joined #openstack-nova | 06:21 | |
*** unicell has quit IRC | 06:22 | |
*** zenoway has quit IRC | 06:22 | |
*** esracelik has joined #openstack-nova | 06:23 | |
*** mdnadeem has quit IRC | 06:27 | |
bharathk | no ,i did not get any message haukebruno | 06:27 |
haukebruno | <haukebruno> bharathk, make sure you have enough remaining space under /var/lib/nova/instances on your compute noe | 06:27 |
*** mdnadeem has joined #openstack-nova | 06:27 | |
haukebruno | <haukebruno> *node | 06:27 |
haukebruno | <haukebruno> bharathk, alternatively size down your flavors | 06:27 |
bharathk | there is no directory name nova in /var/lib | 06:29 |
bharathk | i have ~/data/nova/instances in my system | 06:30 |
haukebruno | bharathk, what linux distro do you use? | 06:30 |
haukebruno | ah ok, nevermind. you need to make sure that your flavors fit your filesystem remaining space | 06:31 |
*** thorst has joined #openstack-nova | 06:31 | |
bharathk | okay , ill do that | 06:31 |
*** unicell has joined #openstack-nova | 06:31 | |
bharathk | thank you haukebruno | 06:31 |
*** gongysh has quit IRC | 06:32 | |
*** irenab has joined #openstack-nova | 06:33 | |
*** edtubill has quit IRC | 06:35 | |
*** edtubill has joined #openstack-nova | 06:36 | |
*** cfarquhar has quit IRC | 06:37 | |
*** Kevin_Zheng has joined #openstack-nova | 06:37 | |
eliqiao_ | hi pkoniszewski | 06:38 |
*** mylu has quit IRC | 06:38 | |
*** eliqiao_ is now known as eliqiao | 06:39 | |
*** thorst has quit IRC | 06:39 | |
*** mylu has joined #openstack-nova | 06:39 | |
openstackgerrit | Feng Xi Yan proposed openstack/nova: <VMWare> Attach more than one SCSI LUN to VM https://review.openstack.org/287055 | 06:39 |
*** cfarquhar has joined #openstack-nova | 06:40 | |
*** cfarquhar has quit IRC | 06:40 | |
*** cfarquhar has joined #openstack-nova | 06:40 | |
*** mylu_ has joined #openstack-nova | 06:41 | |
*** vilobhmm11 has quit IRC | 06:43 | |
*** mylu has quit IRC | 06:44 | |
*** tojuvone_ is now known as tojuvone | 06:44 | |
*** sridharg has joined #openstack-nova | 06:45 | |
*** unicell has quit IRC | 06:52 | |
*** yarkot_ has quit IRC | 06:52 | |
*** hoangcx has quit IRC | 06:53 | |
*** edtubill has quit IRC | 06:54 | |
*** mylu_ has quit IRC | 06:54 | |
*** shaohe_f- has quit IRC | 06:58 | |
*** lpetrut has joined #openstack-nova | 06:58 | |
*** shaohe_feng_ has joined #openstack-nova | 06:59 | |
*** hoangcx has joined #openstack-nova | 07:00 | |
*** takedakn has quit IRC | 07:01 | |
*** haomaiwang has quit IRC | 07:01 | |
*** takedakn has joined #openstack-nova | 07:01 | |
*** haomaiwang has joined #openstack-nova | 07:01 | |
openstackgerrit | Tomi Juvonen proposed openstack/nova-specs: Get servers filtered by host status https://review.openstack.org/276671 | 07:06 |
*** sudipto has joined #openstack-nova | 07:08 | |
*** Daisy has quit IRC | 07:13 | |
*** Daisy has joined #openstack-nova | 07:13 | |
*** jaosorior has joined #openstack-nova | 07:15 | |
*** mdnadeem has quit IRC | 07:16 | |
*** mdnadeem has joined #openstack-nova | 07:16 | |
*** Daisy has quit IRC | 07:18 | |
*** paul-carlton2 has joined #openstack-nova | 07:18 | |
*** scheuran has joined #openstack-nova | 07:20 | |
*** haomaiw__ has joined #openstack-nova | 07:20 | |
*** haomaiwang has quit IRC | 07:20 | |
openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Add changes-since support when list servers https://review.openstack.org/286423 | 07:20 |
*** mjura has joined #openstack-nova | 07:22 | |
pkoniszewski | eliqiao: hi | 07:23 |
*** rdopiera has joined #openstack-nova | 07:23 | |
*** claudiub|2 has joined #openstack-nova | 07:26 | |
*** claudiub|2 has quit IRC | 07:28 | |
eliqiao | hi pkoniszewski, I replied you on https://review.openstack.org/287363 | 07:28 |
eliqiao | pkoniszewski: thanks for proposing that patch. | 07:29 |
*** takedakn has quit IRC | 07:29 | |
eliqiao | tdurakov: hi, are you there? | 07:29 |
*** takedakn has joined #openstack-nova | 07:30 | |
*** achanda has quit IRC | 07:31 | |
*** boris-42 has quit IRC | 07:34 | |
*** takedakn has quit IRC | 07:34 | |
*** boris-42 has joined #openstack-nova | 07:35 | |
pkoniszewski | eliqiao: please see my answer | 07:36 |
*** thorst has joined #openstack-nova | 07:37 | |
alex_xu_ | pkoniszewski: 'in' work for object | 07:38 |
openstackgerrit | Vasyl Saienko proposed openstack/nova: WIP Allow to update resource per single node. https://review.openstack.org/287485 | 07:38 |
pkoniszewski | alex_xu_: well, let me double check it | 07:38 |
*** lpetrut has quit IRC | 07:39 | |
eliqiao | pkoniszewski: yeah, I remember that dsmth have commented me on my patch. I will find you the link | 07:39 |
eliqiao | pkoniszewski: that's the link https://review.openstack.org/#/c/254101/9/nova/objects/migrate_data.py | 07:40 |
eliqiao | pkoniszewski: oh, bad link I will find you new. | 07:41 |
pkoniszewski | alex_xu_: yeah, i see, i missed something | 07:41 |
alex_xu_ | pkoniszewski: no worries, for another concern, actually ComputeManager shouldn't know driver specific also | 07:42 |
pkoniszewski | alex_xu_: yep, but conductor is much worse place to put such logic | 07:42 |
alex_xu_ | pkoniszewski: actually I'm same with eliqiao, we should stop that mess early before those mess pass to the compute node | 07:42 |
eliqiao | pkoniszewski: that's the example https://review.openstack.org/#/c/275585/5/nova/virt/libvirt/driver.py@5495 | 07:42 |
pkoniszewski | alex_xu_: okay, i will do that, as this does not work in any scenario | 07:43 |
*** thorst has quit IRC | 07:43 | |
eliqiao | pkoniszewski: we need to cleanup block_migration from live_migration in future. | 07:43 |
alex_xu_ | pkoniszewski: thanks :) | 07:43 |
eliqiao | since we do have it in migrate_data | 07:43 |
alex_xu_ | eliqiao: +1 | 07:44 |
*** andymaier has joined #openstack-nova | 07:44 | |
sheel | auggy: hi there | 07:44 |
sheel | regarding bug #1535224 | 07:45 |
openstack | bug 1535224 in OpenStack Compute (nova) "meta data of nova instance with case sensitive has problem when deleting" [Undecided,In progress] https://launchpad.net/bugs/1535224 - Assigned to Sheel Rana (ranasheel2000) | 07:45 |
sheel | auggy: need some information about approach finalization for fixing this issue | 07:45 |
*** MVenesio has joined #openstack-nova | 07:45 | |
*** pcaruana has joined #openstack-nova | 07:45 | |
*** achanda has joined #openstack-nova | 07:49 | |
*** MVenesio has quit IRC | 07:51 | |
*** EinstCra_ has joined #openstack-nova | 07:52 | |
*** e0ne has joined #openstack-nova | 07:55 | |
*** EinstCrazy has quit IRC | 07:56 | |
*** e0ne has quit IRC | 07:56 | |
sheel | auggy: I could find update on 1538011 | 07:57 |
*** jed56 has joined #openstack-nova | 07:57 | |
*** haomaiw__ has quit IRC | 08:01 | |
*** jichen has joined #openstack-nova | 08:01 | |
*** eliqiao has quit IRC | 08:01 | |
*** haomaiwang has joined #openstack-nova | 08:01 | |
*** amotoki has quit IRC | 08:02 | |
*** markvoelker has quit IRC | 08:03 | |
*** takedakn has joined #openstack-nova | 08:04 | |
*** erno has joined #openstack-nova | 08:04 | |
*** EinstCra_ has quit IRC | 08:06 | |
*** EinstCrazy has joined #openstack-nova | 08:06 | |
*** zenoway has joined #openstack-nova | 08:07 | |
*** eliqiao has joined #openstack-nova | 08:07 | |
*** andymaier has quit IRC | 08:09 | |
*** neophy has joined #openstack-nova | 08:14 | |
*** paul-carlton2 has quit IRC | 08:14 | |
*** claudiub|2 has joined #openstack-nova | 08:16 | |
openstackgerrit | Pawel Koniszewski proposed openstack/nova: Make block_migration and migrate_data.block_migration consistent https://review.openstack.org/287363 | 08:17 |
*** vilobhmm11 has joined #openstack-nova | 08:17 | |
*** taneee is now known as tanee | 08:18 | |
*** andymaier has joined #openstack-nova | 08:25 | |
*** paul-carlton2 has joined #openstack-nova | 08:26 | |
*** ihrachys has joined #openstack-nova | 08:32 | |
*** achanda has quit IRC | 08:32 | |
*** matrohon has joined #openstack-nova | 08:34 | |
*** eliqiao_ has joined #openstack-nova | 08:35 | |
*** eliqiao has quit IRC | 08:35 | |
*** e0ne has joined #openstack-nova | 08:36 | |
*** jichen has quit IRC | 08:37 | |
*** browne has quit IRC | 08:37 | |
*** jlanoux has joined #openstack-nova | 08:38 | |
*** jichen has joined #openstack-nova | 08:38 | |
*** e0ne has quit IRC | 08:38 | |
*** thorst has joined #openstack-nova | 08:41 | |
bauzas | good morning Nova | 08:43 |
*** atuvenie has joined #openstack-nova | 08:44 | |
*** haomaiwang has quit IRC | 08:45 | |
*** haomaiwang has joined #openstack-nova | 08:48 | |
*** macsz has joined #openstack-nova | 08:49 | |
*** hoangcx has quit IRC | 08:49 | |
*** thorst has quit IRC | 08:49 | |
*** ndipanov has joined #openstack-nova | 08:49 | |
openstackgerrit | javeme proposed openstack/nova: VMware: Prefer ShutdownGuest to PowerOffVM_Task https://review.openstack.org/181634 | 08:51 |
*** mgoddard has joined #openstack-nova | 08:52 | |
*** shuquan has joined #openstack-nova | 08:54 | |
*** haomaiwang has quit IRC | 09:01 | |
*** BobBall_AWOL is now known as BobBall | 09:01 | |
*** 18VAACMSD has joined #openstack-nova | 09:01 | |
*** andymaier has quit IRC | 09:03 | |
*** shuquan has quit IRC | 09:03 | |
*** markvoelker has joined #openstack-nova | 09:03 | |
*** shuquan has joined #openstack-nova | 09:05 | |
*** ishant has joined #openstack-nova | 09:07 | |
*** dedery has joined #openstack-nova | 09:07 | |
pkoniszewski | alex_xu_: what in case of vmware? i wasn't sure about it | 09:07 |
*** eliqiao_ is now known as eliqiao | 09:07 | |
*** ducttape_ has joined #openstack-nova | 09:08 | |
tdurakov | eliqiao, hi | 09:09 |
*** liverpooler has joined #openstack-nova | 09:11 | |
*** ducttape_ has quit IRC | 09:12 | |
*** Fdaisuke has joined #openstack-nova | 09:12 | |
pkoniszewski | alex_xu_: nvm, found out that there won't be any problem | 09:12 |
*** andymaier has joined #openstack-nova | 09:15 | |
*** jistr has joined #openstack-nova | 09:17 | |
*** lucas-dinner is now known as lucasagomes | 09:17 | |
*** EinstCrazy has quit IRC | 09:18 | |
*** EinstCrazy has joined #openstack-nova | 09:19 | |
*** daemontool_ has joined #openstack-nova | 09:19 | |
*** takedakn has quit IRC | 09:20 | |
*** josecastroleon has joined #openstack-nova | 09:23 | |
*** e0ne has joined #openstack-nova | 09:24 | |
*** Daisy has joined #openstack-nova | 09:26 | |
*** hoangcx has joined #openstack-nova | 09:26 | |
*** yassine__ has joined #openstack-nova | 09:27 | |
*** Daisy has quit IRC | 09:27 | |
*** Daisy has joined #openstack-nova | 09:28 | |
*** Daisy has quit IRC | 09:29 | |
*** Daisy has joined #openstack-nova | 09:29 | |
*** Daisy has quit IRC | 09:30 | |
*** Daisy has joined #openstack-nova | 09:30 | |
*** qeelee has quit IRC | 09:31 | |
*** Daisy has quit IRC | 09:32 | |
*** Daisy has joined #openstack-nova | 09:33 | |
*** Daisy has quit IRC | 09:33 | |
*** Daisy has joined #openstack-nova | 09:33 | |
*** Daisy has quit IRC | 09:35 | |
*** Daisy has joined #openstack-nova | 09:36 | |
bauzas | johnthetubaguy: so, I'm just digging into grenade code to see why it takes so long in https://jenkins02.openstack.org/job/gate-grenade-dsvm-multinode/3180/console | 09:36 |
*** Daisy has quit IRC | 09:36 | |
johnthetubaguy | ah, is that normal? | 09:36 |
*** Daisy has joined #openstack-nova | 09:37 | |
bauzas | well, I'm unfortunately not a grenade contributor, so I just see it's probably here https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L259 | 09:37 |
BobBall | When we _remove_ a config option (i.e. not just deprecate) will config files that have that old option fail to parse? or will the option be ignored? | 09:38 |
*** houming has quit IRC | 09:39 | |
johnthetubaguy | BobBall: I thought you just get a warning log, or something, but I don't 100% remember, although a vaguely remember checking that recently when we removed one | 09:39 |
johnthetubaguy | BobBall: are you seeing different? | 09:40 |
BobBall | Not seeing - commenting on https://review.openstack.org/#/c/213112/21/nova/virt/xenapi/driver.py - I had a vague feeling that the config would fail to load if it included unknown options | 09:40 |
BobBall | and therefore deprecating options that were being ignored still made sense, to give the admin a chance to remove them from config files before nova failed to start | 09:40 |
BobBall | Actually I can test really simply can't I... | 09:41 |
BobBall | Ignore me. | 09:41 |
*** qeelee has joined #openstack-nova | 09:41 | |
markus_z | claudiub|2: Are you around? | 09:41 |
johnthetubaguy | BobBall: let me know what you find, I am curious | 09:41 |
claudiub|2 | markus_z: hello. :) | 09:41 |
markus_z | claudiub|2: I'm currently looking into missing config options for the sample config file and I found one in os-win | 09:41 |
markus_z | claudiub|2: https://github.com/openstack/os-win/blob/d3902e33024eadd196d09c3b133fac90a173ed23/os_win/utilsfactory.py#L27 | 09:41 |
*** mjura_ has joined #openstack-nova | 09:42 | |
markus_z | claudiub|2: That one is configurable within the "etc/nova/nova.conf" at the end, right? | 09:42 |
*** mjura_ has quit IRC | 09:42 | |
BobBall | johnthetubaguy: Seems to work with a random config existing; so removal is "safe" | 09:42 |
*** mjura_ has joined #openstack-nova | 09:42 | |
*** danpb has joined #openstack-nova | 09:42 | |
*** paul-carlton2 has quit IRC | 09:43 | |
openstackgerrit | Jianghua Wang proposed openstack/nova-specs: XenAPI: support VGPU via passthrough PCI https://review.openstack.org/280099 | 09:43 |
*** gongysh has joined #openstack-nova | 09:44 | |
claudiub|2 | markus_z: indeed it is | 09:44 |
claudiub|2 | markus_z: hm, it should be deprecated. I'll send a patch for that soon. | 09:44 |
markus_z | BobBall: Let me know the result please, I'm not sure anymore if the service start fails then. | 09:45 |
*** hoangcx has quit IRC | 09:45 | |
BobBall | The service appears to start successfully in a quick test | 09:46 |
johnthetubaguy | BobBall: Ok, lets go for the removal, thats what I saw in the oslo_config code I think | 09:46 |
markus_z | claudiub|2: ok, are there others in os-win I might have overlooked? If so, they need to be imported from nova code to make it appear in http://docs.openstack.org/developer/nova/sample_config.html | 09:46 |
*** MVenesio has joined #openstack-nova | 09:46 | |
*** thorst has joined #openstack-nova | 09:46 | |
*** mdnadeem_ has joined #openstack-nova | 09:47 | |
markus_z | claudiub|2: We do that for other libs too, for example the db options: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L31 | 09:47 |
openstackgerrit | Wenzhi Yu proposed openstack/nova: Fix missing of unit in HostState.__repr__() https://review.openstack.org/287032 | 09:47 |
markus_z | claudiub|2: That's a thing you should be aware of when introducing options in os-win. | 09:48 |
claudiub|2 | markus_z: no, I don't think there is. We had force_v1_utils or something like that, that was deprecated in liberty and we removed it in mitaka. | 09:49 |
*** mdnadeem has quit IRC | 09:50 | |
*** belmoreira has joined #openstack-nova | 09:51 | |
claudiub|2 | markus_z: I suppose we can make a config.py file in os-win, and have an os_win_opts there, that can be imported in nova, right? | 09:51 |
*** MVenesio has quit IRC | 09:52 | |
markus_z | claudiub|2: yep, right, that would be the way | 09:52 |
*** grassy is now known as davechen_afk | 09:52 | |
bauzas | johnthetubaguy: so, tbc, we're here https://github.com/openstack-dev/grenade/blob/22ee7f82df8e3aa4ea97517d3c0083e41bf529e5/inc/plugin#L102 | 09:52 |
markus_z | claudiub|2: That would establish a contract, which means renaming that option in the lib would break nova | 09:52 |
bauzas | johnthetubaguy: well, rather https://github.com/openstack-dev/grenade/blob/22ee7f82df8e3aa4ea97517d3c0083e41bf529e5/inc/plugin#L118 | 09:53 |
bauzas | with all projects running the create phase | 09:53 |
bauzas | but I can't say which one is holding | 09:53 |
claudiub|2 | markus_z: you mean renaming os_win_opts to something else or renaming config options in the list? | 09:53 |
markus_z | claudiub|2: renaming "os_win_opts" | 09:53 |
claudiub|2 | markus_z: there's debtcollector though, that helps with stuff like this. | 09:54 |
*** openstackgerrit_ has joined #openstack-nova | 09:54 | |
*** thorst has quit IRC | 09:54 | |
johnthetubaguy | bauzas: hmm, when will that job timeout? | 09:54 |
markus_z | claudiub|2: hmm, sounds interesting | 09:55 |
claudiub|2 | markus_z: plus, I don't think there's going to be a reason to rename it. :) | 09:55 |
bauzas | johnthetubaguy: that's an excellent question, and I still don't know | 09:55 |
tdurakov | pkoniszewski, hi, are you around? | 09:55 |
markus_z | claudiub|2: yeah, I think so too. I just wanted to mention it, as it could be easy to overlook this when working within the os-win lib. | 09:56 |
pkoniszewski | tdurakov: hello | 09:56 |
bauzas | johnthetubaguy: there is a for loop, so I need to introspect each plugin create() condition to see whether we assume we can timeout | 09:56 |
*** vilobhmm11 has quit IRC | 09:56 | |
markus_z | claudiub|2: ok, I think we're clear here. Thanks for the debtcollector info, I'll look into that. | 09:56 |
*** haomaiwang has joined #openstack-nova | 09:56 | |
*** 18VAACMSD has quit IRC | 09:57 | |
claudiub|2 | markus_z: sure, no problem. :) I'll let you know when the commit has been merged and tagged. | 09:57 |
tdurakov | pkoniszewski, missed that during review, got question about this https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5264 | 09:57 |
*** qeelee has quit IRC | 09:57 | |
tdurakov | pkoniszewski, why not to pass migration object, as compute will go back to api to get migration by id | 09:57 |
pkoniszewski | tdurakov: we would need to refresh object anyway to check status, so i'm not sure it really matters whether we send ID or migration object there | 09:58 |
tdurakov | pkoniszewski, extra hops | 09:58 |
pkoniszewski | tdurakov: doesnt refresh generate the same? | 09:59 |
tdurakov | pkoniszewski, which refresh? | 09:59 |
pkoniszewski | object.refresh() | 09:59 |
*** e0ne has quit IRC | 10:00 | |
tdurakov | pkoniszewski, i thought if we pass object no additional queries will be done | 10:00 |
*** haomaiwang has quit IRC | 10:01 | |
pkoniszewski | don't we need to refresh the object to get actual state after RPC call? | 10:01 |
pkoniszewski | so in case RPC took a while we have actual state of migration in manager | 10:01 |
*** haomaiwang has joined #openstack-nova | 10:01 | |
*** openstackgerrit_ has quit IRC | 10:02 | |
*** Jeffrey4l has joined #openstack-nova | 10:03 | |
tdurakov | pkoniszewski, oh, that's what you mean, is it make sense at all then, if migration could finish after check on line 5273? | 10:03 |
*** mjura_ has quit IRC | 10:04 | |
*** boris-42 has quit IRC | 10:04 | |
bauzas | johnthetubaguy: so, honestly, reading the plugin repo is hard because I don't have much hooks to get in | 10:04 |
*** mjura has quit IRC | 10:04 | |
bauzas | johnthetubaguy: but that said, the job finally failed | 10:04 |
*** mjura has joined #openstack-nova | 10:04 | |
pkoniszewski | tdurakov: it can, but by refreshing status and checking it we are reducing chances to the minimum | 10:04 |
bauzas | johnthetubaguy: so the gate is unlocked now, although with a huge delayu | 10:04 |
johnthetubaguy | bauzas: OK, well, I guess the gate reset, and we roll the dice again | 10:04 |
bauzas | yup | 10:05 |
bauzas | and that's a real problem IMHO | 10:05 |
pkoniszewski | tdurakov: still need to implement one more check at driver level, we really never want to pause vm in post migration | 10:05 |
pkoniszewski | tdurakov: i mean, in rollback, post is not a case there | 10:05 |
bauzas | johnthetubaguy: oh, I got the failure \o/ | 10:05 |
bauzas | and that was a timeout which unlocked that indeed | 10:05 |
tdurakov | pkoniszewski, true, here is the point, if we do this check on api side it's imo reduce this chance at the same level | 10:05 |
johnthetubaguy | pkoniszewski: tdurakov: I don't like passing the id myself, much prefer passing the object, but I ended up not blocking on that because of the refresh that was needed anyways | 10:06 |
*** Daisy has quit IRC | 10:06 | |
pkoniszewski | tdurakov: no, there might be a significant delay between API and manager | 10:06 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Sample nova.conf file is missing config options https://review.openstack.org/287685 | 10:06 |
*** Daisy has joined #openstack-nova | 10:07 | |
tdurakov | johnthetubaguy, pkoniszewski delay almost the same, as we do rpc to get migration from db | 10:07 |
pkoniszewski | tdurakov: well, in ha environments it might vary | 10:09 |
tdurakov | if we do this check as a last step in api before calling compute we end up with same delay on rpc, anyway, it's better to check in driver real state rather db, agree. | 10:09 |
pkoniszewski | tdurakov: anyway we still need to make sure that there is an active job in libvirt before actually pausing an instance | 10:09 |
bauzas | johnthetubaguy: see the problem http://logs.openstack.org/81/283781/6/gate/gate-grenade-dsvm-multinode/3c1b54b/logs/grenade.sh.txt.gz#_2016-03-03_09_51_29_865 | 10:09 |
tdurakov | pkoniszewski, tbh not sure that ha env affects anything here | 10:10 |
*** e0ne has joined #openstack-nova | 10:10 | |
johnthetubaguy | tdurakov: the message could be delayed, but its all a bit borderline, agreed | 10:10 |
johnthetubaguy | tdurakov: I would +2 a patch that passed the object and dropped the referesh | 10:10 |
*** Daisy has quit IRC | 10:11 | |
johnthetubaguy | I think the compute manager task check is probably good enough | 10:11 |
pkoniszewski | we don't check task in compute manager | 10:12 |
pkoniszewski | right now, i mean | 10:12 |
bauzas | johnthetubaguy: which is related to https://github.com/openstack-dev/grenade/blob/master/projects/70_cinder/resources.sh#L141 | 10:12 |
*** EinstCrazy has quit IRC | 10:13 | |
*** EinstCrazy has joined #openstack-nova | 10:13 | |
*** sfinucan has joined #openstack-nova | 10:13 | |
johnthetubaguy | pkoniszewski: oh, yeah you are right, we don't for this flow, we usually do for most instance operations, update the task state when it reaches the compute node, and use expected state to check whats going on. | 10:14 |
bauzas | okay, point raised to #openstack-infra, we'll see | 10:14 |
bauzas | johnthetubaguy: fancy tagging today? | 10:14 |
johnthetubaguy | pkoniszewski: tdurakov: this all boils down to getting someone to fix our async task handling | 10:14 |
johnthetubaguy | bauzas: keep getting distracted, still reading through | 10:14 |
johnthetubaguy | I mean, probably... just not yet | 10:15 |
bauzas | johnthetubaguy: don't invest too much on the grenade side, the gate is unlocked now, we'll see later with QA folks | 10:15 |
bauzas | johnthetubaguy: I'm just wanting to let the boat leave | 10:15 |
*** amotoki has joined #openstack-nova | 10:16 | |
bauzas | I mean the Mitaka-3 tag | 10:16 |
*** amotoki has quit IRC | 10:16 | |
johnthetubaguy | yep, thats what I was meaning | 10:16 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Sample nova.conf file is missing config options https://review.openstack.org/287685 | 10:18 |
*** EinstCrazy has quit IRC | 10:18 | |
pkoniszewski | johnthetubaguy: i have a question regarding you comment there https://review.openstack.org/#/c/287363/2/nova/conductor/tasks/live_migrate.py | 10:19 |
bauzas | johnthetubaguy: okay, my only concern is that https://review.openstack.org/278124 is +W'd but not yet merged | 10:19 |
bauzas | johnthetubaguy: for the other cells v2 and resource-providers changes, they won't have a +W in time, but that's life | 10:20 |
pkoniszewski | johnthetubaguy: at this point migrate_data.block_migration is already calculated | 10:20 |
pkoniszewski | johnthetubaguy: i can me code use, e.g., only migrate_data.block_migration so we will be able to deprecate block_migraiton argument and finally get rid of this if this is what you think about | 10:21 |
pkoniszewski | johnthetubaguy: and the real problem is compute manager, not drivers | 10:21 |
eliqiao | johnthetubaguy: pkoniszewski block_migration won't be used when we cast live_migration to compute node | 10:21 |
bauzas | johnthetubaguy: so my question is, should we wait for 278124 for providing the mitaka-3 tag or are we confident enough to do the tag today and include the change in the RC | 10:22 |
pkoniszewski | eliqiao: in case of Xen it will be, https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5309 | 10:22 |
johnthetubaguy | pkoniszewski: so I think the argument should represent what comes from the API, so None is a valid value there, the code should use migrate_data for its choices, I think. | 10:22 |
*** sulo_ is now known as sulo | 10:22 | |
johnthetubaguy | bauzas: I am really looking at what gets thrown under the bus as soon as we cut the tag | 10:23 |
johnthetubaguy | bauzas: stuff with a +W on it now can go into the next tarball, and thats OK | 10:23 |
bauzas | johnthetubaguy: I can tell you for the cellsv2 and resource-providers series | 10:23 |
johnthetubaguy | bauzas: the cells stuff looks like a few nits away from being merged, but it doesn't deliver enough thats worth waiting for, I fear | 10:26 |
bauzas | johnthetubaguy: so the nova-manage command requires a respin, and the BuildRequest object patch using it in the compute API isn't yet approved with some concerns | 10:27 |
johnthetubaguy | bauzas: right, they are mostly nits though, need to go check the nova-manage | 10:27 |
bauzas | agreed | 10:28 |
johnthetubaguy | bauzas: have you had chance to review https://blueprints.launchpad.net/nova/+spec/host-state-level-locking yet? | 10:29 |
bauzas | johnthetubaguy: well, I did a few, but not with the really needed thing for +Wing it | 10:30 |
johnthetubaguy | bauzas: the key bits already have one +2, I would love to see about a +W before we cut the branch | 10:30 |
johnthetubaguy | bauzas: I mean these two: https://review.openstack.org/#/c/262939 | 10:30 |
*** deepakcs has joined #openstack-nova | 10:31 | |
tdurakov | johnthetubaguy, will submit patch for this | 10:32 |
*** paul-carlton2 has joined #openstack-nova | 10:36 | |
*** sgordon` is now known as sgordon | 10:37 | |
*** Daisy has joined #openstack-nova | 10:38 | |
*** sudipto has quit IRC | 10:38 | |
eliqiao | pkoniszewski johnthetubaguy for block_migration, I think we can do that in driver side, some of driver doesn't use block_migration at all, also, block_migration are already contained in migrate_data(no matter it is passed or we calculated). We need to always keeping grab block_migration from migration and deprecate it in future ? | 10:39 |
pkoniszewski | eliqiao: thats what im working on right now | 10:39 |
*** shuquan has quit IRC | 10:39 | |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Sample nova.conf file is missing config options https://review.openstack.org/287685 | 10:39 |
pkoniszewski | eliqiao: use migrate_data.block_migration everywhere instead of block_migration | 10:39 |
eliqiao | but the parameter passed to compute_api.live_migrate() should be clearly leave a note to avoid miss using | 10:39 |
*** sfinucan has quit IRC | 10:40 | |
eliqiao | pkoniszewski: thanks for doing that, and sorry about involve this critical bug :( | 10:41 |
pkoniszewski | johnthetubaguy: one more question about upgradeability from liberty to mitaka | 10:41 |
eliqiao | johnthetubaguy: tdurakov: can you take a look at the tempest bug for testing block_migration='auto' | 10:41 |
openstackgerrit | Abhishek Talwar proposed openstack/python-novaclient: flavor-id should be only integer or UUID https://review.openstack.org/287702 | 10:41 |
tdurakov | eliqiao, which one? | 10:42 |
pkoniszewski | johnthetubaguy: i ran into a situation where my control plane is already mitaka and all compute nodes are liberty | 10:42 |
eliqiao | tdurakov: https://review.openstack.org/#/c/287605/ | 10:42 |
*** Daisy has quit IRC | 10:42 | |
*** raildo-afk is now known as raildo | 10:42 | |
*** Daisy has joined #openstack-nova | 10:42 | |
*** ishant has quit IRC | 10:43 | |
tdurakov | eliqiao, sure, triggered experimental also | 10:43 |
pkoniszewski | johnthetubaguy: did we decide to completely turn off live migrations in the process of upgrading nova from liberty to mitaka? | 10:43 |
johnthetubaguy | pkoniszewski: no, that was never the plan | 10:43 |
*** mvk has joined #openstack-nova | 10:43 | |
*** josecastroleon has quit IRC | 10:43 | |
johnthetubaguy | pkoniszewski: we ensured that users requesting the new API microversion are unable to use that until the upgrade has completed | 10:43 |
johnthetubaguy | all other live-migrates should be unaffected | 10:44 |
johnthetubaguy | assuming we didn't break it for some other reason (its untested in grenade right now) | 10:44 |
johnthetubaguy | we must keep live-migrate working, as its something people do to actually peform their upgrade | 10:44 |
pkoniszewski | johnthetubaguy: well, hmm, so with the new API I can't pass block_migrate=False, it always defaults to 'auto', i can force it to be True, but... | 10:44 |
pkoniszewski | johnthetubaguy: i can't pass disk_over_commit, we completely delete it from the API | 10:45 |
tdurakov | eliqiao, would like to see live-migration job results, ping me please once experimental finished, l-m job is here for tempest to | 10:45 |
tdurakov | s/to/too | 10:45 |
*** astupnikov has joined #openstack-nova | 10:45 | |
pkoniszewski | johnthetubaguy: it makes me think this check https://review.openstack.org/#/c/254102/35/nova/compute/rpcapi.py is always true in this case, which means live migration does not work in the upgrade process | 10:45 |
johnthetubaguy | pkoniszewski: I think we are miss understanding each other here | 10:46 |
eliqiao | pkoniszewski: you can not use v2.25 until all compute are upgraed to new version. | 10:46 |
johnthetubaguy | pkoniszewski: the new API is not available during upgrade, the old API just work like it always did, people will have to change their code and usage patterns to activate the new API | 10:46 |
pkoniszewski | eliqiao: got it | 10:46 |
pkoniszewski | johnthetubaguy: okay | 10:47 |
pkoniszewski | thanks! | 10:47 |
*** Daisy has quit IRC | 10:47 | |
*** kylek3h has joined #openstack-nova | 10:48 | |
eliqiao | tdurakov: the tempest will be only passed after pkoniszewski's fix get merged? I mean is 'Depends-On' will using pkoniszewski's patch to do test? | 10:48 |
tdurakov | eliqiao, it should | 10:49 |
eliqiao | tdurakov: okay, cool. | 10:49 |
*** thorst has joined #openstack-nova | 10:51 | |
tdurakov | pkoniszewski, about force_complete live migration is it make sense to check on the api layer that instance.host == migration.source_node? what do you think? | 10:53 |
pkoniszewski | tdurakov: hmm, it might make sense in case migration has failed just after triggering force_complete | 10:54 |
openstackgerrit | Ihar Hrachyshka proposed openstack/os-vif: mtu: don't attempt to set link mtu if it's invalid https://review.openstack.org/287709 | 10:54 |
tdurakov | pkoniszewski, I mean we don't want to pause instance once it's on dest node, right? | 10:55 |
pkoniszewski | tdurakov: anyway it will fail in libvirt driver because the domain will not be there | 10:55 |
*** shaohe_feng_ has quit IRC | 10:55 | |
*** lpetrut has joined #openstack-nova | 10:56 | |
*** sivaramakrishna has quit IRC | 10:56 | |
*** cdent has joined #openstack-nova | 10:56 | |
pkoniszewski | tdurakov: if instance.host is already new then yes, it makes sense | 10:56 |
*** gongysh has quit IRC | 10:57 | |
*** thorst has quit IRC | 10:59 | |
*** doug-fish has joined #openstack-nova | 11:00 | |
*** haomaiwang has quit IRC | 11:01 | |
*** alex_xu_ has quit IRC | 11:01 | |
*** qeelee has joined #openstack-nova | 11:01 | |
*** haomaiwang has joined #openstack-nova | 11:01 | |
*** salv-orl_ has quit IRC | 11:01 | |
*** salv-orlando has joined #openstack-nova | 11:02 | |
*** alex_xu has joined #openstack-nova | 11:02 | |
*** shaohe_feng_ has joined #openstack-nova | 11:02 | |
*** jed56 has quit IRC | 11:03 | |
*** mdnadeem_ has quit IRC | 11:03 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/os-vif: mtu: don't attempt to set link mtu if it's invalid https://review.openstack.org/287709 | 11:04 |
*** doug-fish has quit IRC | 11:05 | |
*** ankit_ag has quit IRC | 11:06 | |
*** ociuhandu has joined #openstack-nova | 11:06 | |
*** ankit_ag has joined #openstack-nova | 11:07 | |
bauzas | johnthetubaguy: so, the boat is sailing for the BuildRequest patches, I just rechecked those | 11:07 |
*** deepakcs has quit IRC | 11:08 | |
*** links has quit IRC | 11:08 | |
*** sridharg has quit IRC | 11:08 | |
*** kashyap has quit IRC | 11:09 | |
*** deepakcs has joined #openstack-nova | 11:09 | |
*** sridharg has joined #openstack-nova | 11:10 | |
*** links has joined #openstack-nova | 11:10 | |
*** raildo is now known as raildo-afk | 11:12 | |
*** kashyap has joined #openstack-nova | 11:12 | |
*** dims has joined #openstack-nova | 11:14 | |
*** shaohe_robot has joined #openstack-nova | 11:14 | |
paul-carlton2 | johnthetubaguy, are you aware that id you disable a compute host you can't use force host to force an instance to that node anymore? | 11:14 |
paul-carlton2 | Also the host is no longer listed in the availability zone | 11:15 |
johnthetubaguy | paul-carlton2: that came up at the midycle, I think I said we should revert that fix, IMHO. | 11:15 |
johnthetubaguy | not sure about the AZ thing, not really thought about that one, sounds wrong though | 11:16 |
*** bharathk has quit IRC | 11:16 | |
paul-carlton2 | This was not the case way back in icehouse | 11:16 |
paul-carlton2 | ok, I'm looking this now | 11:16 |
johnthetubaguy | cool, that would be a good fix to get in | 11:16 |
*** neophy has quit IRC | 11:17 | |
paul-carlton2 | my initial analysis is that commit c42fd6de5903cdb7c5519bd773e0859767f68043 broke thus | 11:18 |
paul-carlton2 | this | 11:18 |
*** salv-orl_ has joined #openstack-nova | 11:19 | |
paul-carlton2 | It relates to host and node handling, but I'm not familiar with the concept of a host having nodes, seen it in the code but never paid much attention | 11:19 |
paul-carlton2 | Any guidance on where to look to understand this concept | 11:20 |
openstackgerrit | Daniel Berrange proposed openstack/os-vif: linux_bridge: actually apply the iptables rules https://review.openstack.org/287724 | 11:20 |
openstackgerrit | Daniel Berrange proposed openstack/os-vif: linux_bridge: convert over to use privsep module https://review.openstack.org/287725 | 11:20 |
*** salv-orlando has quit IRC | 11:22 | |
bauzas | paul-carlton2: well, the force_hosts concept is indeed making sure that you force on a non-disabled host, that's not a pure "force" | 11:25 |
*** shaohe_robot1 has joined #openstack-nova | 11:25 | |
johnthetubaguy | paul-carlton2: basically think ironic and vmware, were a compute node reports many nodes for the one host | 11:25 |
*** dims has quit IRC | 11:26 | |
*** shaohe_robot has quit IRC | 11:27 | |
*** dims has joined #openstack-nova | 11:27 | |
*** phuongnh has quit IRC | 11:28 | |
*** jed56 has joined #openstack-nova | 11:28 | |
bauzas | paul-carlton2: that said, the scheduler only filters disabled hosts by the ComputeFilter | 11:28 |
bauzas | paul-carlton2: if you're using the AZ hack that makes force_hosts, it should give you the right hosts even if disabled | 11:29 |
*** sahid has joined #openstack-nova | 11:30 | |
bauzas | paul-carlton2: so, see I'm wondering why you have problems with disabled hosts | 11:30 |
*** paul-carlton2 has quit IRC | 11:30 | |
bauzas | if you're using something when booting like nova boot --az myfake:myhost | 11:31 |
bauzas | arrrgh | 11:31 |
bauzas | ping timeout | 11:31 |
bauzas | too many paul-carlton... | 11:31 |
*** mvk has quit IRC | 11:32 | |
bauzas | paul-carlton: paul-carlton1 : so, like I said, if you force hosts using the AZ flag, the scheduler *should* return you that host even if it's disabled | 11:32 |
*** pece has joined #openstack-nova | 11:34 | |
openstackgerrit | Cao ShuFeng proposed openstack/python-novaclient: Over Usage's get() function https://review.openstack.org/287735 | 11:35 |
openstackgerrit | Cao ShuFeng proposed openstack/python-novaclient: Overwrite Usage class's get() function https://review.openstack.org/287735 | 11:36 |
*** mdnadeem has joined #openstack-nova | 11:37 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/os-vif: mtu: don't attempt to set link mtu if it's invalid https://review.openstack.org/287709 | 11:37 |
*** yamahata has quit IRC | 11:39 | |
*** ivase has quit IRC | 11:43 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/os-vif: mtu: don't attempt to set link mtu if it's invalid https://review.openstack.org/287709 | 11:44 |
*** roeyc has joined #openstack-nova | 11:45 | |
*** paul-carlton2 has joined #openstack-nova | 11:46 | |
johnthetubaguy | bauzas: question on https://review.openstack.org/#/c/262938/8/nova/scheduler/host_manager.py@241 what makes you say that? | 11:46 |
*** smatzek has joined #openstack-nova | 11:47 | |
*** MVenesio has joined #openstack-nova | 11:47 | |
bauzas | johnthetubaguy: it's a nit, but since the existing claim raises this one, I think we should explain that it's maybe not due to a ComputeResource | 11:47 |
johnthetubaguy | bauzas: the other code is catching only ComputeResource one though | 11:48 |
johnthetubaguy | bauzas: as in https://review.openstack.org/#/c/262938/8/nova/scheduler/filter_scheduler.py@159 | 11:48 |
*** qeelee has quit IRC | 11:49 | |
bauzas | johnthetubaguy: yeah but https://review.openstack.org/#/c/262938/8/nova/compute/claims.py@161 is raising the same exception | 11:49 |
*** sdague has joined #openstack-nova | 11:49 | |
bauzas | johnthetubaguy: I just wanted to explain that we're not checking the same things, even if using the same framework | 11:50 |
johnthetubaguy | bauzas: not sure I get you | 11:51 |
bauzas | johnthetubaguy: we're claiming against different sets of resources | 11:51 |
bauzas | but that's a nit, don't overthink on that one | 11:52 |
bauzas | and I already suck at naming, so probably my point is not really nice | 11:52 |
*** MVenesio_ has joined #openstack-nova | 11:52 | |
*** rfolco has joined #openstack-nova | 11:52 | |
*** MVenesio has quit IRC | 11:53 | |
johnthetubaguy | bauzas: oh, you mean you don't like the name of this: https://review.openstack.org/#/c/262938/8/nova/scheduler/claims.py@119 | 11:53 |
bauzas | johnthetubaguy: yup sorry | 11:53 |
johnthetubaguy | bauzas: but its a straight cut and paste, but I think I see your point | 11:54 |
johnthetubaguy | they are all compute resources though, really | 11:54 |
bauzas | johnthetubaguy: yup, agreed, they are all related to compute resources, but the model is different | 11:54 |
*** EinstCrazy has joined #openstack-nova | 11:55 | |
bauzas | johnthetubaguy: we have some slight differences between the ComputeNode object and the HostState object in terms on how fields are set | 11:55 |
johnthetubaguy | bauzas: have you done the follow up patch where the cut and paste is removed? | 11:55 |
bauzas | while it could be interesting to stop using HostState and rather ComputeNode object in the HostManager, that's not yet the case, and I just wanted to say that | 11:56 |
*** dave-mccowan has joined #openstack-nova | 11:56 | |
bauzas | johnthetubaguy: I was reviewing it yeah, not yet uploaded my point | 11:56 |
*** thorst has joined #openstack-nova | 11:56 | |
bauzas | johnthetubaguy: which is basically that I feel we should somehow reconsider UTs | 11:57 |
bauzas | but that's not a blocker also | 11:57 |
*** mvk has joined #openstack-nova | 11:59 | |
*** qeelee has joined #openstack-nova | 12:00 | |
*** haomaiwang has quit IRC | 12:01 | |
*** haomaiwang has joined #openstack-nova | 12:01 | |
*** openstackgerrit has quit IRC | 12:03 | |
johnthetubaguy | bauzas: so this file is totally new, with random cut and paste bits all over it: https://review.openstack.org/#/c/262938/8/nova/scheduler/claims.py | 12:03 |
johnthetubaguy | bauzas: that comment is very missleading | 12:03 |
*** openstackgerrit has joined #openstack-nova | 12:03 | |
*** thorst has quit IRC | 12:04 | |
bauzas | johnthetubaguy: which comment ? | 12:04 |
bauzas | johnthetubaguy: I did https://review.openstack.org/#/c/262938/8/nova/compute/claims.py@80 | 12:04 |
johnthetubaguy | bauzas: it says its a verbatum copy, its not | 12:04 |
bauzas | oh yeah | 12:04 |
*** aysyd has joined #openstack-nova | 12:04 | |
bauzas | johnthetubaguy: hence me wanting to reorganize by abstracting first the Claim model | 12:05 |
*** paul-carlton2 has quit IRC | 12:06 | |
ndipanov | bauzas, it's a bad idea | 12:06 |
ndipanov | why would the other patch not make it | 12:07 |
ndipanov | because of random deadlines | 12:07 |
ndipanov | ? | 12:07 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Sample nova.conf file has missing/duplicated config options https://review.openstack.org/287685 | 12:08 |
johnthetubaguy | ndipanov: its really, really hard to review those patches, I have been trying for the last hour to get my head around this series | 12:09 |
*** aysyd has quit IRC | 12:09 | |
*** deepakcs has quit IRC | 12:09 | |
johnthetubaguy | I am going to get some lunch and try again | 12:09 |
*** aysyd has joined #openstack-nova | 12:10 | |
ndipanov | johnthetubaguy, hmmm you do have a point that that class is not a verbatim copy | 12:11 |
ndipanov | I missed that | 12:11 |
*** PaulMurray has joined #openstack-nova | 12:12 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Nuke cliutils from oslo-incubator https://review.openstack.org/287753 | 12:18 |
*** doug-fish has joined #openstack-nova | 12:19 | |
openstackgerrit | Gábor Antal proposed openstack/nova: Replaced unittest and unittest2 to testtools https://review.openstack.org/285209 | 12:20 |
openstackgerrit | Sean Dague proposed openstack/nova: WIP: add new microversion header per standards https://review.openstack.org/287755 | 12:22 |
*** paul-carlton2 has joined #openstack-nova | 12:22 | |
markus_z | johnthetubaguy: I'm writing the ML post for the handling of potential rc blockers. We are planning to release RC1 in ~2 weeks, right? | 12:23 |
markus_z | johnthetubaguy: http://releases.openstack.org/mitaka/schedule.html#m-rc1 | 12:23 |
*** edtubill has joined #openstack-nova | 12:23 | |
markus_z | johnthetubaguy: I would suggest to use the Launchpad Milestones again to split up the potential blockers. As we did for Liberty. | 12:24 |
*** haomaiwang has quit IRC | 12:24 | |
*** shuquan has joined #openstack-nova | 12:24 | |
*** haomaiwang has joined #openstack-nova | 12:25 | |
*** rk4n has joined #openstack-nova | 12:27 | |
*** shuquan has quit IRC | 12:29 | |
*** haomaiwang has quit IRC | 12:29 | |
*** macsz has quit IRC | 12:29 | |
johnthetubaguy | markus_z: right, we use tags to identify possible ones, then the milestone target to agree the blocker, I believe | 12:30 |
*** josecastroleon has joined #openstack-nova | 12:30 | |
johnthetubaguy | markus_z: sorry, getting my lunch, but happy to help draft something after lunch | 12:30 |
*** shuquan has joined #openstack-nova | 12:30 | |
*** doug-fish has quit IRC | 12:31 | |
*** belmoreira has quit IRC | 12:31 | |
markus_z | johnthetubaguy: no worries, I'm drafting right now. Takes a bit to get it straight. I link the ML post here and you can comment there if I got something wrong. | 12:31 |
*** belmoreira has joined #openstack-nova | 12:31 | |
*** AndChat457281 has joined #openstack-nova | 12:33 | |
*** doug-fish has joined #openstack-nova | 12:34 | |
*** shuquan has quit IRC | 12:35 | |
*** thorst has joined #openstack-nova | 12:37 | |
*** qeelee has quit IRC | 12:37 | |
*** jaosorior has quit IRC | 12:39 | |
*** jaosorior has joined #openstack-nova | 12:39 | |
*** jaosorior has quit IRC | 12:39 | |
*** haomaiwang has joined #openstack-nova | 12:40 | |
*** jaosorior has joined #openstack-nova | 12:40 | |
alex_xu | sdague: sorry, I didn't get idea for how to fix testr. only find testr will return fault when only one test run but will success for multiple test, and also find this http://lists.openstack.org/pipermail/openstack-dev/2015-March/058446.html | 12:41 |
*** haomaiwang has quit IRC | 12:44 | |
*** rtheis has joined #openstack-nova | 12:46 | |
*** qeelee has joined #openstack-nova | 12:48 | |
sdague | alex_xu: what's up? | 12:51 |
alex_xu | sdague: just feedback to you what I find for why https://review.openstack.org/281143 can pass the regression test | 12:52 |
alex_xu | sdague: there is bug for testr https://bugs.launchpad.net/testrepository/+bug/1429196, looks like same thing, but didn't get fix | 12:52 |
openstack | Launchpad bug 1429196 in Testrepository "testr is inconsistent with testtools over unexpected successes" [High,Triaged] | 12:52 |
alex_xu | sdague: if I only run this one regression test https://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_1541691.py, testr will return fault | 12:54 |
*** cdent has quit IRC | 12:54 | |
alex_xu | if I run multiple tests, testr will tell me everything passed | 12:54 |
sdague | alex_xu: oh, right, so we can make that a fail in ostestr, that's where we should probably file the bug | 12:55 |
sdague | alex_xu: oh, interesting.... | 12:56 |
*** haomaiwang has joined #openstack-nova | 12:56 | |
alex_xu | sdague: ok, got it, will file a bug | 12:56 |
sdague | alex_xu: it's fine, we should just fix up the test now that the bug is fixed and then we're good | 12:57 |
alex_xu | sdague: emm..how to fix the test? | 12:58 |
sdague | alex_xu: well, let's land this patch, I'll redo the test after | 12:58 |
sdague | if you unWIP it | 12:58 |
alex_xu | sdague: ok, got it | 12:59 |
*** haomaiwang has quit IRC | 12:59 | |
*** haomaiwa_ has joined #openstack-nova | 12:59 | |
alex_xu | sdague: done | 12:59 |
*** haomaiwa_ has quit IRC | 13:01 | |
*** haomaiwa_ has joined #openstack-nova | 13:01 | |
*** baoli has joined #openstack-nova | 13:03 | |
*** haomaiwa_ has quit IRC | 13:07 | |
*** haomaiwa_ has joined #openstack-nova | 13:10 | |
markus_z | johnthetubaguy: mitaka-rc-blockers: http://lists.openstack.org/pipermail/openstack-dev/2016-March/088205.html | 13:10 |
*** raildo-afk is now known as raildo | 13:11 | |
*** markvoelker has quit IRC | 13:11 | |
*** markvoelker has joined #openstack-nova | 13:11 | |
markus_z | larsks: Are there news about https://review.openstack.org/#/c/264157/ ? You talked with thorst about that. | 13:13 |
*** pradk has joined #openstack-nova | 13:13 | |
*** sfinucan has joined #openstack-nova | 13:13 | |
*** ducttape_ has joined #openstack-nova | 13:13 | |
*** qeelee has quit IRC | 13:14 | |
*** qeelee has joined #openstack-nova | 13:14 | |
*** ndipanov has quit IRC | 13:14 | |
*** gongysh has joined #openstack-nova | 13:14 | |
*** astupnikov has quit IRC | 13:15 | |
*** efried has joined #openstack-nova | 13:16 | |
sdague | markus_z: I was staring at the config opts draft... there are some weird things in there. | 13:17 |
markus_z | sdague: An example? | 13:18 |
sdague | also, do we know if that's been updated recently, conductor manager I thought was deprecated | 13:18 |
sdague | markus_z: not weird as anything you did wrong, just some other options that maybe we want to deprecate | 13:18 |
sdague | let me dig up a thing once I file this bug | 13:18 |
markus_z | sdague: I put the odd options to the section "Config options to re-evaluate" of https://etherpad.openstack.org/p/config-options for a further analysis. | 13:19 |
*** gongysh has quit IRC | 13:19 | |
*** haukebruno has quit IRC | 13:19 | |
alex_xu | sdague: one more thing, i'm in the travel next week, may not available the api meeting, if i'm not here, would you mind running the meeting? | 13:20 |
*** haomaiwa_ has quit IRC | 13:23 | |
*** haukebruno has joined #openstack-nova | 13:23 | |
*** cdent has joined #openstack-nova | 13:25 | |
thorst | markus_z: I'm still setting up my env on 264157 for the RHEL test. Hoping to have that going today. | 13:27 |
*** coreywright has quit IRC | 13:27 | |
*** mgoddard_ has joined #openstack-nova | 13:28 | |
markus_z | thorst: cool, maybe it can make it into the release then | 13:28 |
thorst | markus_z: I hope! Just getting the RHEL image built and set up properly is where I had a hiccup. But I think I'm through that now | 13:30 |
*** qeelee has quit IRC | 13:30 | |
*** mgoddard has quit IRC | 13:31 | |
markus_z | sdague: https://bugs.launchpad.net/nova/+bug/1544044 The status doesn't fit your comment. Should it be incomplete or is a comment missing which explains why "confirmed" is right? | 13:31 |
openstack | Launchpad bug 1544044 in OpenStack Compute (nova) " Add new API to force live migration to complete" [Low,Confirmed] | 13:31 |
markus_z | sdague: I would also change the official tag "documentation" to "doc" to make it equal to the incoming bug reports from "DocImpact" merges. Any concerns? | 13:32 |
sdague | markus_z: sure, that's cool | 13:32 |
*** ivase has joined #openstack-nova | 13:33 | |
*** ducttape_ has quit IRC | 13:33 | |
*** links has quit IRC | 13:34 | |
sdague | markus_z: yeh, I don't know on that bug | 13:34 |
sdague | I think I misread it the first time and ignore my #1 comment | 13:35 |
kashyap | markus_z: Nice analysis on the config stuff, was just catching up with that thread on the list | 13:36 |
markus_z | pkoniszewski: Could you have a look at https://bugs.launchpad.net/nova/+bug/1544044 please? It's based on your patch. The manuals team would at least need technical input from us. | 13:37 |
openstack | Launchpad bug 1544044 in OpenStack Compute (nova) " Add new API to force live migration to complete" [Low,Confirmed] | 13:37 |
pkoniszewski | markus_z: sure | 13:37 |
*** xyang1 has joined #openstack-nova | 13:37 | |
markus_z | kashyap: thanks! nice to hear that. | 13:37 |
andreykurilin | johnthetubaguy: Hi! I wonder why novaclient releases don't have the same notification at mailing-list(from no-reply@openstack.org ) as oslo libraries have. | 13:37 |
johnthetubaguy | markus_z: thanks, that looks good. | 13:38 |
johnthetubaguy | andreykurilin: unsure, the release team generally handles all of those | 13:38 |
andreykurilin | johnthetubaguy: ok, will try to ping mriedem when he will be online:) | 13:39 |
*** gszasz has joined #openstack-nova | 13:39 | |
johnthetubaguy | andreykurilin: maybe we are supposed to do those then, I might be assuming wrongly | 13:39 |
*** Kevin_Zheng has quit IRC | 13:40 | |
*** kiran-r has quit IRC | 13:40 | |
sdague | markus_z: any idea how this is generated - http://docs.openstack.org/draft/config-reference/compute/config-options.html | 13:40 |
sdague | because I think it's pretty out of date | 13:40 |
markus_z | eliqiao: Could you have a look at https://bugs.launchpad.net/nova/+bug/1551782 please? It's based on your patch which states "DocImpact" but doesn't explain what the doc should look like. | 13:40 |
*** shaohe_feng has quit IRC | 13:40 | |
openstack | Launchpad bug 1551782 in OpenStack Compute (nova) " API: Improve os-migrateLive input parameters" [Undecided,New] | 13:40 |
*** shaohe_feng_ is now known as shaohe_feng | 13:40 | |
johnthetubaguy | sdague: markus_z: is that the one the docs team have to run a manual script to generate? | 13:41 |
markus_z | sdague: yeah, I looked into that. Let me find the pointer. | 13:41 |
markus_z | sdague: johnthetubaguy: Right, it's manually triggered | 13:41 |
sdague | johnthetubaguy: I don't know, there are some deprecations that aren't in there that I know we did a month ago | 13:41 |
sdague | it's honestly a much better way to view the world | 13:41 |
sdague | markus_z: anyway, I added some things to the etherpad | 13:42 |
johnthetubaguy | That might be part of the doc impact stuff, we might need to trigger that ourselves now, I keep meaning to dig into that more, but stuck doing release things atm. | 13:42 |
markus_z | sdague: thanks. Do they need immediate action or is the N cycle sufficient? | 13:42 |
sdague | well, I do wonder if we want to deprecate all the manager plug points | 13:43 |
dims | sdague : markus_z : http://git.openstack.org/cgit/openstack/openstack-manuals/tree/doc/config-reference/source/compute/config-options.rst that one? | 13:43 |
sdague | dims: ... maybe? | 13:44 |
johnthetubaguy | sdague: it seems worth trying to get that in ASAP | 13:44 |
pkoniszewski | sdague: hi, what do you mean by 'cleanup' there https://bugs.launchpad.net/nova/+bug/1544044/comments/2 ? | 13:44 |
openstack | Launchpad bug 1544044 in OpenStack Compute (nova) " Add new API to force live migration to complete" [Low,Confirmed] | 13:44 |
markus_z | dims: that's what gets generated, but there is one script who does that, I'm still digging | 13:44 |
*** unicell has joined #openstack-nova | 13:44 | |
sdague | pkoniszewski: typically we need to explain very clearly what openstack-manual change we would expect | 13:44 |
dims | andreykurilin : there should be no difference between the two, if you do, let me know please (release team hangs out on #openstack-release) | 13:44 |
sdague | then transfer the bug to the openstack manuals project | 13:45 |
sdague | johnthetubaguy: ok, I can work on a patch once I get breakfast | 13:45 |
*** coreywright has joined #openstack-nova | 13:47 | |
*** zenoway has quit IRC | 13:47 | |
*** zenoway has joined #openstack-nova | 13:47 | |
andreykurilin | dims: ok, let's move to openstack-release chat :) | 13:47 |
*** edmondsw has joined #openstack-nova | 13:48 | |
markus_z | sdague: dims: This one generates the config docs for the guide: https://github.com/openstack/openstack-doc-tools/tree/master/autogenerate_config_docs | 13:49 |
dims | markus_z : ah interesting | 13:49 |
dims | andreykurilin : ack | 13:49 |
markus_z | That was my source back then when I wanted to know how many options we have. | 13:49 |
*** kiran-r has joined #openstack-nova | 13:50 | |
*** Kevin_Zheng has joined #openstack-nova | 13:52 | |
bauzas | sdague: saw the grenade multinode CI job issue I provided here and in #openstack-qa ? | 13:52 |
*** anush has quit IRC | 13:52 | |
bauzas | sdague: basically, it takes a whiiiiile to pass thru the cinder plugin (because retries take 2 hours) | 13:52 |
bauzas | so the gate is taking like 7 hours delay atm because of some single job blocking the full others$ | 13:53 |
*** krtaylor has quit IRC | 13:54 | |
sdague | bauzas: no, I haven't seen the issue, what is it? | 13:54 |
*** vladikr has joined #openstack-nova | 13:55 | |
*** ankit_ag has quit IRC | 13:55 | |
*** EinstCrazy has quit IRC | 13:55 | |
*** EinstCrazy has joined #openstack-nova | 13:56 | |
*** sfinucan has quit IRC | 13:56 | |
openstackgerrit | John Garbutt proposed openstack/nova: Add Service.get_minimum_version_multi() for multiple binaries https://review.openstack.org/279312 | 13:56 |
*** unicell has quit IRC | 13:57 | |
bauzas | sdague: so the gate was blocked on a single job | 13:58 |
*** unicell has joined #openstack-nova | 13:58 | |
bauzas | sdague: which is gate-grenade-dsvm-multinode | 13:58 |
bauzas | for a specific change | 13:58 |
sdague | bauzas: currently? | 13:58 |
bauzas | no, earlier todaty | 13:58 |
sdague | ok | 13:58 |
*** busterswt has joined #openstack-nova | 13:58 | |
*** rlrossit has joined #openstack-nova | 13:58 | |
bauzas | so, the job was running for like 2.5h without doing anything | 13:58 |
sdague | ok | 13:58 |
bauzas | after some findings, it was due to a periodic retry in the cinder plugin | 13:59 |
bauzas | so the general devstack timeout killed it | 13:59 |
bauzas | sdague: http://logs.openstack.org/81/283781/6/gate/gate-grenade-dsvm-multinode/3c1b54b/logs/grenade.sh.txt.gz#_2016-03-03_09_51_29_865 | 13:59 |
sdague | ok, is this summarized somewhere in detail? Like a bug or mailing list post? | 13:59 |
bauzas | not yet | 13:59 |
bauzas | I was waiting you to be awaken to discuss on that :-) | 14:00 |
bauzas | as the timeout unblocked the gate | 14:00 |
sdague | right, it's why we have a hard backstop | 14:00 |
smcginnis | bauzas: Do you know what driver was being used? | 14:00 |
bauzas | but I guess it's probably related to the delay we already observed yesterday | 14:00 |
bauzas | smcginnis: not really but all the stuff is in http://logs.openstack.org/81/283781/6/gate/gate-grenade-dsvm-multinode/3c1b54b/logs/grenade.sh.txt.gz#_2016-03-03_09_51_29_865 | 14:01 |
bauzas | well, I mean in the logs I mean | 14:01 |
smcginnis | bauzas: Found it. LVM. That's odd. | 14:01 |
bauzas | lemme check localrc | 14:01 |
sdague | bauzas: this looks like a new provider | 14:01 |
sdague | vexxhost | 14:01 |
sdague | so are we sure that multinode jobs ever work on vexxhost? | 14:01 |
*** busterswt has quit IRC | 14:01 | |
bauzas | let's move that convo in -qa | 14:02 |
bauzas | jpittier had already an eye on it | 14:02 |
markus_z | sdague: AJeager in #openstack-doc told me that the draft config-reference docs got updated in Nov 2015 when they switched to rst. The next update will happen shortly before the release. | 14:02 |
*** busterswt has joined #openstack-nova | 14:03 | |
sdague | markus_z: can we get that more often | 14:03 |
sdague | it's really useful to see | 14:03 |
sdague | and audit in that format | 14:03 |
*** jed56 has quit IRC | 14:03 | |
dims | markus_z : sdague : seems like a matter of checking out openstack/openstack-doc-tools and running "autohelp-wrapper update nova" from autogenerate_config_docs directory | 14:04 |
bauzas | I think the config-references one is semi-automated | 14:04 |
bauzas | so that requires some human intervention | 14:04 |
markus_z | sdague: I agree. I'll ask on the ML. The openstack-doc-tools are the key but I cannot commit myself to do that. | 14:04 |
bauzas | at least that's what I saw when I was digging on how they did the November push | 14:04 |
johnthetubaguy | bauzas: so I think its time to do the tag for mitaka-3, I can't see us getting any further in the next hour or two, I have -2ed the remaining bits now | 14:05 |
markus_z | dims: right, that gives you the config-reference manual locally. | 14:05 |
bauzas | johnthetubaguy: I just uploaded a DNM for quick modifications when we are ready | 14:05 |
bauzas | johnthetubaguy: https://review.openstack.org/287810 | 14:05 |
*** rodrigods has quit IRC | 14:06 | |
*** rodrigods has joined #openstack-nova | 14:06 | |
bauzas | johnthetubaguy: I was actually waiting https://review.openstack.org/#/c/278125/14 and https://review.openstack.org/#/c/278598/13 to land but we had a job failure | 14:06 |
*** ctrath has joined #openstack-nova | 14:06 | |
johnthetubaguy | bauzas: yeah, I just gave them a FFE, since they have +W | 14:07 |
alaski | bauzas: yeah I was just looking at that. I'm not sure what Jenkins is struggling with "This change depends on a change that failed to merge." but the parent change is going through check just fine | 14:07 |
johnthetubaguy | I don't think they critically need to get in the tarball, as I read that | 14:07 |
bauzas | okay, so I can just edit the commit msg and ask ttx to take a look | 14:07 |
bauzas | alaski: the dependency merge problem is a known issue | 14:08 |
alaski | okay | 14:08 |
alaski | what's the best workaround? | 14:08 |
bauzas | alaski: it's just telling you that it has been reset because anyway your dependent patch failed to merge | 14:08 |
bauzas | alaski: well, recheck ? :D | 14:08 |
bauzas | provided your dependent change is able to make it done | 14:08 |
alaski | your recheck earlier didn't seem to trigger anything | 14:09 |
bauzas | wait ? seriously? | 14:09 |
bauzas | lemme see zuul | 14:09 |
*** unicell has quit IRC | 14:09 | |
bauzas | alaski: okay, so I guess 278598 was aborted by zuul because the dependent change had problems | 14:10 |
bauzas | the neutron job failure | 14:10 |
bauzas | but I asked to recheck https://review.openstack.org/#/c/278125/14 | 14:11 |
alaski | right. do we need to wait until that succeeds? | 14:11 |
*** jaypipes has joined #openstack-nova | 14:12 | |
bauzas | alaski: we can possibly ask to recheck 278598 but given that we're cutting m3 without that, I'd prefer to wait for it being at least verified by Jenkins | 14:12 |
bauzas | in order to save a bit of CI resources | 14:12 |
*** mdrabe has joined #openstack-nova | 14:12 | |
bauzas | alaski: john gave you FFEs anyway so no real rush to try to land both at same time | 14:12 |
alaski | sure. I'm just trying to understand what to do in this situation to appease jenkins | 14:12 |
bauzas | they will just be RC | 14:12 |
*** cdelatte has joined #openstack-nova | 14:13 | |
alaski | if the answer is wait until the parent patch merges that's fine | 14:13 |
bauzas | alaski: well, the problem is that a gate failure implies you asking the check pipeline again | 14:13 |
bauzas | so, that's twice consuming CI resources | 14:13 |
bauzas | alaski: yeah indeed | 14:13 |
*** mylu has joined #openstack-nova | 14:14 | |
bauzas | alaski: at least making sure that the dependent one is passing the check pipeline I feel | 14:14 |
bauzas | the gate is quite in a good shape now with 1h15 delay | 14:14 |
*** links has joined #openstack-nova | 14:14 | |
alaski | okay, so once it gets into the gate pipeline a recheck might work. I'll wait, just wondering for my own edification | 14:14 |
*** mylu has quit IRC | 14:15 | |
johnthetubaguy | bauzas: just edited the commit message, I think we are all good otherwise? https://review.openstack.org/#/c/287810/ | 14:15 |
bauzas | johnthetubaguy: just refreshing my local copy to verify the SHA | 14:16 |
bauzas | but yeah, I feel so | 14:16 |
*** tobasco_ is now known as tobasco | 14:16 | |
*** doug-fish has quit IRC | 14:17 | |
*** delattec has joined #openstack-nova | 14:17 | |
*** zengine has joined #openstack-nova | 14:17 | |
*** atuvenie has quit IRC | 14:20 | |
*** cdelatte has quit IRC | 14:20 | |
openstackgerrit | Pawel Koniszewski proposed openstack/nova: Use migrate_data.block_migration instead of block_migration https://review.openstack.org/287363 | 14:20 |
openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add unit tests for live_migration_cleanup_flags https://review.openstack.org/287821 | 14:20 |
*** delattec has quit IRC | 14:21 | |
*** mriedem has joined #openstack-nova | 14:21 | |
*** haomaiwang has joined #openstack-nova | 14:21 | |
*** ndipanov has joined #openstack-nova | 14:21 | |
dims | sdague : markus_z : https://review.openstack.org/#/c/287822/ | 14:22 |
*** doug-fish has joined #openstack-nova | 14:22 | |
dims | sdague : markus_z : "autohelp-wrapper update nova" then "autohelp-wrapper rst nova" from autogenerate_config_docs/ directory. then upload the changes from autogenerate_config_docs/sources/openstack-manuals/ directory | 14:24 |
*** doug-fish has quit IRC | 14:24 | |
*** edtubill has quit IRC | 14:24 | |
*** doug-fish has joined #openstack-nova | 14:24 | |
markus_z | dims: I would have expected more changes | 14:25 |
*** mgoddard has joined #openstack-nova | 14:26 | |
*** mgoddard_ has quit IRC | 14:26 | |
*** edtubill has joined #openstack-nova | 14:26 | |
*** geekinutah has joined #openstack-nova | 14:27 | |
markus_z | dims: I have a meeting in a few minutes, let's talk about it after that | 14:27 |
dims | markus_z : y, we'll have to peel the onion | 14:27 |
markus_z | yeah, last minute meeting cancel, I have time | 14:27 |
*** fishbone has joined #openstack-nova | 14:28 | |
ndipanov | johnthetubaguy, is there going to be a FFE process? I remember you saying that this is really the last call... | 14:28 |
johnthetubaguy | ndipanov: we only have two weeks till RC this time, so I think the release team suggested its basically more exceptional that normal | 14:30 |
johnthetubaguy | ndipanov: if there is something we really need, I can go ask, but right now its just stuff that has +W but not quite merged, for various reasons | 14:30 |
*** links has quit IRC | 14:30 | |
ndipanov | johnthetubaguy, ok cool | 14:31 |
*** busterswt has quit IRC | 14:31 | |
*** eharney has quit IRC | 14:31 | |
*** edtubill has quit IRC | 14:31 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Enable rebuild tests in cellsv1 job https://review.openstack.org/287830 | 14:31 |
*** erlon has joined #openstack-nova | 14:33 | |
*** erno has quit IRC | 14:36 | |
*** Guest60497 is now known as dansmith | 14:36 | |
*** thumpba has joined #openstack-nova | 14:38 | |
jaypipes | morning everyone. | 14:40 |
openstackgerrit | John Garbutt proposed openstack/nova: Use db connection from RequestContext during queries https://review.openstack.org/272329 | 14:41 |
*** Jeffrey4l has quit IRC | 14:41 | |
*** lykinsbd has joined #openstack-nova | 14:41 | |
*** knikolla has joined #openstack-nova | 14:41 | |
jaypipes | dansmith: up yet? I'm looking at the compute_node_statistics() patch now. looks like you and edleafe- were looking into the failures on that? /me looking at the PostgreSQL barfs now.. | 14:42 |
lucasagomes | johnthetubaguy, hi there, re FF, do you think that the host state level locking feature would be eligible for a feature freeze exception this cycle ( https://review.openstack.org/#/c/262939 ) ? | 14:42 |
johnthetubaguy | lucasagomes: I don't think it is really. | 14:42 |
dansmith | jaypipes: yeah, I had a ping from him last night that made it look like he thought it was good. is it still failing? | 14:42 |
*** akshai has joined #openstack-nova | 14:42 | |
*** mriedem has left #openstack-nova | 14:42 | |
lucasagomes | johnthetubaguy, the feature is fixing a annoying race that hits the ironic users a lot, we even introduced a workaround in the ironic driver to help mitigate it | 14:42 |
johnthetubaguy | lucasagomes: its a very tight time between m-3 and RC this cycle | 14:42 |
*** mriedem has joined #openstack-nova | 14:43 | |
lucasagomes | johnthetubaguy, right, there's anything that we can do to help? Reviewing that code? or having 2 cores in nova to sponsor that feature to amke sure it land this cycle? | 14:43 |
johnthetubaguy | lucasagomes: if you run a single nova-scheduler, and using the caching scheduler, it should avoid the worst of that | 14:43 |
*** sfinucan has joined #openstack-nova | 14:43 | |
*** mylu has joined #openstack-nova | 14:43 | |
lucasagomes | johnthetubaguy, we do run 1 sched, not sure about the caching tho... something to look at I guess | 14:44 |
*** irenab has quit IRC | 14:44 | |
jaypipes | dansmith: yeah. here is the failure from the PG logs: http://paste.openstack.org/show/489156/ | 14:44 |
markus_z | sdague: FYI http://lists.openstack.org/pipermail/openstack-dev/2016-March/088217.html | 14:45 |
*** mgoddard_ has joined #openstack-nova | 14:45 | |
dansmith | jaypipes: yep... greek to me | 14:45 |
lucasagomes | johnthetubaguy, ok cool, I will try to see what can be done and may talk to you later | 14:46 |
lucasagomes | thanks | 14:46 |
markus_z | dims: for you too http://lists.openstack.org/pipermail/openstack-dev/2016-March/088217.html | 14:46 |
edleafe- | jaypipes: did you see the fix I made? According to the docs, that should have worked, but obviously did not | 14:46 |
johnthetubaguy | lucasagomes: if it were lower risk or it was the same cycle length as other times, and if the code was already ready to merge, maybe, it just seems like too many hurdles at this point | 14:46 |
*** nelsnels_ has joined #openstack-nova | 14:46 | |
jaypipes | edleafe-: looking into it now... :) | 14:47 |
dims | markus_z : +1 | 14:47 |
johnthetubaguy | lucasagomes: I would be surpized that would make a big impact for ironic, if I am honest, it feels more important for PCI/NUMA related things, but maybe I am missing something | 14:47 |
lucasagomes | johnthetubaguy, yeah I understand... just wanted to kinda reach out to show that we had high hopes this would land this cycle | 14:47 |
lucasagomes | johnthetubaguy, it's more like Ironic + TripleO | 14:47 |
edleafe- | jaypipes: I was following the pattern at http://docs.sqlalchemy.org/en/latest/core/compiler.html - search for 'FunctionElement' | 14:48 |
lucasagomes | johnthetubaguy, cause when they deploy the undercloud the register N machines and try to deploy N machines (same number) | 14:48 |
*** Jeffrey4l has joined #openstack-nova | 14:48 | |
*** mgoddard has quit IRC | 14:48 | |
lucasagomes | johnthetubaguy, this can cause this race to be very apparent | 14:48 |
jaypipes | zzzeek: you around? could use your PostgreSQL wizard hat. | 14:48 |
johnthetubaguy | lucasagomes: yes, agreed its racey, I just don't see how the fix you are pointing at helps that race | 14:49 |
dansmith | johnthetubaguy: yeah, and if it does, it's buried in the refactor, which makes it really hard to review | 14:49 |
*** nelsnelson has quit IRC | 14:49 | |
*** ljxiash has joined #openstack-nova | 14:49 | |
lucasagomes | johnthetubaguy, this may help to understand https://review.openstack.org/#/c/226235/ | 14:50 |
jaypipes | edleafe-: I have a feeling it's a type conversion thing... | 14:50 |
edleafe- | jaypipes: what gives you that feeling? | 14:50 |
edleafe- | the log said it was an ifnull problem | 14:50 |
*** haomaiwang has quit IRC | 14:50 | |
*** haomaiwang has joined #openstack-nova | 14:51 | |
johnthetubaguy | lucasagomes: agreed about the race, I just don't understand how that patch actually helps | 14:51 |
jaypipes | edleafe-: because of this: "You might need to add explicit type casts." | 14:52 |
jaypipes | edleafe-: I'm currently looking at this: http://www.postgresql.org/docs/9.4/static/typeconv-func.html | 14:52 |
jaypipes | but there's no mention of ifnull() | 14:52 |
lucasagomes | johnthetubaguy, it helps because since we have many nodes being deployed at the same the schedule may return the same node for 2 spawn() workers | 14:52 |
edleafe- | jaypipes: that's because ifnull doesn't exist in postgres | 14:52 |
edleafe- | postgres uses coalesce() | 14:52 |
lucasagomes | johnthetubaguy, and the way we detect it in Ironic is because the first thing we do is to create a bound between the instance in Nova and the node in Ironic by setting the instance_uuid in the node | 14:53 |
bauzas | lucasagomes: so, see, I'm very concerned by all the changes it introduces so close to a release | 14:53 |
bauzas | lucasagomes: while it's something we can workaround by increasing the number of retries | 14:53 |
lucasagomes | johnthetubaguy, so when it tries to set the instance_uuid and the ironic API returns a conflict (409) it means that the node was already taken | 14:53 |
lucasagomes | even tho the scheduler returned it as it was suppose to be free | 14:53 |
jaypipes | edleafe-: COALESCE() should function identically... | 14:53 |
*** pradk has quit IRC | 14:53 | |
jaypipes | edleafe-: jinx. | 14:53 |
lucasagomes | bauzas, sure, I'm just trying to show up and say "hey, if we can make it ffe maybe I can try to help landing it" | 14:54 |
edleafe- | jaypipes: of course. But sqla doesn't alias the two. | 14:54 |
johnthetubaguy | lucasagomes: agreed about the problem, I just didn't understand how that patch series helps really, I could be missing something in that patch series | 14:54 |
*** AndChat457281 has quit IRC | 14:54 | |
bauzas | lucasagomes: if the change was less impacting, maybe, but it does try a verbatim copy with some slight differences that make me very concerned about creating some unexpected regressions if we're not enough verifying | 14:54 |
lucasagomes | bauzas, I understand the concerns and it's up to the nova project to decide | 14:54 |
edleafe- | jaypipes: your code used func.ifnull, and that doesn't get aliased to func.coalesce when running postgres | 14:54 |
lucasagomes | johnthetubaguy, that patch just makes it fail faster | 14:54 |
lucasagomes | johnthetubaguy, so the retry filter goes and pick another node for the deployment | 14:54 |
edleafe- | jaypipes: see http://version2beta.com/articles/migrating-from-mysql-to-postresql-using-sqlalchemy/ - search for ifnull | 14:55 |
lucasagomes | johnthetubaguy, prior to that it kept retrying to set the instance_uuid until it gave up and failed the deployment | 14:55 |
*** mgoddard_ has quit IRC | 14:55 | |
*** eharney has joined #openstack-nova | 14:55 | |
*** mgoddard has joined #openstack-nova | 14:55 | |
lucasagomes | so with that feature + using only 1 scheduler, that race would be solved and we wouldn't need to workaround it in the nova driver | 14:56 |
*** zengine has quit IRC | 14:56 | |
johnthetubaguy | lucasagomes: but the claim happens on the same stale data that the filters act on | 14:57 |
*** esracelik has quit IRC | 14:57 | |
bauzas | johnthetubaguy: it just makes the failure happen earlier in the scheduler, and not on the compute | 14:58 |
bauzas | johnthetubaguy: so, not technically a retry, see | 14:58 |
johnthetubaguy | bauzas: but the claim is on the same stale data as the filters right? | 14:58 |
bauzas | but that's not really changing the fact that ironic wants to place as many VMs as there free slots | 14:58 |
bauzas | johnthetubaguy: sure, but that's not a retry, and that's what ironic folks are concerned | 14:59 |
bauzas | anyway, you can't have an algorithm that is able to place optimistically N instances on N hosts | 15:00 |
bauzas | you necessarly need retries | 15:00 |
johnthetubaguy | bauzas: I guess I don't understand ironic scheduling, the filter should have ruled it out if the claim would fail, I thought? | 15:00 |
lucasagomes | johnthetubaguy, I probably need to dig more into the change to see how it actually works... I probably just had high hopes that the state locking thingy would fix that race for us | 15:00 |
lucasagomes | that was my impression by reading the spec at least | 15:00 |
bauzas | johnthetubaguy: no, because we consume the resources after we filter | 15:00 |
*** haomaiwang has quit IRC | 15:01 | |
*** belmoreira has quit IRC | 15:01 | |
johnthetubaguy | bauzas: but the resources are still in same in memory objects, I thought? | 15:01 |
*** haomaiwang has joined #openstack-nova | 15:01 | |
bauzas | johnthetubaguy: having the consume_from_requests() method being synchronized really helped their problem | 15:01 |
*** liverpooler has quit IRC | 15:02 | |
johnthetubaguy | bauzas: sure, that will help, but that has merged already | 15:02 |
bauzas | johnthetubaguy: having the scheduler claims in place just prevents some kind of back-and-forth but yeah, maybe it's unnecessary | 15:02 |
*** haomaiwang has quit IRC | 15:02 | |
johnthetubaguy | bauzas: the claims stop us duplicating logic in filters for all the complicated PCI and NUMA resources, I like that, I just don't see how it helps ironic | 15:03 |
bauzas | johnthetubaguy: the point is that when decrease your resource usage, it won't fail on the scheduler if you're not having enough space | 15:03 |
bauzas | ie. you can have negative values AFAIR | 15:03 |
*** zengine has joined #openstack-nova | 15:03 | |
*** ducttape_ has joined #openstack-nova | 15:03 | |
*** haomaiwang has joined #openstack-nova | 15:03 | |
bauzas | johnthetubaguy: it's just for accounting that you took some resources | 15:03 |
*** mrkz has joined #openstack-nova | 15:04 | |
*** haomaiwang has quit IRC | 15:04 | |
bauzas | so, 2 instances could end up having the same destination by doubling their resources | 15:04 |
bauzas | but | 15:04 |
bauzas | in the ironic case, there can only be a winner | 15:04 |
*** haomaiwang has joined #openstack-nova | 15:04 | |
bauzas | where one instance necessarly takes all the compute space | 15:04 |
*** rcernin has joined #openstack-nova | 15:04 | |
openstackgerrit | Vasyl Saienko proposed openstack/nova: Allow to update resource per single node. https://review.openstack.org/287485 | 15:04 |
bauzas | so, while the scheduler could be happy saying "meh, I have -8GB RAM available", it wouldn't fail, and that would be the compute claim which would say "sorry buddy, not enough space" which raises a retry | 15:05 |
johnthetubaguy | bauzas: but if you have the correct filters, they will fail were you to hit -ve resources, and the claim and filter are happing on the same stale, pre request (if not using caching scheduler) copy of the host state | 15:05 |
*** amotoki has joined #openstack-nova | 15:06 | |
bauzas | johnthetubaguy: no, because it's not atomic | 15:07 |
bauzas | johnthetubaguy: ie. you can have 2 concurrent requests running thru the filters before entering the synchronized portion | 15:07 |
johnthetubaguy | bauzas: but each request has its own copy of the host state | 15:07 |
zzzeek | jaypipes: something quick sure, im on PTO :) | 15:08 |
*** EinstCrazy has quit IRC | 15:08 | |
*** EinstCrazy has joined #openstack-nova | 15:08 | |
bauzas | johnthetubaguy: sec, verifying something | 15:08 |
*** mylu has quit IRC | 15:08 | |
jaypipes | zzzeek: no worries, I think I solved it already. | 15:09 |
zzzeek | jaypipes: k | 15:09 |
openstackgerrit | Sean Dague proposed openstack/nova: deprecate manager class options https://review.openstack.org/287867 | 15:09 |
jaypipes | zzzeek: thx though :) | 15:09 |
sdague | johnthetubaguy: for consideration - https://review.openstack.org/287867 | 15:09 |
bauzas | johnthetubaguy: so, there are 3 steps | 15:09 |
bauzas | johnthetubaguy: get_all_host_states, filter, consume_from_request | 15:10 |
edleafe- | jaypipes: I have the changes that explicitly calculate the expected values in the stats test | 15:10 |
edleafe- | jaypipes: want to share your fix, and I can merge them both? | 15:10 |
bauzas | johnthetubaguy: two concurrent requests could issue two calls to get_all_host_state and consequently have the same resource information, right? | 15:10 |
johnthetubaguy | bauzas: the same copy of that data, yes | 15:11 |
bauzas | right | 15:11 |
bauzas | that's not the same memory object, I agree | 15:11 |
ndipanov | johnthetubaguy, but stuff on compute host is always accurate | 15:11 |
ndipanov | the problem with ironic's use case is that | 15:11 |
ndipanov | you usually deploy X instances onto exactly X hosts | 15:12 |
jaypipes | edleafe-: yup, just running local test | 15:12 |
ndipanov | which means it's easy to hit 3 retries | 15:12 |
bauzas | johnthetubaguy: the problem is then that the filters won't deny one of the two requests if they don't know there is a concurrent request, right? | 15:12 |
ndipanov | bauzas, right | 15:12 |
ndipanov | and normally that gets corrected with the claim | 15:12 |
ndipanov | but triple O use case is | 15:12 |
johnthetubaguy | bauzas: I agree, but the claims wouldn't deny those either, in the scheduler claim | 15:12 |
ndipanov | johnthetubaguy, they would with the locking | 15:13 |
*** READ10 has joined #openstack-nova | 15:13 | |
openstackgerrit | Jay Pipes proposed openstack/nova: Make compute_node_statistics() use new schema https://review.openstack.org/286931 | 15:13 |
jaypipes | edleafe-: ^ | 15:13 |
johnthetubaguy | ndipanov: but the data is separate set of objects for each request, I thought? unless you are using the caching scheduler | 15:13 |
*** zz_dimtruck is now known as dimtruck | 15:13 | |
bauzas | johnthetubaguy: sec, lemme read the proposal again | 15:13 |
ndipanov | johnthetubaguy, nope | 15:13 |
edleafe- | jaypipes: looking now... | 15:13 |
bauzas | oh wait, ndipanov is right | 15:14 |
bauzas | that's the same reference | 15:14 |
ndipanov | now that's all good when there is plenty of capacity to make sure that 3 retries is usually fine | 15:14 |
ndipanov | but triple O use case is exactly the opposite | 15:14 |
ndipanov | there is the exact amount of capacity | 15:14 |
edleafe- | jaypipes: heh, that was one of my first approaches, but when I saw the docs, I thought that it wouldn't work | 15:15 |
bauzas | well, technically, that's not a pure Ironic problem, that's also affecting other drivers, that's just the one-off model from ironic makes it visible | 15:15 |
*** pushkaru has joined #openstack-nova | 15:15 | |
*** zenoway has quit IRC | 15:15 | |
bauzas | makes it *more visible | 15:16 |
ndipanov | bauzas, yeah | 15:16 |
*** zenoway has joined #openstack-nova | 15:16 | |
ndipanov | similar thing with pci devices in NFV proof-of-concept scenarios | 15:16 |
lucasagomes | bauzas, indeed | 15:16 |
ndipanov | sadly | 15:16 |
bauzas | but yeah, I need to confirm but if you're running only one oslo.messaging service worker, that does sounds your memory is shared when you have two concurrent greenthreads right? | 15:17 |
johnthetubaguy | ndipanov: bauzas: so I totally miss read that before, yeah, I see how the claims help now, even without the caching scheduler | 15:17 |
*** sridharg has quit IRC | 15:17 | |
johnthetubaguy | hmm, although a different race, the compute node updates, become more of an issue | 15:18 |
ndipanov | johnthetubaguy, so the only trick with that BP is that we don't hold a global lock during running of all the filters | 15:18 |
bauzas | so, you can have two requests but since you only have a shared HostManager storing the host_map info, that does impact that | 15:18 |
ndipanov | but 2 short lived locks | 15:18 |
bauzas | ndipanov: right, that's what I said to johnthetubaguy, the synchronized section is *after* filtering | 15:19 |
ndipanov | well before and after | 15:19 |
bauzas | by adding a check that if you have negative values, it will suck | 15:19 |
bauzas | for the moment, you can just decrement your usage, and you don't care whether it's silly or not | 15:19 |
johnthetubaguy | yeah, thats what I was thinking that happens with the caching_scheduer, but yeah, we already share that | 15:19 |
johnthetubaguy | the problem we have with ironic, is each build, when it hits the compute node, updates the central state, which then wipes out some of the previously cached stuff in the scheduler, and we get other races, I think, but I could be wrong | 15:20 |
bauzas | that's why I'm fine with that BP, and I feel it could help reducing the number of retries, but I still think it's something workaroundable for that release and that we could land very soon in Newton | 15:20 |
*** zenoway has quit IRC | 15:20 | |
mriedem | sdague: on https://review.openstack.org/#/c/287867/ | 15:20 |
mriedem | what about things like the security group api and network api and identity backend? | 15:21 |
ndipanov | bauzas, I don't think it's critical for most users, however it's kinda silly we landed half of it (locking) | 15:21 |
ndipanov | that said - not my patches so can't really speak on turnaround times | 15:21 |
bauzas | johnthetubaguy: well, fortunately, we write on DB every time we claim in the compute node | 15:22 |
johnthetubaguy | bauzas: right, thats the bit I am talking about | 15:22 |
bauzas | johnthetubaguy: so a scheduler request could read that fresh data right after | 15:22 |
bauzas | johnthetubaguy: then, not sure I see your point | 15:22 |
ndipanov | bauzas, yes that's correct | 15:22 |
ndipanov | every claim is a DB write that | 15:22 |
*** krtaylor has joined #openstack-nova | 15:23 | |
ndipanov | and holds a global lock | 15:23 |
ndipanov | so that's consistent | 15:23 |
*** tbachman has joined #openstack-nova | 15:23 | |
bauzas | I feel so | 15:23 |
ndipanov | well | 15:24 |
ndipanov | to the best of our knowledge :) | 15:24 |
edleafe- | jaypipes: pep8 bites again | 15:24 |
ndipanov | there's still some known races around resizes etc | 15:24 |
edleafe- | jaypipes: fixing | 15:24 |
bauzas | and that's also why trying to modify those writes to happen in the DB is subject to discussion, eh jaypipes ;) | 15:24 |
ndipanov | but they are very small windows | 15:24 |
johnthetubaguy | well, you loose the inflight things, I think | 15:24 |
bauzas | ndipanov: that's another problem | 15:24 |
bauzas | ndipanov: some move operations are not claiming resources | 15:24 |
ndipanov | bauzas, yeah not relevant to this discussion at all | 15:24 |
ndipanov | bauzas, even the onest that are | 15:24 |
jaypipes | edleafe-: guh, thx | 15:24 |
*** efoley has joined #openstack-nova | 15:24 | |
ndipanov | are not doing it exactly right | 15:24 |
bauzas | ndipanov: plus we force some operations to bypass the scheduler, which makes the scheduler blind about the resource usage | 15:25 |
ndipanov | bauzas, that is true | 15:25 |
ndipanov | and sad | 15:25 |
*** geekinutah has quit IRC | 15:25 | |
bauzas | jaypipes: that's probably another concern about trying to have the scheduler ubiquitous about resource usage | 15:25 |
bauzas | if you see the above point | 15:25 |
ndipanov | but also the scheduler will refresh if it's internal memory state is stale compared to the computes | 15:26 |
ndipanov | so now that we have exclusivity between that update of in-memory state | 15:26 |
ndipanov | and consume update after the filter ran | 15:26 |
ndipanov | we should be mostly good I think | 15:26 |
bauzas | yeah, like I said, the semaphore on consume_from_requests is a great benefit | 15:27 |
ndipanov | it has to be coupled with the semafore on _update_from_compute_node | 15:27 |
ndipanov | as both of those write to the memory variables | 15:27 |
edleafe- | bauzas: that bypassing is a problem that should be fixed | 15:27 |
ndipanov | bauzas, ^ | 15:28 |
edleafe- | bauzas: not designed around | 15:28 |
bauzas | edleafe-: it's an operator usecase, you know | 15:28 |
ndipanov | edleafe-, we are not designing around it at all :D | 15:28 |
*** dave-mccowan has quit IRC | 15:28 | |
doffm | What opererations bypass the scheduler? | 15:28 |
johnthetubaguy | yes, very much on purpose | 15:28 |
doffm | I'm curious. | 15:28 |
ndipanov | rebuild and live mig | 15:28 |
bauzas | doffm: live-migrate myinstance myhost | 15:28 |
ndipanov | and even boot | 15:28 |
*** dave-mccowan has joined #openstack-nova | 15:28 | |
ndipanov | if you use AZs | 15:28 |
bauzas | ndipanov: well, boot with force_hosts does call the scheduler | 15:28 |
bauzas | ndipanov: it's just not reading the filters | 15:29 |
ndipanov | bauzas, lool | 15:29 |
*** mtanino has joined #openstack-nova | 15:29 | |
bauzas | yeah I know | 15:29 |
* bauzas whispers | 15:29 | |
bauzas | mmm, not whisper | 15:29 |
edleafe- | ndipanov: they can operate without first checking with the scheduler, but they should also make their changes visible | 15:30 |
johnthetubaguy | doffm: think about patching a host and rotating your VMs, and patching the next host, etc. you really want to target a specific host, regardless of and server group host affinity, etc | 15:30 |
* bauzas shrugs (rather) | 15:30 | |
ndipanov | edleafe-, well currently they don't until the next RT periodic refresh :( | 15:30 |
bauzas | edleafe-: no, in some emergency situation like driving on ice, you want to cut the ABS | 15:30 |
bauzas | shutdown | 15:30 |
jaypipes | bauzas: I just read back through your conversation with ndipanov and I'm not understanding if you have a specific question for me? | 15:31 |
bauzas | jaypipes: I was just (again) pointing out how much I care about having the compute nodes baby-sitting the resource usage :D | 15:31 |
*** angdraug has joined #openstack-nova | 15:31 | |
edleafe- | bauzas: of course you do those things in an emergency. But the resultant state should be surfaced, no? | 15:32 |
openstackgerrit | Andrew Bogott proposed openstack/nova: Prevent scheduling of instances on full hosts. https://review.openstack.org/242251 | 15:32 |
*** john5223 has joined #openstack-nova | 15:32 | |
doffm | johnthetubaguy: Total aside, but I looked in to the aggregate_hosts mapping table. No strong opinion on API vs Cell. Performance wise its probably fine in cell. | 15:32 |
bauzas | jaypipes: by giving another example where sometimes ops want to migrate by bypassing the scheduler | 15:32 |
doffm | Its only accessed when initialising the HostState data in teh scheduler. | 15:32 |
doffm | *the | 15:32 |
bauzas | edleafe-: so it would be an eventual consistency | 15:32 |
johnthetubaguy | doffm: cools, yeah, it feels like it grows in size as you add cells | 15:32 |
edleafe- | bauzas: in emergencies, yes | 15:32 |
edleafe- | bauzas: in normal operation, no | 15:32 |
*** achanda has joined #openstack-nova | 15:33 | |
doffm | johnthetubaguy: Yep, I'll plan for cell for now. If anyone has strong opinions the other way... spec argument. | 15:33 |
jaypipes | bauzas: and I don't really care to reinforce bad practice like that :) | 15:33 |
bauzas | edleafe-: what if you want to migrate all your instances from HostA to HostB, and "regular" instances being scheduler on HostB ? | 15:33 |
bauzas | scheduled* | 15:33 |
bauzas | (dammit, I'm tired) | 15:33 |
mriedem | lucasagomes: umm, does ironic import nova's compute manager? https://github.com/openstack/ironic/blob/master/ironic/nova/compute/manager.py#L90 | 15:34 |
edleafe- | jaypipes: update posted to https://review.openstack.org/#/c/286931/ - somehow didn't show in IRC | 15:34 |
jaypipes | edleafe-: thx :) | 15:34 |
edleafe- | bauzas: what about it? During the migration you are going to have lots of scheduler failures | 15:34 |
edleafe- | bauzas: once the migrations are done, the failures should quickly return to near zero | 15:35 |
openstackgerrit | Jay Pipes proposed openstack/nova: DNM: Canary test https://review.openstack.org/287248 | 15:36 |
lucasagomes | mriedem, well we have this custom hostmanager yes | 15:36 |
jaypipes | dansmith: ^^ just a rebase for your canary. | 15:36 |
dansmith | jaypipes: okay | 15:36 |
bauzas | edleafe-: nothing to say much about that :-) | 15:36 |
*** jichen has quit IRC | 15:36 | |
mriedem | lucasagomes: can ironic use notifications or something? | 15:37 |
lucasagomes | mriedem, ironic *service* does not use it, we orient ironic users to set it as the "scheduler_host_manager" for nova | 15:37 |
lucasagomes | mriedem, not at the moment | 15:37 |
lucasagomes | mriedem, fyi http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-compute-to-use-the-bare-metal-service | 15:37 |
mriedem | lucasagomes: ok, so compute_manager=ironic.nova.compute.manager.ClusteredComputeManager | 15:38 |
mriedem | you might want to see sdague's patch which makes the compute manager no longer pluggable | 15:38 |
mriedem | b/c that will totally break this | 15:38 |
lucasagomes | mriedem, yup | 15:38 |
lucasagomes | mriedem, oh... you have it handy there? | 15:38 |
lucasagomes | jroll, ^ | 15:39 |
mriedem | https://review.openstack.org/#/c/287867/1/nova/service.py | 15:39 |
*** achanda has quit IRC | 15:39 | |
* edleafe- goes for some more coffee... | 15:39 | |
jroll | yikes | 15:39 |
*** ccarmack has joined #openstack-nova | 15:39 | |
ndipanov | we were just namedropping you jaypipes | 15:39 |
lucasagomes | mriedem, thanks for pointing it out | 15:39 |
mriedem | lucasagomes: to be fair, the ironic clustered compute manager module has a comment that this is basically a short term hack workaroudn | 15:40 |
mriedem | but that was from 2014 and it hasn't been fixed | 15:40 |
*** atuvenie has joined #openstack-nova | 15:41 | |
lucasagomes | mriedem, yeah, I think this is something that http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/ironic-multiple-compute-hosts.html | 15:41 |
lucasagomes | will fi | 15:42 |
lucasagomes | fix* | 15:42 |
lucasagomes | mriedem, but it depends on some stuff in ironic which is not in place yet | 15:42 |
lucasagomes | jroll, ^ | 15:42 |
mriedem | markus_z: this is going to be an rc blocker https://bugs.launchpad.net/nova/+bug/1548980 | 15:43 |
openstack | Launchpad bug 1548980 in OpenStack Compute (nova) "nova list --deleted as admin fails with 404" [High,In progress] - Assigned to Matt Riedemann (mriedem) | 15:43 |
mriedem | patch is here https://review.openstack.org/#/c/286423/ | 15:43 |
mriedem | was a regression in the 2.16 microversion | 15:43 |
sdague | mriedem: I thought we had a specific call out for that instead | 15:44 |
mriedem | sdague: i'm just going based on the ironic docs pointed out above | 15:44 |
*** yarkot_ has joined #openstack-nova | 15:44 | |
lucasagomes | sdague, yeah i think it was discussed in Tokyo | 15:45 |
sdague | mriedem: oh, I guess it was https://github.com/openstack/nova/blob/c702ab87c30081625358136ca910add60f650f2b/nova/conf/scheduler.py#L247 | 15:45 |
sdague | there is a constants bit there | 15:45 |
sdague | so is there another bit that we need to do like that? | 15:46 |
bauzas | sdague: that's what is indeed concerned by the ironic spec that we discussed in YVR | 15:46 |
bauzas | lucasagomes: why are you needing a fresh compute manager ? | 15:46 |
mriedem | sdague: the compute manager | 15:46 |
mriedem | sdague: see http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-compute-to-use-the-bare-metal-service | 15:46 |
mriedem | compute_manager=ironic.nova.compute.manager.ClusteredComputeManager | 15:46 |
sdague | mriedem: hmmm... so are we long term satisfied having an out of tree compute manager? | 15:47 |
lucasagomes | bauzas, IIRC, otherwise nova would mess up tracking the the available resouces in Ironic | 15:47 |
bauzas | lucasagomes: you want that ironic compute manager because you want some synchronized method around consuming the resource usage ? | 15:48 |
dansmith | that's the short term hack, right? | 15:48 |
bauzas | lucasagomes: that's something we fixed in an earlier release | 15:48 |
sdague | lucasagomes: right, ok, agreed there is currently a need | 15:48 |
lucasagomes | bauzas, cause, in Ironic every conductor can potentially manage every node in the cluster | 15:48 |
mriedem | dansmith: the 2014 short term hack | 15:48 |
dansmith | mriedem: right | 15:48 |
dansmith | just saying, long term, we do not want that | 15:48 |
lucasagomes | bauzas, where in nova, the computes manage a part of the cluster | 15:48 |
dansmith | and there is something in place to address it going forward | 15:48 |
bauzas | lucasagomes: sec, lemme verify but I think we put some semaphore in Kilo | 15:48 |
bauzas | lucasagomes: in the RT | 15:48 |
sdague | dansmith: what's the long term bit to address this? | 15:49 |
lucasagomes | bauzas, so if you have more than 1 instance of compute loaded with the Ironic driver they will report the usage for the same nodes in the nova model | 15:49 |
mriedem | dansmith: i'm assuming that's one of the 20 scheduler specs? | 15:49 |
bauzas | lucasagomes: which is litterally doing what you do in your module | 15:49 |
*** yarkot_ has quit IRC | 15:49 | |
mriedem | resource / inventory / allocation schmorgasboard? | 15:49 |
lucasagomes | sdague, yeah, sorry for that... We are working in Ironic to get the bits in place | 15:49 |
dansmith | sdague: ironic doing dynamic management of multiple services owning nodes at the time of deployment | 15:49 |
dansmith | sdague: the "jroll doctrine" | 15:49 |
sdague | so I think deprecating this is probably still fine | 15:49 |
mriedem | the jroll tuna roll?! | 15:49 |
dansmith | yes definitely | 15:49 |
bauzas | mriedem: sdague: this is what we want http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/ironic-multiple-compute-hosts.html | 15:49 |
sdague | we just don't remove until ironic is sorted | 15:49 |
dansmith | sdague: yes | 15:49 |
lucasagomes | bauzas, not sure what you mean by put some semaphore in Kilo | 15:50 |
bauzas | sec | 15:50 |
sdague | that will signal to other folks this is going away as well | 15:50 |
*** EinstCrazy has quit IRC | 15:50 | |
sdague | which is kind of the important bit | 15:50 |
*** Daisy has joined #openstack-nova | 15:50 | |
*** EinstCrazy has joined #openstack-nova | 15:50 | |
bauzas | lucasagomes: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L528-L529 | 15:51 |
*** jlanoux has quit IRC | 15:52 | |
bauzas | 2014.2 | 15:52 |
lucasagomes | bauzas, right, but I think the problem still no? The ironic model does not fit the nova one very well | 15:52 |
lucasagomes | bauzas, cause computes are "clusterized" | 15:52 |
bauzas | lucasagomes: oh, because you have N RTs, right? | 15:52 |
lucasagomes | where in nova model, each compute will just report the usage/available resources of a small part of the whole cluster | 15:53 |
bauzas | lucasagomes: ah okay, so you want a global lock, I see | 15:53 |
bauzas | not something in the RT | 15:53 |
lucasagomes | I think it's more like an architectural problem | 15:53 |
bauzas | I agree | 15:53 |
lucasagomes | not sure if we can solve it with locks | 15:53 |
lucasagomes | e.g 2 computes will report the same available resources | 15:53 |
bauzas | that's exactly why we agreed on http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/ironic-multiple-compute-hosts.html :-) | 15:54 |
lucasagomes | nova will then think we have twice as much as ironic actually have available | 15:54 |
lucasagomes | bauzas, yes | 15:54 |
lucasagomes | and that spec depends on some changes in the ironic side, like a claim/filter api | 15:54 |
lucasagomes | which haven't landed yet | 15:54 |
*** EinstCra_ has joined #openstack-nova | 15:55 | |
bauzas | well | 15:55 |
*** pece has quit IRC | 15:55 | |
bauzas | pushing back the removal because of that borked model would make me sad :) | 15:55 |
*** EinstCrazy has quit IRC | 15:57 | |
*** yamahata has joined #openstack-nova | 15:57 | |
lucasagomes | bauzas, honestly, I beleive there are some hypervisors that also works in a "clustered" way | 15:58 |
lucasagomes | not sure if it's something nova wants to address properly in the future or not | 15:58 |
*** anush has joined #openstack-nova | 15:58 | |
lucasagomes | it sounds like a big change in architectural terms | 15:58 |
*** amotoki has quit IRC | 15:59 | |
jroll | lucasagomes: sdague: dansmith: I agree long term we don't want that compute manager. I didn't realize it was in our docs, it definitely isn't required, just provides some form of HA (with extra races). I do think deprecating it is totally cool. | 16:00 |
bauzas | we can do big architectural changes :) | 16:00 |
sdague | jroll: cool | 16:00 |
*** haomaiwang has quit IRC | 16:01 | |
*** EinstCra_ has quit IRC | 16:01 | |
*** baoli has quit IRC | 16:01 | |
*** ssurana has joined #openstack-nova | 16:01 | |
*** EinstCrazy has joined #openstack-nova | 16:01 | |
*** haomaiwa_ has joined #openstack-nova | 16:01 | |
dansmith | jroll: yeah | 16:02 |
*** baoli has joined #openstack-nova | 16:02 | |
*** e0ne has quit IRC | 16:02 | |
lucasagomes | right, yeah the compute manager is only needed if we have >=2 n-compute loaded with ironic | 16:03 |
sdague | ok, so I screwed up the cells one, I'll fix that | 16:03 |
sdague | mriedem: what were your other concerns? | 16:03 |
lucasagomes | sdague, so deprecating is ok, but let's make sure we don't remove until we sort it out for ironic | 16:03 |
sdague | lucasagomes: sure, I'll leave a code comment | 16:03 |
lucasagomes | or it will impact in scaling out ironic with nova | 16:03 |
lucasagomes | sdague, ack, thanks much | 16:03 |
openstackgerrit | Ludovic Beliveau proposed openstack/nova: Update binding:profile for SR-IOV ports https://review.openstack.org/242573 | 16:03 |
*** yhvh- is now known as yhvh | 16:03 | |
mriedem | sdague: well, network api and security group api and key manager are all pluggable too | 16:04 |
mriedem | i'm not sure how far you want to take this | 16:04 |
mriedem | at least auth_strategy is no longer open ended | 16:04 |
mriedem | volume_api_class | 16:05 |
mriedem | https://github.com/openstack/nova/blob/master/nova/volume/__init__.py#L22 | 16:05 |
mriedem | so the volume api is pluggable | 16:05 |
dansmith | deprecate them all, IMHO | 16:05 |
markus_z | mriedem: ok, thanks | 16:05 |
*** EinstCrazy has quit IRC | 16:06 | |
*** rcernin has quit IRC | 16:06 | |
*** ssurana has quit IRC | 16:07 | |
*** roeyc has quit IRC | 16:07 | |
markus_z | mikal: Do you think you can have a look at bug fix https://review.openstack.org/#/c/215613/8 ? 4 months old and fixes a race condition. | 16:07 |
*** haplo37 has joined #openstack-nova | 16:07 | |
sdague | ok, I'll do another round and look for them | 16:07 |
mriedem | i can't tell if the image api is pluggable | 16:08 |
sdague | I don't think so | 16:08 |
*** jistr is now known as jistr|call | 16:08 | |
mriedem | the code appears to use glance always but there is the s3 image backend | 16:08 |
sdague | security_group api is more complicated, right? | 16:08 |
mriedem | so i think it is pluggable somehow, i actually traced that down a couple of weeks ago | 16:08 |
sdague | because n-net and neutron | 16:08 |
mriedem | well, | 16:08 |
*** mdrabe has quit IRC | 16:08 | |
mriedem | if n-net, security_group_api is nova.compute.api.SecurityGroupAPI or whatever | 16:08 |
mriedem | we have 2 sg drivers in tree, one for nova-net and one for neutron | 16:09 |
sdague | right | 16:09 |
mriedem | i'd probably just remove that even as an option | 16:09 |
mriedem | if your network api is neutron, you use the neutron security group driver, and the same for n-net | 16:09 |
mriedem | it seems silly not to | 16:09 |
*** thorst is now known as thorst_afk | 16:09 | |
*** ssurana has joined #openstack-nova | 16:09 | |
sdague | sure, do we actually have a big global switch for that though? all that code is fuzzy to me | 16:10 |
sdague | I'll look there | 16:10 |
mriedem | for network api yes, | 16:10 |
mriedem | BUT | 16:10 |
bauzas | sdague: I just said in the review that a scheduler manager entrypoint is not something we use atrm | 16:10 |
mriedem | it checks neutron and nova and then imports from classpath | 16:10 |
sdague | bauzas: yep | 16:11 |
sdague | saw that | 16:11 |
*** jdurgin1 has joined #openstack-nova | 16:11 | |
sdague | but we still tell people we'll deprecate before remove | 16:11 |
sdague | so that seems fine | 16:11 |
bauzas | okay, good | 16:11 |
bauzas | wfm | 16:11 |
bauzas | just as a fyi | 16:11 |
mriedem | sdague: quota_driver is another one | 16:11 |
mriedem | which is pluggable | 16:11 |
mriedem | that should probably either be DB or None | 16:11 |
cfriesen_ | just discovered this internally: https://bugs.launchpad.net/nova/+bug/1552777 It looks like nova can't resize from a flavor with swap to one without | 16:11 |
openstack | Launchpad bug 1552777 in OpenStack Compute (nova) "resizing from flavor with swap to one without breaks" [Undecided,New] | 16:11 |
mriedem | s/None/noop/ i mean | 16:12 |
*** permalac has joined #openstack-nova | 16:12 | |
cfriesen_ | heh...timing. I thought the automated message was lost or something. | 16:12 |
sdague | mriedem: ok, well, how about this. I'll try to get one patch up with is all the ones that can just become a constant | 16:12 |
*** mspreitz has joined #openstack-nova | 16:13 | |
sdague | and for the things that need to be an option of some kind I'll try to stack those up a bit differently. Those are going to take some time as they are more than just writing DEPRECATED | 16:13 |
*** browne has joined #openstack-nova | 16:13 | |
*** BobBall is now known as BobBall_AWOL | 16:14 | |
mriedem | sure.... | 16:14 |
mriedem | i think you should probably post a thread about this also | 16:14 |
bauzas | lucasagomes: btw. a second eye would be appreciated for https://review.openstack.org/#/c/287244/1/releasenotes/notes/ironic_api_version_opt_deprecated-50c9b0486e78fe6e.yaml | 16:15 |
sdague | mriedem: sure, as an FYI? or are you looking for feedback? | 16:16 |
mriedem | at least fyi | 16:16 |
*** nkrinner has quit IRC | 16:16 | |
sdague | I'm good with it as an FYI | 16:16 |
sdague | will do | 16:16 |
mriedem | i don't think this is a trivial shift | 16:16 |
lucasagomes | bauzas, oh right, thanks! Yeha I bet we forgot to add a release note when the patch for microversion merged in nova | 16:16 |
sdague | sure | 16:16 |
mriedem | i realize public cloud interop rah rah rah, but there are still a lot of people running small private clouds or building products off openstack | 16:16 |
sdague | it's also part of the release notes, I'll clean them up to list all the options | 16:16 |
mriedem | yeah, i think this kind of thing is bigger than just a release note entry though | 16:17 |
sdague | it's not just public cloud interop | 16:17 |
*** Daisy has quit IRC | 16:17 | |
sdague | it's off the shelf product interop as well | 16:17 |
sdague | but I do get the concern | 16:17 |
*** Daisy has joined #openstack-nova | 16:17 | |
mriedem | i think if you're buying an off the shelf product from a vendor, you get any vendorism issues that come with it | 16:17 |
mriedem | and you can't report bugs for that stuff upstream and expect us to care | 16:18 |
*** hemnafk is now known as hemna | 16:18 | |
lucasagomes | bauzas, reviewed | 16:20 |
*** zengine has quit IRC | 16:21 | |
bauzas | coolness, that's not super urgent, we're just preparing RC1 | 16:21 |
sdague | mriedem: if you are buying an off the shelf product from a vendor that is OpenStack (tm), then I disagree | 16:21 |
*** mdrabe has joined #openstack-nova | 16:21 | |
mriedem | sdague: then openstack should have a different license | 16:21 |
mriedem | openstack (tm) | 16:21 |
sdague | remember, this is all python, you can monkey patch this class out if you like | 16:21 |
mriedem | right, and vendors will just revert this patch | 16:21 |
*** Daisy has quit IRC | 16:22 | |
sdague | mriedem: sure, that's fine, if they change the behavior in a way that's not defcore, then they aren't openstack (tm). | 16:22 |
mriedem | yeah i guess i didn't think about defcore | 16:23 |
mriedem | i rarely do honestly | 16:23 |
mriedem | but if interop is a thing you care about as a customer you will want to see the defcore stamp of approval on said vendored product | 16:23 |
markus_z | johnthetubaguy: you -2ed two reviews which are on the "mitaka-3 blockers" list in the priorities etherpad, that's OK I guess? I just mention it in the rare case it slipped through. | 16:24 |
*** jlanoux has joined #openstack-nova | 16:24 | |
*** armax has joined #openstack-nova | 16:25 | |
*** yamahata has quit IRC | 16:26 | |
openstackgerrit | Jianghua Wang proposed openstack/nova-specs: XenAPI: support VGPU via passthrough PCI https://review.openstack.org/280099 | 16:26 |
*** jistr|call is now known as jistr | 16:26 | |
bauzas | markus_z: M-3 has weighed anchor today already | 16:29 |
*** ctrath has quit IRC | 16:30 | |
*** burt has joined #openstack-nova | 16:30 | |
sdague | mriedem: right | 16:30 |
*** Swami has joined #openstack-nova | 16:30 | |
sdague | and in past vendors have said - we want to invalidate these tests because you have a config option that lets us radicially change our behavior | 16:30 |
bauzas | mriedem: lucasagomes: so IIUC, I should mention in https://review.openstack.org/#/c/287244/1/releasenotes/notes/ironic_api_version_opt_deprecated-50c9b0486e78fe6e.yaml that now it pins nova to 1.8 or 1.9 ? | 16:30 |
bauzas | by lucasagomes's comment, it's v1.9 for mitaka, right? | 16:31 |
bauzas | well, I guess it will depend on the client version | 16:31 |
lucasagomes | bauzas, the ironic nova driver pins it at 1.8 | 16:31 |
markus_z | bauzas: OK, just wanted to double-check | 16:31 |
lucasagomes | bauzas, 1.9 is the default in the python client | 16:31 |
*** yamahata has joined #openstack-nova | 16:32 | |
mriedem | lucasagomes: and ironic api v1.8 was added in liberty right? | 16:32 |
lucasagomes | mriedem, yup | 16:32 |
mriedem | so nova's ironic driver in mitaka will require an ironic api from at least liberty | 16:32 |
mriedem | i think that's what i was trying to get across in the release note review | 16:32 |
bauzas | lucasagomes: so like mriedem said, it implies an ironic liberty deployment at least | 16:32 |
*** ctrath has joined #openstack-nova | 16:32 | |
*** RuiChen has quit IRC | 16:33 | |
bauzas | okay, that's clear for me | 16:33 |
lucasagomes | yeah | 16:33 |
*** RuiChen has joined #openstack-nova | 16:33 | |
bauzas | lucasagomes: and thanks for Vim, but I use Atom :) | 16:34 |
mriedem | sdague: this came up in the ML, it's a weird one | 16:35 |
mriedem | https://gist.github.com/mriedem/1299a15007e413ff646a | 16:35 |
lucasagomes | bauzas, np, you should enable some vim mode for it then ;D | 16:35 |
* lucasagomes is jk | 16:35 | |
mriedem | a non-admin can't list deleted instances using --status=deleted | 16:35 |
PaulMurray | bauzas, paul-carlton re forcing to a host that is disabled | 16:35 |
mriedem | but a non-admin can use nova list --deleted, but rather than error, it just returns non-deleted instances :) | 16:35 |
*** haukebruno has quit IRC | 16:35 | |
bauzas | PaulMurray: yup ? | 16:36 |
bauzas | (I'll have to leave soon) | 16:36 |
PaulMurray | bauzas, paul-carlton the filter scheduler host manager explicitly excludes disabled hosts in get_all_host_states | 16:36 |
*** jaypipes has quit IRC | 16:36 | |
PaulMurray | bauzas, paul-carlton so they are not available for scheduling let alone forcing | 16:36 |
*** Jeffrey4l has quit IRC | 16:36 | |
PaulMurray | that's why we can't force to a disable host | 16:37 |
PaulMurray | bauzas, paul-carlton see: https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L547-L560 | 16:39 |
*** mjura has quit IRC | 16:39 | |
PaulMurray | objects.ServiceList.get_by_binary( | 16:39 |
PaulMurray | ) does not return disabled hosts | 16:39 |
*** paul-carlton2 has quit IRC | 16:39 | |
PaulMurray | (or rather service binary) | 16:40 |
*** harlowja_at_home has joined #openstack-nova | 16:40 | |
PaulMurray | bauzas, no worries, paul-carlton is dealing with it | 16:40 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Reno for libvirt libosinfo with OS https://review.openstack.org/287328 | 16:40 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Reno for VMWare opaque networks https://review.openstack.org/287210 | 16:40 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Reno for Ironic api_version opt deprecation https://review.openstack.org/287244 | 16:40 |
bauzas | PaulMurray: oh man, that's terrible | 16:40 |
*** baoli has quit IRC | 16:41 | |
*** yamahata has quit IRC | 16:41 | |
*** baoli has joined #openstack-nova | 16:41 | |
bauzas | PaulMurray: we already have the ComputeFilter for that case, it's just sad to filter out the disabled nodes IMHO | 16:42 |
*** scheuran has quit IRC | 16:42 | |
*** busterswt has joined #openstack-nova | 16:42 | |
*** diogogmt has joined #openstack-nova | 16:43 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Versioned notification transformation https://review.openstack.org/286675 | 16:43 |
bauzas | and we're absolutely not consistent across all our Service DB API methods :/ | 16:43 |
bauzas | so, depending on which you call, you won't get the same set of results even if you think so | 16:43 |
mriedem | tjones_zzzz: new vmware bug https://bugs.launchpad.net/nova/+bug/1552786 | 16:44 |
openstack | Launchpad bug 1552786 in OpenStack Compute (nova) "VMware: Port Group and Port ID not explicit from port binding" [Undecided,New] | 16:44 |
tjones_zzzz | mriedem: checking | 16:44 |
tjones_zzzz | i'll get someone to take a look | 16:45 |
mriedem | tjones_zzzz: it actually sounds like a blueprint | 16:45 |
tjones_zzzz | :-( | 16:45 |
*** rk4n has quit IRC | 16:45 | |
PaulMurray | bauzas, its been working by coincidence for years ! | 16:45 |
sdague | mriedem: do you have a reproduce for that? | 16:46 |
sdague | because it does look odd | 16:46 |
mriedem | sdague: yeah | 16:46 |
*** atuvenie has quit IRC | 16:46 | |
mriedem | i can push a functional regression test patch | 16:46 |
sdague | ok, I need to meet some folks for lunch | 16:46 |
mriedem | your favorite! | 16:46 |
sdague | mriedem: it is! | 16:46 |
sdague | if you do that, I'll fix the bug :) | 16:46 |
sdague | I'll look once I get back online this afternoon | 16:47 |
mriedem | ndipanov: i added you to this since you probably know right off the bat if it's legit or not https://review.openstack.org/#/c/252836/ | 16:48 |
ndipanov | 2 legit 2 quit | 16:49 |
ndipanov | mriedem, it is | 16:49 |
*** armax has quit IRC | 16:49 | |
mriedem | ok | 16:49 |
mriedem | i thought so...given the comments in the code | 16:49 |
mriedem | i'll +W if you +2 | 16:49 |
*** armax has joined #openstack-nova | 16:51 | |
mriedem | cfriesen_: did you resize to the same host or migrate? | 16:51 |
mriedem | i guess same host since it's single-node, nvm | 16:52 |
*** sdague has quit IRC | 16:52 | |
*** gszasz has quit IRC | 16:53 | |
ndipanov | mriedem, I want better tests | 16:53 |
ndipanov | I think I'm entitled to them | 16:53 |
ndipanov | :) | 16:53 |
mriedem | geez, and you're not even a millenial | 16:54 |
ndipanov | make nova tests great again | 16:54 |
mriedem | *millennial | 16:54 |
ndipanov | what do you mean I'm not a millennial? | 16:54 |
mriedem | ndipanov: you could be the republican candidate with talk like that | 16:54 |
mriedem | ndipanov: how old are you? | 16:55 |
ndipanov | 32 | 16:55 |
*** shangxdy has joined #openstack-nova | 16:55 | |
mriedem | shit, i guess i technically am too | 16:55 |
ndipanov | yeah man! | 16:55 |
edleafe- | babies! | 16:55 |
mriedem | i thought it was only rlrossit and his new hire buddies | 16:55 |
ndipanov | it's twitter and organic coffee all day long | 16:55 |
mriedem | crap | 16:56 |
rlrossit | what am I doing? | 16:56 |
rlrossit | my ears are burning | 16:56 |
*** mspreitz has quit IRC | 16:56 | |
rlrossit | vote green | 16:57 |
rlrossit | vote independent | 16:57 |
rlrossit | mriedem: were you still in SD when the body was the governer? | 16:57 |
mriedem | i can't be lumped into the same demographic as someone that doesn't know about the wizard | 16:57 |
mriedem | http://www.imdb.com/title/tt0098663/?ref_=nm_flmg_act_31 | 16:57 |
*** RuiChen has quit IRC | 16:57 | |
mriedem | rlrossit: yeah...of MN you mean right? | 16:58 |
mriedem | however, i did enjoy jesse's role in predator at the age of ~6 | 16:58 |
*** RuiChen has joined #openstack-nova | 16:58 | |
mriedem | because he ain't got time to bleed | 16:58 |
*** thorst_afk is now known as thorst_ | 16:59 | |
rlrossit | mriedem: yeah. we had a cool governor before california go the governator | 17:00 |
*** yamahata has joined #openstack-nova | 17:00 | |
rlrossit | Who *doesn't* want a famous XFL announcer to be their governor?! | 17:00 |
*** haomaiwa_ has quit IRC | 17:01 | |
mriedem | or reality tv show boss to be their president | 17:01 |
mriedem | what a country | 17:01 |
*** burt has quit IRC | 17:01 | |
*** haomaiwa_ has joined #openstack-nova | 17:01 | |
ccarmack | mriedem: have time for a question? | 17:02 |
*** doug-fish has quit IRC | 17:02 | |
mriedem | not unless it's about tv and movie stars running for office | 17:02 |
*** rdopiera has quit IRC | 17:02 | |
ccarmack | sorry | 17:02 |
mriedem | danpb: you might like this one :) https://bugs.launchpad.net/nova/+bug/1552023 | 17:02 |
openstack | Launchpad bug 1552023 in OpenStack Compute (nova) "live-migration error message points to wrong webpage" [Low,Confirmed] | 17:02 |
*** jaosorior has quit IRC | 17:04 | |
mriedem | oh i see the problem | 17:05 |
mriedem | ccarmack: you can ask your question | 17:05 |
mriedem | w/o asking if you can ask | 17:05 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: trivial: remove unused argument from a method https://review.openstack.org/286743 | 17:06 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: libvirt: Refactor _update_xml https://review.openstack.org/286742 | 17:06 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 17:06 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML https://review.openstack.org/286744 | 17:06 |
ccarmack | mriedem: we are getting this message in the logs "While synchronizing instance power states, found 5 instances in the database and 3 instances on the hypervisor." Is the only way to get rid of this to delete instances? | 17:07 |
mriedem | umm, you could... | 17:07 |
mriedem | you might want to debug what happened to those other 2 instances | 17:07 |
mriedem | unless someone knows they deleted them outside nova-api | 17:08 |
ccarmack | I doubt that | 17:08 |
*** doug-fish has joined #openstack-nova | 17:09 | |
*** sfinucan_ has joined #openstack-nova | 17:11 | |
*** paul-carlton2 has joined #openstack-nova | 17:14 | |
*** sfinucan has quit IRC | 17:14 | |
cfriesen_ | mriedem: sorry, was sidetracked. for Bug 1552777 it was on the same host | 17:15 |
openstack | bug 1552777 in OpenStack Compute (nova) "resizing from flavor with swap to one without swap puts instance into Error status" [Medium,Confirmed] https://launchpad.net/bugs/1552777 | 17:15 |
*** apoorvad has joined #openstack-nova | 17:16 | |
*** Marga_ has quit IRC | 17:16 | |
*** Marga_ has joined #openstack-nova | 17:17 | |
*** nic has joined #openstack-nova | 17:17 | |
openstackgerrit | Timofey Durakov proposed openstack/nova: force_live_migration remove redundant check https://review.openstack.org/287997 | 17:19 |
*** salv-orlando has joined #openstack-nova | 17:20 | |
*** salv-orl_ has quit IRC | 17:22 | |
cfriesen_ | mriedem: There's a similar issue if we resize from having ephemeral disk to no ephemeral disk. Should I add the info in the existing bug or make a new bug? | 17:22 |
cfriesen_ | Failure mode is slightly different | 17:23 |
*** sahid has quit IRC | 17:24 | |
*** ljxiash has quit IRC | 17:24 | |
*** andymaier has quit IRC | 17:25 | |
*** jaosorior has joined #openstack-nova | 17:26 | |
*** ndipanov has quit IRC | 17:28 | |
*** ljxiash has joined #openstack-nova | 17:29 | |
cfriesen_ | So normally we don't allow resizing to a smaller disk. What should be the behaviour if we try to resize from "some ephemeral disk" to "no ephemeral disk"? Currently it accepts the resize action and then silently fails with "ResizeError: Resize error: Unable to resize disk down." in the compute logs. | 17:30 |
cfriesen_ | I suppose an argument could be made that ephemeral disks are supposed to preserve data for the life of the instance, so it makes sense to not allow resizing to an instance with smaller ephemeral disks. | 17:32 |
*** pushkaru has quit IRC | 17:33 | |
*** jistr has quit IRC | 17:33 | |
*** ccarmack has left #openstack-nova | 17:33 | |
*** jaypipes has joined #openstack-nova | 17:36 | |
*** sc68cal has joined #openstack-nova | 17:38 | |
*** matrohon has quit IRC | 17:39 | |
*** vilobhmm11 has joined #openstack-nova | 17:45 | |
*** vilobhmm111 has joined #openstack-nova | 17:46 | |
PaulMurray | tdurakov, just looking at your patch https://review.openstack.org/287997 - that is a tough one | 17:47 |
*** claudiub|2 has quit IRC | 17:47 | |
tdurakov | PaulMurray, hey, why? | 17:48 |
PaulMurray | tdurakov, just looking hold on | 17:48 |
tdurakov | PaulMurray, sure | 17:48 |
PaulMurray | just checking my facts before I open my mouth | 17:48 |
*** vilobhmm11 has quit IRC | 17:49 | |
*** salv-orl_ has joined #openstack-nova | 17:49 | |
*** paul-carlton2 has quit IRC | 17:50 | |
*** sc68cal_ has joined #openstack-nova | 17:51 | |
PaulMurray | tdurakov, actually, its probably not so bad in this particular case | 17:51 |
tdurakov | PaulMurray, patch? | 17:52 |
alaski | cfriesen_: xen hit a similar issue https://bugs.launchpad.net/nova/+bug/1488635 | 17:52 |
openstack | Launchpad bug 1488635 in OpenStack Compute (nova) "xen: resize assumes an ephemeral was migrated if the new flavor has one" [Undecided,Fix released] - Assigned to Andrew Laski (alaski) | 17:52 |
PaulMurray | tdurakov, the argument that there is a small window between api and compute doesn't really hold in general | 17:52 |
*** salv-orlando has quit IRC | 17:52 | |
alaski | cfriesen_: although xen also allows resize down so the considerations are different | 17:52 |
PaulMurray | tdurakov, when there is a disruption in messaging the api is usually still accessible to the user because its a rest interface | 17:53 |
PaulMurray | tdurakov, so a user request can get queued up but not deliverd | 17:53 |
alaski | cfriesen_: but I would argue that any decrease in disk size of root/swap/ephemeral is a resize down | 17:53 |
PaulMurray | tdurakov, when that happens users tend to start repeating what they did | 17:53 |
*** baoli has quit IRC | 17:53 | |
*** jdurgin1 has quit IRC | 17:53 | |
PaulMurray | tdurakov, so you end up with several requests all queued up | 17:53 |
*** sc68cal has quit IRC | 17:54 | |
PaulMurray | tdurakov, if that happens in this case its likely that the instance will have moved to another host so the pause will be | 17:54 |
PaulMurray | just fail | 17:55 |
*** e0ne has joined #openstack-nova | 17:55 | |
*** sfinucan_ has quit IRC | 17:55 | |
PaulMurray | but its those kind of situations that we need to think about | 17:55 |
*** baoli has joined #openstack-nova | 17:55 | |
*** danpb has quit IRC | 17:55 | |
*** ijw has joined #openstack-nova | 17:57 | |
PaulMurray | tdurakov, I think doing a pause on an instance that is not migrating will actually pause the instance | 17:58 |
*** ducttape_ has quit IRC | 17:58 | |
*** rk4n has joined #openstack-nova | 17:58 | |
PaulMurray | tdurakov, so checking there is a migration that we think is still running will make the window between the check and the action really small | 17:58 |
PaulMurray | tdurakov, but you are right, if we go through messaging to do the check it doesn't really help | 17:59 |
rtheis | osc functional tests for creating an aggregate are now failing within the past couple hours...400 error from the nova api. osc is creating an aggregate without specifying an AZ. Is this an expected behavior change? I suspect it may be related to https://github.com/openstack/nova/commit/e5f3d9a5ebbb4a9238e5d5057c8565c9c0fa0fa6 | 18:00 |
cfriesen_ | alaski: I'm not sure about swap, since it's expected to get wiped on reboot and there's no filesystem involved. Data on the other two is supposed to be preserved. | 18:00 |
rtheis | I'm bringing up a recent devstack environment now to verify | 18:00 |
*** haomaiwa_ has quit IRC | 18:01 | |
tdurakov | PaulMurray, this window is already small, if we got messaging problem check on compute small will also be useless, another thing in this patch - target host is not current instance host, but migration.source_host instead | 18:01 |
tdurakov | s/small/side | 18:01 |
*** haomaiwa_ has joined #openstack-nova | 18:01 | |
tdurakov | so we will not end up with pause instance that is already migrated | 18:01 |
*** vilobhmm111 has quit IRC | 18:02 | |
*** salv-orl_ has quit IRC | 18:02 | |
*** ijw has quit IRC | 18:02 | |
alaski | cfriesen_: that's fair. my understanding is that resize down is disabled due to the complexity of ensuring it will succeed since a user may have filled the filesystem. swap would be a special case there, but it is a reduction in resources available. *shrug* | 18:02 |
*** salv-orl_ has joined #openstack-nova | 18:03 | |
tdurakov | PaulMurray, if this change is ok, I'd make it to abort live-migration too | 18:03 |
*** baoli has quit IRC | 18:04 | |
*** paul-carlton2 has joined #openstack-nova | 18:04 | |
PaulMurray | tdurakov, I have been called out in the night to fix these problems - they do happen | 18:04 |
PaulMurray | tdurakov, that's why I'm thinking so hard about it | 18:04 |
*** baoli has joined #openstack-nova | 18:04 | |
PaulMurray | tdurakov, is there any kind of check we can do on the host ? | 18:05 |
*** mariusv_ has quit IRC | 18:05 | |
*** mariusv has quit IRC | 18:05 | |
PaulMurray | tdurakov, in this case I am not so bothered because pause is not destructive | 18:06 |
johnthetubaguy | PaulMurray: tdurakov: I think checking the libvirt live-migrate job is still running would help, at least it would make me feel less worried about things | 18:06 |
*** mariusv has joined #openstack-nova | 18:06 | |
*** mariusv has quit IRC | 18:06 | |
*** mariusv has joined #openstack-nova | 18:06 | |
johnthetubaguy | PaulMurray: also depends if unpause works when we get confused about the state of the VM | 18:06 |
* johnthetubaguy has to go sort out food, etc | 18:07 | |
PaulMurray | johnthetubaguy, better check the food is still in date before and after putting in oven | 18:07 |
PaulMurray | johnthetubaguy, I'm cautious like that :) | 18:07 |
*** cdent has quit IRC | 18:08 | |
tdurakov | PaulMurray, let me explain my thinking on this: check on compute still not guarantee that live-migration is in progress, also db request is go over rpc, so the period remains the same in both cases, whether we check this on compute side or not. johnthetubaguy, will add check in libvirt, this will make things more secure here | 18:09 |
*** lucasagomes is now known as lucas-afk | 18:09 | |
*** penick has joined #openstack-nova | 18:09 | |
*** jlanoux has quit IRC | 18:10 | |
PaulMurray | tdurakov, I agree with you about rpc being just as bad in general. | 18:10 |
mriedem | cfriesen_: same bug or different, doesn't matter to me much | 18:10 |
*** mriedem has quit IRC | 18:10 | |
*** salv-orl_ has quit IRC | 18:10 | |
PaulMurray | tdurakov, some kind of local check would be good if possible | 18:11 |
PaulMurray | tdurakov, not sure what check you can do though - do yo uknow? | 18:11 |
*** ctrath has quit IRC | 18:11 | |
*** salv-orlando has joined #openstack-nova | 18:11 | |
*** ctrath has joined #openstack-nova | 18:11 | |
tdurakov | PaulMurray, domainjobinfo should be none if no job here | 18:13 |
tdurakov | checking this | 18:13 |
PaulMurray | tdurakov, what would be cool is if we had a local stash of what was being done to the instance | 18:13 |
*** anush has quit IRC | 18:14 | |
PaulMurray | tdurakov, but wouldn't bother with that right now | 18:14 |
tdurakov | PaulMurray, local stash is in progress:) | 18:14 |
*** anush has joined #openstack-nova | 18:14 | |
PaulMurray | tdurakov, now if libvirt was that clever we would be sorted | 18:16 |
*** zenoway has joined #openstack-nova | 18:16 | |
*** permalac has quit IRC | 18:17 | |
*** shangxdy has quit IRC | 18:18 | |
*** ihrachys has quit IRC | 18:18 | |
*** ducttape_ has joined #openstack-nova | 18:19 | |
*** Marga_ has quit IRC | 18:19 | |
*** dedery has quit IRC | 18:20 | |
*** mriedem has joined #openstack-nova | 18:20 | |
*** dedery has joined #openstack-nova | 18:21 | |
*** zenoway has quit IRC | 18:21 | |
*** mtanino__ has quit IRC | 18:22 | |
*** anush has quit IRC | 18:23 | |
mriedem | doffm: https://github.com/openstack/gerrit-dash-creator | 18:23 |
mriedem | doffm: https://goo.gl/1vTS0Z | 18:24 |
tdurakov | PaulMurray, http://xsnippet.org/361491/ | 18:25 |
*** apoorvad has quit IRC | 18:25 | |
tdurakov | first element is virDomainJobType if it is 0 VIR_DOMAIN_JOB_NONE=0 No job is active | 18:26 |
tdurakov | so yes it's possible | 18:26 |
PaulMurray | tdurakov, that is good. I think danpb said before that if a job is running its not possibly to tell what kind of job it is? | 18:27 |
PaulMurray | tdurakov, but that does seem to help | 18:28 |
tdurakov | PaulMurray, yes, this is true, but we check it on nova side in api migration status | 18:28 |
tdurakov | PaulMurray, I need to make several tests, after that I'll resubmit this patch | 18:29 |
PaulMurray | tdurakov, I think that's as good as we can do at the moment. it is likely that if a job is not running the migration has failed or the vm has gone | 18:29 |
*** lpetrut has quit IRC | 18:29 | |
*** takedakn has joined #openstack-nova | 18:30 | |
tdurakov | PaulMurray, there will be another error if vm has gone:-p but we could also properly handle that | 18:31 |
*** jaosorior has quit IRC | 18:31 | |
*** geekinutah has joined #openstack-nova | 18:31 | |
*** cdent has joined #openstack-nova | 18:31 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: Compute API: omit disk/container formats when creating images of snapshots https://review.openstack.org/226069 | 18:32 |
PaulMurray | tdurakov, yeah, not worried about that case at all. Maybe catch an exception and log it like the abort code does. | 18:32 |
tdurakov | PaulMurray, need to go now, you are in review list, so you will be notified about new version. | 18:32 |
*** sdague has joined #openstack-nova | 18:32 | |
PaulMurray | tdurakov, thanks | 18:33 |
*** Guest7786 has quit IRC | 18:33 | |
*** PaulMurray has quit IRC | 18:35 | |
*** Sam-I-Am has left #openstack-nova | 18:36 | |
*** vishwanathj_zzz is now known as vishwanathj | 18:37 | |
*** gjayavelu has joined #openstack-nova | 18:37 | |
*** daemontool_ has quit IRC | 18:38 | |
*** kiran-r has quit IRC | 18:38 | |
openstackgerrit | Moshe Levi proposed openstack/nova-specs: Add scheduling with NIC capabilities https://review.openstack.org/286073 | 18:39 |
*** baoli has quit IRC | 18:41 | |
*** takedakn has quit IRC | 18:42 | |
mriedem | eliqiao: are you planning on working on this docs bug? https://bugs.launchpad.net/nova/+bug/1551782 | 18:43 |
openstack | Launchpad bug 1551782 in OpenStack Compute (nova) " API: Improve os-migrateLive input parameters" [Undecided,New] | 18:43 |
*** baoli has joined #openstack-nova | 18:43 | |
*** READ10 has quit IRC | 18:43 | |
*** baoli has quit IRC | 18:43 | |
*** takedakn has joined #openstack-nova | 18:44 | |
*** rcernin has joined #openstack-nova | 18:44 | |
mriedem | cfriesen_: would you be able to triage this? https://bugs.launchpad.net/nova/+bug/1551504 | 18:44 |
openstack | Launchpad bug 1551504 in OpenStack Compute (nova) "I/O (PCIe) Based NUMA Scheduling can't really achieve pci numa binding in some cases." [Undecided,New] - Assigned to jinquanni(ZTE) (ni-jinquan) | 18:44 |
mriedem | it's pci/numa stuff but kilo | 18:44 |
*** takedakn has quit IRC | 18:45 | |
mriedem | cfriesen_: basically wondering if it's a known issue that is fixed now | 18:45 |
*** mgoddard_ has joined #openstack-nova | 18:49 | |
*** geekinutah has quit IRC | 18:50 | |
*** jichen has joined #openstack-nova | 18:51 | |
*** zenoway has joined #openstack-nova | 18:51 | |
*** ijw has joined #openstack-nova | 18:51 | |
*** paul-carlton2 has quit IRC | 18:51 | |
mriedem | Vek: are you around? | 18:52 |
*** mgoddard has quit IRC | 18:52 | |
*** gjayavelu has quit IRC | 18:52 | |
*** vilobhmm11 has joined #openstack-nova | 18:55 | |
*** ijw has quit IRC | 18:56 | |
*** mgoddard_ has quit IRC | 18:56 | |
mriedem | andreykurilin: or you? i have a novaclient question | 18:56 |
*** mvk has quit IRC | 18:57 | |
*** e0ne has quit IRC | 18:57 | |
*** pushkaru has joined #openstack-nova | 18:58 | |
*** neophy has joined #openstack-nova | 18:58 | |
*** baoli has joined #openstack-nova | 18:58 | |
*** coreywright has quit IRC | 19:00 | |
*** ljxiash has quit IRC | 19:00 | |
*** haomaiwa_ has quit IRC | 19:01 | |
*** ljxiash has joined #openstack-nova | 19:01 | |
*** 20WAAG6RV has joined #openstack-nova | 19:01 | |
*** Marga_ has joined #openstack-nova | 19:01 | |
*** baoli has quit IRC | 19:02 | |
*** andymaier has joined #openstack-nova | 19:02 | |
*** baoli has joined #openstack-nova | 19:03 | |
*** Marga_ has quit IRC | 19:03 | |
*** apoorvad has joined #openstack-nova | 19:04 | |
*** burt has joined #openstack-nova | 19:04 | |
*** ljxiash has quit IRC | 19:06 | |
*** ducttape_ has quit IRC | 19:06 | |
*** Marga_ has joined #openstack-nova | 19:06 | |
*** ducttape_ has joined #openstack-nova | 19:07 | |
*** ijw has joined #openstack-nova | 19:07 | |
*** mgoddard has joined #openstack-nova | 19:07 | |
*** ociuhandu has quit IRC | 19:09 | |
*** coreywright has joined #openstack-nova | 19:14 | |
dansmith | zomg.. yet another service_version rebase | 19:14 |
*** baoli has quit IRC | 19:16 | |
*** sheel has quit IRC | 19:17 | |
mriedem | you love it | 19:19 |
dansmith | I really don't | 19:19 |
mriedem | so: nova interface-attach --net-id 1 --net-id 2 test only stores --net-id 2, i didn't realize argparse did that w/o warning | 19:20 |
mriedem | and you have to provide your own custom action to puke if an option is specified more than once and you don't want it to be a list | 19:20 |
mriedem | crazy | 19:20 |
mriedem | dansmith: you should love this https://review.openstack.org/#/c/284945/ | 19:21 |
mriedem | intel pci test was ok | 19:21 |
openstackgerrit | Thomas Bachman proposed openstack/nova: VMware: Use Port Group and Key in binding details https://review.openstack.org/288076 | 19:26 |
openstackgerrit | Sean Dague proposed openstack/nova: deprecate manager class options https://review.openstack.org/287867 | 19:26 |
openstackgerrit | Sean Dague proposed openstack/nova: deprecate ``volume_api_class`` https://review.openstack.org/288077 | 19:26 |
dansmith | mriedem: yep cool | 19:26 |
sdague | mriedem: I think the managers deprecation probably should be a dedicated patch, because they are like options | 19:26 |
sdague | I pulled out the volume_api on in a follow on | 19:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: DNM: see where instance.pci_devices is getting lazy-loaded https://review.openstack.org/284839 | 19:26 |
*** gjayavelu has joined #openstack-nova | 19:28 | |
mriedem | vif['details'] is another big fun bag of unversioned dict mess | 19:29 |
mriedem | allowing vendor neutron drivers to put whatever things they want in the vif object and nova can special case a bunch of it | 19:29 |
mriedem | maybe os-vif will hide those details | 19:29 |
openstackgerrit | Dan Smith proposed openstack/nova: Migrate compute node resource information to Inventory objects https://review.openstack.org/279313 | 19:31 |
openstackgerrit | Dan Smith proposed openstack/nova: Add Service.get_minimum_version_multi() for multiple binaries https://review.openstack.org/279312 | 19:31 |
dansmith | jaypipes: ^ rebased on your latest | 19:31 |
dansmith | I assume the cells fail was spurious | 19:31 |
mriedem | sdague: what about things like network_api_class? | 19:31 |
sdague | mriedem: yeh, I'm working through some of the others. | 19:32 |
sdague | honestly, there are some awesome ones | 19:33 |
mriedem | horribly awesome? | 19:33 |
mriedem | sdague: some typos in the release note on the manager one | 19:33 |
openstackgerrit | Tim Pownall proposed openstack/nova: Updating while loop for tar extraction in utils.py. https://review.openstack.org/287426 | 19:33 |
mriedem | otherwise seems ok | 19:33 |
sdague | https://github.com/openstack/nova/blob/02f0e4694f660a329e214dc759495d6d9aa5c190/nova/api/metadata/base.py#L51 | 19:34 |
dansmith | ouch | 19:34 |
sdague | what's even better is that also has a config option for the json file it loads | 19:35 |
sdague | which is even more confusing | 19:35 |
sdague | because if the answer is "here is an arbitrary json file" | 19:35 |
*** ducttape_ has quit IRC | 19:35 | |
sdague | then why did you put an arbitrary loaders in front | 19:35 |
*** pwnall has joined #openstack-nova | 19:36 | |
*** Sukhdev has joined #openstack-nova | 19:38 | |
sdague | dansmith: https://github.com/openstack/nova/blob/02f0e4694f660a329e214dc759495d6d9aa5c190/nova/db/base.py#L23 is also neat | 19:39 |
mriedem | yeah i've seen that | 19:39 |
dansmith | sdague: rax uses that one today | 19:39 |
dansmith | AFAIK | 19:39 |
mriedem | i also noticed that oslo.db makes the backend optional. as in not using sqlalchemy | 19:39 |
sdague | dansmith: really? | 19:40 |
openstackgerrit | Mark Doffman proposed openstack/nova-specs: Move aggregates tables to the API database https://review.openstack.org/288084 | 19:40 |
dansmith | sdague: yeah they override our db_api with one that replaces a few operations with very mysql-specific higher-performance ones | 19:40 |
mriedem | sdague: https://review.openstack.org/#/c/243822/ | 19:40 |
mriedem | ^ that | 19:40 |
mriedem | https://review.openstack.org/#/c/243822/1/nova/db/api.py | 19:40 |
dansmith | yeah | 19:40 |
*** ihrachys has joined #openstack-nova | 19:41 | |
sdague | ok, well I'll leave that one alone, but it still seems odd | 19:41 |
dansmith | I'm not saying you should | 19:41 |
dansmith | I'm just explaining why I know about it :) | 19:42 |
sdague | heh | 19:42 |
alaski | there are a few queries that weren't working well at scale | 19:42 |
alaski | but addressing that within the sqlalchemy db api is a better path forward than having a separate db api | 19:42 |
sdague | alaski: ok, so does that mean you are pro marking this as deprecated? | 19:43 |
alaski | sdague: marking as deprecated, yes. not pro removal yet. | 19:43 |
*** lpetrut has joined #openstack-nova | 19:43 | |
mriedem | alaski doesn't have to care anymore :) | 19:43 |
mriedem | let them eat cake | 19:43 |
sdague | mriedem: are there other ones like the volume / network api? | 19:44 |
*** ljxiash has joined #openstack-nova | 19:44 | |
sdague | I think those are probably good as a patch together | 19:44 |
*** efoley has quit IRC | 19:45 | |
mriedem | key manager | 19:45 |
sdague | ok, I will look | 19:45 |
mriedem | idk, whatever i pointed out in the patch earlier :) | 19:45 |
mriedem | quota driver | 19:46 |
sdague | also - nova/compute/resource_tracker.py: default='nova.compute.stats.Stats', | 19:46 |
sdague | 19:46 | |
mriedem | quota driver is really a choice | 19:46 |
mriedem | but the choies are db or noop | 19:46 |
sdague | mriedem: right, quota driver is different | 19:46 |
dims | sdague : mriedem : how about cloudpipe stuff? | 19:46 |
dims | sdague : mriedem : how about nuking cloudpipe stuff? | 19:46 |
mriedem | sdague: you want to really piss everyone off? | 19:46 |
*** sgordon has quit IRC | 19:46 | |
mriedem | compute_driver | 19:46 |
mriedem | :) | 19:46 |
sdague | dims: it's in our api, unfortunately | 19:46 |
dims | ouch | 19:46 |
sdague | mriedem: I thought we had stand names for that? | 19:46 |
sdague | mriedem: keymanager is different | 19:47 |
sdague | because we have multiple implementations in tree | 19:48 |
sdague | that needs to be a selector as well | 19:48 |
mriedem | sdague: yeah, i'm talking about choices | 19:48 |
mriedem | but enforcing choices where it was open ended before is in the same vein | 19:49 |
*** gjayavelu has quit IRC | 19:49 | |
*** ljxiash has quit IRC | 19:49 | |
mriedem | sdague: compute_driver is not limited to choices today | 19:49 |
mriedem | however, if you did that, it would go against what we've allowed in the hv_type module | 19:50 |
mriedem | for out of tree tihngs | 19:50 |
sdague | well, if we did that it would have to allow for hv_type based loading, possibly off of entry points | 19:52 |
sdague | the point isn't to break things we have told people to use | 19:52 |
sdague | it's about marking a bunch of stuff we really don't want people using | 19:52 |
*** browne has quit IRC | 19:53 | |
*** boris-42 has joined #openstack-nova | 19:55 | |
*** rlrossit has quit IRC | 19:57 | |
*** angdraug has quit IRC | 19:57 | |
*** ihrachys_ has joined #openstack-nova | 19:58 | |
openstackgerrit | Tim Pownall proposed openstack/nova: Updating while loop for tar extraction in utils.py. https://review.openstack.org/287426 | 19:58 |
*** rlrossit_ has joined #openstack-nova | 19:59 | |
*** ducttape_ has joined #openstack-nova | 20:00 | |
*** 20WAAG6RV has quit IRC | 20:01 | |
*** ihrachys has quit IRC | 20:01 | |
*** haomaiwang has joined #openstack-nova | 20:01 | |
*** anush has joined #openstack-nova | 20:02 | |
mriedem | i might be misunderstanding this bug, but it doesn't seem valid https://bugs.launchpad.net/nova/+bug/1550919 | 20:04 |
openstack | Launchpad bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,Triaged] - Assigned to leehom (feli5) | 20:04 |
mriedem | rebuild/evacuate recreates a new instance, and if the driver spawn fails, i'm not sure why it shouldn't delete the disk | 20:04 |
*** Sukhdev has quit IRC | 20:07 | |
openstackgerrit | Tim Pownall proposed openstack/nova: Updating while loop for tar extraction in utils.py. https://review.openstack.org/287426 | 20:08 |
mriedem | only issue i can see is shared storage | 20:08 |
sdague | mriedem: typo fixes coming up. Also, I pulled network_manager from the list, because that's really a selector | 20:13 |
openstackgerrit | Sean Dague proposed openstack/nova: deprecate ``volume_api_class`` and ``network_api_class`` https://review.openstack.org/288077 | 20:13 |
openstackgerrit | Sean Dague proposed openstack/nova: deprecate manager class options https://review.openstack.org/287867 | 20:13 |
openstackgerrit | Sean Dague proposed openstack/nova: Deprecate db_driver config option https://review.openstack.org/288106 | 20:13 |
openstackgerrit | Sean Dague proposed openstack/nova: Deprecate the ``vendordata_driver`` config option. https://review.openstack.org/288107 | 20:13 |
sdague | which, speaking of deprecations. When are we going to deprecate nova-network again? | 20:13 |
dansmith | I don't want to deprecate it until we are through the cellsv2 woods | 20:14 |
dansmith | and certainly not until it's default in devstack (for more than five minutes), IMHO | 20:14 |
*** yamahata has quit IRC | 20:14 | |
sdague | ok | 20:15 |
mriedem | yeah... | 20:15 |
mriedem | i think you can add get-me-a-network in that boat | 20:15 |
sdague | those are acceptable data points | 20:15 |
dansmith | mriedem: agree | 20:15 |
*** breton has quit IRC | 20:16 | |
*** penick has quit IRC | 20:16 | |
sdague | anyway, I think those are some of the easy deprecate bits that get us a ways forward in stop loading arbitrary code in every module | 20:17 |
*** rlrossit_ has quit IRC | 20:19 | |
*** eharney has quit IRC | 20:19 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: don't destroy shared disks during rebuild if vif plug fails https://review.openstack.org/288109 | 20:19 |
mriedem | is ^ crazy? cfriesen_: nic: ^? | 20:19 |
*** ctrath has quit IRC | 20:22 | |
*** matrohon has joined #openstack-nova | 20:24 | |
*** ctrath has joined #openstack-nova | 20:24 | |
mriedem | zigo: per https://bugs.launchpad.net/nova/+bug/1552897 | 20:26 |
openstack | Launchpad bug 1552897 in OpenStack Compute (nova) "Unit test failure when buildnig debian package for Mitaka b3" [Undecided,New] | 20:26 |
mriedem | zigo: which version of oslo.cache are you using? | 20:26 |
*** dave-mccowan has quit IRC | 20:26 | |
*** cdelatte has joined #openstack-nova | 20:27 | |
*** penick has joined #openstack-nova | 20:28 | |
*** atuvenie has joined #openstack-nova | 20:29 | |
*** claudiub|2 has joined #openstack-nova | 20:30 | |
mriedem | zigo: actually, which version of dogpile.cache | 20:31 |
zigo | mriedem: 0.5.4 | 20:32 |
mriedem | zigo: ok, i think we're using 0.5.7 | 20:32 |
mriedem | this is just asking for trouble https://github.com/openstack/nova/blob/master/nova/tests/unit/test_cache.py#L87 | 20:32 |
*** e0ne has joined #openstack-nova | 20:32 | |
zigo | mriedem: Right, that's what's in the upper-constraints.txt | 20:32 |
zigo | I can try upgrading the package ... | 20:32 |
mriedem | dims: shouldn't these tests live in oslo.cache? https://github.com/openstack/nova/blob/master/nova/tests/unit/test_cache.py | 20:33 |
zigo | mriedem: Thanks for the hint, I'll report back in the bug report whatever the result is. | 20:33 |
mriedem | zigo: np | 20:34 |
mriedem | dims: nova mocking an oslo.cache dependency for unit tests seems bound to fail | 20:34 |
openstackgerrit | Thomas Bachman proposed openstack/nova: VMware: Use Port Group and Key in binding details https://review.openstack.org/288076 | 20:36 |
mriedem | zigo: which oslo.cache version? | 20:37 |
zigo | mriedem: The latest from upper-constaints (whatever it is). | 20:37 |
mriedem | 1.4.0? | 20:37 |
mriedem | https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L207 | 20:37 |
zigo | mriedem: I always upgrade fully oslo + clients before attempting to package a new OpenStack release. | 20:37 |
mriedem | ok | 20:37 |
zigo | Wont be easy, the latest dogpile.cache has unit test failures ... :( | 20:38 |
*** yassine__ has quit IRC | 20:43 | |
mriedem | i tried looking at the nova, oslo.cache and dogpile.cache code to figure out what is now passing wrap kwarg or not, and i'm lost | 20:43 |
mriedem | oslo.cache has quite a bit of magic goin gon | 20:43 |
*** irenab has joined #openstack-nova | 20:44 | |
zigo | Oh, I remember, these tests aren't deterministic. | 20:44 |
zigo | (the dogpile.cache ones...) | 20:44 |
zigo | Rebuilding nova... | 20:45 |
zigo | (and going to sleep... ;P) | 20:45 |
*** sdague has quit IRC | 20:45 | |
openstackgerrit | Jay Faulkner proposed openstack/nova: Ironic: Clean up if configdrive build fails https://review.openstack.org/287520 | 20:45 |
*** ZZelle_ has joined #openstack-nova | 20:46 | |
*** alex_xu has quit IRC | 20:46 | |
openstackgerrit | Jay Faulkner proposed openstack/nova: Ironic: Clean up if configdrive build fails https://review.openstack.org/287520 | 20:46 |
dims | mriedem : we have a new cache_utils.py, this tests if that works right | 20:47 |
openstackgerrit | Tim Pownall proposed openstack/nova: Updating while loop for tar extraction in utils.py. https://review.openstack.org/287426 | 20:47 |
*** ihrachys has joined #openstack-nova | 20:47 | |
*** rk4n has quit IRC | 20:48 | |
*** alex_xu has joined #openstack-nova | 20:49 | |
*** doug-fis_ has joined #openstack-nova | 20:49 | |
*** doug-fish has quit IRC | 20:50 | |
mriedem | gah, sean sneaky exit again | 20:50 |
*** ihrachys_ has quit IRC | 20:50 | |
mriedem | alex_xu: fyi https://bugs.launchpad.net/nova/+bug/1552888 | 20:50 |
openstack | Launchpad bug 1552888 in OpenStack Compute (nova) "ERROR (BadRequest): expected string or buffer" [Undecided,New] - Assigned to jichenjc (jichenjc) | 20:50 |
mriedem | sounds like a regression | 20:50 |
*** anush has quit IRC | 20:51 | |
*** dave-mccowan has joined #openstack-nova | 20:51 | |
*** dave-mcc_ has joined #openstack-nova | 20:54 | |
*** e0ne has quit IRC | 20:54 | |
*** salv-orlando has quit IRC | 20:55 | |
*** salv-orlando has joined #openstack-nova | 20:55 | |
*** gyee has joined #openstack-nova | 20:55 | |
*** tdurakov_ has joined #openstack-nova | 20:57 | |
*** dave-mccowan has quit IRC | 20:57 | |
*** rlrossit_ has joined #openstack-nova | 20:59 | |
*** tdurakov_ has quit IRC | 20:59 | |
*** Sukhdev has joined #openstack-nova | 20:59 | |
*** lykinsbd has quit IRC | 20:59 | |
*** e0ne has joined #openstack-nova | 21:00 | |
tonyb | isn't there a meeting now? | 21:00 |
claudiub|2 | mriedem: hello. wanna hear a horror story? the tempest test LiveBlockMigrationTestJSON.test_iscsi_volume hasn't been run since before October 2015. :) because of this, there's this boo-boo: https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_live_migration.py#L164 | 21:00 |
*** haomaiwang has quit IRC | 21:01 | |
*** markusz has joined #openstack-nova | 21:01 | |
*** Sukhdev has quit IRC | 21:01 | |
mriedem | doesn't pass volume_backed huh | 21:01 |
*** haomaiwa_ has joined #openstack-nova | 21:01 | |
claudiub|2 | yup | 21:01 |
claudiub|2 | fun times. :) | 21:01 |
*** rfolco has quit IRC | 21:01 | |
bauzas | tonyb: right, I think so | 21:02 |
claudiub|2 | fix: https://review.openstack.org/#/c/287837/ | 21:02 |
claudiub|2 | although, I haven't run the test yet.. | 21:02 |
openstackgerrit | jichenjc proposed openstack/nova: Revert "Generate better validation error message when using name regexes" https://review.openstack.org/288125 | 21:02 |
*** pwnall has quit IRC | 21:02 | |
*** baoli has joined #openstack-nova | 21:06 | |
*** raildo is now known as raildo-afk | 21:08 | |
*** tdurakov_ has joined #openstack-nova | 21:08 | |
*** baoli_ has joined #openstack-nova | 21:08 | |
tonyb | jaypipes: I think it's me in the dunce chair | 21:08 |
*** tdurakov_ has quit IRC | 21:10 | |
* tbachman occupies said chair regularly | 21:10 | |
jaypipes | edleafe-: I've rechecked that stats patch yet again. | 21:10 |
*** tdurakov_ has joined #openstack-nova | 21:11 | |
edleafe- | jaypipes: did you see http://logs.openstack.org/31/286931/6/check/gate-tempest-dsvm-postgres-full/9afed6a/logs/screen-n-api.txt.gz?level=WARNING | 21:11 |
* Vek tries to avoid the dunce chair by saying little--"better to say nothing and be thought a fool than to open your mouth and remove all doubt" ;) | 21:11 | |
*** baoli has quit IRC | 21:11 | |
tdurakov_ | folks, who could clarify what VIR_DOMAIN_JOB_LAST means for libvirt? | 21:11 |
jaypipes | edleafe-: yeah but that is no relation to our patch. | 21:12 |
tbachman | Vek: I hate ambiguity ;_ | 21:12 |
tbachman | ;) | 21:12 |
edleafe- | jaypipes: didn't think so | 21:12 |
Vek | ;) | 21:12 |
* tbachman ducks out | 21:12 | |
jaypipes | edleafe-: and it's expected AFAICT | 21:12 |
jaypipes | tdurakov: it means that libvirt intends to retire. | 21:13 |
edleafe- | jaypipes: expected? why? | 21:13 |
*** rk4n has joined #openstack-nova | 21:13 | |
*** rk4n has quit IRC | 21:14 | |
tdurakov_ | jaypipes, lol | 21:14 |
jaypipes | edleafe-: meaning I believe it's a negative test where the test is deliberately tossing a bad IP at something. | 21:14 |
*** smatzek has quit IRC | 21:14 | |
jaypipes | tdurakov: glad you got my joke :) | 21:14 |
edleafe- | jaypipes: but it was a postgres syntax error, because it wasn't an inet value. Shouldn't the test then catch that? | 21:15 |
tdurakov_ | jaypipes, https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainJobType - no info, some secret type | 21:15 |
tdurakov_ | jaypipes, hm, I think, i got the point for that kind of last flags | 21:16 |
jaypipes | kashyap: can you help tdurakov? | 21:16 |
jaypipes | edleafe-: not sure :( | 21:16 |
tdurakov_ | kashyap, jaypipes is it just current max(last) value for this type? | 21:16 |
jaypipes | edleafe-: all I know is we didn't touch that code even remotely :) | 21:16 |
edleafe- | jaypipes: I mean, then it should throw that error on every patch | 21:17 |
edleafe- | jaypipes: of course | 21:17 |
*** e0ne has quit IRC | 21:17 | |
*** e0ne has joined #openstack-nova | 21:17 | |
*** penick has quit IRC | 21:18 | |
mriedem | thomasem: are you around? | 21:20 |
mriedem | your agenda thing is happening in #openstack-meeting | 21:20 |
thomasem | oh, sorry | 21:20 |
thomasem | prod issue, be in the meeting | 21:20 |
*** angdraug has joined #openstack-nova | 21:21 | |
openstackgerrit | melanie witt proposed openstack/nova: Add a cell and host mapping utility to nova-manage https://review.openstack.org/270565 | 21:22 |
auggy | hey so i might be interested in the I18n CP liason thing since i have some experience working with internationalization/unicode... buuuut | 21:22 |
auggy | i'm still pretty new to nova | 21:22 |
*** tdurakov_ has quit IRC | 21:22 | |
auggy | so if anyone else was going to do it then i wouldn't mind helping out | 21:22 |
openstackgerrit | Timofey Durakov proposed openstack/nova: force_live_migration remove redundant check https://review.openstack.org/287997 | 21:23 |
auggy | i used to work with this guy - https://twitter.com/novapatch/ | 21:23 |
auggy | he's an internationalization nut | 21:23 |
auggy | and when i landed my first nova patch i totally tweeted his name cuz lol | 21:24 |
JayF | I have a reproducable problem in a new fedora 23 VM of being unable to run nova tests; it's hanging while running the same test. I'm running a small patch rebased on master far away from the test that it's freezing on. Everything works fine, it freezes, and after 5m I ^C and see what test is frozen. Logs are here (scroll to bottom) https://gist.github.com/jayofdoom/96dbcfe26e32c889eb8c | 21:24 |
*** ihrachys_ has joined #openstack-nova | 21:24 | |
*** ihrachys has quit IRC | 21:25 | |
*** e0ne has quit IRC | 21:28 | |
*** ihrachys has joined #openstack-nova | 21:28 | |
*** tdurakov_ has joined #openstack-nova | 21:28 | |
*** ihrachys_ has quit IRC | 21:29 | |
*** tdurakov_ has quit IRC | 21:29 | |
*** tdurakov_ has joined #openstack-nova | 21:30 | |
*** mingyu has joined #openstack-nova | 21:31 | |
*** dave-mcc_ has quit IRC | 21:31 | |
*** ljxiash has joined #openstack-nova | 21:33 | |
*** tdurakov_ has quit IRC | 21:33 | |
*** tdurakov_ has joined #openstack-nova | 21:34 | |
tdurakov_ | eliqiao, hi, are you around? | 21:35 |
*** e0ne has joined #openstack-nova | 21:37 | |
*** mingyu has quit IRC | 21:37 | |
*** ljxiash has quit IRC | 21:37 | |
*** dave-mccowan has joined #openstack-nova | 21:40 | |
*** penick has joined #openstack-nova | 21:42 | |
*** ihrachys_ has joined #openstack-nova | 21:43 | |
*** mgoddard has quit IRC | 21:44 | |
*** edleafe- is now known as edleafe | 21:44 | |
*** rcernin has quit IRC | 21:45 | |
*** ihrachys has quit IRC | 21:45 | |
*** rk4n has joined #openstack-nova | 21:46 | |
*** rk4n has quit IRC | 21:46 | |
openstackgerrit | Jay Faulkner proposed openstack/nova: Ironic: Clean up if configdrive build fails https://review.openstack.org/287520 | 21:48 |
*** atuvenie has quit IRC | 21:49 | |
mtreinish | mriedem, tonyb: so my pings in the meeting. https://review.openstack.org/288126 is what you were looking for | 21:50 |
mtreinish | the plan is to use the plugin for all the gating ceph jobs in the short term | 21:50 |
*** sc68cal_ is now known as sc68cal | 21:52 | |
*** penick has quit IRC | 21:53 | |
*** rk4n has joined #openstack-nova | 21:53 | |
*** whenry_ has quit IRC | 21:53 | |
*** whenry_ has joined #openstack-nova | 21:54 | |
*** thorst_ has quit IRC | 21:54 | |
*** fawadkhaliq has joined #openstack-nova | 21:55 | |
*** edtubill has joined #openstack-nova | 21:56 | |
*** rk4n has quit IRC | 21:57 | |
*** tdurakov_ has quit IRC | 22:00 | |
*** thorst_ has joined #openstack-nova | 22:01 | |
*** haomaiwa_ has quit IRC | 22:01 | |
*** tdurakov_ has joined #openstack-nova | 22:01 | |
*** haomaiwang has joined #openstack-nova | 22:01 | |
*** gjayavelu has joined #openstack-nova | 22:02 | |
*** ihrachys has joined #openstack-nova | 22:02 | |
*** gjayavelu has quit IRC | 22:03 | |
*** vladikr has quit IRC | 22:04 | |
mriedem | auggy: it's yours | 22:05 |
*** ihrachys_ has quit IRC | 22:05 | |
*** thorst_ has quit IRC | 22:05 | |
*** ctrath has quit IRC | 22:05 | |
*** amotoki has joined #openstack-nova | 22:05 | |
auggy | mriedem: ok cool, i'll reply to that email | 22:05 |
*** thorst_ has joined #openstack-nova | 22:06 | |
auggy | hehe i would like to work on i18n on Nova because i used to work on i18n with Nova | 22:06 |
*** yarkot_ has joined #openstack-nova | 22:06 | |
*** ihrachys_ has joined #openstack-nova | 22:07 | |
*** rtheis has quit IRC | 22:07 | |
*** MVenesio_ has quit IRC | 22:09 | |
*** tdurakov_ has quit IRC | 22:09 | |
openstackgerrit | jichenjc proposed openstack/nova: Catch ProjectQuotaNotFound when do update_limit https://review.openstack.org/288153 | 22:09 |
*** ihrachys has quit IRC | 22:10 | |
*** tdurakov_ has joined #openstack-nova | 22:10 | |
*** thorst_ has quit IRC | 22:11 | |
*** lpetrut has quit IRC | 22:11 | |
mriedem | markus_z: thanks for herding the cats in https://review.openstack.org/#/c/287685 | 22:11 |
*** vilobhmm11 has quit IRC | 22:11 | |
*** vilobhmm11 has joined #openstack-nova | 22:11 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Extend FakeCryptoCertificate.cert_not_valid_after to 2 hours https://review.openstack.org/288154 | 22:12 |
*** ociuhandu has joined #openstack-nova | 22:14 | |
*** tdurakov_ has quit IRC | 22:15 | |
openstackgerrit | jichenjc proposed openstack/nova: Catch ProjectQuotaNotFound when do update_limit https://review.openstack.org/288153 | 22:16 |
mriedem | ha | 22:16 |
mriedem | dansmith: sound familiar? https://bugs.launchpad.net/nova/+bug/1543010 | 22:16 |
openstack | Launchpad bug 1543010 in OpenStack Compute (nova) "Nova clears DB if ESX nova-compute node restarted" [Undecided,New] - Assigned to Radoslav Gerganov (rgerganov) | 22:16 |
*** yarkot_ has quit IRC | 22:16 | |
*** ctrath has joined #openstack-nova | 22:16 | |
*** ctrath has quit IRC | 22:17 | |
*** jichen has quit IRC | 22:19 | |
*** knikolla has quit IRC | 22:20 | |
*** ctrath has joined #openstack-nova | 22:20 | |
*** claudiub|2 has quit IRC | 22:21 | |
*** baoli_ has quit IRC | 22:22 | |
*** thorst_ has joined #openstack-nova | 22:23 | |
dansmith | mriedem: nice | 22:25 |
*** thorst_ has quit IRC | 22:27 | |
*** edmondsw has quit IRC | 22:28 | |
*** penick has joined #openstack-nova | 22:29 | |
*** aysyd has quit IRC | 22:29 | |
tonyb | mtreinish: Thanks. | 22:30 |
tonyb | mtreinish: I have a *new* plugin that I want to add to git.o.o and was hoping we could plce it undert QA and seek buy-in from the devstack team to be core on it. | 22:30 |
*** diana_clarke1 has joined #openstack-nova | 22:31 | |
*** dimtruck is now known as zz_dimtruck | 22:31 | |
mtreinish | tonyb: we'll see, tbh I don't like the idea of owning a plugin in qa. If it's that important why doesn't it just live in devstack proper | 22:31 |
mtreinish | which is likely going to be a summit session (for the ceph case) | 22:31 |
mtreinish | tonyb: https://etherpad.openstack.org/p/newton-qa-summit-topics | 22:32 |
tonyb | mtreinish: Hmm okay. I have to admit I was hoping to land this before that. | 22:33 |
mriedem | knife in the back of boot from volume https://bugs.launchpad.net/nova/+bug/1523447 | 22:33 |
openstack | Launchpad bug 1523447 in OpenStack Compute (nova) "When multiple cinder backend enabled ,Cinder picks up first enabled backend during boot from image(create new volume) " [Undecided,Won't fix] | 22:33 |
mtreinish | tonyb: there's no barrier to adding the plugin right now | 22:33 |
openstackgerrit | melanie witt proposed openstack/nova: Use db connection from RequestContext during queries https://review.openstack.org/272329 | 22:33 |
*** pwnall has joined #openstack-nova | 22:34 | |
mtreinish | tonyb: the only discussion is whether we make it a qa project (which tbh I'd probably be fine with as long as it's small, self-contained, and provides real value) | 22:34 |
*** matrohon has quit IRC | 22:35 | |
*** e0ne has quit IRC | 22:35 | |
mtreinish | I just don't want qa to be known as the land for devstack-plugins to live under openstack governance | 22:35 |
*** andymaier_ has joined #openstack-nova | 22:35 | |
*** Sukhdev has joined #openstack-nova | 22:35 | |
mtreinish | because there are a lot of them out there, and most don't make sense under qa | 22:35 |
tonyb | mtreinish: okay. I'll worry about final resting place later | 22:35 |
*** efried has quit IRC | 22:36 | |
*** thorst_ has joined #openstack-nova | 22:36 | |
tonyb | mtreinish: I'll try to add it and see what happens. Right now I'll just make markusz and I core and then try to convince you and sdague to mentor me in to making it better. | 22:37 |
mtreinish | tonyb: you're talking about the libvirt from source thing? | 22:37 |
tonyb | mtreinish: libvirt from packages, but yeah | 22:37 |
tonyb | I've split the work into 2 parts, the devstack plugin just points at a non-distro repo to pull the libvirt/qemu packge | 22:38 |
*** andymaier has quit IRC | 22:38 | |
*** zenoway has quit IRC | 22:38 | |
*** rlrossit_ has quit IRC | 22:38 | |
mtreinish | tonyb: from packages? where's the fun in that? needs -O3 -funroll-loops :) | 22:39 |
tonyb | rigth now it's just UCA but my plan is to build them for Ubuntu myself | 22:39 |
tonyb | but I really want that seperate from the gate process | 22:39 |
mtreinish | tonyb: yeah, that makes sense | 22:39 |
mtreinish | that's just wasting cycles when we can just do it periodically in a package repo | 22:39 |
tonyb | it's generic enough that neutron could use it for ovs or dpdk | 22:40 |
tonyb | mtreinish: and adds several failure vectors that we dont want. | 22:40 |
mtreinish | yep | 22:40 |
tonyb | mtreinish: https://github.com/tbreeds/tbreeds-devstack-plugin-additional-pkg-repos is what I have so far | 22:41 |
*** mriedem has quit IRC | 22:41 | |
*** busterswt has quit IRC | 22:42 | |
*** ctrath has quit IRC | 22:42 | |
*** baoli has joined #openstack-nova | 22:43 | |
*** sc68cal has quit IRC | 22:45 | |
*** ducttape_ has quit IRC | 22:47 | |
*** penick has quit IRC | 22:52 | |
*** zenoway has joined #openstack-nova | 22:53 | |
*** ihrachys has joined #openstack-nova | 22:56 | |
*** zenoway has quit IRC | 22:57 | |
*** edtubill has quit IRC | 22:58 | |
*** ihrachys has quit IRC | 22:58 | |
*** markusz has quit IRC | 22:58 | |
*** ijw has quit IRC | 22:59 | |
*** ihrachys_ has quit IRC | 22:59 | |
*** ijw has joined #openstack-nova | 23:00 | |
*** haomaiwang has quit IRC | 23:01 | |
*** haomaiwang has joined #openstack-nova | 23:01 | |
*** vilobhmm11 has quit IRC | 23:04 | |
*** vilobhmm11 has joined #openstack-nova | 23:04 | |
*** vilobhmm11 has quit IRC | 23:04 | |
*** vilobhmm11 has joined #openstack-nova | 23:04 | |
*** xyang1 has quit IRC | 23:06 | |
*** xyang1 has joined #openstack-nova | 23:07 | |
*** marcusvrn_ has quit IRC | 23:07 | |
*** baoli has quit IRC | 23:08 | |
*** baoli has joined #openstack-nova | 23:09 | |
*** xyang1 has quit IRC | 23:09 | |
*** geekinutah has joined #openstack-nova | 23:10 | |
*** penick has joined #openstack-nova | 23:13 | |
*** cdelatte has quit IRC | 23:14 | |
*** cdelatte has joined #openstack-nova | 23:14 | |
*** amotoki has quit IRC | 23:16 | |
*** mdrabe has quit IRC | 23:17 | |
*** fawadkhaliq has quit IRC | 23:18 | |
*** fawadkhaliq has joined #openstack-nova | 23:18 | |
*** haplo37 has quit IRC | 23:18 | |
*** ljxiash has joined #openstack-nova | 23:20 | |
*** fawadkhaliq has quit IRC | 23:21 | |
*** fawadkhaliq has joined #openstack-nova | 23:21 | |
*** Jeffrey4l has joined #openstack-nova | 23:23 | |
*** ljxiash has quit IRC | 23:25 | |
*** baoli has quit IRC | 23:26 | |
*** ivase has quit IRC | 23:27 | |
*** baoli has joined #openstack-nova | 23:27 | |
*** Sukhdev has quit IRC | 23:27 | |
*** gjayavelu has joined #openstack-nova | 23:28 | |
*** ianw has quit IRC | 23:28 | |
*** neophy has quit IRC | 23:34 | |
*** yamahata has joined #openstack-nova | 23:35 | |
openstackgerrit | Tim Pownall proposed openstack/nova: Updating while loop for tar extraction in utils.py. https://review.openstack.org/287426 | 23:36 |
*** ivase has joined #openstack-nova | 23:36 | |
*** penick has quit IRC | 23:37 | |
*** ivase_ has joined #openstack-nova | 23:37 | |
*** haomaiwang has quit IRC | 23:40 | |
*** ivase has quit IRC | 23:41 | |
*** zz_dimtruck is now known as dimtruck | 23:41 | |
*** ivase_ has quit IRC | 23:41 | |
*** knikolla has joined #openstack-nova | 23:44 | |
*** ijw has quit IRC | 23:45 | |
*** ijw has joined #openstack-nova | 23:46 | |
*** Jeffrey4l has quit IRC | 23:47 | |
*** baoli has quit IRC | 23:48 | |
*** diana_clarke1 has quit IRC | 23:49 | |
*** cdent has quit IRC | 23:50 | |
*** ijw has quit IRC | 23:50 | |
*** knikolla has quit IRC | 23:51 | |
*** salv-orl_ has joined #openstack-nova | 23:52 | |
*** cdelatte has quit IRC | 23:52 | |
*** salv-orlando has quit IRC | 23:55 | |
*** mrkz has quit IRC | 23:56 | |
*** ianw has joined #openstack-nova | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!