Monday, 2017-01-16

*** a-pugachev has quit IRC00:02
*** tuanluong has joined #openstack-manila00:42
*** gouthamr has joined #openstack-manila01:14
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Add cast_rules_to_readonly to share instances  https://review.openstack.org/41916301:19
openstackgerritzhongjun proposed openstack/manila: Enable IPv6 in manila(allow access)  https://review.openstack.org/31232101:20
openstackgerritzhongjun proposed openstack/manila: Fix column name error in migration script  https://review.openstack.org/41816801:25
openstackgerritzhongjun proposed openstack/manila: Enable IPv6 in manila(network plugins and drivers)  https://review.openstack.org/40677601:33
*** gouthamr has quit IRC01:41
*** dsariel has quit IRC01:47
*** gcb has joined #openstack-manila01:54
*** nkrinner_afk has quit IRC02:12
*** nkrinner_afk has joined #openstack-manila02:12
*** sticker has joined #openstack-manila03:24
*** bswartz has quit IRC04:03
*** bswartz has joined #openstack-manila04:06
*** bswartz has quit IRC04:10
*** gouthamr has joined #openstack-manila04:24
*** bswartz has joined #openstack-manila04:30
*** bswartz has quit IRC04:32
*** bswartz has joined #openstack-manila04:33
*** bswartz has quit IRC04:44
*** bswartz has joined #openstack-manila04:49
*** bswartz has quit IRC04:52
*** bswartz has joined #openstack-manila04:53
*** bswartz1 has joined #openstack-manila05:01
*** bswartz1 has quit IRC05:01
openstackgerritzhongjun proposed openstack/python-manilaclient: Add the ability to check tenant quota detail  https://review.openstack.org/36856706:08
openstackgerritTina Tang proposed openstack/manila: [Unity driver] VLAN enhancement  https://review.openstack.org/41003706:37
*** lpetrut has joined #openstack-manila06:42
openstackgerritzhongjun proposed openstack/python-manilaclient: Add the ability to check tenant quota detail  https://review.openstack.org/36856706:43
*** mnaser has quit IRC06:53
*** a-pugachev has joined #openstack-manila07:02
*** a-pugachev has quit IRC07:12
*** mnaser has joined #openstack-manila07:14
*** nkrinner_afk is now known as nkrinner07:24
*** jprovazn has joined #openstack-manila07:27
*** gouthamr has quit IRC07:32
*** pcaruana has joined #openstack-manila07:34
*** gcb has quit IRC07:44
*** gcb has joined #openstack-manila07:46
*** dsariel has joined #openstack-manila08:51
openstackgerritTuan Luong-Anh proposed openstack/manila: Replace yaml.load() with yaml.safe_load()  https://review.openstack.org/42058208:56
tommylikehuping vponomaryov09:05
*** houming has joined #openstack-manila09:09
*** a-pugachev has joined #openstack-manila09:16
openstackgerritzhongjun proposed openstack/manila: Add manila-manage db purge command  https://review.openstack.org/30821209:17
*** ganso has joined #openstack-manila09:43
tommylikehuhey ganso09:45
tommylikehumorning, could you take a look at this patch? https://review.openstack.org/#/c/418168/09:46
gansotommylikehu: hi tommylikehu09:48
gansotommylikehu: ok09:48
tommylikehuhey tbarron, vponomaryov, bswartz: I just uploaded another patch for DB purge, and I think maybe this time we can drop the manual SORTED_MODELS. https://review.openstack.org/#/c/308212/09:52
*** yumiriam has joined #openstack-manila10:00
*** tuanluong has quit IRC10:07
openstackgerritValeriy Ponomaryov proposed openstack/manila: Rename consistency group modules to share groups  https://review.openstack.org/40986410:46
openstackgerritzhongjun proposed openstack/manila: Fix column name error in migration script  https://review.openstack.org/41816810:46
gansotommylikehu: that change still does not look clear to me10:48
tommylikehuwhy?10:49
gansotommylikehu: commit message now says: '5077ffcc5f1c_add_share_instances' script.10:49
gansotommylikehu: which is related to the change you did in the 5077ffcc5f1c migration, which is the bug you fixed10:49
gansotommylikehu: could you please explain the other change?10:49
gansotommylikehu: did your change break eb6d5544cbbd?10:50
tommylikehuok wait a second10:50
openstackgerritValeriy Ponomaryov proposed openstack/python-manilaclient: Add share group support to Manila client  https://review.openstack.org/33512010:53
vponomaryovganso: it is ok for me10:55
vponomaryovganso: he just says that he fixes bug in script and adds tests for it10:56
gansovponomaryov: I would just like to know if it broke another test, otherwise the change to eb6d5544cbbd is unrelated10:56
vponomaryovganso: i tis ok10:57
vponomaryovganso: new test adds records10:57
vponomaryovganso: that influence existing one, that was not expecting additional records10:58
gansovponomaryov: great, that was the explaination I was looking for10:58
vponomaryovganso: so, it is colpletely correct approach to fix existing test10:58
vponomaryovganso: it is obvious )10:58
gansovponomaryov: but not stated in the commit message. Ideally it should, but now that it is clear I think it is fine11:00
*** lpetrut has quit IRC11:03
vponomaryovganso: sure, insist on describing )11:04
vponomaryovganso: be a man!11:04
gansovponomaryov: I said Ideally, I would do it, but really no need to11:04
vponomaryovbe tough like a rock!11:04
tommylikehuganso, vponomaryov: Oh I get what  ganso's point now :(11:05
tommylikehuvponomaryov: can we add a database cleaning action before each testcases?11:06
vponomaryovtommylikehu: better to write correct tests11:06
tommylikehuvponomaryov: also can you take a review on this patch ? https://review.openstack.org/#/c/308212/11:07
*** JoseMello has joined #openstack-manila11:13
*** erlon_pto has joined #openstack-manila11:15
*** lpetrut has joined #openstack-manila11:26
*** lpetrut has quit IRC11:30
openstackgerritzhongjun proposed openstack/manila-specs: Add spec for share backup  https://review.openstack.org/33030611:39
*** alyson_ has joined #openstack-manila11:44
*** lseki has joined #openstack-manila11:51
*** lpetrut has joined #openstack-manila11:59
openstackgerritAlyson proposed openstack/python-manilaclient: Add mountable snapshots support to manila client  https://review.openstack.org/34562511:59
*** jprovazn has quit IRC12:06
*** jprovazn has joined #openstack-manila12:07
*** ociuhandu has quit IRC12:15
*** ociuhandu has joined #openstack-manila12:19
tommylikehuhey ganso: could you review this patch again? https://review.openstack.org/#/c/418168/   the Jenkins looks good12:25
gansotommylikehu: done12:26
tommylikehugaso: thanks12:34
openstackgerritValeriy Ponomaryov proposed openstack/python-manilaclient: Add support of microversion 2.26  https://review.openstack.org/42069612:37
openstackgerritValeriy Ponomaryov proposed openstack/python-manilaclient: Add share group support to Manila client  https://review.openstack.org/33512012:38
*** houming has quit IRC12:49
*** gcb has quit IRC13:01
*** rraja has joined #openstack-manila13:04
openstackgerritJeremy Liu proposed openstack/manila: Update .gitignore  https://review.openstack.org/42070713:06
*** zhonghua has quit IRC13:17
*** zhonghua has joined #openstack-manila13:19
openstackgerritValeriy Ponomaryov proposed openstack/manila: Manila Share Groups  https://review.openstack.org/33509313:23
openstackgerritValeriy Ponomaryov proposed openstack/manila: [Tempest] Add functional tests for share groups feature  https://review.openstack.org/35526413:28
*** erlon_pto is now known as erlon13:33
openstackgerritTom Barron proposed openstack/python-manilaclient: Remove Nova Net option for share networks  https://review.openstack.org/41623913:35
*** jprovazn has quit IRC13:39
*** xyang1 has joined #openstack-manila13:44
*** zengyingzhe has joined #openstack-manila13:52
*** rraja has quit IRC13:53
*** mkoderer has joined #openstack-manila14:20
*** jprovazn has joined #openstack-manila14:21
openstackgerritJeremy Liu proposed openstack/manila: Update .gitignore  https://review.openstack.org/42070714:26
vponomaryovtbarron: actually, your change was first14:40
vponomaryovtbarron: I created my own now knowing about yours14:41
tbarronvponomaryov: of course, but I really don't care.  Let's just get the client changes merged so the functional tests work.14:42
tbarronvponomaryov: I meant to get the changes in at pretty much the same time.14:42
tbarronvponomaryov: your change set is a little simpler and mine still has a couple issues14:42
vponomaryovtbarron: my has issues too14:43
vponomaryov))14:43
tbarronvponomaryov: but it makes the tests pass14:43
vponomaryovtbarron: forgot to update "update" method in "shell" module14:43
vponomaryov))14:43
tbarronok, I'll push a new change set, you can too, and whichever merges, merges.  As I say, I really don't care on this one :D14:44
tbarronvponomaryov: do you think the way you are generating an error if 'nova-net-id' is supplied with the new microversion is better?  Mine just does a usage error ...14:45
*** gcb has joined #openstack-manila14:50
openstackgerritVictoria Martinez de la Cruz proposed openstack/manila-image-elements: Enables end user to pick share protocol  https://review.openstack.org/40041114:52
openstackgerritMerged openstack/manila: Fix error message in Share Networks API  https://review.openstack.org/42044414:52
openstackgerritDirk Mueller proposed openstack/manila: Allow access to test vm floating IP  https://review.openstack.org/42074814:52
*** mtanino has joined #openstack-manila15:01
vponomaryovtbarron: waiting for your update15:21
tbarronvponomaryov: k, just finishing a @#$# meeting15:22
openstackgerritTom Barron proposed openstack/python-manilaclient: Remove Nova Net option for share networks  https://review.openstack.org/41623915:24
tbarronvponomaryov: ^^15:24
openstackgerritRodrigo Barbieri proposed openstack/manila: Share Migration Ocata Improvements  https://review.openstack.org/40630515:28
*** dustins has joined #openstack-manila15:33
*** nkrinner is now known as nkrinner_afk15:35
vponomaryovtbarron: shouldn't it be holiday for you today?15:35
tbarronvponomaryov: should be but it isn't :D15:36
vponomaryovtbarron: [gouthamr-mode]can you please add reno releasenote too[/gouthamr-mode]15:36
tbarronvponomaryov: oh yeah, forgot, just a minute'15:36
openstackgerritVictoria Martinez de la Cruz proposed openstack/manila-image-elements: Enables end user to pick share protocol  https://review.openstack.org/40041115:40
*** gouthamr has joined #openstack-manila15:48
dmelladovponomaryov: tbarron now that I've got your attention, could you have a look at this backport for newton? https://review.openstack.org/#/c/420429/15:51
dmelladothanks!15:51
vponomaryovdmellado: you need bswartz15:52
openstackgerritTom Barron proposed openstack/python-manilaclient: Remove Nova Net option for share networks  https://review.openstack.org/41623915:52
dmelladovponomaryov: I'll add him to the review, thanks!15:52
*** lpetrut has quit IRC15:52
vponomaryovdmellado: only he is able to merge it15:52
dmelladoyumiriam: ack15:52
dmelladovponomaryov: I meant15:52
*** eharney has joined #openstack-manila15:58
*** eharney has quit IRC16:05
openstackgerritRodrigo Barbieri proposed openstack/python-manilaclient: Implement Share Migration Ocata improvements  https://review.openstack.org/40630616:09
openstackgerritRodrigo Barbieri proposed openstack/python-manilaclient: Implement Share Migration Ocata improvements  https://review.openstack.org/40630616:10
openstackgerritRodrigo Barbieri proposed openstack/manila-ui: Implement Share Migration Ocata improvements  https://review.openstack.org/40630716:18
*** eharney has joined #openstack-manila16:18
openstackgerritRodrigo Barbieri proposed openstack/manila-ui: Implement Share Migration Ocata improvements  https://review.openstack.org/40630716:18
openstackgerritTom Barron proposed openstack/python-manilaclient: Remove Nova Net option for share networks  https://review.openstack.org/41623916:38
*** dustins_ has joined #openstack-manila16:40
*** gcb has quit IRC16:42
tbarrongouthamr: ^^16:44
gouthamrthanks tbarron16:45
*** dustins_ has quit IRC16:47
openstackgerritRodrigo Barbieri proposed openstack/manila: Share Migration Ocata Improvements  https://review.openstack.org/40630517:09
openstackgerritTom Barron proposed openstack/python-manilaclient: Remove Nova Net option for share networks  https://review.openstack.org/41623917:12
*** dsariel has quit IRC17:12
tbarrongouthamr: vponomaryov sorry for sloppiness, this time for sure :D ^17:12
openstackgerritOpenStack Proposal Bot proposed openstack/manila: Updated from global requirements  https://review.openstack.org/42084017:20
*** a-pugachev has quit IRC17:24
openstackgerritRodrigo Barbieri proposed openstack/manila: Share Migration Ocata Improvements  https://review.openstack.org/40630517:55
openstackgerritTom Barron proposed openstack/manila: Some share api test cleanup  https://review.openstack.org/42088618:01
openstackgerritTom Barron proposed openstack/manila: tests: remove useless variables in db_utils methods  https://review.openstack.org/42088718:01
openstackgerritTom Barron proposed openstack/manila: Remove deprecated share_type share model properties  https://review.openstack.org/42088818:01
openstackgerritTom Barron proposed openstack/manila: Remove deprecated availability zone from share model  https://review.openstack.org/42088918:01
openstackgerritTom Barron proposed openstack/manila: Remove deprecated share_network_id from share model  https://review.openstack.org/42089018:01
openstackgerritTom Barron proposed openstack/manila: Remove deprecated share_server_id from share model  https://review.openstack.org/42089118:01
openstackgerritTom Barron proposed openstack/manila: Remove deprecated host property from share model  https://review.openstack.org/42089218:01
gansoO_O18:01
tbarronI got really sick of those deprecated property messages in the logs18:01
gouthamro.O18:02
gouthamron a roll :D18:02
tbarronno microversion bumps or schema changes18:02
vponomaryovtbarron has the lowest boiling point ))18:03
gouthamrwill remember to vent to tbarron every time i get annoyed with manila18:04
vponomaryovgouthamr: you get annoyed with manila?18:05
gansowell, people reactions to deprecated property message in logs in different scales18:05
tbarronvponomaryov: about 90 degrees C18:05
openstackgerritMerged openstack/manila: Fix column name error in migration script  https://review.openstack.org/41816818:06
gouthamrvponomaryov: no, i just like to vent18:06
vponomaryovganso: I guess our priority now is https://review.openstack.org/#/c/36966818:08
vponomaryovaccording to https://etherpad.openstack.org/p/manila-ocata-code-review-focus18:09
gansovponomaryov: my priority right now is to update my patches first18:09
gansovponomaryov: but yes, I already told gouthamr that I'll be reviewing his access rule patch once I finish updating mine18:10
gouthamrcrosses-fingers18:10
vponomaryovgouthamr: "today" in etherpad looks really funny18:10
gouthamrvponomaryov: see what i did there?18:10
vponomaryovgouthamr: whre there?18:11
gouthamrtbarron: why not just create @property for all those deprecated properties, we don't need those deprecation msges anymore18:11
tbarronthe ethernal presnt18:11
tbarrongouthamr: I took seriously the idea that they are truly deprecated.18:12
tbarrongouthamr: it actually shows some test issues that had to be fixed.18:12
tbarrongouthamr: where fake shares were being passed to rpcs that wanted instances for example18:12
gansotbarron takes deprecation seriously18:12
gouthamrtbarron: they are.. but the assumption that the share is it's instance holds in many places..18:12
tbarronmany, or all18:13
tbarronit is confusing18:13
tbarronbut if the community wants to leave all the proxies and throw away the deprecation msgs we could do that18:14
tbarrongouthamr: note though it makes handling races trickier18:14
tbarrongouthamr: harder to tell when you are locking around a simple DB transition and when it's a bunch of gobbledybook with multiple tables18:15
tbarronunderstanding races is one of the reasons I am trying to clean up18:15
gouthamrtbarron: hmmm, in code, we should refer to the instance at all times and get share properties in it by default -> i fixed this for access rules..18:16
gouthamrtbarron: the only place where share would be applicable is in the API..18:16
gouthamrtbarron: that deprecation message made no sense to me, because it looks like it was ignored since liberty..18:17
gouthamrtbarron: and operators/admins (principal consumers of logs) didn't cause any errors..18:17
vponomaryovgouthamr: have you had any plans about updating tempest with your change of access rules -> https://review.openstack.org/#/c/369668/ ?18:19
vponomaryovgouthamr: at least simple change like checking presence of new attr of an access rule18:20
gouthamrvponomaryov: sure, i meant to do it here: https://review.openstack.org/#/c/390380/18:20
gouthamrvponomaryov: thanks for reminding me. i'll update ^18:21
tbarrongouthamr: I think these changes ensure that what you say *should* be true is in fact true.  I assert that in the current codebase what you assert *should* be true *tends* to be true, i.e. there are exceptions, and18:22
tbarronthe unnecessary proxying allows them to exist.18:22
vponomaryovtbarron: it is true that there is too many trues in your comment ))18:23
tbarronexceptions to the rule, they don't generate python exceptions :D18:23
gouthamrtbarron: hmmm, but you'd still need to refer to the share in the API18:23
tbarronvponomaryov: true that, verily18:23
tbarrongouthamr: well, let's see if tempest tells me I broke anything.  I haven't run it on these changes yet.18:23
openstackgerritAlyson proposed openstack/python-manilaclient: Add mountable snapshots support to manila client  https://review.openstack.org/34562518:23
* gouthamr esker and tbarron, man18:23
tbarrongouthamr: you really know how to insult esker18:24
gouthamrhe ain't here ;)18:24
tbarronno fun then18:24
*** lpetrut has joined #openstack-manila18:44
openstackgerritOpenStack Proposal Bot proposed openstack/manila: Updated from global requirements  https://review.openstack.org/42084018:46
openstackgerritTom Barron proposed openstack/manila-image-elements: Suppress some bashate warnings  https://review.openstack.org/42091119:05
openstackgerritMerged openstack/python-manilaclient: Remove Nova Net option for share networks  https://review.openstack.org/41623919:11
*** yumiriam has quit IRC19:18
bswartzgouthamr tbarron: he isn't here but this channel is logged and the archives are available publicly forever19:19
tbarronbswartz: true that, and true that I can be more verbose even sometimes19:20
tbarronbut my vocabulary isn't as rich.19:21
* gouthamr pictures esker googling for his irc nick and going o.O and mailing in my pink slip 19:21
*** jcsp has joined #openstack-manila19:35
openstackgerritRodrigo Barbieri proposed openstack/manila: Share Migration Ocata Improvements  https://review.openstack.org/40630519:43
*** jcsp has quit IRC20:09
openstackgerritGoutham Pacha Ravi proposed openstack/manila: NetApp cDOT: Add Intra-vserver migration support  https://review.openstack.org/41378320:19
*** lseki has quit IRC20:20
openstackgerritGoutham Pacha Ravi proposed openstack/manila: NetApp cDOT: Add Intra-vserver migration support  https://review.openstack.org/41378320:21
*** jcsp has joined #openstack-manila20:24
*** lpetrut has quit IRC20:27
*** alyson_ has quit IRC21:07
*** JoseMello has quit IRC21:15
*** jprovazn has quit IRC21:15
*** jcsp has quit IRC21:29
*** dustins has quit IRC21:31
*** jcsp has joined #openstack-manila21:41
*** absubram_ has joined #openstack-manila21:46
openstackgerritMerged openstack/manila: Update .gitignore  https://review.openstack.org/42070721:51
openstackgerritMerged openstack/manila: Some share api test cleanup  https://review.openstack.org/42088621:58
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Fix license and E265 errors in doc/source/conf.py  https://review.openstack.org/41329821:59
openstackgerritMerged openstack/manila: Updated from global requirements  https://review.openstack.org/42084021:59
*** absubram_ has quit IRC22:00
openstackgerritRodrigo Barbieri proposed openstack/manila: Share Migration Ocata Improvements  https://review.openstack.org/40630522:06
*** eharney has quit IRC22:08
openstackgerritRodrigo Barbieri proposed openstack/manila: Add mountable snapshots support  https://review.openstack.org/34552622:36
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Refactor Access Rules APIs  https://review.openstack.org/36966822:37
openstackgerritYogesh proposed openstack/manila: Share Migration Ocata Improvements  https://review.openstack.org/40630522:38
openstackgerritYogesh proposed openstack/manila: Improve test coverage for share migration  https://review.openstack.org/41855922:38
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Share Migration Ocata Improvements  https://review.openstack.org/40630522:44
openstackgerritRodrigo Barbieri proposed openstack/manila: Share Migration Ocata Improvements  https://review.openstack.org/40630523:03
openstackgerritRodrigo Barbieri proposed openstack/manila: Add mountable snapshots support  https://review.openstack.org/34552623:17
*** xyang1 has quit IRC23:34
openstackgerritMerged openstack/manila: Fix license and E265 errors in doc/source/conf.py  https://review.openstack.org/41329823:47

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