Posted	Nick	Remark
#openstack-nova - 2017-03-23
22:08:55	mriedem	*no
22:09:09	openstackgerrit	Sujitha proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228
22:09:23	openstackgerrit	Sujitha proposed openstack/nova master: Change tags to default field in Instance object. https://review.openstack.org/415298
22:09:40	mriedem	Apoorva_: we have an NFS CI job running in the community infra, are you running a 3rd party Cinder NFS driver CI?
22:09:50	mriedem	this is the failure
22:09:56	openstackgerrit	Sujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185
22:09:57	mriedem	TypeError: connect_volume() takes exactly 3 arguments (4 given)
22:10:21	Apoorva_	mriedem our driver inherits nfs driver.
22:10:41	melwitt	I see something, hang on
22:10:48	mriedem	my guess would be https://github.com/openstack/nova/commit/f0153fa4c87528e1dcbda8cba55d34ab7aec4e7a
22:11:33	melwitt	https://review.openstack.org/#/c/438970/
22:11:53	melwitt	yeah, I was trying to find the review
22:12:28	mriedem	yeah, so mdbooth missed the nfs driver
22:12:37	mriedem	in that change,
22:12:43	mriedem	probably because he's handling it in https://review.openstack.org/#/c/383859/ which isn't merged
22:12:46	mriedem	Apoorva_: please open a bug
22:12:59	Apoorva_	mriedem will do that
22:15:10	mriedem	Apoorva_: thanks, post it here and we'll get it fixed
22:17:48	openstackgerrit	Sujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185
22:18:14	openstackgerrit	Sujitha proposed openstack/nova master: Reduce calls to load_tags() to 0 https://review.openstack.org/435146
22:19:00	openstack	Launchpad bug 1675584 in OpenStack Compute (nova) "TypeError: connect_volume() takes exactly 3 arguments (4 given) for NFS Cinder drivers" [Undecided,New]
22:19:00	Apoorva_	mriedem melwitt https://bugs.launchpad.net/nova/+bug/1675584
22:28:00	openstackgerrit	Sujitha proposed openstack/nova master: Add description to policies in cells_scheduler.py https://review.openstack.org/448260
22:31:51	tjones	mriedem: thanks - my schedule is messed up. stupid outloook…. i hate it
22:54:36	mriedem	i also hate outlook
22:54:52	mriedem	Apoorva_: thanks, will push a fix shortly
22:55:20	Apoorva_	mriedem appreciate it!
22:56:51	mriedem	gluster is also broken
22:57:58	mriedem	and quobyte
23:00:07	openstackgerrit	Anusha Unnam proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730
23:00:16	kaisers	mriedem: oh, good this is in the works. Saw the fails earlier and did not come around to take a more detailed look so far. Thanks!
23:00:44	kaisers	*ka-ching* and it's midnight
23:01:46	mriedem	np
23:01:57	mriedem	i'm making these abc's, they should have been anyway
23:04:07	mriedem	i have no idea why scality doesn't implement disconnect_volume
23:10:19	kaisers	hmm, same here. But i'm beyond useful anyways. Off for a few hrs of zzzzZZzz
23:10:23	mriedem	man the scality driver was added 4 years ago
23:25:44	mriedem	weird, the rbd driver doesn't implement connect_volume
23:33:47	openstackgerrit	Matt Riedemann proposed openstack/nova master: Revert "libvirt: Pass instance to connect_volume and ..." https://review.openstack.org/449362
23:34:00	mriedem	melwitt: dansmith: witness https://review.openstack.org/#/c/449362/
23:34:38	dansmith	omy
23:35:00	mriedem	i was fixing it locally but decided, you know what, nothing uses this, it's 6:34pm, and i have other things to do,
23:35:01	mriedem	so fuck it
23:35:31	dansmith	oh I was going to +W it for you but.. I guess you did :)
23:35:42	mriedem	feel free to pile on
23:38:03	mriedem	Apoorva_: https://review.openstack.org/#/c/449362/ is the fix for now
23:39:08	mriedem	oh also the scality volume driver was removed from cinder in ocata,
23:39:20	mriedem	so the libvirt volume driver for scality does nothing right now anyway
23:39:29	mriedem	so i'll follow up on removing that too
23:40:08	Apoorva_	mriedem thanks!
23:41:36	melwitt	revert makes sense to me too
23:47:50	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444
23:55:34	openstackgerrit	Matt Riedemann proposed openstack/nova master: libvirt: remove scality volume driver https://review.openstack.org/449364
23:55:36	mriedem	jgriffith: there you go ^
23:59:25	jgriffith	mriedem you have my 2cents
#openstack-nova - 2017-03-24
00:54:02	openstackgerrit	Matt Riedemann proposed openstack/nova master: Regression test for local delete with an attached volume https://review.openstack.org/449334
01:17:54	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Re-propose nested resource providers spec https://review.openstack.org/449381
01:18:00	mriedem	jaypipes: ^
01:34:41	mriedem	god this is nice http://logs.openstack.org/62/449362/1/check/gate-tempest-dsvm-py35-ubuntu-xenial/ec959b4/logs/screen-n-cpu.txt.gz?level=TRACE#_2017-03-24_00_00_35_648
01:34:45	mriedem	thanks osinfo
01:34:48	mriedem	i get it
01:45:21	openstackgerrit	Danfly proposed openstack/nova master: remove mox from unit/api/openstack/compute/test_disk_config.py https://review.openstack.org/449054
01:47:15	openstackgerrit	Yoshihiko Atsumi proposed openstack/nova master: Add confirm resized server functional negative tests https://review.openstack.org/421074
01:48:20	gmann	Danfly: reviewed this, can you convert those as decorator or self.stub_out - https://review.openstack.org/#/c/445504
02:08:36	openstackgerrit	Matt Riedemann proposed openstack/nova master: Do not attempt to load osinfo if we do not have os_distro https://review.openstack.org/449397
02:48:29	openstackgerrit	ChangBo Guo(gcb) proposed openstack/nova master: Fix wrong unit test about config option enabled_apis https://review.openstack.org/446898
03:20:08	openstackgerrit	Zhou Jianming proposed openstack/nova master: code comments incorrectness https://review.openstack.org/449416
03:56:07	openstackgerrit	Sivasathurappan Radhakrishnan proposed openstack/nova master: Raise general error message when volume attach fails https://review.openstack.org/320163
05:15:14	openstackgerrit	Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_virtual_interfaces.py https://review.openstack.org/448377
05:22:46	openstackgerrit	Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_migrations.py https://review.openstack.org/448370
05:37:33	openstackgerrit	Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_quotas.py https://review.openstack.org/448393
05:46:29	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199
05:46:29	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201
05:47:40	openstackgerrit	Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_virtual_interfaces.py https://review.openstack.org/448377
05:55:43	openstackgerrit	Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_migrations.py https://review.openstack.org/448370
06:00:57	openstackgerrit	lan proposed openstack/os-vif master: Remove log translations https://review.openstack.org/447865
06:16:15	openstackgerrit	Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_migrations.py https://review.openstack.org/448370
06:21:26	openstackgerrit	Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_migrations.py https://review.openstack.org/448370
06:25:02	openstackgerrit	Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/consoleauth/test_consoleauth.py https://review.openstack.org/448386
06:32:26	openstackgerrit	Hironori Shiina proposed openstack/nova-specs master: Fix rendering sub-directories in README https://review.openstack.org/449000
06:43:47	openstackgerrit	Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_quotas.py https://review.openstack.org/448393
06:46:44	openstackgerrit	Huan Xie proposed openstack/nova master: Make xenapi driver compatible with assert_can_migrate https://review.openstack.org/439960
06:47:05	openstackgerrit	Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_virtual_interfaces.py https://review.openstack.org/448377
07:17:16	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix api-ref for adminPass behavior https://review.openstack.org/449450
07:17:43	nkorabli	Hi there, I've got a question related to multitenancy, I believe. I already asked it once, but have some new details..
07:18:33	nkorabli	login as demo user and try to unshelve the instance shelved by admin user
07:18:33	nkorabli	login as a admin user navigate to admin panel and shelve the instance (as admin user is able shelve any instance)
07:18:33	nkorabli	login as demo user and create an instance .
07:18:35	nkorabli	expected : instance should be unshelved
07:18:37	nkorabli	Actual : instance is not shelved but is went to error state. There are two conditions here 1.If this scenarios is not valid admin user should not have an option to shelve the instance , this option should be removed .
07:18:39	nkorabli	2.If this is a valid flow , instance should be unsheleved by the demo user. During the shelve process a snap shot will be created in the instance panel and it will be removed automatically when instance is unshelved. But when admin user is trying to shelve instance a snapshot is created under admin projects instead of demo project . This may be the reason for unshelve failure
07:19:02	nkorabli	So, basically what is a valid workflow for this scenario?
07:34:24	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add description to policies in lock_server.py https://review.openstack.org/448468
07:34:32	openstackgerrit	Tomi Juvonen proposed openstack/nova-specs master: Planned host maintenance https://review.openstack.org/310510
07:43:26	tojuvone	johnthetubaguy, updated the spec: https://review.openstack.org/310510
07:44:30	openstackgerrit	Sivasathurappan Radhakrishnan proposed openstack/nova master: Use mask_dict_password from oslo.utils https://review.openstack.org/388345
07:45:42	openstackgerrit	jichenjc proposed openstack/nova master: refactory some code on get_password https://review.openstack.org/449458
07:46:20	openstackgerrit	jichenjc proposed openstack/nova master: check hw:mem_page_size when change extra_spec https://review.openstack.org/408610
07:47:48	openstackgerrit	jichenjc proposed openstack/nova master: remove keypair server extension https://review.openstack.org/396611
07:48:53	openstackgerrit	Sivasathurappan Radhakrishnan proposed openstack/nova master: Use mask_dict_password from oslo.utils https://review.openstack.org/388345

Earlier   Later