*** heyongli has quit IRC | 00:00 | |
*** heyongli has joined #openstack-nova | 00:00 | |
*** dave-mccowan has joined #openstack-nova | 00:05 | |
*** tbachman_ has joined #openstack-nova | 00:07 | |
*** tbachman has quit IRC | 00:08 | |
*** stvnoyes has quit IRC | 00:09 | |
*** tbachman has joined #openstack-nova | 00:09 | |
*** heyongli has quit IRC | 00:10 | |
*** heyongli has joined #openstack-nova | 00:10 | |
*** tbachman_ has quit IRC | 00:12 | |
*** heyongli has quit IRC | 00:20 | |
*** heyongli has joined #openstack-nova | 00:20 | |
*** cdent has quit IRC | 00:25 | |
*** jamesdenton has joined #openstack-nova | 00:26 | |
*** mriedem has quit IRC | 00:29 | |
*** heyongli has quit IRC | 00:30 | |
*** heyongli has joined #openstack-nova | 00:31 | |
*** edmondsw has joined #openstack-nova | 00:31 | |
*** mlavalle has quit IRC | 00:32 | |
*** edmondsw has quit IRC | 00:36 | |
*** heyongli has quit IRC | 00:41 | |
*** heyongli has joined #openstack-nova | 00:41 | |
*** heyongli has quit IRC | 00:51 | |
*** heyongli has joined #openstack-nova | 00:51 | |
*** larainema has joined #openstack-nova | 00:52 | |
*** ljjjustin has joined #openstack-nova | 01:00 | |
*** heyongli has quit IRC | 01:01 | |
*** heyongli has joined #openstack-nova | 01:01 | |
*** markvoelker has quit IRC | 01:08 | |
*** catintheroof has quit IRC | 01:10 | |
openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512 | 01:11 |
---|---|---|
*** catintheroof has joined #openstack-nova | 01:11 | |
*** heyongli has quit IRC | 01:11 | |
*** heyongli has joined #openstack-nova | 01:12 | |
*** liverpooler has joined #openstack-nova | 01:13 | |
*** bjhuangr has joined #openstack-nova | 01:14 | |
*** catintheroof has quit IRC | 01:15 | |
*** gyee has quit IRC | 01:16 | |
*** heyongli has quit IRC | 01:22 | |
*** heyongli has joined #openstack-nova | 01:22 | |
openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216 | 01:25 |
*** esberglu has quit IRC | 01:26 | |
*** Apoorva_ has joined #openstack-nova | 01:27 | |
*** Apoorva has quit IRC | 01:30 | |
*** Apoorva_ has quit IRC | 01:32 | |
*** heyongli has quit IRC | 01:32 | |
*** heyongli has joined #openstack-nova | 01:32 | |
*** dave-mccowan has quit IRC | 01:34 | |
*** tuanla____ has joined #openstack-nova | 01:37 | |
*** liverpooler has quit IRC | 01:42 | |
*** heyongli has quit IRC | 01:42 | |
*** heyongli has joined #openstack-nova | 01:42 | |
*** namnh has joined #openstack-nova | 01:46 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use neutron port_list when filtering instance by ip https://review.openstack.org/525505 | 01:47 |
*** tonyb has quit IRC | 01:47 | |
*** tonyb has joined #openstack-nova | 01:49 | |
*** yangyapeng has quit IRC | 01:51 | |
*** yangyapeng has joined #openstack-nova | 01:52 | |
*** heyongli has quit IRC | 01:52 | |
*** heyongli has joined #openstack-nova | 01:53 | |
*** dave-mccowan has joined #openstack-nova | 01:55 | |
*** phuongnh has joined #openstack-nova | 01:57 | |
openstackgerrit | Chen Hanxiao proposed openstack/nova master: manager: more detailed info of unsupported compute driver https://review.openstack.org/530701 | 02:02 |
*** heyongli has quit IRC | 02:03 | |
*** heyongli has joined #openstack-nova | 02:03 | |
*** markvoelker has joined #openstack-nova | 02:05 | |
openstackgerrit | Wangpan proposed openstack/nova master: Handle glance exception during rotating instance backup https://review.openstack.org/523395 | 02:07 |
*** Brin has joined #openstack-nova | 02:10 | |
*** heyongli has quit IRC | 02:13 | |
*** heyongli has joined #openstack-nova | 02:13 | |
*** annp has joined #openstack-nova | 02:14 | |
*** frank_wang has joined #openstack-nova | 02:14 | |
*** dave-mccowan has quit IRC | 02:18 | |
*** dave-mccowan has joined #openstack-nova | 02:19 | |
*** edmondsw has joined #openstack-nova | 02:19 | |
*** heyongli has quit IRC | 02:23 | |
*** heyongli has joined #openstack-nova | 02:23 | |
*** edmondsw has quit IRC | 02:24 | |
*** markmcclain has quit IRC | 02:25 | |
*** gouthamr has quit IRC | 02:27 | |
*** yamahata has quit IRC | 02:30 | |
*** heyongli has quit IRC | 02:33 | |
*** heyongli has joined #openstack-nova | 02:34 | |
*** owalsh_ has joined #openstack-nova | 02:42 | |
*** trungnv has quit IRC | 02:43 | |
*** heyongli has quit IRC | 02:44 | |
*** heyongli has joined #openstack-nova | 02:44 | |
*** owalsh has quit IRC | 02:45 | |
*** karthiks has joined #openstack-nova | 02:49 | |
*** heyongli has quit IRC | 02:54 | |
*** heyongli has joined #openstack-nova | 02:54 | |
*** xinliang has quit IRC | 02:57 | |
*** heyongli has quit IRC | 03:04 | |
*** heyongli has joined #openstack-nova | 03:04 | |
*** markmcclain has joined #openstack-nova | 03:05 | |
*** xinliang has joined #openstack-nova | 03:10 | |
*** heyongli has quit IRC | 03:14 | |
*** heyongli has joined #openstack-nova | 03:15 | |
*** flwang has quit IRC | 03:16 | |
*** flwang has joined #openstack-nova | 03:17 | |
*** bzhao__ has joined #openstack-nova | 03:18 | |
*** Apoorva has joined #openstack-nova | 03:23 | |
*** Apoorva has quit IRC | 03:24 | |
*** heyongli has quit IRC | 03:25 | |
*** heyongli has joined #openstack-nova | 03:25 | |
*** Apoorva has joined #openstack-nova | 03:25 | |
*** gongysh has joined #openstack-nova | 03:28 | |
*** masber has quit IRC | 03:32 | |
*** markmcclain has quit IRC | 03:33 | |
*** heyongli has quit IRC | 03:35 | |
*** heyongli has joined #openstack-nova | 03:35 | |
*** bjhuangr_ has joined #openstack-nova | 03:36 | |
*** abhishekk has joined #openstack-nova | 03:43 | |
*** gcb has quit IRC | 03:44 | |
*** nicolasbock has quit IRC | 03:44 | |
*** heyongli has quit IRC | 03:45 | |
*** heyongli has joined #openstack-nova | 03:45 | |
*** jaianshu has joined #openstack-nova | 03:53 | |
*** heyongli has quit IRC | 03:55 | |
*** heyongli has joined #openstack-nova | 03:56 | |
*** jaianshu_ has joined #openstack-nova | 03:56 | |
*** jaianshu has quit IRC | 03:59 | |
*** hongbin has joined #openstack-nova | 03:59 | |
*** jaianshu__ has joined #openstack-nova | 04:03 | |
*** heyongli has quit IRC | 04:06 | |
*** heyongli has joined #openstack-nova | 04:06 | |
*** jaianshu_ has quit IRC | 04:06 | |
*** edmondsw has joined #openstack-nova | 04:07 | |
*** edmondsw has quit IRC | 04:12 | |
*** vladikr_ has joined #openstack-nova | 04:13 | |
*** udesale has joined #openstack-nova | 04:15 | |
*** heyongli has quit IRC | 04:16 | |
*** heyongli has joined #openstack-nova | 04:16 | |
*** gmann has joined #openstack-nova | 04:19 | |
*** Apoorva has quit IRC | 04:20 | |
*** masber has joined #openstack-nova | 04:21 | |
*** Apoorva has joined #openstack-nova | 04:21 | |
*** Apoorva has quit IRC | 04:25 | |
*** frank_wang has quit IRC | 04:26 | |
*** heyongli has quit IRC | 04:26 | |
*** heyongli has joined #openstack-nova | 04:26 | |
*** hemna_ has quit IRC | 04:29 | |
*** Apoorva has joined #openstack-nova | 04:30 | |
*** heyongli has quit IRC | 04:36 | |
*** heyongli has joined #openstack-nova | 04:37 | |
*** ratailor has joined #openstack-nova | 04:45 | |
*** crushil_ has joined #openstack-nova | 04:46 | |
*** heyongli has quit IRC | 04:47 | |
*** heyongli has joined #openstack-nova | 04:47 | |
*** links has joined #openstack-nova | 04:48 | |
*** crushil_ has quit IRC | 04:51 | |
*** crushil_ has joined #openstack-nova | 04:51 | |
*** psachin has joined #openstack-nova | 04:56 | |
*** heyongli has quit IRC | 04:57 | |
*** heyongli has joined #openstack-nova | 04:57 | |
*** links has quit IRC | 04:58 | |
*** markmcclain has joined #openstack-nova | 05:01 | |
*** links has joined #openstack-nova | 05:03 | |
*** sridharg has joined #openstack-nova | 05:03 | |
*** vladikr_ has quit IRC | 05:05 | |
*** dave-mccowan has quit IRC | 05:06 | |
*** heyongli has quit IRC | 05:07 | |
*** heyongli has joined #openstack-nova | 05:07 | |
*** yamamoto has joined #openstack-nova | 05:10 | |
*** ratailor has quit IRC | 05:12 | |
*** ratailor has joined #openstack-nova | 05:12 | |
*** Apoorva has quit IRC | 05:14 | |
*** heyongli has quit IRC | 05:17 | |
*** heyongli has joined #openstack-nova | 05:18 | |
*** crushil_ has quit IRC | 05:25 | |
*** gongysh has quit IRC | 05:26 | |
*** heyongli has quit IRC | 05:28 | |
*** heyongli has joined #openstack-nova | 05:28 | |
*** hieulq has quit IRC | 05:30 | |
*** annp has quit IRC | 05:30 | |
*** hoangcx has quit IRC | 05:30 | |
*** phuongnh has quit IRC | 05:30 | |
*** namnh has quit IRC | 05:30 | |
*** annp has joined #openstack-nova | 05:31 | |
*** phuongnh has joined #openstack-nova | 05:31 | |
*** namnh has joined #openstack-nova | 05:31 | |
*** hieulq has joined #openstack-nova | 05:31 | |
*** hoangcx has joined #openstack-nova | 05:31 | |
*** crushil_ has joined #openstack-nova | 05:32 | |
*** crushil_ has quit IRC | 05:33 | |
*** tuanla____ has quit IRC | 05:34 | |
*** crushil_ has joined #openstack-nova | 05:34 | |
*** tuanla____ has joined #openstack-nova | 05:34 | |
*** heyongli has quit IRC | 05:38 | |
*** heyongli has joined #openstack-nova | 05:38 | |
*** gcb has joined #openstack-nova | 05:42 | |
*** frank_wang has joined #openstack-nova | 05:44 | |
*** tuanla____ has quit IRC | 05:45 | |
*** tuanla____ has joined #openstack-nova | 05:46 | |
*** zhurong has joined #openstack-nova | 05:47 | |
*** artom_ has quit IRC | 05:48 | |
*** heyongli has quit IRC | 05:48 | |
*** heyongli has joined #openstack-nova | 05:48 | |
*** edmondsw has joined #openstack-nova | 05:56 | |
*** heyongli has quit IRC | 05:58 | |
*** heyongli has joined #openstack-nova | 05:59 | |
*** edmondsw has quit IRC | 06:00 | |
*** heyongli has quit IRC | 06:09 | |
*** heyongli has joined #openstack-nova | 06:09 | |
*** masber has quit IRC | 06:13 | |
*** heyongli has quit IRC | 06:19 | |
*** heyongli has joined #openstack-nova | 06:19 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova master: Use method validate_integer from oslo.utils https://review.openstack.org/528620 | 06:21 |
*** markvoelker has quit IRC | 06:28 | |
*** heyongli has quit IRC | 06:29 | |
*** heyongli has joined #openstack-nova | 06:29 | |
*** armax has quit IRC | 06:30 | |
*** armax has joined #openstack-nova | 06:30 | |
*** armax has quit IRC | 06:31 | |
*** armax has joined #openstack-nova | 06:32 | |
*** armax has quit IRC | 06:32 | |
*** crushil_ has quit IRC | 06:32 | |
*** armax has joined #openstack-nova | 06:33 | |
*** armax has quit IRC | 06:34 | |
*** Tom-Tom has joined #openstack-nova | 06:38 | |
*** heyongli has quit IRC | 06:39 | |
*** esberglu has joined #openstack-nova | 06:40 | |
*** heyongli has joined #openstack-nova | 06:40 | |
*** esberglu has quit IRC | 06:40 | |
*** esberglu has joined #openstack-nova | 06:40 | |
*** moshele has joined #openstack-nova | 06:44 | |
*** yamamoto has quit IRC | 06:49 | |
*** heyongli has quit IRC | 06:50 | |
*** heyongli has joined #openstack-nova | 06:50 | |
*** heyongli has quit IRC | 07:00 | |
*** heyongli has joined #openstack-nova | 07:00 | |
openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216 | 07:01 |
*** Eran_Kuris has quit IRC | 07:01 | |
*** Eran_Kuris has joined #openstack-nova | 07:03 | |
*** gongysh has joined #openstack-nova | 07:09 | |
*** pcaruana has joined #openstack-nova | 07:09 | |
*** heyongli has quit IRC | 07:10 | |
*** heyongli has joined #openstack-nova | 07:10 | |
*** david-lyle has quit IRC | 07:12 | |
*** david-lyle has joined #openstack-nova | 07:12 | |
*** goutham has joined #openstack-nova | 07:13 | |
*** heyongli has quit IRC | 07:20 | |
*** heyongli has joined #openstack-nova | 07:21 | |
*** rcernin has quit IRC | 07:25 | |
*** markmcclain has quit IRC | 07:30 | |
*** heyongli has quit IRC | 07:31 | |
*** heyongli has joined #openstack-nova | 07:31 | |
*** markmcclain has joined #openstack-nova | 07:32 | |
*** yamamoto has joined #openstack-nova | 07:38 | |
*** heyongli has quit IRC | 07:41 | |
*** heyongli has joined #openstack-nova | 07:41 | |
*** edmondsw has joined #openstack-nova | 07:44 | |
*** mdnadeem has joined #openstack-nova | 07:46 | |
*** edmondsw has quit IRC | 07:48 | |
*** yamamoto has quit IRC | 07:49 | |
*** heyongli has quit IRC | 07:51 | |
*** heyongli has joined #openstack-nova | 07:51 | |
*** gongysh has quit IRC | 07:52 | |
Roamer` | johnthetubaguy, when you get back, could you please take a look at https://review.openstack.org/140733/ - is there a chance that you might remove your procedural -2 from a previous release cycle and possibly consider the almost trivial StorPool Nova volume attachment driver for inclusion in Queens? Thanks a lot in advance! Right now, with our Cinder and os-brick drivers merged, it's a bit weird - | 07:55 |
Roamer` | the main use case for the Cinder driver still requires manual patching :) | 07:55 |
*** damien_r has joined #openstack-nova | 07:55 | |
*** damien_r1 has joined #openstack-nova | 07:56 | |
*** damien_r has quit IRC | 07:56 | |
*** hongbin has quit IRC | 08:01 | |
*** heyongli has quit IRC | 08:01 | |
*** heyongli has joined #openstack-nova | 08:02 | |
*** tesseract has joined #openstack-nova | 08:05 | |
*** karthiks has quit IRC | 08:07 | |
*** fragatina has joined #openstack-nova | 08:07 | |
*** amodi has quit IRC | 08:08 | |
*** heyongli has quit IRC | 08:12 | |
*** heyongli has joined #openstack-nova | 08:12 | |
*** medy has joined #openstack-nova | 08:15 | |
*** medy has quit IRC | 08:16 | |
*** medy has joined #openstack-nova | 08:16 | |
*** medy has quit IRC | 08:18 | |
*** medy has joined #openstack-nova | 08:18 | |
*** medy has quit IRC | 08:19 | |
*** medy has joined #openstack-nova | 08:19 | |
*** medy has quit IRC | 08:21 | |
*** medy has joined #openstack-nova | 08:21 | |
*** heyongli has quit IRC | 08:22 | |
*** medy has quit IRC | 08:22 | |
*** heyongli has joined #openstack-nova | 08:22 | |
*** medy has joined #openstack-nova | 08:22 | |
*** artom_ has joined #openstack-nova | 08:22 | |
*** medy has quit IRC | 08:24 | |
*** medy has joined #openstack-nova | 08:24 | |
*** frank_wang has quit IRC | 08:24 | |
*** frank_wang has joined #openstack-nova | 08:25 | |
*** medy has quit IRC | 08:25 | |
*** medy has joined #openstack-nova | 08:25 | |
*** medy has quit IRC | 08:27 | |
*** medy has joined #openstack-nova | 08:27 | |
*** medy has quit IRC | 08:28 | |
*** medy has joined #openstack-nova | 08:28 | |
*** markvoelker has joined #openstack-nova | 08:29 | |
*** medy has quit IRC | 08:29 | |
*** heyongli has quit IRC | 08:32 | |
*** heyongli has joined #openstack-nova | 08:32 | |
*** gmann has quit IRC | 08:33 | |
*** moshele has quit IRC | 08:35 | |
*** moshele has joined #openstack-nova | 08:37 | |
*** fragatina has quit IRC | 08:38 | |
*** ragiman has joined #openstack-nova | 08:41 | |
*** heyongli has quit IRC | 08:42 | |
*** heyongli has joined #openstack-nova | 08:43 | |
*** artom_ has quit IRC | 08:45 | |
*** fragatina has joined #openstack-nova | 08:47 | |
*** yamamoto has joined #openstack-nova | 08:49 | |
*** moshele has quit IRC | 08:50 | |
*** fragatina has quit IRC | 08:50 | |
*** moshele has joined #openstack-nova | 08:50 | |
*** fragatina has joined #openstack-nova | 08:51 | |
*** heyongli has quit IRC | 08:53 | |
*** heyongli has joined #openstack-nova | 08:53 | |
*** karthiks has joined #openstack-nova | 08:54 | |
*** yamamoto has quit IRC | 08:57 | |
*** gmann has joined #openstack-nova | 09:00 | |
*** heyongli has quit IRC | 09:03 | |
*** heyongli has joined #openstack-nova | 09:03 | |
*** markvoelker has quit IRC | 09:03 | |
*** yassine has joined #openstack-nova | 09:05 | |
*** sshwarts has joined #openstack-nova | 09:05 | |
*** yassine is now known as Guest71543 | 09:05 | |
openstackgerrit | Merged openstack/nova master: Deprecate API extensions policies https://review.openstack.org/526258 | 09:08 |
stephenfin | mdbooth: Done | 09:11 |
*** heyongli has quit IRC | 09:13 | |
*** heyongli has joined #openstack-nova | 09:13 | |
*** derekh has joined #openstack-nova | 09:17 | |
*** fragatina has quit IRC | 09:17 | |
*** fragatina has joined #openstack-nova | 09:18 | |
*** heyongli has quit IRC | 09:23 | |
*** heyongli has joined #openstack-nova | 09:24 | |
*** edmondsw has joined #openstack-nova | 09:32 | |
openstackgerrit | Rajesh Tailor proposed openstack/nova master: Allow force-delete even if task_state is not None https://review.openstack.org/530879 | 09:32 |
*** heyongli has quit IRC | 09:34 | |
*** heyongli has joined #openstack-nova | 09:34 | |
*** Brin has quit IRC | 09:35 | |
*** edmondsw has quit IRC | 09:37 | |
*** mdnadeem has quit IRC | 09:39 | |
*** lucas-afk is now known as lucasagomes | 09:40 | |
*** Tom-Tom has quit IRC | 09:43 | |
*** heyongli has quit IRC | 09:44 | |
*** ameeda has quit IRC | 09:44 | |
*** heyongli has joined #openstack-nova | 09:44 | |
*** gszasz has joined #openstack-nova | 09:48 | |
*** Tom-Tom has joined #openstack-nova | 09:51 | |
*** mdnadeem has joined #openstack-nova | 09:52 | |
*** yamamoto has joined #openstack-nova | 09:53 | |
*** heyongli has quit IRC | 09:54 | |
*** heyongli has joined #openstack-nova | 09:54 | |
*** ralonsoh has joined #openstack-nova | 10:00 | |
*** markvoelker has joined #openstack-nova | 10:01 | |
*** heyongli has quit IRC | 10:04 | |
*** heyongli has joined #openstack-nova | 10:05 | |
mdbooth | stephenfin: Thanks | 10:05 |
*** ralonsoh_ has joined #openstack-nova | 10:06 | |
*** ralonsoh has quit IRC | 10:06 | |
*** mvk has quit IRC | 10:06 | |
*** mvk has joined #openstack-nova | 10:07 | |
*** phuongnh has quit IRC | 10:07 | |
*** pcaruana has quit IRC | 10:09 | |
*** Tom-Tom has quit IRC | 10:09 | |
*** pcaruana has joined #openstack-nova | 10:10 | |
*** namnh has quit IRC | 10:11 | |
*** Tom-Tom has joined #openstack-nova | 10:12 | |
*** jaianshu_ has joined #openstack-nova | 10:14 | |
*** heyongli has quit IRC | 10:15 | |
*** heyongli has joined #openstack-nova | 10:15 | |
*** frank_wang has quit IRC | 10:15 | |
*** Tom-Tom has quit IRC | 10:16 | |
*** damien_r1 has quit IRC | 10:17 | |
*** jaianshu__ has quit IRC | 10:18 | |
*** ljjjustin has quit IRC | 10:18 | |
*** annp has quit IRC | 10:21 | |
*** owalsh has joined #openstack-nova | 10:21 | |
*** owalsh_ has quit IRC | 10:22 | |
*** Trident has joined #openstack-nova | 10:25 | |
*** heyongli has quit IRC | 10:25 | |
*** heyongli has joined #openstack-nova | 10:25 | |
*** tuanla____ has quit IRC | 10:25 | |
*** jaianshu__ has joined #openstack-nova | 10:26 | |
*** claudiub has joined #openstack-nova | 10:26 | |
*** cdent has joined #openstack-nova | 10:27 | |
*** jaianshu_ has quit IRC | 10:30 | |
ralonsoh_ | stephenfin: hello, about https://review.openstack.org/#/c/449257/52/nova/pci/request.py | 10:34 |
stephenfin | ralonsoh_: Shoot | 10:34 |
*** markvoelker has quit IRC | 10:34 | |
ralonsoh_ | stephenfin: if I accept this suggestion, I need to refactor the whole patch | 10:34 |
ralonsoh_ | stephenfin: the scope of this feature is to store a list of strings in spec | 10:35 |
stephenfin | ralonsoh_: Yup, afraid so :( I realize | 10:35 |
*** heyongli has quit IRC | 10:35 | |
stephenfin | I realize that, but as soon as we create the object that way, we're stuck with it | 10:35 |
ralonsoh_ | stephenfin: not to modify how InstacePCIRequestSpec was storing the other parameters | 10:35 |
*** heyongli has joined #openstack-nova | 10:35 | |
ralonsoh_ | stephenfin: but "spec" was storing parameters in key=value fashion | 10:36 |
stephenfin | Right, but you're already modifying how the other parameters were stored | 10:36 |
stephenfin | Previously they were stored in a 'ListOfDictOfNullableStringsField'. Now they're stored in the 'InstancePCIRequestSpec' object | 10:37 |
ralonsoh_ | in a generic way. But there were no specific fields for those parameters (product, vendor, etc) | 10:37 |
ralonsoh_ | ok, I'll submit this modification next release | 10:37 |
ralonsoh_ | First, I'll submit again the spec | 10:38 |
stephenfin | ralonsoh_: I realize it's a good chunk of rework, but it just seems like a bad idea to not do this when we're already doing the work here | 10:38 |
stephenfin | I could help in the rework, but if I do then I can't review :/ | 10:38 |
stephenfin | ralonsoh_: So that is your call | 10:39 |
ralonsoh_ | ok, I'll abandond the patch for now | 10:39 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: Don't persist could-be-stale InstanceGroup fields in RequestSpec https://review.openstack.org/528226 | 10:39 |
stephenfin | ralonsoh_: I wouldn't, personally. Better to keep it as -W so we can see the reasoning when it's reworked | 10:40 |
stephenfin | But that's me | 10:41 |
ralonsoh_ | and then there will be another reason to refactor it again | 10:41 |
*** Tom-Tom has joined #openstack-nova | 10:42 | |
stephenfin | ralonsoh_: That hardly seems fair :( I'm not asking for nit-picky stuff here and I (hope) I've explained the rationale for the change as much as possible (plus offered to help where possible) | 10:43 |
*** damien_r has joined #openstack-nova | 10:44 | |
*** heyongli has quit IRC | 10:45 | |
*** heyongli has joined #openstack-nova | 10:46 | |
*** Tom-Tom has quit IRC | 10:46 | |
*** abhishekk has quit IRC | 10:49 | |
*** frank_wang has joined #openstack-nova | 10:53 | |
*** heyongli has quit IRC | 10:56 | |
*** priteau has joined #openstack-nova | 10:56 | |
*** heyongli has joined #openstack-nova | 10:56 | |
*** frank_wang has quit IRC | 10:59 | |
*** moshele has quit IRC | 11:01 | |
*** moshele has joined #openstack-nova | 11:01 | |
*** heyongli has quit IRC | 11:06 | |
*** zhurong has quit IRC | 11:06 | |
*** heyongli has joined #openstack-nova | 11:06 | |
*** Drankis_ has joined #openstack-nova | 11:12 | |
*** heyongli has quit IRC | 11:16 | |
*** heyongli has joined #openstack-nova | 11:16 | |
*** edmondsw has joined #openstack-nova | 11:20 | |
*** mvk has quit IRC | 11:23 | |
*** edmondsw has quit IRC | 11:24 | |
*** heyongli has quit IRC | 11:26 | |
*** heyongli has joined #openstack-nova | 11:27 | |
*** bjhuangr_ has quit IRC | 11:29 | |
*** bjhuangr has quit IRC | 11:29 | |
*** markvoelker has joined #openstack-nova | 11:31 | |
*** moshele has quit IRC | 11:32 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Modify PciDevice.uuid generation code https://review.openstack.org/530487 | 11:33 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Add an online migration for PciDevice.uuid https://review.openstack.org/530905 | 11:33 |
*** Drankis_ has quit IRC | 11:35 | |
*** moshele has joined #openstack-nova | 11:35 | |
*** nicolasbock has joined #openstack-nova | 11:36 | |
*** edmondsw has joined #openstack-nova | 11:36 | |
*** heyongli has quit IRC | 11:37 | |
*** heyongli has joined #openstack-nova | 11:37 | |
*** moshele has quit IRC | 11:40 | |
*** heyongli has quit IRC | 11:47 | |
*** heyongli has joined #openstack-nova | 11:47 | |
*** frank_wang has joined #openstack-nova | 11:48 | |
*** tbachman has quit IRC | 11:49 | |
*** damien_r has quit IRC | 11:54 | |
*** frank_wang has quit IRC | 11:54 | |
*** heyongli has quit IRC | 11:57 | |
*** heyongli has joined #openstack-nova | 11:57 | |
*** mvk has joined #openstack-nova | 11:59 | |
*** markvoelker has quit IRC | 12:05 | |
*** yikun has quit IRC | 12:05 | |
*** gszasz has quit IRC | 12:05 | |
*** yikun has joined #openstack-nova | 12:06 | |
*** smatzek has joined #openstack-nova | 12:07 | |
*** heyongli has quit IRC | 12:07 | |
*** heyongli has joined #openstack-nova | 12:08 | |
*** yikun has quit IRC | 12:09 | |
*** sambetts|afk is now known as sambetts | 12:09 | |
*** gcb has quit IRC | 12:09 | |
*** gcb has joined #openstack-nova | 12:11 | |
*** edmondsw has quit IRC | 12:15 | |
*** damien_r has joined #openstack-nova | 12:17 | |
*** heyongli has quit IRC | 12:18 | |
*** heyongli has joined #openstack-nova | 12:18 | |
*** gszasz has joined #openstack-nova | 12:22 | |
*** yamamoto has quit IRC | 12:24 | |
*** heyongli has quit IRC | 12:28 | |
*** gcb has quit IRC | 12:28 | |
*** heyongli has joined #openstack-nova | 12:28 | |
*** fragatina has quit IRC | 12:31 | |
*** gcb has joined #openstack-nova | 12:31 | |
*** fragatina has joined #openstack-nova | 12:32 | |
*** lucasagomes is now known as lucas-hungry | 12:34 | |
*** heyongli has quit IRC | 12:38 | |
*** heyongli has joined #openstack-nova | 12:38 | |
*** yangyapeng has quit IRC | 12:45 | |
*** yangyapeng has joined #openstack-nova | 12:46 | |
*** jangutter has joined #openstack-nova | 12:47 | |
*** heyongli has quit IRC | 12:48 | |
*** heyongli has joined #openstack-nova | 12:49 | |
*** sdague has joined #openstack-nova | 12:49 | |
*** Jack_Iv has joined #openstack-nova | 12:50 | |
*** yangyapeng has quit IRC | 12:50 | |
*** moshele has joined #openstack-nova | 12:52 | |
*** frank_wang has joined #openstack-nova | 12:52 | |
*** tbachman has joined #openstack-nova | 12:56 | |
*** yamamoto has joined #openstack-nova | 12:56 | |
*** yamamoto has quit IRC | 12:56 | |
*** frank_wang has quit IRC | 12:58 | |
*** catintheroof has joined #openstack-nova | 12:58 | |
*** heyongli has quit IRC | 12:59 | |
*** frank_wang has joined #openstack-nova | 12:59 | |
*** heyongli has joined #openstack-nova | 12:59 | |
*** catintheroof has quit IRC | 12:59 | |
*** catintheroof has joined #openstack-nova | 12:59 | |
*** tbachman has quit IRC | 13:00 | |
*** jaianshu__ has quit IRC | 13:01 | |
*** jaianshu__ has joined #openstack-nova | 13:01 | |
*** markvoelker has joined #openstack-nova | 13:02 | |
openstackgerrit | Silvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts https://review.openstack.org/522245 | 13:02 |
*** tbachman has joined #openstack-nova | 13:02 | |
*** frank_wang has quit IRC | 13:05 | |
*** heyongli has quit IRC | 13:09 | |
*** heyongli has joined #openstack-nova | 13:09 | |
*** mkoderer_4 has quit IRC | 13:10 | |
*** tpatzig_7 has quit IRC | 13:10 | |
*** mkoderer_ has quit IRC | 13:10 | |
*** tpatzig_ has quit IRC | 13:10 | |
*** carthaca_ has quit IRC | 13:10 | |
*** david_ has quit IRC | 13:10 | |
*** mkoderer_3 has joined #openstack-nova | 13:10 | |
*** dgonzalez_5 has joined #openstack-nova | 13:10 | |
*** tpatzig_2 has joined #openstack-nova | 13:10 | |
*** dgonzalez_3 has joined #openstack-nova | 13:10 | |
*** sapcc-bot1 has joined #openstack-nova | 13:10 | |
*** david_ has joined #openstack-nova | 13:10 | |
*** carthaca_ has joined #openstack-nova | 13:10 | |
*** tpatzig_ has joined #openstack-nova | 13:10 | |
*** mkoderer_ has joined #openstack-nova | 13:10 | |
*** dgonzalez_1 has joined #openstack-nova | 13:10 | |
*** tpatzig_1 has joined #openstack-nova | 13:10 | |
*** mkoderer_1 has joined #openstack-nova | 13:10 | |
*** carthaca_1 has joined #openstack-nova | 13:10 | |
*** tpatzig_1 has quit IRC | 13:12 | |
*** dgonzalez_1 has quit IRC | 13:12 | |
*** yangyapeng has joined #openstack-nova | 13:12 | |
*** mkoderer_1 has quit IRC | 13:12 | |
*** carthaca_1 has quit IRC | 13:12 | |
*** READ10 has joined #openstack-nova | 13:13 | |
*** edmondsw has joined #openstack-nova | 13:13 | |
*** dgonzalez_4 has quit IRC | 13:14 | |
*** dgonzalez_ has quit IRC | 13:14 | |
*** sapcc-bot has quit IRC | 13:14 | |
*** cleong has joined #openstack-nova | 13:15 | |
*** ratailor has quit IRC | 13:16 | |
*** yangyapeng has quit IRC | 13:17 | |
*** Jack_Iv has quit IRC | 13:18 | |
*** edmondsw has quit IRC | 13:18 | |
*** Jack_Iv has joined #openstack-nova | 13:19 | |
*** markvoelker has quit IRC | 13:19 | |
*** heyongli has quit IRC | 13:19 | |
*** heyongli has joined #openstack-nova | 13:19 | |
*** markvoelker has joined #openstack-nova | 13:20 | |
*** yangyapeng has joined #openstack-nova | 13:20 | |
*** diga has joined #openstack-nova | 13:21 | |
bhagyashris | ianw_pto: Hi, | 13:22 |
*** karthiks has quit IRC | 13:22 | |
openstackgerrit | Chen Hanxiao proposed openstack/nova master: manager: more detailed info of unsupported compute driver https://review.openstack.org/530701 | 13:23 |
*** pchavva has joined #openstack-nova | 13:24 | |
*** jaianshu__ has quit IRC | 13:26 | |
*** gszasz has quit IRC | 13:27 | |
*** yamamoto has joined #openstack-nova | 13:28 | |
*** heyongli has quit IRC | 13:29 | |
*** psachin has quit IRC | 13:29 | |
*** heyongli has joined #openstack-nova | 13:30 | |
*** lucas-hungry is now known as lucasagomes | 13:32 | |
*** yamamoto has quit IRC | 13:33 | |
*** jaypipes has joined #openstack-nova | 13:35 | |
*** karthiks has joined #openstack-nova | 13:36 | |
*** Drankis has joined #openstack-nova | 13:38 | |
*** gszasz has joined #openstack-nova | 13:39 | |
*** psachin has joined #openstack-nova | 13:40 | |
*** heyongli has quit IRC | 13:40 | |
*** heyongli has joined #openstack-nova | 13:40 | |
*** goutham has quit IRC | 13:41 | |
*** psachin has quit IRC | 13:43 | |
*** karthiks has quit IRC | 13:44 | |
*** karthiks has joined #openstack-nova | 13:45 | |
*** READ10 has quit IRC | 13:48 | |
*** heyongli has quit IRC | 13:50 | |
*** dave-mccowan has joined #openstack-nova | 13:50 | |
*** heyongli has joined #openstack-nova | 13:50 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move additional nova-net options https://review.openstack.org/530923 | 13:50 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Use new-style choice values https://review.openstack.org/530924 | 13:50 |
*** frank_wang has joined #openstack-nova | 13:51 | |
*** mriedem has joined #openstack-nova | 13:52 | |
*** Tom-Tom has joined #openstack-nova | 13:56 | |
openstackgerrit | Merged openstack/nova master: Add an online migration for BDM.uuid https://review.openstack.org/525599 | 13:56 |
*** READ10 has joined #openstack-nova | 13:56 | |
*** frank_wang has quit IRC | 13:57 | |
*** links has quit IRC | 13:57 | |
*** heyongli has quit IRC | 14:00 | |
*** Tom-Tom has quit IRC | 14:00 | |
*** heyongli has joined #openstack-nova | 14:00 | |
*** larainema has left #openstack-nova | 14:03 | |
*** lyan has joined #openstack-nova | 14:04 | |
*** purplerbot has quit IRC | 14:04 | |
*** purplerbot_ has joined #openstack-nova | 14:04 | |
*** purplerbot_ is now known as purplerbot__ | 14:04 | |
*** purplerbot__ is now known as purplerbot | 14:04 | |
*** fanzhang has quit IRC | 14:10 | |
*** fanzhang has joined #openstack-nova | 14:10 | |
*** purplerbot has quit IRC | 14:10 | |
*** heyongli has quit IRC | 14:10 | |
*** purplerbot has joined #openstack-nova | 14:10 | |
*** archit has joined #openstack-nova | 14:11 | |
*** heyongli has joined #openstack-nova | 14:11 | |
*** archit is now known as amodi | 14:11 | |
*** sdake has quit IRC | 14:12 | |
*** sdake has joined #openstack-nova | 14:13 | |
*** sdake has quit IRC | 14:13 | |
*** sdake has joined #openstack-nova | 14:13 | |
*** diga has quit IRC | 14:16 | |
*** mriedem has quit IRC | 14:17 | |
*** heyongli has quit IRC | 14:21 | |
*** heyongli has joined #openstack-nova | 14:21 | |
*** hongbin has joined #openstack-nova | 14:21 | |
*** smatzek has quit IRC | 14:24 | |
*** smatzek has joined #openstack-nova | 14:25 | |
*** yamamoto has joined #openstack-nova | 14:28 | |
*** lyan has quit IRC | 14:28 | |
*** lyan has joined #openstack-nova | 14:29 | |
*** smatzek has quit IRC | 14:29 | |
*** heyongli has quit IRC | 14:31 | |
*** heyongli has joined #openstack-nova | 14:31 | |
*** yamamoto has quit IRC | 14:32 | |
jaypipes | stephenfin: so is there something funky going on with zuul? I see a lot of -2V's around a dependent change failing to merge and then rechecks right after... | 14:33 |
stephenfin | jaypipes: I'd expect to see the -2s if a dependant, +W'd patch failed to merge. Those errors do seem more frequent than they should though | 14:34 |
jaypipes | well, I've noticed *you* recheck'ing a lot after those -2s (and before any other patches are merged/pushed/rebased for those series) which is why I was asking you what was up :) | 14:35 |
stephenfin | Aha. Nope, they're dumb rechecks. The failures are all timeouts and unrelated to the patches in question | 14:36 |
*** edmondsw has joined #openstack-nova | 14:37 | |
*** kuzko has quit IRC | 14:40 | |
*** mriedem has joined #openstack-nova | 14:41 | |
*** heyongli has quit IRC | 14:41 | |
*** heyongli has joined #openstack-nova | 14:41 | |
ildikov | mriedem: hi | 14:44 |
mriedem | hi | 14:44 |
ildikov | mriedem: just a quick note that stvnoyes is helping out with the tempest tests for multi-attach | 14:44 |
mriedem | i pushed up the tempest multiattach patch yesterday after a rebase, i basically had to start over | 14:45 |
mriedem | https://review.openstack.org/#/c/266605/ | 14:45 |
* johnthetubaguy peaks over the wall to see how people are doing | 14:45 | |
ildikov | mriedem: yeah, I saw your ping this morning | 14:45 |
*** yamamoto has joined #openstack-nova | 14:46 | |
*** yamamoto has quit IRC | 14:46 | |
ildikov | mriedem: I pointed Steve to the patch yesterday, I didn't know you're planning to work on it | 14:46 |
mriedem | it still needs some additional test coverage, and needs to actually pass first | 14:48 |
*** frank_wang has joined #openstack-nova | 14:48 | |
ildikov | we can chat about it tomorrow on the meeting | 14:48 |
ildikov | I will try to catch Steve later today | 14:49 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Modify PciDevice.uuid generation code https://review.openstack.org/530487 | 14:50 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Add an online migration for PciDevice.uuid https://review.openstack.org/530905 | 14:50 |
*** heyongli has quit IRC | 14:51 | |
*** heyongli has joined #openstack-nova | 14:52 | |
*** jobewan has joined #openstack-nova | 14:52 | |
*** frank_wang has quit IRC | 14:54 | |
*** cz2 has joined #openstack-nova | 14:55 | |
*** karthiks has quit IRC | 14:56 | |
*** awaugama has joined #openstack-nova | 14:58 | |
*** eharney has joined #openstack-nova | 14:59 | |
*** heyongli has quit IRC | 15:02 | |
*** heyongli has joined #openstack-nova | 15:02 | |
*** trinaths has joined #openstack-nova | 15:03 | |
*** frank_wang has joined #openstack-nova | 15:03 | |
*** frank_wang has quit IRC | 15:03 | |
stephenfin | jaypipes: Think this is something you could tackle before the end of the cycle? https://review.openstack.org/#/q/status:open+project:openstack/nova+topic:bp/websocket-proxy-to-host-security | 15:03 |
jaypipes | stephenfin: definitely not my area of expertise, but I will review just because you asked nicely. | 15:04 |
stephenfin | sdague had volunteered to take the blow at the PTG, but he's not about much these days. It's been about since...Kilo and would be good to finally close out | 15:04 |
stephenfin | jaypipes: Excellent. Appreciate it :) | 15:04 |
* stephenfin needs to start picking up features that are in people's areas of expertise, heh | 15:05 | |
*** liverpooler has joined #openstack-nova | 15:05 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add test for assignment of uuid to a deleted BDM https://review.openstack.org/530774 | 15:06 |
*** Apoorva has joined #openstack-nova | 15:06 | |
*** sridharg has quit IRC | 15:10 | |
*** smatzek has joined #openstack-nova | 15:10 | |
*** heyongli has quit IRC | 15:12 | |
*** heyongli has joined #openstack-nova | 15:12 | |
*** udesale has quit IRC | 15:13 | |
*** mlavalle has joined #openstack-nova | 15:14 | |
*** armax has joined #openstack-nova | 15:19 | |
mriedem | stephenfin: on that series, there is this devstack change https://review.openstack.org/#/c/396186/ | 15:21 |
mriedem | stephenfin: is tempest already running a test to exercise any of this code? | 15:21 |
mriedem | i thought at one point danpb had a devstack-gate or project-config change to one of the jobs to enable this | 15:21 |
*** karthiks has joined #openstack-nova | 15:21 | |
mriedem | this https://review.openstack.org/#/c/396289/ | 15:22 |
stephenfin | If you're running under libvirt then some of it is exercised | 15:22 |
*** heyongli has quit IRC | 15:22 | |
*** heyongli has joined #openstack-nova | 15:22 | |
stephenfin | You're referring to this job, I imagine? https://review.openstack.org/#/c/396289/ | 15:22 |
*** armax has quit IRC | 15:23 | |
mriedem | i see melwitt moved that to a zuulv3 job definition within nova https://review.openstack.org/#/c/513160/4 | 15:23 |
mriedem | however, the nova-next job is still in the openstack-zuul-jobs repo | 15:23 |
stephenfin | The stuff that's tested is the basic negotiation. The VeNCrypt scheme is untested without that job by melwitt | 15:24 |
mriedem | so, can we put some effort into getting the new stuff tested before landing the series? | 15:24 |
stephenfin | I had patches up to move that, I thought? | 15:24 |
mriedem | you had patches up to move like all jobs into the nova repo, or something close, | 15:24 |
mriedem | and i'd like to avoid doing a giant job migration right now | 15:24 |
*** links has joined #openstack-nova | 15:25 | |
mriedem | because, e.g. pike is blocked on the tempest-full job migration for master | 15:25 |
mriedem | so maybe we start with moving the nova-next job into the nova repo, | 15:25 |
mriedem | then tweak it on top with mel's change to flip the new bit in devstack | 15:26 |
mriedem | the NOVA_CONSOLE_PROXY_COMPUTE_TLS variable | 15:26 |
mriedem | would have been easier if https://review.openstack.org/#/c/513160/ was on the same topic branch as the rest of the series | 15:26 |
stephenfin | Hmm, I just moved nova-lvm and nova-wsgi https://review.openstack.org/#/c/514309/ I don't think nova-next was supposed to move... | 15:27 |
*** devananda has joined #openstack-nova | 15:27 | |
stephenfin | But I need to investigate. Leave me with it | 15:27 |
stephenfin | Though, even still, the first two patches _can_ go in. They're tested using existing code | 15:27 |
stephenfin | As seen by the many CI failures I'd to work through, heh | 15:28 |
mriedem | well, an easier short-term solution is just push a patch on top of https://review.openstack.org/#/c/396186/ which sets the variable to True in devstack | 15:28 |
mriedem | btw, not sure why we need the nova-wsgi job in master - everything is running wsgi by default since pike | 15:30 |
mriedem | so that nova-wsgi job is only useful while working up to wsgi support in pike, right? | 15:30 |
*** links has quit IRC | 15:30 | |
stephenfin | We don't. I kept it in there so we can backport all the way to Pike. It's removed in a follow-up https://review.openstack.org/#/c/527140/ | 15:30 |
mriedem | but pike also runs wsgi by default in devstack | 15:30 |
mriedem | so it seems we just don't need it at all | 15:31 |
stephenfin | Ocata? | 15:31 |
mriedem | we don't support wsgi mode in ocata | 15:31 |
stephenfin | Oh, well then | 15:31 |
mriedem | let me make sure i'm not blowing smoke first | 15:31 |
mriedem | https://github.com/openstack-dev/devstack/blob/stable/pike/lib/nova#L79 | 15:32 |
*** heyongli has quit IRC | 15:32 | |
*** moshele has quit IRC | 15:32 | |
*** heyongli has joined #openstack-nova | 15:33 | |
stephenfin | It's in Ocata too though https://github.com/openstack-dev/devstack/blob/stable/ocata/lib/nova#L69 | 15:33 |
stephenfin | and set to False | 15:33 |
mriedem | i think that's for grenade | 15:33 |
mriedem | well, | 15:33 |
mriedem | https://github.com/openstack-dev/devstack/commit/d5537c1dc835413f1911ab797e3007d85322eace | 15:34 |
mriedem | that was added when dims added experimental support for running nova-api under wsgi long ago, | 15:34 |
mriedem | but we don't support it, and with cellsv2 stuff in ocata, the api literally doesn't work in that omde | 15:34 |
mriedem | *mode | 15:34 |
openstackgerrit | Chen Hanxiao proposed openstack/nova master: manager: more detailed info of unsupported compute driver https://review.openstack.org/530701 | 15:34 |
mriedem | which tripleo found out the hard way | 15:34 |
mriedem | https://github.com/openstack/nova/blob/stable/ocata/nova/wsgi/nova-api.py#L15 | 15:34 |
mriedem | https://github.com/openstack/nova/commit/e846c32ce3c3aee2cd83dec7561dc14cb3f0ada8 | 15:34 |
*** tbachman has quit IRC | 15:34 | |
*** tidwellr has joined #openstack-nova | 15:36 | |
mriedem | stephenfin: commented on https://review.openstack.org/#/c/514309/ | 15:36 |
mriedem | i think you can just squash those two patches | 15:36 |
openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Support aggregate affinity filters/weighers https://review.openstack.org/529135 | 15:38 |
mriedem | here is that devstack patch to run with the tls stuff https://review.openstack.org/530136 | 15:39 |
mriedem | err this one i mean https://review.openstack.org/530950 | 15:40 |
*** Drankis has quit IRC | 15:40 | |
*** heyongli has quit IRC | 15:43 | |
*** heyongli has joined #openstack-nova | 15:43 | |
*** gmann has quit IRC | 15:43 | |
*** hrw has joined #openstack-nova | 15:46 | |
hrw | morning | 15:46 |
*** yamamoto has joined #openstack-nova | 15:46 | |
mriedem | seems we shouldn't need to trace on an expired token... http://logs.openstack.org/60/513160/4/check/tempest-dsvm-neutron-nova-next-full/6458405/logs/screen-n-novnc.txt.gz?level=TRACE#_Dec_08_23_14_32_787200 | 15:47 |
hrw | can someone tell me does https://bugs.linaro.org/attachment.cgi?id=905 have sense? On AArch64 cpu_mode = host-model results in "libvirtError: unsupported configuration: CPU mode 'host-model' for aarch64 kvm domain on aarch64 host is not supported by hypervisor" and instance is not started. | 15:48 |
hrw | if I set cpu_mode = 'host-passthrough' in [libvirt] section of nova.conf then it works. | 15:48 |
hrw | but I would prefer to have it as default rather than work around it in every deploy method | 15:49 |
stephenfin | mriedem: All the above sounds good. I'll squash those patches and and drop that log.exception to...log.debug, I guess | 15:50 |
mriedem | stephenfin: i don't think that logging is coming from nova | 15:50 |
mriedem | i think it's coming from websockify | 15:50 |
stephenfin | I think this is it https://github.com/openstack/nova/blob/master/nova/console/websocketproxy.py#L156-L157 | 15:52 |
mriedem | right that's raising the exception | 15:52 |
stephenfin | Ohh, gotcha | 15:52 |
mriedem | and this is catching it https://github.com/novnc/websockify/blob/v0.8.0/websockify/websocket.py#L938 | 15:52 |
*** heyongli has quit IRC | 15:53 | |
stephenfin | Guess we could could configure | 15:53 |
*** heyongli has joined #openstack-nova | 15:53 | |
stephenfin | *websocket proxy's logger in DevStack | 15:53 |
stephenfin | *websockify's logger | 15:53 |
ericyoung | with queens-3 coming up... can someone review https://review.openstack.org/#/c/407440/ | 15:54 |
*** yamamoto has quit IRC | 15:54 | |
*** armax has joined #openstack-nova | 15:57 | |
*** imacdonn has quit IRC | 15:58 | |
*** imacdonn has joined #openstack-nova | 15:58 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309 | 15:59 |
*** heyongli has quit IRC | 16:03 | |
*** heyongli has joined #openstack-nova | 16:03 | |
*** burt has joined #openstack-nova | 16:04 | |
openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Support aggregate affinity filters/weighers https://review.openstack.org/529135 | 16:07 |
*** dims has quit IRC | 16:09 | |
openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: Deprecates support for Windows / Hyper-V Server 2012 https://review.openstack.org/530953 | 16:10 |
*** dims has joined #openstack-nova | 16:10 | |
*** heyongli has quit IRC | 16:13 | |
*** heyongli has joined #openstack-nova | 16:14 | |
*** artom has joined #openstack-nova | 16:14 | |
*** trinaths has quit IRC | 16:16 | |
*** Jack_Iv has quit IRC | 16:18 | |
melwitt | mriedem: hey, thanks for all the reviews on consoles | 16:19 |
*** openstackgerrit has quit IRC | 16:19 | |
mriedem | stephenfin: couple questions inline https://review.openstack.org/#/c/514309/ | 16:21 |
*** heyongli has quit IRC | 16:24 | |
*** heyongli has joined #openstack-nova | 16:24 | |
*** dave-mccowan has quit IRC | 16:24 | |
*** gouthamr has joined #openstack-nova | 16:24 | |
*** mvk has quit IRC | 16:24 | |
*** lpetrut has joined #openstack-nova | 16:24 | |
*** dave-mccowan has joined #openstack-nova | 16:25 | |
stephenfin | mriedem: All done | 16:26 |
stephenfin | tl;dr: It's a copy-paste patch. The real work happens once it's in-tree | 16:26 |
stephenfin | mordred won't know what hit him | 16:26 |
mriedem | ok just waiting for the new job to hit the experimental queue | 16:28 |
*** sshwarts has quit IRC | 16:33 | |
mordred | stephenfin: uhoh | 16:33 |
*** heyongli has quit IRC | 16:34 | |
*** heyongli has joined #openstack-nova | 16:34 | |
*** pcaruana has quit IRC | 16:35 | |
*** tidwellr_ has joined #openstack-nova | 16:36 | |
*** tidwellr has quit IRC | 16:36 | |
stephenfin | mordred: Just more work for you that you won't have time for. Nothing to see here :) | 16:36 |
mordred | stephenfin: \o/ | 16:37 |
mordred | stephenfin: my favorite!!! | 16:37 |
dims | hey mriedem, did you see the OpenStack++ related papers? (ops / edge computing) they have details on how they extended nova for better live migration scenarios | 16:37 |
*** mvk has joined #openstack-nova | 16:37 | |
dims | mriedem : [8][9] at https://etherpad.openstack.org/p/2017_edge_computing_working_sessions_existing_gaps | 16:37 |
dims | LOL stephenfin. right on the money :) | 16:38 |
melwitt | mriedem: do you know if we have a cells meeting today? | 16:38 |
*** openstackgerrit has joined #openstack-nova | 16:38 | |
openstackgerrit | Stephen Finucane proposed openstack/os-vif master: zuul: Enable functional tests in gate https://review.openstack.org/530961 | 16:38 |
stephenfin | ralonsoh_: ^ | 16:39 |
mriedem | dims: nope haven't seen it | 16:39 |
mriedem | heads down until feature freeze | 16:39 |
mriedem | melwitt: i assume so | 16:39 |
stephenfin | ralonsoh_: Let's see how them functional tests fare. I'm assuming you don't need a full-fledged DevStack environment to test them? | 16:39 |
ralonsoh_ | stephenfin: not in os-vif, just simple command checks | 16:40 |
*** hemna_ has joined #openstack-nova | 16:40 | |
openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Use UEFI as the default boot for AArch64 https://review.openstack.org/489951 | 16:41 |
stephenfin | ralonsoh_: Sweet. That should test it so, I hope. If it does, I might squash it into the main patch | 16:41 |
ralonsoh_ | stephenfin: which patch? | 16:42 |
stephenfin | ralonsoh_: Oh, sorry https://review.openstack.org/#/c/530961/ | 16:42 |
*** moshele has joined #openstack-nova | 16:42 | |
ralonsoh_ | stephenfin: I mean, you said you need to squash this patch with othe one | 16:43 |
*** mvk has quit IRC | 16:43 | |
stephenfin | The preceding one https://review.openstack.org/#/c/484386/30 | 16:43 |
stephenfin | i.e. the one that adds said functional tests | 16:43 |
ralonsoh_ | stephenfin: I'll rebase mine to your patch | 16:43 |
stephenfin | No, you need functional tests first | 16:44 |
stephenfin | So they need to go in in that order or together | 16:44 |
*** heyongli has quit IRC | 16:44 | |
jaypipes | dims: ugh. I hate when academics invent some new term like "cloudlet" and act like it's an industry-standard term. | 16:44 |
ralonsoh_ | stephenfin: perfect | 16:44 |
*** heyongli has joined #openstack-nova | 16:44 | |
stephenfin | ralonsoh_: Should wait and see if I've done it properly first too :) | 16:44 |
ralonsoh_ | stephenfin: I'll wait for the result | 16:45 |
dims | jaypipes : y once you get past some interesting stuff in there | 16:45 |
dims | jaypipes : they have to justify grants :) | 16:45 |
jaypipes | dims: there's some flat-out incorrect stuff in there, too. :) | 16:45 |
dims | yep. i was focused on how/what they changed in Nova | 16:46 |
openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965 | 16:47 |
hrw | I hope I did that right | 16:47 |
*** stvnoyes has joined #openstack-nova | 16:47 | |
jaypipes | dims: this picture perfectly represents the coherence of most edge-computing literature. | 16:48 |
jaypipes | dims: https://i.pinimg.com/originals/d7/cc/31/d7cc31df947e74fc98942bcdedb1de31.png | 16:48 |
dims | LOLOLOL | 16:48 |
ildikov | LOL | 16:49 |
ildikov | jaypipes: I thought cloudlet is an older one | 16:50 |
jaypipes | cloudlet. it's like saying a banana is a yellow orange. | 16:50 |
ildikov | you mean a lemon? | 16:52 |
ildikov | to which we Hungarians would say that it's a little yellow, a little sour, but it's ours... :) | 16:53 |
mriedem | https://www.youtube.com/watch?v=FT7YZQEHHVE | 16:53 |
*** moshele has quit IRC | 16:54 | |
*** heyongli has quit IRC | 16:54 | |
*** heyongli has joined #openstack-nova | 16:55 | |
jaypipes | ildikov: :) | 16:55 |
*** catintheroof has quit IRC | 16:55 | |
*** mvk has joined #openstack-nova | 16:55 | |
*** gyee has joined #openstack-nova | 16:58 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't log a warning for InstanceNotFound in detach_interface https://review.openstack.org/530969 | 16:59 |
*** fragatina has quit IRC | 17:00 | |
*** Apoorva has quit IRC | 17:01 | |
*** fragatina has joined #openstack-nova | 17:01 | |
*** heyongli has quit IRC | 17:05 | |
*** heyongli has joined #openstack-nova | 17:05 | |
*** felipemonteiro has joined #openstack-nova | 17:06 | |
*** ragiman has quit IRC | 17:07 | |
openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Use UEFI as the default boot for AArch64 https://review.openstack.org/489951 | 17:09 |
stephenfin | claudiub: Mind taking a look at this? https://review.openstack.org/#/c/487405/ | 17:09 |
hrw | stephenfin: thanks for review. updated commit message and fixed that ':' I skipped | 17:10 |
*** kaisers has quit IRC | 17:10 | |
stephenfin | hrw: Sweet. Still need a bug report though :) | 17:11 |
hrw | stephenfin: can be done ;d | 17:11 |
stephenfin | i.e. why this is necessary | 17:11 |
stephenfin | Cheers (y) | 17:11 |
hrw | https://bugs.launchpad.net/nova/+bug/1740824 already exists... | 17:12 |
hrw | will add to commit | 17:12 |
openstack | Launchpad bug 1740824 in OpenStack Compute (nova) "Make UEFI as the default properties for AArch64" [Undecided,New] | 17:12 |
stephenfin | hrw: Cool. If you were feeling generous, a little more info in that bug would be appreciated. Why is it essential, etc. | 17:13 |
hrw | sure | 17:13 |
openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Use UEFI as the default boot for AArch64 https://review.openstack.org/489951 | 17:15 |
*** heyongli has quit IRC | 17:15 | |
*** heyongli has joined #openstack-nova | 17:15 | |
*** kaisers has joined #openstack-nova | 17:15 | |
*** smatzek has quit IRC | 17:16 | |
*** yamamoto has joined #openstack-nova | 17:16 | |
*** smatzek has joined #openstack-nova | 17:17 | |
hrw | stephenfin: added comment to the bug | 17:19 |
*** smatzek_ has joined #openstack-nova | 17:19 | |
hrw | stephenfin: please read and tell me is it enough or should I expand it | 17:19 |
claudiub | stephenfin: sure, will do. and btw, there are plenty of OVS users on Windows. :) We've added OVS vif plugging support in nova some time ago. I think it was Mitaka. | 17:19 |
claudiub | or newton | 17:19 |
hrw | Mitaka... my favourite openstack release. | 17:19 |
hrw | due to uefi support ;) | 17:19 |
stephenfin | claudiub: Ah, so I wonder how come we're not using os-vif code for that? | 17:20 |
claudiub | stephenfin: we are | 17:20 |
*** smatzek__ has joined #openstack-nova | 17:20 | |
claudiub | stephenfin: https://github.com/openstack/nova/blob/master/nova/virt/hyperv/vif.py#L88 | 17:20 |
stephenfin | claudiub: I'm confused. How does that work with the code that's there. Surely that's the whole point of that patch? | 17:20 |
* stephenfin clicks | 17:20 | |
*** yamamoto has quit IRC | 17:21 | |
stephenfin | Ahhhh... http://git.openstack.org/cgit/openstack/compute-hyperv/tree/compute_hyperv/nova/vif.py | 17:21 |
*** smatzek has quit IRC | 17:21 | |
*** moshele has joined #openstack-nova | 17:23 | |
*** mdnadeem has quit IRC | 17:23 | |
*** smatzek_ has quit IRC | 17:23 | |
*** smatzek__ has quit IRC | 17:24 | |
*** smatzek has joined #openstack-nova | 17:25 | |
*** heyongli has quit IRC | 17:25 | |
*** heyongli has joined #openstack-nova | 17:25 | |
stephenfin | hrw: So to summarize, if you don't set that property, the instance doesn't boot? | 17:26 |
*** gszasz has quit IRC | 17:26 | |
hrw | stephenfin: unless it is setup to boot kernel+initrd which no one does nowadays | 17:26 |
stephenfin | hrw: Cool. +2d | 17:27 |
melwitt | uh, what's happened to tempest job logs? can't filter by log level anymore | 17:27 |
hrw | stephenfin: cool! thanks | 17:28 |
*** yamamoto has joined #openstack-nova | 17:29 | |
hrw | now have to find someone else to take a look ;D | 17:29 |
claudiub | stephenfin: hm, I think the main point of the patch is to make os-vif a bit more OS agnostic. If you look in the vif_plug_ovs.ovs module, linux_net.device_exists is being used, even though that is also executed on Windows | 17:29 |
claudiub | and it also adds a Windows specific implementation for device_exists, which is nice. | 17:30 |
openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: Deprecates support for Windows / Hyper-V Server 2012 https://review.openstack.org/530953 | 17:31 |
*** devananda has quit IRC | 17:31 | |
*** slunkad has quit IRC | 17:32 | |
hrw | stephenfin: can you look at https://review.openstack.org/#/c/530965/ one too? It is probably wrong way of doing it and I am open for suggestions | 17:32 |
*** Jack_Iv has joined #openstack-nova | 17:32 | |
*** yamamoto has quit IRC | 17:34 | |
*** jangutter has quit IRC | 17:35 | |
*** heyongli has quit IRC | 17:35 | |
*** cdent has quit IRC | 17:35 | |
*** heyongli has joined #openstack-nova | 17:36 | |
*** cdent has joined #openstack-nova | 17:36 | |
*** tidwellr has joined #openstack-nova | 17:36 | |
*** tidwellr_ has quit IRC | 17:36 | |
*** tesseract has quit IRC | 17:38 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530980 | 17:39 |
*** lucasagomes is now known as lucas-afk | 17:42 | |
*** mvk has quit IRC | 17:43 | |
ralonsoh_ | stephenfin: I don't know why it's failing | 17:45 |
ralonsoh_ | stephenfin: I can't reproduce the error | 17:45 |
*** heyongli has quit IRC | 17:46 | |
*** heyongli has joined #openstack-nova | 17:46 | |
*** smatzek has quit IRC | 17:47 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530981 | 17:48 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530982 | 17:49 |
*** smatzek has joined #openstack-nova | 17:51 | |
*** smatzek has quit IRC | 17:51 | |
*** smatzek has joined #openstack-nova | 17:51 | |
mriedem | stephenfin: https://review.openstack.org/#/c/514309/ is fine except it has to be renamed nova-lvm i guess | 17:54 |
*** Apoorva has joined #openstack-nova | 17:55 | |
*** heyongli has quit IRC | 17:56 | |
*** Apoorva has quit IRC | 17:56 | |
*** heyongli has joined #openstack-nova | 17:56 | |
*** andrewbogott has joined #openstack-nova | 17:57 | |
*** lpetrut has quit IRC | 17:57 | |
mriedem | mmedvede: are you powerkvm ci? | 17:59 |
*** liusheng has quit IRC | 17:59 | |
*** Swami has joined #openstack-nova | 17:59 | |
*** yamamoto has joined #openstack-nova | 17:59 | |
mriedem | yeah; https://review.openstack.org/#/c/530982/ on stable/newton failed but +1ed the change | 17:59 |
mmedvede | mriedem: I maintain kvm on power CI, yes | 17:59 |
mriedem | should we just not run pkvm ci on newton changes? | 17:59 |
mriedem | that branch is nearly eol | 18:00 |
*** derekh has quit IRC | 18:00 | |
mriedem | 2018-01-03 17:51:43.289 | error: pathspec 'stable/newton' did not match any file(s) known to git. | 18:00 |
*** liusheng has joined #openstack-nova | 18:00 | |
*** tbachman has joined #openstack-nova | 18:00 | |
*** Apoorva has joined #openstack-nova | 18:00 | |
mmedvede | mriedem: there is actually a fix for that in the pipeline, let me dig it out | 18:01 |
mmedvede | it affects all non-zuulv3 third-party CIs | 18:01 |
*** Apoorva has quit IRC | 18:02 | |
*** Apoorva_ has joined #openstack-nova | 18:02 | |
mmedvede | mriedem: https://review.openstack.org/#/c/523925/ | 18:03 |
*** Jack_Iv has quit IRC | 18:03 | |
*** yamamoto has quit IRC | 18:04 | |
mmedvede | the CI failing but still +1 the change is due to there being only non-voting job in the set | 18:05 |
mriedem | i feel like i recently made a similar change to devstack in newton | 18:05 |
mriedem | ah yes https://review.openstack.org/#/c/519758/ | 18:05 |
mriedem | and apparently it was my devstack-gate patch they are referencing | 18:06 |
mriedem | https://review.openstack.org/#/c/520095/ | 18:06 |
*** heyongli has quit IRC | 18:06 | |
*** heyongli has joined #openstack-nova | 18:06 | |
mriedem | sdague: you around for devstack-gate +2? https://review.openstack.org/#/c/523925/ | 18:07 |
mmedvede | d-g is accumulating lots of cruft due to having to support zuulv3/v2 at the same time :( | 18:07 |
mmedvede | mriedem: I'll watch the patch, if the fix would not land I'll disable our CI on stable/newton | 18:09 |
mriedem | ok | 18:11 |
*** yamamoto has joined #openstack-nova | 18:16 | |
*** yamamoto has quit IRC | 18:16 | |
*** heyongli has quit IRC | 18:16 | |
*** heyongli has joined #openstack-nova | 18:17 | |
*** yamahata has joined #openstack-nova | 18:21 | |
*** ChanServ sets mode: +r | 18:21 | |
*** heyongli has quit IRC | 18:27 | |
mriedem | cdent: you want to take a crack at a release note for this? https://review.openstack.org/#/c/448034/ | 18:28 |
cdent | mriedem: ya, thanks, will look, I don't know much about that one (just did the rebase) but should be able to do it up | 18:30 |
mriedem | stephenfin: seeing a TypeError in testing the websocket proxy series http://logs.openstack.org/50/530950/1/check/legacy-tempest-dsvm-py35/1dffae7/logs/screen-n-novnc.txt.gz?level=TRACE#_Jan_03_16_09_08_830407 | 18:31 |
mriedem | in the py35 job | 18:31 |
cdent | will check with rado first | 18:31 |
mriedem | huzzah, i've done my job https://review.openstack.org/#/c/345398/ | 18:33 |
melwitt | mriedem: weird, I wonder how the tempest py35 passed on my DNM patch but not with yours on the console proxy with TLS stuff | 18:37 |
mriedem | melwitt: because you were configuring one specific job | 18:38 |
mriedem | which doesn't use py35 | 18:38 |
melwitt | oh, right. derp | 18:38 |
* mriedem goes to feast on smoked pig carcass | 18:39 | |
*** pchavva has quit IRC | 18:43 | |
*** felipemonteiro has quit IRC | 18:44 | |
*** yamamoto has joined #openstack-nova | 18:46 | |
*** yamamoto has quit IRC | 18:54 | |
openstackgerrit | Chris Dent proposed openstack/nova master: VMware: add support for different firmwares https://review.openstack.org/448034 | 18:55 |
cdent | I had an amusing conversation with someone outside of openstack today about the complex bits of placement and scheduling and he said what many people have said before: maybe there needs to be an openstack for "give me a vm" and a different openstack for "I am a telco" | 18:56 |
jaypipes | cdent: bingo. | 18:57 |
cdent | I thought that might draw you out :) | 18:57 |
*** pchavva has joined #openstack-nova | 18:59 | |
melwitt | hah, I like that. openstack Telco Edition(tm) | 19:01 |
cdent | now including a super upper ontology | 19:01 |
ildikov | and it's only the 3rd of January... :) | 19:07 |
*** jhesketh has quit IRC | 19:08 | |
*** efried has quit IRC | 19:09 | |
hrw | guys: can you look at https://review.openstack.org/#/c/530965/ one? It is probably wrong way of doing it and I am open for suggestions | 19:10 |
*** jhesketh has joined #openstack-nova | 19:11 | |
*** armax has quit IRC | 19:14 | |
*** tidwellr has quit IRC | 19:19 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: VMware: add support for different firmwares https://review.openstack.org/448034 | 19:23 |
stvnoyes | mriedem: Hi Matt, I was talking to ildikov and she suggested I could help with https://review.openstack.org/#/c/266605 , the tempest multiattach test. I see you've been working on it recently. Any objections to me taking it on? | 19:25 |
*** dtruong_ has quit IRC | 19:26 | |
ildikov | stvnoyes: I mentioned it mriedem today that we talked about that test and that we can talk about the desired test coverage in Tempest on the meeting tomorrow | 19:31 |
mriedem | i'm debugging the failure now - can't create a multiattach volume | 19:32 |
ildikov | mriedem: it's disabled in Cinder | 19:32 |
ildikov | mriedem: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/lvm.py#L279 | 19:33 |
mriedem | Jan 03 14:42:03.502967 ubuntu-xenial-citycloud-sto2-0001666901 cinder-scheduler[12541]: DEBUG cinder.scheduler.filters.capabilities_filter [None req-ecbc7305-ea6f-438b-af55-028c5b6eeb03 tempest-AttachVolumeMultiAttachTest-1915307512 None] Volume type extra spec requirement "multiattach=<is> True" does not match reported capability "False" {{(pid=12541) _satisfies_extra_specs /opt/stack/cinder/cinder/scheduler/filters/capabili | 19:33 |
mriedem | _filter.py:74}} | 19:33 |
stvnoyes | @page { margin: 0.79in } pre { direction: ltr; text-align: left; orphans: 2; widows: 2 } pre.western { font-family: "Times New Roman", serif; font-size: 12pt } pre.cjk { font-family: "SimSun"; font-size: 12pt; so-language: zh-CN } pre.ctl { font-family: "Times New Roman"; font-size: 12pt } p { margin-bottom: 0.1in; direction: ltr; line-height: 120%; text-align: left; orphans: 2; widows: 2 } p.western { fo | 19:33 |
stvnoyes | 19:33 | |
stvnoyes | ugh! | 19:34 |
stvnoyes | - Need the lvm driver to support MA - in cinder/volume/drivers/lvm.py , line 280 > | 19:34 |
stvnoyes | multiattach = True | 19:34 |
stvnoyes | there are also issues in tempest detach, I'm working on that now | 19:35 |
ildikov | stvnoyes: what's the issue with detach? | 19:35 |
mriedem | tempest assumes that when you detach the volume will eventually be 'available' | 19:35 |
mriedem | but with multiattach it will go from in-use to in-use | 19:36 |
mriedem | until the last attachment is gone | 19:36 |
ildikov | stvnoyes: there's a whole chain to address shared_targets issues: https://review.openstack.org/#/c/529695/ | 19:36 |
stvnoyes | from what I see so far, at the end of the test it tries to detach, but when it detaches the vol from the first server, the vol status doesn't go to abailable (as we would expect) but the test wants to see available | 19:36 |
ildikov | however I'm not sure that's applicable for lvm... | 19:36 |
mriedem | ildikov: different issues | 19:36 |
mriedem | i didn't realize that cinder hard-coded this in all drivers across the board | 19:37 |
ildikov | ah ok, sorry, I thought actual issues and not things we need to update | 19:37 |
mriedem | that should have probably been a global policy change in the API | 19:37 |
stvnoyes | I think we will need to pass in an expected status to detach so it knows what success means | 19:37 |
ildikov | I think they also wanted to ensure that driver maintainers think about it for at least two seconds before turning it on | 19:37 |
mriedem | stvnoyes: not sure that is necessary - we can just count the attachments first, and if the volume has multiattach=True | 19:38 |
stvnoyes | y, that should work too | 19:38 |
stvnoyes | and is simpler... | 19:38 |
mriedem | ildikov: so is https://review.openstack.org/#/c/428365/ going to be reverted at the same time that a policy rule is put in place? | 19:39 |
mriedem | not even a release note on that change which effectively breaks the API | 19:39 |
ildikov | well, it was never really supported anyway... | 19:40 |
ildikov | I would think that we turn back the drivers more one-by-one | 19:40 |
mriedem | ok so i'll push a WIP cinder patch to turn this on for lvm to at least get some test coverage going | 19:41 |
ildikov | but I can double check it with jgriffith or someone who feels brave enough to answer that question... :) | 19:42 |
mriedem | i just need to move the testing along, so i'm ok for now | 19:42 |
*** yangyapeng has quit IRC | 19:44 | |
ildikov | mriedem: is there anything stvnoyes can help out or you're dedicated to make this one work? | 19:44 |
mriedem | i'm ok for now, wait until i'm stuck or don't have time to look | 19:45 |
*** pcaruana has joined #openstack-nova | 19:45 | |
stvnoyes | ok, that's fine. If there's something else I can help with, let me know. | 19:46 |
ildikov | mriedem: ok, cool, tnx | 19:46 |
mriedem | stvnoyes: have you read my comments in https://review.openstack.org/#/c/271047/ ? | 19:47 |
openstackgerrit | Chris Dent proposed openstack/nova master: Document lack of side-effects in AllocationList.create_all() https://review.openstack.org/530997 | 19:47 |
mriedem | could use more eyes on what, if any, restriction we're going to put in the compute api for multiattach and the new microversion | 19:47 |
stvnoyes | mriedem: i haven't looked at it yet. I'll take a look. | 19:47 |
*** awaugama has quit IRC | 19:48 | |
*** yamamoto has joined #openstack-nova | 19:56 | |
*** dave-mccowan has quit IRC | 19:56 | |
mriedem | oh nice, | 19:57 |
mriedem | http://paste.openstack.org/show/637525/ | 19:57 |
mriedem | trying to attach a volume to more than one instance results in a 500 | 19:58 |
*** pcaruana has quit IRC | 19:59 | |
*** sambetts is now known as sambetts|afk | 20:00 | |
mriedem | Unable to create attachment for volume (Invalid volume: Volume 373c2a29-f6e7-4931-adda-6b87a193e9f9 status must be available or downloadi | 20:01 |
ildikov | mriedem: for a multiattach volume? | 20:03 |
mriedem | no, not multiattach | 20:03 |
mriedem | i wanted to make sure that still is prohibited | 20:03 |
openstackgerrit | Chris Dent proposed openstack/nova master: Fix documentation nits in set_and_clear_allocations https://review.openstack.org/531001 | 20:04 |
ildikov | oh, ok :) | 20:04 |
mriedem | hmm, why would i get a 400 in the case of os-reserve but a 500 from cinder in the case of attachment_create? | 20:08 |
mriedem | i would think the ResourceExceptionHandler would be used in both cases | 20:10 |
mriedem | that converts InvalidVolume to a 400 response | 20:10 |
*** Jeffrey4l has quit IRC | 20:11 | |
*** damien_r has quit IRC | 20:14 | |
*** Jeffrey4l has joined #openstack-nova | 20:24 | |
*** mriedem has left #openstack-nova | 20:42 | |
*** mriedem has joined #openstack-nova | 20:42 | |
openstackgerrit | Lance Bragstad proposed openstack/nova master: Simplify logic in get_enforcer https://review.openstack.org/531008 | 20:44 |
*** elod has quit IRC | 20:46 | |
edleafe | mriedem: got a minute to talk about resize? | 20:55 |
mriedem | sure | 20:55 |
*** ociuhandu has joined #openstack-nova | 20:55 | |
*** eharney has quit IRC | 20:55 | |
edleafe | we recently changed it to do 'replace_allocation_with_migration' to have the allocations use the migration uuid | 20:56 |
*** ociuhandu has quit IRC | 20:56 | |
edleafe | It checks to verify that the instance is allocated against the source node, and if it isn't raises an error | 20:57 |
edleafe | But with claiming in the scheduler, select_destinations will claim against the target host | 20:57 |
edleafe | So the replace method always fails | 20:58 |
mriedem | edleafe: i can see that being true in the reschedule case, | 20:58 |
*** armax has joined #openstack-nova | 20:59 | |
edleafe | exactly | 20:59 |
mriedem | because on the 2nd loop through conductor, the source node allocation will be held by the migration uuid and the first chosen and failed host is owned by the instance | 20:59 |
edleafe | (I forgot to mention that I'm working on functional tests for reschedules | 20:59 |
mriedem | ah - i was going to start working on a functional test this afternoon but got sidetracked with cinder | 20:59 |
mriedem | edleafe: so, we should (1) report a bug for the thing and (2) fix it separately before your alternate hosts patch for resize i think | 21:00 |
edleafe | I've been working on it, but it took a while to track down exactly why they were all failing | 21:00 |
mriedem | i was going to push the functional test before your alternate hosts patch in the series | 21:00 |
mriedem | yeah we need the functional resize + reschedule test regardless of alternate hosts | 21:00 |
mriedem | because it would have caught this earlier | 21:00 |
edleafe | makes sense | 21:00 |
mriedem | cool | 21:00 |
edleafe | so have you started on it? Should I wait and rebase on top of yours? | 21:01 |
mriedem | i haven't started | 21:01 |
mriedem | so feel free to report the regression bug and push a patch in nova/tests/functional/regressions | 21:01 |
mriedem | for that bug | 21:01 |
edleafe | ok, I can do it. I only have a little time left today, thanks to IBM adding a 2-hour each way commute to my schedule | 21:02 |
*** hemna_ has quit IRC | 21:02 | |
mriedem | how about report the bug and push what you have for the test | 21:02 |
mriedem | can work on a fix and such tomorrow | 21:02 |
edleafe | ok, I can do that. | 21:02 |
mriedem | thanks | 21:02 |
mriedem | i should probably start thinking about an rc1 bugs etherpad... | 21:05 |
*** yamahata has quit IRC | 21:14 | |
*** Apoorva_ has quit IRC | 21:16 | |
openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle not found error on taking snapshot https://review.openstack.org/511074 | 21:20 |
*** cleong has quit IRC | 21:20 | |
*** pchavva has quit IRC | 21:21 | |
*** smatzek has quit IRC | 21:21 | |
*** edmondsw has quit IRC | 21:22 | |
*** READ10 has quit IRC | 21:22 | |
edleafe | mriedem: https://bugs.launchpad.net/nova/+bug/1741125 | 21:26 |
openstack | Launchpad bug 1741125 in OpenStack Compute (nova) "Instance resize always fails when rescheduling" [High,New] - Assigned to Ed Leafe (ed-leafe) | 21:26 |
mriedem | thanks; resize in general is probably just broken for the CachingScheduler | 21:27 |
mriedem | dansmith: we probably can't error out here https://github.com/openstack/nova/blob/f95f165b49fbc0efe29450b0e858a3ccadecedea/nova/conductor/tasks/migrate.py#L47-L48 | 21:31 |
mriedem | we won't have allocations if you're using the caching scheduler | 21:32 |
dansmith | hrm | 21:32 |
dansmith | and not create them either? that'll kinda screw with all the downstream logic for confirm/delete/undo-swap | 21:33 |
edleafe | if we do check, we should check against the instance or the migration | 21:33 |
mriedem | dansmith: compute won't create them once everything is pike+ | 21:33 |
dansmith | that's not what I meant, | 21:34 |
dansmith | but I guess we can't create the new allocations (on the dest) if we don't have some to copy from the source | 21:35 |
mriedem | edleafe: i think we can handle the logic in the conductor task based on whether or not we know we're doing a reschedule | 21:35 |
mriedem | edleafe: in the case of a reschedule, we'd swap the allocation from desthost1 to desthost2 for the migration uuid | 21:35 |
mriedem | and just leave the instance uuid allocation on the source node | 21:35 |
mriedem | not really a swap, just an update to the allocations for the migration consumer | 21:36 |
edleafe | yeah, I knew what you meant | 21:37 |
dansmith | reschedule of a migration is changing the instance uuid's allocation though not the migration one | 21:39 |
mriedem | oh right yeah, got those mixed up | 21:39 |
mriedem | migration stays on the source node, instance goes to the eventual dest host | 21:39 |
dansmith | so just need to update the instance's allocation for the new node | 21:40 |
dansmith | right | 21:40 |
melwitt | mriedem: what did you mean by your comment here? https://review.openstack.org/#/c/481700/7/nova/db/sqlalchemy/api.py@1867 update what, model_query? | 21:41 |
mriedem | the comment | 21:42 |
melwitt | oh, k | 21:42 |
*** gouthamr has quit IRC | 21:42 | |
mriedem | there is no 'deleted' in 'tags' or 'consoleauthtoken' tables | 21:42 |
melwitt | thanks | 21:43 |
*** hongbin has quit IRC | 21:46 | |
mriedem | stvnoyes: got further on the multiattach tempest test; it appears to be failing to create the 2nd attachment | 21:47 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | 21:52 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Add functional test for resize reschedules https://review.openstack.org/531022 | 21:52 |
edleafe | mriedem: ^^ resize reschedule func test | 21:52 |
mriedem | ok | 21:52 |
rybridges | Hey Guys! Had a really quick question. Is ephemeral GB included in snapshots for VMs? | 21:58 |
*** eharney has joined #openstack-nova | 22:04 | |
*** rcernin has joined #openstack-nova | 22:05 | |
*** priteau has quit IRC | 22:05 | |
*** burt has quit IRC | 22:42 | |
*** lyan has quit IRC | 22:43 | |
*** karthiks has quit IRC | 22:43 | |
*** moshele has quit IRC | 22:48 | |
clarkb | I am pretty sure only the root device is snapshotted | 22:58 |
clarkb | (but I'm saying that as a user of random clouds not a nova dev that would know for sure) | 22:58 |
*** cdent has quit IRC | 23:03 | |
*** ralonsoh_ has quit IRC | 23:03 | |
*** takashin has joined #openstack-nova | 23:09 | |
*** jdurgin has quit IRC | 23:14 | |
*** jdurgin has joined #openstack-nova | 23:18 | |
openstackgerrit | Merged openstack/nova master: Add test for assignment of uuid to a deleted BDM https://review.openstack.org/530774 | 23:19 |
openstackgerrit | Merged openstack/nova master: Add support for listing hosts in cellv2 https://review.openstack.org/524755 | 23:19 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: [libvirt] Allow multiple volume attachments https://review.openstack.org/267587 | 23:20 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: [api] Allow multi-attach in compute api https://review.openstack.org/271047 | 23:20 |
*** mlavalle has quit IRC | 23:26 | |
mriedem | stvnoyes: running the test again based on these changes to the libvirt patch ^ | 23:26 |
*** esberglu has quit IRC | 23:38 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!