| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-09 | |||
| 16:02:51 | sean-k-mooney | we do that in some places but not others | |
| 16:02:54 | bauzas | for managing libvirt versions (and even kernel versions) against trait names | |
| 16:03:13 | sean-k-mooney | bauzas: not really its considerdf part of the public api of the cpu | |
| 16:03:28 | sean-k-mooney | if they change it will break gcc and many many other things | |
| 16:06:31 | sean-k-mooney | bauzas: anyway for what its worth here is the cpu fetaru flag mapping table | |
| 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 | |