Posted	Nick	Remark
#openstack-nova - 2019-08-04
06:09:20	openstackgerrit	Brin Zhang proposed openstack/nova master: WIP: Filter migrations by user_id/project_id https://review.opendev.org/674243
14:05:25	zhouyao	@sean-k-mooney May i ask if you could check this patch? https://review.opendev.org/#/c/621646
20:22:56	openstackgerrit	Stephen Finucane proposed openstack/nova master: trivial: Rewrap definitions of 'NUMACell' https://review.opendev.org/674395
20:22:57	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Differentiate between shared and dedicated CPUs https://review.opendev.org/671800
20:22:57	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Remove unnecessary argument https://review.opendev.org/674396
20:22:58	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Rename 'fields' import to 'obj_fields' https://review.opendev.org/674103
20:22:58	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add support translating CPU policy extra specs, image meta https://review.opendev.org/671801
20:22:59	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add proxy fields to NUMATopology object https://review.opendev.org/674104
23:57:32	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Filter UnsupportedServiceVersion warning https://review.opendev.org/674515
23:59:33	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add a live migration regression test https://review.opendev.org/641200
23:59:52	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Retrun 400 if invalid query parameters are specified https://review.opendev.org/670440
#openstack-nova - 2019-08-05
00:59:25	openstackgerrit	Brin Zhang proposed openstack/nova master: WIP: Filter migrations by user_id/project_id https://review.opendev.org/674243
01:01:32	openstackgerrit	Brin Zhang proposed openstack/nova master: WIP: Add user_id and project_id colume to Migration https://review.opendev.org/673990
03:44:20	openstackgerrit	ya.wang proposed openstack/nova master: Support report driver migrate feature https://review.opendev.org/674522
03:44:21	openstackgerrit	ya.wang proposed openstack/nova master: Scheduelr translate properties to traits https://review.opendev.org/674524
03:44:21	openstackgerrit	ya.wang proposed openstack/nova master: Add property check when create/rebuild/resize. https://review.opendev.org/674523
03:44:22	openstackgerrit	ya.wang proposed openstack/nova master: Add releasenote and update docs. https://review.opendev.org/674526
03:44:22	openstackgerrit	ya.wang proposed openstack/nova master: Change method: _live_migration_flags https://review.opendev.org/674525
06:11:15	alex_xu	bauzas: dansmith looking for some feedback on https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:claim_for_instance, both mdev and vpmem are implement
06:45:52	openstackgerrit	Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476
07:14:48	alex_xu	dansmith: efried, one more corner case found. For same host resize, we only have one domain xml in the libvirt before confirming the resize. So if the host is down, and then the host back again, we will lose the source allocation.
07:16:09	alex_xu	dannins: efried, one method is we write the device allocations(vgpu, vpmem..) into the filesystem under the dir "instance_base_path + '_resize'"...
07:25:15	alex_xu	dansmith: efried or we just auto confrim that same host resize after the host back, and cleanup other free vpmem.
08:48:15	openstackgerrit	ya.wang proposed openstack/nova master: vCPU mdoels selection https://review.opendev.org/670298
08:48:16	openstackgerrit	ya.wang proposed openstack/nova master: Support report multi CPU model traits https://review.opendev.org/670300
08:48:16	openstackgerrit	ya.wang proposed openstack/nova master: Add compatibility checks for CPU mode and CPU models and extra flags https://review.opendev.org/670299
08:48:17	openstackgerrit	ya.wang proposed openstack/nova master: Add release note https://review.opendev.org/670441
09:05:44	openstackgerrit	ya.wang proposed openstack/nova master: Add compatibility checks for CPU mode and CPU models and extra flags https://review.opendev.org/670299
09:05:44	openstackgerrit	ya.wang proposed openstack/nova master: vCPU model selection https://review.opendev.org/670298
09:05:45	openstackgerrit	ya.wang proposed openstack/nova master: Add release note https://review.opendev.org/670441
09:05:45	openstackgerrit	ya.wang proposed openstack/nova master: Support report multi CPU model traits https://review.opendev.org/670300
09:21:59	mkrai	mgoddard, Hi Mark! Can you please take a look at the reconfigure spec whenever you've time? https://review.opendev.org/#/c/672252/
10:58:23	openstackgerrit	Huachang Wang proposed openstack/nova-specs master: Use PCPU and VCPU in one instance https://review.opendev.org/668656
11:15:38	openstackgerrit	Huachang Wang proposed openstack/nova-specs master: Use PCPU and VCPU in one instance https://review.opendev.org/668656
11:52:53	kashyap	alex_xu: When you get a min, want to put this through? This fixes a couple of bugs: https://review.opendev.org/#/c/348394/10/
11:54:48	openstackgerrit	Merged openstack/nova master: Add the support of CPU feature 'AVX512-VNNI' https://review.opendev.org/673480
12:08:13	gibi	kashyap: all those nice emails waiting for me...
12:08:24	gibi	kashyap: have a nice vacation!
12:09:16	kashyap	Thanks!
12:20:07	openstackgerrit	Brin Zhang proposed openstack/nova master: WIP: Filter migrations by user_id/project_id https://review.opendev.org/674243
12:55:11	alex_xu	kashyap: I will try tomorrow morning
12:55:48	aspiers	kashyap: any chance you could review https://review.opendev.org/#/c/673151/ before you go?
12:56:03	aspiers	kashyap: I mean, finish reviewing it. It blocks the entire SEV series :-(
12:56:22	aspiers	welcome back gibi :)
12:56:44	kashyap	alex_xu: Sure
12:56:48	kashyap	aspiers: Looking
12:57:08	gibi	aspiers: thanks :)
12:58:32	kashyap	aspiers: I had opened it, and apparently have a little draft (nothing substantial) to your reply. Will hit send today
13:00:15	aspiers	kashyap: thanks. If at all possible I would really like to get your +1 before you go
13:00:25	kashyap	aspiers: Yes, doing that
13:03:40	kashyap	aspiers: A silly XML question
13:05:13	aspiers	?
13:06:02	kashyap	aspiers: http://paste.openstack.org/show/755517/
13:06:12	kashyap	Thought it's better to use the pastebin
13:07:42	aspiers	c.text
13:07:50	aspiers	c.attrib is for the ... attributes :)
13:07:53	kashyap	Err, indeed. What am I smoking
13:08:01	aspiers	also, don't use getchildren
13:08:25	kashyap	.text holds the additional data...
13:10:22	kashyap	aspiers: Oh, I recall you had a patch to replace that, because of new XML library changes
13:11:11	kashyap	Right, Element.getchildren() is deprecated
13:11:31	aspiers	I haven't done it yet
13:11:35	aspiers	but I will
13:15:23	kashyap	Nod
13:32:23	kashyap	aspiers: Is this readable to you? :D
13:32:24	kashyap	elif c.tag == "enum" and c.attrib["name"] == "firmware":
13:32:25	kashyap	if child.tag == "value"]
13:32:25	kashyap	self.firmwares = [child.text for child in c
13:32:25	kashyap	firmwares = []
13:33:04	aspiers	yes but couldn't it cause a KeyError if there's an <enum> without a 'name' attribute?
13:33:17	aspiers	which in theory shouldn't happen, but ya know, defensive programming and all that
13:33:32	aspiers	this is probably the kind of thing sean-k-mooney was talking about
13:33:35	kashyap	aspiers: Hmm, possibly. How do you guard?
13:33:45	aspiers	it's just a dict :)
13:33:59	aspiers	c.attrib.get("name")
13:34:29	belmoreira	Hi. I'm testing the nova upgrade from Rocky to Stein. I was trying to do it without control plane interruption but I'm failing. I was hoping that upgrade_levels for the control plane would do it... but no.
13:34:43	aspiers	I think sean-k-mooney would advocate //enum[@name='firmware']/value/text() or whatever
13:34:55	kashyap	aspiers: Nod on the attrib / dict :-)
13:35:04	kashyap	aspiers: Right ... that's the 'xpath' parsing
13:35:44	kashyap	aspiers: I wrote this for QE folks: https://kashyapc.fedorapeople.org/Querying-guest-XML-with-xpath.txt
13:36:04	aspiers	how is xpath different from xq?
13:37:04	belmoreira	afaik this is not supported, but what are we missing?
13:38:13	kashyap	1 sec; call
13:39:53	aspiers	belmoreira: not sure anyone who could answer that is awake yet, unless maybe lyarwood
13:48:44	mriedem	stephenfin: i'm having a hard time reasoning why this libvirt functional test you wrote here https://logs.opendev.org/22/674422/1/check/nova-tox-functional/ddac573/testr_results.html.gz is failing on a simple optimization change here https://review.opendev.org/#/c/674422/
13:48:52	mriedem	looks like the test expects something to fail which isn't now
13:49:16	mriedem	so it's hitting a StopIteration on a mocked call to open a console file for the guest when spawning it for the 3rd time (1 create, 2 rebuilds)
13:54:16	Rubueno	Hi, quick question. We're running OS in accp and prod environments, currently running the same version. Today the nova api returns an unexpected api error in accp when calling "nova list" or "openstack server list --all". It asks to report it at launchpad, just wondering what temlate to follow and whether there's anything I can do at this moment.
13:56:20	Rubueno	"Unexpected exception in API method: TypeError: __init__() takes at least 4 arguments (2 given)"
13:57:10	kashyap	aspiers: BTW, earlier, c.get() does it; no need for c.attrib.get() :-) -- you know why
13:58:11	aspiers	kashyap: oh yeah, that's even better
14:00:46	aspiers	kashyap: just to be 100% clear before you vanish for a significant length of time, you're intending to reply to https://review.opendev.org/#/c/673151/ later today right? If you have any nits which you want fixing, I really want to get those done before you go so that you can give an honest +1 :) Not trying to pester as I'm around all day - just making sure :)
14:01:43	kashyap	aspiers: Writing right now
14:01:48	aspiers	awesome, thanks!
14:03:01	kashyap	aspiers: Also, note, don't worry - I've been looking at this for a couple of days, not doing it in a hurry :-)
14:03:08	aspiers	cool
14:03:50	kashyap	aspiers: Also, congrats ... you are one of the few people who can distinguish the getCapabilities() and getDomainCapabilities() at feature-level :D
14:04:01	aspiers	heh
14:04:31	aspiers	Well they're kind of confusing
14:05:29	alex_xu	mriedem: do you want to give a help on this https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:claim_for_instance? I remember you reviewed the original vgpu code also

Earlier   Later