| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-11 | |||
| 20:12:32 | opendevreview | Dan Smith proposed openstack/nova master: Make resource tracker use UUIDs instead of names https://review.opendev.org/c/openstack/nova/+/863919 | |
| 20:12:32 | opendevreview | Dan Smith proposed openstack/nova master: Add get_available_node_uuids() to virt driver https://review.opendev.org/c/openstack/nova/+/863917 | |
| 20:12:33 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 20:12:33 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 20:38:11 | opendevreview | Dan Smith proposed openstack/nova master: Make resource tracker use UUIDs instead of names https://review.opendev.org/c/openstack/nova/+/863919 | |
| 20:38:12 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 20:38:12 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 21:24:04 | opendevreview | Merged openstack/nova master: Support same host resize with PCI in placement https://review.opendev.org/c/openstack/nova/+/854441 | |
| #openstack-nova - 2023-01-12 | |||
| 05:06:05 | opendevreview | Merged openstack/nova master: Test reschedule with PCI in placement https://review.opendev.org/c/openstack/nova/+/854626 | |
| 05:12:25 | opendevreview | Merged openstack/nova master: Test multi create with PCI in placement https://review.opendev.org/c/openstack/nova/+/854663 | |
| 07:26:01 | opendevreview | Tobias Urdin proposed openstack/nova master: Use get_rpc_client helper from oslo.messaging https://review.opendev.org/c/openstack/nova/+/869900 | |
| 07:26:23 | tobias-urdin | gibi: dansmith ^ maybe that | |
| 08:42:13 | auniyal | Hi sean-k-mooney, gibi | |
| 08:42:23 | auniyal | how can I validate this bug - https://bugs.launchpad.net/nova/+bug/1996732 | |
| 08:42:39 | auniyal | like after my changes | |
| 08:46:40 | gibi | stephenfin: if you have time could you get back to this https://review.opendev.org/c/openstack/nova/+/854924 this is now the bottom of the PCI series and I fixed your comments | |
| 08:46:45 | gibi | tobias-urdin: added to my queue | |
| 08:48:34 | gibi | auniyal: I think you can trigger late affinity check failures in a func test by injecting a fault there | |
| 08:50:47 | gibi | and observing the effect of the BuildFailureWeigher can be done by crafting the settings of the existing weighers so that thd BuildFailureWeigher is overrepresented in the weighing result | |
| 08:54:32 | gibi | tobias-urdin: I left feedback there | |
| 08:54:39 | gibi | thanks for proposing a fix for this | |
| 08:56:47 | tobias-urdin | gibi: ack, thx! | |
| 09:10:26 | gibi | bauzas: I've approved the privsep refactor spec | |
| 09:10:34 | bauzas | thanks | |
| 09:10:44 | bauzas | I was looking into it | |
| 09:10:50 | bauzas | but I need to go to Uggla's spec | |
| 09:11:10 | gibi | I saw your +2 from before and I had a -1 that was answered so with sean-k-mooney +2 I felt we have corum | |
| 09:18:55 | opendevreview | Merged openstack/nova-specs master: Review usage of oslo-privsep library on Nova https://review.opendev.org/c/openstack/nova-specs/+/865432 | |
| 09:42:56 | opendevreview | Lukas Piwowarski proposed openstack/nova stable/yoga: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869912 | |
| 09:47:47 | opendevreview | Lukas Piwowarski proposed openstack/nova stable/xena: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869913 | |
| 09:51:15 | opendevreview | Lukas Piwowarski proposed openstack/nova stable/wallaby: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869914 | |
| 09:52:29 | opendevreview | Lukas Piwowarski proposed openstack/nova stable/victoria: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869915 | |
| 09:53:40 | opendevreview | Merged openstack/nova-specs master: Use extend volume completion action https://review.opendev.org/c/openstack/nova-specs/+/855490 | |
| 09:53:52 | opendevreview | Lukas Piwowarski proposed openstack/nova stable/ussuri: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869916 | |
| 09:55:08 | opendevreview | Lukas Piwowarski proposed openstack/nova stable/train: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869917 | |
| 10:38:56 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace usage of compareCPU() with compareHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/869950 | |
| 10:39:10 | kashyap | gibi: When you're about --^ | |
| 11:13:30 | auniyal | gibi, right now I have single node devstack setup in that, during this test sql-connection is getting failed at here - https://github.com/openstack/nova/blob/2eb358cdcec36fcfe5388ce6982d2961ca949d0a/nova/compute/manager.py#L1753 | |
| 11:13:53 | auniyal | I think its a env issue, so will be deploying new devstack | |
| 11:14:17 | auniyal | should I go with node have 2 compute node ? or single node would be enough ? | |
| 11:14:52 | auniyal | exact error I am getting is this - "oslo_messaging.rpc.client.RemoteError: Remote error: CantStartEngineError No sql_connection parameter is established" | |
| 12:16:15 | sean-k-mooney | gibi: regardign the hacking check for privsep | |
| 12:17:01 | sean-k-mooney | we dont have to stickly enforce "never import privladged fucntions" it might be nice to but as long as we are deliberate in keeping the contract narrow some limited shareing is proably ok | |
| 12:17:15 | sean-k-mooney | we just dont want to repate teh current issues we have with the shared ones. | |
| 12:18:13 | sean-k-mooney | i dont really know how easy it woudl be to do what you suggested in hacking but its not something im against trying | |
| 12:31:14 | gibi | sean-k-mooney: ack | |
| 12:31:22 | gibi | I agree that this is not a super imporant thing | |
| 12:35:40 | gibi | kashyap: I will check it | |
| 12:36:27 | gibi | auniyal: "No sql_connection parameter is established" means the configuration is incomplete. Yeah try to rebuild devstack | |
| 12:37:10 | kashyap | gibi: 🙏 | |
| 12:37:17 | gibi | auniyal: if you want to reproduce the problem in devstack first then you need two nodes as the BuildFailureWeigher won't fail a build but deproritize the affected compute | |
| 12:37:49 | gibi | auniyal: but I'm not sure how easy it is to fail the late affinity check in devstack without modifying the nova code itself | |
| 12:40:48 | sean-k-mooney | in genreally if you are wokring on schudler or move op related code its good to do at least some testign in a multi node devstack env but i agree with gibi that repoducing in a functional test is goign to be easier then repoducign in a devstack env | |
| 12:43:20 | gibi | kashyap: I'm OK with the direction of https://review.opendev.org/c/openstack/nova/+/869950 | |
| 12:43:51 | gibi | sean-k-mooney: this ^^ is now a patch that do a drop in replacement of the old compare cpu with a new compar hypervisor cpu, much cleaner than the past trials | |
| 12:45:21 | sean-k-mooney | ack that is what i was hoping for ill take a look | |
| 12:46:47 | kashyap | sean-k-mooney: gibi: Thanks! Yeah, it looks clean. I have also asked a Red Hat person to get it tested in a real env | |
| 12:47:08 | sean-k-mooney | this does not seam to be taking account fo the cpu_extra flags | |
| 12:47:17 | kashyap | gibi: Heh, about the wrapping, I did it locally and didn't just commit it | |
| 12:50:09 | gibi | sean-k-mooney: did the old code took account of it? | |
| 12:50:37 | gibi | if so the I think this will too | |
| 12:50:38 | sean-k-mooney | gibi: no which was a bug in the current implemenation | |
| 12:50:42 | gibi | ahh | |
| 12:57:38 | sean-k-mooney | i also dont think compareHypervisorCPU is correct | |
| 12:59:28 | opendevreview | Merged openstack/nova-specs master: Allow local scaphandre directory to be mapped to an instance using virtiofs https://review.opendev.org/c/openstack/nova-specs/+/861881 | |
| 13:16:09 | kashyap | sean-k-mooney: What do you mean by not correct? | |
| 13:16:40 | kashyap | gibi: sean-k-mooney: Again, the extra flags are handled elsewhere and will be considered "together" | |
| 13:17:27 | kashyap | Extra flags are handled here: _get_guest_cpu_model_config() | |
| 13:17:53 | kashyap | (gibi's reading is correc here.) | |
| 13:18:20 | sean-k-mooney | in that case that does not coralate with the know bug | |
| 13:18:48 | kashyap | sean-k-mooney: I gave the fix for the buggy environment to a Red Hat colleague. I'm waiting for that result too | |
| 13:21:27 | sean-k-mooney | kashyap: myu questions are basically related to the fixutre https://review.opendev.org/c/openstack/nova/+/869950/1/nova/tests/fixtures/libvirt.py | |
| 13:21:53 | sean-k-mooney | you seam to have change very little and its surpising that that is sufficent to emulate the new behavior | |
| 13:21:58 | sean-k-mooney | of the new api | |
| 13:22:35 | kashyap | sean-k-mooney: Exactly, I didn't change much of the fixture, as it is indeed a "drop-in" replacement and I don't think we should implement all the internal mechanics of libvirt | |
| 13:23:06 | kashyap | sean-k-mooney: Oh, we can also keep the vendor check there too. I don't think that'll affect it either | |
| 13:23:24 | sean-k-mooney | well then its identical to the ohter one :) | |
| 13:23:34 | sean-k-mooney | so we can just have one function call the other or alias thme | |
| 13:23:47 | kashyap | Yes :) | |
| 13:23:57 | kashyap | How do I alias? | |
| 13:24:01 | sean-k-mooney | have you any concenrs withthe delta in behavior of the fixture/real code | |
| 13:24:18 | sean-k-mooney | compareHypervisorCPU = compareCPU | |
| 13:24:49 | kashyap | I don't have any concerns. I'm just waiting for a real-life test to come back | |
| 13:25:16 | kashyap | Thanks for looking! | |
| 13:29:31 | sean-k-mooney | actually the alais wont work in this case | |
| 13:29:40 | sean-k-mooney | kashyap: the fucntion signiture is differnt | |
| 13:30:00 | kashyap | sean-k-mooney: Err, oh, right. Indeed | |
| 13:30:01 | sean-k-mooney | but you can jsut havc compareHypervisorCPU call compareCPU in the fixutre | |
| 13:30:37 | sean-k-mooney | assuming the vendor bit does not matter | |
| 13:30:56 | kashyap | Yep, it does not matter. I'll do the call and run the tests | |
| 13:49:06 | kashyap | sean-k-mooney: Good you raised the fixture point, I caught a few more buggy unit tests :) | |
| 13:51:29 | kashyap | Hmm, I fixed all but 2 of 'em seem a bit tricky; /me digs | |
| 14:09:11 | opendevreview | Tobias Urdin proposed openstack/nova master: Use get_rpc_client helper from oslo.messaging https://review.opendev.org/c/openstack/nova/+/869900 | |
| 14:20:23 | tobias-urdin | sean-k-mooney: weird question, if one implements a scheduler host filter the RequestSpec.image.properties is a ImageProps class, does that get populated for BFV instances? i.e would it read cached properties of image props for that volume and populate ImageProps or read existing image props for image volume spawned from an populate ImageProps? | |
| 14:20:56 | tobias-urdin | I'm guessing that is cached since it exists in RequestSpec from when it was spawned | |
| 14:29:12 | sean-k-mooney | tobias-urdin: yes its gets populated for BFV instances if the volume was created from a glance image it will have the image properties stored in the volume metadata and we also copy them into the nova databasein two locaionts | |
| 14:29:29 | sean-k-mooney | the request spec and teh instance_system_metadata in teh cell db | |
| 14:31:22 | sean-k-mooney | tobias-urdin: the request spec copy is used for schdulign to avoid needing to get the info out of the cell db | |
| 14:31:24 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py#L64 | |
| 14:32:21 | sean-k-mooney | the image filed is an nova imagemeta object which has the image properies embded in it https://github.com/openstack/nova/blob/master/nova/objects/image_meta.py#L75 | |