Monday, 2018-04-23

*** germs has quit IRC00:00
*** dpawlik has joined #openstack-nova00:03
*** fragatin_ has quit IRC00:03
*** dpawlik has quit IRC00:08
*** fragatina has joined #openstack-nova00:11
*** jogo has quit IRC00:11
*** yamamoto has joined #openstack-nova00:20
*** fragatina has quit IRC00:21
*** yamamoto has quit IRC00:24
*** yamamoto has joined #openstack-nova00:41
*** hoangcx has joined #openstack-nova00:46
*** hshiina|afk is now known as hshiina00:46
*** tiendc has joined #openstack-nova00:52
*** dave-mccowan has joined #openstack-nova01:00
*** dave-mcc_ has joined #openstack-nova01:02
*** jichen has joined #openstack-nova01:05
*** dave-mccowan has quit IRC01:05
*** phuongnh has joined #openstack-nova01:08
*** jogo has joined #openstack-nova01:08
*** namnh has joined #openstack-nova01:17
*** tomtom001 has quit IRC01:19
*** tomtom001 has joined #openstack-nova01:20
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765801:23
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add snapshot function  https://review.openstack.org/53424001:24
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334001:24
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334401:24
*** salv-orlando has joined #openstack-nova01:28
*** salv-orl_ has quit IRC01:31
*** cfriesen has joined #openstack-nova01:32
*** zhaochao has joined #openstack-nova01:34
*** bkopilov has quit IRC01:37
*** cfriesen has quit IRC01:39
*** Tom-Tom has joined #openstack-nova01:40
*** sapd has joined #openstack-nova01:43
*** takashin has joined #openstack-nova01:44
*** tuanla____ has joined #openstack-nova01:53
*** dougshel_ has quit IRC01:59
*** masuberu has quit IRC02:06
*** masuberu has joined #openstack-nova02:06
*** annp has joined #openstack-nova02:16
*** jichen has quit IRC02:18
*** Guest84910 has quit IRC02:22
openstackgerritZhenyu Zheng proposed openstack/nova master: Send out notification when instance metadata updated  https://review.openstack.org/55814002:22
*** Guest84910 has joined #openstack-nova02:25
*** moshele has joined #openstack-nova02:27
*** phuongnh has quit IRC02:28
*** armaan has quit IRC02:29
*** armaan has joined #openstack-nova02:29
openstackgerritMerged openstack/python-novaclient master: Trivial: Update pypi url to new url  https://review.openstack.org/56306002:33
*** Dinesh_Bhor has joined #openstack-nova02:39
*** dklyle has joined #openstack-nova02:41
*** psachin has joined #openstack-nova02:44
*** bkopilov has joined #openstack-nova02:49
*** mdnadeem has joined #openstack-nova03:00
*** dklyle has quit IRC03:00
*** masuberu has quit IRC03:06
*** masuberu has joined #openstack-nova03:08
*** masber has joined #openstack-nova03:15
*** Tom-Tom has quit IRC03:17
*** Tom-Tom has joined #openstack-nova03:18
*** masuberu has quit IRC03:18
*** tomtom002 has joined #openstack-nova03:22
*** Tom-Tom has quit IRC03:23
*** Tom-Tom has joined #openstack-nova03:24
*** tomtom001 has quit IRC03:24
*** Tom-Tom_ has joined #openstack-nova03:26
*** Tom-Tom has quit IRC03:29
*** rcernin has quit IRC03:31
*** rcernin has joined #openstack-nova03:35
*** tomtom002 has quit IRC03:37
*** tomtom001 has joined #openstack-nova03:38
*** lpetrut has joined #openstack-nova03:39
*** psachin has quit IRC03:39
*** takashin has quit IRC03:43
*** Tom-Tom_ has quit IRC03:45
*** Guest84910 has quit IRC03:47
*** hongbin_ has quit IRC03:50
openstackgerritZhenyu Zheng proposed openstack/nova master: Send out notification when instance metadata updated  https://review.openstack.org/55814003:50
*** mdnadeem has quit IRC03:53
*** edmondsw has joined #openstack-nova03:55
*** dave-mcc_ has quit IRC04:00
*** Guest84910 has joined #openstack-nova04:01
*** Nisha_Agarwal has joined #openstack-nova04:02
*** phuongnh has joined #openstack-nova04:02
*** bhagyashris has joined #openstack-nova04:10
*** lpetrut has quit IRC04:11
*** lpetrut has joined #openstack-nova04:22
*** sridharg has joined #openstack-nova04:32
*** Tom-Tom has joined #openstack-nova04:35
*** Tom-Tom has quit IRC04:39
*** abhishekk has joined #openstack-nova04:46
*** lpetrut has quit IRC04:46
*** pooja_jadhav has joined #openstack-nova04:47
*** takashin has joined #openstack-nova04:52
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765804:53
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add snapshot function  https://review.openstack.org/53424004:53
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334004:53
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334404:53
*** armaan has quit IRC04:54
*** armaan has joined #openstack-nova04:54
*** moshele has quit IRC04:55
*** psachin has joined #openstack-nova05:00
*** edmondsw has quit IRC05:03
*** edmondsw has joined #openstack-nova05:03
*** Nisha_Agarwal has quit IRC05:08
*** edmondsw has quit IRC05:08
*** tuanla____ has quit IRC05:26
*** vivsoni__ has quit IRC05:27
*** vivsoni has joined #openstack-nova05:29
*** sridharg has quit IRC05:30
*** vivsoni has quit IRC05:31
*** ratailor has joined #openstack-nova05:35
*** Zames has joined #openstack-nova05:40
*** Zames has quit IRC05:43
*** sridharg has joined #openstack-nova05:43
*** stakeda has joined #openstack-nova05:43
*** Tom-Tom has joined #openstack-nova05:47
*** moshele has joined #openstack-nova05:50
*** Lingwu has quit IRC06:07
*** Lingwu has joined #openstack-nova06:08
*** blkart has quit IRC06:08
*** sar has joined #openstack-nova06:12
*** links has joined #openstack-nova06:13
*** lajoskatona has joined #openstack-nova06:14
*** blkart has joined #openstack-nova06:17
*** lpetrut has joined #openstack-nova06:21
*** kaisers1 has quit IRC06:31
*** kaisers has joined #openstack-nova06:33
*** ircuser-1 has joined #openstack-nova06:39
*** tiendc has quit IRC06:45
*** tiendc has joined #openstack-nova06:46
*** tomtom001 has quit IRC06:52
*** tomtom001 has joined #openstack-nova06:52
*** ccamacho has joined #openstack-nova06:56
*** sahid has joined #openstack-nova06:58
*** tesseract has joined #openstack-nova07:04
*** tomtom001 has quit IRC07:06
*** tomtom001 has joined #openstack-nova07:08
*** jpena|off is now known as jpena07:10
*** sapd has quit IRC07:13
*** xinliang has joined #openstack-nova07:14
*** xinliang has quit IRC07:14
*** xinliang has joined #openstack-nova07:14
*** janki has joined #openstack-nova07:15
*** slaweq has joined #openstack-nova07:16
openstackgerritMoshe Levi proposed openstack/nova stable/pike: Don't overwrite binding-profile  https://review.openstack.org/56350107:17
*** ralonsoh has joined #openstack-nova07:19
*** rcernin has quit IRC07:19
*** dpawlik has joined #openstack-nova07:19
*** jchhatbar has joined #openstack-nova07:21
*** dpawlik has quit IRC07:22
*** dpawlik has joined #openstack-nova07:22
*** janki has quit IRC07:23
*** pcaruana has joined #openstack-nova07:25
*** jogo has quit IRC07:27
*** AlexeyAbashkin has joined #openstack-nova07:36
*** damien_r has joined #openstack-nova07:37
*** zz9pzza has quit IRC07:39
*** Tom-Tom has quit IRC07:40
*** Tom-Tom has joined #openstack-nova07:41
*** markmc`` is now known as markmc07:41
openstackgerritZhenyu Zheng proposed openstack/nova master: WIP new migration threads control  https://review.openstack.org/56350507:42
*** damien_r has quit IRC07:42
*** amoralej|off is now known as amoralej07:42
*** damien_r has joined #openstack-nova07:42
*** markmc is now known as markmc``07:42
gibigood morning nova07:47
*** zz9pzza has joined #openstack-nova07:51
*** alexchadin has joined #openstack-nova07:56
*** kholkina has joined #openstack-nova07:58
*** Dinesh__Bhor has joined #openstack-nova08:03
*** Dinesh_Bhor has quit IRC08:05
*** mgoddard has joined #openstack-nova08:07
*** mdnadeem has joined #openstack-nova08:13
*** jaosorior has joined #openstack-nova08:15
*** lucas-afk is now known as lucasagomes08:15
*** threestrands has quit IRC08:17
*** finucannot is now known as stephenfin08:20
sahidsean-k-mooney[m]: how can i update tempest-dsvm-ovsdpdk-nfv-networking-xenial ?08:22
*** armaan has quit IRC08:24
*** jogo has joined #openstack-nova08:24
*** jogo has joined #openstack-nova08:24
*** armaan has joined #openstack-nova08:25
*** derekh has joined #openstack-nova08:27
*** takashin has left #openstack-nova08:30
*** jaosorior has quit IRC08:32
*** Tom-Tom has quit IRC08:37
*** phuongnh has quit IRC08:41
*** Tom-Tom has joined #openstack-nova08:44
*** sambetts_ is now known as sambetts08:46
*** Alexey_Abashkin has joined #openstack-nova08:48
*** Dinesh__Bhor has quit IRC08:49
*** Alexey_Abashkin has quit IRC08:50
*** Alexey_Abashkin has joined #openstack-nova08:50
*** AlexeyAbashkin has quit IRC08:51
*** sapd has joined #openstack-nova08:53
*** Dinesh_Bhor has joined #openstack-nova08:53
*** Tom-Tom_ has joined #openstack-nova08:53
*** AlexeyAbashkin has joined #openstack-nova08:54
*** Alexey_Abashkin has quit IRC08:54
*** sapd1 has joined #openstack-nova08:56
*** Tom-Tom has quit IRC08:56
*** Alexey_Abashkin has joined #openstack-nova09:02
*** AlexeyAbashkin has quit IRC09:05
*** Alexey_Abashkin is now known as AlexeyAbashkin09:05
*** jaosorior has joined #openstack-nova09:07
*** brault has joined #openstack-nova09:10
*** alexchad_ has joined #openstack-nova09:10
*** alexchadin has quit IRC09:12
*** armaan has quit IRC09:13
*** armaan has joined #openstack-nova09:14
*** hshiina is now known as hshiina|afk09:17
*** theanalyst has joined #openstack-nova09:18
*** namnh has quit IRC09:22
*** Nisha_Agarwal has joined #openstack-nova09:23
*** namnh has joined #openstack-nova09:23
*** annp has quit IRC09:26
*** hoangcx has quit IRC09:26
*** hoangcx has joined #openstack-nova09:26
*** annp has joined #openstack-nova09:27
*** bkopilov has quit IRC09:27
*** liuzz has quit IRC09:29
*** Dinesh_Bhor has quit IRC09:37
*** liuzz has joined #openstack-nova09:38
*** liuzz_ has joined #openstack-nova09:40
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Add a debug log entry before / after invoking migrate()  https://review.openstack.org/56355009:41
*** liuzz has quit IRC09:43
*** cdent has joined #openstack-nova09:48
*** Tom-Tom_ has quit IRC09:49
*** Tom-Tom has joined #openstack-nova09:50
kashyapstephenfin: ^ If you're looking for the first +2 of the day :-)09:50
*** Tom-Tom has quit IRC09:52
*** rmart04 has joined #openstack-nova09:53
*** markmc`` is now known as markmc09:55
*** Tom-Tom has joined #openstack-nova09:56
*** Nisha_Agarwal has quit IRC10:06
*** nicolasbock has joined #openstack-nova10:09
*** dougshelley66 has joined #openstack-nova10:09
*** hoangcx has quit IRC10:09
*** sdague has joined #openstack-nova10:14
*** dtantsur|pto is now known as dtantsur10:16
*** Nisha_Agarwal has joined #openstack-nova10:16
*** vivsoni has joined #openstack-nova10:16
*** namnh has quit IRC10:17
*** armaan has quit IRC10:17
*** armaan has joined #openstack-nova10:18
*** armaan has quit IRC10:22
*** diga has joined #openstack-nova10:22
*** pcaruana has quit IRC10:26
openstackgerritsahid proposed openstack/nova master: libvirt: fix wrong driver name for vhostuser interface  https://review.openstack.org/56156210:27
openstackgerritsahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes  https://review.openstack.org/48499710:27
*** yamamoto has quit IRC10:31
*** pcaruana has joined #openstack-nova10:33
*** armaan has joined #openstack-nova10:35
*** nicolasbock has quit IRC10:39
*** abhishekk has quit IRC10:41
*** tssurya has joined #openstack-nova10:46
*** dougshelley66 has quit IRC10:46
*** AlexeyAbashkin has quit IRC10:47
*** nicolasbock has joined #openstack-nova10:50
*** alexchad_ has quit IRC10:50
*** AlexeyAbashkin has joined #openstack-nova10:51
*** sapd1 has quit IRC10:54
*** mdbooth has joined #openstack-nova11:00
*** hoonetorg has quit IRC11:03
*** lucasagomes is now known as lucas-hungry11:06
*** yikun has quit IRC11:10
*** yikun_ has joined #openstack-nova11:10
*** stakeda has quit IRC11:10
*** yikun_ has quit IRC11:13
*** dougshelley66 has joined #openstack-nova11:13
*** yikun_ has joined #openstack-nova11:13
*** hoonetorg has joined #openstack-nova11:16
*** cdent has quit IRC11:19
*** yikun_ has quit IRC11:20
*** hoonetorg has quit IRC11:21
*** yamamoto has joined #openstack-nova11:31
*** sdeath1 has quit IRC11:32
*** jpena is now known as jpena|lunch11:33
*** tiendc has quit IRC11:33
*** amoralej is now known as amoralej|lunch11:36
*** yamamoto has quit IRC11:37
*** hoonetorg has joined #openstack-nova11:39
*** cdent has joined #openstack-nova11:41
*** diga has quit IRC11:42
*** edmondsw has joined #openstack-nova11:47
*** Tom-Tom has quit IRC11:48
*** Tom-Tom has joined #openstack-nova11:48
*** Tom-Tom_ has joined #openstack-nova11:51
*** Nisha_Agarwal has quit IRC11:52
*** sean-k-mooney has joined #openstack-nova11:52
*** Tom-Tom has quit IRC11:53
*** pcaruana has quit IRC11:54
*** Tom-Tom_ has quit IRC11:56
*** yamamoto has joined #openstack-nova12:05
*** yamamoto has quit IRC12:06
*** lucas-hungry is now known as lucasagomes12:06
*** pcaruana has joined #openstack-nova12:10
*** Tom-Tom has joined #openstack-nova12:12
*** liverpooler has joined #openstack-nova12:16
*** Tom-Tom has quit IRC12:17
*** pchavva has joined #openstack-nova12:19
*** QianYu has joined #openstack-nova12:20
*** Alex_ has quit IRC12:20
*** ktibi has joined #openstack-nova12:23
*** QianYu has quit IRC12:24
*** amoralej|lunch is now known as amoralej12:25
*** jpena|lunch is now known as jpena12:27
*** zhaochao has quit IRC12:29
*** zhaochao has joined #openstack-nova12:30
*** trozet has quit IRC12:32
*** edmondsw has quit IRC12:37
*** yamamoto has joined #openstack-nova12:37
*** jchhatbar is now known as janki12:37
*** yamamoto has quit IRC12:42
*** trozet has joined #openstack-nova12:44
*** dave-mccowan has joined #openstack-nova12:49
*** yamamoto has joined #openstack-nova12:51
* efried will be in and out all week - conference12:54
* efried waves12:54
*** efried has quit IRC12:54
*** takashin has joined #openstack-nova12:54
*** MultipleCrashes has joined #openstack-nova12:55
*** yamamoto has quit IRC12:55
sahidsean-k-mooney: did you get chance to see my last message?12:56
MultipleCrasheshttp://openstack-3rd-party-virtuozzo-ci-logs.virtuozzo.com/18/563418/1/check/check-dsvm-tempest-vz7-exe-minimal/70e8d0a/console.html.gz12:56
MultipleCrashesthis task is failing since disk space is full12:56
MultipleCrashesPlz check12:56
*** cdent has quit IRC12:56
*** MultipleCrashes has quit IRC12:58
*** lyan has joined #openstack-nova13:00
*** lyan is now known as Guest7797913:00
*** hkominos has joined #openstack-nova13:00
hkominosHi all. Is this the right channel to ask a question regarding libvirt/driver.py ?13:01
bauzashkominos: yup13:02
*** READ10 has joined #openstack-nova13:03
*** vladikr has joined #openstack-nova13:04
*** Nisha_Agarwal has joined #openstack-nova13:07
*** eharney has joined #openstack-nova13:08
*** sar has quit IRC13:10
openstackgerritAndrey Volkov proposed openstack/osc-placement master: RP delete inventories (v1.5)  https://review.openstack.org/51464213:11
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI for traits (v1.6)  https://review.openstack.org/51464313:11
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Resource class set (v1.7)  https://review.openstack.org/51464413:11
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9)  https://review.openstack.org/51464613:11
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10)  https://review.openstack.org/51464713:11
openstackgerritAndrey Volkov proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12)  https://review.openstack.org/54281913:11
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Transactionally update allocations (v1.13)  https://review.openstack.org/54667413:11
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Add nested resource providers (v1.14)  https://review.openstack.org/54667513:11
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16)  https://review.openstack.org/54804313:11
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17)  https://review.openstack.org/54832613:11
*** Nisha_Agarwal has quit IRC13:13
sean-k-mooneysahid: hi no sorry13:13
sean-k-mooneysahid: what was it?13:14
sahidsean-k-mooney: it's about tempest-dsvm-ovsdpdk-nfv-networking-xenial13:15
sean-k-mooneysahid: oh the fact that its offline?13:15
sean-k-mooneyor at least it was last week13:15
*** esberglu has joined #openstack-nova13:15
sahidno no it's about to update it to configure virtio-net queue sizez13:15
sean-k-mooneysahid: ah you want to update it to test with queue configured13:16
sahidyep exactly13:16
*** moshele has quit IRC13:16
*** psachin has quit IRC13:17
sean-k-mooneysahid: im nologer the product owner for that ci but i can reach out internally  and ask if they can enable it13:17
sean-k-mooneyinfact we should proably also enable multiqueue also13:17
sahidsean-k-mooney: that would be great, i did some internal tests, but dansmith really would like to see some ci running13:17
sahidhttps://review.openstack.org/#/c/484997/13:18
sahidsean-k-mooney: that would be great yes13:18
hkominoshi: 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
hkominosJust to clarify, I want nova to add another pci-slot to the Vms that it creates13:20
*** Nisha_Agarwal has joined #openstack-nova13:20
*** ratailor has quit IRC13:21
sean-k-mooneysahid: so out of interest what was the issue that was found just before merging?13:21
*** cdent has joined #openstack-nova13:21
*** ttsiouts_ has joined #openstack-nova13:24
sahidlibvirt rejects driver name=vhost-user13:24
sahidonly acceptable values are qemu and vhost13:24
*** jackie-truong has joined #openstack-nova13:28
kashyapYeah, it is also documented so in the formatdomain.html13:29
*** dougshelley66 has quit IRC13:29
kashyap(Probably the upstream doc of that sub section needs a bit more tweaking.)13:30
*** dougshelley66 has joined #openstack-nova13:30
openstackgerritMargarita Shakhova proposed openstack/nova master: Do not skip migrations in _destroy_evacuated_instances()  https://review.openstack.org/56362313:30
*** Nisha_Agarwal has quit IRC13:31
dansmithsean-k-mooney: does that ci not support depends-on?13:32
hkominosbauzas: 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 IRC13:35
bauzashkominos: the libvirt driver gets the number of PCI devices by calling libvirtd and then creating a config13:35
sean-k-mooneydansmith: that ci does not it is still using zuul 2.5 with jenkins13:35
bauzashkominos: then the config is used by the driver.py module to create a XML13:35
bauzaswhen I say a config, I mean a config object13:36
sean-k-mooneydansmith: well the old depends on might work13:36
bauzashkominos: see virt.libvirt.config13:36
sean-k-mooneydansmith: 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 that13:37
*** r-daneel has joined #openstack-nova13:37
dansmithsean-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-nova13:38
sean-k-mooneydansmith: the intel nfv ci runs on changes to nova also13:38
hkominosbauzas: 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
dansmithsean-k-mooney: oh does it? then we could just use a DNM change to default those queues to something right?13:39
sean-k-mooneydansmith: 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-xenial13:39
sean-k-mooneydansmith: yep you could13:39
dansmithah yeah I see13:39
dansmithsahid: easy ^13:40
*** QianYu has joined #openstack-nova13:42
kashyapWhat is Eric Fried's Friday nick?13:43
cdentkashyap: usually fried_rice13:43
kashyapcdent: Ah, thanks.  I think he's AFK now.13:43
*** felipemonteiro__ has joined #openstack-nova13:44
sahiddansmith, sean-k-mooney are the repo of the template to i can provide a patch?13:44
sahidwhere*13:44
cdentkashyap: he's at a conference this week13:46
dansmithsahid: just nova.. I'll do it, just a sec13:46
*** felipemonteiro_ has quit IRC13:47
kashyapcdent: Noted; thanks.13:47
openstackgerritDan Smith proposed openstack/nova master: DNM: Test virtio queue sizes  https://review.openstack.org/56362613:48
dansmithsahid: ^13:48
sahiddansmith: oh... smart :)13:49
cdentscheduler meeting #openstack-meeting-alt in ten minutes13:49
bauzashkominos: 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-nova13:50
kashyapAs 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 IRC13:51
*** yamamoto has joined #openstack-nova13:51
*** rmart04 has quit IRC13:51
*** rmart04 has joined #openstack-nova13:52
hkominosbauzas:yes13:53
hkominosbauzas:I think it does provide one anyway but I need one more.13:54
*** QianYu has quit IRC13:54
bauzashkominos: well, I don't know why, so you should ask for that by a blueprint13:54
*** QianYu has joined #openstack-nova13:54
tssuryadansmith: 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 IRC13:54
*** awaugama_away has joined #openstack-nova13:54
dansmithtssurya: doing it while adding the column should not be done because it can lock the table and be a problem for pre-applying schema13:55
dansmithtssurya: if you can, the best way is on loading/saving the object out of the db13:55
*** liuzz has joined #openstack-nova13:55
dansmithtssurya: 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 immediately13:55
tssuryadansmith: oh okay, yep will go that way then13:55
dansmithtssurya: example: https://github.com/openstack/nova/blob/master/nova/objects/service.py#L259-L26413:56
*** dougshelley66 has joined #openstack-nova13:56
dansmiththat was how we generated uuids on the service objects13:56
*** tbachman has quit IRC13:56
tssuryadansmith: thanks13:56
*** yamamoto has quit IRC13:57
tssuryaI will use this approach13:57
*** rmart04 has quit IRC13:57
*** cfriesen has joined #openstack-nova13:57
*** rmart04 has joined #openstack-nova13:57
*** edmondsw has joined #openstack-nova13:58
*** felipemonteiro_ has joined #openstack-nova14:00
*** felipemonteiro__ has quit IRC14:00
*** ralonsoh_ has joined #openstack-nova14:02
*** efried has joined #openstack-nova14:04
sean-k-mooneysahid: 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 effect14:04
stephenfinbauzas: All comments addressed here. Fancy hitting it again? https://review.openstack.org/#/c/379525/14:05
*** ralonsoh has quit IRC14:06
bauzasstephenfin: yup, I can sneak peek on it14:06
sean-k-mooneysahid: 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-nova14:07
*** mriedem has joined #openstack-nova14:08
sahidsean-k-mooney: ok thanks14:08
*** mlavalle has joined #openstack-nova14:09
openstackgerritMatt Riedemann proposed openstack/nova master: Add policy rule to block image-backed servers with 0 root disk flavor  https://review.openstack.org/56128414:09
kashyapCan anyone remind me what is the way to recheck this job: 'legacy-tempest-dsvm-multinode-live-migration'14:14
*** cfriesen has quit IRC14:14
kashyapSeems 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.gz14:17
* kashyap digs further14:17
*** hkominos has quit IRC14:18
*** cdent has quit IRC14:18
*** tbachman has joined #openstack-nova14:20
*** felipemonteiro__ has joined #openstack-nova14:20
*** liuzz has quit IRC14:23
*** felipemonteiro_ has quit IRC14:24
*** liuzz has joined #openstack-nova14:24
mriedemkashyap: you can't recheck specific jobs14:26
mriedemyou just leave a 'recheck' comment14:26
kashyapmriedem: 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
mriedemmnaser: 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 bfv14:27
openstackgerritMerged openstack/nova master: libvirt: Add a debug log entry before / after invoking migrate()  https://review.openstack.org/56355014:29
*** hjensas is now known as hjensas|afk14:29
openstackgerritMerged openstack/nova master: Mention that users need noVNC >= 0.6  https://review.openstack.org/56284714:29
*** links has quit IRC14:30
kashyapmriedem: 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 IRC14:31
*** jroll has quit IRC14:31
*** jroll has joined #openstack-nova14:32
*** shoffmann_ has joined #openstack-nova14:33
mriedemwas just looking at it14:33
*** namnh has joined #openstack-nova14:34
*** awaugama_away is now known as awaugama14:35
*** bkopilov has joined #openstack-nova14:38
*** hemna_ has joined #openstack-nova14:39
kashyapThanks.14:39
kashyapstephenfin: efried: Thanks for the quick review (commit: 880f628)!14:40
efriedkashyap: yahyoubetcha.  Easy one.14:40
kashyapefried: More context on that is - before that commit, it was incredibly difficult to tell if that migrate() method has run or not14:41
efriedI get it.14:41
*** QianYu has quit IRC14:41
efriedkashyap: 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
kashyapAlthough there's a comment snuck in to Eventlet schedule a new thread, but that's too subtle14:42
kashyapefried: Hehe14:42
*** cdent has joined #openstack-nova14:42
efriedkashyap: I'm counting on major brownie points from this series: https://review.openstack.org/56044414:43
kashyapefried: Don't worry, I'll send more your way; mechanical clean up of constants :P14:43
mnasermriedem: nice patch, is there a way we can track things like "things to do for next release" ?14:43
kashyapUntil you might get sick of it :P  (Once Matt merges the earlier MIN_ bump)14:43
mriedemmnaser: we don't have anything formal for that besides the TODOs in the code14:43
mnaseri see14:45
*** kholkina has quit IRC14:45
kashyapefried: Interesting.  Have to still educate myself on that (ProviderTree stuff)14:45
efriedkashyap: 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
kashyapefried: I don't claim to be fresh :P14:46
kashyapefried: But yeah, got a URL to that doc?14:47
efriedkashyap: working on that (I pretty much suck at searching the docs)14:47
kashyapAah, I see.  The past tense word "wrote" made me think it's already in tree14:47
efriedkashyap: It is.14:48
efriedkashyap: Mahbad, it's a reference doc: https://docs.openstack.org/nova/latest/reference/update-provider-tree.html14:48
* kashyap will find it14:48
efriedkashyap: So yeah, in recompense for that two-line review, you get to read my doc and give me feedback.14:49
efriedkashyap: No mistake, you got the raw end of that deal.14:49
kashyapefried: 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-nova14:50
* kashyap is in a hurry as he needs to be AFK a bit early on Monday evenings14:50
efriedkashyap: thanks in advance14:50
*** felipemonteiro__ has quit IRC14:51
*** felipemonteiro__ has joined #openstack-nova14:51
kashyapefried: 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
efriedkashyap: I know nothing from z.14:52
efriedkashyap: And PowerKVM != PowerVM.  I know nothing about the former.14:52
*** gcb has joined #openstack-nova14:53
*** yamamoto has joined #openstack-nova14:53
efriedkashyap: 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-nova14:53
kashyapOops, sorry for the mix-up14:54
*** artom has joined #openstack-nova14:54
*** zhaochao has quit IRC14:54
kashyap(Nod)14:54
efriedYeah, I'm sorry too.14:54
efriedIt didn't need to be that confusing.14:54
efriedBut you know branding/marketing folk.14:54
kashyapHehe, totally14:54
kashyapefried: The KVM author still regrets the name of it, I hear...14:54
*** AlexeyAbashkin has quit IRC14:54
*** Alexey_Abashkin is now known as AlexeyAbashkin14:54
kashyaphttps://en.wikipedia.org/wiki/KVM_switch vs. https://en.wikipedia.org/wiki/Kernel-based_Virtual_Machine14:55
efriedgibi, mriedem: FYI: https://review.openstack.org/#/c/561724/14:57
*** ttsiouts_ has quit IRC14:57
*** yamamoto has quit IRC14:58
*** tbachman has quit IRC14:58
kashyapmriedem: Hmm, you gave +2, but you noted a whitespace nit & the needless comment.  Respinning to remove the needless cruft.14:59
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Rocky"  https://review.openstack.org/55878315:02
openstackgerritKashyap Chamarthy proposed openstack/nova master: Drop MIN_LIBVIRT_SET_ADMIN_PASSWD  https://review.openstack.org/56231215:02
efriedgibi: 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-nova15:05
*** QianYu has joined #openstack-nova15:05
*** rmart04 has quit IRC15:06
kashyapRemind me again, if I update the commit message, will I "lose" the +2?15:07
* kashyap has an OCD to remove needless commas15:08
*** rajinir has joined #openstack-nova15:08
kashyap(Only one spurious comma this time, though)15:08
stephenfinkashyap: You will, yup15:09
*** sridharg has quit IRC15:09
kashyapOkay, I'll leave it as is, not worth to bother reviewers about it15:09
stephenfinIf it's not a bigger issue, you're usually better of just leaving it unless someone else comments on that or something else15:09
*** felipemonteiro_ has joined #openstack-nova15:09
stephenfin(If it was a bigger issue, you wouldn't have got the +2)15:09
kashyapYeah, it's not an issue all, only OCD.15:09
kashyapstephenfin: 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
sahiddansmith: the intel ci just finished, that looks to be OK15:11
*** QianYu has quit IRC15:12
*** felipemonteiro__ has quit IRC15:13
stephenfinkashyap: Ooh, for sure15:16
bauzasstephenfin: 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 usage15:16
kashyapGracias15:16
stephenfinbauzas: I've it done15:16
bauzasstephenfin: where ? I could do the same15:16
stephenfin:)15:16
stephenfinsec15:16
stephenfinbauzas: https://github.com/stephenfin/nova/commit/58b53e888feb146123087f2e3f54f677f1fdccdc15:16
bauzasor make it generic15: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
dansmithsahid: yep, cool15:17
*** janki has quit IRC15:17
bauzasstephenfin: I'm not sure we should dynamicall add the groups in nova/config15:18
bauzasstephenfin: rather closer to the caller15:18
bauzasstephenfin: my thoughts was to use devices.enabled_vgpu_types as the ListOpt value for knowing which opt groups to check15:19
stephenfinbauzas: 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
openstackgerritMatt Riedemann proposed openstack/nova master: Remove vestigial system_metadata param from info_from_instance()  https://review.openstack.org/56172415:19
bauzasstephenfin: or use MultiStrOpt for that15:19
stephenfinbauzas: Aye, I used 'CONF.neutron.physnets' for that15:19
*** pcaruana has quit IRC15:19
stephenfinhttps://github.com/stephenfin/nova/commit/58b53e888feb146123087f2e3f54f677f1fdccdc#diff-bf484fde349b127980f9341543ce3c9aR3815:20
stephenfinso '[neutron] physnets = foo, bar'15:20
*** felipemonteiro_ has quit IRC15:20
bauzasright15:20
bauzasstephenfin: see the context in https://review.openstack.org/#/c/557065/2/specs/rocky/approved/vgpu-rocky.rst@8615:20
*** felipemonteiro_ has joined #openstack-nova15:20
bauzasstephenfin: for MultiStrOpt15:20
stephenfinI guess I could use prefixed attribute names but someone (jaypipes or dansmith, I think) thought that was eewy15:21
* stephenfin clicks15:21
dansmithjaypipes: 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
jaypipesdansmith: ack, will do.15:22
jaypipesah, this is the one I'd asked sahid to combine the first two patches. yeah... one minute.15:23
sahiddansmith: it was a really nice idea your DNM patch, i'm going to reuse it a lot :)15:23
dansmithjaypipes: yep15:24
dansmithsahid: :)15:24
bauzasstephenfin: 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 code15:26
stephenfinbauzas: Got an example?15:26
bauzassec15:26
stephenfinThey have the same key by definition15:26
*** lajoskatona has quit IRC15:27
jaypipesdansmith: off to the test pits it goes.15:28
efriedgibi: Ready for you: https://review.openstack.org/#/c/561724/    self.NotificationEmail().AndRemove(bug_1764390)15:28
dansmithjaypipes: the test pits ahead won't really test it, but it came back from other test pits that did recently, so.. thanks :)15:29
bauzasstephenfin: actually, what I want is a MultiDictOpt15:29
* bauzas gets confused by oslo.config15:29
jaypipesbauzas: you do *not* want a MultiDictOpt.15:30
bauzasI don't know15:30
jaypipesbauzas: dansmith might kill you if you used that.15:30
bauzaswhat I want is a pony15:30
jaypipesheh15:30
jaypipesbauzas: your dog is as big as a pony.15:30
stephenfinkashyap: 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
bauzasI just want a fucking easy way to describe a transitive relationship between a thing and a list of other things15:30
bauzasjaypipes: not as big as a pony, I saw taller yesterday when I was in a dog show15:31
jaypipesbauzas: relational DBs are pretty good at storing relations :P15:31
bauzasjaypipes: I know15:31
bauzasjaypipes: I have a crazy idea to just add a new nova-manage helper if you want my own thoughhts15:32
*** sar has joined #openstack-nova15:32
bauzasjust for something super specific like that15:32
stephenfinbauzas: s/in a dog show/at a dog show/ ;)15:32
bauzasbecause libvirtd doesn't help me with that15:32
*** dklyle has joined #openstack-nova15:32
bauzasso, things are, we know we're doomed with defining that relationship elsewhere but nova15:32
jaypipesstephenfin: an important clarification :)15:32
jrollstephenfin: "in" was correct, bauzas was a contestant :P15:32
bauzasbut I want to make it the less painful15:32
sean-k-mooneybauzas: whay are the entities you are trying to define the relationship of?15:33
bauzasjaypipes: stephenfin: jroll: if you care https://lh3.googleusercontent.com/fFHAwwIKjuW9uKAYkPrsTjsw_bOO6gQgA7xw-B0d2-gmC24J8Z1DUpSzKHwkIJesK_aTGeKGQ8ts1P67SZhmsxUiVEEABuFv7Hg4CvQ-ROw8TNcOgKOgw7HjUqL6ZMsLPYgLee8mDkyj0PpCTRUiD936luaHuThLkW__s1Ym9UAXmMPqXo7J1cBv1GkYjyunJfWbJTUBklcJHd6zwTz51M1DhIMRd0zYpr9xm8kCG3Qbn0NnzVK6jt7ppA00jqUbl-NfReOZyCirPZvrMLJyrtm9nza7AwwOuCm0NGYrVBMZlpnGKQ8VmmlG3r8wz9uPKvUjREitqqN4WgTjhDgRrIomD6Oe9bCMRnRX215:34
bauzasx8kAufXHD_RF1-sfVGt6FEMv2MOY7Y2WH5d4MhvFQT3WF37ZLRJxAxXXmhloPjqF6qh374jvsopMPVaKykwW9zuTiZ87CNcQbfXeGze46RC_CAPmccFyYeA84_y7t0TIJOzStqFXA5LvLivtFfxJtGq06ZRD_P0ZSHcw_cdigRN84R-jDPXGRxnbuAkm5f16HfyLzxARcz1OPfVdUBFliYvHR4h8vZ9VI2ajmprFcxZp2mxIhuvD8FC6lzAwyRLayjnfn4ElfZSPVcm8zVJ9VBFnSwBL5xSq6ZaayKfdL1KEPAYgpQYIj0bdLk6w=w1689-h950-no15:34
sean-k-mooneybauzas: you can use somthing like ovo in somecases also15:34
bauzassean-k-mooney: I just want to say "that GPU type is for those PCI devices"15:34
bauzasperiod15:34
bauzasbut 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 most15:35
bauzasso, I'm out of ideas15:35
sean-k-mooneybauzas: ah ok ya so like the pci alais with out all the other symantics that is layered on that15:35
bauzasexcept sealing in the nova codebase something I don't want15:35
bauzasbecause that's just libvirtd isn't mature enough on that feature to provide CLIs for that15:36
bauzasso a config option is just a 'mid-term' solution until, I hope, something comes up that allows to specify that outside of nova15:36
sean-k-mooneybauzas: you can find the vgpu types in virsh capablitites or virsh nodedev-list though right?15:37
*** lpetrut has joined #openstack-nova15:37
bauzassean-k-mooney: sure, but then I need to say which type is for which pGPU15:37
sean-k-mooneybauzas: one of those should expose the mdev types that correstpond the the different vgpus15:37
bauzassean-k-mooney: context is https://review.openstack.org/#/c/557065/2/specs/rocky/approved/vgpu-rocky.rst15:38
bauzassean-k-mooney: sure, but each pGPU supports multiple types15:38
*** chyka has joined #openstack-nova15:38
*** moshele has joined #openstack-nova15:38
bauzasXen doesn't have that problem, because the xen CLI allows to provide such tool out of nova15:38
bauzasI don't have that in my bucket, so I'm doomed15:39
sean-k-mooneybauzas: 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 flavor15:39
bauzassean-k-mooney: we already have the former15:39
bauzassean-k-mooney: what we miss is the latter15:39
*** fragatina has joined #openstack-nova15:39
bauzasagain, we can technically have a StrOpt and do some magical JSON import15:40
*** moshele has quit IRC15:40
sean-k-mooneybauzas: 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 api15:40
bauzasjust because oslo.config misses that15:40
bauzasno15:40
bauzassean-k-mooney: we currently have enabled_vgpu_types that lists all the supported types accepted by that compute15:40
*** fragatin_ has joined #openstack-nova15:41
bauzassean-k-mooney: what I miss is a way to say for each of those types, which pGPU(s) is/are associated with15:41
sean-k-mooneybauzas: ah ok.15:41
bauzasjaypipes: 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.config15:42
bauzasjaypipes: I'm not super happy with the second option that carries a lot of misunderstandings15:43
*** jpena is now known as jpena|brb15:43
sean-k-mooneybauzas: i belive stephenfin is looking at reusing how cindier does dynmaic config for the numa aware vswitch spec15:44
sean-k-mooneybauzas: could we use the same approch here or is that overkill15:44
*** tbachman has joined #openstack-nova15:44
bauzasI think that's the only option I have tbh, if we still care about nova.conf15:44
*** abhishekk has joined #openstack-nova15:44
*** fragatina has quit IRC15:45
sean-k-mooneybauzas: 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 clean15:46
* bauzas goes back to code15:47
jaypipesbauzas: 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
bauzasok, I can try then15:47
*** cfriesen has joined #openstack-nova15:48
*** dklyle has quit IRC15:49
dansmithjaypipes: yeah15:49
openstackgerritMargarita Shakhova proposed openstack/nova master: Do not skip migrations in _destroy_evacuated_instances()  https://review.openstack.org/56362315:49
*** yamamoto has joined #openstack-nova15:54
*** owalsh is now known as owalsh_afk15:58
*** yamamoto has quit IRC16:00
*** gcb has quit IRC16:03
*** gyee has joined #openstack-nova16:03
*** armaan has quit IRC16:03
*** namnh has quit IRC16:04
*** ccamacho has quit IRC16:04
*** damien_r has quit IRC16:04
*** armaan has joined #openstack-nova16:04
openstackgerritsahid proposed openstack/nova master: add mtu to libvirt xml for ethernet and bridge types  https://review.openstack.org/55307216:05
sahidsean-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 you16:06
*** dpawlik has quit IRC16:06
sean-k-mooneysahid: ya no worries sorry i had ment to get back to that patch sooner16:08
*** armaan has quit IRC16:09
*** armaan has joined #openstack-nova16:09
sean-k-mooneysahid: 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 fixes16:11
*** ccamacho has joined #openstack-nova16:11
openstackgerritMatt Riedemann proposed openstack/nova master: Remove explicit instance.info_cache.delete()  https://review.openstack.org/56368916:11
*** shoffmann_ has left #openstack-nova16:12
sahidsean-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/174749616:12
openstackLaunchpad 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-nova16:13
*** munimehan has joined #openstack-nova16:13
*** sahid has quit IRC16:14
sean-k-mooneysahid: 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 IRC16:15
*** tianhui has joined #openstack-nova16:15
*** hjensas|afk is now known as hjensas16:18
*** sambetts is now known as sambetts|afk16:20
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Add policy rule to block image-backed servers with 0 root disk flavor  https://review.openstack.org/56369216:21
*** jpena|brb is now known as jpena16:22
openstackgerritMerged openstack/nova master: Cleanup RP and HM records while deleting a compute service.  https://review.openstack.org/55492016:27
*** tbachman has quit IRC16:29
*** lucasagomes is now known as lucas-afk16:29
*** edmondsw has quit IRC16:29
*** Alexey_Abashkin has joined #openstack-nova16:31
openstackgerritEric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates  https://review.openstack.org/51775716:32
*** efried has quit IRC16:33
openstackgerritEric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates  https://review.openstack.org/51775716:33
*** AlexeyAbashkin has quit IRC16:34
*** Alexey_Abashkin is now known as AlexeyAbashkin16:34
*** sdeath has joined #openstack-nova16:41
*** dpawlik has joined #openstack-nova16:46
*** sdeath has quit IRC16:47
*** tbachman has joined #openstack-nova16:48
*** jpena is now known as jpena|off16:50
*** dpawlik has quit IRC16:50
mriedemmelwitt: 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
mriedemif 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
mriedemso i left some notes in the etherpad16:53
*** mgoddard has quit IRC16:53
*** yamamoto has joined #openstack-nova16:56
openstackgerritSurya Seetharaman proposed openstack/nova stable/queens: Cleanup RP and HM records while deleting a compute service.  https://review.openstack.org/56369816:57
*** AlexeyAbashkin has quit IRC16:59
jaypipeslbragstad: some comments inline on your set scope_types for admin password policies patch.17:01
jaypipeslbragstad: comments and questions that is...17:01
*** yamamoto has quit IRC17:01
*** ralonsoh_ has quit IRC17:03
*** jaosorior has quit IRC17:05
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Add policy rule to block image-backed servers with 0 root disk flavor  https://review.openstack.org/56370017:05
*** sar has quit IRC17:05
lbragstadjaypipes: awesome - checking17:08
*** derekh has quit IRC17:08
*** arvindn0_ has joined #openstack-nova17:09
*** tbachman_ has joined #openstack-nova17:09
*** mdbooth has quit IRC17:10
*** tbachman has quit IRC17:11
*** tbachman_ is now known as tbachman17:11
openstackgerritZack Cornelius proposed openstack/nova-specs master: Libvirt file backed memory  https://review.openstack.org/56370417:16
*** mdnadeem has quit IRC17:17
*** mvk has quit IRC17:22
openstackgerritMerged openstack/nova master: libvirt: fix wrong driver name for vhostuser interface  https://review.openstack.org/56156217:22
*** tesseract has quit IRC17:23
*** tbachman_ has joined #openstack-nova17:30
jaypipeslbragstad: thx for the answers. so what does the 'all' in system={'all': True} refer to?17:31
*** mgoddard has joined #openstack-nova17:31
*** tbachman has quit IRC17:32
*** tbachman_ is now known as tbachman17:32
*** dtantsur is now known as dtantsur|afk17:33
*** edmondsw has joined #openstack-nova17:34
*** chyka_ has joined #openstack-nova17:36
*** edmondsw has quit IRC17:37
*** edmondsw has joined #openstack-nova17:37
artomdansmith, https://review.openstack.org/#/c/563704/ you're some sort of ninja dude17:38
dansmithartom: um, what?17:38
artom20 minutes since patch posted, 13 comments, that's close to 1 comment per minute :)17:39
*** efried has joined #openstack-nova17:39
*** chyka has quit IRC17:39
dansmithartom: only 14 between post and comment.. don't sell me short17:39
artomGomen, senpai17:40
artom(Sincere apologies to all Japanese speakers here)17:40
dansmithheh17:41
*** chyka_ has quit IRC17:41
* artom goes back to wondering wtf Ubuntu named his nic "rename3" and why it gets a new IP every boot17:42
openstackgerritJulia Kreger proposed openstack/nova master: DNM: Test: Ironic: Lock the instance if plug vifs  https://review.openstack.org/56371417:42
* artom just corrected dansmith17:43
* artom is obviously having second thoughts, immediately after doing it.17:44
dansmithartom: when you're right you're right17:44
artomI shall speak of this moment to my grandchildren17:45
*** tssurya has quit IRC17:47
melwittmriedem: ack17:47
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Add policy rule to block image-backed servers with 0 root disk flavor  https://review.openstack.org/56371917:49
*** jaosorior has joined #openstack-nova17:49
*** yamahata has quit IRC17:50
openstackgerritChris Dent proposed openstack/nova master: Use nova.db.api directly  https://review.openstack.org/54326217:52
*** tomtom001 has quit IRC17:54
openstackgerritJim Rollenhagen proposed openstack/nova master: WIP: ironic: add instance_uuid before any other spawn activity  https://review.openstack.org/56372217:54
*** tomtom001 has joined #openstack-nova17:55
*** chyka has joined #openstack-nova17:55
*** yamamoto has joined #openstack-nova17:58
lbragstadjaypipes: it's supposed to refer to the entire deployment system17:58
*** dpawlik has joined #openstack-nova17:58
* lbragstad is terrible with naming things though17:58
lbragstadjaypipes: following the documentation here - http://specs.openstack.org/openstack/keystone-specs/specs/keystone/queens/system-scope.html#authenticating-for-a-system-scoped-token17:59
jaypipeslbragstad: ok. so the string 'all' is special?17:59
lbragstadyeah17:59
jaypipesgotcha17:59
openstackgerritMatt Riedemann proposed openstack/nova master: doc: add note about xenapi aggregate upcall being resolved  https://review.openstack.org/56372417:59
lbragstadour intention was to build it so that eventually we could do something like system: {$service: $id}18:00
*** chyka has quit IRC18:00
lbragstadbut that would be relayed in the token18:00
*** r-daneel has quit IRC18:00
*** r-daneel has joined #openstack-nova18:01
*** yamamoto has quit IRC18:02
*** dpawlik has quit IRC18:02
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: mark block_device_mapping_v2.boot_index as required  https://review.openstack.org/56372818:06
mriedemalex_xu: see ^ i don't know if we should revert https://review.openstack.org/#/c/524208/ or not18:07
mriedemi tend to think we should18:08
*** felipemonteiro__ has joined #openstack-nova18:08
*** abhishekk has quit IRC18:11
*** felipemonteiro_ has quit IRC18:12
*** cfriesen has quit IRC18:30
*** Zames has joined #openstack-nova18:31
*** Zames has quit IRC18:33
openstackgerritMatt Riedemann proposed openstack/nova master: Fix dropped check for boot_index 0 in _validate_bdm  https://review.openstack.org/56373218:35
*** tbachman is now known as tbachmanAway18:39
openstackgerritEric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates  https://review.openstack.org/51775718:41
*** mgoddard has quit IRC18:43
*** harlowja has joined #openstack-nova18:44
openstackgerritEric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates  https://review.openstack.org/51775718:48
openstackgerritMatt Riedemann proposed openstack/nova master: docs: link to volume multi-attach demo recording  https://review.openstack.org/56373418:49
openstackgerritLance Bragstad proposed openstack/nova master: Set scope_types for admin password policies  https://review.openstack.org/55361318:51
mriedemjackie-truong: can you post a spec amendment per L68 https://etherpad.openstack.org/p/nova-runways-rocky ?18:53
*** sar has joined #openstack-nova18:53
jackie-truongmriedem: Yep, Peter Hamilton from our team is working on it18:54
mriedemok, thanks18:55
jackie-truongIn the mean time, Brianna has started a WIP patch for notifications: https://review.openstack.org/#/c/563269/18:55
*** jaosorior has quit IRC18:55
*** cfriesen has joined #openstack-nova18:55
*** yamamoto has joined #openstack-nova18:59
*** fragatin_ has quit IRC19:02
*** yamamoto has quit IRC19:03
*** amoralej is now known as amoralej|off19:06
*** felipemonteiro__ has quit IRC19:06
*** felipemonteiro__ has joined #openstack-nova19:07
*** harlowja has quit IRC19:07
*** rajinir has quit IRC19:08
*** chyka has joined #openstack-nova19:10
*** mgoddard has joined #openstack-nova19:10
*** tbachmanAway is now known as tbachman19:10
mriedemarvindn05: 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-nova19:11
mriedemmelwitt: 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
mriedemof 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 go19:13
*** imacdonn has quit IRC19:13
*** imacdonn has joined #openstack-nova19:14
*** mgoddard has quit IRC19:16
mriedemesberglu: are you updating https://review.openstack.org/#/c/526094/ /19:18
mriedem?19:18
esberglumriedem: Yep working on it right now19: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 runway19:26
*** Nisha_Agarwal has joined #openstack-nova19:27
mriedemarvindn0_: my understanding is that runways are for things that are ready to go19:28
mriedemand i wouldn't consider this ready to go19:28
mriedembtw, i'm trying to summarize alternatives for this in the ML to get some visibility19:28
openstackgerritMerged openstack/nova master: Add root and parent provider uuid to group by clause  https://review.openstack.org/56237919:30
*** Nisha_Agarwal has quit IRC19:32
arvindn0_mriedem: thanks. should be fine to move to end of runway...i guess the runway maintainer is going to make this change19:32
*** yamahata has joined #openstack-nova19:36
jaypipesartom: 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
artomjaypipes, looking19:39
jaypipesartom: cheers19:40
artomjaypipes, I'd go with gibi19:40
artomMaybe a "real" example would make more sense:19:40
artom1. Just interface attachment: nova interface-attach port-id=blah19:40
artom2. Tagged attachment: nova interface-attach port-id=blah,tag=foo19:41
*** lpetrut has quit IRC19:41
*** jackie-truong has quit IRC19:41
*** ktibi has quit IRC19:43
openstackgerritArvind Nadendla proposed openstack/nova-specs master: Handle rebuild of instance with new image  https://review.openstack.org/56071819:44
*** cdent has quit IRC19:45
*** READ10 has quit IRC19: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 direction19:47
openstackgerritMerged openstack/nova master: Leave a hint when populate_schema fails  https://review.openstack.org/56275719:52
jaypipesarvindn0_: 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
artomOh hey are we reopening the rebuild with new image can of worms19:56
jaypipesartom: 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
artomjaypipes, because the driver is what knows about the actual hardware metadata, like PCI and MAC address for NICs19:57
jaypipesarvindn0_: no, I am not.19:57
jaypipesarvindn0_: if there is nothing to allocate, there's no reason to call GET /allocation_candidates.19:57
*** sambetts|afk has quit IRC19:58
arvindn0_jaypipes: if there are strong opinions against it, then we need to drop that proposal...19:58
jaypipesarvindn0_: ok, gimme a bit.19:59
jaypipesarvindn0_: 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-nova19:59
*** sambetts_ has joined #openstack-nova20:00
jaypipesartom: but why is the capability to associated a tag of "foo" with a PCI address something that is virt-driver dependent?20:00
dansmithjaypipes: because the virt driver is what knows the address of the thing20:01
dansmithjaypipes: if you ask for two nics, one tagged as foo and one as bar,20:01
artomjaypipes, because some virt drivers don't have the code for it :)20:01
dansmithonly the virt driver knows how and where those are attached20:01
artom(Wasn't trying to be condescending, I swear!)20:02
jaypipes*sigh*20:02
*** dpawlik has joined #openstack-nova20:04
*** dpawlik has quit IRC20:05
*** liverpooler has quit IRC20:05
jaypipesdansmith, 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 IRC20:05
*** dpawlik has joined #openstack-nova20:05
artomjaypipes, yes20:06
jaypipesthat's lovely.20:06
jaypipesimplementation-defined interfaces.20:06
dansmithjaypipes: you know that some virt drivers don't even provide network interface attachments at all right?20:07
dansmith*gasp*20:07
artomI 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
jaypipesartom: why doesn't something above the virt layer save the tag information for the interface?20:07
dansmithjaypipes: it does happen above the virt driver level, but it doesn't have sufficient information to be useful20:07
artomjaypipes, it does, and we could conceivably just put "device: {tag: foo}" in the metadata20:07
artomBut that's useless for the guest OS20:07
jaypipeswhy is that useless for the guest OS20:08
jaypipes?20:08
dansmithjaypipes: 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
dansmiththose are the things the guest needs to know which tag applies to which20:08
artomFor the guest to do anything useful with that it would need to know something about the hardware "associated" with that tag20:08
dansmithand compute manager can't know that20:08
artomjaypipes, https://notartom.net/2017/06/20/virtual-device-role-tagging-better-explained/ /shameless plug20:08
*** owalsh_afk is now known as owalsh20:08
*** tomtom001 has quit IRC20:09
jaypipesartom: 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)" question20:10
artomjaypipes, it doens't20:10
dansmithby not supporting more than one virt method?20:10
*** tomtom001 has joined #openstack-nova20:10
artomjaypipes, wait, ec2... the metadata API?20:10
dansmithhe means EC2 the service I believe20:10
jaypipesdansmith: no, how can an EC2 API user inform their guest that a particular NIC is "for this specific network"?20:10
dansmithjaypipes: that's totally different than applying a tag to a thing20:11
artomdansmith, to be fair, that's kinda what tags were made for20:11
dansmithjaypipes: you may impute some meaning from the tag, but that's different20:11
jaypipesexactly what they were made for, actually.20:11
dansmithjaypipes: so lets use the volume case20:12
dansmithjaypipes: doesn't ec2 let you say "this volume will be vdb"?20:12
jaypipesdansmith: you can specify one, but it might rename it behind the scenes.20:13
artomAlso, and this may be stupid, but what does EC2 have to do with any of this?20:13
dansmithartom: zero, because they don't do many hypervisors20:14
*** dave-mccowan has quit IRC20:14
*** armaan has quit IRC20:14
dansmithjaypipes: 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 this20:14
dansmithjaypipes: https://hangouts.google.com/call/Jcmcsrr1qa7qmYfuZ_myAAEE ?20:15
dansmithartom: of course, if jaypipes'll join20:16
artomOf course what?20:16
dansmithartom: join ^20:17
jaypipesI'm on.20:17
artomdansmith, uh, sure, except it keeps telling me I'm not allowed20:17
dansmithartom: we're on, it's just public20:18
artomOh hey, the gmail account works20:18
melwittmriedem: 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#alternatives20:23
mriedemmelwitt: exactly. ok i'll start a thread.20:23
melwittthanks20:24
*** sar has quit IRC20:26
melwittarvindn0_, mriedem: which thing are y'all talking about "moving to the end of the runway" earlier?20:28
mriedemglance-image-traits20:28
mriedemL6220:28
mriedemend-o-queue20:29
arvindn0_Support Traits in Glance: https://blueprints.launchpad.net/nova/+spec/glance-image-traits20:29
melwittoh, I see20:29
melwittI got confused with the rebuild talk right after it20:29
*** felipemonteiro__ has quit IRC20:29
arvindn0_we are discussing  an ammendement to the originally approved spec...20:29
melwittgotcha. I see the note in the runways etherpad20:30
arvindn0_rebuild is related to the amendment :)20:30
melwittah, okay20:30
*** dpawlik has quit IRC20:34
*** dpawlik has joined #openstack-nova20:37
melwittmriedem: 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 there20:38
mriedemcool20:38
mriedemhttp://lists.openstack.org/pipermail/openstack-dev/2018-April/129736.html20:38
melwittthanks20:38
arvindn0_mriedem: wanted to quick check something on the scenario 2 i detailed in the comment20:41
arvindn0_a host with two SRIOV nic. One is normal SRIOV nic, another one with some kind of offload feature.20:43
melwittyikun: 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-rocky20: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 nic20:44
*** rajinir has joined #openstack-nova20:45
*** cfriesen has quit IRC20:45
*** mvk has joined #openstack-nova20:45
arvindn0_but the original allocation happens against nic1 and since in rebuild the original allocations are not changed, we have wrong allocations20:45
arvindn0_mriedem: is the above scenario an issue if we use GET /resource_providers/{rp_uuid}/traits approch?20:46
mriedemarvindn0_: did you see efried's reply in the mailing list?20:49
*** dpawlik has quit IRC20:49
*** hemna_ has quit IRC20:49
*** dpawlik has joined #openstack-nova20:50
mriedemeven with efried's suggestion, that scenario is likely a gap yes20:50
arvindn0_mriedem: trying to figure out the mailing list...i only get digests....do you have a link?20:50
mriedemwe need to have some way of knowing, will a rebuild with a new image result in new allocations and if so, fail the rebuild20:50
mriedemyou can change your subscription to not be digests :)20:51
mriedemhttp://lists.openstack.org/pipermail/openstack-dev/2018-April/129734.html20:51
mriedemarvindn0_: also, what you're describing above is this bug https://bugs.launchpad.net/nova/+bug/176376620:51
openstackLaunchpad bug 1763766 in OpenStack Compute (nova) "nova needs to disallow topology changes on image rebuild" [Medium,Triaged]20:51
efriedmriedem, 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
mriedemefried: i'm not sure we do, we just have the root provider uuid in the scheduler filter20:52
mriedemunless we are going to build a ProviderTree object or something20:52
efriedmriedem: Do you have access to a SchedulerReportClient from wherever you are?20:52
mriedemsure20:53
efriedmriedem: If so, then yeah, you have that ability with a single call.20:53
*** hemna_ has joined #openstack-nova20:53
efriedmriedem: https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L986 would be that call.20:54
efriedThe 'ensure_root' part is slightly uncomfortable, but you know the root exists when you call this.20:54
efriedAnd we can factor that out of there if it's an issue.20:54
munimehanAll, 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 order20: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 IRC20:59
efriedarvindn0_: Do you have a resourcetracker?21:00
efriedarvindn0_: or a schedulerclient?21:00
arvindn0_nope...currently the filters make 0 api calls....so no clients of any sort21:01
*** tbachman has quit IRC21:01
efriedarvindn0_: 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
efriedarvindn0_: So yeah, you're not getting around that.  And those should be pretty quick calls, one would hope.21:02
efriedarvindn0_: The set logic around required vs forbidden will be a little tricky.21:02
*** gjayavelu has joined #openstack-nova21: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 node21:02
arvindn0_we dont support forbidden traits in images i think...21:03
efriedarvindn0_: 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
efriedarvindn0_: 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 traits21:06
efriedk21:06
*** pchavva has quit IRC21:06
efriedarvindn0_: 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 IRC21:08
efriedarvindn0_: Aha, actually, it's easier than I thought.21:08
efriedarvindn0_: 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 IRC21:09
*** fragatina has joined #openstack-nova21:09
*** fragatina has quit IRC21: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-nova21:11
efriedYes, you pass in your compute node UUID21:11
efriedYou'll get back a ProviderTree.21:11
efriedwait, hold on.21:11
arvindn0_efried: not too familiar with the provider tree object...my understanding provider tree will return all traits etc21:12
arvindn0_but not which RP is allocated to us...21:12
efriedyes it will.  For the whole tree, and associated sharing providers.  And you need ...21:12
efriedexactly.21:12
efriedarvindn0_: Now, we're focused on an instance here, right?21:12
efriedarvindn0_: Not the whole host.21:12
arvindn0_yup21:13
arvindn0_this is rebuild of that instance...so we need to know what resource was allocated with what traits...21:13
efriedSo 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 mapping21:14
efriedarvindn0_: That doesn't make sense.21:14
efriedarvindn0_: What do you mean by 1:1 mapping?21:14
efriedarvindn0_: 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
efriedarvindn0_: 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
efriedAnd I think the other .1% is an acceptable risk.21:15
*** tomtom001 has joined #openstack-nova21: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 scenario21:16
efriedarvindn0_: You can't do that with image traits, at all, sorry.21:17
*** cdent has joined #openstack-nova21:17
efriedarvindn0_: Because one thing we explicitly said in that spec review was that we can't correlate image traits with individual granular groups.21:17
efriedarvindn0_: 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 traits21:18
efriedarvindn0_: 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
efriedarvindn0_: 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
efriedyes.  Use the instance UUID as the consumer_uuid.21:20
efriedarvindn0_: Email followup coming...21:24
openstackgerritJulia Kreger proposed openstack/nova master: DNM: Test: Ironic: Lock the instance if plug vifs  https://review.openstack.org/56371421:25
arvindn0_efried: thanks21:25
arvindn0_efried: wait...another potential issue21:27
efriedarvindn0_: http://lists.openstack.org/pipermail/openstack-dev/2018-April/129740.html21:27
efriedoh.  Too late :)21:27
arvindn0_efried: nvm...it should work21:27
efriedarvindn0_: 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 fail21:27
efriedexactly.21:28
openstackgerritMichael Still proposed openstack/nova master: Move image conversion to privsep.  https://review.openstack.org/55443721:28
openstackgerritMichael Still proposed openstack/nova master: We don't need utils.trycmd any more.  https://review.openstack.org/55443921:28
openstackgerritMichael Still proposed openstack/nova master: We no longer need rootwrap.  https://review.openstack.org/55443821:28
arvindn0_efried: you think fast :) glad i could catch up in IRC21:30
openstackgerritMichael Still proposed openstack/nova master: Move IVS VIF plugging into the new utilities module.  https://review.openstack.org/55755221:30
openstackgerritMichael Still proposed openstack/nova master: Move create_tap_dev to the new utility module.  https://review.openstack.org/55783521:30
openstackgerritMichael Still proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module.  https://review.openstack.org/55848721:30
*** dougshelley66 has quit IRC21:31
*** awaugama has quit IRC21:31
*** dklyle has quit IRC21:31
*** cdent has quit IRC21: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-nova21:35
*** r-daneel has quit IRC21:36
*** r-daneel_ is now known as r-daneel21:36
efriedarvindn0_: 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
efriedI should abandon that...21:36
arvindn0_cool...thanks the history helps so i dont spend time exploring that possibility21:37
efriednod21:38
*** munimehan has quit IRC21:39
efriedarvindn0_: 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
efriedarvindn0_: ...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 IRC21: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 yes21:42
efriedgotcha21:43
efriedWell, I leave it in your capable hands.  Let me know if further questions.  Good luck!21:43
*** slaweq has quit IRC21:45
*** tbachman has joined #openstack-nova21:46
arvindn0_thanks!21:46
*** slaweq has joined #openstack-nova21:46
*** dpawlik has quit IRC21:47
*** tbachman_ has joined #openstack-nova21:48
*** tbachman has quit IRC21:50
*** tbachman_ is now known as tbachman21:50
*** slaweq has quit IRC21:52
*** yamamoto has joined #openstack-nova22:02
*** yamamoto has quit IRC22:07
*** Guest77979 has quit IRC22:14
*** gjayavelu has quit IRC22:17
*** gjayavelu has joined #openstack-nova22:18
*** dougshelley66 has joined #openstack-nova22:18
*** lbragstad has quit IRC22:18
*** esberglu has quit IRC22:18
*** esberglu has joined #openstack-nova22:19
*** esberglu has quit IRC22:21
*** mlavalle has quit IRC22:22
*** arvindn0_ has quit IRC22:23
*** gjayavelu has quit IRC22:25
*** dklyle has joined #openstack-nova22:26
*** rcernin has joined #openstack-nova22:31
*** sdague has quit IRC22:37
*** dklyle has quit IRC22:39
*** threestrands has joined #openstack-nova22:41
*** threestrands has quit IRC22:41
*** threestrands has joined #openstack-nova22:41
*** felipemonteiro__ has joined #openstack-nova22:42
*** efried has quit IRC22:42
*** edmondsw has quit IRC22:43
*** gjayavelu has joined #openstack-nova22:59
*** yamamoto has joined #openstack-nova23:04
*** felipemonteiro__ has quit IRC23:07
*** yamamoto has quit IRC23:10
openstackgerritMerged openstack/nova stable/pike: libvirt: disconnect volume from host during detach  https://review.openstack.org/56069023:19
*** r-daneel has quit IRC23:21
*** yamamoto has joined #openstack-nova23:26
*** hongbin_ has quit IRC23:27
openstackgerritZack Cornelius proposed openstack/nova-specs master: Libvirt file backed memory  https://review.openstack.org/56370423:46
*** cfriesen has joined #openstack-nova23:49
*** Tom-Tom has joined #openstack-nova23:50
*** artom has joined #openstack-nova23:53
*** Tom-Tom has quit IRC23:54
*** takashin has joined #openstack-nova23:55

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!