| *** haomaiwa_ has quit IRC | 00:01 | |
| *** haomaiwang has joined #openstack-cinder | 00:01 | |
| *** gouthamr has joined #openstack-cinder | 00:03 | |
| *** cknight has quit IRC | 00:04 | |
| *** gouthamr_ has joined #openstack-cinder | 00:05 | |
| *** mylu has joined #openstack-cinder | 00:05 | |
| *** thurloat_isgone is now known as thurloat | 00:08 | |
| *** gouthamr has quit IRC | 00:08 | |
| *** thurloat is now known as thurloat_isgone | 00:09 | |
| *** thurloat_isgone is now known as thurloat | 00:15 | |
| *** hemna is now known as hemnafk | 00:16 | |
| openstackgerrit | Alex Meade proposed openstack/cinder-specs: User facing error messages https://review.openstack.org/274958 | 00:17 |
|---|---|---|
| *** sb- is now known as sb | 00:17 | |
| aorourke | jgriffith, I will submit the additions for now. If things need to be changed, by all means. Don't want to create any chaos :) | 00:18 |
| *** sid14726 has quit IRC | 00:18 | |
| *** cknight has joined #openstack-cinder | 00:20 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: Replication v2.1 https://review.openstack.org/275797 | 00:24 |
| *** mylu has quit IRC | 00:26 | |
| *** bardia has quit IRC | 00:27 | |
| *** alonma has joined #openstack-cinder | 00:27 | |
| jgriffith | aorourke: sounds good... I think it's mucho less chaotic for you to just submit on that patch at this stage :) | 00:27 |
| *** xyang1_ has joined #openstack-cinder | 00:28 | |
| *** xyang1 has quit IRC | 00:28 | |
| *** xyang1_ is now known as xyang1 | 00:28 | |
| *** mylu has joined #openstack-cinder | 00:29 | |
| aorourke | agreed | 00:30 |
| *** alonma has quit IRC | 00:32 | |
| *** alonma has joined #openstack-cinder | 00:36 | |
| *** sgotliv has quit IRC | 00:37 | |
| *** cknight has quit IRC | 00:38 | |
| *** cknight has joined #openstack-cinder | 00:39 | |
| *** mylu has quit IRC | 00:40 | |
| *** mylu has joined #openstack-cinder | 00:40 | |
| *** alonma has quit IRC | 00:40 | |
| *** bardia has joined #openstack-cinder | 00:41 | |
| *** martyturner has joined #openstack-cinder | 00:43 | |
| *** alonma has joined #openstack-cinder | 00:45 | |
| *** tonyb has joined #openstack-cinder | 00:45 | |
| tonyb | Any chance I can get stable-cores to look at: https://review.openstack.org/#/c/265182 | 00:45 |
| *** mylu has quit IRC | 00:46 | |
| tonyb | It's a g-r sync that needs to be released to unblock glance_store | 00:46 |
| *** gouthamr_ has quit IRC | 00:46 | |
| *** merooney has joined #openstack-cinder | 00:47 | |
| *** mylu has joined #openstack-cinder | 00:47 | |
| *** mylu has quit IRC | 00:49 | |
| *** mylu has joined #openstack-cinder | 00:49 | |
| *** alonma has quit IRC | 00:50 | |
| *** mylu has quit IRC | 00:52 | |
| *** boichev has quit IRC | 00:55 | |
| *** mylu has joined #openstack-cinder | 00:55 | |
| *** angela-s has quit IRC | 00:56 | |
| *** mylu has quit IRC | 00:59 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** mylu has joined #openstack-cinder | 01:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 01:02 | |
| *** mylu has quit IRC | 01:03 | |
| openstackgerrit | Michael Rowden proposed openstack/cinder: Correcting thin provisioning behavior https://review.openstack.org/275408 | 01:03 |
| *** mylu has joined #openstack-cinder | 01:06 | |
| *** merooney has quit IRC | 01:06 | |
| *** esker has joined #openstack-cinder | 01:08 | |
| *** mylu has quit IRC | 01:08 | |
| *** mylu has joined #openstack-cinder | 01:08 | |
| *** merooney has joined #openstack-cinder | 01:09 | |
| *** garthb has quit IRC | 01:12 | |
| *** mylu has quit IRC | 01:13 | |
| *** mylu has joined #openstack-cinder | 01:15 | |
| *** harlowja has quit IRC | 01:18 | |
| *** mylu has quit IRC | 01:18 | |
| *** chlong has joined #openstack-cinder | 01:18 | |
| *** diablo_rojo has joined #openstack-cinder | 01:23 | |
| *** martyturner has quit IRC | 01:23 | |
| *** wN has quit IRC | 01:24 | |
| *** jungleboyj has joined #openstack-cinder | 01:27 | |
| smcginnis | tonyb: Done | 01:27 |
| *** diablo_rojo has quit IRC | 01:27 | |
| *** esker has quit IRC | 01:27 | |
| *** wN has joined #openstack-cinder | 01:30 | |
| *** wN has joined #openstack-cinder | 01:30 | |
| *** jasondotstar has quit IRC | 01:35 | |
| *** cknight has quit IRC | 01:35 | |
| *** gouthamr has joined #openstack-cinder | 01:36 | |
| *** sid14726 has joined #openstack-cinder | 01:36 | |
| *** dims_ has joined #openstack-cinder | 01:37 | |
| *** jasondotstar has joined #openstack-cinder | 01:37 | |
| *** dims has quit IRC | 01:38 | |
| *** dims_ has quit IRC | 01:41 | |
| *** apoorvad has quit IRC | 01:41 | |
| *** leeantho has quit IRC | 01:43 | |
| *** IlyaG has quit IRC | 01:43 | |
| *** alonma has joined #openstack-cinder | 01:45 | |
| *** cknight has joined #openstack-cinder | 01:48 | |
| *** dims has joined #openstack-cinder | 01:49 | |
| *** alonma has quit IRC | 01:50 | |
| *** mylu has joined #openstack-cinder | 01:51 | |
| *** alonma has joined #openstack-cinder | 01:52 | |
| *** mylu has quit IRC | 01:54 | |
| *** alonma has quit IRC | 01:56 | |
| openstackgerrit | Merged openstack/python-cinderclient: Return wrapper classes with request_ids attribute https://review.openstack.org/257180 | 01:58 |
| *** merooney has quit IRC | 01:59 | |
| *** akshai has quit IRC | 01:59 | |
| *** haomaiwa_ has quit IRC | 02:01 | |
| *** haomaiwang has joined #openstack-cinder | 02:03 | |
| *** edmondsw has quit IRC | 02:03 | |
| openstackgerrit | Merged openstack/cinder: EMC VMAX - Method not being called for V3 https://review.openstack.org/252066 | 02:03 |
| *** mtanino has quit IRC | 02:04 | |
| *** alonma has joined #openstack-cinder | 02:05 | |
| *** mudassirlatif_ has joined #openstack-cinder | 02:06 | |
| *** mragupat has joined #openstack-cinder | 02:08 | |
| *** mragupat has quit IRC | 02:08 | |
| *** mragupat has joined #openstack-cinder | 02:08 | |
| *** cknight1 has joined #openstack-cinder | 02:08 | |
| *** alonma has quit IRC | 02:10 | |
| *** alonma has joined #openstack-cinder | 02:11 | |
| *** jungleboyj has quit IRC | 02:11 | |
| *** ByPasS_ has joined #openstack-cinder | 02:14 | |
| *** cknight has quit IRC | 02:15 | |
| *** mudassirlatif has quit IRC | 02:15 | |
| *** openstackgerrit has quit IRC | 02:15 | |
| *** ByPasS has quit IRC | 02:15 | |
| *** ByPasS_ is now known as ByPasS | 02:15 | |
| *** mudassirlatif_ is now known as mudassirlatif | 02:15 | |
| *** chlong has quit IRC | 02:15 | |
| *** alonma has quit IRC | 02:16 | |
| *** alonma has joined #openstack-cinder | 02:18 | |
| *** alonma has quit IRC | 02:23 | |
| *** mragupat has quit IRC | 02:24 | |
| *** cknight1 has quit IRC | 02:24 | |
| *** openstackgerrit has joined #openstack-cinder | 02:24 | |
| *** xyang1 has quit IRC | 02:29 | |
| *** chlong has joined #openstack-cinder | 02:29 | |
| *** laughterwym has joined #openstack-cinder | 02:33 | |
| *** tpsilva has quit IRC | 02:36 | |
| *** arch-nemesis has joined #openstack-cinder | 02:39 | |
| *** Poornima has joined #openstack-cinder | 02:41 | |
| *** sid14726 has quit IRC | 02:47 | |
| *** mudassirlatif has quit IRC | 02:48 | |
| *** laughterwym has quit IRC | 02:48 | |
| *** laughterwym has joined #openstack-cinder | 02:49 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:01 | |
| *** bardia has quit IRC | 03:03 | |
| *** Poornima has quit IRC | 03:05 | |
| *** Poornima has joined #openstack-cinder | 03:06 | |
| *** mylu has joined #openstack-cinder | 03:06 | |
| *** arch-nemesis has quit IRC | 03:07 | |
| *** links has joined #openstack-cinder | 03:10 | |
| *** Poornima has quit IRC | 03:13 | |
| *** bill_az_ has quit IRC | 03:15 | |
| *** sid14726 has joined #openstack-cinder | 03:15 | |
| tonyb | smcginnis: Thanks. | 03:18 |
| *** alonma has joined #openstack-cinder | 03:21 | |
| openstackgerrit | Merged openstack/python-cinderclient: Fix some flake8 violations https://review.openstack.org/277925 | 03:24 |
| *** alonma has quit IRC | 03:25 | |
| *** arch-nemesis has joined #openstack-cinder | 03:27 | |
| *** Poornima has joined #openstack-cinder | 03:27 | |
| *** esker has joined #openstack-cinder | 03:32 | |
| *** Poornima has quit IRC | 03:32 | |
| *** mylu has quit IRC | 03:34 | |
| *** mylu has joined #openstack-cinder | 03:35 | |
| *** esker has quit IRC | 03:53 | |
| *** laughterwym has quit IRC | 03:54 | |
| *** gouthamr has quit IRC | 03:58 | |
| *** esker has joined #openstack-cinder | 03:58 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:01 | |
| *** esker has quit IRC | 04:03 | |
| *** laughterwym has joined #openstack-cinder | 04:04 | |
| *** funeutron has joined #openstack-cinder | 04:06 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Migration: take the direct path for the source volume https://review.openstack.org/250220 | 04:06 |
| *** arch-nemesis has quit IRC | 04:08 | |
| *** laughterwym has quit IRC | 04:12 | |
| *** boris-42 has quit IRC | 04:14 | |
| *** laughterwym has joined #openstack-cinder | 04:20 | |
| *** mragupat has joined #openstack-cinder | 04:21 | |
| *** alonma has joined #openstack-cinder | 04:22 | |
| *** funeutron_ has joined #openstack-cinder | 04:22 | |
| *** funeutron has quit IRC | 04:24 | |
| *** laughterwym has quit IRC | 04:25 | |
| *** alonma has quit IRC | 04:26 | |
| *** sid14726 has quit IRC | 04:27 | |
| *** chhavi has joined #openstack-cinder | 04:28 | |
| *** jgriffith is now known as jgriffith_away | 04:31 | |
| *** mylu has quit IRC | 04:34 | |
| *** mylu has joined #openstack-cinder | 04:38 | |
| *** salv-orl_ has joined #openstack-cinder | 04:41 | |
| *** lprice has joined #openstack-cinder | 04:42 | |
| *** dims has quit IRC | 04:43 | |
| *** salv-orlando has quit IRC | 04:44 | |
| *** isaacb has quit IRC | 04:50 | |
| *** cknight has joined #openstack-cinder | 04:58 | |
| *** pradipm has joined #openstack-cinder | 04:59 | |
| *** julim has joined #openstack-cinder | 05:00 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 05:01 | |
| *** markvoelker has quit IRC | 05:03 | |
| *** sid14726 has joined #openstack-cinder | 05:04 | |
| *** cknight1 has joined #openstack-cinder | 05:05 | |
| *** jamielennox is now known as jamielennox|away | 05:06 | |
| *** cknight has quit IRC | 05:08 | |
| *** funeutron_ has quit IRC | 05:14 | |
| *** dave-mcc_ has quit IRC | 05:15 | |
| *** shyama has joined #openstack-cinder | 05:16 | |
| jgriffith_away | ./cinder/db/sqlalchemy/migrate_repo/versions/064_add_replication_info_to_service.py:1:1: H102 Apache 2.0 license header not found | 05:21 |
| jgriffith_away | ^^ that's a thing now? | 05:21 |
| *** jgriffith_away is now known as jgriffith | 05:21 | |
| *** alonma has joined #openstack-cinder | 05:22 | |
| *** mylu has quit IRC | 05:24 | |
| *** shout28 has quit IRC | 05:25 | |
| *** alonma has quit IRC | 05:26 | |
| *** mylu has joined #openstack-cinder | 05:27 | |
| *** alonma has joined #openstack-cinder | 05:28 | |
| *** garthb has joined #openstack-cinder | 05:28 | |
| *** jamielennox|away is now known as jamielennox | 05:31 | |
| *** alonma has quit IRC | 05:32 | |
| *** sid14726 has quit IRC | 05:39 | |
| *** lpetrut has joined #openstack-cinder | 05:39 | |
| *** sid14726 has joined #openstack-cinder | 05:49 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 https://review.openstack.org/275797 | 05:50 |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257187 | 05:56 |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257195 | 05:57 |
| *** vgridnev has joined #openstack-cinder | 05:58 | |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257199 | 05:58 |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add release notes for return-request-id-to-caller https://review.openstack.org/273930 | 05:59 |
| *** haomaiwa_ has quit IRC | 06:01 | |
| *** cknight1 has quit IRC | 06:01 | |
| *** 18VAACNL5 has joined #openstack-cinder | 06:01 | |
| *** ChubYann has quit IRC | 06:03 | |
| *** lpetrut has quit IRC | 06:03 | |
| *** markvoelker has joined #openstack-cinder | 06:04 | |
| openstackgerrit | Merged openstack/os-brick: Updated from global requirements https://review.openstack.org/278702 | 06:09 |
| openstackgerrit | Merged openstack/cinder: Correct opt type for nexenta_chunksize/blocksize https://review.openstack.org/269880 | 06:14 |
| openstackgerrit | Merged openstack/cinder: Remove access_mode 'rw' setting in drivers https://review.openstack.org/265443 | 06:14 |
| *** lpetrut has joined #openstack-cinder | 06:19 | |
| *** abhi has quit IRC | 06:21 | |
| *** sid14726 has quit IRC | 06:21 | |
| *** abhi has joined #openstack-cinder | 06:23 | |
| openstackgerrit | Merged openstack/cinder: Cleanup in backup reset status https://review.openstack.org/246748 | 06:25 |
| openstackgerrit | Merged openstack/cinder: Replace exit() by sys.exit() https://review.openstack.org/271966 | 06:25 |
| *** laughterwym has joined #openstack-cinder | 06:26 | |
| *** lprice1 has joined #openstack-cinder | 06:27 | |
| *** alonma has joined #openstack-cinder | 06:28 | |
| *** lprice has quit IRC | 06:29 | |
| *** sid14726 has joined #openstack-cinder | 06:30 | |
| *** alonma has quit IRC | 06:32 | |
| *** bkumar has joined #openstack-cinder | 06:34 | |
| *** markvoelker has quit IRC | 06:34 | |
| openstackgerrit | Merged openstack/cinder: Return updated volume type after updating https://review.openstack.org/273317 | 06:34 |
| openstackgerrit | Merged openstack/cinder: Remove 'external=True' in Huawei driver https://review.openstack.org/271597 | 06:35 |
| openstackgerrit | Merged openstack/cinder: VMware: optimize in get_cluster_refs https://review.openstack.org/261715 | 06:35 |
| openstackgerrit | Merged openstack/cinder: Storwize: Implement v2 replication https://review.openstack.org/237387 | 06:36 |
| *** liverpooler has quit IRC | 06:37 | |
| openstackgerrit | Merged openstack/cinder: XtremIO add support for create CG from CG src https://review.openstack.org/245548 | 06:37 |
| *** cfriesen has quit IRC | 06:39 | |
| *** vgridnev has quit IRC | 06:43 | |
| *** alonma has joined #openstack-cinder | 06:44 | |
| *** alonma has quit IRC | 06:48 | |
| *** bardia has joined #openstack-cinder | 06:52 | |
| *** lpetrut has quit IRC | 06:53 | |
| *** sheel has joined #openstack-cinder | 06:53 | |
| *** vgridnev has joined #openstack-cinder | 06:55 | |
| *** laughterwym has quit IRC | 06:59 | |
| *** 18VAACNL5 has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| *** sid14726 has quit IRC | 07:05 | |
| *** vgridnev has quit IRC | 07:10 | |
| *** baojg has joined #openstack-cinder | 07:13 | |
| *** sid14726 has joined #openstack-cinder | 07:15 | |
| *** vgridnev has joined #openstack-cinder | 07:18 | |
| *** arecknag has joined #openstack-cinder | 07:22 | |
| *** baojg has quit IRC | 07:35 | |
| *** baojg has joined #openstack-cinder | 07:35 | |
| *** sid14726 has quit IRC | 07:36 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/277228 | 07:38 |
| *** alonma has joined #openstack-cinder | 07:39 | |
| *** baojg has quit IRC | 07:42 | |
| *** bardia has quit IRC | 07:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/266163 | 07:44 |
| *** alonma has quit IRC | 07:44 | |
| *** sid14726 has joined #openstack-cinder | 07:45 | |
| *** alonma has joined #openstack-cinder | 07:48 | |
| *** sgotliv has joined #openstack-cinder | 07:49 | |
| *** alonma has quit IRC | 07:52 | |
| *** ndipanov has quit IRC | 07:55 | |
| *** baojg has joined #openstack-cinder | 07:55 | |
| *** nkrinner has joined #openstack-cinder | 07:59 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-cinder | 08:01 | |
| *** vgridnev has quit IRC | 08:06 | |
| *** sinese_ has joined #openstack-cinder | 08:10 | |
| *** mylu has quit IRC | 08:12 | |
| *** sid14726 has quit IRC | 08:13 | |
| *** garthb has quit IRC | 08:15 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: Support backup import on another Storage database https://review.openstack.org/153969 | 08:24 |
| *** nkrinner has quit IRC | 08:26 | |
| *** sid14726 has joined #openstack-cinder | 08:28 | |
| *** nkrinner has joined #openstack-cinder | 08:32 | |
| *** liverpooler has joined #openstack-cinder | 08:34 | |
| *** sinese_ has quit IRC | 08:34 | |
| *** rcernin has joined #openstack-cinder | 08:35 | |
| *** isaacb has joined #openstack-cinder | 08:36 | |
| *** mragupat has quit IRC | 08:40 | |
| *** e0ne has joined #openstack-cinder | 08:42 | |
| *** mylu has joined #openstack-cinder | 08:43 | |
| *** hemna_ has joined #openstack-cinder | 08:46 | |
| *** bkumar4 has joined #openstack-cinder | 08:46 | |
| *** openstackgerrit has quit IRC | 08:47 | |
| *** mylu has quit IRC | 08:47 | |
| *** openstackgerrit_ has joined #openstack-cinder | 08:47 | |
| *** openstackgerrit_ is now known as openstackgerrit | 08:48 | |
| *** alonma has joined #openstack-cinder | 08:48 | |
| *** liverpooler has quit IRC | 08:51 | |
| *** ildikov has quit IRC | 08:51 | |
| *** liverpooler has joined #openstack-cinder | 08:51 | |
| *** nkrinner has quit IRC | 08:52 | |
| *** bkumar has quit IRC | 08:52 | |
| *** shyama has quit IRC | 08:52 | |
| *** hemnafk has quit IRC | 08:52 | |
| *** alonma has quit IRC | 08:52 | |
| *** sgotliv has quit IRC | 08:52 | |
| *** shyama has joined #openstack-cinder | 08:54 | |
| *** alonma has joined #openstack-cinder | 08:54 | |
| *** slunkad has quit IRC | 08:56 | |
| *** nkrinner has joined #openstack-cinder | 08:57 | |
| *** alonma has quit IRC | 08:58 | |
| *** jordanP has joined #openstack-cinder | 09:00 | |
| *** alonma has joined #openstack-cinder | 09:00 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 09:01 | |
| *** sid14726 has quit IRC | 09:04 | |
| *** alonma has quit IRC | 09:05 | |
| *** alonma has joined #openstack-cinder | 09:06 | |
| *** slunkad has joined #openstack-cinder | 09:08 | |
| *** alonma has quit IRC | 09:11 | |
| *** jordanP has quit IRC | 09:12 | |
| *** alonma has joined #openstack-cinder | 09:12 | |
| *** mylu has joined #openstack-cinder | 09:13 | |
| *** alonma has quit IRC | 09:16 | |
| *** mylu has quit IRC | 09:18 | |
| *** alonma has joined #openstack-cinder | 09:18 | |
| *** alonma has quit IRC | 09:23 | |
| *** sid14726 has joined #openstack-cinder | 09:24 | |
| *** alonma has joined #openstack-cinder | 09:25 | |
| *** vgridnev has joined #openstack-cinder | 09:25 | |
| *** ildikov has joined #openstack-cinder | 09:26 | |
| *** mvk has joined #openstack-cinder | 09:27 | |
| *** alonma has quit IRC | 09:29 | |
| *** markvoelker has joined #openstack-cinder | 09:31 | |
| *** ildikov has quit IRC | 09:33 | |
| *** markvoelker has quit IRC | 09:36 | |
| *** Poornima has joined #openstack-cinder | 09:43 | |
| *** ociuhandu has quit IRC | 09:44 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Operations and timeout issues https://review.openstack.org/272760 | 09:45 |
| *** ildikov has joined #openstack-cinder | 09:45 | |
| *** ccesario has quit IRC | 09:50 | |
| *** andymaier has joined #openstack-cinder | 09:51 | |
| *** kaisers has joined #openstack-cinder | 09:54 | |
| *** andymaier has quit IRC | 09:56 | |
| *** baojg_ has joined #openstack-cinder | 09:58 | |
| *** baojg has quit IRC | 09:58 | |
| *** haomaiwa_ has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:01 | |
| *** ccesario has joined #openstack-cinder | 10:01 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX-Replacing deprecated API EMCGetTargetEndpoints https://review.openstack.org/244328 | 10:02 |
| *** vgridnev has quit IRC | 10:03 | |
| *** vgridnev has joined #openstack-cinder | 10:08 | |
| *** vgridnev has quit IRC | 10:09 | |
| *** vgridnev has joined #openstack-cinder | 10:11 | |
| *** vgridnev has quit IRC | 10:12 | |
| *** vgridnev has joined #openstack-cinder | 10:12 | |
| *** vgridnev has quit IRC | 10:13 | |
| *** sid14726 has quit IRC | 10:13 | |
| openstackgerrit | Edward Hope-Morley proposed openstack/cinder: Conditionally restore display_name https://review.openstack.org/278629 | 10:13 |
| *** mylu has joined #openstack-cinder | 10:14 | |
| *** vgridnev has joined #openstack-cinder | 10:15 | |
| *** ndipanov has joined #openstack-cinder | 10:16 | |
| *** alonma has joined #openstack-cinder | 10:19 | |
| *** mylu has quit IRC | 10:19 | |
| *** alonma has quit IRC | 10:19 | |
| *** alonma has joined #openstack-cinder | 10:20 | |
| *** sid14726 has joined #openstack-cinder | 10:24 | |
| *** funeutron has joined #openstack-cinder | 10:28 | |
| *** vgridnev has quit IRC | 10:34 | |
| *** tpsilva has joined #openstack-cinder | 10:36 | |
| openstackgerrit | Shilpa Jagannath proposed openstack/python-cinderclient: Allow "cinder backup-delete" to delete multiple backups in one request https://review.openstack.org/277895 | 10:38 |
| openstackgerrit | Shilpa Jagannath proposed openstack/python-cinderclient: Allow "cinder backup-delete" to delete multiple backups in one request https://review.openstack.org/277895 | 10:40 |
| *** Poornima has quit IRC | 10:41 | |
| *** salv-orlando has joined #openstack-cinder | 10:42 | |
| *** vgridnev has joined #openstack-cinder | 10:42 | |
| *** ankit_ag has joined #openstack-cinder | 10:44 | |
| openstackgerrit | Oleksii Butenko proposed openstack/python-cinderclient: Test https://review.openstack.org/278971 | 10:44 |
| *** salv-orl_ has quit IRC | 10:45 | |
| *** vgridnev has quit IRC | 10:45 | |
| *** vgridnev has joined #openstack-cinder | 10:46 | |
| *** dims has joined #openstack-cinder | 10:48 | |
| *** lpetrut has joined #openstack-cinder | 10:50 | |
| *** mvk has quit IRC | 10:54 | |
| *** Poornima has joined #openstack-cinder | 10:55 | |
| *** Poornima_K has joined #openstack-cinder | 10:56 | |
| *** Poornima_K has quit IRC | 10:57 | |
| *** sid14726 has quit IRC | 10:57 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 11:01 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Limit SG and MV to 64 characters https://review.openstack.org/272777 | 11:02 |
| *** ildikov has quit IRC | 11:03 | |
| *** alonma has quit IRC | 11:07 | |
| *** sid14726 has joined #openstack-cinder | 11:08 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Changing PercentSynced to CopyState in isSynched https://review.openstack.org/246992 | 11:10 |
| *** baojg_ has quit IRC | 11:12 | |
| *** alonma has joined #openstack-cinder | 11:14 | |
| *** pradipm has quit IRC | 11:15 | |
| *** baojg has joined #openstack-cinder | 11:16 | |
| *** alonma has quit IRC | 11:18 | |
| *** alonma has joined #openstack-cinder | 11:20 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder-specs: Job Distribution to support HA A/A https://review.openstack.org/232595 | 11:20 |
| *** alonma has quit IRC | 11:24 | |
| *** funeutron has quit IRC | 11:26 | |
| *** alonma has joined #openstack-cinder | 11:26 | |
| *** mvk has joined #openstack-cinder | 11:26 | |
| *** alonma has quit IRC | 11:30 | |
| *** alonma has joined #openstack-cinder | 11:32 | |
| *** markvoelker has joined #openstack-cinder | 11:32 | |
| *** alonma has quit IRC | 11:36 | |
| *** markvoelker has quit IRC | 11:37 | |
| *** alonma has joined #openstack-cinder | 11:38 | |
| *** Poornima has quit IRC | 11:39 | |
| *** sid14726 has quit IRC | 11:41 | |
| *** alonma has quit IRC | 11:42 | |
| *** sid14726 has joined #openstack-cinder | 11:45 | |
| *** alonma has joined #openstack-cinder | 11:46 | |
| *** alonma has quit IRC | 11:51 | |
| *** alonma has joined #openstack-cinder | 11:52 | |
| *** vgridnev has quit IRC | 11:56 | |
| *** vgridnev has joined #openstack-cinder | 11:56 | |
| *** alonma has quit IRC | 11:57 | |
| *** alonma has joined #openstack-cinder | 11:58 | |
| *** haomaiwa_ has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-cinder | 12:01 | |
| *** alonma has quit IRC | 12:02 | |
| *** ig0r_ has joined #openstack-cinder | 12:12 | |
| *** mylu has joined #openstack-cinder | 12:14 | |
| *** mylu has quit IRC | 12:18 | |
| *** sid14726 has quit IRC | 12:21 | |
| *** bkumar4 has quit IRC | 12:30 | |
| *** markvoelker has joined #openstack-cinder | 12:33 | |
| *** markvoelker has quit IRC | 12:37 | |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS driver: Fix SSH and cluster_admin_ip0 bug https://review.openstack.org/277495 | 12:38 |
| *** dims has quit IRC | 12:40 | |
| *** alonma has joined #openstack-cinder | 12:44 | |
| *** sid14726 has joined #openstack-cinder | 12:45 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races on extend and volume_upload_image https://review.openstack.org/216378 | 12:46 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from consistency groups https://review.openstack.org/259429 | 12:46 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from migrate and retype https://review.openstack.org/221442 | 12:46 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races on replication methods https://review.openstack.org/221441 | 12:46 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from backup https://review.openstack.org/255430 | 12:46 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add ordering possibilities to conditional update https://review.openstack.org/231936 | 12:46 |
| *** alonma has quit IRC | 12:49 | |
| *** alonma has joined #openstack-cinder | 12:50 | |
| *** gouthamr has joined #openstack-cinder | 12:52 | |
| *** raildo-afk is now known as raildo | 12:53 | |
| *** alonma has quit IRC | 12:54 | |
| *** alonma has joined #openstack-cinder | 12:56 | |
| *** sgotliv has joined #openstack-cinder | 12:56 | |
| *** sheel has quit IRC | 12:57 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** bkumar4 has joined #openstack-cinder | 13:01 | |
| *** haomaiwang has joined #openstack-cinder | 13:01 | |
| *** alonma has quit IRC | 13:01 | |
| *** bkumar4 has quit IRC | 13:01 | |
| *** bkumar4 has joined #openstack-cinder | 13:02 | |
| *** alonma has joined #openstack-cinder | 13:02 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add SIGHUP handlers to reset RPC version pins https://review.openstack.org/279039 | 13:03 |
| *** alonma has quit IRC | 13:03 | |
| *** alonma has joined #openstack-cinder | 13:04 | |
| dulek | DuncanT: ^ - that wasn't trivial, but I think I've got it. | 13:04 |
| *** delattec has joined #openstack-cinder | 13:10 | |
| *** bkumar4 has quit IRC | 13:10 | |
| *** delatte has quit IRC | 13:13 | |
| *** mylu has joined #openstack-cinder | 13:14 | |
| *** esp has joined #openstack-cinder | 13:17 | |
| *** gouthamr has quit IRC | 13:17 | |
| *** eharney has joined #openstack-cinder | 13:18 | |
| *** martyturner has joined #openstack-cinder | 13:18 | |
| *** mylu has quit IRC | 13:19 | |
| *** esker has joined #openstack-cinder | 13:20 | |
| *** dims has joined #openstack-cinder | 13:20 | |
| *** sid14726 has quit IRC | 13:21 | |
| *** markvoelker has joined #openstack-cinder | 13:23 | |
| *** esp has quit IRC | 13:23 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: Support backup import on another Storage database https://review.openstack.org/153969 | 13:23 |
| *** xyang1 has joined #openstack-cinder | 13:24 | |
| *** sborkows has joined #openstack-cinder | 13:25 | |
| *** akerr has joined #openstack-cinder | 13:31 | |
| *** delatte has joined #openstack-cinder | 13:32 | |
| *** akerr_ has joined #openstack-cinder | 13:33 | |
| *** akerr_ has quit IRC | 13:35 | |
| *** delattec has quit IRC | 13:35 | |
| *** akerr_ has joined #openstack-cinder | 13:35 | |
| *** akerr has quit IRC | 13:36 | |
| *** akerr has joined #openstack-cinder | 13:38 | |
| *** funeutron has joined #openstack-cinder | 13:38 | |
| *** edmondsw has joined #openstack-cinder | 13:38 | |
| scottda | e0ne: Were you saying the other day that there was something causing the Rally tests to fail? | 13:39 |
| *** lpetrut has quit IRC | 13:39 | |
| e0ne | scottda: AFAIK, I've just pointed you on rally fail on your patch | 13:39 |
| *** akerr has quit IRC | 13:39 | |
| scottda | e0ne: OK, thanks. | 13:40 |
| *** akerr_ has quit IRC | 13:40 | |
| *** gouthamr has joined #openstack-cinder | 13:40 | |
| e0ne | scottda: the last microversions patch looks good. backup scenarios failures are known issue :( | 13:40 |
| *** akerr has joined #openstack-cinder | 13:40 | |
| scottda | e0ne: YOu mean the Rally failures? They are not something to worry about? | 13:41 |
| e0ne | scottda: yes. rally backup scenarios on gates are broken:( | 13:41 |
| e0ne | scottda: I'm trying to find what happen | 13:41 |
| *** akerr_ has joined #openstack-cinder | 13:41 | |
| scottda | e0ne: ahhh..ok. But I still have apache errors... | 13:41 |
| e0ne | looking | 13:42 |
| *** akerr has quit IRC | 13:42 | |
| scottda | 2016-02-11 02:01:45.352 | + /opt/stack/new/devstack/lib/cinder:create_volume_types:L552: openstack volume type create --property volume_backend_name=lvmdriver-1 lvmdriver-1 | 13:43 |
| scottda | volume types works when I run the commands myself. | 13:43 |
| *** akerr has joined #openstack-cinder | 13:43 | |
| scottda | but test gets "500 error" | 13:43 |
| *** sid14726 has joined #openstack-cinder | 13:44 | |
| e0ne | scottda: http://logs.openstack.org/10/224910/25/check/gate-cinder-dsvm-apache/0426202/logs/apache/c-api.txt.gz#_2016-02-11_02_01_58_200310 | 13:44 |
| *** ankit_ag has quit IRC | 13:45 | |
| *** akerr_ has quit IRC | 13:46 | |
| e0ne | scottda: I don't know why it happens | 13:46 |
| e0ne | I'll setup devstack for it | 13:46 |
| scottda | e0ne: is six.text_type unicode? | 13:47 |
| e0ne | yes | 13:48 |
| *** links has quit IRC | 13:48 | |
| e0ne | it's unicode in python 2.x | 13:49 |
| *** laughterwym has joined #openstack-cinder | 13:50 | |
| scottda | Well, I did add that translation. | 13:51 |
| *** cbader has quit IRC | 13:52 | |
| e0ne | did you find where is it broken? | 13:52 |
| e0ne | I'm intalling devstack with cinder+apache now | 13:52 |
| scottda | e0ne: No, you found where it was broken: | 13:55 |
| scottda | http://logs.openstack.org/10/224910/25/check/gate-cinder-dsvm-apache/0426202/logs/apache/c-api.txt.gz#_2016-02-11_02_01_58_135316 | 13:55 |
| scottda | cinder/api/openstack/wsgi.py L#1065 | 13:55 |
| scottda | I added the translations to use six.text_type | 13:55 |
| scottda | and it fails with: | 13:56 |
| scottda | TypeError: expected byte string object for header value, value of type unicode found | 13:56 |
| *** rlrossit has joined #openstack-cinder | 13:57 | |
| scottda | But it seems that would always fail, if 2.x translates to unicode | 13:57 |
| e0ne | https://review.openstack.org/#/c/224910/25/cinder/api/openstack/wsgi.py - line #1154? | 13:59 |
| e0ne | # Headers must be utf-8 strings | 13:59 |
| e0ne | response.headers[hdr] = six.text_type(val) | 13:59 |
| *** edmondsw has quit IRC | 13:59 | |
| e0ne | thes must be in utf-8, but it must be a string | 14:00 |
| e0ne | response.headers[hdr] = val.encode('utf-8') | 14:00 |
| *** haomaiwang has quit IRC | 14:01 | |
| scottda | e0ne: Yes, that could be it. Good catch | 14:01 |
| *** haomaiwang has joined #openstack-cinder | 14:01 | |
| e0ne | scottda: yes, looks like it will always fail under apache/nginx | 14:01 |
| scottda | I don't really understand that apache test. Do you know a good way to repro that? | 14:02 |
| e0ne | I believe... | 14:02 |
| scottda | so I can test the fix before submitting | 14:02 |
| *** ildikov has joined #openstack-cinder | 14:02 | |
| e0ne | gimme 2 minutes, plz | 14:02 |
| scottda | please, take your time. You are a great help. | 14:02 |
| *** julim has quit IRC | 14:05 | |
| e0ne | to reproduce the issue you have to setup cinder+apache | 14:05 |
| e0ne | the easiest way is to add "CINDER_USE_MOD_WSGI=True" to yours localrc/local.conf and install devstack | 14:05 |
| e0ne | I've just tried to do it myself | 14:05 |
| e0ne | fix with "response.headers[hdr] = val.encode('utf-8') " works | 14:06 |
| e0ne | but | 14:06 |
| e0ne | you can configure apache manually w/o destack | 14:06 |
| *** laughter_ has joined #openstack-cinder | 14:06 | |
| *** alonma has quit IRC | 14:07 | |
| *** esker has quit IRC | 14:08 | |
| *** alonma has joined #openstack-cinder | 14:08 | |
| *** dave-mccowan has joined #openstack-cinder | 14:08 | |
| e0ne | scottda: I've added a note about it to gerrit | 14:08 |
| e0ne | scottda: thanks a lot for not ignoring non-voting job! | 14:09 |
| *** laughte__ has joined #openstack-cinder | 14:09 | |
| *** laughterwym has quit IRC | 14:09 | |
| scottda | Thanks for the help and reviews. | 14:10 |
| *** laughter_ has quit IRC | 14:11 | |
| e0ne | you're welcome! | 14:11 |
| *** dave-mcc_ has joined #openstack-cinder | 14:11 | |
| *** alonma has quit IRC | 14:12 | |
| *** alonma has joined #openstack-cinder | 14:14 | |
| *** dave-mccowan has quit IRC | 14:14 | |
| *** dustins has joined #openstack-cinder | 14:14 | |
| *** krotscheck_dcm is now known as krotscheck | 14:15 | |
| *** mriedem has joined #openstack-cinder | 14:17 | |
| smcginnis | e0ne: I haven't looked in a while. How stable has the rally job been? | 14:17 |
| e0ne | smcginnis: except bug https://bugs.launchpad.net/cinder/+bug/1359428 (TBH, tempests job are affected too) is was stable enough | 14:18 |
| openstack | Launchpad bug 1359428 in Cinder "Checksum error\n Volume group for uuid not found; libdevmapper exiting with 2 device(s) still suspended" [High,Confirmed] | 14:18 |
| *** esker has joined #openstack-cinder | 14:18 | |
| smcginnis | e0ne: Great! | 14:19 |
| e0ne | smcginnis: I'm missed point 1-2 weeks ago when it was broken | 14:19 |
| smcginnis | e0ne: Do you recall what was broken? | 14:19 |
| e0ne | smcginnis: I'm investigating this issue, bug to cinder (for now to cinder) is filed | 14:19 |
| *** laughte__ has quit IRC | 14:19 | |
| e0ne | smcginnis: may be it's related to keystone v3 api. I'm not sure yet. I can't reproduce it locally:( | 14:20 |
| smcginnis | e0ne: Ah, one of the fun ones. ;) | 14:20 |
| e0ne | :) | 14:20 |
| *** merooney has joined #openstack-cinder | 14:21 | |
| *** knomura has joined #openstack-cinder | 14:21 | |
| *** esker has quit IRC | 14:23 | |
| scottda | smcginnis: Have you ever used the cinder youtube user to stream? I think I need a user email and password to use it, but am not certain about that. | 14:24 |
| *** martyt has joined #openstack-cinder | 14:25 | |
| smcginnis | scottda: No, just your own account. | 14:28 |
| *** links has joined #openstack-cinder | 14:28 | |
| smcginnis | scottda: Then you have rights to stream into that channel. | 14:28 |
| scottda | OK, I'll test it in a bit and see if that works. Thanks. | 14:28 |
| smcginnis | scottda: Yeah, we can test ahead of time and I'm pretty sure we can delete whatever ends up out there. | 14:29 |
| smcginnis | scottda: Or leave it for fun. ;) | 14:29 |
| scottda | Yeah, Cinder team doesn't have enough fun :) | 14:29 |
| smcginnis | We're way too serious. | 14:29 |
| openstackgerrit | Wang Bo proposed openstack/cinder: Python3: fix indexing of dict.keys() in python3 https://review.openstack.org/267720 | 14:34 |
| *** JoseMello has joined #openstack-cinder | 14:34 | |
| DuncanT | dulek: I've got your SIGHUP patch bookmarked, need to spend some time on it, looks good at a first pass though | 14:35 |
| *** alonma has quit IRC | 14:35 | |
| *** liverpooler has quit IRC | 14:36 | |
| *** martyt has quit IRC | 14:36 | |
| *** alonma has joined #openstack-cinder | 14:36 | |
| *** baojg has quit IRC | 14:37 | |
| *** akshai has joined #openstack-cinder | 14:38 | |
| *** jgregor has joined #openstack-cinder | 14:38 | |
| *** ociuhandu has joined #openstack-cinder | 14:39 | |
| *** lpetrut has joined #openstack-cinder | 14:41 | |
| *** alonma has quit IRC | 14:41 | |
| sborkows | geguileo Hi, I have a question about conditional_update. I'm just passing dict of values, expected = None and filters in that case are an empty list. When I process the create volume request, I call conditional update and after that call I got an error: http://codepaste.net/s25qsn . Can it be a None value of expected attribute an issue? | 14:45 |
| *** akerr has quit IRC | 14:46 | |
| geguileo | sborkows: Why are you calling conditional update without anything in expected? | 14:46 |
| geguileo | sborkows: Then you don't need it to be conditional | 14:47 |
| *** alonma has joined #openstack-cinder | 14:47 | |
| *** cbader has joined #openstack-cinder | 14:47 | |
| geguileo | sborkows: You should read the docstring of the conditional_update method | 14:47 |
| geguileo | If no expected_values are passed in we will default to make sure | 14:48 |
| geguileo | that all fields have not been changed in the DB. Since we cannot | 14:48 |
| geguileo | know the original value in the DB for dirty fields in the object | 14:48 |
| geguileo | those will be excluded. | 14:48 |
| *** knikolla has joined #openstack-cinder | 14:48 | |
| geguileo | sborkows: If you don't want that to happen you can pass an empty dict | 14:49 |
| *** diablo_rojo has joined #openstack-cinder | 14:49 | |
| *** sid14726 has quit IRC | 14:50 | |
| *** alonma has quit IRC | 14:51 | |
| *** julim has joined #openstack-cinder | 14:52 | |
| *** alonma has joined #openstack-cinder | 14:53 | |
| knomura | smcginnis: Hi, Could you review this patch? (https://review.openstack.org/#/c/271763/) | 14:54 |
| *** erlon has joined #openstack-cinder | 14:55 | |
| *** baumann has joined #openstack-cinder | 14:55 | |
| *** cknight has joined #openstack-cinder | 14:55 | |
| smcginnis | knomura: I can try. Still not happy with 6000+ lines of code to look through. | 14:55 |
| *** markstur_ has joined #openstack-cinder | 14:57 | |
| *** cfriesen has joined #openstack-cinder | 14:57 | |
| *** alonma has quit IRC | 14:57 | |
| knomura | smcginnis: Great thanks. hitachi-hbsd2-ci don't work now. But CI will work well before long. | 14:57 |
| smcginnis | knomura: We will definitely need a successful report from the CI before we can merge that one. | 14:58 |
| JoseMello | knomura smcginnis the patch is the next on the line for the CI run | 14:58 |
| smcginnis | JoseMello: OK, thanks. | 14:59 |
| *** alonma has joined #openstack-cinder | 14:59 | |
| *** markstur_ has left #openstack-cinder | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-cinder | 15:01 | |
| *** nug has left #openstack-cinder | 15:01 | |
| *** alyson_ has joined #openstack-cinder | 15:02 | |
| *** eharney has quit IRC | 15:02 | |
| knomura | smcginnis: Sure. we'll do our best for the CI runs. | 15:02 |
| dulek | geguileo: I think he's calling it with filters being non-empty. | 15:02 |
| *** crose has joined #openstack-cinder | 15:03 | |
| geguileo | dulek: It's the same, he still need to pass an empty dict to expected if he doesn't want the method to match all attributes | 15:03 |
| *** alonma has quit IRC | 15:03 | |
| dulek | geguileo: Sure, makes sense - empty dict is the solution. :) | 15:03 |
| *** jungleboyj has joined #openstack-cinder | 15:04 | |
| openstackgerrit | Michael Rowden proposed openstack/cinder: Correcting thin provisioning behavior https://review.openstack.org/275408 | 15:04 |
| *** alonma has joined #openstack-cinder | 15:05 | |
| *** jwang_ has joined #openstack-cinder | 15:08 | |
| *** alonma has quit IRC | 15:09 | |
| *** alonma has joined #openstack-cinder | 15:09 | |
| *** lprice has joined #openstack-cinder | 15:12 | |
| *** jwang has quit IRC | 15:12 | |
| *** lprice1 has quit IRC | 15:12 | |
| *** mragupat has joined #openstack-cinder | 15:18 | |
| *** martyt has joined #openstack-cinder | 15:20 | |
| *** ig0r_ has quit IRC | 15:21 | |
| *** sheel has joined #openstack-cinder | 15:21 | |
| *** eharney has joined #openstack-cinder | 15:21 | |
| sborkows | geguileo: Thanks I'll try this | 15:24 |
| *** martyt has quit IRC | 15:24 | |
| sborkows | geguileo: Unfortunately, it does not help. I think it is a 'values' issue, because the log mentions 'MetaData' | 15:30 |
| dulek | sborkows: Hm, maybe it's the MetaData object from cinder.db.sqlalchemy.models? | 15:30 |
| dulek | sborkows: Oh, wait, no MetaData model there. | 15:30 |
| *** garthb has joined #openstack-cinder | 15:31 | |
| sborkows | In values I have the metadata which is the empty dict | 15:31 |
| geguileo | sborkows: That's wrong... | 15:32 |
| geguileo | sborkows: Metadata is another table | 15:33 |
| dulek | geguileo: Ah, I see. | 15:33 |
| openstackgerrit | Oleksii Butenko proposed openstack/python-cinderclient: Test https://review.openstack.org/278971 | 15:33 |
| geguileo | sborkows: conditional update does not construct a multi table update | 15:33 |
| geguileo | sborkows: It is not supported by postgresql | 15:33 |
| geguileo | sborkows: But if you are passing an empty dictionary as metadata | 15:33 |
| geguileo | sborkows: It will have the same effect if you don't pass anything | 15:34 |
| dulek | geguileo: It's bad that conditional_update method *on o.vo* operates on fields from the *SQLAlchemy model*. | 15:34 |
| geguileo | dulek: What do you mean? | 15:34 |
| geguileo | dulek: The error is from sqlalchemy | 15:34 |
| dulek | geguileo: Well, these may not match. | 15:34 |
| *** garthb has quit IRC | 15:35 | |
| *** garthb has joined #openstack-cinder | 15:35 | |
| dulek | geguileo: An example is volume_metadata on Volume. | 15:35 |
| *** zul_ has joined #openstack-cinder | 15:36 | |
| *** zul has quit IRC | 15:36 | |
| dulek | geguileo: In the DB it's "volume_metadata", on object it's "metadata". Moreover it's dict on object, but list in SQLAlchemy obj. | 15:36 |
| openstackgerrit | Oleksii Butenko proposed openstack/python-cinderclient: Testv2 https://review.openstack.org/278971 | 15:36 |
| sborkows | geguileo: I did not even think about multi-update, my intention was to update the almost empty volume with ids of e.g. snapshots, source volumes with complex conditionals | 15:37 |
| geguileo | dulek: It doesn't matter, those should not be used for a conditional update | 15:37 |
| *** dave-mcc_ has quit IRC | 15:37 | |
| geguileo | sborkows: But you should know that metadata is another table in the database | 15:37 |
| *** dave-mccowan has joined #openstack-cinder | 15:37 | |
| dulek | geguileo: Right, so no joined updates. Metadata should go as a normal update then. | 15:38 |
| sborkows | geguileo: ok, I will figure out which parameter makes mess. | 15:38 |
| *** akerr has joined #openstack-cinder | 15:38 | |
| geguileo | sborkows: You can do joined updates | 15:38 |
| geguileo | sborkows: What you cannot do is multi-table updates | 15:39 |
| geguileo | sborkows: Only 1 table can be updated | 15:39 |
| geguileo | sborkows: If multiple tables needs to be updated then you need a transaction with the right isolation level | 15:39 |
| dulek | geguileo: Right, I was thinking of JOIN referring to multi-table. Anyway I get it. | 15:40 |
| geguileo | sborkows: Sorry, those were for dulek :-) | 15:40 |
| *** mtanino has joined #openstack-cinder | 15:40 | |
| geguileo | dulek: You can see the problem if you look at the results of the backup API races removal patch | 15:40 |
| geguileo | dulek: Because I have to redo it | 15:40 |
| geguileo | dulek: It works fine with mysql but doesn't work on postgresql | 15:40 |
| geguileo | dulek: Because I was doing a multitable update :''-( | 15:41 |
| *** arecknag has quit IRC | 15:41 | |
| dulek | geguileo: BTW multi-table update evaluates to sth like "UPDATE foo, bar SET foo.abc=1, bar.def=2;"? | 15:41 |
| geguileo | dulek: Depending on the backend | 15:42 |
| dulek | geguileo: Evaluates to error on PostgreSQL. ;) | 15:42 |
| geguileo | dulek: I think it yields different results for mysql and sqlserver | 15:42 |
| geguileo | dulek: Yep, but a weird error I didn't know what it meant | 15:42 |
| geguileo | dulek: And I only realized that it was a problem when the patch was completed :''-( | 15:42 |
| *** edmondsw has joined #openstack-cinder | 15:43 | |
| *** mylu has joined #openstack-cinder | 15:44 | |
| scottda | e0ne: If you are around, I'd like to bother you again... | 15:47 |
| sborkows | geguileo: so the 'metadata' attribute made a mess, even if it is an empty dict | 15:47 |
| geguileo | sborkows: Yep | 15:47 |
| e0ne | scottda: what's up? | 15:47 |
| geguileo | sborkows: The problem is not the value, but using metadata key | 15:48 |
| scottda | I put up the changes we discussed, and 3 unit tests failed... | 15:48 |
| scottda | Then I backed out the changes, and the 3 tests still failed on my workstation. | 15:48 |
| scottda | The failures don't happen in patch set 24, but do in patch set 25 | 15:48 |
| scottda | Between 24 & 25 I did a rebase (for thin provisioning fixes) | 15:48 |
| scottda | And upstream, the tests pass on patch set 25. But on my workstation they fail | 15:49 |
| scottda | I cannot understand why this would happen. | 15:49 |
| sborkows | geguileo: so, should I avoid putting that key in values for conditional update, and when it just succeeded, would do the normal update? | 15:49 |
| scottda | e0ne: if you have a devstack handy, can you please test this on patch set 25: | 15:49 |
| geguileo | sborkows: You mean the normal update to update the metadata with an empty dict? | 15:49 |
| scottda | tox -epy27 -- -n cinder.tests.unit.test_service.TestWSGIService.test_workers_set_default | 15:49 |
| *** lcurtis has joined #openstack-cinder | 15:49 | |
| e0ne | scottda: to you mean run tests with running devstack? | 15:50 |
| scottda | yes. | 15:50 |
| dulek | geguileo: Well, it's create_volume, metadata can be anything there. I think sborkows just tested it without passing any. | 15:50 |
| scottda | I tried restack and reboot, but still get the failure | 15:50 |
| sborkows | geguileo: yes, but dulek just gave me a sugesstion to put it in insert, at the beginning, before the volume.create() call | 15:50 |
| scottda | And I tried on a different VM | 15:50 |
| *** amoturi has joined #openstack-cinder | 15:50 | |
| *** alonma has quit IRC | 15:51 | |
| geguileo | If it's an empty dict there's no need to do anything afaik | 15:51 |
| e0ne | scottda: why tests are depended on devstack? | 15:51 |
| *** alonma has joined #openstack-cinder | 15:51 | |
| geguileo | If it's something different and you want it to be written atomically you need a transaction with the right isolation level | 15:51 |
| *** sgotliv has quit IRC | 15:51 | |
| *** andrei__ has quit IRC | 15:52 | |
| scottda | e0ne: Well, they're not. I just use Devstack to run the tests. Could be done on my workstation in a different directory. I just use Devstack for everything | 15:52 |
| *** dobson has quit IRC | 15:52 | |
| e0ne | scottda: so what should I do? just run the test mentioned above on my env? or do it in devstack? | 15:53 |
| *** martyt has joined #openstack-cinder | 15:53 | |
| *** EmilienM has quit IRC | 15:53 | |
| e0ne | I'm using different VMs for devstack and unit tests. | 15:54 |
| scottda | e0ne: either one, should not matter. But if it makes a difference, maybe that's my problem. | 15:54 |
| *** NightKhaos has quit IRC | 15:54 | |
| scottda | e0ne: I see. I don't do that | 15:54 |
| scottda | maybe I should.... | 15:54 |
| *** precurse has quit IRC | 15:54 | |
| sborkows | geguileo: indeed. But do we want to make it atomically, when the user does even not know about the fact, that the volume entry has already been created in db? The metadata in that case cannot be deleted I think. | 15:54 |
| *** martyt has quit IRC | 15:55 | |
| geguileo | sborkows: I agree | 15:55 |
| geguileo | sborkows: I was just mentioning as a general concept | 15:55 |
| e0ne | TBH, I run unit tests on my desktop, and devstack on VMs | 15:55 |
| *** HenryG has quit IRC | 15:55 | |
| geguileo | sborkows: When creating a volume you don't have those constraints | 15:55 |
| *** alonma has quit IRC | 15:55 | |
| *** v12aml has quit IRC | 15:56 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Split out NestedQuotas into a separate driver https://review.openstack.org/274825 | 15:57 |
| *** alonma has joined #openstack-cinder | 15:57 | |
| *** laughterwym has joined #openstack-cinder | 15:57 | |
| scottda | e0ne: Well, you are a smart man. When I ./unstack, the unit tests pass | 15:58 |
| e0ne | scottda: or maybe I'm a happy and have some resources for VMs ;) | 15:58 |
| sborkows | geguileo: Thanks for help, now I can make a further progress :) | 15:59 |
| *** martyt has joined #openstack-cinder | 15:59 | |
| geguileo | sborkows: Thanks for working on that | 15:59 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Fix calls to non-existent assert methods https://review.openstack.org/279153 | 15:59 |
| geguileo | sborkows: If you run into any more problems with the conditional_update method just ping me | 15:59 |
| *** fthiagogv has joined #openstack-cinder | 15:59 | |
| *** mriedem is now known as mriedem_meeting | 16:00 | |
| sborkows | geguileo: sure | 16:00 |
| *** jordanP has joined #openstack-cinder | 16:00 | |
| e0ne | scottda: confirmed, it doesn't work with running devstack | 16:00 |
| *** haomaiwang has quit IRC | 16:01 | |
| e0ne | scottda: and I know the best solution for it | 16:01 |
| e0ne | (I realy really hope that it's the best) | 16:01 |
| scottda | e0ne: Thanks for confirming | 16:01 |
| *** haomaiwa_ has joined #openstack-cinder | 16:01 | |
| *** alonma has quit IRC | 16:01 | |
| *** laughterwym has quit IRC | 16:01 | |
| *** knomura has quit IRC | 16:01 | |
| e0ne | scottda: move these tests to functional (depends on https://review.openstack.org/267801) | 16:01 |
| scottda | e0ne: You mean you have a solution? other than "stop running devstack while running unit tests"? | 16:01 |
| *** HenryG has joined #openstack-cinder | 16:02 | |
| e0ne | scottda: "stop devstack" is a 1st step:) | 16:02 |
| *** andrei__ has joined #openstack-cinder | 16:02 | |
| *** knomura has joined #openstack-cinder | 16:02 | |
| scottda | ahh, ok. you have an even better long-term solution. | 16:02 |
| *** NightKhaos has joined #openstack-cinder | 16:02 | |
| scottda | Very good. | 16:02 |
| *** v12aml has joined #openstack-cinder | 16:02 | |
| *** alonma has joined #openstack-cinder | 16:03 | |
| *** funeutron has quit IRC | 16:03 | |
| *** links has quit IRC | 16:04 | |
| Swanson | Okay I'm learning to have the review page thingy. | 16:04 |
| *** dobson has joined #openstack-cinder | 16:05 | |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: cinder-api-microversions code https://review.openstack.org/224910 | 16:05 |
| *** EmilienM has joined #openstack-cinder | 16:06 | |
| e0ne | scottda: I've tested your patch with old client - it works for me | 16:06 |
| *** knomura has quit IRC | 16:06 | |
| smcginnis | Swanson: Review page thingy? | 16:06 |
| e0ne | old==latest for pypi | 16:06 |
| scottda | e0ne: Thanks. | 16:07 |
| e0ne | scottda: np. I've promised to do it almost one moth ago:( | 16:07 |
| scottda | e0ne: I need to update my patches for cinderclient to use /v3 enpoint. Hopefully I can get to that today. | 16:07 |
| *** timcl has joined #openstack-cinder | 16:07 | |
| e0ne | I'll add note about my tests once CI passed | 16:07 |
| scottda | cool | 16:07 |
| *** alonma has quit IRC | 16:07 | |
| *** precurse has joined #openstack-cinder | 16:09 | |
| *** alonma has joined #openstack-cinder | 16:09 | |
| *** mylu has quit IRC | 16:12 | |
| *** mylu has joined #openstack-cinder | 16:12 | |
| *** alonma has quit IRC | 16:13 | |
| *** cbader has quit IRC | 16:14 | |
| *** vgridnev has quit IRC | 16:15 | |
| *** cbader has joined #openstack-cinder | 16:15 | |
| *** mylu_ has joined #openstack-cinder | 16:17 | |
| *** mylu has quit IRC | 16:17 | |
| sheel | scottda hi | 16:18 |
| sheel | scottda: "Update the expected versions in affected tests." | 16:18 |
| diablo_rojo | jbernard: Can you send me those two etherpads you mentioned creating yesterday in the meeting? | 16:18 |
| scottda | sheel: Cool. I'll review and test later today. | 16:18 |
| sheel | scottda: In microversion patch, could you give some details what need to do for "Update the expected versions in affected tests." | 16:19 |
| *** garthb has quit IRC | 16:20 | |
| sheel | scottda: this was in reference of https://review.openstack.org/#/c/224910/24/doc/source/devref/api_microversion_dev.rst - L287 | 16:21 |
| *** alonma has joined #openstack-cinder | 16:21 | |
| *** vgridnev has joined #openstack-cinder | 16:21 | |
| scottda | sheel: I may just have to work with you on which tests are affected for your patch, and how to change for both pre- and post- micorversion. | 16:22 |
| openstackgerrit | Jon Bernard proposed openstack/cinder: RBD: use versioned objects https://review.openstack.org/279158 | 16:22 |
| *** vgridnev has quit IRC | 16:22 | |
| scottda | sheel: I'm not sure that I can put something generic enough in the microversion patch that is also specific, but I'll try after we figure out what changes in unit tests for your patches. | 16:22 |
| sheel | scottda: ok. So, may be i submit patch for now without this and later on work on this? | 16:23 |
| scottda | sheel: You can submit the patch at any time, but I think it should have the appropriate unit tests before it merges. | 16:24 |
| *** mvk has quit IRC | 16:24 | |
| sheel | scottda: yes sure | 16:25 |
| scottda | sheel: Since your patch will depend on microversion patches, and microversions have not merged, I don't think there is any great hurry. We'll get it done, just going to take a little time.... | 16:25 |
| sheel | scottda: agreed.. | 16:26 |
| *** alonma has quit IRC | 16:27 | |
| *** jordanP has quit IRC | 16:27 | |
| *** alonma has joined #openstack-cinder | 16:30 | |
| *** jdurgin1 has joined #openstack-cinder | 16:30 | |
| *** martyt has quit IRC | 16:30 | |
| *** alonma has quit IRC | 16:34 | |
| *** jgregor has quit IRC | 16:35 | |
| *** alonma has joined #openstack-cinder | 16:36 | |
| *** mylu has joined #openstack-cinder | 16:36 | |
| *** cppforlife_ has quit IRC | 16:38 | |
| *** tpsilva has quit IRC | 16:38 | |
| *** mylu_ has quit IRC | 16:38 | |
| *** zhenguo_ has quit IRC | 16:38 | |
| *** tpeoples has quit IRC | 16:38 | |
| *** arch-nemesis has joined #openstack-cinder | 16:38 | |
| *** JoseMello has quit IRC | 16:39 | |
| *** sheel has quit IRC | 16:39 | |
| *** krtaylor has quit IRC | 16:40 | |
| *** alonma has quit IRC | 16:40 | |
| *** salv-orl_ has joined #openstack-cinder | 16:41 | |
| *** esker has joined #openstack-cinder | 16:42 | |
| *** sheel has joined #openstack-cinder | 16:43 | |
| *** JoseMello has joined #openstack-cinder | 16:43 | |
| *** isaacb has quit IRC | 16:43 | |
| *** cppforlife_ has joined #openstack-cinder | 16:43 | |
| *** tpeoples has joined #openstack-cinder | 16:43 | |
| *** zhenguo_ has joined #openstack-cinder | 16:44 | |
| *** salv-orlando has quit IRC | 16:44 | |
| *** timcl has quit IRC | 16:51 | |
| *** r-daneel has joined #openstack-cinder | 16:52 | |
| *** mriedem_meeting is now known as mriedem | 16:52 | |
| *** tpsilva has joined #openstack-cinder | 16:53 | |
| *** rcernin has quit IRC | 16:56 | |
| smcginnis | hemna_: ping | 16:56 |
| hemna_ | yough | 16:56 |
| smcginnis | hemna_: Are we waiting for anything in os-brick. | 16:57 |
| hemna_ | yah | 16:57 |
| *** krtaylor has joined #openstack-cinder | 16:57 | |
| smcginnis | hemna_: Other than the recently approved patch? | 16:57 |
| hemna_ | we have 4 patches in gate | 16:57 |
| smcginnis | hemna_: Anything else here: https://review.openstack.org/#/q/project:openstack/os-brick+status:open | 16:57 |
| hemna_ | https://review.openstack.org/#/c/264678/ | 16:57 |
| hemna_ | I haven't looked at that one closely yet | 16:58 |
| hemna_ | trying to figure it out | 16:58 |
| smcginnis | hemna_: Hah, was just going to ask you about that. | 16:58 |
| smcginnis | hemna_: Looks good to me. | 16:58 |
| *** sgotliv has joined #openstack-cinder | 16:58 | |
| smcginnis | hemna_: But would definitely like other eyes on it. | 16:58 |
| *** esker has quit IRC | 16:58 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: WIP: Add devref on rolling upgrades https://review.openstack.org/279186 | 16:58 |
| hemna_ | it's a pretty decent change | 16:58 |
| smcginnis | hemna_: Other than that one, once all the in-flight changes merge, think we're clear to make the final | 17:00 |
| smcginnis | Mitaka os-brick cut? | 17:00 |
| hemna_ | yah I think so | 17:00 |
| hemna_ | well | 17:00 |
| smcginnis | hemna_: Awesome. I can try to avoid the crunch then. ;) | 17:00 |
| hemna_ | I was hoping to get the privsep patch to land | 17:00 |
| hemna_ | but, it looks like there are some serious problems with it still :( | 17:00 |
| smcginnis | hemna_: How's that looking. | 17:00 |
| hemna_ | I've been working with gus daily on it | 17:00 |
| smcginnis | I know we really wanted that in place. | 17:00 |
| *** jgregor has joined #openstack-cinder | 17:01 | |
| *** haomaiwa_ has quit IRC | 17:01 | |
| hemna_ | it's working with my command line scripts that call brick's connector methods | 17:01 |
| hemna_ | but | 17:01 |
| hemna_ | when it's in place with Nova, Nova locks up on the first call into brick. | 17:01 |
| hemna_ | :( | 17:01 |
| hemna_ | totally freezes nova | 17:01 |
| smcginnis | hemna_: :( | 17:01 |
| *** haomaiwang has joined #openstack-cinder | 17:01 | |
| hemna_ | https://review.openstack.org/#/c/277606/ | 17:01 |
| hemna_ | that one needs to land as well | 17:01 |
| smcginnis | Weird. Wondering what it's getting wedged on? | 17:01 |
| hemna_ | it's a good bug fix | 17:01 |
| hemna_ | for systems when you have LUN ids > 255 | 17:02 |
| *** esker has joined #openstack-cinder | 17:02 | |
| hemna_ | I've had customers run into it. | 17:02 |
| smcginnis | That's right. Yeah, I could be impacted by that as well. I'm sure others. | 17:02 |
| hemna_ | just need diablo_rojo to respond to the -1 and get er done | 17:02 |
| smcginnis | :) | 17:02 |
| hemna_ | diablo_rojo, ping | 17:03 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: WIP: Add devref on rolling upgrades https://review.openstack.org/279186 | 17:03 |
| smcginnis | OK, I'll wait on that and time to review the multipath one. | 17:03 |
| hemna_ | jungleboyj, ping | 17:03 |
| smcginnis | And hope for goodness on the privsep stuff. | 17:03 |
| hemna_ | the privsep patch seems risky at this point | 17:04 |
| hemna_ | it has yet to work | 17:04 |
| hemna_ | so, I'm inclined to push it out after this next drop | 17:04 |
| jungleboyj | hemna_: What's up? | 17:04 |
| hemna_ | jungleboyj, any word from diablo_rojo on your -1 for https://review.openstack.org/#/c/277606/ | 17:05 |
| hemna_ | we'd like to get that one updated and in the gate today | 17:05 |
| jungleboyj | hemna_: Been in meetings all morning. I am guessing it is next on diablo_rojo 's list. | 17:05 |
| hemna_ | ok, we are holding up os-brick for that guy | 17:05 |
| *** jwcroppe has joined #openstack-cinder | 17:06 | |
| *** ndipanov has quit IRC | 17:06 | |
| jungleboyj | hemna_: I just talked to her. She is addressing your comments as well and working through unit test issues. | 17:06 |
| jungleboyj | So, actively working it. | 17:06 |
| diablo_rojo | hemna_: Working on it now. Jacob just suggested I move the method into the method into the class in linuxscsi | 17:06 |
| *** sgotliv has quit IRC | 17:06 | |
| *** esker has quit IRC | 17:07 | |
| diablo_rojo | hemna_: Want me to do that too? I had it defined in file, but outside the method | 17:07 |
| diablo_rojo | I mean class | 17:07 |
| diablo_rojo | lol | 17:07 |
| hemna_ | oh yah, oopsie | 17:07 |
| hemna_ | easy enough change | 17:07 |
| diablo_rojo | hemna_: Doing it now | 17:08 |
| hemna_ | ok thanks | 17:08 |
| *** ociuhandu has quit IRC | 17:12 | |
| *** porrua has joined #openstack-cinder | 17:12 | |
| *** e0ne has quit IRC | 17:14 | |
| *** esker has joined #openstack-cinder | 17:14 | |
| *** esker has quit IRC | 17:14 | |
| *** esker has joined #openstack-cinder | 17:14 | |
| *** apoorvad has joined #openstack-cinder | 17:17 | |
| mc_nair | diablo_rojo: I had some suggestions to cleanup the code some, but they're not urgent. I'll comment on the patch but not -1 and you can fix them up after the release | 17:18 |
| *** nkrinner has quit IRC | 17:19 | |
| sborkows | geguileo: creating basic volume (specifying only size) and from snapshot works for me now. But I got stuck at case when we are creating volume from another volume. The root cause is the query to the db -> update volumes ... where ... and exists(select * from volumes ...) | 17:20 |
| sborkows | http://dev.mysql.com/doc/refman/5.6/en/update.html | 17:20 |
| diablo_rojo | mc_nair: Sounds good. There are a lot of refactors coming down the road in brick so I can clean that up then. Thanks for looking at it. | 17:20 |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Adds support for min/max volume size on vol_type https://review.openstack.org/278648 | 17:20 |
| geguileo | sborkows: Where's the problem? | 17:21 |
| geguileo | sborkows: In the exists filter? | 17:21 |
| sborkows | in the almost last paragraph it is written that one cannot select the updated table in a subquery | 17:21 |
| openstackgerrit | Merged openstack/os-brick: Remove multipath -l logic from ISCSI connector https://review.openstack.org/267085 | 17:21 |
| *** leeantho has joined #openstack-cinder | 17:21 | |
| hemna_ | diablo_rojo, ^^ w00t! | 17:22 |
| openstackgerrit | Merged openstack/os-brick: Fix output returned from get_all_available_volumes https://review.openstack.org/276876 | 17:22 |
| sborkows | however, sqlalchemy does not prevent to do that, it is an error on the db level unfortunately | 17:22 |
| diablo_rojo | hemna_: YAY :) | 17:22 |
| hemna_ | diablo_rojo, congrats. :) | 17:22 |
| *** fernnest has quit IRC | 17:22 | |
| geguileo | sborkows: What's the query you want to run? | 17:22 |
| *** fernnest has joined #openstack-cinder | 17:23 | |
| geguileo | sborkows: I mean, what is the restriction, not the sql query itself | 17:23 |
| *** fernnest has quit IRC | 17:24 | |
| *** fernnest has joined #openstack-cinder | 17:24 | |
| diablo_rojo | hemna_: Thanks :) Can I go 2 for 2 is the real question. | 17:24 |
| sborkows | geguileo: http://paste.openstack.org/show/486736/ | 17:24 |
| sborkows | the exact query | 17:24 |
| hemna_ | :) | 17:25 |
| geguileo | sborkows: The problems seems to be that you need to use an alias for the second query | 17:27 |
| *** ebalduf has joined #openstack-cinder | 17:28 | |
| geguileo | sborkows: I think I had to do that for one of my patches, but I don't remember which one | 17:28 |
| geguileo | sborkows: By the second query I mean the exists filter | 17:28 |
| sborkows | geguileo: I agree, but is it doable somehow from the sqlalchemy layer? | 17:29 |
| geguileo | sborkows: Yep | 17:29 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder-specs: Deleting volume metadata keys with a single request https://review.openstack.org/279202 | 17:30 |
| geguileo | sborkows: Ok, it's in the migrate and retype patch (Change Id: I2de07b9cf7f0d670e4c531e8be9ae1c879c890f6) | 17:30 |
| geguileo | sborkows: Look in the sqlalchemy/api.py | 17:30 |
| sborkows | geguileo: ok I have found it in sqlalchemy documentation | 17:30 |
| geguileo | sborkows: Method volume_has_same_encryption_type | 17:30 |
| *** esker has quit IRC | 17:31 | |
| geguileo | sborkows: It's only a problem with the query because the DB engine cannot tell the difference between the volume referenced in the subquery and the one from the query | 17:31 |
| *** ociuhandu has joined #openstack-cinder | 17:33 | |
| *** esker has joined #openstack-cinder | 17:33 | |
| sborkows | geguileo: Thanks, I will try this out | 17:34 |
| geguileo | sborkows: Good luck :-) | 17:34 |
| *** alonma has joined #openstack-cinder | 17:36 | |
| *** esker has quit IRC | 17:39 | |
| *** sid14726 has joined #openstack-cinder | 17:39 | |
| *** esker has joined #openstack-cinder | 17:39 | |
| sborkows | geguileo: That does not help me, it is still the same problem, becuase in where clause it is still the volumes table, but now: 'from volumes as volumes1' | 17:40 |
| geguileo | sborkows: And that doesn't work? r:-?? | 17:41 |
| *** alonma has quit IRC | 17:41 | |
| *** garthb has joined #openstack-cinder | 17:42 | |
| ameade | anyone know what openstackid exists and why we want to replace launchpad? | 17:42 |
| ameade | why* | 17:42 |
| sborkows | geguileo: unfortunately, not :/ | 17:42 |
| *** mudassirlatif has joined #openstack-cinder | 17:42 | |
| sborkows | It is a DBAPIError wrapped from pymysql.err.InternalError | 17:43 |
| geguileo | sborkows: Well, that's what I would have done before trying to look for alternatives | 17:43 |
| *** jdurgin1 has quit IRC | 17:43 | |
| *** esker has quit IRC | 17:45 | |
| *** garthb has quit IRC | 17:45 | |
| *** garthb has joined #openstack-cinder | 17:46 | |
| *** alonma has joined #openstack-cinder | 17:47 | |
| apoorvad | Hello cores! Please review https://review.openstack.org/#/c/261126/ and https://review.openstack.org/#/c/268370/ | 17:47 |
| sborkows | geguileo: this sounds bad. I am now finishing the work today and will return back tommorow. Bye | 17:52 |
| *** alonma has quit IRC | 17:52 | |
| openstackgerrit | Merged openstack/os-brick: Raise exception in find_multipath_device https://review.openstack.org/249231 | 17:52 |
| geguileo | sborkows: ok | 17:52 |
| *** sborkows has quit IRC | 17:52 | |
| *** esker has joined #openstack-cinder | 17:53 | |
| mc_nair | diablo_rojo: added the comments. One of the comments is actually a functional difference that would probably be good to fix for the next changeset | 17:53 |
| hemna_ | mc_nair, your assumption about always using hex #'s is completely incorrect | 17:54 |
| hemna_ | you can't do that | 17:54 |
| *** knikolla has quit IRC | 17:54 | |
| openstackgerrit | Merged openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/266163 | 17:55 |
| mc_nair | hemna_: wasn't trying to suggest doing that, was asking for expanding my own understanding *why* that wasn't possible | 17:56 |
| hemna_ | you definitely can't do that. | 17:56 |
| hemna_ | it would be easier, but it just wouldn't work because you'd never find the volumes. | 17:57 |
| mc_nair | hemna_: again wasn't trying to say that's what should be done. Was asking why that's the case | 17:57 |
| mc_nair | but regardless, the functional part was about passing exactly 255 | 17:57 |
| mc_nair | I think with the current code you'd get '255' if you passed '255' but [hex(255)] if you passed [255] | 17:58 |
| hemna_ | sure that's fine | 17:58 |
| hemna_ | maybe some unit tests to ensure that you get the expected int or hex depending on the value | 17:59 |
| *** timcl has joined #openstack-cinder | 18:00 | |
| mc_nair | yea - there's unit tests, just doesn't hit the edge case for both passing list and single val | 18:00 |
| mc_nair | anyway, should be a simple change | 18:00 |
| hemna_ | yah I just saw the unit tests. the boundary case wasn't tested | 18:00 |
| hemna_ | when the last int happens and the first hex. | 18:00 |
| hemna_ | it's a good test case. | 18:00 |
| *** haomaiwang has quit IRC | 18:01 | |
| *** ChubYann has joined #openstack-cinder | 18:01 | |
| mc_nair | hemna_: the udev info you left in the comments was what I was looking for. I *figured* there was a reason, but I didn't know it so was trying to learn for next time. Thanks for that info | 18:01 |
| openstackgerrit | Kendall Nelson proposed openstack/os-brick: Lun id's > 255 should be converted to hex https://review.openstack.org/277606 | 18:01 |
| hemna_ | np | 18:01 |
| *** haomaiwang has joined #openstack-cinder | 18:01 | |
| diablo_rojo | hemna_: ^^ | 18:01 |
| hemna_ | diablo_rojo, coolio | 18:01 |
| hemna_ | lets get this thing in today | 18:01 |
| diablo_rojo | hemna_: DId you want me to do another patchset with your comment from two min ago? | 18:02 |
| hemna_ | diablo_rojo, the boundary unit test case would be nice to have | 18:02 |
| hemna_ | yes please | 18:02 |
| diablo_rojo | hemna_: I had 255 in one of my test cases | 18:02 |
| hemna_ | ok...so | 18:03 |
| diablo_rojo | hemna_: So you want me to do a test case with 256? | 18:03 |
| hemna_ | I think the formatting of the hex # isn't correct | 18:03 |
| *** timcl1 has joined #openstack-cinder | 18:04 | |
| hemna_ | iqn.1989-10.jp.co.xxx:storage.ttsp.2ec7fdda.ff5c4a16.0-lun-0x0200000000000000 | 18:04 |
| diablo_rojo | the hex method returns a lowercase hex number prefaced with 0x. | 18:04 |
| hemna_ | iqn.1989-10.jp.co.xxx:storage.ttsp.2ec7fdda.ff5c4a16.0-lun-0x016b000000000000 | 18:04 |
| hemna_ | not just 0x016b | 18:04 |
| diablo_rojo | hemna_: So in the method I need to add some mask or something that makes it have however many 00's after it? | 18:05 |
| hemna_ | the bug has the information in it | 18:05 |
| *** timcl has quit IRC | 18:05 | |
| hemna_ | also, I can do a test against my array | 18:05 |
| hemna_ | and force the lun ID to 300 | 18:05 |
| hemna_ | or whatever | 18:05 |
| hemna_ | I think I did that and posted the results in the bug | 18:05 |
| hemna_ | to show the final format | 18:05 |
| diablo_rojo | hemna_: So I need to do the mask thing or what? | 18:06 |
| *** sid14726 has quit IRC | 18:07 | |
| diablo_rojo | hemna_: Basically I just need to know if I need to tell the people that were waiting for me to go to lunch to go without me or not :) | 18:07 |
| hemna_ | yes we have to fix it | 18:08 |
| diablo_rojo | hemna_: Okay cool. | 18:08 |
| hemna_ | or we'll get paths like iqn.1989-10.jp.co.xxx:storage.ttsp.2ec7fdda.ff5c4a16.0-lun-0x016b | 18:08 |
| hemna_ | instead of | 18:08 |
| hemna_ | iqn.1989-10.jp.co.xxx:storage.ttsp.2ec7fdda.ff5c4a16.0-lun-0x016b000000000000 | 18:08 |
| hemna_ | which won't be found | 18:08 |
| hemna_ | :( sorry | 18:08 |
| hemna_ | probably should test this against your array as well | 18:08 |
| hemna_ | just to make sure | 18:08 |
| diablo_rojo | hemna_: No worries. I just wanted to confirm. So. I need to create a mask or something to fill in the extra 0's ? | 18:09 |
| *** esker has quit IRC | 18:10 | |
| hemna_ | well I don't think they are always 0's :) | 18:10 |
| hemna_ | it depends on the value itself | 18:10 |
| diablo_rojo | hemna_: Okay. | 18:12 |
| hemna_ | diablo_rojo, mc_nair FWIW, the original udev rule for creating the device paths is in /lib/udev/rules.d/60persistent-storage.rules | 18:13 |
| jungleboyj | hemna_: If the value 0x016b shouldn't it actually be 0x000000000000016b ? | 18:13 |
| hemna_ | diablo_rojo, mc_nair FWIW, the original udev rule for creating the device paths is in /lib/udev/rules.d/60-persistent-storage.rules | 18:14 |
| hemna_ | at least for ubuntu. I hope that isn't different with redhate and others | 18:14 |
| JoseMello | knomura smcginnis CI passed for patch (https://review.openstack.org/#/c/271763/) :) | 18:14 |
| hemna_ | jungleboyj, no | 18:14 |
| hemna_ | I don't think so | 18:14 |
| hemna_ | udevadm and some testing will vet that out | 18:14 |
| smcginnis | JoseMello: Thanks | 18:15 |
| hemna_ | udevadm will dump out the ID_SERIAL that it discovers from the SCSI device | 18:15 |
| *** e0ne has joined #openstack-cinder | 18:15 | |
| hemna_ | actually ID_SERIAL is the scsi WWN | 18:16 |
| mc_nair | hemna_: shweet - good to know | 18:16 |
| hemna_ | not the lun id | 18:16 |
| hemna_ | run | 18:17 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Duplicate code in volume manager and base driver https://review.openstack.org/271331 | 18:17 |
| hemna_ | udevadm info /dev/sdX | 18:17 |
| hemna_ | that pukes out what udev sees and creates | 18:17 |
| *** rcernin has joined #openstack-cinder | 18:17 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Remove duplicated code in volume manager and base driver https://review.openstack.org/271331 | 18:17 |
| hemna_ | for example, http://paste.openstack.org/show/486742/ | 18:17 |
| *** jwcroppe_ has joined #openstack-cinder | 18:21 | |
| *** esker has joined #openstack-cinder | 18:21 | |
| *** jwcroppe_ has quit IRC | 18:22 | |
| *** jwcroppe_ has joined #openstack-cinder | 18:22 | |
| *** mriedem has quit IRC | 18:22 | |
| *** jwcroppe has quit IRC | 18:23 | |
| *** mriedem has joined #openstack-cinder | 18:24 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Added Bootable filter for cinder list command https://review.openstack.org/278605 | 18:25 |
| *** jwcroppe_ has quit IRC | 18:26 | |
| erlon | patrickeast: ping | 18:29 |
| patrickeast | erlon: hey | 18:29 |
| erlon | patrickeast: hey Patrick, | 18:29 |
| erlon | patrickeast: I'm trying to use the image cache feature | 18:29 |
| erlon | patrickeast: I set these options listed in the config reference: http://docs.openstack.org/admin-guide-cloud/blockstorage_image_volume_cache.html | 18:30 |
| erlon | patrickeast: but, it does not seen to be working | 18:30 |
| patrickeast | erlon: hmm ok, well there are a few things to check for | 18:30 |
| *** nkrinner has joined #openstack-cinder | 18:31 | |
| patrickeast | erlon: first off, what openstack are you using (liberty, current master, something else?) | 18:31 |
| erlon | patrickeast: im using devstack | 18:31 |
| erlon | patrickeast: so, master | 18:31 |
| patrickeast | erlon: and do a grep for like 'cache' or 'image-volume' in the logs | 18:31 |
| patrickeast | erlon: and for any exceptions/errors/warnings | 18:31 |
| erlon | patrickeast: cinder-volume? | 18:32 |
| patrickeast | erlon: yea should be c-vol | 18:32 |
| patrickeast | erlon: i made it so it would be pretty verbose when it doesn't work | 18:32 |
| patrickeast | erlon: there should be somethin explaining why in the logs | 18:33 |
| patrickeast | well.. unless its broken, again | 18:33 |
| *** timcl1 has quit IRC | 18:33 | |
| patrickeast | but last i saw was still working on my ci | 18:33 |
| *** zhenguo_ has quit IRC | 18:33 | |
| *** jseiler has quit IRC | 18:33 | |
| erlon | mhm, Ill run cinder-volume outside screen, I might be missing something | 18:34 |
| *** mtanino__ has quit IRC | 18:35 | |
| *** mtanino__ has joined #openstack-cinder | 18:35 | |
| *** zhenguo_ has joined #openstack-cinder | 18:35 | |
| *** alyson_ has quit IRC | 18:36 | |
| *** garthb has quit IRC | 18:37 | |
| *** garthb has joined #openstack-cinder | 18:37 | |
| *** alyson_ has joined #openstack-cinder | 18:37 | |
| *** timcl has joined #openstack-cinder | 18:40 | |
| erlon | patrickeast: find something missing, hnas-nfs.image_volume_cache_enabled = False, though defaut.image_volume_cache_enabled =True | 18:42 |
| patrickeast | erlon: whats your cinder.conf look like? using multi-backend syntax? | 18:42 |
| erlon | patrickeast: I'v created 3 volumes from image... should I do anything else to get the caches created? | 18:42 |
| erlon | patrickeast: yes | 18:42 |
| patrickeast | erlon: nope, it should "just work" whenever you create from image | 18:43 |
| patrickeast | erlon: ah yea, thats the issue | 18:43 |
| erlon | patrickeast: I didnt restart scheduler or api, should I? | 18:43 |
| erlon | only c-vol | 18:43 |
| patrickeast | erlon: so if you are using multi-backend stuff you need to put the config option to enable it and size stuff in the backend definition stanza | 18:43 |
| patrickeast | erlon: thats an issue with a bunch of the backend config options iirc, the ones in 'default' don't really do what you would expect if you define a backend with the multi-backend stuff | 18:44 |
| patrickeast | erlon: should only need c-vol | 18:44 |
| erlon | https://www.irccloud.com/pastebin/vuSiYmkm/cinder.conf | 18:44 |
| patrickeast | that should be good | 18:45 |
| erlon | patrickeast: like that? | 18:45 |
| patrickeast | erlon: yep, that paste looks correct | 18:46 |
| patrickeast | erlon: the tenant stuff you can leave in default since it isn't specific to the backend | 18:46 |
| *** knikolla has joined #openstack-cinder | 18:46 | |
| patrickeast | erlon: but stuff that is specific has to go in the backend definition | 18:47 |
| patrickeast | erlon: like the enabled and cache size restrictions | 18:47 |
| erlon | patrickeast: mhm, but I should see 1 adicional volume, after I create one from image right? | 18:47 |
| patrickeast | erlon: one owned by the internal tenant | 18:47 |
| patrickeast | erlon: you should see details of it happening in the c-vol log | 18:48 |
| erlon | hmm, I only see the 3 Iv created, ill move the tenant/user to default | 18:48 |
| *** garthb has quit IRC | 18:48 | |
| *** garthb has joined #openstack-cinder | 18:49 | |
| *** bardia has joined #openstack-cinder | 18:49 | |
| *** garthb has quit IRC | 18:50 | |
| erlon | the tenant stuff 'must' be outside the stanza? or is optional? | 18:50 |
| *** esker has quit IRC | 18:51 | |
| patrickeast | erlon: looks like it must be | 18:51 |
| *** timcl has quit IRC | 18:51 | |
| *** garthb has joined #openstack-cinder | 18:51 | |
| patrickeast | erlon: which isn't so weird since it is for all of cinder, not specific to the backend | 18:51 |
| jungleboyj | hemna_: That is so weird that they are using hex values backwards in the formatting. | 18:53 |
| erlon | patrickeast: mhm, makes sense, what does not make sense is that default.image_volume_cache_enabled =True is not inherited in the backend confs, if they are not set | 18:53 |
| patrickeast | erlon: oh, yea there was an email thread about that a while ago | 18:53 |
| patrickeast | erlon: so the problem is that "DEFAULT" is kind of a bad name, it isn't the *default* for driver specific things that can be modified | 18:54 |
| patrickeast | erlon: its just that we let you define a single backend in the default section, or you can define them in their own for multi-backend | 18:54 |
| hemna_ | jungleboyj, https://gist.github.com/WaltHP/f7b50b9ad6e415262aa0 | 18:54 |
| patrickeast | erlon: but if you do multi-backend you have to put all of their specific config stuff in the stanza | 18:55 |
| erlon | patrickeast: so, whenever, the driver have the same option as default, it will overlap default, right? | 18:55 |
| erlon | patrickeast: mhm | 18:55 |
| patrickeast | erlon: they are treated differently, like one is 'default.foo' and the other is 'mydriver.foo', and anything the driver/volume manager see is the 'mydriver.X' one | 18:55 |
| patrickeast | erlon: iff multi-backend is enabled | 18:55 |
| erlon | patrickeast: hmm | 18:56 |
| jungleboyj | hemna_: Intersting. | 18:56 |
| erlon | patrickeast: confusing | 18:56 |
| patrickeast | erlon: there was talk about having some kind of like 'actually-default-inherited' section | 18:56 |
| patrickeast | erlon: but imo the whole thing is kind of borked | 18:56 |
| erlon | patrickeast: yep | 18:57 |
| patrickeast | erlon: ah yea, here it is this thread http://lists.openstack.org/pipermail/openstack-dev/2015-September/074972.html | 18:57 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Migration: take the direct path for the source volume https://review.openstack.org/250220 | 19:00 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Migration: take the direct path for the source volume https://review.openstack.org/250220 | 19:00 |
| *** haomaiwang has quit IRC | 19:01 | |
| *** vincent_hou has joined #openstack-cinder | 19:01 | |
| *** haomaiwang has joined #openstack-cinder | 19:01 | |
| *** lpetrut has quit IRC | 19:01 | |
| vincent_hou | eharney: Hi Eric. Please help me review https://review.openstack.org/#/c/250220/ again. Thank you. | 19:03 |
| *** harlowja has joined #openstack-cinder | 19:04 | |
| erlon | patrickeast: it looks like it worked | 19:04 |
| *** mudassirlatif_ has joined #openstack-cinder | 19:04 | |
| openstackgerrit | Kendall Nelson proposed openstack/os-brick: Lun id's > 255 should be converted to hex https://review.openstack.org/277606 | 19:05 |
| erlon | patrickeast: one think weird, I created 2 volumes at the same time, before it had anything in cache. Now I have 2 images in cache + the 2 I created | 19:05 |
| erlon | patrickeast: is the code handling this? | 19:06 |
| jungleboyj | hemna_: So quickly pounding out C. I am far too out of practice with such things. :-) | 19:06 |
| *** mudassirlatif has quit IRC | 19:07 | |
| *** mudassirlatif_ is now known as mudassirlatif | 19:07 | |
| patrickeast | erlon: yep, the cache doesn't do any locking or checks | 19:09 |
| patrickeast | erlon: so it will just let you cache teh same thing more than once | 19:09 |
| patrickeast | erlon: eventually (if its size is capped) one would be evicted | 19:09 |
| patrickeast | erlon: since it would always pick one over the other | 19:09 |
| erlon | patrickeast: hmm, taht doesn't look right, I should be just a matter of checking any caches with the same image id and not caching then doesnt it? | 19:10 |
| erlon | patrickeast: otherwise, the user will always have to create 1 volume from image to avoid waste | 19:10 |
| *** angela-s has joined #openstack-cinder | 19:11 | |
| patrickeast | erlon: unfortunately it isn't so simple, to make it work you would need synchronization and make other requests block while the cache entry is being created | 19:11 |
| erlon | patrickeast: hmm | 19:11 |
| patrickeast | erlon: well, the assumption is that the cache probably wont be unlimited in size | 19:11 |
| hemna_ | jungleboyj, it had been years for me. | 19:11 |
| hemna_ | but it wasn't hard to do :) | 19:11 |
| patrickeast | erlon: so for a time there might be duplicate entries, but they won't *always* eat up space | 19:11 |
| patrickeast | erlon: its a trade off | 19:12 |
| erlon | patrickeast: that if the backend clones without eating the space | 19:12 |
| erlon | patrickeast: but for drivers like LVM i think it will eat | 19:12 |
| patrickeast | erlon: yea, they should limit the cache size | 19:13 |
| erlon | patrickeast: the caching start before the image cloning right? | 19:13 |
| *** akshai has quit IRC | 19:13 | |
| patrickeast | erlon: a proposed extention to this was to add a cache expiration on entries, so they would go away after some time of not being used | 19:13 |
| *** akshai has joined #openstack-cinder | 19:13 | |
| patrickeast | erlon: eh yea sort of, cinder knows it is going to be using the cache before it clones | 19:14 |
| patrickeast | erlon: but the cache entry being created is the last step | 19:14 |
| erlon | patrickeast: I thinkg the right way would be add a lock before creating a lock | 19:14 |
| erlon | patrickeast: we have a use case where our client needs to create 200 images at same time | 19:15 |
| erlon | patrickeast: that will be a problem | 19:15 |
| patrickeast | erlon: with that image having never seen use? | 19:15 |
| patrickeast | erlon: for like a POC or demo testing the system those scenarios suck | 19:15 |
| patrickeast | erlon: for like "normal" usage the cache is effective | 19:15 |
| erlon | patrickeast: I don't know, but, if so, the admin will need to manually create a volume to cache taht | 19:16 |
| patrickeast | erlon: it wasn't designed with kick-ass demo's of 0 to thousands in mind, if you want to do that just pre-cache it | 19:16 |
| erlon | patrickeast: I don't know how often the images will change | 19:16 |
| patrickeast | erlon: but by all means, if you have a need for it i'm supportive of adding a configurable way to make it lock and prevent other creates until the cache entry is done | 19:17 |
| *** vincent_hou has quit IRC | 19:17 | |
| patrickeast | erlon: just didn't do it initially because i dont have a need | 19:18 |
| *** mudassirlatif has quit IRC | 19:18 | |
| erlon | patrickeast: mhm, I think this will work the way it is, thogth it might require som manual work in that case, the lock would be a improvment | 19:19 |
| patrickeast | erlon: heh, i think improvement is debatable, for instance, if you only did a 'bulk' create with 2 or 3 you force them to become serial, which may use less bandwidth but probably won't be any faster than just downloading/attaching cloning a few images/volumes | 19:22 |
| patrickeast | erlon: but definitely if 200 at a time is the use-case | 19:22 |
| *** rlrossit has quit IRC | 19:24 | |
| *** boris-42 has joined #openstack-cinder | 19:26 | |
| *** rlrossit_ has joined #openstack-cinder | 19:28 | |
| *** funeutron has joined #openstack-cinder | 19:29 | |
| *** baumann has quit IRC | 19:29 | |
| *** lpetrut has joined #openstack-cinder | 19:30 | |
| eharney | e0ne: can you look at the dependent patches in that series too | 19:34 |
| eharney | https://review.openstack.org/#/c/278595/ https://review.openstack.org/#/c/278594/ | 19:34 |
| e0ne | eharney: hi. do you mean profiler-related patches? | 19:34 |
| eharney | e0ne: yes, there are two easy ones before it | 19:35 |
| openstackgerrit | Merged openstack/cinder: Updated from global requirements https://review.openstack.org/277228 | 19:35 |
| *** alonma has joined #openstack-cinder | 19:35 | |
| *** baumann has joined #openstack-cinder | 19:37 | |
| *** mudassirlatif has joined #openstack-cinder | 19:38 | |
| *** Suyash has joined #openstack-cinder | 19:38 | |
| *** alonma has quit IRC | 19:40 | |
| e0ne | eharney: I'm not sure that https://review.openstack.org/#/c/278594/1/cinder/tests/unit/test_service.py is a good solution | 19:40 |
| eharney | e0ne: how so? | 19:41 |
| e0ne | test_service_listen_port=0 by default | 19:41 |
| e0ne | will it work correct w/o root permissions? | 19:41 |
| eharney | port=0 means pick something dynamically | 19:41 |
| eharney | many of the tests already work that way | 19:41 |
| e0ne | oh... great! I didn't know it | 19:41 |
| eharney | they use %s_listen_port where %s is the service name | 19:42 |
| *** mylu has quit IRC | 19:42 | |
| eharney | so most use test_service_listen_port from the top of the file, but these change the service name, so they need a different cfg opt | 19:42 |
| *** alonma has joined #openstack-cinder | 19:43 | |
| openstackgerrit | Kendall Nelson proposed openstack/os-brick: Lun id's > 255 should be converted to hex https://review.openstack.org/277606 | 19:43 |
| *** mylu has joined #openstack-cinder | 19:44 | |
| *** mudassirlatif has quit IRC | 19:44 | |
| e0ne | eharney: now I understand how it works. thanks for the explanation | 19:44 |
| e0ne | eharney: IMO, we have to move these tests to integration directory | 19:45 |
| e0ne | eharney: it's not real unit tests | 19:45 |
| eharney | e0ne: yeah, tests that fire up a service and listen on ports are not unit tests :) | 19:45 |
| thingee | smcginnis: do you know the reason for the bitwise usage here https://review.openstack.org/#/c/263081/3/cinder/volume/manager.py on line 1627 | 19:47 |
| e0ne | eharney: added comment | 19:47 |
| *** alonma has quit IRC | 19:47 | |
| *** alonma has joined #openstack-cinder | 19:51 | |
| e0ne | eharney: we talk about functional tests a lot. I hope, we'll implement them soon | 19:51 |
| eharney | e0ne: i hope so too, but i'm going to keep fixing the unit tests | 19:52 |
| e0ne | eharney: sure, I won't block it | 19:52 |
| *** alonma has quit IRC | 19:55 | |
| *** ivase has joined #openstack-cinder | 19:55 | |
| *** mudassirlatif has joined #openstack-cinder | 19:58 | |
| *** mylu has quit IRC | 19:59 | |
| *** ociuhandu has quit IRC | 19:59 | |
| e0ne | thingee: it's common way to concatenate to sets | 19:59 |
| smcginnis | thingee: Looking... | 20:00 |
| thingee | huh, first time I've seen it if it's common :) | 20:00 |
| thingee | makes sense though | 20:00 |
| *** haomaiwang has quit IRC | 20:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 20:01 | |
| smcginnis | Hmm, guessing that was supposed to be an "or", not a bitwise or. | 20:01 |
| erlon | guys, quick question, Im booting some VMs from images/creating volumes, but my hypervisor is limiting the number of VMs based on the local disk consumed. Is there a way/config to change that? | 20:01 |
| e0ne | thingee: afaik, it's equals to 'union' method. not sure what is faster | 20:01 |
| smcginnis | e0ne: Oh, yep, you're right. | 20:02 |
| erlon | I can boot 21VMs of 10GB disk, the compute host has 211GB, but after the creation it still has +-200 free space | 20:02 |
| thingee | smcginnis: right that's why I was asking. | 20:02 |
| erlon | as spected | 20:02 |
| thingee | e0ne: yea, not concerned which is faster. just curious if that was meant to be bitwise or raelly an || | 20:02 |
| smcginnis | thingee: Looks intentional. Adding 'host' into the set of keys that get removed in the next line. | 20:03 |
| *** chhavi has quit IRC | 20:04 | |
| thingee | smcginnis: yeah, wouldn't make sense otherwise | 20:04 |
| thingee | thanks all | 20:04 |
| smcginnis | thingee: Yeah, good to check. | 20:04 |
| *** mudassirlatif has quit IRC | 20:06 | |
| *** vgridnev has joined #openstack-cinder | 20:07 | |
| *** timcl has joined #openstack-cinder | 20:08 | |
| patrickeast | erlon: if you are booting from volumes you can create a flavor with 0 root disk size so it won't try and block the vm placement based on local disk on hypervisor | 20:08 |
| *** esker has joined #openstack-cinder | 20:09 | |
| erlon | patrickeast: hmmm, good idea | 20:09 |
| *** timcl1 has joined #openstack-cinder | 20:17 | |
| *** timcl has quit IRC | 20:17 | |
| *** mudassirlatif has joined #openstack-cinder | 20:23 | |
| *** rcernin has quit IRC | 20:24 | |
| openstackgerrit | Merged openstack/cinder: Support ZeroMQ messaging driver in cinder https://review.openstack.org/271848 | 20:26 |
| *** ebalduf has quit IRC | 20:28 | |
| *** ebalduf has joined #openstack-cinder | 20:29 | |
| *** funeutron has quit IRC | 20:32 | |
| *** timcl1 has quit IRC | 20:38 | |
| *** ebalduf has quit IRC | 20:39 | |
| *** julim has quit IRC | 20:40 | |
| *** ebalduf has joined #openstack-cinder | 20:40 | |
| scottda | smcginnis: or anyone, will you please go to the Cinder Youtube channel and see if my test video worked? | 20:43 |
| scottda | smcginnis: nevermind. I can see now that it didn't. I'm not sure what is missing. | 20:45 |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - get iscsi ip from port in existing MV https://review.openstack.org/245997 | 20:45 |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - necessary updates for CG changes https://review.openstack.org/268223 | 20:46 |
| smcginnis | scottda: I should see something once it does work. | 20:46 |
| scottda | Is that some kind of Zen koan | 20:46 |
| scottda | ? | 20:46 |
| smcginnis | Hah, I guess so. | 20:46 |
| smcginnis | I'll go back to zazen for now. | 20:46 |
| *** alonma has joined #openstack-cinder | 20:51 | |
| *** rlrossit_ has quit IRC | 20:52 | |
| *** e0ne has quit IRC | 20:53 | |
| mc_nair | jgriffith: know you've been sickly, but just a heads up I have a first pass of the separated NestedQuota driver here - https://review.openstack.org/#/c/274825/. Tried to make it as close as I could to the discussion we had (for a first pass) | 20:53 |
| *** alonma has quit IRC | 20:56 | |
| scottda | smcginnis: OK, well I have a channel called 'openstack-cinder' and then there's the channel 'openstack-cinder' and they are not connected..... | 20:56 |
| *** sid14726 has joined #openstack-cinder | 20:57 | |
| *** ociuhandu has joined #openstack-cinder | 20:57 | |
| scottda | So, I'm not going to send out any info on a microversion demo until I figure this out, but I'm also not going to spend any time on it any time soon. | 20:57 |
| *** alonma has joined #openstack-cinder | 20:57 | |
| smcginnis | scottda: OK, sounds good. Maybe hemna_ can help out. | 21:00 |
| smcginnis | scottda: I know he was able to post something recorded under his own into the channel, so maybe that will work for now. | 21:00 |
| *** haomaiwa_ has quit IRC | 21:01 | |
| scottda | Yeah. Might be simpler for me to just record something and post it. | 21:01 |
| *** haomaiwang has joined #openstack-cinder | 21:01 | |
| apoorvad | eharney: Thanks for checking out. I have replied to your comment on https://review.openstack.org/#/c/268370/. | 21:01 |
| scottda | Which I seem to be able to do with my personal Hangout account, just not to openstack-cinder. Except when I renamed my channel to open-stack cinder. Which seems bad and wrong... | 21:01 |
| *** alonma has quit IRC | 21:02 | |
| *** notmorgan is now known as morganfainberg | 21:02 | |
| *** baumann has quit IRC | 21:03 | |
| *** alonma has joined #openstack-cinder | 21:04 | |
| *** merooney has quit IRC | 21:06 | |
| eharney | apoorvad: i replied there... but don't see how it being short-lived helps anything. You shouldn't be using native threads here | 21:06 |
| apoorvad | eharney alright, I will work on it | 21:07 |
| *** alonma has quit IRC | 21:08 | |
| *** raildo is now known as raildo-afk | 21:09 | |
| *** morganfainberg is now known as notmorgan | 21:12 | |
| openstackgerrit | Michael Rowden proposed openstack/cinder: Correcting thin provisioning behavior https://review.openstack.org/275408 | 21:16 |
| *** NightKhaos has quit IRC | 21:19 | |
| *** edtubill has joined #openstack-cinder | 21:19 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder-specs: Deleting volume metadata keys with a single request https://review.openstack.org/279202 | 21:21 |
| *** NightKhaos has joined #openstack-cinder | 21:21 | |
| *** flip214 has quit IRC | 21:23 | |
| *** esker has quit IRC | 21:23 | |
| *** flip214 has joined #openstack-cinder | 21:24 | |
| *** flip214 has quit IRC | 21:24 | |
| *** flip214 has joined #openstack-cinder | 21:24 | |
| *** esker has joined #openstack-cinder | 21:26 | |
| *** jungleboyj has quit IRC | 21:29 | |
| *** esker has quit IRC | 21:31 | |
| *** rlrossit has joined #openstack-cinder | 21:38 | |
| *** mylu has joined #openstack-cinder | 21:39 | |
| *** akerr has quit IRC | 21:41 | |
| *** mylu has quit IRC | 21:42 | |
| *** sid14726 has quit IRC | 21:42 | |
| *** cknight has quit IRC | 21:43 | |
| *** esker has joined #openstack-cinder | 21:46 | |
| openstackgerrit | Vivek Dhayaal proposed openstack/cinder: Support ZeroMQ messaging in cinder multibackend https://review.openstack.org/277113 | 21:50 |
| *** eharney has quit IRC | 21:52 | |
| *** ociuhandu has quit IRC | 21:52 | |
| *** ociuhandu has joined #openstack-cinder | 21:53 | |
| *** timcl has joined #openstack-cinder | 21:55 | |
| *** jgregor has quit IRC | 21:57 | |
| *** garthb has quit IRC | 21:57 | |
| *** garthb has joined #openstack-cinder | 21:58 | |
| *** jgregor has joined #openstack-cinder | 21:58 | |
| *** baumann has joined #openstack-cinder | 22:00 | |
| *** nkrinner has quit IRC | 22:00 | |
| *** delatte has quit IRC | 22:00 | |
| *** haomaiwang has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-cinder | 22:01 | |
| *** esker has quit IRC | 22:02 | |
| *** chlong has quit IRC | 22:03 | |
| *** ebalduf has quit IRC | 22:03 | |
| *** esker has joined #openstack-cinder | 22:03 | |
| *** bwallis has joined #openstack-cinder | 22:06 | |
| *** crose has quit IRC | 22:06 | |
| *** mudassirlatif has quit IRC | 22:08 | |
| *** timcl has left #openstack-cinder | 22:09 | |
| *** mudassirlatif has joined #openstack-cinder | 22:10 | |
| *** mudassirlatif has quit IRC | 22:12 | |
| *** amoturi has left #openstack-cinder | 22:14 | |
| *** mudassirlatif has joined #openstack-cinder | 22:15 | |
| *** tpsilva has quit IRC | 22:16 | |
| *** sheel has quit IRC | 22:17 | |
| *** mudassirlatif has quit IRC | 22:17 | |
| *** esker has quit IRC | 22:17 | |
| *** cknight has joined #openstack-cinder | 22:18 | |
| *** jgregor has quit IRC | 22:19 | |
| *** knikolla has quit IRC | 22:19 | |
| *** vgridnev has quit IRC | 22:19 | |
| *** mudassirlatif has joined #openstack-cinder | 22:21 | |
| *** mragupat has quit IRC | 22:23 | |
| *** dims has quit IRC | 22:23 | |
| *** vincent_hou has joined #openstack-cinder | 22:24 | |
| *** porrua has quit IRC | 22:24 | |
| *** dims has joined #openstack-cinder | 22:26 | |
| *** gouthamr has quit IRC | 22:28 | |
| *** sgotliv has joined #openstack-cinder | 22:28 | |
| *** bwallis has quit IRC | 22:28 | |
| *** mragupat has joined #openstack-cinder | 22:29 | |
| *** jungleboyj has joined #openstack-cinder | 22:29 | |
| *** fthiagogv has quit IRC | 22:31 | |
| *** alonma has joined #openstack-cinder | 22:31 | |
| *** sid14726 has joined #openstack-cinder | 22:32 | |
| *** mudassirlatif has quit IRC | 22:32 | |
| *** mudassirlatif has joined #openstack-cinder | 22:32 | |
| openstackgerrit | Angela Smith proposed openstack/cinder-specs: Spec for Brocade Zone Driver Virtual Fabrics support https://review.openstack.org/144389 | 22:32 |
| *** alonma has quit IRC | 22:36 | |
| *** mudassirlatif has quit IRC | 22:37 | |
| *** alonma has joined #openstack-cinder | 22:38 | |
| *** baumann has quit IRC | 22:40 | |
| *** vincent_hou has quit IRC | 22:41 | |
| *** salv-orlando has joined #openstack-cinder | 22:42 | |
| *** mudassirlatif has joined #openstack-cinder | 22:42 | |
| *** alonma has quit IRC | 22:42 | |
| *** cdelatte has joined #openstack-cinder | 22:42 | |
| *** baumann has joined #openstack-cinder | 22:43 | |
| *** ebalduf has joined #openstack-cinder | 22:43 | |
| *** diablo_rojo has quit IRC | 22:44 | |
| *** salv-orl_ has quit IRC | 22:44 | |
| *** mylu has joined #openstack-cinder | 22:45 | |
| *** dustins has quit IRC | 22:46 | |
| *** cknight1 has joined #openstack-cinder | 22:46 | |
| angela-s | hemna: argh! let me remove that entire section. the author was rambling... | 22:46 |
| hemna_ | angela-s, sorry :( | 22:47 |
| hemna_ | I'm ok with it after that | 22:48 |
| angela-s | hemna_: the author was trying to say that there is a performance optimization that can be done, but this spec is not the place for that | 22:48 |
| hemna_ | yah that's true. | 22:48 |
| *** JoseMello has quit IRC | 22:48 | |
| hemna_ | if there is a perf. impact it should be noted fwiw | 22:48 |
| *** cknight has quit IRC | 22:49 | |
| angela-s | hemna_: there is no performance impact with the addition of the VF support | 22:49 |
| openstackgerrit | Apoorva Deshpande proposed openstack/cinder: Tintri image cache cleanup https://review.openstack.org/268370 | 22:50 |
| hemna_ | ok cool, just nuke it then :) | 22:50 |
| *** ebalduf has quit IRC | 22:50 | |
| openstackgerrit | Angela Smith proposed openstack/cinder-specs: Spec for Brocade Zone Driver Virtual Fabrics support https://review.openstack.org/144389 | 22:51 |
| *** arch-nemesis has quit IRC | 22:53 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls https://review.openstack.org/252458 | 22:53 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec in create volume calls to ovo https://review.openstack.org/246600 | 22:53 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO https://review.openstack.org/253033 | 22:53 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype) https://review.openstack.org/257305 | 22:53 |
| *** baumann has quit IRC | 22:53 | |
| *** mriedem has quit IRC | 22:53 | |
| *** mudassirlatif has quit IRC | 22:54 | |
| *** baumann has joined #openstack-cinder | 22:55 | |
| *** xyang1 has quit IRC | 22:58 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwang has joined #openstack-cinder | 23:01 | |
| *** mudassirlatif has joined #openstack-cinder | 23:01 | |
| *** cknight1 has quit IRC | 23:01 | |
| openstackgerrit | Angela Smith proposed openstack/cinder-specs: Spec for Brocade Zone Driver Virtual Fabrics support https://review.openstack.org/144389 | 23:04 |
| *** martyturner has quit IRC | 23:07 | |
| *** vincent_hou has joined #openstack-cinder | 23:08 | |
| *** rlrossit has quit IRC | 23:09 | |
| *** baumann has quit IRC | 23:09 | |
| *** wN has quit IRC | 23:10 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Lazy load a project's default values https://review.openstack.org/279304 | 23:10 |
| *** mragupat has quit IRC | 23:11 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Lazy load a project quotas's default values https://review.openstack.org/279304 | 23:12 |
| *** lpetrut has quit IRC | 23:13 | |
| *** edtubill has quit IRC | 23:15 | |
| *** wN has joined #openstack-cinder | 23:15 | |
| *** wN has joined #openstack-cinder | 23:15 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Update replication to v2.1 https://review.openstack.org/279307 | 23:16 |
| *** sid14726 has quit IRC | 23:18 | |
| openstackgerrit | Merged openstack/python-cinderclient: Put py34 first in the env order of tox https://review.openstack.org/266761 | 23:21 |
| *** vincent_hou has quit IRC | 23:21 | |
| *** vincent_hou has joined #openstack-cinder | 23:22 | |
| *** alonma has joined #openstack-cinder | 23:23 | |
| *** jungleboyj has quit IRC | 23:23 | |
| *** alonma has quit IRC | 23:28 | |
| jgriffith | aorourke: you around? | 23:28 |
| aorourke | jgriffith yep just reading your comment right now | 23:28 |
| jgriffith | aorourke: cool, just realized might be easier to talk real-time here ;) | 23:28 |
| aorourke | jgriffith, that makes sense. When i first read through the spec i was under the assumption there was something where all volumes could be replicated on the backend by default | 23:29 |
| *** alonma has joined #openstack-cinder | 23:29 | |
| aorourke | but now i understand it is volume type driven still | 23:29 |
| jgriffith | aorourke: Swanson Remember we have things like default volume types too :) | 23:29 |
| Swanson | lies | 23:30 |
| jgriffith | :) | 23:30 |
| *** mudassirlatif has quit IRC | 23:30 | |
| *** chlong has joined #openstack-cinder | 23:30 | |
| Swanson | jgriffith, I'm happiest with using volume_types | 23:31 |
| jgriffith | Swanson: ditto! | 23:31 |
| aorourke | same here, i just misunderstood the intention at first. this works though | 23:32 |
| *** alonma has quit IRC | 23:33 | |
| openstackgerrit | Michael Rowden proposed openstack/cinder: Correcting thin provisioning behavior https://review.openstack.org/275408 | 23:34 |
| Swanson | jgriffith, Now +2/+A your own patch and get this rollin. | 23:34 |
| aorourke | jgriffith, Swanson while we are on the subject, do you think there is a need to list potential targets to fail-over to still? I think that if we support multiple replication devices, there should be some mechanism to list fail-over options. | 23:36 |
| hemna_ | someone needs a +A ? | 23:36 |
| hemna_ | :P | 23:36 |
| Swanson | aorourke, jgriffith, I should think so. Even if they are just being pulled out of options... | 23:37 |
| Swanson | At least give the admin the name to use for the secondary. | 23:37 |
| jgriffith | Swanson: LOL | 23:38 |
| jgriffith | aorourke: my opinion was not really | 23:38 |
| jgriffith | aorourke: it's in the config file which the admin set up | 23:38 |
| jgriffith | aorourke: and we do store the active-backend-id in the db | 23:39 |
| jgriffith | aorourke: although it would be useful I guess so you can do a "failover to" | 23:39 |
| jgriffith | aorourke: I'll work that back in | 23:40 |
| aorourke | jgriffith, i think it would be helpful. Then it seems the admin would have to pass in a host or a backend name then to get the targets for that backend | 23:41 |
| jgriffith | aorourke: fair enough | 23:41 |
| *** lcurtis has quit IRC | 23:41 | |
| openstackgerrit | Merged openstack/cinder: Fix image volume creation error https://review.openstack.org/263081 | 23:42 |
| *** mudassirlatif has joined #openstack-cinder | 23:45 | |
| openstackgerrit | Merged openstack/cinder: 3PAR: Create consistency group from source CG https://review.openstack.org/274908 | 23:47 |
| *** marcusvrn_ has quit IRC | 23:47 | |
| *** ivase has quit IRC | 23:49 | |
| *** yhayashi has joined #openstack-cinder | 23:51 | |
| *** mylu has quit IRC | 23:52 | |
| *** angela-s has quit IRC | 23:53 | |
| *** mylu has joined #openstack-cinder | 23:53 | |
| *** mudassirlatif has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!