Posted	Nick	Remark
#openstack-nova - 2019-07-25
23:51:16	gmann	and before we do that we can trigger the notification to users via deprecating those APIs
23:51:29	melwitt	gmann: oh, sorry, I didn't know HTTPGone is different. my bad
23:52:07	melwitt	yeah, I'd like to be able to do that but wasn't sure about the API perspective of deprecating something when the new thing is opt-in and not on by default
23:52:41	gmann	melwitt: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/floating_ip_dns.py#L25
23:53:02	openstack	Launchpad bug 1837955 in OpenStack Compute (nova) "MaxRetriesExceeded sometime fails with messaging exception" [Undecided,New]
23:53:02	eandersson	sean-k-mooney, https://bugs.launchpad.net/nova/+bug/1837955
23:54:42	gmann	melwitt: the only things make me feel uncomfortable to do that was the comment johnthetubaguy added in alternate section about Forum discussion with operators about keeping the old tooling. But i hope that is only for transition period not permanently
23:57:27	gmann	melwitt: and second point is about GET quotas API to get the limits which are not going on new system. keep existing GET quotas APIs for them ?
23:57:52	melwitt	limits mode is on by default
23:57:52	melwitt	gmann: ok. the treatment of /os-quota-sets and /os-quota-class-sets is definitely temporary for the transition. the /limits API we're not completely sure because if we HTTPGone that one, there will be no more ability for users to show limits + usage in one API. but if community could be OK with having to go to two different API (keystone for limits and placement for usage) then we could 410 /limits too in the future when unified
23:59:10	gmann	melwitt: +1 we can decide the limit thing later once hierarchy unified limits are there.
#openstack-nova - 2019-07-26
00:01:06	gmann	melwitt: but can we query on placement for nova owned limits usage (server groups, injected_files etc) ? also not about showing the limit from keystone so we need to keep the GET /os-quota-sets for those limits al least.
00:01:45	melwitt	gmann: I'm not sure I understand the second question about the existing GET quotas API for limits. we have to keep that as long as old quotas are the default, I think
00:02:11	melwitt	gmann: we cannot, no. so that's another thing
00:03:17	gmann	yeah.
00:04:35	melwitt	gmann: it's a little unclear because there's overlap with /limits and GET /os-quota-sets. I'm not sure we need the latter if we have the former (once unified limits is default down the road)
00:05:17	melwitt	the (server groups, injected_files etc) limits are shown from /limits and will only be modifiable via the config files. no more API access for those
00:05:35	melwitt	*no more API access to update those
00:06:13	gmann	yeah, if we keep /limits then it is enough. But we need one API to show those limits and usage.
00:06:41	melwitt	yeah, agreed
00:12:18	openstackgerrit	melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
00:20:43	openstackgerrit	melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
00:22:03	openstackgerrit	melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
03:00:44	openstackgerrit	karim proposed openstack/nova-specs master: Support VM memory encryption using Intel's MKTME https://review.opendev.org/666769
03:22:36	openstackgerrit	Erik Olof Gunnar Andersson proposed openstack/nova master: Always cleanup when hitting MaxRetriesExceeded https://review.opendev.org/672855
03:23:42	openstackgerrit	pengyuesheng proposed openstack/os-vif master: Blacklist sphinx 2.1.0 (autodoc bug) https://review.opendev.org/672856
03:25:24	eandersson	sean-k-mooney, no clue if that is the fix, but looks reasonable to me ^
03:26:08	openstackgerrit	pengyuesheng proposed openstack/os-vif master: Bump the openstackdocstheme extension to 1.20 https://review.opendev.org/672857
03:27:17	openstackgerrit	pengyuesheng proposed openstack/os-vif master: Bump the openstackdocstheme extension to 1.20 https://review.opendev.org/672857
03:46:35	openstackgerrit	Erik Olof Gunnar Andersson proposed openstack/nova master: Always cleanup when hitting MaxRetriesExceeded https://review.opendev.org/672855
03:49:14	openstackgerrit	Erik Olof Gunnar Andersson proposed openstack/nova master: Always cleanup when hitting MaxRetriesExceeded https://review.opendev.org/672855
04:25:41	openstackgerrit	Merged openstack/nova-specs master: Add user_id to the migrations https://review.opendev.org/667894
04:31:37	alex_xu	sean-k-mooney: stephenfin the real_time_mask indeed neeed the '^', you can check my reply https://review.opendev.org/#/c/668656/4/specs/train/approved/use-pcpu-vcpu-in-one-instance.rst@84
04:49:20	openstackgerrit	Artom Lifshitz proposed openstack/nova master: [WIP] Functional test for NUMA live migration https://review.opendev.org/672595
07:02:41	openstackgerrit	Huachang Wang proposed openstack/os-traits master: CPU: add a trait for AVX512-VNNI support https://review.opendev.org/672888
07:48:57	openstackgerrit	Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420
07:48:57	openstackgerrit	Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
07:48:58	openstackgerrit	Adam Spiers proposed openstack/nova master: Add and to config.py https://review.opendev.org/636318
07:48:58	openstackgerrit	Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334
07:48:59	openstackgerrit	Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616
07:48:59	openstackgerrit	Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565
07:51:36	openstackgerrit	Alex Xu proposed openstack/nova master: Add the virt driver interface for claim and unclaim the devices https://review.opendev.org/670782
07:51:37	openstackgerrit	Alex Xu proposed openstack/nova master: Calling the virt driver's claim/unclaim_for_instance in resource tracker https://review.opendev.org/670784
07:51:37	openstackgerrit	Alex Xu proposed openstack/nova master: Moves the allocation retrieving early https://review.opendev.org/670783
07:51:38	openstackgerrit	Alex Xu proposed openstack/nova master: Populates the existing mediated devices in the libvirt device manager https://review.opendev.org/670787
07:51:38	openstackgerrit	Alex Xu proposed openstack/nova master: Add DeviceManager to the libvirt virt driver https://review.opendev.org/671388
07:51:39	openstackgerrit	Alex Xu proposed openstack/nova master: Adds functional test for creating the instance with vgpus https://review.opendev.org/671398
07:51:39	openstackgerrit	Alex Xu proposed openstack/nova master: Using the claim/unclaim_for_instance for mdevs https://review.opendev.org/671222
10:11:55	aspiers	sean-k-mooney, efried, bbobrov: exciting progress! http://paste.openstack.org/show/754889/
10:13:21	aspiers	cdent: inventory stuff seems to be working nicely :) ^^^
10:14:01	cdent	excellent, glad to hear it
10:42:27	sean-k-mooney	aspiers: yep that does look like its working well
10:42:59	sean-k-mooney	aspiers: you said you finally booted an sev instance with openstack yesterday right or is that the goal for today
10:43:35	sean-k-mooney	aspiers: cdent actully should th max unit on that inventory not be 1
10:44:02	sean-k-mooney	you can only have one mem_encryption_context per vm right
10:45:19	openstackgerrit	sean mooney proposed openstack/os-vif master: only disable mac ageing for ovs hybrid plug https://review.opendev.org/672834
10:45:33	sean-k-mooney	this a relitivly minor nit
10:55:12	aspiers	sean-k-mooney: booting is the goal for today
10:56:34	aspiers	sean-k-mooney: I don't get why you think max_unit should be 1? there are 16 slots configured on that machine
10:56:57	aspiers	ohhh, are you saying max_unit represents the maximum per VM, not the maximum per host?
10:56:57	sean-k-mooney	yes but max_unit is the maxium amount you can request in a singe request
10:57:03	sean-k-mooney	yes
10:57:04	aspiers	ok I didn't know that
10:57:10	aspiers	in that case yeah it should be 1
10:57:12	aspiers	good catch
10:57:14	sean-k-mooney	total is the total capasity
10:57:21	aspiers	right that never quite made sense before
10:57:23	aspiers	easy to fix
10:57:39	sean-k-mooney	ya as i said minor nit
10:57:46	aspiers	minor but v important!
10:57:50	sean-k-mooney	we will only ever ask for 1 anyway
10:57:56	aspiers	true
10:57:59	aspiers	well
10:58:14	aspiers	yeah
10:58:32	aspiers	unless some operator requested the resource directly via the flavor
10:58:37	sean-k-mooney	still it does not hurt to be defensive
10:58:40	sean-k-mooney	ya
10:59:48	aspiers	sean-k-mooney: is min_unit=1 correct though?
11:00:11	sean-k-mooney	can you ask for less then 1 allocation of something if you want a resouce
11:00:14	sean-k-mooney	yes it is
11:00:45	sean-k-mooney	the min unit only applie if you ask for a resouce of that type
11:01:22	sean-k-mooney	and since we dont want people saying resouce:MEM_ENCRYPTION_CONTEXT=0 it is correct
11:01:28	aspiers	got it
11:01:30	aspiers	thanks
11:02:35	sean-k-mooney	resouce:MEM_ENCRYPTION_CONTEXT=0 would actully for the vm to land on a host that supported sev but not consume sev resouces so it could be used incorrectly by opertors to stave those nodes or ram cpus and disk
11:03:02	sean-k-mooney	/for/force
11:03:51	aspiers	good point
11:28:25	openstackgerrit	Luyao Zhong proposed openstack/nova master: Report pmem namespaces resources by provider tree https://review.opendev.org/634551
11:28:25	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configures PMEM namespace https://review.opendev.org/634549
11:28:26	openstackgerrit	Luyao Zhong proposed openstack/nova master: Populates existing pmems in libvirt device manager https://review.opendev.org/672957
11:28:26	openstackgerrit	Luyao Zhong proposed openstack/nova master: Indroduce new LibvirtConfigGuestPMEM https://review.opendev.org/672956
11:28:27	openstackgerrit	Luyao Zhong proposed openstack/nova master: Support instance creation and resize with pmems https://review.opendev.org/672958
11:31:38	openstackgerrit	Surya Seetharaman proposed openstack/nova master: API microversion 2.75: Add 'power-update' external event https://review.opendev.org/645611
11:39:58	efried	code: https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/openstacksdk-in-nova
11:39:58	efried	spec: http://specs.openstack.org/openstack/nova-specs/specs/train/approved/openstacksdk-in-nova.html
11:39:58	efried	mnaser: re openstacksdk:
11:40:58	efried	mnaser: note that we're not using clouds.yaml. We're using newly introduced functionality to consume ksa.loading oslo.config opts to build the Connection obj.
11:42:27	efried	aspiers, sean-k-mooney: You can't ask placement for an allocation of 0.
11:43:13	sean-k-mooney	efried: because we explcitly check or because we set min_unit to 1
11:43:15	efried	If you want to land on a SEV host but not consume SEV resources (why??) you would have to require the SEV trait. Which of course wouldn't be generic for SEV/MKTME like the rc is.

Earlier   Later