Friday, 2018-08-03

*** tonyb has quit IRC00:16
openstackgerritBradley Jones proposed openstack/cinder master: [DNM] Testing Cisco Third party CI  https://review.openstack.org/58841100:22
*** edmondsw has joined #openstack-cinder00:37
*** edmondsw has quit IRC00:42
openstackgerritMerged openstack/cinder master: NetApp ONTAP: Fix driver force detach operations  https://review.openstack.org/58808200:58
*** mriedem_afk has quit IRC01:34
*** tonyb has joined #openstack-cinder02:22
*** edmondsw has joined #openstack-cinder02:25
*** edmondsw has quit IRC02:30
openstackgerritzhufl proposed openstack/cinder master: Fix none-ascii char in doc  https://review.openstack.org/58842402:33
*** psachin has joined #openstack-cinder02:34
openstackgerrityenai proposed openstack/cinder master: Add multiattach attribute when do backup restore  https://review.openstack.org/58766702:35
*** dave-mccowan has quit IRC02:35
*** itlinux_ has joined #openstack-cinder03:31
openstackgerritcaixiaoyu proposed openstack/cinder master: Add test case: create volume from source volume  https://review.openstack.org/58843103:32
openstackgerritBoxiang Zhu proposed openstack/cinder master: swap out easily readable strings for constants  https://review.openstack.org/58818903:55
*** dklyle has quit IRC04:39
*** gnufied has quit IRC05:27
openstackgerritBoxiang Zhu proposed openstack/cinder master: swap out easily readable strings for constants  https://review.openstack.org/58818906:10
*** Luzi has joined #openstack-cinder06:13
openstackgerritKumar Prashant proposed openstack/cinder master: VMAX Driver - Fix for multiple clones failure  https://review.openstack.org/57848606:19
openstackgerritcheng li proposed openstack/cinder master: Using "availability_zone" property when restoring  https://review.openstack.org/58844706:24
*** whoami-rajat__ has quit IRC06:34
*** rcernin has quit IRC06:54
*** kaisers has quit IRC07:03
openstackgerritYu Zhang proposed openstack/cinder master: IBM XIV: fix issues for replication  https://review.openstack.org/58845307:09
openstackgerritcaixiaoyu proposed openstack/cinder master: Add test case: create volume from source volume  https://review.openstack.org/58843107:25
openstackgerritcaixiaoyu proposed openstack/cinder master: Add test case: create volume from source volume  https://review.openstack.org/58843107:33
openstackgerrityenai proposed openstack/cinder master: Add Flase check when do _retype_error  https://review.openstack.org/58814407:49
*** tommylikehu is now known as tommylikehu208:02
*** tommylikehu2 is now known as tommylikehu08:03
*** tommylikehu is now known as tommylikehu_afk08:04
*** tommylikehu_afk is now known as tommylikehu08:09
*** vivsoni_ has quit IRC08:29
*** deepak_mourya has quit IRC09:25
*** vivsoni has joined #openstack-cinder09:31
openstackgerritcheng li proposed openstack/cinder master: Using "availability_zone" property in restoration  https://review.openstack.org/58844709:48
*** obre is now known as obre_09:58
*** obre_ is now known as obre09:58
*** obre has quit IRC10:04
*** obre has joined #openstack-cinder10:06
openstackgerritcaixiaoyu proposed openstack/cinder master: Add test case: create volume from source volume  https://review.openstack.org/58843110:09
*** luizbag has joined #openstack-cinder10:47
*** e0ne has joined #openstack-cinder11:01
*** e0ne has quit IRC11:01
*** e0ne has joined #openstack-cinder11:02
*** e0ne has quit IRC11:02
*** e0ne has joined #openstack-cinder11:17
*** e0ne has quit IRC11:17
*** vivsoni has quit IRC11:17
*** walshh_ has joined #openstack-cinder11:38
*** dave-mccowan has joined #openstack-cinder11:43
*** tpsilva has joined #openstack-cinder11:48
*** abishop has joined #openstack-cinder11:52
*** walshh_ has quit IRC11:56
*** zul has joined #openstack-cinder12:02
*** hemna_ has quit IRC12:04
*** hemna_ has joined #openstack-cinder12:15
*** mriedem has joined #openstack-cinder12:39
*** nicolasbock has joined #openstack-cinder13:00
*** edmondsw_ has joined #openstack-cinder13:00
*** eharney has joined #openstack-cinder13:07
*** dustins has joined #openstack-cinder13:07
*** dave-mccowan has quit IRC13:25
*** mriedem is now known as mriedem_afk13:25
*** lbragstad has quit IRC13:26
*** stephenfin is now known as finucannot13:28
*** psachin has quit IRC13:33
*** ganso has joined #openstack-cinder13:37
*** _ix has joined #openstack-cinder13:46
*** gnufied has joined #openstack-cinder13:49
*** gnufied has quit IRC13:55
kukacz_hi, what is the status of capacity based qos implementation? is there a release where this works with front-end policy?14:03
*** edmondsw_ is now known as edmondsw14:08
kukacz_I would like to implement qos with specs like `total_iops_sec_per_gb=30`, as in the example at https://docs.openstack.org/cinder/latest/admin/blockstorage-capacity-based-qos.html14:09
*** Luzi has quit IRC14:15
jungleboyjkukacz_:  By front end do you mean at the attachment point?14:20
kukacz_hmm, I mean by Nova with KVM - so libvirt/qemu perhaps14:22
kukacz_ie. defined the same way as in the example14:22
*** _ix has quit IRC14:22
jungleboyjkukacz_:  Ok.  So that has been implemented for a while.  The documentation should be correct.14:22
kukacz_I did a test and it did not work - applied as in the example, created new volume from the appropriate volume type (the one where this qos was associated), attached to existing instance and tested using fio from within an instance14:24
smcginnisluizbag: Morning. I didn't see a backport proposed for that os-brick patch. Did you need some help with that?14:24
kukacz_jungleboyj: is there perhaps something which needs to be configured in nova or cinder controllers or compute nodes, to make it work?14:25
luizbagNo, thanks. I need to pass it through our internal code review on NetApp14:25
smcginnisluizbag: Should I just do it then? It's just a backport. I thought NetApp got better about those things.14:26
jungleboyjkukacz_:  I didn't think there was anything extra that needs to be done.14:26
jungleboyjWhat storage backend do you have?14:26
luizbagsmcginnis: no need, I'll get it through this afternoon14:26
smcginnisHmm, OK. We could have it merged by now. :)14:27
kukacz_jungleboyj: solidfire and ceph14:29
kukacz_jungleboyj: does it make a difference?14:31
*** cmart has quit IRC14:31
jungleboyjkukacz_:  Interesting.14:32
jungleboyjkukacz_:  So, I am not as familiar with how this all works.  According to the support matrix RBD doesn't support QoS but it should work on Solidfire.14:32
kukacz_jungleboyj: I thought that "front-end" consumer means that the qos is applied at hypervisor, disregard the type of backend14:35
jungleboyjI think that it still requires some support from the backend?14:36
jungleboyjsmcginnis:  ^^^ Am I off the rails here?14:36
kukacz_jungleboyj: should mention that common "total_iops_sec" applied by "front-end" consumer works. the solidfire specific qos specs (all 3 of them) applied by "back-end" consumer works as well14:38
kukacz_jungleboyj: what I'm looking for though, is the capacity based model. I could not find it in solidfire driver description, so I thought I would implement it using front-end14:39
jungleboyjOh, so it is just the iops/gb ?14:40
jungleboyjThat isn't working?14:40
smcginnisjungleboyj: I would look to eharney or maybe mnaser on that. I have not touched QoS much.14:41
jungleboyjsmcginnis:  That was what I was thinking.14:41
jungleboyjkukacz_:  What release are you on?14:41
kukacz_jungleboyj: yes, the iops/gb is what I'm seeking14:41
kukacz_jungleboyj: ocata14:42
jungleboyjkukacz_:  Hmmm, let me look here.  The iops/gb was added more recently.  I am not totally sure when.  Give me a minute to look here.14:42
smcginnisCapacity based merged in pike or queens I think.14:42
jungleboyjsmcginnis:  Ah, there would be the rub.  :-)14:42
kukacz_thanks, that would explain it14:43
jungleboyjkukacz_:  That was my first thought but I missed the capacity based part of your first note.  :-)14:43
smcginnisYep, spec was merged in pike - https://specs.openstack.org/openstack/cinder-specs/specs/pike/capacity_based_qos.html14:43
*** mriedem_afk is now known as mriedem14:43
jungleboyjsmcginnis:  Thank you for confirming sir.14:44
kukacz_pike, great! that's the easier upgrade path for me :-)14:44
kukacz_smcginnis, jungleboyj: thanks a lot guys!14:44
jungleboyjkukacz_:  No problem.  Glad to help.14:45
jungleboyjkukacz_:  Out of curiosity what are you going to do instead?14:45
kukacz_hmm, sorry, what do you mean? are you asking about the usecase?14:46
*** cmart has joined #openstack-cinder14:47
jungleboyjSince it isn't implemented in ocata what will you do instead?14:48
kukacz_I'll consider proceeding with upgrade or will have to build a pricing model specifying both the $/IOPS and $/GiB values to stay on the safe side with storage utilization14:52
kukacz_my motivation for the IOPS/GB is to keep a static ratio on a performance tier and to avoid bearing costs for the extreme situations where customers consume large IOPS with just small GiBs and vice versa14:55
kukacz_... means I prefer the GCE storage pricing model over the AWS one :-)14:57
* mnaser reads buffer14:58
jungleboyjkukacz_:  Sounds good.  Hope you are able to do the upgrade to get the functionality in place.14:58
mnaserah yeah, looks like jungleboyj figured it out14:59
mnaserit was added in pike so yeah14:59
jungleboyjmnaser:  The credit goes to smcginnis  :-)14:59
mnasergah i fixed the pep8 issue yesterday but forgot to psuh up change15:00
mnaserdarn15:00
smcginnisLikely story. :)15:00
openstackgerritMohammed Naser proposed openstack/cinder master: Add ability to specify minimum value when using capacity based QoS  https://review.openstack.org/58830215:00
mnaservoila15:00
*** gnufied has joined #openstack-cinder15:16
*** cmart has quit IRC15:23
*** gnufied has quit IRC15:25
*** pooja_jadhav has quit IRC15:25
*** cmart has joined #openstack-cinder15:41
*** Shilpa has quit IRC15:43
*** itlinux_ has quit IRC15:43
*** itlinux has joined #openstack-cinder15:49
*** mriedem is now known as hansmoleman15:57
-openstackstatus- NOTICE: The infra team is renaming projects in Gerrit. There will be a short ~10 minute Gerrit downtime in a few minutes as a result.16:01
*** melwitt is now known as jgwentworth16:14
*** cmart has quit IRC16:27
*** pocketprotector- has quit IRC16:27
*** pocketprotector- has joined #openstack-cinder16:27
*** pocketprotector- is now known as pocketprotector16:27
*** psachin has joined #openstack-cinder16:29
*** imacdonn has quit IRC16:37
*** imacdonn has joined #openstack-cinder16:37
*** cmart has joined #openstack-cinder16:41
*** openstackgerrit has quit IRC16:49
*** zhaochao has quit IRC16:52
hansmolemansmcginnis: anyone ever talk about upstreaming something like this? https://github.com/starlingx-staging/stx-cinder/commit/ac9c475f6f598be8312b7480918525ac87eddf21#diff-ce3ab5d94902facf73ddd54d10739347R11617:13
hansmolemanthis is mriedem btw17:13
*** gouthamr has quit IRC17:23
*** erlon has quit IRC17:26
*** gouthamr has joined #openstack-cinder17:26
-openstackstatus- NOTICE: Project renames and review.openstack.org downtime are complete without any major issue.17:27
smcginnishansmoleman: We have the messaging API for a more general way of getting details on failures with async operations.17:28
smcginnishansmoleman: Does that track any faults with the volume, even outside of async operations.17:28
smcginnis(I know you are not involved enough to expect you to know that)17:28
*** psachin has quit IRC17:29
smcginnisQuick skim it looks like it's just on volume create, so we do have that now with the messages API.17:31
*** gouthamr has quit IRC17:33
hansmolemanok i sort of knew about the messages API so i figured that was the answer17:33
smcginnishansmoleman: Looks like we could add some better description here - https://developer.openstack.org/api-ref/block-storage/v3/index.html#messages17:34
smcginnisTrying to track down the spec from that too.17:34
hansmolemani'd think people using the messages API would also want to filter messages based on resource_uuid17:35
hansmolemanso gimme all the messages for my volume17:35
smcginnisHmm, I thought we did that, but it wouldn't appear so from the api-ref.17:36
smcginnisOriginal spec - https://specs.openstack.org/openstack/cinder-specs/specs/newton/summarymessage.html17:37
*** hemna_ has quit IRC17:41
jungleboyjhansmoleman:  That would be nice.17:47
hansmolemansimilar for listing attachments for a given volume17:48
hansmolemanwe need that on the nova side for some stuff,17:48
hansmolemanbut i'm too busy (lazy) to write a spec / change to cinder for that17:48
hansmolemanthe volume also has the attachments list in it though17:48
*** e0ne has joined #openstack-cinder18:15
*** gnufied has joined #openstack-cinder18:42
*** luizbag has quit IRC18:47
*** hemna_ has joined #openstack-cinder18:48
*** gouthamr has joined #openstack-cinder19:04
*** openstackgerrit has joined #openstack-cinder19:17
openstackgerritMerged openstack/cinder master: Add ability to specify minimum value when using capacity based QoS  https://review.openstack.org/58830219:17
*** woojay has left #openstack-cinder19:29
*** gouthamr has quit IRC20:00
openstackgerritSean McGinnis proposed openstack/cinder master: Improve messages api-ref description  https://review.openstack.org/58865320:21
*** harlowja has joined #openstack-cinder20:21
*** tswanson has quit IRC20:27
smcginnisjungleboyj: We still have a bug with python-cinderclient for v3 volume transfers. Filing a bug now.20:35
jungleboyjDang was going to try and test that. What is wrong?20:37
smcginnisNot sure yet, but I can repro the failure using the cinder CLI.20:37
smcginnisWorks with v2 API, not with v3.20:37
smcginnisDoesn't work with the 3.55 microversion either.20:38
*** e0ne has quit IRC20:40
*** e0ne has joined #openstack-cinder20:41
*** abishop has quit IRC20:45
*** e0ne has quit IRC20:45
smcginnisPffffft21:05
openstackgerritSean McGinnis proposed openstack/cinder master: Correct volume-transfers API endpoint  https://review.openstack.org/58866421:17
*** dustins has quit IRC21:27
*** pcaruana has quit IRC21:38
openstackgerritSean McGinnis proposed openstack/cinder master: Add api-ref for mv 3.55 volume transfer  https://review.openstack.org/58866621:40
*** gouthamr has joined #openstack-cinder21:47
*** KeithMnemonic has quit IRC21:53
openstackgerritSean McGinnis proposed openstack/python-cinderclient master: Fix backwards compat for volume transfer < 3.55  https://review.openstack.org/58867322:05
*** Guest43093 is now known as amrith22:12
*** ganso has quit IRC22:18
openstackgerritSean McGinnis proposed openstack/cinder master: Add api-ref for mv 3.55 volume transfer  https://review.openstack.org/58866622:36
*** nicolasbock has quit IRC22:43
*** tpsilva has quit IRC22:54
openstackgerritSean McGinnis proposed openstack/cinder master: Correct volume-transfers API endpoint  https://review.openstack.org/58866422:56
*** harlowja has quit IRC23:48
*** tswanson has joined #openstack-cinder23:48

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