| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-01-26 | |||
| 13:57:11 | gibi | sean-k-mooney: yeah, os-trait releases always do that | |
| 13:57:16 | gibi | sean-k-mooney: ping me and I will approve the fix | |
| 13:57:23 | sean-k-mooney | will do | |
| 13:58:53 | sean-k-mooney | im just trying to see how gabbit tests are run | |
| 14:00:55 | sean-k-mooney | i wonder if there are any flags we can pass to https://github.com/cdent/gabbi | |
| 14:01:56 | sean-k-mooney | maybe -v https://github.com/cdent/gabbi/blob/main/gabbi/runner.py#L246-L250 i have never actuly run those | |
| 14:04:25 | gibi | sean-k-mooney: that would log the HTTP request / response information. I would need to look at the placement service logs | |
| 14:05:32 | 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:06:13 | sean-k-mooney | it would be nice to beable to use gabbit for debuging this way | |
| 14:06:23 | gibi | https://github.com/openstack/placement/blob/master/placement/tests/functional/fixtures/gabbits.py#L62-L71 | |
| 14:06:31 | gibi | I probably need to tweak ^^ | |
| 14:07:11 | sean-k-mooney | ah you found it i did not know where the gabit fixutres were but i proably shoudl have looked there | |
| 14:08:11 | sean-k-mooney | if the loggs were reddirected to standard out/error then they woudl be aviable on failure | |
| 14:08:49 | sean-k-mooney | https://github.com/openstack/placement/blob/master/placement/tests/functional/fixtures/capture.py#L38-L60 | |
| 14:09:14 | sean-k-mooney | so its using the null handeler | |
| 14:09:26 | sean-k-mooney | we shoudl updat that to log to standard out/error | |
| 14:09:49 | sean-k-mooney | or do that based on an enviornment varable | |
| 14:09:58 | sean-k-mooney | so do that when you do OS_DEBUG=1 | |
| 14:10:45 | sean-k-mooney | that way it wont explode the subunit output normally | |
| 14:25:39 | gibi | sean-k-mooney: thanks | |
| 14:32:01 | 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:53 | sean-k-mooney | gsantos: you cant today | |
| 14:33:10 | sean-k-mooney | gsantos: qemu and the kernel do not suport live migrating with mdevs currently | |
| 14:33:27 | sean-k-mooney | gsantos: that is changing but proably 6 months before upstream can support live migrating | |
| 14:33:53 | sean-k-mooney | then we can add support in nova | |
| 14:34:39 | 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:35:20 | 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:38 | sean-k-mooney | gsantos: so your only option is too use cold migration today | |
| 14:37:05 | 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:45:45 | 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:46:05 | sean-k-mooney | gibi ^ | |
| 14:46:48 | sean-k-mooney | actully this wont pass without the upperconstrait change but the upper constratit change can merge without this | |
| 14:46:53 | sean-k-mooney | how do we resolve that | |
| 14:47:27 | gibi | sean-k-mooney: in the past I did that like this https://review.opendev.org/c/openstack/placement/+/805830 | |
| 14:47:49 | gibi | so a single patch | |
| 14:48:13 | sean-k-mooney | that wont work or at least should not | |
| 14:48:39 | sean-k-mooney | it shoudl still get clamped by https://github.com/openstack/placement/blob/master/tox.ini#L17 | |
| 14:49:07 | gibi | it worked in the past, what changed? | |
| 14:49:20 | 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:32 | sean-k-mooney | gibi: we likely did not have placment functional in the requirements repo | |
| 14:49:43 | gibi | hm | |
| 14:51:05 | 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:52:39 | 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:53:47 | gibi | yepp we did not have placement functional job in the req repo last time | |
| 14:54:17 | gibi | so I think we need to 1) disable the test 2) merge the req bump 3) enable the test | |
| 14:54:26 | gibi | or change the test to e more forgiving | |
| 15:01:23 | gsantos | sean-k-mooney: just out of curiosity, since live migrating doesn't work with mdevs, would it work with pci-passthrough? | |
| 15:02:03 | sean-k-mooney | gsantos: no for the same reason | |
| 15:02:15 | sean-k-mooney | we have a hack for neutron sriov ports | |
| 15:02:42 | 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:03:02 | sean-k-mooney | gsantos: for nics thats kind of ok since you can do bonding to cover the gap | |
| 15:03:09 | sean-k-mooney | but for anything else its not really ok | |
| 15:04:18 | gsantos | ack thanks | |
| 15:04:32 | 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:05:23 | sean-k-mooney | we can see if there is a better long term solution. | |
| 15:06:02 | sean-k-mooney | perhaps addign a functional-next that uses uncapped requiremetn for the requiremts repo job or something like that | |
| 15:06:23 | gibi | sean-k-mooney: I'm OK with the comment-out solution | |
| 15:06:38 | gibi | sean-k-mooney: I think if we need to do this once per cycle then OK | |
| 15:06:57 | sean-k-mooney | well its every os-traits release so its multiple times | |
| 15:07:17 | gibi | yes, but in the last cycle we had one such release | |
| 15:07:18 | sean-k-mooney | ill see if we can do this more intelegently | |
| 15:07:26 | sean-k-mooney | fair | |
| 15:07:50 | sean-k-mooney | for now ill do the quick solution and then think about a clearer approch later | |
| 15:07:59 | gibi | ok | |
| 15:08:03 | gibi | thanks for taking care of it | |
| 15:09:13 | 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:16:56 | 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:56 | 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:54:13 | 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:19 | opendevreview | Balazs Gibizer proposed openstack/placement master: Enhance doc of _get_trees_with_traits https://review.opendev.org/c/openstack/placement/+/825780 | |
| 15:54:19 | 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:20 | 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:35 | 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:40 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add any-traits support for allocation candidates https://review.opendev.org/c/openstack/placement/+/826492 | |
| 15:54:45 | opendevreview | Balazs Gibizer proposed openstack/placement master: Remove unused compatibility code https://review.opendev.org/c/openstack/placement/+/826493 | |
| 15:58:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: run nova tests with any--traits placement feature https://review.opendev.org/c/openstack/nova/+/825914 | |
| 18:01:10 | opendevreview | Gustavo Santos proposed openstack/nova stable/wallaby: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821978 | |
| 18:05:50 | opendevreview | Gustavo Santos proposed openstack/nova stable/victoria: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821980 | |
| 18:07:11 | opendevreview | Gustavo Santos proposed openstack/nova stable/ussuri: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821987 | |
| 20:11:48 | lyarwood | Apologies in advance for the spam | |
| 20:12:05 | 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:05 | opendevreview | Lee Yarwood proposed openstack/nova master: image_meta: Add ephemeral encryption properties https://review.opendev.org/c/openstack/nova/+/760454 | |
| 20:12:06 | opendevreview | Lee Yarwood proposed openstack/nova master: BlockDeviceMapping: Add encryption fields https://review.opendev.org/c/openstack/nova/+/760453 | |
| 20:12:06 | opendevreview | Lee Yarwood proposed openstack/nova master: BlockDeviceMapping: Add is_local property https://review.opendev.org/c/openstack/nova/+/764485 | |
| 20:12:07 | 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:07 | opendevreview | Lee Yarwood proposed openstack/nova master: virt: Add ephemeral encryption flag https://review.opendev.org/c/openstack/nova/+/760455 | |
| 20:12:08 | opendevreview | Lee Yarwood proposed openstack/nova master: scheduler: Add an ephemeral encryption pre filter https://review.opendev.org/c/openstack/nova/+/760456 | |
| 20:12:09 | 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:09 | 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:11 | opendevreview | Lee Yarwood proposed openstack/nova master: block_device_info: Add swap to inline https://review.opendev.org/c/openstack/nova/+/826523 | |
| 20:12:11 | opendevreview | Lee Yarwood proposed openstack/nova master: libvirt: Improve creating images INFO log https://review.opendev.org/c/openstack/nova/+/826524 | |
| 20:12:13 | opendevreview | Lee Yarwood proposed openstack/nova master: libvirt: Remove defunct comment https://review.opendev.org/c/openstack/nova/+/826525 | |
| 20:12:13 | 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:15 | 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:15 | 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:17 | 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:17 | 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 | |
| #openstack-nova - 2022-01-27 | |||
| 10:10:03 | opendevreview | Frode Nordahl proposed openstack/nova master: Ensure binding profile keys added by Nova are cleared https://review.opendev.org/c/openstack/nova/+/826601 | |
| 10:41:42 | gibi | bauzas: can we land this? https://review.opendev.org/q/topic:bug%252F1952941 I'm getting pressure from downstream... | |
| 11:14:34 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Include pf mac and vf num in port updates https://review.opendev.org/c/openstack/nova/+/824833 | |