| *** zzzeek has quit IRC | 00:06 | |
| *** diogogmt has joined #openstack-nova | 00:09 | |
| *** echoingumesh has quit IRC | 00:10 | |
| *** shyu has joined #openstack-nova | 00:15 | |
| *** bapalm has joined #openstack-nova | 00:24 | |
| *** thorst has quit IRC | 00:25 | |
| *** ijw has joined #openstack-nova | 00:29 | |
| *** tsekiyam_ has joined #openstack-nova | 00:30 | |
| *** ijw_ has quit IRC | 00:32 | |
| *** mtanino has quit IRC | 00:32 | |
| *** tsekiyama has quit IRC | 00:33 | |
| *** tsekiyam_ has quit IRC | 00:34 | |
| *** Sukhdev has quit IRC | 00:38 | |
| *** mili_ has joined #openstack-nova | 00:40 | |
| *** mili has quit IRC | 00:42 | |
| *** bapalm has quit IRC | 00:44 | |
| *** bapalm has joined #openstack-nova | 00:45 | |
| *** bapalm has quit IRC | 00:46 | |
| *** bapalm has joined #openstack-nova | 00:46 | |
| *** chhavi has joined #openstack-nova | 00:57 | |
| *** mili_ has quit IRC | 01:01 | |
| *** mvensky has quit IRC | 01:02 | |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: Servicegroup foundational refactoring for Control Plane https://review.openstack.org/202714 | 01:03 |
|---|---|---|
| *** jdurgin has joined #openstack-nova | 01:05 | |
| *** leeantho has quit IRC | 01:06 | |
| *** shihanzhang has quit IRC | 01:08 | |
| openstackgerrit | Marian Horban proposed openstack/nova: Added processing /compute URL https://review.openstack.org/195303 | 01:09 |
| *** baoli has joined #openstack-nova | 01:10 | |
| *** nic has quit IRC | 01:11 | |
| *** meghal has quit IRC | 01:12 | |
| *** agireud has joined #openstack-nova | 01:15 | |
| *** tobe_ has joined #openstack-nova | 01:18 | |
| *** moshele has joined #openstack-nova | 01:18 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Resize: Return to initial state instead of ACTIVE when resize fails https://review.openstack.org/202379 | 01:19 |
| *** ssurana has quit IRC | 01:19 | |
| *** bapalm has quit IRC | 01:21 | |
| *** bapalm has joined #openstack-nova | 01:21 | |
| *** jwcropp__ has quit IRC | 01:23 | |
| *** coolsvap_ has joined #openstack-nova | 01:24 | |
| *** jwcroppe has joined #openstack-nova | 01:24 | |
| *** browne has quit IRC | 01:25 | |
| *** zhenguo has joined #openstack-nova | 01:25 | |
| *** bapalm has quit IRC | 01:26 | |
| *** kiwik has quit IRC | 01:26 | |
| *** RuiChen has joined #openstack-nova | 01:27 | |
| *** liusheng has joined #openstack-nova | 01:27 | |
| *** agireud has quit IRC | 01:28 | |
| *** jwcroppe has quit IRC | 01:28 | |
| openstackgerrit | Kylin CG proposed openstack/python-novaclient: Remove _discover_extensions https://review.openstack.org/198257 | 01:31 |
| *** skylerberg has quit IRC | 01:33 | |
| *** shuquan has joined #openstack-nova | 01:35 | |
| *** coolsvap_ has quit IRC | 01:36 | |
| *** slong_ has joined #openstack-nova | 01:38 | |
| *** slong has quit IRC | 01:38 | |
| *** shuquan has quit IRC | 01:39 | |
| *** shuquan has joined #openstack-nova | 01:40 | |
| *** baoli has quit IRC | 01:40 | |
| *** jwcroppe has joined #openstack-nova | 01:41 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move existing V2 to legacy_v2 - step 2 https://review.openstack.org/211390 | 01:41 |
| *** jwcroppe has quit IRC | 01:41 | |
| *** shyu has quit IRC | 01:42 | |
| *** jwcroppe has joined #openstack-nova | 01:42 | |
| edleafe | alex_xu: johnthetubaguy: ^ second in the series | 01:42 |
| *** jwcroppe has quit IRC | 01:46 | |
| alex_xu | edleafe: good morning! | 01:47 |
| alex_xu | edleafe: looks like you want to separate the move v2 code into two parts | 01:48 |
| alex_xu | edleafe: I'm try to work on contrib/ reference to legacy_v2/contrib modules | 01:50 |
| *** otter768 has joined #openstack-nova | 01:50 | |
| *** shuquan has quit IRC | 01:50 | |
| *** baoli has joined #openstack-nova | 01:52 | |
| *** shyu has joined #openstack-nova | 01:54 | |
| *** otter768 has quit IRC | 01:54 | |
| openstackgerrit | Sachi King proposed openstack/nova: DNM: Run constraints job as py27 for experemental https://review.openstack.org/211392 | 01:58 |
| *** shyu has quit IRC | 01:59 | |
| *** jdandrea has quit IRC | 02:00 | |
| *** tobe_ has quit IRC | 02:05 | |
| *** tobe_ has joined #openstack-nova | 02:06 | |
| openstackgerrit | whitekid proposed openstack/nova: tuning extended volume extensions https://review.openstack.org/211258 | 02:07 |
| *** vilobhmm1 has quit IRC | 02:08 | |
| *** SourabhP has quit IRC | 02:13 | |
| *** unicell has quit IRC | 02:15 | |
| *** patrickeast has quit IRC | 02:16 | |
| *** shuquan has joined #openstack-nova | 02:16 | |
| *** shuquan has quit IRC | 02:16 | |
| *** tpeoples has joined #openstack-nova | 02:19 | |
| *** shuquan has joined #openstack-nova | 02:22 | |
| *** jwcroppe has joined #openstack-nova | 02:25 | |
| *** baoli has quit IRC | 02:26 | |
| *** slong_ has quit IRC | 02:29 | |
| *** slong_ has joined #openstack-nova | 02:30 | |
| edleafe | alex_xu: hi, busy evening here. Yes, the idea is to break the patches into reviewable chunks | 02:31 |
| *** mdrabe has joined #openstack-nova | 02:31 | |
| edleafe | alex_xu: so nothing different from what we were doing before, just smaller steps :) | 02:31 |
| alex_xu | edleafe: yea, thanks, that is good idea, let me continue work on it | 02:31 |
| edleafe | alex_xu: I have to go back now, so I'll see what you get done by my tomorrow morning | 02:32 |
| alex_xu | edleafe: that is good for review | 02:32 |
| alex_xu | edleafe: yea, see you tomorrow | 02:32 |
| *** unicell has joined #openstack-nova | 02:34 | |
| *** annashen has joined #openstack-nova | 02:35 | |
| *** yamahata has quit IRC | 02:36 | |
| *** mdrabe has quit IRC | 02:38 | |
| *** bkopilov has quit IRC | 02:38 | |
| *** browne has joined #openstack-nova | 02:41 | |
| *** venkat_p has joined #openstack-nova | 02:42 | |
| *** haigang has joined #openstack-nova | 02:43 | |
| *** diogogmt has quit IRC | 02:44 | |
| *** xyang1 has quit IRC | 02:44 | |
| *** tobe_ has quit IRC | 02:47 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add instance action events for live migration https://review.openstack.org/195372 | 02:47 |
| *** tobe_ has joined #openstack-nova | 02:49 | |
| *** hakimo_ has joined #openstack-nova | 02:52 | |
| *** meghal has joined #openstack-nova | 02:52 | |
| *** meghal1 has joined #openstack-nova | 02:54 | |
| *** shuquan has quit IRC | 02:54 | |
| *** hakimo has quit IRC | 02:54 | |
| *** meghal has quit IRC | 02:56 | |
| *** tpeoples has quit IRC | 02:57 | |
| *** rotbeard has joined #openstack-nova | 02:58 | |
| *** ijw has quit IRC | 03:00 | |
| *** otter768 has joined #openstack-nova | 03:00 | |
| *** ijw has joined #openstack-nova | 03:00 | |
| *** ijw has quit IRC | 03:00 | |
| *** shuquan has joined #openstack-nova | 03:02 | |
| *** jwcroppe has quit IRC | 03:05 | |
| *** david-lyle has quit IRC | 03:21 | |
| *** jwcroppe has joined #openstack-nova | 03:24 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Allow display project-id for server groups https://review.openstack.org/209011 | 03:24 |
| *** chhavi has quit IRC | 03:26 | |
| *** fawadkhaliq has joined #openstack-nova | 03:29 | |
| *** vilobhmm has joined #openstack-nova | 03:31 | |
| *** tjones has joined #openstack-nova | 03:33 | |
| *** jdurgin has quit IRC | 03:41 | |
| *** annashen has quit IRC | 03:44 | |
| *** logan2 has quit IRC | 03:47 | |
| *** shuquan has quit IRC | 03:51 | |
| *** annashen has joined #openstack-nova | 03:51 | |
| *** hhuang has quit IRC | 03:54 | |
| *** jkraj has joined #openstack-nova | 03:54 | |
| *** logan2 has joined #openstack-nova | 03:55 | |
| *** jwcroppe has quit IRC | 03:57 | |
| *** Sree has joined #openstack-nova | 04:01 | |
| *** Sree has quit IRC | 04:02 | |
| *** Sree has joined #openstack-nova | 04:02 | |
| *** otter768 has quit IRC | 04:03 | |
| *** bkopilov has joined #openstack-nova | 04:03 | |
| *** shuquan has joined #openstack-nova | 04:08 | |
| *** annashen has quit IRC | 04:08 | |
| *** shuquan has quit IRC | 04:09 | |
| *** shyu has joined #openstack-nova | 04:11 | |
| *** tobe_ has quit IRC | 04:12 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Use StrOpt's parameter choices to restritct option auth_strategy https://review.openstack.org/200814 | 04:14 |
| *** tobe_ has joined #openstack-nova | 04:17 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: libvirt: use instance UUID with exception InstanceNotFound https://review.openstack.org/183492 | 04:20 |
| *** skylerberg has joined #openstack-nova | 04:20 | |
| *** tjones has quit IRC | 04:21 | |
| *** fawadkhaliq has quit IRC | 04:21 | |
| *** achanda has quit IRC | 04:22 | |
| *** Sukhdev has joined #openstack-nova | 04:24 | |
| *** jdurgin has joined #openstack-nova | 04:27 | |
| *** agireud has joined #openstack-nova | 04:30 | |
| *** jecarey has joined #openstack-nova | 04:31 | |
| *** deepakcs has joined #openstack-nova | 04:36 | |
| *** jkraj has quit IRC | 04:38 | |
| *** moshele has quit IRC | 04:39 | |
| *** exploreshaifali has joined #openstack-nova | 04:39 | |
| *** david-lyle has joined #openstack-nova | 04:40 | |
| *** rotbeard has quit IRC | 04:40 | |
| *** ildikov has quit IRC | 04:46 | |
| *** shuquan has joined #openstack-nova | 04:49 | |
| *** tpeoples has joined #openstack-nova | 04:51 | |
| *** chhavi has joined #openstack-nova | 04:54 | |
| *** annashen has joined #openstack-nova | 04:55 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Move existing V2 to legacy_v2 - step 2 https://review.openstack.org/211390 | 05:01 |
| openstackgerrit | Alex Xu proposed openstack/nova: Move existing V2 to legacy_v2 https://review.openstack.org/211356 | 05:01 |
| *** geekinutah has quit IRC | 05:02 | |
| *** Sree_ has joined #openstack-nova | 05:02 | |
| *** Sree_ is now known as Guest68623 | 05:03 | |
| *** Marga_ has quit IRC | 05:03 | |
| *** shuquan has quit IRC | 05:03 | |
| *** Marga_ has joined #openstack-nova | 05:04 | |
| *** Sree has quit IRC | 05:05 | |
| *** Sree has joined #openstack-nova | 05:07 | |
| *** jkraj has joined #openstack-nova | 05:07 | |
| *** irenab has quit IRC | 05:08 | |
| *** shuquan has joined #openstack-nova | 05:10 | |
| *** Guest68623 has quit IRC | 05:10 | |
| *** Sree has quit IRC | 05:11 | |
| *** Sree has joined #openstack-nova | 05:12 | |
| *** annashen has quit IRC | 05:14 | |
| *** ssurana has joined #openstack-nova | 05:16 | |
| *** achanda has joined #openstack-nova | 05:17 | |
| *** shyama has joined #openstack-nova | 05:19 | |
| *** annashen has joined #openstack-nova | 05:19 | |
| *** shuquan has quit IRC | 05:21 | |
| *** slong_ has quit IRC | 05:22 | |
| *** fawadkhaliq has joined #openstack-nova | 05:22 | |
| *** moshele has joined #openstack-nova | 05:22 | |
| *** ildikov has joined #openstack-nova | 05:24 | |
| *** exploreshaifali has quit IRC | 05:24 | |
| *** ijw has joined #openstack-nova | 05:26 | |
| *** slong has joined #openstack-nova | 05:27 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Filter out inactive compute before computing avaible resources https://review.openstack.org/180155 | 05:29 |
| *** fawadk has joined #openstack-nova | 05:31 | |
| *** fawadkhaliq has quit IRC | 05:31 | |
| *** meghal1 has quit IRC | 05:32 | |
| *** liusheng has quit IRC | 05:34 | |
| *** sc68cal has quit IRC | 05:37 | |
| *** Nikolay_St has quit IRC | 05:38 | |
| *** Sree has quit IRC | 05:41 | |
| *** jdurgin has quit IRC | 05:41 | |
| *** Sree has joined #openstack-nova | 05:42 | |
| *** belmoreira has joined #openstack-nova | 05:43 | |
| *** liusheng has joined #openstack-nova | 05:44 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Move existing V2 to legacy_v2 - step 2 https://review.openstack.org/211390 | 05:46 |
| *** jecarey_ has joined #openstack-nova | 05:46 | |
| *** irenab has joined #openstack-nova | 05:47 | |
| *** Sukhdev has quit IRC | 05:48 | |
| *** jecarey has quit IRC | 05:49 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Move existing V2 to legacy_v2 - step 2 https://review.openstack.org/211390 | 05:52 |
| *** irenab has quit IRC | 05:55 | |
| *** ijw has quit IRC | 05:55 | |
| *** irenab has joined #openstack-nova | 05:56 | |
| *** josecastroleon has joined #openstack-nova | 05:57 | |
| *** vilobhmm has quit IRC | 05:57 | |
| *** meghal has joined #openstack-nova | 05:57 | |
| *** vilobhmm has joined #openstack-nova | 05:59 | |
| *** SourabhP_ has joined #openstack-nova | 06:02 | |
| *** otter768 has joined #openstack-nova | 06:04 | |
| *** jecarey_ has quit IRC | 06:04 | |
| *** chhavi has quit IRC | 06:05 | |
| openstackgerrit | zhengyue proposed openstack/nova: Fix abnormal quota usage after restore by admin https://review.openstack.org/211432 | 06:07 |
| *** otter768 has quit IRC | 06:08 | |
| *** agireud has quit IRC | 06:10 | |
| *** moshele has quit IRC | 06:11 | |
| *** Murali_ has joined #openstack-nova | 06:11 | |
| *** moshele has joined #openstack-nova | 06:11 | |
| *** sahid has joined #openstack-nova | 06:20 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/209780 | 06:23 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 06:23 |
| *** sahid has quit IRC | 06:25 | |
| *** sahid has joined #openstack-nova | 06:26 | |
| *** chhavi has joined #openstack-nova | 06:26 | |
| *** annashen has quit IRC | 06:30 | |
| *** scheuran has joined #openstack-nova | 06:35 | |
| *** jkraj has quit IRC | 06:36 | |
| *** jwcroppe has joined #openstack-nova | 06:37 | |
| *** Sree_ has joined #openstack-nova | 06:40 | |
| *** Sree_ is now known as Guest42299 | 06:40 | |
| *** Sree has quit IRC | 06:43 | |
| haigang | hi guys | 06:43 |
| haigang | I got a problem during create instance, after nova boot, the host's default root modified | 06:44 |
| haigang | the host default route changed to dev interface qvbXXXX | 06:45 |
| haigang | anybody have idea on this? | 06:45 |
| *** sc68cal has joined #openstack-nova | 06:49 | |
| *** garyk has quit IRC | 06:49 | |
| *** rdopiera has joined #openstack-nova | 06:50 | |
| *** sc68cal has quit IRC | 06:50 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 06:51 |
| *** matrohon has joined #openstack-nova | 06:52 | |
| *** slong has quit IRC | 06:54 | |
| *** garyk has joined #openstack-nova | 06:54 | |
| *** exploreshaifali has joined #openstack-nova | 06:56 | |
| *** slong has joined #openstack-nova | 06:57 | |
| *** wanghua has quit IRC | 06:57 | |
| *** slong has quit IRC | 06:58 | |
| *** wanghua has joined #openstack-nova | 06:58 | |
| *** achanda has quit IRC | 07:00 | |
| *** jlanoux has joined #openstack-nova | 07:00 | |
| *** sc68cal has joined #openstack-nova | 07:02 | |
| *** achanda has joined #openstack-nova | 07:03 | |
| *** alex_klimov has joined #openstack-nova | 07:03 | |
| *** achanda has quit IRC | 07:05 | |
| *** Guest42299 has quit IRC | 07:05 | |
| *** annashen has joined #openstack-nova | 07:06 | |
| *** ssurana has quit IRC | 07:07 | |
| *** exploreshaifali has quit IRC | 07:12 | |
| *** afazekas has joined #openstack-nova | 07:13 | |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: Servicegroup foundational refactoring for Control Plane https://review.openstack.org/202714 | 07:13 |
| *** vilobhmm has quit IRC | 07:14 | |
| *** cfriesen_ has quit IRC | 07:17 | |
| *** unicell has quit IRC | 07:17 | |
| *** unicell has joined #openstack-nova | 07:17 | |
| *** skylerberg has quit IRC | 07:19 | |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/nova: Omit RescheduledException in instance_fault.message https://review.openstack.org/165751 | 07:22 |
| *** heyongli has quit IRC | 07:22 | |
| *** jkraj has joined #openstack-nova | 07:22 | |
| *** kodoku has joined #openstack-nova | 07:32 | |
| openstackgerrit | Jeffrey Zhang proposed openstack/nova: Cleanup the the test_api_validation unittest https://review.openstack.org/207376 | 07:32 |
| kodoku | Hi, nova use what version for contact cinder ? V1 or V2 ? Because I use catalog_info=volume:cinder:internalURL But nova use publicURL CINDER V2. | 07:43 |
| *** sc68cal has quit IRC | 07:44 | |
| *** SourabhP_ has quit IRC | 07:44 | |
| *** Sree has joined #openstack-nova | 07:46 | |
| *** markus_z has joined #openstack-nova | 07:48 | |
| *** browne has quit IRC | 07:50 | |
| *** sc68cal has joined #openstack-nova | 07:52 | |
| *** yamahata has joined #openstack-nova | 07:52 | |
| *** Jeffrey4l has joined #openstack-nova | 07:58 | |
| openstackgerrit | Meghal Gosalia proposed openstack/nova-specs: Quota by Flavor, Quota by AZ or Quota by (Flavor and AZ) https://review.openstack.org/206160 | 07:58 |
| *** dguitarbite has joined #openstack-nova | 07:58 | |
| *** humble_ has joined #openstack-nova | 07:59 | |
| *** shuquan has joined #openstack-nova | 07:59 | |
| *** romainh has joined #openstack-nova | 07:59 | |
| *** jcoufal has joined #openstack-nova | 08:00 | |
| *** wanghua has quit IRC | 08:02 | |
| *** danpb has joined #openstack-nova | 08:02 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 08:02 |
| *** annashen has quit IRC | 08:02 | |
| *** tpeoples has quit IRC | 08:02 | |
| *** annashen has joined #openstack-nova | 08:03 | |
| *** chhavi has quit IRC | 08:04 | |
| *** jistr has joined #openstack-nova | 08:04 | |
| *** otter768 has joined #openstack-nova | 08:05 | |
| *** annashen has quit IRC | 08:08 | |
| *** heyongli has joined #openstack-nova | 08:09 | |
| *** otter768 has quit IRC | 08:09 | |
| *** jordanP has joined #openstack-nova | 08:09 | |
| *** boris-42 has quit IRC | 08:10 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Filter out inactive compute before computing avaible resources https://review.openstack.org/180155 | 08:11 |
| *** kmARC has joined #openstack-nova | 08:12 | |
| *** links has joined #openstack-nova | 08:13 | |
| *** shuquan has quit IRC | 08:13 | |
| *** heyongli has quit IRC | 08:14 | |
| *** sahid has quit IRC | 08:15 | |
| *** yassine__ has joined #openstack-nova | 08:16 | |
| *** sahid has joined #openstack-nova | 08:17 | |
| *** heyongli has joined #openstack-nova | 08:17 | |
| meghal | lxsli: yt ? | 08:18 |
| openstackgerrit | Meghal Gosalia proposed openstack/nova-specs: Quota by Flavor, Quota by AZ or Quota by (Flavor and AZ) https://review.openstack.org/206160 | 08:19 |
| *** lucasagomes has joined #openstack-nova | 08:21 | |
| *** heyongli has quit IRC | 08:22 | |
| *** shuquan has joined #openstack-nova | 08:23 | |
| *** fawadk has quit IRC | 08:25 | |
| *** fawadkhaliq has joined #openstack-nova | 08:26 | |
| *** shuquan_ has joined #openstack-nova | 08:27 | |
| *** kodoku has quit IRC | 08:27 | |
| *** shuquan has quit IRC | 08:27 | |
| *** shuquan_ is now known as shuquan | 08:27 | |
| *** Sree__ has joined #openstack-nova | 08:29 | |
| *** meghal has quit IRC | 08:31 | |
| *** heyongli has joined #openstack-nova | 08:31 | |
| *** Sree has quit IRC | 08:32 | |
| *** ndipanov has quit IRC | 08:33 | |
| *** ndipanov has joined #openstack-nova | 08:33 | |
| *** heyongli has quit IRC | 08:36 | |
| *** shuquan has quit IRC | 08:36 | |
| openstackgerrit | yalei wang proposed openstack/nova: Support unaddressed port https://review.openstack.org/202882 | 08:37 |
| lxsli | meghal: morning | 08:40 |
| *** moshele has quit IRC | 08:40 | |
| lxsli | meghal: I haven't checked your spec again yet, I've been looking at other things | 08:41 |
| *** moshele has joined #openstack-nova | 08:41 | |
| *** moshele has quit IRC | 08:41 | |
| *** shuquan has joined #openstack-nova | 08:41 | |
| *** sahid has quit IRC | 08:41 | |
| *** moshele has joined #openstack-nova | 08:41 | |
| *** shuquan has quit IRC | 08:44 | |
| *** sahid has joined #openstack-nova | 08:44 | |
| *** gszasz has joined #openstack-nova | 08:45 | |
| *** heyongli has joined #openstack-nova | 08:45 | |
| *** arunman has joined #openstack-nova | 08:46 | |
| *** garyk has quit IRC | 08:47 | |
| *** meghal has joined #openstack-nova | 08:48 | |
| *** heyongli has quit IRC | 08:50 | |
| *** meghal1 has joined #openstack-nova | 08:51 | |
| *** meghal has quit IRC | 08:53 | |
| *** ankit_ag has joined #openstack-nova | 08:55 | |
| *** heyongli has joined #openstack-nova | 08:56 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 08:56 |
| *** yassine__ has quit IRC | 09:01 | |
| *** yassine__ has joined #openstack-nova | 09:01 | |
| *** heyongli has quit IRC | 09:02 | |
| *** heyongli has joined #openstack-nova | 09:05 | |
| johnthetubaguy | alex_xu: hows things this morning, I am seeing a green jenkins on those two patches :) | 09:06 |
| *** yassine__ has quit IRC | 09:09 | |
| *** meghal1 has quit IRC | 09:09 | |
| *** alex_klimov has quit IRC | 09:09 | |
| *** heyongli has quit IRC | 09:12 | |
| *** yassine__ has joined #openstack-nova | 09:12 | |
| *** shyu has quit IRC | 09:13 | |
| *** moshele has quit IRC | 09:13 | |
| *** shuquan has joined #openstack-nova | 09:14 | |
| openstackgerrit | whitekid proposed openstack/nova: tuning extended volume extensions https://review.openstack.org/211258 | 09:14 |
| *** moshele has joined #openstack-nova | 09:14 | |
| *** shuquan has quit IRC | 09:16 | |
| *** alex_klimov has joined #openstack-nova | 09:17 | |
| *** heyongli has joined #openstack-nova | 09:17 | |
| *** Nikolay_St has joined #openstack-nova | 09:18 | |
| *** shuquan has joined #openstack-nova | 09:19 | |
| *** shuquan has quit IRC | 09:21 | |
| *** alex_klimov has quit IRC | 09:22 | |
| *** alex_klimov has joined #openstack-nova | 09:22 | |
| *** heyongli has quit IRC | 09:22 | |
| *** shuquan has joined #openstack-nova | 09:23 | |
| *** shuquan has quit IRC | 09:28 | |
| *** heyongli has joined #openstack-nova | 09:28 | |
| *** marzif_ has joined #openstack-nova | 09:29 | |
| *** shuquan has joined #openstack-nova | 09:30 | |
| *** shyu has joined #openstack-nova | 09:30 | |
| lxsli | Simple review: https://review.openstack.org/#/c/207040/ | 09:30 |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Test patch libvirt race condition (do not merge) https://review.openstack.org/191732 | 09:32 |
| johnthetubaguy | lxsli: if you add a bug, it could be part of the bug push tomorrow. | 09:32 |
| lxsli | johnthetubaguy: I'm not sure you can hit it externally, I just noticed it when I was reading code | 09:33 |
| johnthetubaguy | lxsli: probably still worth a bug though, its good to tidy up potential bugs | 09:34 |
| *** Savemech has joined #openstack-nova | 09:34 | |
| lxsli | johnthetubaguy: OK, sure | 09:34 |
| johnthetubaguy | its more for advertising really | 09:34 |
| alex_xu | johnthetubaguy: yea, it ready for review https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1462901,n,z :) | 09:34 |
| *** heyongli has quit IRC | 09:34 | |
| johnthetubaguy | alex_xu: I should give that a look now | 09:34 |
| alex_xu | johnthetubaguy: thanks | 09:34 |
| johnthetubaguy | alex_xu: is the plan to do the v3 move next? | 09:34 |
| alex_xu | johnthetubaguy: yes, that is the plan | 09:35 |
| alex_xu | sdague: we have nova api meeting today? | 09:36 |
| *** shuquan has quit IRC | 09:36 | |
| *** moshele has quit IRC | 09:37 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Prevent (un)pinning unknown CPUs https://review.openstack.org/207040 | 09:38 |
| lxsli | bug #1483613 :) | 09:38 |
| openstack | bug 1483613 in OpenStack Compute (nova) "It may be possible to request (un)pinning of CPUs not in the NUMA cpuset" [Undecided,In progress] https://launchpad.net/bugs/1483613 - Assigned to Alexis Lee (alexisl) | 09:38 |
| *** heyongli has joined #openstack-nova | 09:38 | |
| *** fesp has joined #openstack-nova | 09:40 | |
| openstackgerrit | Jordan Pittier proposed openstack/nova: libvirt:on snapshot delete, use qemu-img to blockRebase if VM is stopped https://review.openstack.org/192736 | 09:42 |
| *** shuquan has joined #openstack-nova | 09:42 | |
| *** hanzhang has joined #openstack-nova | 09:44 | |
| *** moshele has joined #openstack-nova | 09:48 | |
| *** heyongli has quit IRC | 09:50 | |
| openstackgerrit | Alexander Schmidt proposed openstack/nova: libvirt: enable live migration with serial console https://review.openstack.org/191035 | 09:51 |
| openstackgerrit | Michael Still proposed openstack/nova: libvirt: Race condition leads to instance in error https://review.openstack.org/185549 | 09:53 |
| openstackgerrit | whitekid proposed openstack/nova: replace memcache.get to memcache.get_multi for better performance https://review.openstack.org/211250 | 09:55 |
| openstackgerrit | Michael Still proposed openstack/nova: libvirt: Race condition leads to instance in error https://review.openstack.org/185549 | 09:55 |
| *** heyongli has joined #openstack-nova | 09:55 | |
| *** obondarev_ has quit IRC | 09:56 | |
| *** xgsa has quit IRC | 09:57 | |
| *** Sree has joined #openstack-nova | 09:59 | |
| *** shuquan has quit IRC | 10:01 | |
| *** Sree__ has quit IRC | 10:02 | |
| *** annashen has joined #openstack-nova | 10:03 | |
| *** heyongli has quit IRC | 10:03 | |
| *** obondarev has joined #openstack-nova | 10:04 | |
| *** romainh has quit IRC | 10:05 | |
| *** heyongli has joined #openstack-nova | 10:05 | |
| *** otter768 has joined #openstack-nova | 10:05 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Change docstring of api_versions.discover_version https://review.openstack.org/211177 | 10:08 |
| *** annashen has quit IRC | 10:08 | |
| *** otter768 has quit IRC | 10:10 | |
| *** xgsa has joined #openstack-nova | 10:11 | |
| *** lucasagomes has quit IRC | 10:13 | |
| *** dheeraj-gupta-4 has joined #openstack-nova | 10:13 | |
| *** mhorban has quit IRC | 10:13 | |
| *** dheeraj-gupta-4 has left #openstack-nova | 10:14 | |
| sdague | alex_xu: we can, I kind of assumed it would start next week, but I can meet today and not friday | 10:18 |
| sdague | and it looks like the ical merged :) | 10:18 |
| *** IanGovett has joined #openstack-nova | 10:21 | |
| *** fawadkhaliq has quit IRC | 10:22 | |
| *** fawadkhaliq has joined #openstack-nova | 10:22 | |
| *** Sree has quit IRC | 10:25 | |
| *** Sree has joined #openstack-nova | 10:26 | |
| *** heyongli has quit IRC | 10:31 | |
| *** claudiub has joined #openstack-nova | 10:32 | |
| *** heyongli has joined #openstack-nova | 10:34 | |
| *** yamahata has quit IRC | 10:35 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Move existing V2 to legacy_v2 - step 2 https://review.openstack.org/211390 | 10:35 |
| openstackgerrit | Alex Xu proposed openstack/nova: Move existing V2 to legacy_v2 https://review.openstack.org/211356 | 10:35 |
| *** josecastroleon has quit IRC | 10:38 | |
| *** dguitarbite has quit IRC | 10:41 | |
| *** jcoufal_ has joined #openstack-nova | 10:42 | |
| *** jwcroppe has quit IRC | 10:42 | |
| *** lucasagomes has joined #openstack-nova | 10:43 | |
| *** jcoufal__ has joined #openstack-nova | 10:43 | |
| *** jcoufal has quit IRC | 10:45 | |
| *** heyongli has quit IRC | 10:46 | |
| *** jcoufal_ has quit IRC | 10:47 | |
| *** heyongli has joined #openstack-nova | 10:50 | |
| *** shyu has quit IRC | 10:55 | |
| *** heyongli has quit IRC | 10:55 | |
| *** heyongli has joined #openstack-nova | 10:56 | |
| *** marcusvrn has joined #openstack-nova | 10:56 | |
| *** rotbeard has joined #openstack-nova | 10:57 | |
| *** Sree__ has joined #openstack-nova | 10:59 | |
| *** xgsa has quit IRC | 11:00 | |
| *** yassine__ has quit IRC | 11:00 | |
| *** romainh has joined #openstack-nova | 11:00 | |
| *** atuvenie has joined #openstack-nova | 11:01 | |
| *** shyama has quit IRC | 11:01 | |
| *** Sree has quit IRC | 11:03 | |
| *** dikonoor has joined #openstack-nova | 11:04 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Move existing V2 to legacy_v2 - step 2 https://review.openstack.org/211390 | 11:08 |
| *** chhavi has joined #openstack-nova | 11:09 | |
| *** heyongli has quit IRC | 11:10 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Walk through the API, Conductor and Compute code https://review.openstack.org/210467 | 11:10 |
| *** heyongli has joined #openstack-nova | 11:11 | |
| johnthetubaguy | alex_xu: got some comments on https://review.openstack.org/#/c/211390, includes the ones you just fixed, but there are others | 11:12 |
| sahid | sdague: hello, perhaps you could ack this one https://review.openstack.org/#/c/176360/ it's waiting for a while now | 11:14 |
| sahid | i tried to put it here https://etherpad.openstack.org/p/liberty-nova-priorities-tracking but unfortunatly did not get more reviews | 11:15 |
| *** xgsa has joined #openstack-nova | 11:15 | |
| *** heyongli has quit IRC | 11:16 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Move existing V2 to legacy_v2 - step 2 https://review.openstack.org/211390 | 11:17 |
| openstackgerrit | Alex Xu proposed openstack/nova: Move existing V2 to legacy_v2 https://review.openstack.org/211356 | 11:17 |
| *** heyongli has joined #openstack-nova | 11:19 | |
| *** lucasagomes is now known as lucas-hungry | 11:23 | |
| *** humble_ has quit IRC | 11:25 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 11:26 |
| *** heyongli has quit IRC | 11:29 | |
| *** xgsa has quit IRC | 11:31 | |
| *** heyongli has joined #openstack-nova | 11:31 | |
| *** dikonoor has quit IRC | 11:31 | |
| *** jlanoux has quit IRC | 11:35 | |
| *** mwagner_lap has quit IRC | 11:35 | |
| *** yassine__ has joined #openstack-nova | 11:35 | |
| *** belmoreira has quit IRC | 11:35 | |
| *** thorst1 has joined #openstack-nova | 11:36 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Copy cli arguments in api_versions.wraps decorator https://review.openstack.org/210563 | 11:36 |
| *** Jeffrey4l has quit IRC | 11:36 | |
| *** heyongli has quit IRC | 11:36 | |
| *** josecastroleon has joined #openstack-nova | 11:38 | |
| *** eglynn has joined #openstack-nova | 11:38 | |
| alex_xu | johnthetubaguy: thanks, just fixed | 11:40 |
| *** fesp has quit IRC | 11:40 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Copy cli arguments in api_versions.wraps decorator https://review.openstack.org/210563 | 11:40 |
| *** lucas-hungry is now known as lucasagomes | 11:43 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 11:45 |
| *** shyama has joined #openstack-nova | 11:46 | |
| *** liusheng has quit IRC | 11:49 | |
| *** coolsvap has quit IRC | 11:50 | |
| *** tobe_ has quit IRC | 11:50 | |
| *** bkopilov has quit IRC | 11:50 | |
| *** erlon has joined #openstack-nova | 11:51 | |
| *** coolsvap has joined #openstack-nova | 11:51 | |
| *** tobe_ has joined #openstack-nova | 11:52 | |
| alex_xu | nova api meeting in 7 mins wiht new time | 11:53 |
| *** alexschm has joined #openstack-nova | 11:53 | |
| alex_xu | s/wiht/with/ | 11:53 |
| *** xgsa has joined #openstack-nova | 11:54 | |
| *** lyanchih has joined #openstack-nova | 11:55 | |
| *** tobe_ has quit IRC | 11:57 | |
| *** jlanoux has joined #openstack-nova | 11:57 | |
| *** Sree__ has quit IRC | 11:58 | |
| alex_xu | nova api meeting is running gmann sdague johnthetubaguy | 12:01 |
| sdague | coming | 12:01 |
| *** dguitarbite has joined #openstack-nova | 12:02 | |
| *** liusheng has joined #openstack-nova | 12:02 | |
| *** chhavi has quit IRC | 12:02 | |
| *** openstackgerrit_ has joined #openstack-nova | 12:02 | |
| *** xgsa has quit IRC | 12:03 | |
| *** annashen has joined #openstack-nova | 12:04 | |
| *** otter768 has joined #openstack-nova | 12:06 | |
| *** Jeffrey4l has joined #openstack-nova | 12:08 | |
| *** annashen has quit IRC | 12:09 | |
| *** heyongli has joined #openstack-nova | 12:10 | |
| *** otter768 has quit IRC | 12:11 | |
| *** marzif__ has joined #openstack-nova | 12:14 | |
| *** marzif__ has quit IRC | 12:16 | |
| *** marzif__ has joined #openstack-nova | 12:16 | |
| *** marzif_ has quit IRC | 12:17 | |
| *** heyongli has quit IRC | 12:19 | |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: libvirt: Race condition leads to instance in error https://review.openstack.org/185549 | 12:19 |
| *** jecarey has joined #openstack-nova | 12:20 | |
| *** arunman has quit IRC | 12:21 | |
| *** burgerk has joined #openstack-nova | 12:24 | |
| *** thumpba has joined #openstack-nova | 12:25 | |
| *** andreykurilin has quit IRC | 12:25 | |
| *** aysyd has joined #openstack-nova | 12:26 | |
| *** kylek3h has joined #openstack-nova | 12:26 | |
| *** bapalm has joined #openstack-nova | 12:26 | |
| *** bapalm has quit IRC | 12:26 | |
| *** andreykurilin has joined #openstack-nova | 12:27 | |
| *** bapalm has joined #openstack-nova | 12:27 | |
| openstackgerrit | whitekid proposed openstack/nova: use memcache.get_multi instead of memcache.get better performance https://review.openstack.org/211250 | 12:28 |
| *** chhavi has joined #openstack-nova | 12:28 | |
| *** heyongli has joined #openstack-nova | 12:29 | |
| *** chhavi has quit IRC | 12:29 | |
| *** chhavi has joined #openstack-nova | 12:29 | |
| *** tojuvone has joined #openstack-nova | 12:29 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id when list server-groups https://review.openstack.org/209018 | 12:29 |
| *** jwcroppe has joined #openstack-nova | 12:30 | |
| openstackgerrit | James Page proposed openstack/nova: Add canonicalization for non-x86 architectures https://review.openstack.org/211041 | 12:30 |
| *** bnemec has joined #openstack-nova | 12:30 | |
| openstackgerrit | James Page proposed openstack/nova: Add canonicalization for non-x86 architectures https://review.openstack.org/211041 | 12:31 |
| openstackgerrit | James Page proposed openstack/nova: Add canonicalization for non-x86 architectures https://review.openstack.org/211041 | 12:32 |
| *** rfolco has joined #openstack-nova | 12:33 | |
| *** edmondsw has joined #openstack-nova | 12:33 | |
| *** heyongli has quit IRC | 12:34 | |
| *** haigang has quit IRC | 12:35 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Switch from deprecated timeutils.isotime https://review.openstack.org/211146 | 12:36 |
| *** venkat_p has quit IRC | 12:36 | |
| lyanchih | Hi guys, plase help to review our patch https://review.openstack.org/#/c/205766/ | 12:36 |
| *** lyanchih has left #openstack-nova | 12:36 | |
| *** lyanchih has joined #openstack-nova | 12:36 | |
| *** chuckcarmack has joined #openstack-nova | 12:38 | |
| *** rushiagr_away is now known as rushiagr | 12:38 | |
| chhavi | johnthetubaguy: Hi, I'm interesting in seeing if I could help with the multi-attach-volume blueprint that looks like it will occur in the Mitaka release. I work on the PowerVM driver, and can drive the implementation there. But I'm curious if there are any other things I could potentially help out with? | 12:39 |
| *** mdrabe has joined #openstack-nova | 12:41 | |
| *** shyama has quit IRC | 12:43 | |
| *** shyama has joined #openstack-nova | 12:43 | |
| *** jaypipes has joined #openstack-nova | 12:44 | |
| lucasagomes | johnthetubaguy, hi there, thanks for the review in https://review.openstack.org/#/c/206614/ . I replied to that. Do you guys have any guidance of when a spec is needed in Nova? For example if the change impacts in multiple areas of the project (API and database model) then you need a spec | 12:44 |
| lucasagomes | cause for that change I'm unsure we actually need, it's just internal stuff of how we request and retrieve information to/from Ironic | 12:45 |
| *** bapalm_ has joined #openstack-nova | 12:45 | |
| johnthetubaguy | chhavi: hi, yes, we are in bug fixing time now really | 12:45 |
| *** thumpba has quit IRC | 12:45 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Fix incorrect result of host-describe https://review.openstack.org/211083 | 12:45 |
| johnthetubaguy | chhavi: help to review code, and fix bugs is probably the best thing at this point, depends how that fits in with your plans, happy to talk specifics if thats useful :) | 12:46 |
| johnthetubaguy | lucasagomes: lots of the detail is in here: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule and http://docs.openstack.org/developer/nova/blueprints.html#specs | 12:47 |
| *** Piet has quit IRC | 12:47 | |
| johnthetubaguy | lucasagomes: for ironic driver stuff, its probably more just needs a blueprint, rather than needs a blueprint and a spec, it just depends on exactly what it is really | 12:47 |
| johnthetubaguy | lucasagomes: let me know if the docs are unclear, and I can help get them updated | 12:48 |
| *** links has quit IRC | 12:48 | |
| lucasagomes | johnthetubaguy, right... yeah because that patch in nova all it does is to not wait for the node to be deleted (once the request is accepted in Ironic Nova should just trust that the instance is going to be deleted) | 12:48 |
| lucasagomes | sync or async | 12:48 |
| lucasagomes | and Ironic will take care of it | 12:48 |
| *** bapalm has quit IRC | 12:49 | |
| lucasagomes | so it just need to delete a loopingcall there | 12:49 |
| lucasagomes | johnthetubaguy, will read the docs, thanks | 12:49 |
| chhavi | johnthetubaguy: are there any specific bugs surrounding this which i can pick up, or just bugs in general considering we are near to openstack release. | 12:49 |
| ndipanov | johnthetubaguy, it really makes very little sense to have a blueprint for this change in the ironic driver | 12:51 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Don't check flavor disk size when booting from volume https://review.openstack.org/204635 | 12:51 |
| jaypipes | ndipanov, sahid: thx much for the reviews on the PCI patch series. I'll address your comments shortly. | 12:51 |
| *** openstackgerrit_ has quit IRC | 12:51 | |
| ndipanov | jaypipes, I thought I +2ed most of it | 12:51 |
| ndipanov | ah yes the private thing | 12:52 |
| *** thumpba has joined #openstack-nova | 12:52 | |
| johnthetubaguy | ndipanov: its good for our users to know when that feature was added | 12:52 |
| jaypipes | ndipanov: ya. it's only "publicly" called in tests. | 12:52 |
| johnthetubaguy | ndipanov: right now, we want to focus on bugs and priority things, not general features (see previous discussions), hence the feature is blocked | 12:52 |
| johnthetubaguy | ndipanov: more than likely doesn't need a spec, agreed | 12:53 |
| *** jwcroppe has quit IRC | 12:53 | |
| lucasagomes | johnthetubaguy, but it's a bug right? In nova you can call delete for an instance in spawning. Ironic didn't support that and it's broken (as per bug). We are just working to allow it in Ironic so the interface that Nova expects is there | 12:53 |
| lucasagomes | it's not a new thing, it just that it's currently broken in Ironic | 12:53 |
| johnthetubaguy | chhavi: I would look at the high priority ones rather than low priority, and in areas you feel comfortable in, unless you fancy learning a new bit of the code as well, which is fine too :) | 12:53 |
| johnthetubaguy | lucasagomes: thats just a bug fix, then, thats fine | 12:54 |
| johnthetubaguy | lucasagomes: just needs a bug not a blueprint | 12:54 |
| lucasagomes | johnthetubaguy, exactly, there's one there | 12:54 |
| lucasagomes | https://bugs.launchpad.net/nova/+bug/1477490 | 12:54 |
| openstack | Launchpad bug 1477490 in OpenStack Compute (nova) "Ironic: Deleting while spawning can leave orphan ACTIVE nodes in Ironic" [Undecided,In progress] - Assigned to Lucas Alvares Gomes (lucasagomes) | 12:54 |
| johnthetubaguy | lucasagomes: so I blocked it because it attached a blueprint, you got the link? | 12:54 |
| lucasagomes | johnthetubaguy, it's a "related blueprint" The blueprint is actually in ironic | 12:55 |
| lucasagomes | cause big changes goes there | 12:55 |
| lucasagomes | I put in the commit message so one could take a look as it's related | 12:55 |
| lucasagomes | and see what is needed on both projects | 12:55 |
| lucasagomes | (and WIP'd that patch until the blueprint is approved in Ironic) | 12:55 |
| johnthetubaguy | lucasagomes: OK, I just missread the commit message then, if its a bug fix, we can still merge it | 12:56 |
| chhavi | johnthetubaguy: sure let me have a look | 12:56 |
| lucasagomes | johnthetubaguy, cool yeah. Thanks much | 12:56 |
| *** dane-fichter has joined #openstack-nova | 12:56 | |
| *** Kevin_Zheng_ has joined #openstack-nova | 12:56 | |
| *** openstackgerrit_ has joined #openstack-nova | 12:56 | |
| *** jwcroppe has joined #openstack-nova | 12:59 | |
| *** rlrossit has joined #openstack-nova | 13:00 | |
| *** jwcroppe has quit IRC | 13:01 | |
| *** dguitarbite has quit IRC | 13:01 | |
| Kevin_Zheng_ | johnthetubaguy: The patch for adding project_id to os-server-group API response data, the spec and also the fix in novaclient side is ready for review, could you review them and see whether it can be merged to L? Thanks, nova patch : https://review.openstack.org/#/c/209018/ , nova spec : https://review.openstack.org/#/c/209917/ , novaclient fix : | 13:02 |
| Kevin_Zheng_ | https://review.openstack.org/#/c/209011/ | 13:02 |
| alex_xu | sdague: just reminder there is section about extension in devref, in case you didn't know it http://docs.openstack.org/developer/nova/stable_api.html | 13:03 |
| johnthetubaguy | Kevin_Zheng_: I think we need to advertise this with the NovaAPI team, alex_xu and sdague have you seen this one? https://review.openstack.org/#/c/209917/ | 13:04 |
| *** burgerk has quit IRC | 13:04 | |
| *** chhavi has quit IRC | 13:05 | |
| alex_xu | johnthetubaguy: Kevin_Zheng_ add myself as reviewer, will take a look at later | 13:05 |
| edleafe | alex_xu: I'm going to grab some coffee. We can talk about the v3 stuff in a little while. | 13:05 |
| alex_xu | edleafe: ok, no problem | 13:06 |
| *** heyongli has joined #openstack-nova | 13:06 | |
| johnthetubaguy | lucasagomes: sorry, bit confused with how this change is backwards compatible (thinking old ironic, new nova): https://review.openstack.org/#/c/206614/ | 13:06 |
| johnthetubaguy | lucasagomes: read through that upgrade section of the spec, and I guess I am missing something big here | 13:07 |
| *** ankit_ag has quit IRC | 13:07 | |
| johnthetubaguy | lucasagomes: the old system will still need the loop for things to go well, but it looks like we are removing that code from nova? | 13:07 |
| alex_xu | Kevin_Zheng_: can you add APIImpact flag for your patch https://review.openstack.org/#/c/209917/? | 13:08 |
| lucasagomes | johnthetubaguy, yeah we are removing that loop, because for new version of Ironic if Ironic can't delete the instance immediately it will just mark that node and asynchrounously will delete the instance | 13:08 |
| lucasagomes | so no point to loop | 13:08 |
| *** rushiagr is now known as rushiagr_away | 13:08 | |
| lucasagomes | for old versions of Ironic | 13:08 |
| alex_xu | Kevin_Zheng_: that is good for nova api meeting notice your patch | 13:08 |
| *** xyang1 has joined #openstack-nova | 13:08 | |
| alex_xu | s/nova api meeting/nova api team/... | 13:09 |
| lucasagomes | now for old versions of Ironic it will raise Conflict if you try to delete a node that is in DEPLOYING state | 13:09 |
| lucasagomes | saying there's a transition in progress and it can't be aborted | 13:10 |
| *** marzif_ has joined #openstack-nova | 13:10 | |
| lucasagomes | the new code in nova will handle that exception and ignore it | 13:10 |
| lucasagomes | for old versions of nova it won't even call unprovision when a node is in DEPLOYING state | 13:10 |
| lucasagomes | that's the problem | 13:10 |
| lucasagomes | johnthetubaguy, https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L877-L880 | 13:11 |
| johnthetubaguy | lucasagomes: that sounds bad though, doesn't that break users of the old version? I figured we would just detect if we needed the old loop or not? | 13:11 |
| lucasagomes | johnthetubaguy, it's already broken | 13:11 |
| *** marzif_ has quit IRC | 13:11 | |
| *** marzif__ has quit IRC | 13:11 | |
| lucasagomes | johnthetubaguy, for old versions of nova the function which has the loop won't even be invoked | 13:12 |
| *** heyongli has quit IRC | 13:12 | |
| *** marzif_ has joined #openstack-nova | 13:12 | |
| *** dustins has joined #openstack-nova | 13:12 | |
| *** brad_ has quit IRC | 13:12 | |
| lucasagomes | johnthetubaguy, so if you use new version of nova with old version of Ironic, it will still broken (keeping the behavior by ignoring the error that Ironic can't delete an instance that is being deployed) | 13:13 |
| lucasagomes | that was Jim's suggestion on that spec | 13:13 |
| lucasagomes | and that will be fixed as a separated bug | 13:13 |
| *** baoli has joined #openstack-nova | 13:14 | |
| lucasagomes | johnthetubaguy, see https://review.openstack.org/#/c/204162/9/specs/approved/lazy-delete-instances.rst L 241 | 13:15 |
| *** chhavi has joined #openstack-nova | 13:17 | |
| *** ferest has joined #openstack-nova | 13:18 | |
| *** browne has joined #openstack-nova | 13:18 | |
| johnthetubaguy | lucasagomes: can we merge that bug fix before we break the compatibility, bit worried about merge one half of the fix, and not the other half, I should look more into jroll 's suggestion in there | 13:19 |
| lucasagomes | johnthetubaguy, yeah, I first suggested to stop the instance deletion if Ironic doesn't support it. That at least will prevent the instance to be deleted and leave the nodes ACTIVE orphan in Ironic | 13:20 |
| *** fawadkhaliq has quit IRC | 13:20 | |
| lucasagomes | johnthetubaguy, so I put this up just fixing that https://review.openstack.org/#/c/209457/ | 13:21 |
| lucasagomes | I mean, this will make the instance delete to FAIL if Ironic can't delete it | 13:21 |
| lucasagomes | but yeah the full fix requires changes on both Ironic and Nova | 13:22 |
| *** ferest has quit IRC | 13:22 | |
| lucasagomes | that will make the delete instance request to work independent of the state of the deployment in Ironic | 13:22 |
| johnthetubaguy | lucasagomes: but you can't "fix" and already deployed ironic system? | 13:22 |
| *** cfarquhar has joined #openstack-nova | 13:22 | |
| johnthetubaguy | lucasagomes: so the reason for me being worried here, is we assume ever Nova commit is released right, so we can't really merge something that breaks master working with currently deployed ironic, I would love to see the new behaviour for the old version, then adding the bug fix that works with the newer version after that, I hope that makes some sense | 13:23 |
| *** deepakcs has quit IRC | 13:24 | |
| lucasagomes | johnthetubaguy, it won't break because it's already broken | 13:24 |
| sdague | alex_xu: ok, let me figure out how to work from there | 13:24 |
| *** jecarey has quit IRC | 13:24 | |
| sdague | johnthetubaguy: I had not seend that, it also isn't tagged APIImpact | 13:24 |
| *** tpeoples has joined #openstack-nova | 13:25 | |
| johnthetubaguy | sdague: good point, not sure it sounds serious enough to rush it in at this point, but thats a separate discussion | 13:25 |
| johnthetubaguy | sdague: feels more like a feature than a bug fix | 13:25 |
| lucasagomes | johnthetubaguy, that's what I'm trying to say. Cause delete the instance when it's being deployed is already broken. If we merge the nova in code it won't make it better or worse it will continue to be broken until Ironic is also updated | 13:25 |
| *** mwagner_lap has joined #openstack-nova | 13:26 | |
| edleafe | alex_xu: back | 13:27 |
| * edleafe with caffeine | 13:27 | |
| johnthetubaguy | lucasagomes: OK, I am clearly totally confused, I will try come back to this with fresh eyes later | 13:27 |
| lucasagomes | johnthetubaguy, ok, yeah ping myself or jroll when we are available | 13:27 |
| *** irenab has quit IRC | 13:28 | |
| alex_xu | edleafe: just wrote up the etherpad for steps of removing v3 https://etherpad.openstack.org/p/remove_v3 | 13:28 |
| alex_xu | sdague: johnthetubaguy ^ | 13:28 |
| *** annegentle has joined #openstack-nova | 13:28 | |
| jroll | johnthetubaguy: lucasagomes btw, I'm here now for about an hour and then afk for most of the day traveling | 13:28 |
| lucasagomes | jroll, hi there! Going to seattle? | 13:29 |
| jroll | yes sir :) | 13:29 |
| *** ajayaa has joined #openstack-nova | 13:29 | |
| *** heyongli has joined #openstack-nova | 13:29 | |
| lucasagomes | jroll, yeah no I was just talking to johnthetubaguy about the lazy delete instance about what we decided | 13:29 |
| *** ctrath has joined #openstack-nova | 13:29 | |
| *** ivasev has joined #openstack-nova | 13:29 | |
| jroll | right, reading | 13:30 |
| edleafe | alex_xu: so for step 3, do you mean move compute/plugins/v3/* to compute/ ? | 13:30 |
| lucasagomes | jroll, if we use an new version of nova and old version of Ironic, we will handle the Conflict exception that will be raise by the Ironic API and ignore it to keep the old behavior | 13:30 |
| *** xgsa has joined #openstack-nova | 13:30 | |
| alex_xu | edleafe: yes | 13:30 |
| lucasagomes | jroll, so the instance gets delete in Nova but the node still in Ironic (so users won't panic and we can clean the instance in Ironic later), right? | 13:30 |
| *** VW_ has joined #openstack-nova | 13:30 | |
| edleafe | alex_xu: ok, just checking | 13:30 |
| edleafe | alex_xu: I'm not familiar with the resource directory. Why is it needed? | 13:31 |
| jroll | lucasagomes: right... | 13:31 |
| jroll | lucasagomes: it's not handled in that patch though? | 13:31 |
| johnthetubaguy | edleafe: +1 | 13:31 |
| lucasagomes | jroll, no I haven't updated that patch because the spec is not merged yet | 13:31 |
| jroll | lucasagomes: right now it will explode with fire, which isn't the old behavior | 13:31 |
| jroll | ho | 13:31 |
| jroll | oh* | 13:31 |
| lucasagomes | jroll, I just left it as WIP | 13:31 |
| alex_xu | edleafe: it describe in the spec https://github.com/openstack/nova-specs/blob/master/specs/liberty/approved/nova-api-remove-v3.rst | 13:31 |
| alex_xu | just match the directory struct like url | 13:32 |
| johnthetubaguy | alex_xu: lets concentrate on the quickest way to remove the v3 vs v2.1, then follow up with the extra bits | 13:32 |
| jroll | lucasagomes: I think that's what john's confusion is then :P | 13:32 |
| lucasagomes | jroll, yeah the patch _right now_ will just explode. Becuase we had the discussion after I put that patch up and I haven't updated it | 13:32 |
| lucasagomes | ohhhhhh | 13:32 |
| jroll | heh | 13:32 |
| edleafe | alex_xu: ah, I hadn't seen that part | 13:32 |
| lucasagomes | johnthetubaguy, ^ is that? Sorry, yeah the code in the patch *right now* is not updated to match the spec because I'm waiting the spec to be merged | 13:32 |
| alex_xu | johnthetubaguy: ok, I'm ok with that, that can save time for now. But leave similar work in M | 13:33 |
| edleafe | alex_xu: I'd vote for holding off on that until the end | 13:33 |
| johnthetubaguy | jroll lucasagomes: cool, that makes sense now, I just couldn't see how that code would work | 13:33 |
| lucasagomes | johnthetubaguy, yeah sorry... I just left the code in WIP until we land the spec in Ironic | 13:33 |
| lucasagomes | my bad | 13:33 |
| lucasagomes | and then I will update everything | 13:33 |
| alex_xu | sdague: what would you think ^ | 13:34 |
| edleafe | alex_xu: I'll get started on the v3 move. I have lots of meetings today, so I'm not sure how far I'll get. | 13:34 |
| sdague | alex_xu: yeh, restructuring the code to be like the urls can wait | 13:34 |
| sdague | that can be mitaka | 13:34 |
| alex_xu | edleafe: no problem, I will continue your work tomorrow | 13:34 |
| alex_xu | sdague: edleafe ok, cool, we have agreement on, let do it | 13:35 |
| johnthetubaguy | lucasagomes: yeah, no worries, that makes sense now, the info was there, I just wasn't seeing it! | 13:35 |
| lucasagomes | ack, thanks! | 13:36 |
| *** david-lyle has quit IRC | 13:36 | |
| *** burgerk has joined #openstack-nova | 13:36 | |
| *** bkopilov has joined #openstack-nova | 13:36 | |
| edleafe | alex_xu: good. Updated the etherpad | 13:36 |
| alex_xu | edleafe: cool thanks | 13:36 |
| alex_xu | edleafe: step3 just move the v2.1 file without any other rename or move test file | 13:37 |
| *** dikonoo has joined #openstack-nova | 13:38 | |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: libvirt: Race condition leads to instance in error https://review.openstack.org/185549 | 13:38 |
| *** sabari has quit IRC | 13:39 | |
| *** diogogmt has joined #openstack-nova | 13:40 | |
| openstackgerrit | Jordan Pittier proposed openstack/nova: libvirt:on snapshot delete, use qemu-img to blockRebase if VM is stopped https://review.openstack.org/192736 | 13:43 |
| edleafe | alex_xu: I'm in capitalization hell. There is n/a/o/versions.py, that defines both VERSIONS and Versions. Then there is n/a/o/c/versions.py, that defines versions and Versions. Finally there is n/a/o/c/p/v/versions.py, that defines versions and Versions | 13:44 |
| *** jecarey has joined #openstack-nova | 13:44 | |
| edleafe | alex_xu: which of these should remain after the plugins/v3 move? | 13:44 |
| *** baoli_ has joined #openstack-nova | 13:45 | |
| *** sabari has joined #openstack-nova | 13:46 | |
| openstackgerrit | Jordan Pittier proposed openstack/nova: libvirt:on snapshot delete, use qemu-img to blockRebase if VM is stopped https://review.openstack.org/192736 | 13:46 |
| *** baoli_ has quit IRC | 13:46 | |
| *** baoli_ has joined #openstack-nova | 13:46 | |
| *** dannywilson has joined #openstack-nova | 13:47 | |
| *** baoli has quit IRC | 13:48 | |
| openstackgerrit | whitekid proposed openstack/nova: use memcache.get_multi instead of memcache.get better performance https://review.openstack.org/211250 | 13:49 |
| *** edmondsw has quit IRC | 13:51 | |
| *** heyongli has quit IRC | 13:51 | |
| edleafe | alex_xu: I've added the version confusion to the etherpad. Comments? | 13:51 |
| kashyap | sdague: (Or anyone) -- how come people are not hitting this one w/ current DevStack Git master, when you boot a Nova instance -- https://bugs.launchpad.net/python-glanceclient/+bug/1479296 | 13:51 |
| openstack | Launchpad bug 1476770 in OpenStack-Gate "duplicate for #1479296 _translate_from_glance fails with "AttributeError: id" in grenade" [Undecided,Fix committed] - Assigned to Matt Riedemann (mriedem) | 13:51 |
| kashyap | Sigh, it's the bad interplay b/n system-installed versus pip-installed packages. . . | 13:52 |
| *** burt has joined #openstack-nova | 13:53 | |
| *** irenab has joined #openstack-nova | 13:54 | |
| *** openstackgerrit_ has quit IRC | 13:55 | |
| *** exploreshaifali has joined #openstack-nova | 13:56 | |
| *** openstackgerrit_ has joined #openstack-nova | 13:56 | |
| *** afazekas has quit IRC | 13:56 | |
| *** heyongli has joined #openstack-nova | 13:58 | |
| *** krtaylor has joined #openstack-nova | 13:59 | |
| *** eharney has joined #openstack-nova | 13:59 | |
| *** r-daneel has joined #openstack-nova | 14:00 | |
| *** fawadkhaliq has joined #openstack-nova | 14:01 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/nova: Ironic: Use ironicclient native retries for Conflict in ClientWrapper https://review.openstack.org/210369 | 14:01 |
| *** Piet has joined #openstack-nova | 14:01 | |
| *** Anticimex has quit IRC | 14:01 | |
| *** Anticimex has joined #openstack-nova | 14:02 | |
| *** garyk has joined #openstack-nova | 14:02 | |
| *** slagle has quit IRC | 14:02 | |
| *** diogogmt has quit IRC | 14:02 | |
| *** stefanb has quit IRC | 14:03 | |
| *** heyongli has quit IRC | 14:03 | |
| *** slagle has joined #openstack-nova | 14:04 | |
| johnthetubaguy | edleafe: not sure I get your version confusion comments | 14:04 |
| *** thangp has joined #openstack-nova | 14:04 | |
| *** stefanb has joined #openstack-nova | 14:05 | |
| johnthetubaguy | edleafe: v2.0 vs v2.1 is OK in that context I thought? | 14:05 |
| *** Piet has quit IRC | 14:05 | |
| *** annashen has joined #openstack-nova | 14:05 | |
| *** Piet has joined #openstack-nova | 14:05 | |
| edleafe | johnthetubaguy: in a nutshell, what should nova.api.openstack.compute.versions.Versions represent? | 14:05 |
| *** mjturek1 has joined #openstack-nova | 14:06 | |
| johnthetubaguy | edleafe: it doesn't change right? it represents the external facing API version 2.0 and 2.1, I thought? | 14:06 |
| edleafe | johnthetubaguy: currently it's a wsgi.Resource subclass. If I move the plugins/v3/versions, it would be a V#APIExtensionBase subclass | 14:07 |
| johnthetubaguy | edleafe: just happens the single v2.1 code base can support both of those | 14:07 |
| *** otter768 has joined #openstack-nova | 14:07 | |
| johnthetubaguy | edleafe: that would just be V21APIExtensionBase I guess? I get the feeling I am missing something | 14:07 |
| edleafe | johnthetubaguy: things throughout the codebase refer to the current versions.Versions. If I move the v3 stuff, it will now point to a different beast | 14:08 |
| edleafe | johnthetubaguy: so the current versions.Versions has to be renamed or something like that | 14:08 |
| atuvenie | johnthetubaguy: we found a potential DOS issue with ephemerals and bdm https://bugs.launchpad.net/nova/+bug/1483645 | 14:09 |
| openstack | Launchpad bug 1483645 in OpenStack Compute (nova) "Ephemeral disk size in volume can be bypassed when booting instance" [Undecided,New] | 14:09 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Add project id information in os-server-group API https://review.openstack.org/209917 | 14:09 |
| johnthetubaguy | edleafe: do you mean this thing: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/versions.py | 14:10 |
| *** annashen has quit IRC | 14:10 | |
| *** jcoufal has joined #openstack-nova | 14:10 | |
| atuvenie | johnthetubaguy: we can basically make block devices with more allocated space than the flavor allows | 14:10 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Add project id information in os-server-group API https://review.openstack.org/209917 | 14:10 |
| johnthetubaguy | atuvenie: yes, I believe thats totally intentional | 14:10 |
| *** jistr is now known as jistr|mtg | 14:10 | |
| *** bapalm_ has quit IRC | 14:11 | |
| johnthetubaguy | atuvenie: the flavor is talking about local disk, not cinder disks (most of the time) | 14:11 |
| edleafe | johnthetubaguy: that's now in n/a/o/c/legacy_v2/versions.py | 14:11 |
| *** bapalm has joined #openstack-nova | 14:11 | |
| *** jecarey has quit IRC | 14:11 | |
| *** otter768 has quit IRC | 14:12 | |
| johnthetubaguy | edleafe: hmm, maybe it shouldn't have moved | 14:12 |
| *** dave-mccowan has quit IRC | 14:12 | |
| *** shyama has quit IRC | 14:12 | |
| johnthetubaguy | edleafe: oh wait, now I see this one: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/plugins/v3/versions.py | 14:12 |
| rlrossit | dansmith: I think I had my one slight burst of knowledge for the year in my comment on https://review.openstack.org/#/c/211268/1 | 14:12 |
| edleafe | johnthetubaguy: so instead of removing it later on, like the rest of legacy_v2, just remove the reference to v2? | 14:13 |
| johnthetubaguy | edleafe: to be honest, I think we should just do the code shuffle, then it should be much clearer what we do next | 14:13 |
| *** shyama has joined #openstack-nova | 14:13 | |
| *** rotbeard has quit IRC | 14:14 | |
| *** jcoufal__ has quit IRC | 14:14 | |
| alex_xu | johnthetubaguy: edleafe , I didn't catch up your discussion. nova/api/openstack/versions.py for '/' , legacy_v2/versions for '/v2', plugins/v3/versions.py for '/v2.1' | 14:14 |
| dansmith | rlrossit: yeah, I get how it can be done that way, and since __getitem__ is O(1) for us, then it's fine, but not all sequency-like things would be | 14:14 |
| dansmith | rlrossit: but I hadn't had coffee so I didn't go look at the docs | 14:14 |
| *** bapalm has quit IRC | 14:15 | |
| *** bapalm has joined #openstack-nova | 14:15 | |
| *** bapalm has quit IRC | 14:16 | |
| *** raildo has joined #openstack-nova | 14:17 | |
| edleafe | alex_xu: but nova/api/openstack/compute/versions.py needs to be able to handle all the versions, right? | 14:17 |
| *** bapalm has joined #openstack-nova | 14:17 | |
| alex_xu | edleafe: right | 14:17 |
| *** openstackgerrit_ has quit IRC | 14:17 | |
| alex_xu | edleafe: I'm looksing your etherpad, looks good | 14:17 |
| edleafe | alex_xu: I think I'm going to fix the version.py changes before moving v3 | 14:18 |
| *** stefanb has quit IRC | 14:18 | |
| edleafe | alex_xu: that will make things cleaner | 14:18 |
| *** openstackgerrit_ has joined #openstack-nova | 14:18 | |
| *** xgsa has quit IRC | 14:18 | |
| alex_xu | edleafe: what is wrong? | 14:19 |
| edleafe | alex_xu: the logic was moved to legacy_v2 | 14:20 |
| edleafe | as johnthetubaguy pointed out, it probably should have remained in n/a/o/compute | 14:20 |
| *** dannywilson has quit IRC | 14:21 | |
| johnthetubaguy | edleafe: we can move it back though, if we need to, thats fine | 14:21 |
| alex_xu | edleafe: it is for '/v2' endpoint version info, it is v2 specific. it should v2 specific. | 14:21 |
| johnthetubaguy | edleafe: at somepoint we need to move all the configs defined in legacy_v2 into the v2.1 stuff, but thats all later | 14:22 |
| johnthetubaguy | this should be purely the code move, and things to make that work | 14:22 |
| johnthetubaguy | lets not get distracted, its my vote | 14:22 |
| edleafe | johnthetubaguy: alex_xu: so back to my base question: what sort of thing should n/a/o/c/versions.Versions be? | 14:23 |
| edleafe | right now it's a wsgi.Resource | 14:23 |
| *** gszasz has quit IRC | 14:24 | |
| *** edmondsw has joined #openstack-nova | 14:24 | |
| alex_xu | edleafe: currently it is '/' middleware, and '/v2' resource controller | 14:25 |
| alex_xu | edleafe: wsgi.Resource for '/v2' | 14:25 |
| johnthetubaguy | edleafe: oh I see, the v3 one lands on the stub for the old one | 14:26 |
| johnthetubaguy | hmm, not sure | 14:26 |
| edleafe | alex_xu: in the PS now, n/a/o/c/legacy_v2/versions.py has a VersionV2 class | 14:26 |
| johnthetubaguy | dansmith: can I ask you a quick question about https://review.openstack.org/#/c/201733/8? | 14:27 |
| alex_xu | edleafe: yea, I move the v2 bit into that | 14:27 |
| chhavi | ndipanov: https://bugs.launchpad.net/nova/+bug/1291835 for this bug, this can be fixed in reserve_block_device_name by changes the way the bdms are retrieved | 14:27 |
| openstack | Launchpad bug 1291835 in OpenStack Compute (nova) "Repeated volume attach can cause u'message': u'The supplied device path (/dev/vdc) is in use.'" [Medium,Confirmed] | 14:27 |
| johnthetubaguy | dansmith: I have been having impure thoughts about the service object, wondered what you thought about those? | 14:27 |
| alex_xu | edleafe: when moving v3 code, we probably need put v3 versions code into n/a/o/c/versions.py also | 14:27 |
| ndipanov | chhavi, I think that bug is actually fixed now | 14:28 |
| dansmith | johnthetubaguy: um. okay.. what are your impure thoughts? | 14:28 |
| dansmith | oh, I see | 14:28 |
| edleafe | dansmith: you don't want to know | 14:28 |
| *** nelsnelson has joined #openstack-nova | 14:29 | |
| johnthetubaguy | dansmith: yeah, it makes the object less general, but I wonder if the should restrict what folks can pass in for the version | 14:29 |
| *** dave-mccowan has joined #openstack-nova | 14:29 | |
| *** gszasz has joined #openstack-nova | 14:29 | |
| *** Murali_ has quit IRC | 14:29 | |
| alex_xu | edleafe: I think we should change this to new patch https://github.com/openstack/nova/blob/master/etc/nova/api-paste.ini#L121 although we are backward-compatible | 14:29 |
| ndipanov | chhavi, since nowadays we don't allow for the device to be specified at all i.e. it will be always chosen by the driver | 14:29 |
| alex_xu | edleafe: then in the future, we can remove the reference in n/a/o/c/versions.py | 14:30 |
| alex_xu | s/new patch/new path | 14:30 |
| ndipanov | chhavi, since b6733dbdf | 14:30 |
| dansmith | johnthetubaguy: I guess I feel like we should never allow them to pass it in, especially to create | 14:30 |
| edleafe | alex_xu: I thought that changing the api-paste was off-limits | 14:30 |
| *** dustins has quit IRC | 14:30 | |
| *** stefanb has joined #openstack-nova | 14:31 | |
| dansmith | johnthetubaguy: however, I just realized a bug here | 14:31 |
| johnthetubaguy | dansmith: yeah, saves you doing the "wrong" thing | 14:31 |
| *** openstackgerrit_ has quit IRC | 14:31 | |
| chhavi | ndipanov: ok yeah can we mark it fixed then | 14:31 |
| dansmith | johnthetubaguy: it'll create with the conductor version, which isn't what we want | 14:31 |
| *** moshele has quit IRC | 14:31 | |
| dansmith | johnthetubaguy: in the end, we have to take the service version in the object like I'm doing for save | 14:31 |
| dansmith | johnthetubaguy: I just need to not make it easy for people to override it | 14:31 |
| dansmith | johnthetubaguy: which means an __init__ and a readonly property I think | 14:31 |
| dansmith | johnthetubaguy: let me revise | 14:31 |
| ndipanov | chhavi, yes - thanks for noticing will do it now | 14:31 |
| dansmith | and I'll try to add some comments to help clear this up | 14:31 |
| johnthetubaguy | dansmith: yeah, I was just thinking about that... ah, cool | 14:32 |
| jaypipes | edleafe, johnthetubaguy, ndipanov, dansmith: your daily pug fix: https://twitter.com/jaypipes/status/631102588319080448 | 14:32 |
| *** xgsa has joined #openstack-nova | 14:32 | |
| alex_xu | edleafe: we keep the old api-paste works. but we need change the default value in api-paste, then people know we move to new path in the future? | 14:32 |
| dansmith | johnthetubaguy: I'd really like to get this merged soon as it has been conflict-bait :/ | 14:32 |
| dansmith | jaypipes: nice :) | 14:32 |
| johnthetubaguy | dansmith: +1 ping me when its revisted | 14:32 |
| dansmith | jaypipes: everywhere we go, when we see something pug-related, taylor considers buying and sending | 14:33 |
| dansmith | jaypipes: luckily, I'm the voice of reason | 14:33 |
| jaypipes | dansmith: well, sounds like *Taylor* is really the voice of reason. ;) | 14:33 |
| edleafe | jaypipes: you know that in 20 years your face will change so that you will look exactly like a pug | 14:33 |
| dansmith | hehe | 14:33 |
| jaypipes | edleafe: already has mostly happened. :P | 14:33 |
| *** agireud has joined #openstack-nova | 14:34 | |
| ndipanov | jaypipes, lol | 14:35 |
| *** rushiagr_away is now known as rushiagr | 14:35 | |
| johnthetubaguy | jaypipes: lol | 14:36 |
| *** tellesnobrega_ has joined #openstack-nova | 14:42 | |
| *** tellesnobrega_ has quit IRC | 14:42 | |
| johnthetubaguy | alex_xu: I guess this one is not quite there yet: https://review.openstack.org/#/c/200382/ | 14:42 |
| *** tellesnobrega_ has joined #openstack-nova | 14:42 | |
| *** jdandrea has joined #openstack-nova | 14:43 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add version counter to Service object https://review.openstack.org/201733 | 14:43 |
| *** rushiagr is now known as rushiagr_away | 14:44 | |
| *** tellesnobrega_ has quit IRC | 14:44 | |
| openstackgerrit | whitekid proposed openstack/nova: use memcache.get_multi instead of memcache.get better performance https://review.openstack.org/211250 | 14:45 |
| *** tellesnobrega_ has joined #openstack-nova | 14:45 | |
| *** kmARC has quit IRC | 14:46 | |
| *** dustins has joined #openstack-nova | 14:46 | |
| *** jecarey has joined #openstack-nova | 14:46 | |
| dansmith | johnthetubaguy: see what you think about that now ^ | 14:47 |
| johnthetubaguy | dansmith: just taking a look, that looks alot neater | 14:47 |
| *** tellesnobrega_ has quit IRC | 14:48 | |
| *** ajayaa has quit IRC | 14:48 | |
| johnthetubaguy | dansmith: just curious, is there an easy way to add a test to check we get the old version from the DB correctly when we read in the object? | 14:48 |
| edleafe | :qa | 14:49 |
| edleafe | doh! | 14:49 |
| dansmith | johnthetubaguy: ah, good point | 14:49 |
| dansmith | johnthetubaguy: dang | 14:49 |
| dansmith | yeah, let me fix that and write that test | 14:49 |
| johnthetubaguy | dansmith: also, I am guessing we might be always saving the version, as we will set the dirty flag in the __init__ ? or is there magic to protect us? | 14:50 |
| dansmith | johnthetubaguy: yeah, just setting it makes it dirty, so saving is good | 14:51 |
| *** tellesnobrega_ has joined #openstack-nova | 14:51 | |
| johnthetubaguy | dansmith: but that might be too good, like we always save that, even if we didn't change it, I guess? | 14:51 |
| johnthetubaguy | dansmith: although I guess the DB load fix might sort that out anyways | 14:51 |
| dansmith | johnthetubaguy: yeah, I intended to always save it any time we have other stuff to save | 14:51 |
| johnthetubaguy | dansmith: oh yeah, the old code did that... yeah, thats probably what we want | 14:52 |
| *** ajayaa has joined #openstack-nova | 14:52 | |
| dansmith | yeah, so we're good there with this I think | 14:52 |
| dansmith | just the loading needs fixing | 14:52 |
| johnthetubaguy | dansmith: I was thinking we might turn a no-op save into a real save, but thats never going to happen. | 14:52 |
| dansmith | johnthetubaguy: it could, and I could make this object not consider itself changed if only that is touched, but I think the way we use it, we're fine in this case | 14:53 |
| dansmith | we can always optimize that if we see a ton of saves of just that attribute, but I don't think that's ever going to happen the way this object is used | 14:54 |
| johnthetubaguy | dansmith: yeah I think thats good | 14:54 |
| edleafe | alex_xu: why did we move n/a/o/compute/versions.py to its parent directory? | 14:54 |
| edleafe | alex_xu: that's part of the confusion for me | 14:55 |
| johnthetubaguy | dansmith: hmm, it would be a slight bandwidth increase, setting both the datetime and the version, but meh, lets sort that later, if we need to | 14:55 |
| *** achanda has joined #openstack-nova | 14:56 | |
| dansmith | johnthetubaguy: only bandwidth between conductor and the db, but this will only happen if we do self.service.save() in our main loop or something when we're not already changing things | 14:56 |
| dansmith | and we don't do that | 14:56 |
| *** angdraug has joined #openstack-nova | 14:57 | |
| *** xgsa has quit IRC | 14:57 | |
| johnthetubaguy | dansmith: yeah, I think so | 14:57 |
| rlrossit | dansmith: can you see if pulling out a little more common stuff made this cleaner? https://review.openstack.org/#/c/203831/4 | 14:59 |
| dansmith | in a sec | 14:59 |
| *** jistr|mtg is now known as jistr | 14:59 | |
| rlrossit | yup no prob | 14:59 |
| dansmith | johnthetubaguy: that broke some xenapi test, so I'm chasing that | 14:59 |
| *** diogogmt has joined #openstack-nova | 15:00 | |
| *** cfriesen_ has joined #openstack-nova | 15:00 | |
| rlrossit | mriedem: should I break https://review.openstack.org/#/c/195713/ up into 2 patches, one with greenthread and the other for eventlet? greenthread was trivial but eventlet was not. | 15:01 |
| johnthetubaguy | dansmith: oops, let me know if thats my fault! | 15:02 |
| dansmith | johnthetubaguy: actually, no I think that it was just the thing you pointed out, which is loading from the db breaks on the readonly field :) | 15:02 |
| *** xgsa has joined #openstack-nova | 15:02 | |
| johnthetubaguy | dansmith: the system works! | 15:03 |
| dansmith | haha | 15:03 |
| mriedem | rlrossit: yeah | 15:03 |
| rlrossit | mriedem: eventually both will still be the same hacking check though right? or do we want 2 separate? | 15:04 |
| mriedem | rlrossit: i assume would be the same | 15:04 |
| mriedem | since they are very similar | 15:04 |
| rlrossit | indeed | 15:04 |
| mriedem | rlrossit: split would be good since the initial hacking check change was just because https://review.openstack.org/#/q/I3623e60c49e442e2431cf017540422aa59bc285a,n,z dealt only with greenthread.spawn | 15:05 |
| *** mtanino has joined #openstack-nova | 15:05 | |
| *** achanda has quit IRC | 15:05 | |
| *** zzzeek has joined #openstack-nova | 15:05 | |
| *** exploreshaifali has quit IRC | 15:06 | |
| rlrossit | mriedem: so that would almost be 3 then right? because it's 1. fix greenthread 2. add hacking for greenthread 3. fix eventlet 4. add hacking for eventlet | 15:06 |
| rlrossit | 3 & 4 can be made into one though still | 15:06 |
| mriedem | 1 is done, you have 2 (need to split), then yeah 3 and 4 could be a single change | 15:08 |
| *** tpeoples has quit IRC | 15:08 | |
| alex_xu | edleafe: I thought '/' isn't belong to v2 or v2.1. So I move it to parent directory. | 15:09 |
| alex_xu | johnthetubaguy: yea, that is the last one, I will update when other patch merged | 15:09 |
| edleafe | alex_xu: I'm going to try moving it back, since it is compute-specific | 15:10 |
| edleafe | alex_xu: it might also simplify the naming confusions | 15:10 |
| johnthetubaguy | alex_xu: yeah, OK, I guess I wonder about lining up the last three in a row, so its clear its the end, but thats fine | 15:11 |
| alex_xu | edleafe: ok, I'm ok with that | 15:11 |
| *** tjones has joined #openstack-nova | 15:11 | |
| johnthetubaguy | edleafe: what were you going to call the v3 one it drops into that directory? versions_extension.py or something? | 15:11 |
| alex_xu | johnthetubaguy: yea, I just didn't want to clear existed +2 | 15:12 |
| *** dustins has quit IRC | 15:12 | |
| *** yamahata has joined #openstack-nova | 15:13 | |
| * alex_xu is out of power... | 15:13 | |
| *** shyama has quit IRC | 15:14 | |
| edleafe | johnthetubaguy: not sure yet. Could be a separate file, or could be merged into the existing versions.py | 15:14 |
| *** david-lyle has joined #openstack-nova | 15:15 | |
| *** tpeoples has joined #openstack-nova | 15:17 | |
| johnthetubaguy | claudiub: I noticed this really important looking but was not triaged, how did this get past our CI system? https://bugs.launchpad.net/nova/+bug/1463044 | 15:19 |
| openstack | Launchpad bug 1463044 in OpenStack Compute (nova) "Hyper-V: the driver fails to initialize on Windows Server 2008 R2" [High,In progress] - Assigned to Lucian Petrut (petrutlucian94) | 15:19 |
| edleafe | alex_xu: go recharge yourself! :) | 15:19 |
| danpb | johnthetubaguy: wasn't there a message that they stopped testing 2k8 | 15:22 |
| *** gszasz has quit IRC | 15:22 | |
| claudiub | johnthetubaguy: hi. because Hyper-V CI runs on Windows Hyper-V 2012 R2 which doesn't have this issue. | 15:22 |
| *** dannywilson has joined #openstack-nova | 15:23 | |
| sdague | claudiub: so is that a "won't fix"? | 15:24 |
| danpb | johnthetubaguy: "Subject: [openstack-dev] Hyper-V 2008 R2 support" | 15:24 |
| johnthetubaguy | claudiub: yeah, I figured that was why, is that a good idea? | 15:24 |
| danpb | IMHO we *must* fix it | 15:24 |
| johnthetubaguy | danpb: yeah, I missed that | 15:24 |
| danpb | we have previously support 2k8 and haven't told users we're dropping it | 15:24 |
| danpb | so at most we can deprecate it in Liberty and remove it in Mitaka | 15:24 |
| danpb | we can't drop it in Liberty as that'll screw over any users with n prior warning | 15:24 |
| johnthetubaguy | danpb: well, we don't test all versions of libvirt either in a similar way, but agreed | 15:24 |
| dansmith | well, | 15:25 |
| dansmith | we previously had a discussion about vmware | 15:25 |
| johnthetubaguy | danpb: given we are fixing this bug, I guess we are not dropping support, its just not tested | 15:25 |
| danpb | sure but we'd fix any bugs | 15:25 |
| dansmith | and the consensus seemed to be that dropping support for unsupported commercial hypervisors is not a needs-a-deprecation-cycle sort of thing | 15:25 |
| danpb | and when we offiicaly increment the min required version, we'd hjave a period of deprecation warnings of 1 cycle | 15:25 |
| sdague | so, are there fixes posted by hyper-v folks? | 15:25 |
| johnthetubaguy | dansmith: so yeah, thats the direct comparison vmware ESX versions we test | 15:25 |
| dansmith | johnthetubaguy: right | 15:25 |
| dansmith | we talked about it here, in a review, and on the ML, IIRC | 15:26 |
| johnthetubaguy | sdague: so this made me ask the question: https://review.openstack.org/#/c/189308 | 15:26 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add version counter to Service object https://review.openstack.org/201733 | 15:26 |
| dansmith | johnthetubaguy: I did your save no-op thing too ^^ | 15:26 |
| *** dustins has joined #openstack-nova | 15:26 | |
| johnthetubaguy | dansmith: oh, cool, looking | 15:26 |
| danpb | dansmith: IME vendors are much more inclined to say things are unsupported than users are to accept it | 15:27 |
| johnthetubaguy | danpb: I need to do the Feature classification thing, and get that going, it really helps answer these questions | 15:27 |
| dansmith | danpb: well, running an unsupported hypervisor is like a terrible idea | 15:27 |
| danpb | pretty much every company i know will continue running software even if the vendor technically claims it is unsupported | 15:27 |
| dansmith | danpb: and when we asked how many people were running the prehistoric vmware on the ops ML, it was like crickets | 15:27 |
| danpb | and the vendors will often secretly provide them support for it while still claiming it is snupported | 15:27 |
| *** bapalm has quit IRC | 15:27 | |
| *** bapalm has joined #openstack-nova | 15:28 | |
| *** josecastroleon has quit IRC | 15:28 | |
| *** kmARC has joined #openstack-nova | 15:29 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: Fix deleting instance with incorrectly attached volume https://review.openstack.org/204231 | 15:29 |
| sdague | ok, so there is in theory a fix posted. If we merge it. Mark that OS deprecated. And remove support in Mitaka, that seem like a plan? | 15:29 |
| danpb | yeah | 15:29 |
| johnthetubaguy | sdague: yep | 15:30 |
| *** tsekiyama has joined #openstack-nova | 15:30 | |
| johnthetubaguy | claudiub: does the above statement make sense? | 15:30 |
| sdague | I'm not sure exactly how that patch fixes things though, it seems like it sends up a fatal exception still. I guess that's caught somewhere further up the stack? | 15:30 |
| johnthetubaguy | claudiub: I can't remember what the plan was | 15:30 |
| johnthetubaguy | sdague: it moves the exception to usage instead of nova-compute startup, I think | 15:31 |
| sdague | oh, gotcha, behind the property | 15:31 |
| *** moshele has joined #openstack-nova | 15:31 | |
| *** hemnafk is now known as hemna_ | 15:32 | |
| *** krtaylor has quit IRC | 15:32 | |
| sdague | I would like to see the deprecation patch posted by claudiub as well | 15:32 |
| johnthetubaguy | sdague: yeah, we should probably just merge the fix though I guess? | 15:34 |
| johnthetubaguy | sdague: well, once the CI votes on it | 15:34 |
| sdague | the fix is simple, and it's a bug fix | 15:34 |
| johnthetubaguy | yeah, agreed | 15:34 |
| dansmith | johnthetubaguy: I think maybe an item for the at-release-time procedure docs would be checking with driver owners to see if there are any products that have dropped out of support in the last cycle so we can get ahead of these deprecation messages | 15:34 |
| sdague | so anyway I stuck my +2 on that | 15:34 |
| *** paboldin has joined #openstack-nova | 15:35 | |
| johnthetubaguy | dansmith: yeah, thats a good call | 15:35 |
| sdague | dansmith: yeh, though it would be good to catch it by milestone 3 | 15:35 |
| sdague | so it's not a last minute scramble | 15:35 |
| dansmith | well, not sure we have docs for milestone 3 things, | 15:35 |
| dansmith | but we do for release time | 15:35 |
| dansmith | but regardless | 15:35 |
| dansmith | doing it is the important part, whenever it happens | 15:36 |
| sdague | yep | 15:36 |
| *** moshele has quit IRC | 15:37 | |
| *** moshele has joined #openstack-nova | 15:37 | |
| johnthetubaguy | dansmith: I put a place holder here, in the hope it reminds us: https://wiki.openstack.org/wiki/Nova/ReleaseChecklist#Pre-release_Checklist | 15:37 |
| dansmith | cool | 15:37 |
| *** hemna_ has quit IRC | 15:40 | |
| *** hemna_ has joined #openstack-nova | 15:41 | |
| *** krtaylor has joined #openstack-nova | 15:45 | |
| *** moshele has quit IRC | 15:47 | |
| *** changbl has quit IRC | 15:47 | |
| johnthetubaguy | dansmith: that service version looks good to me now, there is a minor nit, but thats not really worth a respin | 15:48 |
| dansmith | johnthetubaguy: might as well | 15:48 |
| dansmith | not sure how I did that | 15:48 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add version counter to Service object https://review.openstack.org/201733 | 15:49 |
| dansmith | bewn | 15:49 |
| dansmith | er, bewm | 15:49 |
| *** kmARC has quit IRC | 15:49 | |
| johnthetubaguy | dansmith: :) | 15:49 |
| *** changbl has joined #openstack-nova | 15:50 | |
| dansmith | ls | 15:51 |
| dansmith | oops | 15:51 |
| *** bnemec has quit IRC | 15:51 | |
| * dansmith needs more coffee | 15:51 | |
| claudiub | sdague: danpb: johnthetubaguy: Sorry, I was AFK for a while. The fix should still get in Liberty. Yes, 2008 R2 is deprecated in Liberty, but it will only be removed in M. I will post the commit with the deprecation message today. | 15:52 |
| sdague | claudiub: great, thank you | 15:52 |
| johnthetubaguy | claudiub: sounds good :) | 15:52 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add hacking check for greenthread.spawn() https://review.openstack.org/195713 | 15:53 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add hacking check for eventlet.spawn() https://review.openstack.org/211643 | 15:53 |
| *** alexschm has quit IRC | 15:54 | |
| *** leeantho has joined #openstack-nova | 15:55 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add hacking check for eventlet.spawn() https://review.openstack.org/211643 | 15:56 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add hacking check for greenthread.spawn() https://review.openstack.org/195713 | 15:56 |
| rlrossit | mriedem: ^ I think the greenthread one at least should be pretty easy and ready to go | 15:56 |
| mriedem | k, sec | 15:56 |
| mriedem | i'm busy making chartware | 15:57 |
| *** scheuran has quit IRC | 15:58 | |
| johnthetubaguy | garyk: can I ask questions about this OVA fix: https://review.openstack.org/#/c/158319 ? | 15:59 |
| *** apuimedo has quit IRC | 15:59 | |
| johnthetubaguy | garyk: oh sorry, you are away, lets catch up in the morning | 15:59 |
| *** ijuwang has joined #openstack-nova | 15:59 | |
| claudiub | johnthetubaguy: question: this config option will have to be deprecated as well: https://github.com/openstack/nova/blob/master/nova/virt/hyperv/utilsfactory.py#L36 , is adding deprecated_for_removal=True enough for that? | 16:00 |
| *** jistr has quit IRC | 16:01 | |
| dansmith | yeah, I think so | 16:01 |
| dansmith | it warns when you do that I believe | 16:01 |
| dansmith | claudiub: ^ | 16:01 |
| *** bnemec has joined #openstack-nova | 16:02 | |
| *** zhenguo has quit IRC | 16:02 | |
| *** jcoufal has quit IRC | 16:04 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move existing V2 to legacy_v2 https://review.openstack.org/211356 | 16:04 |
| *** jlanoux has quit IRC | 16:04 | |
| *** chhavi has quit IRC | 16:06 | |
| *** annashen has joined #openstack-nova | 16:06 | |
| claudiub | dansmith: cool. thanks! | 16:06 |
| sdague | edleafe: was there a reason you changed alex's last revision? | 16:07 |
| openstackgerrit | Tracy Jones proposed openstack/nova: Cleanup of compute api reboot method https://review.openstack.org/127427 | 16:07 |
| *** marzif_ has quit IRC | 16:08 | |
| *** otter768 has joined #openstack-nova | 16:08 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move existing V2 to legacy_v2 - step 2 https://review.openstack.org/211390 | 16:08 |
| *** sahid has quit IRC | 16:08 | |
| *** openstackgerrit_ has joined #openstack-nova | 16:08 | |
| edleafe | sdague: yeah - we agreed that versions.py should be in n/a/o/compute/ | 16:09 |
| sdague | edleafe: ok, but your patch changes a ton of other stuff | 16:09 |
| sdague | oh, never mind bad diff comparison | 16:10 |
| *** annashen has quit IRC | 16:10 | |
| sdague | because of the rebase | 16:10 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add hacking check for eventlet.spawn() https://review.openstack.org/211643 | 16:11 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add hacking check for greenthread.spawn() https://review.openstack.org/195713 | 16:11 |
| sdague | ok, seems fine | 16:12 |
| openstackgerrit | John Garbutt proposed openstack/nova: VMware: Do not untar OVA on the file system https://review.openstack.org/158319 | 16:12 |
| *** beekneemech has joined #openstack-nova | 16:12 | |
| *** Jeffrey4l has quit IRC | 16:13 | |
| *** tellesnobrega_ has quit IRC | 16:13 | |
| *** otter768 has quit IRC | 16:13 | |
| *** bnemec has quit IRC | 16:13 | |
| *** tojuvone has quit IRC | 16:16 | |
| *** ig0r_ has joined #openstack-nova | 16:19 | |
| openstackgerrit | Marian Horban proposed openstack/nova: Added processing /compute URL https://review.openstack.org/195303 | 16:20 |
| *** annashen has joined #openstack-nova | 16:21 | |
| *** alex_klimov has quit IRC | 16:23 | |
| *** beekneemech is now known as bnemec | 16:26 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix the 2.10 formatting in the REST API version history doc https://review.openstack.org/211656 | 16:26 |
| *** thumpba has quit IRC | 16:27 | |
| mriedem | sdague: muwahaha https://review.openstack.org/#/q/status:merged+project:openstack/nova+branch:master+topic:bp/consolidate-libvirt-fs-volume-drivers,n,z | 16:27 |
| *** lucasagomes has quit IRC | 16:27 | |
| *** tjones has quit IRC | 16:28 | |
| *** tjones has joined #openstack-nova | 16:28 | |
| *** thumpba has joined #openstack-nova | 16:28 | |
| *** tjones has quit IRC | 16:28 | |
| *** raildo__ has joined #openstack-nova | 16:29 | |
| *** tjones has joined #openstack-nova | 16:29 | |
| *** mdrabe_ has joined #openstack-nova | 16:29 | |
| *** mdrabe has quit IRC | 16:29 | |
| *** garyk has quit IRC | 16:30 | |
| *** yamahata has quit IRC | 16:31 | |
| *** krtaylor has quit IRC | 16:31 | |
| *** unicell has quit IRC | 16:31 | |
| markus_z | @cores: please review https://review.openstack.org/#/c/209187/ | 16:33 |
| *** raildo-afk has joined #openstack-nova | 16:35 | |
| *** raildo__ has quit IRC | 16:35 | |
| *** bapalm_ has joined #openstack-nova | 16:35 | |
| *** ildikov has quit IRC | 16:37 | |
| *** nic has joined #openstack-nova | 16:37 | |
| *** lyanchih has quit IRC | 16:37 | |
| *** bapalm has quit IRC | 16:39 | |
| *** rotbeard has joined #openstack-nova | 16:40 | |
| *** htruta has quit IRC | 16:41 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: Fix deleting instance with incorrectly attached volume https://review.openstack.org/204231 | 16:43 |
| *** htruta has joined #openstack-nova | 16:43 | |
| *** krtaylor has joined #openstack-nova | 16:43 | |
| *** htruta has quit IRC | 16:44 | |
| mnestratov | mriedem: hi. please look ^^ | 16:44 |
| edleafe | :q | 16:45 |
| edleafe | geez | 16:45 |
| edleafe | that's like the third time today :( | 16:45 |
| *** dane-fichter has quit IRC | 16:45 | |
| mriedem | mnestratov: thanks, in a bit | 16:46 |
| *** rdopiera has quit IRC | 16:46 | |
| *** markus_z has quit IRC | 16:46 | |
| mnestratov | mriedem: is there a way to make a review be dependant on a review from another repo? | 16:47 |
| *** ig0r_ has quit IRC | 16:48 | |
| dansmith | yeah | 16:48 |
| dansmith | Depends-On: $changeid | 16:49 |
| *** meghal has joined #openstack-nova | 16:49 | |
| jordanP | mnestratov, http://docs.openstack.org/infra/manual/developers.html#cross-repository-dependencies | 16:49 |
| mnestratov | dansmith: mriedem: thanks | 16:49 |
| mnestratov | jordanP: thank you | 16:50 |
| *** yamahata has joined #openstack-nova | 16:50 | |
| *** romainh has left #openstack-nova | 16:51 | |
| *** ssurana has joined #openstack-nova | 16:51 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: Fix deleting instance with incorrectly attached volume https://review.openstack.org/204231 | 16:52 |
| *** meghal has quit IRC | 16:53 | |
| *** raildo has quit IRC | 16:53 | |
| *** earlephilhower has joined #openstack-nova | 16:53 | |
| *** jordanP has quit IRC | 16:53 | |
| *** bapalm_ has quit IRC | 16:54 | |
| *** bapalm has joined #openstack-nova | 16:54 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move existing V2 to legacy_v2 - step 2 https://review.openstack.org/211390 | 16:54 |
| *** raildo has joined #openstack-nova | 16:55 | |
| *** htruta has joined #openstack-nova | 16:55 | |
| *** matrohon has quit IRC | 16:55 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: deprecates support for Windows / Hyper-V Server 2008 R2 https://review.openstack.org/211670 | 16:55 |
| *** bapalm_ has joined #openstack-nova | 16:55 | |
| *** vilobhmm has joined #openstack-nova | 16:58 | |
| *** browne has quit IRC | 16:59 | |
| *** bapalm has quit IRC | 16:59 | |
| *** tjones has left #openstack-nova | 17:00 | |
| *** bapalm_ has quit IRC | 17:00 | |
| *** rotbeard has quit IRC | 17:00 | |
| *** skylerberg has joined #openstack-nova | 17:02 | |
| *** unicell has joined #openstack-nova | 17:04 | |
| *** vilobhmm has quit IRC | 17:04 | |
| *** ajayaa has quit IRC | 17:04 | |
| openstackgerrit | Marian Horban proposed openstack/nova: Parsing of URL corrected https://review.openstack.org/195303 | 17:05 |
| *** vilobhmm has joined #openstack-nova | 17:05 | |
| *** meghal has joined #openstack-nova | 17:08 | |
| dannywilson | danpb: ping, question about checking qemu version when trying to add discard functionality | 17:09 |
| *** vilobhmm has quit IRC | 17:10 | |
| *** patrickeast has joined #openstack-nova | 17:11 | |
| *** vilobhmm has joined #openstack-nova | 17:11 | |
| *** vilobhmm has quit IRC | 17:11 | |
| mriedem | mnestratov: so i think there was a misunderstanding about how your nova change is going to work and that cinderclient change which introduces that new exception type | 17:13 |
| mriedem | mnestratov: in nova we have to parse the 400 error message from cinder today, since there isn't a released version of cinderclient that has that new exception type | 17:13 |
| mnestratov | mriedem: maybe | 17:13 |
| mriedem | mnestratov: so i was saying, do the error parsing logic in nova today as a short term fix but have a TODO in there that we need to change it to handle a specific exception from cinderclient | 17:14 |
| mnestratov | mriedem: ok | 17:14 |
| *** busterswt has joined #openstack-nova | 17:14 | |
| mriedem | then once the exception type is in cinderclient and released, and we bump minimum required global-requirements for cinderclient, we can then change the code to use that | 17:14 |
| mnestratov | i thought we should wait for a new ciinderclient | 17:14 |
| mnestratov | mriedem: I revert my change to message parsing then | 17:17 |
| mriedem | mnestratov: we also couldn't backport the change if we wanted for the cinderclient version | 17:18 |
| mriedem | mnestratov: i just added some comments in there to clarify | 17:18 |
| dansmith | johnthetubaguy: my service version patch failed | 17:19 |
| dansmith | johnthetubaguy: because compute_rpc has changed and it detected it | 17:19 |
| mnestratov | mriedem: again I thought wait for cinderclient in kilo either | 17:19 |
| mnestratov | mriedem: again I thought we should wait for cinderclient in kilo either | 17:19 |
| mriedem | mnestratov: well, that would required backporting that change to stable/kilo cinderclient and doing a release there, which i doubt the cinder team would want to do | 17:20 |
| mriedem | mnestratov: and that would require raising the minimum required version of cinderclient in stable/kilo which is a bad idea | 17:20 |
| *** VW_ has quit IRC | 17:21 | |
| *** raildo has quit IRC | 17:21 | |
| *** VW_ has joined #openstack-nova | 17:22 | |
| mnestratov | mriedem: ok, I'll redo the review, no problem | 17:22 |
| *** thumpba has quit IRC | 17:22 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add version counter to Service object https://review.openstack.org/201733 | 17:22 |
| dansmith | sdague: you were previously happy with that ^ | 17:23 |
| dansmith | sdague: john already +2d before a rebase just now | 17:23 |
| *** VW_ has quit IRC | 17:26 | |
| *** raildo-afk is now known as raildo | 17:27 | |
| *** atuvenie has quit IRC | 17:27 | |
| *** raildo is now known as raildo-afk | 17:28 | |
| mriedem | mnestratov: did you see the comment in patch set 6 from ftersin about https://review.openstack.org/#/c/210534/ ? | 17:28 |
| mriedem | i think that would also fix your bug | 17:28 |
| mriedem | if the volume status is not in-use, we shouldn't try to detach it | 17:28 |
| *** raildo has joined #openstack-nova | 17:29 | |
| *** raildo has quit IRC | 17:29 | |
| *** jwcroppe has joined #openstack-nova | 17:30 | |
| *** raildo-afk is now known as raildo | 17:32 | |
| *** boris-42 has joined #openstack-nova | 17:32 | |
| *** agireud_ has joined #openstack-nova | 17:33 | |
| *** agireud has quit IRC | 17:34 | |
| *** agireud_ is now known as agireud | 17:34 | |
| mnestratov | mriedem: yes I saw, but I think it is racy and won't work for every case | 17:36 |
| mriedem | mnestratov: i was just talking about that with xyang1 | 17:37 |
| mriedem | mnestratov: the only race pointed out is if the cinder db is out of sync with the volume backend, | 17:37 |
| mriedem | mnestratov: but i don't think your fix probably handles that case either | 17:37 |
| mriedem | nova gets the 400 if we get to this point in cinder http://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/manager.py#n885 | 17:38 |
| *** browne has joined #openstack-nova | 17:39 | |
| *** praneshp has joined #openstack-nova | 17:39 | |
| *** VW_ has joined #openstack-nova | 17:40 | |
| *** danpb has quit IRC | 17:40 | |
| mriedem | ndipanov: you stepped into something :) | 17:41 |
| *** ildikov has joined #openstack-nova | 17:46 | |
| *** vilobhmm has joined #openstack-nova | 17:48 | |
| mnestratov | mriedem: it would be helpful if we had logs or bug description in https://review.openstack.org/#/c/210534/ | 17:49 |
| mriedem | mnestratov: yeah | 17:49 |
| *** kmARC has joined #openstack-nova | 17:50 | |
| *** tonytan4ever has joined #openstack-nova | 17:51 | |
| *** harlowja has quit IRC | 17:51 | |
| *** armax has quit IRC | 17:52 | |
| mriedem | dansmith: would a version bump be necessary here? https://review.openstack.org/#/c/179390/18/nova/objects/external_event.py | 17:52 |
| mriedem | i guess so since an event enum fields values are changing | 17:53 |
| *** achanda has joined #openstack-nova | 17:54 | |
| *** harlowja has joined #openstack-nova | 17:54 | |
| *** armax has joined #openstack-nova | 17:55 | |
| *** armax has quit IRC | 17:55 | |
| *** derekh has joined #openstack-nova | 17:56 | |
| *** bapalm has joined #openstack-nova | 17:56 | |
| mriedem | rlrossit: question in https://review.openstack.org/#/c/195713/ | 17:56 |
| *** patrickeast has quit IRC | 17:57 | |
| *** baoli__ has joined #openstack-nova | 18:00 | |
| *** bnemec has quit IRC | 18:01 | |
| *** bapalm has quit IRC | 18:01 | |
| *** baoli_ has quit IRC | 18:03 | |
| *** rushil has joined #openstack-nova | 18:03 | |
| *** meghal has quit IRC | 18:03 | |
| *** meghal has joined #openstack-nova | 18:04 | |
| *** baoli_ has joined #openstack-nova | 18:04 | |
| *** baoli_ has quit IRC | 18:04 | |
| *** tjones has joined #openstack-nova | 18:05 | |
| *** bapalm has joined #openstack-nova | 18:05 | |
| kfox1111 | is there a way to get nova to set libvirt/kvm to use uefi instead of the regular bios? | 18:05 |
| *** derekh has quit IRC | 18:06 | |
| rlrossit | responded mriedem | 18:06 |
| *** openstackgerrit_ has quit IRC | 18:06 | |
| sdague | dansmith: looking | 18:06 |
| *** bnemec has joined #openstack-nova | 18:06 | |
| *** baoli has joined #openstack-nova | 18:07 | |
| mriedem | kfox1111: there was a patch up to do that, sec | 18:07 |
| kfox1111 | thx. | 18:07 |
| mriedem | kfox1111: https://review.openstack.org/#/c/185129/ | 18:08 |
| *** baoli__ has quit IRC | 18:08 | |
| kfox1111 | Awesome. thanks. :) | 18:08 |
| *** otter768 has joined #openstack-nova | 18:09 | |
| *** dikonoo has quit IRC | 18:10 | |
| *** haomaiwa_ has joined #openstack-nova | 18:10 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Fix serial port issue on Windows Threshold https://review.openstack.org/192617 | 18:11 |
| *** ijw has joined #openstack-nova | 18:11 | |
| mnestratov | mriedem: regarding race condition: two simultaneous API calls: detach volume and delete instance - isn't this possible? | 18:11 |
| mriedem | mnestratov: yeah, that's possible | 18:12 |
| *** 92AABUE7H has quit IRC | 18:12 | |
| mnestratov | then we will cirtainly hit this race | 18:13 |
| mriedem | ndipanov appears to agree from his comment in the other review | 18:13 |
| mriedem | mnestratov: feel free to update your change as we discussed | 18:13 |
| mnestratov | cirtainly->certainly | 18:13 |
| *** ijw_ has joined #openstack-nova | 18:14 | |
| *** otter768 has quit IRC | 18:14 | |
| mnestratov | mriedem: ok | 18:14 |
| *** meghal has quit IRC | 18:15 | |
| *** ijw has quit IRC | 18:16 | |
| *** ig0r_ has joined #openstack-nova | 18:17 | |
| mriedem | rlrossit: +2 | 18:18 |
| *** ig0r_ has quit IRC | 18:18 | |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: Servicegroup foundational refactoring for Control Plane https://review.openstack.org/202714 | 18:18 |
| *** ijw_ has quit IRC | 18:18 | |
| *** meghal has joined #openstack-nova | 18:19 | |
| *** baoli has quit IRC | 18:24 | |
| *** josecastroleon has joined #openstack-nova | 18:24 | |
| *** baoli has joined #openstack-nova | 18:25 | |
| *** ig0r_ has joined #openstack-nova | 18:27 | |
| *** READ10 has joined #openstack-nova | 18:28 | |
| *** armax has joined #openstack-nova | 18:31 | |
| *** exploreshaifali has joined #openstack-nova | 18:32 | |
| *** salv-orl_ has quit IRC | 18:32 | |
| *** haomaiwang has joined #openstack-nova | 18:33 | |
| *** haomaiwa_ has quit IRC | 18:35 | |
| mriedem | mtreinish: i tend to agree with also fixing this for the ec2 api here https://review.openstack.org/#/c/210684/ | 18:48 |
| mtreinish | mriedem: wait did I agree to that? | 18:49 |
| mriedem | baoli: i think you need some error handling in here https://review.openstack.org/#/c/179390/ | 18:49 |
| mriedem | melwitt: ^ | 18:49 |
| mriedem | then i'd be +2 | 18:49 |
| mriedem | mtreinish: you pointed it out | 18:49 |
| mtreinish | mriedem: heh I thought I prefaced it by saying I don't care about ec2 :) | 18:49 |
| mtreinish | mriedem: so what should I do, add unit tests, add the todo, etc from your first comment? | 18:50 |
| mtreinish | or pivot to a straight 400 passthrough | 18:50 |
| mriedem | mtreinish: i think the original | 18:52 |
| mtreinish | ok, then I'll work on doing that | 18:52 |
| melwitt | mriedem: commented you're right about update_cells=False only for the periodic case | 18:52 |
| *** meghal has quit IRC | 18:53 | |
| *** VW_ has quit IRC | 18:53 | |
| *** meghal has joined #openstack-nova | 18:53 | |
| *** VW_ has joined #openstack-nova | 18:54 | |
| *** meghal has quit IRC | 18:54 | |
| *** davideagnello has quit IRC | 18:54 | |
| *** josecastroleon has quit IRC | 18:55 | |
| *** davideagnello has joined #openstack-nova | 18:55 | |
| *** tjones has quit IRC | 18:55 | |
| *** davideagnello has quit IRC | 18:56 | |
| *** VW_ has quit IRC | 18:58 | |
| *** ig0r_ has quit IRC | 18:59 | |
| *** kmARC has quit IRC | 19:00 | |
| *** Nikolay_St has quit IRC | 19:00 | |
| *** ijw has joined #openstack-nova | 19:01 | |
| *** ig0r_ has joined #openstack-nova | 19:03 | |
| *** bapalm has quit IRC | 19:04 | |
| *** bapalm has joined #openstack-nova | 19:05 | |
| baoli | mriedem: looking ... | 19:06 |
| *** meghal has joined #openstack-nova | 19:07 | |
| *** bapalm_ has joined #openstack-nova | 19:08 | |
| *** davideagnello has joined #openstack-nova | 19:08 | |
| *** bapalm has quit IRC | 19:09 | |
| baoli | mriedem: do we catch, log, ignore or catch, log, reraise? | 19:10 |
| sdague | dansmith: https://review.openstack.org/#/c/201733 has some real test fails to look into | 19:11 |
| mriedem | baoli: i think catch/log/ignore | 19:12 |
| openstackgerrit | Taylor Peoples proposed openstack/nova: Use string substitution before raising exception https://review.openstack.org/211716 | 19:13 |
| mriedem | baoli: log a warning like in the detach_interface method | 19:13 |
| baoli | mriedem: I agree. | 19:13 |
| mriedem | baoli: since neutron detached the port i don't think there is much we could do at that point, | 19:13 |
| mriedem | i just don't want to break the loop for the other events to process | 19:13 |
| baoli | mriedem: sure. Let me load a new patch with that shortly | 19:13 |
| *** exploreshaifali has quit IRC | 19:13 | |
| *** mestery_ has joined #openstack-nova | 19:14 | |
| *** annegentle has quit IRC | 19:14 | |
| *** annegentle has joined #openstack-nova | 19:15 | |
| *** rushil has quit IRC | 19:15 | |
| *** mestery has quit IRC | 19:17 | |
| *** rushil has joined #openstack-nova | 19:17 | |
| *** mili has joined #openstack-nova | 19:18 | |
| dansmith | sdague: yeah, py3 at lrast | 19:18 |
| dansmith | sdague: t'was passing until I wrote that test :/ | 19:18 |
| *** VW_ has joined #openstack-nova | 19:18 | |
| *** eglynn has quit IRC | 19:18 | |
| *** eglynn has joined #openstack-nova | 19:18 | |
| *** mestery_ is now known as mestery | 19:19 | |
| dansmith | sdague: is this safe? https://review.openstack.org/#/c/210369/4 | 19:21 |
| dansmith | sdague: it's very straightforward, but I'm not sure about the version | 19:21 |
| dansmith | sdague: we pin ironicclient to 0.6 in our test-requirements, so I assume that means we're good, right? | 19:21 |
| melwitt | mriedem: did you see my comment on your logging patch? https://review.openstack.org/#/c/188427/ I don't think _version_to_string() works right on lists | 19:22 |
| *** patrickeast has joined #openstack-nova | 19:23 | |
| *** bapalm_ has quit IRC | 19:24 | |
| baoli | mriedem: do you want a UT for it? | 19:24 |
| *** bapalm has joined #openstack-nova | 19:24 | |
| *** alejandrito has joined #openstack-nova | 19:24 | |
| *** eglynn has quit IRC | 19:25 | |
| *** bapalm has quit IRC | 19:25 | |
| *** bapalm has joined #openstack-nova | 19:26 | |
| *** meghal has quit IRC | 19:26 | |
| *** meghal has joined #openstack-nova | 19:26 | |
| *** mili has quit IRC | 19:27 | |
| *** zsais has joined #openstack-nova | 19:28 | |
| mriedem | baoli: we should have a unit test for the error handling yes | 19:28 |
| baoli | mriedem: sounds good | 19:28 |
| *** rushil has quit IRC | 19:29 | |
| mriedem | melwitt: looking, thought i copied that from somewhere else | 19:29 |
| openstackgerrit | Taylor Peoples proposed openstack/nova: Use string substitution before raising exception https://review.openstack.org/211716 | 19:29 |
| *** rods has joined #openstack-nova | 19:30 | |
| *** bapalm has quit IRC | 19:31 | |
| *** bapalm has joined #openstack-nova | 19:31 | |
| dansmith | sdague: did you see my question about that ironicclient thing above? | 19:32 |
| *** annashen has quit IRC | 19:32 | |
| sdague | just looking now | 19:32 |
| sdague | if we have a minimum of 0.6, then yes | 19:32 |
| dansmith | well, what I'm asking is, | 19:32 |
| dansmith | is the test-requirements entry the one we follow? | 19:33 |
| sdague | hmmm | 19:33 |
| dansmith | jroll: you wanna ack that from the ironic perspective? | 19:33 |
| sdague | so, right, there is nothing that currently forces a user up to the right level during install | 19:33 |
| *** bapalm has quit IRC | 19:33 | |
| *** meghal has quit IRC | 19:33 | |
| *** READ10 has quit IRC | 19:33 | |
| *** meghal has joined #openstack-nova | 19:34 | |
| *** bapalm has joined #openstack-nova | 19:34 | |
| sdague | it's a bummer that import_module doesn't support a minimum valid version | 19:35 |
| *** shyama has joined #openstack-nova | 19:36 | |
| dansmith | well, I'm sure we have this kind of thing happen all the time then | 19:36 |
| sdague | dansmith: so, it's error prone, but largely from an install and upgrade perspective because we don't force the correct version anywhere in the field. It would be good if we had a better answer there. But that's a more generic issue. | 19:36 |
| dansmith | yeah | 19:36 |
| sdague | it's no worse than I'm sure a dozen other similar kinds of things in the tree | 19:37 |
| dansmith | yeah | 19:37 |
| sdague | so, I'd say probably fine | 19:37 |
| dansmith | and not retrying a Conflict forever is good :) | 19:37 |
| sdague | yep | 19:37 |
| * dansmith +2d | 19:37 | |
| *** j_king has joined #openstack-nova | 19:38 | |
| *** bapalm has quit IRC | 19:38 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add version counter to Service object https://review.openstack.org/201733 | 19:38 |
| *** meghal has quit IRC | 19:40 | |
| *** meghal has joined #openstack-nova | 19:41 | |
| *** bapalm has joined #openstack-nova | 19:41 | |
| *** nic has quit IRC | 19:41 | |
| *** jkraj has quit IRC | 19:43 | |
| *** browne1 has joined #openstack-nova | 19:44 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: Fix deleting instance with incorrectly attached volume https://review.openstack.org/204231 | 19:44 |
| *** browne has quit IRC | 19:47 | |
| openstackgerrit | Matthew Treinish proposed openstack/nova: Handle neutron exception on ipv6 only floating ip create https://review.openstack.org/210684 | 19:51 |
| *** bapalm has quit IRC | 19:51 | |
| *** bapalm has joined #openstack-nova | 19:51 | |
| mtreinish | mriedem: ^^^ you probably won't like the unit tests (I copied what was there it was mox) | 19:52 |
| mtreinish | also how do I add an ec2 unit test that was a very confusing test dir | 19:52 |
| rods | now that https://review.openstack.org/#/c/123073/27 has been merged, I'd like to back-port it to kilo but looks like patch doesn't apply cleanly and it may need some manual work, what is our policy in a situation like this? | 19:52 |
| *** tjones has joined #openstack-nova | 19:52 | |
| mriedem | rods: you manually resolve the merge conflicts, leave the Conflicts section in the commit message and i like to see a note about what the conflicts were | 19:53 |
| mriedem | or how they were addressed if they are odd | 19:53 |
| mriedem | mtreinish: not sure, i'd have to dig on the ec2 tests | 19:54 |
| mriedem | but i'm about out in 5 min for an appt | 19:54 |
| *** nic has joined #openstack-nova | 19:54 | |
| mtreinish | mriedem: then hurry :) | 19:54 |
| mriedem | gdu | 19:54 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: call host.get_capabilities after checking for bad numa versions https://review.openstack.org/188431 | 19:55 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: log when BAD_LIBVIRT_NUMA_VERSIONS detected https://review.openstack.org/188427 | 19:55 |
| mriedem | melwitt: ^ cleaned up that libvirt version logging tihng, nice catch there - also added a check for it in the test | 19:55 |
| *** bapalm has quit IRC | 19:55 | |
| rods | mriedem cool, thx | 19:55 |
| *** bapalm has joined #openstack-nova | 19:55 | |
| *** ig0r_ has quit IRC | 19:58 | |
| *** kmARC has joined #openstack-nova | 19:59 | |
| *** bapalm has quit IRC | 20:01 | |
| *** bapalm has joined #openstack-nova | 20:01 | |
| j_king | looking for feedback on https://review.openstack.org/#/c/210126/ | 20:02 |
| *** bapalm_ has joined #openstack-nova | 20:04 | |
| openstackgerrit | Baodong (Robert) Li proposed openstack/nova: Handle port delete initiated by neutron https://review.openstack.org/179390 | 20:05 |
| melwitt | mriedem: +2d | 20:05 |
| *** bapalm has quit IRC | 20:05 | |
| *** salv-orlando has joined #openstack-nova | 20:07 | |
| *** ig0r_ has joined #openstack-nova | 20:07 | |
| mnestratov | j_king: test_destroy_with_network_events_neutron_failed_nonfatal_error fails | 20:08 |
| j_king | mnestratov: oh shoot.. will fix. | 20:10 |
| *** otter768 has joined #openstack-nova | 20:10 | |
| *** fawadkhaliq has quit IRC | 20:10 | |
| *** claudiub has quit IRC | 20:11 | |
| edleafe | my grep-fu is failing. How does stevedore get the path for loading plugins? Specifically, where does it get the 'nova/api/openstack/compute/plugins/v3' path for the V3 stuff? | 20:13 |
| *** annashen has joined #openstack-nova | 20:14 | |
| *** otter768 has quit IRC | 20:15 | |
| dansmith | it doesn't | 20:15 |
| dansmith | edleafe: it looks in the entry point config, which is written by setuptools (I think) at setup time | 20:15 |
| dansmith | so /usr/lib/python somewhere | 20:15 |
| melwitt | edleafe, dansmith: yeah, you can see them in setup.cfg for nova | 20:17 |
| edleafe | dansmith: melwitt: ah, I see. Thanks! | 20:17 |
| *** mdrabe_ has quit IRC | 20:18 | |
| *** mdrabe_ has joined #openstack-nova | 20:18 | |
| *** alex_klimov has joined #openstack-nova | 20:19 | |
| *** morgan_503 is now known as morgan_404 | 20:23 | |
| *** bapalm_ has quit IRC | 20:25 | |
| *** bapalm has joined #openstack-nova | 20:25 | |
| *** davideagnello has quit IRC | 20:27 | |
| *** VW_ has quit IRC | 20:27 | |
| *** davideagnello has joined #openstack-nova | 20:28 | |
| *** VW_ has joined #openstack-nova | 20:28 | |
| jaypipes | sdague: nice review on the flavor-cpu-model spec thing. I've been spending most of the afternoon actually thinking about that problem space. | 20:29 |
| *** VW_ has quit IRC | 20:30 | |
| *** bapalm has quit IRC | 20:30 | |
| *** VW_ has joined #openstack-nova | 20:31 | |
| *** bapalm has joined #openstack-nova | 20:31 | |
| jaypipes | sdague: I think that one thing that would make a real difference is to get rid of as much of the hand-keying of extra_specs information (on the compute_nodes table and host_aggregates table) as possible, and instead have a standard hardware discovery mechanism (driven by the virt driver) inventory everything about a compute node's resources and capabilities. | 20:31 |
| dansmith | lots of aggregation to be done there to make it usable | 20:32 |
| jaypipes | sdague: in addition to that, I think it's wrong to couple things like CPU feature flags to anything related to the flavor at all. Those kinds of things belong on the image metadata only, IMHO. | 20:32 |
| *** meghal has quit IRC | 20:32 | |
| jaypipes | dansmith: agreed. | 20:32 |
| dansmith | jaypipes: except that people that want this want to charge more if you use more sophisticated cpus | 20:33 |
| dansmith | which makes it a flavor thing I think | 20:33 |
| jaypipes | dansmith: The CPU features represent something that a guest image requires, though. The CPU model/speed/performance is certainly something that can be related to a flavor, yes. | 20:35 |
| *** meghal has joined #openstack-nova | 20:35 | |
| *** bapalm has quit IRC | 20:35 | |
| dansmith | sure, we kinda need both | 20:35 |
| jaypipes | dansmith: for instance, you can definitely say that your image needs aes-ni instruction sets, but you wouldn't say that yoru image needs a 2.6Mhz IvyBridge v2 CPU... | 20:36 |
| dansmith | or, we get intel to stop innovating and then wait there years for a hardware refresh :) | 20:36 |
| jaypipes | lol | 20:36 |
| jaypipes | or just use AMD | 20:36 |
| dansmith | jaypipes: sure, that's one use case, agreed | 20:36 |
| dansmith | jaypipes: ouch :D | 20:36 |
| *** rushil has joined #openstack-nova | 20:40 | |
| *** chuckcarmack has left #openstack-nova | 20:42 | |
| *** kmARC has quit IRC | 20:46 | |
| *** echoingumesh has joined #openstack-nova | 20:46 | |
| *** VW_ has quit IRC | 20:49 | |
| *** VW_ has joined #openstack-nova | 20:50 | |
| *** VW_ has quit IRC | 20:50 | |
| *** VW_ has joined #openstack-nova | 20:51 | |
| *** rushil has quit IRC | 20:52 | |
| jaypipes | cfriesen_: you around? | 20:53 |
| *** rushil has joined #openstack-nova | 20:54 | |
| *** rlrossit has left #openstack-nova | 20:56 | |
| *** ig0r_ has quit IRC | 20:58 | |
| *** rdo has quit IRC | 20:59 | |
| *** rdo has joined #openstack-nova | 21:01 | |
| *** ijuwang has quit IRC | 21:02 | |
| *** baoli has quit IRC | 21:02 | |
| *** ijuwang has joined #openstack-nova | 21:02 | |
| *** morgan_404 is now known as morgan_410 | 21:04 | |
| *** mdrabe__ has joined #openstack-nova | 21:04 | |
| *** mili has joined #openstack-nova | 21:04 | |
| *** morgan_410 is now known as morgan_404 | 21:05 | |
| *** thorst1 has quit IRC | 21:05 | |
| *** thangp has quit IRC | 21:06 | |
| *** mdrabe_ has quit IRC | 21:07 | |
| *** aysyd has quit IRC | 21:09 | |
| *** ivasev has quit IRC | 21:10 | |
| *** annashen has quit IRC | 21:13 | |
| *** gyee has joined #openstack-nova | 21:17 | |
| *** apuimedo has joined #openstack-nova | 21:18 | |
| *** Piet has quit IRC | 21:18 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: pci: Use PciDeviceList for PciDevTracker.pci_devs https://review.openstack.org/210948 | 21:19 |
| openstackgerrit | Jay Pipes proposed openstack/nova: pci: Move whitelist filtering inside PCI tracker https://review.openstack.org/210182 | 21:19 |
| openstackgerrit | Jay Pipes proposed openstack/nova: pci: Remove get_pci_devices_filter() method https://review.openstack.org/210183 | 21:19 |
| *** mili has quit IRC | 21:22 | |
| *** Piet has joined #openstack-nova | 21:22 | |
| *** liusheng has quit IRC | 21:25 | |
| *** liusheng has joined #openstack-nova | 21:26 | |
| *** raildo is now known as raildo-afk | 21:26 | |
| *** rushil_ has joined #openstack-nova | 21:28 | |
| *** burgerk has quit IRC | 21:29 | |
| *** ctrath has quit IRC | 21:29 | |
| *** mriedem is now known as mriedem_away | 21:30 | |
| *** rushil has quit IRC | 21:31 | |
| *** rdo has quit IRC | 21:31 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create RequestSpec object https://review.openstack.org/145528 | 21:32 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 21:32 |
| *** rdo has joined #openstack-nova | 21:33 | |
| xyang1 | mriedem_away: Hi, I don't see this in the gate queue, maybe need to be re-approved? https://review.openstack.org/#/c/194454/ thanks! | 21:33 |
| *** IanGovett has quit IRC | 21:34 | |
| *** annegent_ has joined #openstack-nova | 21:34 | |
| *** IanGovett has joined #openstack-nova | 21:35 | |
| *** annegentle has quit IRC | 21:37 | |
| *** sachin has joined #openstack-nova | 21:41 | |
| *** alejandrito has quit IRC | 21:41 | |
| *** bapalm has joined #openstack-nova | 21:42 | |
| *** mdrabe__ has quit IRC | 21:45 | |
| *** rushil_ has quit IRC | 21:48 | |
| *** bapalm has quit IRC | 21:49 | |
| *** bapalm has joined #openstack-nova | 21:49 | |
| *** rushil has joined #openstack-nova | 21:50 | |
| *** ildikov has quit IRC | 21:51 | |
| *** dave-mccowan has quit IRC | 21:51 | |
| *** rushil has quit IRC | 21:51 | |
| *** bapalm has quit IRC | 21:56 | |
| *** bapalm has joined #openstack-nova | 21:57 | |
| *** Raildo has joined #openstack-nova | 21:57 | |
| *** Raildo has quit IRC | 21:59 | |
| *** meghal1 has joined #openstack-nova | 22:04 | |
| *** meghal has quit IRC | 22:04 | |
| *** zsais has quit IRC | 22:06 | |
| *** ildikov has joined #openstack-nova | 22:06 | |
| *** yassine__ has quit IRC | 22:08 | |
| *** meghal1 has quit IRC | 22:08 | |
| *** burt has quit IRC | 22:09 | |
| *** mwagner_lap has quit IRC | 22:09 | |
| *** bapalm has quit IRC | 22:11 | |
| *** otter768 has joined #openstack-nova | 22:11 | |
| *** bapalm has joined #openstack-nova | 22:11 | |
| *** jecarey has quit IRC | 22:12 | |
| *** r-daneel has quit IRC | 22:12 | |
| *** VW_ has quit IRC | 22:12 | |
| *** diogogmt has quit IRC | 22:12 | |
| *** agireud has quit IRC | 22:12 | |
| *** VW_ has joined #openstack-nova | 22:13 | |
| *** dustins has quit IRC | 22:14 | |
| *** diogogmt has joined #openstack-nova | 22:14 | |
| *** otter768 has quit IRC | 22:15 | |
| *** claudiub has joined #openstack-nova | 22:15 | |
| *** bapalm has quit IRC | 22:16 | |
| *** agireud has joined #openstack-nova | 22:17 | |
| *** VW_ has quit IRC | 22:17 | |
| *** edmondsw has quit IRC | 22:17 | |
| *** kiwik-chenrui has joined #openstack-nova | 22:19 | |
| *** artom_ has joined #openstack-nova | 22:19 | |
| *** mriedem has joined #openstack-nova | 22:21 | |
| *** Kevin_Zheng_ has quit IRC | 22:21 | |
| *** Kevin_Zheng_ has joined #openstack-nova | 22:21 | |
| *** mriedem_away has quit IRC | 22:21 | |
| *** annegent_ has quit IRC | 22:22 | |
| *** rfolco has quit IRC | 22:22 | |
| *** RuiChen has quit IRC | 22:22 | |
| *** artom has quit IRC | 22:22 | |
| *** tonytan4ever has quit IRC | 22:23 | |
| *** mriedem has quit IRC | 22:23 | |
| *** tonytan4ever has joined #openstack-nova | 22:23 | |
| *** agireud has quit IRC | 22:23 | |
| *** bapalm has joined #openstack-nova | 22:24 | |
| *** mriedem has joined #openstack-nova | 22:24 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Fix serial port issue on Windows Threshold https://review.openstack.org/192617 | 22:25 |
| *** geekinutah has joined #openstack-nova | 22:28 | |
| *** ijuwang has quit IRC | 22:31 | |
| *** tpeoples has quit IRC | 22:31 | |
| mriedem | xyang1: it's in the gate now | 22:32 |
| xyang1 | mriedem: Thanks | 22:33 |
| xyang1 | mriedem: I am hoping it can get merged tonight. It's been more than a week | 22:34 |
| mriedem | yeah, my series is all merged as of today | 22:34 |
| mriedem | so just recheck yours until it's in | 22:34 |
| *** rfolco has joined #openstack-nova | 22:35 | |
| xyang1 | mriedem: Sure, thanks | 22:35 |
| *** jwcroppe has quit IRC | 22:38 | |
| *** bapalm has quit IRC | 22:42 | |
| *** bapalm has joined #openstack-nova | 22:42 | |
| *** VW_ has joined #openstack-nova | 22:43 | |
| *** marcusvrn has quit IRC | 22:43 | |
| *** tonytan4ever has quit IRC | 22:44 | |
| *** bapalm has quit IRC | 22:47 | |
| *** alex_klimov has quit IRC | 22:47 | |
| *** mwagner_lap has joined #openstack-nova | 22:47 | |
| *** jaypipes has quit IRC | 22:50 | |
| *** achanda has quit IRC | 22:50 | |
| *** armax has quit IRC | 22:51 | |
| *** ildikov has quit IRC | 22:58 | |
| *** wverdugo500 has quit IRC | 22:58 | |
| *** bnemec has quit IRC | 22:59 | |
| jroll | dansmith: that ironic change seems fine to me, yeah | 22:59 |
| dansmith | jroll: cool thanks | 23:00 |
| jroll | np | 23:00 |
| jroll | I swear I reviewed that earlier but apparently not :/ | 23:00 |
| mtreinish | jroll: oh, wasn't I supposed to give you a hand with the ironic full stuff? | 23:01 |
| mtreinish | I completely forgot about that | 23:01 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move existing V2 to legacy_v2 - step 3 https://review.openstack.org/211777 | 23:01 |
| jroll | mtreinish: yeah, and vice versa :) | 23:01 |
| jroll | I've been crazy busy, no worries | 23:01 |
| mtreinish | heh, ok | 23:01 |
| jroll | ironic midcycle this week so maybe we should plan to spend some time on it while I have a bunch of people on hand to debug things? | 23:01 |
| mtreinish | jroll: that sounds like a good plan | 23:02 |
| jroll | or more like... if you give me some pointers to what we should be doing here, I can hack on it at the midcycle and bug you as needed | 23:02 |
| * jroll knows almost nothing about tempest | 23:02 | |
| *** achanda has joined #openstack-nova | 23:02 | |
| mtreinish | jroll: heh, well I haven't even looked at the failures yet :) | 23:04 |
| mtreinish | I guess I should do that | 23:04 |
| jroll | neither have I really | 23:04 |
| jroll | I'm going afk for now but I'll poke you sometime tomorrow | 23:04 |
| *** jwcroppe has joined #openstack-nova | 23:08 | |
| *** dave-mccowan has joined #openstack-nova | 23:10 | |
| *** jwcroppe has quit IRC | 23:10 | |
| *** ildikov has joined #openstack-nova | 23:11 | |
| *** jwcroppe has joined #openstack-nova | 23:11 | |
| *** zzzeek has quit IRC | 23:12 | |
| *** annashen has joined #openstack-nova | 23:14 | |
| openstackgerrit | Danny Wilson proposed openstack/nova: libvirt: add discard support for attached volumes https://review.openstack.org/205726 | 23:14 |
| *** jwcroppe has quit IRC | 23:15 | |
| *** annashen has quit IRC | 23:18 | |
| *** sachin has quit IRC | 23:20 | |
| *** sachin has joined #openstack-nova | 23:21 | |
| *** busterswt has quit IRC | 23:21 | |
| edleafe | alex_xu: johnthetubaguy: heading out now. Here is step 3 patch: https://review.openstack.org/211777 | 23:22 |
| *** marzif has joined #openstack-nova | 23:26 | |
| *** morgan_404 has quit IRC | 23:29 | |
| *** VW_ has quit IRC | 23:29 | |
| *** VW_ has joined #openstack-nova | 23:30 | |
| *** morganfainberg has joined #openstack-nova | 23:31 | |
| openstackgerrit | Michael Still proposed openstack/nova: Xenapi: Handle missing aggregate metadata on startup https://review.openstack.org/156126 | 23:31 |
| *** morganfainberg is now known as morgan_404 | 23:32 | |
| *** VW_ has quit IRC | 23:34 | |
| *** ijw_ has joined #openstack-nova | 23:34 | |
| *** ijw__ has joined #openstack-nova | 23:35 | |
| *** ijw has quit IRC | 23:37 | |
| *** apuimedo has quit IRC | 23:37 | |
| *** meghal has joined #openstack-nova | 23:37 | |
| *** rfolco has quit IRC | 23:38 | |
| *** ijw_ has quit IRC | 23:39 | |
| *** armax has joined #openstack-nova | 23:45 | |
| *** echoingumesh has quit IRC | 23:45 | |
| *** ijw__ has quit IRC | 23:45 | |
| *** echoingumesh has joined #openstack-nova | 23:45 | |
| *** ijw has joined #openstack-nova | 23:46 | |
| *** echoingumesh has quit IRC | 23:51 | |
| *** geekinutah has quit IRC | 23:51 | |
| *** VW_ has joined #openstack-nova | 23:52 | |
| *** alejandrito has joined #openstack-nova | 23:57 | |
| *** leeantho has quit IRC | 23:58 | |
| *** david-lyle has quit IRC | 23:58 | |
| *** tpeoples has joined #openstack-nova | 23:58 | |
| *** david-lyle has joined #openstack-nova | 23:58 | |
| *** dannywilson has quit IRC | 23:58 | |
| *** thumpba has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!