| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-01-26 | |||
| 11:31:07 | kashyap | Yep, that's what I meant | |
| 11:33:14 | 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:41 | kashyap | That's fine; following all backports takes up too much cognitive load (at least for me) | |
| 12:56:39 | 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. | |
| 13:29:09 | kashyap | plibeau2: Hi, I've responded to your question on qemu-devel | |
| 13:30:07 | 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:26 | kashyap | After that, any additions on top should be done separately. ("One logical change per patch.") | |
| 13:30:41 | kashyap | (I say it because we've identified a few more things on 'qemu-devel' list and on Gerrit review) | |
| 13:54:33 | gibi | does anybody know if and how the python logging can be enable during placement gabbi tests? | |
| 13:54:58 | sean-k-mooney | i assume you can export OS_DEBUG=1 | |
| 13:55:02 | sean-k-mooney | to get debug logging | |
| 13:55:09 | sean-k-mooney | on failure | |
| 13:55:31 | sean-k-mooney | i would have to look at how the logging fixture we use it implemented beyond that | |
| 13:55:44 | sean-k-mooney | are you looking for logs in the success case | |
| 13:56:36 | gibi | for gabbi tests there is no python logging output at all | |
| 13:56:43 | gibi | even if the test fails | |
| 13:56:47 | 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:55 | gibi | I tried OS_DEBUG without success | |
| 13:56:55 | sean-k-mooney | ill be pushing a ptach for that shortly | |
| 13:56:59 | sean-k-mooney | ack | |
| 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: disable traits count check to allow os-traits 2.7.0 https://review.opendev.org/c/openstack/placement/+/826486 | |
| 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: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: Extend the RP tree DB query to support any-traits https://review.opendev.org/c/openstack/placement/+/825849 | |
| 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: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 | |