Wednesday, 2015-03-11

*** r-daneel has quit IRC00:01
openstackgerritVilobh Meshram proposed openstack/cinder: WIP - Quota Objects
*** e0ne has quit IRC00:06
*** Apoorva has joined #openstack-cinder00:06
*** jogo has left #openstack-cinder00:07
*** ho has joined #openstack-cinder00:09
openstackgerritThang Pham proposed openstack/cinder: Raise exception for invalid mock assert calls
openstackgerritThang Pham proposed openstack/cinder: Switch to oslo_versionedobjects
*** tsekiyama has quit IRC00:18
*** mtanino has quit IRC00:21
*** vilobhmm1 has quit IRC00:25
*** annashen has quit IRC00:26
etta_hi! i am having problem to use ceph as backend. i am receiving error... "no valid host was found. no weighed hosts available" when creating ceph volume00:33
*** david-lyle_afk has quit IRC00:34
*** IanGovett has quit IRC00:34
*** yamada-h has joined #openstack-cinder00:41
*** leeantho has quit IRC00:41
*** yamada-h has quit IRC00:45
*** topshare has joined #openstack-cinder00:51
jungleboyjSaw that on a couple of patches.01:11
*** _cjones_ has quit IRC01:11
*** liusheng has quit IRC01:12
*** liusheng has joined #openstack-cinder01:12
*** topshare has quit IRC01:15
jgriffithjungleboyj: looking01:17
*** chenleji has joined #openstack-cinder01:20
jungleboyjjgriffith: Yeah, that is what I have seen.01:20
*** david-lyle_afk has joined #openstack-cinder01:21
jgriffithjungleboyj: I can try and help track it down tomorrow, if it hasn't already been reported or even fixed01:21
jgriffithjungleboyj: would be curious if it's already known01:21
jungleboyjjgriffith: Ok.  I will search google again in a little bit and open a bug if it hasn't.01:22
jgriffithjungleboyj: might be worth a rebase just for grins01:22
jgriffithjungleboyj: the initial patch set is pretty old01:22
jgriffithjungleboyj: unless there's one in there and I just didn't see it01:22
jgriffithalthough... this is submitted against master01:22
jgriffithhemnafk: jungleboyj DuncanT smcginnis patrickeast
jgriffithI'm good with merging that noting a couple things in my comment01:24
jgriffithI'll leave it to another core for the +A01:24
jgriffithand on that note, gotta get out for tonight01:24
jungleboyjjgriffith: Thanks for the input there!01:24
jungleboyjHave a good night.01:25
*** lcurtis has joined #openstack-cinder01:26
*** annashen has joined #openstack-cinder01:26
*** topshare has joined #openstack-cinder01:27
*** topshare has quit IRC01:28
*** Lee1092 has joined #openstack-cinder01:28
*** devlaps has joined #openstack-cinder01:31
*** annashen has quit IRC01:31
*** mwichmann has quit IRC01:32
*** topshare has joined #openstack-cinder01:33
openstackgerritharsh mishra proposed openstack/cinder: Fix for  inconsistent cinder-services state change
jungleboyjjgriffith: Looks like a bug is open for it now:
openstackLaunchpad bug 1430592 in devstack "testtools-1.7.0 triggering pkg_resources.VersionConflict: (unittest2 0.5.1 (/usr/lib/python2.7/dist-packages), Requirement.parse('unittest2>=1.0.0'))" [Undecided,In progress] - Assigned to Ian Wienand (iwienand)01:42
openstackgerritJay Bryant proposed openstack/cinder: Move to the oslo.middleware library
*** cdelatte has quit IRC01:49
*** devlaps has quit IRC01:51
*** patrickeast has quit IRC01:51
*** rwsu is now known as rwsu-afk01:52
*** diegows has quit IRC01:55
*** changbl has joined #openstack-cinder01:56
*** vilobhmm has joined #openstack-cinder01:56
*** takedakn has joined #openstack-cinder02:04
*** harlowja_ is now known as harlowja_away02:09
*** Apoorva has quit IRC02:14
*** Apoorva has joined #openstack-cinder02:18
*** jcru has joined #openstack-cinder02:24
*** Apoorva has quit IRC02:26
*** haomaiwang has joined #openstack-cinder02:28
*** haomaiwang has quit IRC02:29
*** coolsvap has joined #openstack-cinder02:29
*** haomaiwang has joined #openstack-cinder02:29
*** haomaiwang has quit IRC02:29
*** haomaiwa_ has joined #openstack-cinder02:30
*** haomaiwa_ has quit IRC02:30
*** haomaiwang has joined #openstack-cinder02:31
*** Mandell has quit IRC02:33
*** jcru has quit IRC02:35
*** david-lyle_afk has quit IRC02:36
*** takedakn1 has joined #openstack-cinder02:40
*** takedakn has quit IRC02:40
*** kaisers2 has joined #openstack-cinder02:41
*** kaisers3 has joined #openstack-cinder02:42
*** kaisers1 has quit IRC02:42
*** yamada-h has joined #openstack-cinder02:42
*** kaisers has quit IRC02:43
*** I has joined #openstack-cinder02:44
*** I is now known as Guest5172302:45
*** Guest51723 is now known as centcom02:45
*** takedakn1 has quit IRC02:46
*** yamada-h has quit IRC02:47
*** jcru has joined #openstack-cinder02:48
*** centcom has left #openstack-cinder02:50
*** hemna has joined #openstack-cinder02:51
hemnajungleboyj, do we want the oslo_middleware patch in kilo?02:58
hemnathe one you submitted today?02:58
*** coolsvap is now known as coolsvap|afk03:15
*** jcru has quit IRC03:17
*** david-lyle_afk has joined #openstack-cinder03:17
*** resker has joined #openstack-cinder03:18
*** esker has quit IRC03:21
*** coolsvap|afk is now known as coolsvap03:22
*** jcru has joined #openstack-cinder03:29
*** jcru has quit IRC03:29
*** shyama has joined #openstack-cinder03:37
*** mwichmann has joined #openstack-cinder03:43
*** vilobhmm has quit IRC03:50
*** chenleji has quit IRC03:53
*** patrickeast has joined #openstack-cinder03:53
openstackgerritSHIGEMATSU Mitsuhiro proposed openstack/cinder: Fix typos in LVMVolumeDriver
*** lcurtis has quit IRC04:01
*** asselin_ has joined #openstack-cinder04:01
*** tries has quit IRC04:12
*** BharatK has joined #openstack-cinder04:14
*** tries has joined #openstack-cinder04:23
*** tries has joined #openstack-cinder04:23
*** [1]Thelo has joined #openstack-cinder04:27
*** Thelo has quit IRC04:30
*** [1]Thelo is now known as Thelo04:30
*** harlowja_at_home has joined #openstack-cinder04:35
*** dflorea has joined #openstack-cinder04:44
*** patrickeast has quit IRC04:48
openstackgerritxing-yang proposed openstack/cinder: Sort snapshots in create CG from CG snapshot
*** rushiagr_away is now known as rushiagr04:58
*** anuragpalsule has joined #openstack-cinder05:00
*** xyang1 has quit IRC05:02
*** mwichmann has quit IRC05:15
*** deepakcs has joined #openstack-cinder05:18
*** krishna_vedams has quit IRC05:21
*** BharatK has left #openstack-cinder05:25
*** BharatK has quit IRC05:25
*** nikesh_vedams has joined #openstack-cinder05:28
*** Mandell has joined #openstack-cinder05:46
*** netapp has quit IRC05:49
*** bnemec has quit IRC05:49
*** netapp has joined #openstack-cinder05:49
*** bnemec has joined #openstack-cinder05:52
*** yamada-h has joined #openstack-cinder05:55
*** yamada-h has quit IRC05:56
openstackgerritRichard Hedlind proposed openstack/cinder: Properly set number of remaining retries after failed REST call. Also, update unit test to properly test the retry path.
*** netapp has quit IRC06:02
*** BharatK has joined #openstack-cinder06:06
*** bkopilov has quit IRC06:08
*** netapp has joined #openstack-cinder06:08
*** nlevinki has joined #openstack-cinder06:08
*** pradipta has joined #openstack-cinder06:09
*** nlevinki has quit IRC06:09
*** nlevinki has joined #openstack-cinder06:09
*** bkopilov has joined #openstack-cinder06:09
*** harlowja_at_home has quit IRC06:11
*** hemna has quit IRC06:12
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex
*** stefan_amann has joined #openstack-cinder06:17
*** pradipm has joined #openstack-cinder06:20
*** pradipm has quit IRC06:21
*** pradipm has joined #openstack-cinder06:21
*** pradipm has quit IRC06:22
*** lpetrut has joined #openstack-cinder06:22
*** dflorea has quit IRC06:23
*** annashen has joined #openstack-cinder06:31
*** stefan_amann has quit IRC06:33
openstackgerritVincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails
*** annashen has quit IRC06:36
*** anshul has joined #openstack-cinder06:37
*** Maike has joined #openstack-cinder06:41
*** resker has quit IRC06:49
*** esker has joined #openstack-cinder06:50
*** takedakn has joined #openstack-cinder07:00
*** anand1 has joined #openstack-cinder07:08
*** vnagpal has joined #openstack-cinder07:09
openstackgerritYAMADA Hideki proposed openstack/cinder: Add missing backup/restore methods to Sheepdog driver
*** kaisers3 has quit IRC07:15
*** dulek has joined #openstack-cinder07:30
*** Tross1 has quit IRC07:37
*** bkopilov has quit IRC07:38
*** Tross has joined #openstack-cinder07:38
*** bkopilov has joined #openstack-cinder07:45
*** bkopilov has quit IRC07:50
*** takedakn has quit IRC07:50
*** alecv has joined #openstack-cinder07:53
*** etta_ has quit IRC07:54
*** coolsvap is now known as coolsvap|brb07:54
*** takedakn has joined #openstack-cinder07:57
*** chlong has quit IRC08:01
*** bkopilov has joined #openstack-cinder08:02
*** markus_z has joined #openstack-cinder08:04
*** e0ne has joined #openstack-cinder08:11
openstackgerritVincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails
*** topshare has quit IRC08:15
*** pradipta has quit IRC08:16
*** topshare has joined #openstack-cinder08:18
*** topshare has quit IRC08:21
*** stefan_amann has joined #openstack-cinder08:23
*** topshare has joined #openstack-cinder08:23
*** Mandell has quit IRC08:23
openstackgerritMichal Dulko proposed openstack/cinder: Backup object
*** coolsvap|brb is now known as coolsvap08:26
*** jungleboyj has quit IRC08:28
*** TobiasE has joined #openstack-cinder08:29
*** TobiasE has quit IRC08:31
*** jungleboyj has joined #openstack-cinder08:32
*** HoloIRCUser2 has joined #openstack-cinder08:34
*** lpetrut has quit IRC08:39
openstackgerritVincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails
openstackgerritVincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails
openstackgerritVincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails
openstackgerritLena Novokshonova proposed openstack/cinder: Add notifications about snapshot.update.*
*** ndipanov has joined #openstack-cinder08:47
*** bkopilov has quit IRC08:49
openstackgerritMichal Dulko proposed openstack/cinder: Add is_ready method to scheduler driver
openstackgerritMichal Dulko proposed openstack/cinder: Add waiting for the driver to SchedulerManager
openstackgerritMichal Dulko proposed openstack/cinder: Allow scheduler to receive volume stats when starting service
*** bkopilov has joined #openstack-cinder08:51
*** topshare has quit IRC08:53
*** topshare has joined #openstack-cinder08:55
*** topshare has quit IRC08:59
*** stefan_amann has quit IRC08:59
openstackgerritYAMADA Hideki proposed openstack/cinder: Add missing backup/restore methods to Sheepdog driver
*** topshare has joined #openstack-cinder09:01
*** topshare has quit IRC09:03
*** topshare has joined #openstack-cinder09:05
*** lpetrut has joined #openstack-cinder09:07
*** ronis has joined #openstack-cinder09:10
*** ho has quit IRC09:15
*** afazekas_ has joined #openstack-cinder09:16
openstackgerritTina Tang proposed openstack/cinder: Attach/detach batch processing in VNX driver
*** topshare has quit IRC09:20
*** pradipta has joined #openstack-cinder09:24
*** jistr has joined #openstack-cinder09:25
*** e0ne has quit IRC09:27
*** annashen has joined #openstack-cinder09:34
openstackgerritMichal Dulko proposed openstack/cinder: Remove global mocking from
openstackgerritVincent Hou proposed openstack/cinder: Retype in driver should have the correct parameters
*** annashen has quit IRC09:39
*** ankit_ag has joined #openstack-cinder09:40
*** anand1 has quit IRC09:45
*** anand has joined #openstack-cinder09:46
*** jistr has quit IRC09:50
openstackgerritVincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails
*** jistr has joined #openstack-cinder09:52
*** nshaikh has joined #openstack-cinder09:56
*** yuriy_n17 has joined #openstack-cinder09:57
openstackgerritGorka Eguileor proposed openstack/python-cinderclient: Add cache to Managers' find/findall methods
*** HoloIRCUser2 has quit IRC10:01
*** HoloIRCUser1 has joined #openstack-cinder10:02
*** tbarron has joined #openstack-cinder10:03
*** TobiasE has joined #openstack-cinder10:03
*** deepakcs has quit IRC10:04
*** TobiasE has quit IRC10:05
*** e0ne has joined #openstack-cinder10:16
*** chlong has joined #openstack-cinder10:23
*** stefan_amann has joined #openstack-cinder10:28
*** jistr has quit IRC10:38
*** HoloIRCUser2 has joined #openstack-cinder10:38
*** jistr has joined #openstack-cinder10:39
*** HoloIRCUser1 has quit IRC10:39
*** kmartin has quit IRC10:41
*** takedakn has quit IRC10:42
*** mdenny has quit IRC10:43
*** haomaiwang has quit IRC10:49
*** asselin has quit IRC10:52
nikesh_vedamsi am using devstack with lvm backends in ubuntu 14.0410:55
nikesh_vedamsi want to use a volume group created by me in cinder.conf10:56
*** HoloIRCUser2 has quit IRC10:56
nikesh_vedamsbut its not taking10:56
*** TobiasE has joined #openstack-cinder10:56
*** TobiasE has quit IRC10:59
*** lpetrut has left #openstack-cinder11:05
*** anshul has quit IRC11:06
*** IanGovett has joined #openstack-cinder11:06
*** anshul has joined #openstack-cinder11:09
*** stefan_amann has quit IRC11:14
openstackgerritMarcus V R Nascimento proposed openstack/cinder: Error trying to delete snapshots on Hitachi driver
*** e0ne is now known as e0ne_11:21
*** e0ne_ is now known as e0ne11:26
*** timcl has joined #openstack-cinder11:29
e0nenikesh_vedams: did you restsrt cinder-volume?11:30
*** anshul has quit IRC11:30
*** anshul has joined #openstack-cinder11:42
*** e0ne is now known as e0ne_11:51
*** esker has quit IRC11:55
*** e0ne_ has quit IRC11:56
*** anshul has quit IRC11:59
openstackgerritBob Callaway proposed openstack/cinder: Fix LUN misalignment issue with NetApp iSCSI drivers
nikesh_vedamse0ne: yes12:02
nikesh_vedamsi am using devstack12:03
nikesh_vedamsso rejoined12:03
nikesh_vedamsand restarted12:03
nikesh_vedamscinder volume12:03
nikesh_vedamsbut even though same problem12:03
nikesh_vedamsits not taking explicit volume group12:03
*** erlon has joined #openstack-cinder12:04
*** anshul has joined #openstack-cinder12:05
*** jcru has joined #openstack-cinder12:05
*** timcl has quit IRC12:09
nikesh_vedamse0ne DuncanT :
*** jcru has quit IRC12:11
*** pradipta has quit IRC12:11
*** nlevinki has quit IRC12:12
*** jcru has joined #openstack-cinder12:12
*** nlevinki has joined #openstack-cinder12:14
*** markvoelker has joined #openstack-cinder12:14
*** cdelatte has joined #openstack-cinder12:14
*** delattec has joined #openstack-cinder12:14
*** akerr has joined #openstack-cinder12:14
*** Administrator_ has joined #openstack-cinder12:17
*** e0ne has joined #openstack-cinder12:18
nikesh_vedams$ sudo cinder-rootwrap /etc/cinder/rootwrap.conf env LC_ALL=C LVM_SYSTEM_DIR=/etc/cinder vgs --noheadings --unit=g -o name,size,free,lv_count,uuid --separator : --nosuffix mystack-vg   Volume group "mystack-vg" not found12:19
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Support over subscription in thin provisioning
*** Mandell has joined #openstack-cinder12:24
*** BharatK has quit IRC12:27
*** Mandell has quit IRC12:28
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Support over subscription in thin provisioning
openstackgerritGrzegorz Grasza (xek) proposed openstack/cinder: Service object
*** annashen has joined #openstack-cinder12:36
*** jyoti-ranjan has joined #openstack-cinder12:37
*** dulek has quit IRC12:38
*** annashen has quit IRC12:41
*** lpetrut has joined #openstack-cinder12:44
*** esker has joined #openstack-cinder12:44
*** timcl has joined #openstack-cinder12:44
*** rushiagr is now known as rushiagr_away12:46
*** esker has quit IRC12:47
*** tbarron has quit IRC12:47
*** esker has joined #openstack-cinder12:50
nikesh_vedamsDuncanT: there?12:50
nikesh_vedamse0ne : there?12:50
*** esker has quit IRC12:51
*** bswartz has quit IRC12:51
*** jyoti-ranjan has quit IRC12:51
*** TobiasE has joined #openstack-cinder12:53
*** mwichmann has joined #openstack-cinder12:54
*** TobiasE has quit IRC12:54
*** TobiasE has joined #openstack-cinder12:55
*** anand has quit IRC12:55
*** TobiasE has quit IRC12:59
*** anuragpalsule has quit IRC13:00
*** Yogi1 has joined #openstack-cinder13:01
*** ganso_ has joined #openstack-cinder13:02
*** rushiagr_away is now known as rushiagr13:05
*** esker has joined #openstack-cinder13:10
*** aix has joined #openstack-cinder13:13
openstackgerritRichard Hedlind proposed openstack/cinder: Decrement remaining retries after failed REST call.
*** Longgeek has joined #openstack-cinder13:19
*** rushiagr is now known as rushiagr_away13:20
*** dustins has joined #openstack-cinder13:22
*** mtanino_p has joined #openstack-cinder13:22
*** nellysmitt has joined #openstack-cinder13:24
*** bswartz has joined #openstack-cinder13:25
*** mriedem has joined #openstack-cinder13:27
*** tbarron has joined #openstack-cinder13:27
*** mtanino_p has quit IRC13:29
*** dulek has joined #openstack-cinder13:30
nikesh_vedamshi anyone online13:31
nikesh_vedamsdid any one face this issue13:31
nikesh_vedamsvolume group not found13:32
nikesh_vedamsfor lvm'13:32
*** jungleboyj has quit IRC13:33
*** stefan_amann has joined #openstack-cinder13:34
*** mtanino_p has joined #openstack-cinder13:38
*** mtanino_p has quit IRC13:38
*** nellysmitt has quit IRC13:39
*** nellysmi_ has joined #openstack-cinder13:42
*** hemna has joined #openstack-cinder13:43
*** Longgeek has quit IRC13:43
*** nellysmi_ has quit IRC13:44
*** mtanino_p has joined #openstack-cinder13:45
*** amoturi_ has quit IRC13:46
*** amoturi_ has joined #openstack-cinder13:46
*** annegentle has joined #openstack-cinder13:49
*** thangp has joined #openstack-cinder13:50
*** mtanino_p has quit IRC13:52
*** eharney has joined #openstack-cinder13:55
*** Longgeek has joined #openstack-cinder13:55
*** r-daneel has joined #openstack-cinder13:55
*** TobiasE has joined #openstack-cinder13:56
*** e0ne has quit IRC13:57
*** e0ne has joined #openstack-cinder13:57
*** TobiasE has quit IRC13:58
*** changbl has quit IRC13:58
*** stefan_amann has quit IRC13:59
*** topshare has joined #openstack-cinder14:00
*** topshare has quit IRC14:00
*** tellesnobrega_ has joined #openstack-cinder14:00
*** timcl has left #openstack-cinder14:04
*** xyang has joined #openstack-cinder14:05
*** rushiagr_away is now known as rushiagr14:06
*** topshare has joined #openstack-cinder14:06
*** lpetrut has quit IRC14:06
*** topshare has quit IRC14:06
*** topshare has joined #openstack-cinder14:07
*** changbl has joined #openstack-cinder14:09
*** mdbooth has quit IRC14:09
*** shyama has quit IRC14:09
*** alonmarx_ has quit IRC14:10
*** dulek has quit IRC14:11
*** mdbooth has joined #openstack-cinder14:14
*** dulek has joined #openstack-cinder14:15
openstackgerritEric Harney proposed openstack/cinder: Tests: Harden fake_notifier asserts
*** timcl has joined #openstack-cinder14:19
*** lpetrut has joined #openstack-cinder14:19
*** anand1 has joined #openstack-cinder14:22
*** topshare has quit IRC14:23
*** anand1 has left #openstack-cinder14:24
*** topshare has joined #openstack-cinder14:25
*** esker has quit IRC14:26
*** HoloIRCUser1 has joined #openstack-cinder14:27
*** jungleboyj has joined #openstack-cinder14:27
*** [1]Thelo has joined #openstack-cinder14:27
*** Thelo has quit IRC14:30
*** [1]Thelo is now known as Thelo14:30
*** ankit_ag has quit IRC14:30
*** lcurtis has joined #openstack-cinder14:32
*** e0ne is now known as e0ne_14:32
*** e0ne_ is now known as e0ne14:33
*** eharney has quit IRC14:34
*** mtanino has joined #openstack-cinder14:36
*** dulek has quit IRC14:37
*** annashen has joined #openstack-cinder14:38
*** esker has joined #openstack-cinder14:38
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator
*** jungleboyj has quit IRC14:38
*** BharatK has joined #openstack-cinder14:42
*** marcusvrn has joined #openstack-cinder14:43
*** annashen has quit IRC14:43
*** marcusvrn1 has quit IRC14:45
*** TobiasE has joined #openstack-cinder14:46
*** TobiasE has quit IRC14:48
openstackgerritAnton Arefiev proposed openstack/cinder: Add entry create and cast tasks to manage workflow
openstackgerritAnton Arefiev proposed openstack/cinder: Fix lvm manage existing volume
*** mdenny has joined #openstack-cinder14:49
*** stefan_amann has joined #openstack-cinder14:52
*** fischerw has joined #openstack-cinder14:54
*** timcl1 has joined #openstack-cinder14:54
*** timcl has quit IRC14:56
*** marcusvrn1 has joined #openstack-cinder14:56
*** hemnafk is now known as hemna14:56
*** marcusvrn has quit IRC14:58
*** topshare has quit IRC14:58
*** Mandell has joined #openstack-cinder14:58
*** dannywilson has joined #openstack-cinder15:02
*** dannywilson has quit IRC15:02
*** marcusvrn1 has quit IRC15:03
*** dannywilson has joined #openstack-cinder15:03
*** eharney has joined #openstack-cinder15:03
DuncanTMorning, hemna15:04
*** Yogi11 has joined #openstack-cinder15:05
*** tellesnobrega_ has quit IRC15:07
*** Yogi1 has quit IRC15:08
*** bkopilov has quit IRC15:09
*** marcusvrn has joined #openstack-cinder15:09
*** zhipeng has joined #openstack-cinder15:10
*** alonmarx has joined #openstack-cinder15:11
nikesh_vedamsDuncanT: hi busy?15:12
*** kmartin has joined #openstack-cinder15:12
DuncanTnikesh_vedams: Yes, sorry#15:13
*** alonmarx has quit IRC15:13
*** marcusvrn has quit IRC15:14
nikesh_vedamsi am struggling in this
nikesh_vedamsoh i got the mistake15:16
nikesh_vedamssudo cinder-rootwrap /etc/cinder/rootwrap.conf env LC_ALL=C LVM_SYSTEM_DIR=/etc/lvm vgs --noheadings --unit=g -o name,size,free,lv_count,uuid --separator : --nosuffix mystack-vg   mystack-vg:50.00:50.00:0:5hS1O9-oxVX-WlRE-8krf-Q1Tt-Avor-zcfdX315:17
nikesh_vedamsthis is working15:17
nikesh_vedamsbut sudo cinder-rootwrap /etc/cinder/rootwrap.conf env LC_ALL=C LVM_SYSTEM_DIR=/etc/cinder vgs --noheadings --unit=g -o name,size,free,lv_count,uuid --separator : --nosuffix mystack-vg   Volume group "mystack-vg" not found15:17
nikesh_vedamsthis not working15:17
nikesh_vedamsMight be this is a bug15:18
nikesh_vedamsin devstack15:18
jgriffithnikesh_vedams: it would appear that the /etc/cinder lvm.conf has a filter against your dev15:18
nikesh_vedamswhich is not taking explicitly created volume-group15:19
jgriffithnikesh_vedams: open the file and look for "filter", you'll see that only certain devices are queried15:19
mtaninonikesh_vedams: you should check LVM filter in /etc/cinder/lvm.conf15:19
jgriffithnikesh_vedams: make it match /etc/lvm.conf if you don't understand lvm filtering15:19
jgriffithie copy/paste the entire file15:19
jgriffithmtanino: :)15:20
nikesh_vedamsmtanino jgriffith : ya sure thanks15:20
*** thingee has joined #openstack-cinder15:21
mtaninonikesh_vedams: I think old bash have a bug(in fedora20), so devstack can't proper LVM filter when we use devstack.15:21
mtaninonikesh_vedams: I met same situation.15:21
mtaninocan't/can't create/15:22
*** bkopilov has joined #openstack-cinder15:23
*** TobiasE has joined #openstack-cinder15:24
*** TobiasE has quit IRC15:25
*** Administrator_ has quit IRC15:26
*** Administrator_ has joined #openstack-cinder15:27
*** david-lyle_afk is now known as david-lyle15:28
*** thingee_ has joined #openstack-cinder15:29
*** mtanino is now known as mtanino_away15:30
*** jdurgin1 has joined #openstack-cinder15:31
*** Mandell has quit IRC15:31
*** thingee has quit IRC15:32
*** dulek has joined #openstack-cinder15:36
*** maune has quit IRC15:37
*** Apoorva has joined #openstack-cinder15:38
*** jungleboyj has joined #openstack-cinder15:38
*** zhipeng has quit IRC15:39
thingee_jungleboyj: can we get some to +2/A that systemz one?15:39
thingee_hemna has a minus 1 on it though15:39
jungleboyjthingee_: Yeah, I wanted to +2/A it but thought hemna should do it.15:40
jungleboyjI pinged him earlier this morning.15:40
*** dannywilson has quit IRC15:40
thingee_ok, has the new cert tests been posted?15:40
markus_zthingee_: I'm not quite sure on what the -1 is based. The driver certification is added in the bug description.15:40
*** deepakcs has joined #openstack-cinder15:41
thingee_markus_z: hemna noted there were two failures in it ;)15:41
*** dulek has quit IRC15:41
jungleboyjmarkus_z: thingee_ Yes.  The new results were posted.  hemna Was waiting for the new gate run to pass and there was an issue in the gate last night.15:41
markus_zthingee_: Yes, that's true. We had to add another patch in tempest because we (currently) have to use an image which is 2GB in size.15:41
thingee_jungleboyj, markus_z: excellent, sorry just catching up on this patch. Yeah sounds good to me if hemna gives the OK15:42
markus_zAfter using that patch we were able to run it successfully.15:42
thingee_thanks :)15:42
*** afazekas_ has quit IRC15:42
markus_zthingee_: np :)15:42
jungleboyjthingee_: No problem.  Tried to hold down the fort yesterday.15:42
thingee_jungleboyj: all of you did an awesome job15:42
markus_zFor later reference:
jungleboyjOh good, we got this one in too.
openstackgerritGrzegorz Grasza (xek) proposed openstack/cinder: Service object
thingee_jungleboyj: yeah I just went through and updated k-3 page, etc. lookin' good!15:43
*** thingee_ is now known as thingee15:43
*** Longgeek has quit IRC15:45
*** nellysmitt has joined #openstack-cinder15:45
*** marcusvrn has joined #openstack-cinder15:45
*** marcusvrn1 has joined #openstack-cinder15:47
hemnaok I'm looking at 149256 again15:47
thingeejungleboyj: I saw you and hemna talking about it earlier, but yes I'm not accepting any more feature merges. especially if it's not targeted15:47
thingeeso feel free to -215:47
thingeewe'll make an exception for this since it's very close if hemna is fine with the test results.15:48
hemnathingee, so there is also a companion patch for os-brick that I had as a requirement as well15:48
e0neimo, it's not fair to not merge #131182 (shadow tables) with reason "a feature freeze deadline passed" :(15:48
markus_zhemna: That os-brick patch is here:
hemnamarkus_z, thank you15:49
*** marcusvrn has quit IRC15:49
*** victorfeitosa has joined #openstack-cinder15:49
*** nellysmitt has quit IRC15:50
*** xyang has quit IRC15:50
flip214for devstack, do I have to use trusty, or is vivid an option, too?15:50
thingeee0ne: well it's what we all agreed on.15:50
*** xyang1 has joined #openstack-cinder15:50
e0nehemna: my congrantulations! your patch is almost merged15:50
thingeee0ne: not sure why that's not fair.15:51
thingeee0ne: we need time to just let things sit and catch issues before the cut15:51
hemnae0ne, thank you.  It's been a LONG time coming.15:51
e0nethingee: it was holded on to make others patches less complicated and w/o shadow migrations15:51
markus_zhemna: To give you a heads up, I had issues to add the "depends-on" keyword because of a mishap with assigning the same change-id to two different patch sets.15:52
thingeee0ne: ah that's true15:52
hemnamarkus_z, yah I just read that15:52
thingeee0ne: we'll talk about it in the meeting15:52
hemnae0ne, did you give any more thought to my idea of an alternate way of doing shadow tables ?15:52
*** marcusvrn has joined #openstack-cinder15:52
hemnainstead of forcing it on devs in db migration scripts, etc.15:52
flip214or can I even use debian testing/unstable as a devstack base?15:52
e0nethingee: i was agree with deadlines since we started discussion on meeting15:53
e0nehemna: i'm going to make poc for oslo.db but have not time to it today15:53
markus_zhemna: I talked with jeblair in the infra channel, he too couldn't find a way to fix this. I'm sorry for that.15:53
hemnae0ne, ok cool.   we should talk about it in Vancouver15:53
hemnamarkus_z, np15:54
*** marcusvrn1 has quit IRC15:54
*** coolsvap is now known as coolsvap|afk15:54
e0nehemna: ok, sounds good15:54
e0nethingee, hemna: imo, fair reason to not merge py patch is "we need to make developers life less complicated for new migrations development"15:55
*** rwsu-afk is now known as rwsu15:55
hemnae0ne, yah that would be why I would -2 it at this point.15:55
*** diemt has joined #openstack-cinder15:56
hemnajungleboyj, ok done15:57
hemnamarkus_z, good to go15:57
*** ronis has quit IRC15:57
markus_zhemna: Yeah, thank you! :)15:58
*** tellesnobrega has quit IRC15:58
hemnathanks for the work man.  appreciate it15:58
thingeegood work everyone :)15:58
hemnamarkus_z, I know doing the patch twice (cinder vs. os-brick) is a bummer, but it's the only way we can keep those 2 in sync, until L.15:58
markus_zjungleboyj: Thanks for the time and effort you spent too!15:59
thingeehemna: +115:59
*** tellesnobrega has joined #openstack-cinder15:59
markus_zhemna: Will the transition to os-brick happen in Liberty? I thought I have read something on the ML which said it is planned for Kilo?15:59
hemnaok so I'll post another follow up to multiattach and update the unit tests to change the order, etc.15:59
jungleboyjhemna: Thank you!15:59
*** ebalduf has joined #openstack-cinder15:59
jungleboyjmarkus_z: Thank you for working with us on this.16:00
*** Yogi11 has quit IRC16:00
hemnamarkus_z, the plan is to migrate to os-brick in L.16:00
hemnamarkus_z, I have a WIP patch up already for cinder.   I'll just have to rebase it and such.16:00
e0nehemna: ok. it would be fair enough...16:00
hemnawe should do an official release of os-brick when we ship cinder for K16:00
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements
hemnaso os-brick is in pypi for L.16:00
*** patrickeast has joined #openstack-cinder16:00
markus_zjungleboyj: Anytime again16:01
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add multiattach capabilities injection
markus_zhemna: Yes, I have seen it. I have knowledge of this area in Nova as well if there is help needed.16:01
hemnamarkus_z, nice.16:01
hemnamarkus_z, back in Icehouse I had a WIP patch in nova that removed the internals of libvirt volume drivers and used brick (As it was then).16:02
*** TobiasE has joined #openstack-cinder16:02
jgriffithwe meeting?16:02
tbarronjgriffith: yup16:03
*** _cjones_ has joined #openstack-cinder16:03
*** TobiasE has quit IRC16:03
*** rushil has joined #openstack-cinder16:08
*** victor-feitosa has joined #openstack-cinder16:11
flip214hemna: thanks for the offer, but I've got an appointment in 20 mins. perhaps tomorrow at ~2000 CET?16:12
*** victorfeitosa has quit IRC16:12
*** thrawn01 has quit IRC16:13
*** nkrinner has quit IRC16:13
*** briancurtin has quit IRC16:13
*** rmesta has joined #openstack-cinder16:14
*** marcusvrn has quit IRC16:14
hemnaflip214, I'll be here during normal PST hours :P16:14
*** rmesta has left #openstack-cinder16:14
*** marcusvrn has joined #openstack-cinder16:14
*** russellb has quit IRC16:14
*** zigo_ has quit IRC16:14
*** jbernard has quit IRC16:14
*** zigo_ has joined #openstack-cinder16:14
*** asselin_ has quit IRC16:14
*** jbernard has joined #openstack-cinder16:15
*** russellb has joined #openstack-cinder16:15
*** asselin_ has joined #openstack-cinder16:15
*** thrawn01 has joined #openstack-cinder16:15
flip214yeah, that's why I suggested that time ;)16:16
flip214thanks, will get back to you tomorrow.16:16
*** nkrinner has joined #openstack-cinder16:17
*** fischerw has quit IRC16:18
*** fischerw has joined #openstack-cinder16:18
*** annegentle has quit IRC16:18
*** briancurtin has joined #openstack-cinder16:19
*** anshul has quit IRC16:20
*** kallebe has joined #openstack-cinder16:22
*** marcusvrn1 has joined #openstack-cinder16:22
*** alecv has quit IRC16:22
*** marcusvrn has quit IRC16:23
*** marcusvrn has joined #openstack-cinder16:25
*** nshaikh has left #openstack-cinder16:26
*** esker has quit IRC16:26
*** leeantho has joined #openstack-cinder16:27
*** marcusvrn1 has quit IRC16:27
*** emagana has quit IRC16:28
*** emagana has joined #openstack-cinder16:28
*** mtanino_away has quit IRC16:32
*** emagana has quit IRC16:33
*** timcl1 has quit IRC16:33
*** dulek has joined #openstack-cinder16:33
*** rcallawa has joined #openstack-cinder16:33
*** timcl has joined #openstack-cinder16:33
*** vilobhmm has joined #openstack-cinder16:34
*** amann has joined #openstack-cinder16:34
*** stefan_amann has quit IRC16:37
jgriffithvishy: that doesn't look too bad, thanks for jumping in16:38
hemnajaypipes, so the cinder multiattach code just merged.  :)16:39
jgriffithvishy: I'd really like to go this route rather than introduce the public/private thing on all Cinder objects16:39
*** nlevinki has quit IRC16:39
*** annashen has joined #openstack-cinder16:40
*** annashen has quit IRC16:45
*** jdurgin1 has quit IRC16:45
*** Longgeek has joined #openstack-cinder16:45
jaypipeshemna: I saw that :) yay!16:47
*** asselin__ has joined #openstack-cinder16:48
*** annashen has joined #openstack-cinder16:48
*** TobiasE has joined #openstack-cinder16:50
*** asselin_ has quit IRC16:52
*** vilobhmm has quit IRC16:52
openstackgerritMichal Dulko proposed openstack/cinder: Backup object
*** openstack has joined #openstack-cinder16:55
*** rhagarty__ has joined #openstack-cinder16:56
*** takedakn has joined #openstack-cinder16:57
*** timcl1 has joined #openstack-cinder16:58
*** timcl has quit IRC16:58
*** cbader has joined #openstack-cinder16:59
morganfainbergthingee, let me know next time your in SoCal.16:59
*** patrickeast has quit IRC17:00
thingeemorganfainberg: I will be tomorrow actually!17:00
thingeemorganfainberg: leaving philly now17:00
*** TobiasE has quit IRC17:00
*** rcallawa has quit IRC17:01
morganfainbergthingee, coffee meetup when youre here? or... whiskey?17:01
marcusvrnthangp: xyang1: eharney: hi! I posted the patch fixing the commit message as eharney asked me. would be great if you guys take a look on it :)17:01
thingeemorganfainberg: sounds good17:01
*** Longgeek has quit IRC17:02
*** Yogi1 has joined #openstack-cinder17:04
jungleboyjhemna: afk beating?17:04
* jungleboyj doesn't want to know.17:04
*** dulek has quit IRC17:05
*** thingee has quit IRC17:05
*** takedakn has quit IRC17:05
* jungleboyj is laughing uncontrollably. Been that kind of day.17:06
*** dannywilson has joined #openstack-cinder17:06
*** annashen has quit IRC17:07
*** annashen has joined #openstack-cinder17:07
*** esker has joined #openstack-cinder17:08
*** timcl1 has quit IRC17:08
*** takedakn has joined #openstack-cinder17:09
openstackgerritxing-yang proposed openstack/cinder: Sort snapshots in create CG from CG snapshot
*** Adriano_ has joined #openstack-cinder17:11
jungleboyjsmcginnis: Are you too having the urge to just go sit out in the sun?17:11
smcginnisjungleboyj: Oh yeah. It's lovely. :)17:12
smcginnisjungleboyj: Nice there too I assume? Heard it's actually warmer here than in Austin right now.17:13
jungleboyjsmcginnis: Indeed.  Almost couldn't come back in after lunch yesterday.  Spent part of the afternoon working from my deck.  :-)17:13
*** dannywilson has quit IRC17:13
smcginnisjungleboyj: Almost tempted to ride my motorcycle in the cold this morning so I could ride home this afternoon when it's nice. :)17:13
*** bswartz has quit IRC17:13
*** harlowja_away is now known as harlowja_17:14
jungleboyjAlmost 50 on my deck right now.  Got to 60 yesterday.  Supposed to do taht again today.17:14
*** mgagne_PHL is now known as mgagne17:14
jungleboyjsmcginnis: Motorcycle!?!17:14
smcginnisjungleboyj: Might have to take off early today!17:14
jungleboyjsmcginnis: +217:15
*** jordanP has joined #openstack-cinder17:16
*** bswartz has joined #openstack-cinder17:16
*** HoloIRCUser1 has quit IRC17:16
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Support over subscription in thin provisioning
*** e0ne is now known as e0ne_17:18
*** leeantho has quit IRC17:19
*** amann has quit IRC17:19
*** resker has joined #openstack-cinder17:20
*** vilobhmm has joined #openstack-cinder17:21
*** timcl has joined #openstack-cinder17:21
*** chlong has quit IRC17:21
*** asselin__ has quit IRC17:22
*** markus_z has quit IRC17:23
*** esker has quit IRC17:24
*** yuriy_n17 has quit IRC17:25
*** dflorea has joined #openstack-cinder17:26
*** e0ne_ has quit IRC17:29
*** _cjones_ has quit IRC17:30
*** e0ne has joined #openstack-cinder17:30
*** HoloIRCUser2 has joined #openstack-cinder17:31
*** leeantho has joined #openstack-cinder17:31
*** timcl has quit IRC17:33
e0nezuul shows 167 changes in pipeline :-(17:33
*** ronis has joined #openstack-cinder17:34
*** jdandrea has quit IRC17:35
*** jdandrea has joined #openstack-cinder17:35
*** bkopilov has quit IRC17:35
*** kmartin has quit IRC17:37
*** takedakn has quit IRC17:37
*** _cjones_ has joined #openstack-cinder17:38
*** asselin has joined #openstack-cinder17:38
openstackgerritAlex Meade proposed openstack/cinder: Add tracing facility to NetApp ONTAP drivers
*** bkopilov has joined #openstack-cinder17:39
*** BharatK has quit IRC17:39
*** kmartin has joined #openstack-cinder17:40
*** Mandell has joined #openstack-cinder17:40
*** annashen has quit IRC17:41
*** e0ne has quit IRC17:41
*** annashen has joined #openstack-cinder17:41
*** annegentle has joined #openstack-cinder17:41
*** rhagarty_ has quit IRC17:42
*** rhagarty__ has quit IRC17:42
*** rhagarty has quit IRC17:42
*** devlaps has joined #openstack-cinder17:42
*** nellysmitt has joined #openstack-cinder17:46
*** nellysmitt has quit IRC17:50
*** mtanino has joined #openstack-cinder17:52
*** lpetrut has quit IRC17:55
*** bkopilov has quit IRC17:55
*** kallebe has quit IRC17:55
*** kallebe has joined #openstack-cinder17:56
*** bkopilov has joined #openstack-cinder17:57
openstackgerritVilobh Meshram proposed openstack/cinder: WIP - Quota Objects
*** marcusvrn has quit IRC18:00
hemnajungleboyj, we are having CA winter again today.   overcast, drizzle, 70degrees.   I'm sure some folks are putting sandbags up.18:01
*** Longgeek has joined #openstack-cinder18:02
jungleboyj:-)  We are having spring here, 50 and Sunny.18:02
jungleboyj60's the rest of the week.  :-)  Hoo ray!18:03
DuncanT26 here yesterday18:04
jgriffithDuncanT: Celsius :)18:04
DuncanTHigh seventies in your funny money18:04
jungleboyjDuncanT: Was just converting.  Nice.18:05
*** emagana has joined #openstack-cinder18:05
*** angela-s has joined #openstack-cinder18:06
jungleboyjhemna: Why sandbags?  Because it is drizzling?18:06
*** Longgeek has quit IRC18:09
openstackgerritEric Harney proposed openstack/cinder: Fix sqlalchemy reuse in multi-backend children
hemnajungleboyj, yah18:10
hemnajungleboyj, it gets wet, people freak out, news stations talk about the impending mega storm, etc18:10
eharneythis bug stinks ^18:10
*** Maike has quit IRC18:11
smcginnisSwanson: ^^18:11
*** rhagarty has joined #openstack-cinder18:12
*** rhagarty_ has joined #openstack-cinder18:12
*** takedakn has joined #openstack-cinder18:12
*** kbyrne has quit IRC18:13
smcginniseharney: We want that on every init?18:13
*** kbyrne has joined #openstack-cinder18:14
eharneysmcginnis: all except the first one, i think, and it doesn't have any real penalty there AFAIK18:14
smcginniseharney: So the idea is at the beginning it cleans up anything left over?18:15
eharneysmcginnis: see comments in the LP bug from Mike Bayer18:15
DuncanTsmcginnis: The one time we don't want it, there's nothing in the pool anyway18:15
eharneyDuncanT: right18:15
eharneysmcginnis: it's mostly to clean up things that leaked in from a parent process/thread18:15
smcginniseharney: Ah, hadn't followed the bug yet. Looking now. Makes sense though.18:15
smcginniseharney: I see, makes sense.18:16
*** marcusvrn has joined #openstack-cinder18:16
eharneyi'm still not really sure how this is just showing up now18:17
eharneybut i think the secure NFS patch triggered it to be a lot more likely to occur18:17
eharneysince it actually started using the db before forking some of these children, when it wasn't before18:18
smcginniseharney: That seems to be the problem right? Only if the db is shared between multiple processes. Should that piece be fixed?18:18
*** marcusvrn has quit IRC18:19
DuncanTeharney: Yeah, if nothing touches the db before fork, the pool is empty so nothing gets leaked18:19
eharneysmcginnis: i think this is the quickest way to fix that (it effectively does).  The problem is the code that launches the services etc doesn't keep track of the db objects etc., so there's not really a good place to tie it in18:20
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Fix unit tests for multiattach patch
*** aix has quit IRC18:21
smcginniseharney: Makes sense. Thanks for talking it through.18:21
smcginnishemna: Can we get a tempest test added for multiattach too?18:21
smcginnishemna: That way if a backend supports it we'll get some real coverage?18:22
hemnasmcginnis, so jaypipes was going to do that for nova.18:23
hemnasmcginnis, so that's the plan for when the end to end pieces are all in place.18:23
smcginnishemna: OK, cool. Just thought of that over lunch and thought I should bring it up.18:24
hemnasmcginnis, buy yah, we need to gate on it eventually.18:24
jaypipeshemna: for tempest. TobiE was doing the nova pieces...18:24
hemnajaypipes, ok great.18:24
jaypipeshemna: next step is to get the cinderclient changes in and released I believe.18:24
hemnajaypipes, yah, there isn't much the nova guys can do w/o the cinderclient landing.18:24
hemnaand that isn't happening for K.18:24
hemnaso, I'll push to get the client in L as soon as it opens up.18:25
openstackgerritEric Harney proposed openstack/cinder: Fix sqlalchemy reuse in multi-backend children
eharneyDuncanT: smcginnis: heh, whoops, would be good if it actually passed pep8 ^18:25
DuncanTeharney: Meh, that's what we have jenkins for18:26
hemnaeharney, ew.  so all the c-vol services used the same objects?!18:26
hemnaDuncanT, +1 :P18:26
eharneyhemna: apparently18:26
eharneywell, copies of the same base object18:26
eharneywhich was "ok" until it actually had meaningful state in it18:26
DuncanTDo we ahve any similar issue with RPC connections or similar, I wonder?18:27
*** nellysmitt has joined #openstack-cinder18:27
smcginniseharney: I was actually wondering about an extra line before the constant. Guess I was looking in the wrong place. :)18:28
hemnaeharney, FYI "It is critical to note that dispose() does not guarantee that the application will release all open database connections - only those connections that are checked into the pool are closed. Connections which remain checked out or have been detached from the engine are not affected."18:28
eharneyhemna: well... this was what Mike specifically recommended we do18:30
hemnaok, I noted it in the review, fwiw.18:30
*** lpetrut has joined #openstack-cinder18:30
*** jaypipes is now known as jaypipes-afk18:34
*** marcusvrn has joined #openstack-cinder18:34
*** timcl has joined #openstack-cinder18:38
DuncanTThere's some discussion on the mailing list about trying to fix this in OSLO incubator18:38
kmartinAnyone know if the python-cinder client has the same Kilo deadlines as cinder?18:38
DuncanTkmartin: No, we can cut a client release at any time18:38
DuncanTkmartin: It is only as managed as the PTL makes it18:39
DuncanTkmartin: And you're way bigger than Mike. Put him in a headlock and nuggy him until he does a relase18:39
DuncanTkmartin: Actually, that won't work with his hair, will it?18:40
*** timcl has quit IRC18:40
kmartinDuncanT, lol thanks, so if a cinder feature is currently in gate that requires a client change to be useful then that should be fine to get in without a FFE18:41
DuncanTkmartin: It always has been before, and there's no official process required18:41
*** timcl has joined #openstack-cinder18:41
*** timcl has quit IRC18:43
*** jistr has quit IRC18:46
hemnajungleboyj, ping18:48
*** dflorea has quit IRC18:50
*** dflorea has joined #openstack-cinder18:51
*** ho has joined #openstack-cinder18:52
nikesh_vedamsnow after adding "filter = [ "a/.*/" ]" in /etc/cinder/lvm.conf , i am not getting volume-group not found error18:54
nikesh_vedamsbut still volume creation is failing18:54
DuncanTnikesh_vedams: What is the error?18:54
*** dflorea has quit IRC18:55
openstackgerritTom Barron proposed openstack/cinder: NFS backup driver
*** patrickeast has joined #openstack-cinder18:56
*** Administrator__ has joined #openstack-cinder18:56
*** dannywilson has joined #openstack-cinder18:56
*** mtanino has quit IRC18:57
*** mtanino has joined #openstack-cinder18:57
*** bswartz has quit IRC18:57
*** marcusvrn has quit IRC18:58
*** e0ne has joined #openstack-cinder18:58
*** nellysmitt has quit IRC18:59
*** Mandell has quit IRC18:59
*** nellysmitt has joined #openstack-cinder18:59
*** Administrator_ has quit IRC19:00
*** thingee has joined #openstack-cinder19:00
*** marcusvrn has joined #openstack-cinder19:01
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator
*** bswartz has joined #openstack-cinder19:03
nikesh_vedamsError is  No weighed hosts found for volume with properties: {u'name': u'lvmdriver-2', u'qos_specs_id': None, u'deleted': False, u'created_at': u'2015-03-11T17:26:11.000000', u'updated_at': None, u'extra_specs': {u'volume_backend_name': u'lvmdriver-2'}, u'is_public': True, u'deleted_at': None, u'id': u'b642c6de-845f-418e-af2f-ae192210ae94', u'description': None} 2015-03-12 00:24:02.459 10659 ERROR cinder.scheduler.fl19:05
jgriffithnikesh_vedams: what's the "name" in your spec there?19:05
jgriffithnikesh_vedams: that's not going to work19:06
jgriffithyou need an extra spec key: volume_backend_name=lvmdriver-219:06
e0nedoes anybody know why we have so many patches in check queue?19:06
jgriffithNOT name=lvmdriver-219:06
tbarronxyang, jungleboyj, e0ne, DuncanT:
*** jordanP has quit IRC19:08
*** ebalduf has quit IRC19:09
*** rushiagr is now known as rushiagr_away19:10
*** HoloIRCUser2 has quit IRC19:11
*** Mandell has joined #openstack-cinder19:11
*** annashen has quit IRC19:15
*** kallebe has left #openstack-cinder19:20
*** timcl has joined #openstack-cinder19:20
*** rushil has quit IRC19:20
DuncanTtbarron: Why is backup_compression_model being removed?19:20
marcusvrne0ne: hi! could you take a look on to check if the commit message is ok as you asked me?19:21
tbarronDuncanT: I just put a note in the review.  That option isn't being used.19:21
*** ebalduf has joined #openstack-cinder19:22
DuncanTtbarron: Cool, thanks19:22
tbarronIt's dead code left over from my first (unpushed) rebase attempt this weekend.19:22
marcusvrnxyang1: thangp: thanks for reviewing the patch19:22
thangpthangp: welcome19:22
tbarronOriginally I was going to use a streaming compression model for NFS and a chunked model for Swift.19:22
tbarronand hide the difference.19:22
thangpmarcusvrn: welcome19:22
DuncanTtbarron: Same with get_object_reader ?19:23
*** diegows has joined #openstack-cinder19:23
tbarronBut with xyang's changes for incremental backup I can no longer treat the NFS data store as a tape driver :-)19:23
tbarronDuncanT: yes.19:23
tbarronOriginally I wasn't doing the context handler with ... as object_reader.19:24
tbarronBut I implemented all the Swift and NFS object_reader as context handlers before I19:24
DuncanTtbarron: I'm wondering why backup_file_size and backup_sha_block_size_bytes aren't covered by the base chunked driver? Isn't backup_file_size just chunk size?19:24
tbarronpushed the last change set.19:24
*** timcl has quit IRC19:25
tbarronAnd I invoked them that way, but stupidly left the non-context handler invocation first.19:25
tbarronDidn't hurt Swift, but it was stupid.19:25
tbarronFound it when I did NFS.19:25
tbarronSwift unit tests still pass.19:25
DuncanTbackup_container is unused too19:25
tbarronDuncanT: I want to present config options to the end user with abstractions that make sense to the end user.19:26
tbarronEnd user doesn't have to know a backup file is a "chunk".19:26
nikesh_vedamsjgriffith: its set like that19:26
tbarron"chunk" makes more sense if you are thinking of the backup repo as an object store.19:26
tbarronDuncanT: there is an earlier discussion of this with patrickeast in the review IIRC.19:27
*** resker has quit IRC19:27
DuncanTtbarron: I'll go read.19:28
DuncanTtbarron: Is backup_container used anywhere?19:28
tbarronDunctanT: From a developer perspective they are of course the same thing.19:29
*** emagana has quit IRC19:29
*** marcusvrn has quit IRC19:29
DuncanTtbarron: I see now how the config options work, I hadn't realised how nfs, swift and chunked classes were structured19:30
*** annashen has joined #openstack-cinder19:30
tbarronDuncanT: for NFS the backup container is a posix path ending in xx/yy/backup_id19:31
tbarronwhere xx is the first two chars from the backup id and yy is the 3rd and fourth.19:31
DuncanTduncan@z400-duncan:~/src/cinder-upstream$ grep -R backup_container cinder/backup/19:31
DuncanTcinder/backup/drivers/    cfg.StrOpt('backup_container',19:31
DuncanTcinder/backup/drivers/        backup_default_container = CONF.backup_container19:31
tbarronThis keeps the number of entries in a direcotyr managable19:31
DuncanTNever mind, sorry, I'm an idiot19:32
e0netbarron: thanks for update. i'll look on it right now19:32
DuncanTtbarron: Ok, thanks, I think I've figuring this again19:32
tbarronDuncanT: not at all, you had the idea for doing it this way in the first place!19:32
tbarrone0ne: thanks much19:32
*** Mandell has quit IRC19:32
*** xyang1 has quit IRC19:32
DuncanTtbarron: I'm an idiot /tonight/.19:32
e0nemarcusvrn: i'm sorry. did i asked you in irc?19:33
tbarronDuncanT: well, I'm surprised you are around this late19:33
tbarronand that e0ne is still working too19:33
*** esker has joined #openstack-cinder19:33
erlone0ne: marcusvrn: it was in the review actually19:34
DuncanTtbarron: Failing to do electronincs mostly. Decided to stop after the second time I nearly plugged myself into the mains19:34
*** esker has quit IRC19:34
e0neerlon: i didn't find myself in that review19:34
tbarronDuncanT: that will wake you up, or put you to sleep for good.19:34
*** ronis has quit IRC19:35
DuncanTtbarron: The big sparks didn't wake me up. I saw taht as a danger sign19:35
*** esker has joined #openstack-cinder19:36
tbarronDuncanT: definitely19:36
openstackgerritMike Perez proposed openstack/cinder: Unset auth token before trying to login to Datera
erlone0ne: sorry Ivan, marcusvrn meat eharney19:36
*** kmartin has quit IRC19:37
DuncanTtbarron: +2.19:37
tbarronDuncanT: thanks!19:38
*** rushil has joined #openstack-cinder19:38
vilobhmmtbarron : can you plz send the link for your review req i will have a look19:39
nikesh_vedamsworking now,volume service was down19:39
tbarronDuncanT: vilobhmm: ty, I was away for a moment19:41
vilobhmmthanks DuncanT, tbarron : np19:41
DuncanTRight, I'm off19:43
*** takedakn has quit IRC19:45
thingeetbarron: I think jungleboyj was the other sponsor?19:45
*** nellysmitt has quit IRC19:46
thingeetbarron: yours is the last on the list :)
*** annegentle has quit IRC19:46
*** annegentle has joined #openstack-cinder19:47
tbarronthingee: ty, yes jungleboyj also volunteered :-)19:47
tbarronI owe jungleboyj and DuncanT and xyang and e0ne beers.19:47
tbarronor whatever19:47
tbarronyou too thingee :-)19:48
tbarronin-flight wireless?19:48
jungleboyjtbarron: Sounds good.  :-)19:48
*** Mandell has joined #openstack-cinder19:49
tbarronthingee: I'm taking a screen-shot of that last-on-the-list web page.19:49
e0netbarron: some comments inline, but i tried to explain them in the review19:49
nikesh_vedamsone more doubt,when i am doing ./,its creating a default volume group "stack-volumes-lvmdriver-1" is created which is lost if we reboot system19:50
* tbarron thinks it's not something to brag about, but to remember for sure19:50
tbarrone0ne: looking19:50
tbarrone0ne: I agree with the comments, but all but one of the areas you remarked on are in the original Swift code.19:52
*** Administrator__ has quit IRC19:52
e0netbarron: yes, that's why I put +219:52
tbarronthe one that's my responsibility is the \ continuation19:52
*** Administrator__ has joined #openstack-cinder19:52
*** annegentle has quit IRC19:52
tbarrone0ne: thanks, I'm happy to open a new review as a bug fix to put commas here we have '%' in log msgs, and replace \ continuations with parens continuations.19:53
nikesh_vedamswhy after reboot volume group stack-volumes-lvmdriver-1 is deleted?19:53
*** esker has quit IRC19:53
tbarronOr if you guys want to stay up longer, I can push a new patch now.19:53
thingeetbarron: yeah, I ended up just buying a monthly plan because of how often I was getting day passes on flight.19:54
*** Longgeek has joined #openstack-cinder19:54
thingeetbarron: I was doing just reviews through gertty, and that's nice, but I send so many emails, bug triage, etc that I need access still19:54
tbarrone0ne: I just know that it's late for both of you.19:54
jgriffithnikesh_vedams: because devstack uses loopback files19:54
jgriffithnikesh_vedams: which are not persistent19:54
jgriffithnikesh_vedams: you can modify code to make them persist if you want19:55
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator
jgriffithbut that's not what devstack was intended for19:55
jungleboyjtbarron: I got distracted.  Let me look.19:55
tbarronthingee: good to know we can reach you no matter what :-)  good for us at least.19:55
*** timcl has joined #openstack-cinder19:55
tbarronjungleboyj: np, got two +2s now ...19:56
thingeetbarron: yeah next thing I need to do is setup a bouncer again so I can still be pinged on irc19:56
e0netbarron: next level is to got +A ;)19:56
nikesh_vedamsjgriffith:so then i think we can get back these loopback files after reboot19:56
*** leeantho has quit IRC19:56
nikesh_vedamssure thanks19:56
*** leeantho has joined #openstack-cinder19:57
tbarrone0ne: jungleboyj: let me know if you want another changeset in this review with the changes e0ne proposed.19:57
jungleboyjtbarron: Will do.19:57
*** leeantho has quit IRC19:57
e0nejungleboyj, tbarron: i think my comments should be addressed in a separate patch19:57
tbarronjungleboyj: all except one issue, continuation by '\' rather than parens, are in the original Swift code.19:57
tbarrone0ne: and I'm happy to do that patch.19:58
tbarrone0ne: and besides being lazy atm, I actually believe that this refactor change set should avoid gratuitous changes to the original code.19:58
tbarronthat makes the fact that it's "just a refactor" clearer.19:59
*** timcl has quit IRC20:00
nikesh_vedamsjgriffith:yu said "but that's not what devstack was intended for" ,what that means20:00
*** Longgeek has quit IRC20:00
*** dflorea has joined #openstack-cinder20:01
jgriffithnikesh_vedams: that means, "it's not intended to be a persistent environment"20:02
*** dflorea has quit IRC20:02
nikesh_vedamsasking so that if i some one ask me then i can explain to them,20:02
jgriffithnikesh_vedams: FYI, the other trick is don't use loopbacks20:02
jgriffithnikesh_vedams: create an actual VG on physical disks and give it the cinder-volume name20:02
jgriffiththen devstack just leaves it alone20:02
jungleboyjFYI, +2 from me on the oslo.log patch.  Should get one other person to look at it though:
nikesh_vedamsyes that i know but i didnot know that loopback,thanks for clearance20:04
*** vilobhmm has quit IRC20:04
*** dflorea_ has joined #openstack-cinder20:05
*** vilobhmm1 has joined #openstack-cinder20:05
nikesh_vedamsjgriffith: thanks i might join "Canada Summit" :)20:06
jgriffithnikesh_vedams: awesome!20:06
*** diemt has quit IRC20:06
jgriffithnikesh_vedams: be good to meet you in person20:06
*** thangp has quit IRC20:08
*** annegentle has joined #openstack-cinder20:08
jungleboyjtbarron: You patch looks good to me.20:09
jungleboyje0ne: If you are ok having him address your additional comments separately I will +A it.20:09
tbarronjungleboyj: ty.  I appreciate all your help.20:09
*** esker has joined #openstack-cinder20:09
e0nejungleboyj: sure. i'm not +1/+2  if i don't want it in cinder:)20:10
jungleboyje0ne: Though that that may break the oslo.log patch.  Maybe it won't.20:11
jungleboyje0ne: That is ok though, you go to bed and I will deal with the oslo.log patch if anything goes wrong.20:11
e0nejungleboyj: thank you. i'll be available online about 2 hours more20:13
jungleboyjtbarron: Merry Christmas!20:13
*** fischerw has quit IRC20:13
jungleboyje0ne: You work too much.  ;-020:13
*** fischerw has joined #openstack-cinder20:14
tbarronjungleboyj: let's see what the bots think of it now.20:16
mtaninojungleboyj: Hi, Could you visit this patch again for stable/juno
mtaninoafter your current task will be finished :)20:18
anishsmcginnis: thanks for catching the error in my comment20:25
*** xyang1 has joined #openstack-cinder20:25
jgriffithe0ne: you still online?20:26
smcginnisanish: np - just luck. :)20:27
e0nejgriffith: yes20:27
jgriffithe0ne: I finally got around to setting up my fuel deployment again20:27
e0nejgriffith: :)20:28
jgriffithe0ne: do you by chance know default login/pass to the fuel nodes themeselves?20:28
e0neadmin/admin for web ui20:28
e0neroot/r00tme for ssh20:28
jgriffithe0ne: :)20:28
jgriffiththat's it... thanks!20:28
*** xyang1 has quit IRC20:29
*** kmartin has joined #openstack-cinder20:29
*** xyang has joined #openstack-cinder20:29
e0nejgriffith: you're welcome. we've got in the docs, so i'm comfortable to write it here:)20:30
jgriffithe0ne: hehe.. yeah, I just noticed it there ;)20:30
e0nejgriffith: ssh from master to slaves should work by key20:31
mtaninosmcginnis: anish :)20:31
*** delattec has quit IRC20:31
*** cdelatte has quit IRC20:31
anishmtanino: I'm glad your jenkins issues are finally over. that was amusing to watch20:32
mtaninoanish: oh, it was solved. nice20:33
jgriffithe0ne: well that was EZ :)20:33
jgriffithe0ne: up and running, configured my driver, running tests :)20:33
e0nejgriffith: ez==eazy?20:33
jgriffithe0ne: correct :)20:34
*** jistr has joined #openstack-cinder20:35
jgriffithanish: there ya go with the TM again :)20:36
jgriffithanish: show off!20:36
*** ndipanov has quit IRC20:36
anishjust a good keyboard layout :)20:37
*** scottda_ has joined #openstack-cinder20:39
*** jwang_ has quit IRC20:40
*** leeantho has joined #openstack-cinder20:40
*** scottda_ has quit IRC20:41
*** cdelatte has joined #openstack-cinder20:42
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder-specs: Volume multi attachments
*** takedakn has joined #openstack-cinder20:43
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix HNAS iSCSI driver error on LUN creation
*** akerr has quit IRC20:52
*** jwang_ has joined #openstack-cinder20:55
*** mtanino has quit IRC20:56
*** dustins has quit IRC20:57
*** esker has quit IRC21:02
*** bswartz has quit IRC21:03
*** Yogi1 has quit IRC21:04
*** r-daneel has quit IRC21:05
*** Yogi1 has joined #openstack-cinder21:05
*** tbarron has quit IRC21:05
*** jcru has quit IRC21:06
*** jcru has joined #openstack-cinder21:07
*** jcru has quit IRC21:08
*** tellesnobrega_ has joined #openstack-cinder21:09
*** eharney has quit IRC21:10
*** jistr has quit IRC21:10
*** thingee has quit IRC21:10
*** tellesnobrega_ has quit IRC21:11
*** vilobhmm2 has joined #openstack-cinder21:12
*** ozamiatin has quit IRC21:13
*** aarefiev has quit IRC21:13
*** ozamiatin has joined #openstack-cinder21:14
*** vilobhmm1 has quit IRC21:14
*** aarefiev has joined #openstack-cinder21:14
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator
jungleboyje0ne: Oh no!21:16
e0nejungleboyj: :(21:17
*** Yogi1 has quit IRC21:17
e0nejungleboyj: i forget to update requirements21:18
jungleboyje0ne: Go to bed.  I will keep an eye on it.21:19
e0nejungleboyj: that's i'm going to do right now21:20
jungleboyje0ne: Ok.21:20
jungleboyje0ne: Have a good night.21:20
e0nejungleboyj: thanks!21:21
*** esker has joined #openstack-cinder21:21
*** fischerw has quit IRC21:28
*** fischerw has joined #openstack-cinder21:28
jgriffithAnybody know if Eric noticed the test failures on that patch?21:29
*** annashen has quit IRC21:32
*** boris-42 has quit IRC21:32
smcginnisjgriffith: I hit recheck.21:33
jgriffithsmcginnis: well, it's a valid failure I think21:33
smcginnisAny idea why we're getting a lot of failures: Table 'encryption' specified column 'volume_type_id'...21:33
smcginnisjgriffith: This error keeps popping up, then going away on recheck.21:34
jgriffithsmcginnis: oh.. hmm21:34
smcginnisjgriffith: So I don't think it's valid, unless there actually is a real failure buried in that mess.21:34
jgriffithsmcginnis: I thought that some of those were in fact what this patch was supposed to fix21:34
smcginnisjgriffith: But I've seen it too often now to blame it on Eric. (yet)21:34
jgriffithsmcginnis: so you're seeing that unit test fail?21:35
smcginnisjgriffith: I think it failed one of my patches last night. Recheck passed.21:35
smcginnisjgriffith: Seen at least several others as well.21:35
jgriffithsmcginnis: that's very bad21:35
smcginnisjgriffith: Yeah, just started looking at the recheck page to see if there was anything.21:36
*** scottda_ has joined #openstack-cinder21:36
smcginnisjgriffith: Doesn't look like it's been identified though, otherwise elastic recheck would comment, right?21:36
jgriffithsmcginnis: should yeah21:37
jgriffithsmcginnis: also the fact that it's in unit tests is distrubing to say the least21:37
smcginnisThat's what I've seen a bunch of.21:37
smcginnisjgriffith: It's like the database is getting messed with occasionally.21:37
jgriffithYeah, same thing... very bad21:37
*** rushil_ has joined #openstack-cinder21:38
smcginnisHmmm...  033_add_encryption_unique_key.py21:39
*** annashen has joined #openstack-cinder21:39
*** movielovers has joined #openstack-cinder21:39
*** rushil has quit IRC21:39
*** dflorea_ has quit IRC21:40
openstackLaunchpad bug 1417018 in Cinder "Cinder encounters dbapi error: NoSuchColumnError: "Could not locate column in row for column '%(140070887032400 anon)s.volumes_id'"" [High,In progress] - Assigned to Eric Harney (eharney)21:40
openstackLaunchpad bug 1417018 in Cinder "duplicate for #1430859 Cinder encounters dbapi error: NoSuchColumnError: "Could not locate column in row for column '%(140070887032400 anon)s.volumes_id'"" [High,In progress] - Assigned to Eric Harney (eharney)21:40
*** dflorea has joined #openstack-cinder21:40
smcginnisjgriffith: Those look similar, but a little different. This one looks like conflicting primary key definitions.21:40
jgriffithGiven that's what Eric's patch was referenced for on the ML looks like it doesn't work21:41
jgriffithsmcginnis: I'm betting same root cause here21:41
*** Mandell has quit IRC21:41
jgriffithsmcginnis: the oslo.db fix is the key there21:41
smcginnisjgriffith: So we need to wait for oslo?21:41
dhellmannjgriffith: is zzzeek's patch not working?21:42
jgriffithsmcginnis: not necessarily... check comment 3 and 4 in bug 141701821:42
openstackbug 1417018 in Cinder "Cinder encounters dbapi error: NoSuchColumnError: "Could not locate column in row for column '%(140070887032400 anon)s.volumes_id'"" [High,In progress] - Assigned to Eric Harney (eharney)21:42
*** scottda_ has quit IRC21:42
jgriffithdhellmann: not sure... which that one is21:42
jgriffithdhellmann: I don't think we've pulled that in yet21:42
*** zzzeek has joined #openstack-cinder21:43
jgriffithdhellmann: OH... Mike :)21:43
dhellmannjgriffith: yeah, sorry, I thought he was in the room21:43
jgriffithdhellmann: so no, the fix Eric submitted just failed for the same error21:43
*** jecarey has joined #openstack-cinder21:43
smcginnisI'm still not convinced. It looks a little different to me.21:44
*** xyang has quit IRC21:44
dhellmannjgriffith: oh, sorry, zzzeek gave me that link a while back and I thought it was his patch, so that's the one I meant21:44
*** dflorea has quit IRC21:45
dhellmannjgriffith: we're going to move ahead with in oslo.db, too21:45
zzzeekOK in the console of this failed build I see some table not existing issues, so let me ask, is this SQLite “memory” DB and that constructor is being called in-process perhaps?21:45
dhellmannwe probably want some form of both, though - belt and braces21:46
zzzeekbecause that dispose() will blow away the “memory” database21:46
jgriffithzzzeek: yes21:46
zzzeekjgriffith: OK so that’s the issue here21:46
jgriffithzzzeek: unit tests all run sqlite21:46
jgriffithzzzeek: interesting21:46
jgriffithzzzeek: that complicates things slightly21:46
zzzeekjgriffith: right, so, sqlite memory DB has no on-disk state, you call engine.dispose() and its gone21:46
zzzeekjgriffith: yes :)21:46
jgriffithzzzeek: makes sense21:46
zzzeekjgriffith: which is why, calling dispose() *only* in a forked child, would be best21:46
jgriffithzzzeek: yeah, was just wondering about moving that down in to the actual service21:47
jgriffithzzzeek: not that I'm clear on exactly how that would look21:47
zzzeekjgriffith: unless these tests are actually doing that, which i woud think a sqlite :memory: wouldnt travel there very well21:47
zzzeekjgriffith: there’s no forking in these tests, right?21:47
jgriffithzzzeek: in the unit tests no, there should not be21:48
zzzeekjgriffith: guaranteed way to see if you just started a child is to memoize and os.ppid() and compare21:48
smcginnisThis one actually does seem to be related to sqllite:
zzzeekjgriffith: basically something somewhre needs a “we are starting a child” routine.     id think the best place woudl be in the service library thats doing the fork21:49
smcginnisIN 033_add_encryption_unique_key it tries to drop a primary key of volume_type_id21:49
smcginnisThen add a new primary key constraint to make encryption_id the PK.21:50
* zzzeek thinks openstack’s whole “run the tests against sqlite + memory” is insane :)21:50
jgriffithzzzeek: yeah, that whole thing has been debated in the past21:50
zzzeekimagine how many problems we woudlnt have if we just made people run a fricking mysql db21:50
smcginnisBut then in the exception handling of dropping the previous PK is this:21:50
smcginnis        # NOTE (e0ne): SQLite doesn't support 'drop constraint' statament21:50
smcginnis        if migrate_engine.url.get_dialect().name.startswith('sqlite'):21:50
smcginnis            pass21:50
jgriffithsmcginnis: indeed21:50
smcginnisSo on upgrade if using sqllite you're basically screwed it appears.21:51
*** Mandell has joined #openstack-cinder21:51
zzzeekjgriffith: here’s an easier, more hacky appraoch:  “if engine.driver != ‘sqlite’: engine.dispose()”21:51
smcginnisYou can't get rid of the old PK to add the new PK, causing the error in Eric's patch test run.21:51
zzzeekerr e.name21:52
smcginnisAt least that one instance that's I've seen elsewhere.21:52
smcginniszzzeek: +1 seems simple enough for now at least.21:52
zzzeeksmcginnis: its a nasty hack21:52
smcginniszzzeek: Agreed21:53
zzzeeksmcginnis: its just to get you through the day and see if htis works otherwise21:53
smcginniszzzeek: But gets around it until a better solution is found...21:53
zzzeeksmcginnis: make sure its noted, “remove this when XYZ is fixed”21:53
e0neis something wrong with migration #33>21:53
smcginnise0ne: Oh, just saw a bunch of failures recently from Jenkins.21:54
smcginnisRelated to that one, but usually a recheck passes.21:54
smcginnisIt was pointed out gate uses sqlite. Saw your note that primary key constraint can't be dropped.21:54
e0neis was made few month ago21:54
smcginnisThat appears to be causing the issue, though not sure why it's intermittent then.21:54
smcginnise0ne: Seems fine for the most part.21:55
*** mriedem is now known as mriedem_away21:55
*** esker has quit IRC21:55
smcginnise0ne: Looks like the only way around it with sqlite is to create a new table, migrate data, drop the original, then rename the new one.21:55
e0neupgrade or downgrade?21:56
smcginnisGate uses sqlite. Sqlite doesn't allow primary key to change. :[21:56
smcginnisI rechecked Eric's patch because this was the first thing I saw, but looking further it does look like there are problems with this fix.21:57
e0newhy did it work before?21:57
smcginnisI think the work around zzzeek proposed will probably have to do.21:57
smcginnise0ne: And it doesn't always cause problems, so that's what I'm scratching my head on.21:58
*** arif-ali has quit IRC21:58
smcginnise0ne: Not even sure how you could do the whole migration to a new table and rename with sqlalchemy.22:00
hemnaoops what happened?22:00
e0nesmcginnis: if it works sometimes on gates - it means that different envs on infra22:00
smcginnishemna: You broke it.22:00
* hemna runs home22:00
smcginnishemna: JK :)22:00
hemnaso that looks like it's not a migration failure22:00
e0nesmcginnis: it it reproducible on Eric's patch?22:00
smcginnisAnyone know if they use a mix of backends? That could explain why it works sometimes and not others.22:00
hemnano such table ?22:01
smcginnisSometimes gets mysql, sometimes gets sqlite.22:01
e0nesmcginnis: it's crazy22:01
smcginnise0ne: It was hit once with Eric's patch, but it does look like there are other bigger issues.22:01
zzzeeksmcginnis: the table rename thing is in alembic with the “batch” migration mode, and sqlalchemy-migrate has always had the sqlite rename and drop the table thing22:01
e0nesmcginnis: what is the "other bigger issue"?22:02
smcginniszzzeek: Oh. So 033 could be updated - if it's sqlite do a migrate, otherwise do a drop constraint?22:02
*** lpetrut has quit IRC22:02
smcginnise0ne: If I followed right, engine_dispose() with sqlite in memory DB will actually blow away the DB.22:02
zzzeeksmcginnis: not sure, not looking :)22:02
smcginnise0ne: So Eric's approach loses the whole DB in that case.22:03
smcginniszzzeek: ;)22:03
*** _cjones_ has quit IRC22:03
hemnagoosh, yah it looks like every table is gone22:03
e0nesmcginnis: ok.. why are you talking about migration error?22:03
hemnaship it!22:03
*** akerr has joined #openstack-cinder22:03
*** _cjones_ has joined #openstack-cinder22:03
e0nei can't believe that code works well for a 3 month and it is broken now22:04
hemnaso, maybe it's not able to reconnect to the db instance ?22:04
smcginnise0ne: One out of the many errors was this one:
hemnaare other patches failing in the same way ?22:04
*** akerr_ has joined #openstack-cinder22:04
smcginnise0ne: I've seen that in a few patches (including my own) where a recheck passes.22:04
Swansongate-cinder-python27 is running on that patch right now.  So be interesting to see if it is consistent fail or not.22:04
smcginnisSwanson: +122:05
smcginnise0ne: So sometimes 033 works fine. Other times it fails.22:05
smcginnise0ne: Related to Eric's patch, it was pointed out gate uses sqlite.22:05
e0nesmcginnis: it's a warning22:05
smcginnise0ne: Then I saw your note.22:05
smcginnise0ne: Apparently it treats that warning as a failure.22:05
*** annegentle has quit IRC22:05
hemnawhere are the migration logs associated with this ?22:06
smcginnise0ne: End the attempt to set encryption_id as the primary key fails because volume_type_id is still the primary key.22:06
smcginnise0ne: Only for sqlite cases.22:06
e0neif tests works time to time it means we've got different version of software on different Infra's hosts22:06
smcginnise0ne: So wondering if instead of passing on that exception we need to do the data migration to a new table.22:07
smcginnise0ne: That's what I was wondering.22:07
jgriffithzzzeek: yeah, that's what I was wondering if we could/shoudl do :)22:07
smcginnise0ne: Maybe infra uses a mix of DBs? Only thing I can think of that would explain it.22:07
jgriffithzzzeek: even if just as a temporary hack22:07
e0nei can't believe in it22:07
jgriffithzzzeek: think I may mod eharney's patch and see what folks think22:08
jgriffithalthough I'd rather wait and chat with him22:08
*** akerr has quit IRC22:08
jgriffithzzzeek: actually... since you mentioned it first, do you want to suggest it in the comments of his current review set?22:08
smcginnisjgriffith: Since he's not around, might be a good test to mod it. He can always back it out, right?22:08
jgriffithsmcginnis: correct22:09
*** jungleboyj has quit IRC22:09
smcginnisjgriffith: Or that...22:09
*** arif-ali has joined #openstack-cinder22:10
*** ho__ has joined #openstack-cinder22:12
SwansonIt choked again.22:12
jgriffithhehe... pysqlite22:13
jgriffithok, I'll just mod the test class and submit it22:13
jgriffithpet peave alert: No handlers could be found for logger "cinder.volume.drivers.cloudbyte.cloudbyte"22:14
*** arif-ali has quit IRC22:15
*** arif-ali has joined #openstack-cinder22:18
jgriffithzzzeek: hmm.. that doesn't work out well22:19
jgriffithsmcginnis: so the *good* thing is those errors are 100% reproducible22:22
e0nesmcginnis, jgriffith: i've run test_migrations tests for sqlite on Eric's patch22:23
e0neits' workin on my en22:23
e0ne10 times from 1022:23
jgriffithe0ne: try runnign the fulll test suite22:24
e0neand cinder.tests.test_volume_transfer.VolumeTransferTestCase.test_transfer_volume_create_delete fails everytime22:24
jgriffithe0ne: not just migrations22:24
e0neand yes, i've got sqlalchemy warning on both tests22:24
*** annegentle has joined #openstack-cinder22:25
jgriffithe0ne: smcginnis trying to figure out the relation in test.py22:26
jgriffiththe Database fixture that is22:26
e0neit invokes dispose several times:(22:26
jgriffithe0ne: right... every time22:26
e0nesmcginnis, jgriffith: i can't understand why are you talking that migration fails22:26
jgriffithe0ne: don't think we were22:27
e0nesmcginnis, jgriffith: i can't work or not work for different test22:27
jgriffithe0ne: we were talking about:
e0nejgriffith: ok, so we are on the same page22:27
jgriffithe0ne: sorry, I'm not following?22:28
e0nejgriffith: i mean that you agree that migration works22:28
e0nedo you?22:28
jgriffithe0ne: I dunno... maybe :)22:29
*** annashen has quit IRC22:30
jgriffithe0ne: there are a few hundred test failures, didn't loook at exactly what passes :)22:30
jgriffithwould image migrations are "ok"22:30
jgriffithbut I dunno22:30
*** annashen has joined #openstack-cinder22:30
*** jaypipes-afk is now known as jaypipes22:31
*** annegentle has quit IRC22:34
jgriffithe0ne: so I can fix it by hacking the sqlite check in the db/ layer22:39
jgriffithe0ne: but I'm not sure that's such a "good" thing or not22:39
*** tellesnobrega_ has joined #openstack-cinder22:40
*** tellesnobrega_ has quit IRC22:41
e0newhy when we call engine.dispose() and its gone for in memory sqlite db?22:41
jgriffithe0ne: if I follow correctly because you basicly just deleted the db22:41
*** lcurtis has quit IRC22:42
jgriffithe0ne: smcginnis here's what I did:
jgriffithOhh... hmm22:44
e0nejgriffith: zzzeek said so. but why? i didn't find anything reladed in sqlalchemy docd22:44
*** Longgeek has joined #openstack-cinder22:45
jgriffithe0ne: not sure, but the patch change I put the gist up for certainly fixes it22:47
*** jecarey has quit IRC22:51
openstackgerritJohn Griffith proposed openstack/cinder: Fix sqlalchemy reuse in multi-backend children
*** annashen has quit IRC22:52
*** Adriano_ has quit IRC22:52
e0nejgriffith: i agree that yours patch with small changes would work and looks acceptable22:54
e0nejgriffith: i just want to understand root cause of the errors22:54
jgriffithe0ne: yeah, I'm kinda nervous about how hacky that is22:54
jgriffithe0ne: and yes, I agree with you completely22:54
jgriffithe0ne: as a result i'm certainly ok if that doesn't merge22:54
jgriffithe0ne: no problem waiting22:54
*** Longgeek has quit IRC22:55
jgriffithe0ne: maybe see what eharney thinks when he gets back online22:55
e0nejgriffith: good idea22:55
jgriffithe0ne: also I think there's something different we should be doing22:55
jgriffithe0ne: ie in the way we launch multi-backend22:55
e0neTBH, i don't like out db/ code22:56
*** movielovers has left #openstack-cinder22:57
*** angela-s has quit IRC22:58
jgriffithe0ne: I was just noticing that as well22:58
jgriffithe0ne: also take a look at the stuff we're doing in our test fixture: test.py22:58
jgriffithe0ne: we probably need to get some love on those things22:58
e0nejgriffith: +2 from me. i'm agree about tests refactoring22:59
e0nenot only db-related tests22:59
jgriffithe0ne: yea22:59
e0nejgriffith:  - i trief to refactor it to fix concurrency. but it must be re-writed almost all:(23:00
e0nei'm sorry. i need to go sleep23:02
e0nebe back in ~10 hours23:02
jgriffithe0ne: hve you noticed all of these too:
jgriffithe0ne: oh... ok... go sleep!!!  I'll see you tomorrow23:03
*** dannywil_ has joined #openstack-cinder23:03
*** dannywil_ has quit IRC23:03
e0nejgriffith: yes, but it's an warning in my sqlalchemy.migrate version23:04
* e0ne away23:04
*** e0ne is now known as e0ne_23:04
*** patrickeast_ has joined #openstack-cinder23:05
*** dannywilson has quit IRC23:06
*** patrickeast has quit IRC23:06
*** patrickeast_ is now known as patrickeast23:06
*** ganso_ has quit IRC23:07
openstackgerritVilobh Meshram proposed openstack/cinder: WIP - ConsistencyGroup Object
*** thingee has joined #openstack-cinder23:10
openstackgerritVilobh Meshram proposed openstack/cinder: WIP - ConsistencyGroup Object
*** annashen has joined #openstack-cinder23:12
*** scottda_ has joined #openstack-cinder23:15
*** scottda_ has quit IRC23:16
*** asselin_ has joined #openstack-cinder23:18
*** asselin_ has quit IRC23:23
*** scottda_ has joined #openstack-cinder23:24
*** annashen has quit IRC23:24
*** akerr_ has quit IRC23:27
*** Apoorva has quit IRC23:28
*** jecarey has joined #openstack-cinder23:28
*** Mandell has quit IRC23:29
*** scottda_ has quit IRC23:29
*** annegentle has joined #openstack-cinder23:32
*** bswartz has joined #openstack-cinder23:37
*** scottda_ has joined #openstack-cinder23:38
*** HoloIRCUser1 has joined #openstack-cinder23:39
*** IanGovett has quit IRC23:39
*** chlong has joined #openstack-cinder23:42
*** annegentle has quit IRC23:43
*** sparr has quit IRC23:45
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator
*** sparr has joined #openstack-cinder23:48
*** e0ne_ has quit IRC23:50
*** Apoorva has joined #openstack-cinder23:51
*** dflorea has joined #openstack-cinder23:51
*** mtanino has joined #openstack-cinder23:52
*** dflorea has quit IRC23:54
*** dflorea has joined #openstack-cinder23:54
*** scottda_ has quit IRC23:58
*** patrickeast has quit IRC23:58
*** scottda_ has joined #openstack-cinder23:59
*** dflorea has quit IRC23:59

Generated by 2.14.0 by Marius Gedminas - find it at!