*** salv-orlando has quit IRC | 00:01 | |
openstackgerrit | Merged openstack/cinder master: api-ref cleanup: remove quotas user related calls https://review.openstack.org/406001 | 00:07 |
---|---|---|
*** crushil has joined #openstack-cinder | 00:08 | |
*** crushil has quit IRC | 00:08 | |
*** crushil has joined #openstack-cinder | 00:08 | |
*** mtanino has joined #openstack-cinder | 00:09 | |
*** felipemonteiro has quit IRC | 00:13 | |
*** crushil has quit IRC | 00:14 | |
*** crushil has joined #openstack-cinder | 00:14 | |
*** wanghao has joined #openstack-cinder | 00:25 | |
*** s-shiono has joined #openstack-cinder | 00:25 | |
*** gcb has quit IRC | 00:26 | |
stakeda | hi, we need another +2 on this small patch. NEC Cinder CI successfully rechecked. thanks! https://review.openstack.org/#/c/449869/ | 00:29 |
*** crushil has quit IRC | 00:38 | |
*** crushil has joined #openstack-cinder | 00:39 | |
*** itlinux has joined #openstack-cinder | 00:40 | |
*** itlinux has quit IRC | 00:41 | |
*** itlinux has joined #openstack-cinder | 00:43 | |
*** viks has quit IRC | 00:49 | |
*** nicolasbock has joined #openstack-cinder | 00:56 | |
*** Julien-zte has joined #openstack-cinder | 00:57 | |
*** markstur has quit IRC | 00:59 | |
*** kaisers_ has joined #openstack-cinder | 01:07 | |
*** kaisers_ has quit IRC | 01:12 | |
*** zhangguoqing has joined #openstack-cinder | 01:17 | |
openstackgerrit | wangxiyuan proposed openstack/cinder-specs master: Support get volume metadata summary https://review.openstack.org/466691 | 01:18 |
*** markstur has joined #openstack-cinder | 01:21 | |
*** itlinux has quit IRC | 01:28 | |
*** catintheroof has joined #openstack-cinder | 01:29 | |
*** catintheroof has quit IRC | 01:30 | |
openstackgerrit | qtlu proposed openstack/cinder master: Check the volume metadata value is a string or not https://review.openstack.org/465829 | 01:30 |
*** Shunli has joined #openstack-cinder | 01:33 | |
*** dave-mccowan has quit IRC | 01:36 | |
openstackgerrit | qtlu proposed openstack/cinder master: Check the volume metadata value is a string or not https://review.openstack.org/465829 | 01:41 |
*** edmondsw has quit IRC | 01:45 | |
openstackgerrit | zheng yin proposed openstack/cinder master: eventlet.green.subprocess replace subprocess https://review.openstack.org/466607 | 01:49 |
openstackgerrit | TommyLike proposed openstack/cinder master: Explicit user messages https://review.openstack.org/448384 | 01:50 |
openstackgerrit | wangxiyuan proposed openstack/cinder-specs master: Support get volume metadata summary https://review.openstack.org/466691 | 01:50 |
*** jamielennox is now known as jamielennox|away | 01:54 | |
openstackgerrit | wangxiyuan proposed openstack/cinder master: api-ref: add list manageable resource API https://review.openstack.org/465874 | 01:54 |
tommylikehu | hey hemna , could you take a look at this patch when you free? https://review.openstack.org/#/c/448384, thanks in advance | 01:54 |
*** lamt has joined #openstack-cinder | 01:57 | |
*** links has joined #openstack-cinder | 01:58 | |
tommylikehu | ping winston-d , winston-d_ | 01:59 |
*** mriedem_afk is now known as mriedem | 01:59 | |
openstackgerrit | wangxiyuan proposed openstack/cinder master: Ceph: Fix delete backup with non-existent pool https://review.openstack.org/461082 | 01:59 |
tommylikehu | winston-d: could you take a look at this patch again, when you free. https://review.openstack.org/#/c/445440/ thanks in advance :) | 02:00 |
*** zhaochao has joined #openstack-cinder | 02:02 | |
*** gcb has joined #openstack-cinder | 02:04 | |
*** mtanino has quit IRC | 02:15 | |
*** crushil has quit IRC | 02:16 | |
openstackgerrit | wangxiyuan proposed openstack/cinder master: Python3: Don't use Exception.message https://review.openstack.org/463230 | 02:19 |
winston-d_ | @tommylikehu looking | 02:29 |
*** crushil has joined #openstack-cinder | 02:41 | |
*** reduxio has joined #openstack-cinder | 02:46 | |
*** dikonoor has joined #openstack-cinder | 02:46 | |
*** lamt has quit IRC | 02:49 | |
*** reduxio has quit IRC | 02:50 | |
*** anticw has joined #openstack-cinder | 02:52 | |
*** lamt has joined #openstack-cinder | 02:56 | |
*** edmondsw has joined #openstack-cinder | 02:56 | |
anticw | i have a situation where i'm trying to attach a ceph backed volume using cinder and nova-compute is invoking iscsi initiator logic | 02:58 |
anticw | https://pastebin.com/5Uvjk7hR | 02:58 |
anticw | does anyone know off-hand why this would happen? i'm guessing something about the response from cinder makes nova think it's an iscsi target (vs ceph) | 02:59 |
*** edmondsw has quit IRC | 03:01 | |
openstackgerrit | wangxiyuan proposed openstack/python-cinderclient master: Support list_volume for group show https://review.openstack.org/432179 | 03:03 |
*** Apoorva has joined #openstack-cinder | 03:18 | |
*** zsli_ has joined #openstack-cinder | 03:20 | |
*** Shunli has quit IRC | 03:22 | |
openstackgerrit | xing-yang proposed openstack/cinder master: Tiramisu: replication group support https://review.openstack.org/352228 | 03:22 |
*** zsli_ is now known as Shunli | 03:24 | |
openstackgerrit | zheng yin proposed openstack/cinder master: eventlet.green.subprocess replace subprocess https://review.openstack.org/466607 | 03:27 |
*** itlinux has joined #openstack-cinder | 03:29 | |
*** lamt has quit IRC | 03:41 | |
*** lamt has joined #openstack-cinder | 03:46 | |
*** psachin has joined #openstack-cinder | 03:48 | |
*** mriedem has quit IRC | 03:50 | |
*** wanghao has quit IRC | 03:50 | |
*** gkadam has joined #openstack-cinder | 03:51 | |
*** wanghao has joined #openstack-cinder | 03:55 | |
*** wanghao has quit IRC | 03:59 | |
*** arnewiebalck_ has joined #openstack-cinder | 04:09 | |
*** abhitechie has joined #openstack-cinder | 04:12 | |
*** nicolasbock has quit IRC | 04:13 | |
*** david-lyle has quit IRC | 04:14 | |
*** Apoorva has quit IRC | 04:22 | |
*** jamielennox|away is now known as jamielennox | 04:23 | |
*** itlinux has quit IRC | 04:25 | |
*** markstur has quit IRC | 04:26 | |
*** edmondsw has joined #openstack-cinder | 04:44 | |
*** reduxio has joined #openstack-cinder | 04:48 | |
*** arnewiebalck_ has quit IRC | 04:48 | |
openstackgerrit | Sofie Huang proposed openstack/cinder master: Extend Falconstor driver to utilize multiple FSS storage pools https://review.openstack.org/422462 | 04:48 |
*** edmondsw has quit IRC | 04:49 | |
*** reduxio has quit IRC | 04:52 | |
*** sdatko has joined #openstack-cinder | 04:56 | |
*** dpawlik has joined #openstack-cinder | 04:56 | |
*** pgadiya has joined #openstack-cinder | 04:56 | |
*** danpawlik has quit IRC | 04:56 | |
*** sdatko_ has quit IRC | 04:57 | |
*** dpawlik1 has joined #openstack-cinder | 05:02 | |
*** kaisers_ has joined #openstack-cinder | 05:04 | |
*** dpawlik has quit IRC | 05:04 | |
*** jamielennox is now known as jamielennox|away | 05:11 | |
*** pgadiya has left #openstack-cinder | 05:12 | |
*** dpawlik1 has quit IRC | 05:15 | |
*** sdatko has quit IRC | 05:15 | |
*** Poornima_K has joined #openstack-cinder | 05:15 | |
*** dpawlik1 has joined #openstack-cinder | 05:21 | |
*** dpawlik2 has joined #openstack-cinder | 05:26 | |
*** dpawlik1 has quit IRC | 05:26 | |
*** sdatko has joined #openstack-cinder | 05:28 | |
*** markstur has joined #openstack-cinder | 05:31 | |
openstackgerrit | Gaurang Tapase proposed openstack/cinder master: GPFS: Handle unsupported operations with an exception https://review.openstack.org/466889 | 05:32 |
*** bkopilov__ has quit IRC | 05:34 | |
*** markstur has quit IRC | 05:35 | |
*** lkuchlan has joined #openstack-cinder | 05:35 | |
*** adisky_ has joined #openstack-cinder | 05:36 | |
*** mtreinish has quit IRC | 05:38 | |
*** jamielennox|away is now known as jamielennox | 05:51 | |
*** crushil has quit IRC | 05:54 | |
*** isaacb has joined #openstack-cinder | 06:01 | |
*** wanghao has joined #openstack-cinder | 06:01 | |
*** rcernin has quit IRC | 06:01 | |
openstackgerrit | wangxiyuan proposed openstack/cinder-specs master: Support get volume metadata summary https://review.openstack.org/466691 | 06:12 |
openstackgerrit | wangxiyuan proposed openstack/cinder master: Python3: Don't use Exception.message https://review.openstack.org/463230 | 06:14 |
*** ChubYann has quit IRC | 06:15 | |
*** rcernin has joined #openstack-cinder | 06:18 | |
*** jamielennox is now known as jamielennox|away | 06:23 | |
*** lhx__ has joined #openstack-cinder | 06:25 | |
*** lhx__ has quit IRC | 06:25 | |
*** lhx__ has joined #openstack-cinder | 06:26 | |
*** lkuchlan has quit IRC | 06:32 | |
*** edmondsw has joined #openstack-cinder | 06:32 | |
*** mtreinish has joined #openstack-cinder | 06:33 | |
*** edmondsw has quit IRC | 06:37 | |
*** pcaruana has joined #openstack-cinder | 06:40 | |
*** reduxio has joined #openstack-cinder | 06:44 | |
*** zhangguoqing has quit IRC | 06:45 | |
*** kwathore_ has quit IRC | 06:45 | |
*** belmoreira has joined #openstack-cinder | 06:47 | |
*** lamt has quit IRC | 06:48 | |
*** arnewiebalck_ has joined #openstack-cinder | 06:49 | |
*** arnewiebalck__ has quit IRC | 06:49 | |
openstackgerrit | Andrey Volkov proposed openstack/cinder master: api-ref cleanup: required flag for tenants https://review.openstack.org/406056 | 06:50 |
*** arnewiebalck__ has joined #openstack-cinder | 06:52 | |
*** wanghao has quit IRC | 07:02 | |
*** isaacb has quit IRC | 07:03 | |
*** arnewiebalck_ has quit IRC | 07:05 | |
openstackgerrit | Jianghua Wang proposed openstack/cinder master: Create volume from VHD format image always failed https://review.openstack.org/467039 | 07:06 |
openstackgerrit | Jianghua Wang proposed openstack/cinder master: Create volume from VHD format image always failed https://review.openstack.org/467039 | 07:08 |
*** zhangguoqing has joined #openstack-cinder | 07:08 | |
*** salv-orlando has joined #openstack-cinder | 07:10 | |
*** lhx__ has quit IRC | 07:10 | |
*** lkuchlan has joined #openstack-cinder | 07:11 | |
*** lhx__ has joined #openstack-cinder | 07:11 | |
*** psachin has quit IRC | 07:12 | |
*** rcernin has quit IRC | 07:16 | |
*** rcernin has joined #openstack-cinder | 07:17 | |
*** salv-orlando has quit IRC | 07:25 | |
*** psachin has joined #openstack-cinder | 07:29 | |
openstackgerrit | yuyafei proposed openstack/cinder master: Remove "readonly=True" when creating volume from image cache https://review.openstack.org/466563 | 07:41 |
*** dpawlik2 is now known as danpawlik | 07:42 | |
*** psachin has quit IRC | 07:43 | |
openstackgerrit | Gerhard Muntingh proposed openstack/python-cinderclient master: Add pagination for snapshots, backups https://review.openstack.org/465159 | 07:43 |
*** psachin has joined #openstack-cinder | 07:45 | |
*** reduxio has quit IRC | 07:46 | |
*** reduxio has joined #openstack-cinder | 07:46 | |
*** reduxio has quit IRC | 07:47 | |
*** reduxio has joined #openstack-cinder | 07:47 | |
openstackgerrit | Andrey Volkov proposed openstack/cinder master: api-ref cleanup: required flag for tenants https://review.openstack.org/406056 | 07:54 |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-cinder | 08:00 | |
*** reduxio has quit IRC | 08:10 | |
*** reduxio has joined #openstack-cinder | 08:11 | |
*** reduxio has quit IRC | 08:16 | |
*** lpetrut has joined #openstack-cinder | 08:17 | |
*** lhx__ has quit IRC | 08:17 | |
*** openstackgerrit has quit IRC | 08:18 | |
*** lhx__ has joined #openstack-cinder | 08:19 | |
*** mvk has quit IRC | 08:19 | |
*** edmondsw has joined #openstack-cinder | 08:20 | |
*** edmondsw has quit IRC | 08:25 | |
*** lhx__ has quit IRC | 08:29 | |
*** lhx__ has joined #openstack-cinder | 08:29 | |
*** lhx__ has quit IRC | 08:34 | |
*** lhx__ has joined #openstack-cinder | 08:34 | |
*** Julien-zte has quit IRC | 08:35 | |
*** lhx__ has quit IRC | 08:36 | |
*** lhx__ has joined #openstack-cinder | 08:36 | |
*** Julien-zte has joined #openstack-cinder | 08:37 | |
*** lhx__ has quit IRC | 08:37 | |
*** lhx__ has joined #openstack-cinder | 08:37 | |
*** s-shiono has quit IRC | 08:40 | |
kaisers | FYI: You're CIs DevStack is building no longer because 'babel' is missing? See bugticket https://bugs.launchpad.net/oslo.i18n/+bug/1692773 | 08:40 |
openstack | Launchpad bug 1692773 in oslo.i18n "Removal of Babel in 3.15.2 requirements.txt breaks package" [Undecided,New] - Assigned to Akihiro Motoki (amotoki) | 08:40 |
*** Julien-zte has quit IRC | 08:41 | |
*** lhx__ has quit IRC | 08:43 | |
*** Julien-zte has joined #openstack-cinder | 08:43 | |
*** lhx__ has joined #openstack-cinder | 08:43 | |
*** lhx__ has quit IRC | 08:44 | |
*** wN_ has quit IRC | 08:46 | |
*** Julien-zte has quit IRC | 08:47 | |
*** Julien-zte has joined #openstack-cinder | 08:48 | |
*** Kimmo_ has quit IRC | 08:48 | |
*** wN_ has joined #openstack-cinder | 08:48 | |
*** openstackgerrit has joined #openstack-cinder | 08:55 | |
openstackgerrit | yuyafei proposed openstack/cinder master: Remove "readonly=True" when creating volume from image cache https://review.openstack.org/466563 | 08:55 |
*** jistr is now known as jistr|call | 08:56 | |
*** mvk has joined #openstack-cinder | 08:57 | |
*** aarefiev_afk is now known as aarefiev | 09:04 | |
*** jistr|call is now known as jistr | 09:19 | |
*** dalgaaf has quit IRC | 09:24 | |
*** markstur has joined #openstack-cinder | 09:24 | |
*** e0ne has joined #openstack-cinder | 09:25 | |
*** dalgaaf has joined #openstack-cinder | 09:27 | |
*** Shunli has quit IRC | 09:28 | |
*** mvk has quit IRC | 09:32 | |
*** Julien-zte has quit IRC | 09:33 | |
*** Julien-zte has joined #openstack-cinder | 09:34 | |
*** reduxio has joined #openstack-cinder | 09:34 | |
*** stakeda has quit IRC | 09:36 | |
*** Julien-zte has quit IRC | 09:36 | |
*** Julien-zte has joined #openstack-cinder | 09:37 | |
*** Julien-zte has quit IRC | 09:41 | |
*** Julien-zte has joined #openstack-cinder | 09:42 | |
*** mvk has joined #openstack-cinder | 09:45 | |
*** reduxio has quit IRC | 09:45 | |
*** Julien-zte has quit IRC | 09:45 | |
*** dikonoo has joined #openstack-cinder | 09:46 | |
*** Julien-zte has joined #openstack-cinder | 09:46 | |
*** dikonoor has quit IRC | 09:49 | |
*** Julien-zte has quit IRC | 09:55 | |
*** Cibo_ has joined #openstack-cinder | 09:55 | |
*** Julien-zte has joined #openstack-cinder | 09:56 | |
*** faizy has joined #openstack-cinder | 09:59 | |
*** sdague has joined #openstack-cinder | 10:03 | |
*** Cibo_ has quit IRC | 10:04 | |
*** danpawlik has quit IRC | 10:04 | |
*** dpawlik has joined #openstack-cinder | 10:06 | |
*** faizy has quit IRC | 10:07 | |
*** edmondsw has joined #openstack-cinder | 10:09 | |
*** reduxio has joined #openstack-cinder | 10:09 | |
*** Julien-zte has quit IRC | 10:10 | |
*** edmondsw has quit IRC | 10:13 | |
*** dpawlik has quit IRC | 10:14 | |
*** e0ne has quit IRC | 10:14 | |
*** dpawlik has joined #openstack-cinder | 10:14 | |
*** e0ne has joined #openstack-cinder | 10:14 | |
*** reduxio has quit IRC | 10:21 | |
*** reduxio has joined #openstack-cinder | 10:21 | |
*** lpetrut has quit IRC | 10:22 | |
*** lpetrut has joined #openstack-cinder | 10:22 | |
*** lamt has joined #openstack-cinder | 10:25 | |
*** reduxio has quit IRC | 10:26 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs master: Generic backup implementation https://review.openstack.org/379778 | 10:30 |
*** lamt has joined #openstack-cinder | 10:40 | |
dims | tommylikehu : around? | 10:46 |
dims | tommylikehu : can you please peek at this cinder failure - http://logs.openstack.org/41/466941/1/check/gate-cross-cinder-python27-ubuntu-xenial/86a663b/testr_results.html.gz | 10:46 |
*** reduxio has joined #openstack-cinder | 10:48 | |
*** Kimmo_ has joined #openstack-cinder | 10:48 | |
*** salv-orlando has joined #openstack-cinder | 10:49 | |
*** nicolasbock has joined #openstack-cinder | 10:51 | |
*** salv-orlando has quit IRC | 10:53 | |
*** abhinavtechie has joined #openstack-cinder | 10:56 | |
*** abhitechie has quit IRC | 10:56 | |
tommylikehu | hey dims , looking | 10:58 |
dims | tommylikehu : not urgent though :) | 10:58 |
dims | just weird... since i am updating tooz version number there and that does not seem to be in this code path | 10:59 |
tommylikehu | dims: this testcase will fail randomly? | 10:59 |
gerhardqux | arnewiebalck__: see my comment on https://bugs.launchpad.net/cinder/+bug/1685818 | 11:00 |
openstack | Launchpad bug 1685818 in Cinder "Parallel deletion triggers DB deadlocks " [Medium,Triaged] | 11:00 |
gerhardqux | smcginnis: ^ | 11:00 |
dims | tommylikehu : will recheck again, seems random yes | 11:01 |
dims | tommylikehu : see https://review.openstack.org/#/c/466941/ - 2 failures 2 success, the current run in zuul view is the one i gave you the logs for | 11:03 |
*** chhavi has joined #openstack-cinder | 11:03 | |
tommylikehu | dims: Thanks I will look into it after our internal conference. We had this every Tuesday evening :) | 11:07 |
*** abhinavtechie has quit IRC | 11:08 | |
*** reduxio has quit IRC | 11:09 | |
*** reduxio has joined #openstack-cinder | 11:09 | |
dims | tommylikehu : no rush at all, i know it's very late for you | 11:11 |
*** reduxio has quit IRC | 11:14 | |
arnewiebalck__ | gerhardqux: thx! I’ll give that a try and let you know! | 11:15 |
arnewiebalck__ | gerhardqux: is that documentted somewhere? | 11:16 |
gerhardqux | arnewiebalck__: it's tucked away at https://docs.openstack.org/developer/cinder/devref/threading.html#mysql-access-and-eventlet | 11:19 |
*** Poornima_K has quit IRC | 11:29 | |
openstackgerrit | Gyorgy Szombathelyi proposed openstack/cinder master: Use the deprecated os_privileged_xxx settings https://review.openstack.org/467132 | 11:33 |
*** gouthamr_ has joined #openstack-cinder | 11:36 | |
openstackgerrit | jiamin proposed openstack/cinder master: DS8K: support clone volume asynchroniously https://review.openstack.org/467136 | 11:40 |
*** salv-orlando has joined #openstack-cinder | 11:42 | |
*** lhx__ has joined #openstack-cinder | 11:45 | |
*** chhavi has quit IRC | 11:48 | |
*** Julien-zte has joined #openstack-cinder | 11:48 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/467141 | 11:48 |
*** lhx__ has quit IRC | 11:51 | |
*** lhx__ has joined #openstack-cinder | 11:51 | |
*** salv-orlando has quit IRC | 11:52 | |
arnewiebalck__ | gerhardqux: it works! | 11:52 |
arnewiebalck__ | gerhardqux: with the change you suggested, I cannot reproduce the problem any longer | 11:53 |
gerhardqux | arnewiebalck__: nice! | 11:53 |
arnewiebalck__ | gerhardqux: would you mind explaining this, I’m not sure I understand … | 11:54 |
gerhardqux | I don't understand the locking fully, but the case is this. | 11:54 |
gerhardqux | cinder runs in one thread | 11:54 |
gerhardqux | eventlet mimicks threads by overriding I/O calls | 11:55 |
gerhardqux | when such a green thread waits for I/O, the overridden call gives CPU time to another green thread | 11:55 |
gerhardqux | the mysql:// libc implementation prevents overriding these calls | 11:56 |
gerhardqux | thus the process (the one thread) blocks, waiting for mysql all the time | 11:57 |
gerhardqux | sqlalchemy creates a loop when a resource is locked, because it wants the lock itself | 11:58 |
gerhardqux | so you have around 10 green threads blocking the process, just trying to create locks | 11:59 |
gerhardqux | and the cinder process cannot do anything else | 11:59 |
*** lkuchlan has quit IRC | 12:00 | |
*** gouthamr_ has quit IRC | 12:02 | |
*** Julien-zte has quit IRC | 12:02 | |
*** liverpooler has joined #openstack-cinder | 12:02 | |
arnewiebalck__ | when decreasing the innodb wait lock timeout, things started to move forward more quickly … how is the db server involved when there is only one thread talking to it at any given moment? | 12:04 |
arnewiebalck__ | or is that param attached to the session and the client gives up after the timeout? | 12:05 |
*** Poornima_K has joined #openstack-cinder | 12:05 | |
arnewiebalck__ | jungleboyj: ^ | 12:09 |
*** wN_ has quit IRC | 12:09 | |
*** rmk has quit IRC | 12:11 | |
*** rods has quit IRC | 12:11 | |
*** dgonzalez has quit IRC | 12:12 | |
*** MVenesio has joined #openstack-cinder | 12:13 | |
*** wN_ has joined #openstack-cinder | 12:13 | |
*** rmk has joined #openstack-cinder | 12:14 | |
*** rmk has quit IRC | 12:14 | |
*** rmk has joined #openstack-cinder | 12:14 | |
gerhardqux | I don't know this yet | 12:14 |
*** dgonzalez has joined #openstack-cinder | 12:14 | |
*** dave-mccowan has joined #openstack-cinder | 12:14 | |
*** rods has joined #openstack-cinder | 12:14 | |
smcginnis | gerhardqux, arnewiebalck__: Wow, nice! | 12:17 |
smcginnis | arnewiebalck__: That might explain why I was never able to repro this - I always just used mysql+pymysql | 12:17 |
*** lamt has quit IRC | 12:18 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/467141 | 12:18 |
*** dustins has joined #openstack-cinder | 12:19 | |
*** reduxio has joined #openstack-cinder | 12:20 | |
arnewiebalck__ | smcginnis: exactly, I was thinking the same thing! | 12:21 |
*** ociuhandu has joined #openstack-cinder | 12:21 | |
*** lkuchlan has joined #openstack-cinder | 12:22 | |
*** Julien-zte has joined #openstack-cinder | 12:23 | |
smcginnis | jungleboyj: ^^ | 12:24 |
*** edmondsw has joined #openstack-cinder | 12:25 | |
*** edmondsw_ has joined #openstack-cinder | 12:27 | |
*** Julien-zte has quit IRC | 12:27 | |
*** Julien-zte has joined #openstack-cinder | 12:28 | |
smcginnis | geguileo: Didn't you have a patch that fixed something like this before? http://logs.openstack.org/41/466941/1/check/gate-cross-cinder-python27-ubuntu-xenial/86a663b/console.html#_2017-05-23_10_12_14_072514 | 12:29 |
*** edmondsw has quit IRC | 12:29 | |
*** lhx__ has quit IRC | 12:30 | |
*** lhx__ has joined #openstack-cinder | 12:30 | |
*** edmondsw_ has quit IRC | 12:31 | |
*** edmondsw has joined #openstack-cinder | 12:33 | |
*** liverpooler has quit IRC | 12:33 | |
*** liverpooler has joined #openstack-cinder | 12:34 | |
*** crushil has joined #openstack-cinder | 12:34 | |
smcginnis | geguileo: I think I was thinking of this one: https://review.openstack.org/#/c/363008 | 12:37 |
*** edmondsw has quit IRC | 12:37 | |
*** edmondsw has joined #openstack-cinder | 12:39 | |
*** catintheroof has joined #openstack-cinder | 12:42 | |
*** tomh has joined #openstack-cinder | 12:42 | |
*** bjornar_ has joined #openstack-cinder | 12:44 | |
*** edmondsw has quit IRC | 12:44 | |
bjornar_ | What are the config keys that influences if nova is allowed to Authenticate with X-Service-Token/X-Auth-Token or not? | 12:44 |
bjornar_ | I get a 401 from cinder when trying to attach a volume.. | 12:44 |
bjornar_ | X-Service-Token is then nova and X-Auth-Token whoever is trying to attach the volume (admin in this case) | 12:45 |
bjornar_ | verified that keystone serves back the correct token information to cinder.. | 12:46 |
*** edmondsw has joined #openstack-cinder | 12:47 | |
*** salv-orlando has joined #openstack-cinder | 12:48 | |
*** jamielennox|away is now known as jamielennox | 12:49 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs master: Backup driver initialization https://review.openstack.org/367439 | 12:49 |
*** edmondsw has quit IRC | 12:51 | |
*** salv-orlando has quit IRC | 12:53 | |
*** edmondsw has joined #openstack-cinder | 12:53 | |
*** gcb has quit IRC | 12:55 | |
*** edmondsw has quit IRC | 12:55 | |
*** edmondsw has joined #openstack-cinder | 12:56 | |
*** crushil has quit IRC | 12:56 | |
geguileo | smcginnis: I think I've fixed a couple of issues like that one | 12:57 |
geguileo | although I don't know where this specific one is comming from | 12:57 |
*** crushil has joined #openstack-cinder | 12:57 | |
*** lamt has joined #openstack-cinder | 12:59 | |
*** lhx__ has quit IRC | 13:02 | |
*** crushil has quit IRC | 13:04 | |
*** crushil has joined #openstack-cinder | 13:04 | |
*** lamt has quit IRC | 13:04 | |
*** lamt has joined #openstack-cinder | 13:04 | |
geguileo | smcginnis: But I think I know where the problem is | 13:05 |
geguileo | although it's not probably that since it doesn't seem to be failing all the time, right? | 13:06 |
*** Julien-zte has quit IRC | 13:07 | |
*** Julien-zte has joined #openstack-cinder | 13:08 | |
*** winston-d_ has quit IRC | 13:09 | |
*** mriedem has joined #openstack-cinder | 13:11 | |
*** salv-orlando has joined #openstack-cinder | 13:15 | |
*** lucasxu has joined #openstack-cinder | 13:17 | |
*** lucasxu has quit IRC | 13:17 | |
*** eantyshev has joined #openstack-cinder | 13:19 | |
*** salv-orlando has quit IRC | 13:20 | |
*** Julien-zte has quit IRC | 13:22 | |
eantyshev | e0ne: Hello! Could you please take look here: https://review.openstack.org/#/c/462563 ? | 13:22 |
*** eantyshev has left #openstack-cinder | 13:22 | |
e0ne | eantyshev: hi. sure, I'll review it | 13:22 |
e0ne | eantyshev: I don't see CI results for it | 13:23 |
smcginnis | geguileo: Thanks. Yeah, it seems to fail intermittently. | 13:23 |
smcginnis | geguileo: tommylikehu said he was going to look in to it, so if you have any advice for him... | 13:24 |
geguileo | I'll try to debug it, because my initial idea would mean it always fails... | 13:24 |
geguileo | oh, ok | 13:24 |
*** Poornima_K has quit IRC | 13:26 | |
*** cknight has joined #openstack-cinder | 13:28 | |
*** lucasxu has joined #openstack-cinder | 13:30 | |
*** crushil has quit IRC | 13:32 | |
*** zhangguoqing has quit IRC | 13:35 | |
openstackgerrit | Merged openstack/cinder master: Mock qemu calls in sheepdog tests https://review.openstack.org/465626 | 13:39 |
*** gcb has joined #openstack-cinder | 13:45 | |
*** markstur has quit IRC | 13:46 | |
*** dave-mcc_ has joined #openstack-cinder | 13:47 | |
*** dave-mccowan has quit IRC | 13:49 | |
*** dave-mccowan has joined #openstack-cinder | 13:51 | |
*** crushil has joined #openstack-cinder | 13:53 | |
*** eharney has joined #openstack-cinder | 13:53 | |
*** lkuchlan has quit IRC | 13:53 | |
*** dave-mcc_ has quit IRC | 13:53 | |
*** bjornar_ has quit IRC | 13:54 | |
*** jwcroppe has quit IRC | 13:55 | |
*** jwcroppe has joined #openstack-cinder | 13:55 | |
*** chlong has joined #openstack-cinder | 13:55 | |
*** links has quit IRC | 13:57 | |
openstackgerrit | Shay Halsband proposed openstack/cinder master: XtremIO: reduce number of backend calls https://review.openstack.org/467245 | 13:57 |
*** Julien-zte has joined #openstack-cinder | 13:58 | |
*** jwcroppe has quit IRC | 13:59 | |
*** Julien-zte has quit IRC | 14:02 | |
*** Julien-zte has joined #openstack-cinder | 14:02 | |
*** reduxio has quit IRC | 14:03 | |
*** reduxio has joined #openstack-cinder | 14:04 | |
*** mtanino__ has joined #openstack-cinder | 14:04 | |
*** JoseMello has joined #openstack-cinder | 14:05 | |
jungleboyj | arnewiebalck__: That is great news. | 14:06 |
*** reduxio has quit IRC | 14:08 | |
*** Julien-zte has quit IRC | 14:08 | |
hemna | guwak | 14:08 |
*** Julien-zte has joined #openstack-cinder | 14:08 | |
*** salv-orlando has joined #openstack-cinder | 14:10 | |
*** dikonoor has joined #openstack-cinder | 14:10 | |
*** kfarr has joined #openstack-cinder | 14:11 | |
*** jwcroppe has joined #openstack-cinder | 14:12 | |
jungleboyj | arnewiebalck__: gerhardqux smcginnis I just verified that I also had mysql+pymysql set on the environment where I was trying to recreate this. | 14:13 |
jungleboyj | So, it appears that we found a solution. Thank you so much gerhardqux ! | 14:13 |
*** Cibo_ has joined #openstack-cinder | 14:13 | |
*** dikonoo has quit IRC | 14:13 | |
*** salv-orlando has quit IRC | 14:14 | |
*** dfisher has joined #openstack-cinder | 14:15 | |
jungleboyj | arnewiebalck__: What connection string were you using that failed? | 14:16 |
*** reduxio has joined #openstack-cinder | 14:19 | |
*** xyang has joined #openstack-cinder | 14:20 | |
gerhardqux | maybe oslo.db should log warnings when the mysql libc driver is used in combination with eventlets | 14:22 |
arnewiebalck__ | jungleboyj: mysql://… | 14:23 |
arnewiebalck__ | jungleboyj: I guess this has been carried forward since we set up Cinder some 4 years ago. | 14:24 |
gerhardqux | or create something like "cinder-manage doctor" to give warnings for situations that are not recommended (like running without cache) | 14:26 |
*** digvijay2016 has joined #openstack-cinder | 14:29 | |
openstackgerrit | xing-yang proposed openstack/cinder master: Tiramisu: replication group support https://review.openstack.org/352228 | 14:30 |
jungleboyj | arnewiebalck__: Ok, so you are thinking that this was an old configuration option that was pulled forward from your older config files? | 14:30 |
arnewiebalck__ | jungleboyj: FWIU, pymysql would be the default, no? | 14:32 |
jungleboyj | arnewiebalck__: That is the default I am seeing. | 14:32 |
*** gkadam has quit IRC | 14:33 | |
*** reduxio has quit IRC | 14:33 | |
arnewiebalck__ | jungleboyj: Where does your default come from? | 14:34 |
arnewiebalck__ | The docs do not specify which sql to use, do they? | 14:35 |
jungleboyj | arnewiebalck__: Good question. Let me look here. | 14:35 |
smcginnis | arnewiebalck__: Last time I went through the install guide, it did state to use pymysql. | 14:35 |
smcginnis | gerhardqux: That might be something good to propose to oslo.db. At least a debug warning or something so there's a chance it could be found. | 14:36 |
jungleboyj | arnewiebalck__: In my case it was being set by lib/database in devstack it appears. | 14:38 |
*** eharney has quit IRC | 14:38 | |
jungleboyj | smcginnis: ++ | 14:39 |
*** Cibo_ has quit IRC | 14:41 | |
arnewiebalck__ | jungleboyj: in our case it’s set by the Puppet module which extracts it from our secret store … I guess it was set once and never looked at again (as it “worked”) :) | 14:43 |
lpetrut | Hi guys, I have this chain of patches on the SMBFS driver (some on the base RemoteFS one as well), already have +2s (thanks Jay), could use a second one :) https://review.openstack.org/#/c/424233/ | 14:43 |
arnewiebalck__ | jungleboyj: the default in the Pupper module is 'sqlite:////var/lib/cinder/cinder.sqlite' | 14:44 |
smcginnis | Wow, they default to sqlite? I wonder why they would do that. | 14:44 |
* jungleboyj is baffled. :-) | 14:45 | |
dfisher | it defaults to sqlite likely because it requires literally no additional components to "work" | 14:46 |
dfisher | mysql/maria requires those packages to be installed, services to be running, etc. not saying that's hard (especially with puppet), but it's additional work | 14:46 |
smcginnis | dfisher: Yeah, sqlite is great for tests for that reason. But that's the only case I would use it. :) | 14:47 |
dfisher | oh, absolutely. | 14:47 |
*** nicolasbock has quit IRC | 14:47 | |
*** chhavi has joined #openstack-cinder | 14:47 | |
hemna | jungleboyj, https://goo.gl/pllr6g | 14:47 |
smcginnis | :) | 14:48 |
jungleboyj | hemna: :-) | 14:48 |
openstackgerrit | Andrey Volkov proposed openstack/cinder master: api-ref cleanup: required flag for tenants https://review.openstack.org/406056 | 14:49 |
jungleboyj | I was thinking something more like this: http://gph.is/1jskbwn | 14:49 |
*** psachin has quit IRC | 14:50 | |
arnewiebalck__ | smcginnis: this was mostly to say that users of the Puppet module for Cinder are not misled by the default, they have to actively set a value (which may create sadness down the road like in our case) | 14:51 |
smcginnis | arnewiebalck__: True. | 14:53 |
*** eharney has joined #openstack-cinder | 14:53 | |
arnewiebalck__ | anyway, kudos to gerhardqux for having spotted this! | 14:53 |
smcginnis | Yeah, good catch! | 14:53 |
arnewiebalck__ | I’ll deploy this and run some more serious tests with larger Magnum clusters to see how it does in production. | 14:54 |
jungleboyj | Very good catch. | 14:55 |
jungleboyj | smcginnis: So do we have a follow up here? | 14:55 |
*** lpetrut has quit IRC | 14:57 | |
*** itlinux has joined #openstack-cinder | 14:57 | |
openstackgerrit | Gyorgy Szombathelyi proposed openstack/cinder master: Use the deprecated os_privileged_xxx settings https://review.openstack.org/467132 | 14:57 |
*** diablo_rojo has joined #openstack-cinder | 14:58 | |
*** digvijay2016 has quit IRC | 14:59 | |
smcginnis | jungleboyj: I don't think there's much we can do from the Cinder side. | 14:59 |
smcginnis | jungleboyj: But I like the idea of seeing if there's something that can be done - at least a warning - on the oslo.db side. | 15:00 |
openstackgerrit | xing-yang proposed openstack/python-cinderclient master: Tiramisu: replication group support https://review.openstack.org/352229 | 15:00 |
arnewiebalck__ | smcginnis: +1 | 15:00 |
jungleboyj | arnewiebalck__: smcginnis ++ | 15:01 |
jungleboyj | Yeah, if it known that this could blow up on you some sort of warning should be produced. | 15:01 |
*** dpawlik has quit IRC | 15:02 | |
jungleboyj | arnewiebalck__: or gerhardqux Is that something you could propose to Oslo? | 15:02 |
*** dpawlik has joined #openstack-cinder | 15:02 | |
*** dpawlik has quit IRC | 15:02 | |
arnewiebalck__ | smcginnis: what about the “SQLAlchemy connection string comment” in https://docs.openstack.org/ocata/config-reference/block-storage/samples/cinder.conf.html ? | 15:02 |
*** markstur has joined #openstack-cinder | 15:04 | |
*** dpawlik has joined #openstack-cinder | 15:04 | |
smcginnis | arnewiebalck__: Do people read comments? :D | 15:04 |
smcginnis | arnewiebalck__: Yeah, we could add a note there for sure! | 15:04 |
smcginnis | arnewiebalck__: You want to do that? | 15:04 |
arnewiebalck__ | smcginnis: well, at least you can say “look here” :-D | 15:05 |
smcginnis | +1 | 15:05 |
*** dpawlik has quit IRC | 15:05 | |
jungleboyj | arnewiebalck__: +1 | 15:06 |
*** dpawlik has joined #openstack-cinder | 15:07 | |
*** dpawlik is now known as danpawlik | 15:07 | |
jungleboyj | So if you want to propose an update there we can help get that through. | 15:09 |
*** danpawlik has quit IRC | 15:09 | |
*** dustins has quit IRC | 15:11 | |
*** mlakat has quit IRC | 15:11 | |
arnewiebalck__ | jungleboyj: that potentially affects more projects than just Cinder, no? | 15:11 |
*** lhx__ has joined #openstack-cinder | 15:11 | |
*** dpawlik has joined #openstack-cinder | 15:12 | |
*** dpawlik is now known as danpawlik | 15:12 | |
*** dustins has joined #openstack-cinder | 15:12 | |
*** danpawlik has quit IRC | 15:13 | |
smcginnis | Yeah, that's why I think it would be good to get something in oslo.db. | 15:14 |
arnewiebalck__ | smcginnis: agreed, I can give that a try | 15:14 |
jungleboyj | Yeah, that is the best way to go. | 15:14 |
*** lpetrut has joined #openstack-cinder | 15:14 | |
smcginnis | I wonder if this would be worth a post the the ops ML to spread the word. I can do that. | 15:15 |
jungleboyj | smcginnis: And I can bring it up at the next Oslo meeting. | 15:16 |
*** danpawlik has joined #openstack-cinder | 15:18 | |
smcginnis | jungleboyj: +1 | 15:18 |
openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Base functionality, replacing SMI-S with REST https://review.openstack.org/455428 | 15:20 |
*** rcernin has quit IRC | 15:22 | |
jungleboyj | Ok, so we have Arne proposing a warning to Oslo, Sean sending a note to the mailing list and me making the Oslo team aware of it. | 15:23 |
*** nicolasbock has joined #openstack-cinder | 15:24 | |
smcginnis | jungleboyj: Done | 15:26 |
jungleboyj | Cool. | 15:26 |
openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Retype, replacing SMI-S with REST https://review.openstack.org/456757 | 15:28 |
openstackgerrit | Lucian Petrut proposed openstack/cinder master: Fix fetching VHD images from Glance https://review.openstack.org/467285 | 15:30 |
*** chlong has quit IRC | 15:31 | |
*** Julien-zte has quit IRC | 15:33 | |
*** ociuhandu has quit IRC | 15:35 | |
*** Julien-zte has joined #openstack-cinder | 15:35 | |
*** Julien-zte has quit IRC | 15:36 | |
*** belmoreira has quit IRC | 15:37 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - QoS, replacing SMI-S with REST https://review.openstack.org/456759 | 15:37 |
jungleboyj | lpetrut: You are welcome. Thanks for quickly responding to my comments. | 15:39 |
*** itlinux has quit IRC | 15:39 | |
arnewiebalck__ | jungleboyj: https://bugs.launchpad.net/oslo.db/+bug/1692956 | 15:41 |
openstack | Launchpad bug 1692956 in oslo.db "Warn about potentially misconfigured connection string " [Undecided,New] | 15:41 |
jungleboyj | arnewiebalck__: Cool. | 15:42 |
*** chlong has joined #openstack-cinder | 15:43 | |
*** lucasxu has quit IRC | 15:44 | |
*** pcaruana has quit IRC | 15:47 | |
openstackgerrit | Jianghua Wang proposed openstack/cinder master: Create volume from VHD format image always failed https://review.openstack.org/467039 | 15:49 |
*** lucasxu has joined #openstack-cinder | 15:49 | |
*** bjornar_ has joined #openstack-cinder | 15:51 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Compression, replacing SMI-S with REST https://review.openstack.org/460191 | 15:51 |
*** e0ne has quit IRC | 15:52 | |
*** Apoorva has joined #openstack-cinder | 15:59 | |
*** Apoorva has quit IRC | 15:59 | |
openstackgerrit | Merged openstack/cinder master: Updated from global requirements https://review.openstack.org/467141 | 15:59 |
*** Apoorva has joined #openstack-cinder | 16:00 | |
*** cristicalin has joined #openstack-cinder | 16:05 | |
openstackgerrit | John Griffith proposed openstack/cinder master: Add blockbox to Cinder project https://review.openstack.org/467304 | 16:07 |
smcginnis | jgriffith: woot | 16:07 |
openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Replication, replacing SMI-S with REST https://review.openstack.org/466027 | 16:11 |
*** lucasxu has quit IRC | 16:14 | |
jgriffith | smcginnis worth a shot :) | 16:15 |
*** lpetrut has quit IRC | 16:22 | |
*** chlong has quit IRC | 16:30 | |
*** salv-orlando has joined #openstack-cinder | 16:31 | |
*** winston-d_ has joined #openstack-cinder | 16:32 | |
*** mariusv is now known as lunarlamp | 16:33 | |
*** reduxio has joined #openstack-cinder | 16:35 | |
*** gouthamr has joined #openstack-cinder | 16:35 | |
*** reduxio has quit IRC | 16:40 | |
openstackgerrit | xing-yang proposed openstack/cinder master: Tiramisu: replication group support https://review.openstack.org/352228 | 16:43 |
*** mvk has quit IRC | 16:43 | |
openstackgerrit | Gyorgy Szombathelyi proposed openstack/cinder master: Use the deprecated os_privileged_xxx settings https://review.openstack.org/467132 | 16:45 |
*** aarefiev is now known as aarefiev_afk | 16:45 | |
*** lpetrut has joined #openstack-cinder | 16:49 | |
*** dikonoor has quit IRC | 16:50 | |
openstackgerrit | xing-yang proposed openstack/cinder master: Tiramisu: replication group support https://review.openstack.org/352228 | 16:54 |
*** harlowja has joined #openstack-cinder | 16:56 | |
*** eharney has quit IRC | 16:59 | |
*** Cibo_ has joined #openstack-cinder | 16:59 | |
jungleboyj | jgriffith: Ironic. You pushed that up while I was talking to a co-worker about running Cinder in containers. :-) | 16:59 |
*** salv-orlando has quit IRC | 17:03 | |
*** cristicalin has quit IRC | 17:04 | |
*** Apoorva_ has joined #openstack-cinder | 17:04 | |
*** Apoorva has quit IRC | 17:08 | |
*** cristicalin has joined #openstack-cinder | 17:10 | |
*** lhx__ has quit IRC | 17:13 | |
*** eharney has joined #openstack-cinder | 17:15 | |
*** cristicalin has quit IRC | 17:17 | |
openstackgerrit | Merged openstack/cinder master: Use the list iterator instead of six iterator https://review.openstack.org/449869 | 17:18 |
*** salv-orlando has joined #openstack-cinder | 17:19 | |
*** kfarr has quit IRC | 17:19 | |
*** salv-orlando has quit IRC | 17:23 | |
*** itlinux has joined #openstack-cinder | 17:24 | |
*** Apoorva_ has quit IRC | 17:33 | |
*** ebbex has quit IRC | 17:36 | |
*** Anticimex has quit IRC | 17:38 | |
*** ebbex has joined #openstack-cinder | 17:38 | |
*** Anticimex has joined #openstack-cinder | 17:39 | |
*** dhinesh has joined #openstack-cinder | 17:41 | |
openstackgerrit | Dhinesh Balasubramaniam proposed openstack/cinder master: Hedvig Cinder driver implementation and unit tests. https://review.openstack.org/433341 | 17:42 |
*** lucasxu has joined #openstack-cinder | 17:43 | |
*** ChubYann has joined #openstack-cinder | 17:46 | |
*** salv-orlando has joined #openstack-cinder | 17:51 | |
*** markstur has quit IRC | 17:52 | |
*** cknight has quit IRC | 17:54 | |
*** anticw has left #openstack-cinder | 17:56 | |
*** Apoorva has joined #openstack-cinder | 17:56 | |
*** Cibo_ has quit IRC | 17:57 | |
hemna | jgriffith, ok so I just tried out the cinder patch for the block-box | 17:59 |
hemna | the build worked, but I still can't fire up the devenv container | 17:59 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/467342 | 17:59 |
*** xyang has quit IRC | 18:01 | |
*** itlinux has quit IRC | 18:02 | |
*** cknight has joined #openstack-cinder | 18:04 | |
*** lpetrut has quit IRC | 18:05 | |
jgriffith | hemna you know it just runs a command and exits right? | 18:06 |
jgriffith | Unless you launch bash | 18:06 |
hemna | it just gens the conf | 18:07 |
jgriffith | If that’s all you tell it to do yes | 18:07 |
hemna | yah I just figured that out | 18:07 |
hemna | heh | 18:07 |
hemna | what was throwing me off was the puking of the oslo warnings | 18:07 |
jgriffith | If you want to poke at things just change the command to `bash` to interact with it | 18:07 |
jgriffith | Oh! Yeah, so that happens in normal env too I noticed | 18:07 |
hemna | yup | 18:08 |
hemna | I just ran that outside of the container and saw that....so the lightbulb turned on | 18:08 |
jgriffith | Yeah, it threw me off at first too | 18:08 |
openstackgerrit | Merged openstack/cinder-specs master: Correct the filter name for changes-since spec https://review.openstack.org/462015 | 18:08 |
jgriffith | I’m not quite sure why we don’t get the pretty green success message | 18:08 |
jgriffith | Oh, yeah I know | 18:08 |
jgriffith | That comes from the top layer | 18:08 |
jgriffith | tox | 18:09 |
jgriffith | Stupid auto-correct | 18:09 |
*** cknight has quit IRC | 18:09 | |
hemna | ok the review is in the merge pipeline now :) | 18:09 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext master: Updated from global requirements https://review.openstack.org/467345 | 18:09 |
jgriffith | I misspell the right words, it spells the wrong words correctly | 18:09 |
jgriffith | Don’t know which is better/worse | 18:09 |
jgriffith | Oh sweet!!! | 18:09 |
hemna | yah I really despise autocorrect | 18:09 |
jgriffith | hemna so if you want to tinker, using that devbox as a development environment from a Mac works pretty nice | 18:11 |
jgriffith | I do have to figure out something with number of threads for unit tests, cuz a single thread for our unit test suite is horrifyingly long | 18:11 |
ildikov | hemna: autocorrect on smart phones is pretty useful at least to me, no text communication without that... | 18:11 |
hemna | I find it corrects stuff it shouldn't most of the time. | 18:12 |
smcginnis | jgriffith: Single thread is painfully looon. | 18:12 |
smcginnis | g | 18:12 |
smcginnis | Loony too I guess. | 18:12 |
ildikov | hemna: I'm not saying we don't have our moments, but I wouldn't despise it for sure :) | 18:12 |
jgriffith | There is an option to allocate more cpus, but I haven’t messed with it | 18:12 |
*** itlinux has joined #openstack-cinder | 18:12 | |
jgriffith | smcginnis auto-correct? | 18:13 |
jgriffith | :) | 18:13 |
*** reduxio has joined #openstack-cinder | 18:13 | |
jungleboyj | I like painfully loooooon | 18:13 |
smcginnis | Painfully loooong, painfully loony. | 18:13 |
ildikov | jgriffith: lol :) | 18:13 |
jgriffith | smcginnis bottom line “it ain’t good" | 18:13 |
jgriffith | hemna BTW, what’s kinda neat is using those templates to plug in to a devstack that’s running without Cinder | 18:14 |
jgriffith | So I have a long running devstack with no cinder that just runs, I swap version of cinder in and out depending on what I’m working on | 18:15 |
jgriffith | Anyway…. I hope people check it out and find it useful/improve on it | 18:15 |
hemna | cinder development can't get much better than having all the other services just up and cinder in a container. | 18:15 |
jungleboyj | hemna: Starting to see the value there. | 18:15 |
smcginnis | Sweet - never thought of doing it that way. | 18:15 |
hemna | I'll use separate source trees and another container to test out different stuffs, instead of an entire new vagrant. | 18:15 |
hemna | and, I'd say it's a great way to look at testing out active/active HA cinder | 18:16 |
jungleboyj | jgriffith: That is cool. | 18:16 |
hemna | after you start using docker containers for apps, it's really hard to go back | 18:16 |
jungleboyj | Some really nice guy was good enough to explain this craze to me yesterday. Now I don't feel like I am out of the loop on Containers. :-) | 18:17 |
smcginnis | :) | 18:18 |
mriedem | hemna: smcginnis: jungleboyj: is there no standard format for the keys that go into the device_info dict returned from connect_volume in os-brick? | 18:18 |
mriedem | because i see some volume drivers in nova are using device_info['path'] | 18:18 |
mriedem | but veritas is using device_path https://review.openstack.org/#/c/466437/1/os_brick/initiator/connectors/vrtshyperscale.py | 18:18 |
hemna | some of the connectors are not block types, they are object types | 18:19 |
*** reduxio has quit IRC | 18:19 | |
hemna | mriedem, is the libvirt volume driver looking for device_path for vrts ? | 18:20 |
mriedem | it will be | 18:20 |
hemna | ok, well it's easy to change that to path :) | 18:20 |
hemna | would you like me to? | 18:20 |
mriedem | well, | 18:20 |
mriedem | that's kind of a break in the api isn't it? | 18:20 |
mriedem | to os-brick | 18:20 |
mriedem | i guess right now the libvirt volume driver is not even using it | 18:21 |
hemna | nobody is using that connector right now | 18:21 |
hemna | I'd have to look at the cinder driver to see what it's looking for, in the case of copy image <--> volume case | 18:21 |
hemna | but that's easily changeable too | 18:22 |
mriedem | i don't see the volume driver in cinder using brick at all | 18:23 |
hemna | I was just looking at that | 18:24 |
hemna | looks like all the real work is done in the veritas/utils.py | 18:25 |
hemna | so I think it's probably safe to change that to 'path' from 'device_path' and then -1 the veritas libvirt review | 18:25 |
mriedem | i've already -1ed the nova change, | 18:26 |
mriedem | but because i wanted them to use the 1.13.0 brick version from g-r so they could just do the todo in the nova code now, | 18:26 |
mriedem | but if we're going to change brick, then we have to wait for another release | 18:26 |
mriedem | anywho | 18:26 |
smcginnis | We're going to be at os-brick 1.55.0 by the time this is all straightened out. | 18:27 |
hemna | the vrts connector isn't in a current os-brick release yet | 18:27 |
hemna | it's just in github | 18:27 |
*** felipemonteiro has joined #openstack-cinder | 18:27 | |
hemna | I'll push up a patch to change it | 18:28 |
mriedem | it's in 1.13.0 | 18:28 |
smcginnis | hemna: And 1.12.0 | 18:28 |
smcginnis | And maybe even 1.11.0 | 18:28 |
hemna | bah | 18:28 |
mriedem | maybe need a test to whitelist the keys in the device_info dict | 18:28 |
mriedem | or use a versioned object or something | 18:28 |
hemna | guess nobody caught that in the reviews | 18:28 |
*** felipemonteiro_ has joined #openstack-cinder | 18:28 | |
hemna | I didn't get a chance to review that one before it landed. | 18:29 |
mriedem | went into 1.12.0, the connector i mean | 18:29 |
mriedem | anyway | 18:29 |
mriedem | we have time to wait on the nova side | 18:29 |
mriedem | just do a patch release | 18:29 |
mriedem | 1.13.1 or something | 18:29 |
smcginnis | mriedem: Yeah, this one we should be able to do that. | 18:29 |
hemna | https://gist.github.com/0307e9d63c24da43fc7843effdc49575 | 18:30 |
smcginnis | Last one it wasn't until a bunch of other stuff merged that they realized they needed to change something. | 18:30 |
mriedem | btw, nothing is using it except veritas' k8s product which uses standalone cinder and brick | 18:30 |
smcginnis | hemna: Looks simple enough. | 18:30 |
hemna | are we ok with that? or do you want me to keep device_path in for anyone that might be using it :( | 18:31 |
mriedem | i was joking about the k8s thing, | 18:31 |
hemna | ok :) | 18:31 |
mriedem | but in the future utopia where you guys don't have nova, you might have to care about who is using your code | 18:31 |
smcginnis | hemna: Might be safe to keep it compatible. Or they can comment on the patch if they need it. | 18:32 |
*** arunkant has joined #openstack-cinder | 18:32 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick master: Fixed the veritas connector path https://review.openstack.org/467356 | 18:32 |
*** felipemonteiro has quit IRC | 18:33 | |
*** itlinux has quit IRC | 18:34 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/python-brick-cinderclient-ext master: Added bindep support https://review.openstack.org/466110 | 18:34 |
*** itlinux has joined #openstack-cinder | 18:43 | |
*** salv-orlando has quit IRC | 18:46 | |
*** JoseMello has quit IRC | 18:55 | |
*** salv-orlando has joined #openstack-cinder | 18:56 | |
*** markstur has joined #openstack-cinder | 18:57 | |
*** itlinux has quit IRC | 19:01 | |
*** markstur has quit IRC | 19:01 | |
*** salv-orlando has quit IRC | 19:02 | |
*** lucasxu has quit IRC | 19:02 | |
*** Apoorva has quit IRC | 19:04 | |
*** Apoorva has joined #openstack-cinder | 19:04 | |
*** xyang has joined #openstack-cinder | 19:05 | |
*** mvk has joined #openstack-cinder | 19:06 | |
*** adisky_ has quit IRC | 19:07 | |
*** salv-orlando has joined #openstack-cinder | 19:10 | |
*** salv-orlando has quit IRC | 19:11 | |
openstackgerrit | Merged openstack/cinder master: Support 'LIKE' operator to filter resource https://review.openstack.org/445440 | 19:11 |
openstackgerrit | Merged openstack/cinder master: Fix typo in docstring https://review.openstack.org/464156 | 19:11 |
*** cristicalin has joined #openstack-cinder | 19:15 | |
*** lucasxu has joined #openstack-cinder | 19:15 | |
*** xyang1 has joined #openstack-cinder | 19:16 | |
jgriffith | mriedem you’re being a bit unfair don’t you think? | 19:17 |
mriedem | a bit yes | 19:18 |
jgriffith | mriedem we might not run as tight a ship as some, but it’s tighter than others… and at least we’re trying to work together and to the right thing | 19:19 |
mriedem | my point being we can assume things within the openstack arena, but once there are outside consumers, it will be tougher | 19:19 |
jgriffith | mriedem understood/agreed | 19:20 |
*** crushil has quit IRC | 19:20 | |
jungleboyj | mriedem: Always trying to harsh our mellow. | 19:22 |
jungleboyj | ;-) | 19:22 |
mriedem | can it hippy | 19:22 |
jungleboyj | It is what happens when I am left out here looking at the lake all day. | 19:23 |
openstackgerrit | Merged openstack/cinder master: [Trivial] SMBFS: fix passing image format using volume types https://review.openstack.org/464603 | 19:26 |
openstackgerrit | Merged openstack/cinder master: VMAX driver - Manage/Unmanage performance fix https://review.openstack.org/456209 | 19:27 |
openstackgerrit | Merged openstack/cinder master: VMAX driver - Close ecom connections https://review.openstack.org/463744 | 19:27 |
openstackgerrit | Merged openstack/python-cinderclient master: Cinder client reset-state improvements https://review.openstack.org/446356 | 19:28 |
*** reduxio has joined #openstack-cinder | 19:30 | |
jgriffith | Ha, never thought I’d here jungleboyj described as a *hippy* | 19:33 |
jungleboyj | :-) | 19:34 |
*** reduxio has quit IRC | 19:34 | |
*** crushil has joined #openstack-cinder | 19:41 | |
*** mvk has quit IRC | 19:43 | |
*** eharney has quit IRC | 19:46 | |
jgriffith | smcginnis so I wonder what you and others think about the idea of bfs orchestration living “somewhere else" | 19:46 |
jgriffith | As in OSC or even Cinder | 19:46 |
smcginnis | bfs? | 19:47 |
jgriffith | Bahhh!!! Damn auto-correct!!!! | 19:47 |
jgriffith | I hate you! | 19:47 |
jgriffith | BFV | 19:47 |
jungleboyj | Bathroom Fixture Store | 19:47 |
*** clenimar has quit IRC | 19:47 | |
jgriffith | Well yeah! We all want nice handles on our faucets | 19:47 |
smcginnis | jgriffith: Interesting that you mention that. | 19:47 |
jungleboyj | :-) | 19:48 |
smcginnis | jgriffith: I think there are a few "orchestration" things that don't fit in Cinder or Nova or X. | 19:48 |
*** itlinux has joined #openstack-cinder | 19:48 | |
jgriffith | smcginnis you working on a new project? :) | 19:48 |
smcginnis | jgriffith: And we need some layer/tool/orchestrator/whateveryouwanttocallit that provides a nice simple API and does all the little steps underneath to service it. | 19:48 |
jgriffith | indeed | 19:49 |
smcginnis | jgriffith: Nah, I don't even have time to write Cinder code. :D | 19:49 |
jgriffith | There was a talk about this in Boston | 19:49 |
jgriffith | Can’t remember what it was though... | 19:49 |
jgriffith | But I was thinking, OSC might be an interesting place for such things | 19:49 |
jgriffith | Assuming that dtroyer and team had any remote interest at al | 19:49 |
smcginnis | I need to look into it yet, but what avbout oaktree? | 19:50 |
*** e0ne has joined #openstack-cinder | 19:51 | |
jgriffith | I hadn’t thought of that | 19:52 |
jgriffith | This is what I was thinking of: https://github.com/jaypipes/enamel | 19:52 |
*** markstur has joined #openstack-cinder | 19:52 | |
smcginnis | Oh yeah, that was the other one I've heard mentioned. | 19:52 |
jgriffith | Need to ask jay pipes about that | 19:52 |
smcginnis | jgriffith: Just doing a quick read through of that, it might be the better fit. | 19:53 |
jgriffith | Will look at oak tree, maybe that’s the same deal… but there’s that whole sensible layer there | 19:53 |
smcginnis | Might have to bookmark that one for further reading. | 19:53 |
openstackgerrit | Merged openstack/cinder master: rbd: resize volume not cloned but full copy https://review.openstack.org/295480 | 19:54 |
openstackgerrit | Merged openstack/cinder master: Python3: Don't use Exception.message https://review.openstack.org/463230 | 19:55 |
*** crushil has quit IRC | 19:56 | |
*** crushil has joined #openstack-cinder | 19:56 | |
smcginnis | jgriffith: I think this is an interesting point: http://inaugust.com/talks/oaktree.html#/11 | 19:58 |
smcginnis | It might be nice to separate out our inter-service APIs from our client facing APIs through something like this. | 19:58 |
mriedem | osc isn't an api binding | 19:58 |
mriedem | the sdk is | 19:58 |
openstackgerrit | Merged openstack/python-brick-cinderclient-ext master: Updated from global requirements https://review.openstack.org/467345 | 19:58 |
mriedem | s/the/one of the 50/ | 19:58 |
smcginnis | mriedem: Yeah, I think the idea is looking at what could become a better interface. | 19:59 |
mriedem | k8s will solve all the problems | 20:00 |
*** eharney has joined #openstack-cinder | 20:01 | |
jgriffith | mriedem and create a billion new ones! :) | 20:01 |
jgriffith | I was just thinking it’d be interesting if OSC did more than just wrap up all the existing clients, but I guess that’s crazy | 20:02 |
ildikov | jgriffith: I think it depends also on the timeline you set for that to happen | 20:03 |
jgriffith | ildikov next week :) | 20:03 |
mriedem | jgriffith: what's crazy is you wanting to create servers through cinder when doing bfv | 20:03 |
ildikov | jgriffith: in that case it's *insane* :) | 20:03 |
jgriffith | mriedem :) I said it was an awful idea | 20:03 |
jgriffith | But not much worse than creating volumes from nova when you boot instances :) | 20:04 |
mriedem | someone is already working on an api extension for it in their private cloud now | 20:04 |
jgriffith | mriedem surely you jest | 20:04 |
smcginnis | From the cinder side? | 20:05 |
mriedem | yes i'm joking, | 20:05 |
*** jaypipes has joined #openstack-cinder | 20:05 | |
smcginnis | jaypipes: Hey! | 20:05 |
mriedem | but someone probably thought, "i can write that" | 20:05 |
jaypipes | hola smcginnis :) | 20:05 |
ildikov | mriedem: well, multi-attach is in production, so I can imagine almost anything by now... | 20:05 |
*** cknight has joined #openstack-cinder | 20:06 | |
smcginnis | jaypipes: We were just talking about some upper layer that can coordinate things like creating cinder volumes that then creates nova instances to boot from them. | 20:06 |
jgriffith | Wow… he’s kinda like BeetleJuice | 20:06 |
smcginnis | jaypipes: Is that along the lines of the enamel stuff you've been doing? | 20:06 |
mriedem | ildikov: multiattach is i'm sure in multiple clouds for multiple years now | 20:06 |
smcginnis | jgriffith: Just gotta say his name three times. | 20:06 |
mriedem | and people are having a ball rebasing it | 20:06 |
ildikov | oh they do | 20:07 |
ildikov | at least I don't have those patches up for nothing :) | 20:07 |
jaypipes | smcginnis: enamel was something me, cdent, tdurakov and rpodolyaka played around with in Feb last year. Didn't go anywhere -- not enough time to work on it -- but the intent was definitely to be the thing that would be a porcelain API, yes. | 20:08 |
ildikov | mriedem: I asked the guy to upload his version, they're on Mitaka right now with it: https://review.openstack.org/#/c/466184/ | 20:08 |
jaypipes | thus the name enamel :) | 20:08 |
ildikov | mriedem: and here's the Cinder side: https://review.openstack.org/#/c/466218/ | 20:08 |
mriedem | ildikov: that's ibm right? | 20:08 |
smcginnis | jaypipes: So the idea is basically to have a simplified API layer that has a little business logic to hide the sausage making from the consumer? | 20:08 |
mriedem | they've had multiattach in powervc for years | 20:08 |
mriedem | they never helped upstream, but meh | 20:09 |
ildikov | not sure at this point whether we can reuse anything from it, but worth a look still | 20:09 |
jaypipes | smcginnis: yes, xactly. | 20:09 |
ildikov | mriedem: not IBM | 20:09 |
smcginnis | jaypipes: Cool. I might have to try to play around with what you've got up there. | 20:09 |
smcginnis | jaypipes: There may be more interest for that now. | 20:09 |
ildikov | mriedem: would need to look it up, but I'm sure not IBM | 20:10 |
*** dave-mccowan has quit IRC | 20:10 | |
smcginnis | jaypipes: Thanks for the info. I might ping you if I actually get anywhere with that and have questions. | 20:10 |
*** boris-42_ has joined #openstack-cinder | 20:11 | |
mriedem | ildikov: samsung now, | 20:11 |
mriedem | but used to be ibm | 20:11 |
mriedem | https://www.linkedin.com/in/abillore/?ppe=1 | 20:11 |
mriedem | i remember, from when he was on powervc | 20:11 |
ildikov | mriedem: oh ok, I didn't have that background | 20:12 |
ildikov | mriedem: I'm trying to be more forward looking usually :) | 20:12 |
*** viks has joined #openstack-cinder | 20:15 | |
*** lucasxu has quit IRC | 20:17 | |
*** e0ne has quit IRC | 20:18 | |
jaypipes | smcginnis: sure thing, any time :) | 20:20 |
*** lucasxu has joined #openstack-cinder | 20:21 | |
openstackgerrit | Merged openstack/python-brick-cinderclient-ext master: Added bindep support https://review.openstack.org/466110 | 20:24 |
*** rcernin has joined #openstack-cinder | 20:24 | |
dfisher | is there any way to tell what 3rd party CI systems are still working on a changeset? | 20:25 |
*** crushil has quit IRC | 20:26 | |
*** crushil has joined #openstack-cinder | 20:27 | |
*** crushil has quit IRC | 20:28 | |
smcginnis | dfisher: No, no standard external reporting on that. | 20:28 |
dfisher | bummer. | 20:28 |
*** Apoorva_ has joined #openstack-cinder | 20:32 | |
*** harlowja_ has joined #openstack-cinder | 20:32 | |
*** harlowja has quit IRC | 20:33 | |
*** Apoorva has quit IRC | 20:36 | |
*** jose-phillips has quit IRC | 20:39 | |
*** jose-phillips has joined #openstack-cinder | 20:41 | |
openstackgerrit | xing-yang proposed openstack/cinder master: Tiramisu: replication group support https://review.openstack.org/352228 | 20:49 |
*** david-lyle has joined #openstack-cinder | 20:54 | |
openstackgerrit | xing-yang proposed openstack/python-cinderclient master: Tiramisu: replication group support https://review.openstack.org/352229 | 20:59 |
*** itlinux has quit IRC | 21:00 | |
openstackgerrit | xing-yang proposed openstack/python-cinderclient master: Tiramisu: replication group support https://review.openstack.org/352229 | 21:01 |
*** lucasxu has quit IRC | 21:03 | |
*** harlowja_ has quit IRC | 21:05 | |
*** gouthamr has quit IRC | 21:05 | |
*** Apoorva_ has quit IRC | 21:06 | |
*** Apoorva has joined #openstack-cinder | 21:07 | |
*** harlowja has joined #openstack-cinder | 21:08 | |
*** david-lyle has quit IRC | 21:10 | |
*** eharney has quit IRC | 21:16 | |
-openstackstatus- NOTICE: The logserver has filled up, so jobs are currently aborting with POST_FAILURE results; remediation is underway. | 21:20 | |
*** ChanServ changes topic to "The logserver has filled up, so jobs are currently aborting with POST_FAILURE results; remediation is underway." | 21:20 | |
*** david-lyle has joined #openstack-cinder | 21:22 | |
*** reduxio has joined #openstack-cinder | 21:24 | |
openstackgerrit | Merged openstack/cinder master: Tests: Fix VMAX hash randomization failure https://review.openstack.org/465680 | 21:27 |
openstackgerrit | Merged openstack/cinder master: Tests: Fix Blockbridge hash randomization failure https://review.openstack.org/465681 | 21:28 |
*** reduxio has quit IRC | 21:28 | |
*** jwcroppe has quit IRC | 21:32 | |
*** catintheroof has quit IRC | 21:33 | |
*** imacdonn has joined #openstack-cinder | 21:39 | |
*** MVenesio has quit IRC | 21:46 | |
*** jroll has quit IRC | 21:47 | |
*** jroll has joined #openstack-cinder | 21:47 | |
viks | :) | 21:47 |
*** xyang has quit IRC | 21:48 | |
*** edmondsw has quit IRC | 21:48 | |
*** edmondsw has joined #openstack-cinder | 21:48 | |
*** jroll has quit IRC | 21:49 | |
*** xyang1 has quit IRC | 21:49 | |
*** xyang has joined #openstack-cinder | 21:50 | |
*** salv-orlando has joined #openstack-cinder | 21:50 | |
*** lamt has quit IRC | 21:51 | |
*** edmondsw has quit IRC | 21:53 | |
*** jroll has joined #openstack-cinder | 21:53 | |
*** xyang has quit IRC | 21:57 | |
*** cknight has quit IRC | 21:59 | |
*** jose-phillips has quit IRC | 21:59 | |
*** david-lyle has quit IRC | 22:01 | |
*** chhavi has quit IRC | 22:13 | |
*** jose-phillips has joined #openstack-cinder | 22:14 | |
*** dfisher has quit IRC | 22:16 | |
*** gouthamr has joined #openstack-cinder | 22:21 | |
*** felipemonteiro_ has quit IRC | 22:22 | |
*** sdague has quit IRC | 22:24 | |
*** wolsen has quit IRC | 22:33 | |
*** Julien-zte has joined #openstack-cinder | 22:34 | |
*** Julien-zte has quit IRC | 22:34 | |
*** Julien-zte has joined #openstack-cinder | 22:34 | |
*** wolsen has joined #openstack-cinder | 22:35 | |
*** dustins has quit IRC | 22:37 | |
*** dave-mccowan has joined #openstack-cinder | 22:44 | |
*** Cibo_ has joined #openstack-cinder | 22:45 | |
openstackgerrit | Maciej Kucia proposed openstack/os-brick master: Stop ignoring H405 hacking rule in tox https://review.openstack.org/466553 | 22:46 |
openstackgerrit | Maciej Kucia proposed openstack/os-brick master: Stop ignoring E265 pycodestyle rule in tox https://review.openstack.org/466552 | 22:46 |
openstackgerrit | Maciej Kucia proposed openstack/os-brick master: Removed invalid comments in tox.ini [flake8] https://review.openstack.org/466555 | 22:46 |
openstackgerrit | Maciej Kucia proposed openstack/os-brick master: Stop ignoring H904 hacking rule in tox https://review.openstack.org/466554 | 22:46 |
openstackgerrit | Maciej Kucia proposed openstack/os-brick master: Stop ignoring E123 and E125 pycodestyle rules https://review.openstack.org/466551 | 22:46 |
openstackgerrit | Maciej Kucia proposed openstack/os-brick master: Update hacking version to align with Cinder https://review.openstack.org/466550 | 22:46 |
*** salv-orlando has quit IRC | 22:48 | |
*** edmondsw has joined #openstack-cinder | 22:59 | |
*** viks has quit IRC | 23:00 | |
*** reduxio has joined #openstack-cinder | 23:00 | |
openstackgerrit | Merged openstack/cinder master: SMBFS: avoid unnecessary conversion when uploading volumes https://review.openstack.org/464215 | 23:01 |
*** cristicalin has quit IRC | 23:02 | |
*** edmondsw has quit IRC | 23:04 | |
*** reduxio has quit IRC | 23:06 | |
*** jose-phillips has quit IRC | 23:08 | |
*** jose-phillips has joined #openstack-cinder | 23:09 | |
*** viks has joined #openstack-cinder | 23:10 | |
*** s-shiono has joined #openstack-cinder | 23:22 | |
*** rkrum has joined #openstack-cinder | 23:23 | |
*** jungleboyj has quit IRC | 23:26 | |
*** jungleboyj has joined #openstack-cinder | 23:26 | |
*** mtanino__ has quit IRC | 23:30 | |
*** rkrum has quit IRC | 23:35 | |
*** Julien-zte has quit IRC | 23:49 | |
openstackgerrit | Merged openstack/cinder master: api-ref cleanup: required flag for tenants https://review.openstack.org/406056 | 23:57 |
openstackgerrit | Merged openstack/cinder master: Updated from global requirements https://review.openstack.org/467342 | 23:59 |
*** salv-orlando has joined #openstack-cinder | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!