*** jdurgin1 has joined #openstack-nova | 00:02 | |
*** amrith is now known as _amrith_ | 00:04 | |
*** takedakn has quit IRC | 00:04 | |
*** takedakn has joined #openstack-nova | 00:05 | |
*** diogogmt has quit IRC | 00:05 | |
*** andrewbogott has joined #openstack-nova | 00:12 | |
*** dims_ has quit IRC | 00:13 | |
*** dims_ has joined #openstack-nova | 00:14 | |
*** tsekiyam_ has joined #openstack-nova | 00:15 | |
*** Daisy has joined #openstack-nova | 00:17 | |
*** tsekiyama has quit IRC | 00:18 | |
*** darrenc_afk is now known as darrenc | 00:18 | |
*** dims_ has quit IRC | 00:19 | |
dansmith | FYI to anyone around, we really need to get this trivial patch in: | 00:19 |
---|---|---|
dansmith | https://review.openstack.org/#/c/231686/ | 00:19 |
*** tsekiyam_ has quit IRC | 00:19 | |
*** manishg has joined #openstack-nova | 00:19 | |
*** Daisy has quit IRC | 00:19 | |
*** mtanino has quit IRC | 00:19 | |
dansmith | it's blocking an ovo patch landing to fix an upgrade issue, which prevents us from landing any new object versions without breaking things | 00:19 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/231238 | 00:19 |
*** Daisy has joined #openstack-nova | 00:20 | |
*** Daisy has quit IRC | 00:20 | |
*** kgalanov has joined #openstack-nova | 00:20 | |
*** Daisy has joined #openstack-nova | 00:20 | |
*** tpeoples has quit IRC | 00:23 | |
*** dims_ has joined #openstack-nova | 00:25 | |
*** Daisy has quit IRC | 00:25 | |
*** takedakn1 has joined #openstack-nova | 00:25 | |
*** su_zhang has quit IRC | 00:26 | |
*** takedakn has quit IRC | 00:26 | |
*** takedakn1 is now known as takedakn | 00:26 | |
*** dims_ has quit IRC | 00:26 | |
*** dims_ has joined #openstack-nova | 00:26 | |
*** artom has quit IRC | 00:28 | |
*** zhangjn has quit IRC | 00:29 | |
*** zhangjn has joined #openstack-nova | 00:32 | |
*** takedakn has quit IRC | 00:32 | |
*** manishg has quit IRC | 00:36 | |
*** zhangjn has quit IRC | 00:37 | |
*** Daisy has joined #openstack-nova | 00:37 | |
*** IanGovett has quit IRC | 00:39 | |
*** claudiub has quit IRC | 00:40 | |
*** tpeoples has joined #openstack-nova | 00:42 | |
*** thumpba has joined #openstack-nova | 00:44 | |
*** su_zhang has joined #openstack-nova | 00:47 | |
*** manishg has joined #openstack-nova | 00:50 | |
*** angdraug has quit IRC | 00:51 | |
*** IanGovett has joined #openstack-nova | 00:53 | |
*** IanGovett has quit IRC | 00:54 | |
*** ssurana has quit IRC | 00:54 | |
*** Daisy has quit IRC | 00:55 | |
*** browne has quit IRC | 00:57 | |
*** gyee has quit IRC | 00:58 | |
*** thumpba has quit IRC | 01:01 | |
*** mudassirlatif has quit IRC | 01:04 | |
*** baoli has joined #openstack-nova | 01:05 | |
*** thumpba_ has joined #openstack-nova | 01:06 | |
*** su_zhang has quit IRC | 01:07 | |
*** vilobhmm111 has quit IRC | 01:08 | |
*** stevemar_ has quit IRC | 01:11 | |
*** jhesketh has quit IRC | 01:15 | |
*** apoorvad has quit IRC | 01:16 | |
*** jhesketh has joined #openstack-nova | 01:16 | |
*** emagana has quit IRC | 01:19 | |
*** apoorvad has joined #openstack-nova | 01:25 | |
*** markvoelker has quit IRC | 01:27 | |
*** diogogmt has joined #openstack-nova | 01:34 | |
*** armax has joined #openstack-nova | 01:37 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/231238 | 01:44 |
*** tjones has joined #openstack-nova | 01:45 | |
*** apuimedo has quit IRC | 01:50 | |
*** apuimedo has joined #openstack-nova | 01:51 | |
*** achanda has joined #openstack-nova | 01:52 | |
*** unicell has quit IRC | 01:55 | |
*** fawadkhaliq has joined #openstack-nova | 01:58 | |
*** fawadkhaliq has quit IRC | 01:59 | |
*** thumpba_ has quit IRC | 01:59 | |
*** zhangjn has joined #openstack-nova | 02:00 | |
*** stevemar_ has joined #openstack-nova | 02:00 | |
*** zhangjn has quit IRC | 02:05 | |
*** zhangjn has joined #openstack-nova | 02:07 | |
*** apoorvad has quit IRC | 02:08 | |
*** manishg has quit IRC | 02:08 | |
*** tjones has quit IRC | 02:14 | |
*** kumar has joined #openstack-nova | 02:15 | |
*** VikasC has quit IRC | 02:16 | |
*** haomaiwang has joined #openstack-nova | 02:17 | |
*** thumpba has joined #openstack-nova | 02:17 | |
*** yamahata has joined #openstack-nova | 02:17 | |
*** baoli has quit IRC | 02:18 | |
*** changbl has joined #openstack-nova | 02:19 | |
*** penick has quit IRC | 02:22 | |
*** manishg has joined #openstack-nova | 02:24 | |
*** thumpba has quit IRC | 02:28 | |
*** haomaiwang has quit IRC | 02:29 | |
*** haomaiwang has joined #openstack-nova | 02:30 | |
*** thumpba_ has joined #openstack-nova | 02:31 | |
*** browne has joined #openstack-nova | 02:31 | |
*** tpeoples has quit IRC | 02:31 | |
*** manishg has quit IRC | 02:33 | |
*** julien_ZTE has quit IRC | 02:33 | |
openstackgerrit | Diana Clarke proposed openstack/python-novaclient: Fix incorrect help for 'nova flavor-create' https://review.openstack.org/231803 | 02:34 |
*** thumpba_ has quit IRC | 02:34 | |
*** haomaiwang has quit IRC | 02:35 | |
*** thumpba_ has joined #openstack-nova | 02:37 | |
*** haomaiwa_ has joined #openstack-nova | 02:39 | |
*** mdrabe has joined #openstack-nova | 02:42 | |
*** yamahata has quit IRC | 02:42 | |
*** baoli has joined #openstack-nova | 02:45 | |
*** tpeoples has joined #openstack-nova | 02:46 | |
*** darrenc is now known as darrenc_afk | 02:53 | |
*** zzzeek has quit IRC | 02:55 | |
*** pixelbeat has quit IRC | 02:55 | |
*** deepthi has joined #openstack-nova | 02:57 | |
*** tjones has joined #openstack-nova | 02:58 | |
*** baoli has quit IRC | 02:59 | |
*** haomaiwa_ has quit IRC | 03:00 | |
*** baoli has joined #openstack-nova | 03:01 | |
*** haomaiwang has joined #openstack-nova | 03:03 | |
*** achanda has quit IRC | 03:09 | |
*** burgerk has joined #openstack-nova | 03:15 | |
*** dave-mccowan has quit IRC | 03:16 | |
*** Marga_ has quit IRC | 03:18 | |
*** markvoelker has joined #openstack-nova | 03:19 | |
*** Marga_ has joined #openstack-nova | 03:19 | |
*** links has joined #openstack-nova | 03:21 | |
*** markvoelker_ has joined #openstack-nova | 03:23 | |
*** Marga_ has quit IRC | 03:23 | |
*** markvoelker has quit IRC | 03:25 | |
*** tjones has quit IRC | 03:28 | |
*** markvoelker has joined #openstack-nova | 03:33 | |
*** markvoelker has quit IRC | 03:33 | |
*** markvoelker_ has quit IRC | 03:33 | |
*** baoli has quit IRC | 03:34 | |
*** vilobhmm11 has joined #openstack-nova | 03:38 | |
*** darrenc_afk is now known as darrenc | 03:48 | |
*** Sree has joined #openstack-nova | 03:50 | |
*** Sree has quit IRC | 03:51 | |
*** Sree has joined #openstack-nova | 03:52 | |
*** Sree has quit IRC | 03:54 | |
*** Sree has joined #openstack-nova | 03:56 | |
*** Vimal_ has joined #openstack-nova | 03:57 | |
*** Marga_ has joined #openstack-nova | 03:58 | |
*** Marga_ has quit IRC | 03:59 | |
*** Marga_ has joined #openstack-nova | 03:59 | |
*** Marga_ has quit IRC | 04:01 | |
*** Marga_ has joined #openstack-nova | 04:02 | |
* Vimal_ slaps anish around a bit with a large fishbot | 04:03 | |
*** VikasC has joined #openstack-nova | 04:03 | |
*** Marga_ has quit IRC | 04:06 | |
*** Marga_ has joined #openstack-nova | 04:06 | |
*** fawadkhaliq has joined #openstack-nova | 04:10 | |
*** fawadk has joined #openstack-nova | 04:11 | |
*** wverdugo500 has quit IRC | 04:12 | |
*** hemna has joined #openstack-nova | 04:12 | |
*** fawadkhaliq has quit IRC | 04:15 | |
*** burgerk has quit IRC | 04:16 | |
*** su_zhang has joined #openstack-nova | 04:17 | |
*** fawadkhaliq has joined #openstack-nova | 04:18 | |
*** fawadk has quit IRC | 04:19 | |
*** achanda has joined #openstack-nova | 04:20 | |
*** achanda has quit IRC | 04:20 | |
*** kumar has quit IRC | 04:25 | |
*** mdrabe has quit IRC | 04:35 | |
*** thumpba_ has quit IRC | 04:39 | |
*** csd has joined #openstack-nova | 04:39 | |
openstackgerrit | Michael Still proposed openstack/nova: Switch to oslo.cache lib https://review.openstack.org/203049 | 04:39 |
*** markvoelker has joined #openstack-nova | 04:40 | |
*** csd has quit IRC | 04:40 | |
*** Sree has quit IRC | 04:41 | |
*** fawadk has joined #openstack-nova | 04:42 | |
*** Sree has joined #openstack-nova | 04:42 | |
*** csd has joined #openstack-nova | 04:44 | |
*** thumpba has joined #openstack-nova | 04:44 | |
*** markvoelker has quit IRC | 04:45 | |
*** fawadkhaliq has quit IRC | 04:45 | |
*** deepthi has quit IRC | 04:46 | |
*** thumpba has quit IRC | 04:48 | |
*** markvoelker has joined #openstack-nova | 04:50 | |
*** agireud has joined #openstack-nova | 04:51 | |
*** dims_ has quit IRC | 04:52 | |
*** rushiagr_away is now known as rushiagr | 04:52 | |
*** zhangjn has quit IRC | 04:53 | |
*** markvoelker has quit IRC | 04:55 | |
*** Vimal_ has quit IRC | 04:59 | |
*** markvoelker has joined #openstack-nova | 05:02 | |
*** markvoelker has quit IRC | 05:03 | |
*** markvoelker has joined #openstack-nova | 05:13 | |
*** emagana has joined #openstack-nova | 05:16 | |
*** hemna has quit IRC | 05:18 | |
*** achanda has joined #openstack-nova | 05:19 | |
*** markvoelker has quit IRC | 05:21 | |
*** yassine has joined #openstack-nova | 05:23 | |
*** haomaiwang has quit IRC | 05:23 | |
*** hemna has joined #openstack-nova | 05:24 | |
*** alex_klimov has joined #openstack-nova | 05:25 | |
*** rushiagr is now known as rushiagr_away | 05:27 | |
*** csd has quit IRC | 05:28 | |
*** e0ne has joined #openstack-nova | 05:28 | |
*** markvoelker has joined #openstack-nova | 05:28 | |
*** e0ne has quit IRC | 05:30 | |
*** jamielennox is now known as jamielennox|away | 05:30 | |
*** stevemar_ has quit IRC | 05:31 | |
*** haomaiwang has joined #openstack-nova | 05:33 | |
*** e0ne has joined #openstack-nova | 05:34 | |
*** zhangjn has joined #openstack-nova | 05:36 | |
*** markvoelker has quit IRC | 05:36 | |
*** e0ne has quit IRC | 05:37 | |
*** doron_afk has joined #openstack-nova | 05:39 | |
*** csd has joined #openstack-nova | 05:40 | |
*** markvoelker has joined #openstack-nova | 05:41 | |
*** e0ne has joined #openstack-nova | 05:42 | |
*** markvoelker_ has joined #openstack-nova | 05:42 | |
*** markvoelker_ has quit IRC | 05:43 | |
*** markvoelker has quit IRC | 05:45 | |
*** su_zhang has quit IRC | 05:46 | |
*** e0ne has quit IRC | 05:49 | |
*** unicell has joined #openstack-nova | 05:50 | |
*** moshele has joined #openstack-nova | 05:51 | |
*** e0ne has joined #openstack-nova | 05:53 | |
*** e0ne has quit IRC | 05:54 | |
*** _amrith_ is now known as amrith | 05:55 | |
*** markvoelker has joined #openstack-nova | 05:58 | |
*** e0ne has joined #openstack-nova | 05:58 | |
*** csd has quit IRC | 05:59 | |
*** markvoelker_ has joined #openstack-nova | 06:00 | |
*** markvoelker has quit IRC | 06:00 | |
*** haomaiwang has quit IRC | 06:01 | |
*** kaisers has quit IRC | 06:01 | |
*** haomaiwa_ has joined #openstack-nova | 06:01 | |
*** e0ne has quit IRC | 06:02 | |
*** amrith is now known as _amrith_ | 06:04 | |
*** markvoelker_ has quit IRC | 06:05 | |
*** vilobhmm11 has quit IRC | 06:05 | |
*** nkrinner has joined #openstack-nova | 06:06 | |
*** hemna has quit IRC | 06:06 | |
*** jdurgin1 has quit IRC | 06:09 | |
*** asettle has quit IRC | 06:09 | |
*** alex_klimov has quit IRC | 06:13 | |
*** achanda has quit IRC | 06:13 | |
*** nikhil has quit IRC | 06:15 | |
*** nikhil has joined #openstack-nova | 06:16 | |
*** irenab has joined #openstack-nova | 06:16 | |
*** markvoelker has joined #openstack-nova | 06:19 | |
*** achanda has joined #openstack-nova | 06:19 | |
*** unicell1 has joined #openstack-nova | 06:20 | |
*** unicell has quit IRC | 06:20 | |
*** e0ne has joined #openstack-nova | 06:21 | |
*** jamielennox|away is now known as jamielennox | 06:21 | |
*** nikhil_k has joined #openstack-nova | 06:25 | |
*** markvoelker has quit IRC | 06:25 | |
*** kaisers has joined #openstack-nova | 06:25 | |
*** mtreinish has quit IRC | 06:26 | |
*** e0ne has quit IRC | 06:26 | |
openstackgerrit | Raga lahari proposed openstack/nova: Support for new --status parameter to "nova-manage vm list" command https://review.openstack.org/231447 | 06:26 |
*** nikhil has quit IRC | 06:28 | |
*** cfriesen has quit IRC | 06:29 | |
*** sahid has joined #openstack-nova | 06:29 | |
*** mtreinish has joined #openstack-nova | 06:31 | |
*** e0ne has joined #openstack-nova | 06:31 | |
*** emagana has quit IRC | 06:33 | |
*** rushiagr_away is now known as rushiagr | 06:33 | |
*** e0ne has quit IRC | 06:35 | |
*** alexschm has joined #openstack-nova | 06:37 | |
*** achanda has quit IRC | 06:39 | |
*** e0ne has joined #openstack-nova | 06:39 | |
*** vlaza has joined #openstack-nova | 06:40 | |
*** achanda has joined #openstack-nova | 06:42 | |
*** jcoufal has joined #openstack-nova | 06:43 | |
*** rdopiera has joined #openstack-nova | 06:46 | |
*** scheuran has joined #openstack-nova | 06:47 | |
*** markvoelker has joined #openstack-nova | 06:47 | |
*** fawadk has quit IRC | 06:48 | |
*** pm90_ has joined #openstack-nova | 06:49 | |
*** jcoufal has quit IRC | 06:49 | |
*** fawadkhaliq has joined #openstack-nova | 06:50 | |
*** fawadkhaliq has quit IRC | 06:51 | |
*** jcoufal has joined #openstack-nova | 06:52 | |
openstackgerrit | Vineet Menon proposed openstack/nova-specs: Add flavor tables to API database https://review.openstack.org/231858 | 06:52 |
*** markvoelker has quit IRC | 06:52 | |
*** rotbeard has joined #openstack-nova | 06:53 | |
*** deepthi has joined #openstack-nova | 06:54 | |
*** e0ne has quit IRC | 06:55 | |
*** matrohon has joined #openstack-nova | 06:56 | |
*** Sree has quit IRC | 07:00 | |
*** haomaiwa_ has quit IRC | 07:01 | |
*** romainh has joined #openstack-nova | 07:01 | |
*** haomaiwang has joined #openstack-nova | 07:01 | |
*** markvoelker has joined #openstack-nova | 07:02 | |
*** romainh has left #openstack-nova | 07:03 | |
*** Daisy has joined #openstack-nova | 07:05 | |
*** Daisy has quit IRC | 07:06 | |
*** markvoelker has quit IRC | 07:07 | |
*** Daisy has joined #openstack-nova | 07:07 | |
*** Daisy has quit IRC | 07:09 | |
*** Daisy has joined #openstack-nova | 07:09 | |
*** fawadkhaliq has joined #openstack-nova | 07:11 | |
*** ildikov has quit IRC | 07:14 | |
*** atuvenie has joined #openstack-nova | 07:16 | |
*** markvoelker has joined #openstack-nova | 07:16 | |
*** Daisy has quit IRC | 07:16 | |
*** armax has quit IRC | 07:17 | |
*** Daisy has joined #openstack-nova | 07:17 | |
*** pm90_ has quit IRC | 07:20 | |
*** markvoelker has quit IRC | 07:21 | |
*** Daisy has quit IRC | 07:22 | |
*** vivekd has joined #openstack-nova | 07:22 | |
*** pbrooko has joined #openstack-nova | 07:28 | |
openstackgerrit | Andrey Kurilin proposed openstack/nova: [Py34] api.openstack.test_faults.TestFaultWrapper https://review.openstack.org/231473 | 07:29 |
*** tpeoples has quit IRC | 07:29 | |
*** aloga_ has joined #openstack-nova | 07:30 | |
*** markvoelker has joined #openstack-nova | 07:31 | |
openstackgerrit | Andrey Kurilin proposed openstack/nova: [Py34] api.openstack.compute.legacy_v2.test_servers.Base64ValidationTest https://review.openstack.org/231474 | 07:31 |
*** pbrooko has quit IRC | 07:32 | |
openstackgerrit | Andrey Kurilin proposed openstack/nova: [Py34] api.openstack.compute.legacy_v2.test_servers.Base64ValidationTest https://review.openstack.org/231474 | 07:34 |
*** marzif has joined #openstack-nova | 07:34 | |
*** markvoelker has quit IRC | 07:36 | |
*** achanda has quit IRC | 07:39 | |
openstackgerrit | Raga lahari proposed openstack/nova: Support for new --status parameter to "nova-manage vm list" command https://review.openstack.org/231447 | 07:40 |
bauzas | good morning Nova | 07:41 |
*** Sree has joined #openstack-nova | 07:44 | |
*** doron_afk has quit IRC | 07:44 | |
*** jlanoux has joined #openstack-nova | 07:45 | |
*** jwcroppe_ has joined #openstack-nova | 07:45 | |
*** ociuhandu has quit IRC | 07:45 | |
*** markvoelker has joined #openstack-nova | 07:46 | |
*** doron_afk has joined #openstack-nova | 07:46 | |
*** rdopiera has quit IRC | 07:46 | |
*** rdopiera has joined #openstack-nova | 07:48 | |
*** jwcroppe has quit IRC | 07:48 | |
*** tteggel has quit IRC | 07:48 | |
*** salv-orlando has quit IRC | 07:48 | |
*** MarcF has joined #openstack-nova | 07:49 | |
*** salv-orlando has joined #openstack-nova | 07:49 | |
*** markvoelker has quit IRC | 07:50 | |
*** tteggel has joined #openstack-nova | 07:51 | |
*** tteggel has left #openstack-nova | 07:53 | |
*** lpetrut has joined #openstack-nova | 07:53 | |
*** jwcroppe_ has quit IRC | 07:57 | |
*** jwcroppe has joined #openstack-nova | 07:59 | |
*** markvoelker has joined #openstack-nova | 08:00 | |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwang has joined #openstack-nova | 08:01 | |
*** marzif has quit IRC | 08:02 | |
*** markvoelker has quit IRC | 08:05 | |
*** e0ne has joined #openstack-nova | 08:10 | |
*** MarcF has quit IRC | 08:12 | |
*** pbrooko has joined #openstack-nova | 08:12 | |
*** browne has quit IRC | 08:14 | |
*** markvoelker has joined #openstack-nova | 08:15 | |
*** claudiub has joined #openstack-nova | 08:15 | |
*** doron_afk has quit IRC | 08:16 | |
*** markus_z has joined #openstack-nova | 08:19 | |
*** alex_klimov has joined #openstack-nova | 08:19 | |
*** markvoelker has quit IRC | 08:19 | |
*** fawadkhaliq has quit IRC | 08:21 | |
*** fawadkhaliq has joined #openstack-nova | 08:21 | |
pilgrimstack1 | bauzas: Do you think Nova is in CEST timezone? :p | 08:24 |
bauzas | pilgrimstack1: I'm gentle :p | 08:24 |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Add soft deleting instance actions and events https://review.openstack.org/231887 | 08:24 |
*** Sree has quit IRC | 08:25 | |
*** Sree has joined #openstack-nova | 08:27 | |
*** lucas-dinner has joined #openstack-nova | 08:27 | |
*** lucasagomes has joined #openstack-nova | 08:27 | |
*** doron_afk has joined #openstack-nova | 08:28 | |
*** markvoelker has joined #openstack-nova | 08:29 | |
*** e0ne has quit IRC | 08:30 | |
*** e0ne has joined #openstack-nova | 08:32 | |
*** markvoelker has quit IRC | 08:34 | |
*** jistr has joined #openstack-nova | 08:36 | |
*** markvoelker has joined #openstack-nova | 08:36 | |
*** atuvenie has quit IRC | 08:37 | |
*** danpb has joined #openstack-nova | 08:37 | |
*** Daisy has joined #openstack-nova | 08:37 | |
*** links has quit IRC | 08:38 | |
*** achanda has joined #openstack-nova | 08:39 | |
*** markvoelker has quit IRC | 08:41 | |
*** Daisy has quit IRC | 08:42 | |
*** Daisy has joined #openstack-nova | 08:43 | |
*** markvoelker has joined #openstack-nova | 08:44 | |
*** achanda has quit IRC | 08:46 | |
*** Murali has joined #openstack-nova | 08:46 | |
*** Daisy has quit IRC | 08:47 | |
*** fawadkhaliq has quit IRC | 08:48 | |
*** fawadkhaliq has joined #openstack-nova | 08:48 | |
*** Sree has quit IRC | 08:50 | |
*** gszasz has joined #openstack-nova | 08:50 | |
claudiub | johnthetubaguy: hi. the hyper-v uefi secure boot spec, the one that you previously had a +2 on, needs your +2 again. One minor change added: https://review.openstack.org/#/c/190997/ Thanks! | 08:50 |
*** markvoelker has quit IRC | 08:51 | |
*** rubasov has joined #openstack-nova | 08:51 | |
*** markvoelker has joined #openstack-nova | 08:51 | |
*** Daisy has joined #openstack-nova | 08:52 | |
johnthetubaguy | claudiub: sweet, thanks for letting me know | 08:53 |
*** subscope has joined #openstack-nova | 08:53 | |
*** aix has joined #openstack-nova | 08:53 | |
*** Daisy has quit IRC | 08:55 | |
*** Sree has joined #openstack-nova | 08:55 | |
*** Sree has quit IRC | 08:55 | |
*** markvoelker has quit IRC | 08:56 | |
*** moshele has quit IRC | 08:58 | |
*** nikhil_k has quit IRC | 09:00 | |
*** nikhil_k has joined #openstack-nova | 09:00 | |
*** haomaiwang has quit IRC | 09:01 | |
*** Sree has joined #openstack-nova | 09:01 | |
*** 77CAAYMV4 has joined #openstack-nova | 09:01 | |
*** markvoelker has joined #openstack-nova | 09:01 | |
*** rushiagr is now known as rushiagr_away | 09:06 | |
*** markvoelker has quit IRC | 09:06 | |
*** deepthi has quit IRC | 09:08 | |
*** aloga_ has quit IRC | 09:10 | |
*** ildikov has joined #openstack-nova | 09:11 | |
*** tbh has joined #openstack-nova | 09:14 | |
*** tbh has quit IRC | 09:15 | |
*** tbh has joined #openstack-nova | 09:15 | |
*** markvoelker has joined #openstack-nova | 09:16 | |
openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Moves the Hyper-V NUMA instance spec to Mitaka https://review.openstack.org/231417 | 09:18 |
*** deepthi has joined #openstack-nova | 09:20 | |
*** zhangjn has quit IRC | 09:20 | |
*** markvoelker has quit IRC | 09:20 | |
*** andrearo1 is now known as andrearosa | 09:21 | |
*** lucas-dinner has quit IRC | 09:23 | |
*** links has joined #openstack-nova | 09:25 | |
*** markvoelker has joined #openstack-nova | 09:27 | |
*** deepthi has quit IRC | 09:29 | |
*** markvoelker has quit IRC | 09:31 | |
*** tbh has quit IRC | 09:34 | |
*** moshele has joined #openstack-nova | 09:34 | |
*** atuvenie has joined #openstack-nova | 09:35 | |
*** nikhil_k_ has joined #openstack-nova | 09:36 | |
*** markvoelker has joined #openstack-nova | 09:37 | |
jwcroppe | johnthetubaguy: if you get a moment, would you mind reviewing a fix for a metric bug? https://review.openstack.org/#/c/219153/ Jay's +2'd ... just need one more set of eyes. | 09:38 |
*** tbh has joined #openstack-nova | 09:39 | |
*** romainh has joined #openstack-nova | 09:40 | |
*** nikhil_k has quit IRC | 09:40 | |
bauzas | jwcroppe: still in Rennes or night owl ? | 09:41 |
jwcroppe | bauzas: night owl... who just realized he needs to go to bed! :) | 09:42 |
bauzas | :) | 09:42 |
*** deepthi has joined #openstack-nova | 09:42 | |
jwcroppe | bauzas: lost track of time, again. | 09:42 |
*** markvoelker has quit IRC | 09:42 | |
*** VikasC has quit IRC | 09:42 | |
jwcroppe | bauzas: talk to you when I get up ... in an hour. ;-) | 09:42 |
openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Remove shared storage flag in evacuate API https://review.openstack.org/183033 | 09:43 |
openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Add soft policies for server-group feature https://review.openstack.org/212504 | 09:43 |
*** achanda has joined #openstack-nova | 09:43 | |
*** achanda has quit IRC | 09:46 | |
*** achanda has joined #openstack-nova | 09:47 | |
*** VikasC has joined #openstack-nova | 09:47 | |
*** subscope has quit IRC | 09:49 | |
*** pbrooko has quit IRC | 09:50 | |
*** achanda has quit IRC | 09:52 | |
*** markvoelker has joined #openstack-nova | 09:52 | |
*** kaisers has quit IRC | 09:57 | |
*** markvoelker has quit IRC | 09:57 | |
johnthetubaguy | claudiub: I see a WIP -1 on this: https://review.openstack.org/#/c/231418/2 is there something you need to change on that? | 09:58 |
*** tbh has quit IRC | 09:59 | |
*** kaisers has joined #openstack-nova | 09:59 | |
*** 77CAAYMV4 has quit IRC | 10:01 | |
claudiub | johnthetubaguy: needs a small update in the "Data model impact", adding a new field for ImageMetaProps | 10:01 |
*** haomaiwa_ has joined #openstack-nova | 10:01 | |
claudiub | johnthetubaguy: will update it today. | 10:03 |
johnthetubaguy | claudiub: ah, gotcha, cool | 10:03 |
johnthetubaguy | claudiub: honestly, I keep forget that should have object changes listed, I am used to that just being DB schema changes | 10:04 |
johnthetubaguy | not totally sure why we require that, thinking about it... | 10:04 |
openstackgerrit | Roman Bogorodskiy proposed openstack/nova-specs: User-controlled SR-IOV ports allocation https://review.openstack.org/182242 | 10:04 |
*** e0ne has quit IRC | 10:06 | |
*** romainh has left #openstack-nova | 10:06 | |
*** markvoelker has joined #openstack-nova | 10:06 | |
*** kaisers has quit IRC | 10:06 | |
claudiub | johnthetubaguy: well, updating an object will increment its version, making it incompatible with older versions of the objects. Other than that, dunno. Thanks for re-approving the specs. :) | 10:07 |
johnthetubaguy | claudiub: no problem, it just feels like those details are best covered in code reviews really | 10:08 |
*** zzelle has joined #openstack-nova | 10:08 | |
*** zzelle is now known as Guest31367 | 10:09 | |
*** e0ne has joined #openstack-nova | 10:09 | |
*** kaisers has joined #openstack-nova | 10:11 | |
*** markvoelker has quit IRC | 10:11 | |
*** Guest31367 has quit IRC | 10:13 | |
*** zzelle_ has joined #openstack-nova | 10:15 | |
*** zzelle_ is now known as ZZll | 10:16 | |
*** ZZll is now known as ZZelle_ | 10:16 | |
*** yuntongjin has joined #openstack-nova | 10:18 | |
*** Sree has quit IRC | 10:20 | |
*** Sree has joined #openstack-nova | 10:21 | |
*** markvoelker has joined #openstack-nova | 10:21 | |
openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: update to min required version to 0.10.2 https://review.openstack.org/231917 | 10:22 |
openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Remove service management from nova-manage https://review.openstack.org/231918 | 10:23 |
*** Sree has quit IRC | 10:25 | |
*** markvoelker has quit IRC | 10:26 | |
openstackgerrit | Raga lahari proposed openstack/nova: Support for new --status parameter to "nova-manage vm list" command https://review.openstack.org/231447 | 10:26 |
*** subscope has joined #openstack-nova | 10:26 | |
*** Daisy has joined #openstack-nova | 10:27 | |
*** pbrooko has joined #openstack-nova | 10:27 | |
*** Daisy has quit IRC | 10:31 | |
*** markvoelker has joined #openstack-nova | 10:35 | |
*** mwagner_ has quit IRC | 10:38 | |
*** markvoelker has quit IRC | 10:40 | |
*** mdavidson has joined #openstack-nova | 10:40 | |
*** Daisy has joined #openstack-nova | 10:41 | |
*** kaisers has quit IRC | 10:42 | |
*** pbrooko has quit IRC | 10:44 | |
*** IanGovett has joined #openstack-nova | 10:44 | |
*** markvoelker has joined #openstack-nova | 10:45 | |
*** kaisers has joined #openstack-nova | 10:45 | |
*** Daisy has quit IRC | 10:46 | |
openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: update to min required version to 0.10.2 https://review.openstack.org/231917 | 10:48 |
openstackgerrit | lyanchih proposed openstack/nova: libvirt: serial console ports count upper limit needs to be checked https://review.openstack.org/207373 | 10:49 |
*** salv-orlando has quit IRC | 10:49 | |
*** romainh has joined #openstack-nova | 10:51 | |
*** zhangjn has joined #openstack-nova | 10:52 | |
*** dave-mccowan has joined #openstack-nova | 10:54 | |
*** markvoelker has quit IRC | 10:54 | |
*** pilgrimstack1 has quit IRC | 10:55 | |
openstackgerrit | Alexis Lee proposed openstack/nova: Make add_metrics_to_list side-effect free https://review.openstack.org/230918 | 10:57 |
*** pixelbeat has joined #openstack-nova | 10:58 | |
*** smatzek has joined #openstack-nova | 10:59 | |
*** markvoelker has joined #openstack-nova | 11:00 | |
*** obondarev has quit IRC | 11:00 | |
*** haomaiwa_ has quit IRC | 11:01 | |
*** haomaiwang has joined #openstack-nova | 11:01 | |
*** njirap has joined #openstack-nova | 11:01 | |
*** pilgrimstack has joined #openstack-nova | 11:02 | |
*** garyk has joined #openstack-nova | 11:04 | |
*** gzhai has quit IRC | 11:04 | |
*** markvoelker has quit IRC | 11:05 | |
*** gzhai has joined #openstack-nova | 11:05 | |
openstackgerrit | sonu proposed openstack/nova: Replaces xrange() with range() to provide py 2/3 compatibility https://review.openstack.org/205307 | 11:07 |
*** markvoelker has joined #openstack-nova | 11:14 | |
*** DRoBeR has joined #openstack-nova | 11:15 | |
*** zhangjn has quit IRC | 11:15 | |
johnthetubaguy | claudiub: I meant to ask, you know that python34 fix that we merged, could you get someone to follow up with expanding the list of unit tests that pass with py34 after that change? | 11:16 |
*** pbrooko has joined #openstack-nova | 11:17 | |
*** pbrooko has quit IRC | 11:17 | |
*** markvoelker has quit IRC | 11:19 | |
*** jamielennox is now known as jamielennox|away | 11:19 | |
*** claudiub|2 has joined #openstack-nova | 11:28 | |
*** aix has quit IRC | 11:29 | |
*** claudiub has quit IRC | 11:29 | |
*** markvoelker has joined #openstack-nova | 11:29 | |
*** takedakn has joined #openstack-nova | 11:30 | |
*** pbrooko has joined #openstack-nova | 11:30 | |
*** ndipanov has joined #openstack-nova | 11:31 | |
openstackgerrit | John Garbutt proposed openstack/nova: HyperV: Add IOUtils class for asynchronous IO operations https://review.openstack.org/184035 | 11:33 |
*** markvoelker has quit IRC | 11:34 | |
*** lucasagomes is now known as lucas-hungry | 11:36 | |
*** subscope has quit IRC | 11:37 | |
*** baoli has joined #openstack-nova | 11:38 | |
openstackgerrit | Alexey I. Froloff proposed openstack/nova: Fix more inconsistency between Nova-Net and Neutron https://review.openstack.org/226274 | 11:39 |
*** haomaiwang has quit IRC | 11:39 | |
*** romainh has left #openstack-nova | 11:39 | |
*** baoli_ has joined #openstack-nova | 11:39 | |
*** claudiub has joined #openstack-nova | 11:40 | |
*** ildikov has quit IRC | 11:41 | |
*** yuntongjin has quit IRC | 11:42 | |
*** claudiub|2 has quit IRC | 11:43 | |
*** baoli has quit IRC | 11:43 | |
*** _amrith_ is now known as amrith | 11:43 | |
*** markvoelker has joined #openstack-nova | 11:44 | |
*** jcoufal has quit IRC | 11:45 | |
*** takedakn1 has joined #openstack-nova | 11:46 | |
*** takedakn has quit IRC | 11:46 | |
*** takedakn1 is now known as takedakn | 11:46 | |
*** Daisy has joined #openstack-nova | 11:48 | |
*** markvoelker has quit IRC | 11:49 | |
*** markvoelker has joined #openstack-nova | 11:50 | |
*** Daisy has quit IRC | 11:51 | |
*** thorst1 has joined #openstack-nova | 11:51 | |
*** sambetts has quit IRC | 11:52 | |
*** pbrooko has quit IRC | 11:52 | |
*** sambetts has joined #openstack-nova | 11:54 | |
*** aysyd has joined #openstack-nova | 11:55 | |
*** pbrooko has joined #openstack-nova | 11:56 | |
*** markvoelker has quit IRC | 11:56 | |
*** Sree has joined #openstack-nova | 11:56 | |
*** jcoufal has joined #openstack-nova | 11:57 | |
claudiub | johnthetubaguy: i have no idea on how to expand the list of unit tests for py34. there seems to be a tests-py3.txt file that should be changed, but dunno where it is. It isn't in nova or project-config. | 11:59 |
*** ociuhandu has joined #openstack-nova | 11:59 | |
johnthetubaguy | claudiub: its here: https://github.com/openstack/nova/blob/master/tests-py3.txt | 12:00 |
claudiub | johnthetubaguy: damn, I'm blind. I'll send a commit soon. | 12:02 |
*** rfolco has joined #openstack-nova | 12:02 | |
*** subscope has joined #openstack-nova | 12:05 | |
*** Marga_ has quit IRC | 12:05 | |
*** Marga_ has joined #openstack-nova | 12:05 | |
*** markvoelker has joined #openstack-nova | 12:07 | |
*** david-ly_ has joined #openstack-nova | 12:07 | |
openstackgerrit | Flavio Percoco proposed openstack/nova-specs: (follow up) Nova Image v1 -> v2 https://review.openstack.org/231357 | 12:07 |
openstackgerrit | jichenjc proposed openstack/nova: Remove soft_deleted filter in image cache manage https://review.openstack.org/217118 | 12:08 |
*** jichen has joined #openstack-nova | 12:09 | |
*** david-lyle has quit IRC | 12:09 | |
*** david-ly_ is now known as david-lyle | 12:09 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: Expands python34 unit tests list https://review.openstack.org/231948 | 12:10 |
*** mwagner_ has joined #openstack-nova | 12:11 | |
*** markvoelker has quit IRC | 12:12 | |
*** thumpba has joined #openstack-nova | 12:14 | |
*** fawadkhaliq has quit IRC | 12:15 | |
*** Daisy has joined #openstack-nova | 12:15 | |
*** rfolco has quit IRC | 12:16 | |
*** rfolco has joined #openstack-nova | 12:17 | |
*** obondarev has joined #openstack-nova | 12:17 | |
*** burgerk has joined #openstack-nova | 12:20 | |
*** markvoelker has joined #openstack-nova | 12:21 | |
*** thumpba has quit IRC | 12:22 | |
*** su_zhang has joined #openstack-nova | 12:23 | |
*** russellb has quit IRC | 12:24 | |
openstackgerrit | Alexis Lee proposed openstack/nova: Make add_metrics_to_list side-effect free https://review.openstack.org/230918 | 12:24 |
johnthetubaguy | claudiub: no worries, that keeps moving, was in tox.ini | 12:25 |
*** ccesario_ has joined #openstack-nova | 12:26 | |
*** markvoelker has quit IRC | 12:26 | |
BobBall | danpb: Regarding your comment on https://review.openstack.org/#/c/229351/2/specs/mitaka/approved/xenapi-add-support-for-vgpu.rst - not sure I understand... With using a key in extra_specs we can filter the hosts to find ones that have available vGPUs and use the same extra_specs definition to add the vGPU in Novas code. Why does this approach require the admin to create 2 sets of flavors? | 12:26 |
bauzas | oh johnthetubaguy was really quick with https://review.openstack.org/#/c/231542/ :) | 12:26 |
*** vivekd has quit IRC | 12:27 | |
*** Daisy has quit IRC | 12:27 | |
bauzas | I was about to rebase for removing the prio section :) | 12:27 |
*** Daisy has joined #openstack-nova | 12:27 | |
johnthetubaguy | bauzas: ah, yeah, we should probably do that in bulk, I just started ignoring it for the fast approves | 12:27 |
danpb | BobBall: if we take m1.small and set vgpu=foo, then m1.small will always result in creation of a vGPU, so if you want vms without vGPU, then you'd need to create m1.small and m1.small-with-vgpu | 12:27 |
BobBall | Ah, I see what you mean, yes. | 12:28 |
danpb | with my suggestion, we'd only ever need m1.small in the common case - no need for separate m1.small-with-vgpu | 12:28 |
bauzas | johnthetubaguy: yup, I was about removing that section for my spec, I could just also change the alrady approved ones | 12:28 |
danpb | of course an admin might want both, if they want to restrirt access to and/or charge extra for vGPU support, but its nice not to force use of separate flavours | 12:28 |
*** zhangjn has joined #openstack-nova | 12:28 | |
johnthetubaguy | danpb: most features work like that right now though, that have resource implications? I think the flavor rework jaypipes is thinking about was to fix that issue more generally | 12:29 |
danpb | johnthetubaguy: its not entirely clear-cut - some do, some dont | 12:30 |
BobBall | Which ones don't? I'd just like to have a look at how they are implemented | 12:30 |
*** zhangjn has quit IRC | 12:31 | |
johnthetubaguy | danpb: thats true I guess, I wish that stuff that involved consuming finite resources were only available by flavors really | 12:31 |
danpb | or at least, much of the recent stuff we've done has allow image meta to override what flavour has done | 12:31 |
*** markvoelker has joined #openstack-nova | 12:31 | |
danpb | eg with NUMA, the image meta can specify the number of NUMA nodes | 12:31 |
danpb | provided it is *not* specified against the flavour | 12:31 |
*** Daisy has quit IRC | 12:32 | |
danpb | the admin can set it against the flavour to prevent tenants choosing their own numa settings in image meta | 12:32 |
bauzas | johnthetubaguy: meh, totally missed https://github.com/openstack/nova-specs/commit/99bf65f023fe0e48ce49fbfa48c17024181c2531 | 12:32 |
danpb | but to prevent an explosion of number of flavours private-cloud would typically just let user choose whatever they want against image | 12:32 |
danpb | guess we could do a similar thing for vGPU - allow a choice of enabling it on the flavour, or on the image meta, with flavour taking priority if set | 12:33 |
*** Daisy has joined #openstack-nova | 12:33 | |
BobBall | I'm not sure how the NUMA nodes work - are the number of numa nodes a finite resource? | 12:34 |
*** njirap has quit IRC | 12:34 | |
johnthetubaguy | I need to finish off that patch I started to get agreement on how we use all these leavers | 12:34 |
johnthetubaguy | its got so crazily messy now | 12:34 |
johnthetubaguy | I get there are use cases for both though | 12:35 |
*** Daisy has quit IRC | 12:36 | |
*** markvoelker has quit IRC | 12:36 | |
danpb | BobBall: yeah, to some degree they are | 12:36 |
johnthetubaguy | bauzas: its one where I think we need to see the code really | 12:36 |
*** njirap has joined #openstack-nova | 12:36 | |
bauzas | johnthetubaguy: agreed, I'm a bit torn about that one | 12:36 |
*** doron_afk has quit IRC | 12:36 | |
*** Daisy has joined #openstack-nova | 12:36 | |
bauzas | johnthetubaguy: because it means that it could be updating the DB | 12:36 |
danpb | BobBall: eg if you request a guest has 8 NUMA nodes, then you can only run it on hosts which have 8 physical NUMA nodes, so that restricts the pool of acceptable hosts for the schedular | 12:36 |
bauzas | johnthetubaguy: or if not, just having like an in-memory lock | 12:37 |
*** doron_afk has joined #openstack-nova | 12:37 | |
bauzas | which means it would still have some scale-out problems | 12:37 |
johnthetubaguy | bauzas: I assume its a regular oslo lock, we have banned any more DB locks | 12:37 |
*** edmondsw has joined #openstack-nova | 12:37 | |
bauzas | johnthetubaguy: agreed, but this spec means that if you have 2 processes, then you have 2 possible claims at the same time | 12:37 |
mdbooth | johnthetubaguy: we've banned db locks? | 12:37 |
BobBall | danpb: I see; so usage is tracked by get_available_resource too, right? | 12:38 |
mdbooth | johnthetubaguy: Got a link, I missed that. | 12:38 |
danpb | BobBall: yep | 12:38 |
bauzas | johnthetubaguy: that looks a possible first step, but then we should think about how to sync 2 claims | 12:38 |
BobBall | okies | 12:38 |
bauzas | johnthetubaguy: given we say that Claims are the Facade for sync'ing | 12:38 |
johnthetubaguy | mdbooth: kinda of yeah, we are saying prefer compare and swap style (or whatever the proper name is) | 12:38 |
bauzas | johnthetubaguy: but all of that needs a bit of discussion, I feel | 12:39 |
mdbooth | johnthetubaguy: Ok, so not a hard ban. Just an expressed preference. | 12:39 |
*** zhangjn has joined #openstack-nova | 12:39 | |
mdbooth | Sorry to butt in :) | 12:39 |
*** zhangjn has quit IRC | 12:39 | |
johnthetubaguy | mdbooth: yeah, not totally sure if we got that written down, now you mention it | 12:39 |
johnthetubaguy | bauzas: sure, with the code in front of us | 12:39 |
*** zhangjn has joined #openstack-nova | 12:40 | |
*** Daisy has quit IRC | 12:40 | |
bauzas | johnthetubaguy: I'd even try to see some convo during the summit, provided we have some time for that | 12:40 |
bauzas | johnthetubaguy: because we have lots of things trying to see how we could have sched claims | 12:40 |
*** lucas-hungry is now known as lucasagomes | 12:40 | |
bauzas | at least jaypipes was thniking about it, too | 12:40 |
bauzas | and I'm not sure he's aware of that one, too | 12:40 |
johnthetubaguy | bauzas: I feel we need to see the code before we can debate it | 12:41 |
*** aswadr has joined #openstack-nova | 12:41 | |
bauzas | johnthetubaguy: well, unfortunately the BP is a bit vague about what could be the implementation | 12:41 |
bauzas | so yeah seeing code is good | 12:41 |
bauzas | because I'm +1 on the move | 12:42 |
bauzas | but I'm really worried by the implementation details | 12:42 |
*** Daisy has joined #openstack-nova | 12:42 | |
BobBall | danpb: How/where do we verify that vpug_enabled can only be set/used when vgpu_permitted is also set? Is that logic in the compute driver (i.e. verify presence of both flags when actually allocating the vgpu?) | 12:42 |
danpb | yeah, you'd usually put that logic in the virt driver | 12:43 |
*** garyk has quit IRC | 12:43 | |
alaski | bauzas: melwitt I'm not really around today so I'll leave the cells meeting details to you. melwitts plan to run it in case people show up with topics is great. | 12:43 |
*** garyk has joined #openstack-nova | 12:43 | |
bauzas | alaski: yup, I saw melwitt's point yesterday night my time | 12:43 |
*** Daisy has quit IRC | 12:43 | |
bauzas | alaski: we could just discuss about what we want for Mitaka | 12:43 |
*** aix has joined #openstack-nova | 12:43 | |
BobBall | Can do - was just wondering whether there was some additional validation when setting image props or pre-schedule validation somewhere | 12:43 |
bauzas | alaski: and leave you yelling against us | 12:44 |
bauzas | :p | 12:44 |
snikitin | sdague: Hi Sean. I want to ask you a question about oslo.cache patch https://review.openstack.org/#/c/203049/30/nova/utils.py . You wrote that we can't deprecate it until N. Why? I thought deprecation cycle is consist of deprecating something in current release, and removing it in the next release. Am I wrong? | 12:44 |
*** dims_ has joined #openstack-nova | 12:44 | |
*** zhangjn has quit IRC | 12:44 | |
alaski | bauzas: heh, sounds good. I'm out for the next few days but then my plan is to push some specs up | 12:44 |
*** zhangjn has joined #openstack-nova | 12:45 | |
*** markvoelker has joined #openstack-nova | 12:46 | |
*** zhangjn has quit IRC | 12:46 | |
*** Marga_ has quit IRC | 12:46 | |
lxsli | melwitt: would you look kindly on a --json flag to python-novaclient? | 12:47 |
*** Marga_ has joined #openstack-nova | 12:47 | |
*** zhangjn has joined #openstack-nova | 12:48 | |
sdague | snikitin: but your patch deprecates without any of our deprecation triggers | 12:48 |
sdague | that's my concern, people are expecting certain deprecation signals | 12:49 |
kashyap | bauzas: Hi, I only noticed now your brief ping in the backchat, I didn't read all the scrollback... | 12:49 |
*** markvoelker_ has joined #openstack-nova | 12:49 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova-specs: Remove Priorities section for approved specs https://review.openstack.org/231964 | 12:49 |
johnthetubaguy | bauzas: about the spec being vague, it defines the problem space, and the direction, the stuff thats best discussed with code in front of you is left to later, as it should be | 12:49 |
*** VikasC has quit IRC | 12:50 | |
openstackgerrit | Sudipta Biswas proposed openstack/nova: add helper method to obtain image client version https://review.openstack.org/222150 | 12:50 |
bauzas | johnthetubaguy: well, I'm a bit opiniated about having specs that give more of a direction, but that's MHO :) | 12:50 |
*** raildo-afk is now known as raildo | 12:51 | |
johnthetubaguy | bauzas: its a balance, if its a small thing, and the code would answer more questions, do it in the code, its way less overhead, if you want to avoid re-work, feel free to agree more details up front, but its a lot more work | 12:52 |
johnthetubaguy | lxsli: what would --json do? | 12:52 |
johnthetubaguy | lxsli: i am curious | 12:52 |
*** xyang1 has joined #openstack-nova | 12:52 | |
*** markvoelker has quit IRC | 12:53 | |
johnthetubaguy | lxsli: bauzas: would be awesome if you can tidy up the whiteboard on this blueprint for me please :) https://blueprints.launchpad.net/nova/+spec/check-destination-on-migrations | 12:53 |
johnthetubaguy | just so its clear what needs review, and what has merged, etc | 12:53 |
*** markvoelker_ has quit IRC | 12:54 | |
lxsli | johnthetubaguy: will do | 12:54 |
bauzas | johnthetubaguy: sure it's a trade-off :) | 12:54 |
danpb | bauzas: lol, several approved Mitaka specs are refering to *liberty* project priorities | 12:54 |
lxsli | johnthetubaguy: --json would print JSON output instead of the formatted table | 12:54 |
johnthetubaguy | lxsli: awesome, thank you | 12:54 |
lxsli | johnthetubaguy: like https://review.openstack.org/#/c/57834 but implemented better :) | 12:54 |
danpb | makes me even more sure that removing this section was the right thing todo | 12:54 |
johnthetubaguy | lxsli: ah, so more "script friendly", gotcha | 12:55 |
*** whenry has joined #openstack-nova | 12:55 | |
johnthetubaguy | lxsli: I was trying to compare that with --debug in my head somehow | 12:55 |
*** burgerk has quit IRC | 12:55 | |
lxsli | johnthetubaguy: what's your +1/-1 on the idea? | 12:55 |
*** markvoelker has joined #openstack-nova | 12:56 | |
*** eharney has joined #openstack-nova | 12:56 | |
*** Sree has quit IRC | 12:56 | |
*** sahid has quit IRC | 12:56 | |
*** su_zhang has quit IRC | 12:56 | |
*** Sree has joined #openstack-nova | 12:57 | |
*** sahid has joined #openstack-nova | 12:58 | |
openstackgerrit | John Garbutt proposed openstack/nova: Adds Hyper-V Cluster implementation https://review.openstack.org/199037 | 12:58 |
bauzas | lxsli: johnthetubaguy: saw zigo's thread ? | 13:01 |
*** deepthi has quit IRC | 13:01 | |
johnthetubaguy | bauzas: I guess not, which one? | 13:01 |
bauzas | sec | 13:01 |
*** Sree has quit IRC | 13:01 | |
*** jaypipes has joined #openstack-nova | 13:01 | |
*** rlrossit has joined #openstack-nova | 13:01 | |
bauzas | johnthetubaguy: lxsli: http://lists.openstack.org/pipermail/openstack-dev/2015-October/076112.html | 13:02 |
bauzas | barely related but quite the same usecase | 13:02 |
*** links has quit IRC | 13:02 | |
lxsli | bauzas: good user story indeed | 13:02 |
bauzas | lxsli: I'm a bit concerned by the CLI contract tho | 13:03 |
johnthetubaguy | bauzas: yeah, thats a good point... lxsli there are plans to kill python-novaclient so maybe we just work on the openstack cli instead? | 13:03 |
lxsli | bauzas: I'm not sure whether the openstack client supports JSON output, I think it does? So the question is whether we bother adding it to the old one | 13:03 |
johnthetubaguy | bauzas: with always using the latest version, its a bit dodgy | 13:03 |
bauzas | yeah that of johnthetubaguy | 13:03 |
bauzas | johnthetubaguy: yup, my thoughts was that the novaclient API was good, but that CLI wasn't exacly contracting the same way | 13:04 |
*** pbrooko has quit IRC | 13:04 | |
bauzas | so, giving a chance for openstackclient to call novaclient API sounds cool to me | 13:04 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable options for oslo.reports https://review.openstack.org/216917 | 13:04 |
bauzas | and adding json support to openstackclient sounds better to me | 13:04 |
bauzas | because if not, then just use your own tool | 13:05 |
bauzas | having a command-line interface returning serialized JSON sounds weird, why not directly having novaclient ? | 13:05 |
bauzas | lxsli: any spec I could put my dirty hands on ? | 13:06 |
*** jaypipes has quit IRC | 13:06 | |
lxsli | It might be the easiest way from Ruby/bash | 13:06 |
lxsli | bauzas: no not yet just floating the idea | 13:06 |
danpb | bauzas: openstackclient supports CSV output mode | 13:06 |
bauzas | danpb: right, see my above link | 13:06 |
danpb | bauzas: but it would actually make alot of sense to have a JSON mode, as CSV is incredibly poorly spec | 13:07 |
bauzas | danpb: I'm just explaining that it should probably be done by openstackclient instead of the Nova CLI | 13:07 |
danpb | when it comes to dealing with escaping & quoting, so many CSV parsers & generators have slightly different semantics | 13:07 |
*** mc_nair has joined #openstack-nova | 13:07 | |
bauzas | danpb: agreed with you | 13:07 |
danpb | JSON by comparison is pretty clearly defined and has good interoperability | 13:07 |
bauzas | sure | 13:07 |
johnthetubaguy | +1 all that | 13:08 |
lxsli | OK I thought the openstackclient already did JSON but if it doesn't I'm happy to look at that first | 13:08 |
bauzas | I'm not advocating for not having JSON support, just that it should be done in openstackclient IMHO | 13:08 |
danpb | yeah, | 13:08 |
danpb | doesn'pt make much sense to it in novaclient if openstackclient is the future direction | 13:08 |
lxsli | Are we actively not adding features to novaclient then? IE would having it in both be better than just in openstackclient? | 13:09 |
bauzas | lxsli: to be clear, there are 2 things | 13:09 |
lxsli | s/better/worse/ | 13:09 |
bauzas | #1 novaclient is the python API | 13:09 |
bauzas | #2 the nova CLI is the shell script wrapping the pythonclient | 13:10 |
lxsli | Yes that's clear | 13:10 |
bauzas | lxsli: AFAIK, the plan is to support novaclient but leave openstackclient call it directly | 13:11 |
openstackgerrit | Alexey I. Froloff proposed openstack/nova: Fix more inconsistency between Nova-Net and Neutron https://review.openstack.org/226274 | 13:11 |
lxsli | I expect novaclient itself returns data structures and it's nova CLI which formats into a table | 13:11 |
bauzas | that's what I thought was the consensus in YVR or Paris, I don't remember correctly | 13:11 |
bauzas | lxsli: yup | 13:11 |
lxsli | so this is a "nova CLI"/"openstackclient" feature | 13:11 |
bauzas | not sure what the slash means, but yes | 13:12 |
lxsli | happy to add it to the latter first, just wondering if there's active opposition to also adding it to "nova CLI" | 13:12 |
lxsli | slash meant 'or' | 13:12 |
bauzas | well, I leave that (adding something to Nova CLI) to other folks like melwitt and Vek_ :) | 13:12 |
*** signed8bit has joined #openstack-nova | 13:13 | |
bauzas | I'm just saying that openstackclient seems the first fit for it :) | 13:13 |
lxsli | OK, agreed on that | 13:13 |
*** signed8bit has quit IRC | 13:13 | |
*** doron_afk has quit IRC | 13:13 | |
*** doron_afk has joined #openstack-nova | 13:14 | |
lxsli | bauzas: where do specs for python-openstackclient go, do you know offhand please? | 13:15 |
*** fawadkhaliq has joined #openstack-nova | 13:15 | |
*** mdrabe has joined #openstack-nova | 13:16 | |
*** su_zhang has joined #openstack-nova | 13:16 | |
*** david-lyle has quit IRC | 13:16 | |
*** mriedem has joined #openstack-nova | 13:17 | |
edleafe | lxsli: there's also this: https://wiki.openstack.org/wiki/SDK-Development/PythonOpenStackSDK | 13:17 |
edleafe | lxsli: it's supposed to replace the python module part of novaclient, but it's been dragging without a lot of momentum lately | 13:17 |
bauzas | lxsli: I suspect it's old-style https://blueprints.launchpad.net/python-openstackclient | 13:18 |
mriedem | edleafe: and we were told at the yvr summit not to use the sdk in the server projects | 13:18 |
*** jichen has quit IRC | 13:18 | |
lxsli | bauzas: thanks | 13:18 |
edleafe | mriedem: we should use novaclient instead? | 13:19 |
bauzas | mriedem: yeah that | 13:19 |
mriedem | edleafe: i guess? if you're heat or ceilometer. | 13:19 |
bauzas | mriedem: I remember that discussion, forgot if it was in yvr or paris | 13:19 |
mriedem | or write your own service layer that talks to the rest api like tempest does | 13:19 |
bauzas | +1 | 13:19 |
briancurtin | YVR, and i didn’t understand it at all, but someone said it and everyone immediately agreed | 13:19 |
bauzas | the API bindings are there | 13:19 |
lxsli | I remember us agreeing to consume Neutron's API raw at YVR | 13:20 |
mriedem | i don't understand it either | 13:20 |
bauzas | to be used by other projects | 13:20 |
mriedem | what good is an sdk if the server projects can't use it | 13:20 |
*** fawadkhaliq has quit IRC | 13:20 | |
bauzas | I thought the whole idea was to have one single CLI for the user, but as many as necessary bindings for the services | 13:20 |
mriedem | lxsli: yeah i had a todo to go off and look at that as an alternative to beagles' spec about wrapping the neutronclient stuff in nova | 13:21 |
edleafe | mriedem: there was an inefficiency with keystone that was a problem for OSC, but I don't remember any other specific criticisms | 13:21 |
jroll | mriedem: https://xkcd.com/927/ obviously | 13:21 |
bauzas | oh that | 13:21 |
bauzas | I know that xkcd number by heart | 13:21 |
lxsli | dtroyer: ping | 13:21 |
jroll | lol | 13:21 |
johnthetubaguy | jroll: I thought of you when reading a spec yesterday about resource pool tracking | 13:22 |
jroll | oh? | 13:22 |
johnthetubaguy | jroll: its about tracking groups of resources, like shared storage thats shared across multiple nova-computes | 13:23 |
*** annegentle has joined #openstack-nova | 13:23 | |
jroll | :o | 13:23 |
* jroll finds this spec | 13:23 | |
*** dustins has joined #openstack-nova | 13:23 | |
johnthetubaguy | jroll: I might have the link handy, one sec | 13:23 |
bauzas | jroll: http://www.commitstrip.com/en/2015/09/21/me-my-code-and-i/ also appies | 13:23 |
jroll | I think we talked about this a bit in minnesota yeah? | 13:23 |
bauzas | applies | 13:23 |
johnthetubaguy | jroll: yeah, https://review.openstack.org/#/c/225546/ | 13:23 |
*** signed8bit has joined #openstack-nova | 13:24 | |
jroll | bauzas: heh, I like | 13:24 |
jroll | johnthetubaguy: thanks | 13:24 |
*** rdopiera is now known as rdopiera_afk | 13:24 | |
bauzas | johnthetubaguy: could you put the -W button to that spec ? it could be confusing | 13:25 |
*** ctrath has joined #openstack-nova | 13:25 | |
johnthetubaguy | bauzas: there is the big WIP bit a the top | 13:25 |
* bauzas *facepalm* | 13:25 | |
*** kmartin has quit IRC | 13:25 | |
bauzas | the biggest it is, the less I can see it | 13:26 |
bauzas | the lesser* | 13:26 |
johnthetubaguy | bauzas: heh | 13:26 |
lxsli | the bigger it is, the less I can see it :) | 13:26 |
bauzas | rah | 13:28 |
bauzas | plus c'est gros, moins je le vois | 13:28 |
bauzas | nah | 13:28 |
*** ijuwang has joined #openstack-nova | 13:28 | |
lxsli | English: there are no ruels | 13:29 |
jroll | johnthetubaguy: interesting read, I'm not sure I want to depend on it but it could be useful | 13:31 |
mriedem | danpb: what do you think about moving https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix into the nova devref? | 13:31 |
danpb | mriedem: sure, we could do that | 13:31 |
mriedem | i like the change policy thing you added | 13:32 |
johnthetubaguy | jroll: yeah, it might be more about the longer term thing | 13:32 |
jroll | nod | 13:32 |
*** david-lyle has joined #openstack-nova | 13:32 | |
DRoBeR | <openstackgerrit> Flavio Percoco proposed openstack/nova-specs: (follow up) Nova Image v1 -> v2 <---- I'm actually having some issues trying to get "nova image-list" due to nova using image V1 instead of V2. :/ Is there any way I can set nova to get that through API V2, guys? | 13:32 |
danpb | mriedem: we should probably incorporate some of the background rationale/info from that email of mine you quoted too | 13:32 |
mriedem | the ML from may? | 13:33 |
mriedem | when we bumped to 0.10.2? | 13:33 |
*** smatzek has quit IRC | 13:33 | |
*** tpeoples has joined #openstack-nova | 13:33 | |
danpb | mriedem: yeah | 13:33 |
pkholkin | sdague: Hello Sean. We can't mark the option for example like 'deprecated_for_removal' because we will get conflict with the same option and tests will start fail | 13:35 |
sdague | pkholkin: right, I understand | 13:35 |
pkholkin | do you have some suggestions? | 13:35 |
sdague | however, that seems like a big problem | 13:35 |
*** diogogmt has quit IRC | 13:35 | |
sdague | no, I don't, that's why I raised it as an issue, and also poked on -oslo about it | 13:35 |
sdague | because this seems problematic | 13:35 |
pkholkin | also we can't even change 'help' message | 13:37 |
*** signed8bit has quit IRC | 13:38 | |
*** LukeHinds has joined #openstack-nova | 13:38 | |
pkholkin | sdague: we don't have a good idea how to make it more clear for users | 13:38 |
openstackgerrit | Andrea Rosa proposed openstack/nova-specs: Allow more instance operations during live migration https://review.openstack.org/226199 | 13:39 |
mriedem | huh, seems we should have this in tree https://review.openstack.org/#/c/229013/ | 13:39 |
*** thumpba has joined #openstack-nova | 13:39 | |
pkholkin | sdague: maybe we should add warning here? https://review.openstack.org/#/c/203049/30/nova/servicegroup/drivers/mc.py | 13:39 |
*** diogogmt has joined #openstack-nova | 13:39 | |
pkholkin | at the same place where the note | 13:40 |
sdague | pkholkin: possibly, we need something | 13:40 |
*** Marga_ has quit IRC | 13:41 | |
*** signed8bit has joined #openstack-nova | 13:41 | |
pkholkin | mriedem: Good day Matt! Could you participate in our discussion about oslo.cache too?) https://review.openstack.org/#/c/203049/30 | 13:42 |
pkholkin | we are speaking about good deprecating of the option there | 13:42 |
*** njirap has quit IRC | 13:43 | |
*** perci has joined #openstack-nova | 13:43 | |
pkholkin | sdague, mriedem: we can add warning message in mc.py near the note, will it be ok? | 13:44 |
*** vladikr has joined #openstack-nova | 13:44 | |
openstackgerrit | Dan Smith proposed openstack/nova: Remove obj_relationships from objects https://review.openstack.org/231271 | 13:46 |
openstackgerrit | Dan Smith proposed openstack/nova: Clean up tests for dropping obj_relationships https://review.openstack.org/231270 | 13:46 |
openstackgerrit | Dan Smith proposed openstack/nova: Fix InstanceV1 backports to use context https://review.openstack.org/231649 | 13:46 |
openstackgerrit | Dan Smith proposed openstack/nova: DNM: Canary test the backporting from a liberty compute https://review.openstack.org/231486 | 13:46 |
openstackgerrit | Dan Smith proposed openstack/nova: Fix up Service object for manifest-based backports https://review.openstack.org/231269 | 13:46 |
openstackgerrit | Dan Smith proposed openstack/nova: DNM: Drop Instance v1.x support https://review.openstack.org/220690 | 13:46 |
openstackgerrit | Dan Smith proposed openstack/nova: Drop unused obj_to_primitive() override https://review.openstack.org/231686 | 13:46 |
*** links has joined #openstack-nova | 13:46 | |
*** annegentle has quit IRC | 13:46 | |
sdague | pkholkin: lets see what that looks like | 13:47 |
dansmith | this last one ^^^ is blocking an o.vo bug fix from landing, which we need to get released so we don't break if we have to bump an object version in a specific way | 13:47 |
mriedem | pkholkin: in a minute | 13:48 |
*** Marga_ has joined #openstack-nova | 13:48 | |
mriedem | is it odd that archive_deleted_rows doesn't attempt to sort the tables based on foreign key constraints? https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L5967 | 13:48 |
mriedem | it just gets all of the tables from the nova db models module https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L6043 | 13:49 |
*** signed8bit has quit IRC | 13:50 | |
bauzas | hey nova-drivers, some nit patch for removing the prio section would be loving a +2/+W https://review.openstack.org/#/c/231964/1 :) | 13:50 |
*** mudassirlatif has joined #openstack-nova | 13:51 | |
*** zhangjn has quit IRC | 13:52 | |
*** zhangjn has joined #openstack-nova | 13:52 | |
*** zhangjn has quit IRC | 13:52 | |
*** zzzeek has joined #openstack-nova | 13:53 | |
*** zhangjn has joined #openstack-nova | 13:53 | |
*** zhangjn has quit IRC | 13:53 | |
*** zhangjn has joined #openstack-nova | 13:54 | |
openstackgerrit | Pavel Kholkin proposed openstack/nova: Switch to oslo.cache lib https://review.openstack.org/203049 | 13:54 |
*** EinstCrazy has joined #openstack-nova | 13:55 | |
pkholkin | sdague, mriedem: patch was updated | 13:55 |
*** irenab has quit IRC | 13:58 | |
openstackgerrit | Andrea Rosa proposed openstack/nova-specs: Allow more instance operations during live migration https://review.openstack.org/226199 | 13:58 |
*** takedakn has quit IRC | 13:59 | |
*** signed8bit has joined #openstack-nova | 14:00 | |
*** Piet has joined #openstack-nova | 14:01 | |
*** breitz has quit IRC | 14:02 | |
*** breitz has joined #openstack-nova | 14:02 | |
johnthetubaguy | mriedem: is the archive thing, the shadow table thing? | 14:03 |
mriedem | yeah | 14:03 |
mriedem | there is an ops patch that basically does all of that https://review.openstack.org/#/c/229013/ | 14:03 |
mriedem | showed up in the ops ML | 14:03 |
*** manishg has joined #openstack-nova | 14:04 | |
seanw1 | In nova/compute/manager.py -> _prep_resize() a check is made against instance.host == self.host and if the allow_resize_to_same_host is false an exception is thrown. The prep_resize executes on the current host of the instance, and I cannot find where the instance.host is updated to the migrating host before the call to prep_resize. So without the allow_resize_to_same_host set to true, an error is always reported, but the resize complete | 14:04 |
seanw1 | is this a bug or am I missing something? | 14:04 |
*** markvoelker_ has joined #openstack-nova | 14:04 | |
mriedem | jwcroppe: ^ i'm sure you can field that one | 14:04 |
johnthetubaguy | mriedem: oops, so thats the first I heard of that repo, interesting | 14:05 |
mriedem | i have 20 deleted instances in my devstack and ran "nova-manage db archive_deleted_rows --max_rows 10" and it didn't fail but didn't delete anything either | 14:05 |
mriedem | so it's definitely busted | 14:05 |
*** smatzek has joined #openstack-nova | 14:05 | |
*** burgerk has joined #openstack-nova | 14:05 | |
johnthetubaguy | mriedem: eek, we should totally add a functional test for that thing | 14:06 |
mriedem | yeah... | 14:06 |
*** Sree has joined #openstack-nova | 14:06 | |
mriedem | i think there is a semi functional unit test for it | 14:06 |
*** diogogmt has quit IRC | 14:06 | |
mriedem | test_archive_deleted_rows | 14:07 |
*** emagana has joined #openstack-nova | 14:07 | |
johnthetubaguy | hmm | 14:07 |
*** burgerk_ has joined #openstack-nova | 14:07 | |
mriedem | mysql> select * from nova.shadow_instances; | 14:07 |
mriedem | Empty set (0.01 sec) | 14:07 |
*** markvoelker has quit IRC | 14:08 | |
*** thumpba has quit IRC | 14:08 | |
*** diogogmt has joined #openstack-nova | 14:08 | |
dansmith | jwcroppe: http://xkcd.com/1587/ | 14:08 |
*** emagana has quit IRC | 14:09 | |
*** burgerk has quit IRC | 14:10 | |
*** emagana has joined #openstack-nova | 14:10 | |
jwcroppe | dansmith: that's spot on | 14:11 |
*** xdccMule[60EX] has joined #openstack-nova | 14:11 | |
xdccMule[60EX] | Ciao a tutti :D | 14:11 |
*** alaski is now known as alaski_out | 14:12 | |
xdccMule[60EX] | Hello to all | 14:12 |
xdccMule[60EX] | anyone here? | 14:12 |
*** whenry has quit IRC | 14:12 | |
*** markvoelker_ has quit IRC | 14:12 | |
edleafe | xdccMule[60EX]: you can just ask your question. | 14:13 |
*** diogogmt has quit IRC | 14:13 | |
xdccMule[60EX] | Hello, i'm a student of an Italian university | 14:13 |
xdccMule[60EX] | Prof tell me about OpenStack (little overview) and he want from me | 14:14 |
*** thumpba has joined #openstack-nova | 14:14 | |
xdccMule[60EX] | know what's Nova and information about this | 14:14 |
*** perci is now known as njirap | 14:15 | |
xdccMule[60EX] | now...i'm finding document in docs.openstack | 14:15 |
johnthetubaguy | dansmith: I meant to catch you about those upgrade bugs, do we want to hold RC2 and get those into RC2? | 14:15 |
xdccMule[60EX] | but when i see irc chat i think is more simply ask here a document | 14:15 |
*** Marga_ has quit IRC | 14:16 | |
dansmith | jwcroppe: I don't eat fish, but it's close yeah | 14:16 |
dansmith | johnthetubaguy: you mean the one I mentioned above? | 14:16 |
*** Marga_ has joined #openstack-nova | 14:16 | |
johnthetubaguy | dansmith: yes, late yesterday I think | 14:16 |
edleafe | xdccMule[60EX]: this IRC channel is for people developing OpenStack Nova. You would probably be better off asking questions in either #openstack or #openstack-101 | 14:17 |
xdccMule[60EX] | ok thanks | 14:17 |
edleafe | xdccMule[60EX]: there are lots of people in those channels who would be very helpful | 14:17 |
*** artom has joined #openstack-nova | 14:18 | |
dansmith | johnthetubaguy: hmm, hang on | 14:18 |
dansmith | johnthetubaguy: so actually this one would be really good if we can get it in: https://review.openstack.org/#/c/231649/ | 14:18 |
dansmith | I need to file a bug for it first though | 14:18 |
dansmith | johnthetubaguy: that one actually could affect kilo->liberty | 14:18 |
johnthetubaguy | dansmith: thats the one I was thinking yes | 14:19 |
*** manishg has quit IRC | 14:19 | |
dansmith | johnthetubaguy: okay, let me file a bug and jam in in there and maybe we can land those quick | 14:20 |
johnthetubaguy | dansmith: +1 thank you! | 14:20 |
*** zhangjn has quit IRC | 14:20 | |
*** cfriesen has joined #openstack-nova | 14:20 | |
dansmith | johnthetubaguy: should I target for rc2 or just set the tag? | 14:20 |
johnthetubaguy | dansmith: feel free to target that one right away | 14:20 |
*** artom_ has joined #openstack-nova | 14:21 | |
mriedem | is it odd that we don't actually required admin context when calling db.api.archive_deleted_rows? | 14:21 |
mriedem | we pass one in but we don't check it | 14:21 |
johnthetubaguy | mriedem: yeah, the API work removed the whole admin context needed in the DB layer thing | 14:22 |
johnthetubaguy | mriedem: but no one has gone back to remove all the elevated bits | 14:22 |
mriedem | ok, was going to say, the context used here is only passed in from the nova-manage command | 14:22 |
mriedem | and it's completely unused now | 14:22 |
johnthetubaguy | ah, yuck, some tidying up to do around that, its tempting to put that on the low hanging fruit list to get it fixed up | 14:23 |
mriedem | it sort of seems odd that it was removed from db apis that are only accessed through nova-manage, but i guess we assume only admins have access to nova-manage to start with | 14:23 |
johnthetubaguy | yeah, its basically file access controls on nova.conf I think | 14:23 |
*** artom has quit IRC | 14:23 | |
raildo | johnthetubaguy: hi :) thanks for you comments in the nested quota spec, do you have some time to talk about it? | 14:24 |
*** manishg has joined #openstack-nova | 14:24 | |
openstackgerrit | Dan Smith proposed openstack/nova: Remove obj_relationships from objects https://review.openstack.org/231271 | 14:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Clean up tests for dropping obj_relationships https://review.openstack.org/231270 | 14:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Fix InstanceV1 backports to use context https://review.openstack.org/231649 | 14:24 |
openstackgerrit | Dan Smith proposed openstack/nova: DNM: Canary test the backporting from a liberty compute https://review.openstack.org/231486 | 14:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Fix up Service object for manifest-based backports https://review.openstack.org/231269 | 14:24 |
openstackgerrit | Dan Smith proposed openstack/nova: DNM: Drop Instance v1.x support https://review.openstack.org/220690 | 14:24 |
johnthetubaguy | raildo: yes | 14:24 |
*** manishg has quit IRC | 14:24 | |
*** tsekiyama has joined #openstack-nova | 14:25 | |
johnthetubaguy | dansmith: sweet, +W, I guess we should line up the backport | 14:26 |
dansmith | johnthetubaguy: yep, just a sec | 14:27 |
*** xdccMule[60EX]^8 has joined #openstack-nova | 14:27 | |
*** xdccMule[60EX]^8 has left #openstack-nova | 14:27 | |
johnthetubaguy | dansmith: no rush, needs to merge in master first anyways, I am getting ahead of myself | 14:27 |
raildo | johnthetubaguy: great, so about the "dedicated" resoucers, it's something complicated, beacuse I agree with your comment, it's something that I don't want and I had discussed in the last summit with the cinder folks...but they decided to implement on this way | 14:28 |
raildo | johnthetubaguy: so if we want to maintain the same behaviout for cinder, this will be a tricky point. | 14:29 |
raildo | behaviour* | 14:30 |
dansmith | johnthetubaguy: when we're ready: https://review.openstack.org/232033 | 14:30 |
*** artom_ is now known as artom | 14:30 | |
*** xdccMule[60EX] has quit IRC | 14:31 | |
raildo | johnthetubaguy: maybe we can have both features as a config option? something that we can enabled... | 14:31 |
openstackgerrit | Jesse J. Cook proposed openstack/nova: xenapi_device_id integer, expected string https://review.openstack.org/231703 | 14:31 |
*** ZZelle_ has quit IRC | 14:32 | |
*** markvoelker has joined #openstack-nova | 14:32 | |
johnthetubaguy | dansmith: ack, thank you! | 14:33 |
*** pm90_ has joined #openstack-nova | 14:33 | |
johnthetubaguy | raildo: so I don't like a config that changes API behaviour, its terrible for discovery | 14:34 |
johnthetubaguy | raildo: but I am OK supporting both options in the API, if that makes sense? | 14:34 |
johnthetubaguy | raildo: the key thing, is you can do the same successful API calls with both cinder and nova, I am not questioning that bit | 14:35 |
johnthetubaguy | raildo: I just think nested quotas should feel the same as regular quotas, at least by default | 14:35 |
raildo | johnthetubaguy: ++ totally agree with you | 14:36 |
*** fawadkhaliq has joined #openstack-nova | 14:37 | |
*** pm90__ has joined #openstack-nova | 14:38 | |
raildo | johnthetubaguy: I'm trying implement in a way that willhave a smaller impact on the API, like to update a quota for a subproject, You just have to pass a target-project like happend on nova in others API calls | 14:38 |
*** nikhil_k_ is now known as nikhil | 14:38 | |
*** EinstCrazy has quit IRC | 14:39 | |
raildo | but to handle with both situations we have to be more intrusive, adding something like "?overcommit" | 14:40 |
raildo | in the API call | 14:40 |
*** csd has joined #openstack-nova | 14:40 | |
*** pm90_ has quit IRC | 14:40 | |
bauzas | dansmith: so I'm reviewing https://review.openstack.org/#/c/231269/5/nova/objects/service.py,cm and I'm a bit unclear | 14:41 |
johnthetubaguy | raildo: true, well it would be overcommit by default, as thats what the original API call does right? | 14:41 |
*** amotoki has joined #openstack-nova | 14:42 | |
mriedem | johnthetubaguy: here is why db archive didn't work for me, it was lost in the debug log output from devstack: 2015-10-07 14:18:39.265 WARNING nova.db.sqlalchemy.api [req-149506be-fbdd-452c-bd35-76f4ba269bc3 None None] IntegrityError detected when archiving table instances | 14:42 |
bauzas | dansmith: now, ovoo.obj_make_compatible_from_manifest() is called when primitiving the object, right? | 14:42 |
mriedem | so yeah, totes busted | 14:42 |
dansmith | bauzas: yes | 14:42 |
johnthetubaguy | mriedem: nice | 14:42 |
sdague | mriedem: yeh, that was reported as an issue back in phili | 14:42 |
*** mtanino has joined #openstack-nova | 14:43 | |
bauzas | dansmith: which itself sets the manifest and calls the previous obj_make_compatible() | 14:43 |
raildo | johnthetubaguy: sounds good for me. or we can use a ?dedicated as a option and overcommit by default | 14:43 |
*** nikhil is now known as nikhil_k_ | 14:43 | |
*** agireud has quit IRC | 14:44 | |
*** nikhil_k_ is now known as nikhil__ | 14:44 | |
dansmith | bauzas: ....yeah... | 14:44 |
mriedem | sdague: right, https://bugs.launchpad.net/nova/+bug/1183523 | 14:44 |
mriedem | super old | 14:44 |
openstack | Launchpad bug 1183523 in OpenStack Compute (nova) "db-archiving fails to clear some deleted rows from instances table" [Medium,Confirmed] | 14:44 |
*** nikhil__ is now known as nikhil | 14:45 | |
mriedem | sdague: which has led to operators writing their own archive scripts https://review.openstack.org/#/c/229013/ | 14:45 |
bauzas | dansmith: at the end of the stack, it will call _do_subobject_backport() which will call the object obj_make_obj_compatible | 14:45 |
sdague | yeh | 14:45 |
mriedem | which blows | 14:45 |
bauzas | dansmith: that, I understand | 14:45 |
mriedem | i'm pretty sure simply revert sorting the tables using sqla metadata, like in migration 267, we can fix the Fkey issue | 14:45 |
mriedem | *reverse | 14:45 |
bauzas | dansmith: now, my question is then in https://review.openstack.org/#/c/231269/5/nova/objects/service.py,cm | 14:46 |
mriedem | also, the way max_rows is used, seems like a terrible idea | 14:46 |
bauzas | dansmith: shouldn't we change L129 to call super().obj_make_compatible_from_manifest() to set the manifest for the nested object? | 14:46 |
bauzas | dansmith: sorry if I took time, I needed to make sure I was understanding the stack | 14:47 |
*** moshele has quit IRC | 14:47 | |
*** lpetrut has quit IRC | 14:47 | |
*** unicell has joined #openstack-nova | 14:47 | |
*** claudiub has quit IRC | 14:48 | |
openstackgerrit | Paul Carlton proposed openstack/nova-specs: Blueprint to use libvirt storage pools https://review.openstack.org/232045 | 14:48 |
*** Murali_ has joined #openstack-nova | 14:48 | |
*** unicell1 has quit IRC | 14:49 | |
*** Murali has quit IRC | 14:49 | |
*** HenryG has quit IRC | 14:49 | |
*** Murali_ is now known as Murali | 14:50 | |
*** hogepodge has quit IRC | 14:50 | |
*** dansmith has quit IRC | 14:50 | |
*** hogepodge has joined #openstack-nova | 14:51 | |
*** dansmith has joined #openstack-nova | 14:51 | |
*** claudiub has joined #openstack-nova | 14:51 | |
*** stevemar_ has joined #openstack-nova | 14:51 | |
*** vlaza has quit IRC | 14:51 | |
*** tpeoples has quit IRC | 14:52 | |
johnthetubaguy | raildo: yeah, something like that, well dedicated=true, although thats tricky, because you need all sub teants to be dedicated to get the performance gain you wanted before | 14:52 |
*** HenryG has joined #openstack-nova | 14:52 | |
*** ssurana has joined #openstack-nova | 14:52 | |
*** browne has joined #openstack-nova | 14:53 | |
*** Daisy has joined #openstack-nova | 14:53 | |
*** tjones has joined #openstack-nova | 14:53 | |
raildo | johnthetubaguy: you think they may be at different drivers? | 14:54 |
*** alexschm has quit IRC | 14:54 | |
* raildo trying to do a brainstorm here :P | 14:54 | |
*** matrohon has quit IRC | 14:54 | |
*** rubasov has quit IRC | 14:54 | |
*** alaski_out has quit IRC | 14:55 | |
openstackgerrit | Paul Carlton proposed openstack/nova-specs: Blueprint to use libvirt storage pools https://review.openstack.org/232045 | 14:56 |
*** Sree has quit IRC | 14:57 | |
dansmith | bauzas: good catch! | 14:57 |
*** Sree has joined #openstack-nova | 14:57 | |
*** alejandrito has joined #openstack-nova | 14:57 | |
*** alaski has joined #openstack-nova | 14:58 | |
*** kmartin has joined #openstack-nova | 14:58 | |
*** diogogmt has joined #openstack-nova | 14:59 | |
openstackgerrit | Jesse J. Cook proposed openstack/nova: xenapi_device_id integer, expected string https://review.openstack.org/231703 | 15:00 |
bauzas | dansmith: so, IIUC, now if a novaobject is not yet providing obj_make_obj_compatible it calls the ovoo base method, which itself calls the object obj_make_compatible right? | 15:00 |
*** tpeoples has joined #openstack-nova | 15:00 | |
bauzas | dansmith: so you only need to override obj_make_obj_compatible only when you want to provide the version manifest elsewhere, right? | 15:00 |
dansmith | bauzas: I could have not changed this method to the new one and used the special attribute, | 15:01 |
bauzas | dansmith: that's my thought | 15:01 |
bauzas | dansmith: but since it's a private attribute, I'm fine | 15:01 |
dansmith | but I feel like if you need the manifest, it's better to use the new method and get it passed to you | 15:01 |
bauzas | right | 15:01 |
bauzas | dansmith: honestly, I prefer your way instead of just trying to call self._version_manifest | 15:01 |
dansmith | yes, I just need to fix my super call of course! | 15:02 |
*** Sree has quit IRC | 15:02 | |
bauzas | sure, but like I said, your pattern is better than just getting self._version_manifest | 15:03 |
*** diogogmt has quit IRC | 15:03 | |
bauzas | I should leave a comment to that once you land PS6 | 15:03 |
*** diogogmt_ has joined #openstack-nova | 15:03 | |
bauzas | because some could argue that you just need to call that attr directly | 15:03 |
bauzas | dansmith: ^ | 15:03 |
dansmith | bauzas: some would be wrong in that case :) | 15:04 |
bauzas | heh | 15:04 |
*** thangp has joined #openstack-nova | 15:04 | |
*** agireud has joined #openstack-nova | 15:05 | |
*** csd has quit IRC | 15:05 | |
*** electrocucaracha has joined #openstack-nova | 15:06 | |
mriedem | danpb: you must not have seen this https://review.openstack.org/#/c/229577/ | 15:07 |
*** andrewbogott has quit IRC | 15:07 | |
*** andrewbogott has joined #openstack-nova | 15:07 | |
openstackgerrit | Paul Carlton proposed openstack/nova-specs: Blueprint to migrate libvirt volumes https://review.openstack.org/232053 | 15:08 |
danpb | mriedem: hmm, no | 15:08 |
danpb | mriedem: but i think mine is better, as we should not remove the NEXT_MIN_LIBVIRT_VERSION infrastructure | 15:09 |
danpb | we need that on an ongoing basis | 15:09 |
mriedem | yeah yours is more comprehensive | 15:09 |
*** tjones has quit IRC | 15:09 | |
*** garyk has quit IRC | 15:09 | |
openstackgerrit | Dan Smith proposed openstack/nova: Remove obj_relationships from objects https://review.openstack.org/231271 | 15:10 |
openstackgerrit | Dan Smith proposed openstack/nova: Clean up tests for dropping obj_relationships https://review.openstack.org/231270 | 15:10 |
openstackgerrit | Dan Smith proposed openstack/nova: DNM: Canary test the backporting from a liberty compute https://review.openstack.org/231486 | 15:10 |
openstackgerrit | Dan Smith proposed openstack/nova: Fix up Service object for manifest-based backports https://review.openstack.org/231269 | 15:10 |
openstackgerrit | Dan Smith proposed openstack/nova: DNM: Drop Instance v1.x support https://review.openstack.org/220690 | 15:10 |
*** manishg has joined #openstack-nova | 15:10 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Add testresources used by oslo.db fixture https://review.openstack.org/232055 | 15:12 |
*** hemnafk is now known as hemna | 15:12 | |
*** Murali has quit IRC | 15:12 | |
*** njirap has quit IRC | 15:13 | |
*** jistr is now known as jistr|mtg | 15:14 | |
*** armax has joined #openstack-nova | 15:14 | |
*** tsekiyama has quit IRC | 15:16 | |
*** Daisy has quit IRC | 15:17 | |
*** jistr|mtg is now known as jistr | 15:17 | |
*** claudiub has quit IRC | 15:17 | |
*** Daisy has joined #openstack-nova | 15:17 | |
*** subscope has quit IRC | 15:18 | |
*** Daisy has quit IRC | 15:19 | |
*** Daisy has joined #openstack-nova | 15:19 | |
*** njirap has joined #openstack-nova | 15:20 | |
mriedem | danpb: you don't have anything up for review to update the hypervisor support matrix about required min libvirt version for admin password support do you? if not, i'll do that. | 15:20 |
mriedem | i kind of wish that wasn't landed | 15:20 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Remove unused context parameter from db.archive_deleted_rows* methods https://review.openstack.org/232057 | 15:21 |
*** claudiub has joined #openstack-nova | 15:21 | |
*** mdrabe has quit IRC | 15:22 | |
danpb | mriedem: nope, not don't anything | 15:23 |
danpb | argh | 15:23 |
danpb | not done anything | 15:23 |
openstackgerrit | Paul Carlton proposed openstack/nova-specs: Blueprint to use libvirt storage pools https://review.openstack.org/232045 | 15:25 |
*** EinstCrazy has joined #openstack-nova | 15:25 | |
*** subscope has joined #openstack-nova | 15:28 | |
*** annegentle has joined #openstack-nova | 15:29 | |
*** electrocucaracha has quit IRC | 15:30 | |
*** electrocucaracha has joined #openstack-nova | 15:31 | |
dansmith | bauzas: https://review.openstack.org/#/c/231269/6/nova/objects/service.py,cm | 15:31 |
bauzas | dansmith: yup, just commenting a few nits and double-checking the test | 15:31 |
*** jdurgin1 has joined #openstack-nova | 15:31 | |
dansmith | cool | 15:32 |
jwcroppe | mriedem: any chance we can pull https://bugs.launchpad.net/nova/+bug/1490837 into L? | 15:32 |
openstack | Launchpad bug 1490837 in OpenStack Compute (nova) "Sporadic incoherent metrics when driver.get_host_cpu_stats takes longer than 1 second to execute" [Medium,Fix committed] - Assigned to Joe Cropper (jwcroppe) | 15:32 |
mriedem | umm | 15:33 |
mriedem | sdague: ^ what do you think about that for stable? | 15:33 |
*** dims_ has quit IRC | 15:34 | |
*** EinstCrazy has quit IRC | 15:34 | |
*** nkrinner has quit IRC | 15:36 | |
*** moshele has joined #openstack-nova | 15:37 | |
*** mudassirlatif has quit IRC | 15:37 | |
*** mpaolino has joined #openstack-nova | 15:38 | |
*** mdrabe has joined #openstack-nova | 15:39 | |
openstackgerrit | Alexey I. Froloff proposed openstack/nova: Fix more inconsistency between Nova-Net and Neutron https://review.openstack.org/226274 | 15:40 |
*** kaisers has quit IRC | 15:40 | |
*** njirap has quit IRC | 15:40 | |
lxsli | johnthetubaguy: if I wanted to adopt an old bp, should I just make a new one? | 15:40 |
sdague | mriedem: what's the impact if we don't bring it back? | 15:41 |
mriedem | jwcroppe: ^ | 15:41 |
dansmith | metrics will be skewed a little | 15:41 |
dansmith | when we're under high load | 15:41 |
*** david-lyle has quit IRC | 15:41 | |
dansmith | I think | 15:41 |
dansmith | not the end of the world, IMHO, just means a scheduler filter might make a less than optimal decision, AFAIK | 15:42 |
dansmith | we could wait for a bug where someone says it's really a problem | 15:42 |
*** david-lyle has joined #openstack-nova | 15:42 | |
jwcroppe | right, metrics are messed up when systems under load | 15:42 |
bauzas | dansmith: just one comment puzzling me in https://review.openstack.org/#/c/231269/6/nova/tests/unit/objects/test_service.py,cm | 15:42 |
johnthetubaguy | lxsli: ideally no, just re-use the old one | 15:42 |
*** links has quit IRC | 15:42 | |
mriedem | well, there was a bug for the metrics thing https://bugs.launchpad.net/nova/+bug/1490837 | 15:43 |
openstack | Launchpad bug 1490837 in OpenStack Compute (nova) "Sporadic incoherent metrics when driver.get_host_cpu_stats takes longer than 1 second to execute" [Medium,Fix committed] - Assigned to Joe Cropper (jwcroppe) | 15:43 |
johnthetubaguy | lxsli: I should be able to assign it to you, I think | 15:43 |
bauzas | dansmith: since you call obj_tree_get_versions with ComputeNode being a nested object, you should already have the key set unless I'm wrong? | 15:43 |
mriedem | i just wasn't on the master review so wasn't sure how much is in there that is unsafe for backporting to stable | 15:43 |
mriedem | since it's not a trivial change | 15:43 |
bauzas | dansmith: or do you want to force a specific version ? | 15:43 |
dansmith | mriedem: right, that's his bug | 15:43 |
dansmith | mriedem: I meant a bug from someone running liberty where they say "this is important because ..." | 15:44 |
*** kaisers has joined #openstack-nova | 15:44 | |
*** dims_ has joined #openstack-nova | 15:44 | |
mriedem | someone other than jwcroppe :) | 15:45 |
jwcroppe | lol | 15:45 |
bauzas | mriedem: sdague: I feel it's not really mandatory for liberty | 15:45 |
*** marcusvrn_ has quit IRC | 15:45 | |
bauzas | speaking of bug 1490837 | 15:45 |
openstack | bug 1490837 in OpenStack Compute (nova) "Sporadic incoherent metrics when driver.get_host_cpu_stats takes longer than 1 second to execute" [Medium,Fix committed] https://launchpad.net/bugs/1490837 - Assigned to Joe Cropper (jwcroppe) | 15:45 |
dansmith | mriedem: no offense to jwcroppe, you get my point, right? :) | 15:45 |
mriedem | unless you have a scheduler filter or metrics plugin out of tree that is completely fubar w/o the fix | 15:45 |
mriedem | right? | 15:45 |
bauzas | mriedem: there is only one filter | 15:45 |
bauzas | mriedem: and one weigher | 15:45 |
bauzas | mriedem: consuming those stats | 15:45 |
mriedem | you can't drop anything in there? | 15:45 |
jwcroppe | dansmith: I thought our tentacle-less bond counted for something, man! | 15:45 |
bauzas | mriedem: the filter being there just for checking that we have the metrics | 15:46 |
dansmith | jwcroppe: that sounds.. creepy :) | 15:46 |
jwcroppe | +1 | 15:46 |
bauzas | mriedem: most of the work is done by weighers | 15:46 |
mriedem | are those pluggable? | 15:46 |
jwcroppe | anyway, if mriedem feels it's too risky for L, that's fine - thought I'd ask | 15:46 |
bauzas | which means that if the weigher is a bit buggy, then it's not really bad | 15:46 |
lxsli | johnthetubaguy: great! can you assign https://blueprints.launchpad.net/nova/+spec/migrate-libvirt-volumes to paul-carlton2 please? | 15:46 |
bauzas | just that it could be not exacly the right compute | 15:46 |
jwcroppe | mriedem: I agree life won't end because it's not in L, so I'm cool with 'dat | 15:46 |
mriedem | dis and dat? | 15:47 |
bauzas | so, backporting to Liberty sounds a bit picky | 15:47 |
johnthetubaguy | lxsli: done, it will need a spec, etc, clearly, but that should be assigned now | 15:47 |
lxsli | johnthetubaguy: yep he's working on those, thank you | 15:48 |
johnthetubaguy | lxsli: :) | 15:48 |
dansmith | bauzas: replied, thanks for paying attention to the details | 15:48 |
*** Marga_ has quit IRC | 15:48 | |
*** andreykurilin__ has joined #openstack-nova | 15:49 | |
bauzas | dansmith: I see, so you're overriding the key to an older version | 15:49 |
dansmith | bauzas: yeah | 15:49 |
bauzas | that was my first thought hence the +1 | 15:49 |
andreykurilin__ | sdague: hi! can you look at small change to project-config? https://review.openstack.org/#/c/231877/ | 15:49 |
bauzas | but I needed double-checking | 15:49 |
dansmith | bauzas: in other tests, we had to do majorly confusing antics to get a single process to think it needed to backport something.. the manifest makes it super easy | 15:49 |
*** doron_afk has quit IRC | 15:50 | |
bauzas | okay, I see the relationship ('1.9', '1.10'), gotcha | 15:50 |
bauzas | that's a pretty old version but I'm fine | 15:50 |
* bauzas should read the explicit tuples before asking silly questions | 15:50 | |
*** READ10 has joined #openstack-nova | 15:51 | |
*** artom has quit IRC | 15:52 | |
*** artom has joined #openstack-nova | 15:52 | |
*** READ10 is now known as READ10|lunch | 15:53 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Use DBReferenceError in archive_deleted_rows_for_table https://review.openstack.org/232066 | 15:53 |
*** jdurgin1 has quit IRC | 15:54 | |
*** david-ly_ has joined #openstack-nova | 15:55 | |
*** david-lyle has quit IRC | 15:55 | |
*** david-ly_ has quit IRC | 15:56 | |
*** david-lyle has joined #openstack-nova | 15:56 | |
*** diana_clarke has quit IRC | 15:58 | |
*** flwang has quit IRC | 15:58 | |
*** dims__ has joined #openstack-nova | 15:59 | |
*** subscope has quit IRC | 15:59 | |
*** fawadkhaliq has quit IRC | 16:00 | |
*** fawadk has joined #openstack-nova | 16:00 | |
*** electrocucaracha has quit IRC | 16:02 | |
*** manishg has quit IRC | 16:02 | |
*** dims_ has quit IRC | 16:03 | |
*** pm90__ has quit IRC | 16:04 | |
*** tsekiyama has joined #openstack-nova | 16:04 | |
*** jistr has quit IRC | 16:05 | |
*** artom has quit IRC | 16:07 | |
*** artom has joined #openstack-nova | 16:08 | |
*** jlanoux has quit IRC | 16:08 | |
*** diana_clarke has joined #openstack-nova | 16:10 | |
*** scheuran has quit IRC | 16:10 | |
dims__ | dansmith: mriedem: one of you please bless this https://review.openstack.org/#/c/232055/ will help prevent nova breakage with next oslo.db release | 16:11 |
*** lpetrut has joined #openstack-nova | 16:12 | |
dansmith | yep | 16:12 |
*** annegentle has quit IRC | 16:12 | |
*** annegentle has joined #openstack-nova | 16:13 | |
mriedem | dims__: why doesn't nova just require oslo.db[fixtures]? | 16:13 |
*** marcusvrn_ has joined #openstack-nova | 16:13 | |
dims__ | mriedem: y that would work too | 16:13 |
*** Marga_ has joined #openstack-nova | 16:14 | |
mriedem | -1 | 16:14 |
mriedem | anyway, cinder meeting | 16:14 |
*** su_zhang has quit IRC | 16:14 | |
dansmith | mriedem: dims__: isn't this just a pre-gr-sync? | 16:14 |
dims__ | mriedem: i am afraid to do that as i don't know what tools will break | 16:14 |
dansmith | dims__: mriedem just wanted to get me a disagreement stat.. he loves that | 16:15 |
dims__ | mriedem: as i don't see that syntax yet | 16:15 |
dims__ | lol :) | 16:15 |
mriedem | i believe dhellmann said that the lib would need a major version bump for the extras stuff | 16:15 |
mriedem | and you'd have to opt into using those, so would require nova to use a new min version of oslo.db to use oslo.db[fixtures] | 16:15 |
mriedem | if that was released in a minor version, then well | 16:16 |
mriedem | shrug? | 16:16 |
mriedem | anyway, this is precisely why we started adding the extras stuff to the oslo libs | 16:16 |
dims__ | mriedem: i have not released anything yet. just trying things with master of all oslo branches and filing defensive reviews | 16:16 |
mriedem | ok, we should probably circle back with dhellmann at some point | 16:16 |
dims__ | and lifeless... | 16:17 |
dims__ | worries about the proposal bots/edit constraints etc. | 16:17 |
*** mudassirlatif has joined #openstack-nova | 16:17 | |
dims__ | s/worries/worried/ | 16:17 |
*** nic has joined #openstack-nova | 16:17 | |
*** unicell has quit IRC | 16:19 | |
mriedem | "This should trigger a major version bump for the library, when it lands." was his comment in https://review.openstack.org/#/c/217347/ | 16:19 |
*** dims__ is now known as dims | 16:19 | |
dims | mriedem: y, not worried about version bump | 16:20 |
*** amotoki has quit IRC | 16:20 | |
dims | mriedem: do i have to add an entry in g-r for this? will edit-constraints support the square brackets syntax | 16:20 |
dims | etc... | 16:20 |
mriedem | dims: i'm not totally sure about that | 16:21 |
clarkb | you dont put that in global requirements or constraints | 16:21 |
mriedem | i guess i'd see if the job fails on the nova change that tries to use it | 16:21 |
dims | mriedem: ack good point | 16:22 |
clarkb | global reqs has individual reqs not aggregates | 16:22 |
mriedem | so: oslo.db[fixtures]>=3.0? | 16:22 |
dims | clarkb: just test-requirements i guess | 16:22 |
clarkb | so you want to require optional extras? | 16:22 |
clarkb | why not unoptional them then? | 16:23 |
*** zzzeek has quit IRC | 16:23 | |
dansmith | "unoptional them then" -> awesome | 16:23 |
dims | mriedem: i am also using the "-e " syntax to try latest master https://github.com/dims/nova/blob/master/requirements.txt#L13 | 16:23 |
*** agireud has quit IRC | 16:24 | |
dims | guess, i can do a oslo.db major version bump, make a release and we have 2 ways to fix nova after that | 16:24 |
clarkb | so I think you dont have to worry about constraints at all as it looks at individual items only. global reqs may need to grok it if you list that as a dep somehwere | 16:24 |
clarkb | I think it would just work in global reqs | 16:24 |
openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Versioned notification API https://review.openstack.org/224755 | 16:25 |
dims | clarkb: ack | 16:25 |
*** agireud has joined #openstack-nova | 16:27 | |
*** apoorvad has joined #openstack-nova | 16:31 | |
*** pilgrimstack has left #openstack-nova | 16:31 | |
*** electrocucaracha has joined #openstack-nova | 16:34 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Log DBReferenceError in archive_deleted_rows_for_table https://review.openstack.org/232097 | 16:39 |
*** signed8bit has quit IRC | 16:40 | |
*** ildikov has joined #openstack-nova | 16:43 | |
*** thangp has quit IRC | 16:44 | |
*** pm90_ has joined #openstack-nova | 16:44 | |
*** belmoreira has joined #openstack-nova | 16:45 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: Expands python34 unit tests list https://review.openstack.org/231948 | 16:49 |
*** tonytan4ever has joined #openstack-nova | 16:49 | |
*** pm90__ has joined #openstack-nova | 16:49 | |
*** Murali has joined #openstack-nova | 16:51 | |
jlk | mordred: hey, is there a way to turn off caching in shade? It's making things extremely difficult to walk through a call with pdb | 16:51 |
*** achanda has joined #openstack-nova | 16:51 | |
*** pm90_ has quit IRC | 16:52 | |
*** danpb has quit IRC | 16:52 | |
*** ZZelle has joined #openstack-nova | 16:53 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Make archive_deleted_rows_for_table private https://review.openstack.org/232104 | 16:55 |
*** alaski is now known as alaski_out | 16:55 | |
*** Marga_ has quit IRC | 16:55 | |
*** exploreshaifali has joined #openstack-nova | 16:56 | |
*** garyk has joined #openstack-nova | 16:57 | |
openstackgerrit | Mike Fedosin proposed openstack/nova: Bringing Glance v2 images to a uniform format https://review.openstack.org/231696 | 16:57 |
openstackgerrit | Claudiu Belu proposed openstack/nova: Replaces contextlib.nested with test.nested https://review.openstack.org/232108 | 16:58 |
*** Murali has quit IRC | 16:58 | |
*** nic has quit IRC | 16:58 | |
mordred | jlk: it should default to off | 16:59 |
melwitt | mriedem: do your patches mean you tried out archive_deleted_rows recently? | 16:59 |
mordred | jlk: although, hrm the decorators will make the stacks weird ... lemme try something | 17:00 |
*** manishg has joined #openstack-nova | 17:00 | |
jlk | yeah I had to put my pdb trace further into the code to bypass the decorator rabbit hole | 17:00 |
mordred | jlk: boarding plane ... will have you a patch in an hour | 17:00 |
*** moshele has quit IRC | 17:01 | |
*** rk4n has joined #openstack-nova | 17:01 | |
jlk | no rush, no worries. | 17:02 |
*** penick has joined #openstack-nova | 17:02 | |
jlk | I think keystoneclient bump fucked us though | 17:02 |
*** unicell has joined #openstack-nova | 17:02 | |
*** claudiub has quit IRC | 17:03 | |
*** mpaolino has quit IRC | 17:03 | |
*** yamahata has joined #openstack-nova | 17:04 | |
*** rk4n has quit IRC | 17:05 | |
*** nic has joined #openstack-nova | 17:06 | |
*** rk4n has joined #openstack-nova | 17:06 | |
*** ctrath has quit IRC | 17:06 | |
openstackgerrit | Mike Fedosin proposed openstack/nova: Bringing Glance v2 images to a uniform format https://review.openstack.org/231696 | 17:07 |
mriedem | melwitt: this morning yeah | 17:08 |
mriedem | melwitt: it's busted as ever | 17:08 |
mriedem | melwitt: posted to the dev ML, and doing some cleanup before i actually dug into the reverse table sort | 17:08 |
melwitt | mriedem: bummer :( | 17:08 |
*** krtaylor has quit IRC | 17:09 | |
openstackgerrit | Mike Fedosin proposed openstack/nova: use helper method to remove version hardcoding https://review.openstack.org/228578 | 17:10 |
*** achanda has quit IRC | 17:11 | |
*** su_zhang has joined #openstack-nova | 17:11 | |
mriedem | i kind of like that the novaclient functional tests rely on a devstack so we can boot a real instance | 17:12 |
*** ccarmack1 has quit IRC | 17:12 | |
mriedem | i'm kind of wishing the nova functional tests required a devstack now, or if we had a nova integration or dsvm tests or something | 17:12 |
*** READ10|lunch is now known as READ10 | 17:12 | |
mriedem | because i think the only good way to probably test db archive is with devstack, and we don't have that in the nova tree, tempest wouldn't take it and it doesn't fit in novaclient | 17:13 |
mriedem | could maybe do it in nova functional tests if we monkey with sqlite | 17:13 |
*** tsekiyama has quit IRC | 17:13 | |
*** achanda has joined #openstack-nova | 17:13 | |
*** ctrath has joined #openstack-nova | 17:14 | |
*** sahid has quit IRC | 17:14 | |
*** mnestratov has quit IRC | 17:18 | |
*** ccarmack has joined #openstack-nova | 17:19 | |
mriedem | dansmith: johnthetubaguy: if i want to make a change to nova-manage db archive_deleted_rows to add a --verbose option and dump the tables that were touched along with how many rows were deleted before exit, does that require a specless bp? | 17:19 |
dansmith | mriedem: it'd be like a single patch right? | 17:20 |
mriedem | yeah | 17:20 |
mriedem | just stash results in a dict and pretty print them on exit | 17:20 |
dansmith | I'd say no, but I think he might want it for recording the feature for release notes, etc | 17:20 |
mriedem | ok | 17:20 |
*** subscope has joined #openstack-nova | 17:21 | |
openstackgerrit | Nikola Dipanov proposed openstack/nova-specs: spec document for sriov-physical-function-passthrough blueprint https://review.openstack.org/212472 | 17:22 |
*** krtaylor has joined #openstack-nova | 17:23 | |
*** lucasagomes is now known as lucas-dinner | 17:23 | |
*** ijw has joined #openstack-nova | 17:24 | |
mriedem | dansmith: https://blueprints.launchpad.net/nova/+spec/print-table-archived-rows | 17:24 |
*** changbl has quit IRC | 17:25 | |
mriedem | thanks | 17:26 |
*** mudassirlatif has quit IRC | 17:28 | |
*** achanda has quit IRC | 17:28 | |
*** jcoufal has quit IRC | 17:28 | |
*** jcoufal has joined #openstack-nova | 17:29 | |
*** e0ne has quit IRC | 17:30 | |
mriedem | heh, nova-manage vm list shows all instances, deleted or not | 17:31 |
*** Piet has quit IRC | 17:31 | |
markus_z | mriedem: IIRC there was a but about the nova-manage issue, sec... | 17:32 |
*** Marga_ has joined #openstack-nova | 17:33 | |
markus_z | mriedem: https://bugs.launchpad.net/nova/+bug/1494841 | 17:33 |
openstack | Launchpad bug 1494841 in OpenStack Compute (nova) "nova-manage vm list active" [Low,In progress] - Assigned to Raga lahari (raga-lahari) | 17:33 |
* markus_z has to leave, bye | 17:34 | |
*** markus_z has quit IRC | 17:34 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Make archive_deleted_rows_for_table private https://review.openstack.org/232104 | 17:34 |
*** DRoBeR has quit IRC | 17:34 | |
mordred | jlk: oh? I take it you're trying ot debug to figure out why it broke? (I certainly did not intend to break things - but we did move to keystoneauth1 for getting Sessions so the auth plugins might have a slight difference in behavior that is affecting you but which we did not account for) | 17:34 |
johnthetubaguy | mriedem: dansmith: I don't mind either way for that little one, its nice to advertise the feature though, and I would approve a blueprint if you add it. | 17:36 |
mriedem | dan got it | 17:36 |
*** lpetrut has quit IRC | 17:37 | |
jlk | mordred: yeah. shade glance image list is passing along a "insecure" kwarg, that when it eventually hits keystone client code, the client code throws an exception on unexpected kwarg "insecure" | 17:37 |
mordred | hrm | 17:38 |
jlk | but I'm in a call now and can't debug at the moment | 17:38 |
mordred | jlk: I'm going to loop in Shrews in a different channel - what version of shade are you on? latest? | 17:39 |
jlk | mordred: not sure | 17:39 |
mordred | kk | 17:39 |
*** signed8bit has joined #openstack-nova | 17:39 | |
jlk | on ab1c566cb8afbf477492180e2cc8257817a06893 it seems | 17:39 |
jlk | which, looks "oldish" | 17:39 |
mordred | we're not seeing that in infra which uses shade for images | 17:39 |
mordred | oh. yes. very | 17:40 |
*** electrocucaracha has quit IRC | 17:40 | |
*** vilobhmm11 has joined #openstack-nova | 17:40 | |
mordred | if you got updated client libs and old shade I think that's untested - but makes me think we should maybe think about how to express that | 17:40 |
jlk | yeah I had my automation pinned for various reasons. Trying master | 17:41 |
*** zul has quit IRC | 17:42 | |
*** vilobhmm111 has joined #openstack-nova | 17:42 | |
*** zul has joined #openstack-nova | 17:42 | |
*** yamahata has quit IRC | 17:42 | |
*** vishwanathj has joined #openstack-nova | 17:43 | |
*** LukeHinds has quit IRC | 17:45 | |
*** vilobhmm11 has quit IRC | 17:45 | |
*** rlrossit has quit IRC | 17:46 | |
*** electrocucaracha has joined #openstack-nova | 17:47 | |
*** jichen has joined #openstack-nova | 17:48 | |
*** electrocucaracha has quit IRC | 17:51 | |
*** dims has quit IRC | 17:52 | |
*** dims has joined #openstack-nova | 17:53 | |
*** mriedem has quit IRC | 17:54 | |
*** raildo is now known as raildo-afk | 17:59 | |
*** achanda has joined #openstack-nova | 18:00 | |
*** rlrossit has joined #openstack-nova | 18:00 | |
*** dims_ has joined #openstack-nova | 18:00 | |
*** e0ne has joined #openstack-nova | 18:03 | |
*** pkoniszewski has quit IRC | 18:03 | |
*** dims has quit IRC | 18:04 | |
*** raildo-afk is now known as raildo | 18:04 | |
*** xek has quit IRC | 18:07 | |
*** _gryf has quit IRC | 18:07 | |
*** rfolco has quit IRC | 18:07 | |
*** xek has joined #openstack-nova | 18:08 | |
*** dims_ has quit IRC | 18:08 | |
*** artom has quit IRC | 18:10 | |
*** pkoniszewski has joined #openstack-nova | 18:10 | |
*** tsekiyama has joined #openstack-nova | 18:11 | |
*** mriedem has joined #openstack-nova | 18:11 | |
*** rfolco has joined #openstack-nova | 18:12 | |
*** e0ne has quit IRC | 18:12 | |
*** ccesario_ has quit IRC | 18:14 | |
*** su_zhang has quit IRC | 18:14 | |
*** READ10 has quit IRC | 18:15 | |
*** belmoreira has quit IRC | 18:15 | |
*** pixelbeat has quit IRC | 18:16 | |
*** Daisy has quit IRC | 18:17 | |
*** Daisy has joined #openstack-nova | 18:18 | |
*** e0ne has joined #openstack-nova | 18:18 | |
*** jcoufal has quit IRC | 18:20 | |
*** pkoniszewski has quit IRC | 18:20 | |
*** isq_ has quit IRC | 18:21 | |
*** _gryf has joined #openstack-nova | 18:22 | |
*** Daisy has quit IRC | 18:22 | |
openstackgerrit | Dan Smith proposed openstack/nova: Add test of claim context manager abort https://review.openstack.org/213591 | 18:24 |
*** pkoniszewski has joined #openstack-nova | 18:25 | |
dansmith | bauzas: ^ | 18:25 |
*** aix has quit IRC | 18:26 | |
*** emagana has quit IRC | 18:30 | |
bauzas | dansmith: +1(d | 18:30 |
bauzas | uh | 18:30 |
bauzas | FWIW, +1'd even | 18:30 |
dansmith | cool | 18:30 |
*** achanda has quit IRC | 18:30 | |
*** irenab has joined #openstack-nova | 18:32 | |
*** su_zhang has joined #openstack-nova | 18:34 | |
*** isq_ has joined #openstack-nova | 18:36 | |
*** ociuhandu has quit IRC | 18:38 | |
*** garyk1 has joined #openstack-nova | 18:38 | |
*** garyk has quit IRC | 18:38 | |
openstackgerrit | Gábor Antal proposed openstack/nova: Added appender methods to remove clones from config.py https://review.openstack.org/232145 | 18:39 |
*** dims has joined #openstack-nova | 18:40 | |
*** lalitd has joined #openstack-nova | 18:40 | |
*** yassine has quit IRC | 18:41 | |
*** emagana has joined #openstack-nova | 18:41 | |
lalitd | bauzas: hi | 18:41 |
openstackgerrit | Mike Fedosin proposed openstack/nova: Bringing Glance v2 images to a uniform format https://review.openstack.org/231696 | 18:42 |
*** tsekiyam_ has joined #openstack-nova | 18:43 | |
*** tsekiyam_ has quit IRC | 18:43 | |
*** Piet has joined #openstack-nova | 18:44 | |
*** tsekiyama has quit IRC | 18:46 | |
*** mtanino has quit IRC | 18:46 | |
bauzas | lalitd: hi | 18:46 |
bauzas | lalitd: so, could you please point me out the change requiring attention ? | 18:47 |
*** electrocucaracha has joined #openstack-nova | 18:47 | |
lalitd | bauzas: ya, actually the grenade and tempest tests are failing in the spec https://review.openstack.org/#/c/213041/ | 18:48 |
*** tsekiyama has joined #openstack-nova | 18:48 | |
* bauzas looking | 18:48 | |
bauzas | lalitd: okay, so you have a problem with an upgrade from liberty | 18:49 |
bauzas | sec | 18:49 |
lalitd | in earlier patch set same failures were there , we had discussion on that , it was due to configuration missing for api_db in grenade | 18:49 |
bauzas | mriedem: dansmith: do you know if grenade and n-cpu are updated to use liberty as the original branch ? | 18:49 |
dansmith | yes | 18:50 |
lalitd | bauzas: ^ | 18:50 |
bauzas | dansmith: okay, I'll look when the change landed | 18:50 |
dansmith | bauzas: but I don't think we've landed the patch to actually get the api_db db_sync run yet | 18:50 |
bauzas | lalitd: lemme see when Jenkins ran | 18:50 |
dansmith | because we haven't used it | 18:50 |
dansmith | bauzas: https://review.openstack.org/#/c/190399/ | 18:50 |
lalitd | bauzas: cool | 18:50 |
bauzas | lalitd: see dansmith's change | 18:51 |
*** achanda has joined #openstack-nova | 18:51 | |
*** pixelbeat has joined #openstack-nova | 18:51 | |
bauzas | lalitd: I haven't yet checked the jobs, but until 190399 is merged, that should be normal that it would fail | 18:52 |
*** Marga_ has quit IRC | 18:54 | |
lalitd | bauzas: oks, 190399 is on high priority :) | 18:54 |
*** ntpttr has joined #openstack-nova | 18:56 | |
lalitd | bauzas: so probably when it ll get merge? any idea | 18:57 |
*** Marga_ has joined #openstack-nova | 18:57 | |
dansmith | I will update it for liberty | 18:57 |
dansmith | if we need it, we can merge it | 18:57 |
dansmith | but we need to probably discuss that flavor thing first | 18:57 |
*** pixelbeat has quit IRC | 18:58 | |
bauzas | lalitd: yup confirmed http://logs.openstack.org/41/213041/18/check/gate-grenade-dsvm-partial-ncpu/c604874/logs/new/screen-n-api.txt.gz#_2015-09-29_12_56_36_414 | 18:59 |
*** ctrath has quit IRC | 18:59 | |
*** ctrath has joined #openstack-nova | 19:00 | |
*** electrocucaracha has quit IRC | 19:00 | |
bauzas | dansmith: we had a convo in the last cells meeting about the next things, and until alaski agrees or not, we're thinking to work on data online migrations to the api db for the global tables | 19:01 |
*** flwang has joined #openstack-nova | 19:01 | |
lalitd | bauzas: thanks , nice job.. so it is connection missing in configuration | 19:01 |
*** Marga_ has quit IRC | 19:02 | |
bauzas | that should be the liberty focus for cells - that and some sched bits (persisting the spec at least, plus if possible running a conductor) | 19:02 |
*** achanda has quit IRC | 19:02 | |
bauzas | that still needs alaski's comments | 19:02 |
*** su_zhang has quit IRC | 19:02 | |
*** ntpttr has quit IRC | 19:02 | |
*** rdopiera_afk is now known as rdopiera | 19:02 | |
dansmith | bauzas: I feel like the migration from the lower DB to the API one could be done mostly offline and ahead of time | 19:03 |
bauzas | dansmith: sounds good, we were mostly discussing about the process, ie. how many specs we should be providing | 19:04 |
dansmith | bauzas: like, an initial sync before restarting anything | 19:04 |
dansmith | we should use service_versions to denote this change and stop looking at the non-API database after the minimum has lifted | 19:04 |
bauzas | dansmith: that would ease the migration, I was thinking about the general pattern of expand/live/contract | 19:04 |
bauzas | mmm interesting | 19:05 |
bauzas | melwitt: ^ | 19:05 |
dansmith | yeah, the reason we want to be careful here is just because it requires hitting two databases I think | 19:05 |
*** ntpttr has joined #openstack-nova | 19:05 | |
dansmith | but we can control whether we look in both places based on the service version minimum | 19:05 |
dansmith | I think | 19:06 |
*** ociuhandu has joined #openstack-nova | 19:06 | |
* bauzas thinks about that | 19:06 | |
dansmith | definitely good discussion material for tokyo.. too bad alaski won't be there :? | 19:06 |
dansmith | er -> :/ | 19:06 |
*** rdopiera has quit IRC | 19:07 | |
dansmith | I wonder if there is any reason we could not require the initial sync, | 19:08 |
dansmith | hmm, well, no | 19:08 |
dansmith | requires some thinking for sure | 19:09 |
bauzas | yup | 19:09 |
dansmith | we can't do the migration in a transaction, which makes it harder I think | 19:09 |
bauzas | I mean, we have 2 facades here | 19:09 |
bauzas | the object facade, but we also added some db facade | 19:09 |
*** matrohon has joined #openstack-nova | 19:09 | |
dansmith | I don't think I get what you just said | 19:10 |
bauzas | sorry | 19:10 |
bauzas | thinking out loud | 19:10 |
*** dims has quit IRC | 19:10 | |
melwitt | yeah, it hasn't been clear to me how to move forward with the api database in general. I've been trying to help review the flavor patches and realized I don't know what the plan is | 19:10 |
bauzas | so the general idea is to move what's api related to the api db, and leave the instance related bits to the existing db | 19:11 |
bauzas | which means that we should care about migrating some objects from the legacy db (now called cell db) to the api db | 19:12 |
bauzas | and stop having to sync between 2 DBs like we did with cells v1 and all the lovely races we have | 19:12 |
mriedem | melwitt: have you ever noticed that novaclient.utils.print_dict is basically the same as novaclient.common.cliutils.print_dict? | 19:13 |
bauzas | so IIUC, the patch is caring about migrating the data | 19:13 |
*** dims has joined #openstack-nova | 19:13 | |
dansmith | bauzas: yeah, we get that | 19:13 |
dansmith | I'm just thinking about the mechanics | 19:13 |
melwitt | yeah, same | 19:13 |
melwitt | mriedem: I have not | 19:14 |
* alaski_out pops in briefly | 19:14 | |
bauzas | alaski_out: go away ! | 19:14 |
alaski_out | I'm still at home, nothing exciting going on yet | 19:14 |
bauzas | hah | 19:14 |
bauzas | so the mechanics seem to be described in http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/flavor-cell-api.html | 19:15 |
*** fawadk has quit IRC | 19:16 | |
dansmith | well, | 19:16 |
alaski_out | the idea we discussed for flavors was to do an online migration as things were accessed, and then include a nova-manage command for final migration | 19:16 |
dansmith | right I get that | 19:16 |
alaski_out | much the same as the system_metadata -> instance_extra flavor thing | 19:16 |
dansmith | I was kinda hoping we could avoid having to roll all API and conductor workers | 19:16 |
dansmith | this time | 19:16 |
dansmith | but I think we'll have to, because we can't have old api workers writing to the old location while we're trying to migrate | 19:16 |
*** tonytan4ever has quit IRC | 19:17 | |
dansmith | if we roll api and conductor together, then the old location is read-only and all updates go to the API DB | 19:17 |
*** tsekiyama has quit IRC | 19:17 | |
dansmith | that's relatively trivial, so not a problem | 19:17 |
alaski_out | well, we can have old writing to the old place as long as updated workers read from the old place first | 19:17 |
dansmith | I was just hoping to be able to push the envelope again | 19:17 |
bauzas | that's debatable | 19:17 |
dansmith | alaski_out: yeah, that's just less desirable I think | 19:17 |
*** tsekiyama has joined #openstack-nova | 19:17 | |
alaski_out | it is extra lookups | 19:18 |
bauzas | hence the lookup in the spec | 19:18 |
dansmith | alaski_out: we can't null out the old locations either, | 19:18 |
bauzas | hah, jinxed by alaski_out | 19:18 |
dansmith | which means we have to always read from both places | 19:18 |
bauzas | yup | 19:18 |
bauzas | at least the old one | 19:18 |
mriedem | dims: here is a fun little one https://bugs.launchpad.net/oslo-incubator/+bug/1503841 | 19:18 |
openstack | Launchpad bug 1503841 in python-novaclient "novaclient.utils.print_dict should be replaced with novaclient.common.cliutils.print_dict" [Undecided,New] | 19:18 |
alaski_out | yes, until some cutoff time. and a full upgrade would have to happen before that | 19:18 |
bauzas | yup | 19:19 |
dansmith | alaski_out: we don't have service records for the api nodes, so we don't really have any indication | 19:19 |
dansmith | alaski_out: seems like just rolling api and conductor together (like you have to today anyway) is the easiest | 19:19 |
alaski_out | yeah, that would definitely be easier and avoid a lot of pitfalls | 19:19 |
dansmith | actually, I guess we have to read from both places all the time? | 19:19 |
dansmith | hmm | 19:20 |
dansmith | if we need to list all flavors and some are not migrated, | 19:20 |
dansmith | then we have to list from both | 19:20 |
alaski_out | yes, that's what's been implemented | 19:20 |
dansmith | hrm | 19:20 |
alaski_out | it's not ideal | 19:20 |
dansmith | so we could then have some nova-manage command that does the background sync, | 19:20 |
dansmith | and then once it's synced, it could delete the flavors from the old db, | 19:21 |
dansmith | and have our objects stop looking at the old location if there are zero flavors | 19:21 |
dansmith | and cache that on the first lookup or something | 19:21 |
dansmith | so that once you're migrated, your overhead is pretty low | 19:21 |
alaski_out | I think the spec covered most of that, but didn't address when to stop looking at the old db | 19:22 |
dansmith | then you could use that same tool as pre-sync to seed the api database and get there quicker | 19:22 |
dansmith | okay | 19:22 |
dansmith | I haven't really read it | 19:22 |
alaski_out | but that last bit would be a great addition | 19:22 |
dansmith | oh also I have a super idea | 19:22 |
dansmith | alaski_out: gtfo of here | 19:22 |
dansmith | any +2s? | 19:22 |
alaski_out | lol | 19:23 |
alaski_out | my time is limited now, just trying to make the most of it | 19:23 |
dansmith | alaski_out: stop it | 19:23 |
melwitt | alaski_out: (sorry) for the simple boot with one cell, this is supposed to happen before we deal with finishing the api db? | 19:23 |
*** tonytan4ever has joined #openstack-nova | 19:24 | |
*** annegentle has quit IRC | 19:24 | |
melwitt | I got confused thinking that the single cell would be one api db with one cell db | 19:24 |
alaski_out | melwitt: right. we can continue to query for things in the cell db for now, since there's just one | 19:24 |
*** annegentle has joined #openstack-nova | 19:25 | |
*** nelsnels_ has joined #openstack-nova | 19:25 | |
alaski_out | we can't get to multiple cells until we transition everything to the api db, but we can transition boot and other things along the way | 19:25 |
*** Marga_ has joined #openstack-nova | 19:25 | |
*** vishwana_ has joined #openstack-nova | 19:25 | |
bauzas | right | 19:25 |
melwitt | alaski_out: okay. I had misunderstood that. and for the other global api db tables, do we expect the specs and migrations to be like flavors? | 19:25 |
*** nelsnelson has quit IRC | 19:25 | |
dansmith | alaski_out: http://i1.kym-cdn.com/photos/images/facebook/000/588/827/f67.png | 19:25 |
*** ijw_ has joined #openstack-nova | 19:25 | |
*** manishg_ has joined #openstack-nova | 19:25 | |
*** e0ne has quit IRC | 19:25 | |
alaski_out | dansmith: two seconds and then I swear I'm gone | 19:26 |
dansmith | TIME | 19:26 |
alaski_out | figurative seconds | 19:26 |
dansmith | alaski_out: I have her number, I'll call her and get you in all kinds of trouble | 19:27 |
dansmith | don't make me | 19:27 |
*** manishg has quit IRC | 19:27 | |
alaski_out | melwitt: if they can be like flavors, yes. with the addition that dansmith suggested. but we'll have to consider them case by case to ensure that method will work | 19:27 |
*** vishwanathj has quit IRC | 19:27 | |
*** ijw has quit IRC | 19:27 | |
*** dims has quit IRC | 19:28 | |
alaski_out | melwitt: I'm going to volunteer dansmith to field any further questions for a few days :) | 19:28 |
alaski_out | dansmith: alright, I'm out. no call necessary :p | 19:28 |
dansmith | yeah, I'm happy to help figure out migrations | 19:28 |
* alaski_out goes poof | 19:28 | |
*** dims has joined #openstack-nova | 19:28 | |
bauzas | hehe | 19:28 |
dansmith | next time I'm going to pull out my chanops and ban his ass | 19:29 |
melwitt | alaski_out: thanks | 19:29 |
*** Marga_ has quit IRC | 19:29 | |
*** Vimal has joined #openstack-nova | 19:30 | |
*** Piet has quit IRC | 19:30 | |
dims | mriedem: there's not going to be any oslo-incubator stuff. trying to nuke it :) | 19:30 |
mriedem | dims: where is cliutils going then? | 19:31 |
*** fawadkhaliq has joined #openstack-nova | 19:31 | |
mriedem | b/c i'm pushing a change for it right now :) | 19:31 |
mriedem | 2 changes actually | 19:32 |
*** changbl has joined #openstack-nova | 19:32 | |
dims | mriedem: it's ok to push till we make an official decision. | 19:32 |
*** su_zhang has joined #openstack-nova | 19:33 | |
mriedem | kewl | 19:33 |
mriedem | which brings up another thing i have to post to the ML here at some point | 19:33 |
mriedem | mask_password should move back into oslo-incubator :) | 19:33 |
mriedem | because of backports, https://review.openstack.org/#/c/220622/ | 19:33 |
*** gszasz has quit IRC | 19:34 | |
dims | mriedem: will wait for the email :) | 19:34 |
*** electrocucaracha has joined #openstack-nova | 19:36 | |
*** pixelbeat has joined #openstack-nova | 19:37 | |
*** su_zhang has quit IRC | 19:37 | |
mriedem | done | 19:38 |
*** lalitd has quit IRC | 19:38 | |
*** ntpttr has left #openstack-nova | 19:39 | |
*** tsekiyama has quit IRC | 19:40 | |
vilobhmm111 | alaski_out : ping | 19:42 |
*** e0ne has joined #openstack-nova | 19:43 | |
*** otter768_ has joined #openstack-nova | 19:44 | |
*** otter768 has quit IRC | 19:44 | |
*** sambetts has quit IRC | 19:45 | |
*** sdague_ has joined #openstack-nova | 19:47 | |
*** inara` has quit IRC | 19:47 | |
*** afazekas has quit IRC | 19:47 | |
*** jamiec has quit IRC | 19:47 | |
*** sc68cal has quit IRC | 19:47 | |
*** IanGovett has quit IRC | 19:47 | |
*** sdague has quit IRC | 19:47 | |
*** dhellmann has quit IRC | 19:47 | |
*** salv-orlando has joined #openstack-nova | 19:47 | |
*** dhellmann has joined #openstack-nova | 19:47 | |
*** apoorvad has quit IRC | 19:47 | |
*** sdague_ is now known as sdague | 19:47 | |
*** sc68cal has joined #openstack-nova | 19:48 | |
*** sambetts has joined #openstack-nova | 19:48 | |
*** inara has joined #openstack-nova | 19:48 | |
*** jamiec has joined #openstack-nova | 19:48 | |
*** IanGovett has joined #openstack-nova | 19:48 | |
vilobhmm111 | dansmith : ping | 19:49 |
dansmith | vilobhmm111: in the middle of something right now, sorry | 19:49 |
vilobhmm111 | ok later then…thanks for the review on https://review.openstack.org/#/c/222423/ wanted to discuss something regarding the same…will talk later when you are free | 19:49 |
vilobhmm111 | dansmith : ^^ | 19:50 |
*** apoorvad has joined #openstack-nova | 19:51 | |
*** markvoelker has quit IRC | 19:51 | |
mriedem | ccarmack: regular delete has this state check https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1919 | 19:52 |
mriedem | ccarmack: force delete doesn't have those https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1959 | 19:52 |
ccarmack | mriedem: thanks | 19:53 |
ccarmack | I'm trying to review https://review.openstack.org/#/c/163798/ | 19:53 |
*** tsekiyama has joined #openstack-nova | 19:54 | |
*** afazekas has joined #openstack-nova | 19:55 | |
*** markvoelker has joined #openstack-nova | 19:55 | |
*** yassine has joined #openstack-nova | 19:55 | |
*** pixelbeat has quit IRC | 19:57 | |
*** garyk1 has quit IRC | 19:59 | |
*** penick has quit IRC | 19:59 | |
*** thumpba has quit IRC | 19:59 | |
*** penick has joined #openstack-nova | 19:59 | |
*** penick has quit IRC | 19:59 | |
*** ctrath has quit IRC | 20:02 | |
mriedem | yeah i gave up on that, was hoping alaski_out would get it straightened out | 20:04 |
mriedem | did mrs alaski go into labor yet? | 20:05 |
mriedem | i missed something | 20:05 |
*** e0ne has quit IRC | 20:05 | |
*** ctrath has joined #openstack-nova | 20:06 | |
*** tsekiyama has quit IRC | 20:09 | |
*** electrocucaracha has quit IRC | 20:10 | |
*** electrocucaracha has joined #openstack-nova | 20:10 | |
*** pm90__ has quit IRC | 20:11 | |
*** tsekiyama has joined #openstack-nova | 20:12 | |
*** emagana has quit IRC | 20:13 | |
*** emagana has joined #openstack-nova | 20:15 | |
*** jichen has quit IRC | 20:19 | |
*** rfolco has quit IRC | 20:20 | |
*** BobBall has quit IRC | 20:20 | |
doffm | ccarmack: https://etherpad.openstack.org/p/mitaka-nova-spec-review-tracking | 20:21 |
*** smatzek has quit IRC | 20:21 | |
*** BobBall has joined #openstack-nova | 20:22 | |
*** BobBall is now known as Guest4559 | 20:22 | |
*** rotbeard has quit IRC | 20:23 | |
*** Marga_ has joined #openstack-nova | 20:26 | |
*** seanw1 has quit IRC | 20:30 | |
*** seanw1 has joined #openstack-nova | 20:31 | |
*** Marga_ has quit IRC | 20:31 | |
*** rfolco has joined #openstack-nova | 20:33 | |
*** achanda has joined #openstack-nova | 20:34 | |
*** achanda has quit IRC | 20:34 | |
*** matrohon has quit IRC | 20:35 | |
*** mwagner_ has quit IRC | 20:36 | |
*** rfolco has quit IRC | 20:37 | |
*** rfolco has joined #openstack-nova | 20:38 | |
*** exploreshaifali has quit IRC | 20:39 | |
*** achanda has joined #openstack-nova | 20:41 | |
*** andreykurilin__ has quit IRC | 20:42 | |
*** salv-orlando has quit IRC | 20:42 | |
*** tsekiyama has quit IRC | 20:43 | |
*** rk4n has quit IRC | 20:44 | |
*** tsekiyama has joined #openstack-nova | 20:45 | |
*** Marga_ has joined #openstack-nova | 20:49 | |
*** moshele has joined #openstack-nova | 20:50 | |
*** su_zhang has joined #openstack-nova | 20:52 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable options for oslo.reports https://review.openstack.org/216917 | 20:53 |
*** tsekiyama has quit IRC | 20:55 | |
sdague | ok, so can anyone who's familiar with the scheduler tell me why this reduced to 0 hosts - http://logs.openstack.org/60/231960/3/check/gate-tempest-dsvm-postgres-full/c319701/logs/screen-n-sch.txt.gz#_2015-10-07_19_45_54_270 | 20:58 |
sdague | because I'm staring a lot at this | 20:58 |
*** salv-orlando has joined #openstack-nova | 20:59 | |
*** tsekiyama has joined #openstack-nova | 20:59 | |
sdague | hmmm libvirt explosion - http://logs.openstack.org/60/231960/3/check/gate-tempest-dsvm-postgres-full/c319701/logs/screen-n-cpu.txt.gz#_2015-10-07_19_45_59_597 | 21:00 |
sdague | mriedem: is that an issue you've been tracking | 21:00 |
sdague | I guess we matched a host, landed there, libvirt exploded, and the retry resolved to 0 (for unknown to me reason) | 21:01 |
mriedem | sdague: totes | 21:01 |
sdague | mriedem: ok, I also thought our scheduler logs had gotten better :( | 21:02 |
*** thumpba has joined #openstack-nova | 21:02 | |
mriedem | this is the bug fwiw https://bugs.launchpad.net/nova/+bug/1501366 | 21:02 |
openstack | Launchpad bug 1501366 in OpenStack Compute (nova) "libvirtError: Error while building firewall: Some rules could not be created for interface" [High,Confirmed] | 21:02 |
mriedem | the scheduler logs should just say which filters were processed and what their results were (i think) | 21:02 |
sdague | ok, why does the retry fail so hard in those cases? | 21:02 |
mriedem | edleafe: did that part | 21:02 |
mriedem | i don't understand the question on the retry | 21:03 |
mriedem | we only have one host right? | 21:03 |
sdague | correct | 21:03 |
sdague | retry isn't ever allowed to circle back? | 21:03 |
mriedem | you'd have to ask one of the scheduler trolls | 21:04 |
sdague | ok | 21:04 |
mriedem | http://logs.openstack.org/60/231960/3/check/gate-tempest-dsvm-postgres-full/c319701/logs/screen-n-sch.txt.gz#_2015-10-07_19_46_00_487 | 21:05 |
mriedem | https://github.com/openstack/nova/blob/master/nova/scheduler/filters/retry_filter.py#L41 | 21:05 |
*** bswartz has joined #openstack-nova | 21:05 | |
mriedem | i guess you'd have to extend that for a retry limit on a failed host if you wanted that | 21:05 |
*** pm90_ has joined #openstack-nova | 21:05 | |
mriedem | otherwise i think the retry filter is to retry on a different host up to x times, (3 by default?) assuming you have x hosts | 21:06 |
sdague | or edge case, that if retry filter returns nothing, maybe don't blacklist hsots | 21:06 |
*** annasort has quit IRC | 21:06 | |
*** achanda has quit IRC | 21:07 | |
*** penick has joined #openstack-nova | 21:07 | |
*** mtanino has joined #openstack-nova | 21:07 | |
sdague | because on a full cloud it gets a bit pathologic given that it assumes you failed for capacity, but you might have failed for other reasons, like in this case | 21:07 |
*** ccesario_ has joined #openstack-nova | 21:07 | |
*** harlowja has quit IRC | 21:08 | |
*** mc_nair has quit IRC | 21:08 | |
sdague | ok, well I at least have the recheck bug then. Got to run off to lug meeting, but maybe I'll poke tomorrow. | 21:08 |
cfriesen | sdague: mriedem: retry normally doesn't retry the one that failed, I think | 21:08 |
sdague | cfriesen: yeh, that's what it looks like | 21:09 |
*** thorst1 has quit IRC | 21:09 | |
sdague | however, it does seem if it goes to 0 nodes, maybe it's worth retrying the whole set | 21:09 |
*** tsekiyama has quit IRC | 21:09 | |
cfriesen | sdague: mriedem: is there any interest in better scheduler logs (maybe optionally) ? Currently especially some of the funky stuff with PCI/numa/pinning/hugepages it can be difficult to figure out what failed. | 21:09 |
mriedem | i'm not interested in better logging no | 21:10 |
mriedem | or anything useful in general | 21:10 |
*** Guest62625 is now known as mfisch | 21:10 | |
*** john5223 is now known as zz_john5223 | 21:10 | |
*** mfisch is now known as Guest49167 | 21:10 | |
*** zz_john5223 is now known as john5223 | 21:10 | |
sdague | cfriesen: almost everything is hard to figure out on fail in the scheduler :) | 21:10 |
sdague | honestly, I think we kind of need to unroll the way filters are processed to make it better | 21:11 |
cfriesen | sdague: mriedem: we've got a local patch that passes a "failure reason" parameter into each filter that can get filled in if the filter fails | 21:11 |
sdague | cfriesen: that seems useful, I'd say propose it | 21:11 |
*** tsekiyama has joined #openstack-nova | 21:11 | |
mriedem | cfriesen: you can always propose a thing and be rejected | 21:11 |
mriedem | but if you don't propose it, you never know | 21:12 |
cfriesen | true enough. :) | 21:12 |
mriedem | cfriesen: you can get all sassy like these two http://www.imdb.com/title/tt1041829/ | 21:13 |
*** angdraug has joined #openstack-nova | 21:13 | |
*** pm90__ has joined #openstack-nova | 21:15 | |
*** achanda has joined #openstack-nova | 21:15 | |
*** pm90_ has quit IRC | 21:17 | |
bauzas | catching the convo now :) | 21:18 |
bauzas | well, the RetryFilter generates some confusion IMHO | 21:19 |
*** rlrossit has quit IRC | 21:19 | |
jlk | Has anybody thrown up alarms about stable/kilo issues and glance server interaction? | 21:20 |
cfriesen | mriedem: I prefer blackmail. Now if only I had some dirt on anyone.... | 21:20 |
mriedem | http://docs.openstack.org/developer/nova/filter_scheduler.html?highlight=retryfilter - we could update the docs there | 21:20 |
mriedem | "filters hosts that have been attempted for scheduling. Only passes hosts that have not been previously attempted." | 21:20 |
bauzas | the problem is when your computes are wedged, you think it's a scheduler issue | 21:21 |
mriedem | jlk: nope | 21:21 |
mriedem | jlk: well, need some context | 21:21 |
bauzas | so people are complaining against something which is not really its fault | 21:21 |
mriedem | oh the poor scheduler | 21:21 |
mriedem | :P | 21:21 |
jlk | mriedem: Installs over the last day or so we're getting errors when Nova tries to get info from glance, it's getting a traceback on a key error, | 21:22 |
mriedem | bauzas: i hope your kids aren't jealous of your true love for the scheduler | 21:22 |
bauzas | I have a French cartoon in mind | 21:22 |
bauzas | calimero | 21:22 |
bauzas | but that doesn't help | 21:22 |
jlk | 2015-10-07 21:16:32.411 27912 TRACE nova.api.openstack File "/opt/bbc/openstack-11.0-master/nova/local/lib/python2.7/site-packages/glanceclient/openstack/common/apiclient/base.py", line 494, in __getattr__ | 21:22 |
jlk | 2015-10-07 21:16:32.411 27912 TRACE nova.api.openstack raise AttributeError(k) | 21:22 |
jlk | 2015-10-07 21:16:32.411 27912 TRACE nova.api.openstack AttributeError: id | 21:22 |
*** claudiub has joined #openstack-nova | 21:22 | |
*** john5223 is now known as zz_john5223 | 21:22 | |
mriedem | bauzas: i watched a youtube clip, makes sense i guess | 21:23 |
bauzas | mriedem: my first daughter is born the same week than RAX and Nasa announced OpenStack, so she already knows why I'm overcommitted to the project :p | 21:23 |
*** yassine has quit IRC | 21:23 | |
*** fawadkhaliq has quit IRC | 21:23 | |
mriedem | jlk: i feel like that's a solved issue in newer glanceclient | 21:23 |
mriedem | you'd have to check bug reports though | 21:23 |
jlk | mriedem: it is, but stable/kilo pins to an older glanceclient | 21:23 |
mriedem | jlk: you don't have to use that older glanceclient just b/c g-r has that | 21:23 |
jlk | python-glanceclient<0.18.0,>=0.15.0 | 21:23 |
bauzas | anyway, I agree that we could somehow document better how to troubleshoot a scheduler failure | 21:24 |
mriedem | g-r usually has things capped on stable not for functoinal reasons, but for gate job issues | 21:24 |
jlk | mriedem: sure, just overriding that is somewhat... difficult. | 21:24 |
mriedem | jlk: are you pip installing? | 21:24 |
jlk | I /can/ make it work by bumping it | 21:24 |
jlk | or rather by editing the requirements file in my checkout before pip installing it | 21:24 |
* bauzas thinking of a blogpost, quite classic ;) | 21:24 | |
jlk | (because bumping it after the install breaks all kinds of things) | 21:24 |
mriedem | jlk: alternatively, if we know the fix in glanceclient, we can backport that to stable and do a patch version release | 21:24 |
*** zz_john5223 is now known as john5223 | 21:24 | |
jlk | it's certainly fixed in glance 1.1.0 | 21:25 |
mriedem | jlk: did you find the bug for the fix? | 21:25 |
jlk | er glanceclient | 21:25 |
*** burt has quit IRC | 21:25 | |
mriedem | i found this https://bugs.launchpad.net/python-glanceclient/+bug/1476770 | 21:26 |
openstack | Launchpad bug 1476770 in python-glanceclient "_translate_from_glance fails with "AttributeError: id" in grenade" [Undecided,New] | 21:26 |
jlk | https://bugs.launchpad.net/glance/+bug/1476770 is the bug I think if I just found it right | 21:26 |
mriedem | jinx | 21:26 |
jlk | lolz | 21:26 |
mriedem | oh i remember this | 21:26 |
jlk | what I'm trying to understand is why this just started hitting us last night, and not like the day before | 21:26 |
*** rlrossit has joined #openstack-nova | 21:26 | |
*** dustins has quit IRC | 21:27 | |
*** xgsa has quit IRC | 21:27 | |
mriedem | did you update some dependencies? | 21:27 |
*** xgsa has joined #openstack-nova | 21:27 | |
mriedem | the problem in the bug was capped requests in oslo.vmware on stable which was borking glanceclient somehow | 21:27 |
jlk | well, this is CI, so every run i sa new install | 21:27 |
mriedem | i don't remember the full details | 21:27 |
*** willsama has joined #openstack-nova | 21:27 | |
jlk | if I modify the requirements.txt before pip installing nova intoa venv, it seems to work | 21:28 |
jlk | that'd mean forking nova for us for now | 21:28 |
mriedem | well, i'd first read through that bug report for how we fixed it in the gate | 21:28 |
*** Guest49167 is now known as mfisch | 21:29 | |
mriedem | jlk: https://review.openstack.org/#/c/204193/ | 21:29 |
*** mfisch has quit IRC | 21:29 | |
*** mfisch has joined #openstack-nova | 21:29 | |
doffm | nova doesn't do any sort of device passthrough for lxc guests right? | 21:29 |
mriedem | jlk: this was the root cause for the gate which uses ubuntu 14.04 https://bugs.launchpad.net/python-glanceclient/+bug/1476770/comments/6 | 21:30 |
openstack | Launchpad bug 1476770 in python-glanceclient "_translate_from_glance fails with "AttributeError: id" in grenade" [Undecided,New] | 21:30 |
jlk | mriedem: yeah, I'm struggling to figure out what changed on our end to start tripping on this, because supposedly it's all been fixed for months | 21:31 |
mriedem | jlk: what version of oslo.vmware do you have? | 21:31 |
jlk | would you believe, none? | 21:32 |
*** ccarmack has quit IRC | 21:32 | |
*** jamielennox|away is now known as jamielennox | 21:32 | |
mriedem | i don't believe that no | 21:32 |
jlk | /opt/bbc/openstack-11.0-master/nova/bin/pip freeze |grep vmware | 21:32 |
jlk | show snone | 21:33 |
jlk | what pulls in oslo.vmware ? | 21:33 |
mriedem | well, nova requires it in test-requirements b/c it's optional | 21:34 |
mriedem | https://github.com/openstack/nova/blob/stable/kilo/test-requirements.txt#L26 | 21:34 |
jlk | yeah, we don't install test requirements | 21:34 |
jlk | so our urllib3 isn't capped | 21:34 |
mriedem | i thought glance or glancestore required it though for the vmware backend | 21:34 |
mriedem | or cinder | 21:34 |
jlk | although we don't have urllib3 | 21:34 |
jlk | glance might, but we just install glanceclient | 21:34 |
jlk | glance venv has oslo.vmware==0.11.2 | 21:35 |
*** ccarmack has joined #openstack-nova | 21:35 | |
jlk | and urllib3==1.10.4 | 21:35 |
mriedem | cinder requires oslo.vmware in kilo https://github.com/openstack/cinder/blob/stable/kilo/requirements.txt#L47 | 21:35 |
*** moshele has quit IRC | 21:35 | |
mriedem | 0.11.2 is the version you want | 21:35 |
jlk | yeah, each of our services get their own venv, so the requirements of one don't equate to the requirements of the other | 21:35 |
mriedem | which caps urllib3 | 21:35 |
mriedem | ooo like containers! | 21:36 |
mriedem | fancy | 21:36 |
jlk | Looks like on the system we have ii python-urllib3 1.7.1-1ubuntu3 al | 21:36 |
jlk | there is no pip installed urllib3 in nova's venv | 21:36 |
*** signed8bit has quit IRC | 21:37 | |
mriedem | requests 2.8.0 was released yesterday | 21:37 |
mriedem | requests 2.8.0 2015-10-06 | 21:37 |
mriedem | in case that affects you at all | 21:37 |
jlk | I bet that's it | 21:38 |
*** manishg_ has quit IRC | 21:38 | |
mriedem | which is uncapped in kilo https://github.com/openstack/requirements/blob/stable/kilo/global-requirements.txt#L143 | 21:38 |
jlk | yeah 2.8.0 | 21:38 |
jlk | so... | 21:38 |
jlk | lemme downgrade | 21:38 |
jlk | oh right | 21:39 |
jlk | because pip requests bundles urllib3 | 21:39 |
jlk | and it's including 1.12 | 21:39 |
jlk | this is it | 21:39 |
mriedem | cool | 21:39 |
mriedem | btw, pro tip, i found that by running this https://github.com/openstack/requirements/blob/stable/kilo/tools/what-broke.py | 21:39 |
mriedem | from stable/kilo requirements, python tools/what-broke.py | 21:40 |
jlk | oh that looks useful | 21:40 |
mriedem | yeah, sdague wrote that since this is a common thing that breaks us upstream | 21:40 |
mriedem | well, before upper-constraints in liberty | 21:40 |
*** pixelbeat has joined #openstack-nova | 21:40 | |
mriedem | i'm surprised we haven't seen that blow up in the stable/kilo jobs upstream | 21:41 |
jlk | yeah, I wonder if your requests is capped there somehow | 21:41 |
jlk | using the system package rather than pip installed | 21:41 |
mriedem | maybe | 21:41 |
jlk | but that seems to fix me | 21:41 |
mriedem | urllib3 is definitely capped | 21:41 |
mriedem | https://github.com/openstack/requirements/blob/stable/kilo/global-requirements.txt#L169 | 21:41 |
jlk | ah | 21:41 |
mriedem | so it's probably a thing waiting tobreak | 21:42 |
jlk | maybe | 21:42 |
jlk | wait a sec | 21:43 |
jlk | how does g-r interact with the requirements file in the project itelf? | 21:43 |
jlk | nova requirements.txt doesn't mention urllib, but g-r does? | 21:43 |
*** tsekiyama has quit IRC | 21:43 | |
mriedem | nova's requirements.txt is synced from g-r | 21:43 |
mriedem | via bot | 21:43 |
jlk | so this must have just changed? | 21:43 |
mriedem | requests is a transitive dependency through glanceclient | 21:43 |
jlk | (pinning urllib) | 21:43 |
mriedem | no, that was awhile back | 21:43 |
mriedem | july https://review.openstack.org/#/c/204193/ | 21:44 |
jlk | then why wasn't that synced to the stable/kilo requirements for nova? | 21:44 |
*** rlrossit has left #openstack-nova | 21:44 | |
mriedem | nova doesn't require urllib3 | 21:44 |
mriedem | oslo.vmware does | 21:44 |
mriedem | again, transitive dep | 21:44 |
jlk | oh wait, | 21:44 |
jlk | so the requirements of the projects bubble up into g-r | 21:44 |
jlk | not the other way around? | 21:44 |
mriedem | yes | 21:44 |
mriedem | everything is centralized in g-r | 21:45 |
mriedem | so we're not all using different versions of dependencies and cause conflicts | 21:45 |
mriedem | when g-r is changed, that is synced out to the projects that use the changed dep | 21:45 |
mriedem | via bot | 21:45 |
jlk | gotcha | 21:45 |
jlk | so it's a "if you use this, use this version" | 21:45 |
mriedem | right, like this https://review.openstack.org/#/c/201191/ | 21:45 |
jlk | and your tests install all the services in one pile, yeah? | 21:46 |
jlk | each service isn't isolated? | 21:46 |
mriedem | right | 21:46 |
jlk | that's why you don't see it | 21:46 |
mriedem | devstack for the tempest jobs, otherwise tox for the unit test jobs | 21:46 |
jlk | with combined requirements, you're safe. But with nova requirements on its own, boom. | 21:46 |
mriedem | i prefer to stay safe and warm | 21:47 |
*** tsekiyama has joined #openstack-nova | 21:47 | |
*** Vimal has quit IRC | 21:48 | |
*** manishg has joined #openstack-nova | 21:48 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Add more documentation for RetryFilter https://review.openstack.org/232241 | 21:49 |
bauzas | mriedem: sdague: ^ | 21:49 |
jlk | would a patch to pin requests in nova be frowned upon? | 21:49 |
*** subscope has quit IRC | 21:51 | |
mriedem | jlk: in kilo? | 21:52 |
jlk | at least in kilo, not sure if other versions are impacted | 21:52 |
mriedem | you'd have to first pin requests in g-r stable/kilo | 21:52 |
jlk | oh I could just do it there, sure | 21:53 |
mriedem | it's capped in juno so not a problem there | 21:53 |
mriedem | well, | 21:53 |
mriedem | it's 2 parts | 21:53 |
jlk | no I can't | 21:53 |
jlk | I have to do it there first, then add it in nova | 21:53 |
mriedem | right | 21:53 |
mriedem | cap in g-r stable/juno <2.8.0 | 21:53 |
mriedem | then a nova change that adds requests (same as what's in the g-r change) with a Depends-On tag from the nova change to the g-r change | 21:53 |
*** harlowja has joined #openstack-nova | 21:53 | |
*** rk4n has joined #openstack-nova | 21:54 | |
*** rfolco_ has joined #openstack-nova | 21:54 | |
mriedem | with an explanation in the nova change as to why nova needs to require requests even though nova doesn't use it directly | 21:54 |
jlk | alright, I might need some help on that one when the time comes | 21:54 |
mriedem | heh, if you have the explanation i can push the changes | 21:54 |
jlk | well, maybe it would be better to put the pin on glanceclient | 21:54 |
mriedem | want to just dump it in that bug report? | 21:54 |
jlk | since glanceclient is the one having trouble | 21:54 |
mriedem | glanceclient is pinned in g-r stable/kilo https://github.com/openstack/requirements/blob/stable/kilo/global-requirements.txt#L125 | 21:54 |
mriedem | s/pinned/capped/ | 21:54 |
jlk | what I meant was that glanceclient is the thing that is using requests | 21:55 |
mriedem | if you cap requests in g-r stable/kilo and glanceclient uses requests, it will sync to glanceclient on stalbe/kilo and we can release that as a patch version | 21:55 |
jlk | so old glanceclient pulls in new requests and tries to use it, boom | 21:55 |
*** john5223 is now known as zz_john5223 | 21:55 | |
mriedem | yeah https://github.com/openstack/python-glanceclient/blob/stable/kilo/requirements.txt#L10 | 21:55 |
mriedem | jlk: so how about update that bug report with details | 21:55 |
mriedem | and we can begin the capping game | 21:55 |
mriedem | tonyb: ^ | 21:55 |
jlk | yeah I was just opening that tab again. | 21:56 |
* tonyb reads scroolback | 21:56 | |
*** mwagner_ has joined #openstack-nova | 21:56 | |
*** rfolco has quit IRC | 21:56 | |
*** rfolco__ has joined #openstack-nova | 21:56 | |
*** ccarmack has quit IRC | 21:58 | |
mriedem | jlk: reminded me that i had this https://review.openstack.org/#/c/213310/ | 21:59 |
*** rfolco_ has quit IRC | 22:00 | |
jlk | neat | 22:00 |
jlk | https://bugs.launchpad.net/glance/+bug/1476770 updated | 22:01 |
openstack | Launchpad bug 1476770 in python-glanceclient "_translate_from_glance fails with "AttributeError: id" in grenade" [Undecided,New] | 22:01 |
tonyb | jlk: please drop me the bug number so I can look/help with the capping game. | 22:01 |
tonyb | Gah stupid scrollback. Thanks. | 22:01 |
mriedem | let me update that reqs change on master quick | 22:01 |
tonyb | mriedem: cool. | 22:03 |
*** edmondsw has quit IRC | 22:03 | |
*** penick has quit IRC | 22:03 | |
tonyb | mriedem: I need a coffee anyway. Just got back from a ride | 22:03 |
*** ociuhandu has quit IRC | 22:04 | |
jlk | Thanks for the help, gotta run for a bit. | 22:04 |
*** tsekiyama has quit IRC | 22:04 | |
*** sambetts has quit IRC | 22:05 | |
*** sambetts has joined #openstack-nova | 22:07 | |
*** tsekiyama has joined #openstack-nova | 22:07 | |
*** su_zhang_ has joined #openstack-nova | 22:07 | |
*** annegentle has quit IRC | 22:08 | |
*** penick has joined #openstack-nova | 22:09 | |
*** su_zhang has quit IRC | 22:11 | |
openstackgerrit | Anne Gentle proposed openstack/nova: Rearranges to create new Compute API Guide https://review.openstack.org/230186 | 22:12 |
*** manishg has quit IRC | 22:12 | |
*** tpeoples has quit IRC | 22:13 | |
*** mdrabe has quit IRC | 22:16 | |
*** ctrath has quit IRC | 22:16 | |
mriedem | jlk: https://review.openstack.org/#/c/232250/ | 22:16 |
mriedem | tonyb: ^ | 22:16 |
mriedem | hmm | 22:19 |
mriedem | nova.tests.unit.db.test_db_api.ArchiveTestCase is completely racy | 22:20 |
mriedem | b/c of multiple workers and a single db | 22:20 |
mriedem | mtreinish: tempest has a fixture for making a test case run in serial right? | 22:20 |
*** alex_klimov has quit IRC | 22:20 | |
*** burgerk_ has quit IRC | 22:20 | |
*** tonytan4ever has quit IRC | 22:21 | |
mtreinish | mriedem: well we tell testr to parallelize at the class level | 22:23 |
mriedem | yeah, i want that | 22:23 |
mtreinish | and then there is an external lock fixture we use to lock against running tests at the same time | 22:23 |
mtreinish | mriedem: it's an all or nothing deal to do the class thing | 22:24 |
mtreinish | it's in the testr.conf | 22:24 |
mtreinish | mriedem: http://git.openstack.org/cgit/openstack/tempest/tree/.testr.conf#n9 | 22:24 |
mtreinish | getting that all working was most of my havana cycle | 22:25 |
mriedem | https://github.com/openstack/tempest/blob/master/tempest/common/tempest_fixtures.py#L19 | 22:25 |
mriedem | right? | 22:25 |
mtreinish | yeah that's something we added to oslo_concurrency | 22:25 |
mtreinish | that does the same thing as the syncronized decorator (or whatever the lcok one is) with external=True | 22:25 |
mtreinish | it requires config to work | 22:25 |
mtreinish | mriedem: what exactly are you looking for? | 22:26 |
*** aix has joined #openstack-nova | 22:26 | |
mriedem | let me push it up quick | 22:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: Print number of rows archived per table in db archive_deleted_rows https://review.openstack.org/232252 | 22:27 |
mtreinish | mriedem: wait don't the nova db_api tests use an in memory sqlite db? what is racing? | 22:27 |
mriedem | so what i had here: https://review.openstack.org/#/c/232252/1/nova/tests/unit/db/test_db_api.py | 22:27 |
mriedem | i was originally doing self.assertDictEqual | 22:27 |
mriedem | b/c i expected a very specific result | 22:27 |
mriedem | but b/c there are other tests running i was getting stuff back in results from other tests hitting archive at the same time | 22:28 |
mriedem | honestly i'm not sure how this test class actually works today since it's very global on thedb | 22:28 |
mriedem | or is it a db per test case run? | 22:28 |
mtreinish | mriedem: I was pretty sure you had a fixture in place that setup a new in memory sqlite db for each test (it runs migrations fresh) so the tests are properly isolate | 22:29 |
mtreinish | d | 22:29 |
mtreinish | mriedem: yeah that's how I thought they worked | 22:29 |
mriedem | so we do https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L195 | 22:29 |
mriedem | well wtf | 22:29 |
*** dims has quit IRC | 22:29 | |
mriedem | could be some dumb metadata stuff we're doing in the db api | 22:30 |
mtreinish | mriedem: if you want the total insanity of a shared DB for api testing, you end up with essentially this: https://review.openstack.org/#/c/213481/ | 22:30 |
mtreinish | which is not a direction you want to go | 22:30 |
mriedem | heh, no thanks | 22:30 |
*** markvoelker has quit IRC | 22:31 | |
* mtreinish reminds himself to rework that to not use the shared db now that sqlite works | 22:33 | |
*** markvoelker has joined #openstack-nova | 22:33 | |
mtreinish | mriedem: do you have an example of the failure you were hitting? | 22:34 |
mriedem | sec | 22:34 |
mriedem | it had some entries in the dict that were some crazy random string | 22:34 |
mriedem | like instance_info_cache:[862 char] | 22:35 |
*** Daisy has joined #openstack-nova | 22:36 | |
*** stevemar_ has quit IRC | 22:36 | |
*** stevemar_ has joined #openstack-nova | 22:37 | |
mtreinish | hmm, that sounds weird | 22:37 |
mriedem | mtreinish: with just this: self.assertDictEqual(dict(instance_id_mappings=2), results) | 22:38 |
mriedem | i get http://paste.openstack.org/show/475668/ | 22:38 |
*** sdague has quit IRC | 22:38 | |
mriedem | assertDictEqual is probably not what i want to be using either | 22:38 |
*** annasort has joined #openstack-nova | 22:39 | |
*** sdague has joined #openstack-nova | 22:39 | |
*** rk4n has quit IRC | 22:40 | |
mtreinish | mriedem: yeah, I normally don't like the more involved asserts and just extract the values to check manually and do an equals | 22:40 |
mtreinish | fwiw, I have a hard time following what that error output is trying to tell me | 22:40 |
*** Daisy has quit IRC | 22:40 | |
*** stevemar_ has quit IRC | 22:41 | |
*** rk4n has joined #openstack-nova | 22:41 | |
*** manishg has joined #openstack-nova | 22:42 | |
mriedem | nonsense | 22:43 |
mriedem | it's nonsense | 22:43 |
mriedem | we have a model object comparator assertion thing in the test_db_api module, i'll use that | 22:44 |
mriedem | compares 2 dicts based on length and key/values | 22:44 |
*** ZZelle has quit IRC | 22:45 | |
mriedem | assertDictEqual isn't even testtools, it's std lib https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertDictEqual | 22:45 |
*** Daisy has joined #openstack-nova | 22:46 | |
mriedem | heh, the test result is still completely reandom | 22:46 |
mriedem | *random | 22:46 |
mriedem | oh i know why | 22:47 |
*** su_zhang has joined #openstack-nova | 22:49 | |
*** Daisy has quit IRC | 22:50 | |
*** su_zhang_ has quit IRC | 22:52 | |
*** david-lyle has quit IRC | 22:54 | |
lifeless | mriedem: forgot to update the incubator? | 22:54 |
mriedem | lifeless: ? | 22:54 |
mriedem | for my test problems above? | 22:54 |
lifeless | mriedem: guessing at why its still random order :) | 22:54 |
lifeless | mriedem: but perhaps you've context switched onto something totally different :) | 22:54 |
mriedem | no, the code was stuffing keys with 0 values in the dict and i only expected things with a >0 value | 22:54 |
mriedem | so the test actually found a problem \o/ | 22:55 |
*** david-lyle has joined #openstack-nova | 22:55 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Print number of rows archived per table in db archive_deleted_rows https://review.openstack.org/232252 | 22:55 |
mriedem | now my 1 day blueprint is complete | 22:55 |
mriedem | pending the o-i syncs | 22:55 |
*** pm90__ has quit IRC | 22:56 | |
*** angdraug has quit IRC | 22:57 | |
*** pm90_ has joined #openstack-nova | 22:59 | |
*** stevemar_ has joined #openstack-nova | 22:59 | |
*** davidlenwell has quit IRC | 23:01 | |
*** davidlenwell has joined #openstack-nova | 23:02 | |
*** markvoelker has quit IRC | 23:04 | |
*** alejandrito has quit IRC | 23:04 | |
*** xyang1 has quit IRC | 23:08 | |
*** markvoelker has joined #openstack-nova | 23:08 | |
*** pixelbeat has quit IRC | 23:08 | |
*** ijuwang has quit IRC | 23:09 | |
*** harlowja has quit IRC | 23:10 | |
*** david-lyle has quit IRC | 23:10 | |
*** harlowja has joined #openstack-nova | 23:10 | |
*** willsama has quit IRC | 23:10 | |
*** david-lyle has joined #openstack-nova | 23:10 | |
*** amrith is now known as _amrith_ | 23:11 | |
*** _amrith_ is now known as amrith | 23:12 | |
*** david-lyle has quit IRC | 23:14 | |
*** diogogmt_ has quit IRC | 23:16 | |
*** david-lyle has joined #openstack-nova | 23:17 | |
*** rk4n has quit IRC | 23:18 | |
*** david-ly_ has joined #openstack-nova | 23:19 | |
*** david-lyle has quit IRC | 23:21 | |
*** atuvenie_ has joined #openstack-nova | 23:22 | |
*** manishg has quit IRC | 23:22 | |
*** dims has joined #openstack-nova | 23:22 | |
*** atuvenie has quit IRC | 23:25 | |
*** hemna is now known as hemnafk | 23:26 | |
*** manishg has joined #openstack-nova | 23:27 | |
*** thorst has joined #openstack-nova | 23:30 | |
*** david-ly_ is now known as david-lyle | 23:31 | |
*** mriedem has quit IRC | 23:31 | |
*** mriedem has joined #openstack-nova | 23:32 | |
*** tsekiyama has quit IRC | 23:33 | |
*** penick has quit IRC | 23:34 | |
*** lucas-dinner has quit IRC | 23:35 | |
*** nic has quit IRC | 23:36 | |
*** aysyd has quit IRC | 23:36 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: Replaces contextlib.nested with test.nested https://review.openstack.org/232108 | 23:37 |
*** penick has joined #openstack-nova | 23:37 | |
*** thorst has quit IRC | 23:38 | |
*** asettle has joined #openstack-nova | 23:38 | |
*** mriedem has quit IRC | 23:39 | |
*** dims has quit IRC | 23:40 | |
*** tjones has joined #openstack-nova | 23:42 | |
*** markvoelker has quit IRC | 23:42 | |
*** tjones has quit IRC | 23:43 | |
*** markvoelker has joined #openstack-nova | 23:43 | |
*** tjones has joined #openstack-nova | 23:44 | |
*** amrith is now known as _amrith_ | 23:46 | |
*** achanda has quit IRC | 23:47 | |
*** stevemar_ has quit IRC | 23:47 | |
*** penick has quit IRC | 23:49 | |
*** achanda has joined #openstack-nova | 23:49 | |
*** tjones has left #openstack-nova | 23:50 | |
*** tsekiyama has joined #openstack-nova | 23:50 | |
*** Marga_ has quit IRC | 23:50 | |
*** aswadr has quit IRC | 23:52 | |
*** markvoelker has quit IRC | 23:53 | |
*** achanda has quit IRC | 23:53 | |
*** dims has joined #openstack-nova | 23:54 | |
*** achanda has joined #openstack-nova | 23:55 | |
*** pixelbeat has joined #openstack-nova | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!