Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-04-01
14:51:25 sean-k-mooney cool am for the cyborg evacuate support https://review.opendev.org/#/c/715326/ should i do the arq binding on the conductor like we do for spawn or on the compute node
14:51:45 sean-k-mooney i started doing it on the compute node but i think i should move it to the conductor
14:52:10 sean-k-mooney so that cyborg can start to program the device asyc of nova
14:52:19 sean-k-mooney gibi: any thoughts?
14:54:03 dansmith sean-k-mooney: condutor for sure unless there's some really compelling reason otherwise
14:55:24 sean-k-mooney dansmith: cool only reason not to is i start doing it the other way first which is easily solved via copy paste :)
14:55:40 gibi sean-k-mooney: yeah, qos stuff is also done there
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

Earlier   Later