| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-09 | |||
| 16:06:33 | sean-k-mooney | https://github.com/openstack/nova/blob/c5f3d3b73256ff0d31e1c1a972909228287c3f64/nova/virt/libvirt/utils.py#L51 | |
| 16:07:08 | bauzas | sean-k-mooney: tbc, I think cpu flags are considered with more cautiousness than mdev types, y'know | |
| 16:07:52 | sean-k-mooney | bauzas: maybe but if they ever change the mdev type we are already screwed | |
| 16:08:05 | sean-k-mooney | the existing vms wil not be able to boot | |
| 16:08:34 | bauzas | yeah maybe I'm overthinking it | |
| 16:09:04 | bauzas | but at least having some way to prevent a possible API trait explosion in nova would be nice (and that was drafted in the spec likewise) | |
| 16:10:04 | sean-k-mooney | bauzas: but didnt you hear plamcent and traits will solve all problems :) | |
| 16:10:41 | bauzas | this alleviates some problems but raises other concerns, I'd politically say :-) | |
| 16:16:48 | sean-k-mooney | dansmith: thanks for the review on the cyborg stuff | |
| 16:16:59 | sean-k-mooney | i responded to your comments | |
| 16:17:05 | sean-k-mooney | also is nova meeting now | |
| 16:17:18 | dansmith | ack | |
| 16:17:19 | sean-k-mooney | yes ill go join that | |
| 16:20:51 | bauzas | sean-k-mooney: you mean the warning that was filling the logs ? | |
| 16:20:58 | bauzas | (re: policy and olso) | |
| 16:21:03 | bauzas | oslo* | |
| 16:21:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Change UEFI check to handle AArch64 better https://review.opendev.org/714311 | |
| 16:22:45 | sean-k-mooney | bauzas: yes | |
| 16:23:20 | sean-k-mooney | bauzas: i think we have stop it by disabling it entirly a temp messure but gmann is altering oslo to do it properly | |
| 16:23:35 | bauzas | k, thanks for the heads up | |
| 16:23:43 | sean-k-mooney | so if you manually set it you will get the deprecation warning but not for the defualts | |
| 16:24:25 | gmann | bauzas: sean-k-mooney this is nova change to adopt the new flag, waiting for new version of oslo.policy - https://review.opendev.org/#/c/717884/ | |
| 16:25:17 | gmann | its working fine so once we have olso release then i will update the lower constraint and remvoe WIP | |
| 16:26:18 | sean-k-mooney | cool. the cyborg patch seres is like 100 commits behind master so i still get the wall fo error if i touch nova manage | |
| 16:26:28 | sean-k-mooney | so it will be nice when that is all resovled | |
| 16:31:12 | gmann | lbragstad: on gate, somehow new flag is not reflecting due to oslo checkout etc but tested localyl and it worked fine - https://review.opendev.org/#/c/717943/2 | |
| 17:00:43 | bauzas | gibi: can you put some vote on https://review.opendev.org/#/c/715490/13 before you leave ? | |
| 17:00:55 | gibi | will do | |
| 17:01:00 | bauzas | you already +1d with comment saying you'd want to review the functest | |
| 17:01:04 | bauzas | thanks | |
| 17:01:08 | bauzas | (and I know this is late) | |
| 17:01:37 | bauzas | gibi: working at home btw. during the lockdown or back to the office ? | |
| 17:01:39 | gibi | I will do the cycle highlught patch anyhow | |
| 17:03:42 | gibi | bauzas: I'm home in the last 3 weeks | |
| 17:04:01 | gibi | or 4? I dont even remember | |
| 17:04:31 | bauzas | gibi: I'm facing my 24th day in paradise | |
| 17:04:46 | bauzas | 25th actually | |
| 17:05:10 | gibi | paradise, lol | |
| 17:05:38 | bauzas | I honestly and frankly enjoy this period | |
| 17:05:57 | bauzas | I don't have to taxi the kids 4 times a day | |
| 17:06:07 | bauzas | I can arrange my worktime like I want | |
| 17:06:28 | bauzas | and my wife is stuck with me and discovering remote work | |
| 17:06:35 | bauzas | what dare could I complain ? | |
| 17:08:32 | gmann | gibi: cycle highlights lines for policy, let me know if it need to be shorten, also feel free to rephrase if needed - http://paste.openstack.org/show/791896/ | |
| 17:09:19 | gibi | gmann: thanks a lot! looks good | |
| 17:09:36 | gmann | ok, thanks. | |
| 17:09:57 | gibi | bauzas: I have a fairly small flat in the middle of the capital. Now this place feels too small | |
| 17:11:25 | gibi | bauzas: I'm +2+A on the whole vgpu series | |
| 17:17:07 | gibi | cores: latest cycle highlights patch is up https://review.opendev.org/#/c/712498 | |
| 17:17:24 | gibi | bauzas: btw, do you want to add a highlight about the vgpu work? | |
| 17:17:38 | bauzas | gibi: thanks | |
| 17:17:54 | bauzas | gibi: and nope for the highlights, it's a minor thing | |
| 17:18:04 | gibi | bauzas: ack. I just wanted to double check | |
| 17:19:40 | gibi | I think this is it for me today. I will check the gate tomorrow but will not work much | |
| 17:23:32 | gmann | gibi: sent the FFE request on ML- http://lists.openstack.org/pipermail/openstack-discuss/2020-April/014027.html | |
| 17:33:07 | bauzas | gibi: \o and I'll handover your work | |
| 18:07:22 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM testing enforce new defaults flag https://review.opendev.org/717945 | |
| 18:08:16 | gmann | lbragstad: melwitt now this ^^ can test the new flag for defaults switch. flag needs to be set before load_rules() which happens during setup of test.py | |
| 18:08:51 | melwitt | thanks | |
| 18:08:58 | gmann | previously i was setting it after load_rules called and old defaults already as OrCheck. | |
| 18:12:55 | melwitt | ah ok | |
| 18:13:28 | openstackgerrit | Merged openstack/nova master: api: Add support for new cyborg extra specs https://review.opendev.org/716222 | |
| 18:13:41 | openstackgerrit | Merged openstack/nova master: Pass the actual target in server diagnostics policy https://review.opendev.org/716810 | |
| 18:17:31 | stephenfin | gmann: You're going to address the issues with the policy patches that are failing at the gate, I assume? | |
| 18:17:56 | stephenfin | I started to fix some of them today but got distracted and probably won't have a chance to close it off now | |
| 18:18:03 | gmann | stephenfin: yeah. working on those | |
| 18:19:04 | gmann | stephenfin: np!. | |
| 18:19:18 | gmann | let me check all review if any query for you. | |
| 18:19:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add test coverage of existing shelve policies https://review.opendev.org/717539 | |
| 18:20:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Introduce scope_types in shelve server https://review.opendev.org/717571 | |
| 18:20:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add new default roles in shelve server policies https://review.opendev.org/717581 | |
| 18:22:43 | gmann | stephenfin: replied on this. is ok for you ? -https://review.opendev.org/#/c/718619/2/nova/policies/keypairs.py@26 | |
| 18:30:05 | openstackgerrit | sean mooney proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186 | |
| 18:35:51 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing keypairs policies https://review.opendev.org/718604 | |
| 18:36:10 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in keypairs https://review.opendev.org/718609 | |
| 18:36:24 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in keypairs policies https://review.opendev.org/718619 | |
| 18:36:36 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in keypairs policy https://review.opendev.org/718621 | |
| 19:06:07 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix servers policy for admin_or_owner https://review.opendev.org/718501 | |
| 19:07:02 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server policies https://review.opendev.org/718348 | |
| 19:07:18 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers Policies https://review.opendev.org/662968 | |
| 20:01:49 | openstackgerrit | Merged openstack/nova master: Provide the parent pGPU when creating a new vGPU https://review.opendev.org/715489 | |
| 20:01:58 | openstackgerrit | Merged openstack/nova master: libvirt: Calculate disk_over_committed for raw instances https://review.opendev.org/717037 | |
| 20:11:23 | larsks | Howdy. Has any encountered a situation where 'server list' shows a server, but any attempt to interact with that (server show <uuid>, etc) results in "No server with a name or ID of <uuid> exists"? In the database, the server has vm_state "error" and task_state "NULL". | |
| 20:11:32 | larsks | This is queens. | |
| 20:13:43 | melwitt | yes, let me look for a link | |
| 20:15:06 | melwitt | I think that's likely https://bugs.launchpad.net/nova/+bug/1784093 which was fixed in queens 17.0.11 upstream | |
| 20:15:07 | openstack | Launchpad bug 1784093 in OpenStack Compute (nova) stein "Build requests can be orphaned without instance mappings" [Medium,Fix committed] - Assigned to melanie witt (melwitt) | |
| 20:21:43 | larsks | melwitt: thanks, perfect! | |
| 20:29:06 | dansmith | that looks more like a missing instance mapping to me, | |
| 20:29:44 | dansmith | although potentially missing BR too I guess | |
| 20:29:45 | dansmith | or maybe that bug is about BRs, but the fix is instance mappings? | |
| 20:36:06 | melwitt | yeah, it is. it's a bit different. I think the fix is the same (atomic update of BR, IM, RS) | |
| 20:36:18 | dansmith | ack | |
| 20:36:24 | melwitt | but yes this is a missing IM | |
| 21:19:23 | melwitt | dansmith: ideas welcome on what to do to recover/cleanup when an instance mapping update fails due to DBError https://bugs.launchpad.net/nova/+bug/1871925 | |
| 21:19:24 | openstack | Launchpad bug 1871925 in OpenStack Compute (nova) "Server appears in "openstack server list" but "openstack server (show|delete|etc)" insists it doesn't exist" [Medium,Confirmed] | |
| 21:20:41 | melwitt | the "easy" solution is retry a few times and hope that does it | |
| 21:24:39 | bauzas | gibi: good call on rechecking changes for 2.86 and 2.87, will follow up on this | |
| 21:33:40 | melwitt | I think I'm gonna propose a retry first | |
| 22:24:50 | mriedem | there was a bug like that in novaclient once, related to markers and how novaclient did it's marker stuff when listing servers. since osc uses novaclient, depending on when that was fixed and the version of osc being used here it could be related. | |