Tuesday, 2017-11-28

*** salv-orlando has quit IRC00:12
*** salv-orlando has joined #openstack-cinder00:13
*** AlexeyAbashkin has joined #openstack-cinder00:17
*** salv-orlando has quit IRC00:17
*** AlexeyAbashkin has quit IRC00:21
*** stakeda has joined #openstack-cinder00:33
*** jmlowe has quit IRC00:39
openstackgerritTommyLike proposed openstack/cinder master: Fix 'KeyError' when 'with_count' is not specified  https://review.openstack.org/52291700:43
*** jmlowe has joined #openstack-cinder00:46
smcginnistommylikehu: I don't think it's worth that change if you are going to keep the if statement the same.00:48
* jungleboyj pauses my review00:48
smcginnistommylikehu: I should have just approved the last one. Think I missed that Jay had commented on there already.00:48
*** dklyle has quit IRC00:54
tommylikehusmcginnis:  oh, It's useful if we delete the key in the method 'get_bool_param' someday at least:)00:56
smcginnistommylikehu: We shouldn't ever delete a key inside a method like that.00:57
smcginnistommylikehu: Either the if statement should be updated, or it should go back to how you had it.00:57
smcginnistommylikehu: I would have just approved it before but I didn't notice jungleboyj had given a +2.00:57
tommylikehusmcginnis:  I will keep the if statement00:58
smcginnistommylikehu: Works for me. :)00:58
jungleboyjOk, so there is another patch set coming?00:59
openstackgerritTommyLike proposed openstack/cinder master: Fix 'KeyError' when 'with_count' is not specified  https://review.openstack.org/52291701:00
jungleboyjYes.01:00
*** david-lyle has joined #openstack-cinder01:01
tommylikehujungleboyj: :)01:02
jungleboyjWeird, when you went back to the patchset being like the one I +2'd is automatically put my +2 back.  That is crazy.01:02
tommylikehujungleboyj:  it's confusing me also01:03
jungleboyjYeah, I can't say I like that.01:03
tommylikehujungleboyj:  it's smater than we expected01:03
smcginnisGerrit must be smart enough to know it was OK before.01:03
smcginnis¯\_(ツ)_/¯01:04
jungleboyjGuess so.01:05
*** s-shiono has joined #openstack-cinder01:06
*** liverpooler has joined #openstack-cinder01:11
*** salv-orlando has joined #openstack-cinder01:14
*** yangyapeng has joined #openstack-cinder01:15
*** yangyapeng has quit IRC01:16
*** yangyapeng has joined #openstack-cinder01:16
*** salv-orlando has quit IRC01:19
*** jose-phillips has quit IRC01:21
*** jose-phi_ has joined #openstack-cinder01:21
openstackgerritTommyLike proposed openstack/cinder master: Support create volume from backup  https://review.openstack.org/51294901:22
openstackgerritTommyLike proposed openstack/python-cinderclient master: Support create volume from backup in client  https://review.openstack.org/51617501:26
*** armax has quit IRC01:27
*** dave-mccowan has joined #openstack-cinder01:36
*** s-shiono_ has joined #openstack-cinder01:39
*** s-shiono has quit IRC01:43
*** sdague has quit IRC01:47
*** mriedem has joined #openstack-cinder01:48
openstackgerritMasayuki Igawa proposed openstack/cinder master: Fix indentation in docs  https://review.openstack.org/52302701:50
openstackgerritwanghongxu proposed openstack/os-brick master: set vg_thin_pool_size to float type  https://review.openstack.org/52327302:00
*** cheneydc has joined #openstack-cinder02:00
*** cheneydc has quit IRC02:07
*** salv-orlando has joined #openstack-cinder02:15
*** AlexeyAbashkin has joined #openstack-cinder02:16
*** zhurong has joined #openstack-cinder02:18
*** lhx_ has joined #openstack-cinder02:18
*** dalgaaf has quit IRC02:18
*** dalgaaf has joined #openstack-cinder02:19
*** salv-orlando has quit IRC02:21
*** AlexeyAbashkin has quit IRC02:21
*** Apoorva_ has joined #openstack-cinder02:21
*** Apoorva has quit IRC02:25
*** Apoorva_ has quit IRC02:25
*** gcb has joined #openstack-cinder02:27
*** erlon has quit IRC02:41
openstackgerritMerged openstack/cinder master: Fix 'KeyError' when 'with_count' is not specified  https://review.openstack.org/52291702:42
openstackgerritGuoqiang Ding proposed openstack/cinder master: Update the documentation links  https://review.openstack.org/52328202:50
*** markstur_ has quit IRC02:55
*** dhinesh has quit IRC03:01
*** dave-mccowan has quit IRC03:12
*** dave-mccowan has joined #openstack-cinder03:12
*** dave-mcc_ has joined #openstack-cinder03:16
*** salv-orlando has joined #openstack-cinder03:17
*** dave-mccowan has quit IRC03:18
*** salv-orlando has quit IRC03:23
*** abhishekk has joined #openstack-cinder03:26
*** armax has joined #openstack-cinder03:34
*** links has joined #openstack-cinder03:39
*** dhinesh has joined #openstack-cinder03:44
*** dhinesh_ has joined #openstack-cinder03:46
*** dhinesh has quit IRC03:48
*** gkadam has quit IRC04:07
*** dhinesh_ has quit IRC04:11
*** AlexeyAbashkin has joined #openstack-cinder04:16
*** salv-orlando has joined #openstack-cinder04:19
*** AlexeyAbashkin has quit IRC04:21
*** salv-orlando has quit IRC04:25
*** zhurong has quit IRC04:30
*** jiaopeng_ has joined #openstack-cinder04:30
*** gouthamr has quit IRC04:31
*** jiaop____ has joined #openstack-cinder04:31
*** jiaopeng_ has quit IRC04:35
*** liverpooler has quit IRC04:36
*** psachin has joined #openstack-cinder04:37
*** dave-mcc_ has quit IRC04:42
*** pgadiya has joined #openstack-cinder04:51
*** pgadiya has quit IRC04:52
*** threestrands has quit IRC05:10
*** threestrands has joined #openstack-cinder05:10
*** threestrands has joined #openstack-cinder05:10
*** threestrands has quit IRC05:12
*** threestrands has joined #openstack-cinder05:12
*** threestrands has joined #openstack-cinder05:12
*** pgadiya has joined #openstack-cinder05:13
*** pgadiya has quit IRC05:13
*** markstur has joined #openstack-cinder05:17
*** markstur has quit IRC05:17
*** markstur has joined #openstack-cinder05:17
*** dhinesh has joined #openstack-cinder05:20
*** salv-orlando has joined #openstack-cinder05:20
*** salv-orlando has quit IRC05:27
*** sticker has quit IRC05:49
*** jiaop____ has quit IRC05:55
*** armax has quit IRC05:58
*** armax has joined #openstack-cinder06:01
*** armax has quit IRC06:02
*** lkuchlan has joined #openstack-cinder06:03
*** pcaruana has joined #openstack-cinder06:06
*** pcaruana has quit IRC06:06
*** zhurong has joined #openstack-cinder06:08
*** jiaopen__ has joined #openstack-cinder06:15
*** jiaopen__ has quit IRC06:15
*** lpetrut has joined #openstack-cinder06:22
*** salv-orlando has joined #openstack-cinder06:22
*** chenying has joined #openstack-cinder06:26
*** salv-orlando has quit IRC06:28
openstackgerritRikimaru Honjo proposed openstack/os-brick master: Recover node.startup values after discovering  https://review.openstack.org/46614606:33
*** jiaopeng_ has joined #openstack-cinder06:33
*** jiaopeng_ has quit IRC06:34
openstackgerritRikimaru Honjo proposed openstack/os-brick master: Recover node.startup values after discovering  https://review.openstack.org/46614606:35
*** armaan has joined #openstack-cinder06:36
*** armaan_ has joined #openstack-cinder06:40
*** bkopilov has joined #openstack-cinder06:42
*** armaan has quit IRC06:43
*** markstur has quit IRC06:49
*** salv-orlando has joined #openstack-cinder06:49
*** salv-orlando has quit IRC06:50
*** salv-orlando has joined #openstack-cinder06:50
*** e0ne has joined #openstack-cinder06:52
openstackgerritRikimaru Honjo proposed openstack/os-brick master: Recover node.startup values after discovering  https://review.openstack.org/46614606:57
*** mriedem has quit IRC07:00
openstackgerritChris Yang proposed openstack/cinder master: Add support for enhanced features to the QNAP Cinder driver  https://review.openstack.org/47018807:00
*** threestrands has quit IRC07:04
*** zhurong has quit IRC07:05
*** namnh has joined #openstack-cinder07:06
*** josecastroleon has joined #openstack-cinder07:08
*** lkuchlan has quit IRC07:10
*** spectr has joined #openstack-cinder07:18
*** spectr has quit IRC07:21
*** moshele has joined #openstack-cinder07:23
gmannjungleboyj: this is for you (api-ref)-  https://review.openstack.org/#/c/521747/07:23
*** danpawlik has quit IRC07:29
*** dhinesh has quit IRC07:40
*** AlexeyAbashkin has joined #openstack-cinder07:52
*** rcernin has quit IRC07:53
*** lkuchlan has joined #openstack-cinder07:59
*** e0ne has quit IRC08:01
*** gkadam has joined #openstack-cinder08:13
*** lpetrut has quit IRC08:18
*** amoralej|off is now known as amoralej08:21
*** danpawlik has joined #openstack-cinder08:22
*** danpawlik has quit IRC08:23
*** danpawlik has joined #openstack-cinder08:24
*** lpetrut has joined #openstack-cinder08:25
*** danpawlik has quit IRC08:26
*** pcaruana has joined #openstack-cinder08:28
*** danpawlik has joined #openstack-cinder08:29
openstackgerritTommyLike proposed openstack/cinder master: Update 'force' parameter for volume delete API  https://review.openstack.org/52334408:31
*** alexchadin has joined #openstack-cinder08:31
*** wewe0901 has joined #openstack-cinder08:45
*** AlexeyAbashkin has quit IRC08:48
*** AlexeyAbashkin has joined #openstack-cinder08:51
*** gmann is now known as gmann_afk08:53
openstackgerritGhanshyam Mann proposed openstack/cinder master: Cinder Tempest plugin tests: Switch to stable interface  https://review.openstack.org/52335408:57
*** stvnoyes has quit IRC09:02
*** stvnoyes has joined #openstack-cinder09:02
nsinghi am facing some issues while implementing service token in cinder for cinder to glance interaction. please find the detail http://paste.openstack.org/show/627543/09:12
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Fix error handling and checks for generic volume groups  https://review.openstack.org/52108009:19
*** sbezverk has quit IRC09:22
*** zhurong has joined #openstack-cinder09:22
*** pgadiya has joined #openstack-cinder09:24
*** moshele has quit IRC09:24
*** moshele has joined #openstack-cinder09:24
*** pgadiya has quit IRC09:24
*** moshele has quit IRC09:25
*** moshele_ has joined #openstack-cinder09:26
*** moshele_ has quit IRC09:26
*** moshele has joined #openstack-cinder09:26
*** salv-orlando has quit IRC09:33
*** salv-orlando has joined #openstack-cinder09:33
*** e0ne has joined #openstack-cinder09:36
*** salv-orlando has quit IRC09:38
*** ociuhandu has joined #openstack-cinder09:43
*** lpetrut has quit IRC09:44
*** ociuhandu has quit IRC09:46
*** openstackgerrit has quit IRC09:48
*** erlon has joined #openstack-cinder09:55
*** rcernin has joined #openstack-cinder09:55
*** lkuchlan has quit IRC09:59
*** openstackgerrit has joined #openstack-cinder10:00
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX Doc - clarification and typo fixes  https://review.openstack.org/51895610:00
*** lkuchlan has joined #openstack-cinder10:02
*** sdatko has quit IRC10:06
*** namnh has quit IRC10:07
*** ganso has joined #openstack-cinder10:10
*** zhurong has quit IRC10:19
*** sdague has joined #openstack-cinder10:22
*** lkuchlan_ has joined #openstack-cinder10:22
*** lkuchlan_ has quit IRC10:22
openstackgerritShunei Shiono proposed openstack/cinder master: NEC driver: delete an unused configuration parameter.  https://review.openstack.org/52338210:26
*** alexchadin has quit IRC10:28
*** gcb has quit IRC10:28
*** s-shiono_ has quit IRC10:29
*** lpetrut has joined #openstack-cinder10:35
*** laurent\ has quit IRC10:42
*** alexchadin has joined #openstack-cinder10:51
*** laurent\ has joined #openstack-cinder10:53
*** abhishekk has quit IRC10:57
*** salv-orlando has joined #openstack-cinder11:01
*** salv-orlando has quit IRC11:07
*** salv-orlando has joined #openstack-cinder11:08
*** gnufied has quit IRC11:10
*** salv-orlando has quit IRC11:13
*** salv-orlando has joined #openstack-cinder11:14
*** wewe0901 has quit IRC11:25
*** gnufied has joined #openstack-cinder11:46
*** e0ne has quit IRC12:01
*** bkopilov has quit IRC12:01
*** psachin has quit IRC12:01
*** sapd_ has quit IRC12:03
*** sapd__ has joined #openstack-cinder12:03
*** gnufied has quit IRC12:04
*** e0ne has joined #openstack-cinder12:05
*** armaan_ has quit IRC12:08
*** armaan has joined #openstack-cinder12:09
*** msno has joined #openstack-cinder12:18
*** takedakn has joined #openstack-cinder12:18
msnoi updated my cinder client package from 1.0 to 2.1.012:19
msnoi am getting this error now12:19
msno# cinder list12:19
msnoERROR: publicURL endpoint for volumev3 service in denver region not found12:19
msnoi dont have a endpoint for volumev312:19
msnoany pointers for this error12:20
*** takedakn has quit IRC12:23
*** schmots has joined #openstack-cinder12:33
*** schmots has quit IRC12:38
*** schmots has joined #openstack-cinder12:39
msnook.. i created a volumev3 service and added endoints .. both private and public12:42
*** lpetrut has quit IRC12:42
msnonow i am geting this error12:42
msnohttp://paste.openstack.org/show/627568/12:42
tommylikehuhey smcginnis , jungleboyj  I updated the patch according to your comments: https://review.openstack.org/#/c/512949/12:42
msnoany pointers please12:42
*** armaan has quit IRC12:46
*** armaan has joined #openstack-cinder12:46
*** lhx_ has quit IRC12:50
*** gnufied has joined #openstack-cinder12:51
*** abishop has joined #openstack-cinder12:55
*** yangyapeng has quit IRC12:56
*** amoralej is now known as amoralej|lunch13:02
*** gnufied has quit IRC13:05
*** dave-mccowan has joined #openstack-cinder13:05
*** liverpooler has joined #openstack-cinder13:06
*** links has quit IRC13:16
*** lhx_ has joined #openstack-cinder13:20
smcginnismsno: That response doesn't look right - "http://10.18.10.25:8776/v1/v3/volumes/detail"13:21
smcginnismsno: What is the server-side version?13:22
*** edmondsw has joined #openstack-cinder13:22
nsinghi am facing some issues while implementing service token in cinder for cinder to glance interaction. please find the detail http://paste.openstack.org/show/627543/.. Any help???13:24
*** bkopilov has joined #openstack-cinder13:28
*** gnufied has joined #openstack-cinder13:31
*** yangyapeng has joined #openstack-cinder13:32
*** mriedem has joined #openstack-cinder13:32
msnosmcginnis, openstack 1.0.513:33
msnoserver side version is still this release based13:33
smcginnismsno: I don't know what 1.0.5 is. Cinder Pike was 11.0.0. Ocata was 10.0.0, etc.13:35
msnosmcginnis, let me check ..13:35
*** yangyapeng has quit IRC13:37
msnosmcginnis, http://paste.openstack.org/show/627581/13:40
msnothis the output from another contrller where the original version of cinde client is there13:40
ildikovsmcginnis: coffee is awesome, thank you!!! :)13:42
smcginnisildikov: Glad you like it! :)13:42
*** wN has quit IRC13:43
openstackgerritGhanshyam Mann proposed openstack/cinder master: Cinder Tempest plugin tests: Switch to stable interface  https://review.openstack.org/52335413:48
msnosmcginnis,13:53
msnocinder-api_2014.1.3-7+hLinux_all.deb13:53
msnocinder-backup_2014.1.3-7+hLinux_all.deb13:53
msnocinder-common_2014.1.3-7+hLinux_all.deb13:53
msnocinder-scheduler_2014.1.3-7+hLinux_all.deb13:53
msnocinder-volume_2014.1.3-7+hLinux_all.deb13:53
msnothis is the server version13:53
msnosorry .. ignore it13:53
msnoii  hoscg-cinder                         2015.1.2.153-1                 all          Openstack cinder13:53
openstackgerritOmar Muhtaseb proposed openstack/cinder master: Disallow managed volume to be managed again on manage existing volume:   - On manage existing volume, check if the volume is already managed   - Display an error "Unable to manage existing volume. The volume is already     managed" if the volume is alread  https://review.openstack.org/52341713:55
smcginnismsno: Oh wow, that's Kilo I think. I don't know if we had manage/unmanage in that release.13:56
msnosmcginnis, even at cinder server side .. right?13:56
msnosmcginnis, one doubt .. what if .. i upgrade only cinder [both cli and server ] and try to manage/ unmanage existing volumes13:57
*** salv-orlando has quit IRC13:57
smcginnismsno: Not sure if it's safe to only do the one service. I think nova officially only works with one version back.13:58
msnosmcginnis, ok.. let me check another approach then ..13:58
msnothank you for all your pointers..!13:58
smcginnismsno: No problem.13:58
*** mmethot has joined #openstack-cinder14:00
*** amoralej|lunch is now known as amoralej14:08
*** amoralej is now known as amoralej|off14:10
*** jmlowe has quit IRC14:10
openstackgerritOmar Muhtaseb proposed openstack/cinder master: Disallow managed volume to be managed again:      - On manage existing volume, check if the volume is already managed.   -   If the volume is already managed, display an error     "Unable to manage existing volume. The volume is already managed"  https://review.openstack.org/52341714:12
*** abhishekk has joined #openstack-cinder14:13
*** pcaruana has quit IRC14:13
*** pcaruana has joined #openstack-cinder14:17
*** armaan has quit IRC14:22
openstackgerritOmar Muhtaseb proposed openstack/cinder master: Disallow managed volume to be managed again On manage existing volume, check if the volume is already managed. If the volume is already managed, display an error "Unable to manage existing volume. The volume is already managed"  https://review.openstack.org/52341714:23
openstackgerritOmar Muhtaseb proposed openstack/cinder master: Disallow managed volume to be managed again  https://review.openstack.org/52341714:25
chrisyang_0660eharney, smcginnis :14:26
chrisyang_0660https://review.openstack.org/#/c/470188/14:26
chrisyang_0660I have already re-added the QNAP volume driver first. Now the patch is ready for the review.14:26
chrisyang_0660Ii will be great if you could schedule and review it.14:26
chrisyang_0660Thanks! :)14:26
smcginnischrisyang_0660: Will do, on my review list.14:27
chrisyang_0660smcginnis: thanks, have a nice day :)14:28
*** alexchadin has quit IRC14:32
storshooHi guys, im trying to recover from compute host failure using: http://docs.openstack.org/nova/pike/admin/node-down.html14:33
storshooI performed changes to noba db and i managed to power on server... now it hangs on dev [sda]14:33
*** eharney has quit IRC14:34
storshoosda abort...14:34
storshoodoes anyone have any idea? (using ocata vanilla and ceph luminous backend)14:35
*** armax has joined #openstack-cinder14:40
*** gnufied has quit IRC14:41
*** lpetrut has joined #openstack-cinder14:44
*** jdillaman has joined #openstack-cinder14:45
*** salv-orlando has joined #openstack-cinder14:51
*** gnufied has joined #openstack-cinder14:53
*** salv-orl_ has joined #openstack-cinder14:56
*** salv-orlando has quit IRC14:59
*** abhishekk has quit IRC15:01
lkwanFor review, https://review.openstack.org/#/c/511567/, any more comment for the next step?15:15
*** moshele has quit IRC15:16
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Implement SRDF Metro feature  https://review.openstack.org/51728215:20
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - retypes failing when workload is missing  https://review.openstack.org/51760215:26
*** jmlowe has joined #openstack-cinder15:28
*** jmlowe has quit IRC15:30
*** armaan has joined #openstack-cinder15:30
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - support for manage/unmanage snapshot  https://review.openstack.org/51873515:31
*** crose has joined #openstack-cinder15:33
*** sapd__ has quit IRC15:34
*** sapd_ has joined #openstack-cinder15:34
*** crose has quit IRC15:37
*** crose has joined #openstack-cinder15:38
*** charlesr has joined #openstack-cinder15:38
*** charlesr has quit IRC15:39
*** crose has quit IRC15:40
*** crose has joined #openstack-cinder15:40
*** dklyle has joined #openstack-cinder15:43
*** ntpttr_ has joined #openstack-cinder15:44
*** lkuchlan has quit IRC15:44
*** crose has quit IRC15:44
*** gouthamr has joined #openstack-cinder15:45
*** crose has joined #openstack-cinder15:45
*** ntpttr has quit IRC15:45
openstackgerritMerged openstack/cinder-specs master: Provisioning Improvements  https://review.openstack.org/49011615:46
*** david-lyle has quit IRC15:46
*** nicolasbock has joined #openstack-cinder15:49
*** markstur has joined #openstack-cinder15:58
*** masuberu has joined #openstack-cinder16:01
*** masber has quit IRC16:05
*** lpetrut has quit IRC16:06
*** lpetrut has joined #openstack-cinder16:06
*** sbezverk has joined #openstack-cinder16:07
*** masuberu has quit IRC16:08
jungleboyjjgriffith:  Ping16:08
*** dhinesh has joined #openstack-cinder16:08
*** masuberu has joined #openstack-cinder16:08
ildikovjungleboyj: ha! I just wanted to ping jgriffith as well16:10
ildikovjungleboyj: so I think he'll now hide for the whole week :)16:10
jungleboyjildikov:  Poor guy is going to hide now.16:10
jungleboyjildikov jinx16:11
*** imacdonn has joined #openstack-cinder16:11
ildikovjungleboyj: :)16:11
*** dhinesh has quit IRC16:13
*** crose has quit IRC16:14
*** eharney has joined #openstack-cinder16:14
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Incorrect stats reporting  https://review.openstack.org/51938316:15
*** josecastroleon has quit IRC16:15
*** shaner has quit IRC16:16
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - revert a volume to last snapshot  https://review.openstack.org/52065916:19
*** lpetrut has quit IRC16:28
*** salv-orlando has joined #openstack-cinder16:28
*** salv-orl_ has quit IRC16:31
*** shaner has joined #openstack-cinder16:31
*** gkadam_ has joined #openstack-cinder16:31
*** gkadam has quit IRC16:34
*** jmlowe has joined #openstack-cinder16:38
*** salv-orlando has quit IRC16:42
*** salv-orlando has joined #openstack-cinder16:42
*** lpetrut has joined #openstack-cinder16:43
*** salv-orlando has quit IRC16:46
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Fix error handling and checks for generic volume groups  https://review.openstack.org/52108016:48
*** AlexeyAbashkin has quit IRC16:49
*** armaan has quit IRC17:00
*** armaan has joined #openstack-cinder17:00
openstackgerritMerged openstack/python-cinderclient master: Migrate to Zuul v3  https://review.openstack.org/52172317:03
jungleboyjildikov:  He really is avoiding us.17:03
*** moshele has joined #openstack-cinder17:08
*** vivsoni_ has joined #openstack-cinder17:09
vivsoni_Hi Team, may i request you to please review https://review.openstack.org/#/c/522947/17:10
*** moshele has quit IRC17:12
*** jmlowe has quit IRC17:15
*** lhx_ has quit IRC17:16
*** gkadam_ has quit IRC17:16
openstackgerritGoutham Pacha Ravi proposed openstack/cinder master: Fix provisioned_capacity_gb for drivers that don't report it  https://review.openstack.org/52347817:17
*** shaner has quit IRC17:19
*** shaner has joined #openstack-cinder17:21
vivsoni_Thanks Jay for the review17:27
*** dhinesh_ has joined #openstack-cinder17:29
*** ntpttr_laptop has joined #openstack-cinder17:30
openstackgerritEric Harney proposed openstack/cinder master: Move NFS CI job to .zuul.yaml  https://review.openstack.org/52313817:30
vivsoni_Thanks Sean for the review :)17:37
*** Apoorva has joined #openstack-cinder17:40
*** Apoorva has quit IRC17:40
*** Apoorva has joined #openstack-cinder17:41
*** dhinesh_ has quit IRC17:42
*** dhinesh has joined #openstack-cinder17:43
*** dhinesh has quit IRC17:44
*** dhinesh has joined #openstack-cinder17:44
*** lpetrut has quit IRC17:45
openstackgerritLouie Kwan proposed openstack/cinder master: Allow purging of records less than 1 day old  https://review.openstack.org/51156717:48
dhineshhad a question, should the CI pass all the tempest tests in "tox -e all -- volume"(there are around 159 test)? when the CI is configured with the vendor specific cinder driver. So it will be considered to merge for queens release.17:48
smcginnisdhinesh: Yes, it should pass all volume tests.17:50
dhineshsmcginnis: ok thanks17:50
smcginnisdhinesh: Skipping a few of them is acceptable if there is a known issue you are working through, but it should only be a small amount and not any of the tests for basic operations.17:50
openstackgerritMerged openstack/python-cinderclient master: Revert "Bump MAX version of client to 3.45"  https://review.openstack.org/51641017:51
*** pcaruana has quit IRC17:53
dhineshsmcginnis: ok, what are considered as basic operations? i have failures in mostly testing negative test cases.17:54
*** moshele has joined #openstack-cinder17:55
smcginnisdhinesh: Can you put the failure list in paste.openstack.org? We can take a look.17:57
* smcginnis has to step away for a bit but will be back soon.17:57
dhineshsmcginnis: the failures are on the top and passing cases are in the bottom http://paste.openstack.org/show/627619/18:00
*** moshele has quit IRC18:01
*** josecastroleon has joined #openstack-cinder18:02
*** jmlowe has joined #openstack-cinder18:05
openstackgerritJohn Griffith proposed openstack/cinder master: Add shared_targets flag to Volumes  https://review.openstack.org/52067618:08
*** jmlowe has quit IRC18:25
*** liverpooler has quit IRC18:25
*** stvnoyes has quit IRC18:27
*** liverpooler has joined #openstack-cinder18:28
*** jmlowe has joined #openstack-cinder18:28
*** dhinesh has quit IRC18:29
*** armaan has quit IRC18:32
*** armaan has joined #openstack-cinder18:32
*** moshele has joined #openstack-cinder18:35
*** stvnoyes has joined #openstack-cinder18:35
*** AlexeyAbashkin has joined #openstack-cinder18:35
*** josecastroleon has quit IRC18:36
*** AlexeyAbashkin has quit IRC18:40
*** adisky_ has quit IRC18:43
*** crose has joined #openstack-cinder18:45
*** dhinesh has joined #openstack-cinder18:53
openstackgerritEric Harney proposed openstack/cinder master: Move NFS CI job to .zuul.yaml  https://review.openstack.org/52313818:56
*** AlexeyAbashkin has joined #openstack-cinder18:56
openstackgerritMerged openstack/cinder master: Support create volume from backup  https://review.openstack.org/51294918:57
openstackgerritMerged openstack/cinder master: Use new oslo.db test cases  https://review.openstack.org/52229018:57
*** moshele has quit IRC19:01
*** armaan has quit IRC19:01
*** armaan has joined #openstack-cinder19:02
*** AlexeyAbashkin has quit IRC19:04
smcginnisdhinesh: There are a few there that should pass. test_volume_list, test_snapshots_actions, test_volume_snapshots, etc. are all basic operations.19:04
openstackgerritMerged openstack/cinder master: Move legacy-cinder-tox-* jobs in-tree  https://review.openstack.org/52165419:09
openstackgerritMerged openstack/cinder master: HPE3PAR: Create FC vlun as host sees  https://review.openstack.org/52294719:11
*** rtjure has joined #openstack-cinder19:13
*** vivsoni_ has quit IRC19:13
*** salv-orlando has joined #openstack-cinder19:17
*** rtjure has quit IRC19:18
*** msno has quit IRC19:35
dhineshsmcginnis: ok thanks19:36
*** sdague has quit IRC19:37
*** links has joined #openstack-cinder19:39
jungleboyjsmcginnis:  It was your removal of the old initiator constants that broke cinderclient-brick-ext  Ooops.19:39
*** stvnoyes has quit IRC19:40
*** links has quit IRC19:42
smcginnisjungleboyj: Ah, another place that needed cleanup.19:42
jungleboyjYeah, give me a minute here and I should have a fix.19:42
jungleboyjAt least it is easy to fix.19:47
*** gouthamr_ has joined #openstack-cinder19:53
*** gouthamr has quit IRC19:56
*** stvnoyes has joined #openstack-cinder20:02
*** e0ne has quit IRC20:03
*** armaan has quit IRC20:10
*** lpetrut has joined #openstack-cinder20:14
openstackgerritEric Harney proposed openstack/cinder master: Disallow unmanaging encrypted volumes  https://review.openstack.org/52352220:15
*** eharney has quit IRC20:19
*** pcaruana has joined #openstack-cinder20:21
*** markstur has quit IRC20:24
*** schmots_ has joined #openstack-cinder20:24
*** schmots has quit IRC20:25
*** schmots_ is now known as schmots20:25
*** edmondsw_ has joined #openstack-cinder20:52
*** edmondsw has quit IRC20:56
*** diablo_rojo has quit IRC21:04
*** moshele has joined #openstack-cinder21:04
jungleboyjjgriffith:  If you have a couple moments can you let me know if this is a valid bug: https://bugs.launchpad.net/cinder/+bug/173499921:05
openstackLaunchpad bug 1734999 in Cinder "connection_info returned from attachments API is blank" [High,New]21:05
jungleboyjLooking into it today.  Need a way to pull attachment info from a volume after it has been attached.  Expected to be able to see it there.21:06
openstackgerritJohn Griffith proposed openstack/cinder master: Add shared_targets flag to Volumes  https://review.openstack.org/52067621:09
jgriffithjungleboyj: pong21:09
jgriffithildikov: pong21:09
jungleboyjjgriffith:  Hey, question is basically the bug I put together above.21:09
jgriffithjungleboyj: yeah, just looking; not sure I'm following?21:11
jgriffithjungleboyj: the response is pasted in there no?21:11
jungleboyjYes but connection_info is 'null'.21:12
jgriffithjungleboyj: jungleboyj did you do a show/detail?21:12
jgriffithjungleboyj: and is it "attached"?  Or just reserved?21:12
jgriffithjungleboyj: if you just create an attachment it won't have connection info right?21:12
jungleboyjThis was one that was actually attached.21:13
jungleboyjjgriffith:  I need to poke around some more but it sounds to me that your initial reaction isn't that I am doing something stupid?21:14
jgriffithjungleboyj: I wouldn't use the word "stupid" by any stretch21:14
jgriffithjungleboyj: but you have the Nova changes loaded up and you're using the new api to make the attachment?21:15
smcginnisjungleboyj: Oh, you are using the existing (old) attach, right?21:15
*** stvnoyes has quit IRC21:15
jgriffith:)21:15
jungleboyjjgriffith:  So, that is my second question.  Would that info be populated if attaching using the current nova code?21:16
jgriffithnope21:17
jungleboyjOk, so that may be source of my problem.21:17
jgriffithjungleboyj: you have to use the new cinder api's to get that all set21:17
*** armax has quit IRC21:17
jgriffithjungleboyj: I'll load everything up and look at it this afternoon and make sure something's not messed up21:17
jgriffithjungleboyj: but if it wasn't there, the nova tests certainly wouldn't make it very far :)21:18
jungleboyjjgriffith:  So, in order to see connection_info populated I would need to load up your patches to Cinder, Nova and run with that?21:19
ildikovjgriffith: jungleboyj: the Nova tests are quite happy now :)21:19
jungleboyjildikov:  That is good.21:19
*** diablo_rojo has joined #openstack-cinder21:21
*** schmots has joined #openstack-cinder21:21
jungleboyjjgriffith:  I will give that a try and see if I get the anticipated result.21:21
jgriffithjungleboyj: you don't need any Cinder patches, but you need to set the api version and use it's attachment-create/attachment-complete calls21:21
jgriffithjungleboyj: you will :)21:21
*** gouthamr_ is now known as gouthamr21:21
jgriffithjungleboyj: get the anticipated results that is21:21
*** pcaruana has quit IRC21:21
jungleboyjjgriffith:  Ok, so just the Nova side and set the new microversion.21:21
jungleboyjThat will be a good exercise for me anyway.21:22
jungleboyjjgriffith:  Thanks.  Gotta run.  Will give it a try later.21:22
ildikovjgriffith: I wanted to ask you about the BP we were talking about for multi-attach on the Cinder side21:23
ildikovjgriffith: whether you started that already?21:23
jgriffithI have not, was on vacation :)21:23
smcginnisWho let you do that?21:23
jgriffithsmcginnis: :)21:24
ildikovok, and when/if do you plan to write it up?21:24
jgriffithI guess I should put something together today maybe so we can talk about it in cinder meeting tomorrow perhaps?21:24
ildikovjgriffith: that would be pretty great21:25
jgriffithildikov: well... not for me :)21:25
jgriffithjust kidding21:26
ildikovjgriffith: I know :)21:26
jgriffithsmcginnis: I never did figure out how to fix the stupid NEC unit tests21:26
jgriffithsmcginnis: re: ssh_known_hosts21:27
smcginnisjgriffith: Yeah, me neither.21:27
ildikovjgriffith: sorry, on a call at the same time21:27
smcginnisAnd I've accidentally "git add ." that file several times now.21:27
*** edmondsw_ is now known as edmondsw21:27
jgriffithsmcginnis: :) screw it, I'll add it to .gitignore :)21:27
jgriffithagain... just kidding21:27
smcginnisjgriffith: Ya know... I was thinking the same thing. Really should fix it, but at this point...?21:28
jgriffithwell, I was at least hoping to zero in on which test was creating it and skipping until bug fix21:28
jgriffithbut that could be *all* of them, and it could stay that way for years21:28
smcginnisThat's part of the frustration. Looking through, it's not obvious where it's coming from.21:29
jgriffithsmcginnis: well, I'd be lieing if I didn't say part of me was glad you didn't find it easily21:29
jgriffithsmcginnis: as much time as I spent on it I would've looked even dumber than usual :)21:29
smcginnisHah!21:29
*** rcernin has quit IRC21:33
*** liverpooler has quit IRC21:37
*** liverpooler has joined #openstack-cinder21:38
jgriffithWTF again!!  ERROR: 'Namespace' object has no attribute 'os_domain_id'21:41
jgriffithalright jungleboyj you're on your own21:41
*** openstackstatus has quit IRC21:42
*** openstack has joined #openstack-cinder21:47
*** ChanServ sets mode: +o openstack21:47
jgriffithhttps://bugs.launchpad.net/cinder/+bug/173299221:47
smcginnisjgriffith: Cool, I'll see if I can track it down.21:47
*** crose has quit IRC21:47
jgriffithsweet!!21:47
smcginnisBug bot, where'd you go? :)21:48
*** moshele has quit IRC21:48
openstackLaunchpad bug 1732992 in Cinder "Unit tests for NEC are leaving ssh_known_hosts file in root test dir" [Undecided,New]21:49
jgriffithsmcginnis: how did you do that :)21:49
smcginnis:D21:50
ildikovjgriffith: BTW, do you also plan to have the API patch up for the shared_targets change this week?21:52
jgriffithildikov: if the other patch ever merges yes :)21:55
*** sticker has joined #openstack-cinder21:55
openstackgerritMatthew Edmonds proposed openstack/cinder master: handle no default_cgsnapshot_type  https://review.openstack.org/52060721:55
ildikovjgriffith: do you expect big changes to that patch?21:56
dhineshif there are bug fixes after the code has been merged to cinder, how do we address them?21:56
jgriffithildikov: ?  I don't expect any changes to it21:56
jgriffithit's just waiting, getting rebased21:56
jgriffithtic-toc21:57
ildikovjgriffith: I'm just asking as I guess a few people will disappear for KubeCon next week including you so it would be good to have version up to try to move forward with things21:57
jgriffithildikov: there's still >3 working days left in this week for someone to click on +A21:57
jgriffith:)21:57
jgriffithI'm holding out hope and being the optimist that I am21:58
jgriffith:)21:58
ildikovjgriffith: and as you said that you wanted the current patch to be merged first I thought you might expect changes to that one before it can get accepted21:58
jgriffithildikov: nah, I think it should be set; but depends on if somebody find a problem that I missed21:58
ildikovok, so21:58
ildikovjungleboyj: smcginnis: plz look into this one: https://review.openstack.org/#/c/520676/ :)21:59
edmondswsmcginnis does my comment and commit msg update help on https://review.openstack.org/#/c/520607 ?22:01
*** armax has joined #openstack-cinder22:02
smcginnisedmondsw: Sure, that makes sense. Thanks!22:02
edmondswnp22:02
edmondswsorry for the confusion22:02
*** hemna_ has joined #openstack-cinder22:06
openstackgerritSean McGinnis proposed openstack/cinder master: Mock out CLI execution in NEC driver tests  https://review.openstack.org/52354622:10
smcginnisjgriffith: Boom22:10
hemna_yough22:11
smcginnisHoly cow, it's a hemna_!22:11
jgriffithYou may have outsmarted me this time smcginnis ... but next time... next time!!!22:11
jgriffith:)22:11
smcginnisjgriffith: :)22:11
hemna_cinder migrate an attached nfs volume.   is that supposed to work? :)22:11
jgriffithsmcginnis: OMG seriously!!!!22:11
smcginnisjgriffith: I knew it would end up being a one line change. Well, close enough.22:12
smcginnishemna_: I think so, but I have steered clear of NFS so far.22:12
jgriffithhemna_: yeah it should, there's a tempest test for that IIRC22:13
smcginnisOn that note, time to cook for my poor starving children.22:13
eharneyhemna_: yes22:13
hemna_ok22:15
hemna_I  have a customer complaining that the vm is locking up during the copy22:15
*** schmots has quit IRC22:15
jungleboyjeharney: i guessed that wrong. :-(22:15
*** abishop has quit IRC22:17
*** schmots has joined #openstack-cinder22:18
hemna_so in this case nova handles the copying of data it looks like in the volume/manager.py22:19
hemna_nova_api.update_server_volume()22:19
*** rcernin has joined #openstack-cinder22:23
*** moshele has joined #openstack-cinder22:28
hemna_ah yah, swap_volume()22:30
hemna_which looks like a dev.rebase() eventually.   I'm assuming that's a qemu command ?22:30
eharneyyeah, libvirt/qemu block rebase22:31
*** edmondsw has quit IRC22:36
*** edmondsw has joined #openstack-cinder22:39
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements  https://review.openstack.org/51908822:42
jgriffithjungleboyj: FYI  https://bugs.launchpad.net/cinder/+bug/173499922:43
openstackLaunchpad bug 1734999 in Cinder "connection_info returned from attachments API is blank" [High,Invalid]22:43
*** edmondsw has quit IRC22:43
jungleboyjjgriffith: cool thanks. Going to try too do it using the new APIs from the brick-cinderclient-ext .22:46
*** schmots has quit IRC22:49
*** schmots has joined #openstack-cinder22:51
*** mchlumsky has quit IRC23:07
openstackgerritEric Harney proposed openstack/cinder master: Disallow unmanaging encrypted volumes  https://review.openstack.org/52352223:09
openstackgerritEric Harney proposed openstack/cinder master: Tests: Cleanup utils.create_volume fields  https://review.openstack.org/52356323:09
*** lbragstad has quit IRC23:17
*** edmondsw has joined #openstack-cinder23:18
*** edmondsw has quit IRC23:23
*** schmots has quit IRC23:23
*** lbragstad has joined #openstack-cinder23:24
*** lpetrut has quit IRC23:25
*** wN has joined #openstack-cinder23:25
*** wN has quit IRC23:25
*** wN has joined #openstack-cinder23:25
openstackgerritOpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext master: Updated from global requirements  https://review.openstack.org/52050523:26
*** ganso has quit IRC23:30
*** schmots has joined #openstack-cinder23:35
*** salv-orl_ has joined #openstack-cinder23:37
*** salv-orlando has quit IRC23:40
*** lbragstad has quit IRC23:56
openstackgerritJohn Griffith proposed openstack/cinder-specs master: Propose multi-attach using V3 Attachment API's  https://review.openstack.org/52360823:57

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!