Thursday, 2018-08-02

*** dave-mccowan has quit IRC00:02
*** dr_gogeta86 has quit IRC00:04
*** namnh has joined #openstack-cinder00:04
*** edmondsw has joined #openstack-cinder00:07
*** dr_gogeta86 has joined #openstack-cinder00:07
*** clarkb has quit IRC00:09
*** namnh has quit IRC00:10
*** edmondsw has quit IRC00:12
*** _ix has joined #openstack-cinder00:15
*** tswanson has quit IRC00:21
*** dave-mccowan has joined #openstack-cinder00:22
*** mriedem has quit IRC00:29
*** SatelliteN00b has quit IRC00:35
*** _ix has quit IRC00:36
*** zhaochao has joined #openstack-cinder00:55
*** dave-mccowan has quit IRC00:55
openstackgerritMerged openstack/cinder master: XtremIO: support multiattach  https://review.openstack.org/55452401:02
*** EmilienM has quit IRC01:02
*** EmilienM has joined #openstack-cinder01:03
*** namnh has joined #openstack-cinder01:08
*** namnh has quit IRC01:13
*** itlinux_ has quit IRC01:26
*** edmondsw has joined #openstack-cinder01:29
*** namnh has joined #openstack-cinder01:35
*** liuzz_ has quit IRC01:35
*** dave-mccowan has joined #openstack-cinder01:47
*** edmondsw has quit IRC01:50
*** edmondsw has joined #openstack-cinder01:51
*** edmondsw has quit IRC01:55
*** namnh has quit IRC01:59
*** namnh has joined #openstack-cinder02:08
*** psachin has joined #openstack-cinder02:20
*** e0ne has joined #openstack-cinder02:23
*** e0ne has quit IRC02:23
*** LutzB has quit IRC02:59
*** gcb has quit IRC03:07
*** tswanson has joined #openstack-cinder03:11
*** edmondsw has joined #openstack-cinder03:19
*** edmondsw has quit IRC03:24
*** dave-mccowan has quit IRC03:33
*** dalgaaf has quit IRC03:34
*** namnh has quit IRC03:44
*** fnordahl has joined #openstack-cinder05:05
*** tswanson has quit IRC05:29
*** Luzi has joined #openstack-cinder06:12
openstackgerrityenai proposed openstack/cinder master: Add none check when do _retype_error  https://review.openstack.org/58814406:12
openstackgerritTommyLike proposed openstack/cinder master: Increase length limit on image metadata content  https://review.openstack.org/58673706:16
*** hoonetorg has quit IRC06:42
*** hoonetorg has joined #openstack-cinder06:55
*** rcernin has quit IRC07:03
openstackgerritwu.chunyang proposed openstack/python-cinderclient master: update wrong link  https://review.openstack.org/58815607:11
*** gouthamr has quit IRC07:36
*** devfaz has quit IRC07:45
*** Tahvok has quit IRC07:45
*** strigazi has quit IRC07:45
*** mvk_ has quit IRC07:46
*** tbarron has quit IRC07:46
*** eandersson has quit IRC07:46
*** Tahvok has joined #openstack-cinder07:46
openstackgerritVivek Soni proposed openstack/cinder master: VSA: Concurrent request handling in attachment  https://review.openstack.org/58817108:17
*** gnufied has quit IRC08:33
*** pcaruana has joined #openstack-cinder09:00
openstackgerritMerged openstack/cinder master: nimble: Fix missing ssl support  https://review.openstack.org/58781109:01
openstackgerritBoxiang Zhu proposed openstack/cinder master: replace status value with objects.fields  https://review.openstack.org/58818909:31
*** arne_wiebalck has joined #openstack-cinder09:44
openstackgerritDavid Rabel proposed openstack/cinder master: Fix indentation in docs  https://review.openstack.org/58819309:44
*** edmondsw has joined #openstack-cinder09:48
*** hoonetorg has quit IRC09:54
*** VikasReddy1 has quit IRC09:59
openstackgerritBoxiang Zhu proposed openstack/cinder master: replace status value with objects.fields  https://review.openstack.org/58818910:00
*** tbarron has joined #openstack-cinder10:05
*** hoonetorg has joined #openstack-cinder10:09
openstackgerritBoxiang Zhu proposed openstack/cinder master: remove some invalid variable statement  https://review.openstack.org/58820610:35
*** vivsoni has joined #openstack-cinder10:39
openstackgerritBoxiang Zhu proposed openstack/cinder master: meet the pep8 N806 rule  https://review.openstack.org/58821010:51
*** ganso has joined #openstack-cinder10:55
*** wolsen has quit IRC11:04
*** fmccarthy has quit IRC11:04
*** rajinir has quit IRC11:04
*** lamt has quit IRC11:04
*** simondodsley_ has quit IRC11:04
*** simondodsley_ has joined #openstack-cinder11:05
*** zzzeek has quit IRC11:07
*** zzzeek has joined #openstack-cinder11:13
*** dave-mccowan has joined #openstack-cinder11:27
*** luizbag has joined #openstack-cinder11:32
*** abishop has joined #openstack-cinder11:57
openstackgerritcaixiaoyu proposed openstack/cinder master: volume/rbd: Add judgment whether a rbd snapshot is protected from deletion  https://review.openstack.org/58822612:02
*** vivsoni_ has joined #openstack-cinder12:06
*** vivsoni has quit IRC12:07
*** tpsilva has joined #openstack-cinder12:15
openstackgerritcaixiaoyu proposed openstack/cinder master: volume/rbd: Judge protected attr when deleting snapshot  https://review.openstack.org/58822612:16
*** dustins has joined #openstack-cinder12:20
*** EmilienM has left #openstack-cinder12:21
*** mvenesio has quit IRC12:25
*** jmlowe has quit IRC12:35
*** cmart has quit IRC12:35
*** mriedem has joined #openstack-cinder12:57
*** eharney has quit IRC12:59
openstackgerritBoxiang Zhu proposed openstack/cinder master: replace status value with objects.fields  https://review.openstack.org/58818913:33
*** Luzi has quit IRC13:35
*** bswartz has quit IRC13:38
*** gnufied has joined #openstack-cinder13:40
*** _ix has joined #openstack-cinder13:41
*** eharney has joined #openstack-cinder13:45
*** _ix has quit IRC13:49
hemna_morning13:55
smcginnisHey hemna_ o/13:59
openstackgerritMerged openstack/cinder master: Fix indentation in docs  https://review.openstack.org/58819314:00
*** erlon has quit IRC14:04
*** luizbag has quit IRC14:04
jungleboyjMorning all.14:12
mnaserhi everyone14:15
*** luizbag has joined #openstack-cinder14:15
*** erlon has joined #openstack-cinder14:16
mnaserwe make heavy use of the iops_per_gb feature and one of our private cloud users runs very small volumes which means sometimes they might end up with a tiny volume that barely gets any iops14:16
mnaseri'd like to propose a change that adds an iops_per_gb_min which is a minimum, so it gets either that or the multiplied value14:17
mnaseri don't think it would be a huge change, probably a few lines, but is it too late in the cycle :(14:17
mnaser(there should be a question mark there)14:17
eharneymnaser: sounds like a reasonable thing to work on14:18
smcginnisYeah, I would expect that to be a small change.14:18
jungleboyj++14:18
mnaseri'm just wondering if that type of thing can still get merged or it's too late .. it would be nice to be able to get it for rocky rather than keep it only be able to get it in stein14:18
smcginnisIf we see a patch soon, I would assume it would be a low risk thing to still allow in.14:19
jungleboyjKind of sounds like a bug to me?  ;-)14:19
mnaserjungleboyj: sneaky, aha ;p14:20
mnaseri'll push up something very shortly14:20
jungleboyjSounds good.  Ping us and we will take a look.14:20
openstackgerritMohammed Naser proposed openstack/cinder master: Add ability to specify minimum value when using capacity based QoS  https://review.openstack.org/58830214:45
mnaserjungleboyj, smcginnis, eharney ^ patch with tests, docs and release notes :>14:45
smcginnismnaser: That was quick!14:45
mnaserit was pretty straight forward14:45
mnaserugh i touched one of the original tests14:46
mnaserone sec14:46
openstackgerritMohammed Naser proposed openstack/cinder master: Add ability to specify minimum value when using capacity based QoS  https://review.openstack.org/58830214:46
mnaserokay the diff should look much cleaner now14:46
openstackgerritMohammed Naser proposed openstack/cinder master: Add ability to specify minimum value when using capacity based QoS  https://review.openstack.org/58830214:55
mnasersmcginnis: thanks for that comment, TIL, pop'ing a value that didnt exist was something i was thinking about14:55
mnaserbut i didnt know you could also get a default value when popping so cool :>14:55
mnaserjungleboyj: ^14:56
smcginnismnaser: Always discovering ways python makes simple things even easier. ;)14:56
mnaserabsolutely :D14:56
*** gouthamr has joined #openstack-cinder15:02
*** tomtom001 has joined #openstack-cinder15:14
tomtom001Hello all, trying to setup the cinder backup service with swift for Openstack Newton.  I set a user and key for the cinder user. I keep getting the following error message in the cinder-volume service whenever I try to make a backup. Container PUT failed: http://10.0.30.10:8080/v1/AUTH_744b62717c1946cb95d11909fea98a4e/ 405 Method Not Allowed  -- Anyone know why the method is not allowed?  Is15:17
tomtom001it just an auth thing?  What .conf options should be defined at minimum to get it to authenticate properly?15:17
*** psachin has quit IRC15:21
jungleboyjtomtom001:  Sounds like an authentication issue.15:23
jungleboyjI haven't done much with backup.  e0ne should pop on later and might be able to provide more guidance.15:23
*** tswanson has joined #openstack-cinder15:30
*** cmart has joined #openstack-cinder15:53
openstackgerritMerged openstack/cinder master: Update volume create schema to allow 'null'  https://review.openstack.org/58782116:07
*** gman-tx has joined #openstack-cinder16:09
*** eandersson has joined #openstack-cinder16:10
*** arne_wiebalck has quit IRC16:13
*** walshh_ has joined #openstack-cinder16:16
*** spotz has quit IRC16:18
tomtom001jungleboyj - thank you, that's what I thought, I'll keep playing around with the settings to see if I can get it working.16:28
*** walshh_ has quit IRC16:30
*** walshh_ has joined #openstack-cinder16:32
openstackgerritMerged openstack/cinder master: tintri: Enable SSL with requests  https://review.openstack.org/58744516:34
openstackgerritMerged openstack/cinder master: NEC driver: Return non-random iscsi portal in initialize  https://review.openstack.org/57639016:34
openstackgerritMerged openstack/cinder master: [doc] Use openstack client commands to replace cinder client  https://review.openstack.org/58093516:34
openstackgerritMerged openstack/cinder master: docs: Write high availability devref  https://review.openstack.org/57124216:34
*** imacdonn has quit IRC16:36
*** imacdonn has joined #openstack-cinder16:37
openstackgerritJay Bryant proposed openstack/cinder master: Add sphinx-feature-classification based support matrix  https://review.openstack.org/58075716:41
*** walshh_ has quit IRC16:41
openstackgerritJay Rubenstein proposed openstack/cinder master: SF ensure the correct volume is deleted  https://review.openstack.org/58834217:00
openstackgerritRajat Dhasmana proposed openstack/python-cinderclient master: __repr__ crashes when empty dict passed  https://review.openstack.org/58834617:15
*** gnufied has quit IRC17:31
openstackgerritMerged openstack/cinder master: VSA: Concurrent request handling in attachment  https://review.openstack.org/58817117:34
*** gnufied has joined #openstack-cinder17:47
*** itlinux has joined #openstack-cinder18:14
*** gman-tx has quit IRC18:15
*** tswanson has quit IRC18:37
openstackgerritMerged openstack/cinder master: Fix remotefs driver report wrong value  https://review.openstack.org/57697519:03
luizbagPeople, has anyone noticed failing stable/queens jobs due to os-brick on fibre channel?19:05
smcginnisluizbag: Did it just start?19:06
eharneyluizbag: unit tests?19:06
luizbageharney, tempest tests19:06
luizbagsmcginnis, we started noticing it a few days ago19:07
smcginnisluizbag: Did you change something in your environment? I don't think we've really backported much to stable/queens lately.19:07
luizbagsmcginnis, we haven't changed anything19:09
smcginnisNothing of significance changed there that I see for awhile: https://review.openstack.org/#/q/project:openstack/cinder+status:merged+branch:stable/queens19:09
openstackgerritMerged openstack/os-brick stable/queens: RemoteFS: don't fail in do_mount if already mounted  https://review.openstack.org/58323719:13
*** gnufied has quit IRC19:27
*** dustins has quit IRC19:38
*** gnufied has joined #openstack-cinder19:41
*** eharney has quit IRC19:44
openstackgerritChuck Short proposed openstack/cinder master: solidfire: Enable SSL with requests  https://review.openstack.org/58836519:44
*** lseki has joined #openstack-cinder19:44
jungleboyjluizbag:  There was another question about tests timing out on stable/queens yesterday.20:00
*** zul has quit IRC20:01
luizbaginteresting20:02
jungleboyjsmcginnis:  So, the requirements on stable queens were for os-brick >= 2.2.0 .20:03
jungleboyjWe are up to 2.5.320:03
smcginnisjungleboyj: upper-constraints limits that though.20:03
jungleboyjSo, would that be what is getting pulled in when running tempest since we didn't cap?20:03
smcginnishttps://github.com/openstack/requirements/blob/stable/queens/upper-constraints.txt#L21720:03
jungleboyjsmcginnis:  Ok.  That is good.20:04
*** masayukig has quit IRC20:18
luizbagthe problem is with the os-brick version on upper-constraints.txt20:21
luizbagit's freezing the version on 2.3.220:21
smcginnis2.3.2 has been out for a month and a half, so why would it start failing for you now?20:22
luizbagbecause now we have patches to run on fibre channel20:23
smcginnisluizbag: So you weren't running before and that's why you just noticed it fails?20:24
luizbagno, the upper-constraints.txt is updated on master, but on stable/queens it's with the old version20:25
luizbagwe're working on a backport to stable/queens that's having this issue20:25
luizbagthe patches for master branch are passing20:25
smcginnisluizbag: Right, this is stable/queens we're talking about, not master I thought.20:26
luizbagshould I open a bug for that?20:27
smcginnisFor what?!20:27
smcginnisAre we talking stable/queens or are we talking master?20:27
luizbagstable/queens20:27
luizbagto update the upper-constraints.txt file20:28
smcginnisluizbag: It has the most up to date in upper-constraints already.20:29
luizbagor just create a patch to update it?20:29
luizbagyes, on master branch20:29
luizbagthe stable/queens branch on requirements project is not up to date20:29
smcginnisluizbag: Yes it is.20:30
luizbagbut I'm reading version 2.3.2 on it...20:30
smcginnisYes.20:31
smcginnisThat's the latest stable/queens os-brick version - https://github.com/openstack/releases/blob/master/deliverables/queens/os-brick.yaml#L3520:31
luizbagso I'm kinda lost here20:32
luizbaghow do I fix it?20:32
smcginnisluizbag: It depends. What is actually broken?20:32
smcginnisIf it's something that's been fixed during rocky, then it should be backported to stable/queens and we can do another release.20:33
smcginnisBut other than it's broke and nothing was changed, I don't know what the problem is. :)20:33
luizbagI've found a bug already reported: https://bugs.launchpad.net/os-brick/+bug/177428320:35
openstackLaunchpad bug 1774283 in os-brick "fc rescan issue when looking at specific hbas" [Undecided,Fix released] - Assigned to Patrick East (patrick-east)20:35
luizbagthat's the problem20:35
smcginnisYeah, that looks like a good thing to backport.20:38
jungleboyjsmcginnis: ++20:38
jungleboyjluizbag:  Do you know how to backport?20:38
luizbagyes20:39
smcginnisluizbag: It you put up a patch to backport that, then jungleboyj and I can take a look and get it through if all's good.20:39
smcginnisAnd get a new stable/queens release done.20:39
jungleboyjOk.  Please propose the backport and then I will work on requesting another release.20:39
jungleboyjYeah, what smcginnis said.  ;-)20:40
luizbagok, thanks, I'll do it asap20:46
*** luizbag has quit IRC20:47
*** lseki has quit IRC21:02
*** dcdamien has quit IRC21:17
*** abishop has quit IRC21:24
*** mriedem is now known as mriedem_afk21:28
*** itlinux has quit IRC21:54
*** mchlumsky has quit IRC22:02
*** ganso has quit IRC22:24
*** rcernin has joined #openstack-cinder22:32
openstackgerritChuck Short proposed openstack/cinder master: solidfire: Enable SSL with requests  https://review.openstack.org/58836522:34
*** tswanson has joined #openstack-cinder22:47
*** tpsilva has quit IRC22:53
*** edmondsw has quit IRC23:08

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