| *** xyang2 has quit IRC | 00:01 | |
| *** achanda has joined #openstack-nova | 00:01 | |
| *** salv-orl_ has quit IRC | 00:03 | |
| *** Alexandra_ is now known as dews | 00:07 | |
| *** dews is now known as dewsday | 00:07 | |
| *** baoli has quit IRC | 00:10 | |
| openstackgerrit | Michael Still proposed openstack/nova: libvirt: qemu-img convert should be skipped when migrating https://review.openstack.org/138979 | 00:11 |
|---|---|---|
| *** jecarey has joined #openstack-nova | 00:13 | |
| *** tjones1 has quit IRC | 00:14 | |
| *** agireud has quit IRC | 00:15 | |
| *** markvoelker has joined #openstack-nova | 00:15 | |
| *** zul has joined #openstack-nova | 00:16 | |
| *** vishwanathj has quit IRC | 00:16 | |
| *** xyang2 has joined #openstack-nova | 00:17 | |
| *** achanda has quit IRC | 00:18 | |
| *** xyang2 has quit IRC | 00:18 | |
| *** markvoelker has quit IRC | 00:21 | |
| *** mdrabe has joined #openstack-nova | 00:21 | |
| *** vilobhmm has quit IRC | 00:22 | |
| *** achanda has joined #openstack-nova | 00:25 | |
| *** tsekiyam_ has joined #openstack-nova | 00:29 | |
| *** skylerberg has quit IRC | 00:29 | |
| *** meghal has joined #openstack-nova | 00:30 | |
| *** tjones has joined #openstack-nova | 00:31 | |
| *** tsekiyama has quit IRC | 00:32 | |
| *** mdrabe has quit IRC | 00:32 | |
| *** mtanino has quit IRC | 00:32 | |
| *** tsekiyam_ has quit IRC | 00:33 | |
| *** tjones has quit IRC | 00:34 | |
| *** ijw has quit IRC | 00:35 | |
| *** ijw has joined #openstack-nova | 00:36 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Add project id information in os-server-group API https://review.openstack.org/209917 | 00:36 |
| *** mnestratov has quit IRC | 00:36 | |
| *** thedodd has quit IRC | 00:39 | |
| *** agireud has joined #openstack-nova | 00:43 | |
| *** baoli has joined #openstack-nova | 00:43 | |
| openstackgerrit | Michael Still proposed openstack/nova: Fix collection of metadata for a snapshot of a volume-backed instance https://review.openstack.org/170243 | 00:47 |
| openstackgerrit | Michael Still proposed openstack/nova: Consolidate initialization of instance snapshot metadata https://review.openstack.org/198959 | 00:47 |
| openstackgerrit | Michael Still proposed openstack/nova: Simplify interface for creating snapshot of volume-backed instance https://review.openstack.org/198958 | 00:47 |
| *** ssurana has quit IRC | 00:48 | |
| *** agireud has quit IRC | 00:52 | |
| *** jwcroppe has quit IRC | 00:52 | |
| *** jwcroppe has joined #openstack-nova | 00:52 | |
| *** meghal has quit IRC | 00:54 | |
| *** achanda has quit IRC | 00:55 | |
| *** josecastroleon has quit IRC | 00:56 | |
| *** jeblair has quit IRC | 00:56 | |
| *** jwcroppe has quit IRC | 00:57 | |
| *** achanda has joined #openstack-nova | 00:58 | |
| *** baoli has quit IRC | 00:58 | |
| *** boris-42 has quit IRC | 00:58 | |
| *** jeblair has joined #openstack-nova | 00:58 | |
| *** flwang has quit IRC | 00:59 | |
| *** freyes has quit IRC | 00:59 | |
| *** flwang has joined #openstack-nova | 01:00 | |
| *** boris-42 has joined #openstack-nova | 01:00 | |
| *** freyes has joined #openstack-nova | 01:04 | |
| *** ijw_ has joined #openstack-nova | 01:05 | |
| *** ijw has quit IRC | 01:08 | |
| *** echoingumesh has quit IRC | 01:09 | |
| *** echoingumesh has joined #openstack-nova | 01:09 | |
| *** liusheng has joined #openstack-nova | 01:09 | |
| *** vilobhmm has joined #openstack-nova | 01:10 | |
| *** salv-orlando has joined #openstack-nova | 01:11 | |
| *** echoingumesh has quit IRC | 01:13 | |
| *** echoingumesh has joined #openstack-nova | 01:13 | |
| *** wanghua has joined #openstack-nova | 01:15 | |
| *** ijw has joined #openstack-nova | 01:15 | |
| *** humble__ has joined #openstack-nova | 01:15 | |
| *** salv-orlando has quit IRC | 01:18 | |
| *** ijw_ has quit IRC | 01:18 | |
| *** suresh12_ has joined #openstack-nova | 01:20 | |
| *** suresh12 has quit IRC | 01:20 | |
| *** xyang1 has quit IRC | 01:21 | |
| *** wanghua has quit IRC | 01:21 | |
| *** humble__ has quit IRC | 01:22 | |
| *** wanghua has joined #openstack-nova | 01:22 | |
| *** humble__ has joined #openstack-nova | 01:22 | |
| *** apoorvad has quit IRC | 01:24 | |
| *** apoorvad has joined #openstack-nova | 01:26 | |
| *** Marga_ has joined #openstack-nova | 01:26 | |
| *** vilobhmm has quit IRC | 01:26 | |
| *** Marga_ has quit IRC | 01:26 | |
| *** Marga_ has joined #openstack-nova | 01:27 | |
| *** apoorvad has quit IRC | 01:31 | |
| *** dannywilson has quit IRC | 01:31 | |
| *** tjones has joined #openstack-nova | 01:33 | |
| *** zhenguo has joined #openstack-nova | 01:33 | |
| *** thorst has joined #openstack-nova | 01:34 | |
| *** apoorvad has joined #openstack-nova | 01:34 | |
| *** jwcroppe has joined #openstack-nova | 01:34 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move the v3 api_sample functional tests https://review.openstack.org/212238 | 01:35 |
| *** patrickeast has quit IRC | 01:35 | |
| *** veena has quit IRC | 01:35 | |
| *** unicell has quit IRC | 01:35 | |
| edleafe | alex_xu: ^ the first stab at moving the functional tests | 01:36 |
| edleafe | alex_xu: it isn't working yet, as I do not really understand the mechanism that determines how each test find its template and sample. | 01:37 |
| alex_xu | edleafe: cool, thanks, let me help on that | 01:38 |
| edleafe | alex_xu: if you can fix this, I would appreciate it, and hopefully learn a bit about how this is supposed to work. :) | 01:38 |
| edleafe | alex_xu: great! | 01:38 |
| alex_xu | edleafe: :) | 01:38 |
| edleafe | alex_xu: I'm signing off for the evening - see you tomorrow (my time) morning! | 01:38 |
| alex_xu | edleafe: yea, take a rest, see you tomorrow! | 01:38 |
| *** lyanchih has joined #openstack-nova | 01:39 | |
| *** ijw has quit IRC | 01:39 | |
| *** echoingumesh has quit IRC | 01:40 | |
| *** echoingumesh has joined #openstack-nova | 01:41 | |
| *** meghal has joined #openstack-nova | 01:42 | |
| *** meghal1 has joined #openstack-nova | 01:44 | |
| *** meghal has quit IRC | 01:44 | |
| *** echoingumesh has quit IRC | 01:45 | |
| *** Jeffrey4l has joined #openstack-nova | 01:47 | |
| *** angdraug has quit IRC | 01:49 | |
| *** meghal1 has quit IRC | 01:49 | |
| *** achanda has quit IRC | 01:51 | |
| alex_xu | oomichi: I need rebase this patch https://review.openstack.org/#/c/211777/8 ? | 01:52 |
| alex_xu | oomichi: looks like the patch stop at there, do you know what I need to do? | 01:53 |
| oomichi | alex_xu: morning, I can +A after rebasing without another +2 in this case | 01:53 |
| alex_xu | oomichi: morning, thanks. | 01:53 |
| alex_xu | oomichi: but I didn't see any conflict in my local rebase, strange... | 01:54 |
| *** diogogmt has joined #openstack-nova | 01:54 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Move v2.1 code to the main compute directory - remove v3 step3 https://review.openstack.org/211777 | 01:54 |
| openstackgerrit | Alex Xu proposed openstack/nova: Remove 'v3' directory for v2.1 json-schemas https://review.openstack.org/211846 | 01:54 |
| alex_xu | oomichi: ^ it's done | 01:55 |
| *** thorst has quit IRC | 01:55 | |
| oomichi | alex_xu: done, happy to keep sdague and mikal +2s :) | 01:55 |
| alex_xu | oomichi: thanks :) | 01:56 |
| oomichi | alex_xu: np :) | 01:56 |
| *** apoorvad has quit IRC | 01:59 | |
| *** baoli has joined #openstack-nova | 02:01 | |
| *** haomaiwa_ has joined #openstack-nova | 02:02 | |
| *** markvoelker has joined #openstack-nova | 02:05 | |
| *** shyu has joined #openstack-nova | 02:06 | |
| *** Jeffrey4l has quit IRC | 02:13 | |
| *** haomaiwa_ has quit IRC | 02:13 | |
| *** 16WAAIX9E has joined #openstack-nova | 02:14 | |
| *** salv-orl_ has joined #openstack-nova | 02:23 | |
| *** jecarey has quit IRC | 02:26 | |
| *** zzzeek has quit IRC | 02:27 | |
| *** jecarey has joined #openstack-nova | 02:28 | |
| *** kylek3h has joined #openstack-nova | 02:30 | |
| *** salv-orl_ has quit IRC | 02:31 | |
| *** tjones has left #openstack-nova | 02:31 | |
| *** baoli has quit IRC | 02:34 | |
| *** baoli has joined #openstack-nova | 02:34 | |
| openstackgerrit | shihanzhang proposed openstack/nova: Optimize validate_networks https://review.openstack.org/175742 | 02:36 |
| *** tpeoples has joined #openstack-nova | 02:36 | |
| *** achanda has joined #openstack-nova | 02:45 | |
| *** meghal has joined #openstack-nova | 02:45 | |
| *** suresh12_ has quit IRC | 02:49 | |
| *** meghal has quit IRC | 02:49 | |
| *** jwcroppe has quit IRC | 02:50 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Allow resize root_gb to 0 for volume-backed instances https://review.openstack.org/207794 | 02:50 |
| *** jwcroppe has joined #openstack-nova | 02:51 | |
| *** tpeoples has quit IRC | 02:51 | |
| *** venkat_p has joined #openstack-nova | 02:51 | |
| *** tjones has joined #openstack-nova | 02:51 | |
| *** hakimo_ has joined #openstack-nova | 02:52 | |
| *** hakimo has quit IRC | 02:54 | |
| *** humble__ has quit IRC | 02:54 | |
| *** jwcroppe has quit IRC | 02:55 | |
| *** humble__ has joined #openstack-nova | 02:55 | |
| *** wanghua has quit IRC | 02:58 | |
| *** wanghua has joined #openstack-nova | 02:58 | |
| *** 16WAAIX9E has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-nova | 03:01 | |
| *** Marga_ has quit IRC | 03:02 | |
| *** baoli has quit IRC | 03:04 | |
| *** humble__ has quit IRC | 03:04 | |
| *** wanghua has quit IRC | 03:05 | |
| *** wanghua has joined #openstack-nova | 03:05 | |
| *** Marga_ has joined #openstack-nova | 03:12 | |
| *** baoli has joined #openstack-nova | 03:15 | |
| *** otter768 has quit IRC | 03:16 | |
| *** tpeoples has joined #openstack-nova | 03:19 | |
| *** diogogmt has quit IRC | 03:20 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Allow resize root_gb to 0 for volume-backed instances https://review.openstack.org/207794 | 03:24 |
| openstackgerrit | Rui Chen proposed openstack/nova: Set task_state=None when booting instance failed https://review.openstack.org/187145 | 03:25 |
| *** humble_ has joined #openstack-nova | 03:29 | |
| *** wanghua has quit IRC | 03:31 | |
| *** salv-orlando has joined #openstack-nova | 03:33 | |
| *** sudipto_ has joined #openstack-nova | 03:33 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add instance action events for live migration https://review.openstack.org/195372 | 03:36 |
| openstackgerrit | Rui Chen proposed openstack/nova: Fix permission issue of server group API https://review.openstack.org/193117 | 03:43 |
| *** shuquan has joined #openstack-nova | 03:45 | |
| *** salv-orlando has quit IRC | 03:45 | |
| *** meghal has joined #openstack-nova | 03:46 | |
| *** dave-mccowan has quit IRC | 03:46 | |
| *** tjones has quit IRC | 03:50 | |
| *** meghal has quit IRC | 03:50 | |
| *** mili has joined #openstack-nova | 03:51 | |
| *** vilobhmm has joined #openstack-nova | 03:52 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Fix incorrect result of host-describe https://review.openstack.org/211083 | 03:57 |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-nova | 04:02 | |
| *** mili has quit IRC | 04:05 | |
| *** tpeoples has quit IRC | 04:11 | |
| *** lyanchih has quit IRC | 04:16 | |
| *** meghal has joined #openstack-nova | 04:16 | |
| *** unicell has joined #openstack-nova | 04:21 | |
| openstackgerrit | Balaji Narayanan proposed openstack/nova-specs: Snapshot failure makes instance unusable https://review.openstack.org/208078 | 04:22 |
| openstackgerrit | Balaji Narayanan proposed openstack/nova-specs: Snapshot failure makes instance unusable https://review.openstack.org/208078 | 04:23 |
| *** Marga_ has quit IRC | 04:24 | |
| *** tjones has joined #openstack-nova | 04:26 | |
| *** Sree has joined #openstack-nova | 04:30 | |
| oomichi | alex_xu: can you take a look at https://review.openstack.org/#/c/192043/ again ? | 04:30 |
| *** Sree has quit IRC | 04:31 | |
| *** mili has joined #openstack-nova | 04:31 | |
| *** baoli has quit IRC | 04:31 | |
| oomichi | alex_xu: I wrote a response to your question on https://review.openstack.org/#/c/192043/12/nova/network/security_group/neutron_driver.py | 04:31 |
| *** Sree has joined #openstack-nova | 04:31 | |
| *** fawadkhaliq has joined #openstack-nova | 04:32 | |
| *** Sree has quit IRC | 04:34 | |
| *** fawadkhaliq has quit IRC | 04:39 | |
| *** fawadkhaliq has joined #openstack-nova | 04:39 | |
| alex_xu | oomichi: ec2 already code already useless? | 04:44 |
| *** unicell1 has joined #openstack-nova | 04:44 | |
| *** unicell has quit IRC | 04:45 | |
| oomichi | alex_xu: it is difficult to say "ec2 code of nova is not used at any places", but we can say "ec2 code is not maintained in nova" as deprecated mark. | 04:46 |
| *** sudipto_ has quit IRC | 04:47 | |
| *** rolandchan has joined #openstack-nova | 04:47 | |
| *** salv-orlando has joined #openstack-nova | 04:47 | |
| *** rolandchan has left #openstack-nova | 04:47 | |
| alex_xu | oomichi: I'm not sure about that, as we recently fix a bug for ec2 https://bugs.launchpad.net/nova/+bug/1370250 | 04:51 |
| openstack | Launchpad bug 1370250 in OpenStack Compute (nova) "Can not set volume attributes at instance launch by EC2 API" [Medium,Fix committed] - Assigned to Nikola Đipanov (ndipanov) | 04:51 |
| *** shyama has joined #openstack-nova | 04:52 | |
| oomichi | alex_xu: http://www.gossamer-threads.com/lists/openstack/operators/44191 | 04:55 |
| *** fawadkhaliq has quit IRC | 04:57 | |
| alex_xu | oomichi: let me review patch first, then let me check with Sean tonight | 04:58 |
| oomichi | alex_xu: thanks :) | 04:58 |
| alex_xu | oomichi: np :) | 04:58 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Fix collection of metadata for a snapshot of a volume-backed instance https://review.openstack.org/170243 | 04:58 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Consolidate initialization of instance snapshot metadata https://review.openstack.org/198959 | 04:58 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Simplify interface for creating snapshot of volume-backed instance https://review.openstack.org/198958 | 04:58 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Remove unused root_metadata method of BlockDeviceMappingList https://review.openstack.org/212324 | 04:58 |
| *** salv-orlando has quit IRC | 04:59 | |
| openstackgerrit | wanghao proposed openstack/nova: adding a volume detach_force call https://review.openstack.org/184537 | 05:00 |
| *** haomaiwang has quit IRC | 05:01 | |
| *** takedakn has joined #openstack-nova | 05:01 | |
| *** haomaiwang has joined #openstack-nova | 05:01 | |
| *** takedakn has quit IRC | 05:01 | |
| *** takedakn has joined #openstack-nova | 05:03 | |
| *** lyanchih has joined #openstack-nova | 05:04 | |
| *** fawadkhaliq has joined #openstack-nova | 05:06 | |
| *** takedakn has quit IRC | 05:08 | |
| *** jdurgin has joined #openstack-nova | 05:08 | |
| *** jdurgin has quit IRC | 05:08 | |
| *** deepakcs has joined #openstack-nova | 05:08 | |
| *** tjones has quit IRC | 05:11 | |
| *** shuquan has quit IRC | 05:13 | |
| *** vivekd has joined #openstack-nova | 05:13 | |
| *** mili has quit IRC | 05:15 | |
| *** otter768 has joined #openstack-nova | 05:17 | |
| *** kmARC has joined #openstack-nova | 05:19 | |
| *** otter768 has quit IRC | 05:22 | |
| *** chhavi has joined #openstack-nova | 05:25 | |
| *** shuquan has joined #openstack-nova | 05:27 | |
| *** meghal1 has joined #openstack-nova | 05:28 | |
| *** armax has quit IRC | 05:30 | |
| *** atuvenie has joined #openstack-nova | 05:30 | |
| *** kmARC has quit IRC | 05:30 | |
| *** meghal has quit IRC | 05:31 | |
| *** lyanchih has quit IRC | 05:33 | |
| *** Maike has joined #openstack-nova | 05:33 | |
| *** vilobhmm has quit IRC | 05:35 | |
| *** atuvenie has quit IRC | 05:35 | |
| *** hhuang has joined #openstack-nova | 05:37 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Fix evaluation of host disk usage by volume-backed instances https://review.openstack.org/200870 | 05:37 |
| *** hhuang has quit IRC | 05:38 | |
| *** Vikas has joined #openstack-nova | 05:38 | |
| *** hhuang has joined #openstack-nova | 05:39 | |
| *** Vikas has quit IRC | 05:39 | |
| *** shuquan has quit IRC | 05:39 | |
| *** Vikas has joined #openstack-nova | 05:40 | |
| *** markvoelker has quit IRC | 05:41 | |
| *** vilobhmm has joined #openstack-nova | 05:41 | |
| *** ildikov has quit IRC | 05:42 | |
| *** shyama has quit IRC | 05:45 | |
| *** shyama has joined #openstack-nova | 05:46 | |
| *** Vikas has quit IRC | 05:48 | |
| *** fifieldt has joined #openstack-nova | 05:49 | |
| *** med_ has quit IRC | 05:50 | |
| venkat_p | Hi all, | 05:55 |
| *** shuquan has joined #openstack-nova | 05:55 | |
| *** Nikolay_St has joined #openstack-nova | 05:55 | |
| venkat_p | How to do unit test abstract classes, | 05:56 |
| *** shuquan has quit IRC | 06:00 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-nova | 06:02 | |
| openstackgerrit | Chris Friesen proposed openstack/nova: Improve error logs for start/stop of locked instance https://review.openstack.org/210689 | 06:03 |
| *** shuquan has joined #openstack-nova | 06:03 | |
| *** salv-orlando has joined #openstack-nova | 06:04 | |
| *** lyanchih has joined #openstack-nova | 06:08 | |
| venkat_p | i converted https://github.com/openstack/nova/blob/master/nova/scheduler/driver.py#L39 | 06:09 |
| venkat_p | into ABC | 06:09 |
| venkat_p | existing test cases are failing due to this change | 06:10 |
| venkat_p | now, I want to modify unit test cases | 06:10 |
| venkat_p | How to test abstract classes? | 06:10 |
| *** salv-orlando has quit IRC | 06:13 | |
| *** xgsa has joined #openstack-nova | 06:14 | |
| *** cfriesen_ has quit IRC | 06:18 | |
| *** vivekd has quit IRC | 06:18 | |
| *** oomichi has quit IRC | 06:18 | |
| *** Murali_ has joined #openstack-nova | 06:21 | |
| *** meghal1 has quit IRC | 06:21 | |
| *** sudipto has joined #openstack-nova | 06:22 | |
| *** med_ has joined #openstack-nova | 06:23 | |
| *** med_ is now known as Guest471 | 06:23 | |
| *** lyanchih has quit IRC | 06:25 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/212367 | 06:26 |
| *** shuquan has quit IRC | 06:27 | |
| *** praneshp has quit IRC | 06:28 | |
| *** ildikov has joined #openstack-nova | 06:31 | |
| *** praneshp has joined #openstack-nova | 06:32 | |
| *** shuquan has joined #openstack-nova | 06:33 | |
| *** xiaohui has joined #openstack-nova | 06:33 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 06:33 |
| *** vilobhmm has quit IRC | 06:35 | |
| *** sahid has joined #openstack-nova | 06:35 | |
| *** dewsday has quit IRC | 06:36 | |
| *** yamahata has joined #openstack-nova | 06:36 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Re-arrange the unit test directories https://review.openstack.org/212120 | 06:36 |
| *** afazekas has joined #openstack-nova | 06:40 | |
| *** apuimedo has quit IRC | 06:40 | |
| *** markvoelker has joined #openstack-nova | 06:41 | |
| *** rdo has quit IRC | 06:42 | |
| *** oomichi has joined #openstack-nova | 06:42 | |
| *** rdo has joined #openstack-nova | 06:43 | |
| *** Guest471 has quit IRC | 06:44 | |
| *** coolsvap_ has joined #openstack-nova | 06:45 | |
| *** nkrinner has joined #openstack-nova | 06:45 | |
| *** coolsvap_ has quit IRC | 06:46 | |
| *** markvoelker has quit IRC | 06:46 | |
| *** scheuran has joined #openstack-nova | 06:47 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Move quota delta reserve methods from api to utils https://review.openstack.org/212374 | 06:52 |
| *** alexschm has joined #openstack-nova | 06:53 | |
| *** rdopiera has joined #openstack-nova | 06:57 | |
| *** Marga_ has joined #openstack-nova | 07:00 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-nova | 07:02 | |
| *** xgsa has quit IRC | 07:05 | |
| *** xgsa has joined #openstack-nova | 07:07 | |
| *** lyanchih has joined #openstack-nova | 07:07 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Neutron: Check port binding status https://review.openstack.org/156112 | 07:08 |
| openstackgerrit | Eric Brown proposed openstack/nova: Use min and max on IntOpt option types https://review.openstack.org/212381 | 07:08 |
| *** xgsa1 has joined #openstack-nova | 07:10 | |
| *** salv-orlando has joined #openstack-nova | 07:11 | |
| *** xgsa has quit IRC | 07:12 | |
| *** snikitin has quit IRC | 07:12 | |
| *** takedakn has joined #openstack-nova | 07:12 | |
| *** markus_z has joined #openstack-nova | 07:13 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Fixed indentation https://review.openstack.org/205529 | 07:15 |
| *** med_ has joined #openstack-nova | 07:15 | |
| *** med_ is now known as Guest61394 | 07:15 | |
| *** fifieldt has quit IRC | 07:17 | |
| *** otter768 has joined #openstack-nova | 07:18 | |
| openstackgerrit | Eric Brown proposed openstack/nova: Use min and max on IntOpt option types https://review.openstack.org/212381 | 07:18 |
| *** liusheng has left #openstack-nova | 07:20 | |
| *** browne has quit IRC | 07:20 | |
| *** romainh has joined #openstack-nova | 07:22 | |
| *** otter768 has quit IRC | 07:22 | |
| *** Murali__ has joined #openstack-nova | 07:23 | |
| *** mtreinish has quit IRC | 07:24 | |
| *** leseb has quit IRC | 07:24 | |
| *** leseb has joined #openstack-nova | 07:25 | |
| *** Murali_ has quit IRC | 07:25 | |
| *** mwagner_lap has quit IRC | 07:25 | |
| *** shuquan has quit IRC | 07:26 | |
| *** eliqiao has quit IRC | 07:26 | |
| *** eliqiao has joined #openstack-nova | 07:26 | |
| *** shuquan has joined #openstack-nova | 07:27 | |
| *** mwagner_lap has joined #openstack-nova | 07:27 | |
| *** mtreinish has joined #openstack-nova | 07:27 | |
| *** moshele has joined #openstack-nova | 07:31 | |
| *** rajesht has joined #openstack-nova | 07:32 | |
| *** jlanoux has joined #openstack-nova | 07:37 | |
| *** sahid has quit IRC | 07:38 | |
| *** sahid has joined #openstack-nova | 07:39 | |
| *** alex_klimov has joined #openstack-nova | 07:40 | |
| *** Murali__ has quit IRC | 07:41 | |
| *** takedakn has quit IRC | 07:41 | |
| *** praneshp has quit IRC | 07:41 | |
| *** deepakcs has quit IRC | 07:41 | |
| *** RuiChen has quit IRC | 07:41 | |
| *** changbl has quit IRC | 07:41 | |
| *** Anticimex has quit IRC | 07:41 | |
| *** alex_xu has quit IRC | 07:41 | |
| *** openstackgerrit has quit IRC | 07:41 | |
| *** dosaboy has quit IRC | 07:41 | |
| *** setuid has quit IRC | 07:41 | |
| *** hogepodge has quit IRC | 07:41 | |
| *** pedroalvarez has quit IRC | 07:41 | |
| *** ByPasS has quit IRC | 07:41 | |
| *** ekhugen has quit IRC | 07:41 | |
| *** ttx has quit IRC | 07:41 | |
| *** mirrorbox has quit IRC | 07:41 | |
| *** tteggel_ has quit IRC | 07:41 | |
| *** _fortis has quit IRC | 07:41 | |
| *** wuhao has quit IRC | 07:41 | |
| *** Madasi has quit IRC | 07:41 | |
| *** devananda has quit IRC | 07:41 | |
| *** rmstar has quit IRC | 07:41 | |
| *** oomichi has quit IRC | 07:42 | |
| *** Murali__ has joined #openstack-nova | 07:43 | |
| *** takedakn has joined #openstack-nova | 07:43 | |
| *** deepakcs has joined #openstack-nova | 07:43 | |
| *** RuiChen has joined #openstack-nova | 07:43 | |
| *** changbl has joined #openstack-nova | 07:43 | |
| *** Anticimex has joined #openstack-nova | 07:43 | |
| *** alex_xu has joined #openstack-nova | 07:43 | |
| *** openstackgerrit has joined #openstack-nova | 07:43 | |
| *** dosaboy has joined #openstack-nova | 07:43 | |
| *** setuid has joined #openstack-nova | 07:43 | |
| *** hogepodge has joined #openstack-nova | 07:43 | |
| *** pedroalvarez has joined #openstack-nova | 07:43 | |
| *** ByPasS has joined #openstack-nova | 07:43 | |
| *** ekhugen has joined #openstack-nova | 07:43 | |
| *** ttx has joined #openstack-nova | 07:43 | |
| *** mirrorbox has joined #openstack-nova | 07:43 | |
| *** tteggel_ has joined #openstack-nova | 07:43 | |
| *** _fortis has joined #openstack-nova | 07:43 | |
| *** wuhao has joined #openstack-nova | 07:43 | |
| *** Madasi has joined #openstack-nova | 07:43 | |
| *** devananda has joined #openstack-nova | 07:43 | |
| *** rmstar has joined #openstack-nova | 07:43 | |
| *** Murali_ has joined #openstack-nova | 07:43 | |
| *** kiwik-chenrui has joined #openstack-nova | 07:43 | |
| *** kmARC has joined #openstack-nova | 07:44 | |
| *** _fortis has quit IRC | 07:44 | |
| *** Murali__ has quit IRC | 07:44 | |
| *** takedakn has quit IRC | 07:44 | |
| *** ByPasS has quit IRC | 07:45 | |
| *** alex_xu has quit IRC | 07:45 | |
| *** openstackgerrit has quit IRC | 07:45 | |
| *** ttx has quit IRC | 07:45 | |
| *** devananda has quit IRC | 07:45 | |
| *** rmstar has quit IRC | 07:45 | |
| *** mirrorbox has quit IRC | 07:46 | |
| *** mirrorbox has joined #openstack-nova | 07:46 | |
| *** deepakcs has quit IRC | 07:46 | |
| *** RuiChen has quit IRC | 07:46 | |
| *** changbl has quit IRC | 07:46 | |
| *** Anticimex has quit IRC | 07:46 | |
| *** dosaboy has quit IRC | 07:46 | |
| *** setuid has quit IRC | 07:46 | |
| *** hogepodge has quit IRC | 07:46 | |
| *** pedroalvarez has quit IRC | 07:46 | |
| *** ekhugen has quit IRC | 07:46 | |
| *** tteggel_ has quit IRC | 07:46 | |
| *** wuhao has quit IRC | 07:46 | |
| *** Madasi has quit IRC | 07:46 | |
| *** devananda has joined #openstack-nova | 07:47 | |
| *** rmstar has joined #openstack-nova | 07:47 | |
| *** pedroalvarez has joined #openstack-nova | 07:47 | |
| *** Anticimex has joined #openstack-nova | 07:47 | |
| *** mnestratov has joined #openstack-nova | 07:48 | |
| *** dosaboy has joined #openstack-nova | 07:48 | |
| *** tteggel has joined #openstack-nova | 07:48 | |
| *** Marga_ has quit IRC | 07:50 | |
| *** ttx has joined #openstack-nova | 07:50 | |
| *** Marga_ has joined #openstack-nova | 07:50 | |
| *** alex_xu has joined #openstack-nova | 07:51 | |
| *** shuquan has quit IRC | 07:51 | |
| *** Madasi has joined #openstack-nova | 07:51 | |
| *** ByPasS has joined #openstack-nova | 07:52 | |
| *** ttx has quit IRC | 07:52 | |
| *** ttx has joined #openstack-nova | 07:52 | |
| *** ekhugen has joined #openstack-nova | 07:52 | |
| *** _fortis has joined #openstack-nova | 07:53 | |
| *** wuhao has joined #openstack-nova | 07:53 | |
| *** nelsnelson has quit IRC | 07:53 | |
| *** setuid has joined #openstack-nova | 07:54 | |
| *** deepakcs has joined #openstack-nova | 07:58 | |
| *** jcoufal has joined #openstack-nova | 08:00 | |
| *** openstackgerrit has joined #openstack-nova | 08:00 | |
| *** changbl has joined #openstack-nova | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:02 | |
| *** penick has joined #openstack-nova | 08:03 | |
| *** yassine has joined #openstack-nova | 08:04 | |
| *** shuquan has joined #openstack-nova | 08:05 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Add missing rules in policy.json https://review.openstack.org/188235 | 08:07 |
| *** shuquan has quit IRC | 08:12 | |
| *** shuquan has joined #openstack-nova | 08:13 | |
| *** Kennan has quit IRC | 08:14 | |
| *** lucasagomes has joined #openstack-nova | 08:15 | |
| *** ankit_ag has joined #openstack-nova | 08:15 | |
| *** yamahata has quit IRC | 08:17 | |
| *** shuquan has quit IRC | 08:20 | |
| *** Kennan has joined #openstack-nova | 08:21 | |
| *** danpb has joined #openstack-nova | 08:22 | |
| *** lyanchih has quit IRC | 08:23 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove/deprecate unused parts of the compute node object https://review.openstack.org/186757 | 08:24 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Make ComputeNode.service_id nullable to match db schema https://review.openstack.org/169284 | 08:24 |
| *** rdo has quit IRC | 08:24 | |
| *** apuimedo has joined #openstack-nova | 08:25 | |
| *** matrohon has joined #openstack-nova | 08:25 | |
| *** jistr has joined #openstack-nova | 08:26 | |
| *** jcoufal has quit IRC | 08:27 | |
| *** zhenguo has quit IRC | 08:29 | |
| *** atuvenie has joined #openstack-nova | 08:29 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Avoid querying for Service in resource tracker https://review.openstack.org/169288 | 08:30 |
| *** lyanchih has joined #openstack-nova | 08:31 | |
| *** achanda has quit IRC | 08:33 | |
| *** atuvenie has quit IRC | 08:33 | |
| *** jcoufal has joined #openstack-nova | 08:37 | |
| *** shuquan has joined #openstack-nova | 08:38 | |
| *** rdo has joined #openstack-nova | 08:38 | |
| *** shuquan has quit IRC | 08:41 | |
| *** shuquan has joined #openstack-nova | 08:42 | |
| *** markvoelker has joined #openstack-nova | 08:43 | |
| *** hogepodge has joined #openstack-nova | 08:43 | |
| *** penick has quit IRC | 08:43 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Move V2.1 API unittest to top level directory https://review.openstack.org/212120 | 08:44 |
| openstackgerrit | Alex Xu proposed openstack/nova: Move V2 API unittests under legacy_v2 directory https://review.openstack.org/212393 | 08:44 |
| openstackgerrit | Alex Xu proposed openstack/nova: Add V2.1 API tests parity with V2 API tests https://review.openstack.org/212392 | 08:44 |
| alex_xu | edleafe: I separated moving unittest work also. hope it helpful for review | 08:45 |
| alex_xu | sdague: johnthetubaguy ^ | 08:46 |
| *** markvoelker has quit IRC | 08:47 | |
| *** shuquan has quit IRC | 08:48 | |
| *** jcoufal has quit IRC | 08:49 | |
| *** shuquan has joined #openstack-nova | 08:49 | |
| openstackgerrit | Michal Dulko proposed openstack/nova: Replace get_cinder_client_version in cinder.py https://review.openstack.org/192158 | 08:50 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova: Enable getting connection info for MKS consoles https://review.openstack.org/208998 | 08:51 |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Move quota delta reserve methods from api to utils https://review.openstack.org/212374 | 08:59 |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Move quota delta reserve methods from api to utils https://review.openstack.org/212374 | 09:06 |
| openstackgerrit | Edward Hope-Morley proposed openstack/nova: Ensure failed attach cleaned up https://review.openstack.org/212423 | 09:07 |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Fix quota-update when nova-compute startup finish https://review.openstack.org/212427 | 09:12 |
| *** drankis has joined #openstack-nova | 09:12 | |
| *** rodrigods has quit IRC | 09:13 | |
| *** rodrigods has joined #openstack-nova | 09:13 | |
| *** humble__ has joined #openstack-nova | 09:13 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Fix quota-update when nova-compute startup finish https://review.openstack.org/212427 | 09:15 |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Fix quota-update when nova-compute startup finish https://review.openstack.org/170118 | 09:17 |
| *** humble_ has quit IRC | 09:17 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Fix quota-update when nova-compute startup finish https://review.openstack.org/170118 | 09:18 |
| *** otter768 has joined #openstack-nova | 09:19 | |
| *** rushiagr_away is now known as rushiagr | 09:21 | |
| *** otter768 has quit IRC | 09:23 | |
| openstackgerrit | Edward Hope-Morley proposed openstack/nova: Ensure disk unattached if attach error occurs https://review.openstack.org/212423 | 09:27 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/212264 | 09:28 |
| *** jwcroppe has joined #openstack-nova | 09:28 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/212434 | 09:31 |
| *** chhavi has quit IRC | 09:35 | |
| paboldin | Hello there, guys, please respond here https://bugs.launchpad.net/nova/+bug/1246201 | 09:36 |
| openstack | Launchpad bug 1246201 in OpenStack Compute (nova) "Live migration fails when the instance has a config_drive_format=iso9660" [High,Confirmed] - Assigned to lvmxh (shaohef) | 09:36 |
| *** IanGovett has joined #openstack-nova | 09:38 | |
| *** zhenguo has joined #openstack-nova | 09:39 | |
| *** rushiagr is now known as rushiagr_away | 09:41 | |
| markus_z | paboldin: done | 09:42 |
| markus_z | I did forget to mention that you should set it to "In Progress" as soon as you set yourself as assignee to show others that you actively working on a patch set. | 09:44 |
| markus_z | Would be great to have a fix for that. | 09:44 |
| *** exploreshaifali has joined #openstack-nova | 09:46 | |
| *** sudipto has quit IRC | 09:46 | |
| paboldin | markus_z: thanks | 09:48 |
| *** jcoufal has joined #openstack-nova | 09:53 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/128940 | 09:58 |
| *** boris-42 has quit IRC | 10:00 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwa_ has joined #openstack-nova | 10:02 | |
| *** sudipto has joined #openstack-nova | 10:03 | |
| *** IanGovett1 has joined #openstack-nova | 10:07 | |
| *** Maike_ has joined #openstack-nova | 10:08 | |
| *** IanGovett has quit IRC | 10:08 | |
| *** shuquan has quit IRC | 10:09 | |
| *** Maike__ has joined #openstack-nova | 10:10 | |
| *** Maike has quit IRC | 10:11 | |
| *** Maike_ has quit IRC | 10:11 | |
| *** jeblair has quit IRC | 10:15 | |
| *** romainh has quit IRC | 10:15 | |
| *** jeblair has joined #openstack-nova | 10:16 | |
| *** Maike__ has quit IRC | 10:16 | |
| *** lucasagomes has quit IRC | 10:24 | |
| *** Maike has joined #openstack-nova | 10:26 | |
| *** lyanchih has quit IRC | 10:26 | |
| sdague | alex_xu: https://review.openstack.org/#/c/212393/ is still failing on unit tests | 10:27 |
| *** sudipto has quit IRC | 10:31 | |
| *** Maike has quit IRC | 10:34 | |
| *** Maike has joined #openstack-nova | 10:35 | |
| *** dheeraj has joined #openstack-nova | 10:36 | |
| *** Marga_ has quit IRC | 10:39 | |
| *** Marga_ has joined #openstack-nova | 10:39 | |
| *** Murali_ has quit IRC | 10:42 | |
| *** markvoelker has joined #openstack-nova | 10:43 | |
| *** jcoufal_ has joined #openstack-nova | 10:44 | |
| alex_xu | sdague: yea...let me check | 10:47 |
| *** jcoufal has quit IRC | 10:47 | |
| *** markvoelker has quit IRC | 10:48 | |
| *** sudipto has joined #openstack-nova | 10:49 | |
| *** claudiub has joined #openstack-nova | 10:51 | |
| *** Marga_ has quit IRC | 10:57 | |
| *** rubasov has quit IRC | 10:58 | |
| openstackgerrit | whitekid proposed openstack/nova: tuning extended volume extensions https://review.openstack.org/211258 | 10:59 |
| *** Marga_ has joined #openstack-nova | 11:00 | |
| *** exploreshaifali has quit IRC | 11:00 | |
| *** haomaiwa_ has quit IRC | 11:01 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova-specs: spec document for sriov-physical-function-passthrough blueprint https://review.openstack.org/212472 | 11:01 |
| *** romainh has joined #openstack-nova | 11:02 | |
| *** Maike has quit IRC | 11:02 | |
| *** Maike has joined #openstack-nova | 11:02 | |
| *** haomaiwang has joined #openstack-nova | 11:02 | |
| paboldin | markus_z: well, seems like I will need some assistance. | 11:07 |
| markus_z | paboldin: In which way? | 11:07 |
| paboldin | markus_z: I see that there is bailout on non-empty block_device_mapping. This list is a product of `block_device_info_get_mapping'. Does this list includes the instance image itself? | 11:09 |
| alex_xu | sdague: would you like take a like the comment at line 96 at https://review.openstack.org/#/c/192043/12/nova/network/security_group/neutron_driver.py , it's about whether we should care about ec2 api now | 11:10 |
| markus_z | paboldin: I'm sorry, I don't know. The bug mentions other contributors which did already some work there, maybe you could ask them. | 11:12 |
| sdague | I'm not convinced we really want to fix ec2 in tree for this | 11:12 |
| paboldin | markus_z: seems like stpierre is on vacation :-/ | 11:12 |
| sdague | our strategy here is for folks to use the out of tree code | 11:13 |
| sdague | alex_xu: is https://review.openstack.org/#/c/192043 going to break people out there? | 11:15 |
| sdague | like before could they make the network security group call without description and it worked, and now it won't? | 11:15 |
| *** eglynn has quit IRC | 11:15 | |
| *** lucasagomes has joined #openstack-nova | 11:16 | |
| alex_xu | sdague: there are two behaviors when use nova-network and neutron | 11:16 |
| alex_xu | when use nova-network, user must specifiy description | 11:16 |
| alex_xu | sdague: and this api in the begining (neutron not existed yet) behavior let this. | 11:17 |
| alex_xu | sdague: so that means the neutron implementation is wrong | 11:17 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Resize: Return to initial state instead of ACTIVE when resize fails https://review.openstack.org/202379 | 11:18 |
| *** Maike has quit IRC | 11:18 | |
| *** sudipto has quit IRC | 11:20 | |
| *** otter768 has joined #openstack-nova | 11:20 | |
| *** andreykurilin has quit IRC | 11:20 | |
| *** andreykurilin has joined #openstack-nova | 11:23 | |
| sdague | alex_xu: sure, however, if people are using this code path, it's going to break working code | 11:24 |
| *** jcoufal has joined #openstack-nova | 11:24 | |
| sdague | which we really want to be careful of | 11:24 |
| *** otter768 has quit IRC | 11:25 | |
| alex_xu | sdague: this isn't bug? | 11:25 |
| sdague | sure, it might be, but it's been there for a long time | 11:25 |
| * alex_xu still can get clear mind on those thing... | 11:25 | |
| alex_xu | sdague: yea, it definitely a long time | 11:26 |
| alex_xu | so we need microverions | 11:26 |
| sdague | every time we touch the API we also need to figure out if we're likely to break code that works against our interface | 11:26 |
| *** lyanchih has joined #openstack-nova | 11:26 | |
| sdague | so, honestly, given that neutron is the long term direction, I'd be more enthusiastic to evolving the nova-net behavior towards the neutron one | 11:26 |
| sdague | so make description not required even on the nova net side | 11:27 |
| *** jcoufal_ has quit IRC | 11:27 | |
| alex_xu | do we need microversion also? | 11:27 |
| alex_xu | sdague: 'name' also is a problem, neutron allow name empty also | 11:29 |
| *** gszasz has joined #openstack-nova | 11:29 | |
| alex_xu | sdague: nova-network is not. | 11:29 |
| sdague | we probably need a microversion | 11:29 |
| alex_xu | sdague: the 'name' for nova-network is the key | 11:29 |
| sdague | dropping name as requirement seems bad, it would probably be worth asking the neutron folks why that's not required there | 11:30 |
| sdague | maybe we could meet in a happy middle | 11:30 |
| alex_xu | sdague: there is uuid in neutron | 11:30 |
| *** Maike has joined #openstack-nova | 11:30 | |
| sdague | sure, but uuids are very non user friendly. Do they expect uuid in their cli to change things? | 11:31 |
| alex_xu | both uuid and name, I guess when name is empty, they can return fault | 11:32 |
| alex_xu | why need asking neutron folks, you want the neutron required name in their API? | 11:33 |
| sdague | ok, so how about we get a mailing list thread going with neutron folks here as well about trying to converge the behavior. I think dropping description as required from nova would be fine | 11:33 |
| sdague | ideally that would be nice | 11:33 |
| *** jcoufal_ has joined #openstack-nova | 11:33 | |
| sdague | I don't know how disruptive it would be | 11:33 |
| sdague | it's something we should ask | 11:34 |
| alex_xu | emm...make sense, worth a mail | 11:34 |
| *** thorst has joined #openstack-nova | 11:35 | |
| alex_xu | let me address the mail | 11:35 |
| *** jcoufal has quit IRC | 11:37 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Move V2.1 API unittest to top level directory https://review.openstack.org/212120 | 11:38 |
| openstackgerrit | Alex Xu proposed openstack/nova: Move V2 API unittests under legacy_v2 directory https://review.openstack.org/212393 | 11:38 |
| *** sahid has quit IRC | 11:38 | |
| *** sahid has joined #openstack-nova | 11:39 | |
| alex_xu | sdague: ^ the tests are fixed. at least pass in my local | 11:39 |
| *** haomaiwang has quit IRC | 11:42 | |
| *** markvoelker has joined #openstack-nova | 11:44 | |
| *** Marga_ has quit IRC | 11:45 | |
| *** Marga_ has joined #openstack-nova | 11:46 | |
| *** baoli has joined #openstack-nova | 11:46 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Fix incorrect result of host-describe https://review.openstack.org/211083 | 11:47 |
| *** markvoelker has quit IRC | 11:49 | |
| *** marzif has joined #openstack-nova | 11:51 | |
| *** markvoelker has joined #openstack-nova | 11:53 | |
| sdague | alex_xu: thanks | 11:54 |
| alex_xu | sdague: np | 11:55 |
| *** humble__ has quit IRC | 11:57 | |
| *** dave-mccowan has joined #openstack-nova | 12:00 | |
| *** kiwik-chenrui has quit IRC | 12:01 | |
| *** RuiChen has joined #openstack-nova | 12:01 | |
| *** dheeraj has quit IRC | 12:03 | |
| sdague | alex_xu: why is this change in there - https://review.openstack.org/#/c/212393/3/nova/tests/unit/api/openstack/compute/plugins/v3/test_servers.py,cm | 12:10 |
| *** evgenyf has joined #openstack-nova | 12:10 | |
| *** lyanchih has quit IRC | 12:11 | |
| *** otter768 has joined #openstack-nova | 12:11 | |
| alex_xu | sdague: I think the original code is wrong. It reference to v2 test_servers.py. When move v2 test_servers, this test is borken | 12:11 |
| alex_xu | sdague: so I correct to use it own fake NeutronClass | 12:12 |
| sdague | ok | 12:14 |
| sdague | that's fine | 12:14 |
| edleafe | alex_xu: shouldn't it be changed to point to the legacy_v2.test_servers class? | 12:14 |
| edleafe | not sure if it makes a difference | 12:14 |
| *** liusheng has joined #openstack-nova | 12:15 | |
| alex_xu | sdague: no, it shouldn't, there are NeutronV2Subclass both in v2.1 test_servers.py and v2 test_servers.py, they should use their own copy | 12:15 |
| *** busterswt has joined #openstack-nova | 12:16 | |
| alex_xu | sdague: it should be mistake when create v2.1 test_servers from v2 | 12:16 |
| sdague | ok | 12:16 |
| alex_xu | oops, sorry, point to wrong people, edleafe :) | 12:16 |
| *** kylek3h has quit IRC | 12:16 | |
| edleafe | heh | 12:16 |
| *** lucasagomes has quit IRC | 12:18 | |
| *** erlon has joined #openstack-nova | 12:21 | |
| rajesht | sdague, alaski, dansmith, ndipanov, jaypipes: could you please review https://review.openstack.org/#/c/170118/ | 12:22 |
| edleafe | alex_xu: I saw you made some changes to https://review.openstack.org/#/c/212120/ and changed the commit message | 12:23 |
| edleafe | alex_xu: was there a problem? | 12:23 |
| jaypipes | rajesht: yup, it's in my queue. thx :) | 12:23 |
| rajesht | jaypipes: thanks | 12:23 |
| *** otter768 has quit IRC | 12:24 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Add soft policies for server-group feature https://review.openstack.org/212504 | 12:25 |
| alex_xu | edleafe: I found not all test files under compute are v2 specific. I want to legacy_v2 only include v2 specific test https://review.openstack.org/#/c/212393/3 | 12:25 |
| edleafe | alex_xu: yes; I thought we were going to fix that later. Did you see my comments on the etherpad? | 12:26 |
| * alex_xu is reading... | 12:27 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Switch from deprecated timeutils.isotime https://review.openstack.org/211146 | 12:28 |
| alex_xu | edleafe: let us put v2.1 and v2 mixed test file under toplevel, treat them as v2.1. When we remove v2 api, I think we can cleanup them up | 12:29 |
| *** liusheng has quit IRC | 12:30 | |
| edleafe | alex_xu: there were very few that were v2 only | 12:30 |
| alex_xu | edleafe: yea, like test_servers, it is huge, really hard to merge | 12:31 |
| *** Nikolay_St has quit IRC | 12:31 | |
| alex_xu | edleafe: something put into legacy_v2 can be removed easily when we remove v2 code | 12:31 |
| edleafe | alex_xu: I'll grab your changes. Do you need me to work on these at all, or are they ready? | 12:32 |
| *** aysyd has joined #openstack-nova | 12:32 | |
| alex_xu | edleafe: they are ready, just waiting for review | 12:33 |
| *** jcoufal_ has quit IRC | 12:33 | |
| alex_xu | edleafe: I didn't touch sample test yet, probably tomorrow | 12:33 |
| *** annegentle has joined #openstack-nova | 12:34 | |
| *** bnemec has joined #openstack-nova | 12:34 | |
| *** haomaiwang has joined #openstack-nova | 12:34 | |
| edleafe | alex_xu: ah, ok. I really don't understand all the path calculations going on there, so I was hoping you could enlighten me :) | 12:34 |
| openstackgerrit | Gorka Eguileor proposed openstack/nova: Preserve usage and reservations on quota deletion https://review.openstack.org/163423 | 12:35 |
| alex_xu | edleafe: hope no big trouble with sample test :) | 12:35 |
| edleafe | alex_xu: what I don't understand is how those tests find their samples and templates. | 12:36 |
| edleafe | alex_xu: I see how they work, but it is hard to sort out the duplication | 12:36 |
| *** chuckcarmack has joined #openstack-nova | 12:36 | |
| *** pkarikh has joined #openstack-nova | 12:37 | |
| alex_xu | edleafe: you probably cna get some hint from there https://github.com/openstack/nova/blob/master/nova/tests/functional/api_samples_test_base.py#L77 | 12:38 |
| *** jcoufal has joined #openstack-nova | 12:38 | |
| *** dane-fichter has joined #openstack-nova | 12:38 | |
| edleafe | alex_xu: yes, I read all that code | 12:39 |
| alex_xu | edleafe: ok, I feel I will get some trouble tomorrow :) | 12:40 |
| edleafe | alex_xu: but it is hard to tell which test is v2, and should point to the doc/legacy_v2 directory, and which is v21 | 12:40 |
| alex_xu | edleafe: there also some works on merge v2 and v2.1 sample tests, but not finish yet. | 12:41 |
| *** VW_ has joined #openstack-nova | 12:41 | |
| edleafe | alex_xu: ok | 12:41 |
| alex_xu | edleafe: the v2 sample tests in https://github.com/openstack/nova/blob/master/nova/tests/functional/test_api_samples.py | 12:41 |
| *** mdrabe has joined #openstack-nova | 12:42 | |
| alex_xu | edleafe: all those tests existed in test_api_samples is not merged yet | 12:42 |
| alex_xu | edleafe: others already merged into v2.1 sample tests, those are under v3 directly | 12:42 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Add support for shared volumes between guests https://review.openstack.org/212508 | 12:43 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Add support for shared volumes between guests https://review.openstack.org/212508 | 12:44 |
| *** deepakcs has quit IRC | 12:45 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 12:48 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Move objects registration in tests directory https://review.openstack.org/212159 | 12:48 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Add fields to request spec object https://review.openstack.org/212168 | 12:48 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Create RequestSpec object https://review.openstack.org/145528 | 12:48 |
| openstackgerrit | Alexis Lee proposed openstack/nova: WIP https://review.openstack.org/212510 | 12:48 |
| *** jcoufal has quit IRC | 12:52 | |
| *** jcoufal has joined #openstack-nova | 12:52 | |
| *** fawadkhaliq has quit IRC | 12:52 | |
| openstackgerrit | Pranali Deore proposed openstack/nova: Validate quota class_name https://review.openstack.org/192115 | 12:53 |
| *** rfolco has joined #openstack-nova | 12:55 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move the v3 api_sample functional tests https://review.openstack.org/212238 | 12:55 |
| openstackgerrit | Peter Penchev proposed openstack/nova: Add the Nova libvirt StorPool attachment driver. https://review.openstack.org/140733 | 12:55 |
| *** bapalm has joined #openstack-nova | 12:55 | |
| *** lyanchih has joined #openstack-nova | 12:55 | |
| *** bapalm__ has joined #openstack-nova | 12:57 | |
| *** VW_ has quit IRC | 12:58 | |
| *** VW_ has joined #openstack-nova | 12:58 | |
| *** bapalm___ has joined #openstack-nova | 12:58 | |
| *** jecarey has quit IRC | 12:58 | |
| *** bapalm has quit IRC | 12:59 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** openstackgerrit has quit IRC | 13:01 | |
| *** bapalm__ has quit IRC | 13:01 | |
| *** haomaiwang has joined #openstack-nova | 13:02 | |
| *** baoli_ has joined #openstack-nova | 13:02 | |
| *** openstackgerrit has joined #openstack-nova | 13:02 | |
| *** rlrossit has joined #openstack-nova | 13:03 | |
| *** VW_ has quit IRC | 13:03 | |
| *** xyang1 has joined #openstack-nova | 13:03 | |
| *** mestery has quit IRC | 13:03 | |
| *** annegentle has quit IRC | 13:04 | |
| *** baoli has quit IRC | 13:04 | |
| *** annegentle has joined #openstack-nova | 13:05 | |
| *** edmondsw has joined #openstack-nova | 13:05 | |
| *** bapalm___ has quit IRC | 13:05 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Add REST API impact to the multi-attach spec https://review.openstack.org/192755 | 13:05 |
| *** lyanchih has quit IRC | 13:06 | |
| *** otter768 has joined #openstack-nova | 13:08 | |
| *** sudipto has joined #openstack-nova | 13:10 | |
| *** mestery has joined #openstack-nova | 13:11 | |
| *** fawadkhaliq has joined #openstack-nova | 13:13 | |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: libvirt: Fix snapshot delete for network disk type for blockRebase op https://review.openstack.org/212518 | 13:13 |
| *** ivasev has joined #openstack-nova | 13:16 | |
| *** dustins has joined #openstack-nova | 13:21 | |
| *** ijuwang has joined #openstack-nova | 13:21 | |
| *** Piet has quit IRC | 13:22 | |
| *** rotbeard has joined #openstack-nova | 13:22 | |
| *** Nikolay_St has joined #openstack-nova | 13:22 | |
| *** otter768 has quit IRC | 13:22 | |
| *** bebech has joined #openstack-nova | 13:24 | |
| *** bebech has quit IRC | 13:27 | |
| *** diogogmt has joined #openstack-nova | 13:28 | |
| *** browne has joined #openstack-nova | 13:30 | |
| *** haomaiwang has quit IRC | 13:31 | |
| *** haomaiwang has joined #openstack-nova | 13:31 | |
| evgenyf | Hi folks, who can help with sporadical vm provissioning failures with "no valid host was found, there are not enough hosta available" ? | 13:32 |
| *** zzzeek has joined #openstack-nova | 13:35 | |
| *** VW_ has joined #openstack-nova | 13:36 | |
| *** gszasz has quit IRC | 13:36 | |
| *** jwcroppe has quit IRC | 13:37 | |
| *** jwcroppe has joined #openstack-nova | 13:37 | |
| jaypipes | evgenyf: ask on #openstack (see /topic). I can help you there. | 13:39 |
| *** gszasz has joined #openstack-nova | 13:40 | |
| *** rgerganov has quit IRC | 13:40 | |
| *** shuquan has joined #openstack-nova | 13:41 | |
| *** jwcroppe has quit IRC | 13:42 | |
| *** eharney has quit IRC | 13:43 | |
| *** lyanchih has joined #openstack-nova | 13:46 | |
| *** mriedem has joined #openstack-nova | 13:46 | |
| *** setuid has quit IRC | 13:47 | |
| *** mwagner_lap has quit IRC | 13:48 | |
| *** ankit_ag has quit IRC | 13:48 | |
| *** alex_xu has quit IRC | 13:48 | |
| *** diogogmt has quit IRC | 13:49 | |
| *** gszasz has quit IRC | 13:50 | |
| *** jwcroppe has joined #openstack-nova | 13:50 | |
| *** exploreshaifali has joined #openstack-nova | 13:50 | |
| *** thangp has joined #openstack-nova | 13:50 | |
| *** jswarren has joined #openstack-nova | 13:51 | |
| *** alex_xu has joined #openstack-nova | 13:51 | |
| *** rook has joined #openstack-nova | 13:51 | |
| *** shuquan has quit IRC | 13:53 | |
| *** setuid has joined #openstack-nova | 13:53 | |
| *** mriedem has quit IRC | 13:55 | |
| *** gholler has joined #openstack-nova | 13:56 | |
| *** mriedem has joined #openstack-nova | 14:00 | |
| *** haomaiwang has quit IRC | 14:01 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Limit parallel live migrations in progress https://review.openstack.org/212065 | 14:01 |
| *** haomaiwang has joined #openstack-nova | 14:02 | |
| *** tpeoples has joined #openstack-nova | 14:02 | |
| *** shuquan has joined #openstack-nova | 14:03 | |
| *** thorst has quit IRC | 14:03 | |
| mriedem | scottda: can you remind me the link to the force detach volume spec for nova? i can't seem to find it | 14:03 |
| *** afazekas has quit IRC | 14:03 | |
| *** Maike has quit IRC | 14:03 | |
| *** marzif has quit IRC | 14:04 | |
| scottda | https://review.openstack.org/#/c/84048/44 | 14:04 |
| mriedem | ah, abandoned | 14:04 |
| mriedem | that's why i couldn't find it, thanks | 14:04 |
| scottda | Note that we have changed tactics and are not using this spec.. | 14:04 |
| *** marzif has joined #openstack-nova | 14:04 | |
| scottda | jaypipes: suggested that we use nova-manage, so that we don't have to change the API | 14:04 |
| *** evgenyf has quit IRC | 14:04 | |
| mriedem | yeah, the code patch should be pointing out the blueprint though | 14:05 |
| mriedem | and the code patch still touches the api | 14:05 |
| scottda | Here's the bug: https://bugs.launchpad.net/nova/+bug/1449221 | 14:06 |
| openstack | Launchpad bug 1449221 in OpenStack Compute (nova) "Nova doesn't allow to cleanup volumes stuck in 'attaching' or 'detaching' status" [High,In progress] - Assigned to wanghao (wanghao749) | 14:06 |
| *** ijuwang has quit IRC | 14:06 | |
| *** evgenyf has joined #openstack-nova | 14:06 | |
| mriedem | ok, and johnthetubaguy marked the bp as obsolete https://blueprints.launchpad.net/nova/+spec/add-force-detach-to-nova | 14:06 |
| *** ferest has joined #openstack-nova | 14:06 | |
| scottda | That POC code will have to be abandoned in favor of a new one that uses nova-manage | 14:06 |
| *** ijuwang has joined #openstack-nova | 14:06 | |
| mriedem | sure, if a new patch is posted then i guess we can abandon https://review.openstack.org/#/c/184537/ | 14:07 |
| scottda | andrearosa: Will be working on this, but he is on holiday until tomorrow? or monday | 14:07 |
| mriedem | i put a -2 on it for now due to the api change | 14:07 |
| scottda | ok | 14:07 |
| *** krtaylor has joined #openstack-nova | 14:08 | |
| *** fifieldt has joined #openstack-nova | 14:08 | |
| *** Maike has joined #openstack-nova | 14:09 | |
| scottda | hmm...I see that wanghao has re-assigned that bug to himself. | 14:10 |
| mriedem | scottda: because they rebased the change | 14:10 |
| scottda | wanghao: Are you online? | 14:10 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Allow resize root_gb to 0 for volume-backed instances https://review.openstack.org/207794 | 14:11 |
| *** ferest has quit IRC | 14:11 | |
| scottda | That's fine. I just want to make sure we're on the same page regarding the fix. There's some work in the calls to Cinder that I need to discuss with whomever writes the patch | 14:11 |
| *** thorst has joined #openstack-nova | 14:11 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Remove tie between system_metadata and extra.flavor https://review.openstack.org/209240 | 14:12 |
| andrearosa | scottda: I am back today :( | 14:14 |
| *** shuquan has quit IRC | 14:14 | |
| scottda | What's with the frowny face? | 14:14 |
| *** Murali_ has joined #openstack-nova | 14:15 | |
| andrearosa | I talked to wanghao today and he will be the co-author for that patch | 14:15 |
| lxsli | scottda: I think he prefers loafing around in Italy to sitting in a cubefarm! | 14:15 |
| lxsli | scottda: what a weirdo amirite | 14:16 |
| scottda | Italy would be great. I think he was in Spain! You Europeans have it nice, I go to South Dakota for vacation. | 14:16 |
| lxsli | I'm sure that's... lovely | 14:16 |
| andrearosa | lxsli: you know me well. scottda: I was in Spain and italy is great :) | 14:17 |
| scottda | andrearosa: Let's sync up before you leave today. | 14:17 |
| andrearosa | scottda: sure | 14:17 |
| andrearosa | as I said wanghao is going to work on it starting from taht POC. I talked with him and next week | 14:17 |
| andrearosa | I am goign to sync with him to see how we can put a complete patch as soon as we can according to our differen commitments | 14:18 |
| *** Murali_ has quit IRC | 14:18 | |
| *** eharney has joined #openstack-nova | 14:18 | |
| *** Murali_ has joined #openstack-nova | 14:19 | |
| *** jecarey has joined #openstack-nova | 14:19 | |
| *** Murali_ has quit IRC | 14:20 | |
| mriedem | claudiub: want to take a look at the juno backport too? https://review.openstack.org/#/c/203253/ | 14:20 |
| mriedem | the tempest test is passing https://review.openstack.org/#/c/175949/ | 14:21 |
| *** annegentle has quit IRC | 14:21 | |
| *** shuquan has joined #openstack-nova | 14:21 | |
| *** cfriesen_ has joined #openstack-nova | 14:23 | |
| *** devlaps has quit IRC | 14:24 | |
| *** exploreshaifali has quit IRC | 14:25 | |
| *** gszasz has joined #openstack-nova | 14:27 | |
| *** tsekiyama has joined #openstack-nova | 14:28 | |
| mnestratov | mriedem: hi regarding https://bugs.launchpad.net/nova/+bug/1476806 as far as https://review.openstack.org/#/c/212193/ nova changes for master aren't actual anymore, right? | 14:28 |
| openstack | Launchpad bug 1476806 in OpenStack Compute (nova) "Unable to delete instance with attached volumes which failed to boot" [High,In progress] - Assigned to Maxim Nestratov (mnestratov) | 14:28 |
| mriedem | mnestratov: i just started looking at that | 14:29 |
| mriedem | mnestratov: that doesn't fix stable/kilo unless the cinder team plans on backporting that change to kilo also | 14:29 |
| mriedem | jaypipes: are you around? | 14:29 |
| mriedem | API change question | 14:29 |
| mnestratov | mriedem: by the way I can port to kilo my changes | 14:30 |
| *** mwagner_lap has joined #openstack-nova | 14:30 | |
| claudiub | mriedem: actually wondering how the tempest test passes in Juno, even without the nova juno backport.. | 14:31 |
| mriedem | claudiub: the tempest change uses the depends-on | 14:31 |
| mriedem | so when it runs the juno job it's using the nvoa backport for juno | 14:31 |
| claudiub | mriedem: oh, that's so cool. I didn't know it could do that. | 14:31 |
| claudiub | technology man, it keeps blowing people's minds! | 14:32 |
| *** deepakcs has joined #openstack-nova | 14:32 | |
| mriedem | :) | 14:32 |
| *** shuquan has quit IRC | 14:35 | |
| wanghao | scottda: yes I'm here | 14:35 |
| scottda | wanghao: Hi. I just wanted to sync with you and andrearosa regarding the patch for https://bugs.launchpad.net/nova/+bug/1449221 | 14:36 |
| openstack | Launchpad bug 1449221 in OpenStack Compute (nova) "Nova doesn't allow to cleanup volumes stuck in 'attaching' or 'detaching' status" [High,In progress] - Assigned to wanghao (wanghao749) | 14:36 |
| scottda | I'll PM you | 14:36 |
| wanghao | OK | 14:37 |
| *** gszasz has quit IRC | 14:38 | |
| *** whenry has joined #openstack-nova | 14:43 | |
| *** unicell has joined #openstack-nova | 14:43 | |
| *** unicell1 has quit IRC | 14:43 | |
| *** mtanino has joined #openstack-nova | 14:48 | |
| *** achanda has joined #openstack-nova | 14:53 | |
| *** marzif has quit IRC | 14:54 | |
| *** diogogmt has joined #openstack-nova | 14:54 | |
| *** lucasagomes has joined #openstack-nova | 14:55 | |
| *** morgan_404 is now known as morgan_302 | 14:55 | |
| *** eglynn has joined #openstack-nova | 14:55 | |
| jaypipes | mriedem: yep! | 14:56 |
| *** jistr is now known as jistr|mtg | 14:57 | |
| mriedem | jaypipes: i wanted to get your opinion on https://review.openstack.org/#/c/212193/ since it fixes a bug in nova | 14:58 |
| mriedem | jaypipes: it's changing a 400 to a 200 | 14:58 |
| mriedem | which the api guidelines say is generally ok | 14:58 |
| *** annegentle has joined #openstack-nova | 14:58 | |
| jaypipes | mriedem: looking... | 14:59 |
| *** annegentle has quit IRC | 14:59 | |
| *** annegentle has joined #openstack-nova | 14:59 | |
| *** gholler_ has joined #openstack-nova | 15:00 | |
| *** armax has joined #openstack-nova | 15:00 | |
| *** agireud has joined #openstack-nova | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-nova | 15:02 | |
| *** dane-fichter has quit IRC | 15:02 | |
| *** rlrossit has left #openstack-nova | 15:03 | |
| *** Piet has joined #openstack-nova | 15:03 | |
| *** gholler has quit IRC | 15:03 | |
| *** achanda has quit IRC | 15:03 | |
| *** agireud_ has joined #openstack-nova | 15:04 | |
| *** yamahata has joined #openstack-nova | 15:05 | |
| *** agireud has quit IRC | 15:05 | |
| *** agireud_ is now known as agireud | 15:05 | |
| *** jcoufal has quit IRC | 15:06 | |
| danpb | sigh, the advice for hotel bookings on the openstack website is utterly wrong | 15:07 |
| danpb | it says if you need to stay for extra nights not covered by the OST2015 discount code you have to email the hotel | 15:07 |
| danpb | after 3 days waiting for a response to the email, they respond saying they won't handle requests via email and you just have to book online | 15:08 |
| *** garyk has joined #openstack-nova | 15:08 | |
| *** rlrossit has joined #openstack-nova | 15:08 | |
| *** Maike has quit IRC | 15:09 | |
| *** gszasz has joined #openstack-nova | 15:10 | |
| *** thorst has quit IRC | 15:12 | |
| *** burt has joined #openstack-nova | 15:13 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: return guest informations https://review.openstack.org/191046 | 15:15 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: clean code of NOTE tags https://review.openstack.org/212580 | 15:15 |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: unit tests: disable eventlet thread pool https://review.openstack.org/212581 | 15:17 |
| mnestratov | danpb: could you please take a look at this https://review.openstack.org/#/c/184153/ ? | 15:18 |
| *** suresh12 has joined #openstack-nova | 15:20 | |
| *** deepakcs has quit IRC | 15:21 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: return guest informations https://review.openstack.org/191046 | 15:21 |
| *** ftersin has joined #openstack-nova | 15:22 | |
| *** kfox1111 has left #openstack-nova | 15:22 | |
| *** hhuang has quit IRC | 15:23 | |
| *** tpeoples has quit IRC | 15:23 | |
| *** angdraug has joined #openstack-nova | 15:23 | |
| *** haigang has joined #openstack-nova | 15:24 | |
| *** rlrossit has left #openstack-nova | 15:24 | |
| *** haigang has quit IRC | 15:24 | |
| *** haigang has joined #openstack-nova | 15:25 | |
| *** vishwanathj has joined #openstack-nova | 15:29 | |
| *** rlrossit has joined #openstack-nova | 15:30 | |
| *** haomaiwang has quit IRC | 15:31 | |
| *** gszasz has quit IRC | 15:31 | |
| *** ijuwang has quit IRC | 15:31 | |
| *** romainh has quit IRC | 15:32 | |
| *** dannywilson has joined #openstack-nova | 15:33 | |
| *** jistr|mtg is now known as jistr | 15:34 | |
| mnestratov | danpb: thanks a lot | 15:38 |
| *** tpeoples has joined #openstack-nova | 15:38 | |
| *** scheuran has quit IRC | 15:38 | |
| *** tonytan4ever has joined #openstack-nova | 15:40 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: ensure that get_info raises the correct exception https://review.openstack.org/191558 | 15:40 |
| mriedem | mnestratov: danpb: virtuozzo ci failed on https://review.openstack.org/#/c/184153/ | 15:40 |
| *** nkrinner has quit IRC | 15:40 | |
| *** alexschm has quit IRC | 15:40 | |
| mriedem | mnestratov: it doesn't look like virtuozzo ci even runs server image snapshot tests | 15:42 |
| openstackgerrit | garyk proposed openstack/nova: VMware: treat deletion exception with attached volumes https://review.openstack.org/198674 | 15:42 |
| mriedem | maybe due to this bug, but could you get that test enabled for the run on that patch? | 15:42 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: return guest informations https://review.openstack.org/191046 | 15:43 |
| *** ijuwang has joined #openstack-nova | 15:43 | |
| *** amotoki has joined #openstack-nova | 15:44 | |
| *** jwcroppe has quit IRC | 15:44 | |
| mnestratov | mriedem: at the moment https://review.openstack.org/#/c/190843 cherry pick fails due to recent fs volume code landed | 15:44 |
| *** jwcroppe has joined #openstack-nova | 15:45 | |
| mnestratov | so it is being reworked | 15:45 |
| mriedem | so needs a rebase | 15:45 |
| openstackgerrit | garyk proposed openstack/nova: Cells: Use decorator for checking if cells are enabled https://review.openstack.org/198537 | 15:45 |
| mnestratov | yeah | 15:46 |
| *** praneshp has joined #openstack-nova | 15:46 | |
| *** dprince has joined #openstack-nova | 15:47 | |
| *** jistr has quit IRC | 15:48 | |
| *** haigang has quit IRC | 15:49 | |
| *** haigang has joined #openstack-nova | 15:49 | |
| mnestratov | mriedem: any info about cinder plans on kilo backporting https://review.openstack.org/#/c/212193/ ? | 15:49 |
| *** praneshp_ has joined #openstack-nova | 15:50 | |
| mriedem | mnestratov: i'm waiting for the cinder core team to weigh in on that in the change | 15:50 |
| *** rajesht has quit IRC | 15:50 | |
| *** MarcF has left #openstack-nova | 15:51 | |
| *** praneshp has quit IRC | 15:51 | |
| *** praneshp_ is now known as praneshp | 15:51 | |
| mnestratov | mriedem: just saw your question in gerrit | 15:51 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: ironic: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189362 | 15:53 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: objects: convert config drive to use ImageMeta object https://review.openstack.org/189323 | 15:53 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188119 | 15:53 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: hyperv: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189302 | 15:53 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 15:53 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 15:53 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 15:53 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: xenapi: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188790 | 15:53 |
| *** thangp has quit IRC | 15:54 | |
| danpb | mriedem: could you take a look at this https://review.openstack.org/#/c/176360/ | 15:54 |
| danpb | and/or these migration pathces https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1429220,n,z | 15:55 |
| *** thangp has joined #openstack-nova | 15:55 | |
| mriedem | danpb: yeah, in a bit - might ask cfriesen_ to peek at the first about huge pages | 15:55 |
| *** thorst has joined #openstack-nova | 15:56 | |
| *** tjones has joined #openstack-nova | 15:58 | |
| openstackgerrit | jichenjc proposed openstack/nova: Catch invalid id input in service_delete https://review.openstack.org/198753 | 15:59 |
| *** rushil has joined #openstack-nova | 15:59 | |
| openstackgerrit | jichenjc proposed openstack/nova: Catch OverQuota in volume create function https://review.openstack.org/173985 | 15:59 |
| danpb | mriedem: it basically makes vhost-user VIF actually work | 16:00 |
| danpb | without it, it isnt possible for the app using vhost-user to access the guest memory | 16:00 |
| openstackgerrit | jichenjc proposed openstack/nova: Return HTTPForbidden if over quota https://review.openstack.org/180469 | 16:00 |
| *** hhuang has joined #openstack-nova | 16:01 | |
| mriedem | ok, whatever that means :) | 16:01 |
| mriedem | i'm pushing a few things to unblock stable atm, then i'll circle back on this | 16:01 |
| *** apoorvad has joined #openstack-nova | 16:02 | |
| danpb | mriedem: essentially vhost-user lets an external app provide the network connectivity to the host | 16:02 |
| danpb | the app needs to be able to map QEMU ram into its own address space | 16:02 |
| *** geekinutah has joined #openstack-nova | 16:02 | |
| danpb | hugepages as setup by Nova currently deny that mapping | 16:02 |
| mriedem | ok | 16:02 |
| mriedem | i love me some NFV | 16:03 |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: add vzstorage volumes suppport https://review.openstack.org/190843 | 16:03 |
| danpb | heh | 16:03 |
| mriedem | b/c if it's not shared the app can't access it | 16:03 |
| danpb | indeed, it is marked private to the qemu process currently | 16:03 |
| *** yassine has quit IRC | 16:04 | |
| mriedem | "There is no serious downside to using shared mappings, so Nova might as well just unconditionally request them all the time." | 16:04 |
| *** rushil has quit IRC | 16:04 | |
| mriedem | there is no security issue here? | 16:04 |
| mriedem | potentially | 16:04 |
| *** xgsa1 has quit IRC | 16:04 | |
| danpb | no, to access the RAM you have to be running as the same UID as QEMU, or as root, at which point you've already lost separation | 16:04 |
| *** jlanoux has quit IRC | 16:04 | |
| danpb | unless you have SELinux / AppArmour enforcing it, which they will | 16:05 |
| *** dprince has quit IRC | 16:05 | |
| *** lucasagomes has quit IRC | 16:05 | |
| *** edmondsw has quit IRC | 16:06 | |
| ftersin | ndipanov: hi. i've answered you in https://review.openstack.org/#/c/170243/ | 16:07 |
| ndipanov | yeah saw that | 16:07 |
| ftersin | ndipanov: and what do you think? nothing yet? | 16:08 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: WIP: documentation: virt driver interface (as-is) https://review.openstack.org/212613 | 16:08 |
| openstackgerrit | Martin Schuppert proposed openstack/nova: Make pagination work with changes-since and deleted marker https://review.openstack.org/212614 | 16:09 |
| *** kurtmartin has joined #openstack-nova | 16:09 | |
| *** alexpilotti_ has joined #openstack-nova | 16:11 | |
| *** Marga_ has quit IRC | 16:13 | |
| *** Marga_ has joined #openstack-nova | 16:13 | |
| *** matrohon has quit IRC | 16:14 | |
| *** sahid has quit IRC | 16:14 | |
| *** kmARC has quit IRC | 16:14 | |
| *** lordd has joined #openstack-nova | 16:14 | |
| *** haigang has quit IRC | 16:15 | |
| ndipanov | ftersin, you are probably right but won't be able to get back to it today :( | 16:16 |
| *** Nikolay_St has quit IRC | 16:16 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: Retry check for path existence after delay if needed https://review.openstack.org/212619 | 16:17 |
| *** evgenyf has quit IRC | 16:17 | |
| *** hhuang has quit IRC | 16:18 | |
| *** alexpilotti_ is now known as alexpilotti | 16:19 | |
| *** alexpilotti has quit IRC | 16:19 | |
| *** meghal has joined #openstack-nova | 16:19 | |
| *** alexpilotti has joined #openstack-nova | 16:19 | |
| *** alexpilotti has quit IRC | 16:19 | |
| *** praneshp has quit IRC | 16:20 | |
| *** nic has joined #openstack-nova | 16:20 | |
| *** alejandrito has joined #openstack-nova | 16:22 | |
| ftersin | ndipanov: well, it's not asap. just wouldn't like to wait for Mitaka bugfix to continue with that :) | 16:28 |
| *** edmondsw has joined #openstack-nova | 16:30 | |
| *** diogogmt has quit IRC | 16:31 | |
| *** openstackgerrit has quit IRC | 16:31 | |
| *** unicell has quit IRC | 16:31 | |
| *** diogogmt has joined #openstack-nova | 16:31 | |
| *** openstackgerrit has joined #openstack-nova | 16:32 | |
| mriedem | danpb: i assume i'll be tarred and feathered if i -1 over the unrelated whitespace change in your vhost-user shared thing | 16:32 |
| *** Sukhdev has joined #openstack-nova | 16:32 | |
| danpb | mriedem: lol | 16:32 |
| *** alex_klimov has quit IRC | 16:32 | |
| mriedem | +W | 16:32 |
| mriedem | don't want to lose the +2 | 16:32 |
| *** eglynn has quit IRC | 16:32 | |
| *** rotbeard has quit IRC | 16:34 | |
| *** meghal1 has joined #openstack-nova | 16:34 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: Cleanup of compute api reboot method https://review.openstack.org/127427 | 16:35 |
| *** exploreshaifali has joined #openstack-nova | 16:35 | |
| *** ildikov has quit IRC | 16:36 | |
| *** penick has joined #openstack-nova | 16:36 | |
| *** penick has quit IRC | 16:36 | |
| *** meghal has quit IRC | 16:37 | |
| *** dustins is now known as dustins|lunch | 16:37 | |
| *** FL1SK has joined #openstack-nova | 16:38 | |
| *** moshele has quit IRC | 16:40 | |
| *** jamielennox is now known as jamielennox|away | 16:45 | |
| *** Nikolay_St has joined #openstack-nova | 16:49 | |
| *** lyanchih has quit IRC | 16:51 | |
| *** kmARC has joined #openstack-nova | 16:53 | |
| *** mriedem is now known as mriedem_lunch | 16:54 | |
| *** kmARC has quit IRC | 16:54 | |
| *** browne has quit IRC | 16:54 | |
| *** kmARC has joined #openstack-nova | 16:54 | |
| openstackgerrit | Tiago Mello proposed openstack/nova: libvirt: Handle unsupported Spice feature for PowerKVM https://review.openstack.org/203809 | 16:54 |
| *** apuimedo has quit IRC | 16:55 | |
| *** VW_ has quit IRC | 16:55 | |
| *** VW_ has joined #openstack-nova | 16:56 | |
| *** lordd has quit IRC | 16:56 | |
| *** patrickeast has joined #openstack-nova | 16:56 | |
| *** mwagner_lap has quit IRC | 16:57 | |
| *** rdopiera has quit IRC | 16:57 | |
| *** unicell has joined #openstack-nova | 16:59 | |
| *** VW__ has joined #openstack-nova | 16:59 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: resize ploop disks with prl_disk_tool https://review.openstack.org/182390 | 17:00 |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: virtuozzo instance resize support https://review.openstack.org/182257 | 17:00 |
| *** VW_ has quit IRC | 17:00 | |
| *** meghal1 is now known as meghal | 17:00 | |
| *** suresh12 has quit IRC | 17:02 | |
| *** skylerberg has joined #openstack-nova | 17:02 | |
| *** kmARC has quit IRC | 17:03 | |
| *** zhenguo has quit IRC | 17:03 | |
| *** paul-carlton has joined #openstack-nova | 17:04 | |
| *** praneshp has joined #openstack-nova | 17:08 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move the v3 api_sample functional tests https://review.openstack.org/212238 | 17:10 |
| *** penick has joined #openstack-nova | 17:11 | |
| *** mwagner_lap has joined #openstack-nova | 17:11 | |
| *** boris-42 has joined #openstack-nova | 17:12 | |
| *** mriedem_lunch is now known as mriedem | 17:13 | |
| *** kurtmartin has quit IRC | 17:14 | |
| *** lordd has joined #openstack-nova | 17:14 | |
| *** vishwanathj has quit IRC | 17:15 | |
| *** ssurana has joined #openstack-nova | 17:16 | |
| *** tonytan4ever has quit IRC | 17:17 | |
| *** rushil has joined #openstack-nova | 17:18 | |
| *** dustins|lunch is now known as dustins | 17:21 | |
| *** moshele has joined #openstack-nova | 17:22 | |
| *** thangp is now known as thangp_afk | 17:22 | |
| *** tonytan4ever has joined #openstack-nova | 17:23 | |
| *** ftersin has quit IRC | 17:23 | |
| *** amotoki has quit IRC | 17:25 | |
| paul-carlton | All, I have submitted https://review.openstack.org/#/c/212619/ can someone review this please | 17:27 |
| paul-carlton | There is no new unit test but I could not find any existing unit test for the function in question | 17:28 |
| *** moshele has quit IRC | 17:28 | |
| *** rushil has quit IRC | 17:28 | |
| *** achanda has joined #openstack-nova | 17:29 | |
| *** rushil has joined #openstack-nova | 17:30 | |
| *** asselin has joined #openstack-nova | 17:33 | |
| *** paul-carlton has left #openstack-nova | 17:36 | |
| mriedem | dansmith: on https://review.openstack.org/#/c/212065/6/nova/compute/manager.py | 17:36 |
| *** gholler_ has quit IRC | 17:36 | |
| mriedem | dansmith: asked this a couple of times, in build_and_run_instance there is a lock on instance.uuid before threading off _do_build_and_run_instance, | 17:37 |
| mriedem | i was wondering if there is similar value in doing that for live_migration | 17:37 |
| dansmith | yeah, I answered didn't I? | 17:37 |
| mriedem | don't think so | 17:38 |
| dansmith | well, I did type something, but apparently I never committed it I guess | 17:38 |
| mriedem | i think you told me to eff off or something | 17:39 |
| mriedem | it was really abusive | 17:39 |
| dansmith | we're not locking now, so I don't think this changes anything | 17:39 |
| mriedem | yeah, true | 17:40 |
| * mriedem checks why that lock was added to biuld | 17:40 | |
| dansmith | it is probably a good idea, and it doesn't look like libvirt is doing it, | 17:40 |
| *** danpb has quit IRC | 17:40 | |
| dansmith | but we'd need to make sure the others never lock instance.uuid, or we'd deadlock | 17:40 |
| alejandrito | Hi guys, just one existence question. suppose im a public cloud provider, how can i handle the amazon like "public address" assigment taking into account that "floating IP" has an internal ip addres space, so, what are the best practices to do a floatingIP -> public internet address, like for example take advantage of my few public ips, but using une public to, for example, redirect traffic to specific tenants floating IPS. | 17:41 |
| alejandrito | any experience on this ? | 17:41 |
| *** angdraug has quit IRC | 17:41 | |
| *** VW__ is now known as VW_ | 17:41 | |
| *** fawadkhaliq has quit IRC | 17:42 | |
| *** browne has joined #openstack-nova | 17:42 | |
| *** gholler has joined #openstack-nova | 17:43 | |
| *** ijw has joined #openstack-nova | 17:44 | |
| mriedem | dansmith: heh, ok, yeah and the lock in build_and_run_instance predates diablo | 17:45 |
| mriedem | so i give up looking on why it was added | 17:45 |
| dansmith | heh | 17:46 |
| dansmith | I didn't know anything came from before diably | 17:46 |
| dansmith | er, diablo | 17:46 |
| * dansmith jokes | 17:46 | |
| mriedem | the lock was in compute.api apparently | 17:46 |
| *** ildikov has joined #openstack-nova | 17:46 | |
| mriedem | in the long long ago | 17:46 |
| mriedem | back when that guy from bush and the holler back girl were still happily married | 17:47 |
| *** suresh12 has joined #openstack-nova | 17:47 | |
| * dansmith nods as if he gets it | 17:47 | |
| mriedem | http://www.people.com/article/gwen-stefani-gavin-rossdale-divorce-assets-no-prenup !!! | 17:47 |
| mriedem | it was a callback to a 90s reference, but for a thing that i forgot was actually still a thing until it wasn't anymore | 17:48 |
| dansmith | okay, I don't feel bad about not knowing that | 17:48 |
| mriedem | ha | 17:48 |
| *** markus_z has quit IRC | 17:53 | |
| mnestratov | mriedem: Our CI reported on https://review.openstack.org/#/c/184153/ | 17:53 |
| *** salv-orl_ has joined #openstack-nova | 17:55 | |
| *** fawadkhaliq has joined #openstack-nova | 17:56 | |
| *** sc68cal has quit IRC | 17:56 | |
| *** thangp_afk is now known as thangp | 17:58 | |
| *** gholler has quit IRC | 17:58 | |
| *** salv-orlando has quit IRC | 17:59 | |
| *** dane-fichter has joined #openstack-nova | 17:59 | |
| *** fawadkhaliq has quit IRC | 18:00 | |
| *** fawadkhaliq has joined #openstack-nova | 18:00 | |
| *** kmARC has joined #openstack-nova | 18:00 | |
| *** salv-orl_ has quit IRC | 18:01 | |
| *** salv-orl_ has joined #openstack-nova | 18:01 | |
| *** sc68cal has joined #openstack-nova | 18:02 | |
| *** sc68cal has quit IRC | 18:02 | |
| *** sc68cal has joined #openstack-nova | 18:02 | |
| mriedem | dansmith: one teensy thing in https://review.openstack.org/#/c/212065/ and then i'm +2 | 18:04 |
| *** salv-orl_ has quit IRC | 18:04 | |
| *** suresh12 has quit IRC | 18:04 | |
| *** salv-orl_ has joined #openstack-nova | 18:05 | |
| mriedem | mnestratov: the CI ran but it's not actually running any tempest tests that do an image snapshot right? | 18:05 |
| dansmith | mriedem: k thanks | 18:05 |
| dansmith | I'm already sick of this patch | 18:06 |
| mnestratov | mriedem: right. it is cinder CI only cinder snapshots | 18:06 |
| mriedem | mnestratov: that's cool and all, but if it's going to run on nova changes and be a part of libvirt in nova, | 18:07 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move V2.1 API unittest to top level directory https://review.openstack.org/212120 | 18:07 |
| mriedem | it should be running tempest.api.compute tests also | 18:07 |
| mriedem | like tempest.api.compute.images.test_images_oneserver.ImagesOneServerTestJSON.test_create_delete_image in this case | 18:07 |
| mriedem | to test that image create works for virtuozzo backends | 18:07 |
| sdague | dansmith: I sort of feel like the comment here is monty python esc - https://review.openstack.org/#/c/212065/6/nova/tests/unit/virt/libvirt/fakelibvirt.py,cm - it made me laugh | 18:07 |
| dansmith | sdague: um, okay :) | 18:08 |
| sdague | maybe I was just in a mood | 18:08 |
| dansmith | sdague: you have a problem with it or it's just funny? | 18:08 |
| sdague | no, I think it's funny | 18:08 |
| dansmith | the deal is that returns like a c struct turned into a python array | 18:08 |
| dansmith | heh, okay | 18:08 |
| dansmith | mriedem: so I replied to one comment. On the spawn thing, I think that using mock locally is better than using stubs, no? | 18:09 |
| *** echoingumesh has joined #openstack-nova | 18:09 | |
| sdague | thanks for updating the help string, I think I'm +2 now. | 18:09 |
| *** echoingumesh has quit IRC | 18:09 | |
| openstackgerrit | dane-fichter proposed openstack/nova: libvirt: Raise error when booting from encrypted volumes https://review.openstack.org/203784 | 18:09 |
| dansmith | sdague: cool, I poured over that for a while trying to not say "you'll fsck this up" | 18:09 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 18:09 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Move objects registration in tests directory https://review.openstack.org/212159 | 18:09 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add fields to request spec object https://review.openstack.org/212168 | 18:09 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create RequestSpec object https://review.openstack.org/145528 | 18:09 |
| *** echoingumesh has joined #openstack-nova | 18:10 | |
| mriedem | dansmith: on test_compute_mgr that's fine, as noted it's not an issue - the stub is globally used in the older test_compute style stuff | 18:10 |
| dansmith | okay | 18:10 |
| mriedem | i just remember a stub existing for just that thing | 18:10 |
| *** salv-orlando has joined #openstack-nova | 18:10 | |
| dansmith | yeah, I wrote it :) | 18:10 |
| mriedem | i remember | 18:10 |
| mriedem | i think i approved it | 18:10 |
| mriedem | +W | 18:11 |
| *** kmARC has quit IRC | 18:11 | |
| *** vilobhmm has joined #openstack-nova | 18:11 | |
| *** unicell has quit IRC | 18:11 | |
| *** unicell has joined #openstack-nova | 18:11 | |
| *** salv-orl_ has quit IRC | 18:12 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 18:12 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Move objects registration in tests directory https://review.openstack.org/212159 | 18:12 |
| *** SourabhP has joined #openstack-nova | 18:13 | |
| mnestratov | mriedem: compute CI is being created, I hope it will be ready next week | 18:13 |
| mriedem | mnestratov: oh i see, virtuozzo storage CI is for cinder stuff, so you'll have something like virtuozzo compute CI? | 18:14 |
| *** echoingumesh has quit IRC | 18:14 | |
| mriedem | couldn't they just be the same thing? | 18:14 |
| *** jswarren has quit IRC | 18:14 | |
| mnestratov | mriedem: right, actually waiting for hardware | 18:15 |
| mriedem | i.e. you already have this, you just need to enable the compute api tests for things that now work | 18:15 |
| mriedem | still not understanding.... | 18:15 |
| *** atuvenie has joined #openstack-nova | 18:15 | |
| mriedem | if you're running ci on cinder and nova today, | 18:15 |
| mriedem | it seems you're ready to go | 18:15 |
| mriedem | just need to enable tests for the things that work now | 18:16 |
| mnestratov | mriedem: our current hardware can't stand the workload from both - we checkked | 18:16 |
| *** fawadkhaliq has quit IRC | 18:16 | |
| *** lmtaylor has joined #openstack-nova | 18:16 | |
| mriedem | how is it working today? | 18:16 |
| mriedem | selective rechecks? | 18:16 |
| *** suresh12 has joined #openstack-nova | 18:16 | |
| mnestratov | no, all events from nova and cinder, just selective number of tests | 18:17 |
| mnestratov | related to cinder | 18:17 |
| mriedem | ok, well until the hw shows up, could you just enable tempest.api.compute.images.test_images_oneserver.ImagesOneServerTestJSON.test_create_delete_image ? | 18:18 |
| mnestratov | ok, I'll try | 18:18 |
| mriedem | otherwise i can wait until you have the other stuff too, but i won't drop the -1 | 18:19 |
| *** angdraug has joined #openstack-nova | 18:19 | |
| mnestratov | mriedem: understood | 18:19 |
| *** gholler has joined #openstack-nova | 18:19 | |
| mnestratov | mriedem: and one more thing that differs compute CI from cinder is that cinder CI runs our vzstorage driver with KVM libvirt_type because it is more mature, while compute will run with parallels virt_type | 18:25 |
| mnestratov | mriedem: so enabling tempest.api.compute.images.test_images_oneserver.ImagesOneServerTestJSON.test_create_delete_image on current CI will not test changes in https://review.openstack.org/#/c/184153/ | 18:27 |
| *** echoingumesh has joined #openstack-nova | 18:32 | |
| *** Sukhdev has quit IRC | 18:32 | |
| *** ivasev has quit IRC | 18:32 | |
| mriedem | oh | 18:34 |
| mriedem | hrm | 18:34 |
| *** salv-orlando has quit IRC | 18:39 | |
| *** rushil has quit IRC | 18:39 | |
| rlrossit | mriedem: if you're bored, which I'm sure you are, can you snag a look at this if you feel like it? https://review.openstack.org/#/c/197135/ | 18:42 |
| *** rushil has joined #openstack-nova | 18:42 | |
| mriedem | oh not that again | 18:43 |
| rlrossit | lol | 18:43 |
| rlrossit | at least the dust from the objects flurry has settled | 18:43 |
| rlrossit | I'm back at the top of the rabbit hole | 18:43 |
| *** ndipanov has quit IRC | 18:44 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/nova: Allow virt driver to define binding:host_id https://review.openstack.org/194413 | 18:44 |
| *** cp16net has joined #openstack-nova | 18:46 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/nova: Allow virt driver to define binding:host_id https://review.openstack.org/194413 | 18:46 |
| *** shyama has quit IRC | 18:47 | |
| *** baoli_ has quit IRC | 18:48 | |
| *** exploreshaifali has quit IRC | 18:49 | |
| *** baoli has joined #openstack-nova | 18:49 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/nova: Allow virt driver to define binding:host_id https://review.openstack.org/194413 | 18:49 |
| *** xyang2 has joined #openstack-nova | 18:54 | |
| *** Nikolay_St has quit IRC | 18:54 | |
| *** cp16net has left #openstack-nova | 18:57 | |
| *** xyang1 has quit IRC | 18:57 | |
| *** baoli has quit IRC | 18:58 | |
| *** gholler_ has joined #openstack-nova | 18:58 | |
| *** baoli has joined #openstack-nova | 18:58 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove/deprecate conductor instance_update() https://review.openstack.org/189155 | 18:58 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Make compute manager instance updates use objects https://review.openstack.org/189154 | 18:58 |
| *** baoli has quit IRC | 18:58 | |
| *** vivekd has joined #openstack-nova | 19:00 | |
| *** SourabhP has quit IRC | 19:01 | |
| *** rlrossit has left #openstack-nova | 19:01 | |
| *** gholler has quit IRC | 19:01 | |
| *** fawadkhaliq has joined #openstack-nova | 19:01 | |
| *** SourabhP has joined #openstack-nova | 19:02 | |
| openstackgerrit | whitekid proposed openstack/nova: tuning extended volume extensions https://review.openstack.org/211258 | 19:04 |
| *** mwagner_lap has quit IRC | 19:04 | |
| *** fawadkhaliq has quit IRC | 19:06 | |
| *** bapalm_ is now known as bapalm | 19:07 | |
| *** dustins has quit IRC | 19:12 | |
| *** SourabhP has quit IRC | 19:12 | |
| *** exploreshaifali has joined #openstack-nova | 19:12 | |
| *** SourabhP has joined #openstack-nova | 19:18 | |
| *** lordd has quit IRC | 19:23 | |
| *** armax has quit IRC | 19:23 | |
| *** SourabhP has quit IRC | 19:24 | |
| *** kmARC has joined #openstack-nova | 19:25 | |
| *** mdrabe has quit IRC | 19:31 | |
| *** lordd has joined #openstack-nova | 19:31 | |
| *** salv-orlando has joined #openstack-nova | 19:31 | |
| *** mdrabe has joined #openstack-nova | 19:31 | |
| *** mdrabe has quit IRC | 19:32 | |
| *** lordd has quit IRC | 19:33 | |
| *** ijw_ has joined #openstack-nova | 19:33 | |
| *** mdrabe has joined #openstack-nova | 19:33 | |
| *** fifieldt_ has joined #openstack-nova | 19:33 | |
| *** busterswt has quit IRC | 19:34 | |
| *** lordd has joined #openstack-nova | 19:34 | |
| *** fifieldt has quit IRC | 19:34 | |
| openstackgerrit | Dan Smith proposed openstack/nova: neutron: filter None port_ids from ports list in _unbind_ports https://review.openstack.org/206236 | 19:35 |
| *** ijw has quit IRC | 19:35 | |
| *** salv-orlando has quit IRC | 19:36 | |
| mriedem | wut?! | 19:37 |
| *** rushil has quit IRC | 19:37 | |
| *** rlrossit has joined #openstack-nova | 19:37 | |
| mriedem | heh, i saw that typo the other day too | 19:37 |
| mriedem | dansmith: thanks | 19:37 |
| dansmith | heh | 19:37 |
| mriedem | alaski: not making friends with the openstackclient boyz https://developer.rackspace.com/blog/introducing-rack-global-cli/ | 19:38 |
| *** lordd has quit IRC | 19:38 | |
| *** dane-fichter has quit IRC | 19:39 | |
| mriedem | alaski: btw, in singling you out, i'm intentionally saying that was your idea | 19:40 |
| *** SourabhP has joined #openstack-nova | 19:40 | |
| *** ijw has joined #openstack-nova | 19:43 | |
| dansmith | I assumed it was | 19:43 |
| *** VW_ has quit IRC | 19:44 | |
| *** alejandrito has quit IRC | 19:44 | |
| *** VW_ has joined #openstack-nova | 19:44 | |
| *** hemna has quit IRC | 19:45 | |
| *** ijw_ has quit IRC | 19:46 | |
| *** Sukhdev has joined #openstack-nova | 19:47 | |
| alaski | yep, you know how I love to hold stuff back from upstream | 19:49 |
| *** VW_ has quit IRC | 19:49 | |
| *** salv-orlando has joined #openstack-nova | 19:50 | |
| *** meghal has quit IRC | 19:51 | |
| alaski | they must have gotten tired of the flak for rax-novaclient and decided that rewriting in go would fix everything | 19:51 |
| *** achanda has quit IRC | 19:52 | |
| *** sudipto has quit IRC | 19:54 | |
| *** kmARC has quit IRC | 19:58 | |
| *** annegentle has quit IRC | 20:02 | |
| *** ijw_ has joined #openstack-nova | 20:03 | |
| *** praneshp has quit IRC | 20:04 | |
| *** ijw has quit IRC | 20:06 | |
| *** geekinutah has quit IRC | 20:07 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Rename classes containing 'v3' to 'v21' https://review.openstack.org/212734 | 20:12 |
| *** VW_ has joined #openstack-nova | 20:12 | |
| *** nagyz has joined #openstack-nova | 20:16 | |
| *** meghal has joined #openstack-nova | 20:16 | |
| *** praneshp has joined #openstack-nova | 20:19 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/212751 | 20:20 |
| *** Alexandra_ has joined #openstack-nova | 20:20 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 20:21 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/212764 | 20:21 |
| *** alejandrito has joined #openstack-nova | 20:28 | |
| *** Piet has quit IRC | 20:29 | |
| *** baoli has joined #openstack-nova | 20:29 | |
| *** atuvenie has quit IRC | 20:29 | |
| *** beraldo has joined #openstack-nova | 20:31 | |
| *** _Alexandra_ has joined #openstack-nova | 20:31 | |
| *** rushil has joined #openstack-nova | 20:32 | |
| openstackgerrit | Matthew Treinish proposed openstack/nova: Add sample config file to nova docs https://review.openstack.org/212778 | 20:32 |
| *** xyang2 has quit IRC | 20:32 | |
| mtreinish | mriedem, dansmith: ^^^ I'm not sure that's something you'll want | 20:32 |
| *** Alexandra_ has quit IRC | 20:32 | |
| mtreinish | either way it won't work until the next oslo.config release is cut | 20:33 |
| *** achanda has joined #openstack-nova | 20:33 | |
| *** rushil_ has joined #openstack-nova | 20:33 | |
| *** baoli has quit IRC | 20:34 | |
| mriedem | mtreinish: i'm sure tjones will want it | 20:34 |
| tjones | yes!!! please!!! | 20:35 |
| *** _Alexandra_ is now known as dewsday-WFB | 20:36 | |
| *** chuckcarmack has left #openstack-nova | 20:36 | |
| tjones | now if only it was in kilo.... | 20:36 |
| *** rushil has quit IRC | 20:36 | |
| *** Marga_ has quit IRC | 20:36 | |
| *** krtaylor has quit IRC | 20:37 | |
| mtreinish | tjones: heh, that'd get tricky. But you can hack the oslo.config module into a nova kilo backport | 20:37 |
| tjones | :-) | 20:37 |
| *** Marga_ has joined #openstack-nova | 20:38 | |
| mtreinish | tjones: like just copy what I did for tempest: http://git.openstack.org/cgit/openstack/tempest/tree/doc/source/conf.py#n16 (that's what the oslo module is based on) | 20:38 |
| mtreinish | I'd likely +2 that as a stable patch | 20:38 |
| tjones | cool - let me take alook | 20:38 |
| mtreinish | although config opts probably shouldn't be changing on stable | 20:38 |
| tjones | its not that they change (on kilo) it is that tox -egenconfig misses a bunch of config options. | 20:39 |
| mtreinish | tjones: oh, that likely won't change with this. Since it's just calling oslo-config-generator under the covers | 20:40 |
| mtreinish | this just bakes that into build sphinx so it publishes on each commit | 20:40 |
| *** Marga_ has quit IRC | 20:41 | |
| *** annegentle has joined #openstack-nova | 20:41 | |
| *** Marga_ has joined #openstack-nova | 20:41 | |
| tjones | mtreinish: kilo doens't use the oslo-config-generator. It uses a generate_sample.sh script | 20:42 |
| tjones | and i have not had a chance to dig into why it's skipping oslo and some other projects (which i now cannot recall) | 20:42 |
| tjones | that's why mriedem knew i would be interested in this cause i was asking about it a couple of weeks ago | 20:43 |
| *** ijw_ has quit IRC | 20:44 | |
| mtreinish | tjones: ah ok, its using the oslo-incubator generator. Yeah that one had some issues | 20:44 |
| *** ijw has joined #openstack-nova | 20:44 | |
| *** setuid has quit IRC | 20:45 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add hacking check for eventlet.spawn() https://review.openstack.org/211643 | 20:47 |
| *** fawadkhaliq has joined #openstack-nova | 20:48 | |
| *** krtaylor has joined #openstack-nova | 20:49 | |
| mtreinish | tjones: https://review.openstack.org/#/c/212783/ I'm not sure it'll just work, I imagine the config file might need some pruning | 20:50 |
| mriedem | nova meeting in 10 minutes | 20:50 |
| *** rdo has quit IRC | 20:50 | |
| mtreinish | but if you wanted to run with it that would be the basic patch to fix things on kilo | 20:50 |
| *** Marga_ has quit IRC | 20:50 | |
| *** mjturek1 has quit IRC | 20:51 | |
| *** Guest61394 is now known as med_ | 20:51 | |
| *** med_ has joined #openstack-nova | 20:51 | |
| tjones | mtreinish: thanks! | 20:52 |
| *** rdo has joined #openstack-nova | 20:52 | |
| tjones | i'll give it atry | 20:52 |
| mtreinish | I was actually surprised how cleanly it merged | 20:52 |
| mriedem | we run a tight ship around here | 20:52 |
| mtreinish | mriedem: heh, sure | 20:53 |
| tjones | :-) | 20:53 |
| *** n0ano has joined #openstack-nova | 20:53 | |
| scottda | mriedem: Do you know if 'nova rebuild' is always on the same compute host? | 20:53 |
| mtreinish | tjones: I just ran it locally, and it doesnt work | 20:53 |
| mriedem | scottda: i think it depends on if it's an evacuate | 20:53 |
| mtreinish | the issue is the opts file it creates to generate the lists poing to opts that aren't there on kilo | 20:54 |
| *** kmARC has joined #openstack-nova | 20:54 | |
| mtreinish | which is about what I expected | 20:54 |
| *** setuid has joined #openstack-nova | 20:54 | |
| scottda | ok, thanks | 20:54 |
| *** ijw_ has joined #openstack-nova | 20:55 | |
| mriedem | scottda: dansmith will gladly answer all of your questions about evacuate and rebuild | 20:55 |
| * dansmith goes poof | 20:55 | |
| scottda | haha | 20:55 |
| dansmith | scottda: I believe rebuild is done in place, yeah | 20:55 |
| dansmith | scottda: evacuate is a rebuild on another host by definition | 20:56 |
| mriedem | scottda: this is the evacuate block http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/manager.py#n2605 | 20:56 |
| tjones | mtreinish: wel thanks for trying it out. | 20:56 |
| scottda | thanks guys. | 20:56 |
| *** ijw has quit IRC | 20:58 | |
| *** thangp has quit IRC | 20:58 | |
| *** VW_ has quit IRC | 20:58 | |
| *** kmARC has quit IRC | 20:58 | |
| *** VW_ has joined #openstack-nova | 20:58 | |
| *** raildo is now known as raildo-afk | 20:59 | |
| *** VW_ has quit IRC | 21:03 | |
| openstackgerrit | Michael Still proposed openstack/nova: Removing dummy mac address in the dhcp conf during shelving https://review.openstack.org/85249 | 21:04 |
| *** aysyd has quit IRC | 21:06 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Remove last of the plugins/v3 from unit tests https://review.openstack.org/212793 | 21:12 |
| *** tsbot_ has quit IRC | 21:13 | |
| *** thorst has quit IRC | 21:14 | |
| *** VW_ has joined #openstack-nova | 21:14 | |
| *** kmARC has joined #openstack-nova | 21:14 | |
| *** rfolco has quit IRC | 21:14 | |
| *** Piet has joined #openstack-nova | 21:15 | |
| *** vishwanathj has joined #openstack-nova | 21:17 | |
| *** suresh12 has quit IRC | 21:18 | |
| *** xyang2 has joined #openstack-nova | 21:19 | |
| *** VW_ has quit IRC | 21:19 | |
| *** alex_klimov has joined #openstack-nova | 21:19 | |
| *** tonytan4ever has quit IRC | 21:20 | |
| *** Sukhdev has quit IRC | 21:21 | |
| *** suresh12 has joined #openstack-nova | 21:21 | |
| *** kmARC_ has joined #openstack-nova | 21:22 | |
| *** armax has joined #openstack-nova | 21:22 | |
| *** Sukhdev has joined #openstack-nova | 21:24 | |
| *** otter768 has joined #openstack-nova | 21:26 | |
| *** agireud has quit IRC | 21:26 | |
| *** alex_klimov has quit IRC | 21:27 | |
| *** rlrossit has left #openstack-nova | 21:28 | |
| *** agireud has joined #openstack-nova | 21:28 | |
| *** dewsday-WFB has quit IRC | 21:30 | |
| *** otter768 has quit IRC | 21:30 | |
| *** lmtaylor has quit IRC | 21:32 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: Cleanup of compute api reboot method https://review.openstack.org/127427 | 21:33 |
| tjones | rebasing again - perma rebase ;-) | 21:33 |
| tjones | twice today :-) | 21:33 |
| *** tonytan4ever has joined #openstack-nova | 21:34 | |
| mriedem | perma rebase is trademarked | 21:35 |
| mriedem | you owe me $5 | 21:35 |
| mriedem | perma rebase (tm) | 21:36 |
| tjones | hehe | 21:36 |
| melwitt | alaski: for the request spec persistence, are we planning to have db api methods for getting it and saving it? I was just looking at your patch | 21:37 |
| alaski | melwitt: they're embedded in the objects. I wasn't planning to add them to the api.py file, now | 21:38 |
| alaski | *no | 21:38 |
| *** VW_ has joined #openstack-nova | 21:40 | |
| *** alejandrito has quit IRC | 21:41 | |
| *** tonytan4ever has quit IRC | 21:42 | |
| *** gyee has joined #openstack-nova | 21:42 | |
| *** tonytan4ever has joined #openstack-nova | 21:42 | |
| *** annegentle has quit IRC | 21:45 | |
| *** mriedem is now known as mriedem_away | 21:45 | |
| *** tonytan4ever has quit IRC | 21:46 | |
| bauzas | alaski: I'm back by tomorrow, any patch I should review for persist the ReqSpec ? | 21:49 |
| alaski | bauzas: there are a few dependent on your first object patch | 21:50 |
| alaski | so just follow the trail in gerrit | 21:50 |
| bauzas | alaski: okay, I will look at them | 21:50 |
| bauzas | alaski: I just wanted to make sure not to miss them in my morning | 21:51 |
| alaski | bauzas: understood. my patches start here https://review.openstack.org/#/c/212168/ | 21:51 |
| alaski | and I rebased and updated yours for updated object dependencies | 21:52 |
| bauzas | alaski: FYI I'll be traveling this weekend to Seattle so we could be discussing next week on the same TZ if needed | 21:52 |
| alaski | bauzas: cool. I'm on the other coast though. but still much closer | 21:53 |
| bauzas | alaski: coolio, will look at Gerrit tomorrow morning EU TZ | 21:53 |
| bauzas | np | 21:53 |
| *** burt has quit IRC | 21:58 | |
| *** baoli has joined #openstack-nova | 22:00 | |
| *** alexpilotti has joined #openstack-nova | 22:03 | |
| *** armax has quit IRC | 22:04 | |
| *** morgan_302 has quit IRC | 22:04 | |
| *** baoli has quit IRC | 22:05 | |
| *** morganfainberg has joined #openstack-nova | 22:07 | |
| *** exploreshaifali has quit IRC | 22:09 | |
| *** gyee is now known as gyee_500 | 22:12 | |
| *** agireud has quit IRC | 22:13 | |
| *** annegentle has joined #openstack-nova | 22:15 | |
| *** VW_ has quit IRC | 22:18 | |
| *** meghal has quit IRC | 22:18 | |
| *** VW_ has joined #openstack-nova | 22:18 | |
| *** jecarey has quit IRC | 22:19 | |
| *** bnemec has quit IRC | 22:20 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 22:21 |
| *** VW_ has quit IRC | 22:23 | |
| tjones | thanks alaski!! for your review on my perma rebase (tm) patch | 22:23 |
| *** meghal has joined #openstack-nova | 22:26 | |
| alaski | tjones: np. I'm surprised it needs so much rebasing. It doesn't affect much code | 22:26 |
| *** mdrabe has quit IRC | 22:27 | |
| edleafe | alaski: tjones: it's because of the v3 cleanup work. We're shuffling around a lot of code. | 22:27 |
| tjones | yeah every day at least once it says "patch in merge conflict" | 22:27 |
| tjones | but it's always a trivial rebase | 22:27 |
| alaski | edleafe: well hurry it up already :) | 22:28 |
| edleafe | alaski: heh | 22:29 |
| *** geekinutah has joined #openstack-nova | 22:29 | |
| edleafe | alaski: tjones: See what I mean: https://review.openstack.org/#/q/topic:bug/1462901,n,z | 22:29 |
| *** jamielennox|away is now known as jamielennox | 22:33 | |
| *** vivekd has quit IRC | 22:34 | |
| tjones | wow | 22:35 |
| *** nic has quit IRC | 22:35 | |
| *** meghal has quit IRC | 22:38 | |
| *** kmARC_ has quit IRC | 22:39 | |
| melwitt | alaski: I'm also reviewing that reboot cleanup patch and was wondering, what does it mean, "in practice it could never succeed" with a hard reboot in progress? | 22:40 |
| *** kmARC has quit IRC | 22:40 | |
| *** beraldo has quit IRC | 22:43 | |
| melwitt | if I understand correctly, the current code allows a reboot request while it's "hard rebooting" I'm guessing in the case where it's stuck there. and this patch disallows reboot during hard rebooting but says it never worked? | 22:43 |
| *** eharney has quit IRC | 22:44 | |
| *** eharney has joined #openstack-nova | 22:44 | |
| *** jaypipes has quit IRC | 22:44 | |
| alaski | hmm, I thought there was some protection against a double hard reboot but now I'm not seeing it. | 22:45 |
| tjones | isn't it this vm_state=set(vm_states.ALLOW_HARD_REBOOT), | 22:45 |
| tjones | task_state=task_states.ALLOW_REBOOT) | 22:45 |
| *** annegentle has quit IRC | 22:46 | |
| tjones | melwitt: alaski - ALLOW_HARD_REBOOT = ALLOW_SOFT_REBOOT + [STOPPED, PAUSED, SUSPENDED, ERROR] | 22:48 |
| *** edmondsw has quit IRC | 22:48 | |
| alaski | tjones: melwitt it was this that I was thinking of https://review.openstack.org/#/c/132950/4/nova/compute/api.py,cm | 22:49 |
| alaski | the removed code | 22:49 |
| melwitt | tjones: what I mean is, I'm looking for how removing REBOOTING_HARD is keeping the same behavior | 22:49 |
| alaski | I think it's not now | 22:49 |
| *** alexpilotti has quit IRC | 22:49 | |
| tjones | i think chris was saying that was the wrong behavoir - at least that is how i interpreted it. that you should not allow hard reboot when in hard reboot. the previous code allowed that | 22:50 |
| melwitt | hm, okay. I'm wary of that being that instances getting "stuck" in a task_state is pretty common in my experience and it's helpful to be able to kick them without a reset-state | 22:51 |
| alaski | tjones: we deemed that a bug worth a fix in https://review.openstack.org/#/c/132950/4 | 22:51 |
| melwitt | but the commit message says "it never worked anyway" and that's what I'm trying to understand | 22:51 |
| alaski | when that patch was first proposed it would not have worked | 22:52 |
| *** tpeoples has quit IRC | 22:52 | |
| melwitt | ahhh | 22:52 |
| alaski | https://review.openstack.org/#/c/132950/ merged after that | 22:52 |
| tjones | melwitt: alaski: ok let me add that part back in | 22:52 |
| melwitt | mystery solved | 22:52 |
| tjones | tada | 22:52 |
| melwitt | thanks alaski | 22:52 |
| tjones | sugh | 22:52 |
| tjones | oops | 22:52 |
| tjones | wrong window | 22:52 |
| alaski | melwitt: I was just trying to defend my apparently bad vote. thanks for catching that | 22:53 |
| melwitt | alaski: haha! :) | 22:54 |
| *** rushil_ has quit IRC | 22:55 | |
| *** morganfainberg is now known as morgan_404 | 22:57 | |
| *** earlephilhower has quit IRC | 22:59 | |
| *** meghal has joined #openstack-nova | 23:04 | |
| *** ijuwang has quit IRC | 23:09 | |
| *** cfriesen_ has quit IRC | 23:11 | |
| *** cfriesen_ has joined #openstack-nova | 23:13 | |
| *** suresh12_ has joined #openstack-nova | 23:16 | |
| *** suresh12_ has quit IRC | 23:18 | |
| *** jamielennox is now known as jamielennox|away | 23:18 | |
| *** suresh12_ has joined #openstack-nova | 23:19 | |
| *** suresh12 has quit IRC | 23:19 | |
| *** meghal has quit IRC | 23:21 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: Cleanup of compute api reboot method https://review.openstack.org/127427 | 23:23 |
| cfriesen_ | anyone aware of an updated libvirt package for ubuntu 14.04? I'd like to test out hugepage stuff with devstack. | 23:23 |
| tjones | alaski: melwitt: updated above. added REBOOTING_HARD to ALLOW_REBOOT | 23:24 |
| *** Sukhdev has quit IRC | 23:24 | |
| *** meghal has joined #openstack-nova | 23:27 | |
| *** otter768 has joined #openstack-nova | 23:27 | |
| clarkb | cfriesen_: there is the cloud archive with 1.2.12 for liberty | 23:28 |
| *** dannywilson has quit IRC | 23:29 | |
| *** sachin has quit IRC | 23:29 | |
| cfriesen_ | clarkb: I see it. thanks. | 23:29 |
| *** sachin has joined #openstack-nova | 23:30 | |
| *** otter768 has quit IRC | 23:31 | |
| *** krtaylor has quit IRC | 23:32 | |
| *** eharney has quit IRC | 23:32 | |
| melwitt | tjones: thanks. I think the unit tests will fail that are making sure reboot during hard reboot raises a conflict (they will need to be removed too). sorry I forgot to mention that | 23:35 |
| tjones | i did run them - at least the ones in compute.api | 23:38 |
| tjones | i guess i should run them all :-) | 23:39 |
| tjones | no worries | 23:39 |
| tjones | let me check | 23:39 |
| tjones | hm - the code looks like one should have failed… https://review.openstack.org/#/c/127427/40/nova/tests/unit/compute/test_compute_api.py,cm | 23:40 |
| melwitt | tjones: run the ones in test_server_actions, I think you'll just want to keep the old test_reboot_hard_with_hard_in_progress instead of the two "raises conflicts" ones | 23:40 |
| tjones | thanks! | 23:41 |
| *** jwcroppe has quit IRC | 23:42 | |
| *** zzzeek has quit IRC | 23:42 | |
| *** jwcroppe has joined #openstack-nova | 23:43 | |
| *** jwcroppe has quit IRC | 23:43 | |
| *** thorst has joined #openstack-nova | 23:43 | |
| *** jwcroppe has joined #openstack-nova | 23:43 | |
| *** Sukhdev has joined #openstack-nova | 23:44 | |
| *** SourabhP has quit IRC | 23:44 | |
| *** Alexandra_ has joined #openstack-nova | 23:44 | |
| melwitt | tjones: okay, it looks like soft reboot is only allowed during task state None, so the test that ensures it fails during hard rebooting will remain correct (in test_compute_api). and I don't see a test for hard reboot + hard rebooting so that's why we don't see a change in test running there | 23:45 |
| *** krtaylor has joined #openstack-nova | 23:45 | |
| *** Alexandra_ has quit IRC | 23:45 | |
| melwitt | tjones: which also means the test_reboot_soft_with_hard_in_progress_raises_conflict should still succeed. the only test I expect to fail is test_reboot_hard_with_hard_in_progress_raises_conflict with the current patch set | 23:45 |
| *** Alexandra_ has joined #openstack-nova | 23:46 | |
| tjones | im running the tests now and you are right - test_reboot_hard_with_hard_in_progress_raises_conflict fails. i'll fix that now | 23:46 |
| melwitt | tjones: so only test_reboot_hard_with_hard_in_progress_raises_conflict should be removed, if it's failing | 23:46 |
| tjones | :-) | 23:46 |
| *** jamielennox|away is now known as jamielennox | 23:47 | |
| *** jwcroppe has quit IRC | 23:48 | |
| *** Alexandra_ is now known as dewsday | 23:49 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: Cleanup of compute api reboot method https://review.openstack.org/127427 | 23:50 |
| tjones | melwitt: ^^ thanks for your help | 23:50 |
| *** fawadkhaliq has quit IRC | 23:50 | |
| *** markvoelker has quit IRC | 23:50 | |
| *** diogogmt has quit IRC | 23:51 | |
| melwitt | tjones: cool, will review it | 23:51 |
| *** meghal has quit IRC | 23:51 | |
| tjones | thanks! | 23:52 |
| tjones | i'm going to run to the bus in 7 minutes but will be back online shortly if you have comments | 23:53 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Remove v3 references in unit test 'contrib' https://review.openstack.org/212851 | 23:53 |
| edleafe | alex_xu: Got the functional tests figured out, and a few others, too ^ | 23:53 |
| melwitt | tjones: it looks fine to me. I'm going to wait for jenkins to vote on it | 23:55 |
| tjones | thanks melwitt | 23:55 |
| *** penick has quit IRC | 23:57 | |
| *** apoorvad has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!