Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-02-14
02:55:29 openstackgerrit Liang Fang proposed openstack/nova-specs master: Support volume local cache https://review.opendev.org/689070
02:58:01 openstackgerrit Brin Zhang proposed openstack/nova master: Introduce scope_types in os-instance-action policy https://review.opendev.org/707751
06:41:43 openstackgerrit Brin Zhang proposed openstack/nova master: Introduce scope_types in os-instance-action policy https://review.opendev.org/707751
06:42:13 openstackgerrit Brin Zhang proposed openstack/nova master: Introduce scope_types in os-instance-action policy https://review.opendev.org/707751
07:57:56 openstackgerrit Brin Zhang proposed openstack/nova master: Add test coverage of existing os-instance-actions policies https://review.opendev.org/707777
07:58:47 openstackgerrit Brin Zhang proposed openstack/nova master: Introduce scope_types in os-instance-action policy https://review.opendev.org/707751
08:59:44 openstackgerrit Guo Jingyu proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120
09:03:10 openstackgerrit Brin Zhang proposed openstack/nova master: Add test coverage of existing os-instance-actions policies https://review.opendev.org/707777
09:07:21 openstackgerrit Guo Jingyu proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120
09:28:53 openstackgerrit Brin Zhang proposed openstack/nova master: Introduce scope_types in os-instance-action policy https://review.opendev.org/707751
10:02:02 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM - Test TEMPEST_EXTEND_ATTACHED_ENCRYPTED_VOLUME https://review.opendev.org/707593
10:11:38 openstackgerrit Merged openstack/nova-specs master: Add nova-audit spec https://review.opendev.org/693226
10:23:08 openstackgerrit Sylvain Bauza proposed openstack/nova-specs master: Proposes NUMA topology with RPs https://review.opendev.org/552924
10:23:23 bauzas stephenfin: sean-k-mooney: efried: I made another iteration to clarify ^
10:29:44 openstackgerrit Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735
10:29:45 openstackgerrit Sundar Nadathur proposed openstack/nova master: Enable hard/soft reboot with accelerators. https://review.opendev.org/697940
10:29:45 openstackgerrit Sundar Nadathur proposed openstack/nova master: Enable start/stop of instances with accelerators. https://review.opendev.org/699553
10:29:46 openstackgerrit Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554
10:29:46 openstackgerrit Sundar Nadathur proposed openstack/nova master: Bump compute rpcapi version and reduce Cyborg calls. https://review.opendev.org/704227
10:29:47 openstackgerrit Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999
10:47:01 openstackgerrit Lee Yarwood proposed openstack/nova master: api: Introduce microverion 2.82 allowing boot from volume rescue https://review.opendev.org/701430
10:47:02 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Extract _get_bdm_image_metadata into nova.utils https://review.opendev.org/705212
11:53:14 openstackgerrit Guo Jingyu proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120
12:18:02 openstackgerrit Merged openstack/nova stable/rocky: Mask the token used to allow access to consoles https://review.opendev.org/704255
12:41:43 sean-k-mooney bauzas: i think you missed some of efried's comments. like explainging the fallback query.
12:42:29 sean-k-mooney efried: also did you see my suggestion to have teh numa=false host be added to an aggreate so we can use forbiden aggreates for that case https://review.opendev.org/#/c/552924/21/specs/ussuri/approved/numa-topology-with-rps.rst@235
12:46:33 sean-k-mooney over all however i think there is enough we agree on that we can proceed with this and adject with a fix up patch later based on implementaion discussions
12:47:43 sean-k-mooney i am off today so im not going to be around but i might see pings in the evening as my irc client will be connected
12:59:21 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120
13:48:05 bauzas sean-k-mooney: again, I explained this
13:48:55 bauzas sean-k-mooney: oops my bad, I didn't passed my draft comments
13:48:59 bauzas sean-k-mooney: see them
13:49:31 bauzas stephenfin: can you please review it ? https://review.opendev.org/#/c/552924/
14:19:25 openstackgerrit Balazs Gibizer proposed openstack/nova stable/queens: Mask the token used to allow access to consoles https://review.opendev.org/707845
14:20:33 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120
15:03:39 bauzas efried: stephenfin: the spec is waiting for your comments :) https://review.opendev.org/#/c/552924/
15:03:58 melwitt johnthetubaguy, bauzas: thank you both for your review on the nova-audit spec ++
15:08:04 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120
15:10:57 melwitt alex_xu: hey, did you mean to hold off on +W on the unified limits spec? https://review.opendev.org/602201
15:25:17 openstackgerrit Alexandre arents proposed openstack/nova master: Avoid allocation leak when deleting instance stuck in BUILD https://review.opendev.org/702368
15:31:27 alex_xu melwitt: I'm ok with spec, just see bauzas whether want to look at that again
15:31:42 bauzas alex_xu: the unified limits one ?
15:31:44 melwitt alex_xu: cool, thanks
15:31:45 bauzas I can take a look
15:32:09 alex_xu np
15:42:31 efried alex_xu, johnthetubaguy: and also stephenfin: bauzas: gibi: Would you please have a look at https://review.opendev.org/#/c/580336 (delete on termination) and see if it's ready to +A today?
15:43:50 bauzas efried: /me opens a tab
15:44:18 efried sean-k-mooney, bauzas: I'm looking at the NUMA RP spec again right now. Based on comments I skimmed on my phone, I think we might have missed the boat on the `False` thing. But checking...
15:44:53 bauzas efried: when we discussed this with stephenfin, we agreed on splitting between non-NUMA and NUMA-aware nodes
15:45:13 bauzas so, if you say 'no' for a node, then you shouldn't get NUMA-aware instances
15:45:52 bauzas it's like 'no, I don't want to get a beer', but then the bar tender gives you one
15:46:08 efried bauzas: exactly, and there was a design hole for that in PS21.
15:46:49 gibi efried: ack
15:48:12 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support re-configure deleted_on_termination in server https://review.opendev.org/580336
15:50:13 bauzas efried: ok, look then at the new revision and tell me then if you see some hole
15:50:19 efried bauzas: now I think I understand that you got rid of the fallback query completely, and on purpose. But that doesn't work. The upgrade issue that gibi and stephenfin identified still exists.
15:50:42 efried bauzas: Specifically: without the 'fallback query', NUMA-aware flavors can't land on `None` hosts.
15:51:06 efried But *with* the fallback query, NUMA-aware flavors will *also* (incorrectly) land on `False` hosts.
15:51:08 bauzas efried: ah, possibly
15:51:14 bauzas shit
15:51:55 efried So we need a) the fallback query, and b) some other design element to fix that second problem.
15:52:08 efried sean-k-mooney's suggestion of using an aggregate... that will work, but it seems like a big hammer to me.
15:52:16 bauzas or using another trait then
15:52:22 efried I think I would prefer marking `False` hosts with a `NO_NUMA_HERE` trait... yes
15:52:28 bauzas like HW_NON_NUMA
15:52:40 bauzas ok, I think we can do this
15:52:58 efried It's not pretty. But it's also *temporary*. Once all computes are upgraded to V where the conf is mandatory, we can get rid of that, because we also get rid of the fallback query.
15:53:16 efried please be sure to mention ^ that.
15:53:22 efried sean-k-mooney: does that work for you?
15:53:32 bauzas a trait for which nodes then ?
15:53:47 efried for U+ nodes where the conf opt is marked False explicitly.
15:53:47 bauzas for those having 'false' right?
15:53:49 efried y
15:54:06 bauzas ok, because then we will use the trait even after V
15:54:09 efried no
15:54:37 efried I mean, it wouldn't *hurt* to keep it, but it won't be needed after V.
15:54:41 bauzas if so, I'd prefer to have a trait on 'none'
15:54:45 efried can't
15:54:54 bauzas actually you're right
15:55:00 efried because we can't muck with T hosts
15:55:03 bauzas and yeah we can still have the trait
15:55:07 bauzas in V
15:55:23 bauzas and for example, stoppint to have it when we want (V or W)
15:55:26 efried We could. But we wouldn't need it anymore. But we can make that decision in V. Just mention that we *could* get rid of it if we wanted to.
15:55:33 bauzas agreed
15:55:44 bauzas okay I think I can write *something*
15:55:47 alex_xu bauzas: efried that spec I only not sure about the policy, that API is admin-only API. but I think his usecase is for normal user. maybe need to add new policy
15:55:51 alex_xu brinzhang: ^
15:55:53 bauzas melwitt: +Wd https://review.opendev.org/#/c/602201/17
15:56:24 gibi efried, brinzhang: I have to API related comments in https://review.opendev.org/#/c/580336/
15:56:42 melwitt thanks bauzas!
15:57:15 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120
15:57:45 brinzhang alex_xu, bauzas, melwitt, gibi, stephenfin, efried: the nova-support-webvnc-with-password-anthentication spec was ready, please review, hope it can be merged before freeze SPEC day (Feb 14), thanks.
15:58:50 brinzhang alex_xu: thanks for your review of the noVNC spec
15:59:06 alex_xu brinzhang: np, quick reminder the title https://review.opendev.org/#/c/623120/27//COMMIT_MSG
15:59:35 brinzhang alex_xu: will update
16:01:17 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Proposal for a safer remote console with password authentication https://review.opendev.org/623120
16:01:36 brinzhang alex_xu: done.
16:02:03 efried brinzhang: you're going to have a busy evening :)
16:02:20 efried brinzhang: did you see gibi's comment on https://review.opendev.org/#/c/580336/ above?

Earlier   Later