| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-07 | |||
| 16:27:11 | dansmith | was anything not working when we first tried to do this? | |
| 16:27:24 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/828570 | |
| 16:29:13 | chateaulav | what do you mena in regards to not working? | |
| 16:30:31 | dansmith | chateaulav: you added the ability to select the guest emulation mode right? when you added that, were other things broken that made that non-trivial? | |
| 16:30:52 | dansmith | or, how invasive was the change? it thought it was mostly just a flag | |
| 16:33:04 | chateaulav | yeah, so the main item is the meta property that lets you define the guest architecture | |
| 16:33:28 | chateaulav | everything else was mods to the various checks to account for reading that value along with the host arch | |
| 16:34:40 | kashyap | sean-k-mooney: I franky question the value of this "nova-emulation" job, given dansmith's comment on the impact. | |
| 16:34:57 | kashyap | Also who are the users for this? | |
| 16:35:27 | chateaulav | and then choosing the guest arch if it was defined. so the ci is just to ensure the emulation works. it is highly likely that changes to nova wont affect its functionality, because it follows the logical paths for the physical architecture support | |
| 16:35:50 | sean-k-mooney | kashyap: well chateaulav for one :) | |
| 16:36:08 | kashyap | Hmm, still | |
| 16:36:19 | kashyap | chateaulav: Also, please note: https://www.qemu.org/docs/master/system/security.html#non-virtualization-use-case | |
| 16:37:34 | sean-k-mooney | kashyap: they are aware. there are many production uscase for it even with that in mind | |
| 16:37:51 | dansmith | yeah, really seems pretty low-impact in terms of a feature, and a whole job on every change is very high cost | |
| 16:37:59 | sean-k-mooney | probly not public cloud | |
| 16:38:05 | dansmith | I tend to think that even a scenario in every job is more expensive than we need | |
| 16:38:15 | dansmith | a weekly periodic is fine if we want, but.. | |
| 16:38:48 | kashyap | Yeah, 30% impact on other nodes is just too much | |
| 16:39:05 | sean-k-mooney | well its not 30% from this job | |
| 16:39:08 | kashyap | sean-k-mooney: "many cases" - I'm assuming they don't give a hoot about security | |
| 16:39:13 | sean-k-mooney | we are loosing one of the providers i assume | |
| 16:39:39 | sean-k-mooney | kashyap: much of our downstream ci use qemu some uses kvm | |
| 16:40:17 | sean-k-mooney | so for ci, package building it think its fine | |
| 16:40:23 | kashyap | sean-k-mooney: Well, near as I know, most is exercising nested KVM | |
| 16:40:32 | kashyap | Internal CI is fine | |
| 16:40:58 | sean-k-mooney | dont forget that rackspace used to run there public cloud on power provideign x86 vms | |
| 16:41:32 | dansmith | um, what? that's news to me :) | |
| 16:41:56 | dansmith | I think they toyed with that, probably for second-source reasons but.. not to my knowledge for anything real | |
| 16:42:23 | sean-k-mooney | they used ot have xen but also ppc host | |
| 16:42:23 | dansmith | even still, that doesn't mean it makes sense, or is a good idea with qemu, and arm on x86 :) | |
| 16:43:00 | dansmith | yeah, probably for political reasons :) | |
| 16:43:11 | sean-k-mooney | perhaps | |
| 16:44:34 | chateaulav | yeah initial use of this is not meant to real-world systems. it is to bring testing and validation forward a little more so you dont have to run physical, and then work towards greater parity going forward | |
| 16:48:26 | kashyap | chateaulav: Okay, as long as you're clear that for any production usage this cross-arch emulation is entirely unfit. | |
| 16:49:28 | kashyap | Depending on the (cross-arch emulation) config, you still have _massive_ holes for a truck to comfortably drive through ;-) | |
| 16:50:06 | chateaulav | correct, this is entirely meant to bring security testing, validation testing, and providing simulated environments (which doesnt exist anywhere) to the common person within openstack | |
| 16:50:44 | dansmith | chateaulav: yeah, so it's cool if this is a toy, useful for developers or whatever, but that means the ci impact has to be negligible, IMHO | |
| 16:58:08 | chateaulav | dansmith: I was requested to add a ci in, so from the Nova Core Dev community perspective use it as you see fit. no need to waste ci time if it is exhuasting a lot of extra. I think i would be useful to have a periodic check to ensure that it remains functional; however, i can see it also being added to an existing ci as a scenario for the long term support of its testing | |
| 16:59:49 | dansmith | chateaulav: yeah, I understand, I'm not blaming you | |
| 17:00:24 | chateaulav | for sure, just want to make sure you understand our overall intent for this feature as a whole | |
| 17:23:45 | bauzas | chateaulav: dansmith: fwiw, I explain in the prelude that this is experimental and not tested in our CI | |
| 17:23:53 | bauzas | not false promises | |
| 17:23:56 | bauzas | no* | |
| 17:24:25 | bauzas | plus in the cycle highlights, hoping the marketing folks don't freak out and write something wrong | |
| 17:25:00 | dansmith | bauzas: okay but it is tested in our ci, has already broken for me this morning, and is costing a fair bit in terms of resource | |
| 17:25:15 | dansmith | but if you mean to describe it that way (and make the job reflect that) then ++ | |
| 17:25:45 | bauzas | I'm just testing the prelude as I write, so I'll upload it | |
| 17:25:54 | bauzas | heh, done | |
| 17:26:01 | bauzas | uploading it so reviews are welcome | |
| 17:26:12 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add the Yoga prelude section https://review.opendev.org/c/openstack/nova/+/832292 | |
| 17:26:15 | bauzas | dansmith: gibi: sean-k-mooney: gmann: ^ | |
| 17:27:15 | gibi | bauzas: ack, I will look at it tomorrow morning | |
| 17:27:32 | gmann | bauzas: thanks. will check in my after noon | |
| 19:28:33 | sean-k-mooney | gibi: my func test now show that the device are claimed and freed porperly i had an off by on error | |
| 19:30:39 | sean-k-mooney | claiming a vdpa device decremets the total count by 2 | |
| 19:30:47 | sean-k-mooney | 1 for the vdpa device and 1 for the pf | |
| 20:36:46 | opendevreview | Merged openstack/nova master: Add grenade-skip-level irrelevant-files config https://review.opendev.org/c/openstack/nova/+/831229 | |
| 20:46:08 | opendevreview | sean mooney proposed openstack/nova master: [WIP] add fun tests for VDPA operations that should work. https://review.opendev.org/c/openstack/nova/+/832330 | |
| #openstack-nova - 2022-03-08 | |||
| 02:48:13 | opendevreview | norman shen proposed openstack/nova master: Narrow mdev uuid range https://review.opendev.org/c/openstack/nova/+/832489 | |
| 07:48:26 | bauzas | good morning Nova | |
| 07:48:37 | bauzas | a bit remotely working, so on and off for the morning | |
| 08:21:11 | rribaud__ | bauzas, good morning. | |
| 08:21:35 | bauzas | good morning | |
| 08:50:44 | kashyap | bauzas: How much remote can you become :D Bonjour, in any case | |
| 08:51:29 | bauzas | like I said in the internal chan, https://photos.app.goo.gl/jkMW5MmAjD11r8xP9 | |
| 09:13:49 | gibi | bauzas: that looks nice :) | |
| 09:20:48 | kashyap | bauzas: Nice! | |
| 10:06:02 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/830010 | |
| 10:16:23 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/828570 | |
| 12:31:10 | admin1 | if a migration fails to happen, do i debug the nova-compute or the nova-placement ? | |
| 12:32:06 | sean-k-mooney | a migration invovles all the nova componetns so it depens on how it failed | |
| 12:32:16 | sean-k-mooney | and when in the migration | |
| 12:34:01 | sean-k-mooney | bauzas: oh your picking up your new car congrats | |
| 13:47:46 | kashyap | admin1: Yeah, migration problems can also be in the virtualization layer too. So it depends on the precise error (whether it be from -compute, -api, or wherever in Nova) | |
| 13:50:53 | bauzas | sean-k-mooney: nah I already had it since 2 weeks, but I was there because I asked for a tow hitch :) | |
| 13:51:06 | sean-k-mooney | ah ok | |
| 13:51:11 | sean-k-mooney | happy with it so far | |
| 13:52:47 | erlon | @sean-k-mooney hey sean, can you revisit these patches when you have a chance? https://review.opendev.org/q/topic:bug%252F1944619 | |
| 13:53:11 | erlon | when is the cut date for RC1? | |
| 13:53:24 | erlon | would be nice to have those included | |
| 13:53:55 | sean-k-mooney | ya i can have a look bauzas its this thursday? | |
| 13:54:02 | sean-k-mooney | rc1 | |
| 13:54:28 | sean-k-mooney | Mar 07 - Mar 11 | |
| 13:54:39 | sean-k-mooney | so ya it will be thursday of this week | |
| 13:55:04 | erlon | great, let me know as soon as possible just in case I need to adjust something else | |
| 13:55:36 | erlon | but I believe I addressed all your concerns listed there | |
| 13:58:37 | sean-k-mooney | im reviewing them now, i added yoga-rc-potential to it and added that as an offical tag | |
| 13:58:57 | sean-k-mooney | not sure why that was not added to the offical tag list before | |
| 14:02:11 | yuval | Is there a weekly meeting now? | |
| 14:03:04 | sean-k-mooney | its in 2 hours i think | |
| 14:03:39 | sean-k-mooney | Regular MEETING TIME: Tuesdays 16:00 UTC (#openstack-nova on OFTC) | |
| 14:06:13 | AndroUser | Test1 | |
| 14:06:28 | yuval | Thanks sean-k-mooney | |
| 14:14:15 | sean-k-mooney | erlon: can you add a release not to the second patch | |
| 14:14:29 | sean-k-mooney | otherwise im +2 on the repoducere and +1 on the fix | |
| 14:15:03 | erlon | ok, give some | |
| 14:15:44 | sean-k-mooney | im happy to upgrade to +2 on the second patch once added so feel free to ping me when its up | |
| 15:00:34 | opendevreview | Erlon R. Cruz proposed openstack/nova master: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/815324 | |
| 15:03:44 | bauzas | yuval: sean-k-mooney: sorry was discussing with Uggla | |