Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-01-27
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
16:37:56 stephenfin ack, thanks
16:38:43 openstackgerrit Vlad Gusev proposed openstack/nova stable/stein: tests: Add regression test for bug 1894966 https://review.opendev.org/c/openstack/nova/+/772720
16:38:45 openstack bug 1894966 in OpenStack Compute (nova) stein "Create servergroup failed with unexpected error" [Undecided,Confirmed] https://launchpad.net/bugs/1894966
16:42:58 openstackgerrit Vlad Gusev proposed openstack/nova stable/stein: api: Set min, maxItems for server_group.policies field https://review.opendev.org/c/openstack/nova/+/772721
16:46:36 openstackgerrit Stephen Finucane proposed openstack/nova master: tox: Enable parallel docs build https://review.opendev.org/c/openstack/nova/+/751034
16:47:02 gmann lbragstad: did your single tests for all persona approach worked? https://review.opendev.org/c/openstack/placement/+/760238/9/placement/tests/functional/gabbits/inventory-legacy-rbac.yaml#20
16:47:56 gmann nova did the same to avoid 409 or creating new resource per persona though nova tests are with stub for non-api layer
16:47:56 lbragstad gmann i haven't gotten back around to it - i've been helping cinder setup test plumbing
16:48:07 gmann ohk
16:49:48 lbragstad i can repropose the beginning of that series since it sounds like we have consensus on the check string
16:50:10 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Add support for virtio-based input devices https://review.opendev.org/c/openstack/nova/+/756552
16:51:35 gmann lbragstad: for using common constraint from oslo?
16:52:19 lbragstad oh - no, i was just going to rebase them and cleanup the concern in the aggregate patch
16:52:56 gmann lbragstad: ah this https://review.opendev.org/c/openstack/placement/+/760235/10/placement/policies/aggregate.py#52
16:53:34 lbragstad yeah
16:55:08 gmann lbragstad: ok, as you are rebasing what you think of https://review.opendev.org/c/openstack/placement/+/760240/21/placement/policies/resource_provider.py#50
16:55:29 gmann this can leave all explicit deprecation in policies.
16:56:22 gmann and deprecate current admin with system-admin rule in base.py and use system-admin rule as check_str
16:56:43 gmann and in system-admin rule we will use common string from oslo whenever we are ready
16:57:34 gmann lbragstad: if that is too much asking at this stage then we can leave that i can do that in follow up for all policy together
16:57:50 lbragstad gmann stephenfin has a patch to do the deprecation https://review.opendev.org/c/openstack/placement/+/772334/4
16:58:14 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: libvirt: start tracking NUMACell.socket for hosts https://review.opendev.org/c/openstack/nova/+/766816

Earlier   Later