Posted	Nick	Remark
#openstack-nova - 2018-03-05
17:09:28	stephenfin	That's not a bad idea
17:09:34	mriedem	it's a *great* idea
17:09:46	gibi	mlavalle: I have to run now, talk to you later
17:26:08	Spazmotic	whew, third shifters shouldn't be up t his late
17:59:30	jgriffith	mriedem: hey.. there's a little bit of a snag in the plan
17:59:52	jgriffith	mriedem: it seems the attached_host field wasn't being translated into the viewbuilder from cinder :(
18:00:15	jgriffith	Not sure if it was dropped somewhere or if we just missed it altogether
18:00:29	jgriffith	mriedem: easy enough to add, but then we get into the whole versioning debate
18:00:43	mriedem	jgriffith: we don't want to rely on the attached_host field,
18:00:53	jgriffith	mriedem: oh
18:01:01	jgriffith	well then I suppose that's good news :)
18:01:02	mriedem	because we discussed at the ptg that we need to control that via policy rule, and default that to admin-only
18:01:37	jgriffith	mriedem: ok... so on the disconnect, I have a list of attachment-ids
18:01:50	jgriffith	and of course can get the list of attachments from said id's
18:02:06	mriedem	jgriffith: so i think the flow is: get volume details, from the volume['attachments'], get the list of server uuids attach to that volume, get the list of instances on that host (InstanceList.get_uuids_by_host - something like that), and then cross-reference those 2 lists
18:02:11	jgriffith	so are you suggesting then iterate through bdms?
18:02:19	mriedem	no i don't think we need bdms
18:02:33	jgriffith	mriedem: ahh
18:02:41	mriedem	i think we just need to see if there are >1 servers attached to the same volume, and if >1 of those servers are on *this* host
18:02:42	jgriffith	I think I see what you're getting at
18:02:46	mriedem	if so, don't call disconnect
18:03:16	jgriffith	Ok, I think that will work; let me try and put it all together, thanks!
19:03:50	openstackgerrit	Matt Riedemann proposed openstack/nova master: Make nova-cells-v1 run with neutron https://review.openstack.org/549789
19:43:59	openstackgerrit	Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Localdisk https://review.openstack.org/549300
19:58:31	openstackgerrit	Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862
20:06:42	openstackgerrit	Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Localdisk https://review.openstack.org/549300
20:19:57	openstackgerrit	John Griffith proposed openstack/nova master: Check for multiattach before removing connections https://review.openstack.org/548427
20:21:40	jgriffith	mriedem: ^^ I should have the unit tests finished up here when I get back to my desk after a bit, but just want to make sure you and others ( stvnoyes ) are good with the approach
20:21:48	jgriffith	bb in a couple hours :(
20:31:07	mriedem	ok
21:02:25	openstackgerrit	Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094
21:16:49	mriedem	appears that CI is borked,
21:16:56	mriedem	seeing RETRY_LIMIT on a most jobs on most patches
22:06:34	openstackgerrit	Eric Berglund proposed openstack/nova master: PowerVM Driver: DiskAdapter parent class https://review.openstack.org/549053
22:38:54	openstackgerrit	Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Network interface attach/detach https://review.openstack.org/546813
23:02:04	openstackgerrit	John Griffith proposed openstack/nova master: Check for multiattach before removing connections https://review.openstack.org/548427
23:34:06	openstackgerrit	John Griffith proposed openstack/nova master: Check for multiattach before removing connections https://review.openstack.org/548427
23:50:09	openstackgerrit	Matt Riedemann proposed openstack/nova master: Make the nova-next job voting and gating https://review.openstack.org/549893
#openstack-nova - 2018-03-06
01:02:52	Spazmotic	Well at least I got SOME sleep anyway.. Morning Novaers
01:12:47	openstackgerrit	John Griffith proposed openstack/nova master: Check for multiattach before removing connections https://review.openstack.org/548427
01:31:49	Kevin_Zheng	gibi Thanks alot for the update
01:32:22	Kevin_Zheng	mriedem I got kicked everyday T_T
01:43:14	Kevin_Zheng	gibi what about fault and traceback in notification?
03:13:57	Spazmotic	I want to do more XenAPI devvvvv
06:19:40	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772
06:51:09	openstackgerrit	Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805
07:50:19	toabctl	melwitt, could you have a look at https://review.openstack.org/#/c/548572/ please? I think this is really something that we want backported to Queens...
07:57:28	openstackgerrit	Zhenyu Zheng proposed openstack/nova-specs master: Add extra-specs to the flavor show and detail API calls https://review.openstack.org/544158
08:15:31	openstackgerrit	Zhenyu Zheng proposed openstack/nova-specs master: Add extra-specs to the flavor show and detail API calls https://review.openstack.org/544158
08:24:00	openstackgerrit	Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805
08:50:29	openstackgerrit	sahid proposed openstack/nova-specs master: update: introducing isolate emulthreads on host https://review.openstack.org/511188
08:55:29	gibi	Kevin_Zheng: regarding fault and traceback in notification I still have a todo to compare the content of the legacy and versioned notifications and if they are different then do some digging why we did it that way
08:55:57	gibi	Kevin_Zheng: and if they are different then I will file a bug against the versioned one
08:56:14	Kevin_Zheng	gibi Thanks, our product seems to be also interested in those
08:57:25	gibi	Kevin_Zheng: give me couple of days. I will try to progress with this later this week
08:57:37	Kevin_Zheng	gibi cool, thanks alot
08:58:14	gibi	Kevin_Zheng: meanwhile any review on https://review.openstack.org/#/c/502306/ is highly appreciated.
08:59:06	Kevin_Zheng	gibi, sure, will do
09:07:22	bauzas	good chilly morning
09:15:17	openstackgerrit	Radoslav Gerganov proposed openstack/nova master: VMware: fix TypeError while get console log https://review.openstack.org/549182
09:25:33	openstackgerrit	sahid proposed openstack/nova-specs master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/539605
10:10:03	openstackgerrit	Tetsuro Nakamura proposed openstack/nova-specs master: Support shared/dedicated vCPUs in one instance https://review.openstack.org/545734
10:17:23	Spazmotic	Morning UK nova team
10:17:40	Spazmotic	Welcome home and all that jazz :)
10:18:02	Spazmotic	UK/EU/Europe/Not America rather
11:25:21	openstackgerrit	Tetsuro Nakamura proposed openstack/nova-specs master: Enable NUMA Features for Libvirt/QEMU Driver https://review.openstack.org/533077
11:27:50	openstackgerrit	Pranab proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
11:38:34	sean-k-m1	stephenfin: https://review.openstack.org/#/c/546588/1 looking at this i think we are ok to merge this in master however back porting to ocata may be an issue if the bug with adding interfaces to the datapath is only fixed in ovs 2.6
11:39:29	stephenfin	sean-k-mooney: That's a good point. I wonder how old 2.6.0 is?
11:39:56	stephenfin	sean-k-mooney: Although, to be fair, I think that bugfix is broken, given that it introduces another, potentially more serious bug
11:40:08	sean-k-mooney	stephenfin: september 2016
11:41:08	sean-k-mooney	well both bugs are resolved with a vm hard reboot so i dont think this new bug is more serious
11:41:26	sean-k-mooney	they both resolt in the same user observable behavior
11:41:32	stephenfin	sean-k-mooney: So Newton was released 6 October 2016 (per Wikipedia). I wonder if it's fair to say anyone using...Ocata and upwards would be using OVS 2.6+ ?
11:41:51	stephenfin	We could include a release note to that effect (do we do release notes for os-vif?)
11:42:06	sean-k-mooney	i belive neutron has a minium version for ovs as we have for libvirt
11:42:37	sean-k-mooney	well what im considering is should we confige this change to only when plugging a dpdkvhostuser port
11:43:19	sean-k-mooney	s/confige/confine/
11:43:23	stephenfin	Hmm, that could be possible
11:43:47	stephenfin	But if we do, I'd like to remove that check immediately afterwards and not backport that
11:44:13	stephenfin	That could be discussed separately though
11:44:17	sean-k-mooney	stephenfin: well no i would like it in the backport as the old bug was specifically for kernel ovs
11:45:02	stephenfin	You mean you would like the check to be included in the backport?
11:45:03	sean-k-mooney	when https://github.com/openstack/nova/commit/33cc64fb817 was done this fuction did not support vhost user
11:45:09	stephenfin	If so, that's what I mean :)
11:45:24	stephenfin	Add the check, backport that patch, then remove that check in a follow-up patch
11:45:50	sean-k-mooney	ya so in the backport i think we should have the check so that we only change the behavior for vhost user ports
11:45:53	stephenfin	...given that we don't need it anymore (since OVS 2.6.0), and using '--if-exists' seems far saner to me
11:46:05	stephenfin	OK, if that's possible then I've no objections
11:46:09	stephenfin	sahid: Thoughts? ^
11:46:56	sean-k-mooney	stephenfin: did you mean '--may-exists' seams saner?
11:47:03	sean-k-mooney	for master
11:47:54	stephenfin	I did 🙈
11:48:22	stephenfin	i.e. don't special case dpdkvhostuser on master
11:48:23	openstackgerrit	Pranab proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
11:50:47	sean-k-mooney	stephenfin: yes agreed. also i would like to tackover rodolfo patches to add support for using the python binding so that we can stop using the commandline clients
11:51:58	stephenfin	sean-k-mooney: +1 from me. I saw an email on openstack-dev about that earlier this week
11:52:14	stephenfin	I'll review...probably not this week, but definitely next week
11:54:13	sean-k-mooney	ill leave a comment regarding the backport and +w the patch for master.
11:55:22	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486

Earlier   Later