Wednesday, 2016-12-21

« Tuesday, 2016-12-20 Index Thursday, 2016-12-22 »
openstackgerritpangliye proposed openstack/manila: Add some spaces to pass the check of pep8  https://review.openstack.org/41329800:50
openstackgerritTony Breeds proposed openstack/manila-image-elements: Add Constraints support  https://review.openstack.org/41329900:50
*** harlowja has joined #openstack-manila01:06
openstackgerrityunfeng zhou proposed openstack/puppet-manila: add CONTRIBUTING.rst  https://review.openstack.org/41291801:17
*** harlowja has quit IRC01:44
openstackgerritMerged openstack/manila-image-elements: Add Constraints support  https://review.openstack.org/41329901:53
openstackgerrityankee proposed openstack/manila: Allow use of <is> True and <is> False with DHSS and snapshot support  https://review.openstack.org/41333802:05
openstackgerritTony Breeds proposed openstack/python-manilaclient: Add Constraints support  https://review.openstack.org/41333902:06
*** tuanluong has joined #openstack-manila02:31
*** gcb has joined #openstack-manila02:35
*** gouthamr has quit IRC02:58
*** mtanino has quit IRC04:16
*** senk has joined #openstack-manila05:25
*** senk has quit IRC06:24
*** lpetrut has joined #openstack-manila06:43
*** senk has joined #openstack-manila07:05
*** pcaruana has joined #openstack-manila07:34
*** nherciu has joined #openstack-manila07:45
*** openstackgerrit has quit IRC07:48
*** lpetrut has quit IRC07:51
*** tuanluong has quit IRC08:05
*** cfey has quit IRC08:16
*** cfey has joined #openstack-manila08:23
*** openstackgerrit has joined #openstack-manila08:39
openstackgerritzhongjun proposed openstack/python-manilaclient: Add share_type filter to pool_list  https://review.openstack.org/36852808:39
*** dsariel has joined #openstack-manila08:40
*** cfey has quit IRC08:50
*** akapil has joined #openstack-manila09:03
openstackgerritzhongjun proposed openstack/manila: Enable IPv6 in manila(allow access)  https://review.openstack.org/31232109:07
*** dsariel has quit IRC09:13
*** cfey has joined #openstack-manila09:16
openstackgerritValeriy Ponomaryov proposed openstack/manila: [Grenade] Do not run tempest tests  https://review.openstack.org/41179609:18
openstackgerritValeriy Ponomaryov proposed openstack/manila: [DNM] debug 3.0  https://review.openstack.org/41036609:21
*** breitz has joined #openstack-manila09:22
*** a-pugachev has joined #openstack-manila09:25
*** breitz_ has quit IRC09:26
*** chlong has quit IRC09:34
openstackgerritValeriy Ponomaryov proposed openstack/manila: Manila Share Groups  https://review.openstack.org/33509309:49
*** ganso has joined #openstack-manila09:51
openstackgerritValeriy Ponomaryov proposed openstack/manila: Manila Share Groups  https://review.openstack.org/33509309:58
*** alyson_ has joined #openstack-manila10:03
openstackgerritTina Tang proposed openstack/manila: [Unity driver] VLAN enhancement  https://review.openstack.org/41003710:09
*** tuanluong has joined #openstack-manila10:10
*** lpetrut has joined #openstack-manila10:14
*** ociuhandu has quit IRC10:14
*** akapil has quit IRC10:28
*** akapil has joined #openstack-manila10:30
*** ociuhandu has joined #openstack-manila10:31
*** tuanluong has quit IRC10:34
*** gcb has quit IRC10:58
*** dsariel has joined #openstack-manila10:59
*** akapil has quit IRC11:09
*** arnewiebalck has quit IRC11:10
*** gcb has joined #openstack-manila11:15
*** akapil has joined #openstack-manila11:29
openstackgerritMerged openstack/python-manilaclient: Add Python 3.5 classifier and venv  https://review.openstack.org/39069311:29
*** akapil has quit IRC11:34
*** akapil has joined #openstack-manila11:34
*** gcb has quit IRC11:40
*** gcb has joined #openstack-manila11:52
*** tommylik_ has joined #openstack-manila12:14
*** tommylik_ has quit IRC12:16
*** tommylikehu_ has joined #openstack-manila12:16
openstackgerritHelen Walsh proposed openstack/manila: VMAX manila plugin - Support for VMAX in Manila  https://review.openstack.org/40485912:22
*** ociuhandu has quit IRC12:39
*** pcaruana has quit IRC12:49
*** rraja has joined #openstack-manila12:54
openstackgerritzhongjun proposed openstack/manila: Trivial fix LOG.exception issues  https://review.openstack.org/41268812:58
*** ociuhandu has joined #openstack-manila13:06
*** gouthamr has joined #openstack-manila13:18
*** yankee has joined #openstack-manila13:31
*** pcaruana has joined #openstack-manila13:32
openstackgerritDigvijay Ukirde proposed openstack/manila: Add support for manage/unmanage in GPFS driver  https://review.openstack.org/37470513:39
*** xinyanzhang has quit IRC13:41
*** xinyanzhang has joined #openstack-manila13:42
*** xinyanzhang has quit IRC13:45
*** xinyanzhang has joined #openstack-manila13:45
openstackgerritMerged openstack/manila: [Grenade] Do not run tempest tests  https://review.openstack.org/41179613:49
*** yankee has quit IRC13:51
*** xyang1 has joined #openstack-manila13:52
*** eharney has joined #openstack-manila14:21
*** senk has quit IRC14:31
*** senk has joined #openstack-manila14:32
*** akapil has quit IRC14:35
*** pcaruana has quit IRC14:38
*** zengyingzhe_ has joined #openstack-manila14:39
*** senk has quit IRC14:41
*** senk has joined #openstack-manila14:42
*** zengyingzhe has quit IRC14:43
*** akapil has joined #openstack-manila14:46
*** porrua has joined #openstack-manila14:47
*** pcaruana has joined #openstack-manila14:51
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Add create_share_from_snapshot_support extra spec  https://review.openstack.org/35668214:57
openstackgerritVitaliy Levitski proposed openstack/manila: Add MapR-FS native driver  https://review.openstack.org/35003515:12
*** lpetrut has quit IRC15:25
vponomaryovgouthamr: what about client changes https://review.openstack.org/#/c/356683/  ?15:28
*** lpetrut has joined #openstack-manila15:30
*** senk has quit IRC15:33
gouthamrvponomaryov: cknight was looking at them15:35
*** cknight has joined #openstack-manila15:41
*** lpetrut1 has joined #openstack-manila15:43
*** lpetrut has quit IRC15:44
*** lpetrut1 is now known as lpetrut15:44
*** tommylikehu_ has quit IRC15:46
*** openstackgerrit_ has joined #openstack-manila15:47
*** openstackgerrit_ has quit IRC15:49
gansocknight: pong15:50
*** mtanino has joined #openstack-manila15:52
*** chlong has joined #openstack-manila15:52
*** lpetrut has quit IRC15:53
*** nherciu has quit IRC15:54
*** akapil has quit IRC15:54
*** nherciu has joined #openstack-manila15:59
*** a-pugachev has quit IRC16:02
*** a-pugachev has joined #openstack-manila16:02
*** nherciu_ has joined #openstack-manila16:05
*** nherciu has quit IRC16:05
*** akapil has joined #openstack-manila16:08
cknightganso: Hi, Rodrigo.  Goutham & I didn't understand your comment here.  https://review.openstack.org/#/c/356682/28/contrib/ci/post_test_hook.sh  What are you looking for, please?  Y'all need this patch as well, so we're trying to get it wrapped up soon.16:10
openstackgerritVitaliy Levitski proposed openstack/manila: Add MapR-FS native driver  https://review.openstack.org/35003516:10
*** jcsp has quit IRC16:11
*** jcsp has joined #openstack-manila16:11
gansocknight: I meant to move it to line 238, either using an else or a "define if not defined"16:12
*** pcaruana has quit IRC16:13
*** senk has joined #openstack-manila16:14
cknightganso: Ok, I wondered if you were looking for an 'else' here.  There are other things in this file that would also fit into that else block, so I could add the else only for my addition, or we could do my part and anything else all together in a separate patch.16:14
*** tommylikehu_ has joined #openstack-manila16:15
gansocknight: either way is ok, but if you created the else block now, later new changes could use it16:16
gansocknight: I believe the revert and mountable patch will use it16:18
cknightganso: ok, I'll discuss with Goutham and see which he prefers.  He's been helping me with this one while I attend to other things.16:19
gansocknight: ok, thanks Clinton!16:19
cknightganso: I assume y'all have been rebasing your mountable snapshots patch on this one.  Has it been working for you?16:20
gansocknight: Latest mountable snapshot patch has, with a few workarounds, but it is still based on an older version of your extra_specs patch16:21
*** lpetrut has joined #openstack-manila16:21
gansocknight: I will probably rebase on the latest later today or tomorrow to test16:22
gansocknight: I started working on mountable covering for tpsilva this monday16:22
gansocknight: so I am still getting familiar16:22
cknightganso: That should be OK, I haven't made any substantive changes in several weeks.  Please let us know how it goes, and we will look to you for one of the +2's.  Thanks!16:22
cknightganso: It'd be great to get this merged before Christmas!16:23
gansocknight: my vacation starts this Saturday16:23
gansocknight: I'll be back Jan 5th16:24
cknightganso: Same here16:24
gansocknight: if all my tests pass, I think it will be ready to merge16:25
* markstur drifts off into a mental vacation16:27
*** dsariel has quit IRC16:39
marksturI'll be out from Saturday through Jan 4th.  I might be available while I'm out but not sure yet how useful that will be.16:41
*** eharney has quit IRC16:47
*** lseki has joined #openstack-manila16:51
*** eharney has joined #openstack-manila16:58
*** a-pugachev has quit IRC17:00
*** openstackstatus has joined #openstack-manila17:00
*** ChanServ sets mode: +v openstackstatus17:00
openstackgerritTom Barron proposed openstack/manila: DO NOT MERGE  https://review.openstack.org/35260617:00
*** openstackstatus has quit IRC17:02
*** openstack has quit IRC17:02
*** openstack has joined #openstack-manila17:06
gansogouthamr: ping17:15
*** a-pugachev has quit IRC17:17
*** lpetrut has quit IRC17:22
*** senk has quit IRC17:23
*** panatl has quit IRC17:26
openstackgerritTom Barron proposed openstack/manila: WIP: Remove nova net from service_instance  https://review.openstack.org/41044917:36
openstackgerritTom Barron proposed openstack/manila: WIP - Remove NovaNetworkPlugin  https://review.openstack.org/41045017:36
openstackgerritValeriy Ponomaryov proposed openstack/manila: [DNM] Test 4.0  https://review.openstack.org/41373717:45
*** alyson_ has quit IRC17:46
openstackgerritValeriy Ponomaryov proposed openstack/manila: [DNM] Test 4.0  https://review.openstack.org/41373717:48
*** akapil has quit IRC17:50
*** nherciu has joined #openstack-manila17:54
openstackgerritValeriy Ponomaryov proposed openstack/manila: [DNM] Test 4.0  https://review.openstack.org/41373717:54
*** senk has joined #openstack-manila17:55
*** nherciu_ has quit IRC17:55
*** mtanino has quit IRC17:55
vponomaryovcknight, ganso: lets just get it merged17:56
vponomaryovcknight, ganso: and fix bugs later if any17:56
gansovponomaryov: the same could be said for migration patches :P17:56
-openstackstatus- NOTICE: Gerrit is being restarted to update its OpenID SSO configuration17:58
vponomaryovganso: if you are talking about https://review.openstack.org/#/c/406305/ then it does not touch api microversions and db migrations17:58
vponomaryovso, we gain more merging the most affecting other commits commit17:59
vponomaryovganso: and you gain too for mountable snapshots feature )18:00
vponomaryovgansO; your turn ))18:00
openstackgerritRodrigo Barbieri proposed openstack/manila: Share Migration Ocata Improvements  https://review.openstack.org/40630518:00
gansovponomaryov: which patch does not touch microversions and db migrations?18:02
gansovponomaryov: my patch does, so does cknight's18:02
*** senk has quit IRC18:04
vponomaryovganso: hm, ok, api changed. Not DB. + your one is smaller18:04
*** ociuhandu has quit IRC18:04
*** senk has joined #openstack-manila18:05
openstackgerritValeriy Ponomaryov proposed openstack/manila: [DNM] Test 4.0  https://review.openstack.org/41373718:06
*** jcsp has quit IRC18:09
gouthamrganso: pong18:12
gouthamrvponomaryov: +118:12
*** jkremer has joined #openstack-manila18:12
*** jkremer has quit IRC18:13
*** jkremer has joined #openstack-manila18:13
*** jkremer has quit IRC18:14
gansovponomaryov: see, gouthamr agrees that we should merge the migration patches :P18:14
gouthamrganso: lol18:14
gansogouthamr: I am re-reading the access rules spec18:15
*** jcsp has joined #openstack-manila18:15
gansogouthamr: I can't find where it is explained how the recovery mode is triggered, besides restarting the service18:15
*** ociuhandu has joined #openstack-manila18:15
gansogouthamr: so based on that I assume that there is no more recovery mode based on error-ed rules?18:16
gouthamrganso: yep. there's no recovery mode18:17
gouthamrganso: what sort of recovery are you expecting?18:17
gansogouthamr: if we have rules that are conflicting, and cannot be removed individually, they always error out18:17
gouthamrganso: access-denies should not error out at the driver, ever; unless we genuinely can't deny a rule for a legit reason18:18
gouthamrganso: now, you could have a smaller set of rules errored out than before....18:18
gansogouthamr: and the only way to fix that is passing add_rules=[] and delete_rules=[] so drivers can clear everything out and reapply... restarting the service looks very brutal way to do trigger that18:18
gansogouthamr: this is the recovery mechanism that was discussed in Tokyo, before we had update_access, there was a use case for this, having to clear all the rules and add them back18:19
gouthamrganso: i wonder what the use case is. if there is a use case, shouldn't there be an API for it?18:20
*** ociuhandu has quit IRC18:20
gouthamrganso: which process internally needs a recovery mechanism, today?18:20
gouthamrganso: iirc migration doesn't.18:21
gansogouthamr: only access rules18:21
gansogouthamr: bswartz mentioned that yesterday, that is one of the benefits of update_access, and we would be crippling that18:22
gouthamrganso: hmmm, yes, we don't have any need for it imo. You saw me introduce a "resync" parameter.. but i didn't mention it in the spec because it's going to go away18:22
gansogouthamr: are you saying the updates are not going to run recursively anymore?18:24
gouthamrganso: not sure what you mean by that.. we get a request, we ask for the driver to update rules and while the driver is updating rules, we ignore any further requests.. when the driver returns, we call it again if there's more work to be done.18:29
gansogouthamr: "call it again": that's the resync, isn't it?18:29
gouthamrganso: that's not "resync" - that's just apply the remaining changes... resync would mean to call the driver with no rules in transitional states18:30
gouthamrganso: i.e, just say, these are the rules i'm expecting you to have - and add_rules and delete_rules will be []18:30
gouthamrganso: like you mentioned before18:30
gansogouthamr: I call that recovery mode lol18:31
gouthamrganso: names.18:32
gouthamrganso: yes, so there's no recovery mode18:32
*** pcaruana has joined #openstack-manila18:34
tbarronso if there is a use case we could add a resync api that puts all rules in add rules and passes a flag to the driver saying to delete any rules not in add rules.  But there is no way for OpenStack to know what the driver has that might ought to be deleted in a resync.18:40
*** ociuhandu has joined #openstack-manila18:41
tbarronor a driver could return a status to the share manager that says "resync me" to get all the current rules as add-rules18:41
gouthamrtbarron: the resync API can actually pass add_rules=[] and delete_rules=[]18:47
gouthamrtbarron: currently drivers expect that to "resync"18:48
gouthamrtbarron: drivers that don't have a way to mass-commit rules onto the back end..18:48
tbarrongouthamr: yeah, I realized as I wrote that you'd need to pass the deletes that are in flight.18:48
gouthamri'm wondering if it's time to introduce the "resync" API, or just not worry about it until we have a use case. there's no need for it in any current workflow...18:49
tbarrongouthamr: YAGNI18:50
tbarrongouthamr: and if you do, you'll find out when you do18:50
gouthamrtbarron: +118:50
*** nherciu has quit IRC19:08
gansogouthamr, tbarron: we had motivations to introduce that before, for me it doesn't feel like it won't cause any damage if we remove19:10
*** rraja has quit IRC19:13
*** mtanino has joined #openstack-manila19:24
*** gouthamr has quit IRC19:39
*** gouthamr has joined #openstack-manila20:03
*** senk has quit IRC20:03
openstackgerritMark Sturdevant proposed openstack/manila: GPFS KNFS fix deny access to succeed when possible  https://review.openstack.org/41376220:07
openstackgerritClinton Knight proposed openstack/python-manilaclient: Add create_share_from_snapshot_support extra spec  https://review.openstack.org/35668320:08
*** pcaruana has quit IRC20:11
*** senk has joined #openstack-manila20:21
openstackgerritMark Sturdevant proposed openstack/manila: GPFS KNFS: Do not reuse ssh prefix in loop  https://review.openstack.org/41323920:32
*** senk has quit IRC20:32
*** lpetrut has joined #openstack-manila20:32
openstackgerritMark Sturdevant proposed openstack/manila: GPFS KNFS fix deny access to succeed when possible  https://review.openstack.org/41376220:33
*** senk has joined #openstack-manila20:36
*** chlong has quit IRC20:36
*** senk has quit IRC20:38
*** chlong has joined #openstack-manila20:52
openstackgerritClinton Knight proposed openstack/python-manilaclient: Add create_share_from_snapshot_support extra spec  https://review.openstack.org/35668320:55
openstackgerritGoutham Pacha Ravi proposed openstack/manila: NetApp cDOT: Add Intra-vserver migration support  https://review.openstack.org/41378320:56
*** porrua has quit IRC21:07
openstackgerritTom Barron proposed openstack/manila: WIP - Remove NovaNetworkPlugin  https://review.openstack.org/41045021:09
openstackgerritDirk Mueller proposed openstack/python-manilaclient: Add Constraints support  https://review.openstack.org/41333921:51
*** cknight has quit IRC21:52
*** gouthamr has quit IRC21:58
*** lpetrut has quit IRC22:01
*** chlong has quit IRC22:04
*** ganso has quit IRC22:23
*** tommylikehu_ has quit IRC22:45
*** tommylikehu_ has joined #openstack-manila22:47
*** tommylik_ has joined #openstack-manila22:47
*** tommylikehu_ has quit IRC22:47
*** gouthamr has joined #openstack-manila22:53
*** xyang1 has quit IRC22:58
*** lseki has quit IRC23:00
*** tommylik_ has quit IRC23:03
openstackgerritMark Sturdevant proposed openstack/manila: GPFS KNFS: Do not reuse ssh prefix in loop  https://review.openstack.org/41323923:34
*** catintheroof has joined #openstack-manila23:48
« Tuesday, 2016-12-20 Index Thursday, 2016-12-22 »

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