| *** kaisers has joined #openstack-cinder | 00:04 | |
| *** _cjones_ has quit IRC | 00:05 | |
| *** _cjones_ has joined #openstack-cinder | 00:06 | |
| *** drjones has joined #openstack-cinder | 00:07 | |
| *** yangyapeng has quit IRC | 00:10 | |
| *** _cjones_ has quit IRC | 00:10 | |
| *** kamtamtun has quit IRC | 00:10 | |
| *** alenavolk has quit IRC | 00:11 | |
| *** drjones has quit IRC | 00:11 | |
| *** alenavolk has joined #openstack-cinder | 00:12 | |
| *** cknight has joined #openstack-cinder | 00:14 | |
| *** salv-orl_ has joined #openstack-cinder | 00:17 | |
| *** salv-orlando has quit IRC | 00:20 | |
| *** yuvalb has quit IRC | 00:21 | |
| *** yuvalb has joined #openstack-cinder | 00:30 | |
| *** wanghao has joined #openstack-cinder | 00:31 | |
| *** kaisers has quit IRC | 00:34 | |
| *** zsli has joined #openstack-cinder | 00:58 | |
| *** alenavolk has quit IRC | 01:10 | |
| *** yangyapeng has joined #openstack-cinder | 01:13 | |
| *** kaisers has joined #openstack-cinder | 01:30 | |
| *** sdague has joined #openstack-cinder | 01:42 | |
| *** mriedem has quit IRC | 01:42 | |
| *** cknight has quit IRC | 01:52 | |
| *** dave-mccowan has joined #openstack-cinder | 01:52 | |
| *** huanxuan has joined #openstack-cinder | 01:55 | |
| *** sdague has quit IRC | 02:01 | |
| *** gcb has joined #openstack-cinder | 02:03 | |
| *** _cjones_ has joined #openstack-cinder | 02:09 | |
| *** _cjones_ has quit IRC | 02:14 | |
| *** alenavolk has joined #openstack-cinder | 02:14 | |
| *** alenavolk has quit IRC | 02:19 | |
| *** gouthamr has quit IRC | 02:20 | |
| *** kaisers_ has joined #openstack-cinder | 02:26 | |
| *** kaisers has quit IRC | 02:29 | |
| *** huanxuan has quit IRC | 02:40 | |
| *** kaisers_ has quit IRC | 02:41 | |
| *** sudipto has joined #openstack-cinder | 03:05 | |
| *** sudipto_ has joined #openstack-cinder | 03:05 | |
| *** ircuser-1 has joined #openstack-cinder | 03:09 | |
| *** sudipto has quit IRC | 03:12 | |
| *** sudipto_ has quit IRC | 03:12 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder master: Set access_policy for messaging's dispatcher https://review.openstack.org/413446 | 03:18 |
|---|---|---|
| *** mars has joined #openstack-cinder | 03:26 | |
| *** pgadiya has joined #openstack-cinder | 03:36 | |
| *** kaisers has joined #openstack-cinder | 03:38 | |
| *** mars has quit IRC | 03:39 | |
| *** gcb has quit IRC | 03:39 | |
| *** dhinesh has quit IRC | 03:49 | |
| *** mars has joined #openstack-cinder | 03:52 | |
| *** gcb has joined #openstack-cinder | 03:52 | |
| *** dhinesh has joined #openstack-cinder | 03:53 | |
| *** _cjones_ has joined #openstack-cinder | 03:54 | |
| *** kaisers has quit IRC | 03:57 | |
| *** _cjones_ has quit IRC | 03:58 | |
| *** _cjones_ has joined #openstack-cinder | 03:59 | |
| *** psachin has joined #openstack-cinder | 04:03 | |
| *** zsli has quit IRC | 04:09 | |
| *** lamt has joined #openstack-cinder | 04:18 | |
| *** kaisers has joined #openstack-cinder | 04:20 | |
| *** kaisers has quit IRC | 04:24 | |
| *** mdnadeem has joined #openstack-cinder | 04:29 | |
| *** adisky_ has joined #openstack-cinder | 04:31 | |
| *** alonma has joined #openstack-cinder | 04:31 | |
| *** alonma has quit IRC | 04:36 | |
| *** dave-mccowan has quit IRC | 04:36 | |
| *** pck has quit IRC | 04:40 | |
| *** pck has joined #openstack-cinder | 04:41 | |
| *** zsli has joined #openstack-cinder | 04:41 | |
| *** lamt has quit IRC | 04:45 | |
| *** alonma has joined #openstack-cinder | 04:49 | |
| *** alonma has quit IRC | 04:53 | |
| *** kaisers has joined #openstack-cinder | 04:55 | |
| *** sudipto_ has joined #openstack-cinder | 04:57 | |
| *** sudipto has joined #openstack-cinder | 04:57 | |
| *** Poornima has joined #openstack-cinder | 05:01 | |
| *** zsli has quit IRC | 05:01 | |
| *** drjones has joined #openstack-cinder | 05:02 | |
| *** zsli has joined #openstack-cinder | 05:02 | |
| *** gcb has quit IRC | 05:04 | |
| *** _cjones_ has quit IRC | 05:05 | |
| *** zsli has quit IRC | 05:07 | |
| *** mars has quit IRC | 05:08 | |
| *** rezroo has quit IRC | 05:13 | |
| *** drjones has quit IRC | 05:22 | |
| *** _cjones_ has joined #openstack-cinder | 05:23 | |
| *** kaisers has quit IRC | 05:23 | |
| *** kaisers has joined #openstack-cinder | 05:23 | |
| *** zhangguoqing has joined #openstack-cinder | 05:25 | |
| *** _cjones_ has quit IRC | 05:27 | |
| *** rezroo has joined #openstack-cinder | 05:27 | |
| *** kaisers has quit IRC | 05:28 | |
| *** pdeore has joined #openstack-cinder | 05:30 | |
| *** kamtamtun has joined #openstack-cinder | 05:30 | |
| *** yangyapeng has quit IRC | 05:31 | |
| *** gcb has joined #openstack-cinder | 05:32 | |
| *** mars has joined #openstack-cinder | 05:32 | |
| *** sandanar has joined #openstack-cinder | 05:36 | |
| *** yangyapeng has joined #openstack-cinder | 05:37 | |
| *** cknight has joined #openstack-cinder | 05:37 | |
| *** jiaohaolin1 has quit IRC | 05:39 | |
| *** zhangguoqing has quit IRC | 05:42 | |
| *** zhangguoqing has joined #openstack-cinder | 05:42 | |
| *** wanghao has quit IRC | 05:46 | |
| *** wanghao has joined #openstack-cinder | 05:46 | |
| *** alonmarx has joined #openstack-cinder | 05:53 | |
| *** rezroo has quit IRC | 05:58 | |
| *** Jack_Iv has joined #openstack-cinder | 05:58 | |
| *** isaacb has joined #openstack-cinder | 06:01 | |
| *** PoornimaK has joined #openstack-cinder | 06:10 | |
| *** lamt has joined #openstack-cinder | 06:12 | |
| *** alonmarx has quit IRC | 06:13 | |
| *** cknight has quit IRC | 06:17 | |
| *** salv-orlando has joined #openstack-cinder | 06:17 | |
| *** salv-orlando has quit IRC | 06:20 | |
| *** salv-orlando has joined #openstack-cinder | 06:20 | |
| *** salv-orl_ has quit IRC | 06:21 | |
| *** lamt has quit IRC | 06:21 | |
| *** Administrator_ has joined #openstack-cinder | 06:22 | |
| *** nikeshm has joined #openstack-cinder | 06:23 | |
| *** dhinesh has quit IRC | 06:28 | |
| *** kaisers has joined #openstack-cinder | 06:29 | |
| nikeshm | hi | 06:29 |
| *** mars has quit IRC | 06:31 | |
| *** nkrinner_afk is now known as nkrinner | 06:41 | |
| *** kaisers has quit IRC | 06:49 | |
| nikeshm | still our Kaminario CI is failing due to setuptooks | 06:51 |
| nikeshm | due to setuptools issue | 06:51 |
| nikeshm | this fix did not work https://review.openstack.org/#/c/437741 | 06:52 |
| *** yangyapeng has quit IRC | 06:52 | |
| *** Cibo_ has joined #openstack-cinder | 06:57 | |
| *** yangyapeng has joined #openstack-cinder | 07:03 | |
| *** kaisers has joined #openstack-cinder | 07:05 | |
| *** sp___ has joined #openstack-cinder | 07:06 | |
| *** alonmarx has joined #openstack-cinder | 07:11 | |
| *** wanghao has quit IRC | 07:13 | |
| openstackgerrit | huyang proposed openstack/cinder master: Fix storage pools key of Huawei driver https://review.openstack.org/390132 | 07:13 |
| *** wanghao has joined #openstack-cinder | 07:15 | |
| *** kaisers has quit IRC | 07:19 | |
| *** rcernin has joined #openstack-cinder | 07:20 | |
| *** pcaruana has joined #openstack-cinder | 07:20 | |
| *** nikeshm has quit IRC | 07:21 | |
| *** PoornimaK has quit IRC | 07:21 | |
| *** rcernin has quit IRC | 07:23 | |
| *** rcernin has joined #openstack-cinder | 07:23 | |
| *** wanghao_ has joined #openstack-cinder | 07:25 | |
| *** ChubYann has quit IRC | 07:26 | |
| *** Cibo_ has quit IRC | 07:27 | |
| *** wanghao has quit IRC | 07:28 | |
| openstackgerrit | huyang proposed openstack/cinder master: Huawei driver supports specifying copy speed https://review.openstack.org/368516 | 07:28 |
| *** Julien-zte has joined #openstack-cinder | 07:32 | |
| *** Poornima has quit IRC | 07:33 | |
| *** alonmarx has quit IRC | 07:38 | |
| *** kamtamtun has quit IRC | 07:40 | |
| *** Poornima has joined #openstack-cinder | 07:50 | |
| *** alonma has joined #openstack-cinder | 07:54 | |
| *** mjura has joined #openstack-cinder | 08:10 | |
| *** Guest68960 has quit IRC | 08:14 | |
| *** kaisers has joined #openstack-cinder | 08:16 | |
| *** lkuchlan has joined #openstack-cinder | 08:18 | |
| *** Poornima has quit IRC | 08:19 | |
| *** mgagne has joined #openstack-cinder | 08:20 | |
| *** mgagne is now known as Guest11443 | 08:20 | |
| *** Poornima has joined #openstack-cinder | 08:21 | |
| *** salv-orlando has quit IRC | 08:22 | |
| *** wanghao_ has quit IRC | 08:27 | |
| *** bks has joined #openstack-cinder | 08:29 | |
| *** Sprvn has joined #openstack-cinder | 08:30 | |
| *** e0ne has joined #openstack-cinder | 08:30 | |
| *** akapil has joined #openstack-cinder | 08:31 | |
| *** ccesario has quit IRC | 08:35 | |
| *** lkuchlan has quit IRC | 08:38 | |
| *** wanghao has joined #openstack-cinder | 08:41 | |
| *** mvk has quit IRC | 08:44 | |
| *** ccesario has joined #openstack-cinder | 08:46 | |
| *** kaisers has quit IRC | 08:53 | |
| *** slunkad has joined #openstack-cinder | 08:57 | |
| *** pgadiya has quit IRC | 08:59 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-cinder | 09:00 | |
| *** bks has quit IRC | 09:11 | |
| *** mvk has joined #openstack-cinder | 09:12 | |
| *** pgadiya has joined #openstack-cinder | 09:15 | |
| *** jordanP has joined #openstack-cinder | 09:24 | |
| *** pgadiya has quit IRC | 09:26 | |
| *** isaacb has quit IRC | 09:34 | |
| *** isaacb has joined #openstack-cinder | 09:35 | |
| *** Julien-zte has quit IRC | 09:37 | |
| *** psachin has quit IRC | 09:38 | |
| *** pgadiya has joined #openstack-cinder | 09:38 | |
| *** isaacb has quit IRC | 09:47 | |
| *** wanghao has quit IRC | 09:47 | |
| *** Administrator_ has quit IRC | 09:50 | |
| *** savihou has joined #openstack-cinder | 09:50 | |
| *** Administrator_ has joined #openstack-cinder | 09:50 | |
| *** gcb has quit IRC | 09:55 | |
| *** Poornima has quit IRC | 10:18 | |
| *** yangyapeng has quit IRC | 10:33 | |
| *** Poornima has joined #openstack-cinder | 10:34 | |
| *** strigazi_ is now known as strigazi | 10:35 | |
| *** Poornima has quit IRC | 10:37 | |
| *** Poornima has joined #openstack-cinder | 10:40 | |
| *** aarefiev_afk is now known as aarefiev | 10:47 | |
| *** kaisers has joined #openstack-cinder | 10:48 | |
| *** dalgaaf has quit IRC | 10:49 | |
| *** nikeshm has joined #openstack-cinder | 10:53 | |
| *** dalgaaf has joined #openstack-cinder | 10:54 | |
| *** psachin has joined #openstack-cinder | 10:56 | |
| *** strigazi has quit IRC | 10:56 | |
| *** strigazi has joined #openstack-cinder | 10:56 | |
| nikeshm | hi | 11:06 |
| nikeshm | the patch "https://review.openstack.org/#/c/437880" helps in fixing Kaminario CI failure while installing devstack due to setuptools. | 11:06 |
| *** Cibo_ has joined #openstack-cinder | 11:09 | |
| *** kaisers has quit IRC | 11:14 | |
| *** nicolasbock has joined #openstack-cinder | 11:14 | |
| *** nicolasbock has quit IRC | 11:16 | |
| *** nicolasbock has joined #openstack-cinder | 11:16 | |
| *** pdeore has quit IRC | 11:17 | |
| *** alonma has quit IRC | 11:18 | |
| *** alonma has joined #openstack-cinder | 11:19 | |
| *** alonma has quit IRC | 11:23 | |
| *** sandanar has quit IRC | 11:29 | |
| *** sdague has joined #openstack-cinder | 11:31 | |
| *** akapil has quit IRC | 11:48 | |
| *** alonma has joined #openstack-cinder | 11:50 | |
| *** yangyapeng has joined #openstack-cinder | 11:51 | |
| *** sdague_ has joined #openstack-cinder | 11:53 | |
| *** sdague__ has joined #openstack-cinder | 11:53 | |
| *** gouthamr has joined #openstack-cinder | 11:53 | |
| *** sdague has quit IRC | 11:53 | |
| *** sdague__ has quit IRC | 11:54 | |
| *** sdague_ has quit IRC | 11:54 | |
| nikeshm | in my CI, v1 tests are skipping | 11:55 |
| *** sdague has joined #openstack-cinder | 11:55 | |
| *** sdague_ has joined #openstack-cinder | 11:55 | |
| *** pgadiya has quit IRC | 11:57 | |
| *** wanghao has joined #openstack-cinder | 11:58 | |
| *** sudipto_ has quit IRC | 11:58 | |
| *** sudipto has quit IRC | 11:58 | |
| *** nikeshm has quit IRC | 11:59 | |
| *** kaisers has joined #openstack-cinder | 12:10 | |
| *** wanghao_ has joined #openstack-cinder | 12:10 | |
| *** akapil has joined #openstack-cinder | 12:11 | |
| *** wanghao has quit IRC | 12:11 | |
| *** Poornima has quit IRC | 12:14 | |
| *** nikeshm has joined #openstack-cinder | 12:17 | |
| *** wanghao has joined #openstack-cinder | 12:18 | |
| *** wanghao_ has quit IRC | 12:19 | |
| *** dave-mccowan has joined #openstack-cinder | 12:21 | |
| *** Cibo_ has quit IRC | 12:23 | |
| *** wanghao_ has joined #openstack-cinder | 12:24 | |
| *** wanghao has quit IRC | 12:25 | |
| *** sdague has quit IRC | 12:32 | |
| *** sdague_ is now known as sdague | 12:33 | |
| *** wanghao_ has quit IRC | 12:43 | |
| *** Julien-zte has joined #openstack-cinder | 12:47 | |
| *** gouthamr has quit IRC | 12:49 | |
| *** sudipto has joined #openstack-cinder | 12:50 | |
| *** sudipto_ has joined #openstack-cinder | 12:50 | |
| *** nkrinner has quit IRC | 12:57 | |
| *** nkrinner has joined #openstack-cinder | 13:01 | |
| *** edmondsw has joined #openstack-cinder | 13:08 | |
| *** gouthamr has joined #openstack-cinder | 13:09 | |
| *** zhangguoqing has quit IRC | 13:09 | |
| *** zhangguoqing has joined #openstack-cinder | 13:10 | |
| *** lamt has joined #openstack-cinder | 13:11 | |
| *** lamt has quit IRC | 13:12 | |
| *** e0ne has quit IRC | 13:15 | |
| *** liverpooler has joined #openstack-cinder | 13:18 | |
| *** Jack_Iv has quit IRC | 13:25 | |
| *** Jack_Iv has joined #openstack-cinder | 13:26 | |
| *** danpawlik has joined #openstack-cinder | 13:28 | |
| *** ericyoung has joined #openstack-cinder | 13:29 | |
| danpawlik | Hello. Could someone tell me, why os-brick library is using hardcoded path for ceph keyring instead of using variables set in cinder config file ? | 13:30 |
| *** Jack_Iv has quit IRC | 13:30 | |
| smcginnis | danpawlik: Oversight maybe. | 13:32 |
| *** kaisers has quit IRC | 13:33 | |
| *** dustins has joined #openstack-cinder | 13:35 | |
| danpawlik | smcginnis: I guess yes... This hardcoded code is problematic :D https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connectors/rbd.py#L72 | 13:37 |
| *** chhavi has joined #openstack-cinder | 13:37 | |
| dulek | danpawlik: Wow, that's clearly a bug. | 13:38 |
| smcginnis | danpawlik: Maybe jbernard knows why that was done, or can fix it. | 13:38 |
| *** bkopilov has joined #openstack-cinder | 13:39 | |
| danpawlik | jbernard: Are you here? | 13:40 |
| *** jwcroppe has quit IRC | 13:44 | |
| *** tongli has joined #openstack-cinder | 13:53 | |
| *** Poornima has joined #openstack-cinder | 13:56 | |
| *** Poornima has quit IRC | 13:57 | |
| *** chlong has joined #openstack-cinder | 13:59 | |
| *** chlong has quit IRC | 14:04 | |
| *** erhudy has joined #openstack-cinder | 14:05 | |
| *** cdelatte has joined #openstack-cinder | 14:08 | |
| *** nkrinner has quit IRC | 14:09 | |
| *** porrua has joined #openstack-cinder | 14:10 | |
| smcginnis | ildikov: check_attach change is through, time for some champagne. ;) | 14:11 |
| *** gman-tx has joined #openstack-cinder | 14:12 | |
| *** mriedem has joined #openstack-cinder | 14:12 | |
| *** crushil has joined #openstack-cinder | 14:13 | |
| *** nkrinner has joined #openstack-cinder | 14:14 | |
| *** xyang1 has joined #openstack-cinder | 14:15 | |
| *** salv-orlando has joined #openstack-cinder | 14:15 | |
| *** nikeshm has quit IRC | 14:20 | |
| *** kaisers has joined #openstack-cinder | 14:29 | |
| *** tlian has joined #openstack-cinder | 14:32 | |
| *** kaisers has quit IRC | 14:33 | |
| *** lamt has joined #openstack-cinder | 14:39 | |
| *** kaisers has joined #openstack-cinder | 14:40 | |
| *** gman-tx has quit IRC | 14:50 | |
| jbernard | smcginnis: from pm, i wrote it to follow the expected naming convention for conf and keyring paths | 14:55 |
| jbernard | so that the provided cluster name could be used to derive those paths and addition arguments would not need to be passed | 14:55 |
| smcginnis | danpawlik: ^^ | 14:55 |
| jbernard | howerver, if the conf and keyring paths don't follow the convention - then RBDClient will fail to find the nessesary files | 14:56 |
| jbernard | and will need additional arguments passed in, so that was can fall back gracefuly | 14:56 |
| *** Cibo_ has joined #openstack-cinder | 14:56 | |
| jbernard | admins are allowed to do this, so a bug and patch are probably in order here | 14:57 |
| openstackgerrit | Yogesh proposed openstack/cinder master: Fix internal tempest tests https://review.openstack.org/437604 | 14:57 |
| jbernard | danpawlik: so a bug for this is appropriate | 14:59 |
| jbernard | danpawlik: and if you don't want to write the patch you can assign it to me | 14:59 |
| *** cknight has joined #openstack-cinder | 14:59 | |
| *** lennyb_ is now known as lennyb | 15:01 | |
| *** Poornima has joined #openstack-cinder | 15:01 | |
| danpawlik | jbernard: ok | 15:02 |
| danpawlik | jbernard: should I create a bug on launchpad | 15:02 |
| *** akapil has quit IRC | 15:02 | |
| jbernard | danpawlik: yes, that will be need so we can reference it in the patch | 15:02 |
| *** Sprvn has quit IRC | 15:02 | |
| *** lamt has quit IRC | 15:03 | |
| *** chhavi has quit IRC | 15:03 | |
| danpawlik | proper answer before finish my sentence :D | 15:03 |
| danpawlik | jbernard: I will be much faster when you will create a patch, but if you have too much work to do, I can do it | 15:04 |
| *** kaisers has quit IRC | 15:07 | |
| jbernard | danpawlik: no problem i can do it, that one is not too bad | 15:07 |
| *** Cibo_ has quit IRC | 15:08 | |
| *** belmoreira has joined #openstack-cinder | 15:10 | |
| *** gcb has joined #openstack-cinder | 15:10 | |
| *** lucasxu has joined #openstack-cinder | 15:10 | |
| *** akapil has joined #openstack-cinder | 15:13 | |
| *** diablo_rojo has joined #openstack-cinder | 15:14 | |
| *** diablo_rojo has quit IRC | 15:15 | |
| *** diablo_rojo has joined #openstack-cinder | 15:15 | |
| *** lamt has joined #openstack-cinder | 15:16 | |
| *** kong has quit IRC | 15:19 | |
| *** _ducttape_ has joined #openstack-cinder | 15:21 | |
| *** isaacb has joined #openstack-cinder | 15:21 | |
| *** mriedem1 has joined #openstack-cinder | 15:22 | |
| *** mriedem has quit IRC | 15:23 | |
| *** nkrinner is now known as nkrinner_afk | 15:25 | |
| *** chlong has joined #openstack-cinder | 15:25 | |
| *** chlong_ has joined #openstack-cinder | 15:27 | |
| *** chlong_ has quit IRC | 15:28 | |
| *** Cibo_ has joined #openstack-cinder | 15:31 | |
| *** zhangguoqing has quit IRC | 15:34 | |
| mriedem1 | jungleboyj: easy wins for your new people https://bugs.launchpad.net/cinder/+bug/1668305 https://bugs.launchpad.net/cinder/+bug/1668301 | 15:37 |
| openstack | Launchpad bug 1668305 in Cinder "api-ref: v2 API does not mention DELETE for volume metadata" [Undecided,New] | 15:37 |
| openstack | Launchpad bug 1668301 in Cinder "api-ref: v2 update volume metadata description is wrong" [Undecided,New] | 15:37 |
| *** mriedem1 is now known as mriedem | 15:37 | |
| jungleboyj | mriedem: Nice. Thank you! | 15:38 |
| *** liverpooler has quit IRC | 15:43 | |
| *** crose has joined #openstack-cinder | 15:44 | |
| *** mdnadeem has quit IRC | 15:46 | |
| *** Julien-zte has quit IRC | 15:47 | |
| *** liverpooler has joined #openstack-cinder | 15:48 | |
| *** Julien-zte has joined #openstack-cinder | 15:49 | |
| *** lamt has quit IRC | 15:49 | |
| *** liverpooler has quit IRC | 15:49 | |
| *** Julien-zte has quit IRC | 15:49 | |
| *** liverpooler has joined #openstack-cinder | 15:49 | |
| *** lennyb is now known as lennyb_ | 15:51 | |
| *** lennyb_ is now known as lennyb | 15:51 | |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: Fix iSCSI multipath rescan https://review.openstack.org/433104 | 15:56 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: Fix multipath flush when using friendly names https://review.openstack.org/433102 | 15:57 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: Retry multipath flush when map is in use https://review.openstack.org/433103 | 15:57 |
| *** lucasxu has quit IRC | 15:58 | |
| mdovgal | geguileo, hi. Are you here? | 16:00 |
| *** Cibo has joined #openstack-cinder | 16:02 | |
| *** kaisers has joined #openstack-cinder | 16:04 | |
| *** Cibo_ has quit IRC | 16:05 | |
| *** Cibo has quit IRC | 16:07 | |
| *** Cibo_ has joined #openstack-cinder | 16:07 | |
| *** lamt has joined #openstack-cinder | 16:14 | |
| *** jwcroppe has joined #openstack-cinder | 16:14 | |
| *** rcernin has quit IRC | 16:15 | |
| *** alonma has quit IRC | 16:16 | |
| *** lucasxu has joined #openstack-cinder | 16:16 | |
| *** alonma has joined #openstack-cinder | 16:16 | |
| openstackgerrit | Michael Dovgal proposed openstack/cinder master: Fix size update error for snapshot-manage operation https://review.openstack.org/426162 | 16:17 |
| *** belmoreira has quit IRC | 16:17 | |
| *** alonma has quit IRC | 16:21 | |
| *** psachin_ has joined #openstack-cinder | 16:21 | |
| *** Poornima has quit IRC | 16:22 | |
| *** crose has quit IRC | 16:24 | |
| *** dhajare_ has joined #openstack-cinder | 16:25 | |
| *** dhajare_ has left #openstack-cinder | 16:25 | |
| *** dhajare has joined #openstack-cinder | 16:27 | |
| *** mdovgal has left #openstack-cinder | 16:29 | |
| *** dhajare has quit IRC | 16:29 | |
| *** alonmarx has joined #openstack-cinder | 16:31 | |
| ildikov | smcginnis: \o/ :) | 16:31 |
| ildikov | smcginnis: special thanks goes to mriedem for it! :) | 16:32 |
| *** psachin has quit IRC | 16:36 | |
| *** psachin_ has quit IRC | 16:37 | |
| *** akapil has quit IRC | 16:40 | |
| *** gatuus has joined #openstack-cinder | 16:41 | |
| *** mriedem has quit IRC | 16:44 | |
| *** mriedem has joined #openstack-cinder | 16:46 | |
| *** savihou has quit IRC | 16:50 | |
| *** fmccrthy has quit IRC | 16:53 | |
| *** Cibo_ has quit IRC | 16:54 | |
| *** fmccrthy has joined #openstack-cinder | 16:55 | |
| *** takedakn has joined #openstack-cinder | 16:56 | |
| *** takedakn has quit IRC | 16:58 | |
| *** rajinir_ has joined #openstack-cinder | 17:06 | |
| *** mjura has quit IRC | 17:07 | |
| *** lamt has quit IRC | 17:07 | |
| *** mvk has quit IRC | 17:12 | |
| *** Jack_Iv has joined #openstack-cinder | 17:13 | |
| *** Jack_Iv has quit IRC | 17:13 | |
| *** Jack_Iv has joined #openstack-cinder | 17:14 | |
| *** sudipto_ has quit IRC | 17:15 | |
| *** sudipto has quit IRC | 17:15 | |
| *** david-lyle has joined #openstack-cinder | 17:17 | |
| *** Apoorva has joined #openstack-cinder | 17:20 | |
| *** Apoorva has quit IRC | 17:20 | |
| *** david-lyle has quit IRC | 17:21 | |
| *** lucasxu has quit IRC | 17:22 | |
| *** Apoorva has joined #openstack-cinder | 17:23 | |
| *** _cjones_ has joined #openstack-cinder | 17:26 | |
| xinli | stakeda: I tried several times in NEC Cinder CI, it all failed since last Thursday, 02/23/16. Could you please help to take a look? | 17:30 |
| *** kaisers has quit IRC | 17:36 | |
| xinli | nikeshm: one question regarding Vedams CI | 17:44 |
| *** david-lyle has joined #openstack-cinder | 17:44 | |
| *** david-lyle has quit IRC | 17:45 | |
| *** david-lyle has joined #openstack-cinder | 17:46 | |
| *** e0ne has joined #openstack-cinder | 17:48 | |
| *** earlephilhower has joined #openstack-cinder | 17:49 | |
| *** adisky_ has quit IRC | 17:49 | |
| earlephilhower | Hi folks, been a while. I recently changed my CI's logserver to a new host and need to update the IP that reviews.openstack.org links to in completion. | 17:51 |
| earlephilhower | Except I can't find where that's set. :( My Jenkins job is SFTP'ing to the new site, and the log header has the right url, but I'm still going to the old host @ reviews. | 17:52 |
| earlephilhower | Can someone point me to the right spot to check? grep-r on /etc/jenkins not showing anything and I'm kinda stumped... | 17:52 |
| asselin | earlephilhower, if you're using the common ci solution: http://git.openstack.org/cgit/openstack-infra/puppet-openstackci/tree/contrib/single_node_ci_data.yaml#n32 | 17:52 |
| asselin | you might need to restart zuul | 17:52 |
| *** jordanP has quit IRC | 17:53 | |
| *** nikeshm has joined #openstack-cinder | 17:53 | |
| nikeshm | smcginnis: hi | 17:53 |
| earlephilhower | asselin, thanks. I think I'm somewhat behind the times in terms of infra. "It works, don't touch it" is my mantra.. | 17:53 |
| earlephilhower | Will check | 17:53 |
| *** chlong has quit IRC | 17:55 | |
| *** sandanar has joined #openstack-cinder | 17:56 | |
| gouthamr | earlephilhower: in zuul 2.5, it is "url_pattern" under [zuul] in zuul.conf | 17:56 |
| earlephilhower | gouthamr, thanks! That was it. D'oh. | 17:58 |
| *** david-lyle has quit IRC | 17:58 | |
| *** Jack_Iv has quit IRC | 17:59 | |
| *** crushil has quit IRC | 17:59 | |
| *** aarefiev is now known as aarefiev_afk | 18:00 | |
| *** nikeshm has quit IRC | 18:00 | |
| *** david-lyle has joined #openstack-cinder | 18:01 | |
| *** nikeshm has joined #openstack-cinder | 18:05 | |
| nikeshm | smcginnis: Following links helped us to resolve Kaminario CI failures: | 18:06 |
| nikeshm | http://lists.openstack.org/pipermail/openstack-dev/2017-February/112822.html https://review.openstack.org/#/c/326585/21/devstack-vm-gate.sh | 18:06 |
| nikeshm | https://review.openstack.org/#/c/437880/ | 18:07 |
| nikeshm | patch 437880 is under review and cherry picking this patch is resolving setuptools issue | 18:08 |
| *** kamtamtun has joined #openstack-cinder | 18:09 | |
| *** chlong has joined #openstack-cinder | 18:11 | |
| earlephilhower | nikeshm: Were the Kaminario failures like "2017-02-27 00:58:08.044 | localhost | FAILED | network_sanity_check : Perform ping check | rc=2 >> ping: unknown host mirror.regionone..openstack.org to retry, use: --limit @/home/jenkins/workspace/TegileFCDriver-dsvm-tempest/devstack-gate/playbooks/setup_host.retry" | 18:12 |
| earlephilhower | Or were they different. I'm trying to help clear a patch 336092 and seeing those on my own CI and a bunch of other failing CIs. | 18:13 |
| *** isaacb has quit IRC | 18:14 | |
| *** crushil has joined #openstack-cinder | 18:14 | |
| nikeshm | earlephilhower: [ERROR] /opt/stack/new/devstack/stackrc:759 Could not determine host ip address. See local.conf for suggestions on setting HOST_IP. | 18:16 |
| nikeshm | this was due to use of both local.conf and localrc | 18:16 |
| nikeshm | files | 18:16 |
| nikeshm | then we used only local.conf according to http://lists.openstack.org/pipermail/openstack-dev/2017-February/112822.html https://review.openstack.org/#/c/326585/21/devstack-vm-gate.sh | 18:17 |
| nikeshm | so that issue solved | 18:17 |
| *** ChubYann has joined #openstack-cinder | 18:17 | |
| *** alonma has joined #openstack-cinder | 18:17 | |
| earlephilhower | nikeshm, got it. I think this is something different now. Just happened recently, I've got to backtrack and see when change happened. Thx! | 18:17 |
| *** salv-orl_ has joined #openstack-cinder | 18:18 | |
| nikeshm | then we started getting setuptools issue | 18:18 |
| *** salv-orlando has quit IRC | 18:20 | |
| *** alonma has quit IRC | 18:23 | |
| *** lamt has joined #openstack-cinder | 18:24 | |
| *** sandanar has quit IRC | 18:26 | |
| *** sandanar has joined #openstack-cinder | 18:26 | |
| *** nikeshm has quit IRC | 18:27 | |
| *** sudipto_ has joined #openstack-cinder | 18:40 | |
| *** sudipto has joined #openstack-cinder | 18:40 | |
| *** lamt has quit IRC | 18:41 | |
| openstackgerrit | Elad Zucker proposed openstack/cinder master: Delete consistency group fails on xtremio driver https://review.openstack.org/438109 | 18:42 |
| *** _ducttape_ has quit IRC | 18:43 | |
| *** ducttape_ has joined #openstack-cinder | 18:44 | |
| mtreinish | jgriffith: http://blog.kortar.org/wp-content/uploads/2017/02/c-vol.svg I generated one of those graphs for cinder volume from my test patch: http://logs.openstack.org/44/429844/9/check/gate-tempest-dsvm-neutron-src-oslo.service-ubuntu-xenial/0a731f0/logs/ | 18:45 |
| *** ducttape_ has quit IRC | 18:45 | |
| *** ducttape_ has joined #openstack-cinder | 18:45 | |
| earlephilhower | So for posterity the CI failures on HGST, Lenovo, Tegile, and others in network_sanity_check seem to be related to nodepool_mirror_host (and nodepool_*) being undefined, causing an invalid URL to be generated for a ping test. | 18:46 |
| earlephilhower | https://github.com/openstack-infra/devstack-gate/commit/4d7b92ca16c5ac04af71512d39230004abf3b4db | 18:46 |
| *** mtanino has joined #openstack-cinder | 18:46 | |
| *** dhinesh has joined #openstack-cinder | 18:46 | |
| earlephilhower | How it passed for the prior months, I just don't know. But if NODEPOOL_* isn't defined in /etc/nodepool/provider it makes a hostname with extra periods like "mirror...openstack.org" | 18:47 |
| *** dustins has quit IRC | 18:47 | |
| *** lamt has joined #openstack-cinder | 18:48 | |
| *** gatuus has quit IRC | 18:50 | |
| *** esker has joined #openstack-cinder | 18:54 | |
| *** Cibo_ has joined #openstack-cinder | 18:54 | |
| *** pots has joined #openstack-cinder | 18:59 | |
| *** xyang1 has quit IRC | 19:04 | |
| *** xyang1 has joined #openstack-cinder | 19:04 | |
| *** Jack_Iv has joined #openstack-cinder | 19:08 | |
| *** e0ne has quit IRC | 19:12 | |
| *** crushil has quit IRC | 19:13 | |
| *** kfarr has joined #openstack-cinder | 19:14 | |
| *** ntpttr_ has joined #openstack-cinder | 19:14 | |
| *** sudipto_ has quit IRC | 19:17 | |
| *** sudipto has quit IRC | 19:17 | |
| *** gatuus has joined #openstack-cinder | 19:19 | |
| *** alkhodos has joined #openstack-cinder | 19:20 | |
| *** crushil has joined #openstack-cinder | 19:23 | |
| *** sudipto_ has joined #openstack-cinder | 19:27 | |
| *** sudipto has joined #openstack-cinder | 19:27 | |
| *** kaisers has joined #openstack-cinder | 19:27 | |
| *** e0ne has joined #openstack-cinder | 19:27 | |
| *** pcaruana has quit IRC | 19:29 | |
| *** sudipto_ has quit IRC | 19:32 | |
| *** sudipto has quit IRC | 19:32 | |
| *** sandanar has quit IRC | 19:33 | |
| *** Gorian- is now known as Gorian | 19:34 | |
| *** e0ne has quit IRC | 19:34 | |
| *** earlephilhower has quit IRC | 19:36 | |
| *** mjura has joined #openstack-cinder | 19:37 | |
| *** mjura has quit IRC | 19:37 | |
| *** nikeshm has joined #openstack-cinder | 19:37 | |
| nikeshm | hi | 19:38 |
| nikeshm | how to enable api_v1 in CI | 19:38 |
| nikeshm | now i am using only local.conf | 19:38 |
| *** mjura has joined #openstack-cinder | 19:39 | |
| *** sudipto has joined #openstack-cinder | 19:39 | |
| *** sudipto_ has joined #openstack-cinder | 19:39 | |
| nikeshm | xinli: hi | 19:40 |
| xinli | nickeshm: hi | 19:41 |
| *** mjura has quit IRC | 19:41 | |
| patrickeast | nikeshm: you should just need to set "enable_v1_api = True" in the cinder post-config part of your local.conf | 19:42 |
| xinli | patrickeast: are you talking about the pure cinder CI broken isse? | 19:43 |
| patrickeast | xinli: no, well, not directly | 19:43 |
| nikeshm | xinli: we are not maintaining HPMSA CI | 19:44 |
| xinli | patrickeack: IC | 19:45 |
| *** sudipto has quit IRC | 19:46 | |
| *** sudipto_ has quit IRC | 19:46 | |
| patrickeast | xinli: looks like currently its blocked because of trusty being dropped by devstack... https://github.com/openstack-dev/devstack/commit/ff10ac318ce4be49d8bceafb7fea92ae1655b497 | 19:46 |
| smcginnis | patrickeast: Just discovered that as well. | 19:47 |
| patrickeast | guess its time to figure out whats wrong with iscsi multipathing on xenial | 19:47 |
| patrickeast | smcginnis: good thing customers aren't using trusty anymore ;) | 19:47 |
| nikeshm | for me, multipathing is working on xenial | 19:47 |
| smcginnis | patrickeast: Heck, I've still got a 12.x version running somehwere. | 19:48 |
| nikeshm | now, our CI is working with xenial | 19:48 |
| patrickeast | yea, there was a patch to fix it upstream, it might have merged in | 19:49 |
| patrickeast | iirc they just needed to patch multipath-tools | 19:49 |
| *** lkuchlan has joined #openstack-cinder | 19:51 | |
| nikeshm | i hope i was not only the victim of setuptools issue which is temporary fixed by cherry pick https://review.openstack.org/#/c/437880 | 19:52 |
| nikeshm | patrickeast: thanks | 19:52 |
| patrickeast | nikeshm: iirc that was breaking mine too, before this new one | 19:53 |
| *** thumpba has joined #openstack-cinder | 19:54 | |
| xinli | alkhodos: could you please help to take a look regarding nexenta CI broken issue? http://140.174.232.106/refs-changes-92-336092-28/ns_iscsi/ | 19:56 |
| nikeshm | xinli: may be you can ping HP guys here about HPMSA CI | 19:57 |
| xinli | nickeshm: do you have any irc? | 19:57 |
| alkhodos | xinli: hi, looks like there are some issues with devstack. I will look into this today | 19:58 |
| xinli | alkhodos: thx | 19:59 |
| nikeshm | xinli: i am facing some network issues so not able to login | 19:59 |
| nikeshm | continuosly | 19:59 |
| *** kfarr has quit IRC | 20:00 | |
| nikeshm | continuously | 20:00 |
| *** kfarr has joined #openstack-cinder | 20:00 | |
| xinli | nikeshm: Chris M <cinder@tristero.net> is helping to fix the HPMAS CI issue now, thx | 20:00 |
| xinli | patrickeast: Could you pleas ehelp me to understand why suddenly all the CI broekn due to devStack issue? | 20:01 |
| nikeshm | xinli: Chris M, great | 20:01 |
| *** dustins has joined #openstack-cinder | 20:02 | |
| nikeshm | good n8 guys, bye | 20:06 |
| *** nikeshm has left #openstack-cinder | 20:06 | |
| *** salv-orl_ has quit IRC | 20:12 | |
| *** Jack_Iv has quit IRC | 20:12 | |
| *** Jack_Iv has joined #openstack-cinder | 20:13 | |
| *** akapil has joined #openstack-cinder | 20:14 | |
| *** kaisers has quit IRC | 20:15 | |
| patrickeast | xinli: there were a handful of commits merged around the PTG timeframe that broke a few different things for 3rd party ci's, just bad timing | 20:15 |
| patrickeast | xinli: most aren't that bad, but require some intervention by the ci maintainers to fix, and some need upstream changes like that devstack issue nikeshm linked to | 20:16 |
| xinli | patrickeast: got it, the bad thing is that before the issue occured, all the CIs for the 336092 either sucess or passing (the two key tempest test scenarios: test_volumes_clone.* and test-volume_from_snapshot.*), new I have to wait all the Third party CIs get fixed. thank you for yhr info. | 20:18 |
| patrickeast | xinli: yea, the timing makes it very hard to merge that patch | 20:19 |
| *** akapil has quit IRC | 20:19 | |
| *** alonma has joined #openstack-cinder | 20:19 | |
| *** ducttape_ has quit IRC | 20:22 | |
| *** ccesario has quit IRC | 20:22 | |
| xinli | erlon: http://paste.openstack.org/show/600667/ | 20:22 |
| *** Jack_Iv_ has joined #openstack-cinder | 20:23 | |
| *** alonma has quit IRC | 20:24 | |
| *** _ducttape_ has joined #openstack-cinder | 20:25 | |
| *** Jack_Iv has quit IRC | 20:26 | |
| *** salv-orlando has joined #openstack-cinder | 20:39 | |
| *** crushil has quit IRC | 20:45 | |
| *** Jack_Iv_ has quit IRC | 20:46 | |
| *** dave-mccowan has quit IRC | 20:47 | |
| *** liverpooler has quit IRC | 20:50 | |
| *** Jack_Iv has joined #openstack-cinder | 20:51 | |
| *** rcernin has joined #openstack-cinder | 20:52 | |
| *** porrua has quit IRC | 20:53 | |
| *** mwhahaha has left #openstack-cinder | 20:56 | |
| *** Guest11443 is now known as mgagne | 20:57 | |
| *** mgagne has quit IRC | 20:57 | |
| *** mgagne has joined #openstack-cinder | 20:57 | |
| *** Jack_Iv has quit IRC | 20:57 | |
| *** crushil has joined #openstack-cinder | 20:58 | |
| *** tongli has quit IRC | 21:02 | |
| *** e0ne has joined #openstack-cinder | 21:19 | |
| *** nicolasbock has quit IRC | 21:22 | |
| *** smcginnis has quit IRC | 21:26 | |
| *** smcginnis has joined #openstack-cinder | 21:32 | |
| xinli | rick: Could you please help to check the ProphetStor CI status? it seems that CI not working. | 21:32 |
| *** gouthamr has quit IRC | 21:33 | |
| *** kfarr has quit IRC | 21:35 | |
| *** ChanServ sets mode: +o smcginnis | 21:36 | |
| *** jonesn has joined #openstack-cinder | 21:39 | |
| *** lpetrut has joined #openstack-cinder | 21:40 | |
| *** gouthamr has joined #openstack-cinder | 21:42 | |
| *** lpetrut has quit IRC | 21:42 | |
| gouthamr | hey cinder cores: in-tree tempest tests are currently broken and need this minor fix: https://review.openstack.org/#/c/437604/ | 21:45 |
| *** chlong has quit IRC | 21:50 | |
| *** dave-mccowan has joined #openstack-cinder | 21:52 | |
| *** shaner has quit IRC | 21:52 | |
| *** shaner has joined #openstack-cinder | 21:55 | |
| *** crushil has quit IRC | 21:58 | |
| *** esker has quit IRC | 21:59 | |
| *** mvk has joined #openstack-cinder | 21:59 | |
| *** edmondsw has quit IRC | 22:00 | |
| *** alonmarx has quit IRC | 22:06 | |
| *** chris_morrell has joined #openstack-cinder | 22:06 | |
| openstackgerrit | Raunak Kumar proposed openstack/cinder master: Fix Iscsi login for multi array pools for Nimble https://review.openstack.org/438132 | 22:07 |
| *** _ducttape_ has quit IRC | 22:07 | |
| *** gouthamr has quit IRC | 22:08 | |
| *** kaisers has joined #openstack-cinder | 22:10 | |
| *** kamtamtun has quit IRC | 22:10 | |
| *** jonesn has quit IRC | 22:14 | |
| *** e0ne has quit IRC | 22:14 | |
| *** catintheroof has joined #openstack-cinder | 22:15 | |
| *** lkuchlan has quit IRC | 22:24 | |
| xinli | patrickeast: does this patch fix the upstream issues which caused the CI broken? https://review.openstack.org/#/c/437604/ | 22:28 |
| *** asselin_ has joined #openstack-cinder | 22:28 | |
| openstackgerrit | Patrick East proposed openstack/cinder master: Add support for shared "backend_defaults" config https://review.openstack.org/335299 | 22:29 |
| *** asselin has quit IRC | 22:31 | |
| patrickeast | xinli: one of the issues, not all of them... | 22:36 |
| *** _ducttape_ has joined #openstack-cinder | 22:36 | |
| xinli | patrickeast: thx | 22:36 |
| *** kaisers has quit IRC | 22:39 | |
| *** raunak has joined #openstack-cinder | 22:42 | |
| *** raunak_ has joined #openstack-cinder | 22:43 | |
| *** mriedem has quit IRC | 22:44 | |
| *** raunak_ has quit IRC | 22:47 | |
| *** dave-mccowan has quit IRC | 22:48 | |
| *** _ducttape_ has quit IRC | 22:50 | |
| *** _ducttape_ has joined #openstack-cinder | 22:52 | |
| *** dustins has quit IRC | 23:01 | |
| *** Apoorva has quit IRC | 23:03 | |
| *** Apoorva has joined #openstack-cinder | 23:04 | |
| patrickeast | jgriffith: ping, has question about connect api stuff | 23:13 |
| jgriffith | patrickeast hey, sure, what's up? | 23:13 |
| patrickeast | jgriffith: i was looking at the updates on the nova spec, and fiddling around with how it might work to return back that shared flag thing | 23:13 |
| patrickeast | jgriffith: one thing i saw was that we don't seem to actually store the connection info anywhere, is that correct? or am i missing something? | 23:14 |
| jgriffith | patrickeast on the cinder side? | 23:14 |
| patrickeast | yea | 23:14 |
| * patrickeast digs up a github link | 23:14 | |
| jgriffith | patrickeast we should be dumping it into the attachment_specs table | 23:14 |
| smcginnis | We were at least on one of the earlier versions I tested. | 23:15 |
| jgriffith | smcginnis :) | 23:15 |
| smcginnis | Hey, if you want to get rid of 2000 lines of useless code: https://review.openstack.org/#/c/438230/ | 23:16 |
| jgriffith | I'm not crazy, you're the one that's crazy | 23:16 |
| jgriffith | All I wanted was a Pepsi | 23:16 |
| patrickeast | https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1953 | 23:16 |
| jgriffith | patrickeast https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L4537 | 23:16 |
| smcginnis | You're going to an institution. | 23:17 |
| patrickeast | jgriffith: smcginnis we set it on the object... but the object doesn't have the field? | 23:17 |
| jgriffith | @smcginnis :) | 23:17 |
| smcginnis | jgriffith: I get your movie AND music references. ;) | 23:17 |
| jgriffith | patrickeast it's an FK in the object | 23:17 |
| patrickeast | jgriffith: ah yea, that looks like its the connector object nova passes us, not the connection_info back from the driver? | 23:17 |
| jgriffith | patrickeast ahh, yeah; didn't know we needed that :) | 23:18 |
| jgriffith | other than what's there already | 23:18 |
| patrickeast | jgriffith: ah yea, i guess i wasn't sure if we intended to or not | 23:18 |
| * jgriffith isn't trying to solve "old" problems, just new ones :) | 23:18 | |
| patrickeast | jgriffith: that line i linked to makes it look like we try to save it | 23:18 |
| patrickeast | jgriffith: but it doesn't do anything (if i understand how the objects work... which is unlikely) | 23:18 |
| jgriffith | patrickeast well, yeah; only because if you looked at the history of those I got dinged for not returning an object | 23:19 |
| jgriffith | so I stuffed that in as an objecty kinda thing; but not necessarily a db thing | 23:19 |
| jgriffith | don't know if that even made sense or not | 23:19 |
| *** Julien-zte has joined #openstack-cinder | 23:19 | |
| patrickeast | dunno | 23:20 |
| patrickeast | lol | 23:20 |
| jgriffith | patrickeast fair enough :) | 23:20 |
| patrickeast | jgriffith: well, anyway, ORM weirdness aside | 23:20 |
| jgriffith | patrickeast so the answer to your question is "no" I'm not storing the drivers connection_info | 23:20 |
| jgriffith | patrickeast I had assumed we had everything we needed in the volume provider info | 23:20 |
| patrickeast | jgriffith: looking at how we would return that shared flag thing on create, it seems like an easy change... i was playing around with it here https://github.com/patrick-east/cinder/commit/be77a1a21e5ee89d8e4e230d62df7ac0b41521ef | 23:21 |
| jgriffith | patrickeast I was just stuffing that response info into the return to the caller to clean that up a bit | 23:21 |
| *** sp___ has quit IRC | 23:21 | |
| jgriffith | patrickeast :) | 23:21 |
| patrickeast | jgriffith: but if we don't save that info, it means nova needs to hold on to that flag, which doesn't line up with the nova spec right now | 23:21 |
| jgriffith | patrickeast so what you have is almost perfect IMHO | 23:21 |
| patrickeast | jgriffith: dunno which side of things is easier to shift to make them meet in the middle | 23:22 |
| jgriffith | patrickeast I would just want to add a column in the attachment table for it | 23:22 |
| *** gouthamr has joined #openstack-cinder | 23:22 | |
| jgriffith | I even think I have a TODO or something about that somewhere | 23:22 |
| patrickeast | jgriffith: gotcha, so not as an attachment spec | 23:22 |
| *** stakeda has quit IRC | 23:22 | |
| jgriffith | patrickeast I hadn't planned on that no, but I'm open to ideas if someobody (I'm looking at you) has a better way of doing it | 23:22 |
| *** lamt has quit IRC | 23:23 | |
| patrickeast | jgriffith: one thing that might be useful is stashing the whole conn_info thing, like we do with the connector from the other side | 23:23 |
| patrickeast | jgriffith: so that nova or whoever can just grab that info again later without calling create again | 23:23 |
| jgriffith | patrickeast but we do that anyway on restarts when we do verify no? | 23:24 |
| jgriffith | patrickeast I mean, we can dump it but I think that needs to be "another" table linked to the attachment | 23:24 |
| patrickeast | jgriffith: we do? | 23:24 |
| patrickeast | jgriffith: i thought we just updated the exports or whatever | 23:24 |
| jgriffith | patrickeast yea, that's what I mean | 23:25 |
| patrickeast | jgriffith: those aren't the same, are they? >.< | 23:25 |
| jgriffith | patrickeast no, it's provider info, but that's what that info is anyway I tihnk | 23:26 |
| jgriffith | but I'm not sure we're on the same page still :) | 23:26 |
| patrickeast | haha yea i dunno | 23:26 |
| jgriffith | patrickeast I can look at that next though | 23:26 |
| patrickeast | so like, here is what i'm thinking | 23:26 |
| jgriffith | patrickeast so I'm getting ready to push the new bdm column for Nova and the detach pieces, then will be sorting through attach; can revisit it then too | 23:26 |
| patrickeast | the stuff you jam into os-brick to detach/attach whatever | 23:27 |
| jgriffith | oh, you mean the Nova stuff? | 23:27 |
| patrickeast | whichever thingy that is (for our drivers its only returned from intialize_attachment, create_export is a noop) | 23:27 |
| patrickeast | nah brick needs the combination of nova initiator stuff and the connection info cinder gives back | 23:27 |
| jgriffith | you mean initialize_connection :) | 23:27 |
| patrickeast | iqn, lun, portal, etc for iscsi | 23:27 |
| jgriffith | right | 23:28 |
| *** lamt has joined #openstack-cinder | 23:28 | |
| jgriffith | patrickeast that should all be in provider info of the volume no? | 23:28 |
| patrickeast | so like, later on if they need to re-attach or detach they just do a get attachment, and have it | 23:28 |
| patrickeast | without calling initialize_connection on the drivers again | 23:28 |
| patrickeast | jgriffith: i don't think it is, unless the manager is stashing it away in there somewhere | 23:28 |
| jgriffith | patrickeast right, but we do that already (or at least I do) | 23:28 |
| jgriffith | I'll double check | 23:28 |
| patrickeast | jgriffith: maybe thats the problem, it might not be universal | 23:29 |
| jgriffith | my stuffs all tied up doing *things* right now, but I'll look | 23:29 |
| jgriffith | well that could be :) | 23:29 |
| patrickeast | no rush on it, just figured i'd throw rocks at things to start monday off | 23:29 |
| *** Julien-zte has quit IRC | 23:30 | |
| jgriffith | patrickeast I would've expected all of that stuff to be here: https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L277 | 23:30 |
| jgriffith | patrickeast but if that's not the case then ruh-roh | 23:30 |
| jgriffith | patrickeast I don't know how you restore connections on a reboot today :) | 23:30 |
| jgriffith | although, I guess that's not something somebody that's not LVM (running on Cinder node) would've necessarily thought of | 23:31 |
| patrickeast | jgriffith: haha, yea :( | 23:31 |
| patrickeast | jgriffith: afaik thats for things like lvm that runs the target info | 23:31 |
| patrickeast | jgriffith: so that it can rebuild targets | 23:31 |
| jgriffith | patrickeast right but we also have the ensure_export call for that | 23:32 |
| *** cknight has quit IRC | 23:32 | |
| patrickeast | jgriffith: afaik thats also optional, we don't manage exports at all | 23:33 |
| jgriffith | patrickeast ha! "optional" ? | 23:33 |
| jgriffith | LOL.. yeah, ya'll are pretty well F*&$(# then :) | 23:33 |
| jgriffith | serves ya right, slackers | 23:33 |
| patrickeast | i think previously the exports were only used by the drivers to know how to do things, and like you mentioned, to rebuild on startup | 23:33 |
| patrickeast | jgriffith: why? the backend array handles all that, we just tell it when to create or delete connections at intialize/terminate connections | 23:34 |
| patrickeast | cinder doesn't do anything else | 23:34 |
| patrickeast | need to do* | 23:34 |
| jgriffith | patrickeast sure... likely story | 23:34 |
| jgriffith | :) | 23:34 |
| patrickeast | lol | 23:34 |
| jgriffith | so yeah I see what you're saying | 23:35 |
| *** kaisers has joined #openstack-cinder | 23:36 | |
| jgriffith | There was a point where I thought we also called this from Nova on a restart; but anyway... what you're saying is you want a way to just stuff that and return it with the attachment | 23:36 |
| jgriffith | seems like a reasonable thing, since we won't do things like call initialize-connection directly | 23:36 |
| *** cdelatte has quit IRC | 23:36 | |
| jgriffith | patrickeast but honestly my hope was that you wouldn't ever do that | 23:36 |
| jgriffith | if you needed to rebuild something you just threw away the attachment and built a new one | 23:36 |
| patrickeast | jgriffith: yea i mean, if we get that info from *somewhere* that works for me | 23:37 |
| jgriffith | maybe that's not the best idea, but I like "disposable" things | 23:37 |
| jgriffith | patrickeast what do you think about just saying "if this is toast I don't requery, I just build a new one"? | 23:37 |
| jgriffith | I should let you finish describing the use-case/problem you want to solve here in detail :) | 23:38 |
| jgriffith | In other words, I'm a gonna shut up now and listen | 23:38 |
| patrickeast | jgriffith: that should be fine... but i think some flows did do that, iirc things like migrate did silly things | 23:38 |
| jgriffith | patrickeast migrate is evil, horrible and awful | 23:38 |
| patrickeast | jgriffith: i guess the concern is just that nova needs to know they can't do that kinda crap anymore | 23:38 |
| jgriffith | patrickeast yeah, I think I get ya | 23:39 |
| patrickeast | jgriffith: if they play by the rules, and do just create a new one when lost, and hold on to the info, they are ok | 23:39 |
| patrickeast | jgriffith: its just that they won't have any good way to get the info again after the call to create the attachment | 23:39 |
| jgriffith | patrickeast and I think that's where my plan of "create a new attachment" came in, but maybe that's going to be more problematic than I was thinking | 23:39 |
| jgriffith | patrickeast right, but they store that in the bdm | 23:39 |
| jgriffith | patrickeast so they're already persisting it | 23:40 |
| jgriffith | I hate duplicating like that | 23:40 |
| patrickeast | jgriffith: well, i recall comments about moving towards the bdm storing only an attachment id | 23:40 |
| *** crushil has joined #openstack-cinder | 23:40 | |
| jgriffith | patrickeast maybe some day | 23:40 |
| jgriffith | patrickeast I suppose that would be nice.. then just attachment-get and voila | 23:40 |
| jgriffith | patrickeast alright, I'll look at adding that, seems reasonable | 23:40 |
| *** rcernin has quit IRC | 23:41 | |
| patrickeast | jgriffith: cool, and if it turns out we don't need to worry about it, even better :D | 23:41 |
| jgriffith | thoughts on having it be something like a "connection_info_specs" table, just like the attachment_specs table? | 23:41 |
| patrickeast | jgriffith: yea its basically the same format, just for the other direction | 23:41 |
| jgriffith | well, it might afford some more robust things in the future if nothing else | 23:41 |
| jgriffith | patrickeast so I'm thinking for first pass Nova changes we may/may-not use it, but have it there | 23:42 |
| patrickeast | jgriffith: sounds good to me | 23:43 |
| *** catintheroof has quit IRC | 23:44 | |
| *** salv-orlando has quit IRC | 23:48 | |
| *** asselin has joined #openstack-cinder | 23:49 | |
| *** kong has joined #openstack-cinder | 23:49 | |
| *** edmondsw has joined #openstack-cinder | 23:49 | |
| *** edmondsw has quit IRC | 23:50 | |
| *** asselin__ has joined #openstack-cinder | 23:50 | |
| *** edmondsw has joined #openstack-cinder | 23:51 | |
| *** asselin_ has quit IRC | 23:52 | |
| *** xyang1 has quit IRC | 23:52 | |
| *** asselin has quit IRC | 23:54 | |
| *** gcb has quit IRC | 23:55 | |
| *** _ducttape_ has quit IRC | 23:58 | |
| *** jwcroppe has quit IRC | 23:59 | |
| *** jwcroppe has joined #openstack-cinder | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
