Wednesday, 2016-11-16

openstackgerritTom Swanson proposed openstack/cinder: WIP: Dell SC: Retype fixes  https://review.openstack.org/39797000:08
*** jwcroppe has quit IRC00:13
*** zul has quit IRC00:13
*** alonma has joined #openstack-cinder00:18
*** alonma has quit IRC00:23
*** laughterwym has joined #openstack-cinder00:31
*** laughterwym has quit IRC00:32
*** laughterwym has joined #openstack-cinder00:32
*** Julien-zte has joined #openstack-cinder00:43
*** zul has joined #openstack-cinder00:44
*** wanghao_ has joined #openstack-cinder00:45
*** jwcroppe has joined #openstack-cinder00:59
*** alonma has joined #openstack-cinder01:00
*** alonma has quit IRC01:04
*** rajinir has quit IRC01:16
*** yangyapeng has joined #openstack-cinder01:21
*** victorhe has joined #openstack-cinder01:29
*** victorhe has quit IRC01:32
*** victorhe has joined #openstack-cinder01:33
*** zhangguoqing has joined #openstack-cinder01:35
hemnawhat'd I do now?01:35
*** alonma has joined #openstack-cinder01:42
*** huanxuan has joined #openstack-cinder01:42
*** mriedem has joined #openstack-cinder01:45
*** alonma has quit IRC01:46
*** gcb has joined #openstack-cinder01:50
*** juzuluag has joined #openstack-cinder01:52
*** links has joined #openstack-cinder01:55
*** zhonghua-lee has quit IRC02:04
*** zhonghua-lee has joined #openstack-cinder02:05
*** links has quit IRC02:07
mriedemanyone else seeing this in the gate lately? http://logs.openstack.org/69/396469/7/check/gate-tempest-dsvm-cells-ubuntu-xenial/29e8b58/logs/screen-c-vol.txt.gz?level=TRACE#_2016-11-15_22_06_36_79102:17
*** alonma has joined #openstack-cinder02:23
*** alonma has quit IRC02:28
*** laughterwym has quit IRC02:29
*** laughterwym has joined #openstack-cinder02:31
*** victorhe has quit IRC02:32
*** yangyapeng has quit IRC02:33
*** yangyapeng has joined #openstack-cinder02:34
*** victorhe has joined #openstack-cinder02:35
*** yangyape_ has joined #openstack-cinder02:36
*** yangyapeng has quit IRC02:37
*** ckonstanski has quit IRC02:38
*** Apoorva has quit IRC02:39
*** bardia has quit IRC02:50
*** laughterwym has quit IRC02:55
*** laughterwym has joined #openstack-cinder02:56
*** Lee1092 has joined #openstack-cinder02:58
*** laughterwym has quit IRC03:01
*** laughterwym has joined #openstack-cinder03:01
*** laughterwym has quit IRC03:02
*** laughterwym has joined #openstack-cinder03:02
*** alonma has joined #openstack-cinder03:05
*** laughterwym has quit IRC03:07
*** laughterwym has joined #openstack-cinder03:07
openstackgerritxianming.mao proposed openstack/cinder: Delete the redundant expression expected_attrs  https://review.openstack.org/39805803:08
*** ckonstanski has joined #openstack-cinder03:08
*** alonma has quit IRC03:09
*** laughterwym has quit IRC03:09
*** laughterwym has joined #openstack-cinder03:10
*** victorhe has quit IRC03:12
*** victorhe has joined #openstack-cinder03:13
*** laughterwym has quit IRC03:14
*** laughterwym has joined #openstack-cinder03:15
*** victorhe has quit IRC03:17
*** juzuluag has quit IRC03:17
*** mriedem has quit IRC03:18
*** laughter_ has joined #openstack-cinder03:18
*** laughterwym has quit IRC03:19
*** laughter_ has quit IRC03:22
*** udesale has joined #openstack-cinder03:25
openstackgerritxing-yang proposed openstack/cinder: WIP: Migrate consistency groups to groups  https://review.openstack.org/35035003:38
*** alonma has joined #openstack-cinder03:46
*** alonma has quit IRC03:51
*** psachin has joined #openstack-cinder03:54
*** laughterwym has joined #openstack-cinder03:56
*** GB21 has joined #openstack-cinder04:12
*** takedakn has joined #openstack-cinder04:21
*** sudipto has joined #openstack-cinder04:24
*** bks has joined #openstack-cinder04:24
*** takedakn has quit IRC04:26
*** victorhe has joined #openstack-cinder04:26
*** kwathore__ has quit IRC04:26
*** kwathore__ has joined #openstack-cinder04:27
*** alonma has joined #openstack-cinder04:28
*** nicolasbock has quit IRC04:28
*** laughter_ has joined #openstack-cinder04:30
*** victorhe has quit IRC04:30
*** takedakn has joined #openstack-cinder04:31
*** alonma has quit IRC04:32
*** laughterwym has quit IRC04:34
*** khamtamtun has joined #openstack-cinder04:40
*** vladani_ has quit IRC04:43
*** khamtamtun has quit IRC04:44
*** sudipto has quit IRC04:53
*** archies has joined #openstack-cinder04:54
*** kwathore__ has quit IRC04:55
*** kwathore has joined #openstack-cinder04:55
*** kwathore has quit IRC05:07
*** kwathore has joined #openstack-cinder05:07
*** alonma has joined #openstack-cinder05:10
*** laughter_ has quit IRC05:11
*** laughterwym has joined #openstack-cinder05:11
*** takedakn has quit IRC05:14
*** alonma has quit IRC05:14
*** dims has quit IRC05:20
*** takedakn has joined #openstack-cinder05:22
*** pdeore has joined #openstack-cinder05:32
*** Julien-zte has quit IRC05:38
*** Julien-zte has joined #openstack-cinder05:41
*** Julien-zte has quit IRC05:41
*** Julien-zte has joined #openstack-cinder05:42
*** alonma has joined #openstack-cinder05:42
*** sandanar has joined #openstack-cinder05:42
*** alonma has quit IRC05:46
*** Julien-zte has quit IRC05:50
*** yangyape_ has quit IRC05:50
*** Julien-zte has joined #openstack-cinder05:50
*** sandanar has quit IRC05:51
*** alonmarx has joined #openstack-cinder05:54
*** harlowja has quit IRC05:55
*** gcb has quit IRC05:58
*** dims has joined #openstack-cinder05:59
*** gcb has joined #openstack-cinder05:59
*** kwathore_ has joined #openstack-cinder06:01
*** kwathore has quit IRC06:04
*** Poornima has joined #openstack-cinder06:06
*** pbandark has joined #openstack-cinder06:09
*** ChubYann has quit IRC06:10
*** victorhe has joined #openstack-cinder06:11
*** Julien-zte has quit IRC06:12
*** lpetrut has joined #openstack-cinder06:12
*** Julien-zte has joined #openstack-cinder06:13
*** victorhe has quit IRC06:17
*** Julien-zte has quit IRC06:18
*** Julien-zte has joined #openstack-cinder06:18
*** alonma has joined #openstack-cinder06:23
*** Julien-zte has quit IRC06:27
*** alonma has quit IRC06:28
*** Julien-zte has joined #openstack-cinder06:28
*** markvoelker has quit IRC06:28
*** yangyapeng has joined #openstack-cinder06:30
*** harlowja_at_home has joined #openstack-cinder06:32
*** harlowja_at_home has quit IRC06:33
*** lpetrut has quit IRC06:42
*** Poornima has quit IRC06:43
*** Julien-zte has quit IRC06:48
*** Julien-zte has joined #openstack-cinder06:49
*** Julien-zte has quit IRC06:56
*** Julien-zte has joined #openstack-cinder06:57
*** snarwade has quit IRC06:57
*** snarwade has joined #openstack-cinder07:00
*** laughterwym has quit IRC07:01
*** laughterwym has joined #openstack-cinder07:03
*** laughterwym has quit IRC07:04
*** pgadiya has joined #openstack-cinder07:04
*** laughterwym has joined #openstack-cinder07:04
*** alonma has joined #openstack-cinder07:05
*** alonma has quit IRC07:09
*** arecknag has joined #openstack-cinder07:09
*** mjura has joined #openstack-cinder07:11
*** laughterwym has quit IRC07:11
*** laughterwym has joined #openstack-cinder07:12
*** GB21 has quit IRC07:16
*** laughterwym has quit IRC07:16
*** mjura has quit IRC07:19
*** mjura has joined #openstack-cinder07:19
openstackgerritxianming.mao proposed openstack/cinder: Modify the position of try-except  https://review.openstack.org/39814907:20
*** yangyape_ has joined #openstack-cinder07:25
*** yangyapeng has quit IRC07:25
*** GB21 has joined #openstack-cinder07:29
*** laughterwym has joined #openstack-cinder07:30
*** nherciu has joined #openstack-cinder07:35
*** houming has joined #openstack-cinder07:36
*** lpetrut has joined #openstack-cinder07:37
*** pcaruana has joined #openstack-cinder07:41
*** lpetrut has quit IRC07:41
*** rha has joined #openstack-cinder07:42
*** laughterwym has quit IRC07:42
*** laughterwym has joined #openstack-cinder07:43
*** alonmarx has quit IRC07:43
*** alonma has joined #openstack-cinder07:47
*** laughterwym has quit IRC07:47
*** alonma has quit IRC07:51
*** laughterwym has joined #openstack-cinder07:54
*** Julien-zte has quit IRC07:58
openstackgerritint32bit proposed openstack/cinder: Add host check while create snapshot  https://review.openstack.org/39684807:58
*** Julien-zte has joined #openstack-cinder08:01
*** houming has quit IRC08:02
*** gcb has quit IRC08:02
*** takedakn has quit IRC08:06
*** Poornima has joined #openstack-cinder08:08
*** houming has joined #openstack-cinder08:12
*** gcb has joined #openstack-cinder08:15
*** amoralej|off is now known as amoralej08:20
*** jordanP has joined #openstack-cinder08:23
*** gcb has quit IRC08:27
*** alonma has joined #openstack-cinder08:28
*** markvoelker has joined #openstack-cinder08:29
*** takedakn has joined #openstack-cinder08:29
*** e0ne has joined #openstack-cinder08:30
*** gcb has joined #openstack-cinder08:30
*** alonma has quit IRC08:33
*** cknight has joined #openstack-cinder08:33
*** markvoelker has quit IRC08:34
*** jordanP has quit IRC08:35
*** laughterwym has quit IRC08:36
*** laughterwym has joined #openstack-cinder08:36
*** GB21 has quit IRC08:38
*** lpetrut has joined #openstack-cinder08:39
*** laughterwym has quit IRC08:41
*** Julien-zte has quit IRC08:45
*** Julien-zte has joined #openstack-cinder08:45
*** Poornima has quit IRC08:51
*** GB21 has joined #openstack-cinder08:51
*** Poornima has joined #openstack-cinder08:57
*** zzzeek has quit IRC09:00
*** Swanson has quit IRC09:00
*** e0ne has quit IRC09:02
*** victorhe has joined #openstack-cinder09:03
*** zzzeek has joined #openstack-cinder09:03
*** victorhe has quit IRC09:07
*** alonma has joined #openstack-cinder09:10
*** cknight has quit IRC09:13
*** cknight1 has joined #openstack-cinder09:13
*** gouthamr has joined #openstack-cinder09:13
*** alonma has quit IRC09:14
*** Poornima has quit IRC09:15
*** Poornima has joined #openstack-cinder09:17
*** gouthamr has quit IRC09:17
*** gouthamr has joined #openstack-cinder09:17
*** takedakn has quit IRC09:19
*** Julien-zte has quit IRC09:20
*** jordanP has joined #openstack-cinder09:23
openstackgerritLee Yarwood proposed openstack/os-brick: encryptors: Mock os_brick.executor correctly  https://review.openstack.org/39793309:28
openstackgerritLee Yarwood proposed openstack/os-brick: encryptors: Workaround mangled passphrases  https://review.openstack.org/39793409:28
*** cknight has joined #openstack-cinder09:35
*** cknight1 has quit IRC09:36
*** gouthamr has quit IRC09:43
*** khushbu has joined #openstack-cinder09:45
*** gouthamr has joined #openstack-cinder09:50
*** alonma has joined #openstack-cinder09:51
*** alonmarx has joined #openstack-cinder09:52
*** alonma has quit IRC09:56
*** yangyape_ has quit IRC09:56
*** pdeore has quit IRC09:58
*** victorhe has joined #openstack-cinder10:02
*** laughterwym has joined #openstack-cinder10:03
*** alonmarx has quit IRC10:05
*** e0ne has joined #openstack-cinder10:05
*** victorhe has quit IRC10:06
*** laughterwym has quit IRC10:07
*** siddhi has joined #openstack-cinder10:08
*** siddhi has quit IRC10:10
*** cknight1 has joined #openstack-cinder10:10
*** cknight has quit IRC10:13
*** Poornima has quit IRC10:14
*** pdeore has joined #openstack-cinder10:14
*** stakeda has quit IRC10:16
*** Poornima has joined #openstack-cinder10:20
*** zengyingzhe has quit IRC10:20
*** zengyingzhe has joined #openstack-cinder10:21
*** GB21 has quit IRC10:22
*** udesale has quit IRC10:26
*** mvk has quit IRC10:30
*** markvoelker has joined #openstack-cinder10:30
*** victorhe has joined #openstack-cinder10:31
*** alonma has joined #openstack-cinder10:33
*** GB21 has joined #openstack-cinder10:34
*** markvoelker has quit IRC10:35
*** victorhe has quit IRC10:36
*** alonma has quit IRC10:38
*** gouthamr has quit IRC10:38
*** gouthamr has joined #openstack-cinder10:39
*** zengyingzhe_ has joined #openstack-cinder10:41
*** tpsilva has joined #openstack-cinder10:41
*** victorhe has joined #openstack-cinder10:42
*** zengyingzhe has quit IRC10:44
*** victorhe has quit IRC10:46
*** huanxuan has quit IRC10:53
*** laughterwym has joined #openstack-cinder10:53
*** khushbu has quit IRC10:53
*** mvk has joined #openstack-cinder10:54
*** laughterwym has quit IRC10:59
*** laughterwym has joined #openstack-cinder10:59
openstackgerritxianming.mao proposed openstack/cinder: Modify the position of try-except  https://review.openstack.org/39814911:01
*** cknight1 has quit IRC11:02
*** bks has quit IRC11:04
*** laughterwym has quit IRC11:04
*** houming has left #openstack-cinder11:04
*** yangyapeng has joined #openstack-cinder11:04
*** nikeshm has joined #openstack-cinder11:06
*** yangyape_ has joined #openstack-cinder11:06
*** gouthamr has quit IRC11:07
openstackgerritRodion Tikunov proposed openstack/python-cinderclient: Fix typo in set unicode metadata key  https://review.openstack.org/39151511:09
*** yangyapeng has quit IRC11:09
*** victorhe has joined #openstack-cinder11:11
openstackgerritint32bit proposed openstack/cinder: Add host check while create snapshot  https://review.openstack.org/39684811:13
openstackgerritxianming.mao proposed openstack/cinder: Modify the position of try-except  https://review.openstack.org/39814911:14
*** alonma has joined #openstack-cinder11:15
*** victorhe has quit IRC11:16
*** GB21 has quit IRC11:18
openstackgerritErlon R. Cruz proposed openstack/cinder: Introduce Hitachi VSP driver  https://review.openstack.org/38937711:18
*** alonma has quit IRC11:19
erlon-airlong.11:20
*** adisky has joined #openstack-cinder11:20
*** cknight has joined #openstack-cinder11:26
*** gouthamr has joined #openstack-cinder11:26
*** Swanson has joined #openstack-cinder11:30
openstackgerritErlon R. Cruz proposed openstack/cinder: [WIP] Introduce Hitachi VSP iSCSI driver  https://review.openstack.org/39830011:30
*** nicolasbock has joined #openstack-cinder11:31
*** GB21 has joined #openstack-cinder11:31
*** vbala has joined #openstack-cinder11:32
*** vbala_ has joined #openstack-cinder11:34
openstackgerritErlon R. Cruz proposed openstack/cinder: [WIP] Introduce Hitachi VSP iSCSI driver  https://review.openstack.org/39830011:35
openstackgerritErlon R. Cruz proposed openstack/cinder: Introduce Hitachi VSP driver  https://review.openstack.org/38937711:35
*** vbala has quit IRC11:35
*** vbala_ has quit IRC11:36
*** alonma has joined #openstack-cinder11:47
*** huanxuan has joined #openstack-cinder11:50
*** alonma has quit IRC11:51
*** gouthamr has quit IRC11:51
*** gouthamr has joined #openstack-cinder11:52
*** timcl has joined #openstack-cinder11:55
*** vgridnev has joined #openstack-cinder12:03
geguileodulek: ping - oslo messaging namespaces12:08
*** archies has quit IRC12:08
*** vgridnev has left #openstack-cinder12:13
*** alonma has joined #openstack-cinder12:13
*** thurloat has quit IRC12:15
*** thurloat has joined #openstack-cinder12:16
dulekgeguileo: I'm at the meeting, will ping you in 45 minutes.12:17
*** alonma has quit IRC12:18
*** pdeore has quit IRC12:18
dulekgeguileo: Right now you can take a look on my yesterday's discussion with sileht: http://eavesdrop.openstack.org/irclogs/%23openstack-oslo/%23openstack-oslo.2016-11-15.log.html#t2016-11-15T09:59:4612:18
geguileodulek: Ok12:20
geguileodulek: I could have saved some time looking into this if I had just pinged you earlier12:22
dulekgeguileo: I've took a look on your email - I would want only to switch AMQP message routing somehow, not change anything in the DB.12:22
dulekgeguileo: Just RPCServer and RPCClient layers.12:22
geguileodulek: My results where the same as yours12:22
geguileodulek: namespaces don't help us12:22
geguileodulek: Because they route in the process12:22
*** GB21 has quit IRC12:22
geguileodulek: But let's talk after your meeting12:23
geguileo:-)12:23
*** catintheroof has joined #openstack-cinder12:23
dulekgeguileo: Yup!12:23
openstackgerritGorka Eguileor proposed openstack/cinder: Make Image Volume Cache cluster aware  https://review.openstack.org/38618612:24
openstackgerritGorka Eguileor proposed openstack/cinder: Add more operations to cluster  https://review.openstack.org/35596812:24
openstackgerritGorka Eguileor proposed openstack/cinder: Support A/A on Scheduler operations  https://review.openstack.org/34422612:24
openstackgerritGorka Eguileor proposed openstack/cinder: [PoC][Don't review] Testing delay  https://review.openstack.org/35306912:24
openstackgerritGorka Eguileor proposed openstack/cinder: Support A/A in attach/detach operations  https://review.openstack.org/31857312:24
openstackgerritGorka Eguileor proposed openstack/cinder: Allow triggering cleanup from API  https://review.openstack.org/36301012:24
openstackgerritGorka Eguileor proposed openstack/cinder: Support A/A in delete actions and get_capabilities  https://review.openstack.org/34422512:24
openstackgerritGorka Eguileor proposed openstack/cinder: Make Replication support Active-Active  https://review.openstack.org/38183512:24
openstackgerritGorka Eguileor proposed openstack/cinder: Cosmetic changes to scheduler  https://review.openstack.org/34604112:25
openstackgerritGorka Eguileor proposed openstack/cinder: Prevent Active-Active on drivers by default  https://review.openstack.org/39346012:25
*** cknight has quit IRC12:27
*** victorhe has joined #openstack-cinder12:30
*** cknight has joined #openstack-cinder12:31
*** gouthamr has quit IRC12:32
*** victorhe has quit IRC12:34
*** alonma has joined #openstack-cinder12:37
*** gcb has quit IRC12:38
*** alonma has quit IRC12:42
*** khushbu_ has joined #openstack-cinder12:42
*** gcb has joined #openstack-cinder12:42
openstackgerritint32bit proposed openstack/cinder: Add host check while create snapshot  https://review.openstack.org/39684812:43
*** alonma has joined #openstack-cinder12:47
*** khushbu_ has quit IRC12:49
*** akapil has joined #openstack-cinder12:50
*** zhipeng has joined #openstack-cinder13:01
*** vgridnev has joined #openstack-cinder13:02
*** ducttape_ has joined #openstack-cinder13:08
*** ducttape_ has quit IRC13:09
*** ducttape_ has joined #openstack-cinder13:10
*** Poornima has quit IRC13:10
*** Poornima has joined #openstack-cinder13:12
*** pgadiya has quit IRC13:12
*** Julien-zte has joined #openstack-cinder13:19
dulekgeguileo: I'm available now. So right now I'm looking into using topics for routing.13:20
dulekgeguileo: The only problem is the fanout message from scheduler, that should go to every backend.13:20
dulekgeguileo: And again - I'm looking only on changes to the transport side, nothing about how we're processing host field and how we're saving it in the DB.13:20
dulekgeguileo: I think _at least_ A/A code is assuming one process per Manager's instance, so using namespaces and routing in the process doesn't really work for us.13:20
dulekgeguileo: Even without clusters there may be even more dragons. :(13:20
geguileodulek: But why do we want to change the routing we have now?13:20
* dulek was writing into the void until IRC reconnected… 13:20
dulekgeguileo: Problem is - zmq.13:20
dulekgeguileo: oslo.messaging doesn't assume you to mess with host field.13:21
dulekgeguileo: It works in RabbitMQ by chance, but in zmq host/server should be set to target hostname.13:21
*** alonma has quit IRC13:21
geguileoBut isn't "server" the same as "topic queue" is oslo messaging?13:22
*** alonma has joined #openstack-cinder13:22
* geguileo saying it from memory13:22
dulekgeguileo: Well, apparently not. Target for RPCServer routes based on server and topic fields.13:23
*** akapil has quit IRC13:23
dulekgeguileo: Now we're abusing this Target.server, which breaks zmq.13:23
geguileodulek: Last time I check oslo messaging I arrived at the conclussion that it's toooooooooooooooo restrictive to do anything13:24
geguileodulek: I had problems with the cluster queues/exchanges13:24
dulekgeguileo: Guys were trying to get zmq working by switching this routing to topic *only* for zmq case. But this is ugly, as we're basing our logic on setting of abstraction layer, which basically sucks.13:24
dulekgeguileo: Hm, what problem?13:25
*** jamielennox is now known as jamielennox|away13:25
geguileodulek: I don't remember exactly13:25
geguileodulek: It was due to the restrictions set on messaging13:25
geguileodulek: It's intended for a very specific use case13:25
geguileodulek: There's no way to control exchanges/queues, etc13:26
*** dave-mccowan has joined #openstack-cinder13:26
*** cknight has quit IRC13:26
* geguileo would have to check the code again13:26
dulekgeguileo: BTW - isn't this all implemented for RabbitMQ as oslo.messaging.RPCServer listening on two queues - one server-specific and one topic-specific?13:26
*** alonma has quit IRC13:26
dulekgeguileo: Sure, and I wouldn't oppose to that - that's the purpose of abstraction layer.13:26
geguileodulek: Ooooh, maybe it was the RPC server limitation13:26
*** Poornima has quit IRC13:27
*** GB21 has joined #openstack-cinder13:27
*** bkopilov_ is now known as bkopilov13:28
dulekAnyway getting back to the point - I would like to implement our multibackend routing using oslo.messaging behaviors that aren't RabbitMQ-specific.13:28
dulekgeguileo: Right now we're using RabbitMQ backend implementation details.13:29
duleks/using/relying on13:29
geguileodulek: This was the problem: https://review.openstack.org/#/c/381835/12/cinder/service.py@26213:29
*** akapil has joined #openstack-cinder13:29
geguileodulek: That sounds good to me13:30
geguileodulek: I'm just not too sure that I want that to delay my A/A work another 6 months13:30
dulekgeguileo: I believe you have the priority here, so I'm looking for a way that wouldn't affect you.13:30
*** cknight has joined #openstack-cinder13:31
geguileodulek: I wouldn't mind making a few changes if it can save us from the upgrading nightmare later on13:31
geguileodulek: Do you have a proposal on how to change it to be ZMQ compatible?13:31
dulekgeguileo: Oh, this code snippet is interesting, let me take a look.13:31
dulekgeguileo: I was only looking at your already merged code.13:31
*** xyang1 has joined #openstack-cinder13:32
dulekgeguileo: So maybe I'm missing some relationship here.13:32
geguileodulek: That code is to create a broadcasting exchange that will go to all the services within a clustered backend13:33
dulekgeguileo: Oh, okay, I get it - you want to be able to send fanout messages to all cluster members, but not all volumes, right?13:34
dulekgeguileo: Yeah, exactly.13:34
geguileodulek: Because when we complete a failover we have to tell all the services that the failover has been done13:34
*** akapil has quit IRC13:34
geguileodulek: You can also do it to all volumes13:34
geguileodulek: I just need this additional level of broadcasting13:34
geguileodulek: Specific to the services in the cluster13:34
geguileodulek: Because everybody else doesn't care that backend X has been failed over and those services need to point to the new storage site13:35
*** takedakn has joined #openstack-cinder13:35
dulekgeguileo: Yes, yes, I get it. Just members of one cluster/backend should receive it.13:35
geguileodulek: Yup13:36
*** zhangguoqing has quit IRC13:36
*** Lee1092 has quit IRC13:36
dulekgeguileo: And to address all volumes you only need Target(topic='cinder-volume', fanout=True) - we're already calling that in scheduler to get all reports on scheduler start.13:36
geguileodulek: Yup13:37
*** ducttape_ has quit IRC13:38
*** ducttape_ has joined #openstack-cinder13:38
geguileodulek: If I'm reading this correctly I'm already doing what you wanted me to do...13:39
dulekgeguileo: Okay, cool. So we can agree namespaces doesn't work, as inter-process routing could be dangerous, because there probably are assumptions that Managers are working in different processes.13:39
geguileodulek:yes13:39
dulekgeguileo: Yeah, exactly, your code looks more or less like we should do multibackend in the first place.13:39
geguileodulek: I'm setting the topic with the whole @backend and the server is just using the host part of the host13:39
geguileodulek: So that patch is already making the change you wanted  :-)13:40
dulekgeguileo: Miracle! :D13:40
*** alonma has joined #openstack-cinder13:41
geguileoGreat!!!13:41
geguileodulek: Funny thing is that I think you can use that same change for the normal topic and it will work13:41
dulekgeguileo: Yes and no. Can I keep writing? ;)13:42
geguileodulek: Ooooooooooooooooooooooooooooooooook  ;-)13:42
dulekgeguileo: Getting out of cluster RPCServer now, and back to the standard one.13:42
*** e0ne has quit IRC13:42
*** ducttape_ has quit IRC13:43
dulekgeguileo: It should be possible to apply your change to standard RPCServer above.13:43
dulekgeguileo: + add a legacy RPCServer for compatibility purposes.13:43
*** edmondsw has joined #openstack-cinder13:43
geguileodulek: Are you sure a legacy one is required?13:43
geguileodulek: Because I think for rabbit mq this will result in the same addressing13:43
dulekgeguileo: Upgrade order tells you that scheduler goes first, and so on, but c-vol can communicate with c-vols.13:44
*** markvoelker has joined #openstack-cinder13:44
geguileodulek: I think it will work for rabbitmq13:44
geguileodulek: Regardless of the order you upgrade13:44
*** zhipeng has quit IRC13:44
dulekgeguileo: So during upgrade you should listen on both Target(server='host', topic='host@backend') and Target(server='host@backend', topic='cinder-volume')13:45
*** zhipeng has joined #openstack-cinder13:45
*** alonma has quit IRC13:45
dulekgeguileo: Because oslo.messaging will listen on both queues - topic and server?13:45
geguileodulek: I think so13:46
geguileodulek: We would have to check, but I think it would work13:46
dulekgeguileo: Still you'll have to listen on both server='host' (new way), and server='host@backend' (old way).13:46
geguileoBut you are already listening in host@backend since it's the new topic...13:47
dulekgeguileo: Or is "@" special character? Or RabbitMQ is matching queues with regex?13:47
geguileoNo, it's not a special character13:47
dulekgeguileo: Ah, interesting. I wonder if there isn't something special added to server queues, let me check.13:48
*** huanxuan has quit IRC13:48
dulekgeguileo: I would need to prefix it with cinder-volume, but it could work…13:49
dulekgeguileo: cinder-volume.vagrant-ubuntu-trusty-64.localdomain      013:49
dulekgeguileo: cinder-volume.vagrant-ubuntu-trusty-64.localdomain      013:49
*** yangyape_ has quit IRC13:49
dulekgeguileo: Aw, I mean:13:49
dulekgeguileo: cinder-volume.vagrant-ubuntu-trusty-64.localdomain@lvmdriver-1  013:50
geguileodulek: That's what my code is doing13:50
geguileodulek: topic='%s.%s' % (self.topic, self.cluster)13:50
geguileolol13:50
dulekgeguileo: Oh, right, I've missed it!13:50
*** gouthamr has joined #openstack-cinder13:50
geguileodulek: I made it so that it would be compatible13:50
dulekgeguileo: Okay, so this *could* work, but there's one more problem.13:50
geguileoOh, no more problems for today!!13:51
geguileoXD13:51
dulekgeguileo: Scheduler wants to send fanout to all cinder volumes. It's using fanout to cinder-volume queue now.13:51
dulekgeguileo: Idea 1 - make scheduler iterate and send it to every backend.13:51
*** Administrator_ has quit IRC13:51
dulekgeguileo: Idea 2 - additionally listen on cinder-volume.13:51
*** victorhe has joined #openstack-cinder13:51
dulek1 is ugly, 2 adds third, required RPCServer.13:52
*** Administrator_ has joined #openstack-cinder13:52
geguileo1 does not work because it doesn't know all available backends13:52
dulekgeguileo: What do you think about both? Or am I missing something obvious again.13:52
geguileo2 is the only option  :-(13:52
*** amoralej is now known as amoralej|lunch13:52
*** yangyapeng has joined #openstack-cinder13:52
*** zhipeng has quit IRC13:53
*** zhipeng has joined #openstack-cinder13:53
dulekOkay. I'll dig into it while writing the code. Thank you very much, this talk was extremely helpful! :)13:53
*** gouthamr has quit IRC13:54
*** takedakn has quit IRC13:55
*** victorhe has quit IRC13:56
geguileodulek: glad I could help (and I don't have to change my code)  :-)13:56
*** udesale has joined #openstack-cinder13:56
*** zhipeng has quit IRC13:59
*** akapil_ has joined #openstack-cinder14:00
*** laughterwym has joined #openstack-cinder14:02
*** akapil_ has quit IRC14:04
*** porrua has joined #openstack-cinder14:05
openstackgerritmorfy proposed openstack/cinder: Cluster volume group fix in lvm  https://review.openstack.org/39626914:05
*** dustins has joined #openstack-cinder14:06
*** mriedem has joined #openstack-cinder14:07
*** alonma has joined #openstack-cinder14:08
*** akapil has joined #openstack-cinder14:10
*** gouthamr has joined #openstack-cinder14:11
*** victorhe has joined #openstack-cinder14:12
*** alonma has quit IRC14:12
*** Administrator_ has quit IRC14:13
*** dustins has quit IRC14:13
*** Administrator_ has joined #openstack-cinder14:14
*** Cibo has joined #openstack-cinder14:18
*** ducttape_ has joined #openstack-cinder14:20
*** akapil has quit IRC14:20
*** akapil has joined #openstack-cinder14:21
*** vgridnev has quit IRC14:22
*** jgregor has joined #openstack-cinder14:22
*** victorhe_ has joined #openstack-cinder14:25
*** dustins has joined #openstack-cinder14:27
*** victorhe has quit IRC14:28
*** victorhe_ has quit IRC14:29
*** dave-mccowan has quit IRC14:30
*** wanghao_ has quit IRC14:31
*** ducttape_ has quit IRC14:31
*** ducttape_ has joined #openstack-cinder14:32
*** dustins has quit IRC14:32
*** amoralej|lunch is now known as amoralej14:32
*** dustins has joined #openstack-cinder14:33
*** e0ne has joined #openstack-cinder14:36
*** ducttape_ has quit IRC14:36
*** akapil has quit IRC14:37
*** Lee1092 has joined #openstack-cinder14:38
*** gouthamr has quit IRC14:39
*** cknight has quit IRC14:40
*** tongli has joined #openstack-cinder14:44
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Storage assisted volume migration.  https://review.openstack.org/39786214:45
*** e0ne has quit IRC14:46
*** victorhe has joined #openstack-cinder14:48
*** akapil has joined #openstack-cinder14:48
*** victorhe has quit IRC14:50
*** psachin has quit IRC14:51
*** victorhe has joined #openstack-cinder14:51
*** swamireddy has quit IRC14:52
dulekgeguileo: Ha, I've just hacked something that seems functional in less than an hour. I'll switch to zmq and retest, but I don't think it will explode.14:54
*** ducttape_ has joined #openstack-cinder14:54
*** GB21 has quit IRC14:54
*** nherciu has quit IRC14:54
geguileodulek: Awesome!!!14:55
*** andymaier has joined #openstack-cinder14:55
*** akapil has quit IRC14:56
*** slayer has joined #openstack-cinder14:57
*** andymaier has quit IRC14:57
*** e0ne has joined #openstack-cinder15:01
*** adrianofr has joined #openstack-cinder15:01
*** zhipeng has joined #openstack-cinder15:05
*** alonma has joined #openstack-cinder15:06
*** jungleboyj has joined #openstack-cinder15:06
*** mpjetta has quit IRC15:07
*** cknight has joined #openstack-cinder15:11
*** swamireddy_ has joined #openstack-cinder15:15
*** pbandark has quit IRC15:17
Cibohello everybody, will cinder meeting be today?15:17
dulekCibo: Looks like agenda isn't empty, so most probably yes.15:18
dulekCibo: https://wiki.openstack.org/wiki/CinderMeetings#Next_Cinder_Team_meeting15:18
*** jgregor_ has joined #openstack-cinder15:21
*** yangyapeng has quit IRC15:22
*** victorhe has quit IRC15:23
*** jgregor has quit IRC15:25
Cibodulek: Ok, Thank you.15:27
*** chris_morrell has joined #openstack-cinder15:28
*** arecknag has quit IRC15:28
*** rajinir has joined #openstack-cinder15:30
*** ckonstanski has quit IRC15:33
*** alonma has quit IRC15:34
*** alonma has joined #openstack-cinder15:34
*** ducttape_ has quit IRC15:35
*** udesale has quit IRC15:35
*** jwcroppe has quit IRC15:37
*** akapil has joined #openstack-cinder15:38
*** alonma has quit IRC15:39
smcginnisgeguileo: If you have a moment, it would help if you could update the HA A/A patches in the review etherpad: https://etherpad.openstack.org/p/cinder-spec-review-tracking15:39
geguileosmcginnis: I'll get to it right away  :-)15:39
smcginnisgeguileo: Thanks! I think that would help get some more reviews going.15:40
geguileosmcginnis: I would settle for "some review"  ;-)15:40
smcginnis;)15:42
*** chris_morrell has quit IRC15:43
*** akapil has quit IRC15:43
*** zhipeng has quit IRC15:49
geguileosmcginnis: Done15:49
smcginnisgeguileo: Thank you15:50
geguileonp15:50
*** ociuhandu has joined #openstack-cinder15:51
smcginnisgeguileo: Cleaning things up. This was in the Mitaka section: https://review.openstack.org/#/c/315541/15:52
smcginnisgeguileo: Was that done in another patch, or no longer needed?15:53
geguileosmcginnis: I moved it out of the dependency chain since there were some concerns15:53
geguileosmcginnis: I should probably update it though15:54
*** jseiler has joined #openstack-cinder15:54
geguileosmcginnis: In summary, it is not needed15:54
smcginnisgeguileo: OK, I'll leave the reference to it in there for now.15:54
geguileosmcginnis: you can remove it15:54
smcginnisgeguileo: Oh, OK. Then I'll remove it. :)15:54
geguileothanks15:54
*** jwcroppe has joined #openstack-cinder15:55
*** lpetrut has quit IRC15:55
SwansonDaily beg for reviews... https://review.openstack.org/#/c/396343/15:59
smcginniseharney: Was the code for this spec merged? https://review.openstack.org/#/c/320052/15:59
*** bardia has joined #openstack-cinder16:00
eharneysmcginnis: nope, still in (minimal) review16:00
*** ducttape_ has joined #openstack-cinder16:02
*** dave-mccowan has joined #openstack-cinder16:04
*** mjura has quit IRC16:06
*** jgregor has joined #openstack-cinder16:07
*** gouthamr has joined #openstack-cinder16:09
*** jgregor_ has quit IRC16:10
*** jgregor_ has joined #openstack-cinder16:11
*** clenimar has joined #openstack-cinder16:11
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Create a CG from a source CG  https://review.openstack.org/37662016:13
hemnadoink16:14
Swansonhemna is Blair Walsh?16:14
*** chris_morrell has joined #openstack-cinder16:14
*** jgregor has quit IRC16:15
hemnaChewbacca16:15
openstackgerritJon Bernard proposed openstack/cinder: RBD: improve readability in over provisioning tests  https://review.openstack.org/39192716:15
*** diablo_rojo has joined #openstack-cinder16:18
openstackgerritMichal Dulko proposed openstack/cinder: Make BackupManager not a SchedulerDependentManager  https://review.openstack.org/39626416:18
openstackgerritMichal Dulko proposed openstack/cinder: WIP: Use oslo.messaging topics for multibackend  https://review.openstack.org/39845216:18
*** ducttape_ has quit IRC16:22
*** ducttape_ has joined #openstack-cinder16:23
*** erhudy has joined #openstack-cinder16:24
duleksmcginnis: Can you take a glance at what I'm trying to do in https://review.openstack.org/398452 ?16:29
duleksmcginnis: I'm starting to wonder if I shouldn't create a spec or at least a blueprint.16:29
*** ckonstanski has joined #openstack-cinder16:30
Swansonso did we switch from consistency groups to groups with consistence or what?16:30
smcginnisdulek: I thought there was one out there already for zmq support.16:30
smcginnisSwanson: In process now.16:31
duleksmcginnis: Oh, good point. This patch is however extending the idea from the spec for all RPC backends, without if "zmc://" conditional.16:32
Swansonsmcginnis, Actually looking for what we are calling this.16:32
smcginnisdulek: Ah, good.16:33
smcginnisSwanson: Apple crisp16:33
SwansonWell, look for a dell/applecrisp branch.16:33
duleksmcginnis: The reason for that is - we shouldn't really base our logic on abstraction layer implementation details.16:33
*** jseiler has left #openstack-cinder16:34
smcginnisdulek: I like that. That was some of the negative feedback on the zmq work is it was too implementation specific.16:34
smcginnisdulek: Quick review, but nothing jumped out at me.16:34
smcginnisdulek: Will be good to see how Jenkins likes it.16:34
smcginnisAt least for the tempest and grenade tests like you mention.16:35
duleksmcginnis: Getting back to my question - should I do a spec update, and mention the blueprint in the commit message?16:36
duleksmcginnis: Or a new spec? Or a specless blueprint?16:37
smcginnisdulek: I suppose. Seems minor enough, but I guess if someone goes looking it will be good to have links.16:37
smcginnisdulek: I think the spec update.16:37
duleksmcginnis: Okay, thanks!16:37
smcginnisdulek: Or even just reference that blueprint. Doesn't seem like the effort of creating a new spec would be worth it.16:38
*** slayer has quit IRC16:38
*** slayer has joined #openstack-cinder16:39
smcginnisdiablo_rojo: Hey, didn't you have a patch for this? https://review.openstack.org/#/c/286234/16:39
SwansonDear god please tell me that just reloads the drivers.16:42
*** slayer has quit IRC16:43
*** e0ne has quit IRC16:45
*** laughterwym has quit IRC16:45
*** jistr is now known as jistr|biab16:48
*** alonma has joined #openstack-cinder16:52
*** chenying has quit IRC16:53
*** chenying has joined #openstack-cinder16:54
*** chris_morrell has quit IRC16:54
*** alonma has quit IRC16:56
*** Zer0Byte__ has joined #openstack-cinder17:01
*** swamireddy_ has quit IRC17:02
*** slayer has joined #openstack-cinder17:02
*** xyang has joined #openstack-cinder17:04
*** slayer has quit IRC17:06
diablo_rojosmcginnis: What do you mean?17:06
diablo_rojosmcginnis: That's the spec, I had seen something you pushed moving it to ocata, but I hadn't gotten started on it yet.17:07
SwansonESL?17:08
smcginnisdiablo_rojo: OK, I thought I might have seen the start of the actual implementation. Must be thinking of something else.17:08
smcginnisEnglish as a Second Language.17:08
diablo_rojosmcginnis ah okay :) It's nearing the top of my todo list. Should be able to get started soon.17:08
smcginnisdiablo_rojo: Cool17:09
SwansonI see 10.0.0.0b1 is out.17:09
*** chris_morrell has joined #openstack-cinder17:11
*** Poornima has joined #openstack-cinder17:13
*** ociuhandu has quit IRC17:14
*** cknight has quit IRC17:14
*** slayer has joined #openstack-cinder17:19
*** zengyingzhe_ has quit IRC17:21
*** zengyingzhe_ has joined #openstack-cinder17:21
*** swamireddy has joined #openstack-cinder17:26
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Create a CG from a source CG  https://review.openstack.org/37662017:26
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Storage assisted volume migration.  https://review.openstack.org/39786217:26
*** xyang has quit IRC17:27
*** xyang has joined #openstack-cinder17:33
*** lennyb has quit IRC17:35
*** jistr|biab is now known as jistr17:36
*** Poornima has quit IRC17:39
*** Apoorva has joined #openstack-cinder17:41
*** lennyb has joined #openstack-cinder17:46
nikeshmhi17:47
nikeshmreport_discard_supported is not supported in liberty r817:47
nikeshmit is supported mitaka onwards r817:47
*** david-lyle has quit IRC17:48
*** david-lyle has joined #openstack-cinder17:48
*** slayer has quit IRC17:52
*** jgregor_ has quit IRC17:53
*** slayer_ has joined #openstack-cinder17:54
*** adisky has quit IRC17:55
*** slayer_ has quit IRC17:56
*** slayer has joined #openstack-cinder17:57
*** jgregor has joined #openstack-cinder17:57
*** mvk has quit IRC17:58
*** cdelatte has joined #openstack-cinder18:00
*** jgregor has quit IRC18:02
*** alonma has joined #openstack-cinder18:04
*** chris_morrell has quit IRC18:05
*** chris_morrell has joined #openstack-cinder18:06
*** sam64_ has quit IRC18:07
*** sam64 has joined #openstack-cinder18:07
*** pkoniszewski has quit IRC18:07
*** alonma has quit IRC18:08
patrickeastnikeshm: thats correct, the shared config option and nova side of the changes didn't land until mitaka18:09
*** pkoniszewski has joined #openstack-cinder18:10
nikeshmthanks18:10
*** e0ne has joined #openstack-cinder18:12
*** diogogmt has joined #openstack-cinder18:13
*** catinthe_ has joined #openstack-cinder18:13
*** harlowja has joined #openstack-cinder18:14
*** catintheroof has quit IRC18:15
*** catintheroof has joined #openstack-cinder18:15
*** uberjay has quit IRC18:16
*** catinthe_ has quit IRC18:18
*** victorhe_ has joined #openstack-cinder18:18
*** Administrator_ has quit IRC18:32
*** Administrator_ has joined #openstack-cinder18:32
*** tpsilva has quit IRC18:35
*** alonmarx has joined #openstack-cinder18:35
*** e0ne has quit IRC18:36
*** kaisers_ has joined #openstack-cinder18:40
*** uberjay has joined #openstack-cinder18:40
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Reject thaw call when backend is failed over  https://review.openstack.org/39634318:44
*** jordanP has quit IRC18:48
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: ibm_storage driver: Added fczm decorators  https://review.openstack.org/39465518:49
*** zhonghua-lee has quit IRC18:51
*** zhonghua-lee has joined #openstack-cinder18:51
*** chris_morrell has quit IRC18:52
*** chris_morrell has joined #openstack-cinder18:52
*** chlong has joined #openstack-cinder18:53
*** gouthamr has quit IRC18:57
*** gouthamr has joined #openstack-cinder18:59
*** nherciu has joined #openstack-cinder19:00
*** jwcroppe has quit IRC19:00
*** jgregor has joined #openstack-cinder19:03
*** slayer has quit IRC19:04
*** jgregor_ has joined #openstack-cinder19:04
*** slayer has joined #openstack-cinder19:05
*** jgregor__ has joined #openstack-cinder19:05
*** kaisers_ has quit IRC19:07
*** jgregor has quit IRC19:07
*** slayer has quit IRC19:08
*** jgregor_ has quit IRC19:08
*** ccesario has quit IRC19:11
*** amoralej is now known as amoralej|off19:12
*** jwcroppe has joined #openstack-cinder19:15
*** Lee1092 has quit IRC19:16
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Reject thaw call when backend is failed over  https://review.openstack.org/39634319:19
smcginnisAnyone looked into this? https://bugs.launchpad.net/cinder/+bug/164211119:21
openstackLaunchpad bug 1642111 in Cinder "create lvm volume from snapshot fails with "device-mapper: reload ioctl on (252:4) failed: Invalid argument"" [Undecided,Confirmed]19:21
*** slayer has joined #openstack-cinder19:21
smcginnisSomething change in lvchange on Xenial?19:22
smcginnisHmm, https://access.redhat.com/solutions/2124019:22
*** chris_morrell has quit IRC19:25
*** jamielennox|away is now known as jamielennox19:26
*** lpetrut has joined #openstack-cinder19:29
*** ChubYann has joined #openstack-cinder19:32
smcginnishemna: You were fighting some xenial issues, right? Seen anything with the lvchange command with device-mapper?19:34
*** lpetrut has quit IRC19:36
*** tongli has quit IRC19:40
*** khushbu has joined #openstack-cinder19:40
openstackgerritErlon R. Cruz proposed openstack/cinder: Gate migration tests: Add Cinder tempest hook  https://review.openstack.org/38173619:46
*** e0ne has joined #openstack-cinder19:49
*** e0ne has quit IRC19:50
*** pcaruana has quit IRC19:51
*** e0ne has joined #openstack-cinder19:52
*** tpsilva has joined #openstack-cinder19:54
*** e0ne has quit IRC19:54
*** diablo_rojo_phon has joined #openstack-cinder19:56
hemnaI had problems with vagrant and xenial due to the change in ethernet naming19:58
*** diablo_rojo has quit IRC19:59
hemnaare we seeing this in the lvm code in cinder ?19:59
smcginnishemna: Yep. Lot's of gate failures popping up.20:00
hemna:(20:00
*** e0ne has joined #openstack-cinder20:01
hemnaheading to lunch...bbaib20:01
*** alonma has joined #openstack-cinder20:04
*** alonma has quit IRC20:09
*** khushbu has quit IRC20:12
*** e0ne has quit IRC20:12
*** kaisers_ has joined #openstack-cinder20:12
*** porrua has quit IRC20:16
*** kaisers_ has quit IRC20:17
*** e0ne has joined #openstack-cinder20:20
*** kaisers_ has joined #openstack-cinder20:30
*** chenying has quit IRC20:30
*** jwcroppe has quit IRC20:30
*** chenying has joined #openstack-cinder20:30
*** jwcroppe has joined #openstack-cinder20:32
*** alonma has joined #openstack-cinder20:36
*** alonma has quit IRC20:40
*** zhonghua-lee has quit IRC20:40
*** porrua has joined #openstack-cinder20:40
*** zhonghua-lee has joined #openstack-cinder20:41
mriedemi'm having a hell of a time sorting out what a boot from volume test in tempest is failing with NFS backed cinder20:42
mriedemthe test fails when creating the volume,20:43
mriedemthe volume goes to ERROR state20:43
mriedemand it's trying to create the volume from a given image20:43
smcginnismriedem: Just a plain old volume create on NFS?20:43
mriedemthe POST request to create the volume is req-1614c0df-7216-4683-bf79-2180582b54cb in http://logs.openstack.org/87/395887/4/check/gate-tempest-dsvm-full-devstack-plugin-nfs-nv/817e301/logs/screen-c-vol.txt.gz20:43
mriedemsmcginnis: yes20:43
mriedemthere are clone errors all over c-vol but they aren't for that same request id20:44
smcginnisgit rm cinder/volume/drivers/nfs.py20:44
mriedemlooks like the volume create gets rescheduled 3 times and then fails, for whatever reason20:44
mriedemand there are stacktraces like this http://logs.openstack.org/87/395887/4/check/gate-tempest-dsvm-full-devstack-plugin-nfs-nv/817e301/logs/screen-c-vol.txt.gz#_2016-11-13_15_37_51_62220:45
mriedembut i thought a clone was when you create a new volume from an existing volume20:45
mriedemin this case it should just be a new volume created from a backing image20:45
smcginnismriedem: Ah. Image caching has been enabled by default now. So it will try to create a volume of the image, then just clone the volume on subsequent requests so it doesn't need to download again.20:46
mriedemso basically, image cache in cinder can't be used with NFS20:46
mriedemb/c it triggers the clone which isn't implemented20:46
smcginnismriedem: Yep, that appears to be the case. I just checked out the nfs code.20:47
SwansonAh ha! Image caching is on by default now?20:47
smcginnisSwanson: Yep. It's all patrickeast's fault.20:47
SwansonThis why I get an extra volume on first create?20:47
smcginnisSwanson: That'd do it.20:47
mriedemshit balls20:47
mriedemok, well that does it20:48
mriedemi was really wondering why this create volume part of tempest was failing on a clone20:48
smcginnismriedem: Odd though, I thought the behavior was supposed to be to fall back if it couldn't get the clone.20:48
mriedemwhen clone has nothing to do with the test20:48
mriedemsmcginnis: _create_image_cache_volume_entry handles CinderException, not NotImplementedError20:50
mriedemotherwise it would just log a warning and move on20:50
*** timcl has quit IRC20:50
mriedemshouldn't that code check if the volume driver is an instance of CloneableImageVD ?20:51
mriedemwell let me report a cinder bug first20:52
Swansonjgriffith, Per our convo the other day image caching is on by default now. That would be where my extra volume came from. Doodle doodle dee wubba wubba wubba20:52
mriedembtw, is the image caching on by default a cinder default option thing or a devstack thing?20:53
smcginnismriedem: devstack20:53
mriedem# Enable the image volume cache for this backend. (boolean value) #image_volume_cache_enabled = false20:53
mriedemfun....20:54
mriedemhttps://github.com/openstack-dev/devstack/commit/dddb2c7b5f85688de9c9b92f025df25d2f2d301620:54
*** e0ne has quit IRC20:56
*** rcernin has joined #openstack-cinder20:58
mriedemalright here you go https://bugs.launchpad.net/cinder/+bug/164239421:02
openstackLaunchpad bug 1642394 in Cinder "NFS: Creating multiple volumes from the same image fails when image caching is turned on by default" [Undecided,New]21:02
smcginnismriedem: Thanks21:02
mriedembtw, the only reason i care about getting the nfs job working is because of some libvirt nfs volume driver refactoring going on in nova21:04
mriedemi would like to see the nfs job passing on nova changes like that in the experimental queue21:05
mriedemif you guys just want to drop nfs from cinder and nova, sure, let's go nuts21:05
mriedemthis is also wrong btw https://wiki.openstack.org/wiki/CinderSupportMatrix21:05
smcginnismriedem: If we can't figure out a fix quickly I'll put up a revert for that devstack change.21:05
mriedemsays that the nfs backend has supported clone since grizzly21:05
mriedemsmcginnis: you don't really need to revert the devstack change, it's fine for lvm21:05
smcginnismriedem: That's one of the weird things here. I thought it did.21:05
mriedemi can workaround this in the devstack-plugin-nfs repo21:06
smcginnismriedem: Well, if we can't handle this for all backends, I kind of think we shouldn't enable it by default.21:06
smcginnismriedem: But we'll see.21:06
mriedembtw, clone is listed in defcore21:07
smcginnisAnd snapshots...21:07
mriedemhttps://github.com/openstack/defcore/blob/master/2016.08.json#L263621:07
mriedemyeah, so if you're running an NFS backed cloud i guess you're not refstack compliant21:07
*** alonma has joined #openstack-cinder21:07
smcginnisRight now, that is correct. We've been working on it, but starting to have some doubts.21:08
mriedemhttps://wiki.openstack.org/wiki/CinderSupportMatrix#OpenStack_Block_Storage_.28aka_Cinder.29_Drivers updated btw21:08
*** porrua has quit IRC21:09
*** mvk has joined #openstack-cinder21:11
*** alonma has quit IRC21:12
* patrickeast catches up on scrollback21:15
patrickeastsmcginnis: mriedem: oh weird, yea clone was supposed to be something *all* drivers supported21:16
eharneythat this is in progress for NFS shouldn't really be news21:16
eharney*cough* block device driver *cough*21:17
patrickeastheh good point21:17
openstackgerritSean McGinnis proposed openstack/cinder: Handle NotImplementedError for image cache cloning  https://review.openstack.org/39858121:17
smcginniseharney: I could have sworn it supported clone but not snapshot.21:18
patrickeast^ thats what i thought21:18
eharneysmcginnis: if it supports clone of attached volumes, it's corrupting data21:18
patrickeastclone should be easy... just cp21:18
smcginniseharney: But non-attached should be fine.21:18
smcginniseharney: Just can't do the attached-so-lets-snap-and-clone-from-that case.21:19
smcginnisWell, quick stab at working around it. ^^21:19
smcginnisNot set up to actually test that at the moment, so if someone else is, that would be grand.21:19
smcginnisWould also be grand if we had an nfs test in the gate.21:19
eharneyit's in "check experimental"21:20
smcginniseharney: Oh, because we can't do the snap tests.21:20
mriedemeharney: that's what i'm working on21:20
smcginnisI'm kind of paying attention I guess.21:20
mriedemeharney: https://review.openstack.org/#/c/395887/21:20
mriedem-1 on the PTLs patch21:21
mriedemso sweet21:21
eharneyi looked at the block device driver... it does do clones in a way that corrupt data from what i can tell.  at least it can run tempest...21:21
smcginnismriedem: :)21:21
smcginnismriedem: Tests schmests21:22
*** nherciu has quit IRC21:28
*** catinthe_ has joined #openstack-cinder21:32
*** cdelatte has quit IRC21:34
*** alonmarx has quit IRC21:35
*** catintheroof has quit IRC21:36
*** gouthamr has quit IRC21:37
*** alonma has joined #openstack-cinder21:39
*** jordanP has joined #openstack-cinder21:39
*** alonmarx has joined #openstack-cinder21:41
*** alonma has quit IRC21:43
*** zengyingzhe_ has quit IRC21:43
*** kaisers_ has quit IRC21:43
*** zengyingzhe_ has joined #openstack-cinder21:43
*** jordanP has quit IRC21:46
*** jwcroppe has quit IRC21:50
*** catintheroof has joined #openstack-cinder21:51
*** catinthe_ has quit IRC21:51
*** jgregor__ has quit IRC22:02
*** xyang has quit IRC22:04
*** jwcroppe has joined #openstack-cinder22:07
*** xyang has joined #openstack-cinder22:10
*** alonma has joined #openstack-cinder22:10
*** lifeless has quit IRC22:11
*** lifeless has joined #openstack-cinder22:12
*** alonma has quit IRC22:15
*** tpsilva has quit IRC22:25
*** alonmarx_ has joined #openstack-cinder22:28
*** alonmarx has quit IRC22:32
*** alonma has joined #openstack-cinder22:32
*** xyang has quit IRC22:33
*** slayer has quit IRC22:34
*** alonma has quit IRC22:36
*** mriedem has quit IRC22:41
*** jessegler has joined #openstack-cinder22:42
*** xyang has joined #openstack-cinder22:44
*** catintheroof has quit IRC22:54
*** jessegler has quit IRC22:57
*** chlong has quit IRC22:59
*** victorhe_ has quit IRC23:01
*** alonma has joined #openstack-cinder23:03
*** dave-mccowan has quit IRC23:07
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Storage assisted volume migration.  https://review.openstack.org/39786223:08
*** alonma has quit IRC23:08
*** Cibo has quit IRC23:08
*** jwcroppe has quit IRC23:09
*** jwcroppe has joined #openstack-cinder23:10
*** jgravel has joined #openstack-cinder23:12
*** aimeeu has left #openstack-cinder23:12
*** jwcroppe has quit IRC23:14
*** xyang1 has quit IRC23:15
*** xyang has quit IRC23:16
*** ducttape_ has quit IRC23:16
*** chlong has joined #openstack-cinder23:17
*** edmondsw has quit IRC23:17
*** khushbu_ has joined #openstack-cinder23:20
*** diogogmt has quit IRC23:25
*** jwcroppe has joined #openstack-cinder23:26
*** chlong has quit IRC23:27
*** jwcroppe has quit IRC23:31
*** alonma has joined #openstack-cinder23:35
*** jungleboyj has quit IRC23:35
*** cknight has joined #openstack-cinder23:37
*** zhugaoxiao has joined #openstack-cinder23:38
*** alonma has quit IRC23:39
*** khamtamtun has joined #openstack-cinder23:39
*** Administrator_ has quit IRC23:40
*** Julien-zte has quit IRC23:41
*** khamtamtun has quit IRC23:42
*** khushbu_ has quit IRC23:43
*** kaisers_ has joined #openstack-cinder23:44
openstackgerritMerged openstack/cinder: Add VolumeAttachStatus Enum  https://review.openstack.org/29088523:46
*** kaisers_ has quit IRC23:48
*** cdelatte has joined #openstack-cinder23:49
*** david-lyle_ has joined #openstack-cinder23:50
*** erhudy has quit IRC23:51
*** david-lyle has quit IRC23:51
*** rcernin has quit IRC23:52
*** dustins has quit IRC23:55

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!