*** germs has quit IRC | 00:00 | |
*** dpawlik has joined #openstack-nova | 00:03 | |
*** fragatin_ has quit IRC | 00:03 | |
*** dpawlik has quit IRC | 00:08 | |
*** fragatina has joined #openstack-nova | 00:11 | |
*** jogo has quit IRC | 00:11 | |
*** yamamoto has joined #openstack-nova | 00:20 | |
*** fragatina has quit IRC | 00:21 | |
*** yamamoto has quit IRC | 00:24 | |
*** yamamoto has joined #openstack-nova | 00:41 | |
*** hoangcx has joined #openstack-nova | 00:46 | |
*** hshiina|afk is now known as hshiina | 00:46 | |
*** tiendc has joined #openstack-nova | 00:52 | |
*** dave-mccowan has joined #openstack-nova | 01:00 | |
*** dave-mcc_ has joined #openstack-nova | 01:02 | |
*** jichen has joined #openstack-nova | 01:05 | |
*** dave-mccowan has quit IRC | 01:05 | |
*** phuongnh has joined #openstack-nova | 01:08 | |
*** jogo has joined #openstack-nova | 01:08 | |
*** namnh has joined #openstack-nova | 01:17 | |
*** tomtom001 has quit IRC | 01:19 | |
*** tomtom001 has joined #openstack-nova | 01:20 | |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 01:23 |
---|---|---|
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 01:24 |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 01:24 |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 01:24 |
*** salv-orlando has joined #openstack-nova | 01:28 | |
*** salv-orl_ has quit IRC | 01:31 | |
*** cfriesen has joined #openstack-nova | 01:32 | |
*** zhaochao has joined #openstack-nova | 01:34 | |
*** bkopilov has quit IRC | 01:37 | |
*** cfriesen has quit IRC | 01:39 | |
*** Tom-Tom has joined #openstack-nova | 01:40 | |
*** sapd has joined #openstack-nova | 01:43 | |
*** takashin has joined #openstack-nova | 01:44 | |
*** tuanla____ has joined #openstack-nova | 01:53 | |
*** dougshel_ has quit IRC | 01:59 | |
*** masuberu has quit IRC | 02:06 | |
*** masuberu has joined #openstack-nova | 02:06 | |
*** annp has joined #openstack-nova | 02:16 | |
*** jichen has quit IRC | 02:18 | |
*** Guest84910 has quit IRC | 02:22 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Send out notification when instance metadata updated https://review.openstack.org/558140 | 02:22 |
*** Guest84910 has joined #openstack-nova | 02:25 | |
*** moshele has joined #openstack-nova | 02:27 | |
*** phuongnh has quit IRC | 02:28 | |
*** armaan has quit IRC | 02:29 | |
*** armaan has joined #openstack-nova | 02:29 | |
openstackgerrit | Merged openstack/python-novaclient master: Trivial: Update pypi url to new url https://review.openstack.org/563060 | 02:33 |
*** Dinesh_Bhor has joined #openstack-nova | 02:39 | |
*** dklyle has joined #openstack-nova | 02:41 | |
*** psachin has joined #openstack-nova | 02:44 | |
*** bkopilov has joined #openstack-nova | 02:49 | |
*** mdnadeem has joined #openstack-nova | 03:00 | |
*** dklyle has quit IRC | 03:00 | |
*** masuberu has quit IRC | 03:06 | |
*** masuberu has joined #openstack-nova | 03:08 | |
*** masber has joined #openstack-nova | 03:15 | |
*** Tom-Tom has quit IRC | 03:17 | |
*** Tom-Tom has joined #openstack-nova | 03:18 | |
*** masuberu has quit IRC | 03:18 | |
*** tomtom002 has joined #openstack-nova | 03:22 | |
*** Tom-Tom has quit IRC | 03:23 | |
*** Tom-Tom has joined #openstack-nova | 03:24 | |
*** tomtom001 has quit IRC | 03:24 | |
*** Tom-Tom_ has joined #openstack-nova | 03:26 | |
*** Tom-Tom has quit IRC | 03:29 | |
*** rcernin has quit IRC | 03:31 | |
*** rcernin has joined #openstack-nova | 03:35 | |
*** tomtom002 has quit IRC | 03:37 | |
*** tomtom001 has joined #openstack-nova | 03:38 | |
*** lpetrut has joined #openstack-nova | 03:39 | |
*** psachin has quit IRC | 03:39 | |
*** takashin has quit IRC | 03:43 | |
*** Tom-Tom_ has quit IRC | 03:45 | |
*** Guest84910 has quit IRC | 03:47 | |
*** hongbin_ has quit IRC | 03:50 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Send out notification when instance metadata updated https://review.openstack.org/558140 | 03:50 |
*** mdnadeem has quit IRC | 03:53 | |
*** edmondsw has joined #openstack-nova | 03:55 | |
*** dave-mcc_ has quit IRC | 04:00 | |
*** Guest84910 has joined #openstack-nova | 04:01 | |
*** Nisha_Agarwal has joined #openstack-nova | 04:02 | |
*** phuongnh has joined #openstack-nova | 04:02 | |
*** bhagyashris has joined #openstack-nova | 04:10 | |
*** lpetrut has quit IRC | 04:11 | |
*** lpetrut has joined #openstack-nova | 04:22 | |
*** sridharg has joined #openstack-nova | 04:32 | |
*** Tom-Tom has joined #openstack-nova | 04:35 | |
*** Tom-Tom has quit IRC | 04:39 | |
*** abhishekk has joined #openstack-nova | 04:46 | |
*** lpetrut has quit IRC | 04:46 | |
*** pooja_jadhav has joined #openstack-nova | 04:47 | |
*** takashin has joined #openstack-nova | 04:52 | |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 04:53 |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 04:53 |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 04:53 |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 04:53 |
*** armaan has quit IRC | 04:54 | |
*** armaan has joined #openstack-nova | 04:54 | |
*** moshele has quit IRC | 04:55 | |
*** psachin has joined #openstack-nova | 05:00 | |
*** edmondsw has quit IRC | 05:03 | |
*** edmondsw has joined #openstack-nova | 05:03 | |
*** Nisha_Agarwal has quit IRC | 05:08 | |
*** edmondsw has quit IRC | 05:08 | |
*** tuanla____ has quit IRC | 05:26 | |
*** vivsoni__ has quit IRC | 05:27 | |
*** vivsoni has joined #openstack-nova | 05:29 | |
*** sridharg has quit IRC | 05:30 | |
*** vivsoni has quit IRC | 05:31 | |
*** ratailor has joined #openstack-nova | 05:35 | |
*** Zames has joined #openstack-nova | 05:40 | |
*** Zames has quit IRC | 05:43 | |
*** sridharg has joined #openstack-nova | 05:43 | |
*** stakeda has joined #openstack-nova | 05:43 | |
*** Tom-Tom has joined #openstack-nova | 05:47 | |
*** moshele has joined #openstack-nova | 05:50 | |
*** Lingwu has quit IRC | 06:07 | |
*** Lingwu has joined #openstack-nova | 06:08 | |
*** blkart has quit IRC | 06:08 | |
*** sar has joined #openstack-nova | 06:12 | |
*** links has joined #openstack-nova | 06:13 | |
*** lajoskatona has joined #openstack-nova | 06:14 | |
*** blkart has joined #openstack-nova | 06:17 | |
*** lpetrut has joined #openstack-nova | 06:21 | |
*** kaisers1 has quit IRC | 06:31 | |
*** kaisers has joined #openstack-nova | 06:33 | |
*** ircuser-1 has joined #openstack-nova | 06:39 | |
*** tiendc has quit IRC | 06:45 | |
*** tiendc has joined #openstack-nova | 06:46 | |
*** tomtom001 has quit IRC | 06:52 | |
*** tomtom001 has joined #openstack-nova | 06:52 | |
*** ccamacho has joined #openstack-nova | 06:56 | |
*** sahid has joined #openstack-nova | 06:58 | |
*** tesseract has joined #openstack-nova | 07:04 | |
*** tomtom001 has quit IRC | 07:06 | |
*** tomtom001 has joined #openstack-nova | 07:08 | |
*** jpena|off is now known as jpena | 07:10 | |
*** sapd has quit IRC | 07:13 | |
*** xinliang has joined #openstack-nova | 07:14 | |
*** xinliang has quit IRC | 07:14 | |
*** xinliang has joined #openstack-nova | 07:14 | |
*** janki has joined #openstack-nova | 07:15 | |
*** slaweq has joined #openstack-nova | 07:16 | |
openstackgerrit | Moshe Levi proposed openstack/nova stable/pike: Don't overwrite binding-profile https://review.openstack.org/563501 | 07:17 |
*** ralonsoh has joined #openstack-nova | 07:19 | |
*** rcernin has quit IRC | 07:19 | |
*** dpawlik has joined #openstack-nova | 07:19 | |
*** jchhatbar has joined #openstack-nova | 07:21 | |
*** dpawlik has quit IRC | 07:22 | |
*** dpawlik has joined #openstack-nova | 07:22 | |
*** janki has quit IRC | 07:23 | |
*** pcaruana has joined #openstack-nova | 07:25 | |
*** jogo has quit IRC | 07:27 | |
*** AlexeyAbashkin has joined #openstack-nova | 07:36 | |
*** damien_r has joined #openstack-nova | 07:37 | |
*** zz9pzza has quit IRC | 07:39 | |
*** Tom-Tom has quit IRC | 07:40 | |
*** Tom-Tom has joined #openstack-nova | 07:41 | |
*** markmc`` is now known as markmc | 07:41 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: WIP new migration threads control https://review.openstack.org/563505 | 07:42 |
*** damien_r has quit IRC | 07:42 | |
*** amoralej|off is now known as amoralej | 07:42 | |
*** damien_r has joined #openstack-nova | 07:42 | |
*** markmc is now known as markmc`` | 07:42 | |
gibi | good morning nova | 07:47 |
*** zz9pzza has joined #openstack-nova | 07:51 | |
*** alexchadin has joined #openstack-nova | 07:56 | |
*** kholkina has joined #openstack-nova | 07:58 | |
*** Dinesh__Bhor has joined #openstack-nova | 08:03 | |
*** Dinesh_Bhor has quit IRC | 08:05 | |
*** mgoddard has joined #openstack-nova | 08:07 | |
*** mdnadeem has joined #openstack-nova | 08:13 | |
*** jaosorior has joined #openstack-nova | 08:15 | |
*** lucas-afk is now known as lucasagomes | 08:15 | |
*** threestrands has quit IRC | 08:17 | |
*** finucannot is now known as stephenfin | 08:20 | |
sahid | sean-k-mooney[m]: how can i update tempest-dsvm-ovsdpdk-nfv-networking-xenial ? | 08:22 |
*** armaan has quit IRC | 08:24 | |
*** jogo has joined #openstack-nova | 08:24 | |
*** jogo has joined #openstack-nova | 08:24 | |
*** armaan has joined #openstack-nova | 08:25 | |
*** derekh has joined #openstack-nova | 08:27 | |
*** takashin has left #openstack-nova | 08:30 | |
*** jaosorior has quit IRC | 08:32 | |
*** Tom-Tom has quit IRC | 08:37 | |
*** phuongnh has quit IRC | 08:41 | |
*** Tom-Tom has joined #openstack-nova | 08:44 | |
*** sambetts_ is now known as sambetts | 08:46 | |
*** Alexey_Abashkin has joined #openstack-nova | 08:48 | |
*** Dinesh__Bhor has quit IRC | 08:49 | |
*** Alexey_Abashkin has quit IRC | 08:50 | |
*** Alexey_Abashkin has joined #openstack-nova | 08:50 | |
*** AlexeyAbashkin has quit IRC | 08:51 | |
*** sapd has joined #openstack-nova | 08:53 | |
*** Dinesh_Bhor has joined #openstack-nova | 08:53 | |
*** Tom-Tom_ has joined #openstack-nova | 08:53 | |
*** AlexeyAbashkin has joined #openstack-nova | 08:54 | |
*** Alexey_Abashkin has quit IRC | 08:54 | |
*** sapd1 has joined #openstack-nova | 08:56 | |
*** Tom-Tom has quit IRC | 08:56 | |
*** Alexey_Abashkin has joined #openstack-nova | 09:02 | |
*** AlexeyAbashkin has quit IRC | 09:05 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 09:05 | |
*** jaosorior has joined #openstack-nova | 09:07 | |
*** brault has joined #openstack-nova | 09:10 | |
*** alexchad_ has joined #openstack-nova | 09:10 | |
*** alexchadin has quit IRC | 09:12 | |
*** armaan has quit IRC | 09:13 | |
*** armaan has joined #openstack-nova | 09:14 | |
*** hshiina is now known as hshiina|afk | 09:17 | |
*** theanalyst has joined #openstack-nova | 09:18 | |
*** namnh has quit IRC | 09:22 | |
*** Nisha_Agarwal has joined #openstack-nova | 09:23 | |
*** namnh has joined #openstack-nova | 09:23 | |
*** annp has quit IRC | 09:26 | |
*** hoangcx has quit IRC | 09:26 | |
*** hoangcx has joined #openstack-nova | 09:26 | |
*** annp has joined #openstack-nova | 09:27 | |
*** bkopilov has quit IRC | 09:27 | |
*** liuzz has quit IRC | 09:29 | |
*** Dinesh_Bhor has quit IRC | 09:37 | |
*** liuzz has joined #openstack-nova | 09:38 | |
*** liuzz_ has joined #openstack-nova | 09:40 | |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Add a debug log entry before / after invoking migrate() https://review.openstack.org/563550 | 09:41 |
*** liuzz has quit IRC | 09:43 | |
*** cdent has joined #openstack-nova | 09:48 | |
*** Tom-Tom_ has quit IRC | 09:49 | |
*** Tom-Tom has joined #openstack-nova | 09:50 | |
kashyap | stephenfin: ^ If you're looking for the first +2 of the day :-) | 09:50 |
*** Tom-Tom has quit IRC | 09:52 | |
*** rmart04 has joined #openstack-nova | 09:53 | |
*** markmc`` is now known as markmc | 09:55 | |
*** Tom-Tom has joined #openstack-nova | 09:56 | |
*** Nisha_Agarwal has quit IRC | 10:06 | |
*** nicolasbock has joined #openstack-nova | 10:09 | |
*** dougshelley66 has joined #openstack-nova | 10:09 | |
*** hoangcx has quit IRC | 10:09 | |
*** sdague has joined #openstack-nova | 10:14 | |
*** dtantsur|pto is now known as dtantsur | 10:16 | |
*** Nisha_Agarwal has joined #openstack-nova | 10:16 | |
*** vivsoni has joined #openstack-nova | 10:16 | |
*** namnh has quit IRC | 10:17 | |
*** armaan has quit IRC | 10:17 | |
*** armaan has joined #openstack-nova | 10:18 | |
*** armaan has quit IRC | 10:22 | |
*** diga has joined #openstack-nova | 10:22 | |
*** pcaruana has quit IRC | 10:26 | |
openstackgerrit | sahid proposed openstack/nova master: libvirt: fix wrong driver name for vhostuser interface https://review.openstack.org/561562 | 10:27 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | 10:27 |
*** yamamoto has quit IRC | 10:31 | |
*** pcaruana has joined #openstack-nova | 10:33 | |
*** armaan has joined #openstack-nova | 10:35 | |
*** nicolasbock has quit IRC | 10:39 | |
*** abhishekk has quit IRC | 10:41 | |
*** tssurya has joined #openstack-nova | 10:46 | |
*** dougshelley66 has quit IRC | 10:46 | |
*** AlexeyAbashkin has quit IRC | 10:47 | |
*** nicolasbock has joined #openstack-nova | 10:50 | |
*** alexchad_ has quit IRC | 10:50 | |
*** AlexeyAbashkin has joined #openstack-nova | 10:51 | |
*** sapd1 has quit IRC | 10:54 | |
*** mdbooth has joined #openstack-nova | 11:00 | |
*** hoonetorg has quit IRC | 11:03 | |
*** lucasagomes is now known as lucas-hungry | 11:06 | |
*** yikun has quit IRC | 11:10 | |
*** yikun_ has joined #openstack-nova | 11:10 | |
*** stakeda has quit IRC | 11:10 | |
*** yikun_ has quit IRC | 11:13 | |
*** dougshelley66 has joined #openstack-nova | 11:13 | |
*** yikun_ has joined #openstack-nova | 11:13 | |
*** hoonetorg has joined #openstack-nova | 11:16 | |
*** cdent has quit IRC | 11:19 | |
*** yikun_ has quit IRC | 11:20 | |
*** hoonetorg has quit IRC | 11:21 | |
*** yamamoto has joined #openstack-nova | 11:31 | |
*** sdeath1 has quit IRC | 11:32 | |
*** jpena is now known as jpena|lunch | 11:33 | |
*** tiendc has quit IRC | 11:33 | |
*** amoralej is now known as amoralej|lunch | 11:36 | |
*** yamamoto has quit IRC | 11:37 | |
*** hoonetorg has joined #openstack-nova | 11:39 | |
*** cdent has joined #openstack-nova | 11:41 | |
*** diga has quit IRC | 11:42 | |
*** edmondsw has joined #openstack-nova | 11:47 | |
*** Tom-Tom has quit IRC | 11:48 | |
*** Tom-Tom has joined #openstack-nova | 11:48 | |
*** Tom-Tom_ has joined #openstack-nova | 11:51 | |
*** Nisha_Agarwal has quit IRC | 11:52 | |
*** sean-k-mooney has joined #openstack-nova | 11:52 | |
*** Tom-Tom has quit IRC | 11:53 | |
*** pcaruana has quit IRC | 11:54 | |
*** Tom-Tom_ has quit IRC | 11:56 | |
*** yamamoto has joined #openstack-nova | 12:05 | |
*** yamamoto has quit IRC | 12:06 | |
*** lucas-hungry is now known as lucasagomes | 12:06 | |
*** pcaruana has joined #openstack-nova | 12:10 | |
*** Tom-Tom has joined #openstack-nova | 12:12 | |
*** liverpooler has joined #openstack-nova | 12:16 | |
*** Tom-Tom has quit IRC | 12:17 | |
*** pchavva has joined #openstack-nova | 12:19 | |
*** QianYu has joined #openstack-nova | 12:20 | |
*** Alex_ has quit IRC | 12:20 | |
*** ktibi has joined #openstack-nova | 12:23 | |
*** QianYu has quit IRC | 12:24 | |
*** amoralej|lunch is now known as amoralej | 12:25 | |
*** jpena|lunch is now known as jpena | 12:27 | |
*** zhaochao has quit IRC | 12:29 | |
*** zhaochao has joined #openstack-nova | 12:30 | |
*** trozet has quit IRC | 12:32 | |
*** edmondsw has quit IRC | 12:37 | |
*** yamamoto has joined #openstack-nova | 12:37 | |
*** jchhatbar is now known as janki | 12:37 | |
*** yamamoto has quit IRC | 12:42 | |
*** trozet has joined #openstack-nova | 12:44 | |
*** dave-mccowan has joined #openstack-nova | 12:49 | |
*** yamamoto has joined #openstack-nova | 12:51 | |
* efried will be in and out all week - conference | 12:54 | |
* efried waves | 12:54 | |
*** efried has quit IRC | 12:54 | |
*** takashin has joined #openstack-nova | 12:54 | |
*** MultipleCrashes has joined #openstack-nova | 12:55 | |
*** yamamoto has quit IRC | 12:55 | |
sahid | sean-k-mooney: did you get chance to see my last message? | 12:56 |
MultipleCrashes | http://openstack-3rd-party-virtuozzo-ci-logs.virtuozzo.com/18/563418/1/check/check-dsvm-tempest-vz7-exe-minimal/70e8d0a/console.html.gz | 12:56 |
MultipleCrashes | this task is failing since disk space is full | 12:56 |
MultipleCrashes | Plz check | 12:56 |
*** cdent has quit IRC | 12:56 | |
*** MultipleCrashes has quit IRC | 12:58 | |
*** lyan has joined #openstack-nova | 13:00 | |
*** lyan is now known as Guest77979 | 13:00 | |
*** hkominos has joined #openstack-nova | 13:00 | |
hkominos | Hi all. Is this the right channel to ask a question regarding libvirt/driver.py ? | 13:01 |
bauzas | hkominos: yup | 13:02 |
*** READ10 has joined #openstack-nova | 13:03 | |
*** vladikr has joined #openstack-nova | 13:04 | |
*** Nisha_Agarwal has joined #openstack-nova | 13:07 | |
*** eharney has joined #openstack-nova | 13:08 | |
*** sar has quit IRC | 13:10 | |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642 | 13:11 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643 | 13:11 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | 13:11 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646 | 13:11 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647 | 13:11 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819 | 13:11 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674 | 13:11 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675 | 13:11 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043 | 13:11 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326 | 13:11 |
*** Nisha_Agarwal has quit IRC | 13:13 | |
sean-k-mooney | sahid: hi no sorry | 13:13 |
sean-k-mooney | sahid: what was it? | 13:14 |
sahid | sean-k-mooney: it's about tempest-dsvm-ovsdpdk-nfv-networking-xenial | 13:15 |
sean-k-mooney | sahid: oh the fact that its offline? | 13:15 |
sean-k-mooney | or at least it was last week | 13:15 |
*** esberglu has joined #openstack-nova | 13:15 | |
sahid | no no it's about to update it to configure virtio-net queue sizez | 13:15 |
sean-k-mooney | sahid: ah you want to update it to test with queue configured | 13:16 |
sahid | yep exactly | 13:16 |
*** moshele has quit IRC | 13:16 | |
*** psachin has quit IRC | 13:17 | |
sean-k-mooney | sahid: im nologer the product owner for that ci but i can reach out internally and ask if they can enable it | 13:17 |
sean-k-mooney | infact we should proably also enable multiqueue also | 13:17 |
sahid | sean-k-mooney: that would be great, i did some internal tests, but dansmith really would like to see some ci running | 13:17 |
sahid | https://review.openstack.org/#/c/484997/ | 13:18 |
sahid | sean-k-mooney: that would be great yes | 13:18 |
hkominos | hi: in order to spawn an openstackVM nova will define an xml file describing the VM.In the XML I think nova also defines a number of pci slot for taht VM. Is that default number of PciE slots taken from some config file? | 13:19 |
hkominos | Just to clarify, I want nova to add another pci-slot to the Vms that it creates | 13:20 |
*** Nisha_Agarwal has joined #openstack-nova | 13:20 | |
*** ratailor has quit IRC | 13:21 | |
sean-k-mooney | sahid: so out of interest what was the issue that was found just before merging? | 13:21 |
*** cdent has joined #openstack-nova | 13:21 | |
*** ttsiouts_ has joined #openstack-nova | 13:24 | |
sahid | libvirt rejects driver name=vhost-user | 13:24 |
sahid | only acceptable values are qemu and vhost | 13:24 |
*** jackie-truong has joined #openstack-nova | 13:28 | |
kashyap | Yeah, it is also documented so in the formatdomain.html | 13:29 |
*** dougshelley66 has quit IRC | 13:29 | |
kashyap | (Probably the upstream doc of that sub section needs a bit more tweaking.) | 13:30 |
*** dougshelley66 has joined #openstack-nova | 13:30 | |
openstackgerrit | Margarita Shakhova proposed openstack/nova master: Do not skip migrations in _destroy_evacuated_instances() https://review.openstack.org/563623 | 13:30 |
*** Nisha_Agarwal has quit IRC | 13:31 | |
dansmith | sean-k-mooney: does that ci not support depends-on? | 13:32 |
hkominos | bauzas: in order to spawn an openstackVM nova will define an xml file describing the VM.In the XML I think nova also defines a number of pci slot for taht VM. Is that default number of PciE slots taken from some config file? | 13:34 |
*** dougshelley66 has quit IRC | 13:35 | |
bauzas | hkominos: the libvirt driver gets the number of PCI devices by calling libvirtd and then creating a config | 13:35 |
sean-k-mooney | dansmith: that ci does not it is still using zuul 2.5 with jenkins | 13:35 |
bauzas | hkominos: then the config is used by the driver.py module to create a XML | 13:35 |
bauzas | when I say a config, I mean a config object | 13:36 |
sean-k-mooney | dansmith: well the old depends on might work | 13:36 |
bauzas | hkominos: see virt.libvirt.config | 13:36 |
sean-k-mooney | dansmith: we had planned to move it to zuul v3 at some point but not sure if the new team that is managing that intend to do that | 13:37 |
*** r-daneel has joined #openstack-nova | 13:37 | |
dansmith | sean-k-mooney: and that only runs on neutron, so no way to propose some sort of change that would end up with some nova config change? | 13:37 |
*** felipemonteiro_ has joined #openstack-nova | 13:38 | |
sean-k-mooney | dansmith: the intel nfv ci runs on changes to nova also | 13:38 |
hkominos | bauzas: thx I will take a look.Should I by intuitio,n assume that it is possible to just ask libvirt to create another empty pci slot? | 13:38 |
dansmith | sean-k-mooney: oh does it? then we could just use a DNM change to default those queues to something right? | 13:39 |
sean-k-mooney | dansmith: currently it runs 3 jobs on nova tempest-dsvm-ovsdpdk-nfv-networking-xenial tempest-dsvm-multinode-ovsdpdk-nfv-networking-xenial and tempest-dsvm-intel-nfv-xenial | 13:39 |
sean-k-mooney | dansmith: yep you could | 13:39 |
dansmith | ah yeah I see | 13:39 |
dansmith | sahid: easy ^ | 13:40 |
*** QianYu has joined #openstack-nova | 13:42 | |
kashyap | What is Eric Fried's Friday nick? | 13:43 |
cdent | kashyap: usually fried_rice | 13:43 |
kashyap | cdent: Ah, thanks. I think he's AFK now. | 13:43 |
*** felipemonteiro__ has joined #openstack-nova | 13:44 | |
sahid | dansmith, sean-k-mooney are the repo of the template to i can provide a patch? | 13:44 |
sahid | where* | 13:44 |
cdent | kashyap: he's at a conference this week | 13:46 |
dansmith | sahid: just nova.. I'll do it, just a sec | 13:46 |
*** felipemonteiro_ has quit IRC | 13:47 | |
kashyap | cdent: Noted; thanks. | 13:47 |
openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Test virtio queue sizes https://review.openstack.org/563626 | 13:48 |
dansmith | sahid: ^ | 13:48 |
sahid | dansmith: oh... smart :) | 13:49 |
cdent | scheduler meeting #openstack-meeting-alt in ten minutes | 13:49 |
bauzas | hkominos: well, not sure I understant your question, you're asking if Nova should provide an empty pci slot anyway? | 13:49 |
*** burt has joined #openstack-nova | 13:50 | |
kashyap | As a second pair of eyes to MattR, can anyone else be able to take a look at this: https://review.openstack.org/#/c/558783/ | 13:50 |
*** r-daneel has quit IRC | 13:51 | |
*** yamamoto has joined #openstack-nova | 13:51 | |
*** rmart04 has quit IRC | 13:51 | |
*** rmart04 has joined #openstack-nova | 13:52 | |
hkominos | bauzas:yes | 13:53 |
hkominos | bauzas:I think it does provide one anyway but I need one more. | 13:54 |
*** QianYu has quit IRC | 13:54 | |
bauzas | hkominos: well, I don't know why, so you should ask for that by a blueprint | 13:54 |
*** QianYu has joined #openstack-nova | 13:54 | |
tssurya | dansmith: if I want to populate a newly added column (say queued_for_delete), based on existing data from another column (say instances.deleted), what is the best way of doing this ? -> migration tool (seems to be the usual way in the past) ? or is there a way to do this while adding the new column ? | 13:54 |
*** liuzz_ has quit IRC | 13:54 | |
*** awaugama_away has joined #openstack-nova | 13:54 | |
dansmith | tssurya: doing it while adding the column should not be done because it can lock the table and be a problem for pre-applying schema | 13:55 |
dansmith | tssurya: if you can, the best way is on loading/saving the object out of the db | 13:55 |
*** liuzz has joined #openstack-nova | 13:55 | |
dansmith | tssurya: so in _from_db_obj or something like that, set the field on the loaded object so it will be saved later, optionally saving it right then if you need it to be saved immediately | 13:55 |
tssurya | dansmith: oh okay, yep will go that way then | 13:55 |
dansmith | tssurya: example: https://github.com/openstack/nova/blob/master/nova/objects/service.py#L259-L264 | 13:56 |
*** dougshelley66 has joined #openstack-nova | 13:56 | |
dansmith | that was how we generated uuids on the service objects | 13:56 |
*** tbachman has quit IRC | 13:56 | |
tssurya | dansmith: thanks | 13:56 |
*** yamamoto has quit IRC | 13:57 | |
tssurya | I will use this approach | 13:57 |
*** rmart04 has quit IRC | 13:57 | |
*** cfriesen has joined #openstack-nova | 13:57 | |
*** rmart04 has joined #openstack-nova | 13:57 | |
*** edmondsw has joined #openstack-nova | 13:58 | |
*** felipemonteiro_ has joined #openstack-nova | 14:00 | |
*** felipemonteiro__ has quit IRC | 14:00 | |
*** ralonsoh_ has joined #openstack-nova | 14:02 | |
*** efried has joined #openstack-nova | 14:04 | |
sean-k-mooney | sahid: the nfv cis are corntolled by internal repos in intel. you wont be able to change the test setup. you will however be able to hardcode the use of a queue lenght in nova in a DNM patch and get the same effect | 14:04 |
stephenfin | bauzas: All comments addressed here. Fancy hitting it again? https://review.openstack.org/#/c/379525/ | 14:05 |
*** ralonsoh has quit IRC | 14:06 | |
bauzas | stephenfin: yup, I can sneak peek on it | 14:06 |
sean-k-mooney | sahid: we might be able to change the setup via a patch to networking-ovs-dpdk to add a local.sh or late running section to the devstack pluging to alter the flavors used by the ci but i would need to look into that. | 14:06 |
*** hongbin_ has joined #openstack-nova | 14:07 | |
*** mriedem has joined #openstack-nova | 14:08 | |
sahid | sean-k-mooney: ok thanks | 14:08 |
*** mlavalle has joined #openstack-nova | 14:09 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add policy rule to block image-backed servers with 0 root disk flavor https://review.openstack.org/561284 | 14:09 |
kashyap | Can anyone remind me what is the way to recheck this job: 'legacy-tempest-dsvm-multinode-live-migration' | 14:14 |
*** cfriesen has quit IRC | 14:14 | |
kashyap | Seems like I'm hitting a problem that is related to the CI env: http://logs.openstack.org/12/562312/4/check/legacy-tempest-dsvm-multinode-live-migration/88f1334/job-output.txt.gz | 14:17 |
* kashyap digs further | 14:17 | |
*** hkominos has quit IRC | 14:18 | |
*** cdent has quit IRC | 14:18 | |
*** tbachman has joined #openstack-nova | 14:20 | |
*** felipemonteiro__ has joined #openstack-nova | 14:20 | |
*** liuzz has quit IRC | 14:23 | |
*** felipemonteiro_ has quit IRC | 14:24 | |
*** liuzz has joined #openstack-nova | 14:24 | |
mriedem | kashyap: you can't recheck specific jobs | 14:26 |
mriedem | you just leave a 'recheck' comment | 14:26 |
kashyap | mriedem: Yeah, that I'm aware. Just thought I'd check if I can avoid a "full blown recheck" | 14:26 |
kashyap | (To be mindful to not cause needless CI churn.) | 14:27 |
mriedem | mnaser: i got https://review.openstack.org/#/c/561284/ updated to use a policy rule instead of a config option, this is the thing to disable booting from a 0 root disk flavor if you're not doing bfv | 14:27 |
openstackgerrit | Merged openstack/nova master: libvirt: Add a debug log entry before / after invoking migrate() https://review.openstack.org/563550 | 14:29 |
*** hjensas is now known as hjensas|afk | 14:29 | |
openstackgerrit | Merged openstack/nova master: Mention that users need noVNC >= 0.6 https://review.openstack.org/562847 | 14:29 |
*** links has quit IRC | 14:30 | |
kashyap | mriedem: When you have time this week: Zuul blessed this, and I've addressed your remarks: https://review.openstack.org/#/c/558783/ (libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Rocky") | 14:30 |
*** lpetrut has quit IRC | 14:31 | |
*** jroll has quit IRC | 14:31 | |
*** jroll has joined #openstack-nova | 14:32 | |
*** shoffmann_ has joined #openstack-nova | 14:33 | |
mriedem | was just looking at it | 14:33 |
*** namnh has joined #openstack-nova | 14:34 | |
*** awaugama_away is now known as awaugama | 14:35 | |
*** bkopilov has joined #openstack-nova | 14:38 | |
*** hemna_ has joined #openstack-nova | 14:39 | |
kashyap | Thanks. | 14:39 |
kashyap | stephenfin: efried: Thanks for the quick review (commit: 880f628)! | 14:40 |
efried | kashyap: yahyoubetcha. Easy one. | 14:40 |
kashyap | efried: More context on that is - before that commit, it was incredibly difficult to tell if that migrate() method has run or not | 14:41 |
efried | I get it. | 14:41 |
*** QianYu has quit IRC | 14:41 | |
efried | kashyap: Mainly, I get credit for a libvirt review I can point to so people don't accuse me of being PowerVM/IBM-centric. | 14:42 |
kashyap | Although there's a comment snuck in to Eventlet schedule a new thread, but that's too subtle | 14:42 |
kashyap | efried: Hehe | 14:42 |
*** cdent has joined #openstack-nova | 14:42 | |
efried | kashyap: I'm counting on major brownie points from this series: https://review.openstack.org/560444 | 14:43 |
kashyap | efried: Don't worry, I'll send more your way; mechanical clean up of constants :P | 14:43 |
mnaser | mriedem: nice patch, is there a way we can track things like "things to do for next release" ? | 14:43 |
kashyap | Until you might get sick of it :P (Once Matt merges the earlier MIN_ bump) | 14:43 |
mriedem | mnaser: we don't have anything formal for that besides the TODOs in the code | 14:43 |
mnaser | i see | 14:45 |
*** kholkina has quit IRC | 14:45 | |
kashyap | efried: Interesting. Have to still educate myself on that (ProviderTree stuff) | 14:45 |
efried | kashyap: Let me know if you want help there. We wrote a nice contributor doc - maybe since you claim to be fresh, you could scan it and let me know if it gives you what you need... | 14:46 |
kashyap | efried: I don't claim to be fresh :P | 14:46 |
kashyap | efried: But yeah, got a URL to that doc? | 14:47 |
efried | kashyap: working on that (I pretty much suck at searching the docs) | 14:47 |
kashyap | Aah, I see. The past tense word "wrote" made me think it's already in tree | 14:47 |
efried | kashyap: It is. | 14:48 |
efried | kashyap: Mahbad, it's a reference doc: https://docs.openstack.org/nova/latest/reference/update-provider-tree.html | 14:48 |
* kashyap will find it | 14:48 | |
efried | kashyap: So yeah, in recompense for that two-line review, you get to read my doc and give me feedback. | 14:49 |
efried | kashyap: No mistake, you got the raw end of that deal. | 14:49 |
kashyap | efried: Hehe, oh on the first look, I appreciate the structure & detail. This needs a bit of concentration to read cover-to-cover; queued it. | 14:50 |
*** Alexey_Abashkin has joined #openstack-nova | 14:50 | |
* kashyap is in a hurry as he needs to be AFK a bit early on Monday evenings | 14:50 | |
efried | kashyap: thanks in advance | 14:50 |
*** felipemonteiro__ has quit IRC | 14:51 | |
*** felipemonteiro__ has joined #openstack-nova | 14:51 | |
kashyap | efried: A quick question -- do you also do "hypervisor stuff" with PowerVM / IBM (z)? | 14:51 |
kashyap | (This bit: https://www-03.ibm.com/systems/power/software/linux/powerkvm/) | 14:52 |
efried | kashyap: I know nothing from z. | 14:52 |
efried | kashyap: And PowerKVM != PowerVM. I know nothing about the former. | 14:52 |
*** gcb has joined #openstack-nova | 14:53 | |
*** yamamoto has joined #openstack-nova | 14:53 | |
efried | kashyap: And depending what you mean by "hypervisor" - I get kinda close. I at least have first-order contact with people who do the nuts and bolts. | 14:53 |
*** r-daneel has joined #openstack-nova | 14:53 | |
kashyap | Oops, sorry for the mix-up | 14:54 |
*** artom has joined #openstack-nova | 14:54 | |
*** zhaochao has quit IRC | 14:54 | |
kashyap | (Nod) | 14:54 |
efried | Yeah, I'm sorry too. | 14:54 |
efried | It didn't need to be that confusing. | 14:54 |
efried | But you know branding/marketing folk. | 14:54 |
kashyap | Hehe, totally | 14:54 |
kashyap | efried: The KVM author still regrets the name of it, I hear... | 14:54 |
*** AlexeyAbashkin has quit IRC | 14:54 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 14:54 | |
kashyap | https://en.wikipedia.org/wiki/KVM_switch vs. https://en.wikipedia.org/wiki/Kernel-based_Virtual_Machine | 14:55 |
efried | gibi, mriedem: FYI: https://review.openstack.org/#/c/561724/ | 14:57 |
*** ttsiouts_ has quit IRC | 14:57 | |
*** yamamoto has quit IRC | 14:58 | |
*** tbachman has quit IRC | 14:58 | |
kashyap | mriedem: Hmm, you gave +2, but you noted a whitespace nit & the needless comment. Respinning to remove the needless cruft. | 14:59 |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Rocky" https://review.openstack.org/558783 | 15:02 |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: Drop MIN_LIBVIRT_SET_ADMIN_PASSWD https://review.openstack.org/562312 | 15:02 |
efried | gibi: That last low-priority bug with the abandoned patches - I didn't look deeply, but does that qualify as low-hanging fruit? | 15:05 |
*** takashin has left #openstack-nova | 15:05 | |
*** QianYu has joined #openstack-nova | 15:05 | |
*** rmart04 has quit IRC | 15:06 | |
kashyap | Remind me again, if I update the commit message, will I "lose" the +2? | 15:07 |
* kashyap has an OCD to remove needless commas | 15:08 | |
*** rajinir has joined #openstack-nova | 15:08 | |
kashyap | (Only one spurious comma this time, though) | 15:08 |
stephenfin | kashyap: You will, yup | 15:09 |
*** sridharg has quit IRC | 15:09 | |
kashyap | Okay, I'll leave it as is, not worth to bother reviewers about it | 15:09 |
stephenfin | If it's not a bigger issue, you're usually better of just leaving it unless someone else comments on that or something else | 15:09 |
*** felipemonteiro_ has joined #openstack-nova | 15:09 | |
stephenfin | (If it was a bigger issue, you wouldn't have got the +2) | 15:09 |
kashyap | Yeah, it's not an issue all, only OCD. | 15:09 |
kashyap | stephenfin: Okido; sorry for the bait, but since you like clean-up patches, you might be the second +2er: https://review.openstack.org/#/c/558783/ | 15:10 |
sahid | dansmith: the intel ci just finished, that looks to be OK | 15:11 |
*** QianYu has quit IRC | 15:12 | |
*** felipemonteiro__ has quit IRC | 15:13 | |
stephenfin | kashyap: Ooh, for sure | 15:16 |
bauzas | stephenfin: jaypipes: dansmith: efried: FWIW, given all the convos we had about https://review.openstack.org/#/c/557065/2/specs/rocky/approved/vgpu-rocky.rst, I'm about to write a patch that'd show some dynamic group usage | 15:16 |
kashyap | Gracias | 15:16 |
stephenfin | bauzas: I've it done | 15:16 |
bauzas | stephenfin: where ? I could do the same | 15:16 |
stephenfin | :) | 15:16 |
stephenfin | sec | 15:16 |
stephenfin | bauzas: https://github.com/stephenfin/nova/commit/58b53e888feb146123087f2e3f54f677f1fdccdc | 15:16 |
bauzas | or make it generic | 15:16 |
stephenfin | (I haven't pushed it to Gerrit yet because it's not end-to-end tested and I don't want to swamp the CI as I iterate) | 15:17 |
dansmith | sahid: yep, cool | 15:17 |
*** janki has quit IRC | 15:17 | |
bauzas | stephenfin: I'm not sure we should dynamicall add the groups in nova/config | 15:18 |
bauzas | stephenfin: rather closer to the caller | 15:18 |
bauzas | stephenfin: my thoughts was to use devices.enabled_vgpu_types as the ListOpt value for knowing which opt groups to check | 15:19 |
stephenfin | bauzas: I wanted it to be done at startup so I could quickly fail. However, that's really good feedback for when it's on Gerrit :) | 15:19 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove vestigial system_metadata param from info_from_instance() https://review.openstack.org/561724 | 15:19 |
bauzas | stephenfin: or use MultiStrOpt for that | 15:19 |
stephenfin | bauzas: Aye, I used 'CONF.neutron.physnets' for that | 15:19 |
*** pcaruana has quit IRC | 15:19 | |
stephenfin | https://github.com/stephenfin/nova/commit/58b53e888feb146123087f2e3f54f677f1fdccdc#diff-bf484fde349b127980f9341543ce3c9aR38 | 15:20 |
stephenfin | so '[neutron] physnets = foo, bar' | 15:20 |
*** felipemonteiro_ has quit IRC | 15:20 | |
bauzas | right | 15:20 |
bauzas | stephenfin: see the context in https://review.openstack.org/#/c/557065/2/specs/rocky/approved/vgpu-rocky.rst@86 | 15:20 |
*** felipemonteiro_ has joined #openstack-nova | 15:20 | |
bauzas | stephenfin: for MultiStrOpt | 15:20 |
stephenfin | I guess I could use prefixed attribute names but someone (jaypipes or dansmith, I think) thought that was eewy | 15:21 |
* stephenfin clicks | 15:21 | |
dansmith | jaypipes: we've validated this with the nfv ci, if you want to have another look at it and the one below: https://review.openstack.org/#/c/484997/ | 15:21 |
jaypipes | dansmith: ack, will do. | 15:22 |
jaypipes | ah, this is the one I'd asked sahid to combine the first two patches. yeah... one minute. | 15:23 |
sahid | dansmith: it was a really nice idea your DNM patch, i'm going to reuse it a lot :) | 15:23 |
dansmith | jaypipes: yep | 15:24 |
dansmith | sahid: :) | 15:24 |
bauzas | stephenfin: you're better than me on oslo.config, do you know if we can have a MultiStrOpt where two lines would have the same key but different values ? | 15:25 |
* bauzas needs to look at MultiStrOpt code | 15:26 | |
stephenfin | bauzas: Got an example? | 15:26 |
bauzas | sec | 15:26 |
stephenfin | They have the same key by definition | 15:26 |
*** lajoskatona has quit IRC | 15:27 | |
jaypipes | dansmith: off to the test pits it goes. | 15:28 |
efried | gibi: Ready for you: https://review.openstack.org/#/c/561724/ self.NotificationEmail().AndRemove(bug_1764390) | 15:28 |
dansmith | jaypipes: the test pits ahead won't really test it, but it came back from other test pits that did recently, so.. thanks :) | 15:29 |
bauzas | stephenfin: actually, what I want is a MultiDictOpt | 15:29 |
* bauzas gets confused by oslo.config | 15:29 | |
jaypipes | bauzas: you do *not* want a MultiDictOpt. | 15:30 |
bauzas | I don't know | 15:30 |
jaypipes | bauzas: dansmith might kill you if you used that. | 15:30 |
bauzas | what I want is a pony | 15:30 |
jaypipes | heh | 15:30 |
jaypipes | bauzas: your dog is as big as a pony. | 15:30 |
stephenfin | kashyap: One question before I send that on it's way: how come you didn't use the "NEXT" libvirt/qemu values already specified by berrange? | 15:30 |
bauzas | I just want a fucking easy way to describe a transitive relationship between a thing and a list of other things | 15:30 |
bauzas | jaypipes: not as big as a pony, I saw taller yesterday when I was in a dog show | 15:31 |
jaypipes | bauzas: relational DBs are pretty good at storing relations :P | 15:31 |
bauzas | jaypipes: I know | 15:31 |
bauzas | jaypipes: I have a crazy idea to just add a new nova-manage helper if you want my own thoughhts | 15:32 |
*** sar has joined #openstack-nova | 15:32 | |
bauzas | just for something super specific like that | 15:32 |
stephenfin | bauzas: s/in a dog show/at a dog show/ ;) | 15:32 |
bauzas | because libvirtd doesn't help me with that | 15:32 |
*** dklyle has joined #openstack-nova | 15:32 | |
bauzas | so, things are, we know we're doomed with defining that relationship elsewhere but nova | 15:32 |
jaypipes | stephenfin: an important clarification :) | 15:32 |
jroll | stephenfin: "in" was correct, bauzas was a contestant :P | 15:32 |
bauzas | but I want to make it the less painful | 15:32 |
sean-k-mooney | bauzas: whay are the entities you are trying to define the relationship of? | 15:33 |
bauzas | jaypipes: stephenfin: jroll: if you care https://lh3.googleusercontent.com/fFHAwwIKjuW9uKAYkPrsTjsw_bOO6gQgA7xw-B0d2-gmC24J8Z1DUpSzKHwkIJesK_aTGeKGQ8ts1P67SZhmsxUiVEEABuFv7Hg4CvQ-ROw8TNcOgKOgw7HjUqL6ZMsLPYgLee8mDkyj0PpCTRUiD936luaHuThLkW__s1Ym9UAXmMPqXo7J1cBv1GkYjyunJfWbJTUBklcJHd6zwTz51M1DhIMRd0zYpr9xm8kCG3Qbn0NnzVK6jt7ppA00jqUbl-NfReOZyCirPZvrMLJyrtm9nza7AwwOuCm0NGYrVBMZlpnGKQ8VmmlG3r8wz9uPKvUjREitqqN4WgTjhDgRrIomD6Oe9bCMRnRX2 | 15:34 |
bauzas | x8kAufXHD_RF1-sfVGt6FEMv2MOY7Y2WH5d4MhvFQT3WF37ZLRJxAxXXmhloPjqF6qh374jvsopMPVaKykwW9zuTiZ87CNcQbfXeGze46RC_CAPmccFyYeA84_y7t0TIJOzStqFXA5LvLivtFfxJtGq06ZRD_P0ZSHcw_cdigRN84R-jDPXGRxnbuAkm5f16HfyLzxARcz1OPfVdUBFliYvHR4h8vZ9VI2ajmprFcxZp2mxIhuvD8FC6lzAwyRLayjnfn4ElfZSPVcm8zVJ9VBFnSwBL5xSq6ZaayKfdL1KEPAYgpQYIj0bdLk6w=w1689-h950-no | 15:34 |
sean-k-mooney | bauzas: you can use somthing like ovo in somecases also | 15:34 |
bauzas | sean-k-mooney: I just want to say "that GPU type is for those PCI devices" | 15:34 |
bauzas | period | 15:34 |
bauzas | but oslo.config doesn't give me a good way to express that except something dumb like we merged for pci whitelist, and which I dislike at most | 15:35 |
bauzas | so, I'm out of ideas | 15:35 |
sean-k-mooney | bauzas: ah ok ya so like the pci alais with out all the other symantics that is layered on that | 15:35 |
bauzas | except sealing in the nova codebase something I don't want | 15:35 |
bauzas | because that's just libvirtd isn't mature enough on that feature to provide CLIs for that | 15:36 |
bauzas | so a config option is just a 'mid-term' solution until, I hope, something comes up that allows to specify that outside of nova | 15:36 |
sean-k-mooney | bauzas: you can find the vgpu types in virsh capablitites or virsh nodedev-list though right? | 15:37 |
*** lpetrut has joined #openstack-nova | 15:37 | |
bauzas | sean-k-mooney: sure, but then I need to say which type is for which pGPU | 15:37 |
sean-k-mooney | bauzas: one of those should expose the mdev types that correstpond the the different vgpus | 15:37 |
bauzas | sean-k-mooney: context is https://review.openstack.org/#/c/557065/2/specs/rocky/approved/vgpu-rocky.rst | 15:38 |
bauzas | sean-k-mooney: sure, but each pGPU supports multiple types | 15:38 |
*** chyka has joined #openstack-nova | 15:38 | |
*** moshele has joined #openstack-nova | 15:38 | |
bauzas | Xen doesn't have that problem, because the xen CLI allows to provide such tool out of nova | 15:38 |
bauzas | I don't have that in my bucket, so I'm doomed | 15:39 |
sean-k-mooney | bauzas: yes but form a config type we need to seperate things. a whitelist to declare what pGPUs can be used and a mapping that allows you to name the types and use that in the flavor | 15:39 |
bauzas | sean-k-mooney: we already have the former | 15:39 |
bauzas | sean-k-mooney: what we miss is the latter | 15:39 |
*** fragatina has joined #openstack-nova | 15:39 | |
bauzas | again, we can technically have a StrOpt and do some magical JSON import | 15:40 |
*** moshele has quit IRC | 15:40 | |
sean-k-mooney | bauzas: the pci whitelist is the former right. and your looking for a clean way to lable the diferent types so we dont leak the host implemtaiton details via the api | 15:40 |
bauzas | just because oslo.config misses that | 15:40 |
bauzas | no | 15:40 |
bauzas | sean-k-mooney: we currently have enabled_vgpu_types that lists all the supported types accepted by that compute | 15:40 |
*** fragatin_ has joined #openstack-nova | 15:41 | |
bauzas | sean-k-mooney: what I miss is a way to say for each of those types, which pGPU(s) is/are associated with | 15:41 |
sean-k-mooney | bauzas: ah ok. | 15:41 |
bauzas | jaypipes: if we say here that a dict opt isn't the best match, it only allows us some kind of dynamic import, or some external file that'd be out of oslo.config | 15:42 |
bauzas | jaypipes: I'm not super happy with the second option that carries a lot of misunderstandings | 15:43 |
*** jpena is now known as jpena|brb | 15:43 | |
sean-k-mooney | bauzas: i belive stephenfin is looking at reusing how cindier does dynmaic config for the numa aware vswitch spec | 15:44 |
sean-k-mooney | bauzas: could we use the same approch here or is that overkill | 15:44 |
*** tbachman has joined #openstack-nova | 15:44 | |
bauzas | I think that's the only option I have tbh, if we still care about nova.conf | 15:44 |
*** abhishekk has joined #openstack-nova | 15:44 | |
*** fragatina has quit IRC | 15:45 | |
sean-k-mooney | bauzas: well if you have a config var that listed all the phyciscal gpus and then had a dynmic section for each that listed the allowed types i think that would be resonably clean | 15:46 |
* bauzas goes back to code | 15:47 | |
jaypipes | bauzas: I'm not sure if you're asking me a question. I think dansmith would like to see you use a strategy similar to stephenfin's code that adds a config section to nova.conf for each physical device? So, something like [gpu_<some identifier>] and then have CONF options inside that group? | 15:47 |
bauzas | ok, I can try then | 15:47 |
*** cfriesen has joined #openstack-nova | 15:48 | |
*** dklyle has quit IRC | 15:49 | |
dansmith | jaypipes: yeah | 15:49 |
openstackgerrit | Margarita Shakhova proposed openstack/nova master: Do not skip migrations in _destroy_evacuated_instances() https://review.openstack.org/563623 | 15:49 |
*** yamamoto has joined #openstack-nova | 15:54 | |
*** owalsh is now known as owalsh_afk | 15:58 | |
*** yamamoto has quit IRC | 16:00 | |
*** gcb has quit IRC | 16:03 | |
*** gyee has joined #openstack-nova | 16:03 | |
*** armaan has quit IRC | 16:03 | |
*** namnh has quit IRC | 16:04 | |
*** ccamacho has quit IRC | 16:04 | |
*** damien_r has quit IRC | 16:04 | |
*** armaan has joined #openstack-nova | 16:04 | |
openstackgerrit | sahid proposed openstack/nova master: add mtu to libvirt xml for ethernet and bridge types https://review.openstack.org/553072 | 16:05 |
sahid | sean-k-mooney: I updated you patch ^ based on matt comments and i also have updated a point with the mtu function to update libvirt configure xml I hope that was ok for you | 16:06 |
*** dpawlik has quit IRC | 16:06 | |
sean-k-mooney | sahid: ya no worries sorry i had ment to get back to that patch sooner | 16:08 |
*** armaan has quit IRC | 16:09 | |
*** armaan has joined #openstack-nova | 16:09 | |
sean-k-mooney | sahid: the real answer to matts question is when os-vif actully plug the interface it does set teh mtu but there are still a few code paths where libvirt plugs the interfaces and then nothing set the mtu which is what that patch fixes | 16:11 |
*** ccamacho has joined #openstack-nova | 16:11 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove explicit instance.info_cache.delete() https://review.openstack.org/563689 | 16:11 |
*** shoffmann_ has left #openstack-nova | 16:12 | |
sahid | sean-k-mooney: launchpad updated the bug report to assign myself, i'm not able to re-assign you, so when you have a moment, please do https://bugs.launchpad.net/nova/+bug/1747496 | 16:12 |
openstack | Launchpad bug 1747496 in OpenStack Compute (nova) "MTUs are not set for VIFs if using kernel ovs + hybrid plug = false" [Medium,In progress] | 16:12 |
*** pcaruana has joined #openstack-nova | 16:13 | |
*** munimehan has joined #openstack-nova | 16:13 | |
*** sahid has quit IRC | 16:14 | |
sean-k-mooney | sahid: done :) thanks for adressing the comment. to your backport point im also hopping we can backport this change. i would like to move this to os-vif also in a seperate patch but that would chage the xml we generate so that is not back portable hence the current approch. | 16:14 |
*** tianhui has quit IRC | 16:15 | |
*** tianhui has joined #openstack-nova | 16:15 | |
*** hjensas|afk is now known as hjensas | 16:18 | |
*** sambetts is now known as sambetts|afk | 16:20 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Add policy rule to block image-backed servers with 0 root disk flavor https://review.openstack.org/563692 | 16:21 |
*** jpena|brb is now known as jpena | 16:22 | |
openstackgerrit | Merged openstack/nova master: Cleanup RP and HM records while deleting a compute service. https://review.openstack.org/554920 | 16:27 |
*** tbachman has quit IRC | 16:29 | |
*** lucasagomes is now known as lucas-afk | 16:29 | |
*** edmondsw has quit IRC | 16:29 | |
*** Alexey_Abashkin has joined #openstack-nova | 16:31 | |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | 16:32 |
*** efried has quit IRC | 16:33 | |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | 16:33 |
*** AlexeyAbashkin has quit IRC | 16:34 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 16:34 | |
*** sdeath has joined #openstack-nova | 16:41 | |
*** dpawlik has joined #openstack-nova | 16:46 | |
*** sdeath has quit IRC | 16:47 | |
*** tbachman has joined #openstack-nova | 16:48 | |
*** jpena is now known as jpena|off | 16:50 | |
*** dpawlik has quit IRC | 16:50 | |
mriedem | melwitt: the next runway slot is open since https://review.openstack.org/#/c/553288/ is done. i'm not sure, however, if the zvm stuff is ready to go back into the runway (i haven't been following the ML thread), | 16:52 |
mriedem | if not, the next one is https://blueprints.launchpad.net/nova/+spec/list-show-all-server-migration-types and i have reservations about even supporting that now (i think i always did but got tired of pushing against it) | 16:52 |
mriedem | so i left some notes in the etherpad | 16:53 |
*** mgoddard has quit IRC | 16:53 | |
*** yamamoto has joined #openstack-nova | 16:56 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova stable/queens: Cleanup RP and HM records while deleting a compute service. https://review.openstack.org/563698 | 16:57 |
*** AlexeyAbashkin has quit IRC | 16:59 | |
jaypipes | lbragstad: some comments inline on your set scope_types for admin password policies patch. | 17:01 |
jaypipes | lbragstad: comments and questions that is... | 17:01 |
*** yamamoto has quit IRC | 17:01 | |
*** ralonsoh_ has quit IRC | 17:03 | |
*** jaosorior has quit IRC | 17:05 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add policy rule to block image-backed servers with 0 root disk flavor https://review.openstack.org/563700 | 17:05 |
*** sar has quit IRC | 17:05 | |
lbragstad | jaypipes: awesome - checking | 17:08 |
*** derekh has quit IRC | 17:08 | |
*** arvindn0_ has joined #openstack-nova | 17:09 | |
*** tbachman_ has joined #openstack-nova | 17:09 | |
*** mdbooth has quit IRC | 17:10 | |
*** tbachman has quit IRC | 17:11 | |
*** tbachman_ is now known as tbachman | 17:11 | |
openstackgerrit | Zack Cornelius proposed openstack/nova-specs master: Libvirt file backed memory https://review.openstack.org/563704 | 17:16 |
*** mdnadeem has quit IRC | 17:17 | |
*** mvk has quit IRC | 17:22 | |
openstackgerrit | Merged openstack/nova master: libvirt: fix wrong driver name for vhostuser interface https://review.openstack.org/561562 | 17:22 |
*** tesseract has quit IRC | 17:23 | |
*** tbachman_ has joined #openstack-nova | 17:30 | |
jaypipes | lbragstad: thx for the answers. so what does the 'all' in system={'all': True} refer to? | 17:31 |
*** mgoddard has joined #openstack-nova | 17:31 | |
*** tbachman has quit IRC | 17:32 | |
*** tbachman_ is now known as tbachman | 17:32 | |
*** dtantsur is now known as dtantsur|afk | 17:33 | |
*** edmondsw has joined #openstack-nova | 17:34 | |
*** chyka_ has joined #openstack-nova | 17:36 | |
*** edmondsw has quit IRC | 17:37 | |
*** edmondsw has joined #openstack-nova | 17:37 | |
artom | dansmith, https://review.openstack.org/#/c/563704/ you're some sort of ninja dude | 17:38 |
dansmith | artom: um, what? | 17:38 |
artom | 20 minutes since patch posted, 13 comments, that's close to 1 comment per minute :) | 17:39 |
*** efried has joined #openstack-nova | 17:39 | |
*** chyka has quit IRC | 17:39 | |
dansmith | artom: only 14 between post and comment.. don't sell me short | 17:39 |
artom | Gomen, senpai | 17:40 |
artom | (Sincere apologies to all Japanese speakers here) | 17:40 |
dansmith | heh | 17:41 |
*** chyka_ has quit IRC | 17:41 | |
* artom goes back to wondering wtf Ubuntu named his nic "rename3" and why it gets a new IP every boot | 17:42 | |
openstackgerrit | Julia Kreger proposed openstack/nova master: DNM: Test: Ironic: Lock the instance if plug vifs https://review.openstack.org/563714 | 17:42 |
* artom just corrected dansmith | 17:43 | |
* artom is obviously having second thoughts, immediately after doing it. | 17:44 | |
dansmith | artom: when you're right you're right | 17:44 |
artom | I shall speak of this moment to my grandchildren | 17:45 |
*** tssurya has quit IRC | 17:47 | |
melwitt | mriedem: ack | 17:47 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Add policy rule to block image-backed servers with 0 root disk flavor https://review.openstack.org/563719 | 17:49 |
*** jaosorior has joined #openstack-nova | 17:49 | |
*** yamahata has quit IRC | 17:50 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Use nova.db.api directly https://review.openstack.org/543262 | 17:52 |
*** tomtom001 has quit IRC | 17:54 | |
openstackgerrit | Jim Rollenhagen proposed openstack/nova master: WIP: ironic: add instance_uuid before any other spawn activity https://review.openstack.org/563722 | 17:54 |
*** tomtom001 has joined #openstack-nova | 17:55 | |
*** chyka has joined #openstack-nova | 17:55 | |
*** yamamoto has joined #openstack-nova | 17:58 | |
lbragstad | jaypipes: it's supposed to refer to the entire deployment system | 17:58 |
*** dpawlik has joined #openstack-nova | 17:58 | |
* lbragstad is terrible with naming things though | 17:58 | |
lbragstad | jaypipes: following the documentation here - http://specs.openstack.org/openstack/keystone-specs/specs/keystone/queens/system-scope.html#authenticating-for-a-system-scoped-token | 17:59 |
jaypipes | lbragstad: ok. so the string 'all' is special? | 17:59 |
lbragstad | yeah | 17:59 |
jaypipes | gotcha | 17:59 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: add note about xenapi aggregate upcall being resolved https://review.openstack.org/563724 | 17:59 |
lbragstad | our intention was to build it so that eventually we could do something like system: {$service: $id} | 18:00 |
*** chyka has quit IRC | 18:00 | |
lbragstad | but that would be relayed in the token | 18:00 |
*** r-daneel has quit IRC | 18:00 | |
*** r-daneel has joined #openstack-nova | 18:01 | |
*** yamamoto has quit IRC | 18:02 | |
*** dpawlik has quit IRC | 18:02 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: mark block_device_mapping_v2.boot_index as required https://review.openstack.org/563728 | 18:06 |
mriedem | alex_xu: see ^ i don't know if we should revert https://review.openstack.org/#/c/524208/ or not | 18:07 |
mriedem | i tend to think we should | 18:08 |
*** felipemonteiro__ has joined #openstack-nova | 18:08 | |
*** abhishekk has quit IRC | 18:11 | |
*** felipemonteiro_ has quit IRC | 18:12 | |
*** cfriesen has quit IRC | 18:30 | |
*** Zames has joined #openstack-nova | 18:31 | |
*** Zames has quit IRC | 18:33 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix dropped check for boot_index 0 in _validate_bdm https://review.openstack.org/563732 | 18:35 |
*** tbachman is now known as tbachmanAway | 18:39 | |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | 18:41 |
*** mgoddard has quit IRC | 18:43 | |
*** harlowja has joined #openstack-nova | 18:44 | |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | 18:48 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: docs: link to volume multi-attach demo recording https://review.openstack.org/563734 | 18:49 |
openstackgerrit | Lance Bragstad proposed openstack/nova master: Set scope_types for admin password policies https://review.openstack.org/553613 | 18:51 |
mriedem | jackie-truong: can you post a spec amendment per L68 https://etherpad.openstack.org/p/nova-runways-rocky ? | 18:53 |
*** sar has joined #openstack-nova | 18:53 | |
jackie-truong | mriedem: Yep, Peter Hamilton from our team is working on it | 18:54 |
mriedem | ok, thanks | 18:55 |
jackie-truong | In the mean time, Brianna has started a WIP patch for notifications: https://review.openstack.org/#/c/563269/ | 18:55 |
*** jaosorior has quit IRC | 18:55 | |
*** cfriesen has joined #openstack-nova | 18:55 | |
*** yamamoto has joined #openstack-nova | 18:59 | |
*** fragatin_ has quit IRC | 19:02 | |
*** yamamoto has quit IRC | 19:03 | |
*** amoralej is now known as amoralej|off | 19:06 | |
*** felipemonteiro__ has quit IRC | 19:06 | |
*** felipemonteiro__ has joined #openstack-nova | 19:07 | |
*** harlowja has quit IRC | 19:07 | |
*** rajinir has quit IRC | 19:08 | |
*** chyka has joined #openstack-nova | 19:10 | |
*** mgoddard has joined #openstack-nova | 19:10 | |
*** tbachmanAway is now known as tbachman | 19:10 | |
mriedem | arvindn05: i kind of think we should remove https://blueprints.launchpad.net/nova/+spec/glance-image-traits from the runways queue until there is agreement on the direction in https://review.openstack.org/#/c/560718/ | 19:11 |
*** dklyle has joined #openstack-nova | 19:11 | |
mriedem | melwitt: do you want me to send something to the ML re https://blueprints.launchpad.net/nova/+spec/list-show-all-server-migration-types and if we should still do it? | 19:12 |
mriedem | of the 5 things next in line, i think https://review.openstack.org/#/q/topic:bp/add-host-to-instance-action-events+status:open is the only one that's really ready to go | 19:13 |
*** imacdonn has quit IRC | 19:13 | |
*** imacdonn has joined #openstack-nova | 19:14 | |
*** mgoddard has quit IRC | 19:16 | |
mriedem | esberglu: are you updating https://review.openstack.org/#/c/526094/ / | 19:18 |
mriedem | ? | 19:18 |
esberglu | mriedem: Yep working on it right now | 19:19 |
arvindn0_ | mriedem: does the runway queue allow for closure on the specs? | 19:25 |
arvindn0_ | mriedem: i was thinking we can discuss the spec in the runway if thats the case. Also, we are trying to make progress on other patches which hopefully can also be reviewed in the runway | 19:26 |
*** Nisha_Agarwal has joined #openstack-nova | 19:27 | |
mriedem | arvindn0_: my understanding is that runways are for things that are ready to go | 19:28 |
mriedem | and i wouldn't consider this ready to go | 19:28 |
mriedem | btw, i'm trying to summarize alternatives for this in the ML to get some visibility | 19:28 |
openstackgerrit | Merged openstack/nova master: Add root and parent provider uuid to group by clause https://review.openstack.org/562379 | 19:30 |
*** Nisha_Agarwal has quit IRC | 19:32 | |
arvindn0_ | mriedem: thanks. should be fine to move to end of runway...i guess the runway maintainer is going to make this change | 19:32 |
*** yamahata has joined #openstack-nova | 19:36 | |
jaypipes | artom: if you have a minute, would you mind giving your thoughts on https://review.openstack.org/#/c/546713/1/os_traits/compute/net.py please? | 19:38 |
artom | jaypipes, looking | 19:39 |
jaypipes | artom: cheers | 19:40 |
artom | jaypipes, I'd go with gibi | 19:40 |
artom | Maybe a "real" example would make more sense: | 19:40 |
artom | 1. Just interface attachment: nova interface-attach port-id=blah | 19:40 |
artom | 2. Tagged attachment: nova interface-attach port-id=blah,tag=foo | 19:41 |
*** lpetrut has quit IRC | 19:41 | |
*** jackie-truong has quit IRC | 19:41 | |
*** ktibi has quit IRC | 19:43 | |
openstackgerrit | Arvind Nadendla proposed openstack/nova-specs master: Handle rebuild of instance with new image https://review.openstack.org/560718 | 19:44 |
*** cdent has quit IRC | 19:45 | |
*** READ10 has quit IRC | 19:46 | |
arvindn0_ | jaypipes: can you add your thoughts on the above BP? if we can get direction from you whether modifyiny `GET /allocation_candidates` by adding an in_tree parameter AND make resources optional is something we can agree to, i can start thiking in that direction | 19:47 |
openstackgerrit | Merged openstack/nova master: Leave a hint when populate_schema fails https://review.openstack.org/562757 | 19:52 |
jaypipes | arvindn0_: I don't have any thoughts on the above BP. I don't know much about rebuild at all and trust everything mriedem says. | 19:54 |
artom | Oh hey are we reopening the rebuild with new image can of worms | 19:56 |
jaypipes | artom: sorry, coming back to you... so I don't understand why that is a particular capability of the virt driver... is there a reason adding metadata about devices is specific to a virt driver? | 19:56 |
arvindn0_ | jaypipes: are you OK with modifying allocation_candiadates API to not take resources as a required parameter? | 19:57 |
artom | jaypipes, because the driver is what knows about the actual hardware metadata, like PCI and MAC address for NICs | 19:57 |
jaypipes | arvindn0_: no, I am not. | 19:57 |
jaypipes | arvindn0_: if there is nothing to allocate, there's no reason to call GET /allocation_candidates. | 19:57 |
*** sambetts|afk has quit IRC | 19:58 | |
arvindn0_ | jaypipes: if there are strong opinions against it, then we need to drop that proposal... | 19:58 |
jaypipes | arvindn0_: ok, gimme a bit. | 19:59 |
jaypipes | arvindn0_: I see mriedem just posted to the ML. I will respond there. | 19:59 |
arvindn0_ | no problem...i know rebuild is a can of worms like artom mentioned :) | 19:59 |
*** yamamoto has joined #openstack-nova | 19:59 | |
*** sambetts_ has joined #openstack-nova | 20:00 | |
jaypipes | artom: but why is the capability to associated a tag of "foo" with a PCI address something that is virt-driver dependent? | 20:00 |
dansmith | jaypipes: because the virt driver is what knows the address of the thing | 20:01 |
dansmith | jaypipes: if you ask for two nics, one tagged as foo and one as bar, | 20:01 |
artom | jaypipes, because some virt drivers don't have the code for it :) | 20:01 |
dansmith | only the virt driver knows how and where those are attached | 20:01 |
artom | (Wasn't trying to be condescending, I swear!) | 20:02 |
jaypipes | *sigh* | 20:02 |
*** dpawlik has joined #openstack-nova | 20:04 | |
*** dpawlik has quit IRC | 20:05 | |
*** liverpooler has quit IRC | 20:05 | |
jaypipes | dansmith, artom: so you're saying that nova interface-attach port-id=blah,tag=foo only works if specific virt drivers (libvirt only I guess?) are in use for the instance? | 20:05 |
*** yamamoto has quit IRC | 20:05 | |
*** dpawlik has joined #openstack-nova | 20:05 | |
artom | jaypipes, yes | 20:06 |
jaypipes | that's lovely. | 20:06 |
jaypipes | implementation-defined interfaces. | 20:06 |
dansmith | jaypipes: you know that some virt drivers don't even provide network interface attachments at all right? | 20:07 |
dansmith | *gasp* | 20:07 |
artom | I don't disagree, but hasn't that been the case for a while? It's sort of why we have the support matrix in the first place... | 20:07 |
jaypipes | artom: why doesn't something above the virt layer save the tag information for the interface? | 20:07 |
dansmith | jaypipes: it does happen above the virt driver level, but it doesn't have sufficient information to be useful | 20:07 |
artom | jaypipes, it does, and we could conceivably just put "device: {tag: foo}" in the metadata | 20:07 |
artom | But that's useless for the guest OS | 20:07 |
jaypipes | why is that useless for the guest OS | 20:08 |
jaypipes | ? | 20:08 |
dansmith | jaypipes: if we're attaching a pci nic at pci addres 1.2.3, or a virtio nic in slot 7, or a xenback device at /xenstore/foo/bar -- | 20:08 |
dansmith | those are the things the guest needs to know which tag applies to which | 20:08 |
artom | For the guest to do anything useful with that it would need to know something about the hardware "associated" with that tag | 20:08 |
dansmith | and compute manager can't know that | 20:08 |
artom | jaypipes, https://notartom.net/2017/06/20/virtual-device-role-tagging-better-explained/ /shameless plug | 20:08 |
*** owalsh_afk is now known as owalsh | 20:08 | |
*** tomtom001 has quit IRC | 20:09 | |
jaypipes | artom: how does EC2 handle this functionality? | 20:09 |
artom | ^^^ is about boot-time device tagging, but it explains the "why (is that useless to the guest OS)" question | 20:10 |
artom | jaypipes, it doens't | 20:10 |
dansmith | by not supporting more than one virt method? | 20:10 |
*** tomtom001 has joined #openstack-nova | 20:10 | |
artom | jaypipes, wait, ec2... the metadata API? | 20:10 |
dansmith | he means EC2 the service I believe | 20:10 |
jaypipes | dansmith: no, how can an EC2 API user inform their guest that a particular NIC is "for this specific network"? | 20:10 |
dansmith | jaypipes: that's totally different than applying a tag to a thing | 20:11 |
artom | dansmith, to be fair, that's kinda what tags were made for | 20:11 |
dansmith | jaypipes: you may impute some meaning from the tag, but that's different | 20:11 |
jaypipes | exactly what they were made for, actually. | 20:11 |
dansmith | jaypipes: so lets use the volume case | 20:12 |
dansmith | jaypipes: doesn't ec2 let you say "this volume will be vdb"? | 20:12 |
jaypipes | dansmith: you can specify one, but it might rename it behind the scenes. | 20:13 |
artom | Also, and this may be stupid, but what does EC2 have to do with any of this? | 20:13 |
dansmith | artom: zero, because they don't do many hypervisors | 20:14 |
*** dave-mccowan has quit IRC | 20:14 | |
*** armaan has quit IRC | 20:14 | |
dansmith | jaypipes: can we jump on a hangout here to figure out what the real concern is? because I feel like something must be confused if you're really worried about this | 20:14 |
dansmith | jaypipes: https://hangouts.google.com/call/Jcmcsrr1qa7qmYfuZ_myAAEE ? | 20:15 |
dansmith | artom: of course, if jaypipes'll join | 20:16 |
artom | Of course what? | 20:16 |
dansmith | artom: join ^ | 20:17 |
jaypipes | I'm on. | 20:17 |
artom | dansmith, uh, sure, except it keeps telling me I'm not allowed | 20:17 |
dansmith | artom: we're on, it's just public | 20:18 |
artom | Oh hey, the gmail account works | 20:18 |
melwitt | mriedem: yeah, I guess so. I was thinking similar because it's not clear to me if this has utility beyond abort cold migrate. the alternatives section is empty https://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/list-show-all-server-migration-types.html#alternatives | 20:23 |
mriedem | melwitt: exactly. ok i'll start a thread. | 20:23 |
melwitt | thanks | 20:24 |
*** sar has quit IRC | 20:26 | |
melwitt | arvindn0_, mriedem: which thing are y'all talking about "moving to the end of the runway" earlier? | 20:28 |
mriedem | glance-image-traits | 20:28 |
mriedem | L62 | 20:28 |
mriedem | end-o-queue | 20:29 |
arvindn0_ | Support Traits in Glance: https://blueprints.launchpad.net/nova/+spec/glance-image-traits | 20:29 |
melwitt | oh, I see | 20:29 |
melwitt | I got confused with the rebuild talk right after it | 20:29 |
*** felipemonteiro__ has quit IRC | 20:29 | |
arvindn0_ | we are discussing an ammendement to the originally approved spec... | 20:29 |
melwitt | gotcha. I see the note in the runways etherpad | 20:30 |
arvindn0_ | rebuild is related to the amendment :) | 20:30 |
melwitt | ah, okay | 20:30 |
*** dpawlik has quit IRC | 20:34 | |
*** dpawlik has joined #openstack-nova | 20:37 | |
melwitt | mriedem: agreed that the host/hostId to instance action events API is the only thing that looks ready for a runway, so I'm gonna move it there | 20:38 |
mriedem | cool | 20:38 |
mriedem | http://lists.openstack.org/pipermail/openstack-dev/2018-April/129736.html | 20:38 |
melwitt | thanks | 20:38 |
arvindn0_ | mriedem: wanted to quick check something on the scenario 2 i detailed in the comment | 20:41 |
arvindn0_ | a host with two SRIOV nic. One is normal SRIOV nic, another one with some kind of offload feature. | 20:43 |
melwitt | yikun: hi, I know you're not around right now but FYI your blueprint "Add host/hostId to instance action events API" has been added to a review runway https://etherpad.openstack.org/p/nova-runways-rocky | 20:43 |
arvindn0_ | Initial instance launch happens with SRIOV_VF:1 allocated, rebuild lauches with modified request with traits=HW_NIC_OFFLOAD_X, so basically we want the instance to be allocated the second nic | 20:44 |
*** rajinir has joined #openstack-nova | 20:45 | |
*** cfriesen has quit IRC | 20:45 | |
*** mvk has joined #openstack-nova | 20:45 | |
arvindn0_ | but the original allocation happens against nic1 and since in rebuild the original allocations are not changed, we have wrong allocations | 20:45 |
arvindn0_ | mriedem: is the above scenario an issue if we use GET /resource_providers/{rp_uuid}/traits approch? | 20:46 |
mriedem | arvindn0_: did you see efried's reply in the mailing list? | 20:49 |
*** dpawlik has quit IRC | 20:49 | |
*** hemna_ has quit IRC | 20:49 | |
*** dpawlik has joined #openstack-nova | 20:50 | |
mriedem | even with efried's suggestion, that scenario is likely a gap yes | 20:50 |
arvindn0_ | mriedem: trying to figure out the mailing list...i only get digests....do you have a link? | 20:50 |
mriedem | we need to have some way of knowing, will a rebuild with a new image result in new allocations and if so, fail the rebuild | 20:50 |
mriedem | you can change your subscription to not be digests :) | 20:51 |
mriedem | http://lists.openstack.org/pipermail/openstack-dev/2018-April/129734.html | 20:51 |
mriedem | arvindn0_: also, what you're describing above is this bug https://bugs.launchpad.net/nova/+bug/1763766 | 20:51 |
openstack | Launchpad bug 1763766 in OpenStack Compute (nova) "nova needs to disallow topology changes on image rebuild" [Medium,Triaged] | 20:51 |
efried | mriedem, arvindn0_: With what I suggested, you will still know that. Because you know the RPs you've already allocated from (and which you haven't). | 20:51 |
mriedem | efried: i'm not sure we do, we just have the root provider uuid in the scheduler filter | 20:52 |
mriedem | unless we are going to build a ProviderTree object or something | 20:52 |
efried | mriedem: Do you have access to a SchedulerReportClient from wherever you are? | 20:52 |
mriedem | sure | 20:53 |
efried | mriedem: If so, then yeah, you have that ability with a single call. | 20:53 |
*** hemna_ has joined #openstack-nova | 20:53 | |
efried | mriedem: https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L986 would be that call. | 20:54 |
efried | The 'ensure_root' part is slightly uncomfortable, but you know the root exists when you call this. | 20:54 |
efried | And we can factor that out of there if it's an issue. | 20:54 |
munimehan | All, when nova generates VM definition it don't generate the PCIs for guest addresses, is there a way to manipulate those to make it in sync for the interface order | 20:57 |
arvindn0_ | efried: good suggestion...will look into that. | 20:57 |
arvindn0_ | btw currently in the image_props_filter.py we dont have SchedulerReportClient...we can create and get access...but now we have 2 API calls from the filter.... | 20:58 |
*** idlemind has quit IRC | 20:59 | |
efried | arvindn0_: Do you have a resourcetracker? | 21:00 |
efried | arvindn0_: or a schedulerclient? | 21:00 |
arvindn0_ | nope...currently the filters make 0 api calls....so no clients of any sort | 21:01 |
*** tbachman has quit IRC | 21:01 | |
efried | arvindn0_: But we were going to have to call into placement one way or another (unless we went with option 1 and just blocked the whole thing). | 21:01 |
efried | arvindn0_: So yeah, you're not getting around that. And those should be pretty quick calls, one would hope. | 21:02 |
efried | arvindn0_: The set logic around required vs forbidden will be a little tricky. | 21:02 |
*** gjayavelu has joined #openstack-nova | 21:02 | |
arvindn0_ | yup...i was just pointing out that we might now have to make 2 calls, but like you said they should be quick since they are focussed on only the current compute node | 21:02 |
arvindn0_ | we dont support forbidden traits in images i think... | 21:03 |
efried | arvindn0_: If you did, I think you would probably have to do two separate GET /resource_providers calls (so three placement calls total) - one for required and one for forbidden. | 21:04 |
efried | arvindn0_: Sorry, I take that back - I would have to think that through when I'm not so distracted. | 21:05 |
arvindn0_ | efried: will save you time, we dont support forbidden traits in image...we only support required traits | 21:06 |
efried | k | 21:06 |
*** pchavva has quit IRC | 21:06 | |
efried | arvindn0_: I'm thinking through the logic, though, trying to figure out how you will actually know you're good, even with required. | 21:07 |
*** artom has quit IRC | 21:08 | |
efried | arvindn0_: Aha, actually, it's easier than I thought. | 21:08 |
efried | arvindn0_: You actually *only* need the call to get_provider_tree_and_ensure_root. You can walk that guy and collect the set of all traits from all the RPs you're already allocated from. Subtract that from the set of traits in your image. If there's anything left over, fail. Otherwise you're good. | 21:09 |
*** tomtom001 has quit IRC | 21:09 | |
*** fragatina has joined #openstack-nova | 21:09 | |
*** fragatina has quit IRC | 21:10 | |
arvindn0_ | how do we know our allocated RP's? | 21:10 |
arvindn0_ | is that part of the provider tree? | 21:10 |
*** fragatina has joined #openstack-nova | 21:11 | |
efried | Yes, you pass in your compute node UUID | 21:11 |
efried | You'll get back a ProviderTree. | 21:11 |
efried | wait, hold on. | 21:11 |
arvindn0_ | efried: not too familiar with the provider tree object...my understanding provider tree will return all traits etc | 21:12 |
arvindn0_ | but not which RP is allocated to us... | 21:12 |
efried | yes it will. For the whole tree, and associated sharing providers. And you need ... | 21:12 |
efried | exactly. | 21:12 |
efried | arvindn0_: Now, we're focused on an instance here, right? | 21:12 |
efried | arvindn0_: Not the whole host. | 21:12 |
arvindn0_ | yup | 21:13 |
arvindn0_ | this is rebuild of that instance...so we need to know what resource was allocated with what traits... | 21:13 |
efried | So what you'll *actually* want to get is the *allocations* for the instance. From that you can get the providers associated with your instance. Walk the ProviderTree on just those, peeling out the traits. Subtract that from the set of traits in your image. If there's anything left over, fail. Otherwise, you're good. | 21:14 |
arvindn0_ | we cant also do an aggregate of all the traits, because we want to ensure a 1:1 mapping | 21:14 |
efried | arvindn0_: That doesn't make sense. | 21:14 |
efried | arvindn0_: What do you mean by 1:1 mapping? | 21:14 |
efried | arvindn0_: This is one of the limitations with the entire "traits in images" deal. We don't have any way to correlate the traits to nested providers. | 21:15 |
efried | arvindn0_: So as I mentioned in my email, we just have to assume "anywhere in the tree". Which will actually work 99.9% of the time. | 21:15 |
efried | And I think the other .1% is an acceptable risk. | 21:15 |
*** tomtom001 has joined #openstack-nova | 21:16 | |
arvindn0_ | if a instance has multiple allocations to multiple sriov nics with different traits we need to make sure each allocation maps to each instance of sriov nic....i think...let me think about it more and provide a scenario | 21:16 |
efried | arvindn0_: You can't do that with image traits, at all, sorry. | 21:17 |
*** cdent has joined #openstack-nova | 21:17 | |
efried | arvindn0_: Because one thing we explicitly said in that spec review was that we can't correlate image traits with individual granular groups. | 21:17 |
efried | arvindn0_: The best you're going to be able to do is say, "do we have *any* nic already allocated that has this trait?" | 21:18 |
arvindn0_ | ahh..yea...we dont support granular groups/numbered traits | 21:18 |
efried | arvindn0_: Which is the exact same level of granularity you would have had when you initially created the instance, if you were using image traits at that time. | 21:18 |
efried | arvindn0_: And it's just a known limitation that if you used a flavor for creation, and an image for the rebuild, you've got reduced capabilities. | 21:18 |
arvindn0_ | efried: is this the API GET /allocations/{consumer_uuid} for getting allocations for an instance? | 21:20 |
efried | yes. Use the instance UUID as the consumer_uuid. | 21:20 |
efried | arvindn0_: Email followup coming... | 21:24 |
openstackgerrit | Julia Kreger proposed openstack/nova master: DNM: Test: Ironic: Lock the instance if plug vifs https://review.openstack.org/563714 | 21:25 |
arvindn0_ | efried: thanks | 21:25 |
arvindn0_ | efried: wait...another potential issue | 21:27 |
efried | arvindn0_: http://lists.openstack.org/pipermail/openstack-dev/2018-April/129740.html | 21:27 |
efried | oh. Too late :) | 21:27 |
arvindn0_ | efried: nvm...it should work | 21:27 |
efried | arvindn0_: Okay. Let me know if you see holes. Like I say, I'm distracted atm, so no guarantees. | 21:27 |
arvindn0_ | If there's anything left over on the image traits, we fail | 21:27 |
efried | exactly. | 21:28 |
openstackgerrit | Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437 | 21:28 |
openstackgerrit | Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439 | 21:28 |
openstackgerrit | Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438 | 21:28 |
arvindn0_ | efried: you think fast :) glad i could catch up in IRC | 21:30 |
openstackgerrit | Michael Still proposed openstack/nova master: Move IVS VIF plugging into the new utilities module. https://review.openstack.org/557552 | 21:30 |
openstackgerrit | Michael Still proposed openstack/nova master: Move create_tap_dev to the new utility module. https://review.openstack.org/557835 | 21:30 |
openstackgerrit | Michael Still proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module. https://review.openstack.org/558487 | 21:30 |
*** dougshelley66 has quit IRC | 21:31 | |
*** awaugama has quit IRC | 21:31 | |
*** dklyle has quit IRC | 21:31 | |
*** cdent has quit IRC | 21:32 | |
arvindn0_ | efried: btw, GET /allocations/{instance_uuid} does not provide traits? | 21:32 |
arvindn0_ | that would have made it much easier...any plans to add this? | 21:32 |
*** r-daneel_ has joined #openstack-nova | 21:35 | |
*** r-daneel has quit IRC | 21:36 | |
*** r-daneel_ is now known as r-daneel | 21:36 | |
efried | arvindn0_: I've proposed a similar "improvement" in the past (https://review.openstack.org/#/c/521875/), but the PTB deemed this unnecessary, expressing a preference for lots of smaller calls and reduced API breadth/complexity. | 21:36 |
efried | I should abandon that... | 21:36 |
arvindn0_ | cool...thanks the history helps so i dont spend time exploring that possibility | 21:37 |
efried | nod | 21:38 |
*** munimehan has quit IRC | 21:39 | |
efried | arvindn0_: I suspect your biggest overhead is going to be building up the client. Once you have the comm channel to placement open, the individual calls should be very fast. | 21:40 |
efried | arvindn0_: ...which is why I asked if you had a resource tracker or scheduler client or any other way to get at an already-built SchedulerReportClient. mriedem, any way to get at those things from a filter without having to build afresh? | 21:41 |
*** eharney has quit IRC | 21:42 | |
arvindn0_ | i think the implementation hasnt been figured out...we could implement in the scheduler, so we dont need to recreate some clients...if we do it in the filter, then we have the overhead yes | 21:42 |
efried | gotcha | 21:43 |
efried | Well, I leave it in your capable hands. Let me know if further questions. Good luck! | 21:43 |
*** slaweq has quit IRC | 21:45 | |
*** tbachman has joined #openstack-nova | 21:46 | |
arvindn0_ | thanks! | 21:46 |
*** slaweq has joined #openstack-nova | 21:46 | |
*** dpawlik has quit IRC | 21:47 | |
*** tbachman_ has joined #openstack-nova | 21:48 | |
*** tbachman has quit IRC | 21:50 | |
*** tbachman_ is now known as tbachman | 21:50 | |
*** slaweq has quit IRC | 21:52 | |
*** yamamoto has joined #openstack-nova | 22:02 | |
*** yamamoto has quit IRC | 22:07 | |
*** Guest77979 has quit IRC | 22:14 | |
*** gjayavelu has quit IRC | 22:17 | |
*** gjayavelu has joined #openstack-nova | 22:18 | |
*** dougshelley66 has joined #openstack-nova | 22:18 | |
*** lbragstad has quit IRC | 22:18 | |
*** esberglu has quit IRC | 22:18 | |
*** esberglu has joined #openstack-nova | 22:19 | |
*** esberglu has quit IRC | 22:21 | |
*** mlavalle has quit IRC | 22:22 | |
*** arvindn0_ has quit IRC | 22:23 | |
*** gjayavelu has quit IRC | 22:25 | |
*** dklyle has joined #openstack-nova | 22:26 | |
*** rcernin has joined #openstack-nova | 22:31 | |
*** sdague has quit IRC | 22:37 | |
*** dklyle has quit IRC | 22:39 | |
*** threestrands has joined #openstack-nova | 22:41 | |
*** threestrands has quit IRC | 22:41 | |
*** threestrands has joined #openstack-nova | 22:41 | |
*** felipemonteiro__ has joined #openstack-nova | 22:42 | |
*** efried has quit IRC | 22:42 | |
*** edmondsw has quit IRC | 22:43 | |
*** gjayavelu has joined #openstack-nova | 22:59 | |
*** yamamoto has joined #openstack-nova | 23:04 | |
*** felipemonteiro__ has quit IRC | 23:07 | |
*** yamamoto has quit IRC | 23:10 | |
openstackgerrit | Merged openstack/nova stable/pike: libvirt: disconnect volume from host during detach https://review.openstack.org/560690 | 23:19 |
*** r-daneel has quit IRC | 23:21 | |
*** yamamoto has joined #openstack-nova | 23:26 | |
*** hongbin_ has quit IRC | 23:27 | |
openstackgerrit | Zack Cornelius proposed openstack/nova-specs master: Libvirt file backed memory https://review.openstack.org/563704 | 23:46 |
*** cfriesen has joined #openstack-nova | 23:49 | |
*** Tom-Tom has joined #openstack-nova | 23:50 | |
*** artom has joined #openstack-nova | 23:53 | |
*** Tom-Tom has quit IRC | 23:54 | |
*** takashin has joined #openstack-nova | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!