Thursday, 2016-11-10

*** akapil has quit IRC00:03
*** StraubTW has quit IRC00:13
*** chlong has joined #openstack-manila00:23
openstackgerritzhongshengping proposed openstack/puppet-manila: Fix common options position and remove unused parameters  https://review.openstack.org/39470400:35
openstackgerritzhongshengping proposed openstack/manila: Update method cfg.set_defaults  https://review.openstack.org/39587600:39
TommyLikeHuping gouthamr00:46
openstackgerritzhongjun proposed openstack/manila-specs: Manila db purge utility  https://review.openstack.org/39240601:02
openstackgerritzhongjun proposed openstack/manila-specs: Manila db purge utility  https://review.openstack.org/39240601:05
*** StraubTW has joined #openstack-manila01:14
*** vbellur has joined #openstack-manila01:15
*** StraubTW has quit IRC01:19
*** gouthamr has quit IRC01:21
*** gouthamr has joined #openstack-manila01:22
*** tuanluong has joined #openstack-manila01:30
*** gouthamr has quit IRC01:34
*** breitz has quit IRC02:05
*** gcb has joined #openstack-manila02:16
*** houming has joined #openstack-manila02:35
*** StraubTW has joined #openstack-manila02:54
*** StraubTW has quit IRC04:06
*** rhagarty has quit IRC04:13
*** chlong has quit IRC04:13
*** dsariel has joined #openstack-manila04:54
*** akapil has joined #openstack-manila04:59
*** akapil has quit IRC05:04
*** sticker has quit IRC05:29
*** sandanar has joined #openstack-manila05:44
*** lpetrut has joined #openstack-manila06:04
*** akapil has joined #openstack-manila06:07
*** StraubTW has joined #openstack-manila06:07
openstackgerritDeepak proposed openstack/python-manilaclient: Remove vestiges of oslo-incubator  https://review.openstack.org/39596106:09
*** StraubTW has quit IRC06:12
*** harlowja_ has quit IRC06:33
*** tinwood has quit IRC06:41
*** tinwood has joined #openstack-manila06:42
*** kamtamtun has joined #openstack-manila06:42
*** lpetrut has quit IRC06:45
*** pcaruana has joined #openstack-manila06:51
*** chlong has joined #openstack-manila06:58
*** chlong has quit IRC07:09
*** akapil has quit IRC07:11
*** lpetrut has joined #openstack-manila07:17
*** belmoreira has joined #openstack-manila07:26
*** akapil has joined #openstack-manila07:29
*** lpetrut has quit IRC07:30
*** openstackgerrit has quit IRC07:48
*** openstackgerrit has joined #openstack-manila07:49
*** jprovazn has joined #openstack-manila07:55
*** akapil has quit IRC08:06
*** akapil has joined #openstack-manila08:09
*** akapil has quit IRC08:31
*** lpetrut has joined #openstack-manila08:45
*** akapil has joined #openstack-manila08:47
openstackgerritPony Chou proposed openstack/manila: Add QNAP Manila Driver  https://review.openstack.org/39470308:53
*** akapil has quit IRC09:06
*** akapil has joined #openstack-manila09:07
*** akapil has quit IRC09:11
*** lpetrut has quit IRC09:34
*** lpetrut has joined #openstack-manila09:41
*** belmoreira has quit IRC09:46
*** pcaruana has quit IRC09:49
*** akapil has joined #openstack-manila09:55
*** pcaruana has joined #openstack-manila10:01
*** akapil has quit IRC10:01
openstackgerritPony Chou proposed openstack/manila: Add QNAP Manila Driver  https://review.openstack.org/39470310:04
openstackgerritzhongjun proposed openstack/manila-specs: Add spec for enable IPv6 in manila  https://review.openstack.org/36278610:04
*** akapil has joined #openstack-manila10:05
*** akapil has quit IRC10:06
*** StraubTW has joined #openstack-manila10:09
openstackgerritDaniel Mellado proposed openstack/manila: Duplicate remote_client into Manila  https://review.openstack.org/39606710:13
*** StraubTW has quit IRC10:15
*** akapil has joined #openstack-manila10:32
TommyLikeHuanyone would like to review this feature specs: https://review.openstack.org/#/c/392406/  and  https://review.openstack.org/#/c/390052/10:44
*** ganso has joined #openstack-manila10:48
*** lpetrut has quit IRC10:50
openstackgerritMarc Koderer proposed openstack/python-manilaclient: Fix misleading help text  https://review.openstack.org/39610010:51
openstackgerritCedric Zhuang proposed openstack/manila: Add "update_access" interface support for VNX.  https://review.openstack.org/39540410:56
openstackgerritJan Provaznik proposed openstack/manila: Check ceph backend connection on driver setup  https://review.openstack.org/39496110:56
openstackgerritJan Provaznik proposed openstack/manila: Check ceph backend connection on driver setup  https://review.openstack.org/39496110:58
*** akapil has quit IRC11:15
*** houming has quit IRC11:17
*** akapil has joined #openstack-manila11:17
*** tpsilva has joined #openstack-manila11:35
openstackgerritTiago Pasqualini da Silva proposed openstack/manila: Fix share manage tempest test cleanup  https://review.openstack.org/39438811:37
*** lpetrut has joined #openstack-manila11:38
*** chlong has joined #openstack-manila11:43
*** chlong has quit IRC11:44
*** akapil has quit IRC11:54
*** akapil has joined #openstack-manila11:54
*** mkoderer has joined #openstack-manila11:54
*** akapil has quit IRC11:57
*** jcsp has quit IRC12:04
openstackgerritDaniel Mellado proposed openstack/manila: Add testscenario to test-requirements  https://review.openstack.org/39614612:04
*** akapil has joined #openstack-manila12:06
*** TommyLikeHu_ has joined #openstack-manila12:14
*** TommyLikeHu has quit IRC12:17
*** akapil has quit IRC12:27
*** akapil has joined #openstack-manila12:27
*** akapil has quit IRC12:28
*** tuanluong has quit IRC12:43
*** sandanar has quit IRC12:47
*** makowals has quit IRC12:48
*** makowals has joined #openstack-manila12:49
*** gcb has quit IRC13:02
*** catintheroof has joined #openstack-manila13:08
openstackgerritMerged openstack/puppet-manila: Deprecate rabbitmq connection parameters  https://review.openstack.org/39515113:14
*** jprovazn has quit IRC13:42
TommyLikeHu_tommyhu321450013:45
TommyLikeHu_.....13:45
*** kamtamtun has quit IRC13:45
*** akerr has joined #openstack-manila13:49
openstackgerritMauricio Lima proposed openstack/manila-specs: Spec for openstack client support  https://review.openstack.org/39577513:53
*** StraubTW has joined #openstack-manila14:09
openstackgerritTom Barron proposed openstack/manila: DO NOT MERGE  https://review.openstack.org/35260614:12
*** vbellur has quit IRC14:12
*** eharney has quit IRC14:19
openstackgerritValeriy Ponomaryov proposed openstack/manila-specs: Add spec for Manila share groups  https://review.openstack.org/31573014:21
*** cknight has joined #openstack-manila14:23
*** dustins has joined #openstack-manila14:24
openstackgerritValeriy Ponomaryov proposed openstack/manila-specs: Add spec for Manila share groups  https://review.openstack.org/31573014:31
*** porrua has joined #openstack-manila14:31
*** mkoderer has quit IRC14:33
*** vbellur has joined #openstack-manila14:34
openstackgerritDaniel Mellado proposed openstack/manila: Add testscenario to test-requirements  https://review.openstack.org/39614614:42
tbarrondmellado: great! is this at root a bug in oslo.db ?14:43
dmelladotbarron: that's what I'm trying to figure out14:43
tbarrondmellado: +114:44
dmelladoinstalling oslo.db on its own seems to work for me14:44
dmelladoso now I'm checking for more cross-dependencies issues14:44
TommyLikeHu_ping ganso14:44
cknighttoabctl, markstur: Six +2's so far on the snapshot revert spec (which we've all reviewed in Newton).  Any further feedback on it?14:47
toabctlcknight, I'm fine with merging it14:47
cknighttoabctl: great, thanks14:48
cknighttoabctl: https://review.openstack.org/#/c/391269/614:48
openstackgerritzhongjun proposed openstack/manila-specs: Manila db purge utility  https://review.openstack.org/39240614:48
*** xyang_ has joined #openstack-manila14:49
toabctlcknight, is that -1 valid? or just a levftover from somebody?14:49
* toabctl needs to read the backlog...14:49
cknighttoabctl: It's a new -1, but I don't feel the comments require any changes.  I've answered each of them.14:49
TommyLikeHu_does anyone have time to review this spec? https://review.openstack.org/#/c/390052/14:50
*** eharney has joined #openstack-manila14:50
TommyLikeHu_maybe after the meeting14:50
openstackgerritBen Swartzlander proposed openstack/manila-specs: Eliminate Race Conditions Spec  https://review.openstack.org/39625514:51
cknightTommyLikeHu_: It's less an issue of time and more an issue of interest.  Our current quota system is fundamentally flawed, and there isn't much enthusiasm for adding to it.14:52
cknightTommyLikeHu_: Add it to the meeting agenda if you feel it needs further discussion.14:53
gansoTommyLikeHu_: pong14:53
TommyLikeHu_we do have the spec topic today, right?14:53
TommyLikeHu_hey ganso, I updated the spec on db purge14:53
gansoTommyLikeHu_: according to https://wiki.openstack.org/wiki/Manila/Meetings yes14:54
cknightTommyLikeHu_: yes.  https://wiki.openstack.org/wiki/Manila/Meetings14:54
*** xyang_ has quit IRC14:54
gansoTommyLikeHu_: thanks, gonna take a look14:54
TommyLikeHu_cool~14:54
*** makowals has quit IRC14:55
TommyLikeHu_cknight, what do you mean Our current quota system is fundamentallly flawed>14:55
TommyLikeHu_?14:55
TommyLikeHu_it's only manila or other compontents included?14:55
gansoTommyLikeHu_: I still do not agree with the current approach for db purge14:56
TommyLikeHu_sure, for which one?14:56
cknightTommyLikeHu_: Ask tbarron. Quotas often get out of sync with reality. A complete redesign is needed.14:56
gansoTommyLikeHu_: the one that you specify the amount of days prior to NOW to delete14:56
openstackgerritValeriy Ponomaryov proposed openstack/manila-specs: Add spec for Manila share groups  https://review.openstack.org/31573014:57
*** xyang_ has joined #openstack-manila14:57
TommyLikeHu_yes, I talked with tom barron, he said it's ok about this one14:57
*** makowals has joined #openstack-manila14:57
tbarroncknight: https://review.openstack.org/#/c/390052/ doesn't really extend the current quota system, it just extends cli/ui to report on stuff that is currently not exposed14:57
TommyLikeHu_maybe I did not express the intention well14:57
TommyLikeHu_what do you get from this expression here?14:58
cknighttbarron: I understand, but getting users hooked on that will be a disservice if the quota system is rewritten.14:58
tbarroncknight: not sure I agree, this change will make it easier to see when the system is out of sync, etc.14:59
TommyLikeHu_cknight, if we wanna rewrite the base or the skeleton14:59
TommyLikeHu_we still need add this spec14:59
tbarroncknight: w/o this change it is harder to tell when you are broken14:59
TommyLikeHu_this feature14:59
*** zengyingzhe_ has joined #openstack-manila14:59
tbarroncknight: and it's really just cli (and potentially ui), we aren't extending the underlying infra15:00
cknighttbarron: ok, I'll review to understand what is being proposed15:00
TommyLikeHu_cknight, thanks15:00
tbarroncknight: thanks.  I started out wanting to torpedo for just the reasons you expressed :)15:00
TommyLikeHu_meeting started?15:01
*** kaisers_ has joined #openstack-manila15:01
gansoTommyLikeHu_: yes15:01
TommyLikeHu_ganso:I see nothing on my irc channel15:02
gansoTommyLikeHu_: #openstack-meeting-alt15:03
*** kaisers_ has quit IRC15:32
*** zengyingzhe_ has quit IRC15:40
*** jprovazn has joined #openstack-manila15:44
tpsilvacknight: ping16:00
cknighttpsilva: pong16:00
tpsilvacknight: Hello Clinton16:00
tbarronerlon-airlong: over here16:01
cknighttpsilva: Hi, Tiago.  What's up?16:01
tpsilvacknight: can we discuss the mountable snapshots for a minute?16:01
erlon-airlongcknight: tbarron, tbarron: hey16:01
tbarronTommyLikeHu_: zhonghua-lee ^^^16:01
cknighttpsilva: sure16:01
TommyLikeHu_yep~16:01
erlon-airlongcknight: put me in the queue16:01
TommyLikeHu_tbarron: zhonghua-lee is away right noew16:02
cknighterlon-airlong: :-)16:02
TommyLikeHu_s/noew/now16:02
tpsilvacknight: so, my main motivation to have a different set of rules for the snapshots is to allow different systems or users that do not originally have access to it, for instance, I can see scenarios where an administrator would not originally have access to the share, but might want to mount the snapshot16:02
cknighttpsilva: Yep, understood.16:03
tpsilvacknight: or even some external systems, even manila-data16:03
cknighttpsilva: And no argument.  I just wanted that use case described in the spec.16:03
tpsilvacknight: ah ok16:03
*** falavinha has joined #openstack-manila16:04
tpsilvacknight: I thought you would want to propose some modification to the original proposal16:04
tpsilvacknight: but ok, I can add that to the spec16:04
cknighttpsilva: Nope, I was looking for a paragraph in "use cases" that describes a use case that requires separate access rules.16:04
*** falavinha has quit IRC16:05
cknighttpsilva: So anyone else reading the spec would understand why the extra complexity is warranted.16:05
tpsilvacknight: understood, just for clarity16:05
tpsilvacknight: ok, I'll add that to the spec... thank you!16:05
cknighttpsilva: np, thanks!16:06
tbarronerlon-airlong: on the cinder spec you asked questions about why snapshot-id is needed.  manila spec says why.16:06
tbarronTommyLikeHu_: ^^16:07
erlon-airlongtbarron: hmmm, yeah, I saw that today, seems reasonable16:07
TommyLikeHu_tbarron: I will modify the api in cinder's spec16:08
cknightTommyLikeHu_: great, thanks16:08
erlon-airlongtbarron: I hadn't tough about that case, Im talking with Tommy to make all features be as similar as possible16:08
TommyLikeHu_sure16:08
*** falavinha has joined #openstack-manila16:08
erlon-airlongcknight: about the extend, I mentioned because in Cinder we have a but related to extend that does not go trough the scheduler16:09
erlon-airlongcknight: the one in the like I posted16:09
tbarronerlon-airlong: TommyLikeHu_ thanks much!!16:09
cknighterlon-airlong: reading the bug now…16:09
TommyLikeHu_cknight: much appreciated if you have time to review this spec https://review.openstack.org/#/c/390052/ and please leave any comments you come to16:11
*** zhonghua has joined #openstack-manila16:14
TommyLikeHu_bswartz: I have some questions on your comments on ipv6  https://review.openstack.org/#/c/362786/, and these all needs your confirm16:19
tbarronbswartz: vponomaryov https://review.openstack.org/#/c/396146/ unblocks our gate16:26
tbarronbswartz: vponomaryov see https://review.openstack.org/352606 for the current failure16:29
*** harlowja has joined #openstack-manila16:31
*** harlowja has quit IRC16:32
*** harlowja has joined #openstack-manila16:34
TommyLikeHu_ping cknight16:35
tbarronbswartz: xyang_ toabctl ganso vponomaryov markstur vponomaryov should either get spec-review focus or get pushed to pike IMO16:39
tbarron^^^ openstack client proposal for manila, reflects our APIs ...16:40
gansotbarron: I'm sorry, which proposal?16:41
tbarronganso: woops :) https://review.openstack.org/#/c/395775/2/specs/ocata/manila-support-in-python-openstackclient16:42
*** StraubTW_ has joined #openstack-manila16:44
*** catinthe_ has joined #openstack-manila16:45
*** catintheroof has quit IRC16:46
*** zhonghua has quit IRC16:47
*** StraubTW has quit IRC16:47
*** xyang_ has quit IRC16:48
gansotbarron: last time we discussed this, at BCN, IIRC we were against it... but I guess this is an unstoppable force16:49
tbarronganso: well, I think it will happen eventually, but we can influence the pace and requirements - e.g., still allow standalone deployment of manila somehow.16:51
*** tongli has joined #openstack-manila16:51
tbarronganso: but even if that's right, a -2 for now from PTL might be part of getting that goal met16:52
gansotbarron: must be a proper reason for a -216:52
tbarronganso: define "proper reason" - I think it could be that this change is not in scope for ocata16:53
tbarronganso: or maybe I'm confused16:53
gansotbarron: in this case we could just not merge it16:53
tbarronganso: right16:53
tbarronganso: just seems like something we should decide together, not piecemeal, i.e. if we really don't want to merge it we don't want it to somehow get a couple of plus-2s and workflow16:55
gansotbarron: yes16:56
gansotbarron: we should discuss this16:56
*** kaisers_ has joined #openstack-manila17:20
*** dsariel has quit IRC17:29
*** gouthamr has joined #openstack-manila17:32
*** lpetrut has quit IRC17:40
cknightTommyLikeHu_: pong17:44
openstackgerritMerged openstack/manila: Add testscenario to test-requirements  https://review.openstack.org/39614617:45
*** gouthamr has quit IRC17:51
*** xyang_ has joined #openstack-manila17:55
*** breitz has joined #openstack-manila17:55
*** xyang_ has quit IRC17:56
*** vbellur has quit IRC18:02
*** xyang_ has joined #openstack-manila18:02
*** xyang_ has quit IRC18:03
*** igajsin has quit IRC18:03
*** igajsin has joined #openstack-manila18:06
bswartztbarron: I have no problems with OSC supporting manila -- in fact I encourage it18:10
tbarronbswartz: me either, but I'm not sure we're ready for that work for ocata18:11
bswartzthe only thing I take issue with is if they expect us to maintain the integration or deprecate python-manilaclient or change our APIs in any way18:11
tbarronbswartz: it's a matter of focus18:11
bswartzit shouldn't distract us from things we care about as long as they don't as for those 3 things I mentioned18:12
bswartzask18:12
tbarronbswartz: I'd like python-manilaclient core code to be used as a plugin for osc client, so that they differ only in the shell facing the user18:12
bswartztbarron: yes the ideal outcome for both teams is if the client code can be somehow shared18:13
tbarronbswartz: that way standalone manila is still possible18:13
bswartzI suspect that doing that is a lot of work though because they're going to expect us to conform to them rather than the other way around18:13
bswartzin that case I'd prefer to postpone the work18:14
tbarronbswartz: that's why I don't think it's appropriate for our focus for ocata18:14
bswartzagreed18:14
*** breitz has quit IRC18:14
tbarronbswartz: if I'm wrong and external labor can solve that problem with manila - perhaps choosing it as a young and less messy project to pioneer with -- then I would change my opinion.18:14
*** breitz has joined #openstack-manila18:15
tbarroneven then, we can't have de-stabiliztion of the client past feature freeze, etc18:15
bswartztbarron: early indications are that they have a bunch of rules they're not flexible about so there will be many impedance mismatches if we try to share code18:15
tbarronbswartz: and I'm not opposed to our client code being re-enginnered to conform so that code can be shared, I just don't think we have time for it, nor does it fit with a short release cycle and focus on finishing already-started feature work and retiring tech debt18:17
tbarronbswartz: if a team wants to *begin* work on that now, including beginning the spec with an expecation of landing it in pike, that makes a lot of sense to me18:18
*** vbellur has joined #openstack-manila18:18
bswartzagain, as long as it doesn't create distractions18:18
bswartzthe whole microversions thing in OSC left a bad taste in my mouth18:18
tbarroni didn't really follow that, but clearly osc has to support microversions across manila, nova, cinder, etc. or it will fail18:20
bswartzright18:20
bswartzthey pushed us to change our microversion design to match them because they didn't want to special case manila18:20
bswartzI told them no18:21
openstackgerritRodrigo Barbieri proposed openstack/manila-specs: Add spec for Data Jobs table  https://review.openstack.org/39226218:21
tbarronthe cross-component semantic issues are the hard part, not superficial syntax conformity.  The latter is valuable of course to end users - I'm not poo-pooing it - but it doesn't really work unless one deals with the harder former issues.18:21
bswartztbarron: more importantly you can't change it without breaking backward compatibility and defeating the whole point of microversions18:22
tbarronquotas across compoenents are another area where the semantics vary and the differences are glossed over by superficial regularization of syntax in osc18:23
tbarronyeah, agree18:23
* bswartz resists starting a #dieoscdie twitter war18:23
tbarronwhich argues for allowing some degree of semantic difference in compoenents in osc, and perhaps again use of compoentnt client plugins by osc18:24
openstackgerritRodrigo Barbieri proposed openstack/manila-specs: Add spec for Share Migration Ocata improvements  https://review.openstack.org/39229118:25
bswartzI admire what OSC is trying to do -- I just think it's an impossible task given the structure of the community18:26
*** dsariel has joined #openstack-manila18:27
tbarronbswartz: I admire it too, don't want to say it's impossible, but don't want it to be *our* focus for ocata.18:30
gansobswartz, tbarron, toabctl, cknight, markstur: Could you please review this patch that is waiting to merge to be backported when you have some time? https://review.openstack.org/#/c/393271 Thanks in advance18:30
bswartzganso: APIImact and backport?18:31
bswartztbarron: I'm making the argument that it's impossible unless we change something -- I alluded to this in my run for TC18:32
gansobswartz: it adds an additional value for task_state18:32
bswartzganso: so no chance of breakage of existing newton users?18:32
gansobswartz: it may affect adm existing scrips, but not break18:33
gansobswartz: like, scripts may be expecting to see "migration_success" after issuing migration-complete. Right now it goes from "migration_driver_phase1_done" to "migration_success" directly. With that patch, it transitions to "migration_completing" before. It is now more consistent with host-assisted and will behave the same way, so better scripts for admins as18:36
gansowell18:36
gansobswartz: thanks!18:39
*** makowals_ has joined #openstack-manila18:41
*** lpetrut has joined #openstack-manila18:43
*** makowals has quit IRC18:44
*** xyang_ has joined #openstack-manila18:58
*** xyang_ has quit IRC18:59
*** vbellur has quit IRC19:05
openstackgerritMerged openstack/manila: Fix missing 'migration_completing' task state  https://review.openstack.org/39327119:10
*** xyang_ has joined #openstack-manila19:13
*** scottda has quit IRC19:20
*** vbellur has joined #openstack-manila19:20
*** xyang1 has joined #openstack-manila19:21
*** akapil has joined #openstack-manila19:43
*** akapil has quit IRC19:48
*** akerr has quit IRC19:48
*** openstackgerrit has quit IRC19:48
*** openstackgerrit has joined #openstack-manila19:49
*** tpsilva has quit IRC20:04
*** tpsilva has joined #openstack-manila20:06
*** johnthetubaguy has quit IRC20:10
*** johnthetubaguy has joined #openstack-manila20:11
*** eharney has quit IRC20:18
*** porrua has quit IRC20:33
*** eharney has joined #openstack-manila20:34
*** xyang_ has quit IRC20:52
*** xyang_ has joined #openstack-manila20:53
*** xyang_ has quit IRC20:54
*** xyang_ has joined #openstack-manila20:55
*** xyang_ has quit IRC20:57
*** xyang_ has joined #openstack-manila20:57
openstackgerritBen Swartzlander proposed openstack/manila-specs: Eliminate Race Conditions Spec  https://review.openstack.org/39625521:13
*** vbellur has quit IRC21:18
*** cknight has quit IRC21:53
*** xyang_ has quit IRC21:56
*** catinthe_ has quit IRC21:59
*** jprovazn has quit IRC22:02
*** eharney has quit IRC22:03
*** dustins has quit IRC22:03
*** ganso has quit IRC22:16
*** tongli has quit IRC22:17
*** tpsilva has quit IRC22:24
*** xyang_ has joined #openstack-manila22:30
*** cknight has joined #openstack-manila22:45
*** xyang_ has quit IRC22:46
*** xyang_ has joined #openstack-manila22:47
*** xyang_ has quit IRC22:56
*** cknight has quit IRC22:59
*** catintheroof has joined #openstack-manila23:05
*** lpetrut has quit IRC23:05
*** catintheroof has quit IRC23:09
*** cknight has joined #openstack-manila23:13
*** catintheroof has joined #openstack-manila23:17
*** xyang1 has quit IRC23:20
*** xyang_ has joined #openstack-manila23:23
*** xyang_ has quit IRC23:24
*** StraubTW_ has quit IRC23:31
*** xyang_ has joined #openstack-manila23:51
*** akapil has joined #openstack-manila23:59

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