| *** emagana has quit IRC | 00:00 | |
| *** baoli has joined #openstack-nova | 00:00 | |
| *** haomaiwa_ has quit IRC | 00:01 | |
| *** burgerk_ has quit IRC | 00:01 | |
| *** haomaiwang has joined #openstack-nova | 00:01 | |
| melwitt | dansmith: looking at the cells tests, it looks like mox.IsA(objects.Instance) is expecting instance being passed to be an InstanceV1 but with the backport it's not? | 00:01 |
|---|---|---|
| *** edtubill has joined #openstack-nova | 00:03 | |
| *** baoli has quit IRC | 00:05 | |
| *** xyang1 has quit IRC | 00:06 | |
| *** baoli has joined #openstack-nova | 00:06 | |
| *** edtubill has quit IRC | 00:08 | |
| *** pixelbeat has quit IRC | 00:10 | |
| *** baoli has quit IRC | 00:12 | |
| *** morgabra_ is now known as morgabra | 00:15 | |
| *** morgabra has quit IRC | 00:15 | |
| *** morgabra has joined #openstack-nova | 00:15 | |
| *** penick has quit IRC | 00:16 | |
| *** EinstCrazy has quit IRC | 00:19 | |
| *** annegentle has quit IRC | 00:22 | |
| *** annegentle has joined #openstack-nova | 00:24 | |
| *** penick has joined #openstack-nova | 00:26 | |
| dims | dansmith mriedem_away melwitt - o.vo break under control? | 00:26 |
| dansmith | dims: yes | 00:26 |
| dims | whew thanks | 00:26 |
| *** thorst_ has joined #openstack-nova | 00:29 | |
| *** yonglihe has quit IRC | 00:30 | |
| *** thorst_ has quit IRC | 00:30 | |
| *** thorst_ has joined #openstack-nova | 00:31 | |
| *** hemna is now known as hemnafk | 00:34 | |
| *** thorst_ has quit IRC | 00:35 | |
| *** pratikma_ has joined #openstack-nova | 00:35 | |
| *** pratikmallya has quit IRC | 00:35 | |
| *** agireud has quit IRC | 00:37 | |
| *** jerrygb has quit IRC | 00:40 | |
| *** jerrygb has joined #openstack-nova | 00:42 | |
| *** jerrygb has quit IRC | 00:42 | |
| *** otter768 has joined #openstack-nova | 00:44 | |
| *** annegentle has quit IRC | 00:47 | |
| *** penick has quit IRC | 00:47 | |
| *** annegentle has joined #openstack-nova | 00:47 | |
| *** otter768 has quit IRC | 00:48 | |
| *** mlavalle has quit IRC | 00:56 | |
| *** aginwala has joined #openstack-nova | 00:57 | |
| *** Yanyanhu has joined #openstack-nova | 00:59 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** haomaiwang has joined #openstack-nova | 01:01 | |
| *** baoli has joined #openstack-nova | 01:03 | |
| *** jyuso1 has joined #openstack-nova | 01:03 | |
| *** penick has joined #openstack-nova | 01:03 | |
| *** EinstCrazy has joined #openstack-nova | 01:04 | |
| *** davideagnello has quit IRC | 01:05 | |
| *** davideagnello has joined #openstack-nova | 01:06 | |
| *** salv-orlando has quit IRC | 01:12 | |
| *** penick has quit IRC | 01:13 | |
| *** smatzek has joined #openstack-nova | 01:13 | |
| *** rushil has joined #openstack-nova | 01:14 | |
| sdague | is there a pin up? | 01:15 |
| dims | sdague : a unit test fix is being merged | 01:16 |
| sdague | ok | 01:16 |
| openstackgerrit | Sean Dague proposed openstack/nova: Removed extra spaces from double line strings https://review.openstack.org/242989 | 01:17 |
| sdague | that was the last nova patch ahead of it that wasn't already failing | 01:17 |
| *** aginwala has quit IRC | 01:19 | |
| *** smatzek has quit IRC | 01:20 | |
| *** aginwala has joined #openstack-nova | 01:20 | |
| *** penick has joined #openstack-nova | 01:22 | |
| melwitt | dansmith: I left you a comment on the backport review that hopefully might be helpful | 01:25 |
| *** aginwala has quit IRC | 01:25 | |
| *** sacharya has joined #openstack-nova | 01:25 | |
| *** penick has quit IRC | 01:27 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Split-network-plane-for-live-migration https://review.openstack.org/245005 | 01:28 |
| *** thorst_ has joined #openstack-nova | 01:28 | |
| *** thorst_ has quit IRC | 01:30 | |
| *** rushil_ has joined #openstack-nova | 01:30 | |
| *** thorst_ has joined #openstack-nova | 01:30 | |
| *** ssurana1 has quit IRC | 01:32 | |
| *** rushil has quit IRC | 01:33 | |
| *** thorst_ has quit IRC | 01:35 | |
| *** aginwala has joined #openstack-nova | 01:36 | |
| *** zhenguo has joined #openstack-nova | 01:36 | |
| *** takashin has joined #openstack-nova | 01:36 | |
| *** pratikma_ has quit IRC | 01:36 | |
| *** markvoelker has joined #openstack-nova | 01:37 | |
| *** jdurgin1 has quit IRC | 01:41 | |
| *** annegentle has quit IRC | 01:41 | |
| *** jdurgin1 has joined #openstack-nova | 01:44 | |
| *** otter768 has joined #openstack-nova | 01:45 | |
| *** achanda has joined #openstack-nova | 01:46 | |
| *** suro-patz has quit IRC | 01:46 | |
| *** aix has quit IRC | 01:46 | |
| *** haomaiwang has quit IRC | 01:49 | |
| *** yangyapeng has joined #openstack-nova | 01:49 | |
| *** mriedem_away is now known as mriedem | 01:59 | |
| mriedem | if i could travel back in time, i'd go back 10 years and thank myself for keeping copies of furniture receipts so i can use them when the cleaning service damages said furniture | 02:00 |
| mriedem | \o/ | 02:00 |
| *** yonglihe has joined #openstack-nova | 02:02 | |
| *** mwagner has joined #openstack-nova | 02:03 | |
| *** eglynn_ has quit IRC | 02:05 | |
| *** jmccrory has joined #openstack-nova | 02:05 | |
| *** fawadkhaliq has joined #openstack-nova | 02:05 | |
| *** mriedem has quit IRC | 02:07 | |
| *** alejandrito has joined #openstack-nova | 02:08 | |
| *** alejandrito has quit IRC | 02:08 | |
| *** READ10 has joined #openstack-nova | 02:08 | |
| *** alejandrito has joined #openstack-nova | 02:08 | |
| *** otter768 has quit IRC | 02:10 | |
| *** pece has quit IRC | 02:10 | |
| *** baoli has quit IRC | 02:11 | |
| *** baoli has joined #openstack-nova | 02:11 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/245248 | 02:14 |
| *** sacharya has quit IRC | 02:16 | |
| *** alejandrito has quit IRC | 02:21 | |
| *** pratikmallya has joined #openstack-nova | 02:21 | |
| *** armax has joined #openstack-nova | 02:24 | |
| *** jwcroppe has quit IRC | 02:26 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id and user-id when list server-groups https://review.openstack.org/209018 | 02:26 |
| *** leseb has quit IRC | 02:28 | |
| *** klkumar has joined #openstack-nova | 02:28 | |
| *** mtanino has quit IRC | 02:29 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/245248 | 02:29 |
| *** haomaiwang has joined #openstack-nova | 02:31 | |
| *** leseb has joined #openstack-nova | 02:32 | |
| *** alejandrito has joined #openstack-nova | 02:33 | |
| *** haomaiwang has quit IRC | 02:36 | |
| *** haomaiwa_ has joined #openstack-nova | 02:36 | |
| *** leseb has quit IRC | 02:37 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/245248 | 02:38 |
| *** apoorvad has quit IRC | 02:38 | |
| *** unicell has quit IRC | 02:40 | |
| *** aginwala has quit IRC | 02:40 | |
| *** aginwala has joined #openstack-nova | 02:41 | |
| *** shz has joined #openstack-nova | 02:41 | |
| *** vladikr has joined #openstack-nova | 02:42 | |
| *** aginwala has quit IRC | 02:44 | |
| *** shihanzhang has quit IRC | 02:44 | |
| *** aginwala has joined #openstack-nova | 02:44 | |
| *** baoli has quit IRC | 02:45 | |
| *** baoli has joined #openstack-nova | 02:45 | |
| *** takashin has left #openstack-nova | 02:48 | |
| *** aginwala has quit IRC | 02:48 | |
| *** aginwala has joined #openstack-nova | 02:51 | |
| *** leseb has joined #openstack-nova | 02:51 | |
| *** vilobhmm has quit IRC | 02:52 | |
| *** agireud has joined #openstack-nova | 02:52 | |
| *** _RuiChen has joined #openstack-nova | 02:52 | |
| *** baoli has quit IRC | 02:52 | |
| Kevin_Zheng | python27 and python34 always fails | 02:56 |
| *** leseb has quit IRC | 02:56 | |
| *** RuiChen has quit IRC | 02:56 | |
| *** leseb has joined #openstack-nova | 02:56 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Split-network-plane-for-live-migration https://review.openstack.org/245005 | 02:57 |
| *** baoli has joined #openstack-nova | 02:58 | |
| *** venkat_p has joined #openstack-nova | 02:58 | |
| *** annegentle has joined #openstack-nova | 03:02 | |
| *** Sree has joined #openstack-nova | 03:05 | |
| *** bapalm has quit IRC | 03:06 | |
| *** rushil_ has quit IRC | 03:07 | |
| *** Sree has quit IRC | 03:07 | |
| *** Sree has joined #openstack-nova | 03:08 | |
| *** READ10 has quit IRC | 03:10 | |
| *** Yanyanhu has quit IRC | 03:11 | |
| *** EinstCrazy has quit IRC | 03:11 | |
| *** EinstCrazy has joined #openstack-nova | 03:12 | |
| *** EinstCra_ has joined #openstack-nova | 03:12 | |
| oomichi | mrodden: like "back to the future" :-) | 03:13 |
| *** pratikmallya has quit IRC | 03:16 | |
| *** EinstCrazy has quit IRC | 03:16 | |
| *** salv-orlando has joined #openstack-nova | 03:16 | |
| *** davideagnello has quit IRC | 03:18 | |
| *** coreywright has quit IRC | 03:18 | |
| *** devananda has quit IRC | 03:18 | |
| *** coreywright has joined #openstack-nova | 03:21 | |
| *** davideagnello has joined #openstack-nova | 03:21 | |
| *** shuquan has joined #openstack-nova | 03:24 | |
| *** Marga__ has quit IRC | 03:24 | |
| *** annegentle has quit IRC | 03:25 | |
| *** jwcroppe has joined #openstack-nova | 03:26 | |
| *** shuquan has quit IRC | 03:26 | |
| *** eliqiao_ has joined #openstack-nova | 03:29 | |
| *** devananda has joined #openstack-nova | 03:31 | |
| *** aginwala has quit IRC | 03:31 | |
| *** ccard__ has joined #openstack-nova | 03:31 | |
| *** eliqiao has quit IRC | 03:31 | |
| *** aginwala has joined #openstack-nova | 03:32 | |
| *** ccard_ has quit IRC | 03:34 | |
| *** Marga_ has joined #openstack-nova | 03:35 | |
| *** aginwala has quit IRC | 03:36 | |
| *** Marga_ has quit IRC | 03:36 | |
| *** vilobhmm has joined #openstack-nova | 03:39 | |
| *** salv-orlando has quit IRC | 03:41 | |
| *** jwcroppe has quit IRC | 03:44 | |
| *** ccard_ has joined #openstack-nova | 03:46 | |
| *** shuquan has joined #openstack-nova | 03:48 | |
| *** ccard__ has quit IRC | 03:49 | |
| *** ccard__ has joined #openstack-nova | 03:50 | |
| *** achanda has quit IRC | 03:50 | |
| *** ccard_ has quit IRC | 03:50 | |
| *** shuquan has quit IRC | 03:53 | |
| *** gcb has joined #openstack-nova | 03:58 | |
| *** Yanyanhu has joined #openstack-nova | 04:01 | |
| *** dims has quit IRC | 04:06 | |
| *** klkumar has quit IRC | 04:06 | |
| *** Yanyanhu has quit IRC | 04:10 | |
| *** Yanyanhu has joined #openstack-nova | 04:11 | |
| *** otter768 has joined #openstack-nova | 04:11 | |
| *** otter768 has quit IRC | 04:16 | |
| *** alejandrito has quit IRC | 04:16 | |
| *** liusheng has quit IRC | 04:17 | |
| *** liusheng has joined #openstack-nova | 04:17 | |
| *** Yanyanhu has quit IRC | 04:18 | |
| *** chenli has joined #openstack-nova | 04:18 | |
| *** klkumar has joined #openstack-nova | 04:18 | |
| *** achanda has joined #openstack-nova | 04:21 | |
| oomichi | alex_xu: hi, can you take a look at https://review.openstack.org/#/c/222068/ ? | 04:22 |
| oomichi | alex_xu: you are familar with this, I believe :-) | 04:23 |
| *** sacharya has joined #openstack-nova | 04:29 | |
| *** zhangjn has joined #openstack-nova | 04:35 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/nova-specs: Support hypervisor_type option https://review.openstack.org/246154 | 04:37 |
| *** Yanyanhu has joined #openstack-nova | 04:38 | |
| *** takashin has joined #openstack-nova | 04:39 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/nova-specs: Volume support for multi hypervisors https://review.openstack.org/245986 | 04:42 |
| *** sudipto has joined #openstack-nova | 04:42 | |
| *** Yanyan has joined #openstack-nova | 04:43 | |
| *** Yanyanhu has quit IRC | 04:44 | |
| *** zhangjn has quit IRC | 04:45 | |
| *** dave-mccowan has quit IRC | 04:45 | |
| *** jwang__ has joined #openstack-nova | 04:50 | |
| *** artom_ has joined #openstack-nova | 04:50 | |
| *** jwcroppe has joined #openstack-nova | 04:51 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: libvirt: Add ppc to supported arch for NUMA https://review.openstack.org/245765 | 04:52 |
| *** liusheng has quit IRC | 04:53 | |
| *** mari0jv has joined #openstack-nova | 04:53 | |
| *** salv-orlando has joined #openstack-nova | 04:53 | |
| *** liusheng has joined #openstack-nova | 04:53 | |
| *** nikhil|afk has joined #openstack-nova | 04:54 | |
| *** Nakato_ has joined #openstack-nova | 04:55 | |
| *** baffle has joined #openstack-nova | 04:55 | |
| *** doffm has joined #openstack-nova | 04:56 | |
| *** sshen_ has joined #openstack-nova | 04:57 | |
| *** _d34dh0r53_ has joined #openstack-nova | 04:57 | |
| *** jyuso has joined #openstack-nova | 04:58 | |
| *** jhesketh_ has joined #openstack-nova | 04:58 | |
| *** cfarquhar_ has joined #openstack-nova | 04:58 | |
| *** Vek_ has joined #openstack-nova | 04:59 | |
| *** cburgess_ has joined #openstack-nova | 04:59 | |
| *** artom has quit IRC | 04:59 | |
| *** baoli has quit IRC | 05:00 | |
| *** mariojv has quit IRC | 05:00 | |
| *** jwang_ has quit IRC | 05:00 | |
| *** serverascode has quit IRC | 05:00 | |
| *** flaper87 has quit IRC | 05:00 | |
| *** xnox has quit IRC | 05:00 | |
| *** jyuso1 has quit IRC | 05:00 | |
| *** cburgess has quit IRC | 05:00 | |
| *** codebauss has quit IRC | 05:00 | |
| *** baffle_ has quit IRC | 05:00 | |
| *** Vek has quit IRC | 05:00 | |
| *** keekz has quit IRC | 05:00 | |
| *** leseb has quit IRC | 05:00 | |
| *** jhesketh has quit IRC | 05:00 | |
| *** sshen has quit IRC | 05:00 | |
| *** d34dh0r53 has quit IRC | 05:00 | |
| *** Nakato has quit IRC | 05:00 | |
| *** nikhil has quit IRC | 05:00 | |
| *** jamielennox has quit IRC | 05:00 | |
| *** mgagne has quit IRC | 05:00 | |
| *** adam_g has quit IRC | 05:00 | |
| *** jroll has quit IRC | 05:00 | |
| *** sbadia has quit IRC | 05:00 | |
| *** wasmum has quit IRC | 05:00 | |
| *** doffm_ has quit IRC | 05:00 | |
| *** kbyrne has quit IRC | 05:00 | |
| *** afazekas has quit IRC | 05:00 | |
| *** cfarquhar has quit IRC | 05:00 | |
| *** stevemar_ has quit IRC | 05:00 | |
| *** jroll has joined #openstack-nova | 05:00 | |
| *** jhesketh_ is now known as jhesketh | 05:00 | |
| *** adam_g` has joined #openstack-nova | 05:00 | |
| *** klkumar has quit IRC | 05:00 | |
| *** klkumar has joined #openstack-nova | 05:00 | |
| *** jhesketh has quit IRC | 05:00 | |
| *** mgagne has joined #openstack-nova | 05:00 | |
| *** jhesketh has joined #openstack-nova | 05:01 | |
| *** flaper87 has joined #openstack-nova | 05:01 | |
| *** keekz has joined #openstack-nova | 05:01 | |
| *** jamielennox has joined #openstack-nova | 05:01 | |
| *** wasmum- has joined #openstack-nova | 05:01 | |
| *** leseb has joined #openstack-nova | 05:01 | |
| *** jerrygb has joined #openstack-nova | 05:01 | |
| *** kbyrne has joined #openstack-nova | 05:01 | |
| *** edtubill has joined #openstack-nova | 05:01 | |
| *** jroll has quit IRC | 05:02 | |
| *** jroll has joined #openstack-nova | 05:02 | |
| *** jroll is now known as Guest89040 | 05:02 | |
| *** xnox has joined #openstack-nova | 05:02 | |
| *** afazekas has joined #openstack-nova | 05:02 | |
| *** codebauss has joined #openstack-nova | 05:03 | |
| *** sudipto has quit IRC | 05:03 | |
| *** serverascode has joined #openstack-nova | 05:04 | |
| *** sbadia has joined #openstack-nova | 05:04 | |
| *** edtubill has quit IRC | 05:05 | |
| *** salv-orlando has quit IRC | 05:05 | |
| *** mdrabe has joined #openstack-nova | 05:06 | |
| *** jamielennox is now known as jamielennox|away | 05:06 | |
| *** edtubill has joined #openstack-nova | 05:06 | |
| *** Marga_ has joined #openstack-nova | 05:10 | |
| *** Marga_ has quit IRC | 05:11 | |
| *** Marga_ has joined #openstack-nova | 05:11 | |
| *** suro-patz has joined #openstack-nova | 05:12 | |
| *** mdrabe has quit IRC | 05:13 | |
| alex_xu | oomichi: hi, how about we change the test name for https://review.openstack.org/#/c/222068/ | 05:13 |
| alex_xu | oomichi: that is a little confuse for what it is test for | 05:14 |
| oomichi | alex_xu: nice comment for fitting the above test_validate_v2compatible_request :-) | 05:16 |
| alex_xu | oomichi: thanks | 05:16 |
| *** achanda_ has joined #openstack-nova | 05:16 | |
| *** betherly has quit IRC | 05:17 | |
| *** rpodolyaka1 has joined #openstack-nova | 05:17 | |
| *** annasort_ has joined #openstack-nova | 05:17 | |
| *** junjie has quit IRC | 05:18 | |
| *** auggy has quit IRC | 05:18 | |
| *** kgalanov has quit IRC | 05:18 | |
| *** greghaynes has quit IRC | 05:19 | |
| *** tpeoples_away has quit IRC | 05:19 | |
| *** bnemec has joined #openstack-nova | 05:19 | |
| *** rubasov1 has joined #openstack-nova | 05:19 | |
| *** doffm_ has joined #openstack-nova | 05:20 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Add microversions schema unit test for None https://review.openstack.org/222068 | 05:21 |
| *** rpodolyaka1 has quit IRC | 05:22 | |
| oomichi | alex_xu: ^^^ done | 05:22 |
| alex_xu | oomichi: done :) | 05:23 |
| oomichi | alex_xu: :) | 05:23 |
| *** sirushti_ has joined #openstack-nova | 05:24 | |
| *** doffm has quit IRC | 05:25 | |
| *** achanda has quit IRC | 05:25 | |
| *** yonglihe has quit IRC | 05:25 | |
| *** beekneemech has quit IRC | 05:25 | |
| *** rubasov has quit IRC | 05:25 | |
| *** maurosr has quit IRC | 05:25 | |
| *** annasort has quit IRC | 05:25 | |
| *** garyk has quit IRC | 05:25 | |
| *** DuncanT has quit IRC | 05:25 | |
| *** sballe has quit IRC | 05:25 | |
| *** andrewbogott has quit IRC | 05:25 | |
| *** markmcclain has quit IRC | 05:25 | |
| *** sirushti has quit IRC | 05:25 | |
| *** jbernard has quit IRC | 05:25 | |
| *** onder has quit IRC | 05:25 | |
| *** sirushti_ is now known as sirushti | 05:25 | |
| *** maurosr has joined #openstack-nova | 05:26 | |
| *** DuncanT has joined #openstack-nova | 05:26 | |
| *** garyk has joined #openstack-nova | 05:27 | |
| *** tpeoples_away has joined #openstack-nova | 05:28 | |
| *** andrewbogott has joined #openstack-nova | 05:29 | |
| *** auggy has joined #openstack-nova | 05:30 | |
| *** sballe has joined #openstack-nova | 05:30 | |
| *** jbernard has joined #openstack-nova | 05:30 | |
| *** onder has joined #openstack-nova | 05:30 | |
| *** markmcclain has joined #openstack-nova | 05:31 | |
| *** suro-patz has quit IRC | 05:31 | |
| *** gus has quit IRC | 05:33 | |
| *** gus has joined #openstack-nova | 05:34 | |
| *** junjie has joined #openstack-nova | 05:34 | |
| *** greghaynes has joined #openstack-nova | 05:35 | |
| *** yonglihe has joined #openstack-nova | 05:38 | |
| *** eliqiao has joined #openstack-nova | 05:38 | |
| *** jerrygb has quit IRC | 05:38 | |
| *** betherly has joined #openstack-nova | 05:39 | |
| *** kgalanov has joined #openstack-nova | 05:39 | |
| *** eliqiao_ has quit IRC | 05:41 | |
| *** rpodolyaka1 has joined #openstack-nova | 05:42 | |
| *** adam_g` is now known as adam_g | 05:42 | |
| *** adam_g has quit IRC | 05:42 | |
| *** adam_g has joined #openstack-nova | 05:42 | |
| *** rushiagr_away is now known as rushiagr | 05:42 | |
| *** klkumar has quit IRC | 05:44 | |
| *** jmccrory has quit IRC | 05:48 | |
| *** sacharya has quit IRC | 05:50 | |
| *** jmccrory has joined #openstack-nova | 05:51 | |
| *** aswadr has joined #openstack-nova | 05:54 | |
| *** yangyapeng has quit IRC | 05:59 | |
| *** moshele has joined #openstack-nova | 06:00 | |
| *** edtubill has quit IRC | 06:02 | |
| *** edtubill has joined #openstack-nova | 06:05 | |
| *** rushil has joined #openstack-nova | 06:06 | |
| *** unicell has joined #openstack-nova | 06:07 | |
| *** sirushti has quit IRC | 06:11 | |
| *** sirushti has joined #openstack-nova | 06:11 | |
| *** otter768 has joined #openstack-nova | 06:12 | |
| *** otter768 has quit IRC | 06:16 | |
| openstackgerrit | Vincent Untz proposed openstack/python-novaclient: Fix typo in error message https://review.openstack.org/246199 | 06:19 |
| *** gcb has quit IRC | 06:19 | |
| *** abhishekk has joined #openstack-nova | 06:21 | |
| *** rcernin has joined #openstack-nova | 06:22 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/242727 | 06:23 |
| *** aginwala has joined #openstack-nova | 06:24 | |
| *** vilobhmm has quit IRC | 06:25 | |
| *** korzen has joined #openstack-nova | 06:25 | |
| *** aginwala_ has joined #openstack-nova | 06:30 | |
| *** aginwala has quit IRC | 06:32 | |
| *** gcb has joined #openstack-nova | 06:32 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id and user-id when list server-groups https://review.openstack.org/209018 | 06:32 |
| *** rushil has quit IRC | 06:33 | |
| *** markmc has quit IRC | 06:33 | |
| *** markmc has joined #openstack-nova | 06:34 | |
| *** salv-orlando has joined #openstack-nova | 06:34 | |
| *** aginwala_ has quit IRC | 06:35 | |
| *** edtubill has quit IRC | 06:35 | |
| *** pratikmallya has joined #openstack-nova | 06:36 | |
| *** aginwala has joined #openstack-nova | 06:36 | |
| *** salv-orlando has quit IRC | 06:37 | |
| *** Jeffrey4l has quit IRC | 06:39 | |
| *** jerrygb has joined #openstack-nova | 06:39 | |
| *** ankit_ag has joined #openstack-nova | 06:41 | |
| *** venkat_p has left #openstack-nova | 06:41 | |
| *** eliqiao has quit IRC | 06:44 | |
| *** jerrygb has quit IRC | 06:45 | |
| *** eliqiao has joined #openstack-nova | 06:45 | |
| *** sudipto has joined #openstack-nova | 06:47 | |
| *** sileht has joined #openstack-nova | 06:49 | |
| *** josecastroleon has joined #openstack-nova | 06:50 | |
| *** Jeffrey4l has joined #openstack-nova | 06:51 | |
| *** angdraug has joined #openstack-nova | 06:53 | |
| *** aswadr has quit IRC | 06:55 | |
| *** rpodolyaka1 has quit IRC | 06:56 | |
| *** Yanyan has quit IRC | 07:00 | |
| *** unicell1 has joined #openstack-nova | 07:01 | |
| *** dhellmann has quit IRC | 07:01 | |
| *** dhellmann has joined #openstack-nova | 07:02 | |
| *** unicell has quit IRC | 07:02 | |
| openstackgerrit | Wen Zhi Yu proposed openstack/nova: Use SHA-256 algorithm in nova/crypto.py https://review.openstack.org/246217 | 07:03 |
| *** pkoniszewski has quit IRC | 07:04 | |
| *** yonglihe has quit IRC | 07:04 | |
| *** unicell has joined #openstack-nova | 07:05 | |
| *** unicell1 has quit IRC | 07:05 | |
| *** penick has joined #openstack-nova | 07:07 | |
| *** rpodolyaka1 has joined #openstack-nova | 07:07 | |
| *** sahid has joined #openstack-nova | 07:08 | |
| *** mjura has joined #openstack-nova | 07:09 | |
| *** moshele has quit IRC | 07:10 | |
| *** mleroy has quit IRC | 07:12 | |
| openstackgerrit | Tang Chen proposed openstack/nova-specs: Support triggering crash dump in a server. https://review.openstack.org/229255 | 07:13 |
| *** deepakcs has joined #openstack-nova | 07:15 | |
| tangchen__ | alex_xu: Hi alex, on November 19, we are going to review spec, right ? | 07:18 |
| tangchen__ | alex_xu: What time shall we start on that day ? | 07:19 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova-specs: Pause VM during live migration https://review.openstack.org/229040 | 07:19 |
| *** moshele has joined #openstack-nova | 07:19 | |
| alex_xu | tangchen__: anytime in that day | 07:20 |
| *** jyuso has quit IRC | 07:20 | |
| *** jyuso has joined #openstack-nova | 07:21 | |
| alex_xu | tangchen__: that is for local time | 07:21 |
| sahid | jaypipes: hello, any chance you take a look in my PR, also please when you have moment let me know how i can try to involve myself better, don't know what was the concensus about design | 07:21 |
| *** pratikmallya has quit IRC | 07:22 | |
| *** rpodolyaka1 has quit IRC | 07:23 | |
| *** paul-carlton has joined #openstack-nova | 07:25 | |
| *** rpodolyaka1 has joined #openstack-nova | 07:25 | |
| *** Jeffrey4l has quit IRC | 07:26 | |
| *** garyk1 has joined #openstack-nova | 07:28 | |
| openstackgerrit | Tom Patzig proposed openstack/nova: Add 'insecure' option to ironic sect in nova.conf https://review.openstack.org/232560 | 07:28 |
| *** lpetrut has joined #openstack-nova | 07:29 | |
| tangchen__ | alex_xu: Thanks. :) | 07:32 |
| alex_xu | tangchen__: np :) | 07:32 |
| *** alexschm has joined #openstack-nova | 07:34 | |
| openstackgerrit | Alex Xu proposed openstack/nova: POC: generate swagger spec from api code base https://review.openstack.org/233446 | 07:35 |
| *** achanda_ has quit IRC | 07:36 | |
| *** nkrinner has joined #openstack-nova | 07:37 | |
| *** Jeffrey4l has joined #openstack-nova | 07:39 | |
| alex_xu | oomichi: ^ something you expected :) | 07:39 |
| oomichi | alex_xu: thanks, will see it :-) | 07:40 |
| *** artom__ has joined #openstack-nova | 07:40 | |
| alex_xu | oomichi: thanks :) | 07:40 |
| *** artom_ has quit IRC | 07:40 | |
| *** artom__ has quit IRC | 07:41 | |
| *** artom__ has joined #openstack-nova | 07:41 | |
| *** mleroy has joined #openstack-nova | 07:45 | |
| *** aginwala has quit IRC | 07:45 | |
| *** aginwala has joined #openstack-nova | 07:49 | |
| *** longsube has joined #openstack-nova | 07:50 | |
| *** Jeffrey4l has quit IRC | 07:53 | |
| *** longsube has quit IRC | 07:54 | |
| *** irenab has quit IRC | 07:54 | |
| *** irenab has joined #openstack-nova | 07:54 | |
| *** romainh has joined #openstack-nova | 07:56 | |
| *** tyagiprince has joined #openstack-nova | 07:57 | |
| tyagiprince | hey everyone.. I am facing problem in booting up my instance.. It gives the following error.. ERROR (ClientException): The server has either erred or is incapable of performing the requested operation. (HTTP 500) | 07:57 |
| tyagiprince | unable to find the cause root of this problem.. | 07:57 |
| *** gongysh has joined #openstack-nova | 07:58 | |
| *** rotbeard has joined #openstack-nova | 07:58 | |
| tangchen__ | tyagiprince: Hi, what is the status of your instance ? | 07:59 |
| tangchen__ | tyagiprince: If it is ERROR, you can use nova reset-state --active. | 08:00 |
| tangchen__ | tyagiprince: And try to boot it again. | 08:00 |
| tangchen__ | tyagiprince: And also, please try to see the logs. Maybe you can find something. | 08:01 |
| *** rotbart has joined #openstack-nova | 08:02 | |
| *** rotbeard has quit IRC | 08:04 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs: Make block_migration and host flags in live-migration API https://review.openstack.org/245543 | 08:04 |
| *** tyagiprince has quit IRC | 08:06 | |
| *** lpetrut has quit IRC | 08:08 | |
| moshele | johnthetubaguy, danp: can you please review https://review.openstack.org/#/c/199488/ and https://review.openstack.org/#/c/227160/ | 08:13 |
| *** otter768 has joined #openstack-nova | 08:13 | |
| *** jichen has joined #openstack-nova | 08:14 | |
| *** matrohon has joined #openstack-nova | 08:15 | |
| *** scheuran has joined #openstack-nova | 08:15 | |
| *** otter768 has quit IRC | 08:18 | |
| *** deepakcs has quit IRC | 08:22 | |
| *** ildikov has quit IRC | 08:28 | |
| *** mpavone has joined #openstack-nova | 08:30 | |
| *** penick has quit IRC | 08:33 | |
| *** rdopiera has joined #openstack-nova | 08:37 | |
| *** liverpooler has joined #openstack-nova | 08:38 | |
| *** lpetrut has joined #openstack-nova | 08:38 | |
| *** jerrygb has joined #openstack-nova | 08:40 | |
| *** liverpoo1er has joined #openstack-nova | 08:41 | |
| *** jlanoux has joined #openstack-nova | 08:41 | |
| *** liverpooler has quit IRC | 08:43 | |
| *** mpavone has quit IRC | 08:43 | |
| jichen | alex_xu: hi, another look at https://review.openstack.org/#/c/232721/? | 08:43 |
| *** korzen has quit IRC | 08:43 | |
| markus_z | tonyb: I guess we both have the most terrible timezones when trying to communicate in real time :) | 08:44 |
| *** deepakcs has joined #openstack-nova | 08:44 | |
| tonyb | markus_z: Yeah, but as it turns out I'm on now ;p | 08:44 |
| tonyb | markus_z: how can I help? | 08:44 |
| markus_z | tonyb: woah, hey :) | 08:44 |
| markus_z | tonyb: about the bleeding edge libvirt gate job | 08:44 |
| alex_xu | jichen: emm...just looks like strange, we have some fake at db layer and one at compute api layer | 08:45 |
| tonyb | markus_z: Ahh okay. | 08:45 |
| *** tyagiprince has joined #openstack-nova | 08:45 | |
| *** salv-orlando has joined #openstack-nova | 08:45 | |
| tyagiprince | <tangchen>: instance is unable to boot.. when i boot an instance with --nic net-id="something-here", it gives an error after a while.. If I dont pass this argument the instance gets booted up and comes in active state | 08:45 |
| markus_z | tonyb: I haven't created a gate job before and was unsure where exactly to start | 08:45 |
| tonyb | markus_z: So my idea is to make a devstack plugin that will add $repo which contains newwer libvirt and qwemu packages | 08:46 |
| jichen | um... maybe, but I don't think it has much effect, right? we focus on unit test for the code path instead of others, right? | 08:46 |
| tyagiprince | <tangchen__>: instance is unable to boot.. when i boot an instance with --nic net-id="something-here", it gives an error after a while.. If I dont pass this argument the instance gets booted up and comes in active state | 08:46 |
| tonyb | markus_z: To be honnest I'm not 100% certain myself ;P | 08:46 |
| tonyb | markus_z: but I figure the plugin is a good place then we'll work out how to use it ;P | 08:46 |
| alex_xu | jichen: yea, ideally we should use mock instead of those stubs | 08:46 |
| *** jerrygb has quit IRC | 08:46 | |
| alex_xu | jichen: the stub of key_pair_create is still needed? as we already have mock at compute api layer? | 08:47 |
| *** rpodolyaka1 has quit IRC | 08:47 | |
| markus_z | tonyb: Yeah, that's a better start than what I head in mind. I wanted to create a node which already has the latest libvirt/qemu. | 08:47 |
| jichen | alex_xu: for other test cases ... | 08:47 |
| jichen | for simplility and less changes needed | 08:47 |
| tonyb | markus_z: Yeah that'll be hard. | 08:47 |
| markus_z | tonyb: yeah, sounds pretty dumb after I typed it :D | 08:48 |
| tonyb | markus_z: Although it might not be terrible to get a nodepool image for Ubuntu 15.10 | 08:48 |
| tonyb | that would be newer, but the custom repo could be build daily or whenever we thought there was something of interest to test | 08:48 |
| bauzas | good morning Nova | 08:48 |
| tonyb | heck the devstack plugin can even try different repos | 08:49 |
| markus_z | tonyb: Yepp, sounds more flexible | 08:49 |
| tonyb | I feel like having conttol over it would be good for testing the latest and greatest. | 08:49 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Live migration failure in API doesn't revert task_state of vm https://review.openstack.org/168916 | 08:50 |
| tonyb | markus_z: I don't have a good handle on how we test the serial console stuff with tempest. | 08:50 |
| tonyb | markus_z: We should probably talk to mtreinish and sdague about that. | 08:50 |
| markus_z | tonyb: OK, assumed we have a node with U1510 and the devstack plugin, the gate job will then trigger the download from $repo and build it from source? | 08:51 |
| *** deepakcs has quit IRC | 08:52 | |
| tonyb | markus_z: No I was thinking that $repo would contain Ubuntu packages for qemu/libvirt | 08:52 |
| *** sahid_ has joined #openstack-nova | 08:53 | |
| *** mpavone has joined #openstack-nova | 08:53 | |
| markus_z | tonyb: I used "novaconsole" from larsks back then to test it manually: https://github.com/larsks/novaconsole/ | 08:53 |
| *** mpavone has quit IRC | 08:53 | |
| tonyb | markus_z: I we *could* build form source but thsat creates a massive delay in the dsvm jobs and an additional failure mode | 08:53 |
| alex_xu | jichen: strange...I can pass the test if I remove the line 364 | 08:53 |
| *** sahid has quit IRC | 08:53 | |
| tyagiprince | hey people need help with nova boot.. instance is unable to boot.. when i boot an instance with --nic net-id="something-here", it gives an error after a while.. If I dont pass this argument the instance gets booted up and comes in active state | 08:53 |
| tyagiprince | does anyone know what might be the problem | 08:53 |
| jichen | alex_xu: ok only 364, might, I removed 360-367 for test | 08:53 |
| alex_xu | jichen: ok | 08:54 |
| bauzas | tyagiprince: hi, I feel you should get more help in the #openstack channel | 08:54 |
| bauzas | tyagiprince: see that channel $topic | 08:54 |
| tyagiprince | thanks bauzas | 08:54 |
| markus_z | tonyb: Right, the job would take longer. I was just thinking about the delay when upstream libvirt provides the new version until Ubuntu has it ins packages. | 08:54 |
| bauzas | tyagiprince: although I guess you'd get more support in the general channel by US working hours | 08:54 |
| markus_z | *in its | 08:55 |
| tonyb | markus_z: Well I was thinking that we'd build the packages ourselves | 08:55 |
| markus_z | tonyb: OK, got it | 08:55 |
| tonyb | markus_z: initially I was thinking I'd build $latest version. | 08:55 |
| *** ihrachys has joined #openstack-nova | 08:56 | |
| tonyb | markus_z: but we could have nightly builds | 08:56 |
| tonyb | markus_z: of course I'll look again to see if anyone is already doing these builds | 08:56 |
| tonyb | markus_z: I know they used for for Fedora but Fedora in the gate can be problematic | 08:56 |
| openstackgerrit | James Page proposed openstack/nova: Add 'lxd' to the list of recognized hypervisors https://review.openstack.org/245811 | 08:56 |
| markus_z | tonyb: "nightly" gets blurry for me with all the different timezones :p | 08:57 |
| bauzas | tonyb: why Fedora in the gate seems problematic ? | 08:57 |
| bauzas | tonyb: I thought there were some efforts for running it | 08:57 |
| tonyb | markus_z: Well checkout git master at 00:00 UTC ;P | 08:57 |
| *** sahid_ has quit IRC | 08:57 | |
| *** sahid has joined #openstack-nova | 08:58 | |
| tonyb | bauzas: Sure, its there. Last time I looked it was F21 (which is now EOLd) | 08:58 |
| tonyb | bauzas: I'd be happy if I was wrong and it was all F23 and all signing and dancing :) | 08:58 |
| * markus_z will skip the dancing part | 08:59 | |
| *** salv-orlando has quit IRC | 08:59 | |
| bauzas | tonyb: I mean, I just think that using Fedora would help, right? | 08:59 |
| jichen | alex_xu: I will try this and update patch if it pass | 08:59 |
| bauzas | tonyb: if so, I can discuss internally to see how we can help you | 09:00 |
| tonyb | bauzas: As long as we didn't have to close a backlog of other issues sure. | 09:00 |
| tonyb | bauzas: Cool :) | 09:00 |
| *** markus_z has quit IRC | 09:00 | |
| bauzas | tonyb: fersur, you not happy with getting lots of regressions ? nah. | 09:00 |
| tonyb | bauzas: For the record I'm a fedora package maintainer and use it a lot at home so I have no problem with Fedora | 09:00 |
| bauzas | tonyb: I mean, I'm not really opiniated | 09:01 |
| tonyb | bauzas: :) | 09:01 |
| bauzas | tonyb: but it seems Fedora is a bit more goodness for getting the latest libvirt releases, right? | 09:01 |
| *** exploreshaifali has joined #openstack-nova | 09:02 | |
| bauzas | tonyb: so, I can try to see how I can bring back some help for you | 09:02 |
| tonyb | bauzas: Yeah There was someone doing builds for supported Fedora releases on each libvirt release | 09:02 |
| tonyb | bauzas: That'd be very welcome. | 09:02 |
| *** markus_z has joined #openstack-nova | 09:03 | |
| *** xek has joined #openstack-nova | 09:03 | |
| bauzas | tonyb: if that doesn't match because of X, then feel free to play with Ubuntu :D | 09:03 |
| markus_z | tonyb: sorry, my network kicked me out | 09:03 |
| *** lucas-afk is now known as lucasagomes | 09:03 | |
| tonyb | bauzas: I can build .debs but .rpms make more sense to me :) | 09:04 |
| tonyb | markus_z: np | 09:04 |
| bauzas | hah | 09:04 |
| *** deepakcs has joined #openstack-nova | 09:04 | |
| *** eglynn_ has joined #openstack-nova | 09:06 | |
| *** ildikov has joined #openstack-nova | 09:07 | |
| *** Marga_ has quit IRC | 09:07 | |
| markus_z | tonyb: bauzas: Just to double-check if I understand it. Please correct me if I'm wrong. We have 4 layers now. 1) $repo will contain the $latest libvirt version 2) We need a node image which can run $latest libvirt version (maybe Ubuntu15.10) 3) Devstack pulls $latest libvirt from $repo in a gate test 4) tempest needs addional test cases to test "serial console" | 09:07 |
| markus_z | It makes sense in my head, which confuses me... | 09:08 |
| *** liverpoo1er has quit IRC | 09:11 | |
| *** liverpooler has joined #openstack-nova | 09:11 | |
| tonyb | markus_z: 1) We create $repo to contain latest released libvirt and qemu compatible with $nodepool image 2) we create a devstack job that ultilises $repo 3) We modifiy tempest to be capable of dealing with both serial console and normal "log to file" consoles | 09:12 |
| openstackgerrit | jichenjc proposed openstack/nova: use NoDBTestCase for KeypairPolicyTest https://review.openstack.org/232721 | 09:12 |
| tonyb | markus_z: eventually 3 goes away once we have a unified way of dealing with consoles regardless of how the operator has configured the system. | 09:12 |
| tonyb | markus_z: does that make sense? | 09:12 |
| *** deepakcs has quit IRC | 09:13 | |
| openstackgerrit | ShaoHe Feng proposed openstack/nova: do not log twice with different level. https://review.openstack.org/246269 | 09:14 |
| markus_z | tonyb: Yepp, it does. I can work on 3. I don't think that it goes away, because we have 2 separate REST API's we have to fulfill. | 09:14 |
| *** yassine has joined #openstack-nova | 09:15 | |
| *** salv-orlando has joined #openstack-nova | 09:15 | |
| *** salv-orlando has quit IRC | 09:15 | |
| markus_z | I also think 3 can be implemented independently from 1+2 | 09:15 |
| tonyb | markus_z: You'll want to sync up with mtreinish and sdague on that. | 09:15 |
| *** klkumar has joined #openstack-nova | 09:15 | |
| *** salv-orlando has joined #openstack-nova | 09:15 | |
| tonyb | markus_z: I could be very wrong about needing it. | 09:15 |
| *** deepakcs has joined #openstack-nova | 09:16 | |
| *** moshele has quit IRC | 09:16 | |
| *** Marga_ has joined #openstack-nova | 09:16 | |
| markus_z | tonyb: OK, I'm jumping to conclusions right now. | 09:17 |
| markus_z | tonyb: I think I get the flow and potential work items. I would post this on the ML to get also feedback from the infra folks? | 09:17 |
| tonyb | markus_z: Sounds good. | 09:17 |
| *** lpetrut has quit IRC | 09:18 | |
| markus_z | tonyb: Awesome, thanks a lot! Just chime in on the ML post when I say something crazy | 09:18 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Live migration failure in API leaves VM in MIGRATING state https://review.openstack.org/168916 | 09:18 |
| tonyb | markus_z: Sure :) | 09:18 |
| markus_z | cool, have nice evening | 09:19 |
| tonyb | Has anyone seen 'TypeError: Cannot mock a MockAnything! Did you remember to call UnsetStubs in your previous test?' before? | 09:19 |
| *** salv-orlando has quit IRC | 09:20 | |
| *** salv-orlando has joined #openstack-nova | 09:20 | |
| *** moshele has joined #openstack-nova | 09:21 | |
| *** klkumar has quit IRC | 09:21 | |
| openstackgerrit | li,chen proposed openstack/nova: Add unit tests for nova.virt.firewall.IpTablesFirewallDriver https://review.openstack.org/246274 | 09:22 |
| *** ekarlso has quit IRC | 09:22 | |
| *** rpodolyaka1 has joined #openstack-nova | 09:23 | |
| markus_z | tonyb: I could only imagine a missing teardown in the test case class | 09:24 |
| *** ihrachys has quit IRC | 09:24 | |
| tonyb | markus_z: Hmm perhaps. | 09:24 |
| sudipto | markus_z, +1 | 09:24 |
| tonyb | The same code passed the check pipline and failed in the gate so I was thinking that something other than the code was to blame ... | 09:25 |
| *** ihrachys has joined #openstack-nova | 09:26 | |
| *** mgoddard has joined #openstack-nova | 09:27 | |
| markus_z | Could still be the code. AFAIK the test methods get executed in a random order. Which results in different starting positions when a teardown is missing. | 09:27 |
| markus_z | Which would explain why it passes sometimes. | 09:28 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: WIP: enginefacade: 'instance' https://review.openstack.org/244146 | 09:29 |
| tonyb | markus_z: Oh. I have to admit I didn't know that. (about the order of execution) | 09:30 |
| markus_z | tonyb: wohoo, I could help :D | 09:30 |
| *** eglynn_ is now known as eglynn | 09:31 | |
| *** openstackgerrit has quit IRC | 09:31 | |
| *** ihrachys has quit IRC | 09:31 | |
| * tonyb now debates if I shoudl just recheck this review. | 09:31 | |
| *** openstackgerrit has joined #openstack-nova | 09:32 | |
| markus_z | Which review is it? | 09:32 |
| *** klkumar has joined #openstack-nova | 09:33 | |
| tonyb | https://review.openstack.org/#/c/246013/ | 09:34 |
| tonyb | markus_z: it | 09:34 |
| *** aginwala has quit IRC | 09:34 | |
| tonyb | s kinda slowing things down | 09:34 |
| johnthetubaguy | tonyb: i did see that one before, but not in the gate | 09:35 |
| *** ihrachys has joined #openstack-nova | 09:35 | |
| bauzas | tonyb: I remember that story, sec | 09:35 |
| tonyb | sure | 09:35 |
| bauzas | johnthetubaguy: yeah that | 09:35 |
| *** chenli has quit IRC | 09:35 | |
| * tonyb goes to put the chickens in for the night .... | 09:35 | |
| bauzas | it's a py34 job failing | 09:36 |
| bauzas | since py34 is only running a few tests, I wonder if that's just now a brand new test | 09:36 |
| bauzas | that said, it should have failed in the check pipeline, not in gate | 09:36 |
| *** ekarlso has joined #openstack-nova | 09:37 | |
| *** achanda has joined #openstack-nova | 09:38 | |
| *** ekarlso has quit IRC | 09:38 | |
| *** ekarlso has joined #openstack-nova | 09:38 | |
| kashyap | markus_z: (Just catching up on scroll.) There was a long discussion previously about running newest libvirt on the Gate, trying to fish that. . . | 09:39 |
| tonyb | bauzas: I checked and nothing merged between the 2 runs so it's not like the failing test was added in between | 09:39 |
| bauzas | tonyb: yup | 09:39 |
| tonyb | kashyap: We're only talking about doing that for *some* runs not all. | 09:40 |
| oomichi | alex_xu: thanks for https://review.openstack.org/#/c/233446 , that is very cool :-) | 09:40 |
| bauzas | tonyb: checking e-r | 09:40 |
| tonyb | kashyap: Which is different to last time it came up ... | 09:40 |
| kashyap | tonyb: Still reading scroll, what are these "some" runs? (If you still have time.) | 09:40 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: WIP: enginefacade: 'instance' https://review.openstack.org/244146 | 09:40 |
| tonyb | kashyap: well intially it'd be only on the experimental queue, then (clearly) a dedicated job | 09:40 |
| tonyb | kashyap: it's not in the scrool back, but basically we want to be able test code paths that have a requirement on libvirt that's newer than in $distro | 09:41 |
| kashyap | tonyb: Yes, experimental-only sounds reasonable. And, we're diligent in analyzing the results :-) | 09:42 |
| *** danpb has joined #openstack-nova | 09:42 | |
| tonyb | kashyap: yeah to start with :) | 09:43 |
| kashyap | s/we're/if we're/ | 09:43 |
| tonyb | kashyap: perhaps for all of mitaka, that parts not very well defined yet ;P | 09:43 |
| *** rpodolyaka1 has quit IRC | 09:43 | |
| kashyap | Cool, is there an intention to start a thread about it on ML? | 09:44 |
| *** claudiub has joined #openstack-nova | 09:44 | |
| *** rpodolyaka1 has joined #openstack-nova | 09:45 | |
| tonyb | kashyap: I believe that markus_z will commited to doing that. | 09:45 |
| markus_z | kashyap: tonyb: Yepp, I'm going to post that on the ML today to get more feedback. | 09:46 |
| *** exploreshaifali has quit IRC | 09:47 | |
| kashyap | markus_z: Cool. | 09:48 |
| *** e0ne has joined #openstack-nova | 09:48 | |
| johnthetubaguy | tonyb: totally sounds like a random test re-ordering bug | 09:49 |
| tonyb | johnthetubaguy: okay. | 09:50 |
| *** Marga_ has quit IRC | 09:50 | |
| * tonyb recheck's it | 09:50 | |
| *** mgoddard has quit IRC | 09:50 | |
| johnthetubaguy | tonyb: how is the pass rate, its quite possible not that patch | 09:51 |
| bauzas | johnthetubaguy: +1 | 09:51 |
| bauzas | tonyb: johnthetubaguy: I checked logstash and it only occurred as a spike | 09:51 |
| tonyb | Huh, how does a review get into the gate pipeline and fail pep8? | 09:51 |
| *** belmoreira has joined #openstack-nova | 09:52 | |
| bauzas | meh, the py34 output formatting breaks logstash... | 09:52 |
| bauzas | it makes the message very big | 09:52 |
| johnthetubaguy | tonyb: thats oddd | 09:52 |
| bauzas | tonyb: which one ? | 09:52 |
| bauzas | tonyb: someone the PEP8 check fails because of some other causes (timeout or file upload failing etc.) | 09:53 |
| tonyb | https://review.openstack.org/#/c/218644/1 | 09:53 |
| tonyb | it went throught check in August to it's possible we changed the rules in between | 09:53 |
| johnthetubaguy | yeah, change in rules would do it | 09:54 |
| bauzas | tonyb: 218644 is okay | 09:54 |
| bauzas | AFAICS | 09:54 |
| bauzas | unless I'm blind | 09:54 |
| johnthetubaguy | so we do merge lots of stuff on top of each other | 09:54 |
| johnthetubaguy | so it could be a patch underneath that fails them all, I guess | 09:55 |
| tonyb | johnthetubaguy: Yeah that was it. | 09:55 |
| *** ihrachys has quit IRC | 09:55 | |
| johnthetubaguy | cools | 09:55 |
| tonyb | johnthetubaguy, bauzas: It just got re-queued in zuul .... | 09:55 |
| bauzas | tonyb: haha, got it | 09:55 |
| johnthetubaguy | we used to recheck before joining gate, but it did waste a lot of resources I guess | 09:55 |
| johnthetubaguy | (well and multiplied the issue of failure rates) | 09:56 |
| tonyb | Ah so much history I don't have ... that's good and bad ;P | 09:56 |
| bauzas | johnthetubaguy: right https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L53-L56 | 09:56 |
| *** ekarlso has quit IRC | 09:57 | |
| bauzas | johnthetubaguy: tonyb: there is no longer a newer-than tag | 09:58 |
| *** achanda has quit IRC | 10:00 | |
| *** rk4n has joined #openstack-nova | 10:00 | |
| bauzas | oooooh, the liberty gate is also impacted by the o.vo issue... | 10:01 |
| *** Sree_ has joined #openstack-nova | 10:01 | |
| tonyb | bauzas: Yeah. | 10:01 |
| bauzas | tonyb: heard of any backport ? | 10:02 |
| *** Sree_ is now known as Guest50273 | 10:02 | |
| bauzas | tonyb: I left the chan before the issue popped up, so I need to scroll back | 10:02 |
| tonyb | bauzas: Yeah but I can't approve it and melwitt had a question about it .... | 10:02 |
| *** tyagiprince has quit IRC | 10:02 | |
| johnthetubaguy | we don't pin on unit tests yet, as I understand it | 10:02 |
| johnthetubaguy | not sure why we don't do that for stable/liberty | 10:02 |
| bauzas | links ? | 10:02 |
| openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Cope with more Cinder backends https://review.openstack.org/244789 | 10:03 |
| bauzas | FWIW, the reno liberty changes are now stuck because of that - which blocks the reno CI job - which blocks us approving relnote changes | 10:03 |
| tonyb | bauzas: melwitt's question is a comment on your backport: | 10:04 |
| bauzas | tonyb: which backport ? | 10:04 |
| johnthetubaguy | hmm, I am unsure now, maybe they are pinned by upper constraints | 10:05 |
| bauzas | https://review.openstack.org/#/q/I2aaa42784b2cc5c6898a3fac0c951dd57dd4f4c6,n,z | 10:05 |
| tonyb | bauzas: https://review.openstack.org/#/c/246022/ Oh dansmith did it not you | 10:05 |
| *** markvoelker has quit IRC | 10:05 | |
| *** Sree has quit IRC | 10:05 | |
| bauzas | tonyb: okay, I got the change-id | 10:05 |
| tonyb | bauzas: Yeah. | 10:05 |
| tonyb | bauzas: I was checking .... | 10:05 |
| openstackgerrit | lyanchih proposed openstack/nova: NUMATopologyFilter raise exception and not continue filter next node https://review.openstack.org/200630 | 10:06 |
| bauzas | okay, I need to understand what was the problem and what was doing the fix | 10:06 |
| openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Fix race in rotate_xen_guest_logs https://review.openstack.org/245898 | 10:07 |
| *** e0ne has quit IRC | 10:07 | |
| *** mgoddard has joined #openstack-nova | 10:08 | |
| *** ociuhandu has quit IRC | 10:09 | |
| *** achanda has joined #openstack-nova | 10:12 | |
| *** ildikov has quit IRC | 10:12 | |
| *** daemontool has joined #openstack-nova | 10:12 | |
| *** ihrachys has joined #openstack-nova | 10:13 | |
| *** eliqiao has quit IRC | 10:13 | |
| *** ptm_away is now known as PaulMurray | 10:13 | |
| *** otter768 has joined #openstack-nova | 10:14 | |
| *** daemontool has quit IRC | 10:14 | |
| *** alex_klimov has joined #openstack-nova | 10:14 | |
| openstackgerrit | Zhongcheng Lao proposed openstack/nova: VMware: Omit exceptions during snapshot cleanup https://review.openstack.org/216086 | 10:14 |
| *** daemontool has joined #openstack-nova | 10:15 | |
| alex_xu | oomichi: thanks, glad to see you like it | 10:15 |
| oomichi | alex_xu: yeah, very nice for me. can you check my comment again? | 10:15 |
| *** eliqiao has joined #openstack-nova | 10:16 | |
| oomichi | alex_xu: and we need to build a consensus for the direction soon for making it in Mitaka | 10:16 |
| alex_xu | oomichi: yea, I checked, I remember swagger support something for include | 10:16 |
| openstackgerrit | Maxime Leroy proposed openstack/nova: Libvirt: Support fp plug in vhostuser vif https://review.openstack.org/245369 | 10:16 |
| openstackgerrit | Maxime Leroy proposed openstack/nova: Libvirt: Support ovs fp plug in vhostuser vif https://review.openstack.org/237680 | 10:16 |
| alex_xu | oomichi: but let me check more to ensure that | 10:16 |
| *** achanda has quit IRC | 10:16 | |
| alex_xu | oomichi: yea | 10:16 |
| *** Guest50273 has quit IRC | 10:17 | |
| oomichi | alex_xu: api thing needs a lot of time/patches always ;-) | 10:17 |
| alex_xu | oomichi: what suggestion you have for build a consensus direction | 10:17 |
| *** Sree has joined #openstack-nova | 10:17 | |
| alex_xu | oomichi: yea...a lot | 10:17 |
| *** EinstCra_ has quit IRC | 10:18 | |
| *** otter768 has quit IRC | 10:18 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Added support for new block device format in Hyper-V https://review.openstack.org/246298 | 10:18 |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova: Added support for new block device format in vmops https://review.openstack.org/246299 | 10:18 |
| oomichi | alex_xu: need to update the spec for getting a consensus | 10:18 |
| alex_xu | danpb: PaulMurray, good morning, hope you can give some opinion about whether this propose is useful for live-migration works https://review.openstack.org/245543, thanks in advance! | 10:19 |
| alex_xu | oomichi: yea, also need talk with doc team, I missed that in this week | 10:19 |
| alex_xu | oomichi: I have a little worry about this https://github.com/russell/fairy-slipper didn't very active recently | 10:20 |
| johnthetubaguy | alex_xu: is that the bit that converts swagger into html output? | 10:20 |
| alex_xu | johnthetubaguy: yea | 10:21 |
| *** lpetrut has joined #openstack-nova | 10:21 | |
| oomichi | alex_xu: yeah, I am imaging russell's script need to be merged into tempest-lib in long-term I guess | 10:21 |
| PaulMurray | alex_xu, looking | 10:22 |
| oomichi | alex_xu: but it should be out of scope now from swagger thing | 10:22 |
| alex_xu | PaulMurray: thanks :) | 10:22 |
| alex_xu | oomichi: ok | 10:22 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: WIP: enginefacade: 'security_group' https://review.openstack.org/246302 | 10:24 |
| sahid | hello johnthetubaguy i saw that you made moving branch realtime but unfortunatly you stopped to review, did you find a problem or something like that ? | 10:25 |
| johnthetubaguy | sahid: ran out of hours in the day, and I think that went deep into libvirt bits I don't really know about | 10:26 |
| *** exploreshaifali has joined #openstack-nova | 10:27 | |
| *** ihrachys has quit IRC | 10:27 | |
| sahid | johnthetubaguy: yes unfortunatly only danpb is involved on libvirt driver and he already have acked them | 10:28 |
| sahid | thanks | 10:28 |
| *** subscope has joined #openstack-nova | 10:30 | |
| *** alexschm has quit IRC | 10:31 | |
| *** haomaiwa_ has quit IRC | 10:31 | |
| *** openstackgerrit has quit IRC | 10:31 | |
| *** ildikov has joined #openstack-nova | 10:31 | |
| *** thorst has joined #openstack-nova | 10:32 | |
| PaulMurray | alex_xu, seems related to https://review.openstack.org/#/c/227278/ | 10:32 |
| *** openstackgerrit has joined #openstack-nova | 10:32 | |
| *** haomaiwang has joined #openstack-nova | 10:32 | |
| PaulMurray | I think ultimately there may be no need for the block-migration flag altogether, but maybe not just yet | 10:32 |
| PaulMurray | Personally I think the spec is in scope for live migration subteam | 10:32 |
| PaulMurray | I'll add it to the pages for review | 10:33 |
| alex_xu | PaulMurray: thanks :) | 10:33 |
| alex_xu | PaulMurray: I will take a look at the patch you pointed out | 10:33 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Fixes disk overhead claim issue https://review.openstack.org/234283 | 10:33 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Fixes disk overhead claim issue https://review.openstack.org/246307 | 10:33 |
| *** thorst has quit IRC | 10:35 | |
| *** ekarlso has joined #openstack-nova | 10:36 | |
| johnthetubaguy | PaulMurray: +1 for removing that as soon as we can | 10:36 |
| *** ociuhandu has joined #openstack-nova | 10:37 | |
| *** Sree has quit IRC | 10:38 | |
| *** achanda has joined #openstack-nova | 10:38 | |
| *** zhenguo has quit IRC | 10:38 | |
| *** Sree has joined #openstack-nova | 10:39 | |
| *** ekarlso has quit IRC | 10:39 | |
| PaulMurray | johnthetubaguy, do you know if its needed for non-libvirt hypervisors? | 10:39 |
| claudiub | what is needed by other hypervisors? | 10:40 |
| johnthetubaguy | PaulMurray: as far as I know, its mostly as stupid with the other hypervisors | 10:40 |
| johnthetubaguy | block_migrate flag for live-migrate | 10:40 |
| alex_xu | johnthetubaguy: PaulMurray I thought, there will have different shared storage pools in the env, so block_migration can be used control for whether the live-migration will across the shared storage pool | 10:40 |
| *** ihrachys has joined #openstack-nova | 10:40 | |
| claudiub | ah, I see. | 10:41 |
| johnthetubaguy | alex_xu: possibly, but we should know that an just to the right thing | 10:41 |
| johnthetubaguy | now there is a use cases where we might want to say, make sure no block migration, error out if that needed | 10:41 |
| johnthetubaguy | but I much prefer the idea of "doing whats needed" by default | 10:42 |
| johnthetubaguy | leaking too many internal details out the API with the current approach | 10:42 |
| johnthetubaguy | ... I have been meaning to sort out that migrate/live-migrate API for years, just never had the time to do it, so happy to see movement in that direction | 10:43 |
| *** jerrygb has joined #openstack-nova | 10:43 | |
| PaulMurray | johnthetubaguy, I thought same, but to be honest, its a bit like saying "do X but tell me if you think I'm wrong because I can't be bothered to figure it out" :) | 10:43 |
| alex_xu | johnthetubaguy: yea, agree | 10:43 |
| *** Sree has quit IRC | 10:43 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Compute: Adds driver disk_gb instance overhead estimation https://review.openstack.org/234283 | 10:44 |
| johnthetubaguy | PaulMurray: heh | 10:44 |
| PaulMurray | johnthetubaguy, I tended to agree with jaypipes comment in the pause VM spec https://review.openstack.org/#/c/229040/ | 10:44 |
| PaulMurray | I know we started with a "force" name and decided against it | 10:45 |
| PaulMurray | but there does seem to be somehting more generic that can be done instead of pause | 10:45 |
| PaulMurray | in the future | 10:45 |
| PaulMurray | like: invoke agressive migration algorithm even though its bad for the user | 10:45 |
| PaulMurray | Any htoughts? | 10:46 |
| johnthetubaguy | "force-complete" makes more sense to me | 10:46 |
| johnthetubaguy | its clearer about the intention | 10:46 |
| *** haomaiwang has quit IRC | 10:47 | |
| *** jerrygb has quit IRC | 10:47 | |
| PaulMurray | Yes (although I can never settle on a name so I'll leave that to others) | 10:48 |
| PaulMurray | Just seems to be far from normal pause API in my mind now | 10:48 |
| PaulMurray | Appart from the exact name I think the spec is ready | 10:48 |
| *** haomaiwang has joined #openstack-nova | 10:49 | |
| *** haomaiwang has quit IRC | 10:49 | |
| *** haomaiwa_ has joined #openstack-nova | 10:49 | |
| *** anshprat_ has joined #openstack-nova | 10:49 | |
| *** rpodolyaka1 has quit IRC | 10:50 | |
| *** subscope has quit IRC | 10:50 | |
| *** rk4n has quit IRC | 10:50 | |
| anshprat_ | hi, I am wondering how do I see the commit https://github.com/openstack/nova/commit/1cfcfa5aa8ff2437f289c9af0d1ded5047388c11 in openstack git? I mean its not an upstream change via gerrit etc so how did it land in there..? | 10:50 |
| *** yassine has quit IRC | 10:51 | |
| *** rk4n has joined #openstack-nova | 10:52 | |
| andrearosa | PaulMurray: My point about using Force is that you can never guarantee that the live migration will complete even if you pause the instance. | 10:52 |
| PaulMurray | danpb, I looked at your comment on the pause vm spec - what do you think about using a more generic name for live-migrate-pause-vm - thinking that it could | 10:53 |
| PaulMurray | options in future for different approaches? | 10:53 |
| johnthetubaguy | andrearosa: really? what goes wrong after the pause? | 10:53 |
| *** ihrachys has quit IRC | 10:54 | |
| johnthetubaguy | andrearosa: I am fine if the pause then makes the operation error out quickly, just wondering what could go into an infinite loop after the pause? | 10:54 |
| andrearosa | johnthetubaguy: I think paul-carlton1 has more use cases than me, I can think to some network issue or the target host goes down or other stuff like that | 10:55 |
| andrearosa | johnthetubaguy: not infinite loop but failed migration | 10:55 |
| johnthetubaguy | andrearosa: yeah, that stuff is fine, its the loop for ever bit that it needs to stop | 10:55 |
| johnthetubaguy | force operations can often fail, so I think its OK | 10:56 |
| andrearosa | johnthetubaguy: ok so when you think to force live miggration to finish you do not assume anything about the final result. It can fail or success but it reaches a final state | 10:56 |
| *** ihrachys has joined #openstack-nova | 10:57 | |
| andrearosa | s/success/succeed | 10:57 |
| *** e0ne has joined #openstack-nova | 10:57 | |
| johnthetubaguy | andrearosa: thats a good way of putting it, force a final state | 10:58 |
| PaulMurray | johnthetubaguy, andrearosa I think what we will need to decide is if there is any intent to make this a more generic command or if it will only ever do a pause | 10:58 |
| andrearosa | johnthetubaguy: force a final state makes sense | 10:59 |
| johnthetubaguy | PaulMurray: I am fine with the body of the API call getting a method argument eventually | 10:59 |
| johnthetubaguy | andrearosa: does live-migrate-force-end sound better? | 11:00 |
| *** takashin has quit IRC | 11:01 | |
| *** aysyd has joined #openstack-nova | 11:02 | |
| johnthetubaguy | PaulMurray: +1 your previous statement about the spec, it does seem ready apart from the name thing, and the details around that | 11:02 |
| andrearosa | johnthetubaguy: to me yes, it is clear what that means. | 11:02 |
| *** liusheng has quit IRC | 11:03 | |
| PaulMurray | johnthetubaguy, I'm also fine with setting the intent that this API would be extended if another alternative to pause comes up | 11:04 |
| *** shz has quit IRC | 11:05 | |
| *** markvoelker has joined #openstack-nova | 11:06 | |
| *** shz has joined #openstack-nova | 11:06 | |
| johnthetubaguy | andrearosa: just looking at this nova-manage add, I thought we already had a blueprint for that somewhere? | 11:06 |
| *** dims has joined #openstack-nova | 11:08 | |
| johnthetubaguy | andrearosa: thinking about this one: https://review.openstack.org/#/c/184537/24 | 11:08 |
| *** rfolco has joined #openstack-nova | 11:08 | |
| andrearosa | johnthetubaguy: not sure as it is supposed to be a bug fixing | 11:08 |
| andrearosa | johnthetubaguy: let me know if we need a blueprint | 11:08 |
| johnthetubaguy | andrearosa: I think we should add one, just to advertise it, folks have been wanting this for a long time, its good to shout about having added it | 11:09 |
| johnthetubaguy | andrearosa: if you can create a blueprint, give me the link and I can approve that for you | 11:09 |
| *** eliqiao has quit IRC | 11:10 | |
| andrearosa | johnthetubaguy: ok I will write a BP not now but hopefully later today. Thanks | 11:10 |
| johnthetubaguy | andrearosa: not sure its worth re-spinning the patch though, and loosing the +2 | 11:10 |
| *** daemontool has quit IRC | 11:11 | |
| johnthetubaguy | andrearosa: found it: https://blueprints.launchpad.net/nova/+spec/add-force-detach-to-nova | 11:11 |
| *** markvoelker has quit IRC | 11:11 | |
| *** daemontool has joined #openstack-nova | 11:11 | |
| *** eliqiao has joined #openstack-nova | 11:12 | |
| *** zhangjn has joined #openstack-nova | 11:12 | |
| *** zhangjn has quit IRC | 11:12 | |
| andrearosa | johnthetubaguy: oh good, I knew we had a spec (abandoned) not sure about the BP | 11:12 |
| *** zhangjn has joined #openstack-nova | 11:13 | |
| *** zhangjn has quit IRC | 11:14 | |
| *** zhangjn has joined #openstack-nova | 11:14 | |
| *** zhangjn has quit IRC | 11:15 | |
| *** salv-orl_ has joined #openstack-nova | 11:15 | |
| *** zhangjn has joined #openstack-nova | 11:15 | |
| *** zhangjn has quit IRC | 11:15 | |
| *** zhangjn has joined #openstack-nova | 11:16 | |
| *** zhangjn has quit IRC | 11:16 | |
| *** salv-orlando has quit IRC | 11:17 | |
| *** zhangjn has joined #openstack-nova | 11:17 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Replaced private field to public method in enginefacade https://review.openstack.org/246337 | 11:17 |
| tangchen__ | alex_xu: Hi, alex, when is the deadline of the doc review work ? | 11:18 |
| *** zhangjn has quit IRC | 11:18 | |
| *** zhangjn has joined #openstack-nova | 11:18 | |
| alex_xu | tangchen__: I guess there isn't deadline | 11:18 |
| johnthetubaguy | andrearosa: didn't we need to call cinder's force-detach in the end? | 11:18 |
| *** zhangjn has quit IRC | 11:19 | |
| tangchen__ | alex_xu: Oh, I see. Thx. :) | 11:19 |
| alex_xu | tangchen__: np | 11:19 |
| *** zhangjn has joined #openstack-nova | 11:20 | |
| *** zhangjn has quit IRC | 11:20 | |
| *** zhangjn has joined #openstack-nova | 11:21 | |
| Kevin_Zheng | Nova jenkins python27 and python34 failed all day? | 11:21 |
| tangchen__ | <tyagiprince>: Sorry, I didn't see your reply. Hope you have fixed your problem. | 11:21 |
| andrearosa | johnthetubaguy: nope, because of two main reason. 1 cinder force-detach has a bug and doens't do what it claims. I know that the cinder guys are wokring on a fix. 2. we have decided that nova needs to clean up its bits and don't bother about trying to fix cinder side. It's up to the operator to use/call the cinder tools to fix the situation in cinder. | 11:23 |
| *** rmart04 has joined #openstack-nova | 11:23 | |
| andrearosa | johnthetubaguy: not sure if I explained it very well | 11:23 |
| *** daemontool has quit IRC | 11:23 | |
| bauzas | Kevin_Zheng: https://bugs.launchpad.net/nova/+bug/1516805 | 11:23 |
| openstack | Launchpad bug 1516805 in OpenStack Compute (nova) liberty "nova unit tests failing with infinite recursion with oslo.versionedobjects 0.13.0" [Critical,In progress] - Assigned to Dan Smith (danms) | 11:23 |
| *** EinstCrazy has joined #openstack-nova | 11:23 | |
| *** romainh has quit IRC | 11:23 | |
| *** daemontool has joined #openstack-nova | 11:24 | |
| *** sudipto has quit IRC | 11:24 | |
| bauzas | PaulMurray: FYI https://review.openstack.org/#/c/245993/ | 11:24 |
| Kevin_Zheng | Ah, thanks | 11:24 |
| Kevin_Zheng | bauzas: Thanks | 11:24 |
| *** daemontool has quit IRC | 11:25 | |
| bauzas | Kevin_Zheng: the fix for master is in the gate pipeline, hoping it could be merged by the next hours | 11:26 |
| *** xnox has left #openstack-nova | 11:26 | |
| *** daemontool has joined #openstack-nova | 11:26 | |
| Kevin_Zheng | bauzas: cool | 11:26 |
| *** flaper87 has quit IRC | 11:27 | |
| *** flaper87 has joined #openstack-nova | 11:27 | |
| openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Cope with more Cinder backends https://review.openstack.org/244789 | 11:28 |
| PaulMurray | bauzas, thanks | 11:29 |
| *** atuvenie has joined #openstack-nova | 11:30 | |
| bauzas | PaulMurray: btw. any idea when you'll get quotes for room blocks in Bristol ? | 11:31 |
| *** salv-orl_ has quit IRC | 11:31 | |
| *** salv-orlando has joined #openstack-nova | 11:32 | |
| atuvenie | johnthetubaguy: Hey, I have a trivial bug fix to make create configdrive work with py3. It's currently stuck due to a -1, can you take a look and see what you think? https://review.openstack.org/#/c/237786/ | 11:33 |
| *** josecastroleon has quit IRC | 11:36 | |
| *** ihrachys has quit IRC | 11:37 | |
| openstackgerrit | ShaoHe Feng proposed openstack/nova: do not log twice with different level. https://review.openstack.org/246269 | 11:37 |
| *** jhesketh has quit IRC | 11:37 | |
| *** paul-carlton has quit IRC | 11:40 | |
| *** jhesketh has joined #openstack-nova | 11:40 | |
| *** pixelbeat has joined #openstack-nova | 11:40 | |
| johnthetubaguy | atuvenie: have you reached out to sdague on IRC? he is usually online quite soon | 11:43 |
| *** ihrachys has joined #openstack-nova | 11:43 | |
| *** shz has quit IRC | 11:43 | |
| *** rook has quit IRC | 11:43 | |
| *** achanda has quit IRC | 11:43 | |
| openstackgerrit | Mark McLoughlin proposed openstack/nova: servicegroup: remove the zookeeper driver https://review.openstack.org/246343 | 11:43 |
| *** shz has joined #openstack-nova | 11:43 | |
| alex_xu | bauzas: currently we only use reno add upgrade note? | 11:44 |
| bauzas | alex_xu: you mean ? | 11:44 |
| bauzas | alex_xu: for the moment, we're holding off reno relnotes because we're not gating them | 11:44 |
| alex_xu | bauzas: just want to ask whether should ask feature patch add feature note | 11:44 |
| johnthetubaguy | atuvenie: we also have a blueprint for all the python3 work, attaching it to the blueprint might get the attention of the folks who focus on python3 stuff: https://blueprints.launchpad.net/nova/+spec/nova-python3-mitaka | 11:45 |
| alex_xu | bauzas: ok, got it | 11:45 |
| bauzas | alex_xu: yup, you should | 11:45 |
| bauzas | alex_xu: eg. https://review.openstack.org/#/c/197912/ | 11:45 |
| claudiub | johnthetubaguy: ahem, I have like 7-8 python 3 related patches that haven't been reviewed. :D | 11:46 |
| alex_xu | bauzas: cool, thanks | 11:46 |
| johnthetubaguy | claudiub: agreed, we have 600 outstanding patches right now | 11:46 |
| PaulMurray | bauzas, Just asked - there was a correction we had to make, will try to get it out today. It will work out around £97 per night for the hotel near us | 11:46 |
| johnthetubaguy | claudiub: just giving hints for folks to reach your patches sooner | 11:46 |
| atuvenie | johnthetubaguy: Thanks :) I will try to reach Sean and will attach the patch to that BP | 11:47 |
| PaulMurray | approx - detail to come | 11:47 |
| alex_xu | bauzas: for such single patch feature https://review.openstack.org/#/c/209018/, I should ask add feature note, right? | 11:47 |
| johnthetubaguy | claudiub: FWIW, I saw a jenkins -1 at the base of one chain on py34 patches, so skipped that chain till thats fixed | 11:47 |
| bauzas | PaulMurray: I got a ? for the currency, I bet it's pounds? | 11:47 |
| PaulMurray | bauzas, really, ha. Yes pounds | 11:48 |
| bauzas | PaulMurray: cool | 11:48 |
| PaulMurray | which is not good for you Euro folks right now | 11:48 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Replaced private field in get_session/engine with public method https://review.openstack.org/246337 | 11:48 |
| bauzas | PaulMurray: well, UK will get Euro soon, right? :p | 11:48 |
| *** deepakcs has quit IRC | 11:48 | |
| *** smatzek has joined #openstack-nova | 11:48 | |
| *** vladikr has quit IRC | 11:49 | |
| bauzas | PaulMurray: thanks for sorting that out | 11:49 |
| PaulMurray | bauzas, will send out a web site link with everything you need | 11:50 |
| PaulMurray | very soon | 11:50 |
| bauzas | PaulMurray: well, tbh, I'm not that worried | 11:50 |
| bauzas | :) | 11:50 |
| *** ZZelle has quit IRC | 11:50 | |
| *** ZZelle has joined #openstack-nova | 11:51 | |
| *** markus_z has quit IRC | 11:51 | |
| *** ihrachys has quit IRC | 11:51 | |
| *** ihrachys has joined #openstack-nova | 11:52 | |
| * alex_xu reminders api meeting in 8 mins | 11:52 | |
| *** ihrachys has quit IRC | 11:52 | |
| *** markus_z has joined #openstack-nova | 11:53 | |
| *** jlanoux has quit IRC | 11:53 | |
| *** tyagiprince has joined #openstack-nova | 11:53 | |
| *** ndipanov has joined #openstack-nova | 11:54 | |
| *** paul-carlton has joined #openstack-nova | 11:54 | |
| sdague | atuvenie: looking, ok, explanation seems fine | 11:55 |
| tyagiprince | Hey nova people.. I need some help.. I created a 2 node setup and when I boot an instance, it goes into the error state.. and when i try to delete the same, it gives this error "an error occurred while refreshing the network cache." | 11:55 |
| tyagiprince | This error comes when i boot an instance.. | 11:55 |
| tyagiprince | ERROR nova.compute.manager [req-9669c1e0-7afd-4810-988e-83f505ece99d f24656405a8648399c079e26ed0b81e6 7d5099e4dc2e4269b4632adfcc94c132 - - -] [instance: f559751c-dcd4-4ac3-a4ba-69f99a09ad5f] Failed to deallocate network for instance. | 11:55 |
| atuvenie | sdague: Thanks! | 11:55 |
| *** rpodolyaka1 has joined #openstack-nova | 11:57 | |
| *** jwcroppe has quit IRC | 11:57 | |
| *** ParsectiX has joined #openstack-nova | 11:57 | |
| ParsectiX | Hi guys just a small question. Did openstack deletes old instance info from MariaDB ? | 11:58 |
| *** rpodolyaka1 has quit IRC | 11:59 | |
| alex_xu | nova api meeting is running | 12:00 |
| *** venkat_p has joined #openstack-nova | 12:02 | |
| *** Sree_ has joined #openstack-nova | 12:03 | |
| *** lpetrut1 has joined #openstack-nova | 12:03 | |
| *** Sree_ is now known as Guest24040 | 12:03 | |
| *** romainh has joined #openstack-nova | 12:04 | |
| *** lpetrut has quit IRC | 12:04 | |
| *** lpetrut1 is now known as lpetrut | 12:04 | |
| *** ndipanov has quit IRC | 12:05 | |
| *** ParsectiX has quit IRC | 12:06 | |
| *** achanda has joined #openstack-nova | 12:06 | |
| *** stevemar_ has joined #openstack-nova | 12:06 | |
| *** ParsectiX has joined #openstack-nova | 12:06 | |
| *** josecastroleon has joined #openstack-nova | 12:07 | |
| *** ParsectiX has quit IRC | 12:08 | |
| *** rpodolyaka1 has joined #openstack-nova | 12:09 | |
| *** achanda has quit IRC | 12:09 | |
| mdbooth | danpb: I'm thinking about virStorageVolUpload in the context of a potential performance impact on https://review.openstack.org/#/c/232053/9/specs/mitaka/approved/migrate-libvirt-volumes.rst | 12:10 |
| *** rushiagr is now known as rushiagr_away | 12:10 | |
| mdbooth | Do you happen to know anything about its performance? | 12:11 |
| *** venkat_p has quit IRC | 12:11 | |
| mdbooth | A major concern is the lack of efficient sparse handling, which is usually a huge win when copying disk images. | 12:12 |
| *** links has joined #openstack-nova | 12:12 | |
| mdbooth | Raw disk images, anyway. qcow2 is probably less affected. | 12:12 |
| *** links has quit IRC | 12:12 | |
| mdbooth | I notice that you call it and it returns a stream pointer. I believe the transfer channel is multiplexed on the libvirt connection, right? | 12:13 |
| *** haomaiwa_ has quit IRC | 12:14 | |
| *** otter768 has joined #openstack-nova | 12:15 | |
| mdbooth | Is setting up that multiplex an expensive operation? It occurs to me that if it was cheap you could simply do sparse detection at source and only transfer non-sparse segments. However, it it's expensive that would have a very high overhead, as this would result in massively more calls. | 12:15 |
| *** jyuso has quit IRC | 12:15 | |
| *** rubasov1 has quit IRC | 12:17 | |
| *** rubasov has joined #openstack-nova | 12:19 | |
| *** otter768 has quit IRC | 12:19 | |
| *** ndipanov has joined #openstack-nova | 12:21 | |
| *** tojuvone has joined #openstack-nova | 12:21 | |
| mdbooth | danpb: Thinking about it, if libvirt exposed a raw data channel, you could just attach rsync at both ends... | 12:23 |
| *** shaohe_feng has joined #openstack-nova | 12:24 | |
| shaohe_feng | johnthetubaguy: hi | 12:25 |
| johnthetubaguy | shaohe_feng: hi, how can I help? | 12:26 |
| shaohe_feng | just want to check the meeting time | 12:26 |
| danpb | mdbooth: as the name suggests, the API is designing for streaming, so it doesn't do sparse detection | 12:26 |
| shaohe_feng | live migration | 12:26 |
| *** fawadkhaliq has quit IRC | 12:26 | |
| shaohe_feng | johnthetubaguy: ^ | 12:26 |
| *** fawadkhaliq has joined #openstack-nova | 12:27 | |
| mdbooth | danpb: Right, just thought I'd check. There's a flags argument, though. Don't suppose anybody's proposed anything which might be relevant in this area? | 12:27 |
| *** chhavi has joined #openstack-nova | 12:28 | |
| johnthetubaguy | shaohe_feng: not sure about the time, I would have to check the ML for details, but PaulMurray will know | 12:28 |
| danpb | mdbooth: at a protocol level we're just asynchronously sending the data - there's no offsets present - its assumed to be a continuous stream of data | 12:29 |
| *** klkumar has quit IRC | 12:30 | |
| danpb | mdbooth: we could theoretically extend the protocol to add the notion of a "data hole" packet which just provided a length, but no data | 12:30 |
| mdbooth | danpb: So, the api seems to specify an offset that the stream refers to. If it's cheap to create a stream we could simply create lots of streams. | 12:30 |
| danpb | so if the source detected sparseness it would put a data hole into the stream | 12:30 |
| mdbooth | That would also be good. | 12:30 |
| mdbooth | Much better, in fact. | 12:30 |
| shaohe_feng | johnthetubaguy: thank you. let me check it with PaulMurray | 12:30 |
| shaohe_feng | PaulMurray: ping | 12:31 |
| danpb | not sure off hand how hard that would be to wire up but its certainly something that could be explored | 12:31 |
| *** fawadkhaliq has quit IRC | 12:31 | |
| *** jerrygb has joined #openstack-nova | 12:31 | |
| *** kylek3h has joined #openstack-nova | 12:31 | |
| *** kylek3h has quit IRC | 12:31 | |
| *** pkoniszewski has joined #openstack-nova | 12:31 | |
| *** kylek3h has joined #openstack-nova | 12:31 | |
| johnthetubaguy | shaohe_feng: ah, there is a new message on the ML: http://lists.openstack.org/pipermail/openstack-dev/2015-November/079631.html | 12:32 |
| danpb | mdbooth: NB I get the feeling that most clouds without shared storage are likely to be using qcow2, rather than raw | 12:32 |
| danpb | so while its desirable to improve performance, I don't think its the end of the world for openstack if we don't detect sparseness | 12:32 |
| mdbooth | danpb: Yeah, that's why I'm not totally against the idea. | 12:32 |
| danpb | oh wait no, i'm stupid | 12:32 |
| mdbooth | I think we should be explicit about that, though. There are going to be pathological cases. | 12:33 |
| danpb | we fallocate the qcow2 file to the full size | 12:33 |
| mdbooth | Ah... | 12:33 |
| mdbooth | Ok, it's all bad :) | 12:33 |
| danpb | sigh | 12:33 |
| kashyap | danpb: Yeah, I think there's a patch to change that to use 'fallocate' option of qemu-img? | 12:33 |
| * mdbooth would nak it on that basis :( | 12:33 | |
| mdbooth | If I had a nak, that is. | 12:33 |
| danpb | kashyap: that's not helpful | 12:33 |
| danpb | mdbooth: on the flip side there's many clouds which 100% refuse to deploy rsync+ssh | 12:34 |
| *** gcb has quit IRC | 12:34 | |
| mdbooth | danpb: Yup. PaulMurray was saying. | 12:34 |
| kashyap | Any reason why they refuse to use rsync+ssh? | 12:34 |
| danpb | mdbooth: so with the choice between no feature at all, or the slow method in current libvirt, they might choose the slow method | 12:34 |
| danpb | kashyap: remote root access is a security risk | 12:34 |
| shaohe_feng | johnthetubaguy: got it. the time has changed. It is not the poll time http://doodle.com/poll/rbta6n3qsrzcqfbn. | 12:35 |
| mdbooth | danpb: I don't think we can force it, though. It would be a severe performance regression for anybody who was previously using rsync. | 12:35 |
| kashyap | Ah, right. | 12:35 |
| danpb | mdbooth: yeah, looks like we'd need to make choice of transports configurable | 12:35 |
| danpb | so people can opt in to continuing to use rsync if desired | 12:36 |
| *** Guest89040 is now known as jroll | 12:36 | |
| mdbooth | :( | 12:36 |
| danpb | arguably we need to make it configurable regardless, as we need a deprecation period of at least one cycle for rsync method before we remove it | 12:36 |
| danpb | we can't just force people to switch methods with no warning on upgrade | 12:37 |
| mdbooth | The argument in the spec is that the new method is functionally equivalent, but simpler to use, which I agree with. | 12:37 |
| *** markvoelker has joined #openstack-nova | 12:37 | |
| mdbooth | On that basis alone I don't think there's a reason to keep it. It's just the performance impact. | 12:37 |
| kashyap | mdbooth: You might find some commentary here relevant (comments on the left hand side) - https://review.openstack.org/#/c/171098/1/nova/virt/libvirt/driver.py,cm | 12:39 |
| mdbooth | kashyap: Reading, thanks. | 12:39 |
| kashyap | You might know most of it already. | 12:39 |
| *** annegentle has joined #openstack-nova | 12:39 | |
| *** ihrachys has joined #openstack-nova | 12:40 | |
| kashyap | shaohe_feng: That meeting is at 14:00 UTC. | 12:41 |
| * mdbooth finds it odd that we automatically use every new version of every python package as soon as it hits pypi, but we continue using libvirt that's years old. | 12:42 | |
| *** markvoelker has quit IRC | 12:42 | |
| shaohe_feng | kashyap: got it. thantk you. | 12:42 |
| *** alejandrito has joined #openstack-nova | 12:42 | |
| kashyap | mdbooth: Yeah, earlier in the day there was a topic about running newest libvirt on Gate for features that require new enough libvirt | 12:42 |
| *** jgjhgsonchn88523 has joined #openstack-nova | 12:44 | |
| *** annegentle has quit IRC | 12:44 | |
| *** klkumar has joined #openstack-nova | 12:45 | |
| *** shaohe_feng1 has joined #openstack-nova | 12:45 | |
| *** ihrachys has quit IRC | 12:46 | |
| *** shaohe_feng has quit IRC | 12:46 | |
| *** annegentle has joined #openstack-nova | 12:47 | |
| *** pkoniszewski is now known as pkoniszewski_ | 12:50 | |
| *** lucasagomes is now known as lucas-hungry | 12:52 | |
| *** baoli has joined #openstack-nova | 12:52 | |
| *** korzen has joined #openstack-nova | 12:55 | |
| *** thorst has joined #openstack-nova | 12:55 | |
| *** achanda has joined #openstack-nova | 12:56 | |
| *** ihrachys has joined #openstack-nova | 12:57 | |
| *** ihrachys has quit IRC | 12:58 | |
| *** exploreshaifali has quit IRC | 12:59 | |
| *** tyagiprince has quit IRC | 12:59 | |
| *** tyagiprince has joined #openstack-nova | 12:59 | |
| jichen | alex_xu: maybe a quick talk about https://bugs.launchpad.net/nova/+bug/1516158? | 12:59 |
| openstack | Launchpad bug 1516158 in OpenStack Compute (nova) "os-instance_usage_audit_log is used instead of os-instance-usage-audit-log" [Undecided,New] - Assigned to jichenjc (jichenjc) | 12:59 |
| sdague | jichen: can you explain this more - https://bugs.launchpad.net/nova/+bug/1516158 - like what's the setup condition to get there on a devstack and what's going on? | 12:59 |
| sdague | because I'm not really understanding the expected vs. actual behavior | 13:00 |
| *** jgjhgsonchn88523 has quit IRC | 13:00 | |
| *** venkat_p has joined #openstack-nova | 13:01 | |
| jichen | sdague: the end point is os-instance_usage_audiog, we always use x-x-x-x in the resource type | 13:01 |
| *** paul-carlton has quit IRC | 13:01 | |
| jichen | sdague: I am curious why we did so for this one only | 13:01 |
| sdague | I do not know, but that's definitely the code from what I can see | 13:02 |
| *** doug-fish has joined #openstack-nova | 13:02 | |
| sdague | https://github.com/openstack/nova/blob/e6100f593413de77cc825cb9aeb5b86a739874e9/nova/api/openstack/compute/instance_usage_audit_log.py#L136-L137 | 13:02 |
| jichen | sdague: yeah, this is from http://developer.openstack.org/api-ref-compute-v2.1.html#listusageaudits | 13:03 |
| alex_xu | jichen: looks like we can't change that | 13:04 |
| alex_xu | if just for consistent | 13:04 |
| sdague | yeh, I would fix the documentation | 13:04 |
| sdague | this is what it is | 13:05 |
| jichen | ok, I can fix the doc but this is really weird compare to other resource we have | 13:05 |
| *** achanda has quit IRC | 13:05 | |
| sdague | it is, I agree, but it's what we have | 13:05 |
| jichen | sdague: ok, I see, I will close the bug | 13:06 |
| markus_z | danpb: Hi, just to be sure, the libvirt subteam meeting is no more, right? It's still listed (with you as host) here: eavesdrop.openstack.org/#Libvirt_Meeting | 13:06 |
| markus_z | danpb: Apologies If I already asked that some time ago, I'm not sure anymore | 13:07 |
| sdague | jichen: well, there is still a doc change needed | 13:07 |
| sdague | so don't close the bug until the doc patch is landed | 13:08 |
| jichen | ok, I thought we tracked this in doc API change bug , and you are right, I will keep it ,thx | 13:08 |
| lbelivea_ | ndipanov: No SR-IOV meeting today ? | 13:09 |
| kashyap | markus_z: I think it was disbaneded because not enough people were showing up consistently I guess. But a lot of libvirt related topics gets discussed on this channel anyway. | 13:11 |
| *** nkrinner has quit IRC | 13:12 | |
| *** gcb has joined #openstack-nova | 13:12 | |
| *** dave-mccowan has joined #openstack-nova | 13:12 | |
| markus_z | kashyap: I see. I wanted to ask a question there without to make some noise here. | 13:14 |
| sdague | markus_z: here is fine | 13:15 |
| kashyap | markus_z: It's absolutely on-topic here, not noise at all. Type away all you want :-) | 13:15 |
| *** paul-carlton has joined #openstack-nova | 13:15 | |
| *** pradk has joined #openstack-nova | 13:15 | |
| *** pradk has quit IRC | 13:15 | |
| markus_z | OK, cool :) | 13:15 |
| *** nkrinner has joined #openstack-nova | 13:15 | |
| markus_z | I wanted to ask if someone (or more people) like to be a contact point when it comes to untriaged libvirt bugs. | 13:16 |
| *** ctrath has joined #openstack-nova | 13:16 | |
| markus_z | I guess the entry here is not up to date anymore: https://wiki.openstack.org/wiki/Nova/BugTriage#Step_2:_Triage_Tagged_Bugs | 13:16 |
| *** ctrath has quit IRC | 13:16 | |
| *** jlanoux has joined #openstack-nova | 13:17 | |
| *** rushil has joined #openstack-nova | 13:17 | |
| *** signed8bit has joined #openstack-nova | 13:17 | |
| *** lbelivea_ is now known as lbelivea | 13:17 | |
| markus_z | The ML post 2 weeks ago didn't get much feedback: http://lists.openstack.org/pipermail/openstack-dev/2015-November/078252.html | 13:17 |
| *** ff has joined #openstack-nova | 13:18 | |
| *** ff has quit IRC | 13:18 | |
| *** rook has joined #openstack-nova | 13:18 | |
| *** jlanoux_ has joined #openstack-nova | 13:19 | |
| openstackgerrit | jichenjc proposed openstack/nova: Change Invalid exception to a specified exception https://review.openstack.org/232506 | 13:20 |
| *** rushil_ has joined #openstack-nova | 13:20 | |
| markus_z | That's all I wanted to say, thanks for listening :D | 13:20 |
| kashyap | markus_z: You can add me there. | 13:20 |
| kashyap | markus_z: I'm supposed to be more diligent with them, so feel free to shame me here if I'm not being so in future | 13:21 |
| *** jlanoux has quit IRC | 13:21 | |
| *** yassine has joined #openstack-nova | 13:22 | |
| markus_z | kashyap: Yeah, very cool, thanks! | 13:22 |
| kashyap | markus_z: I did read that email (it was not ignored). | 13:22 |
| kashyap | markus_z: I'll update the entry | 13:22 |
| sdague | jichen: I uploaded a doc fix - https://review.openstack.org/#/c/246383/ - let me know what you think | 13:22 |
| jichen | sdague: ok, I will take a look at that, thanks~ | 13:23 |
| *** rushil has quit IRC | 13:23 | |
| sdague | I think that's the right doc fix, though wadl is so hard to know | 13:23 |
| *** ankit_ag has quit IRC | 13:26 | |
| *** gcb has quit IRC | 13:27 | |
| markus_z | kashyap: Thanks a lot! I think you know that this doesn't mean you have to solve it by your own. | 13:27 |
| kashyap | markus_z: Of course. | 13:27 |
| kashyap | markus_z: Some of the bugs are only reproducible in OpenStack Gate env. | 13:27 |
| *** markvoelker has joined #openstack-nova | 13:27 | |
| kashyap | markus_z: Some have very poor info, or special storage hardware that I don't have access to, etc. | 13:28 |
| *** raildo-afk is now known as raildo | 13:28 | |
| kashyap | markus_z: Realize the goal is to actively triage, without letting things rot. . . | 13:28 |
| markus_z | kashyap: Yeah, I think I'm still to hesitant to close bugs when they don't provide enough information. | 13:28 |
| markus_z | *too | 13:29 |
| kashyap | markus_z: Well, if the reporter doesn't provide the details in a month or two, can be closed as INSUFFICIENT_INFO | 13:29 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: WIP: enginefacade: 'security_group' https://review.openstack.org/246302 | 13:30 |
| sdague | hey everyone, please stop approving nova patches until https://review.openstack.org/#/c/246013/ lands, they will all fail | 13:30 |
| PaulMurray | shaohe_feng1, meeting https://wiki.openstack.org/wiki/Meetings/NovaLiveMigration | 13:30 |
| PaulMurray | was at lunch | 13:30 |
| kashyap | markus_z: If you haven't already seen, see some notes from sdague here: http://lists.openstack.org/pipermail/openstack-dev/2014-September/046517.html | 13:30 |
| openstackgerrit | Sean Dague proposed openstack/nova: Replace "all_mappings" variable by "block_device_mappings" https://review.openstack.org/208511 | 13:30 |
| kashyap | FWIW, I agree with a lot of things there. | 13:30 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Compute: Adds driver disk_gb instance overhead estimation https://review.openstack.org/234283 | 13:31 |
| openstackgerrit | Sean Dague proposed openstack/nova: Use flavor instead of flavour https://review.openstack.org/218644 | 13:31 |
| openstackgerrit | Sean Dague proposed openstack/nova: VMware: fix docstring for cluster management https://review.openstack.org/218641 | 13:31 |
| openstackgerrit | Sean Dague proposed openstack/nova: Refactor get_filtered_hosts in filter host manager https://review.openstack.org/211307 | 13:31 |
| openstackgerrit | Sean Dague proposed openstack/nova: Sync L3Driver, NullL3 interface with LinuxNetL3 https://review.openstack.org/189001 | 13:31 |
| markus_z | kashyap: Yepp, thanks, I've read it and it makes perfectly sense for me. My fear is to close a valid critical bug because I didn't know better. | 13:31 |
| sdague | yeh, I guess mikal didn't look at the current state of the world before approving a bunch of things | 13:31 |
| *** Guest24040 has quit IRC | 13:32 | |
| kashyap | markus_z: Yeah, I hear you. If it's critical enough, I'm sure enough people will yell :-) | 13:32 |
| *** burgerk has joined #openstack-nova | 13:32 | |
| *** Sree has joined #openstack-nova | 13:32 | |
| openstackgerrit | jichenjc proposed openstack/nova-specs: Add display-flavor-id-in-migration.rst https://review.openstack.org/219431 | 13:33 |
| *** rubasov has quit IRC | 13:33 | |
| sdague | fyi, dansmith, next time there is a break like this it should also get an email to the list to get people to stop approving code. The fix bounced and ruined everyone else's merge overnight | 13:33 |
| *** rubasov has joined #openstack-nova | 13:33 | |
| markus_z | kashyap: A previous manager I had often said "just imaging Sisyphus was a happy person" :D | 13:33 |
| *** alexschm has joined #openstack-nova | 13:34 | |
| *** amotoki has joined #openstack-nova | 13:34 | |
| markus_z | kashyap: Again, I'm glad you offered your help here. I'm heading back to some code stuff. | 13:34 |
| *** jichen has quit IRC | 13:34 | |
| *** rpodolyaka1 has quit IRC | 13:35 | |
| kashyap | markus_z: :-) | 13:35 |
| danpb | markus_z: correct, i just don't have the time or organization skills to rnu it | 13:36 |
| *** artom__ has quit IRC | 13:36 | |
| danpb | best to just ask questons here, or on the mailing list | 13:36 |
| *** Sree has quit IRC | 13:37 | |
| *** gcb has joined #openstack-nova | 13:40 | |
| *** cdent has joined #openstack-nova | 13:41 | |
| *** cdent has left #openstack-nova | 13:41 | |
| *** pradk has joined #openstack-nova | 13:43 | |
| *** oomichi has quit IRC | 13:43 | |
| *** pkoniszewski has joined #openstack-nova | 13:45 | |
| *** abhishekk has quit IRC | 13:45 | |
| *** markus_z_bnc has joined #openstack-nova | 13:45 | |
| *** haomaiwang has joined #openstack-nova | 13:47 | |
| sdague | markus_z: hey, a bug triage question. Any chance we could get per tag stats reported into graphana? It would be super nice to have some burndown stats on bugs, especially per tag. | 13:48 |
| sdague | markus_z: also, I triaged those 4 api bugs | 13:48 |
| *** paul-carlton has left #openstack-nova | 13:49 | |
| *** paul-carlton has joined #openstack-nova | 13:49 | |
| *** yassine has quit IRC | 13:49 | |
| *** diegows has joined #openstack-nova | 13:49 | |
| markus_z | sdague: I haven't used graphana before. I have a poc at home which uses j3ds. That's a stretch goal I had but didn't announce yet. | 13:49 |
| bauzas | do we have grafana already? | 13:50 |
| markus_z | sdague: And I agree that a bug dashboard would be helpful IMO. | 13:50 |
| bauzas | ooooh http://grafana.openstack.org/ | 13:50 |
| markus_z | bauzas: ohhhh, shiny, I'll take a look :) | 13:51 |
| markus_z | sdague: Thanks for the triaging of the api bugs! | 13:51 |
| *** eharney has quit IRC | 13:52 | |
| bauzas | sdague: seems that the LP OAuth is not for grafana ? | 13:52 |
| *** haomaiwang has quit IRC | 13:52 | |
| markus_z | god I loooove data visualization | 13:52 |
| *** atuvenie_ has joined #openstack-nova | 13:53 | |
| *** samueldmq has joined #openstack-nova | 13:53 | |
| *** sacharya has joined #openstack-nova | 13:55 | |
| *** rpodolyaka1 has joined #openstack-nova | 13:56 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 13:56 | |
| *** atuvenie has quit IRC | 13:56 | |
| *** claudiub has quit IRC | 13:56 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'bw_usage', 'vol_usage' and 's3_image' https://review.openstack.org/246397 | 13:57 |
| *** jgjhgsonchn88523 has joined #openstack-nova | 13:57 | |
| *** claudiub has joined #openstack-nova | 13:57 | |
| *** markus_z is now known as markus_z_meeting | 13:58 | |
| sdague | markus_z_bnc: well, I was chatting with jeblair the other day, and that basically just consumes anything in our graphite | 13:58 |
| *** moshele has quit IRC | 13:58 | |
| sdague | so if there was a periodic job that fed in graphite counters it could be pulled back in there | 13:59 |
| *** mdrabe has joined #openstack-nova | 13:59 | |
| *** ijuwang has joined #openstack-nova | 13:59 | |
| sdague | so chatting with some infra folks about what's needed to do that, it could be a replacement for http://status.openstack.org//bugday/ | 13:59 |
| *** sacharya has quit IRC | 13:59 | |
| PaulMurray | Nova Live Migration meeting on #openstack-meeting-3 | 14:00 |
| *** jlanoux_ has quit IRC | 14:00 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 14:00 | |
| *** obondarev_ is now known as obondarev | 14:01 | |
| *** klkumar has quit IRC | 14:01 | |
| *** ekarlso has joined #openstack-nova | 14:01 | |
| *** jlanoux has joined #openstack-nova | 14:01 | |
| ndipanov | lbelivea, I think we said every second week? | 14:02 |
| *** lucas-hungry is now known as lucasagomes | 14:03 | |
| kashyap | mdbooth: ^ (from Paul) Guess you wanted to join it, you asked here the other day? | 14:03 |
| kashyap | Ah, you waved there, disregard me. | 14:03 |
| mdbooth | :P | 14:03 |
| *** rubasov has quit IRC | 14:04 | |
| *** vladikr has joined #openstack-nova | 14:04 | |
| *** cdent has joined #openstack-nova | 14:04 | |
| *** _gryf has quit IRC | 14:04 | |
| *** claudiub has quit IRC | 14:04 | |
| *** _gryf has joined #openstack-nova | 14:05 | |
| *** _gryf has quit IRC | 14:05 | |
| *** claudiub has joined #openstack-nova | 14:05 | |
| *** rubasov has joined #openstack-nova | 14:05 | |
| *** _gryf has joined #openstack-nova | 14:06 | |
| *** rushil_ has quit IRC | 14:07 | |
| *** READ10 has joined #openstack-nova | 14:07 | |
| *** rlrossit has joined #openstack-nova | 14:10 | |
| *** venkat_p has quit IRC | 14:13 | |
| lbelivea | ndipanov: Ok my bad I taught it was every week | 14:13 |
| *** rushiagr_away is now known as rushiagr | 14:14 | |
| *** burgerk has quit IRC | 14:15 | |
| *** otter768 has joined #openstack-nova | 14:15 | |
| *** kfarr has quit IRC | 14:16 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add DiskFormat as Enum in fields https://review.openstack.org/229854 | 14:16 |
| openstackgerrit | jichenjc proposed openstack/nova: Make ImageMeta:disk_format use DiskFormatField https://review.openstack.org/231460 | 14:16 |
| *** ctrath has joined #openstack-nova | 14:16 | |
| *** cdent has left #openstack-nova | 14:17 | |
| lbelivea | ndipanov: What is your plan for sriov-physical-function-passthrough ? We might be interested to participate in the design/implementation. | 14:17 |
| *** signed8bit is now known as signed8bit_ZZZzz | 14:19 | |
| *** moshele has joined #openstack-nova | 14:19 | |
| *** Marga_ has joined #openstack-nova | 14:19 | |
| *** otter768 has quit IRC | 14:20 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 14:21 | |
| ajo | obondarev , rdopiera | 14:23 |
| ajo | on live migration VM resume / RARP supression | 14:23 |
| ajo | what's the nova status? | 14:23 |
| ajo | obondarev , I think you told me there were patches already? | 14:24 |
| ndipanov | lbelivea, that is a good question | 14:24 |
| *** vlaza has joined #openstack-nova | 14:24 | |
| *** pratikmallya has joined #openstack-nova | 14:25 | |
| *** venkat_p has joined #openstack-nova | 14:25 | |
| dansmith | sdague: okay | 14:25 |
| *** tyagiprince has quit IRC | 14:26 | |
| *** yassine has joined #openstack-nova | 14:28 | |
| ndipanov | so my current plan is to just get the Nova internal bits ready for the fact that pfs can be assigned | 14:29 |
| ndipanov | lbelivea, ^ | 14:29 |
| ndipanov | there is some work to be done on top of that, and on the neutron side as well | 14:29 |
| *** liverpooler has quit IRC | 14:29 | |
| *** smatzek has quit IRC | 14:29 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Replaced deprecated timeutils methods https://review.openstack.org/241179 | 14:29 |
| *** klkumar has joined #openstack-nova | 14:30 | |
| ndipanov | lbelivea, vladikr, was looking into that | 14:31 |
| *** mriedem has joined #openstack-nova | 14:32 | |
| moshele | ndipanov: can you review https://review.openstack.org/#/c/227160/ ? | 14:32 |
| mriedem | https://review.openstack.org/#/c/246013/ is still in the gate? | 14:33 |
| *** neiljerram_bb has joined #openstack-nova | 14:33 | |
| *** dustins has joined #openstack-nova | 14:33 | |
| *** yassine has quit IRC | 14:34 | |
| ndipanov | moshele, hmmm I thought that sriov_numfs was only an ixgbe thing... | 14:34 |
| *** yassine has joined #openstack-nova | 14:34 | |
| *** ttx has quit IRC | 14:36 | |
| *** breitz has quit IRC | 14:36 | |
| *** breitz has joined #openstack-nova | 14:36 | |
| moshele | ndipanov: I think it is a standard we have it on the mellanox driver and also baoli +1 it so I guess it good for cisco | 14:37 |
| ndipanov | moshele, fair enough... | 14:38 |
| *** haomaiwang has joined #openstack-nova | 14:39 | |
| *** nikhil_k is now known as nikhil | 14:39 | |
| *** mrkz has joined #openstack-nova | 14:40 | |
| *** ekarlso has quit IRC | 14:40 | |
| *** ttx has joined #openstack-nova | 14:40 | |
| *** doug-fish has quit IRC | 14:42 | |
| *** eharney has joined #openstack-nova | 14:43 | |
| *** artom has joined #openstack-nova | 14:44 | |
| *** salv-orl_ has joined #openstack-nova | 14:45 | |
| lbelivea | ndipanov: Let's talk about it next SR-IOV meeting. I'll put together some more comments about the BP by then. | 14:46 |
| *** burgerk has joined #openstack-nova | 14:47 | |
| *** fawadkhaliq has joined #openstack-nova | 14:48 | |
| ndipanov | lbelivea, ok but what did you have in mind when it comes to that particular BP? | 14:48 |
| *** salv-orlando has quit IRC | 14:48 | |
| ndipanov | do you see some issues with the proposed solution, or are you looking to add more enhancements? | 14:49 |
| * ndipanov maybe we do need to have a meeting every week in the beginning? | 14:49 | |
| lbelivea | ndipanov: I had a chat with our architect and we might have some customers that are interested in something similar. I need to gather input first. | 14:49 |
| ndipanov | lbelivea, ah ok awesome! more use cases sound excellent! | 14:50 |
| *** burt has joined #openstack-nova | 14:50 | |
| *** jlanoux has quit IRC | 14:50 | |
| lbelivea | ndipanov: I would not disagree with weekly meetings. | 14:50 |
| cfriesen__ | hopefully a quick question. Do we expect to be allowed to delete a flavor at any point, regardless of whether it's in use by an instance? | 14:51 |
| *** jlanoux has joined #openstack-nova | 14:52 | |
| cfriesen__ | I'm looking at nova.conductor.manager.ComputeTaskManager.migrate_server() and it does a call to objects.Flavor.get_by_id(). If the flavor is deleted, I think that could cause problems. | 14:52 |
| *** Marga_ has quit IRC | 14:52 | |
| *** hamzy has quit IRC | 14:53 | |
| lbelivea | ndipanov: Side question - do you know the historic reason why nova networking information wrt to ports are stored in neutron and not in nova (like PCI address, etc.) ? You mention something about nova interface with neutron ... | 14:54 |
| *** doug-fish has joined #openstack-nova | 14:54 | |
| *** mriedem has quit IRC | 14:55 | |
| *** rubasov has quit IRC | 14:55 | |
| ndipanov | lbelivea, so wrt weekly meetings - let's leave it like this for this week, and if we see next week that we are hitting the hour with plenty of stuff to talk about - we'll increase the freq | 14:56 |
| *** rubasov has joined #openstack-nova | 14:57 | |
| *** yonglihe has joined #openstack-nova | 14:57 | |
| *** doug-fish has quit IRC | 14:57 | |
| *** sneti has joined #openstack-nova | 14:58 | |
| *** achanda has joined #openstack-nova | 14:58 | |
| *** jwcroppe has joined #openstack-nova | 14:58 | |
| *** doug-fish has joined #openstack-nova | 14:59 | |
| *** zhangjn has quit IRC | 14:59 | |
| *** doug-fish has quit IRC | 14:59 | |
| *** smatzek has joined #openstack-nova | 14:59 | |
| *** doug-fish has joined #openstack-nova | 15:00 | |
| *** ctrath has quit IRC | 15:00 | |
| *** doug-fish has quit IRC | 15:01 | |
| *** jwcroppe has quit IRC | 15:01 | |
| *** jwcroppe has joined #openstack-nova | 15:02 | |
| *** jwcroppe has quit IRC | 15:02 | |
| moshele | ndipanov: just to understand the process now that we have focus of SR-IOV in nova, should we had our bp/commits in https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking ? | 15:02 |
| ndipanov | moshele, yes | 15:02 |
| ndipanov | moshele, the idea is though | 15:02 |
| ndipanov | that you put there only stuff that is "ready" | 15:02 |
| moshele | ndipanov: also who is include in the sub team? can I add myself ? | 15:02 |
| ndipanov | to be reviewed by cores | 15:03 |
| ndipanov | so don't add the patches there as soon as you post them | 15:03 |
| ndipanov | but let's get some team folks to review them first | 15:03 |
| ndipanov | moshele, yes of course | 15:03 |
| ndipanov | and lbelivea | 15:03 |
| *** mriedem has joined #openstack-nova | 15:03 | |
| ndipanov | and I assume there will be more interested folks that just missed the first meeting | 15:04 |
| moshele | ndipanov: ok I will update the etherpad | 15:04 |
| lbelivea | same here | 15:04 |
| *** venkat_p has quit IRC | 15:04 | |
| *** venkat_p has joined #openstack-nova | 15:05 | |
| *** tdurakov has joined #openstack-nova | 15:06 | |
| PaulMurray | tdurakov, go ahead | 15:06 |
| *** ccarmack has joined #openstack-nova | 15:07 | |
| tdurakov | update for live-migration job: initial part is merged to project-config and it's possible to schedule it with check experimental. There is currently bug in devstack-gate, that i'm going to submit in 15 minutes, after that everything should be ok. next step would be shared storage | 15:07 |
| mriedem | performance team meeting in #openstack-performance if anyone is interested, | 15:07 |
| mriedem | there is a topic on nova-conductor in there | 15:07 |
| mriedem | alaski: dansmith: ^ | 15:08 |
| tdurakov | i'd prefer to address nfs first, but it's up to you to choose one | 15:08 |
| tdurakov | PaulMurray, ^ | 15:08 |
| *** ctrath has joined #openstack-nova | 15:08 | |
| tdurakov | pkoniszewski^ | 15:08 |
| pkoniszewski | well, nfs is a tricky one, sounds good to me | 15:09 |
| pkoniszewski | great to hear that we are close | 15:09 |
| bauzas | -ETOOMUCHMEETINGS | 15:09 |
| pkoniszewski | could you add to me review when you submit a fix? | 15:09 |
| PaulMurray | tdurakov, to be honest, you're doing the work so you get to pick the order | 15:10 |
| pkoniszewski | +1 | 15:10 |
| *** amotoki_ has joined #openstack-nova | 15:10 | |
| tdurakov | pkoniszewski sure, it's easy one, just string reordering | 15:10 |
| tdurakov | PaulMurray, ok, then it gonna be nfs first:) | 15:10 |
| *** annegentle has quit IRC | 15:11 | |
| PaulMurray | tdurakov, glad to make you happy :) | 15:11 |
| *** blaisebool has joined #openstack-nova | 15:11 | |
| *** amotoki has quit IRC | 15:11 | |
| *** tongli has joined #openstack-nova | 15:11 | |
| bauzas | alaski: had a few questions on https://review.openstack.org/#/c/211753/29 | 15:11 |
| *** annegentle has joined #openstack-nova | 15:11 | |
| PaulMurray | tdurakov, let jlanoux know if there is anything he can do to help - but be warned, we don't generally use shared storage | 15:11 |
| openstackgerrit | Mark McLoughlin proposed openstack/nova: servicegroup: remove the zookeeper driver https://review.openstack.org/246343 | 15:11 |
| bauzas | alaski: in particular about setting the id field | 15:12 |
| tdurakov | btw, here is the problem, first site to check timezone forget about daylight saving:http://www.timeanddate.com/worldclock/fixedtime.html?month=10&day=14&year=2005&hour=14&min=0&sec=0&p1=0 | 15:12 |
| *** READ10 has quit IRC | 15:13 | |
| *** venkat_p has quit IRC | 15:13 | |
| *** amotoki_ is now known as amotoki | 15:13 | |
| tdurakov | PaulMurray, i've already set up nfs for devstack, so it should be ok, jlanoux already helped me:) so yep | 15:13 |
| *** venkat_p has joined #openstack-nova | 15:14 | |
| *** sneti has quit IRC | 15:14 | |
| *** sneti_ has joined #openstack-nova | 15:14 | |
| PaulMurray | tdurakov, good - glad you are getting this moving - its very important | 15:14 |
| alaski | bauzas: okay, looking | 15:15 |
| *** pumaranikar has joined #openstack-nova | 15:15 | |
| tdurakov | PaulMurray, yes | 15:15 |
| obondarev | rdopiera: ping | 15:15 |
| *** mdrabe has quit IRC | 15:15 | |
| rdopiera | obondarev: pong | 15:16 |
| * PaulMurray heads off in search of coffee | 15:16 | |
| obondarev | rdopiera: ajo said you're looking into https://bugs.launchpad.net/neutron/+bug/1414559 ? | 15:16 |
| openstack | Launchpad bug 1414559 in neutron "OVS drops RARP packets by QEMU upon live-migration - VM temporarily disconnected" [Undecided,Invalid] - Assigned to Oleg Bondarev (obondarev) | 15:16 |
| obondarev | rdopiera: do you? | 15:16 |
| *** Guest80875 is now known as mfisch | 15:16 | |
| *** mfisch is now known as Guest20594 | 15:17 | |
| rdopiera | obondarev: not at this exact moment, but I do want to fix it | 15:17 |
| obondarev | rdopiera: just wanted to sync with you, as I have neutron and nova patches which I'm testing right now | 15:17 |
| *** Guest20594 is now known as mfisch | 15:18 | |
| obondarev | rdopiera: should publish them by tomorrow eod | 15:18 |
| *** mfisch has quit IRC | 15:18 | |
| *** mfisch has joined #openstack-nova | 15:18 | |
| rdopiera | obondarev: that's awesome, please let me know when you do and I will test too | 15:18 |
| obondarev | rdopiera: thanks, will do | 15:19 |
| *** venkat_p has quit IRC | 15:19 | |
| *** jaypipes has quit IRC | 15:19 | |
| *** venkat_p has joined #openstack-nova | 15:19 | |
| *** claudiub has quit IRC | 15:20 | |
| rdopiera | obondarev: funny how it is marked as "invalid" | 15:20 |
| *** doug-fish has joined #openstack-nova | 15:21 | |
| obondarev | rdopiera: not easy to reproduce I guess | 15:21 |
| *** dustins has quit IRC | 15:22 | |
| *** mtanino has joined #openstack-nova | 15:23 | |
| *** obondarev is now known as obondarev_afk | 15:23 | |
| *** jwcroppe has joined #openstack-nova | 15:24 | |
| *** andreykurilin__ has joined #openstack-nova | 15:24 | |
| alaski | bauzas: commented. I'll double check on 'id', but I was under the impression that sqlalchemy handled that | 15:25 |
| bauzas | alaski: that's my open question :) | 15:25 |
| bauzas | alaski: if the SQLA object is updated by calling .save(), I'm fine | 15:25 |
| bauzas | alaski: tbh, I was lazy, I should have verified by myself :/ | 15:26 |
| *** venkat_p has quit IRC | 15:26 | |
| *** venkat_p has joined #openstack-nova | 15:26 | |
| *** dustins has joined #openstack-nova | 15:27 | |
| alaski | it's my code, I should be able to defend what it does | 15:27 |
| *** tonytan4ever has joined #openstack-nova | 15:28 | |
| *** garyk has quit IRC | 15:29 | |
| *** doug-fish has quit IRC | 15:29 | |
| *** ctrath has quit IRC | 15:29 | |
| *** doug-fish has joined #openstack-nova | 15:30 | |
| *** moshele has quit IRC | 15:31 | |
| *** doug-fis_ has joined #openstack-nova | 15:31 | |
| *** doug-fi__ has joined #openstack-nova | 15:33 | |
| *** doug-fi__ has quit IRC | 15:33 | |
| *** mc_nair has quit IRC | 15:33 | |
| *** doug-fi__ has joined #openstack-nova | 15:34 | |
| *** ctrath has joined #openstack-nova | 15:34 | |
| *** dkliban has joined #openstack-nova | 15:34 | |
| *** doug-fish has quit IRC | 15:34 | |
| BobBall | Anyone know of a new bug for oslo.versionobjects recursion depth failures? https://bugs.launchpad.net/nova/+bug/1470302 It's constantly killing https://review.openstack.org/#/c/244789/ and seems to be showing up in logstash quite a lot ATM (based on query from 1470302). I've not found a new bug yet, but it seems to be happening too regularly for there not to be a bug already? | 15:34 |
| openstack | Launchpad bug 1470302 in OpenStack Compute (nova) "gate-nova-python27 fails with "RuntimeError: maximum recursion depth exceeded"" [High,Fix released] - Assigned to Dan Smith (danms) | 15:34 |
| *** daemontool has quit IRC | 15:34 | |
| dims | BobBall : https://review.openstack.org/#/c/246013/ is about to merge | 15:35 |
| BobBall | ah awesome | 15:35 |
| bauzas | dims: hopefully :) | 15:36 |
| *** doug-fis_ has quit IRC | 15:36 | |
| *** vlaza has quit IRC | 15:36 | |
| bauzas | 20 mins left :) | 15:36 |
| *** klkumar has quit IRC | 15:36 | |
| *** dkliban has left #openstack-nova | 15:36 | |
| * BobBall will be patient | 15:36 | |
| *** markus_z_meeting is now known as markus_z | 15:37 | |
| *** mdrabe has joined #openstack-nova | 15:37 | |
| tdurakov | pkoniszewski, added you for review | 15:37 |
| *** pratikma_ has joined #openstack-nova | 15:38 | |
| *** klkumar has joined #openstack-nova | 15:39 | |
| *** pratikmallya has quit IRC | 15:41 | |
| *** paul-carlton has quit IRC | 15:43 | |
| *** amotoki_ has joined #openstack-nova | 15:44 | |
| *** garyk1 has quit IRC | 15:45 | |
| *** kfarr has joined #openstack-nova | 15:45 | |
| *** amotoki has quit IRC | 15:47 | |
| *** pratikma_ is now known as pratikmallya | 15:48 | |
| *** bnemec has quit IRC | 15:48 | |
| *** electrocucaracha has joined #openstack-nova | 15:49 | |
| *** armax has quit IRC | 15:49 | |
| *** Piet has quit IRC | 15:50 | |
| *** mc_nair has joined #openstack-nova | 15:51 | |
| *** venkat_p has quit IRC | 15:51 | |
| *** diegows has quit IRC | 15:51 | |
| *** venkat_p has joined #openstack-nova | 15:51 | |
| *** dustins has quit IRC | 15:52 | |
| *** dustins has joined #openstack-nova | 15:52 | |
| *** venkat_p has quit IRC | 15:53 | |
| *** tdurakov has quit IRC | 15:56 | |
| *** vilobhmm has joined #openstack-nova | 15:56 | |
| *** daemontool has joined #openstack-nova | 15:57 | |
| *** EinstCrazy has quit IRC | 15:57 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/nova-specs: Tenant networking support for Ironic driver. https://review.openstack.org/237067 | 15:58 |
| mriedem | dansmith: i was going to add a note to the kilo release notes that the flavor migrations will put extra load on conductor so you may need to scale those out until liberty, are you ok with that? | 15:58 |
| *** vilobhmm1 has joined #openstack-nova | 15:58 | |
| *** atuvenie_ has quit IRC | 15:58 | |
| dansmith | mriedem: yeah | 15:58 |
| *** paul-carlton has joined #openstack-nova | 15:58 | |
| *** daemontool has quit IRC | 15:59 | |
| *** jaypipes has joined #openstack-nova | 15:59 | |
| *** daemontool has joined #openstack-nova | 15:59 | |
| *** kfarr has quit IRC | 16:00 | |
| mriedem | done https://wiki.openstack.org/wiki/ReleaseNotes/Kilo#Upgrade_Notes_2 | 16:00 |
| mriedem | may not need to be bold, idk | 16:01 |
| *** mgoddard_ has joined #openstack-nova | 16:01 | |
| *** mjura has quit IRC | 16:01 | |
| *** vilobhmm has quit IRC | 16:01 | |
| mriedem | bbiab | 16:01 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: review guide: add location details for config options https://review.openstack.org/245789 | 16:02 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: config options: centralize section "serial_console" https://review.openstack.org/244177 | 16:02 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: config options: enhance help test of section "serial_console" https://review.openstack.org/246465 | 16:02 |
| dansmith | mriedem: yep, seems fine | 16:02 |
| *** bnemec has joined #openstack-nova | 16:03 | |
| *** mgoddard has quit IRC | 16:03 | |
| *** tdurakov has joined #openstack-nova | 16:03 | |
| *** diegows has joined #openstack-nova | 16:04 | |
| *** rcernin has quit IRC | 16:04 | |
| dansmith | klindgren: now that you're all done with your migrations on kilo, you have your [upgrade_levels] all cleared out right? | 16:04 |
| bauzas | dansmith: mriedem: correct me if I'm wrong but getting a GuruMeditationReport for the conductor could help finding if most of the green threads are running the flavor mig, nope ? | 16:04 |
| *** diegows has quit IRC | 16:04 | |
| dansmith | bauzas: if you get lucky, maybe | 16:05 |
| dansmith | bauzas: depends on what the problem is | 16:05 |
| bauzas | dansmith: for sure, I'm thinking of long-running tasks | 16:05 |
| johnthetubaguy | PaulMurray: re your questions on error handling in those live-migrate specs, I think we should use this: http://developer.openstack.org/api-ref-compute-v2.1.html#os-instance-actions-v2.1 | 16:05 |
| dansmith | bauzas: yeah, I just don't know where the problem is | 16:05 |
| bauzas | dansmith: agreed, that's just something really quick to do without really needing to profile | 16:06 |
| bauzas | I meant, not sure if klindgren is knowing that | 16:06 |
| dansmith | yep, if they're all in the same function it might help | 16:06 |
| *** mriedem has quit IRC | 16:06 | |
| *** tdurakov has quit IRC | 16:06 | |
| markus_z | johnthetubaguy: That's the split up in the "cofig options" change we talked about yesterday: https://review.openstack.org/#/c/244177/4 Let me know if you think this is the right granularity. | 16:07 |
| *** gcb has quit IRC | 16:07 | |
| johnthetubaguy | markus_z: ah, so you caught me at a good point, I will take a quick peak now | 16:07 |
| *** ildikov has quit IRC | 16:07 | |
| klindgren | dansmith, yea we made sure of that a few months ago. We don't have any lingering upgradelevels | 16:09 |
| dansmith | klindgren: okay | 16:09 |
| dansmith | klindgren: can you check your system_metadata table to see if you have any instances with non-deleted instance_type_* keys? | 16:10 |
| sdague | cburgess_: how's that fix / workaround coming for ebtables? | 16:10 |
| *** mriedem has joined #openstack-nova | 16:11 | |
| klindgren | dansmith, sure - one second | 16:12 |
| PaulMurray | johnthetubaguy, I think I wrote a confusing comment | 16:12 |
| lxsli | sdague: mind checking https://review.openstack.org/#/c/215568 please? Has a few +1s, been open for a while. The followup patch too if you'd be so kind :) | 16:13 |
| *** Yingxin1 has joined #openstack-nova | 16:13 | |
| PaulMurray | johnthetubaguy, I meant that a failure of the migration itself would not need to be reported because it is already being reported by existing code | 16:13 |
| *** belmoreira has quit IRC | 16:13 | |
| PaulMurray | johnthetubaguy, the failure to initiate the pause would need to be reported as you say | 16:13 |
| PaulMurray | johnthetubaguy, the question was about should a failure to pause also fail the migration | 16:14 |
| johnthetubaguy | PaulMurray: ah, it sort of is, but I think we might be missing the instance action at the moment. | 16:14 |
| PaulMurray | johnthetubaguy, and I think the answer is no | 16:14 |
| johnthetubaguy | PaulMurray: so it depends, the pause might trigger the live-migrate to fail, in which case you might end up with both failing, and the server in an ERROR state | 16:15 |
| openstackgerrit | Yingxin Cheng proposed openstack/nova: Use stevedore for scheduler host manager https://review.openstack.org/246476 | 16:15 |
| johnthetubaguy | PaulMurray: if the pause fails, but live migration succeeds, thats fine, only report the failed pause | 16:15 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Remove SoftDeleteMixin from BandwidthUsage https://review.openstack.org/240361 | 16:15 |
| *** rubasov has quit IRC | 16:16 | |
| *** mlavalle has joined #openstack-nova | 16:16 | |
| mlavalle | johnthetubaguy: ping | 16:16 |
| PaulMurray | johnthetubaguy, Is it part of this spec if we need to add instance action to the live migration | 16:16 |
| *** otter768 has joined #openstack-nova | 16:16 | |
| johnthetubaguy | PaulMurray: that can probably be a separate spec, probably best to concentrate on the pause | 16:16 |
| johnthetubaguy | PaulMurray: same argument for cancel of course | 16:17 |
| sdague | lxsli: looking | 16:17 |
| PaulMurray | johnthetubaguy, sounds good | 16:17 |
| johnthetubaguy | markus_z: thats it, got a few nits I am adding, but yeah, I think thats easier to review | 16:17 |
| johnthetubaguy | mlavalle: hows things? | 16:17 |
| bauzas | so, the nova patch for o.vo got rebalanced :( | 16:18 |
| mlavalle | johnthetubaguy: great, thanks for the review to https://review.openstack.org/#/c/90150/21/specs/mitaka/approved/internal-dns-resolution.rst | 16:18 |
| *** mwagner has quit IRC | 16:18 | |
| vilobhmm1 | johnthetubaguy : ping this spec has couple of reviews already https://review.openstack.org/#/c/222423/ if the cores find it ok…and if this gets merged I can submit the code | 16:18 |
| mlavalle | johnthetubaguy: do you have a few minutes to discuss comment to line 122? | 16:18 |
| johnthetubaguy | mlavalle: sure thing | 16:19 |
| *** amotoki_ has quit IRC | 16:19 | |
| mlavalle | johnthetubaguy: I don't quite understand your comment. Would you elaborate a little bit? | 16:19 |
| johnthetubaguy | vilobhmm1: sorry, not sure I understand the question | 16:19 |
| johnthetubaguy | mlavalle: ah, right | 16:19 |
| dansmith | bauzas: rebalanced? | 16:20 |
| vilobhmm1 | johnthetubaguy : https://review.openstack.org/#/c/222423/Â needs one more review from the core member | 16:20 |
| johnthetubaguy | mlavalle: so I am not keen on us detected when the passed in host name is a FQDN | 16:20 |
| johnthetubaguy | mlavalle: it feels a little odd somehow | 16:20 |
| vilobhmm1 | to be merged | 16:20 |
| vilobhmm1 | this is the servicegroup spec | 16:20 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add service status notification https://review.openstack.org/245678 | 16:20 |
| sdague | bauzas: yeh, the libvirt ebtables race is the primary culprit (also folks approving nova patches overnight, which basically halted all gate progress until we kicked them all out this morning) | 16:20 |
| bauzas | dansmith: I meant back in the zuul queue | 16:20 |
| johnthetubaguy | vilobhmm1: ah, yes, it does, hopefully folks will get to that sooner now it has a +2 on it | 16:21 |
| sdague | the heat job that just failed, failed on the ebtables issue | 16:21 |
| sdague | which reset us again | 16:21 |
| johnthetubaguy | mlavalle: so I had a few follow on thoughts from that, which go on a bit | 16:21 |
| *** otter768 has quit IRC | 16:21 | |
| bauzas | sdague: okay, thanks for the heads-up | 16:21 |
| mlavalle | johnthetubaguy: I am not proposing any change to nova's behaviour. All I am going to do is to take the data in the instance.hostname and send it to Neutron. Are we clear on that? | 16:21 |
| vilobhmm1 | johnthetubaguy : ok..lets hope so :) | 16:21 |
| johnthetubaguy | mlavalle: first was, well its not really part of this spec, as thats neutron behaviour, so I guess Nova just passes that stuff to Neutron | 16:22 |
| *** _d34dh0r53_ is now known as d34dh0r53 | 16:22 | |
| dansmith | sdague: you haven't sent out an email about not approving nova jobs.. do you still think we should do that? | 16:22 |
| johnthetubaguy | mlavalle: yep, the basic idea seems sound | 16:22 |
| bauzas | dansmith: all the changes are impacted, right? | 16:22 |
| sdague | well, this is the top nova patch right now, let's give it an hour and see what happens | 16:22 |
| lxsli | johnthetubaguy: I discussed an oslo.config opt flag "becoming default in next release" with someone, was it you? | 16:23 |
| *** thangp has joined #openstack-nova | 16:23 | |
| dansmith | sdague: I guess I didn't see the point since nothing should be getting into the gate queue because this is 100% fail | 16:23 |
| *** alex_klimov has quit IRC | 16:23 | |
| johnthetubaguy | lxsli: unsure | 16:23 |
| sdague | dansmith: because we don't check for fresh test results | 16:23 |
| sdague | so mikal went through and approved a bunch of things that were jenkins +1 | 16:23 |
| dansmith | sdague: okay so a few things can make it in | 16:23 |
| mlavalle | johnthetubaguy: on the neutron side there is a domain defined. All I am saying is that if the trailing part of the hostname from nova matches the domain in Neutron, I am going to consider it a FQDN. | 16:23 |
| johnthetubaguy | mlavalle: sorry, not making this very clear yet... | 16:23 |
| dansmith | okay | 16:23 |
| johnthetubaguy | mlavalle: yeah, the problem is the official list of top level domains is expanding these days, etc, it seems a little odd | 16:24 |
| johnthetubaguy | mlavalle: what does cloud-init do with that today? I guess it just sets the hostname to what is specified, its possible we trim the hostname before passing it there, I don't remember now | 16:24 |
| *** rpodolyaka1 has quit IRC | 16:24 | |
| sdague | yeh, about ~30 patches - https://review.openstack.org/#/q/project:openstack/nova+label:Verified%253E%253D1%252Cjenkins+status:open+label:Code-Review%253E%253D2,n,z | 16:25 |
| johnthetubaguy | mlavalle: anyways, I think the easy fix is to remove that detail from the nova spec, as its technically neutron behaviour, but honestly it sounds/feels like it will create a whole heap of bugs and issues with miss-matches between what happens today with cloud-init, etc, and what will happen with the configured DNS. I could be over thinking it though. | 16:26 |
| *** salv-orl_ has quit IRC | 16:26 | |
| *** hemnafk is now known as hemna | 16:26 | |
| dansmith | sdague: okay well I didn't think it was that big of a deal because I expected check to run on most everything | 16:26 |
| *** mspreitz has joined #openstack-nova | 16:26 | |
| *** nkrinner has quit IRC | 16:27 | |
| bauzas | nope, it's no longer the case AFAICS | 16:27 |
| bauzas | it only verifies a +1 from Jenkins, not the age | 16:27 |
| dansmith | I'm pretty sure it considers the age | 16:28 |
| dansmith | sdague: ? | 16:28 |
| *** READ10 has joined #openstack-nova | 16:28 | |
| sdague | it does not | 16:28 |
| *** armax has joined #openstack-nova | 16:28 | |
| dansmith | WAT | 16:28 |
| bauzas | dansmith: I can't see https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L34-L68 | 16:28 |
| dansmith | I can't keep it straight | 16:28 |
| sdague | that was changed sometime last spring | 16:28 |
| *** shaohe_feng1 has quit IRC | 16:28 | |
| klindgren | dansmith, select * from instance_system_metadata where deleted_at is null and `key` LIKE 'instance_type_%'; | 16:28 |
| klindgren | = 15670 rows | 16:29 |
| mlavalle | johnthetubaguy: so, the host name set by cloud-init in the instance might not be the same thing as instance.hostname in nova? | 16:29 |
| dansmith | klindgren: are those instances deleted? | 16:29 |
| dansmith | klindgren: I don't think you want deleted for the sysmeta row, but for the instance | 16:29 |
| johnthetubaguy | mlavalle: unsure, I was just checking this logic: https://github.com/openstack/nova/blob/aa141f0d8ea90a082558a9969340ba655a07c417/nova/utils.py#L774 going a bit deeper | 16:30 |
| mlavalle | johnthetubaguy: but is that the possibility that worries you? | 16:30 |
| *** rpodolyaka1 has joined #openstack-nova | 16:30 | |
| johnthetubaguy | mlavalle: right now, yeah | 16:30 |
| mlavalle | johnthetubaguy: ok, let me dig a little deeper. I'll get back to you in a bit | 16:31 |
| PaulMurray | pkoniszewski, are you still around | 16:31 |
| lxsli | johnthetubaguy: looking at https://wiki.openstack.org/wiki/Nova/Process again - it's OK for me to just edit it? | 16:31 |
| *** Piet has joined #openstack-nova | 16:31 | |
| lxsli | to get https://review.openstack.org/#/c/206070 off my plate | 16:32 |
| johnthetubaguy | lxsli: yeah, honestly I have a TODO to add that into devref so its in the tree rather than the wiki | 16:32 |
| lxsli | johnthetubaguy: probably easiest for me to edit the wiki then convert to devref than try to do both at once though | 16:33 |
| johnthetubaguy | lxsli: +1 | 16:33 |
| lxsli | johnthetubaguy: cool, thanks | 16:33 |
| johnthetubaguy | lxsli: edit the wiki, then convert, makes sense I think | 16:33 |
| *** josecastroleon has quit IRC | 16:33 | |
| *** rk4n has quit IRC | 16:33 | |
| *** mspreitz has quit IRC | 16:34 | |
| *** rk4n has joined #openstack-nova | 16:34 | |
| *** gokrokve has joined #openstack-nova | 16:35 | |
| *** pkoniszewski has quit IRC | 16:36 | |
| johnthetubaguy | mlavalle: seems like cloud-init considers everything after the . as an FQDN: http://bazaar.launchpad.net/~cloud-init-dev/cloud-init/trunk/view/head:/cloudinit/util.py#L932 | 16:37 |
| * mlavalle looking | 16:37 | |
| *** armax has quit IRC | 16:37 | |
| bauzas | johnthetubaguy: FWIW, will respin https://review.openstack.org/#/c/221280/ | 16:38 |
| *** rk4n has quit IRC | 16:38 | |
| bauzas | alaski: I'm also tempted to resubmit the very old Tasks spec, you happy with that? | 16:38 |
| johnthetubaguy | mlavalle: seems like this bit is also important: http://bazaar.launchpad.net/~cloud-init-dev/cloud-init/trunk/view/head:/cloudinit/sources/__init__.py#L171 | 16:39 |
| *** armax has joined #openstack-nova | 16:40 | |
| *** rk4n has joined #openstack-nova | 16:40 | |
| mlavalle | johnthetubaguy: thanks, going through it | 16:40 |
| *** e0ne has quit IRC | 16:41 | |
| *** browne has joined #openstack-nova | 16:42 | |
| johnthetubaguy | mlavalle: cool, I am leaning towards treating everything with a "." in it as an FQDN, but I think for the sake of the Nova spec, you could just say Neutron is planning to be consistent with cloud-init's use, or something like that? | 16:42 |
| sdague | does anyone know netfilter folks? | 16:43 |
| sdague | it seems a little odd that ebtables requires all users of it to say "please don't punch me in the face" to not be punched in the face | 16:43 |
| *** doug-fi__ is now known as doug-fish | 16:43 | |
| sdague | i.e. why is --concurrent even an option vs. just the default | 16:43 |
| *** rk4n has quit IRC | 16:44 | |
| *** rk4n has joined #openstack-nova | 16:44 | |
| *** rmart04 has quit IRC | 16:45 | |
| mlavalle | johnthetubaguy: ah, ok. Upon encountering the first '.', then we assume it is a fqdn. right? the hostname is the string before the first '.'. correct? | 16:46 |
| *** gjayavelu has joined #openstack-nova | 16:46 | |
| *** rk4n has quit IRC | 16:48 | |
| danpb | sdague: looking at the code they wanted to be able to consider failure to lock to be a fatal error when --concurrent is used | 16:49 |
| *** sahid has quit IRC | 16:49 | |
| danpb | sdague: if they had made it the default, then this would have caused a behaviour change for existing users in some cases | 16:50 |
| sdague | danpb: hmmm, yeh, it's just unfortunate, because it means if another tool shows up that uses it, then you get a race again | 16:51 |
| *** njirap has joined #openstack-nova | 16:51 | |
| alaski | bauzas: go for it | 16:51 |
| bauzas | alaski: you made good points in the manifesto, I first need to respin it | 16:52 |
| sdague | and there is no way to specify at a system level, always lock | 16:52 |
| bauzas | alaski: I just had one question for you, with "This is only true for API instantiated actions. Something like an evacuate would not be tracked." | 16:52 |
| *** vilobhmm1 has quit IRC | 16:52 | |
| bauzas | alaski: I don't understand, evacuate is API-triggered, right? | 16:52 |
| *** electrocucaracha has quit IRC | 16:52 | |
| *** unicell has quit IRC | 16:52 | |
| johnthetubaguy | mlavalle: yeah, that sounds about right | 16:53 |
| *** e0ne has joined #openstack-nova | 16:53 | |
| bauzas | alaski: sorry, the context being https://review.openstack.org/#/c/221280/3..4/doc/source/tasks.rst,cm L36 | 16:54 |
| bauzas | nah | 16:54 |
| bauzas | https://review.openstack.org/#/c/221280/3/doc/source/tasks.rst,cm L36 | 16:54 |
| alaski | bauzas: yeah, not sure what I was thinking there. what I meant to say was that anything initiated by a periodic task wouldn't be tracked | 16:54 |
| bauzas | alaski: oh yeah, for sure | 16:54 |
| mlavalle | johnthetubaguy: it actually makes that whole section about fqdn and pqdn irrelevant in the nova spec. in hostname I am just getting the first label. I should remove that section and just indicate that we are going to comply with cloud-init. Makes sense? | 16:55 |
| bauzas | alaski: you probably meant when the compute is restarted, and the evacuations are checked | 16:55 |
| alaski | bauzas: yeah, that. and checks for destroyed instances as well | 16:55 |
| *** vishwana_ is now known as vishwanathj | 16:55 | |
| bauzas | alaski: okay, I will make that point clear | 16:56 |
| bauzas | alaski: but I don't remember periodic tasks changing an instance state ? | 16:56 |
| alaski | bauzas: it could confirm a resize | 16:58 |
| alaski | or offload a shelved instance | 16:58 |
| alaski | and I just noticed the virt lifecycle events which aren't covered | 16:58 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Fix max concurrent builds's unlimited semaphore https://review.openstack.org/239054 | 16:58 |
| bauzas | mmm | 16:58 |
| *** mriedem is now known as mriedem_meeting | 16:59 | |
| bauzas | alaski: could you please leave a comment accordingly ? | 16:59 |
| alaski | sure | 16:59 |
| *** angdraug has quit IRC | 16:59 | |
| *** njirap has quit IRC | 16:59 | |
| *** ctrath1 has joined #openstack-nova | 17:00 | |
| *** ljxiash has joined #openstack-nova | 17:00 | |
| *** njirap has joined #openstack-nova | 17:00 | |
| *** korzen_ has joined #openstack-nova | 17:01 | |
| *** korzen has quit IRC | 17:02 | |
| *** ctrath has quit IRC | 17:02 | |
| *** mwagner has joined #openstack-nova | 17:03 | |
| *** daemontool has quit IRC | 17:04 | |
| *** sacharya has joined #openstack-nova | 17:04 | |
| *** daemontool has joined #openstack-nova | 17:04 | |
| *** tpeoples_away is now known as tpeoples | 17:04 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Fix max concurrent builds's unlimited semaphore https://review.openstack.org/239054 | 17:05 |
| *** rushil has joined #openstack-nova | 17:05 | |
| mlavalle | johnthetubaguy: in light of this exchange, I realize that the whole section about PQDN and FQDN was really rambling. It is irrelevant on the Nova side. I'll remove it. Thanks for the guidance! :-) | 17:05 |
| *** ctrath1 has quit IRC | 17:05 | |
| *** rlrossit has quit IRC | 17:05 | |
| johnthetubaguy | mlavalle: no problems, yeah, I think just dropping it is probably best. Less is often more when it comes to specs. | 17:06 |
| *** mgoddard_ has quit IRC | 17:06 | |
| *** jlanoux has quit IRC | 17:07 | |
| *** yassine has quit IRC | 17:07 | |
| cburgess_ | sdague Working on it today, should have it up soon. Did have 1 question when you get a minute. | 17:08 |
| edleafe | markus_z: still around? Got a question about the centralized conf stuff | 17:09 |
| dansmith | klindgren: are you still working on trying to do that query? | 17:09 |
| markus_z | edleafe: yepp, I'm here | 17:09 |
| openstackgerrit | Moshe Levi proposed openstack/nova: libvirt: mlnx_direct vif type removal https://review.openstack.org/181671 | 17:10 |
| klindgren | I was but - fires came up - I am not the best at cross table joins. By best I mean - its been a long time since i had to do something more complex than a select <stuff> from <table> where <blah>; | 17:11 |
| klindgren | if you have a querry I can run it - but working on a prod issue | 17:11 |
| edleafe | markus_z: so should all conf options under, say, nova/scheduler/ be moved to one file: nova/conf/scheduler.py? Or should it be nova/conf/scheduler/manager.py, etc? | 17:11 |
| edleafe | markus_z: IOW, one conf file for each original file? | 17:12 |
| *** rubasov has joined #openstack-nova | 17:12 | |
| dansmith | klindgren: well, if you want to make sure that something else is not going on, it'd be good to get that when you have a chance | 17:13 |
| openstackgerrit | Samuel Matzek proposed openstack/nova: Reschedule boot when attach volume fails https://review.openstack.org/246505 | 17:13 |
| *** blaisebool has quit IRC | 17:13 | |
| *** rlrossit has joined #openstack-nova | 17:13 | |
| markus_z | edleafe: The name of the "nova.conf" section (like: "[DEFAULT]") determines the name of the new module | 17:13 |
| *** e0ne has quit IRC | 17:13 | |
| markus_z | I see now that there is no "[scheduler]" section, is that right? | 17:13 |
| dansmith | klindgren: I don't really want to try to make a query for you to run that could have issues, but basically joining instances with sysmeta on uuid, where instance is not deleted and the key is as you had it.. | 17:13 |
| edleafe | markus_z: not following | 17:14 |
| markus_z | edleafe: sorry, I look to your patch set again to pick a better example | 17:14 |
| sdague | cburgess_: sure, fire away | 17:14 |
| sdague | also, I wrote the terrible part of the work around for devstack right now | 17:14 |
| cburgess_ | sdague OK | 17:15 |
| sdague | https://review.openstack.org/#/c/246501 | 17:15 |
| sdague | because this is hurting us pretty hard in landing patches | 17:15 |
| cburgess_ | sdague So... there is no timeout for ebtables. What I mean is if you invoke it with --concurrent then you will wait.. forever. | 17:15 |
| dansmith | sdague: can you make that only happen if some config is set? so we don't all destroy our machines if we run that? | 17:15 |
| *** unicell has joined #openstack-nova | 17:15 | |
| cburgess_ | sdague Oh sorry I thought it was one of those occassionaly thing happens things. Not OMG hair on fire. | 17:16 |
| markus_z | edleafe: http://docs.openstack.org/developer/nova/sample_config.html | 17:16 |
| *** korzen has joined #openstack-nova | 17:16 | |
| sdague | dansmith: yes, that would be reasonable. Though it doesn't destroy your machine any more than the rest of devstack. | 17:17 |
| *** pratikmallya has quit IRC | 17:17 | |
| sdague | and it's only set that way for affected platforms of the bug | 17:17 |
| *** korzen_ has quit IRC | 17:17 | |
| dansmith | sdague: we move things in /sbin elsewhere in devstack/ | 17:17 |
| edleafe | markus_z: I guess I'm not asking clearly | 17:17 |
| markus_z | edleafe: The idea was to have one python module per "nova.conf" section. Sections are the "[...]" things in the config file. | 17:17 |
| cburgess_ | sdague So the question is.. do we need a timeout? I ask because while its not hard to set a timer in python to do this, due to how we invoke things via the oslo libraries and rootwrap vs the new fancy daemon thing its actually a bit of work to get all the bits to properly create a timeout. | 17:18 |
| sdague | dansmith: we modify so much of /etc | 17:18 |
| edleafe | markus_z: so nearly everything is in [DEFAULT] | 17:18 |
| cburgess_ | dansmith Yeah sdague has a point. devstack pretty much stomp /etc | 17:18 |
| dansmith | sdague: yeah, but dicking around in /sbin is a different thing I think | 17:18 |
| sdague | cburgess_: because you are concerned that ebtables deadlocks? | 17:18 |
| *** shaohe_feng has joined #openstack-nova | 17:19 | |
| cburgess_ | sdague I have no evidence of that. Just thinking/coding defensibly. | 17:19 |
| markus_z | edleafe: Yes, I didn't know that before. | 17:19 |
| sdague | cburgess_: right, that's a good question | 17:19 |
| sdague | cburgess_: let me look for a sec | 17:19 |
| cburgess_ | sdague We can try it without a timeout. If it proves to be an issue we can fix it later. | 17:20 |
| edleafe | markus_z: for reviewing simplicity, I've been thinking about 1:1 files in nova/conf/ to the location of the original CONF declaration | 17:20 |
| edleafe | markus_z: IOW, if I move an option from nova/scheduler/manager.py, it will go in nova/conf/scheduler/manager.py | 17:20 |
| *** pradk has quit IRC | 17:21 | |
| *** e0ne has joined #openstack-nova | 17:21 | |
| johnthetubaguy | edleafe: can't we just group all the scheduler options in one file? | 17:22 |
| markus_z | edleafe: I think I get you. | 17:22 |
| cburgess_ | sdague There are a few ways to do a timeout, but all of them would require us to muck with oslo_concurrency | 17:22 |
| markus_z | edleafe: what johnthetubaguy said | 17:22 |
| edleafe | johnthetubaguy: that was the other option I was asking about | 17:22 |
| sdague | cburgess_: yeh, honestly, for all the kwargs that oslo_concurrency takes, the fact that it doesn't support command timeout is a thing | 17:22 |
| dims | cburgess_ : help us fix things in oslo_concurrency? :) | 17:22 |
| *** njirap has quit IRC | 17:23 | |
| sdague | that being said, is this going to be in an interuptable state? | 17:23 |
| johnthetubaguy | edleafe: possibly keep the filters and weights ones in their own files, thinking about it | 17:23 |
| mdbooth | Updating a spec here. Remind me, can I add random sections, or will a bot complain at me? | 17:23 |
| edleafe | markus_z: johnthetubaguy: so everything under nova/scheduler/ will be in nova/conf/scheduler.py, and everything under nova/foo/ will be in nova/conf/foo.py | 17:23 |
| cburgess_ | sdague dims Right so... we can fix oslo but it takes even more time to land the fix. We could maybe land a fix no timeout, then add timeout to oslo, then land a time fix with timeout. | 17:23 |
| sdague | cburgess_: yeh, lets do that path | 17:23 |
| cburgess_ | sdague Yeah as I recall its just sitting attempting to lock a file. | 17:23 |
| sdague | land nova without the timeout | 17:23 |
| sdague | that makes things better | 17:24 |
| openstackgerrit | Tom Patzig proposed openstack/nova: Add 'insecure' option to ironic sect in nova.conf https://review.openstack.org/232560 | 17:24 |
| johnthetubaguy | edleafe: not sure we need to be that clear cut, see what the largest natural groupings turn out to be | 17:24 |
| cburgess_ | sdague I think I wrote a test along time ago around this and determined that it was possible to kill it. | 17:24 |
| cburgess_ | sdague OK | 17:24 |
| sdague | cburgess_: might be worth seeing if libvirt is keeping an eye on a timeout here as well, danpb, any ideas there? | 17:24 |
| sdague | because if we are as good as libvirt, we're probably fine | 17:24 |
| cburgess_ | sdague So the gate has a new enough ebtables and libvirt right? We just aren't going to do any version testing and just assume we have the right versions moving forwad right? | 17:24 |
| johnthetubaguy | edleafe: like, common stuff, lots of optional stuff, might be common, like scheduler.py and scheduler_filter_weights.py maybe? not sure | 17:25 |
| sdague | it has new enough ebtables | 17:25 |
| markus_z | edleafe: johnthetubaguy: The grouping most people agreed on was using the sections ("[libvirt]", "[default]", ...) for the grouping. | 17:25 |
| cburgess_ | sdague I don't think there is a timeout in libvirt, its been a while since looked at the code though. | 17:25 |
| cburgess_ | sdague OK | 17:25 |
| *** tdurakov has joined #openstack-nova | 17:25 | |
| *** unicell has quit IRC | 17:25 | |
| cburgess_ | This should be fairly easy. | 17:25 |
| sdague | it does *not* have new enough libvirt on ubunut | 17:25 |
| *** rushiagr is now known as rushiagr_away | 17:25 | |
| *** matrohon has quit IRC | 17:25 | |
| edleafe | johnthetubaguy: that's a very subjective grouping, no? | 17:25 |
| johnthetubaguy | markus_z: yeah, that makes sense, but I don't want all of the default ones in one file, I suspect | 17:25 |
| sdague | the fedora jobs (which are non voting) do, so you could probably see results there | 17:25 |
| cburgess_ | sdague Right so thats the ugly work around thing you put into devstack. I'll merge a similiar thing to nova in a latter patch for the ugly workaround. | 17:25 |
| edleafe | markus_z: we're gonna have one ginormous default.py file! | 17:25 |
| johnthetubaguy | edleafe: yes, we want a logical grouping | 17:26 |
| sdague | cburgess_: right, and I'll make devstack use the nova ugly work around | 17:26 |
| cburgess_ | sdague Right | 17:26 |
| sdague | I just wanted to try to deal with the bleeding here | 17:26 |
| danpb | cburgess_: there's no timeout in libvirt for ebtables and i'm not really seing a reason to add one either | 17:26 |
| *** gjayavel_ has joined #openstack-nova | 17:26 | |
| johnthetubaguy | markus_z: edleafe: this is a horrible answer, but I think it will be easier to tell whats correct when we see the true horror of the specific change. | 17:26 |
| cburgess_ | danpb Didn't think there was. | 17:26 |
| markus_z | edleafe: johnthetubaguy: Right, we wanted also figure out what the natural grouping is. | 17:27 |
| edleafe | johnthetubaguy: markus_z: ok, well I've started putting all into one file. When that's ready I'll push it and we can argue on gerrit, ok? | 17:27 |
| danpb | IME adding timeouts on this kind of thing has only ever led to use *causing* failures where something would have in fact succeeeded with a longer timeout | 17:27 |
| cburgess_ | sdague So.. land nova no timeout, unrelated we can go add a timeout to oslo but.. without a timeout we are as good as libvirt. | 17:27 |
| johnthetubaguy | edleafe: makes sense | 17:27 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 17:27 |
| sdague | cburgess_: yeh | 17:27 |
| cburgess_ | danpb I'm totally fine with that. | 17:27 |
| markus_z | edleafe: I need a second to think about it | 17:27 |
| johnthetubaguy | edleafe: maybe just do a few models at first, to see how it looks? | 17:27 |
| danpb | any ebtables command should complete eventually unless the kernel is so fubar that you are doomed regardless | 17:27 |
| cburgess_ | danpb As in, if we are as a good as libvirt thats good enough for nova. | 17:27 |
| cburgess_ | sdague ^^ | 17:27 |
| sdague | yeh | 17:28 |
| danpb | (ie only time i've seen ebtables hang is when the kernel has already reported mild panics and tried to carry on) | 17:28 |
| sdague | I think the only fail scenario is if ebtables gets killed before it's lock file is cleaned out | 17:28 |
| sdague | then starts stacking up on stale lock file | 17:28 |
| johnthetubaguy | markus_z: sorry, I forgot to submit my questions earlier, done that now: https://review.openstack.org/#/c/244177/4 | 17:28 |
| sdague | I don't know how well it recovers from said condition | 17:28 |
| danpb | if correctly implemented there should be no chance of stale lock files | 17:28 |
| cburgess_ | sdague Yeah so I don't think thats possible the way it does locking. I will look at the code to confirm but I think we are good. | 17:28 |
| johnthetubaguy | edleafe: its tempting to get one merged before we start on uploading new ones | 17:29 |
| *** rk4n has joined #openstack-nova | 17:29 | |
| danpb | ie, if it holds a fcntl lock, those get released when the process exits | 17:29 |
| cburgess_ | Right | 17:29 |
| markus_z | edleafe: yeah, ok, let's discuss in code. Makes probably more sense. | 17:29 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Move Process and Mentoring pages to devref https://review.openstack.org/206070 | 17:29 |
| cburgess_ | Pretty sure when I looked last year it was using fcntl | 17:29 |
| danpb | it certainly shouldn't rely on mere existance of the file | 17:29 |
| johnthetubaguy | markus_z: I am wondering about explicitly importing the used options, to help grep make it clear where things are coming from? or is that overkil? | 17:29 |
| markus_z | johnthetubaguy: thanks, I'll take a look | 17:29 |
| edleafe | johnthetubaguy: sure, mine is dependent on markus_z's initial patch | 17:29 |
| johnthetubaguy | edleafe: makes sense | 17:29 |
| *** gjayavelu has quit IRC | 17:29 | |
| cburgess_ | sdague Just as soon as I can remember how to use nova-network in devstack I'll get this patch up. Been spending way to much time with neutron. | 17:30 |
| *** salv-orl_ has joined #openstack-nova | 17:32 | |
| *** romainh has left #openstack-nova | 17:32 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/nova-specs: Volume support for multi hypervisors https://review.openstack.org/245986 | 17:32 |
| klindgren | dansmith, select * as inst_row_count from instance_system_metadata im join instances i on im.instance_uuid = i.id where im.deleted_at is null and im.key LIKE 'instance_type_%' and i.vm_state = 'deleted'; | 17:32 |
| sdague | cburgess_: heh | 17:32 |
| klindgren | result == 2200 rows | 17:32 |
| cburgess_ | sdague You can't wash that out. | 17:32 |
| openstackgerrit | Ed Leafe proposed openstack/nova: DO NOT MERGE - WIP https://review.openstack.org/245891 | 17:33 |
| edleafe | johnthetubaguy: markus_z: ^^ | 17:33 |
| dansmith | klindgren: you need d.deleted != null not vm_state to be sure | 17:33 |
| *** ssurana has joined #openstack-nova | 17:33 | |
| dansmith | klindgren: sorry, i.deleted | 17:33 |
| *** jdurgin has quit IRC | 17:33 | |
| *** mgoddard_ has joined #openstack-nova | 17:33 | |
| edleafe | johnthetubaguy: markus_z: I posted what I have so far | 17:33 |
| *** scheuran has quit IRC | 17:34 | |
| *** stackdump has joined #openstack-nova | 17:34 | |
| *** ctrath has joined #openstack-nova | 17:34 | |
| *** rcernin has joined #openstack-nova | 17:36 | |
| klindgren | dansmith, select * from instance_system_metadata im join instances i on im.instance_uuid = i.id where im.deleted_at is null and im.key LIKE 'instance_type_%' and i.deleted != null; | 17:37 |
| klindgren | Empty set (0.20 sec) | 17:37 |
| dansmith | klindgren: okay cool | 17:37 |
| dansmith | klindgren: I'm not positive about that im.deleted bit, tbh | 17:38 |
| dansmith | klindgren: but since you were getting results before and now not, I think that's probably fine | 17:38 |
| dansmith | klindgren: so are you interested in trying something hacky? | 17:38 |
| dansmith | klindgren: if I could get devstack on kilo to work, I'd be testing it, but alas | 17:38 |
| markus_z | johnthetubaguy: I answered your questions in https://review.openstack.org/#/c/244177/3 | 17:39 |
| mriedem_meeting | not deleted is 0 i think | 17:39 |
| *** mriedem_meeting is now known as mriedem | 17:39 | |
| mriedem | i.deleted != 0 | 17:39 |
| klindgren | I am willing to do a moderate amount of hackyness, balanced by the fact this is a production environment | 17:39 |
| sdague | cburgess_ / danpb : http://git.netfilter.org/ebtables/tree/libebtc.c#n187 which I think means we're fine without a timeout | 17:39 |
| *** ljxiash has quit IRC | 17:40 | |
| klindgren | i.deleted != 0 = 2200 | 17:40 |
| dansmith | klindgren: deleted!=0 means deleted | 17:40 |
| dansmith | so those you don't care about | 17:40 |
| *** pratikmallya has joined #openstack-nova | 17:42 | |
| cburgess_ | sdague danpb Well they call open with O_EXCL and only mask SIGINT. SIGKILL won't unlock the file. But.. I think the kernel will still clean the FD up on kill. | 17:43 |
| *** sacharya_ has joined #openstack-nova | 17:43 | |
| *** alexschm has quit IRC | 17:43 | |
| *** Piet has quit IRC | 17:44 | |
| *** jmccrory has quit IRC | 17:44 | |
| *** electrocucaracha has joined #openstack-nova | 17:45 | |
| danpb | cburgess_: sigh, yes, it is stupidly implemented - they really ought to use fcntl locks as they're the only guaranteed race-free auto-cleaned impl | 17:45 |
| *** electrocucaracha has quit IRC | 17:45 | |
| *** electrocucaracha has joined #openstack-nova | 17:45 | |
| *** sacharya has quit IRC | 17:45 | |
| cburgess_ | danpb But I think the FD should get cleaned up anyways right? Its a dirty implementation but I *think* its safe. | 17:45 |
| *** e0ne has quit IRC | 17:46 | |
| *** korzen has quit IRC | 17:47 | |
| *** apoorvad has joined #openstack-nova | 17:47 | |
| *** salv-orl_ has quit IRC | 17:47 | |
| *** browne has quit IRC | 17:47 | |
| *** ctrath has quit IRC | 17:48 | |
| danpb | cburgess_: nothing ensures the lockfile is deleted in SIGKILL, only in clean exit | 17:48 |
| markus_z | edleafe: I get it now. code helps me a lot :) I'll make comments | 17:48 |
| *** ZZelle_ has joined #openstack-nova | 17:48 | |
| *** salv-orlando has joined #openstack-nova | 17:48 | |
| danpb | of course if admins use sigkill they get what they deserve to some extent | 17:48 |
| *** garyk has joined #openstack-nova | 17:48 | |
| *** e0ne has joined #openstack-nova | 17:49 | |
| *** greghaynes has quit IRC | 17:49 | |
| dansmith | klindgren: you don't have any preprod to run this in? | 17:49 |
| *** auggy has quit IRC | 17:49 | |
| dansmith | klindgren: to measure potential performance improvement? | 17:49 |
| *** yonglihe has quit IRC | 17:50 | |
| cburgess_ | danpb /sigh yeah you are right. I missed that. | 17:50 |
| *** junjie has quit IRC | 17:50 | |
| *** betherly has quit IRC | 17:51 | |
| klindgren | dansmith, we do - but its no where near what we have in prod | 17:51 |
| *** READ10 has quit IRC | 17:51 | |
| klindgren | and I cant drive the same load in it - because I dont run all the same crap that the users do | 17:52 |
| dansmith | klindgren: well, obviously, but surely you can hamper your conductor, hammer your preprod, then apply this and see how it goes? | 17:52 |
| klindgren | IE 95% sure load is coming from metadata | 17:52 |
| klindgren | yea - like I said I am willing to do some hacky stuffs | 17:52 |
| *** tpeoples has quit IRC | 17:53 | |
| *** kgalanov has quit IRC | 17:53 | |
| *** rpodolyaka1 has quit IRC | 17:55 | |
| *** haomaiwang has quit IRC | 17:55 | |
| mhorban_ | Hi sdague, johnthetubaguy, Could you please look at https://review.openstack.org/#/c/237588/ ? | 17:55 |
| *** yonglihe has joined #openstack-nova | 17:56 | |
| *** _RuiChen_ has joined #openstack-nova | 17:56 | |
| *** artom_ has joined #openstack-nova | 17:56 | |
| *** thangp has quit IRC | 17:57 | |
| dansmith | klindgren: https://review.openstack.org/246530 | 17:57 |
| *** markvoelker_ has joined #openstack-nova | 17:57 | |
| *** burgerk_ has joined #openstack-nova | 17:57 | |
| *** ctrath has joined #openstack-nova | 17:58 | |
| *** lpetrut1 has joined #openstack-nova | 17:58 | |
| *** lpetrut has quit IRC | 17:58 | |
| *** lpetrut1 is now known as lpetrut | 17:58 | |
| dansmith | alaski: ^^ in case you want to try it too | 17:58 |
| *** jmccrory has joined #openstack-nova | 17:58 | |
| *** pkoniszewski_ has quit IRC | 17:59 | |
| *** baffle has quit IRC | 17:59 | |
| cburgess_ | dansmith LOL I love that option name and commit message | 17:59 |
| *** rlrossit has quit IRC | 17:59 | |
| *** bnemec has quit IRC | 17:59 | |
| *** mdrabe has quit IRC | 17:59 | |
| *** keekz has quit IRC | 17:59 | |
| *** agireud has quit IRC | 17:59 | |
| dansmith | cburgess_: I was going to go for more obscenity, but decided against it | 17:59 |
| *** _gryf has quit IRC | 17:59 | |
| *** ZZelle has quit IRC | 17:59 | |
| *** artom has quit IRC | 18:00 | |
| *** markvoelker has quit IRC | 18:00 | |
| *** sbadia has quit IRC | 18:00 | |
| *** _RuiChen has quit IRC | 18:00 | |
| *** yonglihe has quit IRC | 18:00 | |
| cburgess_ | dansmith Yeah its public so have to maintain a certain level of decorum. | 18:00 |
| *** bnemec has joined #openstack-nova | 18:00 | |
| *** burgerk has quit IRC | 18:00 | |
| *** xek has quit IRC | 18:00 | |
| *** maurosr has quit IRC | 18:00 | |
| *** baffle has joined #openstack-nova | 18:00 | |
| *** whenry has joined #openstack-nova | 18:01 | |
| *** lpetrut has quit IRC | 18:02 | |
| *** pratikma_ has joined #openstack-nova | 18:02 | |
| *** thangp has joined #openstack-nova | 18:04 | |
| *** unicell has joined #openstack-nova | 18:05 | |
| *** sbadia has joined #openstack-nova | 18:05 | |
| *** keekz has joined #openstack-nova | 18:05 | |
| dansmith | alaski: klindgren: had to fix a typo, updated now. | 18:05 |
| *** pratikmallya has quit IRC | 18:05 | |
| *** ihrachys has joined #openstack-nova | 18:05 | |
| *** agireud has joined #openstack-nova | 18:05 | |
| dansmith | klindgren: if you hammer that real good, I'd be willing to at least propose it seriously for kilo as an opt-in band-aid | 18:06 |
| *** maurosr has joined #openstack-nova | 18:06 | |
| *** ctrath has quit IRC | 18:06 | |
| *** armax has quit IRC | 18:06 | |
| *** pkoniszewski has joined #openstack-nova | 18:07 | |
| *** ZZelle has joined #openstack-nova | 18:07 | |
| alaski | dansmith: we're past that point now so I don't have a good place to test anymore | 18:08 |
| dansmith | alaski: oh, even better :) | 18:09 |
| dansmith | alaski: and congrats :) | 18:09 |
| alaski | yep, and thanks :) | 18:09 |
| *** ctrath has joined #openstack-nova | 18:10 | |
| *** _gryf has joined #openstack-nova | 18:11 | |
| *** sacharya has joined #openstack-nova | 18:11 | |
| *** mdrabe has joined #openstack-nova | 18:11 | |
| *** ihrachys has quit IRC | 18:11 | |
| cburgess_ | sdague danpb So another question, we currently use utils.sync on ebtables call. If we are using --concurrent should we remove that? | 18:12 |
| *** sacharya_ has quit IRC | 18:14 | |
| *** ihrachys has joined #openstack-nova | 18:14 | |
| *** neiljerram_bb has quit IRC | 18:14 | |
| *** tdurakov has quit IRC | 18:16 | |
| mriedem | 246013 is at the top of the gate queue | 18:16 |
| *** vilobhmm has joined #openstack-nova | 18:17 | |
| danpb | cburgess_: yeah, iirc there's a comment in code saying it should be removed when using --concurrent | 18:17 |
| mriedem | dansmith: bluebox was just moving to kilo i think, so jlk might be interested | 18:17 |
| *** otter768 has joined #openstack-nova | 18:17 | |
| * mriedem jumps to -operators | 18:17 | |
| *** rdopiera has quit IRC | 18:18 | |
| *** tdurakov has joined #openstack-nova | 18:18 | |
| dansmith | mriedem: merged now | 18:18 |
| mriedem | sweet | 18:18 |
| *** tonytan4ever has quit IRC | 18:19 | |
| *** achanda has quit IRC | 18:21 | |
| *** lucasagomes is now known as lucas-dinner | 18:21 | |
| *** otter768 has quit IRC | 18:22 | |
| *** ihrachys has quit IRC | 18:25 | |
| *** fawadkhaliq has quit IRC | 18:26 | |
| *** armax has joined #openstack-nova | 18:26 | |
| sdague | finally that nova objects fix merged | 18:27 |
| *** liverpooler has joined #openstack-nova | 18:27 | |
| *** Vek_ has quit IRC | 18:28 | |
| *** baoli has quit IRC | 18:29 | |
| *** chhavi has quit IRC | 18:29 | |
| *** ihrachys has joined #openstack-nova | 18:30 | |
| *** penick has joined #openstack-nova | 18:31 | |
| *** baoli has joined #openstack-nova | 18:32 | |
| *** junjie has joined #openstack-nova | 18:34 | |
| *** kfarr has joined #openstack-nova | 18:35 | |
| vilobhmm | melwitt : ping | 18:36 |
| melwitt | bauzas: what I meant on the test_objects.py backport was, it won't merge until we do something about the cells tests, right? it's not passing py27 and py34 unit tests because of what I commented about | 18:36 |
| vilobhmm | melwitt : why did this didn't merge https://review.openstack.org/#/c/241016/ any idea ? | 18:36 |
| *** tyagiprince has joined #openstack-nova | 18:37 | |
| *** PaulMurray is now known as ptm_away | 18:37 | |
| markus_z | sdague: That was the fix which blocked the gate, right? | 18:37 |
| *** dims has quit IRC | 18:37 | |
| sdague | markus_z: yes, for nova | 18:37 |
| markus_z | sdague: Ah, cool, good to head. | 18:37 |
| *** gjayavel_ has quit IRC | 18:38 | |
| markus_z | sdague: Can I see the start of the issue at http://status.openstack.org/openstack-health/#/project/openstack%252Fnova ? | 18:38 |
| melwitt | vilobhmm: there was a problem blocking the gate yesterday, you can recheck it now | 18:38 |
| *** jichen has joined #openstack-nova | 18:38 | |
| vilobhmm | sure | 18:39 |
| sdague | markus_z: maybe, it was only the nova project affected | 18:39 |
| markus_z | sdague: I'm asking because I still fail to detect gate failures early. | 18:39 |
| sdague | it wasn't a cross project thing that was breaking everyone | 18:39 |
| sdague | also, that's only tempest jobs | 18:39 |
| *** lpetrut has joined #openstack-nova | 18:39 | |
| sdague | it doesn't handle unit tests | 18:39 |
| *** danpb has quit IRC | 18:40 | |
| *** gokrokve has quit IRC | 18:40 | |
| *** browne has joined #openstack-nova | 18:40 | |
| markus_z | oh, you're right | 18:40 |
| markus_z | I thought that could be the single source of truth if something is burning | 18:40 |
| *** dims has joined #openstack-nova | 18:40 | |
| *** junjie has quit IRC | 18:40 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/nova-specs: Virt image properties boot time override via flavor https://review.openstack.org/246548 | 18:40 |
| *** tonytan4ever has joined #openstack-nova | 18:41 | |
| *** mriedem has quit IRC | 18:42 | |
| openstackgerrit | Maxime Leroy proposed openstack/nova: Libvirt: Support fp plug in vhostuser vif https://review.openstack.org/245369 | 18:42 |
| openstackgerrit | Maxime Leroy proposed openstack/nova: Libvirt: Support ovs fp plug in vhostuser vif https://review.openstack.org/237680 | 18:42 |
| samueldmq | is nova microversioned already ? | 18:42 |
| *** ihrachys has quit IRC | 18:42 | |
| *** e0ne has quit IRC | 18:43 | |
| *** ihrachys has joined #openstack-nova | 18:43 | |
| *** suro-patz has joined #openstack-nova | 18:43 | |
| *** jaypipes has quit IRC | 18:43 | |
| *** unicell1 has joined #openstack-nova | 18:44 | |
| *** daemontool has quit IRC | 18:44 | |
| *** mriedem has joined #openstack-nova | 18:44 | |
| *** pratikma_ has quit IRC | 18:45 | |
| cburgess_ | sdague danpb so given that ebtables isn't smart about its locking.. are we still ok with no timeout? Any admin that issues a SIGKILL against ebtables gets what they deserve? | 18:46 |
| *** rlrossit has joined #openstack-nova | 18:46 | |
| *** andreykurilin__ has quit IRC | 18:46 | |
| *** unicell has quit IRC | 18:46 | |
| *** nic has joined #openstack-nova | 18:47 | |
| *** w_verdugo has quit IRC | 18:47 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/nova-specs: Volume support for multi hypervisors https://review.openstack.org/245986 | 18:48 |
| *** sacharya_ has joined #openstack-nova | 18:48 | |
| *** boris-42 has quit IRC | 18:48 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Use o.vo VersionPredicateField https://review.openstack.org/245327 | 18:49 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Removed extra spaces from double line strings https://review.openstack.org/242989 | 18:49 |
| *** EinstCrazy has joined #openstack-nova | 18:49 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Use o.vo FlexibleBooleanField https://review.openstack.org/245336 | 18:49 |
| *** ociuhandu has quit IRC | 18:49 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Use o.vo DictOfListOfStringsField https://review.openstack.org/245344 | 18:50 |
| *** sacharya has quit IRC | 18:51 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Use ContainerFormat instead of strings https://review.openstack.org/245835 | 18:51 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add ContainerFormat field https://review.openstack.org/245834 | 18:51 |
| *** tpeoples has joined #openstack-nova | 18:51 | |
| *** betherly has joined #openstack-nova | 18:52 | |
| *** lpetrut has quit IRC | 18:52 | |
| *** armax has quit IRC | 18:53 | |
| *** lpetrut has joined #openstack-nova | 18:53 | |
| edleafe | samueldmq: yes: http://docs.openstack.org/developer/nova/api_microversions.html | 18:54 |
| *** armax has joined #openstack-nova | 18:55 | |
| samueldmq | edleafe: nice, I was wondering how useful that could be for keystone | 18:55 |
| samueldmq | edleafe: I will take a look at it, thanks for the link | 18:55 |
| *** EinstCrazy has quit IRC | 18:56 | |
| *** gokrokve has joined #openstack-nova | 18:57 | |
| *** pratikmallya has joined #openstack-nova | 18:57 | |
| *** sc68cal_ is now known as sc68cal | 19:00 | |
| *** irenab has quit IRC | 19:00 | |
| *** gokrokve has quit IRC | 19:02 | |
| *** rotbart has quit IRC | 19:03 | |
| *** rushil has quit IRC | 19:04 | |
| *** ociuhandu has joined #openstack-nova | 19:05 | |
| *** penick has quit IRC | 19:06 | |
| edleafe | samueldmq: np | 19:07 |
| *** ihrachys has quit IRC | 19:07 | |
| *** andrewbogott has quit IRC | 19:09 | |
| *** andrewbogott has joined #openstack-nova | 19:09 | |
| *** sacharya_ has quit IRC | 19:11 | |
| markus_z | tonyb: kashyap: bauzas: FYI: http://lists.openstack.org/pipermail/openstack-dev/2015-November/079679.html | 19:11 |
| bauzas | melwitt: heya sorry was afk | 19:12 |
| mriedem | markus_z: fwiw this has come up before, and russellb had ideas on the same kind of thing | 19:12 |
| *** Marga_ has joined #openstack-nova | 19:13 | |
| russellb | i think i even built a job at one point? | 19:13 |
| *** sacharya has joined #openstack-nova | 19:13 | |
| russellb | or maybe i'm crazy | 19:13 |
| mriedem | could be the neutron talking | 19:13 |
| russellb | very well could be. | 19:13 |
| markus_z | :D | 19:13 |
| melwitt | bauzas: np | 19:14 |
| russellb | markus_z: i believe i was messing with having a custom nodepool image like you talk about | 19:14 |
| russellb | but i didn't want us to have to maintain packages | 19:14 |
| russellb | Fedora has a repo with bleeding edge virt components that i used | 19:14 |
| russellb | and then a similar thing was being developed for centos that i was going to switch to | 19:15 |
| russellb | and that's roughly where my memory gets fuzzy and i don't remember what happened to it all | 19:15 |
| *** gokrokve has joined #openstack-nova | 19:15 | |
| russellb | but i'm not working on it :) | 19:15 |
| *** pixelbeat has quit IRC | 19:15 | |
| russellb | https://fedoraproject.org/wiki/Virtualization_Preview_Repository | 19:15 |
| markus_z | russellb: When did that happen? | 19:15 |
| russellb | couple years ago | 19:16 |
| *** pushkaru has joined #openstack-nova | 19:16 | |
| *** pumaranikar has quit IRC | 19:16 | |
| *** ihrachys has joined #openstack-nova | 19:16 | |
| russellb | https://review.openstack.org/#/c/132976/ | 19:17 |
| *** rushil has joined #openstack-nova | 19:18 | |
| markus_z | russellb: Hm, wanna do it again? ;P | 19:18 |
| *** kgalanov has joined #openstack-nova | 19:18 | |
| *** jdurgin1 has quit IRC | 19:18 | |
| mriedem | sdague: would you agree with the granularity comment here https://review.openstack.org/#/c/240401/ ? | 19:18 |
| mriedem | i.e. lumping too many changes into a single microversion | 19:18 |
| russellb | markus_z: virt-preview for fedora is still around ... there was something being built for CentOS but i can't find status this minute | 19:19 |
| *** paul-carlton has left #openstack-nova | 19:19 | |
| *** tdurakov has quit IRC | 19:19 | |
| russellb | markus_z: my only advice is to try to piggy back on one of the efforts for maintaining packages like that, seems like less work | 19:19 |
| *** gokrokve has quit IRC | 19:20 | |
| markus_z | russellb: yeah, it makes sense to re-use the efforts from others. Thanks for your pointers. | 19:20 |
| *** ildikov has joined #openstack-nova | 19:20 | |
| markus_z | I have to go home now, see you folks tomorrow | 19:20 |
| russellb | np, good luck | 19:20 |
| *** markus_z has quit IRC | 19:22 | |
| *** signed8__ has joined #openstack-nova | 19:23 | |
| *** ekarlso has joined #openstack-nova | 19:24 | |
| *** signed8bit has quit IRC | 19:25 | |
| *** Piet has joined #openstack-nova | 19:26 | |
| *** ctrath has quit IRC | 19:26 | |
| *** jaypipes has joined #openstack-nova | 19:27 | |
| *** e0ne has joined #openstack-nova | 19:27 | |
| *** ctrath has joined #openstack-nova | 19:29 | |
| *** RichardRaseley has joined #openstack-nova | 19:30 | |
| *** tpeoples_ has joined #openstack-nova | 19:30 | |
| bauzas | melwitt: sorry, again afk, okay I see your point, my bad | 19:31 |
| *** tyagiprince has quit IRC | 19:33 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add service status notification https://review.openstack.org/245678 | 19:33 |
| *** josecastroleon has joined #openstack-nova | 19:34 | |
| *** electrocucaracha has quit IRC | 19:34 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/nova-specs: Virt image properties boot time override via flavor https://review.openstack.org/246548 | 19:35 |
| *** greghaynes has joined #openstack-nova | 19:36 | |
| *** junjie has joined #openstack-nova | 19:36 | |
| *** ihrachys has quit IRC | 19:36 | |
| *** yonglihe has joined #openstack-nova | 19:36 | |
| openstackgerrit | ShaoHe Feng proposed openstack/nova: HyperV: do not log twice with different level. https://review.openstack.org/246269 | 19:37 |
| *** josecastroleon has quit IRC | 19:38 | |
| *** klkumar has quit IRC | 19:38 | |
| *** auggy has joined #openstack-nova | 19:41 | |
| *** guilh has joined #openstack-nova | 19:41 | |
| *** sneti_ has quit IRC | 19:42 | |
| *** w_verdugo has joined #openstack-nova | 19:42 | |
| guilh | hi | 19:43 |
| *** sneti has joined #openstack-nova | 19:43 | |
| guilh | Hello all, | 19:43 |
| guilh | I have create an aggregate and flavor and I also create a new cinder volume-type called 'fast-iscsi'. I can create new volume from that volume-type without problem, I can also attach that volume to any VM, and it work. | 19:43 |
| guilh | I still have a lot of trouble to just set the default volume-type to my flavor called Client.fast-iSCSI. | 19:43 |
| guilh | Does anyone know how to dedicated a volume-type to a flavor? | 19:43 |
| guilh | Regards, | 19:43 |
| guilh | Guillaume HARVEY | 19:43 |
| *** aginwala has joined #openstack-nova | 19:43 | |
| *** ihrachys has joined #openstack-nova | 19:43 | |
| guilh | Hello all, I have create an aggregate and flavor and I also create a new cinder volume-type called 'fast-iscsi'. I can create new volume from that volume-type without problem, I can also attach that volume to any VM, and it work. I still have a lot of trouble to just set the default volume-type to my flavor called Client.fast-iSCSI. Does anyone know how to dedicated a volume-type to a flavor? Regards, Guillaume HARVEY | 19:45 |
| *** jamielennox|away is now known as jamielennox | 19:45 | |
| RichardRaseley | guilh: This is a development channel. Please use #openstack for support issues. | 19:46 |
| guilh | oh thx | 19:46 |
| *** penick has joined #openstack-nova | 19:46 | |
| *** exploreshaifali has joined #openstack-nova | 19:47 | |
| *** angdraug has joined #openstack-nova | 19:48 | |
| *** salv-orlando has quit IRC | 19:48 | |
| *** ccarmack has quit IRC | 19:49 | |
| *** ccarmack has joined #openstack-nova | 19:49 | |
| *** e0ne has quit IRC | 19:50 | |
| *** eglynn has quit IRC | 19:51 | |
| *** ihrachys has quit IRC | 19:51 | |
| *** jamielennox is now known as jamielennox|away | 19:52 | |
| *** baoli has quit IRC | 19:53 | |
| *** stackdump has quit IRC | 19:54 | |
| *** sacharya has quit IRC | 19:55 | |
| *** doug-fis_ has joined #openstack-nova | 19:57 | |
| *** armax has quit IRC | 19:57 | |
| *** ed___ has joined #openstack-nova | 19:58 | |
| *** ed___ is now known as edleafe_ | 19:59 | |
| *** eliqiao has quit IRC | 19:59 | |
| gibi_ | nova notification subteam meeting starting now on openstack-meeting-alt | 20:00 |
| openstackgerrit | Chet Burgess proposed openstack/nova: Use --concurrent with ebtables https://review.openstack.org/246580 | 20:00 |
| openstackgerrit | Chet Burgess proposed openstack/nova: ebtables/libvirt workaround https://review.openstack.org/246581 | 20:00 |
| *** aginwala has quit IRC | 20:00 | |
| *** doug-fish has quit IRC | 20:00 | |
| cburgess_ | sdague Reviews for --concurrent posted | 20:01 |
| *** doug-fis_ has quit IRC | 20:01 | |
| *** eliqiao has joined #openstack-nova | 20:01 | |
| cburgess_ | Heading to lunch back in a bit | 20:01 |
| sdague | cburgess_: thanks! | 20:01 |
| *** doug-fish has joined #openstack-nova | 20:01 | |
| *** ccesario has quit IRC | 20:01 | |
| sdague | I will review later today, TC meeting now | 20:01 |
| cburgess_ | sdague First review is proper fix, 2nd one is using your workaround. I need to ammend the commit to indicate you wrote that. my bad. | 20:01 |
| sdague | cburgess_: meh, I don't care that much | 20:01 |
| sdague | unless you are just trying to spread the blame :) | 20:02 |
| *** artom_ has quit IRC | 20:02 | |
| cburgess_ | sdague BINGO! | 20:02 |
| openstackgerrit | Chet Burgess proposed openstack/nova: ebtables/libvirt workaround https://review.openstack.org/246581 | 20:02 |
| cburgess_ | sdague Plus I need a reason why I'm commiting something with the HP copyright. | 20:03 |
| sdague | heh | 20:03 |
| *** baoli has joined #openstack-nova | 20:03 | |
| *** pratikmallya has quit IRC | 20:05 | |
| openstackgerrit | Mark McLoughlin proposed openstack/nova: servicegroup: remove the zookeeper driver https://review.openstack.org/246343 | 20:06 |
| *** claudiub has joined #openstack-nova | 20:07 | |
| sdague | cburgess_: I think there is an issue with the first patch, comment left | 20:07 |
| cburgess_ | sdague reading | 20:08 |
| cburgess_ | sdague Good catch | 20:08 |
| *** jamielennox|away is now known as jamielennox | 20:09 | |
| openstackgerrit | Chet Burgess proposed openstack/nova: Use --concurrent with ebtables https://review.openstack.org/246580 | 20:10 |
| cburgess_ | sdague fixed | 20:10 |
| *** aginwala has joined #openstack-nova | 20:10 | |
| *** tdurakov has joined #openstack-nova | 20:10 | |
| openstackgerrit | Chet Burgess proposed openstack/nova: ebtables/libvirt workaround https://review.openstack.org/246581 | 20:11 |
| sdague | oh, I'm going to hit you for one more administrative issue, we should probably add a release note that nova requires ebtables >= 2... (whatever it is) | 20:11 |
| *** Marga_ has quit IRC | 20:11 | |
| sdague | http://docs.openstack.org/developer/reno/usage.html#creating-new-release-notes | 20:11 |
| sdague | that should be part of that patch, then I'm +2 | 20:11 |
| cburgess_ | sdague OK thats cool. I'll fix that after lunch. | 20:11 |
| sdague | cburgess_: great, thanks again | 20:11 |
| *** doug-fish has quit IRC | 20:12 | |
| cburgess_ | sdague So just to be clear, I update the commit message to indicate we now require ebtables blah blah blah and then I follow this release note process you linked me? | 20:12 |
| *** ijw has joined #openstack-nova | 20:12 | |
| cburgess_ | Or this process is part of the same commit? | 20:12 |
| *** doug-fish has joined #openstack-nova | 20:12 | |
| sdague | it's part of the same commit | 20:12 |
| sdague | it's a tool that builds release notes into the tree | 20:12 |
| cburgess_ | sdague OK I'll give it a shot. Thanks. | 20:12 |
| *** penick has quit IRC | 20:14 | |
| *** sacharya has joined #openstack-nova | 20:16 | |
| *** Marga_ has joined #openstack-nova | 20:17 | |
| *** otter768 has joined #openstack-nova | 20:18 | |
| bauzas | sdague: I'm tempted to respin the stable reno change with a different Change-Id so we could merge the CI job patch | 20:20 |
| bauzas | sdague: because for the moment, stable/liberty is wedged | 20:20 |
| bauzas | due to an ovo issue that can't be fixed easily as melwitt spotted in https://review.openstack.org/#/c/246022/1 | 20:21 |
| bauzas | sdague: thoughts on that? | 20:21 |
| sdague | oh, the ovo thing wreaked havoc there as well? | 20:21 |
| dansmith | bauzas: it's not an ovo issue | 20:21 |
| dansmith | bauzas: the ovo fix is blocked on a cells issue | 20:21 |
| bauzas | dansmith: sorry I was unclear | 20:21 |
| dansmith | sdague: just a backport of the other fix | 20:22 |
| bauzas | dansmith: the backport fix for the ovo release is -1'd by Jenkins because of a cells job issue, right (to be clear) | 20:22 |
| dansmith | yes | 20:22 |
| *** pixelbeat has joined #openstack-nova | 20:23 | |
| *** otter768 has quit IRC | 20:23 | |
| sdague | bauzas: on cells unit tests actually | 20:23 |
| melwitt | I'm not sure it's cells-specific in that somehow with the backport it makes a remote call of objects.InstanceV1.get_by_uuid() return an instance 2.0 | 20:23 |
| bauzas | correct, not a cells job, dammit me | 20:23 |
| dansmith | melwitt: I thought I tried it on base stable and it failed there too | 20:24 |
| melwitt | dansmith: by base stable do you mean without the backport? | 20:25 |
| dansmith | yeah | 20:25 |
| sdague | http://logs.openstack.org/22/246022/1/check/gate-nova-python27/45d1d61/console.html#_2015-11-16_23_20_19_491 hmmmm it says it's returning a V1 object there, though what was the thing it was returning before | 20:25 |
| sdague | which actually has a real repr | 20:25 |
| *** rook has quit IRC | 20:25 | |
| melwitt | dansmith: without the backport on base stable with o.vo 0.13.0 I was getting the recursion depth exceeded | 20:25 |
| dansmith | melwitt: yeah, I forced the package | 20:25 |
| bauzas | melwitt: dansmith: that was my original thought, I don't get how the change can be related to the problem - even if it's a bit close | 20:26 |
| dansmith | but let me re-do that and see if I did what I thought I did | 20:26 |
| *** aginwala has quit IRC | 20:26 | |
| bauzas | I can play too | 20:27 |
| dansmith | hmm, nope, | 20:27 |
| dansmith | it works without the backport | 20:27 |
| dansmith | I must've not done the right thing when I was rushing last night | 20:27 |
| *** doug-fish has quit IRC | 20:27 | |
| *** doug-fish has joined #openstack-nova | 20:28 | |
| dansmith | so, let me work on that | 20:28 |
| *** sacharya_ has joined #openstack-nova | 20:28 | |
| melwitt | dansmith: yeah, sorry I don't understand. you mean it works without the backport with o.vo 0.13.0 by some modification you did? | 20:29 |
| ansiwen | mriedem: I addressed your comments, if you wanna have a look: https://review.openstack.org/#/c/243616/ | 20:29 |
| dansmith | I mean it must be related to the backport | 20:29 |
| bauzas | dansmith: okay, so I'm holding off my possible respin of the reno stable backport with a different change-id | 20:29 |
| dansmith | it's weird that it's just cells tests tho | 20:30 |
| dansmith | wait a sec | 20:30 |
| *** sacharya has quit IRC | 20:31 | |
| *** apoorvad has quit IRC | 20:31 | |
| melwitt | sdague: that's the expected call, wanting InstanceV1 which it used to get before the backport | 20:32 |
| sdague | I thought + meant it was what it got and - is what it expected? | 20:32 |
| sdague | or do I have that totally backwards | 20:32 |
| sdague | god I had mox | 20:32 |
| sdague | hate | 20:32 |
| *** rfolco has quit IRC | 20:33 | |
| melwitt | it says unexpected:- expected:+ but I guess that depends on what order you put them which is why people have done patches to change order of assertEquals for example? | 20:33 |
| melwitt | I put a bunch of prints in the test and verified it gets Instance 2.0 after the get_by_uuid call where objects.Instance has been mocked as objects.InstanceV1 | 20:34 |
| *** edtubill has joined #openstack-nova | 20:35 | |
| melwitt | I just don't understand how that's happening. don't know how the version makes its way from the class to the remote side, and then sent back | 20:35 |
| *** baoli has quit IRC | 20:36 | |
| dansmith | I expect it's just a race to setup the test mocks | 20:37 |
| dansmith | I'm looking at it | 20:37 |
| *** apoorvad has joined #openstack-nova | 20:37 | |
| *** pratikmallya has joined #openstack-nova | 20:37 | |
| melwitt | okay | 20:38 |
| *** Vek_ has joined #openstack-nova | 20:38 | |
| *** greghaynes has quit IRC | 20:38 | |
| *** greghaynes_ is now known as greghaynes | 20:38 | |
| *** jichen has quit IRC | 20:38 | |
| *** pratikmallya has quit IRC | 20:39 | |
| *** pratikmallya has joined #openstack-nova | 20:39 | |
| dansmith | yeah | 20:40 |
| dansmith | lemme run this a couple more times | 20:41 |
| *** doug-fish has quit IRC | 20:41 | |
| dansmith | this cells save test is kinda hariy | 20:41 |
| dansmith | also hairy | 20:41 |
| *** doug-fish has joined #openstack-nova | 20:42 | |
| dansmith | updated, let's see if that looks better | 20:43 |
| cburgess_ | sdague Which section should this be in? prelude? | 20:44 |
| *** e0ne has joined #openstack-nova | 20:44 | |
| bauzas | dansmith: IIUC, that call is not first checked https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_instance.py#L330-L331 | 20:45 |
| bauzas | dansmith: because of the mock call you make in the IndirectionAPIFixture | 20:45 |
| bauzas | dansmith: when calling https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_instance.py#L329 | 20:45 |
| sdague | cburgess_: I'd actually put it in the upgrade section I think | 20:46 |
| cburgess_ | sdague Cool | 20:46 |
| sdague | because it's a new prereq before you deploy this code | 20:46 |
| bauzas | and that's also the same problem with https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_instance.py#L324-L327 | 20:46 |
| *** doug-fish has quit IRC | 20:46 | |
| wznoinsk | dims: are you around? | 20:46 |
| sdague | and I'll leave it as an exercise for you about figuring out what the actual min required version is here | 20:46 |
| bauzas | dansmith: but I could be wrong | 20:46 |
| dims | wznoinsk pong | 20:46 |
| *** e0ne has quit IRC | 20:47 | |
| wznoinsk | dims: hi, a (possibly) quick question for you | 20:47 |
| dims | wznoinsk sure | 20:47 |
| dansmith | bauzas: what call? | 20:47 |
| wznoinsk | nova-docker, was it not getting any traction or had some fundamental limitation and didn't evolve? | 20:48 |
| sdague | wznoinsk: it was broken all the time, no one cared enough to keep it working for more than a day | 20:48 |
| wznoinsk | I heard about lack of interest for sure | 20:48 |
| sdague | or to address some of the limitations | 20:49 |
| dims | wznoinsk : some people are using it. i keep it alive when i get a chance especially around release boundaries. | 20:49 |
| wznoinsk | sdague: sure, but was lack of interest cause by some major (design) issues ? | 20:49 |
| wznoinsk | s/cause/caused | 20:49 |
| bauzas | dansmith: for example wiht https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_instance.py#L362 | 20:49 |
| bauzas | dansmith: it goes to https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_instance.py#L329-L331 | 20:49 |
| dims | wznoinsk : if you see launchpad for nova-docker you can see the tempest tests yet to be fixed and some other work needed | 20:49 |
| bauzas | dansmith: then you have http://logs.openstack.org/22/246022/1/check/gate-nova-python27/45d1d61/console.html#_2015-11-16_23_20_19_307 | 20:50 |
| dansmith | bauzas: ... I'm not sure what your point is | 20:50 |
| *** tonytan4ever has quit IRC | 20:50 | |
| wznoinsk | sdague, dims: I'm trying to understand what was the reason it got dropped... without spending couple of days trying to figure out there were some major flows in how it was designed/implemented to work | 20:50 |
| *** rmart04 has joined #openstack-nova | 20:50 | |
| *** salv-orlando has joined #openstack-nova | 20:51 | |
| dims | wznoinsk : folks switched to magnum as it uses docker swarm, kubernetes etc | 20:51 |
| dansmith | wznoinsk: it's not a good fit inside nova | 20:51 |
| bauzas | dansmith: sorry... if you see the above log, that's because https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_instance.py#L332-L333 is not called IIUC | 20:51 |
| wznoinsk | if it's lack of use cases I hope I may have one but if it's fundamental flow of some sort that made people saying 'nah... that won't work' then it's a bigger problem | 20:51 |
| dansmith | wznoinsk: and like sdague said, it was broken all the time and never getting fixed | 20:51 |
| *** salv-orlando has quit IRC | 20:51 | |
| wznoinsk | dims: sure, slightly different use case when using magnum tho | 20:51 |
| *** rmart04 has left #openstack-nova | 20:52 | |
| *** salv-orlando has joined #openstack-nova | 20:52 | |
| dims | wznoinsk : one blocker was mounting of cinder volumes last i checked needed some more work on docker side too | 20:52 |
| dansmith | bauzas: it's called, just with different than expected arguments | 20:52 |
| dansmith | bauzas: did you see my change? | 20:52 |
| dims | wznoinsk : right | 20:52 |
| wznoinsk | dansmith: I can understand not a good fit inside nova where 'broken all the time/nobody interested' is probably the result of that | 20:52 |
| dansmith | wznoinsk: I don't think they were related tbh | 20:53 |
| dansmith | wznoinsk: the broken all the time was just a matter of nobody giving it attention for the things that it could do (which was a small subset) | 20:53 |
| bauzas | dansmith: https://review.openstack.org/#/c/246022/1/nova/tests/unit/objects/test_objects.py,cm ? sure | 20:53 |
| dims | wznoinsk : it worked fine for a specific set of scenarios for some folks. then they all went away | 20:53 |
| dansmith | bauzas: this: https://review.openstack.org/#/c/246022/1..2/nova/tests/unit/objects/test_instance.py,cm | 20:54 |
| *** penick has joined #openstack-nova | 20:54 | |
| wznoinsk | dansmith: re fitting nova, does that mean supporting different features/request (the ones for containers) than nova was designed for originally? | 20:54 |
| dansmith | wznoinsk: didn't provide things that nova generally considers table stakes, and nova doesn't expose the things that makes docker useful for most people | 20:55 |
| dansmith | wznoinsk: thus, the intersection between the two is pretty small | 20:55 |
| wznoinsk | or because nova scheduling is good for VMs but not necessarily for quick spinning up containers? (religion discussion alert) | 20:55 |
| melwitt | wznoinsk: fwiw, I thought I remember dims reached out on the ML to ask if anyone was interested in bringing the feature back and it was discussed at the nova midcycle before last, and we couldn't find anyone interested in the driver | 20:55 |
| bauzas | dansmith: okay, fair enough... | 20:56 |
| wznoinsk | dansmith: ok, understood | 20:56 |
| * bauzas hides | 20:56 | |
| wznoinsk | melwitt: I'm interested in that only recently, don't feel guilty :-) | 20:56 |
| dims | wznoinsk : we were looking for help implementing all the mandatory features for a nova driver | 20:56 |
| wznoinsk | s/, /, I | 20:56 |
| *** mari0jv is now known as mariojv | 20:57 | |
| *** ccesario has joined #openstack-nova | 20:57 | |
| dims | like melwitt mentioned, folks did not show up | 20:57 |
| wznoinsk | dims: I would need to look deeper on what exactly we can do there still using nova api, I'm thinking having docker containers as compute nodes may have it's valid use cases but only if you can make use of their full potential | 20:58 |
| dansmith | well, you can't and you never will, if that helps :) | 20:59 |
| dansmith | wznoinsk: and you mean s/nodes/instances/ right? | 20:59 |
| wznoinsk | dansmith: that's my impression after your point few lines back ;-) | 20:59 |
| dansmith | because nova-docker was not about running compute nodes in docker containers | 20:59 |
| dansmith | wznoinsk: I don't suffer from subtlety :) | 21:00 |
| *** dims_ has joined #openstack-nova | 21:00 | |
| openstackgerrit | Chet Burgess proposed openstack/nova: Use --concurrent with ebtables https://review.openstack.org/246580 | 21:00 |
| melwitt | wznoinsk: yeah, nova-docker means it will put a barebones container on a compute node when you request an instance. so you get a container instead of a VM | 21:00 |
| wznoinsk | dansmith: I was using nodes interchangeable with instances because a container to me is closer to a baremetal but I guess that might have been a wrong wording | 21:01 |
| *** ctrath has quit IRC | 21:01 | |
| wznoinsk | dansmith, melwitt: sure I know that, bad wording | 21:02 |
| dansmith | wznoinsk: nodes is not the right term here and comes with a lot of other meanings :) | 21:02 |
| melwitt | well, "compute node" anyway | 21:02 |
| *** aginwala has joined #openstack-nova | 21:02 | |
| dansmith | well, nodes too | 21:02 |
| *** dims has quit IRC | 21:02 | |
| dansmith | but anyway :) | 21:02 |
| melwitt | okay :P | 21:03 |
| cburgess_ | sdague So... I think maybe I did the release notes thing correct, please take a look at let me know | 21:03 |
| wznoinsk | I'd consider 'node' more general term - 'a node' of any kind, where compute instance is of some specific type like instatiation of nova compute | 21:03 |
| sdague | cburgess_: yeh, I think that's right | 21:03 |
| sdague | except for the libvirt version you wrote :) | 21:03 |
| dansmith | melwitt: https://github.com/openstack/nova/blob/master/nova/db/api.py#L481 | 21:03 |
| cburgess_ | sdague Whats the version? | 21:03 |
| dansmith | wznoinsk: but it's used in nova for a specific thing | 21:03 |
| cburgess_ | sdague OMG | 21:04 |
| cburgess_ | sdague Thats what I get for doing this while on an annoying conference call /sigh | 21:04 |
| mriedem | so we're brining nova-docker back?! | 21:04 |
| dansmith | wznoinsk: see what you did? | 21:04 |
| * wznoinsk sets eyes down | 21:04 | |
| dansmith | heh | 21:04 |
| openstackgerrit | Chet Burgess proposed openstack/nova: Use --concurrent with ebtables https://review.openstack.org/246580 | 21:05 |
| cburgess_ | sdague fixed /sigh | 21:05 |
| *** pixelbeat has quit IRC | 21:05 | |
| wznoinsk | mriedem: if you ask me, probably (for sure) not | 21:05 |
| melwitt | wznoinsk: I personally see some usefulness there, even if you can't control the container much with nova api after you create it. but I thought we abandoned the idea for lack of interest from people wanting to use it when there is magnum out there. and we do have lxc driver, so maybe that covered the usecase | 21:05 |
| *** doug-fish has joined #openstack-nova | 21:06 | |
| mriedem | dansmith: ctrath and doffm_ want to integrate CIM with nova :) | 21:06 |
| melwitt | dansmith: oh yeah, host + node! | 21:07 |
| mriedem | melwitt: there is also lxd out of tree | 21:07 |
| dansmith | mriedem: come on dude, am I not having a bad enough day already? | 21:07 |
| mriedem | dansmith: heh | 21:07 |
| wznoinsk | melwitt: with support of docker container format you get some use cases... my thinking was slightly further tho... I'm getting my head around lkvm, clear container have unofficial docker support which I wanted to try in Openstack | 21:07 |
| *** ctrath has joined #openstack-nova | 21:07 | |
| mriedem | melwitt: we also have virtuozzo in tree | 21:08 |
| melwitt | mriedem: ah, right | 21:08 |
| mriedem | those are all hidden in the libvirt driver though | 21:08 |
| *** rushil has quit IRC | 21:08 | |
| wznoinsk | so in short while I see use cases for compute (hic!) instances being containers, that's not necessarily native docker containers I'm interested in | 21:08 |
| dansmith | thank goodness | 21:08 |
| *** salv-orlando has quit IRC | 21:08 | |
| sdague | cburgess_: +2 | 21:09 |
| cburgess_ | sdague cool | 21:09 |
| sdague | anyone else for that? | 21:09 |
| openstackgerrit | Chet Burgess proposed openstack/nova: ebtables/libvirt workaround https://review.openstack.org/246581 | 21:09 |
| cburgess_ | sdague Well so there is the ugly workaround too https://review.openstack.org/#/c/246581/ | 21:09 |
| cburgess_ | sdague So we need to figure out what to do with this. | 21:10 |
| sdague | yeh, I'll review in the morning, I need to do an errand | 21:10 |
| cburgess_ | sdague Cool | 21:10 |
| cburgess_ | sdague Once that merges we probably should update your devstack fix to actually directly use the nova stuff. | 21:10 |
| *** salv-orlando has joined #openstack-nova | 21:10 | |
| *** sacharya_ has quit IRC | 21:11 | |
| *** sacharya has joined #openstack-nova | 21:11 | |
| wznoinsk | with that scheduler refactoring and possibly supporting more schedulers, should there be nova api extended too? | 21:12 |
| mriedem | native VMs modeled with CIM...i can see it all now | 21:12 |
| ctrath | mriedem: already working on it | 21:12 |
| *** lpetrut has quit IRC | 21:12 | |
| wznoinsk | sdague, dansmith ^ | 21:13 |
| dansmith | wznoinsk: no | 21:13 |
| mriedem | like....nova api extensions... | 21:13 |
| mriedem | where is jaypipes when you want him | 21:13 |
| mriedem | probably writing nova api extensions | 21:13 |
| wznoinsk | guys, apologies if some parts are not making sense ... I'm not close to nova's and it's code | 21:15 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: DO NOT MERGE https://review.openstack.org/245050 | 21:16 |
| mriedem | you're not by chance a phd student are you? | 21:16 |
| dansmith | lol | 21:16 |
| *** Nakato_ is now known as Nakato | 21:16 | |
| wznoinsk | mriedem: I didn't know I sound that bad | 21:16 |
| mriedem | heh, no offence of course | 21:17 |
| mriedem | there is just usually this pattern of, 'hey, i'm new, have you guys ever thought about doing x? no. oh, well, i'm thinking about doing it for my thesis.' | 21:17 |
| mrodden | thesis-ware? | 21:18 |
| mriedem | wznoinsk: there are some people running nova-docker thingies, but it's really not maintained past probably kilo | 21:18 |
| *** electrocucaracha has joined #openstack-nova | 21:21 | |
| *** smatzek has quit IRC | 21:22 | |
| *** rpodolyaka1 has joined #openstack-nova | 21:23 | |
| *** shaohe_feng has quit IRC | 21:23 | |
| wznoinsk | mriedem: I'm looking it more in terms of provisioning containers (as being between baremetal and VMs) but without all the clutter of magnum, coe | 21:23 |
| wznoinsk | in term of NFVI if you (for some reason) like NFV terms | 21:24 |
| dims_ | mriedem : still works with liberty :) | 21:24 |
| *** vladikr has quit IRC | 21:25 | |
| *** kfarr has quit IRC | 21:25 | |
| dims_ | wznoinsk : there's a nova-docker channel if we want to avoid ruffling feathers :) | 21:26 |
| melwitt | my feathers remain unruffled | 21:27 |
| wznoinsk | dims_: didn't know, I'm not sure I have anything interesting to say/ask as of now tho, I'll catch you when I do | 21:28 |
| mriedem | dims_: it does? | 21:28 |
| mriedem | https://review.openstack.org/#/q/status:open+project:openstack/nova-docker,n,z | 21:29 |
| dims_ | wznoinsk ack | 21:29 |
| jaypipes | mriedem: sorry, what's up? | 21:29 |
| *** aginwala has quit IRC | 21:29 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 21:29 |
| *** eglynn has joined #openstack-nova | 21:29 | |
| mriedem | dims_: yeah i guess it must http://logs.openstack.org/54/245954/1/check/gate-tempest-dsvm-docker/993481b/ | 21:29 |
| mriedem | jaypipes: nothing | 21:29 |
| dims_ | mriedem i cut a branch too https://github.com/openstack/nova-docker/tree/stable/liberty | 21:29 |
| mriedem | dims_: heh, stable/icehouse https://review.openstack.org/#/c/236508/ | 21:30 |
| mriedem | https://review.openstack.org/#/admin/groups/304,members | 21:30 |
| mriedem | time to put that one out to pasture | 21:30 |
| dims_ | y gotta get rid of that one | 21:30 |
| dims_ | mriedem may be if wznoinsk comes back :) | 21:31 |
| wznoinsk | dims_: on nova-docker most likely not, I'm afraid on touching something in nova cores see limitations on ;-) | 21:31 |
| dims_ | wznoinsk was just kidding, no worries | 21:32 |
| *** krtaylor has quit IRC | 21:32 | |
| jroll | "limitations" is a fun word there :) | 21:32 |
| * jroll sees node/host discussion | 21:32 | |
| *** rpodolyaka1 has quit IRC | 21:32 | |
| dansmith | melwitt: where is instance 1.23 requested | 21:33 |
| dansmith | ? | 21:33 |
| melwitt | dansmith: forgive the ignorance that will follow | 21:34 |
| dansmith | melwitt: nope, I must be missing something | 21:34 |
| *** sneti_ has joined #openstack-nova | 21:36 | |
| dansmith | alaski: rs236.mailgun.us is on a blacklist | 21:36 |
| melwitt | dansmith: but I thought if you do objects.InstanceV1.get_by_uuid() where it has VERSION = '1.23', and eventually object_class_action_versions is called to de/serialize, how does object_versions contain 2.0 and give that back | 21:36 |
| *** sneti has quit IRC | 21:36 | |
| dansmith | melwitt: well, first off, objects.InstanceV1 shouldn't be a thing | 21:36 |
| alaski | dansmith: do you know which one? | 21:36 |
| *** tonytan4ever has joined #openstack-nova | 21:37 | |
| melwitt | dansmith: I thought '1.23' would get passed to object_class_action_versions and it would in turn return the right version back | 21:37 |
| dansmith | alaski: http://mxtoolbox.com/SuperTool.aspx?action=blacklist%3ars236.mailgun.us&run=toolpage | 21:37 |
| dansmith | melwitt: nope, when we go to call object_class_action_versions() we'll get the current process's manifest | 21:38 |
| dansmith | melwitt: which will be whatever the highest version is that is supported | 21:38 |
| melwitt | dansmith: okay, so that explains that. so next I wonder, how did it get 1.x back in o.vo 0.12.0? | 21:38 |
| alaski | dansmith: there are a lot more blacklists there than I remember. I'll send out a signal, though they're probably already aware. | 21:39 |
| dansmith | melwitt: well, there was no ovo support for object_class_action_versions() in 0.12.0 | 21:39 |
| dansmith | melwitt: I had this all worked out yesterday, but it'll take me a minute to figure out all the mocking | 21:40 |
| dansmith | alaski: just FYI.. it's delaying a confirmation email for me, which is annoying me.. and I was already annoyed :) | 21:40 |
| *** aysyd has quit IRC | 21:40 | |
| melwitt | dansmith: sorry, didn't want to cause more work. I guess before the object_class_action_versions there was kind of a no-op | 21:40 |
| dansmith | melwitt: I think that what was happening here is that we were falling back to the non-versioned case | 21:41 |
| dansmith | er, the non-manifest case | 21:41 |
| alaski | dansmith: sure. But I still maintain a slight interest in our email. Though I started on OpenStack right after mailgun joined. | 21:41 |
| *** aginwala has joined #openstack-nova | 21:42 | |
| dansmith | alaski: yeah, hence the ping :D | 21:42 |
| *** pushkaru has quit IRC | 21:42 | |
| *** pushkaru has joined #openstack-nova | 21:42 | |
| *** mrkz has quit IRC | 21:43 | |
| *** rpodolyaka1 has joined #openstack-nova | 21:44 | |
| dansmith | melwitt: ah, right, so, | 21:44 |
| dansmith | melwitt: because o.vo wasn't going to support this before liberty, | 21:44 |
| dansmith | melwitt: I made the conductor rpcapi redirect the non-manifest call to the manifest call, first looking up the manifest | 21:45 |
| *** mrkz has joined #openstack-nova | 21:45 | |
| *** krtaylor has joined #openstack-nova | 21:45 | |
| dansmith | melwitt: https://github.com/openstack/nova/blob/master/nova/conductor/rpcapi.py#L226-L233 | 21:45 |
| dansmith | melwitt: so now that 0.13 is out, we're hitting o.vo's fixture for class_action_versions() directly | 21:46 |
| dansmith | which I think means we're looking up the manifest not right at the call time, but earlier | 21:47 |
| *** electrocucaracha has quit IRC | 21:47 | |
| dansmith | before we do the indirection swap | 21:47 |
| *** electrocucaracha has joined #openstack-nova | 21:48 | |
| *** dustins has quit IRC | 21:48 | |
| melwitt | dansmith: and the pre 0.13 manifest had 1.23 as the Instance version and post 0.13 has 2.0, and that's because of when the manifest is looked up? | 21:49 |
| melwitt | or you're saying it's before the objects.Instance gets set to objects.InstanceV1 | 21:49 |
| dansmith | I think it's because of when we generate the manifest | 21:50 |
| *** tongli has quit IRC | 21:50 | |
| melwitt | okay, I'll look at that. for my own curiosity | 21:51 |
| *** mtanino has quit IRC | 21:51 | |
| *** dims_ has quit IRC | 21:51 | |
| *** electrocucaracha has quit IRC | 21:51 | |
| *** rpodolyaka1 has quit IRC | 21:52 | |
| dansmith | melwitt: print out the manifest in conductor's rpc and in the new fixture | 21:53 |
| *** Marga_ has quit IRC | 21:53 | |
| dansmith | melwitt: you think that there's something actually wrong or what? | 21:53 |
| melwitt | dansmith: no, I understand it has to do with how the test runs now. I meant I'm going to look at the manifest generation timing for my own knowledge | 21:56 |
| *** dims has joined #openstack-nova | 21:57 | |
| *** sneti_ has quit IRC | 21:57 | |
| *** thangp has quit IRC | 21:59 | |
| *** w_verdugo has quit IRC | 22:01 | |
| *** penick has quit IRC | 22:01 | |
| *** electrocucaracha has joined #openstack-nova | 22:02 | |
| *** angdraug has quit IRC | 22:04 | |
| *** baoli has joined #openstack-nova | 22:06 | |
| *** rpodolyaka1 has joined #openstack-nova | 22:06 | |
| *** penick has joined #openstack-nova | 22:06 | |
| *** rushil has joined #openstack-nova | 22:06 | |
| *** doffm_ is now known as doffm | 22:06 | |
| *** rushil_ has joined #openstack-nova | 22:07 | |
| *** mtanino has joined #openstack-nova | 22:08 | |
| *** eliqiao has quit IRC | 22:09 | |
| *** eliqiao has joined #openstack-nova | 22:10 | |
| *** sneti has joined #openstack-nova | 22:10 | |
| *** sacharya has quit IRC | 22:11 | |
| *** baoli has quit IRC | 22:11 | |
| *** rushil has quit IRC | 22:11 | |
| *** akshai_ has joined #openstack-nova | 22:12 | |
| *** pkholkin has quit IRC | 22:13 | |
| *** rlrossit has left #openstack-nova | 22:13 | |
| akshai_ | has anyone tried nova network with liberty? | 22:13 |
| *** sacharya has joined #openstack-nova | 22:14 | |
| *** vilobhmm has quit IRC | 22:14 | |
| *** vilobhmm has joined #openstack-nova | 22:14 | |
| *** rpodolyaka1 has quit IRC | 22:15 | |
| *** vilobhmm has quit IRC | 22:15 | |
| *** vilobhmm has joined #openstack-nova | 22:16 | |
| *** pkholkin has joined #openstack-nova | 22:16 | |
| *** rushil_ has quit IRC | 22:16 | |
| *** vilobhmm has quit IRC | 22:17 | |
| *** vilobhmm has joined #openstack-nova | 22:17 | |
| *** thorst has quit IRC | 22:19 | |
| *** otter768 has joined #openstack-nova | 22:19 | |
| *** thorst has joined #openstack-nova | 22:19 | |
| *** rcernin has quit IRC | 22:21 | |
| *** edleafe_ has quit IRC | 22:21 | |
| *** thorst has quit IRC | 22:23 | |
| *** otter768 has quit IRC | 22:24 | |
| *** edleafe_ has joined #openstack-nova | 22:25 | |
| *** browne has quit IRC | 22:27 | |
| *** rushil has joined #openstack-nova | 22:27 | |
| *** lykinsbd has quit IRC | 22:30 | |
| *** stackdump has joined #openstack-nova | 22:30 | |
| *** sneti has quit IRC | 22:31 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: DO NOT MERGE https://review.openstack.org/245050 | 22:33 |
| *** Marga_ has joined #openstack-nova | 22:33 | |
| *** mc_nair has quit IRC | 22:34 | |
| *** tdurakov has quit IRC | 22:35 | |
| *** tdurakov has joined #openstack-nova | 22:36 | |
| mriedem | hmm, so it looks like when we delete instances, we don't delete instance_actions | 22:36 |
| mriedem | and that breaks the archive stuff | 22:36 |
| bauzas | orly? | 22:38 |
| bauzas | will look at that tomorrow | 22:38 |
| *** mlavalle has quit IRC | 22:40 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: Reverse sort tables before archiving https://review.openstack.org/246635 | 22:40 |
| mriedem | bauzas: yeah, found it with ^ | 22:41 |
| mriedem | http://paste.openstack.org/show/479188/ | 22:41 |
| *** mlavalle has joined #openstack-nova | 22:41 | |
| dims | mriedem is this worth merging? https://review.openstack.org/#/c/244827/ | 22:42 |
| mriedem | dims: yeah, the dict_value kwarg was something i moved to o-i from novaclient actually :) | 22:43 |
| mriedem | so we could replace novaclient's print_dict util with the one from o-i | 22:43 |
| mriedem | that's kind of a lost cause now | 22:43 |
| *** ccarmack has quit IRC | 22:44 | |
| mriedem | dims: you know you need to provide the git hashes in the commit message right? :P | 22:44 |
| mriedem | don't let the last o-i sync break all of the pleasure of doing those | 22:44 |
| dims | it's going away dude :) | 22:44 |
| *** apoorvad has quit IRC | 22:44 | |
| *** burgerk has joined #openstack-nova | 22:45 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/python-novaclient: Last sync from oslo-incubator https://review.openstack.org/244827 | 22:45 |
| *** burgerk_ has quit IRC | 22:46 | |
| dims | mriedem luckily i had that handy :) | 22:46 |
| mriedem | awesome | 22:46 |
| mriedem | wait a minute | 22:46 |
| mriedem | that's not the git log of the actual changes in cliutils | 22:46 |
| *** annegentle has quit IRC | 22:47 | |
| dims | mriedem that was the SHA when i ran the sync script | 22:47 |
| *** annegentle has joined #openstack-nova | 22:47 | |
| *** claudiub has quit IRC | 22:47 | |
| *** lucas-dinner is now known as lucas-afk | 22:48 | |
| *** edtubill has quit IRC | 22:48 | |
| openstackgerrit | Miguel Lavalle proposed openstack/nova-specs: Integrate DNS resolution with Neutron using Nova hostname https://review.openstack.org/90150 | 22:49 |
| *** aginwala has quit IRC | 22:49 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient: Last sync from oslo-incubator https://review.openstack.org/244827 | 22:49 |
| mriedem | dims: now +2 | 22:49 |
| dims | mriedem thanks! | 22:50 |
| mriedem | time to head out | 22:50 |
| dims | good night | 22:50 |
| *** liverpooler has quit IRC | 22:53 | |
| *** mriedem has quit IRC | 22:53 | |
| *** thumpba has joined #openstack-nova | 22:54 | |
| *** liverpooler has joined #openstack-nova | 22:54 | |
| *** aginwala has joined #openstack-nova | 22:54 | |
| *** mwagner has quit IRC | 22:55 | |
| *** stackdump has quit IRC | 22:56 | |
| cburgess_ | Is the citrix xenserver CI job voting? | 22:56 |
| cburgess_ | Sorry I meant the XenProject CI job | 22:56 |
| *** stackdump has joined #openstack-nova | 22:57 | |
| *** burgerk has quit IRC | 22:58 | |
| *** liverpooler has quit IRC | 23:00 | |
| *** dims has quit IRC | 23:00 | |
| *** dims has joined #openstack-nova | 23:01 | |
| *** jhesketh has quit IRC | 23:01 | |
| *** mdrabe has quit IRC | 23:01 | |
| *** mhorban__ has joined #openstack-nova | 23:02 | |
| *** vilobhmm has quit IRC | 23:04 | |
| *** vilobhmm has joined #openstack-nova | 23:04 | |
| *** dims_ has joined #openstack-nova | 23:04 | |
| cfriesen__ | anyone left on here? I'm curious why we don't allow resize-to-same-node by default? | 23:04 |
| *** jhesketh has joined #openstack-nova | 23:04 | |
| *** aginwala_ has joined #openstack-nova | 23:05 | |
| *** sacharya_ has joined #openstack-nova | 23:05 | |
| *** sacharya has quit IRC | 23:05 | |
| *** apoorvad has joined #openstack-nova | 23:05 | |
| *** thumpba has quit IRC | 23:07 | |
| *** dims has quit IRC | 23:07 | |
| *** aginwala has quit IRC | 23:08 | |
| *** thumpba has joined #openstack-nova | 23:08 | |
| *** ctrath has quit IRC | 23:08 | |
| *** aginwala_ has quit IRC | 23:09 | |
| *** aginwala has joined #openstack-nova | 23:09 | |
| *** exploreshaifali has quit IRC | 23:10 | |
| *** gokrokve has joined #openstack-nova | 23:11 | |
| *** burt has quit IRC | 23:14 | |
| *** tonytan4ever has quit IRC | 23:14 | |
| *** ctrath has joined #openstack-nova | 23:14 | |
| *** gokrokve has quit IRC | 23:15 | |
| *** pixelbeat has joined #openstack-nova | 23:19 | |
| *** gokrokve has joined #openstack-nova | 23:19 | |
| *** rook has joined #openstack-nova | 23:20 | |
| *** wuhg has joined #openstack-nova | 23:26 | |
| *** salv-orl_ has joined #openstack-nova | 23:28 | |
| *** tdurakov has quit IRC | 23:28 | |
| *** electrocucaracha has quit IRC | 23:28 | |
| melwitt | cfriesen__: my guess is because resize == cold migrate and you don't want it to migrate to the same host it's on, by default | 23:29 |
| *** doug-fish has quit IRC | 23:30 | |
| *** jdurgin has joined #openstack-nova | 23:30 | |
| *** salv-orlando has quit IRC | 23:30 | |
| cfriesen__ | melwitt: why not? | 23:31 |
| *** suro-patz has quit IRC | 23:31 | |
| cfriesen__ | i.e. how is it any worse | 23:31 |
| *** aginwala_ has joined #openstack-nova | 23:31 | |
| *** mwagner has joined #openstack-nova | 23:31 | |
| *** alejandrito has quit IRC | 23:31 | |
| cfriesen__ | it seems like it can even offer opportunities for optimization...like if you're just adding memory/cpus you could skip all the disk stuff | 23:32 |
| *** aginwal__ has joined #openstack-nova | 23:32 | |
| melwitt | cfriesen__: again guessing, because "migrate" implies you want the instance to move, not have it stay in the same place | 23:32 |
| *** ctrath has quit IRC | 23:32 | |
| cfriesen__ | melwitt: but the end-user asked for a resize, not a migration. so the fact that it shares code with cold migration is purely an implementation detail | 23:32 |
| melwitt | cfriesen__: I agree if you're on the "resize" flow it would seem fine to let it stay on the same host | 23:32 |
| *** electrocucaracha has joined #openstack-nova | 23:32 | |
| melwitt | so I'm not sure other than the fact that the share the code path | 23:33 |
| cburgess_ | This issue comes up from time to time. I think the reason at this point is, because its always been that way and changing it would be a problem. | 23:33 |
| cfriesen__ | cburgess: :) | 23:33 |
| cburgess_ | As in no one wants to document/deal with the change. | 23:33 |
| *** mlavalle has quit IRC | 23:34 | |
| melwitt | heh, I hope not. I don't know the history of why resize == cold migrate but decoupling at least some part of it for the initial checks makes sense to me | 23:34 |
| *** thumpba has quit IRC | 23:34 | |
| *** aginwala has quit IRC | 23:34 | |
| *** w_verdugo has joined #openstack-nova | 23:34 | |
| cburgess_ | I could be wrong. | 23:35 |
| cburgess_ | Seems like this issue comes up from time to time and we never change it. | 23:35 |
| cfriesen__ | melwitt: they do share a lot of the code...resize is basically a cold migration to a different flavor (but optionally to the same host) | 23:35 |
| cfriesen__ | melwitt: sort of similar to how evacuate and rebuild share a lot of code | 23:35 |
| *** aginwala_ has quit IRC | 23:35 | |
| dansmith | cfriesen__: resize is not a resize in place | 23:36 |
| dansmith | cfriesen__: it's a two-phase commit, | 23:36 |
| dansmith | which means you do the disk part anyway | 23:36 |
| dansmith | we could do it a little more efficiently if it was on the same box, | 23:36 |
| melwitt | cfriesen__: ah, okay | 23:36 |
| dansmith | but since lots of people pack smalls with smalls and larges with larges, it means we have to consult the scheduler | 23:36 |
| cfriesen__ | dansmith: but we still consult the scheduler with the option set to true | 23:37 |
| dansmith | so it's largely the reason that cburgess_ said, but also because it was designed to be one thing | 23:37 |
| *** mrkz has quit IRC | 23:37 | |
| *** ijuwang has quit IRC | 23:37 | |
| dansmith | cfriesen__: right, I understand, I meant the reason why it's a migrate under the covers | 23:37 |
| dansmith | cfriesen__: the same-host thing was added purely so we could test it in devstack | 23:38 |
| *** EinstCrazy has joined #openstack-nova | 23:39 | |
| cfriesen__ | dansmith: yeah, I got that. I just don't see the downside of allowing it (and even removing the option and setting it permanently to true). Aside from the fact that it's a change, I guess. | 23:39 |
| melwitt | I wonder if it could be as simple as "if user requested resize then treat it as allow same host" for the later check | 23:39 |
| *** signed8__ has quit IRC | 23:42 | |
| dansmith | cfriesen__: I don't see any reason why one default is better than another | 23:43 |
| dansmith | cfriesen__: because as you say, people who assume the current default will have a change anyway | 23:44 |
| *** josecastroleon has joined #openstack-nova | 23:45 | |
| *** mhorban__ has quit IRC | 23:46 | |
| *** jgjhgsonchn88523 has quit IRC | 23:46 | |
| *** mhorban__ has joined #openstack-nova | 23:46 | |
| cfriesen__ | dansmith: I'm actually not seeing why anyone would ever want to prevent a resize to the same host. (Whereas cold migration to same host only makes sense for development) | 23:47 |
| cburgess_ | cfriesen__ As dansmith said, scheduling related reasons might block it. | 23:47 |
| dansmith | there was some other reason a while ago, and I'm blanking on it | 23:47 |
| cfriesen__ | cburgess: then that's fine, it'll go to another host | 23:47 |
| cburgess_ | cfriesen__ Some people setup aggregates for running only a specific flavor and target every flavor to a specific aggregate. | 23:47 |
| cburgess_ | So if you resize you need to move. | 23:48 |
| cfriesen__ | cburgess: which is fine, since we'd still go through the scheduler | 23:48 |
| dansmith | I want to say it also screws with quotas or reservations or something | 23:48 |
| cburgess_ | dansmith At one point it was broken locally. But that was fixed long ago. | 23:48 |
| dansmith | yeah | 23:48 |
| *** EinstCrazy has quit IRC | 23:48 | |
| cburgess_ | dansmith unrelated, is xenproject CI voting? | 23:49 |
| dansmith | cburgess_: running or voting? | 23:49 |
| cburgess_ | dansmith voting? As in do I need to care it gave me a -1? | 23:49 |
| dansmith | cburgess_: no | 23:49 |
| *** salv-orl_ has quit IRC | 23:49 | |
| dansmith | cburgess_: it giving you a -1 means it's voting, but it's not gating | 23:50 |
| dansmith | cburgess_: and also, it's been failing a lot recently, IIRC | 23:50 |
| cburgess_ | dansmith yeah the error looked totally unrelated to me but I didn't know if that -1 would be a bad thing or not. | 23:50 |
| *** salv-orlando has joined #openstack-nova | 23:50 | |
| *** gokrokve has quit IRC | 23:51 | |
| *** ZZelle_ has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!