Posted	Nick	Remark
#openstack-nova - 2017-05-23
03:01:24	jichen	mriedem: so some of my colleagues thought the only request is to report 'gate' result while my understanding is we need report in 'check' work
03:01:49	mriedem	jichen: there is no guide really
03:01:59	jichen	mriedem: https://github.com/openstack/nova-zvm-virt-driver is the one we are working on
03:02:18	mriedem	honestly no one upstream is even thinking about adding zvm in tree because it hasn't come up
03:03:10	jichen	mriedem: actually we have been work on open source for a few years but our CI is not ready and need more resource, we knew CI is mandatory so we didn't bring it up until now
03:03:27	mriedem	i think you might have asked me about this in barcelona? or someone did in atlanta?
03:03:34	mriedem	the zHPM (?) thing also came up
03:03:39	mriedem	which is a different team
03:03:52	mriedem	and zKVM still never started any CI, which they talked about doing since Paris
03:04:12	jichen	yes, they are from India team and you can think they zHPM are barematel on z
03:04:52	mriedem	jichen: well, nothing is going to happen in pike,
03:04:59	jichen	there are 3 different hypervisors z/VM is old one and last for some big customers like ADP etc, zKVM is a new one and libvirt based , DPM (zHPM) is more like a barematel thing
03:05:17	mriedem	i think we've previously told z driver people and huawei fusionsphere people to follow what the powervm team did,
03:05:25	jichen	mriedem: yes, I know, so we want to bring this out at next PTG
03:05:56	jichen	mriedem: ok, for the CI , we need to be as good as or better to PowerVM, need 'check' for every patch, right?
03:06:36	mriedem	i'd need to ask thorst or efried what they were doing, but i thought their ci was pretty solid on their out of tree driver first, including on check queue changes
03:07:01	mriedem	but their driver is also quite different from the zvm driver i believe, but honestly i don't really know anything about the zvm driver architecture,
03:07:03	mriedem	so i can't really say
03:07:26	mriedem	this probably needs some advertising on the dev ML first
03:07:46	mriedem	if you guys have summit sessions / slides / whatever that might be a good start
03:08:38	jichen	ok, we do have some thing there, I already drafted a WIP spec there, I will try to collect whatever info needed and send in the ML
03:08:48	jichen	thanks a lot for your info and guide
03:15:28	mriedem	jichen: sure, sorry i can't be of more help. i'd talk to thorst and the powervm team about their experience.
03:16:51	jichen	mriedem: thanks a lot for your help and we will try to make PTG/summit and show more info there
03:21:59	openstackgerrit	Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995
03:31:12	openstackgerrit	John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595
03:33:18	mriedem	ha ^
03:36:24	mriedem	cburgess: ^
03:36:51	mriedem	cburgess: i think if you can get enough different cloud providers to open separate specs to pass through the volume type, at some point we just have to give in
03:37:15	mriedem	it's in page 45 of the nova core dungeon master's guide
03:50:53	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
04:04:28	openstackgerrit	falseuser proposed openstack/nova master: [BugFix] Change the condition to delete a resizing instance https://review.openstack.org/463466
07:06:19	openstackgerrit	Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995
07:34:25	openstackgerrit	jichenjc proposed openstack/nova master: Deprecate unused policy from policy doc https://review.openstack.org/465505
07:45:58	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
08:03:59	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Amend the "list-show-all-server-migration-types" spec https://review.openstack.org/460847
08:26:25	openstackgerrit	sahid proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820
08:26:25	openstackgerrit	sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514
08:26:26	openstackgerrit	sahid proposed openstack/nova master: network: add command to configure trusted mode for VFs https://review.openstack.org/458513
08:32:29	sfinucan	johnthetubaguy, alex_xu: Could you take a look at [1] as it's blocking 4 other patches [1] https://review.openstack.org/#/c/461630/
08:33:49	sfinucan	oomichi too ^
08:35:29	openstackgerrit	Huan Xie proposed openstack/nova master: XenAPI use os-xenapi v2 in nova https://review.openstack.org/453493
09:32:08	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in os-services.inc https://review.openstack.org/467099
09:47:19	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923
09:54:14	openstackgerrit	Felipe Alfaro Solana proposed openstack/nova-specs master: Implements: blueprint nova-lxd-profiles https://review.openstack.org/467104
09:57:14	openstackgerrit	jichenjc proposed openstack/nova master: Use plain routes list for os-migrations endpoint instead of stevedore https://review.openstack.org/461628
09:59:50	openstackgerrit	jichenjc proposed openstack/nova master: Use plain routes list for os-migrations endpoint instead of stevedore https://review.openstack.org/461628
10:02:09	openstackgerrit	jichenjc proposed openstack/nova master: Use plain routes list for os-services endpoint instead of stevedore https://review.openstack.org/460818
10:09:48	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
10:18:30	openstackgerrit	Vu Cong Tuan proposed openstack/nova master: Repair links in Nova documentation https://review.openstack.org/467114
11:15:36	artom	o_O
11:16:05	artom	https://review.openstack.org/#/c/400763/3 merged without unit tests
11:16:30	artom	Is that legit? We're allowed to do that? I'd have though making sure the exception is caught would be a good test
11:32:54	sfinucan	artom: Hmm, yeah. It should have tests, yes
11:47:48	sfinucan	sdague: If I want to check out a particular commit of nova and run it with Devstack, I simply check it out and reboot all nova services with 'systemctl restart devstack@n-*', right?
11:48:10	sfinucan	(First time tinkering with it since the systemd changes went in)
11:48:21	sdague	yep
11:50:08	sfinucan	cool
12:16:46	openstackgerrit	Feodor Tersin proposed openstack/nova master: Implement ScaleIO image backend https://review.openstack.org/407440
12:17:06	ftersin	mdbooth: hi. did you have a chance to look at ^^^^ ?
12:18:25	mdbooth	ftersin: Only very briefly, I'm afraid. I'm stuck on downstream work right now. It's top of my review list, though.
12:22:46	ftersin	mdbooth: ok, i understand. Probably you might review it by part. E.g. driver.py at the first, or whatever you assume is the most 'suspicious' and requires attention
12:23:14	mdbooth	ftersin: That is indeed where I started :)
12:23:54	ftersin	mdbooth: to let us an opportunity to discuss and to rework complex things not being in rush
12:24:06	ftersin	cool
12:24:07	mdbooth	Incidentally, if the driver could be confined to imagebackend.py I'd be in favour pretty much on your word that it works.
12:24:21	mdbooth	IIRC it's a bit differenty in a couple of places, though.
12:25:22	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/467170
12:29:18	ftersin	mdbooth: i think there are 3 kings of changes in driver.py. the 1st is trivial, i.e. the adding 'sio' into some places where 'rbd', 'lvm', etc already exist
12:30:07	ftersin	the 2nd is intended to map/unmap feature. it cannot be removed but might be reworked
12:30:58	ftersin	the 3rd one are some side changes, which might be separated into different patches or patch series
12:31:21	openstackgerrit	Felipe Alfaro Solana proposed openstack/nova-specs master: Implements: blueprint nova-lxd-profiles https://review.openstack.org/467104
12:31:26	mdbooth	ftersin: Are the side changes generally applicable, or specific to this driver?
12:33:43	ftersin	both. e.g. the fix of swap device is generally applicable, but the filtering of disk_info items (to prevent use them as files) is required for SIO driver
12:34:07	ftersin	however it is required for Cinder driver as well
12:34:20	ftersin	it or some other solution of the problem
12:35:18	mdbooth	ftersin: If you have time, I'd appreciate splitting them out, I think. I'll review without, though.
12:35:43	openstackgerrit	Eric Berglund proposed openstack/nova master: Add strict option to discover_hosts https://review.openstack.org/467218
12:38:21	ftersin	mdbooth: i proposed a series, which resolve that problem separately: https://review.openstack.org/#/q/topic:disk_info_processing
12:50:47	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Exclude deleted service records when calling hyervisor statistics https://review.openstack.org/467220
12:56:39	openstackgerrit	Eric Berglund proposed openstack/nova master: Add strict option to discover_hosts https://review.openstack.org/467218
13:00:26	openstackgerrit	sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514
13:11:44	mriedem	ahoy matey
13:11:49	mriedem	(s)
13:11:57	cdent	is it talk like a pirate day again?
13:12:12	asettle	Yar
13:12:15	asettle	it is
13:12:45	cdent	since I'm in cornwall, I claim authenticity
13:13:08	asettle	Please, please tell me you're in Penzance
13:15:57	mriedem	dansmith: when you're decent i have some questions about your bottom cells change for target_cell
13:19:01	cdent	asettle: sadly right this moment I am in Truro
13:19:54	asettle	Truro is nice!
13:20:02	asettle	But not Penzance
13:30:41	dansmith	mriedem: in the review or you want to ask them here?
13:31:12	pabelanger	morning!
13:31:24	mriedem	dansmith: either
13:31:24	dansmith	mriedem: I still need to follow up with a few more of your earlier comments I think, but removing the object targeting seems to not break anything
13:31:35	mriedem	i just went through PS13
13:31:39	dansmith	mriedem: okay well I'm yet to be caffeinated so give me a few
13:31:45	mriedem	ok

Earlier   Later