Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-08-26
20:42:00 mnaser this is very new
20:42:08 sean-k-mooney mnaser: and we will be supporting it in the next major verions
20:42:15 mnaser gosh, this is exactly what we're doing
20:42:16 mnaser lol
20:42:17 sean-k-mooney altough proably as tech preview
20:42:29 mnaser sean-k-mooney: who should i reach out to lol
20:43:15 sean-k-mooney that is a good question i can follow up internally tommorow and ask
20:44:18 mnaser sean-k-mooney: let me know because it pretty much already just works here... so we're really doing the same thing essentially :)
20:44:19 sean-k-mooney mdbooth and mschuppert are involed with the nova part
20:44:37 mnaser i.e. the keystone todo is already done in our case, heh
20:45:12 sean-k-mooney do you have a fully contiarerised nova contol plane
20:46:54 sean-k-mooney oh you are actully doing it in opendev
20:46:57 mnaser sean-k-mooney: yes :)
20:47:01 sean-k-mooney nice
20:47:13 mnaser and comptues too, the only thing that runs on the physical nodes is openvswitch and libvirt
20:47:26 mnaser the plan is to move openvswitch to containers too because we rely on the kernel data path so that should be ok
20:47:52 mnaser and libvirt, planning to move towards the split daemon model and run only the kvm virt driver on the host (and i think even then that has a way of being containerized but not being killed)
20:47:54 sean-k-mooney kolla uses ovs and libvirt in continers
20:48:10 mnaser right but they have the advantage of managing a docker container and access to host
20:48:14 mnaser i only can do things via k8s api
20:48:37 sean-k-mooney right but you can run it with pid=host and as a deamonset
20:48:59 sean-k-mooney that way the qemu instance are not tied to the lifetime of the libvirt pod
20:49:02 mnaser well you could but there's like some tricky things to do with that in making sure the process doesnt disappear/etc/etc -- but yeah, not impossible!
20:49:27 sean-k-mooney they are doint this in the nova operator i think
20:49:48 sean-k-mooney they started with runnign the contoles in vms and only contianerisn the comptue nodes
20:50:00 sean-k-mooney they are currently working on the contol plane
20:50:05 mnaser yeah we just haven't gotten around that bit yet, but yeah, totally possible, we run nova-compute in containers too
20:50:18 sean-k-mooney nova-compute is trivial
20:50:25 sean-k-mooney libvirt is harder
20:51:01 mnaser yep
20:51:23 mnaser anyways, yeah, we should totally talk because we're probably doing the same thing..
20:51:31 sean-k-mooney yep
20:51:53 sean-k-mooney and as i said this will be supported as a deployment model going forward
20:52:03 sean-k-mooney in our downstream product
20:52:19 sean-k-mooney it was anouched at the redhat summit a few months ago
20:52:51 mnaser it's honestly the way to go :>
21:03:45 openstackgerrit Merged openstack/nova master: Ensure source compute is up when confirming a resize https://review.opendev.org/699291
#openstack-nova - 2020-08-27
02:03:15 openstackgerrit Wenping Song proposed openstack/nova master: Process exception of delete resource provider https://review.opendev.org/748339
02:40:14 openstackgerrit Brin Zhang proposed openstack/nova master: Cyborg evacuate support https://review.opendev.org/715326
03:48:03 openstackgerrit Harshavardhan Metla proposed openstack/nova master: Raising a deprecated warning in case of 403 https://review.opendev.org/735068
05:10:12 openstackgerrit Merged openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460
05:54:48 openstackgerrit Johannes Kulik proposed openstack/nova master: vmware: Handle exception in destroy with attached volumes https://review.opendev.org/738416
08:08:53 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: Ensure source compute is up when confirming a resize https://review.opendev.org/748369
08:32:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934
08:33:00 stephenfin gibi: Respun that to address the failing test /o\ ^
08:33:05 stephenfin so dumb :(
08:36:53 gibi no worries, I put my +2 back
08:37:07 stephenfin thanks
08:39:38 bauzas gibi: I want to provide the implementation for the routed networks spec, can I supersed https://review.opendev.org/#/c/656885/7 ?
08:39:52 bauzas I mean, not providing a new revision, rather a new change
08:43:53 gibi bauzas: sure
08:44:10 bauzas thanks
08:44:11 gibi sean-k-mooney: do you have ideas what happenening here https://bugs.launchpad.net/nova/+bug/1892361 ?
08:44:11 openstack Launchpad bug 1892361 in OpenStack Compute (nova) "SRIOV instance gets type-PF interface, libvirt kvm fails" [Undecided,New]
08:46:31 gibi it seems libvirt driver generates wrong xml for a PF dev
08:51:20 stephenfin gibi: btw, finally got around to this https://review.opendev.org/#/c/748377/
08:52:01 stephenfin It's the backport-candidate label change for Gerrit. I think that's how one does it. I've asked hberaud (who did it for oslo) for advice
08:52:03 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP compute: Only call _local_delete_cleanup when scheduled instance is found https://review.opendev.org/748380
08:53:21 lyarwood now to work out how to exercise that in a regression test
08:56:17 gibi stephenfin: ack, thanks
09:02:17 gibi artom, bauzas: there is a vGPU related bugs in the tracker https://bugs.launchpad.net/nova/+bug/1887380
09:02:17 openstack Launchpad bug 1887380 in OpenStack Compute (nova) "Attaching virtual GPU devices to guests in nova" [Undecided,New]
09:03:42 bauzas gibi: ack
09:21:58 openstackgerrit Vlad Gusev proposed openstack/nova stable/ussuri: docs: Resolve issue with deprecated extra specs https://review.opendev.org/748386
09:22:19 openstackgerrit Vlad Gusev proposed openstack/nova stable/ussuri: replace the "hide_hypervisor_id" to "hw:hide_hypervisor_id" https://review.opendev.org/747189
09:35:05 openstackgerrit Alexandre Arents proposed openstack/nova master: Add a lock to prevent race during detach/attach of interface https://review.opendev.org/747957
09:46:53 jsuchome Hi stephenfin, could we get https://review.opendev.org/#/c/574301/ merged? I think your ,ain concerns were adressed
10:01:22 stephenfin jsuchome: I think lyarwood might have accidentally regressed your changes from PS37 when updating the patch below it. That's why my -1 is still there
10:01:30 stephenfin the docs are still wrong here https://review.opendev.org/#/c/574301/39/nova/conf/glance.py
10:03:09 stephenfin jsuchome: Also, would it impact you if I asked for a new boolean config option to turn on rbd direct download instead of reviving 'allowed_direct_url_schemes'? The more I think about it, the less I like it /o\
10:03:30 stephenfin that would let you drop the following patch and finish this in one patch
10:05:09 gibi stephenfin, jsuchome: if it matters I can quickly re-review the patch(s) after such change
10:05:26 lyarwood stephenfin: did I?
10:05:33 lyarwood stephenfin: crap my bad sorry
10:06:02 stephenfin easily fixed
10:09:52 stephenfin I really do think the new option is better idea though. Much easier to discover and just better UX in general, IMHO. Looks like it'll have a minimal impact of the patch too
10:14:57 jsuchome I'm not exactly in favor of that ... I think reusing that old option was described as a way to go in the blueprint
10:20:04 stephenfin jsuchome: Right, but the spec was first proposed in Queens when that option wasn't yet deprecated. I realize the approved spec notes the deprecation and its impact, but is there a clear technical reason for undeprecating the option or was it simply inertia?
10:22:29 jsuchome I don't think there's technical reason, main reason is that it all takes so much time...
10:24:45 jsuchome I hear there are some users using this patch already so I would think it's nice to just keep the option the same, but that's just it. Maybe the inconsistency with the bp is not an issue, it just looks ugly in a different way
10:29:48 openstackgerrit Jiri Suchomel proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD https://review.opendev.org/574301
10:55:07 openstackgerrit Stephen Finucane proposed openstack/nova master: Introduce '[glance] enable_rbd_download' https://review.opendev.org/748411
10:55:18 stephenfin jsuchome, (lyarwood): ^
11:10:45 openstackgerrit Balazs Gibizer proposed openstack/nova master: Use absolute path during qemu img rebase https://review.opendev.org/739246
11:12:48 gibi stephenfin, lyarwood: rebased and fixed up ^^
11:18:55 tony_su stephenfin: gibi: thanks for your kind assistance for code review and all patches were merged today.
11:19:27 gibi tony_su: thanks for taking care of those patches. nicely done!
11:20:49 tony_su I will continue to take care of any remaining nit comments as a follow-up patch.
11:22:52 gibi thanks
11:37:22 jsuchome stephenfin: I understand it can be done and that the change is small (altough you see youreself that it is affecting the testsuite) ... it just means back to the step 1 with reviews after months getting it to the current state...
11:55:33 stephenfin jsuchome: Hardly. The idea is sound. Just a tweak to the config option
11:56:08 stephenfin and the change is to the new test, not to anything existing
11:56:43 stephenfin gibi: Done
12:05:01 gibi stephenfin: thanks
12:23:57 brinzhang_ stephenfin: hope you can review the cyborg evacuate support patch, it has +2 by gibi, https://review.opendev.org/#/c/715326/
12:24:09 brinzhang_ stephenfin, gibi: thanks
12:27:09 openstackgerrit Balazs Gibizer proposed openstack/nova master: Only unplug vif after the device is detached from libvirt https://review.opendev.org/744934
13:22:23 stephenfin brinzhang_: Done. I'm -1 but it's just a collection of mostly trivial stuff
13:29:32 brinzhang_ stephenfin: thanks, I will update tomorrow
13:30:16 brinzhang_ stephenfin: https://review.opendev.org/#/c/715326/22/nova/accelerator/cyborg.py@313 this function I would not like to change, and Iwant keep it in the cyborg file

Earlier   Later