Monday, 2018-09-17

*** dpawlik has quit IRC00:01
*** brinzhang has joined #openstack-cinder00:03
*** openstackgerrit has quit IRC00:09
*** brinzh has joined #openstack-cinder00:09
*** brinzh has quit IRC00:12
*** ircuser-1 has joined #openstack-cinder00:53
*** openstackgerrit has joined #openstack-cinder01:08
openstackgerritLiang Fang proposed openstack/cinder master: Specify image size to avoid glance backend resize when uploading volume  https://review.openstack.org/60257101:08
*** zhaochao has joined #openstack-cinder01:20
*** mschuppert has quit IRC01:44
*** jmccrory has quit IRC01:54
*** jmccrory has joined #openstack-cinder01:57
openstackgerritChuck Short proposed openstack/cinder master: Move hscli to privsep  https://review.openstack.org/60300402:03
*** lbragstad has joined #openstack-cinder02:08
*** e0ne has joined #openstack-cinder02:33
*** lbragstad has quit IRC02:36
*** dave-mccowan has quit IRC02:37
*** lbragstad has joined #openstack-cinder02:41
*** brinzh has joined #openstack-cinder03:03
*** brinzhang has quit IRC03:07
*** vivsoni has joined #openstack-cinder03:12
*** e0ne has quit IRC03:21
*** e0ne has joined #openstack-cinder03:54
*** dpawlik has joined #openstack-cinder03:57
*** pooja_jadhav has joined #openstack-cinder04:00
*** dpawlik has quit IRC04:02
*** jiaopengju has quit IRC04:38
*** jiaopengju has joined #openstack-cinder04:39
*** sapd1_ has quit IRC04:44
*** sapd1__ has joined #openstack-cinder04:44
*** e0ne has quit IRC04:55
*** whoami-rajat has joined #openstack-cinder05:06
vivsonismcginnis: need workflow approval for https://review.openstack.org/#/c/599478/05:12
vivsonican you please approve05:12
openstackgerritVivek Soni proposed openstack/cinder master: 3PAR: Update Storage Driver docs  https://review.openstack.org/60252305:15
openstackgerritRajat Dhasmana proposed openstack/cinder master: Replace VolumeAttachStatus strings with enum fields  https://review.openstack.org/59968605:35
*** e0ne has joined #openstack-cinder05:42
*** sapd1__ has quit IRC05:50
*** sapd1 has joined #openstack-cinder05:51
*** belmoreira has joined #openstack-cinder05:55
*** Luzi has joined #openstack-cinder06:00
*** dpawlik has joined #openstack-cinder06:07
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata  https://review.openstack.org/60302406:29
openstackgerritLiang Fang proposed openstack/cinder master: Specify image size to avoid glance backend resize when uploading volume  https://review.openstack.org/60257106:31
*** vivsoni has quit IRC06:48
*** e0ne_ has joined #openstack-cinder06:50
*** e0ne has quit IRC06:53
*** e0ne_ has quit IRC06:54
*** mdih has joined #openstack-cinder07:06
mdihhi there, if using ceph as the storage cluster for cinder, would there be a flag or option to freeup unused blocks? seems like fstrim doesn't work07:10
*** rcernin has quit IRC07:11
mdihoh okay, found a link sorry for my previous question. seems like discard support has not been supported in cinder yet. or at least as of 2015. but is it supported now?07:45
mdihugh, nevermind, okay it has been supported already, at least in mitaka..sorry for my question hehe..thanks07:52
*** lpetrut has joined #openstack-cinder07:52
*** vivsoni has joined #openstack-cinder08:03
*** vivsoni_ has joined #openstack-cinder08:03
*** plestang has quit IRC08:08
tommylikehuhey geguileo, do you mind to share the discussion and next step for the placement stuff for cinder?08:25
tommylikehuthere should be a topic on Friday with Nova guy, but I didn't get a chance for that:)08:27
geguileotommylikehu: Hi, we had a summary session on the Cinder room at some point, I think it's summarized there08:40
geguileotommylikehu: but there was no Friday meeting in the end08:40
geguileotommylikehu: Jay was going to write an email explaining what they did08:40
geguileoI'm still checking emails, so I don't know if he sent it or not08:40
tommylikehugeguileo:  ok, thanks, let's check jungleboyj's email:)08:41
geguileotommylikehu: The email will be from Jay Pipes08:47
tommylikehugeguileo: ooh, misunderstood:), so basically you will handle this in Stein?08:48
geguileotommylikehu: Since nobody else wanted to work on this, it fell on me  XD08:48
*** mschuppert has joined #openstack-cinder09:03
*** vivsoni_ has quit IRC09:10
*** lpetrut has quit IRC09:59
*** brinzh has quit IRC10:03
*** alexchadin has joined #openstack-cinder10:10
*** belmoreira has quit IRC10:22
*** finucannot is now known as stephenfin10:34
tommylikehugeguileo:  we can do some help if that helps for the merge of feature by the end of Stein:)10:38
*** alexchadin has quit IRC10:43
openstackgerritTommyLike proposed openstack/cinder master: [WIP] Support transfer pagination  https://review.openstack.org/60218110:53
*** alexchadin has joined #openstack-cinder10:53
tommylikehueharney, geguileo, updated the patch according to the PTG discussion:) https://review.openstack.org/#/c/593556/10:55
*** luizbag has joined #openstack-cinder10:55
*** ganso has joined #openstack-cinder11:07
*** imacdonn has quit IRC11:18
*** imacdonn has joined #openstack-cinder11:19
*** belmoreira has joined #openstack-cinder11:23
*** erlon has quit IRC11:36
*** psachin has joined #openstack-cinder11:43
*** alexchadin has quit IRC11:48
*** zul has joined #openstack-cinder11:48
openstackgerritGorka Eguileor proposed openstack/os-brick stable/rocky: Improve detection of multipathd running  https://review.openstack.org/60310811:52
*** dave-mccowan has joined #openstack-cinder11:54
*** alexchadin has joined #openstack-cinder12:05
*** alexchadin has quit IRC12:06
*** alexchadin has joined #openstack-cinder12:07
jungleboyjtommylikehu: and geguileo  I had a chat with Jay Pipes after the Crosss Project session and answered a number of questions.  I am thinking he got the info he needed then and that is why we didn't have the follow up meeting.12:11
jungleboyjtommylikehu: geguileo  Basically he is proposing that we set up a field in the database to increment a generation per pool and keep the pool information there.  When an update comes in, if it changes the pool information changes then the generation is updated.12:12
jungleboyjtommylikehu: geguileo  If the generation changes between when the data is looked up and when the process tries to save data then the scheduler will have to get the latest information and make the scheduler request again.12:14
geguileojungleboyj: oh, is that all???  then it's something similar to what I do with the workers table iirc12:14
jungleboyjgeguileo:  Yeah, it really didn't sound that bad from what he said.12:15
jungleboyjgeguileo:  I would watch for an e-mail from him to make sure I am right or send something out as follow up.12:15
geguileojungleboyj: I'll try to find the code in the placement project and see if I understand it12:15
jungleboyjIt sounded like he was still planning to send it.12:15
geguileojungleboyj: great  :-)12:15
jungleboyjgeguileo:  :-)  Let me know if there is anything else I can do to help.12:16
openstackgerritJay Rubenstein proposed openstack/cinder master: SF ensure the correct volume is deleted  https://review.openstack.org/60311312:16
geguileojungleboyj: thanks12:16
jungleboyjgeguileo:  Take it that you made it home safely?12:16
*** AJaeger has left #openstack-cinder12:16
geguileojungleboyj: I did, but I'm still pretty much out of it...  %-D12:16
* geguileo is getting old...12:17
jungleboyj:-)  I was doing ok yesterday but feeling very tired today.12:17
*** Roamer` has joined #openstack-cinder12:18
*** tpsilva has joined #openstack-cinder12:24
*** prashkre has joined #openstack-cinder12:25
*** erlon has joined #openstack-cinder12:39
*** prashkre has quit IRC12:40
*** lseki has joined #openstack-cinder12:43
openstackgerritMerged openstack/cinder master: Imported Translations from Zanata  https://review.openstack.org/60302412:45
*** dustins has joined #openstack-cinder13:04
openstackgerritChuck Short proposed openstack/cinder master: Move hscli to privsep  https://review.openstack.org/60300413:10
*** alexchadin has quit IRC13:14
openstackgerritChuck Short proposed openstack/cinder master: Remove sg_scan  https://review.openstack.org/60312413:17
*** mriedem has joined #openstack-cinder13:18
*** alexchadin has joined #openstack-cinder13:29
*** psachin has quit IRC13:29
*** zhaochao has quit IRC13:30
*** Luzi has quit IRC13:32
*** belmorei_ has joined #openstack-cinder13:33
*** belmoreira has quit IRC13:33
*** alexchadin has quit IRC13:38
*** alexchadin has joined #openstack-cinder13:47
*** alexchadin has quit IRC13:56
*** gnufied has joined #openstack-cinder14:00
*** alexchadin has joined #openstack-cinder14:01
*** KeithMnemonic has joined #openstack-cinder14:03
KeithMnemonicjungleboyj: could you or someone else with WF+1 review this when you het a minute https://review.openstack.org/60052814:13
openstackgerritAlan Bishop proposed openstack/cinder master: Fix image volume cache max size and max count limits  https://review.openstack.org/60314514:15
*** e0ne has joined #openstack-cinder14:30
*** e0ne has quit IRC14:38
*** belmorei_ has quit IRC14:43
*** vivsoni has quit IRC14:44
jungleboyjKeithMnemonic:  Done.14:44
*** vivsoni has joined #openstack-cinder14:44
*** belmoreira has joined #openstack-cinder14:45
KeithMnemonicthanks!14:46
jungleboyjNP.14:46
smcginnisjungleboyj: I'm going to send something to the ML asking infra to delete the driverfixes/ocata branch now that we can use stable/ocata.14:48
smcginnisjungleboyj: Any objection to doing that now?14:48
jungleboyjsmcginnis:  No, that is a good idea.  Avoid confusion there.14:49
smcginnisWe synced all patches, so I want to make sure we don't accidentally push something into driverfixes when it should go to stable.14:49
smcginnisjungleboyj: Cool, thanks.14:49
jungleboyjYeah.  Thank you for working on making that all happen.14:49
smcginnisNo problem14:50
*** mdih has quit IRC15:01
*** alexchadin has quit IRC15:08
*** ganso has quit IRC15:09
*** alexchadin has joined #openstack-cinder15:18
*** alexchadin has quit IRC15:23
*** _alastor_ has joined #openstack-cinder15:26
*** trident has quit IRC15:38
*** trident has joined #openstack-cinder15:39
*** alexchadin has joined #openstack-cinder15:40
*** alexchadin has quit IRC15:44
*** dklyle has quit IRC15:49
*** dklyle has joined #openstack-cinder15:50
*** KeithMnemonic has quit IRC15:52
*** belmoreira has quit IRC15:55
*** icey has quit IRC15:56
*** icey has joined #openstack-cinder15:57
*** _alastor_ has quit IRC16:03
*** _alastor_ has joined #openstack-cinder16:11
*** psachin has joined #openstack-cinder16:28
*** psachin has quit IRC16:43
*** psachin has joined #openstack-cinder16:48
openstackgerritMerged openstack/cinder stable/pike: VMware: Optimize volume creation from image  https://review.openstack.org/60052816:53
*** prashkre has joined #openstack-cinder16:59
*** viveksoni has joined #openstack-cinder17:15
*** yumiriam has joined #openstack-cinder17:25
*** yumiriam has quit IRC17:26
*** yumiriam has joined #openstack-cinder17:27
viveksoniHi Team, Please review https://review.openstack.org/#/c/599478/17:35
viveksonineeds workflow approval17:35
viveksoniThanks jungleboyj17:36
openstackgerritChuck Short proposed openstack/cinder master: Remove sg_scan  https://review.openstack.org/60312417:38
*** psachin has quit IRC17:40
openstackgerritJay Bryant proposed openstack/os-brick master: Ignore volume disconnect if it is not connected  https://review.openstack.org/60235017:41
*** tbarron_ is now known as tbarron17:41
openstackgerritiain MacDonnell proposed openstack/cinder master: ZFSSA volume driver REST client python3 fixes  https://review.openstack.org/60319417:48
jungleboyjgeguileo: or jbernard  Since Eric is out do you guys know anything about Andreas' question about this failing from last night?  http://zuul.openstack.org/builds.html?job_name=cinder-tempest-plugin-lvm-lio17:50
smcginnisThat's been failing for some time. I think eharney was looking into it, but I don't know the current status or if anything was found.17:52
jungleboyjsmcginnis:  Ok.  If it has been failing for a while it can wait until Eric gets back.17:52
smcginnisStill would be good to have it working. ;)17:53
geguileojungleboyj: I think this is the Swift credentials issue Eric mentioned on the PTG...17:53
smcginnisAh, that sounds familiar.17:53
smcginnisI think Eric might have been looking at a devstack change to fix this.17:53
jungleboyjHmmm, don't remember that one.17:54
geguileohttp://logs.openstack.org/99/601099/1/check/cinder-tempest-plugin-lvm-lio/839431d/controller/logs/screen-c-bak.txt.gz?level=ERROR17:54
jungleboyjsmcginnis:  Oh, I do remember him saying something about needing to change devstack.17:54
*** david-lyle has joined #openstack-cinder17:54
geguileoLooks like a credentials issue17:54
geguileoHe was thinking about looking into it17:55
geguileoor he was trying to get me curious enough about it to have a look17:55
geguileoone or the other   XD17:55
jungleboyjHe he he.17:55
smcginnisIIRC, admin credentials are set up, but this doesn't call swift as admin. Something along those lines.17:56
*** dklyle has quit IRC17:57
*** mszwed has quit IRC17:57
geguileonow that you say it, I think that was Eric's working theory...17:57
openstackgerritMerged openstack/cinder-specs master: Fix code-block language  https://review.openstack.org/57548118:04
*** openstackgerrit has quit IRC18:10
*** openstackgerrit has joined #openstack-cinder18:10
openstackgerritKumar Prashant proposed openstack/cinder master: VMAX Driver - Rollback for manage existing volume  https://review.openstack.org/60320218:10
*** jiapei has joined #openstack-cinder18:11
*** aspiers has quit IRC18:13
*** ganso has joined #openstack-cinder18:22
*** viveksoni has quit IRC18:49
*** hemna_ has quit IRC18:55
*** hemna_ has joined #openstack-cinder18:57
*** alexchadin has joined #openstack-cinder18:59
*** aspiers has joined #openstack-cinder19:06
*** zul has quit IRC19:08
*** raunakkumar has joined #openstack-cinder19:19
raunakkumarcore reviewers could you please review https://review.openstack.org/#/c/601492/ ?19:20
openstackgerritMerged openstack/cinder master: Move hscli to privsep  https://review.openstack.org/60300419:24
openstackgerritMerged openstack/cinder master: 3PAR: Update Storage Driver docs  https://review.openstack.org/60252319:24
openstackgerritMerged openstack/cinder stable/rocky: 3PAR: Added retries on volume deletion  https://review.openstack.org/59947819:24
*** zul has joined #openstack-cinder19:25
openstackgerritMerged openstack/os-brick stable/rocky: Improve detection of multipathd running  https://review.openstack.org/60310819:31
*** alexchadin has quit IRC19:32
*** raunakkumar has quit IRC19:36
*** aspiers has quit IRC19:48
*** KeithMnemonic has joined #openstack-cinder20:01
*** yumiriam has left #openstack-cinder20:05
openstackgerritDhinesh Balasubramaniam proposed openstack/cinder master: Hedvig Cinder driver implementation  https://review.openstack.org/43334120:09
*** aspiers has joined #openstack-cinder20:09
*** prashkre has quit IRC20:10
*** prashkre has joined #openstack-cinder20:10
*** luizbag has quit IRC20:24
*** lbragstad has quit IRC20:26
*** lbragstad has joined #openstack-cinder20:27
*** zul has quit IRC20:32
openstackgerritMerged openstack/cinder stable/queens: Rename devstack-plugin-ceph jobs  https://review.openstack.org/60135520:34
openstackgerritMerged openstack/cinder stable/ocata: Rename devstack-plugin-ceph jobs  https://review.openstack.org/60135820:34
openstackgerritMerged openstack/cinder stable/pike: Rename devstack-plugin-ceph jobs  https://review.openstack.org/60135620:34
KeithMnemonicReviewers Please review when you get a minute https://review.openstack.org/#/c/600538/20:38
imacdonnanyone else able to access this? https://etherpad.openstack.org/p/cinder-ptg-stein-thursday20:50
imacdonnI can get Wednesday and Friday, but not Thursday :/20:51
smcginnisStuck on "Loading..." for me.20:51
imacdonnyeah, that's what I'm getting20:51
smcginnisMust be everyone doing their PTG recaps. ;)20:51
imacdonnI was looking for the discussion about cascading deletes20:52
imacdonnwas that Thurs ?20:52
smcginnisI missed the morning, so maybe it was then.20:52
imacdonnI think you were there (or at least here)20:53
imacdonntalking about deleting a snapshot that has dependent child volumes20:53
smcginnisI don't recall that being discussed again. I thought we took care of that a cycle or two ago.20:53
smcginnisOh!20:54
smcginnisYeah, I think I was there. :)20:54
imacdonnheh20:54
smcginnisThat some drivers need to internally do a clone instead so that the snapshot can actually be deleted before its dependent volume.20:55
smcginnisNow I remember.20:55
imacdonnI was thinking "one of us must be losing it" :)20:55
smcginnisThat would be me. ;)20:55
imacdonnso yeah ... I was reviewing the skipping of tests for ZFSSA CI (as promised)20:55
imacdonnI believe I'm down to two, and they're both due to that issue20:55
smcginnisI'm really not sure how we should proceed on that.20:55
smcginnisIt's an incredible speed advantage for many backends that creating a volume from a snapshot is almost instantaneous,20:56
smcginnisWhile creating a clone will be a much longer operation.20:56
imacdonnyes ... that's one of our main reasons for booting from volume at all20:56
smcginnisYeah, that's a big one. I know when I worked on Dell SC, boot from cached image was really quick because of the ability to just instantly have a boot volume.20:57
imacdonnright ... combined with our obnoxiously large images ("our" => my day job), it makes a big difference20:58
smcginnisI think I may have had some experience with your obnoxiously large images. ;)21:03
*** raunakkumar has joined #openstack-cinder21:04
*** prashkre has quit IRC21:06
imacdonnhmmm ok :)21:06
imacdonnso yeah, we'll have to figure out what to do about that .. meanwhile I'll have at least these two tests disabled21:09
imacdonnthis is the obvious one: tempest.api.volume.test_volume_delete_cascade.VolumesDeleteCascade.test_volume_from_snapshot_cascade_delete21:10
imacdonnthis one also fails: tempest.scenario.test_volume_boot_pattern.TestVolumeBootPattern.test_create_server_from_volume_snapshot21:10
imacdonnit appears that it's trying to delete the snapshot before nova is done deleting the child volume21:10
smcginnisThat last one might require an update to the tempest test to actually wait for the deletion to finish. There have been other cases of that.21:12
imacdonnyeah. I'm not sure where the deletion is happening, actually ... it must be some sort of automatic cleanup21:12
imacdonntest is https://github.com/openstack/tempest/blob/master/tempest/scenario/test_volume_boot_pattern.py#L167-L20421:14
imacdonnbut it doesn't explicitly delete anything (AFAICT)21:14
smcginnisYeah, probably in the cleanup handling.21:14
*** erlon has quit IRC21:18
*** gnufied has quit IRC21:21
*** lseki has quit IRC21:24
openstackgerritChuck Short proposed openstack/cinder master: Remove aoe-revalidate, aoe-discover, aoe-flush  https://review.openstack.org/60323121:24
*** gnufied has joined #openstack-cinder21:38
*** hemna_ has quit IRC21:51
*** rcernin has joined #openstack-cinder22:08
*** dustins has quit IRC22:33
*** raunakkumar has quit IRC22:41
*** mriedem is now known as mriedem_away23:23
*** jiapei has quit IRC23:30

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