| *** r-daneel has quit IRC | 00:01 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: WIP - Quota Objects https://review.openstack.org/163232 | 00:04 |
|---|---|---|
| *** e0ne has quit IRC | 00:06 | |
| *** Apoorva has joined #openstack-cinder | 00:06 | |
| *** jogo has left #openstack-cinder | 00:07 | |
| *** ho has joined #openstack-cinder | 00:09 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Raise exception for invalid mock assert calls https://review.openstack.org/160208 | 00:14 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Switch to oslo_versionedobjects https://review.openstack.org/160209 | 00:14 |
| *** tsekiyama has quit IRC | 00:18 | |
| *** mtanino has quit IRC | 00:21 | |
| *** vilobhmm1 has quit IRC | 00:25 | |
| *** annashen has quit IRC | 00: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 volume | 00:33 |
| *** david-lyle_afk has quit IRC | 00:34 | |
| *** IanGovett has quit IRC | 00:34 | |
| *** yamada-h has joined #openstack-cinder | 00:41 | |
| *** leeantho has quit IRC | 00:41 | |
| *** yamada-h has quit IRC | 00:45 | |
| *** topshare has joined #openstack-cinder | 00:51 | |
| jungleboyj | jgriffith: http://logs.openstack.org/56/149256/16/check/check-grenade-dsvm/c2cbbfd/ | 01:10 |
| jungleboyj | Saw that on a couple of patches. | 01:11 |
| *** _cjones_ has quit IRC | 01:11 | |
| *** liusheng has quit IRC | 01:12 | |
| *** liusheng has joined #openstack-cinder | 01:12 | |
| *** topshare has quit IRC | 01:15 | |
| jgriffith | jungleboyj: looking | 01:17 |
| jgriffith | jungleboyj: http://goo.gl/tlS5V7 | 01:19 |
| *** chenleji has joined #openstack-cinder | 01:20 | |
| jungleboyj | jgriffith: Yeah, that is what I have seen. | 01:20 |
| *** david-lyle_afk has joined #openstack-cinder | 01:21 | |
| jgriffith | jungleboyj: I can try and help track it down tomorrow, if it hasn't already been reported or even fixed | 01:21 |
| jgriffith | jungleboyj: would be curious if it's already known | 01:21 |
| jungleboyj | jgriffith: Ok. I will search google again in a little bit and open a bug if it hasn't. | 01:22 |
| jgriffith | jungleboyj: might be worth a rebase just for grins | 01:22 |
| jgriffith | jungleboyj: the initial patch set is pretty old | 01:22 |
| jgriffith | jungleboyj: unless there's one in there and I just didn't see it | 01:22 |
| jungleboyj | Ok. | 01:22 |
| jgriffith | although... this is submitted against master | 01:22 |
| jgriffith | hmmm | 01:22 |
| jgriffith | hemnafk: jungleboyj DuncanT smcginnis patrickeast https://review.openstack.org/#/c/85847/ | 01:24 |
| jgriffith | I'm good with merging that noting a couple things in my comment | 01:24 |
| jgriffith | I'll leave it to another core for the +A | 01:24 |
| jgriffith | and on that note, gotta get out for tonight | 01:24 |
| jgriffith | cya | 01:24 |
| jungleboyj | jgriffith: Thanks for the input there! | 01:24 |
| jungleboyj | Have a good night. | 01:25 |
| *** lcurtis has joined #openstack-cinder | 01:26 | |
| *** annashen has joined #openstack-cinder | 01:26 | |
| *** topshare has joined #openstack-cinder | 01:27 | |
| *** topshare has quit IRC | 01:28 | |
| *** Lee1092 has joined #openstack-cinder | 01:28 | |
| *** devlaps has joined #openstack-cinder | 01:31 | |
| *** annashen has quit IRC | 01:31 | |
| *** mwichmann has quit IRC | 01:32 | |
| *** topshare has joined #openstack-cinder | 01:33 | |
| openstackgerrit | harsh mishra proposed openstack/cinder: Fix for inconsistent cinder-services state change https://review.openstack.org/160104 | 01:36 |
| jungleboyj | jgriffith: Looks like a bug is open for it now: https://bugs.launchpad.net/openstack-gate/+bug/1430592 | 01:42 |
| openstack | Launchpad 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 |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Move to the oslo.middleware library https://review.openstack.org/163257 | 01:46 |
| *** cdelatte has quit IRC | 01:49 | |
| *** devlaps has quit IRC | 01:51 | |
| *** patrickeast has quit IRC | 01:51 | |
| *** rwsu is now known as rwsu-afk | 01:52 | |
| *** diegows has quit IRC | 01:55 | |
| *** changbl has joined #openstack-cinder | 01:56 | |
| *** vilobhmm has joined #openstack-cinder | 01:56 | |
| *** takedakn has joined #openstack-cinder | 02:04 | |
| *** harlowja_ is now known as harlowja_away | 02:09 | |
| *** Apoorva has quit IRC | 02:14 | |
| *** Apoorva has joined #openstack-cinder | 02:18 | |
| *** jcru has joined #openstack-cinder | 02:24 | |
| *** Apoorva has quit IRC | 02:26 | |
| *** haomaiwang has joined #openstack-cinder | 02:28 | |
| *** haomaiwang has quit IRC | 02:29 | |
| *** coolsvap has joined #openstack-cinder | 02:29 | |
| *** haomaiwang has joined #openstack-cinder | 02:29 | |
| *** haomaiwang has quit IRC | 02:29 | |
| *** haomaiwa_ has joined #openstack-cinder | 02:30 | |
| *** haomaiwa_ has quit IRC | 02:30 | |
| *** haomaiwang has joined #openstack-cinder | 02:31 | |
| *** Mandell has quit IRC | 02:33 | |
| *** jcru has quit IRC | 02:35 | |
| *** david-lyle_afk has quit IRC | 02:36 | |
| *** takedakn1 has joined #openstack-cinder | 02:40 | |
| *** takedakn has quit IRC | 02:40 | |
| *** kaisers2 has joined #openstack-cinder | 02:41 | |
| *** kaisers3 has joined #openstack-cinder | 02:42 | |
| *** kaisers1 has quit IRC | 02:42 | |
| *** yamada-h has joined #openstack-cinder | 02:42 | |
| *** kaisers has quit IRC | 02:43 | |
| *** I has joined #openstack-cinder | 02:44 | |
| *** I is now known as Guest51723 | 02:45 | |
| *** Guest51723 is now known as centcom | 02:45 | |
| *** takedakn1 has quit IRC | 02:46 | |
| *** yamada-h has quit IRC | 02:47 | |
| *** jcru has joined #openstack-cinder | 02:48 | |
| *** centcom has left #openstack-cinder | 02:50 | |
| *** hemna has joined #openstack-cinder | 02:51 | |
| hemna | yough | 02:53 |
| hemna | jungleboyj, do we want the oslo_middleware patch in kilo? | 02:58 |
| hemna | the one you submitted today? | 02:58 |
| *** coolsvap is now known as coolsvap|afk | 03:15 | |
| *** jcru has quit IRC | 03:17 | |
| *** david-lyle_afk has joined #openstack-cinder | 03:17 | |
| *** resker has joined #openstack-cinder | 03:18 | |
| *** esker has quit IRC | 03:21 | |
| *** coolsvap|afk is now known as coolsvap | 03:22 | |
| *** jcru has joined #openstack-cinder | 03:29 | |
| *** jcru has quit IRC | 03:29 | |
| *** shyama has joined #openstack-cinder | 03:37 | |
| *** mwichmann has joined #openstack-cinder | 03:43 | |
| *** vilobhmm has quit IRC | 03:50 | |
| *** chenleji has quit IRC | 03:53 | |
| *** patrickeast has joined #openstack-cinder | 03:53 | |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/cinder: Fix typos in LVMVolumeDriver https://review.openstack.org/162581 | 03:56 |
| *** lcurtis has quit IRC | 04:01 | |
| *** asselin_ has joined #openstack-cinder | 04:01 | |
| *** tries has quit IRC | 04:12 | |
| *** BharatK has joined #openstack-cinder | 04:14 | |
| *** tries has joined #openstack-cinder | 04:23 | |
| *** tries has joined #openstack-cinder | 04:23 | |
| *** [1]Thelo has joined #openstack-cinder | 04:27 | |
| *** Thelo has quit IRC | 04:30 | |
| *** [1]Thelo is now known as Thelo | 04:30 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:35 | |
| *** dflorea has joined #openstack-cinder | 04:44 | |
| *** patrickeast has quit IRC | 04:48 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Sort snapshots in create CG from CG snapshot https://review.openstack.org/163288 | 04:56 |
| *** rushiagr_away is now known as rushiagr | 04:58 | |
| *** anuragpalsule has joined #openstack-cinder | 05:00 | |
| *** xyang1 has quit IRC | 05:02 | |
| *** mwichmann has quit IRC | 05:15 | |
| *** deepakcs has joined #openstack-cinder | 05:18 | |
| *** krishna_vedams has quit IRC | 05:21 | |
| *** BharatK has left #openstack-cinder | 05:25 | |
| *** BharatK has quit IRC | 05:25 | |
| *** nikesh_vedams has joined #openstack-cinder | 05:28 | |
| *** Mandell has joined #openstack-cinder | 05:46 | |
| *** netapp has quit IRC | 05:49 | |
| *** bnemec has quit IRC | 05:49 | |
| *** netapp has joined #openstack-cinder | 05:49 | |
| *** bnemec has joined #openstack-cinder | 05:52 | |
| *** yamada-h has joined #openstack-cinder | 05:55 | |
| *** yamada-h has quit IRC | 05:56 | |
| openstackgerrit | Richard Hedlind proposed openstack/cinder: Properly set number of remaining retries after failed REST call. Also, update unit test to properly test the retry path. https://review.openstack.org/163296 | 05:57 |
| *** netapp has quit IRC | 06:02 | |
| *** BharatK has joined #openstack-cinder | 06:06 | |
| *** bkopilov has quit IRC | 06:08 | |
| *** netapp has joined #openstack-cinder | 06:08 | |
| *** nlevinki has joined #openstack-cinder | 06:08 | |
| *** pradipta has joined #openstack-cinder | 06:09 | |
| *** nlevinki has quit IRC | 06:09 | |
| *** nlevinki has joined #openstack-cinder | 06:09 | |
| *** bkopilov has joined #openstack-cinder | 06:09 | |
| *** harlowja_at_home has quit IRC | 06:11 | |
| *** hemna has quit IRC | 06:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/152401 | 06:15 |
| *** stefan_amann has joined #openstack-cinder | 06:17 | |
| *** pradipm has joined #openstack-cinder | 06:20 | |
| *** pradipm has quit IRC | 06:21 | |
| *** pradipm has joined #openstack-cinder | 06:21 | |
| *** pradipm has quit IRC | 06:22 | |
| *** lpetrut has joined #openstack-cinder | 06:22 | |
| *** dflorea has quit IRC | 06:23 | |
| *** annashen has joined #openstack-cinder | 06:31 | |
| *** stefan_amann has quit IRC | 06:33 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails https://review.openstack.org/160682 | 06:33 |
| *** annashen has quit IRC | 06:36 | |
| *** anshul has joined #openstack-cinder | 06:37 | |
| *** Maike has joined #openstack-cinder | 06:41 | |
| *** resker has quit IRC | 06:49 | |
| *** esker has joined #openstack-cinder | 06:50 | |
| *** takedakn has joined #openstack-cinder | 07:00 | |
| *** anand1 has joined #openstack-cinder | 07:08 | |
| *** vnagpal has joined #openstack-cinder | 07:09 | |
| openstackgerrit | YAMADA Hideki proposed openstack/cinder: Add missing backup/restore methods to Sheepdog driver https://review.openstack.org/162927 | 07:14 |
| *** kaisers3 has quit IRC | 07:15 | |
| *** dulek has joined #openstack-cinder | 07:30 | |
| *** Tross1 has quit IRC | 07:37 | |
| *** bkopilov has quit IRC | 07:38 | |
| *** Tross has joined #openstack-cinder | 07:38 | |
| *** bkopilov has joined #openstack-cinder | 07:45 | |
| *** bkopilov has quit IRC | 07:50 | |
| *** takedakn has quit IRC | 07:50 | |
| *** alecv has joined #openstack-cinder | 07:53 | |
| *** etta_ has quit IRC | 07:54 | |
| *** coolsvap is now known as coolsvap|brb | 07:54 | |
| *** takedakn has joined #openstack-cinder | 07:57 | |
| *** chlong has quit IRC | 08:01 | |
| *** bkopilov has joined #openstack-cinder | 08:02 | |
| *** markus_z has joined #openstack-cinder | 08:04 | |
| *** e0ne has joined #openstack-cinder | 08:11 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails https://review.openstack.org/160682 | 08:13 |
| *** topshare has quit IRC | 08:15 | |
| *** pradipta has quit IRC | 08:16 | |
| *** topshare has joined #openstack-cinder | 08:18 | |
| *** topshare has quit IRC | 08:21 | |
| *** stefan_amann has joined #openstack-cinder | 08:23 | |
| *** topshare has joined #openstack-cinder | 08:23 | |
| *** Mandell has quit IRC | 08:23 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Backup object https://review.openstack.org/157085 | 08:24 |
| *** coolsvap|brb is now known as coolsvap | 08:26 | |
| *** jungleboyj has quit IRC | 08:28 | |
| *** TobiasE has joined #openstack-cinder | 08:29 | |
| *** TobiasE has quit IRC | 08:31 | |
| *** jungleboyj has joined #openstack-cinder | 08:32 | |
| *** HoloIRCUser2 has joined #openstack-cinder | 08:34 | |
| *** lpetrut has quit IRC | 08:39 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails https://review.openstack.org/160682 | 08:41 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails https://review.openstack.org/160682 | 08:42 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails https://review.openstack.org/160682 | 08:44 |
| openstackgerrit | Lena Novokshonova proposed openstack/cinder: Add notifications about snapshot.update.* https://review.openstack.org/133041 | 08:44 |
| *** ndipanov has joined #openstack-cinder | 08:47 | |
| *** bkopilov has quit IRC | 08:49 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add is_ready method to scheduler driver https://review.openstack.org/163098 | 08:50 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add waiting for the driver to SchedulerManager https://review.openstack.org/163099 | 08:50 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Allow scheduler to receive volume stats when starting service https://review.openstack.org/158623 | 08:50 |
| *** bkopilov has joined #openstack-cinder | 08:51 | |
| *** topshare has quit IRC | 08:53 | |
| *** topshare has joined #openstack-cinder | 08:55 | |
| *** topshare has quit IRC | 08:59 | |
| *** stefan_amann has quit IRC | 08:59 | |
| openstackgerrit | YAMADA Hideki proposed openstack/cinder: Add missing backup/restore methods to Sheepdog driver https://review.openstack.org/162927 | 08:59 |
| *** topshare has joined #openstack-cinder | 09:01 | |
| *** topshare has quit IRC | 09:03 | |
| *** topshare has joined #openstack-cinder | 09:05 | |
| *** lpetrut has joined #openstack-cinder | 09:07 | |
| *** ronis has joined #openstack-cinder | 09:10 | |
| *** ho has quit IRC | 09:15 | |
| *** afazekas_ has joined #openstack-cinder | 09:16 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Attach/detach batch processing in VNX driver https://review.openstack.org/154350 | 09:19 |
| *** topshare has quit IRC | 09:20 | |
| *** pradipta has joined #openstack-cinder | 09:24 | |
| *** jistr has joined #openstack-cinder | 09:25 | |
| *** e0ne has quit IRC | 09:27 | |
| *** annashen has joined #openstack-cinder | 09:34 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Remove global mocking from test_pure.py https://review.openstack.org/163336 | 09:37 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Retype in driver should have the correct parameters https://review.openstack.org/163338 | 09:39 |
| *** annashen has quit IRC | 09:39 | |
| *** ankit_ag has joined #openstack-cinder | 09:40 | |
| *** anand1 has quit IRC | 09:45 | |
| *** anand has joined #openstack-cinder | 09:46 | |
| *** jistr has quit IRC | 09:50 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails https://review.openstack.org/160682 | 09:51 |
| *** jistr has joined #openstack-cinder | 09:52 | |
| *** nshaikh has joined #openstack-cinder | 09:56 | |
| *** yuriy_n17 has joined #openstack-cinder | 09:57 | |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Add cache to Managers' find/findall methods https://review.openstack.org/162139 | 09:57 |
| *** HoloIRCUser2 has quit IRC | 10:01 | |
| *** HoloIRCUser1 has joined #openstack-cinder | 10:02 | |
| *** tbarron has joined #openstack-cinder | 10:03 | |
| *** TobiasE has joined #openstack-cinder | 10:03 | |
| *** deepakcs has quit IRC | 10:04 | |
| *** TobiasE has quit IRC | 10:05 | |
| *** e0ne has joined #openstack-cinder | 10:16 | |
| *** chlong has joined #openstack-cinder | 10:23 | |
| *** stefan_amann has joined #openstack-cinder | 10:28 | |
| *** jistr has quit IRC | 10:38 | |
| *** HoloIRCUser2 has joined #openstack-cinder | 10:38 | |
| *** jistr has joined #openstack-cinder | 10:39 | |
| *** HoloIRCUser1 has quit IRC | 10:39 | |
| *** kmartin has quit IRC | 10:41 | |
| *** takedakn has quit IRC | 10:42 | |
| *** mdenny has quit IRC | 10:43 | |
| *** haomaiwang has quit IRC | 10:49 | |
| nikesh_vedams | Hi | 10:52 |
| *** asselin has quit IRC | 10:52 | |
| e0ne | hi | 10:53 |
| nikesh_vedams | http://paste.openstack.org/show/191554/ | 10:54 |
| nikesh_vedams | i am using devstack with lvm backends in ubuntu 14.04 | 10:55 |
| nikesh_vedams | i want to use a volume group created by me in cinder.conf | 10:56 |
| *** HoloIRCUser2 has quit IRC | 10:56 | |
| nikesh_vedams | but its not taking | 10:56 |
| *** TobiasE has joined #openstack-cinder | 10:56 | |
| *** TobiasE has quit IRC | 10:59 | |
| *** lpetrut has left #openstack-cinder | 11:05 | |
| *** anshul has quit IRC | 11:06 | |
| *** IanGovett has joined #openstack-cinder | 11:06 | |
| *** anshul has joined #openstack-cinder | 11:09 | |
| *** stefan_amann has quit IRC | 11:14 | |
| openstackgerrit | Marcus V R Nascimento proposed openstack/cinder: Error trying to delete snapshots on Hitachi driver https://review.openstack.org/161837 | 11:14 |
| *** e0ne is now known as e0ne_ | 11:21 | |
| *** e0ne_ is now known as e0ne | 11:26 | |
| *** timcl has joined #openstack-cinder | 11:29 | |
| e0ne | nikesh_vedams: did you restsrt cinder-volume? | 11:30 |
| *** anshul has quit IRC | 11:30 | |
| *** anshul has joined #openstack-cinder | 11:42 | |
| *** e0ne is now known as e0ne_ | 11:51 | |
| *** esker has quit IRC | 11:55 | |
| *** e0ne_ has quit IRC | 11:56 | |
| *** anshul has quit IRC | 11:59 | |
| openstackgerrit | Bob Callaway proposed openstack/cinder: Fix LUN misalignment issue with NetApp iSCSI drivers https://review.openstack.org/162376 | 12:02 |
| nikesh_vedams | e0ne: yes | 12:02 |
| nikesh_vedams | i am using devstack | 12:03 |
| nikesh_vedams | so rejoined | 12:03 |
| nikesh_vedams | and restarted | 12:03 |
| nikesh_vedams | cinder volume | 12:03 |
| nikesh_vedams | but even though same problem | 12:03 |
| nikesh_vedams | its not taking explicit volume group | 12:03 |
| *** erlon has joined #openstack-cinder | 12:04 | |
| *** anshul has joined #openstack-cinder | 12:05 | |
| *** jcru has joined #openstack-cinder | 12:05 | |
| *** timcl has quit IRC | 12:09 | |
| nikesh_vedams | e0ne DuncanT : http://paste.openstack.org/show/191568/ | 12:10 |
| *** jcru has quit IRC | 12:11 | |
| *** pradipta has quit IRC | 12:11 | |
| *** nlevinki has quit IRC | 12:12 | |
| *** jcru has joined #openstack-cinder | 12:12 | |
| *** nlevinki has joined #openstack-cinder | 12:14 | |
| *** markvoelker has joined #openstack-cinder | 12:14 | |
| *** cdelatte has joined #openstack-cinder | 12:14 | |
| *** delattec has joined #openstack-cinder | 12:14 | |
| *** akerr has joined #openstack-cinder | 12:14 | |
| *** Administrator_ has joined #openstack-cinder | 12:17 | |
| *** e0ne has joined #openstack-cinder | 12:18 | |
| nikesh_vedams | e0ne: http://paste.openstack.org/show/191568/ | 12:19 |
| 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 found | 12:19 |
| openstackgerrit | Bharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Support over subscription in thin provisioning https://review.openstack.org/162718 | 12:24 |
| *** Mandell has joined #openstack-cinder | 12:24 | |
| *** BharatK has quit IRC | 12:27 | |
| *** Mandell has quit IRC | 12:28 | |
| openstackgerrit | Bharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Support over subscription in thin provisioning https://review.openstack.org/162718 | 12:31 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/cinder: Service object https://review.openstack.org/160417 | 12:35 |
| *** annashen has joined #openstack-cinder | 12:36 | |
| *** jyoti-ranjan has joined #openstack-cinder | 12:37 | |
| *** dulek has quit IRC | 12:38 | |
| *** annashen has quit IRC | 12:41 | |
| *** lpetrut has joined #openstack-cinder | 12:44 | |
| *** esker has joined #openstack-cinder | 12:44 | |
| *** timcl has joined #openstack-cinder | 12:44 | |
| *** rushiagr is now known as rushiagr_away | 12:46 | |
| *** esker has quit IRC | 12:47 | |
| *** tbarron has quit IRC | 12:47 | |
| *** esker has joined #openstack-cinder | 12:50 | |
| nikesh_vedams | DuncanT: there? | 12:50 |
| nikesh_vedams | e0ne : there? | 12:50 |
| *** esker has quit IRC | 12:51 | |
| *** bswartz has quit IRC | 12:51 | |
| *** jyoti-ranjan has quit IRC | 12:51 | |
| *** TobiasE has joined #openstack-cinder | 12:53 | |
| *** mwichmann has joined #openstack-cinder | 12:54 | |
| *** TobiasE has quit IRC | 12:54 | |
| *** TobiasE has joined #openstack-cinder | 12:55 | |
| *** anand has quit IRC | 12:55 | |
| *** TobiasE has quit IRC | 12:59 | |
| *** anuragpalsule has quit IRC | 13:00 | |
| *** Yogi1 has joined #openstack-cinder | 13:01 | |
| *** ganso_ has joined #openstack-cinder | 13:02 | |
| *** rushiagr_away is now known as rushiagr | 13:05 | |
| *** esker has joined #openstack-cinder | 13:10 | |
| *** aix has joined #openstack-cinder | 13:13 | |
| openstackgerrit | Richard Hedlind proposed openstack/cinder: Decrement remaining retries after failed REST call. https://review.openstack.org/163296 | 13:18 |
| *** Longgeek has joined #openstack-cinder | 13:19 | |
| *** rushiagr is now known as rushiagr_away | 13:20 | |
| *** dustins has joined #openstack-cinder | 13:22 | |
| *** mtanino_p has joined #openstack-cinder | 13:22 | |
| *** nellysmitt has joined #openstack-cinder | 13:24 | |
| *** bswartz has joined #openstack-cinder | 13:25 | |
| *** mriedem has joined #openstack-cinder | 13:27 | |
| *** tbarron has joined #openstack-cinder | 13:27 | |
| *** mtanino_p has quit IRC | 13:29 | |
| *** dulek has joined #openstack-cinder | 13:30 | |
| nikesh_vedams | hi anyone online | 13:31 |
| nikesh_vedams | http://paste.openstack.org/show/191568/ | 13:31 |
| nikesh_vedams | did any one face this issue | 13:31 |
| nikesh_vedams | volume group not found | 13:32 |
| nikesh_vedams | for lvm' | 13:32 |
| *** jungleboyj has quit IRC | 13:33 | |
| *** stefan_amann has joined #openstack-cinder | 13:34 | |
| *** mtanino_p has joined #openstack-cinder | 13:38 | |
| *** mtanino_p has quit IRC | 13:38 | |
| *** nellysmitt has quit IRC | 13:39 | |
| *** nellysmi_ has joined #openstack-cinder | 13:42 | |
| *** hemna has joined #openstack-cinder | 13:43 | |
| *** Longgeek has quit IRC | 13:43 | |
| *** nellysmi_ has quit IRC | 13:44 | |
| *** mtanino_p has joined #openstack-cinder | 13:45 | |
| *** amoturi_ has quit IRC | 13:46 | |
| *** amoturi_ has joined #openstack-cinder | 13:46 | |
| *** annegentle has joined #openstack-cinder | 13:49 | |
| *** thangp has joined #openstack-cinder | 13:50 | |
| *** mtanino_p has quit IRC | 13:52 | |
| *** eharney has joined #openstack-cinder | 13:55 | |
| *** Longgeek has joined #openstack-cinder | 13:55 | |
| *** r-daneel has joined #openstack-cinder | 13:55 | |
| *** TobiasE has joined #openstack-cinder | 13:56 | |
| *** e0ne has quit IRC | 13:57 | |
| *** e0ne has joined #openstack-cinder | 13:57 | |
| *** TobiasE has quit IRC | 13:58 | |
| *** changbl has quit IRC | 13:58 | |
| *** stefan_amann has quit IRC | 13:59 | |
| *** topshare has joined #openstack-cinder | 14:00 | |
| *** topshare has quit IRC | 14:00 | |
| *** tellesnobrega_ has joined #openstack-cinder | 14:00 | |
| *** timcl has left #openstack-cinder | 14:04 | |
| *** xyang has joined #openstack-cinder | 14:05 | |
| *** rushiagr_away is now known as rushiagr | 14:06 | |
| *** topshare has joined #openstack-cinder | 14:06 | |
| *** lpetrut has quit IRC | 14:06 | |
| *** topshare has quit IRC | 14:06 | |
| *** topshare has joined #openstack-cinder | 14:07 | |
| *** changbl has joined #openstack-cinder | 14:09 | |
| *** mdbooth has quit IRC | 14:09 | |
| *** shyama has quit IRC | 14:09 | |
| *** alonmarx_ has quit IRC | 14:10 | |
| *** dulek has quit IRC | 14:11 | |
| *** mdbooth has joined #openstack-cinder | 14:14 | |
| *** dulek has joined #openstack-cinder | 14:15 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Harden fake_notifier asserts https://review.openstack.org/163191 | 14:19 |
| *** timcl has joined #openstack-cinder | 14:19 | |
| *** lpetrut has joined #openstack-cinder | 14:19 | |
| *** anand1 has joined #openstack-cinder | 14:22 | |
| *** topshare has quit IRC | 14:23 | |
| *** anand1 has left #openstack-cinder | 14:24 | |
| *** topshare has joined #openstack-cinder | 14:25 | |
| *** esker has quit IRC | 14:26 | |
| *** HoloIRCUser1 has joined #openstack-cinder | 14:27 | |
| *** jungleboyj has joined #openstack-cinder | 14:27 | |
| *** [1]Thelo has joined #openstack-cinder | 14:27 | |
| *** Thelo has quit IRC | 14:30 | |
| *** [1]Thelo is now known as Thelo | 14:30 | |
| *** ankit_ag has quit IRC | 14:30 | |
| *** lcurtis has joined #openstack-cinder | 14:32 | |
| *** e0ne is now known as e0ne_ | 14:32 | |
| *** e0ne_ is now known as e0ne | 14:33 | |
| *** eharney has quit IRC | 14:34 | |
| *** mtanino has joined #openstack-cinder | 14:36 | |
| *** dulek has quit IRC | 14:37 | |
| *** annashen has joined #openstack-cinder | 14:38 | |
| *** esker has joined #openstack-cinder | 14:38 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator https://review.openstack.org/157441 | 14:38 |
| *** jungleboyj has quit IRC | 14:38 | |
| *** BharatK has joined #openstack-cinder | 14:42 | |
| *** marcusvrn has joined #openstack-cinder | 14:43 | |
| *** annashen has quit IRC | 14:43 | |
| *** marcusvrn1 has quit IRC | 14:45 | |
| *** TobiasE has joined #openstack-cinder | 14:46 | |
| *** TobiasE has quit IRC | 14:48 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add entry create and cast tasks to manage workflow https://review.openstack.org/139071 | 14:49 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix lvm manage existing volume https://review.openstack.org/156939 | 14:49 |
| *** mdenny has joined #openstack-cinder | 14:49 | |
| *** stefan_amann has joined #openstack-cinder | 14:52 | |
| *** fischerw has joined #openstack-cinder | 14:54 | |
| *** timcl1 has joined #openstack-cinder | 14:54 | |
| *** timcl has quit IRC | 14:56 | |
| *** marcusvrn1 has joined #openstack-cinder | 14:56 | |
| *** hemnafk is now known as hemna | 14:56 | |
| *** marcusvrn has quit IRC | 14:58 | |
| *** topshare has quit IRC | 14:58 | |
| *** Mandell has joined #openstack-cinder | 14:58 | |
| hemna | mornin | 15:00 |
| *** dannywilson has joined #openstack-cinder | 15:02 | |
| *** dannywilson has quit IRC | 15:02 | |
| *** marcusvrn1 has quit IRC | 15:03 | |
| *** dannywilson has joined #openstack-cinder | 15:03 | |
| *** eharney has joined #openstack-cinder | 15:03 | |
| DuncanT | Morning, hemna | 15:04 |
| *** Yogi11 has joined #openstack-cinder | 15:05 | |
| *** tellesnobrega_ has quit IRC | 15:07 | |
| *** Yogi1 has quit IRC | 15:08 | |
| *** bkopilov has quit IRC | 15:09 | |
| *** marcusvrn has joined #openstack-cinder | 15:09 | |
| *** zhipeng has joined #openstack-cinder | 15:10 | |
| *** alonmarx has joined #openstack-cinder | 15:11 | |
| nikesh_vedams | DuncanT: hi busy? | 15:12 |
| *** kmartin has joined #openstack-cinder | 15:12 | |
| DuncanT | nikesh_vedams: Yes, sorry# | 15:13 |
| *** alonmarx has quit IRC | 15:13 | |
| *** marcusvrn has quit IRC | 15:14 | |
| nikesh_vedams | i am struggling in this http://paste.openstack.org/show/191568/ | 15:14 |
| nikesh_vedams | ok | 15:15 |
| nikesh_vedams | oh i got the mistake | 15:16 |
| nikesh_vedams | sudo 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-zcfdX3 | 15:17 |
| nikesh_vedams | this is working | 15:17 |
| nikesh_vedams | but 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 found | 15:17 |
| nikesh_vedams | this not working | 15:17 |
| nikesh_vedams | Might be this is a bug | 15:18 |
| nikesh_vedams | in devstack | 15:18 |
| jgriffith | nikesh_vedams: it would appear that the /etc/cinder lvm.conf has a filter against your dev | 15:18 |
| nikesh_vedams | which is not taking explicitly created volume-group | 15:19 |
| jgriffith | nikesh_vedams: open the file and look for "filter", you'll see that only certain devices are queried | 15:19 |
| mtanino | nikesh_vedams: you should check LVM filter in /etc/cinder/lvm.conf | 15:19 |
| jgriffith | nikesh_vedams: make it match /etc/lvm.conf if you don't understand lvm filtering | 15:19 |
| jgriffith | ie copy/paste the entire file | 15:19 |
| jgriffith | mtanino: :) | 15:20 |
| nikesh_vedams | mtanino jgriffith : ya sure thanks | 15:20 |
| *** thingee has joined #openstack-cinder | 15:21 | |
| mtanino | nikesh_vedams: I think old bash have a bug(in fedora20), so devstack can't proper LVM filter when we use devstack. | 15:21 |
| mtanino | nikesh_vedams: I met same situation. | 15:21 |
| mtanino | can't/can't create/ | 15:22 |
| *** bkopilov has joined #openstack-cinder | 15:23 | |
| *** TobiasE has joined #openstack-cinder | 15:24 | |
| *** TobiasE has quit IRC | 15:25 | |
| *** Administrator_ has quit IRC | 15:26 | |
| *** Administrator_ has joined #openstack-cinder | 15:27 | |
| *** david-lyle_afk is now known as david-lyle | 15:28 | |
| *** thingee_ has joined #openstack-cinder | 15:29 | |
| *** mtanino is now known as mtanino_away | 15:30 | |
| *** jdurgin1 has joined #openstack-cinder | 15:31 | |
| *** Mandell has quit IRC | 15:31 | |
| *** thingee has quit IRC | 15:32 | |
| *** dulek has joined #openstack-cinder | 15:36 | |
| *** maune has quit IRC | 15:37 | |
| *** Apoorva has joined #openstack-cinder | 15:38 | |
| *** jungleboyj has joined #openstack-cinder | 15:38 | |
| *** zhipeng has quit IRC | 15:39 | |
| thingee_ | jungleboyj: can we get some to +2/A that systemz one? | 15:39 |
| thingee_ | hemna has a minus 1 on it though | 15:39 |
| jungleboyj | thingee_: Yeah, I wanted to +2/A it but thought hemna should do it. | 15:40 |
| jungleboyj | I pinged him earlier this morning. | 15:40 |
| *** dannywilson has quit IRC | 15:40 | |
| thingee_ | ok, has the new cert tests been posted? | 15:40 |
| markus_z | thingee_: 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-cinder | 15:41 | |
| thingee_ | markus_z: hemna noted there were two failures in it ;) | 15:41 |
| *** dulek has quit IRC | 15:41 | |
| jungleboyj | markus_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_z | thingee_: 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 OK | 15:42 |
| markus_z | After using that patch we were able to run it successfully. | 15:42 |
| thingee_ | thanks :) | 15:42 |
| *** afazekas_ has quit IRC | 15:42 | |
| markus_z | thingee_: np :) | 15:42 |
| jungleboyj | thingee_: No problem. Tried to hold down the fort yesterday. | 15:42 |
| thingee_ | jungleboyj: all of you did an awesome job | 15:42 |
| thingee_ | seriously | 15:42 |
| markus_z | For later reference: https://review.openstack.org/#/c/163038/ | 15:42 |
| jungleboyj | Oh good, we got this one in too. https://review.openstack.org/#/c/149725/ | 15:43 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/cinder: Service object https://review.openstack.org/160417 | 15:43 |
| thingee_ | jungleboyj: yeah I just went through and updated k-3 page, etc. lookin' good! | 15:43 |
| *** thingee_ is now known as thingee | 15:43 | |
| *** Longgeek has quit IRC | 15:45 | |
| jungleboyj | :-) | 15:45 |
| *** nellysmitt has joined #openstack-cinder | 15:45 | |
| *** marcusvrn has joined #openstack-cinder | 15:45 | |
| *** marcusvrn1 has joined #openstack-cinder | 15:47 | |
| hemna | ok I'm looking at 149256 again | 15:47 |
| thingee | jungleboyj: I saw you and hemna talking about it earlier, but yes I'm not accepting any more feature merges. especially if it's not targeted | 15:47 |
| thingee | so feel free to -2 | 15:47 |
| thingee | we'll make an exception for this since it's very close if hemna is fine with the test results. | 15:48 |
| hemna | thingee, so there is also a companion patch for os-brick that I had as a requirement as well | 15:48 |
| e0ne | imo, it's not fair to not merge #131182 (shadow tables) with reason "a feature freeze deadline passed" :( | 15:48 |
| markus_z | hemna: That os-brick patch is here: https://review.openstack.org/#/c/162678/ | 15:49 |
| hemna | markus_z, thank you | 15:49 |
| *** marcusvrn has quit IRC | 15:49 | |
| *** victorfeitosa has joined #openstack-cinder | 15:49 | |
| *** nellysmitt has quit IRC | 15:50 | |
| *** xyang has quit IRC | 15:50 | |
| flip214 | for devstack, do I have to use trusty, or is vivid an option, too? | 15:50 |
| thingee | e0ne: well it's what we all agreed on. | 15:50 |
| *** xyang1 has joined #openstack-cinder | 15:50 | |
| e0ne | hemna: my congrantulations! your patch is almost merged | 15:50 |
| thingee | e0ne: not sure why that's not fair. | 15:51 |
| thingee | e0ne: we need time to just let things sit and catch issues before the cut | 15:51 |
| hemna | e0ne, thank you. It's been a LONG time coming. | 15:51 |
| e0ne | thingee: it was holded on to make others patches less complicated and w/o shadow migrations | 15:51 |
| markus_z | hemna: 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 |
| thingee | e0ne: ah that's true | 15:52 |
| hemna | markus_z, yah I just read that | 15:52 |
| thingee | e0ne: we'll talk about it in the meeting | 15:52 |
| hemna | e0ne, did you give any more thought to my idea of an alternate way of doing shadow tables ? | 15:52 |
| *** marcusvrn has joined #openstack-cinder | 15:52 | |
| hemna | instead of forcing it on devs in db migration scripts, etc. | 15:52 |
| flip214 | or can I even use debian testing/unstable as a devstack base? | 15:52 |
| e0ne | thingee: i was agree with deadlines since we started discussion on meeting | 15:53 |
| e0ne | hemna: i'm going to make poc for oslo.db but have not time to it today | 15:53 |
| markus_z | hemna: 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 |
| hemna | e0ne, ok cool. we should talk about it in Vancouver | 15:53 |
| hemna | markus_z, np | 15:54 |
| *** marcusvrn1 has quit IRC | 15:54 | |
| *** coolsvap is now known as coolsvap|afk | 15:54 | |
| e0ne | hemna: ok, sounds good | 15:54 |
| e0ne | thingee, 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 rwsu | 15:55 | |
| hemna | e0ne, yah that would be why I would -2 it at this point. | 15:55 |
| *** diemt has joined #openstack-cinder | 15:56 | |
| hemna | jungleboyj, ok done | 15:57 |
| hemna | markus_z, good to go | 15:57 |
| *** ronis has quit IRC | 15:57 | |
| markus_z | hemna: Yeah, thank you! :) | 15:58 |
| *** tellesnobrega has quit IRC | 15:58 | |
| hemna | thanks for the work man. appreciate it | 15:58 |
| thingee | good work everyone :) | 15:58 |
| hemna | markus_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_z | jungleboyj: Thanks for the time and effort you spent too! | 15:59 |
| thingee | hemna: +1 | 15:59 |
| *** tellesnobrega has joined #openstack-cinder | 15:59 | |
| markus_z | hemna: 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 |
| hemna | ok so I'll post another follow up to multiattach and update the unit tests to change the order, etc. | 15:59 |
| jungleboyj | hemna: Thank you! | 15:59 |
| *** ebalduf has joined #openstack-cinder | 15:59 | |
| jungleboyj | markus_z: Thank you for working with us on this. | 16:00 |
| *** Yogi11 has quit IRC | 16:00 | |
| hemna | markus_z, the plan is to migrate to os-brick in L. | 16:00 |
| hemna | markus_z, I have a WIP patch up already for cinder. I'll just have to rebase it and such. | 16:00 |
| e0ne | hemna: ok. it would be fair enough... | 16:00 |
| hemna | we should do an official release of os-brick when we ship cinder for K | 16:00 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/157642 | 16:00 |
| hemna | so os-brick is in pypi for L. | 16:00 |
| *** patrickeast has joined #openstack-cinder | 16:00 | |
| markus_z | jungleboyj: Anytime again | 16:01 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add multiattach capabilities injection https://review.openstack.org/159650 | 16:01 |
| markus_z | hemna: Yes, I have seen it. I have knowledge of this area in Nova as well if there is help needed. | 16:01 |
| hemna | markus_z, nice. | 16:01 |
| hemna | markus_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-cinder | 16:02 | |
| jgriffith | we meeting? | 16:02 |
| tbarron | jgriffith: yup | 16:03 |
| *** _cjones_ has joined #openstack-cinder | 16:03 | |
| *** TobiasE has quit IRC | 16:03 | |
| *** rushil has joined #openstack-cinder | 16:08 | |
| *** victor-feitosa has joined #openstack-cinder | 16:11 | |
| flip214 | hemna: thanks for the offer, but I've got an appointment in 20 mins. perhaps tomorrow at ~2000 CET? | 16:12 |
| *** victorfeitosa has quit IRC | 16:12 | |
| *** thrawn01 has quit IRC | 16:13 | |
| *** nkrinner has quit IRC | 16:13 | |
| *** briancurtin has quit IRC | 16:13 | |
| *** rmesta has joined #openstack-cinder | 16:14 | |
| *** marcusvrn has quit IRC | 16:14 | |
| hemna | flip214, I'll be here during normal PST hours :P | 16:14 |
| *** rmesta has left #openstack-cinder | 16:14 | |
| *** marcusvrn has joined #openstack-cinder | 16:14 | |
| *** russellb has quit IRC | 16:14 | |
| *** zigo_ has quit IRC | 16:14 | |
| *** jbernard has quit IRC | 16:14 | |
| *** zigo_ has joined #openstack-cinder | 16:14 | |
| *** asselin_ has quit IRC | 16:14 | |
| *** jbernard has joined #openstack-cinder | 16:15 | |
| *** russellb has joined #openstack-cinder | 16:15 | |
| *** asselin_ has joined #openstack-cinder | 16:15 | |
| *** thrawn01 has joined #openstack-cinder | 16:15 | |
| flip214 | yeah, that's why I suggested that time ;) | 16:16 |
| flip214 | thanks, will get back to you tomorrow. | 16:16 |
| *** nkrinner has joined #openstack-cinder | 16:17 | |
| *** fischerw has quit IRC | 16:18 | |
| *** fischerw has joined #openstack-cinder | 16:18 | |
| *** annegentle has quit IRC | 16:18 | |
| *** briancurtin has joined #openstack-cinder | 16:19 | |
| *** anshul has quit IRC | 16:20 | |
| *** kallebe has joined #openstack-cinder | 16:22 | |
| *** marcusvrn1 has joined #openstack-cinder | 16:22 | |
| *** alecv has quit IRC | 16:22 | |
| *** marcusvrn has quit IRC | 16:23 | |
| *** marcusvrn has joined #openstack-cinder | 16:25 | |
| *** nshaikh has left #openstack-cinder | 16:26 | |
| *** esker has quit IRC | 16:26 | |
| *** leeantho has joined #openstack-cinder | 16:27 | |
| *** marcusvrn1 has quit IRC | 16:27 | |
| *** emagana has quit IRC | 16:28 | |
| *** emagana has joined #openstack-cinder | 16:28 | |
| *** mtanino_away has quit IRC | 16:32 | |
| *** emagana has quit IRC | 16:33 | |
| *** timcl1 has quit IRC | 16:33 | |
| *** dulek has joined #openstack-cinder | 16:33 | |
| *** rcallawa has joined #openstack-cinder | 16:33 | |
| *** timcl has joined #openstack-cinder | 16:33 | |
| *** vilobhmm has joined #openstack-cinder | 16:34 | |
| *** amann has joined #openstack-cinder | 16:34 | |
| *** stefan_amann has quit IRC | 16:37 | |
| jgriffith | vishy: that doesn't look too bad, thanks for jumping in | 16:38 |
| hemna | jaypipes, so the cinder multiattach code just merged. :) | 16:39 |
| jgriffith | vishy: I'd really like to go this route rather than introduce the public/private thing on all Cinder objects | 16:39 |
| *** nlevinki has quit IRC | 16:39 | |
| *** annashen has joined #openstack-cinder | 16:40 | |
| *** annashen has quit IRC | 16:45 | |
| *** jdurgin1 has quit IRC | 16:45 | |
| *** Longgeek has joined #openstack-cinder | 16:45 | |
| jaypipes | hemna: I saw that :) yay! | 16:47 |
| *** asselin__ has joined #openstack-cinder | 16:48 | |
| *** annashen has joined #openstack-cinder | 16:48 | |
| *** TobiasE has joined #openstack-cinder | 16:50 | |
| *** asselin_ has quit IRC | 16:52 | |
| *** vilobhmm has quit IRC | 16:52 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Backup object https://review.openstack.org/157085 | 16:52 |
| *** openstack has joined #openstack-cinder | 16:55 | |
| *** rhagarty__ has joined #openstack-cinder | 16:56 | |
| *** takedakn has joined #openstack-cinder | 16:57 | |
| *** timcl1 has joined #openstack-cinder | 16:58 | |
| *** timcl has quit IRC | 16:58 | |
| *** cbader has joined #openstack-cinder | 16:59 | |
| morganfainberg | thingee, let me know next time your in SoCal. | 16:59 |
| *** patrickeast has quit IRC | 17:00 | |
| thingee | morganfainberg: I will be tomorrow actually! | 17:00 |
| thingee | morganfainberg: leaving philly now | 17:00 |
| *** TobiasE has quit IRC | 17:00 | |
| *** rcallawa has quit IRC | 17:01 | |
| hemna | afk...beating. | 17:01 |
| morganfainberg | thingee, coffee meetup when youre here? or... whiskey? | 17:01 |
| marcusvrn | thangp: 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 |
| thingee | morganfainberg: sounds good | 17:01 |
| marcusvrn | link:https://review.openstack.org/#/c/161837 | 17:01 |
| *** Longgeek has quit IRC | 17:02 | |
| *** Yogi1 has joined #openstack-cinder | 17:04 | |
| jungleboyj | hemna: afk beating? | 17:04 |
| * jungleboyj doesn't want to know. | 17:04 | |
| smcginnis | LOL! | 17:05 |
| *** dulek has quit IRC | 17:05 | |
| *** thingee has quit IRC | 17:05 | |
| *** takedakn has quit IRC | 17:05 | |
| * jungleboyj is laughing uncontrollably. Been that kind of day. | 17:06 | |
| *** dannywilson has joined #openstack-cinder | 17:06 | |
| *** annashen has quit IRC | 17:07 | |
| *** annashen has joined #openstack-cinder | 17:07 | |
| *** esker has joined #openstack-cinder | 17:08 | |
| *** timcl1 has quit IRC | 17:08 | |
| *** takedakn has joined #openstack-cinder | 17:09 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Sort snapshots in create CG from CG snapshot https://review.openstack.org/163288 | 17:09 |
| *** Adriano_ has joined #openstack-cinder | 17:11 | |
| jungleboyj | smcginnis: Are you too having the urge to just go sit out in the sun? | 17:11 |
| smcginnis | jungleboyj: Oh yeah. It's lovely. :) | 17:12 |
| smcginnis | jungleboyj: Nice there too I assume? Heard it's actually warmer here than in Austin right now. | 17:13 |
| jungleboyj | smcginnis: Indeed. Almost couldn't come back in after lunch yesterday. Spent part of the afternoon working from my deck. :-) | 17:13 |
| *** dannywilson has quit IRC | 17:13 | |
| smcginnis | jungleboyj: 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 IRC | 17:13 | |
| *** harlowja_away is now known as harlowja_ | 17:14 | |
| jungleboyj | Almost 50 on my deck right now. Got to 60 yesterday. Supposed to do taht again today. | 17:14 |
| *** mgagne_PHL is now known as mgagne | 17:14 | |
| jungleboyj | smcginnis: Motorcycle!?! | 17:14 |
| smcginnis | jungleboyj: Might have to take off early today! | 17:14 |
| jungleboyj | smcginnis: +2 | 17:15 |
| *** jordanP has joined #openstack-cinder | 17:16 | |
| *** bswartz has joined #openstack-cinder | 17:16 | |
| *** HoloIRCUser1 has quit IRC | 17:16 | |
| openstackgerrit | Bharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Support over subscription in thin provisioning https://review.openstack.org/162718 | 17:18 |
| *** e0ne is now known as e0ne_ | 17:18 | |
| *** leeantho has quit IRC | 17:19 | |
| *** amann has quit IRC | 17:19 | |
| *** resker has joined #openstack-cinder | 17:20 | |
| *** vilobhmm has joined #openstack-cinder | 17:21 | |
| *** timcl has joined #openstack-cinder | 17:21 | |
| *** chlong has quit IRC | 17:21 | |
| *** asselin__ has quit IRC | 17:22 | |
| *** markus_z has quit IRC | 17:23 | |
| *** esker has quit IRC | 17:24 | |
| *** yuriy_n17 has quit IRC | 17:25 | |
| *** dflorea has joined #openstack-cinder | 17:26 | |
| *** e0ne_ has quit IRC | 17:29 | |
| *** _cjones_ has quit IRC | 17:30 | |
| *** e0ne has joined #openstack-cinder | 17:30 | |
| *** HoloIRCUser2 has joined #openstack-cinder | 17:31 | |
| *** leeantho has joined #openstack-cinder | 17:31 | |
| *** timcl has quit IRC | 17:33 | |
| e0ne | zuul shows 167 changes in pipeline :-( | 17:33 |
| *** ronis has joined #openstack-cinder | 17:34 | |
| *** jdandrea has quit IRC | 17:35 | |
| *** jdandrea has joined #openstack-cinder | 17:35 | |
| *** bkopilov has quit IRC | 17:35 | |
| *** kmartin has quit IRC | 17:37 | |
| *** takedakn has quit IRC | 17:37 | |
| *** _cjones_ has joined #openstack-cinder | 17:38 | |
| *** asselin has joined #openstack-cinder | 17:38 | |
| openstackgerrit | Alex Meade proposed openstack/cinder: Add tracing facility to NetApp ONTAP drivers https://review.openstack.org/157439 | 17:38 |
| *** bkopilov has joined #openstack-cinder | 17:39 | |
| *** BharatK has quit IRC | 17:39 | |
| *** kmartin has joined #openstack-cinder | 17:40 | |
| *** Mandell has joined #openstack-cinder | 17:40 | |
| *** annashen has quit IRC | 17:41 | |
| *** e0ne has quit IRC | 17:41 | |
| *** annashen has joined #openstack-cinder | 17:41 | |
| *** annegentle has joined #openstack-cinder | 17:41 | |
| *** rhagarty_ has quit IRC | 17:42 | |
| *** rhagarty__ has quit IRC | 17:42 | |
| *** rhagarty has quit IRC | 17:42 | |
| *** devlaps has joined #openstack-cinder | 17:42 | |
| *** nellysmitt has joined #openstack-cinder | 17:46 | |
| *** nellysmitt has quit IRC | 17:50 | |
| *** mtanino has joined #openstack-cinder | 17:52 | |
| *** lpetrut has quit IRC | 17:55 | |
| *** bkopilov has quit IRC | 17:55 | |
| *** kallebe has quit IRC | 17:55 | |
| *** kallebe has joined #openstack-cinder | 17:56 | |
| *** bkopilov has joined #openstack-cinder | 17:57 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: WIP - Quota Objects https://review.openstack.org/163232 | 17:58 |
| *** marcusvrn has quit IRC | 18:00 | |
| hemna | jungleboyj, 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-cinder | 18:02 | |
| jungleboyj | :-) We are having spring here, 50 and Sunny. | 18:02 |
| jungleboyj | 60's the rest of the week. :-) Hoo ray! | 18:03 |
| DuncanT | 26 here yesterday | 18:04 |
| jgriffith | DuncanT: Celsius :) | 18:04 |
| DuncanT | High seventies in your funny money | 18:04 |
| jgriffith | LOL | 18:04 |
| jungleboyj | DuncanT: Was just converting. Nice. | 18:05 |
| *** emagana has joined #openstack-cinder | 18:05 | |
| *** angela-s has joined #openstack-cinder | 18:06 | |
| jungleboyj | hemna: Why sandbags? Because it is drizzling? | 18:06 |
| *** Longgeek has quit IRC | 18:09 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Fix sqlalchemy reuse in multi-backend children https://review.openstack.org/163551 | 18:10 |
| hemna | jungleboyj, yah | 18:10 |
| hemna | jungleboyj, it gets wet, people freak out, news stations talk about the impending mega storm, etc | 18:10 |
| eharney | this bug stinks ^ | 18:10 |
| *** Maike has quit IRC | 18:11 | |
| smcginnis | Swanson: ^^ | 18:11 |
| *** rhagarty has joined #openstack-cinder | 18:12 | |
| *** rhagarty_ has joined #openstack-cinder | 18:12 | |
| *** takedakn has joined #openstack-cinder | 18:12 | |
| *** kbyrne has quit IRC | 18:13 | |
| smcginnis | eharney: We want that on every init? | 18:13 |
| *** kbyrne has joined #openstack-cinder | 18:14 | |
| eharney | smcginnis: all except the first one, i think, and it doesn't have any real penalty there AFAIK | 18:14 |
| smcginnis | eharney: So the idea is at the beginning it cleans up anything left over? | 18:15 |
| eharney | smcginnis: see comments in the LP bug from Mike Bayer | 18:15 |
| DuncanT | smcginnis: The one time we don't want it, there's nothing in the pool anyway | 18:15 |
| eharney | DuncanT: right | 18:15 |
| eharney | smcginnis: it's mostly to clean up things that leaked in from a parent process/thread | 18:15 |
| smcginnis | eharney: Ah, hadn't followed the bug yet. Looking now. Makes sense though. | 18:15 |
| smcginnis | eharney: I see, makes sense. | 18:16 |
| *** marcusvrn has joined #openstack-cinder | 18:16 | |
| eharney | i'm still not really sure how this is just showing up now | 18:17 |
| eharney | but i think the secure NFS patch triggered it to be a lot more likely to occur | 18:17 |
| eharney | since it actually started using the db before forking some of these children, when it wasn't before | 18:18 |
| smcginnis | eharney: 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 IRC | 18:19 | |
| DuncanT | eharney: Yeah, if nothing touches the db before fork, the pool is empty so nothing gets leaked | 18:19 |
| eharney | smcginnis: 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 in | 18:20 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Fix unit tests for multiattach patch https://review.openstack.org/163554 | 18:20 |
| *** aix has quit IRC | 18:21 | |
| smcginnis | eharney: Makes sense. Thanks for talking it through. | 18:21 |
| smcginnis | hemna: Can we get a tempest test added for multiattach too? | 18:21 |
| smcginnis | hemna: That way if a backend supports it we'll get some real coverage? | 18:22 |
| hemna | smcginnis, so jaypipes was going to do that for nova. | 18:23 |
| hemna | smcginnis, so that's the plan for when the end to end pieces are all in place. | 18:23 |
| smcginnis | hemna: OK, cool. Just thought of that over lunch and thought I should bring it up. | 18:24 |
| hemna | smcginnis, buy yah, we need to gate on it eventually. | 18:24 |
| jaypipes | hemna: for tempest. TobiE was doing the nova pieces... | 18:24 |
| hemna | jaypipes, ok great. | 18:24 |
| jaypipes | hemna: next step is to get the cinderclient changes in and released I believe. | 18:24 |
| hemna | jaypipes, yah, there isn't much the nova guys can do w/o the cinderclient landing. | 18:24 |
| hemna | and that isn't happening for K. | 18:24 |
| hemna | so, I'll push to get the client in L as soon as it opens up. | 18:25 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Fix sqlalchemy reuse in multi-backend children https://review.openstack.org/163551 | 18:25 |
| eharney | DuncanT: smcginnis: heh, whoops, would be good if it actually passed pep8 ^ | 18:25 |
| DuncanT | eharney: Meh, that's what we have jenkins for | 18:26 |
| hemna | eharney, ew. so all the c-vol services used the same objects?! | 18:26 |
| hemna | DuncanT, +1 :P | 18:26 |
| eharney | hemna: apparently | 18:26 |
| hemna | doh | 18:26 |
| eharney | well, copies of the same base object | 18:26 |
| eharney | which was "ok" until it actually had meaningful state in it | 18:26 |
| DuncanT | Do we ahve any similar issue with RPC connections or similar, I wonder? | 18:27 |
| *** nellysmitt has joined #openstack-cinder | 18:27 | |
| DuncanT | s/agve/have/ | 18:27 |
| smcginnis | eharney: I was actually wondering about an extra line before the constant. Guess I was looking in the wrong place. :) | 18:28 |
| hemna | eharney, 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 |
| eharney | hemna: well... this was what Mike specifically recommended we do | 18:30 |
| hemna | ok, I noted it in the review, fwiw. | 18:30 |
| *** lpetrut has joined #openstack-cinder | 18:30 | |
| *** jaypipes is now known as jaypipes-afk | 18:34 | |
| *** marcusvrn has joined #openstack-cinder | 18:34 | |
| *** timcl has joined #openstack-cinder | 18:38 | |
| DuncanT | There's some discussion on the mailing list about trying to fix this in OSLO incubator | 18:38 |
| kmartin | Anyone know if the python-cinder client has the same Kilo deadlines as cinder? | 18:38 |
| DuncanT | kmartin: No, we can cut a client release at any time | 18:38 |
| DuncanT | kmartin: It is only as managed as the PTL makes it | 18:39 |
| DuncanT | kmartin: And you're way bigger than Mike. Put him in a headlock and nuggy him until he does a relase | 18:39 |
| DuncanT | kmartin: Actually, that won't work with his hair, will it? | 18:40 |
| *** timcl has quit IRC | 18:40 | |
| kmartin | DuncanT, 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 FFE | 18:41 |
| DuncanT | kmartin: It always has been before, and there's no official process required | 18:41 |
| *** timcl has joined #openstack-cinder | 18:41 | |
| *** timcl has quit IRC | 18:43 | |
| *** jistr has quit IRC | 18:46 | |
| hemna | jungleboyj, ping | 18:48 |
| *** dflorea has quit IRC | 18:50 | |
| *** dflorea has joined #openstack-cinder | 18:51 | |
| *** ho has joined #openstack-cinder | 18:52 | |
| nikesh_vedams | hi | 18:53 |
| nikesh_vedams | now after adding "filter = [ "a/.*/" ]" in /etc/cinder/lvm.conf , i am not getting volume-group not found error | 18:54 |
| nikesh_vedams | but still volume creation is failing | 18:54 |
| DuncanT | nikesh_vedams: What is the error? | 18:54 |
| *** dflorea has quit IRC | 18:55 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: NFS backup driver https://review.openstack.org/149726 | 18:56 |
| *** patrickeast has joined #openstack-cinder | 18:56 | |
| *** Administrator__ has joined #openstack-cinder | 18:56 | |
| *** dannywilson has joined #openstack-cinder | 18:56 | |
| *** mtanino has quit IRC | 18:57 | |
| *** mtanino has joined #openstack-cinder | 18:57 | |
| nikesh_vedams | http://paste.openstack.org/show/191672/ | 18:57 |
| *** bswartz has quit IRC | 18:57 | |
| *** marcusvrn has quit IRC | 18:58 | |
| *** e0ne has joined #openstack-cinder | 18:58 | |
| *** nellysmitt has quit IRC | 18:59 | |
| *** Mandell has quit IRC | 18:59 | |
| *** nellysmitt has joined #openstack-cinder | 18:59 | |
| *** Administrator_ has quit IRC | 19:00 | |
| *** thingee has joined #openstack-cinder | 19:00 | |
| nikesh_vedams | DuncanT: http://paste.openstack.org/show/191672/ http://paste.openstack.org/show/191674/ | 19:01 |
| *** marcusvrn has joined #openstack-cinder | 19:01 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator https://review.openstack.org/157441 | 19:03 |
| *** bswartz has joined #openstack-cinder | 19:03 | |
| nikesh_vedams | Error 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.fl | 19:05 |
| jgriffith | nikesh_vedams: what's the "name" in your spec there? | 19:05 |
| jgriffith | nikesh_vedams: that's not going to work | 19:06 |
| jgriffith | you need an extra spec key: volume_backend_name=lvmdriver-2 | 19:06 |
| e0ne | does anybody know why we have so many patches in check queue? | 19:06 |
| jgriffith | NOT name=lvmdriver-2 | 19:06 |
| tbarron | xyang, jungleboyj, e0ne, DuncanT: https://review.openstack.org/149726 | 19:08 |
| *** jordanP has quit IRC | 19:08 | |
| *** ebalduf has quit IRC | 19:09 | |
| *** rushiagr is now known as rushiagr_away | 19:10 | |
| *** HoloIRCUser2 has quit IRC | 19:11 | |
| *** Mandell has joined #openstack-cinder | 19:11 | |
| *** annashen has quit IRC | 19:15 | |
| *** kallebe has left #openstack-cinder | 19:20 | |
| *** timcl has joined #openstack-cinder | 19:20 | |
| *** rushil has quit IRC | 19:20 | |
| DuncanT | tbarron: Why is backup_compression_model being removed? | 19:20 |
| marcusvrn | e0ne: hi! could you take a look on https://review.openstack.org/#/c/161837 to check if the commit message is ok as you asked me? | 19:21 |
| tbarron | DuncanT: I just put a note in the review. That option isn't being used. | 19:21 |
| *** ebalduf has joined #openstack-cinder | 19:22 | |
| DuncanT | tbarron: Cool, thanks | 19:22 |
| tbarron | It's dead code left over from my first (unpushed) rebase attempt this weekend. | 19:22 |
| marcusvrn | xyang1: thangp: thanks for reviewing the patch | 19:22 |
| thangp | thangp: welcome | 19:22 |
| tbarron | Originally I was going to use a streaming compression model for NFS and a chunked model for Swift. | 19:22 |
| tbarron | and hide the difference. | 19:22 |
| thangp | marcusvrn: welcome | 19:22 |
| DuncanT | tbarron: Same with get_object_reader ? | 19:23 |
| *** diegows has joined #openstack-cinder | 19:23 | |
| tbarron | But with xyang's changes for incremental backup I can no longer treat the NFS data store as a tape driver :-) | 19:23 |
| tbarron | DuncanT: yes. | 19:23 |
| tbarron | Originally I wasn't doing the context handler with ... as object_reader. | 19:24 |
| tbarron | But I implemented all the Swift and NFS object_reader as context handlers before I | 19:24 |
| DuncanT | tbarron: 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 |
| tbarron | pushed the last change set. | 19:24 |
| *** timcl has quit IRC | 19:25 | |
| tbarron | And I invoked them that way, but stupidly left the non-context handler invocation first. | 19:25 |
| tbarron | Didn't hurt Swift, but it was stupid. | 19:25 |
| tbarron | Found it when I did NFS. | 19:25 |
| tbarron | Swift unit tests still pass. | 19:25 |
| DuncanT | backup_container is unused too | 19:25 |
| tbarron | DuncanT: I want to present config options to the end user with abstractions that make sense to the end user. | 19:26 |
| tbarron | End user doesn't have to know a backup file is a "chunk". | 19:26 |
| nikesh_vedams | jgriffith: its set like that | 19:26 |
| tbarron | "chunk" makes more sense if you are thinking of the backup repo as an object store. | 19:26 |
| nikesh_vedams | http://paste.openstack.org/show/191674/ | 19:27 |
| tbarron | DuncanT: there is an earlier discussion of this with patrickeast in the review IIRC. | 19:27 |
| *** resker has quit IRC | 19:27 | |
| DuncanT | tbarron: I'll go read. | 19:28 |
| DuncanT | tbarron: Is backup_container used anywhere? | 19:28 |
| tbarron | DunctanT: From a developer perspective they are of course the same thing. | 19:29 |
| *** emagana has quit IRC | 19:29 | |
| *** marcusvrn has quit IRC | 19:29 | |
| DuncanT | tbarron: I see now how the config options work, I hadn't realised how nfs, swift and chunked classes were structured | 19:30 |
| *** annashen has joined #openstack-cinder | 19:30 | |
| tbarron | DuncanT: for NFS the backup container is a posix path ending in xx/yy/backup_id | 19:31 |
| tbarron | where xx is the first two chars from the backup id and yy is the 3rd and fourth. | 19:31 |
| DuncanT | duncan@z400-duncan:~/src/cinder-upstream$ grep -R backup_container cinder/backup/ | 19:31 |
| DuncanT | cinder/backup/drivers/nfs.py: cfg.StrOpt('backup_container', | 19:31 |
| DuncanT | cinder/backup/drivers/nfs.py: backup_default_container = CONF.backup_container | 19:31 |
| tbarron | This keeps the number of entries in a direcotyr managable | 19:31 |
| DuncanT | Never mind, sorry, I'm an idiot | 19:32 |
| e0ne | tbarron: thanks for update. i'll look on it right now | 19:32 |
| DuncanT | tbarron: Ok, thanks, I think I've figuring this again | 19:32 |
| tbarron | DuncanT: not at all, you had the idea for doing it this way in the first place! | 19:32 |
| tbarron | e0ne: thanks much | 19:32 |
| *** Mandell has quit IRC | 19:32 | |
| *** xyang1 has quit IRC | 19:32 | |
| DuncanT | tbarron: I'm an idiot /tonight/. | 19:32 |
| e0ne | marcusvrn: i'm sorry. did i asked you in irc? | 19:33 |
| tbarron | DuncanT: well, I'm surprised you are around this late | 19:33 |
| tbarron | and that e0ne is still working too | 19:33 |
| *** esker has joined #openstack-cinder | 19:33 | |
| erlon | e0ne: marcusvrn: it was in the review actually | 19:34 |
| DuncanT | tbarron: Failing to do electronincs mostly. Decided to stop after the second time I nearly plugged myself into the mains | 19:34 |
| *** esker has quit IRC | 19:34 | |
| e0ne | erlon: i didn't find myself in that review | 19:34 |
| tbarron | DuncanT: that will wake you up, or put you to sleep for good. | 19:34 |
| *** ronis has quit IRC | 19:35 | |
| DuncanT | tbarron: The big sparks didn't wake me up. I saw taht as a danger sign | 19:35 |
| *** esker has joined #openstack-cinder | 19:36 | |
| tbarron | DuncanT: definitely | 19:36 |
| openstackgerrit | Mike Perez proposed openstack/cinder: Unset auth token before trying to login to Datera https://review.openstack.org/163583 | 19:36 |
| erlon | e0ne: sorry Ivan, marcusvrn meat eharney | 19:36 |
| erlon | *meant | 19:37 |
| *** kmartin has quit IRC | 19:37 | |
| DuncanT | tbarron: +2. | 19:37 |
| tbarron | DuncanT: thanks! | 19:38 |
| *** rushil has joined #openstack-cinder | 19:38 | |
| vilobhmm | tbarron : can you plz send the link for your review req i will have a look | 19:39 |
| nikesh_vedams | working now,volume service was down | 19:39 |
| nikesh_vedams | :) | 19:39 |
| DuncanT | vilobhmm: https://review.openstack.org/149726 | 19:40 |
| tbarron | DuncanT: vilobhmm: ty, I was away for a moment | 19:41 |
| vilobhmm | thanks DuncanT, tbarron : np | 19:41 |
| DuncanT | Right, I'm off | 19:43 |
| *** takedakn has quit IRC | 19:45 | |
| thingee | tbarron: I think jungleboyj was the other sponsor? | 19:45 |
| *** nellysmitt has quit IRC | 19:46 | |
| thingee | tbarron: yours is the last on the list :) https://launchpad.net/cinder/+milestone/kilo-3 | 19:46 |
| *** annegentle has quit IRC | 19:46 | |
| *** annegentle has joined #openstack-cinder | 19:47 | |
| tbarron | thingee: ty, yes jungleboyj also volunteered :-) | 19:47 |
| tbarron | I owe jungleboyj and DuncanT and xyang and e0ne beers. | 19:47 |
| tbarron | or whatever | 19:47 |
| tbarron | you too thingee :-) | 19:48 |
| tbarron | in-flight wireless? | 19:48 |
| jungleboyj | tbarron: Sounds good. :-) | 19:48 |
| *** Mandell has joined #openstack-cinder | 19:49 | |
| tbarron | thingee: I'm taking a screen-shot of that last-on-the-list web page. | 19:49 |
| e0ne | tbarron: some comments inline, but i tried to explain them in the review | 19:49 |
| nikesh_vedams | one more doubt,when i am doing ./stack.sh,its creating a default volume group "stack-volumes-lvmdriver-1" is created which is lost if we reboot system | 19:50 |
| * tbarron thinks it's not something to brag about, but to remember for sure | 19:50 | |
| tbarron | e0ne: looking | 19:50 |
| tbarron | e0ne: 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 IRC | 19:52 | |
| e0ne | tbarron: yes, that's why I put +2 | 19:52 |
| tbarron | the one that's my responsibility is the \ continuation | 19:52 |
| *** Administrator__ has joined #openstack-cinder | 19:52 | |
| *** annegentle has quit IRC | 19:52 | |
| tbarron | e0ne: 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_vedams | why after reboot volume group stack-volumes-lvmdriver-1 is deleted? | 19:53 |
| *** esker has quit IRC | 19:53 | |
| tbarron | Or if you guys want to stay up longer, I can push a new patch now. | 19:53 |
| thingee | tbarron: 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-cinder | 19:54 | |
| thingee | tbarron: I was doing just reviews through gertty, and that's nice, but I send so many emails, bug triage, etc that I need access still | 19:54 |
| tbarron | e0ne: I just know that it's late for both of you. | 19:54 |
| jgriffith | nikesh_vedams: because devstack uses loopback files | 19:54 |
| jgriffith | nikesh_vedams: which are not persistent | 19:54 |
| jgriffith | nikesh_vedams: you can modify code to make them persist if you want | 19:55 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator https://review.openstack.org/157441 | 19:55 |
| jgriffith | but that's not what devstack was intended for | 19:55 |
| jungleboyj | tbarron: I got distracted. Let me look. | 19:55 |
| tbarron | thingee: good to know we can reach you no matter what :-) good for us at least. | 19:55 |
| *** timcl has joined #openstack-cinder | 19:55 | |
| tbarron | jungleboyj: np, got two +2s now ... | 19:56 |
| thingee | tbarron: yeah next thing I need to do is setup a bouncer again so I can still be pinged on irc | 19:56 |
| e0ne | tbarron: next level is to got +A ;) | 19:56 |
| nikesh_vedams | jgriffith:so then i think we can get back these loopback files after reboot | 19:56 |
| nikesh_vedams | ok | 19:56 |
| *** leeantho has quit IRC | 19:56 | |
| nikesh_vedams | sure thanks | 19:56 |
| *** leeantho has joined #openstack-cinder | 19:57 | |
| tbarron | e0ne: jungleboyj: let me know if you want another changeset in this review with the changes e0ne proposed. | 19:57 |
| jungleboyj | tbarron: Will do. | 19:57 |
| *** leeantho has quit IRC | 19:57 | |
| e0ne | jungleboyj, tbarron: i think my comments should be addressed in a separate patch | 19:57 |
| tbarron | jungleboyj: all except one issue, continuation by '\' rather than parens, are in the original Swift code. | 19:57 |
| tbarron | e0ne: and I'm happy to do that patch. | 19:58 |
| tbarron | e0ne: and besides being lazy atm, I actually believe that this refactor change set should avoid gratuitous changes to the original code. | 19:58 |
| tbarron | that makes the fact that it's "just a refactor" clearer. | 19:59 |
| *** timcl has quit IRC | 20:00 | |
| nikesh_vedams | jgriffith:yu said "but that's not what devstack was intended for" ,what that means | 20:00 |
| *** Longgeek has quit IRC | 20:00 | |
| *** dflorea has joined #openstack-cinder | 20:01 | |
| jgriffith | nikesh_vedams: that means, "it's not intended to be a persistent environment" | 20:02 |
| *** dflorea has quit IRC | 20:02 | |
| nikesh_vedams | asking so that if i some one ask me then i can explain to them, | 20:02 |
| jgriffith | nikesh_vedams: FYI, the other trick is don't use loopbacks | 20:02 |
| nikesh_vedams | ok | 20:02 |
| jgriffith | nikesh_vedams: create an actual VG on physical disks and give it the cinder-volume name | 20:02 |
| jgriffith | then devstack just leaves it alone | 20:02 |
| jungleboyj | FYI, +2 from me on the oslo.log patch. Should get one other person to look at it though: https://review.openstack.org/#/c/157441/ | 20:03 |
| nikesh_vedams | yes that i know but i didnot know that loopback,thanks for clearance | 20:04 |
| *** vilobhmm has quit IRC | 20:04 | |
| *** dflorea_ has joined #openstack-cinder | 20:05 | |
| *** vilobhmm1 has joined #openstack-cinder | 20:05 | |
| nikesh_vedams | jgriffith: thanks i might join "Canada Summit" :) | 20:06 |
| jgriffith | nikesh_vedams: awesome! | 20:06 |
| *** diemt has quit IRC | 20:06 | |
| jgriffith | nikesh_vedams: be good to meet you in person | 20:06 |
| nikesh_vedams | :) | 20:07 |
| *** thangp has quit IRC | 20:08 | |
| *** annegentle has joined #openstack-cinder | 20:08 | |
| jungleboyj | tbarron: You patch looks good to me. | 20:09 |
| jungleboyj | e0ne: If you are ok having him address your additional comments separately I will +A it. | 20:09 |
| tbarron | jungleboyj: ty. I appreciate all your help. | 20:09 |
| *** esker has joined #openstack-cinder | 20:09 | |
| e0ne | jungleboyj: sure. i'm not +1/+2 if i don't want it in cinder:) | 20:10 |
| jungleboyj | e0ne: Though that that may break the oslo.log patch. Maybe it won't. | 20:11 |
| jungleboyj | e0ne: That is ok though, you go to bed and I will deal with the oslo.log patch if anything goes wrong. | 20:11 |
| e0ne | jungleboyj: thank you. i'll be available online about 2 hours more | 20:13 |
| jungleboyj | tbarron: Merry Christmas! | 20:13 |
| *** fischerw has quit IRC | 20:13 | |
| jungleboyj | e0ne: You work too much. ;-0 | 20:13 |
| jungleboyj | ;-) | 20:13 |
| *** fischerw has joined #openstack-cinder | 20:14 | |
| tbarron | jungleboyj: let's see what the bots think of it now. | 20:16 |
| mtanino | jungleboyj: Hi, Could you visit this patch again for stable/juno https://review.openstack.org/#/c/161328/. | 20:17 |
| mtanino | after your current task will be finished :) | 20:18 |
| anish | smcginnis: thanks for catching the error in my comment | 20:25 |
| *** xyang1 has joined #openstack-cinder | 20:25 | |
| jgriffith | e0ne: you still online? | 20:26 |
| smcginnis | anish: np - just luck. :) | 20:27 |
| e0ne | jgriffith: yes | 20:27 |
| jgriffith | e0ne: I finally got around to setting up my fuel deployment again | 20:27 |
| e0ne | jgriffith: :) | 20:28 |
| jgriffith | e0ne: do you by chance know default login/pass to the fuel nodes themeselves? | 20:28 |
| e0ne | admin/admin for web ui | 20:28 |
| e0ne | root/r00tme for ssh | 20:28 |
| jgriffith | e0ne: :) | 20:28 |
| jgriffith | that's it... thanks! | 20:28 |
| jgriffith | root/r00tme | 20:28 |
| *** xyang1 has quit IRC | 20:29 | |
| *** kmartin has joined #openstack-cinder | 20:29 | |
| *** xyang has joined #openstack-cinder | 20:29 | |
| e0ne | jgriffith: you're welcome. we've got in the docs, so i'm comfortable to write it here:) | 20:30 |
| jgriffith | e0ne: hehe.. yeah, I just noticed it there ;) | 20:30 |
| e0ne | jgriffith: ssh from master to slaves should work by key | 20:31 |
| mtanino | smcginnis: anish :) | 20:31 |
| *** delattec has quit IRC | 20:31 | |
| *** cdelatte has quit IRC | 20:31 | |
| anish | mtanino: I'm glad your jenkins issues are finally over. that was amusing to watch | 20:32 |
| mtanino | anish: oh, it was solved. nice | 20:33 |
| jgriffith | e0ne: well that was EZ :) | 20:33 |
| jgriffith | e0ne: up and running, configured my driver, running tests :) | 20:33 |
| e0ne | jgriffith: ez==eazy? | 20:33 |
| jgriffith | e0ne: correct :) | 20:34 |
| *** jistr has joined #openstack-cinder | 20:35 | |
| anish | EZ™ | 20:35 |
| jgriffith | anish: there ya go with the TM again :) | 20:36 |
| jgriffith | anish: show off! | 20:36 |
| *** ndipanov has quit IRC | 20:36 | |
| anish | just a good keyboard layout :) | 20:37 |
| *** scottda_ has joined #openstack-cinder | 20:39 | |
| *** jwang_ has quit IRC | 20:40 | |
| *** leeantho has joined #openstack-cinder | 20:40 | |
| *** scottda_ has quit IRC | 20:41 | |
| *** cdelatte has joined #openstack-cinder | 20:42 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder-specs: Volume multi attachments https://review.openstack.org/154135 | 20:43 |
| *** takedakn has joined #openstack-cinder | 20:43 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix HNAS iSCSI driver error on LUN creation https://review.openstack.org/163606 | 20:48 |
| *** akerr has quit IRC | 20:52 | |
| *** jwang_ has joined #openstack-cinder | 20:55 | |
| *** mtanino has quit IRC | 20:56 | |
| *** dustins has quit IRC | 20:57 | |
| *** esker has quit IRC | 21:02 | |
| *** bswartz has quit IRC | 21:03 | |
| *** Yogi1 has quit IRC | 21:04 | |
| *** r-daneel has quit IRC | 21:05 | |
| *** Yogi1 has joined #openstack-cinder | 21:05 | |
| *** tbarron has quit IRC | 21:05 | |
| *** jcru has quit IRC | 21:06 | |
| *** jcru has joined #openstack-cinder | 21:07 | |
| *** jcru has quit IRC | 21:08 | |
| *** tellesnobrega_ has joined #openstack-cinder | 21:09 | |
| *** eharney has quit IRC | 21:10 | |
| *** jistr has quit IRC | 21:10 | |
| *** thingee has quit IRC | 21:10 | |
| *** tellesnobrega_ has quit IRC | 21:11 | |
| *** vilobhmm2 has joined #openstack-cinder | 21:12 | |
| *** ozamiatin has quit IRC | 21:13 | |
| *** aarefiev has quit IRC | 21:13 | |
| *** ozamiatin has joined #openstack-cinder | 21:14 | |
| *** vilobhmm1 has quit IRC | 21:14 | |
| *** aarefiev has joined #openstack-cinder | 21:14 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator https://review.openstack.org/157441 | 21:14 |
| jungleboyj | e0ne: Oh no! | 21:16 |
| e0ne | jungleboyj: :( | 21:17 |
| *** Yogi1 has quit IRC | 21:17 | |
| e0ne | jungleboyj: i forget to update requirements | 21:18 |
| jungleboyj | e0ne: Go to bed. I will keep an eye on it. | 21:19 |
| e0ne | jungleboyj: that's i'm going to do right now | 21:20 |
| jungleboyj | e0ne: Ok. | 21:20 |
| jungleboyj | e0ne: Have a good night. | 21:20 |
| e0ne | jungleboyj: thanks! | 21:21 |
| *** esker has joined #openstack-cinder | 21:21 | |
| *** fischerw has quit IRC | 21:28 | |
| *** fischerw has joined #openstack-cinder | 21:28 | |
| jgriffith | Anybody know if Eric noticed the test failures on that patch? | 21:29 |
| *** annashen has quit IRC | 21:32 | |
| *** boris-42 has quit IRC | 21:32 | |
| smcginnis | jgriffith: I hit recheck. | 21:33 |
| jgriffith | smcginnis: well, it's a valid failure I think | 21:33 |
| smcginnis | Any idea why we're getting a lot of failures: Table 'encryption' specified column 'volume_type_id'... | 21:33 |
| smcginnis | jgriffith: This error keeps popping up, then going away on recheck. | 21:34 |
| jgriffith | smcginnis: oh.. hmm | 21:34 |
| smcginnis | jgriffith: So I don't think it's valid, unless there actually is a real failure buried in that mess. | 21:34 |
| jgriffith | smcginnis: I thought that some of those were in fact what this patch was supposed to fix | 21:34 |
| smcginnis | jgriffith: But I've seen it too often now to blame it on Eric. (yet) | 21:34 |
| jgriffith | hehe | 21:34 |
| jgriffith | smcginnis: so you're seeing that unit test fail? | 21:35 |
| jgriffith | smcginnis: http://goo.gl/E340cg | 21:35 |
| smcginnis | jgriffith: I think it failed one of my patches last night. Recheck passed. | 21:35 |
| smcginnis | jgriffith: Seen at least several others as well. | 21:35 |
| jgriffith | smcginnis: that's very bad | 21:35 |
| smcginnis | jgriffith: Yeah, just started looking at the recheck page to see if there was anything. | 21:36 |
| *** scottda_ has joined #openstack-cinder | 21:36 | |
| smcginnis | jgriffith: Doesn't look like it's been identified though, otherwise elastic recheck would comment, right? | 21:36 |
| jgriffith | smcginnis: should yeah | 21:37 |
| jgriffith | smcginnis: also the fact that it's in unit tests is distrubing to say the least | 21:37 |
| smcginnis | http://logs.openstack.org/51/163551/2/check/gate-cinder-python27/ab6e955/console.html#_2015-03-11_20_16_59_220 | 21:37 |
| smcginnis | That's what I've seen a bunch of. | 21:37 |
| smcginnis | jgriffith: It's like the database is getting messed with occasionally. | 21:37 |
| jgriffith | Yeah, same thing... very bad | 21:37 |
| *** rushil_ has joined #openstack-cinder | 21:38 | |
| smcginnis | Hmmm... 033_add_encryption_unique_key.py | 21:39 |
| *** annashen has joined #openstack-cinder | 21:39 | |
| *** movielovers has joined #openstack-cinder | 21:39 | |
| *** rushil has quit IRC | 21:39 | |
| *** dflorea_ has quit IRC | 21:40 | |
| jgriffith | https://bugs.launchpad.net/oslo.db/+bug/1417018 | 21:40 |
| openstack | Launchpad 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 |
| jgriffith | https://bugs.launchpad.net/cinder/+bug/1430859 | 21:40 |
| openstack | Launchpad 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-cinder | 21:40 | |
| smcginnis | jgriffith: Those look similar, but a little different. This one looks like conflicting primary key definitions. | 21:40 |
| jgriffith | Given that's what Eric's patch was referenced for on the ML looks like it doesn't work | 21:41 |
| jgriffith | smcginnis: I'm betting same root cause here | 21:41 |
| *** Mandell has quit IRC | 21:41 | |
| jgriffith | smcginnis: the oslo.db fix is the key there | 21:41 |
| smcginnis | jgriffith: So we need to wait for oslo? | 21:41 |
| dhellmann | jgriffith: is zzzeek's patch not working? | 21:42 |
| jgriffith | smcginnis: not necessarily... check comment 3 and 4 in bug 1417018 | 21:42 |
| openstack | 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] https://launchpad.net/bugs/1417018 - Assigned to Eric Harney (eharney) | 21:42 |
| *** scottda_ has quit IRC | 21:42 | |
| jgriffith | dhellmann: not sure... which that one is | 21:42 |
| jgriffith | dhellmann: I don't think we've pulled that in yet | 21:42 |
| *** zzzeek has joined #openstack-cinder | 21:43 | |
| jgriffith | dhellmann: OH... Mike :) | 21:43 |
| dhellmann | jgriffith: yeah, sorry, I thought he was in the room | 21:43 |
| jgriffith | dhellmann: so no, the fix Eric submitted just failed for the same error | 21:43 |
| *** jecarey has joined #openstack-cinder | 21:43 | |
| dhellmann | :-| | 21:43 |
| jgriffith | dhellmann: https://review.openstack.org/#/c/163551/ | 21:43 |
| smcginnis | I'm still not convinced. It looks a little different to me. | 21:44 |
| *** xyang has quit IRC | 21:44 | |
| dhellmann | jgriffith: oh, sorry, zzzeek gave me that link a while back and I thought it was his patch, so that's the one I meant | 21:44 |
| *** dflorea has quit IRC | 21:45 | |
| dhellmann | jgriffith: we're going to move ahead with https://review.openstack.org/#/c/156725/ in oslo.db, too | 21:45 |
| jgriffith | ahh | 21:45 |
| jgriffith | cool | 21:45 |
| zzzeek | OK 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 |
| dhellmann | we probably want some form of both, though - belt and braces | 21:46 |
| zzzeek | because that dispose() will blow away the “memory” database | 21:46 |
| jgriffith | zzzeek: yes | 21:46 |
| zzzeek | jgriffith: OK so that’s the issue here | 21:46 |
| jgriffith | zzzeek: unit tests all run sqlite | 21:46 |
| jgriffith | zzzeek: interesting | 21:46 |
| jgriffith | zzzeek: that complicates things slightly | 21:46 |
| zzzeek | jgriffith: right, so, sqlite memory DB has no on-disk state, you call engine.dispose() and its gone | 21:46 |
| zzzeek | jgriffith: yes :) | 21:46 |
| jgriffith | zzzeek: makes sense | 21:46 |
| zzzeek | jgriffith: which is why, calling dispose() *only* in a forked child, would be best | 21:46 |
| jgriffith | zzzeek: yeah, was just wondering about moving that down in to the actual service | 21:47 |
| jgriffith | zzzeek: not that I'm clear on exactly how that would look | 21:47 |
| zzzeek | jgriffith: unless these tests are actually doing that, which i woud think a sqlite :memory: wouldnt travel there very well | 21:47 |
| jgriffith | sigh | 21:47 |
| zzzeek | jgriffith: there’s no forking in these tests, right? | 21:47 |
| jgriffith | zzzeek: in the unit tests no, there should not be | 21:48 |
| zzzeek | jgriffith: guaranteed way to see if you just started a child is to memoize os.pid() and os.ppid() and compare | 21:48 |
| smcginnis | This one actually does seem to be related to sqllite: http://goo.gl/pJnPxU | 21:49 |
| zzzeek | jgriffith: 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 fork | 21:49 |
| smcginnis | IN 033_add_encryption_unique_key it tries to drop a primary key of volume_type_id | 21:49 |
| smcginnis | Then 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 | |
| jgriffith | zzzeek: yeah, that whole thing has been debated in the past | 21:50 |
| zzzeek | imagine how many problems we woudlnt have if we just made people run a fricking mysql db | 21:50 |
| smcginnis | But then in the exception handling of dropping the previous PK is this: | 21:50 |
| smcginnis | # NOTE (e0ne): SQLite doesn't support 'drop constraint' statament | 21:50 |
| smcginnis | if migrate_engine.url.get_dialect().name.startswith('sqlite'): | 21:50 |
| smcginnis | pass | 21:50 |
| jgriffith | smcginnis: indeed | 21:50 |
| smcginnis | So on upgrade if using sqllite you're basically screwed it appears. | 21:51 |
| *** Mandell has joined #openstack-cinder | 21:51 | |
| zzzeek | jgriffith: here’s an easier, more hacky appraoch: “if engine.driver != ‘sqlite’: engine.dispose()” | 21:51 |
| smcginnis | You can't get rid of the old PK to add the new PK, causing the error in Eric's patch test run. | 21:51 |
| zzzeek | err e.name | 21:52 |
| zzzeek | engine.name | 21:52 |
| smcginnis | At least that one instance that's I've seen elsewhere. | 21:52 |
| smcginnis | zzzeek: +1 seems simple enough for now at least. | 21:52 |
| zzzeek | smcginnis: its a nasty hack | 21:52 |
| smcginnis | zzzeek: Agreed | 21:53 |
| zzzeek | smcginnis: its just to get you through the day and see if htis works otherwise | 21:53 |
| smcginnis | zzzeek: But gets around it until a better solution is found... | 21:53 |
| smcginnis | :) | 21:53 |
| zzzeek | smcginnis: make sure its noted, “remove this when XYZ is fixed” | 21:53 |
| e0ne | is something wrong with migration #33> | 21:53 |
| e0ne | ? | 21:53 |
| smcginnis | e0ne: Oh, just saw a bunch of failures recently from Jenkins. | 21:54 |
| smcginnis | Related to that one, but usually a recheck passes. | 21:54 |
| smcginnis | It was pointed out gate uses sqlite. Saw your note that primary key constraint can't be dropped. | 21:54 |
| e0ne | is was made few month ago | 21:54 |
| smcginnis | That appears to be causing the issue, though not sure why it's intermittent then. | 21:54 |
| smcginnis | e0ne: Seems fine for the most part. | 21:55 |
| *** mriedem is now known as mriedem_away | 21:55 | |
| *** esker has quit IRC | 21:55 | |
| smcginnis | e0ne: 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 |
| smcginnis | Uggggg | 21:55 |
| e0ne | upgrade or downgrade? | 21:56 |
| smcginnis | upgrade | 21:56 |
| smcginnis | Gate uses sqlite. Sqlite doesn't allow primary key to change. :[ | 21:56 |
| smcginnis | I 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 |
| e0ne | why did it work before? | 21:57 |
| smcginnis | I think the work around zzzeek proposed will probably have to do. | 21:57 |
| smcginnis | e0ne: And it doesn't always cause problems, so that's what I'm scratching my head on. | 21:58 |
| e0ne | hm... | 21:58 |
| *** arif-ali has quit IRC | 21:58 | |
| smcginnis | e0ne: Not even sure how you could do the whole migration to a new table and rename with sqlalchemy. | 22:00 |
| hemna | oops what happened? | 22:00 |
| e0ne | smcginnis: if it works sometimes on gates - it means that different envs on infra | 22:00 |
| smcginnis | hemna: You broke it. | 22:00 |
| * hemna runs home | 22:00 | |
| smcginnis | hemna: JK :) | 22:00 |
| e0ne | :) | 22:00 |
| hemna | so that looks like it's not a migration failure | 22:00 |
| e0ne | smcginnis: it it reproducible on Eric's patch? | 22:00 |
| smcginnis | Anyone know if they use a mix of backends? That could explain why it works sometimes and not others. | 22:00 |
| hemna | no such table ? | 22:01 |
| hemna | wth | 22:01 |
| smcginnis | Sometimes gets mysql, sometimes gets sqlite. | 22:01 |
| e0ne | smcginnis: it's crazy | 22:01 |
| smcginnis | e0ne: It was hit once with Eric's patch, but it does look like there are other bigger issues. | 22:01 |
| zzzeek | smcginnis: 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 thing | 22:01 |
| e0ne | smcginnis: what is the "other bigger issue"? | 22:02 |
| smcginnis | zzzeek: Oh. So 033 could be updated - if it's sqlite do a migrate, otherwise do a drop constraint? | 22:02 |
| *** lpetrut has quit IRC | 22:02 | |
| smcginnis | e0ne: If I followed right, engine_dispose() with sqlite in memory DB will actually blow away the DB. | 22:02 |
| zzzeek | smcginnis: not sure, not looking :) | 22:02 |
| smcginnis | e0ne: So Eric's approach loses the whole DB in that case. | 22:03 |
| smcginnis | zzzeek: ;) | 22:03 |
| *** _cjones_ has quit IRC | 22:03 | |
| hemna | goosh, yah it looks like every table is gone | 22:03 |
| e0ne | smcginnis: ok.. why are you talking about migration error? | 22:03 |
| hemna | ship it! | 22:03 |
| *** akerr has joined #openstack-cinder | 22:03 | |
| *** _cjones_ has joined #openstack-cinder | 22:03 | |
| e0ne | i can't believe that code works well for a 3 month and it is broken now | 22:04 |
| hemna | so, maybe it's not able to reconnect to the db instance ? | 22:04 |
| smcginnis | e0ne: One out of the many errors was this one: http://goo.gl/pJnPxU | 22:04 |
| hemna | are other patches failing in the same way ? | 22:04 |
| *** akerr_ has joined #openstack-cinder | 22:04 | |
| smcginnis | e0ne: I've seen that in a few patches (including my own) where a recheck passes. | 22:04 |
| Swanson | gate-cinder-python27 is running on that patch right now. So be interesting to see if it is consistent fail or not. | 22:04 |
| smcginnis | Swanson: +1 | 22:05 |
| smcginnis | e0ne: So sometimes 033 works fine. Other times it fails. | 22:05 |
| smcginnis | e0ne: Related to Eric's patch, it was pointed out gate uses sqlite. | 22:05 |
| e0ne | smcginnis: it's a warning | 22:05 |
| smcginnis | e0ne: Then I saw your note. | 22:05 |
| smcginnis | e0ne: Apparently it treats that warning as a failure. | 22:05 |
| *** annegentle has quit IRC | 22:05 | |
| hemna | where are the migration logs associated with this ? | 22:06 |
| smcginnis | e0ne: End the attempt to set encryption_id as the primary key fails because volume_type_id is still the primary key. | 22:06 |
| smcginnis | e0ne: Only for sqlite cases. | 22:06 |
| e0ne | if tests works time to time it means we've got different version of software on different Infra's hosts | 22:06 |
| smcginnis | e0ne: So wondering if instead of passing on that exception we need to do the data migration to a new table. | 22:07 |
| smcginnis | e0ne: That's what I was wondering. | 22:07 |
| jgriffith | zzzeek: yeah, that's what I was wondering if we could/shoudl do :) | 22:07 |
| smcginnis | e0ne: Maybe infra uses a mix of DBs? Only thing I can think of that would explain it. | 22:07 |
| jgriffith | zzzeek: even if just as a temporary hack | 22:07 |
| e0ne | i can't believe in it | 22:07 |
| jgriffith | zzzeek: think I may mod eharney's patch and see what folks think | 22:08 |
| jgriffith | although I'd rather wait and chat with him | 22:08 |
| *** akerr has quit IRC | 22:08 | |
| jgriffith | zzzeek: actually... since you mentioned it first, do you want to suggest it in the comments of his current review set? | 22:08 |
| smcginnis | jgriffith: Since he's not around, might be a good test to mod it. He can always back it out, right? | 22:08 |
| jgriffith | smcginnis: correct | 22:09 |
| *** jungleboyj has quit IRC | 22:09 | |
| smcginnis | jgriffith: Or that... | 22:09 |
| *** arif-ali has joined #openstack-cinder | 22:10 | |
| *** ho__ has joined #openstack-cinder | 22:12 | |
| Swanson | It choked again. | 22:12 |
| jgriffith | hehe... pysqlite | 22:13 |
| jgriffith | ok, I'll just mod the test class and submit it | 22:13 |
| jgriffith | pet peave alert: No handlers could be found for logger "cinder.volume.drivers.cloudbyte.cloudbyte" | 22:14 |
| *** arif-ali has quit IRC | 22:15 | |
| *** arif-ali has joined #openstack-cinder | 22:18 | |
| jgriffith | zzzeek: hmm.. that doesn't work out well | 22:19 |
| jgriffith | smcginnis: so the *good* thing is those errors are 100% reproducible | 22:22 |
| e0ne | smcginnis, jgriffith: i've run test_migrations tests for sqlite on Eric's patch | 22:23 |
| e0ne | its' workin on my en | 22:23 |
| e0ne | s/en/env | 22:23 |
| e0ne | 10 times from 10 | 22:23 |
| jgriffith | e0ne: try runnign the fulll test suite | 22:24 |
| e0ne | and cinder.tests.test_volume_transfer.VolumeTransferTestCase.test_transfer_volume_create_delete fails everytime | 22:24 |
| jgriffith | e0ne: not just migrations | 22:24 |
| e0ne | and yes, i've got sqlalchemy warning on both tests | 22:24 |
| *** annegentle has joined #openstack-cinder | 22:25 | |
| jgriffith | e0ne: smcginnis trying to figure out the relation in test.py | 22:26 |
| jgriffith | the Database fixture that is | 22:26 |
| e0ne | it invokes dispose several times:( | 22:26 |
| jgriffith | e0ne: right... every time | 22:26 |
| e0ne | smcginnis, jgriffith: i can't understand why are you talking that migration fails | 22:26 |
| jgriffith | e0ne: don't think we were | 22:27 |
| e0ne | smcginnis, jgriffith: i can't work or not work for different test | 22:27 |
| jgriffith | e0ne: we were talking about: https://review.openstack.org/#/c/163551/2 | 22:27 |
| e0ne | jgriffith: ok, so we are on the same page | 22:27 |
| jgriffith | e0ne: sorry, I'm not following? | 22:28 |
| e0ne | jgriffith: i mean that you agree that migration works | 22:28 |
| e0ne | do you? | 22:28 |
| jgriffith | e0ne: I dunno... maybe :) | 22:29 |
| e0ne | :) | 22:29 |
| *** annashen has quit IRC | 22:30 | |
| jgriffith | e0ne: there are a few hundred test failures, didn't loook at exactly what passes :) | 22:30 |
| jgriffith | would image migrations are "ok" | 22:30 |
| jgriffith | but I dunno | 22:30 |
| *** annashen has joined #openstack-cinder | 22:30 | |
| *** jaypipes-afk is now known as jaypipes | 22:31 | |
| *** annegentle has quit IRC | 22:34 | |
| jgriffith | e0ne: so I can fix it by hacking the sqlite check in the db/api.py layer | 22:39 |
| jgriffith | e0ne: but I'm not sure that's such a "good" thing or not | 22:39 |
| *** tellesnobrega_ has joined #openstack-cinder | 22:40 | |
| *** tellesnobrega_ has quit IRC | 22:41 | |
| e0ne | why when we call engine.dispose() and its gone for in memory sqlite db? | 22:41 |
| jgriffith | e0ne: if I follow correctly because you basicly just deleted the db | 22:41 |
| e0ne | erm... | 22:42 |
| *** lcurtis has quit IRC | 22:42 | |
| jgriffith | e0ne: smcginnis here's what I did: http://goo.gl/Blveri | 22:43 |
| jgriffith | Ohh... hmm | 22:44 |
| e0ne | jgriffith: zzzeek said so. but why? i didn't find anything reladed in sqlalchemy docd | 22:44 |
| *** Longgeek has joined #openstack-cinder | 22:45 | |
| jgriffith | e0ne: not sure, but the patch change I put the gist up for certainly fixes it | 22:47 |
| *** jecarey has quit IRC | 22:51 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Fix sqlalchemy reuse in multi-backend children https://review.openstack.org/163551 | 22:52 |
| *** annashen has quit IRC | 22:52 | |
| *** Adriano_ has quit IRC | 22:52 | |
| e0ne | jgriffith: i agree that yours patch with small changes would work and looks acceptable | 22:54 |
| e0ne | jgriffith: i just want to understand root cause of the errors | 22:54 |
| jgriffith | e0ne: yeah, I'm kinda nervous about how hacky that is | 22:54 |
| jgriffith | e0ne: and yes, I agree with you completely | 22:54 |
| jgriffith | e0ne: as a result i'm certainly ok if that doesn't merge | 22:54 |
| jgriffith | e0ne: no problem waiting | 22:54 |
| *** Longgeek has quit IRC | 22:55 | |
| jgriffith | e0ne: maybe see what eharney thinks when he gets back online | 22:55 |
| e0ne | jgriffith: good idea | 22:55 |
| jgriffith | e0ne: also I think there's something different we should be doing | 22:55 |
| jgriffith | e0ne: ie in the way we launch multi-backend | 22:55 |
| e0ne | TBH, i don't like out db/base.py code | 22:56 |
| *** movielovers has left #openstack-cinder | 22:57 | |
| *** angela-s has quit IRC | 22:58 | |
| jgriffith | e0ne: I was just noticing that as well | 22:58 |
| jgriffith | e0ne: also take a look at the stuff we're doing in our test fixture: test.py | 22:58 |
| jgriffith | e0ne: we probably need to get some love on those things | 22:58 |
| e0ne | jgriffith: +2 from me. i'm agree about tests refactoring | 22:59 |
| e0ne | not only db-related tests | 22:59 |
| jgriffith | e0ne: yea | 22:59 |
| e0ne | jgriffith: test_huawei_t_dorado.py - i trief to refactor it to fix concurrency. but it must be re-writed almost all:( | 23:00 |
| jgriffith | sad | 23:01 |
| e0ne | i'm sorry. i need to go sleep | 23:02 |
| e0ne | be back in ~10 hours | 23:02 |
| jgriffith | e0ne: hve you noticed all of these too: http://goo.gl/quelUH | 23:02 |
| jgriffith | e0ne: oh... ok... go sleep!!! I'll see you tomorrow | 23:03 |
| *** dannywil_ has joined #openstack-cinder | 23:03 | |
| *** dannywil_ has quit IRC | 23:03 | |
| e0ne | jgriffith: yes, but it's an warning in my sqlalchemy.migrate version | 23:04 |
| * e0ne away | 23:04 | |
| *** e0ne is now known as e0ne_ | 23:04 | |
| *** patrickeast_ has joined #openstack-cinder | 23:05 | |
| *** dannywilson has quit IRC | 23:06 | |
| *** patrickeast has quit IRC | 23:06 | |
| *** patrickeast_ is now known as patrickeast | 23:06 | |
| *** ganso_ has quit IRC | 23:07 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: WIP - ConsistencyGroup Object https://review.openstack.org/163641 | 23:10 |
| *** thingee has joined #openstack-cinder | 23:10 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: WIP - ConsistencyGroup Object https://review.openstack.org/163641 | 23:12 |
| *** annashen has joined #openstack-cinder | 23:12 | |
| *** scottda_ has joined #openstack-cinder | 23:15 | |
| *** scottda_ has quit IRC | 23:16 | |
| *** asselin_ has joined #openstack-cinder | 23:18 | |
| *** asselin_ has quit IRC | 23:23 | |
| *** scottda_ has joined #openstack-cinder | 23:24 | |
| *** annashen has quit IRC | 23:24 | |
| *** akerr_ has quit IRC | 23:27 | |
| *** Apoorva has quit IRC | 23:28 | |
| *** jecarey has joined #openstack-cinder | 23:28 | |
| *** Mandell has quit IRC | 23:29 | |
| *** scottda_ has quit IRC | 23:29 | |
| *** annegentle has joined #openstack-cinder | 23:32 | |
| *** bswartz has joined #openstack-cinder | 23:37 | |
| *** scottda_ has joined #openstack-cinder | 23:38 | |
| *** HoloIRCUser1 has joined #openstack-cinder | 23:39 | |
| *** IanGovett has quit IRC | 23:39 | |
| *** chlong has joined #openstack-cinder | 23:42 | |
| *** annegentle has quit IRC | 23:43 | |
| *** sparr has quit IRC | 23:45 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator https://review.openstack.org/157441 | 23:45 |
| *** sparr has joined #openstack-cinder | 23:48 | |
| *** e0ne_ has quit IRC | 23:50 | |
| *** Apoorva has joined #openstack-cinder | 23:51 | |
| *** dflorea has joined #openstack-cinder | 23:51 | |
| *** mtanino has joined #openstack-cinder | 23:52 | |
| *** dflorea has quit IRC | 23:54 | |
| *** dflorea has joined #openstack-cinder | 23:54 | |
| *** scottda_ has quit IRC | 23:58 | |
| *** patrickeast has quit IRC | 23:58 | |
| *** scottda_ has joined #openstack-cinder | 23:59 | |
| *** dflorea has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!