*** shaohe_feng has quit IRC | 00:00 | |
*** moshele has joined #openstack-nova | 00:00 | |
*** shaohe_feng has joined #openstack-nova | 00:00 | |
*** eharney has quit IRC | 00:04 | |
*** shaohe_feng has quit IRC | 00:10 | |
*** shaohe_feng has joined #openstack-nova | 00:11 | |
*** tetsuro has joined #openstack-nova | 00:14 | |
*** moshele has quit IRC | 00:16 | |
*** dave-mccowan has joined #openstack-nova | 00:19 | |
openstackgerrit | Merged openstack/nova master: add parameter docstring for 'params' to libvirt.guest.Guest.migrate() https://review.openstack.org/571236 | 00:20 |
---|---|---|
*** shaohe_feng has quit IRC | 00:20 | |
*** shaohe_feng has joined #openstack-nova | 00:21 | |
*** suresh12 has joined #openstack-nova | 00:22 | |
*** dave-mccowan has quit IRC | 00:30 | |
*** shaohe_feng has quit IRC | 00:30 | |
*** shaohe_feng has joined #openstack-nova | 00:31 | |
*** salv-orlando has joined #openstack-nova | 00:36 | |
*** edmondsw has joined #openstack-nova | 00:38 | |
*** shaohe_feng has quit IRC | 00:41 | |
*** salv-orlando has quit IRC | 00:41 | |
*** shaohe_feng has joined #openstack-nova | 00:41 | |
*** hshiina|afk is now known as hshiina | 00:42 | |
*** Dinesh_Bhor has joined #openstack-nova | 00:42 | |
*** edmondsw has quit IRC | 00:43 | |
openstackgerrit | Merged openstack/nova master: mirror nova host aggregate members to placement https://review.openstack.org/553597 | 00:45 |
openstackgerrit | Merged openstack/nova master: Remove mox in libvirt/test_driver.py (3) https://review.openstack.org/569320 | 00:45 |
openstackgerrit | Merged openstack/nova master: libvirt: Skip fetching the virtual size of block devices https://review.openstack.org/569062 | 00:46 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (3) https://review.openstack.org/568184 | 00:50 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (4) https://review.openstack.org/568462 | 00:50 |
*** shaohe_feng has quit IRC | 00:51 | |
*** Dinesh_Bhor has quit IRC | 00:51 | |
*** shaohe_feng has joined #openstack-nova | 00:52 | |
*** chyka has joined #openstack-nova | 00:52 | |
*** phuongnh has joined #openstack-nova | 00:55 | |
*** chyka has quit IRC | 00:57 | |
*** Dinesh_Bhor has joined #openstack-nova | 00:58 | |
eandersson | mriedem_afk, we do something similar already for upgrades | 01:00 |
eandersson | we move computes into a upgrade aggregate and once there are no more vms we autoamtically upgrade os, firmware etc | 01:00 |
eandersson | (we just don't enforce a timelimit yet at least :p) | 01:01 |
*** shaohe_feng has quit IRC | 01:01 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy https://review.openstack.org/571166 | 01:01 |
*** shaohe_feng has joined #openstack-nova | 01:02 | |
*** jichen has joined #openstack-nova | 01:06 | |
*** mdnadeem has joined #openstack-nova | 01:08 | |
*** shaohe_feng has quit IRC | 01:11 | |
*** shaohe_feng has joined #openstack-nova | 01:12 | |
*** yamamoto has joined #openstack-nova | 01:14 | |
*** sree has joined #openstack-nova | 01:16 | |
*** hongbin has joined #openstack-nova | 01:16 | |
*** liuzz has joined #openstack-nova | 01:17 | |
*** gyankum has joined #openstack-nova | 01:19 | |
*** harlowja has quit IRC | 01:19 | |
*** liuzz_ has quit IRC | 01:20 | |
*** sree has quit IRC | 01:21 | |
*** shaohe_feng has quit IRC | 01:22 | |
*** mriedem_afk has quit IRC | 01:23 | |
*** shaohe_feng has joined #openstack-nova | 01:23 | |
*** tiendc has joined #openstack-nova | 01:24 | |
*** xinliang has quit IRC | 01:28 | |
*** zhaochao has joined #openstack-nova | 01:30 | |
*** shaohe_feng has quit IRC | 01:32 | |
*** shaohe_feng has joined #openstack-nova | 01:33 | |
*** test_ has joined #openstack-nova | 01:33 | |
*** suresh12 has quit IRC | 01:35 | |
openstackgerrit | wanghongtao proposed openstack/nova master: Fix the metadata re to match the unicode https://review.openstack.org/536236 | 01:36 |
openstackgerrit | wanghongtao proposed openstack/nova master: Fix the metadata re to match the unicode https://review.openstack.org/571352 | 01:36 |
*** salv-orlando has joined #openstack-nova | 01:38 | |
*** suresh12 has joined #openstack-nova | 01:38 | |
*** lifeless_ has joined #openstack-nova | 01:40 | |
*** xinliang has joined #openstack-nova | 01:40 | |
*** lifeless has quit IRC | 01:40 | |
*** salv-orlando has quit IRC | 01:42 | |
*** shaohe_feng has quit IRC | 01:42 | |
*** shaohe_feng has joined #openstack-nova | 01:43 | |
*** suresh12 has quit IRC | 01:43 | |
openstackgerrit | tianhui proposed openstack/nova master: Fix some wrong urls in doc https://review.openstack.org/571147 | 01:44 |
*** namnh has joined #openstack-nova | 01:48 | |
*** shaohe_feng has quit IRC | 01:52 | |
*** sree has joined #openstack-nova | 01:53 | |
*** shaohe_feng has joined #openstack-nova | 01:53 | |
*** markvoelker has joined #openstack-nova | 01:54 | |
*** sree has quit IRC | 01:57 | |
*** thomasem has quit IRC | 01:59 | |
*** thomasem has joined #openstack-nova | 01:59 | |
*** gongysh has joined #openstack-nova | 02:00 | |
*** markvoelker has quit IRC | 02:02 | |
*** shaohe_feng has quit IRC | 02:03 | |
*** shaohe_feng has joined #openstack-nova | 02:03 | |
*** suresh12 has joined #openstack-nova | 02:05 | |
*** mvk has quit IRC | 02:06 | |
*** mdnadeem has quit IRC | 02:06 | |
*** Tom-Tom has joined #openstack-nova | 02:08 | |
*** markvoelker has joined #openstack-nova | 02:08 | |
*** suresh12 has quit IRC | 02:09 | |
*** liuyulong_ is now known as liuyulong | 02:11 | |
*** shaohe_feng has quit IRC | 02:13 | |
*** annp has joined #openstack-nova | 02:13 | |
*** fragatina has quit IRC | 02:14 | |
*** markvoelker has quit IRC | 02:14 | |
*** fragatina has joined #openstack-nova | 02:15 | |
*** markvoelker has joined #openstack-nova | 02:15 | |
*** shaohe_feng has joined #openstack-nova | 02:15 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: metadata: add vf_trusted field to device metadata https://review.openstack.org/566344 | 02:16 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | 02:16 |
*** fragatina has quit IRC | 02:18 | |
*** mvk has joined #openstack-nova | 02:19 | |
*** markvoelker has quit IRC | 02:19 | |
*** fragatina has joined #openstack-nova | 02:19 | |
*** fragatin_ has joined #openstack-nova | 02:23 | |
*** fragatina has quit IRC | 02:23 | |
*** shaohe_feng has quit IRC | 02:23 | |
*** shaohe_feng has joined #openstack-nova | 02:24 | |
*** fragatina has joined #openstack-nova | 02:25 | |
*** edmondsw has joined #openstack-nova | 02:26 | |
*** slaweq has joined #openstack-nova | 02:26 | |
*** fragatin_ has quit IRC | 02:27 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: metadata: add vf_trusted field to device metadata https://review.openstack.org/566344 | 02:28 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | 02:28 |
*** fragatina has quit IRC | 02:29 | |
*** slaweq has quit IRC | 02:31 | |
*** edmondsw has quit IRC | 02:31 | |
*** mdnadeem has joined #openstack-nova | 02:33 | |
*** shaohe_feng has quit IRC | 02:33 | |
*** shaohe_feng has joined #openstack-nova | 02:34 | |
*** artom has quit IRC | 02:34 | |
*** yamahata has quit IRC | 02:35 | |
*** Tom-Tom_ has joined #openstack-nova | 02:38 | |
*** Tom-Tom has quit IRC | 02:41 | |
*** fragatina has joined #openstack-nova | 02:43 | |
*** shaohe_feng has quit IRC | 02:44 | |
*** shaohe_feng has joined #openstack-nova | 02:45 | |
*** fragatina has quit IRC | 02:48 | |
*** xinliang has quit IRC | 02:53 | |
*** xinliang has joined #openstack-nova | 02:54 | |
*** shaohe_feng has quit IRC | 02:54 | |
*** shaohe_feng has joined #openstack-nova | 02:57 | |
*** vladikr has quit IRC | 03:00 | |
*** Tom-Tom has joined #openstack-nova | 03:01 | |
*** vladikr has joined #openstack-nova | 03:01 | |
*** Tom-Tom_ has quit IRC | 03:03 | |
*** shaohe_feng has quit IRC | 03:04 | |
*** trungnv_ has quit IRC | 03:05 | |
*** trungnv_ has joined #openstack-nova | 03:05 | |
*** shaohe_feng has joined #openstack-nova | 03:05 | |
*** slaweq has joined #openstack-nova | 03:11 | |
*** shaohe_feng has quit IRC | 03:14 | |
*** slaweq has quit IRC | 03:16 | |
*** shaohe_feng has joined #openstack-nova | 03:18 | |
*** shaohe_feng has quit IRC | 03:25 | |
*** annp has quit IRC | 03:25 | |
*** shaohe_feng has joined #openstack-nova | 03:26 | |
*** vladikr has quit IRC | 03:26 | |
*** vladikr has joined #openstack-nova | 03:30 | |
*** annp has joined #openstack-nova | 03:31 | |
*** shaohe_feng has quit IRC | 03:35 | |
*** rcernin has quit IRC | 03:35 | |
bhagyashris | efried; yes | 03:35 |
bhagyashris | efried: yes | 03:36 |
*** shaohe_feng has joined #openstack-nova | 03:36 | |
*** sree has joined #openstack-nova | 03:38 | |
*** sree has quit IRC | 03:39 | |
*** sree has joined #openstack-nova | 03:41 | |
*** shaohe_feng has quit IRC | 03:45 | |
*** shaohe_feng has joined #openstack-nova | 03:46 | |
*** bhujay has quit IRC | 03:47 | |
*** Tom-Tom has quit IRC | 03:48 | |
*** Tom-Tom has joined #openstack-nova | 03:49 | |
*** artom has joined #openstack-nova | 03:49 | |
*** hongbin has quit IRC | 03:52 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: metadata: add vf_trusted field to device metadata https://review.openstack.org/566344 | 03:53 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | 03:53 |
*** lpetrut has joined #openstack-nova | 03:53 | |
*** Tom-Tom has quit IRC | 03:54 | |
*** rcernin has joined #openstack-nova | 03:54 | |
*** shaohe_feng has quit IRC | 03:55 | |
*** shaohe_feng has joined #openstack-nova | 03:56 | |
*** mvk has quit IRC | 03:56 | |
*** artom has quit IRC | 03:57 | |
openstackgerrit | tianhui proposed openstack/nova master: Fix bug to api-ref https://review.openstack.org/571375 | 03:57 |
*** slaweq has joined #openstack-nova | 03:58 | |
*** zul has quit IRC | 03:58 | |
*** vladikr has quit IRC | 04:02 | |
*** slaweq has quit IRC | 04:03 | |
*** shaohe_feng has quit IRC | 04:06 | |
*** udesale has joined #openstack-nova | 04:06 | |
*** shaohe_feng has joined #openstack-nova | 04:07 | |
*** suresh12 has joined #openstack-nova | 04:08 | |
*** edmondsw has joined #openstack-nova | 04:14 | |
*** vladikr has joined #openstack-nova | 04:16 | |
*** mikal_ has joined #openstack-nova | 04:16 | |
*** shaohe_feng has quit IRC | 04:16 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: always create consumer records https://review.openstack.org/567678 | 04:16 |
openstackgerrit | Jay Pipes proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958 | 04:16 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405 | 04:16 |
openstackgerrit | Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | 04:16 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 04:16 |
*** edmondsw has quit IRC | 04:19 | |
*** mikal has quit IRC | 04:19 | |
*** shaohe_feng has joined #openstack-nova | 04:20 | |
*** Eran_Kuris has quit IRC | 04:20 | |
*** gyee has quit IRC | 04:21 | |
*** lpetrut has quit IRC | 04:22 | |
*** shaohe_feng has quit IRC | 04:26 | |
*** slaweq has joined #openstack-nova | 04:27 | |
*** rcernin has quit IRC | 04:28 | |
*** shaohe_feng has joined #openstack-nova | 04:28 | |
*** rcernin has joined #openstack-nova | 04:29 | |
*** fragatina has joined #openstack-nova | 04:31 | |
*** slaweq has quit IRC | 04:31 | |
*** fragatina has quit IRC | 04:33 | |
*** fragatina has joined #openstack-nova | 04:34 | |
openstackgerrit | Merged openstack/nova stable/pike: Fix shelving a paused instance https://review.openstack.org/569651 | 04:34 |
*** sridharg has joined #openstack-nova | 04:35 | |
*** shaohe_feng has quit IRC | 04:36 | |
*** shaohe_feng has joined #openstack-nova | 04:37 | |
*** links has joined #openstack-nova | 04:38 | |
*** lpetrut has joined #openstack-nova | 04:39 | |
*** lpetrut has quit IRC | 04:43 | |
*** vladikr has quit IRC | 04:46 | |
*** vladikr has joined #openstack-nova | 04:46 | |
*** shaohe_feng has quit IRC | 04:47 | |
*** shaohe_feng has joined #openstack-nova | 04:48 | |
*** gongysh has quit IRC | 04:50 | |
*** janki has joined #openstack-nova | 04:50 | |
*** shaohe_feng has quit IRC | 04:57 | |
*** slaweq has joined #openstack-nova | 04:58 | |
*** shaohe_feng has joined #openstack-nova | 04:59 | |
*** Dinesh_Bhor has quit IRC | 04:59 | |
*** slaweq has quit IRC | 05:02 | |
*** AlexeyAbashkin has joined #openstack-nova | 05:04 | |
*** shaohe_feng has quit IRC | 05:07 | |
*** shaohe_feng has joined #openstack-nova | 05:09 | |
openstackgerrit | Vishakha Agarwal proposed openstack/python-novaclient master: No requirement of –all-tenants while listing servers https://review.openstack.org/569090 | 05:13 |
*** suresh12 has quit IRC | 05:15 | |
*** shaohe_feng has quit IRC | 05:17 | |
*** shaohe_feng has joined #openstack-nova | 05:18 | |
*** AlexeyAbashkin has quit IRC | 05:23 | |
*** Eran_Kuris has joined #openstack-nova | 05:26 | |
*** fragatina has quit IRC | 05:26 | |
*** phuongnh has quit IRC | 05:26 | |
*** namnh has quit IRC | 05:26 | |
*** fragatina has joined #openstack-nova | 05:27 | |
*** phuongnh has joined #openstack-nova | 05:27 | |
*** namnh has joined #openstack-nova | 05:27 | |
*** ratailor has joined #openstack-nova | 05:27 | |
*** shaohe_feng has quit IRC | 05:28 | |
*** fragatina has quit IRC | 05:28 | |
*** fragatina has joined #openstack-nova | 05:28 | |
*** shaohe_feng has joined #openstack-nova | 05:28 | |
*** AlexeyAbashkin has joined #openstack-nova | 05:28 | |
*** slaweq has joined #openstack-nova | 05:30 | |
*** slaweq has quit IRC | 05:33 | |
*** slaweq has joined #openstack-nova | 05:34 | |
*** AlexeyAbashkin has quit IRC | 05:35 | |
*** shaohe_feng has quit IRC | 05:38 | |
*** shaohe_feng has joined #openstack-nova | 05:38 | |
*** slaweq has quit IRC | 05:39 | |
*** shaohe_feng has quit IRC | 05:48 | |
*** suresh12 has joined #openstack-nova | 05:49 | |
*** abhishekk has joined #openstack-nova | 05:50 | |
*** shaohe_feng has joined #openstack-nova | 05:50 | |
*** suresh12 has quit IRC | 05:53 | |
*** moshele has joined #openstack-nova | 05:54 | |
*** Tom-Tom has joined #openstack-nova | 05:57 | |
*** sar has quit IRC | 05:57 | |
*** jaosorior has quit IRC | 05:58 | |
*** shaohe_feng has quit IRC | 05:58 | |
*** vladikr has quit IRC | 06:00 | |
*** shaohe_feng has joined #openstack-nova | 06:00 | |
*** vladikr has joined #openstack-nova | 06:00 | |
*** edmondsw has joined #openstack-nova | 06:03 | |
*** hamdyk has joined #openstack-nova | 06:03 | |
*** markvoelker has joined #openstack-nova | 06:05 | |
*** vladikr has quit IRC | 06:06 | |
*** bhujay has joined #openstack-nova | 06:06 | |
*** vladikr has joined #openstack-nova | 06:07 | |
*** edmondsw has quit IRC | 06:07 | |
*** shaohe_feng has quit IRC | 06:09 | |
*** shaohe_feng has joined #openstack-nova | 06:10 | |
*** Dinesh_Bhor has joined #openstack-nova | 06:10 | |
*** Luzi has joined #openstack-nova | 06:11 | |
*** jaosorior has joined #openstack-nova | 06:14 | |
*** imacdonn has quit IRC | 06:16 | |
*** imacdonn has joined #openstack-nova | 06:16 | |
*** do3meli has joined #openstack-nova | 06:19 | |
*** shaohe_feng has quit IRC | 06:19 | |
*** sahid has joined #openstack-nova | 06:19 | |
*** vladikr has quit IRC | 06:20 | |
*** vladikr has joined #openstack-nova | 06:21 | |
*** shaohe_feng has joined #openstack-nova | 06:21 | |
*** tv2 is now known as Kvisle | 06:21 | |
*** liuzz has quit IRC | 06:24 | |
openstackgerrit | Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802 | 06:24 |
*** sar has joined #openstack-nova | 06:26 | |
*** lpetrut has joined #openstack-nova | 06:28 | |
*** shaohe_feng has quit IRC | 06:29 | |
*** shaohe_feng has joined #openstack-nova | 06:30 | |
*** vladikr has quit IRC | 06:30 | |
*** vladikr has joined #openstack-nova | 06:31 | |
*** shaohe_feng has quit IRC | 06:39 | |
*** slaweq has joined #openstack-nova | 06:42 | |
*** avolkov has joined #openstack-nova | 06:43 | |
*** pcaruana has joined #openstack-nova | 06:43 | |
*** shaohe_feng has joined #openstack-nova | 06:43 | |
*** liuyulong has quit IRC | 06:44 | |
*** liuyulong has joined #openstack-nova | 06:45 | |
*** slaweq has quit IRC | 06:46 | |
*** shaohe_feng has quit IRC | 06:50 | |
*** shaohe_feng has joined #openstack-nova | 06:50 | |
*** shaohe_feng has quit IRC | 07:00 | |
*** shaohe_feng has joined #openstack-nova | 07:01 | |
*** kholkina has joined #openstack-nova | 07:01 | |
*** rajinir has quit IRC | 07:02 | |
*** namnh has quit IRC | 07:04 | |
*** apple01 has joined #openstack-nova | 07:05 | |
*** tssurya has joined #openstack-nova | 07:07 | |
openstackgerrit | Jan Gutter proposed openstack/os-vif master: Add multiqueue field to InstanceInfo https://review.openstack.org/570959 | 07:08 |
*** damien_r has joined #openstack-nova | 07:10 | |
*** shaohe_feng has quit IRC | 07:10 | |
*** rcernin has quit IRC | 07:11 | |
*** shaohe_feng has joined #openstack-nova | 07:11 | |
*** vladikr has quit IRC | 07:12 | |
*** vladikr has joined #openstack-nova | 07:13 | |
*** apple01 has quit IRC | 07:17 | |
*** vladikr has quit IRC | 07:18 | |
*** vladikr has joined #openstack-nova | 07:18 | |
*** shaohe_feng has quit IRC | 07:20 | |
*** shaohe_feng has joined #openstack-nova | 07:21 | |
*** tesseract has joined #openstack-nova | 07:22 | |
*** apple01 has joined #openstack-nova | 07:26 | |
*** lpetrut has quit IRC | 07:30 | |
*** lpetrut has joined #openstack-nova | 07:30 | |
*** AlexeyAbashkin has joined #openstack-nova | 07:30 | |
*** shaohe_feng has quit IRC | 07:31 | |
*** shaohe_feng has joined #openstack-nova | 07:31 | |
*** apple01 has quit IRC | 07:38 | |
*** shaohe_feng has quit IRC | 07:41 | |
*** ccamacho has joined #openstack-nova | 07:44 | |
*** shaohe_feng has joined #openstack-nova | 07:44 | |
openstackgerrit | tianhui proposed openstack/nova master: Fix bug to api-ref https://review.openstack.org/571375 | 07:45 |
*** vladikr has quit IRC | 07:49 | |
*** jpena|off is now known as jpena | 07:49 | |
*** vladikr has joined #openstack-nova | 07:49 | |
*** edmondsw has joined #openstack-nova | 07:51 | |
*** shaohe_feng has quit IRC | 07:51 | |
*** shaohe_feng has joined #openstack-nova | 07:52 | |
*** amoralej|off is now known as amoralej | 07:55 | |
*** edmondsw has quit IRC | 07:56 | |
*** moshele has quit IRC | 08:00 | |
*** gongysh has joined #openstack-nova | 08:01 | |
*** efried has quit IRC | 08:01 | |
*** shaohe_feng has quit IRC | 08:01 | |
*** efried has joined #openstack-nova | 08:02 | |
*** shaohe_feng has joined #openstack-nova | 08:03 | |
*** alexchadin has joined #openstack-nova | 08:03 | |
*** andreas_s has joined #openstack-nova | 08:03 | |
*** hoonetorg has quit IRC | 08:11 | |
*** shaohe_feng has quit IRC | 08:12 | |
*** rcernin has joined #openstack-nova | 08:12 | |
*** rcernin has quit IRC | 08:12 | |
*** shaohe_feng has joined #openstack-nova | 08:13 | |
*** apple01 has joined #openstack-nova | 08:13 | |
*** ttsiouts has joined #openstack-nova | 08:14 | |
*** markvoelker has quit IRC | 08:15 | |
*** GonZo2000 has joined #openstack-nova | 08:17 | |
*** ttsiouts_ has joined #openstack-nova | 08:18 | |
*** ttsiouts has quit IRC | 08:18 | |
*** ratailor has quit IRC | 08:21 | |
*** shaohe_feng has quit IRC | 08:22 | |
*** shaohe_feng has joined #openstack-nova | 08:24 | |
*** hoonetorg has joined #openstack-nova | 08:24 | |
*** Luzi has quit IRC | 08:27 | |
*** Luzi has joined #openstack-nova | 08:28 | |
*** ratailor has joined #openstack-nova | 08:29 | |
*** bhujay has quit IRC | 08:30 | |
naichuans | bauzas, Hi, Sylvain. I have some questions about your email: http://lists.openstack.org/pipermail/openstack-dev/2018-May/130783.html. You said keep old and new rps, do you mean we will have an old compute node rp and a new compute node rp? or just a new compute node rp but keep old inventory in it? | 08:32 |
*** shaohe_feng has quit IRC | 08:32 | |
bauzas | naichuans: no, I was considering to keep the same behaviour for providing the VGPU RC to the compute inventory for the first type | 08:34 |
bauzas | naichuans: and just create a child RP with a new inventory for a second or more type | 08:34 |
*** mgoddard has joined #openstack-nova | 08:35 | |
*** shaohe_feng has joined #openstack-nova | 08:35 | |
*** GonZo2000 has quit IRC | 08:36 | |
*** wolverineav has quit IRC | 08:37 | |
*** wolverineav has joined #openstack-nova | 08:37 | |
*** wolverineav has quit IRC | 08:37 | |
naichuans | bauzas: got it. But when could we delete it? if we use n-rp as that, the compute node rp will always have the old inventory, it prevent us from use a true n-rp implement. | 08:37 |
*** wolverineav has joined #openstack-nova | 08:38 | |
openstackgerrit | jichenjc proposed openstack/nova master: not reraise DiskNotFound if instance is resized https://review.openstack.org/571410 | 08:38 |
bauzas | naichuans: by providing an online migration in Rocky | 08:38 |
bauzas | naichuans: I mean, upgrading shouldn't modify the existing | 08:38 |
openstackgerrit | jichenjc proposed openstack/nova master: not reraise DiskNotFound if instance is resized https://review.openstack.org/571410 | 08:39 |
bauzas | naichuans: but then, operators would need to use a CLI online migration for passing the VGPU inventory to a new child in between Rocky and Stein | 08:39 |
naichuans | bauzas: not sure about "passing the VGPU inventory to a new child", could you explain more? thanks. | 08:41 |
bauzas | naichuans: do you know about online migrations ? | 08:41 |
*** wolverineav has quit IRC | 08:42 | |
*** lifeless_ is now known as lifeless | 08:42 | |
naichuans | bauzas: no, it's something about upgrade? | 08:42 |
*** avolkov has quit IRC | 08:42 | |
*** shaohe_feng has quit IRC | 08:42 | |
bauzas | naichuans: yup | 08:43 |
*** shaohe_feng has joined #openstack-nova | 08:43 | |
bauzas | naichuans: DB online migrations help operators to modify the DB during a cycle | 08:43 |
bauzas | instead of needing to upgrade the DB offline, they just need to run the schema upgrades when upgrading offline, but then they can run the online migrations when they want | 08:44 |
bauzas | after running again the computes | 08:44 |
*** moshele has joined #openstack-nova | 08:45 | |
naichuans | bauzas: operator use this to modify the DB to remove old inv? | 08:46 |
naichuans | bauzas: if so, we don't need to return old inv to placement in driver. we can directly move to new n-rp implement, and let operator modify DB after upgrade? | 08:48 |
*** avolkov has joined #openstack-nova | 08:49 | |
*** derekh has joined #openstack-nova | 08:50 | |
bauzas | naichuans: if we do that, operators need to run that upgrade script *before* restarting the compute after upgrading, which is not something we want, right? | 08:51 |
*** shaohe_feng has quit IRC | 08:53 | |
*** ttsiouts_ has quit IRC | 08:53 | |
*** shaohe_feng has joined #openstack-nova | 08:55 | |
naichuans | bauzas: could you give a rough flow? modify DB -> upgrade -> retarting service? | 08:56 |
openstackgerrit | zhulingjie proposed openstack/nova master: Update overriden to overridden https://review.openstack.org/571418 | 08:58 |
*** Tahvok has quit IRC | 09:00 | |
naichuans | bauzas: If we modify the driver code to return vGPU inventories both for compute node rp and vGPU node rp, the old inventories would exist in placement db forever. I realy confused about that. Maybe I misunderstand your point,,, | 09:00 |
bauzas | naichuans: I was thinking of something like 'upgrade additive DB schema modifications' -> upgrade & restart n-cpu -> run the DB online migration tool | 09:01 |
bauzas | naichuans: lemme point you some code | 09:01 |
naichuans | bauzas: thanks | 09:01 |
*** wolverineav has joined #openstack-nova | 09:01 | |
bauzas | naichuans: first, docs https://docs.openstack.org/nova/latest/user/upgrade.html | 09:01 |
*** ttsiouts has joined #openstack-nova | 09:01 | |
bauzas | naichuans: I mean https://docs.openstack.org/nova/latest/user/upgrade.html#data-migrations | 09:02 |
*** shaohe_feng has quit IRC | 09:03 | |
bauzas | naichuans: and then code https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L375 | 09:03 |
*** markvoelker has joined #openstack-nova | 09:04 | |
naichuans | bauzas: will check it, thanks | 09:04 |
*** shaohe_feng has joined #openstack-nova | 09:06 | |
*** slaweq has joined #openstack-nova | 09:06 | |
*** alexchadin has quit IRC | 09:10 | |
*** hamdyk has quit IRC | 09:10 | |
*** hamdykhader has joined #openstack-nova | 09:10 | |
*** slaweq has quit IRC | 09:10 | |
*** alexchadin has joined #openstack-nova | 09:10 | |
*** zul_ has joined #openstack-nova | 09:11 | |
bauzas | naichuans: replied to the thread with a proposal, lemme give you the link | 09:11 |
*** zul_ is now known as zul | 09:11 | |
naichuans | bauzas: OK | 09:12 |
*** Tahvok has joined #openstack-nova | 09:12 | |
bauzas | naichuans: there it is http://lists.openstack.org/pipermail/openstack-dev/2018-May/130955.html | 09:12 |
bauzas | naichuans: does Xen support vGPU live migration ? | 09:13 |
naichuans | bauzas: got it~ | 09:13 |
*** shaohe_feng has quit IRC | 09:13 | |
*** shaohe_feng has joined #openstack-nova | 09:13 | |
bauzas | because QEMU folks say it's a very difficult thing to implement because there are a huge number of corner cases to consider | 09:13 |
kashyap | bauzas: Which part is this? | 09:22 |
*** sree has quit IRC | 09:22 | |
kashyap | Ah, Xen | 09:22 |
bauzas | kashyap: which part of what ? | 09:22 |
kashyap | Difficulty in implementing in QEMU | 09:22 |
bauzas | maybe the wording is inappropriate | 09:23 |
bauzas | it's not "difficult", it's just quite having a lot of cases to consider | 09:23 |
*** shaohe_feng has quit IRC | 09:23 | |
*** shaohe_feng has joined #openstack-nova | 09:24 | |
*** bhujay has joined #openstack-nova | 09:29 | |
naichuans | bauzas: sorry, just saw your message about VGPU live migration. XenServer do support live migration | 09:30 |
*** tidwellr has joined #openstack-nova | 09:30 | |
*** ratailor has quit IRC | 09:31 | |
*** bhujay has quit IRC | 09:31 | |
*** bhujay has joined #openstack-nova | 09:31 | |
*** udesale has quit IRC | 09:32 | |
*** Dinesh_Bhor has quit IRC | 09:32 | |
*** ratailor has joined #openstack-nova | 09:33 | |
*** shaohe_feng has quit IRC | 09:34 | |
*** jichen has quit IRC | 09:34 | |
*** shaohe_feng has joined #openstack-nova | 09:35 | |
*** tidwellr has quit IRC | 09:35 | |
*** Dinesh_Bhor has joined #openstack-nova | 09:36 | |
*** Dinesh_Bhor has quit IRC | 09:36 | |
*** apple01 has quit IRC | 09:37 | |
*** apple01 has joined #openstack-nova | 09:37 | |
*** udesale has joined #openstack-nova | 09:38 | |
naichuans | bauzas: just finish reading your conversation history about n-rp in vGPU. You are using DB online migration to avoid compute service restart, right? | 09:44 |
*** shaohe_feng has quit IRC | 09:44 | |
*** shaohe_feng has joined #openstack-nova | 09:45 | |
*** alexchadin has quit IRC | 09:45 | |
*** ratailor has quit IRC | 09:46 | |
*** alexchadin has joined #openstack-nova | 09:48 | |
*** ratailor has joined #openstack-nova | 09:49 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: handle DiskNotFound during update_available_resource https://review.openstack.org/571424 | 09:49 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: Skip fetching the virtual size of block devices https://review.openstack.org/571425 | 09:49 |
*** apple01 has quit IRC | 09:50 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: libvirt: handle DiskNotFound during update_available_resource https://review.openstack.org/571426 | 09:50 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: libvirt: Skip fetching the virtual size of block devices https://review.openstack.org/571427 | 09:50 |
*** ratailor has quit IRC | 09:51 | |
*** ratailor has joined #openstack-nova | 09:52 | |
*** ttx has quit IRC | 09:52 | |
*** shaohe_feng has quit IRC | 09:54 | |
*** shaohe_feng has joined #openstack-nova | 09:55 | |
*** lifeless has quit IRC | 09:55 | |
*** lifeless has joined #openstack-nova | 09:56 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject interface attach with QoS aware port https://review.openstack.org/570078 | 09:56 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject networks with QoS policy https://review.openstack.org/570079 | 09:56 |
*** apple01 has joined #openstack-nova | 09:57 | |
openstackgerrit | wanghongtao proposed openstack/nova master: Fix the metadata re to match the unicode https://review.openstack.org/536236 | 09:58 |
*** avolkov has quit IRC | 10:03 | |
*** shaohe_feng has quit IRC | 10:04 | |
openstackgerrit | Merged openstack/nova master: pci: don't consider case when match tags specs https://review.openstack.org/565808 | 10:07 |
*** shaohe_feng has joined #openstack-nova | 10:07 | |
*** cz2 has joined #openstack-nova | 10:07 | |
*** lifeless has quit IRC | 10:08 | |
*** lifeless_ has joined #openstack-nova | 10:08 | |
*** slaweq has joined #openstack-nova | 10:11 | |
*** ttx has joined #openstack-nova | 10:11 | |
*** artom has joined #openstack-nova | 10:12 | |
*** apple01 has quit IRC | 10:13 | |
*** ttx has quit IRC | 10:14 | |
*** ttx has joined #openstack-nova | 10:14 | |
*** apple01 has joined #openstack-nova | 10:15 | |
*** shaohe_feng has quit IRC | 10:15 | |
*** slaweq has quit IRC | 10:15 | |
*** apple01 has quit IRC | 10:16 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: libvirt: handle DiskNotFound during update_available_resource https://review.openstack.org/571432 | 10:16 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: libvirt: Skip fetching the virtual size of block devices https://review.openstack.org/571433 | 10:16 |
*** shaohe_feng has joined #openstack-nova | 10:16 | |
*** stakeda has quit IRC | 10:19 | |
sahid | artom: thanks for what you did on the vf trusted serie :) | 10:20 |
artom | sahid, oh, hah, I'd just apologized on the downstream channel | 10:21 |
sahid | artom: yes but, please don't, i have appreciated thanks again | 10:21 |
artom | sahid, cool, you're welcome :) | 10:22 |
artom | We *really* need to unfsck the metadata API | 10:22 |
artom | The fact that we're not using templates, and adding/removing stuff based on ifs... | 10:23 |
artom | *Versioned* templates, I should say | 10:23 |
sahid | artom: i see, yes that make sense to have such mech | 10:23 |
artom | sahid, well the "real" API uses those, I believe | 10:24 |
*** shaohe_feng has quit IRC | 10:25 | |
*** ttsiouts has quit IRC | 10:26 | |
*** shaohe_feng has joined #openstack-nova | 10:26 | |
sahid | artom: i did not get time to look at your serie about the live migration, i saw that it's still in WIP - my understanding is i could wwait a bit more before to start reviewing it | 10:27 |
sahid | or do you think we should start to make some reviews? | 10:27 |
artom | sahid, yeah, no worries, there's not much there yet | 10:27 |
artom | Well | 10:27 |
artom | https://review.openstack.org/#/c/566398/7 and https://review.openstack.org/#/c/566723/8 and ready | 10:28 |
artom | The last one is marked WIP because the one below it is | 10:28 |
artom | It's like a WIP sandwich | 10:28 |
*** mdnadeem has quit IRC | 10:28 | |
*** janki has quit IRC | 10:29 | |
*** tetsuro has quit IRC | 10:29 | |
*** tetsuro has joined #openstack-nova | 10:31 | |
*** nicolasbock has joined #openstack-nova | 10:33 | |
*** armaan has joined #openstack-nova | 10:33 | |
*** shaohe_feng has quit IRC | 10:35 | |
*** gcb has quit IRC | 10:36 | |
*** shaohe_feng has joined #openstack-nova | 10:39 | |
*** shaohe_feng has quit IRC | 10:45 | |
*** shaohe_feng has joined #openstack-nova | 10:46 | |
*** tbachman has quit IRC | 10:48 | |
*** abalutoiu_ has quit IRC | 10:49 | |
*** gongysh has quit IRC | 10:50 | |
*** shaohe_feng has quit IRC | 10:56 | |
*** wolverineav has quit IRC | 10:56 | |
*** shaohe_feng has joined #openstack-nova | 10:56 | |
*** apple01 has joined #openstack-nova | 10:57 | |
*** Tom-Tom has quit IRC | 10:57 | |
*** Tom-Tom has joined #openstack-nova | 10:57 | |
*** annp has quit IRC | 10:57 | |
*** tetsuro has quit IRC | 10:58 | |
*** jpena is now known as jpena|lunch | 11:00 | |
*** ratailor_ has joined #openstack-nova | 11:00 | |
*** gyankum has quit IRC | 11:00 | |
*** Tom-Tom has quit IRC | 11:02 | |
*** ratailor has quit IRC | 11:03 | |
*** markvoelker_ has joined #openstack-nova | 11:04 | |
*** shaohe_feng has quit IRC | 11:06 | |
*** markvoelker has quit IRC | 11:06 | |
*** jogo has quit IRC | 11:06 | |
*** shaohe_feng has joined #openstack-nova | 11:07 | |
*** tesseract has quit IRC | 11:07 | |
*** ttx has quit IRC | 11:07 | |
*** tesseract has joined #openstack-nova | 11:08 | |
*** ttx has joined #openstack-nova | 11:08 | |
*** vladikr has quit IRC | 11:11 | |
*** vladikr has joined #openstack-nova | 11:11 | |
*** ttsiouts has joined #openstack-nova | 11:12 | |
*** janki has joined #openstack-nova | 11:12 | |
*** markvoelker has joined #openstack-nova | 11:14 | |
*** markvoelker_ has quit IRC | 11:15 | |
*** shaohe_feng has quit IRC | 11:16 | |
*** abhishekk has quit IRC | 11:17 | |
*** shaohe_feng has joined #openstack-nova | 11:20 | |
*** takashin_ has joined #openstack-nova | 11:25 | |
*** jogo has joined #openstack-nova | 11:25 | |
*** shaohe_feng has quit IRC | 11:26 | |
*** takashin has quit IRC | 11:27 | |
*** edmondsw has joined #openstack-nova | 11:27 | |
openstackgerrit | Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | 11:27 |
*** shaohe_feng has joined #openstack-nova | 11:28 | |
*** vladikr has quit IRC | 11:28 | |
*** Tom-Tom has joined #openstack-nova | 11:28 | |
*** vladikr has joined #openstack-nova | 11:28 | |
*** bhujay has quit IRC | 11:29 | |
*** bhujay has joined #openstack-nova | 11:30 | |
*** bhujay has quit IRC | 11:31 | |
*** edmondsw has quit IRC | 11:31 | |
*** ttsiouts has quit IRC | 11:31 | |
*** bhujay has joined #openstack-nova | 11:32 | |
*** Tom-Tom has quit IRC | 11:32 | |
*** phuongnh has quit IRC | 11:32 | |
*** udesale has quit IRC | 11:33 | |
*** chyka has joined #openstack-nova | 11:33 | |
*** pchavva has joined #openstack-nova | 11:33 | |
*** ttx has quit IRC | 11:34 | |
*** vladikr has quit IRC | 11:34 | |
*** vladikr has joined #openstack-nova | 11:35 | |
*** ttx has joined #openstack-nova | 11:35 | |
*** dave-mccowan has joined #openstack-nova | 11:36 | |
*** sambetts|afk is now known as sambetts | 11:36 | |
*** shaohe_feng has quit IRC | 11:37 | |
*** udesale has joined #openstack-nova | 11:37 | |
*** chyka has quit IRC | 11:38 | |
*** shaohe_feng has joined #openstack-nova | 11:40 | |
*** dave-mcc_ has joined #openstack-nova | 11:40 | |
*** yamamoto has quit IRC | 11:41 | |
*** vladikr has quit IRC | 11:42 | |
*** vladikr has joined #openstack-nova | 11:43 | |
*** dave-mccowan has quit IRC | 11:43 | |
*** bhujay has quit IRC | 11:46 | |
*** udesale has quit IRC | 11:47 | |
*** ttx has quit IRC | 11:47 | |
*** ttx has joined #openstack-nova | 11:47 | |
*** shaohe_feng has quit IRC | 11:47 | |
*** edmondsw has joined #openstack-nova | 11:47 | |
*** shaohe_feng has joined #openstack-nova | 11:48 | |
*** bhujay has joined #openstack-nova | 11:48 | |
*** bhujay has quit IRC | 11:48 | |
*** armaan has quit IRC | 11:50 | |
*** armaan has joined #openstack-nova | 11:50 | |
*** udesale has joined #openstack-nova | 11:51 | |
openstackgerrit | Hironori Shiina proposed openstack/nova master: WIP: virt/ironic: Implement rescue and unrescue https://review.openstack.org/571450 | 11:51 |
*** ttx has quit IRC | 11:51 | |
*** ttx has joined #openstack-nova | 11:51 | |
*** pchavva has quit IRC | 11:52 | |
*** pchavva has joined #openstack-nova | 11:53 | |
*** shaohe_feng has quit IRC | 11:57 | |
openstackgerrit | Jan Gutter proposed openstack/os-vif master: Add virtio_multiqueue field to InstanceInfo https://review.openstack.org/570959 | 11:57 |
*** pchavva has quit IRC | 11:58 | |
*** shaohe_feng has joined #openstack-nova | 11:58 | |
*** udesale has quit IRC | 12:03 | |
*** udesale has joined #openstack-nova | 12:03 | |
*** tbachman has joined #openstack-nova | 12:04 | |
*** hamdykhader has quit IRC | 12:04 | |
*** udesale_ has joined #openstack-nova | 12:07 | |
*** belmoreira has joined #openstack-nova | 12:07 | |
*** wolverineav has joined #openstack-nova | 12:07 | |
*** shaohe_feng has quit IRC | 12:07 | |
*** tbachman_ has joined #openstack-nova | 12:09 | |
*** tbachman has quit IRC | 12:09 | |
*** tbachman_ is now known as tbachman | 12:09 | |
*** udesale has quit IRC | 12:09 | |
*** shaohe_feng has joined #openstack-nova | 12:11 | |
*** armaan has quit IRC | 12:13 | |
*** armaan has joined #openstack-nova | 12:14 | |
*** brad[] has quit IRC | 12:16 | |
*** takashin_ has left #openstack-nova | 12:16 | |
*** takashin has joined #openstack-nova | 12:17 | |
*** shaohe_feng has quit IRC | 12:18 | |
*** shaohe_feng has joined #openstack-nova | 12:18 | |
*** abhishekk has joined #openstack-nova | 12:20 | |
*** jpena|lunch is now known as jpena | 12:25 | |
*** eharney has joined #openstack-nova | 12:25 | |
*** shaohe_feng has quit IRC | 12:28 | |
*** shaohe_feng has joined #openstack-nova | 12:29 | |
*** udesale_ has quit IRC | 12:29 | |
*** udesale_ has joined #openstack-nova | 12:30 | |
*** vladikr has quit IRC | 12:30 | |
*** vladikr has joined #openstack-nova | 12:31 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: WIP abort live migration in queue. https://review.openstack.org/568542 | 12:34 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: WIP abort live migration in queue. https://review.openstack.org/568542 | 12:36 |
openstackgerrit | Jan Gutter proposed openstack/nova master: Convert vrouter legacy plugging to os-vif https://review.openstack.org/571325 | 12:37 |
openstackgerrit | Jan Gutter proposed openstack/nova master: Use vif.vif_name in _set_config_VIFGeneric https://review.openstack.org/571461 | 12:37 |
openstackgerrit | Jan Gutter proposed openstack/nova master: Pass virtio multiqueue info to os-vif plugins https://review.openstack.org/571462 | 12:37 |
*** shaohe_feng has quit IRC | 12:38 | |
*** sahid has quit IRC | 12:39 | |
*** READ10 has joined #openstack-nova | 12:40 | |
*** shaohe_feng has joined #openstack-nova | 12:40 | |
*** yamamoto has joined #openstack-nova | 12:41 | |
*** READ10 has quit IRC | 12:42 | |
*** udesale has joined #openstack-nova | 12:45 | |
*** udesale_ has quit IRC | 12:45 | |
*** yamamoto has quit IRC | 12:46 | |
*** shaohe_feng has quit IRC | 12:48 | |
*** shaohe_feng has joined #openstack-nova | 12:49 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: WIP: Adapt _validate_instance_group_policy to new policy model https://review.openstack.org/571465 | 12:51 |
*** slaweq has joined #openstack-nova | 12:54 | |
*** yamamoto has joined #openstack-nova | 12:56 | |
*** slaweq has quit IRC | 12:58 | |
*** shaohe_feng has quit IRC | 12:59 | |
*** jistr is now known as jistr|mtg | 12:59 | |
*** shaohe_feng has joined #openstack-nova | 13:00 | |
*** mgoddard has quit IRC | 13:01 | |
*** tiendc has quit IRC | 13:03 | |
*** hshiina is now known as hshiina|afk | 13:05 | |
*** mriedem has joined #openstack-nova | 13:07 | |
*** shaohe_feng has quit IRC | 13:09 | |
*** READ10 has joined #openstack-nova | 13:09 | |
*** slaweq has joined #openstack-nova | 13:10 | |
*** shaohe_feng has joined #openstack-nova | 13:11 | |
*** armaan has quit IRC | 13:11 | |
*** armaan has joined #openstack-nova | 13:12 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Delete orphaned RS and IM records of archived instances https://review.openstack.org/560042 | 13:13 |
openstackgerrit | Chen proposed openstack/nova master: WIP Closes-Bug: #1449221 https://review.openstack.org/571472 | 13:14 |
openstack | bug 1449221 in OpenStack Compute (nova) "Nova doesn't allow cleanup of volumes stuck in 'attaching' or 'detaching' status" [High,Confirmed] https://launchpad.net/bugs/1449221 | 13:14 |
*** slaweq has quit IRC | 13:15 | |
*** hughsaunders has joined #openstack-nova | 13:16 | |
*** armaan has quit IRC | 13:17 | |
*** shaohe_feng has quit IRC | 13:19 | |
*** apple01 has quit IRC | 13:19 | |
*** shaohe_feng has joined #openstack-nova | 13:20 | |
*** sahid has joined #openstack-nova | 13:22 | |
*** armaan has joined #openstack-nova | 13:23 | |
*** ratailor_ has quit IRC | 13:24 | |
*** apple01 has joined #openstack-nova | 13:28 | |
*** links has quit IRC | 13:29 | |
*** shaohe_feng has quit IRC | 13:29 | |
*** shaohe_feng has joined #openstack-nova | 13:30 | |
*** burt has joined #openstack-nova | 13:32 | |
*** udesale_ has joined #openstack-nova | 13:34 | |
*** tetsuro has joined #openstack-nova | 13:34 | |
*** markvoelker_ has joined #openstack-nova | 13:35 | |
*** sar has quit IRC | 13:36 | |
*** markvoelker has quit IRC | 13:36 | |
*** udesale has quit IRC | 13:37 | |
*** markvoelker has joined #openstack-nova | 13:38 | |
*** armaan has quit IRC | 13:38 | |
*** shaohe_feng has quit IRC | 13:40 | |
*** markvoelker_ has quit IRC | 13:40 | |
*** shaohe_feng has joined #openstack-nova | 13:41 | |
*** gyankum has joined #openstack-nova | 13:42 | |
*** mvenesio has joined #openstack-nova | 13:43 | |
*** gbarros has quit IRC | 13:44 | |
*** belmorei_ has joined #openstack-nova | 13:49 | |
*** belmoreira has quit IRC | 13:50 | |
*** shaohe_feng has quit IRC | 13:50 | |
melwitt | nova meeting in 10 minutes | 13:50 |
*** esberglu has joined #openstack-nova | 13:50 | |
*** shaohe_feng has joined #openstack-nova | 13:50 | |
*** throwsb1 has joined #openstack-nova | 13:50 | |
*** ttsiouts has joined #openstack-nova | 13:51 | |
*** markvoelker has quit IRC | 13:52 | |
mriedem | dansmith: can you get this queens backport? https://review.openstack.org/#/c/571425/ | 13:53 |
*** slaweq has joined #openstack-nova | 13:53 | |
mriedem | lyarwood: looks like you have something to fix in this one https://review.openstack.org/#/c/571433/ | 13:54 |
*** hongbin has joined #openstack-nova | 13:56 | |
dansmith | hmm, not sure why that depends on the update resource patch | 13:57 |
*** slaweq has quit IRC | 13:57 | |
lyarwood | mriedem: yeah, couldn't run tox locally for Ocata for some reason, I'll get to it later today | 13:57 |
*** tssurya_mob has joined #openstack-nova | 13:58 | |
lyarwood | dansmith: it just keeps the backport clean, I was already looking to backport the resource patch anyway so... | 13:58 |
*** awaugama has joined #openstack-nova | 13:58 | |
dansmith | hmm, okay | 13:59 |
*** shaohe_feng has quit IRC | 14:00 | |
openstackgerrit | sahid proposed openstack/os-vif master: Add virtio_multiqueue field to InstanceInfo https://review.openstack.org/570959 | 14:00 |
*** shaohe_feng has joined #openstack-nova | 14:01 | |
*** brad[] has joined #openstack-nova | 14:01 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: always create consumer records https://review.openstack.org/567678 | 14:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958 | 14:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405 | 14:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | 14:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 14:04 |
*** mlavalle has joined #openstack-nova | 14:06 | |
*** ttsiouts has quit IRC | 14:07 | |
*** andreas_s has quit IRC | 14:08 | |
jaypipes | btw, dansmith, fixed the silly pep8 error in ^ | 14:09 |
dansmith | ack | 14:09 |
jaypipes | dansmith: let me know if you'd like further tests added for ensure_consumer() | 14:09 |
dansmith | you know I will | 14:09 |
jaypipes | :) | 14:09 |
mriedem | i had some random thoughts about a nova-status upgrade check for the missing consumer stuff, | 14:10 |
mriedem | but haven't thought about it much | 14:10 |
mriedem | basically query the api db for allocations w/o a consumer | 14:10 |
mriedem | but not sure what to do if we find them | 14:10 |
jaypipes | mriedem: the bottom patch on that series adds an online db migration for "incomplete" consumers. | 14:10 |
dansmith | you want nova-status to connect directly to the placement database? | 14:10 |
*** shaohe_feng has quit IRC | 14:10 | |
mriedem | nova-status already connects directly to the nova_api db, which is the placement db today | 14:11 |
mriedem | jaypipes: yeah i forgot about that | 14:11 |
dansmith | right, but it muddies the water for a split | 14:11 |
mriedem | yeah if we have an online data migration i'm not sure we need an upgrade status check thing | 14:12 |
*** moshele has quit IRC | 14:12 | |
*** ttsiouts has joined #openstack-nova | 14:13 | |
*** mvenesio has quit IRC | 14:13 | |
mriedem | i think i was thinking, | 14:13 |
mriedem | somehow healing the allocations / creating missing consumers based on the actual instance for the allocation, rather than the config options | 14:13 |
mriedem | which is what jay's online data migration is using it looks like | 14:13 |
*** shaohe_feng has joined #openstack-nova | 14:14 | |
dansmith | hmm | 14:14 |
dansmith | I mean, I guess we can do that now, but we won't be able to do things like that past the split, so I would think we should stick to being cleanly separated even before | 14:15 |
jaypipes | mriedem: your heal_allocations() work would already do that (once the always create consumer patch is merged, of course) | 14:15 |
*** gbarros has joined #openstack-nova | 14:15 | |
mriedem | jaypipes: the heal_allocations CLI won't do anything if the instance already has allocations | 14:16 |
mriedem | but it doesn't check if the project/user is missing | 14:16 |
jaypipes | mriedem: ah, yes, true nuf | 14:16 |
mriedem | https://review.openstack.org/#/c/565886/9/nova/cmd/manage.py@1794 | 14:16 |
*** r-daneel has joined #openstack-nova | 14:19 | |
*** shaohe_feng has quit IRC | 14:21 | |
sahid | jangutter: i have question for you in the osvif patch | 14:21 |
jangutter | shoot! | 14:21 |
mriedem | jaypipes: having said that, we could make heal_allocations look to see if the existing allocations are missing consumer/project and re-post the allocations w/ the consumer/project if so | 14:21 |
*** shaohe_feng has joined #openstack-nova | 14:21 | |
sahid | jangutter: is that could help whether we have that new virtio_multiqueu attribute an integer, so we can pass the exact number of queue enabled? | 14:22 |
*** tidwellr has joined #openstack-nova | 14:22 | |
*** tidwellr has quit IRC | 14:23 | |
jangutter | sahid: I dunno, this is weird territory for me... literally the only code I could find used iproute2 to enable "multiqueue" and that's it. | 14:23 |
*** tidwellr has joined #openstack-nova | 14:23 | |
jaypipes | mriedem: IIRC, that was my original suggestion that I could do that in the online DB migration | 14:23 |
jaypipes | mriedem: but it was decided that the single CONF option was a simpler solution for the majority case. | 14:24 |
*** tetsuro has quit IRC | 14:24 | |
sahid | jangutter: it's not something that will be used by vrouter so? | 14:24 |
jaypipes | mriedem: part of the issue with that is that I would have to import nova objects and DB into the placement object files in order to grab instance information. | 14:24 |
jangutter | sahid: It seems to be a bit of an aberration, almost all of the other VIF types can learn the multiqueue configs from the hypervisor, just TAP seem to need something on the plug side. | 14:24 |
dansmith | jaypipes: which we can't do post-split, so we shouldn't do now, if we're serious about maintaining that separation | 14:25 |
jangutter | sahid: yeah I think vrouter's the only remaining bit that requires that... | 14:25 |
jangutter | sahid: I think things like vhostuser negotiates it over-the-wire or something. I haven't looked closely. | 14:26 |
jaypipes | dansmith: right, that was the concern. | 14:26 |
*** Phuongnh has joined #openstack-nova | 14:27 | |
mriedem | jaypipes: sure i get that, | 14:28 |
mriedem | the heal_allocations CLI is all REST API stuff wrt placement | 14:28 |
mriedem | so there is no mixed db naughtiness | 14:28 |
*** hughsaunders has quit IRC | 14:28 | |
mriedem | if we ever get to counting quotas using placement, i think we'll have to fix those consumer records to use the same project/user as the instance, one way or another | 14:29 |
mriedem | because counting quotas against a "real" project/user won't include any usage marked against the config options | 14:30 |
mriedem | heal_allocations could be told to also compare the allocation project/user against the instance project/user and if not the same, update the allocation | 14:30 |
dansmith | yes, definitely, but I think jaypipes' plan was to make sure that happened first, and I thought heal_allocations was the way | 14:30 |
jaypipes | mriedem: agreed. | 14:30 |
jaypipes | dansmith: yes, it was. | 14:30 |
mriedem | right so i think we're on the same page: fill the hole we have now with the config options, then rely on heal_allocations to properly update the allocations per instance later | 14:31 |
*** shaohe_feng has quit IRC | 14:31 | |
dansmith | yup | 14:31 |
mriedem | ok | 14:31 |
mriedem | whew | 14:31 |
jaypipes | perfecto | 14:31 |
mriedem | we should write this down... | 14:31 |
*** shaohe_feng has joined #openstack-nova | 14:32 | |
mriedem | i might amend something into https://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/add-consumer-generation.html#upgrade-impact later just as a reminder | 14:33 |
melwitt | ++ reminders are good | 14:33 |
sahid | jangutter: ok so i just checked on code, yes it seems that vrouter is the only which needs to created tap device with multiqueue bit so far | 14:33 |
sahid | so let keep that simple | 14:34 |
melwitt | takashin: thanks for fixing the meeting times on the agenda -- I didn't realize I had incorrectly swapped them after summit week | 14:34 |
jangutter | sahid: yeah, it can spiral out of control quickly, txqueues, rxqueues.... | 14:34 |
sahid | jangutter: yes right... | 14:36 |
sahid | but i think we do not have any other choice, "normally" it's libvirt which creates the tap and configures the queues | 14:37 |
sahid | but vrouter is using ethernet interface | 14:38 |
*** alexchadin has quit IRC | 14:39 | |
*** rajinir has joined #openstack-nova | 14:40 | |
*** shaohe_feng has quit IRC | 14:41 | |
*** gongysh has joined #openstack-nova | 14:41 | |
takashin | melwitt: I'm glad to help. | 14:42 |
*** gongysh has quit IRC | 14:42 | |
*** ttsiouts has quit IRC | 14:42 | |
dansmith | mriedem: so I need to raise upper constraints for oslo.messaging prior to my patch to use the new version yeah? that's not automatic I assume... | 14:44 |
*** shaohe_feng has joined #openstack-nova | 14:44 | |
*** gcb has joined #openstack-nova | 14:44 | |
dansmith | actually maybe it gets auto-proposed/ | 14:45 |
dansmith | I see a bunch of those in requirements right now | 14:45 |
*** armaan has joined #openstack-nova | 14:46 | |
mriedem | it's automatic | 14:48 |
mriedem | you'll have to bump lower-constraints in your nova change | 14:48 |
dansmith | um | 14:49 |
*** kholkina has quit IRC | 14:49 | |
mriedem | upper-constraints in the requirements repo gets updated automatically via a bot | 14:49 |
dansmith | okay I guess I'm not sure why we have lower-constraints in our tree | 14:49 |
mriedem | the in-tree lower-constraints file (and requirements.txt i guess) in nova will need to be updated | 14:50 |
*** suresh12 has joined #openstack-nova | 14:50 | |
mriedem | it's in all trees, to run the lower-constraints job | 14:50 |
mriedem | since all other jobs run against upper-constraints | 14:50 |
mriedem | and to decouple g-r from a minimum baseline across all projects | 14:50 |
dansmith | I guess because we how have our own set of lowers or whatev3er | 14:50 |
dansmith | okay | 14:50 |
mriedem | i guess, i'm not really following all of that work, but it's what i know | 14:50 |
mriedem | yup | 14:50 |
*** shaohe_feng has quit IRC | 14:51 | |
*** shaohe_feng has joined #openstack-nova | 14:53 | |
*** suresh12 has quit IRC | 14:54 | |
*** tssurya_mob has quit IRC | 14:54 | |
*** mriedem has quit IRC | 14:55 | |
jangutter | sahid: yeah, vrouter is using the 'this taints libvirt' ethernet interface. Maybe with Tungsten Fabric there's a chance to add that to libvirt, but who knows. | 14:55 |
*** suresh12 has joined #openstack-nova | 14:55 | |
belmorei_ | mriedem: do you think that I can already use: https://review.openstack.org/#/c/565886 | 14:55 |
*** zhaochao has quit IRC | 14:56 | |
*** tbachman has quit IRC | 14:56 | |
*** Eran_Kuris has quit IRC | 14:57 | |
*** belmorei_ has quit IRC | 14:57 | |
*** jistr|mtg is now known as jistr | 14:57 | |
*** belmoreira has joined #openstack-nova | 14:58 | |
*** Luzi has quit IRC | 15:00 | |
*** abhishekk has quit IRC | 15:00 | |
*** tbachman has joined #openstack-nova | 15:01 | |
*** mriedem has joined #openstack-nova | 15:01 | |
*** shaohe_feng has quit IRC | 15:02 | |
*** shaohe_feng has joined #openstack-nova | 15:02 | |
*** suresh12 has quit IRC | 15:03 | |
belmoreira | mriedem do you think that I can already use: https://review.openstack.org/#/c/565886 | 15:04 |
*** Cardoe has quit IRC | 15:04 | |
*** Cardoe has joined #openstack-nova | 15:04 | |
jaypipes | sahid: "The whole NUMA thing is basically a cartography for the scheduler to help it making good decisions." <-- you're referring to the kernel process scheduler, right? not the nova-scheduler,... | 15:06 |
mriedem | belmoreira: what are you trying to solve with it? | 15:07 |
mriedem | missing allocations? | 15:07 |
sahid | jaypipes: yes yes :) | 15:07 |
jaypipes | sahid: ok, just making sure :) | 15:07 |
belmoreira | mriedem we had an issue with ironic that triggered the recreation of all compute_nodes; now the new compute_nodes can't be registered in placement because the previous resource_provider/allocations | 15:07 |
mriedem | so can't create the new resource provider because another with the same name/uuid already exists right? | 15:08 |
jaypipes | sahid, stephenfin: you'll note I tried as much as possible to remove any and all mention of NUMA topology (host or guest/virtual) in the CPU resources spec. | 15:08 |
*** mgoddard has joined #openstack-nova | 15:08 | |
mriedem | and can't delete the existing one because the existing one has allocations already? | 15:08 |
belmoreira | mriedem: yes | 15:08 |
belmoreira | the cleanest way to recover would be to delete everything related to ironic resources providers and recreate again | 15:08 |
jaypipes | sahid, stephenfin: at least in so much as the NUMA topology stuff wasn't really germane to the "request different providers supply some X amount of dedicated or shared CPU resources for the guest" | 15:09 |
belmoreira | unfortunately I have ~2000 nodes in this state | 15:09 |
*** takashin has left #openstack-nova | 15:09 | |
mriedem | belmoreira: yeah i think what i'd probably do is pick one resource provider / allocation combo since they should be 1:1 with ironic, | 15:09 |
mriedem | clean it up using the osc-placement CLI | 15:09 |
mriedem | and then run heal_allocations but probably want to put a cap on the --max-count | 15:09 |
stephenfin | jaypipes: ack. In a meeting/working on NUMA-aware vswitches, but I'll take a look before EOD | 15:09 |
mriedem | belmoreira: this is why i had a note in the code about allowing passing in a single instance id to heal | 15:10 |
mriedem | that would be ideal in this case as a test | 15:10 |
jaypipes | stephenfin: no worries, you already reviewed again (and I will update to your comments) | 15:10 |
belmoreira | mriedem I will first test all on copies of the DBs | 15:10 |
mriedem | belmoreira: ok yeah definitely doing it in a test env would be good for both of us :) | 15:11 |
*** shaohe_feng has quit IRC | 15:12 | |
belmoreira | other use-case for this code is updating ironic allocations that were created before a resource_class was defined | 15:12 |
mriedem | so the existing allocations for the ironic instance are against vcpu/memory_mb/disk_gb and you want to change then to be against custom_baremetal_belmiro, yes? | 15:13 |
mriedem | *them | 15:13 |
belmoreira | mriedem: but the recreation of the compute_node in nova is something that needs to be reviewed; has been creating few issues. Don't know yet why this happened... | 15:14 |
mriedem | that's probably where the --force option mentioned in the comments would come in, since currently the command will skip any instances that already have allocations, not attempt to overwrite them | 15:14 |
*** shaohe_feng has joined #openstack-nova | 15:14 | |
*** ccamacho has quit IRC | 15:15 | |
belmoreira | mriedem:yes, that's way I was pointing this as another use-case | 15:15 |
mriedem | belmoreira: so i thought that sounded very much like https://review.openstack.org/#/c/508555/ | 15:15 |
*** ccamacho has joined #openstack-nova | 15:15 | |
*** Swami_ has joined #openstack-nova | 15:15 | |
belmoreira | mriedem: this was not rebalance... we had an operator working in ironic/nova-compute and know is very hard to track why this happened | 15:16 |
*** awaugama has quit IRC | 15:18 | |
belmoreira | mriedem: but anyway it looks a very dangerous operation to do when having placement. Before placement this was OK (recreating a new compute_node) | 15:18 |
mriedem | was the related nova-compute service deleted by any chance? | 15:18 |
belmoreira | mriedem: no, I don't yet | 15:18 |
mriedem | ok | 15:18 |
mriedem | for ironic the compute node record should have a predictable uuid | 15:18 |
mriedem | unless... | 15:19 |
mriedem | we don't use the ironic node uuid when creating the compute node | 15:19 |
*** gbarros has quit IRC | 15:19 | |
*** ttsiouts has joined #openstack-nova | 15:20 | |
stephenfin | mriedem, dansmith: We've been talking about extending the limits dict but I just noticed this https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1968-L1991 | 15:20 |
mriedem | looks like we don't https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L580 | 15:20 |
stephenfin | mriedem, dansmith: Given that we've essentially deprecated 'limits' and have 'requested_networks' available to us, couldn't I just use that as is? | 15:21 |
jaypipes | stephenfin: who's been talking about extending the limits dict? | 15:21 |
belmoreira | mriedem: that's the problem. When the compute_node is recreated we get a new uuid | 15:21 |
mriedem | belmoreira: so i think in that code, resources['hypervisor_hostname'] for ironic is always the node uuid, | 15:21 |
stephenfin | jaypipes: not me | 15:21 |
mriedem | we should use that when creating the compute node record if we think it's new | 15:21 |
* stephenfin hides | 15:21 | |
mriedem | belmoreira: yup, seems like a trivial fix | 15:21 |
mriedem | belmoreira: not sure if you have enough words to open a bug? | 15:21 |
*** munimeha1 has joined #openstack-nova | 15:22 | |
*** munimeha2 has joined #openstack-nova | 15:22 | |
*** shaohe_feng has quit IRC | 15:22 | |
mriedem | stephenfin: my guess is requested_networks aren't passed back to the cell conductor during a reschedule | 15:22 |
mriedem | oh it looks like they are https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1890 | 15:22 |
belmoreira | This is related with https://review.openstack.org/#/c/508555/ | 15:22 |
*** shaohe_feng has joined #openstack-nova | 15:23 | |
belmoreira | sorry not this one... | 15:23 |
stephenfin | mriedem: Yeah, I checked - the info is there | 15:23 |
stephenfin | For migrate and unshelve, I would need to check 'instance.info_cache.network_info' but that's a-ok | 15:24 |
*** suresh12 has joined #openstack-nova | 15:24 | |
kashyap | stephenfin: Thanks for the review here, https://review.openstack.org/#/c/567258/. Yep, that could go... | 15:25 |
*** itlinux has joined #openstack-nova | 15:25 | |
kashyap | I'll respin. | 15:25 |
*** fragatina has quit IRC | 15:25 | |
*** gyee has joined #openstack-nova | 15:26 | |
*** munimeha1 has quit IRC | 15:26 | |
mriedem | belmoreira: hmm, this should discover if there is already a compute node in the db with the same hypervisor_hostname, which is the ironic node uuid https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L518 | 15:26 |
belmoreira | mriedem: https://bugs.launchpad.net/nova/+bug/1771806 (the main issue here is the new UUID that's generated. Don't think is required to have a different bug) | 15:26 |
openstack | Launchpad bug 1771806 in OpenStack Compute (nova) "Ironic nova-compute failover creates new resource provider removing the resource_provider_aggregates link " [Medium,Confirmed] - Assigned to Surya Seetharaman (tssurya) | 15:26 |
openstackgerrit | Merged openstack/nova stable/ocata: Fix shelving a paused instance https://review.openstack.org/569655 | 15:26 |
mriedem | so it might not be a rebalance, but that code should have found it | 15:26 |
mriedem | unless you're hitting https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L531 | 15:27 |
*** tidwellr has quit IRC | 15:27 | |
mriedem | belmoreira: yeah, and in https://bugs.launchpad.net/nova/+bug/1771806 i asked why https://review.openstack.org/#/c/508555/ doesn't fix the issue, and i'm still kind of wondering that | 15:27 |
openstack | Launchpad bug 1771806 in OpenStack Compute (nova) "Ironic nova-compute failover creates new resource provider removing the resource_provider_aggregates link " [Medium,Confirmed] - Assigned to Surya Seetharaman (tssurya) | 15:27 |
mriedem | are you seeing the error from this condition? https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L531 | 15:28 |
*** pcaruana has quit IRC | 15:28 | |
mriedem | jroll: oath is cells v2 now right? | 15:28 |
mriedem | + placement | 15:28 |
mriedem | has to be if ocata | 15:28 |
jroll | mriedem: yeah | 15:29 |
jroll | single cell | 15:29 |
mriedem | is that cell VMs or BMs? | 15:29 |
*** tidwellr has joined #openstack-nova | 15:29 | |
jroll | VM | 15:29 |
mriedem | just wondering if you've seen similar issues to what belmiro is describing above | 15:29 |
mriedem | oh | 15:29 |
jroll | still working on the baremetal upgrade | 15:29 |
mriedem | ok | 15:29 |
jroll | about to jump in a meeting but will read back shortly | 15:30 |
*** ttsiouts has quit IRC | 15:30 | |
*** ttsiouts has joined #openstack-nova | 15:31 | |
belmoreira | mriedem: I didn't hit that because in that case doesn't rebalance | 15:31 |
*** ttsiouts has quit IRC | 15:31 | |
*** itlinux has quit IRC | 15:31 | |
*** itlinux has joined #openstack-nova | 15:31 | |
mriedem | belmoreira: sure but you don't need to rebalance to hit this check | 15:31 |
mriedem | the code that creates the new compute node record with the new uuid is here https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L580 | 15:31 |
mriedem | and right before that, we check the rebalance thing https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L574 | 15:32 |
mriedem | which will query the db for existing compute node recoreds by nodename (ironic node uuid) https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L518 | 15:32 |
jroll | mriedem: belmoreira: at a skim, this looks like https://bugs.launchpad.net/nova/+bug/1750450 | 15:32 |
openstack | Launchpad bug 1750450 in OpenStack Compute (nova) "ironic: n-cpu fails to recover after losing connection to ironic-api and placement-api" [Low,Fix released] - Assigned to Jim Rollenhagen (jim-rollenhagen) | 15:32 |
mriedem | so unless the node uuid changed, we should find the existing compute node reord | 15:32 |
*** shaohe_feng has quit IRC | 15:32 | |
*** vladikr has quit IRC | 15:33 | |
jroll | or similar | 15:33 |
mriedem | similar, but that says, | 15:33 |
mriedem | "When ironic-api and placement-api return, nova will see nodes, create compute_node records for them, and try to create new resource providers (as they are new compute_node records). This will fail with a name conflict, and the nodes will be unusable." | 15:33 |
mriedem | which, unless the node uuids change, should still find the existing compute node records in the db | 15:33 |
*** shaohe_feng has joined #openstack-nova | 15:33 | |
mriedem | here https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L518 | 15:33 |
*** chyka has joined #openstack-nova | 15:34 | |
*** awaugama has joined #openstack-nova | 15:34 | |
tssurya | mriedem: could be we return false here : https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L511 ? | 15:34 |
mriedem | tssurya: not if it's the ironic driver | 15:35 |
*** vladikr has joined #openstack-nova | 15:35 | |
mriedem | that's the only one that has rebalances_nodes = True | 15:35 |
mriedem | for the libvirt driver, yes | 15:35 |
tssurya | oh okay | 15:35 |
mriedem | but for the libvirt driver, you should only be able to get into this weird scenario if the physical hostname changes | 15:35 |
mriedem | for the compute host | 15:35 |
tssurya | yea understood | 15:36 |
*** mgoddard has quit IRC | 15:36 | |
mriedem | we probably want/need https://review.openstack.org/#/c/545479/ in stable branches | 15:37 |
*** AlexeyAbashkin has quit IRC | 15:37 | |
belmoreira | wouldn't the logic be simpler if compute_nodes aren't recreated and only have the "host" updated | 15:37 |
mriedem | belmoreira: that's what that _check_for_nodes_rebalance method tries to do | 15:38 |
mriedem | if it finds a compute node with the same nodename (ironic node uuid), update the host on the existing compute node record | 15:38 |
mriedem | we only create a compute node record if we can't find an existing one with the same nodename (or if we found >1) | 15:39 |
mriedem | which is why i was asking if you saw the >1 error in the logs | 15:39 |
belmoreira | give me few minutes... | 15:39 |
mriedem | i think probably regardless of this, we should probably have the virt drivers report a uuid up to the RT so it can use that to set ComputeNode.uuid if the virt driver has something it wants to use (like the ironic node uuid), so we don't just generate a random uuid | 15:40 |
*** udesale_ has quit IRC | 15:40 | |
mriedem | that would likely make debugging ironic stuff easier in the future | 15:40 |
belmoreira | +1 | 15:40 |
mriedem | stephenfin: regarding your question about using the requested_networks rather than the limits, that could technically work, but what i don't really like about that is then we have this special edge case of getting limits from 2 places | 15:42 |
mriedem | stephenfin: limits are more or less deprecated because of the filter scheduler and placement but that *only* covers vcpu/ram/disk, and the FilterScheduler, | 15:42 |
*** shaohe_feng has quit IRC | 15:43 | |
mriedem | the CachingScheduler and anything related to numa relies on limits from the scheduler still | 15:43 |
jroll | mriedem | we probably want/need https://review.openstack.org/#/c/545479/ in stable branches <- right, that's the bugfix for the thing I linked | 15:43 |
mriedem | jroll: yeah i know | 15:43 |
jroll | ok | 15:43 |
mriedem | jroll: the patch seemed to have some controversy, but it was merged so might as well be backported | 15:43 |
*** shaohe_feng has joined #openstack-nova | 15:43 | |
mriedem | stephenfin: i would need to double check, but i think the only thing we do claims on in the RT w/o the limits dict is the pci requests | 15:43 |
mriedem | because those were done differently, for whatever reason | 15:44 |
* jroll doesn't remember controversy, re-reads comments | 15:44 | |
mriedem | and the inconsistency is hella confusing | 15:44 |
*** moshele has joined #openstack-nova | 15:44 | |
stephenfin | mriedem: Quick debugging showed the limits dict was always empty | 15:44 |
stephenfin | Using filter_scheduler, anyway | 15:44 |
mriedem | stephenfin: for vcpu/ram/disk yes | 15:44 |
mriedem | the numa filter was, i thought, the only one that put stuff into it | 15:44 |
mriedem | if using filter scheduler | 15:44 |
mriedem | but you made some comment in vancouver about how that might not be true, which dansmith ack'ed and i didn't get | 15:45 |
mriedem | pci_requests are the snowflake right now https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L211 | 15:45 |
mriedem | if we passed through network_requests, then we'd have (1) limits (2) pci_requests and (3) network_requests, all for doing similar claims type stuff | 15:46 |
mriedem | which makes me want to do bad things, physically | 15:46 |
mriedem | maybe i'm the only one that feels this way though | 15:46 |
stephenfin | mriedem: You're referring to https://github.com/openstack/nova/blob/master/nova/scheduler/filters/numa_topology_filter.py#L102 ? | 15:46 |
mriedem | stephenfin: yes | 15:46 |
stephenfin | That does clear things up slightly. I was looking at the limits arg to 'numa_fit_instance_to_host' which is an object, not a dict. Hella confusing | 15:47 |
*** apple01 has quit IRC | 15:47 | |
mriedem | i think https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L354 is where we take the limits from the HostState after the filters run and shove them into the Selection object which gets passed back to conducotr | 15:48 |
mriedem | *conductor | 15:48 |
mriedem | yup https://github.com/openstack/nova/blob/master/nova/objects/selection.py#L53 | 15:48 |
mriedem | and then those are passed to compute here https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L1265 | 15:49 |
mriedem | so unless dansmith feels otherwise, i'd prefer to just continue using the limits dict for now for limits specific things | 15:49 |
mriedem | rather than add another wrinkle to where we get limits in the RT | 15:50 |
stephenfin | mriedem: So you're thinking I shouldn't be doing this https://review.openstack.org/#/c/564448/3/nova/virt/hardware.py@1609 right? | 15:50 |
stephenfin | i.e. no extra parameter | 15:50 |
stephenfin | instead, do limits['network_requests'] or the likes inside that function | 15:50 |
*** moshele has quit IRC | 15:51 | |
mriedem | i can't hardly read that code so i'm not sure what it's doing | 15:51 |
dansmith | mriedem: I'm on a call right now, but I asserted in our talk that limits is really just a temporary mechanism for communicating decisions from the scheduler filters to the lower layers, | 15:51 |
dansmith | which this is I think, so I think it fits | 15:51 |
dansmith | are you agreeing with that or asserting something else? | 15:52 |
mriedem | dansmith: you and i are in agreement, | 15:52 |
mriedem | stephenfin found that we pass requested_networks down to compute also, which will have the same type of informatoin in them so we could use that, and i'm asserting we shouldn't b/c it's confusing | 15:52 |
dansmith | agree | 15:52 |
mriedem | stephenfin: numa_fit_instance_to_host doesn't modify the limits dict today does it? | 15:53 |
*** shaohe_feng has quit IRC | 15:53 | |
*** mvk has joined #openstack-nova | 15:54 | |
*** Luzi has joined #openstack-nova | 15:54 | |
stephenfin | mriedem: Narp. _numa_fit_instance_cell consumes any NUMATopologyLimits objects contained therein but doesn't modify it | 15:54 |
mriedem | stephenfin: i don't think that numa_fit_instance_to_host should modify the limits dict, the scheduler filter should do that | 15:54 |
mriedem | i would basically like to keep all of this as decoupled as reasonably possible | 15:54 |
mriedem | because right now it's a tightly coupled mess | 15:55 |
*** shaohe_feng has joined #openstack-nova | 15:55 | |
mriedem | and it's like the last part of nova i'm not really familiar with, and don't really ever want to be if i can help it :) | 15:55 |
stephenfin | By scheduler filter you mean the NUMATopologyFilter? | 15:56 |
mriedem | yes | 15:56 |
mriedem | the thing that already puts the numa limits in the limits dict today | 15:56 |
mriedem | the scheduler filters are the only things that modify the limits dict | 15:56 |
mriedem | we should maintain that pattern | 15:56 |
openstackgerrit | Merged openstack/nova stable/ocata: Add ssbd and virt-ssbd flags to cpu_model_extra_flags whitelist https://review.openstack.org/570514 | 15:56 |
stephenfin | We're still going to need some way to get that info into the filter though | 15:57 |
stephenfin | So we'd still be modifying RequestSpec | 15:57 |
mriedem | the request spec will have the requested_networks right? | 15:57 |
mriedem | yes i know and agreed with that already | 15:57 |
*** tidwellr has quit IRC | 15:57 | |
mriedem | api -> modify reqspec.requested_networks like pci requests -> scheduler -> numa filter + modify limits -> conductor -> compute -> RT (claim from limits) -> allocate_for_instance -> update instance info cache with network physnet and tunneled info -> driver.spawn | 15:58 |
mriedem | done! | 15:58 |
stephenfin | Oh, so I still have the RequestSpec changes which means for the scheduler I'll just be undoing this change https://review.openstack.org/#/c/564448/3/nova/virt/hardware.py@1609 | 15:59 |
stephenfin | (where I add 'network_requests' | 15:59 |
stephenfin | ...and instead consuming 'network_requests' from limits | 15:59 |
belmoreira | mriedem: I don't have the logs for https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L531 | 15:59 |
stephenfin | Then I can use that for the call to 'instance_claim' | 16:00 |
stephenfin | Gotcha | 16:00 |
mriedem | stephenfin: i don't really know what that code is doing, so hard for me to say | 16:00 |
stephenfin | mriedem: I'm just referring to the function signature | 16:00 |
*** do3meli has quit IRC | 16:00 | |
mriedem | stephenfin: i'm missing the context at which numa_fit_instance_to_host gets called | 16:00 |
mriedem | i know the numa filter calls it, | 16:00 |
mriedem | but the RT does also correct? | 16:00 |
mriedem | well, _test_numa_topology in the claims code, called by the RT | 16:01 |
stephenfin | mriedem: Yeah, that's here https://review.openstack.org/#/c/564449/3/nova/compute/claims.py | 16:01 |
*** suresh12 has quit IRC | 16:01 | |
stephenfin | I could undo most of those changes | 16:01 |
mriedem | right so i think you don't need to pass requested_networks in there, just get it from the limits | 16:01 |
*** ispp has joined #openstack-nova | 16:01 | |
stephenfin | Exactly | 16:01 |
stephenfin | OK, that clears things up massively. Cheers :) | 16:01 |
mriedem | do you need to pass anything into numa_fit_instance_to_host from the numa filter? | 16:01 |
mriedem | stephenfin: heh, yeah, you were over-complicating this it sounds like | 16:02 |
stephenfin | I'll need to set the 'network_requests' field in limits but that's it | 16:02 |
mriedem | remember, at any given point in any given location in nova, we have at least 3 random dicts of stuff you can leverage for master hackery | 16:02 |
stephenfin | mriedem: Yeah, PCI requests probably wasn't the best pattern to copy | 16:02 |
mriedem | definitely not | 16:03 |
mriedem | i cringe whenver i see that | 16:03 |
*** shaohe_feng has quit IRC | 16:03 | |
*** chyka_ has joined #openstack-nova | 16:04 | |
*** shaohe_feng has joined #openstack-nova | 16:05 | |
*** belmoreira has quit IRC | 16:05 | |
*** Phuong_nh has joined #openstack-nova | 16:06 | |
*** tidwellr has joined #openstack-nova | 16:06 | |
*** tidwellr has quit IRC | 16:07 | |
*** itlinux has quit IRC | 16:07 | |
*** chyka has quit IRC | 16:07 | |
*** tidwellr has joined #openstack-nova | 16:07 | |
*** tidwellr has quit IRC | 16:07 | |
*** tidwellr has joined #openstack-nova | 16:07 | |
*** Phuongnh has quit IRC | 16:08 | |
openstackgerrit | Merged openstack/nova master: Update overriden to overridden https://review.openstack.org/571418 | 16:08 |
stephenfin | jaypipes: What's your thoughts on all the above? | 16:10 |
*** tidwellr has quit IRC | 16:10 | |
*** oomichi has joined #openstack-nova | 16:10 | |
mriedem | tssurya: i've added some detailed comments to https://bugs.launchpad.net/nova/+bug/1771806/comments/3 | 16:11 |
openstack | Launchpad bug 1771806 in OpenStack Compute (nova) "Ironic nova-compute failover creates new resource provider removing the resource_provider_aggregates link " [Medium,Confirmed] - Assigned to Surya Seetharaman (tssurya) | 16:11 |
*** markvoelker has joined #openstack-nova | 16:11 | |
stephenfin | jaypipes: tl;dr: We're going to stuff the NetworkRequestList object into the limits dictionary like we do for NUMATopologyLimits https://github.com/openstack/nova/blob/master/nova/scheduler/filters/numa_topology_filter.py#L102 | 16:11 |
mriedem | stephenfin: i didn't think it would be that entire NetworkRequestList object | 16:11 |
*** test_ has quit IRC | 16:11 | |
mriedem | i thought it would be actual specific limits things | 16:11 |
*** tianhui_ has quit IRC | 16:12 | |
stephenfin | mriedem: A new object so? | 16:12 |
*** sahid has quit IRC | 16:12 | |
mriedem | maybe, i'm not sure what your data structure needs to look like for the limits stuff | 16:12 |
mriedem | but i'm pretty sure it could certainly be stripped down from the full NetworkRequestList right? | 16:12 |
stephenfin | It can, yeah. I could reuse NUMANetworkInfo https://review.openstack.org/#/c/564439/3/nova/objects/numa.py | 16:12 |
*** derekh has quit IRC | 16:13 | |
*** Phuong__ has joined #openstack-nova | 16:13 | |
*** Phuong_nh has quit IRC | 16:13 | |
*** shaohe_feng has quit IRC | 16:13 | |
mriedem | maybe, whatever it is will go into the SchedulerLimits object https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py#L774 | 16:13 |
*** shaohe_feng has joined #openstack-nova | 16:14 | |
tssurya | mriedem: looking | 16:14 |
mriedem | tssurya: just the same stuff i said in irc, but with detailed links | 16:14 |
*** ispp has quit IRC | 16:14 | |
*** tidwellr has joined #openstack-nova | 16:15 | |
*** hughsaunders has joined #openstack-nova | 16:15 | |
*** amoralej is now known as amoralej|off | 16:15 | |
oomichi | alex_xu: are you still online now? happy if you take a look at https://review.openstack.org/#/c/571335 | 16:15 |
*** jpena is now known as jpena|off | 16:16 | |
*** sambetts is now known as sambetts|afk | 16:18 | |
melwitt | mriedem: I was thinking queens release will be a minor version bump because of this, do you agree? https://git.openstack.org/cgit/openstack/nova/commit/?h=stable/queens&id=45331d3ea185d08687e9b24e716f3d5411210874 | 16:19 |
*** Phuongnh has joined #openstack-nova | 16:20 | |
*** Eran_Kuris has joined #openstack-nova | 16:20 | |
mriedem | melwitt: umm, | 16:21 |
mriedem | i personally don't really see the need for a minor version bump, i don't know exactly what that will signal since we already have a release note | 16:21 |
*** ccamacho has quit IRC | 16:21 | |
melwitt | api change? I wasn't sure | 16:21 |
mriedem | about the only time i've seen stable branch minor version bumps is when we have to adjust dependencies, | 16:21 |
melwitt | okay | 16:21 |
mriedem | or maybe with a new db migration | 16:21 |
mriedem | the api change is to fix broken behavior | 16:22 |
mriedem | so 409 rather than screw up your data | 16:22 |
*** Phuong__ has quit IRC | 16:22 | |
melwitt | okay, I've got the bugfix version bump proposed at https://review.openstack.org/571494 | 16:23 |
*** tianhui has joined #openstack-nova | 16:23 | |
*** shaohe_feng has quit IRC | 16:24 | |
*** test_ has joined #openstack-nova | 16:24 | |
mriedem | i'd like to get https://review.openstack.org/#/c/571425/ in if we can | 16:24 |
*** itlinux has joined #openstack-nova | 16:24 | |
mriedem | the patch below it needs a +W | 16:24 |
*** markvoelker has quit IRC | 16:24 | |
mriedem | that was related to the nova-lvm regression we backported | 16:24 |
mriedem | bauzas: can you hit this? https://review.openstack.org/#/c/571424/1 | 16:24 |
melwitt | k | 16:24 |
*** Phuongnh has quit IRC | 16:25 | |
*** Phuong_nh has joined #openstack-nova | 16:25 | |
*** shaohe_feng has joined #openstack-nova | 16:25 | |
*** damien_r has quit IRC | 16:29 | |
melwitt | mriedem: will do similar for pike and ocata. we also have this for pike and ocata that just need +W https://review.openstack.org/561850 | 16:30 |
*** moshele has joined #openstack-nova | 16:32 | |
*** shaohe_feng has quit IRC | 16:34 | |
*** shaohe_feng has joined #openstack-nova | 16:35 | |
*** amotoki has quit IRC | 16:35 | |
*** Luzi has quit IRC | 16:36 | |
*** moshele has quit IRC | 16:37 | |
*** amotoki has joined #openstack-nova | 16:38 | |
*** artom has quit IRC | 16:38 | |
melwitt | release proposed for pike https://review.openstack.org/571521 and ocata https://review.openstack.org/571522 | 16:39 |
mriedem | jaypipes: bauzas: thanks for the comments on the heal_allocations change, i've replied https://review.openstack.org/#/c/565886/ | 16:40 |
mriedem | melwitt: ok i'll look at those backpots | 16:41 |
mriedem | *ports | 16:41 |
melwitt | thanks | 16:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add 'numa-aware-vswitches' spec https://review.openstack.org/541290 | 16:42 |
*** shaohe_feng has quit IRC | 16:44 | |
*** shaohe_feng has joined #openstack-nova | 16:45 | |
*** armaan has quit IRC | 16:47 | |
*** armaan has joined #openstack-nova | 16:48 | |
mriedem | melwitt: +W on both of those backports | 16:51 |
melwitt | sweet thanks | 16:51 |
*** suresh12 has joined #openstack-nova | 16:51 | |
melwitt | need to round up some reviews for the skip fetching virtual size patches | 16:52 |
melwitt | something in this one isn't passing unit test jobs tho https://review.openstack.org/571432 | 16:52 |
*** fragatina has joined #openstack-nova | 16:52 | |
*** shaohe_feng has quit IRC | 16:54 | |
mriedem | yeah i already talked with lee about it | 16:55 |
mriedem | i can call in stable maint core ringers if needed on the rest | 16:55 |
*** shaohe_feng has joined #openstack-nova | 16:55 | |
mriedem | i.e. mtreinish | 16:55 |
jaypipes | mriedem: "The missing consumer records" <-- worst Nancy Drew title ever. | 17:02 |
mriedem | easy libvirt driver change for the trusted vf series https://review.openstack.org/#/c/458514/ | 17:02 |
mriedem | jaypipes: it was a dark and stormy night and the consumer records were....MISSING | 17:02 |
mriedem | duh duh duh | 17:03 |
jaypipes | :) | 17:03 |
*** shaohe_feng has quit IRC | 17:05 | |
*** shaohe_feng has joined #openstack-nova | 17:05 | |
*** slaweq has joined #openstack-nova | 17:11 | |
*** shaohe_feng has quit IRC | 17:15 | |
*** felipemonteiro has joined #openstack-nova | 17:15 | |
*** slaweq has quit IRC | 17:16 | |
*** shaohe_feng has joined #openstack-nova | 17:16 | |
*** suresh12 has quit IRC | 17:16 | |
*** tssurya has quit IRC | 17:16 | |
*** shaohe_feng has quit IRC | 17:25 | |
*** shaohe_feng has joined #openstack-nova | 17:27 | |
*** tidwellr has quit IRC | 17:29 | |
*** tssurya has joined #openstack-nova | 17:31 | |
*** AlexeyAbashkin has joined #openstack-nova | 17:32 | |
*** Phuong__ has joined #openstack-nova | 17:34 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Match ComputeNode.uuid to ironic node uuid in RT https://review.openstack.org/571535 | 17:35 |
*** gyankum has quit IRC | 17:35 | |
*** shaohe_feng has quit IRC | 17:35 | |
*** shaohe_feng has joined #openstack-nova | 17:36 | |
*** Phuong_nh has quit IRC | 17:37 | |
*** felipemonteiro has quit IRC | 17:37 | |
*** felipemonteiro has joined #openstack-nova | 17:37 | |
*** Phuong__ has quit IRC | 17:37 | |
*** lpetrut has quit IRC | 17:37 | |
*** markvoelker has joined #openstack-nova | 17:39 | |
*** moshele has joined #openstack-nova | 17:40 | |
mriedem | is artom around today? | 17:41 |
*** tidwellr has joined #openstack-nova | 17:42 | |
*** gbarros has joined #openstack-nova | 17:44 | |
*** AlexeyAbashkin has quit IRC | 17:45 | |
*** AlexeyAbashkin has joined #openstack-nova | 17:45 | |
*** AlexeyAbashkin has quit IRC | 17:45 | |
*** shaohe_feng has quit IRC | 17:46 | |
melwitt | he was around earlier this morning but logged off about an hour ago | 17:46 |
openstackgerrit | karim proposed openstack/nova master: Handle rebuild of instances with image traits https://review.openstack.org/571539 | 17:46 |
*** AlexeyAbashkin has joined #openstack-nova | 17:46 | |
*** shaohe_feng has joined #openstack-nova | 17:47 | |
*** yamahata has joined #openstack-nova | 17:50 | |
*** edmondsw_ has joined #openstack-nova | 17:51 | |
*** markvoelker has quit IRC | 17:51 | |
*** moshele has quit IRC | 17:52 | |
*** itlinux has quit IRC | 17:53 | |
*** edmondsw has quit IRC | 17:54 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: metadata: add vf_trusted field to device metadata https://review.openstack.org/566344 | 17:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | 17:55 |
*** shaohe_feng has quit IRC | 17:56 | |
*** shaohe_feng has joined #openstack-nova | 17:57 | |
mriedem | melwitt: ok i saw you +2ed that trusted vf patch, the rest of the series has my +2, and the next 2 are pretty easy as well | 17:57 |
mriedem | the rest of the series is all pretty easy actually | 17:57 |
*** itlinux has joined #openstack-nova | 17:57 | |
mriedem | it's all just plumbing | 17:57 |
melwitt | coolness, I'll go through the rest of it later | 17:58 |
*** Tom-Tom has joined #openstack-nova | 17:58 | |
*** itlinux has quit IRC | 17:58 | |
*** moshele has joined #openstack-nova | 17:59 | |
*** gbarros has quit IRC | 18:02 | |
*** Tom-Tom has quit IRC | 18:02 | |
*** Alexey_Abashkin has joined #openstack-nova | 18:04 | |
*** itlinux has joined #openstack-nova | 18:05 | |
*** shaohe_feng has quit IRC | 18:06 | |
*** AlexeyAbashkin has quit IRC | 18:07 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 18:07 | |
*** shaohe_feng has joined #openstack-nova | 18:07 | |
*** moshele has quit IRC | 18:08 | |
*** AlexeyAbashkin has quit IRC | 18:13 | |
*** shaohe_feng has quit IRC | 18:16 | |
*** janki has quit IRC | 18:17 | |
*** shaohe_feng has joined #openstack-nova | 18:18 | |
*** armaan has quit IRC | 18:21 | |
*** armaan has joined #openstack-nova | 18:21 | |
openstackgerrit | Merged openstack/nova stable/pike: Avoid showing password in log https://review.openstack.org/561850 | 18:24 |
*** tbachman has quit IRC | 18:24 | |
*** tssurya has quit IRC | 18:24 | |
openstackgerrit | karim proposed openstack/nova master: Handle rebuild of instances with image traits https://review.openstack.org/571539 | 18:24 |
*** shaohe_feng has quit IRC | 18:27 | |
*** shaohe_feng has joined #openstack-nova | 18:27 | |
*** tidwellr has quit IRC | 18:31 | |
*** tesseract has quit IRC | 18:34 | |
*** oomichi has quit IRC | 18:35 | |
*** tianhui has quit IRC | 18:36 | |
*** tianhui has joined #openstack-nova | 18:36 | |
*** shaohe_feng has quit IRC | 18:37 | |
*** tidwellr has joined #openstack-nova | 18:37 | |
*** munimeha2 has quit IRC | 18:38 | |
*** shaohe_feng has joined #openstack-nova | 18:38 | |
*** pchavva has joined #openstack-nova | 18:38 | |
*** lifeless_ has quit IRC | 18:39 | |
*** tidwellr has quit IRC | 18:42 | |
*** ccamacho has joined #openstack-nova | 18:45 | |
*** shaohe_feng has quit IRC | 18:47 | |
*** shaohe_feng has joined #openstack-nova | 18:48 | |
openstackgerrit | Merged openstack/nova stable/queens: libvirt: handle DiskNotFound during update_available_resource https://review.openstack.org/571424 | 18:48 |
*** slaweq has joined #openstack-nova | 18:50 | |
*** mvk has quit IRC | 18:52 | |
*** lpetrut has joined #openstack-nova | 18:53 | |
*** slaweq has quit IRC | 18:54 | |
*** slaweq has joined #openstack-nova | 18:54 | |
*** tidwellr has joined #openstack-nova | 18:55 | |
mriedem | esberglu: you should add the on-demand volume check job to https://wiki.openstack.org/wiki/ThirdPartySystems/IBM_PowerVM_CI | 18:57 |
mriedem | but +W on https://review.openstack.org/#/c/526094/ | 18:57 |
*** shaohe_feng has quit IRC | 18:57 | |
esberglu | mriedem: ack. Thanks for the review! | 18:58 |
*** cdent has joined #openstack-nova | 18:58 | |
*** edmondsw_ is now known as edmondsw | 18:59 | |
*** shaohe_feng has joined #openstack-nova | 19:00 | |
mriedem | melwitt: with ^ the powervm series can be taken out of the runway | 19:01 |
melwitt | awesome | 19:01 |
melwitt | so next is going to be cert validation or cpu features as traits ... cert validation recently landed the spec amendment so should be good to go. thoughts? | 19:02 |
*** vladikr has quit IRC | 19:02 | |
*** lpetrut has quit IRC | 19:03 | |
*** lpetrut has joined #openstack-nova | 19:04 | |
*** armaan has quit IRC | 19:04 | |
*** fragatina has quit IRC | 19:05 | |
cfriesen | eandersson: you asked earlier about CPU pinning. In upstream the only way to see it is in the nova-compute logs or by looking at the DB. | 19:06 |
cfriesen | eandersson: internally we found it useful information and added a way to get it via the API, but never created a spec/blueprint for it. | 19:07 |
*** shaohe_feng has quit IRC | 19:08 | |
*** shaohe_feng has joined #openstack-nova | 19:10 | |
*** trozet has quit IRC | 19:12 | |
*** vladikr has joined #openstack-nova | 19:15 | |
*** dklyle has quit IRC | 19:18 | |
*** dklyle has joined #openstack-nova | 19:18 | |
*** shaohe_feng has quit IRC | 19:18 | |
*** dklyle has quit IRC | 19:19 | |
*** david-lyle has joined #openstack-nova | 19:19 | |
*** shaohe_feng has joined #openstack-nova | 19:20 | |
mriedem | melwitt: yes to cert validation | 19:20 |
melwitt | thanks | 19:20 |
mriedem | cfriesen: to which API? | 19:21 |
*** eharney has quit IRC | 19:23 | |
dansmith | I would like to see cert validation get another go and hopefully get finished | 19:24 |
melwitt | me too. thanks for the input, will move that into the runway next | 19:26 |
*** cdent has quit IRC | 19:27 | |
*** shaohe_feng has quit IRC | 19:28 | |
melwitt | mriedem: I'm working on getting the osc queens + novaclient queens stuff working with these patches https://review.openstack.org/#/q/topic:openstackclient-novaclient+(status:open+OR+status:merged) but from what I understand, nothing in the gate will verify those go together | 19:29 |
*** david-lyle has quit IRC | 19:29 | |
*** dklyle has joined #openstack-nova | 19:29 | |
*** shaohe_feng has joined #openstack-nova | 19:30 | |
melwitt | I was thinking I might need to do some DNM patches to devstack to pull osc from git stable/queens and then a DNM revert of this https://review.openstack.org/538541 in grenade with depends-on to do it. would that work? is there a better way? | 19:30 |
*** sridharg has quit IRC | 19:32 | |
*** toabctl has quit IRC | 19:33 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Use oslo.messaging per-call monitoring https://review.openstack.org/566696 | 19:34 |
*** toabctl has joined #openstack-nova | 19:36 | |
*** armaan has joined #openstack-nova | 19:37 | |
*** lifeless has joined #openstack-nova | 19:38 | |
mriedem | melwitt: your question is breaking my brain | 19:38 |
*** shaohe_feng has quit IRC | 19:38 | |
mriedem | you want to test osc 3.14.2 with novaclient 10.1.0? | 19:39 |
mriedem | in queens, novaclient 10.x broke grenade and horizon, which got fixed in queens, | 19:39 |
melwitt | sorry ... that was too low-level a way to ask. yes, but osc 3.14.2 is not released yet, I was hoping to be able to verify it solves the problem before landing it if possible | 19:39 |
mriedem | the only remaining things that i knew of from the 10.x fallout was osc | 19:39 |
melwitt | yeah, that's my understanding too. I think it will work but realized I don't think anything in the gate run on the constraint update patch will actually run anything with those two together to make super sure it works | 19:40 |
*** shaohe_feng has joined #openstack-nova | 19:40 | |
mriedem | so you're thinking revert https://review.openstack.org/#/c/538541/ on stable/queens and depend on https://review.openstack.org/#/c/571540/ to get novaclient 10.1.0 so osc in grenade will use the osc proxy code that used to rely on novaclient, | 19:42 |
mriedem | i think the problem with that is the grenade revert won't run with the osc fix until that's released in https://review.openstack.org/#/c/571537/ | 19:42 |
mriedem | i think you can hack around that, | 19:42 |
mriedem | by depending on a stable/queens devstack patch that installs osc from stable/queens source | 19:43 |
melwitt | yeah. I was thinking I could make that happen tho with a devstack DNM patch that does LIBS_FROM_GIT=python-openstackclient on branch stable/queens and then depends-on that | 19:43 |
*** armaan has quit IRC | 19:43 | |
mriedem | so you'd have something like, queens grenade revert -> depends-on -> devstack queens osc from source -> depends-on -> novaclient constraints update https://review.openstack.org/#/c/571540/ | 19:43 |
melwitt | yeah exactly. just wanted to run it by you since your depends-on patch-verification-fu is strong | 19:43 |
mriedem | that's what i'd try i guess | 19:44 |
mriedem | OR | 19:44 |
mriedem | just get a queens devstack local and kick the tires | 19:44 |
melwitt | seeing something pass in the gate is more reassuring since I might screw up something in devstack somehow. but point taken | 19:45 |
melwitt | and also other people reviewing the changes can see it work, otherwise it's just "I tested it in devstack, you can trust me *wink*" | 19:48 |
mriedem | trust by verify? | 19:48 |
mriedem | i didn't know you were so into reagan | 19:48 |
*** READ10 has quit IRC | 19:49 | |
*** shaohe_feng has quit IRC | 19:49 | |
*** shaohe_feng has joined #openstack-nova | 19:50 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove support for /os-virtual-interfaces REST API https://review.openstack.org/569923 | 19:55 |
*** armaan has joined #openstack-nova | 19:56 | |
*** artom has joined #openstack-nova | 19:57 | |
*** shaohe_feng has quit IRC | 19:59 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove support for /os-virtual-interfaces REST API https://review.openstack.org/569923 | 20:00 |
*** shaohe_feng has joined #openstack-nova | 20:00 | |
*** harlowja has joined #openstack-nova | 20:01 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove support for /os-virtual-interfaces REST API https://review.openstack.org/569923 | 20:02 |
eandersson | cfriesen, I see - yea was thinking of doing the same internally | 20:02 |
mriedem | eandersson: or, OR, you could work with the upstream community on digging ourself out of your hole | 20:02 |
mriedem | *yourself | 20:03 |
*** armaan has quit IRC | 20:03 | |
eandersson | haha yea that would probably be the more sensable thing to do =] | 20:04 |
*** trozet has joined #openstack-nova | 20:09 | |
*** shaohe_feng has quit IRC | 20:09 | |
*** shaohe_feng has joined #openstack-nova | 20:11 | |
mriedem | i'm trying hard to not make a starlingx comment here | 20:12 |
mriedem | if you called it rock-n-roll-racing-x then i could maybe buy into it | 20:13 |
*** pchavva has quit IRC | 20:14 | |
*** eharney has joined #openstack-nova | 20:17 | |
cfriesen | mriedem: i'm probably corporately obligated to keep my mouth shut. :) | 20:19 |
*** tbachman has joined #openstack-nova | 20:19 | |
*** shaohe_feng has quit IRC | 20:19 | |
*** shaohe_feng has joined #openstack-nova | 20:20 | |
mriedem | heh | 20:21 |
mriedem | it's not like huawei doesn't make changes | 20:21 |
cfriesen | mriedem: sorry, missed your earlier question. I was referring to providing CPU pinning (and hugepages) consumption data to an admin user via the API. | 20:21 |
mriedem | but i fight the same battle internally | 20:21 |
mriedem | cfriesen: sure but on the servers API or the hypervisors API? | 20:22 |
cfriesen | mriedem: ours is on the servers API, but we're private cloud and let people see more host-side info | 20:22 |
mriedem | well, you could do it on the servers API sure, we expose host/node etc, and that's controlled via policy | 20:23 |
mriedem | the same could be done here | 20:23 |
mriedem | someone just has to, you know, propose that api change in a spec first | 20:23 |
mriedem | artom: thanks for making those changes to the trusted vf series btw | 20:24 |
cfriesen | mriedem: http://paste.openstack.org/show/722471/ <-- an example of the stuff we added to "nova show" | 20:24 |
*** cfriesen is now known as cfriesen_away | 20:25 | |
artom | mriedem, sure, thank you for finalizing your own nits :) | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Match ComputeNode.uuid to ironic node uuid in RT https://review.openstack.org/571535 | 20:26 |
mriedem | cfriesen_away: those don't seem crazy, and the nics thing is actually something that's been discussed a few times upstream, i.e. cleaning up GET /servers/{id}/ips and making the 'addresses' field in the existing response more useful | 20:27 |
melwitt | this is something where I feel like if we had enough info to put together a backlog spec, it might be something various contributors in the community might like to work on | 20:28 |
mriedem | reminds me of https://review.openstack.org/#/q/topic:521392+(status:open+OR+status:merged) which was approved, then needed a spec update, and seems to have been abandoned now | 20:30 |
*** shaohe_feng has quit IRC | 20:30 | |
*** shaohe_feng has joined #openstack-nova | 20:30 | |
*** lpetrut has quit IRC | 20:31 | |
*** AlexeyAbashkin has joined #openstack-nova | 20:33 | |
*** lifeless_ has joined #openstack-nova | 20:36 | |
*** lifeless has quit IRC | 20:36 | |
*** wolverineav has quit IRC | 20:37 | |
dansmith | mriedem: I'm surprised that nics thing doesn't run contrary to your no-proxies leanings | 20:37 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add osprofiler config options to generated reference https://review.openstack.org/571225 | 20:39 |
mriedem | dansmith: i lean whichever way the winds of cash blow me | 20:39 |
dansmith | oh, well, then | 20:39 |
*** shaohe_feng has quit IRC | 20:40 | |
*** trozet has quit IRC | 20:40 | |
mriedem | we still have things like os-volume_attachments to show bdms attached to an instance, and os-interfaces to show ports attached to an instance, | 20:40 |
mriedem | and we've talked about leveraging those to return the tags associated with those devices | 20:41 |
mriedem | so we'll still have some things in the API as long as they are per-server | 20:41 |
mriedem | definitely not the stuff we had before, like CRUD ops on volumes/images/ports/networks | 20:41 |
*** shaohe_feng has joined #openstack-nova | 20:43 | |
openstackgerrit | Merged openstack/nova master: PowerVM Driver: vSCSI Fibre Channel volume adapter https://review.openstack.org/526094 | 20:46 |
openstackgerrit | Merged openstack/nova master: PowerVM snapshot cleanup https://review.openstack.org/568988 | 20:46 |
dansmith | volume attachments shows the mapping, but exposing things like the mtu is getting pretty specific | 20:47 |
dansmith | os-interfaces definitely exposes more stuff like that | 20:47 |
dansmith | but I dunno, I just don't know how to answer the question of why we don't have CRUD operations for all the values we expose from other services | 20:48 |
dansmith | I get the desire and need to be able to easily get all the IPs for the interfaces associated with the host, but not really network-specific things like mtu | 20:48 |
dansmith | s/host/instance/ | 20:48 |
*** moshele has joined #openstack-nova | 20:50 | |
*** shaohe_feng has quit IRC | 20:50 | |
*** AlexeyAbashkin has quit IRC | 20:50 | |
*** markvoelker has joined #openstack-nova | 20:51 | |
melwitt | dansmith: last patch for powervm merged so I've updated the runways, if you could do the honors updating the channel topic pls https://etherpad.openstack.org/p/nova-runways-rocky | 20:52 |
*** shaohe_feng has joined #openstack-nova | 20:52 | |
*** ChanServ sets mode: +o dansmith | 20:53 | |
*** dansmith changes topic to "Current runways: granular-placement-policy / vgpu-rocky / nova-validate-certificates -- This channel is for Nova development. For support of Nova deployments, please use #openstack." | 20:54 | |
melwitt | thanks | 20:54 |
*** ChanServ sets mode: -o dansmith | 20:55 | |
dansmith | yup | 20:55 |
*** fragatina has joined #openstack-nova | 20:55 | |
*** bpoulos has joined #openstack-nova | 20:56 | |
*** harlowja has quit IRC | 21:00 | |
*** r-daneel has quit IRC | 21:00 | |
*** shaohe_feng has quit IRC | 21:00 | |
*** shaohe_feng has joined #openstack-nova | 21:01 | |
*** throwsb1 has quit IRC | 21:03 | |
openstackgerrit | Zack Cornelius proposed openstack/nova master: Implement file backed memory for instances in libvirt https://review.openstack.org/567876 | 21:04 |
*** bpoulos has quit IRC | 21:05 | |
*** cfriesen_away is now known as cfriesen | 21:06 | |
*** mikal has joined #openstack-nova | 21:08 | |
cfriesen | dansmith: personally I agree...I'd actually be fine with declaring that you have to pre-create ports in neutron and then pass them to nova. if desired the client could cross-reference with neutron for detailed info. | 21:09 |
dansmith | yeah, I also understand the network-id being there to assist in the mapping process, but anything that's not mapping should really be fetched from neutron I think | 21:10 |
*** mikal_ has quit IRC | 21:10 | |
*** shaohe_feng has quit IRC | 21:11 | |
*** tidwellr has quit IRC | 21:11 | |
*** moshele has quit IRC | 21:12 | |
cfriesen | in case it's not apparent, for dedicated cpus or multi-numa-node instances the "wrs-res:topology" line would have additional information | 21:13 |
*** shaohe_feng has joined #openstack-nova | 21:13 | |
*** moshele has joined #openstack-nova | 21:17 | |
*** arvindn05 has quit IRC | 21:17 | |
*** dklyle has quit IRC | 21:17 | |
*** dklyle has joined #openstack-nova | 21:18 | |
*** arvindn05 has joined #openstack-nova | 21:19 | |
*** wolverineav has joined #openstack-nova | 21:19 | |
cfriesen | what's the story with "meta.js" in the guest...is it still expected to work? I thought I remembered something about getting away from file injection. | 21:19 |
*** shaohe_feng has quit IRC | 21:19 | |
*** slaweq has quit IRC | 21:19 | |
*** slaweq has joined #openstack-nova | 21:20 | |
*** shaohe_feng has joined #openstack-nova | 21:21 | |
*** slaweq has quit IRC | 21:24 | |
*** markvoelker has quit IRC | 21:26 | |
*** markvoelker has joined #openstack-nova | 21:26 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Mention running rootwrap in daemon mode if hitting vif plug timeouts https://review.openstack.org/571577 | 21:28 |
*** moshele has quit IRC | 21:29 | |
*** tidwellr has joined #openstack-nova | 21:30 | |
*** markvoelker has quit IRC | 21:31 | |
*** shaohe_feng has quit IRC | 21:31 | |
*** shaohe_feng has joined #openstack-nova | 21:32 | |
openstackgerrit | Julia Kreger proposed openstack/nova master: ironic: add instance_uuid before any other spawn activity https://review.openstack.org/563722 | 21:34 |
*** tidwellr has quit IRC | 21:35 | |
*** cdent has joined #openstack-nova | 21:40 | |
*** shaohe_feng has quit IRC | 21:41 | |
*** links has joined #openstack-nova | 21:42 | |
*** shaohe_feng has joined #openstack-nova | 21:43 | |
*** sar has joined #openstack-nova | 21:43 | |
*** shaohe_feng has quit IRC | 21:52 | |
*** sar has quit IRC | 21:52 | |
*** shaohe_feng has joined #openstack-nova | 21:53 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: always create consumer records https://review.openstack.org/567678 | 21:54 |
openstackgerrit | Jay Pipes proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958 | 21:54 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405 | 21:54 |
openstackgerrit | Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | 21:54 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 21:54 |
*** hoonetorg has quit IRC | 21:55 | |
openstackgerrit | Jan Gutter proposed openstack/os-vif master: Add multiqueue port profile for VIFGeneric https://review.openstack.org/570959 | 21:56 |
*** Swami_ has quit IRC | 21:57 | |
openstackgerrit | Jan Gutter proposed openstack/nova master: Pass virtio multiqueue info to os-vif plugins https://review.openstack.org/571462 | 21:57 |
openstackgerrit | Jan Gutter proposed openstack/nova master: Convert vrouter legacy plugging to os-vif https://review.openstack.org/571325 | 21:57 |
*** liuyulong has quit IRC | 21:57 | |
*** Tom-Tom has joined #openstack-nova | 21:58 | |
mriedem | cfriesen: personality files (file injection) was deprecated from the api in queens | 21:58 |
*** awaugama has quit IRC | 21:58 | |
mriedem | https://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/deprecate-file-injection.html | 21:59 |
*** jangutter has quit IRC | 21:59 | |
mriedem | cfriesen: and i assume you meant meta_data.json? | 21:59 |
mriedem | you just wouldn't get the 'files' key in there anymore | 22:00 |
*** shaohe_feng has quit IRC | 22:02 | |
*** Tom-Tom has quit IRC | 22:03 | |
*** shaohe_feng has joined #openstack-nova | 22:03 | |
*** itlinux has quit IRC | 22:04 | |
*** mriedem has quit IRC | 22:05 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 22:05 |
*** hoonetorg has joined #openstack-nova | 22:08 | |
*** slaweq has joined #openstack-nova | 22:11 | |
*** shaohe_feng has quit IRC | 22:12 | |
*** shaohe_feng has joined #openstack-nova | 22:13 | |
*** rcernin has joined #openstack-nova | 22:13 | |
*** esberglu has quit IRC | 22:14 | |
*** esberglu has joined #openstack-nova | 22:14 | |
*** slaweq has quit IRC | 22:15 | |
*** links has quit IRC | 22:16 | |
*** lbragstad has quit IRC | 22:18 | |
*** eharney has quit IRC | 22:18 | |
*** esberglu has quit IRC | 22:19 | |
*** harlowja has joined #openstack-nova | 22:21 | |
*** shaohe_feng has quit IRC | 22:22 | |
*** lbragstad has joined #openstack-nova | 22:23 | |
*** shaohe_feng has joined #openstack-nova | 22:24 | |
*** fragatina has quit IRC | 22:25 | |
*** fragatina has joined #openstack-nova | 22:25 | |
*** jangutter has joined #openstack-nova | 22:25 | |
*** takedakn has joined #openstack-nova | 22:29 | |
*** shaohe_feng has quit IRC | 22:33 | |
*** lifeless_ has quit IRC | 22:33 | |
*** felipemonteiro has quit IRC | 22:33 | |
*** shaohe_feng has joined #openstack-nova | 22:34 | |
*** lifeless has joined #openstack-nova | 22:34 | |
*** cdent has quit IRC | 22:40 | |
*** hongbin has quit IRC | 22:41 | |
*** wolverineav has quit IRC | 22:42 | |
*** shaohe_feng has quit IRC | 22:43 | |
*** shaohe_feng has joined #openstack-nova | 22:44 | |
*** shaohe_feng has quit IRC | 22:53 | |
*** shaohe_feng has joined #openstack-nova | 22:55 | |
*** chyka_ has quit IRC | 23:03 | |
*** shaohe_feng has quit IRC | 23:03 | |
*** shaohe_feng has joined #openstack-nova | 23:04 | |
*** SpamapS has joined #openstack-nova | 23:05 | |
*** takedakn has quit IRC | 23:08 | |
openstackgerrit | Merged openstack/nova stable/ocata: Avoid showing password in log https://review.openstack.org/561851 | 23:11 |
*** lyan has quit IRC | 23:13 | |
*** shaohe_feng has quit IRC | 23:14 | |
*** shaohe_feng has joined #openstack-nova | 23:16 | |
*** lyan has joined #openstack-nova | 23:18 | |
*** shaohe_feng has quit IRC | 23:24 | |
*** shaohe_feng has joined #openstack-nova | 23:25 | |
*** moshele has joined #openstack-nova | 23:25 | |
*** moshele has quit IRC | 23:30 | |
*** mlavalle has quit IRC | 23:32 | |
*** shaohe_feng has quit IRC | 23:34 | |
*** shaohe_feng has joined #openstack-nova | 23:35 | |
*** shaohe_feng has quit IRC | 23:44 | |
*** shaohe_feng has joined #openstack-nova | 23:47 | |
*** tidwellr has joined #openstack-nova | 23:52 | |
*** shaohe_feng has quit IRC | 23:55 | |
*** shaohe_feng has joined #openstack-nova | 23:55 | |
*** tidwellr has quit IRC | 23:56 | |
*** rajinir has quit IRC | 23:57 | |
*** cdent has joined #openstack-nova | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!