| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-11 | |||
| 13:06:58 | zigo | Or is it a copy of the image meta? | |
| 13:07:23 | sean-k-mooney | a copy but i assume its uefi | |
| 13:09:39 | sean-k-mooney | you might be higing a kernel api change where some parmater changed for 1/0 to y/n | |
| 13:09:59 | sean-k-mooney | can you check the output of virsh domcapabilities | |
| 13:11:05 | zigo | https://paste.opendev.org/show/bEp66voE4chYd3lF3eme/ | |
| 13:11:10 | zigo | What am I looking for? | |
| 13:12:14 | zigo | The <os supported='yes'> bits? | |
| 13:12:22 | sean-k-mooney | <loader supported='yes'> | |
| 13:12:31 | sean-k-mooney | line 12 | |
| 13:12:51 | zigo | Is it supposed to be 0/1 instead ? | |
| 13:13:14 | sean-k-mooney | no i was thinkin of a change for secure boot | |
| 13:13:18 | sean-k-mooney | which is seperate | |
| 13:16:12 | opendevreview | Merged openstack/nova stable/train: Adapt websocketproxy tests for SimpleHTTPServer fix https://review.opendev.org/c/openstack/nova/+/866201 | |
| 13:18:09 | zigo | sean-k-mooney: That bit of code is supposed to remove the disk containing the NVRAM values, right? | |
| 13:18:54 | sean-k-mooney | its not really a disk its a file that is uesd to store the uefi firmware data | |
| 13:19:09 | zigo | Right, that's what I had in mind. | |
| 13:19:26 | sean-k-mooney | but yes its to tell libvirt that its oke to delete the domain | |
| 13:19:47 | sean-k-mooney | inclucding undefining the nvram | |
| 13:38:59 | zigo | According to this patch's patch header https://review.opendev.org/c/openstack/nova/+/621646/ the UEFI NVRAM variable store isn't preserved on stop/start, hard reboot, cold migration, resize and live migration... | |
| 13:39:07 | zigo | Is it still the case? | |
| 13:41:17 | sean-k-mooney | that i am not sure about. i know that was a proablem in the past | |
| 13:42:17 | sean-k-mooney | i think this bug is still a thing | |
| 13:42:54 | sean-k-mooney | honestly i proably should escalte this internally too becasue this keeps getting stalled out | |
| 13:43:04 | sean-k-mooney | the people that started fixing it have moved on form openstack | |
| 14:38:03 | opendevreview | Merged openstack/nova master: Add mock to avoid loading guestfs in unit test https://review.opendev.org/c/openstack/nova/+/862769 | |
| 14:47:09 | opendevreview | Jorge San Emeterio proposed openstack/nova-specs master: Review usage of oslo-privsep library on Nova https://review.opendev.org/c/openstack/nova-specs/+/865432 | |
| 15:14:35 | dansmith | 2023-01-11 07:00:50,878 WARNING [oslo_messaging.rpc.client] Using RPCClient manually to instantiate client. Please use get_rpc_client to obtain an RPC client instance. | |
| 15:14:47 | dansmith | thousands of those in each run now.. I guess something in o.msg changed | |
| 15:24:00 | gibi | dansmith: I think https://review.opendev.org/c/openstack/requirements/+/869340 pulled in https://review.opendev.org/c/openstack/oslo.messaging/+/862419 that has the new warning | |
| 15:25:35 | opendevreview | ribaudr proposed 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 | |
| 15:31:15 | opendevreview | ribaudr proposed 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 | |
| 15:47:49 | gmann | dansmith: replied and updated the rbac default switch change, please check when you have time https://review.opendev.org/c/openstack/nova/+/866218 | |
| 15:48:48 | gmann | I have pushed devstack changes (depends-on) to keep running all existing jobs on old default and new jobs run with new defaults. After we do 2023.1 release we can switch it to run all existing jobs to new defaults and one job to run on old defaults | |
| 15:50:39 | dansmith | cool | |
| 17:58:35 | opendevreview | sean mooney proposed openstack/placement master: support multiple config files with apache https://review.opendev.org/c/openstack/placement/+/869863 | |
| 17:58:59 | sean-k-mooney | ^ is the placement version fo https://review.opendev.org/c/openstack/nova/+/867162 | |
| 20:12:31 | opendevreview | Dan Smith proposed openstack/nova master: Add virt/node module for stable uuids https://review.opendev.org/c/openstack/nova/+/863915 | |
| 20:12:31 | opendevreview | Dan Smith proposed openstack/nova master: Pass service ref to init_host(), if exists https://review.opendev.org/c/openstack/nova/+/863916 | |
| 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: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:33 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 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: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: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 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 | |
| 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 | |