| *** hemna8 is now known as hemna | 07:37 | |
| kashyap | sean-k-mooney: I don't think we want to backport this to Xena/Wallaby, do we? - https://review.opendev.org/c/openstack/nova/+/798680 | 10:58 |
|---|---|---|
| kashyap | sean-k-mooney: Downstream distros may backport it for their own needs, but from an upstream PoV, not - because the releases were already out | 10:58 |
| *** Uggla|afk is now known as Uggla | 11:02 | |
| sean-k-mooney | no this is not a backport candiate upstream its a blueprint not a bug | 11:23 |
| sean-k-mooney | so by definition it cant be backported upstream | 11:23 |
| kashyap | Yeah, yeah...there's always potential exceptions (not in this case) :) | 11:25 |
| sean-k-mooney | kashyap: no i dont think we have broken that rule | 11:28 |
| sean-k-mooney | this would not be a candiate as you said | 11:28 |
| sean-k-mooney | but i dont think we have backported blueprints as an excption in the the past | 11:29 |
| sean-k-mooney | we have converted them to a bug premtivly when we agreed it shoudl be adressed as a bug but i dont think we have ever done it retro activly | 11:29 |
| kashyap | I think we definitely did at one point, I'm forgetting which one. Not obviously a huge feature, but a targetted one that was also helped w/ a security fix | 11:29 |
| sean-k-mooney | i guess a cve might be a valid reason to cirumvent our normal process if it could not be fixed any other way | 11:30 |
| kashyap | Yep, that's what I meant | 11:31 |
| sean-k-mooney | that specific case is not ringing a bell but i also dont follow all the backports we do as well as i proably should | 11:33 |
| kashyap | That's fine; following all backports takes up too much cognitive load (at least for me) | 11:33 |
| plibeau2 | hello, I have add some comment on my proposal: https://review.opendev.org/c/openstack/nova/+/803713 I'm available to discuss with you. QEMU community has reply also. | 12:56 |
| kashyap | plibeau2: Hi, I've responded to your question on qemu-devel | 13:29 |
| kashyap | plibeau2: I think we can at least do the first step here as a patch: moving the freeze call after end of disk mirroring. | 13:30 |
| kashyap | After that, any additions on top should be done separately. ("One logical change per patch.") | 13:30 |
| kashyap | (I say it because we've identified a few more things on 'qemu-devel' list and on Gerrit review) | 13:30 |
| * kashyap --> afk for a bit | 13:30 | |
| *** dasm|off is now known as dasm | 13:48 | |
| gibi | does anybody know if and how the python logging can be enable during placement gabbi tests? | 13:54 |
| sean-k-mooney | i assume you can export OS_DEBUG=1 | 13:54 |
| sean-k-mooney | to get debug logging | 13:55 |
| sean-k-mooney | on failure | 13:55 |
| sean-k-mooney | i would have to look at how the logging fixture we use it implemented beyond that | 13:55 |
| sean-k-mooney | are you looking for logs in the success case | 13:55 |
| gibi | for gabbi tests there is no python logging output at all | 13:56 |
| gibi | even if the test fails | 13:56 |
| sean-k-mooney | gibi: by the way i broke placment with the os-traits release as i need to update the number of expecte traits which was expected. | 13:56 |
| sean-k-mooney | ill be pushing a ptach for that shortly | 13:56 |
| gibi | I tried OS_DEBUG without success | 13:56 |
| sean-k-mooney | ack | 13:56 |
| gibi | sean-k-mooney: yeah, os-trait releases always do that | 13:57 |
| gibi | sean-k-mooney: ping me and I will approve the fix | 13:57 |
| sean-k-mooney | will do | 13:57 |
| sean-k-mooney | im just trying to see how gabbit tests are run | 13:58 |
| sean-k-mooney | i wonder if there are any flags we can pass to https://github.com/cdent/gabbi | 14:00 |
| sean-k-mooney | maybe -v https://github.com/cdent/gabbi/blob/main/gabbi/runner.py#L246-L250 i have never actuly run those | 14:01 |
| gibi | sean-k-mooney: that would log the HTTP request / response information. I would need to look at the placement service logs | 14:04 |
| sean-k-mooney | ack gabbit is spwaning a placment instance for each test right i wonder how logging is configured and if we are passing a config | 14:05 |
| sean-k-mooney | it would be nice to beable to use gabbit for debuging this way | 14:06 |
| gibi | https://github.com/openstack/placement/blob/master/placement/tests/functional/fixtures/gabbits.py#L62-L71 | 14:06 |
| gibi | I probably need to tweak ^^ | 14:06 |
| sean-k-mooney | ah you found it i did not know where the gabit fixutres were but i proably shoudl have looked there | 14:07 |
| sean-k-mooney | if the loggs were reddirected to standard out/error then they woudl be aviable on failure | 14:08 |
| sean-k-mooney | https://github.com/openstack/placement/blob/master/placement/tests/functional/fixtures/capture.py#L38-L60 | 14:08 |
| sean-k-mooney | so its using the null handeler | 14:09 |
| sean-k-mooney | we shoudl updat that to log to standard out/error | 14:09 |
| sean-k-mooney | or do that based on an enviornment varable | 14:09 |
| sean-k-mooney | so do that when you do OS_DEBUG=1 | 14:09 |
| sean-k-mooney | that way it wont explode the subunit output normally | 14:10 |
| gibi | sean-k-mooney: thanks | 14:25 |
| gsantos | Hey folks, I'm working on a few vGPU features and trying to find a way to live migrate guests with a vGPU/mdev attached, but stumbled upon a libvirt function that says migrating guests with non-USB devices is forbidden. Have any of you guys tried this in the past and can confirm if it is, in fact, not possible to be done? | 14:32 |
| sean-k-mooney | gsantos: you cant today | 14:32 |
| sean-k-mooney | gsantos: qemu and the kernel do not suport live migrating with mdevs currently | 14:33 |
| sean-k-mooney | gsantos: that is changing but proably 6 months before upstream can support live migrating | 14:33 |
| sean-k-mooney | then we can add support in nova | 14:33 |
| sean-k-mooney | speaking to people internally we are hoping that a new live migration capablity will be reported via sysfs which can then be reported by libvirt or quired by nova. | 14:34 |
| sean-k-mooney | both the dest and source host have a new enough kernel and qemu then nova could allow the migratoin to proceed but right now its not possibel to transfer the guest vgpu state as part of the live migration | 14:35 |
| sean-k-mooney | gsantos: so your only option is too use cold migration today | 14:35 |
| gsantos | sean-k-mooney: ah, I see. I was afraid that there would be no workarounds for it :( but thank you very much for the info | 14:37 |
| *** dansmith is now known as Guest661 | 14:38 | |
| opendevreview | sean mooney proposed openstack/placement master: update placement for os-traits 2.7.0 release https://review.opendev.org/c/openstack/placement/+/826478 | 14:45 |
| sean-k-mooney | gibi ^ | 14:46 |
| sean-k-mooney | actully this wont pass without the upperconstrait change but the upper constratit change can merge without this | 14:46 |
| sean-k-mooney | how do we resolve that | 14:46 |
| gibi | sean-k-mooney: in the past I did that like this https://review.opendev.org/c/openstack/placement/+/805830 | 14:47 |
| gibi | so a single patch | 14:47 |
| sean-k-mooney | that wont work or at least should not | 14:48 |
| sean-k-mooney | it shoudl still get clamped by https://github.com/openstack/placement/blob/master/tox.ini#L17 | 14:48 |
| gibi | it worked in the past, what changed? | 14:49 |
| sean-k-mooney | so if i raise it which i realise i should in requirements and lower-constratis the upper constraits will still clamp it | 14:49 |
| sean-k-mooney | gibi: we likely did not have placment functional in the requirements repo | 14:49 |
| gibi | hm | 14:49 |
| sean-k-mooney | ill make those chagnes quickly but i expect it to not be able to install the tox env when i do | 14:51 |
| opendevreview | sean mooney proposed openstack/placement master: update placement for os-traits 2.7.0 release https://review.opendev.org/c/openstack/placement/+/826478 | 14:52 |
| gibi | yepp we did not have placement functional job in the req repo last time | 14:53 |
| gibi | so I think we need to 1) disable the test 2) merge the req bump 3) enable the test | 14:54 |
| gibi | or change the test to e more forgiving | 14:54 |
| gsantos | sean-k-mooney: just out of curiosity, since live migrating doesn't work with mdevs, would it work with pci-passthrough? | 15:01 |
| sean-k-mooney | gsantos: no for the same reason | 15:02 |
| sean-k-mooney | we have a hack for neutron sriov ports | 15:02 |
| sean-k-mooney | where if ti addev via a macvtap devie we allow it or if its a direct assinged sriov port we will do a detach and attach | 15:02 |
| sean-k-mooney | gsantos: for nics thats kind of ok since you can do bonding to cover the gap | 15:03 |
| sean-k-mooney | but for anything else its not really ok | 15:03 |
| gsantos | ack thanks | 15:04 |
| sean-k-mooney | gibi: for now ill put a patch underneat to comment out the test, update the requiremetns patch to depend on it then make the placment patch depend on the requireemts patch and updte and uncomment the test | 15:04 |
| sean-k-mooney | we can see if there is a better long term solution. | 15:05 |
| sean-k-mooney | perhaps addign a functional-next that uses uncapped requiremetn for the requiremts repo job or something like that | 15:06 |
| gibi | sean-k-mooney: I'm OK with the comment-out solution | 15:06 |
| gibi | sean-k-mooney: I think if we need to do this once per cycle then OK | 15:06 |
| sean-k-mooney | well its every os-traits release so its multiple times | 15:06 |
| gibi | yes, but in the last cycle we had one such release | 15:07 |
| sean-k-mooney | ill see if we can do this more intelegently | 15:07 |
| sean-k-mooney | fair | 15:07 |
| sean-k-mooney | for now ill do the quick solution and then think about a clearer approch later | 15:07 |
| gibi | ok | 15:07 |
| gibi | thanks for taking care of it | 15:08 |
| sean-k-mooney | no worries. i just want to make sure that the inflight things are not blocked and didnt want to dump this on a new nova contributor to figure out | 15:09 |
| opendevreview | sean mooney proposed openstack/placement master: update placement for os-traits 2.7.0 release https://review.opendev.org/c/openstack/placement/+/826478 | 15:16 |
| opendevreview | sean mooney proposed openstack/placement master: disable traits count check to allow os-traits 2.7.0 https://review.opendev.org/c/openstack/placement/+/826486 | 15:16 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Extend the RP db query to support any-traits https://review.opendev.org/c/openstack/placement/+/825848 | 15:54 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Enhance doc of _get_trees_with_traits https://review.opendev.org/c/openstack/placement/+/825780 | 15:54 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Extend the RP tree DB query to support any-traits https://review.opendev.org/c/openstack/placement/+/825849 | 15:54 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: DB layer should only depend on trait id not names https://review.opendev.org/c/openstack/placement/+/826490 | 15:54 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Add any-traits support for listing resource providers https://review.opendev.org/c/openstack/placement/+/826491 | 15:54 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Add any-traits support for allocation candidates https://review.opendev.org/c/openstack/placement/+/826492 | 15:54 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Remove unused compatibility code https://review.opendev.org/c/openstack/placement/+/826493 | 15:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: run nova tests with any--traits placement feature https://review.opendev.org/c/openstack/nova/+/825914 | 15:58 |
| *** dansmith_ is now known as dansmith | 16:09 | |
| *** artom is now known as artom_not_the_sdk_one | 16:28 | |
| *** artom_not_the_sdk_one is now known as artom | 16:29 | |
| *** Uggla is now known as Uggla|afk | 17:38 | |
| opendevreview | Gustavo Santos proposed openstack/nova stable/wallaby: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821978 | 18:01 |
| opendevreview | Gustavo Santos proposed openstack/nova stable/victoria: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821980 | 18:05 |
| opendevreview | Gustavo Santos proposed openstack/nova stable/ussuri: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821987 | 18:07 |
| *** dansmith is now known as Guest688 | 19:01 | |
| *** Guest688 is now known as dansmith | 19:40 | |
| *** dansmith is now known as dansmith_ | 19:41 | |
| *** dansmith_ is now known as dansmith | 19:41 | |
| lyarwood | Apologies in advance for the spam | 20:11 |
| opendevreview | Lee Yarwood proposed openstack/nova master: func: Allow compute_driver to be set and used by _IntegratedTestBase https://review.opendev.org/c/openstack/nova/+/764484 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: image_meta: Add ephemeral encryption properties https://review.opendev.org/c/openstack/nova/+/760454 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: BlockDeviceMapping: Add encryption fields https://review.opendev.org/c/openstack/nova/+/760453 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: BlockDeviceMapping: Add is_local property https://review.opendev.org/c/openstack/nova/+/764485 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: compute: Update bdms with ephemeral encryption details when requested https://review.opendev.org/c/openstack/nova/+/764486 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: virt: Add ephemeral encryption flag https://review.opendev.org/c/openstack/nova/+/760455 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: scheduler: Add an ephemeral encryption pre filter https://review.opendev.org/c/openstack/nova/+/760456 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: blockinfo: Add encryption details to the disk_info mappings when provided https://review.opendev.org/c/openstack/nova/+/772272 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: WIP libvirt: Enable ephemeral encryption support for qcow2 https://review.opendev.org/c/openstack/nova/+/772273 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: block_device_info: Add swap to inline https://review.opendev.org/c/openstack/nova/+/826523 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: libvirt: Improve creating images INFO log https://review.opendev.org/c/openstack/nova/+/826524 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: libvirt: Remove defunct comment https://review.opendev.org/c/openstack/nova/+/826525 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: imagebackend: default by_name image_type to config correctly https://review.opendev.org/c/openstack/nova/+/826526 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: block_device: Add DriverImageBlockDevice to block_device_info https://review.opendev.org/c/openstack/nova/+/826527 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: block_device: Add encryption attributes to image and ephemeral disks https://review.opendev.org/c/openstack/nova/+/826528 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: virt: Add block_device_info helper to find encrypted disks https://review.opendev.org/c/openstack/nova/+/826529 | 20:12 |
| opendevreview | Lee Yarwood proposed openstack/nova master: imagebackend: Add disk_info_mapping as an optional attribute of Image https://review.opendev.org/c/openstack/nova/+/826530 | 20:12 |
| *** dasm is now known as dasm|off | 23:24 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!