| *** mtanino has quit IRC | 00:00 | |
| asselin | patrickeast, yes...discussion on that going on in -infra | 00:01 |
|---|---|---|
| *** markvoelker has joined #openstack-cinder | 00:01 | |
| *** tsekiyam_ has quit IRC | 00:01 | |
| *** Apoorva has quit IRC | 00:06 | |
| *** akerr_ has quit IRC | 00:06 | |
| *** Apoorva has joined #openstack-cinder | 00:06 | |
| *** ebalduf has joined #openstack-cinder | 00:07 | |
| *** rongze has joined #openstack-cinder | 00:10 | |
| *** ebalduf has quit IRC | 00:12 | |
| *** akerr has joined #openstack-cinder | 00:13 | |
| *** rongze has quit IRC | 00:15 | |
| *** akerr has quit IRC | 00:27 | |
| *** heyun has joined #openstack-cinder | 00:36 | |
| *** _cjones_ has quit IRC | 00:38 | |
| *** _cjones_ has joined #openstack-cinder | 00:39 | |
| *** Apoorva has quit IRC | 00:39 | |
| *** vilobhmm1 has quit IRC | 00:41 | |
| *** annashen has joined #openstack-cinder | 00:45 | |
| *** leeantho has quit IRC | 00:49 | |
| *** mtanino has joined #openstack-cinder | 00:58 | |
| *** annashen has quit IRC | 01:01 | |
| *** annashen has joined #openstack-cinder | 01:06 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Remove force check from copy_volume_to_image https://review.openstack.org/176174 | 01:06 |
| *** annashen has quit IRC | 01:06 | |
| jgriffith | zigo: sorry, I was away: http://54.164.167.86/solidfire-ci-logs/cinder-conf-samples/ | 01:08 |
| jgriffith | zigo: I generate one nightly and publish it to that web-site | 01:08 |
| *** rongze has joined #openstack-cinder | 01:11 | |
| *** _cjones_ has quit IRC | 01:15 | |
| *** rongze has quit IRC | 01:15 | |
| *** akerr has joined #openstack-cinder | 01:24 | |
| *** akerr_ has joined #openstack-cinder | 01:26 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add resource tag to logging in volume.manager.py https://review.openstack.org/175077 | 01:27 |
| jgriffith | hemnafk: tbarron ^^ I hope that satisfies the two of you. Not worth arguing, but I would appreciate you at least read the comments and look at the logging guidelines. | 01:28 |
| *** Lee1092 has joined #openstack-cinder | 01:28 | |
| *** akerr has quit IRC | 01:29 | |
| jgriffith | hemnafk: tbarron particularly comment from me at 19:17, and the fact that we duplicate a single Info log message 376 times in a little more than a half hour period | 01:29 |
| jgriffith | sad | 01:29 |
| *** kaisers has quit IRC | 01:31 | |
| *** kaisers has joined #openstack-cinder | 01:32 | |
| *** markvoelker has quit IRC | 01:33 | |
| *** xyang has quit IRC | 01:37 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Switch get_all_snapshots to use objects https://review.openstack.org/161077 | 01:41 |
| *** haigang has joined #openstack-cinder | 01:41 | |
| *** IanGovett has quit IRC | 01:44 | |
| *** Longgeek has joined #openstack-cinder | 01:46 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Complete switch to snapshot objects https://review.openstack.org/163910 | 01:49 |
| openstackgerrit | Dave Chen proposed openstack/python-cinderclient: Add commands for modifying image metadata https://review.openstack.org/147077 | 01:51 |
| *** Longgeek has quit IRC | 01:56 | |
| *** Longgeek has joined #openstack-cinder | 01:56 | |
| *** davechen1 has joined #openstack-cinder | 01:57 | |
| *** david-lyle has quit IRC | 01:59 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Switch to oslo_versionedobjects https://review.openstack.org/160209 | 02:02 |
| *** annashen has joined #openstack-cinder | 02:07 | |
| *** ebalduf has joined #openstack-cinder | 02:09 | |
| *** nikesh has quit IRC | 02:11 | |
| *** annashen has quit IRC | 02:12 | |
| *** haigang has quit IRC | 02:12 | |
| *** harlowja is now known as harlowja_away | 02:12 | |
| *** annegentle has joined #openstack-cinder | 02:14 | |
| *** ebalduf has quit IRC | 02:14 | |
| *** _cjones_ has joined #openstack-cinder | 02:15 | |
| *** patrickeast has quit IRC | 02:18 | |
| *** adurbin_ has quit IRC | 02:19 | |
| *** annegentle has quit IRC | 02:19 | |
| *** markvoelker has joined #openstack-cinder | 02:22 | |
| *** lcurtis has quit IRC | 02:23 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Switch to oslo_versionedobjects https://review.openstack.org/160209 | 02:36 |
| *** haigang has joined #openstack-cinder | 02:39 | |
| *** dannywilson has joined #openstack-cinder | 02:40 | |
| *** dannywilson has quit IRC | 02:44 | |
| tbarron | jgriffith: I think the overall principle is good, but not quite precise enough to fit this particular init-time-only, very-useful log msg. | 02:45 |
| tbarron | jgriffith: thanks for your flexibility on this. You have my +1 on this now fwiw. | 02:46 |
| tbarron | We've got to find some way to get the log messages coralled properly without diverting so much energy/resources on this. | 02:47 |
| tbarron | I want to move on the sp3llieng 3rr0rs | 02:47 |
| tbarron | good night | 02:47 |
| thingee | jgriffith: yeah, I might have to add a condition to not do discovery if the endpoint contains a version already | 02:57 |
| thingee | so it doesn't give those warnings | 02:57 |
| *** rongze has joined #openstack-cinder | 03:02 | |
| *** akerr_ has quit IRC | 03:03 | |
| *** _cjones_ has quit IRC | 03:04 | |
| openstackgerrit | Jeegn Chen proposed openstack/cinder: Remove redundant code from VNX Cinder Driver https://review.openstack.org/173183 | 03:07 |
| *** vilobhmm1 has joined #openstack-cinder | 03:10 | |
| *** lan_ is now known as lan | 03:10 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add resource tag to logging in volume.manager.py https://review.openstack.org/175077 | 03:15 |
| *** rongze has quit IRC | 03:15 | |
| *** annashen has joined #openstack-cinder | 03:30 | |
| *** rongze has joined #openstack-cinder | 03:30 | |
| *** ebalduf has joined #openstack-cinder | 03:31 | |
| *** xyang has joined #openstack-cinder | 03:39 | |
| *** xyang has quit IRC | 03:42 | |
| *** esker has joined #openstack-cinder | 03:49 | |
| *** rongze has quit IRC | 03:58 | |
| *** rongze has joined #openstack-cinder | 04:01 | |
| *** breitz has quit IRC | 04:01 | |
| *** breitz1 has joined #openstack-cinder | 04:01 | |
| *** rongze has quit IRC | 04:04 | |
| *** annashen has quit IRC | 04:04 | |
| *** TobiasE has quit IRC | 04:04 | |
| *** rongze has joined #openstack-cinder | 04:06 | |
| *** rongze has quit IRC | 04:07 | |
| *** krtaylor has quit IRC | 04:08 | |
| *** krtaylor has joined #openstack-cinder | 04:11 | |
| *** esker has quit IRC | 04:16 | |
| *** esker has joined #openstack-cinder | 04:17 | |
| *** coolsvap|afk is now known as coolsvap | 04:31 | |
| *** coolsvap is now known as coolsvap|afk | 04:32 | |
| *** annashen has joined #openstack-cinder | 04:35 | |
| *** annashen has quit IRC | 04:48 | |
| *** esker has quit IRC | 04:51 | |
| *** david-lyle has joined #openstack-cinder | 04:59 | |
| *** sgotliv has joined #openstack-cinder | 05:00 | |
| *** mtanino has quit IRC | 05:01 | |
| *** pradipta has joined #openstack-cinder | 05:04 | |
| *** sks has joined #openstack-cinder | 05:05 | |
| *** deepakcs has joined #openstack-cinder | 05:10 | |
| *** bill_az has quit IRC | 05:10 | |
| *** nkrinner has joined #openstack-cinder | 05:12 | |
| *** bill_az has joined #openstack-cinder | 05:14 | |
| *** vilobhmm1 has quit IRC | 05:15 | |
| *** vilobhmm1 has joined #openstack-cinder | 05:17 | |
| *** krtaylor has quit IRC | 05:19 | |
| *** vilobhmm11 has joined #openstack-cinder | 05:19 | |
| *** annashen has joined #openstack-cinder | 05:20 | |
| *** markvoelker_ has joined #openstack-cinder | 05:21 | |
| *** markvoelker has quit IRC | 05:21 | |
| *** bswartz has quit IRC | 05:21 | |
| *** rwsu has quit IRC | 05:21 | |
| *** vilobhmm1 has quit IRC | 05:22 | |
| *** med_ has quit IRC | 05:24 | |
| *** ebalduf has quit IRC | 05:27 | |
| *** krtaylor has joined #openstack-cinder | 05:33 | |
| *** bswartz has joined #openstack-cinder | 05:35 | |
| *** med_ has joined #openstack-cinder | 05:36 | |
| *** med_ has quit IRC | 05:36 | |
| *** med_ has joined #openstack-cinder | 05:36 | |
| *** Longgeek has quit IRC | 05:44 | |
| *** _cjones_ has joined #openstack-cinder | 05:48 | |
| openstackgerrit | yogeshprasad proposed openstack/cinder: Add chap support to CloudByte cinder driver https://review.openstack.org/173733 | 05:48 |
| *** xyang1 has quit IRC | 05:48 | |
| *** lpetrut has joined #openstack-cinder | 05:50 | |
| *** _cjones_ has quit IRC | 05:53 | |
| *** haomaiwang has quit IRC | 05:55 | |
| *** Longgeek has joined #openstack-cinder | 05:57 | |
| *** nikesh has joined #openstack-cinder | 05:57 | |
| *** emagana has joined #openstack-cinder | 05:59 | |
| *** Maike has joined #openstack-cinder | 06:01 | |
| *** BharatK has joined #openstack-cinder | 06:04 | |
| *** ankit_ag has joined #openstack-cinder | 06:08 | |
| *** afazekas_ has joined #openstack-cinder | 06:11 | |
| *** Longgeek has quit IRC | 06:16 | |
| *** anshul has quit IRC | 06:20 | |
| *** rongze has joined #openstack-cinder | 06:22 | |
| *** lpetrut has quit IRC | 06:23 | |
| *** Longgeek has joined #openstack-cinder | 06:26 | |
| *** ebalduf has joined #openstack-cinder | 06:28 | |
| *** ebalduf has quit IRC | 06:32 | |
| *** rongze has quit IRC | 06:32 | |
| *** anshul has joined #openstack-cinder | 06:33 | |
| *** Longgeek_ has joined #openstack-cinder | 06:43 | |
| *** Longgeek has quit IRC | 06:46 | |
| *** annashen has quit IRC | 06:46 | |
| *** sileht has quit IRC | 06:48 | |
| *** sgotliv has quit IRC | 06:50 | |
| *** haomaiwang has joined #openstack-cinder | 06:54 | |
| *** vilobhmm11 has quit IRC | 06:54 | |
| zigo | jgriffith: Well, see by yourself, http://54.164.167.86/solidfire-ci-logs/cinder-conf-samples/cinder.conf.sample_04_15_2015.txt | 07:05 |
| zigo | there's lots of directives missing in the keystone_authtoken, as per: https://bugs.launchpad.net/cinder/+bug/1447380 which I reported. | 07:06 |
| openstack | Launchpad bug 1447380 in Cinder "wrong cinder.conf.sample generation: missing directives for keystone_authtoken (at least)" [Undecided,New] | 07:06 |
| *** Maike has quit IRC | 07:06 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Create consistgroup from cgsnapshot support in VNX driver https://review.openstack.org/163706 | 07:09 |
| *** Longgeek_ has quit IRC | 07:09 | |
| *** sileht has joined #openstack-cinder | 07:10 | |
| *** Longgeek has joined #openstack-cinder | 07:11 | |
| *** ronis has joined #openstack-cinder | 07:11 | |
| *** rushiagr_away is now known as rushiagr | 07:12 | |
| *** davechen has joined #openstack-cinder | 07:14 | |
| *** davechen1 has quit IRC | 07:16 | |
| *** lpetrut has joined #openstack-cinder | 07:17 | |
| *** davechen1 has joined #openstack-cinder | 07:22 | |
| *** davechen has quit IRC | 07:24 | |
| *** bkopilov has quit IRC | 07:26 | |
| *** bill_az has quit IRC | 07:26 | |
| *** _cjones_ has joined #openstack-cinder | 07:37 | |
| *** jistr has joined #openstack-cinder | 07:38 | |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: Windows SMBFS: Fix image resize errors during volume creation https://review.openstack.org/174291 | 07:38 |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: SMBFS: Fix retrieving total allocated size https://review.openstack.org/158713 | 07:40 |
| *** e0ne has joined #openstack-cinder | 07:41 | |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: SMBFS: Add minimum qemu-img version requirement https://review.openstack.org/161299 | 07:41 |
| *** _cjones_ has quit IRC | 07:42 | |
| *** bkopilov has joined #openstack-cinder | 07:44 | |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: Windows: Improve vhdutils error messages https://review.openstack.org/174973 | 07:44 |
| openstackgerrit | Jeegn Chen proposed openstack/cinder: Make VNX Cinder Driver aware of VNX Pool Full Threshold https://review.openstack.org/175247 | 07:44 |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: Fix fetch_to_volume_format if vhd is requested https://review.openstack.org/161363 | 07:52 |
| *** coolsvap|afk is now known as coolsvap | 07:52 | |
| *** coolsvap is now known as coolsvap|afk | 07:53 | |
| *** Maike has joined #openstack-cinder | 07:53 | |
| *** e0ne is now known as e0ne_ | 07:53 | |
| *** jordanP has joined #openstack-cinder | 08:11 | |
| *** e0ne_ is now known as e0ne | 08:14 | |
| *** bkopilov has quit IRC | 08:19 | |
| *** jordanP has quit IRC | 08:19 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Attach/detach batch processing in VNX driver https://review.openstack.org/154350 | 08:20 |
| *** e0ne is now known as e0ne_ | 08:25 | |
| openstackgerrit | Julien Danjou proposed openstack/cinder: tests: replace mox by mox3, clean out mox usage https://review.openstack.org/147476 | 08:26 |
| *** e0ne_ is now known as e0ne | 08:26 | |
| *** ebalduf has joined #openstack-cinder | 08:29 | |
| *** Longgeek has quit IRC | 08:31 | |
| *** Longgeek has joined #openstack-cinder | 08:31 | |
| *** e0ne has quit IRC | 08:32 | |
| *** ebalduf has quit IRC | 08:34 | |
| openstackgerrit | Julien Danjou proposed openstack/cinder: Leverage timeutils, drop strtime() usage https://review.openstack.org/165798 | 08:35 |
| openstackgerrit | Bharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Using mount method in RemoteFsClient https://review.openstack.org/165424 | 08:41 |
| *** jordanP has joined #openstack-cinder | 08:43 | |
| *** vincepii has joined #openstack-cinder | 08:44 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Handle concurrent inventory folder create https://review.openstack.org/167989 | 08:50 |
| *** alexpilotti has joined #openstack-cinder | 08:52 | |
| *** jwang has joined #openstack-cinder | 08:54 | |
| openstackgerrit | Bharat Kumar Kobagana proposed openstack/cinder: Using 'fallocate' instead of 'dd' to create raw volumes https://review.openstack.org/164527 | 08:56 |
| *** jwang_ has quit IRC | 08:57 | |
| *** btran has quit IRC | 08:58 | |
| *** jwang_ has joined #openstack-cinder | 08:59 | |
| openstackgerrit | Rick Chen proposed openstack/cinder: Add patch for consistency group update in ProphetStor driver https://review.openstack.org/169653 | 08:59 |
| *** jwang has quit IRC | 09:01 | |
| *** haomaiwang has quit IRC | 09:05 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Create iSCSI lio portals with right IPs and port https://review.openstack.org/161209 | 09:06 |
| *** e0ne has joined #openstack-cinder | 09:13 | |
| openstackgerrit | YAMADA Hideki proposed openstack/cinder: Add missing backup/restore methods to Sheepdog driver https://review.openstack.org/162927 | 09:18 |
| openstackgerrit | Bharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Support over subscription in thin provisioning https://review.openstack.org/162718 | 09:18 |
| *** ndipanov has joined #openstack-cinder | 09:18 | |
| *** kaisers has quit IRC | 09:20 | |
| *** kaisers has joined #openstack-cinder | 09:23 | |
| *** alecv has joined #openstack-cinder | 09:24 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add missing backups entry to default quota class https://review.openstack.org/174941 | 09:25 |
| *** d0ugal has quit IRC | 09:29 | |
| *** d0ugal has joined #openstack-cinder | 09:29 | |
| *** d0ugal is now known as Guest81472 | 09:29 | |
| *** e0ne is now known as e0ne_ | 09:31 | |
| *** haigang has quit IRC | 09:31 | |
| *** anshul has quit IRC | 09:32 | |
| *** aix has joined #openstack-cinder | 09:35 | |
| *** e0ne_ is now known as e0ne | 09:35 | |
| *** Guest81472 is now known as d0ugal2 | 09:40 | |
| *** anshul has joined #openstack-cinder | 09:44 | |
| *** annashen has joined #openstack-cinder | 09:47 | |
| *** d0ugal2 is now known as d0ugal | 09:47 | |
| *** d0ugal has quit IRC | 09:47 | |
| *** d0ugal has joined #openstack-cinder | 09:47 | |
| *** annashen has quit IRC | 09:51 | |
| *** e0ne is now known as e0ne_ | 09:53 | |
| openstackgerrit | Rushi Agrawal proposed openstack/cinder-specs: Snapshot sharing https://review.openstack.org/173779 | 09:57 |
| *** Longgeek has quit IRC | 09:57 | |
| *** yuriy_n17 has joined #openstack-cinder | 09:59 | |
| *** Longgeek has joined #openstack-cinder | 09:59 | |
| *** davechen1 has quit IRC | 09:59 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement blueprint: check-volume-exists https://review.openstack.org/175809 | 10:03 |
| *** e0ne_ has quit IRC | 10:03 | |
| *** e0ne has joined #openstack-cinder | 10:09 | |
| *** IanGovett has joined #openstack-cinder | 10:13 | |
| *** afazekas_ has quit IRC | 10:14 | |
| *** yamada-h has joined #openstack-cinder | 10:23 | |
| *** yamada-h has quit IRC | 10:23 | |
| *** afazekas has joined #openstack-cinder | 10:32 | |
| *** haomaiwa_ has joined #openstack-cinder | 10:38 | |
| *** rushiagr is now known as rushiagr_away | 10:40 | |
| *** pratyu has joined #openstack-cinder | 10:45 | |
| *** pratyu has quit IRC | 10:47 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Preserve usage and reservations on quota deletion https://review.openstack.org/162722 | 10:47 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Preserve usage and reservations on quota deletion https://review.openstack.org/162722 | 10:49 |
| *** heyun has quit IRC | 10:50 | |
| *** dalgaaf has joined #openstack-cinder | 10:53 | |
| *** Maike has quit IRC | 11:04 | |
| *** _cjones_ has joined #openstack-cinder | 11:14 | |
| *** timcl has joined #openstack-cinder | 11:18 | |
| *** _cjones_ has quit IRC | 11:19 | |
| *** aix has quit IRC | 11:22 | |
| *** yamada-h has joined #openstack-cinder | 11:23 | |
| *** yamada-h has quit IRC | 11:28 | |
| *** david-lyle has quit IRC | 11:29 | |
| *** ebalduf has joined #openstack-cinder | 11:32 | |
| *** deepakcs has quit IRC | 11:36 | |
| *** ebalduf has quit IRC | 11:36 | |
| *** Maike has joined #openstack-cinder | 11:43 | |
| *** akerr has joined #openstack-cinder | 11:47 | |
| *** BharatK has quit IRC | 11:58 | |
| *** ociuhandu has joined #openstack-cinder | 11:59 | |
| *** aix has joined #openstack-cinder | 12:00 | |
| *** aix has quit IRC | 12:00 | |
| *** aix has joined #openstack-cinder | 12:00 | |
| *** yamada-h has joined #openstack-cinder | 12:02 | |
| *** dencaval has joined #openstack-cinder | 12:02 | |
| *** e0ne is now known as e0ne_ | 12:04 | |
| *** david-lyle has joined #openstack-cinder | 12:05 | |
| *** yamada-h has quit IRC | 12:06 | |
| *** zhenguo has quit IRC | 12:12 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to import/export snapshots https://review.openstack.org/144590 | 12:14 |
| *** rooneym has quit IRC | 12:14 | |
| openstackgerrit | Joel Coffman proposed openstack/cinder: Add test case for volume_encryption_metadata_get https://review.openstack.org/168177 | 12:16 |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to import/export snapshots https://review.openstack.org/144590 | 12:19 |
| openstackgerrit | wanghao proposed openstack/cinder: query volume detail support volume_glance_metadata https://review.openstack.org/147738 | 12:24 |
| *** e0ne_ is now known as e0ne | 12:24 | |
| rmstar | hi guys. is thre a way to increase the download speed when i create a volume from an image? | 12:28 |
| *** david-lyle has quit IRC | 12:31 | |
| *** kaisers has quit IRC | 12:33 | |
| openstackgerrit | Joel Coffman proposed openstack/cinder: Remove unnecessary checks for encrypted types https://review.openstack.org/168179 | 12:35 |
| *** rooneym has joined #openstack-cinder | 12:37 | |
| *** kaisers has joined #openstack-cinder | 12:37 | |
| *** bswartz has quit IRC | 12:40 | |
| *** xyang1 has joined #openstack-cinder | 12:45 | |
| *** Longgeek has quit IRC | 12:47 | |
| *** erlon has joined #openstack-cinder | 12:47 | |
| *** Longgeek has joined #openstack-cinder | 12:48 | |
| *** pradipta has quit IRC | 12:49 | |
| *** Adriano_ has joined #openstack-cinder | 12:49 | |
| *** bill_az has joined #openstack-cinder | 12:51 | |
| openstackgerrit | wanghao proposed openstack/cinder-specs: Adds support to force-delete backups https://review.openstack.org/98325 | 12:54 |
| *** Longgeek_ has joined #openstack-cinder | 12:55 | |
| *** Longgeek has quit IRC | 12:56 | |
| openstackgerrit | wanghao proposed openstack/cinder: non-admin user to query volume filter by az https://review.openstack.org/156034 | 12:56 |
| *** vincepii has quit IRC | 12:57 | |
| *** rushil has joined #openstack-cinder | 13:00 | |
| openstackgerrit | wanghao proposed openstack/cinder: Fix response when query host detail by host name https://review.openstack.org/162601 | 13:00 |
| *** Yogi1 has joined #openstack-cinder | 13:01 | |
| *** _cjones_ has joined #openstack-cinder | 13:03 | |
| *** yamada-h has joined #openstack-cinder | 13:03 | |
| *** davechen has joined #openstack-cinder | 13:05 | |
| *** zhipeng has joined #openstack-cinder | 13:06 | |
| *** _cjones_ has quit IRC | 13:09 | |
| *** yamada-h has quit IRC | 13:10 | |
| *** bswartz has joined #openstack-cinder | 13:15 | |
| *** davechen1 has joined #openstack-cinder | 13:16 | |
| *** davechen has quit IRC | 13:19 | |
| *** mriedem_away has quit IRC | 13:21 | |
| *** dustins has joined #openstack-cinder | 13:21 | |
| *** eduardo has joined #openstack-cinder | 13:21 | |
| *** annegentle has joined #openstack-cinder | 13:22 | |
| *** e0ne has quit IRC | 13:25 | |
| *** sks has quit IRC | 13:25 | |
| *** davechen1 has left #openstack-cinder | 13:26 | |
| *** e0ne has joined #openstack-cinder | 13:27 | |
| *** aswadr has joined #openstack-cinder | 13:30 | |
| smcginnis | rmstar: Not really. I know at SolidFire has added some interesting image caching to their driver. | 13:32 |
| *** ebalduf has joined #openstack-cinder | 13:34 | |
| *** mwichmann has joined #openstack-cinder | 13:35 | |
| *** jungleboyj has quit IRC | 13:36 | |
| *** ebalduf has quit IRC | 13:38 | |
| *** mriedem has joined #openstack-cinder | 13:47 | |
| *** kaisers has quit IRC | 14:00 | |
| *** Svedrin has quit IRC | 14:01 | |
| *** kaisers has joined #openstack-cinder | 14:02 | |
| *** ganso_ has joined #openstack-cinder | 14:03 | |
| *** ebalduf has joined #openstack-cinder | 14:07 | |
| *** eharney has joined #openstack-cinder | 14:08 | |
| jgriffith | zigo: That's just awesome! | 14:12 |
| *** esker has joined #openstack-cinder | 14:12 | |
| jgriffith | zigo: sorry, I wasn't aware yesterday you found this problem | 14:13 |
| jgriffith | zigo: I thought you were just asking if there was a reference available :) | 14:13 |
| zigo | jgriffith: Do you know what's the solution? | 14:13 |
| jgriffith | zigo: not at the moment... looking | 14:13 |
| zigo | The generation seems quite broken to me! :) | 14:13 |
| jgriffith | zigo: this is the first I've been aware of it | 14:13 |
| zigo | Ok. | 14:13 |
| jgriffith | zigo: well, dependins on your point of view | 14:14 |
| jgriffith | zigo: keystone/generation or something in oslo.config | 14:14 |
| zigo | jgriffith: I'm using genrated config file other projects (like Ceilometer for example), with the exact same lib versions, and it works. | 14:14 |
| jgriffith | zigo: This is why I was not a fan of not gating on config | 14:14 |
| *** lcurtis has joined #openstack-cinder | 14:14 | |
| jgriffith | zigo: curious about Nova... bu that's a good data point | 14:15 |
| zigo | Though it's using oslo-config-generator directly, not the <project>/common stuff. | 14:15 |
| zigo | Hum... | 14:15 |
| zigo | Let me check. | 14:15 |
| jgriffith | zigo: yeah, the project/common stuff may very well be a problem on our side | 14:15 |
| zigo | For Nova, I use my own baked nova.conf, and just provide the sample as a doc. | 14:15 |
| *** lpetrut has quit IRC | 14:15 | |
| jgriffith | zigo: :) | 14:15 |
| *** nkrinner has quit IRC | 14:16 | |
| jgriffith | zigo: Dang... thought I may have found somebody that's been able to build a nova.conf | 14:16 |
| jgriffith | zigo: I've had bad luck with that | 14:16 |
| zigo | jgriffith: Oh, fun, I have the same issue in Nova !!! :) | 14:16 |
| jgriffith | zigo: Yeah, that was what I suspected | 14:17 |
| zigo | Which I didn't see, because of my custom nova.conf. | 14:17 |
| zigo | So, the issue may well be simply in oslo-incubator. | 14:17 |
| jgriffith | zigo: It may be intentional. | 14:18 |
| zigo | jgriffith: To remove some directives for the authtoken? | 14:18 |
| jgriffith | dhellmann: might have some insite ^^ | 14:18 |
| zigo | This would be a weird intention, no? | 14:18 |
| jgriffith | dhellmann: zigo noticed all the keystone directives are no longer in cinder/nova.conf files | 14:18 |
| jgriffith | dhellmann: https://bugs.launchpad.net/cinder/+bug/1447380 | 14:18 |
| openstack | Launchpad bug 1447380 in Cinder "wrong cinder.conf.sample generation: missing directives for keystone_authtoken (at least)" [Undecided,Confirmed] | 14:18 |
| dhellmann | jgriffith: how is that file being generated? | 14:19 |
| zigo | dhellmann: More precisely, the 2nd half of all the directives, including login/pass/tenant/hostname. | 14:19 |
| jgriffith | dhellmann: the old tox genconfig | 14:19 |
| zigo | dhellmann: No problem with packages using oslo-config-generator directly, though there's the issue for nova and cinder which are using tools/config/generate_sample.sh. | 14:19 |
| jgriffith | zigo: dhellmann maybe I should just move us to use the generator directly | 14:20 |
| jgriffith | at least see what happens :) | 14:20 |
| zigo | jgriffith: If you know the full command line to it, that's enough for me! | 14:20 |
| jgriffith | FWIW: https://github.com/openstack/cinder/tree/master/tools/config | 14:20 |
| zigo | jgriffith: I'd need a --namespace list, and that's it ... | 14:20 |
| jgriffith | is what we use currently | 14:20 |
| dhellmann | jgriffith: where are the missing options defined? | 14:20 |
| zigo | dhellmann: In oslo.messaging. | 14:21 |
| zigo | But that's the only one I *saw*. | 14:21 |
| zigo | Maybe there's more? | 14:21 |
| dhellmann | ok, I think the old generator had support for the libraries through the entry point system, so the invocation may just need to be updated to add the right libs | 14:22 |
| *** haomaiw__ has joined #openstack-cinder | 14:22 | |
| jgriffith | dhellmann: thanks... so honestly I literally *just* saw zigo 's bug so I have no info other than that yet | 14:22 |
| *** haomaiwa_ has quit IRC | 14:22 | |
| jgriffith | dhellmann: thanks for the quick look/suggestion | 14:22 |
| jgriffith | I'll see if I can determine what's actually missing and if it *matters* | 14:23 |
| dhellmann | jgriffith: sure, and let me know if you have trouble making it work. today is going to be busy, but I think I'll have time tomorrow | 14:23 |
| jgriffith | zigo: there's a new keystone discovery thingy-magigy that might eliminate the need here depending on what's missing | 14:23 |
| jgriffith | dhellmann: awesome, thanks | 14:23 |
| jgriffith | zigo: first thing we need to verify is those opts aren't just dead as a result of auto discovery | 14:24 |
| *** rwsu has joined #openstack-cinder | 14:24 | |
| *** anshul has quit IRC | 14:24 | |
| *** zhipeng has quit IRC | 14:25 | |
| e0ne | jgriffith: morning. | 14:28 |
| *** mtanino has joined #openstack-cinder | 14:28 | |
| *** jungleboyj has joined #openstack-cinder | 14:30 | |
| *** haomaiw__ has quit IRC | 14:32 | |
| *** haomaiwa_ has joined #openstack-cinder | 14:32 | |
| zigo | e0ne: Hi there! | 14:33 |
| e0ne | it it ok that cinder-backup use swift if it failed to initialize other driver? ceph in my case | 14:33 |
| e0ne | zigo: hi | 14:34 |
| jgriffith | e0ne: hello | 14:35 |
| *** jistr is now known as jistr|mtg | 14:35 | |
| *** anshul has joined #openstack-cinder | 14:37 | |
| *** rmesta has joined #openstack-cinder | 14:37 | |
| e0ne | jgriffith: i'm trying to understans how cinder-backup should works | 14:38 |
| tbarron | e0ne: since only one backup service can be confiured (no scheduler, no multi-backend) I would think that we should hard-fail in that case rather than falling-back to Swift. | 14:38 |
| *** haomaiwa_ has quit IRC | 14:38 | |
| tbarron | DuncanT ^^^^^ | 14:38 |
| e0ne | jgriffith: in my case i've got not well-configured cinder-backup service: http://paste.openstack.org/show/205292/ | 14:39 |
| *** haomaiwang has joined #openstack-cinder | 14:39 | |
| e0ne | tbarron: i'm agree with you. but it is not what we have now | 14:39 |
| *** garysmith_ has joined #openstack-cinder | 14:40 | |
| tbarron | e0ne: sounds like a bug to me | 14:40 |
| *** thangp has joined #openstack-cinder | 14:41 | |
| *** emagana has quit IRC | 14:41 | |
| jgriffith | e0ne: so I"m not sure I follow, but it should work "just like" cinder-volume backends, sans multi-backend | 14:42 |
| *** emagana has joined #openstack-cinder | 14:42 | |
| tbarron | e0ne: that paste is short, but are you sure it's not coming from a volume driver? Backup manager loads the volume driver for each configured volume backend and runs its backup/restore method. That's independent of whether Swift, Ceph, NFS, etc. is configured for the service itself. | 14:43 |
| e0ne | tbarron: i configured rbd backend for cinder-volume but didn't set backup_driver | 14:44 |
| tbarron | (I consider it a serious scaling issue that the backup service has to do this today.) | 14:44 |
| tbarron | e0ne: then you should get Swift | 14:45 |
| tbarron | that's the default. | 14:45 |
| e0ne | tbarron: but it tries to connect to ceph | 14:45 |
| tbarron | backup service is orthogonal to the cinder volume backends | 14:45 |
| e0ne | tbarron: and works with ceph if it is available | 14:45 |
| tbarron | e0ne: it should connect with Ceph for Ceph backend volumes, but back them up to Swift unless you have configured backup service to other than default. | 14:46 |
| e0ne | tbarron: if ceph is not available, it uses swift because it set as default | 14:46 |
| *** emagana has quit IRC | 14:46 | |
| e0ne | tbarron: thanks. your're right | 14:47 |
| e0ne | i've missed it:( | 14:47 |
| e0ne | i'm sorry for confuse | 14:47 |
| tbarron | I was very confuseed by the same thing when I started working on backup. | 14:48 |
| tbarron | backup manager loads volume drivers. They in turn use the backup service. It's not intuitive. | 14:48 |
| *** ankit_ag has quit IRC | 14:49 | |
| *** haomaiwang has quit IRC | 14:49 | |
| e0ne | tbarron: thanks for help! | 14:49 |
| akerr | that sounds like circular dependency... | 14:50 |
| *** haomaiwang has joined #openstack-cinder | 14:50 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add resource tag to logging in volume.manager.py https://review.openstack.org/175077 | 14:50 |
| e0ne | tbarron: we need more user friendly logs there | 14:50 |
| tbarron | Having to load the backup drivers locally rather than doing a remote attach implies that the backup service node has to run all volume backends too. | 14:50 |
| tbarron | That's a scaling limitation. | 14:51 |
| akerr | tbarron: no one backs up anyway, so not a problem :) | 14:51 |
| tbarron | Instead of scaling cinder services "out" one has to scale them "up" with a big honking single node. | 14:51 |
| tbarron | akerr: :-) | 14:51 |
| jgriffith | tbarron: I'd agree, TBH I haven't quite understood that architecture. | 14:52 |
| jgriffith | tbarron: assumed my ignorance is the problem, as DuncanT is typically very cognisant of things like scale | 14:53 |
| tbarron | jgriffith: I think Duncan told me that when we get remote attach for LVM we can address the issue to which I point. | 14:53 |
| jgriffith | tbarron: hmm... well, we already have remote attach for LVM :) | 14:54 |
| jgriffith | tbarron: its' called iSCSI | 14:54 |
| tbarron | jgriffith: so I'm probably confused then. | 14:54 |
| DuncanT | tbarron: The backup service just does an attach... | 14:54 |
| DuncanT | tbarron: Except for LVM | 14:54 |
| tbarron | here's the man | 14:54 |
| DuncanT | tbarron: It needs some fixing up, but the principle is, I think, sound | 14:54 |
| DuncanT | tbarron: The service is usually CPU bound, particularly if you use SSL to swift | 14:55 |
| *** xyang has joined #openstack-cinder | 14:55 | |
| DuncanT | (just got an alert on my phone, not read scrollback yet) | 14:55 |
| akerr | DuncanT: what does the attaching? The backup node? | 14:55 |
| *** alexpilotti has quit IRC | 14:55 | |
| jgriffith | sdague: you working that same push to Cinder? | 14:56 |
| * tbarron stays tuned ... | 14:56 | |
| jgriffith | sdague: https://review.openstack.org/#/c/176777/1 | 14:56 |
| DuncanT | akerr: The backup service. It uses the backup method in the driver, because different drivers can do it differently | 14:56 |
| DuncanT | akerr: But most will just do a iSCSI attach | 14:56 |
| akerr | so wouldn't you still be bound by a single node having to perform all backups? | 14:56 |
| DuncanT | akerr: No, the idea is you can have as many as you like | 14:57 |
| akerr | so you'd have multiple backup services? | 14:57 |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series: Fix instance live-migration with attached volumes https://review.openstack.org/170990 | 14:57 |
| DuncanT | akerr: Yes, they should be active/active/active/... scalable | 14:57 |
| akerr | and whoever gets the rabbit message first wins? | 14:58 |
| DuncanT | akerr: Correct | 14:58 |
| *** thangp has quit IRC | 14:58 | |
| DuncanT | akerr: At the moment it sends it to the hostname of the hosting volume node | 14:58 |
| DuncanT | akerr: That is because of the LVM driver | 14:58 |
| akerr | ok, Thanks! I'll have to take another hard look at the code in my abundant free time | 15:00 |
| DuncanT | akerr: As I said, at the moment it goes to a named node, that needs fixing | 15:00 |
| DuncanT | akerr: It got changed at the last minute for LVM, and never got fixed up | 15:01 |
| sdague | jgriffith: I thought someone else had that for cinder | 15:01 |
| akerr | DuncanT: ok, thanks | 15:01 |
| jgriffith | sdague: I'm doing it now | 15:02 |
| jgriffith | sdague: I'll look and make sure it's not already in the pipeline | 15:03 |
| *** lpetrut has joined #openstack-cinder | 15:08 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Sync oslo service module for SIGTERM exit https://review.openstack.org/176802 | 15:08 |
| jgriffith | sdague: thanks for tracking that down | 15:09 |
| *** adurbin_ has joined #openstack-cinder | 15:09 | |
| *** ronis has quit IRC | 15:14 | |
| jgriffith | zigo: I think I see the problem with config | 15:20 |
| zigo | Cool! :) | 15:20 |
| zigo | jgriffith: Add me to the review! :) | 15:20 |
| jgriffith | zigo: will do, assuming I'm correct :) | 15:20 |
| smcginnis | jgriffith: Could have sworn someone else had just submitted a patch for the SIGTERM issue. Is this related? | 15:22 |
| jgriffith | smcginnis: maybe I missed it, sean thought the same thing | 15:22 |
| jgriffith | smcginnis: I didn't see it though | 15:23 |
| smcginnis | jgriffith: Ah, I see why I couldn't find it. Merged last night. | 15:23 |
| smcginnis | jgriffith: https://review.openstack.org/#/c/176455/ | 15:23 |
| smcginnis | jgriffith: Haven't looked close enough to see if they are different issues. | 15:23 |
| jgriffith | smcginnis: umm, that's weird, if it merged why is the file still out of date? | 15:23 |
| jgriffith | smcginnis: OH, he just cherry picked the line of the fix. Didn't do a proper sync | 15:24 |
| jgriffith | smcginnis: Ok, have to leave judgement on mine to thingee jungleboyj and dhellmann | 15:25 |
| jgriffith | typically we used to discourage cherry picking rather than full syncs, guess that's not valid any longer | 15:25 |
| smcginnis | jgriffith: In my two second review, looks like what you are adding is needed/good. | 15:25 |
| smcginnis | jgriffith: Need to walk through it more to make sure I understand. | 15:25 |
| jgriffith | smcginnis: seems like a sync is worthwhile IMHO, but I should change the commit message probably | 15:26 |
| *** IanGovett has quit IRC | 15:26 | |
| jgriffith | zigo: fixed | 15:27 |
| jgriffith | zigo: patch incoming | 15:27 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/176823 | 15:28 |
| dhellmann | jgriffith: did you sync service.py from master, or a specific commit? | 15:28 |
| jgriffith | dhellmann: I synced from master | 15:29 |
| jungleboyj | jgriffith: Need to run to a meeting. Will look. | 15:29 |
| *** david-lyle has joined #openstack-cinder | 15:29 | |
| dhellmann | jgriffith: ok, I'm not seeing some changes I expected, but let me make sure I have my local copies up to date | 15:29 |
| jgriffith | dhellmann: hmmm | 15:29 |
| dhellmann | jgriffith: oh, I have the juno copy of cinder checked out, ignore me | 15:30 |
| *** thangp has joined #openstack-cinder | 15:30 | |
| jgriffith | dhellmann: phewww :) | 15:30 |
| jungleboyj | We shouldn't be doing single line fixes of incubator code. | 15:30 |
| jgriffith | jungleboyj: umm, you gave it a +2 | 15:30 |
| jungleboyj | Doh! And I plus 2'd that. I mis interpreted what was going on there. | 15:30 |
| jungleboyj | jgriffith: Yeah, I see that. :-( | 15:31 |
| jgriffith | and then thingee approved/merged it | 15:31 |
| jungleboyj | Yesterday was one of those days. | 15:31 |
| *** jdurgin1 has joined #openstack-cinder | 15:31 | |
| thingee | yeah was a mistake. we can still revert it | 15:32 |
| thingee | jgriffith: checked with ttx, we're making an exception on this one | 15:34 |
| thingee | check openstack-relmgr-office | 15:34 |
| jgriffith | thingee: cool | 15:34 |
| jgriffith | thingee: would've been good to communicate that for idiots like me :) | 15:35 |
| jgriffith | thingee: maybe a note in the commit mesg since it goes against usual process | 15:35 |
| thingee | definitely | 15:35 |
| *** sandywalsh has quit IRC | 15:37 | |
| *** sandywalsh has joined #openstack-cinder | 15:38 | |
| thingee | so unless people start screaming really loud about blockers in Cinder, we'll be having an rc2 tag soon | 15:39 |
| openstackgerrit | John Griffith proposed openstack/cinder: Remove incubator config and update genconfig https://review.openstack.org/176857 | 15:39 |
| jgriffith | thingee: you may want to consider adding that one ^^ | 15:40 |
| jgriffith | :) | 15:40 |
| thingee | sigh | 15:42 |
| e0ne | thingee: what dou you think about getting https://review.openstack.org/#/c/175555/ to Kilo? | 15:43 |
| openstackgerrit | John Griffith proposed openstack/cinder: Sync oslo service module https://review.openstack.org/176802 | 15:43 |
| thingee | e0ne: too late | 15:43 |
| e0ne | thingee: ok, thanks | 15:44 |
| jgriffith | zigo: https://review.openstack.org/176857 try that out | 15:44 |
| jgriffith | zigo: I kinda rushed/multi-tasked, may have missed something... lemme look | 15:45 |
| *** bkopilov has joined #openstack-cinder | 15:46 | |
| *** akerr has quit IRC | 15:50 | |
| *** tsekiyama has joined #openstack-cinder | 15:52 | |
| *** _cjones_ has joined #openstack-cinder | 15:53 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix range check for NFS used ratio https://review.openstack.org/175573 | 15:55 |
| *** david-lyle has quit IRC | 15:56 | |
| * jungleboyj is back. | 15:56 | |
| *** _cjones_ has quit IRC | 15:57 | |
| jungleboyj | jgriffith: Just added a link to one of my reviews with the format we have been using for Oslo syncs. | 15:58 |
| openstackgerrit | Eric Harney proposed openstack/python-cinderclient: Remove "OPTIONAL:" from optional argument help text (v2) https://review.openstack.org/176879 | 16:00 |
| *** _cjones_ has joined #openstack-cinder | 16:01 | |
| xyang | thingee: question for you one cinderclient release | 16:01 |
| * jungleboyj is so confused. | 16:01 | |
| thingee | xyang: ok | 16:01 |
| xyang | thingee: s/one/on | 16:01 |
| xyang | thingee: is it released already for liberty? | 16:01 |
| thingee | no | 16:01 |
| xyang | thingee: is it released for kilo? | 16:02 |
| thingee | no | 16:02 |
| xyang | thingee: kmartin found a link yesterday about a release | 16:02 |
| xyang | thingee: that's good | 16:02 |
| xyang | thingee: as I have two patches that I want to get in | 16:02 |
| xyang | thingee: do you have a release date in mind? | 16:03 |
| thingee | no | 16:03 |
| xyang | thingee: that's easy:) no for all questions | 16:03 |
| thingee | working on RC2 of cinder atm | 16:03 |
| xyang | thingee: sure. | 16:04 |
| jgriffith | jungleboyj: ? | 16:04 |
| jgriffith | jungleboyj: oh | 16:04 |
| jgriffith | sure | 16:05 |
| jungleboyj | jgriffith: I am looking at https://review.openstack.org/176857 | 16:05 |
| jgriffith | jungleboyj: yeah, I'm with ya now | 16:05 |
| jungleboyj | I had started working on moving to the new config generator but that work was decided to be pushed off to Liberty. | 16:05 |
| jungleboyj | I didn't think we could just change which tool we were pointing to and go. Otherwise I would have done that long ago. | 16:06 |
| thingee | jungleboyj: https://review.openstack.org/#/c/176857/ | 16:06 |
| thingee | can I get an ok on that? | 16:06 |
| jungleboyj | I am trying your patch right now. | 16:06 |
| thingee | oh heh | 16:07 |
| *** garysmith_ has quit IRC | 16:07 | |
| jungleboyj | thingee: That is what I am trying right now. I am totally baffled that it works. | 16:07 |
| *** hemnafk is now known as hemna | 16:07 | |
| thingee | jungleboyj: blocking rc2 on this right now | 16:07 |
| jungleboyj | thingee: Ok, I am looking at it as we speak. | 16:07 |
| jungleboyj | Ok, that was what I was afraid of. | 16:08 |
| jungleboyj | Oh wait. | 16:08 |
| *** emagana has joined #openstack-cinder | 16:08 | |
| jungleboyj | Ok, I am not nuts. Thank god. | 16:10 |
| thingee | no, you're still nuts | 16:10 |
| jungleboyj | thingee: :p | 16:10 |
| hemna | jgriffith, ping | 16:11 |
| jungleboyj | jgriffith: I am looking at the bug. | 16:13 |
| *** emagana has quit IRC | 16:13 | |
| *** emagana has joined #openstack-cinder | 16:16 | |
| *** aix has quit IRC | 16:17 | |
| hemna | jgriffith, https://bugs.launchpad.net/cinder/+bug/1447697 | 16:18 |
| openstack | Launchpad bug 1447697 in Cinder "3PAR drivers are logging version numbers a lot" [Undecided,New] | 16:18 |
| hemna | :) | 16:18 |
| *** jistr|mtg is now known as jistr | 16:18 | |
| thingee | jungleboyj: thanks. I would be a bit nervous at this time with your initial patch. just because of size | 16:20 |
| jungleboyj | thingee: Totally agree. | 16:21 |
| *** Apoorva has joined #openstack-cinder | 16:21 | |
| jungleboyj | thingee: I have an idea. | 16:21 |
| jungleboyj | thingee: I am thinking that we can use the old genconfig to create the cinder options and use the new one to pull in the options from the libraries. | 16:22 |
| jungleboyj | A temporary fix until we work out how best to use the new one. Give me a few minutes to see if I can get something that looks right. | 16:23 |
| *** vilobhmm1 has joined #openstack-cinder | 16:25 | |
| *** crose has joined #openstack-cinder | 16:28 | |
| *** _cjones_ has quit IRC | 16:29 | |
| *** vilobhmm1 has quit IRC | 16:29 | |
| *** _cjones_ has joined #openstack-cinder | 16:31 | |
| *** patrickeast has joined #openstack-cinder | 16:31 | |
| jungleboyj | thingee: I need to go get my son to daycare. Will be back in 30 min or so. I am working on a solution that will get us by until I can get us fully moved to the new genconfig. | 16:34 |
| jungleboyj | I think we can use the old config generator to get Cinder's options pulled in and then combine that with the output from the new generator for the libraries. | 16:35 |
| *** dannywilson has joined #openstack-cinder | 16:35 | |
| *** e0ne has quit IRC | 16:36 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR don't log version numbers every stats update https://review.openstack.org/176905 | 16:36 |
| *** nestorf has joined #openstack-cinder | 16:38 | |
| *** alecv has quit IRC | 16:40 | |
| *** Yogi1 has quit IRC | 16:40 | |
| *** akerr has joined #openstack-cinder | 16:40 | |
| *** leeantho has joined #openstack-cinder | 16:43 | |
| *** jistr has quit IRC | 16:43 | |
| nestorf | is it possible to set logging level for just my driver? | 16:44 |
| openstackgerrit | Patrick East proposed openstack/cinder: Enable use of filter_function in PureISCIDriver https://review.openstack.org/176543 | 16:45 |
| hemna | nestorf, we do something like that for the 3PAR client | 16:45 |
| hemna | we have a cinder.conf entry that allows you to turn on hp3par_debug = True | 16:45 |
| hemna | which bascially creates a new logger with that new level. | 16:46 |
| hemna | nestorf, so yes, it's possible. You just have to do it all yourself in your own driver. | 16:46 |
| hemna | I wouldn't really recommend it though. | 16:46 |
| hemna | :P | 16:46 |
| *** ronis has joined #openstack-cinder | 16:47 | |
| *** jdurgin1 has quit IRC | 16:47 | |
| *** yuriy_n17 has quit IRC | 16:50 | |
| *** jordanP has quit IRC | 16:52 | |
| *** dustins_ has joined #openstack-cinder | 16:56 | |
| *** vilobhmm11 has joined #openstack-cinder | 16:57 | |
| *** vilobhmm12 has joined #openstack-cinder | 16:58 | |
| *** Yogi1 has joined #openstack-cinder | 16:58 | |
| *** dustins has quit IRC | 16:59 | |
| *** vilobhmm11 has quit IRC | 17:01 | |
| *** rushil has quit IRC | 17:02 | |
| *** cbader has quit IRC | 17:03 | |
| *** cbader has joined #openstack-cinder | 17:03 | |
| *** rushil has joined #openstack-cinder | 17:04 | |
| *** alexpilotti has joined #openstack-cinder | 17:05 | |
| *** rushiagr_away is now known as rushiagr | 17:07 | |
| *** Longgeek_ has quit IRC | 17:07 | |
| jgriffith | hemna: thanks | 17:11 |
| jgriffith | jungleboyj: thingee wait... what about genconfig? | 17:12 |
| *** garthb has joined #openstack-cinder | 17:12 | |
| *** garthb_ has joined #openstack-cinder | 17:12 | |
| *** dannywilson has quit IRC | 17:12 | |
| jgriffith | good lord! | 17:13 |
| *** harlowja_away is now known as harlowja | 17:13 | |
| *** alexpilotti has quit IRC | 17:16 | |
| *** marcusvrn has quit IRC | 17:20 | |
| *** marcusvrn has joined #openstack-cinder | 17:20 | |
| *** Maike has quit IRC | 17:23 | |
| *** ndipanov has quit IRC | 17:23 | |
| jungleboyj | jgriffith: I am back. | 17:23 |
| jungleboyj | jgriffith: I am working that. | 17:23 |
| jgriffith | jungleboyj: I abandoned my change, but I think you should reconsider the direction on your patch | 17:24 |
| jgriffith | jungleboyj: rather than the crazy deepcopy +..... | 17:24 |
| jgriffith | use an iterator and just build a list | 17:24 |
| jgriffith | not sure how/if that can be automated, should be able to | 17:24 |
| jungleboyj | jgriffith: Yeah, I am open to doing that one differently. That isn't going to happen for Kilo though. | 17:24 |
| jgriffith | just walk all the files | 17:25 |
| jungleboyj | jgriffith: That patch needs to be completely rethought. | 17:25 |
| jgriffith | jungleboyj: yeah, we pretty much F'd up there | 17:25 |
| *** marcusvrn1 has joined #openstack-cinder | 17:26 | |
| jungleboyj | That was the result of discussion with the Oslo community. They sent me off to see what I could do. | 17:26 |
| jungleboyj | So, we get to set the approach there and others will follow. | 17:27 |
| jungleboyj | That isn't going to happen today though. | 17:27 |
| *** marcusvrn has quit IRC | 17:27 | |
| jgriffith | jungleboyj: so actually, it's not hard I don't think | 17:27 |
| jgriffith | jungleboyj: you can walk the files and gather the register_opts files dynamically | 17:27 |
| *** jwcroppe has joined #openstack-cinder | 17:28 | |
| jgriffith | build a list from them | 17:28 |
| jungleboyj | Ok, that would be better than what I did. | 17:28 |
| jungleboyj | jgriffith: Wasn't sure if it was valid to build that file up dynamically though. | 17:28 |
| jgriffith | jungleboyj: ummm... why not? | 17:28 |
| jgriffith | jungleboyj: considering that's how it's worked for *years* until this change? | 17:29 |
| jungleboyj | Well right, but we built up a list in the script now building another .py file. | 17:29 |
| jungleboyj | If you are ok with it though, I can look into it. | 17:30 |
| jgriffith | jungleboyj: so you're kinda screwed as you still have to build up the import list | 17:31 |
| jungleboyj | Yeah. | 17:31 |
| jgriffith | jungleboyj: but you could then walk each of the imports dynamically at least | 17:31 |
| jgriffith | jungleboyj: and get rid of that deppcopy + stuff | 17:32 |
| jgriffith | that's kinda fugly | 17:32 |
| jgriffith | might be a more clever way... gimmie a minute | 17:32 |
| jungleboyj | jgriffith: Hold on. | 17:33 |
| jgriffith | Ok | 17:33 |
| jungleboyj | jgriffith: I don't think we want to go that direction last minute. | 17:33 |
| jgriffith | jungleboyj: yeah, much better to just leave it broken :) | 17:33 |
| jgriffith | and useless :) | 17:33 |
| jungleboyj | jgriffith: No no. :-) | 17:33 |
| jgriffith | jungleboyj: I know, your two staged diff idea | 17:34 |
| jungleboyj | Let me try something here that is less of a departure from the current approach thatwas working. | 17:34 |
| jgriffith | that's fine for kilo | 17:34 |
| jungleboyj | jgriffith: Right. | 17:34 |
| *** AlkaD has joined #openstack-cinder | 17:34 | |
| jgriffith | It works | 17:34 |
| jgriffith | FWIW | 17:34 |
| jgriffith | I have it running | 17:34 |
| jungleboyj | jgriffith: What works? | 17:35 |
| jgriffith | I run genconfig-global out using the libs to get external stuff.... | 17:35 |
| jgriffith | Then run the incubator config gen for local stuff | 17:35 |
| jgriffith | and merge them together into a single file | 17:35 |
| jungleboyj | Right. That was what I was going to put together. | 17:35 |
| jungleboyj | You already have that written? | 17:36 |
| *** patrickeast_ has joined #openstack-cinder | 17:36 | |
| jgriffith | Yeah, but it's not anything I can submit right now | 17:36 |
| jungleboyj | Ok, that was what I was trying to put together for submission. | 17:36 |
| jgriffith | It's two scripts | 17:36 |
| jungleboyj | Do you want to keep working that or do you want me to get something together? | 17:37 |
| *** patrickeast has quit IRC | 17:37 | |
| *** patrickeast_ is now known as patrickeast | 17:37 | |
| jgriffith | jungleboyj: I can do it if you guys won't object and -1 it to death :) | 17:41 |
| jgriffith | jungleboyj: so I just modified the generate_config.sh script... | 17:42 |
| jgriffith | added: | 17:42 |
| jgriffith | oslo-config-generator --namespace keystonemiddleware.auth_token >> etc/cinder/cinder.conf.sample | 17:42 |
| jgriffith | etc | 17:42 |
| jungleboyj | Ok, that is similar to what I was just looking at doing. | 17:43 |
| jungleboyj | jgriffith: If you want to push that up I will give it a try. | 17:44 |
| *** sileht has quit IRC | 17:44 | |
| *** angela-s has joined #openstack-cinder | 17:45 | |
| *** lpetrut has quit IRC | 17:45 | |
| *** annegent_ has joined #openstack-cinder | 17:46 | |
| *** jwcroppe has quit IRC | 17:48 | |
| *** jwcroppe has joined #openstack-cinder | 17:48 | |
| *** annegentle has quit IRC | 17:50 | |
| jgriffith | jungleboyj: quite frankly we should probably just stick with something like this anyway | 17:52 |
| jgriffith | jungleboyj: hybrid of the old incubator code and the new oslo.confg tool | 17:52 |
| jungleboyj | Really? | 17:52 |
| jgriffith | jungleboyj: maybe... beats the hell out of harcoding all that nonsense IMO | 17:53 |
| *** AlkaD has quit IRC | 17:53 | |
| *** e0ne has joined #openstack-cinder | 17:53 | |
| *** sileht has joined #openstack-cinder | 17:55 | |
| *** crose has quit IRC | 17:55 | |
| jungleboyj | jgriffith: Agreed. I still think we should move to using the new newer generator as others have done that. I just need to come up with a better approach than I currently have that doesn't require a big hacking check. | 17:58 |
| jungleboyj | jgriffith: Just haven't gotten back to that yet. | 17:58 |
| jgriffith | jungleboyj: yeah, so what I'm saying is use both :) | 17:58 |
| jgriffith | jungleboyj: but modify the methods in incubator a bit | 17:59 |
| jgriffith | brb | 17:59 |
| jungleboyj | Ok. | 17:59 |
| jgriffith | I'll push that change up when I get back... 5 minutes or so | 17:59 |
| jungleboyj | Cool. | 17:59 |
| jgriffith | too many interruptions | 17:59 |
| *** jwcroppe has quit IRC | 17:59 | |
| jungleboyj | Tell me about it. | 17:59 |
| *** aswadr has quit IRC | 18:01 | |
| *** e0ne is now known as e0ne_ | 18:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/176823 | 18:03 |
| *** jwcroppe has joined #openstack-cinder | 18:05 | |
| *** Yogi11 has joined #openstack-cinder | 18:05 | |
| *** e0ne_ has quit IRC | 18:06 | |
| *** david-lyle has joined #openstack-cinder | 18:07 | |
| *** Yogi1 has quit IRC | 18:07 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add external genconfig calls https://review.openstack.org/176930 | 18:09 |
| *** melwitt has joined #openstack-cinder | 18:11 | |
| *** e0ne has joined #openstack-cinder | 18:13 | |
| *** breitz1 is now known as breitz | 18:21 | |
| thingee | jgriffith, jungleboyj disappearing for a bit, bb in an hour | 18:23 |
| *** morganfainberg is now known as grebniafnagrom | 18:24 | |
| thingee | grebniafnagrom: it's not casual nick friday :) | 18:24 |
| *** grebniafnagrom is now known as morganfainberg | 18:24 | |
| thingee | unless you're going incognito | 18:24 |
| thingee | hehe | 18:24 |
| morganfainberg | someone made some "reflection" joke | 18:25 |
| *** bswartz has quit IRC | 18:26 | |
| *** bswartz has joined #openstack-cinder | 18:27 | |
| *** alexpilotti has joined #openstack-cinder | 18:27 | |
| *** e0ne is now known as e0ne_ | 18:28 | |
| *** e0ne_ is now known as e0ne | 18:29 | |
| ganso_ | Hello guys. I am having difficulties finding the CLI command for volume retype... could anyone help me? | 18:33 |
| mtanino | jungleboyj: hi, | 18:34 |
| mtanino | jungleboyj: May I ask a question related to this fix? https://review.openstack.org/#/c/175349/ | 18:34 |
| jungleboyj | mtanino: Sure. | 18:35 |
| mtanino | jungleboyj: so the fix is backporting partial patch from Kilo tree. In this case, we also need to add cherry-pick comment? | 18:36 |
| *** dalgaaf has quit IRC | 18:37 | |
| jungleboyj | Looking. | 18:37 |
| mtanino | thank you :) | 18:38 |
| mtanino | partialy or fully is not matter for adidng cherry-pick comment? | 18:39 |
| *** e0ne is now known as e0ne_ | 18:40 | |
| jungleboyj | Looking again, I probably made that comment too quickly. | 18:41 |
| mtanino | sure. | 18:41 |
| mtanino | just a confirmation for the policy. | 18:41 |
| jungleboyj | Yeah, this is ok since they couldn't bring the whole thing over. | 18:43 |
| *** e0ne_ is now known as e0ne | 18:44 | |
| mtanino | yes. whole patch can't backport to Juno/stable, this bug need just one method to back-port. | 18:44 |
| jungleboyj | Updated. | 18:44 |
| jungleboyj | mtanino: Thanks for pointing that out. | 18:45 |
| *** alexpilotti_ has joined #openstack-cinder | 18:46 | |
| mtanino | jungleboyj: thank you for updating :) | 18:47 |
| jungleboyj | mtanino: Welcome. | 18:47 |
| *** alexpilotti has quit IRC | 18:49 | |
| *** alexpilotti_ is now known as alexpilotti | 18:49 | |
| jungleboyj | jgriffith: Patch was what I was thinking of. Just need to change the namespace stuff to oslo_db instead of oslo.db, etc. | 18:49 |
| jungleboyj | jgriffith: Just in case we keep using this approach for a while. | 18:49 |
| *** esker has quit IRC | 18:50 | |
| *** xyang has quit IRC | 18:51 | |
| *** AlkaD has joined #openstack-cinder | 18:54 | |
| *** jwcroppe has quit IRC | 18:54 | |
| jungleboyj | It is a known issue that trying to attach using 'auto' will some times result in an already used mount point being attempted. Right? | 18:55 |
| *** jwcroppe_ has joined #openstack-cinder | 18:58 | |
| *** kaisers has quit IRC | 19:05 | |
| *** kaisers has joined #openstack-cinder | 19:06 | |
| *** patrickeast has quit IRC | 19:10 | |
| *** _cjones_ has quit IRC | 19:12 | |
| *** rushiagr is now known as rushiagr_away | 19:14 | |
| *** jgravel has left #openstack-cinder | 19:18 | |
| *** _cjones_ has joined #openstack-cinder | 19:19 | |
| *** esker has joined #openstack-cinder | 19:29 | |
| *** bill_az has quit IRC | 19:32 | |
| *** liusheng has quit IRC | 19:34 | |
| *** ociuhandu has quit IRC | 19:38 | |
| *** annegent_ has quit IRC | 19:40 | |
| *** _cjones_ has quit IRC | 19:40 | |
| *** annegentle has joined #openstack-cinder | 19:41 | |
| *** lpetrut has joined #openstack-cinder | 19:41 | |
| *** ronis has quit IRC | 19:42 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Sync oslo service module https://review.openstack.org/176802 | 19:47 |
| jungleboyj | jgriffith: eharney ^^^ I fixed up the commit message since I am generally the one using those to make updates. :-) | 19:49 |
| *** dannywilson has joined #openstack-cinder | 19:49 | |
| *** dannywilson has quit IRC | 19:49 | |
| *** dannywilson has joined #openstack-cinder | 19:50 | |
| *** annashen has joined #openstack-cinder | 19:58 | |
| *** mtanino has quit IRC | 19:58 | |
| *** patrickeast has joined #openstack-cinder | 19:59 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add external genconfig calls https://review.openstack.org/176930 | 19:59 |
| jgriffith | jungleboyj: thanks! | 19:59 |
| jgriffith | jungleboyj: long meeting then lunch | 20:00 |
| jungleboyj | Welcome. I have the process down and figured you wouldn't mind me doing the busy work. | 20:00 |
| ameade | DuncanT: had to rebase since the unittests moved, https://review.openstack.org/#/c/170990/ | 20:00 |
| jgriffith | jungleboyj: do we have that documented anywhere? | 20:00 |
| jgriffith | we == openstack | 20:00 |
| jungleboyj | jgriffith: Good question. I just always go back to the last one I did and use that as my template. | 20:01 |
| jgriffith | jungleboyj: hehe | 20:01 |
| jgriffith | sounds familiar | 20:01 |
| e0ne | jgriffith, jungleboyj: is it http://specs.openstack.org/openstack/oslo-specs/specs/policy/incubator.html#syncing-code-from-incubator that you ask? | 20:03 |
| jgriffith | e0ne: Perfect | 20:04 |
| *** dencaval has quit IRC | 20:04 | |
| jungleboyj | jgriffith: Yeah, that looks like the place to update. | 20:04 |
| *** annashen has quit IRC | 20:04 | |
| jgriffith | except it doesn't have the req for the commit messages | 20:04 |
| jgriffith | jungleboyj: e0ne syncing is easy, I do that just fine on my own thanks | 20:05 |
| jungleboyj | :-) | 20:05 |
| jgriffith | jungleboyj: e0ne the "requirement" for the commit/change ID's is the part that I got -1'd on | 20:05 |
| jgriffith | that I don't know that we've ever really documented | 20:05 |
| jgriffith | or why we don't automate that in the tool perahps | 20:06 |
| jgriffith | perhaps | 20:06 |
| *** annashen has joined #openstack-cinder | 20:06 | |
| e0ne | jgriffith: good question:) | 20:06 |
| jungleboyj | Automating it has been discussed and deemed too hard. | 20:06 |
| jungleboyj | Maybe that isn't the right answer though. | 20:06 |
| jungleboyj | +2 on the genconfig change. | 20:06 |
| jungleboyj | I need to drop off for a while. | 20:07 |
| *** Apoorva has quit IRC | 20:08 | |
| *** _cjones_ has joined #openstack-cinder | 20:09 | |
| *** annashen has quit IRC | 20:12 | |
| *** mtanino has joined #openstack-cinder | 20:12 | |
| vilobhmm12 | Unable to update stats, RBDDriver -1.1.0 driver is uninitialized mostly we get this when the volume_driver is not specified in cinder.conf right i am seeing this consistently on 2014.1.3 tag any suggestions ? | 20:13 |
| *** jungleboyj has quit IRC | 20:13 | |
| vilobhmm12 | i think i have made the needed changes in cinder.conf | 20:13 |
| vilobhmm12 | to enable it | 20:13 |
| e0ne | jgriffith: looks like this requirement is cinder only | 20:14 |
| e0ne | jgriffith: i found request for heat https://review.openstack.org/#/c/168224/ | 20:14 |
| kmartin | jgriffith, asselin checked and saw that nova is running a live migration test (tempest.api.compute.admin.test_live_migration.LiveBlockMigrationTestJSON.test_live_block_migration) as part of dsvm multinode full see http://logs.openstack.org/17/168317/9/check/check-tempest-dsvm-multinode-full/7932cfe/logs/ | 20:14 |
| e0ne | and for glance: | 20:15 |
| e0ne | https://review.openstack.org/#/c/161907/ | 20:15 |
| kmartin | jgriffith, cinder is currently skipping the test | 20:15 |
| jgriffith | kmartin: because cinder doesn't run "multi-node" | 20:17 |
| jgriffith | kmartin: if it is, you or asselin should enable the test :) | 20:17 |
| jgriffith | kmartin: btw, tested again last night, works fine for LVM and SolidFire | 20:17 |
| jgriffith | kmartin: dind't hear if xyang1 had a chance to try it or not | 20:18 |
| asselin | jgriffith, it's easier said than done as it requires multi-node setup. | 20:18 |
| kmartin | jgriffith, yep, we are aware of the problem with 3PAR and working on fixing | 20:18 |
| xyang1 | jgriffith: Live migration? Not yet | 20:18 |
| asselin | jgriffith, but we'll get to it | 20:18 |
| jgriffith | asselin: I'm not sure if my text is garbled or what... | 20:18 |
| jgriffith | asselin: but yes, that's correct, and that's what I've been saying since yesterday morning :) | 20:19 |
| jgriffith | asselin: I wasn't saying you should enable multinode for Cinder today either :) | 20:19 |
| jgriffith | asselin: I was just telling kmartin that that's why it's "skipped" for Cinder | 20:19 |
| jgriffith | asselin: and that if I was out of the loop and the multi-node devstack for Cinder was in place somewhere then by all means we shoudl enable it. That's all | 20:19 |
| jgriffith | asselin: sorry for the confusion there | 20:20 |
| asselin | jgriffith, ok...I missed yesterday's conversation.... | 20:20 |
| jgriffith | asselin: No worries | 20:20 |
| *** Apoorva has joined #openstack-cinder | 20:20 | |
| jgriffith | I just don't want confusion/fud from statements like: 20:22 < kmartin> jgriffith, cinder is currently skipping the test | 20:20 |
| asselin | jgriffith, well we can enable it for cinder easily enough in upstream. probably 1 line needs to be added | 20:21 |
| jgriffith | asselin: right, but until we do the node config for dsvm-mutli it's irrelevant :) | 20:21 |
| asselin | ok..dsvm full is skipping the test. dsvm multinode full runs it | 20:21 |
| jgriffith | asselin: :) | 20:21 |
| jgriffith | asselin: so that's perfect | 20:21 |
| jgriffith | as it should be | 20:22 |
| *** AlkaD has quit IRC | 20:23 | |
| smcginnis | patrickeast: ping | 20:24 |
| patrickeast | smcginnis: hola | 20:24 |
| smcginnis | patrickeast: Hey! Did you see my response here: https://review.openstack.org/#/c/173368/ | 20:24 |
| smcginnis | patrickeast: Wondering if you are cool with that or if you'd prefer I change some of those back. | 20:24 |
| smcginnis | patrickeast: And earlier reviewer wanted me to change the two step process. | 20:25 |
| smcginnis | patrickeast: I was picking my battles. :) | 20:25 |
| patrickeast | smcginnis: ah yea, sry, forgot to change my review, i’m fine with it | 20:25 |
| smcginnis | patrickeast: OK, cool. | 20:25 |
| patrickeast | smcginnis: i figured i would at least point it out, but its not like it breaks anything | 20:25 |
| smcginnis | patrickeast: Good to know of others opinions though. I will use that as backing next time. | 20:25 |
| smcginnis | patrickeast: Yeah, my thoughts too. I appreciate it. | 20:26 |
| smcginnis | patrickeast: Thanks! | 20:26 |
| zigo | jgriffith: Running the command that you wrote in the tox.ini worked for me. | 20:26 |
| *** e0ne is now known as e0ne_ | 20:26 | |
| zigo | jgriffith: I mean, the keystone_authtoken was right. | 20:26 |
| zigo | jgriffith: I'll now run a full install and tempest run with it... | 20:27 |
| *** e0ne has joined #openstack-cinder | 20:27 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR don't log version numbers every stats update https://review.openstack.org/176905 | 20:31 |
| *** dustins_ has quit IRC | 20:40 | |
| thingee | jgriffith: hey back. what's new with genconfig? | 20:42 |
| *** annashen has joined #openstack-cinder | 20:42 | |
| thingee | ah found https://review.openstack.org/#/c/176930/ | 20:45 |
| *** akerr has quit IRC | 20:47 | |
| *** e0ne has quit IRC | 20:47 | |
| *** Lee1092 has quit IRC | 20:51 | |
| *** Svedrin has joined #openstack-cinder | 20:57 | |
| thingee | jgriffith: thanks https://review.openstack.org/#/c/176986/ | 21:07 |
| *** lpetrut has quit IRC | 21:07 | |
| *** bswartz has quit IRC | 21:08 | |
| jgriffith | thingee: awesome | 21:14 |
| jgriffith | zigo: to the problem with that was it only did the external bits. | 21:15 |
| *** annegentle has quit IRC | 21:15 | |
| jgriffith | zigo: should be fine for what was "missing" and what you needed. | 21:15 |
| hemna | http://www.engadget.com/2015/04/23/amazon-q1-2015/ | 21:15 |
| jgriffith | zigo: I've updatd that patch now to just do all of that in the gen-config script | 21:15 |
| *** thangp has quit IRC | 21:15 | |
| zigo | jgriffith: I thought you abandonned it? | 21:16 |
| jgriffith | hemna: yeah, read that... always wondered how much they "really" made | 21:16 |
| hemna | yah interesting | 21:16 |
| jgriffith | zigo: I did.. but added: https://review.openstack.org/#/c/176930/ | 21:16 |
| jgriffith | hemna: Nice to know a big chunk of that comes from me :) | 21:17 |
| jgriffith | Like 10 bucks!!! | 21:17 |
| *** Yogi11 has quit IRC | 21:17 | |
| zigo | jgriffith: That's the only bits missing? | 21:17 |
| hemna | lol | 21:17 |
| jgriffith | zigo: yep, that should be it | 21:17 |
| zigo | Interesting... | 21:17 |
| jgriffith | zigo: unless we're using other libs that weren't included | 21:18 |
| jgriffith | zigo: which is certainly possible :) | 21:18 |
| openstackgerrit | Julie Gravel proposed openstack/cinder: Validate volume type extra specs https://review.openstack.org/161480 | 21:18 |
| *** jgravel has joined #openstack-cinder | 21:18 | |
| zigo | jgriffith: I have my own tooling to do the requirements.txt -> debian/control Depends:, which I use to for double checking, so I'm quite sure I have it correct ! :) | 21:19 |
| zigo | jgriffith: Thanks a lot for you help btw. | 21:19 |
| jgriffith | zigo: absolutely | 21:20 |
| *** annegentle has joined #openstack-cinder | 21:20 | |
| jgriffith | ls | 21:21 |
| *** timcl has quit IRC | 21:22 | |
| *** cbader has quit IRC | 21:26 | |
| *** eharney has quit IRC | 21:28 | |
| *** marcusvrn1 has quit IRC | 21:29 | |
| *** marcusvrn has joined #openstack-cinder | 21:29 | |
| thingee | hemna: https://review.openstack.org/176996 | 21:36 |
| thingee | hemna: updated os-brick jobs...that explains why pypi publish didn't work | 21:36 |
| hemna | oh! | 21:36 |
| hemna | huh | 21:36 |
| thingee | missed that when I was going through the docs verifying things | 21:36 |
| hemna | ah ok. | 21:36 |
| hemna | dang | 21:36 |
| hemna | sorry about that. I just followed the docs at the time | 21:37 |
| thingee | hemna: yeah I think they were updated after you did things | 21:37 |
| thingee | also now we have docs being gated | 21:37 |
| thingee | :) | 21:37 |
| hemna | :) awesome | 21:37 |
| thingee | hemna: someone from infra will manually trigger the publish so we don't have to retag | 21:38 |
| hemna | ok cool | 21:38 |
| *** akerr has joined #openstack-cinder | 21:43 | |
| *** annegentle has quit IRC | 21:44 | |
| *** akerr has quit IRC | 21:48 | |
| hemna | thingee, jenkins failed | 21:55 |
| *** jungleboyj has joined #openstack-cinder | 21:55 | |
| jungleboyj | Anyone have a concern with me merging this: https://review.openstack.org/#/c/176802 | 21:55 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Sync oslo service module https://review.openstack.org/176802 | 21:56 |
| hemna | needed a rebase | 21:56 |
| jungleboyj | Ah, thank you. | 21:59 |
| jungleboyj | That should probably also get into RC2. Do you agree thingee ? | 22:00 |
| thingee | hemna: yeah I think I fixed it...just reup'd | 22:00 |
| thingee | jungleboyj: why? | 22:01 |
| hemna | jungleboyj, dunno about putting that into RC2 | 22:02 |
| jungleboyj | Well, because it fixes namespace deprecation warnings and is the change that should have been pulled in and backported for RC. | 22:02 |
| hemna | that's not really needed unless something is broken no ? | 22:02 |
| *** rushil has quit IRC | 22:02 | |
| *** Guest47750 has joined #openstack-cinder | 22:03 | |
| thingee | unless things are really broken, I don't want to hold things up anymore if possible | 22:03 |
| hemna | jungleboyj, are deprecation warnings the worst of it ? | 22:04 |
| hemna | to me, that's a 'meh'. | 22:04 |
| jungleboyj | hemna: Yes. | 22:04 |
| hemna | compared to the risk. | 22:04 |
| *** mriedem is now known as mriedem_away | 22:04 | |
| jungleboyj | hemna: I agree but people here, internally are grumbling at me about it. | 22:04 |
| jungleboyj | I can just fix it internally for now. | 22:04 |
| hemna | # | 22:05 |
| hemna | :P | 22:05 |
| jungleboyj | ? | 22:05 |
| hemna | there I fixed it. | 22:05 |
| hemna | just add a # | 22:05 |
| thingee | jungleboyj: need to get rid of the warnings from the version discovery patch in cinderclient. | 22:05 |
| hemna | done | 22:05 |
| hemna | next | 22:05 |
| thingee | jungleboyj: jgriffith reminded me about it yesterday | 22:05 |
| thingee | if a version is passed into the endpoint, we should avoid discovery | 22:05 |
| thingee | otherwise keystone complains | 22:05 |
| morganfainberg | thingee: this is the version discovery warning thing in logS? | 22:06 |
| thingee | morganfainberg: it's a valid warning from keystone http://paste.openstack.org/raw/155906/ | 22:07 |
| morganfainberg | ah yeah | 22:07 |
| morganfainberg | that is valid | 22:08 |
| morganfainberg | either you fix the versioned endpoint so discovery still works or you avoid it | 22:08 |
| morganfainberg | i think the former is better. | 22:08 |
| morganfainberg | but i defer to your choice on attacking that | 22:08 |
| morganfainberg | though discovery with the replaced values is weird | 22:09 |
| *** openstackstatus has quit IRC | 22:09 | |
| dannywilson | ls | 22:19 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR don't log version numbers every stats update https://review.openstack.org/176905 | 22:19 |
| *** Apoorva has quit IRC | 22:20 | |
| *** jwcroppe_ has quit IRC | 22:30 | |
| vilobhmm12 | thingee : ping | 22:31 |
| vilobhmm12 | hemna : ping | 22:31 |
| hemna | vilobhmm12, yough | 22:31 |
| vilobhmm12 | quick question when trying to create volume on ceph backend | 22:32 |
| vilobhmm12 | getting "Unable to update stats, RBDDriver -1.1.0 driver is uninitialized" | 22:32 |
| vilobhmm12 | so from my understanding | 22:32 |
| *** Apoorva has joined #openstack-cinder | 22:32 | |
| hemna | something happened at driver startup | 22:32 |
| vilobhmm12 | volume manager->rbd.py->librbd->librados | 22:32 |
| vilobhmm12 | and that talks to ceph backend | 22:32 |
| vilobhmm12 | cinder-volume is running on api node | 22:32 |
| vilobhmm12 | so during volume creation the compute should not come into picture neither qemu etc stuff | 22:33 |
| jungleboyj | hemna: Thanks for adding that comment in the code. Thought there was something I didn't know going on there. | 22:33 |
| vilobhmm12 | its simply betwn the node runing cinder volume service and the backend on which to create volume righ ? | 22:33 |
| hemna | jungleboyj, np. I figured after jdg asked about it, I should say something in the code to mention it. | 22:34 |
| jungleboyj | I had pinged you asking about it too. :-) Don't know if you saw that. | 22:34 |
| hemna | jungleboyj, ah crap, I didn't see that sorry | 22:34 |
| jungleboyj | That is ok. I was going to harass you again at some point. | 22:35 |
| hemna | :) | 22:35 |
| jungleboyj | I feel a lot better now that I know I wasn't the only one baffled. | 22:35 |
| vilobhmm12 | hemna : https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L527 ^^ | 22:35 |
| hemna | vilobhmm12, if you are getting that message above, the create_volume is never being called. | 22:36 |
| *** alexpilotti has quit IRC | 22:36 | |
| hemna | the volume manager won't call create_volume if the driver isn't marked as initialized. | 22:36 |
| hemna | or it shouldn't | 22:36 |
| hemna | :) | 22:36 |
| vilobhmm12 | and what are the reason for which the driver isn't marked as initialized ? | 22:38 |
| vilobhmm12 | can see anything obvious here https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L401 | 22:38 |
| hemna | something in the driver failed at volume manager init_host time. | 22:39 |
| hemna | misconfiguration ? | 22:39 |
| *** lcurtis has quit IRC | 22:39 | |
| hemna | vilobhmm12, https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L298-L311 | 22:39 |
| hemna | and in fact init_host has to do a bunch of stuff, before marking the driver 'initialized' | 22:40 |
| hemna | that happens here:https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L369 | 22:40 |
| hemna | vilobhmm12, that is the only time the driver gets a chance to get marked as initialized. | 22:40 |
| vilobhmm12 | sure…i did have a look at the second link | 22:40 |
| vilobhmm12 | sure | 22:40 |
| vilobhmm12 | thanks will check the conf again | 22:41 |
| vilobhmm12 | and see if i am missing something | 22:41 |
| *** nestorf has quit IRC | 22:41 | |
| hemna | if you can, restart cinder and look at the log output right before all the conf entries start to get dumped | 22:41 |
| *** nestorf has joined #openstack-cinder | 22:41 | |
| hemna | that's where you'll most likely see an exception for init_host time failures. | 22:41 |
| vilobhmm12 | oh okay | 22:42 |
| vilobhmm12 | lemme try it | 22:42 |
| *** Guest47750 has quit IRC | 22:44 | |
| vilobhmm12 | hemna : you were right it pukes here https://gist.github.com/vilobhmm/00fb9c5e0f31fa686c95 | 22:46 |
| vilobhmm12 | looks like some access permission will fix it | 22:46 |
| hemna | yup. there ya go. | 22:46 |
| vilobhmm12 | thanks | 22:46 |
| hemna | np. glad I could help! | 22:48 |
| *** erlon has quit IRC | 22:51 | |
| *** rhagarty has quit IRC | 22:51 | |
| *** rhagarty has joined #openstack-cinder | 22:52 | |
| *** angela-s has quit IRC | 22:57 | |
| *** annashen has quit IRC | 23:00 | |
| *** annashen has joined #openstack-cinder | 23:00 | |
| *** melwitt has quit IRC | 23:02 | |
| *** annegentle has joined #openstack-cinder | 23:03 | |
| *** rmesta has quit IRC | 23:05 | |
| *** annashen has quit IRC | 23:05 | |
| thingee | vilobhmm12: hi | 23:05 |
| *** hemna is now known as hemnafk | 23:07 | |
| *** annegentle has quit IRC | 23:14 | |
| vilobhmm12 | thingee : never mind got it resolved after talking to hemna…some weird accept permission was blocking me was not obvious after looking at code | 23:15 |
| vilobhmm12 | access* | 23:15 |
| *** ganso_ has quit IRC | 23:17 | |
| *** julim has quit IRC | 23:28 | |
| uberjay | exception raised from volume drivers: are there any rules or guidelines about what exceptions to raise? is any meaning derived from what exception class is used, or is it just dumped to the log? (i.e., should I be raising VolumeBackendAPIException, or does it not matter?) along the same lines, if there's a failure in create_volume (for example), should I bother logging something to the tune of "Create volume failed for reason [X]", or should I | 23:28 |
| uberjay | just log details of the underlying error condition and assume that a higher layer will make it clear what context the failure occurred in? | 23:28 |
| uberjay | I've been watching all of the logging related churn, and would like to save time and try to get it right before submitting for review | 23:30 |
| thingee | uberjay: so the manager most has some catchall exception in place. It'll take care of the logging. | 23:32 |
| thingee | uberjay: example https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L1065 | 23:32 |
| thingee | uberjay: if you look at other drivers, they'll define their own exceptions inheriting VolumeDriverException https://github.com/openstack/cinder/blob/master/cinder/exception.py#L754 | 23:33 |
| *** adurbin_ has quit IRC | 23:33 | |
| uberjay | thingee: ok, so there's really no reason to log it myself, unless there is additional diagnostic information worth recording that doesn't really fit into an exception message? | 23:34 |
| *** zhenguo has joined #openstack-cinder | 23:34 | |
| uberjay | thingee: ah, gotcha | 23:35 |
| thingee | if there are things specific to your driver that warrants additional logging you're more than welcome to add what you feel is necessary. You can find guidelines for logging here http://specs.openstack.org/openstack/openstack-specs/specs/log-guidelines.html | 23:35 |
| * thingee adds this to https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver | 23:36 | |
| uberjay | mm, yeah, I've been watching the recent logging-related commits and that's what got me down this path. i'd like to make sure the driver is doing the right thing and save everyone's reviewing time :) | 23:36 |
| uberjay | thanks! | 23:37 |
| thingee | uberjay: you're welcome :) | 23:37 |
| *** ebalduf has quit IRC | 23:41 | |
| *** vilobhmm12 has quit IRC | 23:42 | |
| *** vilobhmm1 has joined #openstack-cinder | 23:42 | |
| *** vilobhmm11 has joined #openstack-cinder | 23:46 | |
| *** vilobhmm1 has quit IRC | 23:47 | |
| *** garthb_ has quit IRC | 23:51 | |
| *** garthb has quit IRC | 23:51 | |
| *** bswartz has joined #openstack-cinder | 23:54 | |
| *** atan8 has joined #openstack-cinder | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!