Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-06-19
21:46:03 sean-k-mooney i suspcet this would need a short spec or maybe a specless blueprint
21:48:16 sean-k-mooney arjunbaindur: looking at the open blueprints no
21:48:37 sean-k-mooney so the feature you need is support for "multifunction pci device passthough"
21:49:17 sean-k-mooney if you want to file a blueprint for that and add the details of your specific usecase we could bring it up in the next meeting or dicsuss it here next week when more people are online
21:50:07 sean-k-mooney it has been asked for before and should not be too invasive to add but on the other hand not many people review the pci code so getting reviews might be the bigger stubleing block to addign this
21:51:06 sean-k-mooney its a cpablity i suspect cyborg will want eventually too so the libvirt driver work to supprot generintg the correct xml is likely to be need regardless of if we add the feature to novas pci pasthough
22:29:40 sean-k-mooney dansmith: it finally passed https://review.opendev.org/#/c/734184/ :)
22:31:04 sean-k-mooney dansmith: ah you modified devstack in the end https://review.opendev.org/#/c/736231
23:26:23 artom sean-k-mooney, if by some weirdness of circumstance you're still working: https://review.opendev.org/#/c/737052/2
23:32:41 sean-k-mooney im watching a c++ code review of command and conquer https://www.youtube.com/watch?v=Oee7gje-XRc&t=929s
23:33:02 sean-k-mooney but since my laptop is broken im using my work one
23:33:12 sean-k-mooney until i buy a new one
23:33:31 sean-k-mooney so ya you unabandonted that?
23:33:44 sean-k-mooney or is this a different fix
23:34:34 sean-k-mooney that is instead of https://review.opendev.org/#/c/737038/
23:36:34 sean-k-mooney artom: ah ok i see whats happening we are installing pacages via pip before devstack runs for crudini
23:36:43 sean-k-mooney so devstack has not allready installed pip
23:44:34 sean-k-mooney artom: so this is happing because of http://lists.openstack.org/pipermail/openstack-discuss/2020-June/015204.html
23:44:49 sean-k-mooney artom: so +2 from me the ci passes and the patch makes sense
23:46:32 artom I dunno what I expected
23:46:34 artom Thank you!
23:46:37 artom Also, you're a lunatic
23:46:51 sean-k-mooney for likeing c++
23:46:56 artom Sure, why not
23:47:15 sean-k-mooney or for having irc open on a different monitor while i watch youtube
23:48:05 artom That last part is almost reasonnable
23:48:16 dansmith sean-k-mooney: it didn't setup glance properly though, for some devstack reason I don't understand
23:48:24 dansmith so I'll have to work on that on monday
23:49:02 sean-k-mooney dansmith: ah ok i can take a look on monday too if you like the main issue is i dont know how to configure what you want to configure
23:49:10 sean-k-mooney dansmith: but let me know if i can help
23:49:32 dansmith sean-k-mooney: well, you can see what I've got in local.conf, and that didn't end up in glance-api.conf like it did before
23:49:47 dansmith but anyway, I can take a look and then come crawling to you, which'll make it even more sweet for you :)
23:50:00 sean-k-mooney :)
23:52:51 sean-k-mooney oh ya https://zuul.opendev.org/t/openstack/build/d083edcf35d649768506fbbf7dbbf64c/log/controller/logs/etc/glance/glance-api_conf.txt does not have the values but https://zuul.opendev.org/t/openstack/build/d083edcf35d649768506fbbf7dbbf64c/log/controller/logs/etc/glance/glance-image-import_conf.txt does
23:53:17 dansmith I gotta begin my weekend, later
23:53:25 sean-k-mooney dansmith: o/
23:59:04 sean-k-mooney dansmith: for what its worth i think your hitting a devstack bug since its in the local.conf correctly. so i think your zuul file is correct.
23:59:31 sean-k-mooney e.g. someitng in devstack is modifying the config after the post config phase which is a bug
23:59:39 sean-k-mooney just a guess
#openstack-nova - 2020-06-20
14:28:41 jcath hello, friends, is there a way to make kvm up with both vnc and spice? I try to add spice_listen_address in /etc/libvirt/qemu.conf, but it doesnt work
#openstack-nova - 2020-06-21
02:01:17 openstackgerrit Merged openstack/nova master: Add missing test bindep on openssl https://review.opendev.org/644238
06:46:47 openstackgerrit Merged openstack/nova master: Remove hooks https://review.opendev.org/728968
12:28:03 tbarron Is it possible via openstack/nova apis to do expose compute-host extra disks to guest VMs via somthing like pci-passthru?
#openstack-nova - 2020-06-22
05:56:45 openstackgerrit Harshavardhan Metla proposed openstack/nova master: Moved the quoted section https://review.opendev.org/737215
07:42:34 bauzas good morning Nova
07:52:00 gibi bauzas: good morning
08:38:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM: Test the state of VMware NSX 3pp CI https://review.opendev.org/734114
09:19:05 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Filter instances by tenant_id https://review.opendev.org/737241
09:20:51 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Filter instances by tenant_id https://review.opendev.org/737241
09:38:50 gibi bauzas: left feedback in https://review.opendev.org/#/c/733703
09:38:57 bauzas thanks
09:40:37 bauzas gibi: ack, seen your comments
09:40:43 bauzas gibi: honestly, it's a good question
09:40:58 bauzas I'm not opiniated but,
09:41:23 bauzas if we go with the neutron direction, I like it but I won't be able to provide the implementation I think :p
09:41:48 bauzas gibi: so, tbh, I like your concern
09:42:19 bauzas but maybe we should discuss it with some other folks :)
09:48:52 brinzhang gibi, bauzas: I submit a spec, but it has an error of docs. I cannot find which is wrong in line 22
09:49:03 brinzhang error: /home/zuul/src/opendev.org/openstack/nova-specs/doc/source/specs/victoria/approved/filter-instances-by-tenant-id.rst:22:Unknown target name: "1".
09:49:12 brinzhang https://review.opendev.org/#/c/737241
09:49:27 brinzhang can you fast check, where caused the error?
09:52:07 gibi bauzas: sure. this is why I said that put every neutron segment - aggregate related code in a single place in nova, so that later we can easily remove it, but I know this part is then becomes implementation detail
10:00:55 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Filter instances by tenant_id https://review.opendev.org/737241
10:19:21 gibi brinzhang: the latest PS worked for me locally
12:05:26 jcath friends, I try to use "openstack server add volume --device /dev/hdc instance vol-name" to attach a volume to the instance as an IDE device, but it always attach as a virtio device (qemu-kvm as hypervisor) . as I check nova/virt/libvirt/driver.py, it seems that the device name parameter is ignored... so How I can force to attach a volume on IDE bus? thanks!
12:06:17 stephenfin elod: Could you take a look at https://review.opendev.org/#/c/708617/ today?
12:12:19 elod stephenfin: yes, looking
12:12:24 stephenfin ta
13:14:54 tbarron Is it possible via openstack/nova apis to do expose compute-host extra disks to guest VMs via somthing like pci-passthru?
13:15:17 gibi dansmith, stephenfin, melwitt: sorry I was out Friday afternoon. Thanks for reviewing the image cache bugfix. However I don't like the fact that we are mixing the question of "is this on the same dev?" with the question "is this exists?"
13:15:45 tbarron This has been suggested as a way for kubernetes clusters running with nova vms to do software defined storage (like ceph) without indirection
13:15:56 sean-k-mooney tbarron: in general no. if you have nvme disk you can confiture them for pci pasthough but it expects stateless pci device
13:16:13 tbarron in the data path like when they use RBD backed nova ephemeral or cinder storage
13:16:20 sean-k-mooney so we will not correctly clean them when a vm is deleted and there data will not be copied if you mvoe the guest
13:16:40 sean-k-mooney tbarron: so basically today no
13:16:49 sean-k-mooney tbarron: there is no way to do that
13:16:54 stephenfin gibi: Yeah, I'm on the fence about that now too
13:17:08 stephenfin especially given the confusion it resulted in
13:17:24 stephenfin I don't know what dansmith and melwitt settled on when I left
13:17:29 tbarron sean-k-mooney: thanks, I already said "no" but thought I better check with those who really know :D
13:17:57 gibi stephenfin: I will try to come up with a better factored solution in the next PS
13:18:05 stephenfin ack
13:18:18 sean-k-mooney tbarron: the quickest way to enable something like that would be to write a cyborg dirver to manage disk on the host and then extend libivt to accpet disks form cyborg
13:21:17 tbarron sean-k-mooney: interesting, and that makes sense
13:22:59 sean-k-mooney tbarron: a disk is not really an acclearator but if you think of cyborg as a generic device managment service it think it fit. espacially when you consider that imaging or erasing a disk is basically the same as programing or reseting an fpga
13:23:14 sean-k-mooney just with less $$$ for the hardware
13:24:08 tbarron sean-k-mooney: yeah, it fits with the general expose-hardware-features to VMs direction, I get it
13:25:10 tbarron and is somewhat in tension with present compute-instances as abstractions idea, thouhh I say "tension" and not contradiction
13:26:10 tbarron we have a similar tension in storage where different backends have different capabilities and it's tricky figuring which to expose as abstractions
13:34:03 sean-k-mooney tbarron: well the tension is relived in a sense that it would be abstracted via a device-profile and we would should based on the aviablity of the resouce like any other
13:34:29 sean-k-mooney tbarron: cinder does nto quite workin this usecase since you dont have an ideal of a local only cinder backend
13:35:17 dansmith gibi: I commented
13:36:14 dansmith gibi: making it non-public and specific to "decide if we should report zero" addresses my original concern I guess, but I don't understand what the problem currently is
13:38:36 gibi dansmith: my problem that it makes a coupling between nova.virt.libvirt.imagebackend.Image.cache and nova.virt.libvirt.imagecache.ImageCacheManager.cache_dir_is_on_same_dev_as_instances_dir as the later assumes how the former will create the directory
13:39:39 dansmith gibi: cache_dir is a property of the imagecache no?
13:40:09 gibi for me the reasoning like "the directory does not exists therefore it occupies 0 space" is easier to accept than "the directory is on the same dev as it is not created but we know that when it is created it will be a call to mkdir that creates it on the same dev"
13:40:19 dansmith are you just saying that the behavior of creating the cache_dir if it doesn't exist is something in the libvirt code?
13:40:52 dansmith gibi: until the directory exists, the same exact thing is returned right? zero?
13:41:52 tbarron sean-k-mooney: ack, cinder volumes are intended to have a life-cycle independent of compute instances or compute instance hosts
13:41:55 dansmith once the directory exists, we'll report what we see, which will almost definitely be the same dev, but if not, we'll report the value according to how the directory is at that point

Earlier   Later