| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-09 | |||
| 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 | |
| 02:45:17 | openstackgerrit | Li Zhouzhou proposed openstack/nova stable/queens: SRIOV pci_numa_policy dosen't working when create instance with 'cpu_policy' and 'numa_nodes https://review.openstack.org/651429 | |
| 06:15:09 | kaisers | efried: Thanks for looking. I'm busy with other stuff atm but will follow up on your feedback later on! | |
| 07:09:14 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove deprecated 'default_flavor' config option https://review.openstack.org/645476 | |
| 07:47:34 | openstackgerrit | Adrian Chiris proposed openstack/nova master: libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | |
| 08:00:13 | sean-k-mooney | alex_xu: which comment wast that? | |
| 08:00:49 | sean-k-mooney | i didnt get time to review as much specs yesterday as i would have liked but i assume one of the cpu frequencey specs? | |
| 08:02:09 | sean-k-mooney | alex_xu: ah i see you replied on https://review.openstack.org/#/c/649882/ ill take another look in an hour or so | |
| 08:09:49 | openstackgerrit | Boxiang Zhu proposed openstack/nova-specs master: Add host and hypervisor_hostname flag to create server https://review.openstack.org/645458 | |
| 08:27:21 | openstackgerrit | Boxiang Zhu proposed openstack/nova-specs master: Add host and hypervisor_hostname flag to create server https://review.openstack.org/645458 | |
| 08:44:21 | openstackgerrit | Boxiang Zhu proposed openstack/nova-specs master: Add host and hypervisor_hostname flag to create server https://review.openstack.org/645458 | |
| 08:58:07 | alex_xu | sean-k-mooney: thanks! | |
| 08:58:26 | alex_xu | sean-k-mooney: I mean all of them, they are good comments :) | |
| 09:02:16 | alex_xu | sean-k-mooney: I saw you like the extra spec defined in https://review.openstack.org/#/c/651024/5/specs/train/approved/rmd-plugin-energy-efficiency-core-p-states-control.rst@310, but I'm not quite understand, it will be great you have an example for it, if it is good, I'm happy to change also | |
| 09:02:29 | alex_xu | for now, I need to offline for catching my daughter | |
| 09:02:44 | sean-k-mooney | ok o/ | |
| 09:02:51 | alex_xu | thanks! | |
| 09:03:02 | sean-k-mooney | and ya i like it more as its a finite set | |