| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-09 | |||
| 18:06:17 | dansmith | which seems silly to me | |
| 18:06:19 | efried | that's pretty ugly | |
| 18:06:41 | dansmith | because checking policy of an unlock operation during get is not okay for some reason | |
| 18:07:13 | efried | becuase the policy is conditional on the owner of the lock? | |
| 18:07:23 | dansmith | I dunno | |
| 18:07:23 | efried | a thing that can't be determined at policy enforcement time? | |
| 18:07:35 | dansmith | what I want is a boolean that tells me "can I unlock this thing?" | |
| 18:07:49 | efried | locked_by == 'owner' | |
| 18:08:06 | dansmith | but if I'm an admin, I can unlock it too | |
| 18:08:06 | efried | tssurya: Okay, so I'm with dansmith on this. Who's the dissenter? mriedem? | |
| 18:08:12 | dansmith | or if the policy is such that I can unlock.. | |
| 18:08:34 | dansmith | efried: just to be clear, I'm fine with not exposing locked_by | |
| 18:08:45 | mriedem | efried: locked_by won't tell you "hey, what did I lock?" anyway | |
| 18:08:51 | efried | can_I_unlock = am_I_admin or locked_by == 'owner' | |
| 18:08:51 | efried | Okay, | |
| 18:09:14 | mriedem | and owner = the instance is in a project that i'm in | |
| 18:09:18 | dansmith | not sure that's enough | |
| 18:09:19 | mriedem | i think... | |
| 18:09:40 | dansmith | can't I write complex policy to allow a role that can unlock things for other projects? | |
| 18:10:08 | dansmith | regardless, I'd really suggest we not get back into this, remove locked_by and let tssurya proceed | |
| 18:10:24 | tssurya | dansmith: thanks :) | |
| 18:10:29 | tssurya | efried you okay with that ? | |
| 18:10:51 | efried | sure | |
| 18:11:06 | mriedem | i also really don't want to think about locked_by anymore, so if the goal is add a reason field, let's do that - making locked_by not suck is a whole other can of worms | |
| 18:11:09 | mriedem | and likely a separate spec | |
| 18:11:20 | efried | I can get on board with that ^ | |
| 18:11:32 | efried | I had actually drafted a comment that read a lot like that | |
| 18:11:45 | tssurya | ok thanks everyone I am deleting locked_by from my memory | |
| 18:11:55 | efried | tssurya: you can't delete it from memory. | |
| 18:12:09 | efried | you should put something in Alternatives so we remember we had this conversation. | |
| 18:12:11 | mriedem | not persistent memory anyway https://review.openstack.org/#/c/601596/ | |
| 18:12:13 | mriedem | hi-o!!!! | |
| 18:12:59 | efried | tssurya: ...even if it's just a link to the eavesdrop of this conversation | |
| 18:13:12 | openstackgerrit | Surya Seetharaman proposed openstack/nova-specs master: Support adding the reason behind a server lock https://review.openstack.org/638629 | |
| 18:14:33 | mriedem | i feel like at this point in my day my time would be better served raking up dead grass outside since it's supposed to dump a foot a snow on us tomorrow | |
| 18:14:41 | mriedem | *of snow | |
| 18:24:44 | gmann | tssurya: mriedem efried dansmith can we remove the locked_by from valid sort/filter key also as that is not going to be exposed anyway. | |
| 18:26:17 | efried | gmann: If it does nothing right now? Then sure, why don't we do that in a totally unrelated patch? | |
| 18:27:37 | gmann | efried: that need microversion bump so doing in unrelated patch need another spec and version bump | |
| 18:27:57 | efried | gmann: So it *does* do something right now? | |
| 18:28:14 | efried | It actually works, you just don't see the locked_by field in your result? | |
| 18:28:22 | efried | or it's ignored, but removing it will cause a 400? | |
| 18:28:31 | dansmith | anybody want to blow their brains out when I'm done with mine? | |
| 18:30:09 | gmann | efried: yeah it does not show locked_by in results so user will just assume it worked well but no confirmation. in code we do consider that. | |
| 18:30:26 | efried | then why remove it? | |
| 18:30:52 | gmann | there is no way for user to know what is valid value of locked_by as nova does not expose that at all | |
| 18:30:53 | efried | rather than put it off until later and fix it properly | |
| 18:31:05 | ganso | Hi efried. Sorry to interrupt. When you have some time, could you please review https://review.openstack.org/#/c/650437/ and https://review.openstack.org/#/c/649421/ ? Thanks in advance | |
| 18:31:53 | efried | ganso: Is tomorrow soon enough? I'd like to focus on specs as much as possible today. | |
| 18:32:03 | ganso | efried: sure, thank you! | |
| 18:32:17 | gmann | ok, if we do something with locked_by later then only fix sort/filter too / that is ok for me. | |
| 18:32:18 | efried | ganso: Oh, those are stein. I'm not stable, so I can't really help you :) | |
| 18:32:35 | efried | gmann: Okay, I think that's the plan. | |
| 18:32:46 | ganso | efried: oh, sorry... I'll look for the stable maintainers | |
| 18:32:48 | gmann | +1 | |
| 18:33:01 | efried | gmann: Current bp is going to add sorting/filtering by the locked bool. | |
| 18:33:05 | efried | iiuc | |
| 18:33:16 | efried | but ignore locked_by entirely - not fix it or remove it. | |
| 18:34:29 | mriedem | ganso: https://review.openstack.org/#/admin/groups/540,members | |
| 18:34:38 | mriedem | ganso: but also, those can't merge yet anyway b/c we haven't GA'ed stien | |
| 18:34:38 | gmann | efried: you mean current spec will start ignoring 'locked_by' from sort/filter ? | |
| 18:34:40 | mriedem | *stein | |
| 18:35:04 | ganso | mriedem: oh right! release is friday, correct? | |
| 18:35:05 | gmann | i feel leave it as it is currently. | |
| 18:35:06 | mriedem | hence my -W on the bottom patch | |
| 18:35:11 | mriedem | ganso: tomorrow i think | |
| 18:35:17 | efried | gmann: No, sorry, I mean the current spec will do nothing to change locked_by in any way. | |
| 18:35:29 | mriedem | dansmith: grab a rake | |
| 18:35:41 | gmann | efried: +1 | |
| 18:46:51 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.openstack.org/572790 | |
| 18:46:52 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Keep attach_mode as top-level field in _translate_attachment_ref https://review.openstack.org/574413 | |
| 18:47:34 | lyarwood | mriedem: ^ hope you don't mind, this came up downstream last week, finally getting back around to it now and noticed some whitespace. | |
| 19:34:44 | tssurya | mriedem, dansmith: thanks for reviewing https://review.openstack.org/#/c/636132/ , sorry didn't get time to respond, will go through it tomorrow. | |
| 19:35:12 | dansmith | tssurya: hopefully we can make this one every bit as painful as the locked_by one! | |
| 19:35:14 | dansmith | (not) | |
| 19:41:57 | mriedem | lyarwood: fine with me, but you'll have to convince mdbooth since i think it was held up on him, plus this tempest change https://review.openstack.org/#/c/573025/ | |
| 19:44:11 | lyarwood | mriedem: ack thanks will do | |
| 19:44:50 | mriedem | lyarwood: btw i was told you own this now https://review.openstack.org/#/c/551349/ | |
| 19:46:10 | lyarwood | mriedem: yeah, I've just not found my way back to it since the last PS | |
| 19:46:25 | mriedem | let your heart guide you young lee | |
| 19:47:05 | lyarwood | mriedem: I'll queue it up for the morning, thanks for the review! | |
| 20:24:18 | mriedem | so we agree we're not going to sort/filter on locked_reason right | |
| 20:24:19 | mriedem | ? | |
| 20:27:11 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Support adding the reason behind a server lock https://review.openstack.org/638629 | |
| 20:28:51 | gmann | mriedem: right. filter/sort by 'locked' only | |
| 20:29:12 | mriedem | efried_afk: well, got one spec approved today | |
| 20:29:30 | mriedem | who has the champagne? is dansmith still alive? | |
| 20:31:00 | dansmith | heh | |
| 20:38:16 | openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs master: Spec for API inconsistency cleanup https://review.openstack.org/603969 | |
| 20:38:37 | gmann | mriedem: updated the readonly info for OS_EXt prefix fields ^^. OS-DCF:diskConfig i propose to change in request also. | |
| 20:38:54 | gmann | checked all the extensions fields and listed the one with prefix | |
| 20:48:20 | openstackgerrit | Merged openstack/nova-specs master: Support adding the reason behind a server lock https://review.openstack.org/638629 | |
| 21:27:38 | efried | mriedem: \o/ but a lot got done today. | |
| 21:27:58 | efried | and especially leading up to today, with people fleshing their specs. | |
| 22:37:20 | mordred | efried: heh libvirt gorp | |
| 22:38:31 | openstackgerrit | Dakshina Ilangovan proposed openstack/nova-specs master: Nova LLC allocation - RMD plugin for RDT CAT https://review.openstack.org/651233 | |
| 23:31:40 | alex_xu | mriedem: hah, hope that is cold medicine :) | |
| #openstack-nova - 2019-04-10 | |||
| 00:46:49 | openstackgerrit | Merged openstack/nova stable/rocky: Replace openstack.org git:// URLs with https:// https://review.openstack.org/646686 | |
| 01:51:44 | openstackgerrit | melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172 | |
| 02:44:33 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Use the functional test test_parallel_evacuate_with_server_group https://review.openstack.org/649963 | |