| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-01-27 | |||
| 00:11:02 | sean-k-mooney | its defiend right now as project admin or system admin | |
| 00:11:21 | gmann | https://github.com/openstack/nova/blob/3a6c1cbc3a07814b3fecfdc23f28da9294779bcc/nova/policies/migrate_server.py#L27 | |
| 00:11:51 | gmann | scope_type both means it allow both scoped token but check_str is what controlling it | |
| 00:12:08 | sean-k-mooney | which is 'rule:system_admin_api | |
| 00:12:14 | gmann | with special string in check_str | |
| 00:12:29 | sean-k-mooney | which becomes rule:admin_api | |
| 00:13:00 | gmann | system_admin_api is 'role:admin and system_scope:all', | |
| 00:13:32 | sean-k-mooney | wher eis that defiend | |
| 00:13:42 | sean-k-mooney | i was looking at https://github.com/openstack/nova/blob/3a6c1cbc3a07814b3fecfdc23f28da9294779bcc/nova/policies/base.py#L16 | |
| 00:13:45 | gmann | https://github.com/openstack/nova/blob/3a6c1cbc3a07814b3fecfdc23f28da9294779bcc/nova/policies/base.py#L104 | |
| 00:14:04 | sean-k-mooney | oh here https://github.com/openstack/nova/blob/3a6c1cbc3a07814b3fecfdc23f28da9294779bcc/nova/policies/base.py#L103-L108 | |
| 00:14:04 | gmann | this one ^^ | |
| 00:14:26 | sean-k-mooney | see this is already a problem in that the definiton are different beteen nova and placment | |
| 00:14:57 | gmann | placement has already scope_type so current change for ADMIN-SYSTEM_ADMIN is not needed as such, that is my point on that review | |
| 00:15:35 | sean-k-mooney | ii know its not needed but i dont think we should have default RBAC personcs unless they are the same across all services | |
| 00:18:18 | gmann | ok for consistency if we want to have same default then I am ok. then we can do same in in aggregate API too https://review.opendev.org/c/openstack/placement/+/760235/10/placement/policies/aggregate.py#52 | |
| 00:18:32 | gmann | let's see what lbragstad and stephenfin prefer. | |
| 00:20:48 | sean-k-mooney | keystone has the same definiton for system reader as nova for what its worth | |
| 00:20:50 | sean-k-mooney | https://github.com/openstack/keystone/blob/a98f006f854be02e5682390012d8bb917f4f3940/keystone/common/policies/base.py#L48 | |
| 00:23:35 | gmann | like base rule we have in nova it is easy for operator to only override the 4-5 base rule only in policy file to make changes for all the policy instead of 200 rules override. that is why we defined thee as rule instead of string | |
| 00:24:04 | sean-k-mooney | so with my downstrem had on im in two minds | |
| 00:24:11 | sean-k-mooney | first we dont support custom policy | |
| 00:24:24 | sean-k-mooney | so our customer are not able to override any fo them | |
| 00:24:56 | sean-k-mooney | on the other had if they where i would have to check the policy difeintion when looking at api issue or suggesting what steps they could take for there given cloud | |
| 00:25:38 | sean-k-mooney | so while i understand its useful for operators to quickly redfien things in reality it will make debug ing much harder | |
| 00:26:03 | sean-k-mooney | e.g. if we have upstream bug reports or they are working with a vendor | |
| 00:26:30 | sean-k-mooney | the interop part of me is schreaming policy is config diriven api behavior | |
| 00:26:38 | sean-k-mooney | which is bad | |
| 00:27:12 | sean-k-mooney | its componded by the fact as a client i have no way of determining what i am alowed to do | |
| 00:27:22 | sean-k-mooney | there is no policy endpoin i can query to figure it out | |
| 00:27:55 | sean-k-mooney | so for me if we have default RBAC personas across multipel project they shoudl be imutable | |
| 00:28:52 | sean-k-mooney | we could provide a way to simpley replce SYSTEM_ADMIN with CUSTOM_SYSTEM_ADMIN via policy.yaml but i think that hsould be the excption rahter then what we expect people to do | |
| 00:29:29 | sean-k-mooney | lbragstad: gmann that is the main context around my view on this and why i think we should have shared constants for the personas | |
| 00:31:40 | gmann | sean-k-mooney: yeah i agree on common persona which provide much needed consistency but we need to see if those should be as constant or rule or rule-with-set-method to add deprecated rule | |
| 00:32:28 | gmann | may be best way is to proceed with constant string. that is one of the possible option we discussed in policy meeting last week | |
| 00:34:45 | brinzhang_ | nightmare_unreal> ack, add to my list, after done my things I will try to fill this issue | |
| 00:34:49 | brinzhang_ | gmann: thanks ^ | |
| 01:05:25 | openstackgerrit | Wenping Song proposed openstack/nova master: Replace all_tenants with all_projects in List Server APIs https://review.opendev.org/c/openstack/nova/+/765311 | |
| 01:05:26 | openstackgerrit | Wenping Song proposed openstack/nova master: Replaces tenant_id with project_id from Rebuild Server API https://review.opendev.org/c/openstack/nova/+/766380 | |
| 01:05:26 | openstackgerrit | Wenping Song proposed openstack/nova master: Replaces tenant_id with project_id from List SG API https://review.opendev.org/c/openstack/nova/+/766726 | |
| 01:18:19 | sapd1_x | bauzas, Hi, do we need RHEL for vGPU feature? in the supported matrix, they dont mention Ubuntu or CentOS (https://docs.nvidia.com/grid/latest/product-support-matrix/index.html) | |
| 01:19:40 | sapd1_x | we are running Ubuntu + KVM. | |
| 02:09:43 | mnaser | sean-k-mooney: sorry for the late reply, been dealing with a lot of stuff (move and puppy) | |
| 02:10:08 | mnaser | sean-k-mooney: we run it daily because we noticed when we don't, too many records accumualte and we end up with a near impossible to clean state | |
| 02:10:21 | mnaser | sapd1_x: it should work fine without rhel | |
| 02:14:49 | openstackgerrit | Merged openstack/nova master: libvirt: Drop support for UML https://review.opendev.org/c/openstack/nova/+/743230 | |
| 02:33:25 | lbragstad | sean-k-mooney gmann fwiw - i'm fine with constant strings somewhere if that helps move things along | |
| 02:35:45 | lbragstad | i agree having the common personas represented as objects would be ideal | |
| 07:28:57 | openstackgerrit | Wenping Song proposed openstack/os-traits master: remove babel.cfg https://review.opendev.org/c/openstack/os-traits/+/772634 | |
| 08:44:01 | MrClayPole | admin0 & lyarwood Thanks for both you help recently. We were able to trace it through to a possible issue with our storage cinder driver. | |
| 09:04:14 | lyarwood | MrClayPole: awesome, np! | |
| 09:28:17 | bauzas | sapd1_x: you're unfortunately absolutely correct | |
| 09:47:43 | openstackgerrit | Wenping Song proposed openstack/os-traits master: remove babel.cfg https://review.opendev.org/c/openstack/os-traits/+/772634 | |
| 10:25:50 | stephenfin | bauzas: If you have time this week, there are a couple of DB compaction patches that still need a final +2, starting here https://review.opendev.org/c/openstack/nova/+/758397/ | |
| 10:26:18 | stephenfin | Lower priority than gibi's series, but it would still be nice to close that out in Wallaby so I can work on getting alembic integrated (assuming we decide to do that) | |
| 10:27:49 | bauzas | stephenfin: ack, currently reviewing gibi's series | |
| 10:28:08 | bauzas | even if I have to look at https://review.opendev.org/c/openstack/nova/+/769942/ | |
| 10:28:13 | bauzas | mypy my love <3 | |
| 10:35:27 | stephenfin | bauzas: don't you dare diss mypy | |
| 10:36:06 | bauzas | I love cakes | |
| 10:36:12 | bauzas | but surely it's not mypy | |
| 10:36:20 | stephenfin | boo this man | |
| 10:36:22 | stephenfin | boooooo | |
| 10:37:05 | bauzas | emojis are for the others | |
| 13:13:33 | lyarwood | https://review.opendev.org/q/topic:bug/1909120 & https://review.opendev.org/q/topic:bug/1908075 could use reviews from cores if anyone has time in the coming weeks, simple enough bugfixes. | |
| 13:20:24 | lyarwood | elod / bauzas ; https://review.opendev.org/c/openstack/nova/+/764611 & https://review.opendev.org/c/openstack/nova/+/764612 - can I get stable reviews on these please? | |
| 13:20:50 | bauzas | ack, will try | |
| 13:22:36 | elod | lyarwood: looking | |
| 13:24:19 | lyarwood | many thanks both | |
| 13:26:31 | lyarwood | https://review.opendev.org/c/openstack/nova/+/752489/ and the series below it on stable/train is also ready | |
| 13:35:29 | openstackgerrit | Merged openstack/nova master: zuul: Allow encryption tests in nova-ceph-multistore again https://review.opendev.org/c/openstack/nova/+/771837 | |
| 13:47:28 | openstackgerrit | Merged openstack/nova master: libvirt: Drop support for Xen https://review.opendev.org/c/openstack/nova/+/743231 | |
| 13:58:48 | lyarwood | \o/ | |
| 14:08:06 | openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Move the LibvirtDistroSupportMatrix wiki page into our docs https://review.opendev.org/c/openstack/nova/+/771981 | |
| 14:49:12 | stephenfin | lyarwood: I have an open API policy question on https://review.opendev.org/c/openstack/nova/+/768352/ | |
| 14:53:12 | lyarwood | stephenfin: ah my bad, looking | |
| 14:53:17 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Increase SWIFT_LOOPBACK_DISK_SIZE within nova-lvm job https://review.opendev.org/c/openstack/nova/+/772702 | |
| 14:57:10 | openstackgerrit | Merged openstack/nova master: Refactor _claim_pci_device_for_interface_attach to prepare for qos https://review.opendev.org/c/openstack/nova/+/756895 | |
| 15:08:07 | dansmith | sean-k-mooney: gdi, I do that *every* dang time :/ | |
| 15:11:28 | openstackgerrit | Stephen Finucane proposed openstack/nova master: rpc: Rework 'get_notifier', 'wrap_exception' https://review.opendev.org/c/openstack/nova/+/741663 | |
| 15:11:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add type annotations to 'nova.pci' https://review.opendev.org/c/openstack/nova/+/676209 | |
| 15:11:32 | openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add nova.cmd, nova.conf, nova.console https://review.opendev.org/c/openstack/nova/+/705657 | |
| 15:11:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add type annotations to top-level modules https://review.opendev.org/c/openstack/nova/+/705658 | |
| 15:11:36 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Clean manager.Manager, service.Service signatures https://review.opendev.org/c/openstack/nova/+/764806 | |
| 15:11:37 | openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: Expand type hints for nova.block_device https://review.opendev.org/c/openstack/nova/+/742170 | |
| 15:24:50 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1909120 https://review.opendev.org/c/openstack/nova/+/768351 | |
| 15:24:52 | openstack | bug 1909120 in OpenStack Compute (nova) "n-api should reject requests to detach a volume when the compute is down" [Medium,Confirmed] https://launchpad.net/bugs/1909120 - Assigned to Lee Yarwood (lyarwood) | |
| 15:24:58 | openstackgerrit | Lee Yarwood proposed openstack/nova master: api: Reject requests to detach a volume when the compute is down https://review.opendev.org/c/openstack/nova/+/768352 | |
| 15:41:45 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1908075 https://review.opendev.org/c/openstack/nova/+/766976 | |
| 15:41:46 | openstack | bug 1908075 in OpenStack Compute (nova) "Nova allows a non-multiattach volume to be attached to multiple instances *if* its volume state is reset by an admin" [Undecided,New] https://launchpad.net/bugs/1908075 - Assigned to Lee Yarwood (lyarwood) | |
| 15:41:47 | openstackgerrit | Lee Yarwood proposed openstack/nova master: api: Reject volume attach requests when an active bdm exists https://review.opendev.org/c/openstack/nova/+/768472 | |
| 15:53:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Rationalize attachment of USB tablet https://review.opendev.org/c/openstack/nova/+/756551 | |
| 15:53:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Add support for virtio-based input devices https://review.opendev.org/c/openstack/nova/+/756552 | |
| 16:10:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Enable 'vmcoreinfo' feature by default https://review.opendev.org/c/openstack/nova/+/752912 | |
| 16:11:51 | gmann | stephenfin: lbragstad sean-k-mooney replied on https://review.opendev.org/c/openstack/placement/+/760240/21/placement/policies/base.py#20 | |
| 16:12:30 | gmann | Let's go with SYSTEM_ADMIN so that we donot depends on the timing of removing the old rule and enforce_scope=true by default | |
| 16:24:25 | openstackgerrit | sean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif https://review.opendev.org/c/openstack/nova/+/602432 | |
| 16:31:51 | stephenfin | lyarwood: Two of those three patches from me above ^ previously had +2 from you but conflicted with the just-merged Xen removal patch. Can you (re)look at them real quick? | |
| 16:33:16 | stephenfin | If you could get to https://review.opendev.org/c/openstack/nova/+/756552 too that'd be extra awesome. Spec is approved, fwiw | |
| 16:37:43 | lyarwood | stephenfin: need to run for a while now but I'll try to look later or first thing tomorrow | |