Posted	Nick	Remark
#openstack-nova - 2020-04-01
14:55:51	sean-k-mooney	dansmith: should i alther the rpc to pass the arqs
14:55:53	gibi	I mean in the conducotr for evac
14:56:21	dansmith	sean-k-mooney: we did for spawn because it's so much more efficient
14:56:46	dansmith	sean-k-mooney: the early stuff landed without it because it was less change, but that's the only reason
14:57:24	sean-k-mooney	so you would be in favor of just doing it for evac too
14:59:34	sean-k-mooney	ok ill rework the change to bump the rpc and move the recreation of the arq and bindign to the conductor and do the same wait on the compute we do for spawn
14:59:45	sean-k-mooney	thanks
15:11:54	openstackgerrit	melanie witt proposed openstack/nova master: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/701478
15:11:55	openstack	bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929
15:39:10	gibi	jkulik: thanks for the bugfix https://review.opendev.org/#/c/713863 it looks good to me
16:01:08	openstackgerrit	melanie witt proposed openstack/nova master: Move nova-manage db purge to nova-audit https://review.opendev.org/708783
16:01:09	openstackgerrit	melanie witt proposed openstack/nova master: Move nova-manage db archive_deleted_rows to nova-audit https://review.opendev.org/708784
16:01:09	openstackgerrit	melanie witt proposed openstack/nova master: Move nova-manage cell_v2 discover_hosts to nova-manage https://review.opendev.org/708785
16:01:10	openstackgerrit	melanie witt proposed openstack/nova master: Move nova-manage cell_v2 map_instances to nova-audit https://review.opendev.org/708786
16:01:10	openstackgerrit	melanie witt proposed openstack/nova master: Move nova-manage placement sync_aggregates to nova-audit https://review.opendev.org/708787
16:01:11	openstackgerrit	melanie witt proposed openstack/nova master: Move nova-manage placement heal_allocations to nova-audit https://review.opendev.org/708788
16:02:26	melwitt	bauzas: fyi this is what I have so far ^
16:05:44	bauzas	melwitt: all cool, will look at those
16:06:08	melwitt	thanks :)
16:11:34	gmann	melwitt: in case you missed. I hope it covers the requirement you mentioned during policy spec - https://review.opendev.org/#/c/716134/
16:12:35	gmann	melwitt: basically for server action, we are going with same approach. default to system admin ( for current admin-only actions) but give operator a options to assign that role to project also. keeping scope as system and project both
16:14:18	melwitt	gmann: I think that makes sense, but I'm initially unclear how this didn't show up as a bug fix like some of the others?
16:16:28	gmann	melwitt: they are admin only in policy also and does not pass server project id. other policy bugs are on mismatch on what policy say and what API pass to policy.
16:17:22	melwitt	gmann: oh right, derp. thanks. sorry, I get confused by some of these
16:17:29	gmann	or we can say we are extending these action to be do-able by project with new defaults
16:27:59	openstackgerrit	Merged openstack/nova master: Add new default roles in os-flavor-access policies https://review.opendev.org/713697
16:28:06	openstackgerrit	Merged openstack/nova master: Add test coverage of existing migrate server policies https://review.opendev.org/716128
16:28:12	openstackgerrit	Merged openstack/nova master: Add test coverage of existing migrations policies https://review.opendev.org/716136
16:48:49	gmann	gibi: brinzhang_ this lgtm, 1 comment on few more tests - https://review.opendev.org/#/c/693828/25
16:54:55	openstackgerrit	Merged openstack/nova master: Fix unlock server policy to be admin_or_owner https://review.opendev.org/716071
17:09:33	openstackgerrit	Merged openstack/nova master: Fix unpause server policy to be admin_or_owner https://review.opendev.org/716165
17:09:40	openstackgerrit	Merged openstack/nova master: Add test coverage of existing pause server policies https://review.opendev.org/716161
17:09:51	openstackgerrit	Merged openstack/nova master: Add test coverage of existing lock server policies https://review.opendev.org/716057
17:10:00	openstackgerrit	Merged openstack/nova master: Introduce scope_types in pause server policy https://review.opendev.org/716187
17:10:08	openstackgerrit	Merged openstack/nova master: Add new default roles in pause server policies https://review.opendev.org/716191
17:10:16	openstackgerrit	Merged openstack/nova master: Introduce scope_types in migrate server https://review.opendev.org/716130
17:10:23	openstackgerrit	Merged openstack/nova master: Add new default roles in migrate server policies https://review.opendev.org/716132
17:11:30	openstackgerrit	Merged openstack/nova master: Pass the actual target in migrate server policy https://review.opendev.org/716134
17:11:36	openstackgerrit	Merged openstack/nova master: Add new default roles in lock server policies https://review.opendev.org/716122
17:11:42	openstackgerrit	Merged openstack/nova master: Pass the actual target in unlock override policy https://review.opendev.org/716428
17:11:50	openstackgerrit	Merged openstack/nova master: Add test coverage of existing rescue policies https://review.opendev.org/716486
17:11:59	openstackgerrit	Merged openstack/nova master: Introduce scope_types in rescue server policy https://review.opendev.org/716488
17:12:09	openstackgerrit	Merged openstack/nova master: Add test coverage of existing remote console policies https://review.opendev.org/716482
17:12:17	openstackgerrit	Merged openstack/nova master: Introduce scope_types in remote consoles policy https://review.opendev.org/716483
17:22:52	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: DNM: showing policy enforcement for PUT attachments https://review.opendev.org/716679
17:27:56	gmann	gibi: brinzhang_ dansmith commented on policy enforcement for PUT attachments. showed example for covering all cases- https://review.opendev.org/716679
17:52:08	_erlon_	hi guys, is the Nova meeting for tomorrow up? I don't see any topics on the agenda: https://wiki.openstack.org/wiki/Meetings/Nova
17:54:56	gmann	_erlon_: i think yes (it is not explicitly cancneld). may be gibi will update agenda and dates soon .
17:55:24	_erlon_	gmann: ok, thanks, Ill fill up my topic tomorrow
18:53:39	openstackgerrit	Merged openstack/nova master: libvirt: Always provide the size in bytes when calling virDomainBlockResize https://review.opendev.org/707590
18:53:45	openstackgerrit	Merged openstack/nova master: images: Remove Libvirt specific configurable use from qemu_img_info https://review.opendev.org/707591
18:54:54	openstackgerrit	Merged openstack/nova master: libvirt: Remove QEMU_VERSION_REQ_SHARED https://review.opendev.org/710239
18:55:00	openstackgerrit	Merged openstack/nova master: Pass the actual target in os-hypervisors policy https://review.opendev.org/715074
18:55:06	openstackgerrit	Merged openstack/nova master: Correct limits policy check_str https://review.opendev.org/715672
18:57:48	openstackgerrit	Victor Coutellier proposed openstack/python-novaclient master: Microversion 2.83 - Add more filters for the nova list command https://review.opendev.org/713089
19:58:26	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Combine the limits policies in single place https://review.opendev.org/715678
20:02:27	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in remote console policies https://review.opendev.org/716484
21:30:12	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing security groups policies https://review.opendev.org/716779
22:12:50	rs91	Has anyone run into this os-brick issue with multipath failure reported in https://bugs.launchpad.net/nova/+bug/1414527? Is there any known workaround for this issue or plan to fix it in nova?
22:12:51	openstack	Launchpad bug 1414527 in OpenStack Compute (nova) "The multipath device descriptors remove failed when the volume has partition" [Undecided,Confirmed]
22:26:11	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Correct secutrity groups policy check_str https://review.opendev.org/716782
22:27:41	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing security groups policies https://review.opendev.org/716779
22:41:48	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing security groups policies https://review.opendev.org/716779
23:10:18	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in security groups policy https://review.opendev.org/716786
23:30:00	openstackgerrit	Merged openstack/nova master: Add new default roles in rescue server policies https://review.opendev.org/716496
23:57:40	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in security group policies https://review.opendev.org/716793
23:58:37	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing security groups policies https://review.opendev.org/716779
23:58:53	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in security groups policy https://review.opendev.org/716786
23:59:52	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in security group policies https://review.opendev.org/716793
#openstack-nova - 2020-04-02
00:13:30	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in security group policies https://review.opendev.org/716797
00:37:46	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server diagnostics policies https://review.opendev.org/716800
00:39:10	brinzhang_	gmann:I think you mean I should merged https://review.opendev.org/#/c/716679 to https://review.opendev.org/#/c/716679, right?
00:40:25	gmann	brinzhang_: that is just example patch i showed instead of comment which was little hard to explain. i will abandon this once you do those changes in your patch.
00:42:15	brinzhang_	gmann: I think I know what you want to say, if the user want to swap, need to check the v2.85 and swap policy,
00:43:24	brinzhang_	gmann: if the user want to swap + update, they should check update policy for d-o-t, because of other parameters we doesnot allow to update (update only)
00:43:30	gmann	brinzhang_: swap policy check for swap only or update+swap both. swap only can be with any microversion
00:43:51	brinzhang_	yes
00:44:02	gmann	brinzhang_: correct. because that is place where we know request is for update
00:45:10	brinzhang_	and I can say, swap policy can do anything in update api, contains update policy
00:47:12	brinzhang_	gmann: I will update and add the these test said by gibi
00:48:55	gmann	brinzhang_: yeah, tests covering all those cases will be good
00:57:46	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in server diagnostics https://review.opendev.org/716803
01:05:31	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in security group policies https://review.opendev.org/716793
01:08:45	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in server diagnostics policies https://review.opendev.org/716805
01:30:51	brinzhang_	gmann: context.can(va_policies.POLICY_ROOT % 'swap') this policy check, does we need to add target={} in it?
01:31:33	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server diagnostics policy https://review.opendev.org/716810
01:36:45	openstackgerrit	melanie witt proposed openstack/nova master: Move nova-manage placement heal_allocations to nova-audit https://review.opendev.org/708788
01:51:10	melwitt	gmann: hey, just to clarify, if I understand correctly we're going to address the warning log issue in a later patch before FF with the plan being to remove the warning log and use Upgrade reno instead? and we'll have a nova doc page explaining the detail of the change in defaults with instructions for users on how to review the new defaults and use them? (enforce_scope = True and how to generate new defaults for review)?
03:45:05	openstackgerrit	Brin Zhang proposed openstack/nova master: Separate update and swap volume policies https://review.opendev.org/711194
03:46:37	openstackgerrit	Guo Jingyu proposed openstack/nova master: WIP: nova supports password encrypted VNC https://review.opendev.org/622336
03:48:35	brinzhang_	gibi, gmann: I addressed your comments in https://review.opendev.org/#/c/711194/11, but when I want to add the "policy check fails for swap + update due to update policy failure" case, that cannot get what I want, see http://paste.openstack.org/show/791501/
03:48:56	brinzhang_	gibi, gmann: Anything I missed?
07:08:08	gibi	good morning nova
07:08:16	gibi	brinzhang_: will get to it soon
07:08:47	brinzhang_	good morning gibi
07:08:48	gibi	_erlon_: it is up as usual. I update the agenda as soon
07:08:58	brinzhang_	gibi: thanks, wait for your check
07:11:24	brinzhang_	gibi: the noVnc feature, my colleague pushed the PoC code, maybe we can done of that feature, I will give him help as soon as possiable https://review.opendev.org/#/c/622336/
07:12:06	brinzhang_	gibi: you dont too review now, when it's ready, I will respin
07:14:14	gibi	brinzhang_: ack, I saw the patch getting up

Earlier   Later