| opendevreview | Merged openstack/nova master: Test reschedule with PCI in placement https://review.opendev.org/c/openstack/nova/+/854626 | 05:06 |
|---|---|---|
| opendevreview | Merged openstack/nova master: Test multi create with PCI in placement https://review.opendev.org/c/openstack/nova/+/854663 | 05:12 |
| 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 |
| tobias-urdin | gibi: dansmith ^ maybe that | 07:26 |
| auniyal | Hi sean-k-mooney, gibi | 08:42 |
| auniyal | how can I validate this bug - https://bugs.launchpad.net/nova/+bug/1996732 | 08:42 |
| auniyal | like after my changes | 08:42 |
| 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 |
| gibi | tobias-urdin: added to my queue | 08:46 |
| gibi | auniyal: I think you can trigger late affinity check failures in a func test by injecting a fault there | 08:48 |
| 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:50 |
| gibi | tobias-urdin: I left feedback there | 08:54 |
| gibi | thanks for proposing a fix for this | 08:54 |
| tobias-urdin | gibi: ack, thx! | 08:56 |
| gibi | bauzas: I've approved the privsep refactor spec | 09:10 |
| bauzas | thanks | 09:10 |
| bauzas | I was looking into it | 09:10 |
| bauzas | but I need to go to Uggla's spec | 09: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:11 |
| opendevreview | Merged openstack/nova-specs master: Review usage of oslo-privsep library on Nova https://review.opendev.org/c/openstack/nova-specs/+/865432 | 09:18 |
| opendevreview | Lukas Piwowarski proposed openstack/nova stable/yoga: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869912 | 09:42 |
| opendevreview | Lukas Piwowarski proposed openstack/nova stable/xena: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869913 | 09:47 |
| opendevreview | Lukas Piwowarski proposed openstack/nova stable/wallaby: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869914 | 09:51 |
| opendevreview | Lukas Piwowarski proposed openstack/nova stable/victoria: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869915 | 09:52 |
| opendevreview | Merged openstack/nova-specs master: Use extend volume completion action https://review.opendev.org/c/openstack/nova-specs/+/855490 | 09:53 |
| opendevreview | Lukas Piwowarski proposed openstack/nova stable/ussuri: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869916 | 09:53 |
| opendevreview | Lukas Piwowarski proposed openstack/nova stable/train: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/nova/+/869917 | 09:55 |
| opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace usage of compareCPU() with compareHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/869950 | 10:38 |
| kashyap | gibi: When you're about --^ | 10:39 |
| 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 |
| auniyal | I think its a env issue, so will be deploying new devstack | 11:13 |
| auniyal | should I go with node have 2 compute node ? or single node would be enough ? | 11:14 |
| auniyal | exact error I am getting is this - "oslo_messaging.rpc.client.RemoteError: Remote error: CantStartEngineError No sql_connection parameter is established" | 11:14 |
| *** labedz_ is now known as labedz | 11:29 | |
| sean-k-mooney | gibi: regardign the hacking check for privsep | 12:16 |
| 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 |
| sean-k-mooney | we just dont want to repate teh current issues we have with the shared ones. | 12:17 |
| 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:18 |
| gibi | sean-k-mooney: ack | 12:31 |
| gibi | I agree that this is not a super imporant thing | 12:31 |
| gibi | kashyap: I will check it | 12:35 |
| gibi | auniyal: "No sql_connection parameter is established" means the configuration is incomplete. Yeah try to rebuild devstack | 12:36 |
| kashyap | gibi: 🙏 | 12:37 |
| 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 |
| 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:37 |
| 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:40 |
| gibi | kashyap: I'm OK with the direction of https://review.opendev.org/c/openstack/nova/+/869950 | 12:43 |
| 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:43 |
| sean-k-mooney | ack that is what i was hoping for ill take a look | 12:45 |
| 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:46 |
| sean-k-mooney | this does not seam to be taking account fo the cpu_extra flags | 12:47 |
| kashyap | gibi: Heh, about the wrapping, I did it locally and didn't just commit it | 12:47 |
| gibi | sean-k-mooney: did the old code took account of it? | 12:50 |
| gibi | if so the I think this will too | 12:50 |
| sean-k-mooney | gibi: no which was a bug in the current implemenation | 12:50 |
| gibi | ahh | 12:50 |
| sean-k-mooney | i also dont think compareHypervisorCPU is correct | 12:57 |
| 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 | 12:59 |
| *** blarnath is now known as d34dh0r53 | 13:05 | |
| *** dasm|off is now known as dasm | 13:10 | |
| kashyap | sean-k-mooney: What do you mean by not correct? | 13:16 |
| kashyap | gibi: sean-k-mooney: Again, the extra flags are handled elsewhere and will be considered "together" | 13:16 |
| kashyap | Extra flags are handled here: _get_guest_cpu_model_config() | 13:17 |
| kashyap | (gibi's reading is correc here.) | 13:17 |
| sean-k-mooney | in that case that does not coralate with the know bug | 13:18 |
| 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:18 |
| 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 |
| sean-k-mooney | you seam to have change very little and its surpising that that is sufficent to emulate the new behavior | 13:21 |
| sean-k-mooney | of the new api | 13:21 |
| 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:22 |
| 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 |
| sean-k-mooney | well then its identical to the ohter one :) | 13:23 |
| sean-k-mooney | so we can just have one function call the other or alias thme | 13:23 |
| kashyap | Yes :) | 13:23 |
| kashyap | How do I alias? | 13:23 |
| sean-k-mooney | have you any concenrs withthe delta in behavior of the fixture/real code | 13:24 |
| sean-k-mooney | compareHypervisorCPU = compareCPU | 13:24 |
| kashyap | I don't have any concerns. I'm just waiting for a real-life test to come back | 13:24 |
| kashyap | Thanks for looking! | 13:25 |
| sean-k-mooney | actually the alais wont work in this case | 13:29 |
| sean-k-mooney | kashyap: the fucntion signiture is differnt | 13:29 |
| kashyap | sean-k-mooney: Err, oh, right. Indeed | 13:30 |
| sean-k-mooney | but you can jsut havc compareHypervisorCPU call compareCPU in the fixutre | 13:30 |
| sean-k-mooney | assuming the vendor bit does not matter | 13:30 |
| kashyap | Yep, it does not matter. I'll do the call and run the tests | 13:30 |
| kashyap | sean-k-mooney: Good you raised the fixture point, I caught a few more buggy unit tests :) | 13:49 |
| kashyap | Hmm, I fixed all but 2 of 'em seem a bit tricky; /me digs | 13:51 |
| opendevreview | Tobias Urdin proposed openstack/nova master: Use get_rpc_client helper from oslo.messaging https://review.opendev.org/c/openstack/nova/+/869900 | 14:09 |
| 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 |
| tobias-urdin | I'm guessing that is cached since it exists in RequestSpec from when it was spawned | 14:20 |
| 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 |
| sean-k-mooney | the request spec and teh instance_system_metadata in teh cell db | 14:29 |
| 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 |
| sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py#L64 | 14:31 |
| 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 | 14:32 |
| tobias-urdin | sean-k-mooney: thanks that good then it was like I expected :) as always, thanks for the detailed answer! | 14:35 |
| opendevreview | Sofia Enriquez proposed openstack/nova master: WIP: Implement encryption on backingStore https://review.opendev.org/c/openstack/nova/+/870012 | 14:45 |
| opendevreview | Sofia Enriquez proposed openstack/nova master: WIP: Implement encryption on backingStore https://review.opendev.org/c/openstack/nova/+/870012 | 14:50 |
| opendevreview | Tobias Urdin proposed openstack/nova master: Use get_rpc_client helper from oslo.messaging https://review.opendev.org/c/openstack/nova/+/869900 | 15:11 |
| opendevreview | Tobias Urdin proposed openstack/nova master: Use get_rpc_client helper from oslo.messaging https://review.opendev.org/c/openstack/nova/+/869900 | 15:12 |
| bauzas | shall I propose chatgpt for nova-core ? https://twitter.com/sylvainbauza/status/1613556643945672704 | 15:24 |
| kashyap | bauzas: :D | 15:24 |
| kashyap | bauzas: Please ask it to also fix my unit tests! | 15:24 |
| kashyap | (Or even better, just I submit a blueprint to it, write the rest of the code + docs + tests :P) | 15:24 |
| bauzas | kashyap: its trained data dates from 2021 | 15:24 |
| bauzas | kashyap: but you can ask it why your test is failing, I'm sure it will find you the bug | 15:25 |
| sean-k-mooney | how do you know im not chatgpt in disguise | 15:26 |
| sean-k-mooney | oh chatgpt can spell | 15:27 |
| sean-k-mooney | that a problem if i want to use it to automate my job in secret | 15:27 |
| sean-k-mooney | i should ask it if can respond in seanspeak | 15:27 |
| dansmith | hah | 15:27 |
| kashyap | sean-k-mooney: LOL | 15:41 |
| kashyap | sean-k-mooney: I'd totally believe if you're SeanGPT5 | 15:41 |
| kashyap | ! | 15:41 |
| tobias-urdin | hehe | 15:45 |
| bauzas | sean-k-mooney: hahaha | 15:47 |
| opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace usage of compareCPU() with compareHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/869950 | 15:49 |
| bauzas | sean-k-mooney: well, chatgpt sometimes says some wrong answers, but eventually it finds the right one | 15:49 |
| opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace usage of compareCPU() with compareHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/869950 | 15:51 |
| noonedeadpunk | hey folks! I was trying to find a way to get `failed_builds` counter through API but I'm not sure I udnerstand what provides ComputeNode object there. As it's not hypervisor, hosts are deperecated... | 15:57 |
| noonedeadpunk | it looks a bit like internal scheduler thing to me at this point... | 15:58 |
| noonedeadpunk | I mean specifically this thing https://opendev.org/openstack/nova/src/branch/master/nova/scheduler/host_manager.py#L261-L262 | 15:59 |
| noonedeadpunk | What I'm trying to do is monitor this counter to see when node falls out from scheduling and fix the issue/restart nova-compute to return it back | 16:01 |
| noonedeadpunk | I do see it's in cell database but still no idea how to fetch with api :( | 16:11 |
| noonedeadpunk | (in compute_nodes table) | 16:12 |
| opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace usage of compareCPU() with compareHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/869950 | 17:07 |
| kashyap | gibi: --^ Fixed your nits. Thanks for your continued reviews. | 17:10 |
| gmann | dansmith: did you get chance to review RBAC flag switch change https://review.opendev.org/c/openstack/nova/+/866218 | 17:52 |
| dansmith | gmann: I haven't since your update yet sorry | 17:52 |
| gmann | dansmith: ok, please check its depends-on also when you have time https://review.opendev.org/c/openstack/placement/+/869525 https://review.opendev.org/c/openstack/devstack/+/869781 | 17:53 |
| melwitt | sean-k-mooney: I'm +2 on the ironic vnc spec after my questions answered, if you want to send it https://review.opendev.org/c/openstack/nova-specs/+/863773 | 18:10 |
| sean-k-mooney | melwitt: i can i was more or less holding off for you to get your questions answered without interferring | 18:13 |
| sean-k-mooney | i know the responce form the previous iteration but i was away i was filling in context that was not in the spec | 18:13 |
| melwitt | sean-k-mooney: I know. wanted to let you know I'm good so you can go ahead | 18:13 |
| sean-k-mooney | so wanted ot let you get happy with the spec content without that bias | 18:13 |
| melwitt | I appreciate that :) | 18:13 |
| melwitt | I knew you were +2 already but I didn't feel comfortable +2ing until I understood it better | 18:14 |
| sean-k-mooney | hehe well i was almost -2 intilaly when it was storign password. with the pivot to addign the ironic vnc proxy that removed most of my concners and this just became a ironic dirver change to add partiy for an exisitn api | 18:17 |
| sean-k-mooney | the double proxying is a little odd but i see use in the ironic vnc proxy for standalone ironic too | 18:18 |
| melwitt | I was also pretty much -2 with the initial proposal. so I saw that was removed which was good but didn't quite understand where the second proxy fit into the flow | 18:19 |
| melwitt | *saw that password storage was removed | 18:20 |
| sean-k-mooney | ya so the current vnc proxy really just wraps the vnc tcp session provided by qemu in a web socket | 18:20 |
| sean-k-mooney | in the ironic case its more or less the same it will take the tcp session form the ironic proxy and convert it to a websocket for horizon | 18:21 |
| melwitt | yeah, I think it makes sense now but at first I was confused | 18:22 |
| opendevreview | Merged openstack/nova-specs master: new spec: support of vnc console for ironic https://review.opendev.org/c/openstack/nova-specs/+/863773 | 18:26 |
| gmann | sean-k-mooney: gibi: I know you might have this in your list but just a review reminder for placement RBAC change https://review.opendev.org/c/openstack/placement/+/865618 | 18:40 |
| sean-k-mooney | it is on it but i can try and give it more priority :) | 18:41 |
| sean-k-mooney | is this a depency for the nova patch to merge or was that something else | 18:41 |
| opendevreview | Merged openstack/nova master: Remove deleted projects from flavor access list https://review.opendev.org/c/openstack/nova/+/849131 | 18:50 |
| gmann | sean-k-mooney: that was something else. there is no deps. | 18:52 |
| sean-k-mooney | ok that was changing the jobs to enable srbac right | 18:52 |
| sean-k-mooney | we needed to do that for placment then nova | 18:52 |
| gmann | sean-k-mooney: yes | 18:52 |
| sean-k-mooney | ok and this is adding the service role | 18:53 |
| sean-k-mooney | and the other changes for the srbac role | 18:53 |
| sean-k-mooney | ok ill take a look shortly | 18:53 |
| gmann | sean-k-mooney: for placement it was a change in fixture which is needed for nova to enable new defaults, this one https://review.opendev.org/c/openstack/placement/+/869525 | 18:53 |
| gmann | sean-k-mooney: thanks | 18:53 |
| sean-k-mooney | ill look at that after so | 18:56 |
| opendevreview | Tobias Urdin proposed openstack/nova master: Use get_rpc_client helper from oslo.messaging https://review.opendev.org/c/openstack/nova/+/869900 | 19:24 |
| opendevreview | Tobias Urdin proposed openstack/nova master: Use new get_rpc_client API from oslo.messaging https://review.opendev.org/c/openstack/nova/+/869900 | 19:25 |
| opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | 21:20 |
| opendevreview | Dan Smith proposed openstack/nova master: Make resource tracker use UUIDs instead of names https://review.opendev.org/c/openstack/nova/+/863919 | 21:20 |
| opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | 21:21 |
| *** dasm is now known as dasm|off | 22:31 | |
| opendevreview | Merged openstack/nova master: Allow enabling PCI scheduling in Placement https://review.opendev.org/c/openstack/nova/+/854924 | 23:08 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!