| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-03 | |||
| 09:04:59 | openstackgerrit | Merged openstack/nova master: Remove no-op fake calls from network_info tests https://review.opendev.org/749661 | |
| 09:05:43 | openstackgerrit | Brin Zhang proposed openstack/nova stable/ussuri: Add note and daxio version to the vPMEM document https://review.opendev.org/749681 | |
| 09:07:07 | openstackgerrit | Johannes Kulik proposed openstack/nova master: vmware: Use cookiejar from oslo.vmware client directly https://review.opendev.org/749682 | |
| 09:14:37 | openstackgerrit | Mamduh proposed openstack/os-vif master: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/746673 | |
| 09:33:16 | gibi | dansmith: hi! can we finalize documentation for the multistore spec? https://review.opendev.org/#/c/731577/ | |
| 09:33:57 | gibi | dansmith: I would like to mention that feature in the cycle highlights and link to the documentation | |
| 09:46:13 | gibi | stephenfin: could you hit this doc patch about rdb export https://review.opendev.org/#/c/728411 ? | |
| 09:46:28 | stephenfin | sure | |
| 09:46:42 | gibi | thanks | |
| 09:47:44 | lyarwood | gibi: that needs to be updated | |
| 09:48:15 | stephenfin | lyarwood: I'd say I've pointed out why? | |
| 09:48:22 | stephenfin | (wrong section for conf options) | |
| 09:50:54 | stephenfin | lyarwood: Got time to do the respin quickly while we're looking at it? I can do so if not | |
| 09:51:48 | lyarwood | stephenfin: yeah I can | |
| 09:52:46 | gibi | thanks ^^ | |
| 10:00:25 | openstackgerrit | Brin Zhang proposed openstack/nova stable/ussuri: Reject resize operation for accelerator https://review.opendev.org/749698 | |
| 10:02:11 | lyarwood | stephenfin: stupid question, but how would you list user defined things like rbd_user etc | |
| 10:02:27 | lyarwood | stephenfin: rbd_user=$rbd_user or rbd_user=foo etc? | |
| 10:02:33 | stephenfin | can you rephrase that? | |
| 10:02:34 | stephenfin | oh | |
| 10:02:53 | stephenfin | really up to you. I prefer dummy values that somewhat resemble reality | |
| 10:03:09 | stephenfin | http://example.com for URLs, 1.2.3.4 for IP addresses etc. | |
| 10:03:10 | lyarwood | meh I was thinking that could confuse people tbh | |
| 10:03:14 | lyarwood | kk | |
| 10:03:40 | stephenfin | Yeah, tbc that's just what I do. I don't know if there are recommendations in the OpenStack Style Guide or not | |
| 10:08:35 | openstackgerrit | Brin Zhang proposed openstack/nova stable/ussuri: Reject resize operation for accelerator https://review.opendev.org/749698 | |
| 10:10:53 | openstackgerrit | Yao wang proposed openstack/nova master: Use VIR_DOMAIN_XML_INACTIVE for detach volume https://review.opendev.org/749489 | |
| 10:11:26 | brinzhang | stephenfin: thanks, done ^ | |
| 10:14:06 | stephenfin | brinzhang: I think you need to abandon that and re-propose with the same change ID as the original | |
| 10:16:42 | brinzhang | stephenfin: ack, let me try | |
| 10:19:05 | stephenfin | alex_xu: Could I ask you to review https://review.opendev.org/#/c/747745/ with the expectation that I'll fix the live migration path also but in a separate, follow-up change? | |
| 10:20:14 | brinzhang | stephenfin: I cannot run git commit, if I run git commit -s, is it ok? | |
| 10:20:22 | brinzhang | root@ubuntu-OpenStack:/home/my_work/code/backup/nova# git commit | |
| 10:20:22 | brinzhang | Unova/compute/api.py | |
| 10:20:22 | brinzhang | Unova/tests/unit/compute/test_compute_api.py | |
| 10:20:22 | brinzhang | error: Committing is not possible because you have unmerged files. | |
| 10:20:22 | brinzhang | hint: Fix them up in the work tree, and then use 'git add/rm <file>' | |
| 10:20:22 | brinzhang | hint: as appropriate to mark resolution and make a commit. | |
| 10:20:24 | brinzhang | fatal: Exiting because of an unresolved conflict. | |
| 10:20:44 | stephenfin | oh, sorry, 'git add <changed files>' | |
| 10:20:45 | stephenfin | then commit | |
| 10:20:56 | brinzhang | ^ | |
| 10:21:35 | stephenfin | I think 'git cherry-pick --continue' will work instead of 'git commit', but that also needs the merge conflict marked as resolved using 'git add' or whatever | |
| 10:22:11 | openstackgerrit | Brin Zhang proposed openstack/nova stable/ussuri: Reject resize operation for accelerator https://review.opendev.org/749701 | |
| 10:22:49 | brinzhang | stephenfin: yes, we cannot run "git cherry-pick --continue", that will generate a new commit message | |
| 10:23:55 | brinzhang | and I tried many times in my evn, I think I get the way to cherry-pick if there is a conflict in the stable release | |
| 10:27:07 | brinzhang | stephenfin: good guideline, thanks ^^ | |
| 10:31:05 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Adapt the admin guide to describe the direct RBD export https://review.opendev.org/728411 | |
| 10:31:06 | lyarwood | stephenfin: ^ sorry had to deal with something downstream | |
| 10:32:10 | songwenping__ | bauzas: hi, my ubuntu env is ok now. i set two vgpu_types and vgpu_nvidia-180 and vgpu_nvidia-181 in nova-compute.conf, but i cant get vgpu_nvidia-180 and vgpu-nvidia-181 group. | |
| 10:36:38 | songwenping__ | i find the _get_supported_vgpu_types function is first called and then register_dynamic_opts is called. | |
| 10:38:25 | songwenping__ | so when i boot vm with vgpu flavor, i cant get device_addresses. | |
| 10:46:50 | songwenping__ | sean-k-mooney: hi, i reinstall vgpu-kvm.run package and restart my host, i can find nvidia-* at /sys/class/mdev_bus/0000\:af\:00.0/mdev_supported_types/. | |
| 11:10:18 | alex_xu | stephenfin: yea, will take a look at | |
| 11:24:10 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION and NEXT_MIN_{LIBVIRT,QEMU}_VERSION https://review.opendev.org/746981 | |
| 11:24:11 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_FILE_BACKED_DISCARD_VERSION https://review.opendev.org/746982 | |
| 11:24:12 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_NATIVE_TLS_VERSION https://review.opendev.org/746983 | |
| 11:24:12 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_BETTER_SIGKILL_HANDLING https://review.opendev.org/746984 | |
| 11:24:13 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_VIDEO_MODEL_VERSIONS https://review.opendev.org/746985 | |
| 11:24:13 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_PMEM_SUPPORT https://review.opendev.org/746986 | |
| 11:24:14 | openstackgerrit | Lee Yarwood proposed openstack/nova master: fakelibvirt: Use versionutils to set min versions found in the driver https://review.opendev.org/749707 | |
| 11:24:14 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_QEMU_VTPM https://review.opendev.org/749708 | |
| 11:59:07 | CeeMac | afternoon | |
| 11:59:44 | CeeMac | is it possible to 'retrofit' a qemu guest agent channel to a volume backed instance after deployment? | |
| 12:00:17 | CeeMac | if i dumpxml for the domain of an instance that has it enabled and working in can see the unix device with the sock binding etc | |
| 12:00:40 | CeeMac | this device is missing from the other instance that weren't deployed from an image with the guest agent metadata tag | |
| 12:00:59 | CeeMac | wasn't sure if there was a way to generate the sock and binding manually and update the xml / refresh the instance? | |
| 12:01:35 | CeeMac | I've tried adding the metadata item to the instance and power cycling but no dice | |
| 12:03:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add reproducer for bug #1894095 https://review.opendev.org/749713 | |
| 12:03:05 | openstack | bug 1894095 in OpenStack Compute (nova) "Running periodic task during live migration results in incorrect usage" [Medium,Confirmed] https://launchpad.net/bugs/1894095 - Assigned to Stephen Finucane (stephenfinucane) | |
| 12:03:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: Move live migration rollback under semaphore https://review.opendev.org/749714 | |
| 12:07:56 | stephenfin | CeeMac: There isn't any way to modify that kind of configuration without a rebuild (new image) or resize (new flavor), unfortunately | |
| 12:08:19 | stephenfin | and the latter isn't an option because, weirdly, there's no equivalent flavor extra spec for the image metadata property | |
| 12:08:25 | CeeMac | ah | |
| 12:08:36 | CeeMac | boo, i was about to jump on that as a workable option | |
| 12:08:46 | CeeMac | its an odd one | |
| 12:08:49 | stephenfin | Well I don't see any reason we couldn't add the latter | |
| 12:08:57 | stephenfin | though I'm not sure about backportability | |
| 12:09:23 | CeeMac | as just having the metadata on the image isn't sufficient for it to work on a volume backed image, the metadata also needs adding to the instance at point of deployment from what I can see | |
| 12:09:37 | stephenfin | if it was reasonable to do but not backportable, you'd have to carry that yourself/prod your vendor to carry the backport downstream | |
| 12:09:59 | CeeMac | sure | |
| 12:10:13 | CeeMac | thanks stephenfin very helpful, if not much help :D | |
| 12:10:30 | CeeMac | I figured this would be the case though, just wanted some validation I wasn't going mad | |
| 12:10:42 | CeeMac | now I know, so I'll need to work around it | |
| 12:11:34 | CeeMac | i'm still on rocky, so I'm not sure the backport option would be acceptable, its probably fairly niche as far as requirements go | |
| 12:30:21 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: compute: Validate a BDMs disk_bus when provided https://review.opendev.org/744552 | |
| 13:49:14 | dansmith | gibi: ++ for mentioning in the highlights, especially since a lot of cross-project collab has come out of it | |
| 13:49:47 | dansmith | gibi: I was kinda waiting for lyarwood to come back (which he did a while ago) and hopefully get some more meat in that doc about general rbd setup instructions and choices | |
| 13:50:15 | dansmith | gibi: since it's not as helpful to have a "so assume you already know a lot about a not-very-well-documented feature, here's some more details" doc | |
| 13:55:58 | lyarwood | dansmith: yeah sorry did you have a draft change somewhere for that? | |
| 13:56:05 | dansmith | lyarwood: yeah | |
| 13:56:13 | dansmith | lyarwood: https://review.opendev.org/#/c/731577/ | |
| 13:56:29 | lyarwood | dansmith: cool thanks, I'll try to sort something out shortly | |
| 13:56:45 | dansmith | lyarwood: thanks, sorry to play the "shame is a powerful motivator" card :P | |
| 13:57:34 | lyarwood | dansmith: np I said I would do this and totally forgot so it's valid :) | |
| 13:57:42 | artom | gibi, was a func test for https://review.opendev.org/#/c/748453/ ever discussed? | |
| 13:57:44 | dansmith | :) | |
| 13:59:41 | sean-k-mooney | artom: not for that but there are some func tests later in the series | |
| 13:59:52 | sean-k-mooney | in https://review.opendev.org/#/c/740995/14 | |
| 14:00:24 | CeeMac | stephenfin: I managed to find a workable solution! | |