| *** scottda_ has quit IRC | 00:04 | |
| *** Manshoon has joined #openstack-cinder | 00:04 | |
| *** scottda_ has joined #openstack-cinder | 00:08 | |
| *** ebalduf has joined #openstack-cinder | 00:09 | |
| *** diegows has joined #openstack-cinder | 00:12 | |
| *** Manshoon has quit IRC | 00:13 | |
| *** ebalduf has quit IRC | 00:13 | |
| *** markvoelker has joined #openstack-cinder | 00:16 | |
| *** Manshoon has joined #openstack-cinder | 00:19 | |
| *** scottda_ has quit IRC | 00:20 | |
| *** markvoelker has quit IRC | 00:20 | |
| *** scottda_ has joined #openstack-cinder | 00:23 | |
| *** scottda_ has quit IRC | 00:27 | |
| *** Manshoon has quit IRC | 00:32 | |
| *** Yogi1 has joined #openstack-cinder | 00:48 | |
| *** markvoelker has joined #openstack-cinder | 00:50 | |
| *** markvoelker has quit IRC | 00:55 | |
| *** scottda_ has joined #openstack-cinder | 01:01 | |
| *** scottda_ has quit IRC | 01:04 | |
| *** harlowja_at_home has joined #openstack-cinder | 01:06 | |
| *** ebalduf has joined #openstack-cinder | 01:10 | |
| *** boris-42 has joined #openstack-cinder | 01:13 | |
| *** ebalduf has quit IRC | 01:14 | |
| *** kbyrne has quit IRC | 01:23 | |
| *** kbyrne has joined #openstack-cinder | 01:25 | |
| *** ho has joined #openstack-cinder | 01:25 | |
| *** markvoelker has joined #openstack-cinder | 01:51 | |
| *** Yogi1 has quit IRC | 01:55 | |
| *** markvoelker has quit IRC | 01:57 | |
| *** scottda_ has joined #openstack-cinder | 01:59 | |
| *** markvoelker has joined #openstack-cinder | 02:00 | |
| *** diegows has quit IRC | 02:04 | |
| *** ebalduf has joined #openstack-cinder | 02:11 | |
| *** ebalduf has quit IRC | 02:15 | |
| *** markvoelker has quit IRC | 02:26 | |
| *** scottda_ has quit IRC | 02:26 | |
| *** markvoelker has joined #openstack-cinder | 02:27 | |
| *** scottda_ has joined #openstack-cinder | 02:29 | |
| *** markvoelker has quit IRC | 02:31 | |
| *** vilobhmm has quit IRC | 02:36 | |
| *** kaisers1 has joined #openstack-cinder | 02:47 | |
| *** kaisers has quit IRC | 02:48 | |
| *** scottda_ has quit IRC | 02:58 | |
| *** scottda_ has joined #openstack-cinder | 03:02 | |
| *** scottda_ has quit IRC | 03:04 | |
| *** mriedem has left #openstack-cinder | 03:06 | |
| *** jcru has joined #openstack-cinder | 03:10 | |
| *** ebalduf has joined #openstack-cinder | 03:11 | |
| *** ebalduf has quit IRC | 03:16 | |
| *** tellesnobrega_ has joined #openstack-cinder | 03:31 | |
| *** tellesnobrega_ has quit IRC | 03:31 | |
| *** annashen has joined #openstack-cinder | 03:47 | |
| *** pdeore_ has joined #openstack-cinder | 03:52 | |
| *** harlowja_at_home has quit IRC | 03:53 | |
| *** pdeore_ has quit IRC | 04:00 | |
| *** garthb has joined #openstack-cinder | 04:11 | |
| *** garthb_ has joined #openstack-cinder | 04:11 | |
| *** garthb_ has quit IRC | 04:12 | |
| *** garthb has quit IRC | 04:12 | |
| *** ebalduf has joined #openstack-cinder | 04:12 | |
| *** vilobhmm has joined #openstack-cinder | 04:12 | |
| *** Mandell has joined #openstack-cinder | 04:15 | |
| *** ebalduf has quit IRC | 04:16 | |
| *** deepakcs has joined #openstack-cinder | 04:19 | |
| *** rwsu-afk has quit IRC | 04:19 | |
| *** annashen has quit IRC | 04:22 | |
| *** annashen has joined #openstack-cinder | 04:24 | |
| *** annashen has quit IRC | 04:24 | |
| *** annashen has joined #openstack-cinder | 04:27 | |
| *** vilobhmm1 has joined #openstack-cinder | 04:28 | |
| *** BharatK has joined #openstack-cinder | 04:28 | |
| *** vilobhmm has quit IRC | 04:30 | |
| *** bswartz has quit IRC | 04:34 | |
| *** bswartz has joined #openstack-cinder | 04:35 | |
| *** rushiagr_away is now known as rushiagr | 04:56 | |
| *** deepakcs has quit IRC | 05:00 | |
| *** pradipta has joined #openstack-cinder | 05:12 | |
| *** ebalduf has joined #openstack-cinder | 05:13 | |
| *** ebalduf has quit IRC | 05:17 | |
| *** jcru has quit IRC | 05:18 | |
| *** annashen has quit IRC | 05:31 | |
| *** anshul has joined #openstack-cinder | 05:41 | |
| *** anshul has quit IRC | 05:42 | |
| *** anshul has joined #openstack-cinder | 05:43 | |
| *** pcaruana has quit IRC | 05:45 | |
| *** Ilja has joined #openstack-cinder | 05:57 | |
| *** bkopilov has quit IRC | 06:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/152401 | 06:06 |
|---|---|---|
| *** ankit_ag has joined #openstack-cinder | 06:08 | |
| *** Tross has joined #openstack-cinder | 06:10 | |
| *** ebalduf has joined #openstack-cinder | 06:14 | |
| *** ebalduf has quit IRC | 06:18 | |
| *** bkopilov has joined #openstack-cinder | 06:19 | |
| *** rushiagr is now known as rushiagr_away | 06:20 | |
| *** coolsvap_ is now known as coolsvap | 06:27 | |
| *** Lee1092 has joined #openstack-cinder | 06:27 | |
| openstackgerrit | Anish Bhatt proposed openstack/cinder: Improve error handling in refactored Tgt driver https://review.openstack.org/154713 | 06:41 |
| *** nshaikh has joined #openstack-cinder | 06:41 | |
| *** nshaikh has quit IRC | 06:44 | |
| *** nshaikh has joined #openstack-cinder | 06:44 | |
| *** deepakcs has joined #openstack-cinder | 06:45 | |
| *** navid__ has joined #openstack-cinder | 06:46 | |
| *** nshaikh has left #openstack-cinder | 06:47 | |
| *** nshaikh has joined #openstack-cinder | 06:47 | |
| *** navid__ has quit IRC | 06:48 | |
| *** cnesa has joined #openstack-cinder | 07:00 | |
| *** vilobhmm1 has quit IRC | 07:01 | |
| *** afazekas_ has joined #openstack-cinder | 07:06 | |
| *** vilobhmm has joined #openstack-cinder | 07:14 | |
| *** ebalduf has joined #openstack-cinder | 07:14 | |
| *** ebalduf has quit IRC | 07:19 | |
| *** rushiagr_away is now known as rushiagr | 07:34 | |
| *** vilobhmm1 has joined #openstack-cinder | 07:42 | |
| *** alecv has joined #openstack-cinder | 07:44 | |
| *** vilobhmm has quit IRC | 07:44 | |
| *** Ilja has quit IRC | 07:46 | |
| *** bkopilov has quit IRC | 07:49 | |
| *** bkopilov has joined #openstack-cinder | 07:49 | |
| *** e0ne has joined #openstack-cinder | 07:49 | |
| *** e0ne is now known as e0ne_ | 07:49 | |
| *** Miouge has joined #openstack-cinder | 07:51 | |
| *** e0ne_ is now known as e0ne | 07:53 | |
| *** Ilja has joined #openstack-cinder | 07:57 | |
| *** kaisers1 has quit IRC | 07:58 | |
| *** Mandell has quit IRC | 08:00 | |
| *** kaisers has joined #openstack-cinder | 08:05 | |
| *** kaisers has quit IRC | 08:10 | |
| *** theanalyst has quit IRC | 08:14 | |
| *** kaisers has joined #openstack-cinder | 08:14 | |
| *** theanalyst has joined #openstack-cinder | 08:15 | |
| *** ebalduf has joined #openstack-cinder | 08:15 | |
| *** vilobhmm1 has quit IRC | 08:16 | |
| *** jcru has joined #openstack-cinder | 08:19 | |
| *** ebalduf has quit IRC | 08:19 | |
| *** jcru has quit IRC | 08:24 | |
| *** ndipanov has joined #openstack-cinder | 08:24 | |
| *** enterprisedc has quit IRC | 08:31 | |
| *** Miouge_ has joined #openstack-cinder | 08:37 | |
| *** Miouge has quit IRC | 08:38 | |
| *** Miouge_ is now known as Miouge | 08:38 | |
| *** ronis has joined #openstack-cinder | 08:42 | |
| *** sgotliv has joined #openstack-cinder | 08:45 | |
| *** TobiasE has joined #openstack-cinder | 08:49 | |
| *** emaisne has joined #openstack-cinder | 08:54 | |
| *** rushiagr is now known as rushiagr_away | 08:54 | |
| *** jistr has joined #openstack-cinder | 08:55 | |
| *** jordanP has joined #openstack-cinder | 08:56 | |
| *** e0ne has quit IRC | 08:57 | |
| *** dulek has joined #openstack-cinder | 08:59 | |
| *** aarefiev has quit IRC | 09:06 | |
| *** aarefiev has joined #openstack-cinder | 09:11 | |
| *** emaisne_ has joined #openstack-cinder | 09:16 | |
| *** ebalduf has joined #openstack-cinder | 09:16 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Backup object https://review.openstack.org/157085 | 09:18 |
| *** emaisne has quit IRC | 09:19 | |
| *** emaisne_ has quit IRC | 09:20 | |
| *** ebalduf has quit IRC | 09:20 | |
| *** emaisne has joined #openstack-cinder | 09:23 | |
| openstackgerrit | Anish Bhatt proposed openstack/cinder: Improve error handling in refactored Tgt driver https://review.openstack.org/154713 | 09:36 |
| openstackgerrit | Anish Bhatt proposed openstack/cinder: Improve error handling in refactored Tgt driver https://review.openstack.org/154713 | 09:39 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Import only modules: H302 https://review.openstack.org/145780 | 09:46 |
| *** sgotliv has quit IRC | 09:47 | |
| *** emaisne has quit IRC | 09:51 | |
| *** rushiagr_away is now known as rushiagr | 09:58 | |
| *** e0ne has joined #openstack-cinder | 09:59 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add entry create and cast tasks to manage workflow https://review.openstack.org/139071 | 10:07 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix lvm manage existing volume https://review.openstack.org/156939 | 10:07 |
| *** sgotliv has joined #openstack-cinder | 10:12 | |
| *** sgotliv has quit IRC | 10:12 | |
| *** sgotliv has joined #openstack-cinder | 10:12 | |
| *** pdeore_ has joined #openstack-cinder | 10:16 | |
| *** ebalduf has joined #openstack-cinder | 10:17 | |
| *** marcusvrn has joined #openstack-cinder | 10:18 | |
| *** e0ne is now known as e0ne_ | 10:21 | |
| *** ebalduf has quit IRC | 10:21 | |
| *** marcusvrn1 has joined #openstack-cinder | 10:23 | |
| *** marcusvrn has quit IRC | 10:25 | |
| *** marcusvrn1 has quit IRC | 10:25 | |
| *** e0ne_ is now known as e0ne | 10:28 | |
| *** marcusvrn has joined #openstack-cinder | 10:29 | |
| *** sgotliv has quit IRC | 10:34 | |
| *** kbyrne has quit IRC | 10:49 | |
| *** kbyrne has joined #openstack-cinder | 10:49 | |
| *** sgotliv has joined #openstack-cinder | 10:49 | |
| *** lpetrut has joined #openstack-cinder | 10:53 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix lvm manage existing volume https://review.openstack.org/156939 | 10:56 |
| *** dulek_ has joined #openstack-cinder | 10:57 | |
| *** dulek has quit IRC | 11:00 | |
| *** kallebe has joined #openstack-cinder | 11:05 | |
| *** jaypipes has joined #openstack-cinder | 11:05 | |
| *** sgotliv has quit IRC | 11:09 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Skip vSAN for preallocated image download https://review.openstack.org/156904 | 11:09 |
| *** ebalduf has joined #openstack-cinder | 11:17 | |
| *** ebalduf has quit IRC | 11:22 | |
| *** cnesa has quit IRC | 11:27 | |
| *** aix has joined #openstack-cinder | 11:28 | |
| *** nellysmitt has joined #openstack-cinder | 11:29 | |
| *** diegows has joined #openstack-cinder | 11:35 | |
| *** ho has quit IRC | 11:36 | |
| openstackgerrit | nikeshmahalka proposed openstack/cinder: Add CHAP persistence to SCST target helper https://review.openstack.org/157356 | 11:39 |
| *** pradipta has quit IRC | 11:54 | |
| openstackgerrit | Kanchan Gupta proposed openstack/cinder: Checking name for volume creation in cinder. https://review.openstack.org/157714 | 11:59 |
| *** TobiasE1 has joined #openstack-cinder | 12:00 | |
| *** TobiasE has quit IRC | 12:00 | |
| *** Mandell has joined #openstack-cinder | 12:01 | |
| *** ganso_ has joined #openstack-cinder | 12:05 | |
| *** Mandell has quit IRC | 12:05 | |
| *** Miouge_ has joined #openstack-cinder | 12:06 | |
| *** Miouge has quit IRC | 12:06 | |
| *** Miouge_ is now known as Miouge | 12:06 | |
| *** Ilja has quit IRC | 12:07 | |
| *** Miouge has quit IRC | 12:09 | |
| *** emaisne has joined #openstack-cinder | 12:09 | |
| *** Ilja has joined #openstack-cinder | 12:09 | |
| *** Miouge has joined #openstack-cinder | 12:10 | |
| *** aviramb has quit IRC | 12:17 | |
| *** sgotliv has joined #openstack-cinder | 12:18 | |
| *** ebalduf has joined #openstack-cinder | 12:18 | |
| *** sgotliv has quit IRC | 12:18 | |
| *** sgotliv has joined #openstack-cinder | 12:19 | |
| *** Miouge has quit IRC | 12:20 | |
| *** Miouge has joined #openstack-cinder | 12:21 | |
| *** ebalduf has quit IRC | 12:23 | |
| *** cdelatte has joined #openstack-cinder | 12:25 | |
| *** Miouge has quit IRC | 12:27 | |
| *** lpetrut has quit IRC | 12:31 | |
| *** dulek_ has quit IRC | 12:43 | |
| *** TobiasE has joined #openstack-cinder | 12:43 | |
| *** Miouge has joined #openstack-cinder | 12:44 | |
| *** IanGovett has joined #openstack-cinder | 12:45 | |
| *** zigo has quit IRC | 12:52 | |
| *** marcusvrn1 has joined #openstack-cinder | 12:53 | |
| *** diegows has quit IRC | 12:53 | |
| *** zigo has joined #openstack-cinder | 12:56 | |
| *** marcusvrn has quit IRC | 12:57 | |
| *** lpabon has joined #openstack-cinder | 12:59 | |
| *** deepakcs has quit IRC | 13:00 | |
| *** BharatK has quit IRC | 13:01 | |
| *** dulek_ has joined #openstack-cinder | 13:07 | |
| *** dulek_ is now known as dulek | 13:18 | |
| *** akerr has joined #openstack-cinder | 13:19 | |
| *** ebalduf has joined #openstack-cinder | 13:19 | |
| *** bswartz has quit IRC | 13:20 | |
| *** ebalduf has quit IRC | 13:23 | |
| *** Ilja has quit IRC | 13:25 | |
| *** marcusvrn has joined #openstack-cinder | 13:28 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/157642 | 13:28 |
| *** marcusvrn1 has quit IRC | 13:31 | |
| *** bill_az has joined #openstack-cinder | 13:40 | |
| *** sgotliv has quit IRC | 13:43 | |
| *** sgotliv has joined #openstack-cinder | 13:43 | |
| *** timcl has joined #openstack-cinder | 13:44 | |
| *** Guest30952 has joined #openstack-cinder | 13:48 | |
| *** bswartz has joined #openstack-cinder | 13:50 | |
| *** jungleboyj_ has quit IRC | 13:53 | |
| *** dulek_ has joined #openstack-cinder | 13:59 | |
| *** dulek has quit IRC | 14:01 | |
| *** cdelatte has quit IRC | 14:04 | |
| *** ankit_ag has quit IRC | 14:08 | |
| *** kaufer has joined #openstack-cinder | 14:09 | |
| *** lpabon has quit IRC | 14:10 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Refactor initialize_connection unit tests https://review.openstack.org/155300 | 14:10 |
| *** diegows has joined #openstack-cinder | 14:12 | |
| *** Guest28285 is now known as mgagne | 14:13 | |
| *** mgagne has joined #openstack-cinder | 14:13 | |
| *** tsekiyama has joined #openstack-cinder | 14:13 | |
| *** dulek_ has quit IRC | 14:14 | |
| *** Yogi1 has joined #openstack-cinder | 14:16 | |
| *** dustins has joined #openstack-cinder | 14:19 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware:Use datastore selection logic in new module https://review.openstack.org/155313 | 14:19 |
| *** ebalduf has joined #openstack-cinder | 14:20 | |
| *** mtanino has quit IRC | 14:22 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Relocate volume to compliant datastore https://review.openstack.org/155702 | 14:23 |
| *** ebalduf has quit IRC | 14:24 | |
| *** primechuck has joined #openstack-cinder | 14:25 | |
| *** Tross has quit IRC | 14:27 | |
| *** anshul has quit IRC | 14:29 | |
| *** e0ne is now known as e0ne_ | 14:30 | |
| *** e0ne_ is now known as e0ne | 14:31 | |
| *** nshaikh has quit IRC | 14:32 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Skip vSAN for preallocated image download https://review.openstack.org/156904 | 14:37 |
| *** nlevinki has joined #openstack-cinder | 14:38 | |
| *** dulek has joined #openstack-cinder | 14:38 | |
| *** mriedem has joined #openstack-cinder | 14:40 | |
| *** Guest30952 is now known as annegentle | 14:42 | |
| *** rushiagr is now known as rushiagr_away | 14:42 | |
| *** timcl has quit IRC | 14:43 | |
| *** tsekiyama has quit IRC | 14:43 | |
| *** timcl has joined #openstack-cinder | 14:43 | |
| *** e0ne is now known as e0ne_ | 14:44 | |
| *** thangp has joined #openstack-cinder | 14:47 | |
| *** beekneemech is now known as bnemec | 14:48 | |
| *** superdan is now known as dansmith | 14:53 | |
| *** e0ne_ has quit IRC | 14:54 | |
| *** e0ne has joined #openstack-cinder | 14:56 | |
| *** deepakcs has joined #openstack-cinder | 14:57 | |
| *** patrickeast has joined #openstack-cinder | 14:58 | |
| *** rushil has joined #openstack-cinder | 14:59 | |
| *** pdeore_ has quit IRC | 15:02 | |
| *** dulek has quit IRC | 15:03 | |
| *** BharatK has joined #openstack-cinder | 15:06 | |
| *** patrickeast_ has joined #openstack-cinder | 15:06 | |
| *** patrickeast has quit IRC | 15:07 | |
| *** patrickeast_ is now known as patrickeast | 15:07 | |
| *** tbarron has joined #openstack-cinder | 15:11 | |
| *** amoturi has quit IRC | 15:11 | |
| *** amoturi has joined #openstack-cinder | 15:11 | |
| *** Miouge has quit IRC | 15:15 | |
| *** nlevinki has quit IRC | 15:16 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 15:16 | |
| *** rushiagr_away is now known as rushiagr | 15:18 | |
| *** enterprisedc has joined #openstack-cinder | 15:18 | |
| *** Miouge has joined #openstack-cinder | 15:20 | |
| *** ebalduf has joined #openstack-cinder | 15:21 | |
| *** Miouge has quit IRC | 15:23 | |
| *** ebalduf has quit IRC | 15:25 | |
| *** Miouge has joined #openstack-cinder | 15:26 | |
| openstackgerrit | Matthew Treinish proposed openstack/cinder: Use subunit-trace to enable output during unit test runs https://review.openstack.org/158020 | 15:28 |
| *** kallebe has quit IRC | 15:31 | |
| *** nlevinki has joined #openstack-cinder | 15:32 | |
| openstackgerrit | Steven Kaufer proposed openstack/cinder: Generic filter support for volume queries https://review.openstack.org/155343 | 15:34 |
| *** zigo_ has joined #openstack-cinder | 15:35 | |
| *** jcru has joined #openstack-cinder | 15:36 | |
| *** jcru has quit IRC | 15:37 | |
| *** alexpilotti has joined #openstack-cinder | 15:37 | |
| *** jcru has joined #openstack-cinder | 15:37 | |
| *** jcru has quit IRC | 15:38 | |
| *** jcru has joined #openstack-cinder | 15:38 | |
| *** Miouge has quit IRC | 15:39 | |
| *** emaisne_ has joined #openstack-cinder | 15:40 | |
| *** asselin_ has joined #openstack-cinder | 15:40 | |
| *** nlevinki has quit IRC | 15:42 | |
| *** jungleboyj_ has joined #openstack-cinder | 15:43 | |
| *** emaisne has quit IRC | 15:44 | |
| *** emaisne_ has quit IRC | 15:44 | |
| *** hemnafk is now known as hemna | 15:44 | |
| *** Miouge has joined #openstack-cinder | 15:45 | |
| *** pschaef has joined #openstack-cinder | 15:45 | |
| *** tsekiyama has joined #openstack-cinder | 15:45 | |
| *** aix has quit IRC | 15:47 | |
| *** Miouge has quit IRC | 15:50 | |
| *** rushiagr is now known as rushiagr_away | 15:55 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Remove warnings for long vgs and lvs calls https://review.openstack.org/157582 | 15:56 |
| *** lpetrut has joined #openstack-cinder | 15:56 | |
| *** xyang1 has joined #openstack-cinder | 15:57 | |
| guitarzan | jgriffith: did you get that figured out? | 15:57 |
| jgriffith | guitarzan: which? | 15:57 |
| jgriffith | guitarzan: the DB thing? | 15:57 |
| guitarzan | the lvm filter thing | 15:57 |
| jgriffith | guitarzan: haha... kinda | 15:57 |
| jgriffith | guitarzan: I figured out that the rescue stuff is an absolute PITA | 15:58 |
| guitarzan | haha! | 15:58 |
| jgriffith | guitarzan: and that negative tests in tempest are even more so :) | 15:58 |
| *** Mandell has joined #openstack-cinder | 15:59 | |
| *** kallebe has joined #openstack-cinder | 16:00 | |
| *** patrickeast has quit IRC | 16:00 | |
| jgriffith | anybody running kernel 3.16 on 14.04? | 16:02 |
| *** jungleboyj_ is now known as jungleboyj | 16:02 | |
| *** ndipanov has quit IRC | 16:04 | |
| jgriffith | ahh well.. how bad can it be :) | 16:04 |
| openstackgerrit | Steven Kaufer proposed openstack/cinder: Generic filter support for volume queries https://review.openstack.org/155343 | 16:05 |
| openstackgerrit | Steven Kaufer proposed openstack/cinder: Replication status periodic task optimization https://review.openstack.org/154673 | 16:05 |
| hemna | mornin | 16:06 |
| hemna | what's blowing up this fine Monday? | 16:06 |
| hemna | 3.13 here | 16:07 |
| jgriffith | hemna: pishh... that's ancient :) | 16:08 |
| hemna | heh | 16:08 |
| *** TobiasE has quit IRC | 16:09 | |
| *** Miouge has joined #openstack-cinder | 16:10 | |
| Swanson | jgriffith: I was going to mention that the only way I could get multibackend to install correctly was if I put the CINDER_ENABLED_BACKENDS=driver:backend,driver2:backend2 line above the post-config bit. | 16:14 |
| *** Miouge has quit IRC | 16:14 | |
| jgriffith | Swanson: yep.... | 16:14 |
| jgriffith | OHHH | 16:14 |
| jgriffith | Swanson: my snippet only did the names, not the types :( | 16:14 |
| jgriffith | bahhhh... I messed you up | 16:15 |
| Swanson | jgriffith: Not an issue as you posted https://bugs.launchpad.net/cinder/+bug/1424104 | 16:16 |
| openstack | Launchpad bug 1424104 in Cinder "Unable to restart c-vol services when using multi-backend due to set_voldb_empty_at_startup_indicator" [Undecided,New] | 16:16 |
| Swanson | jgriffith: I've seen that one on and off. | 16:17 |
| jgriffith | Swanson: yeah, that one is a big scary IMO | 16:17 |
| jgriffith | s/big/bit/ | 16:18 |
| jungleboyj | hemna: 3.13 ? | 16:19 |
| hemna | Linux devstack-master 3.13.0-32-generic | 16:19 |
| hemna | yup | 16:19 |
| *** dannywilson has joined #openstack-cinder | 16:20 | |
| jungleboyj | hemna: :-) | 16:21 |
| *** devlaps has joined #openstack-cinder | 16:21 | |
| *** mtanino has joined #openstack-cinder | 16:21 | |
| *** rushiagr_away is now known as rushiagr | 16:21 | |
| *** ebalduf has joined #openstack-cinder | 16:21 | |
| *** dannywil_ has joined #openstack-cinder | 16:23 | |
| *** dannywilson has quit IRC | 16:24 | |
| *** rmesta has joined #openstack-cinder | 16:25 | |
| *** ebalduf has quit IRC | 16:25 | |
| *** jdandrea has quit IRC | 16:28 | |
| *** jdandrea has joined #openstack-cinder | 16:29 | |
| *** pschaef has quit IRC | 16:32 | |
| *** e0ne has quit IRC | 16:32 | |
| *** nkrinner has quit IRC | 16:33 | |
| *** Mandell has quit IRC | 16:33 | |
| *** e0ne has joined #openstack-cinder | 16:33 | |
| *** ronis has quit IRC | 16:34 | |
| *** lpetrut has quit IRC | 16:35 | |
| hemna | jungleboyj, https://review.openstack.org/#/c/157184/ | 16:35 |
| hemna | when you have time | 16:35 |
| hemna | I'm trying to get all the brick related things to land | 16:36 |
| hemna | so we can cut a build to pypi | 16:36 |
| jungleboyj | hemna: Ok, will take a look. | 16:37 |
| *** deepakcs has quit IRC | 16:39 | |
| hemna | thanks man | 16:40 |
| *** devlaps has quit IRC | 16:42 | |
| *** Miouge has joined #openstack-cinder | 16:42 | |
| jgriffith | hemna: jungleboyj https://review.openstack.org/#/c/140877/ | 16:47 |
| *** Miouge has quit IRC | 16:48 | |
| *** devlaps has joined #openstack-cinder | 16:48 | |
| hemna | k I'll take a look | 16:48 |
| hemna | currently reviewing the CG update | 16:48 |
| jgriffith | hemna: jungleboyj the thing that I don't really get is there seems to be a completely different impl for the image-copy stuff that I just noticed this morning | 16:48 |
| jgriffith | but I could be completely wrong :) | 16:48 |
| hemna | oh? | 16:48 |
| jgriffith | hemna: lemme see if I can find it again | 16:49 |
| *** sgotliv has quit IRC | 16:50 | |
| jgriffith | hemna: https://review.openstack.org/#/c/134681/ | 16:50 |
| jgriffith | hemna: why not use the same structure? | 16:50 |
| *** deepakcs has joined #openstack-cinder | 16:51 | |
| hemna | 2 different things | 16:51 |
| hemna | 1 is for multipath support | 16:51 |
| hemna | the other is for alternative paths to the same volume | 16:51 |
| jgriffith | hehe... but that's my point | 16:51 |
| jgriffith | they "could" very easily and probably "should" IMHO be the same thing | 16:52 |
| jgriffith | alternative path, vs multi path | 16:52 |
| jgriffith | seems like a distinction without much of a difference | 16:52 |
| hemna | so there needs to be additional data then passed back to brick/nova to tell it what it needs to do then. | 16:52 |
| hemna | if it's trying to do a multipath attach vs. a single attach is different | 16:52 |
| jgriffith | hemna: sure | 16:52 |
| hemna | just because you have a list of target_portals doesn't mean you want multipath | 16:53 |
| hemna | so I guess that's why there was a distinction | 16:53 |
| jgriffith | hemna: understood | 16:53 |
| jgriffith | hemna: but why not just use the same data structure and a separate 'thing' to specify whihc you want? | 16:53 |
| hemna | yah I suppose that would be ok | 16:53 |
| jgriffith | hemna: if you look at the DS's they seem to duplicate an awful lot of code | 16:54 |
| openstackgerrit | Danny Wilson proposed openstack/cinder: Over subscription for Pure Storage iSCSI driver. https://review.openstack.org/157681 | 16:54 |
| hemna | multipath: True | 16:54 |
| hemna | False | 16:54 |
| hemna | done | 16:54 |
| jungleboyj | Seems sensible to me. | 16:54 |
| jgriffith | hemna: and like i said in my review, I really don't like the "if this format, else that format" thing | 16:54 |
| jgriffith | That's a compatability/support nightmare waiting to happen IMHO | 16:54 |
| hemna | ok I'm cool with that | 16:54 |
| jgriffith | hemna: The only other question is should they both be changed :) | 16:55 |
| jgriffith | hemna: to something that plugs seemlessly into the existing model | 16:55 |
| nikesh_vedams | hi | 16:55 |
| jgriffith | hemna: you're way more involved in this than I so I'll defer to you | 16:55 |
| hemna | I like target_portals | 16:55 |
| jgriffith | hemna: yeah... I do too | 16:56 |
| jgriffith | hemna: but that means we're "stuck" until a change is made on the Nova side | 16:56 |
| hemna | and an additional flag telling brick/nova what to do with them. Multipath: True | 16:56 |
| jgriffith | no? | 16:56 |
| hemna | if that's not there, then it's considered alternative portals | 16:56 |
| nikesh_vedams | i am trying to run cinder tests under /opt/stack/cinder but getting errors http://paste.openstack.org/show/180654/ | 16:56 |
| hemna | jgriffith, yah I think so | 16:56 |
| nikesh_vedams | is any facing this issue,i am using ubuntu 14.04 | 16:57 |
| jgriffith | nikesh_vedams: try deleting/rebuilding your venv | 16:57 |
| jgriffith | nikesh_vedams: it looks to me like youre venv never actually built correctly | 16:57 |
| jgriffith | hemna: so that's up to you guys I suppose, but I just thought we could make the DS in such a way that the existing code would all still "just work" | 16:58 |
| nikesh_vedams | okk let me check | 16:58 |
| hemna | jgriffith, I don't think the nova side has landed | 16:58 |
| smcginnis | nikesh_vedams: ./run_test.sh -f | 16:58 |
| jgriffith | hemna: no it hasn't... and it's not going to based on the looks of it | 16:58 |
| hemna | I think they were waiting on brick | 16:58 |
| hemna | :) | 16:58 |
| hemna | so that part is good | 16:58 |
| jgriffith | hemna: hence my concern for having this stuff half in/ half out | 16:58 |
| hemna | so we can fix this here in K before it's release | 16:59 |
| *** pdeore_ has joined #openstack-cinder | 16:59 | |
| jgriffith | That's what I'm thinking... if you and Tomoki agree with me | 16:59 |
| hemna | yah I do | 16:59 |
| jgriffith | coolness | 16:59 |
| hemna | fwiw, the spec was approved though. and you did +2 it :P | 17:00 |
| xyang1 | jgriffith: hemna there are already a couple of drivers made changes based on the multipath enhancement | 17:00 |
| jgriffith | hemna: sure | 17:00 |
| hemna | anyway | 17:00 |
| hemna | xyang1, which ? | 17:00 |
| jgriffith | hemna: but also TBF that was two specs | 17:01 |
| xyang1 | jgriffith: hemna emc vnx and dell. It is backward compatible | 17:01 |
| jgriffith | hemna: and I'm more concerned with impl than with the concept | 17:01 |
| xyang1 | So it works with current nova code too | 17:01 |
| hemna | sure | 17:01 |
| hemna | xyang1, well, the current nova code ignores the new entries | 17:01 |
| hemna | target_portals, target_alterative_portals | 17:01 |
| xyang1 | I know | 17:01 |
| hemna | so yes, it 'works' | 17:01 |
| xyang1 | I mean the driver still keeps the existing behavior | 17:02 |
| xyang1 | If nova side gets in, it will work with the new cide | 17:02 |
| xyang1 | Code | 17:02 |
| jgriffith | hemna: xyang1 so the one wth the portal lists etc that already landed I didn't really have an issue with | 17:02 |
| hemna | I don't think we want the nova side to land though | 17:02 |
| jgriffith | xyang1: hemna the concern for me was when I saw this one: https://review.openstack.org/#/c/140877/ | 17:03 |
| jgriffith | xyang1: hemna that IMO could be consolidated | 17:03 |
| *** pdeore_ has quit IRC | 17:03 | |
| jgriffith | I think hemna and I are on the same page | 17:03 |
| nikesh_vedams | jgriffith smcginnis : rm -rf .venv ./run_tests.sh targets and selected yes for virtual environment and getting this errror http://paste.openstack.org/show/180656/ | 17:03 |
| hemna | to just use target_portals | 17:03 |
| hemna | with the multipath: false | 17:03 |
| xyang1 | Ok, i am fine with that | 17:03 |
| jgriffith | nikesh_vedams: yeah, you need the pre-reqs | 17:04 |
| jgriffith | nikesh_vedams: "sudo apt-get install python-dev libssl-dev python-pip git-core libxml2-dev libxslt-dev pkg-config libffi-dev libpq-dev libmysqlclient-dev libvirt-dev graphviz libsqlite3-dev python-tox" | 17:04 |
| hemna | xyang1, do you have the EMC review that uses this change ? | 17:04 |
| Swanson | jgriffith: One other multibackend thing. I've had a failure with a DB deadlock message in addition to the failure you posted. (Haven't seen it for a while.) I think that is just a different flavor of the same bug. | 17:05 |
| *** rwsu has joined #openstack-cinder | 17:05 | |
| * tsekiyama is scrolling back | 17:05 | |
| jungleboyj | hemna: jgriffith I think it makes sense to use the same data structure. | 17:05 |
| xyang1 | 154774 | 17:05 |
| hemna | xyang1, https://review.openstack.org/#/c/154774 ? | 17:06 |
| xyang1 | https://review.openstack.org/#/c/154772 | 17:06 |
| hemna | oh | 17:06 |
| jgriffith | and by the way, I'm NEVER approving another spec or review if people keep coming back and saying "you approved it" any time I log a bug or raise a suggestion | 17:06 |
| hemna | :P | 17:06 |
| hemna | the other 4 | 17:06 |
| xyang1 | Sorry, wrong one | 17:06 |
| jgriffith | :) | 17:06 |
| xyang1 | :) | 17:06 |
| tsekiyama | hmm, using target_portals/iqns/luns even when multipath is False ? | 17:07 |
| jgriffith | I've had that thrown in my face way too many times the past couple monts | 17:07 |
| jgriffith | tsekiyama: oh... hellooooo! | 17:07 |
| jgriffith | :) | 17:07 |
| nikesh_vedams | jgriffith : thanks | 17:07 |
| jgriffith | for some reason I thought you'd be in a "sleeping" Time zone | 17:07 |
| dannywil_ | ls | 17:07 |
| tsekiyama | jgriffith: :) | 17:08 |
| hemna | xyang1, so it looks like the emc code could be changed to simply return multipath: True when it populates target_portals. | 17:08 |
| jgriffith | tsekiyama: so did you see my thoughts on the review? And the conversation with hemna ? | 17:08 |
| hemna | xyang1, that patch doesn't include support for target_alternative_portals | 17:08 |
| tsekiyama | When multipath = True, it omits target_portal/iqn/lun and add target_portals/iqns/luns. it is ok because multipath=True means the connector knows how to deal with new multipath structure. | 17:08 |
| tsekiyama | But when mutlipath = False, we need to think about backward compatibility .. | 17:08 |
| xyang1 | hemna: That is right. We don't support it | 17:08 |
| hemna | xyang1, so the EMC patch is ok. | 17:08 |
| *** alecv has quit IRC | 17:08 | |
| xyang1 | hemna: Ok. Good | 17:09 |
| tsekiyama | so we can't omit target_poratl/iqn/lun | 17:09 |
| jgriffith | tsekiyama: yeah, but I'm wondering if we could just consolidate on a single supported data-structure with optional keys for the multi-path and fail over | 17:09 |
| hemna | tsekiyama, so the change we are thinking of here is to eliminate the target_alternative_portals | 17:09 |
| *** Miouge has joined #openstack-cinder | 17:09 | |
| *** Saba has joined #openstack-cinder | 17:09 | |
| hemna | and simply reuse target_portals | 17:09 |
| hemna | as a list | 17:09 |
| hemna | and the driver says, multipath: True or False | 17:09 |
| jgriffith | tsekiyama: yeah... kinda what hemna said :) | 17:09 |
| *** Saba is now known as Guest87858 | 17:09 | |
| hemna | if the driver returns multipath: False, then target_portals is the same logic as target_alternative_portals was meant to be. | 17:10 |
| jgriffith | tsekiyama: my main point was not proliferating data structures, but using them more intelligently | 17:10 |
| Guest87858 | Hi , I am wondering if get_volume_stats is exposed as metric in cinder for reading free capacity | 17:10 |
| hemna | just a list to iterate over to try and get a connection to work at all | 17:10 |
| tsekiyama | hemna: I'm ok with reusing traget_portals | 17:10 |
| jgriffith | Guest87858: nope | 17:10 |
| Guest87858 | Hi , I am wondering if get_volume_stats is exposed as metric in cinder for reading free capacity | 17:10 |
| jgriffith | Guest87858: and again... "nope" :) | 17:10 |
| hemna | :) | 17:10 |
| jgriffith | xyang1: I think it's ok if we need to fix drivers that implemented it already | 17:11 |
| jgriffith | xyang1: it's all the same 'stuff' just presented slightly differently | 17:11 |
| tsekiyama | hemna: jgriffith: just concidering we should have both 'target_portals' and 'target_portal' when multipath=False, for old nova | 17:11 |
| xyang1 | jgriffith: That's fine. We just need to know | 17:11 |
| smcginnis | +1 on consolidating structures. That seems simpler IMO. | 17:11 |
| hemna | tsekiyama, yah | 17:11 |
| hemna | tsekiyama, we'll land this in brick | 17:11 |
| jgriffith | tsekiyama: so that's kinda my original thought | 17:11 |
| hemna | nova won't get anything | 17:12 |
| hemna | for L | 17:12 |
| jgriffith | but not sure how anybody else feels | 17:12 |
| smcginnis | tsekiyama: I think that part is good. You are correct, we definitely need backward support. | 17:12 |
| hemna | err K | 17:12 |
| jgriffith | tsekiyama: if you look at my comments in the review | 17:12 |
| hemna | K nova will ignore target_portals | 17:12 |
| jgriffith | tsekiyama: that was before I realized that your other patch had already merged | 17:12 |
| hemna | and L nova will get brick (with any luck) | 17:12 |
| Guest87858 | Hi , I am wondering if get_volume_stats is exposed as metric in cinder for reading free capacity | 17:12 |
| hemna | Guest87858, nope | 17:12 |
| jgriffith | Guest87858: you're going to get booted in a second | 17:12 |
| hemna | Guest87858, 3rd time we said nope. | 17:12 |
| jgriffith | You've asked the same question 4 times and received a prompt response every time | 17:13 |
| hemna | lag ? | 17:13 |
| jgriffith | hemna: nah... we wouldn't see it out of order on our side I don't think | 17:13 |
| jgriffith | hemna: but maybe | 17:13 |
| jgriffith | I dunno | 17:13 |
| Swanson | Perhaps they aren't seeing anything? | 17:14 |
| *** Miouge has quit IRC | 17:14 | |
| jgriffith | Swanson: maybe not | 17:14 |
| jgriffith | anyway.... | 17:14 |
| Guest87858 | I thought it was related to other conversation . Thanks. Why are we not exposing this metric ? Is is a good idea to expose free capacity gb as metric for monitoring ? | 17:14 |
| jgriffith | hemna: xyang1 tsekiyama Just some ideas | 17:14 |
| jgriffith | Guest87858: no worries | 17:15 |
| jgriffith | Guest87858: Well.. I believe it is sent up to Ceilometer for example | 17:15 |
| *** Miouge has joined #openstack-cinder | 17:15 | |
| jgriffith | Guest87858: but we typically don't want to expose details like that to end user | 17:15 |
| Guest87858 | But no metric of ceilometer exposes it too | 17:15 |
| Guest87858 | If a system wants to use this for metering then is there any way ? | 17:16 |
| jgriffith | Guest87858: I believe so yes, that's what I was getting at | 17:16 |
| jgriffith | Guest87858: we emit that information in various places | 17:16 |
| *** Miouge has quit IRC | 17:16 | |
| Guest87858 | Can you tell me which meters ? | 17:17 |
| jgriffith | Guest87858: nope :( | 17:17 |
| jgriffith | Guest87858: I don't mess with that much, but maybe others here do | 17:17 |
| Guest87858 | Thanks | 17:17 |
| *** JayJ has joined #openstack-cinder | 17:17 | |
| *** _cjones_ has joined #openstack-cinder | 17:18 | |
| *** lpetrut has joined #openstack-cinder | 17:21 | |
| guitarzan | jgriffith: is it sent to ceilometer? I'm not sure | 17:21 |
| *** ebalduf has joined #openstack-cinder | 17:22 | |
| *** bkopilov has quit IRC | 17:26 | |
| *** ebalduf has quit IRC | 17:26 | |
| *** leeantho has joined #openstack-cinder | 17:27 | |
| jgriffith | guitarzan: yeah, I'm not certain either | 17:27 |
| *** jistr has quit IRC | 17:29 | |
| jgriffith | guitarzan: seems like it would be of key interest, but who konws | 17:29 |
| jgriffith | knows | 17:29 |
| guitarzan | oh, I'm sure it's of interest, but I don't think it's currently sent | 17:30 |
| jgriffith | :) | 17:30 |
| jgriffith | guitarzan: so what "is" sent... just usage I suppose? | 17:30 |
| jgriffith | and not even based on backend based on what I can see | 17:30 |
| guitarzan | I think the only thing sent is our explicit notifications | 17:30 |
| jgriffith | guitarzan: yeah, the "emitters" | 17:31 |
| jgriffith | guitarzan: suppose we could easily add a bunch o'stuff | 17:31 |
| guitarzan | so unless there's one for get_volume_stats or maybe a periodic one in the scheduler or something | 17:31 |
| *** emaisne has joined #openstack-cinder | 17:35 | |
| *** e0ne has quit IRC | 17:38 | |
| *** emaisne has quit IRC | 17:39 | |
| *** emaisne_ has joined #openstack-cinder | 17:39 | |
| *** bkopilov has joined #openstack-cinder | 17:41 | |
| *** jordanP has quit IRC | 17:43 | |
| *** yuriy_n17 has quit IRC | 17:45 | |
| *** tbarron has quit IRC | 17:46 | |
| *** tbarron has joined #openstack-cinder | 17:47 | |
| *** ronis has joined #openstack-cinder | 17:48 | |
| *** Apoorva has joined #openstack-cinder | 17:49 | |
| *** coolsvap is now known as coolsvap_ | 17:52 | |
| *** kallebe has quit IRC | 17:55 | |
| *** kallebe has joined #openstack-cinder | 17:59 | |
| *** annashen has joined #openstack-cinder | 18:00 | |
| *** Mandell has joined #openstack-cinder | 18:05 | |
| *** beecee has joined #openstack-cinder | 18:07 | |
| *** afazekas_ has quit IRC | 18:07 | |
| *** dustins_ has joined #openstack-cinder | 18:09 | |
| *** david-lyle_afk is now known as david-lyle | 18:09 | |
| *** _cjones_ has quit IRC | 18:10 | |
| *** dustins has quit IRC | 18:12 | |
| *** _cjones_ has joined #openstack-cinder | 18:15 | |
| *** deepakcs has quit IRC | 18:16 | |
| *** garthb_ has joined #openstack-cinder | 18:17 | |
| *** garthb has joined #openstack-cinder | 18:17 | |
| *** ebalduf has joined #openstack-cinder | 18:23 | |
| *** patrickeast has joined #openstack-cinder | 18:25 | |
| *** vilobhmm has joined #openstack-cinder | 18:27 | |
| *** rushiagr is now known as rushiagr_away | 18:27 | |
| *** ebalduf has quit IRC | 18:27 | |
| *** Guest87858 has quit IRC | 18:28 | |
| *** vilobhmm has quit IRC | 18:29 | |
| *** vilobhmm has joined #openstack-cinder | 18:31 | |
| *** dustins_ has quit IRC | 18:31 | |
| *** dustins has joined #openstack-cinder | 18:32 | |
| *** vilobhmm1 has joined #openstack-cinder | 18:32 | |
| *** vilobhmm_ has joined #openstack-cinder | 18:32 | |
| *** harlowja has joined #openstack-cinder | 18:33 | |
| *** diemt has joined #openstack-cinder | 18:34 | |
| *** dustins has quit IRC | 18:35 | |
| *** vilobhmm has quit IRC | 18:35 | |
| *** vilobhmm_ is now known as vilobhmm | 18:35 | |
| *** dustins has joined #openstack-cinder | 18:36 | |
| *** casusbelli has joined #openstack-cinder | 18:39 | |
| *** thingee has joined #openstack-cinder | 18:44 | |
| mkoderer | thingee: ping | 18:46 |
| mkoderer | thingee: could you also have a look to https://review.openstack.org/#/c/138999/ | 18:46 |
| *** asselin_ has quit IRC | 18:47 | |
| thingee | mkoderer: maybe today. I was absent friday because of a conference. Definitely tomorrow though. I got a lot of emails of "please look at this" :) | 18:47 |
| casusbelli | jgriffith: Hi! Can i take a few mins regarding sos-ci, again? :) | 18:53 |
| mkoderer | thingee: sure sure :) I just want to have the ABC stuff somehow delivered | 18:54 |
| mkoderer | thingee: since jgriffith already ask for the next steps :) | 18:55 |
| thingee | xyang1: are we going to see an update this week with https://review.openstack.org/#/c/152973/ ? | 18:55 |
| thingee | jgriffith: https://review.openstack.org/#/c/153918/ | 18:56 |
| thingee | jgriffith: happy to see this | 18:56 |
| thingee | jgriffith: thanks again for your work on v2 | 18:56 |
| xyang1 | thingee: I hope so. He got one problem overriding the existing max over sub ratio | 18:56 |
| xyang1 | thingee: I need to take a look, not sure why it does not work | 18:57 |
| xyang1 | thingee: So he wants to override the default value | 18:57 |
| thingee | I know, we talked about this last week :) | 18:58 |
| xyang1 | thingee: Yes, i sent him examples. He tried but said it does not work | 18:58 |
| kallebe | Just a quick question regarding pep8 style: do you think we should enforce the pep8 recommendation of newline >after< the binary operator? I mean, should it be "foo +\nbar" or "foo\n+bar"? | 18:59 |
| thingee | kallebe: space is sufficient to me. | 19:00 |
| kallebe | thingee I don't know if I understand what you mean. I am asking this because there are many cases of both coexisting | 19:01 |
| guitarzan | kallebe: if you want to add a hacking check, see if people like it | 19:02 |
| kallebe | when the code would pass 80 characters, it would start a newline and place the operator | 19:02 |
| thingee | 1 + 1 | 19:03 |
| thingee | there are spaces in between | 19:03 |
| thingee | that's sufficient to me | 19:03 |
| guitarzan | thingee: he's talking about | 19:03 |
| guitarzan | i | 19:03 |
| guitarzan | + 1 | 19:03 |
| kallebe | thingee no, that's right. I agree, but my question is about \n | 19:03 |
| *** Apoorva_ has joined #openstack-cinder | 19:03 | |
| guitarzan | v | 19:03 |
| guitarzan | i + | 19:03 |
| guitarzan | 1 | 19:03 |
| * guitarzan didn't realize pep8 even cared | 19:03 | |
| *** hemna has quit IRC | 19:03 | |
| thingee | I understand that. kallebe asked for an opinion. My opinion is a space is sufficient. I don't see the appeal with a new line. | 19:03 |
| guitarzan | thingee: 80 character limit... | 19:04 |
| thingee | guitarzan: got an example where we hit that case? | 19:04 |
| thingee | that might help me see ugly :) | 19:04 |
| guitarzan | I'm trying to find it in pep8... | 19:04 |
| kallebe | I will fetch one here | 19:04 |
| *** dustins has quit IRC | 19:04 | |
| thingee | kallebe: thanks | 19:05 |
| kallebe | here is an example: | 19:06 |
| kallebe | https://github.com/openstack/cinder/blob/master/cinder/zonemanager/drivers/cisco/cisco_fc_zone_driver.py#L289 | 19:06 |
| *** Apoorva has quit IRC | 19:06 | |
| kallebe | imagine it passes 80 characters. Should we put a + at the end or at the beginning? | 19:06 |
| *** dustins has joined #openstack-cinder | 19:07 | |
| thingee | xyang1: are these bugs still happening for kilo? https://bugs.launchpad.net/cinder/+bug/1385450 https://bugs.launchpad.net/cinder/+bug/1372635 ... I noticed we finally got some patches for bugs in 2/20 :) | 19:07 |
| openstack | Launchpad bug 1385450 in Cinder "EMC volume create_cloned_volume failure leaves target volume" [Medium,Triaged] - Assigned to Xing Yang (xing-yang) | 19:07 |
| openstack | Launchpad bug 1372635 in Cinder "MITM vulnerability with EMC VMAX driver" [High,In progress] - Assigned to Xing Yang (xing-yang) | 19:07 |
| thingee | kallebe: Oh I see, my mistake. umm front? | 19:08 |
| kallebe | And the pep8 section is this one: https://www.python.org/dev/peps/pep-0008/#maximum-line-length | 19:08 |
| thingee | that's my immediate thought anyways | 19:08 |
| kallebe | see the snippet for that class Rectangle there | 19:09 |
| thingee | kallebe: yeah I don't care one way or another then. :) | 19:09 |
| guitarzan | "break around a binary operator is after the " | 19:09 |
| guitarzan | gotcha | 19:09 |
| xyang1 | thingee: One has patch submitted, the other fixed already. I'll update them | 19:09 |
| thingee | if that's what people do, sure | 19:09 |
| guitarzan | took a lot for me to see it | 19:09 |
| thingee | xyang1: thanks! | 19:09 |
| guitarzan | personally I like it the other way, but *shrug* | 19:09 |
| thingee | xyang1: any luck with devstack to test cinder objects? | 19:09 |
| xyang1 | thingee: Yes, I did some testing and it worked for me | 19:10 |
| kallebe | thingee ok. I was wondering if a general pep8 cleanup would be accepted :) | 19:10 |
| kallebe | I would like to work on it | 19:10 |
| guitarzan | kallebe: if it isn't in hacking, it won't be accepted | 19:10 |
| kallebe | I have seen hacking, but it was more about pep8 | 19:10 |
| guitarzan | right, but optional "I like this better" stuff doesn't usually make it | 19:11 |
| kallebe | the specific recommendations did not mention that case, for example | 19:11 |
| guitarzan | if it isn't enforced, it shouldn't be changed | 19:11 |
| thingee | so much green "implemented" ... happy right now :) https://launchpad.net/cinder/+milestone/kilo-3 | 19:11 |
| xyang1 | thingee: So I don't see big problems with it. Just want to go thru the code more to make sure | 19:11 |
| kallebe | guitarzan yes, so I was here asking how to know if it is enforced. | 19:11 |
| thingee | xyang1: ok, but did you try it out? schema changes while things are in progress? | 19:11 |
| kallebe | is there any place where I can see the enforced ones? | 19:11 |
| kallebe | the ones jenkins tests, for example | 19:12 |
| guitarzan | kallebe: in here maybe? https://github.com/openstack-dev/hacking | 19:12 |
| xyang1 | thingee: I tried migrate down and also add a column in snapshot table | 19:12 |
| guitarzan | kallebe: or better: https://github.com/openstack/cinder/tree/master/cinder/hacking | 19:12 |
| *** hemna has joined #openstack-cinder | 19:12 | |
| thingee | leeantho: client landing soon or did? so that we can get an update on this? https://review.openstack.org/#/c/155127/ | 19:12 |
| thingee | hemna: ^ | 19:13 |
| leeantho | thingee, yes, it is being released to pypi this week | 19:13 |
| kallebe | guitarzan ok, thanks. I will take a look. Just one more question, again related to pep8 haha About keyword arguments. | 19:13 |
| thingee | winston-d: last week I asked for an update on this: https://review.openstack.org/#/c/141868/10 it's block a couple of bps at the moment. | 19:13 |
| kallebe | what about that whitespace between keyword arguments? | 19:14 |
| patrickeast | thingee: oh hey, sorry to change direction after asking you to approve it right at the deadline, but can you move https://blueprints.launchpad.net/cinder/+spec/pure-iscsi-improved-multipath out of k-3? We’ve decided to do it in L once the nova side of those iscsi changes can get in (otherwise right now we don’t really get any benefit from it) | 19:14 |
| patrickeast | i mistakenly thought the nova changes were going to make it in K | 19:14 |
| thingee | patrickeast: making my day :) | 19:15 |
| thingee | patrickeast: marked it for next | 19:15 |
| thingee | patrickeast: thanks for the update | 19:15 |
| patrickeast | thingee: yay, thanks | 19:15 |
| patrickeast | np | 19:15 |
| *** hemna has quit IRC | 19:16 | |
| thingee | don't forget everyone, summit sessions proposals have started https://etherpad.openstack.org/p/cinder-liberty-proposed-sessions | 19:18 |
| thingee | leeantho: thanks, please don't forget about http://lists.openstack.org/pipermail/openstack-dev/2015-February/056964.html | 19:18 |
| thingee | leeantho: march 1st for jenkins passing patches | 19:19 |
| *** Mandell has quit IRC | 19:22 | |
| *** hemna has joined #openstack-cinder | 19:22 | |
| leeantho | thingee, thanks, hemna and kmartin are finishing a few things for it. it should be up before end of the week so there is time for corrections / reviews to the patch | 19:22 |
| *** ebalduf has joined #openstack-cinder | 19:24 | |
| hemna | thingee, we are working on finalizing the client release | 19:24 |
| *** alexpilotti has quit IRC | 19:24 | |
| thingee | leeantho, hemna: excellent thanks! just wanted to communicate I'm watching the patch :) | 19:24 |
| hemna | ok coolio | 19:24 |
| thingee | seriously though, thank you everyone for helping with reviews. 28/46 targeted things is still a bit to do, but progress that's nothing to sneeze at. | 19:27 |
| *** Mandell has joined #openstack-cinder | 19:28 | |
| *** kvidvans has quit IRC | 19:28 | |
| *** ebalduf has quit IRC | 19:28 | |
| *** sgotliv has joined #openstack-cinder | 19:30 | |
| openstackgerrit | Bob Callaway proposed openstack/cinder: NetApp eseries implementation for manage/unmanage https://review.openstack.org/138093 | 19:33 |
| *** Tross has joined #openstack-cinder | 19:45 | |
| *** kallebe has left #openstack-cinder | 19:47 | |
| openstackgerrit | Patrick East proposed openstack/cinder-specs: Add DB table for driver private data https://review.openstack.org/153333 | 19:51 |
| *** e0ne has joined #openstack-cinder | 19:51 | |
| *** openstackgerrit has quit IRC | 19:51 | |
| *** openstackgerrit has joined #openstack-cinder | 19:52 | |
| *** ronis has quit IRC | 19:53 | |
| patrickeast | jgriffith: hemna: ^ I’ve updated that spec to where I’m pretty happy with it after testing it out a bit with some POC code and things seem to work correctly so hopefully this thing should be ready for some final reviews | 19:53 |
| *** Tross has quit IRC | 19:54 | |
| *** Tross has joined #openstack-cinder | 19:54 | |
| *** Mandell has quit IRC | 19:54 | |
| *** Mandell has joined #openstack-cinder | 19:56 | |
| *** Mandell has quit IRC | 19:57 | |
| *** markvoelker has joined #openstack-cinder | 20:07 | |
| *** drjones has joined #openstack-cinder | 20:10 | |
| *** _cjones_ has quit IRC | 20:10 | |
| *** BharatK has quit IRC | 20:11 | |
| e0ne | jungleboyj: hi! are you around? | 20:12 |
| jungleboyj | e0ne: Yep. | 20:13 |
| e0ne | jungleboyj: few questions from me | 20:13 |
| openstackgerrit | Danny Wilson proposed openstack/cinder: Over subscription for Pure Storage iSCSI driver. https://review.openstack.org/157681 | 20:14 |
| e0ne | jungleboyj: did you se my comment for https://review.openstack.org/#/c/149446/? not sure now is the best way to do | 20:14 |
| e0ne | it's about volume copy bandwidth | 20:14 |
| *** marcusvrn has quit IRC | 20:15 | |
| jungleboyj | e0ne: Looking. | 20:16 |
| e0ne | jungleboyj: thank you | 20:16 |
| jungleboyj | e0ne: Oh, you think it should be renamed 'throttling' so you don't have to do 'import throttle as throttling'? | 20:17 |
| e0ne | jungleboyj: yes, because it is used as 'import throttle as throttling' everywhere | 20:18 |
| jungleboyj | I guess I can see that argument. | 20:20 |
| e0ne | i won't -1 it if you thinl it's not important | 20:21 |
| jungleboyj | I just updated and asked Tomoki's input. See what they say. | 20:22 |
| e0ne | thanks | 20:22 |
| jungleboyj | e0ne: No problem. Thanks for looking through the incubator patches. I am getting close to getting the scheduler patch fixed up. That that point I can do the rest of the cleanup in cinder/openstack/common and then get the move to oslo.log done. | 20:23 |
| jungleboyj | e0ne: Will be excited to have all this done. | 20:24 |
| *** ebalduf has joined #openstack-cinder | 20:24 | |
| e0ne | jungleboyj: great! it was the second question for you:) | 20:26 |
| jungleboyj | :-) Yeah, I hope we can get this wrapped up this week. | 20:27 |
| jungleboyj | patrickeast: I owe you one. I got Edwin to dig into that issue with the flashsystem CI and he discovered he had a downlevel version of the CI code that wasn't properly pulling the patches down. | 20:28 |
| patrickeast | jungleboyj: ahh nice, that would explain it | 20:28 |
| jungleboyj | patrickeast: Thanks for catching that. Still waiting for the storwize CI contact to return from Chinese holiday. I am thinking she has the same issue though. | 20:28 |
| *** Mandell has joined #openstack-cinder | 20:28 | |
| patrickeast | jungleboyj: glad it wasn’t anything to crazy | 20:29 |
| jungleboyj | patrickeast: Yeah, something that asselin had fixed a while ago and we missed it. | 20:29 |
| *** ebalduf has quit IRC | 20:29 | |
| jungleboyj | patrickeast: Yep, thanks for calling it out! | 20:29 |
| patrickeast | jungleboyj: np | 20:30 |
| *** erlon has joined #openstack-cinder | 20:30 | |
| *** Apoorva_ has quit IRC | 20:33 | |
| *** Apoorva has joined #openstack-cinder | 20:33 | |
| *** harlowja has quit IRC | 20:34 | |
| *** harlowja_ has joined #openstack-cinder | 20:34 | |
| *** annashen has quit IRC | 20:35 | |
| *** mriedem has quit IRC | 20:35 | |
| *** rcallawa_ has joined #openstack-cinder | 20:36 | |
| jgriffith | any core want to help a brother out and merge this: https://review.openstack.org/#/c/157582/ | 20:40 |
| jgriffith | think I've iterated on spelling issues enough :) | 20:40 |
| jungleboyj | Wheee. Unit tests now pass with the new scheduler code. | 20:41 |
| *** markvoelker has quit IRC | 20:51 | |
| *** lpetrut has quit IRC | 20:59 | |
| *** mdenny has quit IRC | 21:01 | |
| *** sandywalsh_ has joined #openstack-cinder | 21:02 | |
| *** sandywalsh has quit IRC | 21:05 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: NFS backup driver https://review.openstack.org/149726 | 21:05 |
| *** dustins has quit IRC | 21:14 | |
| *** markvoelker has joined #openstack-cinder | 21:15 | |
| *** markvoelker has quit IRC | 21:15 | |
| *** markvoelker has joined #openstack-cinder | 21:16 | |
| e0ne | jgriffith: it was too quick for me to vote for your change:( | 21:16 |
| *** dustins has joined #openstack-cinder | 21:16 | |
| *** jwang_ has joined #openstack-cinder | 21:18 | |
| *** jwang has quit IRC | 21:20 | |
| *** jwang_ has quit IRC | 21:20 | |
| *** jwang has joined #openstack-cinder | 21:21 | |
| *** ebalduf has joined #openstack-cinder | 21:25 | |
| *** vilobhmm2 has joined #openstack-cinder | 21:25 | |
| *** vilobhmm has quit IRC | 21:25 | |
| *** vilobhmm1 has quit IRC | 21:25 | |
| anish | e0ne: your comment is in direct contradiction to what jgriffith xyang1 said on https://review.openstack.org/#/c/154713/ | 21:26 |
| anish | technically all 3 of you have different suggestions. | 21:26 |
| anish | jgriffith wants no six., xyang1 wants it only for debug, e0ne wants it for all | 21:27 |
| anish | welp ? | 21:27 |
| *** rushiagr_away is now known as rushiagr | 21:27 | |
| e0ne | anish: hm... | 21:28 |
| *** jungleboyj has quit IRC | 21:29 | |
| *** ebalduf has quit IRC | 21:29 | |
| jgriffith | anish: :) | 21:30 |
| jgriffith | anish: I'm easy, I'll got with what others say | 21:30 |
| jgriffith | anish: admittedly it's a messy thing | 21:30 |
| jgriffith | anish: and I was just reading the doc, which may be wrong... or perhaps I interpretted it wrong | 21:30 |
| jgriffith | anish: I'll be away for a bit, but send me a note and I'll do whatever you all think is right in this case | 21:31 |
| xyang1 | anish: If you follow the example in the guideline, which is an info, then you are fine:) | 21:31 |
| *** annashen has joined #openstack-cinder | 21:32 | |
| xyang1 | jgriffith: You discovered that bug? If you try to print ex without _ or six, it throws unicode error | 21:33 |
| e0ne | anish, jgriffith, xyang1: sometimes it's not easy for me to make right decision. need to know what error message will be | 21:34 |
| xyang1 | anish: If you use _, then no need for six | 21:34 |
| thingee | jgriffith: sorry, would've helped. looks ninja'd :) | 21:35 |
| thingee | I approved it earlier. I blame jenkins | 21:35 |
| anish | e0ne: xyang1 jgriffith sounds good, thanks ! | 21:35 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Failover to alternative iSCSI portals on login failure https://review.openstack.org/140877 | 21:36 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/os-brick: Use target_portals/iqns/luns for alternative target information https://review.openstack.org/158457 | 21:36 |
| anish | e0ne: can I convince you for something better than 0 maybe :) ? | 21:37 |
| e0ne | xyang1: do you mean something like _('err: %s' % exc)? | 21:37 |
| xyang1 | e0ne: Yes, _LI,etc, according to guideline | 21:37 |
| xyang1 | e0ne: It uses six underneath | 21:38 |
| smcginnis | e0ne: I don't think around the formatting?? | 21:38 |
| anish | hemna: os-brick doesn't get it's own -specs ? | 21:38 |
| hemna | anish, not yet | 21:38 |
| anish | hemna: I have a WIP spec for brick made in cinder-specs. I should just let it be for now then ? | 21:39 |
| hemna | url? | 21:39 |
| *** annashen has quit IRC | 21:39 | |
| *** annashen has joined #openstack-cinder | 21:40 | |
| e0ne | xyang1: i read guidelines once more, thanks | 21:41 |
| anish | hemna: https://review.openstack.org/#/c/152740/ | 21:41 |
| hemna | so, I don't think we have specs for cinderclient either | 21:42 |
| xyang1 | e0ne: It doesn't cover the debug msg case though. That's the problem | 21:42 |
| hemna | so since os-brick is a subproject of cinder, I think cinder-specs is fine for now | 21:42 |
| hemna | thingee, may want to revisit that later. | 21:42 |
| anish | ko ! | 21:42 |
| *** akerr has quit IRC | 21:42 | |
| *** dustins has quit IRC | 21:43 | |
| e0ne | xyang1: agree, it doesn't cover all cases | 21:45 |
| *** karimb has joined #openstack-cinder | 21:45 | |
| *** btran has joined #openstack-cinder | 21:45 | |
| *** dustins has joined #openstack-cinder | 21:46 | |
| thingee | hemna: ok | 21:46 |
| * thingee heads out to lunch | 21:46 | |
| *** jwang has quit IRC | 21:48 | |
| *** karimb has quit IRC | 21:48 | |
| *** dustins has quit IRC | 21:48 | |
| anish | e0ne: xyang1 documentation needs to be updated to remove ambiguities wrt six+debug ? | 21:48 |
| *** bswartz has quit IRC | 21:53 | |
| *** jwang has joined #openstack-cinder | 21:55 | |
| *** annashen has quit IRC | 21:56 | |
| smcginnis | anish, e0ne, xyang1: Tested exception logging: http://paste.openstack.org/show/180813/ | 21:59 |
| smcginnis | Not what I expected. | 21:59 |
| smcginnis | Unless I'm doing something different here. | 21:59 |
| *** annashen has joined #openstack-cinder | 21:59 | |
| xyang1 | smcginnis: I think it has to do with the exception itself | 22:00 |
| xyang1 | smcginnis: I tested it a couple of months ago | 22:00 |
| smcginnis | xyang1: Depending on the type of exception? | 22:00 |
| *** Yogi1 has quit IRC | 22:00 | |
| xyang1 | smcginnis: Yes | 22:00 |
| smcginnis | xyang1: Or if the exception contains unicode? | 22:00 |
| xyang1 | smcginnis: Correct | 22:01 |
| xyang1 | smcginnis: Can you check the patch i referenced there | 22:01 |
| smcginnis | xyang1: Sorry, can you post again? | 22:01 |
| *** emaisne_ has quit IRC | 22:02 | |
| *** Apoorva_ has joined #openstack-cinder | 22:03 | |
| anish | smcginnis: my understanding was the correct way is this http://paste.openstack.org/show/180819/ | 22:03 |
| xyang1 | https://review.openstack.org/#/c/143162/ | 22:03 |
| smcginnis | anish: That looks like what I would expect. | 22:03 |
| smcginnis | Though not sure why it would be different between debug and info. | 22:04 |
| smcginnis | I would think either the logger handles unicode or it doesn't. | 22:04 |
| *** Apoorva has quit IRC | 22:06 | |
| anish | erm, the documentation seems way off. Uses % in logging. From a review that I cannot remember, someone mentioned that % + dict can have issues in logging as well | 22:06 |
| smcginnis | Line 325 would support anish's understanding, at least for info level: https://review.openstack.org/#/c/143162/3/cinder/brick/iscsi/iscsi.py | 22:06 |
| anish | hopefully same for all non-debug levels | 22:08 |
| smcginnis | anish: It does appear so: http://paste.openstack.org/show/180820/ | 22:09 |
| smcginnis | The i18m guidelines do state: "Whenever possible translation should not be forced by use of str(), unicode(), or six.text_type() on a message being used with a format string." | 22:09 |
| anish | smcginnis++ thank you for the research | 22:10 |
| smcginnis | anish: No problem. I wanted to make sure I understood this. Been hearing conflicting information. | 22:10 |
| *** thangp has quit IRC | 22:11 | |
| smcginnis | xyang1, e0ne, jgriffith: ^^ Making sure you saw http://paste.openstack.org/show/180820/ | 22:14 |
| *** primechuck has quit IRC | 22:15 | |
| xyang1 | smcginnis: Unless if they wrap everything by default now. It threw unicode error for me last time | 22:16 |
| *** JayJ has quit IRC | 22:16 | |
| *** nellysmitt has quit IRC | 22:16 | |
| *** nellysmitt has joined #openstack-cinder | 22:16 | |
| xyang1 | smcginnis: I tried one of the command line in brick | 22:16 |
| smcginnis | xyang1: Hmm... I hate inconsistent results. | 22:17 |
| *** nellysmitt has quit IRC | 22:17 | |
| *** JayJ has joined #openstack-cinder | 22:17 | |
| smcginnis | xyang1: Are you sure it was passed as an arg to the log command and not formatted as a string first? | 22:17 |
| xyang1 | smcginnis: That's why i prefer it is wrapped | 22:17 |
| smcginnis | i.e. LOG.info('Test: %s', e) vs LOG.info('TEst: %s' % e) | 22:17 |
| *** vilobhmm1 has joined #openstack-cinder | 22:18 | |
| xyang1 | smcginnis: Yes | 22:18 |
| *** vilobhmm2 has quit IRC | 22:18 | |
| xyang1 | smcginnis: Tried some command line in brick and print that | 22:18 |
| xyang1 | smcginnis: Make sure it fails | 22:19 |
| smcginnis | xyang1: Can you give me a pointer to where you tested that? I just want to make sure I'm looking at the same thing. | 22:20 |
| *** Mandell has quit IRC | 22:20 | |
| xyang1 | smcginnis: I don't remember exactly which command I tried. It is in that patch I sent earlier. Maybe revert it back? Because jgriffith ran into the problem first and fixed it | 22:21 |
| *** JayJ has quit IRC | 22:23 | |
| *** JayJ has joined #openstack-cinder | 22:24 | |
| smcginnis | xyang1: OK, in that patch exceptions are just being passed in to LOG.error, but six.text_type(e) is used for debug. | 22:25 |
| smcginnis | But that actually didn't change with his changes. | 22:25 |
| smcginnis | Those appear to only have been changed to go from % to , | 22:26 |
| smcginnis | I'll have to set up and see. | 22:26 |
| *** ebalduf has joined #openstack-cinder | 22:26 | |
| *** aix has joined #openstack-cinder | 22:26 | |
| xyang1 | smcginnis: Some are removed. Look at code on the left | 22:26 |
| *** timcl has quit IRC | 22:27 | |
| smcginnis | xyang1: I just saw one changed from e to e.stdout. | 22:27 |
| smcginnis | or rather e.stderr. | 22:28 |
| *** JayJ has quit IRC | 22:28 | |
| xyang1 | smcginnis: Some debug on the left are removed | 22:28 |
| *** rushiagr is now known as rushiagr_away | 22:29 | |
| *** harlowja_ has quit IRC | 22:29 | |
| *** JayJ has joined #openstack-cinder | 22:29 | |
| smcginnis | xyang1: The logging of "out" and "err" from the _run() command? | 22:29 |
| xyang1 | Yes | 22:29 |
| *** harlowja has joined #openstack-cinder | 22:30 | |
| *** ebalduf has quit IRC | 22:30 | |
| smcginnis | That doesn't look directly related. At least from the commit message. | 22:30 |
| smcginnis | Commit message actually gives "LOG.error("%s" % e)" as the offending format. | 22:30 |
| smcginnis | Nothing about using six.text_type(). | 22:31 |
| smcginnis | So I think it probably comes down to doing LOG.debug('Test: %s', e) vs LOG.debug('Test: %s' % e) | 22:31 |
| smcginnis | With the first being the correct way. | 22:31 |
| smcginnis | It appears (and I would expect) the logger to handle it correctly. | 22:32 |
| smcginnis | Although this is all speculation. I'll spin it up and see if I can verify. | 22:32 |
| *** JayJ has quit IRC | 22:33 | |
| *** JayJ has joined #openstack-cinder | 22:34 | |
| smcginnis | Ah, and of course it's gone in my current stacked view. | 22:34 |
| *** JayJ has quit IRC | 22:34 | |
| * smcginnis shakes fist at hemna | 22:34 | |
| *** jwang_ has joined #openstack-cinder | 22:34 | |
| smcginnis | :) | 22:35 |
| hemna | what did I do ? | 22:35 |
| *** rcallawa_ has quit IRC | 22:36 | |
| smcginnis | hemna: Ah, nothing. Testing something that was in brick. :) | 22:36 |
| *** jwang has quit IRC | 22:38 | |
| *** Apoorva_ has quit IRC | 22:39 | |
| *** Apoorva has joined #openstack-cinder | 22:39 | |
| *** Mandell has joined #openstack-cinder | 22:40 | |
| *** mtanino has quit IRC | 22:40 | |
| *** Mandell has quit IRC | 22:47 | |
| *** asmith_brcd has joined #openstack-cinder | 22:52 | |
| *** xyang has joined #openstack-cinder | 22:53 | |
| xyang | smcginnis:hi | 22:53 |
| xyang | smcginnis: I got some results | 22:54 |
| smcginnis | xyang: Oh! | 22:54 |
| smcginnis | xyang: What did you see? | 22:54 |
| smcginnis | xyang: I'm still trying to sort my devstack setup. :\ | 22:55 |
| xyang | smcginnis: I think you are right. If we use %, we'll get unicode error right away from the oslo i18n library. If we use ",", it will be handled correctly | 22:55 |
| smcginnis | xyang: Oh good, that's what I would expect. And want. | 22:55 |
| smcginnis | xyang: We still need to watch for uses of msg = 'Unicode %s' % blah, LOG.info(msg), raise exception(msg). | 22:56 |
| smcginnis | xyang: But that's better than every logging call. | 22:56 |
| xyang | smcginnis:yes | 22:56 |
| smcginnis | xyang: Thanks for checking! | 22:57 |
| xyang | smcginnis: no problem. I want to get it right as well | 22:57 |
| *** kaufer has quit IRC | 22:59 | |
| *** e0ne has quit IRC | 23:00 | |
| *** Apoorva_ has joined #openstack-cinder | 23:02 | |
| *** harlowja has quit IRC | 23:04 | |
| *** rwsu is now known as rwsu-afk | 23:04 | |
| *** Mandell has joined #openstack-cinder | 23:05 | |
| *** Apoorva has quit IRC | 23:06 | |
| *** devlaps has quit IRC | 23:14 | |
| hemna | smcginnis, broken ? | 23:14 |
| *** devlaps has joined #openstack-cinder | 23:14 | |
| smcginnis | hemna: Nothings broken, don't worry. | 23:14 |
| smcginnis | hemna: Just having some fun. ;) | 23:14 |
| hemna | heh ok | 23:15 |
| hemna | I'm fixing a fczm issue at the moment | 23:15 |
| smcginnis | hemna: Sorry to distract you. | 23:15 |
| hemna | can't load the cisco driver | 23:15 |
| hemna | the config is borked | 23:15 |
| hemna | *sigh* | 23:15 |
| smcginnis | hemna: Ooh, not good. | 23:15 |
| hemna | looks like this wasn't tested by the cisco folks | 23:15 |
| hemna | :( | 23:15 |
| smcginnis | hemna: Something in the fczm code itself, or specific to cisco? | 23:15 |
| hemna | fczm code | 23:15 |
| *** harlowja has joined #openstack-cinder | 23:16 | |
| hemna | it tries to load the config for the fc-zone-manager group | 23:16 |
| hemna | and it never sees it | 23:16 |
| asmith_brcd | hemna, what's the fczm issue? | 23:16 |
| hemna | because it's only looking at the volume manager group | 23:16 |
| smcginnis | hemna: None of that ends up getting covered by CI unless the block vendors happen to use it, is it | 23:16 |
| asmith_brcd | can i help? | 23:16 |
| hemna | smcginnis, yah | 23:16 |
| hemna | asmith_brcd, I'm working on a fix at the moment. | 23:16 |
| hemna | asmith_brcd, https://bugs.launchpad.net/cinder/+bug/1424852 | 23:16 |
| openstack | Launchpad bug 1424852 in Cinder "Cinder fc_zone_manger with Cisco configured" [High,Confirmed] | 23:16 |
| hemna | for that | 23:16 |
| *** ganso_ has quit IRC | 23:17 | |
| *** mfisch has joined #openstack-cinder | 23:19 | |
| asmith_brcd | hemna, glad you are fixing it, thank you. FYI, any issues in fczm, i'm available | 23:19 |
| *** tbarron has quit IRC | 23:20 | |
| *** tbarron has joined #openstack-cinder | 23:20 | |
| hemna | asmith_brcd, sure | 23:21 |
| *** tbarron has quit IRC | 23:21 | |
| *** Mandell_ has joined #openstack-cinder | 23:26 | |
| *** ebalduf has joined #openstack-cinder | 23:27 | |
| *** Mandell has quit IRC | 23:29 | |
| *** ebalduf has quit IRC | 23:31 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: HP 3par driver filter and evaluator function https://review.openstack.org/151353 | 23:32 |
| *** krtaylor has quit IRC | 23:35 | |
| *** mfisch has quit IRC | 23:36 | |
| *** alexpilotti has joined #openstack-cinder | 23:36 | |
| *** jcru has quit IRC | 23:40 | |
| *** krtaylor has joined #openstack-cinder | 23:47 | |
| *** MasterPiece has joined #openstack-cinder | 23:53 | |
| *** mfisch has joined #openstack-cinder | 23:54 | |
| *** mfisch has quit IRC | 23:54 | |
| *** mfisch has joined #openstack-cinder | 23:54 | |
| *** marcusvrn has joined #openstack-cinder | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!