| *** smoriya has joined #openstack-cinder | 00:04 | |
| *** ducttape_ has quit IRC | 00:06 | |
| *** yangyapeng has quit IRC | 00:07 | |
| *** salv-orlando has joined #openstack-cinder | 00:13 | |
| *** baater has joined #openstack-cinder | 00:15 | |
| baater | anyone here | 00:16 |
|---|---|---|
| hemna | Zzzzzzzzz | 00:17 |
| baater | hello | 00:17 |
| *** salv-orlando has quit IRC | 00:17 | |
| baater | i got an issue when i intergrate cinder with ceph. i saw volume log and it always start the lvmdriver, | 00:18 |
| baater | can not boot the rbd drvier | 00:18 |
| hemna | sounds like a cinder.conf issue | 00:18 |
| baater | where is the problem occured? | 00:18 |
| hemna | lvm is the default driver if there is a cinder.conf problem | 00:18 |
| patrickeast | that usually happens if you don't have the enabled_backends thingy setup correctly | 00:19 |
| baater | i put those conf in cinder conf correctly | 00:19 |
| baater | mine is Liberty | 00:19 |
| hemna | baater, pastebin your cinder.conf (scrub passwords...) | 00:19 |
| patrickeast | or with the passwords :D | 00:20 |
| hemna | and IPs and usernames | 00:20 |
| hemna | :) | 00:20 |
| patrickeast | mhmm mhmm | 00:20 |
| baater | please more detail | 00:20 |
| patrickeast | use paste.openstack.org and copy what your cinder.conf looks like into it, we can probably tell you what to change | 00:20 |
| baater | some issue on paste file | 00:21 |
| *** sdake has quit IRC | 00:21 | |
| openstackgerrit | wanghao proposed openstack/python-cinderclient: Remove deprecated tempest_lib and use tempest.lib https://review.openstack.org/318618 | 00:21 |
| *** sdake has joined #openstack-cinder | 00:21 | |
| *** sdake has quit IRC | 00:21 | |
| baater | ok wait a sec | 00:22 |
| *** ozialien10 has quit IRC | 00:23 | |
| *** winston-d_ has joined #openstack-cinder | 00:23 | |
| baater | http://paste.openstack.org/show/497821/ | 00:24 |
| *** enriquetaso has joined #openstack-cinder | 00:25 | |
| hemna | heh | 00:25 |
| hemna | you don't have a volume driver specified | 00:25 |
| baater | yeah i just saw that my mistake hehe | 00:26 |
| baater | damn it my brain | 00:26 |
| baater | anyway many thanks | 00:26 |
| baater | hemna | 00:26 |
| hemna | np | 00:26 |
| *** enriquetaso has quit IRC | 00:27 | |
| patrickeast | you are also going to need to create a [rbd] section for the enabled_backends thing to pick it up | 00:27 |
| patrickeast | check out the example over at http://docs.ceph.com/docs/master/rbd/rbd-openstack/#configuring-cinder | 00:27 |
| hemna | if you use the enabled_backends yah | 00:27 |
| *** salv-orlando has joined #openstack-cinder | 00:27 | |
| patrickeast | either that or remove the enabled_backends section | 00:27 |
| baater | it is not in default section ? | 00:27 |
| patrickeast | you can do it in the default section, but if you do you have to remove the enabled_backends variable | 00:28 |
| baater | should pull out and put it into rbd section ? | 00:28 |
| patrickeast | i like using different sections, but it doesn't really matter much | 00:28 |
| baater | i just affraid that there will be issue when i put that in rbd section | 00:29 |
| *** salv-orlando has quit IRC | 00:30 | |
| *** jamielennox|away is now known as jamielennox | 00:31 | |
| baater | i still got this log 2016-05-19 20:31:16.276 2500 INFO cinder.volume.manager [req-45900df5-0e75-4e8e-ad76-b2faa08fd5e2 - - - - -] Starting volume driver LVMVolumeDriver (3.0.0) | 00:32 |
| *** laughterwym has joined #openstack-cinder | 00:33 | |
| *** laughterwym has quit IRC | 00:34 | |
| *** laughterwym has joined #openstack-cinder | 00:34 | |
| baater | even i removed the lvm section in cinder conf but still not working with rbd driver | 00:36 |
| hemna | pastebin.... | 00:36 |
| hemna | :) | 00:36 |
| baater | i just put the enaled_backend into rbd section it works | 00:39 |
| *** Apoorva has quit IRC | 00:40 | |
| baater | Starting volume driver RBDDriver (1.2.0 | 00:40 |
| baater | i got this | 00:40 |
| hemna | sweet | 00:40 |
| *** itzdilip has quit IRC | 00:41 | |
| baater | i will try to create a volume to chk it is working with rbd | 00:42 |
| *** itzdilip has joined #openstack-cinder | 00:45 | |
| *** Suyi has quit IRC | 00:49 | |
| *** jwcroppe_ has joined #openstack-cinder | 00:55 | |
| *** ozialien10 has joined #openstack-cinder | 00:55 | |
| *** jwcroppe has quit IRC | 00:57 | |
| *** itzdilip has quit IRC | 00:57 | |
| *** cheneydc has joined #openstack-cinder | 00:58 | |
| *** cknight1 has quit IRC | 01:00 | |
| *** bardia has quit IRC | 01:05 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder-specs: Migrate volume between backends in an async way https://review.openstack.org/312853 | 01:07 |
| *** iceyao has joined #openstack-cinder | 01:10 | |
| *** yangyapeng has joined #openstack-cinder | 01:10 | |
| *** raunak has quit IRC | 01:10 | |
| baater | i got issue when i create volume in rbd please help http://paste.openstack.org/show/497824/ | 01:11 |
| *** Apoorva has joined #openstack-cinder | 01:12 | |
| baater | when creating volume with image, image MUST be in the rbd also? | 01:14 |
| baater | my image is qcow2 | 01:14 |
| baater | MUST be raw ??? | 01:14 |
| patrickeast | the logs say its a permission error, from the callstack it seems to indicate a permission issue with rbd stuff, i don't know much about that stuff | 01:24 |
| baater | client.cinder | 01:24 |
| baater | key: AQDsVz1XKJB3ORAAhqnWH+5ktWdpcX1sT93zsg== | 01:24 |
| baater | caps: [mon] allow r | 01:24 |
| baater | caps: [osd] allow object_prefix class-read rbd_children, allow rwx pool=volumes, allow rx pool=vms | 01:24 |
| baater | i run ceph auth list i chk there is cinder user | 01:25 |
| baater | [root@controller ~]# rados --id cinder -p volumes ls | 01:25 |
| baater | rados returned (1) Operation not permitted | 01:25 |
| baater | i run this i got this | 01:25 |
| baater | is this trigger the issue when i create volume? | 01:25 |
| lixiaoy1 | baater: I think so. | 01:26 |
| baater | but i already set this allow rwx pool=volumes, | 01:26 |
| lixiaoy1 | baater: I remember there is a doc in ceph which tells you how to set auth step by step | 01:26 |
| openstackgerrit | KongWei proposed openstack/cinder: Add ZTE Block Storage Driver https://review.openstack.org/312355 | 01:27 |
| lixiaoy1 | baater: http://docs.ceph.com/docs/master/rbd/rbd-openstack/ | 01:29 |
| *** Guest60864 is now known as jgriffith | 01:29 | |
| baater | many thks | 01:30 |
| *** Lee1092 has joined #openstack-cinder | 01:32 | |
| *** EinstCrazy has joined #openstack-cinder | 01:36 | |
| *** salv-orlando has joined #openstack-cinder | 01:37 | |
| openstackgerrit | wanghao proposed openstack/cinder: Clean up image tmp file if c-vol gets restarted https://review.openstack.org/225081 | 01:37 |
| *** yangxi has joined #openstack-cinder | 01:38 | |
| *** itzdilip has joined #openstack-cinder | 01:40 | |
| *** salv-orlando has quit IRC | 01:41 | |
| *** Apoorva has quit IRC | 01:44 | |
| openstackgerrit | Bin Zhou proposed openstack/cinder: add function call chain https://review.openstack.org/317295 | 01:45 |
| openstackgerrit | wanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly https://review.openstack.org/232436 | 01:46 |
| *** jwcroppe_ has quit IRC | 01:51 | |
| *** jwcroppe has joined #openstack-cinder | 01:52 | |
| *** itzdilip has quit IRC | 01:57 | |
| *** gcb has quit IRC | 02:00 | |
| *** EinstCrazy has quit IRC | 02:01 | |
| *** iceyao_ has joined #openstack-cinder | 02:03 | |
| *** yangxi has quit IRC | 02:03 | |
| *** yangyape_ has joined #openstack-cinder | 02:04 | |
| *** EinstCrazy has joined #openstack-cinder | 02:04 | |
| *** yangxi has joined #openstack-cinder | 02:05 | |
| *** iceyao__ has joined #openstack-cinder | 02:05 | |
| *** yangyapeng has quit IRC | 02:06 | |
| *** iceyao has quit IRC | 02:06 | |
| *** iceyao_ has quit IRC | 02:07 | |
| *** itzdilip has joined #openstack-cinder | 02:10 | |
| *** yangyapeng has joined #openstack-cinder | 02:12 | |
| *** gcb has joined #openstack-cinder | 02:15 | |
| *** yangyape_ has quit IRC | 02:16 | |
| *** karthikp has joined #openstack-cinder | 02:23 | |
| *** karthikp_ has joined #openstack-cinder | 02:24 | |
| *** raunak has joined #openstack-cinder | 02:25 | |
| *** jungleboyj has joined #openstack-cinder | 02:32 | |
| *** ducttape_ has joined #openstack-cinder | 02:37 | |
| *** iceyao__ has quit IRC | 02:39 | |
| *** yangxi has quit IRC | 02:39 | |
| *** bill_az has quit IRC | 02:39 | |
| *** yangxi has joined #openstack-cinder | 02:39 | |
| *** iceyao has joined #openstack-cinder | 02:39 | |
| *** houming has joined #openstack-cinder | 02:39 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Remove unused logging import and LOG global var https://review.openstack.org/261472 | 02:43 |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Fix useless api_version of Manager class https://review.openstack.org/309283 | 02:47 |
| *** xyang has joined #openstack-cinder | 02:51 | |
| *** baater has quit IRC | 02:51 | |
| *** karthikp_ has quit IRC | 02:52 | |
| *** iceyao has quit IRC | 02:52 | |
| *** EinstCrazy has quit IRC | 02:52 | |
| *** iceyao has joined #openstack-cinder | 02:53 | |
| *** EinstCrazy has joined #openstack-cinder | 02:53 | |
| *** baater has joined #openstack-cinder | 02:54 | |
| *** dims_ has quit IRC | 02:55 | |
| *** dims has joined #openstack-cinder | 03:00 | |
| *** karthikp has quit IRC | 03:01 | |
| *** karthikp has joined #openstack-cinder | 03:02 | |
| *** gcb has quit IRC | 03:04 | |
| *** gouthamr_ has quit IRC | 03:05 | |
| *** salv-orlando has joined #openstack-cinder | 03:07 | |
| *** salv-orlando has quit IRC | 03:12 | |
| *** xyang has quit IRC | 03:15 | |
| *** salv-orlando has joined #openstack-cinder | 03:16 | |
| *** gcb has joined #openstack-cinder | 03:17 | |
| *** salv-orlando has quit IRC | 03:18 | |
| openstackgerrit | xiexs proposed openstack/python-cinderclient: Fix the incorrect alignment https://review.openstack.org/319038 | 03:26 |
| *** iceyao has quit IRC | 03:28 | |
| *** EinstCrazy has quit IRC | 03:28 | |
| *** yangxi has quit IRC | 03:28 | |
| *** yangyapeng has quit IRC | 03:28 | |
| *** iceyao has joined #openstack-cinder | 03:29 | |
| *** EinstCrazy has joined #openstack-cinder | 03:30 | |
| *** yangyapeng has joined #openstack-cinder | 03:30 | |
| *** yangxi has joined #openstack-cinder | 03:30 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: [WIP] Switch service capabilities to ovo https://review.openstack.org/319040 | 03:39 |
| *** EinstCra_ has joined #openstack-cinder | 03:41 | |
| *** laughterwym has quit IRC | 03:44 | |
| *** laughterwym has joined #openstack-cinder | 03:45 | |
| *** EinstCrazy has quit IRC | 03:46 | |
| *** ducttape_ has quit IRC | 03:48 | |
| *** harlowja_at_home has joined #openstack-cinder | 03:49 | |
| *** laughterwym has quit IRC | 03:49 | |
| *** jungleboyj has quit IRC | 03:55 | |
| *** karthikp has quit IRC | 04:03 | |
| *** dave-mccowan has quit IRC | 04:03 | |
| *** harlowja_at_home has quit IRC | 04:07 | |
| openstackgerrit | xiexs proposed openstack/python-cinderclient: Add max-width feature to limit the width of the printed table https://review.openstack.org/319044 | 04:07 |
| *** asselin__ has joined #openstack-cinder | 04:11 | |
| *** links has joined #openstack-cinder | 04:13 | |
| *** asselin has quit IRC | 04:15 | |
| *** Poornima has joined #openstack-cinder | 04:19 | |
| *** Poornima is now known as Poornima|osp-boo | 04:19 | |
| *** Poornima|osp-boo is now known as Poornima | 04:19 | |
| *** pgadiya has joined #openstack-cinder | 04:23 | |
| *** sdake_ has joined #openstack-cinder | 04:30 | |
| *** xyang has joined #openstack-cinder | 04:35 | |
| *** salv-orlando has joined #openstack-cinder | 04:37 | |
| *** raunak has quit IRC | 04:37 | |
| *** pgadiya is now known as pgadiya|OSPBootC | 04:39 | |
| *** karthikp has joined #openstack-cinder | 04:44 | |
| *** lpetrut has joined #openstack-cinder | 04:50 | |
| *** salv-orlando has quit IRC | 04:53 | |
| *** EinstCra_ has quit IRC | 04:53 | |
| *** EinstCrazy has joined #openstack-cinder | 04:54 | |
| *** karthikp has quit IRC | 05:01 | |
| *** EinstCrazy has quit IRC | 05:03 | |
| *** EinstCrazy has joined #openstack-cinder | 05:04 | |
| *** xyang has quit IRC | 05:04 | |
| *** yangxi has quit IRC | 05:06 | |
| *** lprice has quit IRC | 05:06 | |
| *** Nakato has quit IRC | 05:07 | |
| *** EinstCrazy has quit IRC | 05:07 | |
| *** houming has quit IRC | 05:15 | |
| *** rchurch has quit IRC | 05:19 | |
| *** houming has joined #openstack-cinder | 05:22 | |
| *** iceyao has quit IRC | 05:22 | |
| *** yangyapeng has quit IRC | 05:22 | |
| guy | Hi everyone, is there a minimum requirement for a backup driver for the N release? Thank you. | 05:23 |
| *** iceyao has joined #openstack-cinder | 05:23 | |
| *** yangyapeng has joined #openstack-cinder | 05:23 | |
| openstackgerrit | xiexs proposed openstack/python-cinderclient: Add max-width feature to limit the width of the printed table https://review.openstack.org/319044 | 05:32 |
| *** salv-orlando has joined #openstack-cinder | 05:40 | |
| *** yangyape_ has joined #openstack-cinder | 05:41 | |
| *** yangyapeng has quit IRC | 05:43 | |
| *** houming has quit IRC | 05:43 | |
| *** ChubYann has quit IRC | 05:45 | |
| *** rcernin has joined #openstack-cinder | 05:48 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/318843 | 05:57 |
| *** yangxi has joined #openstack-cinder | 06:01 | |
| *** EinstCrazy has joined #openstack-cinder | 06:06 | |
| *** belmoreira has joined #openstack-cinder | 06:09 | |
| *** salv-orl_ has joined #openstack-cinder | 06:14 | |
| *** mjura has joined #openstack-cinder | 06:14 | |
| *** yangxi has quit IRC | 06:14 | |
| *** yangxi has joined #openstack-cinder | 06:15 | |
| *** salv-orlando has quit IRC | 06:17 | |
| *** lpetrut has quit IRC | 06:18 | |
| *** mjura has quit IRC | 06:19 | |
| *** cheneydc has quit IRC | 06:19 | |
| *** mjura has joined #openstack-cinder | 06:19 | |
| *** cheneydc has joined #openstack-cinder | 06:20 | |
| openstackgerrit | wanghao proposed openstack/cinder: Clean up image tmp file if c-vol gets restarted https://review.openstack.org/225081 | 06:21 |
| *** yangxi has quit IRC | 06:23 | |
| *** david-lyle_ has joined #openstack-cinder | 06:26 | |
| *** yangxi has joined #openstack-cinder | 06:26 | |
| *** mmotiani_ is now known as mmotiani | 06:27 | |
| *** david-lyle has quit IRC | 06:28 | |
| *** boris-42 has quit IRC | 06:28 | |
| *** guitarzan has quit IRC | 06:28 | |
| *** boris-42 has joined #openstack-cinder | 06:29 | |
| *** guitarzan has joined #openstack-cinder | 06:29 | |
| *** yangyape_ has quit IRC | 06:30 | |
| *** Nakato has joined #openstack-cinder | 06:31 | |
| *** al_indigo has quit IRC | 06:32 | |
| *** al_indigo has joined #openstack-cinder | 06:32 | |
| *** iceyao has quit IRC | 06:38 | |
| *** yangxi has quit IRC | 06:38 | |
| *** EinstCrazy has quit IRC | 06:38 | |
| *** yangxi has joined #openstack-cinder | 06:39 | |
| *** iceyao has joined #openstack-cinder | 06:39 | |
| *** EinstCrazy has joined #openstack-cinder | 06:40 | |
| *** adisky has joined #openstack-cinder | 06:41 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Retype encrypted volumes https://review.openstack.org/319078 | 06:45 |
| openstackgerrit | LisaLi proposed openstack/cinder: Retype encrypted volumes https://review.openstack.org/319078 | 06:46 |
| *** salv-orl_ has quit IRC | 06:47 | |
| *** EinstCrazy has quit IRC | 06:48 | |
| *** EinstCrazy has joined #openstack-cinder | 06:48 | |
| *** sgotliv_ has joined #openstack-cinder | 06:52 | |
| *** EinstCra_ has joined #openstack-cinder | 07:04 | |
| *** iceyao_ has joined #openstack-cinder | 07:04 | |
| *** EinstCrazy has quit IRC | 07:07 | |
| *** iceyao has quit IRC | 07:08 | |
| *** realitix has joined #openstack-cinder | 07:11 | |
| realitix | Hi geguileo ! I'm fixing the OverQuota problem (should inherit QuotaError) but I'm not sure if I have to open a launchpad bug on this ? What do you think ? | 07:13 |
| *** nkrinner_afk is now known as nkrinner | 07:14 | |
| *** iceyao has joined #openstack-cinder | 07:16 | |
| *** iceyao_ has quit IRC | 07:20 | |
| *** yangxi_ has joined #openstack-cinder | 07:20 | |
| *** yangxi has quit IRC | 07:24 | |
| *** salv-orlando has joined #openstack-cinder | 07:32 | |
| *** arecknag has joined #openstack-cinder | 07:32 | |
| *** savihou has joined #openstack-cinder | 07:36 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Retype encrypted volumes https://review.openstack.org/319078 | 07:37 |
| *** jamielennox is now known as jamielennox|away | 07:49 | |
| *** markvoelker has joined #openstack-cinder | 07:56 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** yangxi has joined #openstack-cinder | 08:02 | |
| *** yangxi_ has quit IRC | 08:02 | |
| *** markvoelker has quit IRC | 08:05 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| geguileo | realitix: I kind of remember the problem, but would you mind refreshing my memory on the specifics, please? | 08:10 |
| *** lpetrut has joined #openstack-cinder | 08:12 | |
| *** salv-orlando has quit IRC | 08:15 | |
| *** jistr has joined #openstack-cinder | 08:15 | |
| *** salv-orlando has joined #openstack-cinder | 08:16 | |
| realitix | geguileo : I was catching all exceptions in _extend method to return a Bad Request. But you said to me that wsgi handles exception. But you saw that OverQuota exception was not inheriting from QuotaError exception ans you said to me that I can do it. So I don't know if a bug appears because of that bu it allows to be more consistent | 08:18 |
| geguileo | realitix: Ok, I'm looking at the code and I think maybe the reason why it's not inheriting from QuotaError is because that exception was meant to be caught by the caller and converted | 08:25 |
| geguileo | to any of the other exceptions (VolumeLimitExceeded, VolumeSizeExceedsAvailableQuota, etc. | 08:26 |
| geguileo | So it was never meant to go all the way back | 08:27 |
| geguileo | So now I'm not so sure this is a bug... r:-??? | 08:27 |
| geguileo | Or if this should be left as it is | 08:27 |
| geguileo | :-( | 08:27 |
| *** houming has joined #openstack-cinder | 08:34 | |
| realitix | Ok geguileo. Why in extends method, cinder catch InvalidVolume and return a HTTPBadRequest. Maybe it should be like for the others exceptions, handled by the caller ? | 08:36 |
| geguileo | realitix: It should, and I'm working on making the changes to both NotFound and Invalid exceptions | 08:37 |
| geguileo | realitix: Here's the NotFound patch https://review.openstack.org/#/c/314193/ | 08:37 |
| geguileo | realitix: And I have most of the Invalid exceptions patch also done | 08:37 |
| realitix | yes I saw it. | 08:37 |
| realitix | Great | 08:37 |
| geguileo | realitix: But I'm finding difficulties convincing other people that this is OK | 08:38 |
| geguileo | :-( | 08:38 |
| realitix | Why ? | 08:38 |
| geguileo | Because they want more negative tests on the API to confirm everything is OK | 08:38 |
| realitix | I can help with that, how much tests to add ? | 08:39 |
| realitix | how many* | 08:39 |
| geguileo | realitix: No idea how many they want | 08:41 |
| geguileo | realitix: Let me look at where are the negative tests... | 08:42 |
| geguileo | realitix: Ok, here are the volume tests: https://github.com/openstack/tempest/blob/master/tempest/api/volume/test_volumes_negative.py | 08:44 |
| geguileo | And here the snapshots: https://github.com/openstack/tempest/blob/master/tempest/api/volume/test_volumes_snapshots_negative.py | 08:44 |
| realitix | They want tempest tests ? Not only unit ? | 08:44 |
| geguileo | Yep | 08:44 |
| geguileo | We have enough unittests I believe | 08:45 |
| realitix | Ok | 08:45 |
| geguileo | So we would need admin_actions, backups, cgsnapshots, consistencygroups, hosts, qos, services, volume_type... | 08:45 |
| realitix | When I have time, I will add some tests to tesmpest | 08:46 |
| geguileo | realitix: Awesome! | 09:01 |
| *** e0ne has joined #openstack-cinder | 09:06 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Remove unused logging import and LOG global var https://review.openstack.org/261472 | 09:18 |
| *** mvk has joined #openstack-cinder | 09:21 | |
| baater | hello all i hv questions. how does the ceph intergrate with cinder and nove by using iSCSI? tell me about the novaconf and cinderconf . thx | 09:33 |
| *** savihou has quit IRC | 09:33 | |
| *** wanghao has quit IRC | 09:35 | |
| *** wanghao has joined #openstack-cinder | 09:36 | |
| *** houming has quit IRC | 09:40 | |
| openstackgerrit | Lucian Petrut proposed openstack/cinder: Windows iSCSI: properly handle config option https://review.openstack.org/319170 | 09:54 |
| *** dave-mccowan has joined #openstack-cinder | 09:59 | |
| *** yangxi has quit IRC | 10:00 | |
| *** EinstCra_ has quit IRC | 10:01 | |
| *** cheneydc has quit IRC | 10:02 | |
| *** xyang has joined #openstack-cinder | 10:05 | |
| *** xyang has quit IRC | 10:06 | |
| *** iceyao has quit IRC | 10:07 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Minimize cost of DB exists decorator checks https://review.openstack.org/319177 | 10:12 |
| *** sdague has joined #openstack-cinder | 10:14 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fail on disabled_reasons with more than 255 chars https://review.openstack.org/318744 | 10:15 |
| *** baater has quit IRC | 10:24 | |
| *** jwcroppe has quit IRC | 10:26 | |
| *** jwcroppe has joined #openstack-cinder | 10:26 | |
| *** jwcroppe has quit IRC | 10:26 | |
| *** alaski is now known as lascii | 10:43 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Move OVO methods from CinderObject to Persistent https://review.openstack.org/312930 | 10:55 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove remotable from OVOs https://review.openstack.org/312929 | 10:55 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Refactor create, save, and destroy OVO methods https://review.openstack.org/315541 | 10:55 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add auto relationship mappings to VOs https://review.openstack.org/307075 | 10:55 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Link VOs with their list counterparts https://review.openstack.org/307074 | 10:55 |
| *** laurent\_ is now known as laurent\ | 11:01 | |
| *** laurent\ has quit IRC | 11:02 | |
| *** laurent\ has joined #openstack-cinder | 11:02 | |
| dulek | geguileo: I'm not much opinionated on that, but what's the point of having new constraint code added into try-except? | 11:02 |
| dulek | geguileo: https://review.openstack.org/#/c/318744/2/cinder/utils.py | 11:02 |
| geguileo | dulek: New constraint? r:-?? | 11:10 |
| geguileo | dulek: You mean the allow_spaces? | 11:10 |
| geguileo | dulek: In general you want to check the length of a string, but you may want it to not be all blank (like we do in disabled_reason) | 11:11 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from consistency groups https://review.openstack.org/259429 | 11:11 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from migrate and retype https://review.openstack.org/221442 | 11:11 |
| *** zzzeek has quit IRC | 11:12 | |
| openstackgerrit | Kushal Wathore proposed openstack/cinder: Implemented unit test cases for HPE LeftHand driver https://review.openstack.org/319207 | 11:28 |
| openstackgerrit | xiexs proposed openstack/cinder-specs: Enhance error messages for API user https://review.openstack.org/293306 | 11:29 |
| dulek | geguileo: I get the point of the change, I just think that it should go outside of try-except block there. What do you think? | 11:40 |
| geguileo | Oh, let me see | 11:40 |
| geguileo | dulek: Yep, that's a -1 right there | 11:41 |
| geguileo | dulek: I originally intended to raise Value Error and let the except catch it | 11:41 |
| geguileo | But that was absurd | 11:41 |
| geguileo | I'll move it now :-) | 11:41 |
| geguileo | Thanks! | 11:42 |
| dulek | Cool :) | 11:42 |
| nikeshm | hi | 11:43 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fail on disabled_reasons with more than 255 chars https://review.openstack.org/318744 | 11:43 |
| nikeshm | https://github.com/openstack/cinder/blob/stable/kilo/cinder/volume/driver.py#L253 | 11:45 |
| nikeshm | where is default implementation of clone volume | 11:45 |
| *** yhayashi has quit IRC | 11:47 | |
| *** raildo-afk is now known as raildo | 11:55 | |
| *** ducttape_ has joined #openstack-cinder | 11:59 | |
| *** julim has joined #openstack-cinder | 12:02 | |
| *** Lee1092 has quit IRC | 12:02 | |
| *** iceyao has joined #openstack-cinder | 12:05 | |
| *** vnogin has quit IRC | 12:07 | |
| *** sgotliv_ has quit IRC | 12:14 | |
| *** ducttape_ has quit IRC | 12:15 | |
| *** bill_az has joined #openstack-cinder | 12:17 | |
| *** Poornima has quit IRC | 12:23 | |
| *** kfarr has joined #openstack-cinder | 12:24 | |
| openstackgerrit | wanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly https://review.openstack.org/232436 | 12:25 |
| *** salv-orl_ has joined #openstack-cinder | 12:33 | |
| *** salv-orlando has quit IRC | 12:37 | |
| *** jwcroppe has joined #openstack-cinder | 12:41 | |
| e0ne | geguileo: hi | 12:45 |
| geguileo | e0ne: Hi | 12:45 |
| e0ne | geguileo: I've got few questions about A/A HA things | 12:45 |
| geguileo | e0ne: HA A/A or the API races? | 12:45 |
| e0ne | geguileo: both of them | 12:46 |
| geguileo | Ok | 12:46 |
| geguileo | Shoot! | 12:46 |
| e0ne | geguileo: I'm going to test it both for Mitaka and Newton | 12:46 |
| geguileo | :-) | 12:46 |
| e0ne | geguileo: is it documented anywhere what is done and what is still in progress? | 12:46 |
| *** timcl has joined #openstack-cinder | 12:46 | |
| geguileo | e0ne: In the LP, but I have to update it | 12:47 |
| *** marcusvrn_ has joined #openstack-cinder | 12:47 | |
| e0ne | geguileo: it will be awsome, if you'll do it early next week | 12:47 |
| geguileo | Oh, I'll do it right now | 12:47 |
| e0ne | geguileo: I'm preprearing envs for testing | 12:48 |
| geguileo | AWESOME!! | 12:48 |
| *** julim has quit IRC | 12:49 | |
| geguileo | e0ne: Ok, updated | 12:51 |
| *** pgadiya|OSPBootC has quit IRC | 12:51 | |
| e0ne | geguileo: https://blueprints.launchpad.net/cinder/+spec/cinder-volume-active-active-support - this one? | 12:52 |
| geguileo | e0ne: Yes, sorry I forgot to give you the link | 12:53 |
| geguileo | :-( | 12:53 |
| e0ne | geguileo: don't worry. thanks for update | 12:53 |
| geguileo | np | 12:53 |
| geguileo | Thank you for working on the stress tests :-) | 12:54 |
| *** cknight has joined #openstack-cinder | 12:54 | |
| *** savihou has joined #openstack-cinder | 12:54 | |
| *** salv-orl_ has quit IRC | 12:55 | |
| *** julim has joined #openstack-cinder | 12:57 | |
| *** salv-orlando has joined #openstack-cinder | 12:58 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/os-brick: Add support for processutils.execute https://review.openstack.org/309201 | 12:58 |
| *** cknight1 has joined #openstack-cinder | 13:01 | |
| *** darosale has joined #openstack-cinder | 13:02 | |
| *** laughterwym has joined #openstack-cinder | 13:03 | |
| *** pgadiya|OSPBootC has joined #openstack-cinder | 13:04 | |
| *** cknight has quit IRC | 13:04 | |
| *** porrua has joined #openstack-cinder | 13:04 | |
| *** akerr has joined #openstack-cinder | 13:06 | |
| *** kfarr has quit IRC | 13:07 | |
| *** laughterwym has quit IRC | 13:08 | |
| *** darosale has quit IRC | 13:09 | |
| *** xyang1 has joined #openstack-cinder | 13:10 | |
| openstackgerrit | wanghao proposed openstack/python-cinderclient: Remove deprecated tempest_lib and use tempest.lib https://review.openstack.org/318618 | 13:13 |
| *** rlrossit has joined #openstack-cinder | 13:15 | |
| *** bill_az has quit IRC | 13:15 | |
| *** salv-orl_ has joined #openstack-cinder | 13:19 | |
| *** EinstCrazy has joined #openstack-cinder | 13:20 | |
| *** salv-orlando has quit IRC | 13:22 | |
| *** jwcroppe has quit IRC | 13:26 | |
| *** mjura has quit IRC | 13:27 | |
| *** EinstCrazy has quit IRC | 13:28 | |
| *** EinstCrazy has joined #openstack-cinder | 13:29 | |
| *** akshai has joined #openstack-cinder | 13:31 | |
| *** dansmith is now known as superdan | 13:31 | |
| *** gouthamr has joined #openstack-cinder | 13:32 | |
| *** chuck__ is now known as zul | 13:33 | |
| *** salv-orl_ has quit IRC | 13:38 | |
| *** arecknag has quit IRC | 13:43 | |
| *** diablo_rojo has joined #openstack-cinder | 13:43 | |
| *** winston-d_ has quit IRC | 13:45 | |
| *** links has quit IRC | 13:46 | |
| *** smoriya has quit IRC | 13:46 | |
| *** EinstCrazy has quit IRC | 13:47 | |
| *** EinstCrazy has joined #openstack-cinder | 13:48 | |
| *** ducttape_ has joined #openstack-cinder | 13:50 | |
| openstackgerrit | Merged openstack/cinder: Capacity weigher: Pass correct arg to super() https://review.openstack.org/318749 | 13:51 |
| openstackgerrit | Merged openstack/cinder: Remove unused logging import and LOG global var https://review.openstack.org/261472 | 13:51 |
| openstackgerrit | Merged openstack/cinder: Tests: Fix mock imports https://review.openstack.org/318877 | 13:52 |
| openstackgerrit | Merged openstack/cinder: Tests: Don't attempt connection for Coho driver init https://review.openstack.org/317071 | 13:52 |
| *** ametts has joined #openstack-cinder | 13:53 | |
| *** dustins has joined #openstack-cinder | 13:58 | |
| *** nikeshm has quit IRC | 13:59 | |
| *** EinstCrazy has quit IRC | 14:01 | |
| *** timcl has quit IRC | 14:06 | |
| *** timcl has joined #openstack-cinder | 14:06 | |
| openstackgerrit | Alfredo Moralejo proposed openstack/cinder: Add additional translation files for compile_catalog https://review.openstack.org/319265 | 14:07 |
| *** dustins has quit IRC | 14:13 | |
| *** ozialien10 has quit IRC | 14:14 | |
| *** darosale has joined #openstack-cinder | 14:21 | |
| openstackgerrit | Sven Anderson proposed openstack/cinder: Let setup.py compile_catalog process all language files https://review.openstack.org/319276 | 14:22 |
| *** xyang has joined #openstack-cinder | 14:22 | |
| *** akshai has quit IRC | 14:23 | |
| *** xyang has quit IRC | 14:24 | |
| *** raunak has joined #openstack-cinder | 14:24 | |
| *** nkrinner is now known as nkrinner_afk | 14:26 | |
| *** sgotliv_ has joined #openstack-cinder | 14:26 | |
| *** baumann has joined #openstack-cinder | 14:28 | |
| *** gouthamr_ has joined #openstack-cinder | 14:29 | |
| *** akerr_ has joined #openstack-cinder | 14:29 | |
| *** cknight has joined #openstack-cinder | 14:29 | |
| *** cknight1 has quit IRC | 14:29 | |
| *** timcl has quit IRC | 14:29 | |
| *** timcl has joined #openstack-cinder | 14:30 | |
| *** hemna_ has joined #openstack-cinder | 14:30 | |
| *** kfarr has joined #openstack-cinder | 14:30 | |
| *** gouthamr has quit IRC | 14:30 | |
| *** porrua has quit IRC | 14:31 | |
| *** jungleboyj has joined #openstack-cinder | 14:31 | |
| *** akshai has joined #openstack-cinder | 14:31 | |
| *** akerr has quit IRC | 14:31 | |
| *** pgadiya|OSPBootC has quit IRC | 14:32 | |
| *** laughterwym has joined #openstack-cinder | 14:40 | |
| *** jdurgin1 has joined #openstack-cinder | 14:46 | |
| *** diogogmt has joined #openstack-cinder | 14:47 | |
| *** mtanino has joined #openstack-cinder | 14:48 | |
| *** pgadiya has joined #openstack-cinder | 14:49 | |
| *** lprice has joined #openstack-cinder | 14:49 | |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs https://review.openstack.org/261549 | 14:51 |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: add ploop volume format support https://review.openstack.org/262246 | 14:51 |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: fix create/delete snapshots of in-use volumes https://review.openstack.org/276465 | 14:51 |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Use versionedobjects in remotefs.py https://review.openstack.org/261558 | 14:51 |
| *** dillaman has joined #openstack-cinder | 15:02 | |
| *** realitix has quit IRC | 15:04 | |
| *** iceyao has quit IRC | 15:05 | |
| *** hemna_ has quit IRC | 15:05 | |
| *** diogogmt has quit IRC | 15:06 | |
| *** gouthamr_ is now known as gouthamr | 15:06 | |
| dulek | geguileo: There are no downsides of reducing this number of version bumps, right? (I'm just making sure my comment was valid and I'm not breaking something by misunderstanding your intentions). | 15:09 |
| geguileo | dulek: No, it has no real downsides | 15:10 |
| geguileo | As far as I can think of right now | 15:10 |
| geguileo | But maybe a month ago when I did that I thought of something... | 15:10 |
| dulek | Oh, I know that feeling. | 15:11 |
| geguileo | lol | 15:11 |
| dulek | And you don't really remember why you've put them in the first place, right? ;) | 15:11 |
| *** diogogmt has joined #openstack-cinder | 15:11 | |
| *** xyang has joined #openstack-cinder | 15:12 | |
| *** xyang has quit IRC | 15:13 | |
| *** raunak has quit IRC | 15:15 | |
| *** akshai has quit IRC | 15:15 | |
| *** markvoelker has joined #openstack-cinder | 15:18 | |
| *** markvoelker has quit IRC | 15:18 | |
| geguileo | I think I did to makr the synchronization | 15:18 |
| geguileo | But not sure anymore | 15:18 |
| *** markvoelker has joined #openstack-cinder | 15:18 | |
| *** akshai has joined #openstack-cinder | 15:21 | |
| *** belmoreira has quit IRC | 15:28 | |
| *** pgadiya has quit IRC | 15:36 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Minimize cost of DB exists decorator checks https://review.openstack.org/319177 | 15:42 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve DB volume check in LVM manage volume https://review.openstack.org/319323 | 15:42 |
| *** jungleboyj has quit IRC | 15:44 | |
| openstackgerrit | Rajini Ram proposed openstack/cinder: Volume manage/unmanage support for Eqlx driver https://review.openstack.org/318748 | 15:48 |
| *** Nakato has quit IRC | 15:49 | |
| *** rcernin has quit IRC | 15:51 | |
| openstackgerrit | Rajini Ram proposed openstack/cinder: Volume manage/unmanage support for Eqlx driver https://review.openstack.org/318748 | 15:52 |
| *** geguileo has quit IRC | 15:53 | |
| *** markvoelker has quit IRC | 15:54 | |
| *** xinli has joined #openstack-cinder | 15:55 | |
| *** leeantho has joined #openstack-cinder | 15:55 | |
| *** baumann has quit IRC | 15:56 | |
| *** baumann has joined #openstack-cinder | 15:58 | |
| *** sdake_ has quit IRC | 15:59 | |
| *** Apoorva has joined #openstack-cinder | 15:59 | |
| *** porrua has joined #openstack-cinder | 16:00 | |
| *** sdake has joined #openstack-cinder | 16:01 | |
| *** salv-orlando has joined #openstack-cinder | 16:02 | |
| *** darosale has quit IRC | 16:04 | |
| *** timcl has quit IRC | 16:04 | |
| *** raunak has joined #openstack-cinder | 16:05 | |
| *** jdurgin1 has quit IRC | 16:05 | |
| *** salv-orlando has quit IRC | 16:06 | |
| *** e0ne has quit IRC | 16:06 | |
| *** diablo_rojo has quit IRC | 16:09 | |
| *** darosale has joined #openstack-cinder | 16:12 | |
| *** sdake has quit IRC | 16:12 | |
| ildikov | hemna: hey, around? | 16:12 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Switched REST calls to async where available https://review.openstack.org/315226 | 16:14 |
| ildikov | hemna: I played a bit with attaching an existing volume at instance boot and I think that flow does not call reserve | 16:16 |
| *** jungleboyj has joined #openstack-cinder | 16:16 | |
| xinli | @smcgnnis: if code review and Jenkins test passed, but the CI never starts (flashsyst), both me and the component owner rerun the 'run flashci" for 5 times, what can I do? https://review.openstack.org/#/c/307451/6 | 16:17 |
| ildikov | jgriffith: hey | 16:17 |
| jgriffith | ildikov: hi ere | 16:17 |
| jgriffith | there | 16:17 |
| Swanson | xinli, contact the CI owner. | 16:18 |
| ildikov | jgriffith: I just wanted to ask about those patches we talked about on Tuesday :) | 16:19 |
| jgriffith | I'm going to push a series of things up today | 16:19 |
| jgriffith | doing some refactoring now | 16:19 |
| ildikov | cool, I basically wanted to ask whether you need more hands | 16:20 |
| ildikov | but it sounds like good progress :) | 16:21 |
| jgriffith | ildikov: the patch will certainly be a WIP and need help | 16:21 |
| jgriffith | ildikov: it's just a foundation to build on | 16:21 |
| ildikov | jgriffith: sure, I would like to have a reference to put into the multi-attach spec and get that approved before the deadline in two weeks | 16:21 |
| ildikov | jgriffith: and to make myself useful I'm happy to chime in if we can figure out a good setup to not step on each other's toes | 16:22 |
| ildikov | jgriffith: thanks for the update | 16:24 |
| *** hemna_ has joined #openstack-cinder | 16:25 | |
| openstackgerrit | Sven Anderson proposed openstack/cinder: Let setup.py compile_catalog process all language files https://review.openstack.org/319276 | 16:25 |
| *** salv-orlando has joined #openstack-cinder | 16:25 | |
| *** salv-orlando has quit IRC | 16:26 | |
| *** salv-orlando has joined #openstack-cinder | 16:26 | |
| *** diablo_rojo has joined #openstack-cinder | 16:27 | |
| *** cknight has quit IRC | 16:28 | |
| *** timcl has joined #openstack-cinder | 16:30 | |
| Swanson | jgriffith, on failover_host failback (failback to default) is there a timeout on that call? Or didn't you do that bit? | 16:31 |
| jgriffith | Swanson: just the rpc timeout | 16:31 |
| jgriffith | Swanson: IIRC that was an issue for some | 16:32 |
| Swanson | jgriffith, Probably an issue for me. How long is that? | 16:32 |
| jgriffith | Swanson: It's configurable, but I think the default is like 60 or 90 seconds | 16:32 |
| *** raunak has quit IRC | 16:33 | |
| *** cknight has joined #openstack-cinder | 16:33 | |
| Swanson | jgriffith, Didn't someone change that to an async call of some sort? | 16:34 |
| jgriffith | Swanson: could be | 16:34 |
| Swanson | jgriffith, I know where to look now. Thanks! | 16:34 |
| jgriffith | sure | 16:35 |
| openstackgerrit | Carl Pecinovsky proposed openstack/cinder: Avoid ascii code error for request body strings https://review.openstack.org/308626 | 16:40 |
| xinli | swanson: the CI owner seems also no solution, the CI just not working | 16:41 |
| *** rajinir has quit IRC | 16:42 | |
| Swanson | xinli, They should wander over to #openstack-infra and see of they can get some help fixing it. That is the CI associated with that driver, right? | 16:44 |
| xinli | swanson: got it | 16:45 |
| openstackgerrit | Aleksey Ruban proposed openstack/cinder: Nexenta patchset for Newton release https://review.openstack.org/319349 | 16:46 |
| *** diogogmt has quit IRC | 16:47 | |
| Swanson | xinli, If it is any consolation I'm having the same kind of issues with my patches this cycle. Getting them through my own CI is an adventure. | 16:47 |
| *** laughterwym has quit IRC | 16:47 | |
| *** diogogmt has joined #openstack-cinder | 16:48 | |
| *** karthikp has joined #openstack-cinder | 16:56 | |
| *** kfarr has quit IRC | 16:59 | |
| *** raunak has joined #openstack-cinder | 17:09 | |
| *** jistr has quit IRC | 17:09 | |
| *** tpsilva has joined #openstack-cinder | 17:11 | |
| *** diogogmt has quit IRC | 17:13 | |
| *** diogogmt has joined #openstack-cinder | 17:14 | |
| *** adelia has joined #openstack-cinder | 17:16 | |
| *** mimizone has joined #openstack-cinder | 17:16 | |
| mimizone | Hi all. | 17:17 |
| mimizone | anybody can make sense of these logs when trying to delete a volume? (actually deleting an instance with its vda on ceph) http://paste.openstack.org/show/497945/ | 17:18 |
| mimizone | it's openstack icehouse. so quite old. | 17:18 |
| mimizone | the weird thing is that it sometimes work. | 17:18 |
| *** david-lyle_ is now known as david-lyle | 17:22 | |
| *** hemna_ has quit IRC | 17:23 | |
| *** akerr_ is now known as akerr_away | 17:33 | |
| *** akerr_away is now known as akerr_ | 17:33 | |
| *** salv-orlando has quit IRC | 17:36 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Add driver interface checks https://review.openstack.org/297140 | 17:40 |
| *** ChubYann has joined #openstack-cinder | 17:40 | |
| *** sdague has quit IRC | 17:43 | |
| smcginnis | I've given up on this for now: https://review.openstack.org/#/c/290464/ | 17:44 |
| smcginnis | Owner is looking for someone to pick it up. | 17:44 |
| smcginnis | I've found some interesting issues so far, but the unit tests need a lot more work. | 17:44 |
| smcginnis | So if anyone is looking for something interesting to dig into - have at it. | 17:44 |
| *** akshai has quit IRC | 17:44 | |
| *** dustins has joined #openstack-cinder | 17:45 | |
| *** e0ne has joined #openstack-cinder | 17:46 | |
| *** akshai has joined #openstack-cinder | 17:47 | |
| *** laughterwym has joined #openstack-cinder | 17:48 | |
| *** ntpttr has quit IRC | 17:51 | |
| openstackgerrit | Sean McGinnis proposed openstack/python-cinderclient: WIP: switch to keystoneauth https://review.openstack.org/290464 | 17:51 |
| *** ntpttr_ is now known as ntpttr | 17:51 | |
| e0ne | smcginnis: hi. looks like it's a high or critical for us https://bugs.launchpad.net/python-cinderclient/+bug/1584056:( | 17:53 |
| openstack | Launchpad bug 1584056 in python-cinderclient "upload_to_image() takes exactly 7 arguments (5 given)" [Undecided,New] | 17:53 |
| *** ducttape_ has quit IRC | 17:53 | |
| *** laughterwym has quit IRC | 17:53 | |
| smcginnis | e0ne: Err, yeah. That doesn't look good. | 17:53 |
| e0ne | smcginnis: for now, only rally gates are affected | 17:54 |
| e0ne | but who knows... | 17:54 |
| *** markvoelker has joined #openstack-cinder | 17:54 | |
| e0ne | any existing user could be affected | 17:55 |
| *** sdague has joined #openstack-cinder | 17:55 | |
| smcginnis | Yeah, looks like it. Those new args should have defaults so it stay backwards compatible. :[ | 17:55 |
| e0ne | smcginnis: I'm going to make a patch | 17:55 |
| smcginnis | e0ne: OK, great. Thanks! I'll watch for it. | 17:56 |
| ntpttr | e0ne smcginnis: ah that was my patch that recently got merged - not sure how that didn't get caught I'm sorry! | 17:56 |
| smcginnis | ntpttr: Looks like we all missed it. :/ | 17:56 |
| ntpttr | should just be fixed by giving them 'private' and 'False' defaults | 17:56 |
| smcginnis | ntpttr: It happens. Luckily it was caught quick. | 17:56 |
| *** mtanino has quit IRC | 17:57 | |
| *** markvoelker has quit IRC | 17:59 | |
| *** rlrossit has quit IRC | 18:03 | |
| e0ne | smcginnis, ntpttr: I'm not sure that adding defauls is a right way | 18:03 |
| ntpttr | e0ne: why not? The images were already being uploaded as private and unprotected by default I believe | 18:04 |
| e0ne | smcginnis, ntpttr: yes, but technically, it means that we add these params for v2 | 18:05 |
| e0ne | smcginnis, ntpttr: I hope, something like http://paste.openstack.org/show/497946/ will work | 18:05 |
| ntpttr | e0ne: won't that not work if you try to use those parameters? How does that happen with python, I guess I'm not sure | 18:06 |
| ntpttr | ah wait nvm | 18:07 |
| ntpttr | I see that the 3.1 version wrap has them there, that's a good call | 18:07 |
| ntpttr | just didn't have that in the paste | 18:08 |
| e0ne | smcginnis, ntpttr: IMO, we don't need to pass visibility, protected for v2.0 & v3.0 | 18:09 |
| ntpttr | e0ne: but what about this line here https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volumes.py#L118 | 18:09 |
| ntpttr | it will try to pass those along anyways | 18:09 |
| ntpttr | maybe put another api version check there | 18:10 |
| ntpttr | and make the call in two different ways based on the way it returns? | 18:10 |
| *** lpetrut has quit IRC | 18:10 | |
| *** lpetrut has joined #openstack-cinder | 18:11 | |
| e0ne | ntpttr: it's a good question | 18:11 |
| ntpttr | I think that would probably work | 18:13 |
| *** mtanino has joined #openstack-cinder | 18:14 | |
| *** ducttape_ has joined #openstack-cinder | 18:16 | |
| openstackgerrit | Aleksey Ruban proposed openstack/cinder: Replaced nas_ip option with nas_host in NFS driver for NexentaStor 5. Added NexentaEdge NBD driver. Added HA support in NexentaEdge iSCSI driver. Added unit tests for NexentaEdge NBD driver. Refactored code to pass PEP8, Python 2,3 tests. https://review.openstack.org/319349 | 18:20 |
| *** adisky has quit IRC | 18:20 | |
| *** lpetrut has quit IRC | 18:20 | |
| *** ntpttr_ has joined #openstack-cinder | 18:23 | |
| *** julim has quit IRC | 18:27 | |
| *** akerr_ is now known as akerr | 18:28 | |
| *** sgotliv_ has quit IRC | 18:31 | |
| *** rlrossit has joined #openstack-cinder | 18:32 | |
| *** Redcavalier has joined #openstack-cinder | 18:33 | |
| Redcavalier | Hi, I think I may have found a bug in cinder, but before I report anything, I'd just like to check if it is a bug, or if I'm missing something. | 18:34 |
| Redcavalier | Basically, we're using the dell iscsi driver with cinder. I am testing out availability-zones in nova and I figured out before long that I need to set the same availability-zone in cinder too. I did so, but instances do not boot from volume. I can create volumes from images by themselves, no problem. But when I try to attach those volumes as boot drives to a nova instance, libvirt thinks the storage is local and tries to boot the drive | 18:38 |
| Redcavalier | locally. It's the only way this happens, as if I don't specify availability-zones, all works well. | 18:38 |
| Swanson | Redcavalier, asking around. | 18:44 |
| Swanson | Redcavalier, any notable errors in the c-vol log? | 18:44 |
| *** sdake has joined #openstack-cinder | 18:46 | |
| Redcavalier | Swanson, nothing notable. There is the "created volume successfully" message and that's about it. Debug is not activated though | 18:48 |
| *** diogogmt has quit IRC | 18:48 | |
| Redcavalier | when I try to boot an instance from the volume, I litterally get libvirtError: Cannot access storage file '/var/lib/nova/instances/73fe56dd-1104-4ae7-90e4-caeb4bf976af/disk' (as uid:107, gid:107): No such file or directory | 18:49 |
| *** rajinir has joined #openstack-cinder | 18:51 | |
| *** charlesr has joined #openstack-cinder | 18:53 | |
| openstackgerrit | David Rosales proposed openstack/cinder: Remove excess database calls in volume attach https://review.openstack.org/318324 | 18:55 |
| *** dustins has quit IRC | 18:59 | |
| *** darosale has quit IRC | 19:02 | |
| *** Yogi1 has joined #openstack-cinder | 19:03 | |
| *** lpetrut has joined #openstack-cinder | 19:04 | |
| *** dustins has joined #openstack-cinder | 19:12 | |
| *** ericksonsantos has quit IRC | 19:15 | |
| scottda | ntpttr: e0ne Is one of you working on the upload_to_image cinderclient bug? It is not assigned.....and I can work on it if you are not. | 19:17 |
| e0ne | Ido | 19:17 |
| ntpttr | scottda: I think e0ne said he wanted to | 19:17 |
| e0ne | s/Ido/ I do | 19:17 |
| ntpttr | e0ne: maybe you should assign yourself so nobody snags it | 19:17 |
| *** Yogi1 has quit IRC | 19:17 | |
| *** hemna_ has joined #openstack-cinder | 19:18 | |
| *** julim has joined #openstack-cinder | 19:20 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: WIP. Fix upload_to_image method https://review.openstack.org/319419 | 19:21 |
| e0ne | scottda: ^^ here is WIP patch | 19:21 |
| *** marksei has joined #openstack-cinder | 19:27 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Fix upload_to_image method https://review.openstack.org/319419 | 19:28 |
| scottda | e0ne: I think you will need some code to deal with this: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v2/shell.py#L20 | 19:28 |
| e0ne | scottda, ntpttr: please, review | 19:28 |
| scottda | We translate v3/shell.py volumev3 stuff to v2/shell.py volumev2 | 19:29 |
| e0ne | oh... | 19:30 |
| *** charlesr has quit IRC | 19:30 | |
| smcginnis | Sorry, had stepped away. Looks like you have things mostly figured out. | 19:30 |
| smcginnis | I was going to suggest pinging scottda, so glad to see he chimed in. :) | 19:31 |
| smcginnis | Redcavalier: That sounds like it may be a Nova issue. I know there have been some problems with AZs. Not sure if its related, but might be worth checking on that channel. | 19:31 |
| e0ne | scottda: omg... is't a bad idea to use v3 in v2.shell. did I +2 on it? | 19:32 |
| Redcavalier | smcginnis, yea, I wasn't sure if I should poke cinder or nova. thanks for the info. | 19:32 |
| e0ne | scottda: sorry, I missunderstood what retype_method doo | 19:33 |
| *** raunak has quit IRC | 19:33 | |
| scottda | Yeah, Gorka and I worked on several ideas to re-use the shell for v3 and v2. | 19:33 |
| *** akerr has quit IRC | 19:33 | |
| smcginnis | Redcavalier: Hard to really say, but if there aren't any errors creating the volume, and the error is from libvirt, I'm leaning that way until proven otherwise. ;) | 19:34 |
| smcginnis | v2 should be == to v3. It's different for v3.1, 3.2, etc. | 19:34 |
| Swanson | Redcavalier, smcginnis, that's what I was thinking. Could be an issue but I'd check Nova first. | 19:35 |
| scottda | Yeah, I think what is missing is versioning for 3.1 in v3/shell.py | 19:35 |
| Redcavalier | Swanson, smcginnis, to be quite honest, we might not even use availability zones after all so that bug is not much of a concern for me, but I'll make sure to inform the nova team, just so they know. | 19:36 |
| e0ne | scottda: now, I broke unit tests for 'shell' module:( | 19:36 |
| *** timcl has quit IRC | 19:37 | |
| Swanson | Redcavalier, can I ask who 'we" are? Always like to know who's using our stuff. | 19:38 |
| Redcavalier | Swanson, I'm working for PlanetHoster. A small-to-medium sized host based in montreal, quebec. | 19:39 |
| Redcavalier | hoster* | 19:39 |
| Swanson | Redcavalier, thanks! | 19:39 |
| scottda | e0ne: I *think* the answer is to put do_upload_to_image(..) in v2/shell with the proper v2 args. That seems ugly. The original way I wrote it , the code lived in v2/shell.py and v3/shell.py would have the code that overrides. | 19:39 |
| Redcavalier | Swanson, you're welcome :) | 19:39 |
| e0ne | scottda: that's what I'm doing right know | 19:40 |
| e0ne | scottda: IMO, we have to fix it asap and release a new version | 19:40 |
| e0ne | scottda: then, we'll have a time to find a better solution if needed | 19:40 |
| *** jwcroppe has joined #openstack-cinder | 19:40 | |
| scottda | e0ne: Well, I think that we'll always have to add a version of the original code in v2/shell.py for anything that changes in v3/shell.py. | 19:41 |
| e0ne | scottda: or start use OOP | 19:42 |
| e0ne | scottda: we've already have a lot of duplicated code in client | 19:42 |
| scottda | Yes, it would be good to clean that up. | 19:43 |
| e0ne | scottda: TBH, I've added some copy-pasted code right now | 19:43 |
| *** hemna_ has quit IRC | 19:45 | |
| *** ericksonsantos has joined #openstack-cinder | 19:50 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Fix upload_to_image method https://review.openstack.org/319419 | 19:50 |
| *** laughterwym has joined #openstack-cinder | 19:50 | |
| e0ne | smcginnis: shold we change https://github.com/openstack/requirements/blob/master/global-requirements.txt#L185 here to not use 1.7.0? | 19:51 |
| smcginnis | e0ne: Yeah, we should block it now. This will cause problems for any existing deployments. :( | 19:53 |
| e0ne | smcginnis: thanks for the confirmation. will do it right now | 19:54 |
| smcginnis | e0ne: Cool, thanks. | 19:55 |
| *** laughterwym has quit IRC | 19:55 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Fix upload_to_image method https://review.openstack.org/319419 | 19:56 |
| e0ne | smcginnis: https://review.openstack.org/#/c/319431/ | 19:59 |
| smcginnis | e0ne: A couple comments there. | 20:02 |
| *** salv-orlando has joined #openstack-cinder | 20:03 | |
| e0ne | smcginnis: thanks, Sean | 20:03 |
| e0ne | looks like I have to take a break to not do more stupid mistakes | 20:04 |
| *** Apoorva has quit IRC | 20:04 | |
| smcginnis | e0ne: Hah! Getting late there on a Friday, isn't it? :) | 20:04 |
| scottda | e0ne: I think at this hour "break" is called "sleep" | 20:04 |
| e0ne | smcginnis: you're right | 20:05 |
| scottda | e0ne: I just reviewed and tested and it looks like a good fix to me. | 20:05 |
| openstackgerrit | xing-yang proposed openstack/cinder: Testing: Do Not Merge https://review.openstack.org/319434 | 20:05 |
| smcginnis | e0ne: Ah, another issue I'm afraid. Where'd the rest of the file go? | 20:05 |
| e0ne | smcginnis: 11pm is a late enough time for Friday's work | 20:05 |
| scottda | ntpttr: Are you around to test this ? https://review.openstack.org/#/c/319431/ | 20:06 |
| e0ne | smcginnis: "rest of the file" what exactly do you mean? | 20:06 |
| smcginnis | e0ne: Too late IMO. You should be out partying it up. :D | 20:06 |
| e0ne | :) | 20:06 |
| smcginnis | e0ne: All lines before the cinderclient line are gone in that latest one. | 20:06 |
| * e0ne takes a beer | 20:06 | |
| smcginnis | Unless gerrit is playing tricks on me again. | 20:06 |
| e0ne | ooops | 20:06 |
| e0ne | fised | 20:07 |
| e0ne | fixed | 20:07 |
| *** salv-orlando has quit IRC | 20:08 | |
| e0ne | have a good Friday! see you soon | 20:08 |
| *** e0ne has quit IRC | 20:08 | |
| scottda | e0ne: Have a good one. | 20:08 |
| smcginnis | Good night! He got out quick! :) | 20:09 |
| scottda | smcginnis: I don't see any issue with PS#4 | 20:09 |
| smcginnis | scottda: Yep, looks good. | 20:09 |
| scottda | smcginnis: Yeah, i tried to type "e0ne" with tab competion 2 seconds after his goodbye and he was gone! | 20:09 |
| smcginnis | scottda: Now we just need a new client so folks installing it stand along don't get the 1.7.0 release. | 20:10 |
| smcginnis | *alone | 20:10 |
| scottda | smcginnis: Yeah, does that mean a 1.7.1 release? | 20:10 |
| *** IlyaG has joined #openstack-cinder | 20:10 | |
| smcginnis | scottda: I think so... :| | 20:10 |
| smcginnis | And it's a Friday, so no releases. But I'll see if we can get one through since it's an issue. | 20:11 |
| ntpttr | scottda: sorry I was out - still want some help w/ testing? I'd be happy to | 20:12 |
| scottda | ntpttr: Yes, test and review if you've time. | 20:12 |
| ntpttr | sure | 20:12 |
| *** e0ne has joined #openstack-cinder | 20:12 | |
| smcginnis | https://review.openstack.org/#/c/319419/ | 20:12 |
| scottda | ntpttr: ^^ that one. I sent you the wrong link before. | 20:13 |
| ntpttr | scottda: gotcha, I thought that was probably it | 20:13 |
| smcginnis | scottda: So, microversion wise, did we do this correctly on the cinder side? https://review.openstack.org/#/c/230617/ | 20:14 |
| smcginnis | Should only be 3.1, right? | 20:14 |
| ntpttr | smcginnis: I'm pretty sure we did | 20:14 |
| smcginnis | ntpttr: OK, I thought so. | 20:15 |
| ntpttr | the error is just happening on the client side w/ the extra parameters being non-backwards compatible | 20:15 |
| * smcginnis thinks there are too many things to watch for in reviews now. :( | 20:15 | |
| ntpttr | smcginnis: gotta have like a sticky on your desktop with reminders for everything or something | 20:16 |
| smcginnis | ntpttr: Hah, no doubt. | 20:16 |
| scottda | smcginnis: Yes, that server code still looks good. | 20:18 |
| smcginnis | scottda: Thanks. Just double checking. ;) | 20:19 |
| scottda | smcginnis: sure. Don't worry, this was the first use of microversions. We'll get better at reviewing... | 20:19 |
| * scottda feels guilty for introducing the scourge of microversions | 20:19 | |
| smcginnis | Yeah, hopefully some of this just becomes natural. | 20:19 |
| smcginnis | Haha. | 20:19 |
| smcginnis | We had to go there sooner or later. | 20:20 |
| ntpttr | the mistake wasn't even really microversions fault | 20:20 |
| smcginnis | ntpttr: Right! | 20:20 |
| scottda | I'm kinda like Sir Walter Raleigh bringing tobacco to Europe | 20:20 |
| smcginnis | It just adds that little bit of doubt.. | 20:20 |
| ntpttr | I put in new params into a function, should've realized it wasn't backwards compatible and fixed it :( | 20:20 |
| smcginnis | scottda: Hah | 20:20 |
| ntpttr | scottda: that's a really good and obscure simile there | 20:21 |
| smcginnis | No kidding! | 20:21 |
| scottda | Well, it seemed like a good idea at the time... | 20:21 |
| smcginnis | ntpttr: scottda is trying to show of his trivia prowess I think. :) | 20:21 |
| *** sdake has quit IRC | 20:22 | |
| ntpttr | hah yeah that is some prime trivia knowledge! | 20:22 |
| *** sdake has joined #openstack-cinder | 20:22 | |
| scottda | s/Cards Against Humanity/Trivial Pursuit | 20:23 |
| *** porrua has quit IRC | 20:23 | |
| *** marksei has quit IRC | 20:24 | |
| ntpttr | scottda: love that game. Another one that should be pretty fun along similar lines is the 'joking hazard' game the cyanide and happiness webcomic guys are coming out with | 20:24 |
| *** sdake has quit IRC | 20:25 | |
| smcginnis | I like the comic. | 20:25 |
| *** lpetrut has quit IRC | 20:27 | |
| *** lpetrut has joined #openstack-cinder | 20:27 | |
| *** akerr has joined #openstack-cinder | 20:29 | |
| *** akerr has quit IRC | 20:29 | |
| *** jungleboyj has quit IRC | 20:29 | |
| *** raunak has joined #openstack-cinder | 20:31 | |
| *** e0ne has quit IRC | 20:32 | |
| *** delattec has quit IRC | 20:33 | |
| xinli | team need help, how can I use pdb to debug the test ocde? tox -e py27 -m pdb -- -n cinder.tests.unit.test_gpfs failed | 20:42 |
| ntpttr | xinli: in the code, put "import pdb; pdb.set_trace()" in your test and then just run it individually | 20:42 |
| jgriffith | hemna: you around? | 20:42 |
| ntpttr | in my experience pdb doesn't always work if you're running multiple tests, maybe it does now | 20:42 |
| ntpttr | but that should get it to work for you without the -m pdb in your command | 20:43 |
| jgriffith | ntpttr: it does | 20:43 |
| jgriffith | ntpttr: -- -n | 20:43 |
| ntpttr | jgriffith: cool good to know | 20:43 |
| xinli | ntpttr: let me try now | 20:44 |
| ntpttr | xinli: all right let me know if it works for you | 20:44 |
| jgriffith | jbernard: maybe you can help me with a question I have? | 20:45 |
| *** sdake has joined #openstack-cinder | 20:45 | |
| *** gouthamr has quit IRC | 20:45 | |
| jgriffith | jbernard: oh, wait... think I just found what I was looking for :) | 20:45 |
| jgriffith | never mind | 20:45 |
| *** gouthamr has joined #openstack-cinder | 20:46 | |
| *** gouthamr has quit IRC | 20:46 | |
| smcginnis | ntpttr: Did you get a chance to try out https://review.openstack.org/#/c/319419/ | 20:47 |
| *** Apoorva has joined #openstack-cinder | 20:47 | |
| ntpttr | smcginnis: had to re-spin up and stack my devstack VM, I had deleted it to make space for all the storage simulators for coprhd | 20:48 |
| ntpttr | smcginnis: should be done stacking soon and I'll give it a go | 20:48 |
| smcginnis | ntpttr: Hah, I'm sure that takes a lot to simulate all the various simulators. :) | 20:48 |
| smcginnis | Er, storage. | 20:49 |
| ntpttr | yeah they're a bit of a memory hog, but super convenient for running some quick tests | 20:49 |
| *** bardia has joined #openstack-cinder | 20:49 | |
| smcginnis | Definitely | 20:49 |
| openstackgerrit | Merged openstack/cinder: Avoid ascii code error for request body strings https://review.openstack.org/308626 | 20:49 |
| *** Redcavalier has quit IRC | 20:50 | |
| *** cknight has quit IRC | 20:52 | |
| *** raildo is now known as raildo-afk | 20:53 | |
| *** karthikp_ has joined #openstack-cinder | 20:57 | |
| *** IlyaG has quit IRC | 20:57 | |
| *** karthikp has quit IRC | 20:59 | |
| *** karthikp_ is now known as karthikp | 20:59 | |
| *** bardia has quit IRC | 21:00 | |
| *** IlyaG has joined #openstack-cinder | 21:01 | |
| *** baumann has left #openstack-cinder | 21:03 | |
| *** diablo_rojo has quit IRC | 21:06 | |
| *** akshai has quit IRC | 21:06 | |
| ntpttr | smcginnis: tested and reviewed, it works | 21:16 |
| ntpttr | crisis averted | 21:16 |
| smcginnis | ntpttr: Awesome, thanks for checking it. | 21:18 |
| ntpttr | smcginnis: np | 21:18 |
| smcginnis | jgriffith: Mind taking a quick look: https://review.openstack.org/#/c/319419/4 | 21:19 |
| jgriffith | sure | 21:19 |
| jgriffith | oh lord | 21:19 |
| smcginnis | :) | 21:19 |
| smcginnis | Oops | 21:19 |
| *** rcernin has joined #openstack-cinder | 21:20 | |
| *** markvoelker has joined #openstack-cinder | 21:21 | |
| jgriffith | smcginnis: done | 21:23 |
| *** chenying has quit IRC | 21:23 | |
| smcginnis | jgriffith: Thanks! | 21:23 |
| *** chenying has joined #openstack-cinder | 21:23 | |
| smcginnis | jgriffith: How goes the attach battle? | 21:23 |
| smcginnis | jgriffith: Better if folks stopped interupting you with stupid questions? :) | 21:23 |
| jgriffith | I'd like to scream at our code... well I have been screaming at it :) | 21:24 |
| smcginnis | Hah | 21:24 |
| jgriffith | I keep trying to whittle some things down | 21:24 |
| *** markvoelker has quit IRC | 21:24 | |
| jgriffith | Our nightmarish unit tests are making my life difficult though | 21:24 |
| *** adelia has quit IRC | 21:24 | |
| *** timcl has joined #openstack-cinder | 21:24 | |
| *** markvoelker has joined #openstack-cinder | 21:24 | |
| jgriffith | and it also seems that over the years we've decided that *everything* should be optional | 21:24 |
| smcginnis | Oh man. I banged my head on that keystoneauth patch for way too long and it's all because of unit tests. :/ | 21:25 |
| *** rlrossit has quit IRC | 21:25 | |
| jgriffith | so I'm finding I can't count on any sort of consistency | 21:25 |
| jgriffith | yeah... I hate our unit tests | 21:25 |
| jgriffith | contrib/api tests inparticular | 21:25 |
| smcginnis | Are they coded as optional but in reality pretty much required? | 21:25 |
| *** markvoelker_ has joined #openstack-cinder | 21:25 | |
| jgriffith | they do things like "issue a rest call" and go through the ENTIRE code path | 21:25 |
| jgriffith | which kind of goes against the definition of a unit test | 21:26 |
| jgriffith | IMO | 21:26 |
| jgriffith | smcginnis: well... I don't know how you determine that? | 21:26 |
| smcginnis | We have a lot that I wouldn't really consider "unit tests" in my definition of it | 21:26 |
| jgriffith | smcginnis: unless I had all 80 drivers loaded up :) | 21:26 |
| smcginnis | And then on the other spectrum, the ones that test that mock works. | 21:26 |
| jgriffith | smcginnis: yeah.. it's a shame | 21:26 |
| jgriffith | smcginnis: I'm like Goldie Lox... I want it to be "juuuuust right" | 21:26 |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: [WIP] Switch service capabilities to ovo https://review.openstack.org/319040 | 21:27 |
| smcginnis | Haha. I'll send you some porridge. ;) | 21:27 |
| jgriffith | anyway... I'm going to push up a WIP of this stuff today/tonight regardless of where it's at. At least to get some eyes on it | 21:27 |
| jgriffith | hehe... porridge :) | 21:27 |
| jgriffith | mmmmm | 21:27 |
| *** dustins has quit IRC | 21:27 | |
| *** IlyaG has quit IRC | 21:27 | |
| smcginnis | A WIP is probably good. Then we can see if there is something specific to each of our drivers that would cause problems for the whole thing. | 21:28 |
| jgriffith | I have a patch for Nova, cinderclient and cinder all queued up | 21:28 |
| smcginnis | Oh nice! | 21:28 |
| *** markvoelker_ has quit IRC | 21:28 | |
| *** timcl1 has joined #openstack-cinder | 21:28 | |
| jgriffith | smcginnis: yeah.. the good thing is that functionally in devstack everything is working. | 21:28 |
| jgriffith | going to wait on the multi-attach changes though until the first wave gets some eyes on it | 21:28 |
| smcginnis | jgriffith: Just against LVM? Or did you also do SF? | 21:28 |
| *** markvoelker_ has joined #openstack-cinder | 21:28 | |
| jgriffith | LVM and SF | 21:28 |
| jgriffith | and multi-attach | 21:29 |
| smcginnis | Well, 1/40th verified at least. ;) | 21:29 |
| jgriffith | but that was easy for those two drivers anyway | 21:29 |
| jgriffith | haha | 21:29 |
| jgriffith | that's better than .0000001/40'th | 21:29 |
| *** timcl has quit IRC | 21:29 | |
| smcginnis | It's at least part way there! | 21:29 |
| *** markvoelker has quit IRC | 21:30 | |
| *** ametts has quit IRC | 21:31 | |
| *** timcl1 has quit IRC | 21:34 | |
| *** ducttape_ has quit IRC | 21:36 | |
| *** xyang1 has quit IRC | 21:37 | |
| *** IlyaG has joined #openstack-cinder | 21:40 | |
| *** lprice has quit IRC | 21:42 | |
| scottda | What? Everyone missed a good nit: | 21:43 |
| scottda | Correct misspelt words in msg in cinder | 21:43 |
| scottda | Should be misspelled | 21:43 |
| scottda | The irony. | 21:44 |
| jgriffith | lol | 21:47 |
| jgriffith | scottda: did folks *miss it* or *choose to ignore it* | 21:47 |
| *** savihou has quit IRC | 21:48 | |
| jgriffith | :) | 21:49 |
| scottda | I dunno, just funny to misspell mispelled | 21:49 |
| *** lascii is now known as alaski | 21:50 | |
| *** jungleboyj has joined #openstack-cinder | 21:50 | |
| smcginnis | That's not one of those funny British spellings, is it? :) | 21:50 |
| *** diogogmt has joined #openstack-cinder | 21:57 | |
| *** sdake has quit IRC | 21:59 | |
| scottda | Haha.. just might be. | 22:00 |
| *** alyson_ has quit IRC | 22:07 | |
| *** lpetrut has quit IRC | 22:08 | |
| *** sdake has joined #openstack-cinder | 22:09 | |
| *** adrianofr has quit IRC | 22:12 | |
| karthikp | Hi All... I am trying to implement have one of the fields for oslo object as a list of dicts... Do I need to create new item in fields.py for that? I remember we create enum field in fields.py ..never seen a list of dicts there.. | 22:13 |
| karthikp | is list of dicts already a part of oslo-versioned.fields? | 22:14 |
| karthikp | oslo_versioned | 22:14 |
| openstackgerrit | Merged openstack/python-cinderclient: Fix upload_to_image method https://review.openstack.org/319419 | 22:17 |
| *** hemna_ has joined #openstack-cinder | 22:19 | |
| *** stevemar is now known as notstevemar | 22:24 | |
| *** IlyaG has quit IRC | 22:24 | |
| *** gouthamr has joined #openstack-cinder | 22:25 | |
| *** tpsilva has quit IRC | 22:27 | |
| *** xinli has quit IRC | 22:28 | |
| *** jungleboyj has quit IRC | 22:29 | |
| *** jwcroppe has quit IRC | 22:34 | |
| *** dave-mccowan has quit IRC | 22:36 | |
| *** jamielennox|away is now known as jamielennox | 22:48 | |
| *** krotscheck is now known as krotscheck_dcm | 22:57 | |
| *** salv-orlando has joined #openstack-cinder | 22:59 | |
| *** laughterwym has joined #openstack-cinder | 23:01 | |
| *** sdague has quit IRC | 23:03 | |
| *** laughterwym has quit IRC | 23:05 | |
| *** asselin has joined #openstack-cinder | 23:11 | |
| *** marcusvrn_ has quit IRC | 23:13 | |
| *** asselin__ has quit IRC | 23:14 | |
| *** karthikp_ has joined #openstack-cinder | 23:31 | |
| *** salv-orlando has quit IRC | 23:33 | |
| *** jwcroppe has joined #openstack-cinder | 23:35 | |
| *** leeantho has quit IRC | 23:36 | |
| *** jwcroppe has quit IRC | 23:41 | |
| *** sbezverk has quit IRC | 23:42 | |
| *** markvoelker_ has quit IRC | 23:49 | |
| *** karthikp_ has quit IRC | 23:53 | |
| *** raunak has quit IRC | 23:54 | |
| *** karthikp_ has joined #openstack-cinder | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!