Posted	Nick	Remark
#openstack-nova - 2018-04-20
06:04:11	phuongnh	phuongnh
06:38:39	openstackgerrit	Bhagyashri Shewale proposed openstack/nova master: libvirt: get_inventory => update_provider_tree https://review.openstack.org/560444
06:38:40	openstackgerrit	Bhagyashri Shewale proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459
07:05:52	openstackgerrit	Tetsuro Nakamura proposed openstack/nova-specs master: Return resources of entire trees in Placement https://review.openstack.org/559466
08:32:47	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
08:32:48	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
08:57:22	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
09:05:12	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
09:05:13	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
09:05:14	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
09:05:15	openstackgerrit	jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
09:08:18	openstackgerrit	sahid proposed openstack/nova master: libvirt: don't set interface backend for vhostuser https://review.openstack.org/562708
09:08:19	openstackgerrit	sahid proposed openstack/nova master: libvirt: fix wrong driver name for vhostuser interface https://review.openstack.org/561562
09:08:20	openstackgerrit	sahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997
09:20:03	sahid	jaypipes, dansmith, can i ask you to have a look at the serie https://review.openstack.org/#/q/topic:bp/libvirt-virtio-set-queue-sizes+(status:open+OR+status:merged) ?
09:20:13	sahid	i did not internal tests to ensure everything is working
09:59:57	openstackgerrit	Chris Dent proposed openstack/nova master: Use nova.db.api directly https://review.openstack.org/543262
10:09:35	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Do not use SameHostFilter in APi sample tests https://review.openstack.org/563037
10:10:41	kashyap	Anyone familiar with the IBM zVM CI/
10:10:43	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Do not use SameHostFilter in APi sample tests https://review.openstack.org/563037
10:13:56	artom	kashyap, Andreas Scheuring is the one posting recheck-zkvm, usually
10:14:36	kashyap	artom: I see, trying to figure out why it's failing here: https://review.openstack.org/#/c/558783/11
10:17:07	artom	kashyap, are we looking at the same patch/CI?
10:17:19	artom	IBM zKVM CIApr 19 7:53 AM
10:17:19	artom	check-tempest-dsvm-neutron-full-ubuntu-xenial-s390xSUCCESS in 1h 45m 53s
10:17:41	artom	Doh, zvm != zkvm
10:17:57	kashyap	Stung by CI!
10:18:06	kashyap	Sorry
10:18:59	artom	kashyap, 2018-04-19 11:10:56.583 ERROR nova.compute.manager [instance: b1df65c5-f5cb-47bc-86db-54269c141a77] ZVMDriverException: zVM Cloud Connector request failed
10:19:00	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
10:19:09	artom	In http://extbasicopstackcilog01.podc.sl.edst.ibm.com/test_logs/jenkins-check-nova-master-16984/logs/n-cpu.log
10:19:14	kashyap	Saw that. artom -- it's not my patch's problem, is it
10:19:20	artom	Doesn't look related to your patch
10:19:28	kashyap	Phew, thanks for looking
10:19:43	kashyap	I'm just going a little mental at looking all the non-Zuul CI stuff that's failing
10:19:52	artom	I mean, I know fsck all about zvm
10:19:54	kashyap	And ending up at the conclusion that it's unrelated to what I'm doing
10:20:01	kashyap	:D
10:20:03	artom	But that looks like our "can't connect to libvirt" error message
10:23:02	openstackgerrit	Tovin Seven proposed openstack/python-novaclient master: Trivial: Update pypi url to new url https://review.openstack.org/563060
10:23:18	kashyap	Oh
10:28:10	openstackgerrit	Merged openstack/nova stable/ocata: Handle spawning error on unshelving https://review.openstack.org/548622
10:29:04	openstackgerrit	Merged openstack/nova stable/ocata: Increase cpu time for image conversion https://review.openstack.org/562145
10:38:40	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
10:38:41	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
10:42:24	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: WIP: complex policy https://review.openstack.org/553776
11:06:50	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
11:29:00	openstackgerrit	Chris Dent proposed openstack/nova master: Fix link in placement contributor doc https://review.openstack.org/563086
11:29:19	cdent	stephenfin, fried_rice : trival doc fix ^
11:29:28	fried_rice	...
11:30:59	fried_rice	cdent: Either that should have failed sphinx build previously, or there's a dummy target in the doc somewhere
11:31:09	fried_rice	...but I can't find the latter; and clearly the former didn't happen
11:31:12	fried_rice	So /me confused.
11:31:14	cdent	yeah, I dunno
11:31:45	cdent	I figured I would just fix it
11:32:23	cdent	fried_rice: might have been before warnings were made errors?
11:32:51	fried_rice	That's been a while, and this was recent.
11:33:03	fried_rice	Maybe warnings-as-errors is only in place for the placement docs?
11:35:10	fried_rice	indeed 'unknown target name' before this fix.
11:35:16	fried_rice	(in my IDE)
11:35:18	fried_rice	building...
11:37:06	cdent	which IDE do you use fried_rice ?
11:37:11	fried_rice	pycharm
11:37:41	fried_rice	fyi, openstack has licenses for the pro edition
11:38:05	fried_rice	though tbh I haven't noticed much of a difference. (I probably just don't use the features that are enabled in pro)
11:40:51	cdent	I'm old and inelastic enough that the I want to carry on using vim until I die. I was surveying out of curiosity, not seeking
11:42:34	cdent	"that the I want" don't know what I was trying to say there
11:44:05	fried_rice	cdent: Do you have a bunch of customizations and stuff in your vim?
11:44:13	fried_rice	Anything python-specific beyond syntax coloring?
11:45:07	openstackgerrit	Surya Seetharaman proposed openstack/nova master: Delete orphaned RS and IM records of archived instances https://review.openstack.org/560042
11:45:43	openstackgerrit	Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802
11:46:15	cdent	no, not really. I'm not much of a customizer. And if it wasn't there already in a sort of semi-automatic fashion, I'd not turn on the syntax coloring. The main things I care about are auto-indent and paren matching. And since nova uses such a bad indent style, I'm not getting much help from my editor on that front (I could fix my editor to behave better, but haven't got around to it)
11:49:10	fried_rice	cdent: I love vim just fine, but IDE ability to jump from a symbol to its definition, find all usages of a method, jump directly to method overrides, search for symbols by (wildcarded) name, see class/method/attribute overview trees, etc. -- all pretty serious productivity boosters for me.
11:49:28	cdent	tags
11:50:01	fried_rice	There was a time I used to be able to do interactive debugging, but it didn't (used to) work in the nova project, and I haven't tried it recently or tried to fix it.
11:50:24	fried_rice	Do you tag the whole project?
11:50:47	cdent	fried_rice: do you ever wonder or worry if IDE superpowers can result in code that is harder for other people to read/maintain?
11:51:08	cdent	when I use them, yes, but I frequently don't
11:51:14	cdent	because I don't need/want to
11:51:51	fried_rice	cdent: You mean because I'd be less inclined to write new stuff in sensible locations because I know I can always navigate by hotkey?
11:52:37	cdent	not just that, not just about navigation, and not really anything in particular: more just "do you think about that?"
11:52:41	fried_rice	Given formatting restrictions (which pycharm also validates on the fly) I'm not sure what other pitfalls there might be.
11:52:52	cdent	me neither, just noodling
11:53:11	fried_rice	And given our review process, I would expect people to gripe if stuff was in the wrong place.
11:53:38	cdent	Yeah, I'm not really thinking about issue of where code is
11:53:55	cdent	More about the actual code itself: how it is written, how it is structured internally to itself
11:54:23	fried_rice	Mm. No, I don't feel like I would do any of that differently in a primitive editor.
11:54:56	fried_rice	If code generation were a thing, I could see that being an issue (though I would also expect that to get filtered by reviewers)
11:55:19	fried_rice	So like, if this was Java, and I was using "generate a bean" or whatever.
11:56:07	fried_rice	But - whether because I just don't know about / use the code generation features, or they don't exist, or they wouldn't apply to the kind of dev we do in openstack, or whatever - I don't use any code gen tools in my IDE.
11:56:59	fried_rice	btw, I tried quick-building that doc and apparently my venv wasn't set up, so I'm going through tox.... and the venv is *still* installing deps.
11:57:22	fried_rice	Which says to me that we're installing the wrong set of deps for building docs.
11:58:04	fried_rice	I think stephenfin has some patches that clean some of that up. But I think they just consolidate some of the venvs. Not sure whether docs was folded into one of the others.
11:58:37	cdent	the gate now builds docs outside of the 'docs' venv that can be used locally
11:59:00	cdent	but yes, the deps are probably wrong. probalby building nova deps, not docs deps
12:04:38	openstackgerrit	sahid proposed openstack/nova master: libvirt: don't set interface backend for vhostuser https://review.openstack.org/562708
12:04:39	openstackgerrit	sahid proposed openstack/nova master: libvirt: fix wrong driver name for vhostuser interface https://review.openstack.org/561562
12:04:40	openstackgerrit	sahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997
12:06:27	sahid	giblet: i added a test i hope it's what you were looking for
12:11:15	frickler	melwitt: gah, if you had created a bug for https://review.openstack.org/561370 it might have saved me searching half a day to come to the same result. but then maybe not, so no hard feelings ;)

Earlier   Later