Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-23
22:07:32 openstackgerrit Sujitha proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228
22:08:26 mriedem Apoorva_: have you diffed the _connect_volume method to see what changed the signature?
22:08:43 Apoorva_ lyarwood mriedem would this https://review.openstack.org/#/c/437665/ cause Cinder nfs based driver connections to fail?
22:08:53 mriedem on
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

Earlier   Later