*** shaohe_feng has quit IRC | 00:01 | |
*** shaohe_feng has joined #openstack-nova | 00:02 | |
*** yamamoto has joined #openstack-nova | 00:03 | |
*** yamamoto has quit IRC | 00:09 | |
*** shaohe_feng has quit IRC | 00:11 | |
*** shaohe_feng has joined #openstack-nova | 00:11 | |
*** bigdogstl has joined #openstack-nova | 00:12 | |
*** itlinux has joined #openstack-nova | 00:13 | |
*** bigdogstl has quit IRC | 00:16 | |
*** jmlowe has joined #openstack-nova | 00:19 | |
*** shaohe_feng has quit IRC | 00:21 | |
*** shaohe_feng has joined #openstack-nova | 00:23 | |
*** rnoriega_ has joined #openstack-nova | 00:28 | |
*** weshay_ has joined #openstack-nova | 00:28 | |
*** markmc` has joined #openstack-nova | 00:28 | |
*** kashyap` has joined #openstack-nova | 00:28 | |
*** markmc has quit IRC | 00:31 | |
*** rnoriega has quit IRC | 00:31 | |
*** kashyap has quit IRC | 00:31 | |
*** shaohe_feng has quit IRC | 00:32 | |
*** weshay has quit IRC | 00:32 | |
*** shaohe_feng has joined #openstack-nova | 00:33 | |
*** suresh12 has quit IRC | 00:39 | |
*** mriedem_afk has quit IRC | 00:40 | |
*** bigdogstl has joined #openstack-nova | 00:41 | |
*** suresh12 has joined #openstack-nova | 00:41 | |
*** yassine_ has quit IRC | 00:41 | |
*** markmc has joined #openstack-nova | 00:42 | |
*** kashyap has joined #openstack-nova | 00:42 | |
*** shaohe_feng has quit IRC | 00:42 | |
*** shaohe_feng has joined #openstack-nova | 00:42 | |
*** markmc` has quit IRC | 00:43 | |
*** rnoriega has joined #openstack-nova | 00:43 | |
*** kashyap` has quit IRC | 00:43 | |
*** weshay has joined #openstack-nova | 00:43 | |
*** weshay_ has quit IRC | 00:44 | |
*** rnoriega_ has quit IRC | 00:44 | |
*** bigdogstl has quit IRC | 00:46 | |
*** Dinesh_Bhor has joined #openstack-nova | 00:47 | |
*** eharney has quit IRC | 00:47 | |
*** chyka has joined #openstack-nova | 00:50 | |
*** mriedem has joined #openstack-nova | 00:51 | |
*** edmondsw has joined #openstack-nova | 00:51 | |
*** shaohe_feng has quit IRC | 00:52 | |
*** shaohe_feng has joined #openstack-nova | 00:53 | |
*** chyka has quit IRC | 00:55 | |
*** edmondsw has quit IRC | 00:55 | |
*** shaohe_feng has quit IRC | 01:02 | |
*** shaohe_feng has joined #openstack-nova | 01:03 | |
*** phuongnh has joined #openstack-nova | 01:04 | |
*** yamamoto has joined #openstack-nova | 01:06 | |
mriedem | Kevin_Zheng: fyi https://review.openstack.org/#/c/569386/ | 01:10 |
---|---|---|
*** harlowja has quit IRC | 01:10 | |
*** yamamoto has quit IRC | 01:11 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Ensure resource class cache when listing usages https://review.openstack.org/572652 | 01:12 |
*** xinliang has quit IRC | 01:12 | |
*** shaohe_feng has quit IRC | 01:13 | |
*** shaohe_feng has joined #openstack-nova | 01:13 | |
*** bigdogstl has joined #openstack-nova | 01:14 | |
*** bigdogstl has quit IRC | 01:19 | |
*** Nel1x has joined #openstack-nova | 01:20 | |
*** namnh has joined #openstack-nova | 01:22 | |
Kevin_Zheng | mriedem ack | 01:23 |
*** shaohe_feng has quit IRC | 01:23 | |
*** mdnadeem has joined #openstack-nova | 01:23 | |
Kevin_Zheng | mriedem: BTW, the max bulk boot BP will need further discussion? | 01:24 |
*** shaohe_feng has joined #openstack-nova | 01:24 | |
mriedem | Kevin_Zheng: https://review.openstack.org/#/c/510235/ ? yeah, it's a very rough work in progress | 01:25 |
mriedem | ideally i'd get some operator feedback before revising it | 01:25 |
mriedem | not really pushing to get that in for rocky | 01:26 |
Kevin_Zheng | OK, maybe we will take it over togother with the dettach root volume for next cycle | 01:26 |
mriedem | i think it's arguably also a bug fix / security hardening opportunity | 01:26 |
*** zhaochao has joined #openstack-nova | 01:28 | |
*** cdent has quit IRC | 01:32 | |
*** shaohe_feng has quit IRC | 01:33 | |
*** suresh12 has quit IRC | 01:33 | |
*** threestrands_ has joined #openstack-nova | 01:33 | |
*** shaohe_feng has joined #openstack-nova | 01:34 | |
*** threestrands has quit IRC | 01:36 | |
openstackgerrit | Merged openstack/nova-specs master: Rename spec to match blueprint https://review.openstack.org/568870 | 01:39 |
mriedem | lyarwood: i put some notes in my spec for you https://review.openstack.org/#/c/552078/ - i tried a readonly to readonly swap volume and didn't hit any failures | 01:40 |
*** shaohe_feng has quit IRC | 01:43 | |
*** shaohe_feng has joined #openstack-nova | 01:45 | |
*** gyee has quit IRC | 01:46 | |
*** nguyenhai93 has quit IRC | 01:47 | |
*** nguyenhai93 has joined #openstack-nova | 01:48 | |
*** tiendc has joined #openstack-nova | 01:48 | |
*** hongbin has joined #openstack-nova | 01:48 | |
*** bigdogstl has joined #openstack-nova | 01:50 | |
*** shaohe_feng has quit IRC | 01:54 | |
*** shaohe_feng has joined #openstack-nova | 01:55 | |
*** bigdogstl has quit IRC | 01:55 | |
*** yamahata has quit IRC | 01:59 | |
*** shaohe_feng has quit IRC | 02:04 | |
*** mriedem is now known as mriedem_away | 02:04 | |
*** shaohe_feng has joined #openstack-nova | 02:04 | |
pvc | h | 02:06 |
pvc | guys | 02:06 |
*** pvc has quit IRC | 02:06 | |
gmann | alex_xu: what you think of this - https://review.openstack.org/#/c/571335/1 | 02:07 |
*** yamamoto has joined #openstack-nova | 02:07 | |
alex_xu | gmann: I will check it soon | 02:08 |
gmann | thanks | 02:08 |
*** fragatina has quit IRC | 02:09 | |
*** antosh has joined #openstack-nova | 02:10 | |
*** yassine has joined #openstack-nova | 02:11 | |
*** yassine is now known as Guest79569 | 02:11 | |
*** yamamoto has quit IRC | 02:12 | |
*** s1061123 has quit IRC | 02:13 | |
*** shaohe_feng has quit IRC | 02:14 | |
*** shaohe_feng has joined #openstack-nova | 02:15 | |
*** s1061123 has joined #openstack-nova | 02:15 | |
openstackgerrit | tianhui proposed openstack/nova master: Fix bug to api-ref https://review.openstack.org/571375 | 02:16 |
*** s1061123 has quit IRC | 02:17 | |
*** s1061123 has joined #openstack-nova | 02:17 | |
openstackgerrit | Merged openstack/nova-specs master: trivial: Fix file permissions https://review.openstack.org/566064 | 02:19 |
*** hshiina has joined #openstack-nova | 02:21 | |
*** Guest79569 has quit IRC | 02:22 | |
*** Guest79569 has joined #openstack-nova | 02:22 | |
*** shaohe_feng has quit IRC | 02:24 | |
*** shaohe_feng has joined #openstack-nova | 02:25 | |
*** bigdogstl has joined #openstack-nova | 02:27 | |
*** antosh has quit IRC | 02:30 | |
openstackgerrit | Chen Hanxiao proposed openstack/nova master: sync_guest_time: use the proper errno https://review.openstack.org/572346 | 02:30 |
*** antosh has joined #openstack-nova | 02:30 | |
*** liverpooler has quit IRC | 02:31 | |
*** bigdogstl has quit IRC | 02:32 | |
*** shaohe_feng has quit IRC | 02:35 | |
*** shaohe_feng has joined #openstack-nova | 02:35 | |
*** antosh has quit IRC | 02:36 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 02:38 |
*** psachin has joined #openstack-nova | 02:39 | |
*** edmondsw has joined #openstack-nova | 02:39 | |
*** Tom-Tom has joined #openstack-nova | 02:40 | |
*** heyongli has joined #openstack-nova | 02:40 | |
*** fragatina has joined #openstack-nova | 02:41 | |
*** heyongli has quit IRC | 02:43 | |
*** heyongli has joined #openstack-nova | 02:44 | |
*** edmondsw has quit IRC | 02:44 | |
takashin | 02:45 | |
*** shaohe_feng has quit IRC | 02:45 | |
*** shaohe_feng has joined #openstack-nova | 02:46 | |
*** jmlowe has quit IRC | 02:47 | |
*** bigdogstl has joined #openstack-nova | 02:51 | |
*** suresh12 has joined #openstack-nova | 02:51 | |
openstackgerrit | Merged openstack/nova master: Add granular policy rules for /resource_classes* https://review.openstack.org/565578 | 02:51 |
*** tbachman has quit IRC | 02:52 | |
*** heyongli has quit IRC | 02:54 | |
*** heyongli has joined #openstack-nova | 02:54 | |
*** shaohe_feng has quit IRC | 02:55 | |
*** bigdogstl has quit IRC | 02:55 | |
openstackgerrit | Merged openstack/nova master: Add granular policy rules for resource providers inventories https://review.openstack.org/568666 | 02:55 |
openstackgerrit | Merged openstack/nova master: Add granular policy rules for usages https://review.openstack.org/568706 | 02:55 |
*** shaohe_feng has joined #openstack-nova | 02:56 | |
*** suresh12 has quit IRC | 02:56 | |
*** jmlowe has joined #openstack-nova | 02:56 | |
*** bigdogstl has joined #openstack-nova | 03:02 | |
*** heyongli has quit IRC | 03:04 | |
*** heyongli has joined #openstack-nova | 03:04 | |
*** shaohe_feng has quit IRC | 03:05 | |
*** shaohe_feng has joined #openstack-nova | 03:06 | |
*** bigdogstl has quit IRC | 03:07 | |
*** tbachman has joined #openstack-nova | 03:07 | |
*** yamamoto has joined #openstack-nova | 03:08 | |
*** artom has joined #openstack-nova | 03:09 | |
*** liuzz has joined #openstack-nova | 03:10 | |
*** fragatina has quit IRC | 03:10 | |
*** rajinir has quit IRC | 03:11 | |
*** bigdogstl has joined #openstack-nova | 03:11 | |
*** yamamoto has quit IRC | 03:14 | |
*** heyongli has quit IRC | 03:14 | |
*** heyongli has joined #openstack-nova | 03:14 | |
*** bigdogstl has quit IRC | 03:15 | |
*** shaohe_feng has quit IRC | 03:16 | |
*** bigdogstl has joined #openstack-nova | 03:16 | |
*** shaohe_feng has joined #openstack-nova | 03:17 | |
*** Guest79569 has quit IRC | 03:22 | |
*** Guest79569 has joined #openstack-nova | 03:23 | |
*** heyongli has quit IRC | 03:24 | |
*** heyongli has joined #openstack-nova | 03:25 | |
*** shaohe_feng has quit IRC | 03:26 | |
*** shaohe_feng has joined #openstack-nova | 03:27 | |
*** bigdogstl has quit IRC | 03:29 | |
*** pvc has joined #openstack-nova | 03:34 | |
*** antosh has joined #openstack-nova | 03:34 | |
*** heyongli has quit IRC | 03:35 | |
*** heyongli has joined #openstack-nova | 03:35 | |
*** shaohe_feng has quit IRC | 03:36 | |
*** shaohe_feng has joined #openstack-nova | 03:37 | |
*** bzhao__ has joined #openstack-nova | 03:38 | |
*** antosh has quit IRC | 03:39 | |
*** bigdogstl has joined #openstack-nova | 03:39 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Abort live migration in queue (1/2). https://review.openstack.org/568542 | 03:42 |
*** r-daneel has joined #openstack-nova | 03:42 | |
*** udesale has joined #openstack-nova | 03:44 | |
*** lpetrut has joined #openstack-nova | 03:44 | |
*** bigdogstl has quit IRC | 03:45 | |
*** heyongli has quit IRC | 03:45 | |
*** heyongli has joined #openstack-nova | 03:45 | |
*** shaohe_feng has quit IRC | 03:46 | |
*** shaohe_feng has joined #openstack-nova | 03:47 | |
*** bigdogstl has joined #openstack-nova | 03:48 | |
*** yamahata has joined #openstack-nova | 03:48 | |
*** bigdogstl has quit IRC | 03:53 | |
*** heyongli has quit IRC | 03:55 | |
*** heyongli has joined #openstack-nova | 03:55 | |
*** shaohe_feng has quit IRC | 03:57 | |
*** shaohe_feng has joined #openstack-nova | 03:57 | |
*** Nel1x has quit IRC | 03:57 | |
openstackgerrit | Merged openstack/python-novaclient master: Revert "Fix listing of instances above API max_limit" https://review.openstack.org/572539 | 03:59 |
*** bigdogstl has joined #openstack-nova | 04:00 | |
*** germs has quit IRC | 04:00 | |
*** germs has joined #openstack-nova | 04:00 | |
*** germs has joined #openstack-nova | 04:00 | |
*** Guest79569 has quit IRC | 04:01 | |
*** Tom-Tom has quit IRC | 04:01 | |
*** Tom-Tom has joined #openstack-nova | 04:01 | |
*** links has joined #openstack-nova | 04:02 | |
*** suresh12 has joined #openstack-nova | 04:03 | |
*** masuberu has quit IRC | 04:04 | |
*** vladikr has quit IRC | 04:04 | |
*** germs has quit IRC | 04:04 | |
*** bigdogstl has quit IRC | 04:04 | |
*** heyongli has quit IRC | 04:05 | |
*** artom has quit IRC | 04:05 | |
*** heyongli has joined #openstack-nova | 04:06 | |
*** vladikr has joined #openstack-nova | 04:06 | |
*** boris_42_ has quit IRC | 04:06 | |
*** shaohe_feng has quit IRC | 04:07 | |
*** shaohe_feng has joined #openstack-nova | 04:09 | |
*** takashin has quit IRC | 04:09 | |
*** bigdogstl has joined #openstack-nova | 04:10 | |
*** yamamoto has joined #openstack-nova | 04:10 | |
*** mvenesio has joined #openstack-nova | 04:13 | |
*** hongbin has quit IRC | 04:13 | |
*** Bhujay has joined #openstack-nova | 04:14 | |
*** lpetrut has quit IRC | 04:15 | |
*** yamamoto has quit IRC | 04:16 | |
*** heyongli has quit IRC | 04:16 | |
*** heyongli has joined #openstack-nova | 04:16 | |
*** Guest79569 has joined #openstack-nova | 04:16 | |
*** shaohe_feng has quit IRC | 04:17 | |
*** harlowja has joined #openstack-nova | 04:17 | |
*** abhishekk has joined #openstack-nova | 04:20 | |
*** shaohe_feng has joined #openstack-nova | 04:21 | |
pvc | hi | 04:25 |
pvc | how can i hide kvm for gpu ? | 04:25 |
pvc | so i can test it | 04:25 |
*** bigdogstl has quit IRC | 04:25 | |
*** heyongli has quit IRC | 04:26 | |
*** heyongli has joined #openstack-nova | 04:26 | |
*** shaohe_feng has quit IRC | 04:27 | |
*** edmondsw has joined #openstack-nova | 04:27 | |
pvc | gibi | 04:28 |
pvc | how can i hide the hypervisor | 04:28 |
pvc | ? | 04:28 |
*** shaohe_feng has joined #openstack-nova | 04:29 | |
*** edmondsw has quit IRC | 04:32 | |
melwitt | mriedem_away: fyi I just added your neutron new port binding API to a runway. I skipped "report CPU features as traits" because the last -1 was 7 days ago with no response from the author | 04:34 |
melwitt | naichuans: fyi your blueprint xenapi-image-handler-option-improvement has been added to a runway again | 04:35 |
*** pvc has quit IRC | 04:35 | |
melwitt | naichuans: also, feel free to add feedback about your experience with vgpu-rocky in runway at L186 https://etherpad.openstack.org/p/nova-runways-rocky as it's been moved out of a runway | 04:36 |
*** heyongli has quit IRC | 04:36 | |
*** heyongli has joined #openstack-nova | 04:36 | |
*** shaohe_feng has quit IRC | 04:38 | |
*** pvc has joined #openstack-nova | 04:38 | |
pvc | gibi | 04:38 |
*** myrat has joined #openstack-nova | 04:40 | |
*** myrat has quit IRC | 04:41 | |
*** myrat has joined #openstack-nova | 04:41 | |
*** shaohe_feng has joined #openstack-nova | 04:42 | |
*** myrat has quit IRC | 04:43 | |
*** myrat has joined #openstack-nova | 04:44 | |
*** sridharg has joined #openstack-nova | 04:45 | |
*** harlowja has quit IRC | 04:45 | |
*** heyongli has quit IRC | 04:46 | |
*** heyongli has joined #openstack-nova | 04:47 | |
*** shaohe_feng has quit IRC | 04:48 | |
*** shaohe_feng has joined #openstack-nova | 04:48 | |
*** mvk has joined #openstack-nova | 04:48 | |
*** vladikr has quit IRC | 04:50 | |
*** vladikr has joined #openstack-nova | 04:51 | |
*** udesale has quit IRC | 04:56 | |
*** yamamoto has joined #openstack-nova | 04:56 | |
*** heyongli has quit IRC | 04:57 | |
*** heyongli has joined #openstack-nova | 04:57 | |
*** udesale has joined #openstack-nova | 04:57 | |
*** udesale has quit IRC | 04:57 | |
*** armaan has quit IRC | 04:58 | |
*** udesale has joined #openstack-nova | 04:58 | |
*** armaan has joined #openstack-nova | 04:58 | |
*** shaohe_feng has quit IRC | 04:58 | |
*** Dinesh_Bhor has quit IRC | 04:59 | |
*** shaohe_feng has joined #openstack-nova | 05:00 | |
*** heyongli has quit IRC | 05:07 | |
*** heyongli has joined #openstack-nova | 05:07 | |
*** shaohe_feng has quit IRC | 05:08 | |
*** shaohe_feng has joined #openstack-nova | 05:09 | |
*** bigdogstl has joined #openstack-nova | 05:11 | |
*** nmathew has joined #openstack-nova | 05:14 | |
*** bigdogstl has quit IRC | 05:15 | |
*** heyongli has quit IRC | 05:17 | |
*** lpetrut has joined #openstack-nova | 05:17 | |
*** heyongli has joined #openstack-nova | 05:17 | |
*** shaohe_feng has quit IRC | 05:19 | |
*** shaohe_feng has joined #openstack-nova | 05:19 | |
*** takashin has joined #openstack-nova | 05:19 | |
*** Bhujay has quit IRC | 05:21 | |
*** Dinesh_Bhor has joined #openstack-nova | 05:22 | |
*** bigdogstl has joined #openstack-nova | 05:23 | |
*** suresh12 has quit IRC | 05:25 | |
*** McNinja has quit IRC | 05:25 | |
*** Eran_Kuris has joined #openstack-nova | 05:26 | |
*** lpetrut has quit IRC | 05:27 | |
*** heyongli has quit IRC | 05:27 | |
*** mvenesio has quit IRC | 05:28 | |
*** heyongli has joined #openstack-nova | 05:28 | |
*** sar has quit IRC | 05:28 | |
*** bigdogstl has quit IRC | 05:28 | |
*** Bhujay has joined #openstack-nova | 05:28 | |
*** shaohe_feng has quit IRC | 05:29 | |
*** vladikr has quit IRC | 05:29 | |
*** shaohe_feng has joined #openstack-nova | 05:30 | |
*** McNinja has joined #openstack-nova | 05:30 | |
*** vladikr has joined #openstack-nova | 05:31 | |
*** tssurya has joined #openstack-nova | 05:32 | |
*** antosh has joined #openstack-nova | 05:32 | |
*** myrat has quit IRC | 05:33 | |
*** links has quit IRC | 05:33 | |
*** tssurya has quit IRC | 05:36 | |
*** antosh has quit IRC | 05:37 | |
*** heyongli has quit IRC | 05:38 | |
*** heyongli has joined #openstack-nova | 05:38 | |
*** shaohe_feng has quit IRC | 05:39 | |
*** bigdogstl has joined #openstack-nova | 05:40 | |
*** shaohe_feng has joined #openstack-nova | 05:42 | |
*** psachin has quit IRC | 05:43 | |
*** bigdogstl has quit IRC | 05:44 | |
*** myrat has joined #openstack-nova | 05:45 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix nits in nested provider allocation candidates https://review.openstack.org/572673 | 05:46 |
*** bigdogstl has joined #openstack-nova | 05:47 | |
*** heyongli has quit IRC | 05:48 | |
*** Dinesh_Bhor has quit IRC | 05:48 | |
*** heyongli has joined #openstack-nova | 05:48 | |
*** links has joined #openstack-nova | 05:49 | |
*** fragatina has joined #openstack-nova | 05:49 | |
*** Dinesh_Bhor has joined #openstack-nova | 05:49 | |
*** shaohe_feng has quit IRC | 05:49 | |
*** shaohe_feng has joined #openstack-nova | 05:50 | |
*** Eran_Kuris has quit IRC | 05:52 | |
*** bigdogstl has quit IRC | 05:52 | |
*** nmathew- has joined #openstack-nova | 05:52 | |
*** jaosorior has joined #openstack-nova | 05:52 | |
*** suresh12 has joined #openstack-nova | 05:54 | |
*** nmathew has quit IRC | 05:56 | |
*** Eran_Kuris has joined #openstack-nova | 05:56 | |
*** Bhujay has quit IRC | 05:56 | |
*** heyongli has quit IRC | 05:58 | |
*** heyongli has joined #openstack-nova | 05:59 | |
*** suresh12 has quit IRC | 05:59 | |
*** dklyle has quit IRC | 05:59 | |
*** shaohe_feng has quit IRC | 06:00 | |
*** shaohe_feng has joined #openstack-nova | 06:00 | |
*** germs has joined #openstack-nova | 06:01 | |
*** germs has quit IRC | 06:01 | |
*** germs has joined #openstack-nova | 06:01 | |
*** myrat has quit IRC | 06:01 | |
*** masber has joined #openstack-nova | 06:04 | |
*** dpawlik has joined #openstack-nova | 06:05 | |
*** germs has quit IRC | 06:05 | |
*** hamdyk has joined #openstack-nova | 06:07 | |
*** heyongli has quit IRC | 06:08 | |
*** heyongli has joined #openstack-nova | 06:09 | |
*** dpawlik has quit IRC | 06:09 | |
*** shaohe_feng has quit IRC | 06:10 | |
*** shaohe_feng has joined #openstack-nova | 06:10 | |
*** bigdogstl has joined #openstack-nova | 06:13 | |
*** dpawlik has joined #openstack-nova | 06:13 | |
*** myrat has joined #openstack-nova | 06:14 | |
*** sar has joined #openstack-nova | 06:15 | |
*** edmondsw has joined #openstack-nova | 06:15 | |
*** vladikr has quit IRC | 06:16 | |
*** bigdogstl has quit IRC | 06:18 | |
*** vladikr has joined #openstack-nova | 06:18 | |
*** heyongli has quit IRC | 06:19 | |
*** heyongli has joined #openstack-nova | 06:19 | |
*** Luzi has joined #openstack-nova | 06:20 | |
*** shaohe_feng has quit IRC | 06:20 | |
*** edmondsw has quit IRC | 06:20 | |
*** shaohe_feng has joined #openstack-nova | 06:21 | |
*** myrat has quit IRC | 06:24 | |
*** heyongli has quit IRC | 06:29 | |
*** heyongli has joined #openstack-nova | 06:29 | |
*** andreas_s has joined #openstack-nova | 06:29 | |
*** ccamacho has joined #openstack-nova | 06:29 | |
*** shaohe_feng has quit IRC | 06:30 | |
*** bigdogstl has joined #openstack-nova | 06:33 | |
*** brault has joined #openstack-nova | 06:33 | |
*** shaohe_feng has joined #openstack-nova | 06:34 | |
*** pvc has quit IRC | 06:35 | |
*** myrat has joined #openstack-nova | 06:36 | |
*** Dinesh_Bhor has quit IRC | 06:37 | |
*** brault has quit IRC | 06:37 | |
*** Dinesh_Bhor has joined #openstack-nova | 06:38 | |
*** heyongli has quit IRC | 06:39 | |
*** heyongli has joined #openstack-nova | 06:39 | |
*** bkopilov has joined #openstack-nova | 06:40 | |
*** shaohe_feng has quit IRC | 06:41 | |
*** bigdogstl has quit IRC | 06:41 | |
*** shaohe_feng has joined #openstack-nova | 06:41 | |
*** slaweq has joined #openstack-nova | 06:43 | |
*** links has quit IRC | 06:44 | |
*** pcaruana has joined #openstack-nova | 06:44 | |
*** fragatina has quit IRC | 06:44 | |
*** jaosorior has quit IRC | 06:47 | |
*** nmathew has joined #openstack-nova | 06:48 | |
*** ispp has joined #openstack-nova | 06:48 | |
*** heyongli has quit IRC | 06:49 | |
*** heyongli has joined #openstack-nova | 06:50 | |
*** nmathew- has quit IRC | 06:51 | |
*** shaohe_feng has quit IRC | 06:51 | |
*** shaohe_feng has joined #openstack-nova | 06:51 | |
openstackgerrit | jichenjc proposed openstack/nova master: Fix doc nit https://review.openstack.org/572682 | 06:54 |
*** janki has joined #openstack-nova | 06:55 | |
*** brault has joined #openstack-nova | 06:58 | |
*** heyongli has quit IRC | 07:00 | |
*** heyongli has joined #openstack-nova | 07:00 | |
*** shaohe_feng has quit IRC | 07:01 | |
*** bigdogstl has joined #openstack-nova | 07:02 | |
*** lifeless has quit IRC | 07:02 | |
*** shaohe_feng has joined #openstack-nova | 07:02 | |
*** lifeless has joined #openstack-nova | 07:02 | |
*** links has joined #openstack-nova | 07:03 | |
*** artom has joined #openstack-nova | 07:04 | |
*** bigdogstl has quit IRC | 07:06 | |
*** rcernin has quit IRC | 07:07 | |
*** heyongli has quit IRC | 07:10 | |
*** heyongli has joined #openstack-nova | 07:10 | |
*** shaohe_feng has quit IRC | 07:11 | |
*** links has quit IRC | 07:12 | |
*** shaohe_feng has joined #openstack-nova | 07:14 | |
*** brault has quit IRC | 07:14 | |
*** tssurya has joined #openstack-nova | 07:14 | |
*** armaan has quit IRC | 07:15 | |
*** ratailor has joined #openstack-nova | 07:16 | |
openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459 | 07:16 |
openstackgerrit | Merged openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | 07:18 |
*** openstackgerrit has quit IRC | 07:19 | |
*** Dinesh_Bhor has quit IRC | 07:20 | |
*** heyongli has quit IRC | 07:20 | |
*** heyongli has joined #openstack-nova | 07:20 | |
naichuans | melwitt: got it, thank you very much for the reminder | 07:21 |
*** shaohe_feng has quit IRC | 07:22 | |
*** lifeless has quit IRC | 07:22 | |
*** shaohe_feng has joined #openstack-nova | 07:22 | |
*** Dinesh_Bhor has joined #openstack-nova | 07:24 | |
*** ispp has quit IRC | 07:25 | |
*** bigdogstl has joined #openstack-nova | 07:26 | |
*** masber has quit IRC | 07:27 | |
*** ralonsoh has joined #openstack-nova | 07:27 | |
*** ralonsoh has quit IRC | 07:27 | |
*** lifeless has joined #openstack-nova | 07:28 | |
*** myrat has quit IRC | 07:29 | |
*** takashin has left #openstack-nova | 07:30 | |
*** alexchadin has joined #openstack-nova | 07:30 | |
*** heyongli has quit IRC | 07:30 | |
*** bigdogstl has quit IRC | 07:31 | |
*** heyongli has joined #openstack-nova | 07:31 | |
*** shaohe_feng has quit IRC | 07:32 | |
*** links has joined #openstack-nova | 07:33 | |
*** armaan has joined #openstack-nova | 07:33 | |
*** antosh has joined #openstack-nova | 07:33 | |
*** shaohe_feng has joined #openstack-nova | 07:33 | |
*** masber has joined #openstack-nova | 07:35 | |
*** antosh has quit IRC | 07:38 | |
*** ttsiouts has joined #openstack-nova | 07:38 | |
*** masuberu has joined #openstack-nova | 07:39 | |
*** heyongli has quit IRC | 07:41 | |
*** myrat has joined #openstack-nova | 07:41 | |
*** amoralej|off is now known as amoralej | 07:41 | |
*** heyongli has joined #openstack-nova | 07:41 | |
*** shaohe_feng has quit IRC | 07:42 | |
*** masber has quit IRC | 07:43 | |
*** openstackgerrit has joined #openstack-nova | 07:43 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Update launch-from-volume doc to latest version. https://review.openstack.org/572241 | 07:43 |
*** shaohe_feng has joined #openstack-nova | 07:45 | |
*** ttsiouts has quit IRC | 07:47 | |
*** jangutter has joined #openstack-nova | 07:48 | |
*** heyongli has quit IRC | 07:51 | |
*** heyongli has joined #openstack-nova | 07:51 | |
*** shaohe_feng has quit IRC | 07:52 | |
*** jpena|off is now known as jpena | 07:52 | |
*** shaohe_feng has joined #openstack-nova | 07:53 | |
*** bigdogstl has joined #openstack-nova | 07:56 | |
*** brault has joined #openstack-nova | 07:57 | |
*** dklyle has joined #openstack-nova | 07:57 | |
*** brault has quit IRC | 07:57 | |
*** brault has joined #openstack-nova | 07:58 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 07:58 |
openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 07:58 |
openstackgerrit | Chris Dent proposed openstack/nova master: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 07:58 |
*** sahid has joined #openstack-nova | 07:58 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067 | 07:59 |
*** heyongli has quit IRC | 08:01 | |
*** bigdogstl has quit IRC | 08:01 | |
*** germs has joined #openstack-nova | 08:01 | |
*** germs has quit IRC | 08:01 | |
*** germs has joined #openstack-nova | 08:01 | |
*** heyongli has joined #openstack-nova | 08:01 | |
*** shaohe_feng has quit IRC | 08:03 | |
*** threestrands_ has quit IRC | 08:03 | |
*** edmondsw has joined #openstack-nova | 08:03 | |
*** tssurya has quit IRC | 08:04 | |
*** shaohe_feng has joined #openstack-nova | 08:04 | |
*** ttsiouts has joined #openstack-nova | 08:06 | |
*** armaan has quit IRC | 08:06 | |
*** germs has quit IRC | 08:06 | |
*** edmondsw has quit IRC | 08:08 | |
*** bigdogstl has joined #openstack-nova | 08:09 | |
*** dklyle has quit IRC | 08:09 | |
*** mgoddard has joined #openstack-nova | 08:11 | |
*** heyongli has quit IRC | 08:11 | |
*** dklyle has joined #openstack-nova | 08:12 | |
*** heyongli has joined #openstack-nova | 08:12 | |
*** shaohe_feng has quit IRC | 08:13 | |
*** brault has quit IRC | 08:13 | |
*** armaan has joined #openstack-nova | 08:13 | |
*** bigdogstl has quit IRC | 08:13 | |
*** tssurya has joined #openstack-nova | 08:14 | |
*** shaohe_feng has joined #openstack-nova | 08:14 | |
*** armaan has quit IRC | 08:17 | |
*** heyongli has quit IRC | 08:22 | |
*** heyongli has joined #openstack-nova | 08:22 | |
*** shaohe_feng has quit IRC | 08:23 | |
*** shaohe_feng has joined #openstack-nova | 08:24 | |
*** dklyle has quit IRC | 08:29 | |
*** dklyle has joined #openstack-nova | 08:29 | |
*** bigdogstl has joined #openstack-nova | 08:29 | |
openstackgerrit | sahid proposed openstack/nova master: libvirt: remove unused get_ovs_interfaceid() https://review.openstack.org/572699 | 08:30 |
*** heyongli has quit IRC | 08:32 | |
*** heyongli has joined #openstack-nova | 08:32 | |
*** armaan has joined #openstack-nova | 08:32 | |
*** antosh has joined #openstack-nova | 08:33 | |
*** shaohe_feng has quit IRC | 08:33 | |
*** ispp has joined #openstack-nova | 08:34 | |
*** shaohe_feng has joined #openstack-nova | 08:34 | |
*** bigdogstl has quit IRC | 08:34 | |
*** derekh has joined #openstack-nova | 08:36 | |
*** antosh has quit IRC | 08:37 | |
*** brault has joined #openstack-nova | 08:39 | |
*** pvc has joined #openstack-nova | 08:40 | |
pvc | i have an instance with GPU on it but my hypervisor is always restarting and i dont know why | 08:40 |
*** heyongli has quit IRC | 08:42 | |
*** heyongli has joined #openstack-nova | 08:42 | |
*** shaohe_feng has quit IRC | 08:44 | |
*** shaohe_feng has joined #openstack-nova | 08:46 | |
*** ispp has quit IRC | 08:47 | |
openstackgerrit | Merged openstack/nova master: api-ref: mention that you can't re-parent a resource provider https://review.openstack.org/572501 | 08:48 |
openstackgerrit | Merged openstack/nova master: Fix some inconsistencies in doc https://review.openstack.org/570407 | 08:48 |
*** bigdogstl has joined #openstack-nova | 08:48 | |
pvc | can anyone help me | 08:52 |
*** heyongli has quit IRC | 08:52 | |
*** heyongli has joined #openstack-nova | 08:53 | |
*** bigdogstl has quit IRC | 08:53 | |
*** alexchadin has quit IRC | 08:53 | |
*** shaohe_feng has quit IRC | 08:54 | |
*** alexchadin has joined #openstack-nova | 08:54 | |
*** shaohe_feng has joined #openstack-nova | 08:57 | |
*** ttsiouts has quit IRC | 08:58 | |
*** sar has quit IRC | 08:59 | |
*** sar has joined #openstack-nova | 08:59 | |
mdbooth | Can anybody remind me which repo the ci job definitions are in? | 09:01 |
*** heyongli has quit IRC | 09:03 | |
*** heyongli has joined #openstack-nova | 09:03 | |
*** evin has joined #openstack-nova | 09:04 | |
*** shaohe_feng has quit IRC | 09:04 | |
*** shaohe_feng has joined #openstack-nova | 09:04 | |
*** lifeless has quit IRC | 09:05 | |
kashyap | mdbooth: "project-config", IIRC | 09:05 |
* kashyap looks | 09:05 | |
*** sar has quit IRC | 09:05 | |
kashyap | Maybe it's this, since upstream has moved to Zuul a while ago: https://github.com/openstack-infra/openstack-zuul-jobs | 09:06 |
*** penick has quit IRC | 09:06 | |
*** ttsiouts has joined #openstack-nova | 09:08 | |
kashyap | mdbooth: Yes, the above is the repo. From its description: | 09:08 |
kashyap | This repo contains a set of ansible playbooks which are used by the | 09:08 |
kashyap | OpenStack project CI system Zuul. It also contains job and | 09:08 |
kashyap | project-template definitions for the OpenStack project. You should | 09:08 |
kashyap | edit these files to make configuration changes to OpenStack | 09:08 |
*** andreas_s has quit IRC | 09:08 | |
kashyap | Infrastructure CI. | 09:08 |
mdbooth | kashyap: Thanks. I'll take a look. | 09:08 |
*** bigdogstl has joined #openstack-nova | 09:10 | |
*** penick has joined #openstack-nova | 09:11 | |
*** heyongli has quit IRC | 09:13 | |
*** heyongli has joined #openstack-nova | 09:14 | |
lyarwood | kashyap: http://logs.openstack.org/33/571433/2/gate/legacy-tempest-dsvm-neutron-full/a007290/logs/screen-n-cpu.txt.gz?#_2018-06-06_04_56_16_327 - if you have anytime this morning would you mind looking over this cold snapshot failure, appears we can't resume the paused/saved domain but I can't see a clear reason why in the libvirtd logs. | 09:14 |
*** shaohe_feng has quit IRC | 09:14 | |
lyarwood | kashyap: https://review.openstack.org/#/c/571433/2 is the change FWIW | 09:15 |
*** shaohe_feng has joined #openstack-nova | 09:15 | |
*** tetsuro has quit IRC | 09:15 | |
kashyap | lyarwood: Hi, will look | 09:15 |
*** bigdogstl has quit IRC | 09:15 | |
*** lpetrut has joined #openstack-nova | 09:16 | |
*** andreas_s has joined #openstack-nova | 09:17 | |
*** heyongli has quit IRC | 09:23 | |
*** heyongli has joined #openstack-nova | 09:23 | |
*** bigdogstl has joined #openstack-nova | 09:24 | |
*** andreas_s has quit IRC | 09:24 | |
*** myrat has quit IRC | 09:24 | |
*** andreas_s has joined #openstack-nova | 09:24 | |
*** shaohe_feng has quit IRC | 09:25 | |
*** shaohe_feng has joined #openstack-nova | 09:25 | |
*** bigdogstl has quit IRC | 09:28 | |
jangutter | sahid: Thanks for taking a look at https://review.openstack.org/567148 . I agree that there's "duplicate" info passed through for VIFHostDevice, but at least it's consistent with other VIF's. I think it has to do with the distinction between the VIF objects and the VIFPortProfile objects. | 09:29 |
jangutter | sahid: The os-vif VIF describes "how to wire it into the VM". The os-vif port-profile describes "how to wire it into the datapath". Because the distinction is not particularly clear with libvirt, there's some muddy overlap. | 09:31 |
*** Dinesh_Bhor has quit IRC | 09:32 | |
openstackgerrit | Bhagyashri Shewale proposed openstack/python-novaclient master: Modify novaclient to support basic attributes https://review.openstack.org/572285 | 09:33 |
*** heyongli has quit IRC | 09:33 | |
*** heyongli has joined #openstack-nova | 09:34 | |
*** shaohe_feng has quit IRC | 09:35 | |
*** shaohe_feng has joined #openstack-nova | 09:36 | |
*** armaan has quit IRC | 09:37 | |
*** armaan has joined #openstack-nova | 09:37 | |
kashyap | lyarwood: Meta comment: The libvirt debug log on the URL is supposedly: 3.2M. But my download of it is still running beyond 7MB | 09:38 |
kashyap | Maybe `wget` extracts it downloads | 09:38 |
*** links has quit IRC | 09:39 | |
kashyap | The full download (on a remote machine) was: 47M :-) | 09:40 |
*** alexchad_ has joined #openstack-nova | 09:41 | |
*** lifeless has joined #openstack-nova | 09:42 | |
*** armaan has quit IRC | 09:43 | |
*** alexchadin has quit IRC | 09:43 | |
*** armaan has joined #openstack-nova | 09:43 | |
*** heyongli has quit IRC | 09:44 | |
*** heyongli has joined #openstack-nova | 09:44 | |
*** shaohe_feng has quit IRC | 09:45 | |
*** shaohe_feng has joined #openstack-nova | 09:46 | |
mdbooth | kashyap: I'm looking at the libvirt logs from this multiattach job: http://logs.openstack.org/58/567258/5/check/nova-multiattach/d23fad8/logs/libvirt/libvirtd.txt.gz | 09:46 |
mdbooth | Specifically I'm trying to see what goes on during the successful multiattach swap volume test | 09:47 |
* kashyap clicks | 09:48 | |
mdbooth | It looks like the drive of interest is added around 2018-06-04 10:56:57.865+0000 | 09:49 |
kashyap | Can you tell the drive ID? | 09:49 |
kashyap | Is it? 'drive-virtio-disk1' | 09:50 |
mdbooth | drive-virtio-1 | 09:50 |
*** bigdogstl has joined #openstack-nova | 09:50 | |
mdbooth | I'm slightly confused, because I don't see shared in the 'human-monitor-command' | 09:50 |
kashyap | I don't see a "drive-virtio-1". Did you mean: 'drive-virtio-disk1'? | 09:51 |
mdbooth | But there is "share-rw":"on" in what seems to be sent to qemu | 09:51 |
mdbooth | kashyap: Yeah | 09:51 |
kashyap | libvirt uses QMP (the JSON RPC) with QEMU. Not HMP. (only rarely) | 09:51 |
kashyap | Yes: | 09:51 |
kashyap | 2018-06-04 11:13:18.158+0000: 32308: debug : qemuMonitorJSONCommandWithFd:301 : Send command '{"execute":"device_add","arguments":{"driver":"virtio-blk-pci","scsi":"off","bus":"pci.0","addr" | 09:51 |
kashyap | :"0x6","share-rw":"on","drive":"drive-virtio-disk1","id":"virtio-disk1"},"id":"libvirt-20"}' for write with FD -1 | 09:51 |
*** edmondsw has joined #openstack-nova | 09:52 | |
*** armaan has quit IRC | 09:52 | |
mdbooth | Right, so that's 'shared', right? | 09:52 |
kashyap | mdbooth: Yes. And libvirt will _not_ migrate it. | 09:52 |
mdbooth | i.e. it's multiattach | 09:52 |
kashyap | Here is the libvirt source itself saying as such: | 09:52 |
kashyap | 288 /* Default is to migrate only non-shared non-readonly disks | 09:52 |
kashyap | 289 * with source */ | 09:52 |
kashyap | 290 return !disk->src->shared && !disk->src->readonly && | 09:52 |
kashyap | 291 !virStorageSourceIsEmpty(disk->src); | 09:52 |
kashyap | 292 } | 09:52 |
kashyap | 293 | 09:52 |
*** armaan has joined #openstack-nova | 09:52 | |
mdbooth | kashyap: Except that it does | 09:52 |
*** antosh has joined #openstack-nova | 09:53 | |
mdbooth | Keep searching down on that disk id | 09:53 |
*** andreas_s has quit IRC | 09:53 | |
mdbooth | 2018-06-04 10:57:05.201+0000 | 09:53 |
mdbooth | It does a drive mirror | 09:53 |
*** ispp has joined #openstack-nova | 09:53 | |
*** andreas_s has joined #openstack-nova | 09:54 | |
mdbooth | This isn't live migration, btw | 09:54 |
*** heyongli has quit IRC | 09:54 | |
mdbooth | This is volume migration | 09:54 |
*** heyongli has joined #openstack-nova | 09:54 | |
mdbooth | So the data in volume a is moved to volume b and a is seamlessly substituted for b in the domain | 09:54 |
*** bigdogstl has quit IRC | 09:54 | |
*** ispp has quit IRC | 09:55 | |
kashyap | mdbooth: Hang on -- | 09:55 |
mdbooth | So it looks like this version of qemu/libvirt at least does permit drive mirror of a shared disk | 09:55 |
kashyap | The `drive-mirror` is done fir 'disk0' | 09:55 |
kashyap | s/fir/for/ | 09:55 |
*** shaohe_feng has quit IRC | 09:55 | |
kashyap | 2018-06-04 10:59:30.956+0000: 32306: debug : qemuMonitorJSONCommandWithFd:301 : Send command '{"execute":"drive-mirror","arguments":{"device":"drive-virtio-disk0","target":"/opt/stack/data/n | 09:55 |
kashyap | ova/instances/snapshots/tmpQeWSxD/8d2e75dbf0a4466fb31f11ca06a21a96.delta","sync":"top","mode":"existing","format":"qcow2"},"id":"libvirt-28"}' for write with FD -1 | 09:55 |
mdbooth | kashyap: That's much later | 09:56 |
mdbooth | Different operation | 09:56 |
*** alexchad_ has quit IRC | 09:56 | |
*** shaohe_feng has joined #openstack-nova | 09:56 | |
mdbooth | Look at 2018-06-04 10:57:05.201+0000 | 09:56 |
*** edmondsw has quit IRC | 09:56 | |
kashyap | mdbooth: Okay, I seem to have misread the code :-( | 09:57 |
kashyap | mdbooth: libvirt _doesn't_ forbid | 09:57 |
*** links has joined #openstack-nova | 09:57 | |
*** antosh has quit IRC | 09:57 | |
* kashyap looks closer | 09:57 | |
pvc | hi | 09:58 |
kashyap | mdbooth: So libvirt should _not_ forbid because, the said shared disk might be in use at any given point. | 09:59 |
kashyap | But probably that's what you were trying to tell me. | 09:59 |
mdbooth | kashyap: Actually I think it *should* forbid it, but doesn't | 10:00 |
pvc | kashyap | 10:00 |
pvc | can help me with gpu instnace? | 10:00 |
kashyap | mdbooth: libvirt developer says the above reason is why it shouldn't forbid it | 10:00 |
mdbooth | Although... meh it should probably give us the rope to hang ourselves with | 10:00 |
kashyap | pvc: Hi, in the middle of something. Please ask such questions to the operators list or ask.openstack.org | 10:00 |
kashyap | (And I don't know the answer to it, top off my head) | 10:01 |
*** andreas_s has quit IRC | 10:01 | |
*** andreas_s has joined #openstack-nova | 10:01 | |
mdbooth | kashyap: I don't understand why what 'be in use' means. Can you explain? | 10:02 |
kashyap | mdbooth: BTW, yes. I did see the timestamp you pointed out. And indeed the shared disk is being `drive-mirror`ed: | 10:02 |
mdbooth | why what? | 10:02 |
kashyap | 2018-06-04 10:57:05.201+0000: 32306: debug : qemuMonitorJSONCommandWithFd:301 : Send command '{"execute":"drive-mirror","arguments":{"device":"drive-virtio-disk1","target":"/dev/sdb","sync":"full","mode":"existing","format":"raw"},"id":"libvirt-23"}' for write with FD -1 | 10:02 |
mdbooth | s/why// | 10:02 |
*** germs has joined #openstack-nova | 10:02 | |
*** germs has quit IRC | 10:02 | |
*** germs has joined #openstack-nova | 10:02 | |
kashyap | mdbooth: The shared disk might be in use by the other VM | 10:02 |
*** myrat has joined #openstack-nova | 10:03 | |
mdbooth | Exactly, and iiuc libvirt/qemu has no way to know if this is the case | 10:03 |
mdbooth | So doing a drive mirror isn't a safe operation, because it can't take account of changes made by another vm | 10:03 |
kashyap | (Hmm, the libvirt dev isn't sure if QEMU disallows forbids or not.) | 10:04 |
mdbooth | You'd need a side-channel between all attached vms to make that safe. | 10:04 |
*** heyongli has quit IRC | 10:04 | |
mdbooth | And you can't do it in the guest os, because this operation isn't visible to the guest os | 10:04 |
*** heyongli has joined #openstack-nova | 10:04 | |
kashyap | mdbooth: Mind filing a bug for this? | 10:05 |
mdbooth | kashyap: Which channel are you on? | 10:05 |
kashyap | #virt, OFTC | 10:05 |
mdbooth | kashyap: I'm not convinced it's a bug, tbh. | 10:05 |
*** shaohe_feng has quit IRC | 10:06 | |
mdbooth | Well actually I do think it's a bug, but in Nova. | 10:06 |
mdbooth | I don't think we should be attempting to do this in the first place. | 10:06 |
kashyap | (That's what I meant, Nova LP bug) | 10:06 |
*** shaohe_feng has joined #openstack-nova | 10:06 | |
*** armaan has quit IRC | 10:07 | |
mdbooth | The problem with that is that to fix it we have to break an api | 10:07 |
*** armaan has joined #openstack-nova | 10:07 | |
mdbooth | i.e. make something fail which previously succeeded | 10:07 |
*** germs has quit IRC | 10:07 | |
*** jaosorior has joined #openstack-nova | 10:08 | |
*** bkopilov has quit IRC | 10:09 | |
kashyap | Hmm | 10:11 |
*** abhishekk is now known as abhishekk|afk | 10:11 | |
kashyap | mdbooth: Can you step back a bit and remind me the actual Nova test? | 10:13 |
kashyap | It is doing "swap volume", i.e. volume migration? But in "mult-attach" context? | 10:14 |
*** heyongli has quit IRC | 10:14 | |
*** bigdogstl has joined #openstack-nova | 10:15 | |
*** heyongli has joined #openstack-nova | 10:15 | |
*** shaohe_feng has quit IRC | 10:16 | |
*** links has quit IRC | 10:16 | |
*** namnh has quit IRC | 10:16 | |
*** shaohe_feng has joined #openstack-nova | 10:16 | |
*** pvc has quit IRC | 10:17 | |
*** tbachman has quit IRC | 10:19 | |
*** bigdogstl has quit IRC | 10:20 | |
kashyap | mdbooth: So, the _default_ behaviour for libvirt is that it won't migrate the shared disks if you don't ask for them to be shared. | 10:20 |
kashyap | If you _do_ aske the shared disks to be migrated, libvirt will honor that. | 10:20 |
kashyap | I think Nova shouldn't do that is what you are saying. | 10:21 |
*** annp has quit IRC | 10:21 | |
lyarwood | kashyap: volume migration in this context is swap volume, so rebasing in libvirt terms. | 10:21 |
kashyap | Yep. blockRebase()/blockCopy() --> calls QEMU 'drive-mirror' | 10:21 |
lyarwood | kashyap: right, just checking you didn't have it confused with actual block LM. | 10:22 |
kashyap | Yeah, my head already auto-maps _swap_volume() to the above APIs. But thank you :-) | 10:23 |
kashyap | lyarwood: Actual block LM also goes down to that 'drive-mirror' call, you know that? | 10:23 |
lyarwood | kashyap: yeah without going through blockRebase right? | 10:24 |
lyarwood | kashyap: it's just a copy | 10:24 |
*** heyongli has quit IRC | 10:25 | |
*** heyongli has joined #openstack-nova | 10:25 | |
*** bigdogstl has joined #openstack-nova | 10:26 | |
kashyap | Not sure on that, have to check | 10:26 |
*** shaohe_feng has quit IRC | 10:26 | |
*** lifeless has quit IRC | 10:26 | |
*** lifeless has joined #openstack-nova | 10:27 | |
*** andreas_s has quit IRC | 10:28 | |
*** antosh has joined #openstack-nova | 10:28 | |
*** andreas_s has joined #openstack-nova | 10:28 | |
*** shaohe_feng has joined #openstack-nova | 10:28 | |
*** links has joined #openstack-nova | 10:29 | |
*** bigdogstl has quit IRC | 10:31 | |
*** antosh has quit IRC | 10:32 | |
*** heyongli has quit IRC | 10:35 | |
*** heyongli has joined #openstack-nova | 10:35 | |
*** jamesdenton has quit IRC | 10:35 | |
*** shaohe_feng has quit IRC | 10:36 | |
*** andreas_s has quit IRC | 10:37 | |
*** shaohe_feng has joined #openstack-nova | 10:38 | |
*** tbachman has joined #openstack-nova | 10:38 | |
*** bigdogstl has joined #openstack-nova | 10:39 | |
*** tbachman_ has joined #openstack-nova | 10:42 | |
*** tbachman has quit IRC | 10:43 | |
*** tbachman_ is now known as tbachman | 10:43 | |
*** bigdogstl has quit IRC | 10:43 | |
*** priteau has joined #openstack-nova | 10:45 | |
*** heyongli has quit IRC | 10:45 | |
*** heyongli has joined #openstack-nova | 10:45 | |
*** andreas_s has joined #openstack-nova | 10:46 | |
*** shaohe_feng has quit IRC | 10:47 | |
*** shaohe_feng has joined #openstack-nova | 10:48 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840 | 10:48 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267 | 10:48 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847 | 10:48 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | 10:48 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459 | 10:48 |
*** bigdogstl has joined #openstack-nova | 10:48 | |
*** myrat has quit IRC | 10:49 | |
openstackgerrit | Merged openstack/nova master: Ensure resource class cache when listing usages https://review.openstack.org/572652 | 10:50 |
*** chyka has joined #openstack-nova | 10:50 | |
*** bigdogstl has quit IRC | 10:53 | |
*** dtantsur|afk is now known as dtantsur | 10:54 | |
*** heyongli has quit IRC | 10:55 | |
*** chyka has quit IRC | 10:55 | |
*** phuongnh has quit IRC | 10:56 | |
*** heyongli has joined #openstack-nova | 10:56 | |
*** mdnadeem has quit IRC | 10:56 | |
*** shaohe_feng has quit IRC | 10:57 | |
*** lifeless_ has joined #openstack-nova | 10:57 | |
*** lifeless has quit IRC | 10:57 | |
*** shaohe_feng has joined #openstack-nova | 10:58 | |
*** andreas_s has quit IRC | 10:59 | |
*** myrat has joined #openstack-nova | 11:01 | |
*** antosh has joined #openstack-nova | 11:01 | |
*** dklyle has quit IRC | 11:02 | |
*** antosh has quit IRC | 11:05 | |
*** heyongli has quit IRC | 11:06 | |
*** jpena is now known as jpena|lunch | 11:06 | |
*** heyongli has joined #openstack-nova | 11:06 | |
*** shaohe_feng has quit IRC | 11:07 | |
*** shaohe_feng has joined #openstack-nova | 11:08 | |
*** andreas_s has joined #openstack-nova | 11:08 | |
*** Tom-Tom has quit IRC | 11:08 | |
*** bigdogstl has joined #openstack-nova | 11:12 | |
*** edmondsw has joined #openstack-nova | 11:12 | |
*** dpawlik has quit IRC | 11:15 | |
*** heyongli has quit IRC | 11:16 | |
*** heyongli has joined #openstack-nova | 11:16 | |
*** edmondsw has quit IRC | 11:16 | |
*** bigdogstl has quit IRC | 11:17 | |
*** shaohe_feng has quit IRC | 11:17 | |
*** shaohe_feng has joined #openstack-nova | 11:18 | |
*** brault has quit IRC | 11:19 | |
*** abhishekk|afk is now known as abhishekk | 11:19 | |
*** edmondsw has joined #openstack-nova | 11:20 | |
*** heyongli has quit IRC | 11:26 | |
*** heyongli has joined #openstack-nova | 11:26 | |
*** liuzz has quit IRC | 11:27 | |
*** armaan has quit IRC | 11:27 | |
*** shaohe_feng has quit IRC | 11:28 | |
*** armaan has joined #openstack-nova | 11:28 | |
*** shaohe_feng has joined #openstack-nova | 11:29 | |
*** dtantsur is now known as dtantsur|brb | 11:31 | |
*** felipemonteiro has joined #openstack-nova | 11:31 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove network info stubbing in functional test https://review.openstack.org/570018 | 11:32 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840 | 11:32 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267 | 11:32 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847 | 11:32 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | 11:32 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459 | 11:32 |
*** dklyle has joined #openstack-nova | 11:36 | |
*** heyongli has quit IRC | 11:36 | |
*** heyongli has joined #openstack-nova | 11:37 | |
*** antosh has joined #openstack-nova | 11:37 | |
*** shaohe_feng has quit IRC | 11:38 | |
*** shaohe_feng has joined #openstack-nova | 11:39 | |
*** amoralej is now known as amoralej|lunch | 11:41 | |
*** antosh has quit IRC | 11:41 | |
*** dpawlik has joined #openstack-nova | 11:43 | |
*** dpawlik has quit IRC | 11:44 | |
*** Tom-Tom has joined #openstack-nova | 11:46 | |
*** jamesdenton has joined #openstack-nova | 11:46 | |
*** heyongli has quit IRC | 11:47 | |
*** heyongli has joined #openstack-nova | 11:47 | |
*** tbachman has quit IRC | 11:48 | |
*** shaohe_feng has quit IRC | 11:48 | |
*** shaohe_feng has joined #openstack-nova | 11:48 | |
*** myrat has quit IRC | 11:48 | |
*** felipemonteiro has quit IRC | 11:48 | |
*** felipemonteiro has joined #openstack-nova | 11:49 | |
*** dpawlik has joined #openstack-nova | 11:49 | |
*** abhishekk has quit IRC | 11:56 | |
*** heyongli has quit IRC | 11:57 | |
*** heyongli has joined #openstack-nova | 11:57 | |
*** shaohe_feng has quit IRC | 11:58 | |
*** shaohe_feng has joined #openstack-nova | 11:59 | |
*** myrat has joined #openstack-nova | 12:02 | |
*** germs has joined #openstack-nova | 12:03 | |
*** germs has quit IRC | 12:03 | |
*** germs has joined #openstack-nova | 12:03 | |
*** felipemonteiro has quit IRC | 12:04 | |
*** felipemonteiro has joined #openstack-nova | 12:04 | |
*** brault has joined #openstack-nova | 12:04 | |
*** hemna_ has joined #openstack-nova | 12:05 | |
*** abhishekk_ has joined #openstack-nova | 12:06 | |
*** heyongli has quit IRC | 12:07 | |
*** GonZo2000 has joined #openstack-nova | 12:07 | |
*** GonZo2000 has quit IRC | 12:07 | |
*** GonZo2000 has joined #openstack-nova | 12:07 | |
*** liverpooler has joined #openstack-nova | 12:07 | |
*** heyongli has joined #openstack-nova | 12:07 | |
*** germs has quit IRC | 12:08 | |
*** hamdyk has quit IRC | 12:08 | |
*** shaohe_feng has quit IRC | 12:09 | |
*** shaohe_feng has joined #openstack-nova | 12:09 | |
*** jpena|lunch is now known as jpena | 12:10 | |
*** abhishekk_ has quit IRC | 12:14 | |
*** heyongli has quit IRC | 12:17 | |
*** heyongli has joined #openstack-nova | 12:18 | |
*** shaohe_feng has quit IRC | 12:19 | |
*** GonZo2000 has quit IRC | 12:19 | |
*** shaohe_feng has joined #openstack-nova | 12:19 | |
frickler | mriedem_away: yep, lgtm | 12:21 |
*** bpoulos has joined #openstack-nova | 12:22 | |
*** r-daneel has quit IRC | 12:22 | |
*** tiendc has quit IRC | 12:26 | |
*** heyongli has quit IRC | 12:28 | |
*** heyongli has joined #openstack-nova | 12:28 | |
*** armaan has quit IRC | 12:28 | |
*** armaan has joined #openstack-nova | 12:28 | |
*** AlexeyAbashkin has joined #openstack-nova | 12:29 | |
*** shaohe_feng has quit IRC | 12:29 | |
*** shaohe_feng has joined #openstack-nova | 12:31 | |
*** myrat has quit IRC | 12:31 | |
*** r-daneel has joined #openstack-nova | 12:32 | |
*** links has quit IRC | 12:33 | |
*** eharney has joined #openstack-nova | 12:34 | |
*** felipemonteiro_ has joined #openstack-nova | 12:35 | |
*** lyan has quit IRC | 12:35 | |
openstackgerrit | wanghongtao proposed openstack/nova master: Fix the metadata re to match the unicode https://review.openstack.org/536236 | 12:38 |
*** heyongli has quit IRC | 12:38 | |
*** felipemonteiro has quit IRC | 12:38 | |
*** heyongli has joined #openstack-nova | 12:38 | |
*** shaohe_feng has quit IRC | 12:39 | |
*** shaohe_feng has joined #openstack-nova | 12:40 | |
*** andreas_s has quit IRC | 12:41 | |
*** andreas_s has joined #openstack-nova | 12:41 | |
*** amoralej|lunch is now known as amoralej | 12:44 | |
*** armaan has quit IRC | 12:45 | |
*** armaan has joined #openstack-nova | 12:45 | |
*** tidwellr has joined #openstack-nova | 12:47 | |
*** antosh has joined #openstack-nova | 12:48 | |
*** heyongli has quit IRC | 12:48 | |
openstackgerrit | wanghongtao proposed openstack/nova master: Fix the metadata re to match the unicode https://review.openstack.org/536236 | 12:48 |
*** heyongli has joined #openstack-nova | 12:48 | |
*** sapcc-bot2 has quit IRC | 12:49 | |
*** sapcc-bot has quit IRC | 12:49 | |
*** shaohe_feng has quit IRC | 12:50 | |
*** sapcc-bot has joined #openstack-nova | 12:50 | |
*** sapcc-bot9 has joined #openstack-nova | 12:50 | |
*** shaohe_feng has joined #openstack-nova | 12:51 | |
*** antosh has quit IRC | 12:53 | |
*** antosh_ has joined #openstack-nova | 12:53 | |
*** mriedem_away is now known as mriedem | 12:53 | |
*** Nel1x has joined #openstack-nova | 12:54 | |
*** hshiina has quit IRC | 12:54 | |
*** tidwellr has quit IRC | 12:56 | |
*** hongbin has joined #openstack-nova | 12:57 | |
*** antosh_ has quit IRC | 12:58 | |
*** efried has quit IRC | 12:58 | |
*** heyongli has quit IRC | 12:58 | |
*** heyongli has joined #openstack-nova | 12:59 | |
*** shaohe_feng has quit IRC | 13:00 | |
*** shaohe_feng has joined #openstack-nova | 13:02 | |
mriedem | lyarwood: i did find via code inspection at least one cinder bug while looking at the access mode logic in the volume manager, comments in the spec https://review.openstack.org/#/c/552078/ | 13:04 |
*** pchavva has joined #openstack-nova | 13:04 | |
mriedem | lyarwood: i don't really have a great way to test this stuff until i get the stack written through the compute api, so i figure any issues that are on the cinder side with modeling per-attachment access mode will have to be resolved as dependent bugs | 13:04 |
artom | mdbooth, so, re: mailing list post about swap volume - I believe we could just change the response code in the multi-attach case to a 400 or 409 or whatever, without a new microversion | 13:05 |
artom | https://docs.openstack.org/nova/latest/contributor/microversions.html | 13:05 |
mriedem | lyarwood: beyond that, what is holding up the spec? i'll rename attach_mode to access_mode to match what's in the connection_info | 13:05 |
mdbooth | artom: I wasn't sure, but that was my intention. I don't think we should preserve this behaviour at all, even behind a microversion. | 13:06 |
mdbooth | artom: But I'm not clear on the policy here, hence the ML post. | 13:06 |
*** felipemonteiro_ has quit IRC | 13:06 | |
artom | mdbooth, I agree, which is why I would think using a response code that's already possible for that request would be OK | 13:07 |
*** felipemonteiro_ has joined #openstack-nova | 13:07 | |
*** lyan has joined #openstack-nova | 13:07 | |
*** heyongli has quit IRC | 13:09 | |
*** Bhujay has joined #openstack-nova | 13:09 | |
*** heyongli has joined #openstack-nova | 13:09 | |
*** shaohe_feng has quit IRC | 13:10 | |
*** shaohe_feng has joined #openstack-nova | 13:10 | |
*** felipemonteiro_ has quit IRC | 13:11 | |
*** READ10 has joined #openstack-nova | 13:12 | |
*** Nel1x has quit IRC | 13:12 | |
*** bigdogstl has joined #openstack-nova | 13:12 | |
sahid | jangutter: yes i understand what you mean, i commented back on the spec | 13:13 |
sahid | mriedem, stephenfin, if you have a moment to review https://review.openstack.org/#/c/553072/ | 13:14 |
mriedem | that old thing | 13:15 |
*** lifeless_ has quit IRC | 13:16 | |
sahid | yes, that is why i wanted ping you | 13:16 |
*** bigdogstl has quit IRC | 13:17 | |
sahid | melwitt: is this | 13:19 |
*** heyongli has quit IRC | 13:19 | |
*** heyongli has joined #openstack-nova | 13:19 | |
sahid | https://review.openstack.org/#/c/567472/ cannot have to be considered part of: bug/1686116 | 13:19 |
sahid | s/cannot have/cannot be | 13:20 |
*** jaosorior has quit IRC | 13:20 | |
*** shaohe_feng has quit IRC | 13:20 | |
*** shaohe_feng has joined #openstack-nova | 13:21 | |
mriedem | sahid: there was a blueprint for that | 13:23 |
mriedem | not sure why it's a bug now | 13:23 |
mriedem | sahid: i'll fix my nits in https://review.openstack.org/#/c/553072/ | 13:24 |
*** throwsb1 has joined #openstack-nova | 13:26 | |
*** heyongli has quit IRC | 13:29 | |
*** heyongli has joined #openstack-nova | 13:29 | |
*** nicolasbock has joined #openstack-nova | 13:30 | |
*** shaohe_feng has quit IRC | 13:31 | |
openstackgerrit | Chen proposed openstack/python-novaclient master: Fix typos https://review.openstack.org/572770 | 13:31 |
*** dpawlik has quit IRC | 13:32 | |
mriedem | dansmith: not sure who else should take a look at this network_data.json metadata API spec, but you're probably closest https://review.openstack.org/#/c/312626/ | 13:32 |
*** udesale has quit IRC | 13:32 | |
dansmith | I thought we had already done that actually | 13:33 |
*** ttsiouts has quit IRC | 13:33 | |
*** shaohe_feng has joined #openstack-nova | 13:34 | |
jangutter | sahid: Thanks. It doesn't particularly bother me to use naked VIFHostDevice aside from breaking the pattern. | 13:35 |
*** Roamer` has quit IRC | 13:39 | |
sahid | jangutter: which pattern? for VIFHostDevice that looks redondant information | 13:39 |
*** Roamer` has joined #openstack-nova | 13:39 | |
*** heyongli has quit IRC | 13:39 | |
mriedem | dansmith: i think that was a slightly different thing that was a bug fix | 13:39 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: add mtu to libvirt xml for ethernet and bridge types https://review.openstack.org/553072 | 13:39 |
*** heyongli has joined #openstack-nova | 13:40 | |
mriedem | dansmith: this https://github.com/openstack/nova/commit/4b333b989dfc778a8b61db4a1b8552e988a10471#diff-083754c4012f16fa0f9dd9cd703ee0c1 | 13:40 |
dansmith | ah yep | 13:41 |
jangutter | sahid: yep, but in https://github.com/openstack/nova/blob/stable/queens/nova/network/os_vif_util.py#L287-L297 it's used, and in https://github.com/openstack/nova/blob/stable/queens/nova/network/os_vif_util.py#L329-L347 the apparent redundancy is explained (poorly, by me) | 13:41 |
*** shaohe_feng has quit IRC | 13:41 | |
openstackgerrit | Merged openstack/nova master: Add traits check in nested provider candidates https://review.openstack.org/567150 | 13:41 |
mriedem | dansmith: and looking back, i'm still not sure why we didn't have a new metadata api version for that since it was adding a new key to the response | 13:41 |
mriedem | we'd never let that fly in the compute rest api | 13:41 |
mriedem | but metadata api is the wild west pew pew | 13:42 |
* jaypipes gets his revolver. | 13:42 | |
dansmith | well, probably because it's a fairly small iteration | 13:42 |
*** shaohe_feng has joined #openstack-nova | 13:42 | |
dansmith | and because of how metadata is | 13:42 |
dansmith | but yeah | 13:42 |
mriedem | sahid: +2 on https://review.openstack.org/#/c/553072/ now | 13:42 |
dansmith | mriedem: so I haven't looked at this spec in the past, but has someone gone through and looked at what the delta looks like from the existing? | 13:42 |
sahid | mriedem: ok cool | 13:43 |
dansmith | or is it just that we only ever populate networks[0] now? | 13:43 |
mriedem | dansmith: it adds a new ip_addresses field to the response which is a list of fixed ips rather than the first one from the list | 13:43 |
mriedem | we only take the first fixed ip today | 13:43 |
mriedem | which is the ip_address field | 13:43 |
sahid | jangutter: hum... i don't understand the note | 13:43 |
mriedem | this exposes the list of fixed ips | 13:43 |
mriedem | and still maintains the single ip_address field - which arguably we don't need to do since it's versioned | 13:44 |
mriedem | i don't know how sensitive apps are to doing version discovery with metadata | 13:44 |
dansmith | mriedem: this certainly looks like we're asserting multiple interfaces and ips in the test no? https://github.com/openstack/nova/blob/4b333b989dfc778a8b61db4a1b8552e988a10471/nova/tests/unit/network/test_network_info.py | 13:44 |
dansmith | er hang on | 13:44 |
dansmith | https://github.com/openstack/nova/blob/4b333b989dfc778a8b61db4a1b8552e988a10471/nova/tests/unit/network/test_network_info.py#L1118 | 13:45 |
mriedem | dansmith: how so? | 13:45 |
mriedem | https://github.com/openstack/nova/blob/4b333b989dfc778a8b61db4a1b8552e988a10471/nova/tests/unit/network/test_network_info.py#L1121 | 13:45 |
*** tbachman has joined #openstack-nova | 13:45 | |
mriedem | that's a single ipv4 fixed ip | 13:45 |
mriedem | but you can have multiple fixed ips per port in neutron | 13:45 |
mriedem | which is what this spec is trying to expose via network_data.json | 13:45 |
dansmith | oh, | 13:46 |
dansmith | I thought the spec was saying it was the subsequent networks and the ips there that it needed to add | 13:46 |
jangutter | sahid: Back in the day, when I wrote that, my rationale was something like: the VIF is used to generate the xml for libvirt, the port_profile is used to find the representor and plug it into the bridge. | 13:46 |
*** nmathew has quit IRC | 13:46 | |
mriedem | there is also something in the spec about multiple subnet routes which i didn't fully follow | 13:46 |
mriedem | i believe the networks list is per-vif (port) | 13:46 |
dansmith | I see, multiple fixed per port | 13:46 |
dansmith | is what they're enabling | 13:47 |
mriedem | similar to how the server "addresses" field works in the compute api | 13:47 |
mriedem | dansmith: yeah | 13:47 |
dansmith | sorry, I just skimmed when you sent and haven't really read it | 13:47 |
mriedem | so you didn't see the dan smith easter egg in there yet | 13:47 |
jangutter | sahid: Also, the symmetry will break either way, there's only so far you can take the abstraction, I'm agnostic on this, but if the pattern is continued, then at least the os-vif plugins look similar. | 13:48 |
dansmith | mriedem: well now I can't approve until I find something I think qualifies | 13:49 |
*** heyongli has quit IRC | 13:50 | |
*** heyongli has joined #openstack-nova | 13:50 | |
*** ttsiouts has joined #openstack-nova | 13:51 | |
*** shaohe_feng has quit IRC | 13:51 | |
*** Alexey_Abashkin has joined #openstack-nova | 13:51 | |
*** shaohe_feng has joined #openstack-nova | 13:51 | |
*** bkopilov has joined #openstack-nova | 13:51 | |
mriedem | melwitt: ack, and thanks for hitting those placement policy changes | 13:52 |
*** AlexeyAbashkin has quit IRC | 13:53 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 13:53 | |
kashyap | lyarwood: Seems like your recheck has resulted in a success for the 'legacy-tempest-dsvm-neutron-full'? | 13:53 |
kashyap | (For this: https://review.openstack.org/#/c/571433/) | 13:54 |
kashyap | But there's other failures, though | 13:54 |
*** david-lyle has joined #openstack-nova | 13:56 | |
kashyap | (Like this one 'test_volume_extend_when_volume_has_snapshot') | 13:57 |
*** dave-mccowan has joined #openstack-nova | 13:58 | |
*** dklyle has quit IRC | 13:58 | |
*** dtantsur|brb is now known as dtantsur | 13:58 | |
*** heyongli has quit IRC | 14:00 | |
*** heyongli has joined #openstack-nova | 14:00 | |
*** namnh has joined #openstack-nova | 14:01 | |
openstackgerrit | Merged openstack/nova-specs master: Multiple Fixed-IPs support in network information https://review.openstack.org/312626 | 14:01 |
*** shaohe_feng has quit IRC | 14:01 | |
*** antosh has joined #openstack-nova | 14:02 | |
*** shaohe_feng has joined #openstack-nova | 14:03 | |
*** antosh has quit IRC | 14:03 | |
mriedem | mdbooth: artom: replied in the ML, but if we're going to block retype/live migration of multiattach volumes with >1 read/write attachment, that should happen in the cinder API | 14:04 |
*** rodolof has joined #openstack-nova | 14:04 | |
*** antosh has joined #openstack-nova | 14:04 | |
mriedem | failing in the swap volume api in nova is about 3/4 of the way through the operatoin which is a real shitty place to be failing | 14:04 |
*** evin has quit IRC | 14:04 | |
*** awaugama has joined #openstack-nova | 14:05 | |
mdbooth | mriedem: Reading. Was just responding to jaypipes understandable horror at this messy interface. | 14:05 |
artom | mriedem, unless it's called directly, no? Which apparently some people are doing | 14:06 |
*** dave-mcc_ has joined #openstack-nova | 14:06 | |
*** dpawlik has joined #openstack-nova | 14:07 | |
mriedem | mgagne: https://blueprints.launchpad.net/nova/+spec/multiple-fixed-ips-network-information is now approved so you can post code | 14:07 |
*** sahid has quit IRC | 14:07 | |
mriedem | artom: who is doing that? | 14:07 |
artom | mriedem, see mdbooth's reply :) | 14:07 |
mriedem | swap volume in nova is an admin-only api | 14:07 |
artom | We also got a downstream but about it | 14:07 |
mdbooth | mriedem: We really do have customers who call swap volume directly. | 14:07 |
*** tidwellr has joined #openstack-nova | 14:07 | |
mdbooth | More than one. | 14:07 |
mriedem | why? | 14:08 |
kashyap | Presumably to move storage from one type (local) to another (NFS)? | 14:08 |
*** dave-mccowan has quit IRC | 14:08 | |
mdbooth | I still haven't nailed it down, but I *think* it's a workaround related to BFV. | 14:08 |
artom | mriedem, I mean, I would agree, it's a weird thing to do, but we can't honestly put an API out there and get defensive when people use it in unexpected ways | 14:08 |
mriedem | kashyap: that would be a retype | 14:08 |
mriedem | which you should initiate through cinder | 14:08 |
kashyap | Ah, damned if I ever remember the names | 14:08 |
mdbooth | mriedem: But regardless, even if we think it's dumb, it's a public api and they're using it. | 14:09 |
*** efried has joined #openstack-nova | 14:09 | |
*** felipemonteiro has joined #openstack-nova | 14:09 | |
mriedem | awesome | 14:10 |
mdbooth | Indeed | 14:10 |
mriedem | i made it 70 minutes today before hating my life | 14:10 |
mdbooth | \o/ | 14:10 |
kashyap | This morning we chatted here about mdbooth's e-mail (which I haven't pulled down yet). At least in context of what libvirt APi does by default | 14:10 |
* dansmith golf glaps | 14:10 | |
*** heyongli has quit IRC | 14:10 | |
jaypipes | mdbooth: my annoyance is that this is a libvirt/QEMU-specific behaviour/implementation that is leaking out of the Compute API. | 14:10 |
*** heyongli has joined #openstack-nova | 14:10 | |
kashyap | jaypipes: By _default_ libvirt doesn't copy shared disks | 14:10 |
mdbooth | jaypipes: That was the thrust of my reply. It's not libvirt/qemu specific at all. | 14:11 |
jaypipes | mdbooth: BTW, where exactly does "swap volume" imply or denote that there is a copy operation going on? :( | 14:11 |
kashyap | But if you _ask_ for it, it will migrate it | 14:11 |
mriedem | mdbooth: the only virt driver that implements this is libvirt | 14:11 |
*** felipemonteiro_ has joined #openstack-nova | 14:11 | |
mdbooth | jaypipes: It could be implemented by any hypervisor, and they'd have to do the copy. | 14:11 |
jaypipes | mdbooth: yes, it is. | 14:11 |
mriedem | same with guest-assisted volume snapshot | 14:11 |
mdbooth | jaypipes: I replied in depth on the ML, anyway. | 14:11 |
jaypipes | yes, I read your reply. | 14:11 |
mriedem | i have no idea how non-libvirt drivers handle volume retype/live migration either | 14:11 |
*** shaohe_feng has quit IRC | 14:12 | |
jaypipes | mriedem: they don't. wisely. | 14:12 |
*** dpawlik has quit IRC | 14:12 | |
mdbooth | But Cinder can't do this, regardless of which hypervisor is attached. | 14:12 |
jaypipes | mdbooth: it's the "live" part that Cinder can't do, yes? | 14:12 |
mriedem | i must be missing something but i don't see any reply to jaypipes from mdbooth in the ML | 14:12 |
mdbooth | jaypipes: No, that's just an optimisation. | 14:12 |
mdbooth | jaypipes: If anything is attached, Cinder needs to ask Nova for help. | 14:12 |
mdbooth | Nova could just pause, copy, unpause | 14:12 |
kashyap | jaypipes: To your "swap volume" denoting copy operation, you wouldn't know it, unless you look at the Nova _swap_volume() method | 14:12 |
jaypipes | mdbooth: then I don't understand what is different between the pause VM, volume migrate, start VM. | 14:13 |
mriedem | i wonder when k8s is going to implement swap volume for cinder to use | 14:13 |
mriedem | that will be awesome | 14:13 |
kashyap | jaypipes: And then, map it to libvirt's blockRebase() API, which is called by _swap_volume(). | 14:13 |
mdbooth | jaypipes: How would you prevent unpause during the volume migrate? | 14:13 |
jaypipes | kashyap: no... unless you look at the LIBVIRT driver _swap_volume() method. | 14:13 |
jaypipes | kashyap: which is my point. | 14:13 |
kashyap | jaypipes: That's what I said :-) | 14:13 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | 14:13 |
*** shaohe_feng has joined #openstack-nova | 14:13 | |
jaypipes | kashyap: no, you said "the Nova _swap_volume() method". | 14:14 |
kashyap | jaypipes: Yes, you're right - Nova's libvirt driver. | 14:14 |
*** esberglu has joined #openstack-nova | 14:14 | |
jaypipes | mdbooth: it's an administrative action that involves multiple orchestrated pieces. | 14:14 |
mdbooth | jaypipes: Note that we're redefining swap_volume here, which is a bit out of scope. My concern relates to multiattach. | 14:15 |
*** felipemonteiro has quit IRC | 14:15 | |
jaypipes | mdbooth: and just because libvirt/QEMU has some magic foo to do all that orchestration in a (sometimes) live fashion, we've made the Compute API look like the libvirt/QEMU API and embedded expectations of full coordination/orchestration into our API. and that's wrong IMHO. | 14:15 |
*** MultipleCrashes has joined #openstack-nova | 14:15 | |
mdbooth | However, there are 2 parts to it. | 14:16 |
MultipleCrashes | Please have a look at this review and leave a feedback : https://review.openstack.org/#/c/570370/ | 14:16 |
jaypipes | mdbooth: yes, I understand your concern about multiattach and artom's suggestion of just barfing back a 400 Bad Request when any of the source or dest volumes in the swap operation are multiattach is a perfectly reasonable "solution" to that. | 14:16 |
mdbooth | If we removed swap volume and implemented this with pause/resume, we would still need an additional api in nova to 'lock' the pause. | 14:16 |
jaypipes | mdbooth: I'm just annoyed at everything, sorry. | 14:17 |
mdbooth | Otherwise, orchestration or not, a concurrent operation could unpause it. | 14:17 |
artom | jaypipes, would you like a lawn and some kids to yell at? ;) | 14:17 |
jaypipes | mdbooth: precisely. which is why it belongs in an orchestration layer, not the virt driver layer :) | 14:17 |
mdbooth | Also, customers who don't like 6 hours of downtime would be grumpy. | 14:17 |
jaypipes | artom: :) | 14:17 |
mdbooth | jaypipes: But you *can't* orchestrate an *inability* for somebody else to do something without a new api. | 14:18 |
mdbooth | Your orchestrator can pause, migrate, unpause | 14:18 |
mriedem | mdbooth: so your use case in the ML reply to jay was that people are doing retype to get from old shitty storage to new shiny storage, | 14:18 |
mriedem | but that's still initiated through cinder's retype API | 14:18 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459 | 14:18 |
mriedem | what's the use case for using the swap volume API directly w/o cinder? | 14:18 |
mdbooth | But it can't prevent my user, frustrated that his data has been unavailable for 6 hours, from unpausing | 14:18 |
mdbooth | mriedem: No idea. I don't think it matters here, though. | 14:19 |
mriedem | i think it matters to understand how people are using or abusing the APIs | 14:20 |
mriedem | os-server-external-events is a public REST API which i could call directly with admin creds, i don't know why i would, but i can | 14:20 |
mriedem | doesn't mean i should | 14:20 |
jaypipes | mriedem: ++ | 14:20 |
mdbooth | mriedem: Agreed. I don't think it's specific to this case, though. | 14:20 |
*** heyongli has quit IRC | 14:20 | |
mdbooth | mriedem: It doesn't impact the multiattach thing. | 14:21 |
artom | mriedem, I think the difference here is that, calling os-server-external-events doesn't potentially corrupt data | 14:21 |
*** heyongli has joined #openstack-nova | 14:21 | |
mriedem | you guys are missing the point | 14:21 |
jaypipes | mdbooth: BTW, I'm not insinuating that you personally have anything to do with this swap_volume() code. Just pointing out I don't think this particular API call belongs in the compute API. | 14:21 |
artom | For instance, if we discovered that calling it with a certain event caused a VM to go down (suspend your disbelief, work with me) | 14:21 |
artom | We'd fix that, regardless of how silly the "use case" (or lack thereof) was | 14:21 |
bhagyashri_s | efried, jaypipes, bauzas: Hi, Addressed review comments on https://review.openstack.org/#/c/560459 and replied to few comments. Request to review the same. Thank you in advanced :) | 14:21 |
mdbooth | jaypipes: Don't get me wrong, I've wasted many, many curse words on swap_volume. However, I don't see how you can implement this without something like it. | 14:22 |
*** munimeha1 has joined #openstack-nova | 14:22 | |
*** shaohe_feng has quit IRC | 14:22 | |
*** felipemonteiro_ has quit IRC | 14:22 | |
mriedem | mdbooth: have you opened a nova bug for this yet? | 14:22 |
*** shaohe_feng has joined #openstack-nova | 14:23 | |
*** felipemonteiro has joined #openstack-nova | 14:23 | |
mdbooth | mriedem: No, but I will. I was kinda hoping somebody, either Nova or Cinder, would pipe up with a simple fix. | 14:24 |
mriedem | mdbooth: i can but wanted a but quick | 14:24 |
artom | mdbooth, I mean, I did? | 14:25 |
artom | Return a 400? | 14:25 |
mdbooth | Oh, I think you're saying that the direct call use case is relevant here because it's all that's stopping us from implementing this in cinder? | 14:25 |
artom | jaypipes seems on board with it | 14:25 |
mdbooth | In which case, yeah. | 14:25 |
mriedem | mdbooth: yes | 14:26 |
mriedem | i would prefer to just put the blocker in cinder | 14:26 |
mriedem | reasons are in my ML reply | 14:26 |
mriedem | 1. source of truth so less racy and 2. no rollbacks for retype or live migration for volumes | 14:26 |
* mdbooth is going to take a few minutes for coffee and ML reading... | 14:26 | |
artom | mriedem, so Cinder would barf if we "fix" it *just* in Nova, right? | 14:26 |
openstackgerrit | Jan Gutter proposed openstack/nova master: Add support for vrouter HW offloads https://review.openstack.org/572082 | 14:26 |
mriedem | artom: yes, but way late | 14:26 |
smcginnis | I haven't been following the discussion at all, but if we want to block something in cinder, make sure it makes sense for use cases outside of nova. | 14:27 |
mriedem | and then you better pray that cinder rolls back properly | 14:27 |
mriedem | smcginnis: you probably missed my k8s snark earlier | 14:27 |
*** pchavva has quit IRC | 14:27 | |
artom | mriedem, so we'd have to 1. block it in cinder early on 2. block it in nova for the ludicrous direct-call case | 14:27 |
mriedem | how would a volume live migratoin / retype work with k8s when k8s doesn't implement a swap volume callback API? | 14:27 |
mriedem | artom: i guess | 14:27 |
mriedem | i can hack something up for the nova side blocker | 14:27 |
artom | And I then I don't think we'd need a "3. make cinder handle 2" because we'd never get there | 14:28 |
*** AlexeyAbashkin has quit IRC | 14:28 | |
smcginnis | mriedem + k8s == snark :) | 14:28 |
*** armaan has quit IRC | 14:30 | |
*** heyongli has quit IRC | 14:31 | |
*** heyongli has joined #openstack-nova | 14:31 | |
*** sambetts|afk is now known as sambetts | 14:31 | |
*** armaan has joined #openstack-nova | 14:32 | |
*** shaohe_feng has quit IRC | 14:32 | |
*** mlavalle has joined #openstack-nova | 14:32 | |
*** shaohe_feng has joined #openstack-nova | 14:33 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Block swap volume with multiattach volumes https://review.openstack.org/572790 | 14:33 |
mriedem | there ^ - afk for a couple of hours for a kid's school thing | 14:33 |
dansmith | mriedem: back for cells meeting? | 14:34 |
artom | mriedem, https://bugzilla.redhat.com/show_bug.cgi?id=1446446 <--- downstream bug with direct call to volume update | 14:35 |
openstack | artom: Error: Could not parse XML returned by bugzilla.redhat.com: Unknown host. (https://bugzilla.redhat.com/show_bug.cgi?id=1446446&ctype=xml) | 14:36 |
*** AlexeyAbashkin has joined #openstack-nova | 14:36 | |
artom | openstack, ok :) | 14:36 |
artom | (It just took me forever to find it) | 14:36 |
*** felipemonteiro has quit IRC | 14:37 | |
*** felipemonteiro has joined #openstack-nova | 14:37 | |
*** sar has joined #openstack-nova | 14:37 | |
*** heyongli has quit IRC | 14:41 | |
*** heyongli has joined #openstack-nova | 14:41 | |
mnaser | dansmith: regarding the build failure limit change, nova does include all weighers by default, right? | 14:41 |
dansmith | mnaser: yup | 14:41 |
dansmith | mnaser: you adjust the weight to make a weigher a no-op | 14:42 |
mnaser | dansmith: cool, with that, its lgtm, there are a few copy-paste thing but it's not a big deal, only if you have to do another revision you can fix it i guess | 14:42 |
*** shaohe_feng has quit IRC | 14:42 | |
dansmith | mnaser: ah, not copy-paste, but "mriedem decided he wanted to s/boot/build/" and I missed one :) | 14:43 |
*** sahid has joined #openstack-nova | 14:43 | |
dansmith | oh, | 14:43 |
dansmith | the ram one is copy pasta yeah | 14:43 |
*** shaohe_feng has joined #openstack-nova | 14:43 | |
mnaser | :P | 14:43 |
dansmith | I'll update so it's clean since this is backportable | 14:44 |
dansmith | plus nobody has really looked yet :/ | 14:44 |
*** vabada has joined #openstack-nova | 14:44 | |
mnaser | your call :) | 14:45 |
dansmith | mnaser: | 14:45 |
openstackgerrit | Dan Smith proposed openstack/nova master: Change consecutive build failure limit to a weigher https://review.openstack.org/572195 | 14:45 |
mnaser | dansmith: awesome, thank you very much for your work on that | 14:46 |
dansmith | np | 14:46 |
*** r-daneel has quit IRC | 14:46 | |
*** felipemonteiro_ has joined #openstack-nova | 14:50 | |
*** itlinux has quit IRC | 14:50 | |
*** heyongli has quit IRC | 14:51 | |
*** heyongli has joined #openstack-nova | 14:51 | |
*** MultipleCrashes has quit IRC | 14:52 | |
*** shaohe_feng has quit IRC | 14:53 | |
*** Bhujay has quit IRC | 14:53 | |
*** felipemonteiro has quit IRC | 14:53 | |
efried | bauzas, jaypipes: I think https://review.openstack.org/#/c/560459/ is ready when you get a chance. I'm not +2ing because I contributed. | 14:54 |
bauzas | ok | 14:54 |
*** dave-mcc_ has quit IRC | 14:56 | |
*** shaohe_feng has joined #openstack-nova | 14:57 | |
*** rodolof has quit IRC | 14:58 | |
*** tbachman has quit IRC | 14:58 | |
*** rodolof has joined #openstack-nova | 14:59 | |
*** heyongli has quit IRC | 15:01 | |
sahid | stephenfin: thanks for the reviex on https://review.openstack.org/#/c/553072/ | 15:01 |
*** heyongli has joined #openstack-nova | 15:02 | |
*** pchavva has joined #openstack-nova | 15:02 | |
*** rodolof has quit IRC | 15:03 | |
*** shaohe_feng has quit IRC | 15:03 | |
*** rodolof has joined #openstack-nova | 15:03 | |
*** shaohe_feng has joined #openstack-nova | 15:04 | |
bauzas | dansmith: just saw https://review.openstack.org/#/c/572195/ | 15:04 |
bauzas | dansmith: fortunately to https://review.openstack.org/#/c/572195/4/nova/scheduler/host_manager.py, it will work with queens computes | 15:05 |
*** tbachman has joined #openstack-nova | 15:05 | |
stephenfin | sahid: no problem | 15:05 |
dansmith | bauzas: right that's the point | 15:05 |
bauzas | dansmith: but https://review.openstack.org/#/c/572195/4/nova/conf/scheduler.py is asking for a default number that's huge | 15:06 |
*** dave-mccowan has joined #openstack-nova | 15:06 | |
dansmith | bauzas: did you read the justification for that number in the commit message? | 15:06 |
bauzas | sec, I maybe missed it | 15:06 |
bauzas | if so, my bad | 15:06 |
*** pchavva has quit IRC | 15:07 | |
*** ratailor has quit IRC | 15:08 | |
dansmith | if it's wrong, please do call it out, but it _was_ an intentional plan :) | 15:10 |
bauzas | dansmith: /me looking at the related bug to understand the problem | 15:10 |
*** artom has quit IRC | 15:11 | |
*** heyongli has quit IRC | 15:12 | |
dansmith | the first paragraph pretty much sums it up, but the related bug is also good context | 15:12 |
*** heyongli has joined #openstack-nova | 15:12 | |
*** shaohe_feng has quit IRC | 15:13 | |
*** bigdogstl has joined #openstack-nova | 15:13 | |
*** AlexeyAbashkin has quit IRC | 15:14 | |
*** shaohe_feng has joined #openstack-nova | 15:14 | |
bauzas | dansmith: humpf, I wasn't knowing https://github.com/openstack/nova/commit/f93f675a :) | 15:14 |
dansmith | really? okay :) | 15:15 |
bauzas | yeah, really | 15:15 |
dansmith | you were in boston, no? | 15:15 |
bauzas | ok, so now I understand why we want to shuffle computes per number of failed builds | 15:15 |
*** AlexeyAbashkin has joined #openstack-nova | 15:16 | |
bauzas | dansmith: yup, sorry but I don't remember that discussion :( | 15:16 |
openstackgerrit | Curt Moore proposed openstack/nova-specs master: Add spec for downloading images via RBD https://review.openstack.org/572805 | 15:16 |
dansmith | okay | 15:16 |
bauzas | at least, I now understand the problem | 15:16 |
*** bigdogstl has quit IRC | 15:17 | |
*** imacdonn has quit IRC | 15:20 | |
*** imacdonn has joined #openstack-nova | 15:20 | |
*** tbachman has quit IRC | 15:21 | |
bauzas | dansmith: in general, we said a couple of times to operators to randomize more the computes by using host_subset_size or shuffle_best_same_weighed_hosts | 15:21 |
bauzas | dansmith: so that a new instance was asking for a separate compute | 15:21 |
*** ttsiouts has quit IRC | 15:21 | |
dansmith | yeah, but people that want strict packing might not want that | 15:21 |
bauzas | dansmith: agreed | 15:21 |
jaypipes | efried: done. | 15:21 |
bauzas | asking to pack is a problem | 15:21 |
efried | jaypipes: thanks | 15:21 |
bauzas | dansmith: what I was more thinking was maybe to deprecate one of the two options I mentioned | 15:22 |
*** heyongli has quit IRC | 15:22 | |
bauzas | dansmith: and recommend using your own weigher | 15:22 |
*** janki has quit IRC | 15:22 | |
*** heyongli has joined #openstack-nova | 15:22 | |
bauzas | because once we merge your weigher, we'll get 3 opts for quite the same concern | 15:22 |
*** ttsiouts has joined #openstack-nova | 15:22 | |
dansmith | um, why? | 15:23 |
dansmith | I don't think those overlap | 15:23 |
bauzas | dansmith: I think shuffle_best_same_weighed_hosts overlaps | 15:23 |
*** pcaruana has quit IRC | 15:23 | |
*** shaohe_feng has quit IRC | 15:23 | |
*** Luzi has quit IRC | 15:23 | |
dansmith | with my weigher? | 15:24 |
bauzas | dansmith: it's two different implementations | 15:24 |
dansmith | I don't get that | 15:24 |
bauzas | but at the end, we want to make sure we pack correctly | 15:24 |
dansmith | without this weigher, hosts that have been failing will be considered at the same score as those that haven't.. shuffle just changes the likelihood you'll land on one of those, modulo the subset size | 15:25 |
dansmith | with my weigher they're likely not in consideration, moving your chance closer to 100% | 15:25 |
*** tbachman has joined #openstack-nova | 15:25 | |
bauzas | right, that's my point | 15:26 |
bauzas | asking to shuffle is just because you want to make sure you go elsewhere | 15:26 |
dansmith | so, without this weigher, | 15:26 |
dansmith | lets say your subset size is 3 | 15:26 |
dansmith | and shuffle is turned on | 15:26 |
dansmith | if you have a few hosts that are empty, they score high. if one of those is misconfigured, | 15:27 |
dansmith | your chance of booting is 66% | 15:27 |
*** r-daneel has joined #openstack-nova | 15:27 | |
dansmith | with this weigher, you'd get two empty hosts, plus one that has stuff on it in the subset | 15:27 |
bauzas | right, I'm saying your weigher improves the changes | 15:27 |
bauzas | chances* | 15:27 |
* dansmith is so confused | 15:27 | |
bauzas | that's why I don't see the need for shuffle_best_same_weighed_hosts | 15:28 |
bauzas | I'm asking to deprecate shuffle_best_same_weighed_hosts | 15:28 |
*** shaohe_feng has joined #openstack-nova | 15:28 | |
bauzas | because this option just does a random shuffle | 15:28 |
dansmith | but, in the absence of failing hosts, shuffle still helps to balance concurrent boot traffic right? | 15:28 |
dansmith | I don't see shuffle as being used for avoiding faults at all | 15:29 |
*** rajinir has joined #openstack-nova | 15:29 | |
*** tbachman has quit IRC | 15:31 | |
bauzas | maybe my concern is just that I dislike this shuffle option :) | 15:32 |
*** heyongli has quit IRC | 15:32 | |
bauzas | if you want to spread on a specific metric, just ask for a weigher using that metric | 15:32 |
*** heyongli has joined #openstack-nova | 15:32 | |
bauzas | instead of just hoping you'll get spread for free | 15:32 |
bauzas | anyway | 15:32 |
*** zhaochao has quit IRC | 15:33 | |
*** shaohe_feng has quit IRC | 15:34 | |
*** tbachman has joined #openstack-nova | 15:34 | |
*** zhaochao has joined #openstack-nova | 15:34 | |
dansmith | shuffle is purely to avoid super strict packing right? | 15:34 |
*** shaohe_feng has joined #openstack-nova | 15:34 | |
dansmith | if you have big computes and small instances, | 15:34 |
dansmith | you might build 100 instances in the same exact compute before it's completely full and you move onto the next bucket in the line | 15:35 |
dansmith | shuffle seems good for just saying "we generally want packing, but not suuuuuper strict" | 15:35 |
*** itlinux has joined #openstack-nova | 15:36 | |
*** germs has joined #openstack-nova | 15:37 | |
*** germs has quit IRC | 15:37 | |
*** germs has joined #openstack-nova | 15:37 | |
*** germs has quit IRC | 15:37 | |
*** germs has joined #openstack-nova | 15:38 | |
*** germs has quit IRC | 15:38 | |
*** germs has joined #openstack-nova | 15:38 | |
bauzas | dansmith: "life, uh, always finds a way". Replace "life" by "weigher" and you'll get my mind here :) | 15:38 |
*** felipemonteiro_ has quit IRC | 15:38 | |
bauzas | anyway, back to review | 15:39 |
*** heyongli has quit IRC | 15:42 | |
*** heyongli has joined #openstack-nova | 15:43 | |
openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Update multiple fixed-IPs support with services field https://review.openstack.org/572814 | 15:43 |
*** shaohe_feng has quit IRC | 15:44 | |
*** shaohe_feng has joined #openstack-nova | 15:45 | |
openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Standardize CPU resource tracking https://review.openstack.org/555081 | 15:46 |
*** pchavva has joined #openstack-nova | 15:47 | |
*** andreas_s has quit IRC | 15:48 | |
*** janki has joined #openstack-nova | 15:49 | |
*** chyka has joined #openstack-nova | 15:49 | |
*** vabada has quit IRC | 15:50 | |
*** itlinux has quit IRC | 15:52 | |
*** heyongli has quit IRC | 15:53 | |
*** heyongli has joined #openstack-nova | 15:53 | |
*** shaohe_feng has quit IRC | 15:54 | |
*** fragatina has joined #openstack-nova | 15:55 | |
mriedem | dansmith: for the follow up https://review.openstack.org/#/c/572814/ | 15:55 |
*** shaohe_feng has joined #openstack-nova | 15:55 | |
dansmith | mriedem: don't we have to wait two years first? | 15:56 |
dansmith | like fine aged cheddar | 15:56 |
mriedem | artom linked a bz i don't have access to | 15:57 |
dansmith | mriedem: I looked and I don't think it has any useful context in it | 16:00 |
mriedem | ok | 16:00 |
mriedem | i'll proceed with todo cleanupification | 16:01 |
openstackgerrit | Merged openstack/nova master: Fix doc nit https://review.openstack.org/572682 | 16:02 |
openstackgerrit | Curt Moore proposed openstack/nova-specs master: Add spec for downloading images via RBD https://review.openstack.org/572805 | 16:02 |
*** lpetrut has quit IRC | 16:03 | |
*** heyongli has quit IRC | 16:03 | |
*** heyongli has joined #openstack-nova | 16:03 | |
mriedem | johnthetubaguy: jaypipes: bauzas: the vmware live migration spec needs another +2 https://review.openstack.org/#/c/299207/ | 16:03 |
*** homeski has quit IRC | 16:03 | |
bauzas | mriedem: looking | 16:04 |
*** homeski has joined #openstack-nova | 16:04 | |
*** shaohe_feng has quit IRC | 16:04 | |
openstackgerrit | Curt Moore proposed openstack/nova-specs master: Add spec for downloading images via RBD https://review.openstack.org/572805 | 16:05 |
*** shaohe_feng has joined #openstack-nova | 16:06 | |
openstackgerrit | Merged openstack/nova-specs master: Update multiple fixed-IPs support with services field https://review.openstack.org/572814 | 16:07 |
*** pchavva has quit IRC | 16:07 | |
bauzas | mriedem: in pre_live_migration() on the dest compute, we don't manage exceptions raised by the virt driver, right? | 16:08 |
bauzas | mriedem: I think we only do this kind of pre-flight check in the conductor layer | 16:08 |
*** _pewp_ has joined #openstack-nova | 16:08 | |
efried | stephenfin: You around? I need some undumbification about tunnel providers. | 16:10 |
stephenfin | efried: shoot | 16:10 |
efried | stephenfin: https://review.openstack.org/#/c/564440/3/nova/conf/neutron.py | 16:10 |
mriedem | bauzas: pre_live_migration is an rpc call from the source compute | 16:10 |
*** hongbin has quit IRC | 16:10 | |
mriedem | so yes if that fails, the source sets the migration as an error and stops | 16:10 |
mriedem | bauzas: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6077 | 16:11 |
efried | stephenfin: What I thought I saw there is that you can only have your tunnel provider (singular) associated with one NUMA node. Is that right? | 16:11 |
stephenfin | efried: Nope | 16:11 |
stephenfin | I'm using the term tunnel provider but what I really mean is tunnel endpoint | 16:12 |
bauzas | mriedem: ok, so that will work, because I wasn't seeing any exception coming from the virt layer handled by the dest host | 16:12 |
*** homeski has quit IRC | 16:12 | |
stephenfin | namely, the logical NIC where all tunneled traffic ingresses and egresses from the compute node | 16:12 |
*** homeski has joined #openstack-nova | 16:12 | |
bauzas | mriedem: the spec is a bit missing of details, but okay | 16:12 |
stephenfin | logical because it can actually be two or more physical NICs bonded together | 16:12 |
efried | stephenfin: Ah. And those multiple NICs may live on multiple NUMA nodes. | 16:13 |
openstackgerrit | Murali Annamneni proposed openstack/nova master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446643 | 16:13 |
stephenfin | you're only allowed one tunnel endpoint in neutron. Every tunneled network must use that endpoint | 16:13 |
stephenfin | efried: Correct | 16:13 |
stephenfin | Ditto for physnet-type networks (e.g. layer 2) | 16:13 |
*** heyongli has quit IRC | 16:13 | |
*** heyongli has joined #openstack-nova | 16:13 | |
stephenfin | You could have bonded NICs spread across the various host NUMA nodes | 16:14 |
efried | k, I get that. | 16:14 |
*** yamamoto has quit IRC | 16:14 | |
*** yamamoto has joined #openstack-nova | 16:14 | |
efried | Thanks stephenfin. | 16:14 |
*** shaohe_feng has quit IRC | 16:15 | |
stephenfin | no problemo | 16:15 |
*** shaohe_feng has joined #openstack-nova | 16:15 | |
*** yamamoto has quit IRC | 16:17 | |
*** yamamoto has joined #openstack-nova | 16:19 | |
*** zhaochao has quit IRC | 16:19 | |
*** homeski has quit IRC | 16:21 | |
jaypipes | mriedem: +W | 16:21 |
*** Kevin_Zheng has quit IRC | 16:22 | |
*** homeski has joined #openstack-nova | 16:22 | |
*** fragatina has quit IRC | 16:22 | |
bauzas | jaypipes: snap, I litterally +Wd 10 secs after | 16:22 |
bauzas | but I have comments on it | 16:22 |
*** heyongli has quit IRC | 16:23 | |
*** heyongli has joined #openstack-nova | 16:24 | |
*** shaohe_feng has quit IRC | 16:25 | |
mriedem | bauzas: i replied to your comments about pre live migratoin | 16:26 |
mriedem | i don't really understand the concern | 16:26 |
*** ttsiouts has quit IRC | 16:26 | |
*** _pewp_ has quit IRC | 16:26 | |
*** shaohe_feng has joined #openstack-nova | 16:26 | |
bauzas | mriedem: it's a pure implementation thing | 16:27 |
bauzas | mriedem: it's just the fact we pass straight to the compute the migrate_data object that we build by the conductor | 16:27 |
bauzas | or the api | 16:27 |
stephenfin | efried: You're not on openstack-oslo but https://review.openstack.org/#/c/564822/ | 16:27 |
mriedem | "you somehow need to hook into this and call the virt driver so that you can raise the exception." | 16:27 |
mriedem | bauzas: ^ is all over rpc call | 16:27 |
mriedem | so if the virt driver raises an exception, it goes back to the source and fails | 16:28 |
bauzas | mriedem: but the source host never touches it *before* the dest compute amends it | 16:28 |
*** _pewp_ has joined #openstack-nova | 16:28 | |
stephenfin | efried: I spotted the same issue on oslo.logging (I think) earlier this week | 16:28 |
mriedem | bauzas: it doesn't need to | 16:28 |
bauzas | mriedem: sure, but the spec was saying "we pass the source params to the dest so it can fail there" | 16:28 |
mriedem | the dest host populates that information and sends it back to the source | 16:28 |
mriedem | bauzas: where does it say that? | 16:29 |
mriedem | it just says, implement pre_live_migratoin for the vmware driver | 16:29 |
mriedem | like any virt driver that supports live migratoin | 16:29 |
*** abhishekk has joined #openstack-nova | 16:31 | |
bauzas | mriedem: it just says "implement pre migration checks" | 16:33 |
bauzas | oh shit | 16:33 |
bauzas | s/of/to | 16:33 |
*** heyongli has quit IRC | 16:34 | |
openstackgerrit | Merged openstack/nova-specs master: VMware: add support for live migration https://review.openstack.org/299207 | 16:34 |
bauzas | "which will carry foo, bar, baz *of* the target" | 16:34 |
*** heyongli has joined #openstack-nova | 16:34 | |
bauzas | and not "to the target" | 16:34 |
bauzas | mriedem: I think I misunderstood the point | 16:35 |
bauzas | because of s/of/to | 16:35 |
*** shaohe_feng has quit IRC | 16:35 | |
*** homeski has quit IRC | 16:37 | |
*** homeski has joined #openstack-nova | 16:37 | |
*** tbachman has quit IRC | 16:37 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add NUMANetworkInfo object https://review.openstack.org/564439 | 16:38 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440 | 16:38 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_info field https://review.openstack.org/564441 | 16:38 |
*** armaan has quit IRC | 16:38 | |
*** shaohe_feng has joined #openstack-nova | 16:38 | |
*** armaan has joined #openstack-nova | 16:38 | |
bauzas | mriedem: tbc, the exception that would be raised by the virt driver because of incompatible vcluster datasets would be here https://github.com/openstack/nova/blob/47423dfb7fb22924b6b06b99bb571c49bbc54af8/nova/compute/manager.py#L6097-L6101 | 16:39 |
bauzas | but I'm overengineering | 16:39 |
*** smatzek has joined #openstack-nova | 16:39 | |
mriedem | bauzas: maybe you mean this? https://review.openstack.org/#/c/270116/10/nova/virt/vmwareapi/vmops.py@1623 | 16:41 |
*** antosh has quit IRC | 16:41 | |
bauzas | mriedem: *facepalms* | 16:42 |
bauzas | I searched for the implementation | 16:42 |
bauzas | but since they used a fucking wrong gerrit name, I was fcked | 16:42 |
mriedem | i think on the dest, the driver just stores what the cluster and data store is, so the source has to find a compatible esxi host in the same cluster and datastore | 16:42 |
mriedem | if it can't, then it's a failure | 16:43 |
*** sahid has quit IRC | 16:43 | |
*** armaan has quit IRC | 16:43 | |
bauzas | note for myself : read the fscking launchpad whiteboard and don't assume people use gerrit correctly | 16:43 |
mriedem | johnthetubaguy: you can drop the procedural -2 on this now https://review.openstack.org/#/c/270116/ | 16:44 |
*** heyongli has quit IRC | 16:44 | |
*** heyongli has joined #openstack-nova | 16:44 | |
*** antosh has joined #openstack-nova | 16:44 | |
bauzas | mriedem: yup, I just feel https://review.openstack.org/#/c/270116/10/nova/virt/vmwareapi/driver.py@277 needs to return the object they said | 16:45 |
bauzas | but anyway, let's voice on the change | 16:45 |
*** shaohe_feng has quit IRC | 16:45 | |
*** shaohe_feng has joined #openstack-nova | 16:46 | |
*** abhishekk has quit IRC | 16:47 | |
mriedem | bauzas: that's this https://review.openstack.org/#/c/270116/10/nova/virt/vmwareapi/driver.py@304 | 16:50 |
mriedem | it's the same object | 16:50 |
mriedem | conductor calls check_can_live_migrate_destination to get migrate_data, passes it to live_migration, which passes it to pre_live_migration, which passes it bakc | 16:51 |
mriedem | *back | 16:51 |
mriedem | it's like the live migration hot potato | 16:51 |
*** ttsiouts has joined #openstack-nova | 16:51 | |
bauzas | yeah, it's just yet another way of doing the same check | 16:52 |
*** smatzek has quit IRC | 16:52 | |
*** lyan has quit IRC | 16:53 | |
dansmith | mriedem: melwitt: tssurya: assume we're having a cells meeting? | 16:53 |
bauzas | if we were using the result of check_can_live_migrate_dest() on the compute by calling the source compute too before we run the migration, that would save us one roundtrip | 16:53 |
*** heyongli has quit IRC | 16:54 | |
*** heyongli has joined #openstack-nova | 16:54 | |
*** brault has quit IRC | 16:55 | |
*** shaohe_feng has quit IRC | 16:56 | |
openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Use generic metadata term instead of config drive https://review.openstack.org/572837 | 16:56 |
*** tbachman has joined #openstack-nova | 16:56 | |
*** ktibi has joined #openstack-nova | 16:56 | |
*** shaohe_feng has joined #openstack-nova | 16:57 | |
*** AlexeyAbashkin has quit IRC | 16:57 | |
tssurya | dansmith: I guess so ? | 16:58 |
dansmith | let's do it just to say we did | 16:58 |
*** david-lyle has quit IRC | 16:59 | |
*** ttsiouts has quit IRC | 17:03 | |
*** ttsiouts has joined #openstack-nova | 17:04 | |
*** heyongli has quit IRC | 17:04 | |
*** heyongli has joined #openstack-nova | 17:05 | |
*** fragatina has joined #openstack-nova | 17:05 | |
*** derekh has quit IRC | 17:06 | |
*** shaohe_feng has quit IRC | 17:06 | |
*** ttsiouts has quit IRC | 17:08 | |
*** shaohe_feng has joined #openstack-nova | 17:08 | |
*** janki has quit IRC | 17:09 | |
*** yamamoto has quit IRC | 17:10 | |
*** pchavva has joined #openstack-nova | 17:12 | |
*** yamamoto has joined #openstack-nova | 17:13 | |
*** rodolof has quit IRC | 17:13 | |
*** suresh12 has joined #openstack-nova | 17:13 | |
*** rodolof has joined #openstack-nova | 17:13 | |
*** bigdogstl has joined #openstack-nova | 17:14 | |
*** lpetrut has joined #openstack-nova | 17:14 | |
*** heyongli has quit IRC | 17:15 | |
*** heyongli has joined #openstack-nova | 17:15 | |
openstackgerrit | Merged openstack/nova-specs master: Use generic metadata term instead of config drive https://review.openstack.org/572837 | 17:15 |
*** shaohe_feng has quit IRC | 17:16 | |
*** shaohe_feng has joined #openstack-nova | 17:17 | |
*** yamamoto has quit IRC | 17:18 | |
* dansmith notes that mgagne's stats are through the roof today | 17:18 | |
*** bigdogstl has quit IRC | 17:19 | |
mgagne | got 3 nova-specs change merged in 1 day. more in 1 day than 2 whole years? /s =) | 17:19 |
dansmith | heh | 17:19 |
mriedem | lyarwood: btw, connection_info has access_mode but the volume attachment records in cinder have attach_mode | 17:20 |
mriedem | so that's probably where i got it | 17:20 |
mriedem | mgagne: you are now at your quota limit for nova specs | 17:20 |
mgagne | haha, for the next 2 years, see you at OpenStack Xylophone | 17:21 |
dansmith | heh | 17:21 |
mriedem | openstack xenophobe coming to a denver conference center near you! | 17:22 |
mriedem | lyarwood: so on https://review.openstack.org/#/c/552078/ i'm inclined to leave it attach_mode to mirror what's in the cinder api | 17:22 |
mriedem | lyarwood: given that, i'm not sure if there is additional stuff you need for that spec | 17:23 |
mriedem | i also realize i'm talking to a dev/null here and hopefully you'll come out of your FFU dungeon for awhile to reply | 17:23 |
*** ralonsoh has joined #openstack-nova | 17:23 | |
*** ralonsoh has quit IRC | 17:23 | |
*** sapd1 has joined #openstack-nova | 17:24 | |
mgagne | mriedem: I got Queens up with CellsV1 so there is that. I didn't try to boot an instance but services were not complaining. Now working my way up from Mitaka to Queens. | 17:25 |
*** heyongli has quit IRC | 17:25 | |
*** sapd2 has joined #openstack-nova | 17:25 | |
*** heyongli has joined #openstack-nova | 17:25 | |
mriedem | nice | 17:25 |
mriedem | just cell0? | 17:25 |
mriedem | or 1:1 cell mappings? | 17:25 |
*** shaohe_feng has quit IRC | 17:26 | |
*** yamamoto has joined #openstack-nova | 17:28 | |
*** shaohe_feng has joined #openstack-nova | 17:29 | |
*** yamamoto has quit IRC | 17:30 | |
*** namnh has quit IRC | 17:30 | |
*** yamamoto has joined #openstack-nova | 17:30 | |
*** gyee has joined #openstack-nova | 17:30 | |
*** lyan has joined #openstack-nova | 17:32 | |
*** edmondsw has quit IRC | 17:32 | |
*** suresh12 has quit IRC | 17:32 | |
mgagne | cell0 since it's required and old setup with cells.json | 17:33 |
*** lpetrut has quit IRC | 17:33 | |
*** suresh12 has joined #openstack-nova | 17:34 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova-specs master: Handling a down cell https://review.openstack.org/557369 | 17:35 |
*** heyongli has quit IRC | 17:35 | |
*** heyongli has joined #openstack-nova | 17:35 | |
*** shaohe_feng has quit IRC | 17:37 | |
tssurya | mriedem, melwitt, dansmith: ^^ | 17:37 |
*** jamiec_ has quit IRC | 17:37 | |
*** dpawlik has joined #openstack-nova | 17:37 | |
tssurya | jaypipes: whenever you have the time would appreciate your expert opinion on https://review.openstack.org/#/c/560042/ | 17:38 |
tssurya | thanks in advance | 17:38 |
*** jamiec has joined #openstack-nova | 17:38 | |
*** shaohe_feng has joined #openstack-nova | 17:38 | |
*** jpena is now known as jpena|off | 17:40 | |
*** dklyle has joined #openstack-nova | 17:40 | |
dansmith | jaypipes: yeah I cc'd you on the above a couple weeks ago looking for your comment from a db perspective, so would appreciate a read from you | 17:41 |
*** dpawlik has quit IRC | 17:42 | |
*** tssurya has quit IRC | 17:43 | |
*** yamamoto_ has joined #openstack-nova | 17:43 | |
openstackgerrit | Curt Moore proposed openstack/nova-specs master: Add spec for downloading images via RBD https://review.openstack.org/572805 | 17:45 |
*** suresh12 has quit IRC | 17:45 | |
*** edmondsw has joined #openstack-nova | 17:45 | |
*** heyongli has quit IRC | 17:45 | |
jaypipes | dansmith, tssurya: k, will do a review. | 17:46 |
*** fragatina has quit IRC | 17:46 | |
*** heyongli has joined #openstack-nova | 17:46 | |
*** fragatina has joined #openstack-nova | 17:46 | |
*** tidwellr has quit IRC | 17:46 | |
*** shaohe_feng has quit IRC | 17:47 | |
*** yamamoto has quit IRC | 17:47 | |
*** tidwellr has joined #openstack-nova | 17:47 | |
*** rodolof has quit IRC | 17:47 | |
*** shaohe_feng has joined #openstack-nova | 17:48 | |
*** lpetrut has joined #openstack-nova | 17:48 | |
*** tssurya has joined #openstack-nova | 17:48 | |
*** sapd1 has quit IRC | 17:48 | |
*** dave-mccowan has quit IRC | 17:52 | |
*** amoralej is now known as amoralej|off | 17:54 | |
*** tssurya has quit IRC | 17:55 | |
openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads https://review.openstack.org/567148 | 17:55 |
*** rodolof has joined #openstack-nova | 17:55 | |
*** heyongli has quit IRC | 17:56 | |
*** heyongli has joined #openstack-nova | 17:56 | |
*** shaohe_feng has quit IRC | 17:57 | |
*** pchavva has quit IRC | 17:57 | |
*** shaohe_feng has joined #openstack-nova | 17:59 | |
*** pchavva has joined #openstack-nova | 17:59 | |
openstackgerrit | Jeremy Freudberg proposed openstack/nova master: DNM Revert "Use ksa session for cinder microversion check" https://review.openstack.org/572853 | 17:59 |
*** suresh12 has joined #openstack-nova | 17:59 | |
*** heyongli has quit IRC | 18:06 | |
*** heyongli has joined #openstack-nova | 18:07 | |
*** boris_42_ has joined #openstack-nova | 18:07 | |
*** shaohe_feng has quit IRC | 18:07 | |
*** shaohe_feng has joined #openstack-nova | 18:09 | |
*** pchavva has quit IRC | 18:14 | |
*** heyongli has quit IRC | 18:16 | |
*** heyongli has joined #openstack-nova | 18:16 | |
*** shaohe_feng has quit IRC | 18:18 | |
*** yamamoto_ has quit IRC | 18:18 | |
*** shaohe_feng has joined #openstack-nova | 18:18 | |
*** cdent has joined #openstack-nova | 18:21 | |
*** jmlowe has quit IRC | 18:23 | |
*** heyongli has quit IRC | 18:26 | |
*** heyongli has joined #openstack-nova | 18:27 | |
*** yamamoto has joined #openstack-nova | 18:28 | |
*** shaohe_feng has quit IRC | 18:28 | |
*** shaohe_feng has joined #openstack-nova | 18:29 | |
*** rajinir has quit IRC | 18:29 | |
*** dtantsur is now known as dtantsur|afk | 18:30 | |
*** pchavva has joined #openstack-nova | 18:30 | |
*** yamahata has quit IRC | 18:31 | |
*** hongbin has joined #openstack-nova | 18:32 | |
*** yamamoto has quit IRC | 18:32 | |
openstackgerrit | Merged openstack/nova master: Add granular placement policy rules for aggregates https://review.openstack.org/570621 | 18:32 |
*** itlinux has joined #openstack-nova | 18:33 | |
*** yamamoto has joined #openstack-nova | 18:36 | |
*** heyongli has quit IRC | 18:37 | |
*** heyongli has joined #openstack-nova | 18:37 | |
*** sambetts is now known as sambetts|afk | 18:38 | |
*** shaohe_feng has quit IRC | 18:38 | |
*** shaohe_feng has joined #openstack-nova | 18:39 | |
*** dave-mccowan has joined #openstack-nova | 18:40 | |
*** tssurya has joined #openstack-nova | 18:44 | |
*** heyongli has quit IRC | 18:47 | |
*** heyongli has joined #openstack-nova | 18:47 | |
*** shaohe_feng has quit IRC | 18:48 | |
*** mgoddard has quit IRC | 18:49 | |
*** namnh has joined #openstack-nova | 18:49 | |
*** shaohe_feng has joined #openstack-nova | 18:50 | |
*** lyan has quit IRC | 18:50 | |
*** yamahata has joined #openstack-nova | 18:51 | |
*** lyan has joined #openstack-nova | 18:53 | |
*** namnh has quit IRC | 18:54 | |
*** mvk has quit IRC | 18:54 | |
jaypipes | dansmith, tssurya: k, reviewed. sorry for the delay on that. | 18:56 |
tssurya | jaypipes: thanks,! | 18:56 |
*** heyongli has quit IRC | 18:57 | |
*** pchavva has quit IRC | 18:57 | |
*** heyongli has joined #openstack-nova | 18:57 | |
jaypipes | tssurya: np. mriedem's heal_allocations patch is a really good example of doing that batch processing stuff. | 18:58 |
*** shaohe_feng has quit IRC | 18:59 | |
tssurya | absolutely! I will use similar logic then :) | 18:59 |
*** suresh12 has quit IRC | 18:59 | |
jaypipes | tssurya: cool. just remember the key to that is to ensure you pass a sort_key consistently so you get a stable paging system. | 18:59 |
mriedem | jaypipes: btw, i've got a +2 on heal_allocatoins now if you want to send it home | 19:00 |
tssurya | jaypipes: ack | 19:00 |
jaypipes | mriedem: well ahead of you :) | 19:00 |
jaypipes | mriedem: it's already off to the sarlacc pits. | 19:00 |
jaypipes | ok, now on to jangutter's spec. | 19:01 |
jangutter | jaypipes: that's not the sarlacc pits, that's R'lyeh. | 19:02 |
jaypipes | hehe | 19:02 |
*** shaohe_feng has joined #openstack-nova | 19:02 | |
mriedem | just aim a litter higher | 19:03 |
*** jmlowe has joined #openstack-nova | 19:03 | |
mriedem | dansmith: are you following all of this hot star wars action? | 19:03 |
*** jmlowe has quit IRC | 19:04 | |
jaypipes | mriedem: really? I mean, I'm in Florida and was able to feel dansmith's eyeroll over here. | 19:04 |
jaypipes | :) | 19:04 |
*** r-daneel_ has joined #openstack-nova | 19:04 | |
*** r-daneel has quit IRC | 19:04 | |
*** r-daneel_ is now known as r-daneel | 19:04 | |
*** jmlowe has joined #openstack-nova | 19:05 | |
dansmith | um yeah | 19:05 |
jaypipes | hehe | 19:05 |
dansmith | not a huge star wars fan | 19:05 |
*** heyongli has quit IRC | 19:07 | |
*** heyongli has joined #openstack-nova | 19:08 | |
*** shaohe_feng has quit IRC | 19:09 | |
*** dave-mccowan has quit IRC | 19:10 | |
*** priteau has quit IRC | 19:11 | |
*** shaohe_feng has joined #openstack-nova | 19:11 | |
*** dave-mccowan has joined #openstack-nova | 19:11 | |
*** Tom-Tom has quit IRC | 19:14 | |
*** bigdogstl has joined #openstack-nova | 19:15 | |
*** cdent has quit IRC | 19:18 | |
*** heyongli has quit IRC | 19:18 | |
*** rajinir has joined #openstack-nova | 19:18 | |
*** heyongli has joined #openstack-nova | 19:18 | |
*** shaohe_feng has quit IRC | 19:19 | |
*** bigdogstl has quit IRC | 19:19 | |
*** shaohe_feng has joined #openstack-nova | 19:20 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583 | 19:24 |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583 | 19:24 |
*** bigdogstl has joined #openstack-nova | 19:27 | |
openstackgerrit | Doug Hellmann proposed openstack/osc-placement master: fix tox python3 overrides https://review.openstack.org/572917 | 19:28 |
openstackgerrit | Doug Hellmann proposed openstack/os-traits master: fix tox python3 overrides https://review.openstack.org/572921 | 19:28 |
openstackgerrit | Doug Hellmann proposed openstack/os-vif master: fix tox python3 overrides https://review.openstack.org/572922 | 19:28 |
*** heyongli has quit IRC | 19:28 | |
*** heyongli has joined #openstack-nova | 19:28 | |
*** shaohe_feng has quit IRC | 19:29 | |
mriedem | dansmith: want to flip to +W now https://review.openstack.org/#/c/567682/ | 19:30 |
*** shaohe_feng has joined #openstack-nova | 19:30 | |
*** ttsiouts has joined #openstack-nova | 19:31 | |
dansmith | hell yes I do | 19:31 |
mriedem | sayonara fping?! | 19:31 |
*** sapd2 has quit IRC | 19:31 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583 | 19:32 |
mriedem | oolala | 19:32 |
*** bigdogstl has quit IRC | 19:32 | |
*** ttsiouts has quit IRC | 19:34 | |
*** ttsiouts has joined #openstack-nova | 19:34 | |
*** lifeless has joined #openstack-nova | 19:35 | |
efried | mriedem: working on that requirement now - does it need to be removed from test-requirements? | 19:35 |
mriedem | efried: you need to rebase | 19:36 |
mriedem | the sphinx thing should go in doc/requirements.txt | 19:36 |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583 | 19:37 |
efried | mriedem: Did the move ^. Rebase on what? | 19:37 |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583 | 19:38 |
*** rodolof has quit IRC | 19:38 | |
*** heyongli has quit IRC | 19:38 | |
*** heyongli has joined #openstack-nova | 19:38 | |
*** slaweq has quit IRC | 19:39 | |
*** rodolof has joined #openstack-nova | 19:39 | |
*** dpawlik has joined #openstack-nova | 19:39 | |
*** ttsiouts has quit IRC | 19:39 | |
*** issp has joined #openstack-nova | 19:39 | |
*** shaohe_feng has quit IRC | 19:40 | |
*** shaohe_feng has joined #openstack-nova | 19:41 | |
*** slaweq_ has joined #openstack-nova | 19:42 | |
mriedem | efried: the PTI change merged yesterday | 19:43 |
efried | mriedem: ack. I rebased. Thanks. | 19:43 |
*** dpawlik has quit IRC | 19:43 | |
*** ttsiouts has joined #openstack-nova | 19:44 | |
efried | mriedem: actually, looks like I got on top of it the first time, by a whisker. Anyway... | 19:44 |
*** slaweq has joined #openstack-nova | 19:44 | |
*** Tom-Tom has joined #openstack-nova | 19:44 | |
mriedem | le whew | 19:45 |
mriedem | mdbooth: so did you ever report a nova bug? | 19:45 |
mriedem | because i'm ready to push my fix | 19:46 |
mriedem | nvm https://bugs.launchpad.net/nova/+bug/1775418 | 19:46 |
openstack | Launchpad bug 1775418 in OpenStack Compute (nova) "Swap volume of multiattached volume will corrupt data" [Undecided,New] | 19:46 |
*** suresh12 has joined #openstack-nova | 19:46 | |
*** r-daneel has quit IRC | 19:46 | |
*** r-daneel has joined #openstack-nova | 19:47 | |
*** suresh12 has quit IRC | 19:47 | |
*** suresh12 has joined #openstack-nova | 19:47 | |
*** suresh12 has quit IRC | 19:47 | |
*** suresh12 has joined #openstack-nova | 19:48 | |
*** suresh12 has quit IRC | 19:48 | |
*** heyongli has quit IRC | 19:48 | |
*** suresh12 has joined #openstack-nova | 19:49 | |
*** heyongli has joined #openstack-nova | 19:49 | |
*** Tom-Tom has quit IRC | 19:49 | |
*** jmlowe has quit IRC | 19:50 | |
*** shaohe_feng has quit IRC | 19:50 | |
*** shaohe_feng has joined #openstack-nova | 19:50 | |
*** slaweq has quit IRC | 19:51 | |
*** yamamoto has quit IRC | 19:51 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.openstack.org/572790 | 19:52 |
*** dave-mcc_ has joined #openstack-nova | 19:52 | |
*** dave-mccowan has quit IRC | 19:53 | |
*** slaweq has joined #openstack-nova | 19:54 | |
*** slaweq__ has joined #openstack-nova | 19:54 | |
*** slaweq__ has quit IRC | 19:54 | |
*** slaweq has quit IRC | 19:54 | |
*** slaweq has joined #openstack-nova | 19:54 | |
*** slaweq__ has joined #openstack-nova | 19:55 | |
*** slaweq has quit IRC | 19:55 | |
*** slaweq__ has quit IRC | 19:55 | |
*** slaweq has joined #openstack-nova | 19:55 | |
openstackgerrit | Merged openstack/nova master: Add granular policy rules for traits in placement https://review.openstack.org/570625 | 19:55 |
*** slaweq_ has quit IRC | 19:56 | |
*** slaweq has quit IRC | 19:57 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.openstack.org/572790 | 19:57 |
*** yamamoto has joined #openstack-nova | 19:58 | |
*** heyongli has quit IRC | 19:59 | |
*** heyongli has joined #openstack-nova | 19:59 | |
*** shaohe_feng has quit IRC | 20:00 | |
*** shaohe_feng has joined #openstack-nova | 20:01 | |
*** yamamoto has quit IRC | 20:02 | |
*** throwsb1 has quit IRC | 20:06 | |
*** jmlowe has joined #openstack-nova | 20:07 | |
*** heyongli has quit IRC | 20:09 | |
*** heyongli has joined #openstack-nova | 20:09 | |
*** liverpooler has quit IRC | 20:10 | |
*** shaohe_feng has quit IRC | 20:10 | |
*** READ10 has quit IRC | 20:10 | |
*** suresh12 has quit IRC | 20:12 | |
*** shaohe_feng has joined #openstack-nova | 20:12 | |
*** pooja-jadhav has joined #openstack-nova | 20:13 | |
*** pooja_jadhav has quit IRC | 20:13 | |
*** bhagyashri_s has quit IRC | 20:13 | |
*** bhagyashri_s has joined #openstack-nova | 20:13 | |
*** dave-mcc_ has quit IRC | 20:15 | |
*** suresh12 has joined #openstack-nova | 20:15 | |
*** heyongli has quit IRC | 20:19 | |
*** heyongli has joined #openstack-nova | 20:19 | |
*** yamamoto has joined #openstack-nova | 20:20 | |
*** suresh12 has quit IRC | 20:20 | |
*** yamamoto has quit IRC | 20:20 | |
*** shaohe_feng has quit IRC | 20:21 | |
*** shaohe_feng has joined #openstack-nova | 20:22 | |
*** namnh has joined #openstack-nova | 20:22 | |
*** namnh has quit IRC | 20:26 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova-specs master: Handling a down cell https://review.openstack.org/557369 | 20:29 |
*** heyongli has quit IRC | 20:29 | |
*** heyongli has joined #openstack-nova | 20:30 | |
*** shaohe_feng has quit IRC | 20:31 | |
*** shaohe_feng has joined #openstack-nova | 20:32 | |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262 | 20:32 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 20:32 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | 20:32 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 20:32 |
openstackgerrit | Merged openstack/nova master: Add nova-manage placement heal_allocations CLI https://review.openstack.org/565886 | 20:32 |
*** tbachman has quit IRC | 20:34 | |
*** lpetrut has quit IRC | 20:35 | |
*** hemna_ has quit IRC | 20:39 | |
*** heyongli has quit IRC | 20:40 | |
*** heyongli has joined #openstack-nova | 20:40 | |
*** artom has joined #openstack-nova | 20:40 | |
*** shaohe_feng has quit IRC | 20:41 | |
*** chyka has quit IRC | 20:42 | |
*** chyka has joined #openstack-nova | 20:42 | |
*** esberglu_ has joined #openstack-nova | 20:42 | |
*** shaohe_feng has joined #openstack-nova | 20:42 | |
*** tssurya has quit IRC | 20:42 | |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 20:44 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262 | 20:44 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 20:44 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | 20:44 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 20:44 |
*** esberglu has quit IRC | 20:44 | |
* melwitt laments while trying to write a multi-cell functional test | 20:45 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove support for /os-virtual-interfaces REST API https://review.openstack.org/569923 | 20:45 |
*** bpoulos has quit IRC | 20:46 | |
mriedem | melwitt: https://review.openstack.org/#/c/565886/10/nova/tests/functional/test_nova_manage.py | 20:46 |
melwitt | ty | 20:46 |
melwitt | let's see where I'm going wrong here | 20:47 |
*** heyongli has quit IRC | 20:50 | |
*** heyongli has joined #openstack-nova | 20:50 | |
*** shaohe_feng has quit IRC | 20:51 | |
*** suresh12 has joined #openstack-nova | 20:52 | |
*** shaohe_feng has joined #openstack-nova | 20:52 | |
*** esberglu_ has quit IRC | 20:56 | |
*** tbachman has joined #openstack-nova | 20:58 | |
*** tssurya has joined #openstack-nova | 20:58 | |
*** fragatina has quit IRC | 20:58 | |
*** fragatina has joined #openstack-nova | 20:59 | |
*** heyongli has quit IRC | 21:00 | |
openstackgerrit | Merged openstack/nova master: Remove support for /os-fping REST API https://review.openstack.org/567682 | 21:00 |
openstackgerrit | Merged openstack/nova master: Add contributor docs on deprecating and removing compute REST APIs https://review.openstack.org/567687 | 21:01 |
*** heyongli has joined #openstack-nova | 21:01 | |
*** fragatina has quit IRC | 21:01 | |
*** fragatina has joined #openstack-nova | 21:01 | |
*** shaohe_feng has quit IRC | 21:02 | |
*** felipemonteiro has joined #openstack-nova | 21:02 | |
*** tbachman has quit IRC | 21:02 | |
*** shaohe_feng has joined #openstack-nova | 21:02 | |
*** felipemonteiro_ has joined #openstack-nova | 21:03 | |
*** tssurya has quit IRC | 21:03 | |
*** tssurya has joined #openstack-nova | 21:03 | |
*** jmlowe has quit IRC | 21:05 | |
*** issp has quit IRC | 21:06 | |
*** felipemonteiro has quit IRC | 21:07 | |
*** ktibi has quit IRC | 21:08 | |
*** felipemonteiro_ has quit IRC | 21:09 | |
*** edmondsw has quit IRC | 21:09 | |
*** sar has quit IRC | 21:10 | |
*** heyongli has quit IRC | 21:10 | |
*** heyongli has joined #openstack-nova | 21:11 | |
*** tbachman has joined #openstack-nova | 21:11 | |
*** shaohe_feng has quit IRC | 21:12 | |
*** shaohe_feng has joined #openstack-nova | 21:13 | |
*** nicolasbock has quit IRC | 21:14 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove support for /os-virtual-interfaces REST API https://review.openstack.org/569923 | 21:14 |
*** tbachman has quit IRC | 21:15 | |
*** lyan has quit IRC | 21:18 | |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 21:19 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262 | 21:19 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 21:19 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | 21:19 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 21:19 |
*** yamamoto has joined #openstack-nova | 21:20 | |
*** suresh12 has quit IRC | 21:21 | |
*** heyongli has quit IRC | 21:21 | |
*** suresh12 has joined #openstack-nova | 21:21 | |
*** heyongli has joined #openstack-nova | 21:21 | |
*** shaohe_feng has quit IRC | 21:22 | |
*** lifeless_ has joined #openstack-nova | 21:22 | |
*** lifeless has quit IRC | 21:23 | |
*** shaohe_feng has joined #openstack-nova | 21:23 | |
*** ttsiouts has quit IRC | 21:25 | |
*** jmlowe has joined #openstack-nova | 21:25 | |
*** awaugama has quit IRC | 21:26 | |
*** ttsiouts has joined #openstack-nova | 21:26 | |
*** ttsiouts has quit IRC | 21:27 | |
*** ttsiouts has joined #openstack-nova | 21:27 | |
*** lyan has joined #openstack-nova | 21:28 | |
*** bigdogstl has joined #openstack-nova | 21:28 | |
*** heyongli has quit IRC | 21:31 | |
*** heyongli has joined #openstack-nova | 21:31 | |
mriedem | melwitt: not sure if anyone talked to you about this yet, but https://review.openstack.org/#/c/572583/ is likely going to be a major priority to get done in rocky since nested provider stuff is going to be held up on us handling the data migration properly | 21:32 |
*** shaohe_feng has quit IRC | 21:32 | |
mriedem | i think we might want to have a call or something to go over relative priorities for things we really need to get done in the 3rd milestone, because i have a feeling there are a few things floating around | 21:32 |
mriedem | this and handling a down cell are the 2 main ones i'm thinking of | 21:33 |
*** sridharg has quit IRC | 21:33 | |
*** yamamoto has quit IRC | 21:33 | |
*** shaohe_feng has joined #openstack-nova | 21:33 | |
melwitt | I'm aware of the importance of the data migration stuff, for whatever reason I didn't realize the "reshape" spec was about that | 21:33 |
melwitt | yeah, sounds cool to me | 21:34 |
mriedem | i ignored the whole nrp upgrade thread in the ML, just jumped on the call yesterday that led to the spec | 21:35 |
mriedem | tl;dr there will be a new placement api to atomically change the inventory and allocations for a given set of providers/consumers, which we'll run on start of a compute when upgrading to rocky, and we'll also have an offline hook to do that also for FFU | 21:35 |
*** itlinux has quit IRC | 21:36 | |
*** tidwellr has quit IRC | 21:36 | |
melwitt | right. I wasn't around in time for the call but I read through the etherpad afterward | 21:36 |
*** itlinux has joined #openstack-nova | 21:37 | |
*** bigdogstl has quit IRC | 21:37 | |
*** itlinux has quit IRC | 21:37 | |
*** dpawlik has joined #openstack-nova | 21:40 | |
*** bigdogstl has joined #openstack-nova | 21:40 | |
melwitt | my multi-cell func test fail is related to booting with a server group. boot without a group works fine. argh. | 21:41 |
*** heyongli has quit IRC | 21:41 | |
*** heyongli has joined #openstack-nova | 21:41 | |
melwitt | mriedem: a call sounds cool. are you thinking sometime tomorrow morning? since tomorrow is spec freeze | 21:42 |
mriedem | umm | 21:42 |
mriedem | maybe | 21:42 |
mriedem | i haven't gone through efried's spec in detail yet, and need to go through surya's latest revision | 21:42 |
*** shaohe_feng has quit IRC | 21:43 | |
mriedem | friday at least | 21:43 |
mriedem | once the spec freeze dust settles | 21:43 |
mriedem | and i'd consider these spec freeze exception candidates | 21:43 |
*** shaohe_feng has joined #openstack-nova | 21:43 | |
melwitt | okay, that's what I was about to ask | 21:43 |
melwitt | if we needed to do all the things by EOD tomorrow or if we have some time as exceptions | 21:44 |
mriedem | we can always make exceptions | 21:44 |
*** dpawlik has quit IRC | 21:44 | |
*** antosh has quit IRC | 21:44 | |
melwitt | alright | 21:45 |
dansmith | mriedem: reading the docs for the other weights we have makes me realize we have some that accept negative values | 21:45 |
*** bigdogstl has quit IRC | 21:45 | |
dansmith | so maybe I should make the default -BIGNUMBER and require them to use negative values to properly weigh failures | 21:46 |
artom | mriedem, drive by review on https://review.openstack.org/#/c/572790/, will go to sleep soon (travelling back to my time zone) | 21:46 |
dansmith | I thought they were all zero/positive | 21:46 |
mriedem | dansmith: i know there was an iops one that i saw | 21:46 |
mriedem | that was the only one i saw i guess | 21:46 |
*** dpawlik has joined #openstack-nova | 21:47 | |
dansmith | mriedem: well, what do you think? | 21:47 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 21:48 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | 21:48 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 21:48 |
*** bigdogstl has joined #openstack-nova | 21:48 | |
openstackgerrit | Merged openstack/nova master: Trivial: let internal use only func has _ prefix https://review.openstack.org/572304 | 21:48 |
mriedem | sorry i got artom'ed | 21:50 |
artom | im in ur patch, artoming ur patch | 21:50 |
dansmith | ouch | 21:50 |
mriedem | dansmith: if negative makes more sense then that's fine by me | 21:50 |
melwitt | dansmith: if none of the other weighers auto-negate things, I would think it's more intuitive if it's specified as a negative number. with the thinking that if the normal pattern is "bigger numbers win" then specifying a larger negative number means weight a failure as far less likely to be selected | 21:51 |
mriedem | honestly the weight multipliers break my head | 21:51 |
melwitt | am I thinking about that right? | 21:51 |
dansmith | well, I dunno | 21:51 |
dansmith | melwitt: the way I think of all our weighers, the bigger the number, the more significance it has | 21:51 |
melwitt | I'd want to pick something that fits in best with what's there so that people who already use it don't have to reverse their thinking for that one option | 21:51 |
dansmith | however, there area few where you may want affinity or the opposite, so you can do negative or positive | 21:51 |
*** heyongli has quit IRC | 21:51 | |
melwitt | *already use weights | 21:51 |
*** dpawlik has quit IRC | 21:51 | |
dansmith | you likely never want to prefer hosts that suck, | 21:52 |
*** heyongli has joined #openstack-nova | 21:52 | |
dansmith | so I dunno that it would ever be positive (if we didn't auto-negate) | 21:52 |
mriedem | https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py#L428 is the one i noticed | 21:52 |
mriedem | oh and this https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py#L737 | 21:52 |
mriedem | weight_of_unavailable... | 21:52 |
*** shaohe_feng has quit IRC | 21:53 | |
dansmith | that's a bit different | 21:53 |
*** bigdogstl has quit IRC | 21:53 | |
dansmith | and the iops one can be positive or negative with useful meaning | 21:54 |
melwitt | do the other negative ones ever make sense as positive values? k | 21:54 |
*** rcernin has joined #openstack-nova | 21:54 | |
dansmith | yes, ops defaults negative, but changing it to positive means something useful | 21:54 |
*** shaohe_feng has joined #openstack-nova | 21:54 | |
dansmith | soft_anti_affinity and soft_affinity are both positive, but have opposite meanings | 21:55 |
*** artom has quit IRC | 21:55 | |
dansmith | https://github.com/openstack/nova/blob/master/nova/scheduler/weights/affinity.py#L93-L96 | 21:55 |
melwitt | so if it auto negates ... to make build fails even less likely to schedule toward, you would increase the positive value. that is, make build failures heavier | 21:55 |
dansmith | Just like I do ^ | 21:55 |
dansmith | melwitt: right, if it auto-negates (as it does now) then you raise the number to make build failures more significant (i.e. less likely to be chosen) | 21:56 |
dansmith | I think that anti-affinity one is my proof that I'm okay | 21:56 |
melwitt | yeah, that makes sense to me | 21:56 |
dansmith | it auto-negates because it makes more sense to have a positive value for significance | 21:56 |
melwitt | yeah it's like a coefficient I guess | 21:56 |
mriedem | i left a question in there, but should we put a min value on the multiplier? | 21:57 |
melwitt | I think either way is find, both make sense in their own way | 21:57 |
mriedem | otherwise anything negative weighs failed hosts higher right? | 21:57 |
melwitt | *fine | 21:57 |
dansmith | mriedem: I dunno, if somene wanted to actually select recent fails, they could set it to -BIG, which ... maybe? | 21:57 |
dansmith | if they do auto-disable on their own, | 21:57 |
dansmith | they may want recently-fixed hosts to be super important or something? | 21:57 |
mriedem | via notifications or something | 21:58 |
melwitt | although allowing negative and positive values could make you favor build failures if you mess up, so maybe that is worse | 21:58 |
dansmith | I wouldn't document that, but if you remove the ability, then they can't | 21:58 |
*** tbachman has joined #openstack-nova | 21:58 | |
mriedem | sure | 21:58 |
dansmith | melwitt: right but if you do ^ then maybe you want that | 21:58 |
mriedem | anywho | 21:58 |
mriedem | watcher could have it's own super awesome auto-disabler | 21:58 |
dansmith | I think I'm going to leave it as-is, I convinced myself with the anti-affinity precedent | 21:58 |
*** r-daneel has quit IRC | 21:58 | |
*** r-daneel has joined #openstack-nova | 21:59 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.openstack.org/572790 | 21:59 |
*** ttsiouts has quit IRC | 21:59 | |
*** ttsiouts has joined #openstack-nova | 22:00 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Change consecutive build failure limit to a weigher https://review.openstack.org/572195 | 22:01 |
*** heyongli has quit IRC | 22:02 | |
*** heyongli has joined #openstack-nova | 22:02 | |
*** shaohe_feng has quit IRC | 22:03 | |
*** ttsiouts has quit IRC | 22:04 | |
*** shaohe_feng has joined #openstack-nova | 22:05 | |
*** edmondsw has joined #openstack-nova | 22:06 | |
*** bigdogstl has joined #openstack-nova | 22:09 | |
*** edmondsw has quit IRC | 22:10 | |
*** heyongli has quit IRC | 22:12 | |
*** heyongli has joined #openstack-nova | 22:12 | |
*** shaohe_feng has quit IRC | 22:13 | |
*** munimeha1 has quit IRC | 22:13 | |
*** bigdogstl has quit IRC | 22:14 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.openstack.org/572790 | 22:14 |
*** mlavalle has quit IRC | 22:15 | |
*** shaohe_feng has joined #openstack-nova | 22:15 | |
dansmith | mriedem: oh, those were all in default previously I guess? we should update that doc | 22:15 |
*** rodolof has quit IRC | 22:17 | |
mriedem | i'm sure they are old yeah | 22:17 |
mriedem | but we can cleanup the rest in a follow up | 22:17 |
mriedem | they likely have a rename alias | 22:17 |
dansmith | yeah they do | 22:18 |
dansmith | I didn't even process that as the config section, | 22:18 |
dansmith | I just copied a block and changed the names | 22:18 |
openstackgerrit | Dan Smith proposed openstack/nova master: Change consecutive build failure limit to a weigher https://review.openstack.org/572195 | 22:19 |
*** ttsiouts has joined #openstack-nova | 22:20 | |
*** heyongli has quit IRC | 22:22 | |
*** heyongli has joined #openstack-nova | 22:23 | |
*** shaohe_feng has quit IRC | 22:24 | |
*** jmlowe has quit IRC | 22:24 | |
*** bigdogstl has joined #openstack-nova | 22:25 | |
mriedem | in trumps america, i process it for you | 22:25 |
*** mrjk__ has quit IRC | 22:25 | |
*** mrjk__ has joined #openstack-nova | 22:26 | |
mriedem | i think i'll just fast approve given sylvain was +2 | 22:26 |
mriedem | is that ok? | 22:26 |
melwitt | I was going to +2 it once you were okay with it | 22:26 |
*** shaohe_feng has joined #openstack-nova | 22:26 | |
mriedem | ok all yours | 22:27 |
melwitt | en route to the sarlacc put | 22:28 |
melwitt | *pit | 22:28 |
mriedem | i hope you get that reference | 22:28 |
mriedem | otherwise, for shame | 22:28 |
*** mgariepy has quit IRC | 22:29 | |
mriedem | oh btw, | 22:29 |
melwitt | of course I do! I'm hip like that | 22:29 |
mriedem | dansmith didn't like evil dead 2 | 22:29 |
mriedem | as expected | 22:29 |
dansmith | correction | 22:29 |
melwitt | lol | 22:29 |
dansmith | I didn't like the first 45 minutes of it | 22:29 |
dansmith | I have no idea about the second 45 minutes | 22:29 |
mriedem | did you at least see where he fights his own hand? | 22:30 |
*** mgariepy has joined #openstack-nova | 22:30 | |
dansmith | yeah | 22:30 |
mriedem | well alright then | 22:30 |
mriedem | also, | 22:30 |
mriedem | efried sorta looks like bruce from evil dead | 22:30 |
mriedem | efried: you should take advantage of that on halloween | 22:30 |
*** ttsiouts has quit IRC | 22:30 | |
*** bigdogstl has quit IRC | 22:30 | |
melwitt | *mind blown* | 22:30 |
mriedem | or really any day | 22:30 |
efried | I've actually heard that before. | 22:30 |
mriedem | do you frequently say to your wife, "gimme some sugar, baby"? | 22:31 |
*** ttsiouts has joined #openstack-nova | 22:31 | |
mriedem | if so, you might be ash | 22:31 |
*** tbachman has quit IRC | 22:31 | |
efried | I may have seen those movies once, 20 years ago. They're not auto-memes that just spring forth from my head, I'm afraid. | 22:31 |
mriedem | i saw the first one between slitted fingers when i was probably <10 | 22:32 |
mriedem | so it's burned in there | 22:32 |
mriedem | the granny in the basement | 22:32 |
melwitt | lol | 22:32 |
*** heyongli has quit IRC | 22:32 | |
efried | I can totally quote Finding Nemo, though. | 22:33 |
*** heyongli has joined #openstack-nova | 22:33 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.openstack.org/572790 | 22:33 |
efried | dansmith sorta looks like Marlin | 22:33 |
* melwitt googles | 22:33 | |
dansmith | isn't that an animated fish? | 22:33 |
efried | Aren't you an animated fish? | 22:33 |
dansmith | don't make me fight you | 22:33 |
melwitt | lol | 22:33 |
dansmith | I know car-ah-tay | 22:33 |
efried | With what, your FINS? | 22:34 |
mriedem | hey, fish are people too | 22:34 |
*** shaohe_feng has quit IRC | 22:34 | |
*** lifeless_ has quit IRC | 22:34 | |
*** shaohe_feng has joined #openstack-nova | 22:34 | |
*** lifeless has joined #openstack-nova | 22:34 | |
*** ttsiouts has quit IRC | 22:35 | |
*** heyongli has quit IRC | 22:43 | |
*** heyongli has joined #openstack-nova | 22:43 | |
*** bigdogstl has joined #openstack-nova | 22:43 | |
*** shaohe_feng has quit IRC | 22:44 | |
*** tbachman has joined #openstack-nova | 22:44 | |
*** jmlowe has joined #openstack-nova | 22:45 | |
*** shaohe_feng has joined #openstack-nova | 22:46 | |
*** bigdogstl has quit IRC | 22:50 | |
*** threestrands has joined #openstack-nova | 22:51 | |
*** heyongli has quit IRC | 22:53 | |
*** heyongli has joined #openstack-nova | 22:53 | |
*** tbachman has quit IRC | 22:54 | |
*** shaohe_feng has quit IRC | 22:54 | |
*** shaohe_feng has joined #openstack-nova | 22:56 | |
*** bigdogstl has joined #openstack-nova | 22:58 | |
*** eandersson_ is now known as eandersson | 22:59 | |
*** heyongli has quit IRC | 23:03 | |
*** heyongli has joined #openstack-nova | 23:03 | |
*** tssurya has quit IRC | 23:04 | |
*** shaohe_feng has quit IRC | 23:05 | |
*** shaohe_feng has joined #openstack-nova | 23:06 | |
*** bigdogstl has quit IRC | 23:07 | |
*** rajinir has quit IRC | 23:09 | |
*** heyongli has quit IRC | 23:13 | |
*** heyongli has joined #openstack-nova | 23:14 | |
*** shaohe_feng has quit IRC | 23:15 | |
*** shaohe_feng has joined #openstack-nova | 23:16 | |
*** takashin has joined #openstack-nova | 23:18 | |
*** bigdogstl has joined #openstack-nova | 23:18 | |
*** namnh has joined #openstack-nova | 23:21 | |
*** jmlowe has quit IRC | 23:22 | |
*** lyan has quit IRC | 23:22 | |
*** bigdogstl has quit IRC | 23:22 | |
*** namnh has quit IRC | 23:22 | |
*** namnh has joined #openstack-nova | 23:23 | |
*** heyongli has quit IRC | 23:24 | |
*** jmlowe has joined #openstack-nova | 23:24 | |
*** heyongli has joined #openstack-nova | 23:24 | |
*** bigdogstl has joined #openstack-nova | 23:25 | |
*** shaohe_feng has quit IRC | 23:25 | |
*** shaohe_feng has joined #openstack-nova | 23:26 | |
*** chyka has quit IRC | 23:31 | |
*** tssurya has joined #openstack-nova | 23:31 | |
*** bigdogstl has quit IRC | 23:32 | |
*** hongbin has quit IRC | 23:33 | |
*** heyongli has quit IRC | 23:34 | |
*** heyongli has joined #openstack-nova | 23:34 | |
*** shaohe_feng has quit IRC | 23:35 | |
*** tssurya has quit IRC | 23:36 | |
*** shaohe_feng has joined #openstack-nova | 23:37 | |
*** suresh12 has quit IRC | 23:39 | |
*** bigdogstl has joined #openstack-nova | 23:42 | |
*** Sundar has joined #openstack-nova | 23:42 | |
Sundar | @edleafe, @efried: Please review https://review.openstack.org/#/c/554717/. Hope to get a +1 from any/both of you, or any other Nova expert for that matter. | 23:44 |
*** heyongli has quit IRC | 23:44 | |
*** heyongli has joined #openstack-nova | 23:44 | |
*** shaohe_feng has quit IRC | 23:46 | |
*** lifeless has quit IRC | 23:46 | |
*** lifeless has joined #openstack-nova | 23:46 | |
*** bigdogstl has quit IRC | 23:47 | |
*** r-daneel has quit IRC | 23:47 | |
*** dpawlik has joined #openstack-nova | 23:48 | |
*** shaohe_feng has joined #openstack-nova | 23:48 | |
*** suresh12 has joined #openstack-nova | 23:49 | |
*** Sundar has quit IRC | 23:51 | |
*** dpawlik has quit IRC | 23:53 | |
*** hshiina has joined #openstack-nova | 23:53 | |
*** edmondsw has joined #openstack-nova | 23:54 | |
*** heyongli has quit IRC | 23:54 | |
*** heyongli has joined #openstack-nova | 23:55 | |
*** shaohe_feng has quit IRC | 23:56 | |
*** namnh has quit IRC | 23:56 | |
*** shaohe_feng has joined #openstack-nova | 23:56 | |
*** edmondsw has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!