| *** akshai has quit IRC | 00:03 | |
| *** zhangjn has joined #openstack-nova | 00:04 | |
| *** zhangjn has quit IRC | 00:04 | |
| *** zhangjn has joined #openstack-nova | 00:04 | |
| *** zenoway has joined #openstack-nova | 00:05 | |
| alex_xu | oomich_away: good morning | 00:08 |
|---|---|---|
| *** zenoway has quit IRC | 00:09 | |
| *** thorst has joined #openstack-nova | 00:12 | |
| *** diogogmt has joined #openstack-nova | 00:14 | |
| *** thorst_ has joined #openstack-nova | 00:16 | |
| *** zhangjn has quit IRC | 00:16 | |
| *** zhangjn has joined #openstack-nova | 00:17 | |
| *** thorst has quit IRC | 00:19 | |
| *** ociuhandu has quit IRC | 00:21 | |
| *** Sukhdev has quit IRC | 00:23 | |
| *** zhangjn has quit IRC | 00:25 | |
| *** apoorvad has joined #openstack-nova | 00:27 | |
| *** carey has joined #openstack-nova | 00:28 | |
| *** Daisy has quit IRC | 00:28 | |
| *** zenoway has joined #openstack-nova | 00:29 | |
| *** Jeffrey4l has joined #openstack-nova | 00:31 | |
| *** mmedvede has joined #openstack-nova | 00:31 | |
| *** apoorvad has quit IRC | 00:33 | |
| *** zenoway has quit IRC | 00:33 | |
| *** hemna is now known as hemnafk | 00:35 | |
| *** otter768 has joined #openstack-nova | 00:35 | |
| *** otter768 has quit IRC | 00:40 | |
| *** ssurana has joined #openstack-nova | 00:42 | |
| kashyap | tdurakov: Heya, sorry for the late pong, I was afk all of this afternoon. | 00:42 |
| kashyap | Only got back to the computer now. I don't see a question addressed to me. If you have one please retype here, I'll check back here in the morning (I'm in CET). | 00:43 |
| *** ljxiash has quit IRC | 00:44 | |
| *** nic has quit IRC | 00:45 | |
| *** mriedem has joined #openstack-nova | 00:50 | |
| *** annegentle has quit IRC | 00:51 | |
| *** dimtruck is now known as zz_dimtruck | 00:51 | |
| *** apoorvad has joined #openstack-nova | 00:52 | |
| *** zz_dimtruck is now known as dimtruck | 00:52 | |
| *** rk4n has joined #openstack-nova | 00:52 | |
| *** zhangjn has joined #openstack-nova | 00:53 | |
| auggy | mriedem: thanks for adding me as a reviewer, i'll take a look and see if i can offer anything useful ;) | 00:53 |
| *** zenoway has joined #openstack-nova | 00:53 | |
| auggy | oh wait nm i'm confused | 00:54 |
| * auggy has been staring at bugs too long | 00:55 | |
| *** SourabhP has quit IRC | 00:56 | |
| auggy | you had left comments on a bug i was a reviewer for and i thought it was the same one mentioned here | 00:56 |
| mriedem | i'm also confused | 00:56 |
| *** vishwanathj_ is now known as vishwanathj_away | 00:57 | |
| auggy | you left a comment on a bug i had reviewed or am watching or something referencing this bug and for some reason my brain combined them into one.. so i thought you'd added me as a reviewer and i thought that'll teach me to open my big mouth ;D | 00:58 |
| *** zenoway has quit IRC | 00:58 | |
| auggy | not bug.. review | 00:58 |
| auggy | ack | 00:58 |
| * auggy feels more like a bug huffer than a bug skimmer | 00:58 | |
| *** zhangjn has quit IRC | 00:58 | |
| *** fawadkhaliq has joined #openstack-nova | 01:00 | |
| openstackgerrit | Dimitry Ushakov proposed openstack/nova: (lxc) Updated regex to ignore failing tests https://review.openstack.org/267724 | 01:02 |
| *** zhangjn has joined #openstack-nova | 01:02 | |
| *** ZZelle_ has quit IRC | 01:03 | |
| *** eil397 has quit IRC | 01:05 | |
| *** diogogmt has quit IRC | 01:05 | |
| *** penick has quit IRC | 01:07 | |
| *** EinstCrazy has joined #openstack-nova | 01:07 | |
| *** zenoway has joined #openstack-nova | 01:07 | |
| *** ljxiash has joined #openstack-nova | 01:09 | |
| *** eil397 has joined #openstack-nova | 01:12 | |
| *** zenoway has quit IRC | 01:13 | |
| gmann | mriedem: another issue with server group functional tests is we have 2 set of that. not sure why | 01:14 |
| *** zhangjn has quit IRC | 01:14 | |
| mriedem | gmann: they are different | 01:14 |
| mriedem | one is api samples tests, | 01:14 |
| mriedem | the other is actually running a request through wsgi and w/o mocking the database | 01:14 |
| mriedem | and using the scheduler | 01:15 |
| gmann | mriedem: ok, but if you see 2.15 version tests were added in another one. I think we should add that too in api sample | 01:16 |
| mriedem | ok, i might be missing something, feel free to push a fix | 01:16 |
| mriedem | or add it to mine on the same bug | 01:16 |
| gmann | mriedem: sure | 01:17 |
| *** nelsnelson has quit IRC | 01:17 | |
| *** ankita_w_ has quit IRC | 01:18 | |
| *** zenoway has joined #openstack-nova | 01:21 | |
| *** gangil has quit IRC | 01:22 | |
| *** gangil has joined #openstack-nova | 01:25 | |
| *** gangil has joined #openstack-nova | 01:25 | |
| *** rk4n has quit IRC | 01:27 | |
| *** zenoway has quit IRC | 01:27 | |
| *** ferest has quit IRC | 01:30 | |
| *** rk4n has joined #openstack-nova | 01:30 | |
| *** ferest has joined #openstack-nova | 01:30 | |
| *** gongysh_ has quit IRC | 01:32 | |
| *** rk4n has quit IRC | 01:33 | |
| *** rk4n has joined #openstack-nova | 01:35 | |
| *** zhangjn has joined #openstack-nova | 01:36 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Split-network-plane-for-live-migration https://review.openstack.org/245005 | 01:36 |
| *** haomaiwang has joined #openstack-nova | 01:37 | |
| *** jacky-zhang has joined #openstack-nova | 01:39 | |
| *** ssurana has quit IRC | 01:40 | |
| *** eil397 has quit IRC | 01:41 | |
| *** dimtruck is now known as zz_dimtruck | 01:41 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 01:42 |
| *** zhangjn has quit IRC | 01:43 | |
| *** smatzek has joined #openstack-nova | 01:44 | |
| *** zhangjn has joined #openstack-nova | 01:46 | |
| *** ferest has quit IRC | 01:46 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Libvirt: Move graphics checking and console verify early when LM https://review.openstack.org/265042 | 01:47 |
| openstackgerrit | Jacky_lei_zhang proposed openstack/python-novaclient: Fix support customer ip protocol when create security-rule using nova-cli https://review.openstack.org/266690 | 01:49 |
| *** zhangjn has quit IRC | 01:50 | |
| *** wanghao has joined #openstack-nova | 01:51 | |
| *** zhangjn has joined #openstack-nova | 01:53 | |
| *** rk4n has quit IRC | 01:55 | |
| *** mvensky_ has quit IRC | 01:57 | |
| *** thorst_ has quit IRC | 01:57 | |
| *** thorst has joined #openstack-nova | 01:58 | |
| *** jacky-zhang_ has joined #openstack-nova | 01:59 | |
| *** jacky-zhang has quit IRC | 02:00 | |
| *** jacky-zhang_ is now known as jacky-zhang | 02:00 | |
| *** haomaiwang has quit IRC | 02:01 | |
| *** rk4n has joined #openstack-nova | 02:01 | |
| *** haomaiwang has joined #openstack-nova | 02:01 | |
| *** zhangjn has quit IRC | 02:02 | |
| *** lihg has quit IRC | 02:05 | |
| *** thorst has quit IRC | 02:06 | |
| *** wer_ has joined #openstack-nova | 02:07 | |
| *** zhangjn has joined #openstack-nova | 02:08 | |
| *** nelsnelson has joined #openstack-nova | 02:13 | |
| *** gongysh_ has joined #openstack-nova | 02:13 | |
| *** nelsnelson has quit IRC | 02:13 | |
| *** lihg has joined #openstack-nova | 02:15 | |
| *** zz_dimtruck is now known as dimtruck | 02:17 | |
| *** rk4n has quit IRC | 02:18 | |
| *** suro-patz has quit IRC | 02:19 | |
| openstackgerrit | Yingxin Cheng proposed openstack/nova: Refactor claim code to eliminate duplication https://review.openstack.org/262939 | 02:22 |
| *** houming has joined #openstack-nova | 02:23 | |
| *** rk4n has joined #openstack-nova | 02:23 | |
| *** cfarquhar has quit IRC | 02:23 | |
| *** cfarquhar has joined #openstack-nova | 02:25 | |
| *** cfarquhar has quit IRC | 02:25 | |
| *** cfarquhar has joined #openstack-nova | 02:25 | |
| *** moshele has quit IRC | 02:27 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Replace stubs.Set with stub_out (db) https://review.openstack.org/267897 | 02:28 |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: Servicegroup Foundational Refactoring https://review.openstack.org/235637 | 02:29 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Replace stubs.Set with stub_out (db) https://review.openstack.org/267897 | 02:32 |
| *** dimtruck is now known as zz_dimtruck | 02:32 | |
| *** diogogmt has joined #openstack-nova | 02:33 | |
| *** mriedem has quit IRC | 02:34 | |
| *** lihg has quit IRC | 02:34 | |
| *** otter768 has joined #openstack-nova | 02:36 | |
| *** zhangjn has quit IRC | 02:37 | |
| *** zz_dimtruck is now known as dimtruck | 02:38 | |
| *** otter768 has quit IRC | 02:41 | |
| *** vladikr has quit IRC | 02:42 | |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: Servicegroup Foundational Refactoring https://review.openstack.org/235637 | 02:43 |
| *** gcb has joined #openstack-nova | 02:44 | |
| *** wer_ has quit IRC | 02:46 | |
| *** vilobhmm11 has quit IRC | 02:47 | |
| *** rk4n has quit IRC | 02:48 | |
| *** edmondsw has quit IRC | 02:49 | |
| *** smatzek has quit IRC | 02:50 | |
| *** rk4n has joined #openstack-nova | 02:50 | |
| *** tonytan4ever has quit IRC | 02:53 | |
| *** gangil has quit IRC | 02:53 | |
| *** fawadkhaliq has quit IRC | 02:54 | |
| *** rk4n has quit IRC | 02:59 | |
| *** dimtruck is now known as zz_dimtruck | 02:59 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-nova | 03:01 | |
| *** zz_dimtruck is now known as dimtruck | 03:01 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 03:02 |
| *** thorst has joined #openstack-nova | 03:04 | |
| *** mtanino has quit IRC | 03:04 | |
| *** Marga__ has joined #openstack-nova | 03:07 | |
| *** Marga_ has quit IRC | 03:09 | |
| *** rk4n has joined #openstack-nova | 03:09 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Replace stubs.Set with stub_out (db) https://review.openstack.org/267897 | 03:11 |
| *** unicell1 has quit IRC | 03:11 | |
| *** thorst has quit IRC | 03:11 | |
| *** dims has quit IRC | 03:11 | |
| *** Marga__ has quit IRC | 03:12 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Replace stubs.Set with stub_out (db) https://review.openstack.org/267897 | 03:16 |
| openstackgerrit | Eric Brown proposed openstack/nova: Use of six.PY3 should be forward compatible https://review.openstack.org/267152 | 03:20 |
| *** yamahata has quit IRC | 03:20 | |
| *** Marga_ has joined #openstack-nova | 03:22 | |
| *** smoriya_ has quit IRC | 03:23 | |
| *** cfarquhar has quit IRC | 03:23 | |
| *** gangil has joined #openstack-nova | 03:25 | |
| *** gangil has joined #openstack-nova | 03:25 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Split-network-plane-for-live-migration https://review.openstack.org/245005 | 03:25 |
| *** ferest has joined #openstack-nova | 03:26 | |
| *** apoorvad has quit IRC | 03:27 | |
| *** Marga_ has quit IRC | 03:27 | |
| *** smoriya_ has joined #openstack-nova | 03:28 | |
| *** dimtruck is now known as zz_dimtruck | 03:28 | |
| *** jacky-zhang has quit IRC | 03:30 | |
| *** cfarquhar has joined #openstack-nova | 03:30 | |
| *** cfarquhar has quit IRC | 03:30 | |
| *** cfarquhar has joined #openstack-nova | 03:30 | |
| *** gjayavelu has quit IRC | 03:31 | |
| *** ferest has quit IRC | 03:32 | |
| *** zz_dimtruck is now known as dimtruck | 03:34 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Replace stubs.Set with stub_out (os) https://review.openstack.org/267921 | 03:36 |
| *** lihg has joined #openstack-nova | 03:37 | |
| *** zenoway has joined #openstack-nova | 03:38 | |
| *** zenoway has quit IRC | 03:42 | |
| *** chaochin has quit IRC | 03:43 | |
| *** annegentle has joined #openstack-nova | 03:44 | |
| *** dimtruck is now known as zz_dimtruck | 03:44 | |
| *** gyee has quit IRC | 03:49 | |
| *** links has joined #openstack-nova | 03:49 | |
| *** zz_dimtruck is now known as dimtruck | 03:51 | |
| *** ljxiash has quit IRC | 03:51 | |
| *** mragupat has joined #openstack-nova | 03:55 | |
| *** mragupat has quit IRC | 03:55 | |
| *** mragupat has joined #openstack-nova | 03:56 | |
| *** links has quit IRC | 03:56 | |
| *** ferest has joined #openstack-nova | 03:57 | |
| openstackgerrit | Dimitry Ushakov proposed openstack/nova: (lxc) Updated regex to ignore failing tests https://review.openstack.org/267724 | 03:59 |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwa_ has joined #openstack-nova | 04:01 | |
| *** thorst has joined #openstack-nova | 04:09 | |
| *** salv-orlando has joined #openstack-nova | 04:11 | |
| *** salv-orl_ has quit IRC | 04:13 | |
| *** thorst has quit IRC | 04:16 | |
| *** zhangjn has joined #openstack-nova | 04:17 | |
| *** fawadkhaliq has joined #openstack-nova | 04:20 | |
| *** gjayavelu has joined #openstack-nova | 04:20 | |
| *** takashin has joined #openstack-nova | 04:20 | |
| *** ijw has joined #openstack-nova | 04:25 | |
| *** zhangjn has quit IRC | 04:25 | |
| *** jacky-zhang has joined #openstack-nova | 04:26 | |
| *** apoorvad has joined #openstack-nova | 04:28 | |
| *** coolsvap|away is now known as coolsvap | 04:28 | |
| *** jacky-zhang_ has joined #openstack-nova | 04:31 | |
| *** jacky-zhang has quit IRC | 04:33 | |
| *** jacky-zhang_ is now known as jacky-zhang | 04:33 | |
| *** zhangjn has joined #openstack-nova | 04:35 | |
| *** baoli has quit IRC | 04:37 | |
| *** otter768 has joined #openstack-nova | 04:37 | |
| *** links has joined #openstack-nova | 04:40 | |
| *** carey has quit IRC | 04:41 | |
| *** otter768 has quit IRC | 04:42 | |
| *** apoorvad has quit IRC | 04:42 | |
| *** dimtruck is now known as zz_dimtruck | 04:50 | |
| *** fawadkhaliq has quit IRC | 04:51 | |
| *** ferest has quit IRC | 04:51 | |
| *** ferest has joined #openstack-nova | 04:51 | |
| *** gjayavelu has quit IRC | 04:52 | |
| *** zhangjn has quit IRC | 04:53 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Fix Mock assert_called_once_with() usage https://review.openstack.org/267931 | 04:53 |
| *** haomaiwa_ has quit IRC | 04:53 | |
| *** 77CAAJ1C4 has joined #openstack-nova | 04:54 | |
| *** rk4n has quit IRC | 04:54 | |
| *** annegentle has quit IRC | 04:55 | |
| *** ferest has quit IRC | 04:56 | |
| *** lihg has quit IRC | 04:58 | |
| *** annegentle has joined #openstack-nova | 04:59 | |
| *** jacky-zhang has quit IRC | 05:00 | |
| *** 77CAAJ1C4 has quit IRC | 05:01 | |
| *** haomaiwa_ has joined #openstack-nova | 05:01 | |
| *** annegentle has quit IRC | 05:01 | |
| *** fawadkhaliq has joined #openstack-nova | 05:08 | |
| *** thorst has joined #openstack-nova | 05:15 | |
| *** vivekd has joined #openstack-nova | 05:16 | |
| *** Marga_ has joined #openstack-nova | 05:17 | |
| *** jaosorior has joined #openstack-nova | 05:19 | |
| *** Daisy has joined #openstack-nova | 05:19 | |
| *** thorst has quit IRC | 05:21 | |
| *** Marga_ has quit IRC | 05:22 | |
| *** Daisy has quit IRC | 05:24 | |
| *** agireud has quit IRC | 05:27 | |
| tangchen__ | Hi alex_xu, I have pasted the error here: http://paste.openstack.org/show/483964/ | 05:27 |
| tangchen__ | alex_xu: if I remove sample_dir = 'servers', the error will be there. | 05:28 |
| *** agireud has joined #openstack-nova | 05:28 | |
| *** gcb has quit IRC | 05:30 | |
| *** sirushti has quit IRC | 05:34 | |
| *** claudiub has quit IRC | 05:34 | |
| *** suro-patz has joined #openstack-nova | 05:34 | |
| *** gjayavelu has joined #openstack-nova | 05:34 | |
| *** zhangjn has joined #openstack-nova | 05:35 | |
| *** zhangjn has quit IRC | 05:36 | |
| *** irenab_ has joined #openstack-nova | 05:37 | |
| *** zhangjn has joined #openstack-nova | 05:37 | |
| *** mdnadeem has joined #openstack-nova | 05:39 | |
| *** zhangjn_ has joined #openstack-nova | 05:39 | |
| *** irenab has quit IRC | 05:40 | |
| *** irenab_ is now known as irenab | 05:40 | |
| *** houming has quit IRC | 05:40 | |
| *** zhangjn_ has quit IRC | 05:40 | |
| *** zhangjn_ has joined #openstack-nova | 05:41 | |
| *** gjayavelu has quit IRC | 05:42 | |
| *** gcb has joined #openstack-nova | 05:42 | |
| *** zhangjn_ has quit IRC | 05:42 | |
| *** zhangjn has quit IRC | 05:43 | |
| *** Sukhdev has joined #openstack-nova | 05:43 | |
| *** ijw has quit IRC | 05:43 | |
| *** zhangjn has joined #openstack-nova | 05:43 | |
| *** zz_dimtruck is now known as dimtruck | 05:44 | |
| *** zhangjn has quit IRC | 05:47 | |
| *** zhangjn has joined #openstack-nova | 05:47 | |
| *** gjayavelu has joined #openstack-nova | 05:49 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Replace stubs.Set with stub_out (utils) https://review.openstack.org/267950 | 05:50 |
| gmann | alex_xu: sdague johnthetubaguy : are we going to add 'description' attribute in PUT response too ? https://review.openstack.org/#/c/254950/5 | 05:50 |
| gmann | actually i did not find that explicitly in spec | 05:51 |
| *** ferest has joined #openstack-nova | 05:51 | |
| *** ljxiash has joined #openstack-nova | 05:52 | |
| *** rk4n has joined #openstack-nova | 05:55 | |
| tojuvone | alex_xu: are you there? | 05:56 |
| *** ljxiash has quit IRC | 05:56 | |
| *** dimtruck is now known as zz_dimtruck | 05:58 | |
| *** rk4n has quit IRC | 06:00 | |
| *** vivekd has quit IRC | 06:01 | |
| *** haomaiwa_ has quit IRC | 06:01 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Replace stubs.Set with stub_out (db) https://review.openstack.org/267897 | 06:01 |
| *** haomaiwang has joined #openstack-nova | 06:01 | |
| *** ferest has quit IRC | 06:02 | |
| *** boris-42 has quit IRC | 06:03 | |
| *** nkrinner has joined #openstack-nova | 06:05 | |
| *** ljxiash has joined #openstack-nova | 06:08 | |
| *** gangil has quit IRC | 06:13 | |
| *** agireud has quit IRC | 06:16 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 06:18 |
| *** agireud has joined #openstack-nova | 06:18 | |
| *** thorst has joined #openstack-nova | 06:19 | |
| *** vilobhmm11 has joined #openstack-nova | 06:21 | |
| *** zenoway has joined #openstack-nova | 06:21 | |
| *** carey has joined #openstack-nova | 06:21 | |
| openstackgerrit | Tang Chen proposed openstack/nova: Add a REST API to trigger crash dump in an instance. https://review.openstack.org/202617 | 06:21 |
| *** gangil has joined #openstack-nova | 06:22 | |
| *** gangil has joined #openstack-nova | 06:22 | |
| *** _diana_1 has quit IRC | 06:22 | |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: Servicegroup Foundational Refactoring https://review.openstack.org/235637 | 06:25 |
| *** ankita_wagh has joined #openstack-nova | 06:25 | |
| *** kaisers has joined #openstack-nova | 06:26 | |
| *** zenoway has quit IRC | 06:26 | |
| *** thorst has quit IRC | 06:26 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: enable uefi boot for x86_64 https://review.openstack.org/262930 | 06:35 |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: update feature support matrix documentation https://review.openstack.org/262931 | 06:35 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/265601 | 06:36 |
| *** otter768 has joined #openstack-nova | 06:38 | |
| *** agireud has quit IRC | 06:42 | |
| *** otter768 has quit IRC | 06:43 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: add checking for new image metadata property 'hw_cpu_realtime_mask' https://review.openstack.org/266233 | 06:43 |
| *** ijw has joined #openstack-nova | 06:43 | |
| *** cfriesen has quit IRC | 06:44 | |
| *** ferest has joined #openstack-nova | 06:44 | |
| *** agireud has joined #openstack-nova | 06:44 | |
| *** jaosorior has quit IRC | 06:45 | |
| *** jaosorior has joined #openstack-nova | 06:46 | |
| *** Sukhdev has quit IRC | 06:46 | |
| *** ijw has quit IRC | 06:48 | |
| *** shyama has quit IRC | 06:51 | |
| *** shyama has joined #openstack-nova | 06:51 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Libvirt: Move graphics checking and console verify early when LM https://review.openstack.org/265042 | 06:53 |
| *** shyama has quit IRC | 06:55 | |
| *** jaosorior has quit IRC | 06:55 | |
| *** shyama has joined #openstack-nova | 06:55 | |
| *** rk4n has joined #openstack-nova | 06:57 | |
| *** teju has joined #openstack-nova | 06:58 | |
| *** zenoway has joined #openstack-nova | 07:00 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** 17WABJ0T0 has joined #openstack-nova | 07:01 | |
| *** rk4n has quit IRC | 07:03 | |
| *** claudiub has joined #openstack-nova | 07:04 | |
| *** mjura has joined #openstack-nova | 07:06 | |
| *** agireud has quit IRC | 07:08 | |
| *** twinklechawla has joined #openstack-nova | 07:08 | |
| *** twinklechawla has left #openstack-nova | 07:09 | |
| *** agireud has joined #openstack-nova | 07:10 | |
| *** jaosorior has joined #openstack-nova | 07:11 | |
| *** scheuran has joined #openstack-nova | 07:11 | |
| *** scheuran has quit IRC | 07:12 | |
| *** shyama_ has joined #openstack-nova | 07:13 | |
| *** scheuran has joined #openstack-nova | 07:14 | |
| *** shyama has quit IRC | 07:15 | |
| *** shyama_ is now known as shyama | 07:15 | |
| *** paul-carlton2 has joined #openstack-nova | 07:15 | |
| *** links has quit IRC | 07:17 | |
| *** shyama has quit IRC | 07:17 | |
| openstackgerrit | Tomi Juvonen proposed openstack/nova: Add object and database support for host_status API https://review.openstack.org/267062 | 07:17 |
| *** shyama has joined #openstack-nova | 07:17 | |
| *** scheuran has quit IRC | 07:18 | |
| *** scheuran has joined #openstack-nova | 07:18 | |
| *** vivekd has joined #openstack-nova | 07:19 | |
| *** Marga_ has joined #openstack-nova | 07:20 | |
| *** thorst has joined #openstack-nova | 07:24 | |
| *** Marga_ has quit IRC | 07:25 | |
| *** coolsvap is now known as coolsvap|away | 07:25 | |
| *** fawadkhaliq has quit IRC | 07:29 | |
| *** lpetrut has joined #openstack-nova | 07:29 | |
| *** fawadkhaliq has joined #openstack-nova | 07:29 | |
| *** mpavone has joined #openstack-nova | 07:29 | |
| *** links has joined #openstack-nova | 07:30 | |
| *** thorst has quit IRC | 07:32 | |
| *** sahid has joined #openstack-nova | 07:33 | |
| *** claudiub has quit IRC | 07:35 | |
| *** ociuhandu has joined #openstack-nova | 07:35 | |
| *** yamahata has joined #openstack-nova | 07:36 | |
| *** links has quit IRC | 07:39 | |
| *** vivekd has quit IRC | 07:41 | |
| *** links has joined #openstack-nova | 07:41 | |
| *** ankita_wagh has quit IRC | 07:43 | |
| *** ijw has joined #openstack-nova | 07:44 | |
| *** vivekd has joined #openstack-nova | 07:45 | |
| *** salv-orlando has quit IRC | 07:47 | |
| *** salv-orlando has joined #openstack-nova | 07:48 | |
| *** ijw has quit IRC | 07:49 | |
| *** ferest has quit IRC | 07:52 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: xenapi: simplify swap_xapi_host() https://review.openstack.org/266258 | 07:53 |
| *** suro-patz has quit IRC | 07:54 | |
| *** moshele has joined #openstack-nova | 07:54 | |
| *** femnad has joined #openstack-nova | 07:55 | |
| *** mragupat has quit IRC | 07:58 | |
| *** takashin has quit IRC | 07:58 | |
| *** SourabhP has joined #openstack-nova | 07:59 | |
| *** rk4n has joined #openstack-nova | 07:59 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Ceph for live-migration job https://review.openstack.org/252283 | 07:59 |
| *** otter768 has joined #openstack-nova | 08:00 | |
| *** 17WABJ0T0 has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** vilobhmm11 has quit IRC | 08:01 | |
| *** otter768 has quit IRC | 08:02 | |
| *** otter768_ has joined #openstack-nova | 08:02 | |
| *** SourabhP_ has joined #openstack-nova | 08:03 | |
| openstackgerrit | Tomi Juvonen proposed openstack/nova: Add object and database support for host_status API https://review.openstack.org/267062 | 08:03 |
| *** otter768_ has quit IRC | 08:04 | |
| *** SourabhP has quit IRC | 08:04 | |
| *** SourabhP_ is now known as SourabhP | 08:04 | |
| *** otter768 has joined #openstack-nova | 08:04 | |
| *** rk4n has quit IRC | 08:04 | |
| *** tdurakov_ has joined #openstack-nova | 08:04 | |
| tdurakov_ | hello everyone | 08:05 |
| *** otter768 has quit IRC | 08:05 | |
| *** rdopiera has joined #openstack-nova | 08:06 | |
| *** otter768 has joined #openstack-nova | 08:06 | |
| *** otter768 has quit IRC | 08:07 | |
| *** otter768 has joined #openstack-nova | 08:08 | |
| *** fawadkhaliq has quit IRC | 08:08 | |
| *** fawadkhaliq has joined #openstack-nova | 08:09 | |
| *** otter768 has quit IRC | 08:09 | |
| *** ljxiash has quit IRC | 08:09 | |
| *** otter768 has joined #openstack-nova | 08:10 | |
| *** paul-carlton2 has quit IRC | 08:10 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove and deprecate conductor provider_fw_rule_get_all() https://review.openstack.org/216241 | 08:11 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove unused provider firewall rules functionality in nova https://review.openstack.org/184027 | 08:11 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove provider_fw_rules tables and associated db api methods https://review.openstack.org/216242 | 08:11 |
| *** ljxiash has joined #openstack-nova | 08:11 | |
| *** otter768_ has joined #openstack-nova | 08:11 | |
| *** ociuhandu has quit IRC | 08:13 | |
| *** ljxiash has quit IRC | 08:13 | |
| *** ljxiash_ has joined #openstack-nova | 08:13 | |
| *** otter76__ has joined #openstack-nova | 08:13 | |
| *** otter768 has quit IRC | 08:14 | |
| *** ociuhandu has joined #openstack-nova | 08:15 | |
| *** otter768 has joined #openstack-nova | 08:15 | |
| *** otter76__ has quit IRC | 08:15 | |
| *** jichen has joined #openstack-nova | 08:16 | |
| *** otter768_ has quit IRC | 08:16 | |
| *** vivekd has quit IRC | 08:16 | |
| *** otter768 has quit IRC | 08:17 | |
| *** otter768 has joined #openstack-nova | 08:17 | |
| *** zhangjn has quit IRC | 08:19 | |
| openstackgerrit | Tomi Juvonen proposed openstack/nova-specs: Get valid server state https://review.openstack.org/260314 | 08:19 |
| *** otter768 has quit IRC | 08:19 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/266160 | 08:19 |
| *** otter768_ has joined #openstack-nova | 08:19 | |
| *** zakora has joined #openstack-nova | 08:20 | |
| *** fawadkhaliq has quit IRC | 08:20 | |
| *** fawadkhaliq has joined #openstack-nova | 08:21 | |
| *** yamahata has quit IRC | 08:21 | |
| *** otter768_ has quit IRC | 08:21 | |
| *** otter768 has joined #openstack-nova | 08:21 | |
| *** otter768 has quit IRC | 08:23 | |
| *** otter768 has joined #openstack-nova | 08:23 | |
| *** paul-carlton2 has joined #openstack-nova | 08:23 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 08:24 |
| *** otter768_ has joined #openstack-nova | 08:25 | |
| *** ihrachys has joined #openstack-nova | 08:26 | |
| *** ferest has joined #openstack-nova | 08:26 | |
| *** otter76__ has joined #openstack-nova | 08:27 | |
| *** lpetrut has quit IRC | 08:27 | |
| *** otter768_ has quit IRC | 08:27 | |
| *** garyk has quit IRC | 08:27 | |
| *** otter768 has quit IRC | 08:27 | |
| *** gjayavelu has quit IRC | 08:28 | |
| *** gjayavelu has joined #openstack-nova | 08:28 | |
| *** otter768 has joined #openstack-nova | 08:29 | |
| *** otter76__ has quit IRC | 08:29 | |
| *** otter768_ has joined #openstack-nova | 08:31 | |
| *** otter768 has quit IRC | 08:31 | |
| *** thorst has joined #openstack-nova | 08:31 | |
| *** houming has joined #openstack-nova | 08:32 | |
| *** ljxiash has joined #openstack-nova | 08:32 | |
| *** ljxiash has quit IRC | 08:32 | |
| *** gjayavelu has quit IRC | 08:32 | |
| *** otter768 has joined #openstack-nova | 08:33 | |
| *** otter768_ has quit IRC | 08:33 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Libvirt: Move graphics checking and console verify early when LM https://review.openstack.org/265042 | 08:33 |
| *** achanda has joined #openstack-nova | 08:34 | |
| *** otter768_ has joined #openstack-nova | 08:35 | |
| *** ljxiash has joined #openstack-nova | 08:35 | |
| tdurakov_ | eliqiao, hi, are you around? | 08:36 |
| *** thorst has quit IRC | 08:36 | |
| eliqiao | hi tdurakov_ | 08:36 |
| *** ljxiash_ has quit IRC | 08:36 | |
| *** daemontool has joined #openstack-nova | 08:36 | |
| tdurakov_ | I've refactored hook, as you suggest, could you review it: https://review.openstack.org/#/c/252283/ | 08:37 |
| tdurakov_ | eliqiao, ^ | 08:37 |
| eliqiao | tdurakov_: sure, thanks, I will take a look at it. | 08:37 |
| *** otter768 has quit IRC | 08:37 | |
| *** otter768_ has quit IRC | 08:39 | |
| *** ljxiash has quit IRC | 08:40 | |
| *** ljxiash_ has joined #openstack-nova | 08:40 | |
| *** jlanoux has joined #openstack-nova | 08:40 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Handle SetAdminPasswdNotSupported raised by libvirt driver https://review.openstack.org/252850 | 08:41 |
| *** matrohon has joined #openstack-nova | 08:41 | |
| eliqiao | tdurakov_: I see currently all live migration testing are tested serially, any plan to explict them, so that we can do it concurrently? | 08:41 |
| *** yamahata has joined #openstack-nova | 08:42 | |
| tdurakov_ | eliqiao, different configuration you mean? | 08:42 |
| bauzas | good morning Nova | 08:44 |
| eliqiao | evening Nova ! haha.. | 08:45 |
| *** ijw has joined #openstack-nova | 08:45 | |
| eliqiao | tdurakov_: I ment, current, we did `block migration test`, after that reconfigure compute node , to do nfs live migration test, then after that , do ceph testing... | 08:46 |
| tdurakov_ | eliqiao, it's not possible to test different storages at the same time | 08:46 |
| eliqiao | tdurakov_: can we do them in seperate job ? | 08:46 |
| eliqiao | tdurakov_: yeah, we can start another job. | 08:46 |
| tdurakov_ | eliqiao, hm, we can, but why we need this? setting up env for testing takes more time, then changing configuration | 08:47 |
| *** gjayavelu has joined #openstack-nova | 08:48 | |
| tdurakov_ | so instead of saving time we will start spending compute resources | 08:48 |
| openstackgerrit | Moshe Levi proposed openstack/os-vif: Remove raise NotImplementedError from abstractmethods https://review.openstack.org/268010 | 08:49 |
| eliqiao | tdurakov_: okay, I am fine with this if time is not a problem to us. | 08:49 |
| *** gjayavelu has quit IRC | 08:50 | |
| eliqiao | tdurakov_: BTW, can you please tell me how can I see the test results after 'check experimental' ? | 08:51 |
| *** ijw has quit IRC | 08:52 | |
| tdurakov_ | eliqiao, well, good question - currently changes in nova/tests directory triggers small amount of jobs to be run, so as a workaround I've made change in nova functional code. So the last patchiest here would not trigger this job. To address this I've submitted patch to project config: https://review.openstack.org/#/c/267669/1 | 08:54 |
| *** gmann has quit IRC | 08:54 | |
| tobasco | johnthetubaguy: can we have a quick review on https://review.openstack.org/#/c/200870/ we have revived a old patch which solves the issue however some people before had concerns | 08:55 |
| *** salv-orlando has quit IRC | 08:55 | |
| tdurakov_ | eliqiao, for now please check results for patchiest 83 instead | 08:55 |
| *** salv-orlando has joined #openstack-nova | 08:56 | |
| *** Marga_ has joined #openstack-nova | 08:56 | |
| eliqiao | tdurakov_: gate-tempest-dsvm-multinode-live-migration SUCCESS in 44m 56s (non-voting) | 08:57 |
| *** Marga_ has quit IRC | 08:57 | |
| *** Marga_ has joined #openstack-nova | 08:57 | |
| eliqiao | tdurakov_: is this correct ? | 08:57 |
| tdurakov_ | eliqiao, yes, it's correct | 08:58 |
| tdurakov_ | p.s. diff between ps-83 and 84 is removing stub change in functional code and adding 1 todo in hook | 08:59 |
| *** haomaiwang has quit IRC | 09:01 | |
| eliqiao | tdurakov_: thanks. checking console logs.. | 09:01 |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| *** ferest has quit IRC | 09:03 | |
| *** paul-carlton2 has quit IRC | 09:04 | |
| *** zhangjn has joined #openstack-nova | 09:04 | |
| *** belmoreira has joined #openstack-nova | 09:05 | |
| openstackgerrit | Abhishek Kekane proposed openstack/nova: Python3: Replace dict.iteritems with six.iteritems https://review.openstack.org/268018 | 09:06 |
| *** fawadkhaliq has quit IRC | 09:09 | |
| *** fawadkhaliq has joined #openstack-nova | 09:09 | |
| *** atuvenie has joined #openstack-nova | 09:11 | |
| *** nkrinner has quit IRC | 09:14 | |
| *** nkrinner has joined #openstack-nova | 09:14 | |
| *** moshele has quit IRC | 09:15 | |
| *** chhavi has joined #openstack-nova | 09:15 | |
| *** SourabhP has quit IRC | 09:16 | |
| openstackgerrit | xiexs proposed openstack/nova-specs: Baremetal migration https://review.openstack.org/261812 | 09:19 |
| *** danpb has joined #openstack-nova | 09:21 | |
| *** Marga_ has quit IRC | 09:23 | |
| *** Marga_ has joined #openstack-nova | 09:23 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 09:23 |
| *** yassine_ has joined #openstack-nova | 09:24 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: virtuozzo instance resize support https://review.openstack.org/182257 | 09:24 |
| *** paul-carlton2 has joined #openstack-nova | 09:24 | |
| openstackgerrit | xiexs proposed openstack/nova-specs: Baremetal migration https://review.openstack.org/261812 | 09:25 |
| *** sfinucan has joined #openstack-nova | 09:25 | |
| *** houming has quit IRC | 09:26 | |
| *** jistr has joined #openstack-nova | 09:27 | |
| *** ljxiash_ has quit IRC | 09:30 | |
| johnthetubaguy | bauzas: gibi: I think we are missing elease notes for this: https://blueprints.launchpad.net/nova/+spec/soft-affinity-for-server-group or did I just miss them? | 09:32 |
| gibi | johnthetubaguy: you are right, it is missing. I will add one now | 09:34 |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: virtuozzo instance rescue mode support https://review.openstack.org/217679 | 09:34 |
| johnthetubaguy | gibi: awesome, thank you! | 09:34 |
| *** zhangjn has quit IRC | 09:34 | |
| *** thorst has joined #openstack-nova | 09:34 | |
| tobasco | johnthetubaguy: see my comment https://bugs.launchpad.net/nova/+bug/1469179/comments/8 regarding your review | 09:36 |
| openstack | Launchpad bug 1469179 in OpenStack Compute (nova) "instance.root_gb should be 0 for volume-backed instances" [Medium,In progress] - Assigned to Tobias Urdin (tobias-urdin) | 09:36 |
| tobasco | johnthetubaguy: and see the answer from alaski | 09:36 |
| tobasco | would it still be a valid solution? i'm unsure | 09:36 |
| bauzas | johnthetubaguy: hell yeah | 09:36 |
| *** teju has quit IRC | 09:36 | |
| bauzas | good catch | 09:36 |
| johnthetubaguy | tobasco: I agree, it only affects the scheduler, but the current patch keeps the scheduler using the old flavor value, I thought? | 09:37 |
| *** ljxiash has joined #openstack-nova | 09:39 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'ec2_instance' and 'instance_fault' https://review.openstack.org/261979 | 09:39 |
| *** achanda has quit IRC | 09:39 | |
| *** thorst has quit IRC | 09:42 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: add ploop support into qemu-img info https://review.openstack.org/260636 | 09:42 |
| tobasco | johnthetubaguy: just confirmed it by dumping the request spec built by build_request_spec and it indeed pushes the root_gb with the volume size for the instance_type and flavor | 09:43 |
| *** ljxiash has quit IRC | 09:43 | |
| *** shyama has quit IRC | 09:43 | |
| tobasco | johnthetubaguy: so you are correct, what would be the best way forward, this means we need to modify the scheduler to fix this? or that really depends on what value the DiskFilter is using for filtering | 09:44 |
| *** gongysh_ has quit IRC | 09:45 | |
| *** efoley_ has joined #openstack-nova | 09:45 | |
| johnthetubaguy | tobasco: I added comments on the patch about where I think needs to be changed | 09:46 |
| johnthetubaguy | tobasco: its the values that get passed to the scheduler that need to be changed | 09:47 |
| *** shihanzhang has joined #openstack-nova | 09:47 | |
| *** ijw has joined #openstack-nova | 09:47 | |
| *** liusheng has quit IRC | 09:48 | |
| tobasco | johnthetubaguy: checking the diskfilter it reads it using "spec_obj.root_gb". The root_gb variable in the request spec is 0 as it should, it's just the flavor and instance_type variable that is having the wrong data for their respective root_gb variables | 09:48 |
| *** derekh has joined #openstack-nova | 09:48 | |
| *** liusheng has joined #openstack-nova | 09:49 | |
| johnthetubaguy | tobasco: maybe thats fine then | 09:49 |
| johnthetubaguy | tobasco: would be good to make sure there is a unit test for that | 09:49 |
| dedery_ | johnthetubaguy: hi, i have issued a new patch-set according to the comments from you and mriedem. It passed all CI and ready for review at https://review.openstack.org/#/c/252565/9 | 09:49 |
| tobasco | johnthetubaguy: i think the diskfilter should pass with that, it's however if we want to make sure the data is the same for the other variables aswell but the diskfilter does however not seem to make any decisions based on it | 09:50 |
| *** vivekd has joined #openstack-nova | 09:51 | |
| *** salv-orlando has quit IRC | 09:52 | |
| *** salv-orlando has joined #openstack-nova | 09:52 | |
| *** ijw has quit IRC | 09:52 | |
| tobasco | johnthetubaguy: i tested to boot an instance with a bigger root volume than i have in total of hypervisor local storage and it worked, with the diskfilter enabled | 09:53 |
| dedery_ | johnthetubaguy: thank you for the review :) | 09:54 |
| ZZelle | johnthetubaguy, hi | 09:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.firewall' options https://review.openstack.org/257995 | 09:55 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.netutils' options https://review.openstack.org/258009 | 09:55 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt.images' conf options documentation https://review.openstack.org/259553 | 09:55 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.imagecache' options https://review.openstack.org/258984 | 09:55 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.images' options https://review.openstack.org/258004 | 09:55 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.disk' options https://review.openstack.org/258912 | 09:55 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.driver' options https://review.openstack.org/257956 | 09:55 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt.driver' conf options documentation https://review.openstack.org/266364 | 09:55 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt.firewall' conf options documentation https://review.openstack.org/266365 | 09:55 |
| *** coolsvap|away is now known as coolsvap | 09:56 | |
| *** atuvenie has quit IRC | 09:59 | |
| *** claudiub has joined #openstack-nova | 09:59 | |
| *** achanda has joined #openstack-nova | 10:00 | |
| *** haomaiwang has quit IRC | 10:01 | |
| johnthetubaguy | ZZelle: hi | 10:01 |
| *** lpetrut has joined #openstack-nova | 10:01 | |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| *** ptm_away is now known as PaulMurray | 10:02 | |
| PaulMurray | Good morning everybody | 10:02 |
| *** sudipto has joined #openstack-nova | 10:02 | |
| kashyap | tdurakov_: Hi, yesterday afternoon pinged, I was not around. You had a question for me? | 10:02 |
| ZZelle | johnthetubaguy, could you relook at https://review.openstack.org/186716 ? It has been proposed during christmas time | 10:03 |
| ZZelle | johnthetubaguy, and i am not sure i fully understood one remark in https://review.openstack.org/#/c/186716/9/nova/virt/vmwareapi/images.py | 10:03 |
| tdurakov_ | kashyap, hi, yes, could we chat in 40 minutes? I'll ping you | 10:03 |
| kashyap | tdurakov_: Yep, I'd be around. | 10:04 |
| *** EinstCrazy has quit IRC | 10:08 | |
| *** gongysh_ has joined #openstack-nova | 10:08 | |
| *** e0ne has joined #openstack-nova | 10:08 | |
| *** EinstCrazy has joined #openstack-nova | 10:08 | |
| *** salv-orl_ has joined #openstack-nova | 10:10 | |
| *** ociuhandu has quit IRC | 10:11 | |
| *** gangil has quit IRC | 10:12 | |
| *** ociuhandu has joined #openstack-nova | 10:13 | |
| *** salv-orlando has quit IRC | 10:13 | |
| *** EinstCrazy has quit IRC | 10:14 | |
| johnthetubaguy | ZZelle: how do you get context to be None | 10:14 |
| * ZZelle looking | 10:16 | |
| alex_xu | tojuvone: hi, i'm here | 10:18 |
| openstackgerrit | Dina Belova proposed openstack/nova: Integrate OSprofiler and Nova https://review.openstack.org/254703 | 10:19 |
| openstackgerrit | Alex Xu proposed openstack/nova: Merge os-user-data extension into servers https://review.openstack.org/258235 | 10:21 |
| *** lpetrut1 has joined #openstack-nova | 10:22 | |
| *** ndipanov has joined #openstack-nova | 10:22 | |
| ZZelle | johnthetubaguy, i doubt it's possible to have image_id == None otherwise it means we don't know which image is associated to the instance | 10:22 |
| ZZelle | johnthetubaguy, so download would fail even if we download from glance | 10:23 |
| *** lpetrut has quit IRC | 10:23 | |
| *** lpetrut1 is now known as lpetrut | 10:23 | |
| johnthetubaguy | ZZelle: same for context I think | 10:24 |
| mdbooth | ZZelle: eavesdropping. Which review? | 10:24 |
| johnthetubaguy | ZZelle: unless thats a specific vmware path? | 10:24 |
| mdbooth | nm, you already posted it | 10:24 |
| *** cdent has joined #openstack-nova | 10:25 | |
| alex_xu | tangchen__: thanks, maybe I miss something, let me check again | 10:25 |
| mdbooth | Interestingly, I'm currently tidying up the code which does this in the libvirt driver (hence my ears burning) | 10:26 |
| *** Thelo has joined #openstack-nova | 10:27 | |
| Thelo | /join #openstack | 10:27 |
| ZZelle | johnthetubaguy, it's not specific, in spawn method context != None | 10:27 |
| Thelo | /join #openstack-qa | 10:27 |
| ZZelle | johnthetubaguy, the same for rescue so i will remove the condition | 10:28 |
| johnthetubaguy | ZZelle: cool, certainly my expectation if for context to never be None | 10:28 |
| johnthetubaguy | s/if/is | 10:28 |
| *** lpetrut has quit IRC | 10:31 | |
| *** lpetrut has joined #openstack-nova | 10:32 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add SerialConsoleOps class https://review.openstack.org/184040 | 10:32 |
| pkholkin | mdbooth, johnthetubaguy: Hello guys! I want to inform you that we created all patches to blueprint bp/new-oslodb-enginefacade, they all are green, waiting for reviews from cores!) | 10:32 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Nova serial console access support https://review.openstack.org/145004 | 10:32 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add serial console handler class https://review.openstack.org/184039 | 10:32 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: objects: Added serial port properties to ImageMetaProps https://review.openstack.org/261035 | 10:32 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add serial console proxy https://review.openstack.org/184038 | 10:32 |
| pkholkin | please ask me and snikitin for any questions | 10:32 |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Adding device_present field to bdm table https://review.openstack.org/268053 | 10:33 |
| pkholkin | our understanding right now is that everything is done according to previous comments and issues | 10:34 |
| *** garyk has joined #openstack-nova | 10:35 | |
| *** snikitin has joined #openstack-nova | 10:35 | |
| *** otter768 has joined #openstack-nova | 10:35 | |
| *** ildikov has quit IRC | 10:37 | |
| ndipanov | johnthetubaguy, are we thinking of merging this one? https://review.openstack.org/#/c/230919/2 | 10:37 |
| ndipanov | I want to approve https://review.openstack.org/#/c/252565/9 | 10:38 |
| *** smoriya_ has quit IRC | 10:39 | |
| *** Thelo has quit IRC | 10:40 | |
| *** otter768 has quit IRC | 10:40 | |
| *** thorst has joined #openstack-nova | 10:40 | |
| *** Thelo has joined #openstack-nova | 10:41 | |
| openstackgerrit | gongysh proposed openstack/nova: Fix the duplicate nics for VMs https://review.openstack.org/252824 | 10:42 |
| johnthetubaguy | ndipanov: I am a bit torn about the first one, the second one seems safer, but I can't really put my finger on why | 10:43 |
| *** sudipto has quit IRC | 10:44 | |
| *** rqw has joined #openstack-nova | 10:44 | |
| openstackgerrit | Ryan Moore proposed openstack/nova: Add better help to rdp options https://review.openstack.org/253556 | 10:44 |
| openstackgerrit | Ryan Moore proposed openstack/nova: config options: centralize section: "rdp" https://review.openstack.org/252389 | 10:44 |
| *** thorst has quit IRC | 10:47 | |
| *** ihrachys_ has joined #openstack-nova | 10:47 | |
| *** sudipto has joined #openstack-nova | 10:47 | |
| *** gongysh_ has quit IRC | 10:47 | |
| *** gongysh has quit IRC | 10:48 | |
| *** sileht has quit IRC | 10:48 | |
| *** sileht has joined #openstack-nova | 10:48 | |
| rqw | danpb: could you help recheck the uefi support patch https://review.openstack.org/#/c/262930/ ? according to alex_xu's comments, I updated the patch and added some test cases. So the previous "+2" was removed. Thanks. | 10:48 |
| ndipanov | johnthetubaguy, my take on the first one is that get_instance_nw_info, if called with no ports, should not add ports | 10:49 |
| ndipanov | otherwise the code is even tricker to reason about :( | 10:49 |
| ndipanov | (and it already is quite mind bending) | 10:49 |
| *** ijw has joined #openstack-nova | 10:49 | |
| *** agireud has quit IRC | 10:49 | |
| *** ihrachys has quit IRC | 10:49 | |
| *** agireud has joined #openstack-nova | 10:51 | |
| *** ijw has quit IRC | 10:53 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Adds relase notes for soft affinity feature https://review.openstack.org/268069 | 10:54 |
| *** wznoinsk has quit IRC | 10:55 | |
| gibi | johnthetubaguy, bauzas: I got distracted but finally here is the soft-affintiy releasenotes ^^ | 10:55 |
| bauzas | coolness | 10:56 |
| bauzas | gibi: meh a nit | 10:56 |
| *** paul-carlton1 has joined #openstack-nova | 10:56 | |
| gibi | bauzas: no problem, shoot | 10:56 |
| gibi | bauzas: ahh what you found is a common typo from me :) | 10:57 |
| gibi | fixing... | 10:57 |
| bauzas | gibi: just a sec | 10:57 |
| bauzas | see http://docs.openstack.org/developer/nova/code-review.html#when-a-release-note-is-needed | 10:57 |
| bauzas | gibi: here you need a reno file because you update a contractual API | 10:57 |
| bauzas | gibi: so I think you should explain what the API impact is | 10:57 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Adds relase notes for soft affinity feature https://review.openstack.org/268069 | 10:57 |
| *** Marga_ has quit IRC | 10:58 | |
| bauzas | gibi: not really providing the API update, just telling that the API resource provides 2 new policies | 10:58 |
| *** Marga_ has joined #openstack-nova | 10:58 | |
| gibi | bauzas: ok, I will rephrase | 10:58 |
| bauzas | gibi: left a comment more in detail | 11:00 |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-nova | 11:01 | |
| *** gjayavelu has joined #openstack-nova | 11:01 | |
| tdurakov_ | kashyap, hey | 11:02 |
| *** atuvenie has joined #openstack-nova | 11:02 | |
| kashyap | tdurakov_: Hi | 11:03 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Adds relase notes for soft affinity feature https://review.openstack.org/268069 | 11:03 |
| tdurakov_ | found failures of live-migration on upstream ci | 11:03 |
| tdurakov_ | http://logs.openstack.org/83/252283/79/experimental/gate-tempest-dsvm-multinode-live-migration/070dcc0/ | 11:03 |
| tdurakov_ | kashyap, from libvirt logs on target: http://xsnippet.org/361328/ | 11:04 |
| *** gszasz has joined #openstack-nova | 11:04 | |
| gibi | bauzas: I fixed | 11:04 |
| kashyap | tdurakov_: Looking | 11:05 |
| bauzas | gibi: fair enough, you're maybe now too much detailing (no need to provide the HTTP method or the full path) but that's okay to me | 11:06 |
| *** rk4n has joined #openstack-nova | 11:06 | |
| *** gjayavelu has quit IRC | 11:06 | |
| tdurakov_ | kashyap, I guess logs would be not enough to find out root cause, what other info could be helpful? | 11:06 |
| kashyap | tdurakov_: What test is it performing? | 11:06 |
| *** rfolco has joined #openstack-nova | 11:06 | |
| tdurakov_ | live-migration of ephemerals on nfs | 11:07 |
| bauzas | johnthetubaguy: https://review.openstack.org/#/c/268069/3 LGTM | 11:07 |
| gibi | bauzas: thanks! | 11:07 |
| *** shinobu-nova has joined #openstack-nova | 11:08 | |
| kashyap | tdurakov_: Looking for other logs from the dir you gave | 11:09 |
| dedery_ | ndipanov: thank you! my next question (probably should be discussed with mriedem) is where should it land in previous releases? would love to hear your thoughts about it. | 11:09 |
| tdurakov_ | kashyap, yep, snippet from this run either | 11:09 |
| kashyap | tdurakov_: Hmm, from here, http://logs.openstack.org/83/252283/79/experimental/gate-tempest-dsvm-multinode-live-migration/070dcc0/logs/libvirt/qemu/instance-00000004.txt.gz | 11:09 |
| *** aysyd has joined #openstack-nova | 11:09 | |
| kashyap | It says, "load of migration failed | 11:10 |
| kashyap | " | 11:10 |
| tdurakov_ | yep, it's emu trace | 11:10 |
| tdurakov_ | s/emu/qemu] | 11:10 |
| *** rk4n has quit IRC | 11:10 | |
| * kashyap is trying to search the versions involved | 11:11 | |
| tdurakov_ | kashyap, http://logs.openstack.org/83/252283/79/experimental/gate-tempest-dsvm-multinode-live-migration/070dcc0/logs/dpkg-l.txt.gz | 11:12 |
| *** yamahata has quit IRC | 11:12 | |
| *** dims has joined #openstack-nova | 11:12 | |
| openstackgerrit | jichenjc proposed openstack/nova: Not using mox stubs in nova.tests.unit.volume https://review.openstack.org/268081 | 11:12 |
| *** lucas-dinner is now known as lucasagomes | 11:12 | |
| kashyap | tdurakov_: Thanks, so it's 2.0.0+dfsg-2ubuntu1.21 | 11:12 |
| kashyap | Whatever that 'dfsg' means | 11:12 |
| gibi | bauzas: if you happen to have time, the version notification patch chain needs a second +2. The chain starts here: https://review.openstack.org/#/c/247024/ | 11:16 |
| bauzas | gibi: yeah, I saw that and I'll look at that today | 11:17 |
| bauzas | I'm very slowly resuming my review duties | 11:17 |
| kashyap | tdurakov_: Can you please point me to the test? | 11:18 |
| gibi | bauzas: thank you! | 11:18 |
| *** sudipto has quit IRC | 11:19 | |
| tdurakov_ | kashyap, sure, it's not block-migration actually, http://logs.openstack.org/83/252283/79/experimental/gate-tempest-dsvm-multinode-live-migration/070dcc0/logs/devstack-gate-post_test_hook.txt.gz#_2016-01-13_22_03_07_768 | 11:19 |
| kashyap | tdurakov_: One more, the log you pointed to me, is that source or destination? | 11:20 |
| tdurakov_ | log in snippet is destination | 11:20 |
| * kashyap looks for source too | 11:21 | |
| tdurakov_ | kashyap, source node libvirt logs: http://logs.openstack.org/83/252283/79/experimental/gate-tempest-dsvm-multinode-live-migration/070dcc0/logs/subnode-2/libvirt/libvirtd.txt.gz#_2016-01-13_22_03_02_389 | 11:21 |
| kashyap | tdurakov_: I wrongly assumed 'subnode-2' implies destinatino | 11:22 |
| kashyap | Thanks for the URL | 11:22 |
| tdurakov_ | kashyap, no, it's not guarantied, one more thing - this issue is floating | 11:23 |
| kashyap | What do you mean "this issue is floating"? | 11:23 |
| tdurakov_ | reproduces from time to time | 11:23 |
| kashyap | tdurakov_: Would it be possible to run QEMU 2.3? It has a lot more debugging in this area, I'm being told | 11:24 |
| tdurakov_ | kashyap, well, it would not be easy | 11:24 |
| kashyap | Which reminds me, Markus(Z) was following up on a Gate job that runs newest Virt packages, not sure what's the status of it | 11:24 |
| *** carey has quit IRC | 11:25 | |
| *** snikitin has quit IRC | 11:26 | |
| tdurakov_ | kashyap, didn't know about job with new packages, will ask him for info as markus_z become online | 11:26 |
| tdurakov_ | kashyap, maybe we could turn-on debug info on current versions? | 11:27 |
| kashyap | tdurakov_: Sure, I thought you already have done so. QEMU devs say modern versions have more debugging messages added. | 11:28 |
| tdurakov_ | kashyap, not yet, didn't change any libvirt/qemu setting for this job | 11:29 |
| kashyap | tdurakov_: On the source, it says: "qemu: terminating on signal 15 from pid 8062" | 11:29 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [python3] byte/string conversions and enable more tests https://review.openstack.org/261045 | 11:29 |
| kashyap | So, 15 means SIGTERM | 11:29 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [python3] Port more code to python3 and enable tests https://review.openstack.org/262083 | 11:29 |
| kashyap | tdurakov_: Okay, in this test, I'd assume both src & dst are of same QEMU versions, right? | 11:30 |
| tdurakov_ | yep | 11:30 |
| *** gongysh has joined #openstack-nova | 11:31 | |
| kashyap | tdurakov_: Do you have an inkling as to what send that SIGTERM from the upper layers libvirt/OpenStack? Have to sift through Compute logs I guess | 11:33 |
| openstackgerrit | Simona Iuliana Toader proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot https://review.openstack.org/209581 | 11:33 |
| kashyap | tdurakov_: Also, if you can reproduce it next time (since you say it's not consistent), a bug would be nice to keep track of. | 11:34 |
| tdurakov_ | kashyap, yep. about SIGTERM, on source node? | 11:35 |
| kashyap | tdurakov_: Yes, having a clue of what triggered it might lead us to root cause | 11:35 |
| tdurakov_ | could it be instance delete after during resources cleanup after test? | 11:35 |
| kashyap | Would be so nice to have a test that I could run locally in a loop :-) | 11:36 |
| johnthetubaguy | kashyap: could you not hack tempest up to do that for you? | 11:36 |
| kashyap | johnthetubaguy: I could try, once I narrowed down the test name from the log files | 11:37 |
| kashyap | tdurakov_: In any case, would be really nice if we can run QEMU 2.3, 2.0 was released about 2 years ago. | 11:39 |
| *** haomaiwang has quit IRC | 11:40 | |
| *** tdurakov_ has quit IRC | 11:42 | |
| *** cdent has quit IRC | 11:42 | |
| *** coolsvap is now known as coolsvap|away | 11:43 | |
| *** tdurakov_ has joined #openstack-nova | 11:43 | |
| tdurakov_ | kashyap, , sorry connection lost( | 11:43 |
| *** efoley_ is now known as efoley | 11:44 | |
| kashyap | tdurakov_: In any case, would be really nice if we can run QEMU 2.3, 2.0 was released about 2 years ago. | 11:44 |
| tdurakov_ | ok, will see what could be done | 11:44 |
| tdurakov_ | johnthetubaguy, do you know any about new package version job? | 11:45 |
| *** thorst has joined #openstack-nova | 11:45 | |
| johnthetubaguy | tdurakov_: not enough to be useful I am afraid | 11:47 |
| tdurakov_ | is Markus on vacation? | 11:47 |
| sfinucan | tdurakov_: Yes, he's back Monday | 11:48 |
| kashyap | tdurakov_: There's a thread on the list that has the current status about it | 11:49 |
| *** ijw has joined #openstack-nova | 11:49 | |
| *** tdurakov_ has quit IRC | 11:49 | |
| kashyap | "Getting a bleeding edge libvirt gate job running" (although, it should include QEMU) | 11:49 |
| *** tdurakov_ has joined #openstack-nova | 11:50 | |
| *** shihanzhang has quit IRC | 11:50 | |
| *** thorst has quit IRC | 11:52 | |
| johnthetubaguy | dansmith: I am marking this as implemented now, that seems correct? https://blueprints.launchpad.net/nova/+spec/service-version-behavior | 11:52 |
| johnthetubaguy | dansmith: but I wonder if we should have some kind of release note added for that effort, now we are enforcing the data migrations to use the service_version | 11:52 |
| johnthetubaguy | alex_xu: do you know if gmann is still working on this? https://review.openstack.org/#/c/243396/ | 11:53 |
| *** ijw has quit IRC | 11:53 | |
| tdurakov_ | sfinucan, acked, thanks | 11:53 |
| tdurakov_ | kashyap, johnthetubaguy I'll file bug for this problem in an 30 minutes, so we could track progress | 11:54 |
| kashyap | tdurakov_: Sure. Please try to include the exact test name, and other related potential reproducer details | 11:55 |
| *** ociuhandu has quit IRC | 11:55 | |
| *** fawadkhaliq has quit IRC | 11:57 | |
| gibi | it seems that with a recent stable/liberty commit the release notes generation started to fail http://logs.openstack.org/69/268069/3/check/gate-nova-releasenotes/aaa7da2/console.html | 11:57 |
| *** fawadkhaliq has joined #openstack-nova | 11:58 | |
| gibi | I'm suspecting b2acc9fa864b6fe10bc0c5f3786b976b472b1b27 | 11:59 |
| alex_xu | johnthetubaguy: I think so, but we didn't give that enough review | 12:01 |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: virtuozzo instance rescue mode support https://review.openstack.org/217679 | 12:01 |
| *** Kennan has quit IRC | 12:04 | |
| *** Kennan has joined #openstack-nova | 12:05 | |
| alex_xu | sdague: johnthetubaguy, after review this api patch https://review.openstack.org/254950, I thought we should enable update 'description' for rebuild, as rebuild allow to change name also. without 'description' change looks like strange. what would you think? | 12:06 |
| *** rk4n has joined #openstack-nova | 12:07 | |
| gibi | bauzas: do you have any idea why reno generation fails after b2acc9fa864b6fe10bc0c5f3786b976b472b1b27? see the logs here http://logs.openstack.org/69/268069/3/check/gate-nova-releasenotes/aaa7da2/console.html | 12:10 |
| *** rk4n has quit IRC | 12:12 | |
| *** smatzek has joined #openstack-nova | 12:13 | |
| bauzas | gibi: oh man | 12:13 |
| *** mdnadeem has quit IRC | 12:15 | |
| *** salv-orl_ has quit IRC | 12:15 | |
| *** salv-orlando has joined #openstack-nova | 12:15 | |
| openstackgerrit | jichenjc proposed openstack/nova: Not using mox stubs in nova.tests.unit.volume https://review.openstack.org/268081 | 12:17 |
| *** cdent has joined #openstack-nova | 12:17 | |
| *** ildikov has joined #openstack-nova | 12:18 | |
| *** baoli has joined #openstack-nova | 12:19 | |
| *** haomaiwa_ has joined #openstack-nova | 12:19 | |
| *** Marga_ has quit IRC | 12:20 | |
| *** ociuhandu has joined #openstack-nova | 12:20 | |
| *** haomaiwa_ has quit IRC | 12:20 | |
| *** agireud has quit IRC | 12:25 | |
| *** agireud has joined #openstack-nova | 12:27 | |
| *** EinstCrazy has joined #openstack-nova | 12:28 | |
| *** achanda has quit IRC | 12:31 | |
| *** haomaiwa_ has joined #openstack-nova | 12:31 | |
| *** cdelatte has joined #openstack-nova | 12:35 | |
| *** otter768 has joined #openstack-nova | 12:36 | |
| *** jwcroppe has joined #openstack-nova | 12:39 | |
| *** thorst has joined #openstack-nova | 12:40 | |
| *** salv-orlando has quit IRC | 12:40 | |
| *** salv-orlando has joined #openstack-nova | 12:40 | |
| *** otter768 has quit IRC | 12:41 | |
| *** efoley has quit IRC | 12:42 | |
| *** chhavi has quit IRC | 12:44 | |
| openstackgerrit | Andrey Pavlov proposed openstack/nova: remove EC2 legacy code https://review.openstack.org/266425 | 12:46 |
| *** kylek3h has joined #openstack-nova | 12:47 | |
| *** kylek3h has quit IRC | 12:48 | |
| *** efoley has joined #openstack-nova | 12:49 | |
| *** ijw has joined #openstack-nova | 12:50 | |
| *** kylek3h has joined #openstack-nova | 12:50 | |
| *** openstackgerrit has quit IRC | 12:50 | |
| *** openstackgerrit has joined #openstack-nova | 12:51 | |
| bauzas | gibi: sorry I had to step away for lunch, will look at the reno issue | 12:52 |
| gibi | bauzas: I tried to understand it but I failed what I know is that before I pulled the origin/stable/liberty the generation was ok in my local repo but after the pull it failed with the same error and the pull only added the above mentioned has to stable/liberty | 12:53 |
| *** kylek3h has quit IRC | 12:53 | |
| gibi | s/has/hash/ | 12:53 |
| *** vladikr has joined #openstack-nova | 12:54 | |
| bauzas | gibi: because reno looks at all the commits in the git repo | 12:54 |
| tojuvone | alex_xu: Hi, for my missing unit test for API, you meant I need to make changes here: test_extended_server_attributes.py? | 12:54 |
| bauzas | gibi: including the branches | 12:54 |
| *** ijw has quit IRC | 12:54 | |
| tojuvone | alex_xu: for https://review.openstack.org/241562/ | 12:55 |
| *** links has quit IRC | 12:55 | |
| tojuvone | alex_xu: Otherwise I kinda have other things ready :) | 12:56 |
| gibi | bauzas: ok I get that but the error message talks about 'announcement' but that diff of the commit I pulled didn't contain the word 'announcement' :) | 12:56 |
| *** links has joined #openstack-nova | 12:58 | |
| *** vladikr has quit IRC | 12:59 | |
| *** haomaiwa_ has quit IRC | 13:01 | |
| *** raildo-afk is now known as raildo | 13:01 | |
| *** haomaiwang has joined #openstack-nova | 13:01 | |
| tobasco | johnthetubaguy: how to move forward with this? https://review.openstack.org/#/c/200870/6 i've been looking into adding unit tests but i dont see where they would fit in, for neither of build_request_spec or diskfilter unit tests | 13:01 |
| *** ccarmack has joined #openstack-nova | 13:03 | |
| *** links has quit IRC | 13:03 | |
| *** baoli_ has joined #openstack-nova | 13:03 | |
| johnthetubaguy | tobasco: hang on, I am a bit confused, root_gb comes from the flavor.root_gb: https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py#L72 | 13:05 |
| *** achanda has joined #openstack-nova | 13:05 | |
| *** ducttape_ has joined #openstack-nova | 13:06 | |
| *** baoli has quit IRC | 13:06 | |
| *** achanda has quit IRC | 13:07 | |
| johnthetubaguy | danpb: are the any updates on the os-vif-lib work at the moment? | 13:08 |
| *** jwcroppe has quit IRC | 13:08 | |
| *** rk4n has joined #openstack-nova | 13:09 | |
| *** vladikr has joined #openstack-nova | 13:11 | |
| *** twinklechawla has joined #openstack-nova | 13:13 | |
| *** twinklechawla has left #openstack-nova | 13:13 | |
| *** rk4n has quit IRC | 13:13 | |
| *** cdent has quit IRC | 13:17 | |
| *** cdent has joined #openstack-nova | 13:18 | |
| *** edleafe is now known as figleaf | 13:18 | |
| sdague | cdent: after a day of grinding on test fixes, I'm taking a new approach. Using the config option to force over the regex to include the words 'fake' and 'openstack', which we can chip off over time | 13:19 |
| cdent | sdague: I guess that makes sense but it kind of makes me sad somehow. | 13:20 |
| * cdent has a complicated set of emotions when it comes to tests | 13:20 | |
| sdague | yeh, it does me as well, but I was about 6 hours into fixing tests, and it looked like at least 100 more hours to get there | 13:21 |
| *** carey has joined #openstack-nova | 13:21 | |
| cdent | ouch | 13:21 |
| cdent | yeah, good plan | 13:21 |
| ildikov | johnthetubaguy: hi. got a minute? | 13:21 |
| tdurakov_ | sdague, hi, could you review: https://review.openstack.org/#/c/267669/ | 13:21 |
| sdague | once I get all this working, I'll push some of my patches that fix things | 13:22 |
| johnthetubaguy | ildikov: how can I help? | 13:22 |
| *** dslevin has quit IRC | 13:22 | |
| johnthetubaguy | sdague: eek, that sounds like a good plan then | 13:22 |
| ildikov | johnthetubaguy: I was wondering to mark the multiattach patches that should go into Mitaka and what is for N | 13:23 |
| *** browne has joined #openstack-nova | 13:23 | |
| ildikov | johnthetubaguy: I just would like to be sure that after all the discussion we're all on the same page | 13:24 |
| openstackgerrit | Cedric Brandily proposed openstack/nova: VMware: Use datastore copy when the image is already in vSphere https://review.openstack.org/186716 | 13:24 |
| *** mnestratov has joined #openstack-nova | 13:24 | |
| tobasco | johnthetubaguy: my bad that req spec is from our test environment running liberty, must get a devstack up on master i've been working on liberty mostly so totally forgot | 13:24 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Add functional tests launcher for py3 env https://review.openstack.org/266895 | 13:25 |
| johnthetubaguy | ildikov: just mark them as Work in progress, I think that should do the trick | 13:26 |
| *** ducttape_ has quit IRC | 13:26 | |
| johnthetubaguy | ildikov: the ones currently for N, that is | 13:26 |
| *** carey has quit IRC | 13:26 | |
| ildikov | johnthetubaguy: I also wonder whether you will have time to check out the first patch and figure out something for the second? | 13:26 |
| *** ducttape_ has joined #openstack-nova | 13:26 | |
| ildikov | johnthetubaguy: ok, I will do it that way | 13:26 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/193135 | 13:29 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Prep Nova to allow multiple volume attachments https://review.openstack.org/193134 | 13:29 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: comp: Allow volume multiattach https://review.openstack.org/267587 | 13:29 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 13:29 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: virt: Pass host when call attach to Cinder https://review.openstack.org/256273 | 13:29 |
| *** fawadkhaliq has quit IRC | 13:30 | |
| *** fawadkhaliq has joined #openstack-nova | 13:30 | |
| *** ducttape_ has quit IRC | 13:31 | |
| *** cdelatte has quit IRC | 13:33 | |
| *** cdelatte has joined #openstack-nova | 13:34 | |
| *** vivekd has quit IRC | 13:36 | |
| *** annegentle has joined #openstack-nova | 13:44 | |
| *** _diana_1 has joined #openstack-nova | 13:46 | |
| *** browne has quit IRC | 13:49 | |
| *** _diana_1 has quit IRC | 13:50 | |
| *** ijw has joined #openstack-nova | 13:50 | |
| *** doug-fish has joined #openstack-nova | 13:53 | |
| *** mdnadeem has joined #openstack-nova | 13:53 | |
| *** dslev has joined #openstack-nova | 13:53 | |
| *** ijw has quit IRC | 13:55 | |
| *** annegentle has quit IRC | 13:57 | |
| *** annegentle has joined #openstack-nova | 13:58 | |
| *** haomaiwang has quit IRC | 14:01 | |
| *** haomaiwang has joined #openstack-nova | 14:01 | |
| *** mdrabe has joined #openstack-nova | 14:04 | |
| *** rlrossit has joined #openstack-nova | 14:04 | |
| *** rqw has quit IRC | 14:04 | |
| *** scheuran_ has joined #openstack-nova | 14:06 | |
| openstackgerrit | dane-fichter proposed openstack/nova: Add image signature verification https://review.openstack.org/189843 | 14:07 |
| *** scheuran has quit IRC | 14:08 | |
| *** thangp has joined #openstack-nova | 14:08 | |
| *** tdurakov_ has quit IRC | 14:08 | |
| *** tdurakov_ has joined #openstack-nova | 14:10 | |
| *** rk4n has joined #openstack-nova | 14:10 | |
| *** ctrath has joined #openstack-nova | 14:11 | |
| *** tdurakov_ has quit IRC | 14:12 | |
| *** pratikmallya has joined #openstack-nova | 14:13 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Make project_id optional in v2.1 urls https://review.openstack.org/233076 | 14:14 |
| sdague | cdent / johnthetubaguy - https://review.openstack.org/#/c/233076/ is the fully collapsed version of this patch, all tests passing locally, the new config option | 14:14 |
| *** tdurakov_ has joined #openstack-nova | 14:15 | |
| sdague | it's got a lot going on in it, but because the test coupling, that's as small as it can be | 14:15 |
| *** claudiub has quit IRC | 14:15 | |
| *** rk4n has quit IRC | 14:16 | |
| *** alaski is now known as lascii | 14:16 | |
| cdent | thanks sdague, will mark it for poking at soon | 14:16 |
| *** gszasz has quit IRC | 14:17 | |
| sdague | I annotated it | 14:20 |
| sdague | honestly, mostly, it's not as bad as I thought it would be | 14:20 |
| *** kylek3h has joined #openstack-nova | 14:20 | |
| *** shyama has joined #openstack-nova | 14:20 | |
| *** paul-carlton2 has quit IRC | 14:21 | |
| *** w_verdugo has quit IRC | 14:22 | |
| *** jichen has quit IRC | 14:22 | |
| *** w_verdugo has joined #openstack-nova | 14:23 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Use get_notification_transport() for notifications https://review.openstack.org/266960 | 14:24 |
| *** shyama has quit IRC | 14:26 | |
| *** chhavi has joined #openstack-nova | 14:27 | |
| *** mdnadeem has quit IRC | 14:27 | |
| *** burgerk has joined #openstack-nova | 14:28 | |
| kashyap | Is it "xen: recheck" or "xenserver: recheck" or "recheck xen" or "recheck xenserver"? | 14:28 |
| *** akshai has joined #openstack-nova | 14:28 | |
| kashyap | (I found all of them recommended in IRC log) | 14:28 |
| cdent | kashyap: I had this a couple days ago, it depends on which one it was, and usually it says in the ci report | 14:28 |
| openstackgerrit | Sean Dague proposed openstack/nova: Make project_id optional in v2.1 urls https://review.openstack.org/233076 | 14:28 |
| *** xyang1 has joined #openstack-nova | 14:28 | |
| BobBall | kashyap: On your change, hit toggle CI (bottom left) then look at the comment left by the CI that failed and you want to recheck | 14:28 |
| sdague | on minor issue tweak | 14:28 |
| kashyap | cdent: Yes, I saw your mention, too. Say if this the one that failed: XenProject CI | 14:28 |
| BobBall | kashyap: Depending on the CI it's either xenserver: recheck or xen: recheck | 14:29 |
| rlrossit | I guess when mriedem has to write unit tests, he just abandons and gives it to me :( | 14:29 |
| cdent | heh, it was BobBall that helped me then too. BobBall++ | 14:29 |
| kashyap | cdent: It doesn't tell me in the "ci report". | 14:29 |
| kashyap | BobBall: Thanks, let me look. | 14:29 |
| cdent | kashyap: the place that BobBall is tell you to look is where I meant | 14:30 |
| BobBall | cdent: Unfortunately for me, I get a flashing screen whenever someone mentions "xen", "xenserver", "citrix" or me... Hard to ignore :D | 14:30 |
| kashyap | BobBall: It's not my change, it's some other change that I'm following aong | 14:30 |
| kashyap | s/aong/along/ | 14:30 |
| *** rk4n has joined #openstack-nova | 14:30 | |
| * cdent arms himself for future BobBall torture | 14:30 | |
| dansmith | johnthetubaguy: I thought we did actually, but I'll check | 14:31 |
| kashyap | BobBall: Few of us reviewed it, just trying to see if I can re-run "XenProject CI", as it's been a couple of days. | 14:31 |
| kashyap | BobBall: Heh, so I triggered all your alarm bells | 14:32 |
| BobBall | It's ok kashyap. I just need to add a snooze button | 14:32 |
| kashyap | BobBall: So, it's not my change, I'm not still clear, which one do I issue: 'xenserver: recheck' or 'xen: recheck'? | 14:33 |
| kashyap | (For a XenProject CI failure) | 14:33 |
| *** smatzek has quit IRC | 14:34 | |
| *** lucasagomes is now known as lucasagomes|dent | 14:34 | |
| *** lucasagomes|dent is now known as lucas|dentist | 14:34 | |
| BobBall | kashyap: You can still press the "toggle CI" button (bottom left) to see the comment the CI left on the review - but in this case it would be xen: recheck | 14:35 |
| *** browne has joined #openstack-nova | 14:36 | |
| kashyap | BobBall: Noted, thank you. | 14:36 |
| *** mriedem has joined #openstack-nova | 14:36 | |
| kashyap | Seems like a legit message: "This change depends on a change that failed to merge." | 14:36 |
| BobBall | which change? | 14:37 |
| *** Marga_ has joined #openstack-nova | 14:37 | |
| *** otter768 has joined #openstack-nova | 14:37 | |
| kashyap | Trying to find out | 14:38 |
| BobBall | I mean which change was the comment left on | 14:38 |
| kashyap | BobBall: https://review.openstack.org/#/c/263432/ | 14:38 |
| *** krtaylor has quit IRC | 14:39 | |
| dansmith | sdague: have you seen this live migration hang before? http://logs.openstack.org/88/251588/26/check/gate-tempest-dsvm-multinode-full/42844b8/logs/subnode-2/screen-n-cpu.txt.gz?level=TRACE#_2016-01-14_16_33_06_904 | 14:40 |
| BobBall | Curious that two CIs failed it (XenProject + Hyper-V) but Jenkins passed. I wonder if there was a race condition with another change that it depends on getting through gate. But yes, I'd suggest a recheck | 14:40 |
| *** nkrinner has quit IRC | 14:41 | |
| dansmith | sdague: would think it's related to me, but no failures before that, just a migration that starts and never really gets going | 14:41 |
| *** otter768 has quit IRC | 14:41 | |
| *** burt has joined #openstack-nova | 14:41 | |
| figleaf | sdague: looking through https://review.openstack.org/#/c/233076, and there seem to be version naming mis-matches. Like when you started you were adding v2.13, but now you're up to v2.16 | 14:42 |
| *** mrkz has joined #openstack-nova | 14:42 | |
| figleaf | sdague: but some of the older names remain | 14:42 |
| sdague | figleaf: where is the 2.13 reference? | 14:42 |
| figleaf | sdague: https://review.openstack.org/#/c/233076/21/nova/tests/functional/api_sample_tests/api_sample_base.py | 14:42 |
| figleaf | sdague: L87 | 14:43 |
| sdague | figleaf: ok, it's a comment, how about we fix that in a follow up | 14:43 |
| *** dansmith is now known as superdan | 14:43 | |
| figleaf | sdague: well, there's also the NoAuthMiddlewareV2_15 | 14:43 |
| sdague | right, that name actually means nothing :) | 14:43 |
| sdague | but I'm fine if you really want all that consistent | 14:44 |
| mriedem | superdan: that's coming from libvirt i think | 14:44 |
| mriedem | from the monitor | 14:44 |
| superdan | mriedem: right | 14:44 |
| mriedem | anything in the libvirt logs? | 14:44 |
| kashyap | BobBall: Thanks | 14:44 |
| superdan | mriedem: I'm trying to figure out if I'm causing it or it's a known failure of some sort, like firewall between workers or something | 14:44 |
| superdan | mriedem: yes, tons, per usual :) | 14:44 |
| figleaf | sdague: only because it could be confusing in the future when someone is trying to grok why that class is there and what it's supposed to do | 14:44 |
| figleaf | sdague: not because I'm OCD or anything :) | 14:45 |
| sdague | I've got a review guide up right now as a set of comments on the current patch, so I'd like to collect feedback before reving again and that getting hidden | 14:45 |
| mriedem | superdan: ovh node too, so it's slower | 14:45 |
| figleaf | sdague: sure, that's great | 14:45 |
| *** jaosorior has quit IRC | 14:45 | |
| *** cfriesen has joined #openstack-nova | 14:45 | |
| *** eharney has quit IRC | 14:46 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: clear instance.root_device_name while ignoring user device name https://review.openstack.org/214314 | 14:46 |
| *** jaosorior has joined #openstack-nova | 14:46 | |
| danpb | superdan: interesting - when it gets stuck like that, we request an abort | 14:46 |
| mriedem | superdan: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Live%20migration%20stuck%20for%5C%22%20AND%20tags%3A%5C%22screen-n-cpu.txt%5C%22&from=7d | 14:46 |
| danpb | superdan: so its strange that the log shows the same message over & over | 14:46 |
| danpb | that indictes the abort isn't working | 14:46 |
| sdague | superdan: looking now | 14:46 |
| superdan | danpb: hmm okay | 14:47 |
| mriedem | superdan: started occurring more frequently around 1/12 | 14:47 |
| superdan | ah, I think mriedem set me free | 14:47 |
| mriedem | multiple nodes | 14:47 |
| mriedem | all check queue, but multiple changes | 14:47 |
| superdan | so maybe related to my stuff, but not this last patch I guess | 14:47 |
| superdan | danpb: since you wrote that monitor stuff would you mind taking a look to see if you can figure out why? | 14:48 |
| mriedem | this merged on 1/11 https://github.com/openstack/nova/commit/f9237774bff25cf908eb4a0dcb083157c1599a2c | 14:48 |
| mriedem | MIN_LIBVIRT_REALTIME_VERSION = (1, 2, 13) | 14:48 |
| mriedem | heh, so that wouldn't be it | 14:48 |
| mriedem | we have nowhere close to that version of libvirt in the gate | 14:48 |
| bauzas | mriedem: superdan: FYI, investigating a reno issue possibly in liberty branch http://logs.openstack.org/69/268069/3/check/gate-nova-releasenotes/aaa7da2/console.html#_2016-01-15_11_27_02_287 | 14:49 |
| mriedem | bauzas: i don't see that note in nova's tree on master | 14:50 |
| mriedem | oslo.reports 1.3.0 was released yesterday, i don't know if that would be related | 14:50 |
| bauzas | mriedem: yup, it's in the stable branch | 14:51 |
| mriedem | https://github.com/openstack/nova/tree/stable/liberty/releasenotes/notes | 14:51 |
| mriedem | https://github.com/openstack/nova/tree/master/releasenotes/notes | 14:51 |
| *** ijw has joined #openstack-nova | 14:51 | |
| mriedem | i don't see new-oslo-reports-option-619c3dbf3ae320fb.yaml in either of those | 14:51 |
| mriedem | reading sources... [100%] unreleased | 14:51 |
| sdague | danpb / superdan - http://logs.openstack.org/88/251588/26/check/gate-tempest-dsvm-multinode-full/42844b8/logs/subnode-2/libvirt/libvirtd.txt.gz#_2016-01-14_16_30_37_298 is where it starts | 14:52 |
| mriedem | bauzas: i have seen some very strange issues running tox -e releasenotes locally, | 14:52 |
| bauzas | mriedem: that's what I'm sorting out | 14:52 |
| mriedem | bauzas: like it wouldn't clean up after itself and i had to rm -rf .tox/releasenotes to get it to work | 14:52 |
| sdague | my libvirt log reading is poor, so I don't know if there are any hints there as to why memory isn't getting transfered at all | 14:52 |
| bauzas | meh | 14:53 |
| superdan | and what is "exit remote" ? | 14:53 |
| bauzas | mriedem: trying to get the sh1 | 14:53 |
| superdan | does that mean the other side canceled or something? | 14:53 |
| bauzas | sha1 | 14:53 |
| mriedem | bauzas: https://review.openstack.org/#/c/253821/ | 14:53 |
| *** esracelik has quit IRC | 14:53 | |
| mriedem | bauzas: heh, so reno isn't handling deleted release notes it looks like | 14:54 |
| bauzas | mriedem: ooooh | 14:54 |
| bauzas | that's what I was looking for | 14:54 |
| bauzas | sec | 14:54 |
| *** ducttape_ has joined #openstack-nova | 14:54 | |
| bauzas | mriedem: that's in master right? | 14:54 |
| mriedem | bauzas: yeah | 14:55 |
| mriedem | bauzas: reported a bug https://bugs.launchpad.net/reno/+bug/1534613 | 14:55 |
| openstack | Launchpad bug 1534613 in reno "reno appears to fail when there are deleted release notes" [Undecided,New] | 14:55 |
| *** ijw has quit IRC | 14:55 | |
| bauzas | mriedem: oooh I think I got the idea | 14:56 |
| *** fawadkhaliq has quit IRC | 14:56 | |
| bauzas | mriedem: reno is iterating over all the commits | 14:57 |
| openstackgerrit | esracelik proposed openstack/nova: Improve the help text for hyperv options https://review.openstack.org/268175 | 14:57 |
| bauzas | mriedem: and since it finds 6ed3bd2b24bf9591dbac5d2f088d694cc6c49709 it tries to call the related file | 14:57 |
| *** scheuran_ has quit IRC | 14:57 | |
| mriedem | must be | 14:58 |
| danpb | mriedem: it goes back longer than 1/12 - still pretty high rate even on 1/6 | 14:58 |
| *** fawadkhaliq has joined #openstack-nova | 14:59 | |
| mriedem | new version of reno was released on 1/11 | 14:59 |
| kashyap | sdague: On that live mig failure, was trying to debug something similar with tdurakov_ in the morning | 14:59 |
| kashyap | sdague: Last we found was, on source QEMU log (subnode-2/libvirt/qemu/instance-00000013.txt.gz), we see this: "qemu: terminating on signal 15 from" | 14:59 |
| kashyap | Where 'singal 15' is SIGTERM. So, we're not sure _what_ is triggering that signal in Nova. | 15:00 |
| kashyap | Timofey said he's going to file a bug with some kind of reproducer details today. | 15:00 |
| *** rk4n has quit IRC | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-nova | 15:01 | |
| *** scheuran has joined #openstack-nova | 15:01 | |
| bauzas | mriedem: upgrading it locally | 15:01 |
| *** eharney has joined #openstack-nova | 15:02 | |
| bauzas | same problem | 15:04 |
| bauzas | mriedem: okay, I feel your bug is valid | 15:04 |
| *** nikoul has joined #openstack-nova | 15:04 | |
| bauzas | mriedem: in the meantime, I'm proposing an empty reno file matching the name | 15:04 |
| mriedem | i'm checking the reno code | 15:04 |
| bauzas | mriedem: so we can fix the problem without waiting the reno folks | 15:04 |
| bauzas | mriedem: I did | 15:04 |
| bauzas | mriedem: hence my point about the iterative approach | 15:05 |
| *** smatzek has joined #openstack-nova | 15:05 | |
| danpb | mriedem: ok, so the thread that performs the migration in libvirtd is pid 8274 and the last 2 things that pid does are | 15:05 |
| *** mhorban has quit IRC | 15:05 | |
| danpb | 2016-01-14 16:30:37.303+0000: 8274: debug : virStreamNew:15667 : conn=0x7f016800f0f0, flags=0 | 15:06 |
| danpb | 2016-01-14 16:30:37.303+0000: 8274: debug : qemuDomainObjEnterRemote:1406 : Entering remote (vm=0x7f016c001520 name=instance-00000003) | 15:06 |
| danpb | and then nothing more after that | 15:06 |
| *** gongysh has quit IRC | 15:06 | |
| danpb | so, that says to me that the remote libvirtd that it is connecting to is not responding | 15:06 |
| superdan | danpb: so maybe something with one of our new clouds and a wrinkle in cross-instance communication? | 15:07 |
| *** EinstCrazy has quit IRC | 15:07 | |
| bauzas | mriedem: oh crazy | 15:08 |
| danpb | on the other host things are a little strange - i'm not seeing as much in the logs as i would expect | 15:08 |
| bauzas | mriedem: you blew up the world | 15:08 |
| bauzas | :p | 15:08 |
| bauzas | mriedem: http://paste.openstack.org/show/484015/ | 15:09 |
| bauzas | mriedem: we can't trick | 15:09 |
| *** pratikma_ has joined #openstack-nova | 15:09 | |
| mriedem | b/c the revert is in the git history | 15:09 |
| mriedem | and that's what it's processing | 15:09 |
| bauzas | exactly | 15:10 |
| *** artom_ has joined #openstack-nova | 15:10 | |
| bauzas | it says that you have a duplicate commit that owns the file but not the one it's looking for | 15:10 |
| mriedem | so, i'm checking the code to see where it's failing so we can handlethat | 15:10 |
| bauzas | well, it needs a reno version bump | 15:10 |
| *** krtaylor has joined #openstack-nova | 15:11 | |
| mriedem | yeah, a fix and a release | 15:11 |
| bauzas | I was hoping for some mitigation in nova only | 15:11 |
| bauzas | but that's stuck | 15:11 |
| *** ericksonsantos has quit IRC | 15:11 | |
| *** jaosorior has quit IRC | 15:11 | |
| bauzas | meh, no sec | 15:11 |
| *** burgerk has quit IRC | 15:11 | |
| bauzas | we could unleash the rules | 15:12 |
| mriedem | i have an idea | 15:12 |
| *** jaosorior has joined #openstack-nova | 15:12 | |
| bauzas | and treat warnings not as errors | 15:12 |
| bauzas | until reno is fixed | 15:12 |
| mriedem | right, that's what i was thinking | 15:12 |
| bauzas | easy peasy | 15:12 |
| bauzas | it's in tox.ini | 15:12 |
| bauzas | lemme check locally | 15:12 |
| mriedem | actually setup.cfg | 15:13 |
| mriedem | testing locally | 15:13 |
| *** pratikmallya has quit IRC | 15:13 | |
| bauzas | say what ? | 15:13 |
| bauzas | I remember I set that flag | 15:13 |
| bauzas | yeah it works | 15:13 |
| bauzas | -commands = sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html | 15:13 |
| bauzas | +commands = sphinx-build -a -E -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html | 15:13 |
| superdan | sdague: would it offend you if I +1 this patch and wait for alex_xu or oomich_away to +2? If those guys are cool then I am, but I'd hate for my +2 to be co-opted by someone evil to land it without one of those guys seeing it | 15:14 |
| mriedem | bauzas: oh, i was looking at the pbr one | 15:14 |
| mriedem | that's in setup.cfg | 15:14 |
| mriedem | bauzas: push up the workaround fix | 15:14 |
| bauzas | mriedem: yup | 15:14 |
| bauzas | mriedem: but if so, we need to be super cautious before +Wing a reno change | 15:15 |
| bauzas | and check the logs | 15:15 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 15:15 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 15:15 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 15:15 |
| sdague | superdan: sure, that's fine, as long as we can get this landed next week. Monday is an HP holiday, so I'll see where things are Tues. | 15:15 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the cert options https://review.openstack.org/268186 | 15:16 |
| dedery_ | mriedem: hi, where do you think that https://review.openstack.org/#/c/252565/ should also land? (Liberty/Kilo/Juno?) | 15:16 |
| sdague | superdan: mostly I wanted to make sure you are good with the config option | 15:16 |
| sdague | and the way that is handled | 15:16 |
| mriedem | dedery_: ? | 15:16 |
| mriedem | dedery_: juno is eol | 15:16 |
| *** daemontool has quit IRC | 15:16 | |
| mriedem | dedery_: kilo is security/crit fix only | 15:16 |
| superdan | sdague: okay and I'm out all next week, after monday | 15:17 |
| *** szaher__ has quit IRC | 15:17 | |
| dedery_ | mriedem: sorry, didn't look at the release timelines. So Liberty yes? | 15:17 |
| mriedem | dedery_: i suppose | 15:17 |
| *** mtanino has joined #openstack-nova | 15:18 | |
| mriedem | dedery_: you have a logging typo in your change | 15:18 |
| *** claudiub has joined #openstack-nova | 15:19 | |
| mriedem | superdan: nice timing with FF :) | 15:19 |
| *** fawadkhaliq has quit IRC | 15:19 | |
| dedery_ | mriedem: correct. didn't notice. should I wait for the merge and then fix? | 15:20 |
| *** yamahata has joined #openstack-nova | 15:20 | |
| mriedem | dedery_: or fix now and we can quick re-approve | 15:20 |
| *** raildo is now known as raildo-afk | 15:20 | |
| mriedem | if you want to backport to liberty, you'd need both changes backported if you did | 15:20 |
| mriedem | which is kind of gros | 15:20 |
| sdague | superdan: ok, how about I fix figleaf's comments | 15:20 |
| mriedem | *gross | 15:20 |
| sdague | upload, and you can +1 that with comments and will poke alex_xu next week on things | 15:20 |
| superdan | mriedem: heh | 15:20 |
| superdan | sdague: sure | 15:21 |
| dedery_ | mriedem: in both you mean this patch and the removal of refresh from neutron patch? | 15:21 |
| mriedem | dedery_: no i mean this patch and a 2nd patch that fixes that logging issue | 15:21 |
| sdague | superdan: oh, and I'll fix the indent comment you put in there | 15:21 |
| mriedem | dedery_: so i'd just update it quick and then we can quickly re-approve since it's a trivial fix | 15:21 |
| mriedem | dedery_: hell, you can edit it inline in gerrit quick | 15:22 |
| dedery_ | hehe :) never did it. will try it now. | 15:22 |
| *** carey has joined #openstack-nova | 15:22 | |
| mriedem | don't mess up | 15:22 |
| *** vishwanathj_away is now known as vishwanathj | 15:22 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Workaround reno reverts by accepting warnings https://review.openstack.org/268196 | 15:24 |
| bauzas | mriedem: ^ | 15:24 |
| bauzas | johnthetubaguy: you could be interested in that too ^ | 15:24 |
| openstackgerrit | David Edery proposed openstack/nova: ports & networks gather should validate existance https://review.openstack.org/252565 | 15:24 |
| *** efoley has quit IRC | 15:24 | |
| *** zz_dimtruck is now known as dimtruck | 15:24 | |
| *** Jeffrey4l has quit IRC | 15:25 | |
| mriedem | bauzas: +2 | 15:25 |
| bauzas | ack | 15:25 |
| * bauzas runs away for preparing the skiing session | 15:26 | |
| bauzas | _o | 15:26 |
| dedery_ | mriedem: cool! nice feature! :) | 15:26 |
| *** carey has quit IRC | 15:27 | |
| * cdent crashed into edleafe | 15:27 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Make project_id optional in v2.1 urls https://review.openstack.org/233076 | 15:27 |
| sdague | figleaf / superdan ^^^ updated | 15:27 |
| mriedem | dedery_: +W | 15:28 |
| kashyap | superdan: That live migration issue, also it's running QEMU 2.0 (which is from April 2014), current release is 2.3 (from Dec). QEMU migration dev this morning (CET) was saying that a lot more debugging details have been added in the newer one. | 15:28 |
| figleaf | cdent: ?? | 15:28 |
| kashyap | If there's a way to get the newer version slipped into that multi-node live mig test job. | 15:28 |
| cdent | in our reviews of sdague's thing | 15:29 |
| superdan | kashyap: okay | 15:29 |
| figleaf | cdent: ah | 15:29 |
| cdent | I left pretty much the same comments, but only realized after submitting | 15:29 |
| sdague | cdent: heh | 15:29 |
| cdent | bit by gertty | 15:29 |
| *** paul-carlton2 has joined #openstack-nova | 15:32 | |
| *** xianghui has quit IRC | 15:33 | |
| *** derekh has quit IRC | 15:33 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Workaround reno reverts by accepting warnings https://review.openstack.org/268196 | 15:33 |
| *** aix has quit IRC | 15:33 | |
| *** derekh has joined #openstack-nova | 15:33 | |
| sdague | cdent: so the work around regex, there are a few limitations to what will actually work and extract the project_id until the routes patch lands | 15:33 |
| *** xianghui has joined #openstack-nova | 15:34 | |
| cdent | ah | 15:34 |
| sdague | https://github.com/bbangert/routes/pull/57 is all ready to go, and the maintainer said he'd take it | 15:34 |
| cdent | feh | 15:34 |
| sdague | but it's still outstanding | 15:34 |
| *** jamie_h has joined #openstack-nova | 15:35 | |
| cdent | cool enough | 15:35 |
| dedery_ | mriedem: about the doc strings in tests - this means that if I want to describe the test I should do it with the usual comment hash-sign prefix? | 15:36 |
| Thelo | Hi all, I am submitting a new driver for cinder ( so it includes a patch in nova, os-brick and cinder). For cinder we have a deadline which said that the code has to be merged on Jan 19th, is there such things for nova? | 15:36 |
| mriedem | dedery_: yeah | 15:36 |
| dedery_ | mriedem: 10x | 15:37 |
| mriedem | Thelo: 1/21 is non-priority blueprint feature freeze for nova | 15:37 |
| cdent | sdague: two of my comments are still relevant on ps22 | 15:37 |
| sdague | cdent: ok, which ones? | 15:37 |
| cdent | line 207, comment on the nova/api/openstack/__init__.py project_id_regex say no - but - is there | 15:38 |
| cdent | and s/user/use/ in the release note | 15:38 |
| sdague | oh, sorry missed those | 15:38 |
| *** mragupat has joined #openstack-nova | 15:40 | |
| cdent | it's a big patch | 15:40 |
| *** andrearosa has quit IRC | 15:40 | |
| *** lucas|dentist is now known as lucasagomes | 15:40 | |
| Thelo | mriedem: so far when I submitted my patch I use this blueprint (https://review.openstack.org/#/c/253353/1), is that acceptable? | 15:40 |
| openstackgerrit | Sean Dague proposed openstack/nova: Make project_id optional in v2.1 urls https://review.openstack.org/233076 | 15:40 |
| sdague | though it actually became a lot smaller and more straight forward than I expected | 15:41 |
| sdague | cdent: ok, fixed | 15:41 |
| mriedem | Thelo: that's a cinder blueprint, usually we have a nova blueprint for the libvirt volume driver shim | 15:41 |
| mriedem | that uses os-brick | 15:41 |
| mriedem | the nova blueprint can depend on the cinder blueprint | 15:41 |
| Thelo | so I should mae a new one right? | 15:41 |
| Thelo | before the 21st? | 15:42 |
| openstackgerrit | Dimitry Ushakov proposed openstack/nova: (lxc) Updated regex to ignore failing tests https://review.openstack.org/267724 | 15:42 |
| Thelo | mriedem: How do I write the blueprint dependency? | 15:43 |
| mriedem | there is a button somewhere in the bp page | 15:44 |
| *** rk4n has joined #openstack-nova | 15:44 | |
| Thelo | mriedem: what project should I pick nova-hyper? | 15:45 |
| *** nelsnelson has joined #openstack-nova | 15:46 | |
| mriedem | just nova | 15:46 |
| mriedem | https://blueprints.launchpad.net/nova | 15:46 |
| openstackgerrit | Sean Dague proposed openstack/nova: Make project_id optional in v2.1 urls https://review.openstack.org/233076 | 15:46 |
| sdague | cdent: damn you and finding typos :) | 15:46 |
| cdent | I can't help it! | 15:47 |
| cdent | I wish I had the same eye for detail in my own stuff. | 15:47 |
| * cdent goes home before he finds another one | 15:48 | |
| *** cdent has quit IRC | 15:48 | |
| mriedem | sdague: heh, i just did | 15:48 |
| mriedem | pile on time! | 15:48 |
| sdague | mriedem: ok, what now :) | 15:48 |
| *** rk4n has quit IRC | 15:48 | |
| mriedem | overlaping isn't a word | 15:48 |
| mriedem | neither is overlapping though | 15:48 |
| mriedem | oh i guess it is | 15:48 |
| mriedem | gerund or present participle:Â overlapping | 15:48 |
| openstackgerrit | Sean Dague proposed openstack/nova: Make project_id optional in v2.1 urls https://review.openstack.org/233076 | 15:49 |
| mriedem | our next dog's name is going to be gerund | 15:49 |
| sdague | mriedem: ok fixed | 15:49 |
| Thelo | mriedem: so if I do my blueprint now, how long do I have before the code need to be merged? | 15:49 |
| mriedem | Thelo: 1/21 | 15:49 |
| *** edtubill has joined #openstack-nova | 15:49 | |
| mriedem | Thelo: but that depends on the cinder driver getting into cinder, and the os-brick change getting into os-brick first | 15:50 |
| mriedem | since your nova change depends on the os-brick change | 15:50 |
| *** dedery_ has quit IRC | 15:50 | |
| *** takedakn has joined #openstack-nova | 15:51 | |
| Thelo | the change has been merged to os-brick already, and is on-going for cinder | 15:52 |
| *** sajeesh has joined #openstack-nova | 15:52 | |
| *** ijw has joined #openstack-nova | 15:52 | |
| Thelo | so I guess tomorrow it will be fine | 15:52 |
| *** twinklechawla has joined #openstack-nova | 15:52 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Workaround reno reverts by accepting warnings https://review.openstack.org/268196 | 15:53 |
| mriedem | how did it get into os-brick when the volume driver isn't in cinder yet? | 15:53 |
| *** tonytan4ever has joined #openstack-nova | 15:55 | |
| *** josecastroleon has quit IRC | 15:57 | |
| *** ijw has quit IRC | 15:57 | |
| *** takedakn has quit IRC | 15:59 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** aix has joined #openstack-nova | 16:01 | |
| *** nikoul has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-nova | 16:01 | |
| *** andrearosa has joined #openstack-nova | 16:02 | |
| johnthetubaguy | sdague: this issue just hit me when reviewing this API addition: https://review.openstack.org/#/c/248989/61 | 16:03 |
| *** tjones has joined #openstack-nova | 16:04 | |
| johnthetubaguy | sdague: we add a new action in a microversion, it adds vm states and task states, but now if you call this with one API version, and list with an older version, you kinda break the API versioning, if we are strict about that. Its like we should have added some catch all "unknown_task" and "unknown_state" for those cases? | 16:05 |
| sdague | johnthetubaguy: we should be hiding them | 16:05 |
| Thelo | mriedem: it is an egg-chicken issue the cinder patch depends of the os-brick patch | 16:05 |
| *** belmoreira has quit IRC | 16:05 | |
| Thelo | mriedem: I made a new blueprint could you tell me if its correct https://blueprints.launchpad.net/nova/+spec/add-cinder-disco-connector-for-libvirt | 16:06 |
| sdague | and that should be tested for | 16:06 |
| sdague | johnthetubaguy: good catch | 16:06 |
| johnthetubaguy | sdague: I know in the past we have used an alias, but I don't think it works in this case, so we probably need "unknown" | 16:06 |
| tobasco | johnthetubaguy: i have verified that the patch is working using master on devstack, here is the request spec and some vars dumped from build_request_spec http://paste.openstack.org/raw/484019/ | 16:06 |
| cfriesen | kind of a random question, but does anyone know why we don't provide quotas on local storage usage? | 16:07 |
| johnthetubaguy | tobasco: doesn't that prove that root_gb = 1 and not 0? | 16:07 |
| johnthetubaguy | tobasco: in the request_spec, I mean | 16:08 |
| sdague | johnthetubaguy: yeh, I don't know if there is a current status that it maps to | 16:08 |
| mriedem | johnthetubaguy: in the essence of time, i'm going to approve this specless blueprint in nova for adding a libvirt volume driver shim https://blueprints.launchpad.net/nova/+spec/add-cinder-disco-connector-for-libvirt | 16:09 |
| mriedem | johnthetubaguy: the os-brick and cinder changes are approved, | 16:09 |
| mriedem | waiting for next week's nova meeting for the specless bp approval would effectively kill it's chances of landing in nova in mitaka | 16:09 |
| mriedem | Thelo: you need to work with the cinder team (probably hemnafk) on getting an os-brick release out with your change | 16:10 |
| *** salv-orl_ has joined #openstack-nova | 16:10 | |
| johnthetubaguy | mriedem: technically, I thought we agreed no more blueprints for mitaka, given we have so many that already need reviewing, and made the deadline? | 16:10 |
| Thelo | mriedem: its on=going we have been trying since yesterday but the release team hasn't approved our release patch yet | 16:10 |
| *** rdopiera has quit IRC | 16:10 | |
| *** cdent has joined #openstack-nova | 16:11 | |
| mriedem | johnthetubaguy: i'm not guaranteeing this will be approved before 1/21, but the code has been up | 16:11 |
| mriedem | johnthetubaguy: seems pretty trivial | 16:11 |
| johnthetubaguy | mriedem: I guess this is a trivial shim, so thats worth slipping in? | 16:11 |
| johnthetubaguy | mriedem: OK | 16:11 |
| mriedem | Thelo: the release team usually releases early in the week | 16:11 |
| tobasco | johnthetubaguy: not really i think, the scheduler reads it as spec_obj.root_gb and does not read the root_gb from the flavor nor the instance_type from what I see | 16:11 |
| mriedem | Thelo: to avoid a friday release when no one is around to fix breakages | 16:11 |
| tobasco | i'm gonna dump some data directly from the Diskfilter and see what it uses | 16:11 |
| tobasco | so we know for sure | 16:11 |
| *** chhavi has quit IRC | 16:12 | |
| johnthetubaguy | tobasco: spec_obj.root_gb is a fake property that just reads the flavor | 16:12 |
| mriedem | Thelo: also note ^ that there are no guarantees that this is going to make nova in mitaka | 16:12 |
| mriedem | Thelo: b/c we already have 100+ bp reviews up | 16:12 |
| johnthetubaguy | tobasco: and the flavor says 1 not 0, I think, if I am reading it correctly | 16:12 |
| Thelo | mriedem: ok in my case my patch is ready I just need the new os-brick releases | 16:12 |
| *** edtubill has quit IRC | 16:12 | |
| mriedem | Thelo: your patch isn't ready | 16:13 |
| mriedem | it has 0 unit tests | 16:13 |
| Thelo | mriedem: got it, thanks | 16:13 |
| *** salv-orlando has quit IRC | 16:13 | |
| *** edtubill has joined #openstack-nova | 16:14 | |
| Thelo | mriedem: wait, I did write some, and forgot to commit them, I will do this right now | 16:14 |
| *** schwicke has joined #openstack-nova | 16:14 | |
| mriedem | Thelo: is there an openstack/releases request up for the os-brick release? | 16:14 |
| mriedem | https://review.openstack.org/#/c/267722/ | 16:15 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Ceph for live-migration job https://review.openstack.org/252283 | 16:15 |
| * cdent owes mriedem a beer or other beverage of his choice | 16:16 | |
| Thelo | mriedem: yes https://review.openstack.org/#/c/267722/ | 16:16 |
| mriedem | Thelo: so your nova change needs to actually depend on this https://review.openstack.org/#/c/267725/ | 16:17 |
| *** dave-mccowan has joined #openstack-nova | 16:18 | |
| *** mdrabe has quit IRC | 16:18 | |
| tobasco | johnthetubaguy: ok i verified it now and yes it reads 1 as root_gb, so from the flavor probably then as you said. So there are two alternatives 1) encourage people that has this setup to not use the diskfilter, 2) change the root_gb data that is passed to the scheduler i.e modify the flavor root_gb from build_request_spec however that needs us to somehow being able to determine if the instance is v | 16:19 |
| tobasco | olume-backed in the scheduler/utils.py file | 16:19 |
| *** zenoway has quit IRC | 16:20 | |
| *** dslev has quit IRC | 16:20 | |
| johnthetubaguy | tobasco: I commented on your patch about doing (2) | 16:21 |
| johnthetubaguy | tobasco: folks are likely to support local and remote volumes, in some cases, so it might matter to them | 16:22 |
| *** gjayavelu has joined #openstack-nova | 16:24 | |
| tobasco | johnthetubaguy: totally agree, I need to head home now, but I will try to have some time this weekend to fix this or I will continue next week at work | 16:24 |
| tobasco | johnthetubaguy: thanks for helping | 16:24 |
| rfolco | johnthetubaguy, dims: could you please take a look at https://review.openstack.org/#/c/226555/ when you get a chance please ? | 16:24 |
| *** pradk has joined #openstack-nova | 16:24 | |
| *** vivekd has joined #openstack-nova | 16:25 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: Mask the token used to allow access to consoles https://review.openstack.org/220622 | 16:28 |
| *** vilobhmm11 has joined #openstack-nova | 16:28 | |
| *** angdraug has joined #openstack-nova | 16:29 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Ceph for live-migration job https://review.openstack.org/252283 | 16:30 |
| *** ericksonsantos has joined #openstack-nova | 16:31 | |
| *** mdrabe has joined #openstack-nova | 16:32 | |
| *** zenoway has joined #openstack-nova | 16:32 | |
| mriedem | rfolco: what version does the powerkvm ci test with? | 16:32 |
| mriedem | nvm, checking | 16:33 |
| rfolco | mriedem, we run upstream kvm from ubuntu, | 16:33 |
| rfolco | let me get the exact version | 16:34 |
| mriedem | 2.2 | 16:34 |
| rfolco | mriedem, currently running qemu-kvm 2.2 | 16:34 |
| mriedem | for qemu | 16:34 |
| mriedem | yeah, and you have min of 2.1 | 16:34 |
| mriedem | is there a link to qemu docs that say power kvm support is in 2.1? | 16:35 |
| *** gjayavelu has quit IRC | 16:35 | |
| gibi | mriedem: will your new reno fix https://review.openstack.org/#/c/268218 fail in the check queue as origin/stable/liberty still contains a commit that produce a warning that is now treated as error? | 16:36 |
| *** devananda has quit IRC | 16:36 | |
| *** ihrachys_ has quit IRC | 16:36 | |
| mriedem | rfolco: i guess https://launchpad.net/ubuntu/utopic/+source/qemu/+changelog | 16:36 |
| mriedem | ppc support is added in 2.1 | 16:36 |
| rfolco | ppc64le, yes | 16:37 |
| rfolco | ppc64 Big Endian supported since 1.6 | 16:37 |
| *** zenoway has quit IRC | 16:37 | |
| *** zakora has quit IRC | 16:37 | |
| *** paul-carlton2 has quit IRC | 16:37 | |
| rfolco | but we equilized to the upper levels dues to nuance around host/guest arches | 16:37 |
| *** zhangjn has joined #openstack-nova | 16:38 | |
| *** devananda has joined #openstack-nova | 16:38 | |
| *** otter768 has joined #openstack-nova | 16:38 | |
| *** unicell has joined #openstack-nova | 16:38 | |
| rfolco | all current distros for ppc have the min levels | 16:39 |
| rfolco | mriedem, ^ | 16:39 |
| mriedem | duplicating those maps in the tests and driver code is nasty | 16:40 |
| danpb | superdan: the ovo.base.VersionedObject constructor iterates over the kwargs calling setattr for each of them | 16:41 |
| *** edtubill has quit IRC | 16:41 | |
| danpb | superdan: shouldn't it only be setting keywords which are listed in self.fields ? | 16:41 |
| *** hemnafk is now known as hemna | 16:42 | |
| danpb | otherwise it doesn't catch mistakes where you typo the attribute name given in the constructor | 16:42 |
| superdan | danpb: well, you could have a property with an @setter for compatibility reasons | 16:42 |
| *** edtubill has joined #openstack-nova | 16:42 | |
| superdan | danpb: and we do have objects with non-field properties for things we never want to go over the wire | 16:42 |
| openstackgerrit | Chuck Carmack proposed openstack/nova: REST API changes for user settable server description https://review.openstack.org/254950 | 16:42 |
| *** otter768 has quit IRC | 16:43 | |
| danpb | it seems a shame to loose all sanity checking of field names for that :-( | 16:43 |
| mriedem | sdague: here is where we could use ddt :) https://review.openstack.org/#/c/226555/6/nova/tests/unit/virt/libvirt/test_driver.py | 16:43 |
| mriedem | sdague: but i'm assuming testscenarios could be used there? | 16:44 |
| superdan | danpb: how about hasattr() as a compromise? | 16:44 |
| superdan | danpb: that way only things declared ahead of time would be caught? | 16:44 |
| danpb | that seems like it'd do the trick | 16:45 |
| ndipanov | superdan, wouldn't they be class attrs then | 16:46 |
| ndipanov | which is similar but not the same thing | 16:46 |
| superdan | ndipanov: not sure what you mean | 16:46 |
| openstackgerrit | Thelo Gaultier proposed openstack/nova: Add ITRI DISCO os-brick connector for libvirt https://review.openstack.org/253353 | 16:46 |
| superdan | ndipanov: it would allow us to use @property.setter things, and if you're subclassing, you can set a thing before calling super() so that it works as you want | 16:46 |
| ndipanov | ah ok | 16:47 |
| *** dimtruck is now known as zz_dimtruck | 16:48 | |
| *** vivekd has quit IRC | 16:48 | |
| *** zenoway has joined #openstack-nova | 16:50 | |
| *** zenoway has quit IRC | 16:50 | |
| *** zenoway has joined #openstack-nova | 16:51 | |
| *** zhangjn has quit IRC | 16:51 | |
| *** tdurakov_ has quit IRC | 16:51 | |
| *** nelsnels_ has joined #openstack-nova | 16:52 | |
| *** nelsnelson has quit IRC | 16:52 | |
| *** armax has joined #openstack-nova | 16:52 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add tests for floating_ip private functions https://review.openstack.org/267768 | 16:52 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Remove DictCompat from FixedIP https://review.openstack.org/265951 | 16:52 |
| *** ijw has joined #openstack-nova | 16:53 | |
| superdan | rlrossit: have you fixed your broken? | 16:54 |
| *** vivekd has joined #openstack-nova | 16:54 | |
| rlrossit | superdan: no.... | 16:54 |
| rlrossit | now I just fail on neutron tempest | 16:54 |
| rlrossit | but still no errors are logged | 16:54 |
| *** fawadkhaliq has joined #openstack-nova | 16:55 | |
| rlrossit | I understand why no one wanted to do DictCompat stuff now... | 16:55 |
| *** READ10 has joined #openstack-nova | 16:55 | |
| superdan | heh | 16:55 |
| superdan | well, this isn't the reason really | 16:55 |
| superdan | it's thankless work, but should be straightforward | 16:55 |
| Thelo | riedem: Thanks for your comments I updated the patch with what was missing ( commit messages changed, add unit test, update requirement) | 16:56 |
| *** annegentle has quit IRC | 16:56 | |
| rlrossit | so when I get frustrated at that I just move on to working on this oslo.messaging update for notifications, but of course ceilometer is currently "blowing up" to put it technically | 16:56 |
| rlrossit | dims: how do you think we should move forward with respect to ^ ? | 16:57 |
| *** ijw has quit IRC | 16:57 | |
| rlrossit | I mean it isn't failing ceilometer tempest, and all of ceilometer is dying on the functional tests, so my notification transport patch "worked" right? | 16:57 |
| *** burgerk has joined #openstack-nova | 16:58 | |
| dims | rlrossit : pong | 16:58 |
| *** sfinucan has quit IRC | 16:59 | |
| *** vilobhmm11 has quit IRC | 17:00 | |
| dims | rlrossit : yes, probably :). so the patch to fix ceilometer should merge in a few hours. so best case scenario let's recheck your review after that. let me find the review you have to wait for to merge | 17:00 |
| dims | rlrossit : https://review.openstack.org/#/c/267983/ | 17:00 |
| *** diogogmt has quit IRC | 17:00 | |
| *** haomaiwang has quit IRC | 17:01 | |
| *** haomaiwang has joined #openstack-nova | 17:01 | |
| rlrossit | thanks dims | 17:01 |
| * rlrossit opens zuul | 17:01 | |
| * rlrossit stares | 17:01 | |
| *** jlanoux has quit IRC | 17:01 | |
| * rlrossit sees top patch has taken 18.5 hours so far | 17:01 | |
| * rlrossit cries | 17:01 | |
| *** diogogmt has joined #openstack-nova | 17:02 | |
| * cdent hands rlrossit a tissue | 17:02 | |
| rlrossit | does this mean I can just go home? | 17:02 |
| dims | rlrossit : i can't put my finger on it yet...it was 5 hours yesterday afternoon and just keeps climbing up | 17:03 |
| *** mgoddard has quit IRC | 17:03 | |
| *** mgoddard has joined #openstack-nova | 17:03 | |
| rlrossit | well, you know what. If it looks like this is right, I'll start pushing the notification stuff up to neutron and cinder too | 17:03 |
| rlrossit | being proactive and such | 17:03 |
| *** lpetrut has quit IRC | 17:03 | |
| *** salv-orl_ has quit IRC | 17:04 | |
| *** salv-orlando has joined #openstack-nova | 17:04 | |
| dims | rlrossit : +1000 | 17:05 |
| tiagogomes | johnthetubaguy any change of that spec being implemented for Mitaka? | 17:07 |
| tiagogomes | ah sorry | 17:07 |
| sdague | mriedem: yeh, testscenarios could be used as well | 17:10 |
| *** matrohon has quit IRC | 17:10 | |
| mdbooth | superdan: Do I recall that I can get a request context from a nova object now? | 17:11 |
| *** zz_dimtruck is now known as dimtruck | 17:11 | |
| superdan | mdbooth: well, it depends.. if it was created with one or passed over the wire then, kinda | 17:12 |
| *** paul-carlton2 has joined #openstack-nova | 17:12 | |
| superdan | but it's not a public "here's yer context" interface | 17:12 |
| superdan | and s/now/since always/ | 17:12 |
| mdbooth | superdan: Ok, thanks. Sounds like I shouldn't rely on it. | 17:12 |
| *** e0ne has quit IRC | 17:13 | |
| *** ijw has joined #openstack-nova | 17:13 | |
| superdan | mdbooth: any chance explaining more will help your case? :) | 17:13 |
| johnthetubaguy | tiagogomes: which spec? | 17:13 |
| superdan | certainly if it's something done inside an object, then it's fine | 17:13 |
| *** apoorvad has joined #openstack-nova | 17:13 | |
| mdbooth | Nothing specific. I'm trying to remove some extraneous arguments, and I've found a single use for context buried several levels deep. | 17:14 |
| mdbooth | If I need to pass context all the way down there then so be it, but it would be tidy not to given that it's almost never required. | 17:14 |
| mdbooth | specifically getting a key in imagebackend.Lvm.create_image | 17:15 |
| superdan | ah | 17:15 |
| *** tdurakov_ has joined #openstack-nova | 17:15 | |
| superdan | yeah, might be a little deep to assume it that low | 17:16 |
| *** eil397 has joined #openstack-nova | 17:22 | |
| *** carey_ has joined #openstack-nova | 17:23 | |
| *** ljxiash has joined #openstack-nova | 17:23 | |
| *** sahid has quit IRC | 17:23 | |
| tiagogomes | johnthetubaguy I summoned you by mistake, sorry | 17:23 |
| *** annegentle has joined #openstack-nova | 17:24 | |
| *** raildo-afk is now known as raildo | 17:24 | |
| *** irenab_ has joined #openstack-nova | 17:24 | |
| *** raildo is now known as raildo-afk | 17:25 | |
| *** ssurana has joined #openstack-nova | 17:25 | |
| *** irenab has quit IRC | 17:25 | |
| *** irenab_ is now known as irenab | 17:25 | |
| *** derekh has quit IRC | 17:25 | |
| *** danpb has quit IRC | 17:26 | |
| *** SourabhP has joined #openstack-nova | 17:26 | |
| *** unicell has quit IRC | 17:26 | |
| *** SourabhP_ has joined #openstack-nova | 17:27 | |
| *** gangil has joined #openstack-nova | 17:27 | |
| *** gangil has joined #openstack-nova | 17:27 | |
| *** carey_ has quit IRC | 17:28 | |
| *** tdurakov_ has quit IRC | 17:30 | |
| johnthetubaguy | tiagogomes: ah, no worries | 17:31 |
| *** SourabhP has quit IRC | 17:31 | |
| *** SourabhP_ is now known as SourabhP | 17:31 | |
| *** sajeesh has quit IRC | 17:34 | |
| *** zenoway has quit IRC | 17:35 | |
| *** dane-fichter has joined #openstack-nova | 17:35 | |
| openstackgerrit | Daniel Berrange proposed openstack/os-vif: remove obsolete requirements https://review.openstack.org/268244 | 17:35 |
| openstackgerrit | Daniel Berrange proposed openstack/os-vif: actually register the various objects we define https://review.openstack.org/268245 | 17:35 |
| openstackgerrit | Daniel Berrange proposed openstack/os-vif: remove dependancy on nova object model https://review.openstack.org/268246 | 17:35 |
| *** tdurakov_ has joined #openstack-nova | 17:35 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add host_status attribute for servers/detail and servers/{server_id} https://review.openstack.org/241562 | 17:36 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add object and database support for host_status API https://review.openstack.org/267062 | 17:36 |
| *** cdent has left #openstack-nova | 17:36 | |
| *** tdurakov_ has quit IRC | 17:38 | |
| *** irenab has quit IRC | 17:38 | |
| *** tdurakov_ has joined #openstack-nova | 17:39 | |
| *** gjayavelu has joined #openstack-nova | 17:39 | |
| *** pratikma_ has quit IRC | 17:40 | |
| *** irenab has joined #openstack-nova | 17:40 | |
| *** tdurakov_ has quit IRC | 17:40 | |
| *** achanda has joined #openstack-nova | 17:43 | |
| *** tdurakov_ has joined #openstack-nova | 17:43 | |
| *** chhavi has joined #openstack-nova | 17:44 | |
| *** browne has quit IRC | 17:44 | |
| *** gangil has quit IRC | 17:44 | |
| *** gjayavelu has quit IRC | 17:44 | |
| *** sudipto has joined #openstack-nova | 17:45 | |
| *** shakamunyi has joined #openstack-nova | 17:46 | |
| *** edtubill has quit IRC | 17:47 | |
| *** achanda has quit IRC | 17:47 | |
| *** zenoway has joined #openstack-nova | 17:47 | |
| *** ankita_wagh has joined #openstack-nova | 17:48 | |
| *** tdurakov_ has quit IRC | 17:48 | |
| *** cdent has joined #openstack-nova | 17:49 | |
| *** paul-carlton2 has quit IRC | 17:49 | |
| *** tdurakov_ has joined #openstack-nova | 17:51 | |
| *** jamielennox is now known as jamielennox|away | 17:52 | |
| *** paul-carlton2 has joined #openstack-nova | 17:52 | |
| *** zenoway has quit IRC | 17:52 | |
| *** ericksonsantos has quit IRC | 17:53 | |
| *** jamie_h has quit IRC | 17:54 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Ceph for live-migration job https://review.openstack.org/252283 | 17:54 |
| tdurakov_ | johnthetubaguy, can I ask you to take a look at https://review.openstack.org/#/c/252283/ | 17:55 |
| tdurakov_ | fully working ci for live-migration: https://jenkins03.openstack.org/job/gate-tempest-dsvm-multinode-live-migration/24/console | 17:55 |
| *** nic has joined #openstack-nova | 17:56 | |
| *** edtubill has joined #openstack-nova | 17:57 | |
| *** _diana_1 has joined #openstack-nova | 17:57 | |
| tdurakov_ | ops, wrong link: https://jenkins05.openstack.org/job/gate-tempest-dsvm-multinode-live-migration/17/console | 17:57 |
| *** browne has joined #openstack-nova | 17:58 | |
| auggy | sdague: this patch is finally green (missed some pep8 errors for some reason) https://review.openstack.org/#/c/263440/ | 17:58 |
| *** browne has quit IRC | 17:58 | |
| auggy | sdague: it looks like we have some direction over how to move forward with the project_id problem, i'm happy to help out with whatever tasks you need helping with :) | 17:59 |
| openstackgerrit | Rossella Sblendido proposed openstack/nova: Wait for neutron events when using xen driver https://review.openstack.org/268258 | 17:59 |
| *** e0ne has joined #openstack-nova | 18:00 | |
| *** irenab_ has joined #openstack-nova | 18:00 | |
| *** haomaiwang has quit IRC | 18:01 | |
| *** haomaiwang has joined #openstack-nova | 18:01 | |
| *** irenab has quit IRC | 18:02 | |
| *** ericksonsantos has joined #openstack-nova | 18:05 | |
| *** mriedem has quit IRC | 18:05 | |
| johnthetubaguy | tdurakov_: is there a quick way to get the tests to run on that patch | 18:05 |
| johnthetubaguy | I really need to run off for the day now, I am afraid | 18:05 |
| johnthetubaguy | off to skegness | 18:05 |
| *** irenab has joined #openstack-nova | 18:06 | |
| *** edtubill has quit IRC | 18:06 | |
| *** irenab_ has quit IRC | 18:07 | |
| tdurakov_ | johnthetubaguy, quick way is in progress and sdague review needed | 18:08 |
| *** irenab_ has joined #openstack-nova | 18:09 | |
| *** hblixt has joined #openstack-nova | 18:11 | |
| *** vivekd has quit IRC | 18:11 | |
| *** zenoway has joined #openstack-nova | 18:11 | |
| *** irenab has quit IRC | 18:12 | |
| *** irenab_ is now known as irenab | 18:12 | |
| *** mriedem has joined #openstack-nova | 18:12 | |
| *** ctrath has quit IRC | 18:14 | |
| *** scheuran has quit IRC | 18:14 | |
| *** _diana_1 has joined #openstack-nova | 18:14 | |
| *** _diana_1 is now known as diana_clarke | 18:14 | |
| *** SpamapS has joined #openstack-nova | 18:14 | |
| SpamapS | Hi! I'm debugging quota things. I know, crazy right. Before I go off making a thread on the mailing list.. is there a reason why the PK for quota_usages isn't just project_id?? | 18:15 |
| *** suro-patz has joined #openstack-nova | 18:15 | |
| *** ljxiash has quit IRC | 18:16 | |
| *** zenoway has quit IRC | 18:16 | |
| *** suro-patz has quit IRC | 18:17 | |
| *** yassine_ has quit IRC | 18:19 | |
| *** marcin12345 has joined #openstack-nova | 18:20 | |
| *** ctrath has joined #openstack-nova | 18:20 | |
| *** ihrachys has joined #openstack-nova | 18:20 | |
| marcin12345 | Hello everybody, I am looking to suggestion about "osapi_compute_workers" could this be higher than number of my cores with no issues? | 18:21 |
| *** gangil has joined #openstack-nova | 18:21 | |
| *** gangil has joined #openstack-nova | 18:21 | |
| *** ctrath has quit IRC | 18:21 | |
| *** tonytan4ever has quit IRC | 18:21 | |
| marcin12345 | I am trying to increase number of concurrent operations and currently icrease that to 2xcores | 18:21 |
| *** ctrath has joined #openstack-nova | 18:22 | |
| *** paul-carlton2 has quit IRC | 18:22 | |
| *** SourabhP has quit IRC | 18:24 | |
| *** unicell has joined #openstack-nova | 18:25 | |
| *** sudipto has quit IRC | 18:25 | |
| figleaf | marcin12345: you'd probably get a better response in #openstack. This channel is for nova development. See topic. | 18:26 |
| *** achanda has joined #openstack-nova | 18:27 | |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 18:28 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 18:28 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Remove releasenotes/build between releasenotes runs https://review.openstack.org/268280 | 18:29 |
| mriedem | nic: fyi, your top change is going to be failing the releasenotes job for a bit | 18:30 |
| mriedem | the fixes for that are going through thegate | 18:30 |
| *** vilobhmm11 has joined #openstack-nova | 18:31 | |
| nic | I'll rebase on top of that to prevent noise (yay gerrit!) | 18:31 |
| *** ducttape_ has quit IRC | 18:32 | |
| *** suro-patz has joined #openstack-nova | 18:33 | |
| *** gjayavelu has joined #openstack-nova | 18:33 | |
| *** jed56 has quit IRC | 18:33 | |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 18:33 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 18:33 |
| *** tonytan4ever has joined #openstack-nova | 18:34 | |
| *** dslev has joined #openstack-nova | 18:34 | |
| *** penick has joined #openstack-nova | 18:35 | |
| *** ociuhandu has quit IRC | 18:35 | |
| *** browne has joined #openstack-nova | 18:35 | |
| *** zenoway has joined #openstack-nova | 18:36 | |
| *** penick has quit IRC | 18:36 | |
| *** salv-orlando has quit IRC | 18:37 | |
| *** salv-orlando has joined #openstack-nova | 18:37 | |
| mriedem | nic: i'm going through the tests, so don't touch anything yet | 18:38 |
| mriedem | nic: also, you moved the definition of the create/remove_snap methods in rbd_utils, why? | 18:38 |
| mriedem | that really throws off the diff | 18:38 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 18:39 |
| *** otter768 has joined #openstack-nova | 18:39 | |
| *** suro-patz has quit IRC | 18:39 | |
| *** pradk has quit IRC | 18:39 | |
| mriedem | gah | 18:39 |
| nic | mriedem: sorry, I didn't see your request until after I pushed the typo fixes | 18:39 |
| nic | oops | 18:39 |
| *** suro-patz has joined #openstack-nova | 18:40 | |
| *** ericksonsantos has quit IRC | 18:40 | |
| *** hblixt has quit IRC | 18:40 | |
| *** SourabhP has joined #openstack-nova | 18:40 | |
| *** ericksonsantos has joined #openstack-nova | 18:40 | |
| *** hblixt has joined #openstack-nova | 18:40 | |
| *** achanda has quit IRC | 18:40 | |
| mriedem | i'll re-apply my comments | 18:41 |
| mriedem | i'm going to mark it -WIP until i'm through it | 18:41 |
| nic | OK | 18:41 |
| nic | Thanks | 18:41 |
| *** chhavi has quit IRC | 18:41 | |
| nic | I didn't entirely intend for those methods to move; the dependent change defined the same methods independently, but with less functionality | 18:42 |
| nic | I can try to move them around for Gerrit's sake if you think it would help | 18:42 |
| mriedem | nic: it looks like brand new code otherwise, | 18:43 |
| *** zenoway has quit IRC | 18:43 | |
| mriedem | so it's probably adding ~100 LOC to the change | 18:43 |
| *** ihrachys_ has joined #openstack-nova | 18:43 | |
| mriedem | i really just want to see the diff | 18:43 |
| nic | I'll move them around to make Gerrit pretty | 18:43 |
| mriedem | probably also a reason why people are scared of reviewing this | 18:43 |
| mriedem | ok, but don't push up any changes yet | 18:43 |
| mriedem | until i drop the -W | 18:43 |
| nic | But I'll defer pushing it until it won't cause "gah" | 18:43 |
| *** otter768 has quit IRC | 18:44 | |
| *** dave-mcc_ has joined #openstack-nova | 18:45 | |
| *** jlk has joined #openstack-nova | 18:45 | |
| *** ihrachys has quit IRC | 18:46 | |
| jlk | Quick question on nova-compute. Lets say we're not using DiskFtiler, so an instance with a sizeable disk gets scheduled to a host that does not have enough free space to satisfy the request, does nova-compute on the compute host realize this and kick back the build, or does it attempt to do the build and eventually fails with a no free space kind of error? | 18:46 |
| lascii | jlk: the resource tracker should fail when making the claim and kick back the build | 18:48 |
| *** thumpba has joined #openstack-nova | 18:48 | |
| *** dave-mccowan has quit IRC | 18:48 | |
| jlk | that's the resource tracker on nova-compute ? | 18:48 |
| lascii | yep | 18:48 |
| *** femnad has quit IRC | 18:48 | |
| jlk | thanks | 18:48 |
| mriedem | weren't there some fixes for this in liberty? | 18:48 |
| mriedem | jlk is on kilo | 18:48 |
| *** lpetrut has joined #openstack-nova | 18:48 | |
| *** atuvenie has quit IRC | 18:49 | |
| lascii | very possible, but I don't remember anything specific | 18:49 |
| lascii | he's also not using DiskFilter apparently which is silly | 18:49 |
| jlk | well, shouldn't DiskFilter be a default then? | 18:49 |
| lascii | I thought it was | 18:50 |
| mriedem | it was in liberty | 18:50 |
| mriedem | not kilo | 18:50 |
| mriedem | i was thinking of bug https://bugs.launchpad.net/nova/+bug/1358566 | 18:50 |
| openstack | Launchpad bug 1358566 in OpenStack Compute (nova) "Disk filter should not filter for boot volume" [Low,In progress] - Assigned to Luo Gangyi (luogangyi) | 18:50 |
| jlk | lascii: it's not a default filter for Kilo and older | 18:50 |
| mriedem | yeah | 18:50 |
| mriedem | we don't change defaults in backports | 18:50 |
| lascii | gotcha | 18:51 |
| *** pratikmallya has joined #openstack-nova | 18:51 | |
| mriedem | https://github.com/openstack/nova/commit/6739df76ece9128e895bc13499ed8e3bf172d67e | 18:51 |
| mriedem | a couple of handsome dudes +2d that | 18:51 |
| lascii | heh | 18:52 |
| *** ericksonsantos has quit IRC | 18:53 | |
| openstackgerrit | Andrey Pavlov proposed openstack/nova: remove EC2 legacy code https://review.openstack.org/266425 | 18:54 |
| *** achanda has joined #openstack-nova | 18:54 | |
| *** lpetrut has quit IRC | 18:56 | |
| openstackgerrit | Chuck Carmack proposed openstack/nova: REST API changes for user settable server description https://review.openstack.org/254950 | 18:57 |
| *** ihrachys_ has quit IRC | 18:58 | |
| ccarmack | Has anyone been having problems with gate-nova-releasenotes? I'm getting errors - http://logs.openstack.org/50/254950/6/check/gate-nova-releasenotes/9488e18/console.html | 18:59 |
| ccarmack | mriedem? ^ | 19:00 |
| nic | There's a fix going through now: https://review.openstack.org/268280 | 19:00 |
| ccarmack | oh, ok.. thanks | 19:00 |
| *** penick has joined #openstack-nova | 19:00 | |
| *** haomaiwang has quit IRC | 19:01 | |
| *** browne has quit IRC | 19:01 | |
| *** mlavalle has joined #openstack-nova | 19:01 | |
| *** haomaiwa_ has joined #openstack-nova | 19:01 | |
| mriedem | that's not the fix | 19:01 |
| *** ihrachys has joined #openstack-nova | 19:01 | |
| mriedem | https://review.openstack.org/#/c/268218/ | 19:01 |
| *** shinobu-nova has quit IRC | 19:01 | |
| mriedem | or https://review.openstack.org/#/c/268196/ - which merges first | 19:01 |
| mriedem | *whichever | 19:02 |
| mlavalle | johnthetubaguy: ping | 19:02 |
| *** rlrossit has quit IRC | 19:02 | |
| ccarmack | ok, mriedem | 19:03 |
| *** zenoway has joined #openstack-nova | 19:06 | |
| mriedem | nic: ok https://review.openstack.org/#/c/205282/ | 19:07 |
| mriedem | nic: there seems to be a serious lack of unit test coverage on the imagebackend changes | 19:07 |
| *** rlrossi__ has joined #openstack-nova | 19:07 | |
| *** rlrossi__ has quit IRC | 19:07 | |
| *** ducttape_ has joined #openstack-nova | 19:11 | |
| *** jichen has joined #openstack-nova | 19:14 | |
| *** ctrath has quit IRC | 19:14 | |
| *** thangp has left #openstack-nova | 19:16 | |
| *** ccarmack has left #openstack-nova | 19:19 | |
| *** ctrath has joined #openstack-nova | 19:20 | |
| *** rlrossit has joined #openstack-nova | 19:21 | |
| *** carey has joined #openstack-nova | 19:23 | |
| openstackgerrit | jichenjc proposed openstack/nova: Check host status before scheduler https://review.openstack.org/216759 | 19:28 |
| *** achanda has quit IRC | 19:28 | |
| *** carey has quit IRC | 19:28 | |
| openstackgerrit | jichenjc proposed openstack/nova: Prevent boot if ephemeral disk size > flavor value https://review.openstack.org/213762 | 19:28 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Replace stubs.Set with stub_out (api) https://review.openstack.org/268300 | 19:29 |
| openstackgerrit | jichenjc proposed openstack/nova: Make test cases test_crypto.py from NoDBTestCase https://review.openstack.org/233786 | 19:31 |
| *** ericksonsantos has joined #openstack-nova | 19:32 | |
| *** dshaw has joined #openstack-nova | 19:35 | |
| *** jistr has quit IRC | 19:37 | |
| *** claudiub has quit IRC | 19:37 | |
| *** erlon has joined #openstack-nova | 19:37 | |
| *** gjayavelu has quit IRC | 19:40 | |
| *** tjones has quit IRC | 19:40 | |
| *** achanda has joined #openstack-nova | 19:43 | |
| *** yamahata has quit IRC | 19:44 | |
| *** ccarmack has joined #openstack-nova | 19:48 | |
| *** lucasagomes is now known as lucas-dinner | 19:49 | |
| *** zenoway has quit IRC | 19:51 | |
| *** zenoway has joined #openstack-nova | 19:52 | |
| *** rfolco has quit IRC | 19:54 | |
| bauzas | mriedem: in case you want to bless a padawan https://review.openstack.org/#/c/250101/ | 19:55 |
| *** browne has joined #openstack-nova | 19:56 | |
| *** tdurakov_ has quit IRC | 19:57 | |
| *** tdurakov_ has joined #openstack-nova | 19:57 | |
| superdan | lascii: you care about the host status thing right? | 19:58 |
| superdan | lascii: https://review.openstack.org/#/c/267062/ | 19:58 |
| superdan | that's just the object and db part of the larger patch, that I split out for tojuvone | 19:58 |
| lascii | cool, I'll take a look | 19:58 |
| superdan | since he's working on the api feedback on the top one we keep having to rebase each other | 19:58 |
| superdan | so it'd be nice to land that | 19:59 |
| superdan | just so it's clear that my fingers are clean, | 19:59 |
| superdan | I split the db and object stuff off of his patch above it with just git commands, no code editing :D | 19:59 |
| lascii | nice | 20:00 |
| *** haomaiwa_ has quit IRC | 20:01 | |
| tojuvone | Just getting the last bits. This actually should now be ready: https://review.openstack.org/267062/ | 20:01 |
| *** ankita_wagh has quit IRC | 20:01 | |
| *** 5EXAAQR7M has joined #openstack-nova | 20:01 | |
| tojuvone | But final test stuff going for: https://review.openstack.org/241562/ | 20:01 |
| superdan | lascii: also as I ask this favor I realize my karma quota on lascii patches is probably super small | 20:01 |
| *** ankita_wagh has joined #openstack-nova | 20:01 | |
| superdan | lascii: pick one patch and I will review it presently | 20:02 |
| *** yamahata has joined #openstack-nova | 20:02 | |
| superdan | lascii: choosing, of course, with the knowledge that I know where you live and am on speaking terms with your spousal unit | 20:03 |
| *** achanda has quit IRC | 20:03 | |
| lascii | superdan: heh, https://review.openstack.org/#/c/266573/1 is a reasonable one, as well as the following one if you feel so inclined :) | 20:04 |
| superdan | lascii: you're too night | 20:04 |
| superdan | hah | 20:05 |
| superdan | *nice* | 20:05 |
| mriedem | nic: comments in the bottom change https://review.openstack.org/#/c/187395/ - i haven't looked at tests yet | 20:05 |
| mriedem | nic: and probably won't until i'm good with the code | 20:05 |
| nic | Got igt | 20:05 |
| nic | *got it | 20:05 |
| mriedem | btw, do other cores have any opinion on checking the instance.task_state within the libvirt driver for conditional logic? https://review.openstack.org/#/c/187395/39/nova/virt/libvirt/driver.py | 20:06 |
| *** twinklechawla has quit IRC | 20:06 | |
| lascii | superdan: there's like 8 patches there if you want to really dig in on a friday :) the later ones are WIP but feedback on the approach would be helpful | 20:06 |
| mriedem | looks like some other virt drivers check instance.task_state so probably not a big deal | 20:07 |
| lascii | superdan: I'm also home alone for the weekend, and just got a new desk, so I'm in a really pleasant mood | 20:07 |
| superdan | lol | 20:07 |
| superdan | lascii: you're easy to please | 20:07 |
| * superdan reserves an other joke about lascii just needing some "new wood" to make him happy | 20:07 | |
| superdan | oh, oops | 20:08 |
| *** aix has quit IRC | 20:08 | |
| lascii | heh, although it's bamboo so technically I can avoid that joke | 20:09 |
| superdan | bamboo is wood, no? | 20:09 |
| superdan | it's also ... shaped appropriately | 20:09 |
| * superdan walks away | 20:09 | |
| superdan | dammit | 20:09 |
| superdan | bamboo is not wood | 20:09 |
| superdan | eff | 20:09 |
| lascii | touche | 20:09 |
| *** achanda has joined #openstack-nova | 20:09 | |
| lascii | it's a grass | 20:09 |
| *** angdraug has quit IRC | 20:09 | |
| superdan | yeah, the joke doesn't work now | 20:10 |
| superdan | now _I_ am in a bad mood! | 20:10 |
| * bauzas always learns | 20:10 | |
| cburgess | lol | 20:10 |
| bauzas | darn, my Friday clothes are still in my closet | 20:10 |
| *** bauzas is now known as bauwser | 20:11 | |
| bauwser | 21:12 not too late for claiming a Friday | 20:11 |
| superdan | good thing I -1d your patch | 20:11 |
| lascii | superdan: glad I could cheer you up :) | 20:11 |
| bauwser | superdan saying he -1'd a patch without telling who, it hurted my heart :p | 20:12 |
| *** tdurakov_ has quit IRC | 20:12 | |
| mriedem | bamboo is a grass huh | 20:13 |
| mriedem | didn't know that | 20:13 |
| bauwser | tomatoes are fruits, right? | 20:13 |
| mriedem | everyone knows that | 20:14 |
| lascii | I've apparently watched too many nature documentaries | 20:14 |
| lascii | superdan: https://review.openstack.org/#/c/267062/7/nova/db/sqlalchemy/models.py does this mean all instance lookups also join service info? | 20:14 |
| *** tdurakov_ has joined #openstack-nova | 20:14 | |
| superdan | lascii: no, because we don't joinedload() it | 20:15 |
| superdan | at least, AFAIK | 20:15 |
| superdan | and then we hit it based on extra_attrs in the instance | 20:15 |
| superdan | which I noted on a previous set as something we could do to optimize, by passing that down to the db api | 20:15 |
| *** zenoway has quit IRC | 20:15 | |
| superdan | I should throw up that patch I guess to make it look like I'm serious | 20:15 |
| *** baoli_ has quit IRC | 20:17 | |
| mriedem | bauwser: +W on https://review.openstack.org/#/c/250101/4 | 20:17 |
| bauwser | mriedem: you made my day | 20:18 |
| *** baoli_ has joined #openstack-nova | 20:18 | |
| superdan | lascii: actually, I think that might all be automatic because of the way we calculate extra cols | 20:19 |
| superdan | so, we are probably joining that when the thing is in expected_attrs properly | 20:20 |
| superdan | I forgot about that for instance | 20:20 |
| lascii | superdan: sqlalchemy docs seem to indicate it won't load just with the backref | 20:20 |
| lascii | okay | 20:20 |
| superdan | yeah, I'm pretty sure on that, | 20:20 |
| superdan | because we have lots of backrefs that don't load until we lazy hit then or joinedload() them | 20:20 |
| superdan | I was thinking we were always hitting this one on lazy load, but I think we're good actually | 20:20 |
| superdan | (sqla lazy load, that is) | 20:20 |
| lascii | the code looks good. what happened to just having compute/api pull the service and not embedding it into instance? | 20:22 |
| superdan | lascii: meaning not having it be a property on the instance// | 20:23 |
| superdan | esh -> ? | 20:23 |
| *** e0ne has quit IRC | 20:23 | |
| lascii | yeah, there was some discussion on ps3 of the original patch around that | 20:23 |
| * superdan looks | 20:23 | |
| *** SourabhP has quit IRC | 20:23 | |
| lascii | I'm not sure I made my point clear there | 20:24 |
| *** dslev has quit IRC | 20:24 | |
| superdan | lascii: that was about getting the actual service object, which he's doing, | 20:25 |
| superdan | lascii: as opposed to calculating the status string deep down low | 20:25 |
| superdan | so this just means we efficiently get the service object when requested, | 20:25 |
| superdan | but the api decides what that means in terms of what to show to the client, | 20:26 |
| superdan | which could change based on microversions in the future | 20:26 |
| *** [1]Thelo has joined #openstack-nova | 20:26 | |
| superdan | so I think this is what I asked for, no? | 20:26 |
| *** e0ne has joined #openstack-nova | 20:26 | |
| lascii | yeah, it's what you asked for. I was thinking of having a compute/api method that could retrieve the service for the instance | 20:27 |
| superdan | the api still needs to talk to the sg api to get the richer status | 20:27 |
| superdan | if we do that, we just end up making two calls to the DB to get it right? | 20:27 |
| *** zenoway has joined #openstack-nova | 20:27 | |
| *** ljxiash has joined #openstack-nova | 20:27 | |
| lascii | yes | 20:28 |
| lascii | I'm motivated by trying to get to the core of what an instance is for the cells scheduling work | 20:28 |
| *** Thelo has quit IRC | 20:28 | |
| *** [1]Thelo is now known as Thelo | 20:28 | |
| lascii | and right now the instance object is a large api for doing lookups | 20:28 |
| lascii | adding service isn't really out of the norm though | 20:29 |
| superdan | well, | 20:29 |
| lascii | I guess what I would eventually like would be a split between the lookup layer and the representation | 20:29 |
| superdan | it is a big api for doing the lookups, that's for sure | 20:29 |
| superdan | at least it's bounded | 20:29 |
| superdan | really? the point of the object was to tie those two together.. why do you want to decouple them? | 20:30 |
| *** dave-mcc_ has quit IRC | 20:30 | |
| superdan | maybe there is some line on which we need to split instance that will make more sense for cells? | 20:30 |
| lascii | yeah, was just thinking that. maybe decoupling isn't what I want, just to split instance | 20:31 |
| lascii | I spent some time this morning trying to determine which subset of instance is necessary for displaying in the api | 20:31 |
| superdan | is service being attached to the instance specifically a problem? | 20:31 |
| lascii | no | 20:32 |
| *** ljxiash has quit IRC | 20:32 | |
| superdan | in cellsv2, service is a global, meaning there's no need to differentiate between a service in two cells or anything | 20:32 |
| superdan | okay | 20:32 |
| lascii | it just got me thinking and I wanted to be vocal | 20:32 |
| *** zenoway has quit IRC | 20:32 | |
| superdan | well, that problem is already kinda annoying because of the existing cells quandry, right? meaning, how (much) to persist in order to satisfy the api contract | 20:33 |
| superdan | I don't know how to make that better without changing the api | 20:33 |
| *** ericksonsantos has quit IRC | 20:33 | |
| superdan | I bet leakypipes would have some ideas on the modeling part | 20:34 |
| lascii | I'm not sure either, maybe compose a few objects. like an Instance object primarily focused on display fields, or fields needed by compute, which could be a field in an InstanceMeta object and things like service could go at that level | 20:35 |
| lascii | random thought that may be terrible | 20:35 |
| *** ericksonsantos has joined #openstack-nova | 20:36 | |
| superdan | meaning a sort of view on the object? | 20:36 |
| lascii | essentially | 20:36 |
| *** ankita_wagh has quit IRC | 20:36 | |
| superdan | I thought about doing that navely really early, | 20:36 |
| superdan | some way to have a "show me what this instance looks like at 1.2" | 20:36 |
| *** ankita_wagh has joined #openstack-nova | 20:36 | |
| * bauwser catches up the conversion | 20:37 | |
| bauwser | conversation even | 20:37 |
| *** davidlenwell has quit IRC | 20:38 | |
| lascii | yeah, "show me an instance for displaying in the api, sending to computes, or with included meta objects like service" | 20:39 |
| superdan | yeah | 20:39 |
| superdan | on the one hand, | 20:39 |
| *** davidlenwell has joined #openstack-nova | 20:39 | |
| superdan | that nicely ties things together, | 20:39 |
| superdan | but on the other, it moves the api responsibility of stability for that representation outside the api | 20:39 |
| superdan | I mean, doing that is effectively what the api does today, | 20:40 |
| *** otter768 has joined #openstack-nova | 20:40 | |
| superdan | provides a view of the internals that is stable | 20:40 |
| lascii | true, that's why my first inclination was to decouple the view from the (current)object | 20:41 |
| lascii | but maybe a split just between instance things and meta things would slightly improve what we have today | 20:42 |
| *** annegentle has quit IRC | 20:42 | |
| lascii | without going as far as to have the instance object know what the api needs | 20:43 |
| *** otter768 has quit IRC | 20:45 | |
| lascii | so move things like service and migration_context into a different view | 20:45 |
| *** dslevin has joined #openstack-nova | 20:45 | |
| superdan | well, | 20:45 |
| *** ankita_wagh has quit IRC | 20:45 | |
| superdan | if the api was more declarative about its models, that would eliminate any problem right? | 20:45 |
| superdan | like, instead of iterating over the object's elements, | 20:45 |
| superdan | putting them in a dict, then removing things the client shouldn't see, etc, | 20:45 |
| *** browne has quit IRC | 20:46 | |
| superdan | if it really built the dict explicitly, then what does it matter what is in the object that it doesn't care about? | 20:46 |
| *** schwicke has quit IRC | 20:47 | |
| lascii | the short answer is that it doesn't. the longer answer is that I want to create a minimal instance in https://review.openstack.org/#/c/263927/1/nova/objects/build_request.py and it would be nice to have a spec to adhere to and test against | 20:48 |
| vilobhmm11 | superdan : I have all the changes for sg ready can you please check whenever free https://review.openstack.org/#/c/235637/7 atleast if you are ok with the approach taken | 20:49 |
| superdan | what if we pulled things out of the instance that are already in the request spec and can't change? | 20:49 |
| superdan | vilobhmm11: I'm in like three conversations right now, so no time at the moment | 20:49 |
| lascii | superdan: that only solves part of it, there's more needed by the instance than just what's in req_spec | 20:50 |
| bauwser | lascii: had a question here https://review.openstack.org/#/c/254434/8 | 20:50 |
| vilobhmm11 | superdan : whenever free | 20:50 |
| *** ctrath has quit IRC | 20:50 | |
| superdan | lascii: sure, I'm just trying to get my head wrapped around where a split would be I guess | 20:51 |
| *** tjones has joined #openstack-nova | 20:51 | |
| *** fawadkhaliq has quit IRC | 20:51 | |
| superdan | lascii: if you want I will fix my nit here: https://review.openstack.org/#/c/266573/1/nova/api/openstack/compute/legacy_v2/servers.py | 20:52 |
| superdan | to make myself feel better | 20:52 |
| lascii | heh, sure. putting them on the same line was on purpose, but the spacing is just embarassing | 20:53 |
| *** fawadkhaliq has joined #openstack-nova | 20:53 | |
| superdan | okay | 20:54 |
| *** artom_ has quit IRC | 20:54 | |
| superdan | buh-booom | 20:55 |
| lascii | superdan: I'll keep working and see if a natural split occurs to me. right now what I have is that http://paste.openstack.org/show/484043/ is the minimal set of fields necessary on an instance for the api | 20:55 |
| openstackgerrit | Dan Smith proposed openstack/nova: Create filter_properties earlier in boot request https://review.openstack.org/254434 | 20:55 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Populate instance_mappings during boot. https://review.openstack.org/263925 | 20:55 |
| openstackgerrit | Dan Smith proposed openstack/nova: Persist the request spec during an instance boot https://review.openstack.org/258628 | 20:55 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Map instances to cell0 on failed schedule https://review.openstack.org/267827 | 20:55 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Create instance mapping when instance is scheduled https://review.openstack.org/267828 | 20:55 |
| *** fawadkhaliq has quit IRC | 20:55 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Parse availability_zone in API https://review.openstack.org/266573 | 20:55 |
| openstackgerrit | jichenjc proposed openstack/nova: WIP:remove OS-EXT-IPS and OS-EXT-IPS-MAC extension https://review.openstack.org/266533 | 20:56 |
| bauwser | superdan: lascii: I was having a bit of concerns in https://review.openstack.org/254434 | 20:56 |
| *** ctrath has joined #openstack-nova | 20:56 | |
| bauwser | not sure you saw that | 20:56 |
| superdan | lascii: cool | 20:56 |
| lascii | and my thinking right now is just that if an api change starts to require a new field I'd like a test against my buildreqeust object method to fail | 20:57 |
| *** ihrachys_ has joined #openstack-nova | 20:57 | |
| superdan | lascii: ah | 20:57 |
| *** ihrachys has quit IRC | 20:57 | |
| lascii | but maybe it's just an education thing as well, that api requests may be serviced by a buildrequest object method so that needs to be checked | 20:57 |
| superdan | lascii: but surely you wouldn't want to have to move things from object to buildrequest any time the api starts wanting to expose it right? | 20:57 |
| *** apoorvad has quit IRC | 20:57 | |
| lascii | bauwser: saw that, will respond in a sec | 20:57 |
| superdan | lascii: I feel like what you want is better api definition of what is available before and after scheduling | 20:58 |
| *** ihrachys_ has quit IRC | 20:58 | |
| superdan | lascii: right now it's undefined and assumed to be "everything" | 20:58 |
| bauwser | lascii: ack, it's just for making sure we don't drop that :) | 20:58 |
| *** ihrachys has joined #openstack-nova | 20:58 | |
| superdan | lascii: like really, what else should be available before scheduling besides metadata, uuid, display_name, and those sorts of things | 20:59 |
| superdan | ? | 20:59 |
| lascii | superdan: that's very much what I would like, but I'm not sure we can change much there since we have to keep the old api working | 20:59 |
| superdan | lascii: well, perhaps, but I'd hate to do things internally because of that lack of definition | 20:59 |
| lascii | superdan: yeah, agreed | 20:59 |
| superdan | lascii: we have refined those assumptions before | 20:59 |
| superdan | we could probably do it for this | 20:59 |
| superdan | ffs, we're pretending that changing the url is okay :) | 21:00 |
| * superdan jabs sdague | 21:00 | |
| lascii | heh | 21:00 |
| *** 5EXAAQR7M has quit IRC | 21:01 | |
| *** angdraug has joined #openstack-nova | 21:01 | |
| auggy | lol | 21:01 |
| *** vladikr has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-nova | 21:01 | |
| *** ankita_wagh has joined #openstack-nova | 21:01 | |
| *** achanda has quit IRC | 21:02 | |
| *** zenoway has joined #openstack-nova | 21:03 | |
| *** jaosorior has quit IRC | 21:05 | |
| *** ihrachys_ has joined #openstack-nova | 21:05 | |
| *** ihrachys has quit IRC | 21:05 | |
| *** tjones has quit IRC | 21:07 | |
| *** tjones has joined #openstack-nova | 21:07 | |
| *** zenoway has quit IRC | 21:08 | |
| *** Marga_ has quit IRC | 21:09 | |
| *** Marga__ has joined #openstack-nova | 21:09 | |
| *** smatzek has quit IRC | 21:10 | |
| *** cdelatte has quit IRC | 21:10 | |
| *** tjones has quit IRC | 21:10 | |
| *** dshaw has quit IRC | 21:11 | |
| bauwser | mriedem: btw. thanks for catching https://review.openstack.org/#/c/268218/ | 21:12 |
| mriedem | bauwser: dhellmann held my hand | 21:12 |
| mriedem | https://etherpad.openstack.org/p/fix-nova-release-notes-build | 21:13 |
| *** tjones has joined #openstack-nova | 21:13 | |
| *** marcin12345 has quit IRC | 21:13 | |
| *** dane-fichter has quit IRC | 21:13 | |
| ildikov | superdan: mriedem: hi | 21:15 |
| * superdan runs for the hills | 21:15 | |
| ildikov | superdan: mriedem: can you take a look at the first patch of the multiattach chain: https://review.openstack.org/#/c/193134 ? | 21:15 |
| ildikov | it's the detach part to send attachment_id to Cinder | 21:16 |
| * ildikov is looking at superdan with the cat eyes from Shrek :) | 21:16 | |
| * mriedem is reviewing superdan's https://review.openstack.org/#/c/251588/ atm | 21:16 | |
| superdan | ildikov: hah | 21:16 |
| * ildikov always had timing issues :S | 21:17 | |
| ildikov | superdan: well, if you hate cats, then please ignore what I wrote :) | 21:18 |
| mriedem | superdan only hates the cat he owns | 21:18 |
| superdan | mriedem: true story | 21:18 |
| ildikov | lol :) | 21:18 |
| superdan | he's an ass | 21:18 |
| *** dslevin has quit IRC | 21:19 | |
| ildikov | yeah, cats can be | 21:19 |
| ildikov | I don't have any as it would starve to death, when I'm travelling and my mom's cat is cool, but I'm not sure I would have the same opinion if I would live with it 24/7 :) | 21:20 |
| *** edmondsw has joined #openstack-nova | 21:20 | |
| ildikov | superdan: sorry about your cat :) | 21:21 |
| *** edmondsw has quit IRC | 21:21 | |
| *** ivase has joined #openstack-nova | 21:21 | |
| superdan | heh | 21:21 |
| *** carey has joined #openstack-nova | 21:24 | |
| superdan | ildikov: what about that patch? | 21:24 |
| *** n0ano has quit IRC | 21:24 | |
| ildikov | superdan: it fixes volume detach | 21:25 |
| ildikov | Cinder needs the attachment_id to identify what to detach, so if we pass it in Mitaka then allowing attaching multiple volumes in Newton should be fine and safe from all perspective | 21:27 |
| superdan | okay that's a really thick patch for me to look at at the moment | 21:28 |
| *** doug-fish has quit IRC | 21:28 | |
| *** zenoway has joined #openstack-nova | 21:28 | |
| *** apoorvad has joined #openstack-nova | 21:29 | |
| *** salv-orlando has quit IRC | 21:29 | |
| *** carey has quit IRC | 21:29 | |
| *** salv-orlando has joined #openstack-nova | 21:29 | |
| *** mdnadeem has joined #openstack-nova | 21:29 | |
| *** daemontool has joined #openstack-nova | 21:31 | |
| *** zenoway has quit IRC | 21:31 | |
| notmorgan | superdan, sdague: https://github.com/openstack/nova/commit/676a53ce44a5624a553e80bcff339300802d5494 is likely to cause issues down the line depending on how session ends up working. convering to KSA we now call session.Session twice, i worry that mocking session.Session is a poor choice here since it means that if loader changes what the session does, we break your unit tests | 21:33 |
| notmorgan | superdan, sdague: not sure how to fix this short of breaking the magic mock completely. - i don't want to accidently break your unit tests in the future because we changed internal workings of the loader | 21:34 |
| mriedem | superdan: before i get further, had a question in here https://review.openstack.org/#/c/251588/26/nova/compute/rpcapi.py | 21:34 |
| notmorgan | trying to un-conflict monty's keystoneclient -> keystoneauth patch | 21:34 |
| superdan | notmorgan: I don't know much about that stuff, but not breaking sounds good | 21:35 |
| notmorgan | superdan: i'm inclined to just remove the check on how often session is called | 21:35 |
| notmorgan | superdan: in your test that is. | 21:35 |
| notmorgan | s/your/nova's | 21:35 |
| *** mdnadeem has quit IRC | 21:36 | |
| notmorgan | checking how oft. barbican is called is reasonable. | 21:36 |
| notmorgan | since nova has control over that. | 21:36 |
| superdan | mriedem: good catch | 21:37 |
| mriedem | superdan: i'm sort of curious as to why the multinode job didn't fail | 21:38 |
| mriedem | well, it did | 21:38 |
| mriedem | but didn't seem to hit tihs | 21:38 |
| mriedem | superdan: however, it does hit this http://logs.openstack.org/88/251588/26/check/gate-tempest-dsvm-multinode-full/748c334/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-01-15_18_55_19_856 | 21:38 |
| superdan | hmm, yeah not sure why I haven't noticed that | 21:39 |
| superdan | but yeah, needs fixing | 21:39 |
| ildikov | superdan: if you could take a look at it before the deadline that would be pretty awesome | 21:40 |
| superdan | ildikov: I'll try | 21:40 |
| superdan | ildikov: I'm out all next week though, except for monday | 21:40 |
| mriedem | was just going to say that | 21:40 |
| superdan | ildikov: and you know, have lots of things :/ | 21:40 |
| mriedem | heh | 21:40 |
| ildikov | superdan: the most part of the patch is tests | 21:40 |
| ildikov | superdan: I don't want ot get this feature delayed for two cycles | 21:41 |
| *** gjayavelu has joined #openstack-nova | 21:41 | |
| superdan | I hear ya | 21:41 |
| ildikov | superdan: so pretty please, sugar on top | 21:42 |
| *** cdent has quit IRC | 21:43 | |
| mriedem | superdan: so, this is merged http://logs.openstack.org/88/251588/26/check/gate-tempest-dsvm-multinode-full/748c334/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-01-15_18_55_19_856 | 21:43 |
| openstackgerrit | Morgan Fainberg proposed openstack/nova: Migrate from keystoneclient to keystoneauth https://review.openstack.org/253793 | 21:43 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22AttributeError%3A%20'LibvirtLiveMigrateData'%20object%20has%20no%20attribute%20'get'%5C%22%20AND%20tags%3A%5C%22screen-n-cpu.txt%5C%22&from=7d | 21:43 |
| *** zenoway has joined #openstack-nova | 21:43 | |
| *** thumpba has quit IRC | 21:44 | |
| mriedem | per https://github.com/openstack/nova/commit/69e01758076d0e89eddfe6945c8c7e423c862a49 i think | 21:44 |
| *** ihrachys_ has quit IRC | 21:44 | |
| notmorgan | superdan, sdague (cc mriedem): ^ de-conflict for migrating nova to keystoneauth session. This is needed as keystoneclient session is on the short list to be deprecated and isn't receiving code updates. | 21:44 |
| *** burgerk has quit IRC | 21:46 | |
| *** gjayavelu has quit IRC | 21:46 | |
| *** zenoway has quit IRC | 21:48 | |
| mriedem | reported a bug https://bugs.launchpad.net/nova/+bug/1534837 | 21:49 |
| openstack | Launchpad bug 1534837 in OpenStack Compute (nova) "AttributeError: 'LibvirtLiveMigrateData' object has no attribute 'get'" [High,Confirmed] | 21:49 |
| *** jichen has quit IRC | 21:49 | |
| mriedem | it didn't fail the job b/c the libvirt live migration monitor code swallows the exception | 21:49 |
| *** mragupat has quit IRC | 21:52 | |
| *** tdurakov_ has quit IRC | 21:53 | |
| *** jamielennox|away is now known as jamielennox | 21:54 | |
| *** tdurakov_ has joined #openstack-nova | 21:54 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create filter_properties earlier in boot request https://review.openstack.org/254434 | 21:56 |
| lascii | superdan: bauwser ^ updated the commit message and added two simple tests | 21:56 |
| mriedem | i'll push a fix for this libvirt driver post-live-migration AttributeError | 21:58 |
| mriedem | easy peasy | 21:58 |
| *** tonytan4ever has quit IRC | 21:58 | |
| rlrossit | mriedem: now can you find where it's sucking up my AttributeError in my FixedIP change? | 21:58 |
| *** cdelatte has joined #openstack-nova | 21:59 | |
| *** dims_ has joined #openstack-nova | 21:59 | |
| *** cdelatte has quit IRC | 22:00 | |
| *** cdelatte has joined #openstack-nova | 22:00 | |
| *** haomaiwang has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-nova | 22:01 | |
| *** dims has quit IRC | 22:01 | |
| *** tdurakov_ has quit IRC | 22:03 | |
| mriedem | heh, we have no unit tests for libvirt driver's cleanup method where migrate_data is passed in | 22:05 |
| mriedem | noice! | 22:05 |
| *** lascii is now known as alaski | 22:07 | |
| *** e0ne has quit IRC | 22:08 | |
| rlrossit | mriedem: you sound like me | 22:08 |
| *** dslevin has joined #openstack-nova | 22:08 | |
| *** dslevin has quit IRC | 22:08 | |
| openstackgerrit | Cedric Brandily proposed openstack/nova: VMware: Use datastore copy when the image is already in vSphere https://review.openstack.org/186716 | 22:08 |
| *** tonytan4ever has joined #openstack-nova | 22:09 | |
| *** salv-orl_ has joined #openstack-nova | 22:10 | |
| *** salv-orlando has quit IRC | 22:13 | |
| *** aysyd has quit IRC | 22:19 | |
| *** eharney has quit IRC | 22:19 | |
| *** zenoway has joined #openstack-nova | 22:19 | |
| *** tonytan4ever has quit IRC | 22:22 | |
| *** gjayavelu has joined #openstack-nova | 22:23 | |
| *** zenoway has quit IRC | 22:23 | |
| *** daemontool has quit IRC | 22:24 | |
| *** Sukhdev has joined #openstack-nova | 22:25 | |
| *** thorst has quit IRC | 22:27 | |
| superdan | mriedem: yeah, I'll fix that one up real fast which should fix it | 22:27 |
| *** penick has quit IRC | 22:27 | |
| mriedem | superdan: i have the fix | 22:27 |
| mriedem | 30 sec | 22:27 |
| superdan | oh okay | 22:27 |
| *** pratikmallya has quit IRC | 22:27 | |
| *** suro-patz has quit IRC | 22:27 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: handle migrate_data as object in cleanup method https://review.openstack.org/268352 | 22:28 |
| mriedem | kabam ^ | 22:28 |
| *** daemontool has joined #openstack-nova | 22:28 | |
| superdan | um | 22:29 |
| superdan | oh, right, right | 22:29 |
| *** dimtruck is now known as zz_dimtruck | 22:29 | |
| rlrossit | superdan: uhhh... that change helps me realize I may have been doing something wrong in my DictCompat changes... | 22:31 |
| *** ducttape_ has quit IRC | 22:31 | |
| rlrossit | I was swapping thing.get(foo, default) with getattr(thing, foo, default) | 22:32 |
| rlrossit | but those aren't the same because getattr() only covers your butt with AttributeError, not NotImplementedError | 22:32 |
| *** achanda has joined #openstack-nova | 22:33 | |
| *** eharney has joined #openstack-nova | 22:33 | |
| *** suro-patz has joined #openstack-nova | 22:34 | |
| *** mlavalle has quit IRC | 22:36 | |
| *** daemontool has quit IRC | 22:37 | |
| *** achanda has quit IRC | 22:37 | |
| *** ijw has quit IRC | 22:38 | |
| superdan | ah, yeah | 22:38 |
| *** tdurakov_ has joined #openstack-nova | 22:38 | |
| mriedem | ildikov: so what i'm curious about is https://review.openstack.org/#/c/266605/ | 22:40 |
| *** tdurakov_ has quit IRC | 22:40 | |
| mriedem | and it depends on https://review.openstack.org/#/c/193135/ | 22:40 |
| mriedem | ildikov: but https://review.openstack.org/#/c/193135/ says it has a dependency which isn't merging | 22:40 |
| mriedem | so i'm not sure if you need to rebase that entire series or what | 22:40 |
| mriedem | but i'd like to see that tempest test passing | 22:40 |
| openstackgerrit | Dan Smith proposed openstack/nova: Allow sending the migrate data objects over the wire https://review.openstack.org/251588 | 22:40 |
| ildikov | mriedem: Jenkins acts weird for two days | 22:40 |
| ildikov | regarding these merge conflicts | 22:40 |
| superdan | mriedem: ^ | 22:40 |
| *** otter768 has joined #openstack-nova | 22:41 | |
| superdan | wait, I missed a comment | 22:41 |
| ildikov | mriedem: I rebased it a few hours ago, without any issues, but I can upload a fresh version if that's needed | 22:41 |
| mriedem | superdan: be advised, i reviewed like 5% of that patch :) | 22:41 |
| *** eharney has quit IRC | 22:42 | |
| superdan | mriedem: np.. I covered your other comment | 22:42 |
| superdan | mriedem: it's fine.. starting monday at 1700, I don't care anymore | 22:42 |
| mriedem | ildikov: well this is the one that failed to merge https://review.openstack.org/#/c/267587/2 | 22:43 |
| mriedem | superdan: right | 22:43 |
| *** ccarmack has quit IRC | 22:43 | |
| ildikov | mriedem: I marked this https://review.openstack.org/#/c/193135/ as WIP as my assumption that it will not make it into Mitaka, but I need https://review.openstack.org/#/c/193134/ , which is super green | 22:43 |
| ildikov | mriedem: no merge conflicts locally, but then I will upload the fresh version now | 22:43 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/193135 | 22:44 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Prep Nova to allow multiple volume attachments https://review.openstack.org/193134 | 22:44 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: comp: Allow volume multiattach https://review.openstack.org/267587 | 22:44 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 22:44 |
| *** suro-patz has quit IRC | 22:44 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: virt: Pass host when call attach to Cinder https://review.openstack.org/256273 | 22:44 |
| *** rlrossit has quit IRC | 22:44 | |
| ildikov | mriedem: ^ | 22:44 |
| openstackgerrit | Dan Smith proposed openstack/nova: Allow sending the migrate data objects over the wire https://review.openstack.org/251588 | 22:45 |
| *** otter768 has quit IRC | 22:45 | |
| *** tdurakov_ has joined #openstack-nova | 22:46 | |
| mriedem | alright, | 22:46 |
| mriedem | well, | 22:46 |
| mriedem | i've had a headache all day and it's almost 5 here so i'm calling it a week | 22:46 |
| *** mriedem has quit IRC | 22:47 | |
| *** ctrath has quit IRC | 22:47 | |
| *** zenoway has joined #openstack-nova | 22:48 | |
| *** pratikmallya has joined #openstack-nova | 22:48 | |
| ildikov | it's almost midnight here, so I'm calling it a day too | 22:49 |
| *** ccarmack has joined #openstack-nova | 22:49 | |
| ildikov | have a nice weekend folks | 22:50 |
| *** ctrath has joined #openstack-nova | 22:53 | |
| *** ccarmack has quit IRC | 22:53 | |
| *** gjayavelu has quit IRC | 22:54 | |
| *** ctrath has quit IRC | 22:58 | |
| *** ankita_wagh has quit IRC | 22:59 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** burt has quit IRC | 23:01 | |
| *** haomaiwang has joined #openstack-nova | 23:01 | |
| *** tdurakov_ has quit IRC | 23:01 | |
| *** Sukhdev has quit IRC | 23:03 | |
| *** pratikmallya has quit IRC | 23:04 | |
| *** mdrabe has quit IRC | 23:05 | |
| *** konetzed has joined #openstack-nova | 23:06 | |
| *** thorst has joined #openstack-nova | 23:07 | |
| konetzed | Anyone know if you can just move a instance between availability zones by just updating instances.availability_zone in the nova database? | 23:07 |
| konetzed | or if there are other tables that need to be updated? | 23:07 |
| *** ijw has joined #openstack-nova | 23:09 | |
| *** ijw has quit IRC | 23:09 | |
| *** ijw has joined #openstack-nova | 23:10 | |
| *** ijw has quit IRC | 23:10 | |
| *** ijw has joined #openstack-nova | 23:11 | |
| *** ijw has quit IRC | 23:11 | |
| *** thorst has quit IRC | 23:11 | |
| *** ijw has joined #openstack-nova | 23:11 | |
| *** ijw has quit IRC | 23:12 | |
| *** ijw has joined #openstack-nova | 23:12 | |
| *** ijw has quit IRC | 23:13 | |
| *** gongysh has joined #openstack-nova | 23:13 | |
| *** ijw has joined #openstack-nova | 23:13 | |
| *** ijw has quit IRC | 23:14 | |
| *** ijw has joined #openstack-nova | 23:14 | |
| *** ijw has quit IRC | 23:15 | |
| konetzed | sorry didnt see the topic, i asked my question in the correct room | 23:16 |
| konetzed | exit | 23:16 |
| *** konetzed has quit IRC | 23:16 | |
| *** ijw has joined #openstack-nova | 23:16 | |
| *** ijw has quit IRC | 23:17 | |
| *** ijw has joined #openstack-nova | 23:17 | |
| *** ijw has quit IRC | 23:18 | |
| *** ijw has joined #openstack-nova | 23:18 | |
| *** ijw has quit IRC | 23:18 | |
| *** penick has joined #openstack-nova | 23:19 | |
| *** ijw has joined #openstack-nova | 23:19 | |
| *** ijw has quit IRC | 23:19 | |
| *** ijw has joined #openstack-nova | 23:20 | |
| *** ijw has quit IRC | 23:20 | |
| *** ijw has joined #openstack-nova | 23:21 | |
| *** ivase has quit IRC | 23:21 | |
| *** ijw has quit IRC | 23:21 | |
| *** andreykurilin__ has joined #openstack-nova | 23:21 | |
| *** ivase has joined #openstack-nova | 23:22 | |
| *** ijw has joined #openstack-nova | 23:22 | |
| *** ijw has quit IRC | 23:22 | |
| *** ijw has joined #openstack-nova | 23:22 | |
| *** ijw_ has joined #openstack-nova | 23:23 | |
| *** ijw has quit IRC | 23:23 | |
| *** ijw_ has quit IRC | 23:24 | |
| *** ijw has joined #openstack-nova | 23:24 | |
| *** ijw has quit IRC | 23:25 | |
| *** carey has joined #openstack-nova | 23:25 | |
| *** ijw has joined #openstack-nova | 23:25 | |
| *** ijw has quit IRC | 23:26 | |
| *** ivase has quit IRC | 23:26 | |
| *** ijw has joined #openstack-nova | 23:26 | |
| *** ijw has quit IRC | 23:27 | |
| *** ijw has joined #openstack-nova | 23:27 | |
| *** ijw has quit IRC | 23:28 | |
| *** ijw_ has joined #openstack-nova | 23:28 | |
| *** zenoway has quit IRC | 23:29 | |
| *** ijw_ has quit IRC | 23:29 | |
| *** ijw has joined #openstack-nova | 23:29 | |
| *** dims_ has quit IRC | 23:29 | |
| *** ivase has joined #openstack-nova | 23:29 | |
| *** ijw has quit IRC | 23:29 | |
| *** xyang1 has quit IRC | 23:29 | |
| *** pratikmallya has joined #openstack-nova | 23:29 | |
| *** ijw has joined #openstack-nova | 23:30 | |
| *** ijw has quit IRC | 23:30 | |
| *** carey has quit IRC | 23:30 | |
| *** ijw has joined #openstack-nova | 23:31 | |
| *** baoli_ has quit IRC | 23:31 | |
| *** ijw has quit IRC | 23:31 | |
| *** baoli has joined #openstack-nova | 23:31 | |
| *** penick has quit IRC | 23:32 | |
| *** ijw_ has joined #openstack-nova | 23:33 | |
| *** ijw_ has quit IRC | 23:33 | |
| *** ivase has quit IRC | 23:33 | |
| *** ijw has joined #openstack-nova | 23:33 | |
| *** achanda has joined #openstack-nova | 23:34 | |
| *** suro-patz has joined #openstack-nova | 23:34 | |
| *** ijw_ has joined #openstack-nova | 23:35 | |
| *** ijw has quit IRC | 23:35 | |
| *** ijw_ has quit IRC | 23:35 | |
| *** ducttape_ has joined #openstack-nova | 23:35 | |
| *** ijw has joined #openstack-nova | 23:35 | |
| *** penick has joined #openstack-nova | 23:36 | |
| *** ijw has quit IRC | 23:36 | |
| *** ijw has joined #openstack-nova | 23:36 | |
| *** dims has joined #openstack-nova | 23:37 | |
| *** ijw has quit IRC | 23:37 | |
| *** baoli has quit IRC | 23:37 | |
| *** ijw has joined #openstack-nova | 23:37 | |
| *** figleaf is now known as edleafe | 23:38 | |
| *** ijw has quit IRC | 23:38 | |
| *** ijw has joined #openstack-nova | 23:38 | |
| *** tjones has quit IRC | 23:38 | |
| *** ijw has quit IRC | 23:39 | |
| *** ijw has joined #openstack-nova | 23:39 | |
| *** achanda has quit IRC | 23:39 | |
| *** zenoway has joined #openstack-nova | 23:41 | |
| *** penick has quit IRC | 23:42 | |
| *** baoli has joined #openstack-nova | 23:43 | |
| *** zenoway has quit IRC | 23:45 | |
| *** mrkz has quit IRC | 23:50 | |
| *** jschlueter has quit IRC | 23:54 | |
| *** gongysh has quit IRC | 23:54 | |
| *** dims has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!