Posted	Nick	Remark
#openstack-nova - 2019-08-22
21:53:05	efried	agreed
21:53:59	mriedem	with that i need to get the hell outside for awhile
21:58:26	sean-k-mooney	thanks and ya sorry my reply were a bit dense on info
22:01:39	sean-k-mooney	melwitt: i can take a look and fix up the backport
22:01:52	openstackgerrit	Eric Fried proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899
22:01:52	openstackgerrit	Eric Fried proposed openstack/nova master: Use SDK for node.list https://review.opendev.org/656027
22:01:53	openstackgerrit	Eric Fried proposed openstack/nova master: Use SDK for validating instance and node https://review.opendev.org/656028
22:01:53	melwitt	sean-k-mooney: thanks!
22:01:53	openstackgerrit	Eric Fried proposed openstack/nova master: Use SDK for setting instance id https://review.opendev.org/659690
22:01:54	openstackgerrit	Eric Fried proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/659691
22:01:54	openstackgerrit	Eric Fried proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/670213
22:03:36	sean-k-mooney	melwitt: im pretty sure that add is missing here https://github.com/openstack/os-vif/blob/stable/stein/os_vif/internal/command/ip/__init__.py
22:04:09	sean-k-mooney	we remove a layer of indirection in train
22:04:26	sean-k-mooney	so ther is 1 addtion plays we need to add the parmater in stien
22:04:26	melwitt	you mean it does not accept a ageing kwarg?
22:04:36	melwitt	oh
22:05:40	sean-k-mooney	correct it does not currently the patch that was backported did not have to update that file because we removed it in master
22:19:31	sean-k-mooney	melwitt: mnaser in attion to adding the aging parmater to 1 more location i will need to update the functional test or backport a second patch
22:19:45	sean-k-mooney	but yes we shoudl be able to make this work
22:20:26	dansmith	mriedem_away: I assume by "outside" you mean either (a) out in super mario world or (b) three feet from your door to plant something
22:21:11	melwitt	super mario world?
22:23:36	dansmith	melwitt: i.e. some video game as opposed to actually going outside
22:24:24	melwitt	haha, ok. that's an old super nintendo game
22:24:40	sean-k-mooney	it is a good game. unfortunetly i dont have my snes anymore. i do have a saga meadrive with sonic plug into by tv
22:25:00	melwitt	I didn't know mriedem_away is a SNES person
22:25:51	melwitt	I liked it ok but not a huge fan of super mario. I prefer rpgs
22:25:54	sean-k-mooney	... tries not to feel old for having fond memories of snes and sage games
22:27:24	melwitt	I <3 snes
22:28:24	openstackgerrit	sean mooney proposed openstack/os-vif stable/stein: only disable mac ageing for ovs hybrid plug https://review.opendev.org/678098
22:28:46	sean-k-mooney	the againg parmater was jsut missing in that one spot after all
22:29:02	sean-k-mooney	but now i need to fiture out why i was getting test failures on rhel8
22:29:37	dansmith	melwitt: yeah, I know.. mriedem_away likes snes, which is why I chose it
22:45:05	sean-k-mooney	efried: if you have the link to the cyborg privsep patch that would be useful i have several cyborg patches added to my review queue and i did comment on there use of privsep i think in the past bug i cant recall which one it was
22:47:35	sean-k-mooney	efried: ah its https://review.opendev.org/#/c/673957/
22:48:15	sean-k-mooney	i was added to it but i have not reviewd it yet ill take a look tomorow.
22:48:53	sean-k-mooney	"This code is modelled strongly on how nova and cinder[0] has chosen
22:48:55	sean-k-mooney	to use privsep.
22:48:57	sean-k-mooney	"
22:49:04	sean-k-mooney	^ that is not a good start.
23:49:27	openstackgerrit	Merged openstack/nova master: Handle websockify v0.9.0 in console proxy https://review.opendev.org/677856
#openstack-nova - 2019-08-23
00:19:12	alex_xu	mriedem_away: thanks :)
00:31:57	mriedem_away	i love SNES
00:32:12	mriedem_away	and am playing FF1 on PS1 lately because
00:32:19	mriedem_away	but i was actually physically at a park with my kid
00:32:24	mriedem_away	with air and sun
00:32:25	mriedem_away	wild
00:38:59	openstackgerrit	Brin Zhang proposed openstack/nova master: Remove the invalid AZ server create test https://review.opendev.org/678116
01:17:24	openstackgerrit	Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
01:58:40	openstackgerrit	Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.opendev.org/612626
02:11:58	openstackgerrit	Merged openstack/nova master: Add docstring to check_availability_zone function https://review.opendev.org/677858
03:30:31	openstackgerrit	Guo Jingyu proposed openstack/nova master: Define and catch InstanceAgentNotRunning for set-password failure https://review.opendev.org/661466
03:38:47	openstackgerrit	zhufl proposed openstack/nova master: [Trivial]Remove used helper get_vif_devname_with_prefix https://review.opendev.org/678136
03:43:25	openstackgerrit	Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
03:54:56	openstackgerrit	Merged openstack/nova master: objects: Rename 'nova.objects.instance_numa_topology' https://review.opendev.org/671789
03:55:04	openstackgerrit	Merged openstack/nova master: libvirt: Remove unnecessary try-catch around 'getCPUMap' https://review.opendev.org/671790
03:55:13	openstackgerrit	Merged openstack/nova master: libvirt: Remove unnecessary argument https://review.opendev.org/674396
03:55:23	openstackgerrit	Merged openstack/nova master: claims: Remove useless caching https://review.opendev.org/671791
05:40:17	openstackgerrit	zhufl proposed openstack/nova master: [Trivial]Remove unused helper get_vif_devname_with_prefix https://review.opendev.org/678136
06:16:03	openstackgerrit	Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476
06:19:51	openstackgerrit	Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476
07:41:13	AJaeger	nova stable cores, could you review https://review.opendev.org/677178, https://review.opendev.org/677176 https://review.opendev.org/677177 ? Those remove experimental openSUSE 42.3 jobs since the distro is end-of-life now.
07:43:55	lyarwood	AJaeger: ack, I'll take a look now
07:45:08	lyarwood	AJaeger: nothing on stable/stein?
07:45:36	AJaeger	lyarwood: no, that one uses the newer opensuse-150 job already
07:45:45	AJaeger	thanks, lyarwood
07:46:27	lyarwood	Ib0b49fead613d06001df348831c022edfe8d0fd1 - got it, thanks
08:47:28	openstackgerrit	Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476
08:51:44	cervigni	hello
08:51:52	cervigni	I am playing around with vgpu
08:52:11	cervigni	It seems that the nova scheduler does not assign properly the cpu-> gpu
08:52:33	cervigni	essentially the first vgpu that is always created by the scheduler, goes to my 2nd gpu
08:52:46	cervigni	while the cores are assigned to first cpu (first numa node)
08:53:22	cervigni	is there a way to tweak the scheduling algorithm or to specify somehow that there is an affinity between CPU and VGPUs ?
08:53:52	cervigni	as it stands, instances always get first cpus / second gpu
08:54:17	cervigni	therefore there is a big numa issue and performance penality where instances are communicating with the "opposite" numa node in the other cpu socket
08:56:15	cervigni	it seems to me a real nova scheduling issue, but I would like someone to confirm that is the case before opening a bug report
08:56:24	openstackgerrit	Mitsuhiro Tanino proposed openstack/nova master: Smaller project quota value than user quota value can't be set https://review.opendev.org/678183
08:57:35	cervigni	GPU 0 has a pci id lower than GPU 1, but for reasons I don't understand, the VGPUs are always first assigned to GPU1
08:57:39	cervigni	|===============================+======================+======================|
08:57:39	cervigni	| 0 Tesla V100-PCIE... On | 00000000:37:00.0 Off | Off |
08:57:39	cervigni	| N/A 38C P0 27W / 250W | 39MiB / 16383MiB | 0% Default |
08:57:39	cervigni	+-------------------------------+----------------------+----------------------+
08:57:39	cervigni	| 1 Tesla V100-PCIE... On | 00000000:86:00.0 Off | Off |
08:57:39	cervigni	| N/A 35C P0 29W / 250W | 51MiB / 16383MiB | 0% Default |
08:57:42	cervigni	+-------------------------------+----------------------+----------------------+
08:59:02	openstackgerrit	Mitsuhiro Tanino proposed openstack/nova master: Smaller project quota value than user quota value can't be set https://review.opendev.org/678183
09:05:25	openstackgerrit	Mitsuhiro Tanino proposed openstack/nova master: Smaller project quota than user quota shouldn't be set https://review.opendev.org/678183
09:53:31	openstackgerrit	Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.opendev.org/612626
10:48:52	stephenfin	alex_xu: You should probably look at https://review.opendev.org/#/c/674894/ given your comments at https://review.opendev.org/#/c/664420/42/nova/scheduler/request_filter.py@206
11:18:52	efried	sean-k-mooney: I agree. I skimmed over your review last night, sounds like what I was expecting.
11:19:28	sean-k-mooney	efried: i might try and submit a basic patch to show how to do it as i suggested instead
11:19:41	sean-k-mooney	efried: i assume we are talking about cyborg privsep
11:19:48	efried	yes
11:20:46	sean-k-mooney	i think i was a little harsh in the review but i also have strong feeling about this topic.
11:26:28	efried	Swhy I was hoping you would look at it.
11:47:05	sean-k-mooney	efried: well i review in cyborg octionally and is mostly negitve when i do. not alway but my +/- raition there is proably not that positive. so i do try to make sure my review that i do there are at least constructive rather then saying "no this is all terible" and suggest how it can be improved. but if i do my usuall 50 billion comments on each indiviaual issue that can come across badly if in a
11:47:07	sean-k-mooney	"drive by" review.
11:53:30	openstackgerrit	Eric Fried proposed openstack/nova master: Use SDK for setting instance id https://review.opendev.org/659690
11:53:31	openstackgerrit	Eric Fried proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/659691
11:53:31	openstackgerrit	Eric Fried proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/670213

Earlier   Later