| *** gouthamr_ has joined #openstack-cinder | 00:04 | |
| *** alonma has joined #openstack-cinder | 00:06 | |
| *** gouthamr has quit IRC | 00:08 | |
| ildikov | smcginnis: on the airport, complementary wifi lasts for 30 minutes, after that you're on your own... | 00:11 |
|---|---|---|
| *** alonma has quit IRC | 00:11 | |
| *** jungleboyj has quit IRC | 00:13 | |
| *** sdake has joined #openstack-cinder | 00:14 | |
| *** leeantho has quit IRC | 00:14 | |
| *** mtanino has quit IRC | 00:16 | |
| *** Suyi_ has quit IRC | 00:16 | |
| *** lixiaoy1 has joined #openstack-cinder | 00:18 | |
| *** akerr has joined #openstack-cinder | 00:19 | |
| smcginnis | ildikov: Safe travels! | 00:25 |
| ildikov | smcginnis: thanks! | 00:26 |
| *** alonma has joined #openstack-cinder | 00:27 | |
| ildikov | smcginnis: I'm aiming for some sleep and dreams of my pillow :) | 00:27 |
| *** Apoorva has quit IRC | 00:27 | |
| lixiaoy1 | smcginni: Hey! May I beg your time to have a look at this patch https://review.openstack.org/#/c/319078/ (retype encrypted volume)? | 00:28 |
| smcginnis | ildikov: ;) | 00:28 |
| smcginnis | lixiaoy1: I'll try to take a look in a bit. | 00:28 |
| smcginnis | lixiaoy1: At least have it open in a tab now. | 00:28 |
| lixiaoy1 | smcginnis: I know you are busy. Sorry | 00:28 |
| ildikov | smcginnis: was your trips back all ok? | 00:29 |
| smcginnis | lixiaoy1: No problem. :) | 00:29 |
| smcginnis | ildikov: Yep. A little delay taking off, but we actually landed ahead of schedule. | 00:30 |
| smcginnis | ildikov: Hopefully yours goes as well. | 00:30 |
| lixiaoy1 | smcginnis: also this patch https://review.openstack.org/#/c/216567/ (created bootable encrypted volume). You and John have ever given +2, and I just rebase it. | 00:30 |
| smcginnis | lixiaoy1: OK, great. Got it. | 00:30 |
| ildikov | smcginnis: that sounds rather usual, so I hope mine will be similar :) | 00:31 |
| lixiaoy1 | smcginnis: Thank you | 00:31 |
| smcginnis | ildikov: No long layovers! | 00:31 |
| *** alonma has quit IRC | 00:32 | |
| ildikov | smcginnis: hope not! By now I could sleep anywhere and I know it will go away the minute I step on the plane... | 00:33 |
| smcginnis | ildikov: I always fall asleep waiting to take off, then as soon as we do I wake up and that's it. | 00:34 |
| *** alonma has joined #openstack-cinder | 00:47 | |
| ildikov | smcginnis: sounds too familiar unfortunately | 00:48 |
| ildikov | smcginnis: but I have a thick book with me for this case :) | 00:49 |
| *** alonma has quit IRC | 00:51 | |
| smcginnis | ildikov: Good time to catch up on some reading. ;) | 00:54 |
| *** akerr has quit IRC | 00:55 | |
| *** cheneydc has joined #openstack-cinder | 00:55 | |
| openstackgerrit | haobing proposed openstack/cinder-specs: Reset Cgsnapshot state https://review.openstack.org/361927 | 00:56 |
| *** laughterwym has joined #openstack-cinder | 01:04 | |
| *** alonma has joined #openstack-cinder | 01:07 | |
| *** bardia has joined #openstack-cinder | 01:08 | |
| *** alonma has quit IRC | 01:11 | |
| *** raunak has quit IRC | 01:13 | |
| *** kaisers_ has quit IRC | 01:14 | |
| ildikov | smcginnis: exactly, I accomplished 122 pages on the trip here in English and I'm a slow reader... :) | 01:14 |
| *** sdake has quit IRC | 01:14 | |
| ildikov | smcginnis: and the aircraft is getting ready, so have s good evening | 01:15 |
| ildikov | smcginnis: talk to you later this week | 01:15 |
| *** yangyapeng has joined #openstack-cinder | 01:18 | |
| *** sdake has joined #openstack-cinder | 01:21 | |
| *** bardia has quit IRC | 01:24 | |
| *** zhangguoqing has joined #openstack-cinder | 01:25 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Retype encrypted volumes https://review.openstack.org/319078 | 01:27 |
| openstackgerrit | LisaLi proposed openstack/cinder: Add encryptor attach/detach in utils https://review.openstack.org/341914 | 01:27 |
| *** alonma has joined #openstack-cinder | 01:28 | |
| *** mriedem_afk has quit IRC | 01:29 | |
| *** ociuhandu has joined #openstack-cinder | 01:32 | |
| *** alonma has quit IRC | 01:32 | |
| *** salv-orlando has joined #openstack-cinder | 01:32 | |
| *** cheneydc has quit IRC | 01:32 | |
| *** cheneydc has joined #openstack-cinder | 01:33 | |
| *** cheneydc has quit IRC | 01:33 | |
| *** ociuhandu has quit IRC | 01:34 | |
| *** Lee1092 has joined #openstack-cinder | 01:36 | |
| *** EinstCrazy has joined #openstack-cinder | 01:38 | |
| *** salv-orlando has quit IRC | 01:43 | |
| openstackgerrit | zhufl proposed openstack/cinder: Remove unused config.CONF https://review.openstack.org/361814 | 01:44 |
| *** bardia has joined #openstack-cinder | 01:50 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Fixes the length of resource field in database for quota tables https://review.openstack.org/355214 | 01:50 |
| *** bardia has quit IRC | 01:52 | |
| *** akerr has joined #openstack-cinder | 01:59 | |
| openstackgerrit | xianming.mao proposed openstack/python-brick-cinderclient-ext: Update flake8 ignore list https://review.openstack.org/359572 | 02:01 |
| *** EinstCra_ has joined #openstack-cinder | 02:03 | |
| openstackgerrit | wanghao proposed openstack/cinder: Fix managing volume stuck in 'creating' https://review.openstack.org/361688 | 02:03 |
| *** EinstCrazy has quit IRC | 02:06 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Switch service capabilities to ovo https://review.openstack.org/319040 | 02:16 |
| *** ducttape_ has joined #openstack-cinder | 02:18 | |
| *** akerr is now known as akerr_away | 02:19 | |
| *** akerr_away is now known as akerr | 02:19 | |
| *** akerr is now known as akerr_away | 02:20 | |
| *** akerr_away is now known as akerr | 02:20 | |
| *** akerr is now known as akerr_away | 02:20 | |
| *** akerr_away is now known as akerr | 02:20 | |
| *** akerr is now known as akerr_away | 02:21 | |
| *** akerr_away is now known as akerr | 02:21 | |
| *** akerr is now known as akerr_away | 02:22 | |
| *** akerr_away is now known as akerr | 02:22 | |
| *** akerr is now known as akerr_away | 02:23 | |
| *** akerr_away is now known as akerr | 02:23 | |
| openstackgerrit | wanghao proposed openstack/cinder: Fix managing volume stuck in 'creating' https://review.openstack.org/361688 | 02:25 |
| openstackgerrit | xianming.mao proposed openstack/cinder: Add programming launguage python 3 in classifier. https://review.openstack.org/362530 | 02:26 |
| *** EinstCrazy has joined #openstack-cinder | 02:28 | |
| *** gouthamr_ has quit IRC | 02:30 | |
| *** EinstCra_ has quit IRC | 02:32 | |
| *** magic has joined #openstack-cinder | 02:32 | |
| *** magic is now known as Guest33431 | 02:32 | |
| *** xiaoyang has quit IRC | 02:35 | |
| openstackgerrit | Pony Chou proposed openstack/cinder: Add QNAP ES Storage Driver https://review.openstack.org/309872 | 02:36 |
| *** alonma has joined #openstack-cinder | 02:39 | |
| openstackgerrit | xianming.mao proposed openstack/cinder: Add py35 in envlist because we support py35 now. https://review.openstack.org/362575 | 02:39 |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/*.py (2/2) https://review.openstack.org/355815 | 02:42 |
| *** alonma has quit IRC | 02:43 | |
| *** thurloat has quit IRC | 02:47 | |
| *** alonma has joined #openstack-cinder | 02:49 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Update attach_volume() with versionedobjects https://review.openstack.org/268608 | 02:49 |
| *** alonma has quit IRC | 02:53 | |
| *** ociuhandu has joined #openstack-cinder | 02:53 | |
| *** gluytium has quit IRC | 02:55 | |
| openstackgerrit | Merged openstack/cinder: Stop using mox in unit/backup/drivers/*.py https://review.openstack.org/359592 | 02:56 |
| openstackgerrit | Merged openstack/cinder: Stop using mox in unit/zonemanager/*.py tests https://review.openstack.org/359598 | 02:57 |
| openstackgerrit | Merged openstack/cinder: Stop using mox in unit/scheduler/*.py tests https://review.openstack.org/359603 | 02:57 |
| openstackgerrit | Merged openstack/cinder: Stop using mox in unit/consistencygroup/*.py tests https://review.openstack.org/359606 | 02:57 |
| openstackgerrit | Merged openstack/cinder: Stop using mox in unit/volume/drivers/emc/*.py tests https://review.openstack.org/359613 | 02:57 |
| *** gluytium has joined #openstack-cinder | 02:58 | |
| openstackgerrit | Merged openstack/python-cinderclient: Replace functions 'Dict.get' and 'del' with 'Dict.pop' https://review.openstack.org/361856 | 02:58 |
| *** akerr_ has joined #openstack-cinder | 03:02 | |
| *** ducttape_ has quit IRC | 03:02 | |
| *** kaisers_ has joined #openstack-cinder | 03:02 | |
| *** akerr has quit IRC | 03:06 | |
| *** ociuhandu has quit IRC | 03:07 | |
| *** kaisers_ has quit IRC | 03:08 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: Storwize: do the correct update after failover https://review.openstack.org/292570 | 03:11 |
| *** itzdilip has joined #openstack-cinder | 03:14 | |
| *** abhitechie has joined #openstack-cinder | 03:22 | |
| openstackgerrit | xianming.mao proposed openstack/cinder: Drop unused import cfg https://review.openstack.org/362583 | 03:22 |
| *** akerr_ has quit IRC | 03:23 | |
| openstackgerrit | xing-yang proposed openstack/cinder: WIP: Add functional tests for groups https://review.openstack.org/362584 | 03:23 |
| *** ManishD has joined #openstack-cinder | 03:33 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Update detach_volume() with versionedobjects https://review.openstack.org/268609 | 03:34 |
| *** abhitechie has quit IRC | 03:42 | |
| jgriffith | Ummmm | 03:44 |
| jgriffith | anybody notice that something broke cinder a bit today? | 03:44 |
| *** salv-orlando has joined #openstack-cinder | 03:45 | |
| lixiaoy1 | jgriffith: how do you know? | 03:49 |
| jgriffith | lixiaoy1: my CI started failing every job for snapshot tests at 4:15 MDT today :) | 03:50 |
| jgriffith | lixiaoy1: logs have some weird issue with the c-vol service just constantly restarting | 03:50 |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Convert unicode to string when disabling a service https://review.openstack.org/361105 | 03:50 |
| jgriffith | lixiaoy1: pulling down latest and having a look locally now | 03:50 |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Add filters support to get_pools https://review.openstack.org/321492 | 03:51 |
| lixiaoy1 | jgriffith: I just submitted my this morning, and jeckins succeeded | 03:51 |
| *** links has joined #openstack-cinder | 03:51 | |
| jgriffith | lixiaoy1: Oh yeah, I don't think your patch has anything to do with it | 03:51 |
| jgriffith | lixiaoy1: and yes, Jenkins seems to be *ok*... I need to check ER though | 03:52 |
| lixiaoy1 | jgriffith: do you have logs online? | 03:52 |
| jgriffith | lixiaoy1: yes | 03:53 |
| *** salv-orlando has quit IRC | 03:53 | |
| jgriffith | lixiaoy1: http://54.164.167.86/solidfire-ci-logs/refs-changes-14-341914-12/logs | 03:53 |
| jgriffith | is one example | 03:54 |
| jgriffith | lixiaoy1: http://paste.openstack.org/show/564729/ | 03:54 |
| jgriffith | lixiaoy1: notice that repeats over and over and over | 03:55 |
| lixiaoy1 | lixiaoy1: let me look | 03:55 |
| jgriffith | lixiaoy1: but looking at some of the other failures that's not consistent | 03:55 |
| jgriffith | lixiaoy1: it's part of the object updates it appears | 03:55 |
| *** laughterwym has quit IRC | 03:58 | |
| *** abhitechie has joined #openstack-cinder | 03:59 | |
| jgriffith | lixiaoy1: I'll have a look in about 1/2 hour when stack finally finishes :( | 03:59 |
| *** alonma has joined #openstack-cinder | 04:00 | |
| *** ducttape_ has joined #openstack-cinder | 04:02 | |
| *** alonma has quit IRC | 04:04 | |
| lixiaoy1 | jgriffith: why do you say that c-vol service constantly restarting? As from the log, I only see it started once: Starting cinder-volume node (version 9.0.0) | 04:07 |
| *** raunak has joined #openstack-cinder | 04:07 | |
| jgriffith | lixiaoy1: yeah... sorry, not really that it's restarting... but it's initializing the DB over and over. that second line in the paste | 04:08 |
| jgriffith | lixiaoy1: typically you see that once per service start/init | 04:08 |
| *** ducttape_ has quit IRC | 04:09 | |
| *** lixiaoy1 has quit IRC | 04:09 | |
| *** lixiaoy1 has joined #openstack-cinder | 04:10 | |
| jgriffith | lixiaoy1: unless that's a new *feature* :) | 04:10 |
| jgriffith | lixiaoy1: which I suppose it very well could be | 04:10 |
| lixiaoy1 | jgriffith: anything you just said before " unless that's a new *feature* :)"? as my network disconnect one minute | 04:12 |
| jgriffith | lixiaoy1: yeah... sorry, not really that it's restarting... but it's initializing the DB over and over. that second line in the paste | 04:12 |
| jgriffith | [22:08:56] jgriffith lixiaoy1: typically you see that once per service start/init | 04:12 |
| jgriffith | [22:10:14] jgriffith lixiaoy1: unless that's a new *feature* :) | 04:12 |
| jgriffith | anyway... maybe it's my CI system or a problem in my code. I'll dig into it | 04:13 |
| *** ducttape_ has joined #openstack-cinder | 04:13 | |
| jgriffith | lixiaoy1: there is an interesting error on my side further down the line there | 04:14 |
| *** ducttape_ has quit IRC | 04:15 | |
| lixiaoy1 | jgriffith: from the logs, I can see lots of this error: SolidFireAPIException: API response: {u'id': None, u'error': {u'message': u'Unknown slice ID sliceID=12800 mServiceID=8', u'code': 500, u'name': u'xSliceNotRegistered'}} | 04:19 |
| lixiaoy1 | jgriffith: the error happens in the backend storage, I guess may we transferred wrong parameter. but if you have environment, may you run the failed test and have a check | 04:20 |
| lixiaoy1 | jgriffith: what is the intersting error? | 04:21 |
| ManishD | Good morning Cinder | 04:22 |
| jgriffith | the missing slice | 04:22 |
| lixiaoy1 | jgriffith: I don't understand the error. But name is xSliceNotRegistered? any problem here? | 04:25 |
| jgriffith | Yes, it's a big problem :) | 04:25 |
| openstackgerrit | Ha Van Tu proposed openstack/cinder: Fix using filter() to meet python2,3 https://review.openstack.org/362601 | 04:26 |
| jgriffith | lixiaoy1: I have a fix, just need to run some cycles against it to verify it does in fact address the issue | 04:28 |
| lixiaoy1 | jgriffith: congrats :) | 04:29 |
| jgriffith | lixiaoy1: haha | 04:29 |
| lixiaoy1 | jgriffith: talk to you later, I am going to have lunch now | 04:29 |
| *** raunak has quit IRC | 04:33 | |
| jgriffith | lixiaoy1: enjoy! | 04:35 |
| *** ntpttr- has quit IRC | 04:36 | |
| *** ntpttr- has joined #openstack-cinder | 04:40 | |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/v2/ tests https://review.openstack.org/356347 | 04:40 |
| *** ntpttr has quit IRC | 04:41 | |
| *** ntpttr- is now known as ntpttr | 04:41 | |
| *** salv-orlando has joined #openstack-cinder | 04:41 | |
| *** ntpttr__ has joined #openstack-cinder | 04:41 | |
| *** ntpttr has quit IRC | 04:44 | |
| *** ntpttr has joined #openstack-cinder | 04:45 | |
| *** lixiaoy1 has quit IRC | 04:50 | |
| *** kaisers_ has joined #openstack-cinder | 04:52 | |
| *** kaisers_ has quit IRC | 04:57 | |
| *** laughterwym has joined #openstack-cinder | 04:58 | |
| *** alonma has joined #openstack-cinder | 05:00 | |
| *** jdurgin has quit IRC | 05:03 | |
| *** laughterwym has quit IRC | 05:03 | |
| *** jdurgin has joined #openstack-cinder | 05:04 | |
| *** lixiaoy1 has joined #openstack-cinder | 05:04 | |
| *** alonma has quit IRC | 05:05 | |
| *** lifeless_ is now known as lifeless | 05:15 | |
| *** ducttape_ has joined #openstack-cinder | 05:16 | |
| *** alonma has joined #openstack-cinder | 05:20 | |
| *** ducttape_ has quit IRC | 05:21 | |
| *** itzdilip has quit IRC | 05:22 | |
| *** alonma has quit IRC | 05:24 | |
| *** bkopilov has joined #openstack-cinder | 05:27 | |
| openstackgerrit | wanghao proposed openstack/cinder: Fix managing volume stuck in 'creating' https://review.openstack.org/361688 | 05:27 |
| *** pdeore has joined #openstack-cinder | 05:32 | |
| *** alonma has joined #openstack-cinder | 05:32 | |
| *** Poornima has joined #openstack-cinder | 05:34 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO: fix the returned value of CG actions https://review.openstack.org/346831 | 05:36 |
| *** alonma has quit IRC | 05:36 | |
| *** sdake has quit IRC | 05:47 | |
| *** sandanar has joined #openstack-cinder | 05:47 | |
| *** nkrinner_afk is now known as nkrinner | 05:48 | |
| *** sdake has joined #openstack-cinder | 05:50 | |
| *** alonma has joined #openstack-cinder | 05:53 | |
| *** kaisers_ has joined #openstack-cinder | 05:56 | |
| *** alonma has quit IRC | 05:57 | |
| *** pbandark has joined #openstack-cinder | 05:59 | |
| *** ChubYann has quit IRC | 06:00 | |
| *** ebalduf has quit IRC | 06:00 | |
| *** arecknag has joined #openstack-cinder | 06:07 | |
| *** salv-orl_ has joined #openstack-cinder | 06:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/362627 | 06:09 |
| lixiaoy1 | jgriffith: here? I just had a look at the problem you said about db initialization over and over, I found three places | 06:11 |
| *** salv-orlando has quit IRC | 06:11 | |
| *** salv-orlando has joined #openstack-cinder | 06:12 | |
| *** alonma has joined #openstack-cinder | 06:13 | |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/v3/ tests https://review.openstack.org/356841 | 06:13 |
| *** salv-orl_ has quit IRC | 06:17 | |
| *** alonma has quit IRC | 06:17 | |
| *** salv-orlando has quit IRC | 06:17 | |
| *** mjura_ has joined #openstack-cinder | 06:17 | |
| *** mjura has joined #openstack-cinder | 06:17 | |
| *** rcernin has joined #openstack-cinder | 06:19 | |
| *** rhefner has quit IRC | 06:20 | |
| *** mjura has quit IRC | 06:21 | |
| *** pgadiya has joined #openstack-cinder | 06:25 | |
| *** pcaruana has joined #openstack-cinder | 06:26 | |
| openstackgerrit | Cedric Zhuang proposed openstack/cinder: Performance issue for VNX attache LUN https://review.openstack.org/358720 | 06:27 |
| openstackgerrit | Duncan Thomas proposed openstack/cinder-specs: Support multiple nova consuming one cinder https://review.openstack.org/362637 | 06:32 |
| *** alonma has joined #openstack-cinder | 06:33 | |
| *** lpetrut has joined #openstack-cinder | 06:36 | |
| *** alonma has quit IRC | 06:37 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder: Support create group with group type name https://review.openstack.org/362643 | 06:39 |
| *** ManishD has quit IRC | 06:39 | |
| *** Manish has joined #openstack-cinder | 06:39 | |
| *** sdake has quit IRC | 06:44 | |
| *** liverpooler has joined #openstack-cinder | 06:51 | |
| *** apuimedo|away is now known as apuimedo | 07:00 | |
| *** tesseract- has joined #openstack-cinder | 07:07 | |
| openstackgerrit | xianming.mao proposed openstack/cinder: Add programming launguage python 3 in classifier. https://review.openstack.org/362530 | 07:07 |
| *** sheel has joined #openstack-cinder | 07:08 | |
| openstackgerrit | Ha Van Tu proposed openstack/cinder: Fix using filter() to meet python2,3 https://review.openstack.org/362601 | 07:08 |
| openstackgerrit | xianming.mao proposed openstack/cinder: Add programming launguage python 3 in classifier. https://review.openstack.org/362530 | 07:10 |
| *** Poornima has quit IRC | 07:11 | |
| *** ducttape_ has joined #openstack-cinder | 07:18 | |
| *** lpetrut has quit IRC | 07:21 | |
| *** rkrum has quit IRC | 07:21 | |
| *** ducttape_ has quit IRC | 07:23 | |
| *** Dinesh_Bhor has quit IRC | 07:23 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 07:24 | |
| openstackgerrit | gengchc2 proposed openstack/cinder: Change assertTrue(isinstance()) with optimal assert https://review.openstack.org/362663 | 07:27 |
| *** akapil has joined #openstack-cinder | 07:32 | |
| *** alonma has joined #openstack-cinder | 07:44 | |
| dulek | lixiaoy1: Can you share insight on that initialization issue? It's morning here, so I can start looking at it. | 07:44 |
| *** alonma has quit IRC | 07:48 | |
| *** lkuchlan has joined #openstack-cinder | 07:49 | |
| *** laughterwym has joined #openstack-cinder | 07:50 | |
| *** kaisers1 has quit IRC | 07:50 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder: Support create group with group type name https://review.openstack.org/362643 | 07:51 |
| *** kaisers has quit IRC | 07:52 | |
| *** kaisers has joined #openstack-cinder | 07:53 | |
| *** kaisers1 has joined #openstack-cinder | 07:53 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:02 | |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Remove unused function from unit/fake_utils.py https://review.openstack.org/362686 | 08:07 |
| *** kaisers_ has quit IRC | 08:07 | |
| *** kaisers_ has joined #openstack-cinder | 08:08 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** magic has joined #openstack-cinder | 08:18 | |
| *** ducttape_ has joined #openstack-cinder | 08:19 | |
| *** magic is now known as Guest99973 | 08:19 | |
| openstackgerrit | Béla Vancsics proposed openstack/cinder: Use assertIn instead of assertTrue(A in B) https://review.openstack.org/362693 | 08:20 |
| *** Guest33431 has quit IRC | 08:22 | |
| openstackgerrit | Szymon Borkowski proposed openstack/cinder: Convert backup_device to OVO https://review.openstack.org/329332 | 08:22 |
| *** ducttape_ has quit IRC | 08:23 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs: Cinder volume rollback snapshot https://review.openstack.org/316540 | 08:24 |
| openstackgerrit | TommyLike proposed openstack/cinder-specs: Cinder volume rollback snapshot https://review.openstack.org/316540 | 08:30 |
| openstackgerrit | TommyLike proposed openstack/cinder-specs: Cinder volume rollback snapshot https://review.openstack.org/316540 | 08:32 |
| openstackgerrit | TommyLike proposed openstack/cinder-specs: Cinder volume rollback snapshot https://review.openstack.org/316540 | 08:34 |
| openstackgerrit | gengchc2 proposed openstack/cinder: Change assertTrue(isinstance()) with optimal assert https://review.openstack.org/362663 | 08:37 |
| *** markd_ has joined #openstack-cinder | 08:40 | |
| *** sborkows has joined #openstack-cinder | 08:41 | |
| *** lennyb has quit IRC | 08:42 | |
| Lehtu | Hello, where can I find cinder block storage WWNs? Or how does cinder know what drives to use on the storage node? | 08:44 |
| lixiaoy1 | dulek: which intialization issue? the issue I talked with jgriffith | 08:51 |
| dulek | lixiaoy1: Exactly. | 08:51 |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/contrib/ tests (2/2) https://review.openstack.org/356859 | 08:52 |
| lixiaoy1 | dulek: jgriffith found three places of this log "MySQL server mode set to STRICT_TRANS_TABLES...", he thought it as db restarts over and over | 08:52 |
| dulek | lixiaoy1: Ah, so it was false alarm? | 08:53 |
| lixiaoy1 | dulek: this is from https://github.com/openstack/oslo.db/blob/master/oslo_db/sqlalchemy/engines.py#L248 | 08:53 |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Wrap volume_backup's update function with api_version https://review.openstack.org/362711 | 08:53 |
| lixiaoy1 | dulek: I am not sure whether it is log for db start | 08:53 |
| lixiaoy1 | dulek: only three places have such logs: /opt/stack/cinder/cinder/service.py(146)__init__() | 08:54 |
| lixiaoy1 | -> self.is_upgrading_to_n = self.is_svc_upgrading_to_n(binary) | 08:54 |
| lixiaoy1 | dulek: second place: /opt/stack/cinder/cinder/volume/manager.py(203)__init__() -> vol_db_empty = self._set_voldb_empty_at_startup_indicator( | 08:54 |
| lixiaoy1 | dulek: third place /opt/stack/cinder/cinder/volume/manager.py(423)init_host() -> volumes = objects.VolumeList.get_all_by_host(ctxt, self.host) | 08:54 |
| lixiaoy1 | dulek: so first thing is to confirm whether this log is for db first time start/connect | 08:55 |
| dulek | lixiaoy1: Oh, maybe that's just another SQLAlchemy connection getting started? | 08:55 |
| *** alonma has joined #openstack-cinder | 08:55 | |
| lixiaoy1 | dulek: then we should have lots of such logs | 08:55 |
| dulek | lixiaoy1: All of these places are in the beginning of c-vol start. | 08:55 |
| lixiaoy1 | dulek: sorry, can't talk to you , I have to take shuttle bus now. Talk to you later | 08:55 |
| dulek | lixiaoy1: Is there something like reusage of MySQL connections in SQLAlchemy? Like connection pool? | 08:56 |
| *** akapil has quit IRC | 08:58 | |
| dulek | lixiaoy1: Sure, have a nice trip. :) | 08:59 |
| *** alonma has quit IRC | 08:59 | |
| *** akapil has joined #openstack-cinder | 08:59 | |
| *** lixiaoy1 has quit IRC | 09:00 | |
| *** links has quit IRC | 09:00 | |
| openstackgerrit | Vinita Deshpande proposed openstack/cinder: CoprHD: Handle create volume from CG snapshot error https://review.openstack.org/361999 | 09:03 |
| openstackgerrit | wangxiyuan proposed openstack/cinder: Support create group with group type name https://review.openstack.org/362643 | 09:05 |
| *** lpetrut has joined #openstack-cinder | 09:06 | |
| *** gluytium has quit IRC | 09:11 | |
| *** gluytium has joined #openstack-cinder | 09:12 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO: fix the returned value of CG actions https://review.openstack.org/346831 | 09:14 |
| *** links has joined #openstack-cinder | 09:16 | |
| openstackgerrit | huanan proposed openstack/cinder: Migrate volume between backends in an async way https://review.openstack.org/334923 | 09:16 |
| *** ducttape_ has joined #openstack-cinder | 09:19 | |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/contrib/ tests (1/2) https://review.openstack.org/356856 | 09:24 |
| *** ducttape_ has quit IRC | 09:24 | |
| *** nkrinner has quit IRC | 09:29 | |
| *** lpetrut1 has joined #openstack-cinder | 09:34 | |
| *** lpetrut has quit IRC | 09:34 | |
| *** lpetrut1 is now known as lpetrut | 09:34 | |
| *** imansari has joined #openstack-cinder | 09:35 | |
| *** nkrinner has joined #openstack-cinder | 09:36 | |
| *** laughterwym has quit IRC | 09:37 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Wrap group type and group spec with api_version https://review.openstack.org/362735 | 09:38 |
| openstackgerrit | huanan proposed openstack/cinder: Migrate volume between backends in an async way https://review.openstack.org/334923 | 09:40 |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Wrap volume_backup's update function with api_version https://review.openstack.org/362711 | 09:40 |
| *** lennyb has joined #openstack-cinder | 09:43 | |
| *** e0ne has joined #openstack-cinder | 09:47 | |
| *** kaisers_ has quit IRC | 09:48 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs: Add volume type filter to Get-Pools https://review.openstack.org/362747 | 09:53 |
| openstackgerrit | TommyLike proposed openstack/cinder-specs: Add volume type filter to Get-Pools https://review.openstack.org/362747 | 09:55 |
| *** GB21 has joined #openstack-cinder | 10:03 | |
| *** alonma has joined #openstack-cinder | 10:06 | |
| *** alonma has quit IRC | 10:10 | |
| *** ducttape_ has joined #openstack-cinder | 10:20 | |
| *** yangyapeng has quit IRC | 10:21 | |
| *** ducttape_ has quit IRC | 10:25 | |
| *** alonma has joined #openstack-cinder | 10:26 | |
| *** salv-orlando has joined #openstack-cinder | 10:27 | |
| *** laughterwym has joined #openstack-cinder | 10:28 | |
| *** apuimedo is now known as apuimedo|lunch | 10:29 | |
| *** alonma has quit IRC | 10:30 | |
| *** ntpttr has quit IRC | 10:30 | |
| *** EinstCrazy has quit IRC | 10:31 | |
| *** EinstCrazy has joined #openstack-cinder | 10:31 | |
| *** ntpttr has joined #openstack-cinder | 10:35 | |
| *** EinstCrazy has quit IRC | 10:36 | |
| *** laughterwym has quit IRC | 10:38 | |
| openstackgerrit | Niko Hermanns proposed openstack/cinder: assertTrue(isinstance()) to assertIsInstance https://review.openstack.org/362770 | 10:38 |
| Lehtu | Hello, where can I find cinder block storage WWNs? Or how does cinder know what drives to use on the storage node? | 10:40 |
| *** lixiaoy1 has joined #openstack-cinder | 10:42 | |
| *** akapil has quit IRC | 10:44 | |
| lixiaoy1 | dulek: how about your investigation? | 10:44 |
| *** alonma has joined #openstack-cinder | 10:46 | |
| lixiaoy1 | dulek: https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/api.py#L76-L107 I think here is reusage of sql connection | 10:46 |
| *** abhinavtechie has joined #openstack-cinder | 10:48 | |
| *** GB21 has quit IRC | 10:49 | |
| *** GB21 has joined #openstack-cinder | 10:50 | |
| *** abhitechie has quit IRC | 10:50 | |
| *** alonma has quit IRC | 10:51 | |
| openstackgerrit | Merged openstack/cinder: Stop using mox in unit/*.py tests (1/2) https://review.openstack.org/357648 | 10:51 |
| *** salv-orl_ has joined #openstack-cinder | 10:52 | |
| *** sdague has joined #openstack-cinder | 10:53 | |
| *** timcl has joined #openstack-cinder | 10:54 | |
| *** salv-orl_ has quit IRC | 10:54 | |
| *** apuimedo|lunch is now known as apuimedo | 10:54 | |
| *** salv-orlando has quit IRC | 10:56 | |
| *** dkehn_ has quit IRC | 10:57 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: set_metadata throws AttributeError: id https://review.openstack.org/308475 | 10:57 |
| *** akapil has joined #openstack-cinder | 10:58 | |
| *** akapil_ has joined #openstack-cinder | 10:59 | |
| *** akapil has quit IRC | 10:59 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Wrap cluster related funciton with api_version https://review.openstack.org/362778 | 11:01 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: set_metadata throws AttributeError: id https://review.openstack.org/308475 | 11:01 |
| openstackgerrit | Merged openstack/cinder: Remove unused config.CONF https://review.openstack.org/361814 | 11:04 |
| *** alonma has joined #openstack-cinder | 11:06 | |
| *** dkehn_ has joined #openstack-cinder | 11:10 | |
| *** adisky has joined #openstack-cinder | 11:10 | |
| *** alonma has quit IRC | 11:11 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs: Add volume type filter to Get-Pools https://review.openstack.org/362747 | 11:12 |
| dulek | lixiaoy1: I haven't followed up honestly. | 11:16 |
| *** alonma has joined #openstack-cinder | 11:19 | |
| *** ducttape_ has joined #openstack-cinder | 11:21 | |
| lixiaoy1 | dulek: :) it is not urgent, feel free to look when you have time | 11:22 |
| *** alonma has quit IRC | 11:23 | |
| *** ducttape_ has quit IRC | 11:26 | |
| *** jordanP has joined #openstack-cinder | 11:29 | |
| *** Manish has quit IRC | 11:36 | |
| *** kaisers_ has joined #openstack-cinder | 11:37 | |
| *** laughterwym has joined #openstack-cinder | 11:39 | |
| *** timcl has quit IRC | 11:39 | |
| *** alonma has joined #openstack-cinder | 11:39 | |
| openstackgerrit | xianming.mao proposed openstack/cinder: Drop unused import cfg. https://review.openstack.org/362583 | 11:39 |
| *** kaisers_ has quit IRC | 11:41 | |
| *** alonma has quit IRC | 11:43 | |
| *** laughterwym has quit IRC | 11:44 | |
| *** tpsilva has joined #openstack-cinder | 11:48 | |
| *** milan has joined #openstack-cinder | 11:52 | |
| *** yangyapeng has joined #openstack-cinder | 11:53 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Retype encrypted volumes https://review.openstack.org/319078 | 11:58 |
| openstackgerrit | LisaLi proposed openstack/cinder: Add encryptor attach/detach in utils https://review.openstack.org/341914 | 11:58 |
| *** alonma has joined #openstack-cinder | 11:59 | |
| lixiaoy1 | geguileo: dulek: Hey, could you kindly help to review patches related to encrypted volume? | 12:02 |
| *** alonma has quit IRC | 12:04 | |
| *** sandanar has quit IRC | 12:04 | |
| *** timcl has joined #openstack-cinder | 12:05 | |
| *** ducttape_ has joined #openstack-cinder | 12:06 | |
| dulek | lixiaoy1: Sure, starting right now. | 12:12 |
| lixiaoy1 | dulek: thank you | 12:13 |
| *** pdeore has quit IRC | 12:13 | |
| *** clenimar has quit IRC | 12:15 | |
| lixiaoy1 | dulek: These are the three patches: https://review.openstack.org/#/c/216567/ , https://review.openstack.org/#/c/341914/ and https://review.openstack.org/#/c/319078/ . | 12:15 |
| openstackgerrit | xianming.mao proposed openstack/cinder-specs: Update flake8 https://review.openstack.org/359555 | 12:16 |
| *** gouthamr has joined #openstack-cinder | 12:17 | |
| *** alonma has joined #openstack-cinder | 12:19 | |
| *** tommylikehu has quit IRC | 12:22 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Create encrypted volumes from images https://review.openstack.org/216567 | 12:23 |
| *** alonma has quit IRC | 12:24 | |
| dulek | lixiaoy1: Is 319078 allowing also retyping from an encrypted type to unencrypted, and vice-versa? | 12:24 |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/v1/ tests https://review.openstack.org/356185 | 12:26 |
| lixiaoy1 | dulek: yes | 12:28 |
| *** abhinavtechie has quit IRC | 12:31 | |
| *** ducttape_ has quit IRC | 12:32 | |
| dulek | lixiaoy1: encryption_key_id is identifier that can let my find my key in Barbican? | 12:34 |
| lixiaoy1 | lixiaoy1: yes | 12:35 |
| dulek | lixiaoy1: I've added a bunch of small remarks on 319078. You should be able to get the fixed pretty quickly. :) | 12:36 |
| lixiaoy1 | dulek: thank you. Fix it now | 12:37 |
| *** alonma has joined #openstack-cinder | 12:40 | |
| *** akapil_ has quit IRC | 12:41 | |
| *** rooneym has joined #openstack-cinder | 12:41 | |
| yuriy_n17 | dulek: Hi. Sorry for disturbing. Please take a look https://review.openstack.org/#/c/308475/ whenever you have free time. | 12:41 |
| *** alonma has quit IRC | 12:44 | |
| *** EinstCrazy has joined #openstack-cinder | 12:44 | |
| openstackgerrit | Vivek Dhayaal proposed openstack/cinder: Support ZeroMQ messaging in cinder multibackend https://review.openstack.org/277113 | 12:48 |
| *** cdelatte has joined #openstack-cinder | 12:49 | |
| *** gouthamr_ has joined #openstack-cinder | 12:49 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix DellStorageCenter docstrings https://review.openstack.org/355945 | 12:51 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Add CI_WIKI_NAME to Synology iSCSI Driver https://review.openstack.org/355923 | 12:51 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Add CI_WIKI_NAME to X-IO Driver https://review.openstack.org/355924 | 12:51 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Add CI_WIKI_NAME to ZTE Driver https://review.openstack.org/355925 | 12:51 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix driver interface checks for ZTE driver https://review.openstack.org/362846 | 12:51 |
| *** gouthamr has quit IRC | 12:51 | |
| *** markvoelker has joined #openstack-cinder | 12:52 | |
| *** watanabe_isao has joined #openstack-cinder | 12:52 | |
| *** akerr has joined #openstack-cinder | 12:53 | |
| *** watanabe_isao has left #openstack-cinder | 12:53 | |
| *** yangyapeng has quit IRC | 12:57 | |
| *** asselin_ has quit IRC | 12:59 | |
| *** clenimar has joined #openstack-cinder | 13:00 | |
| *** alonma has joined #openstack-cinder | 13:00 | |
| *** laughterwym has joined #openstack-cinder | 13:01 | |
| openstackgerrit | xianming.mao proposed openstack/cinder-specs: Update flake8 https://review.openstack.org/359555 | 13:01 |
| *** sbezverk_ has quit IRC | 13:02 | |
| *** xyang1 has joined #openstack-cinder | 13:02 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots - db and objects https://review.openstack.org/328052 | 13:03 |
| *** gcb has quit IRC | 13:04 | |
| *** GB21 has quit IRC | 13:04 | |
| *** alonma has quit IRC | 13:04 | |
| *** xyang has joined #openstack-cinder | 13:05 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Retype encrypted volumes https://review.openstack.org/319078 | 13:05 |
| *** david-lyle has quit IRC | 13:10 | |
| *** pauloewerton has joined #openstack-cinder | 13:10 | |
| *** pgadiya has quit IRC | 13:10 | |
| *** adisky has quit IRC | 13:11 | |
| openstackgerrit | Merged openstack/cinder: Imported Translations from Zanata https://review.openstack.org/362627 | 13:11 |
| *** ducttape_ has joined #openstack-cinder | 13:12 | |
| xyang | dulek: hi Michal, I addressed your comments: https://review.openstack.org/#/c/328052/ Can you please take a look again? | 13:12 |
| *** rlrossit has joined #openstack-cinder | 13:13 | |
| openstackgerrit | Merged openstack/cinder: Use assertIn instead of assertTrue(A in B) https://review.openstack.org/362693 | 13:13 |
| openstackgerrit | Merged openstack/cinder: Change assertTrue(isinstance()) with optimal assert https://review.openstack.org/362663 | 13:13 |
| dulek | xyang: Sure. Do we have some kind of plan on how much of these patches we're planning to merge in Newton? | 13:14 |
| xyang | dulek: I'd like to get the 3 group snapshots patches merged | 13:14 |
| xyang | dulek: the one you reviewed is for db and objects, there's one for APIs and one for manager | 13:15 |
| dulek | xyang: Yeah, I've noticed that. BTW - how can API get in before manager changes? | 13:15 |
| dulek | xyang: Won't API be useless without proper support in manager? | 13:16 |
| *** sheel has quit IRC | 13:16 | |
| xyang | dulek: yes, you can only test it by using the manager | 13:16 |
| xyang | dulek: without api, the manager is also useless | 13:17 |
| *** markvoelker has quit IRC | 13:17 | |
| *** ducttape_ has quit IRC | 13:17 | |
| *** david-lyle has joined #openstack-cinder | 13:18 | |
| dulek | xyang: Sure, but shouldn't it be other way around? If we merge manager part without the API, then nothing bad happens, user won't have access to the feature. But if we merge API without manager we're getting non-functional API facing the user. | 13:19 |
| *** milan has left #openstack-cinder | 13:19 | |
| * dulek haven't looked deeply into the manager and API patches, so I may be missing something. | 13:19 | |
| *** alonma has joined #openstack-cinder | 13:20 | |
| xyang | dulek: I have seen other patches break down this way, first db, then api, then manager and driver | 13:21 |
| xyang | dulek: merge them both, problem solved:) | 13:23 |
| *** GB21 has joined #openstack-cinder | 13:23 | |
| *** alonma has quit IRC | 13:24 | |
| *** dustins has joined #openstack-cinder | 13:26 | |
| dulek | xyang: I'm not entirely convinced to the approach, but I'll look closer later. | 13:27 |
| xyang | dulek: ok, thanks | 13:28 |
| *** mriedem has joined #openstack-cinder | 13:30 | |
| *** EinstCrazy has quit IRC | 13:31 | |
| *** sbezverk has joined #openstack-cinder | 13:33 | |
| *** lixiaoy1 has quit IRC | 13:34 | |
| smcginnis | xyang: API should probably be last and be dependent on the others. | 13:36 |
| openstackgerrit | Chuck Fouts proposed openstack/cinder: Fix missing test coverage for manage existing flow https://review.openstack.org/362872 | 13:36 |
| xyang | @smcginnis: alright. I'll change it. thanks | 13:37 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots - manager https://review.openstack.org/361376 | 13:37 |
| *** asselin_ has joined #openstack-cinder | 13:38 | |
| *** markvoelker has joined #openstack-cinder | 13:40 | |
| *** lkuchlan has quit IRC | 13:40 | |
| *** alonma has joined #openstack-cinder | 13:40 | |
| *** ManishD has joined #openstack-cinder | 13:42 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots - manager https://review.openstack.org/361376 | 13:42 |
| *** alonma has quit IRC | 13:45 | |
| *** markvoelker has quit IRC | 13:45 | |
| *** baojg has joined #openstack-cinder | 13:46 | |
| *** e0ne has quit IRC | 13:46 | |
| *** e0ne has joined #openstack-cinder | 13:48 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots - manager https://review.openstack.org/361376 | 13:48 |
| *** Julien-zte has joined #openstack-cinder | 13:50 | |
| *** laughterwym has quit IRC | 13:50 | |
| *** chris_morrell has joined #openstack-cinder | 13:50 | |
| *** laughterwym has joined #openstack-cinder | 13:51 | |
| *** takedakn has joined #openstack-cinder | 13:52 | |
| *** ducttape_ has joined #openstack-cinder | 13:52 | |
| *** gouthamr_ is now known as gouthamr | 13:53 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots - APIs https://review.openstack.org/361369 | 13:53 |
| *** liverpooler has quit IRC | 13:53 | |
| cFouts | xyang: I added a dependent patch to https://review.openstack.org/#/c/361454 for test coverage. Would appreciate a +2 on 361454 in order to get it merged soon. Thanks! | 13:53 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix HNAS stats reporting https://review.openstack.org/344477 | 13:53 |
| *** diablo_rojo_phon has quit IRC | 13:55 | |
| *** laughterwym has quit IRC | 13:55 | |
| *** laughterwym has joined #openstack-cinder | 13:56 | |
| xyang | cFouts: done | 13:57 |
| cFouts | thanks! | 13:57 |
| *** takedakn has quit IRC | 13:57 | |
| openstackgerrit | Vivek Dhayaal proposed openstack/cinder: Support ZeroMQ messaging in cinder multibackend https://review.openstack.org/277113 | 14:00 |
| *** zzzeek has quit IRC | 14:00 | |
| *** alonma has joined #openstack-cinder | 14:00 | |
| *** sbezverk has quit IRC | 14:01 | |
| cFouts | geguileo: would really appreciate a +2 and workflow on https://review.openstack.org/#/c/361454 | 14:01 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots - db and objects https://review.openstack.org/328052 | 14:01 |
| *** rmcall_ has joined #openstack-cinder | 14:01 | |
| *** faiz89 has joined #openstack-cinder | 14:01 | |
| *** jdillaman has joined #openstack-cinder | 14:02 | |
| *** eharney has joined #openstack-cinder | 14:02 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots - manager https://review.openstack.org/361376 | 14:02 |
| *** jgregor has joined #openstack-cinder | 14:03 | |
| geguileo | cFouts: I don't see a reply from wanghao yet and you said Done to a couple of comments, but there's no new patch... | 14:03 |
| *** rmcall has quit IRC | 14:03 | |
| *** rmcall_ is now known as rmcall | 14:03 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots - APIs https://review.openstack.org/361369 | 14:03 |
| *** zzzeek has joined #openstack-cinder | 14:03 | |
| dulek | cFouts: https://review.openstack.org/#/c/361688 is the same, but provides a regression test case I think, so it's superior IMO. | 14:03 |
| *** cknight has joined #openstack-cinder | 14:04 | |
| *** zzzeek has quit IRC | 14:04 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix extend volume doesn't check capacity https://review.openstack.org/341136 | 14:04 |
| *** zzzeek has joined #openstack-cinder | 14:05 | |
| *** alonma has quit IRC | 14:05 | |
| xyang | dulek, @smcginnis: I changed the dependency order and updated commit msg. API patch is the last now | 14:05 |
| cFouts | dulek: I provided a follow on patch for test coverage that would catch the bug that was introduced | 14:05 |
| cFouts | dulek: I'm not sure how you catch that particular bug without testing get_flow directly but I'll look at wanghao's patch again | 14:06 |
| *** lixiaoy1 has joined #openstack-cinder | 14:06 | |
| dulek | cFouts: The test runs whole manage_existing method in cinder.volume.manager module, so it just checks everything. | 14:07 |
| *** sdake has joined #openstack-cinder | 14:08 | |
| *** sbezverk has joined #openstack-cinder | 14:10 | |
| *** zhangguoqing has quit IRC | 14:11 | |
| dulek | Is there some better way of visualizing job stability than something like goo.gl/g6GO7t? | 14:11 |
| *** sdake_ has joined #openstack-cinder | 14:12 | |
| *** zhangguoqing has joined #openstack-cinder | 14:12 | |
| *** sdake has quit IRC | 14:13 | |
| *** ebalduf has joined #openstack-cinder | 14:14 | |
| erlon | xyang: Hi Xyang, I addressed the comments in the HNAS stats reporting patch: https://review.openstack.org/344477 | 14:15 |
| *** zhangguoqing has quit IRC | 14:15 | |
| xyang | erlon: sure, I'll take a look | 14:16 |
| *** baojg has quit IRC | 14:16 | |
| lixiaoy1 | smcginnis: I updated the patch to retype encrypted volume based on your comments https://review.openstack.org/#/c/319078/ , please have a look. | 14:18 |
| *** baojg has joined #openstack-cinder | 14:18 | |
| *** diablo_rojo has joined #openstack-cinder | 14:19 | |
| *** diablo_rojo has quit IRC | 14:20 | |
| *** diablo_rojo has joined #openstack-cinder | 14:20 | |
| *** alonma has joined #openstack-cinder | 14:21 | |
| *** diablo_rojo_ has joined #openstack-cinder | 14:21 | |
| *** diablo_rojo_ has quit IRC | 14:21 | |
| *** jungleboyj has joined #openstack-cinder | 14:21 | |
| *** Yogi1 has joined #openstack-cinder | 14:22 | |
| *** rlrossit has quit IRC | 14:23 | |
| openstackgerrit | Chuck Fouts proposed openstack/cinder: Fix manage existing MissingDependencies error https://review.openstack.org/361454 | 14:23 |
| *** xyang has quit IRC | 14:23 | |
| openstackgerrit | Merged openstack/python-cinderclient: Make APIVersion's null check more pythonic https://review.openstack.org/351767 | 14:24 |
| *** alonma has quit IRC | 14:25 | |
| *** yangyapeng has joined #openstack-cinder | 14:25 | |
| *** raunak has joined #openstack-cinder | 14:26 | |
| cFouts | geguileo and dulek: https://review.openstack.org/#/c/361688 can you +2 and workflow this patch. This bug is breaking our internal CI and it doesn't really matter to me which patch gets merged. | 14:26 |
| *** rlrossit has joined #openstack-cinder | 14:26 | |
| *** mriedem has quit IRC | 14:27 | |
| openstackgerrit | Merged openstack/cinder: Add encryptor attach/detach in utils https://review.openstack.org/341914 | 14:27 |
| *** mriedem has joined #openstack-cinder | 14:28 | |
| *** baojg has quit IRC | 14:28 | |
| *** xyang has joined #openstack-cinder | 14:29 | |
| *** pbandark has quit IRC | 14:31 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX driver - Cleanup of Initiator group fails https://review.openstack.org/347838 | 14:32 |
| *** e0ne has quit IRC | 14:32 | |
| *** bardia has joined #openstack-cinder | 14:32 | |
| *** zeestrat has quit IRC | 14:33 | |
| *** ociuhandu has joined #openstack-cinder | 14:33 | |
| *** e0ne has joined #openstack-cinder | 14:34 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: ScaleIO over subscription support https://review.openstack.org/356547 | 14:36 |
| *** crose has joined #openstack-cinder | 14:36 | |
| *** nkrinner has quit IRC | 14:36 | |
| *** kaisers_ has joined #openstack-cinder | 14:37 | |
| openstackgerrit | Merged openstack/cinder: Create encrypted volumes from images https://review.openstack.org/216567 | 14:38 |
| ameade | dulek, cFouts: why not both patches? I rather unittests check at the smallest unit possible and I also like the minor cleanup in cfouts patch | 14:39 |
| *** nkrinner has joined #openstack-cinder | 14:39 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs: Add volume type filter to Get-Pools https://review.openstack.org/362747 | 14:40 |
| *** alonma has joined #openstack-cinder | 14:41 | |
| *** zhangguoqing has joined #openstack-cinder | 14:41 | |
| *** arecknag has quit IRC | 14:42 | |
| xyang | ameade: how can you merge both patches? Both are changing volume_ref to volume. do you just refer to the unit test part | 14:42 |
| *** david-lyle has quit IRC | 14:43 | |
| geguileo | They should synchronize since both are fixing the same bug (bugs in LP were not marked as duplicated) | 14:43 |
| ameade | yeah the unittest patch changes | 14:44 |
| ameade | just merge one and the other person adjusts? | 14:44 |
| *** zhangguoqing has quit IRC | 14:45 | |
| *** diogogmt has joined #openstack-cinder | 14:45 | |
| geguileo | This is not the first time that happens, and the way we usually do it is that both patch owners talk between them and decide how they want to proceed | 14:45 |
| *** raunak has quit IRC | 14:45 | |
| *** alonma has quit IRC | 14:46 | |
| ameade | mhm, i rather be in rebase hell than collaboration hell though. but that may just be me | 14:46 |
| *** lprice has joined #openstack-cinder | 14:46 | |
| *** nkrinner has quit IRC | 14:46 | |
| *** sdake_ has quit IRC | 14:47 | |
| *** lixiaoy1 has quit IRC | 14:48 | |
| cFouts | yeah, I'm willing to abandon my first patch and use the follow on patch to get my unit test in after the fix gets merged. :) | 14:48 |
| *** david-lyle has joined #openstack-cinder | 14:48 | |
| *** lixiaoy1 has joined #openstack-cinder | 14:48 | |
| *** enriquetaso has joined #openstack-cinder | 14:49 | |
| *** lixiaoy1 has quit IRC | 14:52 | |
| *** ManishD has quit IRC | 14:54 | |
| *** ManishD has joined #openstack-cinder | 14:54 | |
| DuncanT | cFouts: https://review.openstack.org/#/c/361688/ - I've just put a -1 on it. Maybe you can enlighten me? | 14:57 |
| *** rajinir has joined #openstack-cinder | 14:58 | |
| cFouts | DuncanT: here is the launchpad bug I opened: https://bugs.launchpad.net/cinder/+bug/1617442 | 14:59 |
| openstack | Launchpad bug 1617442 in Cinder "Manage existing volume fails with MissingDependencies" [Critical,In progress] - Assigned to Chuck Fouts (fchuck) | 14:59 |
| *** nkrinner has joined #openstack-cinder | 14:59 | |
| *** alonma has joined #openstack-cinder | 15:01 | |
| cFouts | DuncanT: basically the create_with dict in get_flow defines the parameters that are passed to tasks in a linear task flow. Each task can also add additional parameters in a list called default_provides. If a task (execute method) expects to receive a parameter named 'volume' but doesn't it will throw a MissingDependency error | 15:02 |
| *** david-lyle has quit IRC | 15:02 | |
| *** zhangguoqing has joined #openstack-cinder | 15:02 | |
| *** yangyapeng has quit IRC | 15:04 | |
| *** alonma has quit IRC | 15:05 | |
| *** diogogmt has quit IRC | 15:06 | |
| *** coolsvap is now known as coolsvap_ | 15:07 | |
| *** mjura_ has quit IRC | 15:07 | |
| *** lprice1 has joined #openstack-cinder | 15:08 | |
| *** diogogmt has joined #openstack-cinder | 15:08 | |
| smcginnis | scottda, eharney: Any further thoughts/discoveries on the OOM deal? | 15:09 |
| *** lprice has quit IRC | 15:09 | |
| eharney | i haven't spent time on it this week really | 15:09 |
| scottda | smcginnis: No, but I haven't ran any more tests this week. | 15:09 |
| *** sdake has joined #openstack-cinder | 15:09 | |
| eharney | i assumed he was doing it all ^ :P | 15:09 |
| smcginnis | Seems to just be that our tests have slowly been consuming more and more memory over time and we are just now starting to hit the max of wwhat we get in infra. | 15:09 |
| scottda | I assumed he was doing it all * | 15:10 |
| *** akerr has quit IRC | 15:10 | |
| smcginnis | :) | 15:10 |
| scottda | smcginnis: Once I get through some code reviews this morning I'll run some more tests... | 15:10 |
| smcginnis | Still seems insane to me that we would even need more than 1G to run unit tests. | 15:11 |
| *** jgregor has quit IRC | 15:11 | |
| *** akerr has joined #openstack-cinder | 15:11 | |
| eharney | it is, but, we have a lot of tests that need tuning up | 15:11 |
| *** nkrinner is now known as nkrinner_afk | 15:11 | |
| *** zzzeek has quit IRC | 15:19 | |
| *** kaisers_ has quit IRC | 15:19 | |
| *** alonma has joined #openstack-cinder | 15:21 | |
| *** david-lyle has joined #openstack-cinder | 15:22 | |
| *** sandanar has joined #openstack-cinder | 15:23 | |
| *** dkehn_ has quit IRC | 15:26 | |
| *** alonma has quit IRC | 15:26 | |
| *** links has quit IRC | 15:28 | |
| *** kaisers_ has joined #openstack-cinder | 15:29 | |
| *** sdake has quit IRC | 15:31 | |
| *** bardia has quit IRC | 15:34 | |
| *** zzzeek has joined #openstack-cinder | 15:40 | |
| *** salv-orlando has joined #openstack-cinder | 15:40 | |
| *** david-lyle has quit IRC | 15:40 | |
| *** ManishD has quit IRC | 15:41 | |
| *** zzzeek has quit IRC | 15:41 | |
| *** alonma has joined #openstack-cinder | 15:42 | |
| hemna | smcginnis, https://blueprints.launchpad.net/cinder/+spec/zonemanager-supported-driver | 15:42 |
| hemna | I'm working on this one now | 15:42 |
| hemna | I have the zone manager driver side done. I have to work on the lookup service drivers now. | 15:42 |
| *** faiz89 has quit IRC | 15:42 | |
| hemna | then we can put up a cisco drivers patch to disable them as a follow up if we want. | 15:43 |
| *** zzzeek has joined #openstack-cinder | 15:43 | |
| smcginnis | hemna: Cool, thanks. | 15:46 |
| *** alonma has quit IRC | 15:46 | |
| *** dkehn_ has joined #openstack-cinder | 15:46 | |
| *** zhangguoqing has quit IRC | 15:46 | |
| *** lpetrut has quit IRC | 15:48 | |
| *** david-lyle has joined #openstack-cinder | 15:50 | |
| *** timcl1 has joined #openstack-cinder | 15:51 | |
| *** timcl has quit IRC | 15:54 | |
| *** ntpttr__ has quit IRC | 15:54 | |
| *** ducttape_ has quit IRC | 15:55 | |
| *** ducttape_ has joined #openstack-cinder | 15:56 | |
| smcginnis | eharney: I suppose we don't have some fancy deprecation mechanism for drivers like we do for config options, huh? | 15:57 |
| eharney | smcginnis: not that i know of | 15:58 |
| smcginnis | eharney: Infrequent enough that I couldn't remember. ;) | 15:58 |
| *** sborkows has quit IRC | 15:58 | |
| eharney | where's the new patchset irc bot today? | 15:58 |
| karthikp | dulek: Please could you review this in your free time ... https://review.openstack.org/#/c/319040/ | 15:58 |
| DuncanT | smcginnis: Isn't that what the new unsupported_driver stuff is all about? | 16:00 |
| *** leeantho has joined #openstack-cinder | 16:02 | |
| *** alonma has joined #openstack-cinder | 16:02 | |
| eharney | breaking the upgrade path is not great, so i don't think that option really helps here in the "we are notifying you in advance" case | 16:02 |
| *** mriedem1 has joined #openstack-cinder | 16:04 | |
| hemna | well, there is breaking upgrade and there is breaking upgrade | 16:05 |
| hemna | at least this is nothing more that a small tweak to cinder.conf | 16:05 |
| *** mriedem has quit IRC | 16:06 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR use OSLO versioned Object for volume https://review.openstack.org/339147 | 16:06 |
| hemna | 3par CI is still broken | 16:06 |
| *** alonma has quit IRC | 16:06 | |
| *** mriedem1 is now known as mriedem | 16:07 | |
| *** laughterwym has quit IRC | 16:08 | |
| eharney | speaking of deprecating things, this patch has been hanging here for a while: https://review.openstack.org/#/c/339738/ | 16:08 |
| *** laughterwym has joined #openstack-cinder | 16:09 | |
| *** krotscheck has joined #openstack-cinder | 16:10 | |
| krotscheck | Hey, I'm doing some work against the cinder API right now, and I'm looking for a little historical context. | 16:10 |
| krotscheck | When I retrieve the service catalog from devstack's keystone, I get a URL that looks like this: http://192.168.99.99:8776/v1/8b2aa635109f4d0ab355e18a269d341f | 16:11 |
| krotscheck | (for cinder) | 16:11 |
| hemna | eharney, so I did a little hunting on shred vs dd and I didn't find any consensus why one was better over another | 16:11 |
| krotscheck | However http://192.168.99.99:8776 return http://192.168.99.99:8776/v3 | 16:12 |
| krotscheck | (Adjust ersion numbers as appropriate. | 16:12 |
| eharney | hemna: for our cases, shred is not providing anything useful | 16:12 |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX Plugin - Target WWN change for redundancy https://review.openstack.org/342300 | 16:12 |
| krotscheck | What's the significance of the additional suffix found in the devstack catalog? | 16:12 |
| hemna | eharney, I was under the assumption that safety against data leakage was the reason to use dd/shread after we were done w/ the volime | 16:13 |
| hemna | volume | 16:13 |
| hemna | nova code does the same thing | 16:13 |
| eharney | hemna: if people want more safety than dd provides, they should just use volume encryption, we don't know that the cost of running shred does anything significant anyway since it greatly depends on what's going on in the storage backend | 16:14 |
| eharney | hemna: now that everyone has fancy dedup-ey backends you can't really shred over your data anyway | 16:14 |
| *** raunak has joined #openstack-cinder | 16:15 | |
| hemna | I thought this was really only used on lvm volumes ? | 16:15 |
| eharney | hemna: yes, and block_device driver... but those LVM volumes could be on anything (which is why we don't know) | 16:15 |
| *** ociuhandu has quit IRC | 16:15 | |
| *** laughterwym has quit IRC | 16:16 | |
| scottda | krotscheck: There are 3 services for the 3 major APi versions: cinder, cinderv2, and cinder v3 | 16:16 |
| krotscheck | scottda: Yep, I can see that. | 16:16 |
| hemna | cool, I was just curious. I don't really care either way | 16:17 |
| krotscheck | scottda: Really, what I'm trying to do is create find the root URL for all services from the catalog, so I can query the list of versions and negotiate the one that I know how to talk. | 16:18 |
| krotscheck | scottda: And do so in a robust way so I can reuse it for nova, keystone, etc. | 16:18 |
| *** lprice1 has quit IRC | 16:19 | |
| scottda | krotscheck: If you hit the root endpoint (URL:port) it will show the versions, including DEPRECATED /v1, SUPPORTED V2, and CURRENT V3 with microversion | 16:19 |
| scottda | https://www.irccloud.com/pastebin/6MxfBn3u/ | 16:19 |
| *** lpetrut has joined #openstack-cinder | 16:20 | |
| *** xyang has quit IRC | 16:20 | |
| scottda | krotscheck: But that is not necessarily the same for other services. | 16:20 |
| krotscheck | scottda: So what is the significance of the long sha-like string at the end? | 16:20 |
| scottda | krotscheck: But probably should be the same... | 16:20 |
| scottda | That's my token | 16:20 |
| krotscheck | scottda: That's unique to cinder best I can tell. | 16:20 |
| scottda | I stole it from Manila, so maybe the same for them... | 16:20 |
| krotscheck | scottda: Gotcha. So I can happily ignore it if I provide the token via the x-auth-header? | 16:21 |
| scottda | But we need something for services with microversions, so the user can discover what (micro)versions the sever supports. | 16:21 |
| scottda | krotscheck: Yes, you can ignore the token part in the curl. | 16:21 |
| *** david-lyle has quit IRC | 16:21 | |
| krotscheck | scottda: Sweet. | 16:21 |
| krotscheck | scottda: Yeah, so, in this case I'm actually "the user", building an SDK against the cinder api | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: [WIP] Add service cleanup command https://review.openstack.org/363007 | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add cleanable base object and cleanup request VO https://review.openstack.org/303020 | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Make c-vol use workers table for cleanup https://review.openstack.org/303021 | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Add remaining operations to cluster https://review.openstack.org/355968 | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Support A/A on Scheduler operations https://review.openstack.org/344226 | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [PoC][Don't review] Testing delay https://review.openstack.org/353069 | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Support A/A in attach/detach operations https://review.openstack.org/318573 | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Support A/A in delete actions and get_capabilities https://review.openstack.org/344225 | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Cosmetic changes to scheduler https://review.openstack.org/346041 | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix lazy loading cluster field from Service OVO https://review.openstack.org/363008 | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove host assignment in Snapshot OVO https://review.openstack.org/363009 | 16:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Allow triggering cleanup from API https://review.openstack.org/363010 | 16:22 |
| *** alonma has joined #openstack-cinder | 16:22 | |
| hemna | boom! | 16:22 |
| krotscheck | scottda: Thanks for the explanation. Best I can figure, the only service where I can't reduce the service to the root URL now is keystone. | 16:22 |
| hemna | geguileo, shames us all again.... | 16:22 |
| * krotscheck wanders off to the keystone channel. | 16:23 | |
| scottda | krotscheck: Cool. Probably a good thing for the API working group to come up with a standardization on... | 16:23 |
| geguileo | lol | 16:23 |
| *** david-lyle has joined #openstack-cinder | 16:23 | |
| scottda | ameade: Are you still attending API-wg meetings? | 16:23 |
| geguileo | Many of those are WIP because they lack unit tests... | 16:23 |
| *** jordanP has quit IRC | 16:23 | |
| hemna | geguileo, good to know I'm not the only one that does that :) | 16:23 |
| krotscheck | scottda: They did :) https://specs.openstack.org/openstack/api-wg/guidelines/microversion_specification.html | 16:24 |
| geguileo | hemna: I usually don't, but these patches are already hell on earth when I have to rebase them, no need for more work when they are not going to get merged until O ;-) | 16:25 |
| scottda | krotscheck: Oh yeah, and it looks like Cinder is compliant. | 16:25 |
| krotscheck | scottda: Yep, now I'm just trying to figure out why the service as registered in devstack doesn't provide the root URL. | 16:26 |
| *** alonma has quit IRC | 16:27 | |
| *** kaisers_ has quit IRC | 16:28 | |
| ameade | scottda: no not really, I need someone to take my place as liaison. <- smcginnis | 16:30 |
| smcginnis | ameade: :[ | 16:30 |
| ameade | feel like I've been doing a terrible job with it lately anyhow | 16:30 |
| *** e0ne has quit IRC | 16:31 | |
| scottda | smcginnis: I'll put it on my calendar to start attending api-wg meetings | 16:32 |
| smcginnis | scottda: Awesome, thanks! | 16:32 |
| scottda | man, all these meetings on my calendar. I might as well become a manager... | 16:33 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Error attaching after LV-AFO https://review.openstack.org/361453 | 16:34 |
| smcginnis | scottda: Hehe | 16:34 |
| hemna | scottda, ssh! be careful what you wish for....... | 16:34 |
| scottda | That was not a wish. I trust the discretion of the people around here... | 16:35 |
| *** gothicmindfood has joined #openstack-cinder | 16:35 | |
| *** david-lyle_ has joined #openstack-cinder | 16:36 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix exceed of dd action when copying with path https://review.openstack.org/354270 | 16:36 |
| *** xyang has joined #openstack-cinder | 16:37 | |
| *** jgregor has joined #openstack-cinder | 16:37 | |
| *** ChubYann has joined #openstack-cinder | 16:38 | |
| *** timcl1 has quit IRC | 16:39 | |
| *** david-lyle has quit IRC | 16:39 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: DB: Optimize update methods https://review.openstack.org/332944 | 16:40 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: DB: Optimize volume_update method https://review.openstack.org/332943 | 16:40 |
| ameade | scottda: ty sir! | 16:41 |
| *** alonma has joined #openstack-cinder | 16:42 | |
| openstackgerrit | xing-yang proposed openstack/python-cinderclient: Add generic volume groups https://review.openstack.org/322627 | 16:44 |
| *** alonma has quit IRC | 16:47 | |
| openstackgerrit | xing-yang proposed openstack/python-cinderclient: Add support for group snapshots https://review.openstack.org/329770 | 16:47 |
| *** liverpooler has joined #openstack-cinder | 16:50 | |
| *** xyang has quit IRC | 16:52 | |
| *** rlrossit has quit IRC | 16:52 | |
| *** diogogmt has quit IRC | 16:52 | |
| *** bardia has joined #openstack-cinder | 16:53 | |
| hemna | eharney, so the shread patch | 16:53 |
| hemna | isn't deprecation supposed to leave the capability in and then remove it later | 16:54 |
| hemna | the patch removes shred entirely. | 16:54 |
| hemna | and switches it to dd. | 16:54 |
| hemna | that doesn't seem like a deprecation, but a removal ? | 16:54 |
| hemna | do we care? | 16:55 |
| hemna | smcginnis, | 16:55 |
| smcginnis | hemna: I haven't looked. Is the functionality the same, just a different implementation? | 16:55 |
| hemna | this seems like a grey area to me | 16:55 |
| hemna | the patch removes shred | 16:56 |
| smcginnis | hemna: If it takes away something then definitely it should stay and be removed next release. | 16:56 |
| hemna | but logs saying it's deprecated and will be removed | 16:56 |
| hemna | and then switches the functionality to dd | 16:56 |
| hemna | unless I'm missing something | 16:56 |
| smcginnis | If the user can still request something but we just happen to do it differently under the covers, then that's different. | 16:56 |
| hemna | smcginnis, https://review.openstack.org/#/c/339738/1/cinder/volume/utils.py | 16:56 |
| smcginnis | hemna: Thanks, looking... | 16:56 |
| eharney | hemna: it's not a removal of functionality if the original functionality didn't really do what it advertised anyway | 16:56 |
| eharney | which is my argument there | 16:57 |
| eharney | but you could argue that either way really | 16:57 |
| *** erlon has quit IRC | 16:57 | |
| hemna | yah, grey area | 16:57 |
| eharney | i dunno, doesn't matter too much how we do the intermediate step to me, just wanted to push it in the right direction | 16:57 |
| eharney | i also think that nobody has ever set this option to =shred | 16:58 |
| hemna | I'm not sure how they configure cinder to use shred vs. dd anyway | 16:59 |
| hemna | must be some conf entry ? | 16:59 |
| eharney | yeah, volume_clear | 16:59 |
| smcginnis | So I think since the end result still gives the user what they want and it doesn't break anything, it's OK to just switch this. | 16:59 |
| smcginnis | And deprecate the option just in the sense that that config option will go away in O. | 17:00 |
| hemna | ok sounds good. | 17:00 |
| hemna | I just wanted to make sure others saw this too | 17:00 |
| *** sdague has quit IRC | 17:00 | |
| *** sdague has joined #openstack-cinder | 17:01 | |
| smcginnis | hemna: Yeah, good to check for sure. | 17:02 |
| *** alonma has joined #openstack-cinder | 17:03 | |
| *** Apoorva has joined #openstack-cinder | 17:03 | |
| *** faiz89 has joined #openstack-cinder | 17:05 | |
| *** xyang has joined #openstack-cinder | 17:07 | |
| *** alonma has quit IRC | 17:07 | |
| *** diogogmt has joined #openstack-cinder | 17:07 | |
| *** rcernin has quit IRC | 17:12 | |
| *** arch-nemesis has joined #openstack-cinder | 17:13 | |
| *** arch-nemesis has quit IRC | 17:13 | |
| *** tesseract- has quit IRC | 17:14 | |
| *** Suyi_ has joined #openstack-cinder | 17:14 | |
| openstackgerrit | Merged openstack/cinder: Clean import in code https://review.openstack.org/358480 | 17:18 |
| *** sandanar_ has joined #openstack-cinder | 17:21 | |
| *** alonma has joined #openstack-cinder | 17:23 | |
| *** tesseract- has joined #openstack-cinder | 17:24 | |
| *** sandanar has quit IRC | 17:24 | |
| *** pcaruana has quit IRC | 17:25 | |
| *** sdake has joined #openstack-cinder | 17:27 | |
| *** tesseract- has quit IRC | 17:28 | |
| *** alonma has quit IRC | 17:28 | |
| *** bardia has quit IRC | 17:28 | |
| *** xinli has joined #openstack-cinder | 17:34 | |
| *** alonma has joined #openstack-cinder | 17:43 | |
| *** lpetrut has quit IRC | 17:46 | |
| *** alonma has quit IRC | 17:48 | |
| openstackgerrit | Merged openstack/os-brick: Fix cmd execution stderr, stdout unicode errors https://review.openstack.org/341085 | 17:48 |
| *** rlrossit has joined #openstack-cinder | 17:48 | |
| *** gouthamr has quit IRC | 17:54 | |
| *** markvoelker has joined #openstack-cinder | 17:54 | |
| *** gouthamr has joined #openstack-cinder | 17:54 | |
| *** xinli has quit IRC | 17:55 | |
| *** xinli has joined #openstack-cinder | 17:56 | |
| *** e0ne has joined #openstack-cinder | 18:01 | |
| *** rmcall has quit IRC | 18:03 | |
| *** rmcall has joined #openstack-cinder | 18:03 | |
| *** sandanar_ has quit IRC | 18:03 | |
| *** alonma has joined #openstack-cinder | 18:03 | |
| *** akerr is now known as akerr_away | 18:04 | |
| *** akerr_away is now known as akerr | 18:04 | |
| *** tesseract- has joined #openstack-cinder | 18:05 | |
| *** tesseract- has quit IRC | 18:05 | |
| *** tesseract- has joined #openstack-cinder | 18:06 | |
| *** jgregor has quit IRC | 18:07 | |
| ntpttr | n | 18:07 |
| *** salv-orl_ has joined #openstack-cinder | 18:08 | |
| *** tesseract- has quit IRC | 18:08 | |
| *** alonma has quit IRC | 18:08 | |
| *** Apoorva has quit IRC | 18:10 | |
| *** Apoorva has joined #openstack-cinder | 18:10 | |
| *** tesseract- has joined #openstack-cinder | 18:10 | |
| *** salv-orlando has quit IRC | 18:11 | |
| *** diablo_rojo has quit IRC | 18:12 | |
| *** jungleboyj has quit IRC | 18:12 | |
| *** lkuchlan has joined #openstack-cinder | 18:15 | |
| *** ducttape_ has quit IRC | 18:18 | |
| *** xyang has quit IRC | 18:20 | |
| *** markvoelker has quit IRC | 18:22 | |
| enriquetaso | Hi geguileo, I wonder if you could help me with the 'manage volume' bug. ( /354289/ ) | 18:23 |
| enriquetaso | You are right, manage an already-managed volume is handle in the same way as manage a non existent volume. | 18:23 |
| enriquetaso | Both of them don't match the volume ID with image ID, but I think it shouldn't be the same case. :/ | 18:23 |
| *** alonma has joined #openstack-cinder | 18:24 | |
| geguileo | enriquetaso: Mmmm, but the managing patch doesn't remane the image? | 18:24 |
| geguileo | s/patch/path | 18:24 |
| geguileo | enriquetaso: I mean, when we manage a volume, don't we rename the existing image to match the new ID? | 18:25 |
| geguileo | enriquetaso: And why would an already managed volume get an image not found error from Ceph? | 18:25 |
| geguileo | If we give the right image name? | 18:26 |
| enriquetaso | so so, create a volume and check with rbd: http://pastebin.com/RcFtaR7c ... the rbd id starts with "volume-".. manage only works if "volume-" is present in the syntax | 18:26 |
| geguileo | enriquetaso: I know | 18:27 |
| geguileo | enriquetaso: Then you manage that volume, and we rename it, right? | 18:27 |
| geguileo | enriquetaso: Or Ceph doesn't implement the renaming when managing? | 18:28 |
| geguileo | enriquetaso: Didn't you imlement that functionality? | 18:28 |
| *** alonma has quit IRC | 18:28 | |
| geguileo | enriquetaso: If you prefer we can get out of the channel and speak in Spanish ;-) | 18:28 |
| geguileo | it may be easier for us both | 18:29 |
| enriquetaso | geguileo: yes, please | 18:29 |
| geguileo | ok | 18:29 |
| *** tesseract- has quit IRC | 18:30 | |
| *** tongli has joined #openstack-cinder | 18:30 | |
| *** tesseract- has joined #openstack-cinder | 18:30 | |
| *** tesseract- has quit IRC | 18:30 | |
| *** tesseract- has joined #openstack-cinder | 18:31 | |
| *** tesseract- has quit IRC | 18:37 | |
| *** faiz89 has quit IRC | 18:38 | |
| *** campee_ has quit IRC | 18:39 | |
| *** mtanino has joined #openstack-cinder | 18:41 | |
| *** ducttape_ has joined #openstack-cinder | 18:41 | |
| *** Apoorva has quit IRC | 18:43 | |
| *** GB21 has quit IRC | 18:43 | |
| *** alonma has joined #openstack-cinder | 18:44 | |
| *** lpetrut has joined #openstack-cinder | 18:45 | |
| *** xyang has joined #openstack-cinder | 18:45 | |
| *** Apoorva has joined #openstack-cinder | 18:45 | |
| *** delattec has joined #openstack-cinder | 18:48 | |
| *** alonma has quit IRC | 18:49 | |
| *** salv-orl_ has quit IRC | 18:49 | |
| *** Suyi_1 has joined #openstack-cinder | 18:50 | |
| *** dstepanenko_ has joined #openstack-cinder | 18:51 | |
| *** cdelatte has quit IRC | 18:51 | |
| *** dstepanenko has quit IRC | 18:51 | |
| *** etienneme has quit IRC | 18:51 | |
| *** Suyi_ has quit IRC | 18:52 | |
| *** lennyb has quit IRC | 18:52 | |
| *** wN has quit IRC | 18:52 | |
| *** wxy__ has quit IRC | 18:52 | |
| *** sayalilunkad has quit IRC | 18:52 | |
| *** Yogi1 has quit IRC | 18:56 | |
| *** wN has joined #openstack-cinder | 18:57 | |
| *** wN has quit IRC | 18:57 | |
| *** wN has joined #openstack-cinder | 18:57 | |
| *** salv-orlando has joined #openstack-cinder | 18:59 | |
| *** erlon has joined #openstack-cinder | 18:59 | |
| *** lkuchlan has quit IRC | 19:02 | |
| patrickeast | oo neat, the rally job outputs are pretty fancy http://52.8.17.232/PureISCSIDriver-rally-dsvm/11/ | 19:02 |
| patrickeast | i guess thats os-loganalyze doing magic | 19:03 |
| eharney | the html report requires javascript to read -_- | 19:03 |
| patrickeast | haha, like any good html document :P | 19:04 |
| *** sayalilunkad has joined #openstack-cinder | 19:04 | |
| *** lennyb has joined #openstack-cinder | 19:04 | |
| *** alonma has joined #openstack-cinder | 19:04 | |
| hemna | what's javascript? | 19:05 |
| *** david-lyle_ is now known as david-lyle | 19:05 | |
| *** raunak has quit IRC | 19:05 | |
| *** etienneme has joined #openstack-cinder | 19:05 | |
| patrickeast | its like python with more curly braces | 19:05 |
| *** alonma has quit IRC | 19:09 | |
| *** wxy__ has joined #openstack-cinder | 19:09 | |
| *** lpetrut has quit IRC | 19:10 | |
| *** coolsvap_ has quit IRC | 19:12 | |
| *** sbezverk has quit IRC | 19:16 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: NFS snapshots https://review.openstack.org/147186 | 19:20 |
| *** faiz89 has joined #openstack-cinder | 19:21 | |
| *** Lee1092 has quit IRC | 19:24 | |
| *** alonma has joined #openstack-cinder | 19:25 | |
| openstackgerrit | Merged openstack/cinder: Fix typo in enable_unsupported_driver description https://review.openstack.org/363018 | 19:28 |
| *** alonma has quit IRC | 19:29 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix extend volume doesn't check capacity https://review.openstack.org/341136 | 19:29 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix HNAS stats reporting https://review.openstack.org/344477 | 19:30 |
| *** e0ne has quit IRC | 19:36 | |
| *** lkuchlan has joined #openstack-cinder | 19:39 | |
| *** rcernin has joined #openstack-cinder | 19:39 | |
| *** jgregor has joined #openstack-cinder | 19:41 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots - APIs https://review.openstack.org/361369 | 19:48 |
| *** akshai has joined #openstack-cinder | 19:53 | |
| *** alonma has joined #openstack-cinder | 19:55 | |
| melwitt | hi, I was wondering what's the difference between terminate connection and detach? I'm working on a bug where a volume is stuck in-use under certain circumstances and I noticed it doesn't seem to matter whether terminate connection is called, only detach | 19:55 |
| mriedem | jgriffith: ^ | 19:55 |
| *** jungleboyj has joined #openstack-cinder | 19:56 | |
| *** crose has quit IRC | 19:56 | |
| mriedem | the api ref is a bit lacking on those actions http://developer.openstack.org/api-ref-blockstorage-v2.html | 19:56 |
| melwitt | that is, if I attach a volume to a nova instance and that instance goes in error state, if I call detach and not terminate, it frees the volume for a later attach. and I was wondering if I should be calling terminate also | 19:56 |
| *** harlowja has quit IRC | 19:58 | |
| *** Yogi1_ has joined #openstack-cinder | 19:58 | |
| *** diablo_rojo has joined #openstack-cinder | 19:58 | |
| patrickeast | melwitt: these might help, hemna made some nice flow charts https://goo.gl/ufSOJq | 19:58 |
| patrickeast | melwitt: you should call terminate too iirc | 19:58 |
| patrickeast | some other ones on https://etherpad.openstack.org/p/cinder-nova-api-changes | 19:59 |
| melwitt | patrickeast: cool, thank you. yeah, I figured I'm supposed to but was surprised it didn't seem to make a difference | 19:59 |
| *** alonma has quit IRC | 19:59 | |
| melwitt | this is just in devstack with the standard stuff | 19:59 |
| mriedem | so os-initialize_connection creates the export | 19:59 |
| mriedem | os-terminate_connection removes the export | 19:59 |
| patrickeast | might depend on the backend as to how required it is | 20:00 |
| *** Apoorva has quit IRC | 20:00 | |
| scottda | melwitt: Terminate_connection is what ends up calling the Cinder driver to un-export the volume... | 20:00 |
| melwitt | patrickeast: yeah, mriedem suggested that | 20:00 |
| mriedem | i'm not sure if os-detach does much, on the nova side we disconnect the volume from the host after that | 20:00 |
| scottda | melwitt: So, if you don't call it, the volume will still be exported and possibly attached to the Nova compute host. | 20:00 |
| mriedem | i think os-attach and os-detach are just state change APIs in cinder | 20:01 |
| mriedem | same with os-reserve | 20:01 |
| patrickeast | that sounds right | 20:01 |
| scottda | mriedem: yes, os-detach changes the Cinder DB state to 'available' | 20:01 |
| melwitt | okay, so it was just the illusion of devstack then. thanks patrickeast, scottda, mriedem, that's helpful to know | 20:03 |
| scottda | mriedem: you are right, docs don't have info for os-detach. I filed https://bugs.launchpad.net/cinder/+bug/1618609 | 20:08 |
| openstack | Launchpad bug 1618609 in Cinder "Cinder api documents lack volume_detach info" [Undecided,New] - Assigned to Scott DAngelo (scott-dangelo) | 20:08 |
| *** sdake has quit IRC | 20:11 | |
| *** sdake has joined #openstack-cinder | 20:12 | |
| *** raunak has joined #openstack-cinder | 20:14 | |
| *** alonma has joined #openstack-cinder | 20:15 | |
| *** sbezverk has joined #openstack-cinder | 20:16 | |
| *** akerr has quit IRC | 20:16 | |
| *** ducttape_ has quit IRC | 20:19 | |
| *** alonma has quit IRC | 20:19 | |
| hemna | mriedem, correct, those just tweak the db | 20:21 |
| *** ducttape_ has joined #openstack-cinder | 20:22 | |
| *** zeestrat has joined #openstack-cinder | 20:23 | |
| *** sbezverk_ has joined #openstack-cinder | 20:29 | |
| *** sbezverk has quit IRC | 20:29 | |
| *** rcernin has quit IRC | 20:33 | |
| *** rcernin has joined #openstack-cinder | 20:33 | |
| *** Apoorva has joined #openstack-cinder | 20:34 | |
| *** alonma has joined #openstack-cinder | 20:45 | |
| *** jungleboyj has quit IRC | 20:46 | |
| openstackgerrit | xing-yang proposed openstack/cinder: WIP: Add functional tests for groups https://review.openstack.org/362584 | 20:48 |
| *** alonma has quit IRC | 20:50 | |
| *** faiz89 has quit IRC | 20:52 | |
| *** cknight has quit IRC | 20:53 | |
| *** lpetrut has joined #openstack-cinder | 20:54 | |
| *** jgregor has quit IRC | 20:55 | |
| *** e0ne has joined #openstack-cinder | 20:58 | |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: Add os-detach to api-ref https://review.openstack.org/363203 | 21:00 |
| *** Yogi1_ has quit IRC | 21:02 | |
| openstackgerrit | Drew Balfour proposed openstack/cinder: zfssaiscsi driver initiator group option processing https://review.openstack.org/351413 | 21:02 |
| *** e0ne has quit IRC | 21:03 | |
| *** _alastor1 is now known as _alastor- | 21:03 | |
| *** dustins has quit IRC | 21:04 | |
| *** _alastor- is now known as _alastor__ | 21:04 | |
| *** e0ne has joined #openstack-cinder | 21:06 | |
| scottda | I don't think 'reverify' works anymore for +A's patches. Do we just +A again to get them back into the merge queue? | 21:06 |
| patrickeast | you mean if they failed jenkins? | 21:06 |
| hemna | did they change that recently ? | 21:06 |
| *** jamielennox is now known as jamielennox|away | 21:06 | |
| hemna | I've been doing reverify | 21:06 |
| patrickeast | i think 'recheck' should work unless they changed it | 21:06 |
| scottda | patrickeast: They failed Jenkins check | 21:07 |
| scottda | hemna: Yeah, I thought that re-verify didn't work like it used to. But I could be wrong...I guess I'll test it and see. | 21:07 |
| eharney | there is need for "reverify" at this point, "recheck" has handled that case for a while now | 21:08 |
| eharney | is no need* | 21:08 |
| *** lkuchlan has quit IRC | 21:08 | |
| *** xyang has quit IRC | 21:09 | |
| *** pauloewerton has quit IRC | 21:09 | |
| *** faiz89 has joined #openstack-cinder | 21:10 | |
| *** faiz89 is now known as faiz89_away | 21:10 | |
| *** faiz89_away is now known as faiz89 | 21:10 | |
| *** enriquetaso has quit IRC | 21:11 | |
| *** xyang has joined #openstack-cinder | 21:12 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots - APIs https://review.openstack.org/361369 | 21:13 |
| *** raunak has quit IRC | 21:14 | |
| *** harlowja has joined #openstack-cinder | 21:17 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix HNAS stats reporting https://review.openstack.org/344477 | 21:18 |
| *** raunak has joined #openstack-cinder | 21:19 | |
| diablo_rojo | Swanson why do you hate us? Why won't you come hang out with smcginnis and me ? | 21:19 |
| *** gouthamr has quit IRC | 21:19 | |
| Swanson | Was that you running around in the background while he was on a conference call? | 21:20 |
| *** faiz89 has quit IRC | 21:20 | |
| diablo_rojo | Swanson, No I was sitting on a couch in the basement. | 21:20 |
| smcginnis | eharney: Isn't there a need for reverify? recheck sends it back through the check queue, but if it's already passed check and you just need to kick it through gate again, reverify was always good for that. | 21:22 |
| eharney | smcginnis: it will still go through the gate queue w/ recheck | 21:22 |
| smcginnis | eharney: Yeah, but also the check queue. | 21:23 |
| *** e0ne has quit IRC | 21:23 | |
| hemna | smcginnis, ping | 21:23 |
| smcginnis | hemna: Sup | 21:23 |
| hemna | so... | 21:23 |
| hemna | I'm in a bit of a rathole on the fczm shite | 21:23 |
| *** rhefner has joined #openstack-cinder | 21:23 | |
| smcginnis | Oh? | 21:23 |
| hemna | the fczm stuff is a mess :( | 21:23 |
| smcginnis | :[ | 21:24 |
| hemna | the lookup service | 21:24 |
| hemna | has a set of drivers | 21:24 |
| smcginnis | rm -fr *fibre* | 21:24 |
| hemna | but they aren't drivers, they are subclasses | 21:24 |
| hemna | I'm trying to add the supported stuffs to em | 21:24 |
| smcginnis | Where is that? | 21:24 |
| hemna | I might break this stuff up into 2 patches | 21:24 |
| hemna | one for the fczm | 21:24 |
| hemna | and another for the lookup service stuff | 21:24 |
| hemna | it's all a pain | 21:25 |
| hemna | I don't know why it was written this way | 21:25 |
| hemna | but the fczm has drivers | 21:25 |
| hemna | but the lookup service doesn't | 21:25 |
| hemna | brocade and cisco just extended the base class | 21:25 |
| hemna | anyway long story, I'm getting infinite imports | 21:25 |
| smcginnis | hemna: So... they each have their own lookup service? | 21:25 |
| hemna | bleh | 21:25 |
| hemna | yah | 21:25 |
| smcginnis | hemna: Fun! | 21:25 |
| hemna | I might refactor the lookup service stuff | 21:26 |
| hemna | I started, and then....ratholed | 21:26 |
| smcginnis | hemna: Do we need it both places? Or just in the driver load? | 21:26 |
| hemna | more and more changes | 21:26 |
| hemna | we do need it in both places really | 21:26 |
| smcginnis | hemna: Probably wouldn't hurt to clean that up though. | 21:26 |
| hemna | they are used entirely separately | 21:26 |
| *** sdake has quit IRC | 21:26 | |
| hemna | and they should both have the supported crap | 21:26 |
| *** dustins has joined #openstack-cinder | 21:26 | |
| *** salv-orlando has quit IRC | 21:26 | |
| hemna | so I might just punt on the lookup service | 21:26 |
| hemna | and push up what I have | 21:26 |
| hemna | and then do a follow up | 21:27 |
| smcginnis | OK. Well, if it ends up being too big maybe we shoot for early O. | 21:27 |
| hemna | at least this way, we can get the fczm stuff in | 21:27 |
| smcginnis | Yeah, that'd be good. | 21:27 |
| hemna | as soon as I started on the refactor, I'm deep into new interfaces, and new interfaces need new things n stuff | 21:27 |
| hemna | ratehole ensues | 21:28 |
| smcginnis | Thanks for detangling all that. | 21:28 |
| hemna | sure | 21:28 |
| hemna | I'll just get the fczm done | 21:28 |
| smcginnis | Sounds good for now. | 21:29 |
| hemna | ok cool | 21:29 |
| hemna | it's a start at least | 21:29 |
| eharney | smcginnis: https://review.openstack.org/#/c/111098/ | 21:30 |
| scottda | eharney: Thanks. Nice to see something definitive. | 21:31 |
| *** sdake has joined #openstack-cinder | 21:32 | |
| smcginnis | eharney: Huh. OK, I guess that at least answers that question. :) | 21:32 |
| smcginnis | Wait, that was 2014. I know at least as of this spring, reverify would just retrigger gate queue checks if needed. | 21:33 |
| smcginnis | Oh well. | 21:33 |
| openstackgerrit | Drew Balfour proposed openstack/cinder: zfssaiscsi driver log messges need cleaned up https://review.openstack.org/363219 | 21:40 |
| *** eharney has quit IRC | 21:48 | |
| *** chrichip has joined #openstack-cinder | 21:49 | |
| *** salv-orlando has joined #openstack-cinder | 21:49 | |
| *** dustins has quit IRC | 21:50 | |
| *** alonma has joined #openstack-cinder | 21:56 | |
| *** cknight has joined #openstack-cinder | 21:58 | |
| *** cknight1 has joined #openstack-cinder | 22:00 | |
| *** alonma has quit IRC | 22:00 | |
| *** diablo_rojo has quit IRC | 22:01 | |
| *** mriedem has quit IRC | 22:01 | |
| *** cknight has quit IRC | 22:02 | |
| *** alyson_ has quit IRC | 22:03 | |
| *** rajinir has quit IRC | 22:05 | |
| *** chrichip has quit IRC | 22:06 | |
| scottda | OK, we've had this conversation before: We don't backport to the cinderclient, right? and the stable branches are just something used by infra, and don't really have meaning outside that. (users would grab a specific cinderclient version from pip instead). | 22:07 |
| *** rlrossit has quit IRC | 22:09 | |
| *** xyang has quit IRC | 22:10 | |
| hemna | I think so | 22:12 |
| scottda | hemna: Thanks. I believe I'm correct on this patch: https://review.openstack.org/#/c/299458/1 | 22:12 |
| *** lpetrut has quit IRC | 22:12 | |
| hemna | I believe so yah | 22:13 |
| *** alonma has joined #openstack-cinder | 22:17 | |
| openstackgerrit | Merged openstack/cinder: Add tempest tests for Consistency Groups https://review.openstack.org/355271 | 22:18 |
| *** diogogmt has quit IRC | 22:20 | |
| *** alonma has quit IRC | 22:21 | |
| *** cknight1 has quit IRC | 22:24 | |
| *** alonma has joined #openstack-cinder | 22:27 | |
| *** xinli has quit IRC | 22:28 | |
| *** alonma has quit IRC | 22:31 | |
| *** diogogmt has joined #openstack-cinder | 22:32 | |
| *** ducttape_ has quit IRC | 22:34 | |
| *** sdake has quit IRC | 22:36 | |
| *** hemna is now known as hemnafk | 22:39 | |
| ntpttr | hey all, I just tried to do a db migration from stable/mitaka to master, but after I pulled down the master code (in devstack) cinder-manage won't work any more | 22:42 |
| ntpttr | getting this error for any cinder-manage command http://paste.openstack.org/show/564908/ | 22:42 |
| ntpttr | is this a bug in master? hemnafk do you know why this import error might be happening in brick? | 22:43 |
| patrickeast | that doesn't look good | 22:45 |
| ntpttr | hmm it does work on my one node devstack setup w/ cinder at master | 22:46 |
| patrickeast | ntpttr: looks like https://github.com/openstack/cinder/commit/8107b8529c186865c1ffc2b6eccbec1335e295d2 just merged recently... which is suspiciously named with that import error | 22:46 |
| ntpttr | patrickeast: ah very suspicious there... | 22:47 |
| *** alonma has joined #openstack-cinder | 22:47 | |
| patrickeast | if i had to guess maybe you need an updated os-brick? | 22:47 |
| patrickeast | iirc we cut a release somewhat recently too | 22:47 |
| ntpttr | patrickeast: looks like a pip install -U os-brick fixed the issue | 22:48 |
| ntpttr | patrickeast: thanks! | 22:48 |
| patrickeast | ntpttr: np | 22:48 |
| *** alonma has quit IRC | 22:52 | |
| *** erhudy has quit IRC | 22:52 | |
| *** ducttape_ has joined #openstack-cinder | 23:03 | |
| *** ducttape_ has quit IRC | 23:09 | |
| *** Apoorva has quit IRC | 23:09 | |
| *** rkrum has joined #openstack-cinder | 23:12 | |
| *** Apoorva has joined #openstack-cinder | 23:14 | |
| openstackgerrit | Merged openstack/cinder: Add group snapshots - db and objects https://review.openstack.org/328052 | 23:14 |
| openstackgerrit | Merged openstack/cinder: Use constraints for api-ref environment https://review.openstack.org/359362 | 23:14 |
| *** salv-orlando has quit IRC | 23:21 | |
| *** sdague has quit IRC | 23:31 | |
| *** rcernin has quit IRC | 23:35 | |
| openstackgerrit | xing-yang proposed openstack/cinder: WIP: Add functional tests for groups https://review.openstack.org/362584 | 23:41 |
| *** tpsilva has quit IRC | 23:44 | |
| *** Julien-zte has quit IRC | 23:44 | |
| *** xyang has joined #openstack-cinder | 23:47 | |
| *** alonma has joined #openstack-cinder | 23:48 | |
| *** rooneym has quit IRC | 23:49 | |
| *** alonma has quit IRC | 23:52 | |
| *** mriedem has joined #openstack-cinder | 23:57 | |
| *** akerr has joined #openstack-cinder | 23:58 | |
| *** xyang has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!