Tuesday, 2018-11-20

*** hoonetorg has quit IRC00:35
*** hoonetorg has joined #openstack-manila00:37
*** hoonetorg has quit IRC00:40
*** hoonetorg has joined #openstack-manila00:40
*** markstur has quit IRC02:38
openstackgerritMerged openstack/manila master: [LVM][IPv6] Quagga changes to support Bionic Beaver  https://review.openstack.org/61480203:27
*** markstur has joined #openstack-manila04:08
*** markstur has quit IRC06:03
*** pcaruana has joined #openstack-manila07:20
*** pcaruana has quit IRC07:34
*** pcaruana has joined #openstack-manila07:40
*** e0ne has joined #openstack-manila09:24
*** a-pugachev has joined #openstack-manila09:34
*** a-pugachev has quit IRC09:34
*** a-pugachev has joined #openstack-manila09:39
*** ganso has joined #openstack-manila09:39
openstackgerritRodrigo Barbieri proposed openstack/manila-specs master: Add spec for Manage-Unmanage of Share Servers  https://review.openstack.org/60734210:19
openstackgerritMaurice Schreiber proposed openstack/manila master: [api-ref] Added share servers show and corrected path to details  https://review.openstack.org/60717010:34
*** carthaca has joined #openstack-manila10:35
openstackgerrithulina proposed openstack/manila master: check all_tenants value in share_networks api  https://review.openstack.org/61899111:51
openstackgerritHelen Walsh proposed openstack/manila master: VMAX manila doc - support for IPv6  https://review.openstack.org/61245712:18
openstackgerritMerged openstack/manila master: NeutronBindNetworkPlugin: fix multi segment neutron data save.  https://review.openstack.org/59311112:41
openstackgerritMerged openstack/manila master: [CI][LVM] Run the LVM job on Bionic Beaver  https://review.openstack.org/60876114:12
*** mmethot has quit IRC14:13
*** mmethot has joined #openstack-manila14:17
*** dustins has joined #openstack-manila14:30
*** carthaca has quit IRC15:05
*** carthaca has joined #openstack-manila15:07
*** carthaca2 has joined #openstack-manila15:07
*** carthaca2 has quit IRC15:07
*** e0ne has quit IRC16:21
openstackgerritRodrigo Barbieri proposed openstack/manila-specs master: Create share from snapshots in another pool or back end  https://review.openstack.org/60953716:30
*** luizbag has joined #openstack-manila16:39
*** markstur has joined #openstack-manila16:54
*** a-pugachev_ has joined #openstack-manila17:00
gansogouthamr: ping17:03
*** a-pugachev has quit IRC17:04
*** a-pugachev_ is now known as a-pugachev17:04
*** a-pugachev has quit IRC17:17
*** e0ne has joined #openstack-manila17:31
*** luizbag has quit IRC17:36
*** e0ne has quit IRC17:44
*** e0ne has joined #openstack-manila17:57
*** markstur has quit IRC18:17
*** pcaruana has quit IRC18:47
gouthamrganso: pong18:56
gansogouthamr: hey Goutham!18:57
gansogouthamr: I updated my specs, hopefully adressing your feedback18:57
*** markstur has joined #openstack-manila18:57
gansogouthamr: s/feedback/concerns18:57
gansogouthamr: I am writing the "share server with multiple subnets" spec now18:57
gouthamrganso: ty, i'll take a look sometime today - are we pushing any of these to "T"? or are you writing a different one?18:57
gansogouthamr: when I finish and upload it, and I'll back to reviewing your specs18:57
gansogouthamr: the only "T" spec I am proposing is "share server with multiple subnets"18:58
gouthamrganso: okay, bswartz suggested a call to discuss these concerns if we're unable to sort this out in review18:59
gansogouthamr: bswartz will return on monday I suppose18:59
gansogouthamr: let's try to sort them out sooner if possible19:00
gouthamrganso: ack, yeah.. attendance here will be dwindling everyday this week19:00
gouthamrganso: i don't think you answered my concerns about "Create share from snapshots in another pool or back end"19:05
gouthamrganso: I wasn't suggesting the use of the data-service as an alternative.. "However, any driver-specific optimizations that could allow the operation to be performed more efficiently between compatible drivers would not be able to be used." - I was suggesting its use as a supplement to fully resolve this issue..19:06
gouthamrganso: my concern is the complicated scheduling logic that seems almost like a one-off for this situation19:07
gansogouthamr: what about it is complicated?19:10
gouthamrganso: you're adding a post-weigher for starters.. which limits the way weighers currently work19:12
gouthamrganso: you should be able to add multiple weighers that can work in any order, and if the admin wanted, they could skew the weighing by doing what bswartz proposed with his stochastic weigher19:13
gansogouthamr: this behavior would only be intended for a "create_share_from_snapshot" method in the scheduler. The logic will be split from "create_share"19:14
gouthamrganso: yes, i get that, that's one of the reasons i think this can be done in a more simple/straightforward manner, generically.19:18
gansogouthamr: I don't think we should expose to users anything that could control this. Like a "--take-your-time" parameter or something like that. I believe it is something that should be smart in our end.19:20
gouthamrganso: the spec says that administrators also need to tell us what the spread ratio is (backend_share_spread_required_discrepancy) - how would we expect them to know?19:20
gouthamrganso: yes! it's unfortunate that we'd be exposing limits within a cloud, but as a user, i might expect all clones to be instantaneous19:21
gouthamrganso: and with the proposal, some clones will be instantaneous and some others could take a long time19:22
gouthamrganso: and there's no way for the end user/application to have any expectations19:22
*** e0ne has quit IRC19:22
gansogouthamr: as the spec says, the administrator may not want to have all shares spread out to not lose so much COW efficiency, so it is something the administrator would need to tweak, like the over_provisoning_ratio... It does not really solve the use case of "I want to use this snapshot as an image so for this one I always want to spread out" as this is user-facing and the user shouldn't be concerned about that. For19:23
gansothat use case in particular (which my spec is not focused on), it may be better to use metadata19:23
gansogouthamr: yes, there isn't, that's why at the default discrepancy value the source pool will be preferred. And if it is to be created in another AZ (which is one important use cases to address IMO), then it is completely understandable to take longer, as the user will have specified an additional parameter that results in that19:25
gouthamrthe user won't be thinking about spreading out their data - imo, they'd just want a "clone" and if the cloud can't do it efficiently, to let them know so - it is like level setting their expectations19:26
gouthamrganso: yes, the AZ issue is currently a bug that we must fix19:27
gansogouthamr: so in your proposal, when specifying a different AZ, the user would always have to specify "--take-your-time" ?19:28
gansogouthamr: or else it will always result in the "this cannot be done efficiently, please specify --take-your-time" error, correct?19:28
gouthamrganso: yes19:28
gansogouthamr: ok, so the simpler approach I was thinking about before tbarron's feedback (you can check the earlier patchsets), I was thinking about always using the source pool unless a different AZ is specified. What do you think of that?19:29
gouthamrganso: how about different backends in the same AZ?19:30
gouthamrs/how/what19:30
gansogouthamr: in the example above it would always stick to the source pool if it is to be created in the same AZ19:31
*** a-pugachev has joined #openstack-manila19:32
gouthamrganso: backtracking, theoretically the fastest would be to create the clone in the same pool,  then across pools in the same backend, and then among homogeneous backends in the same AZ and then among homogenous backends in different AZs, and then heterogenous backends in the same AZ and finally heterogenous backends in different AZs19:34
gouthamrif manila were to actively consider the differences between homogenous and heterogenous backends, you'll evolve a replication design19:35
gansogouthamr: yup that was my original proposal, considering "homogenous" only as "compatible". "heterogenous" could be accomplished with the data service19:35
gouthamrokay, i didn't read your spec back then, did you and bswartz discuss this offline?19:37
gouthamrhe had similar concerns as i did, and was suggesting a call to give his opinion19:37
gansogouthamr: yes, but bswartz really wanted a user-facing parameter like "--spread-please", even though users shouldn't know about this stuff. That is his first comment in gerrit19:38
gouthamrganso: yeah, my concern was "time" more than spread19:39
gansogouthamr: probably his suggestion changed a bit after I changed to include the discrepancy config option, as that was my approach for spreading, but then his following comment was about the snapshot as an image use case. Which I believe should not be within the scope of the spec19:39
*** a-pugachev has quit IRC19:39
gansogouthamr: my original concern was as well. tbarron was concerned about having a configurable spread factor19:39
gouthamrganso: i was thinking we can evolve this further by caching snapshots in multiple hosts19:39
gansogouthamr: just to solve that use case?19:41
gouthamrganso: nope, building this out in phases19:41
gansogouthamr: I mean, as a different spec, perfect, in another phase, but unrelated to the current proposal19:41
gansogouthamr: yep19:41
gouthamrganso: can you schedule a call on monday with tbarron and bswartz and anyone else who would like to discuss19:43
gansogouthamr: will tbarron be back from PTO?19:43
gouthamrganso: lm check19:43
gouthamrganso: he's back on 26th/Monday19:45
gansogouthamr: ok perfect, I'll schedule the meeting19:46
gouthamrganso: good stuff19:46
gansogouthamr: are you going to be here from tomorrow to friday?19:46
gouthamrganso: sparingly, tomorrow19:46
gansogouthamr: ok19:46
gouthamrganso: i'm here today... hey, i had a TripleO question for you19:46
gansogouthamr: go ahead19:47
gouthamrganso: did you guys certify manila/ontap/dhss=True on OSP13?19:47
gansogouthamr: yes19:47
gouthamrganso: okay, did you have any deployment issues, wrt neutron?19:48
gansogouthamr: hmmm not really. We configured it like our CI is configured in MSVM jobs19:48
gansogouthamr: since it doesn't run scenario tests, we didn't face problems19:49
gouthamrganso: did you use the heat templates or manually configure on a deployed server?19:49
gansogouthamr: heat templates19:49
gouthamrganso: okay, ty.. i wonder what stage this got introduced: https://bugs.launchpad.net/puppet-manila/+bug/180239319:49
openstackLaunchpad bug 1802393 in puppet-manila "THT deploys manila.conf with deprecated settings for neutron and nova" [Undecided,In progress] - Assigned to David Vallee Delisle (valleedelisle)19:49
gansogouthamr: oh, I remember now, we may have used the standalone plugin. I don't remember for sure19:50
gouthamrganso: ouch19:50
gouthamr:)19:50
gansogouthamr: I didn't have that keystone problem19:50
gouthamrganso: you wouldn't, if you used standalone19:50
gansogouthamr: oooooooooohhhhhhh I remember the problem now19:51
gansogouthamr: we had the network ccleanup problem19:51
gansogouthamr: because of dynamic credentials19:51
gouthamrganso: sure, but you'd've worked around that with using pre-provisioned creds?19:51
gansogouthamr: if I recall correctly, I had to create manual credentials to disable dynamic creds19:52
gansogouthamr: yes19:52
gansogouthamr: then, that was one of the problems solved. So I don't remember if I really ended up using standalone or not. I tried many things, and in the end I may have reverted to not using standalone after I solved the dynamic creds problem19:52
gansogouthamr: if you have access to the certification data we submitted, you could check the configuration19:53
gouthamrganso: okay.. the problem in LP 1802393 is manila interacting with nova and neutron19:54
openstackLaunchpad bug 1802393 in puppet-manila "THT deploys manila.conf with deprecated settings for neutron and nova" [Undecided,In progress] https://launchpad.net/bugs/1802393 - Assigned to David Vallee Delisle (valleedelisle)19:54
gansogouthamr: if not I can check it for you tomorrow19:54
gansogouthamr: I am running late now, gotta run19:54
gouthamrganso: i don't have access to it, but would like to see the templates used if you do :)19:54
gouthamrganso: yep.. no rush, tty tomorrow19:54
*** a-pugachev has joined #openstack-manila20:18
*** ganso has quit IRC22:29
*** markstur has quit IRC23:16
*** dustins has quit IRC23:19

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