Thursday, 2016-02-04

*** haomaiwa_ has quit IRC00:01
*** lcurtis has quit IRC00:01
*** haomaiwang has joined #openstack-cinder00:01
hemnaso, to clarify00:05
hemnaif we have multi-targets00:05
hemnaand you failover00:05
hemnahow do we pick which target ?00:05
hemnaand know later on that we've picked target A vs. B ?00:05
patrickeasthemna: it can be specified by the caller00:05
patrickeastbut otherwise up to the driver00:05
patrickeastthen you return the id of the one you did fail over to00:06
hemnapassed as a param in the cinderclient failover call ?00:06
*** baojg has joined #openstack-cinder00:06
patrickeastand its stored in that like active_backend_id or whatever column in the service table00:06
hemnawe currently have list replication targets to get a list of them for a host00:06
patrickeastyea i think so00:06
patrickeastyea00:06
patrickeastthere was some discussion about that earlier00:06
hemnaand the admin uses one of those in the failover command00:06
patrickeastyep00:06
hemnaok sorry I missed that00:06
patrickeastsame idea00:06
hemnaso same thing00:06
hemnaok00:06
hemnadone00:06
hemnaship it.00:06
patrickeasteasy as cheesecake00:06
jgriffithpatrickeast: hemna exactly... look at the call in cinder.volume.manager in my patch00:07
jgriffithpatrickeast: hemna there's an optional param to specify "where' you want to failover to00:07
jgriffithpatrickeast: hemna and even a comment that points out what to do if blank00:07
hemnaso Host is a target ?00:08
hemnaor is that a cinder host ?00:08
*** JoseMello has quit IRC00:08
patrickeasthemna: its the 'secondary' param on the driver method, which should just be a string that matches the id your driver wants to use00:09
patrickeastlike the id from list_replication_targets00:09
hemnahttps://review.openstack.org/#/c/275797/1/cinder/volume/manager.py00:09
jgriffithpoor choice of names, updating the comment now00:10
hemnain failover_replication ?00:10
patrickeastoh00:10
hemnaI only see context, host=None00:10
patrickeastyea its called secondary on the driver method, but yea that one00:10
patrickeastas;dlkjads00:10
patrickeaststupid fc hbas are going offline again00:10
hemnadamnit00:11
jgriffithhemna: patrickeast see my response to the comment.  Host is a *terrible* name00:11
hemnaok phew!00:11
jgriffithhemna: patrickeast I should change it to "backend_id" or "backend_target_id" or whatever we call it in the cinder.conf file00:11
hemnaI was confused00:11
jgriffithsorry about that00:11
patrickeastsounds good00:11
hemnavery cool00:12
jgriffithremember we're skipping the whole manage/unmanaged nonsense00:12
hemnajgriffith, thanks so much for work on this.00:12
jgriffitheven if it is managed it doesn't matter00:12
jgriffithhemna: sure... who knows, maybe this time I'll get it right and it'll stick00:12
jgriffithI'm not doing it again just so everybody knows :)00:12
hemnaI am kinda thinking this is more like straight up unmanaged, in the respect that the driver instance has to switch arrays it's talking to depending on failed over or not.00:13
jgriffithhemna: right00:13
jgriffithhemna: but the other thing is if that array is configured in Cinder as well... then that's fine too00:13
smcginnisjgriffith: No repl v3 for you?00:13
jgriffithhemna: it flat out doesn't matter00:13
hemnayah00:13
jgriffithsmcginnis: NOOOOOO please00:13
hemna:)00:13
smcginnis:)00:13
hemnahey, I tried multiattach 3 times then gave up.00:14
jgriffithsmcginnis: Not that I don't like this sort of stuff00:14
hemna3rd time's a charm!00:14
hemnaeerr quit!00:14
patrickeasthaha00:14
jgriffithhemna: that's not dead yet00:14
smcginnisJust a flesh wound.00:14
hemnaat least with replication, it's all in cinder.00:14
jgriffithsmcginnis: :)00:15
jgriffithhemna: true-dat00:15
jgriffithalright, I'm out... see ya'll later00:15
patrickeastcya00:15
smcginnisSee ya00:15
*** baojg has quit IRC00:16
patrickeasthuh... if i unbind/bind a pci device and get a notice that root has new mail its probably a bad sigh eh?00:16
hemnapatrickeast, smcginnis if you guys are really bored, I posted this up today and would like some feedback....https://review.openstack.org/#/c/275943/00:16
hemnahehe00:16
smcginnishemna: Oooh, interesting.00:17
smcginnisThe platform filtering is interesting.00:18
hemnaI don't think it's fully flushed out yet00:18
hemnabut it's an idea I was toying with00:18
smcginnisThough I believe the TC has states Linux is the only officially supported platform.00:18
*** baojg has joined #openstack-cinder00:18
hemnabecause there is a WIP Windows connector patch in gerrit.00:18
hemnawhich is needed by the hyperV support in nova00:19
*** PsionTheory has quit IRC00:19
hemnathere are several Nova patches in flight that deal with adding FC support to hyperV support.00:19
smcginnishemna: I'm for that. Would be cool if we can handle different platforms.00:19
hemnayah00:19
hemnaI'm not sure this is the right way to do it, but it's A way00:20
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/27572900:20
hemnaI was hoping there was a pattern for this type of thing already00:20
hemnabut I just hacked this out today00:20
smcginnishemna: Just scanning through so far, but looks good to nme.00:21
hemnabeen waiting for jenkins results on it00:21
hemnabut check has been backed up 300 years now it seems00:21
smcginnisJenkins isn't too happy today. :/00:21
*** mtanino has quit IRC00:22
smcginnisBut it does look like a patch to fix the issues with getting 502's and 503's and causing restarts is making it through! :)00:22
hemnaoh good00:22
hemnacan we just all revert back to the old version :P00:23
hemna*sigh*00:23
smcginnishehe00:23
*** r-daneel has joined #openstack-cinder00:23
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/26616300:25
*** bill_az has quit IRC00:31
*** dustins has quit IRC00:36
*** cburgess_ has quit IRC00:39
*** chlong has joined #openstack-cinder00:40
*** cburgess has joined #openstack-cinder00:42
*** pratap has joined #openstack-cinder00:44
*** zhangjn has joined #openstack-cinder00:46
*** Fdaisuke has quit IRC00:52
*** alonma has joined #openstack-cinder00:54
*** cheneydc has joined #openstack-cinder00:57
*** alonma has quit IRC00:58
*** haomaiwang has quit IRC01:01
*** jamielennox is now known as jamielennox|away01:01
*** haomaiwang has joined #openstack-cinder01:01
*** alonma has joined #openstack-cinder01:02
*** smoriya_ has joined #openstack-cinder01:04
*** qeelee has joined #openstack-cinder01:07
*** alonma has quit IRC01:07
*** alonma has joined #openstack-cinder01:09
*** laughterwym has joined #openstack-cinder01:10
*** laughterwym has quit IRC01:10
*** laughterwym has joined #openstack-cinder01:10
*** laughterwym has quit IRC01:10
*** alonma has quit IRC01:14
*** laughterwym has joined #openstack-cinder01:14
openstackgerritWilson Liu proposed openstack/cinder: Add description when rename LUN in Huawei driver  https://review.openstack.org/27160301:16
*** huanan_L has quit IRC01:17
*** cknight has joined #openstack-cinder01:17
*** huanan_L has joined #openstack-cinder01:17
*** alonma has joined #openstack-cinder01:19
*** zhangjn has quit IRC01:19
*** mylu has quit IRC01:19
*** alonma has quit IRC01:23
*** alonma has joined #openstack-cinder01:25
*** mylu has joined #openstack-cinder01:26
*** garthb has quit IRC01:27
*** alonma has quit IRC01:29
*** mylu has quit IRC01:31
*** alonma has joined #openstack-cinder01:31
*** zhangjn has joined #openstack-cinder01:33
*** alonma has quit IRC01:36
*** Lee1092 has joined #openstack-cinder01:38
*** geguileo has quit IRC01:38
*** lprice1 has quit IRC01:38
*** zhangjn has quit IRC01:40
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Don't show chap password in log  https://review.openstack.org/27159501:40
*** r-daneel has quit IRC01:41
*** leeantho has quit IRC01:41
*** lixiaoy1 has joined #openstack-cinder01:44
*** zhangjn_ has quit IRC01:45
*** zhangjn has joined #openstack-cinder01:46
*** ericksonsantos has quit IRC01:50
*** cknight has quit IRC01:52
*** mgarza_ has joined #openstack-cinder01:55
*** mylu has joined #openstack-cinder01:57
*** ericksonsantos has joined #openstack-cinder01:59
*** laughterwym has quit IRC02:00
*** qeelee has quit IRC02:00
*** laughterwym has joined #openstack-cinder02:00
*** haomaiwang has quit IRC02:01
*** haomaiwa_ has joined #openstack-cinder02:01
*** laughter_ has joined #openstack-cinder02:02
*** laughte__ has joined #openstack-cinder02:03
*** bardia has quit IRC02:03
*** qeelee has joined #openstack-cinder02:04
*** davechen has joined #openstack-cinder02:05
*** laughterwym has quit IRC02:05
*** laughter_ has quit IRC02:06
*** baojg has quit IRC02:07
openstackgerritLisaLi proposed openstack/cinder: Clean uploading volume when c-vol restarts  https://review.openstack.org/27548002:07
*** cknight has joined #openstack-cinder02:08
*** baojg has joined #openstack-cinder02:10
*** lixiaoy1 has quit IRC02:11
*** qeelee has quit IRC02:11
openstackgerritLisaLi proposed openstack/cinder: Clean uploading volume when c-vol restarts  https://review.openstack.org/27548002:11
*** lixiaoy1 has joined #openstack-cinder02:12
*** qeelee has joined #openstack-cinder02:18
*** alejandrito has joined #openstack-cinder02:20
*** baojg has quit IRC02:23
*** baojg has joined #openstack-cinder02:23
*** laughte__ has quit IRC02:27
*** laughterwym has joined #openstack-cinder02:27
*** alonma has joined #openstack-cinder02:31
*** jamielennox|away is now known as jamielennox02:32
*** mudassirlatif has quit IRC02:32
*** qeelee has quit IRC02:35
*** alonma has quit IRC02:35
*** qeelee has joined #openstack-cinder02:36
*** qeelee has quit IRC02:45
openstackgerritApoorva Deshpande proposed openstack/cinder: Tintri image direct clone  https://review.openstack.org/26112602:46
openstackgerritLisaLi proposed openstack/cinder: Create volume in same zone when backup restore  https://review.openstack.org/27240402:48
*** baojg has quit IRC02:51
*** sheel has joined #openstack-cinder02:52
*** baojg has joined #openstack-cinder02:52
openstackgerritAccela Zhao proposed openstack/cinder: Don't create cgsnapshot if empty consistencygroup  https://review.openstack.org/27602302:54
*** thurloat_isgone is now known as thurloat02:54
openstackgerritSheel Rana proposed openstack/cinder: Use of metadata id instead of metadata name  https://review.openstack.org/27458902:56
*** apoorvad has quit IRC02:59
*** haomaiwa_ has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** baojg has quit IRC03:02
*** baojg_ has joined #openstack-cinder03:02
*** qeelee has joined #openstack-cinder03:02
openstackgerritLisaLi proposed openstack/cinder: Create volume in same zone when backup restore  https://review.openstack.org/27240403:04
*** baojg_ has quit IRC03:05
*** jamielennox is now known as jamielennox|away03:06
*** baojg has joined #openstack-cinder03:06
*** baojg has quit IRC03:09
*** baojg has joined #openstack-cinder03:09
*** jamielennox|away is now known as jamielennox03:11
*** qeelee has quit IRC03:12
*** gouthamr_ has quit IRC03:12
*** qeelee has joined #openstack-cinder03:13
*** baojg has quit IRC03:16
*** baojg has joined #openstack-cinder03:16
*** alejandrito has quit IRC03:17
*** gouthamr has joined #openstack-cinder03:18
*** qeelee has quit IRC03:21
*** alonma has joined #openstack-cinder03:32
*** gouthamr has quit IRC03:34
*** alonma has quit IRC03:36
*** dims has joined #openstack-cinder03:37
openstackgerritAccela Zhao proposed openstack/cinder: Don't create cgsnapshot if empty consistencygroup  https://review.openstack.org/27602303:39
*** cheneydc_ has joined #openstack-cinder03:40
*** cheneydc_ has quit IRC03:40
*** alonma has joined #openstack-cinder03:41
openstackgerritAccela Zhao proposed openstack/cinder: Don't create cgsnapshot if empty consistencygroup  https://review.openstack.org/27602303:41
*** alonma has quit IRC03:45
*** baojg has quit IRC03:46
*** alonma has joined #openstack-cinder03:49
*** links has joined #openstack-cinder03:49
openstackgerritSheel Rana proposed openstack/cinder: Bootable filter for listening volumes from CLI  https://review.openstack.org/27222903:50
*** vivekd has joined #openstack-cinder03:52
*** dims has quit IRC03:53
*** alonma has quit IRC03:53
*** salv-orl_ has joined #openstack-cinder03:54
*** baojg has joined #openstack-cinder03:54
*** esker has quit IRC03:55
*** alonma has joined #openstack-cinder03:56
*** salv-orlando has quit IRC03:58
*** vivekd has quit IRC03:59
*** alonma has quit IRC04:00
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-cinder04:01
*** alonma has joined #openstack-cinder04:04
*** esker has joined #openstack-cinder04:05
*** laughter_ has joined #openstack-cinder04:06
*** mgarza_ has quit IRC04:06
*** laughterwym has quit IRC04:08
*** alonma has quit IRC04:09
*** daneyon_ has joined #openstack-cinder04:11
*** boris-42 has quit IRC04:11
*** briancurtin has quit IRC04:11
*** fmccrthy has quit IRC04:12
*** cppforlife_ has quit IRC04:12
*** jamielennox is now known as jamielennox|away04:12
*** Lee1092 has quit IRC04:12
*** daneyon has quit IRC04:12
*** mtanino__ has quit IRC04:12
*** tpeoples has quit IRC04:12
*** Nakato has quit IRC04:12
*** abhi has quit IRC04:13
*** tbarron has quit IRC04:13
*** jroll has quit IRC04:13
*** gus has quit IRC04:13
*** chlong has quit IRC04:13
*** Nakato has joined #openstack-cinder04:13
*** alonma has joined #openstack-cinder04:14
*** dosaboy_ has quit IRC04:14
*** tbarron has joined #openstack-cinder04:14
*** gus has joined #openstack-cinder04:14
*** jroll has joined #openstack-cinder04:14
*** mylu has quit IRC04:14
*** EinstCrazy has joined #openstack-cinder04:14
*** dosaboy has joined #openstack-cinder04:15
*** abhi has joined #openstack-cinder04:15
*** mtanino__ has joined #openstack-cinder04:16
*** fmccrthy has joined #openstack-cinder04:16
*** mylu has joined #openstack-cinder04:17
*** tpeoples has joined #openstack-cinder04:17
*** lprice has joined #openstack-cinder04:17
*** xek_ has joined #openstack-cinder04:18
*** boris-42 has joined #openstack-cinder04:18
*** rwall_ has joined #openstack-cinder04:19
*** amit213 has quit IRC04:19
*** rwall has quit IRC04:19
*** SergeyLukjanov has quit IRC04:19
*** alonma has quit IRC04:19
*** mriedem_afk has quit IRC04:19
*** frakt has quit IRC04:19
*** smcginnis has quit IRC04:19
*** rwall_ is now known as rwall04:19
*** frakt has joined #openstack-cinder04:19
*** xek has quit IRC04:19
*** SergeyLukjanov has joined #openstack-cinder04:20
*** briancurtin has joined #openstack-cinder04:20
*** smcginnis has joined #openstack-cinder04:20
*** cknight has quit IRC04:21
*** shyama has joined #openstack-cinder04:22
*** Lee1092 has joined #openstack-cinder04:23
*** cppforlife_ has joined #openstack-cinder04:23
*** alonma has joined #openstack-cinder04:26
*** haomaiwang has quit IRC04:28
*** chlong has joined #openstack-cinder04:28
*** alonma has quit IRC04:30
*** chhavi has joined #openstack-cinder04:33
*** vivekd has joined #openstack-cinder04:35
*** shausy has joined #openstack-cinder04:42
openstackgerritAccela Zhao proposed openstack/cinder: Don't create cgsnapshot if consistencygroup is empty  https://review.openstack.org/27602304:47
*** vivekd has quit IRC04:48
openstackgerritAccela Zhao proposed openstack/cinder: Don't create cgsnapshot if consistencygroup is empty  https://review.openstack.org/27602304:50
*** bkumar has joined #openstack-cinder04:50
openstackgerritAnthony Lee proposed openstack/cinder: LeftHand: Implement un/manage snapshot support  https://review.openstack.org/25501504:52
openstackgerritSheel Rana proposed openstack/cinder: Bootable filter for listening volumes from CLI  https://review.openstack.org/27222905:04
openstackgerritAccela Zhao proposed openstack/cinder: Don't create cgsnapshot if cg is empty  https://review.openstack.org/27602305:05
*** abhishekk has joined #openstack-cinder05:08
*** DericHorn-HP has joined #openstack-cinder05:10
*** vivekd has joined #openstack-cinder05:11
*** boris-42 has quit IRC05:13
*** DericHorn-HP has quit IRC05:17
*** haomaiwa_ has joined #openstack-cinder05:20
*** alonma has joined #openstack-cinder05:20
*** angela-s has quit IRC05:24
*** alonma has quit IRC05:24
*** laughter_ has quit IRC05:29
*** laughterwym has joined #openstack-cinder05:30
*** jwcroppe has joined #openstack-cinder05:33
*** markvoelker_ has quit IRC05:34
*** shausy has quit IRC05:35
*** leeantho has joined #openstack-cinder05:35
*** shausy has joined #openstack-cinder05:35
*** abhishekk has quit IRC05:38
*** qeelee has joined #openstack-cinder05:39
*** krtaylor has quit IRC05:44
*** baojg has quit IRC05:46
*** vgridnev has joined #openstack-cinder05:51
*** krtaylor has joined #openstack-cinder05:56
*** itzdilip has joined #openstack-cinder06:00
*** haomaiwa_ has quit IRC06:01
*** haomaiwang has joined #openstack-cinder06:01
*** krtaylor has quit IRC06:02
*** baojg has joined #openstack-cinder06:02
*** vivekd_ has joined #openstack-cinder06:11
*** vivekd has quit IRC06:12
*** vivekd_ is now known as vivekd06:12
*** krtaylor has joined #openstack-cinder06:15
*** haomaiwang has quit IRC06:18
*** haomaiwang has joined #openstack-cinder06:19
*** lpetrut has joined #openstack-cinder06:20
*** alonma has joined #openstack-cinder06:20
*** alonma has quit IRC06:24
*** mylu has quit IRC06:26
*** alonma has joined #openstack-cinder06:27
*** tongli has quit IRC06:30
*** alonma has quit IRC06:32
*** alonma has joined #openstack-cinder06:34
*** markvoelker has joined #openstack-cinder06:35
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/27606006:38
*** alonma has quit IRC06:38
*** markvoelker has quit IRC06:39
*** vivekd has quit IRC06:40
*** cfriesen has quit IRC06:40
*** alonma has joined #openstack-cinder06:40
*** mylu has joined #openstack-cinder06:41
*** alonma has quit IRC06:44
*** alonma has joined #openstack-cinder06:46
*** alonma has quit IRC06:50
*** jwcroppe has quit IRC06:51
*** jwcroppe has joined #openstack-cinder06:52
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: manage_existing_get_size for VMDK driver  https://review.openstack.org/27606306:52
*** alonma has joined #openstack-cinder06:53
*** lpetrut has quit IRC06:54
*** krtaylor has quit IRC06:54
*** jwcroppe has quit IRC06:56
*** alonma has quit IRC06:57
*** alonma has joined #openstack-cinder06:59
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: manage_existing_get_size for VMDK driver  https://review.openstack.org/27606307:00
*** haomaiwang has quit IRC07:01
*** haomaiwa_ has joined #openstack-cinder07:01
*** belmoreira has joined #openstack-cinder07:02
*** alonma has quit IRC07:03
*** alonma has joined #openstack-cinder07:05
*** krtaylor has joined #openstack-cinder07:06
*** markus_zz has joined #openstack-cinder07:06
*** markus_zz has quit IRC07:07
*** alonma has quit IRC07:09
*** salv-orl_ has quit IRC07:13
*** alonma has joined #openstack-cinder07:14
*** avishay has joined #openstack-cinder07:15
openstackgerritLisaLi proposed openstack/cinder: Update quota when volume type renames.  https://review.openstack.org/26799907:16
*** alonma has quit IRC07:19
*** lixiaoy1 has quit IRC07:19
*** alonma has joined #openstack-cinder07:21
*** lixiaoy1 has joined #openstack-cinder07:21
openstackgerritSheel Rana proposed openstack/cinder: Use of metadata id instead of metadata name  https://review.openstack.org/27458907:22
*** alonma has quit IRC07:25
*** lpetrut has joined #openstack-cinder07:26
*** alonma has joined #openstack-cinder07:27
*** leeantho has quit IRC07:29
*** lpetrut has quit IRC07:30
*** alonma has quit IRC07:31
*** mylu has quit IRC07:32
*** alonma has joined #openstack-cinder07:33
sheelDuncanT: Hi07:35
sheelDuncanT: Please review https://review.openstack.org/#/c/272229/ as per time availibility on your end..07:36
*** markvoelker has joined #openstack-cinder07:36
sheelDuncanT: thank you...07:36
*** alonma has quit IRC07:37
*** alonma has joined #openstack-cinder07:39
*** markvoelker has quit IRC07:41
*** lixiaoy11 has joined #openstack-cinder07:41
*** lixiaoy1 has quit IRC07:41
*** lixiaoy1 has joined #openstack-cinder07:42
*** lixiaoy11 has quit IRC07:42
*** lixiaoy11 has joined #openstack-cinder07:43
*** lixiaoy1 has quit IRC07:43
*** alonma has quit IRC07:43
*** alonma has joined #openstack-cinder07:45
*** arecknag has joined #openstack-cinder07:47
*** anshul has joined #openstack-cinder07:48
*** nkrinner has joined #openstack-cinder07:49
*** alonma has quit IRC07:50
*** alonma has joined #openstack-cinder07:52
*** alonma has quit IRC07:56
*** mvk_ has quit IRC07:57
*** alonma has joined #openstack-cinder07:58
*** haomaiwa_ has quit IRC08:01
*** haomaiwa_ has joined #openstack-cinder08:01
*** alonma has quit IRC08:02
*** lixiaoy11 has left #openstack-cinder08:03
*** lixiaoy1 has joined #openstack-cinder08:03
*** alonma has joined #openstack-cinder08:04
*** sinese has joined #openstack-cinder08:05
*** ChubYann has quit IRC08:07
*** vgridnev has quit IRC08:08
*** alonma has quit IRC08:08
*** alonma has joined #openstack-cinder08:10
*** alonma has quit IRC08:14
*** laughterwym has quit IRC08:14
*** laughterwym has joined #openstack-cinder08:15
*** laughterwym has quit IRC08:15
*** laughterwym has joined #openstack-cinder08:15
*** laughterwym has quit IRC08:15
*** laughterwym has joined #openstack-cinder08:16
*** alonma has joined #openstack-cinder08:16
openstackgerrityangweiwei proposed openstack/cinder: Update the method of backup-restore  https://review.openstack.org/27554208:18
*** alonma has quit IRC08:20
*** nkrinner has quit IRC08:20
*** laughterwym has quit IRC08:21
*** nkrinner has joined #openstack-cinder08:23
*** vgridnev has joined #openstack-cinder08:25
*** mvk_ has joined #openstack-cinder08:26
*** alonma has joined #openstack-cinder08:27
*** jistr has joined #openstack-cinder08:29
*** alonma has quit IRC08:31
*** xinyong is now known as xiangxinyong08:34
*** andymaier has joined #openstack-cinder08:34
*** vgridnev has quit IRC08:46
*** slunkad has quit IRC08:50
*** jordanP has joined #openstack-cinder08:52
*** slunkad has joined #openstack-cinder08:52
*** liverpooler has joined #openstack-cinder08:53
*** haomaiwa_ has quit IRC09:01
*** haomaiwa_ has joined #openstack-cinder09:01
*** avishay has quit IRC09:01
*** baojg has quit IRC09:04
*** baojg has joined #openstack-cinder09:07
swamireddyis Cinder support the volume encryption?09:11
openstackgerrityangweiwei proposed openstack/cinder: Update the status of backup-restore  https://review.openstack.org/27554209:14
*** jistr has quit IRC09:16
*** avishay has joined #openstack-cinder09:17
*** openstackgerrit has quit IRC09:17
*** openstackgerrit has joined #openstack-cinder09:18
*** alonma has joined #openstack-cinder09:19
sheelswamireddy: yes09:21
sheelswamireddy: but as far as i know this support is only provided by IBM driver09:21
swamireddysheel: Thanks...Ok...I see that - encrytion supports in  K release onwards only..09:25
sheelswamireddy:yup09:25
swamireddysheel:  ceph backend might not be support ATM...09:26
*** lpetrut has joined #openstack-cinder09:30
*** qeelee has quit IRC09:33
*** tpsilva has joined #openstack-cinder09:34
*** markvoelker has joined #openstack-cinder09:37
*** geguileo has joined #openstack-cinder09:37
*** abhishekk has joined #openstack-cinder09:40
*** markvoelker has quit IRC09:41
openstackgerritwanghao proposed openstack/cinder: Uploading volume to specific image  https://review.openstack.org/27007309:42
*** isaacb has joined #openstack-cinder09:42
*** qeelee has joined #openstack-cinder09:43
*** ociuhandu has joined #openstack-cinder09:45
*** ociuhandu has quit IRC09:49
*** ociuhandu has joined #openstack-cinder09:49
*** sgotliv has joined #openstack-cinder09:50
*** dalgaaf has joined #openstack-cinder09:53
*** davechen has left #openstack-cinder09:57
*** haomaiwa_ has quit IRC10:01
*** haomaiwang has joined #openstack-cinder10:01
*** cheneydc has quit IRC10:02
*** ildikov has quit IRC10:07
DuncanTEncryption should be supported at the hypervisor by any iSCSI driver, and at the backend by a bunch of drivers10:11
openstackgerritGyorgy Szombathelyi proposed openstack/cinder: Support https keystone CA checking in volume quotas  https://review.openstack.org/27243710:11
*** qeelee has quit IRC10:12
*** e0ne has joined #openstack-cinder10:14
*** esp has joined #openstack-cinder10:20
*** baojg_ has joined #openstack-cinder10:24
*** esp has quit IRC10:24
*** baojg has quit IRC10:25
sheelswamireddy: hmm, i think so10:27
*** dguryanov has joined #openstack-cinder10:30
*** avishay has quit IRC10:36
*** jwang__ has joined #openstack-cinder10:37
*** baojg_ has quit IRC10:38
*** smoriya has quit IRC10:38
*** smoriya_afk has joined #openstack-cinder10:39
*** smoriya_afk is now known as smoriya10:39
*** kmartin__ has joined #openstack-cinder10:39
*** fernnest__ has joined #openstack-cinder10:39
*** pots2 has joined #openstack-cinder10:39
*** jwang_ has quit IRC10:40
*** pots has quit IRC10:40
*** kmartin_ has quit IRC10:40
*** fernnest_ has quit IRC10:40
*** ndipanov has joined #openstack-cinder10:44
*** alonma has quit IRC10:45
*** alonma has joined #openstack-cinder10:45
*** baojg has joined #openstack-cinder10:45
*** alonma has quit IRC10:49
*** ociuhandu has quit IRC10:50
*** alonma has joined #openstack-cinder10:51
openstackgerritMatan Sabag proposed openstack/cinder: Support for consistency groups in ScaleIO driver  https://review.openstack.org/26962410:52
*** baojg has quit IRC10:53
*** salv-orlando has joined #openstack-cinder10:54
*** haomaiwang has quit IRC10:54
*** alonma has quit IRC10:55
*** alonma has joined #openstack-cinder10:57
*** qeelee has joined #openstack-cinder10:59
*** alonma has quit IRC11:02
*** alonma has joined #openstack-cinder11:04
yuriy_n17geguileo: Hi! Sorry for disturbing. Please review https://review.openstack.org/#/c/263257/ whenever you have free time. Thanks a lot for advance.11:05
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Method not being called for V3  https://review.openstack.org/25206611:08
*** alonma has quit IRC11:08
openstackgerritSheel Rana proposed openstack/python-cinderclient: Bootable filter for listening volumes from CLI  https://review.openstack.org/27223211:09
*** qeelee has quit IRC11:10
*** alonma has joined #openstack-cinder11:11
*** dims has joined #openstack-cinder11:12
*** baojg has joined #openstack-cinder11:13
*** qeelee has joined #openstack-cinder11:15
*** alonma has quit IRC11:15
*** alonma has joined #openstack-cinder11:17
*** alonma has joined #openstack-cinder11:18
*** ankit_ag has joined #openstack-cinder11:21
*** yhayashi has quit IRC11:22
swamireddyDuncanT: yes....so it iSCSI driver should have the encryption functionality and its respective backend also suppor the encryption....is this correct?11:24
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Changing PercentSynced to CopyState in isSynched  https://review.openstack.org/24699211:24
DuncanTswamireddy: for iSCSI, cinder can do the encryption at the hypervisor, so it doesn't matter what the backend does11:25
*** sheel has quit IRC11:32
*** erlon has joined #openstack-cinder11:33
*** haomaiwa_ has joined #openstack-cinder11:37
*** markvoelker has joined #openstack-cinder11:38
*** rockeychen has joined #openstack-cinder11:41
*** markvoelker has quit IRC11:43
openstackgerritAccela Zhao proposed openstack/cinder: Fix variable scope issue in try-except  https://review.openstack.org/27617911:43
*** qeelee has quit IRC11:47
*** qeelee has joined #openstack-cinder11:48
*** bkumar has quit IRC11:50
*** merooney has joined #openstack-cinder11:51
*** jistr has joined #openstack-cinder11:56
*** resker has joined #openstack-cinder11:58
*** chhavi has quit IRC11:58
dguryanovHello!12:00
*** haomaiwa_ has quit IRC12:01
*** esker has quit IRC12:01
dguryanovIs it possible to upload patch to an abandoned review?12:01
*** haomaiwang has joined #openstack-cinder12:01
dguryanovThere is a review, which abandoned until newton https://review.openstack.org/#/c/238894/12:01
dguryanovBut a set of my patches depends on it12:01
dguryanovI'd like to just upload rebased version12:02
*** xiangxinyong has quit IRC12:02
*** xek_ is now known as xek12:08
*** ildikov has joined #openstack-cinder12:09
*** chhavi has joined #openstack-cinder12:10
*** merooney has quit IRC12:10
*** merooney has joined #openstack-cinder12:12
*** vgridnev has joined #openstack-cinder12:13
*** merooney has quit IRC12:13
*** qeelee has quit IRC12:13
*** qeelee has joined #openstack-cinder12:14
e0nedguryanov: hi, no, you have to restrore patch12:17
dguryanovIt seems I don't have permissions12:17
*** merooney has joined #openstack-cinder12:18
*** EinstCrazy has quit IRC12:19
*** qeelee has quit IRC12:19
*** EinstCrazy has joined #openstack-cinder12:20
e0nedguryanov: please, ask DuncanT12:21
flip214can somebody help me with a devstack failure? http://logs.openstack.org/38/276138/1/check/gate-drbd-devstack-bashate/b076b2f/console.html#_2016-02-04_10_56_15_17812:21
e0nedguryanov: from my PoV it's enough to get -212:21
dguryanove0ne: OK, thanks!12:21
flip214or, more correct, bashate.12:21
dguryanove0ne: Do you mean set -2 to review instead of abandoning?12:22
*** raildo-afk is now known as raildo12:23
e0nedguryanov: yes. but maybe DuncanT has some reasons to abandone it, so I would like to ask him12:23
dguryanove0ne: OK12:23
dguryanovDuncanT: Hello! Could you, please, temporary reopen review https://review.openstack.org/#/c/238894/? I'd like to upload rebased version of this patch, because my patches depend on it :(12:26
*** baojg has quit IRC12:28
*** baojg has joined #openstack-cinder12:31
*** salv-orlando has quit IRC12:32
DuncanTDone12:37
DuncanTI was told that patch series is entirely optional, so I abandoned the series since we already have a whole bunch of coded changing for rolling upgrades, and I was reluctant to see more added12:37
DuncanTWe need to keep the amount of the change sane for the inevitable debug work when something goes wrong12:38
openstackgerritMarc Koderer proposed openstack/cinder: Fix issue in hacking with underscore imports  https://review.openstack.org/27619812:44
*** salv-orlando has joined #openstack-cinder12:46
*** tpatzig has quit IRC12:47
*** mkoderer has quit IRC12:49
* DuncanT appologies to mkoderer for that review, but it needs it12:50
*** erlon_ has joined #openstack-cinder12:50
*** fernnest_ has joined #openstack-cinder12:52
*** erlon has quit IRC12:52
*** erlon_ is now known as erlon12:52
*** timcl has joined #openstack-cinder12:52
*** fernnest__ has quit IRC12:53
*** knikolla has quit IRC12:53
*** markvoelker has joined #openstack-cinder12:53
*** mc_nair has quit IRC12:54
*** laughterwym has joined #openstack-cinder12:55
*** takedakn has joined #openstack-cinder12:55
*** knikolla has joined #openstack-cinder12:55
*** mc_nair has joined #openstack-cinder12:57
openstackgerritGyorgy Szombathelyi proposed openstack/cinder: Support https keystone CA checking in volume quotas  https://review.openstack.org/27243712:58
*** markvoelker has quit IRC12:58
*** merooney has quit IRC12:58
*** haomaiwang has quit IRC13:01
*** haomaiwa_ has joined #openstack-cinder13:01
*** tpatzig has joined #openstack-cinder13:03
*** mkoderer has joined #openstack-cinder13:03
*** dguryanov has quit IRC13:04
*** merooney has joined #openstack-cinder13:07
*** laughterwym has quit IRC13:09
*** markvoelker has joined #openstack-cinder13:11
*** alonma has quit IRC13:14
*** ociuhandu has joined #openstack-cinder13:15
*** smoriya_ has quit IRC13:16
*** takedakn has quit IRC13:17
*** alonma has joined #openstack-cinder13:21
*** abhishekk has quit IRC13:22
*** akerr has joined #openstack-cinder13:25
*** baojg has quit IRC13:27
*** sinese has quit IRC13:28
*** baojg has joined #openstack-cinder13:28
*** sinese has joined #openstack-cinder13:29
*** sinese has quit IRC13:29
*** edmondsw has joined #openstack-cinder13:31
*** porrua has joined #openstack-cinder13:32
*** ankit_ag has quit IRC13:33
*** takedakn has joined #openstack-cinder13:33
*** arthurrdg has joined #openstack-cinder13:34
*** rockeychen has quit IRC13:34
*** rockeychen has joined #openstack-cinder13:34
*** ildikov has quit IRC13:34
*** david-lyle has quit IRC13:38
*** diablo_rojo has joined #openstack-cinder13:39
*** mylu has joined #openstack-cinder13:39
*** gouthamr has joined #openstack-cinder13:39
*** merooney has quit IRC13:41
*** arthurrdg_ has joined #openstack-cinder13:41
*** shausy has quit IRC13:42
*** leseb_ has quit IRC13:42
*** gouthamr_ has joined #openstack-cinder13:43
*** mylu has quit IRC13:43
*** arthurrdg has quit IRC13:43
*** gouthamr has quit IRC13:46
*** merooney has joined #openstack-cinder13:50
openstackgerritPhilipp Marek proposed openstack/cinder: DRBD: Compat for current DRBDmanage versions.  https://review.openstack.org/27624213:52
*** links has quit IRC13:56
akerrHiya cores, wondering if I could get a +Workflow on my backport? https://review.openstack.org/#/c/273732/13:57
mkodererDuncanT: do we have unit tests for hacking rules? ( https://review.openstack.org/#/c/276198/ )13:59
smcginnismkoderer: Yep, let me get a link for you.14:00
*** haomaiwa_ has quit IRC14:01
smcginnismkoderer: https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_hacking.py14:01
*** ildikov has joined #openstack-cinder14:01
*** haomaiwa_ has joined #openstack-cinder14:01
*** rlrossit has joined #openstack-cinder14:02
smcginnisakerr: Done14:02
akerrsmcginnis: thanks!14:03
*** sheel has joined #openstack-cinder14:03
mkoderersmcginnis: ok thx! need to check if Manila has them too ..14:03
smcginnismkoderer: They are useful. The few hacking checks I've done - I caught issues writing the tests. :)14:04
*** barra204 has quit IRC14:04
DuncanTmkoderer: What he said ;-)14:06
*** shakamunyi has joined #openstack-cinder14:06
smcginnis:)14:06
*** julim has joined #openstack-cinder14:07
*** merooney has quit IRC14:07
*** dims_ has joined #openstack-cinder14:10
*** merooney has joined #openstack-cinder14:11
*** dims has quit IRC14:12
*** gouthamr_ has quit IRC14:15
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in smbfs volume driver  https://review.openstack.org/26155114:17
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in vzstorage volume driver  https://review.openstack.org/26155014:17
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in netapp_nfs unit test  https://review.openstack.org/26155714:17
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in remotefs.py  https://review.openstack.org/26155814:17
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in glusterfs volume driver  https://review.openstack.org/26155214:17
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in scality volume driver  https://review.openstack.org/26155314:17
openstackgerritDmitry Guryanov proposed openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs  https://review.openstack.org/27090814:17
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in quobyte volume driver  https://review.openstack.org/26155414:17
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in nfs volume driver  https://review.openstack.org/26155514:17
openstackgerritDmitry Guryanov proposed openstack/cinder: Update initialize_connection to use versionedobjects  https://review.openstack.org/23889414:17
openstackgerritLiucheng Jiang proposed openstack/cinder: Huawei: Implement v2 replication (managed)  https://review.openstack.org/26124614:17
*** takedakn has quit IRC14:19
*** salv-orlando has quit IRC14:20
*** mylu has joined #openstack-cinder14:23
*** salv-orlando has joined #openstack-cinder14:24
*** gouthamr has joined #openstack-cinder14:26
*** sgotliv has quit IRC14:28
*** timcl has quit IRC14:28
*** arthurrdg_ has quit IRC14:35
*** merooney has quit IRC14:36
*** resker has quit IRC14:36
*** timcl has joined #openstack-cinder14:37
*** jgregor has joined #openstack-cinder14:37
*** takedakn has joined #openstack-cinder14:38
*** mylu has quit IRC14:40
*** takedakn has quit IRC14:41
*** mylu has joined #openstack-cinder14:43
openstackgerritMarc Koderer proposed openstack/cinder: Fix issue in hacking with underscore imports  https://review.openstack.org/27619814:44
*** takedakn has joined #openstack-cinder14:45
*** xyang1 has joined #openstack-cinder14:47
*** merooney has joined #openstack-cinder14:48
*** takedakn has quit IRC14:49
*** cfriesen has joined #openstack-cinder14:52
*** markstur_ has joined #openstack-cinder14:53
*** markstur_ has left #openstack-cinder14:55
*** dustins has joined #openstack-cinder14:56
*** sgotliv has joined #openstack-cinder14:57
*** eharney has joined #openstack-cinder15:00
*** cknight has joined #openstack-cinder15:00
*** haomaiwa_ has quit IRC15:01
*** haomaiwang has joined #openstack-cinder15:01
*** esp has joined #openstack-cinder15:01
*** daneyon_ has quit IRC15:01
*** edtubill has joined #openstack-cinder15:08
*** esp has quit IRC15:10
openstackgerritSheel Rana proposed openstack/python-cinderclient: Bootable filter for listening volumes from CLI  https://review.openstack.org/27223215:12
*** jwcroppe has joined #openstack-cinder15:14
*** jungleboyj has quit IRC15:15
sheelDear All,15:17
sheelneed some eyes for https://review.openstack.org/#/c/274589/615:17
sheelscottda: hi there15:19
sheelscottda: could we discuss today for micro version for https://review.openstack.org/#/c/272229 ?15:20
*** chhavi has quit IRC15:21
aorourkepatrickeast, congrats on the core position :)15:22
sheel11->12  patrickeast:All the Best for your new Role..15:25
*** baumann has joined #openstack-cinder15:27
diablo_rojopatrickeast: Congrats Patrick :)15:28
*** mgarza has joined #openstack-cinder15:29
*** e0ne has quit IRC15:29
*** e0ne has joined #openstack-cinder15:30
*** esker has joined #openstack-cinder15:32
*** vgridnev has quit IRC15:32
*** baojg has quit IRC15:33
*** julim has quit IRC15:33
*** arch-nemesis has joined #openstack-cinder15:35
*** julim has joined #openstack-cinder15:35
*** mylu has quit IRC15:36
*** baojg has joined #openstack-cinder15:37
*** baojg has quit IRC15:37
*** mylu has joined #openstack-cinder15:39
*** gouthamr_ has joined #openstack-cinder15:40
openstackgerritAccela Zhao proposed openstack/cinder: Fix variable scope issue in try-except  https://review.openstack.org/27617915:43
*** gouthamr has quit IRC15:43
openstackgerritAccela Zhao proposed openstack/cinder: Inherit multiattach from volume source  https://review.openstack.org/27631215:43
*** crose has joined #openstack-cinder15:46
scottdasheel: Hi. I looked at your patch. Yes, when microversions has landed this could  microversioned..15:47
openstackgerritJohn Griffith proposed openstack/cinder: Replication v2.1  https://review.openstack.org/27579715:47
*** alyson_ has joined #openstack-cinder15:48
*** alyson_ has quit IRC15:48
*** JoseMello has joined #openstack-cinder15:48
*** alyson_ has joined #openstack-cinder15:49
*** rockeychen has quit IRC15:51
*** zhipeng has joined #openstack-cinder15:51
*** rockeychen has joined #openstack-cinder15:52
sheelscottda: ok, thanks for confirmation15:53
sheelscottda15:53
sheelscottda: could you give some link to have some idea about micro version15:53
openstackgerritJohn Griffith proposed openstack/cinder: Replication v2.1  https://review.openstack.org/27579715:54
sheelscottda: it wil help me to write code if required to release this patch for micro versioning15:55
*** garthb has joined #openstack-cinder15:55
sheelscottda: may be some code, patch, anything..15:55
scottdasheel: https://review.openstack.org/#/c/224910/1715:59
scottdasheel: There is a devref doc there as well15:59
patrickeastaorourke: sheel: diablo_rojo : thanks!16:00
sheelscottda:hmm, quite large patch :)16:01
*** haomaiwang has quit IRC16:01
scottdaTo use microversions you need the code in a class that inherits from wsgi.Controller16:01
scottdai.e. cinder/api/v2/volumes.py16:01
*** haomaiwa_ has joined #openstack-cinder16:01
scottdaBut I need to add a v3 endpoint first....16:01
smcginnispatrickeast: You see the shiny new -2/+2 options on the review page yet? :)16:01
patrickeastsmcginnis: yep :D16:02
sheelscottda: great16:02
*** dustins_ has joined #openstack-cinder16:02
*** mylu has quit IRC16:02
*** mtanino has joined #openstack-cinder16:02
sheelscottda: thnx for explanation16:02
*** jungleboyj has joined #openstack-cinder16:02
*** dustins has quit IRC16:03
sheelpatrickeast: May be you can have feeling of pressing +2 button on https://review.openstack.org/#/c/272229/16:03
sheelpatrickeast: ;)16:04
smcginnisHah, the review requests are already coming in. :)16:04
patrickeasthaha16:04
*** r-daneel has joined #openstack-cinder16:04
patrickeasti'll check it out in a little bit, on mobile right now16:05
scottdasheel: I could walk you through the changes you will need, but it is getting late in the cycle and I am not sure microversions will get in before your patch. I think reviewers should consider this for your patch, and whether your change will break anyone.16:05
sheelpatrickeast : thanks16:05
sheelscottda: patch is about to release..16:06
*** alkhodos has joined #openstack-cinder16:06
sheelscottda: actually there were 2 versions of patch16:06
sheelfor one we need micro version support so that it does not break existing behavior16:06
scottdaok, well micorversion patch is NOT about to release.16:06
sheelscottda: hehe16:07
sheelscottda:  will it in mitaka or Newton?16:07
sheelscottda:  my patch can wait...there is no urgency as such16:08
scottdasheel: Unsure about mitaka. I don't think it could/should merge at the last minute, which is coming up soon.16:08
*** merooney has quit IRC16:08
sheelscottda: yup, no issues...16:09
*** merooney has joined #openstack-cinder16:09
sheelscottda: I will follow up with your patch(you shared)..may be will ping you once your patch is released..16:09
SwansonCan't help but notice something murdered my (Sean's.  I murdered my)  CI overnight.16:10
*** arecknag has quit IRC16:10
*** lprice1 has joined #openstack-cinder16:10
smcginnisSwanson: puppet apply something bad?16:12
Swansonsmcginnis, you use puppet with your CI?16:13
mtaninosmcginnis: Hi, Could you revisit to this image metadata support? Duncan-T was agreed the approach. I suppose just need few reviews. https://review.openstack.org/#/c/258649/16:14
*** lprice has quit IRC16:14
*** alkhodos has quit IRC16:14
*** nkrinner has quit IRC16:15
*** xyang1 has quit IRC16:15
smcginnismtanino: Sure, I'll take a look.16:16
mtaninosmcginnis: Thank you.16:16
sheelsmcginnis: Hi16:17
sheelsmcginnis: please this one as well : https://review.openstack.org/#/c/274589/16:17
smcginnisAdded to my open tabs...16:18
*** liverpooler has quit IRC16:18
sheelsmcginnis: thank you16:19
mtaninothen firefox will be crashed...16:20
smcginnisUsually other apps start acting weird from lack of memory available I think.16:21
smcginnisThen I know it's time to clean up some tabs. :)16:21
*** mylu has joined #openstack-cinder16:21
*** esker has quit IRC16:23
sheel hehe..16:24
*** david-lyle has joined #openstack-cinder16:24
*** esker has joined #openstack-cinder16:24
*** alonma has quit IRC16:26
*** alonma has joined #openstack-cinder16:27
sheelmtanino: hi16:29
mtaninosheel: hi,16:29
sheelmtanino: sorry, but i have conflicts on comment on https://review.openstack.org/#/c/274589/16:29
sheelmay be need to understand your view on this16:30
*** anshul has quit IRC16:30
*** merooney has quit IRC16:30
sheelmtanino: actually, this functionality will not change existing behavior16:30
sheelmtanino: rather will only eliminate one corner case16:30
sheelmtanino: so, there should not be any APIImpact16:30
sheelmtanino: pleaes let me know your views in case i am missing anything16:31
*** alonma has quit IRC16:31
mtaninosheel: I thought this changes existing behaviour(seems it's a bug to delete all same name metadata).16:32
sheelmtanino: hmm, may be I could have given some more details in commit message..16:32
*** geguileo has quit IRC16:32
mtaninosheel: let me check the definition of APIImpact again.16:33
*** alonma has joined #openstack-cinder16:33
sheelmtanino: Ok sure..16:33
mtaninosheel: but I recommend to add release note too.16:33
*** dguryanov has joined #openstack-cinder16:33
dguryanovDuncanT: Thanks!16:33
sheelmtanino: ah, but this is not changing any expected behavior...16:34
sheelmtanino: I dont think even some one create image metadata with same name16:34
sheelmtanino: this is just to cater to one corner case which was may be left in past16:34
*** belmoreira has quit IRC16:34
*** zhipeng has quit IRC16:35
openstackgerritAccela Zhao proposed openstack/cinder: Dedup code for test_cgsnapshots.py  https://review.openstack.org/27633816:35
mtaninosheel: In the past, we could delete metadatas which has same name abc, ABC, Abc etc unexpectedly, and now the bug was fixed in your patch, right?16:36
sheelmtanino: yes, exactly16:36
hemnapatrickeast, congrats!   much deserved :)16:37
*** salv-orlando has quit IRC16:37
*** alonma has quit IRC16:37
patrickeasthemna: thanks!16:37
*** mvk_ has quit IRC16:37
mtaninosheel: I thought it's better to announce this to admin via release note.16:38
*** baojg has joined #openstack-cinder16:38
sheelmtanino: ok, no issues...16:38
mtaninosheel: thank you.16:39
sheelmatnino: Thanks to you for review16:39
mtaninosheel: As for the APIImpact flag, I thought the patch changes existing behaviour even if it's corner case bug, I thought we needed APIImpact flag, but we need opinion from others. please leave it as is.16:42
*** mylu has quit IRC16:42
*** vgridnev has joined #openstack-cinder16:43
*** mylu has joined #openstack-cinder16:43
*** ndipanov has quit IRC16:44
*** baojg has quit IRC16:44
sheelmtanino : yes i too think so16:45
*** fernnest__ has joined #openstack-cinder16:45
*** fernnest__ has quit IRC16:46
*** mylu has quit IRC16:46
sheelmtanino: plz remove -1, lets have second opinion.16:46
*** mylu has joined #openstack-cinder16:47
*** mylu has quit IRC16:50
*** mylu has joined #openstack-cinder16:51
*** mylu has quit IRC16:54
*** jgriffith is now known as jgriffith_away16:58
*** shyama has quit IRC16:59
*** e0ne has quit IRC16:59
*** haomaiwa_ has quit IRC17:01
*** sgotliv has quit IRC17:01
*** haomaiwang has joined #openstack-cinder17:01
*** mylu has joined #openstack-cinder17:02
*** jistr has quit IRC17:02
*** dustins_ has quit IRC17:03
*** edtubill_ has joined #openstack-cinder17:04
*** edtubill has quit IRC17:05
*** jordanP has quit IRC17:06
*** daneyon has joined #openstack-cinder17:06
*** daneyon_ has joined #openstack-cinder17:06
*** daneyon has quit IRC17:11
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS driver: retry on Connection reset fix  https://review.openstack.org/27635717:14
*** dustins_ has joined #openstack-cinder17:14
*** jistr has joined #openstack-cinder17:16
*** jistr has quit IRC17:18
*** edtubill_ has quit IRC17:19
*** leeantho has joined #openstack-cinder17:20
*** mylu has quit IRC17:22
*** mudassirlatif has joined #openstack-cinder17:23
*** merooney has joined #openstack-cinder17:24
*** alonma has joined #openstack-cinder17:33
*** kfarr has joined #openstack-cinder17:33
openstackgerritMerged openstack/cinder: Tests: Strengthen assertFalse assertions  https://review.openstack.org/26340417:36
*** esker has quit IRC17:37
*** esker has joined #openstack-cinder17:38
*** alonma has quit IRC17:38
openstackgerritMerged openstack/cinder: mock time.sleep in Broadcom unit test  https://review.openstack.org/27595217:38
*** salv-orlando has joined #openstack-cinder17:39
*** krtaylor has quit IRC17:41
*** alonma has joined #openstack-cinder17:42
*** salv-orlando has quit IRC17:44
*** diablo_rojo has quit IRC17:46
*** alonma has quit IRC17:46
*** edtubill has joined #openstack-cinder17:46
*** jaqo78 has joined #openstack-cinder17:47
*** baojg has joined #openstack-cinder17:50
*** alkhodos has joined #openstack-cinder17:53
*** diablo_rojo has joined #openstack-cinder17:55
*** amit213 has joined #openstack-cinder17:55
*** amit213 has quit IRC17:55
*** krtaylor has joined #openstack-cinder17:56
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS driver: retry on Connection reset fix  https://review.openstack.org/27635717:56
*** amit213 has joined #openstack-cinder17:56
*** erlon has quit IRC17:56
*** baojg has quit IRC17:59
*** sheel has quit IRC17:59
*** daneyon_ has quit IRC17:59
*** timcl1 has joined #openstack-cinder18:00
*** haomaiwang has quit IRC18:01
*** haomaiwang has joined #openstack-cinder18:01
*** timcl has quit IRC18:03
*** merooney has quit IRC18:03
*** merooney has joined #openstack-cinder18:03
*** esker has quit IRC18:04
*** arch-nemesis has quit IRC18:05
*** arch-nemesis has joined #openstack-cinder18:06
*** rockeychen has quit IRC18:09
*** jungleboyj has quit IRC18:09
*** rockeychen has joined #openstack-cinder18:09
*** alonma has joined #openstack-cinder18:09
*** jdurgin1 has quit IRC18:10
*** jdurgin1 has joined #openstack-cinder18:11
*** jdurgin1 has quit IRC18:13
jbernardDuncanT: hey, did you have questions about RBD iscsi?18:14
jbernardDuncanT: i may be misremembering18:14
jbernardDuncanT: but i understadn the approach now18:14
*** alonma has quit IRC18:14
*** dustins_ has quit IRC18:15
*** alonma has joined #openstack-cinder18:17
*** alonma has quit IRC18:21
openstackgerritAnthony Lee proposed openstack/cinder: LeftHand: Implement un/manage snapshot support  https://review.openstack.org/25501518:21
*** ociuhandu has quit IRC18:22
*** alonma has joined #openstack-cinder18:24
*** salv-orlando has joined #openstack-cinder18:24
*** jungleboyj has joined #openstack-cinder18:25
*** apoorvad has joined #openstack-cinder18:26
*** salv-orlando has quit IRC18:28
*** e0ne has joined #openstack-cinder18:28
*** alonma has quit IRC18:29
*** lpetrut has quit IRC18:30
*** tongli has joined #openstack-cinder18:31
*** ChubYann has joined #openstack-cinder18:38
*** alonma has joined #openstack-cinder18:38
*** diablo_rojo has quit IRC18:41
*** alonma has quit IRC18:43
*** merooney has quit IRC18:43
*** alonma has joined #openstack-cinder18:45
*** timcl1 has quit IRC18:47
*** rlrossit has quit IRC18:47
*** rlrossit has joined #openstack-cinder18:48
*** bardia has joined #openstack-cinder18:49
*** alonma has quit IRC18:50
*** alonma has joined #openstack-cinder18:51
*** rlrossit has quit IRC18:52
*** rlrossit has joined #openstack-cinder18:53
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/27572918:53
*** esker has joined #openstack-cinder18:54
*** timcl has joined #openstack-cinder18:55
*** alonma has quit IRC18:55
smcginnispatrickeast: ping18:56
patrickeastsmcginnis: pong18:56
smcginnispatrickeast: Hey! Question for you - is your FC CI running?18:57
patrickeastsmcginnis: good question :o18:57
patrickeastsmcginnis: it died last night for a while18:57
smcginnispatrickeast: :)18:57
patrickeastsmcginnis: i *think* everything is ok now though18:57
smcginnispatrickeast: I'm getting some odd failures on mine.18:57
smcginnispatrickeast: Seems to only happen for my FC tests.18:57
patrickeastsmcginnis: oh yea?18:57
smcginnispatrickeast: Curious to see if you're seeing it to, or if it's something localized.18:57
patrickeastsmcginnis: i saw one weird error, but couldn't repro it18:57
hemna!18:58
* patrickeast digs up the link18:58
smcginnisI'm getting:18:58
smcginnisVolumeBackendAPIException: Bad or unexpected response from the storage volume backend API: Validate volume connection failed (error: 'staticmethod' object is not callable)18:58
patrickeastoh dang18:58
hemnawth18:58
patrickeastyea18:58
patrickeasthttp://openstack-logs.purestorage.com/19/275919/1/check/PureFCDriver-tempest-dsvm-volume-multipath/21e6a5e/logs/screen-c-vol.txt.gz?level=TRACE18:58
patrickeasti had some "offline" hba's, figured that was the problem18:58
smcginnisWhich would indicate a wrapper method is incorrectly defined as static.18:58
patrickeastbut maybe that was just hiding it18:58
smcginnisBut I don't see it.18:58
hemnan-cpu ?18:59
patrickeastyea i stared at the code for a while last night and didn't see how it could be working, or any recent changes18:59
smcginnispatrickeast: Hah, at least it's not just me. ;)18:59
hemnaare you guys picking up my brick patch ?18:59
hemnahttps://review.openstack.org/#/c/275943/18:59
smcginnishemna: I don't _think_ so, but maybe?18:59
patrickeastshouldn't be18:59
* patrickeast checks18:59
* smcginnis checks 218:59
hemnayah, I'd be concerned if you were for a Cinder CI run18:59
patrickeasthttp://openstack-logs.purestorage.com/19/275919/1/check/PureFCDriver-tempest-dsvm-volume-multipath/21e6a5e/logs/pip-freeze.txt.gz19:00
hemnabut it sure smells coincidental19:00
patrickeasthemna: mine was for a cinder patch19:00
smcginnisNo, doesn't look like it. I have LIBS_FROM_GIT=os-brick commented out.19:00
*** jaqo78 has quit IRC19:00
hemna-e git+https://review.openstack.org/p/openstack/os-brick@4b3dbdcb2fb52043b1f8d4df1a3702d4e5319b73#egg=os_brick19:00
patrickeasthttp://git.openstack.org/cgit/openstack/os-brick/commit/?id=4b3dbdcb2fb52043b1f8d4df1a3702d4e5319b7319:01
*** haomaiwang has quit IRC19:01
smcginnisLast requirements change was a few days ago.19:01
hemnadamn release notes killed FC!19:01
*** haomaiwang has joined #openstack-cinder19:01
hemnaI knew it19:01
smcginnisHah! :)19:01
e0ne:)19:01
patrickeasthmm so i wonder what happened to the fc tests19:02
hemnaooh19:02
*** timcl1 has joined #openstack-cinder19:02
hemnavalidate_connector19:02
hemnathat's what is puking19:03
patrickeastyea https://github.com/openstack/cinder/blob/master/cinder/volume/driver.py#L285819:03
smcginnisCalls self.driver.validate_connector_has_setting that is static, but this looks like a wrapper.19:03
patrickeastwhich is calling some static stuff...but no idea why that would be a problem19:03
patrickeastgit blame doesn't show any recent changes around here19:03
*** baumann has quit IRC19:03
*** timcl has quit IRC19:04
patrickeastand i didn't see anything suspicious in cinder, brick, or nova19:04
smcginnispatrickeast: You're at least confirming what I've seen. :)19:04
patrickeastwell, in the last few days at least19:04
hemnawhy is validate_Connector_has_setting static ?19:04
hemnaodd19:04
patrickeastdoesn't need self anywhere, so why not?19:04
hemnamaybe unit test?19:05
hemnabut statics are usually because someone outside the class needs to call it directly w/o instantiating the object19:05
hemnameh19:05
*** dansmith has quit IRC19:05
patrickeastyea19:05
*** dansmith has joined #openstack-cinder19:05
patrickeastwell... if using pycharm it will complain and tell you things should be static if they can be19:06
smcginnisNot entirely sure how python does overriding with static methods, but there is another staticmethod validate_connector_has_settings in BaseVD.19:06
hemnaI wonder if this is why my os-brick FC tests are failing as well19:06
smcginnishemna: I wouldn't be surprised.19:06
smcginnishemna: That patch looked fine to me.19:06
hemnapycharm?  what's that ? :P19:06
patrickeastsmcginnis: yea, not sure how that works... but the real confusing part is why it changed recently19:06
patrickeasthaha19:06
smcginnisYeah, I can't find what triggered this to suddenly start failing.19:07
smcginnisThat code has been like that for some time.19:07
smcginnisSo I think the code itself is fine.19:07
patrickeastsmcginnis: looking at my now-online hba jobs (that aren't failing after like 30 seconds) i think its still hitting the issues19:07
*** baumann has joined #openstack-cinder19:08
smcginnisosprofiler is in the stack, and that was updated recently, but not that recent.19:08
*** diablo_rojo has joined #openstack-cinder19:08
*** diablo_rojo has left #openstack-cinder19:08
*** diablo_rojo has joined #openstack-cinder19:08
hemnaso it looks like that will happen if someone uses self inside a static19:09
* hemna is confused19:09
patrickeastthis is the time i wish i had logstash for 3rd party ci's...19:10
hemnayah19:10
patrickeastcould see exactly when it started19:10
hemnaif that wasn't such a bitch to setup19:11
*** david-lyle has quit IRC19:11
hemna....19:11
smcginnisI've thought about setting it up locally, but eh19:11
patrickeasthaha yea seriously19:11
hemnaI tried once...19:11
patrickeastive got it on my backlog19:11
*** baumann has left #openstack-cinder19:11
patrickeastbut the instructions ive seen are daunting19:11
hemnait's almost as bad as common CI itself19:11
hemnawalk away.....don't look back.....19:11
patrickeastit would be cool if we could just point our ci's to push logs to the upstream one19:11
patrickeasti'll start setting up a fc devstack to debug this more19:13
patrickeasttakes a while since i have to steal precious hba's from the ci system without breaking anything19:13
smcginnispatrickeast: Yeah, that's my problem too. Not as easy to do a quick test as it is with iSCSI.19:14
smcginnispatrickeast: Thanks for doing that. I'll work on it here too and let you know if I find anything.19:14
openstackgerritMerged openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/27606019:15
*** angela-s has joined #openstack-cinder19:16
patrickeastsmcginnis: hemna: i filed https://bugs.launchpad.net/cinder/+bug/1541996 to track this19:20
openstackLaunchpad bug 1541996 in Cinder "FC CI jobs are failing with" error: 'staticmethod' object is not callable"" [Undecided,New]19:20
hemnaok19:21
smcginnispatrickeast: Started for me noon yesterday from what I can find in my logs.19:22
hemnahttps://review.openstack.org/#/c/275943/  that went up around 2pm yesterday19:26
hemnaI don't think that's related though19:26
smcginnishemna: Yeah, I'm pretty sure that's not it. And I think it started failing before that for me.19:27
openstackgerritAngela Smith proposed openstack/cinder: Adds support for configuring zoning in a virtual fabric  https://review.openstack.org/27540719:28
*** dustins has joined #openstack-cinder19:29
*** baojg has joined #openstack-cinder19:29
e0newhy do we have 'validate_connector' as static in base driver?19:32
hemnae0ne, heh.  that's what I was wondering as well19:32
hemnasomeone else outside it must be calling it19:32
hemna?19:32
e0neI didn't find any19:33
smcginnisI don't think it needs to be, but I know some folks think if it doesn't need to use anything self.X it should be marked static.19:33
smcginnisEh19:33
hemnathat's....just....odd19:33
smcginnisSeems really weird to me, because other languages don't work right "inheriting" static methods.19:34
hemnayah I'm not sure how that works19:34
e0nehttps://github.com/openstack/cinder/blob/master/cinder/volume/driver.py#L2858-L2867 - wtf?19:36
smcginnisHmm, oslo.messaging was release around then and we don't have a cap on that requirement so it could have been picked up.19:36
smcginnisNothing had changed in Cinder itself between my last successful run and when the failures started.19:37
smcginnisSo it's something external it appears.19:37
e0nesmcginnis: it doesn't mean that cinder is not broken19:37
smcginnise0ne: True, it could have just made something latent surface.19:37
smcginnishemna: BTW, we have 2 weeks to get anything else in os-brick before we need to do a final non-client library cut for Mitaka.19:39
smcginnisI had planned on looking what we have out there before I got pulled in to this.19:40
hemnadamn 2 weeks19:40
hemna!19:40
hemnaok19:41
smcginnishemna: Coming up fast!19:41
hemnashit, the privsep stuff needs to get in19:41
hemnaand tested19:41
hemnabut all we have is the remotefs patch19:41
hemnaI'm trying to test that today19:41
smcginnishemna: OK, cool.19:41
hemnanetwork seems to be against me though19:41
smcginnisMaybe next meeting we should do a call out to anyone willing and able to help with privsep.19:42
smcginnisThat would be a bummer if we can't get that in.19:42
hemnayah, it's a lot of work unforuntately19:42
hemnaand big risk19:42
hemnait needs a lot of testing19:42
smcginnishemna: Yeah, the more runtime on that the better.19:42
hemnayup19:42
*** mragupat has joined #openstack-cinder19:48
hemnamigrating my vagrant image builder to using my apt-mirror19:49
hemnabandersnatch finished, but I can't get pip to work with my local mirror :(19:50
smcginnishemna: Isn't there some environment variable to tell it where to go?19:51
hemnawell pip hits it, but it complains about missing stuffs.19:51
hemna:(19:51
*** alonma has joined #openstack-cinder19:51
hemnajust haven't messed with it much19:51
*** merooney has joined #openstack-cinder19:54
*** tiki45 has joined #openstack-cinder19:54
*** salv-orlando has joined #openstack-cinder19:54
*** alonma has quit IRC19:55
*** stevemar has quit IRC19:56
*** alonma has joined #openstack-cinder19:58
*** timcl1 has quit IRC20:00
*** rlrossit has quit IRC20:00
*** haomaiwang has quit IRC20:01
mc_nairsmcginnis: a while back you sent a link to a site that's basically grep OpenStack online... and then life was good.  I "misplaced" some bookmarks, do you happen to still have that link handy?20:01
*** haomaiwang has joined #openstack-cinder20:01
*** timcl has joined #openstack-cinder20:01
*** jwcroppe has quit IRC20:01
*** jwcroppe has joined #openstack-cinder20:02
*** alonma has quit IRC20:03
*** baojg has quit IRC20:03
scottdamc_nair: this? http://codesearch.openstack.org20:04
mc_nairscottda: that's the one :)20:05
*** alonma has joined #openstack-cinder20:05
mc_nairscottda: thanks! Was driving me nuts trying to remember what it was...20:05
*** jwcroppe has quit IRC20:07
*** alonma has quit IRC20:09
*** vgridnev has quit IRC20:11
*** sgotliv has joined #openstack-cinder20:12
*** andymaier has quit IRC20:13
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Make validate_connector_has_setting non-static  https://review.openstack.org/27642520:14
e0nepatrickeast, smcginnis: could you please run yours CIs against of ^^20:14
smcginnise0ne: Thanks, I was thinking of doing that. Will be interesting to see if that does make a difference.20:14
*** alonma has joined #openstack-cinder20:14
*** baumann has joined #openstack-cinder20:14
patrickeaste0ne: its in the queue for mine, should have results in a little while20:15
e0neanyway, it should not break anything, just a some cleanup to make code cleaner20:16
e0nepatrickeast, smcginnis: thanks20:16
*** xyang1 has joined #openstack-cinder20:16
* e0ne hopes that nobody will ask to write unit-tests fot that patch20:16
smcginnishehe20:17
hemnae0ne, did you find out if anyone external is using that ?20:17
hemnastatically ?20:18
e0neno:(20:18
hemnaI guess CI will say20:18
*** tiki45 has quit IRC20:18
e0nehemna: TBH. I don't know why CIs fail20:18
hemname either20:18
*** jgriffith_away is now known as jgriffith20:19
smcginnisYeah, this one has be really confused.20:19
*** alonma has quit IRC20:19
smcginnisIt has to be something external.20:19
e0nehemna: http://paste.openstack.org/show/486010/20:19
smcginnisBut why it makes it fail in this particular way, I have no idea.20:19
hemnahuh20:19
hemnagot me20:19
e0nethat's why I prefer do not use staticmethods20:20
hemnaunless it's really static20:20
hemnare: needing to be called from outside20:20
e0nethey are not needed in the most cases20:20
e0nehemna: make it simple function, noert class memb20:20
smcginnisclassmethod is another interesting one. Took me a while to figure out the difference and why you would use one or the other.20:21
e0nes/noert class memb/not a class member20:21
hemnawe like things convoluted and complicated here in Cinder.20:21
patrickeastugh devstack isn't work now, anyone else seeing http://paste.openstack.org/show/486011/20:21
smcginnis;)20:21
patrickeastworking*20:21
smcginnispatrickeast: Yeah, I was getting that yesterday.20:21
hemnaI saw that earlier today20:21
* e0ne needs to take a break20:22
patrickeastany work-around?20:22
hemnaI just spun up a new vagrant...fixed!20:22
smcginnisEr, no something different. SOrry.20:22
e0nebb tomorrow (~in 12 hours)20:22
smcginnise0ne: Have a good one.20:22
*** alonma has joined #openstack-cinder20:22
smcginnispatrickeast: Oh, no, that is it.20:22
smcginnisI was getting that no such option -c thing.20:22
smcginnisI think I did several ./clean.sh and apt-get autoremove, apt-get autoclean.20:22
smcginnisUltimately I think I had to blow away by devstack dir and re-git clone it, then it was fine.20:23
smcginniss/by devstack/my devstack/20:23
*** rockeychen has quit IRC20:26
*** rockeychen has joined #openstack-cinder20:26
*** alonma has quit IRC20:27
patrickeasthmm must have been some untracked files, git clean -fdx and re-stacking did the trick20:28
smcginnisWell, now you're just getting fancy. Show off.20:29
smcginnis:P20:29
*** alonma has joined #openstack-cinder20:30
patrickeastfancy git is the only git i know20:30
* smcginnis only know enough git to get by20:31
*** mylu has joined #openstack-cinder20:31
*** jgriffith is now known as jgriffith_away20:31
openstackgerritJacob Gregor proposed openstack/cinder: Reduce redundant code in Storwize SVC tests  https://review.openstack.org/27643320:32
smcginniseharney: Hey. e0ne's patch should probably be marked "TEST" or something.20:32
smcginniseharney: We're trying to figure out what the subtle bug is, or at least get around it.20:33
eharneysmcginnis: i think the problem is that you aren't printing the whole exception that occurs20:33
smcginniseharney: Yeah, that was going to be my next step. We're swallowing some of the details that might help.20:34
patrickeastyea20:34
eharneyyou need a stack trace where it occurs rather than where we repeat it20:34
patrickeastive got a fc devstack almost there (assuming no more suprises from devstack)20:34
*** TheJulia has quit IRC20:34
*** alonma has quit IRC20:35
*** dustins has quit IRC20:35
*** TheJulia has joined #openstack-cinder20:35
*** timcl has quit IRC20:36
*** stevemar has joined #openstack-cinder20:36
*** fthiagogv_ has joined #openstack-cinder20:37
openstackgerritHelen Walsh proposed openstack/cinder: VMAX-Replacing deprecated API EMCGetTargetEndpoints  https://review.openstack.org/24432820:37
*** alonma has joined #openstack-cinder20:38
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - necessary updates for CG changes  https://review.openstack.org/26822320:38
openstackgerritSean McGinnis proposed openstack/cinder: TEST: Troubleshoot FC test failures  https://review.openstack.org/27643620:39
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/27572920:39
smcginniseharney, patrickeast: ^^ Thought I might as well get something in the queue to get those exception details.20:39
smcginnisIt will be a bit before it makes it through my CI's queue.20:40
patrickeastsame :(20:41
patrickeastzuul is saying things are about 6 hours behind on mine20:41
*** wolsen_ has joined #openstack-cinder20:41
smcginnisI'm about brain dead on this for now anyway. I'll check tonight if there's any useful logging.20:42
*** alonma has quit IRC20:42
*** edtubill has quit IRC20:44
*** vgridnev has joined #openstack-cinder20:45
*** timcl has joined #openstack-cinder20:45
*** briancurtin_ has joined #openstack-cinder20:46
*** mattymo_ has joined #openstack-cinder20:46
*** fmccrthy_ has joined #openstack-cinder20:46
*** charlesr has joined #openstack-cinder20:47
*** alonma has joined #openstack-cinder20:47
*** mudassirlatif has quit IRC20:47
*** crose has quit IRC20:47
*** porrua has quit IRC20:47
*** isaacb has quit IRC20:47
*** briancurtin has quit IRC20:47
*** fmccrthy has quit IRC20:47
*** fthiagogv has quit IRC20:47
*** lennyb has quit IRC20:47
*** bswartz has quit IRC20:47
*** v12aml has quit IRC20:47
*** dulek has quit IRC20:47
*** wolsen has quit IRC20:47
*** mattymo has quit IRC20:47
*** haomaiwang has quit IRC20:47
*** fmccrthy_ is now known as fmccrthy20:48
*** lennyb has joined #openstack-cinder20:48
*** briancurtin_ is now known as briancurtin20:48
*** dustins has joined #openstack-cinder20:49
*** mylu has quit IRC20:49
*** baumann1 has joined #openstack-cinder20:49
*** mudassirlatif has joined #openstack-cinder20:49
*** porrua has joined #openstack-cinder20:49
*** v12aml has joined #openstack-cinder20:49
*** dulek has joined #openstack-cinder20:49
*** haomaiwang has joined #openstack-cinder20:49
*** dulek has quit IRC20:50
*** mylu has joined #openstack-cinder20:50
*** baumann has quit IRC20:50
*** bswartz has joined #openstack-cinder20:50
*** mudassirlatif has quit IRC20:51
*** porrua has quit IRC20:51
*** mudassirlatif has joined #openstack-cinder20:52
*** alonma has quit IRC20:52
*** stevemar has quit IRC20:52
*** mylu has quit IRC20:52
*** alonma has joined #openstack-cinder20:53
*** mylu has joined #openstack-cinder20:54
*** alonma has quit IRC20:58
*** merooney has quit IRC20:58
*** isaacb has joined #openstack-cinder20:59
*** apoorvad has quit IRC21:00
*** haomaiwang has quit IRC21:01
*** jungleboyj has quit IRC21:01
*** haomaiwang has joined #openstack-cinder21:01
*** alonma has joined #openstack-cinder21:01
*** e0ne has quit IRC21:04
patrickeastgot a (slightly) better stacktrace http://paste.openstack.org/show/486015/21:04
*** baojg has joined #openstack-cinder21:04
patrickeastan interesting note, i couldn't repro this just by attaching a volume, had to run the DeleteServersTestJSON tempest test21:05
smcginnisSo that makes it look like it is actually due to calling the static method. Hm21:05
eharneyi wonder if osprofiler works as expected w/ staticmethods21:05
smcginniseharney: It did make me a little leary seeing that in the stack.21:06
*** alonma has quit IRC21:06
*** raildo is now known as raildo-afk21:06
eharney'staticmethod' is a decorator, not a method21:06
patrickeastapplying e0ne's fix and seeing how that helps21:06
smcginnisYeah, it's like something is not decorating the decorator correctly.21:06
eharneysmcginnis: there's a reason i wrote a patch long ago to make it possible to not have it hooked into everything21:07
smcginniseharney: Oh? What was that?21:07
*** mylu has quit IRC21:07
eharneysmcginnis: made it possible to run Cinder w/o loading it, so we didn't have to sit here wondering about such things when looking at a failure21:07
smcginniseharney: Is that still out there?21:07
*** alonma has joined #openstack-cinder21:08
eharneysmcginnis: it got abandoned because people didn't want to go that route21:08
smcginnisHmm, this is the last commit on osprofiler:21:10
smcginnishttps://github.com/openstack/osprofiler/commit/395edeaafb96211870521baa76c68552189c1c8f21:10
*** esp_ has joined #openstack-cinder21:10
eharneyhrm21:10
*** xyang has joined #openstack-cinder21:10
patrickeastlast release was 2 days ago https://github.com/openstack/osprofiler/releases21:11
*** mylu has joined #openstack-cinder21:11
patrickeasttiming would be pretty close21:11
patrickeastthe "fix" of removing the static decorators does stop the problem from happening21:12
*** alonma has quit IRC21:12
xyangmtanino: ping21:13
patrickeastwhat channel do we bug people in about osprofiler?21:13
mtaninoxyang: pong21:13
openstackgerritMerged openstack/cinder: Don't create cgsnapshot if cg is empty  https://review.openstack.org/27602321:13
*** mylu has quit IRC21:13
smcginnisBoris should be online in the morning I think.21:13
*** esp_ has quit IRC21:14
xyangmtanino: about this bug: https://bugs.launchpad.net/cinder/+bug/153687321:15
openstackLaunchpad bug 1520102 in Cinder "duplicate for #1536873 Migration: additional volume deletion will fail" [Undecided,Fix released] - Assigned to Thang Pham (thang-pham)21:15
mtaninoxyang: yes,21:15
xyangmtanino: Peter re-tested it.  the original problem was fixed but he ran into a problem deleting temp volume21:15
xyangmtanino: versioned_objects version problem21:16
patrickeastlooks like sudo pip install 'osprofiler<0.4.0' fixes it too21:16
xyangmtanino: have you seen that21:16
*** alonma has joined #openstack-cinder21:16
mtaninoxyang: Did you have a bug# for that new one? I haven't seen it.21:16
xyangmtanino: we didn't open a bug yet.  I don't know if it is a set up issue.  the error is ERROR oslo_messaging.rpc.dispatcher RPCVersionCapError: Requested message version, 1.36 is incompatible.  It needs to be equal in major version and less than or equal in minor version as the specified version cap 1.30.21:17
mtaninoxyang: It's new for me. May be he should set up clean devstack environment and if he still meet same problem, need to report as new bug.21:19
xyangmtanino: ya, it could be the installed versioned_objects is too old.  I'll ask him to test with clean setup21:20
mtaninoxyang: I hope so. thanks.21:20
*** dguryanov has quit IRC21:20
xyangmtanino: thank you21:20
*** alonma has quit IRC21:21
patrickeastsmcginnis: eharney: so as a temporary fix for this (and to make the code less confusing) should we go ahead and remove the static decorators?21:21
patrickeastcan follow up on osprofiler later, but seems like that is easier than pinning the version on it21:21
*** timcl has quit IRC21:21
smcginnispatrickeast: Yeah, if e0ne's patch effectively works around it, that's probably the answer for now.21:22
smcginnispatrickeast: I'm writing some osprofiler unit tests right now to see if I can get it to repro the failure.21:22
*** mvk has joined #openstack-cinder21:22
smcginnispatrickeast: If I can get that (not) working I'll see if I can propose a fix.21:23
eharneythis should be something we can reproduce in cinder unit tests21:23
smcginnisosprofiler is disabled in unit tests.21:23
*** dguryanov has joined #openstack-cinder21:23
eharneysmcginnis: well, partially disabled21:24
eharneyactually maybe it is21:24
eharneyand i wrote that patch, who knew21:24
smcginnis;)21:24
*** akerr has quit IRC21:26
*** gouthamr_ has quit IRC21:28
*** rlrossit has joined #openstack-cinder21:32
*** baojg has quit IRC21:32
*** zul has joined #openstack-cinder21:42
*** cknight has quit IRC21:43
*** sgotliv has quit IRC21:44
*** fmccrthy has quit IRC21:44
*** fmccrthy has joined #openstack-cinder21:47
*** mattymo_ has quit IRC21:47
*** mattymo has joined #openstack-cinder21:49
*** mylu has joined #openstack-cinder21:51
*** xyang has quit IRC21:51
*** xyang has joined #openstack-cinder21:51
*** apoorvad has joined #openstack-cinder21:52
*** julim has quit IRC21:55
*** julim has joined #openstack-cinder21:55
*** mylu has quit IRC21:56
*** baumann1 has left #openstack-cinder21:56
*** rlrossit has quit IRC21:57
*** mylu has joined #openstack-cinder21:57
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in smbfs volume driver  https://review.openstack.org/26155121:57
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in vzstorage volume driver  https://review.openstack.org/26155021:57
openstackgerritDmitry Guryanov proposed openstack/cinder: vzstorage: add ploop volume format support  https://review.openstack.org/26224621:57
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in netapp_nfs unit test  https://review.openstack.org/26155721:57
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in remotefs.py  https://review.openstack.org/26155821:57
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in glusterfs volume driver  https://review.openstack.org/26155221:57
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in scality volume driver  https://review.openstack.org/26155321:57
openstackgerritDmitry Guryanov proposed openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs  https://review.openstack.org/27090821:57
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in quobyte volume driver  https://review.openstack.org/26155421:57
openstackgerritDmitry Guryanov proposed openstack/cinder: Use versionedobjects in nfs volume driver  https://review.openstack.org/26155521:57
hemnawhoa21:57
openstackgerritDmitry Guryanov proposed openstack/cinder: Update initialize_connection to use versionedobjects  https://review.openstack.org/23889421:57
openstackgerritDmitry Guryanov proposed openstack/cinder: vzstorage: fix creating and deleting snapshots of in-use volumes  https://review.openstack.org/27646521:57
*** mylu has quit IRC21:58
*** kfarr has quit IRC21:59
jbernardsmcginnis: heya, im thinking about submitting an outreachy project for cinder22:00
jbernardsmcginnis: and wondered if you had any bit-sized project that might fit into that format well22:00
*** haomaiwang has quit IRC22:01
*** haomaiwa_ has joined #openstack-cinder22:01
jbernardsmcginnis: i spun through the todo list we made last week, but most are a bit more involved22:01
*** julim has quit IRC22:02
*** edtubill has joined #openstack-cinder22:02
*** mudassirlatif_ has joined #openstack-cinder22:02
*** eharney has quit IRC22:03
*** mudassirlatif has quit IRC22:04
*** mudassirlatif_ is now known as mudassirlatif22:04
smcginnisjbernard: Yeah, the smaller stuff seems to get snapped up pretty quick. :)22:06
smcginnisjbernard: I'll see if I can think of something.22:06
*** krtaylor has quit IRC22:06
smcginnisjbernard: We could also bring it up in the weekly meeting to see if anyone has any ideas they'd like to hand off.22:06
jbernardsmcginnis: that's a good call22:06
smcginnisjbernard: That would be cool to get something going!22:06
jbernardsmcginnis: ill add it to the adjenda22:06
*** tpsilva has quit IRC22:06
smcginnisjbernard: Perfect, thanks!22:07
jbernardsmcginnis: dont thank me yet ;)22:07
smcginnishehe22:07
*** jamielennox|away is now known as jamielennox22:10
openstackgerritKendall Nelson proposed openstack/cinder: Remove keystonemiddleware endpoint from setup.cfg  https://review.openstack.org/27481022:12
*** cknight has joined #openstack-cinder22:13
*** isaacb has quit IRC22:14
*** alonma has joined #openstack-cinder22:16
*** vgridnev has quit IRC22:17
mtaninojbernard: Does Transition to LIO means changing the default taget from tgtd to lio?22:18
jbernardmtanino: yeah22:19
mtaninojbernard: I see.22:19
jbernardmtanino: in my mind, it's something we should do at some point22:19
jbernardmtanino: so it seems reasonable to raise it now22:19
mtaninojbernard: I like LIO, easy to use.22:20
*** krtaylor has joined #openstack-cinder22:20
jbernardmtanino: changing the default is trivial, i suspect the migration path is where the concern will lie22:20
*** alonma has quit IRC22:20
mtaninojbernard: yes, I agree on it. We should announce migration path, I guess.22:21
jbernardmtanino: im not sure if we can just document what is needed and be done, or if we need to provide as part of the upgrade a tool to move everything over22:21
jbernardmtanino: the latter scares me a little ;)22:21
mtaninojbernard: upgrade tool... wow..22:22
jbernardmtanino: it's probably something the admin will want to do gradually22:22
*** alonma has joined #openstack-cinder22:22
jbernardas attached volumes will have to reamin until unneeded22:22
jbernardexported targets for attached volumes, rather22:23
mtaninojbernard: Does Red Hat have such kind of migration tool from tgtd to LIO already?22:23
jbernardmtanino: not that im aware of22:23
jbernardmtanino: im really hoping a transition document is sufficient22:23
jbernardmtanino: but ive not propsed such a thing before, so im not sure of the requirements22:24
mtaninojbernard: better to discuss at the meeting.22:24
*** diablo_rojo has quit IRC22:24
jbernardmtanino: yeah22:24
mtaninojbernard: I got it. Thanks.22:25
jbernardmtanino: no problem22:25
*** jgregor has quit IRC22:25
*** alonma has quit IRC22:26
*** savihou has joined #openstack-cinder22:34
anteayasavihou: hello22:34
anteayasmcginnis: savihou is trying to set up a ci system with cinder22:34
savihouanteaya: hello , thanks for your help22:34
anteayasavihou: well I haven't helped much since I'm not entirely sure what it is you want to do22:34
anteayabut if you want to test cinder things meeting the cinder team is a good place to begin22:35
anteayasavihou: so to start I would look in gerrit at current cinder patches22:35
anteayahttps://review.openstack.org/#/q/project:openstack/cinder22:36
anteayaand read the build log for dsvm tempest tests that run on cinder patches22:36
anteayait helps you understand how the build process works22:36
anteayaso that when you try to troubleshoot how it is working with your system you know what step it is stuck on22:36
savihouanteaya: you tried, that's a start... I believe I'm facing an infra issue here - as im not reaching the cinder phase ..22:36
anteayaand then perhaps you can figure out why it is stuck22:37
anteayabelieve me if you want to test cinder it is best to meet the cinder team22:37
savihouanteaya - I'll dig in22:37
anteayathank you22:37
savihouanteaya - thanks22:37
anteayamany helpful folks in cinder with a whole lot of good experience running ci systems22:37
anteayawelcome22:38
savihouanteaya: :)22:38
*** salv-orl_ has joined #openstack-cinder22:39
*** salv-orlando has quit IRC22:42
*** arch-nemesis has quit IRC22:43
*** rockeychen has quit IRC22:43
*** mudassirlatif has quit IRC22:44
*** rockeychen has joined #openstack-cinder22:44
*** mylu has joined #openstack-cinder22:44
*** david-lyle has joined #openstack-cinder22:48
*** david-lyle has quit IRC22:52
*** xyang has quit IRC22:53
*** mudassirlatif has joined #openstack-cinder22:53
*** mudassirlatif has quit IRC22:55
*** angela-s has quit IRC22:57
*** charlesr has quit IRC22:59
*** mudassirlatif has joined #openstack-cinder22:59
*** xyang has joined #openstack-cinder23:00
*** haomaiwa_ has quit IRC23:01
*** mylu has quit IRC23:01
*** haomaiwang has joined #openstack-cinder23:01
*** xyang1 has quit IRC23:02
*** mudassirlatif has quit IRC23:04
*** mylu has joined #openstack-cinder23:06
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - get iscsi ip from port in existing MV  https://review.openstack.org/24599723:16
*** JoseMello has quit IRC23:18
*** alonma has joined #openstack-cinder23:23
*** alonma has quit IRC23:27
*** mudassirlatif has joined #openstack-cinder23:27
*** mgarza has quit IRC23:27
*** chlong has quit IRC23:30
*** dims_ has quit IRC23:31
*** jamielennox is now known as jamielennox|away23:31
*** dims has joined #openstack-cinder23:32
openstackgerritPatrick East proposed openstack/cinder: Allow for eradicating Pure volumes on Cinder delete  https://review.openstack.org/27591923:37
DuncanTjbernard: I did have questions about rdb iSCSI, starting with a general description of the approach, and a link to any code or tutorial - the point being that I have a way of doing it via modifying  the cinder driver, it works but is clunky. It's something I'd like to add as an option to cinder, but I'd like to use best practice not whatever hacky stuff I23:37
DuncanTcame up with23:37
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Operations and timeout issues  https://review.openstack.org/27276023:39
*** mragupat_ has joined #openstack-cinder23:45
*** xyang has quit IRC23:46
*** mragupat has quit IRC23:49
*** mragupat_ has quit IRC23:49
*** gouthamr has joined #openstack-cinder23:52
*** daneyon has joined #openstack-cinder23:58
*** daneyon_ has joined #openstack-cinder23:59

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