| *** jordandh has quit IRC | 00:00 | |
| *** bswartz has joined #openstack-cinder | 00:01 | |
| *** jordandh has joined #openstack-cinder | 00:02 | |
| *** lcurtis has quit IRC | 00:02 | |
| *** markstur has quit IRC | 00:05 | |
| *** rmesta has quit IRC | 00:06 | |
| *** markvoelker has quit IRC | 00:07 | |
| *** chlong has joined #openstack-cinder | 00:08 | |
| *** fischerw has quit IRC | 00:09 | |
| *** Lee1092 has joined #openstack-cinder | 00:10 | |
| *** r-daneel has quit IRC | 00:12 | |
| *** scottda_ has joined #openstack-cinder | 00:19 | |
| *** Mandell has quit IRC | 00:27 | |
| *** Mandell has joined #openstack-cinder | 00:32 | |
| *** Guest46399 has quit IRC | 00:47 | |
| *** smoriya has joined #openstack-cinder | 00:48 | |
| *** cppforlife_ has joined #openstack-cinder | 00:49 | |
| *** Longgeek has joined #openstack-cinder | 00:56 | |
| *** scottda_ has quit IRC | 01:01 | |
| *** Longgeek has quit IRC | 01:15 | |
| *** annashen has quit IRC | 01:18 | |
| *** jogo has left #openstack-cinder | 01:20 | |
| *** markstur has joined #openstack-cinder | 01:22 | |
| *** Mandell has quit IRC | 01:22 | |
| *** bill_az_ has quit IRC | 01:48 | |
| *** diegows has quit IRC | 01:53 | |
| openstackgerrit | QiangGuan proposed openstack/cinder: Remove useless storage_availability_zone import https://review.openstack.org/153932 | 01:55 |
|---|---|---|
| *** IanGovett has quit IRC | 01:56 | |
| *** lcurtis has joined #openstack-cinder | 01:57 | |
| *** lcurtis has quit IRC | 02:02 | |
| *** rwsu is now known as rwsu-afk | 02:02 | |
| *** casusbel_ has joined #openstack-cinder | 02:03 | |
| *** mberlin1 has joined #openstack-cinder | 02:04 | |
| *** mberlin has quit IRC | 02:05 | |
| *** casusbelli has quit IRC | 02:06 | |
| *** jnrao has quit IRC | 02:07 | |
| *** leeantho has quit IRC | 02:11 | |
| *** Longgeek has joined #openstack-cinder | 02:12 | |
| openstackgerrit | Gloria Gu proposed openstack/cinder: HP 3par driver filter and evaluator function https://review.openstack.org/151353 | 02:23 |
| openstackgerrit | harsh mishra proposed openstack/cinder: Fixing cinder-status value when the service is disabled https://review.openstack.org/154124 | 02:27 |
| *** ebalduf has quit IRC | 02:28 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Pool-aware scheduler support in EMC VNX Cinder driver https://review.openstack.org/147408 | 02:35 |
| *** Longgeek has quit IRC | 02:37 | |
| *** annashen has joined #openstack-cinder | 02:43 | |
| openstackgerrit | harsh mishra proposed openstack/cinder: Fixing cinder-status value when the service is disabled https://review.openstack.org/154124 | 02:47 |
| *** markvoelker has joined #openstack-cinder | 02:47 | |
| openstackgerrit | harsh mishra proposed openstack/cinder: Fixing cinder-status value when the service is disabled https://review.openstack.org/154124 | 02:48 |
| *** vilobhmm has quit IRC | 02:55 | |
| *** Yogi1 has joined #openstack-cinder | 03:06 | |
| *** jungleboyj has joined #openstack-cinder | 03:08 | |
| *** rushiagr_away is now known as rushiagr | 03:12 | |
| *** scottda_ has joined #openstack-cinder | 03:15 | |
| *** boris-42 has quit IRC | 03:22 | |
| *** davechen__ has quit IRC | 03:28 | |
| *** david-lyle is now known as david-lyle_afk | 03:29 | |
| *** DaveChen has joined #openstack-cinder | 03:29 | |
| *** ebalduf has joined #openstack-cinder | 03:29 | |
| *** BharatK has joined #openstack-cinder | 03:29 | |
| *** ebalduf has quit IRC | 03:33 | |
| *** Longgeek has joined #openstack-cinder | 03:37 | |
| *** Tross has joined #openstack-cinder | 03:42 | |
| *** kaufer has joined #openstack-cinder | 03:50 | |
| *** vilobhmm has joined #openstack-cinder | 04:06 | |
| *** mriedem has quit IRC | 04:07 | |
| *** Longgeek has quit IRC | 04:14 | |
| *** annashen has quit IRC | 04:16 | |
| *** mtanino has quit IRC | 04:23 | |
| *** scottda_ has quit IRC | 04:24 | |
| *** timcl has quit IRC | 04:24 | |
| *** annashen has joined #openstack-cinder | 04:27 | |
| *** ebalduf has joined #openstack-cinder | 04:30 | |
| *** jdg is now known as jgriffith | 04:34 | |
| *** harlowja is now known as harlowja_away | 04:34 | |
| *** jgriffith is now known as Guest3282 | 04:34 | |
| *** ebalduf has quit IRC | 04:35 | |
| *** Guest3282 is now known as jgriffith | 04:36 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Don't fail target_delete if target doesn't exist https://review.openstack.org/154249 | 04:38 |
| *** pradipta has joined #openstack-cinder | 04:38 | |
| *** tsekiyama has quit IRC | 04:38 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Mock out the wait routine in the VMAX driver https://review.openstack.org/154335 | 04:40 |
| *** Yogi1 has quit IRC | 04:42 | |
| *** Novtopro has joined #openstack-cinder | 04:46 | |
| *** bkopilov has joined #openstack-cinder | 04:46 | |
| *** vilobhmm has quit IRC | 04:49 | |
| *** avishay has joined #openstack-cinder | 04:50 | |
| *** coolsvap_ is now known as coolsvap | 04:52 | |
| *** Mandell has joined #openstack-cinder | 04:57 | |
| *** Novtopro has quit IRC | 05:00 | |
| *** casusbel_ has quit IRC | 05:12 | |
| *** annashen has quit IRC | 05:13 | |
| *** annashen has joined #openstack-cinder | 05:20 | |
| *** junhongl has joined #openstack-cinder | 05:22 | |
| *** Ilja has joined #openstack-cinder | 05:23 | |
| *** harlowja_at_home has joined #openstack-cinder | 05:25 | |
| *** enterprisedc has quit IRC | 05:26 | |
| *** enterprisedc has joined #openstack-cinder | 05:27 | |
| *** Ilja has quit IRC | 05:28 | |
| *** anshul has joined #openstack-cinder | 05:34 | |
| *** Xinxiaohui has joined #openstack-cinder | 05:34 | |
| *** BharatK has quit IRC | 05:41 | |
| *** annashen has quit IRC | 05:42 | |
| *** nkrinner has joined #openstack-cinder | 05:45 | |
| *** oomichi has quit IRC | 05:50 | |
| *** harlowja_at_home has quit IRC | 05:51 | |
| *** annashen has joined #openstack-cinder | 05:52 | |
| *** rushiagr is now known as rushiagr_away | 05:53 | |
| *** BharatK has joined #openstack-cinder | 05:58 | |
| *** avishay_ has joined #openstack-cinder | 06:00 | |
| *** avishay has quit IRC | 06:00 | |
| *** harlowja_away has quit IRC | 06:01 | |
| *** annashen has quit IRC | 06:08 | |
| *** annashen has joined #openstack-cinder | 06:16 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/152401 | 06:17 |
| openstackgerrit | Tina Tang proposed openstack/cinder: Attach/detach batch processing in VNX driver https://review.openstack.org/154350 | 06:19 |
| *** nshaikh has joined #openstack-cinder | 06:24 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Attach/detach batch processing in VNX driver https://review.openstack.org/154350 | 06:28 |
| *** kaufer has quit IRC | 06:30 | |
| *** lpetrut has joined #openstack-cinder | 06:41 | |
| *** nshaikh has quit IRC | 06:41 | |
| *** nshaikh has joined #openstack-cinder | 06:42 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Kilo Consistency Group API update https://review.openstack.org/144561 | 07:00 |
| openstackgerrit | xing-yang proposed openstack/cinder: Mock out the wait routine in the VMAX driver https://review.openstack.org/154335 | 07:01 |
| *** nshaikh has quit IRC | 07:09 | |
| *** Longgeek has joined #openstack-cinder | 07:15 | |
| *** Longgeek has quit IRC | 07:16 | |
| *** nshaikh has joined #openstack-cinder | 07:18 | |
| *** bkopilov has quit IRC | 07:21 | |
| *** Miouge has joined #openstack-cinder | 07:23 | |
| *** TobiasE has joined #openstack-cinder | 07:29 | |
| *** markvoelker has quit IRC | 07:33 | |
| *** markvoelker has joined #openstack-cinder | 07:34 | |
| *** markvoelker has quit IRC | 07:38 | |
| openstackgerrit | QiangGuan proposed openstack/cinder: Remove useless storage_availability_zone import https://review.openstack.org/153932 | 07:40 |
| openstackgerrit | QiangGuan proposed openstack/cinder: Remove useless storage_availability_zone import https://review.openstack.org/153932 | 07:42 |
| *** kaisers has joined #openstack-cinder | 07:52 | |
| *** chlong has quit IRC | 07:54 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Adapt volume manager to new driver structure https://review.openstack.org/138999 | 07:58 |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Split volume driver into ABC classes https://review.openstack.org/127455 | 07:58 |
| openstackgerrit | Nilesh Bhosale proposed openstack/cinder: IBM GPFS Consistency Group Implementation https://review.openstack.org/143987 | 08:00 |
| openstackgerrit | QiangGuan proposed openstack/cinder: Remove useless storage_availability_zone import https://review.openstack.org/153932 | 08:03 |
| *** fifieldt has joined #openstack-cinder | 08:03 | |
| *** markvoelker has joined #openstack-cinder | 08:04 | |
| *** annashen has quit IRC | 08:06 | |
| *** aix has joined #openstack-cinder | 08:07 | |
| *** annashen has joined #openstack-cinder | 08:07 | |
| *** markvoelker has quit IRC | 08:09 | |
| *** Raj_ has joined #openstack-cinder | 08:10 | |
| *** Longgeek has joined #openstack-cinder | 08:14 | |
| *** karimb has joined #openstack-cinder | 08:15 | |
| *** ndipanov has joined #openstack-cinder | 08:15 | |
| *** avishay_ has quit IRC | 08:15 | |
| *** annashen has quit IRC | 08:19 | |
| *** oomichi_ has joined #openstack-cinder | 08:23 | |
| *** avishay_ has joined #openstack-cinder | 08:24 | |
| *** avishay_ has quit IRC | 08:30 | |
| openstackgerrit | Tobias Engelbert proposed openstack/python-cinderclient: Recall mechanism for terminate/initialize https://review.openstack.org/154376 | 08:32 |
| openstackgerrit | Tobias Engelbert proposed openstack/cinder: Recall mechanism for terminate/initialize https://review.openstack.org/154377 | 08:32 |
| *** Mandell has quit IRC | 08:34 | |
| *** pwehrle has joined #openstack-cinder | 08:34 | |
| *** ronis has joined #openstack-cinder | 08:35 | |
| *** jordanP has joined #openstack-cinder | 08:36 | |
| openstackgerrit | Tobias Engelbert proposed openstack/cinder: Recall mechanism for terminate/initialize https://review.openstack.org/154377 | 08:39 |
| *** e0ne has joined #openstack-cinder | 08:43 | |
| *** avishay_ has joined #openstack-cinder | 08:46 | |
| *** jistr has joined #openstack-cinder | 08:47 | |
| *** afazekas has joined #openstack-cinder | 08:52 | |
| *** alecv has joined #openstack-cinder | 08:55 | |
| *** ronis has quit IRC | 08:55 | |
| *** ronis has joined #openstack-cinder | 08:55 | |
| *** Xinxiaohui has quit IRC | 08:57 | |
| *** TobiasE1 has joined #openstack-cinder | 08:59 | |
| *** TobiasE has quit IRC | 08:59 | |
| *** jpich has joined #openstack-cinder | 09:02 | |
| *** markvoelker has joined #openstack-cinder | 09:05 | |
| *** lpetrut has quit IRC | 09:07 | |
| *** lpetrut has joined #openstack-cinder | 09:08 | |
| *** nellysmitt has joined #openstack-cinder | 09:10 | |
| *** pradipta has quit IRC | 09:10 | |
| *** markvoelker has quit IRC | 09:10 | |
| openstackgerrit | Ph. Marek proposed openstack/cinder: DRBD: Use correct function object after DBus disconnect. https://review.openstack.org/150740 | 09:12 |
| *** anshul has quit IRC | 09:17 | |
| *** anshul has joined #openstack-cinder | 09:17 | |
| nikesh_vedams_ | hi | 09:19 |
| *** smoriya has quit IRC | 09:20 | |
| *** e0ne is now known as e0ne_ | 09:24 | |
| *** e0ne_ has quit IRC | 09:35 | |
| *** sgotliv has joined #openstack-cinder | 09:37 | |
| *** vnagpal has joined #openstack-cinder | 09:39 | |
| *** Ilja has joined #openstack-cinder | 09:46 | |
| *** bkopilov has joined #openstack-cinder | 09:47 | |
| *** nlevinki has joined #openstack-cinder | 09:50 | |
| *** e0ne has joined #openstack-cinder | 09:52 | |
| *** lpetrut has joined #openstack-cinder | 09:55 | |
| *** markvoelker has joined #openstack-cinder | 10:06 | |
| mkoderer | DuncanT: hi there, I split the ABC patch into two.. this is the fist one: https://review.openstack.org/#/c/127455/ | 10:10 |
| mkoderer | john asked me to do that.. hope it makes the review easier | 10:10 |
| *** markvoelker has quit IRC | 10:11 | |
| *** anshul has quit IRC | 10:11 | |
| *** anshul has joined #openstack-cinder | 10:12 | |
| DuncanT | Sorry, I thought I'd +2d the first one earlier, done now | 10:12 |
| DuncanT | I"ll get to the second one after lunch, but it looks fine at a glance | 10:13 |
| *** primechuck has joined #openstack-cinder | 10:15 | |
| mkoderer | DuncanT: cool thx :) | 10:24 |
| *** boris-42 has joined #openstack-cinder | 10:29 | |
| *** Longgeek has quit IRC | 10:37 | |
| *** primechuck has quit IRC | 10:42 | |
| *** Longgeek has joined #openstack-cinder | 10:45 | |
| *** rmstar_ has quit IRC | 10:50 | |
| *** dulek has joined #openstack-cinder | 10:56 | |
| *** sgotliv has quit IRC | 10:59 | |
| *** oomichi_ has left #openstack-cinder | 11:07 | |
| *** markvoelker has joined #openstack-cinder | 11:07 | |
| *** haomaiwang has quit IRC | 11:11 | |
| *** vnagpal has quit IRC | 11:11 | |
| *** vnagpal has joined #openstack-cinder | 11:12 | |
| *** markvoelker has quit IRC | 11:12 | |
| *** lpetrut has quit IRC | 11:13 | |
| *** lpetrut has joined #openstack-cinder | 11:18 | |
| *** EmilienM|afk is now known as EmilienM | 11:21 | |
| openstackgerrit | Kallebe Monteiro proposed openstack/python-cinderclient: Fix volume_transfers import in v2 https://review.openstack.org/154105 | 11:25 |
| *** diegows has joined #openstack-cinder | 11:26 | |
| *** aix has quit IRC | 11:26 | |
| *** diegows has quit IRC | 11:32 | |
| *** aviramb has quit IRC | 11:36 | |
| *** Ilja has quit IRC | 11:37 | |
| *** Ilja has joined #openstack-cinder | 11:37 | |
| openstackgerrit | Tobias Engelbert proposed openstack/cinder: Recall mechanism for terminate/initialize https://review.openstack.org/154377 | 11:47 |
| *** tbarron has joined #openstack-cinder | 11:56 | |
| *** rushiagr_away is now known as rushiagr | 11:56 | |
| *** ronis has quit IRC | 11:59 | |
| *** chlong has joined #openstack-cinder | 12:00 | |
| *** anshul has quit IRC | 12:07 | |
| *** anshul has joined #openstack-cinder | 12:07 | |
| *** e0ne is now known as e0ne_ | 12:08 | |
| *** markvoelker has joined #openstack-cinder | 12:08 | |
| *** Raj_ has quit IRC | 12:08 | |
| *** aix has joined #openstack-cinder | 12:11 | |
| *** markvoelker has quit IRC | 12:13 | |
| *** Longgeek has quit IRC | 12:13 | |
| *** e0ne_ has quit IRC | 12:18 | |
| *** aix has quit IRC | 12:20 | |
| *** changbl has quit IRC | 12:29 | |
| *** timcl has joined #openstack-cinder | 12:30 | |
| *** karimb is now known as karimb|lunch | 12:32 | |
| *** Miouge has quit IRC | 12:32 | |
| *** Miouge has joined #openstack-cinder | 12:33 | |
| *** Mandell has joined #openstack-cinder | 12:34 | |
| *** Mandell has quit IRC | 12:39 | |
| *** sgotliv has joined #openstack-cinder | 12:41 | |
| *** sgotliv has quit IRC | 12:43 | |
| *** sgotliv has joined #openstack-cinder | 12:44 | |
| *** IanGovett has joined #openstack-cinder | 12:46 | |
| erlon | DuncanT: ping | 12:56 |
| *** takedakn has joined #openstack-cinder | 12:57 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Adds pool aware scheduling for HNAS drivers https://review.openstack.org/142133 | 12:58 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Adds SSH communication to HNAS drivers https://review.openstack.org/142904 | 12:58 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix HNAS driver parsing errors https://review.openstack.org/141868 | 12:58 |
| *** takedakn has quit IRC | 12:59 | |
| *** takedakn has joined #openstack-cinder | 12:59 | |
| *** tshefi has joined #openstack-cinder | 13:02 | |
| *** markvoelker has joined #openstack-cinder | 13:05 | |
| *** avishay__ has joined #openstack-cinder | 13:06 | |
| *** avishay_ has quit IRC | 13:06 | |
| *** avishay__ is now known as avishay | 13:08 | |
| *** ronis has joined #openstack-cinder | 13:13 | |
| *** Yogi1 has joined #openstack-cinder | 13:16 | |
| *** rmstar has joined #openstack-cinder | 13:18 | |
| *** BharatK has quit IRC | 13:20 | |
| *** Yogi1 has quit IRC | 13:21 | |
| *** Ilja has quit IRC | 13:22 | |
| *** bswartz has quit IRC | 13:22 | |
| *** e0ne has joined #openstack-cinder | 13:25 | |
| *** karimb|lunch is now known as karim | 13:39 | |
| *** karim is now known as Guest24809 | 13:39 | |
| *** Miouge has quit IRC | 13:40 | |
| DuncanT | erlon: Hi | 13:42 |
| *** akerr has joined #openstack-cinder | 13:47 | |
| *** nshaikh has left #openstack-cinder | 13:54 | |
| *** rushiagr is now known as rushiagr_away | 13:54 | |
| dulek | Guys, is it possible to inject ipdb to cinder unit tests? | 13:54 |
| erlon | hi, Im noticing there's a lot of reviews only dealing with LOG formatting '%' vs ',' etc, is there any interest in automating this so run_tests -p also check the formating guidelines?? | 13:55 |
| erlon | DuncanT: ^ http://docs.openstack.org/developer/oslo.i18n/guidelines.html | 13:55 |
| DuncanT | erlon: Adding something to hacking would be fantastic! | 13:56 |
| smcginnis | erlon: I thought about doing that, but there are so many exceptions and format variations. | 13:56 |
| e0ne | erlon: we are talking about such check for a while | 13:56 |
| erlon | DuncanT: e0ne: but is run_check -p (PEP) the right place? this is not a PEP rule | 13:57 |
| DuncanT | erlon: I tried to write something that did arg check counting for LOG.foo lines, like python does already for %, but it turned out to be none-trivial | 13:57 |
| erlon | DuncanT: hmm | 13:57 |
| DuncanT | erlon: We have the hacking checks already that enforce style guidelines that are not from PEPs | 13:57 |
| e0ne | erlon, DuncanT: 'run_tests.sh -p' runs checks for pep8, pylint and hacking rules | 13:58 |
| *** emagana has joined #openstack-cinder | 13:58 | |
| DuncanT | erlon: Just a simple check for % inside LOG calls would catch a bunch of the things that keep coming up | 13:58 |
| *** bill_az_ has joined #openstack-cinder | 13:58 | |
| DuncanT | erlon: And checking for fixed strings that are untraslated / only translated with _() inside LOG.{warning|info|error} would catch a bunch more | 13:59 |
| *** xyang has joined #openstack-cinder | 13:59 | |
| erlon | DuncanT: mhm, Ill try to dig that and see what exceptions I find | 13:59 |
| *** Yogi1 has joined #openstack-cinder | 13:59 | |
| DuncanT | erlon: If you can get something that works, I think it would definitely be a help | 14:00 |
| erlon | DuncanT: nice | 14:00 |
| smcginnis | erlon, DuncanT: Part of the challenge would be being able to detect something like line 256 here: | 14:01 |
| smcginnis | https://review.openstack.org/#/c/143891/15/cinder/volume/drivers/emc/emc_vmax_masking.py | 14:01 |
| xyang | smcginnis: yes. the message is used by the caller | 14:02 |
| *** fifieldt has quit IRC | 14:02 | |
| smcginnis | xyang: That's why I think it's pretty much impossible to implement a hacking check for logging. | 14:02 |
| erlon | smcginnis: hmmm, hard to guess those kind of situations | 14:03 |
| xyang | smcginnis: even if the same error msg is used by an exception, it is hard to detect as well | 14:03 |
| smcginnis | erlon: It might be worth doing one locally and getting a full list that could be manually reviewed and cleaned up. | 14:04 |
| erlon | smcginnis: any other exception cases I should think about? | 14:04 |
| smcginnis | erlon: I just don't think we can make it a standard hacking check. | 14:04 |
| xyang | the usage of "%" vs "," is also not easy to differentiate | 14:05 |
| smcginnis | erlon: xyang's was the only specific one that came to mind, but I'm sure there's more. :) | 14:05 |
| smcginnis | xyang: So true. | 14:05 |
| erlon | smcginnis: is it possible to create a 'warning' checher? | 14:05 |
| smcginnis | Even in reviews folks have been leaving conflicting comments. | 14:05 |
| erlon | that wouldn't vote -1? | 14:05 |
| smcginnis | erlon: Cool, if it's just some kind of warning that can be addressed and cleaned up with possible, I'm all for that. | 14:06 |
| *** tbarron has quit IRC | 14:06 | |
| *** annegentle has joined #openstack-cinder | 14:06 | |
| *** bswartz has joined #openstack-cinder | 14:06 | |
| openstackgerrit | Sasikanth Eda proposed openstack/cinder: Volume Replication support - GPFS Synchronous mirroring https://review.openstack.org/153918 | 14:07 |
| *** david-lyle_afk is now known as david-lyle | 14:07 | |
| *** Ilja has joined #openstack-cinder | 14:08 | |
| *** primechuck has joined #openstack-cinder | 14:08 | |
| erlon | smcginnis: like the oslo.middleware deprecation warning that is appearing in thoses last weeks | 14:09 |
| smcginnis | erlon: My one concern with that is there are cases like the patch I mentioned where it will generate a warning even though it is legitimate. We will be stuck with this warning unless we do something to either flag it as OK in the code or change it so it no longer gets detected as non-compliant. | 14:10 |
| smcginnis | erlon: Maybe that's not really an issue though. Just a concern of mine. | 14:11 |
| *** xyang has quit IRC | 14:12 | |
| nikesh_vedams_ | smcginnis: hi | 14:14 |
| *** jasondot_ has joined #openstack-cinder | 14:14 | |
| nikesh_vedams_ | smcginnis: address your comments on https://review.openstack.org/#/c/145778/ | 14:15 |
| smcginnis | nikesh_vedams_: On my list to get to. Might be a little while. Trying to catch up from two back to back weeks of travel. | 14:15 |
| *** kaufer has joined #openstack-cinder | 14:16 | |
| nikesh_vedams_ | smcginnis: ok :) | 14:16 |
| *** xyang has joined #openstack-cinder | 14:19 | |
| xyang | DuncanT: can you please review the modify CG patch when you get a chance? https://review.openstack.org/#/c/144561/ | 14:20 |
| *** Miouge has joined #openstack-cinder | 14:20 | |
| *** dustins has joined #openstack-cinder | 14:21 | |
| xyang | e0ne: This one fixed the hang problem you ran into. give a try: https://review.openstack.org/#/c/154335/ | 14:24 |
| e0ne | xyang: thanks! i've just cheched out it and verifing | 14:24 |
| xyang | e0ne: ok, let me know if it works | 14:25 |
| e0ne | xyang: sure, i'll do | 14:25 |
| *** mriedem has joined #openstack-cinder | 14:25 | |
| e0ne | xyang: looks like it's working for me:). how did you debug it? i failed with it, adding logs didn't help me | 14:27 |
| xyang | e0n0: my coworker nailed it down. we know it must be one of those wait routines. so just tried to mock out all places where wait routine is used, mainly wait for job complete and wait for sync. lots of trial and error, because you can only repro with all tests running | 14:30 |
| *** takedakn has quit IRC | 14:31 | |
| *** jungleboyj has quit IRC | 14:31 | |
| xyang | e0ne | 14:31 |
| e0ne | thanks | 14:32 |
| xyang | e0ne: no problem. maybe now you can use your +2 power to help get it merged:) | 14:32 |
| e0ne | xyang: :). it is ok to +2 depended patch when parent it not reviewed? | 14:33 |
| xyang | e0ne: yes, it is | 14:33 |
| e0ne | i didn't take a look on a parent yet | 14:33 |
| *** Ilja has quit IRC | 14:33 | |
| xyang | e0ne: no problem. | 14:33 |
| xyang | e0ne: but it won't be merged until parent is merged | 14:34 |
| e0ne | that's why i'm asking about +2 on a child | 14:34 |
| xyang | e0ne: you can +2 on a child without reviewing the parent. | 14:35 |
| xyang | e0ne: it will just sit there | 14:35 |
| xyang | e0ne: actually I saw your name as reviewer on the parent as well. Seems that you've looked at it at some point | 14:35 |
| e0ne | done | 14:36 |
| xyang | e0ne: thanks | 14:36 |
| e0ne | xyang: thanks for fix it! | 14:37 |
| e0ne | btw, could you take a look on https://review.openstack.org/#/c/132648/, please? it hase more than 10 +1:) | 14:37 |
| xyang | e0ne: sure:) | 14:37 |
| *** Mandell has joined #openstack-cinder | 14:38 | |
| *** juzuluag has joined #openstack-cinder | 14:38 | |
| DuncanT | e0ne: I meant to ask you about that patch actually... it has zero info on what it is doing or why | 14:39 |
| *** rushiagr_away is now known as rushiagr | 14:40 | |
| xyang | e0ne: I don't understand it either. Just started reading... | 14:41 |
| e0ne | DuncanT: it removes unused code. i forgot to remove it while migration to oslo.db tools | 14:42 |
| xyang | e0ne: no bug number either | 14:42 |
| DuncanT | Ah, can you add that and a bug number to the commit message please? | 14:42 |
| xyang | that will be helpful. | 14:42 |
| e0ne | DuncanT: sure, i'll file a bug first | 14:42 |
| DuncanT | e0ne: :-) | 14:43 |
| DuncanT | e0ne: I'm happy to +2 once that is done | 14:44 |
| xyang | e0ne: me too. your explanation helps | 14:44 |
| xyang | DuncanT: about incremental backup, I think the parent_id option is still useful | 14:47 |
| *** aix has joined #openstack-cinder | 14:48 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Remove custom lazy loading https://review.openstack.org/132648 | 14:48 |
| DuncanT | xyang: While it has uses, is it worth more than the confusion it causes? The last backup taken is just about always the right one to chose | 14:48 |
| xyang | DuncanT: if a volume has 10 full backups and incremental backups on 3 of them, it is hard to choose | 14:48 |
| e0ne | done | 14:48 |
| xyang | DuncanT: I'm not sure if user will always just want the last one | 14:49 |
| e0ne | personaly, i hate bugs wich takes more time to file it instead of fixing | 14:49 |
| xyang | DuncanT: I can keep the parent_id option, make it optional, but choose the latest backup if parent id not provided? | 14:49 |
| DuncanT | xyang: Under what circumstances is the correct one to choose anything other than the latest one? | 14:50 |
| xyang | DuncanT: I'm just thinking user may want to choose an earlier backup as the base | 14:50 |
| DuncanT | e0ne: Agreed, but we need to keep a minimum level of admin going or history has shown we end up with a mess (quite often a political one at that) | 14:51 |
| DuncanT | xyang: Why would they want to do that? That is what I don't understand | 14:51 |
| DuncanT | xyang: Generally that just means they'll have a bigger set of diffs to backup | 14:51 |
| xyang | DuncanT: if the data on the later one is corrupted? | 14:52 |
| DuncanT | xyang: The backup itself is corrupted? They should do a full backup! | 14:52 |
| e0ne | DuncanT: sure. i can say about all project, but while Diablo and Essex development cycles, nova and hodizon(openstack-dashboard) didn't approve patches w/o links to launchpad issues or blueprints | 14:53 |
| DuncanT | xyang: If the contents of volume got corrupted in a previous backup? That is a sufficiently small corner case that the interface complexity of having the option isn't worth it IMO | 14:54 |
| *** nkrinner has quit IRC | 14:54 | |
| xyang | DuncanT: ok, if you don't see a user case for that, I'll just always pick the latest one. | 14:54 |
| DuncanT | e0ne: We should probably look at doing that, particularly later in the cycle | 14:54 |
| xyang | DuncanT: if there are two backups with identical timestamp, I'll just pick any one of them. sounds good? | 14:54 |
| DuncanT | xyang: Sure, it shouldn't matter | 14:55 |
| xyang | DuncanT: the data could be a little different because one may start a little later but finished quicker | 14:55 |
| DuncanT | xyang: Sure, but that be a tiny difference, and again is really a corner case | 14:56 |
| xyang | DuncanT: sure | 14:56 |
| mkoderer | DuncanT: xyang: folks, are you seeing any reason to have pyxattr in cinders requirements.txt? | 14:57 |
| mkoderer | https://github.com/openstack/cinder/blob/master/requirements.txt#L47 | 14:57 |
| *** jcru has joined #openstack-cinder | 14:57 | |
| mkoderer | It was introduced by https://review.openstack.org/#/c/94186/15 .. | 14:57 |
| mkoderer | the nova ppl rejected that and it was rewritten to use the xattr cli commands for nova volume | 14:58 |
| DuncanT | mkoderer: It looks like that driver imports xattr? | 14:58 |
| mkoderer | DuncanT: yeah... but "why"... IMHO the code looks ugly | 14:58 |
| mkoderer | DuncanT: so just because one driver need it we put it in global requirements.txt? | 14:59 |
| DuncanT | mkoderer: Ah, I see. I can't say I've looked. Can we ping the maintainer and ask about cleaning it up, see how much effort hey think it would be? | 14:59 |
| mkoderer | DuncanT: they already have the code in nova | 15:00 |
| DuncanT | mkoderer: That is the way global requirements work, generally. It is that or a conditional import. | 15:00 |
| *** r-daneel has joined #openstack-cinder | 15:00 | |
| mkoderer | DuncanT: ok we will propose a better solution and talk the the driver maintainer | 15:01 |
| mkoderer | DuncanT: how do I know who the driver maintainer btw? git blame? ;) | 15:01 |
| DuncanT | mkoderer: Sounds like a good first plan | 15:01 |
| mkoderer | dalgaaf: ^^ | 15:02 |
| *** e0ne is now known as e0ne_ | 15:02 | |
| *** nikesh_vedams_ has quit IRC | 15:02 | |
| DuncanT | mkoderer: Git blame is the most reliable at the moment. I tried to get a maintainers list in the source tree, but it has fallen by the wayside at the moment, until I'm less tied up iwith HP stuff | 15:02 |
| mkoderer | DuncanT: are you going to throw out drivers that don't have a maintainer? | 15:03 |
| DuncanT | mkoderer: If they have a third party CI and meet the minimum specs, they probably have somebody who can act as maintainer | 15:04 |
| DuncanT | mkoderer: If they don't have / get 3rd party CI in time, then the plan is to remove them | 15:04 |
| mkoderer | DuncanT: yeah +1 ... I see 3rd vendor test as a mandatory barrier for drivers | 15:05 |
| DuncanT | mkoderer: We've made that very clear as a cinder requirement... and the cutoff date is not far away | 15:05 |
| DuncanT | mkoderer: Mike is now doing the comms for that | 15:06 |
| *** e0ne_ is now known as e0ne | 15:07 | |
| *** emagana has quit IRC | 15:09 | |
| *** tbarron has joined #openstack-cinder | 15:09 | |
| *** emagana has joined #openstack-cinder | 15:09 | |
| e0ne | DuncanT: according to https://bugs.launchpad.net/cinder/+bug/1409012. looks like we need to get changes in oslo.messaging or implement such behavior as in nove-scheduler to get it works well | 15:11 |
| openstack | Launchpad bug 1409012 in Cinder "Volume becomes in 'error' state after scheduler starts" [High,In progress] - Assigned to Ivan Kolodyazhny (e0ne) | 15:11 |
| *** jungleboyj has joined #openstack-cinder | 15:12 | |
| mkoderer | jgriffith: hey, btw I rebased the ABC patches https://review.openstack.org/#/c/127455/ https://review.openstack.org/#/c/138999/ | 15:12 |
| *** Longgeek has joined #openstack-cinder | 15:14 | |
| *** emagana has quit IRC | 15:15 | |
| *** Mandell has quit IRC | 15:15 | |
| *** Longgeek has quit IRC | 15:16 | |
| *** thangp has joined #openstack-cinder | 15:16 | |
| *** ebalduf has joined #openstack-cinder | 15:16 | |
| *** coolsvap is now known as coolsvap_ | 15:17 | |
| *** fischerw has joined #openstack-cinder | 15:18 | |
| *** cnesa has joined #openstack-cinder | 15:18 | |
| *** ebalduf has quit IRC | 15:21 | |
| jgriffith | mkoderer: cool | 15:23 |
| jgriffith | checking it out | 15:23 |
| *** ebalduf has joined #openstack-cinder | 15:24 | |
| *** thingee has joined #openstack-cinder | 15:27 | |
| *** ebalduf has quit IRC | 15:28 | |
| *** nlevinki has quit IRC | 15:30 | |
| erlon | xyang: hi Xyang, you talked about the dependencies on your change (145952 -> 144561), are you working with then in the same local branch? | 15:32 |
| *** ebalduf has joined #openstack-cinder | 15:33 | |
| *** Yogi11 has joined #openstack-cinder | 15:34 | |
| *** dustins_ has joined #openstack-cinder | 15:34 | |
| *** avishay has quit IRC | 15:34 | |
| erlon | xyang: Im having a problem with that. When I rebase on upstream/master, all patches(changes) on my local branch are updated and trigger a new patchset on gerrit, even if I only change one. does that happent with you?? | 15:35 |
| *** TobiasE1 has quit IRC | 15:36 | |
| *** avishay has joined #openstack-cinder | 15:36 | |
| *** dustins has quit IRC | 15:37 | |
| *** Yogi1 has quit IRC | 15:37 | |
| xyang | erlon: yes, it happened to me. you need to do git review -R | 15:37 |
| erlon | xyang: hmm nice, Ill check that | 15:38 |
| *** thingee has quit IRC | 15:39 | |
| *** TobiasE has joined #openstack-cinder | 15:44 | |
| *** nlevinki has joined #openstack-cinder | 15:45 | |
| *** aviramb has joined #openstack-cinder | 15:45 | |
| erlon | xyang: hmm, so basically you do not rebase on gerrit/master. my branch was not automatically rebasing. I usually rebase manually to avoid conflicts | 15:45 |
| erlon | avishay: winston-d : guys, can you revisit this when you have a change? It is blocking 2 changes of HNAS drivers and haven't the comments was addressed quite a while ago | 15:46 |
| erlon | avishay: winston-d: https://review.openstack.org/#/c/141868/ | 15:47 |
| hemna | git pull origin hampster | 15:47 |
| hemna | not quite the same as master | 15:47 |
| jgriffith | hemna: you should make an alias :) | 15:50 |
| hemna | :) | 15:50 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support https://review.openstack.org/85847 | 15:50 |
| openstackgerrit | hadi esiely proposed openstack/cinder: blueprint volume-encryption-metadata https://review.openstack.org/154538 | 15:52 |
| *** rwsu-afk is now known as rwsu | 15:52 | |
| *** avishay has quit IRC | 15:54 | |
| *** cnesa has quit IRC | 15:55 | |
| *** kaufer has quit IRC | 15:57 | |
| *** rmesta has joined #openstack-cinder | 15:57 | |
| *** avishay has joined #openstack-cinder | 15:58 | |
| *** jaypipes has quit IRC | 16:01 | |
| *** rajinir_r has quit IRC | 16:01 | |
| *** rajinir_r has joined #openstack-cinder | 16:02 | |
| hemna | TobiasE, ping | 16:02 |
| *** tbarron has quit IRC | 16:02 | |
| *** rhagarty has quit IRC | 16:03 | |
| *** rhagarty has joined #openstack-cinder | 16:03 | |
| *** tbarron has joined #openstack-cinder | 16:03 | |
| *** tsekiyama has joined #openstack-cinder | 16:04 | |
| *** annashen has joined #openstack-cinder | 16:04 | |
| *** mtanino has joined #openstack-cinder | 16:04 | |
| *** nikesh_vedams has joined #openstack-cinder | 16:06 | |
| *** Mandell has joined #openstack-cinder | 16:06 | |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: quobyte: remove dependency to xattr https://review.openstack.org/154543 | 16:08 |
| openstackgerrit | hadi esiely proposed openstack/cinder: Store volume encryption metadata on each volume https://review.openstack.org/152284 | 16:08 |
| *** Mandell has quit IRC | 16:10 | |
| *** ndipanov has quit IRC | 16:10 | |
| *** pwehrle has quit IRC | 16:10 | |
| *** jkraj has joined #openstack-cinder | 16:11 | |
| DuncanT | mkoderer: I see there's a patch in to get rid of the xattr library usage. Can I ask what your basic objection is to adding a/the dependency is, please? In general I think it is always better to use a well designed library than a shell call | 16:13 |
| *** Miouge has quit IRC | 16:14 | |
| *** dustins_ has quit IRC | 16:15 | |
| openstackgerrit | hadi esiely proposed openstack/cinder: Store volume encryption metadata on each volume https://review.openstack.org/152284 | 16:16 |
| *** Miouge has joined #openstack-cinder | 16:17 | |
| *** annashen has quit IRC | 16:18 | |
| nikesh_vedams | thingee : there ? | 16:22 |
| *** Yogi11 has quit IRC | 16:22 | |
| *** Miouge has quit IRC | 16:24 | |
| *** kaufer has joined #openstack-cinder | 16:25 | |
| *** ollie1 has quit IRC | 16:25 | |
| *** Miouge has joined #openstack-cinder | 16:25 | |
| *** ebalduf has quit IRC | 16:27 | |
| *** Yogi11 has joined #openstack-cinder | 16:29 | |
| *** chlong has quit IRC | 16:30 | |
| *** jdurgin1 has joined #openstack-cinder | 16:30 | |
| *** ebalduf has joined #openstack-cinder | 16:35 | |
| *** e0ne is now known as e0ne_ | 16:36 | |
| *** e0ne_ is now known as e0ne | 16:39 | |
| *** nlevinki has quit IRC | 16:40 | |
| *** bkopilov has quit IRC | 16:41 | |
| e0ne | hemna: hi | 16:42 |
| *** lpetrut has quit IRC | 16:42 | |
| e0ne | hemna: i filed blueprint for https://review.openstack.org/#/c/144739/. looks too late for Kilo. we'll we target bug #1111839 to L too? | 16:43 |
| openstack | bug 1111839 in Cinder "Volume API response should list dependent snapshots" [Medium,In progress] https://launchpad.net/bugs/1111839 - Assigned to Ivan Kolodyazhny (e0ne) | 16:43 |
| hemna | e0ne, I don't see a BP for that review | 16:48 |
| hemna | e0ne, it's not in the commit message FWIW | 16:48 |
| *** rhagarty has quit IRC | 16:49 | |
| e0ne | hemna: because it's not implemented yet:( | 16:49 |
| * hemna is confused | 16:49 | |
| e0ne | hemna: proposed solution is to add 'child_snapshots' to volume view. it does'nt include '--detiails' param. i just want to clarify on which release this bug and blueprint (https://blueprints.launchpad.net/cinder/+spec/volume-details-views) should be landed | 16:51 |
| *** Guest24809 has quit IRC | 16:52 | |
| avishay | i don't understand why that's a bug - it's a feature request | 16:52 |
| e0ne | avishay: agree with you | 16:52 |
| jgriffith | avishay: +1 | 16:52 |
| hemna | yah that seems like a new feature | 16:53 |
| hemna | not a bug | 16:53 |
| e0ne | ok.. if everybody agree with it, lets close bug and i'll continue to work on it in scope of a new bp | 16:53 |
| *** bswartz has quit IRC | 16:53 | |
| jgriffith | e0ne: sounds right to me | 16:54 |
| avishay | yes | 16:54 |
| hemna | e0ne, and since this is a new cinder feature, it needs a spec | 16:55 |
| e0ne | hemna: sure. does it have a change to be landed in K? | 16:55 |
| jgriffith | jungleboyj: another one https://bugs.launchpad.net/cinder/+bug/1420335 | 16:55 |
| openstack | Launchpad bug 1420335 in Cinder "cinder is testing with non-namespaced oslo_i18n in Juno" [Undecided,New] | 16:55 |
| hemna | e0ne, the deadline for BP's is Feb 15th. | 16:56 |
| jungleboyj | jgriffith: Thanks. I know Jim is chasing that this morning. | 16:56 |
| e0ne | thanks. i'll try to implement it for K | 16:56 |
| *** afazekas has quit IRC | 16:56 | |
| *** coolsvap_ is now known as coolsvap | 16:56 | |
| e0ne | what is the valid status for bug whis moved to blueprints? | 16:57 |
| *** leeantho has joined #openstack-cinder | 16:59 | |
| jgriffith | e0ne: I'd just mark it as invalid and comment that you moved it to a BP (reference the BP) | 16:59 |
| avishay | e0ne: imporance->wishlist, status->invalid IMO | 16:59 |
| jgriffith | avishay: :) | 16:59 |
| avishay | jgriffith: who asked you? | 16:59 |
| jgriffith | avishay: eat me | 17:00 |
| avishay | jgriffith: :) | 17:00 |
| e0ne | jgriffith, avishay: thanks for help:). i'll do it later today. need to go offline for a while | 17:00 |
| jgriffith | LOL | 17:00 |
| avishay | i'm off, later all | 17:00 |
| *** avishay has quit IRC | 17:00 | |
| *** sgotliv has quit IRC | 17:00 | |
| *** bswartz has joined #openstack-cinder | 17:01 | |
| *** sgotliv has joined #openstack-cinder | 17:01 | |
| jgriffith | aviramb: night | 17:01 |
| jgriffith | oops... avishay | 17:01 |
| jgriffith | hemna: ok... time for me to look at and test multi-attach | 17:02 |
| hemna | yah! | 17:02 |
| *** e0ne has quit IRC | 17:06 | |
| *** jistr has quit IRC | 17:07 | |
| *** dustins has joined #openstack-cinder | 17:15 | |
| *** jkraj has quit IRC | 17:15 | |
| *** p01s0n has joined #openstack-cinder | 17:16 | |
| jgriffith | mtanino: ping | 17:22 |
| *** jpich has quit IRC | 17:22 | |
| *** coolsvap is now known as coolsvap_ | 17:23 | |
| hemna | testing out the ABC patch | 17:25 |
| *** bswartz has quit IRC | 17:25 | |
| *** anshul has quit IRC | 17:27 | |
| mtanino | jgriffith: pong | 17:28 |
| *** jordanP has quit IRC | 17:28 | |
| jgriffith | mtanino: just wanted to apologize for not getting back to you | 17:28 |
| mtanino | jgriffith: oh!! you gave me +2? | 17:28 |
| jgriffith | mtanino: and let you know that the refactor patch looked really good to me | 17:28 |
| mtanino | jgriffith: no problem | 17:28 |
| jgriffith | mtanino: LOL.. yes! | 17:28 |
| jgriffith | mtanino: I realized there is no breaking that up :) | 17:29 |
| mtanino | jgriffith: I think Duncan-T is intrested in this patch, so I will wait his review. | 17:29 |
| mtanino | jgriffith: anyway, thank you for the rewview. | 17:30 |
| jgriffith | mtanino: sounds like he wants to comb through it more closely | 17:30 |
| jgriffith | mtanino: you're welcome, thanks for working on it | 17:30 |
| jgriffith | mtanino: it's def an improvement | 17:30 |
| mtanino | jgriffith: :) | 17:30 |
| *** jasondot_ has quit IRC | 17:31 | |
| jgriffith | hemna: is there any chance that the Nova side of this is going to make it? | 17:31 |
| hemna | jgriffith, jaypipes was working on it | 17:32 |
| hemna | jgriffith, I asked him to get an extension for nova, but hadn't heard back from him yet on it. | 17:32 |
| *** annegentle has quit IRC | 17:32 | |
| hemna | so.... | 17:32 |
| jgriffith | hemna: what about the CI failures? | 17:33 |
| hemna | he's supposed to be working on the tempest tests | 17:33 |
| jgriffith | hemna: no, what I mean is it *breaks* the tests that are there | 17:33 |
| jgriffith | hemna: or are those a *blip* | 17:33 |
| jgriffith | hemna: lemme look closer | 17:33 |
| openstackgerrit | Thang Pham proposed openstack/cinder-specs: VMware: adapter_type volume type extra spec https://review.openstack.org/129389 | 17:33 |
| hemna | last I saw, Tobias had the unit tests working, and then it looks like jaypipes took it over | 17:33 |
| jgriffith | hemna: yeah, they break | 17:33 |
| hemna | to work on the nova tempest tests | 17:33 |
| hemna | and then it broke | 17:34 |
| jgriffith | hemna: indeed | 17:34 |
| hemna | so I'm waiting to hear from jaypipes about it, but I don't see him online | 17:34 |
| jgriffith | hemna: I would think this could/should be done without breaking compatability? | 17:34 |
| hemna | the cinder side changes shouldn't break compatibility | 17:34 |
| jgriffith | hemna: haha | 17:35 |
| hemna | as I made sure the attachment_id on detach is optional | 17:35 |
| jgriffith | hemna: yeah | 17:35 |
| hemna | and the attachments list in the volume object are the same | 17:35 |
| jgriffith | hemna: but it doesn't do us any good if it breaks Nova | 17:35 |
| hemna | agreed | 17:35 |
| jgriffith | hemna: and never merges there :( | 17:35 |
| hemna | so I'm not sure why the nova side is borked honestly | 17:35 |
| jgriffith | hemna: yeah, I'm looking now | 17:35 |
| hemna | if you look at my cinder patch, all of the tempest tests work | 17:36 |
| hemna | which include nova | 17:36 |
| hemna | so I think the nova failures are due to changes on their patches | 17:36 |
| jgriffith | hemna: ummmm | 17:37 |
| jgriffith | hemna: http://logs.openstack.org/14/143114/26/check/check-tempest-dsvm-full/27cd93b/logs/screen-n-api.txt.gz#_2015-02-02_19_39_25_402 | 17:37 |
| jgriffith | hemna: so wait... you're not working on the Nova patch now? | 17:37 |
| jgriffith | hemna: sorry, maybe I'm confused | 17:38 |
| hemna | I haven't worked on the nova side for this attempt | 17:38 |
| jgriffith | Tobias | 17:38 |
| hemna | Tobias and Jaypipes | 17:38 |
| jgriffith | ok, sorry to bug you on that | 17:38 |
| jgriffith | didn't notice | 17:38 |
| jgriffith | I remember you started it though | 17:38 |
| hemna | so | 17:38 |
| hemna | I bet you that error is happening, because that patch hasn't pulled the outstanding cinder patch | 17:39 |
| hemna | which has the multiattach attribute in it | 17:39 |
| *** anshul has joined #openstack-cinder | 17:39 | |
| hemna | cinder master doesn't | 17:39 |
| *** lpetrut has joined #openstack-cinder | 17:39 | |
| hemna | and hence the failure | 17:39 |
| jgriffith | hemna: no, it's the code itself it looks like | 17:39 |
| hemna | vol.multiattach doesn't exist in cinder master | 17:39 |
| jgriffith | hemna: and besides, it should be able to still maintain back compat here IMO | 17:39 |
| hemna | so that's a failure on their part | 17:40 |
| jgriffith | yeah | 17:40 |
| hemna | to be careful about accessing vol.multiattach | 17:40 |
| jgriffith | yeah | 17:40 |
| guitarzan | chickens and eggs | 17:40 |
| *** emagana has joined #openstack-cinder | 17:40 | |
| hemna | yup | 17:40 |
| jgriffith | hemna: FYI | 17:41 |
| jgriffith | you need to update your commit message | 17:41 |
| jgriffith | hemna: that patch has split | 17:41 |
| jgriffith | hemna: it's now https://review.openstack.org/#/c/153033/ | 17:41 |
| hemna | oh, to include both urls ? | 17:41 |
| jgriffith | hemna: and https://review.openstack.org/#/c/153038/ | 17:41 |
| hemna | ok will do | 17:41 |
| hemna | thanks | 17:41 |
| *** emagana_ has joined #openstack-cinder | 17:41 | |
| jgriffith | hemna: well the one you reference is no longer valid it appears | 17:41 |
| *** bswartz has joined #openstack-cinder | 17:42 | |
| *** TobiasE has quit IRC | 17:42 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support https://review.openstack.org/85847 | 17:42 |
| *** emagana has quit IRC | 17:45 | |
| *** aix has quit IRC | 17:47 | |
| nikesh_vedams | jgriffith: morning | 17:48 |
| *** lcurtis has joined #openstack-cinder | 17:49 | |
| *** annashen has joined #openstack-cinder | 17:49 | |
| *** e0ne has joined #openstack-cinder | 17:51 | |
| nikesh_vedams | jgriffith: your concern is valid but we are consistently active on any reviews given to us and addressed them all till now and we have addressed reviews like db concern etc | 17:51 |
| *** Mandell has joined #openstack-cinder | 17:51 | |
| *** rhagarty has joined #openstack-cinder | 17:53 | |
| nikesh_vedams | jgriffith: so please help us in making this patch for kilo :) | 17:53 |
| openstackgerrit | Gloria Gu proposed openstack/cinder: HP 3par driver filter and evaluator function https://review.openstack.org/151353 | 17:56 |
| nikesh_vedams | jgriffith: we will addressed any new concerns if you have and will not disturb anything in master | 17:56 |
| nikesh_vedams | jgriffith: just now saw your comment thanks | 17:57 |
| nikesh_vedams | sorry for flooding the channel | 17:57 |
| jgriffith | nikesh_vedams: understood | 17:57 |
| jgriffith | nikesh_vedams: yeah, I think we all agree you've been responsive and started early | 17:58 |
| jgriffith | nikesh_vedams: I'm willing to try and help get this merged for Kilo | 17:58 |
| *** changbl has joined #openstack-cinder | 17:58 | |
| *** ebalduf has quit IRC | 18:01 | |
| *** dannywilson has joined #openstack-cinder | 18:01 | |
| *** dannywilson has quit IRC | 18:02 | |
| *** dannywilson has joined #openstack-cinder | 18:02 | |
| *** Ilja has joined #openstack-cinder | 18:02 | |
| *** ebalduf has joined #openstack-cinder | 18:03 | |
| *** patrickeast has joined #openstack-cinder | 18:06 | |
| *** sgotliv has quit IRC | 18:06 | |
| nikesh_vedams | jgriffith: thanks john | 18:06 |
| *** Yogi11 has quit IRC | 18:07 | |
| *** ebalduf has quit IRC | 18:08 | |
| *** nellysmitt has quit IRC | 18:09 | |
| *** harlowja has joined #openstack-cinder | 18:10 | |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: quobyte: remove dependency to xattr https://review.openstack.org/154543 | 18:14 |
| *** jaypipes has joined #openstack-cinder | 18:15 | |
| *** vnagpal has quit IRC | 18:15 | |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: quobyte: remove dependency to xattr https://review.openstack.org/154543 | 18:15 |
| *** ebalduf has joined #openstack-cinder | 18:16 | |
| *** diegows has joined #openstack-cinder | 18:16 | |
| *** dalgaaf has quit IRC | 18:17 | |
| *** diegows has quit IRC | 18:18 | |
| *** Yogi1 has joined #openstack-cinder | 18:20 | |
| *** vilobhmm has joined #openstack-cinder | 18:21 | |
| *** vilobhmm has quit IRC | 18:21 | |
| *** ebalduf has quit IRC | 18:22 | |
| *** vilobhmm has joined #openstack-cinder | 18:22 | |
| *** openstackgerrit has quit IRC | 18:22 | |
| *** vilobhmm has quit IRC | 18:22 | |
| *** openstackgerrit has joined #openstack-cinder | 18:23 | |
| *** alecv has quit IRC | 18:24 | |
| *** annashen has quit IRC | 18:26 | |
| *** vilobhmm has joined #openstack-cinder | 18:27 | |
| *** Ilja1 has joined #openstack-cinder | 18:28 | |
| *** Ilja has quit IRC | 18:29 | |
| openstackgerrit | Gloria Gu proposed openstack/cinder: Update volume type name for volume type API https://review.openstack.org/140906 | 18:31 |
| *** Ilja1 has quit IRC | 18:33 | |
| *** Ilja has joined #openstack-cinder | 18:35 | |
| *** vnagpal has joined #openstack-cinder | 18:38 | |
| openstackgerrit | Gloria Gu proposed openstack/python-cinderclient: Update to change name for volume type client https://review.openstack.org/141187 | 18:39 |
| *** asmith_brcd has joined #openstack-cinder | 18:43 | |
| *** TobiasE has joined #openstack-cinder | 18:44 | |
| *** diemt has joined #openstack-cinder | 18:47 | |
| *** dulek has quit IRC | 18:47 | |
| *** annashen has joined #openstack-cinder | 18:49 | |
| jgriffith | jungleboyj: I think again we're not on the same page yet again | 18:51 |
| jgriffith | jungleboyj: What I'm saying is that backporting to use the oslo18n lib should have NEVER been done | 18:51 |
| jgriffith | jungleboyj: capping may have to be the answer now, but the right answer is/was not to allow that backport (assuming it was a backported change) | 18:52 |
| jgriffith | jungleboyj: see what I'm saying here? | 18:53 |
| jgriffith | jungleboyj: the changes Jim made to import oslo.i18n is the problem | 18:54 |
| jgriffith | that shouldn't be there at all | 18:54 |
| jgriffith | but I feel I'm not getting through again | 18:54 |
| jgriffith | not sure how to clarify this | 18:54 |
| *** bkopilov has joined #openstack-cinder | 18:55 | |
| *** nellysmitt has joined #openstack-cinder | 18:55 | |
| *** p01s0n has quit IRC | 18:57 | |
| *** lpetrut has quit IRC | 18:58 | |
| *** Mandell has quit IRC | 18:59 | |
| *** emagana_ has quit IRC | 19:00 | |
| *** lpetrut has joined #openstack-cinder | 19:04 | |
| *** rushiagr is now known as rushiagr_away | 19:06 | |
| *** Mandell has joined #openstack-cinder | 19:08 | |
| jungleboyj | jgriffith: So, I have been trying to make sense of this. There is a timing window here is the issue. | 19:10 |
| jungleboyj | jgriffith: We can rip out https://github.com/openstack/cinder/commit/3965a5f72984fabfa131ef5359a2959a80787a22#diff-e734ea39ff572cae552bf498a40bb33b but I don't feel that is the right answer. | 19:11 |
| *** EmilienM is now known as EmilienM|afk | 19:11 | |
| jungleboyj | jgriffith: Talking to Jim right now. | 19:14 |
| *** ebalduf has joined #openstack-cinder | 19:22 | |
| openstackgerrit | Gloria Gu proposed openstack/cinder: HP 3par driver filter and evaluator function https://review.openstack.org/151353 | 19:25 |
| openstackgerrit | Danny Wilson proposed openstack/cinder: Enabling volume replication on PureISCSIDriver https://review.openstack.org/145090 | 19:25 |
| *** kallebe has joined #openstack-cinder | 19:25 | |
| *** ebalduf has quit IRC | 19:27 | |
| openstackgerrit | Gloria Gu proposed openstack/cinder: HP 3par driver filter and evaluator function https://review.openstack.org/151353 | 19:27 |
| *** vnagpal has quit IRC | 19:31 | |
| *** ronenkat has joined #openstack-cinder | 19:32 | |
| *** Guest92220 has joined #openstack-cinder | 19:32 | |
| *** TobiasE has quit IRC | 19:33 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: Introducing micro_states for create workflow https://review.openstack.org/124205 | 19:34 |
| jgriffith | jungleboyj: no, there's not a timing window either :) | 19:35 |
| jgriffith | jungleboyj: I'll rework my patch and ask that you or Jim don't modify it again | 19:35 |
| jungleboyj | jgriffith: Jim just updated the bugs with an explanation. | 19:37 |
| jgriffith | jungleboyj: and it's still wrong | 19:38 |
| jungleboyj | Fine. | 19:38 |
| jgriffith | jungleboyj: don't get mad | 19:38 |
| jungleboyj | jgriffith: The proposal from oslo is to ping oslo.i18n at 1.3.1 . That resolves the bugs that are currently open. If that isn't satisfactory to you, you can rework your patch. | 19:39 |
| jungleboyj | *pin | 19:39 |
| jgriffith | jungleboyj: what I've been saying though and still not sure if we're on the same page | 19:39 |
| vilobhmm | xyang : addressed the comments you posted for https://review.openstack.org/#/c/124205/ please have a look | 19:39 |
| jgriffith | jungleboyj: we didn't have the oslo.i18n library in Juno | 19:40 |
| jgriffith | jungleboyj: and the only place it is now is in that test | 19:40 |
| jgriffith | jungleboyj: my point being that that shouldn't have been introduced in the first place to backport a fix for a unit test | 19:40 |
| jgriffith | jungleboyj: see what I mean? | 19:40 |
| smcginnis | jgriffith, jungleboyj: Not to jump in and cause any confusion, but I was confused as to why that was backported to Juno. Can someone explain that? | 19:40 |
| jgriffith | smcginnis: +1 | 19:40 |
| smcginnis | jgriffith: That's what you were question too, right? | 19:41 |
| jgriffith | smcginnis: Yes and no, that's one point, the other point is that there was an iteration that I provided that did not import the lib | 19:41 |
| jungleboyj | jgriffith: You are right. dhellmann agrees. | 19:41 |
| jungleboyj | So, lets back it out. | 19:41 |
| jgriffith | smcginnis: so even if it was broken in Juno it should've never introduced a dep on lib that wasn't there | 19:42 |
| jungleboyj | <dhellmann> jecarey: they shouldn't have done that, we'll be capping the libs to versions that don't include those changes. | 19:42 |
| jgriffith | smcginnis: it should have instead used the hack that I submitted without the fixture | 19:42 |
| smcginnis | jgriffith: Yeah, that's what surprised me about it. | 19:42 |
| jgriffith | smcginnis: indeed | 19:42 |
| jgriffith | but now I've made jungleboyj mad again :( | 19:42 |
| * jgriffith pictures Bruce Banner... "Don't make me angry, you wouldn't like me when I'm angry" | 19:43 | |
| smcginnis | jgriffith: Buy him a creme brulee. Or a milk shake. :) | 19:43 |
| jungleboyj | jgriffith: No, I am not angry. | 19:43 |
| jgriffith | My milk shake, is better than yours.... | 19:43 |
| jungleboyj | jgriffith: I am frustrated that I am not able to keep up with all the issues I am trying to tackle right now. | 19:44 |
| * smcginnis thinks jungleboyj does a lot | 19:44 | |
| jungleboyj | :-) | 19:44 |
| jgriffith | jungleboyj: it's like plugging a whole in the damn with your thumb | 19:44 |
| jungleboyj | jgriffith: Yep. | 19:45 |
| jgriffith | jungleboyj: the water never stops coming | 19:45 |
| jgriffith | That's why it takes a whole community | 19:45 |
| jungleboyj | jgriffith: So, you are right. We should not have backported the change that required namespacing and the fixture. That was our mistake so lets fix it and hopefully once we get the oslo libraries pinned this will be better. | 19:46 |
| jgriffith | jungleboyj: agreed, question is am I fixing it or is Jim? | 19:46 |
| jgriffith | jungleboyj: whole thing is silly anyway, it's just a unit test | 19:47 |
| * jungleboyj is laughing | 19:47 | |
| jungleboyj | I was trying not to think of that. | 19:47 |
| jungleboyj | You just want 3965a5f reverted? | 19:48 |
| jungleboyj | jgriffith: ^^ | 19:48 |
| jgriffith | jungleboyj: that won't work | 19:49 |
| jgriffith | jungleboyj: but you can try it if you want | 19:49 |
| jgriffith | jungleboyj: the whole problem is the "from oslo.i18n import _lazy" remember | 19:50 |
| jungleboyj | Oh yeah, Which we tried to resolve with this patch but we went too far. | 19:50 |
| jgriffith | jungleboyj: exactly | 19:51 |
| jgriffith | jungleboyj: I think the last version I put up that worked was 6 or 7 but not sure | 19:51 |
| jgriffith | jungleboyj: that was without the fixture | 19:51 |
| jgriffith | jungleboyj: if you want me to just do it I will, say the word. Otherwise I'm moving on to other issues now | 19:51 |
| jungleboyj | jgriffith: If you think you know the way to tackle this, why don't you do that. | 19:52 |
| *** kallebe has left #openstack-cinder | 19:53 | |
| jgriffith | k | 19:53 |
| jungleboyj | jgriffith: Then Jim can take a look at it. We need to make sure that we don't just leave i18n enabled for everything, though maybe that won't matter for Juno at this point. | 19:55 |
| jgriffith | jungleboyj: so that goes back to my original debate here | 19:55 |
| jgriffith | jungleboyj: if the unit tests pass and everything works who cares in this case? | 19:56 |
| *** nellysmitt has quit IRC | 19:56 | |
| jungleboyj | jgriffith: Agreed. | 19:56 |
| jungleboyj | Lets remove the dependency on 1.3.1, get the test passing and move on with life. | 19:57 |
| jgriffith | jungleboyj: and if you remember, I proposed adding the enable_lazy(False) and toggling it back off but you and Jim nixed that | 19:57 |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: quobyte: remove dependency to xattr https://review.openstack.org/154543 | 19:58 |
| jungleboyj | jgriffith: I don't remember why I said that. | 19:58 |
| jgriffith | jungleboyj: well that's when you overwrote my patch with your own version :) | 19:58 |
| *** vnagpal has joined #openstack-cinder | 19:59 | |
| * jungleboyj sighs | 19:59 | |
| *** vnagpal has quit IRC | 20:00 | |
| *** vnagpal has joined #openstack-cinder | 20:00 | |
| jgriffith | where's thingee in all of this? | 20:00 |
| *** Guest92220 is now known as annegentle | 20:01 | |
| jungleboyj | jgriffith: He is hiding like I would like to be doing. | 20:01 |
| jgriffith | jungleboyj: nahh... he's probably traveling | 20:01 |
| jungleboyj | In patch set six you are adding enable=True for enable_lazy. | 20:02 |
| jungleboyj | You removed the test cases in test_faults.py that failed. | 20:02 |
| jungleboyj | You updated test_wsgi to not use _lazy. | 20:03 |
| jungleboyj | So, depending on where they land the oslo.i18n pin we should be able to just do a revert. It depends on if they use a version if oslo.i18n that has _lazy in there | 20:07 |
| jungleboyj | trying to confirm that statement. | 20:08 |
| *** atmark has quit IRC | 20:10 | |
| *** Miouge has quit IRC | 20:11 | |
| jungleboyj | jgriffith: So, I propose this: | 20:12 |
| hemna | jgriffith, ping | 20:12 |
| jgriffith | hemna: pong | 20:12 |
| hemna | so, do you remember this guy? https://bugs.launchpad.net/cinder/+bug/1375487 | 20:13 |
| openstack | Launchpad bug 1375487 in Cinder "qemu-img "-t none" parameter fails image conversion on tmpfs" [High,Fix released] - Assigned to John Griffith (john-griffith) | 20:13 |
| *** AlkaD has joined #openstack-cinder | 20:13 | |
| jgriffith | I do indeed | 20:13 |
| hemna | heh ok | 20:13 |
| jungleboyj | jgriffith: Lets give dhellmann and jogo time to land on the library levels they want. Then I can test/propose the revert assuming they land where I think they will. | 20:13 |
| hemna | so I'm seeing a failure with -t none | 20:13 |
| jgriffith | jungleboyj: ok, I'm relinquishing ownership to you | 20:13 |
| hemna | the test for odirect works (dd count=0....) | 20:13 |
| jgriffith | :) | 20:13 |
| jungleboyj | jgriffith: Ok. I will keep you updated. | 20:14 |
| hemna | the -t none works with qemu-img version 2.0.0 | 20:14 |
| hemna | but fails with version 2.1.3 | 20:14 |
| *** bswartz has quit IRC | 20:14 | |
| *** Mandell has quit IRC | 20:15 | |
| hemna | I'm manually building qemu 2.1.3 at the moment to confirm | 20:15 |
| jgriffith | hemna: You'll need to give me something more to go on here I'm afraid | 20:16 |
| hemna | so ok | 20:16 |
| hemna | we have a distro here that has qemu 2.1.3 installed | 20:16 |
| hemna | running cinder | 20:16 |
| hemna | which tries to do copy image to volume | 20:16 |
| hemna | which eventually boils down to running the image_utils.py convert_image | 20:16 |
| hemna | which now does a test to see if it can do O_DIRECT | 20:17 |
| hemna | with call to dd count=0 if=image of=device | 20:17 |
| jgriffith | hemna: ok, so this is something in the qemu util in your custom distro for helion | 20:17 |
| hemna | jgriffith, yah | 20:17 |
| *** cnesa has joined #openstack-cinder | 20:17 | |
| hemna | which happens to use qemu 2.1.3 | 20:17 |
| jgriffith | hemna: yes, I know the code path, I meant the failure you're seeing | 20:17 |
| hemna | ubuntu 14.04 uses 2.0.0 | 20:17 |
| hemna | so the test succeeds | 20:17 |
| hemna | (dd count=0 if=image of=device) | 20:17 |
| hemna | but the usage of qemu-img -t none fails | 20:18 |
| hemna | with qemu-img 2.1.3 | 20:18 |
| hemna | I've tested it on ubuntu 14.04 with qemu-img 2.0.0 works fine | 20:18 |
| jgriffith | hemna: what's the error message? | 20:18 |
| jgriffith | hemna: pin your qemu version :) | 20:18 |
| *** bswartz has joined #openstack-cinder | 20:18 | |
| hemna | so yah, we are telling our folks to pin it | 20:18 |
| jgriffith | hemna: so I was half joking :) | 20:19 |
| *** annashen has quit IRC | 20:19 | |
| hemna | but I'm concerned about when ubuntu updates qemu-img | 20:19 |
| jgriffith | hemna: I mean short term that may be the answer but | 20:19 |
| jgriffith | yeah | 20:19 |
| jgriffith | agreed, looking at it | 20:19 |
| hemna | so | 20:19 |
| jgriffith | what's the error message? | 20:19 |
| hemna | (getting that sec) | 20:19 |
| hemna | so in the mean time, I'm manually building qemu 2.1.3 to do a test | 20:20 |
| kmartin | jgriffith, see http://paste.openstack.org/show/170927/ | 20:20 |
| jgriffith | kmartin: cool, thanks | 20:21 |
| *** vnagpal has quit IRC | 20:21 | |
| hemna | we also ran the commands manually | 20:21 |
| hemna | to prove that the -t none causes the failure | 20:21 |
| hemna | fwiw | 20:21 |
| jgriffith | hemna: oh sure | 20:21 |
| *** changbl has quit IRC | 20:21 | |
| kmartin | jgriffith, running it by hand http://paste.openstack.org/show/170928/ | 20:21 |
| jgriffith | kmartin: hemna FWIW, I believe that a qemu bug in that version | 20:22 |
| hemna | I'm hoping | 20:22 |
| jgriffith | http://lists.gnu.org/archive/html/qemu-devel/2014-11/msg01417.html | 20:22 |
| hemna | "Yeah, we should fix that." | 20:23 |
| hemna | lol | 20:23 |
| jgriffith | hemna: there's a patch up for it | 20:23 |
| jgriffith | hemna: try applying this: https://patchwork.ozlabs.org/patch/408683/ | 20:24 |
| *** ebalduf has joined #openstack-cinder | 20:24 | |
| jgriffith | hemna: should fix you up | 20:24 |
| hemna | awesome thanks | 20:24 |
| hemna | or....maybe update to 2.2.0 or 2.0.0 | 20:24 |
| jgriffith | hemna: and of course you'll have to figure out how helion wants to handle packaging to fix it until your debian distro updates to a version with a fix | 20:24 |
| jgriffith | hemna: yeah, that's up to you :) | 20:24 |
| hemna | yah....but that's not my problem :P | 20:24 |
| hemna | NMP! | 20:24 |
| kmartin | jgriffith, perfect thanks | 20:25 |
| jgriffith | hemna: if it were me and I were running helion distro I'd match the most common denominator | 20:25 |
| hemna | yah I'm not sure why they chose 2.1.3 | 20:25 |
| jgriffith | hemna: well, I think that's what's in 15.04 as well | 20:25 |
| jgriffith | haven't looked | 20:25 |
| hemna | anyway, thanks a bunch for the help! | 20:26 |
| jgriffith | hemna: sure | 20:26 |
| jgriffith | hemna: and even 15.04 is 2.2 it looks like FWIW but I could be wrong and dont' feel like digging in to it :) | 20:26 |
| hemna | https://launchpad.net/ubuntu/vivid/amd64/qemu-utils/1:2.1+dfsg-11ubuntu2 | 20:27 |
| hemna | smells like 2.1.x | 20:27 |
| jgriffith | hemna: yeah, but there's an update proposal out there | 20:27 |
| jgriffith | hemna: but anyway... not my thing | 20:28 |
| jgriffith | hemna: packaging stuff makes my head hurt | 20:28 |
| hemna | yah same here | 20:28 |
| hemna | it's pain | 20:28 |
| jgriffith | maybe they rejected 2.2 because of the bug you hit :) | 20:28 |
| *** ebalduf has quit IRC | 20:28 | |
| hemna | :P | 20:29 |
| btran | Hi, for the cinder CI requirement, I have my local CI running and was able to pass all tempest volume tests (however, some issues now). My jobs are being triggered by changes in the openstack-dev/sandbox, project. | 20:31 |
| btran | What is the next step or process I need to do to have my cinder job be part of the openstack-dev/cinder project? I can't fine any documentation on what to do next. thanks in advance | 20:31 |
| hemna | Liberty! | 20:34 |
| hemna | w00t | 20:34 |
| *** EmilienM|afk is now known as EmilienM | 20:35 | |
| jungleboyj | hemna: Yay! I voted for that. | 20:35 |
| btran | anyone? link me a wiki or doc that would be great | 20:41 |
| *** nellysmitt has joined #openstack-cinder | 20:42 | |
| jgriffith | btran: looking... you just need to change the project your listening to to be cinder | 20:43 |
| jgriffith | btran: but I don't use the std Jenkins/Ci method so need to poke around to figure out where that gets flipped | 20:43 |
| btran | jgriffith: you mean I just need to update Zuul layout.xml file? | 20:43 |
| jgriffith | asselin__: ^^ | 20:43 |
| jgriffith | btran: sure :) | 20:43 |
| btran | i see, that simple huh? there is no requirement or approval that I need to go through? | 20:44 |
| jgriffith | btran: so that assumes you have an account and all that stuff | 20:44 |
| jgriffith | btran: ie submitted a request for a third party ci account and recieved an approval and info back | 20:45 |
| jgriffith | btran: which means you have ssh keys etc | 20:45 |
| btran | jgriffith: cool thanks a lot | 20:45 |
| jgriffith | btran: certainly | 20:45 |
| *** Ilja has quit IRC | 20:53 | |
| *** AlkaD has quit IRC | 20:57 | |
| *** mriedem has quit IRC | 20:59 | |
| *** Mandell has joined #openstack-cinder | 21:01 | |
| *** mriedem has joined #openstack-cinder | 21:02 | |
| *** anshul has quit IRC | 21:02 | |
| *** ronenkat has quit IRC | 21:02 | |
| *** annashen has joined #openstack-cinder | 21:02 | |
| *** jcru has quit IRC | 21:03 | |
| *** tbarron has quit IRC | 21:05 | |
| *** sgotliv has joined #openstack-cinder | 21:05 | |
| *** ebalduf has joined #openstack-cinder | 21:05 | |
| *** thingee has joined #openstack-cinder | 21:05 | |
| scottda | boris-42: Is there a reason osprofiler/profiler.py:trace() decorator repeats 18 times during this Traceback: http://paste.openstack.org/show/170961/ | 21:09 |
| *** tbarron has joined #openstack-cinder | 21:09 | |
| boris-42 | scottda: hey there | 21:18 |
| boris-42 | scottda: seems like some magic regarding to mocking.. | 21:18 |
| scottda | Just curious. I don't think it's the cause of my issue, but I see it repeated 18X | 21:19 |
| scottda | OK. makes sense. | 21:19 |
| scottda | thanks | 21:19 |
| boris-42 | scottda: ya that seems bad | 21:20 |
| scottda | I've got many layers of badness going on here :) | 21:21 |
| *** Mandell has quit IRC | 21:23 | |
| *** annashen has quit IRC | 21:27 | |
| *** bswartz has quit IRC | 21:28 | |
| asselin__ | jgriffith, btran hi | 21:30 |
| *** Lee1092 has quit IRC | 21:30 | |
| *** Mandell has joined #openstack-cinder | 21:30 | |
| *** theanalyst has quit IRC | 21:30 | |
| asselin__ | btran, yes, just update the zuul layout.yaml openstack-dev/sandbox to openstack/cinder | 21:31 |
| *** vnagpal has joined #openstack-cinder | 21:31 | |
| *** timcl has quit IRC | 21:32 | |
| *** emagana has joined #openstack-cinder | 21:32 | |
| asselin__ | btran, no requirement or approval. however if there are issues with the ci system that don't get resolved then the whole account gets disabled | 21:32 |
| openstackgerrit | Steven Kaufer proposed openstack/cinder: Replication status periodic task optimization https://review.openstack.org/154673 | 21:33 |
| kaufer | jungleboyj: ^ | 21:33 |
| jungleboyj | kaufer: You the man. | 21:34 |
| *** theanalyst has joined #openstack-cinder | 21:35 | |
| *** atmark has joined #openstack-cinder | 21:38 | |
| *** atmark is now known as Guest19945 | 21:38 | |
| openstackgerrit | abhiram moturi proposed openstack/cinder: Enhance unit tests for zfssa drivers https://review.openstack.org/152676 | 21:41 |
| *** dannywilson has quit IRC | 21:44 | |
| *** dannywilson has joined #openstack-cinder | 21:45 | |
| *** lpetrut has quit IRC | 21:49 | |
| *** bswartz has joined #openstack-cinder | 21:49 | |
| *** Yogi1 has quit IRC | 21:51 | |
| *** asselin_ has quit IRC | 21:52 | |
| *** dannywilson has quit IRC | 21:53 | |
| *** asselin_ has joined #openstack-cinder | 21:53 | |
| *** asselin_ has quit IRC | 21:53 | |
| *** dannywilson has joined #openstack-cinder | 21:53 | |
| *** annashen has joined #openstack-cinder | 21:55 | |
| *** thingee has quit IRC | 21:55 | |
| *** sgordon_ has joined #openstack-cinder | 21:55 | |
| *** vnagpal has quit IRC | 21:57 | |
| *** thingee has joined #openstack-cinder | 21:57 | |
| *** vilobhmm has quit IRC | 21:58 | |
| *** nellysmitt has quit IRC | 22:03 | |
| *** Mandell has quit IRC | 22:03 | |
| *** e0ne has quit IRC | 22:05 | |
| *** dustins has quit IRC | 22:08 | |
| *** ronis has quit IRC | 22:08 | |
| *** bswartz has quit IRC | 22:11 | |
| *** jungleboyj has quit IRC | 22:11 | |
| *** sgotliv_ has joined #openstack-cinder | 22:11 | |
| *** sgotliv has quit IRC | 22:14 | |
| *** thangp has quit IRC | 22:15 | |
| *** mriedem has quit IRC | 22:19 | |
| *** mriedem has joined #openstack-cinder | 22:20 | |
| *** juzuluag has quit IRC | 22:26 | |
| *** mriedem has quit IRC | 22:27 | |
| *** diemt has quit IRC | 22:30 | |
| *** cnesa has quit IRC | 22:31 | |
| *** diegows has joined #openstack-cinder | 22:43 | |
| *** sgotliv__ has joined #openstack-cinder | 22:44 | |
| *** vilobhmm has joined #openstack-cinder | 22:47 | |
| *** sgotliv_ has quit IRC | 22:47 | |
| *** Mandell has joined #openstack-cinder | 22:58 | |
| *** dannywilson has quit IRC | 22:59 | |
| *** jungleboyj has joined #openstack-cinder | 22:59 | |
| *** dannywilson has joined #openstack-cinder | 23:01 | |
| *** dannywilson has quit IRC | 23:01 | |
| *** patrickeast_ has joined #openstack-cinder | 23:02 | |
| *** patrickeast has quit IRC | 23:02 | |
| *** annashen has quit IRC | 23:02 | |
| *** patrickeast_ is now known as patrickeast | 23:02 | |
| *** dannywil_ has joined #openstack-cinder | 23:03 | |
| *** dannywil_ has quit IRC | 23:04 | |
| *** dannywilson has joined #openstack-cinder | 23:04 | |
| *** thingee has quit IRC | 23:04 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add support to differential backups in cinder https://review.openstack.org/110068 | 23:07 |
| *** xyang has quit IRC | 23:07 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add support to incremental backups in cinder https://review.openstack.org/110068 | 23:08 |
| *** alexpilotti has joined #openstack-cinder | 23:09 | |
| *** alexpilotti has quit IRC | 23:10 | |
| *** alexpilotti has joined #openstack-cinder | 23:13 | |
| *** xyang has joined #openstack-cinder | 23:15 | |
| *** tbarron has quit IRC | 23:16 | |
| *** leeantho has quit IRC | 23:18 | |
| *** leeantho has joined #openstack-cinder | 23:18 | |
| *** fischerw has quit IRC | 23:20 | |
| *** Computer_Czar has joined #openstack-cinder | 23:24 | |
| *** thingee has joined #openstack-cinder | 23:31 | |
| *** annashen has joined #openstack-cinder | 23:35 | |
| *** akerr has quit IRC | 23:35 | |
| *** rmesta has quit IRC | 23:36 | |
| *** chlong has joined #openstack-cinder | 23:39 | |
| *** primechuck has quit IRC | 23:41 | |
| *** emagana has quit IRC | 23:46 | |
| *** bitblt has joined #openstack-cinder | 23:46 | |
| *** emagana has joined #openstack-cinder | 23:47 | |
| bitblt | if i only have one nfs mount point, and i decide i want to change it to something else, eg a re-IP, is it right that I'll have to manually change the provider_location in the db, then update the domain xml file to the new location too? | 23:49 |
| *** bswartz has joined #openstack-cinder | 23:51 | |
| *** emagana has quit IRC | 23:51 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: Introducing micro_states for create workflow https://review.openstack.org/124205 | 23:54 |
| *** annashen has quit IRC | 23:56 | |
| *** asmith_brcd has quit IRC | 23:56 | |
| *** kaufer has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!