Monday, 2017-01-30

*** raunak has quit IRC00:02
*** yangyapeng has joined #openstack-cinder00:04
*** kaisers has quit IRC00:06
*** yangyapeng has quit IRC00:09
*** Fdaisuke has quit IRC00:09
*** s-shiono_ has joined #openstack-cinder00:11
*** crushil has joined #openstack-cinder00:13
*** s-shiono has quit IRC00:14
*** stakeda has joined #openstack-cinder00:20
*** reduxio has joined #openstack-cinder00:22
*** reduxio has quit IRC00:26
*** crushil has quit IRC00:33
*** diablo_rojo has joined #openstack-cinder00:44
*** esker has joined #openstack-cinder00:49
*** kaisers has joined #openstack-cinder01:02
*** yangyapeng has joined #openstack-cinder01:05
*** knomura has joined #openstack-cinder01:06
*** ducttape_ has quit IRC01:09
*** yangyapeng has quit IRC01:10
*** knomura has quit IRC01:10
*** diablo_rojo_ has joined #openstack-cinder01:12
*** diablo_rojo__ has joined #openstack-cinder01:13
*** diablo_rojo__ has quit IRC01:13
*** diablo_rojo_ has quit IRC01:13
*** crushil has joined #openstack-cinder01:16
*** ducttape_ has joined #openstack-cinder01:17
*** esker has quit IRC01:18
openstackgerritSofia Enriquez proposed openstack/cinder: RBD: manage/unmanage snapshot  https://review.openstack.org/35804301:22
*** raunak has joined #openstack-cinder01:31
*** ducttape_ has quit IRC01:39
*** ducttape_ has joined #openstack-cinder01:39
*** alonma has joined #openstack-cinder01:46
*** alonma has quit IRC01:51
*** alonma has joined #openstack-cinder01:52
*** alonma has quit IRC01:57
*** Julien-zte has joined #openstack-cinder01:57
*** ducttape_ has quit IRC02:02
*** ducttape_ has joined #openstack-cinder02:03
*** yangyapeng has joined #openstack-cinder02:06
*** ducttape_ has quit IRC02:06
*** _ducttape_ has joined #openstack-cinder02:06
*** crushil has quit IRC02:06
*** yangyapeng has quit IRC02:11
*** reduxio has joined #openstack-cinder02:24
*** _ducttape_ has quit IRC02:24
*** reduxio has quit IRC02:28
*** kaisers has quit IRC02:35
*** kaisers has joined #openstack-cinder03:00
*** yangyapeng has joined #openstack-cinder03:07
*** rcernin has joined #openstack-cinder03:11
*** yangyapeng has quit IRC03:11
*** kaisers has quit IRC03:13
*** rcernin has quit IRC03:13
*** rcernin has joined #openstack-cinder03:14
*** Julien-zte has quit IRC03:17
*** rcernin has quit IRC03:25
*** nicolasbock has quit IRC03:34
*** diablo_rojo has quit IRC03:44
*** crushil has joined #openstack-cinder03:48
*** links has joined #openstack-cinder03:56
*** links has quit IRC03:57
*** dinob has joined #openstack-cinder04:00
*** raunak has quit IRC04:04
*** kwathore_ has quit IRC04:04
*** crushil has quit IRC04:05
*** alonma has joined #openstack-cinder04:05
*** alonma has quit IRC04:09
*** gouthamr has quit IRC04:09
*** kaisers has joined #openstack-cinder04:11
*** kwathore_ has joined #openstack-cinder04:15
*** dinob has quit IRC04:17
*** ducttape_ has joined #openstack-cinder04:25
*** reduxio has joined #openstack-cinder04:25
*** swamireddy has joined #openstack-cinder04:26
*** mdnadeem has joined #openstack-cinder04:29
*** ducttape_ has quit IRC04:30
*** reduxio has quit IRC04:30
*** maestropandy has joined #openstack-cinder04:31
*** stakeda has quit IRC04:36
*** pdeore has joined #openstack-cinder04:37
*** swamireddy has left #openstack-cinder04:45
*** faizy has joined #openstack-cinder04:46
*** kaisers has quit IRC04:46
*** faizy_ has joined #openstack-cinder04:53
*** faizy has quit IRC04:54
*** sudipto has joined #openstack-cinder04:59
*** sudipto_ has joined #openstack-cinder04:59
*** yangyapeng has joined #openstack-cinder05:08
*** knomura has joined #openstack-cinder05:12
*** yangyapeng has quit IRC05:13
*** knomura has quit IRC05:17
*** mtanino has quit IRC05:19
*** kaisers has joined #openstack-cinder05:21
*** psachin has joined #openstack-cinder05:28
openstackgerritMerged openstack/cinder: Rebrand and move EQLX driver to dell_emc folder  https://review.openstack.org/41556505:32
*** swamireddy has joined #openstack-cinder05:43
*** alonmarx has joined #openstack-cinder05:46
*** Dinesh_Bhor has joined #openstack-cinder05:47
*** Jack_Iv has joined #openstack-cinder05:55
*** adisky_ has joined #openstack-cinder05:59
*** swamireddy has left #openstack-cinder06:00
*** swamireddy_ has joined #openstack-cinder06:01
*** Jack_I has joined #openstack-cinder06:02
*** yangyapeng has joined #openstack-cinder06:09
*** mtanino has joined #openstack-cinder06:13
*** yangyapeng has quit IRC06:13
*** alonma has joined #openstack-cinder06:14
*** lpetrut has joined #openstack-cinder06:14
*** lpetrut has quit IRC06:17
*** nkrinner_afk is now known as nkrinner06:17
*** alonma has quit IRC06:18
*** lpetrut has joined #openstack-cinder06:19
*** mtanino has quit IRC06:19
*** pgadiya has joined #openstack-cinder06:20
*** ducttape_ has joined #openstack-cinder06:26
*** reduxio has joined #openstack-cinder06:27
*** Jack_Iv has quit IRC06:30
*** ducttape_ has quit IRC06:31
*** reduxio has quit IRC06:32
*** shubham0d has joined #openstack-cinder06:38
*** alonma has joined #openstack-cinder06:43
*** alonma has quit IRC06:47
*** lpetrut has quit IRC06:58
*** alonma has joined #openstack-cinder07:04
*** alonma has quit IRC07:09
*** Lee1092 has joined #openstack-cinder07:09
*** yangyapeng has joined #openstack-cinder07:10
*** rcernin has joined #openstack-cinder07:11
*** rcernin has quit IRC07:12
*** rcernin has joined #openstack-cinder07:12
*** yangyapeng has quit IRC07:14
*** alonma has joined #openstack-cinder07:15
*** alonma has quit IRC07:20
*** mkoderer has joined #openstack-cinder07:26
*** alonma has joined #openstack-cinder07:27
*** alonma has quit IRC07:31
*** reduxio has joined #openstack-cinder07:34
*** arne_r has joined #openstack-cinder07:35
*** lennyb has joined #openstack-cinder07:37
*** lennyb_ has joined #openstack-cinder07:37
*** lennyb_ has quit IRC07:37
*** alonma has joined #openstack-cinder07:38
*** alonma has quit IRC07:43
*** shubham0d has quit IRC07:46
*** s-shiono_ has quit IRC07:48
*** alonma has joined #openstack-cinder07:49
*** tesseract has joined #openstack-cinder07:50
*** alonma has quit IRC07:54
*** ChubYann has quit IRC08:00
*** alonma has joined #openstack-cinder08:01
*** alonma has quit IRC08:05
*** yangyapeng has joined #openstack-cinder08:11
*** reduxio has quit IRC08:12
*** alonma has joined #openstack-cinder08:12
*** yangyapeng has quit IRC08:15
*** pcaruana has joined #openstack-cinder08:15
*** alonma has quit IRC08:16
*** alonma has joined #openstack-cinder08:18
*** Cibo has quit IRC08:20
*** Cibo has joined #openstack-cinder08:20
*** eantyshev has left #openstack-cinder08:21
*** alonma has quit IRC08:22
*** ducttape_ has joined #openstack-cinder08:27
*** alonma has joined #openstack-cinder08:29
*** ducttape_ has quit IRC08:32
*** maestropandy has quit IRC08:33
*** alonma has quit IRC08:33
*** alonma has joined #openstack-cinder08:40
*** alonma has quit IRC08:45
*** alonma has joined #openstack-cinder08:47
*** Cibo_ has quit IRC08:48
*** lpetrut has joined #openstack-cinder08:50
*** alonmarx has quit IRC08:52
*** yangyapeng has joined #openstack-cinder08:59
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-cinder09:00
*** faizy_ has quit IRC09:01
*** e0ne has joined #openstack-cinder09:01
*** stakeda has joined #openstack-cinder09:03
*** pgadiya has quit IRC09:03
*** pgadiya has joined #openstack-cinder09:05
*** asselin__ has joined #openstack-cinder09:08
*** sudipto_ has quit IRC09:10
*** sudipto has quit IRC09:11
*** sudipto_ has joined #openstack-cinder09:11
*** asselin_ has quit IRC09:12
*** reduxio has joined #openstack-cinder09:12
*** sudswas__ has joined #openstack-cinder09:13
*** reduxio has quit IRC09:13
*** reduxio has joined #openstack-cinder09:13
*** jschwarz is now known as jschwarz|lunch09:27
*** JoseMello has joined #openstack-cinder09:28
*** pgadiya has quit IRC09:29
*** rcernin has quit IRC09:31
*** rcernin has joined #openstack-cinder09:33
*** pgadiya has joined #openstack-cinder09:45
*** Lee1092 has quit IRC09:46
*** jistr has quit IRC09:49
*** jistr has joined #openstack-cinder09:51
*** Jack_Iv has joined #openstack-cinder09:55
*** maestropandy has joined #openstack-cinder09:56
*** Jack_Iv has quit IRC09:57
openstackgerritGorka Eguileor proposed openstack/cinder: Stop get_notifier mock in tests cleanup  https://review.openstack.org/42669209:59
*** Jack_Iv has joined #openstack-cinder10:00
*** Jack_Iv has quit IRC10:00
*** maestropandy has quit IRC10:00
*** stakeda has quit IRC10:00
*** Jack_Iv has joined #openstack-cinder10:00
*** maestropandy has joined #openstack-cinder10:05
*** maestropandy has left #openstack-cinder10:05
*** thiagolib has joined #openstack-cinder10:07
*** lkuchlan has joined #openstack-cinder10:13
*** alonma has quit IRC10:16
*** mkoderer has quit IRC10:16
*** alonma has joined #openstack-cinder10:17
*** alonma has quit IRC10:21
*** pjm6_ is now known as pjm610:25
*** alyson_ has joined #openstack-cinder10:26
*** alonma has joined #openstack-cinder10:28
*** ducttape_ has joined #openstack-cinder10:28
*** mvk has quit IRC10:28
*** alonma has quit IRC10:32
*** ducttape_ has quit IRC10:33
*** pdeore has quit IRC10:38
*** alonma has joined #openstack-cinder10:39
*** alonma has quit IRC10:44
*** Jack_Iv has quit IRC10:44
*** shubham0d has joined #openstack-cinder10:44
*** alonma has joined #openstack-cinder10:46
*** dalgaaf has quit IRC10:49
*** mvk has joined #openstack-cinder10:57
*** dalgaaf has joined #openstack-cinder10:59
*** pdeore has joined #openstack-cinder11:02
*** Jack_Iv has joined #openstack-cinder11:05
*** Jack_Iv has quit IRC11:08
*** Jack_Iv has joined #openstack-cinder11:09
*** Jack_Iv has quit IRC11:13
*** kwathore_ has quit IRC11:16
*** kwathore has joined #openstack-cinder11:17
*** shubham0d has quit IRC11:21
*** jordanP has joined #openstack-cinder11:23
*** Jack_Iv has joined #openstack-cinder11:24
*** dims has joined #openstack-cinder11:26
*** reduxio has quit IRC11:28
openstackgerritLucian Petrut proposed openstack/cinder: Image utils: fix fetching available space on Windows  https://review.openstack.org/42671911:28
openstackgerritLucian Petrut proposed openstack/cinder: Image utils: fix fetching available space on Windows  https://review.openstack.org/42671911:30
openstackgerritGorka Eguileor proposed openstack/cinder: Short-circuit notifications when not enabled  https://review.openstack.org/42654711:32
*** psachin has quit IRC11:32
*** ducttape_ has joined #openstack-cinder11:35
*** ducttape_ has quit IRC11:52
*** nicolasbock has joined #openstack-cinder11:52
*** pdeore has quit IRC11:54
*** porrua has joined #openstack-cinder11:58
*** reduxio has joined #openstack-cinder11:59
openstackgerritGorka Eguileor proposed openstack/cinder: Add create volume from image policy  https://review.openstack.org/42672811:59
*** bkopilov has quit IRC12:02
*** reduxio has quit IRC12:10
*** Jack_Iv has quit IRC12:14
*** reduxio has joined #openstack-cinder12:21
openstackgerritMerged openstack/python-brick-cinderclient-ext: Remove unnecessary 'service_type' decorator  https://review.openstack.org/42633212:22
*** alonma has quit IRC12:23
*** alonma has joined #openstack-cinder12:24
*** faiz89 has joined #openstack-cinder12:24
*** catintheroof has joined #openstack-cinder12:25
*** alonma has quit IRC12:28
*** Jack_Iv has joined #openstack-cinder12:29
*** alonma has joined #openstack-cinder12:30
*** alonma has quit IRC12:34
*** Jack_Iv has quit IRC12:35
*** Jack_Iv has joined #openstack-cinder12:35
*** Jack_Iv has quit IRC12:35
*** Jack_Iv has joined #openstack-cinder12:36
*** Jack_Iv has quit IRC12:38
*** Jack_Iv has joined #openstack-cinder12:39
*** Jack_Iv_ has joined #openstack-cinder12:40
*** sdague has joined #openstack-cinder12:40
*** adrianofr has joined #openstack-cinder12:40
*** bkopilov has joined #openstack-cinder12:41
*** alonma has joined #openstack-cinder12:41
*** Jack_Iv has quit IRC12:43
*** Jack_Iv_ has quit IRC12:45
*** alonma has quit IRC12:45
*** Jack_Iv has joined #openstack-cinder12:46
*** Jack_Iv has quit IRC12:46
*** Jack_Iv has joined #openstack-cinder12:48
*** Jack_Iv has quit IRC12:52
*** reduxio has quit IRC12:53
*** Jack_Iv has joined #openstack-cinder12:54
*** alonma has joined #openstack-cinder12:57
*** dave-mccowan has joined #openstack-cinder13:00
*** alonma has quit IRC13:01
*** alonma has joined #openstack-cinder13:02
*** ducttape_ has joined #openstack-cinder13:04
*** Jack_Iv has quit IRC13:05
*** Jack_Iv has joined #openstack-cinder13:09
*** esker has joined #openstack-cinder13:10
openstackgerritPooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants  https://review.openstack.org/42674713:14
*** esker has quit IRC13:14
*** Jack_Iv has quit IRC13:14
*** eharney has quit IRC13:17
openstackgerritPooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants  https://review.openstack.org/42674813:17
openstackgerritPooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants  https://review.openstack.org/42674913:19
*** Jack_Iv has joined #openstack-cinder13:20
*** mdnadeem has quit IRC13:21
*** kashyap has left #openstack-cinder13:21
openstackgerritPooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants  https://review.openstack.org/42675013:22
*** Jack_Iv has quit IRC13:22
*** pgadiya has quit IRC13:24
*** lkuchlan has quit IRC13:31
dulekerlon: ping13:33
erlondulek: hey13:33
dulekerlon: I'm trying to test NFS snapshots and I've run into a problem with A/A setup there.13:33
erlondulek: which problem??13:34
dulekerlon: Basically my second node, that connects to the NFS remotely, uses same mount directory as on the primary node when issuing qemu-img command.13:34
dulekerlon: Share is mounted in different directory on that node, so it simply fails with file not found.13:35
*** JoseMello has quit IRC13:35
dulekerlon: I wonder why it tries to use wrong dir there. Is it saving it in provider_info, or somewhere?13:35
erlondulek: in a different directory? what is the cinder_data folder?13:36
erlondulek: for devstack for example is /opt/stack/data/cinder13:36
dulekhome/vagrant/data/cinder13:37
dulek(setup specific thing)13:37
erlondulek: is this the same for both nodes?13:37
*** dustins|out is now known as dustins13:37
dulekOn the primary the share is mounted in ~/data/cinder/mnt/896fb15da6036b68a917322e72ebfe5713:37
dulekOn the secondary on ~/data/cinder/mnt/896fb15da6036b68a917322e72ebfe5713:37
dulekBut when snapshotting (and snapshotting only), secondary uses primary's dir.13:38
*** mkoderer has joined #openstack-cinder13:38
dulekWhich of course doesn't exist on secondary.13:38
dulekOption itself is set to the same on both.13:38
erlondulek: insnt that the same?13:38
dulekOh, right, sorry. On secondary there's different uuid.13:39
dulek~/data/cinder/mnt/fccb9d97171126dd2b9e165d7d3cc359 - that's the one13:39
erlondulek: hmm, this uuid is a hash of the provider location, so, can you check the provider location?13:40
*** ducttape_ has quit IRC13:41
dulekerlon: Oh!13:41
dulekerlon: This would explain it, thank you for the pointer. :)13:41
erlondulek: are you using different names for the server?13:42
dulekerlon: Hm, what do you mean by server name?13:42
duleklocalhost:/srv/nfs113:42
openstackgerritTzur Eliyahu proposed openstack/cinder: clean up replication volume on an xiv backend  https://review.openstack.org/42676313:42
erlonfor example nfs-server.local:/share and in the other you use nfs-server:/share13:42
dulekerlon: Yeah, this is true - I use localhost on the one where my server is located, and IP on the other.13:43
erlondulek: hmm, got it, 1 you put localhost and the other the IP13:43
erlondulek: ok, that explains13:43
dulekerlon: Awesome, this will solve it. Thank you! :)13:43
erlondulek: great! Sure, no problem!13:44
openstackgerritTzur Eliyahu proposed openstack/cinder: clean up replication volume on an xiv backend  https://review.openstack.org/42676313:47
*** faiz89 has quit IRC13:50
smcginniserlon: Is this something we should document somewhere?13:51
*** tlian has joined #openstack-cinder13:52
erlonsmcginnis: hmm, may be not sure where would be the best place. I need to update the NFS documentation including the snapshots, may be its a good addition to add a AA section13:52
smcginniserlon: That would make sense to me. It's kind of an AA only issue.13:53
*** reduxio has joined #openstack-cinder13:53
erlonsmcginnis: yeap13:54
*** jdurgin has quit IRC13:54
erlonsmcginnis: do you know where the bugs for infra are created? the ones created with the DocImpact tag13:54
erlonsmcginnis: talking in that, we should have a AA documentation for this release, do we already have one?13:55
smcginniserlon: Those actually get filed against Cinder now, with the idea that that will remind us we needed to do more work and we can then reassign it over to openstack-docs to use to commit our changes. Tracking all the way through.13:55
smcginniserlon: Good point. No, I don't think I've seen anything yet.13:55
smcginnisWe should make sure we at least have something written up for that.13:55
dulekerlon: Okay, so now it won't find snapshot when deleting it. Is snapshot name a hash from something else?13:56
erlonsmcginnis: yeap, may be we should add it this release under Cinder, and when we officially suppot, move to openstack-docs13:56
smcginnisgeguileo: Have you written up anything for the AA stuff?13:57
smcginniserlon: That may be a good way to do it. Just have it in our devref until it's had more time and testing, then move it over to the official docs.o.o once we're ready to say it's available for production.13:58
erlondulek: no,m the snapshot name is  volume-is.snapshot-id13:58
erlongeguileo: if you can gather what you have written so far and send a patch, I add the NFS specific parts, I haven't tested the NFS with AA though13:59
erlondulek: please keep me up to date with the problems you find14:00
erlondulek: brb14:00
*** jschwarz|lunch is now known as jschwarz14:01
dulekerlon: Problems with delete happen only in case of running concurrent requests. Let my try with DLM configured, it's most probably lock issue. :)14:01
* dulek is glad that he runs only into problems we've predicted. :)14:01
smcginnis:)14:01
*** xyang1 has joined #openstack-cinder14:08
*** yumiriam has joined #openstack-cinder14:09
*** jdurgin has joined #openstack-cinder14:12
*** clenimar has quit IRC14:12
*** gcb has joined #openstack-cinder14:13
*** shubham0d has joined #openstack-cinder14:15
geguileosmcginnis: Not yet, but what we have is the A/A framework, but no driver currently supports it14:15
smcginnisgeguileo: True.14:15
*** clenimar has joined #openstack-cinder14:16
geguileosmcginnis: I have to do a writeup for the devref on A/A so driver maintainers can start workinig on it14:16
smcginnisgeguileo: We should probably have something in the devref for at least the driver developers explaining what to do.14:16
smcginnisgeguileo: Hah, I guess we're on the same page then. ;)14:16
geguileoyes we are!  :-)14:17
smcginnisgeguileo: Thanks for working on that.14:17
geguileonp14:17
*** gouthamr has joined #openstack-cinder14:19
*** ducttape_ has joined #openstack-cinder14:19
*** liverpooler has joined #openstack-cinder14:22
*** ducttape_ has quit IRC14:26
dulekerlon: Oh, snapshots are rebased when one in the middle of the chain gets deleted?14:30
dulekerlon: I've run into situation when one was being rebased on another, while this another got deleted.14:30
dulekerlon: I'm not sure how, as I see there's an exclusive lock on delete_snapshot…14:31
openstackgerritRohan Arora proposed openstack/cinder: Nosec verify=False until driver writer review  https://review.openstack.org/42638514:42
*** jungleboyj has quit IRC14:46
dulekerlon: Ooooh, the external argument is set to False, that's why it's failing…14:47
*** jungleboyj has joined #openstack-cinder14:47
*** rcernin has quit IRC14:48
*** shubham0d has quit IRC14:49
*** ducttape_ has joined #openstack-cinder14:52
*** reduxio has quit IRC14:53
*** knomura has joined #openstack-cinder14:57
dulekerlon: Okay, now I see - patch should be using coordination.synchronized instead of locked_volume_id_operation. I'll try to fix this up and see if it helps.14:59
*** rcernin has joined #openstack-cinder15:00
*** crushil has joined #openstack-cinder15:01
*** mvk has quit IRC15:02
*** bkopilov has quit IRC15:05
*** nkrinner is now known as nkrinner_afk15:08
*** bkopilov has joined #openstack-cinder15:11
*** reduxio has joined #openstack-cinder15:14
*** Jack_Iv has joined #openstack-cinder15:18
*** mvk has joined #openstack-cinder15:18
*** reduxio has quit IRC15:19
*** mriedem has joined #openstack-cinder15:19
*** kfarr has joined #openstack-cinder15:21
*** stvnoyes has quit IRC15:22
*** stvnoyes has joined #openstack-cinder15:23
openstackgerritTzur Eliyahu proposed openstack/cinder: clean up replication volume on an xiv backend  https://review.openstack.org/42676315:23
*** jdurgin has quit IRC15:23
*** knomura has quit IRC15:28
dulekerlon: FYI - this helps in A/A use case. I'll look into ways of solving this without affecting drivers that inherit from NfsDriver class.15:29
openstackgerritBéla Vancsics proposed openstack/cinder: Reduced the complexity of the main() method  https://review.openstack.org/25091415:31
openstackgerritMerged openstack/cinder: Fix double call to "qemu-img create"  https://review.openstack.org/42573915:32
*** kfarr has quit IRC15:33
openstackgerritMerged openstack/cinder: VMAX driver - removed incorrect six.text_type  https://review.openstack.org/42571315:33
*** rajinir has joined #openstack-cinder15:33
*** eharney has joined #openstack-cinder15:38
*** mvk has quit IRC15:40
*** alonmarx has joined #openstack-cinder15:40
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - sync sv issue  https://review.openstack.org/42681315:44
*** arne_r has quit IRC15:48
openstackgerritBéla Vancsics proposed openstack/cinder: Reduced the complexity of the main() method  https://review.openstack.org/25091415:52
*** mvk has joined #openstack-cinder15:53
*** reduxio has joined #openstack-cinder15:57
*** _ducttape_ has joined #openstack-cinder16:00
*** ducttape_ has quit IRC16:01
*** jdillaman has joined #openstack-cinder16:02
*** liverpooler has quit IRC16:06
*** diablo_rojo has joined #openstack-cinder16:06
*** liverpooler has joined #openstack-cinder16:07
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - widen locking around storage groups  https://review.openstack.org/42682416:12
erlondulek: yes the snapshot deletion have a rebase proccess, that can happen both in Cinder or in the compute node16:12
erlondulek: if the primary node fails while a snapshot is being deleted in nova (online) that can lead to errors in the clean up16:13
erlondulek: @coordination.synchronized('{self.driver_prefix}-{snapshot.volume.id}', external=True) ??16:16
erlondulek: if so, it could be fixed by inheriting from RemoteFSSnapDriverDistributed: https://github.com/openstack/cinder/blob/2d77a7a87d625f47deab4955298dad9d21a93f26/cinder/volume/drivers/remotefs.py#L1555-L158316:18
*** xinli has joined #openstack-cinder16:24
*** rcernin has quit IRC16:24
*** alonma has quit IRC16:26
*** alonma has joined #openstack-cinder16:26
*** archies has joined #openstack-cinder16:29
*** alonma has quit IRC16:31
*** e0ne has quit IRC16:37
*** alonma has joined #openstack-cinder16:37
*** knomura has joined #openstack-cinder16:39
*** _ducttape_ has quit IRC16:40
*** ducttape_ has joined #openstack-cinder16:40
*** alonmarx has quit IRC16:41
*** alonma has quit IRC16:42
*** jdurgin has joined #openstack-cinder16:43
*** reduxio has quit IRC16:44
*** reduxio has joined #openstack-cinder16:44
*** reduxio has quit IRC16:49
*** Jack_Iv has quit IRC16:49
*** reduxio has joined #openstack-cinder16:49
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - copy state fix  https://review.openstack.org/42684216:50
*** faiz89 has joined #openstack-cinder16:52
*** alonmarx has joined #openstack-cinder16:53
*** alonma has joined #openstack-cinder16:53
*** alonma has quit IRC16:57
*** tesseract has quit IRC16:58
*** kfarr has joined #openstack-cinder17:04
*** takedakn has joined #openstack-cinder17:04
*** takedakn has quit IRC17:05
*** gcb has quit IRC17:07
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - sync sv issue  https://review.openstack.org/42681317:08
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - widen locking around storage groups  https://review.openstack.org/42682417:10
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - copy state fix  https://review.openstack.org/42684217:13
*** knomura has quit IRC17:14
*** mtanino has joined #openstack-cinder17:14
*** jordanP has quit IRC17:16
*** alonma has joined #openstack-cinder17:18
*** jose-phillips has joined #openstack-cinder17:19
*** alonma has quit IRC17:22
*** mvk has quit IRC17:24
*** Apoorva has joined #openstack-cinder17:24
*** alonma has joined #openstack-cinder17:29
openstackgerritRohan Arora proposed openstack/cinder: Nosec verify=False until driver writer review  https://review.openstack.org/42638517:32
*** alonma has quit IRC17:34
*** Jack_Iv has joined #openstack-cinder17:40
*** crushil has quit IRC17:43
*** kfarr has quit IRC17:43
*** liverpooler has quit IRC17:52
*** liverpooler has joined #openstack-cinder17:53
*** newmember has quit IRC17:54
*** jungleboyj has quit IRC17:58
*** reduxio has quit IRC17:59
archies@smcginnis, thanks for review. My CI is passing now. please can you take a look at https://review.openstack.org/#/c/419335/18:00
*** harlowja has joined #openstack-cinder18:02
*** Jack_Iv has quit IRC18:06
*** mvk has joined #openstack-cinder18:07
*** jungleboyj has joined #openstack-cinder18:08
*** e0ne has joined #openstack-cinder18:12
SwansonAnyone? Buehler? https://review.openstack.org/#/c/425788/18:14
jungleboyjSwanson, *click*18:15
jgriffithSwanson I don't quite understand the logic18:15
*** sudipto_ has quit IRC18:15
*** sudswas__ has quit IRC18:15
jgriffithin dell_storagecenter_api.py18:15
jgriffithwell... I guess I don't understand the bug really18:16
jgriffithSwanson so you're not handling delete on volumes that DNE on the backend by default?18:17
jgriffithSwanson but your code says you do18:17
smcginnisHe lies.18:18
jgriffithAnyone?  Swanson?  Swanson?18:18
jgriffith@smcginnis well, we already knew that :)18:18
scottdapatrickeast: Would you please revisit this when you get a chance? https://review.openstack.org/#/c/424492/18:20
*** archies has quit IRC18:21
Swansonjgriffith, hello18:22
jgriffithSwanson well never mind, jungleboyj and scottda already gave their +2's so must be ok18:24
*** crushil has joined #openstack-cinder18:24
jgriffithand code is +A as well...18:24
* jgriffith moves along whistling18:24
Swansonjgriffith, what was the specific question? For delete I get a volume that is a mismash of stuff. So the provider_id that I used to look at is for the wrong backend. So I need to check that.18:25
jgriffithor you get None back from the find_volume call then try and access that as a blog in get_provider_id and based on what I see puke18:26
jgriffiths/blog/blob/18:26
jgriffithdoesn't matter, I'm probably just too dumb to understand it18:26
jgriffitheverybody else has upvoted it18:26
SwansonIt probably blows up.18:26
scottdayeah, we're probably too dumb to have seen the problem18:27
jungleboyjscottda, Thanks for saying it.18:27
* jungleboyj is out of practice so I probably did something stupid.18:27
*** kfarr has joined #openstack-cinder18:29
Swansonjgriffith, looking through it now...18:29
*** adisky_ has quit IRC18:29
*** johndperkins has joined #openstack-cinder18:29
Swansonjgriffith, I just return None from find_volume which means a None for provider_id which means I just return True.18:30
* jungleboyj will with hold +A's for now. :-)18:30
*** crushil has quit IRC18:30
*** crushil has joined #openstack-cinder18:31
SwansonUnder the theory that it was probably deleted by hand or something so just say "We deleted that. Yeah, that's the ticket..."18:32
jgriffithSwanson ahh, it "if not vol" catches it18:32
jgriffithand the Log.warning is not big deal with is say sNone18:32
jgriffithSwanson it's jut kind of confusing because you have a lot of dependencies in the chain of events18:34
Swansonjgriffith, Holy hell, yes.18:34
jgriffithbetween find_volume and the provider_id stuff18:34
jgriffithI'd rather ask and look dumb than just assume there's no mistakes in there :)18:34
*** dino_ has joined #openstack-cinder18:35
jgriffithSwanson maybe you just short ckt a bunch of that crap18:35
*** dhinesh has joined #openstack-cinder18:36
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Cloned volume with different volume type  https://review.openstack.org/42687818:36
Swansonjgriffith, I need to go through a lot of that. It kinda grew organically.18:36
jgriffithSwanson ahh... so then we go to the self._use_provider_id(provider_id) that catches my other concern :)18:36
jgriffithSwanson yeah, I hear ya!  I need to refactor my driver soonish; after 5 years of cruft and add ons it's gotten stupid18:37
jgriffithSwanson anywho, looks like you cover the things I was looking at, I just needed to follow the chain of events a bit deeper :)18:38
*** crushil has quit IRC18:38
Swansonjgriffith, Thanks for looking closely.18:39
Swansonjgriffith, very appreciated. I'd rather have to answer questions now than have a customer scream at me.18:39
jgriffithSwanson sure, like i said; I'm slow sometimes (well most of the time) but doesn't hurt (much)18:40
*** reduxio has joined #openstack-cinder18:44
*** lpetrut has quit IRC18:46
*** reduxio has quit IRC18:49
*** e0ne has quit IRC18:51
*** e0ne has joined #openstack-cinder18:55
*** diablo_rojo has quit IRC19:06
*** zhonghua has quit IRC19:11
*** zhonghua has joined #openstack-cinder19:11
*** diablo_rojo has joined #openstack-cinder19:15
*** crose has joined #openstack-cinder19:19
*** crose has quit IRC19:24
*** knomura has joined #openstack-cinder19:26
*** lpetrut has joined #openstack-cinder19:28
*** raunak has joined #openstack-cinder19:28
*** knomura has quit IRC19:30
*** alonma has joined #openstack-cinder19:31
*** alonma has quit IRC19:35
*** alonma has joined #openstack-cinder19:38
*** alonma has quit IRC19:42
*** mkoderer has quit IRC19:46
*** Jack_Iv has joined #openstack-cinder19:48
*** Jack_Iv has quit IRC19:54
*** Jack_Iv has joined #openstack-cinder19:56
*** timcl1 has quit IRC20:03
*** timcl has joined #openstack-cinder20:06
*** ChubYann has joined #openstack-cinder20:06
*** crushil has joined #openstack-cinder20:07
patrickeastf20:09
*** e0ne has quit IRC20:12
*** e0ne has joined #openstack-cinder20:16
Swansong20:17
*** jose-phillips has quit IRC20:23
jungleboyjh20:24
xinlihemna: in my new devstack configuration I got a warning message? any idea to fix it? http://paste.openstack.org/show/596931/20:25
*** sdague_ has joined #openstack-cinder20:25
*** jose-phillips has joined #openstack-cinder20:28
xinli@smcginnis: I got a warning message in a new devstack configure, any idea to fix it? http://paste.openstack.org/show/596931/20:30
*** newmember has joined #openstack-cinder20:30
*** _ducttape_ has joined #openstack-cinder20:31
*** ducttape_ has quit IRC20:34
*** reduxio has joined #openstack-cinder20:35
*** reduxio has quit IRC20:39
*** kfarr_ has joined #openstack-cinder20:39
*** kfarr has quit IRC20:39
*** kfarr_ has quit IRC20:40
smcginnisxinli: Looks like it's just a deprecation warning with Nova's client for switching from tenant_id to project_id. Nothing you really need to do at this point.20:40
*** kfarr has joined #openstack-cinder20:40
xinli@smcginnis: thx20:41
*** xyang has joined #openstack-cinder20:41
openstackgerritDrew Balfour proposed openstack/cinder: zfssaiscsi driver initiator group option processing  https://review.openstack.org/35141320:44
*** eharney has quit IRC20:52
*** faiz89 is now known as faiz89_away20:56
*** faiz89_away is now known as faiz8920:59
*** liverpooler has quit IRC21:01
*** _ducttape_ has quit IRC21:02
*** ducttape_ has joined #openstack-cinder21:02
*** Jack_Iv has quit IRC21:04
*** Jack_Iv has joined #openstack-cinder21:05
*** porrua has quit IRC21:05
*** eharney has joined #openstack-cinder21:06
*** alyson_ has quit IRC21:07
*** gouthamr has quit IRC21:07
*** e0ne has quit IRC21:10
*** crushil has quit IRC21:12
*** xinli has quit IRC21:16
*** crushil has joined #openstack-cinder21:21
*** sdague_ has quit IRC21:22
*** xinli has joined #openstack-cinder21:26
*** kaisers has quit IRC21:27
*** diablo_rojo_phon has joined #openstack-cinder21:29
*** yumiriam has quit IRC21:33
*** kfarr has quit IRC21:34
*** gouthamr has joined #openstack-cinder21:35
*** sdague has quit IRC21:38
*** catintheroof has quit IRC21:40
*** alonmarx has quit IRC21:40
*** jose-phillips has quit IRC21:41
*** catintheroof has joined #openstack-cinder21:42
*** alonmarx has joined #openstack-cinder21:45
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: [WIP, DNM]Generate support matrix using driver interfaces  https://review.openstack.org/37116921:46
*** alonma has joined #openstack-cinder21:46
*** newmember has quit IRC21:48
*** jose-phillips has joined #openstack-cinder21:48
waj334Has anyone successfully used pudb to debug cinder? The gui appears, but it hangs. I suspect WSGI is blocking it from running correctly21:50
*** alonma has quit IRC21:51
*** catinthe_ has joined #openstack-cinder21:52
*** catintheroof has quit IRC21:53
*** kaisers has joined #openstack-cinder21:57
*** lpetrut has quit IRC22:02
*** xyang has quit IRC22:02
*** breitz has quit IRC22:03
*** breitz has joined #openstack-cinder22:03
*** catintheroof has joined #openstack-cinder22:04
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: [WIP, DNM]Generate support matrix using driver interfaces  https://review.openstack.org/37116922:05
scottdawaj334: I've used it, it doesn't always work. Make sure you chown the permissions to  you in ~/.config/*22:05
*** eharney has quit IRC22:05
scottdaAnd be aware where the debugger may be running.  You'll need to use screen if on devstack to get to the right service, unless it's triggered from the CLI or client.22:06
*** dustins has quit IRC22:07
*** diablo_rojo has quit IRC22:07
*** catinthe_ has quit IRC22:07
*** xyang has joined #openstack-cinder22:07
waj334scottda: I'm aware of that. Ownership isn't the issue here I don't think. It's gui is showing, but it doesn't seem to be able to process any input22:10
scottdaYeah, I've seen that. I don't remember if I got around it, or how.22:10
waj334scottda: I'm debugging the api, so that's why I'm under the suspicion that WSGI is blocking it's eventloop or something22:11
scottdawaj334: Yeah, but I get the problem if I break out in the manager code running in c-vol as well.22:12
scottdaI usually get frustrated and just go back to pdb. But pudb is a lot nicer, and I've had it working before most/some of the time..22:13
openstackgerritSamantha Blanco proposed openstack/cinder: Adds API documentation for update snapshot status  https://review.openstack.org/40037922:14
waj334scottda: Yea pudb is beautiful lol. I guess use pudb's ugly older sister pdb...22:14
*** dustins has joined #openstack-cinder22:14
jgriffithscottda curious if anybody ever figured out a way to actually use the V3 stuff reliably from Nova22:16
scottdajgriffith: Yes, they merged a patch of mine..just a sec..22:16
jgriffithexcellent22:17
scottdahttps://review.openstack.org/42207822:17
jgriffithcuz I'm about ready to put my head through my monitor :)22:17
jgriffithwhich would be a shame, cuz the monitor is actually worth something :)22:17
scottdajgriffith: so put this in nova.conf:22:17
*** jose-phillips has quit IRC22:17
*** catinthe_ has joined #openstack-cinder22:17
scottdahttps://www.irccloud.com/pastebin/iBiezvYs/22:18
scottdaAnd Cinder will use v322:18
scottdaDon't hurt your head22:18
scottdajgriffith: In fact, you'd better step outside while the sun is still shining.22:18
jgriffithscottda and wrap some error checking around the code in cinder.py22:18
*** catinth__ has joined #openstack-cinder22:18
jgriffithscottda no doubt!22:18
*** catintheroof has quit IRC22:18
jgriffithscottda but I was promised auto-discovery and flying cars :)22:19
jgriffithI want them both, but would settle for intelligence in the disco piece :)22:19
scottdajgriffith: haha..Yeah, I'll update this patch now that cinderclient code has merged:22:19
*** jose-phillips has joined #openstack-cinder22:19
jgriffithscottda thanks!22:19
jgriffithmonitor is saved for now22:19
scottdahttps://review.openstack.org/#/c/42020122:19
scottdaThat can be made better now ^^^22:20
*** crushil has quit IRC22:20
scottdaand now we know the API version for your new cinder API code22:20
jgriffithscottda so we might want to rethink that slightly22:20
jgriffithscottda so ie the case of multiple cinders22:20
scottdajgriffith: yeah, I figured there'd be some devlish details22:20
jgriffithalthough, i'm not sure how to automate that yet22:20
jgriffithor if I want to bother22:21
jgriffithSo I've been doing this crazy url.replace(v2, v3) hack and testing the endpoint_url for an exception to see if V3 is supported or not22:21
jgriffithit works, but it's pretty hacky22:22
*** catinthe_ has quit IRC22:22
scottdaYeah, what do you think, some kind of Global Bool like NEW_CINDER_API that is set from the client, based on version discovery?22:23
scottdaso you can do: if NEW_CINDER_API:22:23
scottda    do new stuff22:23
scottdaelse:22:23
scottda   do old stuff22:23
*** crushil has joined #openstack-cinder22:24
jgriffithscottda yeah, I was thinking something along those lines might work, but the trick is figuring out how we represent multiple volume-services; or I guess the cinder-api would only report LCD22:25
jgriffiththat would probably fix that22:25
*** Jack_Iv has quit IRC22:25
jgriffithbut looking at the code since you're using the endpoint URL I'm not sure how to make that work either :(22:25
scottdaYeah, hitting that endpoint URL gets version info, which comes from the c-api22:26
*** knomura has joined #openstack-cinder22:26
jgriffithby "you're" I mean just in general, not you specifically22:26
scottdaSo if c-api is upgraded, but c-vol is not, they'll be skewed22:27
scottdaI take all the blame22:27
jgriffithscottda right and that means we need a new thing to querie all c-vol services22:27
jgriffithprobably not a huge deal; actually.....22:27
jgriffithhmm22:27
jgriffiththis is going to be kinda F'd anyway :)22:27
jgriffithif people do something out of order22:28
scottdaYou mean upgrade out of order?22:28
scottdaCan't we let some Rolling Upgrade genius solve this?22:28
scottda:)22:28
jgriffithreally wish we could've just specified a version in the request :(22:28
scottdaYou mean the request to the endpoint to get version info?22:29
jgriffithscottda Yeah, let me know when you find that Unicorn and I'll share the Pot of Gold I find at the end of the Rainbow22:29
jgriffithscottda no I mean as a parameter to an api request22:29
*** catinth__ has quit IRC22:29
jgriffithscottda just say:  cinder-endpoint = url-foo22:29
jgriffiththen url-foo/method/version/blah-blah22:29
scottdaI see22:30
jgriffithif url-foo/method doesn't support "version" then error22:30
*** Jack_I has quit IRC22:30
jgriffithremember the white board in Austin?22:30
jgriffithI think that's why I'm banned from white-boards now :)22:30
scottdahahah..I love whiteboards. I have whiteboard paper all over my office walls.22:31
*** knomura has quit IRC22:31
jgriffith:)22:31
*** crushil has quit IRC22:31
jgriffithwell it's on Cinder's buzz-word bingo now so I'm not going near a whiteboard with you folks around!22:31
*** faiz89 has quit IRC22:32
jungleboyjOh, that was what I needed to get at the store the other day.  A white board for my home office.22:34
scottdajgriffith: So, I guess you know you can specify the version in the request via the HTTP header22:35
scottda-H "OpenStack-API-Version: volume 3.12"22:35
scottdaWe should be able to get the client to support the new APIs by sending the appropriate header22:36
jgriffithscottda well... kinda22:36
jgriffithscottda from the client it's one thing, from something consuming said client is different22:37
scottdayeah, there's some work to do to make that happen..22:37
jgriffithscottda and thus what drives me kinda crazy22:37
*** reduxio has joined #openstack-cinder22:37
jgriffithscottda like I said, I can just *try* a version and see if I get an attribute erorr, but that's weak sauce22:37
scottdaWell, I'll try to put something together to make it simpler.22:37
scottdaBBIAB, need to run to the bus stop22:38
jgriffithscottda happy to help, I just don't quite know what22:38
jgriffithhaha22:38
jgriffithalright, I'm getting some more coffee22:38
*** dustins is now known as dustins|out22:40
*** reduxio has quit IRC22:42
*** alonmarx has quit IRC22:52
jgriffithscottda just FYI when you get back, here's the direction I'm going for now:  https://gist.github.com/j-griffith/84bbd6e2eb718fc284eac91f0f52758c22:52
openstackgerritPatrick East proposed openstack/cinder: Add descriptions to Pure drivers  https://review.openstack.org/42696122:56
jgriffithpatrickeast get outa here with that mess!22:57
patrickeastjgriffith: lol22:57
* patrickeast is fixin the *real* issues22:57
jgriffithpatrickeast glad to see somebody is doing it!22:58
patrickeastjgriffith: it was the first thing someone asked about when they saw the list output : /22:58
patrickeasti lost that argument22:58
jgriffithLOL... now that's humorous22:58
jgriffithThat's alright I was asked for another Juno patch today :(22:58
jgriffithI'd rather get requests like yours frankly22:59
patrickeastjgriffith: haha23:01
jgriffithwell, at least that one :)23:01
patrickeastjgriffith: you get asked about backporting replication to kilo yet?23:01
jgriffithpatrickeast nah, I already did that :)23:02
patrickeasthaha23:02
jgriffithpatrickeast hell, I backported it to Nova-Volumes already :)23:02
patrickeastits too easy if you just did the volume-types stuff in the driver!23:02
jgriffithhttps://github.com/j-griffith/nova/tree/essex-sf-replication23:03
jgriffithhehe... you caught me :)23:03
jgriffithfunny thing is those customers are happier than the newer customers... go figure :)23:03
patrickeasteither way, an impressive feat of time travel23:03
patrickeasthaha23:03
jgriffithMagic of Git23:03
jgriffithI had to draw the line at the image-caching stuff though :)23:04
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: [WIP, DNM]Generate support matrix using driver interfaces  https://review.openstack.org/37116923:04
patrickeastthat one gets hard with the glance stuff :(23:05
jgriffithyeah, way too many missing pieces23:05
jgriffithpatrickeast but I was looking at one point of just leveraging the code Nova already has in place for it23:06
jgriffithfor the compute side23:06
jgriffithanyway23:06
jgriffithwhatevs23:06
patrickeastah yea, its pretty similar to what we want to do23:06
patrickeasttheirs is more sophisticated iirc23:06
jgriffithsome things23:07
jgriffiththey're using a qcow so that helps a bit I think23:07
jgriffithwell, I guess it depends on the config23:07
jgriffithwhatever, I'm not doing it :)23:07
patrickeasthaha, exactly23:08
*** ircuser-1 has quit IRC23:08
patrickeastall that file type business... bleh23:08
jgriffithhehe23:08
jgriffithalmost as bad as all that zones and wwpn crap :)23:08
*** mriedem has quit IRC23:10
scottdajgriffith: So, check me if I'm wrong, but if you instantiate the client with version "3.27" it can call client.attachments.list() and it works.23:10
jgriffithscottda correct23:11
jgriffithscottda but the trick also is overriding the url in that diff above that23:11
jgriffithscottda you can't just init the client at 3.27 by itself23:11
scottdaso if we use cinderclient.get_server_version(url) to get the cinder api version, as I have in my POC, and the api (and client version) are >= 3.2723:11
jgriffithbecause every url will still be V223:11
jgriffithscottda yeah, so you Einsteins are going to have to figure all that out, I want nothing to do with it ;)23:12
jgriffithand I mean Einsteins as in smart, not sarcastic23:12
jgriffithI think the micro-version stuff is.... well; you all know what I think23:12
scottdahaha..OK, I'll work on the Nova side tomorrow.23:13
scottdaYeah, I know what you think :)23:13
jgriffithand that's not directed at you or what you did either23:13
* jgriffith is finding he needs to qualify everything he says today23:13
scottdaI pretend to be thick-skinned , but if you cut me , I bleed23:13
scottda:)23:13
jgriffithas do most of us23:14
scottdaNow go home and ride your horse before it gets too cold.23:14
jgriffithhaha... good idea!23:15
*** ircuser-1 has joined #openstack-cinder23:20
*** ircuser-1 has quit IRC23:21
*** ducttape_ has quit IRC23:21
*** sileht has quit IRC23:37
*** xyang has quit IRC23:40
*** xyang has joined #openstack-cinder23:40
*** sileht has joined #openstack-cinder23:41
*** xinli has quit IRC23:41
*** Julien-zte has joined #openstack-cinder23:42
*** xyang1 has quit IRC23:44
*** xyang has quit IRC23:45
*** Julien-zte has quit IRC23:46
*** xyang has joined #openstack-cinder23:47
*** gouthamr has quit IRC23:49
*** catintheroof has joined #openstack-cinder23:50
*** Cibo_ has joined #openstack-cinder23:50
dhinesham trying to write tests for my cinder driver using mock..does anyone have a resource to understand how it works, and possibly samples of how it works with cinder?23:50
dhineshjust beginning  to code on it23:51
*** jamielennox is now known as jamielennox|away23:58
*** jamielennox|away is now known as jamielennox23:59

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